ArrayCollection/PersistentCollection Architectural Disagreement

While Doctrine will provide either an ArrayCollection, or a PersistentCollection, depending on the state of the Entity, no common Class or Interface reflects their shared methods.
More specifically, when defining an entity's collection getter, the return type cannot be specified.
Options include:

PersistentCollection: in most instances, will not be correct.
ArrayCollection: has the same methods as PersistentCollection, but in some instances will be a PersistentCollection.
Collection/AbstractLazyCollection: in addition to the problems above, it is missing the "matching" method.

This missing common agreement complicates all code implementing these objects, which otherwise could be clean and simple.
For the sake of clarity, consider the PHP7 return type declaration on the getter for such a method.
Both of the following week result in errors:
pubic method getOneToMany(): ArrayCollection
pubic method getOneToMany(): PersistentCollection
The best alternative is:
pubic method getOneToMany(): Collection
... leaving out the method: matching.

As a simple suggestion, perhaps Collection should implement Selectable, and the whole issue disappears.

no common Class or Interface reflects their shared methods

This is true unfortunately.

As a simple suggestion, perhaps Collection should implement Selectable, and the whole issue disappears.

Impossible due BC, would be a BC break -- only possible in next major of doctrine/collections.
Also possibly not a wise idea -- consider this architecture more in general / different scenarios outside ORM. it doesn't always make sense to have selectable collection.
This could be only solved in 3.x somehow, if you have any proposals, ideally without breaking doctrine/collections, you're welcome to send PR for that or discuss here. 👍

Notation Key:

() interface
[] abscract
definition or class

this extends defintion

Currently

(Countable), (IteratorAggregate), (ArrayAccess)

(Collection)

(Selectable): public function matching
[AbstractLazyCollection] (Collection)

PersistentCollection (Selectable)

ArrayCollection (Collection, Selectable)

Proposed

(Countable), (IteratorAggregate), (ArrayAccess), (Selectable): public function matching

(Collection)

[AbstractLazyCollection] (Collection)

PersistentCollection

ArrayCollection (Collection)

Result
As this would neither remove nor alter any methods, classes, or interface implementations, I fail to see how this would cause a BC break. The only real limitation here, is that it would require two paired PRs against the Collections repository and the ORM Repository.

I fail to see how this would cause a BC break.

In your proposal, Collection suddenly extends Selectable, thus requires implementation for matching(). This method previously did not exist in the Collection interface so any code implementing (just) Collection is now suddenly required to also implement new matching() method, which is a BC break.

The Selectable interface was added a long time after Collection was finalized. A change to this interface would have been a BC break. The solution would be that "collections" introduces a third interface SelectableCollection that extends from Collection and Selectable and to change both ArrayCollection and PersistentCollection to use that.
This must be done in two steps:
1.) Add SelectableCollection to "doctrine/collections", have ArrayCollection extend it and release a new version (minor bump enough, because its not a BC break).
2.) Bump dependency of "doctrine/orm" to new "doctrine/collections" version and change PersistentCollection.

@WyrdNexus did you figure out a workaround for this issue? (without dropping the returntype typehint)

The only real limitation here, is that it would require two paired PRs against the Collections repository and the ORM Repository.

Please don't forget MongoDB ODM, where PersistentCollection doesn't implement Selectable.

@alcaeus if a new interface is created, there is no issue for the ODM, as Collection would still not require implementing Selectable.

@stof I was replying to @WyrdNexus who suggested that the Collection interface extends Selectable, with PersistentCollection implementing Collection: this would indeed also affect ODM. A new, separate interface (SelectableCollection extends Selectable, Collection) would not directly impact ODM.

Any news/plans on this? This is really frustrating when you are working with PHPStan.


subscribe in slotRenderer - make the component re-render whenever the contributions to the slot change

Looks great! Maybe unsubscribe the SlotRenderer on unmount?

Yes! will add :)


bazel: Update xds

This includes some more go_package annotations in the protos
Signed-off-by: Keith Smiley<EMAIL_ADDRESS>

Up to @htuch to review that one. I'm trying to unblock https://github.com/envoyproxy/envoy/pull/17460

I just merged https://github.com/cncf/xds/pull/14 @phlax

@htuch i dont want to block this PR, so im happy to update after


sndev cleanup

closes #1533
You'll have to delete and restart containers after this is merged. It renames stacker_(lnd|cln) to (lnd|cln) and relies on docker compose labels to provide access to service clis/onions/certs now.
This replaces:

sndev fund_user -> sndev set_balance
sndev (stacker|sn)_(lndcli|clncli|litcli) -> sndev cli (lnd|sn_lnd|cln|litd)
sndev (stacker_(lnd|cln)|lnbits) get_onion -> sndev onion (lnd|cln|lnbits)
sndev stacker_lnd get_cert -> sndev cert lnd

This is unrelated to the pr, but i've noticed  that some sndev commands do not work in my ubuntu workstation if i run it with ./sndev. The cause seems to be the use of bash conditional expressions [[ ]] that are not supported by the real /bin/sh (even though /bin/sh might be a symlink to /bin/bash in some distro).
Imo for correctness we should use the shebang  #!/bin/bash

That's weird because is passed the shellcheck check job we run on prs.

Ah it's not catching that problem because it's just a warning. I'll fix that in this pr. I think I added a few more of those bash syntax conditionals.

Should be fixed. There was just one usages of [[ COND ]] in fund_user (now set_balance).


Error when adding product list with the Best Seller option

Doctrine \ DBAL \ Exception \ NonUniqueFieldNameException HELP An exception occurred while executing 'SELECT p.pID FROM CommunityStoreProducts p LEFT JOIN CommunityStoreProductSearchIndexAttributes csi ON p.pID = csi.pID WHERE (pQty > 0 OR pQtyUnlim = 1) AND (pActive = 1) GROUP BY p.pID ORDER BY pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC, pID = ? DESC' with params [99, 98, 66, 22, 60, 15, 123, 4, 72, 127, 52, 100, 247, 329, 3, 243, 59, 136]: SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'pID' in order clause is ambiguous
I am just trying to add a product list with sort option "Best Sellers". It used to work, so it must have messed up when I did an upgrade.
If need be I can put on full debug mode and give you a link to the error. Also, why is the option:
Related Products Order
Always unselectable?

