yad: 8.0 -> 9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/v1cont/yad/releases.
meta.description for yad is: "GUI dialog tool for shell scripts"
meta.homepage for yad is: "https://sourceforge.net/projects/yad-dialog/"
meta.changelog for yad is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 9.0 with grep in /nix/store/zd9ar6yrk5h1al632bp703gwpib3bv46-yad-9.0

directory tree listing: https://gist.github.com/33c33e8aa51aaa8df994827aaeb57ff2

du listing: https://gist.github.com/921f4454ff05608d6bf62a21649ef239

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

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
mpvScripts.convert
yad

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zd9ar6yrk5h1al632bp703gwpib3bv46-yad-9.0
ls -la /nix/store/zd9ar6yrk5h1al632bp703gwpib3bv46-yad-9.0/bin

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

2 packages built:

mpvScripts.convert
yad

Maintainer pings
cc @grwlf for testing.

Result of nixpkgs-review pr 116467 at 1111ebc4 run on aarch64-linux 1

1 package built successfully:

yad

1 suggestion:

warning: unclear-gpl
gpl3 is a deprecated license, check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/tools/misc/yad/default.nix:46:5:
   |
46 |     license = licenses.gpl3;
   |     ^


Improve AsNoTracking integration tests after EFCore5 upgrade

