syncthingtray-qt6: 1.4.13 -> 1.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Martchus/syncthingtray/releases.
meta.description for syncthingtray-qt6 is: Tray application and Dolphin/Plasma integration for Syncthing
meta.homepage for syncthingtray-qt6 is: https://github.com/Martchus/syncthingtray
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.5.0 with grep in /nix/store/b1l2hd17d7d3mgmwcqbi2g6m114fq8cx-syncthingtray-1.5.0
found 1.5.0 in filename of file in /nix/store/b1l2hd17d7d3mgmwcqbi2g6m114fq8cx-syncthingtray-1.5.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
syncthingtray
syncthingtray-minimal
syncthingtray-qt6

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/b1l2hd17d7d3mgmwcqbi2g6m114fq8cx-syncthingtray-1.5.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 syncthingtray-qt6 https://github.com/r-ryantm/nixpkgs/archive/31becf97cf82b22b874cfc275f1513dbb7cdc1f2.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/31becf97cf82b22b874cfc275f1513dbb7cdc1f2#syncthingtray-qt6

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/b1l2hd17d7d3mgmwcqbi2g6m114fq8cx-syncthingtray-1.5.0
ls -la /nix/store/b1l2hd17d7d3mgmwcqbi2g6m114fq8cx-syncthingtray-1.5.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 --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

3 packages built:

syncthingtray
syncthingtray-minimal
syncthingtray-qt6

Maintainer pings
cc @doronbehar for testing.

duplicate of https://github.com/NixOS/nixpkgs/pull/293550

@doronbehar Don't close bot PRs until the update was merged in another PR. @r-ryantm will just keep opening new PRs with this update.


how build tensorflow-lite

Environment (Hardware)

Hardware: Device, CPU, GPU, etc.
Software: OS, Compiler, etc.
(Please include version information)

Issue Details
A clear and concise description of what the issue is.
How to Reproduce
Steps to reproduce the behavior. Please include your cmake command.
Error Log
error log

Additional Information
Add any other context about the problem here.

https://www.tensorflow.org/lite/guide/build_cmake


fix some typo on log_file_test.go

I think maybe exist some typo for test cases name

add HMGet https://github.com/flower-corp/rosedb/issues/142


No Issue, Just thanks

Thanks so much for making this... all the methods listed in various guides didn't have the full names of a few files... this worked great!

Happy to hear you found it useful !


feat: Simplified System-Managed-Identity output

Description

Simplified System-Managed-Identity output
Successful output (max):

Successful output (min):

Pipeline

@AlexanderSehr - I would expect to see a version bump (Patch) with these changes?

Hey @ChrisSidebotham if there'd be a feature change I'd agree. But the output is literally the same. The implementation is nothing more than a shorter version of the same code using the capabilities offered by Bicep

Hey @ChrisSidebotham if there'd be a feature change I'd agree. But the output is literally the same. The implementation is nothing more than a shorter version of the same code using the capabilities offered by Bicep

Ok agreed. In the eyes of semantic versioning this change is completely internal and does not affect the modules interface or behaviour to an end user.
Reviewing again....


CourseLeaf Round 2 Testing - Missing Credit Values + Incorrect Credit Unit Types

