Bug when debugging jupyter notebook cell

Environment data
VSCode details
Version: 1.64.2
Commit: f80445acd5a3dadef24aa209168452a3d97cc326
Date: 2022-02-09T22:00:58.347Z (1 wk ago)
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 19.6.0

Jupyter Extension version (available under the Extensions sidebar): v2022.1.1301854968
Python Extension version (available under the Extensions sidebar): v2022.0.1814523869
OS (Windows | Mac | Linux distro) and version: MacOS Catalina 10.15.7 (19H15)
Python and/or Anaconda version: 3.8.11 and 4.11.0
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local

Expected behaviour
Bug free debugging
Actual behaviour
Debugging jupyter notebook cell was working like a charm until latest update. I have updated today and I am getting error when I try to debug in notebook.

Steps to reproduce:

Set up a break point in the function getting called from the notebook.
Try to go into the debugging mode in notebook by hitting the debug button.

launch.json is already set and nothing unusual there. Below are the contents:

{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python: Current File",
            "type": "python",
            "request": "launch",
            "program": "${file}",
            "console": "integratedTerminal",
            "editor.formatOnSave": true,
            "python.formatting.provider": "black",
            "args": ["FP_HK", "30-90", "", "", ""],
        }
    ]
}

If I cancel on error prompt then I get another error about child process.

I tried but failed to reproduce this issue in smaller code sample. Closing this issue due to lack of reproducibility for time being.


Fix race-condition in PowerRename progress UI

Summary of the Pull Request
What is this about:
There was a race condition in the PowerRename progress UI that meant that sometimes the progress dialog would continue running for the lifetime of the PowerRename UI. This could lead to a crash when the CPowerRenameProgressUI was deallocated as the progress box would continue to call methods on the deallocated object.
What is include in the PR:
A fix that prevents the crash from occurring.
The issue stems from Start() and Stop() being called in quick succession here:
https://github.com/microsoft/PowerToys/blob/v0.43.0/src/modules/powerrename/ui/PowerRenameUI.cpp#L402-L404
The Start() method starts a thread that when it has initialised goes on to set a pointer to the progress dialog in the m_sppd field on the CPowerRenameProgressUI object.
The Stop() method checks to see if the m_sppd field is truthy, and if it is, stops the dialog and then attempts to stop the thread.
There is a race condition here if the Stop() method runs before the thread has started and initialised the m_sppd field on the CPowerRenameProgressUI object.
The solution is to add in synchronisation to ensure the thread has properly started before Stop() attempts to clear up.
How does someone test / validate:
Run the PowerRename tool and find that it no longer crashes.
Quality Checklist

[x] Linked issue: #12391
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Thank you for the contribution!


Pallet Assets: Create new asset classes from genesis config

This change allows chain developers to create new asset classes via genesis config. This is useful for asset classes controlled by the by the root account of a parachain.
Currently the assets pallet requires that stakeholders create a new asset class via a dispatchable (presumably invoked via an extrinsic). This limits the utility of the assets pallets. Our parachain would like to create new asset classes via a loosely coupled trait,  and/or via genesis config. This PR addresses the latter.
^ @shawntabrizi
If you think this is the right approach, I can go ahead and create companion PRs for Statemint, etc in the Cumulus repo

This seems fine, but you dont want to give any genesis balances?
I would expect a vec of Vec<(AccountId, Balance)> for giving users balances at the start.

I agree, I'll add that too. Also for asset metadata.

@vgeddes please merge master.

/tip small

Awesome, thanks for the tip! I've merged in master and pasted my address in the PR description, as well as below:
Polkadot address: 13qBjpFmDbf5vyumEcqpUji5V2iVzsZb6oN7d1qH7i2ZRHep

/tip small

bot merge


Receiving error when returning a 404 page w/ JSON result

I have a response of

{"error":{"message":"Resource Not Found","status_code":404}}

which produces the following error

Time: 5.05 seconds, Memory: 20.75Mb
There was 1 error:

BatterTest::it_should_show_return_error_on_bad_request
InvalidArgumentException: The current node list is empty.

/home/vagrant/code/rangladex5/vendor/symfony/dom-crawler/Symfony/Component/DomCrawler/Crawler.php:580
/home/vagrant/code/rangladex5/vendor/laracasts/integrated/src/Extensions/Laravel.php:152
/home/vagrant/code/rangladex5/vendor/laracasts/integrated/src/Extensions/IntegrationTrait.php:365
/home/vagrant/code/rangladex5/vendor/laracasts/integrated/src/Extensions/Laravel.php:65
/home/vagrant/code/rangladex5/vendor/laracasts/integrated/src/Extensions/IntegrationTrait.php:74
/home/vagrant/code/rangladex5/tests/functional/battertest.php:38

I need more information here.
Show me your test method, and the route that returns that response.

@JeffreyWay
Here is the test
/** @test */
public function it_should_show_return_error_on_bad_request()
{
    $this->visit("batters/55555")->dump();
         // ->see('Resource Not Found');
}

Simple Route Resource
Route::resource('test','TestController');
and the controller 'show' method
public function show($id)
{
	$batter = Batter::find($id);
	return ( ! $batter ) ? $this->respondNotFound() : $this->fractal->item($batter, $this->transformer);
}

protected function respondNotFound($statusCode = 404, $message = 'Resource Not Found')
{
	return Response::json([
	        'error' => [
	            'message' => $message,
	            'status_code' => $statusCode
	        ]
	    ], $statusCode);
}

The visit method is to simulate visiting a page in the browser. So, as part of that call, it runs an assertion to ensure that a 200 response is returned.
For an API, it's best to use $this->call('GET', 'json/end-point') - assuming you're using the Laravel extension.
I'll have a think on this. Maybe we can add something to simplify/clarify this, because I agree that it could be confusing.

Ah, I see what you are saying.  My test is (properly) failing because the response code is not 200.
That makes good sense and selenium test might not be the best tool to perform an API test (I have used integrated Laravel already to test all end points)
Regarding other topic of possible API, I am thinking
$this->send(method, uri)->isJSON()->shouldReturn(404)

Or you can have separate methods for each type of request
$this->get(uri)
$this->post(uri, data)
$this->put()
$this->patch()
$this->delete()
$this->isJSON() // would be nice
And, after thinking this through, really makes it clear using selenium to test API is wrong solution.
Having said that, the above routines would sure be nice for testing APIs :)

Okay - I've pushed a first draft to dev-master.
https://github.com/laracasts/Integrated/commit/5dcc7b5df56e61c1902ef8702c2edd59cf447f27
Use like so:
 /** @test */
 public function it_makes_api_requests()
 {
     $this->get('/json-endpoint')
          ->seeStatusCode(200)
          ->seeJsonEquals(['name' => 'joe', 'age' => 30])
          ->seeJsonContains(['name' => 'joe']);
 }

Perfect!  I will have a look in the morning...
Go to sleep :)

How about a seeContentType() so we can ensure that we're getting application/json or application/atom+xml for feeds?

@JeffreyWay Thanks for adding the above methods!  All is working great!  Now just need to figure out why Selenium is not hitting URL .

@danwall - Yeah, we could add that.


Use inspect API for DataPlane Viz summary panel policies

Description
Awaiting https://github.com/kumahq/kuma/issues/11293 to be able to map clusterNames to toResourceRules

We decided to parse the clustername on the frontend into name/zone/namespace and then use that to find the correct resource rules, so this is no longer waiting on backend


Convert indicator.lop_target field to Decimal

Convert indicator.lop_target field to Decimal.  First use the convert_lop_targets_numeric --execute to clean the database, then run migrations.

Not quite ready yet.


Auto-assume service account in installation repo

Summary
If the owner's email of the project is not the current user, impersonate that user and update go client.

Labels

Test Plan

Checklist

[ ] If required, I have updated the Plural documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have added a meaningful title and summary to convey the impact of this PR to a user.
[ ] I have added relevant labels to this PR to help with categorization for release notes.

I think if when we assume a service account, a global variable within the pkg/config package is set and any future config.Read() function returns it if non-null it might guarantee all code paths are auto-wired with the service account.  Or at least that's how i guessed an elegant solve might work


Sprockets::FileNotFound error

I'm running bundle bundle exec middleman build and continually get this error using middleman (3.3.7)
/Users/Ackerman/.rvm/gems/ruby-2.0.0-p353@global/gems/sprockets-2.12.3/lib/sprockets/base.rb:153:in `resolve': couldn't find file 'Amount<EMAIL_ADDRESS>(Sprockets::FileNotFound)

I've tried everything possible, the image is definitely there as well.  I'm referencing it in a SCSS file using background-image<EMAIL_ADDRESS>could it be a bug?

what is Amount<EMAIL_ADDRESS>Seems like you may have some kind of parse error which is adding extra content to the name of the file being passed to middleman?

On a fresh init of middleman, I am getting the following error, which is possibly related.

/Users/xxx/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/sprockets-2.12.3/lib/sprockets/base.rb:153:in `resolve': couldn't find file '/Users/xxx/Documents/Projects/new-middleman-test/source/images/background.png' (Sprockets::FileNotFound)

Ruby 2.2.0,
Gems included by the bundle:
  * activesupport (4.1.9)
  * addressable (2.3.7)
  * bundler (1.7.12)
  * celluloid (0.16.0)
  * chunky_png (1.3.3)
  * coffee-script (2.3.0)
  * coffee-script-source (1.9.0)
  * compass (1.0.3)
  * compass-core (1.0.3)
  * compass-import-once (1.0.5)
  * em-websocket (0.5.1)
  * erubis (2.7.0)
  * eventmachine (1.0.6)
  * execjs (2.3.0)
  * ffi (1.9.6)
  * haml (4.0.6)
  * hike (1.2.3)
  * hitimes (1.2.2)
  * hooks (0.4.0)
  * http_parser.rb (0.6.0)
  * i18n (0.6.11)
  * json (1.8.2)
  * kramdown (1.5.0)
  * listen (2.8.5)
  * middleman (3.3.7)
  * middleman-blog (3.5.3)
  * middleman-core (3.3.7)
  * middleman-livereload (3.1.1)
  * middleman-sprockets (3.4.1)
  * minitest (5.5.1)
  * multi_json (1.10.1)
  * padrino-helpers (0.12.4)
  * padrino-support (0.12.4)
  * rack (1.6.0)
  * rack-livereload (0.3.15)
  * rack-test (0.6.3)
  * rb-fsevent (0.9.4)
  * rb-inotify (0.9.5)
  * sass (3.4.11)
  * sprockets (2.12.3)
  * sprockets-helpers (1.1.0)
  * sprockets-sass (1.3.1)
  * thor (0.19.1)
  * thread_safe (0.3.4)
  * tilt (1.4.1)
  * timers (4.0.1)
  * tzinfo (1.2.2)
  * uber (0.0.13)
  * uglifier (2.7.0)