Integration tests for EFCore have all entities tracked by default as they are seeded, To make tracking tests work, you are required to manually detach the tested entity first. This is not a perfect solution and may produce side effects.
EFCore5 supplies ChangeTracker.Clear for cases like this which is much faster and side-effect free (for example while working with multiple entities or Include's)
Involved PR #72
Documentation about ChangeTracker.Clear Method

Thanks, we'll look at this once we have a package depending on EFCore 5.

Thanks, we'll look at this once we have a package depending on EFCore 5.

Hey @Clivar,
Do you want to update your test now that we upgraded to EF Core 5? Your test is here. Please work with v5 branch. Pull that branch and PR to that branch.
It doesn't hurt to have this kind of integration test, but I don't think it's crucial from the package's perspective. For us, it's important to test that if this feature is used then the flag is saved in the specification, and the evaluator passes the operation to the provider. That's all we do. The actual implementation is done on the provider's side (EF Core), and it's not our concern. On the other side, user's in their code-base might have full integration tests, and test for their scenarios.

If you'll be working on this, please add these unit tests. It's a simple one. Sounds OK?
public class SpecificationBuilderExtensions_AsNoTracking
{
    [Fact]
    public void DoesNothing_GivenSpecWithoutAsNoTracking()
    {
        var spec = new StoreEmptySpec();

        spec.AsNoTracking.Should().Be(false);
    }

    [Fact]
    public void FlagsAsNoTracking_GivenSpecWithAsNoTracking()
    {
        var spec = new CompanyByIdAsUntrackedSpec(1);

        spec.AsNoTracking.Should().Be(true);
    }
}

For us, it's important to test that if this feature is used then the flag is saved in the specification, and the evaluator passes the operation to the provider. That's all we do
Basically what I was trying to achieve is exactly that. There seems to be no good way (that I know at least) to test the evaluator for AsNoTracking(). Other tests can rely on result to check if it applied correctly. For lack of better option I used efcore to validate it but I do agree this is also basically testing efcore's behavior.

For now I updated my tests to my suggestion which at least gives it a better AAA (Arrange Act Assert) look.

Great, thanks @Clivar.

Implemented in version 5.0.3


[code review] idempotency of NodePublish/Unpublish

Double check idempotency of NodePublish/Unpublish. For example, if NodePublish gets called twice, and it's already correctly mounted, we should return success. Looking at the current code, it seems like we'll bind mount it twice? Which can potentially cause mounts to leak on cleanup.  Similarly, Unpublish should return success if it's already unpublished.
Usage of IsLikelyNotMountPoint() also doesn't work if the bind mount is from the same fs as the kubelet's rootfs (if you look at the implementation, it just checks if the parent directory is in the same fs or not).
results from https://github.com/deislabs/secrets-store-csi-driver/blob/51d64fcf33d62b84143f666ed8bc387011652de5/pkg/secrets-store/nodeserver.go#L220 are unused?

part of #100

Closed with https://github.com/deislabs/secrets-store-csi-driver/pull/124


Clarification on Using Kustomize with Helmfile for Upstream Kubernetes Component Patching

Summary:
I would like to understand whether using Helmfile with Kustomize is recommended for patching upstream Kubernetes components (e.g., OpenShift storage or image registry). After reviewing the documentation and experimenting, I’ve encountered challenges that make it unclear whether this integration is suitable for declarative patch management. Specifically, I want to avoid relying on one-off imperative commands and manage everything as code.
Context:
My goal is to deploy and manage both infrastructure and applications declaratively, without running ad-hoc commands. For example, after installing an OpenShift cluster, I need to patch certain built-in components like storage operators or the image registry. These components are part of the upstream Kubernetes system and not tied to any specific application deployment.
Challenge:
I’ve encountered an issue when trying to patch these components using Kustomize within Helmfile. It seems I need to declare ownership of certain system components by adding special annotations. Without these annotations, Helm or Helmfile reports errors such as:
•	Resource not managed by Helm: When trying to patch components that were originally created outside of Helm.

To apply patches smoothly, it looks like I need to add annotations like these to the upstream resources:
metadata:
annotations:
meta.helm.sh/release-name:
meta.helm.sh/release-namespace:
labels:
app.kubernetes.io/managed-by: Helm
These annotations effectively “claim ownership” of the resources on behalf of Helm, allowing Helmfile to manage them. However, this approach feels intrusive since it forces us to adopt control over components that should ideally remain under the upstream system’s management.
Question:
•	Is it recommended to use Helmfile with Kustomize for patching upstream components this way?
•	Is there a cleaner way to manage such patches without adding ownership annotations to system resources?
•	Alternatively, is there a better strategy for managing patches declaratively within Helmfile that avoids imperative commands and ownership conflicts?

Thank you for your guidance!

repository has moved. please open your issue here: https://github.com/helmfile/helmfile/issues

I opened a discussion there, but I highly doubt I will get reply soon. LOL>


Submitting Signout for teacher returns a Bad Request 400

Sending email of<EMAIL_ADDRESS>after signing in does not return a 200.

Wrong api is being called on form submission, change to sign in endpoint not sign out.

Fixed in #50


Provide a multi-context mode

It might be possible to have a mode where instead of using the cached application we clone a new version of Steal for each request. We could possibly re-use the module source code.
Would be interested to see the performance of this method.

This could probably be implemented as a zone plugin, or as an option to the existing one.


Upgrade ossf/scorecard-action v2.3.1 -> v2.3.3

This PR contains the following updates:

Package
Type
Update
Change

ossf/scorecard-action
action
patch
v2.3.1 -> v2.3.3

Release Notes

ossf/scorecard-action (ossf/scorecard-action)
v2.3.3
Compare Source

[!NOTE]
There is no v2.3.2 release as a step was skipped in the release process. This was fixed and re-released under the v2.3.3 tag

What's Changed

:seedling: Bump github.com/ossf/scorecard/v4 (v4.13.1) to github.com/ossf/scorecard/v5 (v5.0.0-rc1) by @spencerschrock in https://github.com/ossf/scorecard-action/pull/1366
:seedling: Bump github.com/ossf/scorecard/v5 from v5.0.0-rc1 to v5.0.0-rc2 by @spencerschrock in https://github.com/ossf/scorecard-action/pull/1374
:seedling: Bump github.com/ossf/scorecard/v5 from v5.0.0-rc2 to v5.0.0-rc2.0.20240509182734-7ce860946928 by @spencerschrock in https://github.com/ossf/scorecard-action/pull/1377

For a full changelist of what these include, see the v5.0.0-rc1 and v5.0.0-rc2 release notes.
Documentation

:book: Move token discussion out of main README. by @spencerschrock in https://github.com/ossf/scorecard-action/pull/1279
:book: link to ossf/scorecard workflow instead of maintaining an example by @spencerschrock in https://github.com/ossf/scorecard-action/pull/1352
:book: update api links to new scorecard.dev site by @spencerschrock in https://github.com/ossf/scorecard-action/pull/1376

Full Changelog: https://github.com/ossf/scorecard-action/compare/v2.3.1...v2.3.3
v2.3.2
Compare Source

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade ossf/scorecard-action v2.3.1 -> v2.3.3


Improve CI

:zero: Byte-level

Look for non-ASCII characters
(non-English alphabets, whitespace characters, control characters)

:one: Syntax errors

Check source code for syntax errors

:two: Run unit and functional tests
:three: Static Analysis

Run static analysis: PHPStan

:four: Coding Standards

Check coding style
Adhere to EditorConfig

@DarkGhostHunter Should I fix CS problems?

Should I fix CS problems?

Done 🍏

@DarkGhostHunter Please look at TValue in src/Refresh.php once more.

I see some commented code on the test runs. Care to say what are their purpose?

As CacheRefresh is a library there could be a CI job that installs it into Laravel sample application and does some real-word testing.

The callback will receive TValue (the value exists) or null (cache miss), so the PHPDoc should reflect that always.

Done.
What is the second parameter's (Laragear\CacheRefresh\Expire) default value?

What is the second parameter's (Laragear\CacheRefresh\Expire) default value?

Is itself.
use Illuminate\Support\Facades\Cache;
use Illuminate\Support\Collection;
use Laragear\CacheRefresh\Expire;

Cache::refresh('rofl', function (?Collection $collection, Expire $expire): Collection {
   // ..
}

@DarkGhostHunter Finally done 🍏

Thanks. I'm testing something on Rut 2.0, I'll get back to this tomorrow.

Made some changes: Changed to ubuntu-latest, removes redundant Code Standard.

removes redundant Code Standard

@DarkGhostHunter Does PHP CS Fixer run elsewhere?
The removed job contained a step to check EditorConfig adherence.

Style fixes are done already by StyleCI.
I mistakenly deleted the PHP CS Fixer step, I’ll re add it later.
Italo Baeza C.

El 24-07-2022, a la(s) 02:26, Viktor Szépe @.***> escribió:

removes redundant Code Standard
@DarkGhostHunter Does PHP CS Fixer run elsewhere?
The removed job contained a step to check EditorConfig adherence.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

StyleCI is an external service. Log in and enable it for this repo: https://styleci.io/

Done. The PHP CS Fixer is not needed anyway. Any hiccup will be catch by PHP Stan.


Add customer info to transaction.

How can I add customer info while using the Accept.js gateway?
I am currently using the Accept.dispatchData method on the Accept.js library to create the opaque data values.
And in the backend I use the values.
$response = $gateway->purchase([
    'amount' => $amount,
    'opaqueDataDescriptor' => $request->input('dataDescriptor'),
    'opaqueDataValue' => $request->input('dataValue')
])->send();

How can I add customer info to the transaction?
I see the addBillingData method on the AIMAbstractRequest class, but I am not passing any customer info via Accept.js.
I am using version 2.6.

I realized that you can add customer name via Accept.js.
var secureData = {
    cardData: {
        // ...
        fullName: fullName
    }
};
Accept.dispatchData(secureData, function(){});

But I am still unable to set customer email address.
How can I set the customer email address?

You would normally set the email of the customer through the CreditCard object:
$creditCard = new \Omnipay\Common\CreditCard([
    'email' =><EMAIL_ADDRESS>]);

$response = $gateway->purchase([
    'amount' => $amount,
    'opaqueDataDescriptor' => $request->input('dataDescriptor'),
    'opaqueDataValue' => $request->input('dataValue'),
    'card' => $creditCard,
])->send();

Some more details here:
https://omnipay.thephpleague.com/api/cards/

Thanks @judgej !
Yes, I do understand how to do it with a regular card purchase.
But for my case I am using Accept.js in the front end to create a token and then using the token to finalize the purchase. Not sure in which step the email address can be added. Authorize.net's documentation on their website isn't very helpful.

It's exactly as shown -when you submit the purchase. You can collect those details however you like - in the original form with the Accept form, from the back-end user account, from an additional form in the process. But it;s for you to do, not the Accept.JS front end. That only concerns itself with what it needs to tokenise a card.

@judgej
Yes, example above was more than enough.
Thank you.


✨ Improved password validation rules

refs #9150

Moves the password length fn from models/user to data/validation where the other validators are
Added password validation rules
Passwort rules added:

Disallow obviously bad passwords:<PHONE_NUMBER>, qwertyuiop, asdfghjkl; and asdfghjklm
Disallow passwords that contain the words "password" or "ghost"
Disallow passwords that match the user's email address
Disallow passwords that match the blog domain or blog title

Password validation returns an Object now, that includes the isValid statement and an message property to differentiate between the two messges (password too short or password insecure)
Use a catch predicate in api/authentication on passwordReset, so the correct ValidationError will be thrown rather then an UnauthorizedError
When in setup flow, the blog title is not available yet from the settingsCache. We therefore supply it from the received data in the user setup method to have it accessible for the validation.

Todos:

[x] return at least two different error messages
[x] add and update more tests

When this is good to merge, the commits can be squashed and the PR description can be used as commit message. I kept them separate, to make the review and my way to the solution easier to understand.


Add rule for checking number of arguments

Closes #83

Codecov Report

Merging #180 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #180      +/-   ##
==========================================
- Coverage   98.75%   98.73%   -0.02%
==========================================
  Files          31       31
  Lines        1929     1903      -26
==========================================
- Hits         1905     1879      -26
  Misses         24       24

Impacted Files
Coverage Δ

robocop/checkers/lengths.py
98.03% <100.00%> (+0.06%)
:arrow_up:

tests/utest/test_thresholds.py
95.45% <0.00%> (-0.20%)
:arrow_down:

robocop/checkers/duplications.py
96.82% <0.00%> (-0.05%)
:arrow_down:

robocop/checkers/naming.py
99.25% <0.00%> (-0.01%)
:arrow_down:

robocop/config.py
100.00% <0.00%> (ø)

tests/e2e/test_e2e.py
100.00% <0.00%> (ø)

tests/utest/conftest.py
100.00% <0.00%> (ø)

robocop/checkers/misc.py
100.00% <0.00%> (ø)

tests/utest/test_messages.py
100.00% <0.00%> (ø)

... and 4 more

Continue to review full report at Codecov.

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


Vertical Scrollbar extents are wrong.

When the vertical scrollbar appears, it doesn't seem to calculate the extents of the rows correctly:

The example above is using grouping, but this seems to happen even without grouping.

@grokys  Is the source for application used in the sample above available? I'm having difficulty replicating the issue on my end.

Oh sorry, yes the source is here: https://github.com/grokys/devtools


Suggestion: сhart sets default options for scales

Is your feature request related to a problem? Please describe.
At the moment, some chart options may affect other objects:

ChartOptions['leftPriceScale'] overwrites left price scale options
ChartOptions['rightPriceScale'] overwrites right price scale options
ChartOptions['overlayPriceScales'] overwrites options for EACH overlay price scale
ChartOptions['timeScale'] overwrites time scale options

If for the left, right and time scales the overwrite options are more or less obvious, then for the overlay it is not quite so. You can have multiple overlay scales with different options. Each of them can be given some unique options, but one single chart.setOptions({ overlayPriceScales }) call will overwrite them.
Describe the solution you'd like
I would like to suggest adding or renaming the current chart options. Instead of the usual options, allow you to set default options. That is, we will have the option ChartOptions['defaultOverlayPriceScale'] and it will work a little differently. This option will never overwrite explicitly set options on the scale, i.e. it will do something like this:
class PriceScale {
    setOptions(options) {
        merge(this._options, this._defaultOptionsFromChart, options)
    }
}

chart.applyOptions() only applies the properties deeply with a merge already.
Thus something like this:
chart.applyOptions({
    overlayPriceScales: {
        mode: LightweightCharts.PriceScaleMode.Normal,
    }
});

Would only change the mode option on all the overlay price scales, and leave the rest of the options as they currently are.
If you want to change the options individually for each overlay price scale then you can access the specific price scale with it's id within the chart.priceScale() method.
For example:
chart.priceScale('my-overlay').applyOptions({ autoScale: false });

If you feel that I've misunderstood your issue then please could you provide an example (like a JSFiddle) of the behaviour which you would like to change, and I'll reopen the issue.

I don't have any issue with this behavior.
I just wanted to set default options for newly created price scales, rather than overwriting options for already created and customized ones.

Makes sense. We probably won't implement a new api for this. Not because it is a bad idea but rather because it is something that can be achieved from outside the library and we want to keep the library as 'lightweight' as possible.
The easiest way to achieve something like that at the moment would be to create your own defaultOptions object which you then 'extend' when creating a new price scale. You could also create your own helper (factory) function or class to achieve this if it is a common requirement in your code.
const myDefaultPriceScaleOptions = {
  autoScale: true,
  invertScale: true,
};

// ...

chart.priceScale('my-overlay').applyOptions({
   ...myDefaultPriceScaleOptions, // start with your new defaults
  // then add the specific options for this price scale
  scaleMargins: {
        top: 0.8,
        bottom: 0,
    },
});


why the to_message function in chat_history.py is trying to add an assistent message?

Related code:
https://github.com/OpenBMB/AgentVerse/blob/4dd772de18ac5fb2ed9b8cf4a8731565d224afab/agentverse/memory/chat_history.py#L56
https://github.com/OpenBMB/AgentVerse/blob/4dd772de18ac5fb2ed9b8cf4a8731565d224afab/agentverse/llms/openai.py#L436
If there is no message to add, the structure of the mssages will be system/user
But if there is a history message, the structure will change into system/assistant/user, which will cause an error since its not system/user/assistant/user .....

The second role in LLAMA must be 'user.' Does that mean AgentVerse is not compatible with LLAMA models?

there will be an error if you lunch  llama model with local vllm
openai.BadRequestError: Error code: 400 - {'object': 'error', 'message': 'Conversation roles must alternate user/assistant/user/assistant/...', 'type': 'BadRequestError', 'param': None, 'code': 400}


Add support for array of structs

Closes #232

We've decided to just add Pack = 1 for our structures, and so this math will start working.
See how the types for constant pool are generated if you want to know how to enable Pack = 1 with Cecil.

Integration tests have failed because we cannot determine proper size of a pointer type now :(
Let's go back to the drawing board, this is too much for me.

Assigning to myself, I'll try to start working on #132 which is required to resolve the question here.


Created and updated docs for 1.26-13

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


Implement Drop for SplitGrantR

This should close #81.

@Sympatron could you please add the unit test from https://github.com/jamesmunns/bbqueue/issues/81#issuecomment-742580523 and verify it passes in CI w/ this PR?

Thanks @Sympatron!


Exclude undefined line-height and letter-spacing from JSON

Fixes #29
Adapted from @KimDal-hyeong's proposed solution in the issue discussion 🍻

After digging in further, thanks to @KimDal-hyeong, it appears that this was a Puppeteer issue.
For me, the fix was to stringify the JSON in a Puppeteer context, rather than passing an object reference across the bridge.
e.g.
-const page = await page.evaluate('styleguide2asketch.getASketchPage()');
+const pageJson = await page.evaluate('JSON.stringify(styleguide2asketch.getASketchPage())');

Even though this PR fixes the issue, we don't really want to introduce a Puppeteer workaround in this project unless we have to, so I'm closing this PR.

Thanks Mark! I'll reach out to Puppeteer people to see if that's intended behaviour and update html-sketchapp examples accordingly.


Why is there no new snapshot yet ? users experience probs with <IP_ADDRESS>

See here, users:

got stuck at block 1010000
landed on at least 2 forks
can not actually come onto "this blockchain" without a snapshot

When will a new snapshot be available?
A snapshot is surely easier to make than a new Gridcoin version (for what we wait already for 4 days)

PLEASE, provide signed checksum for the snapshot when it is made.

Is there a new Snapshot jet?

October 7th was latest

October 7, 2017 confirmed. you can see the date of upload in this directory: https://download.gridcoin.us/download/downloadstake/signed/
can we close this now? it's not really a wallet issue, per se.


error with installing zulip dependencies in docker image

Hi.
Trying to build zulip images with this instruction:
https://zulip.readthedocs.io/en/1.6.0/dev-setup-non-vagrant.html#using-docker-experimental
At this stage
/bin/bash /srv/zulip/tools/provision --docker
got this failure
PROVISIONING STARTING.
+ mkdir -p /srv/zulip/var/8b049dfe-2427-40d6-bc05-22bb179117bd
No changes to apt dependencies, so skipping apt operations.
+ sudo -H scripts/lib/install-node
scripts/lib/install-node: line 34: /usr/local/nvm/nvm.sh: No such file or directory

Error running a subcommand of ./lib/provision.py: sudo -H scripts/lib/install-node
Actual error output for the subcommand is just above this.

Traceback (most recent call last):
  File "./lib/provision.py", line 419, in <module>
    sys.exit(main(options))
  File "./lib/provision.py", line 248, in main
    run(["sudo", "-H", "scripts/lib/install-node"])
  File "/srv/zulip/scripts/lib/zulip_tools.py", line 163, in run
    subprocess.check_call(args, **kwargs)
  File "/usr/lib/python3.4/subprocess.py", line 561, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sudo', '-H', 'scripts/lib/install-node']' returned non-zero exit status 1

My OS - ubuntu 16.
How can i fix it?

First, you're using instructions from a very old release of Zulip (1.6.0); is that intentional?

Closing as likely user error and/or already fixed.


Ite Technology- SD Card Reader reboots bug

SD Reader bug causing reboots.

Build 0I39  live

Can be used normally.

Thanks for confirming @louies0623 :+1:

This is how it is done:
https://github.com/helloSystem/ISO/commit/01f7e869c99eb6a7fded4a05e03d083fa1492a3b
Please send pull requests similar to this one if you'd like to add more card readers like this.
Thanks!


Search Tabs command fails for tabs within Chrome tab groups

Extension – Google Chrome
Author: @rgomezcasas
Description
Issue with Search Tabs command to switch to the Chrome Tab.
I use google tab groups, and what I observe is that I can switch to a tab from the Search tabs command when the tab is inside a collapsed tab group.
Steps To Reproduce

Have a tab group with multiple tabs open
Collapse the tab group
Go to Search tabs command on Raycast
Search for one of the tabs in the tab group
Hit Enter on it

The current behavior
Nothing happens i.e. focus doesn't shift to the tab
The expected behavior
Focus shifts to the right tab.
Raycast version
Version: 1.28.0

LMK if you need any more information on this.


Clarified wording in new column master key dialog

Just minor clarification on the wording. Also it refers to a key where I think it should say certificate instead, as the list contains certificates.

@greglow : Thanks for your contribution! The author, @jaszymas, has been notified to review your proposed change.

Please note that the key can be either a certificate (if the user selected Windows Certificate Store in step 3) or a key in Azure Key Vault (which is not a certificate). Could you  rephrase to something like this: "a certificate or an Azure Key Vault key"?

@greglow : Thanks for your contribution! The author, @jaszymas, has been notified to review your proposed change.

Done
Regards,
Greg
Dr Greg Low
1300SQLSQL (1300 775 775) office | +61 419201410 mobile│ +61 3 8676 4913 fax
SQL Down Under | Web:  https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.sqldownunder.com%2F&data=02|01|sspahelp%40microsoft.com|1f0ea4d6b97e4d897f3708d666d1e890|72f988bf86f141af91ab2d7cd011db47|1|0|636809449091516274&sdata=SLHeEGAMmWUY5YIwcC4oAPYr%2F9RIZdi4MNASsdzwX2I%3D&reserved=0 www.sqldownunder.com | https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fgreglow.me%2F&data=02|01|sspahelp%40microsoft.com|1f0ea4d6b97e4d897f3708d666d1e890|72f988bf86f141af91ab2d7cd011db47|1|0|636809449091526278&sdata=IU8tnAITCjBxWafi3A9XpO9lF3PIwZJ8ad3t36lnxvs%3D&reserved=0 http://greglow.me
From: jaszymas<EMAIL_ADDRESS>Sent: Saturday, 23 March 2019 9:17 AM
To: MicrosoftDocs/sql-docs<EMAIL_ADDRESS>Cc: Greg Low<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/sql-docs] Clarified wording in new column master key dialog (#1760)
Please note that the key can be either a certificate (if the user selected Windows Certificate Store in step 3) or a key in Azure Key Vault (which is not a certificate). Could you rephrase to something like this: "a certificate or an Azure Key Vault key"?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/sql-docs/pull/1760#issuecomment-475801170 , or mute the thread https://github.com/notifications/unsubscribe-auth/AEkha0eLjJIuulquFWAbWQIdLpMZHoZqks5vZVZ3gaJpZM4bnvsi .  https://github.com/notifications/beacon/AEkha9L2Jea1kSkaL-jTd4wpBqiN8Rgnks5vZVZ3gaJpZM4bnvsi.gif

#sign-off


git review friendly

what issue did you encounter to make this change?

Notebook searlizer is not formatting notebook.
for suppose:
https://github.com/tanhakabir/rest-book/blob/main/notebooks/spacex-api.restbook#L1
its hard to read, hard to compare changes.
formatting notebook before saving will let users read from non vscode environment (like github, vi, git diff)

I don't believe GitHub can handle rendering custom notebooks at the moment so I don't think adding these parameters will fix how notebooks are rendered in GitHub


showNavigationButtons not working based on radio group question choice

Are you requesting a feature, reporting a bug or ask a question?
Bug
What is the current behavior?
I'm wanting to turn off the Complete button on a survey if the user chooses a "No" answer to a radio group question which means they do not need to continue filling it out.
I'm checking the radio group question's answer via onValueChanged
The current behaviour is that if the user chooses No, the Complete button continues to be displayed when it shouldn't be.
What is the expected behavior?
If the user chooses "No" to the radio group question it should hide the "Complete" button straight away.
How would you reproduce the current behavior (if this is a bug)?
See this Plunker demo http://plnkr.co/edit/73E72SUGXaTHn89cKk27?p=preview
Steps to replicate the issue:

Choose "No" for first question
The HTML message is then displayed but Complete button is still displayed (incorrect behaviour).
Then go back and choose "Yes" for the first question, then choose "No" for the first question again.
The HTML message is then displayed again. This time the Complete button is not displayed.

Provide the test code and the tested page URL (if applicable)
http://plnkr.co/edit/73E72SUGXaTHn89cKk27?p=preview
Specify your

browser: Chrome
browser version: 59.0.3071.115 (Official Build) (64-bit)
surveyjs platform (angular or react or jquery or knockout or vue): jquery
surveyjs version: 0.12.19

@bjdani, Hello! Thanks for your interest, could you please try to add survey.render(); into your  surveyQuestionValueChanged handler?
here an example: http://plnkr.co/edit/w2WAJXq3M3HOlZWLYZiq?p=preview

@bjdani I would recommend to call survey.doComplete() method to complete the survey: http://surveyjs.org/docs/api_surveymodel.html#doComplete
Thank you,
Andrew

Thanks to you both. I'll roll with the survey.render(); which solves it.


Add layout component

This promotes the new Layout component from Experimental to Alpha.
I can do a fast follow of migrating this linter for upstreaming: https://github.com/github/github/pull/198781

This looks great, thanks @jklina!


Sql Database Output Manager

Issue: #480

[x] Tick here to confirm that you have run RunCodeFormatter.ps1

Description:
Added output manager to SQL Azure Database with functionality to auto-create tables and insert stored procedures for incoming data storage. Sample project IngestToSql is added as well to demonstrate how
to use Output Manager.
Test:
Run sample IngestSQLProject together with Nether.Ingest, send some data, ensure data landed to database.

@rzdebskiy,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Initiated a rebase after another PR was approved before this


[skip ci] fix typos in comments

These two messages got accidentally swapped in PR #2545

@bors r+

:pushpin: Commit 86bc203a6ea2d99782aca3dbe49ed71d6f8dd3f2 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 86bc203a6ea2d99782aca3dbe49ed71d6f8dd3f2 with merge<PHONE_NUMBER>46e4c287813bce62e55b7a2200a1b7...

:boom: Test timed out

I think bors is confused because the tests aren't getting run.  Can you bypass bors and merge directly?

Could you remove the skip-ci prefix instead? I'd like to manage Git history with bors as possible.

As you wish

Great, thank you! @bors r+

:pushpin: Commit 95fab0f8460b6b14f49abda98bf245a66922d426 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 95fab0f8460b6b14f49abda98bf245a66922d426 with merge 0292b4eff4dc8f83c2098fa97ae2623b557c299f...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 0292b4eff4dc8f83c2098fa97ae2623b557c299f to main...


Port LogImbue to Postgres

Description
Ports log imbue over. I have created a ticket for the fact that these can grow continuously without ever being pruning and added the ts field so that this is an easy query in the future. There is also another ticket to reduce the size of the payloads
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
CI, plus I guess I can try to break the UI

/test stackrox_branding-images


rustup install fails when rustup-init is owned by another user

Not sure if this is a rustup or Homebrew rustup-init issue, but I found that installing rustup-init using Homebrew, and then running rustup-init as a different user resulted in a permissions error (shown below). The result was that rustup would install toolchains, but never set up the binaries in ~/.cargo/bin.
Current installation options:

   default host triple: x86_64-apple-darwin
     default toolchain: stable
  modify PATH variable: yes

1) Proceed with installation (default)
2) Customize installation
3) Cancel installation
>

