text:
option 'close_connections_on_host_set_change' can't destroy long-live stream

I use envoy to proxy long-live server-side grpc stream, the upstream cluster option 'close_connections_on_host_set_change' has been set to 'true'. When hosts changed, the grpc stream can't destroy.

IIRC this setting invokes draining on the connection which tries to do it gracefully vs. forceful closure. for a long lived stream there would never be any graceful closure so I don't think this is going to work, but cc @nezdolik who can provide more info.

Hello @chijinxina,  'close_connections_on_host_set_change' most likely would not work for long-lived grpc streams. Could you please provide sample Envoy config for me to confirm.

I know this is a long-dead issue, but I've just recently run in to an issue where I would like to terminate long-lived gRPC streams when my Cluster changes. Are there any plans to add a forcefully_close_connections_on_host_set_change?
Having a function that is identical to this, but if it fails to gracefully terminate the connection withing a certain time-frame (ideally configurable as well), it forcefully closes the connection so it can be re-established.

I know this is a long-dead issue, but I've just recently run in to an issue where I would like to terminate long-lived gRPC streams when my Cluster changes. Are there any plans to add a forcefully_close_connections_on_host_set_change?

This has come up several times, and I'm guessing we have tracking issues about this but I haven't searched yet (assuming it hasn't already been implemented). I think it's a reasonable feature to add and I would open an issue if it's not already tracked. cc @snowp

@mattklein123 I'm glad to hear I'm not the only one running in to this problem!
I am currently in the process of trying to interrogate the Envoy xDS for Cluster updates so I can manually terminate my gRPC streams in my application, which does seem like a hack to me, and it's proving quite difficult to get started getting cluster state/config from Envoy, not setting it. All guides out there are for Control Plane implementations.
I feel like a way for Envoy to terminate long-running gRPC streams would be a very good addition.

authors: chijinxina leosunmo mattklein123 nezdolik


text:
http: stream error on invalid HTTP messaging for HTTP/1

In this PR:

HTTP/1.1 API is extended to allow configuration of behaviour on invalid HTTP messaging;
codec interface is extended so that the HCM can pull value for new config from it;
logic for behaviour on invalid HTTP messaging added in #11748 is updated to account for the new HTTP/1.1 option.
Helps with #9846.

Risk Level: high (HCM changes)
Testing: new unit tests and integration tests
Docs Changes: n/a
Release Notes: updated
Runtime guard: envoy.reloadable_features.hcm_stream_error_on_invalid_message (already introduced by #11748)

@alyssawilk I think this is ready for another look.
One question though: do we want to update the old code path for sending errors from the codec as well (https://github.com/envoyproxy/envoy/blob/bfab9edadb1937bc1eac3e50f978175b3ddcdebc/source/common/http/http1/codec_impl.cc#L969) to terminate the connection or not based on the new config param?

Addressed some of the feedback + weird code formatting. Still need to work on adding integration tests though.

@alyssawilk I added integration tests and addressed most of your feedback: could you please take another look if you get a chance?

hm, looks like CI is still unhappy.  Mind taking a look at that first?

@rulex123 Alyssa is out for another 1.5 weeks. Do you want me to take a look or do you want to wait for her to come back? Up to you.
/wait-any

@mattklein123 Would be great if you could take a look!

@alyssawilk ping: any further feedback beside what Matt said?

@alyssawilk no worries! I have reworked the code a bit to take into account your cross-codec-behavior comment. Please take another look when you get a chance :-)

@alyssawilk ping: any further comments or feedback? Would love to wrap this up this week if possible :-)

over to Matt for API sign off.

source: gharchive/pull-request

repo: envoyproxy/envoy

license_type: permissive

license_source: github-api


text:
Add release note for PR #17673

Signed-off-by: Tianyu Xia tyxia@google.com

Sorry I thought you already implemented it since there was code in the other PR. If this is the case the other PR should have had the "not implemented hide" annotation. It's not a big deal but something to remember next time. Thank you.

Got it. Thanks for information! Will do next time.
The code there is most for validation that cannot to be done in PGV check (due to the fact that we cannot promote regular filed into oneof).


text:
[Geoip+ci] Fix flaky release check in ci

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
Partially Fixes #35829
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

cc @phlax


text:
kafka: 2.4.0 support - add support for new data types added in 2.4

Description:
Provide codec support for data types added in 2.4.0.
In detail, we Kafka 2.4 added some new things in protocol:

variable-length uint32
new compact strings / byte-arrays / generic arrays that keep length using this varint, instead of fixed-len types
predefined "tagged_fields" structure (that's very similar to generic array, but doesn't do the +1 length)

This PR should not have any impact on existing codebase, just the volume of changes is IMHO too large to comfortably review.
Relates to https://github.com/envoyproxy/envoy/issues/2852
These deserializers are working with 2.4 request types (they are not part of this PR as I'm still working on that and want to make another PR with them).
Risk Level: Low
Testing: Automated unit tests; embedded integration test (ensures compatibility with 2.3 client); manual tests with 2.3 client
Docs Changes: N/A
Release Notes: N/A

/wait

/wait

/wait

/wait-any

@adamkotwasinski are you waiting on a review here, or still iterating?

@junr03 still iterating, didn't expect that build failure!
will assign the reviewer when I'm done

/wait

/wait

/wait-any

/wait-any

This PR should not have any impact on existing Kafka protocol coverage, as the new classes are not yet referenced from (generated) requests.h / responses.h protocol files.
It can be noticed that there's some duplication here, as we are handling very similar data types all the time (e.g. (nullable) (compact) byte array). I will want to revisit this place to refactor it later, to avoid PR noise.
/assign @mattklein123
/assign @junr03

@adamkotwasinski will the dependency updates come in a follow up PR?

https://github.com/dpkp/kafka-python/releases/tag/2.0.0
https://github.com/apache/kafka/releases/tag/2.4.0
https://github.com/apache/kafka/releases/tag/2.4.0

@moderation yes - when I'm done with https://github.com/envoyproxy/envoy/pull/10000

authors: adamkotwasinski junr03 moderation


text:
How is the envoy gateway kubernetes service name defined?

I would like to be able to set a name for the generated envoy service, or at least make sure the name doesn't ever change.
➜ kubectl get service -n envoy-gateway-system
NAME                                     TYPE        CLUSTER-IP       EXTERNAL-IP   PORT(S)               AGE
envoy-envoy-gateway-system-eg-656e766f   ClusterIP   172.20.249.184   <none>        80/TCP                27d
envoy-gateway                            ClusterIP   172.20.94.26     <none>        18000/TCP,18001/TCP   28d
envoy-gateway-metrics-service            ClusterIP   172.20.0.145     <none>        8443/TCP              28d

Here the name of my eg service, is envoy-envoy-gateway-system-eg-656e766f What components make up this service name? Will it always be the same value?

hi @bholland-bh, currently it is generated by this function
https://github.com/envoyproxy/gateway/blob/71c09f285fcc88ddaf234a7c3864263d8a5bbbd0/internal/infrastructure/kubernetes/proxy/resource.go#L54-L57
It will be the same value, its format consists of a prefix(envoy), gateway namespace, gateway name, and 8 characters of the hashed name.

you'd better query by labels.

hi @bholland-bh, currently it is generated by this function
https://github.com/envoyproxy/gateway/blob/71c09f285fcc88ddaf234a7c3864263d8a5bbbd0/internal/infrastructure/kubernetes/proxy/resource.go#L54-L57
It will be the same value, its format consists of a prefix(envoy), gateway namespace, gateway name, and 8 characters of the hashed name.

Thank you so much for the direct link.

you'd better query by labels.

Due to requirements beyond my control, I've got to tuck Envoy Gateway behind EKS ALB ingress. Which uses service names as the destination.
I'm interacting with the resources via labels, but the ALB uses service names
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  annotations:
    alb.ingress.kubernetes.io/backend-protocol: HTTP
    alb.ingress.kubernetes.io/certificate-arn: <arn>
    alb.ingress.kubernetes.io/listen-ports: '[{"HTTP": 80}, {"HTTPS":443}]'
    alb.ingress.kubernetes.io/scheme: internet-facing
    alb.ingress.kubernetes.io/ssl-redirect: '443'
    alb.ingress.kubernetes.io/success-codes: 200-499
    alb.ingress.kubernetes.io/tags: environment=dev,Name=eg-public,application=eg,client=shared
    alb.ingress.kubernetes.io/target-type: ip
    alb.ingress.kubernetes.io/waf-acl-id: <WAF>
    kubernetes.io/ingress.class: alb
  name: eg-ingress
  namespace: envoy-gateway-system
  labels:
    environment: dev
    team: blue
spec:
  defaultBackend:
    resource:
      kind: Service
      name: envoy-envoy-gateway-system-eg-656e766f

I think this is a valid use case
need someone from the community to brainstorm on the drawbacks of exposing this knob apart from naming collisions (which should fall on the user)

I also want to put Envoy Gateway behind EKS ALB ingress. And I encounter the same problem. Is there any update here?

ptal @envoyproxy/gateway-maintainers
I'm a +1 on this, it makes it easier to author infra intent

+1 even for multiple envoys per gateway deployments

+1

can be solved using the newly added patch field in https://github.com/envoyproxy/gateway/issues/2716
apiVersion: http://gateway.envoyproxy.io/v1alpha1
kind: EnvoyProxy
metadata:
  name: custom-proxy-config
  namespace: envoy-gateway-system
spec:
  logging:
    level:
      default: warn
  provider:
    kubernetes:
      envoyService:
        patch:
          value:
            metadata:
              name: custom-name
        type: ClusterIP
    type: Kubernetes

When i specify a name, my gateway doesn't standup.

Sure thing. I applied
apiVersion: http://gateway.envoyproxy.io/v1alpha1
kind: EnvoyProxy
metadata:
  name: custom-proxy-config
  namespace: envoy-gateway-system
spec:
  logging:
    level:
      default: warn
  provider:
    kubernetes:
      envoyService:
        patch:
          value:
            metadata:
              name: custom-name
        type: Loadbalancer
    type: Kubernetes

The service standup fine with the propername. When I go to deploy the gateway, it doesn't work
I get
❯ kubectl get gateway
NAME            CLASS           ADDRESS   PROGRAMMED   AGE
envoy   envoy             False        3m52s

I revert the changes and it creates the standard loadblancer with the long name
❯ kubectl get gateway
NAME            CLASS           ADDRESS         PROGRAMMED   AGE
envoy   envoy   13.64.208.227   True         34m

ah yes, fixing this in https://github.com/envoyproxy/gateway/pull/3537#issuecomment-2150932097

source: gharchive/issue

authors: arkodg bholland-bh cnvergence davem-git mingshun quyenhoang96 zirain

repo: envoyproxy/gateway


text:
create a release guide

Description:

Describe the issue.

Create a release guide listing all the tasks needed to release a new EG version.
Releasing a new EG version includes a number of tasks scattered in different parts of the repo, for example:

Create a release branch
generate a release note
bump the version in the VERSION file
...

It can be useful to create a guide so the next release manager can easily pick up the job.
[optional Relevant Links:]

Any extra documentation required to understand the issue.

https://gateway.envoyproxy.io/contributions/releasing/ may help?

https://gateway.envoyproxy.io/contributions/releasing/ may help?

Ha, didn't notice that.

authors: zhaohuabing zirain


text:
Protolock plugin for PGV annotations

Protolock is a tool for protecting against breaking changes to proto files. By default, Protolock only validates changes to proto messages and RPCs. It does not understand proto options like PGV.
While Protolock does not natively understand PGV, it does support plugins. We can write a plugin that teaches Protolock how to distinguish breaking changes in PGV annotations.

This is an excellent idea. First order of business would be to define what it means for a change to an annotation to be considered breaking.

A degenerate compatibility check would say that any change to any validation rule is a breaking change.
A more nuanced compatibility check would say that removing a rule is OK and broadening an existing rule is OK, but adding new rules is a breaking change.

authors: rmichela rodaine

repo: envoyproxy/protoc-gen-validate

license_source: bigquery


text:
Define list of notifications show in the app

[ ] Review the user flow to define what touchpoints require a notification.
[ ] Identify notifications categories.
[ ] Define design per notification categories.

Own: @leisterfrancisco
Sup: @steph222
Lifebank-GeneralFlows.pdf

Notification Category
Actions

Welcome to Lifebank
Profile edited correctly

Token Transaction

Offer redeem correctly (donor, sponsor)

News

New offer

Urgency

authors: ldrojas leisterfrancisco

repo: eoscostarica/lifebank

license: MIT


text:
[BUG]: Cannot find model

When python pilot/server/dbgpt_server.py
| WARNING | sentence_transformers.SentenceTransformer | No sentence-transformers model found with name /usr/local/src/DB-GPT/models/text2vec-large-chinese. Creating a new one with MEAN pool
Traceback (most recent call last):
File "/usr/local/src/DB-GPT/pilot/server/dbgpt_server.py", line 117, in
from pilot.server.llmserver import worker
File "/usr/local/src/DB-GPT/pilot/server/llmserver.py", line 123, in
model_path = LLM_MODEL_CONFIG[CFG.LLM_MODEL]
KeyError: 'chinese-alpaca-2-7b-hf'
I have download text2vec-large-chinese and chinese-alpaca-2-7b-hf in models/ and set LLM_MODEL=chinese-alpaca-2-7b-hf in .env.

When python pilot/server/dbgpt_server.py
| WARNING | sentence_transformers.SentenceTransformer | No sentence-transformers model found with name /usr/local/src/DB-GPT/models/text2vec-large-chinese. Creating a new one with MEAN pool
Traceback (most recent call last): File "/usr/local/src/DB-GPT/pilot/server/dbgpt_server.py", line 117, in from pilot.server.llmserver import worker File "/usr/local/src/DB-GPT/pilot/server/llmserver.py", line 123, in model_path = LLM_MODEL_CONFIG[CFG.LLM_MODEL] KeyError: 'chinese-alpaca-2-7b-hf'
I have download text2vec-large-chinese and chinese-alpaca-2-7b-hf in models/ and set LLM_MODEL=chinese-alpaca-2-7b-hf in .env.

now chinese-alpaca-2-7b-hf llm not support, you can switch other llm_model like chatglm2-6b or vicuna-13b-v1.5

When python pilot/server/dbgpt_server.py
| WARNING | sentence_transformers.SentenceTransformer | No sentence-transformers model found with name /usr/local/src/DB-GPT/models/text2vec-large-chinese. Creating a new one with MEAN pool
Traceback (most recent call last): File "/usr/local/src/DB-GPT/pilot/server/dbgpt_server.py", line 117, in from pilot.server.llmserver import worker File "/usr/local/src/DB-GPT/pilot/server/llmserver.py", line 123, in model_path = LLM_MODEL_CONFIG[CFG.LLM_MODEL] KeyError: 'chinese-alpaca-2-7b-hf'
I have download text2vec-large-chinese and chinese-alpaca-2-7b-hf in models/ and set LLM_MODEL=chinese-alpaca-2-7b-hf in .env.

now chinese-alpaca-2-7b-hf llm not support, you can switch other llm_model like chatglm2-6b or vicuna-13b-v1.5

Thank you @Aries-ckt ! Why text2vec-large-chinese cannot be found?

text2vec-large-chinese cannot be found is solved.
Because missing some files.

@8188
I followed your above operations and copied those files, but I still get the following error. What’s going on?
docker logs dbgpt -f
==========
== CUDA ==
CUDA Version 11.8.0
Container image Copyright (c) 2016-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
This container image and its contents are governed by the NVIDIA Deep Learning Container License.
By pulling and using the container, you accept the terms and conditions of this license:
https://developer.nvidia.com/ngc/nvidia-deep-learning-container-license
A copy of this license is made available in this container at /NGC-DL-CONTAINER-LICENSE for your convenience.
2023-12-27 02:30:33 edcf08a3ad08 numexpr.utils[1] INFO Note: NumExpr detected 20 cores but "NUMEXPR_MAX_THREADS" not set, so enforcing safe limit of 8.
2023-12-27 02:30:33 edcf08a3ad08 numexpr.utils[1] INFO NumExpr defaulting to 8 threads.
2023-12-27 02:30:34 edcf08a3ad08 matplotlib.font_manager[1] INFO generated new fontManager
2023-12-27 02:30:35 edcf08a3ad08 dbgpt.component[1] INFO Register component with name dbgpt_thread_pool_default and instance: <dbgpt.util.executor_utils.DefaultExecutorFactory object at 0x7f02edbf3790>
2023-12-27 02:30:35 edcf08a3ad08 dbgpt.component[1] INFO Register component with name dbgpt_model_controller and instance: <dbgpt.model.cluster.controller.controller.ModelControllerAdapter object at 0x7f03551cd9c0>
2023-12-27 02:30:35 edcf08a3ad08 dbgpt.component[1] INFO Register component with name dbgpt_agent_hub and instance: <dbgpt.agent.controller.ModuleAgent object at 0x7f02ed9be1d0>
2023-12-27 02:30:35 edcf08a3ad08 dbgpt.app.initialization.embedding_component[1] INFO Register local LocalEmbeddingFactory
2023-12-27 02:30:35 edcf08a3ad08 dbgpt.model.cluster.worker.embedding_worker[1] INFO [EmbeddingsModelWorker] Parameters of device is None, use cuda
2023-12-27 02:30:35 edcf08a3ad08 dbgpt.app.initialization.embedding_component[1] INFO
=========================== EmbeddingModelParameters ===========================
model_name: text2vec
model_path: /app/models/text2vec-large-chinese
device: cuda
normalize_embeddings: None
======================================================================
=========================== WebServerParameters ===========================
host: 0.0.0.0
port: 5000
daemon: False
controller_addr: None
model_name: vicuna-7b-v1.5
2023-12-27 02:30:35 edcf08a3ad08 sentence_transformers.SentenceTransformer[1] INFO Load pretrained SentenceTransformer: /app/models/text2vec-large-chinese
share: False
remote_embedding: False
log_level: INFO
light: False
log_file: dbgpt_webserver.log
tracer_file: dbgpt_webserver_tracer.jsonl
tracer_storage_cls: None
disable_alembic_upgrade: False
======================================================================
Traceback (most recent call last):
File "/usr/local/bin/dbgpt", line 8, in
sys.exit(main())
File "/app/dbgpt/cli/cli_scripts.py", line 149, in main
return cli()
File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1157, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1078, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "/app/dbgpt/util/parameter_utils.py", line 340, in wrapper
return func(*args, **kwargs)
File "/app/dbgpt/app/_cli.py", line 21, in start_webserver
run_webserver(WebServerParameters(**kwargs))
File "/app/dbgpt/app/dbgpt_server.py", line 228, in run_webserver
param = initialize_app(param)
File "/app/dbgpt/app/dbgpt_server.py", line 154, in initialize_app
initialize_components(param, system_app, embedding_model_name, embedding_model_path)
File "/app/dbgpt/app/component_configs.py", line 41, in initialize_components
_initialize_embedding_model(
File "/app/dbgpt/app/initialization/embedding_component.py", line 26, in _initialize_embedding_model
system_app.register(
File "/app/dbgpt/component.py", line 121, in register
instance = component(self, *args, **kwargs)
File "/app/dbgpt/app/initialization/embedding_component.py", line 70, in init
self._model = self._load_model()
File "/app/dbgpt/app/initialization/embedding_component.py", line 103, in _load_model
return loader.load(self._default_model_name, model_params)
File "/app/dbgpt/model/cluster/embedding/loader.py", line 39, in load
return HuggingFaceEmbeddings(**kwargs)
File "/usr/local/lib/python3.10/dist-packages/langchain_community/embeddings/huggingface.py", line 65, in init
self.client = sentence_transformers.SentenceTransformer(
File "/usr/local/lib/python3.10/dist-packages/sentence_transformers/SentenceTransformer.py", line 77, in init
raise ValueError("Path {} not found".format(model_name_or_path))
ValueError: Path /app/models/text2vec-large-chinese not found

@8188 I followed your above operations and copied those files, but I still get the following error. What’s going on? docker logs dbgpt -f
==========
== CUDA ==
CUDA Version 11.8.0
Container image Copyright (c) 2016-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
This container image and its contents are governed by the NVIDIA Deep Learning Container License. By pulling and using the container, you accept the terms and conditions of this license: https://developer.nvidia.com/ngc/nvidia-deep-learning-container-license
A copy of this license is made available in this container at /NGC-DL-CONTAINER-LICENSE for your convenience.
2023-12-27 02:30:33 edcf08a3ad08 numexpr.utils[1] INFO Note: NumExpr detected 20 cores but "NUMEXPR_MAX_THREADS" not set, so enforcing safe limit of 8. 2023-12-27 02:30:33 edcf08a3ad08 numexpr.utils[1] INFO NumExpr defaulting to 8 threads. 2023-12-27 02:30:34 edcf08a3ad08 matplotlib.font_manager[1] INFO generated new fontManager 2023-12-27 02:30:35 edcf08a3ad08 dbgpt.component[1] INFO Register component with name dbgpt_thread_pool_default and instance: <dbgpt.util.executor_utils.DefaultExecutorFactory object at 0x7f02edbf3790> 2023-12-27 02:30:35 edcf08a3ad08 dbgpt.component[1] INFO Register component with name dbgpt_model_controller and instance: <dbgpt.model.cluster.controller.controller.ModelControllerAdapter object at 0x7f03551cd9c0> 2023-12-27 02:30:35 edcf08a3ad08 dbgpt.component[1] INFO Register component with name dbgpt_agent_hub and instance: <dbgpt.agent.controller.ModuleAgent object at 0x7f02ed9be1d0> 2023-12-27 02:30:35 edcf08a3ad08 dbgpt.app.initialization.embedding_component[1] INFO Register local LocalEmbeddingFactory 2023-12-27 02:30:35 edcf08a3ad08 dbgpt.model.cluster.worker.embedding_worker[1] INFO [EmbeddingsModelWorker] Parameters of device is None, use cuda 2023-12-27 02:30:35 edcf08a3ad08 dbgpt.app.initialization.embedding_component[1] INFO
=========================== EmbeddingModelParameters ===========================
model_name: text2vec model_path: /app/models/text2vec-large-chinese device: cuda normalize_embeddings: None
======================================================================
=========================== WebServerParameters ===========================
host: 0.0.0.0 port: 5000 daemon: False controller_addr: None model_name: vicuna-7b-v1.5 2023-12-27 02:30:35 edcf08a3ad08 sentence_transformers.SentenceTransformer[1] INFO Load pretrained SentenceTransformer: /app/models/text2vec-large-chinese share: False remote_embedding: False log_level: INFO light: False log_file: dbgpt_webserver.log tracer_file: dbgpt_webserver_tracer.jsonl tracer_storage_cls: None disable_alembic_upgrade: False
======================================================================
Traceback (most recent call last): File "/usr/local/bin/dbgpt", line 8, in sys.exit(main()) File "/app/dbgpt/cli/cli_scripts.py", line 149, in main return cli() File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1157, in call return self.main(*args, **kwargs) File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1078, in main rv = self.invoke(ctx) File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1688, in invoke return _process_result(sub_ctx.command.invoke(sub_ctx)) File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1688, in invoke return _process_result(sub_ctx.command.invoke(sub_ctx)) File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 1434, in invoke return ctx.invoke(self.callback, **ctx.params) File "/usr/local/lib/python3.10/dist-packages/click/core.py", line 783, in invoke return __callback(*args, **kwargs) File "/app/dbgpt/util/parameter_utils.py", line 340, in wrapper return func(*args, **kwargs) File "/app/dbgpt/app/_cli.py", line 21, in start_webserver run_webserver(WebServerParameters(**kwargs)) File "/app/dbgpt/app/dbgpt_server.py", line 228, in run_webserver param = initialize_app(param) File "/app/dbgpt/app/dbgpt_server.py", line 154, in initialize_app initialize_components(param, system_app, embedding_model_name, embedding_model_path) File "/app/dbgpt/app/component_configs.py", line 41, in initialize_components _initialize_embedding_model( File "/app/dbgpt/app/initialization/embedding_component.py", line 26, in _initialize_embedding_model system_app.register( File "/app/dbgpt/component.py", line 121, in register instance = component(self, *args, **kwargs) File "/app/dbgpt/app/initialization/embedding_component.py", line 70, in init self._model = self._load_model() File "/app/dbgpt/app/initialization/embedding_component.py", line 103, in _load_model return loader.load(self._default_model_name, model_params) File "/app/dbgpt/model/cluster/embedding/loader.py", line 39, in load return HuggingFaceEmbeddings(**kwargs) File "/usr/local/lib/python3.10/dist-packages/langchain_community/embeddings/huggingface.py", line 65, in init self.client = sentence_transformers.SentenceTransformer( File "/usr/local/lib/python3.10/dist-packages/sentence_transformers/SentenceTransformer.py", line 77, in init raise ValueError("Path {} not found".format(model_name_or_path)) ValueError: Path /app/models/text2vec-large-chinese not found

hi，I encounter the same problem  as you said，have you solved it?

repo: eosphoros-ai/DB-GPT


text:
Proxying AI model images via chat

Description
Currently, DIAL have two components named "chat" and "chat-themes." Both of these containers need to be exposed externally. When trying to expose only the chat container externally, the images for the models stop working.
What is expected:

THEMES_CONFIG_HOST - contain internal link
only chat container exposed to internet

Use case/motivation
The requirement is to have the "chat" container proxy the images it received from "chat-themes," rather than simply redirecting to it.
Related issues
No response
Confidential information

[X] I confirm that do not share any confidential information

It was configured by the team on staging environment successfully

The comment was added to documentation

authors: VolhaBazhkova alexey-ban sr-remsha

repo: epam/ai-dial-chat


text:
Add local docker-compose deployment

Currently, there is no any way to run BadgerDoc in local environment without k8s.
This task is to create single docker-compose file, which may run BadgerDoc with all dependencies locally
What must be done:

[x] Single configuration file for all microservices  (PR https://github.com/epam/badgerdoc/pull/448)
[x] Single docker-compose file for all microservices and dependensies (PR https://github.com/epam/badgerdoc/pull/448)
[x] Front-end (@khyurri)
[x] Documentation: how to setup local environment

Microservices

[x] Annotation (#701)
[x] Users (PR https://github.com/epam/badgerdoc/pull/448)
[x] Assets (@khyurri )
[x] Convert (@spiridonovfed) PR https://github.com/epam/badgerdoc/pull/699
[x] Processing (@khyurri)
[x] Jobs (@khyurri)
[x] Taxonomy

Postponed

[ ] Pipelines (postponded)
[ ] Scheduler (postponed)
[ ] Models (postponed)
[ ] Search (postponed)

All required services were migrated.

repo: epam/badgerdoc


text:
Support versioning in ket format

Is your feature request related to a problem? Please describe.
Adding breaking changes to the ket format.
Describe the solution you'd like
The main goal of the task is to add a version field to the ket format.
We would like to have the ability to add breaking changes to the ket format but keep supporting previous formats which are already used by our users.
We will use SemVer for versioning.

1 to this suggestion, we have a load of data from Ketcher v2.3 and we updated to v2.5 and suddenly all our old reactions wouldn't load. Eventually figured out it was due to schema change in 'arrow' node in the Ket format.

Please include breaking changes to schema in the Release Notes.
Would be even better to provide a converter to update all data.
Best of all would be if Ketcher itself was backwards compatible with older schemas so we didn't have to worry about it at all.

Thanks!

repo: epam/ketcher


text:
🛑 Help Desk is down

In a5fd947, Help Desk ($HELP_DESK) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Help Desk is back up in 3ae73b3.

repo: epederson/Eastbrook-Upptime


text:
negation operator is needed to make json schema working correctly

It seems currently the json schema constrain is not working correctly yet
by loading the example student grammar
 # Load grammar
 with open("examples/grammars/custom_json_grammars/grammars/student.ebnf", "r") as file:
      grammar_str = file.read()

And run a simple generation we get :
{ "name": "\\"\\"", "age": 20, "courses": [ "\\"\\"", "\\"\\"", "\\"\\"", "\\"\\"", "\\"\\"", "\\"\\"", "\\"\\"", "\\"\\"", "\\"\\"", "\\"\\"",']

The origin of the problem is that the grammar converted from the json schema, i.e. examples/grammars/custom_json_grammars/grammars/student.ebnf contains a usage of negation operator ^ at line 4, i.e. char ::= [^"\\]
Since we haven't yet added the support for negation operator , this simply doesn't work and only allows to generate escaped backslashes.   #21
A quick fix is to replace it with char ::= [A-z]
And we get
{ "name": "John", "age": 20, "courses": [ "Math", "English" ] }

As a follow up work of #21 , we should now be able to support json schema. Negation was the only missing operator.  See here for more details of how to use json schema

solved in #21

repo: epfl-dlab/transformers-CFG


text:
Support panics as expressions.

Currently, only assertions work for generating panics because the correct type parameter is not inferred. Hence, the following fails:
fn is_it_123(a: i32) -> bool {
  match a {
    123 => true,
    _ if a < 0 => panic!("This is not allowed"),
    _ => false,
  }
}

Because the panic! is extracted as error[Unit]("This is not allowed") which Stainless' type checker doesn't accept.

If Stainless checks whether errors are reachable, then we could just try to give the Nothing type.

repo: epfl-lara/rust-stainless


text:
Using Extensions

Ok, I feel a bit silly about this, but how would I bring in extension into this? I assumed that dumping the relevant css and js into the project would be enough to give access, but that doesn't seem to be the case. I'm specifically looking at FixedHeader, but I'm sure with an example for one, I could get the rest working.

Sorry for the late reply, I didn't get email notifications about this repo for a long time (I think I've fixed it now) and seem to have missed a bunch of questions.
Can you provide a link to the extension you were trying to use and preferably a link to a public GitHub repo with an attempt to integrate the extension?
It's possible you just need to put the extension in meteor's "client/compatibility" directory

Hi @ephemer ,
I would like to integrate datatables editor extension with keytable : https://editor.datatables.net/examples/extensions/keyTable.html
It would be great if you can have this extension (editor) integrated with this your package.
I will also try to check and see if this work with meteor's "client/compatibility" directory as you mentioned in your post.
Thanks and Best Regards,
Manu

Hi @manusharma1 ,
Let me know how you go with the compatibility directory. It's possible we will need to update the datatables version in the package (or better: swap it out so it's independently updatable via npm). Good luck

Hi @ephemer
Thanks for your reply, I will try it and get back on this soon.
Thanks and Best Regards,
Manu

Closing this for now, we can re-open it if adding extensions in the compatibility directory doesn't do the trick

authors: Architecty ephemer manusharma1

repo: ephemer/meteor-reactive-datatables

license: mit


text:
Make graphs transparent

Hi @m0ksem, can you assign me this issue. I'd like to work on it

@imjwang, please have a look at vuestic-ui setHSLAColor method. This can be helpful for this issue.
https://github.com/epicmaxco/vuestic-ui/blob/073e74f17c8f350c4c46443659b140711bd6e667/packages/ui/src/services/color-config/color-config.ts#L61

I'm having trouble getting project set up with npm run serve
Any ideas?
 ERROR  Error: Rule can only have one resource source (provided resource and test + include + exclude) in {
  "exclude": [
    null
     ],
  "use": [
      {
      "loader": "C:\\Users\\jeffr\\Documents\\repos\\vuestic-admin\\node_modules\\cache-loader\\dist\\cjs.js",
      "options": {
        "cacheDirectory": "C:\\Users\\jeffr\\Documents\\repos\\vuestic-admin\\node_modules\\.cache\\babel-loader",
        "cacheIdentifier": "870faf2e"
      },
      "ident": "clonedRuleSet-38.use[0]"
     },
      {
      "loader": "C:\\Users\\jeffr\\Documents\\repos\\vuestic-admin\\node_modules\\babel-loader\\lib\\index.js",
      "options": "undefined",
      "ident": "undefined"
      }
      ]
      }

nvm, it's working with yarn

Npm install works fine for me on 6.14.10.

Found that this issue was caused because the 4.47.0 webpack version. Rollback to 4.46.0 should fix npm.

@m0ksem Seems like they are already transparent?

We're replacing charts in new version.

authors: asvae imjwang m0ksem

repo: epicmaxco/vuestic-admin


text:
Update gleam import and web export to support cities

General

read_csv_smart optionally prefers larger regions
Fixed broken pandas Region display
Fix DisplayName (contained "nan" for some basins)
Update gleam import
Add overwrite option
Make work for gleam basins (cities)
Ignore missing initial_compartments
Update web export
Mathij's many updates from #44 (cherry-picked)
Ignore cities missing M49
Ignore empty TZ list

@mathijshenquet I would like to merge this to master. Could you take a look at changes to export (mostly yours, very few mine) and check that they do not break the main workflow, and finish any cleanups etc.?
(Or let me know that this is only throwaway code - I also added some useful/general features/fixes that would need separation on my side.)

@mathijshenquet I would like to merge this to master. Could you take a look at changes to export (mostly yours, very few mine) and check that they do not break the main workflow, and finish any cleanups etc.?
(Or let me know that this is only throwaway code - I also added some useful/general features/fixes that would need separation on my side.)

Note: two commit 4122e7da2b1dc86f4157c3ee2538a2bf8eab2b83 should be also part of this but was pushed to master by mistake. (But it is functional on its own)

repo: epidemics/epimodel


text:
Does Not Switch Between Active Spaces.

With brave when tab search is used and the tab is on a space that is currently not selected it will not bring you to that space.

This was fixed in https://github.com/epilande/alfred-browser-tabs/pull/49 and has been released with 🔖 v1.0.6

repo: epilande/alfred-browser-tabs


text:
Removed appchart.Controller

Ref: #2247

Codecov Report
Patch coverage has no change and project coverage change: -48.54 :warning:

Comparison is base (129c37d) 57.14% compared to head (f58f204) 8.61%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #2248       +/-   ##
==========================================
- Coverage   57.14%   8.61%   -48.54%
==========================================
  Files         183      96       -87
  Lines       15320   10820     -4500
==========================================
- Hits         8755     932     -7823
- Misses       5448    9832     +4384
+ Partials     1117      56     -1061

Flag
Coverage Δ

acceptance-api
?

acceptance-apps
?

acceptance-cli
?

unittests
8.61% <ø> (ø)

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

Impacted Files
Coverage Δ

internal/api/v1/router.go
36.66% <ø> (-63.34%)
:arrow_down:

... and 170 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter enrichman

repo: epinio/epinio


text:
в чем отличие derscription и lead во front matter?

Где показываются, как влияют на SEO? lead - возможно это только для блогов?

front matter description отображается как meta name="description". В принципе в SEO это еще может котироваться.
front matter lead просто рендерится на странице как обычный параграф. Его можно вообще не использовать и просто удалить из front matter (если нет планов на его использовать на страницах docs). Но для блога думаю он уместен.

Но для SEO - главное title, верно?

Но для SEO - главное title, верно?

Верно.

authors: epogrebnyak necelentano


text:
Create a "learningedge-config" eclipse project and ant task for easier dev config setup

Previously we just had a template folder which you copied and modified, it would be easier if there was a project and ant task which did most of the work for you.

To use:
ant generate-config

Import the eclipse project in "Dev/learningedge-config"

repo: equella/Equella


text:
[Update] Token Rework

Reworks the Token type to remove dynamic .decimals() normalization as this pattern has the potential to be more confusing to users than it is helpful to developers.

Renames Token to TokenOrEther18

Removes all dynamic decimal normalization from TokenOrEther18, maintains the ether support.

Removes .decimals() getters from each Token type, maintains .name() and .symbol() getters.
Adds ZERO constant to each Token type.
Adds .eq() comparator to each Token type.
Bumps root version to v0.1.3.

Added another isZero() function here as well, could you check that quick @arjun-io

repo: equilibria-xyz/root


text:
ci: increase timeout for e2e tests

What does this pull request change?

Increase timeout for tests to be 60 seconds (default is 30 seconds)

Why is this pull request needed?
Issues related to this change

@eoaksnes I added slowMo: 500 to the config, which seems to work. I have tested it quite a bit locally and it runs green every time now. However, this does slow down the test execution, especially in the CI where we only have 1 worker.

authors: eoaksnes roar-larsen

repo: equinor/dm-core-packages


text:
event cards are not outlined properly

There is some empty space on the right side of the event cards column which makes it look weird.
We should either use the full width or center align them.

duplicate of #1371 - yes I am totally awake

repo: equinor/energyvision


text:
As a editor I want to add a simple menu to satellite pages

Acceptance criteria

Satellite sites should have a simple menu in Studio instead of normal
Web application should render simple menu instead of normal in satellites

Steps to produce

Create a menu in Sanity for a satellite site.
See if it renders on the satellite site

Todo:

[x] Logo in header
[x] Remove static links in satellite
[x] Add link field at root

In order to ensure good quality we should (Keep the ones we need for this US)

[ ] Perform integration testing

[ ] Perform functional testing

[ ] Extra focus on usability

[ ] Test for accessibility

[ ] Perform user testing

[ ] Perform security testing

@BirteThornquist Lets discuss the simple menu design this week too

@nicholas-haggerty @meols ready to add content now

https://web-brazil-energyvision-dev.radix.equinor.com/

The css on desktop view is broken when the text is too long. Item is placed inside a grid that limits its width:

Due to this, on my screen resolution a line break is being introduced when hovering the item:

There is also a mismatch on the position of the Menu button when the menu is opened vs closed. This is also happening for the global menu and I believe this can be considered a case of Cumulative Layout Shift. So I'd suggested we fix this too, for both menus. What do you think @BirteThornquist ?

We had issues with the menu button placement before, it's because of a number of things (scrollbars and offset for that amongst which) but it boils down to that we use 2 different buttons; one for when the menu is closed and one for when the menu is open.
It's been tinkered with several times before. If it keeps messing up and we want to properly fix it, we should look into just using a single button that is positioned absolutely or something.

@SvSven #789 was created to solve that. It was due to the language switch

And btw @adneforslund  when you moved some styles according to Fernando's input, the storybook example for the menu broke as well.

We had a walkthrough about this on Friday and at least the Sanity part and the design looks good 👍

Storybook implementation needs to be looked at as it does not work for desktop now.

https://github.com/equinor/energyvision/commit/338e6bf4ccf1c83b4ef0445332fd008e3eebdd16 is the commit that broke Storybook

@wenche help

In review. Being looked at by me and @wenche

authors: SvSven adneforslund fernandolucchesi nicholas-haggerty nilsml wenche


text:
spec.undef exported as "Nan" for tables

Outgoing metadata for tables gets string value Nan for the undef field.
Expected behavior: Either a valid number (-999.25 or similar) or nothing.

https://github.com/equinor/fmu-dataio/blob/ff35c42cbd1637017acb288b0c3b473f85e4f4b4/src/fmu/dataio/_export_item.py#L393

Specific symptom addressed temporarily through #41 but root cause is probably something that must be discussed further.
undef is different in the context of datatypes, formats and contents (e.g. an Irap Binary surface has pre-defined undef in its format - a float value. In the current schema, the data.spec.undef attribute states the undefined value from a content perspective.
Example: From the metadata, the client should understand that the value -999.25 should be read as null.
This raises also a challenge in situations where null is the actual "value" to be used (e.g. for a specific content and a specific data type, None/null is actually the undefined marker. Using null is also ambiguous and should probably be avoided. There may also be incompatibility here with existing functionality for stripping away attributes with null values before dumping to yaml. In that context, a string (such as the "Nan" that was originally used) could be a solution, but it would be nice to avoid type-mixing for this attribute.
Perhaps -999.25 is the best option in many cases where null is not directly present, and perhaps null can be assumed when the data.spec.undef field is not present? (In Python, use .get("undef")) This way, the field - if present, will always be a number.

repo: equinor/fmu-dataio


text:
feat(components): add documentation to all exported types

Write docstrings for all exported types
Remove test prop on LinearProgress indicator component

closes #103

Don't be alarmed by all the files changed 🥇

repo: equinor/mad


text:
Element naming

"nominee-field-isced-code" corresponds to Subject Area Code in the model. I was under the impression that "subject area" is the general term (in this case encoded by ISCED). In that case, shouldn't the field have the more general name? Conceivably, the coding could be changed at some point, or there could be a need for more than one coding scheme - then it would make sense to use a general name and have a (possibly optional) scheme attribute? Also, why the "field" name part?
"nominee-language-skill" corresponds to Language Competence in the model. I have seen the "competence" term used many places in the background material, and the corresponding field in Cooperation Condition is called Required Language Competence. Whatever we decide, we should at least be consistent in all entities/APIs.
"component-studied" should in my opinion be renamed to "component-planned" - otherwise it's easy to interpret it as the component that is actually studied.

I was under the impression that "subject area" is the general term (in this case encoded by ISCED). In that case, shouldn't the field have the more general name?

This naming strategy was taken from ELMO format. In ELMO subjectArea refers to the (now deprecated) Erasmus codes (for backward-compatibility), so in EMREX we began using iscedCodes to differentiate between them. Since this element naming is used in other formats we build upon, I thought it might be good to keep it here, to avoid confusion. (Note, that EMREX ELMO is used in ToRs API.)

then it would make sense to use a general name and have a (possibly optional) scheme attribute

Would it be possible for the reader to tell - based on the value itself - what type of subject area it is? I always thought that it is essential to clearly differentiate which type the server uses.

Also, why the "field" name part?

This is short for "field of education". I thought it appropriate, because there are other types of ISCED codes (only one of them describes fields of education). On the other hand, EMREX ELMO ignores this issue, so perhaps we can ignore it too. Or, we could name it "isced-f" instead of just "isced", this makes it clear.

"nominee-language-skill" corresponds to Language Competence in the model. I have seen the "competence" term used many places in the background material

@erasmus-without-paper/wp2 - what do you think? Is it better to name thing "language skill" or "language competence"? EUC has used "language skill" in their latest IIA template (section C). I just took a glance on WP2's vocabulary (201601_Data Glossary_v2.docx) - there's a reference to "language skills" term (although it's not defined explicitly).

"component-studied" should in my opinion be renamed to "component-planned" - otherwise it's easy to interpret it as the component that is actually studied.

Perhaps. On the other hand, if we name it "component-planned" then it's easy to interpret it as it being only "initially planned to be studied", but not "actually studied". I tried to think of a better name, but couldn't decide which one's better. Perhaps you have some other suggestions?
BTW - because these three issues are not clearly related, it's better to report them separately to avoid clutter (one issue per element).

This naming strategy was taken from ELMO format. In ELMO subjectArea refers to the (now deprecated) Erasmus codes (for backward-compatibility), so in EMREX we began using iscedCodes to differentiate between them. [...] I thought it might be good to keep it here, to avoid confusion. [...]
[...] I always thought that it is essential to clearly differentiate which type the server uses.

Ok, no problem.

Also, why the "field" name part?

This is short for "field of education". I thought it appropriate, because there are other types of ISCED codes (only one of them describes fields of education). On the other hand, EMREX ELMO ignores this issue, so perhaps we can ignore it too. Or, we could name it "isced-f" instead of just "isced", this makes it clear.

"isced-f" sounds good to me. I will change it in the model.

@erasmus-without-paper/wp2 - what do you think? Is it better to name thing "language skill" or "language competence"? EUC has used "language skill" in their latest IIA template (section C). I just took a glance on WP2's vocabulary (201601_Data Glossary_v2.docx) - there's a reference to "language skills" term (although it's not defined explicitly).

I will align the model with whatever the group decides.

"component-studied" should in my opinion be renamed to "component-planned" - otherwise it's easy to interpret it as the component that is actually studied.

Perhaps. On the other hand, if we name it "component-planned" then it's easy to interpret it as it being only "initially planned to be studied", but not "actually studied". I tried to think of a better name, but couldn't decide which one's better. Perhaps you have some other suggestions?

I don't think "planned" excludes the possibility of a component being actually studied:

If the plan is changed, the planned component is superceded by a replacement
If the plan holds, the planned component is studied as well (i.e. good planning!)

Otherwise, the planned component would have to be "superceded" by an identical "studied" component in alternative 2, which doesn't make sense. Anyway, that's my opinion - maybe others will disagree...

BTW - because these three issues are not clearly related, it's better to report them separately to avoid clutter (one issue per element).

I see your point, but since only the last issue is still under discussion from my point of view, I'm tempted to leave it as-is. But I will be more structured in the future :-)

I don't think "planned" excludes the possibility of a component being actually studied

We need a distinction between:

components studied, or planned to be studied,
components recognized, or planned to be recognized.

Problem with "planned" is that it matches both of these sets because <component-recognized> is also not always recognized - it can be planned to be recognized. So, in this context "component planned" seems actually more vague than "component studied".
Perhaps why should change the tense in both?

We need a distinction between:

components studied, or planned to be studied,
components recognized, or planned to be recognized.

Problem with "planned" is that it matches both of these sets because "component-recognized" is also not always recognized - it can be planned to be recognized. So, in this context "component planned" seems actually more vague than "component studied".

I see - "planned" is a boolean flag orthogonal to studied/recognized. If it's necessary to specify whether it is planned or not, there should be such a flag in the format. Otherwise, it's enough with "studied" and "recognized".

I see your point, but since only the last issue is still under discussion from my point of view, I'm tempted to leave it as-is. But I will be more structured in the future :-)

I believe this can now be closed.

I believe this can now be closed.

Well, there is still the matter of "language skill" vs. "language competence" - but still should of course be a separate issue...

Well, there is still the matter of "language skill" vs. "language competence"

Ah yes, we are waiting for @erasmus-without-paper/wp2's comment on this.

Ah yes, we are waiting for @erasmus-without-paper/wp2's comment on this.

I'm closing this due to lack of input. Also, some teams have already started development, so changing this without good reason wouldn't do much good.

authors: kaiqu wrygiel


text:
islemtipi

Selamlar.
Bu modülü kullanırken islemtipi adlı parametrenin olmadığını gördüm. Kendim elle bu parametreyi ekledim ama repoya da yaptıklarımı merge etmek isterim sizin için de uygunsa.

https://opensource.com/article/19/7/create-pull-request-github

Teşekkür ederim.

Pull request'i attım.

authors: ardaorkin eraycetinay

repo: eraycetinay/node-nestpay


text:
display of generatedhtml

Dear @michaelnmmeyer —
At the moment, a DiplEd html that is already generated does not yet get shown on the .io site.
See DHARMA_DiplEdSarvavajrodayaCodex.html at https://github.com/erc-dharma/tfd-sanskrit-philology/tree/master/docs/diplEd, not shown at https://erc-dharma.github.io/#tfd-collection. Can you correct this? Thanks.

Thanks. But now the DiplEd is shown in the place for CritEd files and not in the place for DiplEd files. See screenshots.

fixed in the meantime, I see!

repo: erc-dharma/tfd-sanskrit-philology


text:
🛑 Stirling PDF is down

In 7330c26, Stirling PDF (https://pdf.erdav.eu.org) was down:

HTTP code: 502
Response time: 2743 ms

Resolved: Stirling PDF is back up in 6c57297 after 7 days, 14 hours, 3 minutes.


text:
Refactor TwoPhaseFlow App

[ ] rewrite parun as pro using argparse
[ ] replace app option as subcommand and move over app-specific options to subcommand
[ ] split off some phases as subcommands (e.g. mesh)
[ ] move _p,_n,_so modules into single file using p/n/so object-based approach
[ ] move petsc options files into same single python file and set through petsc4py
[ ] create subdirectory for apps
[ ] refactor or remove fastarchive option if not needed
[ ] move static docs into doc directory
[ ] move examples into tests and enable travis on them--don't want any untested code in the repo

Sorry for the stuff about fast archiving. I was having some troubles when outputting big files here at Kaust so I introduced that to speed up the process but I was not intending to get that into master. And, by the way, it didn't help much.

No problem. Good to know the story behind it. The archiving needs a good cleaning soon, so I'd be interested it any performance information. We could probably add some options to selectively disable archiving for some fields.

@tridelat and @manuel-quezada  here is a simple example of object-based physics and numerics in a single-file problem description: https://github.com/erdc/proteus/blob/master/proteus/tests/periodic/duct.py
The basic idea is to just walk through the needed models setting up a default object and then overriding the options, then set up the global system parameters. e.g.
from proteus.defaults import (Physics_base, Numerics_base)
# Navier-Stokes physics
nse_p = Physics_base()
nse_p.LevelModel = RANS3PF.LevelModel.
...
# Navier-Stokes numerics
nse_n = Numerics_base()
nse_n =  LinearSolver.KSP_Petsc4py
...
# CLSVOF physics
clsvof_p = Physics_base()
...
# CLSVOF_numerics
clsvof_n = Numerics_base()
...
# global system
from proteus.default_so import *
pnList=[(nse_p, nse_n),
             (clsvof_p, clsvof_n)]
...

This avoids the necessity of doing from proteus.default_p import *, etc. and the dangers that involves due to the fact that Python modules are singleton. I would like to get rid of that last from proteus.default_so import *, but I have thought through the best way to do that yet. We could just have parun check for the existence of a single so object that could be initialized in the same manner.

It's not clear to me what the goal for TwoPhaseFlow is. With it's current capabilities, it looks like it's the interface which most users should be building cases in Proteus since it can handle domain generation, mesh generation, and analysis specification in a single file.
The initialization function of TwoPhaseFlowProblem assumes that the user is using some combination of a flow model and interface tracking models. Should it be allowed to handle single phase problems or should that be some other app like OnePhaseFlowProblem? As of now, it isn't very clean in terms of adding additional models like body dynamics.
For example, in the 2D floating body example, there's a need to specify the indices of the models:
https://github.com/erdc/proteus_tutorial/blob/master/2d/floating2D_chrono.py#L228-L233
In the dambreak case, there's no need to specify those indices because they're automatically defined via TwoPhaseFlow:
https://github.com/erdc/proteus_tutorial/blob/master/2d/dambreak.py

On a separate note, I think the initialization function can be condensed from
def __init__(self,
                 ns_model=0, #0: rans2p, 1: rans3p
                 ls_model=1, #0: vof+ncls+rdls+mcorr, 1: clsvof
                 nd=2,
                 # TIME STEPPING #
                 cfl=0.33,
                 outputStepping=None,
                 # DOMAIN AND MESH #
                 structured=False,
                 he=None,
                 nnx=None,
                 nny=None,
                 nnz=None,
                 domain=None,
                 triangleFlag=0,
                 # INITIAL CONDITIONS #
                 initialConditions=None,
                 # BOUNDARY CONDITIONS #
                 boundaryConditions=None,
                 # AUXILIARY VARIABLES #
                 auxVariables=None,
                 # OTHERS #
                 useSuperlu=True,
                 fastArchive=False)

to
def __init__(self,
                   domain = domain,
                   models = modelList,
                   other = otherOptions)

The boundary and initial conditions need to be attached to their respective models within Proteus anyway, so it's better to just have the user be directly responsible for adding it in after the TwoPhaseFlowProblem instantiation. Several of the other flags such as nd and he can be attributes of the domain object.
The modelList could be a user-defined list of model objects that defines the ordering of the staggered solve and allows users to easily incorporate body dynamics or any other model. To avoid new-user confusion, we could define a default_modelList = [rans2p_model,vof_model,ls_model,rd_model,mc_model].

My understanding is that  https://github.com/erdc/proteus/blob/master/proteus/TwoPhaseFlow/utils/petsc.options.asm is not a needed file anymore because of
https://github.com/erdc/proteus/blob/master/proteus/TwoPhaseFlow/utils/Parameters.py#L1483
from PR #954

authors: cekees manuel-quezada zhang-alvin

repo: erdc/proteus


text:
Create method to inspect for missing sensor readings in the database

See issue #2 for motivation.

Not sure why you referred this issue as part of your pull request. The notebook should address issue #3.

repo: erdl/scrape-util


text:
Failed to connect to peripheral

Hai,
I did al the steps in described in your readme. No errors on my PI. Did the config in config.py
_DOMOTICZ_SERVER_IP = "127.0.0.1"
DOMOTICZ_SERVER_PORT = "8080"
DOMOTICZ_USERNAME = ""
DOMOTICZ_PASSWORD = ""
sensors = { 1: {"MAC": "a4:c1:38:b9:07:19", "TH_IDX": 4516, "VOLTAGE_IDX": -1},_
But no connection to Domoticz. In the loginfo.log I see this error
2020-12-19 16:01:03 INFO     ***************************************
2020-12-19 16:01:03 INFO     Start script...
2020-12-19 16:01:03 INFO     Input parameters:
Domoticz Server IP: 127.0.0.1
Domoticz Server Port: 8080
Domoticz Server User:
Domoticz Server Password:
2020-12-19 16:01:03 INFO     TH_IDX:4516
2020-12-19 16:01:03 ERROR    Failed to connect to peripheral a4:c1:38:b9:07:19, addr type: public
Do you know what could be wrong? Thx
Best Regards, Erik

Hi,
Did you check the device with hcitool lescan? Is it works? If it isn't work then try to switch down and up the BLE

After
sudo hciconfig hci0 down
sudo hciconfig hci0 up
it seems to work
Thx

Please close the issue if it works well!
Thanks!

authors: eleutscher erdose


text:
v9.4.x Keepalive feature breaks build on OpenBSD 7.5

To Reproduce
Clone repo on OpenBSD 7.5 and build it with rust 1.76 via cargo r
myhostname$ cargo r
   Compiling wstunnel v9.4.0 (/home/chris/wstunnel)
warning: unused variable: `ip_transparent`
   --> src/tcp.rs:185:43
    |
185 | ...er(bind: SocketAddr, ip_transparent: bool) -> Result<TcpListenerStre...
    |                         ^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_ip_transparent`
    |
    = note: `#[warn(unused_variables)]` on by default

error[E0599]: no method named `with_interval` found for struct `TcpKeepalive` in the current scope
  --> src/tcp.rs:29:10
   |
27 |       let tcp_keepalive = TcpKeepalive::new()
   |  _________________________-
28 | |         .with_time(Duration::from_secs(60))
29 | |         .with_interval(Duration::from_secs(10))
   | |         -^^^^^^^^^^^^^ method not found in `TcpKeepalive`
   | |_________|
   |

For more information about this error, try `rustc --explain E0599`.
warning: `wstunnel` (bin "wstunnel") generated 1 warning
error: could not compile `wstunnel` (bin "wstunnel") due to 1 previous error; 1 warning emitted

v9.3.0 builds fine.

I pushed a fix on main, would you mind trying with the head of main and let me know ?

Yep, builds!

authors: Sir-Photch erebe

repo: erebe/wstunnel


text:
Delete and/or cleanup a single CFR part

Often we've wanted to clean up single parts or delete a single part. Consider pulling in:
https://github.com/cfpb/regulations-core/pull/63
https://github.com/cfpb/regulations-core/pull/64

Resolved in #58

repo: eregs/regulations-core


text:
Update docs to use latest version

The version used in the current docs doesn't support bzlmod. Let's use the latest commit instead.

thanks! merged

Thanks for the quick review! :)

authors: erenon knatten


text:
[Feature] Allow to configure the LoadBalancerIP

Description
In case that the service is defined as type "Loadbalancer" it should be possible to set the "LoadBalancerIP". For further information see:
https://kubernetes.io/docs/concepts/services-networking/service/#loadbalancer
https://docs.microsoft.com/en-us/azure/aks/static-ip#create-a-service-using-the-static-ip-address
This is the affected position:
https://github.com/ergon/airlock-helm-charts/blob/e13281afab375149201c9aa0f842bafc8d4a3c94/charts/microgateway/templates/service.yaml#L15
Describe the solution you'd like
Extend the current solution with the followin code (untested):
  ...
  type: {{ .Values.service.type }}
  {{- if eq .Values.service.type == "LoadBalancer" }}
    {{- if .Values.service.LoadBalancerIP }}
      loadBalancerIP: {{ .Values.service.loadBalancerIP}}
    {{- end }}
  {{- end }}
  ...

Versions

Microgateway: 1.0
Helm chart: 0.6.0
Kubernetes: Azure Kubernetes Service

Checklist

[x] The issue contains all necessary information a third party requires to understand.

When using Type: LoadBalancer the setting externalTrafficPolicy should be set to Local. Otherwise the original client IP will be natted.
=> externalTrafficPolicy: Local
For more information see: https://kubernetes.io/docs/tasks/access-application-cluster/create-external-load-balancer/#preserving-the-client-source-ip

repo: ergon/airlock-helm-charts


text:
please move snyk from dependencies to devDependencies

Is snyk required to use quagga2? If not, it'll be amazing if it wasn't part of npm install @ericblade/quagga2. I ask because snyk is a pretty large dependency, weighting in at minimum 50 MB, up to 120 MB extra - depending how many of the peers of snyk you already have.

While I don't think it's technically a requirement, it is used as a mechanism to patch some dependencies.  I am not positive if it is currently providing any benefit now that most of this project's dependencies have been updated to the latest versions of things.
https://support.snyk.io/hc/en-us/articles/360000911277-Why-has-Snyk-been-added-as-a-dependency-in-my-project-
It's definitely worth a review of a) do we have the latest version of everything we can update, b) are we using any packages that are no longer being updated? are there alternatives?  c) does running snyk protect do anything for us presently, and should we move it out to a devdep?
i don't right off hand know the answers to any of these questions, but they'd definitely be worth knowing the answer to.
If snyk is only operating on our devdeps, which most definitely do still have some pretty out of date packages listed, or if snyk isn't doing anything at all for us then we should consider moving it.

I didn't indent to ask for removal of snyk from quagga for development or maintenance purpose. Just from the quagga that is published to npm.
If I do:
yarn init -y
yarn set version berry
yarn config set nodeLinker node-modules
yarn add @ericblade/quagga2

then node_modules is 136 MB in size!
When I nullify snyk, as that dependency is just occupied space and doesn't do anything for me, via adding to the package.json:
  "resolutions": {
    "snyk": "https://registry.npmjs.org/@favware/skip-dependency/-/skip-dependency-1.1.1.tgz"
  }

then node_modules is down to 44 MB!
When you look at the package.json of quagga, nearly all of the "dependencies" are actually devDependencies. Most of them are inlined at build time. Only references to get-pixel & ndarray-linear-interpolate remain. So when I force the rest to null:
  "resolutions": {
    "gl-mat2": "https://registry.npmjs.org/@favware/skip-dependency/-/skip-dependency-1.1.1.tgz",
    "gl-vec2": "https://registry.npmjs.org/@favware/skip-dependency/-/skip-dependency-1.1.1.tgz",
    "gl-vec3": "https://registry.npmjs.org/@favware/skip-dependency/-/skip-dependency-1.1.1.tgz",
    "lodash": "https://registry.npmjs.org/@favware/skip-dependency/-/skip-dependency-1.1.1.tgz",
    "ndarray": "https://registry.npmjs.org/@favware/skip-dependency/-/skip-dependency-1.1.1.tgz",
    "snyk": "https://registry.npmjs.org/@favware/skip-dependency/-/skip-dependency-1.1.1.tgz"
  }

my node_modules is down to 18 MB! And it works just like before.

so, presently after doing npm install on quagga, it should run snyk protect, to patch any dependencies that it has patches for that may not have been updated by their devs, or are updated in incompatible versions.  At least, that's what it does if there's any work for it to do, and I don't know if there is or not.
As to the other deps, you do have a few interesting points that i've not really considered in the slightest -- i don't really know 100% what the build step does, so i can't speak to if those modules need to be installed at install time or if they can all be moved to dev deps, due to any inlining that might be happening.
On top of that point, you're bringing me to a realization that some or maybe even all of those modules may only be referenced by the node.js code, which might make them totally irrelevant to browser, and could be a source of additional optimizations.  (or there might be no effect at all, if the packaged bundles don't include them because they aren't referenced in the browser code, which hopefully is how it actually works)
So, I think there's a few actions here

make sure that all our non dev dependencies are using the latest version, and that those packages are still maintained.  If any packages that are regular dependencies are not being maintained anymore, find alternatives that are.

move snyk to a devDep since running snyk protect on the user's side shouldn't be desireable if all our regular dependencies are up to date and maintained

investigate if the dependencies are actually necessary for an end user to install, or if they are inlined, and move any that might be being inlined into devDep

figure out if there's any good way to separate browser deps from node deps   (if it is inlining things like gl-vec2, that might be an indication that the original author already thought of this, and set it up to build in the modules that were not needed for both browser and node, rather than actually depending on the user having them...)

and voila, could be a large improvement.
possible way to check this would be to install the source repo, npm install, do a build, remove the step in the tests that builds before running tests, then start removing dependencies and running tests after each one.
i'm concerned, primarily, that not installing the deps you've nulled out could cause breakage in codepaths that you're not going down in your use case.

are you using node, browser, or both?

We only use quagga in the browser.

move snyk to a devDep since running snyk protect on the user's side shouldn't be desireable if all our regular dependencies are up to date and maintained

I use quagga2 in browser also and due to the significant number of dependencies installed I almost decided against using this package.  I've never used snyk so apologies if everything from here on in makes no sense, but from what I've read the last couple of hours I don't think snyk is doing what you think it does with how you currently have it hooked up.
You have snyk protect hooked up to the npm prepublish script.  The npm prepublish script is now deprecated and replaced with prepare (https://docs.npmjs.com/cli/v7/using-npm/scripts#prepare-and-prepublish).  Regardless of the deprecation however, snyk is not running on the user's side because prepublish/prepare does not work like the postinstall script.  That is to say prepublish/prepare will not be called when someone installs your package, it is only called when you do a local npm install, npm pack or npm publish within this package.  Whether the script runs or not is irrelevant however, as all snyk protect will do is install the patches as specified in your .snyk file, which is empty and has been for nearly 2 years, so it will do nothing.
If you want to continue using snyk, then I think you have two options.  Either switch snyk to use the postinstall script and keep it as a dependency or switch to using the prepare script and move it to a devDependency.
I personally hope you'll choose the latter.  If I want to run snyk, I can install it myself, I'd rather not have a dependency force it on me, especially with the package bloat it brings along with it.
Seeing as moving snyk to devDependencies is the easiest task with the biggest reward, could that be done first please, and then the other dependencies mentioned above tackled appropriately later on down the line?

While I don't think it's technically a requirement, it is used as a mechanism to patch some dependencies. I am not positive if it is currently providing any benefit now that most of this project's dependencies have been updated to the latest versions of things.
https://support.snyk.io/hc/en-us/articles/360000911277-Why-has-Snyk-been-added-as-a-dependency-in-my-project-
It's definitely worth a review of a) do we have the latest version of everything we can update, b) are we using any packages that are no longer being updated? are there alternatives? c) does running snyk protect do anything for us presently, and should we move it out to a devdep?
i don't right off hand know the answers to any of these questions, but they'd definitely be worth knowing the answer to.
If snyk is only operating on our devdeps, which most definitely do still have some pretty out of date packages listed, or if snyk isn't doing anything at all for us then we should consider moving it.

Figured it might also be useful to answer some of the questions you raised in this post also.  With snyk you have test, monitor and protect commands, or you can use wizard which is an interactive version of those three commands. snyk wizard will scan your dependencies and devDependencies and assuming it finds a vulnerability you have four options.  First is to tell snyk to ignore it if you judge it to be a non-issue.  Second is to update the package if an updated version with a fix has been released.  Third (if it's applicable) you can just update the relevant dependency of the package if an updated version with a fix has been released.  Fourth is to install a patch crafted by snyk themselves, which is the route you take if you don't want to update a package due to it maybe bringing along some breaking changes with it, or if an updated version with a fix doesn't exist.
As mentioned above snyk protect is not running for users installing your package, and at the moment because your .snyk file is empty is doing nothing for you either.  You'll need to run test/wizard to determine if your current set of dependencies/devDependencies have vulnerabilities.  Even if they do, you might be able to just update to newer versions which will mean snyk protect is still redundant.

I appreciate the additional analysis.  The situation has quite probably changed quite a bit from when I forked initially.
Good news? i've got a long weekend from work right now, perhaps between time spent catching up on house repairs and mandatory computer upgrades, I'll grab some time to take a look at this.  It might well be possible that just relying on dependabot would be fine.. the landscape for this sort of stuff has changed drastically in the last couple of years.

I'm glad I could be of some help.  By far the best return-on-effort will be moving snyk to devDeps, so if that's all you find time to do I'll still be over the moon and super grateful!  I definitely appreciate your time isn't infinite.

I appreciate the additional help, it definitely helps me to make a movement on something, i made a ton of big changes early on, especially with upgrading deps, when my time was copious, but now with a new job started recently, and a real product to deliver soon, it's become a lot more of a crunch to find time to just validate that this isn't a major change and that my reasoning earlier for keeping this is no longer true.

i didn't get to it last weekend, but unless work piles me up with stuff that needs to get done this weekend, i've got time actually set aside to handle personal project maintenance this weekend.

just went ahead and removed it in https://github.com/ericblade/quagga2/commit/c966812ee968ad8a00d85b66d59741149adf0270

authors: ericblade icleolion sod


text:
Quagga stoped but the camera keeps working on background

I'm developing an app where a barcode is read, then an API call is made and if the response is 200 you are redirected to another page...
Okay everything works here but, for some reason, ( The computer, and i suppose the same happens in mobile ), thinks that the camera is still being used. I thought it would happen because i changed the page but not stopped quagga correctly. So I made it so after a minute it automatically stops. The first time it is stopped, any indicator in the computer that indicates the camera is working, it's turned off, but when I start again the camera, and it is automatically stopped, those icons don't turn off

I tried stopping the camera from CameraAccess API, but it stays the same, and I'm worried that the camera keeps working and, in a mobile device, it consumes the battery of the user.
I don't know if there is something related to my code, or it is the library, and I don't know how to replicate it in a codepen as every example I've seen is not working.
<template>
  <div class="scannerComponent col column justify-center items-center full-width">
    <q-btn
      v-if="!isQuaggaRunning"
      label="Scan"
      class="full-width bg-k-secondary q-mb-md q-py-md"
      unelevated
      rounded

      @click="startQuagga"
    />
    <div
      v-show="isQuaggaRunning"
      id="scanner"
      class="full-width flex row items-center justify-center col "
    >
      <video
        class="scanner"
      />
    </div>
  </div>
</template>

<script setup lang="ts">
import { onMounted, reactive, ref } from 'vue'
import Quagga, {
  QuaggaJSConfigObject
} from '@ericblade/quagga2'

const emits = defineEmits(['detected', 'getStopCallback'])

onMounted(() => {
  startQuagga()
  emits('getStopCallback', () => {
    Quagga.stop()
    isQuaggaRunning.value = false
  })
})

const quaggaState = reactive<QuaggaJSConfigObject>({
  inputStream: {
    name: 'Live',
    type: 'LiveStream',
    target: '#scanner'
  },
  decoder: {
    readers: ['code_128_reader']
  },
  locator: {
    debug: {
      showCanvas: false
    }
  }
})

function startQuagga () {
  Quagga.init(quaggaState, (err: any) => {
    if (err) {
      return console.error(err)
    }
    Quagga.start()
  })
  isQuaggaRunning.value = true
  automaticallyStopQuagga()
}

const isQuaggaRunning = ref(true)
const QUAGGA_TIMEOUT = 1000
function automaticallyStopQuagga () {
  setTimeout(() => {
    Quagga.stop()
    isQuaggaRunning.value = false
  }, QUAGGA_TIMEOUT)
}

Quagga.onDetected((data: any) => {
  emits('detected', data.codeResult.code)
})

</script>

Well... I was doing my tests in local with 1 second, and in this case it fails a lot, but increasing the value to 10 seconds it looks like it works fine. I don't know...

Okay, never mind... I'm an idiot. Because my Stop function was hidding the Quagga canvas, and it was actually displaying images but, as it was hidden, i thought it was not working...


text:
Some structs should be inlined.

LocalObjectReference in v1.SecretKeySelector should have ,inline in its json tag.
This is likely a wrong setting of protoc.
As a result, creating and v1.EnvVar in a container with an env variable drawn from a secret does not work.

I don't think this client is aiming for JSON serialization support. If the generated protobufs work for talking to the API server, I think that's the best we're going to get.

authors: ericchiang ewoutp


text:
Bump @actions/core from 1.9.1 to 1.10.0

See #326

Solution based on https://github.com/uuidjs/uuid/issues/544


text:
🛑 Vault is down

In c508089, Vault (https://vault.drgn.xyz) was down:

HTTP code: 403
Response time: 41 ms

Resolved: Vault is back up in 2fef0a3.


text:
🛑 File Share is down

In 7357543, File Share (https://share.drgn.xyz) was down:

HTTP code: 523
Response time: 16564 ms

Resolved: File Share is back up in 4302fd1.


text:
[ISSUE] Crash on startup after ags 4.0 path was added as external editor

After I added ags 4.0(3.99.99) as an "external" editor and different folder with all the projects(some contains 4.0 projects) the software stops working and crash.
I don't know how to reset the settings, but imo it should ignore broken ags version(assuming that it's causing it to crash) and give you a warning and perhaps a yellow exclamation mark in the settings next to the 4.0 folder(for external editor)

Hi @tarnos12 ! Sorry, I did not look here for a long time... Can you explain the steps to reproduce this?

@ericoporto
Hey, I have missed the notification apparently, just came to see if anything has changed since I've seen some screenshots where you had ags 3.99 in the toolbox.
Basically somewhere in settings you have to add Path to different editor versions.
I have added a path ags 4.0 as an "external" editor and the software crashes for me.
Even now when I download it again, it just crashes when I try to open it.
I need a way to remove some kind of config to run the software once more, but I don't know where it is.

Hum, I think I finally know what can cause the crash. I will need some time to figure out where to add more try/catch to prevent this from happening and handle it when handling the paths.
The settings should be under C:\Users\USERNWME\AppData\Local\eri0o\AgsToolbox\settings.json I think, on Windows 10.

Alright, so the issue might be this:
"manually_installed_editors_search_dirs": [
        "C:\\Program Files (x86)\\Adventure Game Studio 3.5.1",
        "C:\\Program Files (x86)\\Adventure Game Studio 3.6.0",
        "C:/Program Files (x86)/Adventure Game Studio 3.99.99"
    ],
    ```

    Ags 4 path has `/` instead of `\\`
    After changing it, the software still crashed(but I managed to open it for a second)

I still can't reproduce this crash with the binaries I generate on Releases, if it generates any sort of error message screen, please share

I will have to test this again in the future, but from my memory the software simply refused to launch due to broken config file after adding ags 4 as the external editor.
Perhaps it's fixed already.
My projects folder had projects made in various versions of the engine which might have been a cause too.

aaaaahhh, I think I get it, it's the path thing you mentioned. I think I fixed this but in case invalid settings are present it should crash - from a previous broken build. But need to check if my fix actually works everywhere.
I think I need to figure a way to fixup in any way, because the way path works in python, it may be a case someone runs this in some environment where they work differently, and then it writes in a way it can't properly read later...
Need to think a bit how to workout with this.

I don't know the details, but perhaps when it's unable to load a path for any reason it should show a pop up asking you to add a new path, otherwise delete the old broken path info.

ok, made a new release that try to silently ignore invalid paths and will update the settings accordingly. I think it should work now.

This has been fixed for a few releases now

repo: ericoporto/agstoolbox


text:
Tutorial

Feel free to work off of this branch!

Closing since we are not adding tutorial anymore

repo: ericrkuo/Flow


text:
DatePicker looses value upon reopening

Right now, if you select a date, the text field is filled with the selected value. As soon as you open the date picker again, it switches to blank and the date picker shows no selected date.
If you compare this to the behaviour of the bare MUI date picker, you will see that the selected date is still in the field and the selected date is marked in the date picker.
For debugging, I tried to use MUI with redux-form directly and it boiled down that this starts to happen if you add the onBlur/onFocus callbacks. Unfortunately, without this callback the dirty/touched tracking does not work.
Any ideas?

Fixed in v4.0.0.

authors: erikras luxflux

repo: erikras/redux-form-material-ui


text:
An empty object is returned if a field is undefined when calling getValues

reduxForm({
  form: 'simple',
  fields: [ 'name' ]
})(SimpleForm);

// action creator
import { getValues } from 'redux-form';

console.log(getValues(getState().form.simple));
// { name: {} }

Thanks. Will investigate. :mag:

I'm unable to reproduce the bug.
it('should ignore undefined fields', () => {
  const state = {
    name: makeFieldValue({}),
  };
  expect(getValuesFromState(state))
    .toBeA('object')
    .toEqual({});
});

As expected, the result is {}.
@erikras Can we close this one?

Sure. I never got around to investigating. 😦
Go ahead and PR that test if you want. You can never have too many tests, especially if they demonstrate functionality reported as buggy.

authors: erikras ooflorent xcatliu

repo: erikras/redux-form


text:
README.md: Remove Zalora

We have no Haskell code in production. Source: I currently work here.

Thanks!

My pleasure, though I wish I had better news :smiley:.

authors: erkmos vaibhavsagar

repo: erkmos/haskell-companies

license: Unlicense


text:
How to use disposable list only?

How can I use only a disposable list without relying on a service? Setting INDISPOSABLE_ENABLED to false doesn't seem to resolve the issue. Awesome package BTW :)

That is correct, indisposable will enable/disable the package, maybe i will add a new configuration so you can enable list/service/both

repo: erlandmuchasaj/laravel-email-verify


text:
Add logging

It is overdue to implement a logging system. I want it to be simple but flexible. I want

Specify logging of arbitrary "modules" with e.g. LOG_REGISTER_MODULE("scheduler")
Set verbosity level for each module LOG_LEVEL("scheduler", LOG_INFO)
Set verbosity level for all modules in one go.
Do logging specific to each module LOG_ERR("scheduler", "Tried to schedule event into the past %ld", event_time);

Zephyr has a particularly nice logging system and I want our logging to match nicely to that: https://docs.zephyrproject.org/latest/services/logging/index.html
For other targets we can chose to either:

Implement the nice logging system with multiple modules or
Just check global verbosity level and dump everything out over printf.

@LasseRosenow is there a nice riot-os logging module the only thing I found is:
https://doc.riot-os.org/core_2lib_2include_2log_8h.html

I am looking at this currently :)

@LasseRosenow is there a nice riot-os logging module the only thing I found is: https://doc.riot-os.org/core_2lib_2include_2log_8h.html

From my knowledge in RIOT it is common to just use printf and then hide Debug prints behind the DEBUG define, which can be overwritten on a per-file basis.
This logging system that you are referencing there is the better alternative to that in my view, but usage of it is somewhat "random".

A first iteration of flexible logging system is now in main

authors: LasseRosenow erlingrj tanneberger

repo: erlingrj/reactor-uc


text:
Sort link should not modify passed-in parameters

When creating a sort_link, the passed-in parameters should not be modified within the sort_link method.

I'm sorry this took so long. I updated the pull request and submitted against 1-0-stable as well. Thanks!


text:
Add request code

I have two imageview, Now I want to chose which image goes where. As you hardcoded the request code, We can't determine which imageview should we show the image. Hope you understand the problem ..

You can start the Image picker with your own request code.
ex:
ImagePicker.create(this).start(yourRequestCode)

Please add this on your doc.. Thanks.. 😀 😀

authors: ShahadatShaki esafirm

repo: esafirm/android-image-picker


text:
Fix echo consumer example

Use the correct function clauses to be able to overwrite the default ones created in ExRabbitPool.Consumer

everything LGTM thank you, and sorry for the delay reviewing this as I was in a client project.


text:
Function to enable commands

Added the function to enable commands based on the disable one

This was added by Ao in #16 so unfortunately, I have to refuse this one.

authors: FOG-Yamato eslachance

repo: eslachance/guidebot


text:
Add eslint-remote-tester

This runs all our rules on top ESLint plugin repos to ensure that they don't crash. It's like smoke testing.
https://github.com/AriPerkkio/eslint-remote-tester

CC @AriPerkkio Can you review this please?

authors: MichaelDeBoey bmish


text:
Use Consultation

I'm not sure which template to use to create my question, so I started a new template.
I have a question. When writing the following code, I encountered an error warning under my id and username, and the editor prompted me with an Expected indication of 4 spaces but found 2.eslint@stylistic /Js/index.
import { Entity, Column, PrimaryGeneratedColumn } from 'typeorm';

@Entity()
export class UserEntity {
  @PrimaryGeneratedColumn()
  id: string;

  @Column()
  username: string;
}

I'm sorry, but I couldn't find any demonstration or explanation on how to solve this error prompt after reviewing the document. I think this indentation format is an acceptable one. Because, how should I configure it to solve this problem?

Duplicated of https://github.com/eslint-stylistic/eslint-stylistic/issues/208

authors: SuperCodeCandy antfu

repo: eslint-stylistic/eslint-stylistic


text:
Implement OpenSearch?

Implement https://en.wikipedia.org/wiki/OpenSearch for the search?
Background: I'd like to add the search as search shortcut in the browser.
I understand if this is not possible due to Algolia, but I wanted to bring it up.

We can support OpenSearch through google custom site-search. Should be easy enough to add. If you feel like it - send a PR and I'll review it.

Closing this issue since we haven't heard anything back for quite some time. Feel free to create a new issue if this is something you would like to discuss further.

authors: Schweinepriester ilyavolodin kaicataldo


text:
Coex support for all chips

[x] ESP32
[x] ESP32-C2
[x] ESP32-C3
[x] ESP32-C6 (*)
[ ] ESP32-H2 (is IEEE802.15.4 / Bluetooth co-existence possible at all?)
[x] ESP32-S3

(*) needs the big-heap feature (or configuring a big enough wifi heap)

This issue is quite old, and I'm not sure there's any work left to do here, so I will close this. @bjoernQ when you're back from vacation, if anything is actually required for the ESP32-H2 could you please open a new issue with some details?

repo: esp-rs/esp-hal


text:
fail ValueError (RDT-262)

IDF: v4.4.2
macOS: 12.5
We are looking to replace our complex clang-tidy wrapper with idf.py clang-check but our initial findings testing is not running. Happy to help support this effort.
command: idf.py clang-check --run-clang-tidy-py ./scripts/ci/run-clang-tidy.py --exclude-paths managed_components
run command: /Users/ash/dev/iot-module-fw/scripts/ci/run-clang-tidy.py .*  || true
Traceback (most recent call last):
  File "/Users/ash/dev/esp/esp-idf-latest-release/tools/idf.py", line 816, in <module>
    main()
  File "/Users/ash/dev/esp/esp-idf-latest-release/tools/idf.py", line 734, in main
    cli(sys.argv[1:], prog_name=PROG, complete_var='_IDF.PY_COMPLETE')
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/click/core.py", line 1689, in invoke
    return _process_result(rv)
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/click/core.py", line 1626, in _process_result
    value = ctx.invoke(self._result_callback, value, **ctx.params)
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/Users/ash/dev/esp/esp-idf-latest-release/tools/idf.py", line 636, in execute_tasks
    task(ctx, global_args, task.action_args)
  File "/Users/ash/dev/esp/esp-idf-latest-release/tools/idf.py", line 217, in __call__
    self.callback(self.name, context, global_args, **action_args)
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/pyclang/idf_extension.py", line 25, in call_runner
    runner()
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/pyclang/runner.py", line 169, in __call__
    self._run(folder, log_fs, output_dir)
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/pyclang/runner.py", line 140, in _run
    func(folder, log_fs, output_dir)
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/pyclang/runner.py", line 183, in _f
    return func(self, *args, **kwargs)
  File "/Users/ash/.espressif/python_env/idf4.4_py3.9_env/lib/python3.9/site-packages/pyclang/runner.py", line 306, in run_clang_tidy
    raise ValueError(first_line)
ValueError

Hi @AshUK, sorry for replying late. When this error happens, could you please check, what is the contents of warnings.txt file in the same directory?

I can confirm making the file executable fixes the issue, apologies and thank you.

authors: AshUK igrr

repo: espressif/clang-tidy-runner


text:
Does the clock of i2s conflict with the clock of Ethernet ? (IDFGH-10750)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I'm using IDF V4.4.5 and ADF V2.5.select lyrat_v4.3 board.
When I debug the network alone，it's OK.
When I debug the es8388 alone，it's OK too .
When I debug together，something wrong.
the es8388 works OK,but the ethernet will link down.
Pins used of my board:
LAN
LAN_RXD0 ---> IO25
LAN_RXD1 ---> IO26
LAN_CRS_DV ---> IO27
LAN_MDC ---> IO23
LAN_MDIO ---> IO18
LAN_TXD1 ---> IO22
LAN_TXD0 ---> IO19
LAN_TX_EN ---> IO21
LAN_RESET ---> IO5
LAN_REF_CLK ---> IO0
es8388
ASDOUT ---> IO35
DSDIN ---> IO33
LRCK ---> IO14
SCLK ---> IO2
MCLK ---> IO3
SDA ---> IO15
SCL ---> IO13
I find the ethernet link down when the bellow code run.
/* Step 7: Set I2S clocks and start. No need to give parameters since configurations has been set in 'i2s_driver_init' */
ESP_GOTO_ON_ERROR(i2s_set_clk(i2s_num, 0, 0, 0), err, TAG, "I2S set clock failed");
Does the clock of i2s conflict with the clock of Ethernet ?How th solve it?

There's only one APLL in ESP32. I2S can use APLL as a clock source and Ethernet can also use APLL to generate the RMII reference clock. If both of them want to be used together, then APLL must be set to 50MHz on the I2S side.
Starting from esp-idf v5.0, there's a lock for the APLL clock, which makes it easier to detect such conflict.
https://github.com/espressif/esp-idf/commit/4f28b33bbcbcff1a018288cdf9b9bdbec9f1d148

Yes, on v4.4, the APLL in I2S and Ethernet are calculated separately, but they will try to acquire the same APLL since v5.0. The best way is to migrate your IDF to the later version like v5.0 or v5.1, and initialize Ethernet before I2S, then I2S will try to calculate the clock with 50MHz APLL (will still fail if the required MCLK frequency greater than 50MHz).
Another way is to turn off the  use_apll in I2S configuration field, so that I2S will use the 160MHz PLL as the clock source. No worry this clock source might be inaccurate, it has no difference for the normal sample rate like 8KHz ~ 192KHz

If both of them want to be used together, then APLL must be set to 50MHz on the I2S side.

Set use_apll=false.But it doesn't work.Ethernet link down.
i2s_cfg.i2s_config.use_apll = false;
i2s_cfg.i2s_config.fixed_mclk = 0;
Set code bellow .Ethernet link down too.
i2s_cfg.i2s_config.use_apll = true;
i2s_cfg.i2s_config.fixed_mclk = 50000000;
In IDF V4.4.5,is there any way to solve it?
If ethernet using IO0 input extern clock  can  solve it?

As you configure I2S via ADF interface, I don't know too much about how ADF initialize I2S. But the I2S driver in IDF won't configure APLL if use_apll is not set.

authors: ArvinHou L-KAYA suda-morris

repo: espressif/esp-idf


text:
GSL1680 - Touch Screen Support (IDFGH-12257)

Is your feature request related to a problem?
I'm researching support for SSD1963 (LCD controller with i80 interface) and LVGL. In the same post I also found a touchscreen support reference: "These touch controllers are supported: GT911, TT21100, FT5X06"

https://github.com/espressif/esp-idf/tree/v5.2/examples/peripherals/lcd/i80_controller

I would like to indicate the possibility of adding support for the GSL1680 touchscreen driver IC (I2C):

https://github.com/rtek1000/GSL1680_

Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
https://www.buydisplay.com/5-inch-capacitive-touch-panel-screen-with-controller-gsl1680f

Hi @rtek1000 Yes, it's surely possible to add both the LCD controller and touch controller drivers based on the interface provided by the esp_lcd component. The drivers don't have to reside in either esp-idf or other Espressif repos. You can create them under your namespace and then share them in the Espressif Registry.
This video demonstrated a very good step-by-step guide.
If you want also a template of LCD and touch IC device drivers, check the lcd and lcd_touch folders here