Drop support for EOL PHP versions

Support only current versions of php so we can leverage modern language features
http://php.net/supported-versions.php

We will support PHP5.6 at least until the end of the year (when PHP security support will end). But we can drop PHP5.4 and 5.5. Please adjust your PR and I'll merge it.

And don't we at least wanna lift the drop to 5.6 as well so at least 7.0 is required and we can start strict-typing things?

@simPod not really. At least not yet. I think we should do a prior notice before doing such major things. So we will release one more version as 5.6 and the next major version will only support PHP 7.0+

@vladar I must admit I really liked how Doctrine project did that http://www.doctrine-project.org/2017/07/25/php-7.1-requirement-and-composer.html and especially Why dropping PHP support in a minor version is not a BC break section there. But it's up to you, will add 5.6 tomorrow.

Thanks!


fix bugs in disk-expand

Include dmidecode (fixes confusing error message)
Use correct function name for logging
Don't bail on e2fsck errors; resize2fs includes an integrity pre check

/packagebuild

working on:
debian-10 (gpt)
centos-7,rhel-7 (gpt, msdos)
centos-8,rhel-8 (gpt, msdos)
created 10g instance, installed package, resized disk to 30g, rebooted, confirmed 30g
this needs a test specifically to check that the condition where root is not the last partition does not get resized.

/hold (needs packagebuild post-submit before merging)

blocking on GoogleCloudPlatform/oss-test-infra#330


IdType overwritten with UuidType

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Having the following class, where Id is Guid
public class Customer
{
    public Guid Id { get; set; }
    public string Name { get; set; }
    public string Description { get; set; }
}

Is desired to have its field representation in the schema as ID:
public class CustomerType : ObjectType<Customer>
{
    protected override void Configure(IObjectTypeDescriptor<Customer> descriptor)
    {
        base.Configure(descriptor);

        descriptor.Name(nameof(Customer));

        descriptor.Field(t => t.Id)
            .Type<NonNullType<IdType>>().Description("Id of the Customer.");

        descriptor.Field(t => t.Name)
            .Type<NonNullType<StringType>>().Description("Customer Name.");

        descriptor.Field(t => t.Description)
            .Type<NonNullType<StringType>>().Description("Customer Description.");
    }
}

This works ok until an extension over this type is declared:
public class CustomerTypeExtensions: ObjectTypeExtension<Customer>
{
    protected override void Configure(IObjectTypeDescriptor<Customer> descriptor)
    {
    }
}

public void ConfigureServices(IServiceCollection services)
{
      services
          .AddRouting()
          .AddGraphQLServer()
              .AddQueryType(p => p.Name(OperationTypeNames.Query))
              .AddType<CustomerType>()
          .AddTypeExtension<CustomerTypeQuery>()
}

When doing this it doesn't matter if the extension of the type is empty the ID field will be changed to Uuid
Steps to reproduce

Attached is the minimum to reproduce.
When the like  .AddTypeExtension<CustomerTypeQuery>() is uncommented this behaviour can be observed.
content.zip

Relevant log output
No response
Additional Context?
No response
Product
Hot Chocolate
Version
11.3.2

Also confirmed this behavior in 12.0 0-rc.1


Add ability to filter by album type e.g. full vs single

Related to #31.

Added a couple of preferences in the Settings area!
Long term we'll probably let you update this right in the album listing, but having it one place for now simplifies things.


build-plugins: build Base component too

Not sure about this, but I noticed that currently the base plugin is inlined in each plugin's dist file.

We should clean up this script later. I don't see why we need so much duplication.

So, I'm not sure about this. It will add a new dependency for each component. On the other hand, it will reduce duplication in each individual plugin dist file.
If we plan to move more stuff to the Base component, maybe it makes sense to build it separately.
Oh, and we need to document this somehow.


Update test_game_handler_bot.py

tried changing !avatar syntax to html tag

this is for #632

this is for #632

zulipbot

zulipbot

@timabbott please review

@timabbott please review

Closing, for the same reason as in #641.  Next time, I'd recommend doing two commits in a single PR making the changes in multiple files.

Closing, for the same reason as in #641.  Next time, I'd recommend doing two commits in a single PR making the changes in multiple files.


Merge main into 2.x

Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@bharath-techie  @dhruv16dhr please take a look. Thanks!


Account for staked BPT in "my pool balance" card inside invest flow

The "my pool balance" card in the pool page accounts for staked BPT. But once the user enters the invest flow, the equivalent card there does not. Minor issue, but for consistency it should.

@sssmi maybe you could look into this when you have time to work on an issue.


Allows Go Hash proxy hash server.

Go Hash allows for ‘burst mode’ hash useage where you can use a lot of hash on startup and then use a lot less when running and paying only for what you consume instead of buying one large hash key that may only be used to full potential on startup. This is a layer on top of Bossland as Bossland still provide the hashes. This is optional and set off by default parameters so it requires no applications to update and doesn’t break anything.

We don't think api should adapt to another 3d party doing business service.
Already talked about it with goman boss, aiopogo won't add / change headers.


Modality dispatch

What does this PR do?

Add modality_dispatch().
Modality-aware adversary.
Remove list-dispatch in Initializer, Projector and Composer.

GradientModifier is different because of the way we handle trainable parameters.)

[ ] Improvement (non-breaking)
[ ] Bug fix (non-breaking)
[x] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

[x] pytest
[x] CUDA_VISIBLE_DEVICES=0,1 python -m mart experiment=CIFAR10_CNN_Adv trainer=ddp trainer.precision=16 trainer.devices=2 model.optimizer.lr=0.2 trainer.max_steps=2925 datamodule.ims_per_batch=256 datamodule.world_size=2 reports 70% (14 sec/epoch).

Before submitting

[x] The title is self-explanatory and the description concisely explains the PR
[x] My PR does only one thing, instead of bundling different changes together
[ ] I list all the breaking changes introduced by this pull request
[x] I have commented my code
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have run pre-commit hooks with pre-commit run -a command without errors

Did you have fun?
Make sure you had fun coding 🙃

Do you think we should wait to merge this until we change MART to bound inputs between [0, 1] instead of [0, 255]? See #70.


Make mode of ssh identity files RO

This causes flux-helm-operator not to try to chmod them to this mode,
which in kubernetes 1.9.4+ and 1.10+, breaks, as secrets volumes are now
read-only filesystems, to mitigate CVE-2017-1002102. This effectively
reapplied the same fix to the main fluxd deployment.

does [the Helm operator's git-handling code] work OK if you make the secret volume read-only?

@tamarakaufler May be worth a look ^


exit fix

cmd: exit a
cmd: exit  1 1
cmd: exit a 1
cmd: exit<PHONE_NUMBER>999

"If n is specified, but its value is not between 0 and 255 inclusively, the exit status is  undefined.
from: info exit


gansynth: how do you convert the Generator output to the audio waveform ?

First, check out the demo for generating audio in case you haven't already, it has examples: https://colab.sandbox.google.com/notebooks/magenta/gansynth/gansynth_demo.ipynb
More specifically, data_helper.data_to_waves() is what you want https://github.com/tensorflow/magenta/blob/master/magenta/models/gansynth/lib/model.py#L381


test invoke coverage

Codecov Report

Merging #8 (8dbabb4) into main (82874fd) will increase coverage by 29.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            main       #8       +/-   ##
==========================================
+ Coverage   0.00%   29.48%   +29.48%
==========================================
  Files          6        6
  Lines        407      407
  Branches       0       68       +68
==========================================
+ Hits           0      120      +120
+ Misses       407      284      -123
- Partials       0        3        +3


Converting from zarr to standard matrix formats

Estimate: 3 days
Conceptually similar to the HTTP Accept request-header, the current v0 API draft allows the API caller to request one of the following de facto standard formats to be returned, which in most cases requires a conversion from the internal zarr storage format:

zarr [default]
parquet
loom [v0]
csv [v0]
anndata
singlecellexperiment

Prior to starting the v0 API implementation sprint, a prototype needs to be written to validate this approach.

Marcus has completed his research. We're adding CSV to the list of conversion formats and delivering the set formats during both v0 and v1.


Fix sample code on the section "Supports Google App Engine" of README

Thank you for your great job.
I fixed sample code on the section about "Supports Google App Engine" of README

Thank you!


Rendering links and added tabular formatting to fields, Added Emoji file to script

@GeekGuy-29 Please review this.

Thank you so much man!!

@GeekGuy-29 No problem bro. We will make it together❤

@A-Spiral-Forge I wanted to ask one thing, UI/UX for the website is being built so, once hard coded in HTML, can you in-lay the functionalities ??
As in, if you see the conversation in #7 you'll get to see I've sent in a design and in that there are buttons so, can you patch the script of each JSON files with the buttons?

@GeekGuy-29 It's not a big deal. We are dynamically rendering the data so just changing the tag in JS file can change the formatting. We can do this once the design is completely ready for data rendering.

Cool @A-Spiral-Forge
I'll ping you as soon as I have something for you
Until then take care and Happy Hacktoberfest <3

@GeekGuy-29 Sure. Happy Hacktoberfest <3


[snowflake] make various small bug fixes to get majority of tests to pass

Current dependencies on/for this PR:

main

PR #4   👈

This stack of pull requests is managed by Graphite.


Change read chunk size to higher than 20MB

Hi,
We use goofys agains our own S3 strorage, which has a 164MB block size.
Is there any way to define a read ahead chunk size bigger than 20MB ?
Thanks

which S3 system is this? Using a larger readahead chunk size certainly makes sense, could you suggest a heuristic (probably base on file size and available memory) that would work for most people without asking users to specify a particular block size?


安卓 背景闪烁 Android background flashing

Use this library to cause Android background flicker
To Reproduce
Steps to reproduce the behavior if possible, or a link to a reproduction repo:

After adding, deleting this library is normal. I don't know why. Please let me know

Expected behavior
Use this library to cause Android background flicker.
Screenshots

Dependency versions

React Native version: 0.63.2
React version: 16.13.1
React Native Fast Image version:8.3.4

Hey @lhb11. Does your app crashes?

Hey @lhb11. Does your app crashes?
There is no crash, but the background color will flicker back and forth, such as white and blue. Maybe blue will cover white. If the current library is annotated, it will be normal. I wonder if the image is refreshing the page

code?

same


[+] Broken Go Build

Hi, I created https://github.com/mgord9518/aisap/issues/17, and today I had some free time and wanted to see if I can build it myself from scratch. I tried to do it first on x86_64
I followed the build.sh instructions, https://github.com/mgord9518/aisap/blob/main/resources/build.sh
But I got:
github.com/mgord9518/aisap/spooky
github.com/mgord9518/aisap/permissions
# github.com/mgord9518/aisap/spooky
../../spooky/isspooky.go:4:2: "encoding/json" imported and not used
../../spooky/isspooky.go:21:1: missing return
github.com/mgord9518/aisap/profiles
github.com/mgord9518/aisap

I wonder if that only affects me (I have a crooked go env setup), anyways, I fixed it by:
#Removed "encoding/json" from line 4
 import (
- 	"encoding/json"
 	"strings"

#Added a tempp return at line 20 to avoid the "missing return" error
func isSpookyNew(path string) bool {
+ 	return false
}

$ du -sh aisap && file aisap
4.3M    aisap
aisap: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), static-pie linked, stripped

$ aisap --version
v0.10.2-alpha

It seems like @xplshn also ran into this:
https://github.com/mgord9518/aisap/compare/main...xplshn:aisap:main

Fixed on my repo, switched to vendored

@xplshn Looks like you're doing really good work in your repo, would you like to push your changes upstream?

Hi, good (pick yours: evening, afternoon, morning, noon), I very much would, but I don't think a PR would be that easy because I forked various repos and then changed everything to point to those instead, since I was having problems and wanted a bit more freedom to change stuff. You can probably still add this upstream tho!
Anyways, just to give you a heads up; I am working on Chains, mostly a rewrite of AISAP's Go version, will probably become a PR here replacing the entirety of the Go AISAP implementation if I can get it to a decent state, it does take lots of code here, but I am trying to use as most of the STD as possible, and I am trying to follow a procedural paradigm, so that the code remains readable.
:)