error: failed to set permissions for '/Users/p4nathan/.cargo/bin/rustup'
info: caused by: Operation not permitted (os error 1)

Changing the ownership of /usr/local/bin/rustup-init to the user running rustup averted the issue and everything installed as expected. Maybe I'm weird in that I want one installation of rustup-init but have multiple accounts on the same system using rustup. At least for now I can work around this.

Is, by any chance, /usr/local/bin/rustup-init a symlink?  Or perhaps however your user is finding rustup-init on your path a symlink?  if so, we don't copy the file we link to it, which would make the chmod fail for sure.
Given that, perhaps the trick will be to make utils::make_executable() notice if the target is already executable and simply not attempt the chmod().
If it's not a symlink then I'll need to think again.

Yes, with Homebrew everything under /usr/local/bin is a link.
$ ls -l /usr/local/bin/rustup-init
lrwxr-xr-x 1 nfiedler admin 44 Dec 11 08:16 /usr/local/bin/rustup-init -> ../Cellar/rustup-init/1.16.0/bin/rustup-init

$ ls -l /usr/local/Cellar/rustup-init/1.16.0/bin/
total 6500
-rwxr-xr-x 10 nfiedler staff 6655064 Dec  6 12:01 rustup-init

Thanks

Okay, so I believe the right approach will be to check the permissions and not try and alter them if they're okay already.  I'll look at what that will entail, and update the issue.  I think this might end up a nice new-contributor issue if you're okay waiting for a bit for a fix?

No problem, it's easy to work around.

For someone who wanted to ensure that this kind of thing was fixed for the future, then it'll probably be a combination of altering fn make_executable(path: &Path) -> Result<()> which lives in src/rustup-utils/src/utils.rs
I'd expect that something where we can only call set_permissions if the mode changes in the non-Windows inner() would be the right approach.
A test which then validates this could be done by means of creating a read-only directory containing rustup-init already executable; make a symlink to that in another directory, and then run it and see if an install can be done.
My expectation is that making the test will be the hardest part of this change.  I am happy to help someone through this, ping me if you want to try and write a PR to solve it.

Hi @kinnison
I'd like to help here if still needed. I've replicated the issue and am prepared to test. Any further tips? Thanks!

Hi @lallotta and thank you for wanting to help.
I would suggest that since you've managed to replicate the issue you're in a good position to try and narrow down a bare-minimum test case to be added to the test suite which demonstrates the problem.
My un-researched thoughts are above, but in summary, I think that this is a case that set_permissions likely fails because it's unable to alter the source binary, despite it already being executable.  To that end, I think it's worth first trying and seeing if you get a similar error if the binary is owned by the same user, but is in a directory which is read-only rather than read-write.  E.g.
mkdir test-dir
cp target/debug/rustup-init test-dir/
chmod 555 test-dir
test-dir/rustup-init ...