Does the file /Users/xxx/Documents/Projects/new-middleman-test/source/images/background.png exist?

Yes - it is created by the standard middleman init command (running middleman init new-middleman-test)

Yes, my file exists as well

I figured out the problem for me.  When I moved my directory to /Users/Ackerman/Downloads/html it worked fine.  It seems the problem is sprockets-2.12.3/lib/sprockets/base.rb:153 couldn't determine the path when there was parentheses and spaces in it.

Nice catch @bcackerman. Sounds like maybe an upstream Sprockets issue, we'll take a look

Definitely a good catch @bcackerman. Upgraded today and started having the same issue. Renamed a folder with parentheses in the name and Sprockets started working again.

I'm going to try and figure out if this is Sprockets' bug or ours

I'm getting the same error.
mo~/Desktop/www (master):middleman server
WARN: Unresolved specs during Gem::Specification.reset:
      rack (< 2.0, >= 1.4.5)
      uber (~> 0.0.4)
      rack-test (~> 0.6.2)
      thor (< 2.0, >= 0.15.2)
      activesupport (~> 4.1.0)
      padrino-helpers (~> 0.12.3)
      listen (< 3.0, >= 2.7.9)
      i18n (~> 0.6.9)
WARN: Clearing out unresolved specs.
Please report a bug if this causes problems.
== The Middleman is loading
/Library/Ruby/Gems/2.0.0/gems/sprockets-2.12.3/lib/sprockets/base.rb:153:in `resolve': couldn't find file 'uri.js/src/URI.min.js' (Sprockets::FileNotFound)
	from /Library/Ruby/Gems/2.0.0/gems/middleman-sprockets-3.3.10/lib/middleman-sprockets/imported_asset.rb:29:in `resolve_path_with'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-sprockets-3.3.10/lib/middleman-sprockets/extension.rb:79:in `block in manipulate_resource_list'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-sprockets-3.3.10/lib/middleman-sprockets/extension.rb:78:in `each'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-sprockets-3.3.10/lib/middleman-sprockets/extension.rb:78:in `manipulate_resource_list'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/sitemap/store.rb:215:in `block (2 levels) in ensure_resource_list_updated!'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/sitemap/store.rb:214:in `each'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/sitemap/store.rb:214:in `reduce'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/sitemap/store.rb:214:in `block in ensure_resource_list_updated!'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/monitor.rb:211:in `mon_synchronize'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/sitemap/store.rb:208:in `ensure_resource_list_updated!'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/sitemap/extensions/on_disk.rb:31:in `block in initialize'
	from /Library/Ruby/Gems/2.0.0/gems/uber-0.0.13/lib/uber/options.rb:80:in `instance_exec'
	from /Library/Ruby/Gems/2.0.0/gems/uber-0.0.13/lib/uber/options.rb:80:in `proc!'
	from /Library/Ruby/Gems/2.0.0/gems/uber-0.0.13/lib/uber/options.rb:69:in `evaluate_for'
	from /Library/Ruby/Gems/2.0.0/gems/uber-0.0.13/lib/uber/options.rb:60:in `evaluate'
	from /Library/Ruby/Gems/2.0.0/gems/hooks-0.4.0/lib/hooks/hook.rb:53:in `execute_callback'
	from /Library/Ruby/Gems/2.0.0/gems/hooks-0.4.0/lib/hooks/hook.rb:40:in `block in run'
	from /Library/Ruby/Gems/2.0.0/gems/hooks-0.4.0/lib/hooks/hook.rb:39:in `each'
	from /Library/Ruby/Gems/2.0.0/gems/hooks-0.4.0/lib/hooks/hook.rb:39:in `inject'
	from /Library/Ruby/Gems/2.0.0/gems/hooks-0.4.0/lib/hooks/hook.rb:39:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/hooks-0.4.0/lib/hooks.rb:55:in `run_hook_for'
	from /Library/Ruby/Gems/2.0.0/gems/hooks-0.4.0/lib/hooks/instance_hooks.rb:6:in `run_hook'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/core_extensions/request.rb:50:in `inst'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/preview_server.rb:103:in `new_app'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/preview_server.rb:21:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/cli/server.rb:78:in `server'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/cli.rb:72:in `method_missing'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/command.rb:29:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/command.rb:126:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/lib/middleman-core/cli.rb:20:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/middleman-core-3.3.7/bin/middleman:18:in `<top (required)>'
	from /usr/bin/middleman:23:in `load'
	from /usr/bin/middleman:23:in `<main>'

@mofirouz See https://github.com/sstephenson/sprockets/issues/706

@Arcovion  Thanks! That helped :)

Okay, closing this in favor of that.

Just checking if the correct bug has been identified – I’ve stumbled on the same problem today, after upgrading a project from MM 3.3.5 to 3.3.10. My problematic path was /Prace_WWW/D+K/www-website/source/site_images/404.jpg (notice the plus sign in the folder name). After removing plus sign all is working again, both development server and build.

@komor72 Issue looks to be an issue with the Sprockets gem. https://github.com/sstephenson/sprockets/issues/706
I upgraded to MM 3.3.10 and started getting the following sprockets error similar to @bcackerman:
== The Middleman is loading
/Users/bermonpainter/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/sprockets-2.12.3/lib/sprockets/base.rb:153:in `resolve': couldn't find file '/Users/bermonpainter/Dropbox (Personal)/Sites/blendconf.com/blendconf-site/source/assets/images/content/blend-conf-icon.png' (Sprockets::FileNotFound)

I tried to update sprockets 3.0.0.beta.8 based on https://github.com/sstephenson/sprockets/issues/706 but got:
You have requested:
  sprockets ~> 3.0.0.beta.8

The bundle currently has sprockets locked at 2.12.3.
Try running `bundle update sprockets`

I ran  bundle update sprockets and bundle update middleman-sprockets but no luck and ended up with the terminal spitting out:
Bundler could not find compatible versions for gem "sprockets":
  In Gemfile:
    middleman-sprockets (~> 3.1.2) ruby depends on
      sprockets (~> 2.1) ruby

    sprockets (3.0.0.beta.8)

I even tried to manuall update the sprockets version in my gemfile.lock to 3.0.0.beta.8 but still no luck.
== The Middleman is loading
== Activating: sprockets
== Reading:  Local config
== Activating: livereload
== Activating: relative_assets
== Activating: directory_indexes
/Users/bermonpainter/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/sprockets-3.0.0.beta.8/lib/sprockets/ejs_processor.rb:1:in `require': cannot load such file -- ejs (LoadError)
	from /Users/bermonpainter/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/sprockets-3.0.0.beta.8/lib/sprockets/ejs_processor.rb:1:in `<top (required)>'
	from /Users/bermonpainter/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/sprockets-3.0.0.beta.8/lib/sprockets/ejs_template.rb:1:in `require'
	from /Users/bermonpainter/.rbenv/versions/2.1.3/lib/ruby/gems/2.1.0/gems/sprockets-3.0.0.beta.8/lib/sprockets/ejs_template.rb:1:in `<top (required)>'

Because 3.0 is a major version change, we'd have to update middleman-sprockets to allow that. A PR against middleman-sprockets to bump the dependency to the 3.0 betas will test if the major change breaks our support

@tdreyno Makes sense. I can take a swing at a PR if you want.

Would love that. Thanks @bpainter

/home/leonardo/.rvm/gems/ruby-2.1.0/gems/sprockets-2.12.4/lib/sprockets/base.rb:153:in `resolve': couldn't find file '/home/leonardo/psdhtml (3)/source/images/background.png' (Sprockets::FileNotFound)
This problem took me a good time :(. The solution was simple, rename the folder psdtohtml (3) for psdtohtml. (No space)

so what about this error on a fresh middleman installation and no spaces or parenthesis or extension in any folder ?
C:/Users/mirko/www/paridedesign/vendor/bundle/ruby/2.2.0/gems/sprockets-2.12.4/l
ib/sprockets/base.rb:153:in `resolve': couldn't find file 'C:\Users\mirko\www\paridedesign\source\images\backgroundoverlay.jpg' (Sprockets::FileNotFound)

Please open new tickets for new issues.


dartsim: To not call dart::collision::GzOdeCollisionDetector::create() in ConstructEmptyWorld

🦟 Bug fix
Fixes problem described in https://github.com/gazebosim/gz-sim/issues/18#issuecomment-2301643365 .
Summary

Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Unfortunately, this does not work as it is possible that one never calls the SetWorldCollisionDetector method.


Fix PostgreSQL Jdbc URL parsing exception

Fix #11552

[x] Add a unit test to verify that the fix works.
[x] Explain briefly why the bug exists and how to fix it.

Fix multi hosts JDBC url parsing exception

[x] Add a unit test to verify that the fix works.
[x] Explain briefly why the bug exists and how to fix it.

Fix https://github.com/apache/skywalking/issues/11552

[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #11552.
[x] Update the CHANGES log.

I am not sure why, but the UTs on the Windows keep failing.

I am not sure why, but the UTs on the Windows keep failing.

all the new UTs added in URLParserTest?

I am not sure why, but the UTs on the Windows keep failing.

all the new UTs added in URLParserTest?

Don't worry about this. They are passed now.


[HOLD for payment 2024-12-16] [$250] Violation tab briefly blinks when open for the first time

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.68-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Issue was found when executing this PR: https://github.com/Expensify/App/pull/53095
Issue reported by: Applause Internal Team
Action Performed:
Precondition: Enable debug mode in Settings > Troubleshoot (on Hybrid app tap with 4 fingers on display). Expense is submitted in the chat.

Go to chat > tap on expense
Tap on expense details page header
Tap on Debug > View transaction
Tap on Violations tab > Create
Tap Save
Tap on created violation

Expected Result:
Debug - Transaction Violation page opens without blink.
Actual Result:
Debug - Transaction Violation page briefly blinks for the first opening.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/ae294460-887d-4baa-b4a6-e66989e018a5

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @twisterdotcom
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021863571590316301120
Upwork Job ID:<PHONE_NUMBER>316301120
Last Price Increase: 2024-12-02
Automatic offers:

paultsimura | Reviewer | 105164657
daledah | Contributor | 105164658

Payment Summary:

@paultsimura paid $250 here (Reviewer)
@daledah paid $250 here (Contributor)


process output of pose_landmark_lite

Please make sure that this is a solution issue.
System information (Please provide as much relevant information as possible)

Have I written custom code (as opposed to using a stock example script provided in Mediapipe): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04, Android 11, iOS 14.4): Windows 10
MediaPipe version: v<IP_ADDRESS>
Bazel version: 5.2.0
Solution (e.g. FaceMesh, Pose, Holistic): Pose
Programming Language and version ( e.g. C++, Python, Java): Python3

