Python3 native logging does not work properly with Azure ML sdk

Package Name: azureml
Package Version: 1.12 but has seen this since 1.0.62
Operating System: mac & linux
Python Version: python3

Describe the bug
Native logging package does not log anything in a case where azureml sdk was used. It has been issue for me for a while and recently I found the cause. The cause is related to from azureml.pipeline.steps import PythonScriptStep
Tested in Python3 azureml-sdk 1.12. But I have been observing this since azureml-sdk  1.0.62
azureml-core==1.12.0.post2 azureml-dataprep==2.2.3 azureml-dataprep-native==22.0.0 azureml-dataprep-rslex==1.0.1 azureml-dataset-runtime==1.12.0 azureml-defaults==1.14.0 azureml-model-management-sdk==1.0.1b6.post1 azureml-monitoring==0.1.0a21 azureml-pipeline==1.12.0 azureml-pipeline-core==1.12.0 azureml-pipeline-steps==1.12.0
To Reproduce
`
import logging
from azureml.core import Workspace, Dataset, Environment, ScriptRunConfig, Experiment
from azureml.core.runconfig import RunConfiguration
from azureml.pipeline.steps import PythonScriptStep
if name == 'main':
subscription_id = 'xxxxxxxxxx'
resource_group = 'yyyyyyyy'
workspace_name = 'zzzzzzzz-ws'
logging.basicConfig(level=logging.INFO, format='[%(asctime)s][%(name)s:' \
                    '%(filename)s:%(lineno)d]%(levelname)s: %(message)s')
logging.info('Logging starts ....')

workspace = Workspace(subscription_id, resource_group, workspace_name)
run_config = RunConfiguration(framework='python')
logging.debug('workspace loaded {}'.format(workspace))
dataset = Dataset.get_by_name(workspace, name='iris_dataset')
logging.info(dataset.to_pandas_dataframe().head())
# Enable Docker
run_config.environment = Environment(name='LoggingEnvironment')
run_config.environment.docker.enabled = True
run_config.environment.python.user_managed_dependencies = False

experiment = Experiment(workspace=workspace,
                        name='logging-test')
src = ScriptRunConfig(source_directory='.',
                        script='helloworld.py',
                        run_config=run_config)`

stdout:
python3 logging_test.py  Failure while loading azureml_run_type_providers. Failed to load entrypoint hyperdrive = azureml.train.hyperdrive:HyperDriveRun._from_run_dto with exception (azureml-telemetry 1.14.0 (/home/xxxx/.virtualenvmlops/lib/python3.6/site-packages), Requirement.parse('azureml-telemetry~=1.12.0')). Failure while loading azureml_run_type_providers. Failed to load entrypoint automl = azureml.train.automl.run:AutoMLRun._from_run_dto with exception (azureml-telemetry 1.14.0 (/home/xxxx/.virtualenvmlops/lib/python3.6/site-packages), Requirement.parse('azureml-telemetry~=1.12.0')).
Expected behavior
When I commented out from azureml.pipeline.steps import PythonScriptStep line and there were native loggings produced.
`
Failure while loading azureml_run_type_providers. Failed to load entrypoint hyperdrive = azureml.train.hyperdrive:HyperDriveRun._from_run_dto with exception (azureml-telemetry 1.14.0 (/home/xxxx/.virtualenvmlops/lib/python3.6/site-packages), Requirement.parse('azureml-telemetry~=1.12.0')).
Failure while loading azureml_run_type_providers.  Logging offline
[2020-09-30 08:57:30,301][azureml.core.run:run.py:297]INFO: Could not load the run context. Logging offline
[2020-09-30 08:57:34,103][root:logging_test.py:21]INFO:    sepal_length  sepal_width  petal_length  petal_width species   species_a   species_b
0           5.1          3.5           1.4          0.2  setosa   virginica  versicolor
1           4.9          3.0           1.4          0.2  setosa      setosa  versicolor
2           4.7          3.2           1.3          0.2  setosa  versicolor  versicolor
3           4.6          3.1           1.5          0.2  setosa  versicolor      setosa
4           5.0          3.6           1.4          0.2  setosa      setosa   virginica
[2020-09-30 08:57:34,119][azureml._restclient.clientbase:clientbase.py:179]INFO: Created a worker pool for first use
`
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

I believe this is due to AzureML calling the "module level" logger in its import. This forces the root logger to be created, and it will then ignore any subsequent calls to "logging.basicConfig". You might need to either

Run basicConfig prior to loading the AzureML library (any member of azureml.core I believe..)
or
Create a local logger with its own handler, that does not propagate the logs to the root logger which has been "created" by azureml. This is only necessary if you want to change the formatting or similar, if not it is sufficient to do a logger.setLevel(logging.DEBUG) or similar.

Example that would give you a local logger that "works" even with azureml:
import logging
LOG_FORMATTER = logging.Formatter("%(asctime)s — %(name)s — %(levelname)s — %(message)s")
handler = logging.StreamHandler()
handler.setFormatter(LOG_FORMATTER)
logger = logging.getLogger(__name__)
logger.addHandler(handler)
logger.setLevel(logging.DEBUG)
logger.propagate = False

This is just some quick input after me having a similar issue - I'm not saying this is exactly what you're experiencing :)

@storesund Thanks.
Your solution solved  the issue.  I just put the logging.basicConfig(level=logging.INFO, format='[%(asctime)s][%(name)s:' \                     '%(filename)s:%(lineno)d]%(levelname)s: %(message)s') before importing any azure library and I could control logging level.

seems this is solved (and should be relabeled to ML-Pipelines) @v-strudm-msft

#please-close


t001w (unknown_types)

Issue:
./src/ecc/#jcs#ckml.fugr.#jcs#lckmltop.abap[11, 9]                       - t001w (unknown_types)
Source:

Not reproducible in playground. Dependency Repo contains table.
Similar issue:
./src/ecc/#jcs#bdc_erp.fugr.#jcs#lbcf90.abap[103, 35]                    - msgtype (unknown_types)
Source:

tested with 2.38.15

part regarding sy-msgty fixed in 2.38.17

for t001w: try opening the project in vscode with the abaplint extension installed, https://marketplace.visualstudio.com/items?itemName=larshp.vscode-abaplint, and hover the mouse on T001W, then it should better show what is missing,

I've opened /jcs/lckmltop in vcode.

It has these errors:
Object name does not match pattern ^Z: /JCS/CKMLabaplint(object_naming)
Include /JCS/LBCMAC not foundabaplint(check_include)
Both errors seem to be wrong (first upper/lower maybe).

Could it be that not all types are retrieved by the dependency collection?
./src/ecc/#jcs#co_internal_order_balance.prog.abap[11, 9]                - Type of "coas" contains unknown types (unknown_types)

the table COAS has a strange field (also saw it in COVP) :

issue persists in 2.39.2

Can the message be improved to show which field is causing the problem?

yea, good idea

COAS star fixed in 2.39.4
error message improved in 2.39.4

COAS releated issues are gone.
the new error message produced this:
./src/ecc/#jcs#bdc_erp.fugr.#jcs#lbdc_erptop.abap[7, 12]                 - Type of "fb02_rec" contains unknown: Table BSEG, todo, group named INCLUDE (unknown_types)
./src/ecc/#jcs#ckml.fugr.#jcs#lckmltop.abap[11, 9]                       - Type of "t001w" contains unknown: Table T001W, todo, group named INCLUDE (unknown_types)
./src/ecc/#jcs#co_internal_order_balance.prog.abap[11, 21]               - Type of "ionra" contains unknown: Table IONRA, todo, group named INCLUDE (unknown_types)
./src/ecc/#jcs#fi_copa_bdc_fields.prog.abap[15, 6]                       - Type of "in_cobl" contains unknown: Table COBL, todo, group named INCLUDE (unknown_types)

Something goes wrong with the type assesment. The first error is from this code:

there is no BSEG there (tpit references are from a type-pool).
The 2nd one is just referencing itself.
The 3rd is a structure with appends
The 4th is a structure with includes and appends

ahh, okay .APPEND is actually .INCLU--AP after serialization

try again, 2.39.5

Several cases of this error are gone. Thanks
Issue:
./src/ecc/#jcs#fi_copa_bdc_fields.prog.abap[15, 6]                       - Type of "in_cobl" contains unknown: Type error, could not resolve cobl (unknown_types)
Source:
DATA in_cobl LIKE cobl.
COBL has multiple includes in include and also appends with includes.
Other cases also have includes with appends and appends with includes (like LFB1, WITH_ITEM).

try again, 2.39.6

They are all gone. Thanks


changing the coords of the obsidian plateform makes the player spawn at x=0 z=0.

I changed the coords to x=1200 y=50 z=1200, and I spawned inside the central bedrock portal and suffocated.

Fixed in 0.11.0


Enhance error report of Bootstrap demo server.

Thx @scop, integrated in master (commit f95d1da478ab121ec9145e3ae44a596d05c0a211)


doc issue in geometries geom3 toPoints() and invert()

Expected Behavior
the function signature is not properly documented
Actual Behavior
missing argument
Steps to Reproduce the Problem

go to https://www.openjscad.xyz/docs/module-modeling_geometries_geom3.html#.toPoints
go to https://www.openjscad.xyz/docs/module-modeling_geometries_geom3.html#.invert

Specifications

Version: production
Platform: all
Environment: all

Doc change has been released.


ValueError: 'none' is not a valid ControlIdentifier

Before submitting a new issue

[ ] I redownloaded the latest release and restarted Home Assistant
[x] I've checked known issues
[x] I've enabled debug logs and attached the relevant logs to this ticket