If that also replicates the issue then we're a step closer to building a plausible test to run in CI.
Once we have a test in CI, probably as part of tests/cli-self-upd.rs, then we can proceed to the fix.
Good luck, and please don't hesitate to ask if you want further guidance.  I'm also usually on the #wg-rustup channel in Discord

Thank you for your guidance @kinnison
I did some experimenting with your notes, and I found that running the binary as its owner from a read-only directory still succeeds:
$ ls -ld test-dir/ test-dir/rustup-init
dr-xr-xr-x  3 lorenzo  staff        96 Apr 10 18:36 test-dir/
-rwxr-xr-x  1 lorenzo  staff  36419552 Apr 10 18:36 test-dir/rustup-init

$ RUSTUP_HOME=rustup-home CARGO_HOME=rustup-home test-dir/rustup-init --no-modify-path -y
info: syncing channel updates for 'stable-x86_64-apple-darwin'
info: latest update on 2019-04-11, rust version 1.34.0 (91856ed52 2019-04-10)
info: downloading component 'rustc'

Running a symlink succeeds as well:
$ ls -l rustup-test/rustup-init
lrwxr-xr-x  1 lorenzo  staff  35 Apr 10 18:39 rustup-test/rustup-init -> /Users/lorenzo/test-dir/rustup-init

$ RUSTUP_HOME=rustup-home CARGO_HOME=rustup-home rustup-test/rustup-init --no-modify-path -y

I only seem to produce this error under the original conditions:
$ ls -l /usr/local/bin/rustup-init
lrwxr-xr-x  1 lorenzo  admin  44 Apr 10 17:53 /usr/local/bin/rustup-init -> ../Cellar/rustup-init/1.17.0/bin/rustup-init

$ ls -l /usr/local/Cellar/rustup-init/1.17.0/bin/
total 12440
-rwxr-xr-x  12 lorenzo  staff  6365960 Mar  5 18:50 rustup-init

$ RUSTUP_HOME=home CARGO_HOME=home rustup-init --no-modify-path -y
error: failed to set permissions for '/Users/m/home/bin/rustup'
info: caused by: Operation not permitted (os error 1)