Describe the expected behavior:
Since mediapipe cannot use gpu for acceleration under windows, I want to use onnxruntime for acceleration.
I convert tflite to onnx, and make sure the conversion is successful.
pose_landmark = onnxruntime.InferenceSession(
    "pose_landmark_lite.onnx",
    providers=["CUDAExecutionProvider", "CPUExecutionProvider"],
)
capture = cv2.VideoCapture("D:/video/mother.mp4")

if capture.isOpened():
    hasFrame, frame = capture.read()
else:
    hasFrame = False

while hasFrame:
    # img1: 256x256
    # img2: 128x128
    img1, img2, scale, pad = resize_pad(frame)
    input = torch.from_numpy(img1.astype(np.float32))
    input = input.unsqueeze(0)
    (
        pose_landmarks,
        pose_world_landmarks,
        segmentation_mask,
        pose_landmarks_roi,
        pose_detection,
    ) = pose_landmark.run(
        None, {pose_landmark.get_inputs()[0].name: input.cpu().numpy()}
    )
    print(
        pose_landmarks.shape,  # (1, 195)
        pose_world_landmarks.shape,  # (1, 1)
        segmentation_mask.shape,  # (1, 256, 256, 1)
        pose_landmarks_roi.shape,  # (1, 64, 64, 39)
        pose_detection.shape,  # (1, 117)
    )
    hasFrame, frame = capture.read()
    key = cv2.waitKey(1)
    if key == 27:
        break

As shown in the comments of the above code, the shape of pose_landmarks is (1, 195).
When using mp.pose, the output of the pose_landmarks is similar to
landmark {
  x: 0.3057445287704468
  y: 0.9131049513816833
  z: -0.014605664648115635
  visibility: 0.9162100553512573
  presence: 0.9988192915916443
}

...

landmark {
  x: 0.3057445287704468
  y: 0.9131049513816833
  z: -0.014605664648115635
  visibility: 0.9162100553512573
  presence: 0.9988192915916443
}

They are 32 landmark as described in document.
What is the processing between these two outputs ? I didn't find the relevant python code.
Standalone code you may have used to try to get what you need :
If there is a problem, provide a reproducible test case that is the bare minimum necessary to generate the problem. If possible, please share a link to Colab/repo link /any notebook:
Other info / Complete Logs :
Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached:

Hi @liuyulvv, Could you please elaborate your query with complete details. Thank you!

@kuaashish How to convert pose_landmarks(one of the outputs of pose_landmark_lite, a tensor with shape of (1,195)), into
  x: 0.3057445287704468
  y: 0.9131049513816833
  z: -0.014605664648115635
  visibility: 0.9162100553512573
  presence: 0.9988192915916443

What is the relationship between 195 numbers and 33 landmarks' x, y, z, visibility, and presence ? Thanks a lot.

@liuyulvv   I convert tflite to onnx aways fails.Can you give me a model of onnx?

@ted1986122 https://github.com/liuyulvv/mediapipe/tree/main/mediapipe/modules/pose_landmark

Hi liuyulvv,
Did you figure out how to get the landmarks from (1,195)) tensor?
Thanks

@kuaashish How to convert (one of the outputs of pose_landmark_lite, a tensor with shape of (1,195)), intopose_landmarks
  x: 0.3057445287704468
  y: 0.9131049513816833
  z: -0.014605664648115635
  visibility: 0.9162100553512573
  presence: 0.9988192915916443

What is the relationship between and landmarks' ? Thanks a lot.195 numbers``33``x, y, z, visibility, and presence
Have you solved this problem yet? Can you explain how this result was converted if it is resolved?

如何将（形状为 （1，195）的张量 pose_landmark_lite 的输出之一）转换为pose_landmarks
  x: 0.3057445287704468
  y: 0.9131049513816833
  z: -0.014605664648115635
  visibility: 0.9162100553512573
  presence: 0.9988192915916443

和地标之间有什么关系？多谢。195 numbers``33``x, y, z, visibility, and presence

Have you solved this problem yet? Can you explain how this result was converted if it is resolved?

@HzHzHzHzHz @vigneshrk29
landmark_list  = [$x_1$ ,$ y_1$, $ z_1$, $visibility_1$, $presence_1$, $x_2$, ... $presence_2$, ......]
values are stored continuously


Validation doesn't seem to recurse into bundles properly

See ignored test in ca.uhn.fhir.validation.BundleValidationTest

Note that two more ignored tests have been added to ResponseValidatingInterceptorTest.java. Enable these when this is fixed.


Support for annotating tags with a message

There is now a tagMessage property in the tag and tagAndPush tasks that can be set to a closure returning a string.
The tag task exposes a fromSystemProperty(String) and fromProjectProperty(String) closure that can be used to specify a system or project property to use when determining the tag message.
Added tests.

@Vampire since you rebased off this, will just be merging yours.

I think this is a major shortcoming of jacoco in that those useless methods
from implicit closures end up showing up.
I should have taken a close look though, so no excuses.
On Feb 2, 2017 7:07 PM, "Björn Kautler"<EMAIL_ADDRESS>wrote:
@Vampire commented on this pull request.
In build.gradle
https://github.com/vivin/gradle-semantic-build-versioning/pull/61#pullrequestreview-19937231
:

@@ -75,6 +75,10 @@ dependencies {
}

