Duplicate error messages for collection input field items

This happens when items of first element more then second
Here is an example
Configuration file
0 => [
	'name' => 'test1',
	'type' => \Zend\InputFilter\InputFilter::class,
	'test1-child' => [
		'type' => \Zend\InputFilter\CollectionInputFilter::class,
		'input_filter' => [
			'field' => [
				'type' => \Zend\InputFilter\CollectionInputFilter::class,
				'input_filter' => [
					'type' => [
						'required' => true,
						'validator' => [
							0 => [
								'name' => \Zend\Validator\Between::class,
								'options' => [],
							]
						],
					],
					'price' => [
						'required' => true,
						'validator' => [
							0 => [
								'name' => \Zend\Validator\Between::class,
								'options' => [],
							]
						],
					],
				]
			],
		]
	],
]

Sending data
test1[test1-child][1][field][0][type] = 159s -> wrong data
test1[test1-child][1][field][0][price] = 15.2568s -> wrong data
test1[test1-child][1][field][1][type] = 2180s -> wrong data
test1[test1-child][1][field][1][price] = 25.2568s -> wrong data
test1[test1-child][1][field][2][type] = 2180s -> wrong data
test1[test1-child][1][field][2][price] = 25.2568s -> wrong data
test1[test1-child][1][field][4][type] = 2180s -> wrong data
test1[test1-child][1][field][4][price] = 25.2568s -> wrong data

test1[test1-child][2][field][0][type] = 159 -> corect data
test1[test1-child][2][field][0][price] = 15.2568 -> wrong data
test1[test1-child][2][field][1][type] = 2180 -> corect data
test1[test1-child][2][field][1][price] = 25.2568 -> corect data

Validation Response
"validation_messages": {
	"test1-child": {
		"1": {
			"field": [
				{
					"type": {
						"error_key": "Error message",
					},
					"price": {
						"error_key": "Error message"
					}
				},
				{
					"type": {
						"error_key": "Error message",
					},
					"price": {
						"error_key": "Error message"
					}
				},
				{
					"type": {
						"error_key": "Error message",
					},
					"price": {
						"error_key": "Error message"
					}
				},
				{
					"type": {
						"error_key": "Error message",
					},
					"price": {
						"error_key": "Error message"
					}
				}
			]
		},
		"2": {
			"field": [
				{
					"type": {
						"error_key": "Error message",
					},
					"price": {
						"error_key": "Error message"
					}
				},
				{
					"type": {
						"error_key": "Error message",
					},
					"price": {
						"error_key": "Error message"
					}
				},
				{
					"type": {
						"error_key": "Error message",
					},
					"price": {
						"error_key": "Error message"
					}
				},
				{
					"type": {
						"error_key": "Error message",
					},
					"price": {
						"error_key": "Error message"
					}
				}
			]
		},
	}
}

@weierophinney Can you have a look and improve this as soon as you can

@developer-devPHP
Please do not ping a single person in the problem description. There is no need for this.
And please provide a simple and working example. Your example is too complicated and includes errors. (validators not validator!)
Here a simple and working example:
$factory     = new \Zend\InputFilter\Factory();
$inputFilter = $factory->createInputFilter(
    [
        'collection-element' => [
            'type'         => \Zend\InputFilter\CollectionInputFilter::class,
            'required'     => true,
            'input_filter' => [
                'type'  => [
                    'validators' => [
                        [
                            'name'    => \Zend\Validator\Between::class,
                            'options' => [
                                'min' => 50,
                                'max' => 100,
                            ],
                        ],
                    ],
                ],
                'price' => [
                    'validators' => [
                        [
                            'name'    => \Zend\Validator\Between::class,
                            'options' => [
                                'min' => 50,
                                'max' => 100,
                            ],
                        ],
                    ],
                ],
            ],
        ],
    ]
);

$inputFilter->setData(
    [
        'collection-element' => [
            [
                'type'  => 10,
                'price' => 10,
            ],
            [
                'type'  => 20,
                'price' => 20,
            ],
        ],
    ]
);
var_dump($inputFilter->isValid());
var_dump($inputFilter->getMessages());

Result:
false

array (size=1)
  'collection-element' =>
    array (size=2)
      0 =>
        array (size=2)
          'type' =>
            array (size=1)
              'notBetween' => string 'The input is not between '50' and '100', inclusively' (length=52)
          'price' =>
            array (size=1)
              'notBetween' => string 'The input is not between '50' and '100', inclusively' (length=52)
      1 =>
        array (size=2)
          'type' =>
            array (size=1)
              'notBetween' => string 'The input is not between '50' and '100', inclusively' (length=52)
          'price' =>
            array (size=1)
              'notBetween' => string 'The input is not between '50' and '100', inclusively' (length=52)

And to solve your problem, you must add break_on_failure to an input:
'type'  => [
    'break_on_failure' => true,
    'validators'       => [
        [
            'name'    => \Zend\Validator\Between::class,
            'options' => [
                'min' => 50,
                'max' => 100,
            ],
        ],
    ],
],

@froschdesign got your note related person ping.
Here is an example as you want.
$factory = new \Zend\InputFilter\Factory();
$inputFilter = $factory->createInputFilter(
    [
        'element' => [
            'type' => \Zend\InputFilter\InputFilter::class,
            'type1' => [
                'type' => \Zend\InputFilter\CollectionInputFilter::class,
                'input_filter' => [
                    'test_field' => [
                        'type' => \Zend\InputFilter\CollectionInputFilter::class,
                        'input_filter' => [
                            'test_field1' => [
                                'required' => true,
                                'validators' => [
                                    [
                                        'name' => \Zend\Validator\Between::class,
                                        'options' => [
                                            'min' => 50,
                                            'max' => 100,
                                            'message' => '%value% is incorrect',
                                        ],
                                    ],
                                ],
                            ],
                            'price' => [
                                'required' => true,
                                'validators' => [
                                    [
                                        'name' => \Zend\Validator\Between::class,
                                        'options' => [
                                            'min' => 50,
                                            'max' => 100,
                                            'message' => '%value% is incorrect',
                                        ],
                                    ],
                                ],
                            ],
                        ],
                    ],
                ],
            ],
        ],
    ]
);

$inputFilter->setData(
    [
        'element' => [
            'type1' => [
                [
                    'test_field' => [
                        [
                            'test_field' => -20,
                            'price' => 20,
                        ],
                        [
                            'test_field' => -15,
                            'price' => 15,
                        ],
                        [
                            'test_field' => -10,
                            'price' => 10,
                        ],
                    ],
                ],
                [
                    'test_field' => [
                        [
                            'test_field' => -5,
                            'price' => 5,
                        ],
                    ],
                ],
            ],
        ],
    ]
);
var_dump($inputFilter->isValid());
var_dump($inputFilter->getMessages());

Result

false