Hmm, looks like I was mistaken, the read-only directory only prevents directory changes, not changes to the inode data of entries inside it.  Urgh, this makes life harder for trying to get this to a state we can CI it since I don't think our CI systems have multiple users we can set up for testing.  Certainly it'd be unreasonable to expect that of random people running cargo test too ;(
It's possible we'll have to take a fix for the implementation without an easily automatable test for verifying we don't regress again.
For now, I suggest focussing on a fix, and once you have the fix we can see again if we can work out how to build a test for this.  Sorry to have made a suggestion which turned out to not be helpful.

No apologies necessary. I've learned something new.
My changes below appear to fix the issue. I aimed to check if the mode is ok already, and avoid calling set_permissions if so. I appreciate your advice if this can be improved. Thank you again.
https://github.com/rust-lang/rustup.rs/compare/master...lallotta:fix-1638?expand=1

That patch LGTM as the bugfix itself.  I'm happy to merge that if we can't come up with a good way to CI test it.

Don't know if it's related, but I just had to do touch /home/dhardy/.cargo/bin/rustup-init to make rustup self update succeed (previously the file didn't exist). I was updating from a fairly old install, possibly from before this patch hit.

@dhardy I don't think this is related no.


CosmWasmClient: Add an optional msg transformer

Feature Request
Adding an optional MsgTransformer to CosmWasmClient (via its constructor).
MsgTransformer will be an interface that transforms the MsgInstantiateContract and MsgExecuteContract before init, handle and query, and also transforms tx responses and errors.
Suggested interface:
interface MsgTransformer {
  transformInitMsg: (MsgInstantiateContract) => MsgInstantiateContract;
  transformHandleMsg: (MsgExecuteContract) => MsgExecuteContract;
  transformSmartQueryMsg: (object) => object;

  transformInitResult: (MsgInstantiateContract, BroadcastTxResult) => BroadcastTxResult;
  transformHandleResult: (MsgExecuteContract, BroadcastTxResult) => BroadcastTxResult;
  transformSmartQueryResult: (object, JsonObject) => JsonObject;
}

(Not sure if the Result transformers also cover errors)
The default implementation would just look like:
const defaultMsgTransformer: MsgTransformer = {
  transformInitMsg: (x: MsgInstantiateContract) => x,
  transformHandleMsg: (x: MsgExecuteContract) => x,
  transformSmartQueryMsg: (x: object) => x,

  transformInitResult: (x: MsgInstantiateContract, y: BroadcastTxResult) => y,
  transformHandleResult: (x: MsgExecuteContract, y: BroadcastTxResult) => y,
  transformSmartQueryResult: (x: object, y: JsonObject) => y,
};

The Use Case
Currently at Secret Network we are maintaining an old fork of cosmwasmjs with just our encryption code before sending and after receiving a tx response. cosmjs is miles ahead of our fork, and we don't really see the point of maintaining a fork to just basically have this line modified: https://github.com/enigmampc/SecretNetwork/blob/8e8628e648bd897a0fe1a225ab58f8294464950d/cosmwasm-js/packages/sdk/src/signingcosmwasmclient.ts#L203
Also, in the above link you can see that we modified the x/wasmd message to have additional fields: callback_code_hash and callback_sig, and also removed everything admin / migrate / history related. I'm thinking other chains that are going to adopt CosmWasm in the future might also want to modify some of these parts for their use case.
What I have in mind is passing executeMsg to the transformer before this line: https://github.com/cosmos/cosmjs/blob/4fdab7f65d9f2fa3627ce63d569aae3d6cfe6402/packages/cosmwasm/src/signingcosmwasmclient.ts#L330
And then before returning, passing result with its executeMsg again to the transformer.
Let me know what you think. I'm happy to create a PR for this if you think this feature is something that will benefit cosmjs and other chains in the ecosystem. :blush:

@assafmo Thanks for the interesting feature request. I just want to confirm first of all that it doesn't make sense in your situation to perform the transform in the signer rather than in the signing client itself? We already allow for the possibility that the signer wants to modify the sign doc (eg change the fees or whatever) and return something different to the signing client. See eg https://github.com/cosmos/cosmjs/blob/4fdab7f65d9f2fa3627ce63d569aae3d6cfe6402/packages/proto-signing/src/signer.ts#L5-L20

Does the signer allows modifying the result/error as well?

@assafmo This seems like a lot of overhead to maintain for a very uncommon use case. Ethan came up with the idea of having a wrapper class like SecretCosmWasmClient around CosmWasmClient, which does the operations for you. You would need something like that on your side anyways, if you don't want every single user to define the transformers. Is thatis something that works for you?
On current master, the cosmwasm package was renamesd to @cosmjs/cosmwasm-launchpad. This should in theory be CosmWasm 0.10 compatible. However, nobody is testing this right now.

Yes! This is a great idea, let me know what can I do to make this happen 🤩
On Mon, Dec 21, 2020, 17:33 Simon Warta<EMAIL_ADDRESS>wrote:

@assafmo https://github.com/assafmo This seems like a lot of overhead
to maintain for a very uncommon use case. Ethan came up with the idea of
having a wrapper class like SecretCosmWasmClient around CosmWasmClient,
which does the operations for you. You would need something like that on
your side anyways, if you don't want every single user to define the
transformers. Is thatis something that works for you?
On current master, the cosmwasm package was renamesd to
@cosmjs/cosmwasm-launchpad. This should in theory be CosmWasm 0.10
compatible. However, nobody is testing this right now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cosmos/cosmjs/issues/512#issuecomment-749034621, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNWHRP442YTMM4JDCZTACLSV5TDTANCNFSM4TKCYHTQ
.


This KCL (intentionally causing errors) locks code mirror completely

// L-System KCL
// Zoo Corporation ⓒ 2024

// L-system
// Note: it was most concise to encode productions directly in axioms.
// Change them as you need.

deg = pi()*2 / 360

fn setSketch = (state, _) => {
  return {
    depthMax: state.depthMax,
    depth: state.depth + 1,
    currentLength: state.currentLength,
    factor: state.factor,
    currentAngle: state.currentAngle,
    angle: state.angle,
    _: _
  }
}

fn setDepth = (state, _) => {
  return {
    depthMax: state.depthMax,
    depth: _,
    currentLength: state.currentLength,
    factor: state.factor,
    currentAngle: state.currentAngle,
    angle: state.angle,
    _:  state._
  }
}

fn setAngle = (state, _) => {
  return {
    depthMax: state.depthMax,
    depth: state.depth,
    currentLength: state.currentLength,
    factor: state.factor,
    currentAngle: _,
    angle: state.angle,
    _:  state._
  }
}

fn setLength = (state, _) => {
  return {
    depthMax: state.depthMax,
    depth: state.depth,
    currentLength: _,
    factor: state.factor,
    currentAngle: state.currentAngle,
    angle: state.angle,
    _:  state._
  }
}

fn Gt2 = (state) => { return setLength(state, state.currentLength * state.factor) }
fn Lt2 = (state) => { return setLength(state, state.currentLength / state.factor) }
fn Add = (state) => { return setAngle(state, rem(int(state.currentAngle - state.angle), 360)) }
fn Sub = (state) => { return setAngle(state, rem(int(state.currentAngle + state.angle), 360)) }

// Only necessary to get around recursion limitations...
fn X = (state, X) => {
  return if state.depth < state.depthMax {
    stateNext = state |> setDepth(%, state.depth + 1)

    // Produce
    // Note:if you need [ and ], just save state to a variable.
    stateNext
      |> F(%, F) |> Sub(%) |> F(%, F)
      |> Add(%) |> Add(%)
      |> F(%, F) |> Sub(%) |> F(%, F)
      |> setDepth(%, stateNext.depth - 1)

  } else {
    // Pass onto the next instruction
    state |> setSketch(%, angledLine({ angle: state.currentAngle, length: state.currentLength }, state._))
  }
}

// Only necessary to get around recursion limitations...
fn F = (state, F) => {

    // Pass onto the next instruction
    state |> setSketch(%, angledLine({ angle: state.currentAngle, length: state.currentLength }, state._))

}

fn LSystem = (args, axioms) => {
  return axioms({
    depthMax: args.iterations,
    depth: 0,
    currentLength: 1.0,
    factor: args.factor,
    currentAngle: 0,
    angle: args.angle,
    _: startSketchAt([0, 0]),
  })
}

LSystem({
  iterations: 2,
  factor: 1.36,
  angle: 60,
}, (_) => {
  result = _ |> F(%, F) |> Add(%) |> Add(%) |> F(%, F) |> Add(%) |> Add(%) |> F(%, F)
  return result._
})

It's quite frustrating that any problem causes CodeMirror to lock up, meaning the user can't get out of the problematic state. #4240 is another example.

 The problem here is the parser is reporting an out-of-bounds source range because it's interpreting text as bytes and not characters. The copyright symbol is technically 3 bytes, but 1 character, making the source range always off by 2. This has to be fixed on the parser side.

We think this is related to the transcoding of UTF-16 JS string to the UTF-8 Rust &str. See wasm_bindgen docs.


How to transform the fisheye camera intrinsic param, include mirror_parameters, distortion_parameters and projection_parameters, into the OPENCV_FISHEYE parameter, such as k1,k2,k3,k4?

Hi,
I found the fisheye camera's parameter was different the OPENCV_FISHEYE's k1,k2, k3, k4.
How to convert it to OPNECV_FISHEYE's parameters?
    def cam2image(self, points):
        """camera coordinate to image plane"""
        points = points.T
        norm = np.linalg.norm(points, axis=1)

        x = points[:, 0] / norm
        y = points[:, 1] / norm
        z = points[:, 2] / norm

        x /= z + self.fi["mirror_parameters"]["xi"]
        y /= z + self.fi["mirror_parameters"]["xi"]

        k1 = self.fi["distortion_parameters"]["k1"]
        k2 = self.fi["distortion_parameters"]["k2"]
        gamma1 = self.fi["projection_parameters"]["gamma1"]
        gamma2 = self.fi["projection_parameters"]["gamma2"]
        u0 = self.fi["projection_parameters"]["u0"]
        v0 = self.fi["projection_parameters"]["v0"]

        ro2 = x * x + y * y
        x *= 1 + k1 * ro2 + k2 * ro2 * ro2
        y *= 1 + k1 * ro2 + k2 * ro2 * ro2

        x = gamma1 * x + u0
        y = gamma2 * y + v0

        return x, y, norm * points[:, 2] / np.abs(points[:, 2])

The OPENCV_FISHEYE's distorted parameters.

Hi, I believe it cannot be simply converted as different camera models are used. We use the camera model proposed by Mei & Rives:
https://www.robots.ox.ac.uk/~cmei/articles/single_viewpoint_calib_mei_07.pdf
According to this discussion, OpenCV uses the Kannala-Brandt camera model:
https://oulu3dvision.github.io/calibgeneric/Kannala_Brandt_calibration.pdf
If you want to use the Kannala-Brandt camera model, calibration is possible using our provided calibration sequences.

for Mei's fisheye model, I believe guys should refer to [CamOdoCal](https://github.com/hengli/camera_model) for more details.

yes, i fixd the dead link~

Kitti 360 calibration also has p1 and p2 as additional distortion parameter apart from k1 and k2. But in the computation above those two parameters are not considered. Is there any specific reason or is the above calculation is outdated


Coming soon cylinder animation

Describe the project you want to add with tech stack

This project contains the coming soon animation.Can you assign this task to me?
Expected behavior

Screenshots (optional)

Additional context (optional )

@MohitGupta121 ,@TusharKesarwani Can you Please assign this task to me?As Gssoc contribution period is going to end..this issues can level me up ..Please assign this issue to me...Please.

@MohitGupta121 ,@TusharKesarwani Can you Please assign this task to me?As Gssoc contribution period is going to end..this issues can level me up ..Please assign this issue to me...Please.


WARNING: pipe closed by peer or os.write(pipe, data) raised exception

Cannot close spider through SIGINT (ctrl+c)
My code:
import scrapy
from scrapy.linkextractors import LinkExtractor

from scrapy_playwright.page import PageMethod

meta={
    'playwright': True,
    'playwright_include_page': True,
    'playwright_page_methods': [PageMethod('wait_for_load_state','networkidle')]
}

async def error_back(failure):
    page = failure.request.meta["playwright_page"]
    await page.close()

class MsuSpider(scrapy.Spider):
    name = "msu"
    allowed_domains = ["msu.ru"]
    start_urls = ["https://msu.ru"]
    custom_settings = {
        'PLAYWRIGHT_BROWSER_TYPE': 'chromium',
        'HTTPCACHE_ENABLED': False,
        "TWISTED_REACTOR": "twisted.internet.asyncioreactor.AsyncioSelectorReactor",
        "DOWNLOAD_HANDLERS": {
            "https": "scrapy_playwright.handler.ScrapyPlaywrightDownloadHandler",
            "http": "scrapy_playwright.handler.ScrapyPlaywrightDownloadHandler",
        }
    }
    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        self.link_extractor = LinkExtractor(unique=True)

    def start_requests(self) :
        start_urls = ["https://msu.ru"]
        for url in start_urls:
            yield scrapy.Request(
                url=url,
                meta=meta,
                callback=self.parse,
                errback=error_back
            )

    async def parse(self, response, **kwargs):
        page = response.meta["playwright_page"]
        screenshot =  await page.screenshot(path=f"{response.url}.png", full_page=True)
        await page.close()

        links_on_current_page = self.link_extractor.extract_links(response)
        for link in links_on_current_page:
              yield response.follow(link.url, callback=self.parse, meta=meta, errback=error_back)

logs:
2024-10-17 14:08:04 [scrapy.crawler] INFO: Received SIGINT, shutting down gracefully. Send again to force
2024-10-17 14:08:04 [scrapy.core.engine] INFO: Closing spider (shutdown)
2024-10-17 14:25:12 [scrapy-playwright] INFO: Launching browser chromium
2024-10-17 14:25:12 [scrapy.core.scraper] ERROR: Error downloading <GET https://msu.ru/ch>
Traceback (most recent call last):
  File "/home/../../../venv/lib/python3.12/site-packages/twisted/internet/defer.py", line 1999, in _inlineCallbacks
    result = context.run(
  File "/home/../../../venv/lib/python3.12/site-packages/twisted/python/failure.py", line 519, in throwExceptionIntoGenerator
    return g.throw(self.value.with_traceback(self.tb))
  File "/home/../../../venv/lib/python3.12/site-packages/scrapy/core/downloader/middleware.py", line 54, in process_request
    return (yield download_func(request=request, spider=spider))
  File "/home/../../../venv/lib/python3.12/site-packages/twisted/internet/defer.py", line 1251, in adapt
    extracted: _SelfResultT | Failure = result.result()
  File "/home/../../../venv/lib/python3.12/site-packages/scrapy_playwright/handler.py", line 378, in _download_request
    return await self._download_request_with_retry(request=request, spider=spider)
  File "/home/../../../venv/lib/python3.12/site-packages/scrapy_playwright/handler.py", line 397, in _download_request_with_retry
    page = await self._create_page(request=request, spider=spider)
  File "/home/../../.../venv/lib/python3.12/site-packages/scrapy_playwright/handler.py", line 296, in _create_page
    ctx_wrapper = await self._create_browser_context(
  File "/home/../../../venv/lib/python3.12/site-packages/scrapy_playwright/handler.py", line 257, in _create_browser_context
    await self._maybe_launch_browser()
  File "/home/../../../venv/lib/python3.12/site-packages/scrapy_playwright/handler.py", line 205, in _maybe_launch_browser
    self.browser = await self.browser_type.launch(**self.config.launch_options)
  File "/home/../../../venv/lib/python3.12/site-packages/playwright/async_api/_generated.py", line 14115, in launch
    await self._impl_obj.launch(
  File "/home/../../../venv/lib/python3.12/site-packages/playwright/_impl/_browser_type.py", line 95, in launch
    Browser, from_channel(await self._channel.send("launch", params))
  File "/home/../../../venv/lib/python3.12/site-packages/playwright/_impl/_connection.py", line 59, in send
    return await self._connection.wrap_api_call(
  File "/home/../../../venv/lib/python3.12/site-packages/playwright/_impl/_connection.py", line 514, in wrap_api_call
    raise rewrite_error(error, f"{parsed_st['apiName']}: {error}") from None
Exception: BrowserType.launch: Connection closed while reading from the driver
2024-10-17 14:25:12 [scrapy.core.scraper] ERROR: Spider error processing <GET https://msu.ru/ch> (referer: https://msu.ru/)
Traceback (most recent call last):
  File "/home/../../../venv/lib/python3.12/site-packages/twisted/internet/defer.py", line 1251, in adapt
    extracted: _SelfResultT | Failure = result.result()
                                        ^^^^^^^^^^^^^^^
  File "/home/../../../../crawlers/spiders/msu.py", line 28, in error_back
    page = failure.request.meta["playwright_page"]
           ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^
KeyError: 'playwright_page'
playwright._impl._errors.TargetClosedError: Page.screenshot: Target page, context or browser has been closed
Call log:
taking page screenshot
  - waiting for fonts to load...
  - fonts loaded

/home/../../../venv/lib/python3.12/site-packages/playwright/driver/package/lib/server/chromium/crPage.js:491
    this._firstNonInitialNavigationCommittedReject(new _errors.TargetClosedError());
                                                   ^

TargetClosedError: Target page, context or browser has been closed
    at FrameSession.dispose (/home/../../../venv/lib/python3.12/site-packages/playwright/driver/package/lib/server/chromium/crPage.js:491:52)
    at CRPage.didClose (/home/../../../venv/lib/python3.12/site-packages/playwright/driver/package/lib/server/chromium/crPage.js:162:60)
    at CRBrowser._onDetachedFromTarget (/home/../../../venv/lib/python3.12/site-packages/playwright/driver/package/lib/server/chromium/crBrowser.js:200:14)
    at CRSession.emit (node:events:519:28)
    at /home/../../../venv/lib/python3.12/site-packages/playwright/driver/package/lib/server/chromium/crConnection.js:160:14

Node.js v20.17.0

2024-10-17 14:08:25 [asyncio] WARNING: pipe closed by peer or os.write(pipe, data) raised exception.
2024-10-17 14:08:28 [asyncio] WARNING: pipe closed by peer or os.write(pipe, data) raised exception.
...
2024-10-17 14:13:58 [scrapy.extensions.logstats] INFO: Crawled 66 pages (at 0 pages/min), scraped 62 items (at 0 items/min)

last lines continues forever

Is this the same as #62?


Implement Array/TypedArray find index

JerryScript-DCO-1.0-Signed-off-by: Daniella Barsony<EMAIL_ADDRESS>

@rerobika Fixed the style errors you mentioned.

@galpeter Added the missing test case


Update phpunit config

We can prevent risky tests, warnings and tests that doesn't test anything.

It seems testDotenvRequiredCanBeUsedWithoutLoadingFile fails.


Handle overlapping labels in needpie

fix #498

Looks like that the useblocks server does not answer to the Linkcheck script executed by the pipeline. Maybe the useblocks server assumes wrongly an attack?

Looks like that the useblocks server does not answer to the Linkcheck script executed by the pipeline. Maybe the useblocks server assumes wrongly an attack?

Yes I know. It is a bug by our provider and ticket is open
Therefore the link check is currently not needed for a merge.


[Code] conductor-common

Common models used by various conductor modules

Utils

RetryUtil

RetryUtil implementation based on guava retrying

run

SearchResult.java

SearchResult Generic

TaskSummary.java

TimeZone is GMT
It represents base task information.

Workflow.java

WorkflowSummary.java

TimeZone is GMT
It represents base workflow information.
This class is not for Definition

metadata

Auditable.java

events

EventExecution.java
EventHandler.java

tasks

PollData.java
Task.java
TaskDef.java
TaskExecLog.java
TaskResult.java

workflow

DynamicForkJoinTask.java
DynamicForkJoinTaskList.java
RerunWorkflowRequest.java
SkipTaskRequest.java
StartWorkflowRequest.java
SubWorkflowParams.java
WorkflowDef.java
WorkflowTask.java

CopyOnWriteArrayList

https://docs.oracle.com/javase/8/docs/api/?java/util/concurrent/CopyOnWriteArrayList.html
A thread-safe variant of ArrayList in which all mutative operations (add, set, and so on) are implemented by making a fresh copy of the underlying array.
useful when you have few writes and many reads


fix slippages

I think there is a bug when computing slippage percentages:
If you want to set amountSlippage to 50 (0.5%)
 const amount0Min = new Fraction(1n)
      .add(amountSlippage)
      .invert()
      .multiply(amount0).quotient

Using the code above gives 1/51 that you multiply  by the amount..
Can you confirm there is an issue? im not 100% sure.
Here is a proposition to fix the issue. The change in addLiquidityParameters and calculateAmountsToRemove signature are intentionnaly breaking to avoid mixing versions..

The computation works fine, the issue was how slippage percentage are defined in the docs. (see 8affb92)
Thanks for noticing out!

@JoeTheCookie217
Can you gives some details about this code ??
 const amount0Min = new Fraction(1n)
      .add(amountSlippage)
      .invert()
      .multiply(amount0).quotient

Is there a reason to add "1" to amountSlippage ?
why not just this ?
 const amount0Min = amountSlippage
      .invert()
      .multiply(amount0).quotient

@JoeTheCookie217
It seems you have the same issue in swapAmountIn and swapAmountOut when calling swapCallParameters

Let's say amountSlippage is 0.005 (0.5%), inverting it equals 200, thus it would result in amount0Min = 200 x amount0
Whereas 1 + 0.005 = 1.005, which is 0.995 once inverted, results in 0.995 x amount0


[App Service] Fix #14758: az webapp create errors when creating windows app with --runtime dotnetcore

Description
Fixes #14758
az webapp create fails when creating a windows app using a dotnetcore runtime.
Testing Guide

Create windows ASP:
az appservice plan create -g RG -n Plan
Create dotnetcore webapp --runtime "DOTNETCORE|2.1" or "DOTNETCORE|3.1":
az webapp create -g RG -p Plan -n Name --runtime "DOTNETCORE|3.1"

Command fails with unexpected error: cli.azure.cli.core.util : 'configs'. Command should have worked successfully

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

App Service


[dataset] updated GAP corpus

Updated ConvoKit version of the GAP corpus, to match the current status of the repository.
Changes to the individual and group level metadata as some erroneous values in the spreadsheets were fixed.
gap-corpus.zip

Hi @ZiggyFloat, I've checked the corpus you uploaded, but it seems to be identical to the one that's currently available for download. (It also isn't using the latest ConvoKit version, which will have the 'vectors' key in most of the component .json files.)


chore(Demo Data):  Assigned Orgs to testOrgSteward1 and added more Co…

…ntact Types #94

I edited this in Postman which helpfully renumbered and rearrange this data.  The actual changes are small.

:+1:

After talking with Clark and an introduction to Postman, :+1:


modify one word

examples   to  example

This change is

@jianglingxia thanks for the contribution. Please sign the CLA with the same email address that you are using w/ GitHub so that we can merge the PR. Thanks


chore(sdk): Revert update to latest trust registry api pr #780

https://github.com/trustbloc/wallet-sdk/pull/793


Should Drogon have built-in cryptographic functions?

Happy new year!
#646 suggested to add security guidelines. I think that's a great idea. Especially if we could make Drogon secure by default. As of now. Drogon currently doesn't come with "security features" per se. Contrast to PHP have a built-in password_hash.
I've password hashing/verification and secure number/string generation upstream-able in my webapp. But I'm not sure if it should be upstreamed. They pull in extra dependencies that the core framework don't need and is trivial to write. And languages like Go and Rust asks user to pull in their own library.
I guess the question is:

Should Drogon provide cryptographic functions? Or is it the user's job?
How much security could we provide without extra dependencies?
If yes, To what extent should Drogon assist users?

I don't expect drogon to support full-on SRP and GPG. That's bloat.
In the mean while, I want to avoid users storing password in the database.

What are your thoughts?
Thanks.

Happy New Year!
This is a good idea. I think any common features of web applications should be considered for adding to Drogon. Is OpenSSL an enough dependency for this?
Plain text passwords shouldn't be stored in the database, instead, we should store something like hash(hash(password)+salt) in the database.
@rbugajewski what do you think about this topic?

Happy New Year!
This is a good idea. I think any common features of web applications should be considered for adding to Drogon. Is OpenSSL an enough dependency for this?
Plain text passwords shouldn't be stored in the database, instead, we should store something like hash(hash(password)+salt) in the database.
@rbugajewski what do you think about this topic?

No, OpenSSL is not enough. It does not support proper password hashes like argon2 and pbkdf2. At best I can make something like sha256(sha256(password)+salt). That's "good enough" but far form secure. I use Botan2. Which will be an extra dependency and not available on older distros (ex: Ubuntu 16.04).

No, OpenSSL is not enough. It does not support proper password hashes like argon2 and pbkdf2. At best I can make something like sha256(sha256(password)+salt). That's "good enough" but far form secure. I use Botan2. Which will be an extra dependency and not available on older distros (ex: Ubuntu 16.04).

Botan2 looks great, It looks like it can replace the openssl. I plan to create Drogon2 project that uses c++20 and boost.asio, The burden of compatibility with the old system can be thrown away in this version. We could discuss the Drogon2 together in another issue.

Botan2 looks great, It looks like it can replace the openssl. I plan to create Drogon2 project that uses c++20 and boost.asio, The burden of compatibility with the old system can be thrown away in this version. We could discuss the Drogon2 together in another issue.

Can't wait for Drogon2! Would that be a new library or based on drogon? I too have a lot want to improve but can't because of backwards compatibility. <insert a paragraph on how long I've tried to add http2>
Anyway, I rather push nothing than a half-backed solution. Maybe postpone this until drogon2?

Can't wait for Drogon2! Would that be a new library or based on drogon? I too have a lot want to improve but can't because of backwards compatibility. <insert a paragraph on how long I've tried to add http2>
Anyway, I rather push nothing than a half-backed solution. Maybe postpone this until drogon2?

As a workaround, maybe creating a plugin project like this can help u. what do you think?

As a workaround, maybe creating a plugin project like this can help u. what do you think?

Botan2 looks great, It looks like it can replace the openssl. I plan to create Drogon2 project that uses c++20 and boost.asio, The burden of compatibility with the old system can be thrown away in this version. We could discuss the Drogon2 together in another issue.
Great, any rough timeline on Drogon2 project?

Botan2 looks great, It looks like it can replace the openssl. I plan to create Drogon2 project that uses c++20 and boost.asio, The burden of compatibility with the old system can be thrown away in this version. We could discuss the Drogon2 together in another issue.
Great, any rough timeline on Drogon2 project?

Botan2 looks great, It looks like it can replace the openssl. I plan to create Drogon2 project that uses c++20 and boost.asio, The burden of compatibility with the old system can be thrown away in this version. We could discuss the Drogon2 together in another issue.

Great, any rough timeline on Drogon2 project?

Not yet, I'm very busy recently. actually I've create the Drogon2 project and implemented most functions of Trantor by boost.asio.

Botan2 looks great, It looks like it can replace the openssl. I plan to create Drogon2 project that uses c++20 and boost.asio, The burden of compatibility with the old system can be thrown away in this version. We could discuss the Drogon2 together in another issue.

Great, any rough timeline on Drogon2 project?

Not yet, I'm very busy recently. actually I've create the Drogon2 project and implemented most functions of Trantor by boost.asio.

What about JWT ? I can make it as contributor for You guys and I need that.

What about JWT ? I can make it as contributor for You guys and I need that.

Great! I look forward to your contribution.
Please tell us more details about your plan. This is a big feature, and we can have some discussion before you proceed.

Big or not, but i started using drogon a few days ago and i missing JWT for my project (API REST), so u know...
There is a lot of good libs on github, like: https://github.com/adhocore/php-jwt JWT for PHP.
JWT for Drogon could be pretty much like this, same methods, same structure, etc, what u think?

I think we should make a JWTPlugin and a JWTFilter for this.

I think we should make a JWTPlugin and a JWTFilter for this.

should be good

im working on some private project based on drogon. I need hash functions for security: eg. store passwords in database, etc.
I think we need add this future, cuz for now i need add some cpp/hpp files my own to drogon app to use sha256.
Easy way: http://www.zedwood.com/article/cpp-sha256-function
But is it good for us? or just add new dependency to crypto++
if we think about plugin for hashing with about 200 lines maybe this should be implemented in drogon framework. If there could be more lines and features maybe plugin will be better?

@roq3 I've discussed with @an-tao and concluded that we shouldn't add additional dependencies to Drogon's core library. As the embedded users may not have the resource. And writing an supplementary then include/link it in your project will be the proper approach.
This is the password hashing and verification code I use in my private projects. Which additionally depends on libbsd and Botan2. Hope it helps.
https://gist.github.com/marty1885/6615102db63ba879ee51b89537e332f0
Hashing passwords with SHA256 is a bad idea. Use something like Argon2 or PBKDF2.

I also think security related functionality should be kept outside of upstream. We already have enough open ends, features to implement, bugs to fix, and there are not enough volunteers in my opinion to deliver everything in a timely manner. By adding hashing functionality, even if this is “just” 200 LOCs, we will add maintenance burden to the currently active developers, and in the worst case open a can full of security holes.

https://gist.github.com/marty1885/6615102db63ba879ee51b89537e332f0

thx, i will try this.
SHA256 with salt for password hashing should be ok, but i will change to what U suggest.

Closing the issue as we decided to not add dependencies. Fell free to reopen.

Don't add dependencies into the core if u don't wish. But in my opinion, it's good to force the drogon users to use dependency manager (Conan & Vcpkg) for their colourful web application development. This is 2021 and every language is going well with dependency manager. C++ is only one left a little behind. We need to move our dev culture forward.

Any recommendations on how I can issue tokens that are safe?
Everyone says to use JWT tokens, is UUIDv4 safe enough which in included into the drogon web framework?
Thanks.

@rajhlinux JWT is more versatile, yes. I've been considering adding support it in the drogon-assist library. But I'm working on other major improvements right now. UUIDv4 is good enough for most applications. Personally I use secure random + base64.
std::vector<uint8_t> vec(16);
utils::secureRandomBytes(vec.data(), vec.size());
auto token = utils::base64Encode(vec.data(), vec.size());

Or use the secure random string in assist.
auto token = drassist::secureRandomString(32);


Store Trained Model Reference

Good day, how exactly can we store the trained model and use it in a  custom API. The way it is currently it will always load a pickled model and train ?

This is a general question about TensorFlow. Please ask on StackOverflow.
The following message came up before you posted your question, please read it:
https://github.com/Hvass-Labs/TensorFlow-Tutorials/blob/master/.github/ISSUE_TEMPLATE.md


[ERROR] Fix a typo in Chinese install guide

Note: This repository is ONLY used to solve issues related to DOCS.
For other issues, please move to other repositories.

Where is the error
What is the URL/path of the document related to this issue?
https://www.milvus.io/docs/zh-CN/userguide/install_milvus/
How would you like to improve it
若要拉取支持 CPU 的镜像： should be 若要拉取支持 GPU 的镜像：。

Fixed.


Add Auphonic Leveler v1.3

Sorry closing this the download link is behind a login form. While testing I was logged in.


qt: 4.1 - Get my words regression

Update: Get my words works fine in dark mode. Must a style issue in Light.css.

Update 2: Get my words working fine on latest 4.1 branch. Need to test on non-HD wallets. If everything works then I will close this issue.

Closing since issue not present in latest 4.1 branch.


Redesign side panels

With the drone launcher and the deadwheels, the side panels need to be redesigned. (the dead wheels could work with minor alterations but I want to see what the drone launcher needs before changing those)

also look at accommodating #53 if needed

Relevant parts of the Game Manual:

<RG03> Team Number Display - Robots must prominently display their Team number (numerals only, for
example “12345”) on two separate signs. The intent of this rule is for field personnel to easily identify Robots
by Team number from at least 12 feet (3.66 meters) away.
a. Team number must be visible from at least two opposite sides of the Robot (180 degrees apart).
b. The numerals must each be at least 2.5 inches (6.35 cm) high and in a contrasting color from their
background.
c. Team numbers must be robust enough to withstand the rigors of Match play. Example robust materials
include: 1) self-adhesive numbers (mailbox or vinyl numbers) mounted on polycarbonate sheet, wood
panel, metal plate, etc. or 2) Ink jet or laser printed numbers on paper and laminated.
d. If used, illuminated Team numbers must be legible when they are not powered on.
<RG04> Alliance Marker – Robots must include a Team supplied, Alliance specific marker on two opposite
sides of the Robot to easily identify which Alliance a Robot is assigned to. The Alliance marker must be
displayed on the same side of the Robot as the Team number, within a 3-inch (7.62 cm) distance of the
number. The intent of this rule is to allow field personnel to easily identify the Robot’s Alliance.
a. The red Alliance marker must be a solid red square, 2.5 inches x 2.5 inches (6.35 cm x 6.35 cm) +/-
0.25 inches (0.64 cm).
b. The blue Alliance marker must be a solid blue circle, 2.5 inches (6.35 cm) +/- 0.25 inches (0.64 cm) in
diameter.
c. The Alliance marker must be visible to the referees during a Match and must indicate the Alliance color
for the Match.
d. The Alliance marker must be robust enough to withstand the rigors of Match play. Example robust
materials include: 1) Alliance marker template printed and laminated; 2) painted or gaff tape covered
polycarbonate sheet, wood panel, metal plate, etc

also: make room for sponsor logos

also: ideally don't require removal of the motors to attach

for now this is blocked for a bit until we figure out what else we need


[MLS Spec change] Add message/mls MIME type registration (#581)

Link to the exact changes
https://github.com/mlswg/mls-protocol/pull/581
Description of the changes

This is a minimal section for MIME type registration of message/mls.
It assumes that PR #523 include wire_format in all MLS messages (making a format parameter unnecessary),
and that all messages have the MLS version number encoded (making a version parameter unnecessary).
These parameters can be added if needed.

There's nothing to do here but I'll leave it open to make sure it's on the radar when working on #979.


Failed to load the native TensorFlow runtime in Mac OSX

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

Mac OSX 10.11.6
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed with pip
TensorFlow version:  1.12.0
Python version: 3.6.6

Describe the problem
Everything was ok with an older version, but with the current version install pass ok but when try to import, many errors are produced.
Provide the exact sequence of commands / steps that you executed before running into the problem
import tensorflow
Any other info / logs
Traceback (most recent call last):
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "/Users/mikko/dev/wip/lib/python3.6/imp.py", line 243, in load_module
    return load_dynamic(name, filename, file)
  File "/Users/mikko/dev/wip/lib/python3.6/imp.py", line 343, in load_dynamic
    return _load(spec)
ImportError: dlopen(/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so, 6): Symbol not found: _clock_gettime
  Referenced from: /Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/../libtensorflow_framework.so (which was built for Mac OS X 10.12)
  Expected in: /usr/lib/libSystem.B.dylib
 in /Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/../libtensorflow_framework.so

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/__init__.py", line 49, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow.py", line 74, in <module>
    raise ImportError(msg)
ImportError: Traceback (most recent call last):
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "/Users/mikko/dev/wip/lib/python3.6/imp.py", line 243, in load_module
    return load_dynamic(name, filename, file)
  File "/Users/mikko/dev/wip/lib/python3.6/imp.py", line 343, in load_dynamic
    return _load(spec)
ImportError: dlopen(/Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so, 6): Symbol not found: _clock_gettime
  Referenced from: /Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/../libtensorflow_framework.so (which was built for Mac OS X 10.12)
  Expected in: /usr/lib/libSystem.B.dylib
 in /Users/mikko/dev/wip/lib/python3.6/site-packages/tensorflow/python/../libtensorflow_framework.so

Failed to load the native TensorFlow runtime.

See https://www.tensorflow.org/install/errors

for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

I am getting the same error, While trying to pip install 1.5 tensorflow i get this error.
Collecting tensorflow==1.5
Could not find a version that satisfies the requirement tensorflow==1.5 (from versions: 1.13.0rc1, 1.13.0rc2, 1.13.1, 1.13.2, 1.14.0rc0, 1.14.0rc1, 1.14.0, 1.15.0rc0, 1.15.0rc1, 1.15.0rc2, 1.15.0rc3, 1.15.0, 2.0.0a0, 2.0.0b0, 2.0.0b1, 2.0.0rc0, 2.0.0rc1, 2.0.0rc2, 2.0.0)
No matching distribution found for tensorflow==1.5
is there any other workaround?


feat: change av-* module demo bundle config and scripts

Description

change av-* module  demo bundle config and scripts
change demo.config.ts to vite.config.demo.ts
add preview:demo scripts
av-cliper demo static resources use doc-site

@yzydeveloper What is the purpose of this pull request (PR)?

@yzydeveloper What is the purpose of this PR?

av-cliper demo static resources use doc-site

@yzydeveloper
build:demo did encounter some issues. I've now migrated the DEMO build to the docs-site directory for an improved experience.
I've also removed relevant code under the av-* modules, retaining only the code necessary for running the DEMO for debugging purposes.

av-cliper demo static resources use doc-site

Since the migration was not completed yesterday, there were two sets of test resources at that time.  [doge]

av-cliper demo static resources use doc-site

Since the migration was not completed yesterday, there were two sets of test resources at that time. [doge]

Is it a symbolic link? It doesn't work local

Closed ❌


Why create a "global_part_neighbor" in the function "find_neighbor" in the skeleton.py?

Hi,
I was recently reading the source code very carefully in order to understand the whole pipeline of your work.
I was kind of confused that why init a variable named "global_part_neighbor" in the  in the function "find_neighbor" in the line #373 of the models/skeleton.py?
Besides, why append a "edge_num" into those global_part_neighbor's list  in line #375?
Many thanks!

Hi,
Line 373 is because we treat the global position channel as an "extra joint" and it shares the neighbor of the root joint, which is exactly global_part_nieghbor.
For line 375, the "neighbor" relation is symmetric, so we add the extra joint (its index is edge_num, the last one) to the neighbor_list of its neighbors.
Hope this will help.

Hi,
Line 373 is because we treat the global position channel as an "extra joint" and it shares the neighbor of the root joint, which is exactly global_part_neighbor.
For line 375, the "neighbor" relation is symmetric, so we add the extra joint (its index is edge_num, the last one) to the neighbor_list of its neighbors.
Hope this will help.

Got it !Thank you so much for the answer!

Hi, I recently found that there might be a mistake when calculating the neighbors of the "global_part_neighbor".
e.g. If you take read the .bvh file of "BigVegas", you can use debug mode of the IDE to found that the 22th element of the list which returned by the "find_neighbor" function is not the neighbors of the root joint(if you draw a graph of these 23 bones). It should be [0, 1, 4, 5, 8, 9], but got [0, 1, 2, 4, 5, 8, 9] instead.
I thinks it is because: at first, the index of the root joint is 0, but the "edges" eliminate the root joint, so when we obtain the neighbor_list  after the for loop in line #370 of the skeleton.py, the mistake occurs.
 global_part_neighbor = neighbor_list[0].copy()
is not the neighbors of the root joint. I think it represents the neighbors of the "LeftUpLeg" of the "BigVegas"'s skeleton.(you could draw out the skeleton graph of the "BigVegas"'s 23 bones to verify it.)

Thank you very mush for providing your findings! I think it's indeed a bug and we'll fix it later.

Can you describe a way for fixing this? I can retrain my models.

global_part_neighbor should be all the edges(armature) that contains joint with index zero. This condition should be used instead of a copy of neighbor_list[0].

So if neighbor_list = [] how can this be fixed?

global_part_neighbor = [] #新增的全局根节点
#记录与根节点距离小于等于d的骨骼

for edge in edges:
    if edge[0] == 0:
        for i,j in enumerate(edge_mat(edge[1])):
            if j < d:
                global_part_neighbor.append(i)

@PeizhuoLi I'm having trouble with neighbor_list = []

Could you provide more details on how to reproduce this error? Thanks :)


python3Packages.losant-rest: 1.15.1 -> 1.15.2

Motivation for this change
Update to latest upstream release 1.15.2
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 156987 run on x86_64-linux 1

2 packages built:

python310Packages.losant-rest
python39Packages.losant-rest


[SASS-400] Aligning session keys with V&C

Journey tests need to be merged first

Can we address these warns where appropriate and also rebase please.


Add transaction direction

Add transaction direction, which was outgoing by default + tests

Let's check that len(event.Topics[1]) > 3 before event.Topics[1] or event.Topics[2] to prevent any possibilty to have an error

We do have it already https://github.com/trustwallet/blockatlas/blob/master/platform/vechain/transaction.go#L89


Data for Enrollment and Development

Should the data for Enrollment and Development be same?

No. The subjects are the same, however.


Problems with Directory Handling for -d,--dir Flag in NullAway Annotator

Describe the bug
I encountered issues when using the required -d,--dir  flag to specify the directory for output serialization in NullAway Annotator. The tool expects a nonnull_elements.tsv file in the directory, but handling this file results in various exceptions.
To Reproduce
Steps to reproduce the behavior:

Run the NullAway Annotator with the Directory:
java -jar /path/to/annotator-core-1.3.14.jar -d "nullawayResults" .....

Encounter NoSuchFileException:
java.nio.file.NoSuchFileException: nullawayResults/0/nonnull_elements.tsv

Create the missing File:
touch nullawayResults/0/nonnull_elements.tsv

Run the NullAway Annotator Again:
java -jar /path/to/annotator-core-1.3.14.jar -d "nullawayResults" .....

Encounter DirectoryNotEmptyException:
java.nio.file.DirectoryNotEmptyException: nullawayResults/0

Expected behavior
The tool should either:

Create the nonnull_elements.tsv file if it does not exist, or
Properly handle an empty directory without requiring pre-existing files

Stack trace
Stack trace for DirectoryNotEmptyException:
Exception in thread "main" java.lang.RuntimeException: java.nio.file.DirectoryNotEmptyException: nullawayResults/0
	at edu.ucr.cs.riple.core.module.ModuleConfiguration.<init>(ModuleConfiguration.java:74)
	at edu.ucr.cs.riple.core.Config.lambda$new$0(Config.java:418)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197)
	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1625)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499)
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682)
	at edu.ucr.cs.riple.core.Config.<init>(Config.java:420)
	at edu.ucr.cs.riple.core.Main.main(Main.java:43)
