haskell-updates fixes: hevm, hourglass and crypton-x509

Description of changes
I am fully aware the ongoing work at https://github.com/ethereum/hevm/pull/471. But in the interest of unblocking hevm and its downstream packages. I made this fix for now, and I will monitor the upstream work and remove the patch when needed.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Can you follow the examples in configuration-common.nix and fetch this patch from GitHub with fetchpatch?

Can you follow the examples in configuration-common.nix and fetch this patch from GitHub with fetchpatch?

Oh, at the moment, there is no such patch as permanent link from github available. I think I found other examples who store patch under pkgs/development/haskell-modules/patches/. What's the guideline here?

Can you follow the examples in configuration-common.nix and fetch this patch from GitHub with fetchpatch?

Oh, at the moment, there is no such patch as permanent link from github available. I think I found other examples who store patch under pkgs/development/haskell-modules/patches/. What's the guideline here?

I haven’t heard about a patch link which can’t be made into a permanent one. What’s the issue, do you need a commit range? GitHub can do that.
The guideline is: Avoid if sensibly possible.

I haven’t heard about a patch link that can’t be made into a permanent one. What’s the issue? Do you need a commit range? GitHub can do that.

Oh, I didn't say it was not possible. I was just saying it was not there yet. I will create one in the right place. No problem

The guideline is: Avoid if sensibly possible.

Updated the fixes, and now ready for reviewing:

hevm patch now is fetched from github
hourglass needs jailbreaking.
crypton-x509 no longer needs special configuration

CC @maralorn

We prefer not to merge the base branch into pull requests, our git history is long enough as it is.

Thank you!
I will do the hourglass fix manually, since we need it on the branch now.
The url to the hevm patch is unstable. Can you please change that to an explicit commit range? Also can you please add a comment to the upstream PR?

I settled on a different fix for hourglass so you can remove your override from this PR. (after a rebase)

Me again, sorry. I also applied the crypton-x509 fix on the branch, so you can also drop that. :D

We prefer not to merge the base branch into pull requests, our git history is long enough as it is.

okay, I just rebased instead.

I will do the hourglass fix manually, since we need it on the branch now.

Ack, I have removed my fix.

Me again, sorry. I also applied the crypton-x509 fix on the branch, so you can also drop that. :D

ack.

The url to the hevm patch is unstable. Can you please change that to an explicit commit range?

Yes, that makes sense. Done.

Also can you please add a comment to the upstream PR?

It doesn't have an upstream PR for this minimum patch, though. I intend to create a PR with a bigger changeset. Is that okay?

Sure.


Tpm2netnew

1. Tell us something about your changes.
Re-implemented the tpm2.net driver on the new ProviderUdp.
Has had limited testing - looking for testers!
2. If this changes affect the .conf file. Please provide the changed section
"device" :
{
"name"       : "new tpm2 example",
"type"       : "tpm2net-new",
"host"       : "<IP_ADDRESS>",
"port"       : 6502,
"colorOrder" : "rgb"
},
3. Reference an issue (optional)
Note: For further discussions use our forum: forum.hyperion-project.org

thx for implementing!
could you replace my tmp2net driver with yours. Mine was only a way to get rid of old udp driver without loosing a device.

Done!


Bump psycopg2-binary on python 3

See if it's possible to bump

Superseded by the following:

https://github.com/DataDog/integrations-core/pull/14363
https://github.com/DataDog/integrations-core/pull/14364


What is the rate?

What does the rate refer to or how is it calculated?
thx

@0528Makoto it is a good question, you could find an answer in #85

thx


Start keeping a changelog

Small suggestion, but I think it would benefit the project to keep a changelog.

It is about time, isn't it

Added changelog to all 3 crates in #154.


Update Golang and tools version for new language idioms

This update addresses compatibility issues with newer Go language idioms. The current version of moq fails when encountering certain statements, such as:
for range 5 {
  // ...
}

which results in the following error:
couldn't load source package: sample.go:110:12: cannot range over 5 (untyped int constant)

##Changes:

Go version: Bumped from 1.18 to 1.22.5
golang.org/x/tools: Updated from 0.17.0 to 0.24.0
Replaced usage of deprecated ioutils

@arkadiuszos4chain could you add a test case that has some of the new constructs in the source somewhere?

@arkadiuszos4chain could you add a test case that has some of the new constructs in the source somewhere?

@ns-skaranth I'm not sure if I choose best place, but you can check my test case.

@arkadiuszos4chain could you rebase the PR on the master branch? There were some changes made to run tests on the latest stable version

@arkadiuszos4chain could you rebase the PR on the master branch? There were some changes made to run tests on the latest stable version

@sudo-suhas done.


typo in default graph

environTment settings. see screenshot.

Fixed https://github.com/vizorvr/vizor/commit/b64a305d485258f4972a9b75b5c8d12a60005a8d


Create a file using custom data

I'm trying to create a file through custom data in cloud init. So after deploying a VM, I should be able to see a file created by the name test.cfg under /root and it should have a content "var_log=10G,/var/log,0755,remap,ext4,defaults"
"customData": "[base64(concat('#cloud-config\nhostname: ', parameters('CustomDataHostName'), '\nwrite_files:\n- owner: root:root\n- path: /root/test.cfg\ncontent: | ',parameters('customfiledata'),'\nfqdn: ', parameters('CustomDataFQDN'), '\nmanage_etc_hosts: true\n'))]"
path: /root/test.cfg
customfiledata: var_log=10G,/var/log,0755,remap,ext4,defaults

Hi @lirishlal,
We are happy to help but this channel is reserved for documentation issues/bugs. Is this a request to update documentation or are you seeking product support?
For the best possible response to a VM related issue, we recommend posting your issue on our Azure VM Forum Channel
Your other option is to open a service request with our technical support team using these steps.
If you don't have a support plan, please send mail to<EMAIL_ADDRESS>with your subscription ID and a link to this Github thread (for context) and we will assist you with engaging Azure technical support.
We will now close this issue on Github but hope to engage with you further via one of the support channels mentioned above.
Cheers.


Examples for basic instrumental variables usage with R and Python

Hi @aleixrvr ,
thank you for opening this PR. I'll have a look at it in the next days and come back to you with any comments. If everything is alright , I'll merge it and publish it at docs.doubleml.org

Hi @SvenKlaassen and @aleixrvr
I just made a couple of changes and included the examples in the gallery now. I also changed the base for this PR to dev. We'll add it to main together with the next bigger changes & PRs
Thanks @aleixrvr for adding the examples 🙏
After the tests are finished successfully this PR can be merged.


[AutoPR track2_azure-iot-deviceupdate] Merge deviceupdate

Create to sync https://github.com/openapi-env-test/azure-rest-api-specs/pull/3066
You can install the use using pip install of the artificats.

API change check for azure-iot-deviceupdate
API changes have been detected in azure-iot-deviceupdate. You can review API changes here


single() returns Invalid media type: expected no more input.

Describe the bug
When single() doesn't find anything, I get "Invalid media type: expected no more input."
full error
flutter: Event: MatchError, Properties: {error: Error on line 1, column 32: Invalid media type: expected no more input.
╷
1 │ application/json; charset=utf-8,application/vnd.pgrst.object+json; charset=utf-8
│                                ^
╵}
flutter: Error fetching match: Error on line 1, column 32: Invalid media type: expected no more input.
╷
1 │ application/json; charset=utf-8,application/vnd.pgrst.object+json; charset=utf-8
│                                ^
╵
To Reproduce
Steps to reproduce the behavior:

install supabase_flutter: ^2.5.1
try to call do something as
await supabase
.from('profile')
.select()
.eq('id', 'gibberish')
.single();
See error

Expected behavior
Some ordinary exception as "not found"
Version (please complete the following information):
├── supabase_flutter 2.5.1
│   ├── supabase 2.1.1
│   │   ├── functions_client 2.0.0
│   │   ├── gotrue 2.6.0
│   │   ├── postgrest 2.1.1
│   │   ├── realtime_client 2.0.4
│   │   ├── storage_client 2.0.1

I'm also having the same error.
supabase: 1.11.11
supabase_flutter: 1.10.24

Is that all stacktrace you get? Shouldn't there be some info about in which file and line it occurs?

unfortunately and surprisingly, this is the whole stack, I don't get anything else.

Is that all stacktrace you get? Shouldn't there be some info about in which file and line it occurs?

I think you have to use :
await supabase .from('profile') .select() .eq('id', 'gibberish') .maybeSingle();

@Mykyta-Chernenko @allipiopereira
Hmm, I am unable to reproduce this issue. When I use .single() and query in a way that it returns 0 rows, I get the following error.
PostgrestException(message: JSON object requested, multiple (or no) rows returned, code: PGRST116, details: The result contains 0 rows, hint: null).
Are you using the hosted version of Supabase? Does the same issue happen on a fresh Supabase instance?

@Mykyta-Chernenko @allipiopereira Hmm, I am unable to reproduce this issue. When I use .single() and query in a way that it returns 0 rows, I get the following error.
PostgrestException(message: JSON object requested, multiple (or no) rows returned, code: PGRST116, details: The result contains 0 rows, hint: null).
Are you using the hosted version of Supabase? Does the same issue happen on a fresh Supabase instance?