array(1) {
  'element' =>
  array(1) {
    'type1' =>
    array(2) {
      [0] =>
      array(1) {
        'test_field' =>
        array(3) {
          [0] =>
          array(2) {
            'test_field1' =>
            array(1) {
              'isEmpty' =>
              string(36) "Value is required and can't be empty"
            }
            'price' =>
            array(1) {
              'notBetween' =>
              string(15) "20 is incorrect"
            }
          }
          [1] =>
          array(2) {
            'test_field1' =>
            array(1) {
              'isEmpty' =>
              string(36) "Value is required and can't be empty"
            }
            'price' =>
            array(1) {
              'notBetween' =>
              string(15) "15 is incorrect"
            }
          }
          [2] =>
          array(2) {
            'test_field1' =>
            array(1) {
              'isEmpty' =>
              string(36) "Value is required and can't be empty"
            }
            'price' =>
            array(1) {
              'notBetween' =>
              string(15) "10 is incorrect"
            }
          }
        }
      }
      [1] =>
      array(1) {
        'test_field' =>
        array(3) {
          [0] =>
          array(2) {
            'test_field1' =>
            array(1) {
              'isEmpty' =>
              string(36) "Value is required and can't be empty"
            }
            'price' =>
            array(1) {
              'notBetween' =>
              string(14) "5 is incorrect"
            }
          }
          [1] =>
          array(2) {
            'test_field1' =>
            array(1) {
              'isEmpty' =>
              string(36) "Value is required and can't be empty"
            }
            'price' =>
            array(1) {
              'notBetween' =>
              string(15) "15 is incorrect"
            }
          }
          [2] =>
          array(2) {
            'test_field1' =>
            array(1) {
              'isEmpty' =>
              string(36) "Value is required and can't be empty"
            }
            'price' =>
            array(1) {
              'notBetween' =>
              string(15) "10 is incorrect"
            }
          }
        }
      }
    }
  }
}

@developer-devPHP
Sorry, but your example is still too complicated. One collection element illustrates the same problem. Reduce your example to a minimum and it is easier to understand your problem.
This helps us and also you!
Have you seen and tested the solution?

If it's just one simple collection, in that case no this kind of problem. Problem coming, during this situation, when inside of collection new collection.
Please just run my previous example and you can see what's the problem.
I used your solution break_on_failure to an input field, but it's not helpful.
I think problem in $collectionMessages in CollectionInputFilter class during recursive get and set messages (in isValid method)  Zend\InputFilter\CollectionInputFilter Line 203

Please just run my previous example and you can see what's the problem.

This was my first problem, I could not reproduce your problem. Therefore I asked for a working example. Now I can see what you mean. (Thanks!)

I used your solution break_on_failure to an input field, but it's not helpful.

Right, because break_on_failure works only for a single input.

I think problem in $collectionMessages in CollectionInputFilter class during recursive get and set messages…

This not a problem, it's a feature. If you provide 6 inputs and all inputs are invalid, then 6 error messages are returned.
I would override the getMessages method for this special use case, if not all error messages are needed.

Let me clarify one thing. I need all messages, but in example as you can see the messages for first item is duplicated on second item. In first item I send 3 fields value, in second item only 1, but in error messages you can see 3 error messages in second item.

…or first item is duplicated on second item.

Ah, you mean the second test_field after -5 and 5. Right, all these messages are wrong.
I will test it.
(A hard nut to crack! 😉 )

Do you have any news related this issue?

@froschdesign Is this issue abandoned,or maybe you or staff working on it?
Can you tell me any update.
Thanks.

@developer-devPHP

Is this issue abandoned…

No. The issue is still open.

…or maybe you or staff working on it?

This is a community project and everyone can help. It would be great, if you have time and write the unit test. We will review it.

Thanks @svycka  For me issue solved.
@froschdesign If it's not breaking other logical part please push it with  next release.
Thanks.

@developer-devPHP

For me issue solved.

Thanks for the feedback!

…please push it with next release.

A review is needed and then we can merge and release it.

@froschdesign do we have any updates related review statuses?


disable the settings automatic to default after restart and shutdown

it work good but when i shutdown and restart the laptop the settings i made is gone and i must make the settings after the laptop begin running manually

it work good but when i shutdown and restart the laptop the settings i made is gone and i must make the settings after the laptop begin running manually

Thank you for opining this Issue. For your security the settings you made will not be saved to the kernel forever. Unfortunately I can do nothing to this because the kernel do it automatically.


Warnings on the version of pip and pipenv

Hi,
When I deploy automatically on a Cloud Foundry Application in IBM cloud, I get this warning. What should I do get ride of it?
WARNING: You are using pip version 19.2.3, however version 20.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

I tried playing with Dockerfile and Dockerfile-tools files in the distribution but with no effect. When do these files come into play?
Best,
-- M

Thanks for the issue @marc-dms. Just to confirm, what are the commands you are executing?

I am using IBM "delivery pipeline". This takes place during the "Rolling deploy" step. I get an earlier warning BTW:
Deprecation warning: Use of 'buildpack' attribute in manifest is deprecated in favor of 'buildpacks'. Please see https://docs.cloudfoundry.org/devguide/deploy-apps/manifest-attributes.html#deprecated for alternatives and other app manifest deprecations. This feature will be removed in the future.
Does that answer your question?

So for Cloud Foundry deployments, we use the Python buildpack which manages its own dependencies in the operating system. I suspect the warning is coming from that buildpack having an older version of pip. I suggest opening an issue with that buildpack at https://github.com/cloudfoundry/python-buildpack/issues. Since the starter kit is working properly the warning can be safely ignored. I'll close this but please reopen if you find this issue is preventing the created app from working properly.
As for the deprecation warning about the buildpack attribute, we'll track that internally and fix it.


Nest the numbers indicating the version of package in code elements

In RTL, these texts will be broken, because they are a mix of special character, numbers, which follow different direction in RTL than LTR.

~1.2.13
1.2.x
1.x.x

To fix these, add code elements around the version number.
File: https://github.com/freeCodeCamp/freeCodeCamp/blob/659f05558904fe5554b1a93fb87311d6904a9e7f/curriculum/challenges/english/05-back-end-development-and-apis/managing-packages-with-npm/expand-your-project-with-external-packages-from-npm.md
change

Add version "1.1.0"

to

Add version 1.1.0

there are multiple instances in these files
https://github.com/freeCodeCamp/freeCodeCamp/blob/659f05558904fe5554b1a93fb87311d6904a9e7f/curriculum/challenges/english/05-back-end-development-and-apis/managing-packages-with-npm/use-the-caret-character-to-use-the-latest-minor-version-of-a-dependency.md
https://github.com/freeCodeCamp/freeCodeCamp/blob/659f05558904fe5554b1a93fb87311d6904a9e7f/curriculum/challenges/english/05-back-end-development-and-apis/managing-packages-with-npm/use-the-tilde-character-to-always-use-the-latest-patch-version-of-a-dependency.md
All of them should be wrapped in code elements

Speaking of weird interactions. Using special characters that relay on direction will break in RTL, because it will be reverted, in example.

similar to how the <head> section

will be

similar to how the >head< section.