Caused by: java.nio.file.DirectoryNotEmptyException: nullawayResults/0
	at java.base/sun.nio.fs.UnixFileSystemProvider.implDelete(UnixFileSystemProvider.java:246)
	at java.base/sun.nio.fs.AbstractFileSystemProvider.deleteIfExists(AbstractFileSystemProvider.java:110)
	at java.base/java.nio.file.Files.deleteIfExists(Files.java:1191)
	at edu.ucr.cs.riple.core.module.ModuleConfiguration.<init>(ModuleConfiguration.java:68)

Stack trace for NoSuchFileException:
Exception in thread "main" java.lang.RuntimeException: Error happened while loading content of file: nullawayResults/0/nonnull_elements.tsv
	at edu.ucr.cs.riple.core.registries.Registry.lambda$new$0(Registry.java:93)
	at java.base/java.lang.Iterable.forEach(Iterable.java:75)
	at edu.ucr.cs.riple.core.registries.Registry.<init>(Registry.java:88)
	at edu.ucr.cs.riple.core.registries.index.NonnullStore.<init>(NonnullStore.java:42)
	at edu.ucr.cs.riple.core.module.ModuleInfo.<init>(ModuleInfo.java:90)
	at edu.ucr.cs.riple.core.module.ModuleInfo.<init>(ModuleInfo.java:73)
	at edu.ucr.cs.riple.core.Context.<init>(Context.java:78)
	at edu.ucr.cs.riple.core.Annotator.<init>(Annotator.java:63)
	at edu.ucr.cs.riple.core.Main.main(Main.java:45)
