[Bug] Editable Columns - Bulk Function Buttons not activating after editing

Bug report
What I did
Edited a row and then selected the row (checkbox) for a bulk function to be called.
What I expected to happen
Bulk Buttons should become active upon selection of at least one row (through checkbox)
What happened
Bulk Buttons do not become active.
What I've already tried to fix it
This can be solved by reloading the page, which is not user friendly.
Probably a matter of reapplying the trigger after editing.
Is it a bug in the latest version of Backpack?
No, it already happened in prior versions. Just waited with the report to see if it was fixed with the new release.
After I run composer update backpack/crud the bug... is it still there?
Yes
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.1.17 (cli) (built: Mar 16 2023 14:38:37) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.17, Copyright (c) Zend Technologies
with Zend OPcache v8.1.17, Copyright (c), by Zend Technologies
with Xdebug v3.2.0, Copyright (c) 2002-2022, by Derick Rethans
LARAVEL VERSION:
<IP_ADDRESS>
BACKPACK PACKAGE VERSIONS:
backpack/basset: 1.0.0
backpack/crud: 6.0.0
backpack/devtools: 2.0.2
backpack/editable-columns: 3.0.0
backpack/generators: v4.0.0
backpack/permissionmanager: 7.0.0
backpack/pro: 2.0.4
backpack/theme-coreuiv2: 1.1.0
backpack/theme-tabler: 1.0.1

Thanks for the report @srd2010
I can confirm the bug is present, and I already submitted a PR to fix it, should be merged and tagged soon 🙏
Sorry for the bad experience 😞
Cheers

OK TESTED. PR ready to merge
https://github.com/Laravel-Backpack/editable-columns/pull/83

Fixed, merged and tagged.
Thanks @srd2010 🙏 you should be able to get the fix with a composer update.
Cheers


[spinel] allow registering callback to handle compatibility errors

When the RCP capabilities check fails, a mechanism should be provided to notify the user of the error instead of causing the program to crash.
For example, the hardware reboot function provided in the Spinel driver interface could be invoked. The specific hardware reboot implementation is left to the user, such as performing an RCP update.

This change is intended to address compatibility issues between older and newer versions: the older version of the RCP may have fewer capabilities than the newer version. If a new version of the Host uses an older version of the RCP, the initialization may fail during the capabilities check due to a mismatch in the number of capabilities. In such cases, users should be allowed to handle this error instead of the program crashing.

@Irving-cl
I found some spelling errors and fixed them. Could please take a look? Thanks.

@Irving-cl @jwhui
Are there any other issues with this PR? It seems that the CI cases haven't fully passed.


[Improvement][SPI] support load single plugin

related issue #5793
We cloud load plugins from registry.plugin.dir,
If we set registry.plugin.dir=./dolphinscheduler-dist/target/dolphinscheduler-dist-1.3.6-SNAPSHOT/lib/plugin/registry,
it can work well.
But in this path, there may exist multiple plugins, we may only want to load one registry plugin, such as zookeeper, so we want to set registry.plugin.dir=./dolphinscheduler-dist/target/dolphinscheduler-dist-1.3.6-SNAPSHOT/lib/plugin/registry/zookeeper, the program will throw an exception.
This is caused by line 83https://github.com/apache/dolphinscheduler/blob/16986c3c651af38469c6d4cb03a587fd174c9a9b/dolphinscheduler-spi/src/main/java/org/apache/dolphinscheduler/spi/plugin/DolphinPluginLoader.java#L81-L87
This pr fix this problem, now we can set
registry.plugin.dir=./dolphinscheduler-dist/target/dolphinscheduler-dist-1.3.6-SNAPSHOT/lib/plugin/registry
to load all registry plugins(maybe for alert plugins)
or set
registry.plugin.dir=./dolphinscheduler-dist/target/dolphinscheduler-dist-1.3.6-SNAPSHOT/lib/plugin/registry/zookeeper
to just load zookeeper plugin

Codecov Report

Merging #5794 (662d7e9) into dev (16986c3) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #5794      +/-   ##
============================================
- Coverage     45.42%   45.38%   -0.05%
+ Complexity     3694     3691       -3
============================================
  Files           607      607
  Lines         24872    24881       +9
  Branches       2834     2836       +2
============================================
- Hits          11299    11291       -8
- Misses        12484    12501      +17
  Partials       1089     1089

Impacted Files
Coverage Δ

...lphinscheduler/spi/plugin/DolphinPluginLoader.java
0.00% <0.00%> (ø)

...he/dolphinscheduler/common/enums/SqoopJobType.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (ø)

...eduler/server/worker/runner/TaskExecuteThread.java
59.50% <0.00%> (ø)

...dolphinscheduler/remote/future/ResponseFuture.java
83.05% <0.00%> (+1.69%)
:arrow_up:

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 16986c3...662d7e9. Read the comment docs.


asus/battery.nix: fix chargeUpto after suspend/resume, make script optional

Fixes chargeUpto setting getting reset after suspend/resume cycle by using a systemd service which runs after suspend instead of tmpfiles.
Fixes #373

Anything else I need to do before this can be merged?


Add code samples for Service Directory

Service Directory is a new product, client libraries were just created and we should add some snippets.

Feel free to assign to me - I have a PR for this addition already

@makk94 thanks for the offer!
@chingor13 is there a guide that we can share for contributing new samples to the split repos?


Feature/blueprint abstraction

Coverage decreased (-0.5%) to 99.49% when pulling 878a917088bd1b2f1619a95009994a80b1cb2812 on feature/blueprint-abstraction into 82d415355929ab47e6ad5d00dca26dfd72a33072 on master.


Unknown import symbol only when using "stubs"

When making a copy of a file in order to create a "stub" file, the imported symbols that worked ok in regultar .py files raises a "Unknown import symbol" error after create the .pyi file.
Environment data

Language Server version: 2023.12.1
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Python 3.11.5 Anaconda

Repro Steps

Create tree files: foo.py, bar.py and spam.py in the same folder.
Contents of foo.py:
class Test:
    ...

Contents of bar.py:
from foo import Test

Contents of spam.py:
from bar import Test

Check that there are no problems with the imported symbols in all files.

Make a copy of bar.py and name it bar.pyi (i.e. create a stub file)

Now the line inside spam.py ("from bar import Test") shows the following error:
"Test" is unknown import symbol

Expected behavior
I expected the same behaviour in both cases of using bar.py or bar.pyi when the 2 files have the same content, which are type definitions.
Please, clarify if it is not correctly expected.
Actual behavior
There is an error raised after create the stub file.

The behavior you've described is correct, not a bug.
A stub file is meant to be a "stand-in" for a module when that module is untyped or written in a language other than Python. When you import a module, a type checker will attempt to resolve the import first using a type stub if it is present. In your example, you've created a type stub named "bar.pyi", but you haven't populated it with any type information. That means any file that imports from "bar" will see no symbols exported from that module.
If you want to create a proper type stub for "bar", you should provide type information within the stub file. For example, if "bar.py" is intended to export the symbol Test, which it in turn imports from the "foo" module, you should add the following line to bar.pyi:
from foo import Test as Test

Note the redundant import form. This is required by the Python typing standard if the intent is for the type stub to re-export a symbol imported form another module. By default, symbols imported by a type stub are considered private to that stub and are not re-exported.
The behavior of type stubs is documented in the official Python typing spec. Pyright (the type checker upon which pylance is built) follows this standard.

@erictraut thank you for the clarification.
So... to manually create stub files for existent modules, I need also to resolve the imports, not just add type annotations.


question about infer_pipeline

https://github.com/Francis-Rings/StableAnimator/blob/main/animation/pipelines/inference_pipeline_animation.py#L592
I notice that every tile_size 's first frame is frame[0], so I would like to trouble you to ask why it is set like this. Thanks, looking forward to your reply~ @Francis-Rings

Hi, please refer to the Progressive latent fusion for long video generation section in https://arxiv.org/pdf/2406.19680.

okok, thanks a lot~


RUMM-3361: Move SpanEventMapper to public package, move firstPartyHostResolver to InternalSdkCore

What does this PR do?
This PR moves SpanEventMapper from internal package to the public one (because it is used in the public API) and also moves firstPartyHostResolver property from SdkCore to the InternalSdkCore (it is needed only for internal functionality and unlikely is useful for the customers).
NOTE: Cast to InternalSdkCore is done in non-safe way, because it is guaranteed that SDK instance always implements it.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1496 (6f400de) into feature/sdkv2 (f6b6ad2) will decrease coverage by 0.16%.
The diff coverage is 40.00%.

@@                Coverage Diff                @@
##           feature/sdkv2    #1496      +/-   ##
=================================================
- Coverage          83.25%   83.10%   -0.16%
=================================================
  Files                423      423
  Lines              14522    14552      +30
  Branches            2180     2180
=================================================
+ Hits               12090    12092       +2
- Misses              1847     1867      +20
- Partials             585      593       +8

Impacted Files
Coverage Δ

.../android/core/internal/data/upload/UploadWorker.kt
86.49% <0.00%> (-5.41%)
:arrow_down:

.../internal/system/DefaultBuildSdkVersionProvider.kt
100.00% <ø> (ø)

...n/com/datadog/android/trace/TracesConfiguration.kt
100.00% <ø> (ø)

...m/datadog/android/trace/internal/TracingFeature.kt
91.30% <ø> (ø)

...ce/internal/domain/event/SpanEventMapperWrapper.kt
100.00% <ø> (ø)

...n/com/datadog/android/okhttp/DatadogInterceptor.kt
69.29% <ø> (ø)

...com/datadog/android/v2/core/NoOpInternalSdkCore.kt
18.60% <18.60%> (ø)

...ore/src/main/kotlin/com/datadog/android/Datadog.kt
81.97% <100.00%> (ø)

...dog/android/ndk/internal/NdkCrashReportsFeature.kt
66.00% <100.00%> (+4.64%)
:arrow_up:

...datadog/android/okhttp/trace/TracingInterceptor.kt
81.45% <100.00%> (+0.48%)
:arrow_up:

... and 16 files with indirect coverage changes


Wrong number of knots in derivative

Hi,
I noticed some overflow due to a wrong number of knots in building the derivative:

