Conflicting information for Switch.System.Windows.Data.Binding.IListIndexerHidesCustomIndexer

Conflicting information on the doc page:
"Switch.System.Windows.Data.Binding.IListIndexerHidesCustomIndexer	... Controls whether custom IList indexers are used incorrectly (false) or correctly (true) by the System.Windows.Data.Binding class..."
but here, it says the opposite... "...app must opt in to the change by adding the [switch] to the app config file and setting it to false"
see here...
https://github.com/microsoft/dotnet/pull/968/files#diff-400d80d469c7d4b228943e37d839dd5aR19-R29
So is the new behavior enabled via true or false?

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

ID: e59350ee-7837-34f8-b50e-c9c136720155
Version Independent ID: 19747ce7-da26-bf2e-5066-1df11e61b0d0
Content: <AppContextSwitchOverrides> Element
Content Source: docs/framework/configure-apps/file-schema/runtime/appcontextswitchoverrides-element.md
Product: dotnet-framework
GitHub Login: @mairaw
Microsoft Alias: dotnetcontent

Thanks for your feedback @glenn-slayden.
@vatsan-madhavan is this something you can help with?

From <AppContextSwitchOverrides> Element documentation:

Switch Name
Description
Introduced

Switch.System.Windows.Data.Binding.IListIndexerHidesCustomIndexer
Controls whether custom IList indexers are used incorrectly (false) or correctly (true) by the System.Windows.Data.Binding class.
.NET Framework 4.8

From Data Binding improvement for KeyedCollection documentation:

In order for the application to benefit from this change, it must run on the .NET Framework 4.7.2 or later, and it must opt in to the change by adding the following AppContext switch to the  section of the app config file and setting it to false

    <?xml version="1.0" encoding="utf-8"?>
    <configuration>
      <startup>
        <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.7"/>
      </startup>
      <runtime>
        <!-- AppContextSwitchOverrides value attribute is in the form of 'key1=true|false;key2=true|false  -->
        <AppContextSwitchOverrides value="Switch.System.Windows.Data.Binding.IListIndexerHidesCustomIndexer=false" />
      </runtime>
    </configuration>

In general, AppContext based application compatibility flags (sometimes we call this a  quirk) in WPF, work like this:

The flag is only meaningful on a runtime >= vFixIntroduced (v4.8 in this case)
The value of the _flag/quirk depends on the TargetFrameworkMoniker (TFM) of the application (i.e., the version of .NET that the app targets).

The default value of the quirk is false. If the app's TFM>=vFixIntroduced, the _quirk) is set to false by default. (For e.g., when TFM=v4.8, the quirk in discussion will get a default value of false).
When TFM < vFixIntroduced, the quirk gets a default value of true. (In this example, if the TFM of the app were
4.7.2, then the quirk in discussion would have a default value of true)

Typically, the bug-fix is written in such a way that when quirk==false, the bug-fix is active, and when quirk==true, the bug-fix is turned-off. (yeah, a little bit of double-negative going on here... )
The quirk can be set to true or false regardless of TFM using App.config via the AppContextSwitchOverrides mechanism.

What I've described above is the general case. There are situations where the quirk's default-initialization logic may be inverted, the quirk may not be sensitive to TFM at all, etc. This is why each application compatibility flag gets its own documentation.
From what I can tell, Switch.System.Windows.Data.Binding.IListIndexerHidesCustomIndexer follows the general pattern.
@SamBent, does the documentation for  Switch.System.Windows.Data.Binding.IListIndexerHidesCustomIndexer  at <AppContextSwitchOverrides> Element have an error? I suspect that it inverts false/true...

There are two errors.
First, the doc page for AppContext switches says:

Controls whether custom IList indexers are used incorrectly (false) or correctly (true) by the System.Windows.Data.Binding class.

The words 'false' and 'true' should be interchanged.
Second, the doc for "improvement for KeyedCollection" says

In order for the application to benefit from this change, it must run on the .NET Framework 4.7.2 or later...

It should say

In order for an application that targets an older version to benefit from this change, it must run on the .NET Framework 4.8 or later...


QEMU memory adapter

Requires https://github.com/simbricks/qemu/pull/1.

About the failing checks: They are about the disaggregated branch in general. I recommend merging this for now and the doing a separate pass over the disaggregated to fix the failing checks. I tried to at least not make them worse.

Something is still off in the the sysbench_memory experiment, so don't merge yet.

I just want log what the last state when I worked on this was before I completely forget.
The implementation in QEMU is quite hacky. Essentially we are suspending the CPU's execution when receiving a request for an operation on distant memory. This generally works fine when accessing this memory via busybox' devmem. When actually running a program like sysbench purely on this distant memory, page tables are created there. The issue appears when handling a cache fault. The code in QEMU expects it to be handled immediately. We are not doing this and instead trying to suspend the CPU. QEMU, however, will just enter the fault handler again and issue another read request. So essentially, we are deadlocked and the simulation can't continue to actually fetch the data because the loop that sends SimBricks synchronization messages and handles the response to the read request is never executed. The only thing we are doing is indefinitely issuing read requests.


Support StringViewArray interop with python: fix lingering C Data Interface issues for *ViewArray

Which issue does this PR close?
Closes #6366
Rationale for this change
Previously, StringViewArrays and BinaryViewArrays sent to PyArrow will panic.
If you use the updated test_to_pyarrow() from this PR without the additional changes, it will fail.
What changes are included in this PR?
The specification states that the C Data Interface format for StringViewArray and BinaryViewArray includes an extra variadic_buffer_sizes buffer at the end of the various output values.
We were not including that before. There was a prior PR to add it for the IPC format, but not for the C Data Interface.
I cribbed the relevant C++ code for this: https://github.com/apache/arrow/blob/ab0a40ee34217070f14027776682074c55d0b507/cpp/src/arrow/c/bridge.cc#L584-L609
Are there any user-facing changes?
It should now be possible to share StringViewArray and BinaryViewArray over C Data Interface.

@alamb I believe I've implemented all of the requested tests, thanks for reviewing!

Thanks again @a10y


refactor: simplify api.js bundling

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[x] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information

We need to test this on Webpack and CRA. https://github.com/tauri-apps/tauri/issues/2360#issuecomment-903894967

The target bump in tsconfig is not really important and can be reverted, will test CRA anyways.

CRA template created with create-react-app works fine.


Add device trigger support for aqara 86sw2 switches to Deconz

Adds device trigger for Aqara Wireless Light Control (aqara 86sw2) switches

Awesome 🎉


Alternate site for site sync doesnt work for sequences

How to test it:

open Sync Queue
try to do Re-sync active site on representation with a multiple files
all files should be redownloaded (reuploaded)

Task linked: OP-2041 Alternate site for Site Sync doesn't work for sequences


formating the field!

There is any way to format the field to avoid mistakes?
Like phone number : +55619999-9999
or email, dates?
thanks!

@allantorres use tag attribute standard: pattern it takes a regular expression.


Stripe SCA: This transaction requires authentication.

We had one complain to support and a few similar API logs. What's happening:

Our recurring contribution job triggers the payment and is challenged by Stripe SCA
We send an email to the financial contributor to confirm their payment
They click on the link in the email, they arrive on the confirmation page
They briefly see the 3dSecure popup but it is somehow automatically closed
They see the following error on their screen

In the API logs, this is matching with:
error: Unknown Stripe Payment Error: Your card was declined. This transaction requires authentication.

Cause:

3dsecure not supported by the bank?
3dsecure failure at the bank?
Stripe bug?
UI glitch at Open Collective Frontend

Workarounds:

Wait for next attempt and see if it works?
Click on the link in the email again and see if it works?

Todo:

Display a more specific message when we will have more info

@znarf I think we resolved this one? I don't see anything in the logs.


Upgrading pytest-homeassistant-custom-component to 0.0.20

@raman325 - I was looking at upgrading pytest-homeassistant-custom-component from 0.0.8 to 0.0.20.  That requires a few other upgrades, including these in tests/requirements_test.txt:
coverage==5.3
pre-commit==2.8.2
pytest==6.1.2

After adding an empty custom_components/__init__.py file (see this issue), all the tests run except tests/test_apps_modules.py which stalls without any error until you hit ^C.
Would you mind seeing what's wrong with tests/test_apps_modules.py?
Once it runs cleanly then I'll update tests/requirements_test.txt.
Thanks!

To be quite honest with you, at first glance I don't see anything wrong. I also looked at the diff between 0.0.8 and 0.0.20 and nothing sticks out: https://github.com/MatthewFlamm/pytest-homeassistant-custom-component/compare/v0.0.8...0.0.20 I will try to look at this in the upcoming weekend, this week is tough due to Thanksgiving

Run the test suite with pytest --timeout=2 tests and you will see exactly where it was hanging. Common cause for hanging tests is calling a sync method from async context.

