remove limits.cpu from values.yaml

Please provide a description of this PR:
When Istio chart is imported as dependency chart, it is unable to unset limits.cpu. The change is really about supporting wider adoption. Keeping original values commented out for referencing and can be restored if desired.
If the change is accepted, feel free to let me know necessary change to increment chart version.

Hi @shippo-eric. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@hanxiaop thanks for the comment.
It is my belief that setting cpu limits in Kubernetes an anti-pattern.  I am sure there are situations where cpu limit is desired.  But it is also a wide known issue that cpu limit triggers unwanted throttling and leads to artificial higher latency.  The purpose of the PR is simply making that setting optional when importing Istio helm chart as dependency chart.
Here are a few blogs for reference:

https://home.robusta.dev/blog/stop-using-cpu-limits
https://maxanuj.medium.com/cpu-limits-cpu-request-and-aggressive-throttling-in-kubernetes-d0d74e4d64ff
https://erickhun.com/posts/kubernetes-faster-services-no-cpu-limits/
https://www.numeratorengineering.com/requests-are-all-you-need-cpu-limits-and-throttling-in-kubernetes/

You should be able to unset it by zero value:
istiod:
  global:
    proxy:
      resources:
        limits:
          cpu: "0"
        requests:
          cpu: "0"

  pilot:
    resources:
      limits:
        cpu: "0"
      requests:
        cpu: "0"

For Chart defined as:
dependencies:
- name: istiod
  version: "1.20.0"
  repository: "https://istio-release.storage.googleapis.com/charts"

@kirecek Thanks for the suggestion.  Your approach that requires not setting both cpu.request and cpu.limits is a nice work around.  Since we don't normally autoscale our pods based on Istio-proxy sidecar container, I agree that this is a good compromise. On the other hand, this workaround also removes the ability to autoscale ingressgateway, something that is critical to production operations.
I have updated the PR to only comment out gateway cpu.limits, and restored Istio-proxy sidecar original cpu.limits. Thanks.

Hi @hanxiaop Do you have other concerns with the PR? Or is there something else I can help with explaining the request?


Columns are added as full-width columns to the pages with one column

For example: Wall page
https://www.boonex.com/forums/topic/add-column-to-wall-timeline-page.htm

can't reproduce the problem, please describe it in more details with screenshots.


Avoid delay when entering edit mode

I want to enter Jupyter edit mode with i, similar to Vim, because Enter is hard to reach.
I remapped it like this:
        {
            "command": "notebook:enter-edit-mode",
            "keys": [
                // "Enter"
                "I"
            ],
            "selector": ".jp-Notebook:focus",
            "args": {}
        },

This works. However, the cell becomes editable only after a significant delay (3 sec?). Enter still works and does so instantly. Also, bindings like j work fine. So why is there a delay? Does it clash with some other keybinding and waits for the second character?

Hi @nkaretnikov what JupyterLab and jupyterlab-vim versions are you using?

I,  I is mapped to interrupt kernel in notebooks (via kernelmenu:interrupt in lab 4.x). Does overriding this shortcut to something else helps?

Tried commenting I, I out - didn't work.
JupyterLab - Version 4.0.8.
axlair-jupyterlab-vim - 4.1.0 (installed via plugins menu).

I think commenting it out does not disable the shortcut itself. Changing it to a different one would do.

Tried that as well - same delay.

Remapping the key binding doesn't disable the default one. This exact remapping was attempted by someone else, leading to https://github.com/jupyterlab/jupyterlab/issues/15038, and this would need to be solved upstream.


kubeconfig secret name configurable via cr

This was meant to address #4

Added a kubeconfigSecretName field to the CR to specify the name of the secret containing the kubeconfig to be used by the pipeline
Added some validation (some of it can be removed looking for comments)
The same pattern will apply to the other two secrets i.e. github api and pyxis (Will do #8 and #9 based on feedback on this one)
The logging is inconsistent left it to be addressed in #15

Per discussion in slack we should be using a service account for this.

Question about usage of client-go (which introduces a new dependency)

removed 👍


airflow: increase reliability of integration tests

There are two fixes in this PR.
First, retrying library catches SystemExit exception raised by sys.exit(1) and retries execution. This is not what we want, so I replaced this with returning boolean, and checking in from caller.
Second, increase dag timeout - mostly for GreatExpectation dag. It's importing tons of stuff - so much that it sometimes can't initialize in 60 seconds.
Checklist

[ ] You've signed-off your work
[ ] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)