in https://github.com/retuxx/tinyspline/blob/master/library/tinyspline.c#L696 it should be to_ctrlp = malloc( ((nc-1)*dim + nk+2) * sof_f );
in https://github.com/retuxx/tinyspline/blob/master/library/tinyspline.c#L724 it should be derivative->n_knots = nk-2;

Indeed, but I think it should be: to_ctrlp = malloc( ((nc-1)*dim + nk-2) * sof_f ); rather than to_ctrlp = malloc( ((nc-1)*dim + nk+2) * sof_f );, otherwise setting derivative->n_knots = nk-2; makes no sense to me.

Oh yeah, nk-2. Typo...


DNR Encryption

Barebones encryption module - OpenSSL only.
This should satisy those that want "something" built in, while not offending those who don't want "anything" built-in.
libsodium support can be added once it gets bundled with our minimum version, and indeed the module is architected to make it easy to add any other encryption handlers.

You're crushing it tonight @jim-parry, keep it up :)

@lonnieezell No apologies needed - appreciate the feedback.
Changes made, though my local build is having issues with a bunch of non-encryption stuff :-/

I don't know what has gone wrong with the travis build, but definitely something serious :(
I will recreate this branch/PR and see if that helps.

Found & fixed the root problem with my travis build - messing with Services:logger.
Addressing unit testing & code coverage before updting this PR.

@lonnieezell Looking better - simplified, tested & hopefully addressing the concerns you raised :)

@jim-parry Looks good to me. Merge at your discretion.


Question about the history command

When I type the command history I get
Black Screen: command "history" not found.

Is there any way to have any insight into the command history or it that a new feature that needs to be created yet?
On a sidenote, there is no response from CTRL+r (which would normally trigger (reverse-i-search)': as this is related to history it seemed worth mentioning, for the sake of competion.

History and history search are to be implemented.

any update? has been history command is implemented?


Add Magnus badges

Add the builder/architect badges from the recent Magnus event

Only question I have is whether to add support for specifying the badge level by name, i.e. I have the [architect|builder] magnus badge or some variation thereof.

Please reopen this issue/pull-request over at therealklanni/hubot-ingress.

Will try to do later today, thanks again


Error when using root vhost

Ex:
queues:
    queue-with-root-vhost@/:
        durable: true
        auto_delete: false

Yields:
11:37:47.420 [main] DEBUG c.m.p.RabbitPuppy - Ensuring queue queue-with-root-vhost exists at vhost / with configuration QueueData{durable=true, auto_delete=false, arguments={}}
11:37:47.426 [main] INFO  c.m.p.RabbitPuppy - Creating queue queue-with-root-vhost at vhost / with configuration QueueData{durable=true, auto_delete=false, arguments={}}
11:37:47.454 [main] ERROR c.m.p.r.RestUtils - Response with HTTP status 405 Method Not Allowed, expected status code 204
11:37:47.455 [main] ERROR c.m.p.RabbitPuppy - Failed to create exchange 'queue-with-root-vhost@/': Response with HTTP status 405 Method Not Allowed, expected status code 204

Note that the last error log says "exchange" (when it was queue creation that failed) as well.

This seems to be a problem with url encoding/decoding Unirest. Will look for another http library that can handle this request.


New version: goplus.gop 1.2.2

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

No errors to post.

(Automated response - build 792.)

@wingetbot waivers Add Validation-Executable-Error

What's the problem?

Hi @cpunion,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 796.)

@wingetbot run


This extension is no longer deployable from the Azure Compute

Specific to the installation steps.
This extension no longer exists for deployment from Azure Compute. It was replaced with Insights Performance Diagnostics. Which may install that same extension under the covers but the steps to use it are completely different.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b8562769-f387-9577-a1ce-7b003b054339
Version Independent ID: 0ccdec3b-d6c3-1385-f77c-a7d898dd2e5e
Content: Azure Performance Diagnostics VM Extension for Windows - Virtual Machines
Content Source: support/azure/virtual-machines/performance-diagnostics-vm-extension.md
Service: virtual-machines
GitHub Login: @genlin
Microsoft Alias: genli

@wsmelton Thanks for the feedback. I will update the article to reflect the new extension.

@wsmelton You can still install the Azure Performance Diagnostics extension via the Extensions + applications blade as described in the article.

Why close when the doc is still wrong?

This does not exists.

@wsmelton  Thanks for pointing out the problem. I will update the article.

@wsmelton  The article is updated:
https://docs.microsoft.com/en-us/troubleshoot/azure/virtual-machines/performance-diagnostics-vm-extension#install-the-extension
Thanks for helping to improve the document.


Make reference to PR in changelog clickable

Core Library
MSAL.js (@azure/msal-browser)
Wrapper Library
MSAL Angular (@azure/msal-angular)
Public or Confidential Client?
Public
Documentation Location
https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/dev/lib/msal-angular/CHANGELOG.md
Description
It'd be nice to make the PR references in changelogs clickable. This will make accessing PR changes pretty easy.
Here's a screenshot of how it looks today in Angular's Changelog

Note: PR references in the "Releases" page seem to work fine: https://github.com/AzureAD/microsoft-authentication-library-for-js/releases

Thanks for raising this, I agree this would be nice to have. I've marked this as a feature and will update here when we have progress to share.


Don't attempt to define constant if it already exists.

Using the 2.x pecl extension, sodium_compat defines already existing constants in the autoload process.

This patch applies the same check to constants as already done for functions prior to defining them.

I cancelled the build on the first commit so the second can proceed faster.

Cheers :D

Thanks! 👍

No problem 😄


Refactor static_buffer array return values

Use a lambda or utility function to return the arrays since the braced-initialization doesn't work right on all compilers. See #739

Done


[Analytics] Send event when we're opened from deeplink

Goal:
When Eigen launches by way of a deeplink, we should send an event preferably noting which campaign/source referred to the app. (like UTM parameters on web)

@sarahscott you're working on the WFY view that can be triggered from email. Could you have a look at whether we can do this?
cc/ @anipetrov

Closing in favour of https://github.com/artsy/collector-experience/issues/239


should be able to run snakemake with --use-singularity and  --kubernetes without  pulling image locally first