we can wrap <head> in https://github.com/freeCodeCamp/freeCodeCamp/blob/659f05558904fe5554b1a93fb87311d6904a9e7f/curriculum/challenges/english/05-back-end-development-and-apis/managing-packages-with-npm/how-to-use-package.json-the-core-of-any-node.js-project-or-npm-package.md

@Sboonny  I want to contribute towards this issue. Can you assign this issue to me.

The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute, and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum


Update scan to make use of the fastlane_core feature to not look for GH issues

This will fail until https://github.com/fastlane/fastlane/pull/5574 and https://github.com/fastlane/fastlane/pull/5573 are merged

👍

Tests are green 👍


Styleguide example

[ ] Link in docs and hosted during dev
[ ] Fractal js
[ ] icon list
styleguidest?
start with:

buttons?
checkbox?
sm/md/lg?
colors?
font?
page types?

Static:
http://styleguide.sc5.io/
Dynamic:
https://github.com/sourcejs/Source


sonar collector is not supporting at sub project level

I have a project in SONAR, which has multiple sub projects. I need to configure the hygieia collector for the sub projects in sonar.
Currently in GUI the collector is listing only the main project and not the subprojects. There is no option to configure the sub-projects to the collector, where as SONAR rest APIs support fetching the data at subproject level.
Kindly help by adding this feature(configuring SONAR subprojects at team level).
Regards,
Nisar T

Affects:  sonar-codequality-collector-3.0.2

@mohdnisart At this point the collector does not support it, if you have a way to implement this please contribute it back. We would greatly appreciate that!


Analytics is not defined

I read your documentation. that is really complete and nice, but I have a problem
Here is my config inside a App.run()
App.config(function (AnalyticsProvider) {
            //Add configuration code as desired - see below
            AnalyticsProvider
                .logAllCalls(true)
                .setAccount(my_code_is_here)
                .trackPages(true)
                .trackUrlParams(true)
                .delayScriptTag(true)
                .readFromRoute(true);
        });

It is not working, when I go and see my Analytics account, it is not counting my visits!
I thought I need to call Analytics.pageView(), but it returns error which is:

Uncaught ReferenceError: Analytics is not defined

It is not working at all. But I added Google Analytics script  and it can show I am online now, but it is not working when I change routes (And I think this is not the standard way):
       (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
                (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
            m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
        })(window,document,'script','https://www.google-analytics.com/analytics.js','ga');

        ga('create', my_code_is_here, 'auto');
        ga('send', 'pageview');

May I know; what is my problem? And what should I do?

Please see #135 as well.


error when running test profile: Input tuple does not match input set cardinality declared by process count

when running:
nextflow run qbic-pipelines/cellranger
I'm getting:
N E X T F L O W  ~  version 21.04.1
Launching cellranger/main.nf [lonely_turing] - revision: 04c22e90bc

                                    ,--./,-.
    ___     __   __   __   ___     /,-._.--~'

|\ | |__  __ /   /  \ |__) |__         }  {   | \| |       \__, \__/ |  \ |___     \-.,--,                                         .,._,'
qbic-pipelines/cellranger v1.0.1
Core Nextflow options
runName                   : lonely_turing
containerEngine           : docker
container                 : qbicpipelines/cellranger:1.0
launchDir                 : /home/ec2-user/environment/qbic
workDir                   : /home/ec2-user/environment/qbic/work
projectDir                : /home/ec2-user/environment/qbic/cellranger
userName                  : ec2-user
profile                   : test,docker
configFiles               : /home/ec2-user/environment/qbic/cellranger/nextflow.config
Input/output options
input                     : https://raw.githubusercontent.com/qbic-pipelines/cellranger/master/testdata/input_data.tsv
index_file                : true
Reference genome options
save_reference            : false
reference_name            : Homo_sapiens.GRCh38.chr_22
fasta                     : https://raw.githubusercontent.com/qbic-pipelines/cellranger/dev/testdata/Homo_sapiens.GRCh38.dna.chromosome.22.fa
gtf                       : https://raw.githubusercontent.com/qbic-pipelines/cellranger/dev/testdata/Homo_sapiens.GRCh38.104.chr_22.gtf
Generic options
max_multiqc_email_size    : 25 MB
Max job request options
max_cpus                  : 2
max_memory                : 6 GB
max_time                  : 2d
Institutional config options
config_profile_name       : Test profile
config_profile_description: Minimal test dataset to check pipeline function

Only displaying parameters that differ from defaults.
[-        ] process > get_software_versions -
[-        ] process > get_references        -
[-        ] process > get_software_versions -
[-        ] process > get_references        -
[-        ] process > get_software_versions [  0%] 0 of 1
[-        ] process > get_references        -
executor >  local (1)
[-        ] process > get_software_versions [  0%] 0 of 1
executor >  local (1)
[-        ] process > get_software_versions [  0%] 0 of 1
executor >  local (2)
[da/985c31] process > get_software_versions [  0%] 0 of 1
executor >  local (3)
[da/985c31] process > get_software_versions               [100%] 1 of 1 ✔
executor >  local (3)
[da/985c31] process > get_software_versions               [100%] 1 of 1 ✔
executor >  local (4)
[da/985c31] process > get_software_versions               [100%] 1 of 1 ✔
[-        ] process > get_references                      -
[ac/4eafc5] process > build_references (build_references) [100%] 1 of 1 ✔
executor >  local (4)
[da/985c31] process > get_software_versions               [100%] 1 of 1 ✔
[-        ] process > get_references                      -
[ac/4eafc5] process > build_references (build_references) [100%] 1 of 1 ✔
executor >  local (5)
[da/985c31] process > get_software_versions               [100%] 1 of 1 ✔
[-        ] process > get_references                      -
[ac/4eafc5] process > build_references (build_references) [100%] 1 of 1 ✔
executor >  local (5)
[da/985c31] process > get_software_versions               [100%] 1 of 1 ✔
[-        ] process > get_references                      -
[ac/4eafc5] process > build_references (build_references) [100%] 1 of 1 ✔
executor >  local (6)
[da/985c31] process > get_software_versions               [100%] 1 of 1 ✔
[-        ] process > get_references                      -
[ac/4eafc5] process > build_references (build_references) [100%] 1 of 1 ✔
executor >  local (6)
[da/985c31] process > get_software_versions               [100%] 1 of 1 ✔
[-        ] process > get_references                      -
[ac/4eafc5] process > build_references (build_references) [100%] 1 of 1 ✔
[8a/3afba2] process > fastqc (pbmc_1k_v3_mini)            [100%] 2 of 2 ✔[dd/2baeeb] process > count (Library1)                    [  0%] 0 of 1[-        ] process > multiqc                             [  0%] 0 of 1[5f/1aba8d] process > output_documentation                [100%] 1 of 1 ✔WARN: Input tuple does not match input set cardinality declared by process count -- offending value: [Library1, [pbmc_1k_v3_mini, pbmc_1k_v3_mini], [L001, L002], [/qbic-pipelines/cellranger/master/testdata/pbmc_1k_v3_mini_S1_L001_R1_001.fastq.gz, /qbic-pipelines/cellranger/master/testdata/pbmc_1k_v3_mini_S1_L002_R1_001.fastq.gz], [/qbic-pipelines/cellranger/master/testdata/pbmc_1k_v3_mini_S1_L001_R2_001.fastq.gz, /qbic-pipelines/cellranger/master/testdata/pbmc_1k_v3_mini_S1_L002_R2_001.fastq.gz], [/qbic-pipelines/cellranger/master/testdata/pbmc_1k_v3_mini_S1_L001_I1_001.fastq.gz, /qbic-pipelines/cellranger/master/testdata/pbmc_1k_v3_mini_S1_L002_I1_001.fastq.gz]]

Hi @jennomics , thanks it is not an error but a warning, but in any case this will be solved in the next pipeline release, which will be ported to DSL2


added centos support

first run at issue #37

@jmccann let me know if you have any thoughts on this pr please

Thanks for the contribution!


[SPARK-48990][SQL] Unified variable related SQL syntax keywords

What changes were proposed in this pull request?
The pr aims to unified variable related SQL syntax keywords, enable syntax DECLARE (OR REPLACE)? ... and DROP TEMPORARY ... to support keyword: VAR (not only VARIABLE).
Why are the changes needed?
When setting variables, we support (VARIABLE | VAR), but when declaring and dropping variables, we only support the keyword VARIABLE (not support the keyword VAR)

https://github.com/apache/spark/blob/285489b0225004e918b6e937f7367e492292815e/sql/api/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBaseParser.g4#L68-L72
https://github.com/apache/spark/blob/285489b0225004e918b6e937f7367e492292815e/sql/api/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBaseParser.g4#L218-L220
The syntax seems a bit weird, inconsistent experience in SQL syntax related to variable usage by end-users, so I propose to unify it.
Does this PR introduce any user-facing change?
Yes, enable end-users to use variable related SQL with consistent keywords.
How was this patch tested?
Updated existed UT.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @srielau @cloud-fan

thanks, merging to master!


feat: add docker build for arm

This pull requests add ARM builds to docker so one can host the explorer on an ARM server. The approach is to only do amd build on prs to not increase the build time and build both arm + amd on the main branch for the release.
Related: I added ARM support to the stacks-blockchain-api in this pr https://github.com/hirosystems/stacks-blockchain-api/pull/1947

I would love to see this merged.

Is there anything missing from this pr? Would love to run the API on my ARM server :)