Problem description
During setup I get:
2024-09-06 18:10:06.213 ERROR (MainThread) [custom_components.mypyllant.coordinator] Unexpected error fetching myVAILLANT data
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 354, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/mypyllant/coordinator.py", line 213, in _async_update_data
    async for system in await self.hass.async_add_executor_job(
  File "/usr/local/lib/python3.12/site-packages/myPyllant/api.py", line 338, in get_systems
    control_identifier = await self.get_control_identifier(home.system_id)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/myPyllant/api.py", line 1213, in get_control_identifier
    return ControlIdentifier(control_identifier)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/enum.py", line 757, in __call__
    return cls.__new__(cls, value)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/enum.py", line 1171, in __new__
    raise ve_exc
ValueError: 'none' is not a valid ControlIdentifier

But the data collection works! I dont want to attached everything because of personal data, but be assured: The debug log shows all homes!
The line just before the trace is:
2024-09-06 18:10:06.100 DEBUG (MainThread) [custom_components.mypyllant] Refreshing DailyDataCoordinator
2024-09-06 18:10:06.100 DEBUG (MainThread) [custom_components.mypyllant.coordinator] Starting async update data for DailyDataCoordinator
2024-09-06 18:10:06.100 DEBUG (MainThread) [custom_components.mypyllant.coordinator] Waiting 119s until token refresh for<EMAIL_ADDRESS>2024-09-06 18:10:06.101 DEBUG (MainThread) [myPyllant.http_client] Starting GET to https://api.vaillant-group.com/service-connected-control/end-user-app-api/v1/homes
2024-09-06 18:10:06.157 DEBUG (MainThread) [myPyllant.http_client] Got response for GET to https://api.vaillant-group.com/service-connected-control/end-user-app-api/v1/homes: ........,"productMetadata":{"productType":"VR921","productionYear":"22","productionWeek":"42","articleNumber":"0020260962"},"state":"CLAIMED","migrationState":"FINISHED","migrationFinishedAt":"2024-03-19T20:05:30.354270431Z","onlineState":"ONLINE","firmware":{"version":"0357.40.35","connectedToOldIoTPlatform":null,"updateEnabled":true,"updateRequired":false},"nomenclature":"VR 921","cag":false,"countryCode":"DE","productInformation":"VR921","firmwareVersion":"0357.40.35"}]
2024-09-06 18:10:06.158 DEBUG (MainThread) [myPyllant.http_client] Starting GET to https://api.vaillant-group.com/service-connected-control/end-user-app-api/v1/systems/myhome/meta-info/time-zone
2024-09-06 18:10:06.212 DEBUG (MainThread) [myPyllant.http_client] Got response for GET to https://api.vaillant-group.com/service-connected-control/end-user-app-api/v1/systems/myhome/meta-info/time-zone: {"timeZone":"Europe/Berlin"}

Logs
see above

I have the same error, but in my case data collection also does not work. When I remove the integration and add it again, there are no devices or entities:
ValueError: 'none' is not a valid ControlIdentifier
2024-09-24 22:13:33.450 ERROR (MainThread) [custom_components.mypyllant.coordinator] Unexpected error fetching myVAILLANT data
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 354, in _async_refresh
    self.data = await self._async_update_data()
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/mypyllant/coordinator.py", line 213, in _async_update_data
    async for system in await self.hass.async_add_executor_job(
  File "/usr/local/lib/python3.12/site-packages/myPyllant/api.py", line 338, in get_systems
    control_identifier = await self.get_control_identifier(home.system_id)
                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/myPyllant/api.py", line 1213, in get_control_identifier
    return ControlIdentifier(control_identifier)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/enum.py", line 757, in __call__
    return cls.__new__(cls, value)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/enum.py", line 1171, in __new__
    raise ve_exc
ValueError: 'none' is not a valid ControlIdentifier
2024-09-24 22:13:34.815 WARNING (MainThread) [custom_components.mypyllant.binary_sensor] No system data, skipping binary sensors
2024-09-24 22:13:34.815 WARNING (MainThread) [custom_components.mypyllant.calendar] No system data, skipping calendar entities
2024-09-24 22:13:34.815 WARNING (MainThread) [custom_components.mypyllant.climate] No system data, skipping climate
2024-09-24 22:13:34.816 WARNING (MainThread) [custom_components.mypyllant.datetime] No system data, skipping date time entities
2024-09-24 22:13:34.816 WARNING (MainThread) [custom_components.mypyllant.number] No system data, skipping number entities
2024-09-24 22:13:34.816 WARNING (MainThread) [custom_components.mypyllant.sensor] No system data, skipping sensors
2024-09-24 22:13:34.816 WARNING (MainThread) [custom_components.mypyllant.sensor] No daily data, skipping sensors
2024-09-24 22:13:34.816 WARNING (MainThread) [custom_components.mypyllant.switch] No system data, skipping switch entities
2024-09-24 22:13:34.816 WARNING (MainThread) [custom_components.mypyllant.water_heater] No system data, skipping water heater

There should be a debug log for an API endpoint ending in /control-identifier:
Got response for GET to https://api.vaillant-group.com/service-connected-control/end-user-app-api/v1/systems/.../meta-info/control-identifier: {"controlIdentifier":"tli"}

Could you share that?

Yup:
2024-10-09 18:22:09.197 DEBUG (MainThread) [myPyllant.http_client] Starting GET to https://api.vaillant-group.com/service-connected-control/end-user-app-api/v1/systems/5cdd2c55-9bf3-4373-b215-71d1a5f68eef/meta-info/control-identifier
2024-10-09 18:22:09.272 DEBUG (MainThread) [myPyllant.http_client] Got response for GET to https://api.vaillant-group.com/service-connected-control/end-user-app-api/v1/systems/5cdd2c55-9bf3-4373-b215-71d1a5f68eef/meta-info/control-identifier: {"controlIdentifier":"vrc700"}


yarpc: refactor dispatcher to use shard key in transport header for history and matching client

Closing as this is an old issue with no context or details.


Use node syncing using rsync (instead of uploading)

I don't actually know what this means, but I saw it on http://groups.google.com/group/littlechef/browse_thread/thread/82a1838e5f479d15 and it sounded good:

Syncing
As the kitchen repository kept growing with many cookbooks and nodes,
uploading all files on every run was getting a bit slow. Markus
implemented node syncing using rsync (instead of uploading), which
speeds up things massively from the second run on.

How do configure rsync to "sync" instead of "upload"? And why would that be faster?
Should we at least add a --compress option? Would that speed it up at all?

Since we aren't preserving time -t we don't get the benefit of syncing and we upload every time.
Maybe @matschaffer has an opinion on why we aren't using -t.
We should also probably be using -z which compresses files before sending.

Pretty sure I just copied that from the spatula gem, so feel free to
patch if you have a more sane set of arguments.
On Feb 27, 2012, at 10:16 AM, Silas Baronda
<EMAIL_ADDRESS>wrote:

Since we aren't preserving time -t we don't get the benefit of syncing and we upload every time.
Maybe @matschaffer has an opinion on why we aren't using -t.
We should also probably be using -z which compresses files before sending.

Reply to this email directly or view it on GitHub:
https://github.com/matschaffer/knife-solo/issues/32#issuecomment-4195239

@silasb: Oh cool, so you think --times/-t would give us the "syncing" benefit littlechef's post was referring to? I'll give that a try...
Oh, I see -- so that allows rsync exclude any files that don't have a more recent modified time than the modified time on the remote host.
       -t, --times
              This tells rsync to transfer modification times along with the files and update them on the remote system.  Note that if this option is not used, the optimization that excludes files that have not been modified cannot
              be effective; in other words, a missing -t or -a will cause the next transfer to behave as if it used -I, causing all files to be updated (though rsync’s delta-transfer algorithm will make the update fairly  efficient
              if the files haven’t actually changed, you’re much better off using -t).

I agree, --compress/-z seems like it would be a good option to add. (I personally prefer the long version, --compress, since it's easier to read and know what it means -- with the short commands, I always have to go to man rsync and look them up.)
(I just modify my gems/knife-solo-0.0.8/lib/chef/knife/cook.rb file directly...)

+1 for longer arguments rather than shorter (especially since it's in ruby
code anyway). If you have any options that seem more efficient and don't
mess up the integration tests I'll be happy to pull them in.
Thanks for the help everyone!
On Thu, Mar 1, 2012 at 11:53 AM, Tyler Rick <
<EMAIL_ADDRESS>
wrote:

@silasb: Oh cool, so you think --times/-t would give us the "syncing"
benefit littlechef's post was referring to? I'll give that a try...
Oh, I see -- so that allows rsync exclude any files that don't have a more
recent modified time than the modified time on the remote host.
      -t, --times
             This tells rsync to transfer modification times along with
the files and update them on the remote system.  Note that if this option
is not used, the optimization that excludes files that have not been
modified cannot
             be effective; in other words, a missing -t or -a will cause
the next transfer to behave as if it used -I, causing all files to be
updated (though rsyncs delta-transfer algorithm will make the update
fairly  efficient
             if the files havent actually changed, youre much better off
using -t).

I agree, --compress/-z seems like it would be a good option to add. (I
personally prefer the long version, --compress, since it's easier to read
and know what it means -- with the short commands, I always have to go to
man rsync and look them up.)
(I just modify my gems/knife-solo-0.0.8/lib/chef/knife/cook.rb file
directly...)

Reply to this email directly or view it on GitHub:
https://github.com/matschaffer/knife-solo/issues/32#issuecomment-4261866


Compare two Clusterings interactively

It would be great to have a page dedicated to comparing the results of two clusterings of the same data.
This StackExchange post has many useful pointers for potential techniques to enable, as well as some of the challenges to consider.

I see that the MTEB evaluates the clustering capability of embedding models using V-measure (to compare a k-means clustering vs ground-truth labels): https://github.com/embeddings-benchmark/mteb/blob/main/mteb/abstasks/AbsTaskClusteringFast.py

V-measure is a metric that evaluates the quality of clustering by comparing the cluster assignments to the true labels. It's the harmonic mean of two other metrics: homogeneity and completeness.
Homogeneity: Measures whether each cluster contains only members of a single class.
Completeness: Measures whether all members of a given class are assigned to the same cluster.
The V-measure ranges from 0 to 1, where 1 indicates perfect clustering.

Some ideas from Claude for comparing clusters with variable sizes:

Calculate V-measure:
We can still calculate the V-measure between the HDBSCAN clusters and the true labels. The interpretation would be slightly different:
If HDBSCAN finds fewer clusters than true labels, a high V-measure would indicate that the embeddings are grouping semantically similar categories together.
If HDBSCAN finds more clusters than true labels, a high V-measure would suggest that the embeddings are capturing fine-grained semantic distinctions within categories.
Additional metrics:
We could introduce additional metrics to complement the V-measure:
Adjusted Rand Index (ARI) or Adjusted Mutual Information (AMI), which are also suitable for comparing clusterings with different numbers of clusters
Silhouette score to measure how well-separated the HDBSCAN clusters are
A measure of how close the number of HDBSCAN clusters is to the number of true labels


Google Music no longer scrobbling.

It looks like something changed? I'm not sure what, but the scrobbling is not working.

It works for me, though I am seeing some huge delays from last.fm, sometimes longer than a minute for a response.

Seems to be working now, yes. I think it is on Last.fm's end. Thank you for the verification.

On and off over the past few days, sometimes it'll scrobble, other times it won't. My google accounts were all logged out yesterday and after logging back in it's consistent -- no scrobbling through google music.

Not scrobbling Google Music here either.

L.FM was migrating to new website in past days, so that's maybe where those delays and outages came from.
I'm scrobbling just fine from Google Music now. Can you confirm? Also if the problem perists please post some console info

Appears to be working once again for me, as well. The extension has resumed showing notifications when I start playing something (previously it did not detect playback) so perhaps this is resolved—or what I was experiencing is unrelated.

Ok, I'll close this. Feel free to reopen if you get any more info on this


Meshlib creation popup content is misaligned

Tested versions
v4.3.beta.custom_build [62a056aa5]
System information
Godot v4.3.beta (62a056aa5) - openSUSE Tumbleweed
Issue description
Exporting a scene as a meshlib opens a popup to show progress while preview images are generated. In 4.3 the content of the popup is not centered but left aligned.

Steps to reproduce
Create scene with (many) meshes inside, Go to Scene->Export->Meshlib and export as meshlib.
Minimal reproduction project (MRP)
n/a

Thank you for reporting, consolidating in:

https://github.com/godotengine/godot/issues/92971

See there for more details, if you think something was missed about this and it's not the same issue, please comment here and it can be reopened


hipSYCL is treating C sources as C++

Hello,
I am working on a SYCL backend for a C/CUDA/OpenMP library. Turns out that some targets where SYCL is needed are also using C libraries, but when using hipSYCL's add_sycl_to_target, the C files are getting compiled as C++ ones (by syclcc) which does not work.
I tried using CMAKE_CXX_COMPILER_LAUNCHER instead of RULE_LAUNCH_COMPILE, in the cmake, but when targeting openMP the executable do not work. I'm not really sure of to fix that. Maybe, in syclcc-clang, if the input file ends in .c, add the -x c flag to the clang++/omp invocation?

SYCL is a C++ model. It's not possible to compile any sources that need SYCL as C. If you don't use SYCL in some source, don't compile as SYCL. If you tell it to compile things as SYCL, it has to compile as C++ to comply with the SYCL specification and to make the SYCL headers work.
I don't see a bug here.

Thanks for your answer, so I guess the only solution to support hipSYCL would be to build a shared object even if the SYCL backend exposes a C api to the rest of the library

You need some sort of interface layer, but I'm not sure you'd need a shared library. It's probably sufficient if you can separate translation units of code that needs SYCL and code that needs the C libraries.
This is a problem that I would expect with any SYCL implementation though since the SYCL headers are always going to require C++.


Build a MediaPipe AAR  error

Refer to the construction steps of the official website:
https://google.github.io/mediapipe/getting_started/android_archive_library.html

$ cd mediapipe/examples/android/src/java/com/google/mediapipe/apps/
$ mkdir aar_example
$ cd aar_example
$ touch BUILD

Put the following content in the BUILD file.

load("//mediapipe/java/com/google/mediapipe:mediapipe_aar.bzl", "mediapipe_aar")
mediapipe_aar(
name = "mediapipe_face_detection",
calculators = ["//mediapipe/graphs/face_detection:mobile_calculators"],
)

Run the Bazel build command to generate the AAR.

bazel build -c opt --strip=ALWAYS
--host_crosstool_top=@bazel_tools//tools/cpp:toolchain
--fat_apk_cpu=arm64-v8a,armeabi-v7a
--legacy_whole_archive=0
--features=-legacy_whole_archive
--copt=-fvisibility=hidden
--copt=-ffunction-sections
--copt=-fdata-sections
--copt=-fstack-protector
--copt=-Oz
--copt=-fomit-frame-pointer
--copt=-DABSL_MIN_LOG_LEVEL=2
--linkopt=-Wl,--gc-sections,--strip-all
--verbose_failures
//mediapipe/examples/android/src/java/com/google/mediapipe/apps/aar_example:mediapipe_face_detection.aar

Result: There is a build error, can you help me to see what the problem is?

Hi @siliconvallley!, Could you please try to use Android SDK release 30.0.0 or above and NDK version between 18 and 21. Please follow the instructions mentioned in the documentation. Thank you!

Hi @kuaashish，
Thanks for replying to my issue.
My problem is solved.

First I find the file "mediapipe_aar.bzl" which outputs error log 1,
then through error log 2 "/bin/bash:$'\r': command not found"
I guess the current error should be in "mediapipe_aar.bzl" caused by newlines,
so I did the following to the "mediapipe_aar.bzl" file:
sed -i 's/\r$//' mediapipe_aar.bz
solved the problem.

Hi @siliconvallley, Thank you for letting us know. Could we move this issue to closed status?

Hi @kuaashish，Thank you for your reply. I think this issue should be closed.

Hi @siliconvallley, Thanks for the confirmation! Closing this issue as it resolved.


Update kubebuilder to v3

resolve #36

Looks good, please go ahead.

Oops, I approved during review by mistake.

@toshipp I fixed the role name. Would you review again?


Libpqxx7

This change removes a few extra memory allocations that were needed until libpqxx updated.
This also shows that https://github.com/jtv/libpqxx/issues/184 can be closed.

@KayEss I am going to merge this one.


Headers support

Can I with Ashton make header support?
I see there is no ashton intermediate header representations.
If i read Wow! Now iOS can create
NSAttributedString
 from HTMLs! markdown output would be "**Wow**! Now *iOS* can create\n\n**NSAttributedString**\n\nfrom HTMLs!\n\n" but it will be cool if output can be the same as input string.
Sorry, I'm not quite understand how attributedString work yet.
Thanks.

As this request has been open for some time and as we don't require it for our own app, I'll close this issue. If you require this feature please reopen the issue and attach a PR. Thank you.


Log phase completion times for perf profiling

{"level":"info","ts":[...],"logger":"[...]","msg":"[RUN] (Step 9/23) Checking whether the created PVCs are bound","DVM":[...]}
{"level":"info","ts":[...],"logger":"[...]","msg":"Phase completed",[...],"Phase": "StartRefresh", "Elapsed":0.522302}

Logs formatted in this way allows us to use a nasty bash one-liner to generate a performance report of which phases are taking the longest to run.
make run-fast 2>&1 | tee controller.log

echo "PhaseName ElapsedSeconds PercentOfTotal" > phase_timing.csv; total_time=$(cat controller.log | grep '\"Phase\ completed\"' | jq '([inputs | .Elapsed] | add)'); cat controller.log | grep '\"Phase\ completed\"' | jq -r --arg total_time "$total_time" '"\(.Phase) \(.Elapsed) \(.Elapsed / ($total_time|tonumber))"' >> phase_timing.csv

PhaseName ElapsedTime PercentOfTotal
StartRefresh 49.691784 14.405797870070389
WaitForRefresh 17.651623 5.117258680362241
CleanStaleAnnotations 2.386096 0.6917364181286685
CleanStaleResticCRs 3.936635 1.1412423449768787
CleanStaleResticCRs 7.816086 2.265906876096197
CleanStaleVeleroCRs 7.983351 2.314397503454984
RestartVelero 1.007199 0.2919900241242501
CleanStaleStagePods 1.669287 0.4839313297573738
WaitForStaleStagePodsTerminated 2.195697 0.6365391744824445
WaitForVeleroReady 0.843389 0.24450101167309263
EnsureCloudSecretPropagated 5.634364 1.6334190962111823
PreBackupHooks 0.762824 0.2211449754840473
CreateDirectImageMigration 0.912454 0.264523163220246
EnsureInitialBackup 3.609576 1.0464269556644856
Started 2.006556 0.5817066288257422
EnsureInitialBackup 4.150545 1.2032554983461916
Prepare 1.541324 0.4468344705895118
CreateDestinationNamespaces 3.497918 1.0140569651128017
ListImageStreams 2.87607 0.8337813566961764

And from this CSV we can easily create a Google Sheet with color formatting that highlights which phases are taking the longest.

@alaypatel07 updated, PTAL

So this is not for Jaeger for I assume gather this output in a must gather and being able to get this performance report with that awesomeness that is a bash one liner!
What happens if we requeue a phase?

So this is not for Jaeger for I assume gather this output in a must gather and being able to get this performance report with that awesomeness that is a bash one liner!

Yeah I was mainly intending this to be used for local dev to easily turn logs into a perf profile but since must-gather collects logs we can just as easily use this to understand perf on the user clusters.


SIM-2782: Replace the narrative plan in the prompt with "extra" infor…

…mation that is sent by the client.

👀 Found no PR bugs during testing.
Looks great and the responses seem more knowledgeable.


Back again: daft io_glob bug

Describe the bug
daft.daft.io_glob does not work somestimes.
daft.version
'0.1.20'
Ubuntu, x86
Here's the problem. I have some files under s3://cluster-dump/big_hadoop/split_0/
daft.daft.io_glob("s3://cluster-dump/") works and returns files
daft.daft.io_glob("s3://cluster-dump/big_hadoop/split_0/") works and returns files
daft.daft.io_glob("s3://cluster-dump/big_hadoop/") returns empty!
I need to do the last command to figure out how many splits there are.

Currently daft.daft.io_glob should only return files!
We are unable to return directories because of the algorithm that we use (falling back onto prefix-listing does not return "directories").


Fix bug on release publisherid migrations

If the releases already have publisher id, the migration in v156 will override them. It's horrible.
This PR will fix that.

Sorry I missed that it was not intentional - I had thought that was intentional.

make L-G-T-M work.


chore: use cert-mananager.io/v1 group/version

What this PR does / why we need it:
Changes cert-manager resources to those whose group/version are "cert-manager.io/v1"
This was tested on a kind cluster with the latest version of cert-manager.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

@jimmidyson here is what you were suggesting from PR: https://github.com/kubernetes-sigs/kubefed/pull/1288. This would be a breaking change. The alternative approach would be to change the defaults to those we manually set here. Let me know what you prefer.

/assign @irfanurrehman

/lgtm

@alejandroEsc thanks for doing this.
/approve


"Find my device" can't find the device location

Please allow the "find my device" service. It's very necessary sometimes in the emergency.

Redmi note 11, crdroid 8.9, android 12.1
Google play services version 22.33.16<PHONE_NUMBER>95102)
Find my device option is ON, Location is also ON. But both find my device and google maps location is not working.

Thanks. Trying and letting you know

Fixed now. Thanks.


[DOCS] Move installation instructions into INSTALL.md

Description
How about moving the installation instructions from README.md into a file like INSTALL.md?
This would allow README.md to be much more concise -- a project description, table of contents, and highly relevant links (Discord, etc.)

maybe we can name it CONTRIBUTING.md
e.g., my favorite repos:

cypress/CONTRIBUTING.md
Typescript/CONTRIBUTING.md
axios/CONTRIBUTING.md

Description
How about moving the installation instructions from README.md into a file like INSTALL.md?
This would allow README.md to be much more concise -- a project description, table of contents, and highly relevant links (Discord, etc.)

Hi @veganaize I don't think that it is needed at this point of time

The current README.md weighs in at 90 LOC (lines of code) which doesn't strike me as too large at all. In fact, many repos have massive readmes that include everything about a library (take zod's 2224 line beast for example). Granted, the well structured table of contents definitely helps out.
I'll be honest, I'm not even sure it's necessary at this point (I might wait until >200 LOC), but one alternate resolution for this ticket could be an addition of a well-linked Table of Contents to README.md, which would perhaps make it more navigable for newcomers.


Find the rigth hexagons levels

Analyzing the dimension of census' cells between 1991 and 2011, I decide to use as minimum extension the median of surface extention from 1991 and as maximum extension the mean from 1991:

Minimum census cell extension: 0.02 kmq
Maximum census cell extension: 0.77 kmq

Based on the previous data, hexagons cells range is from 8 to 10.


Error in TicketCenter, when regular user access that.(develop branch)

When regular user access ticket center index page, there is throw an exception that is System.NullReferenceException.
Because the default listName has been set up for "myTickets" by Index action in TicketCenterController ,

TicketDesk/TicketDesk-2.5/TicketDesk.Web.Client/Controllers/TicketCenterController.cs
Line: 49
listName = listName ?? (Context.SecurityProvider.IsTdHelpDeskUser ? "unassigned" : "myTickets");

But the listName is "mytickets" in the DefaultListSettings.

TicketDesk/TicketDesk-2.5/TicketDesk.Domain/Model/UserSettingsModel/UserTicketListModel/
Line:185
settings.Add(new UserTicketListSetting("mytickets", TicketDeskDomainText.DefaultListNameMyTickets, disOrder++, 20, mySortColumns, myFilterColumns, disableOwnerColumn));

So, I change this code at UserSetting.cs in my project.

TicketDesk/TicketDesk-2.5/TicketDesk.Domain/Model/UserSetting.cs
Line: 3
return ListSettings.FirstOrDefault(us => us.ListName == listName string.Compare(us.ListName, listName, true) == 0);

The above is provided for your reference.

Thanks again Wootz! I'll get that fixed up asap. I've been moving a lot of code in the last couple of weeks, but I need to do a better job with regression testing.


Update slf4j-api to 2.0.0

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

Files still referring to the old version number
The following files still refer to the old version number (1.7.36).
You might want to review and update them manually.
CHANGELOG.md

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.slf4j", artifactId = "slf4j-api" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.slf4j", artifactId = "slf4j-api" }
}]

