In readme Web3 v2 is mentioned but there is no examples given

Could you clarify README on TypeChain usage with web3 v2, on the current state of Web3 v2 support
I can see some progress in the branch but it looks like there is nothing on that in the master branch.

Simply put - it's not supported. What's the status of development? Is it stable yet?

Uhh i think it might be a typo in readme... I only recall some old PoC.

What's your plan on that, you wait for a stable version?
Current is still alpha web3 2.0.0-alpha.1 but it looks good enough interaction with smart contracts

@artall64 TBH: I don't plan on working on this. I prefer using ethers.
It should be quite simple to tweak an already existing target - so I hope that the community will create this target if it's really needed.

I just removed any mention of web3-v2 from the readme. I will rename this issue.


Vaulted backup restore using prefix support

Please validate if use prefix match to restore blobs is available for vaulted backups or if the doc is outdated.  The portal indicates that the feature is now available and tests successfully restore using blob name.
Currently the doc specifies this:
This capability is currently supported only for operational backups.

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

ID: 4535833e-19cd-2692-b6f8-8ca3a1a8d71e
Version Independent ID: ef895e44-35a4-d96b-19be-693031dede2d
Content: Restore Azure Blobs - Azure Backup
Content Source: articles/backup/blob-restore.md
Service: backup
GitHub Login: @AbhishekMallick-MS
Microsoft Alias: v-abhmallick

@briggsb9 Thank you for your feedback. We will have this reviewed by the content team and take appropriate action.

Hi @briggsb9  - Thanks for bringing this to our attention. We've updated the article and it will be live soon. As the document update is complete, closing this trail. #please-close
CC: @SadiqhAhmed-MSFT


Update gonzalez so css vars can be parsed #537

@aiboy

@tonyganch Hey, can you take a look at this, please?

Fixed in dev ❤


_original_id should not be required in weaviate with Haystack 2.6

I am using the WeaviateEmbeddingRetriever to work with the data.
It works fine with the default class in weaviate.
Once I change it to the data class created by myself with customized schema, I got the issue as below:
  File "/root/TS_ph3/00_WeaviateEmbeddingRetriever.py", line 70, in <module>
    result = query_pipeline.run({"text_embedder": {"text": query}})
  File "/root/.cache/pypoetry/virtualenvs/search-infra-7HLB3Aeo-py3.10/lib/python3.10/site-packages/haystack/core/pipeline/pipeline.py", line 229, in run
    res: Dict[str, Any] = self._run_component(name, components_inputs[name])
  File "/root/.cache/pypoetry/virtualenvs/search-infra-7HLB3Aeo-py3.10/lib/python3.10/site-packages/haystack/core/pipeline/pipeline.py", line 67, in _run_component
    res: Dict[str, Any] = instance.run(**inputs)
  File "/root/.cache/pypoetry/virtualenvs/search-infra-7HLB3Aeo-py3.10/lib/python3.10/site-packages/haystack_integrations/components/retrievers/weaviate/embedding_retriever.py", line 138, in run
    documents = self._document_store._embedding_retrieval(
  File "/root/.cache/pypoetry/virtualenvs/search-infra-7HLB3Aeo-py3.10/lib/python3.10/site-packages/haystack_integrations/document_stores/weaviate/document_store.py", line 538, in _embedding_retrieval
    return [self._to_document(doc) for doc in result.objects]
  File "/root/.cache/pypoetry/virtualenvs/search-infra-7HLB3Aeo-py3.10/lib/python3.10/site-packages/haystack_integrations/document_stores/weaviate/document_store.py", line 538, in <listcomp>
    return [self._to_document(doc) for doc in result.objects]
  File "/root/.cache/pypoetry/virtualenvs/search-infra-7HLB3Aeo-py3.10/lib/python3.10/site-packages/haystack_integrations/document_stores/weaviate/document_store.py", line 306, in _to_document
    document_data["id"] = document_data.pop("_original_id")
KeyError: '_original_id'

I check the codes and find that the predefined function need to get data of _original_id and set it as the Document ID.
I have updated the codes in document_store.py and set set document_data["id"] as generated UUID if the dataset does not have one.
In this case, the expected results are shown.
I do not think that the data in weaviate is forced to have the column as _original_id .
But based on the current codes, it will return errors if no _original_id there.
I prefer to have a if statement to handle the different cases.
Please kindly correct me if any misunderstandings.
The packages I am using are:
haystack-ai = "2.6.1"
fastembed-haystack = "1.3.0"
weaviate-client = "^4.9.0"
weaviate-haystack = "^4.0.0"
    def _to_document(self, data: DataObject[Dict[str, Any], None]) -> Document:
        """
        Converts a data object read from Weaviate into a Document.
        """
        document_data = data.properties
        # The error is raised here and I just set document_data["id"] as generated UUID if the dataset does not have one.
        document_data["id"] = document_data.pop("_original_id")
        if isinstance(data.vector, List):
            document_data["embedding"] = data.vector
        elif isinstance(data.vector, Dict):
            document_data["embedding"] = data.vector.get("default")
        else:
            document_data["embedding"] = None

        if (blob_data := document_data.get("blob_data")) is not None:
            document_data["blob"] = {
                "data": base64.b64decode(blob_data),
                "mime_type": document_data.get("blob_mime_type"),
            }

        # We always delete these fields as they're not part of the Document dataclass

The rationale behind this field is explained here:
https://github.com/deepset-ai/haystack-core-integrations/blob/67e08d0b7e5a7f51f52bb0d40fe40b0ff2caf43a/integrations/weaviate/src/haystack_integrations/document_stores/weaviate/document_store.py#L276-L278
This is done to provide a robust default to users who don't need serious customization.
For simplicity, you can add include this field to your collection configuration:
https://github.com/deepset-ai/haystack-core-integrations/blob/67e08d0b7e5a7f51f52bb0d40fe40b0ff2caf43a/integrations/weaviate/src/haystack_integrations/document_stores/weaviate/document_store.py#L40
Does this create problems?


Use epics-base instead of epicscorelibs

I'm wondering whether it would be possible to switch out the dependency of this package to epics-base, rather than epicscorelibs. I've read through #17, and see that there are concerns that the epics-base package is broken on windows.
If we can resolve the epics-on-windows issues, is it reasonable to switch out the dependency, so that we can try and converge on epics-base as the one-true source of EPICS? I'm not an epics expert, just trying to simplify the packaging of pyepics, pcaspy, and a few other things needed by the LIGO experiment.
cc @jrollins.

On Wed, Nov 20 2019, Duncan Macleod<EMAIL_ADDRESS>wrote:

I'm wondering whether it would be possible to switch out the
dependency of this package to epics-base, rather than
epicscorelibs. I've read through #17, and see that there are
concerns that the epics-base package is broken on windows.
If we can resolve the epics-on-windows issues, is it reasonable to
switch out the dependency, so that we can try and converge on
epics-base as the one-true source of EPICS? I'm not an epics expert,
just trying to simplify the packaging of pyepics, pcaspy, and a
few other things needed by the LIGO experiment.

For what it's worth, we have applications that use both pcaspy and
pyepics, so we need both of those packages to be compiled against the
same version of epics.

@duncanmmacleod if you can work on the windows build for epics-base and get it going that would be great.
I have an open issue (https://github.com/conda-forge/epics-base-feedstock/issues/3) and a PR that updates epics-base to 7.0.3 (https://github.com/conda-forge/epics-base-feedstock/pull/2).
I can help with tests if needed.
 It will not interfere with the epics-base package that you may already have installed and pyepics is not compiled so there is no issue in mixing it with pcaspy.

@hhslepicka, now that https://github.com/conda-forge/epics-base-feedstock/pull/6 was merged, are you interested in using epics-base as a dependency instead of epicscorelibs?
I can make a PR if you are.

I will test epics-base here and get back to you on that.

@beenje I think that it would be okay to do that. Just one request... Would you mind to only set the PYEPICS_LIBCA only if one is not set? That would allow for facilities that have their own EPICS Base compiled to also use that even if PyEPICS was installed via conda.

If we only set PYEPCIS_LIBCA when not already defined, it means we can't unset it in the deactivate script. That's not clean. Not speaking about the fact that the variable could have been set for different reasons and you might end-up with a conda environment not working as expected.
My opinion is that if this package depends on epics-base (from conda) it should use the libraries installed with conda.
conda is meant to install packages and their dependencies in an isolated environment. I don't really see why you would install pyepics with conda if you want to use another epics-base.
If that's what you want, isn't it better to install pyepics with pip (can still be done in a conda env)?
Another way would be to have a separate conda pyepics package that doesn't depend on epics-base.
Anyway I created #21 as I see it.
We can take the discussion there.


Intellisense not working with mapped drive

I have a VM with ssh enabled and use SFTP Net Drive 2017 to map the VM's file system to Windows 10 which works great. VSCode can work on the file without issue, but Intellisense doesn't seem to work, stating that it cannot find files.

VSCode Version: 1.20.1
OS Version: Windows 10

Steps to Reproduce:

Map VM file system using SFTP Net Drive
Open project, and Intellisense will not work.

Here is my configuration:
{
    "configurations": [
        {
            "name": "Win32",
            "includePath": [
                "G:\\working\\hypervisor\\build\\prefixes\\x86_64-vmm-elf\\include",
                "G:\\working\\hypervisor\\build\\prefixes\\x86_64-vmm-elf\\include\\c++\\v1",
                "G:\\working\\hypervisor\\build\\prefixes\\x86_64-userspace-elf\\include",
                "G:\\working\\hypervisor\\build\\prefixes\\x86_64-userspace-elf\\include\\c++\\v1",
                "G:\\working\\hypervisor\\build\\prefixes\\x86_64-test-elf\\include",
                "G:\\working\\hypervisor\\build\\prefixes\\x86_64-test-elf\\include\\c++\\v1",
                "G:\\working\\hypervisor\\bfdriver\\include",
                "G:\\working\\hypervisor\\bfdummy\\include",
                "G:\\working\\hypervisor\\bfelf_loader\\include",
                "G:\\working\\hypervisor\\bfintrinsics\\include",
                "G:\\working\\hypervisor\\bfm\\include",
                "G:\\working\\hypervisor\\bfsdk\\include",
                "G:\\working\\hypervisor\\bfunwind\\include",
                "G:\\working\\hypervisor\\bfvmm\\include",
                "G:\\working\\extended_apis\\include",
                "C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.12.25827/include/*",
                "C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.12.25827/atlmfc/include/*",
                "C:/Program Files (x86)/Windows Kits/10/Include/10.0.16299.0/um",
                "C:/Program Files (x86)/Windows Kits/10/Include/10.0.16299.0/ucrt",
                "C:/Program Files (x86)/Windows Kits/10/Include/10.0.16299.0/shared",
                "C:/Program Files (x86)/Windows Kits/10/Include/10.0.16299.0/winrt"
            ],
            "defines": [
                "_DEBUG",
                "UNICODE",
                "_UNICODE",
                "VMM",
                "NATIVE",
                "ENABLE_BUILD_TEST",
                "_HIPPOMOCKS__ENABLE_CFUNC_MOCKING_SUPPORT"
            ],
            "intelliSenseMode": "msvc-x64",
            "browse": {
                "path": [
                    "C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.12.25827/include/*",
                    "C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.12.25827/atlmfc/include/*",
                    "C:/Program Files (x86)/Windows Kits/10/Include/10.0.16299.0/um",
                    "C:/Program Files (x86)/Windows Kits/10/Include/10.0.16299.0/ucrt",
                    "C:/Program Files (x86)/Windows Kits/10/Include/10.0.16299.0/shared",
                    "C:/Program Files (x86)/Windows Kits/10/Include/10.0.16299.0/winrt"
                ],
                "limitSymbolsToIncludedHeaders": true,
                "databaseFilename": "C:/.vscode/browse.VC.db"
            }
        }
    ],
    "version": 3
}

Does IntelliSense work when you disable all extensions and create a JavaScript project?

Yes, it works fine. Please note that I am using C++.


Error in importing com.jayway.jsonpath from U2 level EI-<IP_ADDRESS>

Description
The following issue can be observed when placing the org.wso2.carbon.framework.extension.json_20090211_1.0.0.jar  file to the <EI_HOME>/dropins directory from U2 level EI-<IP_ADDRESS> onwards.
[2023-12-13 12:48:14,268] ERROR {org.wso2.carbon.application.deployer.internal.ApplicationManager} - Error occurred while deploying Carbon Application org.apache.axis2.deployment.DeploymentException: com/jayway/jsonpath/PathNotFoundException
Steps to Reproduce
1] Update the WSO2 EI-6.6.0 server to the U2 level 148.
2] Add the attached lib folder at the link[2] to the pack and, start the server.
3] Stop the server and, add the dropping jars at the link[2] to the dropins folder.
4] Start the server and, deploy the attached car file.
5] Car app will deploy without any issue.
6] Update the server to the 149.
7] Start the server.
8] The error will be observed while deploying the car file.
9] Remove the "org.wso2.carbon.framework.extension.json_20090211_1.0.0.jar" from the dropping folder and, restart the server.
10] The issue will resolved.
11] Enable the OSGI console and, check the import and, export dependancies for synapse-core_2.1.7.wso2v143_104 (for U2 148) and synapse-core_2.1.7.wso2v143_108 (for U2 149).
12] The com.jayway.jsonpath is not importing to the problematic U2 149.
Affected Component
APIM
Version
6.6.0
Environment Details (with versions)
No response
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Closing hence the issue is introduced from the manually added jar org.wso2.carbon.framework.extension.json_20090211_1.0.0.jar