Sorry for the delay, I'd like to just get sign-off from a repo maintainer before merging. @BLuEScioN or @He1DAr can you review?

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


Some content is not usable when you enlarge the text

Some elements have fixed font sizes, for example <code> elements.

https://tdt-documentation.london.cloudapps.digital/multipage.html#build-a-multipage-site
These elements are sized using pixels which is why they do not scale.
Failing WCAG Success Criteria

1.4.4: Resize text - Level AA

I know many auditors who wouldn't interpret this as a WCAG fail, mostly because zooming didn't exist when WCAG was written. See also https://alastairc.ac/2017/11/is-text-sizing-dead/
I'm on the fence about this one and would generally love to see more research on how people make text bigger. I would consider a "loss of content or functionality" a definite fail, that text size doesn't increase not necessarily, depending on how bad it is.

I think if you required a very large text size you would not be able to read these elements.
It also seems to be a fail based on the wording of the criteria:
"Except for captions and images of text, text can be resized without assistive technology up to 200 percent without loss of content or functionality."

This is very interesting! @nickcolley did you get this result with the zoom tool in the Empathy Lab? It's curious that keyboard zooming in the browser (I tried Chrome and Safari) doesn't lead to this inconsistency 🤔

There's two ways to do zooming, the one you're familiar with and text zooming. Text zooming requires you to update settings in your browser.

On reflection I agree with Anika, we should fix this but I don't think it's a WCAG failure since the content is not obscured.

Just to clarify, there are lots more ways to increase text. Apart from browser zoom and browser text sizing, there are also different kinds of magnification, OS text increase, custom styles, changing the display resolution, using physical magnifiers, etc.
(Sorry for being nitpicky, this is not really related to the issue, just wanted to clarify a potential misconception.)


Find out why  "Javascript not enabled" is picked up by google as relevant to our share page?

Also, how can we avoid this? :smile:

@meols it is the euroland iframe that is used that has a text "javascript is not enabled". If Euroland adds a meta tag telling bots not to index (like  ) then it might be excluded, but it has to be on their end

Thanks @millianapia :smile: - I've forwarded info to the graph/iframe vendor, so this ticket can be closed.


Some characters in Japanese (and likely other languages) don't show up when a font is embedded with subset: true

What were you trying to do?
I'm trying to generate a PDF with user-generated text in a lot of different languages / scripts. I'm using a font that includes a large portion of Japanese, Traditional Chinese, Simplified Chinese, and Korean. It is a huge font, so I'd like to use {subset: true} so that the entire font isn't embedded into the PDF.
How did you attempt to do it?

Embed the font that contains the CJK scripts. Use {subset: true}
Render the text using drawText

What actually happened?
Many character don't show up in the final PDF.
However! If I take out {subset: true} then the character show up, but the PDF is huge because it contains a huge font.
Here is the PDF. It should show these characters:
私は日本語を書かないので機械翻訳を使ってバグ
再現用のこのテキストを作成しました
As you'll see, most are missing. ScribbleTogether (34).pdf
What did you expect to happen?
The character should show up even when {subset: true} is enabled, which results in a PDF with the right characters that is also a manageable size.
How can we reproduce the issue?
I'm really having trouble with this JS Fiddle! It should show the bug but is stuck on a different unrelated error. https://jsfiddle.net/bridger/ehdajb6c/21/
It produces an error when encoding the string, which doesn't happen in my webpack builds. This unrelated error showing up in JSFiddle is "Error: Not a fixed size". It seems to come from https://github.com/foliojs/restructure/blob/7b2f4b07d8ca154827afc9fd2b6aa8e20812a421/src/utils.js#L19 according to https://github.com/foliojs/pdfkit/issues/1359
Version
1.17.1
What environment are you running pdf-lib in?
Browser
Checklist

[X] My report includes a Short, Self Contained, Correct (Compilable) Example.
[X] I have attached all PDFs, images, and other files needed to run my SSCCE.

Additional Notes
No response

I published a new package renamed to pdf-fontkit to fix the subsetting issues. I've tested it and it works fine. Maybe you can use this package to replace @pdf-lib/fontkit.


Include license files

Can we add a license file to the .deb and/or .rpm package?

Yes, there are mechanisms to do this and actually, we'll need to include a few from the other projects we're using if/when we package them statically.
For relay-pkg.so as it is now I think we need to do this for:

PHP
jemalloc
hiredis
ck


payment state is balance_due

Hello I am using gem 'spree_braintree_vzero', github: 'spree-contrib/spree_braintree_vzero', branch: '3-1-stable'
and done following setup:

Run bundle install
bundle exec rails g spree_braintree_vzero:install
Done settings at spree backend payment methods set merchant_id, public_key, private_key and set autocapture-yes
restarted my server
added rake task in heroku scheduler rake spree_braintree_vzero:update_states