labels: library-update, early-semver-major, semver-spec-major, old-version-remains, commit-count:1

Superseded by #340.


Slack is asking to allow popups on Ferdi

Bug description:
clicking on links in slack won't pop up on a browser you need to manually copy them. (not all links)
there is a difference between links shared within slack, and admin preferences which automatically pop up a small ferdi-browser window with the slack-admin web page - which is partially nice - but I think it'll be better to appear on default computer browser)
Steps to reproduce:
Certain links won't open, it's pretty straight forward
Expected behavior
Links should open in the default browser of the computer
Screenshots
The following message appears from the bottom of the slack window.

Environment:

Operating System: windows 11 pro

Ferdi Version: 5.6.2 latest (not nightly)

Server: Ferdi without account

Debug information:
Couldn't get that

I confirm that it happens, and it's a deal-breaker issue.


WIP

The file you are replacing it is coming from one of our repos. Are you just testing this here before setting it up in the swiftnav/toolchains repo?

The file you are replacing it is coming from one of our repos. Are you just testing this here before setting it up in the swiftnav/swift-toolchains repo?

Yes exactly, this is an old local test branch I pushed to share with @IsakTjernberg , I set it as "Invalid" and "do not merge"

I did not see you were automatically assigned as reviewer 😅


Envoy Filter Or Command %REQ(X?Y):Z% not working as expected