Try this change here:
https://github.com/concrete5-community-store/community_store/commit/2677facc3f01228e45e357250a7f55cd7bb219d1#diff-82f99f6e29e208e07ba4979e54841732R185
(other javascript stuff in the same commit isn't relevant)

Doctrine \ DBAL \ Exception \ InvalidFieldNameException HELP An exception occurred while executing 'SELECT p.pID FROM CommunityStoreProducts p LEFT JOIN CommunityStoreProductSearchIndexAttributes csi ON p.pID = csi.pID WHERE (pQty > 0 OR pQtyUnlim = 1) AND (pActive = 1) GROUP BY p.pID ORDER BY p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC' with params [99, 98, 66, 22, 60, 15, 123, 72, 4, 127, 52, 136, 247, 100, 329, 243, 59, 3, 251]: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'p.ID' in 'order clause'
Tonight I'll leave the debug mode on  so you can see the full error report.
https://www.mdius.com/test5

Check the change you made, it's p.pID, not p.ID.
On 3 Oct. 2017 1:56 am, "Joseph Snedaker"<EMAIL_ADDRESS>wrote:
Doctrine \ DBAL \ Exception \ InvalidFieldNameException HELP An exception
occurred while executing 'SELECT p.pID FROM CommunityStoreProducts p LEFT
JOIN CommunityStoreProductSearchIndexAttributes csi ON p.pID = csi.pID
WHERE (pQty > 0 OR pQtyUnlim = 1) AND (pActive = 1) GROUP BY p.pID ORDER BY
p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC,
p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC,
p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC,
p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC, p.ID = ? DESC' with params
[99, 98, 66, 22, 60, 15, 123, 72, 4, 127, 52, 136, 247, 100, 329, 243, 59,
3, 251]: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'p.ID' in
'order clause'
Tonight I'll leave the debug mode on so you can see the full error report.
https://www.mdius.com/test5
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/concrete5-community-store/community_store/issues/285#issuecomment-333569238,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABB5MBMQsJ0nw0ZGg3SwAtE8LkyWoSa2ks5soQCWgaJpZM4Ponnk
.

Check the change you made, it's p.pID, not p.ID
Yup!
That fixed it! Thank you again.


Potential bug in sampling single pixel from with grid_pull?

Hello,
I wanted to use this library for doing tricubic sampling from a voxel grid. I have tested it with trilinear interpolation and it works fine, however, I get weird errors whenever I try increase the interpolation degree past 1.
In this case, img consists of batches of 4 by 4 by 4 cube of points stored in two channels.
"weights" consists of single points we would like to sample from that batch.
Using this library for trilinear sampling works perfectly:
img = img.reshape(-1,2,4,4,4)
c = interpol.grid_pull(img, weights.view(-1, 1, 1, 1, 3), interpolation=1, prefilter=True)

but when I try to use to increase the interpolation to a higher order, it no longer works. I get this error in nd.py:
        c = interpol.grid_pull(img, weights.view(-1, 1, 1, 1, 3), interpolation=3, prefilter=True)
        ...
        idx = [c[n] for c, n in zip(coords, nodes)]
        idx = sub2ind_list(idx, shape)
        idx = idx.expand([batch, channel, idx.shape[-1]])
              ~~~~~~~~~~ <--- HERE
        out1 = inp.gather(-1, idx)
RuntimeError: The expanded size of the tensor (2) must match the existing size (65536) at non-singleton dimension 1.  Target sizes: [65536, 2, 1].  Tensor sizes: [65536, 1]

Am I misunderstanding something about how to call this library for voxel representations or how the interpolation parameter is used? Is actually doing grid_sample possible with higher order interpolations in this library? Is this a bug caused by accidental squeezing/broadcasting?

Hi
Thanks for the report! There was indeed a bug in the way I wrote the broadcasting.
I've just pushed a fix. It does not throw errors anymore, although you may want to make sure that the sampled values make sense (they used to make sense when the batch dimension was 1, which did not trigger the broadcast bug).
Cheers
Yael

That seemed to fix the error at least. 💯 Thanks for the fast response.


Write monotonic sequence, but read is non monotonic

Environment
Delta-rs version: 0.18.2
Binding: python
Environment:

Cloud provider: None
OS: macOS Ventura Version 13.6.2 (22G320)
Other: pyarrow==16.1.0 pyarrow-hotfix==0.6 pandas==2.2.2

Bug
What happened:
I wrote a monotonically incrementing sequence into a deltalake table using the pyarrow engine.
When reading this deltalake table, the data is no longer monotonically incrementing.
What you expected to happen: I expect the data to be monotonically incrementing.
The rust engine appears to work as expected, however the pyarrow engine appears to re-order the data.
How to reproduce it:
Minimal example which reproduces the bug consistently on my laptop.
# out_of_order.py

import argparse
import os

import pandas as pd
import pyarrow as pa
import pyarrow.parquet as pq

from deltalake import write_deltalake, DeltaTable

def write_data_file(file, schema, length, batch_size):
    with pq.ParquetWriter(file,
                          schema=schema,
                          compression='gzip',
                          compression_level=6) as writer:

        for i in range(0, length, batch_size):
            rows = min(i + batch_size, length) - i
            df = pd.DataFrame(range(i, i + rows, 1), columns=['increment'])
            batch = pa.record_batch(schema=schema, data=df)
            writer.write_batch(batch)

    df = pd.read_parquet(file)
    assert df['increment'].is_monotonic_increasing, 'data file not monotonic'

def write_delta(engine, uri, schema, file, batch_size):
    with pq.ParquetFile(file) as data:
        write_deltalake(table_or_uri=uri,
                        data=data.iter_batches(batch_size=batch_size),
                        schema=schema,
                        mode='overwrite',
                        engine=engine)

def assert_monotonic(engine, uri):
    dt = DeltaTable(uri)
    assert dt.to_pandas()['increment'].is_monotonic_increasing, f'{engine} not monotonic'

if __name__ == '__main__':
    parser = argparse.ArgumentParser(usage=__doc__, formatter_class=argparse.ArgumentDefaultsHelpFormatter)
    parser.add_argument('--path', required=True, help='Deltalake table path')
    parser.add_argument('--length', default=62914561, type=int, help='Dataset length')
    parser.add_argument('--batch-size', default=100_000, type=int, help='Batch size')
    args = parser.parse_args()

    schema = pa.schema([
        pa.field('increment', pa.int64(), nullable=False),
    ])

    os.makedirs(args.path, exist_ok=True)
    file = args.path + '/monotonic'
    write_data_file(file, schema, args.length, args.batch_size)

    uri = args.path + '/rust'
    write_delta('rust', uri, schema, file, args.batch_size)
    assert_monotonic('rust', uri)

    uri = args.path + '/pyarrow'
    write_delta('pyarrow', uri, schema, file, args.batch_size)
    assert_monotonic('pyarrow', uri)

Run using the command:
python out_of_order.py --path $PWD/out_of_order

Following exception is raised:
Traceback (most recent call last):
  File ".../out_of_order.py", line 64, in <module>
    assert_monotonic('pyarrow', uri)
  File ".../out_of_order.py", line 40, in assert_monotonic
    assert dt.to_pandas()['increment'].is_monotonic_increasing, f'{engine} not monotonic'
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: pyarrow not monotonic

More details:
The file which appears to be out of order on my machine is part-5 ($PWD/out_of_order/pyarrow/0-6e748f6b-f69e-47dd-8857-dc652c73cfef-5.parquet).
This is reproducible when running multiple times, implying it's somewhat deterministic, however it appears that a different "row group" is out of order each time, given that when inspecting the file a different increment number is non monotonic.

Looks like this is mainly due to a known issue in pyarrow: https://github.com/apache/arrow/issues/39030
However, I did find that a _delta_log transaction which has multiple add actions can have it's part files in an unsorted order, which also contributes to this problem.

This is something we cannot guarantee in the first place and the PyArrow engine will be deprecated as  of v0.19, so I am closing this issue


Missing one category for Advisor

The following sentence "To get recommendations for a specific category, click one of the tabs: Reliability, Security, Performance, or Cost."  Is missing Operational Excellence as one of the tabs/categories.

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

ID: 38f3c39c-4a81-87cd-2741-5f8631b46a4d
Version Independent ID: 46fe35e1-dd95-ef4a-0983-a14d6b94db39
Content: Get started with Azure Advisor - Azure Advisor
Content Source: articles/advisor/advisor-get-started.md
Service: advisor
GitHub Login: @ikhapova
Microsoft Alias: ikhapova

@jamandt We have verified and updated the document as required. The update will go live in less than 24 hours. Thanks again!


Add arg descriptions with doc comments

Added some descriptions to the arguments with /// doc comments.
More should be added but I'm not sure what the args all do yet.

@jopetty is this ready to merge/anything else you wanted to add?
Also happy to discuss in person if there are parts that were confusing

We discovered #112 while documenting the flags.
In general, there are opportunities to clean up/remove flags.

Some args should probably be made Optional<> as well


Add GA tracking to view more button

What's this PR do?
Adds GA tracking to the View More button on the reportback gallery.
How should this be reviewed?
Probably need to check in GA that the event is being tracked properly..
Also, if you check the Network log in inspector, you should see a http://www.google-analytics.com/collect request being made when you click the button.
Relevant tickets
Fixes #6698
Checklist

[ ] Tested on staging.

@weerd updated!

¡OLÉ!
:sheep: it!

👍 🔍 !


Update to rust 1.67

Codecov Report

Merging #1516 (3273054) into main (fbd29f7) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1516   +/-   ##
=======================================
  Coverage   68.72%   68.72%
=======================================
  Files         120      120
  Lines       13094    13094
=======================================
  Hits         8999     8999
  Misses       4095     4095

@utam0k The checks do not pass because it expects that integration tests run with 1.65.0. Looks like this is coming from branch protection rules: https://github.com/orgs/community/discussions/26698. I tried to configure the rules (see docs) but I do not see that option. Do you have permissions to change the rules?

@Furisto Sorry for the late! I updated.


Task's queue is 'undefined'

By some reason the output is the following:
celery_tasks_total{name="local.update_record",namespace="celery",queue="local_develop",state="STARTED"} 0.0
# skipped
celery_tasks_total{name="local.update_record",namespace="celery",queue="undefined",state="STARTED"} 1.0

All recorded tasks have queue as "undefined".

We have the same problem:
celery_tasks_total{name="watcher.process_notifications",namespace="celery",queue="celery",state="RECEIVED"} 0.0
# skipped
celery_tasks_total{name="watcher.process_notifications",namespace="celery",queue="undefined",state="RECEIVED"} 1012.0

I checked how our Celery task events look like and it seems, that they are ok and contain all expected data.

We are aware of this bug it's not easy to refresh the queue list in real time.
@itsx the "celery" queue is the default one is used to run internal celery tasks

Having the same issue:
celery_tasks_total{name="core.tasks.square",namespace="celery",queue="celery",state="SUCCESS"} 0.0
celery_tasks_total{name="core.tasks.square",namespace="celery",queue="undefined",state="SUCCESS"} 4.0

having the same issue, does anyone know how to fix this?

I have try added the following settings.
celery.conf.task_queues=(
    Queue('task1_queue'),
    Queue('task2_queue')
)

celery.conf.task_routes={
        'task1': {'queue': 'task1_queue'},#,'routing_key':'task1'},
        'task2': {'queue': 'task2_queue'}#,'routing_key':'task2'}
}

and send the task with following command however it seems like the metrics added the queue with name=undefined.
celery_app.send_task(
                "task1", args=[], kwargs={"payload": jsonable_encoder(data),},time_limit=CELERYD_TASK_TIME_LIMIT,compression="gzip",queue="task1_queue"
        )

Same here; all queues except celery are named undefined in the metrics output. I run the workers with --task-events and despite that i start the exporter with --enable-events.


Connection Lost: Invalid Player Data

I'm on version 1.18.2, and I keep getting this error.
Setup:

Launch game
Create new world
Generate chunks
Error

Logs have so many errors, I've attached them here:
latest.log
Mods I'm using:
mods.zip

This is because you installed so many origins that the code can't handle the size of the packets.
To fix this problem, either remove some origins, or install this mod: https://www.curseforge.com/minecraft/mc-mods/xl-packets-fabric

Working perfectly fine now, thank you!


feature: add rich container mode

1.Describe what this PR did
In runc, we change container init process to /sbin/init, systemd or dumb-init to support rich container mode.
In order to not modify the containerd, we set rich mode by spec process env.
We set env rich_mode=true to start container in rich mode.
We set env key rich_mode_launch_manner to select init manner, such as systemd, dumb-init and sbin-init.
2.Does this pull request fix one issue?
3.Describe how you did it
4.Describe how to verify it
5.Special notes for reviews

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: wangforthinker:x: hqhqYou have signed the CLA already but the status is still pending? Let us recheck it.

I saw that you implement a hook system, could you please tell me why we need this hook system, because if we need do something before init process execute, hook in runtime-spec  is enough.
Correct me if I miss something.

@Ace-Tang In rich container mode,we should replace the start command while hook in runtime-spec cannot do it.

LGTM


[test] Remove overlays with extra NS stripping from the mock SDK

There were only two tests still using them, and they weren't hard to migrate over to the normal mock SDK overlays.

@swift-ci Please test


⚠ Bofrost.de has degraded performance

In ca7f674, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1461 ms

Resolved: Bofrost.de performance has improved in 901f6eb after 6 minutes.


[Feature] update pre/post_model_hooks for tests

resolves https://github.com/dbt-labs/dbt-snowflake/issues/1059
docs dbt-labs/docs.getdbt.com/#

Problem
We want to be able to correctly handle arbitrary configs in tests correctly, for models this is currently done with pre/post_model_hook and for snowflake this is mainly for swapping snowflake_wareshouse values, currently we don't actually recognize this config in during tests. by adding updating dbt-core methods we have opened up the ability to pass these into tests correctly but must make specific modifications in the adapters themselves.
asks:

update for logging
add tests

Solution
we can use AdapterLogger to add loggger.info as we need in the model_hook methods
add a new tests to handle scenarios i.e we update snowflake_warehouse to a invalid warehouse.

Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

running test without the expect_pass shows this in logs
use warehouse DBT_TEST_DOES_NOT_EXIST
11:01:25.737689 [debug] [Thread-22 ]: Snowflake adapter: Snowflake query id: 01b4cf21-0804-bc1d-000d-3783363da962
11:01:25.739334 [debug] [Thread-22 ]: Snowflake adapter: Snowflake error: 002043 (02000): SQL compilation error:
Object does not exist, or operation cannot be performed.

This is just a test I don't think it is a feature.

This is just a test I don't think it is a feature.

currently labeled as such because it ties with the core prs https://github.com/dbt-labs/dbt-core/pull/10258 and https://github.com/dbt-labs/dbt-core/pull/10245 but yeah could possibly just be a under the hood update.


Encountered an error (ServiceUnavailable) while deploying function

Does this occur consistently? Yes.
Repro steps:

Create Azure function using App Service Plan
Publish function from VScode.

Action: azureFunctions.deploy
Error type: BadRequest
Error Message: Encountered an error (ServiceUnavailable) from host runtime.
Version: 0.22.1
OS: linux
OS Release: 5.3.0-53-generic
Product: Visual Studio Code
Product Version: 1.45.1
Language: en

Call Stack
extension.bundle.js:225:2840812
retryCallback extension.bundle.js:225:114550
retryCallback extension.bundle.js:157:31063
extension.bundle.js:157:32676
handleRedirect extension.bundle.js:171:7752
extension.bundle.js:157:29848
Request._callback extension.bundle.js:1:670871
Request.n._callback.n.callback.n.callback extension.bundle.js:201:4483
Request.emit events.js:203:13
Request.<anonymous> extension.bundle.js:201:17592
Request.emit events.js:208:15
IncomingMessage.<anonymous> extension.bundle.js:201:16382
Object.onceWrapper events.js:291:20
IncomingMessage.emit events.js:208:15
endReadableNT _stream_readable.js:1168:12
processTicksAndRejections task_queues.js:77:11

There's a whole bunch of reasons this could be happening and unfortunately we (the VS Code extension) don't have access to the full logs to understand why it's happening. I would start with these troubleshooting tips or contact Azure support (there should be a help button in the portal) since they should have access to more logs.
If you want us to investigate on the VS Code side - we would need much more detailed repro steps including how you created your local project, how you created the function app in Azure, etc.
Other users have had success with trying to deploy to a new app.

I may have hit this same problem deploying a new version of a timer triggered function; I've raised this on the Azure Q&A: https://docs.microsoft.com/en-us/answers/questions/38253/azure-ttime-triggered-function-fails-to-deploy-wit.html I'd be interested to know if the original posters issue also used a timer trigger, and if the same "Syncing triggers" retry output was seen

@nights99 the link you pasted doesn't seem to be working for me. There's a lot of different reasons people could get a "ServiceUnavailable" error, so probably best for you to create a new issue with your own detailed repro steps


feat: add auto close options for toast

What?
Add auto close options for toast
Why?
Refer: https://github.com/shopware/meteor/issues/168
How?
create a new option autoClose,
when it true, it will auto-close.
when it false, it will never auto-close.
when it undefine, follow the default behavior in toast matrix.
Testing?
N/A
Screenshots (optional)
N/A
Anything Else?
N/A

Unfortunately I can't accept this PR. We spend hours upon hours to determine how a toast should behave in the Shopware Administration. With that in mind we build the Components which are fed by the meteor-admin-sdk.
Here you can find an overview of how toasts need to behave.
If your toast doesn't match that matrix it's not a toast! It needs to be a notification.


ERROR !

hey am stuck with this error(mac os) upon double clicking the app
Traceback (most recent call last):
File "screenipy.py", line 13, in
File "PyInstaller/loader/pyimod02_importers.py", line 499, in exec_module
File "classes/Screener.py", line 12, in
File "PyInstaller/loader/pyimod02_importers.py", line 499, in exec_module
File "talib/init.py", line 93, in
ImportError: dlopen(/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/_ta_lib.cpython-39-darwin.so, 2): Library not loaded: @loader_path/../libta_lib.0.dylib
Referenced from: /var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/_ta_lib.cpython-39-darwin.so
Reason: no suitable image found.  Did find:
/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/../libta_lib.0.dylib: cannot load 'libta_lib.0.dylib' (load command 0x80000034 is unknown)
/private/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/../libta_lib.0.dylib: cannot load 'libta_lib.0.dylib' (load command 0x80000034 is unknown)
[70529] Failed to execute script 'screenipy' due to unhandled exception: dlopen(/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/_ta_lib.cpython-39-darwin.so, 2): Library not loaded: @loader_path/../libta_lib.0.dylib
Referenced from: /var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/_ta_lib.cpython-39-darwin.so
Reason: no suitable image found.  Did find:
/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/../libta_lib.0.dylib: cannot load 'libta_lib.0.dylib' (load command 0x80000034 is unknown)
/private/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/../libta_lib.0.dylib: cannot load 'libta_lib.0.dylib' (load command 0x80000034 is unknown)
[70529] Traceback:
Traceback (most recent call last):
File "screenipy.py", line 13, in
File "PyInstaller/loader/pyimod02_importers.py", line 499, in exec_module
File "classes/Screener.py", line 12, in
File "PyInstaller/loader/pyimod02_importers.py", line 499, in exec_module
File "talib/init.py", line 93, in
ImportError: dlopen(/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/_ta_lib.cpython-39-darwin.so, 2): Library not loaded: @loader_path/../libta_lib.0.dylib
Referenced from: /var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/_ta_lib.cpython-39-darwin.so
Reason: no suitable image found.  Did find:
/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/../libta_lib.0.dylib: cannot load 'libta_lib.0.dylib' (load command 0x80000034 is unknown)
/private/var/folders/_j/yt8t21sn6p1ftcw6158ytxnc0000gn/T/_MEI1EVpWc/talib/../libta_lib.0.dylib: cannot load 'libta_lib.0.dylib' (load command 0x80000034 is unknown)
Regards,
Srinath Rao

Hi @srinathra0
Please try downloading the latest version.
If the issue persists or you are using Mac with M1/M2 chip, please consider cloning this repo, setup python and dependencies, and the screenipy.py rather than using the executable .run file.

hey !
sorry for the late reply ..am using pretty old mac (mac book pro 2011) am sort of experimenting with it's work ability as on today! it's (version) indeed i gave it a try with the latest version but still end up with the same error!

Please clone the code repo and setup your own environment. Build for Mac is having issues due to inconsistent hardware architecture as of now.


Fix image resizing on scalar changes

Fixes #2502.
resizeImage() was not called on scalar changes, hence underlying image was not properly resized according to the enclosing div element. This overrides setPixelSize() so as to always call resizeImage() on scalar changes.

Can one of the admins verify this patch?

@AppInventorWorkerBee ok to test


3.7: added elapsed time logging

Scope & Purpose
added logging of elapsed time of ArangoSearch commit/consolidation/cleanup jobs

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[x] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Testing & Verification

[ ] This change is a trivial rework / code cleanup without any test coverage.
[x] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[ ] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[ ] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/14116/

https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/14116/


⚠ Bofrost.fr has degraded performance

In 41f5a0b, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1561 ms

Resolved: Bofrost.fr performance has improved in 23a409b.


Cutecoin does not start due to network error

Here is the trace:
ERROR:__init__:Future/Task exception was never retrieved
future: Task(<refresh>)<exception=NoPeerAvailable('', 0)>
Traceback (most recent call last):
  File "/usr/lib/python3.4/asyncio/tasks.py", line 302, in _step
    result = next(coro)
  File "/home/gael/cutecoin/src/cutecoin/gui/community_tile.py", line 34, in refresh
  File "/home/gael/cutecoin/src/cutecoin/core/community.py", line 294, in members_pubkeys
  File "/home/gael/cutecoin/src/cutecoin/core/net/api/bma/access.py", line 174, in future_request
cutecoin.tools.exceptions.NoPeerAvailable: Error : No peer answered in  community (0 peers available)

This error occured on 0.11.


Using POST instead of GET parameters

Hello!
I am maintaining a Laravel project and there are 2-3 kB long request URI-s because of the amount of GET params.
Would it be possible to implement an option for the request method?
AFAIK jquery datatables supports POST.

Yes, see this demo for some ref.

Thank you for your support.


Clarify that there has been a cleaning process.

Second, I am wondering whether you had conducted manual screening of collected literature to ensure their close relation to the research target. Different from systematic review aiming to provide in-depth analysis of representative literature, bibliometric analysis is used for big data analysis to enable a comprehensive and general view. Thus, sufficient coverage of relevant literature is especially essential for a bibliometric study. In particular, you included articles which had used any of the search terms in the title, the abstract or the text. The use of title and abstract is understandable since they are commonly used to search articles (you can refer to bibliometric studies such as https://www.sciencedirect.com/science/article/pii/S0360131520300555,
https://bera-journals.onlinelibrary.wiley.com/doi/full/10.1111/bjet.12907?casa_token=vCNbiEvDF5gAAAAA%3Ap9AeWKgtwePj-ABgaPIFecQrTObZdjh26sG49aeHs47Ms7zHROw8I_JhUpvQnOlZ0Vx5oCrzwmCN1kTsVQ,
https://idp.springer.com/authorize/casa?redirect_uri=https://link.springer.com/article/10.1007/s11042-020-09062-7&casa_token=z8E4b9PYoncAAAAA:9X2oCiTpIRj2V6xMkkLsjXvAyD2NSmMxVNYK_K-fLdb67SLc3zUFxoCmmniFJCNprc20XInB9KPum71LMCw
), however, the inclusion of full text can be troublesome as it may result in too many irrelevant articles being included. Thus, manual screening is needed and should be conducted to ensure data quality.

The data set has been cleaned. Paper's that did not fit the narrative were removed. However, there might still a few articles that do not?

Reviewer 2

I was not convinced that the dataset covered all relevant article and only the relevant articles. It was not described at all what steps were taken to find the grey literature that was not covered by the database searches. Did you hand-searched the citation of eligible studies or consulted with experts? These are all very important steps to get a comprehensive review. I know that some steps were taken as the author mentions to include some extra paper. Further, what steps were taken to ensure that only relevant articles were included? An abstract can mention Prisoner’s Dilemma but not analyze this topic. Usually, it is good to scope the abstract (at least) to check whether studies were of interest. Further, the number of duplicates should be reported.

The flow chart will help with explaining the above better.

Possibly include a random sample (n=20) to approximate the mistakes in the data set.


Image Upload script Gfycat feature support

I figured I'd move this to a new issue so we don't pollute the other one with feature-specific discussion.
As I don't expect to have a version of this ready for public testing for several days I figured I'd air some questions I have about work on this PR:

Should I deprecate the code for Chattypics in this feature-fork?

Is it okay if I re-style/re-factor the image uploader to be responsive (as opposed to the current table-based non-responsive monstrosity) as long as I maintain feature-parity?

Are there any additional design considerations I should take into account?

My thoughts:

No.  Let's assume MB will pull through with HTTPS support.

Go nuts.  I really think this script has low use, so I'm not that concerned about removing functionality.

If it all ends up being too much work, this whole feature could be replaced with links to the Imgur, Chattypics, and Gfycat upload pages that open in a new tab.  I think that would be like 95% as good as uploading from the post form.

Any idea on timeframe?  I don't mean to rush you at all, I'm just deciding whether I should wait to release 1.50 for this.  I'm thinking I should release 1.50 to get the scrolling fix out to people.  I can gut the image uploader script temporarily.

You don't need to hold back a release on my account, but I am very close (a day or two at most) to being done enough for a PR. This feature will need some testing though in case I missed anything (which is very likely considering the code diff).

I'll probably also be doing a PR for the Embed Posts script once I'm done with Gfycat support as long as the after-work free time holds out given the holidays coming up.

I'm hitting a major snag with Gfycat support. I spent several hours last night and today trying to figure out why encoding remote URLs is very unreliable with Gfycat and I've come to the conclusion that it's on the API/server side.
I may have to remove URL encoding for Gfycat altogether if I can't find a reliable solution. Either way, Gfycat seems to work fine with Youtube videos, local videos, GIFs, and common image formats.


Class property types without semi-colons break highlighting

In this example, x, y and constructor will not be properly colored. As you add in semi-colons, their highlighting is fixed.
Happens in Atom, Visual Studio Code.
Original bug: https://github.com/TypeStrong/atom-typescript/issues/820

Should be fixed with #48


Add contribution regarded graphql and apollo

Really great work. Note that you also need to create a link to this file in the techstacks.md.


[Task]: Investigate the delay in making api calls in Appsmith cloud.

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
No response

There are two reasons for the delay

We are not using HTTP2 on cloud (should verify on self-hosted), so preloads were hogging the available 6 connections.
With this reverted commit, CDN for preloads was broken.


Screen reader does not read any of the items listed on the Environments overview page

Just says "button" for each link/option. This might be fixed by fixing accspot bugs filed earlier, but this open work item can help validate the scenario.

This has been fixed.


fix: add OpenRouter Sonnet-3.5 beta/self-moderated model

Add self-moderated Sonnet-3.5 OpenRouter endpoint to model list to enable e.g. prompt caching.

Yes, sorry for the oversight, it is fixed now.


Princeton orange title.

Closes #203.

Coverage remained the same at 100.0% when pulling e19bb2dd95a84d9ec8ab45c19497cdb6c6d8e58a on masthead_font into 2f55577884f214c2a4e819b46a0c0b1399512b7c on master.


docs/lokomotive: update branch to 0.6.1 for 0.6 release

This is for the new release of lokomotive.
Signed-off-by: knrt10<EMAIL_ADDRESS>

Thanks @knrt10 . Merged.
It'll be picked up by the next website rebuild, in less than 24h, if it's urgent for you let me know and I will trigger a rebuild ASAP.


Activate Google cloud SR from backend

As a user/customer I want that my commands are recognized by Google speech recognition
40 SP
Tasks:

[x] wakeup works with sphinx
[x] Find a way to activate Google SR from backend (overhead as little as possible)
[x] Listen to speech input from user and send result back to device
[x] Send result to parser

Closed by #322


New approach to JavaScript, hosted at a CDN

Changes the approach for the Trelliscope JavaScript library to point to a CDN rather than embedding the JavaScript files directly into the library and copying them to the output directory.
This will make it much easier to update the JavaScript library in the future. It also means the resources directory is no longer necessary and has been removed.
The negative side effect is that a Trelliscope cannot be built or viewed in a disconnected state. Viewing could be addressed in the future by adding a parameter to save a self-contained version which would pull from the CDN and save the files locally.
If it is desired to build a Trelliscope in a disconnected state, we could create a script to download the files locally and copy from there. (This seems less important than being able to view a self-contained Trelliscope.)
A few other notes:

The default JavaScript library version is stored in a constant in the html_utils.py file. I considered putting this in a config file that would be packaged up, but it seemed like a little bit of overkill for the one value, since there are no other config style values we are saving.
This PR also saves out the id file in the base output directory. This did not seem to be necessary, but it is consistent with the output from R and other uses. As a part of this, I updated this to use an actual uuid, rather than a substring of the first 8 hex digits.

@hafen This completes the update to match the code you sent me.
Let me know if you have any concerns with the two issues mentioned.

@sburton42 thanks this looks great. Regarding your two points, both of those are good.
It's interesting that right when I thought it would be nice to decouple the JS/CSS from the R package, I encountered a situation today where it will be used in a walled-off system with no access to the outside internet and only a mirror of the R package repository. In that case it is best for those dependencies to stick with the package. I don't know if that's a consideration here with the Python package as well. I like the idea a lot better of the repos being clean and those dependencies getting grabbed later.

@hafen. Great context. I added Issue #21 to capture this idea. Thanks.


Add lints test for all templates

We have no test catching

https://github.com/flutter/flutter/issues/115347

Rather than add a specific test for that, we should add a test that does it for all templates (and all lints/warnings/errors).

Hello @dcharkes. Thank you for raising this issue. Labeling for further insights and visibility.

cc @stuartmorgan

Should be pretty easy to do by adding an integration test that flutter creates each template and then analyzes it.

Can someone provide some guidance on how to begin contributing to this repo?

Hey @Kaps61929. You can read https://github.com/flutter/flutter/blob/master/CONTRIBUTING.md to get started :)


HomeKit Status

Hi Ben,
In EWeLink app something devices goes offline, it's possible to notify this with a "no response" status in homekit?
I saw in your last Govee plug-in version that you fixed that

It is something I am looking into for ewelink. in govee there are two different accessory types so implementation is easier.
In ewelink there are 41 different accessory type files so you can see it will take me much longer to change the code in each file!

We definitely love your effort!
If you can start from Sonoff Zigbee Basiczbr3 switch really appreciated 😜

Please check the newest beta, v6.0.0-0 and also the changelog for the latest homebridge/hoobs requirements.
Also note that the accessory will show as no response only when offline and an attempt to control it has been made.

This is a good news, thanks!
Any chance in the future to have a "no response" message tile without tentative? I don't know with automatic refresh each 60 seconds double checking the status trough EWeLink app

This would be a very complex task.

Some devices support LAN mode, some don't
Some devices could be online via LAN mode but offline via cloud, or vice versa
Some users restrict to LAN only, some to cloud only

There are also some cases where a plugin cannot detect a device over local network (so plugin would think it were offline) but with a manual IP the device works as expected.
There are so many variations, the most straightforward being to just check when a device is controlled if it has been successful.
Also in my opinion I don't see why it would be useful to know if a device is online/offline if in any other situation that wanting to control it?

Ben, let me try to explain my issue.
I have a sonoff zigbee hub with 7 zigbee devices connected.
Sometime two devices goes offline an not come back online until I switch off and on the power on child devices offline.
I normally not showing EWeLink app but I spend more time in HomeKit. This request made to understand when the devices goes offline in order to understand when restart the devices

Please try the latest beta, v6.0.0-2

Please try the latest beta, v6.0.0-2

I have hoobs. Which input I need to write in terminal to install the beta ?

https://github.com/bwp91/homebridge-ewelink/wiki/Beta-Version#hoobs-ui

Let me know if you get any errors. I don’t have any zigbee devices so need you to do the testing!

Let me know if you get any errors. I don’t have any zigbee devices so need you to do the testing!

Let me two hours, coming back at home

Let me know if you get any errors. I don’t have any zigbee devices so need you to do the testing!

Ben, fantastic works well. The status is identified without trying access on devices. This was the request
Now I’m coming back to normal version awaiting official release

The official release won’t be for a while yet. There’s still a lot to do plus testing with different users with different device types. No harm in staying on the beta

Ok I will remain on the beta

when the “official” is released, it will appear as an update in hoobs

Excellent. I will leave this issue open until v6 of the plugin is released which includes the new 'offline' status.


Rename ChangeLog.md to CHANGELOG.md

Why you made the change:
I did this so that it stands out more so that people can easily find it.

:+1:


Endless Access violations

I get an endless stream of these error messages when I start the new 1.2 beta version, launching it as administrator makes no difference. The error messages only stop coming when I kill the application. The older version works.

The windows install is a fairly recent one, maybe a month or two old now, I'm on Windows 10 Home v1803. It's a bootcamp partion on a Macbook Pro. One other thing that does particularly set my system apart and may be a cause is that I run an eGPU to drive my monitor, when in Windows this is my only monitor, internal Laptop monitor is blank. Other than that the install is fairly fresh and has little that I can think of that would be conflicting with the software.
Happy to test or provide further information as needed.

I'm getting the same issue. The stable version works like a charm besides the missing multi-monitor support.

Thank you for your feedback, I'm attaching an updated executable fixing an error that shows Access Violation (commit 062e18de84fcce85e69b42dbe23ac03efd4cce33), hotpost timer was trying to read a window for OSD visualization that was not created yet. That might be the error.
The new executable includes an error reporting tool (MadExcept) that will allow to generate a bugreport.txt file in order to send here if error persists.
WinXCornersWithErrorRerporting.zip
Please try this new version, and if error persists, save the error report to a txt file and upload here.

Getting exception message  : Failed to get data for 'SystemUsesLightTheme'.
bugreport.txt
bugreportadmin.txt

My bad, that was an error accessing registry entries only available on Windows 10 May 2019 Update. Thank you, I attached new executable fixing that error.
 Feels just like my macbook now 😄

Confirmed now working without issue, very nice!

I am still seeing this problem with the executable downloaded directly from the github page. Did you update the main file?


Ordinal plural fails to parse without at least one non-wildcard case

Failing example:
{n, selectordinal, =1 {} other {}}

I think it's a bug? Or were we being more strict here? I don't think we happen to use this type very much.


UTM cannot adjust graphics and cursor settings.

UTM can't adjust graphics and cursor settings, and can't left click. The mouse is stuck in the upper left corner and shakes. My mouse is OK, but UTM can't, and the touch screen can't.

UTM Version: v2.1.2
OS Version: iOS 15.0 (19A5318f)
Device Model: iPad Pro 11
Is no jailbroken.
UTM installed with developer signature.

In PS / 2 mode, there is no problem with the mouse, and the touch screen can also be dragged normally. Press the left and right buttons. As long as PS / 2 is turned off, it cannot be used normally. The mouse is stuck in the upper left corner and shakes. When you click the screen, the mouse will follow your finger, but you can't click the left button.

It seems that notification GCMouseDidBecomeCurrentNotification is triggered before UTMQemuManager is initialized. Thus mouse mode is not changed to proper value.


Add Syntax highlighting to the file manager

Add different colours to const names, keywords, strings... like in code editors and IDEs

Probably will require some work, will look into it after I'm done with updating all the dependencies.


Update combine error

The update points users in the direction of the use_cftime argument, which can solve issues where calendars are decoded into different time classes (see also the discussion in #5767)

[x] Tests added
[x] Passes pre-commit run --all-files
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

Alrighty updated, see what you think

The only other thing this needs is a test to check that the error is raised with the correct message (i.e. pytest.raises(TypeError, match="may fix this issue")

I believe that's covered by https://github.com/pydata/xarray/blob/ec2657bfc1a426aa3559243b02ecf48de6673506/xarray/tests/test_combine.py#L1109 (i.e. no change is needed to the existing code)

I'm also happy - so let's merge!

Thanks @znicholls!


TypeError on passing passing onchange to hb.shape

Here's my code:
from vharfbuzz import Vharfbuzz

poppins = Vharfbuzz("Fonts/Poppins//Poppins-Regular.ttf")

def myOnchange(self, stage, lookupid, buf):
    print(self, stage, lookupid, buf)

buf = poppins.shape("Archit", onchange=myOnchange)
print(poppins.buf_to_svg(buf))

This gives me the following error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
TypeError: Vharfbuzz.make_message_handling_function.<locals>.handle_message() missing 1 required positional argument: 'buf2'
Exception ignored in: 'uharfbuzz._harfbuzz.msgcallback'
Traceback (most recent call last):
  File "C:\Users\abhon\AppData\Roaming\Python\Python310\site-packages\vharfbuzz\__init__.py", line 113, in shape
    hb.shape(self.hbfont, buf, features, shapers=shapers)
TypeError: Vharfbuzz.make_message_handling_function.<locals>.handle_message() missing 1 required positional argument: 'buf2'
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
TypeError: Vharfbuzz.make_message_handling_function.<locals>.handle_message() missing 1 required positional argument: 'buf2'
Exception ignored in: 'uharfbuzz._harfbuzz.msgcallback'
Traceback (most recent call last):
  File "C:\Users\abhon\AppData\Roaming\Python\Python310\site-packages\vharfbuzz\__init__.py", line 113, in shape
    hb.shape(self.hbfont, buf, features, shapers=shapers)
TypeError: Vharfbuzz.make_message_handling_function.<locals>.handle_message() missing 1 required positional argument: 'buf2'

The error comes from here but I don't see what I'm doing wrong. Passing the onchange is what triggers this error.

I think I understand what is going on here. The below function calls the handle_message function with only one argument msg.
https://github.com/harfbuzz/uharfbuzz/blob/f8b12cb12f9cdd79be281f3f218293b8ba59a4b2/src/uharfbuzz/_harfbuzz.pyx#L30-L34
While make_message_handling_function creates a callback that takes in two parameters.
https://github.com/simoncozens/vharfbuzz/blob/fdc52bc3c42ff18f58c4522f8742a955051a045c/lib/vharfbuzz/__init__.py#L45-L64
More importantly, Vharfbuzz.history is empty after a call to Vharfbuzz.shape.
def myOnchange(self, stage, lookupid, buf):
    print("hello")
    print(self, stage, lookupid, buf)

buf = poppins.shape("देवनागरी", onchange=myOnchange)
print(poppins.history)  // Outputs: {'GSUB': [], 'GPOS': []}

I think this is probably due to some change to harfbuzz interally. I'll use uharfbuzz for now. I'm quite to new to harfbuzz in general but if I understand the problem I'll create a pull request in the future.


JMX monitoring does not report engine down in some cases

The engine died with the following last words:
23/03 11:39:34.126|WARN |INTERNAL_POLLER;polling orders;         |AbstractLifeCycle|FAILED org.eclipse.jetty.server.Server@70ac70ac: java.lang.OutOfMemoryError
And the monitoriing through JMX does not report this behaviour. For information, previously in this day, the web also crashed with outofmemory.

OOM are tricky beasts, as themselves cannot be caugth. But I'm slightly distrubred by the absence of alert with a non-functioning node.
Exactly how was the node failing? No web UI, jobs not taken, jobs stuck in ATTRIBUTED... ?
Are you sure the JMX monitoring was failing? Should a node not poll for whatever reason, an alert should be visible. It may be the JMX system was fully down - could you check your monitoring history for lapses in monitoring data?

UI was ok  but indeed, log grabbing reported node dow. Job was in RUNNING
state.
In fact, I don't make a JMX call by myself. I relied on our zabbix system that
use the zabbix-jqm agent.
Digging in zabbix-jqm-agent, I see socketexception.
--
S�bastien

Agent stack trace :
java.net.SocketException: Broken pipe
at java.net.SocketOutputStream.socketWrite0(Native Method)
at java.net.SocketOutputStream.socketWrite(Unknown Source)
at java.net.SocketOutputStream.write(Unknown Source)
at org.oxymores.monitoring.ZabbixClientThread.run(ZabbixClientThread.java:153)
at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)
at java.util.concurrent.FutureTask$Sync.innerRun(Unknown Source)
at java.util.concurrent.FutureTask.run(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)
java.net.SocketException: Broken pipe

If I understand correctly, Zabbix could not ask for JMX data anymore. Perfectly understandable given the type of error.
But in that case, would you agree this is not a JQM bug? It was the responsibility of Zabbix to report it could not fetch the engine status anymore. And the OOM is not a JQM issue either - batch programs using too much memory will always end that way in a Java world...

Indeed, its looks like a jqm-zabbix-agent issue. Not a jqm issue.

Closing as invalid then.


flare command of datadog agent uses app.datadoghq.com even if dd_url is private link

in datadog.yaml, dd_url points private link (AWS VPC endpoint in us-east-1 region).
When we execute datadog-agent flare, it tries to connect "app.datadoghq.com" which can not be reached because there is  no route to internet in vpc.

Hi @ShigeruNakagaki, could you confirm that you're using the dd_url value documented in our docs? (i.e. dd_url: https://pvtlink.agent.datadoghq.com)
If that's the case, this isn't expected. Could you open a support case with our support team and manually attach the flare's zip file to the case so we can investigate further? Thanks.

Hi, Oliver
I'm sure dd_url points https://pvtlink.agent.datadoghq.com.
and I already sent a flare zip manually.
thanks

Thanks, could you share the support case ID here? I'll sync with our support team to check your flare and understand why it's not being sent to the correct URL

Hi.
Here is support id.
406688
thanks


mgmt, local generation for search

Add <revapi.skip>true</revapi.skip> in pom.xml for fix ci analyze exception.
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

A link to the release request?

https://github.com/Azure/sdk-release-request/issues/5004


Android 10 Blank squares

Android 10 users reporting that they see blank images, selecting one causes the crash
Anyone having similar issues?
Tried android:requestLegacyExternalStorage="true"
Same issue
@werbhelius any ideas?

Tried android:requestLegacyExternalStorage="true"
Issue resolved

Tried android:requestLegacyExternalStorage="true"
Issue resolved


Marathon has a bug.  Health checks cannot timeout.

https://github.com/mesosphere/marathon/issues/2078

:+1:

+1


Clean up usage of import React in libraries

We have a few conflicting linting rules with react not having a default export which forces us to ignore linting to check if modules have no default export. This can cause unwanted errors in the future if we introduce new libraries or existing ( popperjs, downshift, ramda etc) change their apis.
Either we change the remaining imports the cause errors to import * as React from 'react' or just remove react imports all together as they are not needed in React 17 anymore.
Notes:

https://javascript.plainenglish.io/react-17-new-jsx-transform-c6f983f3464d

@mimarz so in short you want all the import React from "react" statements to be removed? is that it?

@mimarz so in short you want all the import React from "react" statements to be removed? is that it?

Thats correct, we do however need to still need to import react util functions (useEffect, useState, useRef etc..) where they are used.
Its just the default React import we no longer need with v17 :)

yes yes that will be taken care.

Nice!
You will probably have to add this rule exception to our eslintrc.js file when you start working on this.
   'react/react-in-jsx-scope': 'off',


Typescript support (entry level only)

Attempt to fix issue #41 Enhance Public AgenticJS API TypeScript

Added types for relevant entry API
Missing type for unknown context (WIP)
Added "test:types" script for testing typescript validation
Added package type: module

Hi @alienkarma,
Thank you so much for your contributions! Adding TypeScript to AgenticJS is a significant enhancement that will undoubtedly benefit many developers.
I’ve reviewed your pull request and have a few suggestions. Your expertise is truly valuable to us, and your efforts are greatly appreciated. Please know that these suggestions are meant to build on the fantastic foundation you’ve already laid, without any pressure.

Link to Issue & Description in PR: It would be great if you could include a link to the related issue and a brief description in your PR to provide context for other collaborators.

JSDoc Annotations: Do you think it would be beneficial to add JSDoc annotations to the type definitions? I’m under the impression that doing so could enhance IDE autocompletion features for TypeScript users, making the library more intuitive to use. What's your take on this? Would this addition be helpful in your view? If so you can get it from here(https://docs.agenticjs.com/category/core-concepts)

TypeScript Example Project: When you have a moment, could you create a small TypeScript example in the Playground? Including a README with usage examples would be wonderful, and we’d love to showcase this in our main documentation later on.

Thank you again for your hard work and dedication. I’m here to support you in any way I can.

Nice.
Thank you for the feedback.
I will implement the necessary changes and update once done.

Hi @darielnoel
I added the JSDocs and also created a typescript version of the nodejs playground example.
The type seems to be working (fixed a few type issues because of this as well).
I will work on a proper typescript react example later on.
To add the types it has to be declared as a separate package @types/agenticjs, assuming the module name is agenticjs.
I am attaching the node_modules directory for more clarity. (The index.js and index.js.map are module.esm.js and module.esm.js.map files respectively)

I am not sure yet how to produce a proper distributable output yet using rollup. (except as an external type package)
Anyways, let me know if you have any more feedback and/or changes.

Thank you @alienkarma, I'm sorry for the back and forth. Just trying to make sure that every feature we add has a solid foundation. Here's some of the philosophy we are trying to pursue.
https://resend.com/philosophy
Thanks for your patience and understanding 🤗

Very much looking for TypeScript friendlyness; it really helps diving into a new codebase.

Hi @darielnoel
I added a typed package builder that we can import and use as a package for testing (and maybe distribution).
I have also updated the nodejs-ts playground example using the built package and also updated the readme for it as well.
As always, let me know if any changes/feedback. (I really appreciate it 😄)


Evaluating inductive inference

Hello! Thank you for your excellent work.
I enjoyed reading your paper, and I am currently running some experiments with your code.
However, in inductive mode, it seems that the accuracy is being measured using test_targets. i.e, acc = accuracy(scores, test_targets). Is this correct?
Also, could you please check again if the output shape of do_inductive_lp is (test_features.shape[0], num_classes)?

Oh I misunderstood. So you were referring to the nodes that make up the graph Laplacian as 'unlabeled nodes.' Now it's clear to me. Thanks!


Introduce a callback when user presses on cancel button

I am facing an issue where there is no callback when the user presses the cancel button. This results in the UI not getting updated when a cancellation is initiated.

In addition to the selection of the photo in the following code
ZLEditImageViewController.showEditImageVC(parentVC: self, image: image, editModel: editModel) { [weak self] (resImage, editModel) in
    // your code
}

we should introduce am optional closure, something like
didCancel: {
  // callers implementation
}

https://github.com/longitachi/ZLImageEditor/commit/e5ede2810dc39014848c0b4300eb95320b4fcbaf


Getting Emails When I should not.

Hi
I am trying to setup email sending but when I run my code I am geting like 144 emails even though I expect 1
For instance I getting an email with this in(I pulled it from my log that I am saving)
2019-03-01 10:25:21.5693|2|INFO|Microsoft.AspNetCore.Hosting.Internal.WebHost|Request finished in 9538.3354ms 200 application/json; charset=utf-8
I have this
    <target name="email" xsi:type="Mail"
         smtpServer="smtp.sendgrid.net"
         smtpPort="587"
         smtpAuthentication="Basic"
         smtpUserName="apikey"
         smtpPassword="dfa,addjkdajkfajdf715471571575715757157"
         enableSsl="true"
<EMAIL_ADDRESS><EMAIL_ADDRESS>          />

    <logger name="*" minlevel="Error" writeTo="email" />

So I should just be seeing emails from error level or higher
My other rules
  <!--All logs, including from Microsoft-->
    <logger name="*" minlevel="Trace" writeTo="allfile" />
    <logger name="*" minlevel="Error" writeTo="email" />

    <!--Skip non-critical Microsoft logs and so log only own logs-->
    <logger name="Microsoft.*" maxLevel="Info" final="true" />
    <!-- BlackHole without writeTo -->
    <logger name="*" minlevel="Trace" writeTo="ownFile-web" />

Also is there away to specify more fine grained if an email is sent? I see you can do by like namspace but is it possible to put it on the debug line it self?
logger.LogError("Test email"); // can I say this is the only line that will send an email?

See you sure you're editing the correct NLog.config? (E.g is there on in the bin folder?). Please check the internal log.

@304NotModified
Ok, I will check that out. How about controlling which Error Logs will get an email send out? Is that possible?

I think you're looking for: https://github.com/NLog/NLog/wiki/Filtering-log-messages

I assume your question has been answered, if not, please let us know!


Add FinAi.com

Yes, we're hiring - unfortunately no remote for now.

Sorry, I don't get it ;). Order of what? Company names? Seems correct to me. Or in location column? I see no pattern here.

Oh i was completely wrong! @orient-man Sorry about that!


Extension doesn't seem to work nicely with new Azure Updates site.

When attempting to use extension on a page - ie:
https://azure.microsoft.com/en-gb/updates?id=467872
It turns the URL format into:
https://azure.microsoft.com/en-gb/updates?id=467872**&**WT.mc_id=AZ-MVP-5004796 - and loses the filter, and the update.
If I manually add in the ? - so https://azure.microsoft.com/en-gb/updates?id=467872?WT.mc_id=AZ-MVP-5004796 - it works.

Hi Luke,
unfortunately, I am not able to reproduce the issue. Can you provide some more details - e. g. which browser are you using?
BR,
Martin

Hey, Martin - thanks for giving it ago. I am using the Chrome extension. This is my experience:

This behavior is slightly different from what I was experiencing the other day; looks like they might have changed something on the website again.

I think I found where the issue lies:
https://azure.microsoft.com/updates?id=469263 (doesn't work)
https://azure.microsoft.com**/en-us/**updates?id=469263 (does)
If I run the extension I get:
](https://azure.microsoft.com/en-gb/updates?id=469263&WT.mc_id=AZ-MVP-5004796) (not working)
However if I adjust to:
https://azure.microsoft.com/en-us/updates?id=469263?WT.mc_id=AZ-MVP-5004796 (works)


[BEAM-2837] Updated grpc-google-pubsub-v1 dependency

Updated grpc-google-pubsub-v1 dependency.
R: @jkff

Motivation?

@robertwb to fix BEAM-2837. Cloud Spanner expects grpc-google-common-protos of version 0.1.8, otherwise fails on runtime.

Mairbek confirmed that a Dataflow pipeline reading and writing to pubsub works with this PR, but for extra safety I'm gonna run ValidatesRunner tests before merging.

Run Dataflow ValidatesRunner

Coverage remained the same at 69.549% when pulling 8c54b6e59d6d844d9d7508c84754d4e9fc9a4836 on mairbek:deps into 5dab9e109cffefb69ac6be22b2d902d5579797e1 on apache:master.


Backquote position is incorrect with Ricty Diminished font

The backquote display position is incorrect like below

It occur in Ricty based fonts.

VSCode Version: 1.40.0
OS Version: Windows 10

Steps to Reproduce:

Change Font to Ricty Diminished
Show Text Using backquote such as `some_text`

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

@coponhub Please contact the font developer, I believe this to be a problem with the font.

@coponhub Thank you for response.

This problem not happened in emacs.
I think Ricty font users ware enough to many notice this problem If this problem ware continue to occur in long time.
I want to know scope and why occur this problem.

@coponhub VSCode is built with Electron, which is built with Chromium. Chrome is also built with Chromium. I think this is an issue with the font or with the font rendering of Chromium. Can you please open https://jsfiddle.net/yhd06wm2/1/ in Chrome and Firefox, and compare the rendering.


test PR with Github API

Please pull this in!

Docs Build status updates of commit a0aad28:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/Peachying/testinlineedit, working directory: W:\chnf-s, target branch: HEAD, target branch commit: 6d7a880bd04c626426fcf12064cbec150a01e64d, source commit: a0aad2809e06d06be65dd690dc2a76e40c78021f.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit f13fcb1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

node-azure-tools.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


details explain on searching on specific key range

anyone please help i need help on this script.
clBitCrack -d 0 -c -u -o found0.txt --keyspace 1:4000000000000 --continue start.txt -i adresses_50BTC.txt
what does mean 1:4000000000000 this? does that mean bitcrack will search in those key range, but i don't understand if those numbers are hex,decimal or just a random numbers?please help

Yeah, he'il search within that range. in order.
The number you write is weighted as 16. whatever you write is automatically 16.

Yeah. it will search


New Games Not Working

Hello.
Recently, new jeopardy games have not been working. When I enter the show title the game board becomes a black screen, starting with game 8807. I know that there were recently some changes to J-Archive!, so maybe that has something to do with it. I am a big fan of this software and it has made playing jeopardy games with my friends significantly easier. Thank you.

I refresh the games manually every few months--they are not immediately available for play after being aired.


changed card-react-component.js in order to remove dots

@shatran changed the hard coded dots into individual objects for default values.  Like i said in the issue, I was getting bad chars on page load because of the dot encoding issue.
Thanks

Hi @sstern6, I've published release v1.1.20 with your fix. try it out and let me know if it helped


vimdoc?

would be nice to be able to just convert the readme into doc/
done know how possible that is

added in a9576eddf346aa88b7c4bb5e297f4cbb26c49616


Deriving serde::Serialize causes an error to incorrectly be displayed

When a struct has serde::Serialize #[derive]d on it, rust-analyzer shows an error tooltip despite no actual errors occurring. Here is a minimized example.
proc macro returned error: proc-macro panicked: called `Result::unwrap()` on an `Err` value: ParseFloatError { kind: Invalid }

This was also happening to @Animeshz (https://github.com/rust-analyzer/rust-analyzer/issues/7100#issuecomment-858631472) and @Grimmp (https://github.com/rust-analyzer/rust-analyzer/issues/7100#issuecomment-859061227).
I am running b641a6607 2021-08-16 stable on VSCodium 1.57.1.

Which Rust toolchain are you using and do you have any RA settings în Code?

@lnicola It seems like this got fixed by updating rustc to the latest nightly. (RA settings are the default).
Broken:
$ rustc -V
rustc 1.56.0-nightly (a6ece5615 2021-08-03)

Fixed:
$ rustc -V
rustc 1.56.0-nightly (29ef6cf16 2021-08-31)


Rename Skeleton components

Fixes

Related to #495 by @obulat
Description

This PR renames the Skeleton folder and components within it, adding the V prefix.
Testing Instructions

Loading skeletons should work properly without errors or warnings in the console.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

The use of GridSkeleton needs to be updated in VAllResultsGrid, there is an error in the browser console on the results page. Otherwise, thanks for updating these component names!

Thank you for catching this, @krysal ! I've found a couple more places and fixed all of them.


attempting to fix pypi images not appearing

Adding images as absolute URLs instead of relative paths within the repo to resolve issue with images not appearing in the pypi page for the project
Description

Related Issue
https://gitlab.jatic.net/jatic/team-metrostar/t-e-platform/-/issues/544

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

For future reference, this was likely because the data files which exist outside the package were not being included in the pypi package. If that's the case, then an approach from this question should fix it.


Optional causes issue when mapping List property

https://github.com/kswoll/npeg/blob/51d0302348f9574845d3221fe8748e5298498cf0/PEG/Builder/PegBuilder.cs#L366-L376
System.InvalidCastException: Unable to cast object of type 'PEG.SyntaxTree.Optional' to type 'PEG.SyntaxTree.OneOrMore'.

It's logical to process Optional in the same way as ZeroOrMore

That makes sense to me.  Out of curiosity, what line was throwing that cast exception?

@kswoll line 375 of course. But that doesn't matter, since I had to refactor the whole function to fix all arising issues (it wasn't just this one). I'll send a PR with it today.
This project is truly amazing, the use of code generation behind the scenes really simplified the PEG description in natural C#.

The exception was due to my mistake, but it still wouldn't hurt.

This can be considered as closed and fixed by #7.


Introduce prd environment

Add a “PRD” environment to go w/ the current “DEV” environment.

Document and provide an example of how upgrades flow from lower (dev) to higher (prd).

Completed in #31


Add log role to message container

Fixes #936

I wouldn't want to merge this if it's entirely useless for screen readers.

Yeah. I'm not sure what to do though, it might just be one of the many weirdnesses from VoiceOver. Not sure how to get help on that.


Local copy of datasets

Pylearn2 have a feature that is independent of the rest. If the user define an env variable, it will copy the data from a path to another one that should be local. This help lower the NFS server load.
https://github.com/lisa-lab/pylearn2/blob/master/pylearn2/datasets/cache.py
I think this is what is needed:

[ ] Copy this file to fuel
[ ] Modify the env variable name to have a fuel name
[ ] Modify current dataset to use it (mostly just call a function with the original file name)

This function call will return the file name to use, the original or a local copy.
It will ensure there is no parallel copy. It will also check there is enough memory space. It will also set a read lock, so in the future, we could delete that local cache if it miss space.
This lowered significatively the file server load when it was implemented in Pylearn2.

closing as this is a duplicate of https://github.com/mila-udem/fuel/issues/173


cylinder odd result with only r1 given

If you make a cylinder by just giving r1 a default r2=1 is used.  This is both weird and doesn't match base OpenSCAD, which sets r2=r1 in this case.  I wonder if there are other cases of this kind of weirdness when there are two radii?  Or other ways in which the revised primitives disagree with base openscad.

cylinder(h=5,r=5);
right(7)
cylinder(h=5,5);
right(14)
cylinder(5,5);
right(22)
cylinder(5,r2=5);
right(32)
cylinder(5,r=5);
Seems defaults are coming into play.
But in unexpected ways

Just discovered that the same thing happens in 2019-05 without using BOSL2.
cylinder(5,3);

I'm confused.  Same as what?  Is that a cone with radii 3 and 1 and height 5?
The basic problem is that we need for cylinder to behave exactly the same in BOSL2 as out of it so that code doesn't break because you're in BOSL2.

Exactly. That was the result in 2019.05
I was not in BOSL2.

I get the same exact results in OpenSCAD 2020.12.11 whether or not BOSL2 is included, using the following code:

I cannot reproduce this bug.


How CVAT processing images?

My actions before raising this issue

[✅] Read/searched the docs
[✅] Searched past issues
I have CVAT deployed in cloud. I upload a video in task, annotate it, and when i try to export annotated images - CVAT crashes

Expected Behaviour
Expected to get annotated images
Current Behaviour
got a crash of cvat

Please fill the issue template.


extraction of embedded files

Access for embedded files, these are called embedded smart objects by Adobe I think.
I'm using this to save embbedded Illustrator files as SVG without using any Adobe product using
for i, emb in enumerate(psd.embedded):
    emb.save_as_svg('/tmp/psdexport/icon-{0}.svg'.format(i))

Hi @vbraun,
Thanks - the code looks good!
Could you please check the test faiures? It seems this PR breaks parsing of some PSD files (try smart-object-slice.psd file).
If these faiures are hard to fix or if you expect more of them then it is better to stop earlier in the new decode function instead of showing a warning, in order to prevent an expection in later code. This will allow to extract at least some information in cases the existing decoder of embedded objects doesn't work.

It seems related, but is not quite what decode_descriptor understands. It doesn't start with a unicode string, e..g.  In placedLayer.psd the mysterious "file open descriptor" is
'\x00\x00\x00\x10\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00null\x00\x00\x00\x01\x00\x00\x00\x08compInfoObjc\x00\x00\x00\x01\x00\x00\x00\x00\x00\x00null\x00\x00\x00\x02\x00\x00\x00\x06compIDlong\xff\xff\xff\xff\x00\x00\x00\x0eoriginalCompIDlong\xff\xff\xff\xff'

Yeah, that's something different indeed. I don't understand what is going on before \x08compInfoObjc. \x08compInfoObjc and beyond looks like the data decoded by this code: https://github.com/kmike/psd-tools/blob/2a73b59fb72ba7ba35d75267cb7f250be08a71cb/src/psd_tools/decoder/actions.py#L92

\x00\x00\x00\x01 could be "item count", but I'm not sure. "null" is classID. 4 zero bytes before it is classID_length.

Thats what I'm thinking, too:
\x00\x00\x00\x10   # some integer = 16

\x00\x00\x00\x01\x00\x00  # unicode empty string

\x00\x00\x00\x00   # classID length 0 means: 4
null               # classID

\x00\x00\x00\x01   # descriptor item count

\x00\x00\x00\x08   # key length
compInfo           # key
Objc               # ostype = descriptor

\x00\x00\x00\x01\x00\x00  # unicode empty string

\x00\x00\x00\x00   # classID length 0 means: 4
null               # classID

\x00\x00\x00\x02   # descriptor item count

\x00\x00\x00\x06   # key length
compID             # key
long               # ostype = integer
\xff\xff\xff\xff   # integer

\x00\x00\x00\x0e   # key length
originalCompID     # key
long               # ostype = integer
\xff\xff\xff\xff   # integer

yeah, this works: decode_descriptor(None, BytesIO(data[4:]))

Its still a mystery to me why Adobe writes this in the file, but it does decode:
Descriptor(name=u'', classID='null', items=[('compInfo', Descriptor(name=u'', classID='null', items=[('compID', Integer(value=4294967295)), ('originalCompID', Integer(value=4294967295))]))])

Could you please preserve these extra fields (have_file_open_descriptor, file_open_descriptor and uuid, anything else?) in LinkedLayer data structure?

Even if they don't look useful it is nice to be able to restore the original data from the decoded data as much as possible.

Since have_file_open_descriptor == (file_open_descriptor is not None) I don't think we need to store it, agree?

yes, it makes sense

I don't have a preference, whatever you think fits best

The PR looks good :+1:
Could you please add yourself to https://github.com/kmike/psd-tools/blob/master/AUTHORS.txt? I'll merge this PR after this last fix.


timeout_ms docs are misleading

Docs reference timeout_ms as being the 'time to wait for the query to complete':
https://github.com/GoogleCloudPlatform/google-cloud-python/blob/master/bigquery/google/cloud/bigquery/query.py#L386
This is contradictory to the actual use of timeout_ms as the API docs describe it as the request timeout:
https://cloud.google.com/bigquery/docs/reference/rest/v2/jobs/query#timeoutMs
TimeoutMS is distinctly NOT the time 'to wait for query to complete'. The docs should be more explicit since query vs request timeout are vastly different things.

Hi @roscoejp,
Thanks for reporting. I agree that this is in error.

https://github.com/GoogleCloudPlatform/google-cloud-python/pull/3320 to update the docs.

#3320 was merged; closing this.


Fix missing data.

Looks like the update for certain days didn't fix the data missing problem. Not sure why. Need to keep an eye to it later.


async logger + slog_stdlog never flushes

I have this test program:
#[macro_use] extern crate log;
extern crate slog;
extern crate slog_envlogger;
extern crate slog_stdlog;
extern crate slog_term;

use slog::DrainExt;

fn main() {
    let drain = slog_term::StreamerBuilder::new().stderr().async().full().build();
    slog_stdlog::set_logger(slog::Logger::root(slog_envlogger::new(drain).ignore_err(),
                                               None)).unwrap();
    info!("exiting.");
}

compiled with these dependencies:
[dependencies]
log = "0.3"
slog = "1.5"
slog-envlogger = "0.5"
slog-stdlog = "1.1"
slog-term = "1.3"

If I run it with RUST_LOG=info set, I don't see the exiting. log line (at least, not consistently). I do if I remove the .async().
It looks like slog-async::AsyncCore (used internally by slog-term) has a Drop trait which flushes the async logger. But it's never run. A logger installed with slog_stdlog is never destroyed.
IMHO, the best solution to this would be for a logger to not be permanently async. Rather, when you install a logger, you get some handle which you can use to make it async in a scoped way. When leaving the scope, it flushes and reverts to sync, so log messages are never dropped.

Ha. Thank you for reporting it! I have noticed that yesterday evening when porting and testing crates to slog  v2, but did not have a chance to fill an issue or investigate.
I was thinking about just removing slog_stdlog::set_logger/slog-scope::set_logger call and everything will always have to be called through the scope so no globals/statics are involved.
I need to investigate more.
BTW. I think in your investigation you might have looked at some crates from slog v1, and some from v2. The exact structure and interactions changed a bit, so it might be a bit confusing. Sorry for that! And I advise just being cautios about v1 vs v2.

So after some though I am still thinking that removing slog_scope::set_global_logger is the way to go. Thoughts?

I'm confused. How would this interact with the log crate's interface? In the slog-scope-stdlog (which I guess is is the v2 stuff):
pub fn init() -> Result<(), log::SetLoggerError> {
    log::set_logger(|max_log_level| {
        max_log_level.set(log::LogLevelFilter::max());
        Box::new(Logger)
    })
}

doesn't the Logger have to live forever? how does removing set_global_logger change that?

The logger set in the above function is a different Logger. So every info! from legacy log crate is processed by Logger::log function. That function uses with_current_logger to look up the slog's Logger to forward the messages to.
So init will still setup global log's Logger, but there will be no global GLOBAL_LOGGER to fallback to. The root of the problem is that lazy_static never gets a destructor called so async thread might be killed before having a chance to flush all the messages.

Oh, I see thanks. So on a thread that hasn't specifically set up a logging scope, such as one created by a slog-unaware library, the standard logging routines would do nothing? Hmm, that's not too appealing to me. But I have a confession to make: I'm not really into structured logging at all (I only use slog because it was the first logging crate I found that did async). Maybe for people who are taking more advantage of it and adding key/value pairs to their scopes that makes more sense. It's possible I'm just using the wrong crate for my purposes.

Oh... right. The whole reason why the global was there, was just in case to catch the thread's that couldn't setup their own scope. Ehhh... I dislike the whole slog-scope because globals are evil. :D
Sure, if you really want to have just simple global logger, you might look for something else. But the problem (one of the problems) with log based loggers is they are not composable: some will have feature A, some feature B, some C, and there's no way to use them together.
Well, the workaround for your current problem is probably just adding thread::sleep for couple of milliseconds at the end of main until I figure out better api.

I hit the same problem recently. My solution was to place a guard object in the main and reset the global logger to a new Logger::root(Discard, o!()) when it is dropped. Then the usual drop on the async logger gets called.
I'm not sure something can be done to fix the real problem on the library side. But I'd propose adding a note to the documentation with some suggested workaround, so people are aware of it and don't spend hours looking why some messages didn't make it.

fwiw, I tried out the approach I mentioned in a (non-slog) simple logging crate:

IMHO, the best solution to this would be for a logger to not be permanently async. Rather, when you install a logger, you get some handle which you can use to make it async in a scoped way. When leaving the scope, it flushes and reverts to sync, so log messages are never dropped.

I'm happy with it. Source here; the gist is that you build a Handle (which is really just an Arc<Logger>); in addition to a method which installs that with log::set_logger_raw, there's one that returns an AsyncHandle. The logger is async until the AsyncHandle is dropped, then it's flushed and reverted to sync.
I like this because you don't have a point where you arbitrary start discarding stuff, but you still get async while your program is doing most of its work.

I have just published slog-scope = 2.0.0-3.0 with a guard implementation. Please let me know how you like it. It can be changed to something else if it doesn't work well. https://github.com/slog-rs/scope/commit/92c5e454f5f29e7dad69849b66461b2f2088834e

https://docs.rs/slog-scope/2.0.0-3.0/slog_scope/fn.set_global_logger.html

Thanks, it does work (and it even warns for an unused result :-).

Closing for now. Feel free to keep commenting or reopen if needed.


Allow to pass doctype option

ERROR in ./app/index.html
Module parse failed: /Users/jjalonso/napper/app/index.html Line 1: Unexpected token <
You may need an appropriate loader to handle this file type.
|
|
|


Support .ini Files

VPX 10.8 comes with with new .ini files.
Support the renaming of these two (plus other operations?).

Comes with >2.11.1


undefined TypeErrorMessage

All of the methods on the TypeErrorMessage object have the syntax "return undefined.getWhatever()."  Looking forward to the new version.

Thanks for the bug report. Should be fixed in v3.0.4.


Allowing consumer of HS specified device type for each service

This is the change we need for production ,  for more background we recorded here https://github.com/eBay/HomeObject/issues/110
Consumer setting the device type to auto will let HS choose based on what it has.

Codecov Report
Attention: Patch coverage is 50.90909% with 27 lines in your changes are missing coverage. Please review.

Project coverage is 58.36%. Comparing base (2c500c5) to head (fd75dc0).

Files
Patch %
Lines

src/lib/homestore.cpp
44.44%
19 Missing and 6 partials :warning:

src/include/homestore/homestore_decl.hpp
50.00%
1 Missing :warning:

src/lib/blkdata_svc/blkdata_service.cpp
50.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #355      +/-   ##
==========================================
- Coverage   61.26%   58.36%   -2.90%
==========================================
  Files         135      107      -28
  Lines       14361     9363    -4998
  Branches     1727     1209     -518
==========================================
- Hits         8798     5465    -3333
+ Misses       4834     3368    -1466
+ Partials      729      530     -199

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

LGTM, can you also run ./bin/test_raft_repl_dev on your local machine (i don't think this PR will break anything) which currently doesn't run in conan build as well as github CI, just to make sure everything is right.

Merge as test_raft_repl_dev passsed
[       OK ] RaftReplDevTest.Leader_Restart (28386 ms)
[       OK ] RaftReplDevTest.Leader_Restart (28386 ms)
[       OK ] RaftReplDevTest.Leader_Restart (28386 ms)
[----------] 6 tests from RaftReplDevTest (115097 ms total)

[----------] 6 tests from RaftReplDevTest (115090 ms total)

[----------] 6 tests from RaftReplDevTest (115278 ms total)

[----------] Global test environment tear-down
[----------] Global test environment tear-down
[----------] Global test environment tear-down
[==========] 6 tests from 1 test suite ran. (115097 ms total)
[==========] 6 tests from 1 test suite ran. (115090 ms total)
[==========] 6 tests from 1 test suite ran. (115278 ms total)
[  PASSED  ] 6 tests.
[  PASSED  ] 6 tests.
[  PASSED  ] 6 tests.


Type Element is not assignable to type string

Do you want to request a feature or report a bug?
feature
What's the current behavior?
titleAccessor only works with string return type.
What's the expected behavior?
Hi, I am working on a Project where the calendar needs to show multiple types of events like tasks, reminders, events. So, the requirement is to display an icon according to the event type before title. I tried with titleAccessor option. Everything is working fine. but @types/react-big-calendar requires to return string on function which is also stated on the documentation. You can check the code below.
  const handleEventTitle = event => {
    return (
      <span>
        <CalendarOutlined />
         {event.title}
      </span>
    );
  };

return (
  <DragAndDropCalendar
    ....
    titleAccessor={handleEventTitle}
  />
)

It would be helpful if there are any options to do that or should this be a new feature?
Thanks

I just found the components api at the end of the docs which solved the problem. Thanks for this great library.


chore: Re-generated to pick up changes from self.

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/6670e800-5ae0-42a0-9f59-45848d8d694a/targets

[ ] To automatically regenerate this PR, check this box.

Attempting to rename flakybot back to buildcop


How to use EditingWidget

Good morning,
In the example provided to use EditingWidget you needed the mapControl and the EditManager or in the nugget version you also need EditManipulation.
Is it possible to update the wpf document?
I don't understand what to do.
How to redo, as in the documentation, the movement of a point with a visualization of the point in movement.

I will convert this into a discussions.


OperationCanceledException handling in ReadUntilCancelledAsync() and ReadAllAsEnumerablesAsync()

Now ReadUntilCancelledAsync() and ReadAllAsEnumerablesAsync() catches all OperationCanceledException exceptions, even if they were thrown by user code in receiver.
using var httpClient = new HttpClient();
httpClient.Timeout = TimeSpan.FromSeconds(5);
var url = "https://reqres.in/api/users?delay=10"; // responds with a delay of 10 seconds

async ValueTask LoadAndPrint()
{
    var json = await httpClient.GetStringAsync(url, ct); // throws TaskCanceledException due to timeout
    Console.WriteLine(json[..10]);
}

// must throw TaskCanceledException
_ = await Enumerable.Repeat(0, 10)
    .ToChannel(cancellationToken: ct)
    .ReadUntilCancelledAsync(ct, (_, _) => LoadAndPrint());

// must throw TaskCanceledException
await Enumerable.Repeat(0, 10)
    .ToChannel(cancellationToken: ct)
    .ReadAllAsEnumerablesAsync(_ => LoadAndPrint(), cancellationToken: ct);

I don't think this is going to change, because it's intentional to "just bail" if there's a cancellation token that has been cancelled.
The token itself is effectively "owned/borrowed" by the caller, and instead of a try catch, you simply call token.ThrowIfCancellationRequested().
Take a look at the differences between the various methods.  It's intentional.  There are plenty of cases where it absolutely will throw when the token is cancelled because that's expected for something as simple as .ReadAllAsync() or the like.
And again, for .ReadAllAsEnumerables: "cancellelation" is a 'signal' to bail out.  And then the caller can or should own what they want to do with it.

The token itself is effectively "owned/borrowed" by the caller, and instead of a try catch, you simply call token.ThrowIfCancellationRequested().

I'm not sure I got the idea right.
In the example above, the exception TaskCanceledException (inherited from OperationCanceledException) is thrown inside the HttpClient.GetStringAsync() method due to a timeout. At the same time, the ct token is not cancelled.
As a result, ReadAllAsEnumerablesAsync() is executed without exception (although not the whole pipeline is worked out) and ct.ThrowIfCancellationRequested(); will not throw an exception.

I think I see the nuanced edge case here...
You are suggesting that if the delegate itself throws a cancellation exception, but the CancellationToken is not cancelled, then it should not trap the exception.

Fixed: https://www.nuget.org/packages/Open.ChannelExtensions/8.4.0

I will also update the minor versions for 6 and 7.

Another few changes to make things a bit more correct:
https://www.nuget.org/packages/Open.ChannelExtensions/8.4.1
If you'd be so kind as to review these changes too:
https://github.com/Open-NET-Libraries/Open.ChannelExtensions/commit/b1f4df088aa22f030b65471baf6f2006477f9e66

LGTM
As I understand, the changes to ReadUntilCancelledAsync/ReadAllAsEnumerablesAsync (WaitToReadOrCancelAsync -> WaitToReadAsync) change the behaviour on explicit cancellation. Now these methods throw OperationCancelledException if the token passed above in chain is canceled.
This makes sense, now the behavior is similar to the methods from BCL.

I found another example of incorrect behavior when throwing OperationCancelledException.
ValueTask<int> AsyncReceiver()
{
	throw new OperationCanceledException("test exception");
}

// OK, throw exception
_ = await Enumerable.Repeat(0, 10)
	.ToChannel()
	.ReadAllAsync(async _ => await AsyncReceiver());

// NOT OK, doesn't throw exception
_ = await Enumerable.Repeat(0, 10)
	.ToChannel()
	.PipeAsync(1, async _ => await AsyncReceiver())
	.ReadAll(_ => { });

As I understand it, the problem is how ContinueWith handles OperationCancelledException (t.Exception = null and t.Status = Cancelled here)

LGTM

The update is already in for the previous examples.
I'll need some time to investigate the one  you just posted.

I strongly suggest in the future, if you have a fork and want to merge a PR, start with adding a test to the test project that proves the current behavior is incorrect.  Then I can pull that down and decide what to do and ultimately merge the fix.

Investigating now.

As I understand it, the problem is how ContinueWith handles OperationCancelledException (t.Exception = null and t.Status = Cancelled here)

You are correct.  This one is interesting.
I'm not sure how this one is fixable as PipeAsync allows for concurrency and that's how this is handled.
Gonna have to think on this one.  Any ideas?

Fixed and released to 8.4.2.  Great catch.  Thank you.
Added special and simple handling that if the task is cancelled, but not by the token, then be sure to tell the channel why by passing it an OperationCancelledException.
https://github.com/Open-NET-Libraries/Open.ChannelExtensions/blob/master/Open.ChannelExtensions/Extensions.Pipe.cs#L11

I'll update the v6 and v7 versions soon.