Example GE error:
airflow_worker_1     | [2022-02-15 17:02:09,181: ERROR/ForkPoolWorker-15] Process timed out, PID: 81
airflow_worker_1     | [2022-02-15 17:02:09,208: ERROR/ForkPoolWorker-15] Failed to import: /opt/airflow/dags/greatexpectations_dag.py
airflow_worker_1     | Traceback (most recent call last):
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/models/dagbag.py", line 326, in _load_modules_from_file
airflow_worker_1     |     loader.exec_module(new_module)
airflow_worker_1     |   File "<frozen importlib._bootstrap_external>", line 678, in exec_module
airflow_worker_1     |   File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
airflow_worker_1     |   File "/opt/airflow/dags/greatexpectations_dag.py", line 1, in <module>
airflow_worker_1     |     from great_expectations_provider.operators.great_expectations import GreatExpectationsOperator
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations_provider/operators/great_expectations.py", line 34, in <module>
airflow_worker_1     |     import great_expectations as ge
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/__init__.py", line 7, in <module>
airflow_worker_1     |     from great_expectations.data_context import DataContext
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/data_context/__init__.py", line 1, in <module>
airflow_worker_1     |     from .data_context import BaseDataContext, DataContext, ExplorerDataContext
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/data_context/data_context.py", line 50, in <module>
airflow_worker_1     |     from great_expectations.core.usage_statistics.usage_statistics import (
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/core/usage_statistics/usage_statistics.py", line 22, in <module>
airflow_worker_1     |     from great_expectations.core.usage_statistics.anonymizers.data_docs_site_anonymizer import (
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/core/usage_statistics/anonymizers/data_docs_site_anonymizer.py", line 2, in <module>
airflow_worker_1     |     from great_expectations.core.usage_statistics.anonymizers.site_builder_anonymizer import (
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/core/usage_statistics/anonymizers/site_builder_anonymizer.py", line 2, in <module>
airflow_worker_1     |     from great_expectations.render.renderer.site_builder import (
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/render/renderer/__init__.py", line 2, in <module>
airflow_worker_1     |     from .column_section_renderer import (
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/render/renderer/column_section_renderer.py", line 15, in <module>
airflow_worker_1     |     from great_expectations.expectations.core.expect_column_kl_divergence_to_be_less_than import (
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/expectations/core/__init__.py", line 1, in <module>
airflow_worker_1     |     from .expect_column_distinct_values_to_be_in_set import (
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/expectations/core/expect_column_distinct_values_to_be_in_set.py", line 19, in <module>
airflow_worker_1     |     from ..expectation import ColumnExpectation, InvalidExpectationConfigurationError
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/expectations/expectation.py", line 39, in <module>
airflow_worker_1     |     from great_expectations.self_check.util import (
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/great_expectations/self_check/util.py", line 177, in <module>
airflow_worker_1     |     import sqlalchemy.dialects.mysql as mysqltypes
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/sqlalchemy/dialects/mysql/__init__.py", line 8, in <module>
airflow_worker_1     |     from . import base  # noqa
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/sqlalchemy/dialects/mysql/base.py", line 849, in <module>
airflow_worker_1     |     from . import reflection as _reflection
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/sqlalchemy/dialects/mysql/reflection.py", line 10, in <module>
airflow_worker_1     |     from .enumerated import _EnumeratedValues
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/sqlalchemy/dialects/mysql/enumerated.py", line 10, in <module>
airflow_worker_1     |     from .types import _StringType
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/sqlalchemy/dialects/mysql/types.py", line 537, in <module>
airflow_worker_1     |     class TINYTEXT(_StringType):
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/sqlalchemy/sql/visitors.py", line 66, in __init__
airflow_worker_1     |     def __init__(cls, clsname, bases, clsdict):
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/utils/timeout.py", line 37, in handle_timeout
airflow_worker_1     |     raise AirflowTaskTimeout(self.error_message)
airflow_worker_1     | airflow.exceptions.AirflowTaskTimeout: DagBag import timeout for /opt/airflow/dags/greatexpectations_dag.py after 60.0s, PID: 81
airflow_worker_1     | [2022-02-15 17:02:09,295: ERROR/ForkPoolWorker-15] Failed to execute task dag_id could not be found: great_expectations_validation. Either the dag did not exist or it failed to parse..
airflow_worker_1     | Traceback (most recent call last):
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/executors/celery_executor.py", line 117, in _execute_in_fork
airflow_worker_1     |     args.func(args)
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/cli/cli_parser.py", line 48, in command
airflow_worker_1     |     return func(*args, **kwargs)
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/utils/cli.py", line 91, in wrapper
airflow_worker_1     |     return f(*args, **kwargs)
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/cli/commands/task_command.py", line 220, in task_run
airflow_worker_1     |     dag = get_dag(args.subdir, args.dag_id)
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/utils/cli.py", line 194, in get_dag
airflow_worker_1     |     'parse.'.format(dag_id)
airflow_worker_1     | airflow.exceptions.AirflowException: dag_id could not be found: great_expectations_validation. Either the dag did not exist or it failed to parse.
airflow_worker_1     | [2022-02-15 17:02:09,430: ERROR/ForkPoolWorker-15] Task airflow.executors.celery_executor.execute_command[d3c6668b-2a5b-42e5-8da6-f9c84fcaacb5] raised unexpected: AirflowException('Celery command failed on host: d8c5bcc7f9b4',)
airflow_worker_1     | Traceback (most recent call last):
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/celery/app/trace.py", line 412, in trace_task
airflow_worker_1     |     R = retval = fun(*args, **kwargs)
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/celery/app/trace.py", line 704, in __protected_call__
airflow_worker_1     |     return self.run(*args, **kwargs)
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/executors/celery_executor.py", line 88, in execute_command
airflow_worker_1     |     _execute_in_fork(command_to_exec)
airflow_worker_1     |   File "/home/airflow/.local/lib/python3.6/site-packages/airflow/executors/celery_executor.py", line 99, in _execute_in_fork
airflow_worker_1     |     raise AirflowException('Celery command failed on host: ' + get_hostname())
airflow_worker_1     | airflow.exceptions.AirflowException: Celery command failed on host: d8c5bcc7f9b4


"new ScriptElement()..async = false" incorrectly rejected in async method

Dart SDK Version (dart --version)
v2_1_0_dev_7_1
Whether you are using Windows, MacOSX, or Linux (if applicable)
Linux
Whether you are using Chrome, Safari, Firefox, Edge (if applicable)
N/A

import 'dart:html';
import 'package:test/test.dart';

main() {
  setUp(() async {
    new ScriptElement()..type='text/javascript'..async=false..src='foo.js';
  });
}

gives the error
[error] The keywords 'async', 'await', and 'yield' can't be used as identifiers in an asynchronous or generator function. (.../async_test.dart, line 6, col 50)

Only await and yield are reserved in Dart 2 async functions, not async itself.

This is now accepted.


Fixed broken link for doc search on ios

This fixes the link no working on search results on ios.
Seems like ios's focus-within event doesn't work quite right so I changed the css so it doesn't use it.

Still not quite working for me on Safari. It works as expected on Chrome.
$ roc version
roc built from commit 5e28756b1e, committed at 2024-11-04 19:05:19 UTC

@jfkonecn I rebuilt roc from this branch and tried this again. It's working in safari. I must have messed something up when I tested it the first time.

I'm glad because I was out of ideas on how to fix it. 😆


Hillshade with raster-dem

Does this package support hillshade using raster-dem tiles? @greensopinion

no, raster-dem is not supported

Hello @greensopinion
First things first : thank you SO much for all the work put in this vector tiles pub ;)
I'm using MapTiler and I'd like to show hillshades. I found this issue and have two questions :

raster-dem isn't supported but will it be in future? do you have plans/need help for something ?
If raster-dem isn't in roadmap, how would you display hillshades on a maptiler vector map ?

Thanks again, have a nice day :)

Hugo, you're very welcome!
I haven't had a need for raster DEM so far and haven't made any plans to support it. Supporting raster DEM is potentially a lot of effort. If you're interested in looking into it, feel free. It seems like something that would fall within the scope of a vector rendering library.
Alternatively, you may want to find a tileset that has hillshade represented as a vector layer. This is already supported today, which you can see in the examples repository - both Thunderforest and Mapbox Outdoor themes use this approach.


Pin Code / TouchID / FaceID support for accessing the app

Please add Pin Code / TouchID / FaceID support for accessing the app.
Many different usecases for this and its neat to have adult apps like this secured in this manner.

I agree. Could we have this feature please?


Make fixes for issue #213: negative weights

This is to check for negative weights and to fix a case when the automatic weights could be negative.
In addition, I think I found a bug in a test where an or should have been an and.
See if you agree.

If one person checks this and thinks it is OK, then I will accept the PR to the prerelease branch.

@cabouman Yes "or" should be "and".
The changes look good to me.


Create F2_13.c

The application you will be writing for is intended to run on a small, inexpensive embedded device that does not have a floating point unit. This means that the processor can only do integer arithmetic. However, the device does have a display and needs to display the result of floating point arithmetic to the user.  Because this is a small inexpensive processor there is no support for strings or any mathematical library functions. You must write all the code yourselves performing all the floating point math with integers only and no strings! Character arrays, or C strings, can't tell you their size but they do end with a '\0' character.
The requirements for the code are to write one of the functions presented below. All the functions return a boolean value indicating the success or failure of the operation. Improper data is the most likely cause for a function to fail. You may add your own additional helper functions (I had at least a half dozen) but you must include them for the review. Do not change the interface to these functions.
These two functions take as parameters two sets of mantissa and characteristic and a char array to hold the result of the arithmetic operations. The result of the add or subtract should be converted into char's and placed in the result array. The array must end with a '\0'. The 'len' parameter tells how many characters can be placed on the result array. For these functions to return true you must at a minimum store the characteristic of the result. If the result is a non-integer place as many of the digits as will fit in the result after a decimal point.
bool add(int c1, int n1, int d1, int c2, int n2, int d2, char result[], int len);
bool subtract(int c1, int n1, int d1, int c2, int n2, int d2, char result[], int len);
...
...
char answer[100];
int c1, n1, d1;
int c2, n2, d2;

c1 = 1;
n1 = 1;
d1 = 2;

c2 = 2;
n2 = 2;
d2 = 3;

//if the C string could hold at least the characteristic if(add(c1, n1, d1, c2, n2, d2, answer, 100))
{
    //display string with answer 4.166666666...
}
else
{
    //display error message
}
'''

I am working on this pull request number #13

This pull request implements functions for performing floating-point arithmetic on an embedded device that only supports integer arithmetic. This is achieved by breaking down floating-point numbers into integer parts (characteristic and mantissa) and handling the arithmetic operations manually. The functions implemented include add and subtract, which handle the addition and subtraction of these numbers respectively

Pros:
The clear structure and modularity of the code make it easy to extend or modify for future requirements, such as supporting different numeric formats or additional arithmetic operations.
The use of integer arithmetic allows for potential optimizations specific to the target hardware, such as using fixed-point arithmetic for better performance.
The implementation includes basic error checking and validation, reducing the risk of runtime errors.

Potential improvement:
Adding a loop or logic to ensure that the length of the result does not exceed the provided buffer length (len) would enhance robustness.
Include scenarios where the fractional part is zero or very close to the denominator.
Instead of returning a boolean value for success or failure, consider returning error codes


Disallow dotted helpers/modifiers

We don't actually use this "feature" in Ember and it is making a lot of the lookups more complicated than they need to be.
With this change, {{foo.bar}} is statically known to be a get, as opposed to an unknown before, saving us a hasHelper lookup at runtime.
Closes emberjs/ember.js#13404.

Seems good to me, but how will this get updated for 0.20 branch?

@rwjblue at this point I think I'll just wait for the master branch to get fixed, then re-do the Work there. Doesn't seem worth it to land this for now


Enterprise PRT deprecated by New Edge

[Enter feedback here]
Hello Guys,
I think this document should be updated. Indeed today is no more possible to use device conditional access policies, device compliance policies, etc with ADFS onprem federated applications because the new Edge browser doesn't support the enterprise prt token.
This document remains true only for Internet Explorer.
Thanks
<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 74d09c81-d205-dbd0-ea6a-aaf469eadf5c
Version Independent ID: 458db848-1b30-1115-8814-3a639a57c70b
Content: Configure Device-based Conditional Access on-Premises
Content Source: WindowsServerDocs/identity/ad-fs/operations/Configure-Device-Based-Conditional-Access-on-Premises.md
Product: windows-server
Technology: ad-fs
GitHub Login: @billmath
Microsoft Alias: billmath

Identical ticket content to, but different page from, issue ticket #5486 (for reference).

Identical ticket content to, but different page from, issue ticket #5486 (for reference).

This sentence seems to be unclear, so I will replace it with only the word "ref.".


Update jsoniter-scala-core, ... to 2.4.0

Updates

com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core
com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-macros

from 2.2.6 to 2.4.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala" } ]

labels: library-update, semver-minor

Superseded by https://github.com/almond-sh/almond/pull/596.


Add misc package to setup.py

This fixes the flatten benchmarks, which weren't working due to rather confusing import errors.

I've added this change to https://github.com/HIPS/autograd/pull/299.


Unable to write elastic search data

Issue description
Unable to write the data using ESStorage using multi index dynamic write.
I am able to index and write the data without dynamic write.
So when I say es.resource=index_name/some_type which is static, I am able to write it.
if I change it to es.resource.write=index_name/{some_field_name} it doesn't work. It just says Cannot detect ES version , which is misleading too.
Description
Steps to reproduce
Code:
register /tmp/elasticsearch-hadoop-pig-5.0.0.jar ;
define EsStorage org.elasticsearch.hadoop.pig.EsStorage('es.resource={name}/some', 'es.nodes=some server', 'es.port=4080', 'es.http.timeout=2m',  'es_net_proxy_http_host=some_proxy_host', 'es_net_proxy_http_port=4080',  'es.http.retries=2', 'es_batch_write_retry_wait=100s', 'es_batch_write_refresh=false');
layout = load '$LAYOUT_TABLE' using PigStorage(',') as (layout_id, name);
layout = foreach layout generate
REPLACE(layout_id,'"','') as layout_id,
REPLACE(name,'"','') as name;
layout = foreach layout generate CurrentTime() as timestamp, (long)layout_id as layout, (chararray)name as name;
STORE layout into '{name}/some' USING EsStorage();
Test/code snippet

Strack trace:
16/11/10 20:41:25 INFO tez.TezJob: DAG Status: status=FAILED, progress=TotalTasks: 1 Succeeded: 0 Running: 0 Failed: 1 Killed: 0 FailedTaskAttempts: 4, diagnostics=Vertex failed, vertexName=scope-23, vertexId=vertex_1478766437060_121497_1_00, diagnostics=[Task failed, taskId=task_1478766437060_121497_1_00_000000, diagnostics=[TaskAttempt 0 failed, info=[Error: Failure while running task:org.elasticsearch.hadoop.EsHadoopIllegalArgumentException: Cannot detect ES version - typically this happens if the network/Elasticsearch cluster is not accessible or when targeting a WAN/Cloud instance without the proper setting 'es.nodes.wan.only'
at org.elasticsearch.hadoop.rest.InitializationUtils.discoverEsVersion(InitializationUtils.java:247)
at org.elasticsearch.hadoop.rest.RestService.createWriter(RestService.java:545)
at org.elasticsearch.hadoop.mr.EsOutputFormat$EsRecordWriter.init(EsOutputFormat.java:173)
at org.elasticsearch.hadoop.mr.EsOutputFormat$EsRecordWriter.write(EsOutputFormat.java:149)
at org.elasticsearch.hadoop.pig.EsStorage.putNext(EsStorage.java:189)
at org.apache.pig.backend.hadoop.executionengine.mapReduceLayer.PigOutputFormat$PigRecordWriter.write(PigOutputFormat.java:136)
at org.apache.pig.backend.hadoop.executionengine.mapReduceLayer.PigOutputFormat$PigRecordWriter.write(PigOutputFormat.java:95)
at org.apache.tez.mapreduce.output.MROutput$1.write(MROutput.java:503)
at org.apache.pig.backend.hadoop.executionengine.tez.plan.operator.POStoreTez.getNextTuple(POStoreTez.java:129)
at org.apache.pig.backend.hadoop.executionengine.tez.runtime.PigProcessor.runPipeline(PigProcessor.java:378)
at org.apache.pig.backend.hadoop.executionengine.tez.runtime.PigProcessor.run(PigProcessor.java:243)
Stack trace goes here

Version Info
OS:         :
JVM         :
Hadoop/Spark:
ES-Hadoop   :  5.0
ES          :

Could you increase the logging level to TRACE and add the logs here? Thanks.

For some reason I was unable to get the logs when I run it on grid, so I tried running it locally and see another issues, here are the logs, if I turn off auto index create - https://gist.github.com/jayesh4192/fbbd3e7a5280dd12569831993483ba56

My use case is store the data in multiple indices based on a filtering criteria, for ex: if I have a table which contains dep_name and employee records and I need to store the employee data per department. For example - computer_science/students, electrical/students.
Is it possible to create multiple indexes on the fly ? here is my script
register /tmp/elasticsearch-hadoop-pig-5.0.0.jar ;
define EsStorage org.elasticsearch.hadoop.pig.EsStorage('es.resource.write={name}/publisher', 'es.nodes=host', 'es.port=4080', 'es.http.timeout=2m',  'es_net_proxy_http_host=httpproxy', 'es_net_proxy_http_port=4080',  'es.http.retries=2', 'es.index.auto.create=true');
layout = load '$LAYOUT_TABLE' using PigStorage(',') as (layout_id, name);
layout = foreach layout generate
REPLACE(layout_id,'"','') as layout_id,
REPLACE(name,'"','') as name;
layout = foreach layout generate CurrentTime() as timestamp, (long)layout_id as layout, (chararray)name as name;
STORE layout into '{name}/publisher' USING EsStorage();
here is the gist with the error - https://gist.github.com/jayesh4192/011e9c8683224e0944fd103125602df7

@jayesh4192 My guess in this case is that you have index creation disabled in Elasticsearch when a document is indexed. When using a pattern for your resource write you will need to set action.auto_create_index to true on every node in your cluster.


Define CoAP as a relative schema

Current URL spec supports relative URLs with any schema. galimatias v0.3.0 will support that, https://github.com/smola/galimatias/issues/56


Specified path in yaml gets URL encoded

Hello,
when an API specification defines path like this:
/?unused:

the generated client does not preserve the specified URL and encodes it to the path to /%3Funused
The request log:
[org.apa.htt.imp.exe.MainClientExec] (executor-thread-1) Executing request POST /%3Funused HTTP/1.1
[org.apa.htt.headers] (executor-thread-1) http-outgoing-0 >> POST /%3Funused HTTP/1.1
[org.apa.htt.wire] (executor-thread-1) http-outgoing-0 >> "POST /%3Funused HTTP/1.1[\r][\n]"

Please change it to use the original spec without modifying or add some option to disable it for given API file or a path
To reproduce use tests in the attached demo
demo.tar.gz
Thank you very much!

Hi @ennishol thank you for this issue!
If you want, open a pull request to solve this issue, what do you think?

@ennishol, this is not an issue in this project. The path segment per RFC 3986, can't have the ? character, hence the engine is encoding it. So it's working as it's meant to be. See here a more detailed response: https://stackoverflow.com/a/4669755/1354502
In your case, the URI interpreter will see the first ? in the path and it will break it considering unused to be a querystring parameter with an empty var. So it will break your API anyway if we not encode the ? char.
My suggestion: remove this path from your OpenAPI file or change it to unused. It smells a workaround anyway.

@ricardozanini Yes. that is what I'm doing. But I wrongly assumed it would be a problem with the open-api generator and wanted to report a bug/improvement. Sorry for bothering you

No worries! We are happy to have you here and using our project 👍


Move or Resize Windows with mouse/touchpad is not working

I have berry-git git+e5bcb3-1 installed, and i also have this in the autostart file
berryc move_mask       "mod4"
berryc move_button    1
berryc resize_mask      "mod4"
berryc resize_button    3

Still, I can't move or resize windows with touchpad (using a laptop)
https://user-images.githubusercontent.com/26059688/211999607-0659eb95-6fd8-4d32-8486-8ac76312373d.mp4
Previously when there was no such config, it used to work fine.
Solutions?

I had the same problem last week. It manifested itself when using SDDM and SLIM. Installed LightDM - everything worked fine.

It used to work just fine with sddm and replacing sddm is not a option for me, at all.

Faced the same problem again. This time, after switching the layout to my native language, moving and resizing windows with the mouse stops working. With the English keyboard layout enabled, these problems are not present.


CLI does not collect assets

Problem
The CLI no longer collects assets correctly
Steps To Reproduce
Steps to reproduce the behavior:

clone dioxus
run cargo run --package dioxus-cli -- serve --example custom_assets --features collect-assets --platform desktop

Expected behavior
Assets should be copied and optimized when serving with the CLI.
Environment:

Dioxus version: main
Rust version: nightly
OS info: MacOS
App platform: cli

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

This works for me but it's temperamental. It depends on which dir you're serving from which is likely due to how magnanis is doing asset resolution.

We generally need to make magnanis more robust to paths.

Fixed by https://github.com/DioxusLabs/collect-assets/pull/12


[Bug]  TypeError: Cannot convert object to primitive value

Version
v2.1.0
Reproduction link
https://github.com/twbs/bootstrap/issues/30553#issuecomment-612617589
Operating System
Ubuntu
Device
PC and Mobile
Browser & Version
All Browsers
Steps to reproduce

Create a site using material design
Try to add JS9: https://js9.si.edu/
add static files. refer to js9support.min.js as example shows.

What is expected?
When navbar-toggler width data-target of #navigation-example is clicked an animation to open a side bar is expected.
What is actually happening?
An error says TypeError: Cannot convert object to primitive value and no side bar is opened

Solution
https://github.com/twbs/bootstrap/issues/30553#issuecomment-612617589
chanhed this line:
 if (!i && l.toggle && /show|hide/.test(t) && (l.toggle = !1), i || (i = new r(this, l), o.data(n, i)), 'string' == typeof t) {
to this:
if (!i && l.toggle && /show|hide/.test(l) && (l.toggle = !1), i || (i = new r(this, l), o.data(n, i)), 'string' == typeof t) {
Additional comments
Unfortunately my code is not in production. So I cannot create a codepen etc...

Hello @mshemuni,
Thank you for using our products, the TypeError: Cannot convert the object to primitive value error typically occurs when you try to use an object in a context where a primitive value is expected, such as when you try to concatenate an object with a string using the + operator.
To fix this error, you need to convert the object to a primitive value before using it in a context where a primitive value is expected. One way to do this is by using the valueOf method, which returns the primitive value of an object.
Hope this information helps you. Please let us know if we can help you with anything else.
All the best,
Stefan


Update role-capabilities.md

Added documentation for new "view connections" capability allowing admin users to allow right users to view connections.
Purpose of this pull request
This pull request (PR) ...
Issue number: SUMO-225496
Select the type of change:

[ ] New Content - New features, sections, pages, tutorials

I added the "do-not-merge" label.

@rmamilla-sumo, note that the "verifiation/cla-signed" check will pass once someone else on my team approves PR https://github.com/SumoLogic/sumologic-documentation/pull/3193 to add you to the contributors list (clabot file).

@rmamilla-sumo, I now see the "View Connections" role in the Demo environment.


cilium-cli: Test TLS with serverNames

Change the client-egress-l7-tls policy to also require SNI match on the configured external target. Add a new client-egress-tls-sni test case to test SNI enforcement without TLS interception. Remove the trailing dot from the external target default value to make this easier.
Add coverage for SNI enforcement in cilium-cli connectivity tests.

/test

/test

Reopened at https://github.com/cilium/cilium/pull/35887 to get GH workflow changes to be run from the PR.


all routes on modules are not found after update from 4 to 6

My routes was working fine on  nwidart/laravel-modules v 4, but after I update to 6 it's doesn't work. I upgrade to 6 because i just upgraded from laravel 5 to laravel 6
I try to change to nwidart/laravel-modules v5 but return error:
  ErrorException  : require(/Users/yaskur/Sites/laravel/project/Modules/Admin): failed to open stream: No such file or directory

  at /Users/yaskur/Sites/laravel/project/vendor/laravel/framework/src/Illuminate/Routing/RouteFileRegistrar.php:35
    31|     public function register($routes)
    32|     {
    33|         $router = $this->router;
    34|
  > 35|         require $routes;
    36|     }
    37| }
    38|

  Exception trace:

  1   require()
      /Users/yaskur/Sites/laravel/project/vendor/laravel/framework/src/Illuminate/Routing/RouteFileRegistrar.php:35

  2   Illuminate\Routing\RouteFileRegistrar::register("/Users/yaskur/Sites/laravel/project/Modules/Admin")
      /Users/yaskur/Sites/laravel/project/vendor/laravel/framework/src/Illuminate/Routing/Router.php:416

  Please use the argument -v to see more details.
Script @php artisan package:discover --ansi handling the post-autoload-dump event returned with error code

on nwidart/laravel-modules v 6 doesnt show error but the route can not loaded.
What's the code actually need to change? I don't see any documentation about it.

I created new modules and the routes are working fine. why old modules doesnt work?

I created new modules and the routes are working fine. why old modules doesnt work?

I just found the changelog and to migrate the old module just using php artisan module:v6:migrate
Now working fine, thanks

I just found the changelog and to migrate the old module just using php artisan module:v6:migrate
Now working fine, thanks


Width proble,

Hi I use BottomPopUp to select user cards
When BottomPopUp shows width is not equal to by self.view

I need equal width BottomPopUp with my root view

Had the same problem and I did this to solve it. I don't know if it 's correct but it works :)
self.view.frame = CGRect(x: 0, y: 0, width: UIScreen.main.bounds.width, height: self.view.frame.height)

Had the same problem and I did this to solve it. I don't know if it 's correct but it works :)
self.view.frame = CGRect(x: 0, y: 0, width: UIScreen.main.bounds.width, height: self.view.frame.height)

it work


Refactoring: Implement a refactoring action to simplify long number literal format. SR-5746

For instance, converting from "100000" to "100_000", and "1000.0001" to "1_000.000_1".

@swift-ci please smoke test


Fix broken scenery unit tests

I noticed in https://github.com/phetsims/aqua/issues/138 that scenery unit tests are not being run on CT. While testing https://github.com/phetsims/scenery/issues/1300 I found that several of the accessibility unit tests are broken at the moment.

Many of the broken tests are related to PDOM element ids. For example, this one
assert.ok( buttonLabel.getAttribute( 'for' ) === buttonElement.id, 'label for attribute' );
The 'for' attribute is empty and the buttonElement.id is 'display1-primary-4-19'. When setting the for attribute, I am seeing that the element ID is empty.

I suspect the regressions may be caused by https://github.com/phetsims/phet-io/issues/1847, so closing this issue and we will resume over there.

https://github.com/phetsims/phet-io/issues/1847 was causing a fair number of failures, thanks for reporting and I'm sorry for not finding that sooner. I am going to reopen this for two errors I'm still seeing:

Thanks so much for working on https://github.com/phetsims/phet-io/issues/1847!
The keydown/keyup errors are a event.code vs `event.key issue and have possibly been failing for many months since we switched from event.key to event.code.
The globalKeyStateTracker failure is because of the valueType for the Emitters in KeyStateTracker:
    this.keydownEmitter = new Emitter( { parameters: [ { valueType: KeyboardEvent } ] } );
    this.keyupEmitter = new Emitter( { parameters: [ { valueType: KeyboardEvent } ] } );

But the unit tests are creating an event of type Event. Maybe the unit tests can create a KeyboardEvent instead.

OK, fixed in the above commit by dispatching an actual KeyboardEvent in the unit tests.
Unit tests are passing again


Pull request for ELEC 413 project 1

project 1 of Sameen Islam 70997606

Pull request for project 1 Sameen Islam

As an aside, my project only has 7 layouts, as that was all that I could fit in the 605 x 410 space. Could I upload another file that would contain my other 3 layouts?

yes, you can submit multiple files.


[BUG] DEBUG-trace is not always reported

Assumption: even using NEVER_SAMPLE, a debug trace should always be sampled.
From the B3-Propagation spec:

When Debug is set, the trace should be reported to the tracing system and also override any collection-tier sampling policy. Debug implies an accept sampling decision.

There is a bug under the following conditions:

service tracer has sampler that is not ALWAYS_SAMPLE
X-B3-Flags: 1
X-B3-Sampled is not set
X-B3-TraceId/SpanId/ParentSpanId is correctly set

Actual result: the X-B3-Flags: 1 trace is NOT sampled.
Expected result: the X-B3-Flags: 1 trace IS sampled.
According to the B3-Propagation spec, this set of propagation headers are expected in a downstream service:

Debug is encoded as X-B3-Flags: 1. Debug implies an accept decision: don't also send X-B3-Sampled: 1.

In the method B3Propagation#extract(C carrier) the parameters are extracted from the carrier and a TraceContextOrSamplingFlags is returned. This object is created with TraceContextOrSamplingFlags.create(result.sampled(sampledV).debug(debug).build()) which is a TraceContext with field flags = 8.
This results in traceContext.debug() returning true and traceContext.sampled() returning null.  Strictly speaking, this is correct: the extractor only found a debug flag and not a sampling decision. Please note that this is a bit of an anomaly: for SamplingFlags.DEBUG both the fields sampled and debug are set to true.
But let's assume that a TraceContext with debug enabled, without a sampling decision set, is correct:
In Tracer#joinSpan(TraceContext context) the context is queried to find out a sampling decision has been made or not:
// If we are joining a trace, we are sharing IDs with the caller
// If the sampled flag was left unset, we need to make the decision here
if (context.sampled() == null) { // then the caller didn't contribute data
      context = context.toBuilder().sampled(sampler.isSampled(context.traceId())).build();
}

The sampler is asked to make a decision on wether this trace should be sampled or not. Note that .debug() is never checked. If the sampler makes the decision to not sample this trace, the new context has the flags: sample=false and debug=true.
At that point, Tracer#toSpan(TraceContext context) is called with the newly constructed context. The test Boolean.TRUE.equals(decorated.sampled()) returns false and a NoopSpan.create(decorated) is returned.
Result: A NoopSpan is returned and the DEBUG-trace is NOT sampled/recorded/reported.
So which part is wrong ?

The B3Propagation extractor by correctly extracting the debug flag, but not setting sampled=true ?
The Tracer asking the sampler for a sampling decision, without checking the context.debug() state first ?

I can provide a failing unit test for both root-causes, but probably only one is valid ?!

@adriancole comment from https://github.com/openzipkin/openzipkin.github.io/issues/31#issuecomment-218365248 says:

the sampled() method includes the debug flag in its decision

That is currently no longer the case and I think that would resolve this issue!
I will try to create a failing unit test that points this out!

If I understand you correctly.. you have set X-B3-Flags: 1 and not
X-B3-Sampled: 1
You are right that according to how below was written it would seem this
should work
https://github.com/openzipkin/b3-propagation
However, that is more likely just an accident of how it is written.
Are you unable to set X-B3-Sampled: 1? Is there any open source library
that allows you to leave out X-B3-Sampled: 1 when setting  X-B3-Flags: 1?
I don't mind changing this impl, just concerned you may end up needing to
request this in every library..

Is there any open source library that allows you to leave out X-B3-Sampled: 1 when setting  X-B3-Flags: 1?

Yes, brave instrumentation does exactly that ? :-)
Both in the B3-Propagation spec & the implementation, the sampled flag is not injected in the carrier if debug is enabled.
See https://github.com/openzipkin/brave/blob/686379fb2a5b123d54d3f4da78ab9e97f1310005/brave/src/main/java/brave/propagation/B3Propagation.java#L93

anyway I agree! let's get this fixed

release 4.18.2 is on the way. Thank you so much!

Thank you very much for fixing this so fast!
I had some feedback on the PR ... but okay ;-)


Feat: Import functions that are written as ES Modules

Currently only the common js format can be used to create functions, something like:
module.exports = function handler(context) { ... }

and then to use this function: faas-js-runtime index.js
This adds the ability to create the function in the ES module format.  Function could look something like this:
const handle = async function(context) => { ... };

// Export the function
export { handle };

and then to use it faas-js-runtime index.mjs
You can also use this faas-js-runtime index.js if the functions package.json has the type=module field

Codecov Report

Merging #140 (a9fdc56) into main (b1bafc8) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #140      +/-   ##
==========================================
+ Coverage   92.37%   92.40%   +0.02%
==========================================
  Files           8        9       +1
  Lines         236      250      +14
==========================================
+ Hits          218      231      +13
- Misses         18       19       +1

Impacted Files
Coverage Δ

lib/function-loader.js
100.00% <100.00%> (ø)

lib/metrics.js
96.07% <0.00%> (-1.97%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Cannot select text on vscode.dev on iPadOS

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

VS Code Version: 1.91.1 (VSCode.dev)
OS Version: iPadOS 17.5.1

Steps to Reproduce:

Open a file on vscode.dev
Double tap on a word

You can see that the cursor applies from and to the end of a word. The cursor is fixed at the same word (I tried to drag the cursor but it doesn’t extend at all)
https://github.com/user-attachments/assets/1e2731c0-58ad-47da-af39-aed3dcbb22a0

This appears to be a duplicate of https://github.com/microsoft/vscode/issues/167151#issuecomment-2547453307 I also don't repro the original bug on the latest iPadOS


Assertion `hasValidInsertionPoint()' failed in variadic-generic function

Description
No response
Reproduction
@usableFromInline
enum SortOrder {
    case ascending
    case descending
}

@inlinable @inline(__always)
func isLessThan<Root, each Value: Comparable>(
    _ leftRoot: Root,
    _ rightRoot: Root,
    comparing properties: repeat ((Root) -> each Value, SortOrder)
) -> Bool {

    // Hopefully this will get constant-folded.
    var numProperties = 0
    for _ in repeat each properties {
        numProperties += 1
    }

    var propertyIdx = 0
    for (getProperty, mode) in repeat each properties {
        let (left, right) = (getProperty(leftRoot), getProperty(rightRoot))
        if propertyIdx < numProperties {
            guard left != right else {
                propertyIdx += 1
                continue
            }
        }
        switch mode {
        case .ascending:  return left < right
        case .descending: return left > right
        }
    }
    return false
}

I built with -O, but with/without that flag, it seems to crash the compiler in the same spot.
Stack dump
swift-frontend: /home/build-user/swift/include/swift/SIL/SILBuilder.h:3024: void swift::SILBuilder::insertImpl(swift::SILInstruction *): Assertion `hasValidInsertionPoint()' failed.
Please submit a bug report (https://swift.org/contributing/#reporting-bugs) and include the crash backtrace.
Stack dump:
0.	Program arguments: /opt/compiler-explorer/swift-nightly/usr/bin/swift-frontend -frontend -S -primary-file <source> -target x86_64-unknown-linux-gnu -disable-objc-interop -g -debug-info-format=dwarf -dwarf-version=4 -O -empty-abi-descriptor -resource-dir /opt/compiler-explorer/swift-nightly/usr/lib/swift -Xllvm --x86-asm-syntax=intel -module-name output -plugin-path /opt/compiler-explorer/swift-nightly/usr/lib/swift/host/plugins -plugin-path /opt/compiler-explorer/swift-nightly/usr/local/lib/swift/host/plugins -enable-default-cmo -o /app/output.s
1.	Swift version 5.11-dev (LLVM 170b778140971f1, Swift 9b5da0d82fca791)
2.	Compiling with the current language version
3.	While evaluating request ASTLoweringRequest(Lowering AST to SIL for file "<source>")
4.	While silgen emitFunction SIL function "@$s6output10isLessThan__9comparingSbx_xq_xc_AA9SortOrderOtq_QptRv_SLR_r0_lF".
 for 'isLessThan(_:_:comparing:)' (at <source>:12:1)

Expected behavior

Describe the behavior you expected instead of the crash.

Err... no crash, of course :)
Environment
Swift version 5.11-dev (LLVM 170b778140971f1, Swift 9b5da0d82fca791)
Target: x86_64-unknown-linux-gnu
Additional information
No response

Seems to be related to the switch at the end of the loop body.
If I write it like this, it works:

@inlinable @inline(__always)
func isLessThan<Root, each Value: Comparable>(
    _ leftRoot: Root,
    _ rightRoot: Root,
    comparing properties: repeat ((Root) -> each Value, SortOrder)
) -> Bool {

    // This should get constant-folded.
    var numProperties = 0
    for _ in repeat each properties {
        numProperties += 1
    }

    var propertyIdx = 0
    for (getProperty, mode) in repeat each properties {
        let (left, right) = (getProperty(leftRoot), getProperty(rightRoot))
        // We can save an equality check for the final element.
        if propertyIdx < numProperties {
            guard left != right else {
                propertyIdx += 1
                continue
            }
        }
        if mode == .ascending {
          return left < right
        } else if mode == .descending {
          // The 'if' condition on this branch is necessary to avoid a compiler crash.
          return left > right
        }
    }
    return false
}

The else condition needs to be else if mode == .descending (even though that's the only other case) in order to avoid the crash. A plain else makes it crash again.

Reduced test case:
func test<each T>(t: repeat each T) -> Bool {
  for _ in repeat each t {
    let _ = "Sima"
    return "fix" < "this"
  }

  return false
}

Describe the behavior you expected instead of the crash.

Err... no crash, of course :)

A compilation error is not a crash, but you expect this code to compile successfully, right? 🙂
cc @simanerush

Thank you @AnthonyLatsis for reducing the test case!

Minimal test case:
func test<each T>(t: repeat each T) {
  for _ in repeat each t {
    break
  }
}


pnpm tauri info exits with error code 101

Describe the bug
Running pnpm tauri info exits with code 101
To Reproduce
Steps to reproduce the behavior:

pnpm init
pnpm add -D @tauri-apps/cli
pnpm add @tauri-apps/api
tauri init
pnpm tauri info
Observe error

Expected behavior
Display the correct versions and exit with no error. Also note that the incorrect version for @tauri-apps/cli is reported when using pnpm before my proposed fix.
Screenshots
pnpm tauri info
Downloading Rust CLI...
Download Complete
 app:spawn Running "/home/barry/experiment/tauri/demo-1/node_modules/.pnpm/@tauri-apps+cli@1.0.0-beta.9/node_modules/@tauri-apps/cli/bin/tauri-cli tauri info" +0ms

Operating System - Ubuntu, version 20.04 X64

Node.js environment
  Node.js - 14.17.4
  @tauri-apps/cli - 1.0.0-beta.7 (outdated, latest: 1.0.0-beta.9)
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error("unexpected character 't' while parsing major version number")', src/info.rs:323:53
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
  @tauri-apps/api - tauri-apps
 app:spawn Command "/home/barry/experiment/tauri/demo-1/node_modules/.pnpm/@tauri-apps+cli@1.0.0-beta.9/node_modules/@tauri-apps/cli/bin/tauri-cli" failed with exit code: 101 +0ms

Platform and Versions (required):

Operating System - Ubuntu, version 20.04 X64

Node.js environment
  Node.js - 14.17.4
  @tauri-apps/cli - 1.0.0-beta.9
  @tauri-apps/api - 1.0.0-beta.6

Global packages
  npm - 7.20.3
  pnpm - 6.13.0
  yarn - 1.22.4

Rust environment
  rustc - 1.54.0
  cargo - 1.54.0

App directory structure
/src-tauri
/node_modules

App
  tauri.rs - 1.0.0-beta.7
  build-type - bundle
  CSP - default-src blob: data: filesystem: ws: wss: http: https: tauri: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'
  distDir - src-tauri/tauri.conf.json
  devPath - http://localhost:3000

Additional context
My suggested fix produces the above output
Stack Trace

I am still getting this problem; what can I do to work around this?

Note: I reinstalled everything with npm as I thought the error had something to do with pnpm but I get the same error.

@ksnyde the fix is only in the next branch and has not been released yet.

Closed by #2510


"Item charges" plugin not tracking loss of charge for Binding necklace when Hat of the eye is equipped. Possibly any piece of outfit, only have the hat and it caused this issue.

Please check if your issue is not a duplicate by searching existing issues
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: [e.g. Windows, Ubuntu, macOS]
RuneLite version: [e.g 1.4.6]
Launcher version: [e.g 1.5.2]

Additional context
Add any other context about the problem here, e.g. logs. Your client logs can usually be found in
C:\Users\<your_user_name>\.runelite\logs on Windows and ~/.runelite/logs on Linux and macOS.

it does seem to happen with any piece of the outfit, i have the leg piece
on my end it also erroneously shows that it has 17 charges when a necklace breaks, as well as saying in chat that bindng the runes was only partially successful despite getting the full amount
(the latter could be a jagex issue tho, not sure)

If you have the game chat filter off, what chat message do you get when you bind runes with part of the outfit equipped? I assume it is changed since GotR's release, or changed when you have the outfit equipped. The plugin expects it to look like:

You bind the temple's power into lava runes.

When wearing pieces of the outfit the chat message changes to:

You partially succeed to bind the temple's power into steam runes.

This is weird because it's the message you get when not using a binding necklace. But the charge will still be used and the correct amount of runes will be given as well. I was getting the partial success message with any combination of the set pieces.

That sounds like an ingame bug then.

I believe this has been worked around now with this commit https://github.com/runelite/runelite/commit/a350b5a57b688411a8f953ad82c55a0fac412c99


coredns and calico-kube-controller stuck in ContainerCreating: Failed to create pod sandbox failed to setup network for sandbox operation not supported

I did a fresh install of microk8s on a 4-node raspberry pi 4 cluster, following the instructions on https://ubuntu.com/tutorials/how-to-kubernetes-cluster-on-raspberry-pi#6-thats-it. Also did a "microk8s enable dns". Both coredns and calico-kube-controller are stuck in ContainerCreating, and kubectl describe pod ... shows "Failed to create pod sandbox [...] failed to setup network for sandbox operation not supported. Before I did "microk8s enable dns", calico-kube-controller was also already in that state
inspection-report-20211113_183336.tar.gz
.

On one of the nodes, I found the following in /var/log/containers/calico-node[...]kube-system_calico-node[...].
There's an extra package you need to install in the OS.  Check the documentation, where the team recently added some instructions on what to do with ubuntu impish.
https://microk8s.io/docs/troubleshooting#heading--common-issues

Thanks @balchua, that indeed solved my issue.


Caps lost in name and definitions in query

Query for adult antennal lobe. From there choose
'Query for: Neurons with synaptic terminals in adult antennal lobe'
Caps lost:
Name: 'adult antennal lobe projection neuron adpn' should be 'adult antennal lobe projection neuron adPN'
Definition:'An adult antennal lobe projection neuron that is derived from neuroblast alad1 (fbbt:00067346). all of these neurons have axons that fasciculate with the medial antennal lobe tract (malt) and innervate the lateral horn' should be 'An adult antennal lobe projection neuron that is derived from neuroblast ALad1 (FBbt:00067346). All of these neurons have axons that fasciculate with the medial antennal lobe tract (mALT) and innervate the lateral horn.
See screenshot for other examples.

Caused by reinstated capitalisation with new less formatting:  https://github.com/openworm/org.geppetto.frontend/blob/development/src/main/webapp/js/components/interface/query/query.less#L457

Still name being lowercased - need to track down code (not CSS)

CSS change fixed for definitions but not names

Still an issue for name and definition columns

Issue moved to VirtualFlyBrain/geppetto-vfb #191 via ZenHub


Analyzer throws a compile error when async function with Future return type returns dynamic.

Dart VM version: 2.9.0-13.0.dev (dev) (Fri May 29 15:59:05 2020 +0200) on "windows_x64"
This issue is related with SDK Issues #41266, #41437, as well as with co19 issues #603 and 703.
It is similar with #42236, but it's agains analyzer, not dart.
Seems like after the #941  the following code should not throw compile error:
import "dart:async";
dynamic getNull() => null;
Future<bool> test1() async => await getNull();
main() {}

However, still dartanalyzer behaves in such a way.
Sample output is:

$> dartanalyzer --enable-experiment=non-nullable test.dart
Analyzing test.dart...
error - A value of type 'dynamic' can't be returned from function 'test1' because it has a return type of 'Future'. - test.dart:3:31 - return_of_invalid_type
1 error found.

This was fixed in 89782c96d9d02dd0c4758d3b285dd57fb32745be, the same day as the SDK build that you are trying.
But maybe a few hours later?
Works for me with the bleeding edge SDK.


Update to att&ck not working anymore

Hello,
got he following error only for th attack updates:
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): attack.mitre.org
DEBUG:urllib3.connectionpool:https://attack.mitre.org:443 "GET /api.php?action=ask&format=json&query=%5B%5BCategory%3ATactic%5D%5D HTTP/1.1" 404 2176
psexec still working.
tested on parrot / kali / ubuntu lts with last github update and requirements satisfied and got the same issue on all linux.

sorry it was an issue related to caldera


I have problems to test my application from the phone

I am doing developments in visual studio code in ubuntu, and I could do the tests with the phone connected to the laptop the app was created in my phone and I could try after a moment to another I am not sure that it was updated if flutter, ubuntu, visual code, and I could no longer do tests from the phone, I run flutter doctor and he always tells me that there is no device detected :-(
I want to continue testing from the phone directly, since the emulator makes my laptop go a little slower

Hi @EduardEddy
can you please provide your flutter doctor -v and flutter run --verbose? Also, to better address the issue, would be helpful if you could post a minimal code sample to reproduce the problem
Thank you

flutter doctor -v
Waiting for another flutter command to release the startup lock...
[✓] Flutter (Channel stable, v1.9.1+hotfix.4, on Linux, locale es_CO.UTF-8)
• Flutter version 1.9.1+hotfix.4 at /home/eduard/development/flutter
• Framework revision cc949a8e8b (hace 5 meses), 2019-09-27 15:04:59 -0700
• Engine revision b863200c37
• Dart version 2.5.0
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /home/eduard/Android/Sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• Java binary at: /opt/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b16-5323222)
• All Android licenses accepted.
[✓] Android Studio (version 3.4)
• Android Studio at /opt/android-studio
• Flutter plugin version 39.0.1
• Dart plugin version 183.6270
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b16-5323222)
[✓] VS Code (version 1.40.2)
• VS Code at /usr/share/code
• Flutter extension version 3.8.1
[!] Connected device
! No devices available
! Doctor found issues in 1 category.
============ and the other command ===============================
flutter run --verbose
[  +18 ms] executing: [/home/eduard/development/flutter/] git log -n 1
--pretty=format:%H                               [  +35 ms] Exit code 0 from: git log -n 1 --pretty=format:%H
[        ] cc949a8e8b9cf394b9290a8e80f87af3e207dce5
[        ] executing: [/home/eduard/development/flutter/] git describe
--match v*.. --first-parent --long --tags
[   +8 ms] Exit code 0 from: git describe --match v*..
--first-parent --long --tags
[        ] v1.9.1+hotfix.4-0-gcc949a8e8b
[  +10 ms] executing: [/home/eduard/development/flutter/] git
rev-parse --abbrev-ref --symbolic @{u}
[   +8 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic
@{u}
[        ] origin/stable
[        ] executing: [/home/eduard/development/flutter/] git
ls-remote --get-url origin
[  +10 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +62 ms] executing: [/home/eduard/development/flutter/] git
rev-parse --abbrev-ref HEAD
[  +10 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[ +102 ms] executing: /home/eduard/Android/Sdk/platform-tools/adb
devices -l
[  +12 ms] Exit code 0 from:
/home/eduard/Android/Sdk/platform-tools/adb devices -l
[   +1 ms] List of devices attached
[  +17 ms] No supported devices connected.
[   +9 ms] "flutter run" took 152ms.
#0      throwToolExit
(package:flutter_tools/src/base/common.dart:28:3)
#1      RunCommand.validateCommand
(package:flutter_tools/src/commands/run.dart:281:7)

#2      FlutterCommand.verifyThenRunCommand
(package:flutter_tools/src/runner/flutter_command.dart:465:11)

#3      FlutterCommand.run.
(package:flutter_tools/src/runner/flutter_command.dart:407:33)

#4      AppContext.run.
(package:flutter_tools/src/base/context.dart:154:29)

#5      _rootRun (dart:async/zone.dart:1124:13)
#6      _CustomZone.run (dart:async/zone.dart:1021:19)
#7      _runZoned (dart:async/zone.dart:1516:10)
#8      runZoned (dart:async/zone.dart:1463:12)
#9      AppContext.run
(package:flutter_tools/src/base/context.dart:153:18)

#10     FlutterCommand.run
(package:flutter_tools/src/runner/flutter_command.dart:397:20)
#11     CommandRunner.runCommand
(package:args/command_runner.dart:197:27)

#12     FlutterCommandRunner.runCommand.
(package:flutter_tools/src/runner/flutter_command_runner.dart:402:21)

#13     AppContext.run.
(package:flutter_tools/src/base/context.dart:154:29)

#14     _rootRun (dart:async/zone.dart:1124:13)
#15     _CustomZone.run (dart:async/zone.dart:1021:19)
#16     _runZoned (dart:async/zone.dart:1516:10)
#17     runZoned (dart:async/zone.dart:1463:12)
#18     AppContext.run
(package:flutter_tools/src/base/context.dart:153:18)

#19     FlutterCommandRunner.runCommand
(package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)

#20     CommandRunner.run.
(package:args/command_runner.dart:112:25)
#21     new Future.sync (dart:async/future.dart:224:31)
#22     CommandRunner.run (package:args/command_runner.dart:112:14)
#23     FlutterCommandRunner.run
(package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#24     run..
(package:flutter_tools/runner.dart:63:22)

#25     _rootRun (dart:async/zone.dart:1124:13)
#26     _CustomZone.run (dart:async/zone.dart:1021:19)
#27     _runZoned (dart:async/zone.dart:1516:10)
#28     runZoned (dart:async/zone.dart:1500:12)
#29     run.
(package:flutter_tools/runner.dart:61:18)

#30     AppContext.run.
(package:flutter_tools/src/base/context.dart:154:29)

#31     _rootRun (dart:async/zone.dart:1124:13)
#32     _CustomZone.run (dart:async/zone.dart:1021:19)
#33     _runZoned (dart:async/zone.dart:1516:10)
#34     runZoned (dart:async/zone.dart:1463:12)
#35     AppContext.run
(package:flutter_tools/src/base/context.dart:153:18)

#36     runInContext
(package:flutter_tools/src/context_runner.dart:59:24)

#37     run (package:flutter_tools/runner.dart:50:10)
#38     main (package:flutter_tools/executable.dart:65:9)

#39     main
(file:///b/s/w/ir/k/archive/flutter/packages/flutter_tools/bin/flutter
_tools.dart:8:3)
#40     _startIsolate.
(dart:isolate-patch/isolate_patch.dart:303:32)
#41     _RawReceivePortImpl._handleMessage
(dart:isolate-patch/isolate_patch.dart:172:12)

@EduardEddy
do you have enabled usb debugging on your phone?
does it persist after this workaround? https://github.com/flutter/flutter/issues/27175#issuecomment-533930857

@EduardEddy
I hope you've solved the problem, closing the issue.
if you disagree please write in the comments and I will reopen it.
Thank you


npx jsr add hangs forever

I am trying to install my library simple-data-analysis with npx jsr add @nshiab/simple-data-analysis, but it hangs forever.
I suspect it's because duckdb is one of the dependencies and it has to run postinstall scripts,  but I am not sure.
I have no problem installing my other library journalism with npx jsr add @nshiab/journalism.
Thank you!

It looks like it was a problem with node v23. I rolled back to v22 (LTS version) and it works!


[Webhook] Implement Allowed Rate

Is this solved by https://github.com/cloudevents/sdk-go/pull/491 ?

Allowed rate is now negotiated but rate limit is not implemented.

Shall I take a step and create a PR with something like https://github.com/sethvargo/go-limiter (see httplimit) or rather basic using x/rate? Background: I currently need this functionality (also #56) so happy to support :)


drop the use of deprecated feature of webpack

As the source of webpack compiler says, should use the (new) interface.

Thanks, @e-cloud.
Would you like to send the PR?

sorry, i have no time for other project for now :weary: . Just report it

this is the actual output coming out from running webpack tests:
  .....................................................................................
  .....................................................................................
  .........................................................................webpack: Using compiler.parser is deprecated.
Use compiler.plugin("compilation", function(compilation, data) {
  data.normalModuleFactory.plugin("parser", function(parser, options) { parser.plugin(/* ... */); });
}); instead. It was called at I18nPlugin.apply (D:\WS\JS\webpack\node_modules\i18n-webpack-plugin\index.js:43:18).
webpack: Using compiler.parser is deprecated.
Use compiler.plugin("compilation", function(compilation, data) {
  data.normalModuleFactory.plugin("parser", function(parser, options) { parser.plugin(/* ... */); });
}); instead. It was called at I18nPlugin.apply (D:\WS\JS\webpack\node_modules\i18n-webpack-plugin\index.js:43:18).

............
  .....................................................................................

Fixed now.
@BrainCrumbz thanks for tour work, but merged @vivek12345 's work since I'm not sure why your PR didn't passed the CI.

Well, that was 5 months ago, I don't know either! ;)

@EcutDavid  not sure I understand the solution for this. how can I get rid of the error displaying in the terminal? I have the latest, please help.
webpack: Using compiler.parser is deprecated. Use compiler.plugin("compilation", function(compilation, data) {   data.normalModuleFactory.plugin("parser", function(parser, options) { parser.plugin(/* ... */); }); }); instead. It was called at I18nPlugin.apply (/node_modules/i18n-webpack-plugin/index.js:43:18). Error: not a directory     at MemoryFileSystem.mkdirpSync (/node_modules/memory-fs/lib/MemoryFileSystem.js:144:10)     at MemoryFileSystem.(anonymous function) [as mkdirp] (/node_modules/memory-fs/lib/MemoryFileSystem.js:279:34)     at Compiler.<anonymous> (/node_modules/webpack/lib/Compiler.js:322:27)     at /node_modules/async/dist/async.js:3025:16     at eachOfArrayLike (/node_modules/async/dist/async.js:940:9)     at eachOf (/node_modules/async/dist/async.js:990:5)     at Object.eachLimit (/node_modules/async/dist/async.js:3089:3)     at Compiler.emitFiles (/node_modules/webpack/lib/Compiler.js:312:20)     at Immediate._onImmediate (/node_modules/memory-fs/lib/MemoryFileSystem.js:288:4)     at tryOnImmediate (timers.js:534:15)     at processImmediate [as _immediateCallback] (timers.js:514:5)


feat(backing): backingimage backup support

ref: longhorn/longhorn#4165
For BackingImage backup, we implemented 3 main functions

CreateBackingImageBackup()

with BackupBackingImageStatus interface, we can read the BackingImage data and upload it as block.

RestoreBackingImageBackup()

with RestoreBackingImageStatus interface, we can update the sync_file progress in backing-image-data-source

RemoveBackingImageBackup()

and 3 commands

ls-backing-image: list all backing image names in the backupstore, so we can create the CR with backup_target_controller
inspect-backing-image: get the info of the backup backing image, so we can sync the info the CR in the new cluster
rm-backing-image: we can't implement this as grpc in backing-image-manager, since there might have no backing-image-manager pod in the cluster, so deletion can be only implemented as command and be used in the controller.

Hi @innobead
Refactored, thanks!

@ChanYiLin Check the failed PR check.

The first commit history is weird, and that could be related to how you did the rebase as discussed. I will scratch this PR instead. Please make sure the commit history and rebasing are correct in later PRs.
commit e1a25677b56d3f7002b73152f03595cf06f6a48f
Author: Jack Lin<EMAIL_ADDRESS>Date:   Mon Aug 28 17:02:29 2023 +0800

    feat(backing): backingimage backup support

    ref: longhorn/longhorn 4165

    Signed-off-by: Jack Lin<EMAIL_ADDRESS>
    Add code to parse a query at the end of targetURL for NFS mount options.

    Signed-off-by: James Munson<EMAIL_ADDRESS>
    Correct construction of backupURLs.

    Signed-off-by: James Munson<EMAIL_ADDRESS>
    Address review feedback.  Add CIFS code.  Expand unit tests.

    Signed-off-by: James Munson<EMAIL_ADDRESS>
    Revise to always keep nfsOptions during Encode and Decode BackupURL.

    Signed-off-by: James Munson<EMAIL_ADDRESS>
    Add output to ensure test is being run.

    Signed-off-by: James Munson<EMAIL_ADDRESS>
    feat: update golang to v1.21

    Signed-off-by: PoAn Yang<EMAIL_ADDRESS>
    fix(backup): Relax S3 client retry intervals
    ....


Fixed the http client performance issue

In the original http client, we used jmeter to start 200 concurrent requests to call jenkins client, but found tens of thousands of close_wait tcp connection can not be cleaned in system. This will used all free ports in client system which has performance issue in using origial jenkins client. Thus i used the pooling http connection to manage the connections to handle the concurrent requests to improve the concurrent handling performance. And call close method instead of releaseConnection the fully release the system resource and return the connection to pool for later reuse. My submitted code get rid of the close_wait issue and improve the concurrent performance finally.

First many thanks for offering that pull request...is there reason why you change the version to 0.3.9-SNAPSHOT ? Furthermore can you please squash your commits?

i squashed the previous commits, please review it.


[jk] Allow canceling of individual running pipeline runs

Description

There is a button for canceling ALL running pipeline runs for an entire pipeline, which includes runs on different pages in the UI. This could confuse users if they select specific runs and think only the selected runs will be canceled (not all of them).
This PR adds an "Actions" menu that consolidations the actions for retrying selected runs, canceling selected/checked pipeline runs in progress, and canceling all pipeline runs in progress.
Clean up Pipeline Runs Table (rearrange columns, change date font size, change link colors).

How Has This Been Tested?

Checklist

[X] The PR is tagged with proper labels (bug, enhancement, feature, documentation)
[X] I have performed a self-review of my own code

@dy46 Do the rest of the BE changes seem fine? It was just the mage_ai/api/resources/PipelineResource.py file on the BE that was updated.


Update documentation.

Add doc comments for:

Structural protocol
StructuralStruct, StructuralProperty
StructuralEnum, StructuralCase
StructuralCons, StructuralEmpty, StructuralEither

Rename properties:

StructuralStruct.type -> StructuralStruct.representedType
StructuralStruct.properties -> StructuralStruct.body
StructuralEnum.type -> StructuralEnum.representedType
StructuralEnum.cases -> StructuralEnum.body

Documentation changes were discussed during Swift for TensorFlow team meeting.
There's further room for improvement, we can discuss on this PR.

CI fails because I have to update the usages of renamed declarations (e.g. type → representedType).
I will do that later today.

I'm merging this PR now to unblock progress. Happy to address further feedback later!


[Suggestion] Ask if user wants to deploy before creating app

If we ask if they want to deploy before creating the web app, we can immediately go into deploying upon creation.  This will prevent users from missing the prompt and streamline deploying a little bit more.

Nobody likes my idea 🐸


Update Java grammar

This PR fixes property identification in Java. Before this fix an property could have two different state:

variable.other.object.java
variable.other.object.property.java

After the fix it will only be:

variable.other.object.property.java

Original issue: https://github.com/atom/language-java/issues/224
Merged PR: https://github.com/atom/language-java/pull/225
Credits go to @sadikovi

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

@rafaelrenanpacheco I update the grammars with a script about once per month. I think I incorporated these changes. If the changes are still missing please do comment!

Ah ok, thanks :smiley:


7016 migrate background importer

The previous PR was against the now defunct analysis branch

Can one of the admins verify this patch?


Require python >=3.5 in setup

Try to prevent pip from installing from 1.1.x from source when using Python <3.5 (see scikit-image/scikit-image#4258)

Taking a quick look at NumPy, the also do:
https://github.com/numpy/numpy/blob/c1f56c455252e4c2dd2535f0ce140125aff8ece2/setup.py#L30-L31
but don't seem to set python_requires

Hey @rgommers, please take a quick look if you are available. Thanks

I tried this in a Python 2.7 environment and it does fail to install with
"RuntimeError: Python version >= 3.5 required".
However this does error does not occur until after Cythonizing is complete. I suppose adding a case like in NumPy at the top would give a quicker failure.

This seems like it should be ok. Installing from git and pypi have different implications z

included in #10


🐛 BUG - Certain characters break the search in the reference field. e.g. /

Describe the issue that you're seeing. Any Loom videos or screenshots usually help a lot!
I am trying to use the new search feature in the reference collection but it does not seem to work as i would expect.
If i type in content/docs it shows nothing
If i type in contentdocs it shows all items in the content/docs directory
/a
https://github.com/user-attachments/assets/ddec0143-608c-46ef-92a3-7fb75476bcee
Reproduction
n/a
Steps to reproduce
see video
System Info
on current prod version of tina.io (tinacms v2.2.1)
Validations

[x] Follow our Code of Conduct
[x] Read the docs.
[x] Check that there isn't already an issue that reports the same bug.

this error happen due to race condition when we release new experimental custom filter for reference field, here are a quick fix so the new release is backward compatible 😊
https://github.com/tinacms/tinacms/pull/4764

As per conversation with @JackDevAU , user want to search content in a specific folder ('/' is not working). We will have a look into the search function to see if there is a quick fix for this

✅Done fixes - https://github.com/tinacms/tinacms/pull/4782


Supported Java versions of "application"?

What's going wrong and not working?
Your application readme says "Java 17 or higher".
I have Java 22.0.2 and when I try to gradlew bootRun, it fails.
[max@Maxs-Pro application (main)]$ javac -version
javac 22.0.2
[max@Maxs-Pro application (main)]$ java -version
openjdk version "22.0.2" 2024-07-16
OpenJDK Runtime Environment Homebrew (build 22.0.2)
OpenJDK 64-Bit Server VM Homebrew (build 22.0.2, mixed mode, sharing)
[max@Maxs-Pro application (main)]$ ./gradlew bootRun --scan

FAILURE: Build failed with an exception.

* What went wrong:
Could not determine the dependencies of task ':bootRun'.
> Could not resolve all dependencies for configuration ':runtimeClasspath'.
   > Failed to calculate the value of task ':compileJava' property 'javaCompiler'.
      > Cannot find a Java installation on your machine matching this tasks requirements: {languageVersion=17, vendor=any, implementation=vendor-specific} for MAC_OS on aarch64.
         > No locally installed toolchains match and toolchain download repositories have not been configured.

* Try:
> Learn more about toolchain auto-detection at https://docs.gradle.org/8.8/userguide/toolchains.html#sec:auto_detection.
> Learn more about toolchain repositories at https://docs.gradle.org/8.8/userguide/toolchains.html#sub:download_repositories.
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Get more help at https://help.gradle.org.

BUILD FAILED in 923ms

It seems to be looking for specifically version 17, not 17 or above.
Expected outcome
gradlew should serve the application!

@max-allan-cgr Thanks for the input. I've just updated the Java language version to current (alongside with a Gradle update). This should fix this problem.


add typescript compatibility for redis adapter

Please check if the PR fulfills these requirements

[x] Followed the Contributing guidelines.
[x] Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

@christianllv - can you get the testing back up to 100%?

Yes, I’m working on that now.

@jaredwray done.


Keyboard Accessibility issue reCAPTCHA v2

The reCaptcha picture selection challenge is inaccessible for keyboard users. W3C guidelines state that a website must be accessible via keyboard. The picture challenge is currently blocking keyboard interaction, which will need to be resolved to meet accessibility guidelines.

Closing super old issues. Please re-raise if still relevant.


Automated Update

@jkingoliver This PR smells funny. Did y'all mean to restore the .bluemix/ folder? I thought that was dead.

@dolph nope, it needs to stay dead.


where are the source repositories for the npm packages?

I can't seem to figure out where @rspc/react etc sources are located.  Otherwise I would help make a PR to fix #291

It is documented in the README. You can find it on the v0.x branch.


Axial and Radial shading pattern displayed as one color

https://issues.apache.org/jira/secure/attachment/12610815/axial-input.pdf
red ball
https://issues.apache.org/jira/secure/attachment/12610820/radial-input.pdf
blue ball
This is not related to the recent regression.

Possibly related: https://issues.apache.org/jira/secure/attachment/12652009/pattern-shading-2-4-idMatrix.pdf

Closing as fixed by PR #6684.


URL in description and README broken

The URL in the repository description and the one in the README are pointing to githubschool's copy of the game instead of yours.
Please fix both so they point to your copy of the game at https://githubschool.github.io/github-games-joshg253

Updated URL.


force redirect to https ssl

Hi there,
I have configured heroku with my own domain and set up ssl, so now https://myodoo.install works fine.  But now how can I force odoo to redirect to it.
Heroku says: Use the app to check for http and redirect to https
in odoo docs it says use apache or other webserver config for that.
but how can I do it now?
Thanks,
Martin

Will have to look at this, since I'm not running ssl myself at the moment (but I should lol).

I also need to get this working.  For now I do the redirect in JavaScript though the website editor.

I'm currently a little short on time, I can spend some time on Odooku in about a week.

Want some help?

Yes, if anybody wants to contribute to Odooku, feel free to do so. That would be great!

I really need that feature now, so where should I start?

This needs to be added to odooku compat. The wsgi application is a good place to start. I ll look at this monday.

I almost have a new feature ready which enables this, plus more. It enables rewriting. A rules.json file should be made in the following format:
{
   "http://*": {
     "rewrite": "https://${0}",
     "redirect": true
   "
}

I'm running a multi db setup and I name my databases according to the domain. So for example, I have a a db named www_adaptiv_nl for my domain www.adaptiv.nl . Any requests to adaptiv.nl will not match that db. I will fix it like this:
{
   "*://adaptiv.nl": {
     "rewrite": "${0}://www.adaptiv.nl",
     "redirect": true
   "
}

mybe you could provide an example file for that?
Just to know, will this cause 301 permanently moved responses, or which way do you go?
Thanks so much for that work.

Will do. Yes they ll be 301's!

Ok, I have just pushed a new version of odooku-compat. Could you help me test heroku ssl, I can't test this quickly myself now. Instructions:

Run the wsgi command with --proxy-mode
Create a file in your project root named "rules.json"

{
  "http://*": {
    "redirect": "https://{0}"
  }
}

If this works I'll make sure to update the docs asap. I'm reworking a lot of stuff.

This seems just to work for the root / of the url.  Other urls seem not
to be reloaded.
I would need a rule which redirects everything to https except
/.well-known/acme-challenge/*
How can I do this?

I have made some fixes. Above rule now works, I have tested it with Heroku. Docs will follow.
Regarding your rule exception /.well-known/acme-challenge/*, this I cannot support right now. But I assume you want to get a letsencrypt certificate? In that case, heroku now integrates with this service. Just enable automatic certificate management.

yes, the thing is that i use flynn and in flynn we have to wait a bit for letsencrypt.  I wrote an odoo module which implements it.  but maybe letsencrypt doesn't check the certificate so this could work....


Template path not found

Run<EMAIL_ADDRESS>Error: ENOENT: no such file or directory, open '/home/runner/work/fsr_qualification/fsr_qualification/.github/ISSUE_TEMPLATE/fsr-qualification.md'

Think it needs to use https://github.com/actions/checkout with e.g.
- uses: actions/checkout@v4
  with:
    sparse-checkout: |
      .github

Yes that fixed it


Add fastapi basic example

Description

Adding a bit more content to the main README.md file with a list/links to the example folders
Added a basic fastapi example with the most updated version of all packages

Types of Changes

[x] Core
[ ] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[x] Documentation

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

thank you!


Add documentation for VCR Plugin

See php-http/HttplugBundle#333

thanks! i did a few small tweaks and merged on the CLI f79e62aa0671de4e6b845e96fdde9f92594fb36c


feat: Add one-wire uart telemetry

In pico-tts/telem-cmake branch, there is an implementation to request telemetry over one-wire uart. This should be implemented in this repo, similar to how it's done in this diff.
The key requirements for this implementation:

implement one-wire uart to request and store latest telemetry
this should be implemented in a way such that the example file should allow the user to

control the rate at which telemetry is output
be able to turn on/off one-wire telemetry

Broadly, the implementation may/should look like:

/// --- Variables

/// struct to setup up one-wire uart telemetry
typedef struct telem_uart {
  /// --- General config
  uint gpio_esc_one_wire;
  uint baudrate;
  bool req_flag; // Only request telemetry if req_flag is true
  repeating_timer_t send_req_rt; // Repeating timer config for periodically requesting telemetry
  const uint buffer_size = 10; // KISS telemetry protocol outputs 10 bytes

  /// --- Storing uart telemetry output
  volatile size_t telem_uart_buffer_idx = 0 // Keep track of idx in uart telemetry buffer
  volatile char telem_uart_buffer[telem_uart_buffer_size] = {0}
}

/// --- Function prototypes (@todo)

void telem_uart_irq(void){
while (uart_is_readable(ESC_TELEMETRY_UART_GPIO)
}

// Reset the telemetry flag after sending the current dshot packet in `dshot.c::dshot_send_packet()`


Using SetState in Callback reference

Hello,
In your example, you have used set state in render for call back reference due to which it the stack size increases and gives an error of maximum call stack size exceeded.

Good catch! I'll try to get this fixed soon :)

Did you have an update on this?


"An error has occured :(" and #32 fix didn't work.

"An error has occured :(" appears when I try to view it and I have tried the #32 fix to no avail. You can view the page at https://themmodernflash.com/statuspage.

Use the new files from here - it worked for me: https://github.com/nerdbaggy/StatusPage/issues/44

I am now also getting this - even after update of the statusPage.php.  However, I am not sure how to update the JS - I only have statuspage.min.js and no main.js in my build.
@davidus05  what is the purpose of main.js ??  where does that go?  There is no reference to it in the main build v3.


Minor refactor and Laravel 9 working

Hello! I made my own working PR. I already made the tests and it's working.
For now, I had to mute 3 hardcoded failing tests and other that should be checked later when I run a game.
I will deep dive into it later in the week, but for now I think it should works.

I also included some changes from #126 and as you can see already run the tests (working).
I would thank if any of @nicekiwi or @syntaxerrors can take a look on this branch.

@vstyler96 thanks for the effort, but the changes don't go far enough to move forward (undescriptive i know), for now please see the laravel9 branch. Re tests, the tests act as a smoke test for the ever changing and undocumented Steam API, generally we'll leave failing tests to run.


Composite Indexes needed in the Staging Environment for /progresses

Issue Description
Since a new query parameter, date, has been introduced in the /progresses route, we must create new composite indexes in  the staging environment. Please note that some indexes may already exist, but to ensure none are overlooked, I have included them below (the indexes after sl no. 4 might be already existing in our indexes):

Sl No
Collection ID
Fields indexed

1
progresses
taskId Ascending type Ascending date Descending name Descending

2
progresses
type Ascending date Ascending name Ascending

3
progresses
taskId Ascending type Ascending date Ascending name Ascending

4
progresses
userId Ascending date Ascending name Ascending

5
progresses
type Ascending date Descending name Descending

6
progresses
type Ascending userId Ascending date Ascending name Ascending

7
progresses
type Ascending userId Ascending date Descending name Descending

8
progresses
taskId Ascending date Ascending name Ascending

As confirmed by @prakashchoudhary07 here these indexes are created in staging


Upload Imagenet text files (provide links to get the text files)

Hi
While I am training Imagenet data I didn't found the below two files
meta.mat
ILSVRC2012_validation_ground_truth.txt
Can any one provide the above files

Hello @rakashi
you can find those files in the "Development kit (Task 1 & 2)" archive (inside the 'data' folder) in the ImageNet LSVRC 2012 download page.


[v1.14] datapath: Fix redirect from from L3 netdev to tunnel

Manual backport to adjust the test for missing helpers.

[ ] https://github.com/cilium/cilium/pull/33421

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

Needs a CI change -> run from a repo branch: https://github.com/cilium/cilium/pull/35265


Use aave as a pricing asset

For historical pricing data on the WSB ETP

Is this still needed, given that we don't have AAVE/USD pool?

Yes, as discussed in slack, we would need a AAVE/USD pair to get USD pricing data


Service Definition: Do we need to provide an 'id'

Did AS3 implement the 'autogeneration' of 'id' as part of build 34 release. Test this and, if so, remove 'id' values from example service definitions.

In build 34 a declaration without an 'id' key will fail with the following response:
{
    "code": 500,
    "declarationFullId": ""
}

Thanks, Kevin. I got it in the AS3 backlog but hadn't yet tested if it made the release. Thanks for saving me the time!!! I'll keep this here, and open, in case the next release of AS3 is out before I release v0.2...


Fix all tests except spewrequest and dateline-5, both require GLD.

Import gpush and fix gpush test (note what it takes to get 10s of
backpressure here..i guess machines are faster now? ;)
Update cost1's output to match reality, and finally, add a .gitignore
(ignore bazel artifacts).

self-high fiving this one. :P


Port over integration tests from monorepo

Now that sequencing is nearly ready, we can start porting over integration tests from the monorepo. The first set we can port over include:

[ ] Deposits
[ ] Simple sends
[ ] Contract interactions

[ ] Uniswap
[ ] ERC20
[ ] ERC721

At what point does the specs repo turn into the monorepo? Should the specs repo contain specs and the monorepo contain code? This would be easier to delineate once all of the prototyping is done

I think we'd like it to contain both spec & code, so they can be kept in sync (updated by the same PR into main branch from a feature branch).


spirv-val fails on array of Position in Control shader.

test.spv.txt
error: 243: According to the Vulkan spec BuiltIn Position variable needs to be a 4-component 32-bit float vector. ID <3> (OpVariable) is not a float vector.
Spec is a bit vague here, because some kind of array of Position must be allowed (but perhaps only as part of a block).

With SPIRV-Tools top-of-master I do not get that validation error.  What version of the code are you using?
Both Vulkan 1.0 and 1.1 say:

Any variable decorated with Position must be declared as a four-component vector of 32-bit floating-point values.

That seems clear to me.  It should fail validation in the way you report.

Perhaps the Vulkan spec is wrong / out of date?

For the record:
 OpDecorate %3 BuiltIn Position
...
      %31 = OpTypeFloat 32
      %32 = OpTypeVector %31 4
...
      %34 = OpTypeInt 32 0
      %35 = OpConstant %34 3
...
      %48 = OpTypeArray %32 %35
      %49 = OpTypePointer Input %48

        %3 = OpVariable %49 Input

Wait, I ran the validator incorrectly.  We have to specify --target-env vulkan1.1 to get the error.
So:  Yes, I get the error.  Also, the validator is implementing the Vulkan spec as-is.
I understand there is a Khronos-internal issue related to this rule. https://gitlab.khronos.org/vulkan/vulkan/issues/1067

Generally, these rules intended as "after removing the level implicit arrayness that some interfaces add, the type must....".  Maybe there is a place this needs to be more clear?  But, obviously, in practice, for these interfaces that add a level of arrayness, one cannot give an error because the level of arrayness is present.

The competing statement is:

Tessellation control shader per-vertex output variables and blocks, and tessellation control, tessellation evaluation, and geometry shader per-vertex input variables and blocks are required to be declared as arrays, with each element representing input or output values for a single vertex of a multi-vertex primitive. For the purposes of interface matching, the outermost array dimension of such variables and blocks is ignored.

I'm sympathetic if somehow not everything is hooked up right the spec, and that can be clarified, but I don't think there is any question the spec expects this.

Per-vertex don't include per-patch. The ones in question can only be non-per-patch geometry in, tess-control in/out, and tess-evaluation in.  Original/core Vulkan only has the four Position, PointSize, ClipDistance, and CullDistance, but several extensions expand the list.
PrimitiveId, InvocationId, PatchVertices, Layer, TessCoord, and ViewportIndex are not treated this way.


Update template of qqp dataset

Previously, the template for qqp was:
doc_to_text: "\nSentence 1: {{sentence1}}\nSentence 2: {{sentence2}}\nAnswer:"
This would lead to errors:

I printed the log information for the apply_template function. I then realized the error was due to a mismatch between the template and doc, so I fixed it and now everything works normally.
def apply_template(template: str, doc: dict) -> str:
    rtemplate = env.from_string(template)
    return rtemplate.render(**doc)

doc_to_text: "\nSentence 1: {{question1}}\nSentence 2: {{question2}}\nAnswer:"

The test command:
CUDA_VISIBLE_DEVICES=0 lm_eval --model hf
--model_args pretrained=gpt2-large
--tasks qqp
--batch_size auto
--output_path ./eval_out/gpt2-large
--use_cache ./eval_cache/gpt2-large

 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.


Make scratch domain name configurable

Dendron allows you to create a scratch note. Defaults to scratch.* but this should be a configuration. See relevant code here

shipped in f26fb61


Cryostat /api/v1/clienturl was renamed to /api/v1/notifications_url

https://github.com/cryostatio/cryostat/pull/537

I think this is only used for health check purposes, perhaps it makes sense to use /health instead.


Incremental normalization?

According to its documentation, text-icu's collation algorithm uses incremental normalization.  This is very helpful in collation:  when you're comparing two strings, the decision about how to order them is generally one you can make after the first few characters, so no need to normalize the whole thing.
Could unicode-transforms provide a function that does this?  For my purposes, an ideal interface would be
normalizeStreaming :: NormalizationMode -> Text -> [Int]

where the Ints are code points, and the list is produced lazily.

The best way to deal with this would be to use stream based normalization. Streamly is going to support that using a signature like this:
normalize :: (IsStream t, Monad m) => NormalizationMode -> t m Char -> t m Char

See this PR https://github.com/composewell/streamly/pull/698 for a working implementation of the above. We are also going to break streamly into several packages so that it can be a lightweight dependency and also have a streamly-unicode package for stream based unicode algorithms (see https://github.com/composewell/streamly/issues/533).

To work with text we can convert it to stream, normalize it and convert the stream back to text. In fact, with that we can work with any streamable type, not just text.

That sounds very good!


Remove hard-coded references to "master" branch

[Copied from Gitlab.]
The user of RoboTA Core should be able to set the name of the development branch, rather than having this hard-coded.

configreader.py already takes in a configuration variable for the default branch to use. However, many parts of the codebase still use "master" hardcoded (e.g. commit.py:106) or use "master" as default instead of "main". This should be changed to use the config variable. It is also a good idea to try to get further hardcoded URL's from gitlab's API instead.


Experiment with smaller Zustand state updates

The middleware asks Zustand to replace the state with a new state object. This feels inefficient, but I need to test to see which is faster: full replacement or using partial updates.

Had a peek at the Zustand source code and learned that, in fact, the larger updates are more efficient. This is due to the replace parameter bypassing an Object.assign() call when set to true. This skips straight to re-assigning the internal state to the new state.
The difference in performance is a constant time operation (reassignment) versus a linear time operation (Object.assign()).