HTML Macro Bug

I can't get html! to create a tags correctly. I tried it with just a single a tag:
html! {
 Foo
}

And it still errors out:
error: no rules expected the token `stack`
  --> src/lib.rs:66:9
   |
66 | /         html! {
67 | |           Foo
68 | |         }
   | |_________^
   |
   = note: this error originates in a macro outside of the current crate (in Nightly builds, run with -Z external-macro-backtrace for more info)

error: aborting due to previous error
error: Could not compile `rustynzb_frontend`.

error: build failed

Process finished with exit code 101

html! macro supports Rust expressions only (I think we can implement direct text inclusions later, but it's impossible with stable compiler). For now you should use:
html! {
    { "Foo" }
}


Add non string map keys

Support non-string keys in maps (#3).

FYI @omalley.

Looks good. It was a much larger patch than I was expecting.
I usually squash commits like this as I commit them to master to make the history easier to read.

Most of the reason for the additional size was that this moved Avro reads and writes over to ValueReader and ValueWriter like Spark uses. That made it easier to support the map logical type and also made reading and writing those maps more efficient because logical types deserialize to generic classes and then convert. The ValueReader approach can go directly to/from map and an Avro array. The downside was restructuring the Spark and Avro code so much to make it happen.
I debated squashing, but I thought it would be better to have separate commits since as a whole the changes were so large.

Forgot to mention: another bonus is that scanning manifest files seems to be ~10-20% faster.


5 to 6 20211020

➡ Forward port
Port ign-gazebo5 to ign-gazebo6
Branch comparison: https://github.com/ignitionrobotics/ign-gazebo/compare/ign-gazebo6...ign-gazebo5
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

@osrf-jenkins run tests please. Let's see if Windows shows up after https://github.com/ignition-tooling/release-tools/pull/539

@osrf-jenkins run tests again :pray:


Lesson 3 Task 2: Not recognizing correct answer

Hi,
I'm unsure if I'm just being stupid, however for String multiplication, I've put in the answer:
hello = "hello" ten_of_hellos = hello * 10 print(ten_of_hellos)
However, it's telling me to use "Multiplication" and marking it wrong... Am I missing something here?

Hello @CarnegieL
Nope, you are right. It's a known bug with a pending pull request (#11). You can fix it temporary by yourself - edit function test_value() inside the file ~/PycharmProjects/PythonIntroduction/lesson3/task2/tests.py (placed next to the task file):
def test_value():
    file = import_task_file()
    if hasattr(file, "ten_of_hellos") and file.ten_of_hellos == "hellohellohellohellohellohellohellohellohellohello":
        passed()
    else:
        failed("Use multiplication")

This will fix the broken test.

i have the same problem, please fix that!
i want to use that in school

Why isn't this fixed?  JetBrains is only hurting their chance for a potentially massive adoption of this IDE.

REASON:  Windows version fails to remove X:\Users<user-name>\PycharmProjects<...>
So, even after complete uninstall & reinstall, the buggy course projects remain.  This is EXTRA frustrating for non-power users who wish to reset Pycharm's progress.  Anyway.

Hi @veganaize What PyCharm EDU version are you talking about?
This bug was fixed in version 3.5.

@medvector I am talking about Pycharm EDU 3.5 (current stable).  It fails to remove remnants of previous version(s) in user's home folder (on Windows 8.1 64-bit) -- Even when the two checkboxes, which remove those kinds of things, are checked.  Then, when I install 3.5 (after reboot & everything), it still loads the OLD course lessons (v3.0) and my progress still remains at whatever it was (not reset).
It took me a lot of mucking to finally figure out (the obvious?) that I needed to completely remove/archive the *C:\Users<username>\PycharmProjects* folder & subfolders.

These folders are located in a weird place for Windows users, as they don't typically look in the folder named after their username (they usually look in subfolders such as "Downloads", "Documents", etc.).

It is counter-intuitive to offer an option to delete all caches, etc. when uninstalling -- but then leave those particular files laying around (without even a popup to offer to delete them).

It is counter-intuitive that the new version (v3.5) doesn't offer to "fix" (ie. overwrite) the existing corrupt lesson(s) (which should have been deleted already anyway; but since they weren't, it's now 3.x's or 4.x's job to do so).

Thanks!


Error happened when dst IP is in the case: TTL expired

Software Version:
Python3.4/3.3
ply (3.10)
pyasn1 (0.2.3)
pycrypto (2.6.1)
pycryptodome (3.4.6)
pysmi (0.1.3)
pysnmp (4.3.8)
Windows2008R2
Error happened when TTL expired:
Pinging <IP_ADDRESS> with 32 bytes of data:
Reply from <IP_ADDRESS>: TTL expired in transit.
Reply from <IP_ADDRESS>: TTL expired in transit.
Reply from <IP_ADDRESS>: TTL expired in transit.
Reply from <IP_ADDRESS>: TTL expired in transit.
Ping statistics for <IP_ADDRESS>:
Packets: Sent = 4, Received = 4, Lost = 0 (0% loss)
Error:
Traceback (most recent call last):
File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dgram\base.py", line 155, in handle_read
File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dgram\base.py", line 38, in __recvfrom
OSError: [WinError 10052] The connection has been broken due to keep-alive activity detecting a failure while the operation was in progress
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dispatch.py", line 46, in runDispatcher
File "C:\Python34\lib\asyncore.py", line 212, in loop
poll_fun(timeout, map)
File "C:\Python34\lib\asyncore.py", line 153, in poll
read(obj)
File "C:\Python34\lib\asyncore.py", line 87, in read
obj.handle_error()
File "C:\Python34\lib\asyncore.py", line 83, in read
obj.handle_read_event()
File "C:\Python34\lib\asyncore.py", line 442, in handle_read_event
self.handle_read()
File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dgram\base.py", line 171, in handle_read
pysnmp.carrier.error.CarrierError: recvfrom() failed: [WinError 10052] The connection has been broken due to keep-alive activity detecting a failure while the operation was in progress
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\Administrator\Dropbox (NT)\multi-vendor support\Mimic Files\Scripts\SNMP.py", line 83, in
SNMP_set_hostname(dit[1], dit[0])
File "C:\Users\Administrator\Dropbox (NT)\multi-vendor support\Mimic Files\Scripts\SNMP.py", line 33, in SNMP_set_hostname
lookupValues=True
File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\entity\rfc3413\oneliner\cmdgen.py", line 200, in setCmd
File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\hlapi\asyncore\sync\cmdgen.py", line 219, in setCmd
File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dispatch.py", line 50, in runDispatcher
pysnmp.error.PySnmpError: poll error: Traceback (most recent call last):
;  File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dgram\base.py", line 155, in handle_read
incomingMessage, transportAddress = self._recvfrom(self.socket, 65535)
;  File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dgram\base.py", line 38, in __recvfrom
d, a = s.recvfrom(sz)
;OSError: [WinError 10052] The connection has been broken due to keep-alive activity detecting a failure while the operation was in progress
;
During handling of the above exception, another exception occurred:
;Traceback (most recent call last):
;  File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dispatch.py", line 46, in runDispatcher
use_poll=True, map=self.__sockMap, count=1)
;  File "C:\Python34\lib\asyncore.py", line 212, in loop
poll_fun(timeout, map)
;  File "C:\Python34\lib\asyncore.py", line 153, in poll
read(obj)
;  File "C:\Python34\lib\asyncore.py", line 87, in read
obj.handle_error()
;  File "C:\Python34\lib\asyncore.py", line 83, in read
obj.handle_read_event()
;  File "C:\Python34\lib\asyncore.py", line 442, in handle_read_event
self.handle_read()
;  File "C:\Python34\lib\site-packages\pysnmp-4.3.8-py3.4.egg\pysnmp\carrier\asyncore\dgram\base.py", line 171, in handle_read
raise error.CarrierError('recvfrom() failed: %s' % (sys.exc_info()[1],))
;pysnmp.carrier.error.CarrierError: recvfrom() failed: [WinError 10052] The connection has been broken due to keep-alive activity detecting a failure while the operation was in progress

That looks like a network problem to me... SNMP might be totally unrelated to this. I can imagine this kind of behaviour when you have a routing loop on your network, for example.


Questionnaire page size is over 120MB!

:sob: :skull: :bomb:
This is an unacceptable page size even for broadband connections.
The problem seems to mostly stem from the massively redundant and nested form format that surveyor seems to generate. i.e.
A list of radio buttons isn't something like

Question text

<input name="r[7][answer_id] value="4" type="radio"> Option 1
    …

But

    <input name="r[7][question_id]" type="hidden" value="4">

    <input id="r_7_api_id" name="r[7][api_id]" type="hidden" value="90189c97-60bd-4705-9a8a-5791ae59bde0">

    <input id="r_7_autocompleted" name="r[7][autocompleted]" type="hidden" value="1">

      <fieldset class="choices">

              <input class="" id="r_7_answer_id_4" name="r[7][answer_id]" type="radio" value="4">
              a one-off release of a single dataset

      </fieldset>
…

Some of the classes etc are necessary but the form could be a lot simpler with the same functionality.
Unfortunately this would require either uncoupling surveyor from formtastic or removing dependencies on both of the gems which is a large task

I think the string generation and transfer time is probably a significant factor in the response time too as I can't find much in the database or controller to shave off time.

Turns out it was just the rails footnotes extension, it's actually 600KB in dev and about 60KB in production, not sure where that discrepancy is coming from though. Interestingly the footnotes extension delays the DOMContentLoaded event by 5 minutes.


fix: manual deployment and add dev config

Changes

Add dev deployment configs.
Remove EDGEDB_DATABASE override.

Happy to merge this after https://github.com/umccr/elsa-data-aws-deploy/pull/10 @andrewpatto.


Create Coplotremove.ps1

That script removes copliot btw.ots powershell

gh pr checkout 2289

Because thats how winutil works by ps1 files because it can run scripts which we can customise and remove bloat and make windows more useful and usable

And i want to add a way to remove copliot and bing ai so theres alternative to remove copliot by clicking a button and never seen it again so wont switch to linux

Ps1 files are powershell script that do the tweaks and winutil is 98.4 present powershell and there all most ps1 and different types of files so yes there aren't placed the are generated and this ps1 file is for tweak that you can import the other one is editing windows and made to install and uninstall the program

Real mulla c tell me then what files should i add and code i will try to code it and try to putt it on the winutil so i can be a contrybutor

Real mulla c tell me then what files should i add and code i will try to code it and try to putt it on the winutil so i can be a contrybutor

That file is not needed.
You have to edit file config/tweaks.json

Hey guys i didn't tested the tweak yet but the problem is that i can't upload it as a zip file i have a picture that the tweak in virus total can someone test my ps1 tweak its about removing copliot ai permanently even windows update
Coplotremove.ps1.zip

Btw i didn't mention it that i don't have a windows pc

And made a earlile GitHub https://github.com/Nikoskampourakis/Copliot-removal-/blob/main/Copliotdisable.ps1 if you know what is doing

Tell me any issues about the script if it has bugs tell me on GitHub

Tell me any issues about the script if it has bugs tell me on GitHub link https://github.com/Nikoskampourakis/Copliot-removal-/blob/main/Copliotdisable.ps1

You need to put it into the config folder then in tweaks.json file find the bot about removing copilot and etc then putting your code there correctly. Not have a separate file.

I don't need a bot its on group pollicy editor i tweaked it

Tell me any issues about the script if it has bugs tell me on GitHub link https://github.com/Nikoskampourakis/Copliot-removal-/blob/main/Copliotdisable.ps1

You need to put it into the config folder then in tweaks.json file find the bot about removing copilot and etc then putting your code there correctly. Not have a separate file.

I already have an alternative here i already have the script {
"WPFCopliotremovepermanent": {
"Content": "Disable Windows Copilot",
"Description": "This disables Windows Copilot permanently even if windows updates.",
"category": "Essential Tweaks",
"panel": "1",
"Order": "a005_",
"url": "https://github.com/Nikoskampourakis/Copliot-removal-/blob/main/Copliotdisable.ps1"
}
}

The code for the tweak is in the GitHub and thx again


Upgrade to React 15.5 and React Router 4.1

PropTypes were moved to a standalone package and enzyme now requires react-test-renderer/shallow.

LGTM. Thank you for your contribution!


Fix the output directory for compiler execution in unit tests

It should be target so it gets cleaned when mvn clean is called

Fixed


Download File always -1 on Progress

FTP OS: Windows
FTP Server: https://github.com/drftpd-ng/drftpd
Computer OS: Windows 11
FluentFTP Version: <IP_ADDRESS>
I can't DownloadFile. I always have -1 on progress. The server is FTPS TLS Explicit with PRET and PASV
Logs :

I need the FTP logs to debug anything.


Link Kubernetes Namespace and Calico IP Pool

This is a feature request that allows linking a Kubernetes Namespace and Calico IP Pool.
This would cause a pod created in the namespace to use IP addresses from the linked IP Pool.

Created from discussion on issue https://github.com/projectcalico/cni-plugin/issues/288

This is available in Calico v3.3! 🙌


Service health check

Hi,
I'm deploying primeqa on a Kubernetes cluster and I'm looking for a way to perform health-checks.
In general, recent versions of Kubernetes have built-in support for gRPC health check. However, for this to work, the application has to implement the standard gRPC Health Checking Protocol. Looking at primeqa code, I didn't find such implementation.
Is this something on the horizon? Or is there any other health-check mechanism that is already supported?
Thanks

Adding that for now I'm just using TCP health check on the gRPC port.

@assaftibm It's a good suggestion. We will add a health check service in subsequent release.


perf: avoid multi re-rendering when switch the project tree item (step 2)

Description
1. multi re-render
When we switch the tree item from project tree, we will update the recoil state and the reach/router will also update its context. These two will trigger twice re-render.

fix
All design page components only consume the designPageLocation state(recoil state). Add React.memo to check props change, and memorize the result.

2. remove CloneDeep
The clonedeep in visual editor will take about 80ms when switch the dialog. implement it in other way, and the time reduce to 0.3ms

3. The links in sideBar will call setState 9 times
The < Link /> don't have a root provider, so all < Link /> will call setState.

fix
Add a root provider for these  < Link />, they will use the same context

Task Item
#refs #5559

Screenshots

Coverage decreased (-0.01%) to 55.175% when pulling dff021d44a201ae0a52b6ce85798c71a2080b6fe on lei9444:stateupdate into 73c0bcecc04cf809838fc23870cb5613722fcd22 on microsoft:main.


Backfill date limit

Is there any particular reason why you limit the backfill to 7 days? Asking if there are any performance gotchas.

You may modify the limit as per your wish. I kept it 7 days because of the limitations of environment my workloads were running on.


added sh as supported shell

I don't understand why this was not available before -- sh fully supports read -r!
Tested with /usr/bin/env sh -c 'stty -echo; read -r mypassword; stty echo; echo $mypassword' on a system that only has sh(alpine linux minimal).
As composer uses your library and there are images of composer based on alpine linux / other bashless systems, this should be merged to stop people from exposing their private repository passwords.

Oh yeah that's a funny oversight. I guess we never noticed because almost everyone has at least bash on the machine. Anyway thanks for the fix.

read -r seems to be supported in POSIX, so all well.
http://pubs.opengroup.org/onlinepubs/9699919799/utilities/read.html


i18n(de-DE): Translate concepts/islands.md

Create and localise /de/concepts/islands.md.

What kind of changes does this PR include?

Translated content

Description

Create and localise /de/concepts/islands.md.
Checked with pnpm dev

Oh, I almost forgot: Could you please also edit the file src/i18n/de/nav.ts and add the translation for your new page title there? This will make the page title also show up correctly in the left navigation bar. It should be as simple as adding 'concepts/islands': 'Astro-Inseln', to the // Core Concepts section.

Done, de/nav.ts updated and review feedback implemented.


Global left nav: update design to flat model

Overview
Updating global left nav component with new flat design.
Added
n/a
Removed
Removed sections from nav, no more category switching.
Pruned javascript of unused functions like handling children, section animations & flyouts.
Changed
Component html updated to match code in Common.
Made accessibility updates for keyboard navigation.
General css updates to fit new design.
Testing / Reviewing
Made changes to unified-header.html so you can test it

@hellobrian - @aliciahickman addressed your comments (for some reason she can't make comments on this PR?)

Not... sure why @aliciahickman can't, what's the error? With the removal of prior functionality, how would you handle an A/B test? I know that was floated as a possible requirement

@chrisdhanaraj I was getting 'you can't perform this action at this time' notification... but i guess it's ok today /shrug

...also we checked with Mark and A/B testing is not happening.

:+1: no problems from me then, merging it in


Implement vectorized operators

Vectorized operators shall be made to resemble their scalar counterparts. In all cases seen as appropriate, where the scalar operator is notated op, the vectorized operator is notated <op>. For example, vectorized addition would have the symbol <+>. Vectorized operators shall have the same level of precedence as their scalar counterparts.
The primary use case of a vectorized operator should be to perform the corresponding scalar operation on pairs constructed from parallel items (i.e. same position, index, or key) in their respective vectors.
A vector is any iterable container.
Binary Operators
Where the lengths of the operands do not match, the operand with shorter length shall be extended, using repetition of its own elements, to match the length of the longer operand.
Where one of the operands is not a vector, but in fact a scalar, it shall be treated as a single-element vector.
The following operators comprise the minimum implementation for this feature, but others may be added.
   <*>  </>  <//>  <%>  <+>  <->  <$>

Function and function application syntax make this an unnecessary change.


Let users create new QBO accounts from the client

e.g. creating a new Stripe clearing account from the settings form

Hi, I would like to contribute to this issue, can you assign to me?


"Unexpected identifier" SyntaxError thrown by meiosis-setup

I wanted to give the Polaris Form Builder a try, but unfortunately I'm stuck with the following error after installing it:
node_modules/meiosis-setup/mergerino/index.js:1
┃ import commonSetup from "../common";
┃        ^^^^^^^^^^^
┃
┃ SyntaxError: Unexpected identifier

I've tried it with node versions 10 and 12 (v12.18.3) and version 1.2.0 as well as the current version of this repo.
Steps to reproduce the error:

Create a new node app using the Shopify App CLI
Install polaris-form-builder
Add a form to the app according to readme

Hi @pwagner - sorry for slow reply.  I don't actually use the shopify CLI to develop apps but I do know that in order to use the form builder package you'd need a build process like rollup or webpack.
Not 100% sure what the build process used in the CLI is but that error looks like it might be in that ballpark.  I certainly don't see anything like that when compiling my JS (or TS) using webpack.  I'll add a note in the readme about this.

The node app created by Shopify App CLI actually uses webpack for the build process, via nextjs. Maybe its default babel presets are causing a problem here. I'll dig deeper when I have time, I'd really like to play around with the form-builder. Thanks for your reply!


WARNING: Version mismatch for Pod-To-HTML ("https://github.com/perl6/Pod-To-HTML.git") found. META6.json has "0.8.1" but latest 'git tag' is "v0.7.1"

Can someone add a 0.8.1 git tag to this repo, please?
https://github.com/Raku/Pod-To-HTML/tags


Web Extensions Declarative Net Request WIP

14942

 Quality gate: NA
Details

 Quality gate: NA
Details

 Quality gate: NA
Details

 Quality gate: NA
Details

 Quality gate: NA
Details

 Quality gate: NA
Details


any way to change the batch size?

running this on a comfyui backend on my pc, only thing i can change about the number of mages is the number of batches
(this could be a planned feature but i cant tell)

Yes, I do have plans for batching that's a little fancier than just "add a param about it" (tldr, automatically determine batch size based on available resources, available backends, call rate, expected performance results, etc. such that you just generate as many images as you want and it automatically uses an ideal batch size without you ever having to worry about it).
That plan however is going to be delayed for a while as there's much higher priority things to do, so I've gone ahead and just added a simple batch size param, it's under the "Display Advanced Options" checkbox (imo 95% of users never have reason to touch this. It's technically a little bit faster to generate in batches, but usually not by much. You can get a better performance uplift by turning previews off lol).
Note that the seeds displayed in the UI from a comfy backend with batching will be wrong until this Comfy PR https://github.com/comfyanonymous/ComfyUI/pull/1454 is merged.

since comfy doesn't want to merge the seed fix i just went ahead and fixed it myself in swarm https://github.com/Stability-AI/StableSwarmUI/commit/8e21b9b3567d5dc352b6d4eb4f5eaa4c001508b0


Remove excessive requirement input[type=checkbox]

Hi,
In order to use checklist-model with [btn-checkbox from ui-bootstrap] (http://angular-ui.github.io/bootstrap/#/buttons), I had to use a forked version of checklist-model by commenting these two lines:
if (tElement[0].tagName !== 'INPUT' || tAttrs.type !== 'checkbox') {
    throw 'checklist-model should be applied to `input[type="checkbox"]`.';
}

I can confirm this works like a charm. This requirement seems a little bit excessive. Could it be improved or removed?

I added a check like this
if ((tElement[0].tagName !== 'INPUT' || tAttrs.type !== 'checkbox')
&& (tElement[0].tagName !== 'MD-CHECKBOX')
&& (!tAttrs.btnCheckbox)) {

Hi,
Thanks for your support!
However, the recommended way to use btn-checkbox is to add attribute btn-checkbox or btn-checkbox=""
=> tAttrs.btnCheckbox = ""
=> !tAttrs.btnCheckbox = false
=> So I still can not use checklist-model.
Also, the new version of angular bootstrap (0.14) rename this directive into uib-btn-checkbox which will broke again your directive.
Could you not simply remove this checking?
The developer should know how he can use your directive.
Regards.


Outdated package in conda-forge

The major version release has not reflected in the conda-forge and still point to 0.1.14 with the old link to the repo: https://anaconda.org/conda-forge/fake-useragent
The admins need to take over the ownership of the feedstock and merge the existing PRs here: https://github.com/conda-forge/fake-useragent-feedstock

This is now fixed since the pending PRs to the feedstock were merged. @hellysmile and @danger89 are also added as maintainers for the conda feedstock.


Private Repositories vs. Public Repositories

**Private Repositories vs. Public Repositories.
Describe the solution you'd like
A clear and concise description of what you want to happen. When students create their Arcade project and save it on GitHub, MakeCode automatically created a private repository. Students were automatically creating a private repository for every new project created. However, students had a very difficult time navigating their GitHub accounts and finding their repos. Showing students how to navigate GitHub is a very time-consuming process and required an involved teacher directed process.
By storing student projects in a private repo, student projects were not exposed to the public.  Some issues exist with private repos because the instructor is unable to review the student's work as its being completed.
Public repos vs. private repos. It is a recommendation to make public repos the default for new student projects. This way, student projects could be shared with the public for future interviews and job opportunities. Also, teacher could view student work without being made a collaborator. Simply by having a student's username, the teacher can view the student’s projects.
Additional context
It is a recommendation to make public repos the default for new student projects. As a teacher, you should use this opportunity to connect the GitHub documentation inside MakeCode. You might consider embedding some of the GitHub documentation that explains these critical terms beyond simply allowing the user to select between a private repo and public repo. Since this is a educational platform, as a teacher, I would like to see deeper embedding of documentation for the user to explore inside MakeCode. Essentially, how are you teaching the users about GitHub while keeping the users on GitHub? My students generally do not want to be redirected to a different website, i.e. GitHub. However, they  do want to learn more about these key terms

[ ] We changed the default to be public repoes.


Fix podspec file

s.source_files was referencing a nonexistent directory.

Current coverage is 93.96% (diff: 100%)

Merging #77 into master will not change coverage

@@             master        #77   diff @@
==========================================
  Files            57         57
  Lines          3875       3875
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3641       3641
  Misses          234        234
  Partials          0          0

Powered by Codecov. Last update 85dfee9...5fffa81

Awesome, thanks a lot @rob-brown! 🎉


Add stress-ng to SPECS-EXTENDED

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

Introduce stress-ng into SPECS-EXTENDED. stress-ng is a stress testing tool to exercise the OS & HW and collect metrics. The packaging is initially from Fedora 41.
stress-ng will stress test a computer system in various selectable ways. It was designed to exercise various physical subsystems of a computer as well as the various operating system kernel interfaces.
Signed-off-by: Chris Co<EMAIL_ADDRESS>$ stress-ng --cpu 4 --vm 2 --hdd 1 --fork 8 --timeout 2m --metrics
stress-ng: info:  [11227] setting to a 2 mins run per stressor
stress-ng: info:  [11227] dispatching hogs: 4 cpu, 2 vm, 1 hdd, 8 fork
stress-ng: metrc: [11227] stressor       bogo ops real time  usr time  sys time   bogo ops/s     bogo ops/s CPU used per       RSS Max
stress-ng: metrc: [11227]                           (secs)    (secs)    (secs)   (real time) (usr+sys time) instance (%)          (KB)
stress-ng: metrc: [11227] cpu              739116    120.00    444.35      0.60      6159.27        1661.11        92.70          6220
stress-ng: metrc: [11227] vm              1539689    120.01    217.38      4.07     12830.06        6952.78        92.27        149456
stress-ng: metrc: [11227] hdd              163840    124.04      1.79      8.40      1320.82       16086.66         8.21          1996
stress-ng: metrc: [11227] fork             953424    120.00    123.18    152.28      7945.19        3461.25        28.69          2508
stress-ng: metrc: [11227] miscellaneous metrics:
stress-ng: metrc: [11227] hdd                    0.00 MB/sec read rate (harmonic mean of 1 instance)
stress-ng: metrc: [11227] hdd                 1520.88 MB/sec write rate (harmonic mean of 1 instance)
stress-ng: metrc: [11227] hdd                 1520.88 MB/sec read/write combined rate (harmonic mean of 1 instance)
stress-ng: info:  [11227] skipped: 0
stress-ng: info:  [11227] passed: 15: cpu (4) vm (2) hdd (1) fork (8)
stress-ng: info:  [11227] failed: 0
stress-ng: info:  [11227] metrics untrustworthy: 0
stress-ng: info:  [11227] successful run completed in 2 mins, 4.05 secs

Does this affect the toolchain?

NO
Associated issues

#9903

Test Methodology

Pipeline build id: 620070

Style change is cosmetic in nature and the previous buddy build included libbsd-devel BuildRequires.
Just to be sure, new Buddy Build queued and passed: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=620070&view=results


forward MTProto

Hi
I need my proxy forward chat protocols to another MTProto proxy , Is it possible?
Telegram is blocked in my area and only working with MTProto, In the other hand i have to use my openvpn proxy and need the openproxy detect chat protocols and forward them to the MTProto proxy.
Regards

Your issue has nothing to do with this project.


What are article types used for in preprints?

In preprints almost everything is a New Results article. Is this correct?

Before the restructure we'd planned not to include an article type selection or label until additional type are required, which has not been decided yet.
BioRxiv has New Results, Confirmatory Results and Contradictory Results, which makes sense to indicate to readers. And I also imagine we could see value in something similar to short reports to indicate that it's not as complete as the default type. Also perhaps allowing a separate type for new methods or protocols in future could be useful.
All speculation at this point though.


Add documentation for using interactive tools with uv run

e.g., in a project,
uv run --with ipython ipython

or, with uvx (#6272)
uvx --with . ipython

or, without a project,
❯ uvx --with requests ipython
Installed 21 packages in 53ms
Python 3.12.1 (main, Jan  7 2024, 23:31:12) [Clang 16.0.3 ]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.26.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: ...

What about without a project but with inline metadata?
like: https://docs.astral.sh/uv/guides/scripts/#declaring-script-dependencies


Re-order CI steps

The tests are the most likely thing to fail, so let's run those first, so we get earlier feedback.

Split into two jobs (ignore the Windows failures, see https://github.com/Shopify/ruby-lsp-rails/pull/365)
Once this is merged we probably need to add lint to the required CI statuses in the main branch settings on GitHub.

Once this is merged we probably need to add lint to the required CI statuses in the main branch settings on GitHub.

Done.


[offset] Update RA offset page copy to direct user to old build site

The form on https://app.klimadao.finance/#/offset was deprecated to shrink maintainable code footprint and to encourage users to retire on Carbonmark.
There are still a small subset of users who want to retire using pool tokens asset that they own (for example, retire carbon using BCT). After taking down the offset form in the RA, there were a number of complaints in the #klima-chat channel on discord.
There was an internal discussion is this discord thread.
The product team discussed and agreed to following changes:

[x] Take an older build (https://klimadao-3rkq7ymia-klimadao.vercel.app/#/offset) and update the URL domain to legacy-app.klimadao.finance that points to that branch (@Atmosfearful)
[x]  On the current /offset page (https://app.klimadao.finance/#/offset) add a new paragraph to the text component that says:

For users who want to offset paying with a pool token, please use this site.

https://legacy-app.klimadao.finance/#/offset

https://legacy-app.klimadao.finance/

Hey @Atmosfearful The offset page is not available on this page.

https://legacy-app.klimadao.finance/

Hey @Atmosfearful The offset page is not available on this site.

@Atmosfearful Solved this by forcing a deployment by making a dummy commit (that I reverted afterwards). I imagine that since they were no deployments for the legacy branch the app-legacy domain pointed by default to the main branch?


Auto-merge checkbox is still delayed

Testing #4647

Enable githubPullRequests.setAutoMerge
Attempt to create PR, notice the checkbox is set
Cancel the PR creation
Attempt to create PR again, checkbox only becomes checked after >10 seconds

I'm not able to reproduce this. What version of the extension were you using?

I still haven't been seeing this, and now https://github.com/microsoft/vscode-pull-request-github/commit/0118be81c2fb7292ee3aa7d733ff3ef8bd531f0c should also prevent this. Let me know if you still see it after today!


Views not updating

I just created my first iced application, but my view doesn't update when I hit the "root" button and the "next" button.
source code

You seem to be cloning the steps and assinging them to a View variant, these cloned steps are then used in the view logic.
However, you call advance on the original steps, which are not used anywhere in your view logic.

Ah, I see it now. I'm still a noob at this haha, thanks for helping me out!


feat: add low cost browser text to speech output using WebSpeechAPI

Purpose
This PR adds the optional feature to enable the built into browsers text to speech optional feature by setting the USE_SPEECH_OUTPUT_BROWSER boolean.

Does this introduce a breaking change?
When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.
[ ] Yes
[x] No

Does this require changes to learn.microsoft.com docs?
This repository is referenced by this tutorial
which includes deployment, settings and usage instructions. If text or screenshot need to change in the tutorial,
check the box below and notify the tutorial author. A Microsoft employee can do this for you if you're an external contributor.
[ ] Yes
[x] No

Type of change
[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Code quality checklist
See CONTRIBUTING.md for more details.

[x] The current tests all pass (python -m pytest).
[-] I added tests that prove my fix is effective or that my feature works
[-] I ran python -m pytest --cov to verify 100% coverage of added lines
[-] I ran python -m mypy to check for type errors
[x] I either used the pre-commit hooks or ran ruff and black manually on my code.

todo: test, add docs

Thank you! Code looks good from a first glance, I'll check out the branch myself locally and see if I notice anything while using it.

I tried to make the naming match your AzureSpeechOutput in case you want to merge it in one file and do the checks there.
IMHO, the way it is now is cleaner to customize as the features doesn't overlap.

Weird, I did run pre-commit run --all-files.


Update adopters add new company.

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Update adopters add new company.
Special notes for your reviewer:

Thanks for contribution @SAMZONG . Need to sign DCO https://github.com/karmada-io/website/pull/188/checks?check_run_id=8566559638 first.


[SDA-7879] Incorrect flags in message and hidden for upgrade roles

Related issue: https://issues.redhat.com/browse/SDA-7879
What
The correct flags should be prompted
Why
User was prompted with incorrect flag

/lgtm


(Generic) Auth support in frontends

What does this PR do?
Adds functionality to have (all) authentication methods configurable at the front-end layer (instead of the entrypoint level). Support available in "File" and "Kubernetes" backends.
Motivation
Need to have fine grained "service level"/"frontend" access control, most common request is to use "Forwarder" authentication on a per frontend basis; Full discussion in https://github.com/containous/traefik/issues/2162
More
Added/updated tests
[ some ] Added/updated documentation
I updated docs for the providers "File" and "Kubernetes" since these are my primary use cases. Updating the remaining providers should be easy but not something i can not allocate time for.
Additional Notes
Not sure how to best handle the existing BasicAuth logic. For now i left BasicAuth as is since any changes would break configuration contracts currently in place. This means that Basic Auth can be configured in 2 places (adding confusion for users).
For Kubernetes I moved BasicAuth to use the new (generic) auth method as it didn't change the configuration contract (k8 annotations)
I'm feeling hesitant going further with this PR without maintainer feedback.
I have updated but not rebuilt the docs.

Please note that the integration tests run (at least in my environment) but the test-server seem to have had issues:
https://semaphoreci.com/containous/traefik/branches/pull-request-3460/builds/1
if [ -n "$SHOULD_TEST" ]; then ci_retry make pull-images; fi
=>
Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?
Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?
make: *** [pull-images] Error 123
make pull-images failed, attempt 3/3```

@Zatte
---> Making bundle: validate-golint (in .)
Errors from golint:
provider/kubernetes/builder_configuration_test.go:224:25: func parameter forwardUrl should be forwardURL
testhelpers/config.go:147:1: comment on exported function WithFrontEndAuth should be of the form "WithFrontEndAuth ..."

Please fix the above errors. You can test via "golint" and commit the result.

make: *** [validate] Error 1
make validate failed, attempt 3/3

Thanks @ldez , I fixed those issues and other issues appeared, they do however seem unrelated to the changes in this PR.
FAIL: acme_test.go:160: AcmeSuite.TestOnDemandRetrieveAcmeCertificateWithWildcard

acme_test.go:167:
    s.retrieveAcmeCertificate(c, testCase)
acme_test.go:286:
    c.Assert(err, checker.IsNil)
... value *errors.errorString = &errors.errorString{s:"try operation failed: domain traefik.acme.wtf found instead of *.acme.wtf"} ("try operation failed: domain traefik.acme.wtf found instead of *.acme.wtf")
----------------------------------------------------------------------```

I can dive into this if it is not an already known issue. The acme code seem to be somewhat volatile, is there a preferred branch i can rebase on?

@Zatte Could you rebase on master?
You also have to add the feature for all providers:

[x] k8s
[ ] Docker
[ ] Rancher
[ ] Consul Catalog
[ ] Marathon
[ ] Mesos
[ ] ECS
[ ] KV

Then i assume the current approach is acceptable, and no major architecture changes are planned.
I'll start looking into adding more providers but have limited time to allocate on this PR atm. Contributions welcome.
Todo (I will keep this updated as I progress):

[ ] Rebase
[x] k8s
[ ] KV
[ ] Docker
[ ] Rancher
[ ] Consul Catalog
[ ] Marathon
[ ] Mesos
[ ] ECS

Support for other providers will be in another PR.

Is there code in flight for docker anywhere? I'll do it myself if not but I don't want to duplicate someone's work if so.

@khuey we are working to add the support for all other providers.
A PR will coming soon.

@Zatte I met this error too:
value *errors.errorString = &errors.errorString{s:"try operation failed: domain traefik.acme.wtf found instead of *.acme.wtf"} ("try operation failed: domain traefik.acme.wtf found instead of *.acme.wtf")

Would you share how did you fix it? Thanks very much.

Q. Does this enable using frontend level auth in a TOML file? If yes, then how?


text and write function are buggy

The TEXT (fixed font) or WRITE (variable width) do present bugs.
See picture for more

Hi,
TL;DR: Sorry, I can not help you.
The reason:
I think the issue lies in here. The set_area() function sets the area of the memory region of the display controller to be updated when inputs come in.
    uint8_t bufx[4] = {
        ((x0 >> 8) & 0xFF),
        (x0 & 0xFF),
        ((x1 >> 8) & 0xFF),
        (x1 & 0xFF)};
    uint8_t bufy[4] = {
        ((y0 >> 8) & 0xFF),
        (y0 & 0xFF),
        ((y1 >> 8) & 0xFF),
        (y1 & 0xFF)};

These two arrays specify the area and the offset value 0xFF depends on the display controller's specification. This is not necessarily a universal value, hence you might want to check the user manual. (But that is probably not going to be very helpful.)
The user manual states:

2A00h to 2A01h controls the x0, they claims only SC[9:0] are cared, the rest is no-care. But that does not make any sense if this is accurate and the display's memory is point-to-point to the display pixels since you only have 10 bits and all you can control with it are
512 pixels at most. Hence the SC[10] has to be somehow used, or some magic is used in RAMWR, which is very unlikely to happen. Because this is a limitation for both row and column, if SC[10] wasn't used, you will never have precise control over the 513th pixel.
What I am trying to say is: that I have no idea what is going on...
Tho I have a suspicion. This is just a suspicion.
There are 2 restrictions for CASET and RASET:

(1) SC[9:0] always must be equal to or less than EC[9:0].
(2) The SC[9:0] and EC[9:0]-SC[9:0]+1 must can be divisible by 2.

In the text() function you see:
                uint16_t x1 = x0 + width - 1;
                if (x1 < self->width) {
                    set_area(self, x0, y0, x1, y0 + height - 1);
                    write_color(self, (uint8_t *)self->frame_buffer, buf_size);
                }

width is the font's width and it is always even. When x0 is odd, x1 becomes even, so x1 - x0 + 1 is even. When x0 is even, x1 becomes odd, so x1 - x0 + 1 is even. So we can rule out the second one. Maybe due to some memory magic, the first restriction is not met. However, I cannot see how this could happen.

These two arrays specify the area and the offset value 0xFF depends on the display controller's specification. This is not necessarily a universal value, hence you might want to check the user manual. (But that is probably not going to be very helpful.)

I've been checking and FF is the correct value for RM690B0

There are 2 restrictions for CASET and RASET:

(1) SC[9:0] always must be equal to or less than EC[9:0].
(2) The SC[9:0] and EC[9:0]-SC[9:0]+1 must can be divisible by 2.

I've seen that, but why should RM690B0 an RM67162 act different. Both manuals requires the same. And I never had any problem writing to any location within RM67162 library.
I'm digging a bit in the code to understand

A thread explaining the odd/even problem
https://github.com/Xinyuan-LilyGO/T-Display-S3-AMOLED/issues/2

Ugh, thank you for providing the thread! I finnaly read through that thread, seems like nothing new was mentioned to this issue. The driver uses only 16-bits color565 mode and 8bits also 24bits are not implemented. Tho, it let you to choose 8bits or 24bits, but the color will be wrong. TBH, I only checked the text() function, and the logics should work.
Also, the 2 rules about odd/even restrictions should be met as I have explained before:

In the text() function you see:

            uint16_t x1 = x0 + width - 1;
            if (x1 < self->width) {
                set_area(self, x0, y0, x1, y0 + height - 1);
                write_color(self, (uint8_t *)self->frame_buffer, buf_size);
            }

width is the font's width and it is always even. When x0 is odd, x1 becomes even, so x1 - x0 + 1 is even. When x0 is even, x1 becomes odd, so x1 - x0 + 1 is even. So we can rule out the second one. Maybe due to some memory magic, the first restriction is not met. However, I cannot see how this could happen.

Tho, his explainations makes sense and suit the situation in the picture you provided:

Failure to 'round off' your update regions to meet these criteria will result in the kind of intermittent 45 degree skewing that looks like tearing but is in fact a restriction of the display hardware. Your screen shots look to me like this is what was happening. A full screen update avoids the issue as 536x240 meets these criteria.

Then I double checked the  write() function, which, even though not the exact same expression, but the drawing part is essentially the same thing like the text() function. Which adhere to the 2 rules. It is really weird...

Then I double checked the write() function, which, even though not the exact same expression, but the drawing part is essentially the same thing like the text() function. Which adhere to the 2 rules. It is really weird...

I implemented a Odd/Even correction in both fill_rect and text function.
It does correct the very first bug we had (so no more need on framebuffer=false).
I'll implement the write function in the very next days.
I think it does also fix the issue within the RM67162. As a conclusion, there wasn't any bug in our codes, but only a misunderstanding in the way the RM67162 and RM690B0 handles memory copy.

I've implemented the correction I made to text() function in the write() function but I'm too lazy tonight to test the result.
Files compiled straight (only 2 warning)... that's usually not a good news ;-)

As long as the firmware is built, you can sleep in peace. 😆


Paging in admin views returns to profile edit page

Trying to go to the next page of posts or members returns me to the main profile edit page. The public posts list is fine, though.

@mjgardner Thanks for reporting this! Should be fixed now.


Locative device trackers disappear after restarting HA

Home Assistant release with the issue:

arch
amd64

dev
false

docker
false

hassio
false

os_name
FreeBSD

os_version
11.2-STABLE

python_version
3.7.4

timezone
America/Toronto

version
0.101.0

virtualenv
false

Last working Home Assistant release (if known):
Unsure. Did not use Locative prior to the migration to Integrations config.
Integration:

Locative
Description of problem:
After creating a webhook through the Locative integration config flow, my devices are able to submit webhooks successfully. For each device sending POST requests to the webhook, a new device_tracker is created in HA. These device trackers can be linked to a person to track their state in HA.
The problem is that when HA is restarted, the device trackers that had been previously created no longer exist. This causes Persons to be shown as "unknown" (or "UNK" in badges). I've confirmed that the device trackers get re-created when the next POST request is sent to the webhook. I've also noticed that the entity registry does not list Locative device trackers.
Ideally, device trackers would persist after restarts and should be deleted manually if no longer needed. The state of the device tracker at HA startup should be the last state recorded prior to  HA shutdown.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

No configuration.yaml because Locative is configured through the Integrations page
Traceback (if applicable):

Additional information:

Same problem here.

Same issue here. Any solutions?

If the locate device has a unique id it's possible to implement the same solution as done for gpslogger:
https://github.com/home-assistant/home-assistant/blob/d1ed67f44a148c0c4aaaf7c2b4099cf7a0a2f295/homeassistant/components/gpslogger/device_tracker.py#L47-L64
This issue is a feature request and not really a bug as the integration is working according to current implementation. I'll close hee now.

@MartinHjelmare Why closed?

It's a feature request. We don't track feature requests in our issue tracker.
I've outlined the solution above. A PR with that is welcome.

Vote to get the fix:
https://community.home-assistant.io/t/fix-locative-bug/223844


Object function HTMLElement() { [native code] } has no method 'call'

We are seeing the error Object function HTMLElement() { [native code] } has no method 'call' in production.
Sample user agent Mozilla/5.0 (Linux; U; Android 4.2.2; it-it; GT-I9060 Build/JDQ39) AppleWebKit/534.30 (KHTML, like Gecko) Version/4.0 Mobile Safari/534.30
We tried reproing this with the Android 4.2.2 system browser but to no success. It may be flaky. I believe the consequence of this is that the custom element constructor (or createdCallback) is never called. We do, however, then see connectedCallback being called.
I was hoping that this may ring a bell, even without a repro case.

to start with, HTMLElement is overwritten in V1 so you shouldn't see [native code] in there.
I have an android 4 phone but I'm not sure it's 4.2, it's probably a 4.0 and it was tested and it was green.
It's also a quite pricey phone, despite it's deprecated, so unless I find a GT-I9060 it's very difficult for me to debug.
However, I'll try to see if I can reproduce with any Android 4 I can find.

OK, I have a Geeksphone Revolution on Android 4.2.2 and ApplwWebkit/534.30 and also an Android 4.0.4 with the same version of the browser.
I have gotchas with attribute tests but nothing about the constructor.
At this point I need a page/use-case to test against or I won't be able to reproduce (also I have HTMLElement and its call method in their right place)

@cramforce ?

Sorry, for not responding. As I said, we do not have a repro case. The error happens consistently, but only for a tiny percentage of page loads across a wide variety of AMP pages. Feel free to close it. I was mostly hoping you'd have an idea where to look.

Feel free to send me over one those phones 😄
Btw, maybe switching to the latest would solve the issue? 1.4.1 does early patching in some device, it might fix this issue.

Yeah, we'll definitely upgrade. Thanks!


✨ OpenQASM 3.0 support

This PR replaces the existing OpenQASM 2.0 parser with a new OpenQASM 3.0 parser.
The new parser now builds a syntax tree, where type checking, constant evaluation, and translation to the Quantum circuit.
The parser can handle the following new features:
New Syntax
New syntax for declaring bits, qubit, measure operations. The old syntax (creg, qreg) is still supported.
qubit[8] q;
bit[8] c;

c[0] = measure q[0];
measure q[1] -> c[1];

if (c[0] == 1) {
  x q[0];
}

Gate modifiers
Gate modifiers (inv, ctrl, and negctrl) are now supported. This replaces the c prefix.
See the OpenQASM 3.0 specification for more information: https://openqasm.com/language/gates.html#quantum-gate-modifiers
ctrl @ x q[0], q[1]; // Equivalent to cx q
ctrl(2) @ x q[0], q[1], q[2]; // Equivalent to ccx q;

Classical constant values
The parser now supports classical computation with constant values. This can be used to e.g. define the number of quantum registers.
const uint N = 4;
qubit[N * 2];
x qubit[N * 2 - 1];

Additionally, all features of the previous parser are still supported.
The big features from OpenQASM 3.0 still missing are:

classical computational features such as loops, functions, etc. (see #33)
types such as bools, floats, angles, complex types, etc. (see #30, #32)
pow modifier (#27)

From my side I'm happy -- I will address the CodeQL warnings though.
Regarding your questions which issues will be closed

#29 yes
#31 yes
#30 partly there (type checker, constant eval is there for float, bool; not for angles, time)
#32 no
#33 no
#27 no

From my side I'm happy -- I will address the CodeQL warnings though. Regarding your questions which issues will be closed

Support for OpenQASM 3.0 classical bits and registers #29 yes
Support for OpenQASM 3.0 constant values #31 yes
Support for OpenQASM 3.0 additional datatypes #30 partly there (type checker, constant eval is there for float, bool; not for angles, time)
Support for casting of OpenQASM 3.0 types #32 no
Support for OpenQASM 3.0 looping and branching #33 no
Support for the powering modifier of OpenQASM 3.0 #27 no

Great, many thanks. Updated the list of linked issues accordingly.
I also took the liberty of adjusting the title from WIP to a proper title and update with the latest changes from main.
As a last step before merging: Could you please update the PR description so that it is a little more complete and elaborative on what is actually available now (and maybe also that the OQ2 parser is completely replaced by the new parser).
Afterwards, you can request a (final) review and I should be able to merge 🎉

I've updated the description. Thanks!

And I seem to have screwed up one of the tests with one of the changes above. Could you maybe fix that? 😅


TACO-325 fixing bug where when there are no clientconfigs available, …

…we send a 404 instead of just hanging
If we don't have clientconfigs we send a 404 down the pipe

Coverage decreased (-0.1%) to 43.469% when pulling 268bcb62a9dc18913b403da4ca33e2ac0af28156 on dc/TACO-325 into ec20b0904cc85a73afb12f76f3291bd1d102cf86 on master.


percentage based offset

Provide a means to set the offset as a percent of viewport height, and have that percent be honored after a resize event.

done in 0.3


New deployment and hosting solution for shhh-encrypt.com

Shhh is currently down at shhh-encrypt.com
Need to find a new solution for hosting Shhh at shhh-encrypt.com
Heroku have increased their pricing, and also removed free plans for Postgre...
Investigate what's available as another solution - and build it!
Maybe have a look at infrastructure as code and add this to repo for others to use?

Hi @smallwat3r - working on this!

That's awesome, thanks for looking into this @troydieter !


Switch to sailshq/lodash which patched security vulnerabilities

@fhawkes I wanted to point to released tags in package.json instead of just branches. Updated. can you review again

@fhawkes I wanted to point to released tags in package.json instead of just branches. Updated. can you review again


extract: true produces sprite with broken structure (all symbols and gradients included in def)

First I used loader without extraction and it works perfect, just index.html has too heavy - plus  600kb.
Structure of svg sprite in this mode was:

All the symbols on upper level. all the gradients inside defs inside symbols - that is OK.
Then I set extract: true and resulted sprite.svg has a broken structure = symbols and gradients on same level inside def. As a result - svg can't get the gradients:

Why the sprite structure is changed?
How I could return extracted sprite to normal structure?

Node.js version: 14.2.0
webpack version: 4.30.0
svg-sprite-loader version: ^4.3.0
OS type & version: macos 10.15.4

Sincerely, Andrey.

I resolved the issue with recommendation  from this issue, with direct DOM manipulation:
function arrayFrom(arrayLike) {
  return Array.prototype.slice.call(arrayLike, 0);
}

const moveSymbolsDown = (svg) => {
  arrayFrom(svg.querySelectorAll('symbol')).forEach((symbol) => {
    symbol.parentNode.after(symbol);
  });
};

Just keep in mind, when you inject sprite in document do not use "display: none" for hiding the spritestorage, because this will lead to losing gradients. Hide it with positioning, for instance.


Issue on docs

Path: /documentation/develop
Can we get instructions for running the desktop version locally?
My (fortune 100) company is looking at contributing features / fixing bugs in the desktop client and it would help us to have clear instructions for running locally

Hi @jimifredjr , the desktop app is currently in the alpha stage and has some non-open source bits, our team is working on making the app more compatible with both cloud and self-host instances. Once that's done, we'll release a contributing guide dedicated to setting up the desktop app locally for contribution.


Update CentOS makefiles and spec file

@dnobori san, could you please merge that?
Same problem stays for Debian files — I'm maintaining a PPA (https://launchpad.net/~paskal-07/+archive/ubuntu/softethervpn) and minor version is still 9412 — I've already got questions why version is so old, while actually it's built from latest github commit.

I'll appreciate a lot if this will be merged today as well, @dnobori.

Thank you so much for your contribution to enrich the SoftEther VPN source code.
Your patch has been merged on the main source-tree of SoftEther VPN.
As a token of our gratitude, your GitHub username has been added on the AUTHORS.TXT file and on the header of the related source file.
Please see: https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT
Thanks again for your contribution.


bug(Scrolling): Scroll Tracking Causes IFrames to Flicker

Reproduction
Scroll down in this article until you reach the Gist rendered with an IFrame.
https://developer.fireflysemantics.com/guides/guides--recreating-the-ngrx-demo-app-with-firefly-semantics-slice-state-manager
Notice the flickering.
It does that due to the scroll tracking needed to show and hide the toolbar.
Expected Behavior
Ideally no flickering.

Angular:
CDK/Material: 11
Browser(s): Firefox, Chrome
Operating System (Ubuntu):

It seems like something is calling document.write on each scroll which is causing everything to re-render. Closing since this appears to be an issue with your own code.


#2665 - fix maxbitrate property being ignored on Android

Update the documentation

No new property added

Update the changelog

updated change log

Provide an example of how to test the change

Test steps can be found on issue page #2665

Focus the PR on only one area

single fix

Describe the changes

Inside ReactExoplayerView.java videoLoaded(), added guards to check type before allowing track selection overwrite. If type is null, don't do a track selection overwrite as that would cause a default type=auto overwrite to happen where it allows all available tracks to be used for ABR.

@evkwan Thank you! Change looks good but how does it relate to maxBitRate?

@hueniverse when MaxBitRate is set, at first ExoPlayer's AdaptiveTrackSelection will honour the value set and only select tracks that matches the bitrate threshold. However, once the function videoLoaded() runs, that line of setSelectedVideoTrack will run with videoTrackType == null (here I am assuming if user puts a MaxBitRate value they won't be wanting to put a specific track selection, as track selection locks the player to play only 1 track and defeats the purpose of the threshold value set!).
When setSelectedVideoTrack() runs with type null, it cause all tracks to be selected instead. You can see some screenshots from the issue thread here.

Thanks @evkwan.
Hopefully we'll get two reviews quickly and get this merged.

Can we get one more person to verify the fix and approve it?

Hi @hueniverse , sorry for the delay in response. In the process I sort of messed up the rebase a bit 😓  but I think it should be fixed now. I have added one update to update the changelog pointing to the PR number instead of issue number.  Please let me know if this is OK for merge. Thanks and sorry for the inconvenience.


Window cursor with negative colors

Subject of the issue
Whenever I try to load a window icon over the Windows platform, using a 32-bits PNG image, the resulting colors are inverted. I determined that the color format of the image I use is RGBA meanwhile the correct format for a window cursor is BGRA. I then inverted the colors through a loop that changes every pixel in the image and the problem was fixed.
In the documentation there's written that the required format is RGBA, but I find it wrong. I don't know if it's an issue related to how the Windows API manages cursors or if I'm totally wrong.
Your environment

Windows 10 x64 Version 10.0.17134 Build 17134
SFML from git master
Microsoft (R) C/C++ Optimizing Compiler Version 19.14.26433 for x64
No special compiler flags

Steps to reproduce
#include <SFML/Graphics.hpp>

int main()
{
    sf::RenderWindow window(sf::VideoMode(1280, 720), "Minimal, complete and verifiable example");
    window.setFramerateLimit(60);

    // Load the icon
    sf::Image image;
    sf::Cursor cursor;
    if (!image.loadFromFile("icon.png"))
        return -1;
    if (!cursor.loadFromPixels(image.getPixelsPtr(), image.getSize(), { 0, 0 }))
        return -1;
    window.setMouseCursor(cursor);

    while (window.isOpen())
    {
        sf::Event event;
        while (window.pollEvent(event))
        {
            if (event.type == sf::Event::Closed)
                window.close();
        }

        window.clear();
        window.display();
    }
}

Expected behavior
The window cursor should be displayed with the correct image colors.
Actual behavior
The cursor image appears as if the colors were negated.
Fix
// Swap the color format from RGBA to BGRA
for (unsigned x = 0; x < image.getSize().x; ++x)
{
	for (unsigned y = 0; y < image.getSize().y; ++y)
	{
		sf::Color pixel = image.getPixel(x, y);
		image.setPixel(x, y, { pixel.b, pixel.g, pixel.r, pixel.a });
	}
}

Example cursor:

Branch bugfix/windows-cursor-colors should fix this, give it a spin, if you can.

While the proposed fix might fix it, I still wonder where the information can be found that BGRA is expected?

Fixed with #1465


Excel crashes when printing XLS from https://sheetjs.com/demo/table.html

Steps to produce the problem:

From https://sheetjs.com/demo/table.html click [Export to XLS!]
For comparison, export XLSX and XLSB too.
Open the file in Excel 365. In Excel, click [Enable Editing].
Ctrl-P to print. ** CRASH **
OR
From https://sheetjs.com/demo/table.html click [Export to XLS!]
Open the file in Excel 2010. Excel title bar shows 'compatibility mode'.
Ctrl-P to print. ** CRASH **

The same process does NOT crash if I choose XLSX or XLSB.
The problem details from Excel show fault module mso20win32client.dll for Excel 365 or MSVCR90.dll for Excel 2010.
Many thanks!
Bug seen on:
Windows 7, Chrome, Excel 365
Windows 7, Chrome Excel 2010 32-bit

Also seen on Windows 10.

the same bug.
Excel crashes when printing XLS
Is there any solution？@lwinsor @SheetJSDev

One-line fix (feel free to send a PR): https://github.com/SheetJS/sheetjs/blob/master/bits/78_writebiff.js#L248
-	if(b8 && _WB.Views) write_biff_rec(ba, "Window2", write_Window2(_WB.Views[0]));
+	if(b8) write_biff_rec(ba, "Window2", write_Window2((_WB.Views||[])[0]));

it work for me.Thanks @SheetJSDev

0.16.8 update added the fix.  the XLS file exported by the page does not cause Excel to crash when printing.


Flask server stops responding to new HTTP connections

After a day or two, my Flask app stops noticing new connections; strace says:
mk@pling /proc/31539/fd> strace -p 31539 -e '!stat'
Process 31539 attached - interrupt to quit
select(0, NULL, NULL, NULL, {0, 805175}) = 0 (Timeout)
select(0, NULL, NULL, NULL, {1, 0})     = 0 (Timeout)
select(0, NULL, NULL, NULL, {1, 0})     = 0 (Timeout)
select(0, NULL, NULL, NULL, {1, 0}^C <unfinished ...>
Process 31539 detached
(note that the stat() calls between each select() have been omitted).
So select() is being called without any fds, basically as a blocking primitive, but I seem to have the following fds:
lsof -p etc:
python  31539   mk    3u  IPv4 141934535      0t0      TCP *:5000 (LISTEN)
python  31539   mk    4u  IPv4 142787301      0t0      TCP <IP_ADDRESS>:5000-><IP_ADDRESS>:60000 (ESTABLISHED)
So it should by rights be select()ing on fd 3 at the very least.

Ah, tiresomely, I forgot -f to strace - so the above output is misleading

Are you using the development server in production? app.run can only serve one connection at a time, it's very easy to freeze the server by leaving one connection open.

And I have to use some external nonsense like gunicorn to work around this?

Yes.
Flask is a web framework, not a server. Not including a scalable server sacrifices a bit of usability for flexibility. I don't know your background, but that distinction is not unique to Flask.

thanks


LocLabels not showing

I have a problem with the version from XrmToolbox, it does not show the text of the LocLabels but rather a reference to the LocLabel. See highlighted block in screendump.
CRM version 2016, <IP_ADDRESS>9 (DB <IP_ADDRESS>9)
XrmToolbox version 2016.11.4
Ribbon Workbench version <IP_ADDRESS>

Could you open a support ticket at ribbonworkbench.uservoice.com
Thanks


When message value is JSON it does not get delivered to the consumer

Version of Ruby: ruby 2.2.3p17
Version of Kafka: 0.10
Version of ruby-kafka: 0.3.15

Steps to reproduce
Consumer is not receiving message when the message is sent as a JSON.
Below is the example which I been using to test:
client = Kafka.new(
  seed_brokers: Rails.configuration.kafka_brokers,
  ssl_ca_cert: Rails.configuration.kafka_ca,
  ssl_client_cert: Rails.configuration.kafka_cert,
  ssl_client_cert_key: Rails.configuration.kafka_private_key
)
producer = client.producer
data = JSON.dump({'foo': 'bar'})
producer.produce(
  data,
  topic: "#{Rails.configuration.kafka_topic_prefix}user-access"
)
producer.deliver_messages

Expected outcome
The consumer should receive the message
b"{'foo': 'bar'}"
Actual outcome
After calling the producer.deliver_messages I would expect the consumer to receive the message, but unfortunately, the consumer does not receive the message.
However, when I try to send an another message. The consumer receives the first and the second message. Not sure why this is happening.
producer.produce(
  'hello',
  topic: "#{Rails.configuration.kafka_topic_prefix}user-access"
)
producer.deliver_messages

Expected outcome
The consumer should receive the message
b"hello"
Actual outcome
b"{'foo': 'bar'}"
b"hello"

Are you sure that you're consuming all the partitions in the topic?

Yes, Its just weird that I have to produce a second message where message body is a string for the first message to reach the consumer.

Kafka has no knowledge of the message body, so I can't see why it should matter that it's JSON. I'm unfortunately not able to reproduce the bug currently, as I'm traveling. Can you try writing a failing spec?

@dasch Thanks for taking a look at the issue.
I have got my ruby producer to send JSON body and python consumer to receive JSON body.
I was memoizing the Kafka ruby client and producer in my Rails app, I removed the memoization, and everything seemed to work as expected.

I'd love to see how that memoization was implemented, as well as which web server you were using.


Remove 'primary' class from tab counter labels

Using the primary color for each label counter makes the use of color redundant, as well as suggesting this is a call to action. Use the base grey color instead.

Hmm… I'm not sure about this PR:
While I can see your point, I also think that the current color is one of the most recognizable elements of the entire theme -
whenever I think of the UI, this tab appearance is the first thing that comes to mind.
So, I'm not sure if it would be a good idea to eliminate this unique characteristic.

Long story short:  I'll stay neutral towards this PR, I won't approve it but I am also fine if it gets merged.

This is how it looks like on the default theme.

I like this as a fix to my suggestion. Right now the count draws more attention than the tab title. And even the tabs shouldn't draw more focus than action buttons, headings, etc.

I'm gonna defer to @silverwind for their review of this

as well as suggesting this is a call to action

That's not really true, Gitea uses red for that (see notification number count badge).
I probably like better as it is now so I will also abstain

I would really like to see this merged in, for multiple reasons:

Primary colors should be saved for call-to-actions (which are -or should be, usually limited to one per page/section).
These labels are simple count indicators, they don't perform any action.
The entire tab can be clicked, no need to highlight the label only.
These indicators are often present in multiple tabs (at least in Issues, PRs, and Projects), leading to visual noise.

This is not totally related to this particular PR so feel free to ignore, but this hardcoded label-colors exists in other areas as well.
e.g. the repositories list in the dashboard. Labels there have the class grey (which is not even needed since the color is taken from context). This makes it hard to theme without getting into !important situations.

Gitea uses red for that [call to action]

I'm confident most users feel called to action by the apparent "unread count" in the tabs, regardless of the intent of the current styles.

There is precedent for gray-count-in-heading in the current dashboard:

I think some folks are used to the way it looks and feel funny when it changes. But I think that feeling will change quickly. I really think removing a bright accent from secondary/tertiary information is better for UX and cognition.

I would also agree this change  is good. A strong reason is: other  sites also do so (say, like GitHub)

Probably a good change, but I see that those grey shades need to be tweaked a bit, but can be done separately.

please update the branch to current head

Needs updating again :)


Use PXSourceList without xib or storyboard

Anyone can help me,using PXSourceList without xib or storyboard.

It show nothing.


Linux support

Adding esbuild for linux, unplugging the open package to solve a crash on start related to yarnpkg/berry/issues/856

More PnP fun 🙃


0.5.0 appears to break Fission function input

Update: this is caused by a transient bug during the build of the release of 0.5.0. This is solved by rebuilding at tag 0.5.0. The new images still need to published on Dockerhub

Did we ever figure out what the issue was?

@soamvasani - no, I haven't been able to reproduce it. It looks like the binary was somehow affected by the build environment (my laptop vs. in-Docker). I'll enforce building in-Docker for releases from now on. ;)


Performance issue: Frequently GC

Tks for your nice code first, but...
while running your demo, I found the memory usage like this (tested on Genymotion 2.4 with intel acceleration, x64, android 5.1.0 on nexus 4):

Obviously, it's cause by:

I have fix that problem, pls see you pull request.
The work I did is:
1, Use a MutableForegroundColorSpan to maintain the foreground color, instead new a ForeGroundColorSpan every time.
2, Change the MutableForegroundColorSpan object's color when needed.
Here is memory usage after my fix(tested on Genymotion 2.4 with intel acceleration, x64, android 5.1.0 on nexus 4):

Fixed with  c62c38fd


Windows Version Notes Incomplete

[Enter feedback here]
Hello,
I have found that the Common Issues -> Windows Version list on this page is incomplete.
Specific to Windows 10 Enterprise, the required version to use .appinstaller files is actually 22H2 (19045.2006). When attempting to use .appinstaller files on Windows 10 Enterprise with versions 21H2 (19044.1288) and lower the following error message is received:
This .appinstaller file is using a new schema which is not valid on this OS version. Please update to the latest version of Windows 10.

Now, while this message is technically correct, it does not match with what the documentation suggests. And further, changing the schema does not resolve the issue. Using schema 2017, 2017/2, or 2018 will not resolve this.
My tests were as follows:

Install new copy of Windows 10 Pro 20H2, success
Install new copy of Windows 10 Enterprise 21H2, failure
Install new copy of Windows 10 Enterprise 22H2, success

Please either update this page or similar documentation pages to note that there is a special case for Windows 10 Enterprise.
Thank you.

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

ID: fd3a8483-a56e-7f3d-8c25-1d8b18943342
Version Independent ID: 3b8e451b-95b5-3d26-c7ec-160fef01d85e
Content: Troubleshoot installation issues with the App Installer file - MSIX
Content Source: msix-src/app-installer/troubleshoot-appinstaller-issues.md
Product: windows
Technology: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

Windows 10 Pro 20H2

Windows 10 Enterprise 21H2

Windows 10 Enterprise 22H2


Polish up usage messages for test name pattern mode

Summary
Polish up usage messages for test name pattern mode.

Test plan

I think this is a good way to solve this for now. Great work!


Add .getProps() to useForm

Is your feature request related to a problem? Please describe.
I want easy access to the defaultValues I've passed in useForm's props
Describe the solution you'd like
A new .getProps() method:
const form = useForm({
  mode: 'onBlur'
  defaultValues: { foo: 'bar' }
})

// what I want:
form.getProps().defaultValues; // { foo: 'bar' }
form.getProps().mode; // 'onBlur'
// and so on

Hi @kaleabmelkie
Could you just hoist your default values to the top?
const defaultValues = { foo: 'bar' };

const form = useForm({
  mode: 'onBlur'
  defaultValues: { foo: 'bar' }
})

Yes, I have considered doing that. I just thought it would be cleaner to have something like .getProps, esp. in typescript projects.
Generally, I would have to do something like this in a .tsx file:

I took the liberty to submit a pull request for this (#185) ... I hope it's okay.

Thanks for the PR. I tempt to reduce more methods if I can in order to keep the API list small. I don't see much value on this getProps() method at this stage. but let's get the other two core contributors' thoughts as well @barrymay @stramel : what do you guys think of this feature?

I would like to mention one more advantage for this design: It allows access to the default options in some scenarios:
Example:
import { Props } from 'react-hook-form/dist/types'

const config: Props<T> = { defaultValues: { foo: 'bar' } };

const form = useForm<T>(config)

config.mode // undefined
config.submitFocusError // undefined

Instead, it could be:
const form = useForm<T>({ defaultValues: { foo: 'bar' } })

config.getProps().mode // 'onSubmit'
config.getProps().submitFocusError // false

@bluebill1049 I think I'm against this change. It expands the public API surface, bundlesize, and code that we have to maintain to handle something that is relatively arbitrary without this change. The only thing that I see being useful of that config object would be the defaultValues.
const defaultValues = { foo: 'bar' }

const form = useForm({ defaultValues })

That was what I thought as well. 👍


Activities missing technical name

Why are there Activities with a friendly name but not a technical name?
if there's no technical name how can we indentify the activity in the log?
are these deprecated activities or in the roadmap and not yet implemented?

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

ID: a0e8bce0-95c1-07f7-d3a3-a1c22bd21f29
Version Independent ID: dddd1e30-4a88-1293-1264-3214cc4b6dc8
Content: Track user activities in Power BI - Power BI
Content Source: powerbi-docs/admin/service-admin-auditing.md
Service: powerbi
Sub-service: powerbi-admin
GitHub Login: @kfollis
Microsoft Alias: kesharab

Sorry, I don't see the activities you mention in the article. It could be that the article has been updated since you submitted.


Readonly attribute not showing up in m4 output

Describe the issue
readonly is not showing up as true even though it's set in the swagger: https://github.com/Azure/azure-rest-api-specs/blob/b4b98c3f08e52adf68f21ea2c01b1e74f3a370ea/specification/keyvault/resource-manager/Microsoft.KeyVault/stable/2019-09-01/keyvault.json#L1665
I'm also confused because it appears the readonly attribute is defined on the schema in the specification, but when I look at various code model outputs, the readonly attribute is defined on the property, not the schema itself.

** REQUIRED **
ModelerFour version: 4.10.252
AutoRest-core version: 3.0.6258
CommandLine : autorest ../azure-rest-api-specs/specification/keyvault/resource-manager/readme.md --python --python-sdks-folder=../azure-sdk-for-python/sdk --output-artifact=code-model-v4-no-tags --use=.
OpenAPI/Swagger file: https://github.com/Azure/azure-rest-api-specs/blob/b4b98c3f08e52adf68f21ea2c01b1e74f3a370ea/specification/keyvault/resource-manager/Microsoft.KeyVault/stable/2019-09-01/keyvault.json

@iscai-msft  -- this got fixed a while back, right? Can you confirm?

@fearthecowboy: yes, apologies, I will go through the fixed please confirm now as well


AWS X-Ray integration for Lambda/Python

Excerpt from the module docs to explain what this is about:

This module bridges the gap of missing Python support in the AWS X-Ray Lambda integration.
After enabling active tracing under Advanced settings on the Configuration tab of your Lambda function in the AWS Console (or using the update_function_configuration API call), you can mark any function or method for tracing by using the @trace_xray_subsegment decorator.
This module also provides wrappers for boto and requests so that any AWS API call, or HTTP request will be automatically traced by X-Ray, but you have to explicitly allow this behavior by calling monkey_patch_botocore_for_xray and/or monkey_patch_requests_for_xray. The best place to do this would be the main handler module where the Lambda entry point is defined.

This looks awesome! Thanks @szilveszter ! 👍

@szilveszter - Will you add a quick docs section to the README for this? It can just be similar to what is there for the boto and requests wrappers.

Thanks for putting this together!! I have the same request as Ryan. Once thats in place I'll happily merge/publish

Thanks for the feedback, I just updated the README, could you check it, please?


pypi & artifact naming

the naming scheme for published artifacts seems to have switched to using _ instead of -
  llm-claude-3-0.3.tar.gz -> llm_claude_3-0.4.1.tar.gz

Is this intended, or will it be reverted in a future release?
I'm the maintainer for your FreeBSD llm-* related ports, and I'm ok to rename the
port to match this new scheme, if that's intentional, but if it will flip back
in a future release I'll just use a small workaround until the next bump.

Not sure what happened here, sorry. Might have been because I switched to using this mechanism for publishing:
https://github.com/simonw/llm-claude-3/blob/b5b19b7644afd671c5d177a5cd980b7f85065a5c/.github/workflows/publish.yml#L44-L51
I do not plan to make any future changes to that.


KeyError: 'resolution' while opening file

Hi. I'm trying to load the samples from the lpd-5-cleansed dataset using pypianoroll.load(path) but getting the following error:
File "/home/xx/anaconda3/envs/xx/lib/python3.6/site-packages/pypianoroll/inputs.py", line 51, in load
    resolution = info_dict["resolution"]
KeyError: 'resolution'

It looks like the resolution information is missing.

Never mind, upgrading pypianoroll solved the issue.

Never mind, upgrading pypianoroll solved the issue.


[Event Request] Application “Tests-SCM” Codeunit 137054 "SCM Supply Planning"

[Event Request]Application “Tests-SCM”, Codeunit 137054 "SCM Supply Planning"
Could you please insert the OnTestInitialize Event at the beginning of the procedures DemandSupplyWithReserveCancelReservationFromSupply and DemandSupplyWithReserveCancelReservationFromDemand?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Emits once when I cast

Hello Jake and team. Thanks for the great work done.
Noticed something.
Casting with as emits only once and stops emitting: For example.
RxTextView.textChanges(phoneNumberEditText)
                .skipInitialValue()
                .subscribe {text ->
                    purchaseButton.isEnabled = purchaseViewModel.isNumberValid(text as String)
                }

But when i do this, it works fine
RxTextView.textChanges(phoneNumberEditText)
                .skipInitialValue()
                .subscribe {text ->
                    purchaseButton.isEnabled = purchaseViewModel.isNumberValid(text.toString())
                }

Probably it's because your cast throws ClassCastException (cause CharSequence isn't necessarily a String), observable emits error notification and stops emitting items.