[geographiclib] build failure

Host Environment

OS: Windows 11 x64
Compiler: Visual Studio 2022 - 17.2.3

To Reproduce
Steps to reproduce the behavior:
./vcpkg install geographiclib:x64-windows
I believe the reason the compile fails is due to the version of the compiler.  The author of this library verified that version 17.1.1 does not exhibit this failure and builds successfully.  The normal log output from the ./vcpkg install geographiclib:x64-windows command is shown below and I also include the log from the ./vcpkg install geographiclib:x64-windows --debug as an attachment (build.log).
The failure in the debug log (shown as the first log snippet below) seems to indicate insufficient memory - however this machine has 16G of ram and another machine that also shows this error has 32G of ram - so I doubt ram is the problem.  All my machines are using 17.2.3 of Vs 2022 and I do not have access to a machine with an earlier version of the compiler to verify that this is the issue
build.log
.
Failure logs
CMake Warning at scripts/cmake/vcpkg_execute_build_process.cmake:60 (message):
  Please ensure your system has sufficient memory.
Call Stack (most recent call first):
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
  ports/geographiclib/portfile.cmake:35 (vcpkg_cmake_install)
  scripts/ports.cmake:146 (include)

[build.log](https://github.com/microsoft/vcpkg/files/8885108/build.log)

PS C:\temp\vcpkg> .\vcpkg.exe install geographiclib:x64-windows
Computing installation plan...
A suitable version of cmake was not found (required v3.22.2). Downloading portable cmake v3.22.2...
Downloading cmake...
  https://github.com/Kitware/CMake/releases/download/v3.22.2/cmake-3.22.2-windows-i386.zip -> C:\temp\vcpkg\downloads\cmake-3.22.2-windows-i386.zip
Extracting cmake...
The following packages will be built and installed:
    geographiclib[core]:x64-windows -> 2.1
  * vcpkg-cmake[core]:x64-windows -> 2022-05-10#1
  * vcpkg-cmake-config[core]:x64-windows -> 2022-02-06#1
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
A suitable version of powershell-core was not found (required v7.2.3). Downloading portable powershell-core v7.2.3...
Downloading powershell-core...
  https://github.com/PowerShell/PowerShell/releases/download/v7.2.3/PowerShell-7.2.3-win-x86.zip -> C:\temp\vcpkg\downloads\PowerShell-7.2.3-win-x86.zip
Extracting powershell-core...
Restored 0 packages from C:\Users\jcoffey\AppData\Local\vcpkg\archives in 520.7 us. Use --debug to see more details.
Installing 1/3 vcpkg-cmake:x64-windows...
Building vcpkg-cmake[core]:x64-windows...
-- Installing: C:/temp/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_configure.cmake
-- Installing: C:/temp/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake
-- Installing: C:/temp/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake
-- Installing: C:/temp/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/vcpkg-port-config.cmake
-- Installing: C:/temp/vcpkg/packages/vcpkg-cmake_x64-windows/share/vcpkg-cmake/copyright
-- Performing post-build validation
-- Performing post-build validation done
A suitable version of 7zip was not found (required v21.7.0). Downloading portable 7zip v21.7.0...
Downloading 7zip...
  https://www.7-zip.org/a/7z2107-extra.7z -> C:\temp\vcpkg\downloads\7z2107-extra.7z
Extracting 7zip...
Stored binary cache: C:\Users\jcoffey\AppData\Local\vcpkg\archives\b9\b9c5cc0c396dcb1f72445cad3fefa70631ad25593055e3d93c73bfda26591ef9.zip
Elapsed time to handle vcpkg-cmake:x64-windows: 2.677 s
Installing 2/3 vcpkg-cmake-config:x64-windows...
Building vcpkg-cmake-config[core]:x64-windows...
-- Installing: C:/temp/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/vcpkg_cmake_config_fixup.cmake
-- Installing: C:/temp/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/vcpkg-port-config.cmake
-- Installing: C:/temp/vcpkg/packages/vcpkg-cmake-config_x64-windows/share/vcpkg-cmake-config/copyright
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\jcoffey\AppData\Local\vcpkg\archives\48\48f6e850b103b8a001ffaa65f9d9aceed645c5f222abd14fd13fc68de95daf4a.zip
Elapsed time to handle vcpkg-cmake-config:x64-windows: 432.4 ms
Installing 3/3 geographiclib:x64-windows...
Building geographiclib[core]:x64-windows...
-- Downloading https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=cfhcable;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=pilotfiber;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=gigenet;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=versaweb;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=ayera;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=netactuate;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=phoenixnap;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=astuteinternet;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=freefr;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=netcologne;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=deac-riga;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=excellmedia;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=iweb;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=jaist;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=jztkft;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=managedway;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=nchc;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=netix;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=ufpr;https://sourceforge.net/projects/geographiclib/files//distrib-C++/GeographicLib-2.1.tar.gz/download?use_mirror=tenet -> GeographicLib-2.1.tar.gz...
-- Extracting source C:/temp/vcpkg/downloads/GeographicLib-2.1.tar.gz
-- Using source at C:/temp/vcpkg/buildtrees/geographiclib/src/efa3bad6a4-8b721f0729.clean
-- Found external ninja('1.10.2').
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:129 (message):
    Command failed: C:/temp/vcpkg/downloads/tools/cmake-3.22.2-windows/cmake-3.22.2-windows-i386/bin/cmake.exe --build . --config Release --target install -- -v -j9
    Working Directory: C:/temp/vcpkg/buildtrees/geographiclib/x64-windows-rel
    See logs for more information:
      C:\temp\vcpkg\buildtrees\geographiclib\install-x64-windows-rel-out.log

Call Stack (most recent call first):
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
  ports/geographiclib/portfile.cmake:35 (vcpkg_cmake_install)
  scripts/ports.cmake:146 (include)

error: building geographiclib:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
    https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+geographiclib
You can submit a new issue at:
    https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[geographiclib]+Build+error
Include '[geographiclib] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
    vcpkg-tool version: 2022-05-05-67e17c1782801cf481be9ac0b3765dff3e4bdeb8
    vcpkg-scripts version: 8e1f46de1 2022-06-10 (30 hours ago)

Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

See logs for more information:
      C:\temp\vcpkg\buildtrees\geographiclib\install-x64-windows-rel-out.log

Here is the log you requested - looks like the error indicates:
mt.exe : general error c101008d: Failed to write the updated manifest to the resource of file "bin\TransverseMercatorProj.exe". Access is denied. [install-x64-windows-rel-out.log](https://github.com/microsoft/vcpkg/files/8885362/install-x64-windows-rel-out.log)

Usually an Anti-Virus thing.

BTW I updated the compiler to Version 17.3.0 Preview 1.1 and the problem seemed to go away.  Unfortunately I am not in a position to test an earlier non-preview version of the compiler to definitively prove that the issue is associated with 17.2.3 however the author of the library sent me his log indicating a successful build using Visual Studio 17.1.1.  The logs below were generated via
.\vcpkg\bootstrap-vcpkg.bat
.\vcpkg\vcpkg.exe install geographiclib:x64-windows --debug > build.log

build-success.log
build-failed.log

It looks a file access problem, it seems bin\TransverseMercatorProj.exe file locked, antivirus software is often the cause of this problem. If you have this problem again, please disable your antivirus software and try again.


RightOn Mobile UI: Phase 1 Nice Try/ Correct Displays

ToDo:

[x] Update headers - spacing and size
[x] Add question image and question text to the card body (see updated figma screenshot below)
[x] Test a question with something in the instructions field to see if they parse out properly
[x] Handle UI for questions with no instructions in that field - maybe remove instructions card entirely if there aren't any?

Source Code

https://github.com/rightoneducation/righton-app/pull/418


Determine common method of focus handling

e.g., focus trapping, input modality, etc. Essentially a common library of functionality for handling focus across all our components.

Superceded by https://github.com/material-components/material-components-web/issues/21


Allow applying own TextTemplate params

There is Text replacePlaceholders(Player player, TextTemplate text) for TextTemplates. I would like to apply my own parameters additionally to the ones from PlayeholderAPI.
How about a method that allows additional params to be passed to the TextTemplate. Or the PlaceholderAPI returns a map of the filled in params and values that the plugin using PAPI applies itself on the TextTemplate?

I actually thought TextTempalte allowed you to add your own placeholders to it without building but I was mistaken. Version 3.8 will support both providing your own to PlaceholderAPI and getting what PlaceholderAPI would use for a template.


Feature request: Tree-like message history

Hello, I recently built a tree-like message history feature (link) and I would like to develop it further to make it more similar to a real ChatGPT conversation. However, implementing this feature would require refactoring the current structure of the conversation. I'm wondering if this is worth doing?

This is a really great idea. I've got a few things that are priority this week, but will keep this open and revisit once those are done. I think this could be really cool.

+1 This would be really useful with the 32k version

I did it.(#339)

awesome work! haven't given it a try yet but curious as to whether the entire tree is included in exports

awesome work! haven't given it a try yet but curious as to whether the entire tree is included in exports

Each conversation is like a tree. When you click the export button, it exports all the trees as a JSON file.


sys/net/aodvv2: split AODVv2 and RFC 5444

This splits the AODVv2 implementation and RFC 5444:

RFC 5444 has now it's own thead/udp server.

Can now send multiple messages on a single packet to the given target (there's an aggregation time of 100 ms).

AODVv2 is thread less.

Comments are appreciated, this work is on-going, needs some structuring.

Archiving repository

Archiving repository


No androrisk.py in Androguard 2.0

Hello.
There used to exist a script androrisk.py in Androguard 1.9 but it has disappeared in version 2.0.
Was it replaced by any other script? Or is there any function in Androguard 2.0 that I can use to calculate risk value of APK files like androrisk.py?
Thanks

A lot of stuff was deleted in this commit 6b24ddf7d385e8a9062de7ae95d7e8dfb12f7b1c
I have no clue why the whole Tainted Stuff was removed, it seems it was removed and not replaced. Maybe there is a new method now? Anyone has information on that?

After looking into androguard a little bit deeper, i found out that the old "tainted" tools are not required, as you can work with the xref stuff now. So for example, where you first would search for all calls of a method, you can now just ask the method for the xref_from list.
For the risk calculation: This part was in my opinion not really reliable and would easily overestimate the risk. Also the permission model in android changed a lot and new API functions are introduced. Maybe it is better to write your own risk analysis module, that calculates the risk how you see it...
You can still use the old script if you need to (I'm not sure if it will still work though):
https://github.com/androguard/androguard/blob/008f1dc438fb0a4cb846045fbbbb180115ad9224/androguard/core/analysis/risk.py

It is not anymore supported !


WIP: Move HaarCascadeFaceDetection to this separate branch

Signed-off-by: choowilson<EMAIL_ADDRESS>When you added a new program / programs , put a cross on the  OS that you have test it on
(Example: [ X ] Windows  )
[ X ] Windows
[  ] Linux Ubuntu
[  ] Mac
[  ] Others  (State here -> xxx )
Assign a reviewer to review your code.

@choowilson Can you request the PR again to merge with dev_branch instead of master ?
I'm keeping master branch untouched till the training end.
@jingzhi93 will review it your PR.

ok, will do that


Fix bug in mapping due to missing short_id field.

@aphelionz @martinseul One bug fix of Zhaokun's didn't get into last PR. This corrects a problem with mapping that causes every box to show up when you click on one marker.

@aphelionz We need to rewrite the whole thing as POST because there is a danger of exceedng GET parameter length limits.

Build finished.

@alvacouch I have performed the edit

Build finished.


Additional properties TCGC Implementation

Replaced by https://github.com/Azure/typespec-azure/issues/664


API browser does not support more complex type anotations

For example if we have function using Unions, Optionals:
@jsonrpc_client.method('object.trip_stop')
@jwt_required()
def object__trip_stop(
        object_id: Union[int, List[int]],
        time_from: str,
        time_to: str,
        min_moving: Optional[int] = 1,
        min_stationary: Optional[int] = 10,
        zones: Optional[List[int]] = [0],
        time_ranges: Optional[List[List[Union[int, str, str]]]] = [],
        zones_only: Optional[bool] = False,
        trip_mode: Optional[int] = 3,
        trips_stops: Optional[int] = 3
) -> Optional[str]:

We get documentation with just Objects:

👋 This PR added support to serializable direct classes Python with the default constructor, Python dataclasses, and Pydantic models, see [here](https://github.com/cenobites/flask-jsonrpc/ blob/master/examples/pydantic/run.py#L158) and example of use. The next step is to enhance the schema to reflect the object at the API Browser and be recognized by it.
Payload:
{
    "jsonrpc": "2.0",
    "method": "Petstore.create_pet",
    "params": {
        "pet": {
            "name": "Lou",
            "tag": "dog"
        }
    },
    "id": "5ad1d47f-4e6c-4f24-bfa9-18a3ec340c57"
}


Getting older sessionRequest and data even if send new request

Describe the bug
I am getting older sessionRequest data on onSessionRequest callback and that too multiple time. Even after I disconnect and connect with new QR code, it's sending older data.
SDK Version

implementation("com.walletconnect:android-core")
implementation("com.walletconnect:web3wallet")
implementation(platform("com.walletconnect:android-bom:1.15.0"))

To Reproduce
Steps to reproduce the behavior:

Pair a dapp with your wallet
It will start sending getNetwork, getMirrorNetwork, call events even if you didn't requested for.
Send a new request from your dapp with a different accountid
Now onSessionRequest, I am getting older request though.

Expected behavior
It should update new request and data.
Screenshots
If applicable, add screenshots to help explain your problem.
Device (please complete the following information):

Device: Motorola G5
OS: Android 8.1.0
Browser [e.g. stock browser, chrome]
Version [e.g. 22]

Additional context
This is the older request and data.
This is request I am getting ->

SessionRequest(topic=fde7b0aab07605ef2bd219e56c9e4084558a667c6f95629c4c768b6f71b7f9cd, chainId=hedera:296, peerMetaData=AppMetaData(name=Blade Wallet Demo App, description=, url=https://<IP_ADDRESS>:3001, icons=[], redirect=, verifyUrl=null), request=JSONRPCRequest(id=1692698541921573, method=call, params={"accountId":"0.0.622648","executable":"Sg4KAhAAEggIABAAGLiAJg==","isTransaction":false}))

This is the dapp with which I am currently paired with. You can check the topic is different with above sessionRequest ->

[Session(pairingTopic=2a862d0b52b8bb7ffafa14e5a9745269704d592602cd36676501113ee0218297, topic=838576a3c283721b88dc950708efed4ac9059774f7239a8d48d9ee396b23d36b, expiry=1693393463, requiredNamespaces={hedera=Proposal(chains=[hedera:296], methods=[signTransaction, call, getAccountBalance, getAccountInfo, getLedgerId, getAccountId, getAccountKey, getNetwork, getMirrorNetwork, sign, getAccountRecords, checkTransaction, populateTransaction], events=[accountsChanged, walletUnlocked, walletLocked])}, optionalNamespaces={}, namespaces={hedera=Session(chains=[hedera:296], accounts=[hedera:296:0.0.622648], methods=[signTransaction, call, getAccountBalance, getAccountInfo, getLedgerId, getAccountId, getAccountKey, getNetwork, getMirrorNetwork, sign, getAccountRecords, checkTransaction, populateTransaction], events=[accountsChanged, walletUnlocked, walletLocked])}, metaData=AppMetaData(name=Blade Wallet Demo App, description=, url=https://<IP_ADDRESS>:3001, icons=[], redirect=, verifyUrl=null))]

Hey @aadityapaliwal thanks for reporting. Couple of questions from my side to understand your problem better:

What happens when you reject the old session request? Can you see the correct one eventually after rejecting all the old ones?
Can you share the dapp's link that you use for testing? Would like to try to reproduce it.
-Can you see this error on older web3wallet SDK versions?

@jakubuid Thanks for revert back.

How can I reject old sessionRequest once I approved a new session proposal. If I tries to respond the older topic id, it's throwing me this error -

Error(throwable=com.walletconnect.android.internal.common.exception.CannotFindSequenceForTopic: Cannot find sequence for given topic: 2fb15c4400fa2f7a4773ac96b158f377000ddd55e4c7499e4f407d2956a7b934)

I am using this Dapp - http://dropp-walletconnect-testbed.s3-website-us-east-1.amazonaws.com/.

I downgraded com.walletconnect:android-bom to 1.14.0 but getting exception and result in crashing the app https://github.com/WalletConnect/WalletConnectKotlinV2/issues/1044. I further downgraded it to 1.13.1 and it's working fine. I am getting correct topic id in the sessionRequest.

Yes, once I uninstall the wallet and install a fresh one, then I get the correct new topic ID but if I again pair it with new topic id then same issue arises.

So, it seems working fine with bom v1.13.1. Should I downgrade it?

Thanks for the reply.
I think I know what the root cause is - I'm working on the fix and we're aiming to release a new version this/early next week

@jakubuid when we will release this version?

@jakubuid I have updated the library on the latest version 1.16.0 and tried it but still getting the older requests. It seems like it's sending the older request which I might have sent few minutes back. Then even if I request the new one for sign transaction, it's sending older. getMirrorNetwork, getNetwork requests are coming as older topic.
This is the active session -
2023-09-06 15:46:56.439 20518-20518 Active sessions D  [Session(pairingTopic=7adf779063d7cab770a62b59221f2ba2a9c6eacaf8a704d39547a089b1b42467, topic=d10f95558829c1fbfda293e3467957e142588499566070e8e27f73f6f697be89, expiry=1694599111, requiredNamespaces={hedera=Proposal(chains=[hedera:296], methods=[signTransaction, call, getAccountBalance, getAccountInfo, getLedgerId, getAccountId, getAccountKey, getNetwork, getMirrorNetwork, sign, getAccountRecords, checkTransaction, populateTransaction], events=[accountsChanged, walletUnlocked, walletLocked])}, optionalNamespaces={}, namespaces={hedera=Session(chains=[hedera:296], accounts=[hedera:296:0.0.622648], methods=[signTransaction, call, getAccountBalance, getAccountInfo, getLedgerId, getAccountId, getAccountKey, getNetwork, getMirrorNetwork, sign, getAccountRecords, checkTransaction, populateTransaction], events=[accountsChanged, walletUnlocked, walletLocked])}, metaData=AppMetaData(name=Blade Wallet Demo App, description=, url=https://dropp-walletconnect-testbed.s3.amazonaws.com, icons=[], redirect=, verifyUrl=null))]

2023-09-06 15:48:01.032 20518-20802 onSessionRequest  D topic af74589c92a296c8f47a1155203ced12b77ff9d030fa3a2b405b4835d4934bab request JSONRPCRequest(id=1693993557098647, method=getMirrorNetwork, params={"accountId":"0.0.622648"})

This error is throwing when I respond to the previous request

FATAL EXCEPTION: DefaultDispatcher-worker-9
Process: PID: 20164
com.walletconnect.android.internal.common.exception.CannotFindSequenceForTopic: Cannot find sequence for given topic: 22ce7f8821630e35a0d0c705b5abc351280de3b68ecd00c012a98db92678e61d
at com.walletconnect.sign.engine.use_case.calls.RespondSessionRequestUseCase$respondSessionRequest$2.invokeSuspend(RespondSessionRequestUseCase.kt:49)
at com.walletconnect.sign.engine.use_case.calls.RespondSessionRequestUseCase$respondSessionRequest$2.invoke(Unknown Source:8)
at com.walletconnect.sign.engine.use_case.calls.RespondSessionRequestUseCase$respondSessionRequest$2.invoke(Unknown Source:4)
at kotlinx.coroutines.intrinsics.UndispatchedKt.startUndispatchedOrReturn(Undispatched.kt:78)
at kotlinx.coroutines.SupervisorKt.supervisorScope(Supervisor.kt:61)
at com.walletconnect.sign.engine.use_case.calls.RespondSessionRequestUseCase.respondSessionRequest(RespondSessionRequestUseCase.kt:46)
at com.walletconnect.sign.engine.domain.SignEngine.respondSessionRequest(Unknown Source:7)
at com.walletconnect.sign.client.SignProtocol$respond$1.invokeSuspend(SignProtocol.kt:213)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
at kotlinx.coroutines.internal.LimitedDispatcher$Worker.run(LimitedDispatcher.kt:115)
at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:100)
at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:584)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:793)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:697)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:684)

@jakubuid Please reopen this issue

@aadityapaliwal @nguyennh-0786
How can we reproduce this issue? Can you provide the steps?
Can you try disconnecting from all the active sessions and try again?

@jakubuid I did try disconnecting with all sessions but it's still sending the old requests.
I am using the Motorola Moto G5 Android version 8.1.0 (API Level 27). Please use the below steps to repro it -

Navigate to https://dropp-walletconnect-testbed.s3.amazonaws.com/index.html
Copy the URI from copy to clipboard and send it in your device.
Connect wallet with URI.
It will ask to approve the request and open a page. Tap on Approve.
As soon as I approve it, I have started getting sessionRequest from Dapp for getNetwork and getMirrorNetwork.
Now go to above Dapp and send any request.
You will get the new request. Now disconnect with that dapp.
Reconnect with the same Dapp and send a new sign request from Dapp.
Now If you check the logs it will send the older request again.

Check my active session and onSessionRequest logs in the above comment.

Hey @aadityapaliwal thank you for your patience
I was trying to reproduce this issue but I don't have access to dropp-walletconnect-testbed.s3.amazonaws.com/index.html in order to connect. Can you help with it?

@jakubuid Please try this - https://dropp-walletconnect-testbed.dropp.cc/

So it seems the problem is that the requests sent from dapp to wallet are not responded to. Meaning that the user neither approved nor rejected them.
In the SDK we keep all the not responded session requests in a queue to prevent any request from being lost. The mechanism will keep emitting the first not responded session request from the queue until it expires or the user responds. Every new request is added to the end of the queue and triggers the first one.
If you need to send sth from dapp to wallet that doesn't require user interaction, I'd recommend using the emit method that emits events that you agreed on during session negotiation.

@jakubuid Thanks for the description. I got the point of having mechanism for the not responded request but if we disconnect with that topic, then it shouldn't send the not responded request of the older topic, isn't it?

Yes, you're right. I'll look deeper into that

Exactly, the problem is that request over an already disconnected session can't be responded, so they are stuck in the queue forever.

The mechanism will keep emitting the first not responded session request from the queue until it expires or the user responds. Every new request is added to the end of the queue and triggers the first one.

I understand the motivation behind not losing requests, but is this a good trade-off? Especially since this is triggered by the user activaly initiating another request. What are the chances that the user wants to Approve/Reject a potentially days-old request, vs. the user mistakenly approving an unwanted request they ignored some days ago because they thought the approve dialog was related to the tx they just initiated? Wouldn't it be much safer to assume any request older than some minutes (or 1h if you want to be safe) is rejected? The problem is that from the UX perspective, ignoring or rejecting a request has the same outcome, but you treat it differently.

So the key problem is why you still see session requests from disconnected sessions. This is what I'll try to reproduce. The team will also reconsider this approach. Thanks!


Added new authentication keyword to support AAD Service Principal

Coverage increased (+0.01%) to 76.705% when pulling 639cf1c4e5f934452ffd86f66db0517c15b78138 on yitam:aadSPA into e0a6b52f7a1d052c052f4b624380503ef5904d06 on microsoft:dev.


throw and catch with SkipMacroException

SkipMacroException is used in the to signal a skip_file macro expression. This is not actually an error. The exception is used to work like throw/catch in Ruby.
We don't have throw/catch in Crystal and I think it's for good reasons (https://forum.crystal-lang.org/t/throw-and-catch/256). Yet we make use of this concept in the compiler. That's definitely a hack because there's no error involved, but it necessarily inherits from Exception which is the base error type.
A side effect of that is that the compiler requires exceptions to function, not just for error handling. This poses difficulties when porting the compiler to new targets where exception support is still missing (https://github.com/crystal-lang/crystal/pull/10870#issuecomment-1000920464).
I think the right solution would be to remove SkipMacroException and properly implement skip_file with an other form of signalling that is embedded in regular code flow. This is probably not going to be trivial because it needs to be handled through several levels of compiler layers.
Alternatively, we could decide this might actually be a good use case for this pattern and we want to keep it. Then we should make sure that it's properly implemented (non-error exception base type, skip the stack trace). Maybe this could also warrant adding throw/catch feature to the language.

Yet we make use of this concept in the compiler

I don't quite agree. We use exceptions for control flow. That's contrived, but valid. People do this in languages with exceptions, it's unrelated to Ruby's throw/catch.
An alternative to using exceptions is requiring that skip file happens as the first node in the file. I think that kind of makes sense because skip_file means "skip the entire file" so having it in the middle of the file is awkward. So we could check if the first node is a macro node, and check if it contains a call to skip_file. If so, we evaluate that. Otherwise, we use regular file evaluation and if we find a skip_file we give a compile error saying that skip_file must happen in the beginning of the file.

Yet we make use of this concept in the compiler

I don't quite agree. We use exceptions for control flow. That's contrived, but valid. People do this in languages with exceptions, it's unrelated to Ruby's throw/catch.
An alternative to using exceptions is requiring that skip file happens as the first node in the file. I think that kind of makes sense because skip_file means "skip the entire file" so having it in the middle of the file is awkward. So we could check if the first node is a macro node, and check if it contains a call to skip_file. If so, we evaluate that. Otherwise, we use regular file evaluation and if we find a skip_file we give a compile error saying that skip_file must happen in the beginning of the file.

Yet another alternative is to deprecate skip_file and stop using it. You can achieve the same thing by not requiring a file.

Yet another alternative is to deprecate skip_file and stop using it. You can achieve the same thing by not requiring a file.

Here are the files that use skip_file after the first {%:

spec/compiler/semantic/macro_spec.cr: The specs that test skip_file place them in the middle of the test code.
spec/std/thread_spec.cr, spec/std/thread/condition_variable_spec.cr, spec/std/thread/mutex_spec.cr: They add a pending test on musl.
src/io/encoding_stubs.cr: Has a flag check, can be rewritten using a suffix conditional.
src/io/encoding.cr: This in turn requires encoding_stubs.cr and then calls skip_file immediately, suggesting that the whole part can be easily refactored into separate files.

I also wonder whether anyone outside this repo is using skip_file. My guess is no, because we use it for platform-de pendent code. In that case, we could refactor our code to not use skip_file and then silently remove that macro method.

A quick search shows that it is indeed used:
{% unless flag?(:anyolite_implementation_ruby_3) %}
  {% skip_file %}
{% end %}
# ...

{% skip_file if Avram::Model.all_subclasses
  .map(&.stringify)
  .includes?("UserOptions")
%}

{% skip_file unless Lucille::JSON.includers
  .map(&.stringify)
  .includes?("UserSettings")
%}
# ...

# ...
{% skip_file if @top_level.has_constant? "Spec" %}
# ...

I'm not sure it would be such a great idea to require skip_file be the first expression. It's a strong restriction and could be really awkward. What about comments, for example?
Also I think it shouldn't be too difficult to handle later in the file.
In order to remove the control flow by exception, we need a way to stop evaluating the current file. I don't think it makes much of a difference whether that only takes the first expression into account because the entire expression including skip_file needs to be processed, that means there's usually already a number of sub-expressions (conditionals, logic operations, flag? calls) involved. I'm pretty sure the implementation would not be very more complicated when it's not limited to the first expression.

Sounds good.
Also note that for example autocasting of numbers/literals is done using exceptions.
I think it's just simpler to spend that effort implementing exceptions on the targets we want to port the language to.

I'm going back and forth on this and my current opinion is that exceptions is a feature of the language like any other and the compiler shouldn't be restricted from using it. Targets that don't support exceptions won't have a native compiler for them and will have to live with cross compilation until they support the full language. It doesn't stop programs being compiled for said target.
If this is changed then it should be based on code clarity or performance on the grounds that exceptions should not be used for normal control flow. That said, I don't have a problem with exceptions being used for control flow.

The compiler uses another exception type, RetryLookupWithLiterals for control flow as well. That's happening much more frequently than SkipMacroException. We improved performance in #15002 by assigning a static, empty call stack to these exceptions (control flow management is not exceptional and we don't care about the call stack).
We could probably do the same for SkipMacroException, although it won't have much effect on performance since skip_file is used very rarely.
This again raises the question whether we should generalize the concept of control flow via stack unwinding.
For example, we could introduce an exception-like base type which doesn't have a call stack (or initializes it to empty like #15002) and use that as common parent for RetryLookupWithLiterals and SkipMacroException.


Is there a trick to stabilize training?

Hi! Thanks for open-sourcing your work! I like the idea, so I tried copying the CFM decoder and use it in my TTS setup. First I had some issues with NaN values after the attention in the estimator was computed for all the values that were padding. I fixed this issue using masked_fill with the x_mask instead of just multiplying with the x_mask, although I'm not sure why that was necessary for me.
But now that I could run a forward pass and calculate the CFM loss and it was not NaN, I thought it's good to go. However more problems came up. No matter what I tried, the CFM decoder would always produce a couple of NaN values after its first update during training.  Do you have any idea what could cause this? Is there any specific thing I need to do to stabilize the training? The setup I am using works very well for lots of other architectures, including e.g. the normalizing flow decoder of PortaSpeech, which seems pretty similar.

Thank you for your interest about StableTTS. In  x_mask, "1" indicates that the element is included in the attention calculation, whereas "0" means that the element does not participate in the attention calculation. So the padded element is set to "0" in x_mask. Could it be possible that your mask is reversed? For example, where "1" represents a padded element.

The mask is not reversed, I am however using a boolean mask instead of a float mask. But I could solve it with the masked_fill method, so that's no longer the problem, just something I found a bit strange. The problem is that after one training update, everything in the CFM decoder turns to NaN, even with a very small learning rate and gradient clipping. Do you have any idea what could cause this?

Try converting the boolean mask into a floating-point mask, for example, attn_mask.to(x.dtype). This might solve the issue.
In PyTorch's attention calculations, it can sometimes be confusing as the same mask in boolean format and floating point format can have dramatically different effects. For instance, in StableTTS, converting the attention_mask to boolean could lead to NaN issues because it makes attention only to focus on padding value.

You're right, it works now, thank you!
I thought a boolean tensor and a float tensor should be equivalent when they are used like this, but apparrently there is a difference.

You're right, it works now, thank you!
I thought a boolean tensor and a float tensor should be equivalent when they are used like this, but apparrently there is a difference.

Hi, it seems that my implement of attention mask is wrong, making the model suffering bad prosody.
In the new model, we have solved this issue:
https://github.com/KdaiP/StableTTS/blob/71dfa4138c511df8e0aedf444df98c6baa44cad4/models/diffusion_transformer.py#L108


Not able to update Request Transformer

Able to create a Request Transformer in Consumer > APIs > Add Plugins > Request Transformer.
Failed to assign values to "replace", "remove", "add", "rename". See screenshots below. After click on Submit Changes, these values remain blank.

I've run into the same issue in Konga 0.10.6

Actually You have to write your thing then press enter and then hit the save button.
As written here https://github.com/pantsel/konga/issues/85

Closing this as it seems to be resolved


Added method to remove include ".c" file from objects linker list.

This pull request implements an automatic method to recognize .c files included to the test module. Once this .c file is included the preprocessor recognize and remove the module from object lists in order to avoid duplicate definition error.
It allows the usage of #include for source files, procedure used to have access to private function and variables when unit testing.

@CezaryGapinski updated the code according your suggestion.

Thank you @beckerzito & @CezaryGapinski. This is good stuff!


Updated NoSchedule taint name for master

Description
node-role.kubernetes.io/master:NoSchedule taint has been replaced with node-role.kubernetes.io/control-plane:NoSchedule from k8s 1.20.

Should we be tainting with both in case it's an old cluster? Or this only happens for new cluster creates and we're definitely using v20 there? Maybe we should be checking the kubernetes version and then using the correct taint?

You are right, we should be checking the version - old clusters were created with node-role.kubernetes.io/master:NoSchedule, so if we are updating a cluster and need to move workload onto a master tainting should be use the old annotations.
My concern is how to handle these issues  going forward, since we'll be hitting changes like this periodically.
@gainsley - what do you think?

I think we need to track the cluster k8s version anyway, to allow for upgrade (triggered by users), and to be able to notify users that the cluster version is old/deprecated and needs to be upgraded. If we are tracking the k8s version, then we can determine the correct taint to use.
It looks like we must use the old taint before 1.20, and we must use the new taint from 1.24. Sources:
https://github.com/kubernetes/enhancements/tree/master/keps/sig-cluster-lifecycle/kubeadm/2067-rename-master-label-taint
https://kubernetes.io/blog/2022/04/07/upcoming-changes-in-kubernetes-1-24/#api-removals-deprecations-and-other-changes-for-kubernetes-1-24

Updated code to check for older k8s cluster version we had. I made it explicit, since that was the only version we had as far as I recall.


How to use GPU for uvq predictions?

I have gone through the code - it stores the features on disk before prediction , is there any way to accelerate the process and using  gpu?

If you have correct drivers for your GPU and CUDA installed, Tensorflow will use it. Here's the reference for compatible versions https://www.tensorflow.org/install/source#gpu. The bulk of the computation happens during feature generation, so you should still see a decent speedup in that phase.


[ldd-msn] Add instrument_clock_start and instrument_start_time

Add to <Mission_Information>:
<instrument_clock_start>  The value of the clock returned by the
instrument at the beginning of the observation.  This may differ from
the spacecraft clock due to several factors such as clock drift, or
differences in definition of start time (for example, spacecraft clock
might be when an image was commanded, but instrument clock might be
after autoexpoure or autofocus processes).
<instrument_start_time>  This is instrument_clock_start converted to UTC.
Both are cardinality 0:1

Modification to definition to be more expansive (to accommodate the helicopter use case, which has its own time unrelated to sclk (based on linux time, actually), which is also converted to sclk):
The value of the clock returned by the instrument at the beginning of the observation. This may differ from the spacecraft clock due to several factors such as clock drift, or differences in definition of start time (for example, spacecraft clock might be when an image was commanded, but instrument clock might be after autoexpoure or autofocus processes), or differences in the time epoch between the instrument and the main spacecraft.

This was previously updated so just changed the definition to allow for helicopter use. It does have a recommended formation rule but it doesn't need to be followed if the helicopter uses something else. Really any ASCII_Short_String_Collapsed can be used up to 255 characters.
     <value_data_type>ASCII_Short_String_Collapsed</value_data_type>       <formation_rule>Value should be of the form: nnnnnnnnn.mmm</formation_rule>       <minimum_characters>1</minimum_characters>       <maximum_characters>255</maximum_characters>


Update cups_de.po

Issues in the translation

This repo is no longer maintained, as it is now maintained by OpenPrinting.
I made a new PR for you here:
https://github.com/OpenPrinting/cups/pull/494


emacs-bootstrap.org seems to be down.

Getting internal server error when trying to load the site.

Hmm,, uwsgi wasn't starting because of a bug in the config. I should probably dockerize this soon.
It should be working now. Btw, it's emacs-bootstrap.com.
Thanks for reporting it!


fix: don't rely on buffer

@ThaUnknown I'm no longer on a contract with Socket so any merging is out of my hands as my commit access then also is gone
Check with @feross

@bmeck sick, please don't forget to publish it

it's not really major, it's backwards compatible with almost all versions of webtorrent
the tests fail because they check against buffer not uint8 so tape hangs because it's deepequal util is poorly written

other chunk stores return uint8 just fine:
https://github.com/mafintosh/memory-chunk-store/blob/master/index.js#L40
https://github.com/ThaUnknown/fsa-chunk-store/blob/master/index.js#L214

@ThaUnknown it changes the type so people using any utilities from Buffer will be unable to use them after this change without converting to a Buffer.

I think ppl should not be using Buffer
it's better to use TextEncoder/decoder(stream), DataView, Uint8Array instead. cuz it's better for cross compatible env who dose not have a global Buffer like in NodeJS and who dose not want to bring in another large dependency on a Buffer polyfill. all resent changes in NodeJS have become more acceptable of writing Uint8Arrays to the disk and responding with typed arrays instead.
another thing i that i think is inherently bad is that they override the .slice() method. if copying is so bad then subarray should be used instead.


Remove level validation check

Codecov Report

Merging #18 (055d826) into main (93a3683) will increase coverage by 0.63%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
+ Coverage   96.52%   97.16%   +0.63%
==========================================
  Files           4        4
  Lines         144      141       -3
==========================================
- Hits          139      137       -2
+ Misses          5        4       -1

Impacted Files
Coverage Δ

sunpy_soar/attrs.py
88.57% <ø> (+1.72%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 93a3683...055d826. Read the comment docs.


Automatically add PTVS diagnostics log files to Report a Problem

Feedback tool team is adding an extensibility point so we can add extension specific logs.

Implemented in https://github.com/huguesv/PTVS/tree/issue-5199, we just have to wait for VS to put their changes in.


Trophy Gold - Broken Translations

Translations aren't working. Selecting it from the drop down has all the red text of broken translations.

[Occupation]

Translation File was misnamed. I've renamed translations.json to translation.json. I'll keep an eye on it to make sure CrowdIn generates the correct files with the next PR.
Nice to see you in these parts @clevett  We've missed you!


CosmosQueryRequestOptions excludes old FeedOptions

Is your feature request related to a problem? Please describe.
Not sure if this is a regression bug or feature request but the PopulateQueryMetrics feed option is not exposed as a member of the CosmosQueryRequestOptions class.
In fact there are a number of feed options which have been left out between the current client and this one. Have these options been left off for a reason or are they configurable else where?
Comparison between FeedOptions & CosmosQueryRequestOptions

Property
Status

ConsistencyLevel
Supported

EnableCrossPartitionQuery
Internally Set (no public access)

EnableLowPrecisionOrderBy
Supported

EnableScanInQuery
Supported

JsonSerializerSettings
Not Supported

MaxBufferedItemCount
Supported

MaxDegreeOfParallelism
Supported (called MaxConcurrency and no longer request option)

MaxItemCount
Supported (no longer request option)

PartitionKey
Supported

PartitionKeyRangeId
Not Supported

PopulateQueryMetrics
Not Supported

RequestContinuation
Not Supported

ResponseContinuationTokenLimitInKb
Supported

SessionToken
Supported

Describe the solution you'd like
Are there any objections to adding PopulateQueryMetrics as an option to CosmosQueryRequestOptions? I've tested and confirmed that CosmosQueryResponse<T> includes the metrics if set. This setting is really useful for performance tuning etc.

We are still working on the v3 diagnostics design. I will create a github issue once we have a design. We want query and point operations to have the same story. In v2 they are completely different which doesn't provide the best user experience.
These properties have been updated.

Property
Status

EnableCrossPartitionQuery
Internally Set (no public access). Automatically set when uses overload with max concurrency.

JsonSerializerSettings
Supported via CosmosJsonSerializer

PartitionKeyRangeId
No plans to support.

PopulateQueryMetrics
Planned

RequestContinuation
Renamed to ContinuationToken

Is this something that will be resolved before GA release?
In the meantime I guess a workaround would be using a handler to add the "x-ms-documentdb-populatequerymetrics" header to the request and read the "x-ms-documentdb-query-metrics" from the Response.

Hi, is there an ETA on the PopulateQueryMetrics property?


dcgm nvlink metrics not available on dcgm 3.1.3

the nvidia-dcgm doc says that metrics like DCGM_FI_PROF_NVLINK_L{id}_TX_BYTES should be avaible on dcgm 3.1
I'm getting the following error when trying to query them (from dcgmi 3.1.3):
$ dcgmi dmon -d 100 -e 1040
#Entity   NVL0T
ID
Error setting watches. Result: -6: Feature not supported

$ dcgmi -v | grep Version
Version : 3.1.3
Version : 3.1.3

is it expected? am I missing intermediate steps to enable the metrics?

@luccabb what is the output of nvidia-smi?

A100

what is the output of nvidia-smi?

+-----------------------------------------------------------------------------+
| NVIDIA-SMI 525.60.13    Driver Version: 525.60.13    CUDA Version: 12.0
...

per https://github.com/NVIDIA/DCGM/issues/149#issuecomment-1922398817 its only available on Hopper+ GPUs
surfacing this on the dcgm docs would be helpful
cc: @dbeer @nikkon-dev


chore: add a Storybook addon to ease working with OUIA ids

Adds an addon to Storybook to visually inspect the OUIA ids in use in the story.
Green ids are statically set, so safe to use. Red ids are autogenerated, so they probably shouldn't be used.
Ids are also listed in a panel called OUIA, with a selector that can be copy-pasted directly to Pendo and that takes into account parent OUIA ids if present.

:tada: This PR is included in version 1.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[BUG] No module named 'Queue' in ibpy2 dependency

Describe the bug
I'm getting a Queue vs queue capitalization error (more details here - it relates to a change in Python 2 vs 3) when trying to run the blotter file. It seems to stem from ibpy2 - is this where it should be gathering these libs from? Is there an alternative for this dependency or a workaround?
I've tried changing the capitalization in the file specified in the stack trace but it fires an exception for trying to print too many exceptions then! So I assume its bad...
Desktop (please complete the following information):

OS: Windows 10 (running from Powershell as Admin)

Additional context
Stack trace:
PS C:\Users\desmo\Documents\sdata> python blotter.py
Traceback (most recent call last):
  File "C:\Users\desmo\Documents\sdata\blotter.py", line 2, in <module>
    from qtpylib.blotter import Blotter
  File "C:\Users\desmo\AppData\Local\Programs\Python\Python310\lib\site-packages\qtpylib-1.5.84-py3.10.egg\qtpylib\blotter.py", line 51, in <module>
    from ezibpy import (
  File "C:\Users\desmo\AppData\Local\Programs\Python\Python310\lib\site-packages\ezibpy-1.12.70-py3.10.egg\ezibpy\__init__.py", line 26, in <module>
  File "C:\Users\desmo\AppData\Local\Programs\Python\Python310\lib\site-packages\ezibpy-1.12.70-py3.10.egg\ezibpy\ezibpy.py", line 31, in <module>
  File "C:\Users\desmo\AppData\Local\Programs\Python\Python310\lib\site-packages\ibpy2-0.8.0-py3.10.egg\ib\opt\__init__.py", line 30, in <module>
    from ib.opt.connection import Connection
  File "C:\Users\desmo\AppData\Local\Programs\Python\Python310\lib\site-packages\ibpy2-0.8.0-py3.10.egg\ib\opt\connection.py", line 20, in <module>
    from ib.opt.dispatcher import Dispatcher
  File "C:\Users\desmo\AppData\Local\Programs\Python\Python310\lib\site-packages\ibpy2-0.8.0-py3.10.egg\ib\opt\dispatcher.py", line 8, in <module>
    from Queue import Queue, Empty
ModuleNotFoundError: No module named 'Queue'

I install qtpylib following this comment https://github.com/ranaroussi/qtpylib/issues/141#issuecomment-1029021829, because the pip install seems not working now. However, using python setup.py install will mess up something if you do not specify the install path, especially when you have anaconda and python at the same time. I am not an expert in package installation and I hope someone can point out the main issue here.
I spent a few hours and solved the problems using brute force. Here is the solution:

Find your package install location, e.g. C:\ProgramData\Anaconda3\Lib\site-packages
Delete folder ib and folder IbPy2-0.8.0.dist-info
Move ibpy2-0.8.0-py3.8.egg (Needs to be downloaded) to this directory
Rerun python setup.py install

I install qtpylib following this comment #141 (comment), because the pip install seems not working now. However, using python setup.py install will mess up something if you do not specify the install path, especially when you have anaconda and python at the same time. I am not an expert in package installation and I hope someone can point out the main issue here.
I spent a few hours and solved the problems using brute force. Here is the solution:

Find your package install location, e.g. C:\ProgramData\Anaconda3\Lib\site-packages
Delete folder ib and folder IbPy2-0.8.0.dist-info
Move ibpy2-0.8.0-py3.8.egg (Needs to be downloaded) to this directory
Rerun python setup.py install

Where do I find ibpy2-0.8.0-py3.8.egg?


D2S Design QA: Trees

Use 20px icon size
Row top and bottom padding 2px
Use sm group button
Dropdown: The dev of dd items is not as per design. Padding 6px vertical padding 16px(horizontal padding) distance between elements 8px.
The hight of tree items should be 32 for child items. It is 24 for now.
There is no hover state for child items. It is present in design
The child element's branch is overlapping on the child and ther left padding should be 6px. It is 8px right now.
There are no confirmation buttons in edit mode.
The action buttons in hover state should sit beside title instead of in the middle and should change position upon increasing or decreasing title character.
there is an extra variant with the actions button on the right. we don't need it.

Props:

There should be props for slots. We are using three slots 2 after plus icon on the left content and 1 on the right side.

@haseeb-repairdesk


Don't accept multi-return function types when enable_multi_value is false.

Thank you for the patch.

Hm, I would have thought this would fail when parsing, before validation, if enable_multi_value was false. @nlewycky did you find that not to be the case? Might be worth a regression test too.

@fitzgen I'm not particularly familiar with the architecture of wasmparser.rs. I think the parser handles the single combined language with all features enabled and streams it through the validator. I mimicked how it rejects a block that returns multiple values to also reject a function type declared to return multiple values.
The testcase is a wasm file with these bytes: [0, 97, 115, 109, 1, 0, 0, 0, 1, 6, 1, 96, 0, 2, 127, 127].


Add support for refreshing tests in a single project/package/module.

Currently, the "Refresh Tests" command and shortcut button in the test view refresh tests across the entire workspace. I have a mono-repo project that I open as a multi-root workspace and I would love to be able to refresh tests from only the specific part of the code I'm currently working as VSCode doesn't always pick up on changes.
The ideal solution would be to add a refresh shortcut next to every expandable test container in the testing view where the Run/Debug/GoTo buttons already pop up on mouseover that would just refresh discovery in that particular module.

Hello @mrburrito, thank you for the request. Currently the UI is handled by an API in vscode core, therefore I cannot make any changes to add that button to the test explorer. @connor4312 is this a request you have gotten before or should we create a new feature request for this in core?
In the meantime @mrburrito would it help if when you saved a file it only ran "refresh test" on that specific test file?
Thanks!

@eleanorjboyd this is actually something you can do by contributing to the testing/item/context menu. Though currently we don't expense the test or controller ID in the when clause, I'll put in a PR for that.

The issue is more when VSCode doesn't refresh, which seems to happen more often when there's an error in the file and pytest can't discover tests. That's when I typically find myself wanting to refresh a specific subtree from the test explorer.

Thank you to everyone who upvoted this issue! Since the community showed interest in this feature request we will leave this issue open as something to consider implementing at some point in the future.
We do encourage people to continue 👍 the first/opening comment as it helps us prioritize our work based on what the community seems to want the most.


Cards: Some a11y fixes

Note: Please transform - [ ] into - (NA) in the description when things are not applicable
Related issues
NA
Description
Removing useless role and adding a small callout at the start to make sure people change the code corresponding to their needs.
Motivation & Context
Some accessibility issues were spotted.
Types of change

Bug fix (non-breaking which fixes an issue)

Live previews

https://deploy-preview-1904--boosted.netlify.app/docs/examples/cards
https://deploy-preview-1904--boosted.netlify.app/docs/examples/cards-rtl

Checklist

Contribution

[x] I have read the contributing guidelines

Accessibility

[x] My change follows accessibility good practices; I have at least run axe

Design

[x] My change respects the design guidelines defined in Orange Design System
[x] My change is compatible with responsive display

Development

[x] My change follows the developer guide
(NA) I have added JavaScript unit tests to cover my changes
(NA) I have added SCSS unit tests to cover my changes

Documentation

[x] My change introduces changes to the documentation and/or I have updated the documentation accordingly

Checklist (for Core Team only)

(NA) My change introduces changes to the migration guide
(NA) My new component is added in Storybook
(NA) My new component is compatible with RTL
[ ] Manually run BrowserStack tests
[ ] Manually test browser compatibility with BrowserStack (Chrome >= 60, Firefox >= 60 (+ ESR), Edge, Safari >= 12, iOS Safari, Chrome & Firefox on Android)
[ ] Code review
[ ] Design review
[ ] A11y review

After the merge

[ ] Manually launch Percy tests

Just one last question: could it be useful to add this warning in the doc too, just like we usually do, and not only on the examples?

I don't know, it might be useful on certain pages and less on other. Maybe a general callout inside accessibility.md or getting-started.md, might do the trick ?

Just one last question: could it be useful to add this warning in the doc too, just like we usually do, and not only on the examples?

I don't know, it might be useful on certain pages and less on other. Maybe a general callout inside accessibility.md or getting-started.md, might do the trick ?

Unfortunately I think that almost no one will have a look on these two pages so...
I honestly think that we should not add any callout because all developers must know the difference between a link and a button. But we have one to explain it in dropdowns.md so it may be usefull after all. 🤔

Changed few things in the migration guide via https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/pull/1904/commits/69d3a318a4c2534c3d9c459bd7099fa5a5a87511:

Moved the new content in the "Examples" section since this modification only impacts the examples
Wording to precise a bit the modified things
Dropped the .text-warning-emphasis and .bg-warning-subtle that doesn't work in dark mode


iFrame test is broken

See: https://abrahamjuliot.github.io/creepjs/tests/iframes.html - the page load hangs because this URL is unreachable:
https://github.com/abrahamjuliot/creepjs/blob/b701222b524b7f576f611ac365b724aabc6fba7d/docs/tests/iframes.js#L140
It would also be good to add an AbortController() to the fetch request (or any fetch requests, really), like this: https://dmitripavlutin.com/timeout-fetch-request/#2-timeout-a-fetch-request
Or the new approach using an AbortSignal: https://developer.mozilla.org/en-US/docs/Web/API/AbortSignal/timeout

Hey Niek, thank you for reporting this.
I plan to host the JSON on GitHub pages and add fetch timeouts. Apps Script is a nice way to manage cron jobs and use is free, but it's too slow if web traffic increases.


how can i create auto generate ID using bread in VOYAGER.

Version information

Laravel: v5.8.17
Voyager: v1.2.6
PHP: 7.3.1
Database: [type] [version] (e.g. MySQL 8.0)

Description of problem
How can I create auto generating unique ID using Bread .
Proposed solution
I want to create a new ID which can be auto generated with unique ID. I want to know if it can be created using bread.
Alternatives considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Question (partly) answered on Slack.
Please follow the issue template and ask questions only there.


Setting up getters with parameters passed

Is it possible to setup @Getter to get the following output?
computed: {
    collectionCategoryObjects() {
         return this.$store.getters["entities/collections/getCollectionCategoryObjects"]({
		department: this.department,
		collectionTitle: this.collectionTitle
	})
    }
}

How can I create @Getter in a compact format?
Is there a way to setup a getter that is less verbose than this?
const Ncollection = namespace('entities/collections')
// ... class .. {
@Ncollection@Getter("getCollectionCategoryObjects") getterCollection: any
get Collection() {
	return this.getterCollection({
		department: this.department,
		collectionTitle: this.collectionTitle
	})
}

In other words, please support curry getter
this.$store.getters['curryGetter'](someParam)

in vuex-class.
See details in How to pass parameters to the function in getters


Add an Explicit Icon for Email Contact

This seems handy and helpful,  right?
The link goes to<EMAIL_ADDRESS>Here is the draft illustration for both header and footer. You should get the idea despite of my poor drawing.

Sorry 我不小心 push 上來了……
目前長得像這樣：

footer:


Constant expression in order-by produces an error

Describe the bug
EnforceSorting produces an error when the sort expression is a constant.
This is the error produced
EnforceSorting
caused by
Internal error: Wrong number of children.
This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker

To Reproduce
Ordering by a constant expression (without any column references) produces the above error, e.g.,
DataFusion CLI v35.0.0
❯ create table a;
0 rows in set. Query took 0.001 seconds.

❯ select * from a order by 'foo';
EnforceSorting
caused by
Internal error: Wrong number of children.
This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker
❯

Expected behavior
No error.
Additional context
No response

@helgikrs when I run the reproducer above on the main branch. Query succeeds. Is there any config parameter, you overwrite the default. Maybe some config parameters trigger this behavior.

I forgot to check latest main, I just checked the v35.0.0 release. I can confirm that this has indeed been fixed there. Closing the issue. Thanks!


Profile creation - skiping mnemonic confirmation by clicking "Next"

Steps:

Go to create profile.
Move mouse to generate mnemonic.
Copy mnemonic somewhere and click next
On mnemonic confirmation choose Next.

Result: Wallet allow create profile without mnemonic confirmation just by clicking Next. Doesn't matter if we put mnemonic or not.
Video: https://share.getcloudapp.com/z8u4Gvb2

Fixed now :+1:


[edge] Uncaught ReferenceError: require is not defined when using @nuxtjs/apollo

Version
nuxt-edge
Reproduction link
https://github.com/Vlaoff/apollonuxt
Steps to reproduce
I get the error when I launch nuxt and open it in the browser
What is expected ?
I should not have this error
What is actually happening?
I think it's a problem with the babel/wepack configuration.
Additional comments?
It does not occur on nuxt 1.4.2, only on the nuxt-edge 2.0.0-xxx (could not select the correct version on cmty.app)
I don't know if the problem is from @nuxtjs/apollo  or nuxt. But as it does not occur on the nuxt stable branch, I submit it here

This bug report is available on Nuxt community (#c7745)

@Vlaoff it seems that something off with graphql 0.13.2 . Nuxt edge have mjs support enabled, and it dont play with with graphql 0.13.2. So its dont seem like a nuxt problem.
You could try to set grapqhl to 14.0.2 and it will make the error go away, although there will be complains about not meet requirements and not sure if it wont break anything.


[Model] 6Marine

The author is Roger [666] Bacon

URL: https://unrealarchive.org/models/unreal-tournament/0/6marine_bcff608d.html
Hash: bcff608d011b5e04f7c8be19c3a7663c199013f5
Current name: 6Marine

Fixed


Which AD role is required to register applications?

You mention that a person must be an admin in the AD tenant, but there are multiple types of admins.  Which specific admin roles have the right permissions?  IT departments don't want to be assigning global admin rights to everyone.

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

ID: 11aaa091-1108-52f6-0066-0f4e939aeef4
Version Independent ID: dbc7e37e-87a7-c124-3e4c-47aa060e6cf3
Content: Create identity for Azure app in portal
Content Source: articles/azure-resource-manager/resource-group-create-service-principal-portal.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@CaryJ Thanks for reaching out! We are investigating and will get back to you ASAP.

@curtand - do you know which AAD admin roles are required for registering an AD app, if the directory does not allow users to register apps? The portal has a flag that indicates whether users can register an app. The help text only says "If this option is set to no, then only users with an administrator role may register these types of applications." But, it doesn't differentiate between the many different types of admin roles.

Or, maybe my question should have gone to @PatAltimore . Can you take a look at the previous comment about the AAD admin role needed to create an AD app?

Assigning to @PatAltimore per update above.

@femsulu  I'm no longer in identity. Please follow up with @mtillman

@PatAltimore Thanks for the redirect.
@mtillman any insights on this please?

@rolyon Do you have any info on the actual role that has perms to register apps? I'm assuming this is an RBAC issue.
@femsulu This issue is still against an article that @tfitzmac owns, so he should remain assigned to it.

@mtillman I'll do some checking to see what I can find out.

@tfitzmac Based on my research, I think you have to be a Global Administrator to register applications. I've also sent a message to the team to confirm.

Thank you for the response.

@CaryJ - thanks for this question. I have updated the article with the requirement of using global administrator. It should get published this afternoon. We will update this article in the future if that requirement changes.
@rolyon - thanks for finding this information.


⚠ DDoS-Relay lin-US-W-lax-01 has degraded performance

In b88df8f, DDoS-Relay lin-US-W-lax-01 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 18 ms

Resolved: DDoS-Relay lin-US-W-lax-01 performance has improved in d95714d after 22 minutes.


Rework the handling of default values for in and out records

This commit addresses both github issues #81 and #67.
The arguments to the a and long record builder functions (aIn, aOut,
longIn, longOut) are brought more into line with the arguments as
used in the epics_device module.  This means that extra unnamed arguments
can be used to specify EGU and PREC (where appropriate), and for out
records the DRV limits take precedence.
As a side effect, the implementation of this is made closer to that in
https://github.com/dls-controls/epics_device/

Regarding the lack of tests, I agree that it would be good to add some more tests.


Model.save

These should be implemented properly. For example, the NeuralNetwork right now only saves the model but not the scaler. I am not sure how to save the Linear one.
There should be a corresponding load.

For linear models, one can simply retrieve the coefficients and intercept from lr.coef_ and lr.intercept_ respectively after fitting. I suppose that would be enough?

Yes, but you should be able to save those coeficients to a json file, and reload the exact same model from the file?


[FRONTEND] Find a way to deactivate getting all localpollens in app.js

Abandoned localpollens for now


Add support for custom project configurations

Hello,
so as the title says, i would like to suggest to add support for custom configurations listed under  in the csproj file. Perhaps we could go further by adding a configuration option for the launch configurations just like with the official .NET MAUI launch configurations.

Hi @AlphaNERD- ! Thank you for your issue. You can customize a default build task in the tasks.json file. Please, check the following wiki:
https://github.com/JaneySprings/DotNet.Meteor/wiki/Predefined-task-customization
For example:
You can create any custom target in the csproj file:
<Target Name="MyCustomTarget">
	<Message Text="Hello from MyCustomTarget" Importance="high" />
</Target>

and create tasks.json file in the .vscode folder with the following content:
{
	"version": "2.0.0",
	"tasks": [
		{
			"type": "dotnet-meteor.task",
			"target": "build",
			"problemMatcher": [
				"$dotnet-meteor.problemMatcher"
			],
			"label": "dotnet-meteor: Build",
			"args": [
				"-t:MyCustomTarget"
			],
			"group": {
				"kind": "build",
				"isDefault": true
			}
		}
	]
}

You can see that your custom target had executed:


Too Many Clicks to Sign Out

I believe this behavior is the same on Windows 10, Linux, and OS X.
If I want to sign out quickly, I can't. It takes five clicks if the app is not in the foreground.

Click on icon in system tray
Click hamburger to get drop-down
Click Settings (why would sign-out be a setting?)
Click Sign Out
Click to confirm

In basically every other chat / instant messaging client I've used, I could right-click on the tray icon and choose to sign out. I much prefer to have that quick sign-out functionality.

Why you can't click on your profile picture, then Sign out? Two clicks for me. Tested on macOS.

Maybe macOS is different. On Windows 10, you click on the tray icon. That brings up a mini window. In the mini window, there is no profile picture. You have to click an icon to bring up the full window. Once you have the full window, you can get to Settings or Profile to Sign out. After you get that far, you still have to confirm. It is still 5 clicks to sign out on Windows 10 and no quicker that I can find.

I will add that the problem is reduced if you uncheck the Always stay logged in advanced option in Settings, because then you can quit the program in three clicks and you will have to enter a password when you open it next time.

I will add that the problem is reduced if you uncheck the Always stay logged in advanced option in Settings, because then you can quit the program in three clicks and you will have to enter a password when you open it next time.

This doesn't work anymore. When I exit the app in Windows and then reopen, it does not require me to log in again.


'make install' needs to be 'sudo make install'

In getting-started/hello_world.md make install needs to be sudo make install.

Merged


Form Validation Styles for radios, checkboxes and toggles

Invalid/Error styles for:

[ ] md-radio-button
[ ] md-checkbox
[ ] md-slide-toggle (if possible)

Also create mixins for validations

We probably wont do anything with this, since most of it comes directly from material.


python311Packages.glyphslib: 6.7.0 -> 6.7.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.glyphslib is: Bridge from Glyphs source files (.glyphs) to UFOs and Designspace files via defcon and designspaceLib
meta.homepage for python311Packages.glyphslib is: https://github.com/googlefonts/glyphsLib
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.7.1 with grep in /nix/store/l2a4qbwp14ljyyvwnv5zrd6a27g49lq1-python3.11-glyphslib-6.7.1
found 6.7.1 in filename of file in /nix/store/l2a4qbwp14ljyyvwnv5zrd6a27g49lq1-python3.11-glyphslib-6.7.1

Rebuild report (if merged into master) (click to expand)

19 total rebuild path(s)

19 package rebuild(s)

First fifty rebuilds by attrpath

fira-math
fontbakery
python311Packages.babelfont
python311Packages.collidoscope
python311Packages.fontbakery
python311Packages.fontfeatures
python311Packages.fontmake
python311Packages.glyphsets
python311Packages.glyphslib
python311Packages.glyphtools
python312Packages.babelfont
python312Packages.collidoscope
python312Packages.fontbakery
python312Packages.fontfeatures
python312Packages.fontmake
python312Packages.glyphsets
python312Packages.glyphslib
python312Packages.glyphtools

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l2a4qbwp14ljyyvwnv5zrd6a27g49lq1-python3.11-glyphslib-6.7.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.glyphslib https://github.com/r-ryantm/nixpkgs/archive/a4639af7d7199d81e12b7b4a379d596a29c30f43.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a4639af7d7199d81e12b7b4a379d596a29c30f43#python311Packages.glyphslib

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l2a4qbwp14ljyyvwnv5zrd6a27g49lq1-python3.11-glyphslib-6.7.1
ls -la /nix/store/l2a4qbwp14ljyyvwnv5zrd6a27g49lq1-python3.11-glyphslib-6.7.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

13 packages failed to build:

fira-math
fontbakery (python311Packages.fontbakery)
fontbakery.dist (python311Packages.fontbakery.dist)
python311Packages.fontmake
python311Packages.fontmake.dist
python311Packages.glyphsets
python311Packages.glyphsets.dist
python312Packages.fontbakery
python312Packages.fontbakery.dist
python312Packages.fontmake
python312Packages.fontmake.dist
python312Packages.glyphsets
python312Packages.glyphsets.dist

20 packages built:

python311Packages.babelfont
python311Packages.babelfont.dist
python311Packages.collidoscope
python311Packages.collidoscope.dist
python311Packages.fontfeatures
python311Packages.fontfeatures.dist
python311Packages.glyphslib
python311Packages.glyphslib.dist
python311Packages.glyphtools
python311Packages.glyphtools.dist
python312Packages.babelfont
python312Packages.babelfont.dist
python312Packages.collidoscope
python312Packages.collidoscope.dist
python312Packages.fontfeatures
python312Packages.fontfeatures.dist
python312Packages.glyphslib
python312Packages.glyphslib.dist
python312Packages.glyphtools
python312Packages.glyphtools.dist

Maintainer pings
cc @barinov274 for testing.

dup of https://github.com/NixOS/nixpkgs/pull/310282


fix osfamily = 'windows' and kernel = 'windows' in lowercase

The value for kernel, osfamily and os.family in Puppet is 'windows' for the Windows OS, not 'Windows' with a capital W.

@kreeuwijk Thank you very much for your contribution.


the KR normalization

Does hictk support KR standardization?

It does support it, in the sense that if a .hic or .[m]cool file contains KR weights, hictk can read and use those weights (e.g. when dumping interactions).
However, there are no plans to add KR support to hictk balance.
The reason being that I do not believe there is a way to implement KR efficiently, so a hictk implementation would be about as slow as e.g. juicer_tools.
If you have some tool that expects KR normalized weights, and that does not allow you to customize the weight names, what you could do is run one of the hictk balance subcommands and tell hictk to name the weights "KR".
Example:
hictk balance scale file.hic --name KR

If you decide to use ICE, you most likely have to specify --no-rescale-weights as well (and if you are balancing .[m]cool files that have already been balanced by cooler or hictk, you may want to pass --no-create-weight-link too).


[E::read_ncigar] No CIGAR operations  issue

Hi,  I'm using strobealing to test alignment performance and I get this error from samtools view command with sam created by strobealign (see below command and output) The .fna reference file is Homo sapiens chromosome 18, GRCh38.p14
the sam creation was succesful, aligning with a fastq that is a pbsim simulated directly from the same NC000018.fna
What am I doing wrong ?
Thanks in advance .
The command samtools view was:
samtools view -bS -T NC000018.fna strobealign_ps01_NC000018.sam -O BAM -o strobealign_ps01_NC000018.bam -@ 12
and the output was
samtools view: error reading file "NC000018.sam"
[E::read_ncigar] No CIGAR operations
samtools view: error closing "NC000018.sam": -5

This is the command and output of sam ceration with strobealign
 /packages/StrobeAlign/build/strobealign NC000018.fna ps01_NC000018.fastq -o strobealign_ps01_NC000018.sam -t 12

Using
k: 23
s: 17
w_min: 5
w_max: 15
Read length (r): 10032
Maximum seed length: 278
Threads: 12
R: 2
Expected [w_min, w_max] in #syncmers: [5, 15]
Expected [w_min, w_max] in #nucleotides: [35, 105]
A: 2
B: 8
O: 12
E: 1
Time reading references: 0.965166 s

ref vector approximate size: 11481897
Ref vector actual size: 10783788
Time generating seeds: 2.99098 s

Time sorting seeds: 1.18937 s

Unique strobemers: 9922754
Total time generating flat vector: 4.18048 s

Flat vector size: 10783788
Total strobemers count: 10783788
Total strobemers occur once: 9840601
Fraction Unique: 0.991721
Total strobemers highly abundant > 100: 783
Total strobemers mid abundance (between 2-100): 81369
Total distinct strobemers stored: 9922754
Ratio distinct to highly abundant: 12672
Ratio distinct to non distinct: 120
Filtered cutoff index: 1984
Filtered cutoff count: 41

Total time generating hash table index: 0.584856 s

Total time indexing: 5.73055 s

Using rescue cutoff: 82
Running SE mode
Done!
Total mapping sites tried: 502929
Total calls to ssw: 502929
Calls to ksw (rescue mode): 0
Did not fit strobe start site: 1585
Tried rescue: 16813
Total time mapping: 91.5354 s.
Total time reading read-file(s): 13.2334 s.
Total time creating strobemers: 2.68648 s.
Total time finding NAMs (non-rescue mode): 0.603002 s.
Total time finding NAMs (rescue mode): 0.0492406 s.
Total time sorting NAMs (candidate sites): 0.0038492 s.
Total time reverse compl seq: 5.79428e-311 s.
Total time base level alignment (ssw): 3.01523 s.
Total time writing alignment to files: 6.92938e-310 s.

Hi @rudigaspardo,
Thanks for testing strobealign. From the output, it looks like you are aligning long reads (median read length is 10,000bp).
Strobealign is currently only a short-read aligner tested for Illumina reads up to 2*500bp. I believe the error comes from the extension alignment we perform, which will stop if forced to aligning through contiguous long segments over 2kbp.
We do have plans to extend strobealign to long-read alignment  to also perform long-read alignment issue #24.
If you have short reads to align we appreciate your feedback.

I’m cleaning up the list of open issues a bit at the moment. I’m going to close this issue as a duplicate of #24.


MudTable: "Rows per page"-Selection not aligned with other elements

Bug type
Component
Component name
MudTable
What happened?
After the update from the Version 5.1.5 to 5.2.0 the Rows per page-Selection is not aligned with the other elements anymore.

Expected behavior
Same behavior as in older version.
Reproduction link
https://mudblazor.com/components/table#table-with-pagination-and-filtering
Version (bug)
5.2.0
Version (working)
5.1.5
What browsers are you seeing the problem on?
Firefox
On what operating system are you experiencing the issue?
Windows, Mac OSX
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Duplicate of #698


Fix total RPC response length byte

I might be wrong, but I think the implementation using the Arduino String class still has this issue. There, no +1 was added. But I was just reading the code, so again this might be totally fine.


Selection not stopping after releasing click on Firefox

When I am trying to select an interval, it looks like the mouseup event is not triggered, and the selection does not stop even if I am not holding click anymore. The only way to stop, is by releasing the click on the expanding zone (little square). I also observed that this only reproduced when selecting from left to right, not from right to left.
Browser: Firefox 81.0.2 (64-bit)
OS: MacOs Catalina 10.15.6

I have the same issue, any solution or workaround?

This still remains an issue. Any possible fixes planned for this?

This still remains an issue. Any possible fixes planned for this?

I'm a bit disappointed that over the course of a full year, there hasn't even been a single reply. Is this something you guys can look into or at least give us an update about?
Thanks!

Being new to this repo, I can say that for the past couple of years, there has only been one person managing this extremely popular project. As you can imagine, he does not have time to answer and respond to every issue or question posted. There are literally thousands of them.
I'm a bit disappointed that the community has not stepped up to help address any of these issues in over a year. I mean, the code is freely available and PRs are always welcomed.
This project needs the community to step up and help wherever they can.

Is there any known WA for this issue?

Hey Everyone
Sorry I know this was closed. but has anyone got a workaround for this?


[Fix] Gluon-NLP benchmark script fix

Description
When I tried to run benchmarks only with CPU I got NaNs in results:
2020-10-20 14:11:39,221 - benchmark_utils - INFO - 1 / 1
2020-10-20 14:11:44,714 - benchmark_utils - INFO - Can't pickle local object 'measure_peak_memory_cpu.<locals>.MemoryMeasureProcess'
2020-10-20 14:11:44,732 - benchmark_utils - INFO - Saving results to csv train_fp32_NT_NT/google_en_uncased_bert_base_4_128.csv.
2020-10-20 14:11:44,733 - benchmark_utils - INFO -
====================            TRAIN - RESULT - SPEED - RESULTS           ====================
2020-10-20 14:11:44,733 - benchmark_utils - INFO - -----------------------------------------------------------------------------------------------
2020-10-20 14:11:44,734 - benchmark_utils - INFO -           Model Name             Batch Size     Seq Length    Latency (ms)      Memory
2020-10-20 14:11:44,734 - benchmark_utils - INFO - -----------------------------------------------------------------------------------------------
2020-10-20 14:11:44,734 - benchmark_utils - INFO -  google_en_uncased_bert_base         4             128            nan            nan
2020-10-20 14:11:44,734 - benchmark_utils - INFO - -----------------------------------------------------------------------------------------------
2020-10-20 14:11:44,734 - benchmark_utils - INFO -
====================        ENVIRONMENT INFORMATION         ====================
2020-10-20 14:11:44,777 - benchmark_utils - INFO - - gluonnlp_version: 1.0.0.dev
- framework_version: 2.0.0
- python_version: 3.8.3
- system: Linux
- cpu: x86_64
- architecture: 64bit
- date: 2020-10-20
- time: 14:11:44.775970
- fp16: False
- cpu_ram_mb: 191906
- use_gpu: False

The reason was that MemoryMeasureProcess class was defined inside function.
This PR moves MemoryMeasureProcess class and get_cpu_memory function outside of measure_peak_memory_cpu function what solves the problem.
Checklist
Essentials

[X] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[X] Changes are complete (i.e. I finished coding on this PR)

cc @dmlc/gluon-nlp-team @sxjscience

LGTM. Thanks for the fix!


Bonners Events can select training or service hour options

Problem
Whenever creating or editing a Bonner event, the option for the event to be a training or service hours. Whenever the service hours or training option is toggled and the event is saved, the event saves as both Bonners Scholar event and the other option.

Change required
Have Bonners event not able to toggle the options training or service hours option, show the user that these options cannot be use, or simply hide them!

 Bonners Events can select training or service hour options #1340


livereload

I have tried many tutorials about angular-cli and docker, the only thing I miss is live reload when I change a file. Does your project has this or do I confuse some things?

I use the --port flag, for detecting changes and it works (with my own angular-cli image tho).


Cuda?

Hi! Thank you for making this project available!
What changes would need to be made to run on the gpu?
I am looking for around 20ms processing speed.
Thanks!

Hi! Thank you for making this project available!
What changes would need to be made to run on the gpu?
I am looking for around 20ms processing speed.
Thanks!
I am using the CPU version of onnxruntime. If you want to use the GPU version, you may need to add AppendExecutionProvider_CUDA (OrtCUDAProviderOptions) to the place where Ort::SessionOptions is initialized. This version of the code does not currently support the use of GPU. For The issue of speed is also the part I always want to improve. If you can implement a version of onnxruntime-gpu, you are very welcome to initiate a PR. I will try to implement it later when I spare time, because the development and application environment has always been a CPU environment.
Thank you for your attention😊

I have made the following changes to the cpp file :
\src\LightGlueOnnxRunner.cpp
and i have this code running on CUDA!
if (cfg.device == "cuda") {
    std::cout << "[INFO] OrtSessionOptions Append CUDAExecutionProvider" << std::endl;
    OrtCUDAProviderOptions cuda_options;

    cuda_options.device_id = 0;
    cuda_options.cudnn_conv_algo_search = OrtCudnnConvAlgoSearchExhaustive;
    cuda_options.gpu_mem_limit = static_cast<int>(SIZE_MAX * 1024 * 1024);
    cuda_options.arena_extend_strategy = 1;
    cuda_options.do_copy_in_default_stream = 1;
    cuda_options.has_user_compute_stream = 1;
    cuda_options.default_memory_arena_cfg = nullptr;
    session_options.AppendExecutionProvider_CUDA(cuda_options);

    session_options.SetGraphOptimizationLevel(GraphOptimizationLevel::ORT_ENABLE_EXTENDED);
}

I see around a 4x speedup on 512 images:
cpu: 1182ms
cuda: 384
Vary interested to see what is costing the most, once you decouple the two models.
Thanks!

I have made the following changes to the cpp file :
\src\LightGlueOnnxRunner.cpp
and i have this code running on CUDA!
if (cfg.device == "cuda") {
    std::cout << "[INFO] OrtSessionOptions Append CUDAExecutionProvider" << std::endl;
    OrtCUDAProviderOptions cuda_options;

    cuda_options.device_id = 0;
    cuda_options.cudnn_conv_algo_search = OrtCudnnConvAlgoSearchExhaustive;
    cuda_options.gpu_mem_limit = static_cast<int>(SIZE_MAX * 1024 * 1024);
    cuda_options.arena_extend_strategy = 1;
    cuda_options.do_copy_in_default_stream = 1;
    cuda_options.has_user_compute_stream = 1;
    cuda_options.default_memory_arena_cfg = nullptr;
    session_options.AppendExecutionProvider_CUDA(cuda_options);

    session_options.SetGraphOptimizationLevel(GraphOptimizationLevel::ORT_ENABLE_EXTENDED);
}

I see around a 4x speedup on 512 images:
cpu: 1182ms cuda: 384
Vary interested to see what is costing the most, once you decouple the two models.
Thanks!

LightGlueDecoupleOnnxRunner supporting decouple model inference of SuperPoint/DISK + LightGlue have finished , and both mode support calling GPU inference and i complete README.md experimental data.Welcome to use and give feedback😊


Updated roles

Team, I am the GBB for customer services AMER
I have noticed that our new roles for service activity scheduling are diferente than the ones in this document.
The current ones are:
Customer Service Scheduler
Customer Service Scheduler Administrator
Can we double check that and if that's the case update the document?
Tahnk you very much

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

ID: c3c19399-ce11-4139-46a0-3a6102ee11e4
Version Independent ID: c06b80df-f556-3c23-51c2-62cc75f67d9b
Content: Manage security roles in Service Scheduling
Content Source: ce/customer-service/manage-security-roles.md
Service: dynamics-365-customerservice
GitHub Login: @lalexms
Microsoft Alias: laalexan

Thanks for your comment, @moliveirapinto. This issue has been assigned to the doc author for review.

Thank you for the heads up about this, Mauricio. I'm reassigning to the writer who is working on this content.

@moliveirapinto : Scheduler Manager and Scheduler are parent security roles, whereas Customer Service Scheduler and Customer Service Scheduler Administrator are roles that need to be added for core service scheduling. The documentation also states the same.
Hope that answers your question. Closing the task. Please feel free to raise another issue if you have any further questions.


Add return type void documentation

Wasn't sure if this deserved to be in the void part of this document. It more has to do with function assignability but I'll see what maintainers have to say about it. In addition, wasn't sure the best way to link to other things so let me know how to change that if necessary.
Rel: https://github.com/microsoft/TypeScript/issues/41138 (already closed by bot)

Yeah, this is great, and a perfect  started for the assignability section. Solid work 👍🏻


Issue with docker-compose ps

Hey project team,
I had issues trying to use polymesh-local due to the docker-compose ps call in the containersUp method.
I'm currently using docker-compose V2:
❯ docker-compose --version
Docker Compose version v2.12.2

Since the docker-compose is being called in the same directory as the docker-compose.yml but without any of the environment variables defined, the ps call is failing with:

WARN[0000] The "FAKETIME" variable is not set. Defaulting to a blank string.
WARN[0000] The "CHAIN" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "PG_HOST" variable is not set. Defaulting to a blank string.
WARN[0000] The "PG_USER" variable is not set. Defaulting to a blank string.
WARN[0000] The "PG_PASSWORD" variable is not set. Defaulting to a blank string.
WARN[0000] The "TOOLING_IMAGE" variable is not set. Defaulting to a blank string.
WARN[0000] The "REST_IMAGE" variable is not set. Defaulting to a blank string.
WARN[0000] The "VAULT_TOKEN" variable is not set. Defaulting to a blank string.
WARN[0000] The "LOCAL_SIGNERS" variable is not set. Defaulting to a blank string.
WARN[0000] The "LOCAL_MNEMONICS" variable is not set. Defaulting to a blank string.
WARN[0000] The "VAULT_URL" variable is not set. Defaulting to a blank string.
WARN[0000] The "TOOLING_API_KEY" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "PG_DB" variable is not set. Defaulting to a blank string.
WARN[0000] The "PG_PORT" variable is not set. Defaulting to a blank string.
WARN[0000] The "PG_HOST" variable is not set. Defaulting to a blank string.
WARN[0000] The "PG_USER" variable is not set. Defaulting to a blank string.
WARN[0000] The "PG_PASSWORD" variable is not set. Defaulting to a blank string.
WARN[0000] The "SUBQUERY_IMAGE" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "POLYMESH_VERSION" variable is not set. Defaulting to a blank string.
WARN[0000] The "DATA_DIR" variable is not set. Defaulting to a blank string.
WARN[0000] The "FAKETIME" variable is not set. Defaulting to a blank string.
WARN[0000] The "CHAIN" variable is not set. Defaulting to a blank string.
WARN[0000] The "FAKETIME" variable is not set. Defaulting to a blank string.
WARN[0000] The "CHAIN" variable is not set. Defaulting to a blank string.
service "subquery" has neither an image nor a build context specified: invalid compose project

I could hack around this issue by creating some sensible defaults in thedocker-compose.yml:
   subquery:
-    image: '${SUBQUERY_IMAGE}'
+    image: '${SUBQUERY_IMAGE:-polymeshassociation/polymesh-subquery}'
     restart: always
     depends_on:
       - 'postgres'
@@ -88,7 +88,7 @@ services:
       - --local

   tooling:
-    image: ${TOOLING_IMAGE}
+    image: ${TOOLING_IMAGE:-polymeshassociation/polymesh-tooling-gql}
     restart: always
     depends_on:
       - 'postgres'
@@ -101,7 +101,7 @@ services:
       STAGE: dev

   rest_api:
-    image: ${REST_IMAGE}
+    image: ${REST_IMAGE:-polymeshassociation/polymesh-rest-api}
     restart: always
     ports:
       - 3004:3000

Cheers,
Hui

@huiwang1  Thanks for opening this, and sorry for the late response, I missed seeing this.
Is this still an issue? Setting sensible defaults for these makes sense to me, however it would be better if it would work consistently with different versions.
I am currently using docker-compose version 1.29.2. It seems like there is something missing for proper v2 support.
I've been meaning to migrate to the built in docker compose instead of needing docker-compose installed as well.
If this is still an issue I can look into getting that prioritized.

Yeah I'm using docker compose v2.13.0 but by adding the sensible defaults I'm able to work around the issue. Just wanted to let you guys know for future reference. Cheers.

For a docker compose v2 compatible version, I've been using a plain docker-compose file:
https://github.com/polymath-eric/scripts/tree/main/compose
I'll likely tweak this for my own development needs, but it should be a good place to copy and get similar functionality to polymesh-local as an alternative.


Latest text-generation-webui

Hey,
Had a quick look at your Dockerfiles but uncertain if it always uses latest text-generation-webui or if there is a way to pull that when running?
Either way I couldn't get thebloke/cuda11.8.0-ubuntu22.04-oneclick:latest running because of a missing requirement, same as this issue. It's fixed in this commit.
A workaround is to just install rich in a web terminal when running on runpod but.. yeah.

Thanks for the report. I have rebuilt the docker now which has brought in the new requirement.
To answer your question: the Docker git pulls the latest text-generation-webui on each boot, so it's kept up-to-date with latest code.  But it doesn't automatically re-install the requirements - that's only done when the Docker is built.
Therefore when new pip packages are needed, I need to re-build the Docker container.
Please stop and re-start your instance, or kill it and make a new one, to get the latest version of the Docker container.  And let me know if it's working OK now.

Great, thanks

No, thank you 😄


Create Adapter.java

hope this helps:) u just need to fix UI and some minor bugs like check if media is playing before stopping it(make btn unclickable if media isn't playin)

u are awesome bro
Keep it Up!!


Update  Sharethrough documentation to include video media type, floors, and safeframes

Update Sharethrough documentation to include video media type, floors true, and safeframes true.

Thanks @poojawins . Is this related to PR on prebid.js ? If yes, could you link it :smile:

I don't believe there is a related PR on prebid.js, this is just to get our documentation synced @muuki88

True that. Integration is already released

https://github.com/prebid/Prebid.js/pull/7290
https://github.com/prebid/Prebid.js/pull/6874

Thanks for the safe_frame information

Hi @muuki88 can this be merged?

Thanks for the ping 🤗 It got lost during Christmas 😅


"cookie="

Hi,
uhm..
Do you have an example for this cookie?
I have several entries in my "facebook cookie" ;)
Thanks,

I just copied a random request as curl and used the whole cookie:

Hm. this make sense.
Thanks!

I can't find it -_-


GM_xmlhttprequest

TamperMonkey / ViolentMonkey only provide GM_xmlhttprequest
can U add an option to config this? or just detect at runtime
current I'm using webpack provide to do the rename stuff
// webpack.config.js
new ProvidePlugin({
   GM: __dirname + '/src/GM.ts',
 })

// src/GM.ts
export const xmlHttpRequest = GM_xmlhttpRequest

I'm pretty sure TamperMonkey provide GM object and it's http request API.

And violent monkey also provide this API https://violentmonkey.github.io/api/gm/#gm_xmlhttprequest

you need to grant the permission of GM.xmlHttprequest instead of GM_xmlhttprequest

Thanks.  I missed that part of violentmonkey doc.


[feat] Precompile MODEXP

Work plan:

[x] Induce and wrap modexp circuit from Gaoxin's work
[x] Parse memory bytes for input and connect with the wrapped modexp circuit via ModExpTable
[x] Error handling: have to prove any invalid input bytes lead to empty output (the precompile always has a successful call)

This work would base on the PR of ECRECOVER to make use of some essential feature
[DO NOT MERGE] until following has completed:

[x] Include #601 so we can enable both ecrecover and modexp
[x] adjust to the behavior identical with l2geth (only len=32 is valid and return error in invalid input)

Pending tasks (not planned in current PR)

[ ] truncate / zero padding the input bytes according to expected input size
[ ] gas cost
[ ] pass a mock prove

i asked gaoxin to not enable ark-std print-trace by default. CI will be fixed then.

Notice that the geth-util apply different (the upstream geth) behavior in modexp. I had to induce another modifier in test_util to tune the input trace for unittest

A small issue: is it possible to pick specific commits of dependencies of gaoxin so we can remove ff v0.13 in cargo lock？

i noticed so many codes is used to hande padding bytes..

i noticed so many codes is used to handle padding bytes..

Yes. Some code is written before PowOfRand table can be used. Now they have been pruned.