Thanks @balloob I am seeing the same exception I saw last night which is an UnknownHandler exception.  For consistent
        # behavior with the other selector classes, we prevent that here
        # (using max). See https://bugs.python.org/issue29255
        max_ev = max(len(self._fd_to_key), 1)
        ready = []
        try:
>           kev_list = self._selector.control(None, max_ev, timeout)
E           Failed: Timeout >2.0s

/usr/local/Cellar/python@3.9/3.9.0_1/Frameworks/Python.framework/Versions/3.9/lib/python3.9/selectors.py:562: Failed
----------------------------------------------------------- Captured stderr setup ------------------------------------------------------------
DEBUG:asyncio:Using selector: KqueueSelector
------------------------------------------------------------ Captured stderr call ------------------------------------------------------------
INFO:homeassistant.loader:Loaded pyscript from custom_components.pyscript
WARNING:homeassistant.loader:You are using a custom integration for pyscript which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant.
INFO:homeassistant.setup:Setting up pyscript
DEBUG:homeassistant.helpers.restore_state:Not creating cache - no saved states found
INFO:homeassistant.setup:Setup of domain pyscript took 0.0 seconds
DEBUG:homeassistant.helpers.restore_state:Dumping states
INFO:pytest_homeassistant_custom_component.common:Writing data to core.restore_state: {'version': 1, 'key': 'core.restore_state', 'data': []}

+++++++++++++++++++++++++++++++++++ Timeout ++++++++++++++++++++++++++++++++++++

~~~~~~~~~~~~~~~~~~~ Stack of SyncWorker_1<PHONE_NUMBER>39648) ~~~~~~~~~~~~~~~~~~~~
  File<EMAIL_ADDRESS>line 908, in _bootstrap
    self._bootstrap_inner()
  File<EMAIL_ADDRESS>line 950, in _bootstrap_inner
    self.run()
  File<EMAIL_ADDRESS>line 888, in run
    self._target(*self._args, **self._kwargs)
  File<EMAIL_ADDRESS>line 75, in _worker
    work_item = work_queue.get(block=True)

~~~~~~~~~~~~~~~~~~~ Stack of SyncWorker_0<PHONE_NUMBER>50144) ~~~~~~~~~~~~~~~~~~~~
  File<EMAIL_ADDRESS>line 908, in _bootstrap
    self._bootstrap_inner()
  File<EMAIL_ADDRESS>line 950, in _bootstrap_inner
    self.run()
  File<EMAIL_ADDRESS>line 888, in run
    self._target(*self._args, **self._kwargs)
  File<EMAIL_ADDRESS>line 75, in _worker
    work_item = work_queue.get(block=True)

+++++++++++++++++++++++++++++++++++ Timeout ++++++++++++++++++++++++++++++++++++
------------------------------------------------------------- Captured log call --------------------------------------------------------------
INFO     homeassistant.loader:loader.py:290 Loaded pyscript from custom_components.pyscript
WARNING  homeassistant.loader:loader.py:463 You are using a custom integration for pyscript which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant.
INFO     homeassistant.setup:setup.py:183 Setting up pyscript
INFO     homeassistant.setup:setup.py:232 Setup of domain pyscript took 0.0 seconds
INFO     pytest_homeassistant_custom_component.common:common.py:935 Writing data to core.restore_state: {'version': 1, 'key': 'core.restore_state', 'data': []}

―――――――――――――――――――――――――――――――――――――――――――――― ERROR at teardown of test_service_exists[pyloop] ――――――――――――――――――――――――――――――――――――――――――――――