Is your feature request related to a problem? Please describe.
I'd like to be able to run snakemake using kubernetes, and also use singularity in some of the jobs. Currently, the way I understand it, the container you'd like to use gets pulled locally first.  So in a kubernetes situation, this means that first the image would be pulled  locally, and then on the node that same image would also be pulled.
I'd like to avoid pulling the image locally first if I'm using kubernetes (I'm triggering this from a node in which singularity is hard to install).
I'm actually not sure but this might be a bug, since at the moment when run my workflow I get:
Traceback (most recent call last):
  File "/home/admin/.venv/lib/python3.8/site-packages/snakemake/__init__.py", line 722, in snakemake
    success = workflow.execute(
  File "/home/admin//.venv/lib/python3.8/site-packages/snakemake/workflow.py", line 1110, in execute
    success = self.scheduler.schedule()
  File "/home/admin/.venv/lib/python3.8/site-packages/snakemake/scheduler.py", line 510, in schedule
    self.run(runjobs)
  File "/home/admin/.venv/lib/python3.8/site-packages/snakemake/scheduler.py", line 558, in run
    executor.run_jobs(
  File "/home/admin//.venv/lib/python3.8/site-packages/snakemake/executors/__init__.py", line 120, in run_jobs
    self.run(
  File "/home/admin/.venv/lib/python3.8/site-packages/snakemake/executors/__init__.py", line 1897, in run
    if job.needs_singularity and self.workflow.use_singularity:
  File "/home/admin//.venv/lib/python3.8/site-packages/snakemake/jobs.py", line 383, in needs_singularity
    return self.container_img is not None
  File "/home/admin//.venv/lib/python3.8/site-packages/snakemake/jobs.py", line 396, in container_img
    return self.dag.container_imgs[self.container_img_url]
KeyError: 'docker://ubuntu'

There is a dictionary self.dag.container_imgs that is empty when running in kubernetes.
Describe the solution you'd like
I'd like the ability to say that if you are running your workflow on kubernetes, and using singularity, then you might not want to pull the image locally first.
I hacked around a little bit, and it works if you modify jobs.py a little bit, to eg look like:
    @property
    def container_img(self):
        if self.dag.workflow.use_singularity and self.container_img_url:
            try:
                img = self.dag.container_imgs[self.container_img_url]
            except KeyError:
                logger.warning("couldn't find the container in the dict")
                img = self.container_img_url
                logger.warning(f"Using {img} instead")
            return img
        return None

but I realise this isn't ideal.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
This is my example workflow:
rule all:
    input:
        "out/b.txt"

rule a:
    output:
        a = "out/a.txt"
    container:
        "docker://ubuntu"
    shell:
        """
        touch {output.a}
        """

rule b:
    input:
        "out/a.txt"
    output:
        "out/b.txt"
    container:
        "docker://jtcressy/cowsay"
    shell:
        """
        /usr/bin/cowsay "both containers ran ok" > {output}
        """

And I run it like:
export REMOTE=S3
export PREFIX=my-bucket

snakemake -j 1 --kubernetes  \
               --default-remote-provider $REMOTE \
               --default-remote-prefix $PREFIX \
               --use-singularity \
               --snakefile ./snakefile \
               --envvar AWS_ACCESS_KEY_ID AWS_SECRET_ACCESS_KEY

I'm using version 7.3.8.

I have this issue too!

@szsctt  I  have been using the modified jobs.py file for while, seems to work ok for me. But it would be good if someone could confirm whether this is a bug or just a lack of my understanding.

Avoiding local pulls would also be useful for the TES backend.


Does this project support AWS Site to Site (Direct Connect)?

Hi,
Wondering if this project supports setting up a K8S cluster on AWS with Direct Connect (site to site VPN)? If so, is there anything special to note about this configuration that needs to be adapted in the "terraform.tfvars" file?
Thanks!

Hi @miaeyg, viya4-iac-aws focuses on provisioning the AWS cloud infrastructure resources that are required to deploy SAS Viya platform product offerings. Use of AWS Direct Connect (site to site VPN) to provide access to AWS resources is not within the scope of this project.

Hi @dhoucgitter
Wait, before you close this issue - I think you did not understand my question... my question is about a pre-existing AWS instance that is configured with Site to Site VPN and I want to setup a Viya cluster on it. I am not expecting the project to configure the Site to Site VPN but asking if any of the project's config variables require special attention in this case.

Hi @miaeyg, are you trying to run viya4-iac-aws from a pre-existing AWS instance in order to create your EKS cluster? If you are an admin for that AWS instance you should be able to install any requirements on that instance and run IaC from there just as you would do from an AWS instance that you SSH'd into from your development network. Is the pre-existing AWS instance part of a bring your own network scenario deployment that you have planned? If so, take a look at https://github.com/sassoftware/viya4-iac-aws/blob/main/docs/user/BYOnetwork.md to understand the configuration requirements depending on your scenario.

Hi @dhoucgitter
At https://github.com/sassoftware/viya4-iac-aws/blob/faf3f0ebcc46cb4287fac70caac6733033d2e8b5/docs/CONFIG-VARS.md#admin-access it says:
NOTE: When deploying infrastructure into a private network (e.g. a VPN), with no public endpoints, the options documented in this block are not applicable.
Q1: Does "private network" include AWS Site to Site VPN?
Q2: What the meaning of "not applicable" does it mean that the options need to be kept empty or deleted or can be with a value and they will be ignored? Also what options specifically are "not applicable" - it is not mentioned.

Hi @miaeyg, added my responses below.

Q1: Does "private network" include AWS Site to Site VPN?
Q2: What the meaning of "not applicable" does it mean that the options need to be kept empty or deleted or can be with a value and they will be ignored? Also what options specifically are "not applicable" - it is not mentioned.

R1: If you are using a VPN to access EC2 instances in your AWS VPC, it's likely that you are accessing a "private network" or one that does not have a public IP address.
R2: When deploying infrastructure into a private network with no public endpoints, the four *_public_access_cidrs configuration variables intended to convey the public IP CIDRs that should be allowed access to your cluster's resources are not so relevant. "Not applicable" in this case means that you could leave all four *_public_access_cidrs variables undefined or set default_public_access_cidrs to [], the empty list, and instead use the *_private_access_cidrs configuration variables to convey which EC2 CIDR ranges you wish to provide access to your cluster resources from. If you created a bastion or IaC client VM in your VPC that you are accessing via your site to site VPN, you could set default_private_access_cidrs to the CIDR range for that VMs private IP address as a starting point. For example, if your IaC client VMs private address is <IP_ADDRESS>, you could set default_private_access_cidrs = ["<IP_ADDRESS>/32"]. If you plan on accessing cluster resources or the application ingress from other private IP addresses in your VPC, you could specify a broader CIDR range that includes them or add additional CIDR ranges to the list for the *_private_access_cidrs variables that you are using.

Hi @dhoucgitter
So if I remove the four *_public_access_cidrs configuration variables and also use an internal PostgreSQL server then IaC will not auto-create any public subnets?

Hi @miaeyg, setting or not setting any of the four *_public_access_cidrs variables and using an internal PostresSQL server will not affect whether IaC will create public subnets, or the other subnets IaC creates for you. Unless you pre-create the required subnets and then supply the subnet ids for those existing subnets in the subnet_ids IaC configuration variable, IaC will create the required subnets for you. Read through the Subnet Requirements section which should help explain which subnets are created if you are not supplying your own via the subnet_ids variable. Providing your own subnet ids qualifies as a BYO networking scenario. Pay particular attention to the Required Variables, Optional Variables and Additional Requirements columns in the table should you choose to go with a BYO networking scenario.

Hi @dhoucgitter, one last question about this...is it OK to set (different) values for both default_private_access_cidrs and default_public_access_cidrs in the same terraform.tfvars file?

Hi @miaeyg, you should be able to set different values for those variables but I'd be curious about why you are trying to both of them. default_private_access_cidrs would be used if you are deploying infrastructure into a private network with no public endpoints. See Admin Access  for guidance. If you are configuring public endpoints for your cluster, you could set default_public_access_cidrs to enable access from your chosen set of CIDR ranges instead.


Why not add an scripting language and design language?

Why not add an scripting language like javascript and a styling language like css?
so you feel like an web application but its an .net assembly.

We currently have a styling system somewhat like CSS:
https://github.com/Perspex/Perspex/blob/master/docs/styles.md
As for a scripting language, I think that is out of scope for the moment, we can consider stuff like that after a 1.0.


Custom css in header/footer

Hey,
Thanks for the great lib!
I'm trying to use a custom font in the header/footer. I tried doing it with puppeteer on it's own, and I managed to do it by having a style tag with an inlined font (base64 representation).
I'd like to do the same here - add some css to header/footer only, but I didn't find a way to do it yet. Would you accept a PR that introduces it? I'd be happy to contribute if this makes sense.

Hi, Thank you for using it :)
I think it quite possible already. Just load the font and apply it to the header and footer id with CSS. what is the problem? that do you have in mind?

Thanks for the quick reply. I think I have found the issue on my side:
Version that works:
<html>
  <head>
    <style>/* some styles here, including footer */</style>
  </head>
  <body>
  </body>
</html>

Version that does not work:
<html>
  <head>
  </head>
  <body>
    <style>/* some styles here, including footer */</style>
  </body>
</html>

All other styles would be applied, except for the special classes like #footer.
Using style outside of head seems to be considered a bad practice for web pages (not sure if that matters much in this context), so I'll just fix it on my side. It wasn't very obvious to find though. Not sure if the lib can do anything to fix this. Maybe a readme update would be good enough? What do you think?

Right. Thank you for pointing that out.
Sure. I think adding an example in the examples folder and mention this matter in the readme will help a lot.
I'm open to any PR. Also, I can do it in the next week.


Error: Cannot find module '../utils/colorManipulator'

↓ Such an error occurred.
Error: Cannot find module '../utils/colorManipulator'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/takata/GitHub/sample/.next/dist/node_modules/material-ui/styles/getMuiTheme.js:17:25)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)

I tried to use material-ui(0.18.0), It did not work.
material-ui is in .next/dist/node_modules/ , but material-ui/utils/colorManipulator.js is not in .next/dist/node_modules/
Is this due to the relative path?
Object.defineProperty(exports, "__esModule", {
  value: true
});
exports.convertColorToString = convertColorToString;
exports.convertHexToRGB = convertHexToRGB;
exports.decomposeColor = decomposeColor;
exports.getContrastRatio = getContrastRatio;
exports.getLuminance = getLuminance;
exports.emphasize = emphasize;
exports.fade = fade;
exports.darken = darken;
exports.lighten = lighten;

You should add material-ui into your package.json.
It should not be in the .next/dist in anyway.
If the error still exists, create a sample repo and send us. (And of course re-open this issue)


Use custom commands to render markdown

Use tools like glow to render documentations inside a terminal buffer

Exactly the first thing i did! fetching the doc and immediately using glow on it

was gonna create an issue for this, so yeah something like glow works perfectly for this cause seeing raw markdown in a buffer isn't exactly functional... id rather just use the browser

was gonna create an issue for this, so yeah something like glow works perfectly for this cause seeing raw markdown in a buffer isn't exactly functional... id rather just use the browser

 As a workaround you could use the same command you open DevDocsOpenFloat and :Glow, and closing the DevDocsOpenFloat markdown but not the :Glow variant, closing it with q also. init.lua:
vim.cmd[[
augroup AutoCloseMarkdownGlow
  autocmd!
  autocmd FileType markdown,glowpreview nnoremap <buffer> q :q<CR>
augroup END
]]

vim.cmd[[
augroup MarkdownKeymaps
  autocmd!
  autocmd FileType markdown nnoremap <buffer> <leader>dd :lua MyGlow()<CR>
  autocmd FileType glowpreview nnoremap <buffer> <leader>dd :<CR>
augroup END
]]

function MyGlow()
  vim.cmd('Glow')
  for _, winid in ipairs(vim.fn.getwininfo()) do
    local bufnr = winid.bufnr
    local filetype = vim.api.nvim_buf_get_option(bufnr, 'filetype')
    if filetype == 'markdown' then
      vim.cmd('bw ' .. bufnr)
      return
    end
  end
end

-- Keymap for DevdocsOpenFloat
vim.api.nvim_set_keymap("n", "<leader>dd", ":DevdocsOpenFloat<CR>", {noremap = true, silent = true})

The only downside is that the first time you will try to do it, it will close the buffer because of plugin loading. But after the first one, it will always work. But you can adjust i guess with some time.sleep from lua
Screencast from 2023-08-12 16-32-04.webm

New in 5b6ba3c
Good news guys... You can now use custom cmds for previewing docs! Checkout the config options for more details.
It's still not available for the telescope previewer however, I'm still thinking on how to improve the performance...

I tried the glow support by adding previewer_cmd = 'glow',.
At first I was getting nothing at all displayed. But I was using glow 1.4.1. I upgraded it to 1.5.1 and now I'm getting something displayed, and it's the glow output, for instance links are handled differently.. However I'm getting no syntax highlighting at all. Not sure why for now.
I'm using linux.

ok, with the --s flags you give in the documentation i get syntax highlighting :+1:

ok, with the --s flags you give in the documentation i get syntax highlighting 👍

could you show me your setup options ?
i cant make it work for me

New in 5b6ba3c Good news guys... You can now use custom cmds for previewing docs! Checkout the new config options for more details. It's still not available for the telescope previewer however, I'm still thinking on how to improve the performance...

would you mind sharing your colorscheme theme and wallpaper background and terminal name ? 🍾

@I-Own-You I just used the settings from the readme, namely:
  previewer_cmd = 'glow'
  cmd_args = { "-s", "dark", "-w", "80" }

All pickers have previews now and support custom commands so I think I can close this.


FlipClock counts down too fast

FlipClock counts down 2 times faster, than it should. Like: 9, 7, 5, 3, 1 ...

Same thing here, when I set a start time.

I can confirm. Happens only for a new Date() object.

Happens even for the minute counter. The seconds countdown almost twice as fast than it should!

I'm also getting this error.

Thank you @aravindsrivats :)

I'm getting this error as well for a normal 24 hour clock.

same bug here when setting a date. No bug when setting countdown in second instead of date object.