jacocoTestReport {

doFirst {

   classDirectories = classDirectories.collect { fileTree(it) {

exclude '**/$_closure*' } }.sum()

}

Do not do this.
Or at least tell me how this brings "better coverage metrics."
It does not at all bring better metrics.
The only thing you achieve with this is the same you achieved the first
time you added it before I first contributed and removed it, it is
self-deception, nothing else.
You might think at first sight that the metrics report looks cleaner if
the closures are not listed as own classes, especially as they are also
included in the full classes. But this is non-sense and the same as with
anonymous classes. If you cut them out, from recording and reporting they
are simply not considered at all and thus you raise the coverage
percentages. But for real values you have to leave them in where they
belong.
I've also prepared some graphics that show you some of the differences,
upper half is this branch as you committed it, lower half the same with
only the excludes removed.
Overview:
Watch the numbers here. 300 vs. 432 branches, 302 vs. 361 Lines, 12 vs. 48
Classes, ...
[image: overview]
https://cloud.githubusercontent.com/assets/325196/22575927/060b8f1a-e9b9-11e6-8c5e-20d404c62b59.png
Package:
Of course the closures look a bit strange here and you might not be able to
easily identify them in the source, but I found no way to leave them out in
only this particular view. But it surely could be done with a custom CSS or
something like that if that is what bugs you most, but even then it would
not really be correct, as their values are not included in the enclosing
classes values, they are completely separate, besides that they share the
same source file.
[image: package]
https://cloud.githubusercontent.com/assets/325196/22575984/49b1c45a-e9b9-11e6-9cea-19bf46905727.png
File:
Here you see the difference best. As you can see in the upper version
almost nothing is even considered (white), in the lower version all code
lines are covered, as the closure which makes most of the file here is
included in the metrics and report.
[image: file]
https://cloud.githubusercontent.com/assets/325196/22576062/c6baf502-e9b9-11e6-9d2b-1cd1fc95fd42.png
If your problem was that you didn't get the 0 missed branches, lines and
classes, don't forget to set the project property disableGroovyOptimizations
when running tests for getting proper covering metrics, otherwise the
Groovy magic does create too much stuff that is non-obvious and not really
testable. That's also why it is enabled in the circleci test task, but not
in the deployment task. This means the sources will need to be recompiled
though they were compiled already for the tests, but you get proper
coverage metrics out.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/pull/61#pullrequestreview-19937231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADbHpMunopKHB82VBdaquNol5PgWGIUks5rYovPgaJpZM4LyS6O
.

They are effectively anonymous inner classes, so it is quite valid to have them listed separately. Also, JaCoCo of course is for measuring Java coverage, but even the I think anonymous classes are listed separately just like closures.

I was referring to the fact that jacoco doesn't provide a way to exclude
methods from coverage -- even in Java code. I think it's something that has
been a requested feature for quite some time.
On Fri, Feb 3, 2017 at 2:26 PM, Björn Kautler<EMAIL_ADDRESS>wrote:

They are effectively anonymous inner classes, so it is quite valid to have
them listed separately. Also, JaCoCo of course is for measuring Java
coverage, but even the I think anonymous classes are listed separately just
like closures.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/pull/61#issuecomment-277366452,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADbHlV0rHYI136j5ySA_1sYVHxE6pJuks5rY5t9gaJpZM4LyS6O
.

--
[vivin.net http://vivin.net :: github http://github.com/vivin ::
linkedin https://www.linkedin.com/in/vivin]

But why should you want to exclude methods? The only thing you reach with excluding something is to raise your coverage percentage while not really having that percentage covered. And I also don't know why we speak about methods, we were at anonymous classes and closures.

It's because, as you said earlier, it's for Java code coverage - so I've
noticed that for closures, it makes references to certain implicit methods
that aren't really found in the groovy code (e.g., the call and doCall
methods on closures).
On Sat, Feb 4, 2017 at 3:57 AM, Björn Kautler<EMAIL_ADDRESS>wrote:

But why should you want to exclude methods? The only thing you reach with
excluding something is to raise your coverage percentage while not really
having that percentage covered. And I also don't know why we speak about
methods, we were at anonymous classes and closures.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/pull/61#issuecomment-277435469,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADbHv0SbDSgUg14pwSXGDspsZyrbfZsks5rZFmvgaJpZM4LyS6O
.

--
[vivin.net http://vivin.net :: github http://github.com/vivin ::
linkedin https://www.linkedin.com/in/vivin]

Ah, I see what you mean, but even then it would be wrong to exclude those. Those are the methods where the code is stuffed in I think. So removing them from the coverage recording again falsifies the measures too much.

Right - I think we need the call/doCall, but some of the implicit methods
we don't. It is unfortunate that it doesn't have such granular exclusions.
On Mon, Feb 6, 2017 at 11:57 AM, Björn Kautler<EMAIL_ADDRESS>wrote:

Ah, I see what you mean, but even then it would be wrong to exclude those.
Those are the methods where the code is stuffed in I think. So removing
them from the coverage recording again falsifies the measures too much.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/pull/61#issuecomment-277777380,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADbHi8EhblkOWAdkfvRWY7EWTZHI-vVks5rZ20NgaJpZM4LyS6O
.

--
[vivin.net http://vivin.net :: github http://github.com/vivin ::
linkedin https://www.linkedin.com/in/vivin]

Well, it's not that important, is it? If you look at line and branch coverage, you get a pretty good number about the real project code if you disabled the optimizations. That's why I got to 0 missed on these metrics.

¯_(ツ)_/¯
Sometimes I get OCD and I want to make sure that we don't count anything
that is useless :) But as long as it has a minimal impact, I think it's
fine.
On Mon, Feb 6, 2017 at 12:58 PM, Björn Kautler<EMAIL_ADDRESS>wrote:

Well, it's not that important, is it? If you look at line and branch
coverage, you get a pretty good number about the real project code if you
disabled the optimizations. That's why I got to 0 missed on these metrics.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/vivin/gradle-semantic-build-versioning/pull/61#issuecomment-277794941,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADbHlrlNL2AiUVZ3t2tFFrpG4OlrftFks5rZ3tygaJpZM4LyS6O
.

--
[vivin.net http://vivin.net :: github http://github.com/vivin ::
linkedin https://www.linkedin.com/in/vivin]

Yeah, as long as you look at the lines and branches coverage, the impact should be 0 :-)

PR #66 supersedes this.


Use own images for fixtures

Hello!
Im new to Box2D and completed the iforce2D tutorial but I have one question. How do I use my own images and not these from the testbed. Is there any possibility to use a png/jpg for fixtures or do I have to use SFML and set the position/angle every timestep again?
I did not find anything about this topic which answered my question!
Thanks!

Box2d does not provide any graphics support. You have to run your own. Typically using glfw or sdl.

SFML is a nice library to start with.


fix(edge): fix error handling on  AMTDevicesDatatable

Closed for https://github.com/portainer/portainer/pull/6409


Serverside rendering support

Hi!
Please accept this PR to support rendering in vm on the server side.
The changes are minor - access to window and document is wrapped with typeof window !== undefined'.

This should have been resolved with the most recent release, as the fixes u referenced are included. Closing this, but please open a new issue if problems persist.


spi: SAM add spin lock around all tx/rx/txrx funcs

The fast tx/rx/txrx functions will leave the SPI bus in an inoperable state if interrupted, potentially spinning forever waiting on some data. Wrapping these operations in what amounts to a critical section using spin locks to avoid the issue.
I had this commit on a branch for awhile and was reminded by @yperess as he ran into trouble with faster data ready interrupts on the icm42688 with the tdk robokit
Signed-off-by: Tom Burdick<EMAIL_ADDRESS>

Thanks @nandojve yes it fixes it a bug for me

I tried cherry-picking this change ontop of my icm42688 interrupt commit (ToT on https://github.com/yperess/zephyr/tree/peress/sensor_shell) and I wasn't able to get things running past 12.5Hz still.

@yperess @teburd
FYI: I have an open PR (#51694) to use the SPI interrupts instead of blocking in loops

I tried cherry-picking this change ontop of my icm42688 interrupt commit (ToT on https://github.com/yperess/zephyr/tree/peress/sensor_shell) and I wasn't able to get things running past 12.5Hz still.

It's unlikely this is the only problem for that particular setup, but its one I definitely ran into.

@yperess @teburd FYI: I have an open PR (#51694) to use the SPI interrupts instead of blocking in loops

I could be wrong but I'm not sure interrupt driven transfers is helpful here... my math could be very wrong, please correct me if so! sam e70 has a 2 byte transmit data register... at 20 MHz its going to be empty in 800ns. That's not very much time to poll, and an interrupt triggering context switching is going to probably be a lot more costly.

Hi @yperess ,
Thank you for the input. I was wondering if you could give us a more context about data sizes and what is the frequency you selected.

Hi @yperess ,
Thank you for the input. I was wondering if you could give us a more context about data sizes and what is the frequency you selected.

Sure thing, I'll be happy to. I'm using the ICM42688 IMU on the TDK Robokit1 board. What I've done (PR will go up for review this week, I'm just not in a rush since we're still in feature freeze) is set up a trigger on SENSOR_CHAN_ALL and SENSOR_TRIG_DATA_READY. The handler will take samples for 5s at a time and print the average per channel. The sensor supports the following data which are sampled for each iteration:

SENSOR_CHAN_ACCEL_X
SENSOR_CHAN_ACCEL_Y
SENSOR_CHAN_ACCEL_Z
SENSOR_CHAN_GYRO_X
SENSOR_CHAN_GYRO_Y
SENSOR_CHAN_GYRO_Z
SENSOR_CHAN_DIE_TEMP

On each interrupt, the sensor driver does the following:

[ISR] Post a k_work to the system work queue
[Handler] Read the status register (1 byte)
[handler] Read data registers (14 bytes)
Add values to the accumulators
Conditional print and reset accumulators if 5s passed

Without this PR
Once the sensor ODR (output-data-rate) was set above 12.5Hz the application would deadlock
** With this PR**
ODR can be set as high as 32kHz without an issue, but I can only handle 8kHz of data (so 75% of the samples are dropped) due to latency in context switching and data format conversion. But, there is no deadlock and the application keeps running.

Just to inform, the XDMAC is a quite complex feature and can be used to do interesting things, see Atmel AT17417: Usage of XDMAC on SAM S/SAM E/SAM V. If you use a linked list all data can be transferred without CPU intervention.
I think it is possible add a feature on SPI that configure an interrupt to trigger a DMA linked list transfer, for instance. In this case, SPI driver reacts to that interrupt to enable the pre-programmed DMA transfer. Another interrupt could be configured to tell the consumer that the full transfer is done, allowing the consumer reschedule the post processing data. If you are luck, DMA can striding data during reception or even after using another DMA linked list operation on a MEM2MEM basis.
The other think that could be checked is to pre-program DMA to start a transfer using a PIOA line. I mean, instead process an interrupt a HW Trigger is made to enable a linked list transfer. The first item on the list is a PIOA line that will read a dummy data but the next linked list should be the SPI peripheral. This may mimics an event driver DMA transfer on the system which don`t require CPU. In this case there is a but, because I guess this works.

Just to inform, the XDMAC is a quite complex feature and can be used to do interesting things, see Atmel AT17417: Usage of XDMAC on SAM S/SAM E/SAM V. If you use a linked list all data can be transferred without CPU intervention.
I think it is possible add a feature on SPI that configure an interrupt to trigger a DMA linked list transfer, for instance. In this case, SPI driver reacts to that interrupt to enable the pre-programmed DMA transfer. Another interrupt could be configured to tell the consumer that the full transfer is done, allowing the consumer reschedule the post processing data. If you are luck, DMA can striding data during reception or even after using another DMA linked list operation on a MEM2MEM basis.
The other think that could be checked is to pre-program DMA to start a transfer using a PIOA line. I mean, instead process an interrupt a HW Trigger is made to enable a linked list transfer. The first item on the list is a PIOA line that will read a dummy data but the next linked list should be the SPI peripheral. This may mimics an event driver DMA transfer on the system which don`t require CPU. In this case there is a but, because I guess this works.

This sounds like a great hardware feature, I think  st has similar abilities to trigger dma transfers with various trigger sources.
I have some thoughts on this stuff for sure, and how it might be exposed generally as a friendlier api than the st/sam specific setups


postDate and expiryDate

postDate and expiryDate only seem to bring out the date. Is it possible to bring out the time as well?

I've managed to do a quick fix on this by editing the ExportService.php file line 364 and removing the (string) from the returned data variable. This returns the timestamp as is. Is there a way to format the ExpotModel::HandlePostDate with the php  date format function?


Bugfix/long wait units import

Enforce threshold_for_looping to be an integer so that the final wait time is also an integer.

I'm not a fan of this, in QUA we automatically convert 1e4 (which is a float) to an int and use it. I don't like it that qualang_tools behaves differently...
With that being said, I'm not completely opposed to it. So what is the reasoning here?

So the scope of this bugfix is limited to the new long_wait macro that I wrote. Despite my testing, when I tried to run it, the QUA compiler complained that my wait_time was not an integer.
This is despite me using unit.(coerce_to_integer=True).to_clock_cycles(1e6), which is unusual, so I decided to go further and write unit.(coerce_to_integer=True).to_clock_cycles(1_000_000), as well as throw an error if the threshold_for_looping parameter is a float.

I'm not a fan of this, in QUA we automatically convert 1e4 (which is a float) to an int and use it. I don't like it that qualang_tools behaves differently...
With that being said, I'm not completely opposed to it. So what is the reasoning here?

So the scope of this bugfix is limited to the new long_wait macro that I wrote. Despite my testing, when I tried to run it, the QUA compiler complained that my wait_time was not an integer.
This is despite me using unit.(coerce_to_integer=True).to_clock_cycles(1e6), which is unusual, so I decided to go further and write unit.(coerce_to_integer=True).to_clock_cycles(1_000_000), as well as throw an error if the threshold_for_looping parameter is a float.

Please write a test (put it in the test folder) that recreates this issue.
There might be a bug in the unit class, and there might also be a bug in the OPX.

There was indeed a bug with th u.to_clock_cycles function that doesn't cast 1e3//4 to an integer...
I create a separated PR to fix it: https://github.com/qua-platform/py-qua-tools/pull/214


Moving some more methods away from daemon.go

Missed some more in my previous pull-request :sweat_smile: 🐙.
daemon.go is still 957 lines long with pretty big methods but at least almost all methods there are for daemon only.
/cc @thaJeztah @runcom @LK4D4
🐸
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

@LK4D4 updated

LGTM

LGTM


ConsoleMessage type and text not interoperable between Chrome and Firefox

Steps to reproduce
Tell us about your environment:

Puppeteer version: 5.2.0
Platform / OS version: macOS 10.15.5
Node.js version: v10.16.3

What steps will reproduce the problem?
Please include code that reproduces the issue.
'use strict';

const puppeteer = require('puppeteer');

async function main() {
  for (const product of ['chrome', 'firefox']) {
    const browser = await puppeteer.launch({product});
    const page = await browser.newPage();
    await page.goto('about:blank');
    const msgPromise = new Promise((resolve) => {
      page.on('console', resolve);
    });
    await page.evaluate('console.log("message")');
    const msg = await msgPromise;
    console.log({
      product,
      type: msg.type(),
      text: msg.text(),
      args: msg.args(),
    });
    await browser.close();
  }
}

main();

Output from running the above is:
{ product: 'chrome',
  type: 'log',
  text: 'message',
  args:
   [ JSHandle {
       _disposed: false,
       _context: [ExecutionContext],
       _client: [CDPSession],
       _remoteObject: [Object] } ] }
{ product: 'firefox',
  type: 'verbose',
  text: [ 'message' ],
  args: [] }

What is the expected result?
The msg.type() and msg.text() should be the same for Chrome and Firefox, to make it possible to use console messages without switching on which product is under test.
What happens instead?

Chrome: msg.type() is "log" and msg.text() is "message".
Firefox: msg.type() is "verbose" and msg.text() is an array with a single item "message"

msg.args() is also different, but I don't know what it should be, and am not using it myself.

The location method also shows different return values. Firefox is missing columnNumber here. I added a PR to increase the test coverage around this area a bit.

@mjzffr this might be a FF issue? I'm not 100% - should we also consider Puppeteer itself translating Chrome/FF consoles into an interface that is consistent regardless of browser?

There are a number of issues here. tl;dr is that Firefox doesn't implement all the CDP events yet + there may be some discrepancies between Blink and Gecko.
In the Chrome case, the sample script is reporting data based on
{"method":"Runtime.consoleAPICalled","params":{"type":"log","args":[{"type":"string","value":"message"}],"executionContextId":3,"timestamp":1.596131012139123e+12,"stackTrace":{"callFrames":[{"functionName":"","scriptId":"4","url":"__puppeteer_evaluation_script__","lineNumber":0,"columnNumber":8}]}},"sessionId":"F24311E58CE491073C2839A39B2E8C83"}

Firefox doesn't yet implement Runtime.consoleAPICalled (https://bugzilla.mozilla.org/show_bug.cgi?id=1549526).
In the Firefox case, the sample script is getting its data from Log.entryAdded, which is only partly implemented. Indeed, no columnNumber is provided yet and the type of text is wrong. (I've filed https://bugzilla.mozilla.org/show_bug.cgi?id=1656314 about that).
{"sessionId":1,"method":"Log.entryAdded","params":{"entry":{"source":"javascript","level":"verbose","text":["message"],"url":"__puppeteer_evaluation_script__","lineNumber":1,"timestamp":1596131013412}}}

I'd have to do more digging to say more about type and args but we've previously noted (on log entries in general) Gecko has different types of logs that don’t conform with Chrome’s, so I suspect that clients like Puppeteer might have to do some translation and/or logging will be unified somehow in WebDriver bidi.

@foolip I implemented Runtime.consoleAPICalled a while ago and it landed in Firefox 85.
Would you mind having a look again if everything satisfies your needs?

Just checked and there is no difference anymore between Chrome and Firefox. Both output the following:
{
  product: 'test',
  type: 'log',
  text: 'message',
  args: [ JSHandle {} ]
}

Thanks @whimboo!


Change Detection - nzPageIndex parameter in nz-table

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-ivy-d9yhyx
Steps to reproduce
While the table page is in the 2nd index, I want to reload the table and go to the 1st step. However, even though I set the nzPageIndex parameter to 1, the 2nd index remains selected in the pagination.
What is expected?
What is expected is that the change detection works and sets the page as the 1st index.
What is actually happening?
Even though I set the nzPageIndex parameter to 1, the 2nd index remains selected in the pagination.

Environment
Info

ng-zorro-antd
12.1.1

Browser
Chrome 97

To solve this problem  add (nzPageIndexChange) (nzPageIndexChange) = "pageIndex = $event"
https://stackblitz.com/edit/ng-zorro-antd-ivy-gs89m7
because all NG-ZORRO's components are running under OnPush mode, this means any mutate to the @Input() data won't trigger change detection, please use immutable way to update array or object.

@souhailharrati Thanks

@merih-bb  you are welcome


kubefwd problem with mktemp suffix argument

Hey, sorry if this is an issue but it is by design.
When using the kubefwd extension on mac, it fails with an error about the --suffix argument.
I believe the version of mktemp on mac doesn't support this argument.
Maybe we can use a prefix instead?
I know the extensions says it requires coreutils to be installed, but homebrew by default prefixs everything with g, so the binary is actually gmktemp and replacing these things on OS by changing the path or something.
The workaround was to execute tilt with PATH=/usr/local/opt/coreutils/gnubin:$PATH. However, even after that the script is trying to run /Users/username/Library/Application because it is not escaping/wrapping in quotes the path which is supposed to be /Users/username/Libarary/Application Support/tilt-dev/tilt_modules/github.com/tilt-dev/tilt-extensions/kubefwd/run-kubefwd.sh, but this seems to be more a problem on how local_resource serve_cmd is setup.
Maybe we can make it more accessible somehow? I'm happy to submit a PR if that is what we are aiming for.
Thanks.

@nicks @bamorim I'm facing the same issue regarding the space escaping.
I've tried to add a basklash in the path:
RUN_KUBEFWD=$(echo  "$RUN_KUBEFWD" | sed 's/ /\\ /g')
DIR=$(echo  "$DIR" | sed 's/ /\\ /g')

but then get a :
+ /usr/bin/osascript -e 'do shell script "/Users/eric/Library/Application\ Support/tilt-dev/tilt_modules/github.com/tilt-dev/tilt-extensions/kubefwd/run-kubefwd.sh /opt/homebrew/bin/kubefwd /Users/eric/.kube/config /opt/homebrew/bin/entr /var/folders/xh/003_hq_j3qq29mtv3nn8kw6c0000gn/T//tilt-kubefwd.nXOFvvLqd" with administrator privileges'
./sudo-kubefwd.sh: line 43: /Users/eric/Library/Application\ Support/tilt-dev/tilt_modules/github.com/tilt-dev/tilt-extensions/kubefwd/watch-namespaces.sh: No such file or directory
49:50: syntax error: Expected “"” but found unknown token. (-2741)


Local exec tmp

let's try merging shadow_working changes with the existing hpc branch of distiller to bring updated pairsamtools to distiller

I'll try git rebase instead!


Support for Intellij 14.1

plugin is no longer compatible with Intellij v14.1

+1

I gonna look into it this evening. Maybe it's just upgrading the manifest, it will depend if the intellij-scala plugin made a binary incompatible scala upgrade.

0.3.1 hit the road: https://plugins.jetbrains.com/plugin/7460?pr=idea
Feel free to close this issue if it works for you - due to lack of time and patience to setup the Idea SDK on this laptop, this is rather a blind, untested release.

Great, working again now, thanks


Edit documentation

Add sparc64-unknown-linux-gnu and x86_64-sun-solaris.
Remove aarch64-unknown-linux-musl.

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit cf70168 has been approved by alexcrichton

:hourglass: Testing commit cf70168621092d8b127f3f0630e6ea26ff07629d with merge 650d58ea4fdbe754a8275f98555d6e8f57a7446f...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 650d58ea4fdbe754a8275f98555d6e8f57a7446f to master...


Sequential UI commands in Example App

Make the handler of the UI messages use the same mutex as the existing
chrome.certificateProvider message handler, so that they never run
concurrently. This corresponds to the typical single-threaded model of
real-world smart card middleware applications.
Also flip the default in the Example App to make the
chrome.certificateProvider message handler sequential (it was previously
controlled by a boolean flag set to false by default).
Implementation-wise, a cleaner approach than to use mutexes would be to
have a dedicated worker thread that would be used for running all
handler jobs, but it'd be a much bigger change to introduce such a
pattern into the code.

PTAL. That's the last planned change for this UI thing.


Updating project name of angular2-data-table to ngx-datatable

@amcdnl Please fix the conflicts and update the PR.


Show Description For Posts

Tasks:

Add Post Description Field in Create View
Show post description

If you want i can do the part in the create-view as i've played around with the medium.js-contenteditable before :)


Lightning Terminal - wrong link for Lightning Node Connect

On BTCpay server version 4.4.0, If you open Lightning Terminal and click on Lightning Node Connect, you get https://YOURBTCPAYSERVER/connect instead of https://YOURBTCPAYSERVER/lit/connect.
Not sure it's a general bug or just for the Lightning Terminal version on BTCpay server version 4.4.0.

Thanks for the report. It looks like BTCPayServer is using LiT v0.6.1-alpha: https://github.com/btcpayserver/btcpayserver-docker/blob/master/docker-compose-generator/docker-fragments/opt-add-lightning-terminal.yml#L7
The problem you mentioned was fixed in https://github.com/lightninglabs/lightning-terminal/pull/301 which was released in v0.6.2-alpha.
So to fix this, the version in BTCPayServer needs to be bumped.


Add support for specifying an exit code using :cquit

This is modeled based on the NeoVim implementation. There is also
an earlier pull request from Thinca in 2014.

There is also an earlier pull request from Thinca in 2014.

The link is here: https://groups.google.com/d/msg/vim_dev/_PjyNbUKyRc/oPgr5_ZXc6AJ


Lasertools

Introducing a new tool (Menu Panes-Tools-Lasertools) to simplify the positioning of (circular) objects on the laserbed.

Use case: place blank, round coaster template on laser and properly place your graphic on it.

You can work with round blanks, then mark (with help of your laserpointer or the camera) three points on the circumference of the circle and MK will either recreate the circle for you (e.g. to act as a reference for further manipulation & orientation of other objects) or move the laserhead to the very center of the circle.

Not yet as useful (but keep tuned, align on reference object will honor rotated objects in the future) is the tool to create a square: mark two points on one side of the rectangle on the bed and another one on the adjacent side: then MK wil create a square for you that can act again as basis for further alignment and manipulation.

I question the utility of this. The coordinates are apparently only half visible. But, this clearly does work, and there's little issue with things that function. I'm not sure I see much a usecase for this but I'll assume there is one.

@tatarize Resizing the window will make them appear... Can look into it. My use case is a simple one: I have some round slates I want to engrave and I want to make sure they are properly centered.


update mopub interstitial adapter and remove admob banner adapters

Looks good.

I couldn't find any reference/documentation as to why AppLovinCustomEventBanner has been removed from the repository except for this commit.
Is it documented anywhere on AppLovin website? Do they not support Admob banners mediation anymore?


[BUG] Inaccurate error message "The specified container does not exist" when uploading data with empty blob name using BlobClient.

Library name and version
Azure.Storage.Blobs 12.14.0
Describe the bug
I created a BlobClient by constructor BlobClient(string connectionString, string blobContainerName, string blobName) with parameter blobName set to an empty string.
When I call UploadAsync(BinaryData content), it shows the error message "The specified container does not exist". But I did have the container created.
Expected behavior
I think the error message should indicate the blob name cannot be null or empty.
Actual behavior
Azure.RequestFailedException: The specified container does not exist.
RequestId:aa50a74f-901e-003c-3647-e776e0000000
Time:2022-10-24T01:23:27.7599225Z
Status: 404 (The specified container does not exist.)
ErrorCode: ContainerNotFound

Content:
<?xml version="1.0" encoding="utf-8"?><Error>ContainerNotFound<Message>The specified container does not exist.
RequestId:aa50a74f-901e-003c-3647-e776e0000000
Time:2022-10-24T01:23:27.7599225Z</Message></Error>

Headers:
Server: Windows-Azure-Blob/1.0,Microsoft-HTTPAPI/2.0
x-ms-request-id: aa50a74f-901e-003c-3647-e776e0000000
x-ms-client-request-id: 89117e0d-204d-4525-a842-8ef0485e58f7
x-ms-version: 2021-10-04
x-ms-error-code: ContainerNotFound
Date: Mon, 24 Oct 2022 01:23:26 GMT
Content-Length: 225
Content-Type: application/xml

Reproduction Steps
Code to reproduce:
var connectingString = // Storage account connection string.
var blobContainerName = // Blob container name.
var blobName = "";

var filePath = // A file path.

var blobClient = new BlobClient(connectingString , blobContainerName, blobName);

var bytes = System.IO.File.ReadAllBytes(filePath);

await blobClient.UploadAsync(new System.BinaryData(bytes));

In the above code, I set the blobName to an Empty string. I did have a blobContainerName created already. It returns error:
Azure.RequestFailedException: The specified container does not exist.
RequestId:aa50a74f-901e-003c-3647-e776e0000000
Time:2022-10-24T01:23:27.7599225Z
Status: 404 (The specified container does not exist.)
ErrorCode: ContainerNotFound

Content:
<?xml version="1.0" encoding="utf-8"?><Error>ContainerNotFound<Message>The specified container does not exist.
RequestId:aa50a74f-901e-003c-3647-e776e0000000
Time:2022-10-24T01:23:27.7599225Z</Message></Error>

Headers:
Server: Windows-Azure-Blob/1.0,Microsoft-HTTPAPI/2.0
x-ms-request-id: aa50a74f-901e-003c-3647-e776e0000000
x-ms-client-request-id: 89117e0d-204d-4525-a842-8ef0485e58f7
x-ms-version: 2021-10-04
x-ms-error-code: ContainerNotFound
Date: Mon, 24 Oct 2022 01:23:26 GMT
Content-Length: 225
Content-Type: application/xml

I think the error message can be improved to maybe "The specified blob name cannot be null or empty" or something similar. Maybe could do the check when creating the instance of BlobClient (I am not sure)?
Environment
Microsoft Visual Studio Professional 2022 (64-bit) - Version 17.3.5

@seanmcc-msft Are we okay to take the change to throw an error if a user passes a empty or default blob name (or blob container name) when attempting to instantiate a BlobClient and possibly our other storage clients?
There might not be much we can do if they pass a full Uri instead, but at least with the constructors that take the parsed out names we can do a check.

@amnguye, I think we are ok with this functionality, let's proceed.

I had the same issue today. The error "The specified container does not exist." still remains confusing. The container is available, but my filename was an empty string.


fix fix toSql of date literal

Proposed changes
toSql should return '2023-2-1 ' for  DateLiteral 2023-2-1

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Scrollback buffer empty after ^L

Hello,
I noticed that when I use [Ctrl]+L to clear my terminal screen my scrollback buffer within Oh My Tmux is empty. However when using the 'clear' command to accomplish this the scrollback buffer still contains content.
Initially I thought that perhaps this had something to do with my configuration changes. But the same problem occurs with a default configuration. I also thought that perhaps it the problem had something to do was my shell environment Zsh. But the problem persists in Bash as well. I've tested this on multiple machines.
Is this by design? Or is this a problem with Oh My Tmux?
Love your work on this project! Thank you so much for providing this to the community!
I look forward to your response.
Let me know if you need any additional information from me. Happy to assist if needed.
Thanks!
--unixn3rd

Hello @unixn3rd 👋
It's by design. I don't believe it's a problem 😅
$ tmux list-keys | grep clear-history
bind-key    -T root         C-l                  send-keys C-l \; run-shell "sleep 0.1" \; clear-history

If you don't like this behavior, edit your~/.tmux.conf.local copy and add
unbind -n C-l

Hope that helps!

Hello @gpakosz,
Looks like I should have reviewed the config files a bit closer.
Anyhow. Thank you very much for the assistance!
--unixn3rd

No problem.
I let you close the issue if you believe we're done.

I have the same requirement , and thanks for your solution.


support JSON functionality in ORM ?

Hi, awesome work! Here is my feature request to make it even awesomier
Have you consider spending some of your time to help bake JSON data in SQL server feature into popular ORMs? I'd vote for Python's SQLAlachemy, but probably working with C# ORM first makes more sense? I already asked about this on SO, please follow the link to check in details what do I mean: SO question
What do you think? Doesn't this look awesome?

@ruskimarcin Thank you for the feedback and for posting the issue to the community.
I'm assigning this to the author for comment.
Also note for the product team's response you need to add this to the SQL Server forum.

Closing this issue due to inactivity.
Please post any product team updates below and refer here for the latest features.


Add addErrorNote to give a name to a common pattern

Motivation

We have "notes" on certain error messages, i.e., suggestions that aren't
associated with locations.
These look bad because most of the time we use ErrorSection titles for these,
which show up entirely in yellow. Doing it this way makes it more consistent
and look nicer.
Test plan

Before

After

@aprocter-stripe, rebasing over this once it lands would mean you could leave a more wordy error message :)

Not encouraging wordy error messages 😅 we should have wordy error messages in srb.help links, and use Note for things that are short and nearly universally applicable, but which don't have locations.


slow-motion mode for test-execution

I feel this makes sense for [ui] and [robot] but someone requested this even for API tests: https://stackoverflow.com/q/61404697/143475
This can be a "canned" ExecutionHook. We can't support all the potential asks, but we can have one that solves the most common cases, which will be

add a sleep before any HTTP call
add a sleep before any UI navigation action (ideally with a highlight)
[robot] already has a highlightDuration https://github.com/intuit/karate/tree/master/karate-robot#robot - maybe we should unify this

we have a solution for API tests, and karate.stop() now takes a mandatory port argument, wonder why I didn't do this in the first place.
so all you need now is the following:
* configure headers = function(){ karate.stop(9000) }

this will pause before every HTTP call, and the nice thing is you can proceed by doing this:
curl http://localhost:9000

or simply pointing a browser to that URL and refreshing it !

highlight and highlightDuration are now unified for both UI and Robot and default to 3000 ms

0.9.6 released


Unable to verify contracts on Polygon Amoy

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (844caa8 2024-04-20T00:16:40.844544000Z)
What command(s) is the bug in?
forge verify-contract --chain polygon-amoy 0xbc4fa1a123ab3afcc803a87e7321f200db7e4ca8 src/MyContract.sol:MyContract
Operating System
macOS (Apple Silicon)
Describe the bug
I am struggling to verify my contracts with etherscan (polygonscan) for amoy contracts I've deployed.
As I execute the verify-contract command, I get a HTML output like this:
2024-04-20T01:16:29.863704Z ERROR etherscan: Failed to deserialize response: expected value at line 1 column 1 res="
<!DOCTYPE html><title>Just a moment...</title><meta http-equiv=\"Content-Type\" content=\"text/html; charset=UTF-8\"><meta http-equiv=\"X-UA-Compatible\" content=\"IE=Edge\"><meta name=\"robots\" content=\"noindex,nofollow\"><meta name=\"viewport\" content=\"width=device-width,initial-scale=1\">


Re-enable JVMTI test SuspendResume1

The test has been fixed by eclipse-openj9/openj9#16805.
Signed-off-by: Dipak Bagadiya<EMAIL_ADDRESS>

@dipak-bagadiya This PR is ready to be reviewed. Please remove it from the draft state.
@llxia @Mesbah-Alam Request for review.


How to reload content using the include plugin?

I'm making extensive use of the include plugin (love it), but I'm wondering what the best way to reload included content. Is there a js hook I'm missing?

You can re-start the plugin by simply re-adding the class="lazyload" (i.e.: element.classList.add('lazyload')).
Please close if this helps.

Really? That will do it? Awesome.

It appears to get stuck in the lazyloading state on Firefox: http://i.imgur.com/kJtyjw2.png

You can see this on http://www.spokesman.com/, the big photo section right below the top headlines section. (BTW, our newspaper site uses lazysizes for everything)

I won't habe time to Look into this before sunday. I will then.

I forgot the API a little bit. This is by design, if you do not change the currently loaded URL the script won't do much because the content is already loaded.
You have three option:

Change the URL by adding a cache buster (might also be good in case the browser has already cached the answer).
Together with adding the lazyload class remove the delte elem.lazyInclude to make the plugin forget that it is already loaded.
Download the master version and use the new lazySizesConfig.include.allowReload = true option.

Awesome. Thank you!

I'm confused as to how to use the lazySizesConfig.include.allowReload = true option. I've set this in the global config, but don't know how to put it to use. So far, I've tried adding a cache buster to the image URL and changing the class to lazyload, but lazysizes seems to still think that it has already loaded the image and thus the image is not reloaded. I didn't see this option listed in the documentation so perhaps I'm missing something.
I also don't know what you mean by:

remove the delte elem.lazyInclude to make the plugin forget that it is already loaded.


Can't sign in with PTC account?

I have config.json setting account as following:
{
    "auth_service": "ptc",
    "username": "MY_USEERNAME",
    "password": "MY_PASSWORD",
...

but the bot say:
Traceback (most recent call last):
  File "pokecli.py", line 38, in <module>
    from pgoapi.exceptions import NotLoggedInException, ServerSideRequestThrottl
ingException, ServerBusyOrOfflineException
ImportError: No module named pgoapi.exceptions

Could the bot sign in with a Pokemon Trainer Club (ptc) account instead of google?

ImportError: No module named pgoapi.exceptions
Try pip2 install -r requirements.txt --upgrade in your bot root.

Work perfectly! Thanks Bobby.


New obsidian version with readable line length destroys format.

Describe the bug
The checkmark box is now not anymore completly visible through updating the version.
Everything worked awesome, I updated obsidian and now the checkmark thing isnt completly visible (for all tasks in every list)
To Reproduce
Steps to reproduce the behavior:
Install obsidian and plugin, log in and implement a list
Turn "readable line length" ON
update to the current newest obsidian version (29.03)
Expected behavior
After the update you can only see the checkmark box for 5% the rest is not visible. The whole dataview is shifted to the left side.
Screenshots

Desktop (please complete the following information):

Plugin Version: 1.9.0
Obsidian Version: v.0.14.2 (installer version: v.0.13.23)

Additional context
no other context

another screenshot because the last is a bit nonsaying
Yes the things on left side should be the colourful checkmarkboxes
Here a total screenshot of obsidian to see it in perspective

Hey @mmfs37, this is likely an issue for the theme you are using. I cannot reproduce this with either the default Obsidian theme or the theme that I use day to day.


[ibex] Enable bit-manipulation extension

Thanks to the great work of @ganoam Ibex has now support for the RISC-V Bit-Manipulation Extension draft version 0.92 and the implementation is fully verified upstream using RISCV-DV. I think we can consider to enable it also in OpenTitan.
The implementation comes in two flavors that can be selected via parameter. The two flavors enable the following subgroups of the extension:

Balanced: Expected increase in core area 2.2 kGE, no impact on ASIC timing, enables:

Zbb - base extension: e.g. count leading/trailing zero, pop count, min/max...
Zbs - single-bit instructions: set, clear, invert, extract (useful for controlling peripherals)
Zbf - bit-field place (very useful for controlling peripherals)
Zbt - ternary: conditional move, mix, funnel shift (useful for constant execution time)

Full: Expected increase in core area 6.4 kGE, no impact on ASIC timing, enables all of the above as well as:

Zbp: generalized reverse, or-combining and shuffling (bitwise reverse and byteswap, zero-byte masking already part of Zbb)
Zbe: parallel bit extract and deposit (unlike in bit-field place, non-contiguous placement and extraction can be handled)
Zbc: carry-less multiplication
Zbr: CRC32

Unless we are going to implement crypto algorithms in software, the balanced implementation is probably sufficient. But I would be interested to hear your opinions.
It's worth nothing that even though this implementation is verified, the spec is still a draft. Before being ratified it might still change and those changes may be backwards incompatible. (Neither GCC or Clang have committed to maintaining support upstream for unratified versions of the specification.)
In addition, compiler support is currently not mature. Only few instructions (30 of 95, only instructions of the balanced flavor above) can be inserted automatically. Consequently, the performance gain is low (+1.7% in CoreMark, +1% in Embench). The remaining ones require the use of intrinsics. But many of the available instructions seem to be helpful for the DIFs/mmio where it seems worth to use intrinsics. My understanding is that both our LLVM and GCC toolchain have support.
For more information

On the implementation: https://ibex-core.readthedocs.io/en/latest/instruction_decode_execute.html
On the draft spec: https://github.com/riscv/riscv-bitmanip/blob/master/bitmanip-0.92.pdf
On compiler support: https://www.embecosm.com/2019/10/22/gcc-risc-v-bit-manipulation-extension/

This is probably relevant for many people. To ping some: @sjgitty , @tomroberts-lowrisc, @tjaychen , @cdgori , @eunchan , @msfschaffner,  @moidx , @asb, @gkelly , @lenary , @alphan ...

My understanding is that both our LLVM and GCC toolchain have support.

This is approximately correct. We have been building a combined toolchain with support for the B extension for about a month now, and could switch to using it instead. It supports assembling B instructions, and in gcc's case, compiling to B instructions (support for this has not landed in LLVM upstream yet). All the toolchain's libraries are built without the use of the B extension so that they can be run on ibex with or without B extension support.

Thanks @lenary. If I understand this correctly, it means that using the current OpenTitan LLVM toolchain:

We could start to use bitmanip instructions via inline assembly e.g. in mmio when dealing with peripherals.
No bitmanip instructions would be automatically inserted as for example with the patched GCC (which can only add a small subset anyway at the moment).

We have been building a toolchain. It is not the one that developers currently download, but a migration would not be as difficult as the one we just did to migrate to clang.
If we switched to it:

We could use B instructions in inline assembly.
GCC would generate B instructions when compiling C code.
Clang would not generate B instructions when compiling C code.

I'm not sure how well you can control which B instructions GCC would generate, as I know ibex will only support a small subset of the entire B extension. It would be useful to know precisely which parts of the B extension that Ibex plans on supporting.

One issue we have is the toolchain chooses the -march and -mabi flags (because these have to be compatible with the built toolchain libraries). We only ship one config file that sets these for meson, and meson_init.sh only looks for one of these configurations (unless manually overriden). These configuration files should not be manually edited.
Switching to the new toolchain will have to happen after bit manipulation was switched on for Ibex, so that we don't get CI failures. In all honesty, it also needs to wait until @luismarques is back so I can work with him to ensure I've not missed anything.

Unless we are going to implement crypto algorithms in software, the balanced implementation is probably sufficient. But I would be interested to hear your opinions.

This seems correct to me.   The carry-less multiply would be the most appealing part of the full implementation, but I think all of our Galois-field stuff is going to be in (dedicated) hardware.
Do we have any idea of potential code size reductions from utilizing the balanced implementation?  Seems like DIFs could become smaller because of single-bit / bit-place instructions.  This alone might be able to justify the incremental area.

Thanks a lot @lenary for the clarification and for providing more details.
Regarding which B instructions are going to be supported: It's the idea of this issue to make a decision. But as you say, GCC does not allow you to control which B instructions are generated. Currently, it only emits instructions in the balanced group described above anyway: Zbb, Zbs, Zbf, Zbt. So for the moment, we would be safe to enable only the balanced group  in Ibex and use GCC without your latest modification. However, having B emission disabled seems to be safer until GCC allows to precisely control which instructions to emit.

Do we have any idea of potential code size reductions from utilizing the balanced implementation? Seems like DIFs could become smaller because of single-bit / bit-place instructions. This alone might be able to justify the incremental area.

Noam did some experiments with CoreMark and Embench and the effect on code size was negligible. However, he wasn't using inline assembly for this and just relied on GCC inserting them. Also, we have support for compressed instructions in Ibex which already gives reductions in code size. This somewhat diminishes potential code size gains of the B extension (B instructions are always 32-bit).
In the PULP systems, we added a custom B extension (long before the RISC-V B extension, I think it even started when we were still using OpenRISC) and we were using these instructions in the runtime functions (interrupt handling, peripherals etc.) a lot. This gave us nice gains in performance and energy.

Code Generation for the B extension has landed in upstream LLVM (not sure if in time for LLVM 11, it just missed the branch, but might get backported). This allows you to enable/disable by sub-extension, so it's certainly something we could look into re code size of Balanced vs Full. It's good to know that GCC currently only emits from the Balanced set, but again this isn't behaviour we want to rely on.
I'm not sure when I can schedule the investigation of how enabling B code generation affects code size, but that feels like the next step: getting data so we have something to base the decision on. I hope that our current software codebase will be representative.


Only 1 connection on Windows

Hi,
I was suspecting it for quite a while already, but today I finally got proof. Twister on Windows has in general only one connection. Even after a couple of hours it mostly still has only one connection. (Only at really rare times it randomly climbs up to 2-4 connections) The amount of peers and DHT peers is quite normal though.
When I used a Linux instance of Twister for the first time today from the same network, it had 9 connection in 2 minutes. Therefore I can finally make the conclusion that Windows is the problem ;-)
@miguelfreitas If I understand it correctly you rather evade developing on Windows. I'm willing to take a look what causes this issue. Maybe you already have an idea what might be the problem though? Or maybe you can give me some hints on where to look?
Arco

When you say "twister on windows" are you actually referring to mingw or cygwin? my experience is that mingw port presents a more smooth operation. cygwin tends to introduce bugs that do not exist when we use it on linux. So please make sure you're using mingw compilation.
You are right to point that I never use/develop anything on windows. Your help is more than appreciated!
I'd start checking netstat results. You should be able to see outgoing connect requests, if any. Also, iirc running twister with -debug may add some verbosity to the "bitcoin side" which is the connection count you must be referring to.

I'm using a Cygwin version, so I will see if I manage to do a MinGW build. And I will try to gather more data using netstat and Twister logging.

See twister website news pages, I've posted a virtualbox image that compiles twister for windows using mingw for you.
My advice: don't try to set your own mingw build enviroment, it's a nightmare.


[RhythmGame][Song] 0.10 Android - Stopping on the now loading

I use the Method 1.
Download the game data dump, extract and copy the data & mx directories to this folder. Android/data/com.UtaMacrossOffline.UtaMacrossOffline/files/
The game launch successfuly to the main page.
Tap start, it continute to the three quizs.
However, when I complete these quizs.
It stop on the page that sd character freya and kaname are in the middle, and now loading on the lower right corner.

Hello,
Can you with at least the version 0.11, enable the debug and error log which is in the UMO option menu (Gamepad icon on bottom left in the title screen) and after reproducung the issue, send the log located in Android/data/com.UtaMacrossOffline.UtaMacrossOffline/files/Log.txt .
Thanks.

No news, closing the bug


Add Open Measurement APIs

Hi Kiro,
I hope all is well.
We noticed that OMID implementation has been updated. could you please let us know how to pass the OMID now?
below is the previous implementation we had
featureFlags:{'enableOmidBeta': true},
omidMode:{
FULL: [ new RegExp('[.+.]moatads.com.$') ]
},
we updated the source files and passing the omidVendorAccess as below, however the access to MOAT is still limited.
omidVendorAccess: {"MOAT":'FULL'},
Test page URL: https://dmscg.com/tech/video/jsplayer/index.html
play the video, and search for OMSDK in the network tab. screenshot: https://prnt.sc/JUCRtJI9dRVY
Thanks

Hello @moeednisardms ,
I would recommend looking at this issue thread. The code snippet there should help you with your OMID implementation.
Please feel free to re-open that thread, if you have any questions on its contents.
Thank you,
Jackson
IMA SDK team

Hi @Kiro705,
Thanks for sharing the issue thread. The issue has been resolved.


Integrate the repository with Travis CI

@alinebrito is working on it.
Reference: https://docs.travis-ci.com/user/languages/javascript-with-nodejs/

ping @alinebrito

ping @alinebrito :trollface:


docs: document the required permissions for the GitHub token

Description
Precise the permissions required to create the Pull Request comment. This is required when the default permissions are set to read in the repository where the action runs.
Testing

I have tested the configuration in various repository with default read permissions. For instance, see https://github.com/bonitasoft/bonita-doc/blob/32b9fe9f04f7f70e7f06e1274816b9dd2febdb78/.github/workflows/contribution-check.yml#L9-L12

Hi @jef
did you have time to have a look at this PR?
Thanks


Framerate/ Refresh Rate mismatch causes microstuttering on Linux and maybe Windows.

SteamVR/VRChat/Beat Saber are the only ones I've tested
Description
I discovered that when messing with the sidequest desktop app if I changed the refresh rate of the headset and its apps to not match what ALVR streamer is set to it will cause micro stuttering like this. I think forcing the android/meta app to sync with the set refresh rate of the ALVR streamer will help make the client run better but you might have a idea on how to fix this then me.
https://github.com/user-attachments/assets/c7098e29-267d-4eeb-9acd-580f61eecaa4
I was able to fix it by change the refresh settings in the side quest desktop app to match my ALVR settings and the micro stuttering is reduced if not gone in my case.

General Troubleshooting

[x] I carefully followed the instructions in the README and successfully completed the setup wizard
[x] I read the ALVR GitHub Wiki

Environment
LIBVA_DRIVER_NAME=radeonsi
VDPAU_DRIVER=radeonsi
Hardware
Note: for Linux, an upload to the hw-probe database is preferred: hw-probe -all -upload
https://linux-hardware.org/?probe=a061c15130
CPU:
AMD Ryzen 9 7900X3D
GPU:
AMD RX 7800 XT
GPU Driver Version:
Mesa 24.3.0 vulkan-radeon 24.2.7
Audio:
pipewire
Compiled with libpipewire 1.2.7
Linked with libpipewire 1.2.7
Installation
ALVR Version:
20.11.1
ALVR Settings File:
session.json
SteamVR Version:
SteamVR beta 2.9.2
Install Type:
Arch AUR alvr-bin
OS Name and Version (winver on Windows or grep PRETTY_NAME /etc/os-release on most Linux distributions):
CachyOS

It happens if I force it to a refresh rate that doesn't match my ALVR streamer settings. I believe it can happen even if I haven't messed with SideQuest.
I don't think there is synchronization of framerates and refresh rates between the streamer and the headset, and this mismatch is causing the framerates to jump. For example, the Quest 3 runs at 120Hz by default, and you can't change that without using an app like SideQuest. It could be that the app on the headset is running at a refresh rate that doesn't match the ALVR streamer's refresh rate, causing it to flash or jump between refresh rates at a very high speed, resulting in a micro-stuttering effect.
The only way to fix something like this is by enabling client sync or providing options to adjust the client refresh rate on the headset. However, this needs to be researched further.

ALVR choses the framerate you set in the settings. if you use sidequest it will mess up and you will get stuttering. if the framerate doesn't reset back to the setting in the dashboard, try to restart the headset

I think there might be fault in how it works because when I set the refresh rate to 90hz in sidequest to match the ALVR dashboard it makes existing stutters go away

I am going to close the issue I still think refresh rate mismatches can happen even without messing with sidequest but as of right now I can't prove that on my end. I just wanted to help people and thought maybe the reason my thing was micro stuttering like that was because of a refresh rate mismatch and something wasn't being checked correctly but sure enough resetting the headset had the same effect as doing that. I feel like a fool and will stop posting bug reports

Well you're right. Meta has a feature of framerate scaling for throttling. Currently we are not handling that case. So if your headset ends up throttling then using sidequest for force the framerate would remove the stutter. We are planning on a fix but unfortunately this is delayed a lot. We are moving towards the v21 stable release, but this fix cannot land until v22, so most probably it will take a year or more.

would including my temp fix in the wiki be a good idea? the thing is I don't know if it will help Windows users all that much my testing on Windows was inconclusive

I am going to close the issue I still think refresh rate mismatches can happen even without messing with sidequest but as of right now I can't prove that on my end. I just wanted to help people and thought maybe the reason my thing was micro stuttering like that was because of a refresh rate mismatch and something wasn't being checked correctly but sure enough resetting the headset had the same effect as doing that. I feel like a fool and will stop posting bug reports

Nah bug report like this are completely fine you'll never be considered a fool.

I just thought I was making noise about nothing again  and feel like a fool for that

In fact this is a perfect example on how bug reports should be written

would including my temp fix in the wiki be a good idea? the thing is I don't know if it will help Windows users all that much my testing on Windows was inconclusive

I'm conflicted on this. because setting the framerate on sidequest would break all other framerates until you restart the headset. this should be stated very clearly, maybe even in bold. I can accept a PR for the wiki

This isn't noise, it may not have gotten all the details right but its definitely not noise
this is noise, for example
https://github.com/alvr-org/ALVR/issues/2494
It wouldve been better to join the discord server or at a bare minimum some text.

would including my temp fix in the wiki be a good idea? the thing is I don't know if it will help Windows users all that much my testing on Windows was inconclusive

I'm conflicted on this. because setting the framerate on sidequest would break all other framerates until you restart the headset. this should be stated very clearly, maybe even in bold. I can accept a PR for the wiki

I am conflicted myself the thing I am doing is using sidequest to set the framerates to match overriding framerate scaling for throttling feature at least that is what I think is happening

I am also having a frametime issue and I believe this is related. I am using windows 11, quest 2. I have this constant staircase latency with (client system (not alvr latency)), when this rises fps jumps between 80/40/26.5, once this latency falls ALVR indicates smooth 40fps, until this latency rises again. I have ALVR set to 80hz and im forcing the game to a 40fps cap with RTSS, also scanline sync at 80 in RTSS as that seems to help a little. Im using this for msfs2024, there is no hitting 80 or even 72hz, so half rate is required for smoothness. I have tried Optimized game latency ON and OFF, usb connection vs. wireless , codec changes, etc.
This photo is with a game 40fps cap

This is with no fps cap (computer running ~45-50 fps avg here), you can still see the staircase latency, and of course the constant jumping between 40 and 80 fps.

This is 72hz mode with a 36fps cap, also confirmed with sidequest 72hz mode on quest side after a fresh reboot.

Running steam Home at 80hz and 80fps still has the staircase latency but not the frametime jumps

I am also having a frametime issue and I believe this is related. I am using windows 11, quest 2. I have this constant staircase latency with (client system (not alvr latency)), when this rises fps jumps between 80/40/26.5, once this latency falls ALVR indicates smooth 40fps, until this latency rises again. I have ALVR set to 80hz and im forcing the game to a 40fps cap with RTSS, also scanline sync at 80 in RTSS as that seems to help a little. Im using this for msfs2024, there is no hitting 80 or even 72hz, so half rate is required for smoothness. I have tried Optimized game latency ON and OFF, usb connection vs. wireless , codec changes, etc.
This photo is with a game 40fps cap
This is with no fps cap (computer running ~45-50 fps avg here), you can still see the staircase latency, and of course the constant jumping between 40 and 80 fps.
This is 72hz mode with a 36fps cap, also confirmed with sidequest 72hz mode on quest side after a fresh reboot.
Running steam Home at 80hz and 80fps still has the staircase latency but not the frametime jumps

I would recommend using sidequest to change the fps to match ALVR

the thing is I discovered this issue on Linux I have no idea if changing the fps affects Windows or not


Cannot differentiate BlogListPage and BlogPostPage for styling

🚀 Feature
Add class names to individuate blog theme components.
Have you read the Contributing Guidelines on issues?
Yes
Motivation
Having a class name to differentiate blog list pages from post pages would allow styling the pages using the global style methodology.
Pitch
Currently, global styling is not possible since there is not variancy between the structures of BlogListPage and BlogPostPage. Adding another class name (e.g. blog-list-page, blog-post-page), alongside the blog-wrapper class name, would allow for global styling.
Furthermore, there doesn't seem to be another way to inject additional class names into these components except to wrap them. I don't feel wrapping the components is a good flow, just to add classes to target for styling, since a global styling methodology often tries to lessen its dependency on structure.
For the docusaurus-theme-classic, these are the proposed changes to the blog components:

[ ] Add the blog-list-page class name to the Layout component for BlogListPage
[ ] Add the blog-post-page class name to the Layout component for BlogPostPage
[ ] Add the blog-tags-list-page class name to the Layout component for BlogTagsListPage
[ ] Add the blog-tags-post-page class name to the Layout component for BlogTagsPostsPage

Make sense, all docusaurus pages should have a convenient way to target them with custom CSS.
We should have one wrapper class per "plugin" (like blog-wrapper), and unique classname per unique page types for better granularity.
Assigning this easy task to MLH fellows

FYI there's also a need to be able to target with CSS a specific doc/version/blog/page: https://github.com/facebook/docusaurus/issues/4280
Ie we probably want a classname like docs-wrapper doc-item-page version-1.0.0 docId?

@ShinteiMai  to work on this.


Instant translator

Description of the new feature / enhancement
Given that when selecting any text anywhere, web, desktop, work, excel at the end of the selection then translate to a configured language of PowerToys.
Scenario when this would be used?
Any location whether on the web or desktop application when text is selected.
Supporting information
No response

Possible Route of Implementation:
Feature Reqeuest #25197 -- PrintScreen Context Menu PowerToy
 or
The proposal is designed to give the end user many options upon hitting PrintScreen and selecting the context (rectangle clip, specific window, etc).
This provides a Translate-anything-on-screen feature.  Even in situations where the app is not localized, and where user content is not translated -- especially useful for chats with foreigners in apps that don't have a built-in translator.

Since Powertoys doesn't have this function,  I'm using DeepL for it, and it works well


New Feature - Socket Communication For Warppers

I'm thinking of developping a new feature in the WebUI. Instead of making all warappers dealing with the complexity of APIs call, memory object sharing between C and other languages wrappers (Nim, Go, V, Rust, Deno, JS, TS, Python...), instead of that, why no a simple socket connection, and WebUI send/receives event in JSON format.
Now
[Browser] <--> WebSocket <--> [WebUI] <--> Direct APIs call <--> [Wrappers] <--> [Application]
Suggestion
[Browser] <--> WebSocket <--> [WebUI] <--> Socket <--> [Wrappers] <--> [Application]

FFI that we are using is faster. Let's keep it.


Add MobileViT-small

Add the model MobileViT-small to model bank

@mschrimpf  have you seen this error before in tests:
@classmethod
    def _tensor_to_numpy(cls, output):
>       return output.cpu().data.numpy()
E       AttributeError: 'BaseModelOutputWithNoAttention' object has no attribute 'cpu'

Seems similar to #683, but not sure what the fix is

looks like output isn't actually a tensor, but rather the Module producing the tensor. It might be that the layer assignment isn't right; or you'll have to debug to see why no tensor is created.


Adding the observation ship model.

Adding the observation ship blender file. The file contains some objects that will need to be removed such as camera and lights. The Frame is on layer 1. The windows are on layer 2. The seats and console are on layer 3.

Eggselent. I'll get in in there after a late dinner. Thank you for this!!
:smile:

No problem at all. Please let me know if you would like anything changed.
I'll create another ship if that one doesn't work out. It doesn't take too
long to do and helps me get better at modeling.
On Wed, Feb 3, 2016 at 7:52 PM, owntheweb<EMAIL_ADDRESS>wrote:

Eggselent. I'll get in in there after a late dinner. Thank you for this!!
[image: :smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/owntheweb/VR-Solar-System-Tour/pull/1#issuecomment-179590278
.


[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]

Forward-merge triggered by push to branch-22.10 that creates a PR to keep branch-22.12 up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge.

SUCCESS - forward-merge complete.


Docs - LDAPConfig/tls description fix

Hi @MichaluxPL, thanks for your contribution! Unfortunately, that part of the doc is autogenerated, so could you also update the string in the code here?

@leonardoce rightly hinted that we need to change also api/v1/cluster_types.go and then run make checks to regenerate the .md file. @MichaluxPL can you please retry?

Please change api/v1/cluster_types.go file and run make checks to regenerate this file.

I've tried to run make checks, but it fails with error:
pkg/configparser/configparser.go:21: File is not gofmt-ed with -s (gofmt)

a struct containing the default values
a map from string to strings (e.g. the one that can be extracted for a Kubernetes ConfigMap)
an environment source like the system environment variables or a "fake"
one suitable for testing.
pkg/utils/imagename.go:83: File is not gofmt-ed with -s (gofmt)
//     GetImageTag("postgres") == "latest"
//     GetImageTag("ghcr.io/cloudnative-pg/postgresql:12.3") == "12.3"
//
pkg/utils/pod_conditions.go:58: File is not gofmt-ed with -s (gofmt)
// IsPodActive checks if a pod is active, copied from:
controllers/pooler_update.go:57: File is not gofumpt-ed (gofumpt)

controllers/pooler_update.go:118: File is not gofumpt-ed (gofumpt)
make: *** [Makefile:154: lint] Error 1

@MichaluxPL if you do make apidoc that should do the changes.

@MichaluxPL if you do make apidoc that should do the changes.

OK, I've pushed additional changes, You've mentioned.

/ok-to-merge

/ok-to-test

/ready-to-test