loop = <_UnixSelectorEventLoop running=False closed=True debug=False>
hass_storage = {'core.restore_state': {'data': [], 'key': 'core.restore_state', 'version': 1}}
request = <SubRequest 'hass' for <Function test_service_exists[pyloop]>>

    @pytest.fixture
    def hass(loop, hass_storage, request):
        """Fixture to provide a test instance of Home Assistant."""

        def exc_handle(loop, context):
            """Handle exceptions by rethrowing them, which will fail the test."""
            # Most of these contexts will contain an exception, but not all.

        Handler key is the domain of the component that we want to set up.
        """
        try:
            integration = await loader.async_get_integration(self.hass, handler_key)
        except loader.IntegrationNotFound as err:
            _LOGGER.error("Cannot find integration %s", handler_key)
            raise data_entry_flow.UnknownHandler from err

        # Make sure requirements and dependencies of component are resolved
        await async_process_deps_reqs(self.hass, self._hass_config, integration)

        try:
            integration.get_platform("config_flow")
        except ImportError as err:
            _LOGGER.error(
                "Error occurred loading configuration flow for integration %s: %s",
                handler_key,
                err,
            )
            raise data_entry_flow.UnknownHandler

        handler = HANDLERS.get(handler_key)

        if handler is None:
>           raise data_entry_flow.UnknownHandler
E           homeassistant.data_entry_flow.UnknownHandler

venv/lib/python3.9/site-packages/homeassistant/config_entries.py:545: UnknownHandler
----------------------------------------------------------- Captured stderr setup ------------------------------------------------------------
DEBUG:asyncio:Using selector: KqueueSelector
------------------------------------------------------------ Captured stderr call ------------------------------------------------------------
INFO:homeassistant.loader:Loaded pyscript from custom_components.pyscript
WARNING:homeassistant.loader:You are using a custom integration for pyscript which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant.
INFO:homeassistant.setup:Setting up pyscript
DEBUG:homeassistant.helpers.restore_state:Not creating cache - no saved states found
INFO:homeassistant.setup:Setup of domain pyscript took 0.0 seconds
DEBUG:homeassistant.helpers.restore_state:Dumping states
INFO:pytest_homeassistant_custom_component.common:Writing data to core.restore_state: {'version': 1, 'key': 'core.restore_state', 'data': []}

+++++++++++++++++++++++++++++++++++ Timeout ++++++++++++++++++++++++++++++++++++

~~~~~~~~~~~~~~~~~~~ Stack of SyncWorker_1<PHONE_NUMBER>39648) ~~~~~~~~~~~~~~~~~~~~
  File<EMAIL_ADDRESS>line 908, in _bootstrap
    self._bootstrap_inner()
  File<EMAIL_ADDRESS>line 950, in _bootstrap_inner
    self.run()
  File<EMAIL_ADDRESS>line 888, in run
    self._target(*self._args, **self._kwargs)
  File<EMAIL_ADDRESS>line 75, in _worker
    work_item = work_queue.get(block=True)

~~~~~~~~~~~~~~~~~~~ Stack of SyncWorker_0<PHONE_NUMBER>50144) ~~~~~~~~~~~~~~~~~~~~
  File<EMAIL_ADDRESS>line 908, in _bootstrap
    self._bootstrap_inner()
  File<EMAIL_ADDRESS>line 950, in _bootstrap_inner
    self.run()
  File<EMAIL_ADDRESS>line 888, in run
    self._target(*self._args, **self._kwargs)
  File<EMAIL_ADDRESS>line 75, in _worker
    work_item = work_queue.get(block=True)

+++++++++++++++++++++++++++++++++++ Timeout ++++++++++++++++++++++++++++++++++++
------------------------------------------------------------- Captured log call --------------------------------------------------------------
INFO     homeassistant.loader:loader.py:290 Loaded pyscript from custom_components.pyscript
WARNING  homeassistant.loader:loader.py:463 You are using a custom integration for pyscript which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant.
INFO     homeassistant.setup:setup.py:183 Setting up pyscript
INFO     homeassistant.setup:setup.py:232 Setup of domain pyscript took 0.0 seconds
INFO     pytest_homeassistant_custom_component.common:common.py:935 Writing data to core.restore_state: {'version': 1, 'key': 'core.restore_state', 'data': []}
---------------------------------------------------------- Captured stderr teardown ----------------------------------------------------------
DEBUG:homeassistant.core:Bus:Handling <Event homeassistant_stop[L]>
ERROR:homeassistant:Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/Users/raman/projects/pyscript/venv/lib/python3.9/site-packages/homeassistant/data_entry_flow.py", line 117, in async_init
    flow = await self.async_create_flow(handler, context=context, data=data)
  File "/Users/raman/projects/pyscript/venv/lib/python3.9/site-packages/homeassistant/config_entries.py", line 545, in async_create_flow
    raise data_entry_flow.UnknownHandler
homeassistant.data_entry_flow.UnknownHandler
DEBUG:homeassistant.helpers.restore_state:Dumping states
DEBUG:homeassistant.core:Waited 60 seconds for task: <Task pending name='Task-3' coro=<_async_setup_component() running at /Users/raman/projects/pyscript/venv/lib/python3.9/site-packages/homeassistant/setup.py:248> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x10a21d490>()]> cb=[<TaskWakeupMethWrapper object at 0x10a200910>()]>
WARNING:homeassistant.core:Timed out waiting for shutdown stage 1 to complete, the shutdown will continue
DEBUG:homeassistant.core:Bus:Handling <Event homeassistant_final_write[L]>
INFO:pytest_homeassistant_custom_component.common:Writing data to core.restore_state: {'version': 1, 'key': 'core.restore_state', 'data': []}
DEBUG:homeassistant.core:Bus:Handling <Event homeassistant_close[L]>
----------------------------------------------------------- Captured log teardown ------------------------------------------------------------
ERROR    homeassistant:runner.py:115 Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/Users/raman/projects/pyscript/venv/lib/python3.9/site-packages/homeassistant/data_entry_flow.py", line 117, in async_init
    flow = await self.async_create_flow(handler, context=context, data=data)
  File "/Users/raman/projects/pyscript/venv/lib/python3.9/site-packages/homeassistant/config_entries.py", line 545, in async_create_flow
    raise data_entry_flow.UnknownHandler
homeassistant.data_entry_flow.UnknownHandler
WARNING  homeassistant.core:core.py:536 Timed out waiting for shutdown stage 1 to complete, the shutdown will continue
INFO     pytest_homeassistant_custom_component.common:common.py:935 Writing data to core.restore_state: {'version': 1, 'key': 'core.restore_state', 'data': []}
                                                                                100% ██████████
========================================================== short test summary info ===========================================================
FAILED tests/test_apps_modules.py::test_service_exists[pyloop] - Failed: Timeout >2.0s
FAILED tests/test_apps_modules.py::test_service_exists[pyloop] - homeassistant.data_entry_flow.UnknownHandler

Results (123.16s):
       1 failed
         - tests/test_apps_modules.py:20 test_service_exists[pyloop]
       1 error

Oh, just import the config flow platform before running the test.

This only happens if you don't use the normal way of instantiating a config flow btw.

I tried instantiating a config flow by running hass.config_entries.flow.async_init but hit the same error. Am I missing something?

Thanks @balloob and @raman325!  I pushed a change that patches just custom_components.pyscript.global_ctx.open with MockOpen() instead of builtins.open, and the test then passes. As you suggest, there's probably a more robust fix.
Anyhow, I'll go ahead and pin pytest-homeassistant-custom-component to 0.0.20, and update tests/requirements_test.txt.

Ah nevermind, looks like @craigbarratt figured it out. My guess is that your original patch was preventing HA from loading the integration/platforms correctly.

Yes, that makes sense.  It was probably the root cause.  Sorry to bother you.

All good glad you got it resolved

Home Assistant needs to load manifests etc. Generally, avoid mocking builtins and do what you did here instead 👍


Do not cache share links

You can't just cache the share links, that will result into every blog post having the same URL.
Just use partial on the getShareLink.html. So now each post will show the correct URL to share the post via social media or alike.

False. There is a bug in the logic of the partial, that is true. But we CAN and will cache those links. A share link must be cacheable for a specific page. Check the $options dict. page here might refer to the outer list page a post is listed on, which is what ruins the cache if you implement share links on list pages. What this needs is $context (defined in the first line) instead of page in the options and then they (should be/are) individual caches per post object (if $context doesn't walk into the same trap). I'll need to check that though, it might be that I will inline the sharelink generation.

Something along the lines of this, not tested yet:
https://github.com/theNewDynamic/gohugo-theme-ananke/pull/763

I am going to merge this now. I opened an issue for going to look at caching after the release.
https://github.com/theNewDynamic/gohugo-theme-ananke/issues/764


Feat/nft training update

I updated the old NFT training to last ligo and taqueria versions

Hi @timothymcmackin
I think I have finished to integrate the last changes to all parts now
Could you please check if everything is fine now ?
Thank you

Ok, your updates look good. Let's move forward.

@LiYuxin44 Do you approve the PR too ?


Use downward API for topology information

Blocked by https://github.com/kubernetes/enhancements/pull/4747.
For network routing, we need topology information to be available at each pod. There's no standard mechanism to do that currently, so for GCP we are doing a metadata service lookup in https://github.com/istio/istio/pull/53821#pullrequestreview-2425416071. Once k8s adds the new API, we should replace it with downward API.

CC @howardjohn @igsong

Not only gcp, currently the region and zone label are also get from node


P2AffinesAdd crashes when executing in alphine docker image

I am getting crash on Cgo execution of P2AffinesAdd. Wonder if you guys have any advice?
Signature and verification are fine so I guess it only affects certain APIs.
Docker base: golang:1.18-alpine
Crash stacktrace:
fatal: morestack on g0
SIGTRAP: trace trap
PC=0x48c2e2 m=8 sigcode=128
signal arrived during cgo execution

goroutine 49 [syscall]:
runtime.cgocall(0xf6cda0, 0xc000de3480)
        /usr/local/go/src/runtime/cgocall.go:157 +0x5c fp=0xc000de3458 sp=0xc000de3420 pc=0x42481c
github.com/photon-storage/blst/bindings/go._Cfunc_blst_p2s_add(0xc000196a20, 0xc003690000, 0x400)
        _cgo_gotypes.go:1115 +0x45 fp=0xc000de3480 sp=0xc000de3458 pc=0x9bafa5
github.com/photon-storage/blst/bindings/go.P2AffinesAdd.func2(0xc000300400?, 0xc000de3508, 0x9baeec?, 0x1543d90)
        /go/pkg/mod/github.com/photon-storage/blst@v1.0.1/bindings/go/blst.go:2599 +0xf3 fp=0xc000de34c8 sp=0xc000de3480 pc=0x9c6833
github.com/photon-storage/blst/bindings/go.P2AffinesAdd({0xc003690000, 0x400, 0x400}, {0x0?, 0x4384d3?, 0x4000?})
        /go/pkg/mod/github.com/photon-storage/blst@v1.0.1/bindings/go/blst.go:2599 +0x65 fp=0xc000de3508 sp=0xc000de34c8 pc=0x9c66e5
github.com/photon-storage/go-photon/crypto/por.(*Verifier).VerifyBlocks(0xc0012e2770, {0xc0034c0000, 0x400, 0xf5394a60a035baa9?},

Is it 100% reproducible or does it fail occasionally? Since you mention it explicitly, is it actually alpine-specific, or is it so you simply happen to use it? Since you mention it explicitly, were previous go versions working, or is 1.18 simply the first go version you compile blst with?
It's not impossible to imagine that the problem is due to an uncertainty introduced by _cgoCheckPointer := func(...interface{}) {} in the function in question. Rationale behind it was that since a) we know that the pointers point to pure data and b) we know that the C side doesn't modify pointers, the pointer check is safe to suppress. The check was causing problems but they were deemed to be false positives. But this assessment was done at specific point in time, and might be not entirely correct by now. Hence the questions.
Is caller's code publicly available? I've found go-photon on github, but there is no VerifyBlocks there...

It is 100% reproducible and always fails. I have tried the same docker image in both EC2 host and my macbook, they both fail. I have also tried docker image based on "golang:1.18.8-bullseye", it is fine.
The crashing docker image is based on "golang:1.18-alpine" with "apk --no-cache --update add gcc musl-dev build-base bash git"

Our repo is not ready to open to public yet as it is still very rough and WIP. I paste the function below. Let me know if you need any further information.
// VerifyBlocks used to batch verify POR proofs of multiple blocks
func (v *Verifier) VerifyBlocks(sigs, blks [][]byte, indexes []uint32) error {
    blkSize := len(blks)
    if len(sigs) != blkSize || len(indexes) != blkSize {
        return ErrBlockSizeMismatch
    }

    mus := make([]*big.Int, v.spb)
    for j := range mus {
        mus[j] = big.NewInt(0)
    }
    for i := range blks {
        for j := uint32(0); j < v.spb; j++ {
            s := BlockSector(blks[i], j)
            mus[j] = new(big.Int).Add(mus[j], s)
        }
    }

    sigmas := make([]*blst.P2Affine, blkSize)
    for i, sig := range sigs {
        sigmas[i] = new(blst.P2Affine).Uncompress(sig)
    }
    sgm := blst.P2AffinesAdd(sigmas)

    var aggs []*blst.P2Affine
    for _, index := range indexes {
        aggs = append(aggs, BlockTag(v.oid, index).ToAffine())
    }
    for i, u := range v.us {
        aggs = append(aggs, blst.P2Mult(u, ReduceToScalar(mus[i])))
    }

    p1, err := v.pk.GetRaw()
    if err != nil {
        return err
    }

    a := blst.Fp12MillerLoop(sgm.ToAffine(), blst.P1Generator().ToAffine())
    b := blst.Fp12MillerLoop(blst.P2AffinesAdd(aggs).ToAffine(), p1)
    if !blst.Fp12FinalVerify(a, b) {
        return ErrVerificationFailure
    }
    return nil
}

I have also tried docker image based on "golang:1.18.8-bullseye", it is fine.

So it sounds like a musl vs glibc thing... Not that it totally explains the failure on alpine, but at least there is some hint... However, I see no reason for why you wouldn't be able to restructure your code to not rely on an array of pointers. And it would be more efficient too. So instead of sigmas := make([]*blst.P2Affine, blkSize) do sigmas := make([]blst.P1Affine, blkSize), in which case you would call sgm := blst.P2Affines(sigmas).Add(). As for the second Miller loop call. Its input appears to be a sum of points and result of a multi-scalar multiplication. You would be better off using the corresponding method for the latter. Note that there are Add and Mult for both P2Affines and P2s. Latter perform efficient conversions to affine so you don't have to do it one by one... As for scalars. How wide are they? Either way, the best option is to lay them down as flat array of bytes. And there also will be a more efficient multi-Miller-loop interface...

How many points are you adding?
When going over 1024 G2 points or 2048 G1 points I have a crash as well (https://github.com/mratsim/nim-blscurve/pull/1#issue-1419851542)
It appears here: https://github.com/supranational/blst/blob/6382d67/src/bulk_addition.c#L148-L149
The C-API seems to have a double pointer/array indirection, could it be that the code assumes the points are in batches of 1024/2048?

Just in case for reference, what's up with internal batching. The algorithm in question requires temporary storage, by where to get is from? blst doesn't do heap allocations, so we take it from stack. But it's not safe to assume that arbitrary amount of stack is available, so we iterate in smaller steps. On the other hand we want to amortize inversion costs across larger amount of points. So where to draw the line? Somewhere within 1-2-3% performance penalty...

Somewhere within 1-2-3% performance penalty...

Though I think that the evaluation was done with slower inversion subroutine... In other words it might turn out that reducing the internal batch size could be acceptable...

Long reply, a bit off-topic regarding OP problem then.

As for the 2nd question. No, C-API doesn't expect specific batch sizes. But it does claim temporary storage on the stack to accommodate the internal batching, 288KB. In Go it's not a problem, because C subroutine is executed on own stack, one with "normal" limit, where 288KB are readily available. But can you be hitting your stack ceiling, @mratsim?

It's unlikely, I'm not using MSVC which limits stack to 1MB,  my ulimit -a shows 8MB

Just in case for reference, what's up with internal batching. The algorithm in question requires temporary storage, by where to get is from? blst doesn't do heap allocations, so we take it from stack. But it's not safe to assume that arbitrary amount of stack is available, so we iterate in smaller steps. On the other hand we want to amortize inversion costs across larger amount of points. So where to draw the line? Somewhere within 1-2-3% performance penalty...

I don't think anyone has issue with internal batching, in particular quoting myself in my own implementation: https://github.com/mratsim/constantine/blob/53a5729/constantine/math/elliptic/ec_shortweierstrass_batch_ops.nim#L269-L286

We chunk the addition to limit memory usage
especially as we allocate on the stack.
From experience in high-performance computing,
here are the constraints we want to optimize for

MSVC limits stack to 1MB by default, we want to use a fraction of that.
We want to use a large fraction of L2 cache, but not more.
We want to use a large fraction of the memory addressable by the TLB.
We optimize for hyperthreading with 2 sibling threads (Xeon Phi hyperthreads have 4 siblings).
Meaning we want to use less than half the L2 cache so that if run on siblings threads (same physical core),
the chunks don't evict each other.

Hardware:

a Raspberry Pi 4 (2019, Cortex A72) has 1MB L2 cache size
Intel Ice Lake (2019, Core 11XXX) and AMD Zen 2 (2019, Ryzen 3XXX) have 512kB L2 cache size

After one chunk is processed we are well within all 64-bit CPU L2 cache bounds
as we halve after each chunk.

The part that is confusing to me is the function signature / double pointer indirection, why
void blst_p2s_add(blst_p2 *ret, const blst_p2_affine *const points[], size_t npoints);

over
void blst_p2s_add(blst_p2 *ret, const blst_p2_affine const points[], size_t npoints);

The part that is confusing to me is the function signature / double pointer indirection, why

For flexibility. Note that it traverses the array of pointers till NULL, after which it treats the last pointer as an array of points. So that if you have an array of points you can simply blst_p2_affine *temp[2] = { array, NULL} and pass temp. But if the subroutine accepted array of points and you had an array of pointers, then you'd have to allocate an array and copy points...

I don't think anyone has issue with internal batching

The remark was more for the OP :-) As already said "just in case" :-)

my ulimit -a shows 8MB

Yes, but the main thread can start threads with smaller stacks... I'm not saying that it does, but it's the only thing I can think of give the circumstances. Can you collect the failing address and compare it to the current stack pointer? Just in case, I can pass more points to the subroutines in question...


ability to reload a recipe, for use in shef

while iterating in shef one must exit out of recipe mode, back in, and include the recipe again to pick up the changes - this adds a "reload_recipe" function to alleviate all those steps
(note, I'm on the approved contributors list)

Hey, thanks, is there a correspond issue open for this in the tracker at http://tickets.opscode.com?
http://wiki.opscode.com/display/chef/How+to+Contribute

here we go:
http://tickets.opscode.com/browse/CHEF-2922

going to close in favor of this revision with changes requested in CHEF-2922:  https://github.com/opscode/chef/pull/237


OAuth AuthorizationGrantCache and the database layer have inconsistent keys for the JWT token cache entries.

Describe the issue:
JWT access token cache entries are stored using the JWT access token in the AuthorizationGrantCache, while the JTI (JSON Web Token ID) is used in the persistence database layer due to the large size of JWT tokens. This inconsistency does not affect opaque tokens since their size is not an issue.

The current codebase addresses this by specifically handling JWT access tokens when accessing the cache. However, the ideal solution should eliminate the inconsistency between the cache keys and the persistence layer keys for the JWT token entries. Or else the inconsistency should be wrapped at the cache layer.
To address a customer issue from the same root cause, the following fixes have been applied as patches within the patch scope which is not the ideal solution.

https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/2503
https://github.com/wso2-support/identity-inbound-auth-oauth/pull/1613

How to reproduce:
This is an architectural level issue and there is no user experience level defect for the moment.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

@indeewari can we close this?

@Thumimku

@indeewari , seems there is a public PR involved, and its fixing refresh grant right? what else we have to fix here .. other grants?


[Outbreak] fixed button macro typo

Changes / Comments

fixed strength typo

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

#6225 covers this, so closing as redundant


tour: Images exercise has misguidance

Context: https://tour.golang.org/methods/25
It is not obvious that the intent is to recreate the implementation while using existing imports.
meaning, I (and come colleagues going through this together) started building EVERYTHING from scratch C-style with arrays and such, and of course getting stuck on the way.
It would be helpful to simply add a comment that says that is not the right direction.

 It seems like it's been written by several people and got abandoned at some point.


feat(orchestrator): initialize shuttle-orchestrator as a library

Description of change
This PR simply creates a new library in the workspace called shuttle-orchestrator.
Let me know if anything else is needed.
How has this been tested? (if applicable)
cargo build

I added a dependency to gateway in e439eafa57232ee142889f8b749e7c8150adf64b since we already going to need it. Let me know if it looks good or I should revert it.


DenseNet-121 is faster than CondenseNet-74 (C=G=4) on GTX 1080 Ti

I compared the forward pass speed of the larger ImageNet model with DenseNet-121 and the latter actually works faster. After benchmarking my guess is that CondenseConv layer is the cause of the slowdown due to memory transfers in ShuffleLayer and torch.index_select.
@ShichenLiu can you comment on this, did you get better performance compared to DenseNet-121 in your experiments?

Hi, thanks for your comment. Although memory transfer does slowdown a little bit, the main reason is the group convolution employed in our model, which could not meet its theoretical speed in GPU currently in PyTorch. In real mobile devices both problems are alleviated much better.

GPUs tend to be memory-bound rather than compute-bound, in particular, for small models that require additional memory transfers such as ShuffleNets and CondenseNets. On mobile devices, embedded systems, etc. the ratio between compute (in FLOPS) and memory bandwidth is very different though: convnets tend to be compute-bound on such platforms. If you did the same comparison on such a platform, you would find that a CondenseNet is much faster than a DenseNet (see Table 5 of the paper for actual timing results on an ARM processor).

Thanks for clarification. I already suspected that is the reason after I measured time spent in bottleneck 1x1 layer and grouped 3x3 layer. Forward pass spends twice as much time in 1x1 compared to 3x3.
I think there is a way to avoid additional memory transfers on GPUs if CUDNN implementation allows you to specify custom feature maps ordering after grouped convolution. I don't know if this feature is available in CUDNN but if I am correct then you could remove all feature shuffling ops.


Updated documentation for modifying machine pools

Updated the Terraform documentation to show how to create machine pools, change the replica count, and enable autoscaling.

/retest

/approve
/lgtm


Add Caduceus to supported networks in redstone docs

We want to display it here: https://docs.redstone.finance/docs/smart-contract-devs/supported-chains
After addition please release the 0.1.6 version of the docs, so that the change is visible fo everyone by default

Done


Implement store info reporting and recover plan execution logic for o…

What problem does this PR solve?
Issue Number: close #10483
Problem Summary:
Support online recovery for Raft majority failure
What is changed and how it works?
Proposal: Design Doc
What's Changed:

Several unsafe recover related PD commands are added.  They are: unsafe remove-failed-store <store id>,<store id>..., unsafe remove-failed-store show and unsafe remove-failed-store history
After PD received unsafe recovery request, it notifies all stores through StoreHeartbeatResponse to send a detailed report (local region and raft info) in the next heartbeat.
Once PD have all the reports, it generates a recover plan and sends them to each individual store (using StoreHeartbeatResponse as well) to execute.
The recover is considered finished if all the target states are met (by comparing following store reports with recovery plan).

Related changes

PR to update pingcap/docs/pingcap/docs-cn: WIP
PR to update pingcap/tidb-ansible: WIP
Need to cherry-pick to the release branch: Not applicable

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression:  Not applicable

Consumes more CPU:  Not applicable
Consumes more MEM: Not applicable

Breaking backward compatibility: Not applicable

Release note
Support online recovery for Raft majority failure

/merge

/test

/merge


NODEjs error

node in vscodium-server don't start on CentOS 7.9, while it can work which in vscode-server.
/lib64/libc.so.6: version GLIBC_2.25' not found /lib64/libstdc++.so.6: version CXXABI_1.3.9' not found
/lib64/libstdc++.so.6: version `GLIBCXX_3.4.20' not found

Hi, this is an vscodium issue, see https://github.com/VSCodium/vscodium/issues/1634, you can workaround by following this comment https://github.com/VSCodium/vscodium/issues/1634#issuecomment-1806526140


Add compile warning if player sprite sheet isn't set

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

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

Add compile warning if player sprite sheet is set to None.

What is the current behavior? (You can also link to an open issue here)

Compile fails with TypeError: Cannot read property 'plugin' of undefined when no sprite sheet has been selected for the Player. See #447

What is the new behavior (if this is a feature change)?

A more clear Player Sprite Sheet isn't set. Please, make sure to select a Sprite Sheet in the Project editor. error is shown on compile.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

No

Other information:

This doesn't solve the core issue of #447 (how did the player sprite sheet ended up being None?). But it should help users to fix the issue by themselves if they get there.

Thanks @pau-tomas I can reproduce the error by just deleting the currently active player sprite sheet png file and building. This error is much better than the random plugin one that appears


Fixed bug - user could not delete their own comment

Closes #178
A user can now delete their own comments on others' posts, without needing admin access.

Great!

Thank you!


Add management_events to COS bucket creation code in SLZ

In https://github.com/IBM-Cloud/terraform-provider-ibm/pull/5277 they are adding support for management_events in the AT config.
Once added, we will want to update code in two places:

https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/blob/a2a4565c0c265db0fc36b8e418b7ca367a08dacb/cos.tf#L140-L144
https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/blob/a2a4565c0c265db0fc36b8e418b7ca367a08dacb/variables.tf#L540-L544

The provider PR was merged today, so this should be ready to go in 1.67.0

Starting on update.


[Localization] Assistance with translation

String Resource ID
0; "TitleAdlsGen2BlobSnapshots"

Source String
Enable ADLS Gen2 snapshots (Preview)

Filename
loc\en\src\Standalone\resources\Shell.json

Question
Please confirm this is still temporary and should NOT be translated.

Correct.


Failed to launch the browser process!

Steps to reproduce

Puppeteer version: 3.0.4
Platform / OS version:MAC OS X / 10.9.5
Node.js version: v12.11.0
Npm version : v6.11.3

What steps will reproduce the problem?
var opt = {};
  if(os == "linux") {
      opt.args = [
          '--no-sandbox',
          '--disable-setuid-sandbox',
          '--disable-gpu',
          '--disable-dev-shm-usage',
          '--no-first-run',
          '--no-zygote'
      ];
  }
  opt.headless = true;
  var browser = await puppeteer.launch(opt)

What happens instead?
UnhandledPromiseRejectionWarning: Error: Failed to launch the browser process!
TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/master/docs/troubleshooting.md
at onClose   node_modules/puppeteer/lib/Launcher.js:547:20)
at Interface. node_modules/puppeteer/lib/Launcher.js:537:65)
at Interface.emit (events.js:215:7)
at Interface.close (readline.js:403:8)
at Socket.onend (readline.js:180:10)
at Socket.emit (events.js:215:7)
at endReadableNT (_stream_readable.js:1183:12)
at processTicksAndRejections (internal/process/task_queues.js:80:21)

I'm having this issue on a Ubuntu box at the moment. Ran fine for months but I did install some new packages and updates via apt earlier so it could be a number of things.


[AUT-179] Fix validation logic and ensure we never get stuck in a connection that never times out

Description of Change

Create a proper app validator class that is slightly more lenient about whether and app can be scanned or not

Apps that 302/303 when requesting their baseUrl were causing issues

Create a standardized Requests Session that can be used throughout the app
Standardized session also ensures we eventually timeout a connection if it hangs too long

Fixes
Internal bugfix revision, does not fix any active GH issues
Did you test your changes?

[x] Yes

Added in tests for the app_validator and csrt_session
Reviewers
@anshumanbh

Quick note: Had to make a quick exception on the Github actions test suite. sslyze has not released a version with cryptography bumped past 3.3, so we have to ignore the warning for now.
Awaiting https://github.com/nabla-c0d3/sslyze/issues/467 to be resolved before we can remove the ignore line in the Github actions.

Quick note: Had to make a quick exception on the Github actions test suite. sslyze has not released a version with cryptography bumped past 3.3, so we have to ignore the warning for now.
Awaiting https://github.com/nabla-c0d3/sslyze/issues/467 to be resolved before we can remove the ignore line in the Github actions.


Project build failed after adding react-jinke-music-player

Version
4.11.2
Description
I am getting below error in .rc-slider which is being used by this library.

Steps
I am using below css loader.
/**
 * MiniCssExtractPlugin.loader,
            {
              loader: 'css-loader',
              options: {
                modules: true,
                localIdentName: '[local]'
              }
            },
            'sass-loader',
 */

Steps to Reproduce
Expected Behavior
Current Behavior

System information
Comments

I think this is your webpack config file problem, It has nothing to do with this player
please search by google to resolve it
Maybe try to exclude ?
{
      test: /\.css$/,
      exclude: /node_modules/,
 }

I'm having the same issue, using razzle for SSR - did you find a solution?

If you take a look at the webpack config for rc-slider, you'll see that it doesn't have a rule to compile CSS yet it imports a css file. The build script compiles the dist via webpack, so my guess is the problem stems from there.


Extending the glossary

Added align, broadcast, merge, concatenate and combine to terminology.
-> Closes #3355

Actually on top of examples maybe some simple text depictions could be useful?
Something like this for aligning:
Values: [ A A A ] + [ B B ] -> [ A B A B A ]
Index:.  [ 1 3 5 ] + [ 2 4 ] -> [ 1 2 3 4 5 ]
But just throwing ideas around

@headtr1ck I have done the changes required, please review

Any reason why you removed the examples?
I didn't follow the review process if this was requested by someone.

Any reason why you removed the examples? I didn't follow the review process if this was requested by someone.

I was unable to build the examples while doing make html, but now I'm able to. I'll add them again.

Thanks @harshitha1201


Game: at_max_dist, at_min_dist, between_dist

TaelinArena.Game
Depends on #161 being merged

[x] at_max_dist
[x] at_min_dist
[x] between_dist

Merged on Moonad/PR9


I can't preview my dot file

Issue Type: Feature Request
Even after installing the Graphviz(dot) language extension I was unable to get the "Open Preview to the Side" on my VS code (Windows 11).I'll be very thankful if you help me with that :)
Extension version: 0.0.1
VS Code version: Code 1.68.1 (30d9c6cd9483b2cc586687151bcbcd635f373630, 2022-06-14T12:48:58.283Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

I have the same problem.

I also have the same problem

I do this and it´s working now:

Installed in my wsl or respective

sudo apt-get install graphviz

I run

which dot

That response

/usr/bin/dot

Then in vscode I went to file > preferences > settings > and write dot in the search box
In config called Dot Path I put /usr/bin/dot
Then I open the dot file and press ctrl+shift+p and search Graphviz: Open preview to the side
Next press enter and ....


[WIP] refactor: make read_batch object safe in WalManager

Which issue does this PR close?
Part of #72
Rationale for this change
Make WalManager object safe, so we can use Arc<dyn WalManager> in other place.
What changes are included in this PR?

Remove LogReader trait.
The item of BlockingLogIterator is Vec<u8> now.
Real value decoding will do in BatchLogIteratorAdapter.
Move LogKeyEncoder/LogValueEncoder etc from wal::rocks_impl::encoding to wal::kv_encoder, which are both used by rocks_impl and table_kv_impl.

Are there any user-facing changes?
No
How does this change test
The refactor is tested by the compiler.

@ShiKaiWi @waynexia PTAL.


Support @mixin directive on fragments

It would be useful to be able to include a mixin class on a generated fragment type, so it isn't necessary to add the directive redundantly in all of the fields that include that fragment.
Something like:
fragment FooFields on Foo @mixin(from: ".mixins", import: "Bar") {
  ...
}

query {
  foo {
    ...FooFields
  }
}

ending up with:
class Foo(FooFields):
    ...

class FooFields(BaseModel, Bar):
    ...

As far as I can tell this is not currently supported.  Is such a thing possible?

Hey, we don't support it at the moment, but I don't see any objections to adding it.

How do I get it on the roadmap?  Or is it my job to submit the diff?  :-)

You don't need to link it anywhere else, this issue is enough. And of course, if you want, you can submit pr :)


Error opening database that has a space character in it.

I tried to access a database from a software we use in our company. The manufacturer called that database Lodgit Database.lxdb. I wasn't able to access it with the connector and another Freeware Software.  I got the error:

After some tryouts I've found out that it's possible to access a newly created database called Test. So I tried to open the database with the following script:
// Connection infos
$host = '<IP_ADDRESS>';
$port = 4430;
$user = 'admin';
$pass = 'admin';
$database = 'Lodgit Test Database.lxdb';
//$database = 'Test';
// Setup Connection
$cdb = new cubeSQLServer;
$cdb->connect_database($host, $port, $user, $pass, $database, 25);
$selectdb = $cdb->execute('USE DATABASE "'.$database.'";');
$tables = $cdb->select("SHOW TABLES");

And it worked. It also worked for the Test database.
So I wanted to suggest to change the following line in the connect_database  method of the cubeSQLServer object.
FROM
$this->db->send_statement($kCOMMAND_EXECUTE, "USE DATABASE $database;");
TO
$this->db->send_statement($kCOMMAND_EXECUTE, 'USE DATABASE "'.$database.'";');
It will send that query always like that: USE DATABASE "database name with spaces"; to the server.
@marcobambini If you think that will cause no conflicts I will make up a new pull request. Just let me know.
Thanks a lot for the connector! Now I'm able to access our live-database using PHP! 👍

Hi @rsteinmann yes you can safely quote database name.
Thanks a lot for your invaluable contribution!

Hi @marcobambini that's good.
You are very welcome. I'm glad that my work is useful! That connector saved me a lot of time since I have no experience in network programming. So thank you for your work!


JS error when Cyclops tmpl file has not loaded

I have noticed intermittent issues (often when refreshing a page) where a JS error occurs because the cyclops tmpl has not yet loaded. So somehow the $.holdReady(true); set whilst the tmpl file loads is not preventing the view models from binding.

Error: Unable to process binding "component: function (){return l}" Message: Component 'pane': Cannot find element with ID cyclops.pane

Has anyone else seen this and figured out a solution?
My workaround is based on: http://swizec.com/blog/how-to-properly-wait-for-dom-elements-to-show-up-in-modern-browsers/swizec/6663

I know this is a silly question but just checking is everything inside a jquery document ready function? can you double check?

Also if you have a constant way to reproduce this that would be help I have never seen this...

Not a silly question and yes. But I'll triple check that some third party code is not doing anything weird that is hard to spot. I'll also try to come up with a reproducible test case. Knowing that you have not seen it makes me think there is something else going on.
Thanks

I'm going to close this for now but If you see it again and or get a good repro please reopen.


🚀 [New Contribution]: A complete guide for Markdown files

Describe Your Contribution Here
This is a blog on guide to Markdown files. Markdown files are an important part to write docs for any open source projects.
I want to add it to this repo as part of Showwcase Hacktoberfest contribution.
Show link:- https://www.showwcase.com/show/17620/a-complete-guide-for-markdown-files

Sure, are you republishing it?

Sure, are you republishing it?

Yes

Isn't it gonna be valid?

Isn't it gonna be valid?

The ground rule is to publish as a first time original one as show. How about you plan another one?

Do you mean writing a new blog?

Do you mean writing a new blog?

That's right.

@atapas So, can I share this: https://www.showwcase.com/show/17617/demystifying-hacktoberfest

This is the original one. Just posted today morning.

Certainly, create a PR

Certainly, create a PR

For the Markdown one or this one?

For the one you published today... Check the readme on how to mark your entry.

For the one you published today... Check the readme on how to mark your entry.

Okay

Completed by #19


feat(cli): add rivet config data-path command to inspect internal data

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#541  👈
#540
#539
#537
#536
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


Non-numpy array support

In this PR I worked on the support for non-numpy array data.
Motivation
Originally, jax arrays could be used to initialize a pynapple Ts object. However, this would eventually create unexpected behaviors. For instance,
import pynapple as nap
import jax.numpy as jnp

tsd = nap.Tsd(t=jnp.array([1,2,3]), d=jnp.array([1,2,3]))
ep = nap.IntervalSet(1.5,2.5)
print(type(tsd.restrict(ep)))

would print <class 'pynapple.core.time_series.Ts'>, converting to Ts.
Design Choices

Convert to NumPy: Pynapple relies on numba for efficiency, many array-like objects in deep learning frameworks (jax, torch...) cannot be handled by numba (which compiles for CPU). It is impossible to rely on numba and operate on jax arrays . For this reason I opted for converting array-like objects to numpy raising a UserWarning.

Suppress UserWarnings: Added a configuration class that can be used to suppress these warnings. This is intended for jax users that can silently convert between numpy and jax arrays.

Testing:  created a MockArray class for testing. Included tests for desired behavior.

I removed the dependency on jax that was creating issues with python 3.8


Add topics for #media-processing channel

Relevant Issues
Closes #1052
Description

Added topics for the .topic command for the recently created #media-processing channel
Did you:

[x] Join the Python Discord Community?
[x] Read all the comments in this template?
[x] Ensure there is an issue open, or link relevant discord discussions?
[x] Read and agree to the contributing guidelines?

Not sure if it helps, but here are a few additional ideas:

Any interesting progress to share?
Any cool trick or optimization you are proud of?
What is your favorite media processing related algorithm?
What is the worst outcome for your generation or processing?
What is the most surprising result you got?
What is your most advanced media processing related achievement?
Do you use a lot of math in your media processing related projects?
Can a computer truly generate art?
Can a computer truly understand art?
What is a good project to get started?

Oh wow, those are actually pretty awesome, @Recursive-Error , thank you so much! However, I would tweak some of them but I don't really have the time right now to write out exactly what I think needs to be tweaked, but I could do it later today if you are interested.

The question of "preferred library for media processing" is too broad to be conducive to discussion. Same with "what is media processing."
I recommend you accept most if not all of recursive_error's suggested topics.
I especially like the suggestions of whether computers can generate or understand art.


merge from arlac77/npm-package-template-esm-only

package.json

fix(engines): add >=12.14.1 remove >=12.14.0 (engines.node)

.travis.yml

chore(travis): (jobs.include)
chore(travis): (jobs.include.node_js)
chore(travis): (script)

README.md

docs(README): update from template

:tada: This PR is included in version 2.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Baha California Sur - Full addresses from INEGI

First test of an entire mexico import

Solved by https://github.com/openaddresses/openaddresses/pull/2743


Failure trying SSHFS. Alternatives?

I have a MacBookPro (MBP) with a folder /opt/tlrepo which I have "mouned" on GNU/Linux Mint with
sshfs utente@MBP:/opt/tlrepo /mnt/tlrepo
and I can use it [BTW, /opt/tlrepo is a local mirror of TeXLive (TL) distribution and I can install TL on GNU/linux using the same repo I have used on MBP.]
Trying the same on WSL fails:
fuse: device not found, try 'modprobe fuse' first
I have seen that this is not yet supported by WSL and that many users want that.
Meanwhile, How I can "mount" that folder in WSL? Which alternatives? [BTW, I have to copy tlrepo on an USB key to install TL on WSL. ALL this works just fine, but to maintain the installation it would be very useful to have only one tlrepo.]
Here is Win 10 Pro 64b 10.0.16299.371 and WSL:
$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.4 LTS
Release:        16.04
Codename:       xenial

(SSHFS package installs on WSL without issues using sudo apt install...)
Thanks.

SSHFS uses FUSE, which is not supported atm.

@fpqc wrote:

SSHFS uses FUSE, which is not supported atm.

Yes, I know and suggest, as others, to add that... but I asked also for alternatives...
How can I do that job ("mounting" a utente@MBP:/opt/tlrepo) on WSL? Notice that
ssh utente@MBP
(and scp to/from MBP) already works on WSL..
Thanks.

@angelog0 alternative: http://www.secfs.net/winfsp/blog/files/winfsp-2017.html
@angelog0 scp is a completely different thing that sshfs (which is on top of FUSE). So what you're basically asking in that ticket (or should), is to please implement FUSE related syscalls.

@christianparpart wrote:

alternative: http://www.secfs.net/winfsp/blog/files/winfsp-2017.html

I don't understand how to use it in relation with WSL.
Do you mean that after I have installed it I can use in WSL
sshfs utente@MBP:/opt/tlrepo /mnt/tlrepo
?
Do you mean that WinFsp will work with sshfs from WSL package? I am reading this page WinFsp and it seems that if I run
memfs-x64 -u \memfs64\share2 -m *
it will create a shared Y: drive. But with which machine?
Please, may you elaborate? How can I "mount" the MBP folder utente@MBP:/opt/tlrepo in WSL /mnt/tlrepo folder? Maybe you need to re-read my OP.
Thanks.

It's not just FUSE syscalls. FUSE needs to be added at the Windows driver level

@fpqc wrote:

FUSE needs to be added at the Windows driver level

So, I download WinFsp MSI file, install it, install the sshfs WSL package and than I can use sshfs utente@MBP:/opt/tlrepo /mnt/tlrepo in WSL?
Please, clarify. Thanks.

Ok, with winfsp and sshfs-win I managed to mount the Mac Book Pro (MBP) repo on Windows as drive Z: and then I mounted this drive to WSL (sudo mount -t drvfs z: /mnt/z) but ALL THIS didn't solve my problem: using a TeXLive (TL) repo on  MBP to install and maintain it on other PCs (Windows, GNU/Linux) in the LAN.
Even if I can read and write the /mnt/z, when I try to update the TL installation, it fails because it does not understand some /mnt/z/tlrepo files. May be they are symlinks on MBP and GNU/Linux but /mnt/z sees them as simple text files. It is a pity.
Let's see if WSL will acquire the FUSE support...
Thanks.

create a samba mount on your MBP, mount it at least once in windows 10 to save the credentials. after that you will be able to mount the UNC path in WSL directly using drvfs

Discussion ran course.

fuse: device not found, try 'modprobe fuse' first
Is anyone having the same problem?

fuse: device not found, try 'modprobe fuse' first
Is anyone having the same problem?

@Hug0Albert0 Having the issue. Any fix?

@Hug0Albert0 @Dibbyo456 The answer is in @fpqc 's comment, the first comment on this issue. The various solutions are the rest of this comment thread.

Keybase also needs Fuse to mount its encrypted and distributed file system, the same problem occurs. Is there any possibility of this being solved?

/dupe #17


Example 'burshing` throws Illegal attribute generation error

Following warning is generated and rendering also looks incorrect, need to viery.
deck.gl deck.gl error during update of <ScatterplotBrushingLayer:'sources'> Error: Illegal attribute generated for instanceTargetPositions
    at AttributeManager._checkAttributeArray (attribute-manager.js:619)
    at AttributeManager._updateBuffer (attribute-manager.js:570)
    at AttributeManager._updateBuffers (attribute-manager.js:553)
    at AttributeManager.update (attribute-manager.js:262)
    at ScatterplotBrushingLayer.updateAttributes (layer.js:189)
    at ScatterplotBrushingLayer.updateLayer (layer.js:414)
    at LayerManager._updateLayer (layer-manager.js:514)
    at loop (layer-manager.js:381)
    at LayerManager._matchSublayers (layer-manager.js:425)
    at LayerManager._updateLayers (layer-manager.js:349)

Following warning is generated and rendering also looks incorrect

I believe the attribute manager checks the first couple of entries in generated Float32Arrays to make sure they are not NaN.


Implemenetd cache on existing communities structure

What does this PR?
This PR adds cache support following existing cache structure on top of existing communities structure implemented in redux.
Issue number
fixes #2351
fixes  #2341
Screenshots/Video
https://user-images.githubusercontent.com/48380998/177032675-0d917e6a-681a-44f7-b923-96f6392033d2.mp4

@noumantahir  added cache in community screen while subscribing/unsubscribing from community page
https://user-images.githubusercontent.com/48380998/177381943-7116dba8-c2a1-4c0c-a4ba-a2065882cbf8.mp4

@feruzm this can be merged as well, it can use some cleaning, but that can come later, we can go for merge and wrap release


Music Assistant Music Provider "Documentation" menu entry leads to 404 link

What version of Music Assistant has the issue?
2.0.4
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Trying to get help on the Music Assistant provider leads to a non-working page
How to reproduce
a. Open MA
b. go to settings
c. select the | menu on the Music Assistant music provider
d. select Documentation
e. end up at https://music-assistant.io/music-providers/builtin/ , which is a 404 page

Music Providers
Music assistant built-in
Player Providers
N/A
Full log output
log.txt
Additional information
No response
What version of Home Assistant Core are your running
2024.6.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Windows

yes known. Fix is awaiting the next update in a day or two. Thanks.


making commercial-paper application-java to fit hyperledger-fabric-2.0.0+

modify digibank/application and magnetocorp/application to fit 2.0.0

Thanks for your working.
About the reslove error, I suggrest  adding orderer.example.org into hosts.


problem with reformatting after zoom on safari

When the example in #710 is zoomed on Safari the image is reformatted incorrectly - the exponent becomes about 20X too big.

This is maybe #99. Can you check if you get the same thing after disabling the "Never use font sizes smaller than" option in Preferences, Advanced?

Sorry, I missed your question. No, it doesn't appear to be the same issue - I didn't have the "Never use font sizes" option set to begin with. The bug is not influenced by setting/resetting the option either.


Fix tests and Travis builds

Add connection method to the WorkerTest::Receiver, the connection is being read from the receiver in Worker#run_periodic_timer and this is causing the tests to fail.
Force EM.next_tick to execute with EM.run_deferred_callbacks
Currently the tests are passing for me, but the build is still failing on travis with NoMethodError: undefined method worker_pool' for #TestServer:0x00000003bf7f68`

Looks like the tests are passing again


update dependencies

resolve https://github.com/karmada-io/website/issues/596

/lgtm
/approve

Thanks for doing this!


LES experiments in ClimaAtmos

SDI: https://github.com/CliMA/ClimaAtmos.jl/issues/2999
This PR consolidates the key changes required to run ClimaAtmos in LES configurations. This links the changes in ClimaCore PR#1736 to updates in parameterized tendencies in ClimaAtmos.

Preliminary 1M microphysics results:
RICO:
summary_rico.pdf
BOMEX:
summary_bomex.pdf


a couple of self-explanatory fixes

Hi,
Thanks for the contributions! I've pulled all the changes (with some minor changes to the commit messages), except the last (7d2d7c9) and the version increment. I may or may not make a patch release with those changes prior to some other, bigger, changes I'm working on.
 It appears the IPAddress class replaces Mask and its related functions. I'm not quite sure how the IPAddress class fits into functions relating to addresses, it seems specific to masking.
Could you elaborate on that commit?

@ version: I'm not experienced enough with git to upload only specific changes. Version increase was useful for local testing to differentiate this version from the hackage one.
@ last commit: It should not have been in the pull request (which was made before the commit). It's not finished :) and might turn out to be useless.
When I wanted to use the package, I could not understand what a Mask was. Being an Integer, which seems more like a number type, than a store some bits type and being the same for both addresses suggested it should be the prefix length. Yet Integer is needlessly big to store a number in the range 0..128.
If a mask is used to and bits of an address with, it seems to me like a good idea to store it in the same underlying type. IPAddress class is an implementation detail with functions applicable only to IP addresses (which have masking and prefixes).
Anyway, address + prefix length is the only interface I like for IP subnets. At least in the app I'm developing.

Yeah I get confused by Mask myself sometimes. It's probably overly complicated since I tried to make the same functions (toMask, fromMask) work for any mask size. It's an integer so it can accommodate larger masks than 128 bits, though I don't think we'll run out of IPv6 address space any time soon. I don't really like the idea of using an IP address as the subnet mask from a semantic point of view, even though as you suggest, the size is the same. It probably does need to be revised.


GetDiscoveredScripts throws a NullReferenceException when using WithScriptsAndCodeEmbeddedInAssembly

Hi
Let's start with the code (using the latest version of DbUp in a .NET Core 3.1 application).
var upgradeEngine = DeployChanges.To
                    .SqlDatabase("snip")
                    .LogToConsole()
                    .WithScriptsAndCodeEmbeddedInAssembly(Assembly.GetExecutingAssembly())
                    .Build();

Console.WriteLine("Discovered scripts:");
foreach (var script in upgradeEngine.GetDiscoveredScripts())
    Console.WriteLine("\t" + script.Name);

Calling GetDiscoveredScripts or GetExecutedButNotDiscoveredScripts will both call GetDiscoveredScriptsAsEnumerable, which in turn will call EmbeddedScriptAndCodeProvider.GetScripts which will then call ScriptsFromScriptClasses. This is a problem since it will call DatebaseConnectionManager.ExecuteCommandsWithManagedConnection and here the 'transactionStrategy' field is null, because OperationStarting has never been called.

It's possible to work around this issue by triggering initialisation of the 'transactionStrategy' field.
var upgradeConfig = DeployChanges.To
                    .SqlDatabase("snip")
                    .LogToConsole()
                    .WithScriptsAndCodeEmbeddedInAssembly(Assembly.GetExecutingAssembly())
                    .BuildConfiguration();
var upgradeEngine = new UpgradeEngine(upgradeConfig);

// This ultimately ensures 'transactionStrategy' is no longer null.
using var disposableOperation = upgradeConfig .ConnectionManager.OperationStarting(
   new DbUp.Engine.Output.NoOpUpgradeLog(),
   new List<SqlScript>());

Console.WriteLine("Discovered scripts:");
foreach (var script in upgradeEngine.GetDiscoveredScripts())
    Console.WriteLine("\t" + script.Name);


Wrong size of children rectangles

Rectangles do not fill the whole area of the parent, but only a small part at the bottom.

Please advise how to fix this.

You have 2 scale functions so you can play with it:
 xScaleFunction: scaleLinear().range([0, this.props.width]),
 yScaleFunction: scaleLinear().range([0, this.props.height]),

I've expose also a prop called paddingInner so you can configure the padding between nodes


Fix sha1 for guava example

The hash in the field is the sha256 sum for the guava jar, rather than the sha1 sum.

Friendly ping.  I am actually a Googler (currently on gRPC), do I still need to sign?

Ah, cool, didn't realize.  Thanks for the patch!

This was merged

Thanks!


SingleThreadedPipelineExecuter + No Transaction = Deadlock

If you add the following to your EtlProcess, it fails:
PipelineExecuter = new SingleThreadedPipelineExecuter();
UseTransaction = false;

@koulkov have you found the reason why? The only thing I've found it was a null check missing.

No, never found the root cause.


docs(cdk/portal): update portal docs

Add the DomPortal example in the cdk-portal-overview-example component and update the Portal markdown documentation adding TemplatePortal and DomPortal.

@googlebot Ping

@norato I think you should squash the commits into one commit

@ajitsinghkaler on this repo we just squash PRs when we merge them


How can I stack multiple animations correctly, such as hands and feet?

I have several hands animations and several leg animations in glb file.
How do I properly stack several different animations?

Are you writing a glTF exporter yourself, or using an existing software package to author glTF?

I wrote my little GLB file parser for my program (thanks for the document "gltf-What the daсk"). I can get animation keys and create mesh transformation matrices. I pass them to the shader and the animation displays fine if I only use one animation at a time.
And I wanted to try using two or more animations and layering them but didn't seem to get it. This is needed to overlay the animation of the arms and the whole body, this will reduce unnecessary duplication of animations. How to do it correctly?

If you have several animations that affect the local transform matrices of nodes, then it should be possible to combine them, but I think that this has some caveats: Imagine one animation that rotates object X clockwise and the other one rotating object X counterclockwise - you cannot say what the result should be. More broadly speaking: It might be possible to combine multiple animations if they are "independent" of each other. But I'm pretty sure that this is not covered by the specification.
But more importantly:

create mesh transformation matrices

That sounds like you're doing vertex skinning. If this is the case: Combining "multiple skinnings" (with animation) is really difficult (it might even be impossible to do this sensibly, but I couldn't give any advice here...)

well. Maybe I chose a dead end path and there is another way to do it. For example, I need to turn my head depending on the direction of the gaze during the playback of the walking animation. How to do it?

It's a bit too early to talk about a "dead end" - at lest for me, it's not entirely clear what the goal is. The last statement sounded like you wanted to make the decision about the actual animations that are played at runtime. That might be difficult, but here is a short pointer to some related discussion: https://github.com/KhronosGroup/glTF/issues/1052:
This eventually led to the implementation note in the spec:

Implementation Note: glTF 2.0 does not specifically define how an animation will be used when imported but, as a best practice, it is recommended that each animation is self contained as an action. For example, "Walk" and "Run" animations might each contain multiple channels targeting a model's various bones. The client implementation may choose when to play any of the available animations.

This sounds pretty similar to your application case. I think that it should be possble do do play several animations simultaneously, with some "caveats":

Other glTF viewers may not be able to play multiple animations at the same time
Your viewer/client would need dedicated controls and an infrastructure so that it is possible to tell it something like "Play the 'walk' animation, for 2.3 seconds, and after 1.8 seconds, play a certain part of the 'turn head' animation".
You'd have to be really careful to not play animations that affect the same bones - otherwise, the results would be unpredictable

It could be a challenge to support this sensibly, but should be certainly be worth a try.

This isn't a core feature of glTF itself.  But for what it's worth, it's possible to implement this on top of glTF, with custom code.
For example, here's the glTF sample fox, playing the "Run" animation and watching the International Space Station fly by at the same time:

In our case this is done with a vendor extension, AGI_articulations.  An "articulation" is a joint that is permitted to move or rotate about particular axis/axes by amounts defined in the extension, but without a pre-scripted animation.  Such transformations are applied separately and subsequently to the transformations applied by the glTF itself and its own built-in animations.
So what you're seeing here is that a bone in the Fox skeleton's head has been given an AGI_articulations extension and allowed to swivel by some number of degrees, along with a defined "pointing vector".  The host software STK aligns this pointing vector with the user's target of choice, in this case the space station passing overhead.  The Fox's "Run" animation is playing, causing the head to bob up and down, and the pointing articulation is applied subsequently to that, causing the head to track the station during the pass.
But again, this is vendor-specific, not part of the glTF standard itself.  If you're not using STK, it's a feature that would need some development effort for your platform of choice.

Yes, this wonderful video inspired me to find new solutions, thank you. I was thinking about a method to change the position of the bone programmatically and recalculate all matrices in real time.
And my idea was that the rotation of the fox's head was recorded in a separate animation and, specifying the number of the current frame of the head, merge it with the current animation of the body.

thank you


Re-Upload a file after a process-error Or reset FileStatus

Hey,
I am currently using the filepond with Vue to upload some files. I am currently a bit helpless to how I can reset the filestatus or if the user re-uploads the file that it will upload the file again(after a processing error), but it does nothing at all, it is not uploading the file again.
I am using a custom processing method, that is set during the initialization of my component.
setOptions({
    server: {
    process: this.process,
  }
});

The process method simply makes a formData and sends it to the server.
I await a status code as a response by my server.
Now if the server would send back a bad status code, how can I make it that the user can press the upload button again to upload the file. (Where either FileStatus is reset for this.$refs.filepond.processFiles(); to work again)
FYI, I do not want to use the controls from filepond, that is not possible with how my form is setup.
How can I with my custom processing method properly re-upload a file that had a processing-error fired. I want to be able to press upload again with this code this.$refs.filepond.processFiles();, also if there was a processing-error, he should just try again. Is that possible for my situation?

You can use the error callback to return an error state to FilePond.
https://pqina.nl/filepond/docs/patterns/api/server/#process-1
Please also make sure you return a uniqueId along with the load callback.


¿How to install Ubuntu 14.04?

I'm sorry I'm a noob at linux and do not know how to install it if someone can explain to you be grateful.

First install npm (Node Package Manager) and node. This should help.
Then

git clone this repo to your workarea
cd to the new directory
sudo npm install to install  the dependencies
node build.js to build the app
The newly built app will reside in build/Messenger/linux{32|64}
You can run it directly with ./build/Messenger/linux32/Messenger

Probably he was only interested in running, not building.
@leninalbertolp Download the latest release for your OS from here.
Then you have to extract it and open Messenger. I'm working on an easier install procedure.

Okay, thank you very much

@leninalbertolp The latest release is just an installer now, instead of an archive. You have to run it and it will install by itself, that's it. I hope this is easier for you.

Has anyone already packaged this great app as a .deb. or provides a ppa for it?

@orschiro Not yet. @Aluxian has used makeself to create an installer for Linux.

@orschiro @faizaanceg Now I'm packaging .deb directly. It will be released soon.

@Aluxian :+1:  That's good news :) !

@Aluxian
Great news! Let me know if you can provide a ppa for it. I am happy to add it to ppa.orzanna.de!

@orschiro The .deb is up at https://github.com/Aluxian/Facebook-Messenger-Desktop/releases/latest. If you add it to your ppa I'll add it on the website.

@Aluxian
What a great release! Integrates perfectly into the elementary OS desktop. One minor thing. Can we get rid of the menubar containing the File entry which only has one entry Quit? I would save we can save this space. :)

Regarding the PPA. I don't have any experience in packaging a package for a PPA myself but I will ask @quassy if he wants to package it in his PPA. Will let you know!

@orschiro I'm still experimenting with how the app should work on each platform. I agree that's a waste of space and could be integrated better.
Thanks for the submission!

If the application is packaged in a deb already, honestly most of the work is already done,  I'm downloading the apps now (32 and 64 bit) and it should be up in my ppa shortly.
As soon as it is i'll report back with the link, once I confirm it's working.

Thanks a lot @JustSomeDood! Did it work out?


amr.py has no license info

The amr.py file is copied (presumably) from smatch, but it does not include smatch's license text. This is a condition of the MIT license. I think just adding a comment at the top saying where the module came from with a link to smatch's LICENSE.txt is sufficient.

I just update the amr.py file.

Excellent. Thank you