Bug Description
Hi, regarding the documentation: https://www.envoyproxy.io/docs/envoy/v1.22.5/configuration/observability/access_log/usage#config-access-log-format
If we have an envoy filter and we set a header with an or command %REQ(X?Y):Z%, the envoy filter will skip this configuration and will skip this header.
Steps to reproduce:
# Install a modern istio
export ISTIO_VERSION=1.15.1
curl -L https://istio.io/downloadIstio | sh -
./istio*/bin/istioctl install -y

# Install some apps
kubectl label ns default istio-injection=enabled
kubectl apply -n default -f https://raw.githubusercontent.com/istio/istio/release-1.15/samples/httpbin/httpbin.yaml
kubectl apply -n default -f https://raw.githubusercontent.com/istio/istio/release-1.15/samples/extauthz/ext-authz.yaml

# Expose a route
kubectl apply -f - <<EOF
apiVersion: networking.istio.io/v1beta1
kind: Gateway
metadata:
  name: api-gateway
spec:
  selector:
    istio: ingressgateway
  servers:
  - port:
      number: 80
      name: http
      protocol: HTTP
    hosts:
    - "*"
---
apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  name: api-gw-vs
spec:
  hosts:
  - "*"
  gateways:
  - api-gateway
  http:
  - route:
    - destination:
        host: httpbin.default.svc.cluster.local
        port:
          number: 8000
EOF

# Apply extauthz
kubectl apply -f - <<EOF
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: authn-filter
  namespace: istio-system
spec:
  configPatches:
    - applyTo: HTTP_FILTER
      match:
        context: GATEWAY
        listener:
          filterChain:
            filter:
              name: envoy.filters.network.http_connection_manager
      patch:
        operation: INSERT_BEFORE
        value:
          name: envoy.filters.http.ext_authz
          typed_config:
            '@type': type.googleapis.com/envoy.extensions.filters.http.ext_authz.v3.ExtAuthz
            http_service:
              server_uri:
                cluster: outbound|8000||ext-authz.default.svc.cluster.local
                timeout: 10s
                uri: http://ext-authz.default.svc.cluster.local:8000/status/200
              authorizationRequest:
                allowedHeaders:
                  patterns:
                  - exact: cookie
                    ignoreCase: true
                  - exact: x-forwarded-access-token
                    ignoreCase: true
                  - exact: x-ext-authz
                    ignoreCase: true
                headersToAdd:
                - key: x-auth-request-redirect
                  value: '%REQ(x-h1)%%REQ(x-h2)%'
  workloadSelector:
    labels:
      istio: ingressgateway
EOF

GATEWAY_IP=$(kubectl get svc -n istio-system istio-ingressgateway -o jsonpath="{.status.loadBalancer.ingress[0].ip}")

# This is accepted
curl -H "x-ext-authz: allow" -H "x-h1: h1" -H "x-h2: h2" http://$GATEWAY_IP/get

# This is rejected
curl -H "x-h1: h1" -H "x-h2: h2" http://$GATEWAY_IP/get

#check logs
kubectl logs deploy/ext-authz

# Now change the value in the envoyfilter for '%REQ(x-h1)%%REQ(x-h3?x-h2)%' , it will stop working, the header will not appear any more nor with h2 nor h3

Version
1.15.2

Additional Information
No response

This seems to be a purely Envoy thing, may make sense to file an issue there instead?

Hi, yes that make sense, I will this one then.

Hi, from envoy, they told us to please submit the issue in Istio: https://github.com/envoyproxy/envoy/issues/23727

@edubonifs When you write the issue in envoy, can you write in the context without Istio?

Hi, I wrote the issue in envoy as requested, but it seems to properly work with purely envoy, may be we missing something from Istio side? https://github.com/envoyproxy/envoy/issues/23753

Can you please share config_dump?