var next_show = new Date('2018-12-01T00:00:00');
var diff = next_show.getTime() - new Date().getTime();
var showtime = Math.floor(diff/1000);
var clock = $('.clock').FlipClock({
clockFace: 'DailyCounter',
autoStart: false,
showSeconds: false,
countdown: true
});
clock.setTime(showtime);
clock.start();
showing wrong output

Has this ever been resolved? I'm facing both issues listed here and can't use the library because of it.

Setting the countdown with the following creates the same error:
var date = new Date("September 30, 2018 23:59:59");
var clock = $('.countdown').FlipClock(date,{
        clockFace: 'DailyCounter',
        countdown: true
    });

The following displays correctly and count downs correctly, as it should.
var timerDate   = "September 30, 2018 23:59:59"
var finaleDate  = new Date(timerDate);
var currentDate = new Date();
 var diff = finaleDate.getTime() / 1000 - currentDate.getTime() / 1000;
    if(diff < 0){
        diff = 0;
    }
    clock = $('.countdown').FlipClock(diff,{
        clockFace: 'DailyCounter',
        countdown: true,
    });

This should be fixed in the latest v1 draft on the js-es6 branch. The library now uses requestAnimationFrame's to handle the timer and animations instead of intervals. Should improve performance and issues like this. In my latest tests, I have seen the clock skip 2 numbers when the GPU starts to lag and catches up. Previously it would just lag behind. Sometimes the browser is intensive and the animations fall behind. Also in the new version animation rate is now half of the interval rate by default. So it executes the animation faster, so it also reduces it lagging behind.

Still broken in master for regular 24h flipclock and settting a non-Date value.
The bug is in compiled/flipclock.js:1348, function getDateObject:
it returns new Date().getTime() + this.getTimeSeconds()*1000, both of which are counting up.

It not going to be fixed in master until I merge js-es6, which will happen in the next day or two. I have the new docs mostly finished, they are in the repo on the new branch if you want to check them out.


RelativeLayout with HeightConstraint, WidthConstraint and YConstraint

Hi,
I need to migrate this code from Xamarin to MAUI using ReactorUI
 <Label
    FontSize="Small"
    Text="{Binding TextPoweredBy}"
    TextColor="{StaticResource Arancione}"
    RelativeLayout.HeightConstraint="{ConstraintExpression Type=RelativeToParent, Property=Height, Factor=1}"
    RelativeLayout.WidthConstraint="{ConstraintExpression Type=RelativeToParent, Property=Width, Factor=1}"
    RelativeLayout.YConstraint="{Binding YConstraintPoweredBy}"
    HorizontalOptions="Center" VerticalOptions="Start" />

but I'm not able to "translate" RelativeLayout.HeightConstraint, RelativeLayout.WidthConstraint and RelativeLayout.YConstraint in C# with ReactorUI.
Its possible to do this thing?
Thank you!

RelativeLayout is not supported in .net maui, use grid instead
https://learn.microsoft.com/en-us/dotnet/maui/migration/layouts


Not able to open encrypted pdf

I'm trying to load these file, but it is always showing same error

z-enc.pdf
Encryption: AES 128-bit
Password: 12345678
Private File
Encryption: RC4 128-bit

Both are opening without any error in any pdf viewer
const buff = await fs.readFile('z-enc.pdf')
const library = await PDFiumLibrary.init();

const password = "12345678";
const document = await library.loadDocument(buff, password);
const page = document.getPage(0)

Error
C:\Users\user\Desktop\PDF\node_modules\@hyzyla\pdfium\dist\library.js:52
                throw new Error(`PDF Loading = ${lastError}`);
                      ^

Error: PDF Loading = 4
    at PDFiumLibrary.<anonymous> (C:\Users\user\Desktop\PDF\node_modules\@hyzyla\pdfium\dist\library.js:52:23)
    at Generator.next (<anonymous>)
    at C:\Users\user\Desktop\PDF\node_modules\@hyzyla\pdfium\dist\library.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (C:\Users\user\Desktop\PDF\node_modules\@hyzyla\pdfium\dist\library.js:4:12)
    at PDFiumLibrary.loadDocument (C:\Users\user\Desktop\PDF\node_modules\@hyzyla\pdfium\dist\library.js:37:16)
    at main (C:\Users\user\Desktop\PDF\pdfium.js:9:33)

Node.js v22.1.0

PDFium was expecting a C string, and I just assumed by mistake (why?) that Emscripten somehow magically converts a JavaScript string to a C string

This issue is fixed in three latest release


Mono Glue generation Fails

Godot version:
c23710a
OS/device including version:
Windows10 / Mono 5.12
Issue description:

Steps to reproduce:
scons p=windows tools=yes module_mono_enabled=yes mono_glue=no

was it some problem with this specific commit? because I built it fine hours ago (c23710a)

When was the last time that mono_glue.gen.cpp file was created?

Damn, I've forgot that auto generated file should be deleted and recreated every time big changes is coming.. Sorry guys, false alarm

No, this file doesn't need to be delete if you pass mono_glue=no. This flag tells scons to not compile this file, so there won't be any errors and you can re-generate it then. When was the last time you generated this file?

^^ Several months ago.. Today, after deletion its compiling perfectly

That explains it then. Not too long ago I made a breaking change that intently required any old generated mono_glue.gen.cpp to be removed.


Show correct URL for CCTray badge

@msteinert thanks! although looks like @macb just beat you to it (#1287)

Haha, funny.

haha, thats great. :)


⚠ ClamAV has degraded performance

In ee0b9a2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in fd9eaa7.


Steps to Reproduce
See above. OPM Transaction: 0x2a523f1b74b93f5d8c1bdc7e5a1d106e7b2c6b1e31c3449a370b05048fec5ada

Bug 2 - Pre FP Withdrawals
OPM Transaction: 0xd45b140c2240ec54153cce72e2176cd0d263a9fbf81620b09dcc9a19faa92c15
This tx needs to be re-proven, but is still stuck on Loading withdrawal status...

Update on #1 - I now see "Ready to Prove" but "Prove" is grayed out

@MSilb7 I'm seeing the button enabled for https://github.com/ethereum-optimism/superchain-relayer/issues/54#issuecomment-2359597390

Hi, yeah I'm able to prove the new transaction as well, so may have been a timing thign before @nitaliano

The pre fp hash is showing ready-to-prove

Nice, just started proving old ones! We are in business.


Integration fails to start with new router hub SPNMX42

My ISP just upgraded to Linksys SPNMX42 and now I get this on startup:
2024-11-20 16:21:29.084 ERROR (MainThread) [custom_components.linksys_velop] Unexpected error fetching linksys_velop speedtest (55P20M16D01000) data
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 382, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/linksys_velop/coordinator.py", line 412, in _async_update_data
    result: dict[str, Any] = responses[0][0]
                             ~~~~~~~~~~~~^^^

I added a debug line at line 412 in coordinator.py:
_LOGGER.warning(responses)
This is now the output:
------------ SNIP -----------------
2024-11-20 17:23:28.687 WARNING (MainThread) [custom_components.linksys_velop.coordinator] [[], '']
2024-11-20 17:23:28.688 ERROR (MainThread) [custom_components.linksys_velop] Unexpected error fetching linksys_velop speedtest (55P20M16D01000) data
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 382, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/linksys_velop/coordinator.py", line 413, in _async_update_data
result: dict[str, Any] = responses[0][0]
~~~~~~~~~~~~^^^
IndexError: list index out of range
------------ SNIP -----------------
So basically responses[0][0] is empty

My guess is that as it is new, no Speedtests have been run.  I'll look to guard against that.  As you can imagine, I have a number of Speedtests that have been run so haven't seen this in testing.  I can see in your warning line that there's an empty list and an empty string so I'll look to use those as the guards.

I've just pushed 2024.11.3b0.  Could you try that and see how you get on please?  The link is: https://github.com/uvjim/linksys_velop/releases/tag/2024.11.3b0 just in case you need it.