I get the error like this.
   final response = await supabase
        .from('app_settings')
        .select('review_version_ios, review_version_android')
        .single();

    id is non-existent

    I get

    [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: Error on line 1, column 32: Invalid media type: expected no more input.
  ╷
1 │ application/json; charset=utf-8,application/vnd.pgrst.object+json; charset=utf-8
  │                                ^
  ╵
#0      wrapFormatException (package:http_parser/src/utils.dart:15:5)
#1      new MediaType.parse (package:http_parser/src/media_type.dart:46:7)
#2      _contentTypeForHeaders (package:http/src/response.dart:79:45)
#3      _encodingForHeaders (package:http/src/response.dart:72:24)
#4      Response.body (package:http/src/response.dart:28:22)
#5      PostgrestBuilder._parseResponse (package:postgrest/src/postgrest_builder.dart:269:31)
#6      PostgrestBuilder._execute (package:postgrest/src/postgrest_builder.dart:164:14)
<asynchronous suspension>
#7      PostgrestBuilder.then (package:postgrest/src/postgrest_builder.dart:372:24)
<asynchronous suspension>

I use supabase hosted version.
I tried to restart the db, the issue is still there
I created a new db with the minimum migration, the issue is still there
here is the migration I applied to the fresh db
-- Migration for the app_settings table
create table if not exists app_settings
(
    id                     serial primary key,
    review_version         text not null,
    review_version_android text not null,
    review_version_ios     text not null,
    version                text not null
);

I use Frankfurt region.
Thanks for you help!

I get the same error message, so I can confirm that. Also I get the same message if .single() is used and multiple results are returned. I tested supabase_flutter: ^2.3.4 and ^2.5.3.

Having same issue

Anyone facing this issue, would you be able to let me know what version of PostgREST you are running on the Supabase instance?

Anyone facing this issue, would you be able to let me know what version of PostgREST you are running on the Supabase instance?

Auth version
2.152.1
PostgREST version
12.0.2
Postgres version
<IP_ADDRESS>

@Mykyta-Chernenko
Still unable to reproduce this issue. I am getting the following response performing the following sample query that you have shared on an iOS simulator.

PostgrestException(message: JSON object requested, multiple (or no) rows returned, code: PGRST116, details: The result contains 0 rows, hint: null)

final response = await supabase
        .from('app_settings')
        .select('review_version_ios, review_version_android')
        .single();

i have the same issue, here is my play around
      final response = await client.from('users').select().eq('id', userId);
      if (response.isEmpty) return null;

      return UserModel.fromJson(response.first);

I upgraded the PostgREST version from 12.0.2 to 12.2.1 and Postgres version from <IP_ADDRESS> to <IP_ADDRESS>. The weird error message is fixed now.

I had the same issue and upgrading to the latest version fixed the error 🚀
Auth version
2.158.1
PostgREST version
12.2.3
Postgres version
<IP_ADDRESS>

I would close this issue as it seems that upgrading the postgrest version seems to solve the issue. Anyone here still having the issue after upgrading?


Mismatch in the published package content.

There seems to be a problem with the npm package currently published. Please check the content in the dist folder.
https://www.npmjs.com/package/nsfwjs/v/4.0.0?activeTab=code
It should look like this instead.
https://www.npmjs.com/package/@hazya/nsfwjs?activeTab=code
It is missing the models folder which contains the model files converted to base64. It also seems to contain the old files as well. The above is what is pushed when doing a simple npm publish which also includes the models folder since it is not npm ignored. I'm not sure why the nsfw package doesn't contain the models folder. Is there something wrong with the shipit script?
@GantMan @mazenchami FYI

@haZya thanks for reporting this. we've been chatting on the side but as of v4.1.0 this has been fixed: https://www.npmjs.com/package/nsfwjs?activeTab=code


NixOS Lima VM flake

Done:
https://github.com/heywoodlh/flakes/commit/9828abc6becc4591d8a19bc6bc0dff71ff0e0338
https://github.com/heywoodlh/nixos-configs/commit/1ede8e5968c7fb401bbf3a0fb48aa7b7e28bad20


DailyFx live mode issue

The problem is the DailyFx custom data in live mode.
At every request the provider returns a JSON with all the events for the actual date (in UTC) and as the day goes by, the fields of the JSON changes.
Every event has a DisplayTime field that as its name indicates, points the exact time the event is fired.
The problem is that in real time it takes some minutes for the event to update all its properties, particularly the field better in the JSON (property Meaning in the DailyFx class).
E.g. an event has a DisplayTime equal to 10:30, and it fires the OnData(DailyFx calendar) but at that precise moment the property Meaning is None. Two minutes later at 10:32 the Meaning of that event is Better or Worse, but of course, the algorithm running in live mode never realizes of that change.
To overcome this problem I made the following:

I made a static property in the DailyFx class that keeps track of the previous values, is a Dictionaty<string, DailyFx>
If the latest JSON has a change in the Meaning property, I update the DisplayTime to UtcNow

But the updated event doesn't fire the OnData(DailyFx calendar).
Here is the modified DailyFx I used for debugging in live mode with an Oanda paper account.
You’ll see in the GetSource method I used a pair of JSON files with a modified event, the first file is called two times and the second one time and should make the time update. Also there are two Debug.Assert to check the event is correctly updated.
The files are available here and here. Please note I use to testing the 2017-09-28, so in order to test it correctly in future days, you’ll should update the DisplayDate field.

Second issue is that the factory object is being re-created every resolution step. Ideally the factory is created once when the enumerator stack is formed.

Fixed in commit 4c2a2087bab8dc23bd7975a7f4516d692deea74b


[1.9] Bump navstar

High-level description
Bump navstar
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-3602 L4LB unstable when something is deployed in the cluster

Related tickets (optional)
Other tickets related to this change:
Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change: there's no changes.md in 1.9
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: none
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: [link to CI job test results for component]
[x] Code Coverage (if available): [link to code coverage report]
https://github.com/dcos/navstar/compare/b1387b4948c48a7a43d7ad9dc63d35b4112b7855...c4cf24f54498ae1feb40276c9384d3ccab34e89e
https://github.com/dcos/minuteman/pull/117
https://circleci.com/gh/dcos/navstar/504
https://codecov.io/gh/dcos/navstar/commit/c4cf24f54498ae1feb40276c9384d3ccab34e89e

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot label [Ready For Review]


Re-implementation of MemoryStorage business logic as a mutable reference

Codecov Report

Merging #597 (0037c0c) into main (2cb67fe) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #597   +/-   ##
=======================================
  Coverage   93.86%   93.86%
=======================================
  Files         178      178
  Lines       12365    12370    +5
=======================================
+ Hits        11606    11611    +5
  Misses        759      759

Impacted Files
Coverage Δ

storages/memory-storage/src/lib.rs
100.00% <100.00%> (ø)

storages/sled-storage/tests/sled_transaction.rs
99.54% <0.00%> (-0.01%)
:arrow_down:

Continue to review full report at Codecov.

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


Embeds have some sort of Invisible Line

Summary
Any Embed has some sort of invisible line which makes it appear bigger than supposed to
Reproduction Steps

Create any Embed
Send it in any channel

Minimal Reproducible Code
embed = nextcord.Embed(

            description = '**[Rules Here](Insert Rules Link)**',

            colour = 3447003

        )

        await ctx.send(embed = embed)

Expected Results
An Embed with a hyper Link Description and appears normally
Actual Results
An Embed is sent as expected but it has some sort of invisibile line making it lookw weirdly

Intents
guild_messages, members, guilds, guild_reactions
System Information

Python v3.10.2-final
nextcord v2.0.0-alpha

nextcord pkg_resources: v2.0.0a8

aiohttp v3.8.1
system info: Windows 10 10.0.19044

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
No response

This looks a lot like something the library cannot control, this would be a discord bug if it's not intended

Discord bug


fix: typos in the README.md

Like the title says, I hope they are good edits!

Thank you for your contribution :)


Upgrade cypress and associated packages

Upgrade cypress and associated packages
This PR includes updates to Cypress and related packages like cypress-axe as well as some required file migrations

Check out a preview of your pull request here: https://app-servicecanadalabs-dev-pr-preview-1.azurewebsites.net

Check out a preview of your pull request here: https://app-servicecanadalabs-dev-pr-preview-1.azurewebsites.net

Check out a preview of your pull request here: https://app-servicecanadalabs-dev-pr-preview-1.azurewebsites.net


[discussion] Return error on duplicated permission tokens

Originally proposed by me here: https://github.com/hyperledger/iroha/pull/4156#discussion_r1427939825

The idea is to return an error from Executor on Register<NewRole> instruction if there are any duplicates in permission tokens.
Pros:

Robust

Cons:

Backward incompatible

I believe we do this, but should be verified


add nukie music

About the PR
i added nukie music
Why / Balance
buffs crew
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes
its jover for surgery
Changelog
:cl: router

add: Added Chip Nightmare to the nuke music roster.

Summary by CodeRabbit

New Features

Introduced a new audio file, chip_nightmare.ogg, to the sound collections.


151948280 User should be able to configure periodic reports

What does this PR do?

Adds a feature User should be able to configure periodic reports, for example monthly, weekly, daily.

Description of the task to be completed?

The dashboard only allows subscribing to monthly report. This should be changed to be user configurable to either be daily, weekly or monthly.

How should this manually be tested?

You can test this manually by issuing this command; _python manage.py test hc.accounts.tests_profile from the project root directory.

Pivotal Tracker story?