@mgord9518 I can try to PR the changes made to wrap.go and helpers.go since that's the biggest change and it fixes the hardcoded env vars issue.
Although the CI will still be failing.

@xplshn That sounds awesome! I'd love to collaborate to ensure compatibility between our versions (permission implementation, profiles, etc). I've been trying to transition to storing most of everything in JSON but the progress has been slow. As long as we agree on the formatting it should be fairly easy to keep things in sync though
@Samueru-sama Thanks! I'll see if I can re-write the build script (probably into a Makefile?) in the next week or so

@xplshn That sounds awesome! I'd love to collaborate to ensure compatibility between our versions (permission implementation, profiles, etc). I've been trying to transition to storing most of everything in JSON but the progress has been slow. As long as we agree on the formatting it should be fairly easy to keep things in sync though

Yeah, it seems like you're moving to Zig, so I wanted to eventually apply to maintain the Go version of AISAP, which for now I'm locally calling chains.
I discussed with Samu about using .json for permissions earlier, but we agree that having them in the .desktop will be easier for appdir maintainers, if you want to reach out to me, I'm on Telegram, and @Azathothas, @Samueru-sama and I are working collaboratively on projects related to having portable Linux binaries, you can join and discuss these kinds of things if you'd like :)
This is my Telegram handle btw: xplshn_2008

@xplshn Unfortunately I don't have Telegram but I have Matrix and Discord if either work (or even just opening a discussion here on Github)
I think desktop files are the way to go for embedding into AppImages, JSON would just be for the internal profile library (which would only be edited by maintainers) and either compiled or embedded directly into the aisap binary.

I've started a discussion here

Hi @mgord9518 , for the go version, which you don't intend to maintain,
Fixing the isspooky.go would be much appreciated as it's the only thing that's breaking the go build.
For the zig version, you say it's not ready yet, so I will default to building the go version for now.
I have also resolved https://github.com/mgord9518/aisap/issues/17 , but I am leaving it open in case you yourself want to include aarch64 builds
There's a matrix space at : https://matrix.to/#/%23official-loonix:matrix.org
But it's abandoned, because everyone wanted bigger file uploads and simplicity, which telegram offered.... (because Appimages...)
If you are still working on appimages or related formats, it would be worth it to join the telegram channel at: https://t.me/official_loonix/63949
We have big plans for appimages, & portable formats in general.
There's a hope we can have a centralized repo (with transparent build process, logs and verification), that will have all the appimages, and other portable formats, similar to my previous work over at: https://github.com/Azathothas/Toolpacks
And to offer it all for free, and with predictable endpoints and metadata so anyone or any package managers, can use it.
If all of that sounds exciting, please join us in the discussions, it would be very helpful.
Thanks!

Btw, I'm making progress with restructuring AISAP and migrating to using the official goappimage library at https://github.com/xplshn/chains :)
We can probably re-organize this repo into two separate folders, one with the Go implementation and one containing the Zig one, and the docs which shall be on the top level. Just leaving it out there, could probably PR all of the changes once I am done :)

@xplshn That sounds perfect!

I'm not sure if I'll ever actually finish it, currently I'm working on other projects. Such as the new AppBundleHUB and its webpage, as well as on the PELF tooling which is used to generate .AppBundles


Messy Project

currently, all the code to generate surfaces is scattered throughout the surfaces module, making it quite difficult to acquire an overview over the codebase.
The various methods should be centralized into a library, which is not in the surfaces module.
This will require some refactoring.

I should also take the time to write some minimal documentation for these functions.


Test package-installed server startup

This new test runs only on travis, normally
Issue #1008

Actually, maybe we should do the install, then run the travis_only test, then pip install requirements.txt, then do the regular tests (to more realistically simulate a pip install that an end user might be doing) ... what do you think, @jeromekelleher ?