I tested 2 CourseLeaf catalogs; this was second round testing.
South Texas College
URL: https://catalog.southtexascollege.edu/courses/
Link to output file: https://docs.google.com/spreadsheets/d/1p0EZ23zV-I8qPST0MBv9sU6SK7McXV2C/edit?usp=sharing&ouid=115685232190749733039&rtpof=true&sd=true
Number of courses look good
Data looks good
ISSUE missing credit values (`3% of records), incorrect credit value type + a small number of course descriptions are truncated
Example: no credit values included in extract, but listed in catalog (https://catalog.southtexascollege.edu/courses/rbtc/)

Example: incorrect credit value type, nothing on page suggests semester (https://catalog.southtexascollege.edu/courses/math/)

These issues seem to be isolated to full sets of courses under a heading (RBTC, MATH, etc.)
There are also a few course descriptions that are truncated:

Here is what appears on the page for these courses:

Deleware Community College
URL: https://catalog.dccc.edu/courses/course-descriptions/
Link to output file: https://docs.google.com/spreadsheets/d/1mr3Aqjr3hw0p5ScvfV-rX90mnmSeQgoV/edit?usp=sharing&ouid=115685232190749733039&rtpof=true&sd=true
Number of courses look good
Data looks good
ISSUE missing credit values (17% of records), incorrect credit value type + truncated course descriptions; I'm not providing screen shots, since it's exactly the same as the above but the output file has some examples highlighted
If there is a pattern to the missing credit values or truncated descriptions, I haven't found it.

@rsaksida Re: your slack question
Can you help me understand how the credits are supposed to be parsed?
CRT HRS:4   LEC HRS:4   LAB HRS:1   OTH HRS:0
I assume this means
Credit hrs - 4
Lecture hrs - 4
Lab hrs - 1
Other hrs - 0
How would this translate to the 4 columns we're using:
Credit Unit Value
Credit Unit Max Value
Credit Unit Type
Credit Unit Type Description_
All of the MATH courses (row 547-563 in the download file) assumed a course unit type = semester hour, when this was not the case for other courses. (Nothing on the MATH URL suggests semester, which is what I thought you'd adjusted the crawler to look for.)
See a constrasting example for instance row 546:

Here is what the URL for that course displays:

Similar display of data between these two examples, yet 546 was parsed correctly:
Credit Unit Value = 4
Credit Unit Max Value
Credit Unit Type
Credit Unit Type Description = This has credit value, but the type cannot be determined
I expected the all of the MATH courses (row 547-563) to be parsed the same way.


How to change item order  by code instead of using  draging gesture

I would like to change item order with animation by code, but i can not find any api to do this.
Would you please tell me how can i change order without using draging gesture

This is not a supported use case. If you really wanted to, you could manually drive the animated values that are currently driven by gestures, and should end up with the same result, but that'd be a lot of work to wire up.


Reordering page thumbnails and the grid view

Closes #746
Closes #332
Closes #517
Uses the same generalized <Reorderable/> component in the layers panel, the page thumbnails and the grid view
/cc @dvoytenko

@dvoytenko @barklund this is ready for review. It seems like we might have overcomplicated the scrolling behavior in #551 , a simple separator.scrollIntoView() works pretty well here!

If we're good with using scrollIntoView for the scroll behavior, this should be good to go

LG with one nit.

Though I don't have a strong opinion on scrollIntoView. IMHO it's ok, but @barklund might feel different.

There is a couple of related warnings in the console:
Received NaN for the `width` attribute. If this is expected, cast the value to a string.
    in div (created by reorderableSeparator__Wrapper)

...

Warning: Each child in a list should have a unique "key" prop.
Check the render method of `Carousel`. See https://fb.me/react-warning-keys for more information.
...

@swissspidy @merapi ready :)

The "dead zone" is still there?

@merapi Sorry, I refactored and totally forgot about that. Should be fixed now :))

@merapi Sorry, I refactored and totally forgot about that. Should be fixed now :))

Can you check the Grid view? I think we need a fix there too.

A couple more bugs specifically related to grid view (but not directly related to any line of code as far as I can see):

You cannot scroll in the modal, even if you in the default view can't see the all the pages (I have 11):

Even without dragging anything, the reordering separators show up at the ends of the rows:

As you reorder, there's no separator before the first item in the second (and subsequent) rows:

I don't know if they're out of scope for this ticket, though. 1. is potentially a general modal bug (with a potentially simple fix).

Did a refactor (went back to using @barklund 's scroller approach since it was smoother) and fixed all of those issues :)

LG!
Just one thing:
I can see the scrollbar (when not needed) in Gridview (Chrome/FF) and I think its style should match other scrollbars.


Fix for High Risk OpenSSL Security Vulnerabilities Announced – Guidance for vcpkg Users

OpenSSL.org announced the release of OpenSSL 3.0.7 to address two security vulnerabilities rated as high risk. This patch is now available, including via vcpkg. The vulnerabilities impact users of OpenSSL 3.0.0 – 3.0.6. If you are relying on a version of OpenSSL in this range, it is strongly recommended to upgrade to 3.0.7 as soon as possible.
If you are a vcpkg user or port author depending on the OpenSSL vcpkg port, see this blog post to learn how to upgrade to the new version: https://devblogs.microsoft.com/cppblog/fix-for-high-risk-openssl-security-vulnerabilities-announced-guidance-for-vcpkg-users/

Port openssl is at version 3.0.8 now. Can this be closed?

Since this issue has been fixed. So I am closing this issue for now.


feat: substitute type parameter when type checking calls

Closes #915
Summary of Changes

Substitute type parameter when type checking calls.
Remove lenient type checking mode. We now use the strict mode everywhere, which means when calling isSubtypeOf(type, other) that the upper bound of type must be a subtype of the lower bound of other.

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

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.8.0
GitHub release

Your semantic-release bot :package::rocket:


docs(#321): better tweet

closes #321

v11.10.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Add support for docker image management

support for add/update/delete docker images
https://github.com/waikato-ufdl/ufdl-backend/issues/19

Added with 2cf7971631106cffd2f99901a830f2f528e709f0


Do not merge: update containers/image

Update to currently-proposed changes for containers/image.  This is mainly a placeholder so that we can be extra-sure that those changes won't negatively impact cri-o.

/test all

/test all

/test all

/test e2e_fedora

/retest

This is fine but we need to wait for containers/image#305 still

LGTM other then the containers/image stuff.

Had to redo some of the bits of #654 that were reverted.

/test all

/test all

/retest

/retest

/test all

/retest

/test ami

/retest

/test all

/test all

/test e2e

/test all

/test e2e

/test e2e

/retest

/test all

/retest

/test all

CI looks down, I'll re-trigger once it's back up

/test all

/test critest

/test critest

/test ami

/test all
/test ami

failure in ami_rhel seems to be because of a runc bug, I'll create another PR to bump runc

/test ami_rhel


Next JS - Revalidate missing from some pages

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
d2skhgz4a0rx0t
AWS Region
eu-west-1
Amplify Hosting feature
Deployments, Frontend builds
Frontend framework
Next.js
Next.js version
12
Next.js router
Pages Router
Describe the bug
We migrated to next 12 with web-compute month ago. We had issue but since 2 month all work well.
Since today (25 july) in some page in my stack transation from server side are missing.
We noticed that after a deploy (of same build of last days).
We use next-i18next for transations. For pages with revalidate or pages using getServerSideProps options it seems that transations are missing.
Other pages witt getStaticProps and without revalide works. using next-i18-next require this code for pages:
export const getStaticProps: GetStaticProps = async () => {
    const locale: T_Locale = "it";

    return {
        props: {
            locale,
            ...(await serverSideTranslations("it", ["common"])),
        },
    };
};

Expected behavior
1- Translations are available in all pages.
2. I can see a log with error
Reproduction steps
Init a next 12 with pages directory app
install next-i18-next.
Create a page with revalidate.
This happend only on amplify
Build Settings
No response
Log output

# Put your logs below this line

Additional information
No response

Hi @multivoltage, I'm not very familiar with the next-i18-next library. Where do you put your translation json files? Are they located in the public folder?

Hi, I cannot reproduce your issue by setting revalidate or using getServerSideProps. This is my test app: https://main.d1rhug8t3v6e4r.amplifyapp.com and
I'm using example app from next-i18-next: https://github.com/Victorccding/simple-i18n-next
Can you help me to reproduce the issue? Thanks

@victorccccc thanks.
I fixed setting localPath to path resolve(something), like the docs write.
But I'm very confused since amplify change something and this fix become mandatory in my codebase.
Can you provide some info about a change?

I was facing the same problem a few days ago, but setting up localePath in next-i18next.config.js solved the problem.

Yes, but did you solve in a "random way" reading stackoverflow or you read some log in some point?

Yes, I found it in a "random way".

Closing the thread since we were not able to reproduce this issue and based on the previous comments it seems to have been resolved.


datasette --root running in Docker doesn't reliably show the magic URL

I followed these steps:
docker run datasetteproject/datasette  pip install datasette-upload-csvs
docker commit $(docker ps -lq) datasette-with-plugins
docker run -p 8001:8001 -v $(pwd):/mnt   datasette-with-plugins   datasette --root -p 8001 -h <IP_ADDRESS>
Visited: http://<IP_ADDRESS>:8001/-/plugins

Visited: http://localhost:8001/-/upload-csvs

I may have missed a step?
Thank you.

Ubuntu 22.04.1 LTS

Confirmed, I just got the same result:
- % docker run datasetteproject/datasette pip install datasette-upload-csvs
~ % docker commit $(docker ps -lq) datasette-with-plugins
sha256:8cde4a6357b9221d6f9e15887a314f2b4d9d1b87b517764d207ccbaec7c0a69f
~ % docker run -p 8001:8001 -v $(pwd):/mnt datasette-with-plugins datasette --root -p 8001 -h <IP_ADDRESS>
INFO:     Started server process [1]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8001 (Press CTRL+C to quit)

^CINFO:     Shutting down
INFO:     Waiting for application shutdown.
INFO:     Application shutdown complete.
INFO:     Finished server process [1]
http://<IP_ADDRESS>:8001/-/auth-token?token=4bd70fdbca215ea55c874eaf889adf8c09f2a00231f7e5e6d0470f3176407a98

Note how the auth-token URL is only displayed after you hit Ctrl+C!

To test the fix, I did a fresh checkout of simonw/datasette and edited Dockerfile to look like this:
FROM python:3.11.0-slim-bullseye as build

RUN apt-get update && \
    apt-get install -y --no-install-recommends libsqlite3-mod-spatialite && \
    apt clean && \
    rm -rf /var/lib/apt && \
    rm -rf /var/lib/dpkg/info/*

RUN pip install https://github.com/simonw/datasette/archive/refs/heads/main.zip && \
    find /usr/local/lib -name '__pycache__' | xargs rm -r && \
    rm -rf /root/.cache/pip

EXPOSE 8001
CMD ["datasette"]

Then I built it like this:
docker build -f Dockerfile -t datasette-pre-click .

And ran like this:
docker run -p 8001:8001 -v $(pwd):/mnt datasette-pre-click datasette --root -p 8001 -h <IP_ADDRESS>

This exhibited the same problem.
Then I pushed a changed to branch and changed the line to:
RUN pip install https://github.com/simonw/datasette/archive/refs/heads/click-echo-root.zip && \

Ran this:
docker build -f Dockerfile -t datasette-post-click .

And this:
docker run -p 8001:8001 -v $(pwd):/mnt datasette-post-click datasette --root -p 8001 -h <IP_ADDRESS>

It fixed the problem!
datasette % docker run -p 8001:8001 -v $(pwd):/mnt datasette-post-click datasette --root -p 8001 -h <IP_ADDRESS>
http://<IP_ADDRESS>:8001/-/auth-token?token=6542dcf5c8f34f8d13f4af9ce728359c602469efb54029098562bd06c87ad26d
INFO:     Started server process [1]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8001 (Press CTRL+C to quit)

This is in the 0.63.x branch now, ready to go out in a bug fix releas.

Awesome. Thank you @simonw.


Separate parts of If, Catch, Check, Quiet statements

Changes

Closes #541.
Format code style of If, Catch, Check, Quiet functions.
Contributing guidelines updated.

Comments

If any other functions that expect compound expressions come to mind, let me know.

Expressions that fit in a single line remain unchanged:

  If[!FileExistsQ[tempBuildInfoFile], ReturnFailed["nobuildinfo", tempBuildInfoFile]];

Examples
  outputWithMetadata = Catch[
    Reap[
      setReplace$wl[initWithMetadata, rulesWithMetadata, stepSpec, vertexIndex, returnOnAbortQ, timeConstraint],
      {$$deletedExpressions, $$events}]
  ,
    $$nonListExpression
  ,
    (Message[caller::nonListExpressions, #]; Return[$Failed]) &
  ];

    result = Catch @ Check[
      If[modifiedEvolution =!= $Failed,
        If[ListQ[property],
          Catch[
            Check[propertyEvaluateWithOptions[#], Throw[$Failed, $propertyMessages]] & /@ property
          ,
            $propertyMessages
          ,
            $Failed &
          ]
        ,
          propertyEvaluateWithOptions @ property
        ] /.
          HoldPattern[WolframModelEvolutionObject[data_Association]] :>
            WolframModelEvolutionObject[Join[data, <|$rules -> rulesSpec|>]]
      ,
        $Failed
      ]
    ,
      $Failed
    ];

This change is

I get this when running ./lint.sh:

Changing which to command -v fixes the issue.

I get this when running ./lint.sh:

Changing which to command -v fixes the issue.

@daneelsan, we need to fix that, however, the error in CI is different. I guess you are using a different version.

@daneelsan, we need to fix that, however, the error in CI is different. I guess you are using a different version.

@maxitg A different version of what?

@maxitg A different version of what?

@maxitg A different version of what?

Of markdownlint.

@maxitg A different version of what?

Of markdownlint.


ARROW-8142: [C++][Compute] Explicit no chunks case for WrapDatumsLike

When kernels which use the WrapDatumsLike utility on a chunked array, they invoke the ChunkedArray constructor without an explicit type. This is forbidden for an empty chunks vector and causes an assertion failure. Since we always have access to the desired type (through Kernel::out_type), we can avoid this by passing that type to WrapDatumsLike.

The aborting tests seem to be limited to several tests dealing with string to dictionary conversion in to_pandas:
python/pyarrow/tests/test_pandas.py::TestConvertStringLikeTypes::test_table_empty_str
python/pyarrow/tests/test_pandas.py::TestConvertStringLikeTypes::test_selective_categoricals
python/pyarrow/tests/test_pandas.py::TestConvertStringLikeTypes::test_table_str_to_categorical_without_na
Smaller reproducer:
In [1]: table = pa.table([pa.array(['a', 'b'], type=pa.string())], ['col1'])

In [2]: table.to_pandas(strings_to_categorical=True)
terminate called after throwing an instance of 'std::bad_cast'
  what():  std::bad_cast
Aborted (core dumped)

In addition, there is also one failing test (python/pyarrow/tests/test_parquet.py::test_direct_read_dictionary), which seems to be due to the table constructor loosing the type of a dictionary encoded chunked array:
In [12]: chunked_array = pa.chunked_array([pa.array(["a", "b"])]).dictionary_encode()

In [13]: pa.table({'f0': chunked_array})
Out[13]:
pyarrow.Table
f0: int32

The second failure comes from this (which might help finding the root cause):
In [8]: chunked_array = pa.chunked_array([pa.array(["a", "b"])]).dictionary_encode()

In [9]: chunked_array
Out[9]:
<pyarrow.lib.ChunkedArray object at 0x7f19c6c99b88>
[

  -- dictionary:
    [
      "a",
      "b"
    ]
  -- indices:
    [
      0,
      1
    ]
]

In [10]: chunked_array.type
Out[10]: DataType(int32)

In [11]: chunked_array.chunk(0).type
Out[11]: DictionaryType(dictionary<values=string, indices=int32, ordered=0>)
``

So when dicitionary encoding a chunked array, the type of the chunked array is set incorrectly (the type of the individual array chunks is correct)

So I suppose that the `out_type()` passed in cast.cc::DictionaryEncode is the wrong type.

@bkietz added two python tests as well (which cover the reproducers from the two JIRA issues)

@jorisvandenbossche thanks!


BAEL-2850: Difference between context:annotation-config vs context:component-scan

BAEL-2850: Difference between context:annotation-config vs context:component-scan

[retest this please]
Note: this comment is for triggering the CI build. No action required from the author.


Error - Error in missing.control() : could not find function "missing.control"

Dear Authors,
Thank you for publishing the code and data. I installed the R environment following your instruction and when I tried to reproduce the experiment by running bash scripts/sh/reproduce_experiments.sh, there is an error occured: "Error in missing.control() : could not find function "missing.control".
As a beginner in R, I would like to ask about which packages should I install/ update to include this missing.control() function? Your answer would be really helpful to me :)

I found missing.control() should be in the package prioritylasso. There is no error after I updated it from 0.2.5 to 0.3.0.

Hi David,
thanks for reaching out and your interest in SurvBoard!
Thank you for pointing this out, this likely means that we have a stale prioritylasso version in our lockfile; I will investigate this further once back from vacation.
Also just FWIW, missing.control specifies how PriorityLasso deals with samples for which some modalities may be missing (details here if curious: https://cran.r-project.org/web/packages/prioritylasso/vignettes/explanation_blockwise_missing.html).
Let us know if we can help further with anything! (will leave this open in the meantime)

Just manually bumped the version, thanks for letting us know!


scale image with orientation on iOS

This PR fixes one problem I've found out recently - when app uses flag RotateImage set to false and scales image down with PhotoSize.xxx it will receive image rotated on iOS. That seems to be caused by the fact that during rescale image is changing it's orientation but the original Orientation flag remains (and is read-only).
I've tried many approaches on setting the flag but only this solution worked for me.

Seems like this will fix #255?

No longer needed


Colors in ColorPicker Default to Semi Transparent

With apologies for being new to WPF... I'm using the pre-release version of the MVVM community toolkit.
If I bind ColorPicker to a Color value like this:
<cpkr:ColorPicker Grid.Column="1" Grid.Row="1"
                              HorizontalAlignment="Left"
                              SelectedColor="{Binding RouteColor, Mode=TwoWay}"/>

and the modelview definition of the property is like this:
        private Color _routeColor = Color.Red;

        public MainViewModel()
        {
            RouteColor = Color.Red;
        }

        public Color RouteColor
        {
            get => _routeColor;

            set
            {
                SetProperty( ref _routeColor, value );

                _appConfig.RouteColor = value;
            }
        }

then the color displays as semi-transparent (alpha channel value 128). In fact, it doesn't appear like the binding is working, since changing the color to Blue doesn't change what gets displayed.

Sigh. I think the problem is elsewhere. I'm closing this issue for now.

Sigh. I think the problem is elsewhere. I'm closing this issue for now.


schemas and metadata

Currently (in the datajoint-pipeline branch), if there is a mismatch between the schema (in dataset.py) of an experiment and metadata.yml then devices with mismatch will not get ingested.
proposal:
First fix: treat the schema as "truth". If there is metadata for a device, great, save it in DJ, but if there is no metadata, just ingest the device without metadata.
Second fix:
Move the reading of metadata into the Device construction, so that we can make smart devices. For example: a stream called list_of_walls which automatically specificies a series of devices.
@ttngu207 @glopesdev

Long fix:
For things like 'WallIterator' for Octagon experiments, python code that parses Bonsai will smartly add each Wall individually to metadata.yml, even though in Bonsai there is no explicit node for each Wall


Add initial Zowe Conformant ZOWE v2

Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you including a repo_url? You need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Does the project meet the guidelines for new entries section?
[ ] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
item does not have a name

Build failed because of:
item does not have a name

Build failed because of:
File Master Plus Plug-in (CLI ZOWE v2) has a homepage_url https://techdocs.broadcom.com/us/en/ca-mainframe-software/devops/ca -brightside/3-0/zowe-cli/available-cli-plug-ins/ca-file-master-plus-plug-in-for-zowe-cli.html with spaces

Build failed because of:
FATAL: entry SymDump (API Mediation Layer ZOWE V2) is missing a logo
Item SymDump (API Mediation Layer ZOWE V2) has no image_data

Build failed because of:
No cached entry, and Test4z (API Mediation Layer ZOWE V2) has issues with logo: Test4z.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/Test4z.svg'
Test4z (API Mediation Layer ZOWE V2) either has no crunchbase entry or it is invalid

Build failed because of:
Test4z (API Mediation Layer ZOWE V2)  has an empty or missing homepage_url

Build failed because of:
IBM® MQ Plug-in for Zowe CLI (CLI ZOWE V2)  homepage_url github.com/zowe/zowe-cli-mq-plugin#readme should start from http:// or https://


Compiler should Emit Warning/Notes when Deduced Type Differs from Annotated Type.

Feature Request
I had some code recursively walking through a tree. The Crystal compiler was very clever, and spotted that I was not handling the base case properly. In this code, I mapped over the node's children, and assigned this to an Array(UInt8?). However, the compiler complained that in order to assign to this, the result of the map had to be Array(UInt8?), not Array(Nil). This was because it had worked out that the nil returned from the base case was wrong, and would propagate upwards. This meant that my recursive function would always return nil, even though it was annotated to return UInt8?, and thus, the array could only ever be an array of nils.
This is great, having the compiler check my logic is the way to do things properly. However, my only issue was that it was hard to work out why this was the case. The function was annotated to return UInt8?, and the array was annotated as Array(UInt8?). If the compiler has inferred in cases like this, that the actual return type differs from what I am expecting, in effect due to unreachable code or similar issues, then I'd like to have a hint in the compiler error stack explaining what it deduced to be different form what I expected, and where this occurs. In this case, if it could tell me that it has deduced that a particular type in a union will never be retuned, then that would help find the source of the error - in particular, because this was ultimately a logic error, not a slip of the fingers or some other semantics-related issue. My code was effectively correct (and other languages would just compile this anyway), but my logic was not, so for this to show up as a type error somewhat hides the source of the issue.
Here's a toy example. Obviously just returning nil would be a bit of a blaring mistake, but it servers as a minimal reproduction for the issue.
def recurse(n) : Int32?
    nil
end

results : Array(Int32?) = (0..10).map{|n|
    recurse n
}

Compiler output:
1 ❯ shards build --debug --error-trace
...
Error target MRE failed to compile:
In src/MRE.cr:5:1

 5 | results : Array(Int32?) = (0..10).map{|n|
      ^------
Error: type must be Array(Int32 | Nil), not Array(Nil)

And what I'd like to see, is something along these lines:
1 ❯ shards build --debug --error-trace
...
Error target MRE failed to compile:
In src/MRE.cr:5:1

 5 | results : Array(Int32?) = (0..10).map{|n|
      ^------
Error: type must be Array(Int32 | Nil), not Array(Nil)

Note: result of expression deduced to have type Array(Nil)
 6 |     recurse n
         ^--------

Note: because method deduced to have return type of Nil.
 1 | def recurse(n) : Int32?
         ^---------

Warning: Method deduced to have return type differing from its explicit annotation. Its effective return type is Nil.
 1 | def recurse(n) : Int32?
                      ^-----
Note: return of type Int32 in union (Int32 | Nil) is unreachable
 1 |     nil
         ^--

I think a warning whenever an explicit type annotation is found not to match what the compiler has deduced would be good in all cases. Even if it's not causing an error elsewhere, it's probably a bug, so I think it should not go by silently.
Modern clang can be quite good at suggesting what I might have meant to do, as well as backend compiler logic has determined something that isn't in the source code; so I don't know if there's some inspiration to be had, either in error messages, or in how it's implemented within LLVM, there.
PS: This bug may well be a duplicate of another; but beyond searching for keywords like "type inference" and getting back tonnes of results, I'm not really sure what search term to use to track it down.

I think a warning whenever an explicit type annotation is found not to match what the compiler has deduced would be good in all cases. Even if it's not causing an error elsewhere, it's probably a bug, so I think it should not go by silently.

This is a controversial topic. It's discussed in more detail in #3803

If I understand correctly what the example exposes is that the annotated return type doesn't coerce the actual return type. It's just a constraint that gets accepted (Nil is a subset of Int32|Nil) and the compiler will generate a recurse(Int32) : Nil function.
The warning would generate too much noise. For example it's common for an abstract method implementation to return only a subset of the annotated types.
Having the compiler explain the issue could be interesting. It's good at reporting how it reached the error, but it doesn't explain why the block is returning Nil.

So, am I right in thinking that the annotation is strictly a constraint? A bit like a concept requirement we now have in C++20. As, admittedly I haven't read up enough on the Crystal theory, it's more been a language I have picked up on the go, but, when I write a function as in the example, I expected that this means the compiler is stamping out binary code that takes those types and returns the one I specify. Is it more like you're just saying "this can return an integer, or nil, or a nullable integer - but refuse to compile this overload for anything else"? It's a generic but with a requirement on the list of permissible types? And the compiler is free to output a specialisation for just a sunset of the types, and/or an overload that handles the possible options at runtime, as would be the case here if I returned nil if rand() was less than 0.5, and some integer if greater, for example?
Also on the verbosity point - could it simply be absorbed into the error-trace option? As the compiler by default only spits out the first problem we encounter, right? If it would be too verbose to have as a warning in general cases, I think showing it in the error stack, or having an option similar to what we can do with macros, would then mean that the user is asking explicitly for more information. I was scratching my head for a good ten minutes so it's stuff I'd like to know given in this scenario my program already can't build.

Note: This discussion would fit better in the forum.
To answer quickly (to the extent of my knowledge): an Int32|Nil return type annotation on a def says that the method may return an Int32 or a Nil. It's fine to return either, but it's an error to return something else (e.g. String). I believe this also stands for argument types.
This doesn't stand for a fun that must be fully typed and generates an extern symbol, compatible with C.

the compiler is free to output a specialisation for just a sunset of the types

Yes, and it will actively try to specialize def methods, even on how they're actually called (as a JIT would).

It's a generic but with a requirement on the list of permissible types?

I believe it can be viewed this way. It is said that "Crystal is a big template language".

Thanks, that makes sense. The template part I have gradually learned the hard way 😂.
Do you want to merge this issue into another re warnings or compiler verbosity and close it?


Question: Can I infer the size of the component type when visiting storage pools?

Hello!
I am trying to generically (without upfront knowledge of the component types) read the byte contents of all existing pools.
Is this possible?
I assume the only way I can access the pools generically is by using visit:
auto visitor = [](entt::sparse_set& pool) {
    // Can I get the pool element size?
};
entity_registry_.visit(visitor);

(BTW, the documentation has this example but it seems to me this is no longer possible:
// create a copy of an entity component by component
for(auto &&curr: registry.storage()) {
    if(auto &storage = curr.second; storage.contains(src)) {
        storage.emplace(dst, storage.get(src));
    }
}

)

the documentation has this example but it seems to me this is no longer possible

Quite the opposite, this is possible since v3.9, that is the last version available upstream.
As for your first question instead, in the generic pool you've a get function that returns a void * at the moment. You can use it and the type info object with a reflection system or similar. This should address your needs.
Let me know if you've still doubts.

@skypjack Thanks for the reply!
Hmmm... I am actually using v3.9!

But if I make a basic registry I cannot call storage without a template parameter. What may I be missing?
Back to my question, I am very sorry but I think I do need more details. I assume that the type info would be what I get by pool.type() (if pool is as my example above), but I have no idea how to get the size info from there!

What may I be missing?

🤔
I don't know honestly, this is the code I've used for a test in v3.9 and I'm pretty sure it works since the CI isn't complaining. 😅
Maybe a CMake project or similar where you've updated the version but that doesn't refresh the local copy after the boostrap? Really dunno, just thinking aloud, it's hard to say from here.
As for your question, the type info can help you to find a meta type with which you can elaborate the void *.
Long story short, pools are lazily instantiated in EnTT and their types are unknown (until the user says - see, this is a T). Therefore, something like a reflection system acts as glue code when you want to work with opaque data.
Can I ask you what's your goal? I mean, why do you need the size? Maybe this is an YZ-problem and I can help more if you tell me what you're trying to achieve. 🙂

If registry.storage() is supposed to compile with 3.9 seems I need to dig around. This is a fresh copy of entt for this project, not an upgrade, but I wonder if I did not setup CMake correctly. Unless I need to include a particular header other than entt.hpp. Anyway, at least now I know!
What I am trying to do is to iterate all the component pools and read the byte data directly without manually registering this code per component type. I am still experimenting but I have two concrete goals right now: Compute a checksum for all existing components and copy entities from one registry to the other without knowing upfront which components where registered.
I can elaborate on why I want to do the later, but I think that goes beyond the specific question.
I am not very familiar with reflection on C++, is there any resource you'd recommend to get me started?
Thank you!

Unless I need to include a particular header other than entt.hp

Uhm... is it the single include file? Maybe it's out-of-date? Otherwise, the entt.hpp file under entt is just a list of includes and should contain the right version of the registry.
I'm cursious now btw, let me know if you find what the problem is.

What I am trying to do is ...

To copy the entity, at the moment you can iterate the pool, then get and emplace entities and values one at a time.
I'm working to make a sort of get-and-emplace-everything but it's not in v3.9, I'm sorry. However, you can have it with some reflection.
C++ doesn't support reflection btw. It's planned for C++23 but it's not even confirmed at the moment.
EnTT offers a runtime reflection module under the meta directory. So, for example, you could:

Create a meta type for T and register a couple of functions to compute the checksum and blindly copy everything across registries
Run ... run ... run
Iterate the pools on the source registry, get the type info and use it to retrieve the meta type, then pass the pool or the value to the right meta functions and get the job done

If this sounds like something that could work for you, I can prepare a super small example if you like. Let me know. 👋

This is certainly of interest to me, so I would very much appreciate an example!
Thank you very much for your help! (And the library of course!)

Sorry, I've been busy today and I haven't found the time to pack a proper example.
Here you can find a snippet that shows what I mean with copying a storage using meta.
Let me know if it's clear enough or if you have any doubt. I'll try to keep up as soon as possible. 👍

This does let me move forward, thank you very much for the example!
Purely for academic interest, what I was hoping for is a way to do this without having to upfront register my components for this, as you do in your example (register_meta_type_for<position>();). I wanted to be able to copy all components that exist in one registry without having forward knowledge of their types. I assumed that the pools would need to know the object size implicitly.
I have not looked into my issues running the untyped storage, but if I find it I will let you know!
Thanks!

Well, technically speaking it's possible already as:
for(auto [id, storage]: src_registry.storage()) {
    auto &&other = dst_registry.storage(storage.type());

    for(auto entity: storage) {
        other.emplace(entity, storage.get(entity);
    }
}

Morever, as I said before, I'm working on something to make it a single line instruction. However, this is only meant to work across storage classes and not across registries.
The fact is that EnTT is designed in such a way that you don't need to use the registry. You can just use the storage classes and that's it. So, the latter has not the knowledge about the registry, for obvious reasons. Therefore, the biggest problem to address is how to create the storage in an empty registry from an opaque reference to a base class (that is, when you don't have the type).
The next version should fill this gap. The solution will likely be to inject a storage class that you obtain as a copy from an existing pool. However, this won't work across registries with different entity types.

Hey!
So that snippet looks exactly like what I need at this moment (still experimenting!). While looking at it, though, I had to figure out why I cannot access storage with no template arguments. The reasons seems to be that the version I got is the one from here: https://github.com/skypjack/entt/releases/tag/v3.9.0
The commit that adds that method was done the next day! :P

🤔
Wait a moment. This is from v3.9:
[[nodiscard]] auto storage() ENTT_NOEXCEPT {
    return iterable_adaptor{internal::storage_proxy_iterator{pools.begin()}, internal::storage_proxy_iterator{pools.end()}};
}

That is, this is possible with v3.9 already:
for(auto [id, storage]: src_registry.storage()) {
    // ...
}

I really don't know why you don't see it but, well, it's there. 😅

You might want to be aware that this link:

leads to a version without it.
Thanks for all the responses so far!

😲 Oh, this I don't know actually, I've always referenced the git tag. What's wrong there? The single include file or the tarball?
I remember it happened one more time too. If I'm not mistaken, it's wrong when I create the release before the git tag or something like that. 🤔

Hello!
A few updates on this.  Finally got around field testing these ideas. First is that your example above needs some tweaks:
Your example:
for(auto [id, storage]: src_registry.storage()) {
    auto &&other = dst_registry.storage(storage.type());

    for(auto entity: storage) {
        other.emplace(entity, storage.get(entity);
    }
}

My (compiling) code:
for (auto pair : source.storage()) {
	const auto& source_store = pair.second;
	auto& target_store = *target.storage(source_store.type().hash());
	for (auto entity : source_store) {
		target_store.emplace(entity, source_store.get(entity));
	}
}

I had to use source_store.type().hash(), because source_store.type() is not the right type, and the id is otherwise private. This works, but using hash feels like a hack, as there really should be no guarantee this will work the same in the future.
Another issue I discovered is that if the target registry has not had the component type initialized this will result in null memory access. I think I can deal with this, though.

auto &&other = dst_registry.storage(storage.type()); only works when dst_registry already has the storage. Otherwise, it returns zero. i was expecting dst_registry.storage(storage.type()) can create such storage in the dst_registry.

Thank you @chengts95!
Any recommendations on how to make sure the storage has the pool, sort of adding/removing a component just to be sure?
I would like to use assure for example, but it is a private member.

To put storage in the pool, you have to use a template because C++ types don't exist at runtime.

Thank you for the reply.
I understand that. My question is what would be a good way to create the pool on the target registry (yes using type specific templates) if no entity on the target registry has received the component yet.
Currently, to get around this I am adding and removing a component of the type on a fake entity, but I wonder if there is a call for such purpose, along the lines of registry.touch<TComponent>().

This is what I am settling with for now:
auto& target_storage = target_registry.storage<TComponent>();
// Hack: Ensure pool exists in target registry
if (target_storage.capacity() == 0) {
    target_storage.clear();
}

The rest (the transfer part) is as above, and as a proof of concept this works pretty well.

Yeah, the non-template storage method is only available upstream and isn't part of any release yet. The goal is to add all is needed to help creating pools across registries. However, it won't be a method in the storage class, mainly because the latter doesn't know about the registry and this thing isn't going to change any time soon.
Probably, the final solution will be a fake vtable directly provided by the registry, something along this line:
for (auto [id, source]: registry.storage()) {
	auto& target = registry.vtable_for(source.type().hash()).emplace(other_registry);
        // ...
}

Though, take this with a grain of salt at the moment, I really don't know how it will look like (and any suggestions is welcome in this sense 🙂).

Thank you, @skypjack.
Looking forward to that!
Any recommended alternatives to my hack above in the mean time?

@Ramito registry.storage<T>() is enough to create it.

@skypjack but then if I don't do anything with it I get a no-discard error, since it gets compiled out. I think it'd be nice to have a call to just ensure it's there without any intention to sue it at the time.

Yeah, I want to remove the [[nodiscard]] in this case. I'm using it to create pools too and it's getting annoying. 😅

😁 Thanks!

this thing isn't going to change any time so

Maybe that is true, a vtable for constructing storage by type id.   I believe the main problem is to new a proper sparse_set with type id only and automate this process. It is true that we can put the class constructor into the registry's storage vtable when new the storage.  But the instance in vtable must have access to other registry's private members to emplace the storage.
Maybe adding something like this to the registry?

    void* registry::emplace_storage(const id_type id , function_ptr  storage_constructor) {

        auto &&cpool = pools[id];

        if(!cpool) {
            cpool.reset( storage_constructor());
            cpool->bind(forward_as_any(*this));
        }

        return  cpool.get();
    }

Mmm probably the vtable should work the other way around, that is:
registry.vtable_for(id).setup(other_registry);

This way you have a function with full knowledge that triggers a .storage<T> call on other_registry.
Actually, the function could even be as simple as &storage<T>. No need to generate a new one.

Mmm probably the vtable should work the other way around, that is:
registry.vtable_for(id).setup(other_registry);

This way you have a function with full knowledge that triggers a .storage<T> call on other_registry. Actually, the function could even be as simple as &storage<T> (if it only returned an opaque storage 😅).

I remember the constructor's pointer cannot be obtained, so I use the save a lambda function to hashmap.

What's the current status on duplicating an entity with all components (possibly to a different registry)? None of the examples listed even compile for me with 3.9.0.
This one:
for (auto pair : source.storage()) {
	const auto& source_store = pair.second;
	auto& target_store = dest.storage(source_store.type());
	for (auto entity : source_store) {
		target_store.emplace(entity, source_store.get(entity));
	}
}

Fails on this line, specifically the storage method:
auto& target_store = dest.storage(source_store.type());
Also, from what i understand from the discussion, this method only works if the destination storage has already been initialized by creating a component of that type. Is there any alternative solution?

You're right, this is available upstream but not part of v3.9:
auto& target_store = dest.storage(source_store.type().hash());

While this:

this method only works if the destination storage has already been initialized by creating a component of that type

Hopefully will be addressed before the next version. With v3.9 you can get close using the storage type function to get a type info object to use to retrieve a meta type, if any.

I've been using poly_storage to copy entities between registries (both to copy entire registries, which I do from the editors registry to the games registry when you press "play", and also for single entities, which I use for "prototype"/prefab entities that can be instantiated by copying from the prototype to the live entities). I do it like this and it seems to work well enough, for the past ~10 months.
I'm wondering how these new changes impact that? Will I be able to implement this then without custom poly_storage? Any tradeoffs I should know about?
Thanks :)

It turned out that this solution is waaaaaay simpler but also incomplete in a sense. Something that I'll fix soon but still.
Briefly, if you use meta, it's almost a direct mapping between the poly storage and the meta type. Otherwise, the opaque get and emplace functions mke you copy components across registries blindly in a blink.
The only issue if compare to the poly storage is that the blind copy doesn't create a storage class in the target registry for you. The fix that I've mentioned above aims to fill the gap.

Funny enough, after struggling with this feature for a while I realized that EnTT already offers everything needed to get what I want in a flexible and customizable way and without library additions.
I'm updating the entity_copy.cpp example to get rid of the infamous TODO 🙂 then I'll slightly review the entity storage machinery to make it cleaner because it requires some non-obvious knowledge at the moment.
CC @Qix- who put the TODO in the entity_copy.cpp file. 👍


More integration tests for TopicMessageSubmitTransaction (PR)

Description:
This PR adds missing integration tests for TopicMessageSubmitTransaction:

ExecuteLargeTopicMessageSubmitTransaction
CannotSubmitTopicMessageWhenTopicIdIsMissing
CannotSubmitTopicMessageWhenMessageIsMissing

Java version of TopicMessageSubmitIntegrationTest.java was used for reference.
Related issue(s):
Fixes: https://github.com/hashgraph/hedera-sdk-cpp/issues/452
Notes for reviewer:
Unfortunately, these 2 integration tests:

CannotSubmitTopicMessageWhenTopicIdIsMissing
CannotSubmitTopicMessageWhenMessageIsMissing

are not testing exactly the missing TopicId or TopicMessage as calling the executeAll() does not throw any exception at this moment. Additional changes in TopicMessageSubmitTransaction could be needed.
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Renamed the following integration tests:

CannotSubmitTopicMessageWhenTopicIdIsMissing -> CanSubmitTopicMessageWithoutTopicId
CannotSubmitTopicMessageWhenMessageIsMissing -> CanSubmitTopicMessageWithoutMessage

Renamed the following integration test:

CanSubmitTopicMessageWithoutTopicId -> CannotSubmitTopicMessageWithoutTopicId

Test CannotSubmitTopicMessageWithoutTopicId will expect throwing a ReceiptStatusException when calling resp.getReceipt(getTestClient()).mStatus.


require external mag (optionally)

I've noticed a number of setups that manage to slip through without an external mag, despite the user thinking one is present.
Ideas

Add a configurable preflight check that requires an external mag by default, but can be easily disabled.
Start explicitly configuring all external devices rather than probing for everything. By requiring the full configuration to be entered we have all the necessary information.

Required sensors

board manifest
user configured externals
preflight checks, useful user facing errors if missing

Optional sensors

does this category even need to exist?

This would be really good.


Modify liquidation mechanics

Liquidate distressed party's position immediately and move it to the network party.
Market's insurance pool is market's party margin account.
Introduce a simple inventory management strategy for the network party.

close https://github.com/vegaprotocol/specs/issues/1641

The core implementation issue is:

https://github.com/vegaprotocol/vega/issues/9945


When choosing the last preset, use tags from the last added object

To speed up adding multiple e.g. benches of the same kind.

You should be able to choose a fresh empty feature, or to use (transferable) tags from,

the last feature of this type that you added,
any nearby (or loaded) features of this type.

Done in 0.5. Did not copy tags from features around, because you never know what's there.

Ugh, I almost filed a bug regarding this behaviour assuming it wasn't intentional. I'll keep an eye on this, but also there's also a question. Is the autofill with data of the last feature of this type performed:

only when adding several points of same type in a row?
with data of previously added point of same type, regardless when?
with data of previously edited point of same type, regardless when?

One more note: for most categories, like shops, this feature isn't really useful. Maybe it should be limited to selected ones?

It would be nice if those remembered tags would also be shown when selecting an object type. The way it is right now it confused me as well that some of them retain what I've entered before and some do not.

Maybe make a selection (slightly) different color if it was autofilled automatically from previous object (as opposed if it was present on the object itself or manually set by the user on this object)?


bump version

SOVERSION needs a bump to (removed function):
Run abidiff old/build/libre.so current/build/libre.so
Functions changes summary: 1 Removed, 0 Changed, [4](https://github.com/baresip/re/actions/runs/9079894989/job/24950077925#step:8:5) Added functions
Variables changes summary: 0 Removed, 0 Changed, 0 Added variable

1 Removed function:

  [D] 'function int dbg_logfile_set(const char*)'    {dbg_logfile_set}

4 Added functions:

  [A] 'function int h2[6](https://github.com/baresip/re/actions/runs/9079894989/job/24950077925#step:8:7)4_stap_decode_annexb_long(mbuf*, mbuf*)'    {h264_stap_decode_annexb_long}
  [A] 'function int http_client_disable_verify_server(http_cli*)'    {http_client_disable_verify_server}
  [A] 'function int pl_strncmp(const pl*, const char*, size_t)'    {pl_strncmp}
  [A] 'function bool sip_request_provrecv(const sip_request*)'    {sip_request_provrecv}


[Enhancement] Field isn't final but should be [grpc SendMessageContext]

Search before asking

[X] I had searched in the issues and found no similar issues.

Enhancement Request

located at:
org/apache/eventmesh/runtime/core/protocol/grpc/producer/SendMessageContext.java 36
analysis and explanation:
This static field public but not final, and could be changed by malicious code or by accident from another package. The field could be made final to avoid this vulnerability.
Describe the solution you'd like
add final modifier field.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

I would like to work on this issue. @Alonexc Can you please assign this task to me?


Can't set distortion amount to 0 via set() method

The documentation says amount should be a value between 0-1 but isn't possible to set the distortion amount to 0 via the set() method as it checks the truthy value of the amount argument:
https://github.com/processing/p5.js-sound/blob/1a9beda8528095e410a45b8822cbd846bba810df/src/distortion.js#L98
When amount is 0 it's false.
It is possible to set amount to 0 via the constructor as that checks for undefined instead of a truthy value.

@davepagurek this issue could be closed

Thanks @KevinGrajeda!


NEXT-14691 - Add pseudo modal twig blocks

1. Why is this change necessary?
Pseudo modal template is missing many twig blocks which may be used to override/extend the template easily from themes/plugins.
2. What does this change do, exactly?
Add the missing twig blocks and deprecate an inappropriately named block.
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
NEXT-14691
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

@tobiasberge I've removed the modal dialog and content blocks. The new blocks are now "overall", "header", "title", "close button", "body" and "back button". Is that okay?

Hi @lacknere thank you. 👍 LGTM.
And good catch with the wrong block naming product_detail_. 🚀

Hi @lacknere, you changes have been merged to trunk: https://github.com/shopware/shopware/commit/f5bc948b784cc53da487f9f62352d989ec0558cf and will also be backported to 6.5.x
Thank you for your contribution. 🎉


Batch job

Hi, Lisa! I’ve got these new scripts working on my new 4k discs. Is there a way to do batch jobs with them? I know with the previous scripts we could, but these seem to run a bit differently. I’d like to be able to submit a few movs and then walk away for a week.
Thanks for the continued development in this space.

@geddy76 Thanks for your enthusiasm! 👍
It should be possible to adapt the batch processing script template I created for my other_video_transcoding project. It's available here:
https://github.com/lisamelton/other_video_transcoding/wiki/Batching
That's basically what I've done.
Does that help?

That certainly points me in the right direction. First attempt is throwing errors, so it looks like I have some learning to do. Thanks very much!

@geddy76 Excellent! Let's keep this open in case you get stuck on something and need to ask questions. We can close it when everything is running smoothly.

Appreciate it. Poking around in this. Full disclosure: I’ve never used Ruby before. I have a basic understanding of JavaScript and python, but that’s about as far as I go. Not asking for this to be spoon fed, just giving context.
I changed the last line of batch.rb to call hevc-transcode.rb instead of other-transcode.rb. That is the only change I made. That batch file, the actual hevc-transcode file and the queue.txt are all in my /usr/local/bin folder. I have two test episodes (mkv) in my user Movies folder, which the queue file points to. In terminal, I cd to /usr/local/bin and run ‘ruby batch.rb’ with no errors, but no apparent result; just a new prompt line. However, when I open the queue file, the first line has been deleted. So it’s doing something, it just doesn’t seem to be calling the hevc-transcode script.

@geddy76 Hmmmm, did you make the hevc-transcode.rb script executable? Like this:
chmod +x hevc-transcode.rb

And are you spelling it hevc-transcode.rb and not hevc-transcode in the batch script?

Definitely appreciate the assistance on this. A few updates:

Chmod doesn’t seem to do anything. No errors, but when I do Get Info, it still shows it as a Plain Text File for type. I have others in that folder that show as Unix Executable.

when I remove *ARGV from the last line of batch.rb, it will actually kick off HandbrakeCLI and get to work on a file. However, it would only do the first of the two files listed in the queue.

it’s actually not working in the last file listed in the queue. I added a third test file to the queue and it transcodes the first two but stops before working on the final.

At this point, I can make this work. I’m not giving Handbrake any options at this point, just using your settings from hevc-transcode. And for the queue, I can just be sure to duplicate the last line, knowing it will stop on that one.

...No errors, but when I do Get Info, it still shows it as a Plain Text File...

This is correct, since it is still a text file. The chmod +x file changes the permissions of the file, setting the "execute bit". On UNIX/Linux systems (macOS is UNIX at the core) being able to "run" something is governed by the "execute bit". If you do a ls -l file, you will see the permissions on the file, and there should be an x in the owner/group/other columns.
For example:
❯ touch foo
❯ ls -l foo
.rw-r--r-- 0 sean  6 Jul 18:45 foo
❯ chmod +x foo
❯ ls -l foo
.rwxr-xr-x 0 sean  6 Jul 18:45 foo
❯ chmod -x foo
❯ ls -l foo
.rw-r--r-- 0 sean  6 Jul 18:45 foo

...That batch file, the actual hevc-transcode file and the queue.txt are all in my /usr/local/bin folder...

In general, you only want to put programs in /usr/local/bin, and not use it as a working directory. Assuming /usr/local/bin is in your path (verifiable by typing echo $PATH, and looking for /usr/local/bin in the list), if you drop a file into /usr/local/bin, and the execute bit is on (i.e. chmod +x /usr/local/bin/file), then you can just run the file by its name.
For example, if I have a file, foo.rb, and I do the following:
❯ chmod a+x foo.rb
❯ mv foo.rb /usr/local/bin/

I can then run foo.rb from anywhere by just typing foo.rb.
Just wanted to give a couple pointers to things that sounded like they might be causing some confusion.

@geddy76 First, don't remove the *ARGV from the batch.rb script or you've never be able to add options at a later time.
Second, your last line listed in your `queue.txt file is not being processed because your last line isn't terminated with a carriage return. In your text editor, make sure your cursor is on a blank line when you're all the way at the end of the file. If the editor you're using is somehow "optimizing" that carriage return away, use a different editor.
I hope that helps.

If you're using a Mac, you might be interested in applying Lisa's command lines using AppleScript droplets. I've made some for use with more-video-transcoding, but they can be used with varying amounts of modification with many types of command lines. Once you have customized the AppleScript, you can save it as an AppleScript Application using Script Editor and drop the appropriate source file or files onto its icon.
two-pass-detelecine-aggr-crop-mp4-out.applescript

This is correct, since it is still a text file. The chmod +x file changes the permissions of the file, setting the "execute bit". On UNIX/Linux systems (macOS is UNIX at the core) being able to "run" something is governed by the "execute bit". If you do a ls -l file, you will see the permissions on the file, and there should be an x in the owner/group/other columns.

Got it. I thought that would be visually reflected in Finder.

In general, you only want to put programs in /usr/local/bin, and not use it as a working directory. Assuming /usr/local/bin is in your path (verifiable by typing echo $PATH, and looking for /usr/local/bin in the list), if you drop a file into /usr/local/bin, and the execute bit is on (i.e. chmod +x /usr/local/bin/file), then you can just run the file by its name.
For example, if I have a file, foo.rb, and I do the following:
❯ chmod a+x foo.rb
❯ mv foo.rb /usr/local/bin/

I can then run foo.rb from anywhere by just typing foo.rb.
Just wanted to give a couple pointers to things that sounded like they might be causing some confusion.

Definitely appreciate that guidance. Now that I’ve made it executable, I can indeed run the file from the initial Terminal prompt. I’ve moved the queue file to my home/user folder.

@geddy76 First, don't remove the *ARGV from the batch.rb script or you've never be able to add options at a later time.

I’ve verified several times that the script will only run for me if I’ve removed *ARGV. I’m way outside of my element, so I won’t begin to guess why.

Second, your last line listed in your `queue.txt file is not being processed because your last line isn't terminated with a carriage return. In your text editor, make sure your cursor is on a blank line when you're all the way at the end of the file. If the editor you're using is somehow "optimizing" that carriage return away, use a different editor.
I hope that helps.

If you're using a Mac, you might be interested in applying Lisa's command lines using AppleScript droplets. I've made some for use with more-video-transcoding, but they can be used with varying amounts of modification with many types of command lines. Once you have customized the AppleScript, you can save it as an AppleScript Application using Script Editor and drop the appropriate source file or files onto its icon.
two-pass-detelecine-aggr-crop-mp4-out.applescript

Interesting. It’s been a couple years since I’ve messed around with AppleScript, though I used to absolutely love it. Thanks!

@geddy76 I'm glad everything is working! Do you want to close this issue now or shall I?

I can close it. But if you ever determine why the *ARGV won’t allow the script to run, I’d love to hear. tomorrow I’ll give the original version of the batch script with other-transcode.

I can close it. But if you ever figure out why I can’t run the script with ARGV included, I’d be interested. Tomorrow I’ll give the original batch script a try with other-transcode, which I already have installed.

Confirming that when I run the original Batch script (calling other-transcode) it runs with the ARGV bit in there. Again, I’m not overly concerned, because I’m not passing any options along for batches (tv shows), but figured I’d pass the info along.
Thanks again to both of you for the assistance.


Update plugin to Zotero 7

Is it possible to update the plugin to be compatible with Zotero 7?

No, it's not recommended. Zotero7 now provides built-in dev tool. So that this plugin can be deprecated.

How can I access the dev tools that inspect HTML elements in Zotero 7?

https://www.zotero.org/support/dev/zotero_7_for_developers


WS-133 - Get nft list by owner

Type of MR:

[x] Feature
[ ] Fix
[ ] HotFix

Small description:

Add getNFTListByOwner function.

Aditional observations/considerations:
...
Checklist:

[x] My code follows the styleguides of the project (successfully pass linters)
[x] I tested my code.
[x] I commented/docummented my code in a way that complex parts are explained.
[ ] If required, i updated project's README file.
[x] I added/updated tests that tests my changes in code.
[x] All previous and new tests pass.
[ ] I commit any new dependency.
[ ] I updated UML diagram in confluence if any model was modified in this MR.

Codecov Report

:exclamation: No coverage uploaded for pull request base (feature/nft-management@fe2d24a). Click here to learn what that means.
The diff coverage is n/a.

@@                    Coverage Diff                    @@
##             feature/nft-management      #45   +/-   ##
=========================================================
  Coverage                          ?   99.20%
=========================================================
  Files                             ?       34
  Lines                             ?     1132
  Branches                          ?      263
=========================================================
  Hits                              ?     1123
  Misses                            ?        9
  Partials                          ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


File Upload with Chatter API

Hi,
I'm struggling with uploading files to chatter, related to a specific object. Does anybody have a working example for this?
Best Regards

If this is still an issue (see latest updates to code) please re-open with more specifics.


Research best practices to build a SDK

We plan to create a SDK for react and react-native

https://chatgpt.com/share/7a087346-51f9-4d47-afb5-d6819d54a3b2


feat/protocol_change: update protocol from http to safe

Updated protocol from http:// to safe://

This change is

r? @ustulation
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Tutorial example "How to optimize GEMM on CPU" not running

I was trying to run the example script from "How to optimize GEMM on CPU" opt_gemm.py, but got the following error on both  server Intel(R) Xeon(R) CPU E5-2640 v3 @ 2.60GHz and laptop Intel i-7 5600U @ 2.6GHz:
Numpy running time: 0.007843
Traceback (most recent call last):
File "opt_gemm.py", line 89, in
func = tvm.build(s, [A, B, C], target=target, name='mmult')
File "/usr/local/lib/python3.5/dist-packages/tvm-0.2.0-py3.5-linux-x86_64.egg/tvm/build_module.py", line 492, in build
mhost = codegen.build_module(fhost, str(target_host))
File "/usr/local/lib/python3.5/dist-packages/tvm-0.2.0-py3.5-linux-x86_64.egg/tvm/codegen.py", line 20, in build_module
return _Build(lowered_func, target)
File "/usr/local/lib/python3.5/dist-packages/tvm-0.2.0-py3.5-linux-x86_64.egg/tvm/_ffi/function.py", line 280, in my_api_func
return flocal(*args)
File "tvm/_ffi/_cython/function.pxi", line 264, in core.FunctionBase.call (tvm/_ffi/_cython/core.cpp:6954)
File "tvm/_ffi/_cython/function.pxi", line 213, in core.FuncCall (tvm/_ffi/_cython/core.cpp:6176)
File "tvm/_ffi/_cython/function.pxi", line 205, in core.FuncCall3 (tvm/_ffi/_cython/core.cpp:6078)
File "tvm/_ffi/_cython/base.pxi", line 131, in core.CALL (tvm/_ffi/_cython/core.cpp:1831)
tvm._ffi.base.TVMError: [17:09:14] src/codegen/codegen.cc:27: Check failed: bf != nullptr Target llvm is not enabled
Anything missed?
Thank you

please ask the general discussion question at https://discuss.tvmlang.org/
In your case, as the error message indicated, you need to build tvm with llvm support http://docs.tvmlang.org/how_to/install.html#customized-building


Fix Julia version in AppVeyor to 1.0

I will merge this for now. When Libtask is ready, we can revert this PR.


Rule Order created by Admin in flow Builder not working

Description: We created a rule "OrderCreatedByAdminRule" and used it in the flow Order placed (checkout.order.placed) as condition. After placing an order as customer in the Storefront the flow always resolve as the order is created as admin.
Checked in multiple Systems, included demo shops.
Environment: Shopware <IP_ADDRESS>
Steps to reproduce:

Create rule "Order created by Admin", set the Condition to no
Change flow Order placed (checkout.order.placed). Add the created rule as condition.
if false set a tag to the order "created by Admin", if true set a tag to the order "created by customer"
Place an order in Storefront.
Create an order in the admin backend.

Expected result:
Both orders get the tag "created by admin".
Actual result:
The order placed in Storefront get the tag "created by customer".

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-33043.
[created from NEXT-33043, comment 484297]


chore: Update pragmas for Step3

Do we need this after some discussion in Zulip? If it is still needed, I would update all pragmas to particular version (0.8.17 for example)


Where to write this code

see the code first
`const puppeteer = require('puppeteer');
(async () => {
const browser = await puppeteer.launch();
const page = await browser.newPage();
await page.goto('https://example.com');
await page.screenshot({path: 'example.png'});
browser.close();
})();`
the above code is pure js code ?
where to write this code in browser console or just the run the code when user click on any button of a web page ?
please guide me.

@karlosRivera
You need the last version of Node.js installed.
Then you need to install puppeteer: npm install puppeteer.
Then you save this code in a file, say, test.js.
Then you launch it as node test.js.

in paper

I use node 8.5 and puppeteer 0.10.2 url = 'https://github.com', it block at
await page.screenshot({page:'example.png'})'

any reason?

@eruca It seems the option should be path, not page:
await page.screenshot({ path: 'example.png' })'


PhoneNumber not validated

Is this a regression?
No
Description
falso can generate number like this one : +33 0 54 43 46 83  when using the FR code country (due to the mask "+33 # ## ## ## ##")
Problem is this number is no validating the  libphonenumber-js
import { isValidPhoneNumber } from "libphonenumber-js";

console.log(isValidPhoneNumber("+33 0 54 43 46 83", "FR")); => false
console.log(isValidPhoneNumber("+33 6 54 43 46 83", "FR")); => true

Would it be possible to know what is considered a valid number for each place?

Number in france
Numbers starts from 01 to 07 for normal numbers, 08 and 09 are for special numbers, there is no 00
So, in international number, there is no +33 0 prefixes, 1 to 9 would be correct I guess
https://codesandbox.io/s/phone-numbers-cmg7mv

Number in france
Numbers starts from 01 to 07 for normal numbers, 08 and 09 are for special numbers, there is no 00
So, in international number, there is no +33 0 prefixes, 1 to 9 would be correct I guess
https://codesandbox.io/s/phone-numbers-cmg7mv

I'm pretty sure that +33 is the international code for France. When I call French mobile numbers from the U.K. I replace the leading 0 with +33.
We could look into using Google's phone number lib as a dev dependency to validate number in our tests (we would still need to research number formats ourselves, but at least we'd know if they are valid)

https://countrycode.org/france

Yes, but no double 0
+33 stand for initial 0
national format is : 0[1-9] ## ## ## ##
international format is : +33 [1-9] ## ## ## ##

@jbdemonte you're welcome to create a PR


QR code can't be decode when qr size is 49x49

python 3.4 (virtualenv)
zbarlight 1.2
PIL 1.1.7
qrcode 5.3
I use qrcode module to generate QR codes. These sizes of QR codes can be 49x49 or 98x98 or 147x147.
49x49 QR code can't be decode, but larger can.

Thanks for the report.
I don't use qrcode and don't know how it works. Do you have examples of QR codes that can not be decoded or a script that allow create one?

import qrcode
import zbarlight
from PIL import Image

img = qrcode.make("alalalal", version=1,
                  error_correction=qrcode.constants.ERROR_CORRECT_L,
                  box_size=1,
                  border=10, )
img.save("t.png")

with open('t.png', 'rb') as img_file:
    image = Image.open(img_file)
    a = zbarlight.scan_codes("qrcode", image)
    print(a)

While box_size=1 and border is an even number, QR generated can't be recognized.
While box_size larger than 1 or border is an odd number, this bug never occurs.


git new-pull-request is broken after solving conflicts during sync

When trying to create a new pull request from a branch that is not synced - and has conflicts, I get the following output:
$ git new-pull-request
<...>
[...] git merge --no-edit develop
<...>
Automatic merge failed; fix conflicts and then commit the result.

To abort, run "git new-pull-request --abort".
To continue after you have resolved the conflicts, run "git new-pull-request --continue".

So, after solving the conflicts, when I try to resume the process - no PR is created and it is finished like this:
$ git new-pull-request --continue
<...>
[...] git commit --no-edit
[...] Merge branch 'develop' into <...>

[...] git push
<...>
/usr/local/Cellar/git-town/0.9.0/libexec/helpers/script_helpers.sh: line 148: create_pull_request: command not found

To abort, run "git new-pull-request --abort".
To continue after you have resolved the conflicts, run "git new-pull-request --continue".

Please upgrade to 0.10. This was recently fixed.

Awesome, thank you.


Fix AWSClient build errors.

Description of changes:
Some args don't have namespace prefixes and cause C2872 errors on build.
In particular, GeneratePresignedUrl was mixed with and without Aws::Http:: prefixes.
Related Issue #2619
It is fixed but after updating sdk to 1.11.147, AWSClient.cpp generates same kind of errors.
Check all that applies:

[x] Did a review by yourself.
[ ] Added proper tests to cover this PR. (If tests are not applicable, explain.)
[ ] Checked if this PR is a breaking (APIs have been changed) change.
[x] Checked if this PR will not introduce cross-platform inconsistent behavior.
[ ] Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

[ ] Linux
[x] Windows
[ ] Android
[ ] MacOS
[ ] IOS
[ ] Other Platforms

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

Hi @ShimYama ,
Thank you a lot for submitting this pull request.
Best regards,
Sergey


What is the font used in the README screenshot?

It looks very nice, and I'd like to use it in my system.

My best guess is that its the Hack font. Please share if you figure out how to use it for Ly.
https://sourcefoundry.org/hack/

Correct :) Not sure if TTYs fonts can easily be changed though.


[backport] Improve README

Backport of #5254

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit a739ba06f073fead4655367c14c475a7c6e462b5, target branch v9) failed with status FAILURE.


Crash after filtering and getting empty result.

[ ] When using the FlexibleAdapter's filter function, the app will crash after no filter results come out.

[ ] The library I use is v5.0.0-b7.

[ ] Here is the problem.

		//Call listener to update EmptyView
		if (mUpdateListener != null &&
				(initialCount == 0 && getItemCount() > 0) ||
				(initialCount > 0 && getItemCount() == 0))
			mUpdateListener.onUpdateEmptyView(getItemCount());

This is the source code in FlexibleAdapter#filterItems() where the crash happened. If the mUpdateListener is null, obviously mUpdateListener.onUpdateEmptyView(getItemCount()); shouldn't be executed. But in fact it will run this code and NullPointerException occurs. The reason is a pair of brackets is missed and the condition turns out true.

[ ] I suppose that the code may be

		//Call listener to update EmptyView
		if (mUpdateListener != null &&
				( (initialCount == 0 && getItemCount() > 0) ||
				(initialCount > 0 && getItemCount() == 0) ) )
			mUpdateListener.onUpdateEmptyView(getItemCount());

Hello @ReinaldLee, please see issue #135, then use Snapshot release of yesterday with more others fix and improvements.


Tracker suddenly stopped working (v1.130)

Tracker was working great after the v1.130 update, then suddenly? Nothing. It refuses to read anything at all. No idea why because nothing in the game/tracker has changed since then? I've tried a random assortment/combination of things such as:

Verifying the integrity of the Steam game files
Restarting my PC
Opening the tracker before, during, and after the opening the game (I know you're meant to do it before, I was just checking)
Re-downloading the tracker and testing the newly downloaded version
Downloading the "manual update" version of the tracker and testing that
Setting the log path manually
Changing the Fall Guys game location from "FallGuys_client_game" to "FallGuys_client" and back (what is the difference between those?)

Nothing worked. It worked perfectly before but now? Nothing.
I'm going to go ahead and also tag @ThreesFG as I had this exact same issue with his version of the tracker and never figured out why it doesn't work.

same it doesnt work again after the fix.

ye for other people its working but not me to

I've just been told Mediatonic changed the log formats again, and this is why it doesn't work.

Mediatonic suddenly started relying on the user date format to print in the logs, but not consistently. So, in parts of the logs the date format is printed as yours, and in parts as the default one from the game (mm/dd/yyyy).
Temporary solution is to set your Windows region to the same as the default from the game (we tested and recommend using English (US) as regional format).
Anyway, I don't have time to fix the parser, but it shouldn't be hard to adjust it to detect both date formats.

Con confirm the above workaround.

my fg is australia and my settings is australia but still doesn't work

You need to set it to United States and English (US). Australian english
will not work, even if your game is set to australian english.
On Mon, Nov 14, 2022 at 17:21 Letrobro @.***> wrote:

my fg is australia and my settings is australia but still doesn't work
—
Reply to this email directly, view it on GitHub
https://github.com/ShootMe/FallGuysStats/issues/152#issuecomment-1314332502,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2OXOJGOTHGJXJQG4EAVODWIKNODANCNFSM6AAAAAAR6TGX44
.
You are receiving this because you commented.Message ID:
@.***>

Will setting windows to US english mess with keyboard settings? Like im in norway and we got 3 more letters than the US. Will it work still?

It may mess up your keyboard. Blame microsoft. Anyway, you should be able
to go to keyboard settings and reconfigure it.
Or you should wait for someone to come up with a patch… this is just a
temporary workaround.
On Mon, Nov 14, 2022 at 17:27 mrsilverhawk @.***> wrote:

Will setting windows to US english mess with keyboard settings? Like im in
norway and we got 3 more letters than the US. Will it work still?
—
Reply to this email directly, view it on GitHub
https://github.com/ShootMe/FallGuysStats/issues/152#issuecomment-1314339076,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2OXOMSZC4DULBU4LH7PWLWIKODHANCNFSM6AAAAAAR6TGX44
.
You are receiving this because you commented.Message ID:
@.***>

like this ?

I can confirm what @amagus said above, it looks like the log (when using a regional formal like United Kingdom) writes the date as dd/MM/YYYY except when you start a game, and then it suddenly shifts to MM/dd/YYYY for the duration of the game. 🤷♂ . Should be fixable.

YES ITS WORKING NOW THANKS

I have submitted a pull request that should fix this issue: https://github.com/ShootMe/FallGuysStats/pull/153


feat: add support for schedules

xref: https://github.com/go-vela/community/issues/538
Part of https://github.com/go-vela/community/pull/772
Dependent on https://github.com/go-vela/server/pull/846
This adds support for running pipelines with event: schedule in the ruleset.

I see that this Skip check extends to other event types, but I'm sort of at a loss for how this ever gets called, since we purge compile-time rulesets here: https://github.com/go-vela/server/blob/5a9a92d279e058b75b37e589e1eef2ec63f550a6/compiler/native/transform.go#L197. AFAICT, the only use this Skip function serves is determining if the status is valid against the ruledata.
Is there a case where Purge wouldn't have caught this?

@ecrupper Purge() does both filtering out steps based off ruleset as well as setting the Number field.
It is a compile-time filter so it doesn't account for status in a ruleset (intentionally).
Skip() is a build-time filter based off ruleset and does account for status in a ruleset.
They do some similar things but aren't an exact match since they have different purposes.

@jbrockopp Yeah, I guess what I'm trying to say is perhaps there is an opportunity for container.Execute to not require all the compile-time ruledata to be set 🤷  — then this wouldn't be necessary any time we want to add a rule in the future.


Add environment variables to specify which local/remote browsers to run tests on

Implement support for two new env vars to override default browsers run by integration tests
export TEST_LOCAL_BROWSERS=chrome,firefox
export TEST_REMOTE_BROWSERS=edge (or whatever remote browser names wct-sauce uses)

Note that SKIP_LOCAL_BROWSERS and SKIP_REMOTE_BROWSERS are already things.


[IMPORTANT] Breaking changes in v4.0.3!!!

You may encounter this version of mov-cli crashing after attempting to play anything.
If this happens update all your plugins with:
pip install {package-name} -U

v4.0.3 introduces some changes to the Series media class. The season parameter is removed and the episode parameter is changed to take EpisodeSelector(). Once again be sure to update your plugins.
class Series(Media):
    """Represents a TV Show. E.g an Anime or Cartoon."""
    def __init__(
        self,
        url: str,
        title: str,
        referrer: str,
-       episode: int,
+       episode: EpisodeSelector,
-       season: int | None,
        subtitles: Optional[dict]
    ) -> None:
        ...

Closing this as the version has now long passed.


Request: add tool to show all VectorDrawables of the project

Currently I can see them only one after another.
This can be useful in order to manage them better.

CTRL + SHIFT + T in Android Studio, inside the projects view. Or right click in project and choose "Show Image thumbnails". Then in the images view click button "Toggle recursive browsing"

click button

it will show all folders. But wait, now I see VectorDrawables are not displayed, sorry, my mistake, will update my answer

@AndroidDeveloperLB I created a plugin showing all vector drawables in your project, hope it helps https://plugins.jetbrains.com/plugin/10741-vector-drawable-thumbnails


Create with Proxy should be updated to pass proxy to TokenCredential.

https://github.com/microsoftgraph/msgraph-snippets-dotnet/blob/9eea7c798a62179d7fd7bcbf23e0b6d7eac822dd/src/SdkSnippets/Snippets/CustomClients.cs#L57
        // URI to proxy
        var proxyAddress = "http://localhost:8888";
        // Create a new System.Net.Http.HttpClientHandler with the proxy
        var handler = new HttpClientHandler
        {
            // Create a new System.Net.WebProxy
            // See WebProxy documentation for scenarios requiring
            // authentication to the proxy
            Proxy = new WebProxy(new Uri(proxyAddress))
        };
        // Create an options object for the credential being used
        // For example, here we're using a ClientSecretCredential so
        // we create a ClientSecretCredentialOptions object
        var options = new ClientSecretCredentialOptions
        {
            // Create a new Azure.Core.HttpClientTransport
            Transport = new HttpClientTransport(handler)
        };
        var credential = new ClientSecretCredential(
            "YOUR_TENANT_ID",
            "YOUR_CLIENT_ID",
            "YOUR_CLIENT_SECRET",
            options
        );
        var scopes = new[] { "https://graph.microsoft.com/.default" };
        // This example works with Microsoft.Graph 5+
        var httpClient = GraphClientFactory.Create(proxy: new WebProxy(new Uri(proxyAddress)));

        var graphClient = new GraphServiceClient(httpClient, new AzureIdentityAuthenticationProvider(credential, scopes: scopes));

Interesting. Isn't this sort of out of scope for Graph? What happens when Azure.Identity changes how there classes interact with proxies? ;)
How about instead I add a comment to the snippet that if token requests need to go through proxy then it has to be configured separately? Maybe with a link to https://learn.microsoft.com/en-us/dotnet/api/azure.core.clientoptions.transport?view=azure-dotnet#azure-core-clientoptions-transport?

Isn't this sort of out of scope for Graph? What happens when Azure.Identity changes how there classes interact with proxies? ;)

I agree @jasonjoh. This is valid.

How about instead I add a comment to the snippet that if token requests need to go through proxy then it has to be configured separately? Maybe with a link to https://learn.microsoft.com/en-us/dotnet/api/azure.core.clientoptions.transport?view=azure-dotnet#azure-core-clientoptions-transport?

This works for me as well. In my head a user may be unaware that the token requests also need to pass through the proxy, so this would be helpful in helping someone realize that.


release: fix for pypi release

fix issue with gh action that prevented release to PyPI

:tada: This PR is included in version 1.0.25 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


deploy an app into two different environments with changes between them

Short description
As a SRE I want to deploy MCCP into dev using sqlite and into UAT with postgress so that I can use gitops across environments
Acceptance criteria

[ ] Deploy MCCP v1 in dev with slqlite and using a shared config repo deploy in staging with external DB
[ ] single configuration repo with both environments
[ ] all manifests in git

Design materials

Screenshots
Design diagrams
WEP
PR/FAQ

Work breakdown
(optional)Task breakdown of the work

[ ] create X
[ ] design X
[ ] ...

Closing - duplicate of #74


test : Complete test cases for the (current-version) protocol by MessageType

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
Complete test cases for the (current-version) protocol by MessageType.  This also provides the basis for comprehensive testing of multiple versions of the protocol.
Ⅱ. Does this pull request fix one issue?

I'll add it later.
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

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

Comparison is base (1a57f2d) 49.03% compared to head (d350894) 48.97%.
Report is 1 commits behind head on 2.x.

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x    #6207      +/-   ##
============================================
- Coverage     49.03%   48.97%   -0.07%
+ Complexity     4786     4780       -6
============================================
  Files           915      915
  Lines         31896    31896
  Branches       3855     3855
============================================
- Hits          15641    15620      -21
- Misses        14699    14717      +18
- Partials       1556     1559       +3

see 6 files with indirect coverage changes

1.Please remove the seata copyright information
2.Please @author @date

建议把mockserver合完再合这个 https://github.com/apache/incubator-seata/pull/6205


Do not redirect 'bootstrap' output to a file

With https://github.com/ceph/ceph/pull/36766 cephadm will log to a file which has the same name uised in ceph-salt [1].
We should consider removing this redirect from ceph-salt (if possible) or, at least, rename it.
[1] https://github.com/ceph/ceph-salt/blob/master/ceph-salt-formula/salt/ceph-salt/apply/cephbootstrap.sls#L84

As I wrote at https://github.com/ceph/ceph/pull/36766#issuecomment-680882277 I don't think we should stop redirecting the output to a file as long as cephadm still contains print statements.

https://github.com/ceph/ceph/pull/36766 has been merged


Feat: Enable setting the install path

On mac /usr/local/bin is owned by root an may lead people to sudo or chown. Enable users to specify install path.

@iamwacko I was validating the change, it looks like the docs refer to https://getsynth.com/install. This path doesn't seem to load the updated install.sh. In reviewing this API path, it is referring to a static dir which I cannot seem to find.
Could we update this API to use https://raw.githubusercontent.com/shuttle-hq/synth/master/tools/install.sh rather than some static asset that has to be updated?

@jrschumacher /static/releases/latest/install.sh is defined in the vercel.json. It should work now though.

@iamwacko verified! Thank you.


ui: Add user initials as avatar if no icon present

Description
Fixes the missing user avatar in the header when no icon is present
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):

@blueorangutan ui

@blueorangutan ui

@davidjumani a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6070 (SL-JID-1223)

@blueorangutan ui

@davidjumani a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6070 (SL-JID-1224)


Loadbalancer setup with 80 and 443?

Just glanced over the code, and I can see that I can do http, https or TCP and then it sets the ports. Wouldn't you need port 80 and 443 in most cases?

Hi,
Yes, in most cases. However, you can customize to use other ports like 8080, 8443, etc.

@thobiast How do you currently do that with your module?

I mean, not the custom port, but two ports on one loadbalancer. Seems like an additional listener is missing. I can send a PR later if you'd take the contribution.

The current version does not support more than one listener per load balancer.
Some time ago, I worked on a new version to add multiple listeners. It is still a WIP, but works. Please, take a look at 'dev' branch.
The syntax is different.
module "openstack-lb" {
  name                  = "multiple_listeners"
  lb_description        = "Test load balancer with multiple listeners"
  lb_security_group_ids = [openstack_networking_secgroup_v2.my_lb.id]
  lb_vip_subnet_id      = xxxxx-yyyyyy
  listeners = {
    https_lb1 = {
      listener_protocol               = "TERMINATED_HTTPS"
      listener_protocol_port          = 443
      lb_pool_protocol                = "HTTP"
      member_address                  = openstack_compute_instance_v2.http[*].access_ip_v4
      member_subnet_id                = aaaaa-bbbbb
      tls_container_ref               = openstack_keymanager_container_v1.tls_1.container_ref
    },
    http_lb1 = {
      listener_protocol               = "HTTP"
      lb_pool_protocol                = "HTTP"
      member_address                  = openstack_compute_instance_v2.http[*].access_ip_v4
      member_subnet_id                = aaaaa-ccccc
    },

@thobiast I have a full example, that could "serve" as a test. I'll PR that first.
Other things:

Want to figure out how to turn on HA-mode
Want to figure out what it takes to assign a FIP to the LB

On the second part, it looks like:
resource "openstack_compute_floatingip_v2" "floating_ip" {
  pool = "public"
}

resource "openstack_networking_port_v2" "port_floating" {
  network_id = "a5bbd213-e1d3-49b6-aed1-9df60ea94b9a"
}

resource "openstack_networking_floatingip_associate_v2" "fip_to_port" {
  floating_ip = openstack_compute_floatingip_v2.floating_ip.address
  port_id     = openstack_networking_port_v2.port_floating.id
}

And then use openstack_networking_port_v2.port_floating.id with your module.
Roughly around here:
https://github.com/thobiast/terraform-openstack-loadbalancer/blob/dev/main.tf#L4-L10
I'll PR that next and then I can add that as an example as well? Maybe a docs folder?

Hi,
Regarding the first point. I am not sure if I got you. HA is configured at Octavia level.
I have not tested, but this should work to assign a FIP to the LB
resource "openstack_networking_floatingip_v2" "floatip_1" {
  pool      = "fip_pool"
  port_id     = module.openstack-lb.loadbalancer_vip_port_id
}

or using two steps:
resource "openstack_networking_floatingip_v2" "floatip_1" {
  pool      = "fip_pool"
}

resource "openstack_networking_floatingip_associate_v2" "fip_1" {
  floating_ip = openstack_networking_floatingip_v2.floatip_1.address
  port_id     = module.openstack-lb.loadbalancer_vip_port_id
}

We can add some different scenario examples in docs folder. You can also add a full working example, that way it is not required to add a test folder.
Thanks

Hi,
Regarding the first point. I am not sure if I got you. HA is configured at Octavia level.
I have not tested, but this should work to assign a FIP to the LB

resource "openstack_networking_floatingip_v2" "floatip_1" {

  pool      = "fip_pool"

  port_id     = module.openstack-lb.loadbalancer_vip_port_id

}

or using two steps:

resource "openstack_networking_floatingip_v2" "floatip_1" {

  pool      = "fip_pool"

}

resource "openstack_networking_floatingip_associate_v2" "fip_1" {

  floating_ip = openstack_networking_floatingip_v2.floatip_1.address

  port_id     = module.openstack-lb.loadbalancer_vip_port_id

}

We can add some different scenario examples in docs folder. You can also add a full working example, that way it is not required to add a test folder.
Thanks

Makes sense, I'll do that in docs.
I am still looking re: ha, our Openstack distro has an option to enable HA. Essentially multiple haproxy instances with keepalived. So I am guessing that is Octavia, but I wanted to enable it through Terraform somehow. But I will dig into terraform and gophercloud to see if that is exposed.
So the reason why I want to add the port variable is that the lb otherwise gets another public IP. And I am trying to run it only with the floating IP to preserve IPs.

Re: fip, I see what you mean now. Thanks for merging though. :)
I am trying to figure out if the openstack loadbalancer flavours is what gets me HA. I might send a PR for flavor_id.

@thobiast so, the last push adds support for the flavor_id, this allows the use of Octavia flavor's, aka, HA.

Hi,
Those issues are fixed in dev branch.
Thank you


Set up test Ruuter instance to pass requests to Turvis

AS A Product Owner
I WANT test instances of Ruuter and Turvis to be set up and Ruuter passing all of its incoming requests to Turvis
SO THAT I would have a test playground to validate the current state of Turvis and its developments
Acceptance Criteria

[ ] Ruuter's requests passing functionality based on https://github.com/buerokratt/Ruuter/issues/9
[ ] Use the latest version of Ruuter
[ ] Use the latest version of Turvis

Docker Compose

[ ] Launch turvis-test on port 8500
[ ] Launch turvis-ruuter-test on port 8505

Blocked as Ruuter does not pass all the incoming parameters to a third-party endpoint (Turvis).

Here we have open questions:

When ruuter is configured to have external forwarding, the configuration states what is the external endpoint and whether the request is forwarded with GET or POST method. This hides the context of how the initial request was initiated. e.g. when client does a GET request to ruuter and ruuter forwards the request via POST to external endpoint then external endpoint does not know if it was originally GET or POST request
When ruuter knows how to use path variables, such as /user/:id/report then the context is not recognizable in external endpoint, e.g. when ruuter does the mapping and replaces /user/431/report then external endpoint does not have the knowledge of what 431 means as the only piece of information is the path string