Caused by: java.nio.file.NoSuchFileException: nullawayResults/0/nonnull_elements.tsv
	at java.base/sun.nio.fs.UnixException.translateToIOException(UnixException.java:92)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:106)
	at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111)
	at java.base/sun.nio.fs.UnixFileSystemProvider.newByteChannel(UnixFileSystemProvider.java:218)
	at java.base/java.nio.file.Files.newByteChannel(Files.java:380)
	at java.base/java.nio.file.Files.newByteChannel(Files.java:432)
	at java.base/java.nio.file.spi.FileSystemProvider.newInputStream(FileSystemProvider.java:422)
	at java.base/java.nio.file.Files.newInputStream(Files.java:160)
	at java.base/java.nio.file.Files.newBufferedReader(Files.java:2922)
	at edu.ucr.cs.riple.core.registries.Registry.populateContent(Registry.java:114)
	at edu.ucr.cs.riple.core.registries.Registry.lambda$new$0(Registry.java:91)

OS:

OS: macOS
Version: Sonoma 14.5

Annotator Version

Annotator Version: 1.3.14
NullAway Version: 3.40.0

Additional context
My understanding is that the specified directory should be any directory I wish to use for storing outputs. This directory should be a new, empty directory to be used by the tool.

@akbarkanso thanks for your report.  We're a bit busy with deadlines until August 2, but after that @nimakarimipour will take a look.