(also, we should get this PR to a point where it is actually failing, so we know that it reproduces the bug we're trying to fix)

I think we'd need to do this in a different travis target that gets run before any of the others. This should be something like
$ python setup.py sdist
$ pip install <tarball>
$ ga4gh_server --version

This should be enough to shake out the majority of problems. We could then proceed with the rest of the tests as before, perhaps after running pip install -r requirements.txt?

Excellent!  This new version catches the failure (from the Travis CI output):
$ ga4gh_client --version
Traceback (most recent call last):
  File "/home/travis/virtualenv/python2.7.9/bin/ga4gh_client", line 7, in <module>
    from ga4gh.cli import client_main
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/ga4gh/cli.py", line 23, in <module>
    import ga4gh.frontend as frontend
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/ga4gh/frontend.py", line 21, in <module>
    import oic.oauth2
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/oic/oauth2/__init__.py", line 9, in <module>
    from oic.oauth2.base import PBase
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/oic/oauth2/base.py", line 7, in <module>
    from oic.utils.keyio import KeyJar
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/oic/utils/keyio.py", line 18, in <module>
    from jwkest.jwk import rsa_load
  File "/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/jwkest/jwk.py", line 13, in <module>
    from Crypto.PublicKey.RSA import RsaKey
ImportError: cannot import name RsaKey

@jeromekelleher ready for review

Excellent, great work @dcolligan! Now, how do we catch the full dependency tree? I think it's enough to use the fancy tool you found to populate  requirements.txt by hand, and to update it periodically. What do you think?

That sounds good, although specifying and updating the libraries is going to be a bit cumbersome with so much specified.  Here is the list for the project:
$ pipdeptree -f 2> /dev/null
flake8==2.3.0
    pyflakes
    mccabe
    pep8==1.6.2
guppy==0.1.10
humanize==0.5.1
nose==1.3.7
lxml==3.4.4
yubico-client==1.9.1
    requests==2.7.0
mock==1.0.0
oic==0.7.6
    requests==2.7.0
    pycrypto==2.6.1
    pyjwkest==1.0.1
      pycrypto==2.6.1
      requests==2.7.0
      six==1.10.0
      future==0.15.2
    Mako==1.0.4
      MarkupSafe==0.23
    Beaker==1.8.0
      funcsigs==0.4
    alabaster==0.7.7
    pyOpenSSL==0.15.1
      six==1.10.0
      cryptography==1.3.1
        idna==2.1
        pyasn1==0.1.9
        six==1.10.0
        setuptools
        enum34==1.1.2
        ipaddress==1.0.16
        cffi==1.5.2
          pycparser==2.14
pyaml==15.03.1
    PyYAML==3.11
wsgiref==0.1.2
coverage==3.7.1
freezegun==0.3.6
    python-dateutil
    six==1.10.0
CherryPy==3.2.4
pysam==0.8.4
PyVCF==0.6.7
    distribute
Flask-Cors==2.0.1
    Flask==0.10.1
      Werkzeug==0.11.5
      Jinja2==2.7.3
        MarkupSafe==0.23
      itsdangerous==0.24
    six==1.10.0
avro==1.7.7
snakefood==1.4

OK, we should be able to separate out the deployment and dev packages out of that easily enough. At first glance, we have
snakefood==1.4
PyVCF==0.6.7
freezegun==0.3.6
    python-dateutil
    six==1.10.0
coverage==3.7.1
CherryPy==3.2.4
flake8==2.3.0
    pyflakes
    mccabe
    pep8==1.6.2
guppy==0.1.10
nose==1.3.7

are all for dev?

@jeromekelleher right, check the most recent push... I think this one is good to merge

(and I don't think it's necessary to pin dependencies of libraries in dev-requirements... the main point of this change is to ensure our shipped installables don't break... development environment breaking is of much lower importance)

Looks good modulo a few extra packages in requirements.txt. I think it's important we pin requirements for development, it would be very frustrating for developers finding their tests running locally but failing in CI (or vice versa) because of some weird dependency issues.
It shouldn't be too hard to do a package refresh every now and again to bring them up to date.

@jeromekelleher updated

Do we use these in deployment?
PyVCF==0.6.7
PyYAML==3.11
requests==2.7.0
Sorry, should have seen these a minute ago.

@jeromekelleher updated
PyVCF and PyYAML are not needed in deployment; I moved them to dev
requests is required by oic, which is needed in deployment

Excellent, thanks @dcolligan. @reece makes great points over in #1007, but we have a particularly troublesome dependency graph here and there's very little we can do about it if we want to support authentication (which we do). Let's try this out for a while and see how it works. We can always have another look before we release.

What do the rest of us do when we add packages needed for new functionality?

Good question.  Here's the algorithm:
Determine if the new library can be reached via a shipped executable.  If so, place it in the bottom section in development.txt and use pipdeptree to find any dependencies of that library that aren't already listed in development.txt and put them in the top section.  If the new library can not be reached via a shipped executable (e.g. only in tests, scripts, etc.), do the same, but in dev-requirements.txt (with the exception that no libraries should be added to dev-requirements.txt that are already in requirements.txt because the former imports the later -- pip will give an error on double-imports anyway).  Every library needs to be pinned to an exact version -- exceptions are only made in case of incompatible version issues.

Every time anyone makes a change to either requirements.txt or dev-requirements.txt they will probably need to do this.  There might be a way to automate this.


Update Pages “other-page”

Automatically generated by Netlify CMS

👷 Deploy Preview for digital-minck-be processing.

Name
Link

🔨 Latest commit
ab76642eadebad51df096a5c07716622fb6eb967

🔍 Latest deploy log
https://app.netlify.com/sites/digital-minck-be/deploys/662c918d579a3200073c7def


pmm: ratings - two ratings (top/center) - ratings/images stacked

Version Number
1.18
What branch are you on?
master
Describe the Bug
using -pmm: ratings (overlay_level: episode)
selecting just two ratings for overlay, selecting:
vertical_position: top,
horizontal_position: center,
alignment: horizontal
leads to stacking ratings / images:
log:
Conditional: rating1_horizontal_offset Condition 1 Failed: rating_alignment "horizontal" is not "vertical" Condition 2 Failed: rating2 "audience" is not "none" Condition 3 Failed: rating2 "audience" is not "none" Conditional Variable: rating1_horizontal_offset is "-<<ch2_offset>>"
and
Conditional: rating2_horizontal_offset Condition 1 Failed: rating_alignment "horizontal" is not "vertical" Condition 2 Failed: rating1 "critic" is not "none" Condition 3 Failed: rating1 "critic" is not "none" Conditional Variable: rating2_horizontal_offset is "-<<ch2_offset>>"
line 240 needs to end with:
value: +<<ch2_offset>> or value: <<ch2_offset>>
instead of:
value: -<<ch2_offset>>
or more detailed rating1_horizontal_offset leads to line 164 which is exactly the same as rating2_horizontal_offset ending in line 240.
It looks like this:

Relevant Collection/Overlay/Playlist Definition
No response
Logs
https://github.com/meisnate12/Plex-Meta-Manager/blob/master/defaults/overlays/ratings.yml

fixed in nightly thanks for letting me know

You are very welcome - I appreciate your work and efforts.


Node under cursor not highlighted as before

A while ago if I open the query buffer and move the cursor around in the code buffer, the node for the code under the cursor would be highlighted/selected in the query buffer. This behavior is also shown in the screenshot in the README of this project.
This was very useful, but it seems like it got broken at some point. Currently it's very difficult to find a node for a code because the query buffer no longer follows the cursor in the code buffer.

What version nvim are you using? I can't reproduce it.
On Sun, Feb 14, 2021, 12:28 AM Ömer Sinan Ağacan<EMAIL_ADDRESS>wrote:

A while ago if I open the query buffer and move the cursor around in the
code buffer, the node for the code under the cursor would be
highlighted/selected in the query buffer. This behavior is also shown in
the screenshot in the README of this project.
This was very useful, but it seems like it got broken at some point.
Currently it's very difficult to find a node for a code because the query
buffer no longer follows the cursor in the code buffer.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nvim-treesitter/playground/issues/24, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPXOSUVBWPACBAH5ZMWEVTS65USNANCNFSM4XS5LHMA
.

NVIM v0.5.0-dev+2894-g5ad32885d
playground 7e373e5

I can't check which version is edge but I would suggest using the latest
version of both. The playground was reworked to support nested languages
that was merged into neovim. I can't check is those versions are the latest
(mobile).
On Sun, Feb 14, 2021, 7:26 AM Ömer Sinan Ağacan<EMAIL_ADDRESS>wrote:

NVIM v0.5.0-dev+2894-g5ad32885d
playground 7e373e5
https://github.com/nvim-treesitter/playground/commit/7e373e5706a2df71fd3a96b50d1f7b0c3e7a0b36

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nvim-treesitter/playground/issues/24#issuecomment-778777699,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPXOSTPAT3IOYERVVAVLDDS67FO3ANCNFSM4XS5LHMA
.

Those versions are latest, yes. I updated both today, from git repos, master branches.

Can you provide a reproducible file/scenario I can use to reproduce?
On Sun, Feb 14, 2021, 7:30 AM Ömer Sinan Ağacan<EMAIL_ADDRESS>wrote:

Those versions are latest, yes. I updated both today, from git repos,
master branches.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nvim-treesitter/playground/issues/24#issuecomment-778778276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPXOSVMZ75AP2XMHSUU7VTS67GA7ANCNFSM4XS5LHMA
.

Unfortunately this is still not fixed. I will upload a gif if I can figure out how to record gifs on Linux.

Hmmm. I'm thinking this may be something localized to your install. The fix
provided fixed the highlighting when not displaying anonymous nodes which
was the issue I was seeing. Can you provide a minimal reproduction with a
minimal init.vim/lua?
On Wed, Feb 17, 2021, 9:33 PM Ömer Sinan Ağacan<EMAIL_ADDRESS>wrote:

Unfortunately this is still not fixed. I will upload a gif if I can figure
out how to record gifs on Linux.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nvim-treesitter/playground/issues/24#issuecomment-781023722,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPXOSXV3MARAH54VCXKRKDS7SDATANCNFSM4XS5LHMA
.

I'm on newest nvim and playground and also get nothing when the cursor is sitting on the capture name
e.g.
[
    "for"
    "while"
    "if"
    "else"
] @keyword

@stumash can you share all the steps you are following and the files you are using?

I guess if this gets resolved it will be here https://github.com/nvim-treesitter/playground/issues/51


Browserstack fail

Yesterday failed with Error: Access to API denied.
Today failed with Error: Cannot queue more than 200 workers.
Should investigate and contact browserstack about these.

For Error: Access to API denied. we got our account activated yesterday.
For Error: Cannot queue more than 200 workers. we have to wait until all queued tasks are complete.
Now the browserstack works.


Add proto messages representing all account balances

Addresses #852 in hashgraph/hedera-services

@ljianghedera should the protos for the account balances file be in its own proto file?

I would think the CryptoGetAccountBalance is an appropriate location for these proto messages as they are all associated the account balances queries.


Set HSTS max-age to 5 minutes

Increase the max-age for the Strict-Transport-Security header to 5 minutes (300).

Closed by #46.


Remove use of code deprecated in Gaffer

For Gaffer alpha-0.1, deprecated code will be removed (https://github.com/gchq/Gaffer/issues/2552). A change is required to gaffer-tools to replace use of code which has been deprecated.

Closed by #970


Stop ignoring order of primitives in colocation constraints

cs_colocation, when used with the pcs provider, makes the incorrect
assumption that the order of primitives in colocation constraints is
arbitrary. It is not, and the following must not be equivalent:
cs_colocation { 'vip_with_service':
primitives => [ 'nginx_vip', 'nginx_service' ],
}
cs_colocation { 'vip_with_service':
primitives => [ 'nginx_service', 'nginx_vip' ],
}
These mean different things. The first example ensures that nginx_vip
runs on whatever node nginx_service runs on, and if nginx_service
can't be started on any node, Pacemaker won't even attempt to start
nginx_vip. However, if nginx_vip cannot be started on any node,
nginx_service will continue to be available.
The second example means the opposite. In it, nginx_service is
configured to run wherever nginx_vip runs, and if nginx_vip can't run
anywhere, nginx_service won't start. And if nginx_vip cannot be
started on any node, nginx_service will not be available.
As a result, drop artificial sorting for the primitives, and use them
exactly as specified.
Fixes issue puppet-community/puppet-corosync#150.

Sorry, accidentally pushed two separate PRs to the same branch. Will close and recreate.


documenting Struct/ Enum/ Constant  outside of contract

Hi, I'm attempting to document an EIP-2535 Diamond contract that utilizies an 'AppStorage' pattern such as in AAVEGotchi  Thing is this pattern makes extensive use of structs outside of contracts.  To which I cant seem to docgen to register it.
Is documenting structs outside of contracts possible, or if not, planned to be implemented?
Thanks!

specifically speaking
/**
* Some documentation about A
* @param b some documentation about
*/
struct A {
uint256 b;
}

contract {
...
}
}

Thanks for reaching out @Joe-mcgee. This is not currently supported. I'm interested in it but the code is structured around documenting contracts, and I haven't figure out how to implement it.

Thanks for looking into it and letting me know @frangio :)

@miltonjonat This is currently supported in the 0.6 release.

Ok @frangio thanks! Is there any open issue covering having NatSpec comments showing up in structs?


no security tests?

add nocode tests

nocode is reliable and secure enough to exclude tests. We're going for notest too!

nocde+notest => nosecurity?
or is it nocode * notest => security?


Remove duplicate reference to Contributing doc in readme

Change Type

[ ] Feature
[ ] Chore
[x] Bug Fix

Change Level

[ ] major
[ ] minor
[x] patch

Further Information (screenshots, bug report links, etc.)
The first link to the Contributing doc in the readme no longer links to Contributing.md successfully. However https://github.com/entropic-dev/entropic/pull/131 added a separate paragraph to the readme with a working link to Contributing.md, so this PR removes the link that no longer works.
Checklist

[x] Added tests / did not decrease code coverage
[x] Tested in supported environments (common browsers or current Node)

Merged in https://github.com/entropic-dev/entropic/commit/5acb8dd6306a491c6b3bd5bb0c87dd77df7059d7 ! Thanks!


Deployable on edge?

What are some known limitations when using Shopware Frontends in a high traffic environment. For example the current PWA solutions requires a lot of resources when deployed in a production environment.
Basically even with a sep. caching layout for api responses you will be limited by the number of instances you can manage because your servers will run out of memory.
Making Shopware Frontends deployable on Edge, for example: https://vercel.com/docs/concepts/edge-network/overview would be a major benefit. Is this currently possible as it is basically Nuxt (https://nuxt.com/docs/getting-started/deployment) ?

Hey @azngeek! Currently project supports deployment on vercel-edge for ./templates/vue-demo-store, but you have to disable ssr for nuxt:
//nuxt.config.ts
ssr: false,
nitro: {
   preset: "vercel-edge",
}

we need to keep in mind that edge functions have a lot of limitations -> https://vercel.com/docs/concepts/functions/edge-functions/limitations#many-node.js-apis-are-not-available in order to make the solution faster (tl;dr: the environment does not have nodejs but only JavaScript API)
Problem I found was that useAsyncData we currently use for page resolving (uses server prefetch mechanism under the hood) does not work well on vercel-edge, and there is a problem with hydration. You can deal with that for example, by enabling lazy mode, or disabling server for the useAsyncData (globally it's solved by disabling mentioned ssr).

Closed for now. If you have feedback feel free to reopen.


ticket_from_message() does not send updates to existing TicketCC objects (subscribers)

TicketCC objects (henceforth referred to as subscribers) get created for email addresses in the CC email field that aren't already subscribed. When the email is sent out, subscribers are skipped (presumably because we wouldn't want to send the messages to the users that were CC'd on this email since they have already received the email). However, the subscribers that existed prior to this will also not receive an email. This includes subscribers that were added via the TicketCC form and subscribers that were added via the HELPDESK_AUTO_SUBSCRIBE_ON_TICKET_RESPONSE setting.
For example,

User creates ticket
Support agent responds to the ticket, but is not assigned to the ticket, and becomes subscribed via the HELPDESK_AUTO_SUBSCRIBE_ON_TICKET_RESPONSE setting
User responds to the ticket via email

In this scenario, the support agent would not receive an email with the user's response in #3.
At line 520 or /management/commands/get_email.py  we can see the only emails that are sent are sent to the person the ticket is assigned to and the updated_ticket_cc email address:
if new:
    # omitted
else:
        context.update(comment=f.comment)
        if t.assigned_to:
            send_templated_mail(
                'updated_owner',
                context,
                recipients=t.assigned_to.email,
                sender=queue.from_address,
                fail_silently=True,
            )
        if queue.updated_ticket_cc:
            send_templated_mail(
                'updated_cc',
                context,
                recipients=queue.updated_ticket_cc,
                sender=queue.from_address,
                fail_silently=True,
            )

What should happen instead is the existing subscribers that are not already included in the email's CC field should have the email response sent to them.

Thanks! The ticket CC's now get a copy of the email. This might result in duplicates as we're not testing for who sent the email, etc., but this is probably better than not getting any message at all.

This is sort of a stop-gap in the 0.2.x series that is in maintenance mode. A more correct solution is going to be part of the 0.3.x release, you can take a look at develop branch to see progress.


Va merge

This is the merge of the variant_annotation branch into master

is it different from #930?

-1 until @david4096 's question is answered.

No, it's the same.  Wasn't aware of that other PR.  Closing this one...


Resolve IP Address for spark.es.nodes param

This request is a proposition to fix the following situation :
With the parameters :
es.nodes.discovery = false
es.nodes.client.only = false
es.nodes.data.only = true
es.nodes = abc.example.com
With abc.example.com resolving to <IP_ADDRESS>
The data node on abc.example is incorrectly filtered because the InitializationUtils class performs a comparison between the ip extracted from "_nodes/http" and the hostname which resolves to that ip.
When enabling debug, you then get the following situation:
"DEBUG ScalaEsRDD: Found data nodes [<IP_ADDRESS>:9200]"
"DEBUG ScalaEsRDD: Filtered discovered only nodes [abc.example.com:9200] to data-only []"
Which raises an EsHadoopIllegalArgumentException : "No data nodes with HTTP-enabled available; node discovery is disabled and none of nodes specified fits the criterion [abc.example.com:9200]"
The proposed solution resolves the ip for the hostname when qualifying nodes which allows consistency for subsequent comparisons.

Thanks for raising this. Please sign the CLA in order to consider it.

Merged in master. Thanks for your contribution!

No problem @costin, thank you looking at it !

This still doesn't work with a URL as such:
https://example.com
@pbamba any possible chance for a fix? It seems like the code is stripping out the 'https://' from the URL and then trying to resolve the hostname.

@shamak This has been fixed through #652 in master. And please, open up a new issue instead of commenting on a close PR.
Thanks,


ScrollToFixed .summary not working when click more button.

Hi,
I have one problem about fixed summary function. I have created a social networking website.I want to use ScrollToFixed plugin for profile picture. So i will try the ScrollToFixed plugin but it is not working when i click load more button. The user main page seeing only 20 post then after 20 post they see show more post button. When user click Show More Post the new profile pictures not fixed.
I have create a video from YouTube: https://www.youtube.com/watch?v=bNGb7OWuXnQ&feature=youtu.be
If anyone can help me i will be very happy. Thanks for this plugin....

Please see answer in #171.


docs: Attach SBOMs as attestations

Summary

As discussed in RFC #https://github.com/sigstore/cosign/issues/1742, we decided to add a note where we suggest users to include the SBOM in an attestation as a way to ensure the SBOM is tamper-proof.
Ticket Link

Fixes #https://github.com/sigstore/cosign/issues/1742
Release Note

docs: add note about attaching SBOMs in attestations

ping @sigstore/codeowners-sigstore-website


setRatingName doesn't work on latest Action Build

Describe your bug here.

[Current Action Build]

[Current Release Build]
Both images are using the same script, which looks like this:
function onUpdate()
    -- Changes Names to Letters on Score Text
    if ratingName == 'Perfect!!' then
    setRatingName('SSS')
    end
    if ratingName == 'Sick!' then
    setRatingName('SS')
    end
    if ratingName == 'Great' then
    setRatingName('S')
    end
    if ratingName == 'Good' then
    setRatingName('A')
    end
    if ratingName == 'Nice' then
    setRatingName('B')
    end
    if ratingName == 'You Suck!' then
    setRatingName('C')
    end
    if ratingName == 'Meh' then
    setRatingName('D')
    end
    if ratingName == 'Bruh' then
    setRatingName('E')
    end
    if ratingName == 'Shit' then
    setRatingName('F')
    end
    end

Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No.

I think it's meant to be used on goodNoteHit() as that's what updates the rating

I think it's meant to be used on goodNoteHit() as that's what updates the rating

it would probably also work with onUpdate since it gets called every frame

Important to note that the script is constant here, so its 100% a source thing


Faster working with LzoBinary data

Use a thread pool for reading lzo indexes can cut launch time for a job from several minutes to the low seconds.

Changes in LzoInputFormat duplicate my work in #434 while providing less RPC savings. I would not put multithreaded status fetch in eb because Hadoop 2.3+ already provides it.

The main win for us has been reading the index in parallel, not actually getting the status that I've seen. Our PR's are independent from each other in effect no?

I see, then we can compose PR's nicely. Reading index in parallel cannot be pushed into hadoop FileInputFormat. With #434 we can submit index file reads to an LzoInputFormat thread pool from listStatus only when they exist, and harvest the results in getSplits

This was done earlier. One of the issues was total memory required to keep the whole index in memory (we can't). We could avoid this with a limit on indexes buffered. I will check why that patch finally didn't make it.


Documentation + examples for Square, Dot, Circle and Rectangle

Motivation
I was browsing the source code and noticed that many of the basic/introductory mobjects did not have documentation or examples, so I thought it would be worthwhile to add some.
Overview / Explanation for Changes
Adds documentation and examples for

Circle + surround + point_at_angle
Dot
Square
Rectangle

Further Comments
None
Acknowledgements

[x] I have read the Contributing Guidelines
[x] I have chosen a descriptive PR title (see top of PR template for examples)

Reviewer Checklist

[ ] Newly added functions/classes are either private or have a docstring
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[x] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings
[ ] The PR title is descriptive enough

I've updated the changes to include all parameters, fix the typing issues, and adjust the circle example. The parameters without descriptions are ones that I suspect do nothing, but that can be addressed in a later issue.

Looks really good!
Thanks a lot.


Add Zuplo to Development

[X] My submission is formatted according to the guidelines in the contributing guide
[X] My addition is ordered alphabetically
[X] My submission has a useful description
[X] The description does not have more than 100 characters
[X] The description does not end with punctuation
[X] Each table column is padded with one space on either side
[X] I have searched the repository for any relevant issues or pull requests
[X] Any category I am creating has the minimum requirement of 3 items
[X] All changes have been squashed into a single commit

@marcelscruz - any chance you can review?

Thank you!


make gam available as a Homebrew package ???

any chance that gam could be in the homebrew repositories?
It would be delightful to just type brew upgrade gam and be done
gam is awesome by the way
I manage groups & lists for a college CS department and it's so nice to have this tool! Especially since I always have to update the lists at the busiest time of year.

Just type the following: bash <(curl -s -S -L https://gam-shortn.appspot.com/gam-install) -l

I know, it's just nice to have one standard way of updating all my dozens of 3rd-party command line tools

As a macOS user, the first thing I look for are Homebrew versions of utilities because they are easier to install and maintain.  While I know the current install of gam is only 1 line, it is far easier for me to remember brew install gam over the current one.  I can also tell brew to update by simply saying brew update gam.  It's a standardized syntax.
As @daethnir mentioned, the bar is higher to get code accepted into Homebrew versus just publishing it as is being done now.  What is appspot.com, anyway?  How does an admin go about validating install links, especially if they're inexperienced?  The point of the previous questions is to illustrate that it is easier to use and trust a tool like Homebrew because there is some validation involved.
Please add Homebrew support!

MacBook-Pro ~ % bash <(curl -s -S -L https://gam-shortn.appspot.com/gam-install)
curl: (7) Failed to connect to raw.githubusercontent.com port 443: Operation timed out

PRs welcome. We have a lot of requests for adding GAM to <yet another packaging system on OS that doesn't actually have native packaging solved> FWIW, I'll never understand how it is Microsoft and Apple have been building OSes for half a decade and still haven't figured out a reliable solution to packaging / installers.
Anyway, PRs are always welcome here if GAM needs some code to integrate or just submit GAM to said packaging system and feel free to let community know about it.


Added repo reference

Contributing
If you want to add a project to this list, please make sure that:

[x] You read the contributing guide
[x] The repository has hacktoberfest as topic and counts for hacktoberfest event
[x] The project can't be just an "add your name" or simply + 1 contribution to hacktoberfest
[x] Place them in alphabetical order
[x] One repository per PR/contribution - avoid conflicts and simplify review
[x] Your pull request is not spammy just adding an empty line, otherwise it will be marked as invalid and not count to hacktoberfest.

If they meet the requirements above, feel free to create a PR. Happy Hacking! :sparkles:

Please accept my pull requerst

Can you add Hacktoberfest-accept label?


fix: minor linting

Description

This addresses some of the items marked as Minor in our Sonar Cloud report
The idea is to reduce down some of the very small issues so that we can more easily triage the intermediate and major items that have been flagged

Things included:

Typos
Options chains
Nullish coalescing
Readonly attributes on classes
Combining type imports
Empty fragments
Unnecessary return

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


getting error while upload the screenshot and generating the code

Hi,
I have attached the error image below can you check and solve the issue?

Thanks & Regards,
Gangadhara Rao Ande.

Hi, please run the open source version or buy a subscription.  We don't offer support for the hosted version with your own API key.


[Bug]: cmdow installed without working shim

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.
[X] I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name
cmdow
Expected/Current Behaviour
cmdow should be installed with working shim.
Current behaviour:
❯ scoop install extras/cmdow
Installing 'cmdow' (1.4.8) [64bit] from 'extras' bucket
Downloading https://github.com/ritchielawrence/cmdow/archive/v1.4.8.zip (-1 B)...

Checking hash of v1.4.8.zip ... ok.
Extracting v1.4.8.zip ... done.
Linking ~\scoop\apps\cmdow\current => ~\scoop\apps\cmdow\1.4.8
Creating shim for 'cmdow'.
InvalidOperation: C:\Users\user\scoop\apps\scoop\current\lib\core.ps1:19
Line |
  19 |          $binaryReader.Close()
     |          ~~~~~~~~~~~~~~~~~~~~~
     | You cannot call a method on a null-valued expression.
InvalidOperation: C:\Users\user\scoop\apps\scoop\current\lib\core.ps1:20
Line |
  20 |          $fileStream.Close()
     |          ~~~~~~~~~~~~~~~~~~~
     | You cannot call a method on a null-valued expression.
'cmdow' (1.4.8) was installed successfully!
❯ cmdow
Shim: Could not determine if target is a GUI app. Assuming console.
Shim: Could not create process with command '"C:\Users\user\scoop\apps\cmdow\current\bin\Release\cmdow.exe" '.

Steps to Reproduce
scoop install extras/cmdow

Possible Solution
I don't know.
Scoop and Buckets Version
❯ scoop --version
Current Scoop version:
be56faf2 (HEAD -> master, tag: v0.5.1, origin/master, origin/HEAD) chore(release): Bump to version 0.5.1 (#6057)

'main' bucket:
6f9be62da (HEAD -> master, origin/master, origin/HEAD) saml2aws: Update to version 2.36.17

'extras' bucket:
6f4630e9e (HEAD -> master, origin/master, origin/HEAD) sidekick-browser: Update to version <IP_ADDRESS>616-1131b84

'versions' bucket:
ce4b950ff (HEAD -> master, origin/master, origin/HEAD) labplot-nightly: Update to version 6972

'nirsoft' bucket:
b38031d (HEAD -> master, origin/master, origin/HEAD) updated manifests

'sysinternals' bucket:
47f9751 (HEAD -> main, origin/main, origin/HEAD) chore(ci): Update 'psmodulecache' to 'main'

'nonportable' bucket:
1000f86 (HEAD -> master, origin/master, origin/HEAD) nodejs-np: Update to version 22.5.1

'java' bucket:
5c1770d3 (HEAD -> master, origin/master, origin/HEAD) zulufx8-jre: Update to version <IP_ADDRESS>

'games' bucket:
6fdf267e4 (HEAD -> master, origin/master, origin/HEAD) duckstation-preview: Update to version 20240721-g533022b

'naderi' bucket:
1e4a7eef (HEAD -> master, origin/master, origin/HEAD) vscode-insider: Update to version<PHONE_NUMBER>039

Scoop Config
❯ scoop config

last_update       : 21.07.2024 16:34:22
use_external_7zip : True
scoop_repo        : https://github.com/ScoopInstaller/Scoop
scoop_branch      : master
use_sqlite_cache  : True

PowerShell Version
❯ echo $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.4.3
PSEdition                      Core
GitCommitId                    7.4.3
OS                             Microsoft Windows 10.0.22631
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Additional Softwares
No response

Depending on the antivirus software you are using and its settings it could block installation or quarantine binary in case of threat detection (see e.g. Is Cmdow Malware?).
Ensure first that it's not your case.


Respect required parameters

We currently don't validate on whether a parameter is mandatory.  We should.

Partially fixed via #62 - this visually indicates required parameters but does not perform additional validation.  Should update cnabjs to do this, which will cascade automatically to the self-installer.


Fix handling repeated atoms

E.g., in the formula forall t1,t2: !(x(t1) <= x(t2)) || (y(t1) <= y(t2) && x(t1) <= x(t2)),  the atom x(t1) <= x(t2) appears twice. Now we pair instances with atoms, but we must pair them with BDD nodes which are uniqe. Since BDD nodes are paired with atoms and the atoms are the same, there will be no re-computation.
Now the code crashes on an assertion, so we do not generate wrong monitors, but we also do not support this case.

Done


Port of sparsepp added

Sparsepp is  "a fast, memory efficient hash map for C++".
It's an header-only library.

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

Awesome, thanks for the PR!


Handle missing patch file

Describe the bug
If a patch is registered in the repo, and it is smaller than the full bundle, the client will try to download the patch.
However, if the patch file is missing, whereas the full bundle is available, the client does not fall back to the full bundle
To Reproduce
Steps to reproduce the behavior:

create a new app bundle
add the bundle to the tufup repo, with patch
upload the metatdata and full app bundle to your update server, but do not upload the patch
try to update an existing client

this results in a status 400 client error due to missing patch file
Expected behavior
probably a fallback to the existing full bundle

Actually, I think this should be expected behavior, because all files in the metadata "manifest" should be available on the server.
If files are missing, it should probably be considered suspicious.


feat(connector): [Airwallex] SEPA, BACS, Sofort, EPS payment methods

Type of Change

[ ] Bugfix
[x] New feature
[x] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context
https://github.com/juspay/hyperswitch/discussions/4145
How did you test it?
Testing currently blocked by https://github.com/juspay/hyperswitch/discussions/4145#discussioncomment-8891468
Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

Closing in favor of https://github.com/juspay/hyperswitch/pull/4222 to satisfy - Convention Checks / Verify PR contains one or more linked issues


Ensure CSR and remove structural symmetry flag

Coverage increased (+0.09%) to 94.156% when pulling 9dbecec76acc1bdc4ec878bd866fe17e966027c7 on dev into dfd4de319a7287d99ed44c4a7854d7250c391eaa on master.

Coverage increased (+0.2%) to 94.318% when pulling 2f585f338b2812f754aa0107d772bb39eb6764ea on dev into dfd4de319a7287d99ed44c4a7854d7250c391eaa on master.

Coverage increased (+0.2%) to 94.318% when pulling 2f585f338b2812f754aa0107d772bb39eb6764ea on dev into dfd4de319a7287d99ed44c4a7854d7250c391eaa on master.


Test API endpoint with actual API address

Circling back: @krubenok is it working?

Yeah seems like everything it working.

Closing then.


fix: new gradle logic for downloading

improving android gradle to be like windows logic, fixing https://github.com/media-kit/media-kit/issues/645

Check your internet connection, the ci works so it's not a problem with the code, also you can rerun without flutter clean and the code will redownload the broken files


Warning about not using Composer

Need to put a note into the installation docs about Composer being a requirement to use the plugin.

Can reference this, http://book.cakephp.org/3.0/en/plugins.html#installing-a-plugin-with-composer
Also that non-composer installs are unsupported
Update readme to have Composer as a requirement

use composer require command instead of manually editing composer.json


Missing metrics on Wallbox Pulsar Plus?

Version of the custom_component
0.4.26

Configuration
Add your logs here.

Describe the bug
In the documentation Voltage and Frequency is listed as "Useful Entities for Wallbox Pulsar Plus". I have a Pulsar Plus with current firmware version 5.5.10 but I do not get any values for those metrics.
Something wrong with my charger, firmware or is the documentation not correct?
Debug log

Add your logs here.

I wrote the User Guide with the list of Useful Entities.  I'm still using v0.4.21 of this integration, but I don't know whether that makes any difference compared to the version you're using.  Please note that Voltage and Frequency only update during a charging session, not when the charger is idle.

While reading through some posts in the Discussions I have noticed that we have different models even though they are all named Pulsar Plus.
My model number is: PLP1-M-2-4
And I can not get any readings when charging.

Good point!  I have PLP1-0-2-2.  It will be interesting to see which version @lbbrhzn is using.

It seems you have a Type 2 version with 7m cable and 3-phase 22kW capability.

Here's a further thought, given that you have a 3-phase version.  Did you try looking in the Extra Attributes of the Voltage Sensor?

I do have the 3-phase, 22kW, type 2 box with 7m cable.
The sensor value is unknown, and has no attributes.

I'm running FW version 5.5.10 on my PLP1-0-2-4

@GrantKeymer, Is it possible you have the additional readings because of the external power meter you installed?

I also have the external power meter (Power Boost, Carlo Gavazzi EM340) using Modbus to communicating with the Pulsar Plus

@lbbrhzn - No, I removed the external energy meter and sent it back many weeks ago.

@Strixx76, I also have that. But I thought @GrantKeymer hade one configured differently, for Eco-Smart charging.

I don't get Frequency or Voltage.

Could it be that it is the EV that is reporting it to the charger?

Seems unlikely.  I think we can close this issue, it is not related to the integration.

Sure. Should the documentation be updated?

Sure. Should the documentation be updated?

Yes, I'll update the User Guide with a note that the Voltage and Frequency metrics don't work on 3-phase versions of the Pulsar Plus.
There are a few improvements I want to make, so this will be one of them.  Thanks guys for checking up on this issue.


enable monitoring

Enable monitoring. Have it so that either all prompts/completions are monitored, or only tasks that are specified (in particular, a draft/redraft steps).

Hey @DaveDeCaprio, it's looking like we can hit 93% on HumanEval with the prompt changes I made today. What I want to do is enable the monitor and send the unsolved challenge tasks to the monitor so it can try to solve them on a separate thread. We'd have to figure out how to do another processor instance and job description in parallel also. If we can solve 50% of the remaining unsolved challenges then we'd get #1 ranking (see https://paperswithcode.com/sota/code-generation-on-humaneval).


ok() function for handle types

Suggestion
To better harmonize with Result-oriented surrounding code, handle types like HANDLE, HBRUSH, HBITMAP, HRGN, HWND etc. should have an ok() function similar to that of BOOL to yield the value wrapped in Ok or an Err with windows::core::Error::from_win32().
It could then be used for https://github.com/microsoft/win32metadata/issues/1758, or, if this issue won't be fulfilled, it would be even more vital to be able to do this on your own without too much trouble.

Here be dragons. The contract for error reporting is a property of the API call. It cannot be implemented in a type that's used across API calls with different error reporting contracts.
Take HBITMAP as an example. The vast majority of GDI functions that return HBITMAPs do not set the calling thread's last error code. As such there's no one correct way to do the Result transformation. The few cases where there is additional error information (e.g., CreateDIBSection) already receive the appropriate treatment (CreateDIBSection).
In essence:

Error reporting contracts cannot be attributed to types.
If there is a single, loss-less Result transformation for a given API, the windows crate provides that transformation already.
For everything else the only tractable solution is to delegate error discovery and propagation to clients.

When you pointed out here that a number of functions don't make an error code available, I understood that this makes this issue shaky. But it still seems to me that basic means to write DRYer code should be provided by the windows crate. Maybe some handle returning function can never fail; but the handle type still has its is_invalid() method. Why can't it have a set of tools used under different circumstances? E.g., two functions that transform it into a Result, but only one of them calls GetLastError(), while the other one simly transforms an invalid handle into an E_HANDLE error.
A problem in this context may also be the unexpressiveness of ok(). It doesn't convey whether GetLastError() is called or just a simple check is performed and nothing more. I'm trying to fight against these imperfections of the windows crate with helper methods like these that make the code DRYer and more streamlined in Result-oriented surroundings.
From my perspective, the windows crate already accommodates the user to some extent. Many functions that in v0.48 still returned a BOOL not return a Result, and some functions like you pointed out return a Result for a fallible function where the return type isn't associated with BOOL. But it's all quite inconsistent, and I'm hoping for more accommodation and consistency.

Why can't it have a set of tools used under different circumstances?

Because the code is generated from metadata, and there isn't a whole lot of information available beyond (absence of) the SetLastError = true attribute. If you'd like to see the windows crate in a position to generate stricter code you'd have to introduce that information at the top of the pipeline (the win32metadata repo).
That is challenging, and I doubt the metadata will ever encode the full set of API contracts (or that it should, for that matter). I still believe there is room for improvement, specifically around capturing SAL annotations (apologies to @riverar, I said the "S-word" 😉 ).
Taking the example from above, CreateDIBSection (in wingdi.h) has this annotation: _Success_(return != NULL). That could conceivably be included in the metadata to drive better code generation, on a per-API basis. There are lots of SAL annotations, and I'd like to see more of them making their way into the metadata, although I understand that the win32metadata team isn't thrilled about this idea.

But it's all quite inconsistent

Such is the nature of an API that has been around for decades. The windows crate cannot fix history, and smoothing out the sharp edges should be delegated to dedicated crates.

This:

Such is the nature of an API that has been around for decades. The windows crate cannot fix history, and smoothing out the sharp edges should be delegated to dedicated crates.


Report status when stack is locked

A given stack may become stuck due to locking.  Can the operator reflect this condition to the user? Perhaps the update could have a Reason for its Failed condition, that would help the Stack controller produce a better status.
Here's an example of how the problem may occur.
 pulumi 2024-10-03T00:08:30.369Z INFO    pulumi  Updating (dev)
 pulumi 2024-10-03T00:08:30.370Z INFO    pulumi
 pulumi 2024-10-03T00:08:45.917Z WARN    pulumi  error: [500] Internal Server Error
 pulumi 2024-10-03T00:08:45.995Z ERROR   server  up completed with an error      {"error": "failed to run update: exit status 255\ncode: 255\nstdout: Updating (dev)\n\n\nstderr: error: [500] Internal Server Error\n\n"}
 pulumi 2024-10-03T00:08:45.996Z ERROR   cmd.serve.grpc  finished streaming call with code Unknown       {"grpc.start_time": "2024-10-03T00:08:28Z", "system": "grpc", "span.kind": "server", "grpc.service": "agent.AutomationService", "grpc.method": "Up", "peer.address": "<IP_ADDRESS>:44098", "error": "failed to run update: exit status 255\ncode: 255\nstdout: Updating (dev)\n\n\nstderr: error: [500] Internal Server Error\n\n", "grpc.code": "Unknown", "grpc.time_ms": 17067.732}
random-yaml-workspace-0 pulumi 2024-10-03T00:08:46.868Z INFO    server  selected a stack        {"name": "dev"}
random-yaml-workspace-0 pulumi 2024-10-03T00:08:48.256Z INFO    cmd.serve.grpc  finished unary call with code OK        {"grpc.start_time": "2024-10-03T00:08:46Z", "system": "grpc", "span.kind": "server", "grpc.service": "agent.AutomationService", "grpc.method": "SelectStack", "peer.address": "<IP_ADDRESS>:44478", "grpc.code": "OK", "grpc.time_ms": 2078.996}
 pulumi 2024-10-03T00:08:49.541Z INFO    pulumi  Updating (dev)
 pulumi 2024-10-03T00:08:49.542Z INFO    pulumi
 pulumi 2024-10-03T00:08:49.895Z WARN    pulumi  error: [409] another update is currently in progress or was interrupted. Try again later.) Try again later
 pulumi 2024-10-03T00:08:49.895Z WARN    pulumi      for more information about this error, see https://pulumi.io/oa5g1o
 pulumi 2024-10-03T00:08:49.895Z WARN    pulumi  To learn more about possible reasons and resolution, visit https://www.pulumi.com/docs/troubleshooting/#conflict

Leading to:
 pulumi 2024-10-03T00:08:53.653Z INFO    pulumi  Updating (dev)
 pulumi 2024-10-03T00:08:53.654Z INFO    pulumi
 pulumi 2024-10-03T00:08:53.960Z WARN    pulumi  error: [409] another update is currently in progress or was interrupted. Try again later.) Try again later
 pulumi 2024-10-03T00:08:53.961Z WARN    pulumi      for more information about this error, see https://pulumi.io/oa5g1o
 pulumi 2024-10-03T00:08:53.962Z WARN    pulumi  To learn more about possible reasons and resolution, visit https://www.pulumi.com/docs/troubleshooting/#conflict

Added to epic https://github.com/pulumi/pulumi-kubernetes-operator/issues/586


WIP: Appveyor for Windows tests

Have just been playing around with it a bit.
Input / help would be appreciated.

Closing for now.  Let me know if you're interested to re-work on this.

It might be useful as reference to get this going again (probably on GitHub Actions), but can be closed for now.


Fix for Zig 0.11

Not sure if this project is still maintained, and I am only just starting on my Zig journey but it no longer builds in the current stable build of Zig due to changes in the build API. I will try to make an PR when I can but I think this page summarises most of the changes:
https://devlog.hexops.com/2023/zig-0-11-breaking-build-changes/
Although, it seems like the way builds changed quite a bit, I think there would need to be a rethink from:
https://github.com/kristoff-it/bork/blob/master/build.zig#L93
inline for (releases) |ct| {

onward.

The project is maintained but all development is paused until the Zig compiler reintrouduces support for async/await. In the meantime you should grab a prebuilt executables from the releases tab.

FYI I just updated bork to latest Zig. Async isn't back in the compiler yet, but I've decided to remove it from Bork and go with OS threads instead.


test_matrix_binfile_read_write fail on various architectures

On armhf, ppc64el, ppc64, and riscv64 Debian machines, one gets the following test failure:
============================= test session starts ==============================
platform linux -- Python 3.10.6, pytest-7.1.2, pluggy-1.0.0+repack
rootdir: /<<PKGBUILDDIR>>, configfile: setup.cfg
collected 7 items

suitesparse_graphblas/tests/test_doctest.py .                            [ 14%]
suitesparse_graphblas/tests/test_exceptions.py .                         [ 28%]
suitesparse_graphblas/tests/test_io.py ..F                               [ 71%]
suitesparse_graphblas/tests/test_package.py .                            [ 85%]
suitesparse_graphblas/tests/test_scalar.py .                             [100%]

=================================== FAILURES ===================================
________________________ test_matrix_binfile_read_write ________________________

tmp_path = PosixPath('/tmp/pytest-of-buildd/pytest-0/test_matrix_binfile_read_write0')

    def test_matrix_binfile_read_write(tmp_path):
        for opener in (Path.open, gzip.open, bz2.open, lzma.open):
            for format in (lib.GxB_BY_ROW, lib.GxB_BY_COL):
                for T in grb_types:
                    for sparsity in (lib.GxB_HYPERSPARSE, lib.GxB_SPARSE, lib.GxB_BITMAP, lib.GxB_FULL):

                        A = matrix.new(T, 2, 2)

                        if T is not lib.GxB_FULL:
                            for args in zip(*_test_elements(T)):
                                f = _element_setters[T]
                                check_status(A, f(A[0], *args))
                        else:
                            Tone = _test_elements(T)[0][0]
                            check_status(
                                A[0],
                                lib.GrB_assign(
                                    A,
                                    NULL,
                                    NULL,
                                    Tone,
                                    lib.GrB_ALL,
                                    0,
                                    lib.GrB_ALL,
                                    0,
                                    NULL,
                                ),
                            )
                        matrix.set_sparsity_control(A, sparsity)
                        matrix.set_format(A, format)

                        binfilef = tmp_path / "binfilewrite_test.binfile"
                        binary.binwrite(A, binfilef, opener=opener)
                        B = binary.binread(binfilef, opener=opener)

                        assert matrix.type(A) == matrix.type(B)
                        assert matrix.nrows(A) == matrix.nrows(B)
                        assert matrix.ncols(A) == matrix.ncols(B)
>                       assert matrix.hyper_switch(A) == matrix.hyper_switch(B)
E                       AssertionError: assert 0.0625 == 0.0
E                        +  where 0.0625 = <function hyper_switch at 0x7fffa61975b0>(<cdata 'struct GB_Matrix_opaque * *' 0x1000e6b6660>)
E                        +    where <function hyper_switch at 0x7fffa61975b0> = matrix.hyper_switch
E                        +  and   0.0 = <function hyper_switch at 0x7fffa61975b0>(<cdata 'struct GB_Matrix_opaque * *' 0x1000e7118d0>)
E                        +    where <function hyper_switch at 0x7fffa61975b0> = matrix.hyper_switch

suitesparse_graphblas/tests/test_io.py:176: AssertionError
=========================== short test summary info ============================
FAILED suitesparse_graphblas/tests/test_io.py::test_matrix_binfile_read_write
========================= 1 failed, 6 passed in 0.24s ==========================

I am using version <IP_ADDRESS>, but the bug was already present with version <IP_ADDRESS>.
Tests are run after building the package from source.
Here is a full Debian build log: https://buildd.debian.org/status/fetch.php?pkg=python-suitesparse-graphblas&arch=ppc64&ver=<IP_ADDRESS>-1&stamp=1668629267&raw=0.

Thanks @ViviCoder for the info and for helping with the Debian builds.
We have recently tried adding support for other architectures, but we have a hard time testing them, so this is actually very helpful. Specifically, cffi has issues when wrapping variadic functions on different architectures. We added tried workarounds for arm64 and ppc64le in #50 and #51.
We may need to detect more platforms here:
https://github.com/GraphBLAS/python-suitesparse-graphblas/blob/95d5fdc15dc014696c36f3317ceb216fbaf04acf/suitesparse_graphblas/__init__.py#L9-L36
@ViviCoder, can you figure out what platform.machine() returns in Python for the failing builds? Can you apply patches to update the logic of which vararg function to use? Once we figure out which platforms need which vararg function, we'll be happy to update python-suitesparse-graphblas and cut a new release.

Here are the values for platform.machine():

armhf (and armel, which will probably fail once https://github.com/DrTimothyAldenDavis/GraphBLAS/issues/178 is solved): armv7l,
ppc64: ppc64,
riscv64: riscv64.

I will try to add these values to those you already have and see if tests succeed.

Here is my understanding of the problem for armv7l: as for arm64, variadic arguments must be send to the stack, but contrary to arm64, casting it to an argument of type char * does not work for double arguments, because char * is only 4 bytes.

Thanks again for your work on this @ViviCoder and the latest updates.
As a potential solution, should we try using int8_t * instead of char *?

The problem is not that char is too small, but that char * is (as a pointer, not as an array).
As far as I know, int8_t * has the same size as char *.
What we could try is to cast double variadic arguments into two separate char * arguments so that they are next to each other in the stack.

I do not know if this can be of any help, but I noticed that the failure for armhf (and armel, I assume) is slightly different: instead of AssertionError: assert 0.0625 == 0.0, one gets AssertionError: assert 0.0625 == -inf.

This problem would go away if GxB_get/set didn't use va_arg, right?
I could instead convert methods like GxB_set/get to use type-specific methods, like a new GxB_Global_Option_set_INT64 (and related types).  The user code would remain unchanged.  I could even keep the va_arg-based methods but allow them to be disabled with an #ifndef NO_VA_ARG_PLEASE ... or something.
This would be a modest change to the GxB_get/set since the user's code would typically not change.  I could even change GxB_Global_Option_set into a macro that use the _Generic keyword.
This would require a bump to GraphBLAS v8, however, since the old va_arg methods would be removed.

Yes, I guess.
It could also be worth reporting the issue to the maintainers of cffi.

This will be pretty quick to add.  See my draft v7.4.0 with the new type-specific GxB_Global_Option_set and get methods, with no va_arg.  The tests pass so far, as well:
https://github.com/DrTimothyAldenDavis/GraphBLAS/tree/v740
You can see from the first commit what differs from v7.3.3, particular the new functions in the GraphBLAS.h file:
https://github.com/DrTimothyAldenDavis/GraphBLAS/commit/989b252b85d8b473c0fc383654cf3d949e5bc89d
I still have to write the type-specific methods for the GxB_Matrix_Option_set/get, GxB_Vector_Option_set/get, and GxB_Desc_set/get, but those should be quick.  I haven't added these to the user guide yet.
You don't need these in a _Generic macro, right?  It would be easier to skip that, and leave my GxB_set and GxB_get macros the same.
I'll keep the existing methods with va_arg.  The Python cffi can just avoid calling any of those methods.

Thanks @DrTimothyAldenDavis! Correct, we don't need (and can't directly use) the macros. I like your plan.
Our automated tooling picked up the additions just fine. See: #58

See the new v7.4.0.beta1: https://github.com/DrTimothyAldenDavis/GraphBLAS/releases/tag/v7.4.0.beta1 .  I now have va_arg-free methods for all GxB set/get functions.  Test coverage is 100% so the beta1 is pretty stable.
I didn't add these to the user guide (see the Doc/Changelog), because they will only be needed until the v2.1 C API Spec comes out, and because most applications will be able to use the existing _Generic GxB_set / GxB_get.  I will keep them "forever"; I'll just declare these new va_arg-free get/set methods as "historical" when the GrB_get/set methods are available.

Thanks @DrTimothyAldenDavis! Avoiding variadic is the best solution for this IMHO.
7.4.0.beta1 passes our normal tests and is able to do our normal builds:

https://github.com/GraphBLAS/python-suitesparse-graphblas/pull/58
https://github.com/conda-forge/graphblas-feedstock/pull/58

I'll update python-graphblas once SuiteSparse:GraphBLAS 7.4.0 is released (we don't have a good workflow for beta and RC releases yet).

I just posted the stable v7.4.0. It passes all my tests
Sent from Gmail Mobile

@ViviCoder, python-suitesparse-graphblas version <IP_ADDRESS> and python-graphblas version 2022.12.0 have been released. These use the non-variadic get and set functions that Tim added in SuiteSparse:GraphBLAS 7.4.0, so I expect them to work on more architectures.
Care to give them a try when you have the time? Thanks again for working on this!

I confirm that python-suitesparse-graphblas <IP_ADDRESS> works on all architectures officially supported by Debian.
It means that #48 is also fixed.

Hooray :tada:
Closing :)


SolverError: Solver 'OSQP' failed when using max_sharpe

I am running into an error:
SolverError: Solver 'OSQP' failed. Try another solver, or solve with verbose=True for more information.
I import the dataset.
yahoo_sp500_adj_close_interim.csv.zip
df = pd.read_csv('yahoo_sp500_adj_close_interim.csv', index_col=['date'])

I am trying to run the dataset over two different risk models and then see the max sharpe ratio but when I take the max sharpe I end up with the error.
I was thinking its because the dataset is to wide because when I reduce it to 300 columns it works and 301 it breaks.
I checked the results with min_volitility and that worked with all assets. I saw this check from #86 and #82 .
When I check

from pypfopt.expected_returns import mean_historical_return
from pypfopt.risk_models import CovarianceShrinkage, sample_cov
from pypfopt.efficient_frontier import EfficientFrontier

mu = mean_historical_return(df)

S = sample_cov(df)
ef = EfficientFrontier(mu, S)
weights = ef.max_sharpe()

S = CovarianceShrinkage(df).ledoit_wolf()
ef = EfficientFrontier(mu, S)
weights = ef.max_sharpe()

S = sample_cov(df)
ef = EfficientFrontier(mu, S)
weights = ef.min_volitility()

Does anyone have an idea what this issue is?
Or point my to a resource that might help me fix the issue.

How many rows do you have in the dataset? Portfolio optimisation normally runs into issues when you don't have enough data relative to the number of assets (ill conditioned cov matrix).

I have a year of data (252) and planned on increasing once I got the optimizer up and running. I updated the data to 10 years and now I am getting a different error.
OptimizationError: Please check your objectives/constraints or use a different solver.
Below is my new large dataset used with the same covariance functions.
yahoo_sp500_adj_close_interim_2.csv.zip

@jujbates
I've tried running this locally. I can reproduce the failure for max_sharpe, but it works fine for min_volatility. I suspect the solver is having difficulty optimising for Sharpe (a comparatively difficult optimisation problem) for such a large set of assets. I was able to get it to work by using a different solver:
df = pd.read_csv("yahoo_sp500_adj_close_interim.csv", parse_dates=["date"], index_col="date")
mu = mean_historical_return(df)
S = sample_cov(df)
ef = EfficientFrontier(mu, S, solver="SCS")
weights = ef.max_sharpe()

I would add a note of caution, however. PyPortfolioOpt is designed for asset allocation rather than asset selection. In general, I would try to narrow down which stocks you are interested in first (<100) before optimising, rather than using portfolio optimisation to select assets. Just me $0.02.
Hope this helps!
Robert


Implementing User Authentication

User Story: As a user, I want to securely log in and manage my pet adoption information.
Acceptance Criteria:

GIVEN: I enter my credentials

WHEN: I submit the login form

THEN: I am authenticated and logged in.

Reopened as Issue contains crashing bugs.

Hey Josh,Cool, thanks for fixing those bugs. Also saw my new assignment, #27.  Best,Marquan WhiteOn Aug 2, 2024, at 10:45 AM, Josh @.***> wrote:
Reopened as Issue contains crashing bugs.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>


Doesn't work for transitive dependencies?

I'm writing a module (A) that requires another module (B). B has assets installable by rnpm link.
If I as an app author use module (A), running rnpm link doesn't seem to install the assets of B. Perhaps a workaround right now is to simply install B's native dependencies explicitly, but just wondering if this is intended behaviour.

Hi @Secretmapper! If you're talking about native dependencies, you're right. We don't provide this functionality yet (but we have an intention to implement it). Currently, you can make a postlink hook (see README) which will run an installation process for your dependencies.

Yeah, that's how it works for now, mainly because module B should be linked with project A and that's how it's done in most cases if I am not wrong (iOS). With Android, gradle will handle all these things for us anyway.

I think there's nothing bad in using postlink hooks for these purposes

Hello guys, thanks for the quick feedback!
What was actually weird was I tried adding rnpm link on module A, thinking that running rnpm link on the actual app would propagate the libraries to the actual app, but it doesn't seem to work (module A's xcodeproj does get added on the app, but it's dependency libraries still seem to be missing)
@Kureev If you don't mind, can you write out a quick sample in using postlinks once you have the time? I don't have native ios experience, so I'm not particularly sure how to create the hook.
How would it look like to use postlink hooks to implement this right now?

It literally one-to-one as npm scripts: https://github.com/rnpm/rnpm#commands.
Can you provide some real-world example? I'll try to sketch it.

I'm working on extracting a part of my recent project to a module, which is an epub reader for react-native.
To render the epub templates, I'm making use of https://github.com/alinz/react-native-webview-bridge and https://www.npmjs.com/package/react-native-zip-archive. I'm unsure of what to put in my module's postlink to add their dependencies.
Thanks again!

@grabbou do you have any snippet to add libz to the project?

We are going to support it thanks to @appden PR if I am not wrong?

Yeah, true. But it's not merged yet, right?

Hi Arian @Secretmapper, please get in touch with me with regards to the Salescamp project - I don't understand why you've just stopped replying to messages for 2 months even after you've received payment and signed a contract? http://arianv.com

@PC123PC I've removed your comment, please sort out any business inconveniences with @Secretmapper outside this repo, thank you :)

@Secretmapper @Kureev until my PR is merged and released, you can accomplish automatically linking libz through a postlink script. Here is the one I created for Realm.

I am planning to review it tomorrow, merge and release, so if you don't mind waiting an extra day, thats awesome!

Thanks for sharing your script, @appden! I think that supposed to solve the initial problem 'till we merge your PR in 😉


FEAT: Improve extensions quality

Description
Add documentation of extensions
Unify UI style
Issue linked
Close #5545
Checklist

[ ] I have tested my changes locally.
[ ] I have added necessary documentation or updated existing documentation.
[ ] I have followed the coding style guidelines of this project.
[ ] I have added appropriate tests (unit, integration, system).
[ ] I have reviewed my changes before submitting this pull request.
[ ] I have linked the issue or issues that are solved by the PR if any.
[ ] I have agreed with the Contributor License Agreement (CLA).

I moved it to draft because the same had to be applied to more files, and I am still fixing the other extensions.
I have also uncommented the unit tests that were failing due to PyEDB issues, and now they are passing :) so we can improve the coverage.

I moved it to draft because the same had to be applied to more files, and I am still fixing the other extensions.
I have also uncommented the unit tests that were failing due to PyEDB issues, and now they are passing :) so we can improve the coverage.

I think this problem were related to the dotnet issues that was fixed two weeks ago. I'll add the same changes in PyAEDT.


Add "param" shortcode

Looking something like this:
{{ $name := .Get 0 }}
{{ with (.Page.Param $name }}{{ . }}{{ else }}{{ errorf "param %q not found in %q" $name .Page.Path }}{{ end }}

Should there be a site params alternative too? Maybe pparam and sparam?

Should there be a site params alternative too? Maybe pparam and sparam?

I assume you know that the above snippet will fall back to Site param if not found in Page. This is the  common requirement. I don't want to add more until I see some neon ads on Time Square in New York saying "we want pparam in Hugo now!" It adds documentation work and those people who really need it can easily add it themselves.

That said, this shortcode should probably have an optional second argument with a default value if not found.

I assume you know that the above snippet will fall back to Site param if not found in Page.

Yes, I read about there somewhere in the docs. But I got misled by the error message {{ errorf "param %q not found in %q" $name .Page.Path }}. Should the error then say that that $name param was not found in both .Site and .Page params?

this shortcode should probably have an optional second argument with a default value if not found.

That will be cool!

Should the error then say that that $name param was not found in both .Site and .Page params?

This is an issue asking for a shortcode. This is not the implementation. Your "error text" was better than what I quickly wrote ...


How are updates sent out?

Looks like an awesome project. I'm digging around to see how realtime updates of subscribed records are propagated to clients and I can't find any info on that.
Is the full JSON blob sent out, or is there some sort of diffing mechanism? This could heavily influence the design of larger records that have frequent but small changes.

If a client sends a PATCH then the same PATCH will be sent to all local subscribers. https://github.com/deepstreamIO/deepstream.io/blob/master/src/record/record-handler.js#L312

Thanks!


fix: use copy into instead of attachment

fix https://github.com/databendcloud/databend-jdbc/issues/38


Signing raw transaction - scriptPubKey not supported

I've been having a problem when trying to sign a transaction, can you give me some insight?
The raw transaction has been created successfully using this library, and here's the hex:
01000000017d2adc7945951368549858b0752571198de1ec1a6e923b9d9725ac7d3d197a6c0000000000ffffffff0290010000000000001976a9147c71e6ee867c0f5ceff3374fd99d5c665af1809988acf4010000000000001976a914bae6a34409b768704eccba0fc5344248eb16c34c88ac00000000
However, I always get this scriptPubKey not supported error whenever I'm trying to sign it.
Here's the code:
$privateKey = $privFactory->fromWif('cNsCfR9TadEJkXeoKMC4fv9g7KfmoHZ4SET6c63cTXY1UwSmecKv');
$address = new PayToPubKeyHashAddress($privateKey->getPublicKey()->getPubKeyHash());
$addressPubKey = $privateKey->getPublicKey();

$amount = 500; // satoshis
$change = 500; // satoshis
$fee = 100; // satoshis
$amountAfterFee = $amount - $fee;

$transaction = TransactionFactory::build()
    ->input(
        '6c7a193d7dac25979d3b926e1aece18d19712575b05898546813954579dc2a7d', // prev unspent tx id
        0 // vout
    )
    ->payToAddress(
        $amountAfterFee, // the amount to send
        $addrCreator->fromString("mrrxXtvaPrksaVkr3oxjyMgRNSUHUaoZY1")) // target address
    ->payToAddress(
        $change, // the amount to send
        $addrCreator->fromString("mxZCGj76gyznyHCsdTKBQApDaSHbWNAqqJ")) // change address
    ->get();

$redeemScript = ScriptFactory::scriptPubKey()->p2wkh($privateKey->getPubKeyHash());
$p2shScript = new P2shScript($redeemScript);
$signData = new SignData();
$signData->p2sh($redeemScript);

$signer = new Signer($transaction, Bitcoin::getEcAdapter());

// foreach ($transaction->getInputs() as $index=>$input) {
//     $signer->sign($index, $privateKey, new TransactionOutput(0, ScriptFactory::fromHex($input->getScript()->getBuffer()->getHex())));
// }

// foreach ($transaction->getInputs() as $index=>$input) {
//     $signer->sign($index, $privateKey, $input->getScript());
// }

$transactionOutputs = [];
foreach ($transaction->getInputs() as $idx => $input) {
    $transactionOutput = new TransactionOutput(0, ScriptFactory::fromHex($input->getScript()->getBuffer()->getHex()));
    array_push($transactionOutputs, $transactionOutput);
}
$signer = new Signer($transaction, Bitcoin::getEcAdapter());
foreach ($transactionOutputs as $idx => $transactionOutput) {
    $signer->sign($idx, $privateKey, $transactionOutput, $signData);
}

// $signer = new Signer($spendTx, Bitcoin::getEcAdapter());
// $signer->sign(0, $privateKey, $transaction->getOutput($spendOutput));

$signedTxHex = $signer->get();

$transactionOutput = new TransactionOutput(0, ScriptFactory::fromHex($input->getScript()->getBuffer()->getHex()));

Inputs don't contain the script of the output, they contain a different script which provides a solution to the script of the previous output. You're passing $input->getScript() instead of $p2shScript

Got it to work, thanks for the help @afk11!
I tried using the $p2shScript and it's always rejected when I tried pushing the hex. However, instead of just using the $p2shScript I tried putting the input script from the unspent list instead.
->input(
    'd81527287fb23c174e3f75ff6e3fc0f684e68a608c1b7c22d6aa0419c1135f72', // prev unspent tx id
    0, // vout
    new Script(Buffer::hex('76a914bae6a34409b768704eccba0fc5344248eb16c34c88ac')) // scriptPubKey
)

That's great news - best of luck


Add PasswordInput component

Description
Adds a PasswordHider component input group that can be appended on the end of password inputs
Closes #1077
Screenshots

Checklist

[x] Are your changes backwards compatible?

[x] Did you QA this? Could we deploy this straight to production?

Only added on the NWC input are there other Password inputs in sn?

[x] For frontend changes: Tested on mobile?

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Enhanced user experience by adding password visibility toggle feature in settings, allowing users to switch between hiding and showing password inputs.
Introduced a new PasswordInput component for handling password inputs in the LNBits settings page.
Improved security and user interaction by replacing ClientInput with PasswordInput in the NWC settings.

I just noticed that the UI/UX is reversed for comments. When the comment is uncollapsed, we show the closed eye. When it's collapsed, we show the collapsed eye.
So I think we should make it consistent here. A classic case of this.

Also true of dark/light mode. It's weird having eyes on all the comments though.


Add FunC to the list of supported languages

Refs https://github.com/highlightjs/highlight.js/issues/3629
Checklist

[ ] Added markup tests, or they don't apply here because...
[x] Updated the changelog at CHANGES.md

@sobolevn Thanks so much!


Fix flaky test on github ci

Folks, just FYI ... this patch made the FHIR tests flaky again. It deleted the assumptions that prevented the code from running on slow systems (such as the ASF CI ones).

Folks, just FYI ... this patch made the FHIR tests flaky again. It deleted the assumptions that prevented the code from running on slow systems (such as the ASF CI ones).

Damn, I am truly sorry, I did not know that fhir code was needed for slow systems, locally it was working with and without that code. Sorry!

Folks, just FYI ... this patch made the FHIR tests flaky again. It deleted the assumptions that prevented the code from running on slow systems (such as the ASF CI ones).

Damn, I am truly sorry, I did not know that fhir code was needed for slow systems, locally it was working with and without that code. Sorry!

Hey, no worries. We can work a fix next week.


feat(meter): add meter budget&event module

这个 PR 实现什么功能/修复什么问题:
添加meter budget&event module
是否需要 backport 到之前的 release 分支:

release/3.7

/area meter
/cc @zexi @yousong

Codecov Report

Merging #9675 (c3f31fc) into master (2b47be4) will increase coverage by 9.93%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9675       +/-   ##
===========================================
+ Coverage    5.99%   15.93%    +9.93%
===========================================
  Files         844      124      -720
  Lines      131395    12773   -118622
===========================================
- Hits         7879     2035     -5844
+ Misses     122815    10547   -112268
+ Partials      701      191      -510

Flag
Coverage Δ

aFlag
15.93% <ø> (+9.93%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/monitor/alerting/engine.go

pkg/apigateway/handler/request.go

pkg/mcclient/options/k8s/repo.go

pkg/compute/models/guestdisks.go

pkg/multicloud/aws/iam_policy.go

pkg/cloudcommon/policy/resources.go

pkg/util/ansible/playbook.go

pkg/util/qemuimg/init.go

pkg/util/sysutils/privilege.go

pkg/compute/regiondrivers/esxi.go

... and 710 more

Continue to review full report at Codecov.

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

/lgtm
/approve


People Tab Navigation

Closes #77
Added a People tab between MyGroups and Profile. People tab has 3 sub tabs Friends, Following, Followers that will eventually be populated once user interaction implemented

Other Minor Changes:

removed the "s" from "Profiles" tab since it didn't make sense
changed some icons around to distinguish each tab from each other
removed headers from main tabs and added toppadding to compensate

LGTM on android
for IOS the people tab works properly but the padding at the top is too small


Update CONTRIBUTING.md link

Fixes #83096

@cmcclister
I see you've made recent updates to this article. Would you be the correct person to review and approve this link change? (GitHub didn't assign an author.)
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"


Node.js -> io.js

Now, electron is using io.js :smile: .

Fair point! :smile_cat: But as things are a bit in flux and as of yesterday io.js is renaming to nodejs and moving to the Node Foundation I think sticking with 'node.js' is the best bet right now.

:+1:  yes, I saw this after sent my pr. Anyway, that's awesome!


Was SharePoint integration example deleted?

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [X] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Try to navigate to the URL https://github.com/Azure-Samples/blockchain/tree/master/blockchain-development-kit/accelerators/attestable-documents-and-media/blockchain-workbench/sharepoint

Any log messages given by the failure

HTTP 404 Error

Expected/desired behavior

Access to the sample provided on https://msdn.microsoft.com/magazine/mt833290

OS and Version?

Windows 10.0.17134.523

Versions

Mention any other details that might be useful

Hi, guys! Please, restore the example presented in this amazing MSDN Magazine article: https://msdn.microsoft.com/magazine/mt833290. Thanks in advance!

Thanks! We'll be in touch soon.

It seems to be moved. I've found it in another path: https://github.com/Azure-Samples/blockchain-devkit/tree/master/accelerators/attestable-documents-and-media/blockchain-workbench/SharePoint
Please, close this issue. Thanks!

Yes, we moved the devkit to it's own repo


Incorporate changes from Live Work Locate application

Incorporate changes from Live Work Locate (https://github.com/Esri/live-work-locate/commit/e581eb33f38959d367de6652e1b8004ac2699b7f) into the Site Selector application.  Summarized below.

@MikeTschudi  @NikkiGolding
Can either of you take a look at this to confirm the change was made?

I see obsolete-search-string handling and special-key handling in the code.  The special-key handling is an improvement over the live-work-locate version because it is using the AMD approach for the 'keys' constants; we should use this new work as our standard for other apps.
The differences in the sharing code between live-work-locate and site-selector are too great to tell if the site-selector version solves the same problem; I can only recommend the thorough testing that brand-new code would get.

Thanks @MikeTschudi

@lindsayking We need to ensure the following work for this scenario
Test Application using place names with Special Characters
Do critical search testing review (ensure we didn't lose functionality)
Do critical sharing testing review (ensure we didn't lose functionality)

Search for "D' " breaks in current and new version of apps
Can't test sharing with special characters because they aren't searchable


Update README.md

I made some changes to your resume. I suggested that you make some changes with work experience