Wow that was a quick response - thank you and much appreciated!
I have installed the branch you just made and now I get this:
2024-11-21 10:13:18.910 ERROR (MainThread) [custom_components.linksys_velop] Unexpected error fetching linksys_velop speedtest (55P20M16D01000) data
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 382, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/linksys_velop/coordinator.py", line 426, in _async_update_data
    ret = SpeedtestResults(
          ^^^^^^^^^^^^^^^^^
TypeError: SpeedtestResults.__init__() missing 6 required positional arguments: 'download_bandwidth', 'exit_code', 'latency', 'result_id', 'timestamp', and 'upload_bandwidth'
2024-11-21 10:13:18.911 DEBUG (MainThread) [custom_components.linksys_velop] Finished fetching linksys_velop speedtest (55P20M16D01000) data in 1.704 seconds (success: False)

Brilliant.  That's one step further forwards... I know what is happening there.  Just needs another guard.  Fix incoming...

I've just pushed 2024.11.3b1 Could you try that and see how you get on please? The link is: https://github.com/uvjim/linksys_velop/releases/tag/2024.11.3b1 just in case you need it.
Let me know how that one goes.

No problem. Glad it's working.


Admin website flashes during navigation

Looks like a dark-mode / light-mode flip when navigating

Looks like this is actually caused by the Dark Reader browser extension, even with the extension set to "off".
The "fix" for this can be observed by disabling or uninstalling the Dark Reader extension, or running the site in a private browsing tab (where browser extensions are disabled).
Apparently, Dark Reader alters default fallback styles in the browser to prevent "white flashes" when navigating between pages, because dark flashes in light mode is a lot less off-putting than light flashes in dark mode.
https://github.com/darkreader/darkreader/issues/4069

It feels like the recommendation then is to close this issue.
Closing it for now.  If something comes up again, we can re-open

Thank you for the research @jbw716 !

I agree. Not much to be done here. Definitely Dark Reader's problem.


How to serve static resources & dynamic content via cloudfront?

You noted that we can use CloudFront to serve static content from S3 and dynamic content via Lambda.
Could you share how the CloudFormation stack file would look?
I have no idea how to setup CloudFront to do this.

Hi @ivands, i'm currently in process of simplifying the build process so that deployment can be done in one command, im hoping to have this version ready in the next few days.

@ivands This new release is now published as 0.3.1 it should now be possible to deploy with a single command to AWS


Errors in running the script

The script gives me these two errors can you guide me how to resolve these errors to get logs  from imperva incapsula.

Hi
please review your configuration file , you should use:
922_413149 in the url and not 922-413149

Thanks alot benyo really appreciate your assistance, thanks once again.
On Mon, Mar 25, 2019 at 7:50 PM benyvo<EMAIL_ADDRESS>wrote:

Hi
please review your configuration file , you should use:
922_413149 in the url and not 922-413149
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Incapsula/logs-downloader/issues/26#issuecomment-476230793,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZq75WjudBpNi_KM-0rKeMLNXhzc2Csqks5vaOIkgaJpZM4cHCg_
.

Thank you for using our py script.
On Mon, Mar 25, 2019 at 5:00 PM Umar174<EMAIL_ADDRESS>wrote:

Thanks alot benyo really appreciate your assistance, thanks once again.
On Mon, Mar 25, 2019 at 7:50 PM benyvo<EMAIL_ADDRESS>wrote:

Hi
please review your configuration file , you should use:
922_413149 in the url and not 922-413149
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/Incapsula/logs-downloader/issues/26#issuecomment-476230793
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AZq75WjudBpNi_KM-0rKeMLNXhzc2Csqks5vaOIkgaJpZM4cHCg_
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Incapsula/logs-downloader/issues/26#issuecomment-476235611,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbSGojA9HH4fQfoGRIsW6jhdJ3EvQjj_ks5vaOSMgaJpZM4cHCg_
.

--
bv

Hi, this repository is deprecated.
please download the script from https://github.com/imperva/incapsula-logs-downloader


Adding minor detail to Configuration profiles step to device-management.md

Closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/7491.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@dstrome PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@dstrome PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@dstrome Hope you are well. Wanted to follow up here to see if there is an update? Please let us know if there is anything we can do to help. Thanks.

Change is no longer applicable to updated article. Closing this PR.


Fix Vagrant so that it uses new CLI

We will be able to avoid this message during the installation:

For more info:
https://optdyn.slack.com/archives/C93MLLTL4/p1535521197000100

Used the commands: vagrant init subutai/stretch-master
Vagrant used the new CLI and has got me a new versions of vagrant plugin and box:

Issue is fixed on master

Tested in prod


Improve error elaboration for types that have a property that references the type recursively

Revert commit 4beedcf from #5761.
It is no longer needed to prevent a crash in cases/compiler/elaborateErrors, and makes errors worse with properties that reference the containing type.
Fixes #7269 and doesn't re-break #5712 (as checked by the regression test/cases/compiler/elaborateErrors).
@ahejlsberg can you take a look? I understand why the change made the errors worse, but I don't understand why it was supposed to prevent the crash in #5712. I guess that some other, later change also fixed the source of the crash.

I wouldn't revert #5761 until you have a clear understanding of why #5712 is no longer an error.
I looks to me like #5761 is still needed as we might otherwise recurse indefinitely attempting to elaborate an error. The code I added ensures that we never recurse more than once and it is not clear to me what would keep that from happening otherwise.
I don't know why #5712 appears to work when you revert my fix. Perhaps you could use git bisect to find out?

@sandersn what is the status of this PR?

I couldn't prove that 4beedcf is no longer needed. If I remember correctly, it's largely because I couldn't show how subsequent changes (there were one or two likely ones) managed to fix both #5712 and whatever bugs they were intended to fix.
It's a mild enough bug that I would recommend closing this PR and the associated bug as Won't Fix. I think it's the safest approach without spending many more hours on this.

works for me.


v-model and :type

:type is not supported by v-model, but works with :value + @input.
If it is just syntactic sugar, why the difference?
https://vuejs.org/v2/guide/components.html#Form-Input-Components-using-Custom-Events
Could there be issues with <input :type="type" :value="value" @input="value = $event.target.value"> ?

That syntactic sugar is for components, it's a bit more complicated for inputs. Basically, some listeners must be set up and different inputs behave differently. You can dig further here: https://github.com/vuejs/vue/blob/dev/src/platforms/web/compiler/directives/model.js


python312Packages.iocsearcher: 1.0.0 -> 2.4.3-unstable-2024-10-08

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 359545

x86_64-linux

:white_check_mark: 4 packages built:

iocsearcher (python312Packages.iocsearcher)
iocsearcher.dist (python312Packages.iocsearcher.dist)
python311Packages.iocsearcher
python311Packages.iocsearcher.dist


Can you make it easier for the user to use?

I find it hard to adjust the value very quickly. Each time I swipe the handle to the side, I have to wait for the animation of it bouncing back to finish before I can swipe again. Also, the + and - on the two sides are too small for tapping. Can you make the + and - buttons' clickable area bigger?

you can customise this code by unlocking this cod. change animation duration and ,+, - sign, and and you can track code by ....... ahooo 💯


test new issue 123

sdsasda

Hi, I have some suggestions to help solving this issue!
Here is a list of similar issues:

1. [test new issue 123](https://github.com/fjsj/python-social-auth/issues/42)

2. [test new issue 123](https://github.com/fjsj/python-social-auth/issues/42)

3. [test new issue 123](https://github.com/fjsj/python-social-auth/issues/42)

4. [test new issue 123](https://github.com/fjsj/python-social-auth/issues/42)

5. [test new issue 123](https://github.com/fjsj/python-social-auth/issues/42)

Hope this is useful!
Cheers,
Cinnabot.


v1.12 Backports 2023-11-20

[ ] #29189 (@brb)

:warning: minor conflict

Once this PR is merged, a GitHub action will update the labels of these PRs:
 29189

/test-backport-1.12

/test-backport-1.12

/test-1.19-4.9

@brlbil whenever you have time, thanks a lot.


⚠ ClamAV has degraded performance

In 6d4cf3a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in bf114a0.


MINOR: Add note on Release management guide to Release/start PARQUET CPP versions

@pitrou this minor note is what was discussed on Zulip yesterday. It is a checklist with some minor notes as a reminder. I don't think it requires further detail but let me know if you think we should expand.


Order refunds

Following on from #1, backend & front end functionality needed to provide refunds.

Now admin page order/detail  has problems.
There are not enough files on the page:

http://..../static/longclaw.longclaworders/js/orders.bundle.js
http://..../static/longclaw.longclaworders/js/orders.bundle.js

@alexfromvl You probably need to compile the JS assets. This is done automatically when running setup.py (e.g. when installing from github) or you can cd into the longclaw/client directory and run npm install followed by npm run build

@JamesRamm Yes, after the manual startup files appear.
A suggestion is to make the collector start automatically when the command is "longclaw start mysite".
What do you think?

@alexfromvl
Building an sdist with setup.py will build the frontend assets so that the first distribution of longclaw submitted to pypi will have the assets already when a user pip install's longclaw. You are basically coming up against this problem since longclaw has not yet been released :)
For developers of longclaw, I think it is OK to expect them to run npm run build since they will need to do this if they make any changes to front end components anyway.
So, long story short: When longclaw 0.1.0 is released, the assets will be included

If I run the installation setup.py sdist, I get an error:
running sdist

><EMAIL_ADDRESS>build /home/longclaw/longclaw/client
> webpack --config webpack.config.js

sh: 1: webpack: not found

Perhaps before starting "'npm run build", it is necessary to do the init npm "npm init"

@alexfromvl
you would need to do an npm install (in the longclaw/client directory) before running npm run build. This should install all dependencies from package.json for building the assets, including webpack

@alexfromvl there is a now a command in the longclaw CLI to build the assets for you::
longclaw build


Please add logo positioning

Would it be possible to add a logo position setting? e.g. logo-position: top-right

Hey, It's open source!
He's a pull request: https://github.com/gitpitch/gitpitch/pull/88


MapAny: Extract

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

14 of 16 (87.5%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 90.307%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

values/mapany.go
14
16
87.5%

Totals

Change from base Build<PHONE_NUMBER>7:
-0.03%

Covered Lines:
1323

Relevant Lines:
1465

💛  - Coveralls


select [Number] does not return floating point, truncates to integer

var_dump from a double-float column.
Once defined for select as 'time [Number]' the other as 'time'.
string(15) "1496468069.4663"
int(1496468069)

change line 992 (or seek for case 'Number') to:
case 'Number':
	$stack[ $column_key ] = (double) $data[ $column_key ];
	break;

Same here!

The change on case 'Number': working fine if the numbers are bigger than 1.
all range between 0.01 - 0.99 returning 0
Is there a fix for this issue too ?


"You may have an infinite update loop in a component render function" when using rules in object format with optional parameters

Versions:

VueJs: 2.1.10
Vee-Validate: 2.0.0-rc.13

Description
Console throws a vue warning when attempting to validate an input that contains rules in object format WITH optional parameters being passed
Steps To Reproduce:
<input v-validate="{required: true, regex: [/^(?=.*[0-9])[- +()0-9]+$/, 'gm']}">  this warning is thrown with any rule that has optional parameters (like between). v-validate="{between: [1, 5]}"
message in console: [Vue warn]: You may have an infinite update loop in a component render function.

We are currently at rc.19 which comes a long way in fixing most issues, try to upgrade and report back if the issue still exists, also a live example would be appreciated.

I thought I might need to update, ill test it and post results with demo here tomorrow


Correction for raw affiliation AGIR EA4294, Medicine Department, Université de Picardie Jules Verne , Amiens , France; Infectious Disease Department, Amiens-Picardie University Hospital , FR-80000, Amiens , France

Correction needed for raw affiliation AGIR EA4294, Medicine Department, Université de Picardie Jules Verne , Amiens , France; Infectious Disease Department, Amiens-Picardie University Hospital , FR-80000, Amiens , France
raw_affiliation_name: AGIR EA4294, Medicine Department, Université de Picardie Jules Verne , Amiens , France; Infectious Disease Department, Amiens-Picardie University Hospital , FR-80000, Amiens , France
new_rors: 01gyxrk03;010567a58
previous_rors: 010567a58;01sjmsj73;01gyxrk03
works_examples: W4390618309
contact: 22a22d2538400bb18fdc110ccf31a9fa:b0f7bf976d6176a643da1eea944dcc8f @ u-picardie.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Add bazel-pipecd to gitignore

What this PR does / why we need it:
Ignore bazel-pipecd because changed the name of packages to pipecd.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice catch! Thank you.
/approve


[Question] How to process refunds?

Hi,
I se how I can make refunds, but how can I make them process? It's in the webhook I guess, but there is no example on how to do this. Would it be to fetch te order, check if there are refunds and if so, just call the handleProcessed() on the Refund model?

Sorry, I know that mollie takes oooon test environments around 2 hours to get processed by their system. However, I aimed to know how I can see the changes in my application. Since I don't see those appearing...

Can you share a code sample so we can reproduce the issue?

So I have made a refund with;
$order = Order::query()->find(x);
$order->refundCompletely();

That makes the refund in mollie. So that's good already.
This also makes table values in the refunds table;

And also in the refund_items table, so that's okay.
Now, the status will never change, since there is never an update when it's been processed.
My webhook liiks like the following;
final class WebhookController extends BaseWebhookController
{
    /**
     * @param Request $request
     * @return Response
     * @throws ApiException Only in debug mode
     */
    public function handleWebhook(Request $request): Response
    {
        $payment = $this->getPaymentById($request->get('id'));

        if ($payment) {
            $order = $this->getOrder($payment);

            if ($order && $order->mollie_payment_status !== $payment->status) {
                switch ($payment->status) {
                    case PaymentStatus::STATUS_PAID:
                        $order->handlePaymentPaid($payment);
                        break;
                    case PaymentStatus::STATUS_FAILED:
                        $order->handlePaymentFailed($payment);
                        break;
                    default:
                        break;
                }
            }
        }

        return new Response(null, 200);
    }

    /**
     * @param Payment $payment
     * @return Order|null
     */
    protected function getOrder(Payment $payment): ?Order
    {
        $order = Order::findByPaymentId($payment->id);

        if (!$order && isset($payment->metadata, $payment->metadata->temporary_mollie_payment_id)) {
            $order = Order::findByPaymentId($payment->metadata->temporary_mollie_payment_id);

            if ($order) {
                // Store the definite payment id.
                $order->update(['mollie_payment_id' => $payment->id]);
            }
        }

        return $order;
    }
}

Am I missing something here for refunds? I bet I am...

Hi @RosiersRobin ,
I am not sure why you have implemented your own controller for this as everything is already handled by Cashier for you...
But essentially the refunds are handled by the AftercareWebhookController.
Also note that - if I remember correctly - Mollie refunds receive no status updates in test mode. So there's no webhook request from Mollie after refund creation

Hi @sandervanhooft
The reason why I made my own callbacks and such, is because I don't have users to do payments. For example, you can create a registration in the system, with catering attached to that, but that isn't connected to a user. the order is then connected to the registration... Which is not really possible, or am I wrong?

@sandervanhooft hi,
I have tried to change all the self-written code to the implementation of the package, however, when I test all of this, I always get a 404 on the first-pament webhook url (/webhooks/mollie/first-payment).
An order gets never made, even after triggering all the webhooks myself (since it's local development).
cashier:run also doesn't make any kind of orders. I followed the full docs, but it just doesn't seem to work.

Can you find the webhook using php artisan route:list?


Wire screen is not pasting correctly

I am using the workshop version of wiremod, and yesterday created an engine-powered contraption to have some fun. However on the wire screen settings I checked "Left alignment" and "Floor value", however these settings did not apply when I pasted the Dupe.

This problem is long gone for me

Yeah, was a wire issue rather than an advdupe2 issue too.


The CLI usage does not consistent with relate doc

The CLI usage still show the name of command is openchain. Does not consistent with https://github.com/openblockchain/obc-docs/blob/master/api/Openchain API.md#open-blockchain-cli

Closed via 02d50f9c7691871253966128031dbe23d77ff34f


Could you pls provide license information on transform-pouch lib for me?

Hi Dear
I am thinking if apply transform-pouch lib on my project.
Could you pls provide license information for me?
Thanks on advance!

My bad; I will add an Apache license. Thanks!


Pre-Run Liquid Set Up for PD protocols

give PD users the starting liquid information as a new step in the pre-protocol flow

Closing since we have this ticketed elsewhere


Different redirect route according to permission

Hello,
How do i make Admin and Normal users redirect to different controllers after login.
Like if logged in user is admin go to
Route::get('/dashboard', 'DashboardController@admin');
if logged in user is normal user
Route::get('/dashboard', 'DashboardController@normal');
Thanks

I've done it a couple different ways, the most recent way was a DashboardController@director which redirected them to either location, or you could have everything point to normal but if you're an admin get a redirect to the admin, or everything point to admin and redirect the normals

I actually did it this way, if there is simpler way or im doing something very stupid, let me know.
in my route will point to DashboardController@index and in the index following
if(Gate::allows('admin'))
        {
           $this->showSuperuserstuff();
        }
        else
        {
             $this->showSupernormalstuff();
        }

Let me know.

Yup I've done it that way before too :) I don't really know an official 'BEST' way

The way I've done it by using AuthenticatesUsers trait.
\App\Http\Controllers\Auth\LoginController.php

Add this method to that controller:
    /**
     * Check user's role and redirect user based on their role
     * @return
     */
    public function authenticated()
    {
        if(auth()->user()->hasRole('admin'))
        {
            return redirect('/admin/dashboard');
        } else {
            return redirect('/user/dashboard');
        }
    }

Hey @amirulfirdaus put in a PR I like that


Add more statistics for instances

Add more statistics for instances in remove_by_properties function. Changes need to be made:

Add a variable to calculate stats. Now remove_by_properties function is only called when some filtering is applied (TEST.POST_PROCESSING.REMOVE_BY_PROPERTIES).
Calculate perimeter (necessary for sphericity)
Calculate sphericity (for 3D). Here is the formula.

Implemented in c720318 using same formulas as in MorphoLibJ but using scikit-image functions.


Bond generated headers don't work for clang in c++20

Bond generates code as follow:
https://godbolt.org/z/a4qnbcjeh
#include <vector>

struct forward_decl;

struct S{
    std::vector <forward_decl> m;

    S(){};
};

struct forward_decl{};

But this does not work in C++20 (in clang) because the constructor will be instantiated with the forward declaration and compilation will fail.
A possible fix is to move the constructor body until after all entities have bee fully defined:
https://godbolt.org/z/6YjarYx3c
#include <vector>

struct forward_decl;

struct S {
    std::vector<forward_decl> m;

    S();
};

struct forward_decl {};

S::S() {}

Office is currently hitting this when trying to update to C++20.

This change makes sense and should be backward compatible to C++11 as well. Feel free to submit a PR with the changes to code generation.
Does S::S() {} need to be marked inline as it won't be inside a class definition?


feat: support multi-format feed

compatibility with hexo-generator-feed 2.1+

https://github.com/hexojs/hexo-theme-landscape/pull/170


I think the text that is printed to console should be more informative

I think it should say the name of your pet instead of "your pet"

Merged branch with these changes successfully.


Add example Docker Compose of running lakeFS with PostgreSQL for K/V store

This issue highlighted that we don't publish a Docker Compose showing lakeFS running with postgres, so I'm adding it in this PR.

@rmoff I was able to create Samples container but it didn't import Samples data to minio so few sample notebooks will fail.


Question: how to deal with haskell packages that have deps behind a flag not reflected in nixpkgs?

Issue description
I'm trying to build a simple haskell project compiled with ghcjs which has a dependency on react-flux package. That package when being compiled with ghcjs adds a conditional dependency on ghcjs-base as seen here. This doesn't appear to be reflected in react-flux package in nixpkgs and I don't know how to conditionally override it to depend on ghcjs-base.
Steps to reproduce

Create simple haskell project.
Make it depend or react-flux.
Try to build it with ghcjs.

Technical details
Built on top of a162c6c828 with few custom commits not related to haskell.

Came up with a work around for this package based on https://github.com/NixOS/nixpkgs/issues/16583


Make the number of repos returned configurable with an environment variable

Default it to 6 if not specified.

I kind of want this implemented ASAP so I can use it in one of my projects. I'm guessing you'd want tests for this too? Or can we let this slide without one?

It will need a test, yes, but that should be very easy: https://github.com/sindresorhus/gh-latest-repos/blob/5cda67aa4b4b68f0bc340e9d7578b78042a93f3b/test.js#L47-L50


Regression: generateScopedName ignored by vitest

Describe the bug
Upgrade from vitest 0.18.1 to 0.24.1 causes my tests fail because generateScopedName setting is being ignored.
Possibly caused by this PR: https://github.com/vitest-dev/vitest/pull/1803
I tried setting classNameStrategy to 'scoped' which should make it respect generateScopedName but it doesn't work
In the reproduction link below, tests fail unless you downgrade vitest version (eg to 0.18.1):
Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-fdmmn9?file=package.json
System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.1 LTS 22.04.1 LTS (Jammy Jellyfish)
    CPU: (8) x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
    Memory: 2.66 GB / 15.33 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 14.19.1 - ~/.nvm/versions/node/v14.19.1/bin/node
    npm: 8.18.0 - ~/.nvm/versions/node/v14.19.1/bin/npm
  Browsers:
    Chrome: 106.0.5249.119
  npmPackages:
    @vitejs/plugin-vue: ^3.1.2 => 3.1.2
    vite: ^3.1.7 => 3.1.7
    vitest: ^0.24.1 => 0.24.1

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is not a regression. You need to use css.include option, if you want your files to gow through Vite pipeline instead of being mocked.

Thanks, with this I can get my generated class names to work again:
css: { modules: { classNameStrategy: 'scoped' }, include: /.*/ },

Imo by default generated class names during tests should be the same as in production to avoid issues going undetected but w/e


only test ipaddr if feature is enabled

Description of changes
only test ipaddr if feature is enabled
Issue #, if available
https://github.com/cedar-policy/cedar/issues/120
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):

[ ] A breaking change requiring a major version bump to cedar-policy (e.g., changes to the signature of an existing API).
[ ] A backwards-compatible change requiring a minor version bump to cedar-policy (e.g., addition of a new API).
[x] A bug fix or other functionality change requiring a patch to cedar-policy.
[ ] A change "invisible" to users (e.g., documentation, changes to "internal" crates like cedar-policy-core, cedar-validator, etc.)
[ ] A change (breaking or otherwise) that only impacts unreleased or experimental code.

I confirm that this PR (choose one, and delete the other options):

[ ] Updates the "Unreleased" section of the CHANGELOG with a description of my change (required for major/minor version bumps).
[x] Does not update the CHANGELOG because my change does not significantly impact released code.

I confirm that cedar-spec (choose one, and delete the other options):

[x] Does not require updates because my change does not impact the Cedar Dafny model or DRT infrastructure.
[ ] Requires updates, and I have made / will make these updates myself. (Please include in your description a timeline or link to the relevant PR in cedar-spec, and how you have tested that your updates are correct.)
[ ] Requires updates, but I do not plan to make them in the near future. (Make sure that your changes are hidden behind a feature flag to mark them as experimental.)
[ ] I'm not sure how my change impacts cedar-spec. (Post your PR anyways, and we'll discuss in the comments.)

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

Lgtm. Will there be a separate PR for decimal?

cargo test --no-default-features --verbose currently passes, so I wasn't planning on anything more.


how to update last value ?

let a = new x.SMA({
    period: 3,
    values: [3, 1, 2, 4, 6]
})
console.log(a.getResult())  //[ 2, 2.3333333333333335, 4 ]

//a.updateLastValue(9)  //return 5      //The latest price is real time updates

let b = new x.SMA({
    period: 3,
    values: [3, 1, 2, 4, 9]
})
console.log(b.getResult())  //[ 2, 2.3333333333333335, 5 ]

There is no update last value. There is only nextValue

I double @fc01 you get live data for say a 1 hour chart that updates every second, how are you supposed to get the latest rsi for that duration without recalculating everything

as said there is no update last values there is only nextValue which means you will be able to get the next value after the bar completes and not intermediate updates


START_SSH_SERVER = true works only in rootless images

Description
Decided to switch to gitea builtin ssh server, so i changed config:
[server]
START_SSH_SERVER = true

But it seems that this config works only in rootless images (1.17.3-rootless in my case). With version 1.17.3, a separate ssh server is always started.
Gitea Version
1.17.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Debian 11.4
How are you running Gitea?
With Docker compose.
# cat docker-compose.yml
version: "3.3"
services:
  server:
    restart: always
    image: gitea/gitea:${GITEA_VERSION}
    volumes:
      - /etc/localtime:/etc/localtime:ro
      - /usr/share/zoneinfo:/usr/share/zoneinfo:ro
      - ${DATA}:/data
    ports:
      - "${HTTP_PORT}:3000"
      - "${SSH_PORT}:22"

# cat .env
GITEA_VERSION=1.17.3
HTTP_PORT=3000
SSH_PORT=2222
DATA=./data

Database
No response

This is by design - although we could do the work to make the root docker start the openssh server as needed it's in a lot of ways unnecessary.
If you do not need to run openssh concurrently with the gitea server and are happy with using the builtin ssh server (or no ssh server) you should use the rootless docker build.

Thank you for your quick answer!

This is by design - although we could do the work to make the root docker start the openssh server as needed it's in a lot of ways unnecessary.

I think this would be worth mentioning in the documentation.

If you do not need to run openssh concurrently with the gitea server and are happy with using the builtin ssh server (or no ssh server) you should use the rootless docker build.

I tried, but: https://github.com/go-gitea/gitea/issues/21647

#21647 has been resolved.


Share image directly

I want to share image to other app directly. There is currently only one button to save the image.

this seems somewhat related to #43


Use steps to charge for gas instead of buckets

Description
Cherry pick gas schedule changes
Test Plan
Running existing tests

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

could not strictly speaking cherry pick so I had to port changes.
I'll check tests once they run.
And I may change differently the limits tests though I am not sure....


feat: add postWhatsappTrackingEntry api

References: CAR-6006
New api to store data collected by the whatsapp modal

@Averethel added the helper function you suggested and tested all 4 forms it affects. Is it ok to merge it?

@Averethel added the helper function you suggested and tested all 4 forms it affects. Is it ok to merge it?

Sure, go for it

Sure, go for it


Desktop notifications

Some sort of notifications for new chat messages, trade offers etc.

Basic desktop notifications added in https://github.com/scholtzm/punk/commit/16cc38e8cc2746d7f6141c36db8734966c100a7c


Bug: Cards not responsive

What happened?
The cards are not responsive, text inside them are not readable in small screen.
Add screenshots

What browsers are you seeing the problem on?
Microsoft Edge
Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

Go ahead @aryansharma220


shields: adafruit_2_8_tft_touch_v2: add arduino nano connector support

Allow shield to be used with any board that has an
arduino nano connector.

@jfischer-no gentle ping on my question

@jfischer-no gentle ping on my question

What question or where?

@jfischer-no are you not seeing this review?

Nope, and according to your screenshot, it is still pending.

@jfischer-no are you not seeing this review?

Nope, and according to your screenshot, it is still pending.

Duh... Sorry for that -ETOOTIRED


Now grid lines can shown at any coordinates. as taking camera

Now grid lines can shown at any coordinates. as taking camera coordinates as referance

Now grid lines can shown at any coordinates. as taking camera as referance


Add generator for coversheets functionality.

Pull Request Test Coverage Report for Build 155

136 of 184 (73.91%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.4%) to 75.854%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/leaf_addons/citation/citation_service.rb
31
32
96.88%

lib/leaf_addons/configuration.rb
38
41
92.68%

lib/leaf_addons/coversheet/coversheet_derivative_path.rb
3
7
42.86%

lib/leaf_addons/coversheet/coversheet_service.rb
53
93
56.99%

Totals

Change from base Build 146:
-1.4%

Covered Lines:
333

Relevant Lines:
439

💛  - Coveralls


Improve installation documentation when using a tarball

The current instructions to install DTK do not work if we are downloading a tarball.
I will cherry-pick this commit.

Codecov Report

Merging #445 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #445   +/-   ##
=======================================
  Coverage   96.83%   96.83%
=======================================
  Files         123      123
  Lines        9094     9094
=======================================
  Hits         8806     8806
  Misses        288      288

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 3d476f9...bf0691d. Read the comment docs.


Functional tests

@ericof I polished the generator until I reached what I've found this morning:
https://github.com/plone/cookieplone-templates/actions/runs/9840829807/job/27165988173#step:15:15
If you run make check inside the backend folder, tox gets crazy and do not find the project.pytoml file.
Also in the backend, there's no make format, the check command is make check while in the frontend is make lint.
I kind of dislike that check also does format, because it leads to tons of confusion.

I recall that we abolished make check because it means "check code", not "reformat and write code". Let's burn it with 🔥. make lint or make format both make sense, depending on what the command actually does.

I'm +1000 with @stevepiercy on this one.


Slow mode support

I'm wondering if it would be feasible using the current features of the library to record a video in slow motion?
Thanks,

Any idea about how that could be implemented?

@jmpaya  Adding new features may be considered, but it is unlikely they will be implemented soon. If any features are added, I will notify you.


Problem with LinkingSetRelation

I have the following field definition with LinkingSetRelation attribute:
    public sealed class PermitRow : Row, IIdRow, INameRow
    {
        [DisplayName("Request"), NotNull, Unique, PrimaryKey, ForeignKey("[dbo].[REQUEST]", "RequestId"), LeftJoin("jRequest"), TextualField("RequestRequestNumber")]
        public Int32? RequestId
        {
            get { return Fields.RequestId[this]; }
            set { Fields.RequestId[this] = value; }
        }
.....
        [DisplayName("Zoning"), NotMapped, LookupEditor(typeof(ZoningRow), Multiple = true)]
        [LinkingSetRelation(typeof(PermitZoningRow), "RequestId", "ZoneId")]
        [MinSelectLevel(SelectLevel.Details), QuickFilter]
        public List<Int32> PermitZones
        {
            get { return Fields.PermitZones[this]; }
            set { Fields.PermitZones[this] = value; }
        }
....
        IIdField IIdRow.IdField
        {
            get { return Fields.RequestId; }
        }
.....
   }

When I try to save PermitRow, I get the following error:
Cannot insert the value NULL into column 'RequestId', table 'permitsTEST.dbo.PERMIT_ZONING'; column does not allow nulls. INSERT fails.
The strange thing is that I've successfully used the LinkingSetRelation in other classes. Searched through other related issues and I analyzed the Northwind Customer representatives scenario "to death". The only thing different is that RequestId is both PrimaryKey and ForeignKey (there is no Identity column in PERMIT table which works just fine). I just can't tell what I'm doing wrong. Any hint will be greatly appreciated.

I don't know the version of SF you are using - but at least until recently I thought it was not possible to not have an identity field which is not an int - and I dont know if it is already possible.
As it seems from the error message, you are trying to save a record with nothing (null) in the RequestId field. Either you have set this field within your database to not allow null (not Null) or you have set the corresponding field definition within your xyzRows.cs to "notNull".
That would be my first gueesses...
With kind regards,
John

Hi @JohnRanger,
The RequestId is defined as int. It is also non-null because it's a primary key of PermitRow. Looking at the LinkingSetRelationBehavior source code, everything should be working. The master key (RequestId) should be automatically set during save (it should come from the PermitRow). I tried to setup VisualStudio to debug Serenity classes but it crashes so I can't step through all the code. I have everything set up identically as in the Customer Representatives sample:
Master key: RequestId -> CustomerId (both have PrimaryKey NotNull attributes)
Item key: ZoneId -> EmployeeId (both have Identity attribute)
Not sure what I am doing wrong...

Spent way too much time struggling with this. Thinks the problem is that the LinkingSetRelation requires Identity field for master key (PrimaryKey will not cut it). As a workaround I stored the multiple zones in a single string field:
    [DisplayName("Zoning"), Size(50), LookupEditor(typeof(ZoningRow), Multiple = true, Delimited = true)]
    public String Zoning
    {
        get { return Fields.Zoning[this]; }
        set { Fields.Zoning[this] = value; }
    }

Not ideal but will work for now.


Nullability attributes are not included in WPF temp projects which are part of the WPF build process

Usage of any nullability attributes in a WPF project result in this:

CS0122 'NotNullWhenAttribute' is inaccessible due to its protection level

The error is reported as coming from a project named YourProjectName_*_wpftmp.csproj.
The reason is that the NullableAttributes.cs file is not included in such a way that this WPF build task can see it:
https://github.com/dotnet/wpf/blob/master/src/Microsoft.DotNet.Wpf/src/PresentationBuildTasks/Microsoft.WinFx.targets#L401
https://github.com/dotnet/wpf/blob/master/src/Microsoft.DotNet.Wpf/src/PresentationBuildTasks/Microsoft/Build/Tasks/Windows/GenerateTemporaryTargetAssembly.cs#L90
Repro:
<Project Sdk="Microsoft.NET.Sdk.WindowsDesktop">

  <PropertyGroup>
    <OutputType>WinExe</OutputType>
    <TargetFramework>net472</TargetFramework>
    <UseWPF>true</UseWPF>
  </PropertyGroup>

  <ItemGroup>
    <PackageDownload Include="Microsoft.NETCore.App.Ref" Version="[3.1.0]" />
    <PackageReference Include="TunnelVisionLabs.ReferenceAssemblyAnnotator" Version="1.0.0-alpha.154" PrivateAssets="all" />
  </ItemGroup>
</Project>

public partial class MainWindow : Window
{
    public MainWindow()
    {
        _ = typeof(NotNullWhenAttribute);
        this.InitializeComponent();
    }
}

Would it be possible to optionally expose the attributes as public? Can we perhaps take advantage of InternalsVisibleToAttribute?

@chucker Public is dangerous. Rather than adding a new option that is a workaround, I think it would be better for WPF projects for now to opt out with <GenerateNullableAttributes>false</GenerateNullableAttributes> and manually add a source file with public attributes. Especially since the WPF team is considering fixing this problem.

I figured out how to fix it.


fix(els-footnotes): add ready decorator to disconnected callback

:tada: This PR is included in version 3.13.0-beta.5 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Index is index within group not overall

In renderItem, the index being passed in is the index of an item within a group. This means when grouping is used, there is no overall "unique" index that can ID an object. Makes poking and prodding at elements a bit difficult!
Even having a "groupidx" and a regular idx to uniquely ID each item would be useful.

Thanks @devlinb. That makes sense. So if im understanding it well, when grouping you get something like this
idx: [0, 1, 2, 3], [0, 1, 2], ...
which makes keying items challenging. And you would like to get something like:
idx: [(1, 0), (2, 0), (3, 0)], [(1, 1), (2, 1)]...
where you have the idx of the item and index of the group, or something like:
idx: [1.0, 2.0, 3,0], [1,1, 2,1]...
right?

This got fixed with following PR: https://github.com/ECorreia45/flatlist-react/pull/7
Changes will be on version 0.1.3(https://github.com/ECorreia45/flatlist-react/releases/tag/0.1.3).


Clarity on exactly which docstring styles are supported?

It's not clear exactly which docstrings are supported. Hoping to clear that up.
What I've gathered below, is this correct?

Markdown: Yes.
Google: Yes.
Numpy: No.
ReST: Sort of.

So docstring parsing is delegated to pytkdocs, which supports Google, and some limited ReST.
Referring to pytkdocs where they state "1: reStructured Text parsing is in active development and is not feature complete yet."

Your summary seems correct

Your summary seems correct

Hello @connerxyz, sorry for the confusing docs :slightly_smiling_face:

I think it states pretty clearly what we support (google-style and restructuredtext style), but we could emphasize restructuredtext is only partially supported, as a style and not a whole markup (mkdocstrings converts markdown only, nothing else, so rest markup will be converted as if it was markdown as well).
There's also a PR for Numpy style pending on pytkdocs :slightly_smiling_face:
Docs will be updated when/if it's merged of course

Hello @connerxyz, sorry for the confusing docs :slightly_smiling_face:

I think it states pretty clearly what we support (google-style and restructuredtext style), but we could emphasize restructuredtext is only partially supported, as a style and not a whole markup (mkdocstrings converts markdown only, nothing else, so rest markup will be converted as if it was markdown as well).
There's also a PR for Numpy style pending on pytkdocs :slightly_smiling_face:
Docs will be updated when/if it's merged of course


⚠ Fortnite has degraded performance

In ee12341, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1689 ms

Resolved: Fortnite performance has improved in 660d15f.


Add queue_name to queued method options for Flavor model

This PR adds a queue_name to the queue options for the Flavor.create_flavor_queue and Flavor#delete_flavor_queue methods. I've also added some specs (these methods were previously uncovered), and some comments on those methods.
Part of https://github.com/ManageIQ/manageiq/issues/19543

@agrare Please review.


feat: support yi ai model

Ⅰ. Describe what this PR did
Support Yi ai model, api documentation（https://platform.lingyiwanwu.com）
Ⅱ. Does this pull request fix one issue?

fixes #957
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
envoy.yaml:
admin:
  address:
    socket_address:
      protocol: TCP
      address: <IP_ADDRESS>
      port_value: 9901
static_resources:
  listeners:
    - name: listener_0
      address:
        socket_address:
          protocol: TCP
          address: <IP_ADDRESS>
          port_value: 10000
      filter_chains:
        - filters:
            - name: envoy.filters.network.http_connection_manager
              typed_config:
                "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
                scheme_header_transformation:
                  scheme_to_overwrite: https
                stat_prefix: ingress_http
                # Output envoy logs to stdout
                access_log:
                  - name: envoy.access_loggers.stdout
                    typed_config:
                      "@type": type.googleapis.com/envoy.extensions.access_loggers.stream.v3.StdoutAccessLog
                # Modify as required
                route_config:
                  name: local_route
                  virtual_hosts:
                    - name: local_service
                      domains: [ "*" ]
                      routes:
                        - match:
                            prefix: "/"
                          route:
                            cluster: yi
                            timeout: 300s
                http_filters:
                  - name: wasmtest
                    typed_config:
                      "@type": type.googleapis.com/udpa.type.v1.TypedStruct
                      type_url: type.googleapis.com/envoy.extensions.filters.http.wasm.v3.Wasm
                      value:
                        config:
                          name: wasmtest
                          vm_config:
                            runtime: envoy.wasm.runtime.v8
                            code:
                              local:
                                filename: /etc/envoy/plugin.wasm
                          configuration:
                            "@type": "type.googleapis.com/google.protobuf.StringValue"
                            value: |
                              {
                                "provider": {
                                  "type": "yi",
                                  "apiTokens": [
                                    "your-apiTokens"
                                  ]
                                }
                              }
                  - name: envoy.filters.http.router
  clusters:
    - name: httpbin
      connect_timeout: 30s
      type: LOGICAL_DNS
      # Comment out the following line to test on v6 networks
      dns_lookup_family: V4_ONLY
      lb_policy: ROUND_ROBIN
      load_assignment:
        cluster_name: httpbin
        endpoints:
          - lb_endpoints:
              - endpoint:
                  address:
                    socket_address:
                      address: httpbin
                      port_value: 80
    - name: yi
      connect_timeout: 30s
      type: LOGICAL_DNS
      dns_lookup_family: V4_ONLY
      lb_policy: ROUND_ROBIN
      load_assignment:
        cluster_name: yi
        endpoints:
          - lb_endpoints:
              - endpoint:
                  address:
                    socket_address:
                      address: api.lingyiwanwu.com
                      port_value: 443
      transport_socket:
        name: envoy.transport_sockets.tls
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.UpstreamTlsContext
          "sni": "api.lingyiwanwu.com"

Request:
curl "http://localhost:10000/v1/chat/completions"  -H "Content-Type: application/json" -d '{
  "model": "yi-large",
  "messages": [
    {
      "role": "user",
      "content": "你好，你是谁？"
    }
  ]
}'

Response:
{
    "id": "cmpl-1cef946a",
    "object": "chat.completion",
    "created": 12063916,
    "model": "yi-large",
    "usage": {
        "completion_tokens": 42,
        "prompt_tokens": 15,
        "total_tokens": 57
    },
    "choices": [
        {
            "index": 0,
            "message": {
                "role": "assistant",
                "content": "你好！我是一个人工智能助手，专门设计来回答问题、提供信息和帮助解决问题。人们通常叫我“AI助手”或者类似的名字。如果你有任何问题或需要帮助，请随时告诉我！"
            },
            "finish_reason": "stop"
        }
    ]
}

Ⅴ. Special notes for reviews

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

cc @CH3CHO


Update rbac for source namespaces

What type of PR is this?
/kind bug
What does this PR do / why we need it:
Add missing permissions and update rolebinding reconciliation logic for sourceNamespaces feature
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:

@jaideepr97 can you PTAL

To be noted, this PR was raised to attempt to fix an issue where apps in non-control namespaces don't show up in the Argo CD UI - this PR onyl fixes it half way as apps are now showing up but the dependent resources in target namespaces don't show up or don't get created even when Argo CD says the app is healthy and sync'd
we will need a follow up PR to address the remaining changes


Clean project deps

Description
I've removed some deps that seem extra or indirect.
Eventually, we should pin everything we have in lock in pyproject.toml to be Konflux compliant. But now, it would be an overkill.
The app still works as usual. But I can't confirm if those aren't used in some corners. Like indexer.py?
Type of change

[x] Refactor

The app still works as usual. But I can't confirm if those aren't used in some corners. Like indexer.py?

indexer.py is deleted
but probably good to run the e2e:
/test e2e-ols-openai-gpt35

/lgtm
/approve


Can't revert change that deletes a single empty line in side-by-side diff editor

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.69-insider
OS Version: MacOS Monterey

Steps to Reproduce:

Having a side-by-side diff editor
Make a change that removes a single empty line
Try to revert the change with the arrow #151826

Due to an inaccurate if check in revertChange method [1], the delete of an empty line is not correctly recognized (sorry about that!)
@hediet
[1] https://github.com/microsoft/vscode/blob/1bf95bcd9ec667f2f9d0a577e4383caf4073adae/src/vs/editor/browser/widget/diffEditorWidget.ts#L646 shoud be ... && originalContent != null) instead of just checking ... && originalContent)

Thanks! It would be awesome if we had some tests for this!