See [Feature #151948280]

Coverage increased (+0.2%) to 80.413% when pulling bb70d14c3b2f1fc7c73ae381075ef39c5c16a896 on ft-user-configure-periodic-reports-151948280 into 58995fa7029ab7fc752c414beb37bd6da67c7216 on develop.

your PR heading and template does not follow our guidelines
Your commit messages does not adhere to our standards

Coverage decreased (-1.04%) to 79.211% when pulling 44056685f9d972c0fe35dfe0d1458d1dcab99be3 on ft-user-configure-periodic-reports-151948280 into 58995fa7029ab7fc752c414beb37bd6da67c7216 on develop.

Coverage decreased (-0.5%) to 79.796% when pulling 5be97a5449baea485d3d7bde06ab0fa78e8e1715 on ft-user-configure-periodic-reports-151948280 into 58995fa7029ab7fc752c414beb37bd6da67c7216 on develop.

Coverage decreased (-0.2%) to 80.0% when pulling ebce12e43bcaaea4d37a7ae94eb3bfc22935795c on ft-user-configure-periodic-reports-151948280 into 58995fa7029ab7fc752c414beb37bd6da67c7216 on develop.

Coverage decreased (-0.04%) to 80.205% when pulling 43ec9b314d550ad947ae8967d7860f027b95a54a on ft-user-configure-periodic-reports-151948280 into 58995fa7029ab7fc752c414beb37bd6da67c7216 on develop.

Coverage decreased (-0.06%) to 80.194% when pulling f4b1493d179157af2b5feebb9fc7c5c1d88b8822 on ft-user-configure-periodic-reports-151948280 into 58995fa7029ab7fc752c414beb37bd6da67c7216 on develop.

Coverage increased (+0.05%) to 80.295% when pulling f392b299c3132fa83d3c8c59c58cd0a859ee1caf on ft-user-configure-periodic-reports-151948280 into 58995fa7029ab7fc752c414beb37bd6da67c7216 on develop.

Coverage decreased (-0.1%) to 80.295% when pulling 72ac325279fe99990a722f7994b3194067944bc6 on ft-user-configure-periodic-reports-151948280 into 713e81c81602e3cc1b6f0041bd95b0f802a8726d on develop.

Coverage decreased (-0.02%) to 80.385% when pulling 02737facdde78f7825c0f0f6a98bfaef37a04832 on ft-user-configure-periodic-reports-151948280 into 713e81c81602e3cc1b6f0041bd95b0f802a8726d on develop.

Coverage decreased (-0.02%) to 80.508% when pulling 72de5c171524426e46a0b874b354f32e6e958504 on ft-user-configure-periodic-reports-151948280 into d67218f3c90159ad0a933c46a61376f593cf5433 on develop.


Fix firing ChangeListener on onGroundProperty() with setScaleX()

It seems like chaning X scale (it doesn't matter whether the change includes sign change such as from +1 to -1, it still apears, for instance, from +1 to +0.5) invokes a shift of PhysicsComponent's onGroundProperty.

Changing the scale of an entity affects its physics shape, which in turn requires the physics world to re-create the fixtures used in collision checks. When previous fixtures are removed, the onGround property becomes false since there is no fixture. Then when a new fixture is created, this triggers the onGround property to become true. So this is not a bug per se.
The above re-creation is necessary since even with just scaleX = -1 asymmetric hitboxes will need to be recalculated.
I don't have a clean solution to this yet, so I'll need some time to think. Any ideas?

For my problem (exclude triggering with mirroring but keep with jumping) I've used a boolean variable . Inside jump() method I've got something like if (!jumping) jumping = true; if the entity isOnGround() (its physics component with a ground sensor), and then add new ChangeListener with overridden method:
public void changed(ObservableValue<? extends Boolean> observable, Boolean wasOnGround, Boolean isOnGround) {
    if (!isOnGround && wasOnGround && jumping) {
        System.out.println("jumping...");
    } else if (isOnGround && !wasOnGround && jumping) {
        System.out.println("on ground");
    }
}

I mean, maybe it's possible to add kind of 'lock'. But I guess it's neither thread-safe nor applicable without a loss of performance due to checking and saving such a variable.


Parsing options in the 'for' example are not quoted.

The parsing options to the 'for' command are described in the text as "a quoted string that contains one or more keywords to specify different parsing options"  The third example on the page shows the keywords without surrounding them with quotes.
This example matches the example that is shown when doing "help for" in a command prompt.  The "help for" version shows the command with the required quotes and executes without errors.
Here is the example from the web page:
for /f eol=; tokens=2,3* delims=, %i in (myfile.txt) do @echo %i %j %k
and here is the example from the help text.
FOR /F "eol=; tokens=2,3* delims=, " %i in (myfile.txt) do @echo %i %j %k

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Ensure Main Command Exits with Non-Zero Status Code

This pull request resolves #123 by causing the main command to exit with status code 1 if there is any error when running the tasks.

Failure in the Test Sample job will be resolved by #125.


edit frecuency in Scramble menu

edit new frecuency in scramble menu

I don't think the BK4819 chip allows for any other scramble frequencies, it seems to allow only the ones listed.

But together with the radios built-in FSK modem, a rolling code scrambler can be written - short burst of FSK data at start of each TX syncs the scramblers in all radios on channel, just like motorola do.

fantastic !!!!!


Add hidden RPC command to activate features

Description:

Activate a protocol feature.

Note: Omni Core ignores activations from unauthorized sources.

Arguments:

fromaddress          (string, required) the address to send from
featureid            (number, required) the identifier of the feature to activate
block                (number, required) the activation block

Feature activations (and alerts) can be tested by whitelisting sources with -omnialertallowsender=address|any, for example in regtest mode or on testnet.

Looks good to me, OK to merge.


Allow the email mechanisms to work in the development environment

The change in src/config/environments/development.rb in this pull request allows you to use the password reset feature in the local development environment.

Coverage remained the same at 52.482% when pulling d049f831dd85199af18743d31c1ba0c9e9fce686 on jhsu802701:pr_password_reset_dev into a454e3dd0769a6aa2ad8f340021a603637097796 on minnestar:master.


Decide the best controls for the driver and operator

Right now, drive controls are pretty much finished. It is pretty much your standard holonomic field-relative drive with the ability to rotate A LITTLE while moving in a relatively straight line. Rotation while stationary is excellent.
Operator controls are a little different. Once we get a workable arm prototype, we can return to it with the ArmBasics branch and start testing different control schemes and subsystem layouts. Until then, progress on this issue is stagnant.

Driver:

Drive
low/med/high score

Operator:

Intake
Ground score
Extender
Unsheathe


Remove deprecation warnings for saveAsFile() and saveAsBitmap()

@burhanrashid52 Is there anything more you want me to add to this PR?


Kenny dev

merge to staging

This is a stable build; the work is from pair programming between Kenny, and Lewis
In this current build, Auth has been structured on the user routes, and the basic express API is functional.
The RESTful routes are connected to MongoDB. Users can be created by creating a POST request.


collapse and expand is not working int  develop batch

jquery version is 2.1.3.

i find this mistake, i caused.
because it cant reference the bootstrap-treeview.css file when i use bower to install the develop bootstrap-treeview.
need to override main proprety, include this file.


exception server

tomochainINFO 04:58:55 at ws/connection.go:48 in func1():Closing connection
tomochainINFO 04:58:55 at ws/notification.go:35 in func1():%v connections before unsubscription 2
tomochainINFO 04:58:55 at ws/notification.go:28 in func1():In unsubscription handler
tomochainINFO 04:58:55 at ws/notification.go:35 in func1():%v connections before unsubscription 2
panic: runtime error: slice bounds out of range
goroutine 2959 [running]:
github.com/tomochain/tomox-sdk/ws.NotificationSocketUnsubscribeHandler.func1(0xc0004bd2c0)
/home/ubuntu/tomox-sdk/ws/notification.go:38 +0x763
created by github.com/tomochain/tomox-sdk/ws.(*Client).closeConnection
/home/ubuntu/tomox-sdk/ws/client.go:56 +0x85
panic: runtime error: slice bounds out of range
goroutine 2960 [running]:
github.com/tomochain/tomox-sdk/ws.NotificationSocketUnsubscribeHandler.func1(0xc0004bd2c0)
/home/ubuntu/tomox-sdk/ws/notification.go:38 +0x763
created by github.com/tomochain/tomox-sdk/ws.(*Client).closeConnection
/home/ubuntu/tomox-sdk/ws/client.go:56 +0x85

fixed in e33dd32ce9ad94f9faed603c5a8e46f6c265b495 commit


Shortcut for focusing omnibox and changing tabs

Fixes #1498
Problem:
We want a quick way to focus the omnibox and toggle the selected tab
Fix:
Map the shortcut cmd+/ to the action for focusing the topmenu, and also capture that shortcut when the topmenu is focused to trigger an action for changing the selected tab.

sometimes selecting a scene on the canvas opens the dropdown automatically, is it remounting?

sometimes selecting a scene on the canvas opens the dropdown automatically, is it remounting?

 Hmmm I guess that means a different approach is needed in the classname-select


install qemu --env=std --cc=gcc-4.2 failed on mac os x 10.11.6-x86_64

If Homebrew was updated on Aug 10-11th 2016 and brew update always says Already up-to-date. you need to run: cd "$(brew --repo)" && git fetch && git reset --hard origin/master && brew update.
Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] If you're seeing permission errors tried running sudo chown -R $(whoami) $(brew --prefix)?

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
brew_config_log.txt
brew_doctor_log.txt
brew_install_qemu_log.txt

We don't support issues when using --env=std or custom --cc, sorry.


Added Bulgarian BIC codes

Hi. Firstly thanks for your work and for maintaining this repository, we are happily using this package at our company, but lately we ran into the limitation that it doesn't support Bulgarian BIC codes yet.
I've manually generated the json file containing Bulgarian mapping.
I did not find any official easily-parsable source, but did manage to find PDF file containing bank subjects by the Bulgarian national bank. Hence I've generated the json manually.

Hi @Krystofee, thank you very much for the PR. @mdomke kudos for merging, would you also be able to make a new release? Thank you so much.


Search functionality

Hi,
I couldn't find any search plugins available for editor :( I wrote my own function but there is an issue with if text splitted by html tag:
Example i need to find sample word but that's splitted by span tag
sample
Can you help or provide search functionality in your editor?
Thanks,
Regards.

ctrl+f (facepalm.jpeg)

so funny.... if you don't understand the question please don't spam.
I meant popup with find for this area with editor

I did not plan to dev something like that. I do not want that in core, but if anyone is interested in dev that as a plugin, I'm ok with that :)


Bug: Failed to pull dependent modules of main module from repo

When using VPM to pull the main module cacheModule.v from https://github.com/rajshadow/4-way-set-associative-cache-verilog/tree/master, the command doesn't get the roughly 30 submodule files required to instantiate the main module.

Command used: vpm include --local cacheModule.v rajshadow/4-way-set-associative-cache-verilog
Architechture: aarch64-apple-darwin

I see, thank you. Fixing it now.

Should be fixed btw, thank you.

Closing the issue if that's all!


⚠ Mitsubishi has degraded performance

In a65651d, Mitsubishi (https://mitsubishimotorsid-mobileapp.com/app/login) experienced degraded performance:

HTTP code: 200
Response time: 7037 ms

Resolved: Mitsubishi performance has improved in 8b7bada after 1 minute.


Compilation terminated

Hi,
I cannot pip install the package. I have

regex_3/_regex.c:48:10: fatal error: Python.h: No such file or directory
#include "Python.h"
^~~~~~~~~~
compilation terminated.
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

Might have something to do with CI failure for the new merge.

managed to install it by running

sudo apt-get install python3 python-dev python3-dev
build-essential libssl-dev libffi-dev
libxml2-dev libxslt1-dev zlib1g-dev
python-pip


Report an Issue does not successfully submit issues

Clicking submit on the report an issue page after filling fields produces error. Fixing this might involve backend.

can successfully report issues with emails, closed.

We need to reopen this issue, I just tried it and it's not working. @danielsjoo @lizyn @crchong1

https://user-images.githubusercontent.com/109312540/179077337-2f4b7f66-e92e-4555-972b-a2dde82244b0.mp4
Here's a video


Fix code scanning alert no. 1: SQL query built from user-controlled sources

Fixes https://github.com/jjmendezrodriguez/skills-introduction-to-codeql/security/code-scanning/1
To fix the problem, we need to use parameterized queries instead of directly concatenating user input into the SQL query strings. Parameterized queries ensure that user input is properly escaped and treated as data rather than executable code, thus preventing SQL injection attacks.

Replace the direct string concatenation in the SQL queries with parameterized queries.
Use placeholders (%s) in the SQL query strings and pass the user-provided values as parameters to the cursor.execute method.

Suggested fixes powered by Copilot Autofix. Review carefully before merging.

fix


Add drop shadow to search bar

Not focused & not expanded ---> normal shadow
Focused & expanded ---> a bit more shadow

Will see what I can do.


[Google Chrome] ...

Extension – Google Chrome
Author: @rgomezcasas
Description
Extension crashes when trying to search Chrome history

Error:
Error: ENOENT: no such file or directory, open '/Users/amit/Library/Application Support/Google/Chrome/Default/History'

Looks like it breaks because I have multiple Chrome profiles, so it fails to find the Default profile.
Raycast version
Version: 1.39.1

Having the same issue today. The temporary workaround for me is forcing Google Chrome to launch with a profile pointed to the directory /Users/.../Library/Application Support/Google/Chrome/Default/ with the command open -na 'Google Chrome' --args --profile-directory='Default' https://drive.google.com/. (Found this solution here.)Then, log in to your google account and enable syncing. Now the History file can be located by google-chrome extension. Make sure you make a backup of your original profile folder. (You can visit chrome://version/ in your browser and check 'Profile Path' to find the current path in used.)

https://github.com/raycast/extensions/blob/efba3ec1978c17317a96bcdcaa914f9153839e04/extensions/google-chrome/src/browserHistory.tsx#L33
Wishing that, instead of this line of code, we can select which profiles to be used. Mine is "Profile 1", not "Default".


fix e2e zk download failed

fix e2e zk download failed

t_ds_user table structure is error in sql/dolphinscheduler-postgre.sql

The problem of field state in table t_ds_user, I have submited the PR
https://github.com/apache/incubator-dolphinscheduler/pull/2710


Nag for deprecation of registering conventions

Fixes: #24232

Starting as draft, to find the repercussions in tests

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

This is looking good! I left a few remarks. There are 3 TODO-RC remove callers that should be taken care of.

Thanks, Paul! Wanted to get your feedback before actually removing that code. Will do as soon as the tests go greem.

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you. Click here to see all build failures.

ReadyForNightly build

@bot-gradle merge

The merge queue build has started. Click here to see all failures if any.


Partial fix

closing branch due to code being uniform across used machines

this request was opened by error, branch is personal on a fork that is not planned to merge with the origin project


Fix blocktransfer stall due to lost ACK

Signed-off-by: Leif Sandstrom<EMAIL_ADDRESS>Fix for #555

Tested that the last ACK of the blocktransfer can be lost and will be resent as 2.04
Tested that an ACK in the middle of the blocktransfer can be lost and will be resent as 2.31

Any chance to get this fix unit-tested?

"Integration testing" is the only realistic way (as I see it) to get decent test coverage with realistic effort. And we should refactor some of the blocktransfer code once we have to make it cleaner and easier to understand.
I plan to look into integration testing now, happy to discuss best way forward

Shoul I integrate this one ?

yes and yes!

Integrated in master (commit f4328b8f997ac7c8815f4f179695953cb193f5d3)


Added websql type definition

Many existing applications and new applications use the deprecated W3C WebSql specification (http://www.w3.org/TR/webdatabase/) for storing data in a client side database.
A number of existing tools such as chrome, safari, and the V8/WebKit bowser on Android, support WebSql and will most likely continue to support it into the future.
This commit contains a basic set of interface definitions to add WebSql functions to TypeScript projects targeting WebSql.
I ensured that the websql-test.ts file compiles without error.  This is my first DefinitelyTyped pull request, please let me know if any changes are needed to make this WebSql TypeScript definition conformant with DefinitelyTyped's standards.

@TeamworkGuy2 Thanks mate!


"ansible_processor_vcpus" reported as 'Undefined' on s390x RHEL7

Running playbooks on new Rhel7.7 system (s390x) which fell over in gmake task because they are reporting ansible_processor_vcpus as undefined.

TASK [gmake : Compile and install make 4.1 from source] ************************
fatal: [jsvt302.hursley.ibm.com]: FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: 'ansible_processor_vcpus' is undefined\n\nThe error appears to be in '/home/sej42282/gitibm/runtimes/infra-can/ansible/playbooks/AdoptOpenJDK_Unix_Playbook/roles/gmake/tasks/main.yml': line 40, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Compile and install make 4.1 from source\n  ^ here\n"}

/proc/cpuinfo contains:
# cat /proc/cpuinfo
vendor_id       : IBM/S390
# processors    : 2
bogomips per cpu: 21881.00
max thread id   : 0
features        : esan3 zarch stfle msa ldisp eimm dfp edat etf3eh highgprs te vx vxd vxe gs sie
facilities      : 0 1 2 3 4 6 7 8 9 10 12 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 30 31 32 33 34 35 36 37 38 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 57 58 59 60 73 74 75 76 77 80 81 82 128 129 130 131 133 134 135 146 147 156 168
cache0          : level=1 type=Data scope=Private size=128K line_size=256 associativity=8
cache1          : level=1 type=Instruction scope=Private size=128K line_size=256 associativity=8
cache2          : level=2 type=Data scope=Private size=4096K line_size=256 associativity=8
cache3          : level=2 type=Instruction scope=Private size=2048K line_size=256 associativity=8
cache4          : level=3 type=Unified scope=Shared size=131072K line_size=256 associativity=32
cache5          : level=4 type=Unified scope=Shared size=688128K line_size=256 associativity=42
processor 0: version = FF,  identification = 313F07,  machine = 3906
processor 1: version = FF,  identification = 313F07,  machine = 3906

Comparing with a working system - also RHEL 7.7, but x86, /proc/cpuinfo contains:
processor       : 0
vendor_id       : AuthenticAMD
cpu family      : 23
model           : 1
model name      : AMD EPYC Processor (with IBPB)
stepping        : 2
microcode       : 0x1000065
cpu MHz         : 2199.998
cache size      : 512 KB
physical id     : 0
siblings        : 1
core id         : 0
cpu cores       : 1
apicid          : 0
initial apicid  : 0
fpu             : yes
fpu_exception   : yes
cpuid level     : 13
wp              : yes
flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm art rep_good nopl extd_apicid eagerfpu pni pclmulqdq ssse3 fma cx16 sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm cmp_legacy svm cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw retpoline_amd ssbd ibpb vmmcall fsgsbase tsc_adjust bmi1 avx2 smep bmi2 rdseed adx smap clflushopt sha_ni xsaveopt xsavec xgetbv1 virt_ssbd arat
bogomips        : 4399.99
TLB size        : 1024 4K pages
clflush size    : 64
cache_alignment : 64
address sizes   : 40 bits physical, 48 bits virtual
power management:

(and similar repeated for each cpu).
There are a lot of instances of this variable being used, so for now I'm setting an override of ansible_processor_vcpus=1 for this system in my hosts file to get around the problem.

I saw a similar problem looking at the playbook for FreeBSD, where it had ansible_processor_cores undefined, and it required the ansible_processor_vcpus variable instead. Solutions that come to mind would be to change the role to have a task specific to Rhel7 on S390x, or to make a task in the ../Common/tasks/Redhat.yml file that sets vcpus to the same value as cores.

@Willsparker If you havent already implemented a solution, I think the following way might be the quickest way to do it
"{{ ansible_processor_vcpus | default(whatever you want as the default) }}"

If tested it roughly, with a simple debug msg command and it works; it outputs the default value if the variable is undefined. I haven't tested it in the gmake task

Theres also a task in the cmake role that i've just spotted which does something similar to what you have mentioned in your comment:
- name: Set ansible_processor_vcpus
  set_fact:
    ansible_processor_vcpus: "{{ ansible_processor_cores }}"
  when:
    - ansible_architecture == "s390x"
  tags: cmake

I'll have a look - if we go for the default value solution (i.e, 1), it would build an awful lot slower if it actually has potential for a lot more processors

You could try setting the default value to be the same value as cores, as suggested in your comment above, if that too isn't undefined

There were 14x files using ansible_processor_vcpus - I don't know if the problem is just associated with the s390x architecture though - it's the first one I've done.
find . -name "*.yml" -exec grep -l ansible_processor_vcpus {} \;
./roles/Common/tasks/CentOS.yml
./roles/Common/tasks/FreeBSD.yml
./roles/Common/tasks/SLES.yml
./roles/Debug/tasks/main.yml
./roles/GIT_Source/tasks/main.yml
./roles/Nagios_Plugins/tasks/nagios_FreeBSD.yml
./roles/Nagios_Plugins/tasks/nagios_RedHat.yml
./roles/Nagios_Plugins/tasks/nagios_SLES.yml
./roles/OpenSSL/tasks/main.yml
./roles/ccache/tasks/main.yml
./roles/cmake/tasks/main.yml
./roles/gcc_48/tasks/main.yml
./roles/gmake/tasks/main.yml
./roles/local_srcinstall/tasks/main.yml

I do remember having this issue with the new z15 RHEL marist machines, but it must have slipped my mind to open an issue

Okay, I'll put (what I think will be) a fix in. For the other architectures, I think we should handle those cases individually, as they come up.

Just FYI, the above PR will needed to be tested as I've not got access to an s390x machine, if someone wouldn't mind doing that...?

@Haroon-Khel If your machiens showed up the problem then we could use one of the s390x Marist machines to test this. If we remove /usr/local/bin/make from the machine and run the playbook that should trigger it to rebuild the role. If you want to do that give me a shout and I'll mark it offline in jenkins for a while

@sxa555 I dont think I need to use the marist machine. I've recreated the error on a s390x RHEL fyre box and am testing the pr on it

I too created an s390x Rhel Fyre ember, and it's completed successfully against @Willsparker 's changes.

I think https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1092#pullrequestreview-342539797 serves as a more succinct solution, but I think we should look to creating a separate role for situations like this; resolving variables that appear undefined


chore(main): release 0.3.1

:robot: I have created a release beep boop
0.3.1 (2024-11-18)
Build System

deps: bump tqdm from 4.64.1 to 4.66.3 (b4d9b4f)
deps: bump tqdm from 4.64.1 to 4.66.3 (ee877f1)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Danderson123/Amira/releases/tag/v0.3.1 :sunflower:


Support Databinding on carbon_cornerRadiusBottomEnd and the rest

Hi I wish to apply databinding on these field in xml but unfortunately unable to do so.
Build Error :
Cannot find a setter for <carbon.widget.ConstraintLayout app:carbon_cornerRadiusBottomEnd> that accepts parameter type 'float'
If a binding adapter provides the setter, check that the adapter is annotated correctly and that the parameter type matches.
<EMAIL_ADDRESS><EMAIL_ADDRESS>app:carbon_cornerRadiusTopEnd="@dimen/space"
app:carbon_cornerRadiusTopStart="@dimen/space"

Well, that's correct, there are no setters for these attributes. If you want, you can add an adapter yourself, but that wouldn't be cheap since corners are set using ShapeAppearanceModel and there's only one setter exposed to set an entire model at once.


w2grid.js: add scrollToColumn(field)

I've a grid where columns represents the calendar days, from a couple months
in the past to a couple months in the future.
I needed a way to position the grid at "today" column when the grid loads,
therefore I've developed the following small utility function to scroll the view
to a specific column programatically.
Feel free to merge if you find that function useful / correct.

Seems correct. Thanks for your pull request.


Fixed bug in result Dockerfile

I ran the command:
docker-compose up
and it didn't worked because there was an error in the Dockerfile of result.
Therefore, I fixed it!

Can you give us the error you're seeing and your environment setup? I've not seen this issue before, so I'm assuming it's due to a unique environment.

I've built master on macOS and Windows 10 Pro (both with Docker Desktop 19.03.1) and don't get this error.

OK, my research shows that this error is caused by a long-standing on-and-off-again npm+alpine bug.  npm has fixed it in 6.11, but we'll need to wait for that to be default in upstream official images, so choices are:

Accept this PR workaround that slightly lowers the security of npm
Install the newer npm in this image ourselves (which to get it installed, may still require this workaround)
Use debian slim version of this image, which I've tested and works. It also prevents multiple other node/npm issues I've seen over the years with using node:alpine.

Since this is an example app that we would rather have minimal future issues with, I'm recommending the 3rd option and have added that to my result update PR #140
Once that's accepted we can close this issue. Thanks for the heads up @FlyingBird95

Verifying that PR #140 fixed this issue and we're OK to close this one? @FlyingBird95 @BretFisher


Unable to launch tasks

Issue Type: Bug

Ctrl+P Tasks: Run Task
Select a task.
It doesn't run.

Tried with and without extensions loaded. Still persists after rebooting the PC.

It works with your tasks.json. My tasks.json is quite long so I'll try to cut it down before sharing. It needs cleaning because we've just moved from ClearCase to git.

I've removed some of the tasks and some of them are working, but not all of them. I'll do further analysis when I have some time.

Any example of one that doesn't work would probably help.

Sorry been busy. I'll try to narrow it down.

The following doesn't work:
      {
         "label": "four",
         "command": "",
         "args": [ "sleep 13" ],
         "type": "shell",
         "problemMatcher": [],
      }

I have a lot of tasks that following that pattern where I have to run shell commands. These ones used to work and now failing. I think it started with the patch release 1.36.1. I also have ones that launch external processes, which seem to be fine.

I've reinstalled version 1.35.1 and its working again.

Thank you for tracking down one of the tasks. The issue is that you don't have a command. It looks like that used to be implicitly supported, but the idea of a task with arguments and no command is a little weird. We did used to support it though.

I pushed a fix, which you should be able to get in the next insiders release. Inisiders installs side by side with stable, so you can have both if you want to get the fix sooner.

I installed 1.35.1 and before I managed to do much it updated itself 1.36.1 because I forgot to turn of the auto update. Now on "1.36.1" I the following works:
      {
         "label": "four",
         "command": "sleep",
         "args": [ "13" ],
         "type": "shell",
         "problemMatcher": [],
      }

I'll try to update my more complex tasks to do that. I have a lot of tasks that run long complex commands in cmd. Something like "(scons parm1 parm2 parm3 parm4 || build_fail.bat) && (scons another1 another2 another3 && build_fail.bat) && build_success.bat". I'm not sure how I would split that between the command and args. I could pull it out into a separate batch file, but I don't want to do that because it is more files to maintain when I change the build parameters that I do frequently.

I installed 1.35.1 and before I managed to do much it updated itself 1.36.1 because I forgot to turn of the auto update. Now on "1.36.1" I the following works:
{
"label": "four",
"command": "sleep",
"args": [ "13" ],
"type": "shell",
"problemMatcher": [],
}
I'll try to update my more complex tasks to do that. I have a lot of tasks that run long complex commands in cmd. Something like "(scons parm1 parm2 parm3 parm4 || build_fail.bat) && (scons another1 another2 another3 && build_fail.bat) && build_success.bat". I'm not sure how I would split that between the command and args. I could pull it out into a separate batch file, but I don't want to do that because it is more files to maintain when I change the build parameters that I do frequently.
Update Thanks for fixing it. I've found a work around, which is put the first item (e.g "(scons") in the command and leaving the rest in the args. (only tested one task which I needed to run). I think I'll use the workaround as it is less disruptive than changing to insiders.

A little change as a developer, will bring endless pain to producers, when I catch the bug, it's so painful😢Thanks for fixing it

Can you clarify the expected behavior?
On linux I get
/bin/bash: sleep 13: command not found
The terminal process terminated with exit code: 127

On windows, there is no error but the task terminates immediate and does not seem to be actually sleeping.

There is no sleep command on Windows, so you can test on Windows with:
		{
			"label": "My Task",
			"type": "shell",
			"command": "",
			"args": ["echo", "hello"],
			"problemMatcher": []
		}

On linux, the way we do escaping means that for the sleep command you should test with:
		{
			"label": "My Task",
			"type": "shell",
			"command": "",
			"args": ["sleep", "3"],
			"problemMatcher": []
		}

Wrote this yesterday, but it was late so forgot to press "Comment". Looks like you've verified it.
I've not tried the insider build yet. I'm using the workaround instead. Not tried on Linux (and not easy for me to do so). On my Windows 10 machine, I have something installed that is providing the command "sleep", so it sleeps for 13 seconds. That is with:
      {
         "label": "four",
         "command": "sleep",
         "args": [ "13" ],
         "type": "shell",
         "problemMatcher": [],
      }

With:
  {
      "label": "four",
      "command": "",
      "args": [ "sleep", "13" ],
      "type": "shell",
      "problemMatcher": [],
   }

Nothing appears to happen. That pattern used work, and I presuming will work in the next release.


feat: Add accordion override

Will fix tests in refactor, going from testing-library/react -> cypress. Cannot adequately test all required style changes with testing-library/react.


Improve documentation for Ray and OrientedPoint

The classes Ray (Ray2d and Ray3d) and DirectedPoint (DirectedPoint2d and DirectedPoint3d) seem to store the exact same information, yet the two classes are very different. Explain in the Javadoc, and in the documentation why both classes exist.

In 2D the difference is the naming convention of the rotZ (vs phi). Class OrientedPoint2d has been removed.


node: add explicit 409 code for version conflicts

Follow up to #85.
I'm still unsure about using the same UUID for all resources in the example Location headers.
And I'd prefer the URLs to end without a trailing slash in all cases.
But for the moment, I've gone for consistency with the Query API examples.

Now updated to match all UUIDs in Location header examples to the relevant example response data, and removed trailing slashes as this demonstrates what a client must use to make subsequent DELETE requests on the Registration API.


Add support for detecting and reading Aztec Runes

This pull adds support for Aztec Runes, as described in Annex A of ISO/IEC 24778:2008.
Runes are identical to the Core Symbol of a Compact aztec code, but with every other bit of the data message inverted to distinguish them from regular Mode Messages. In other words, all we need to do to detect a rune is (a) fail to decode a regular Mode Message and then (b) XOR each 4-bit word in the data message with 0b1010 and try again. If it decodes this time, then we've got a rune.
Let me know if there's any changes you'd like to see!

all done! should probably flag up that while this patch does also include support for encoding runes, it's mainly done to add it to the battery of Encode/Decode tests, and I haven't really done anything to integrate it into the rest of the project. But it's there to build off for a future patch. (Which I will probably also end up contributing, lol.)
(and it's a good thing I did, because I would never have found the issue with runes 68 and 223 otherwise!)

Thanks a lot. About the writer part: I did not look too hard at that, e.g. I did not suggest to use the appropriate 'str -> int' conversion (inverse of the ToString(runeValue(), 3) call). Please note that the writing code is going to be deprecated/removed in the long run and replaced by a libzint backend. See #332 et al. Bottom line: there is not much point putting effort into this.
That said, it is currently not possible to specify the use a Rune instead of a standard Aztec symbol with the new interface.

Bottom line: there is not much point putting effort into this.

Noted! I will put zero more effort into this. Cheers!


二维码识别，在连续识别的情况下，如何设置识别时间隔间？

在连续识别二维码的情况下，我想要二次识别设置下识别间隔时间3秒左右，现在好像是不断的连续识别，调用remoteView.onstop()方法,j来停止识别，让后再调用onstart()方法开启识别，这个界面就不太友好了!不知道官方有设置这个连续识别间隔时间不？

Hello,
According to my research the continuous scan is collecting the results while the camera is in use thus scans the frame without intervals.
In order to wait for a specific time frame while scanning a barcode, you can implement your own camera or use a 3rd party dependency that offers camera as a widget and process the camera output using Huawei Scan's decodeWithBitmap API. For detailed information about decodeWithBitmap API, please check this link.


XD-3283: Add transform & filter, with groovy variations

now replaced by PR https://github.com/spring-cloud/spring-cloud-stream-modules/pull/19


Laravel 5.1 Support

Getting issues with illuminate/support. I believe it is the "illuminate/support" : "5.0.*", require statement that makes this unable to install on Laravel 5.1, which needs laravel/support >= 5.1...?

I deleted my vendor folder and ran composer install (I had originally just updated my composer.json with Laravel 5.1 and ran composer update).
Running a fresh composer install seems to have worked (I think because of Laravel's replaces in its composer.json...?)

Er, nevermind, I deleted that last comment. I forgot to delete my composer.lock. Still haven't issues installing this alongside Laravel 5.1.


Rebuild Viewer on top of the jupyter widgets lib

http://ipywidgets.readthedocs.io/en/latest/examples/Custom Widget - Hello World.html

Done


add ssh command to print device info

This PR adds a new ssh command device-info that responds with the name of the tun device, and the cidr of the tun. Example output:
name=tun0 cidr=<IP_ADDRESS>/24

I need the tun device name because I would like to monitor the network device using other programs, and since the tun device name is dynamic, querying nebula seems like the best way to get it. I actually don't need the cidr, but since its available on the Device interface it seems easy enough to print as well.
If this seems reasonable I can add a json flag like the other print-xxx commands. Also the name of the ssh command is negotiable.

Thanks for the PR! The name of the command seems fine and I believe the a json flag would be helpful if you don't mind adding it.

:+1: ok will do soon

@kazzmir looks good, mind signing the latest CLA?

Sure, where do I go to sign it? Its suprisingly hard to find. Maybe the main readme should have a link to the cla.

That is a wonderful question. A fresh commit might trigger the bot to prompt for it, I am searching for another solution in the mean time.


[WIP] Add propagated inputs to the profile

Testing out solutions to #601.
List of propagated inputs we've found:

depsHostHostPropagated
depsBuildBuildPropagated
depsBuildTargetPropagated
depsTargetTargetPropagated
propagatedNativeBuildInputs or depsBuildHostPropagated
propagatedBuildInputs or depsHostTargetPropagated

This is fixed now by removing naked shell


[9.x] Automatically resolve route parameters into view data

Recently, I attempted to register the following route:
Route::view('event/{event}/sign-in', 'event.sign-in');

While I didn't expect this to work, the magic within Laravel has surprised me numerous times.
Ultimately, I had to write:
Route::get('event/{event}/sign-in', function (\App\Models\Event $event) {
    return view('event.sign-in', ['event' => $event]);
});

However, I think it'd be pretty neat if Laravel had this magic. So that's what this PR adds - resolving route parameters and automatically injecting them into the view data.
How it works
Any explicit route parameter bindings are resolved like any other route. Implicit model bindings are guessed using a fallback similar to how model factories are guessed.
If view data with these names is not already defined, the parameter will be automatically injected into the view data. Said another way, values passed in the $data argument take precedence over parameters.

Note: this is targeted for Laravel 9 due to the change in default behavior of Route::view(). To be compatible with Laravel 8 or to make the behavior opt-in, a chained method could be used. For example, Route::view()->withParameterBinding();

Personally I think I'll hold off on this. I don't love that we have to reach out to the global app helper from deep in the routing layer to try to guess the class names, etc.
Can kinda already be a one-liner like this... 😅
Route::get('/event/{event}/sign-in', fn (Event $event) => view('event.sign-in', compact('event')));

Sure. With Laravel 9's requirement of PHP 8 you could definitely map them explicitly using a type-hinted, short closure.
I thought the implicit mapping was a nice bit of automagic that would work for most apps. However, completely understand there's probably a better implementation.


Availability is false for vocab.getty.edu

http://sparqles.ai.wu.ac.at/availability shows
http://vocab.getty.edu/sparql (2)   0%  0%
I know for a fact this endpoint is up all the time.
Can you provide more info when it was last checked and what was the result?

We've started debugging this issue. Our logs seem to be throwing an error for the http://vocab.getty.edu/sparql endpoint:
12-09-15 14:24:38           ATask:114  WARN - failed SELECT query for http://vocab.getty.edu/sparql, EXC@0#349> QueryException HTTP msg:null cause:null
Obviously there's something wrong on our end because of the "null" messages. We'll keep you updated once this is fixed.

Hi @VladimirAlexiev. Sorry for the delay but it seems like your endpoint isn't read correctly by standard SPARQL tools:
$ java -cp "lib/*" arq.rsparql --service "http://dbpedia.org/sparql" 'SELECT ?s WHERE {?s ?p ?o} LIMIT 1'
----------------------------------------------------------------
| s                                                            |
================================================================
| <http://www.openlinksw.com/virtrdf-data-formats#default-iid> |
----------------------------------------------------------------
$ java -cp "lib/*" arq.rsparql --service "http://vocab.getty.edu/sparql" 'SELECT ?s WHERE {?s ?p ?o} LIMIT 1'
HTTP Exeception
HttpException: 404
	at org.apache.jena.sparql.engine.http.HttpQuery.execGet(HttpQuery.java:346)
	at org.apache.jena.sparql.engine.http.HttpQuery.exec(HttpQuery.java:292)
	at org.apache.jena.sparql.engine.http.QueryEngineHTTP.execResultSetInner(QueryEngineHTTP.java:345)
	at org.apache.jena.sparql.engine.http.QueryEngineHTTP.execSelect(QueryEngineHTTP.java:337)
	at org.apache.jena.sparql.util.QueryExecUtils.doSelectQuery(QueryExecUtils.java:196)
	at org.apache.jena.sparql.util.QueryExecUtils.executeQuery(QueryExecUtils.java:78)
	at arq.rsparql.exec(rsparql.java:76)
	at jena.cmd.CmdMain.mainMethod(CmdMain.java:93)
	at jena.cmd.CmdMain.mainRun(CmdMain.java:58)
	at jena.cmd.CmdMain.mainRun(CmdMain.java:45)
	at arq.rsparql.main(rsparql.java:44)

As you can see, in the first example we run a test query against DBpedia. Running the same query against your endpoint leads to an error. Let us know if you can debug.

Thanks for the report!
I think that arq.rsparql doesn't send an Accept header. Without an Accept header, we return HTML and maybe rsparql fails on that. This returns results in XML:
curl -Haccept:application/sparql-results+xml "http://vocab.getty.edu/sparql?query=SELECT+%3Fs+WHERE+%7B%3Fs+%3Fp+%3Fo%7D+LIMIT+1"
In contrast, http://dbpedia.org/sparql has a drop-down "Results Format" with default value HTML. But if you send the query without an Accept header, it returns application/sparql-results+xml.
However, I tried this and it still fails with a similar exception:
rsparql --results xml --service "http://vocab.getty.edu/sparql" "SELECT ?s WHERE {?s ?p ?o} LIMIT 1"
We'll look into it.

Re the last command: I'm on Windows, so the syntax is slightly different.
A rsparql cheat sheet is at http://richard.cyganiak.de/blog/wp-content/uploads/2013/09/jena-sparql-cli-v1.pdf, that's where I got the --results xml  param

Now rsparql obtains 1 result as expected:
<?xml version="1.0"?>
<sparql xmlns="http://www.w3.org/2005/sparql-results#">
  <head>
    <variable name="s"/>
  </head>
  <results>
    <result>
      <binding name="s">
        <uri>http://www.w3.org/1999/02/22-rdf-syntax-ns#type</uri>
      </binding>
    </result>
  </results>
</sparql>

The Availabiliy shows 100%, but the query reports are still failing. Should we close this?


canvas without width/height attributes -> off-scale drawing

I made a pretty simple signature_pad example:
http://hujhax.austinimprov.com/code/SimpleSig/
Whenever I draw on it, it draws about 2x out of scale, with the new lines coming in out-of-sync with the cursor position.
I assume this is to do with not explicitly setting width and height attributes on the canvas -- I can fix it by adding these lines to the constructor:
    canvas.width = canvas.offsetWidth;
    canvas.height = canvas.offsetHeight;

... but I figured it was worth checking in about this, since  the "draw 2x out of scale" thing doesn't seem like intended behavior, right?

Please check out the demo page and its source code for an example how to set size of the canvas.

Excellent -- I tried adding the resizeCanvas() call to that sample, and it worked correctly.  Thanks!
Q: are there cases where a signature_pad user would ٭not٭ want this resizing behavior?

I have no idea :) It's not part of the library, because some users might want to handle it differently.

Fair enough :) -- thanks for the info!


Add start and stop methods for when code isn't conveniently block wrappable

stackprof supports using start / stop in addition to the block usage. This is helpful for when things aren't cleanly block wrapped.
cc @baweaver who requested it

This would help profiling Rails boot too

@nate-at-gusto how are you thinking you'd use it in that case? I've been using the CLI to do this (with rbspy under the hood)

Add Singed.start to boot.rb, bin/rails runner 'Singed.stop'

I can see the value in this now. I have been using memory_profiler recently, and have used things like at_exit hooks to stop and generate the report. I could see doing the same with singed.


Update README-ja.adoc

as #138

Hi @hinaloe and thanks for the PR. Should I merge #138 or #140? Or should I merge incrementally?

both! (#138 is on master, #140 is on development)

The best process IHMO, i to make PR on development, then we merge and report on gh-pages and master, wdyt?

ACK @anthonny
I'll also make changes on /development as well for any docs changes I do.

@hinaloe I'll merge the #140 PR as this one is on /development. Thanks again for your work on ja-JP translations.


x/tools/go/ssa: the incorrect SSA code was generated

Go version
go1.23
Output of go env in your module/workspace:
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/Users/chai/Library/Caches/go-build'
GOENV='/Users/chai/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/chai/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/chai/go'
GOPRIVATE=''
GOPROXY='https://goproxy.cn,direct'
GOROOT='/usr/local/go'
GOSUMDB='off'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/usr/local/go/pkg/tool/darwin_amd64'
GOVCS=''
GOVERSION='go1.23.1'
GODEBUG=''
GOTELEMETRY='local'
GOTELEMETRYDIR='/Users/chai/Library/Application Support/go/telemetry'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/dev/null'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/d0/q0b3c09s2_nb6sjjhbnf0t1h0000gq/T/go-build2141529561=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?
i use x/tools/go/ssa to print ssa text:
https://go.dev/play/p/dxj4fTu2M5G
package main

import (
	"go/ast"
	"go/parser"
	"go/token"
	"go/types"
	"log"
	"os"

	"golang.org/x/tools/go/ssa"
)

const src = `
package main

func main() {
	root = &mapNode{}
	insert()
	println(root)
}

var root *mapNode

type mapNode struct {
	Child *mapNode
}

func insert() {
	var x *mapNode = root
	var y *mapNode = x
	for x != nil {
		y = x
		println(y)
		x = x.Child
	}
	println(y)
}
`

func main() {
	fset := token.NewFileSet()
	f, err := parser.ParseFile(fset, "hello.go", src, parser.AllErrors)
	if err != nil {
		log.Fatal(err)
	}

	info := &types.Info{
		Types:      make(map[ast.Expr]types.TypeAndValue),
		Defs:       make(map[*ast.Ident]types.Object),
		Uses:       make(map[*ast.Ident]types.Object),
		Implicits:  make(map[ast.Node]types.Object),
		Selections: make(map[*ast.SelectorExpr]*types.Selection),
		Scopes:     make(map[ast.Node]*types.Scope),
	}

	conf := types.Config{Importer: nil}
	pkg, err := conf.Check("hello.go", fset, []*ast.File{f}, info)
	if err != nil {
		log.Fatal(err)
	}

	var ssaProg = ssa.NewProgram(fset, ssa.SanityCheckFunctions)
	var ssaPkg = ssaProg.CreatePackage(pkg, []*ast.File{f}, info, true)

	ssaPkg.Build()

	ssaPkg.WriteTo(os.Stdout)
	ssaPkg.Func("insert").WriteTo(os.Stdout)
}

What did you see happen?
the insert func ssa:
# Name: hello.go.insert
# Package: hello.go
# Location: hello.go:16:6
func insert():
0:                                                                entry P:0 S:1
	t0 = *root                                                     *mapNode
	jump 3
1:                                                             for.body P:1 S:1
	t1 = println(t5)                                                     ()
	t2 = &t5.Child [#0]                                           **mapNode
	t3 = *t2                                                       *mapNode
	jump 3
2:                                                             for.done P:1 S:0
	t4 = println(t6)                                                     ()
	return
3:                                                             for.loop P:2 S:2
	t5 = phi [0: t0, 1: t3] #x                                     *mapNode
	t6 = phi [0: t0, 1: t5] #y                                     *mapNode
	t7 = t5 != nil:*mapNode                                            bool
	if t7 goto 1 else 2

What did you expect to see?
i guess the ssa lose the y = x in for loop. the test code should print(root) twice.
t1 = *root
jump 3

3:
t6 = t1 = *root
t7 = t1 = *toot
t8 = t6 != nil = true
if t8 goto 1 else 2 = goto 1

1:
println(t6) = println(*root)
t3 = &t6.Child = root.Child
t4 = *t3 = t6.Child = root.Child
jump 3

3:
t6 = t4 = t6.Child = root.Child
t7 = t6 = root.Child
t8 = t6 != nil = root.Child != nil = false
if t8 goto 1 else 2 = goto 2

2:
println(t7) = println(root.Child)
return

but the ssa print(root) and print(root.Child).

Related Issues and Documentation

x/tools/go/ssa/interp: TestGorootTest persistently failing on plan9 #60226 (closed)
x/tools/go/ssa: panic: cannot convert interface #62085 (closed)
x/tools/go/ssa: panic: no ssa.Value for function argument #33279
x/tools/go/ssa/builder.go: panic("nil underlyin") while executing *ssa.Program.Build() #68195 (closed)
x/tools/go/ssa: multiple assign ssa.FieldAddr instr order error #55086 (closed)
go/ssa: crash during SSA construction #9569 (closed)
cmd/compile: GOSSAFUNC=main open ssa.html: permission denied #33278 (closed)
cmd/compile/internal/ssa: incorrect deadcode elim #47594 (closed)
x/tools/go/ssa: function.Origin() panics with "index out of range" #59427 (closed)
x/tools/go/ssa: addNamedLocal panic: runtime error: invalid memory address or nil pointer dereference #58600

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Just as the issue9097 I submitted 10 years ago was ignored, and then when the official team proposed a similar issue45624, there was a chorus of acclaim. Once again, from the reply, I sensed the arrogance of some people in the development team: Since you don't understand SSA, don't spout nonsense. cc @rsc @fkarakas
I hope the development team can treat community feedback with mutual respect. This is my newly added test result.
By comparing “go run” and “x/tools/go/ssa/interp”, I still found differences when executing the following:
$ go run main.go
main begin
y.v0: 0xc000015330
y.v1: 0xc000015330
y.v2: <nil>
<nil>
main end
$ go run _hello.go
package command-line-arguments: no Go files in /Users/chai/go/src/github.com/wa-lang/wa/_build/a44
$ go run ./hello/hello.go
main begin
y.v0: 0xc000104000
y.v1: 0xc000104000
y.v2: 0xc000104000
0xc000104000
main end
$

As a gopher who doesn't understand SSA, I am rather confused as to why the above two execution results are different?
Complete code. (play: https://go.dev/play/p/ZwNMYkH4nTd )：
package main

import (
	"fmt"
	"go/ast"
	"go/parser"
	"go/token"
	"go/types"

	"golang.org/x/tools/go/ssa"
	"golang.org/x/tools/go/ssa/interp"
)

func main() {
	prog := NewProgram(map[string]string{
		"main": `
			package main

			func main() {
				println("main begin")
				root = &mapNode{}
				insert()
				println("main end")
			}

			var root *mapNode

			type mapNode struct {
				Child *mapNode
			}

			func insert() {
				var x *mapNode = root
				var y *mapNode = x
				println("y.v0:", y)
				for x != nil {
					y = x
					println("y.v1:", y)
					x = x.Child
				}
				println("y.v2:", y)
				println(y)
			}
		`,
		"runtime": `
			package runtime

			type errorString string

			func (e errorString) RuntimeError() {}
			func (e errorString) Error() string { return "runtime error: " + string(e) }

			type Error interface {
				error
				RuntimeError()
			}
		`,
	})

	prog.LoadPackage("main")
	prog.LoadPackage("runtime")

	var ssaProg = ssa.NewProgram(prog.fset, ssa.SanityCheckFunctions)
	var ssaMainPkg *ssa.Package

	for name, pkg := range prog.pkgs {
		ssaPkg := ssaProg.CreatePackage(pkg, []*ast.File{prog.ast[name]}, prog.infos[name], true)
		if name == "main" {
			ssaMainPkg = ssaPkg
		}
	}
	ssaProg.Build()

	exitCode := interp.Interpret(
		ssaMainPkg, 0, &types.StdSizes{8, 8},
		"main", []string{},
	)
	if exitCode != 0 {
		fmt.Println("exitCode:", exitCode)
	}
}

/*
$ go run ./hello/hello.go
main begin
y.v0: 0xc000104000
y.v1: 0xc000104000
y.v2: 0xc000104000
0xc000104000
main end

$ go run main.go
main begin
y.v0: 0xc000015330
y.v1: 0xc000015330
y.v2: <nil>
<nil>
main end
*/

type Program struct {
	fs    map[string]string
	ast   map[string]*ast.File
	pkgs  map[string]*types.Package
	infos map[string]*types.Info
	fset  *token.FileSet
}

func NewProgram(fs map[string]string) *Program {
	return &Program{
		fs:    fs,
		ast:   make(map[string]*ast.File),
		pkgs:  make(map[string]*types.Package),
		infos: make(map[string]*types.Info),
		fset:  token.NewFileSet(),
	}
}

func (p *Program) LoadPackage(path string) (pkg *types.Package, f *ast.File, err error) {
	if pkg, ok := p.pkgs[path]; ok {
		return pkg, p.ast[path], nil
	}

	f, err = parser.ParseFile(p.fset, path, p.fs[path], parser.AllErrors)
	if err != nil {
		return nil, nil, err
	}

	info := &types.Info{
		Types:      make(map[ast.Expr]types.TypeAndValue),
		Defs:       make(map[*ast.Ident]types.Object),
		Uses:       make(map[*ast.Ident]types.Object),
		Implicits:  make(map[ast.Node]types.Object),
		Selections: make(map[*ast.SelectorExpr]*types.Selection),
		Scopes:     make(map[ast.Node]*types.Scope),
	}

	conf := types.Config{Importer: p}
	pkg, err = conf.Check(path, p.fset, []*ast.File{f}, info)
	if err != nil {
		return nil, nil, err
	}

	p.ast[path] = f
	p.pkgs[path] = pkg
	p.infos[path] = info
	return pkg, f, nil
}

func (p *Program) Import(path string) (*types.Package, error) {
	if pkg, ok := p.pkgs[path]; ok {
		return pkg, nil
	}
	pkg, _, err := p.LoadPackage(path)
	return pkg, err
}

The variable y is not lost. It is translated to a phi node, that is, a node whose value depends on which incoming edge control came through to reach the current block:
	t6 = phi [0: t0, 1: t5] #y

The comment tells you that this value corresponds to y, and the phi notation tells you that y's value is either t0 (*root) when coming from block zero (i.e. on the first iteration), or t5 (x) on all later iterations.
Unlike traditional intermediate representations, there are no local assignments in SSA, so you can't "update" y. The representation expresses the conditionality in a stateless form. This is the nature of SSA. I suggest you read some introductory material on the topic, such as the Wikipedia article.

We know how phi works, this problem is not about losing y, it's about incorrect instructions generated by for.
According to SSA code above, on all iterations after for,  t5 will always been assigned to t6(y), and t5 always comes from x.child, that means when block 2 is reached, t6 must be nil, obviously it's not right.
Using ssa/interp.Interpret() to run the SSA code above shows exactly the same error.

Apologies for my previous comment. I mistakenly thought I was addressing someone unfamiliar with SSA basics, though even in that were so I now see how the tone of my comment came across as condescending.
The subtlety in this case is that the two phis are interdependent:
        t5 = phi(t0, t3) # x
        t6 = phi(t0, t5) # y

The SSA builder acts as if all consecutive phis are executed in parallel, which is consistent with the interpretation that phis are really virtual instructions at the end of each predecessor block. Annotating the purpose of each value below shows that the final print(t6) is correct: it prints the value of y on entry to the final (false) loop test:
entry:
        t0 = *root     // t0 is x or y before loop
        jump test

body:
        println(t5)    // t5 is x at loop entry
	t3 = &t5.Child // t3 is x after loop
	jump test

test:
        t5 = phi(t0, t3) // t5 is x at loop entry
        t6 = phi(t0, t5) // t6 is y at loop entry
	if t5 != nil goto body else done

done:
        println(t6)
	return

However, the ssa/interp logic gets this case wrong because it shouldn't update the environment for t5 and t6 one instruction at a time; it must wait until both phis have been evaluated. So I think this is a problem in the interpreter, not the builder, though perhaps the documentation should clarify the parallel nature of phis.

...
However, the ssa/interp logic gets this case wrong because it shouldn't update the environment for t5 and t6 one instruction at a time; it must wait until both phis have been evaluated. So I think this is a problem in the interpreter, not the builder, though perhaps the documentation should clarify the parallel nature of phis.

That explains all.
Thanks!

For how to efficiently replace phi nodes with explicit copies while preserving parallel semantics, see the "swap problem" in Briggs et al's Practical Improvements to the Construction and Destruction of Static Single Assignment Form.

Thanks for reporting the bug, with a nice reproducible test case.


Support Vite projects in debug:cpu command

Needs #2100 first
This PR adds supports for Vite projects in the h2 debug cpu command, and makes the command also available in diff examples with the --diff example flag.
It's easier to check the code commit by commit.
To tophat:

Check that we can still run it for classic projects: in examples/classic-remix, run h2 debug cpu --diff.
Check that it works in Vite projects: in templates/skeleton, run h2 debug cpu.

You can add the following code, for example, to the top level of server.ts and change the number or 0s in the times variable:
let times = 100000;
while (--times) {
  // @ts-ignore
  globalThis.asdf = 2 + 6;
}

It appears to work, the total start-up time seems correct. But in all honesty, I can't get the cpuprofile to actually be helpful


Extend graphql-playground-html options type

This change is required to be able to provide params like cdnUrl
This allows a fix to #1070

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


0.5.0 as image artefacts (rtsp/live)

Nice work on this video player implementation. I was just about to update to 0.5 but ran into a problem:
The video playback is distorted and shows artefacts that vary depending on the stream used. Screenshots are below.
Pipeline used: uridecodebin uri={} ! videoconvert ! videoscale ! videorate ! appsink name=iced_video caps=video/x-raw,format=RGBA,pixel-aspect-ratio=1/1
Test rtsp stream: rtsp://<IP_ADDRESS>:554/iloveyou

v0.5.0

v0.4.0

Hi, thanks!
0.5.0 switched to hardware-accelerated YUV conversion so iced_video_player now expects NV12 frame data.
Please try format=NV12 in the pipeline instead.

Awesome, thank you for your quick reply, this solves my issue


Popover animations

This branch adds opacity animations specifically for popovers and a few other fades.
The goal is to eventually ship css that can be used to animate popover web components with code like:
       menu: cx(
                'popover-container',
                'popover-container--menu',
                'anim-transition--opacity--s',
                {
                    'anim-opacity--0': !isActive,
                    'anim-opacity--1': isActive,
                    'popover-container--horizontal-left': (align === 'left'),
                    'popover-container--horizontal-right': (align === 'right')
                }
            )
        };

        return (
            <div>
                    ...
                    <ul
                        className={classNames.menu}
                        role='menu'
                        aria-hidden={!isActive}
                    >
                        {this.renderMenuItems()}
                    </ul>
                </nav>
            </div>

updates / additions:

keyframe and transition animations. transitions will be used for animations triggered by user feedback - like triggering a popover appear. the keyframe fade animations could be for page load animations.
added duration constants - still working out the values based on research, look & feel
built out javascript for show and hide (not sure if these would be usefule in react projects or just in meetup classic javascript)
tests for the javascript using jquery-jasmine to load css fixtures
removed bourbon @include and added autoprefixer for vendor prefixes

feedback needed on:

naming conventions in css (eg abbreviating --short to --s)
javascript naming conventions and organization
jasmine tests
documentation https://meetup.github.io/swarm-animation/

We should talk a little more about duration and easing and come up with some guidelines, but here's how I'm thinking about naming conventions...

Some more examples could be:
.trans-fadeTo75--slow
.trans-fadeInFromLeft
.trans-slideRight
.anim-spin
.anim-growDown
.anim-shrinkUp```

Curious to hear how @akdetrick feels about classes like .opacity--0 and .opacity--1 living in here to support animations.
Should we namespace those or do anything to make it clear that they've been added for transition purposes?

opacity--0 and opacity--1 seem like sensible things to live in this repo

I updated the naming to @mperrotti's suggestions. Hopefully we can keep these, but there may need to be changes if the React naming rules/suffixes make these sound odd (eg. trans-fadeIn-enter).

No additional feedback - seems like this is in a good place to merge and grow the lib


Login into Azure not working

Storage Explorer Version: 1.4.4
Platform/OS Version: Windows 10 (1809)
Architecture: ia32
Regression From: Was never working
Steps to Reproduce:
Login to Azure
Expected Experience:
Login into the Azure Storage Explorer with my credentials
Actual Experience:
Login Windows just shows white with "Browser-SSO" in the title bar and never logs me in.
Were using ADFS with Windows Authentication.

I am having a similar problem on Mac OS. I just get a pop-up saying I am being redirected to my organization's sign-in page, and then I get a blank window and nothing happens.

@Theragus, can you please upgrade to 1.6.x and give this another try?  Also, if you use proxy, make sure you first config the proxy.
@juve can you try with the steps outlined in #126?

@jinglouMSFT thank you for the reference. Those steps did not work for me. It does ask for access to the keychain when I start storage explorer, but I still get the same blank window when I try to sign in to my account.

@jinglouMSFT I now upgrade to 1.6.1 but the issue still persists. Redirecting to Orga Sign In Page and blank page

After deactivating all other option except Formular Auth in the ADFS I was prompted for a login from the ADFS Server. The Issue seems to be related with the Windows and the Windows Passport Auth.

@Theragus Thanks for the info.
@juve Your issue will require additional information and therefore I recommend you opening a support request with our support team. Here is a link for the instructions:  https://docs.microsoft.com/en-us/azure/azure-supportability/how-to-create-azure-support-request

Hello to anyone watching this issue! In 1.7.0 we're introducing the ability to optionally sign-in via device code flow, or as you may be more familiar with, copying a code from Storage Explorer into a page in your browser, and then signing in via your browser.
We're currently previewing that feature in our insiders channel. If y'all are intereseted in trying it early, simply go to Help -> Opt-in To Insider Builds.

Then, go to Help -> Check for updates, wait for 1.7.0 to appear, and install it. Then once you install it, you can enable device code flow by going to Experimental -> Use Device Code Flow Sign In.

And then let us know if things work out for y'all! This issue of blank sign in dialog is one we've been fighting for the longest time, and we're hoping this method of sign-in will unblock y'all. 😄

I'm having the same issue in v1.16.0.
Steps:
Select "add an Azure account"
Next a popup window titled "Sign in to your account" appears, I enter my corporate email address and click next
Briefly I see my corporate logo appear but then a blank window with "Browser SSO" appears and that's as far as it gets.
I am using a corporate proxy which does do SSL inspection also, but there is a SSL proxy bypass rule in place for Azure storage accounts in place.
However, when I went in to Edit -> Settings screen and ticked the "Use device code flow sign-in" option, and then followed the instructions to login via a browser, the login worked great and was able to access all storage I had access to.  So that works now.


Build Failures

I'm trying to follow this code lab and have encountered the following build errors:

File not found: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/arc/libarclite_iphonesimulator.a
Linker command failed with exit code 1 (use -v to see invocation)

I tried loading my iPhone 14 Pro simulator prior to building the project and it still failed to build.
I'm using an M1 Pro Mac with Xcode 14.3.

Thanks for reporting. In the meantime, you can work around this by updating the codelab app's deployment target to iOS 13.

I'll take a look.


Should a field containing only spaces be considered "blank"?

Using the same example/use case as for the issue #11, I wonder if using blank should work for cells with only spaces (using trim before isEmpty)?
Here is an example:
[ "   Name, Age"
, "  Alice,  12"
, "    Bob,    "
, "Charlie,  24"
]
|> String.join "\r\n"
-- |> String.replace " " ""
|> decodeCsv FieldNamesFromFirstRow (field "Age" (blank int))

I would expect to get:
Ok [Just 12, Nothing, Just 24]

But instead, I get:
Err (DecodingErrors [{ column = Field "Age" (Just 1), problems = [ExpectedInt ("    ")], row = 2 }])

If I uncomment the line String.replace " " "", it works as expected.
As we discussed previously, this could lead to problems if we do blank string. Maybe someone would expect to get Just "   " instead of Nothing?
However, should it work for blank int and blank float?
I can surely add the following to make it work:
customBlank : Decoder a -> Decode (Maybe a)
customBlank decoder =
    andThen
        (\maybeBlank ->
            if String.isEmpty (String.trim maybeBlank) then
                succeed Nothing

            else
                map Just decoder
        )
        string

[ "   Name, Age"
, "  Alice,  12"
, "    Bob,    "
, "Charlie,  24"
]
|> String.join "\r\n"
|> decodeCsv FieldNamesFromFirstRow (field "Age" (customBlank int))
                                                  ^^^^^^^^^^^

Sorry for this long message. I have seen that you consider the package "done", but I was wondering what you would expect in this situation. Thanks a lot for your work and the nice 3.0.1 upgrade!

I realize that this may be an edge case. I work with a small dataset and visually align the columns to make it nice. But this would not be like it for real datasets. (Besides, I am not blocked by it, because I can still trim the columns myself.)
I do not want to make you lose your time. Feel free to close the issue if this does not help you or go in the direction you want. Otherwise, I am open for discussion.

Thanks for bringing it up. This seems reasonable! Wanna make a PR?
I should clarify the done-ness language: I'd consider this more a bug fix than a feature since it makes the package work more in ways you'd expect.

Great! Sure for the PR.
One question before. Is it OK then to consider any "white space" as blank? Even for strings?

Even for strings? What do you mean?
I think that for the blank function we get to define what a blank field is. If people don't agree with the definition, it's easy to￼ roll your own.￼￼ If people have some field where whitespace is significant, they can use string directly.￼￼
Like with the number decoders, the difference is one of intent. People are signaling their intent that a blank string doesn't mean anything, so we can be a little bit more aggressive in our data handling.￼

Oh do you mean if you specify blank string? Yes, even then—that won't trim the string, it'll just turn an empty string into Nothing. Subtle difference but IMO an acceptable one.

Exactly.
So we are saying that a blank field is composed of any number of white space characters, not only the empty string.
As you said, decodeCsv NoFieldNames (blank string) "    " should decode into Nothing, and we can use string directly if needed.
I'll make a PR then. Thank you for the feedback.


Update MD file on new branch 2017-21-16_00-11-03

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


docs: mention the nexus ver ceiling for 0.3.x

closes #520

Hey @MihaiSandor @sapkra @larsivi please let me know if this is clear enough, thanks! I will merge this in a day or two.

Well, yes, clear in the least helpful way :)


Add Seidel and Zernike aberrations

Add elements and functions for Seidel and Zernike aberrations. Linting already applied.

Can you run Black again after the merge (or push if you already did it). Sorry about this --- I think we should set up automatic formatting at some point.


Update e2e instance count

Issue #, if available:
Description of changes:
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/test eks-anywhere-presubmit

We are increasing the count by 50? is this due to the timeout for e2e tests? was it caused by lack of resources?

We are increasing the count by 50? is this due to the timeout for e2e tests? was it caused by lack of resources?

They're testing that out now. I think the e2e test issues are likely a matter of cleaning up resources.
However, our current instance count is 217, which is why I increased it to 220.

We are increasing the count by 50? is this due to the timeout for e2e tests? was it caused by lack of resources?

They're testing that out now. I think the e2e test issues are likely a matter of cleaning up resources. However, our current instance count is 217, which is why I increased it to 220.

If current instance count is 217 and this changes the value from 170 to 220, what does this change do?

I now remember that the count in our e2e channel is the count of actual tests that failed, not the instance count because many of those tests are actually running on the same instance.So I'm going to close this PR. As folks look into the tests more, we can increase this as needed


Screen Mirroring like Airplay for non-touch screen brands like Audi

Thanks for all the contribution Ethan. Unfortunately, tweak cannot be used on non-touch screen vehicles like Audi's, Mercedes etc. due to joystick control. Can you add a feature similar to "Portal" thing in CarBridge to be able to mirror screen directly? Happy to donate as well. Thanks!

Yes! I'm having this problem too with Mercedes without touch screen.
I suggest to disable the screen showing "Running on carplay screen"
So we can control the apps from our phone.
Thanks @EthanArbuckle