Config dump from gateway shows proper configuration:
                 "headers_to_add": [
                  {
                   "key": "x-auth-request-redirect",
                   "value": "%REQ(x-h1)%%REQ(x-h3?x-h2)%"
                  }

I am looking at this issue and will provide update.

@edubonifs I figured out what is going on. Despite the name similarity, access log's %REQ(...)% is not the same as header manipulator's %REQ(...)%. The documentation you referred to https://www.envoyproxy.io/docs/envoy/v1.22.5/configuration/observability/access_log/usage#config-access-log-format applies only to generating access log output. For manipulating headers (what to add or remove) you have to use this documentation: https://www.envoyproxy.io/docs/envoy/v1.22.5/configuration/http/http_conn_man/headers.html [search for %REQ(]. As you will see, header manipulator does not allow for %REQ(X?Y):Z% format.
Now, this was the case until Envoy's release 1.24. In release 1.24 you can use all access log's formatters to manipulate headers. See https://github.com/envoyproxy/envoy/issues/20389. Let me know if this is clear or you need more info.
This is not istio problem and this issue can be closed.

Thanks! Envoy 1.24 will be in Istio 1.16


ENYO-5365: Fixed overscroll effects on improper timing

Issue Resolved / Feature Added

Fixed not to show an overscroll effect when a focus moves between paging control buttons right after a list reached to the edge.
Fixed to show an overscroll effect by the first wheeling on native lists.

Resolution
The internal status variable is shared for effects on both edges (beginning and ending), so updating the variable by reaching one edge affected an effect for the other edge. In this PR, effects for both edges have their own status variable.
Links
ENYO-5365
Comments
Enact-DCO-1.0-Signed-off-by: Seungcheon Baek<EMAIL_ADDRESS>

Hmm. it's a just clearing code and is not required. I will change the name of the first function.

I found there was one issue with dragging.
If dragging vertically to show overscroll effect on the top or on the bottom while showing overscroll effect on the left or on the right, the vertical and horizontal overscroll effects are hidden at the same time even though the horizontal overscroll effect was shown first.
I think it is the reason to call clearAllOverscrollEffects() in https://github.com/enactjs/enact/blob/8104e49a5352f574126d472734b4fad4a9872031/packages/ui/Scrollable/Scrollable.js#L560, then all overscroll effect are hidden at the same time.

Thanks for reporting. I have already one ticket to handle drag cases and I will check this also.

@ybsung

I found there was one issue with dragging.
If dragging vertically to show overscroll effect on the top or on the bottom while showing overscroll effect on the left or on the right, the vertical and horizontal overscroll effects are hidden at the same time even though the horizontal overscroll effect was shown first.

I thought you pointed out some flicking, but it looks a dragging case. All effects by dragging should be gone at the same time. It is not a problem. If I misunderstood, please let me know.

To make relations between internal functions clearer and to remove redundant check codes, I updated to check conditions (to show effects) within a requesting methods (input handlers, scrolling logics, etc) instead of effect handling methods. It is a simple change of a concept but most code for overscroll effect is affected.

LGTM


Required flags (take 2)

🚧WIP - not in a reviewable state 🚧
builds off of https://github.com/urfave/cli/pull/155
closes https://github.com/urfave/cli/issues/85
🚧WIP - not in a reviewable state 🚧

✔ rebase is done and tests are passing again! I haven't dug into any of the actual logic yet, but this is a good start 🏃♀

Focus points:

app.go :: Run, RunAsSubcommand <= WIP, has a relevant review comment https://github.com/urfave/cli/pull/155#discussion_r21725190
context.go :: checkRequiredFlags <= has tests ✔need to check the coverage
flag.go :: withRequiredHint, withHints <= WIP

Personal note: I have a 🔐private cli 🔐that uses required flags, and it'll be nice to ensure that this implementation can replace the code I have in that private repo.

I will return to this PR in a few day(s) 🕐

Almost ready for 👀! Stuff that's still TODO:

checking that the withHints thing is actually working? I didn't add that
adding tests for sub-command handling (it looks like the flag parsing code is copy pasted for sub-commands?)

Hey do you need any help with this pull request? I am currently needing this feature in my project :)

@MichaelDao I think the pull request will be 99% done once I commit and push my last few local changes. I'll spend an hour on it and see where I get.

@MichaelDao this PR needs 3 things:

manual testing. So pull down this branch, and try to make a required flag.
general code review
a specific review of all of the items I mention in "Current Unknowns" in the PR description

I'm probably not going to work on this much more until someone helps out with the "Current Unknowns" and "Testing and Verification" notes I mention above. It's a large-ish PR that I've invested quite some time in, so I want to get some verification that I'm moving in the right direction.
cc @meatballhat I'd appreciate some help here!

I've got this working on my personal branch now, it looks like it needs a bit more work!

✔  got required flags working in my 🔐private CLI 🔐
I've just got to look at the environment variable bit, I think

This is ready for 👀 again @AudriusButkevicius! Here's a view that mostly only shows the changes since you last saw the PR https://github.com/urfave/cli/pull/819/files/9ec594d5290b846de59b0bc350849b848cabfbd7..60fb2972328d6a7487c6821a58a86d476167c2bd
In summary:

removed all the showFlagError stuff
added environment variable support
added a errRequiredFlags and tests to check for it
generally added a bunch more tests

@asahasrabuddhe would you mind reviewing this too? thanks in advance ✨🙏✨

Let me take a look :)

Thanks for adding in this feature @lynncyrin, is there a guide on how to use the RequiredFlag type? I am a little lost on how i can apply this

@MichaelDao Check #85

I just noticed the followup comments ^^ there should definitely be a tiny readme section about how to use this! someone should add that 👀


djanjo module zerver.worker.queue_processers error

Logger root, from module zerver.worker.queue_processors line 371:
Error generated by Anonymous user (not logged in) on meslinux deployment
Traceback (most recent call last):
File "/home/zulip/deployments/2022-02-05-22-51-49/zerver/worker/queue_processors.py", line 305, in do_consume
consume_func(events)
File "/home/zulip/deployments/2022-02-05-22-51-49/zerver/worker/queue_processors.py", line 530, in consume_batch
do_update_user_activity(user_profile_id, client_id, query, count, log_time)
File "/home/zulip/deployments/2022-02-05-22-51-49/zerver/decorator.py", line 830, in wrapped_func
ret = func(*args, **kwargs)
File "/home/zulip/deployments/2022-02-05-22-51-49/zerver/lib/actions.py", line 5102, in do_update_user_activity
activity.save(update_fields=["last_visit", "count"])
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/models/base.py", line 727, in save
force_update=force_update, update_fields=update_fields)
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/models/base.py", line 765, in save_base
force_update, using, update_fields,
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/models/base.py", line 846, in _save_table
forced_update)
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/models/base.py", line 899, in _do_update
return filtered._update(values) > 0
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/models/query.py", line 802, in _update
return query.get_compiler(self.db).execute_sql(CURSOR)
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/models/sql/compiler.py", line 1559, in execute_sql
cursor = super().execute_sql(result_type)
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/models/sql/compiler.py", line 1175, in execute_sql
cursor.execute(sql, params)
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/backends/utils.py", line 66, in execute
return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/backends/utils.py", line 75, in _execute_with_wrappers
return executor(sql, params, many, context)
File "/home/zulip/deployments/2022-02-05-22-51-49/zulip-py3-venv/lib/python3.6/site-packages/django/db/backends/utils.py", line 84, in _execute
return self.cursor.execute(sql, params)
File "/home/zulip/deployments/2022-02-05-22-51-49/zerver/lib/db.py", line 34, in execute
return wrapper_execute(self, super().execute, query, vars)
File "/home/zulip/deployments/2022-02-05-22-51-49/zerver/lib/db.py", line 19, in wrapper_execute
return action(sql, params)
File "/usr/lib/python3.6/encodings/utf_8.py", line 15, in decode
def decode(input, errors='strict'):
File "/home/zulip/deployments/2022-02-05-22-51-49/zerver/worker/queue_processors.py", line 351, in timer_expired
raise WorkerTimeoutException(limit, len(events))
zerver.worker.queue_processors.WorkerTimeoutException: Timed out after 30 seconds processing 1 events
Deployed code:

git: None
ZULIP_VERSION: 4.9

Request info: none
we updated the SMTP settings on for our outgoing email messages from zulip, then we started getting these messages. I am not sure what they mean. Can someone point me in the right direction?

I don't think this has anything to do with SMTP configuration; the exception is that a simple database query to update "last visit" stats for a single user is taking 30s+ to execute on your database.  My guess is some sort of networking issue in your connection to the database server, or a hardware issue with the database server's disk.
This is unlikely to be a bug in Zulip, so I'd recommend a chat.zulip.org thread if you need further help investigating.  (The issue tracker is intended for things that someone on the Zulip team can address).

Same error, +1. Did you resolve this?

No I have not. We still get this alert randomly. I have not had time to investigate it further. In our system, the database is on the same VM as the linux server. So it seems odd that it would be a connection issue to the database.
I'd be interested if you come up with anything.

Inspired by
https://github.com/zulip/zulip/issues/11788
I added this configuration
https://zulip.readthedocs.io/en/latest/production/email.html#troubleshooting
because (mentioned in these docs)
https://medium.com/intigriti/how-i-hacked-hundreds-of-companies-through-their-helpdesk-b7680ddc2d4c

Having this problem now. Did anyone ever find a fix?


Display for cluster deliveries with a unique color marker

What
We are currently not showing deliveries marked for cluster deliveries on https://crownheightsma.herokuapp.com/delivery-needed.  They should be color coded and differentiable from the other deliveries, maybe even have an option to toggle them.
Why
To help cluster delivery folks out. They are currently using the google maps view in Airtable, which has limitations. There will likely be requests for future features, but for a start, just start showing them on the map.

closed with https://github.com/crownheightsaid/mutual-aid-app/pull/79


Split the UmpleOnline Dockerfile to save space

Description
The header of the Dockerfile that installs all the dependencies is now
split out into an image called umpleonline-base. It should not need to
change much and saves 200MB from the full UmpleOnline image size.
The Travis build has been altered to base its Dockerfile on that image, and
rebuild it on tag builds.
Tests
As with other build system modifications, all tests are live.
The Dockerfiles have been tested independently, hopefully the tag build logic works.

That's odd... all my stuff worked, but the build failed on testbed. Any thoughts @TimLethbridge ?

So two things here. First the pr build is passing, which is what matters. Not sure why the branch build is failing, but I have seen that before. Something to do with resetting version.
Second issue is that when I pull the docker image for this PR I don't see an image that makes use of the new base image. Not sure why.

@TimLethbridge thanks for checking, I've relaunched the push build just as a matter of principle (need to test the PR builds branches correctly, though I think it does). Looks ok one way or another,
As for you not seeing the new base image, I agree in that I can't really see it either. When I pulled the PR image however, it did only download ~40mb of data so I think it works, it's just not obvious. When (hopefully) the branch build passes, you can pull that too and you should see a significant improvement. Since I imagine the PR build was the first one based on my changes you downloaded, it would have had to download the base image as well at that point. That should be a one time cost.

I give up on the push build, I have no idea how to make it pass.
On the other hand, I reran the pr build so if you pull that again you should only get the UmpleOnline files. If you view it in docker history you should see distinct creation times for the 200mb of dependencies and the other files. Your download time should also be shorter.


make computeProxyState more readable

Please provide a description of this PR:

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-istiodremote-tests_istio

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

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-istiodremote-tests_istio

integ-pilot-multicluster-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-multicluster-tests_istio

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

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-istiodremote-tests_istio

integ-pilot-multicluster-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-multicluster-tests_istio

integ-pilot-k8s-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-k8s-tests_istio

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

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-istiodremote-tests_istio

integ-pilot-multicluster-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-multicluster-tests_istio

integ-pilot-k8s-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-k8s-tests_istio

integ-cni-k8s-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-cni-k8s-tests_istio

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

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-istiodremote-tests_istio

integ-pilot-multicluster-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-multicluster-tests_istio

integ-pilot-k8s-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-k8s-tests_istio

integ-cni-k8s-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-cni-k8s-tests_istio

lint_istio
515f3c9e63ade37ae66c95a698d9b5e12ae453b3
link
true
/test lint_istio

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

@hzxuzhonghu: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-k8s-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-pilot-k8s-tests_istio