Hi @nimakarimipour, did you get a chance to check the above problem?

Hi @akbarkanso,
Thank you for reporting this issue, and apologies for the delayed response—we were tied up with a deadline but are now available to assist. From your bug report, it seems that the AnnotatorScanner may not be configured correctly. The error message nonnull_elements.tsv not found suggests this, as nonnull_elements.tsv and several other files are generated by AnnotatorScanner.
AnnotatorScanner is a plugin for Error Prone, similar to NullAway, and must be added to the build configuration before running Annotator. Once the plugin is properly added, Annotator will compile the module, and the required files should be generated by AnnotatorScanner. If you've already added AnnotatorScanner, I recommend the following steps:

Ensure that the path specified in AnnotatorScanner:ConfigPath in the build configuration matches the corresponding value in paths.tsv.
Rerun Annotator using -rboserr or --redirect-build-output-stderr to redirect compilation output. This will help identify any crashes in the Scanner or errors in the build configuration. That would be very helpful if could provide this for us. You should expect to see NullAway warnings in the build output.

Regarding the directory not empty exception, this is likely due to manually creating nonnull_elements.tsv using the touch command. We actually prefer that users always pass an empty directory to Annotator, as it generates intermediate and large files that are unnecessary after execution. This allows you to easily delete the entire directory afterward without searching for individual files.
We will update the README to include more detailed instructions and a sample project for running Annotator. Please let me know if you have any additional information or need further assistance, and sorry for any inconvenience caused.

Hi @nimakarimipour ,
Thanks for the detailed reply.
I double-checked my configuration and re-ran the process, but I’m still encountering the same exception:
Exception in thread "main" java.lang.RuntimeException: Error happened while loading content of file: outDirN/0/nonnull_elements.tsv
        at edu.ucr.cs.riple.core.registries.Registry.lambda$new$0(Registry.java:93)
        at java.base/java.lang.Iterable.forEach(Iterable.java:75)
        at edu.ucr.cs.riple.core.registries.Registry.<init>(Registry.java:88)
        at edu.ucr.cs.riple.core.registries.index.NonnullStore.<init>(NonnullStore.java:42)
        at edu.ucr.cs.riple.core.module.ModuleInfo.<init>(ModuleInfo.java:90)
        at edu.ucr.cs.riple.core.module.ModuleInfo.<init>(ModuleInfo.java:73)
        at edu.ucr.cs.riple.core.Context.<init>(Context.java:78)
        at edu.ucr.cs.riple.core.Annotator.<init>(Annotator.java:63)
        at edu.ucr.cs.riple.core.Main.main(Main.java:45)
Caused by: java.nio.file.NoSuchFileException: outDirN/0/nonnull_elements.tsv
        at java.base/sun.nio.fs.UnixException.translateToIOException(UnixException.java:92)
        at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:106)
        at java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111)
        at java.base/sun.nio.fs.UnixFileSystemProvider.newByteChannel(UnixFileSystemProvider.java:218)
        at java.base/java.nio.file.Files.newByteChannel(Files.java:380)
        at java.base/java.nio.file.Files.newByteChannel(Files.java:432)
        at java.base/java.nio.file.spi.FileSystemProvider.newInputStream(FileSystemProvider.java:422)
        at java.base/java.nio.file.Files.newInputStream(Files.java:160)
        at java.base/java.nio.file.Files.newBufferedReader(Files.java:2922)
        at edu.ucr.cs.riple.core.registries.Registry.populateContent(Registry.java:114)
        at edu.ucr.cs.riple.core.registries.Registry.lambda$new$0(Registry.java:91)
        ... 8 more

My current configuration is as follows:
I am adding the following to my compilation args:
-XepOpt:NullAway:FixSerializationConfigPath=/path/root/outDirN/nullaway_config.xml
-XepOpt:AnnotatorScanner:ConfigPath=/path/root/outDirN/nullaway_scanner_config.xml
My paths.tsv file available in /path/root/ content:
/path/root/outDirN/nullaway_config.xml	/path/root/outDirN/nullaway_scanner_config.xml

the command I am running at root:
java -jar /path/annotator-core-1.3.14.jar -cn NULLAWAY -d "outDirN" -i com.Initializer  -cp paths.tsv -bc "cd module && gradle compileAllJava" -rboserr
The gradle configuration is passing, and at the end of gradle execution, I can see that the task is failing due to current NullAway violations. In the terminal, after the build result, I see the nonnull_elements.tsv exception mentioned above.

Additional information:
If I run the gradle compileAllJava command directly instead of java -jar /path/annotator-core-1.3.14.jar, I notice that the AnnotatorScanner fails to create the outDirN directory, resulting in the following exception:
NoSuchFileException: /path/root/outDirN/nullaway_scanner_config.xml

However, running java -jar /path/annotator-core-1.3.14.jar ...  does create the outDirN directory along with the nullaway_scanner_config.xml and nullaway_config.xml files beside the 0 directory, but it fails to create the outDirN/0/nonnull_elements.tsv file.

Thanks for the further info, @akbarkanso.  @nimakarimipour maybe we should land #239 and cut a release, so that @akbarkanso can provide more detailed info on what is going on here?

Hi @nimakarimipour,
I tried with the latest release, and indeed it fails in the recent code you introduced. The failure occurs after checking for the existence of outDirN/0/nonnull_elements.tsv, which is not being created as mentioned in my previous comments.
Error message:
AnnotatorScanner is not correctly configured for the module: target.
Please verify that the path specified for AnnotatorScanner on line 0 in the configuration file matches the path provided in file with -cp/--config-paths, and that it is identical to the path specified with -XepOpt:AnnotatorScanner:ConfigPath.
If the path is set correctly, rerun annotator with -rboserr/--redirect-build-output-stderr flag and check compilation output and ensure NullAway and AnnotatorScanner is executing properly.

I have a feeling that AnnotatorScanner is not configured correctly, but I haven't been able to figure out what I did wrong. I already shared my configuration in my pervious comment.

@akbarkanso

Rerun Annotator using -rboserr or --redirect-build-output-stderr to redirect compilation output. This will help identify any crashes in the Scanner or errors in the build configuration. That would be very helpful if could provide this for us. You should expect to see NullAway warnings in the build output.

Just want to check, did you run with this option enabled?  I'm wondering if there is a build failure.  If you could run with this output and see if you notice any build failures or crashes that would be great.

@akbarkanso does your Gradle project have multiple modules or a single module?  Sorry we haven't been able to track this down.  If there is any way you can create an open-source skeleton project that exhibits the same crashing behavior, that would help us a lot.