Now issue is my payment is happening successfully but not able to recive payment_status as 'PAID' it always gives balance_due so wis there any api call missing and how to set this.
Thanks
Dinshaw Raje

this issue is came when we skip 'confirm' step of checkout flow.


HAWQ-1342. Fixed QE process hang in shared input scan on segment node

The basic idea for this kinds of hung problem is to:
(1) The error thrown segment will invoke rollback the whole transaction, and all related fd will be closed during transaction end.
(2) The other segment just act as before, when wait for select(), it will loop until the specific fd is closed, then the code will run until process interrupt (the rollback transaction will send cancel signal) again in other place afterward.
So some previous fix (HAWQ-166,  HAWQ-1282) will be changed accordingly.
(1) HAWQ-166: we don't need to skip sending info
(2) HAWQ-1282:

we don't need to close the fd, it will be closed automatically during transaction end.
we just end loop if we find the related FD has already been closed.

Signed-off-by: Amy Bai<EMAIL_ADDRESS>

+1

LGTM. +1


Make swift-operator image FIPS-compliant

Set CGO_ENABLED=1 using extra args and use the ubi-minimal image to make this operator FIPS compliant according to the check-payload tool[1].
[1] https://github.com/openshift/check-payload

@cschwede we might need a follow up to set the features.operators.openshift.io/fips-compliant: "true" annotation in the clusterservice manifest (see https://github.com/openstack-k8s-operators/cinder-operator/pull/330)

@fmount: Thx, follow up PR: https://github.com/openstack-k8s-operators/swift-operator/pull/119


Host_Error: Mod_FindModelOrigin: EOF Without Closing Brace

While this problem is not very common, it is still worth bringing some light onto it.
First, what I was basically doing, is that I am trying to cross over NPCs of Scientist Slaughterhouse into another mod campaign (Typical Disaster is one of them), even to make some of them attack the player like the NA Slaves.
In order to do that, I opened up the secondary training level, copied the MyClass function from one of the monstermaker entities spawning the NA Agents whilst using Raw Edit, I pasted it into the monster_na_slave NPC, and given the MyClass attribute "4," which is definitely the way to turn an NPC evil. Unfortunately, once I made an attempt to load the level where the edited NPC is, the game disconnects me from the level, and an error saying "Host_Error: Mod_FindModelOrigin: EOF Without Closing Brace" had shown up.
I addressed this problem to a1batross first, and he explains the issue this way, at least in his words:
"It looks like there is an entity "monster_na_slave" with key "MyClass" but the problem is that at the end of the key, there is a NULL terminator (basically "MyClass\0", which then makes entities list parser think that it's where EOF is.
It's probably a bug of newbspguy if it allows pasting such data or inserts it with a bug."
So, my suggestion here, is to investigate the root cause of the problem and polish it off. Many thanks.

I think this issue already reported and fixed in one of previous commits.
imgui_stdlib.cpp produced unexpected null bytes in std::string, I think it fixed in latest nightly releases.

@ToxicHolyGrenade just need to use latest build, closed as already completed


Use test database in development mode

In development mode (#69) the app could use a test database (special database name).

There is now a new build type...

The standard build type in the development environment ist debug. Now, debug uses a special database for development purposes.
There is also the new build type staging. staging uses the "normal production database".

The developer has to choose which build type she wants to deploy to her phone: Build -> Select Build Variant...


Uncaught TypeError: a.target.className.indexOf is not a function at bubble_compiled.js:176

I try to use md-ng-icon directive inside md-button of Angular Material and button tag of Lumx.  I keep on to receive the following error when I implement ng-click event at md-button or button tag:
Uncaught TypeError: a.target.className.indexOf is not a function
This error message occurred at bubble_compiled.js:176
Here is my template segment:

You forgot to attach template segment?
It will help if you can send a Plunker.

template segment?
Here is my code

I can go to ui-router state after click but error message always come out.  Any help?

Can you reproduce by forking this?
http://plnkr.co/edit/37No9RsPNLZcDlQYoXqm?p=preview

Closing this for no follow-ups.


LOG in backend don't show in logcat

In the past (v2 branch), LOG lines in backend will appear in logcat. But in the flutter app (master branch), LOG lines in backend don't show in logcat.
p.s. I can see LOG message from cpp/flutter/dart_run_benchmark.cc in logcat. But no lines from backends.

LOG(INFO) <<  ... should work. Is it redefined? @d-uzlov and @freedomtan to check if we can reproduce this.

@freedomtan  are you using the apk generated by make docker/flutter/android/apk ?

nope, docker is slow on my old machines :-)
simply
$ make WITH_MEDIATEK=1 WITH_PIXEL=1
$ cd flutter
$ flutter run

Ah, sorry, it's my mistake. In the past (android version), backends are created for each benchmark. Now backends are created when app started and I missed the log messages.


carbon catabolite repression of transcription from RNA polymerase II promoter by glucose

carbon catabolite repression of transcription from RNA polymerase II promoter by glucose (GO:0000433)
Which term do you propose this term merge into? Provide GO ID and term label
GO:0000122    negative regulation of transcription by RNA polymerase II
Reason for merge
This really describes the phenotype "carbon catabolite repression' and is non standard wording for
GO terms
Any other information
Obsoletion might be better because the correct way to annotate would be
GO:0000122    negative regulation of transcription by RNA polymerase II
during GO:0071333    cellular response to glucose stimulus

in fact it is so confusing, there are even semantically identical terms like
carbon catabolite repression of transcription from RNA polymerase II promoter by glucose (GO:0000433)
AND
GO:0061987  negative regulation of transcription from RNA polymerase II promoter by glucose

To illustrate, we annotated the same thing in 3 ways!

See https://github.com/geneontology/go-annotation/issues/5204
There is a good review here:
https://pubmed.ncbi.nlm.nih.gov/31119370/
It seems this may be a signaling pathway that responds to nutrient sources, leading to transcription. In any case, I think this fit better under 'carbon utilization' than under transcription.
@srengel
@marcfeuermann
Any insights about these terms?

this is specifically about transcription, so 'utilization' is not the best way to describe this.

it's not a pathway that LEADS to transcription, it's a pathway that REPRESSES transcription of genes necessary for utilization of non-preferred carbon sources (the repression part) when preferred carbon sources are present (the glucose part).

but it's not just glucose, it can be other preferred carbon sources.
therefore, @ValWood 's suggestion to annotate as following is correct:
GO:0000122 negative regulation of transcription by RNA polymerase II
during GO:xxxxxxx cellular response to [whichever] stimulus


Only update Ingress status for the configured class

fixes #412

This change is

@gianrubio please check

@aledbf more readable,  lgtm.

@aledbf nice!
One issue though (I think), if I change my ingress service (after its running and correctly got the endpoint), to not match my selected ingress-class, it will not remove the endpoint.

Coverage decreased (-0.07%) to 46.261% when pulling ad247847005f26083c4213e9040ab1dfd20c5efb on aledbf:fix-status-update into db96c9d574ec4c312538c47f112b1348879df573 on kubernetes:master.


Allow empty content type for multipart form field

Sorry for the new PR. I squashed the changes + created a new branch

Fixed

Thanks!


Support running as a daemon on Mac

mac: Add plist file to run nexd as a daemon
docs: Reorganize agent install instructions
docs: Document running nexd with launchd on mac

commit 9f7cc98eaebf7c31943cf35df68f660eca199307
Author: Russell Bryant<EMAIL_ADDRESS>Date:   Fri Apr 28 22:27:35 2023 -0400
mac: Add plist file to run nexd as a daemon

Add the file needed to run `nexd` as a daemon under launchd on mac.

Signed-off-by: Russell Bryant<EMAIL_ADDRESS>
commit<PHONE_NUMBER>27d8559d83d0f87bd756a594c9634c
Author: Russell Bryant<EMAIL_ADDRESS>Date:   Sat Apr 29 19:41:15 2023 -0400
docs: Reorganize agent install instructions

Reorganize the agent install docs to include install+run instructions
together instead of split across 2 sections.

Signed-off-by: Russell Bryant<EMAIL_ADDRESS>
commit 46dcc86db22c3ac24cbc42100dd7766a6794b696
Author: Russell Bryant<EMAIL_ADDRESS>Date:   Sat Apr 29 19:59:17 2023 -0400
docs: Document running nexd with launchd on mac

These instructions describe how to set this up manually. Once it is
merged and present in the `prod` tag, I will update `homebrew-nexodus`
to set this up automatically and adjust the documentation accordingly.

Signed-off-by: Russell Bryant<EMAIL_ADDRESS>

After a few days of using this, I'm not confident we have it right. I haven't narrowed down the exact explanation yet, but I think it might be after updating nexodus, it's broken. If I run nexd manually in a terminal, then I get a pop-up asking if wireguard-go is allowed to accept network connections. After I approve, it works.
I think this needs more testing before we document it as a recommended way to run on mac.

I haven't been able to reproduce this problem, so I guess it's ok to proceed with documenting this now.

@chirino i think this can go in


bugbash - az bot publish should not require --proj-file-path AND --code-dir

Tool
Name: AZ CLI
Version: 2.0
OS: windows
Describe the bug
When publishing a .net bot, providing the .csproj should be enough.  I don't see why --code-dir is necessary
Expected behavior
only the .csproj should be required for c# deployments
[bug]

Since we are deprecating msbot as a provisioning and deployment option, replaced with ARM template support, I'm going to resolve this issue as "won't fix"

@sgellock I believe this is related to az bot (AZ CLI), not to msbot.

we are deprecating msbot as a provisioning and deployment option, replaced with ARM template support

Does this have impact on people who are currently deploying using az bot publish?

@hansmbakker we're suggesting customer use Zip deploy to deploy their bots to Azure.  So for C# bots it would look like the following:
for C#

az bot prepare-deploy --lang Csharp --code-dir "." --proj-file-path "MyBot.csproj"
zip your bot (next cli command assumes a zip named code.zip):  it is the folder that has the .csproj file
az webapp deployment source config-zip --resource-group "<new-group-name>" --name "<name-of-web-app>" --src "code.zip"

we've got updated documentation queued up which will document this much better than what we currently have.  Should land on or about May 1.

@sgellock thank you for the clarification!
Besides whether this works, could you explain how the bot builder team see this fit into a CI/CD pipeline where one deployment package (artifact) is created that is deployed to multiple environments?
Currently that does not work because of different bot files being needed for different environments.

@sgellock if you use --code-dir "." and zip the folder that has the .csproj file, does that still work if there is a project in ..\DependencyProject?
E.g. if you have

the head project in BotSolutionFolder\WebAppBot\WebAppBot.csproj
business logic in BotSolutionFolder\BusinessLogic\BusinessLogic.csproj

And you make BotSolutionFolder\WebAppBot your working folder and only supply --code-dir ".", then you miss your BusinessLogic project I assume?

@hansmbakker I haven't tried the dependency project scenario you've raised.  should be easy to test.  basically zip deployment is simply taking a compiled structure, packaging it up, and then unzipping it on the azure server.  if you want help here, I can ask our support team to work the issue with you.

@hansmbakker, when you use the zipdeploy api via az webapp deployment source config-zip, the API is designed to take a ready-to-go website/app/bot. The code should already be built and working.
This is scenario is applicable if you have not configured your App Service to build on deployment; meaning you haven't set SCM_DO_BUILD_DURING_DEPLOYMENT to true on your App Service's Application Settings.
Note: You don't need to manually configure this environment variable to be false for zipdeploy to do its thing.

@sgellock Thank you for thinking along! I'll try it next Tuesday as the Easter weekend already started here :)


cannot read property 'Unicode' of undefined

Hi, I have success loading font with this plugin, however, it gives me a problem when trying to do splitTextToSize(). I got an error message: can not read property 'Unicode' of undefined. While I am using built-in fonts, it just running fine.
In order to load custom fonts, I am using node makeFonts.js approach.
Here is the font which I used: https://fonts.google.com/specimen/Nunito+Sans . Is there any clue what kind of fonts that I can use with this plugin?
Any help will be highly appreciated. Thank you in advance.

After a while tracking trough jsPDF source code, I have seen that this issue initially caused by getFont() function, inside API.splitTextToSize() function. Somehow when it reach getCharWidthsArray() function it is finally bring the error which need to get option.widths or as alternate is activeFont.metadata.Unicode.widths.
A workaround for this is by giving option parameter to splitTextToSize() function with fontName which recognized by jsPDF library. So it could be something like :
var long_text = "Something realy long text or paragrahps is here"
var width_of_area = 300;
doc.splitTextToSize(long_text, width_of_area, { fontName: 'Sans' });

Notice that i put option value with :  { fontName: 'Sans' }.
Wonder if there any proper solution compared to this hacky ways. Maybe something that can produce properly an array of font character widths from the actual custom font defined, rather than giving another font character widths.

I'm using openfont.js to read a font and look up every character width and character pair kerning. Might be overkill. 😬
Below is how I'm getting widths and kerning after already using openfont.js to read the font.
export function getTextWidthsAndKerning(text, font) {
  const chars = uniq(text);
  const widths = {};
  const kerning = {};

  for (let i = 0; i < chars.length; i += 1) {
    const char = chars[i];
    widths[char] = get(font.charToGlyph(char), 'advanceWidth');

    kerning[char] = {};
    for (let j = 0; j < chars.length; j += 1) {
      const kerningChar = chars[j];
      kerning[char][kerningChar] = font.getKerningValue(
        font.charToGlyph(char),
        font.charToGlyph(kerningChar),
      );
    }
  }

  return { widths, kerning };
}


[Docs] Update Meshery Architecture Page

Current State

Figures used in the following sections are outdated and are in need to be updated
https://docs.meshery.io/concepts/architecture
https://docs.meshery.io/guides/upgrade

Does not include the Docker Extension Architecture description in  architecture page

Desired State

Update the figures with the help of the Meshery Architecture Slide
Add Docker Extension architecture sub section

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I would like to work on this issue .
Could you please give me the link of those figma files which are required to be updated.

@Ghat0tkach sure go ahead 🚀
You can find the slides (Meshery Architecture ) in the community drive, download them as SVG and replace them with current ones

Since you don't have access, I'll take care of this issue, thanks for volunteering though 😄

Oh no worries , Will look for other issues . Thank you

@thisiskaransgit I would like to work on this


Cloud Migration Manager [v1.6.2] Company Selection

In version 1.6.2, there is no  Select Companies to Migrate option in the Cloud Migration Management list page (4003). This option exists solely within the Assisted Setup, Set up Cloud Migration.
Company selection only appears after waiting a long period of time (5+ minutes) for the MS Integration to query the DYNAMICS based on the SQL Connection String and integration runtime.
If users fail to wait for the assisted setup to populate the companies, then users will see an error on the Cloud Migration Management page that no company has been selected for migration with no easy option to select companies outside of the slow Set up Cloud Migration utility.
D365BC Information:
Version: US Business Central 16.2 (Platform 16.0.13440.14360 + Application 16.2.13509.13619)

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

ID: e8064280-eae4-9181-8ef2-c591fd357f7d
Version Independent ID: 952f5564-d7dc-4124-fe6f-724a00beddd2
Content: Manage the cloud migration - Business Central
Content Source: dev-itpro/administration/migration-management.md
Service: dynamics365-business-central
GitHub Login: @bmeier94
Microsoft Alias: bmeier

Thank you for your feedback! This will be fixed in the next doc update.


Close the application on escape key press

It would be cool to emulate the spotlight functionality where the application is closed when escape key is pressed.
Thanks again for putting this example together!

Implemented; see commit 660f714.

Here is what I mean about the focus #4.
At the start the focus is on the text box, then I open the app (Command + K) and the focus is on the app. I press escape, the app is closed but the focus is not back on the text box.
https://user-images.githubusercontent.com/37532050/218580307-852038d3-9061-499c-8d38-e919032bb4c3.mov

I'll close this issue since the  to close window feature implemented.

@elanzini your issue has been fixed in this commit 78c5497.

I have updated the <Esc> implementation to hide the spotlight window when focused. See commit 7e0711c.


Crank gas cost

Problem: crank is currently run without direct incentive mechanism, different ecosystem members are running it pro bono. A more sustainable system would be preferable.
proposed solution: event queue stores extra lamports, cancel/take initiator deposits 1000 lamports per out event (should not require additional writeable accounts) crank collects 1000 lamports per event.
problems: it would require all integrations to pass in either the signer account as writeable or the additional lamports manually in another instruction

Alternative solution: a portion of the referrer rebate goes toward cranker

have cranker pass in their own open order account into consume_events
credit half of the referrer fee to cranker open order account: https://github.com/openbook-dex/program/blob/master/dex/src/state.rs#L3035
cranker passes themself as their referrer in settle funds

Benefits:

only crankers need to modify their client code
no additional fee for users

Cons:

rebates is not denominated in SOL

If I understand Max's suggestion correctly then the difference between the two approaches is that placing/cancelling orders will incur a fee instead of waiting for a match to collect a fee. I'm honestly not sure which is better. I think there's already an incentive for MMs to crank or else they can't quote. I would rather not charge MMs for placing orders and providing liquidity, charging them when there's a fill (fees) make more sense because they should be making PnL then and the platform gets its cut.

Problem: crank is currently run without direct incentive mechanism, different ecosystem members are running it pro bono. A more sustainable system would be preferable.
proposed solution: event queue stores extra lamports, cancel/take initiator deposits 1000 lamports per out event (should not require additional writeable accounts) crank collects 1000 lamports per event.
problems: it would require all integrations to pass in either the signer account as writeable or the additional lamports manually in another instruction

Not familiar enough, but what happens when a bid/ask is matched? Both accounts are closed?
If so, the rent can be the crank incentive, it adds up for high volume.

Problem: crank is currently run without direct incentive mechanism, different ecosystem members are running it pro bono. A more sustainable system would be preferable.
proposed solution: event queue stores extra lamports, cancel/take initiator deposits 1000 lamports per out event (should not require additional writeable accounts) crank collects 1000 lamports per event.
problems: it would require all integrations to pass in either the signer account as writeable or the additional lamports manually in another instruction

I think something similar to this while completely eliminating all other fees would be best, less opinions on how the orderbook should be used and more about ensuring this can be cranked affordably. Factoring in whatever server costs are needed (that scale log in fiat) and txn costs (that scale linearly in SOL) I don't see a reason to reward others as anyone can charge fees separately (dapps/MMs/etc.)

I think the whole thing boils down to can we afford to break the ABI.
The SOL fee approach has a sound economic model imo, and it works for every market regardless of the value of the underlying asset. It's also more friendly to both takers and makers, because it eats less into the existing fees collected.
Chrises approach is more pragmatic, but it eats into the existing fees more, potentially prompting taker fees to be increased (40 bps suggested), and the economic model only works in markets where tokens are sufficiently valuable (maybe not a bad thing). The benefit of this approach is obvs that it preserves the ABI.

If I understand Max's suggestion correctly then the difference between the two approaches is that placing/cancelling orders will incur a fee instead of waiting for a match to collect a fee. I'm honestly not sure which is better. I think there's already an incentive for MMs to crank or else they can't quote. I would rather not charge MMs for placing orders and providing liquidity, charging them when there's a fill (fees) make more sense because they should be making PnL then and the platform gets its cut, so I'd lean towards the second approach by Crisp

My assumptions is that the cost can be easily circumvented by cranking in the same tx as when crank events are created, further if done correctly cranking can actually pay tx costs for both takers and makers.

different ecosystem members are running it pro bono

My understanding is that Openbook is divided between permissioned and permissionless markets. The permissioned markets can only be cranked by the individual teams who created them, and so I assume the problem statement here applies to permissionless markets?
A few months ago, we started working on a program (linked below) to auto-crank Serum markets. It only handles the consume_events call today, but we should be able to extend it to settle_funds and other "automatable" instructions in the future. Our original vision was to allow teams who run permissioned markets to save on server costs and decentralize their operations, shifting the crank authority from a dedicated server to a permissionless network of RPC nodes.
I believe this solution could be similar (in spirit at least) to Max's proposal. Ironically, the Clockwork automation fee starts at a minimum 1000 lamports per instruction as proposed. This can be bumped higher to prioritize an important thread with the workernet, like cranking an Openbook market. I don't see any reason to change the Openbook interfaces or introduce a new user fee. To keep a market cranking, a community simply needs to fund the thread account associated with the market (this is permissionless as anyone can transfer SOL to another account).
There remain some challenges to be solved before I would recommend this in production though... On localnet, things obviously look great; but we've yet to see a per-slot time resolution for successful serial transactions on mainnet. We also need to test against a mainnet market where other MMs are actively cranking and see how it fails. I can post back here when things are looking more suitable for realistic mainnet usage. If you get a chance to read through the code, we would super appreciate any feedback you have!
https://github.com/clockwork-xyz/examples/tree/main/serum_crank

. To keep a market cranking, a community simply needs to fund the thread account associated with the market (this is permissionless as anyone can transfer SOL to another account).

This is definitely an improvement in terms of ease of access from what's happening right now, but only a few communities are actually organized well enough to pay for cranking. The goal of a solution inside the program should be to have the bill paid up by the traders rather than some donor.


Initial commit

Purpose

...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

I assume this was an accidental PR, closing.

Indeed, thank you for your understanding
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Pamela Fox @.>
Sent: Wednesday, October 25, 2023 12:44:43 PM
To: Azure-Samples/azure-search-openai-demo @.>
Cc: Kyriacos Antoniades @.>; Author @.>
Subject: Re: [Azure-Samples/azure-search-openai-demo] Initial commit (PR #858)
I assume this was an accidental PR, closing.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure-Samples/azure-search-openai-demo/pull/858#issuecomment-1778991482, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APLA3FY4AKZLEVVZUSP3OC3YBDURXAVCNFSM6AAAAAA6PDMUZ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZYHE4TCNBYGI.
You are receiving this because you authored the thread.Message ID: @.***>


Fixed #16470 -- Allowed FileResponse to auto-set some Content headers.

Jon, very good catch. I also updated the PR with a separate set_headers() method so subclasses may easily update the headers attribution.
The Windows test failure is not my PR's fault :-(

While the change explicitly handles .tar.gz, I think the gzip issue will apply to all gzipped files, not only .tar.gz files. For example, if a project uses FileResponse with a file named words.txt.gz, I think it is reasonable for the user to receive a gzipped text file (.txt.gz) not an uncompressed text file. IMO, FileResponse should never set the Content-Encoding header by default. Use cases like django/views/static.py may need to continue setting Content-Encoding on their own.
I think this will be a problem for all encodings recognized by Python, not only gzip. Here is the default list:
https://github.com/python/cpython/blob/b1dc07509f78b354e83f5f4a902f1ff80c7bb05d/Lib/mimetypes.py#L390-L395
A potential solution would be to use a custom instance of mimetypes.MimeTypes that has an empty encodings_map attribute.

Hopefully addressed that in that new version.

Claude, please check my "Edits" commit before squashing. The only changes in the tests are some added response.close() calls to avoid ResourceWarning -- perhaps it could be done differently.

Your edits are fine, many thanks for the thorough review.


Monk nodes seems to be stuck at block 500015

My 3 nodes (Windows GUI Wallet, and 2 CLI ubuntu nodes) are stuck at block 500015.
Decenomy explorer is also stuck at the same block.
Waiting for an update ;)

devs have frozen chain for a couple of hours to thwart hacker


Revert "Switch to registry for tasks"

Reverts inventree/InvenTree#3790
Reason: scheduled tasks cannot be registered using the function reference - only using 'dotted string' notation:

@matmair we will need a new PR to perform daily backups as the task scheduling has been removed for the run_backup task


Updating the CODEOWNERS file

Description
Updating the CODEOWNERS file to reflect maintainers list
Issues Resolved
Closes #421
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Couldn't find user @yizheliu-amazon listed on the maintainers.md file. @ohltyler @ylwu-amzn Please confirm if we need to add that user. CODEOWNERS file is not green yet due to this issue.

Codecov Report

Merging #474 (3ea7342) into main (f0e3016) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #474   +/-   ##
=======================================
  Coverage   51.98%   51.98%
=======================================
  Files         149      149
  Lines        5030     5030
  Branches      983      967   -16
=======================================
  Hits         2615     2615
  Misses       2156     2156
  Partials      259      259

Couldn't find user @yizheliu-amazon listed on the maintainers.md file. @ohltyler @ylwu-amzn Please confirm if we need to add that user. CODEOWNERS file is not green yet due to this issue.

We don't need to add that user - you can remove. thanks!

Actually - looks like this is covered in #476. I will close. @bbarani reopen if needed, thanks.


Backward test for acos and xlogy

#6443
CI runs : https://github.com/tenstorrent-metal/tt-metal/actions/runs/8392058236

Did we not need a test change for this update?
https://github.com/tenstorrent-metal/tt-metal/pull/6679 Once this PR is merged will submit the rest of the PR that includes test file changes

Going forward, we really need to have a test included with the underlying change.  I'm approving but this should be the exception.

Going forward, we really need to have a test included with the underlying change. I'm approving but this should be the exception.

@eyonland I have updated the tests in this PR as the depending PR was merged into the main. Please review those changes as well


Add bootstrap test on FreeBSD

This should fail until the issues exposed in #159 are fixed.

@Theodus do you want to update to FreeBSD 13 or should I?

@Theodus do you want to update to FreeBSD 13 or should I?

I'll give it a shot.

Hi @SeanTAllen,
The changelog - changed label was added to this pull request; all PRs with a changelog label need to have release notes included as part of the PR. If you haven't added release notes already, please do.
Release notes are added by creating a uniquely named file in the .release-notes directory. We suggest you call the file 161.md to match the number of this pull request.
The basic format of the release notes (using markdown) should be:
## Title

End user description of changes, why it's important,
problems it solves etc.

If a breaking change, make sure to include 1 or more
examples what code would look like prior to this change
and how to update it to work after this change.

Thanks.

@Theodus as this changes the supported FreeBSD, I changed the PR title and added a changelog. It will need some release notes as well. I'll add the release notes shortly.

@SeanTAllen this should be good to go now. Feel free to merge if you agree with the changes I made.


ddclient: incorrect permissions on file /usr/local/etc/ddclient.json which contains my username/password

[x ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x ] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
The config file for ddclient /usr/local/etc/ddclient.conf contains my credentials to my dyndns provider. It has the correct permissions of u=rw with g and o having no permissions ensuring that it cannot be read except by root.
The file /usr/local/etc/ddclient.json also contains my credentials to my dyndns provider but this file has read permissions for any user.
I have verified that non root users can read the contents of /usr/local/etc/ddclient.json
To Reproduce
Login on the command line, sudo su - to root and then execute ls -l /usr/local/etc/ddclient.*
# ls -l `/usr/local/etc/ddclient.*
-rw-------  1 root  wheel    343 Nov 26 12:49 /usr/local/etc/ddclient.conf
-rw-------  1 root  wheel  10219 Nov 22 04:21 /usr/local/etc/ddclient.conf.sample
-rw-r--r--  1 root  wheel    796 Nov 26 12:49 /usr/local/etc/ddclient.json

Expected behavior
I expect to see:
-rw-------  1 root  wheel    796 Nov 26 12:49 /usr/local/etc/ddclient.json

Describe alternatives you considered
n/a
Screenshots
n/a
Relevant log files
n/a
Additional context
n/a
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 23.7.9-amd64
FreeBSD 13.2-RELEASE-p5
OpenSSL 1.1.1w

It's not so much incorrect but just different. Ddclient did enforce it I think to in order to start it was added. If you don't trust your system administrators you have a different problem. ;) I can see how this can be addressed structurally, but it's not super important.