integ-cni-k8s-tests_istio
c39dd8bf2b7bd8549152037f197c996764d2a45e
link
true
/test integ-cni-k8s-tests_istio

lint_istio
1b6cab9e52b933de21d9584670aa0a6800e00ceb
link
true
/test lint_istio

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

/retest


Fixed bundle problem

This change allows other classes to extend SVProgressHUD.

Hi Michael, danke für den PR. Könntest du kurz erklären warum es das Bundle-Problem löst?

Hey. I am sorry but I do not speak German :(
Problem is that when SVProgressHUD is bundled with CocoaPods it stays in Pods bundle. Then you get NSBundle from Pods. However, when you externalise it in your project then NSBundle returns you bundle of your app that does not have SVProgressHUD.bundle file.

Hi, i thought you might speak german because you are located in Berlin ;). Sorry for that. However, it seems you understood my question and yes very good explanation. Thanks for this good catch.

Google Translate :)


Web app is slow

Ever since the update, the web app is running extremely slow.
Not only do analyses move slowly, but if you refresh the screen after an analysis, it seems to recalculate that -- rather than just showing the previously computed analysis
-Tevis

It does not recalculate. It is just slow at loading things in the background. I am not sure what it is.
This has to be the first priority of @mcrot when he comes back from vacation.

@pastewka, a naive question:
I know the web app stores the results of prior analyses so that it doesn't need to recompute. The purpose of this is to reduce waiting time if a user performs an analysis that has already been performed.
However, it appears that the biggest delay is in visualization, not in calculation.  So is it possible to store the visualization of the prior analysis also?
Probably this is not feasible, but if it was, this would help to achieve the same objective -- fast loading of a recomputed analysis.

@tdbjacobs , we think the performance improvements were effective. Could you have a second look how fast it is now on your side?


Return OS specific default GOPATH if not set

See: https://github.com/golang/go/issues/17262
Fixes: #52

Not sure if this is still live, but there's no reason to ad-hoc it.  The standard library package go/build has a global build.Default.GOPATH that returns the currently set GOPATH, using the environment variable if set, and the default for that operating system if it is not.  You can simply unconditionally return the value of that variable.

@kaedys Thanks for the hint.
After looking at go/build, I found, that maybe even better than using build.Default.GOPATH would be to use build.Default.SrcDirs(), as this would simplify stripGopath as well and it does handle some special/edge cases as well (like unquoted ~ which was not expanded to $HOME and special cases on different OS).
@matryer if you like this, I can update the PR accordingling.

Note that SrcDirs() returns a slice, for which the first item is the GOROOT.  Looking at your updated code, it should work in most cases, but I could see it failing in a couple edge cases, notably if the $GOPATH is nested in the $GOROOT (which is legal, though strongly discouraged).
For example, of $GOROOT = ~/go and $GOPATH = ~/go/code, then the absolute import path for a package might be, for example, ~/go/code/src/github.com/foo/bar.  Our goal is to get back out github.com/foo/bar.  Problem is, it first looks at the $GOROOT, as that's the first item in the list, stripping p to /code/src/github.com/foo/bar.  This no longer matches the $GOPATH, and thus p is returned as code/src/github.com/foo/bar, which isn't a valid canonical import path (unless you happen to have a path that matches ~/go/code/src/code/src/github.com/foo/bar).
For the purposes of just stripping the GOPATH from the front of the package path, I think you really should just use build.Default.GOPATH.  Also not sure what you mean about not expanding the tilde character.  Running my tester program prefixed by GOPATH=~/testtest, my build.Default.GOPATH is properly set to /Users/<me>/testtest.

About ~ see comment here: https://github.com/golang/go/blob/master/src/go/build/build.go#L220
About $GOPATH inside $GOROOT you are right, but this is als true for the (rather unusual) case where someone puts a $GOPATH inside another $GOPATH, which I think would be (technicaly) possible as well. This case is also not covered now.

About ~ see comment here

That comment is about quoted tilde paths, not unquoted ones.  Unquoted ones are already properly expanded into build.Default.GOPATH.  Quoted ones, per that comment, aren't even legal paths, and thus shouldn't be supported anyway.


:package:  fix cwe serialize-javascript

Description
Fix CWE issue reported by Snyk related to dependency serialize-javascript < 6.0.2
Checklist

[x] My changes follow the contributing guidelines of this project.
[x] I have performed a self-review of my code.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my changes work.
[ ] New and existing unit tests pass locally with my changes.

coverage: 94.01%. remained the same
when pulling ad87a01385d0f519411ef3fd9c721b52fcc42acd on chore/fix-cwe-79
into 42b434161182b31c10cf3c43e3868a99ce013c28 on main.


:focus-visible actually work on chrome

I can use :focus-visible on my chrome (69.0.3497.100 mac) although that's red on https://caniuse.com/#feat=css-focus-visible

Disable the required flag and send us a screenshot. Are you sure you have disabled the needed flag before you send this?

According to chrome status, it is still under flag.

You're right, thanks and sorry


animepahe has weird resolutions

Describe the bug

Animepahe api returns resolutions like 692p, 1036p etc. We should think how to handle it
 {
'data':
	{'65723':
		{'692p': {'filesize': 405211235, 'crc32': 'a04be349', 'fansub': 'Reta
il', 'revision': '', 'disc': 'BD', 'url': 'https://kwik.cx/e/dasvaUC0j8xn', 'server': 'lux'},
		'1036p': {'filesize': 711722610, 'crc32': '9480ed25', 'fansub': 'Ret
ail', 'revision': '', 'disc': 'BD', 'url': 'https://kwik.cx/e/sjcJmankXfFk', 'server': 'lux'}}}}

To reproduce

anime dl 'nauroto' --provider animepahe --url

Choose one of the shippuden movies

A convoluted fix probably involves dynamically setting both the qualities on the AnimepaheEpisode class and the AnimePahe class since if you don't do it for the latter an error will occur... on second thought this might also be a no go since the base Anime class checks to see if the quality entered by the user is one of the qualities in the QUALITIES list.
We could avoid the tight coupling between the Anime and AnimEpisode classes following the Single Responsibility Principal. Because right now the mere fact that the AnimEpisode class has to check on the qualities of the Anime class allows it to somehow skip over what it actually has available. So for AnimepaheEpisode maybe override the init or just change the general structure.
Because this will persist even for other grabbers.

I would say to ignore the quality, as that's what we are doing for a lot of sites. Quality is not provided by a lot of sites, therefore we just ignore what the quality is right now. This is not a good option but I can't think of any other way which don't harm the user experience.
Maybe we can somehow round the resolutions to standard ones? I don't want to expose these weird resolutions to the user.

If the we ignore the qualities then we risk being unable to get any of the episodes. The program has done a good enough job ignoring them to begin with so that's okay I think. But perhaps we could have a base episode quality list some of the episode classes can decide if they are capable of fetching episodes?

Closing, resolution change would add dependencies. We should just use the resolutions provided by a provider. We successfully scraped the video, also AnimePahe is broken rn


ollama: 0.1.48 -> 0.2.1

Description of changes
changelog: release 0.2.0
changelog: release 0.2.1
changelog: release 0.2.2
changelog: release 0.2.3

Things done

Built on platform(s)

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

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

[x] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Diff looks good except for the vendorHash, did it not need to be updated?
However, I happened to choose today to update ollama as well, so I just recently created #326867 before seeing that you had created this. Since my pr updates to 0.2.3 which is more recent, should this pr be closed?

Initially I made the PR for 0.2.3 but wasn't sure if I should've used that version or 0.2.1 as it was the latest release according to ollama's repo. I just went back to 0.2.3 again so I think it's best to keep this one.

I just checked ollama's releases page again, and 0.2.1 is now marked as the latest even though 0.2.3 was marked as latest just a couple hours ago. I think you may have been right in only updating to 0.2.1; perhaps 0.2.2 introduced multiple issues that are just now surfacing.
It's probably best to only update to 0.2.1 for now.

I see, that makes sense. I'll revert to 0.2.1

Ollama just declared 0.2.4 as latest version, here we go again xD

Do you want to (in order of personal preference):

leave this pr open to wait and see if 0.2.4 is relatively stable
merge 0.2.1 for now and wait and see if 0.2.4 is relatively stable
merge 0.2.4 and hope for the best

I may prefer option 1, but I'm not against any of them.

Do you want to (in order of personal preference):
1. leave this pr open to wait and see if 0.2.4 is relatively stable

2. merge 0.2.1 for now and wait and see if 0.2.4 is relatively stable

3. merge 0.2.4 and hope for the best

I may prefer option 1, but I'm not against any of them.

I vote for merging 0.2.1 for now, as it's already pretty stable. We can bump the next versions later.

Alright, I'll merge once ofborg is done. Though it would really be best if someone could test that rocm and cuda still work correctly and detect the gpu. I can't test rocm at the moment due to a hardware fault in both the cpu and motherboard rendering my pc unusable; I'm currently on a weak laptop.

Ollama v0.2.5 is already available, so I think it was actually a good idea not merging 0.2.4 hahaha :laughing:  https://github.com/ollama/ollama/releases/tag/v0.2.5

Would you like to merge 0.2.5 within this PR ?

Ollama's releases seem too unstable and bug filled for now; I think we should hold off on updating ollama for 2 to 5 more days. Unless it turns out 0.2.1 also has all the bugs, and updating is the only way to fix them besides reverting back to 0.1.48.


openui5_preload adds sub components to "root" Component-preload.js

I have an app with a sub component:
app
|- Component.js
|- model
|...
|- shared
   |- Component.js
   |- model
   |...

When I build  the project with "components: true" the app/Component-preload.js contains all the resources including the ones from shared.
I would have expected that each component only provides its recources and not the ones of sub components. Or is this intentionally?

Yes, this is intentionally.
You can configure what should be included in the preload using the [src](https://github.com/SAP/grunt-openui5#src-1) option:
components: {
  "app": {
    "src": [
      "app/**", // this is the default
      "!app/shared/**" // excludes the embedded component
    ]
  }
}

I already tried that. The exclusion also applies when the sub component preload is built - so the sub component is not built.

There are two different places where such filters can be set:

On resources level (see here). This is to collect all relevant files (e.g. filtered by type) regardless of libraries/components.
On components / libraries level (see here). Here you can define filters for a specific preload bundle

To build the two components individually the following config should be correct:
components: {
  "app": {
    "src": [
      "app/**", // this is the default
      "!app/shared/**" // excludes the embedded component
    ]
  },
  "app/shared": {} // use default "src" filter ("app/shared/**")
}

Thanks for your workaround. This works.
Still, for me it is an issue that with components:true sub modules are included in the "root" Component-preload.js.
In my opinion when components:true is set the resources covered by sub components shouldn't appear in the in the "root" Component-preload.js.

Alright. As it's still possible to provide the more complex configuration (src option) I would be fine with changing the default behavior as suggested. PR welcome.

This project has been deprecated (see https://github.com/SAP/grunt-openui5/pull/283)


Add most extreme SSP (585) to workflow

Want to add this to the workflow.

[x] Create climate data for this scenario (2041 & 2071)
[x] Re-run predictions for all bugs and plants based on these two additional models

Data added in e5b0589.

After running this, it still highlighted the utility of making "rerun" a bit more fine-grained (see #80 ).


User data stream became unreliable recently?

Hi!
I am using python-binance for the user data stream (websocket). Recently (like 1 week ago), the socket stops sending me updates after 1 - 4 hours (chosen pretty much at random every time I open the connection) without any signs of the connection being closed. It was working perfectly fine before.
I am wondering if anyone else is experiencing the same. Is there a recent change on Binance's side I am not aware of?
Thanks for sharing your thoughts!

Hi, have you solved the problem? I got the same problem and it doesn't work by using the master branch instead of the pip packages.

Works for me now, I run a separate thread (which i wrote myself from scratch) which sends keepalive every 5 minutes.

ok,thank you so much,I will try it

Just submitted a pull request to do it internally with no extra code, just change the user timeout when initializing the socket manager. Default is 30, you may change it to 5:
bm = BinanceSocketManager(client, user_timeout=5*60)

Thanks for pointing out @karlosss


Js functions pizza analyzer

js-functions/pizza-analyzer
📄 Rendered Exercise
🔎 Review
Review via 🔗 CodeSandbox or locally by running this command in your Terminal:
npx ghcd@latest neuefische/web-exercises/tree/js-functions-pizza-analyzer/sessions/js-functions/pizza-analyzer -i

✨ Copyable Markdown for Curriculum
Solve this challenge on
[🔗 **CodeSandbox**](https://codesandbox.io/s/github/neuefische/web-exercises/tree/main/sessions/js-functions/pizza-analyzer?file=/README.md)
or locally by running this command in your Terminal:

```
npx ghcd@latest neuefische/web-exercises/tree/main/sessions/js-functions/pizza-analyzer -i
```

This is a nice and exciting exercise with a lot of real-world importance! ❤
Please see my solution here, but I'm not able to set the correct new width after clicking the buttons, so my pizzas don't change; what am I missing here?
I also tried calculatedDisplaySize.toString(), but it didn't work either.

I think the changes after review work fine! 🚀
Just one note about my issue quoted above:
To be honest, setting CSS styles via JavaScript is not my expertise, so I would say this problem and the not working solution is totally on me.
But anyway, do the students, while solving this challenge, know about how to set the width dynamically because it was part of (let's say) a session two days ago?
If so, everything's fine here; if not, we might add a hint how to do this (because setting a style attribute is not the focus of this challenge, so they should not use too much time to find out how to do this, I would say).

@PReichetanz: Thats a good point there. For changing the width there is a hint, but I could add that they need to add a "px" at the end. For the colors, I could also create two classes which do the trick. What do you think?

@PReichetanz: Thats a good point there. For changing the width there is a hint, but I could add that they need to add a "px" at the end. For the colors, I could also create two classes which do the trick. What do you think?

The hint itself is ok, but I didn't know that I have to add "px" at the end, so students might struggle here as well?
💡 Maybe it's just me because I just had to write markdown files with <img> elements where you don't add the "px" part when setting the width.
I think that the explanation for the colors is fine and we don't have to change it (because students don't need to add anything else when setting outputSection.style.backgroundColor = "var(--red)";.

I am pretty sure the students will struggle with it. So we should make it easier for them. I will think about it.


After the main thread references the electron-store, the dependency is lost

Normal in dev mode, lost after packaging
  "dependencies": {     "electron-store": "^8.1.0",   },

It is these dependencies that are not found in the source code in the electronic store

Vite doesn't seem to pack the dependencies

Maybe this issues will be usefuf to you.
https://github.com/electron-vite/electron-vite-vue/issues/253

也许这个问题对你有用。 电子-vite/电子-vite-vue#253

非常感谢，使用 pnpm i --shamefully-hoist 就可以了，是因为vite的 tree shaking导致的吗？


[Bug]2024-07-05T08:34:36.496758Z DEBUG snarkos_node::prover: Skipping an iteration of the puzzle (no connected peers)

2024-07-05T08:34:36.496758Z DEBUG snarkos_node::prover: Skipping an iteration of the puzzle (no connected peers)
2024-07-05T08:34:36.496768Z DEBUG snarkos_node::prover: Skipping an iteration of the puzzle (no connected peers)
2024-07-05T08:34:36.496773Z DEBUG snarkos_node::prover: Skipping an iteration of the puzzle (no connected peers)
2024-07-05T08:34:36.496776Z DEBUG snarkos_node_router::heartbeat: No connected peers
2024-07-05T08:34:36.496781Z DEBUG snarkos_node::prover: Skipping an iteration of the puzzle (no connected peers)
2024-07-05T08:34:36.496784Z DEBUG snarkos_node::prover: Skipping an iteration of the puzzle (no connected peers)
2024-07-05T08:34:36.496789Z DEBUG snarkos_node::prover: Skipping an iteration of the puzzle (no connected peers)

Seems like it was resolved in https://github.com/AleoNet/snarkOS/issues/3353.


Issue with marker

Hi, I would like to raise a problem while using the TileView lib.
When I insert a marker in the view (with zoom level equal to 0), this is positioned correctly, but when I zoom in on the view the marker previously inserted is automatically translated by a certain amount in Y.
(Zoom level 3)

(Zoom level 0)

How i can solve this problem ?

Sounds like a bug.  What version are you using?

'com.qozix:tileview:2.2.9'

That should be stable.  Post your project?


First step for adding MAVLink support on csd

Related to #12

overall LGTM. Some small comments above.

merged. Reported problems fixed, with the exception of stop using mavlink_parse_char. A TODO was added to improve that later.


fix(trailing-slash): respect fragment if existing

Resolves #174

Two questions:
Would you prefer another argument (e.g. fragment = false) or add fragment support to the queryParams arg?
I have the feeling that another arg would be cleaner but also would add more complexity (4 combinations to consider instead of 2)

Do you think the perf implication applies to withoutTrailingSlash and withTrailingSlash too, or only hasTrailingSlash?

(updated my PR to add the functionality to the queryParams arg but happy to add another param if you think it is necessary.

Another thing: Trailing slash should only applied if it is http/https protocol or relative link. (so no tel/mailto protocol)

@pi0 I've updated the PR to include protocol handling and also wrapped fragment handling so perf overhead is reduced
Do you have further recommendations?
Should I bench the changes?

Last changes fix the issue that a sole fragment should never be changed
/#abc and #abc should stay as is because the semantics are different (current page vs. root page!)


samba: 4.6.3 -> 4.6.4

Motivation for this change
CVE-2017-7494
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[X] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Fixed in 17.03 with 8c7460f890c5c120253044c82676051d3fbb93a1.


Preparing Staging Env Incase Needed

Snyk issues all listed as "probably caused by your .NET framework version", no mitigations available.

Closing as outdated.  Will raise a new PR.


Fix components unmounting on window unload.

Update react-rails to fix components unmounting on window unload.  See https://github.com/reactjs/react-rails/pull/318

:+1:

:+1:


Super improved 500ms development version with parcel

Super improved 500ms development version with parcel

Also, @syrusakbary confirmed everything is working from a fresh clone and install 😄 👍 👏

And one last question, Are you sure Parcel is building all the way up the stack? If I'm not mistaken, I don't think parcel would also be manually bundling all of the deep child dependencies. If you modify packages/wasi/wasi.ts, like add a general console.log('Hello') does that end up getting shown on the next build? 😄 That's super rad if so! 🔥

Yup, it's like that.. with a very small latency of few ms!


Support for seconds and milliseconds in gantt

I'm a collaborator with @cristian-spiescu. The gantt doesn't go beyond minutes and you cannot zoom out after going past this point.

This issue has been fixed in my branch. I also added a new snap variable called "snap"(see problems below)
Problems
Since the snap was titled "snapMinutes" , I added a new var called "snap". "snapMinutes" still exist to not damage current projects, but the new variable takes seconds. The problem I had is, that I could not get it to below seconds, because of rounded pixel calculations. Meaning, even if i gave it 500 ms to snap, it would go to 0,7s, because it was rounded.

@lilfolr @marklawlor I would also like to have this feature, as we have events tracked at "seconds" granularity too. Can we look at merging the PR sent by @Iorda for adding support for it? Thanks in advace.

Added comments to the PR.
Just need to update the unit tests & some variable names and I think its good to go

Thanks Leighton for the merge. Can you also close this issue?


Ensure build.ninja is always in a clean state

The Ninja tool can regenerate build.ninja via SCons whenever
necessary, but up until now SCons left the file handling to Ninja.
Unfortunately, this could result in Ninja having removed build.ninja
before SCons had regenerated it. This change makes SCons own
build.ninja completely so Ninja won't remove it.

[X] I have created a new test or updated the unit tests to cover the new/changed functionality.
[X] I have updated CHANGES.txt (and read the README.rst)
[X] I have updated the appropriate documentation

A unit test for the behavior I am fixing is very hard to do, considering we would have to terminate a Ninja build while it's regenerating build.ninja via SCons. If you can think of a way to do this, I would really like to know.

Please on fixing it, I'm working on it.
There's some issues with the current implementation (not your fixes) which should be changed which would resolve the failures in a simpler way.

Is the NOOP logic related to this issue? or extraneous?

The NOOP rule is what creates the dependency edge between build.ninja.in, which has a depfiles defined (and which we don't care might be removed), and build.ninja, which does not. Unfortunately, Ninja doesn't have a way of creating the kind of dependency edge we want here, so we have to fake it.

@bdbaddog From your suggestions in Discord, I reworked this PR so it doesn't need the synthetic NOOP dependency. Now it works very similar to the way CMake generates its build.ninja files and doesn't use a depfile at all in the process. There's also a small UX change so that specifically requesting a REGENERATE with ninja build.ninja doesn't cause an auto-build - and by extension, only the initial generation would auto-build unless specifically requested otherwise. I think what's here represents good state now. PTAL.

@deriamis  - Unless there's something wrong with my tests. The new logic to generate build.ninja isn't preserving build.ninja when you control-c ninja while it's running SCons to rebuild the build.ninja.


Cannot load bcrypt on Windows 8.1 64-bit

I make a sample (electron-template) to use bcrypt. I install bcrypt without error by npm install command. It shows exception when I run electron-template.exe (Using npm run build to make it) every time:

P.s: This sample works fine on Mac 10.11
My environment:

node 4.2.2
npm 2.14.7
bcrypt 0.8.5
Windows 8.1 64-bit

I don't think electron will work well with this bcrypt module. Maybe try stack overflow for help from others that might have used this particular bcrypt module with electron.


$parent not working in controller

I have the following code in form $parent.checkOut is working fine. But once i try to access this Data in conoller {this.$parent.checkOut} but it showing undefiend. Can you please help me how i can access these model Data into my controller.

                                         <input type="datetime" class="form-control" date-time                                                ng-model="$parent.checkOut" view="date"                                                min-view="date" format="YYYY-MM-DD" auto-close="true">

can i know $parent is currently supported or not in angular-meteor.

@navin10sharma It's related to angular not angular-meteor but I don't think it's works this way.
Here's an explanation.


Pasted markdown text does not render properly

Issue Summary
When text containing markdown is pasted into the editor, the markdown is not parsed. I don't mean in a markdown box, but in the regular editor. If you write the markdown by hand it renders correctly.
Steps to Reproduce

Open the ghost editor
Write HH some text
Observe that as you press the space after HH the editor parses the markdown and changes the format accordingly
Paste HH some text
Observe that this time the editor does not parse and format the the markdown.

I would prefer to write posts in an external markdown editor and then paste them raw in the ghost editor.
Ghost Version
hosted in ghost.org
Node.js Version
hosted in ghost.org
How did you install Ghost?
hosted in ghost.org
Database type
Other
Browser & OS version
mac os monterey, chrome
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

This is currently the expected behaviour - the Ghost editor understands markdown as you type it but is not a markdown based editor.
If you want to copy from a markdown based editor you would need to paste into a markdown card to get your desired effect.


Code blocks are invisible when inline editing

Description
See https://github.com/tinacms/tinacms-site/issues/38
Steps to reproduce

launch documentation website locally
browse to a page containing code blocks with an inline edit button
Click on edit

Expected result
Code blocks content should be displayed
Actual result
Code blocks content are invisible.
Environment
latest macOS/Firefox

I can't reproduce this. Closing for now & also tinacms-site #38


"Go to Symbol in Workspace" (⌘T) becomes very slow in seemingly inconsistent ways due to indexing

Environment data

Language Server version: 2021.2.4
OS and version: both macOS Catalina (when running without dev container) and Ubuntu Focal (with dev container)
Python version: 3.7.10
Multi-root workspace containing several sizeable Python codebases

Expected behaviour
With python.analysis.indexing set to true, I would expect "Go to Symbol in Workspace" (⌘T) to always be fast and not become unusably slow intermittently.
Additionally, I'd also expect to see some sort of visual indication for when indexing is taking place so that I can know that lookup performance might temporarily be affected.
Actual behaviour
Initially (e.g. after a window reload) ⌘T is very slow while the indexing is being performed (which is something I was only able to figure out after setting python.analysis.logLevel to Trace and looking at the Python Language Server output) and after that it gets slow again intermittently whenever Pylance decides to do some more indexing. It's not entirely clear to me what exactly triggers the additional indexing, but initiating a "Go to Symbol in Workspace" (⌘T) also seems to trigger it sometimes (which is kind of unfortunate, because that then makes the ⌘T very slow).
Logs

The initial indexing (e.g. after window reload) eventually stops with this message in the logs:
[Warn  - 5:02:35 PM] Workspace indexing has hit its upper limit: 2000 files

And then whenever ⌘T gets slow again it's pretty obvious from the logs that more indexing is being done.
Additional information
At first I thought the slowness might've been related to the fact that I was working in a dev container, but it turns out I'm getting the same issues when developing directly on macOS. The dev container does add a noticeable delay to ⌘T that makes it a bit less instant, but that's a very short and consistent delay that doesn't impede fluent navigation. The intermittent (extreme) slowness due to indexing happens for both setups.
I've already tried setting python.analysis.indexing to false by the way, but that basically makes ⌘T too slow pretty much all of the time. In fact, I was very excited when I found out about the possibility of enabling indexing because that actually gets us quite a bit closer to having usable symbol-based code navigation in VS Code. It's definitely looking very promising, as does  Pylance in general!

Thanks for the report. We will investigate this performance issue. Could you please provide the logs? Or at least the top of the logs. I'm concerned the devcontainer is opening at the top of the root of the container.

No, that number of source files seems to be correct actually. It's just a large codebase I'm afraid.
For what it's worth: I wouldn't mind if the initial indexing takes a while, as long as it only needs to happen once. Afterwards I'd expect it to only be necessary for new or changed files? It'd be good to have something in the status bar or so to make it clear that indexing is going on then though. As far as I know that's basically what PyCharm does, and they do manage to cope with our large codebase in that way.

Most settings changes can significantly change the analysis; any change or file change event in the search paths can change something and our current method is to just about everything away without much granularity. This works fine for most features (as they are lazy and evaluate on the fly), but indexing is of course ahead-of-time.
The 2000 limit is hardcoded, but we could potentially experiment with some configuration there. A spinner for an in-progress indexing job may also be a good idea.

Making the file limit configurable and showing a spinner when indexing is in progress could already be a big step forward I think! I'd definitely be happy to experiment with that and report back on how much it improves things for us.
I also found out today that excluding subpackages in pyrightconfig.json might be a way of mitigating the problem a bit, by the way. It's not the greatest solution as you also can't search in those subpackages anymore via ⌘T then, but at least it gives you a (crude) way to avoid hitting the hardcoded limit and the slowness did seem to become less of an issue when I did that.

@klbostee can you check whether this still repo with latest release? one of reason this might have happened before was due to multi root workspace causing re-indexing. we have made a few changes around there so wondering whether it is still reproes.
thank you

@heejaechang haven't tested extensively yet, but it does seem to be better now at first sight!


Strenghten TLS cipher selection on secure connection

Context creation (Networking.cpp) already restricted connections to be TLSv1, TLSv1.1 or TLSv1.2 only. However, not specifying a DH params file forbids openssl to choose among DH ciphers.
As stated in man SSL_CTX_set_cipher_list:

[...] On the server side, additional restrictions apply. All ciphers have additional requirements. ADH ciphers don't need a certificate, but DH-parameters must have been set. All other ciphers need a corresponding certificate and key.

This PR allows a user to specify a DH params file which will allow to restrict ciphers to "strong ones only". If no DH params file is provided, cipher list stays as is (TLSv* only).
I spotted that Node.cpp also creates SSL contexts but I would prefer to have your thoughts on this before going further.

Thanks for the PR. OpenSSL is a huge library of a lot of things and I think this PR is too specific to be useful by the masses. If you want specific features you can just subclass the SSLContext and make your changes to the native OpenSSL context. It would be a slippery slope to go down to try and wrap everything of OpenSSL instead of just exposing OpenSSL itself. The same applied to another PR where someone needed some other specific feature I cannot remember -> better to go native OpenSSL in those cases.

Thank you for your reply.

OpenSSL is a huge library of a lot of things

Hell Yeah! 😅
By SSLContext you mean uS::TLS::Context class?


Added How to change the background color of cards in the Board View

Category

[ ] Content fix
[x] New article

Contents of the Pull Request
I wrote an article on how to change the background color of cards in board view.

I have found that after applying the formatting once, clicking [Customize Card] appears to clear the formatting. In reality, the formatting is not cleared until the [Save] button is clicked, but this may be confusing.

So I would like to change the process from [Format current view] instead of [Customize Card]. After formatting once, when I click [Format current view], I get a screen of code and the formatting is not cleared.
I will draft this pull request once. I will comment as soon as the fix is complete.
I am sorry for any inconvenience this may cause you🙇♂

I fixed it.

Thank you for contributing, you are awesome!

Thank you @LuiseFreese 🙏✨!!


TTML percentage of extent and origin scaled down unnecessarily

Have you read the FAQ and checked for duplicate open issues?
yes
What version of Shaka Player are you using?
v3.0.6
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
Both
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
Win & Mac
Chrome v87, Firefox v83, Edge v87
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A
What are the manifest and license server URIs?
https://dash.akamaized.net/dash264/CTA/imsc1/IT1-20171027_dash.mpd
What did you do?

Play this stream
Enable the English subtitle stream
Observe the subtitles throughout

What did you expect to happen?
The specific percentage origin and extent attributes are expected to be honoured for correct placement.
What actually happened?
Subtitles appear at top left and off screen.

If a percentage is specified then the region width and height apply the scaling unnecessarily.  Same issue exists for the origin just after.
https://github.com/google/shaka-player/blob/56bb045ff68325d6b9de7cb3d56265ce3162f164/lib/text/ttml_text_parser.js#L365

Hello @dhmmNagra , thank you for your report! Would you like to submit a PR for this?

Hi @michellezhuogg thanks for the opportunity, I am drafting a PR - I'm a first time contributor so there will be a delay whilst my employer sorts out the CLA signing.

pull request drafted at https://github.com/google/shaka-player/pull/3044

I think that it was resolved with https://github.com/google/shaka-player/pull/3044 and https://github.com/google/shaka-player/commit/d3640d1da15b6470deebd663f3f771c03864ab66 so it could be closed, what do you think?

Yes its all been wrapped up already


KYLIN-1427 Support partition_time_column and partition_time_format

https://issues.apache.org/jira/browse/KYLIN-1427
Support additional partition_time_column and partition_time_format beside partition_date_column and partition_date_format
Unit Test: org.apache.kylin.metadata.model.DefaultPartitionConditionBuilderTest

Merged into 2.x-staging at https://github.com/apache/kylin/commit/6678ae3725d030d4acb08e2aa5f20261aa7bf16b