Program Arguments?

I may have missed something, but I can't seem to figure out how to launch an application with a command-line parameter. I URL encoded the query string parameter...
For example, "c:\windows\system32\cmd.exe /c ping <IP_ADDRESS>" is "program=c%3A%5Cwindows%5Csystem32%5Ccmd.exe%20%2Fc%20ping%20<IP_ADDRESS>"
But it fails to launch with an access denied error.
I can launch cmd, and many others apps, without parameters. I'm also the owner of the server so there isn't any permission issues.

I did see this thread https://github.com/cedrozor/myrtille/issues/41

This is a configuration issue. Check this link: https://social.technet.microsoft.com/Forums/windowsserver/en-US/9c78b735-2929-4f8c-ac7f-3d93efd0f322/using-dynamic-commandline-arguments?forum=winserverTS

Thanks, @cedrozor! That's what I was missing.


the pengouin example is read-only

I can't download the flr file so i can't follow the instructions and I don't understand how to use this package. pls upload another example?

Thanks for bringing this to our attention! The file can be forked now. https://rive.app/a/JuanCarlos/files/flare/penguin/preview


[Research] Multiple contract support

Is your feature request related to a problem? Please describe.
Currently, the wallet selector modal APIs do not allow for managing function call access keys. APIs might have to change in the wallet selector core APIs as well to be able to check if there is an access key for the given contract, or just to ensure they are signed into an injected wallet.
The reason this is useful/necessary is if someone wanted to interact with multiple contracts within the same frontend, they would need to sign in and generate a function call access key for each time they interact with a new contract (using a hosted wallet).
This is a problem because not only is it a bad UX, but also it charges unnecessary gas by throwing away usable function call access keys. The pattern of interact with contract A -> interact with contract B -> interact with contract A requires that you generate two function call access keys for interacting with contract A when you should only need one.
The assumption based on how the code is set up is that there is a 1:1 mapping of wallet -> contract, which is very invalid.
Describe the solution you'd like

Keep contractId mandatory and expose a method to switch the contract, like updateContractId as a callback (along with hide and show)

Alternatively: Modal show method takes a contractId parameter (optional) and make the contractId parameter of ModalOptions optional as well. Assumption is error would be thrown if neither is used. This might be able to be restricted based on TS type system (in theory it should be able to be) so that either is required.

Assumption that wallets maintain contract id -> access key mappings, if not, adjust the selector core APIs to convey the switch in account or expose the generated access keys from the selector. (unclear the semantics around requestSignIn , is the implication that it will force another sign in, or if it's already signed in for the account it will short-circuit?)

Describe alternatives you've considered
Acceptance criteria Allow interacting with multiple contracts with a single wallet selector
Additional context Motivation is for building a frontend that interacts with contracts dynamically at runtime.

@austinabell This currently is not possible to add because wallets like MyNearWallet only allow you to one add access key to one contract when signing in. This will need to first be addressed on the wallet side and/or to update the near-api-js library.
Right now we call the sign in function like this: wallet.requestSignIn({ contractId, methodNames }).
Maybe we can have something like this: wallet.requestSignIn([{ contractId, methodNames }, { contractId, methodNames }]); which will allow us to add multiple access keys with one approve/sign in.
Example of adding full access key to guest-book.testnet when signing in:

In explorer you can see that we only create a full access key for guest-book.testnet.

This looks like a big change, I am not completely sure about how to approach this. Maybe @MaximusHaximus can comment more on this.

Currently with MyNearWallet there is no way to generate multiple function call access keys for different contracts with one sign in. This is the limitation of some wallets that is preventing us to continue with this task.
The new Injected Wallet NEP suggests to use only one or more contract id when signing in. This will allow us to use multiple contracts.
After we align the wallet standards to be like Injected Wallets then we can integrate this feature.

+1

Closing this as it will be covered by the work in: https://github.com/near/wallet-selector/pull/811


Exit game controller

Note

Copy priority queue contracts to here.

Upgrade to solidity 0.5
Use open-zeppelin library: Ownable, SafeMath

Implement ExitGameController and unit tests

close #126
Side notes
If using Visual Studio Code, use the following setting to remove import error (for open-zeppelin).
"solidity.packageDefaultDependenciesContractsDirectory": "",
"solidity.packageDefaultDependenciesDirectory": "plasma_framework/node_modules"

Test
truffle test
Contract: ExitGameController
    hasToken
      ✓ returns true when token already added (177ms)
      ✓ returns false when token not added (52ms)
    addToken
      ✓ rejects when token already added (174ms)
      ✓ generates a priority queue instance to the exitsQueues map (128ms)
      ✓ emits TokenAdded event (110ms)
    enqueue
      ✓ rejects when not called from exit game contract (150ms)
      ✓ rejects when such token has not been added yet (200ms)
      ✓ increases `exitQueueNonce` one (171ms)
      ✓ inserts the new unique priority to the queue (187ms)
      ✓ saves the exit data to map (160ms)
      ✓ can enqueue with the same exit priority multiple times (221ms)
    processExits
      ✓ rejects when such token has not been added yet (165ms)
      ✓ rejects when the exit queue is empty (115ms)
      ✓ rejects when the "top unique priority" mismatch with the specified one (182ms)
      ✓ processes when the "top unique priority" is set to 0 (183ms)
      ✓ processes when the "top unique priority" is set to the exact top one of the queue (289ms)
      ✓ calls the "processExit" function of the exit processor when processes (194ms)
      ✓ deletes the processed exit from queue after processed (236ms)
      ✓ deletes the exit data after processed (222ms)
      ✓ processes no more than the "maxExitsToProcess" limit (242ms)
      ✓ does not process exit that is not able to exit yet (162ms)
      ✓ stops to process when queue becomes empty (209ms)

Contract: ExitGameRegistry
    onlyFromExitGame
      ✓ accepts call when called by registered exit game contract (47ms)
      ✓ reverts when not called by registered exit game contract

Contract: PriorityQueue
    getMin
      ✓ can get the min value (66ms)
      ✓ fails when empty
    currentSize
      ✓ can get current size (40ms)
    insert
      ✓ can insert one value (60ms)
      ✓ can insert multiple values with order (122ms)
      ✓ can insert out of order and still get the minimal number (88ms)
      ✓ is not idempotent (85ms)
    delMin
      ✓ can delete when single value in queue (76ms)
      ✓ can delete when multiple values in queue (121ms)
      ✓ can delete all (141ms)

@pgebal
re: adding to CI
Have we finalized our decision on JS testing? I am sort of waiting the result of https://github.com/omisego/plasma-contracts/issues/116. From my last understanding it seems like people are more and more agreed on using JS because of tooling and to be consistent with deployment tool we are going to use.
@pik694 do you have any idea what's the current state of the python testing? And what's your opinion on the two side since you've probably has most insight on both.


left hand attached to my little finger

So, my right hand is following normally my hand, but the left hand is following my little finger as we can see in the video, this make it really hard to interact with thigs such as UI, i am using the  XR plug-in management with the mrtk, and using the version 2020.3.20f1 of unity.
https://user-images.githubusercontent.com/35118366/146414245-39505485-74d9-4251-bfc2-7ea03d16c09a.mp4
and this is my input system window

forget it, i've just fixed it, change the rotation/handbonedata to pointer/rotation

Glad to hear you were able to resolve it!


Add :editor objed module

Integrates objed with Doom Emacs to provide text-object manipulation for non-evil users.
See https://github.com/clemera/objed

Shouldn't be clear in the README or somewhere that this must not be use alongside evil? I understand the idea of being in the editor section as it is a sort of add-on for editing purposes but at the end is an alternative to evil at least in the modal editing aspect so not sure if it should be in feature instead.

@richerve Thanks, I've updated the README with a note regarding objed vs evil.  Let me know if you think this is clear enough, I'm happy to revise it.
Regarding whether this belongs in :editor or :feature, I'll leave that decision with @hlissner.

This is now ready for final review.

Agree with the category decision to be up to @hlissner, in any case I think :editor works well.

The :feature category is due for removal in the next batch of commits. :feature evil will be in :editor from that point on. In any case, I'll review this over the weekend.

Rebased to fix conflict with modeline segment.

Updated to remove the dubious activation hacks as discussed.  Ready for review!

Looks good! Thanks again, and for your continued work around here. You've been a huge help!

Thanks :)


More able to read light monika diff conflicts

#39

[ ] Works on windows

Looks good on
IntelliJ IDEA 2018.2.3 (Ultimate Edition)
Build #IU-182.4323.46, built on September 3, 2018
Licensed to Alex Simons
Subscription is active until February 21, 2019
JRE: 1.8.0_152-release-1248-b8 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0


Add Ingress option to Keptn installer manifests for control plane

Is your feature request related to a problem? Please describe.
As I would like to use the GitOps paradigm for deploying Keptn to my Kubernetes Cluster
Then I would like the ability to specify ingress in the values.yaml
When Deploying with ArgoCD so I don't have to run ingress separate after
I would much rather it be an option in the helm chart so I have a clean deployment, and everything is declared in one place, instead of running it after.
Describe the solution you'd like
This ingress.yaml should be in the installer/manifests/keptn/charts/control-plane/templates and its config in the values.yaml under a new section called ingress (if under apiGatewayNginx it could create confusion).
The solution needs to have the ability to say enabled: true/false this will not break any existing or future deployments or config.
It also needs to cater for annotations, hosts, and tls as options
Describe alternatives you've considered
Alternative solutions are to not have this and just create my ingress after. eg kubectl create -f ingress.yaml
Additional context
I'm happy to write this code myself

@Dynatrace-Adam-Gardner @grabnerandi

Follow-up documentation ticket: https://github.com/keptn/keptn/issues/5297


Feature/add sparse dense

This PR adds support for sparse-dense vectors.
It addresses bugs mentioned in #24 and #26.

@miararoy - yes this was regenerated. Updated tests.


'UnsafeRawBufferPointer' is not convertible to 'UnsafePointer<_>'

I used this integration to report this error.
use xcode 10.1 ,swift 4.2
I tried pod version 0.11.6 and the issue still exists. I do not want to update my Xcode yet due to other SDKs that I am using my project.

Any solutions?

Having the same issue, it seems that  0.11.6+ should be marked as "Swift 5 Required" and not "Swift 5 Supported"
It seems like possibly replacing the implementation of datatypeValue in Foundation.swift with the following could be a possible solution to continue supporting both versions but I am unclear how safe this change actually is.
        #if swift(>=5.0)
        return withUnsafeBytes { (pointer: UnsafeRawBufferPointer) -> Blob in
            return Blob(bytes: pointer.baseAddress!, length: count)
        }
        #else
        return withUnsafeBytes { (pointer: UnsafePointer<UInt8>) -> Blob in
            return Blob(bytes: pointer, length: count)
        }
        #endif
    }```

If you make this change and also change SWIFT_VERSION to 4.2, all the tests seem to pass on Xcode 10.1 at least

try this
add these code into your podfile
post_install do |installer|   installer.pods_project.targets.each do |target|     target.build_configurations.each do |config|       config.build_settings['SWIFT_VERSION'] = '4.2'       config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '9.0'     end   end end
and set pod 'SQLite.swift', '0.11.5'


Asking Sample code

Can you send me a sample code where you are sending message just to one person not to all of them?

Hello. I'm afraid I cannot write simple small example. Just because it depends on User model in your project and I don't know how you will process direct message.
But I can describe you how to deliver message to the needed client.
You have to implement search function in the onFilteredMessage method. Here you can define target user with assigned socket client and use fwrite to send message to him

Also there is another non-secure approach: You can specify userId in the message and show it only if logged user has the same id

I hope my answer helps you.
I decide to create another project to show you how to build Laravel Messenger. Because this topic about Laravel Chat and your question more relevant to Messenger


fix: amazon uk email format change

Proposed change

Check Amazon email body for order numbers.
Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation update
[ ] Adds a new shipper
[ ] Update existing shipper

Additional information

This PR fixes or closes issue: fixes #575

Codecov Report

Merging #576 (9855502) into dev (d4484d3) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev     #576   +/-   ##
=======================================
  Coverage   98.77%   98.77%
=======================================
  Files           6        6
  Lines        1304     1306    +2
=======================================
+ Hits         1288     1290    +2
  Misses         16       16

Impacted Files
Coverage Δ

custom_components/mail_and_packages/helpers.py
99.58% <100.00%> (+<0.01%)
:arrow_up:


In mkroot: Combination of CROSS= and NOLOGPATH= breaks PATH

Command: ./scripts/mkroot.sh dropbear CROSS=aarch64 NOLOGPATH=1 V=s
Expectation: Root is built with a cross-compiled dropbear, the same as if NOLOGPATH had not been specified
Result: cross compiler is not seen in path. Configure for dropbear detects and uses host compiler, resulting in a build with the host compiler. Then ${CROSS_COMPILE}strip dropbearmulti fails with /home/user/toybox/ccc/aarch64-linux-musleabi-cross/bin/aarch64-linux-musleabi-strip: Unable to recognise the format of the input file 'dropbearmulti'
Mitigation: compiling without NOLOGPATH specified seems to always work correctly

Commit 43f61a67b835

Did you know that if you write a commit with a hash while the push is still ongoing in another window so github doesn't turn the hash into a link, that if you edit and re-save the comment after the push completes it STILL doesn't turn it into a link?
I break everything.

In the absence of follow-up, I'm assuming the commit above fixed the issue.


New Mailto Button - Call for suggestions

Hey there,
the current version has just a "Mailto" Link after the Body text.
Maybe we can add a button with a Mail icon or so? Any suggestions?

Add a new Mailto Button in 0.6.0


build broken

Describe the issue
the package is broken/ won't build
Code or steps to reproduce
install most recent version of klasa ( github/0.5.0 )
run code that uses klasa ( causes entry point error because klasa won't build)

Expected and actual behavior
no errors

Further details

@klasa/core version: n/a
node.js version:12.0.0
Klasa version:0.5.0
[ ] I have modified core files.
[x] I have tested the issue on latest master. Commit hash: 3dd2cbdca0f4a71f0dedddf6bbd779b3173f9de0

Install stable instead.
# Install with npm
$ npm i klasa discord.js

# Install with yarn
$ yarn add klasa discord.js

how about for dev?


No console logging

I'm open to other ideas besides NODE_ENV=debug, but having it print stuff out to the console by default is incredibly annoying, especially when you can't change the names of your columns.

This makes sense to me.  Sorry for adding that annoying logging in there earlier!  Wanted to make things easier to use out of the box, but I understand its a bit presumptuous to log without being able to stop it in someone else's environment. Will push a new version to npm shortly!


antivm improvement

I want most of the stuff to be hard coded into the program so in case something happens to the github repo, it won't affect the program.
Besides, I don't think Anti VM needs any improvements for now.


FAFO Backlash

Suggested new feature or improvement
Don't install it without asking first.
Scenario
"Microsoft research indicates that you are a Developer, so we installed Dev Home for you."  Yeah? This is my home GAMING PC, and I don't 5H1T where I EAT.
Additional details
The OSS community has provided you with functionality that your own devs haven't.  Your suggestion that this relationship needs more invasive insertion of your talent-extraction desires by means of bloatware on ALL of their machines including their personal, non-development, pc's is going to cost you. BACK OFF.

/dup https://github.com/microsoft/devhome/issues/1615


Ensure that CI includes validation of Item Description files

When any Item Description is updated:

run CI
validate content
check all vocab terms exist in the vocab service/lookups

Duplicated elsewhere.


Please confirm these before moving forward

[x] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[x] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Other
Brief description of your issue
After installation the current version is Unknown.
Steps to reproduce

Install the package winget install Mikrotik.TheDude
See that the version shows up as unknown

Name                                      Id               Version          Available        Source
---------------------------------------------------------------------------------------------------
The Dude                                  Mikrotik.TheDude Unknown          7.16             winget

Actual behavior
The installed versions always is Unknown.
Expected behavior
Installed version is correctly understood and winget does not always prompt for updates.
Environment
Windows Package Manager v1.8.1911
Copyright (c) Microsoft Corporation. All rights reserved.
Windows: Windows.Desktop v10.0.19045.5011
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.23.1911.0
Winget Directories
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                D:_sort_downloads
Links
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale
Admin Setting                             State
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled
ProxyCommandLineOptions                   Disabled
DefaultProxy                              Disabled
Screenshots and Logs
Found The Dude [Mikrotik.TheDude] Version 7.16
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.mikrotik.com/routeros/7.16/dude-install-7.16.exe
  ██████████████████████████████  2.22 MB / 2.22 MB
Successfully verified installer hash
Starting package install...
Successfully installed

🎉 Issue reproduced and identified!
The package does not write any version information to Windows Registry, so winget.exe can't get any version information from ARP Entries. winget.exe will mark this package as version Unknown and fall into an "infinite upgrade loop" ...

Since this is a Nullsoft install wizard, winget.exe will not write any ARP entries for the package. All these stuff are done by the installer itself. So this might be a https://github.com/microsoft/winget-pkgs/labels/Blocking-Issue with https://github.com/microsoft/winget-pkgs/labels/Area-External.
16:54:59 D:\...\winget-pkgs  [master ≡] 5ms pwsh> komac show --installer-manifest Mikrotik.TheDude
PackageIdentifier: Mikrotik.TheDude
PackageVersion: '7.16'
InstallerType: nullsoft
ReleaseDate: 2024-09-24
Installers:
- Architecture: x64
  InstallerUrl: https://download.mikrotik.com/routeros/7.16/dude-install-7.16.exe
  InstallerSha256: D3382A4F472FF84F4174BE3A108AF77E06108C2CE7D9355248AC6D543BE8CF07
ManifestType: installer
ManifestVersion: 1.6.0

This is something of a known issue with packages whose installers do not write version data to the Registry.

[Policy] Installer Issue


Not Working in release mode

It's working fine with debug mode but. Not working in release mode . Storage not opened as expected .
targetSdk : 30
Using latest version of flutter and image picker

Maybe it's a problem with storage permission with lower sdks?
Here's the instruction to add the permission needed.
Hit me up if it worked for you.

@officer47p I already tried that flag and set it to true . But it's not working after converting  to apk using flutter build apk

I believe you are referring this
android:requestLegacyExternalStorage="true
But I already tried it still not working when converting it into apk

Is there any other permission I needed to add in menifest

I just created a new project and copy paste all the code but It worked. I don't know how . But I am closing this issue for now


Picoweb request doesn't return all form data

The function read_form_data() calls read() which can result in only part of the expected data being returned.  Seems like it should be calling readexactly() instead.  The problem is described in the micropython forum in detail:
https://forum.micropython.org/viewtopic.php?f=16&t=6647

Good catch. Fixed/new version released.

I'm still experiencing this issue.
        function setwifi() {
            obj = document.getElementById("networks")
            var ssid = obj.options[obj.selectedIndex].value;
            password = document.getElementById("password").value;
            $.ajax({
                url: '/configure',
                data: {'password' : password, 'ssid' : ssid},
                method: 'POST',
                success: function(data) {
                    window.location = data
                }
            });
        }

Only password will be in the form. Doing alert in the js will give me the correct form.
Python code:
@app.route("/configure")
async def wifi_config(req, resp):
    await picoweb.start_response(resp)
    await req.read_form_data()
    print(req.form)


Support specifying branch in /packit test $repo $branch

The syntax is to be decided.
@fernflower requested to support specifying branches in our /packit test $repo#$pr trigger. This RFE should extend the existing funcionality where you can specify PR, with this addition, branch can be specified as well.

Thank you!

I am including two more comments from Ina from our recent discussion on slack (now that I re-read it again) and would like to 100% confirm this request is correct

Now we have '/rerun' for leapp PRs, that will build leapp-pr-version and fetch leapp-repositorymaster; while '/rerun 4242' will build leapp-pr-version, build leapp-repository4242*, and pass both as test artifacts to testing farm.

So it's not about enabling leapp package installation by specifying additional repos (which is what additional_repos is about iiuc). It is about "how do I instruct packit to run tests with both leapp-from-pr-build and leapp-repository-master-or-the-version-user-specified"

Do I understand correctly that you have 3 test scenarios?

Mode
leapp
leapp-repo

default
leapp from PR
latest released leapp-repo

on demand
leapp from PR
leapp-repo build from master

on demand
leapp from PR
specific build of leapp-repo

default is run when a PR in leapp is created
on demand are triggered with a comment in a PR


Build changes

@danielmarbach Did the appveyor have flaws? Just interesting.

I wasn't unhappy with appveyor. It was just sometimes or most of the time really slow. But I'm interested in Github actions to see how it speeds up to build and deploy as well as the integration with the new package repository of github. I'm thinking about leveraging it for pre-release packages. Furthermore the matrix build capability of github action looks promising for getting the build to run on multiple platforms


Stuck @ 'Rounding them up!' 😢

New PC, well specced. Restarted, reinstalled.
Win10 Pro, existing project, no custom snippets.

hi @idusortus
In VS Code, could you go to the menu and select Help> About and tell me the VS Code version info.
Can you run the command with the Debug Console open to see if there is an error message (like below):

Can you tell me where you have installed VS Code on Windows? Are you admin? Do you run it as a window program (no ssh/remote/wsl)? Is there anything unusual/particular about your setup?
I ran the extension on Windows 10 machine recently and it was fine, but I will take another look to see if there is any issue again.

Hey, @robole  !
Uninstalled ranger, closed VSC, reinstalled ranger, ran round'em up, been running about 6 minutes now.
Ranger version is 0.20.1
"C:\Program Files\Microsoft VS Code\Code.exe"
I'm logged in as admin, running VSC as admin.
All local. My day-to-day dev setup.
winver: 20H2 / build 19042.928
VSC Ver: 1.56.2
Thanks!

Hi @idusortus
Can you open VS Code from the CLI with all extension disabled?
code --disable-extensions

Run the command again to see.  That will rule out other extensions.
Do you have any user snippets?
I have no issues on my linux machine. I will check on a windows machine soon to see it is not a Windows-specific.

Thanks again. I'll take a look if you want a hand. Otherwise I'm moving on.
Or... I'll let you know if I give it a try in Linux.

hi @idusortus
I tried the extension on a machine with windows 10 pro (recent install and same version as you), I did a fresh installation of VS Code v1.56.2 installed to the same directory as you ("C:\Program Files\Microsoft VS Code\Code.exe") and had no issue.

There has to be something unusual with your environment. There is no issue with the extension.
I am closing the issue. If you can identify what it is, feel free to open a new issue.
All the best

hi @idusortus
The Snippets Ranger  extension has been updated recently. I believe your issue could have been when no workspace was open and you ran the command Snippets Ranger: Show me that dur Range, Partner. It should be good now. You can try out v1.6 if you are still interested!
All the best


Command-line task reports false success if STDIO left open

At https://dev.azure.com/polymorpheus/BetterRandom/_build/results?buildId=1080&view=logs&s=62add90f-9a1f-5845-6bfd-b54447f526dc&j=f43d4008-7a89-5683-a713-8042507c2c7e, the shell script exited with status 1 (otherwise there would have been log output from more Dieharder tests; see the source at https://github.com/Pr0methean/BetterRandom/blob/dce64e116c1287eb243cb6e4d6a9431297973618/dieharder.sh#L102), but the job was reported as successful. The following status message was triggered:
The STDIO streams did not close within 10 seconds of the exit event from process '/bin/bash'. This may indicate a child process inherited the STDIO streams and has not yet exited.
The bug seems to be that when such children are left hanging, the exit status is suppressed.

@Pr0methean - I believe the issue is that you are | the output to tee. When you do this, the exit status is the exit status of 'tee', not of the command you are piping to.
$ false | tee foo.txt
$ echo $?
0
$ false
$ echo $?
1

No, that's not the problem: dieharder exits with status 0 even when the tests fail. It's lines 99-103 that convert the output to an exit status.

@Pr0methean - ok, I see, you are right, I jumped too quickly at the 'tee'. Looking through the code, I can see where we print out the
The STDIO streams did not close within 10 seconds of the exit event from process '/bin/bash'. This may indicate a child process inherited the STDIO streams and has not yet exited.

message, but, to me, it looks like we continue to process the exit code.
Can you re-run the pipeline and and check the 'Enable system diagnostics' so we can maybe get a better idea of what is going on?

Started a diagnostic run (https://dev.azure.com/polymorpheus/BetterRandom/_build/results?buildId=1113&view=results); logs should be available in a few hours.

Here's the log: https://dev.azure.com/polymorpheus/BetterRandom/_build/results?buildId=1115&view=logs&s=62add90f-9a1f-5845-6bfd-b54447f526dc&j=f43d4008-7a89-5683-a713-8042507c2c7e

I think I see your issue, you have:
) 2>&1 < prng_out | tee ../dieharder.txt report_out) &
if [ "$(grep -q -m 1 'FAILED' report_out)" ]; then
  pkill dieharder
  pkill java
  exit 1
fi

I think what you really want is:
) 2>&1 < prng_out | tee ../dieharder.txt report_out) &&
if [ "$(grep -q -m 1 'FAILED' report_out)" ]; then
  pkill dieharder
  pkill java
  exit 1
fi

A single & runs the command in the background, your grep is going to run and not find anything since report_out doesn't have any content yet

I think I see your issue, you have:
) 2>&1 < prng_out | tee ../dieharder.txt report_out) &
if [ "$(grep -q -m 1 'FAILED' report_out)" ]; then
  pkill dieharder
  pkill java
  exit 1
fi

I think what you really want is:
) 2>&1 < prng_out | tee ../dieharder.txt report_out) &&
if [ "$(grep -q -m 1 'FAILED' report_out)" ]; then
  pkill dieharder
  pkill java
  exit 1
fi

A single & runs the command in the background, your grep is going to run and not find anything since report_out doesn't have any content yet

report_out is a named pipe, not a normal file. The intent is to run dieharder in the background (since otherwise we couldn't reuse prng_out -- also a named pipe -- or its contents from one Dieharder run to the next), but monitor its output in the main shell so that it can be failed fast.

Ok, having looked through a lot of the agent and task code, I am pretty confident that issue is in dieharder.sh script and not in the agent. The agent is getting an exit code of 0.
Is it possible that for this instance, there is a race condition and that the grep is executed before data is written to report_out and it thinks it's done?
Can you not just pipe the output directly to grep?
) 2>&1 < prng_out | tee ../dieharder.txt | $(grep -q -m 1 'FAILED' report_out)) ||\
(
  pkill dieharder
  pkill java
  exit 1
)

@jtpetty I'll be testing that solution overnight. Thanks!

@Pr0methean - Looking at the issue you filed, it seems that the problem you were running into is with the ubuntu image the agent is running on and not with the agent code itself. Is that correct? And if so, can this issue be closed?

Sounds right; https://github.com/microsoft/azure-pipelines-image-generation/issues/1386 is the same issue.

Lately we have the same problem with a .NET Core Task. I reported this also in azure-pipelines-image-generation/1386 because I'm not sure where this fits.
Starting: dotnet tool restore
==============================================================================
Task         : .NET Core
Description  : Build, test, package, or publish a dotnet application, or run a custom dotnet command
Version      : 2.164.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/build/dotnet-core-cli
==============================================================================
/opt/hostedtoolcache/dotnet/dotnet tool restore --ignore-failed-sources --no-cache --verbosity normal

... restoring packages

Restore was successful.
The STDIO streams did not close within 10 seconds of the exit event from process '/opt/hostedtoolcache/dotnet/dotnet'. This may indicate a child process inherited the STDIO streams and has not yet exited.

and the task will remain blocked until build timeout expires.

Same issue here. After running the tests my agent with 2.165.0 say:

Test Run Successful.
Total tests: 46
Passed: 46
Total time: 8.4851 Seconds
[debug]Exit code 0 received from tool '/usr/bin/dotnet'
The STDIO streams did not close within 10 seconds of the exit event from process '/usr/bin/dotnet'. This may indicate a child process inherited the STDIO streams and has not yet exited.
[debug]The STDIO streams did not close within 10 seconds of the exit event from process '/usr/bin/dotnet'. This may indicate a child process inherited the STDIO streams and has not yet exited.

while the old agent with 2.158.0 seems to be good

Test Run Successful.
Total tests: 46
Passed: 46
Total time: 7.7349 Seconds
##[debug]Exit code 0 received from tool '/usr/bin/dotnet'
##[debug]STDIO streams have closed for tool '/usr/bin/dotnet'

I am not even opening any STDIOs in the tests (at least I am not aware of). Starting the task with debug doesn't reveal any more information. This issues is NOT fixed

Reopen please?

I am facing the same issue. i don't want to kill java as other process are running on it.
Is there any other solution for it

This is still happening, we see it frequently with the DotNetCoreCLI@2 yaml task. It writes "The STDIO streams did not close within 10 seconds of the eit event from process 'C:\Program Files\dotnet\dotnet.ee'. This may indicate a child process inherited the STDIO streams and has not yet e***ited." to the log and then does nothing until the job times out (after 60 minutes).
Please fix this.


Encoding Difference OpenAsync and new HtmlParser?

AngleSharp Version v0.14.0
The following two snippets produce different results.
The first uses an HtmlParser, the second uses browsingContext.OpenAsync(...).
The same html, configuration, and browsing context is used in both.
Why?  Which result is expected?
// first snippet
string html = @"<meta http-equiv=""Content-Type"" content=""text/html; charset=Windows-1252"">ü";
IConfiguration configuration = Configuration.Default;
IBrowsingContext browsingContext = BrowsingContext.New(configuration);
HtmlParser parser = new HtmlParser(new HtmlParserOptions(), browsingContext);
IHtmlDocument document1 = parser.ParseDocument(html);
foreach (INode node in document1.Body.ChildNodes)
{
	Console.WriteLine(node.TextContent);
}
// ü

// second snippet
string html = @"<meta http-equiv=""Content-Type"" content=""text/html; charset=Windows-1252"">ü";
IConfiguration configuration = Configuration.Default;
IBrowsingContext browsingContext = BrowsingContext.New(configuration);
Task<IDocument> task = Task.Run<IDocument>(async () =>
{
	return await browsingContext.OpenAsync(response => response.Content(html));
});
IDocument document = task.Result;
foreach (INode node in document.Body.ChildNodes)
{
	Console.WriteLine(node.TextContent);
}
// A¼

In the second one you'll fake an HTTP response. As such you see the result of transporting exactly these bytes over the wire, with a switch in encoding (you provided UTF16 - this is the char format of .NET - and told the browser to use Windows-1252). So this is expected.
In the first case you provide already the pre-evaluated result and no stream is faked for you. Thus an encoding change has no impact.

Thank you


expressWinston.errorLogger: No logging when using level

When using a function for the level option, nothing will be logged by the error logger.
Please find a simple example attached:
  app.use(expressWinston.errorLogger({
    winstonInstance: logger,
    msg: '{{err.message}}',
    level: function() {
      return 'warn';
    }
  }));

I just tested this, and it seems to be working fine. Is it possible that you have your transport(s) on your winstonInstance set not to log at the "warn" level?
Here's the code I used (with some transport options laid out as well):
app.use(expressWinston.errorLogger({
  transports: [
      new winston.transports.Console({}),
      // new winston.transports.Console({level: 'warn'}), // also works
      // new winston.transports.Console({level: 'error'}), // DOES NOT work (reproduces bug)
  ],
  msg: '{{err.message}}',
  level: function() {
    return 'warn';
  }
}));

Thanks for your quick response. You're right – sorry for bothering you with this issue.

No problem!


how to only register www.domain.com and not domain.com

On the server where I have Plesk, I am only keeping the "www.domain.com", while the "domain.com" is on another server.
However I specified the domain in Plesk as "domain.com"
When I attempt to install the SSL certificate with the Let's Encrypt plugin, I select the
"include www.domain.com as an alternative domain name."
but after pressing the install button I get these errors attempting to also register "domain.com" which points to another server:

Error: Let's Encrypt SSL certificate installation failed: Failed letsencrypt execution: Saving debug log to /usr/local/psa/var/modules/letsencrypt/logs/letsencrypt.log
Obtaining a new certificate
Performing the following challenges:
http-01 challenge for domain.com
http-01 challenge for www.domain.com
Waiting for verification...
Cleaning up challenges
Failed authorization procedure. domain.com (http-01): urn:acme:error:unauthorized :: The client lacks sufficient authorization :: The key authorization file from the server did not match this challenge [_LxZP3Xc_tMIczdnlNOsk4n3mB0HiOfNeWJoEXRWAyQ.oEUmCGvNba4HNfEHiZpaI5EOvdBrRctXTBkAZCG32Qw] != [URL not found]
IMPORTANT NOTES:

The following errors were reported by the server:

Domain: domain.com
Type: unauthorized
Detail: The key authorization file from the server did not match
this challenge
[_LxZP3Xc_tMIczdnlNOsk4n3mB0HiOfNeWJoEXRWAyQ.oEUmCGvNba4HNfEHiZpaI5EOvdBrRctXTBkAZCG32Qw]
!= [URL not found]
To fix these errors, please make sure that your domain name was
entered correctly and the DNS A record(s) for that domain
contain(s) the right IP address.

Which is the smoothest way to only install the SSL certificate for www.domain.com and not for domain.com ?

Add www.domain.com as a separate subscription ?

Use this command in CLI:
plesk bin extension --exec letsencrypt cli.php -d www.domain.tld --email<EMAIL_ADDRESS>


Feature request: Documentation for the LSP protocol extensions

Hello!
Thanks for a great extension, it make s editing COBOL and REXX code a lot more pleasant and it's a welcoming addition in the Mainframe development space!
Description of the enhancement requested
One of the great benefits with Language server protocol (LSP) is that it's editor agnostic, one language server can be used with multiple different editors as long as they implement a client with speaks the LSP protocol.
My preferred editor of choice is Emacs and I would love to be able to utilize this LSP server with Emacs for editing Cobol and Rexx, and with my initial try it basically works out of the box (with the built in LSP client Eglot).
What doesn't work is of course your extensions to the LSP protocol.
So my feature request is:

Documentation for the extensions to the LSP protocol, such as document/request, what paramenters and format are being requested and what does the LSP server expect to get back.

Best Regards
Kristofer

Hi @Trisk3lion,
I'm just a user but I'm going to advocate for IBM on this request.
IBM Z Open Editor is not an open source product.
IBM Z Open Editor is made available to users free of charge, without support (although the IBM team does a great job here taking into account the various requests).
IBM Z Open Editor may be supported by IBM but on condition that you subscribe to a licensed product, such as IBM Z Cloud Modernization Stack.
Basically IBM Z Open Editor is intended for companies that use an IBM mainframe, and I assume that IBM finds its way financially by facilitating the use of its mainframes by providing solutions such as IBM Z Open Editor. It is a balance between paid solutions and free solutions which all contribute to the same end, to promote the mainframe.
IBM Z Open Editor also relies on other software elements purely intended for the mainframe such as Zowe Explorer which manages communication between the local workstation and the mainframe.
As you have seen, the Language Server is not enough to make a solution. It is also necessary to exploit its analysis results to benefit from them at the level of the code editor, such as, for example, offering the opening of a copybook or include by simple click, or displaying contextual documentation.
This is the added value provided by IBM in Z Open Editor. And IBM Z Open Editor is not open source.
Regarding the Language Server protocol, there is a document on the official Microsoft site: Language Server Protocol

Hi @FALLAI-Denis
I'm not sure I understand your point, or what you are advocating.
I'm well aware of that IBM Z Open Editor is not open source, is made available free of charge without support. That has nothing to do with my feature request.
Zowe Explorer brings functionality that IBM Z Open Editor uses for fetching files from the Mainframe, and it's a feature of the client side of IBM Z Open Editor, so it also has nothing to do with my feature request.
I don't want to "compete" with IBM Z Open Editor, I want to use it! The only difference is that with (in my specific case) another client/front-end that happens to be Emacs instead of Visual Studio Code. But it could be any client that implements the LSP protocol.
Now this it already works perfectly fine because IBM Z Open Editor is partly and LSP server and implements the LSP protocol.
The only thing I'm asking is if they would like to document their specific extensions to the protocol. For example where IBM Z Open Editor can find a specific copybook. That is an extension to the  Language Server protocol and as such is not documented in the link you posted.
The result is in my opinion only beneficiary to IBM, Mainframe community and anyone developing for the Mainframe.
And in the end, it just a feature request.

Thanks for the proposal and discussion on this topic. We definitely had requests to support other editors before, but enabling users to integrate themselves is a new one. We have to discuss this in our teams and see what kind of support we could and want to provide.
Note, that currently our License Terms at the moment do not allow to disassemble our files, reverse engineer them, or use components such as the language server with other programs. We would have to change our license terms as well if we wanted to allow and support this.

Hello @phaumer , and thanks for your answer.
I understand this is a difficult topic which requires consideration. You do of course care much about your product and the quality it provides.
But I do  hope you consider the possibility to support other editors. I strongly  believe that it would only benefit yourself and the broader Mainframe community in this modernization we are in the middle of, and after all that was what the LSP protocol was made for. :)
From using Z Open Editor in VS Code with COBOL and REXX it's clear that the parsing and code completion is superb and I think it would be sad to keep that only to VS Code.
I will respect the license terms and stop my discoveries into using it with Emacs. But to my defence , I have not disassemble files, or reverse engineer anything. I have just used what information can be gathered from the "DEBUG" output that is built into the extension.
Best Regards
Kristofer


Update to latest node modules

Signed-off-by: Eduard Itrich<EMAIL_ADDRESS>

@maxwolfs I'm aware of your standpoint regarding pushing the nodule modules directly into this repository, but for now I wanted to satisfy Dependabot until we tackle #418 😇


Correctly set batched plot options

Correctly respect the plot options set on the batched elements. Previously inherited all plot options from the NdOverlay, which a) could cause unsupported parameters being set and b) ignored plot options set on the Element. This appropriately combines the plot options from set on the NdOverlay and Element plot.

Is this ready for review?

It certainly sounds like an improvement, and I'd be in favor of merging.

Yes, ready to review/merge.

I notice this is implemented on GenericElementPlot but the tests have passed, the changes look reasonable so I think it is all ok. Merging.


Interaction: Fixes setting an image as lock screen background

Fixes #1319

What version is this supposed to be applied to?

This fix will be available in the next release.


Check by default

The Tree Select component is configured to not enable checkboxes by default. As a result, upon loading, all checkboxes appear in an unchecked state.
To accommodate the requirement, a new parameter named checkOnLoad could be introduced, set to False by default.
This ensures that upon loading, all checkboxes within the Tree Select component are automatically marked as checked.

This is definitely something that's missing and a feature that would be great to have!

Hey,
when I am understanding you correct you want that some values are checked by default. So when you load the component some checkboxes are checked, right?
I think this is already currently possible by passing the "value" in the defaultValue Parameter.
Example:
tree_data = [
  {
    "value": "parent 1",
    "title": """Test    parent HTML test""",
      },
      {
        "value": "parent 1-1 - value",
        "title": "parent 1-1",
      },
]

default_value = tree_data[0]["value"]
st_ant_tree(tree_data, multiple=True, treeCheckable=True, defaultValue=default_value)

This is the state of the first render:

Does this help?


the new version 0.4.1 will cause pandas read_sql() stuck

import pandas as pd
hql="select * from test_tb"
pd.read_sql(hql, hive_conn)
the code above will stuck in read_sql step.
and at the version 0.3.0 it will be ok

Indeed since #10, and in particular the changes from _trans.readAll() to _trans.read(), queries can randomly hang.
The issue is that read() makes no attempt to read the full requested data (e.g. TSocket only does a single socket.recv() call), while readAll() does.
As a consequence, if a frame does not fit in a single read() call, the frame is not fully read. Any subsequent calls will then return garbage data, which may hang on the client side when trying to read a binary with a stupidly large size, or just break altogether.
I just created #23 to fix this.


Failed to resolve entry for package "libp2p". The package may have incorrect main/module/exports specified in its package.json

Version: libp2p 0.37.3

Platform: macOS Monterey v12.4, Darwin Kernel Version 21.5.0: Tue Apr 26 21:08:22 PDT 2022; root:xnu-8020.121.3~4/RELEASE_X86_64 x86_64, Brave Version 1.39.120 Chromium: 102.0.5005.99 (Official Build) (x86_64)

Subsystem: libp2p-in-the-browser

Severity: Medium

Description: When attempting to run the libp2p-in-the-browser example, I cloned the js-libp2p repository and followed the steps: ran npm install in the root, then cd into ./examples/libp2p-in-the-browser and ran npm install again. When attempting to run with npm start, an error is thrown:

When doing a general web search, there are many links referring to Vite, I feel that this may be causing the problem. Here is a direct link to a similar error in the ViteJS issues.

Steps to reproduce the error: Clone repository, npm install in root, cd into ./examples/libp2p-in-the-browser, run npm install again, then npm start

Ran new test, appears to be resolved

same error on wsl2 :(
wonder how you fix it
just retry?
thank you


Add ability to provide custom height className (Fixes #20)

It works by adding height property with the className value to the transitionName object.
<ReactCSSTransitionReplace transitionName={{ height: 'cross-fade-height' }}>
  <div key="page01">
    My page 01 content
  </div>
</ReactCSSTransitionReplace>

Nevermind. I'm maintaining my own fork of this module now: https://github.com/le0nik/react-css-transition-replace-plus

@le0nik you've done some great work on this module! Thanks for all that.
I'm sorry that I wasn't able to get round to it sooner. Everything you've done is now incorporated in this main repo and published as v2.0.0 and I've take care to maintain you as the author on your commits.
It would be a pity to see this module diverge into different packages unnecessarily, so if you want to make large scale changes/additions again in the future and aren't set on maintaining your own fork let me know and I'll make you a collaborator on this repo/npm.

I'm glad you're back!
I've added a deprecation notice to README.md and deprecated the package on npm.
Thanks for taking the time to incorporate the changes into this repo.

No problem, and thanks for deprecating the other. Next time I'll be sure to at least leave a reply sooner! Funny how PRs always seem to arise in the most hectic times. :blush:


cxfreeze windows digest not euqal

test_esky.py on py3k branch
tests/eskytester/script2.py
https://github.com/cloudmatrix/esky/blob/py3k/esky/tests/eskytester/script2.py#L83
this call winds up here
patch.py
https://github.com/cloudmatrix/esky/blob/py3k/esky/patch.py#L704
this error gets caught
finder.py
https://github.com/cloudmatrix/esky/blob/py3k/esky/finder.py#l239
i compared the flow to the master branch. the point of deviation is in patch.py when the digest is not equal.
tests are passing on python3 and python2 using linux

error just went away randomly... nice!


Flutter sample's text input loses focus after submission

Repro steps:

Run the Flutter sample.
Type text and press Enter

Expected result: the text input stays focused after the response is shown
Actual result: the text input loses focus after the response is shown

Thanks for the report! cc @johnpryan (who wrote the sample) and @khanhnwin (who's contributed fixes to it)

@loic-sharma I think this is the right way to fix it, please take a look and let me know what you think: https://github.com/google/generative-ai-dart/pull/61


chore: convert to yarn and workspaces

This is the setup I use with Lerna. I tried once to set up a Lerna monorepo using just NPM and it was a literal nightmare. You constantly have to bootstrap and hoist things where yarn just takes care of all of that for you with symlinks in your workspace

Thanks for taking a look and adding the conversion to yarn


Use functions for buffer reading

Which component is your feature related to?
Forge
Feature Request
https://github.com/foundry-rs/starknet-foundry/pull/1270/files#diff-8aa925b526143c705871a1be89c6ff1c8e46bc7dcbc453e0ce8b5a7a785bcb94R625-R642
These functions should be used in the whole file (currently they are used only in spoof)

Regarding our discussion during the meeting, this is a nice thing to do and should be done, though I would not call array of felts buffer, because there is a different structure named buffer used to write to cairo memory.

Please rename the functions adequately as a part of the issue then.

Functions are fine, I mean argument name buffer

Wasn't it done in #1440? cc @abulenok


Strings not freed in the JS wrapper?

I don't see any references to anoncreds_string_free, but this is likely required for all strings coming from the FFI layer. This could be a significant memory leak over time.

Yeah that does not seem to be too good. I think we have three solutions but they all have some disadvantages

Expose string_free publicly and allow the consumer to deal with it.

Downside is that this is quite the uncommon pattern in JS and likely they will still forget some allocations.

Within the wrapper copy the value and immediately call it (it might even already be copied ffi-napi and c++ side).

Would still require all manual frees but at least we control it and not the consumer.

Wait for Typescript 5.2 to be stable as it introduces the using keyword which would finally allow a cleanup function to be ran when it goes out-of-scope.

The best option by far, but I am unsure of the timeline (https://github.com/microsoft/TypeScript/issues/54298).

IMO it is okay to wait for typeScript 5.2 and leave this as-is for now.

3. Wait for Typescript 5.2 to be stable as it introduces the using keyword which would finally allow a cleanup function to be ran when it goes out-of-scope.

The best option by far, but I am unsure of the timeline (TypeScript 5.2 Iteration Plan microsoft/TypeScript#54298).

Well, if that iteration plan is accurate, it will be out in 4 days from now! So definitely it's worth waiting a bit and doing not only this but also for object handle clear() methods.
If not, I'd go for option 2.

I think the wrapper copies these FFI string buffers to Strings anyway, so it should be possible to free the reference before returning. I noticed later that this would apply to anoncreds_buffer_free as well, for binary buffers.


Improve sections management in info file

Sections names are named with an md5 sum of the source path.
Changes to do:

[x] rename sections to src1, src2, ...
[x] use find_infofile_section in backup to find old backup status

Done in commit d225aa50db052cfd44eaff6b3dcdf5a88ef62279


name and symbol

Instead of class id being stored in name and symbol, it's actual name and symbol from source chain should be transferred, when creating new collection.
Unfortunately name can't easily be changed. Reason is here: https://github.com/public-awesome/cw-ics721/blob/main/packages/ics721/src/ibc.rs#L256-L262
Basically ICS721 does a sub msg for instantiating cw721 contract (aka collection) on target chain. Once done, on reply, it needs to store ClassId and collection address in a map. The only way of getting class id is storing it somewhere in cw721 - in this case in name.
So what I can at least do, is setting symbol from source source.
A possible solutions is storing name and class id into name with this syntax: name: name_from_source_collection (classID).
This way I could extract ClassId from name.
TBD: instantiate2

Resolved using instantiate2 in PR 71: https://github.com/public-awesome/cw-ics721/pull/71


Section headings in documentation

Section headings in documentation
I'm seeing the following pattern used throughout these doc pages for section titles:

Important globals

Screen reader users see the text "Important globals" repeated twice (i.e. via the aria-label); sighted users do not see the contents of aria-label. Worse, we get even more noise from the extraneous "Anchor link for: " in the aria-label.
The effect is that each section title looks something like this:

Anchor link for: important globals
Important globals

P{lease remove the aria-label from all of these section headings; it is not needed, and severely degrades readability for all screen reader users!
Thank you.

We use anchor.js for the anchors.
/CC @bryanbraun

@XhmikosR, I've put together a fix upstream at https://github.com/bryanbraun/anchorjs/pull/90. I'll merge it in the next couple of days unless anyone objects.

@bryanbraun; LGTM, thanks!
I'll wait for a new patch release so that I update anchor.js here.

@XhmikosR, version 4.1.0 is published and up on github / npm.

@bryanbraun: thanks! I'm gonna make a PR now.


Statistics of the percentage of descriptions and tags inserted for each part of the databases

Greetings and Regards
Please advise if there is a section through which you can get the statistics of the descriptions and tags entered for each part of the databases. This case has many applications regarding the completion of metadata by different parts of an organization and control It facilitates it.
thanks a lot.

@alitehranim please do not open github tickets as bugs. Please join httsp://slack.open-metadata.org and confirm if its indeed an issue.
look at the docs for your question https://docs.open-metadata.org/v1.2.x/how-to-guides/data-insights


Add support for ACMEv2 POST-as-GET

What this PR does / why we need it:
Add support for ACME POST-as-GET to the ACME client.
Thanks to @bluecmd for this patch.
Which issue this PR fixes: fixes #1341
Special notes for your reviewer:
I'm holding this patch until v0.9, as this probably shouldn't be included this late in v0.8.
Release note:
Add support for ACMEv2 POST-as-GET

/hold
/milestone v0.9

/lgtm cancel

Sorry about this 🙈
/lgtm cancel

/test all

I have a small question here. I am trying to generate certs for my kubernetes endpoints. I used the tag "8fa533c3" for cert-manager, cert-manager-webhook, cert-manager-cainjector and it works fine.
However if I change the tag to say v0.8.1 or any of the latest ones, it says
"Failed to create order: acme: urn:ietf:params:acme:error:malformed: The request message was malformed"
Is it because, those tags are missing something from "8fa533c3"


Add link to blog post's source file

The blog pages should have a link to their source code in the right column of each page.

The text should read Edit on Github
the blogs are stored in this folder:
https://github.com/build-trust/ockam-website/tree/develop/src/content/blog
The expected behavior is that if a reader finds a typo, or wants to suggest a change with a PR, they will clink the link to the source code and land in the .mdx file. Example:
https://github.com/build-trust/ockam-website/blob/develop/src/content/blog/join_the_team.mdx

PR: https://github.com/build-trust/ockam-website/pull/634


Enter/Return Ignored

I can't use my Enter/Return key in Atom.
OSX Yosemite
Version 10.10.1
MacBook Pro (13-inch, Mid 2012)
Processor 2.9 GHz Intel Core i7
Memory 8GB

try to disable emmet if you have it installed

Yeah, this has come up a lot in relation to the emmet package. Do you have that installed? If not, can you report your installed packages (apm list) and whether or not this occurs in safe mode?

See https://github.com/emmetio/emmet-atom/issues/186

Confirm this is an issue with Emmet


Remove version: Alibaba.aDrive version 4.12.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

For moderators or/and anyone else

[!Caution]
As a result of several personal problems I've had over the last few months + my final exams coming up very soon, I'm currently unable to look at my GitHub notifications every day and maintain my executions correctly.
If you detect a certain problem on a certain pul request:

Either you can close the pull request directly without mentioning me.
Or you can send me a message on X (Twitter): @Exorcism0666

Thank you for your understanding, and I'm sorry if some pull requests can't be approved due to the situation I'm in.

This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

URL: https://cdn.aliyundrive.net/downloads/apps/desktop/update/4.12.0/win32/ia32/aDrive-4.12.0.exe
Response Error: The remote server returned an error: (403) Forbidden.
(Automated message - build 912)


stdout buffering

Hi! Super cool project!
I was looking at your io related code an noticed you are using std::io::stdout, which has issues: https://github.com/rust-lang/rust/issues/60673
I've worked around this in other libraries of my own by using burtsushi's https://docs.rs/grep-cli/0.1.5/grep_cli/, which provides a correctly buffering stdout method when piping to a file, and line buffering when connected to a TTY.
I've seen significant speedups in my own libraries switching to buffered stdout. This may help close the single threaded gap with tsv utils.

Hey, thanks for this pointer! It does indeed look like I should be using this. To set expectations on this improving benchmarks, though, all writes are batched in a local thread and then sent to a background thread to be issued. Beyond that, the background thread uses write_all_vectored, which lets you write a whole batch without necessarily flushing after each line if I'm reading the LineBuffered implementation correctly.
Still, like I said I still think frawk should be using grep_cli; so I'll keep this open until I migrate.


[polish] capitalize the default answer in direnv integration question

Summary
It is common convention to capitalize the default answer in linux CLI prompts.
How was it tested?
the default behavior appears to be No

Current dependencies on/for this PR:

main

PR #545   👈

This comment was auto-generated by Graphite.


Update cds-utils.md

decodeURI and decodeURIComponent were missing the parameter in their heading, which other methods with parameters were more explicit about. Also, I think it would be helpful to link to the original decodeURI and decodeURIComponent documentation. Do we have goto-documentation for native functions? MDN maybe?
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/decodeURI and
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/decodeURIComponent respectively

MDN maybe?

Yes, MDN is by far the best resource for that imo.


Class-Level Event seem to detach between 2 unit test

What steps will reproduce the problem?
i will explain my event structure :
in a bootstrap.php file :
Event::on(
  'yii\db\ActiveRecord',
  ActiveRecord::EVENT_AFTER_VALIDATE,
  ['app\events\ActiveRecordEvents', 'logError']
);

Event::on(
  'yii\db\ActiveRecord',
  ActiveRecord::EVENT_AFTER_UPDATE,
  ['app\events\ActiveRecordEvents', 'logError']
);

Event::on(
  'yii\db\ActiveRecord',
  ActiveRecord::EVENT_AFTER_INSERT,
  ['app\events\ActiveRecordEvents', 'logError']
);
Event::on(
  'yii\db\ActiveRecord',
  ActiveRecord::EVENT_AFTER_DELETE,
  ['app\events\ActiveRecordEvents', 'logError']
);

The event file :
<?php
namespace app\events;

use Yii;
use common\models\User;
use app\models\Traitement;
/**
*/
class ActiveRecordEvents
{

  public static function logError($event)
  {

    $model = $event->sender;

    if (count($model->getErrors()) != 0) {

      yii::$app->params['save_error_attributes'] = self::formateAttributes(
        $model->getAttributes()
      );
      yii::$app->params['save_error_error'] = self::formateErreurs(
        $model->getErrors()
      );
      $appelant = '';
      if (isset(debug_backtrace()[7]['file'])) {
        $appelant .= 'file : ' . debug_backtrace()[7]['file'] . "\n";
      }
      if (isset(debug_backtrace()[7]['line'])) {
        $appelant .= 'line : ' . debug_backtrace()[7]['line'] . "\n";
      }
      if (isset(debug_backtrace()[7]['function'])) {
        $appelant .= 'function : ' . debug_backtrace()[7]['function'] . "\n";
      }
        //SKIPPED code : store in database the message
      return false;
    }

   return true;
  }

i have a file with multiple unit test.
If i test a unit file :
Unit Tests (6) ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Modules: Asserts, Yii2, Db
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
- EnsemblePatCreationTest: Creation avec deux categorie ged  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  Fixture traitement: cleanTraitementDirectory
  Fixture traitement: cleanTraitementDirectory
  [app\models\mysql\PdfEnsemblePatrimonial] {}
  [app\models\mysql\PdfEnsemblePatrimonial] {}
✔ EnsemblePatCreationTest: Creation avec deux categorie ged (0.16s)
- EnsemblePatCreationTest: Testcreation programme  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  Fixture traitement: cleanTraitementDirectory
  [app\models\mysql\PdfEnsemblePatrimonial] {}
✔ EnsemblePatCreationTest: Testcreation programme (0.09s)
- EnsemblePatCreationTest: Testcreation batiment  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  Fixture traitement: cleanTraitementDirectory
  [app\models\mysql\PdfEnsemblePatrimonial] {}
✔ EnsemblePatCreationTest: Testcreation batiment (0.10s)
- EnsemblePatCreationTest: Testcreation escalier  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  Fixture traitement: cleanTraitementDirectory
  [app\models\mysql\PdfEnsemblePatrimonial] {}
✔ EnsemblePatCreationTest: Testcreation escalier (0.10s)
- EnsemblePatCreationTest: Testcreation logement  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  Fixture traitement: cleanTraitementDirectory
  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  [app\models\mysql\PdfEnsemblePatrimonial] {}
✔ EnsemblePatCreationTest: Testcreation logement (0.09s)
- EnsemblePatCreationTest: Testcreation sans categorie ged  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  Fixture traitement: cleanTraitementDirectory
  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  [yii\db\ActiveRecord::insert] 'Model not inserted due to validation error.'
✖ EnsemblePatCreationTest: Testcreation sans categorie ged (0.15s)
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Time: 1.01 seconds, Memory: 16.00MB

There was 1 failure:

---------
1) EnsemblePatCreationTest: Testcreation sans categorie ged
 Test  tests/unit/components/ensemblePatrimonial/EnsemblePatCreationTest.php:testcreationSansCategorieGed
Pdf Categorie Ged ID est invalide.
Failed asserting that '0' matches expected 1.
#1  Codeception\Module\Asserts->assertEquals
#2  /data/htdocs/tests/_support/_generated/UnitTesterActions.php:42
#3  /data/htdocs/tests/unit/components/ensemblePatrimonial/EnsemblePatCreationTest.php:206
#4  tests\unit\components\ensemblePatrimonial\EnsemblePatCreationTest->testcreationSansCategorieGed

FAILURES!
Tests: 6, Assertions: 19, Failures: 1.

But if i launch only the relevant test :
Codeception PHP Testing Framework v2.4.0
Powered by PHPUnit 6.5.7 by Sebastian Bergmann and contributors.

  [Db] Connected to yii2_basic_tests
  [Db] Connected to yii2_basic_tests
  [Db] No SQL loaded, loading dump skipped

Unit Tests (1) ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Modules: Asserts, Yii2, Db
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
- EnsemblePatCreationTest: Testcreation sans categorie ged  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  Fixture traitement: cleanTraitementDirectory
  Fixture traitement: cleanTraitementDirectory
  [yii\db\Connection::open] 'Opening DB connection: mysql:host=<IP_ADDRESS>;dbname=yii2_basic_tests'
  [yii\db\ActiveRecord::insert] 'Model not inserted due to validation error.'
✔ EnsemblePatCreationTest: Testcreation sans categorie ged (0.11s)
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Time: 491 ms, Memory: 12.00MB

OK (1 test, 4 assertions)

When i launch multiple test it seem to loose the event attached beetween test (if i understand it correcly, the app is not "reset" during each test. My event are attached at bootstrap level but event are detached somewhere)
What is the expected result?
no test failure
What do you get instead?
The test search for a datase entry but the event are not fired => test failule
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.0.27

Operating system
debian 9.3

It seems to me that this is due to the update of codeception\base to version 2.4.0
When I upgraded to the latest version of this extension, I also broke tests.
I decided to install in composer.json
"codeception/base": "~2.3.9"

Tests earned as it should.

Thanks for the info, i will confirm it monday

Still go the error after update.
composer update log :
 - Removing codeception/phpunit-wrapper (6.0.5)
  - Updating behat/gherkin (v4.5.1 => v4.4.5): Downloading (100%)
  - Updating symfony/dom-crawler (v3.4.4 => v3.4.6): Downloading (100%)
  - Updating symfony/css-selector (v3.4.4 => v3.4.6): Downloading (100%)
  - Updating symfony/browser-kit (v3.4.4 => v3.4.6): Downloading (100%)
  - Updating symfony/yaml (v3.4.4 => v3.4.6): Downloading (100%)
  - Updating symfony/event-dispatcher (v3.4.4 => v3.4.6): Downloading (100%)
  - Updating symfony/debug (v3.4.4 => v3.4.6): Downloading (100%)
  - Updating symfony/console (v3.4.4 => v3.4.6): Downloading (100%)
  - Updating symfony/finder (v3.4.4 => v3.4.6): Downloading (100%)
  - Updating codeception/base (2.4.0 => 2.3.9): Downloading (100%)

I do not know if it helps you or not, but here are the setup logs when there is a problem with codeception / base 2.4.0
https://travis-ci.org/Dominus77/yii2-advanced-start/builds/346727948
It was worth changing the version to 2.3.9 the tests passed on ok!
https://travis-ci.org/Dominus77/yii2-advanced-start/jobs/349791418

https://github.com/Codeception/Codeception/pull/4621
https://github.com/Codeception/Codeception/pull/4700
I think these discussions are useful for you.

Does not look like something we can fix in yii. Closing the issue.


[CARBONDATA-1707][Streaming] Log the taken time of each stream batch and fix StreamExample issue

[x] Any interfaces changed?
no

[x] Any backward compatibility impacted?
no

[x] Document update required?
no

[x] Testing done
no require
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1111/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1710/

LGTM


Add new names to an existing ORCID record

I'd like to be able to add new names (given and/or family) to an existing ORCID record. I would assume that the most logical way to do this would be to rewrite doi_common.update_existing_orcid(), though I don't super care how the functionality is made available to me. Just that it's available!

New function:
add_orcid_name(lookup_by='employeeId', lookup=None, family=None, given=None, coll=None, write=True)
lookup_by is either "orcid" or "employeeId", lookup is the lookup value. Specify family and/or given, and pass in the collection in coll. Returns the new record.


address review lint issues

address lint errors ( https://ci.woodpecker-ci.org/repos/3780/pipeline/22530/31 )

While its nice that we skip some steps/workflows depending on the changed files, we somehow miss some breaking changes that should have been covered by the introduced PR CI run already.

Ah didnt know, thanks. Is there a reason to use latest instead of pinning + automerge?


Update Dockerfile

Fixed 'AS' casing to remove warning from Docker.

Docker warning in case you're interested :)


Explicitly account for current limitations of RISC-V kernels

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: kexectools, libseccomp
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: kexectools, libseccomp
Partial log (click to expand)
these paths will be fetched (0.00 MiB download, 0.01 MiB unpacked):
  /nix/store/kxfc1kcacriisw2mhb4yq6ki6p7fsmjm-libseccomp-2.3.3
copying path '/nix/store/kxfc1kcacriisw2mhb4yq6ki6p7fsmjm-libseccomp-2.3.3' from 'https://cache.nixos.org'...
/nix/store/fm1ql96dshkg4hkkzzm2vnnd1d9v3qa0-kexec-tools-2.0.16
/nix/store/kxfc1kcacriisw2mhb4yq6ki6p7fsmjm-libseccomp-2.3.3

Success on x86_64-linux (full log)
Attempted: kexectools, libseccomp
Partial log (click to expand)
/nix/store/rg5zjwjl5rnnar3rb6hy649454wr2imd-kexec-tools-2.0.16
/nix/store/ydimk83fkc07sr575qkw8sqxhppsaryp-libseccomp-2.3.3

Hmm so hehe there's two things I mentioned on IRC, and I now see they were in direct conflict:

It's better not to push local package problems into global lib/systems.nix
It's better to allow negative reasoning in arbitrary packages

Maybe it would be a good idea to do a meta.__badPlatforms so meta.platforms stays "good" and "open world friendly" but we still keep local concerns local?

@Ericson2314 Hmm I'm a bit confused, this PR both moves stuff out of lib/systems and allows negative reasoning in individual packages... meta.platforms.{include,exclude} seems isomorphic to meta.{platforms,__badPlatforms}, especially since include is the default if none are specified.

@shlevy hahaha I meant to write "disallow" not "allow". Sorry for that. Indeed it is isomorphic, just trying to quarantine the negative stuff as much as possible.

@Ericson2314 Updated

Success on aarch64-linux (full log)
Attempted: kexectools, libseccomp
Partial log (click to expand)
these paths will be fetched (0.00 MiB download, 0.01 MiB unpacked):
  /nix/store/kxfc1kcacriisw2mhb4yq6ki6p7fsmjm-libseccomp-2.3.3
copying path '/nix/store/kxfc1kcacriisw2mhb4yq6ki6p7fsmjm-libseccomp-2.3.3' from 'https://cache.nixos.org'...
/nix/store/fm1ql96dshkg4hkkzzm2vnnd1d9v3qa0-kexec-tools-2.0.16
/nix/store/kxfc1kcacriisw2mhb4yq6ki6p7fsmjm-libseccomp-2.3.3

Success on x86_64-linux (full log)
Attempted: kexectools, libseccomp
Partial log (click to expand)
/nix/store/rg5zjwjl5rnnar3rb6hy649454wr2imd-kexec-tools-2.0.16
/nix/store/ydimk83fkc07sr575qkw8sqxhppsaryp-libseccomp-2.3.3

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: kexectools, libseccomp
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.


user handling with HTML template

hello, how do I pass a ValidationError or 400 error back to the login.html template?
for example if user enters empty username or incorrect password, they just get a text return,
1 validation error for Request
body -> password
  field required (type=value_error.missing)
Validation Error:

How can I pass this error back to login.html template as a warning?
Thank you

Hi @perfecto25!
FastAPI is intended to be used as a REST API, not a traditional web framework like Django that generates HTML response.
The correct way to do this is to write a bit of JavaScript in your page to handle the form submission, handle the response and make the redirection.

@frankie567, so there is no native way to check in Jinja2, for example, that the user is logged in?
{% if request.user.is_authenticated %}
...
{% endif %}

@frankie567, so there is no native way to check in Jinja2, for example, that the user is logged in?
{% if request.user.is_authenticated %}
...
{% endif %}

You can use the current_user dependency with the optional flag  and pass the result in your template context. Then, you can check if it's None or not.

 Thank you, @frankie567!


watch multiple locations in multi-phoenix umbrella application

Not exactly an issue, so apologize for using this as a forum for a question - but I came to the issues section of this repo to find an answer, so perhaps other will as well.
Is it possible to configure :esbuild such that it will watch assets in multiple phoenix umbrella apps? Right now I'm defining multiple profiles such like:
config :esbuild,
  version: "0.12.18",
  app_one: [
    args:
      ~w(js/app.js --bundle --target=es2016 --outdir=../priv/static/assets --external:/fonts/* --external:/images/*),
    cd: Path.expand("../apps/app_one_web/assets", __DIR__),
    env: %{"NODE_PATH" => Path.expand("../deps", __DIR__)}
  ],
  app_two: [
    args:
      ~w(js/app.js --bundle --target=es2016 --outdir=../priv/static/assets --external:/fonts/* --external:/images/*),
    cd: Path.expand("../apps/app_two_web/assets", __DIR__),
    env: %{"NODE_PATH" => Path.expand("../deps", __DIR__)}
  ]

Then I am adding to my Endpoint configurations like:
config :app_one_web, AppOneWeb.Endpoint,
  # ...
  watchers: [
    esbuild: {Esbuild, :install_and_run, [:app_one, ~w(--sourcemap=inline --watch)]}
  ]

config :app_two_web, AppTwoWeb.Endpoint,
  # ...
  watchers: [
    esbuild: {Esbuild, :install_and_run, [:app_two, ~w(--sourcemap=inline --watch)]}
  ]

Running mix phx.server doesn't seem to start these watchers.
It's also worth noting that the Endpoint configs for both of these Phoenix applications have server: false since I am using :master_proxy to route requests to the appropriate endpoint in my umbrella application.

You could send a pull request to Phoenix that adds :force_watchers which we can fallback to before we read config[:server], this way you can have server: false and force the watchers to run. If you want to use a single executable, then that's more a question for esbuild CLI. :) Closing this for now, feel free to PR phoenix. :)

@josevalim When you have a moment: https://github.com/phoenixframework/phoenix/pull/4614


stop dragging the picker when it reaches the disabled time !

the behavior that I get when the user reaches the disabled time which is in my case from 23:59 to 00:00 instead of stoping the picker there it skips it and start a new picker that starts from 00:00, what I want is that to stop the picker on 23:59 instead of skipping it and starts new picker from 00:00.
can you please fix it 🙏🏻
thank you

I have created a fork of this repo to fix this issue: https://github.com/Jonny1987/flutter_time_range_picker


Update BuildArtifactHubPkg to Ensure Proper Time Format

Ideally, we would want this to be handled within the BuildArtifactHubPkg in Meshkit. The BuildArtifactHubPkg function should take time type as a parameter rather than string. And inside the BuildArtifactHubPkg function, we would try to parse it to RFC3339.
Originally posted by @humblenginr in https://github.com/meshery/meshery/pull/11595#discussion_r1712306508

Awesome, @humblenginr 🙌 .
I created one without knowing this was already filed. The appropriate repo for this issue is meshkit, yeah? And the appropriate labels are on https://github.com/meshery/meshkit/issues/556, I think. Please either mark that issue duplicate and close or this one.


Navbar Section

🎮 feature Request
I would like to add a navbar.
Point down the features
The navbar can contain the contact us, home, FAQ section.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@Durgesh4993 Can you assign me this issue under the gssoc'24 label?

hello, please assign this issue to me under GSSOC'24.

@Durgesh4993 pls respond

Please assign me this issue under @GSS0C24


regexes containing \ need to be double-escaped

in regex-from-string.ts
 /^\/(.*)\/([a-z]*)$/.exec

is performed before a regex is created.
so for regexes that have \ they will need to be doubled escaped.

Interesting.. the problem is not because of /^\/(.*)\/([a-z]*)$/.exec. It is because of string evaluation in javascript!

oops turns out /[\u4e00-\u9fff]/u and the evaluated version /^[一-鿿]+$/u are actually equivalent so both work and there is no bug!

What a time to be alive!


Read/write to same file using  .to_parquet(overwrite=True) in the same task graph should raise an error

Performing the operation .to_parquet produces an error if the Dask dataframe is loaded from a Dask file vs. if it is created within the code and not read from file.
What happened: Way 1 of the code below works as expected - it correctly saves the output to parquet. Way 2 of the below code, does NOT work as expected and fails with FileNotFoundError: [Errno 2] No such file or directory: 'error.parquet/part.1.parquet'
What you expected to happen: I expected Way 2 to work as Way 1
Minimal Complete Verifiable Example:
import numpy as np
import pandas as pd
import dask.dataframe as dd

# -------------
# way 1 - works
# -------------
print('way 1 - start')
A = np.random.rand(200,300)
cols = np.arange(0, A.shape[1])
cols = [str(col) for col in cols]
df = pd.DataFrame(A, columns=cols)
ddf = dd.from_pandas(df, npartitions=11)

# compute and resave
ddf.drop(cols[0:11], axis=1)
dd.to_parquet(
	ddf, 'error.parquet', engine='auto', compression='default',
	write_index=True, overwrite=True, append=False)
print('way 1 - end')

# ----------------------
# way 2 - does NOT work
# ----------------------
print('way 2 - start')
ddf = dd.read_parquet('error.parquet')

# compute and resave
ddf.drop(cols[0:11], axis=1)
dd.to_parquet(
	ddf, 'error.parquet', engine='auto', compression='default',
	write_index=True, overwrite=True, append=False)
print('way 2 - end')

Environment:

Dask version: dask==2021.3.0
Python version: 3.8
Operating System: Mac OS Big Sur 11.2.1
Install method (conda, pip, source): pip

I provided an answer here: https://stackoverflow.com/questions/66794328/saving-to-parquet-throws-an-error-in-dask-dataframe/66794448#66794448
Happy to make a PR, but a decision is needed on how to handle the temporary folder.

Great, thanks - that helped. It would be good to have a workaround or fix for this.

BTW, instead of using
ddf = ddf.persist()

I use
npartitions = ddf.npartitions
ddf = ddf.compute()
ddf = dd.from_pandas(ddf, npartitions=npartitions)

it seems to be faster.

There are many ways to skin a dask dataframe...
On Thursday, March 25, 2021, mengaldo @.***> wrote:

BTW, instead of using
ddf = ddf.persist()
I use
npartitions = ddf.npartitionsddf = ddf.compute()ddf = dd.from_pandas(ddf, npartitions=npartitions)
it seems to be faster.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/7466#issuecomment-806550843, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE2FWEXBQRXCBVTPRGIMRCLTFMIXJANCNFSM4ZYZLPXQ
.

Another solution would be to raise an error when a user adds a task that will delete a file that is referenced by a different task in the graph. I wonder if @rjzamora or @crusaderky have ideas about what the behavior should be here.

Another solution would be to raise an error when a user adds a task that will delete a file that is referenced by a different task in the graph. I wonder if @rjzamora or @crusaderky have ideas about what the behavior should be here.

One of our initial concerns about adding an overwrite= argument was that it should really be the users resoponsibility to deal with file/directory cleanup.  We wanted to avoid the need to deal with issues exactly like this one :).  I do think it makes sense to add an explicit check for a circular dependency, but we should avoid adding any special handling (temporary directory creation, etc) within Dask.  If the user wants to read from a dataset and then overwrite it within the same graph execution, they should probably get an error.   It should be up to the user to decide how to manage resources (i.e. create a temporary directory, or persist the collection in memory before overwriting the data).

BTW, if instead of using ... I use ...
it seems to be faster.

That's strange.

Thanks @jsignell @rjzamora
I think that having overwrite= is a nice feature. Many deployments that we have use a combination of spark and dask, and having some consistency on how the two handle overwriting is good. I think that this issue can be solved, but given my short experience, you are much better placed than me to answer how

Ok like @rjzamora said I think the solution to this issue is:

If the user wants to read from a dataset and then overwrite it within the same graph execution, they should probably get an error

I am updating the title of this issue to reflect that decision.

By changing the way you suggested, it means that
ddf = ddf.persist()

and
npartitions = ddf.npartitions
ddf = ddf.compute()
ddf = dd.from_pandas(ddf, npartitions=npartitions)

would still be a workaround to overwrite, correct?
A simple case example in our pipeline is when updating data in a folder.
Say that you have ./data/filename.parquet
What we typically do is:

read ddf = dd.read_parquet('filename.parquet')
update e.g. ddf = ddf.merge(ddf_tmp, ...)
resave dd.to_parquet(ddf, './data/filename.parquet', write_index=True, overwrite=True, append=False)

would still be a workaround to overwrite, correct?

Yes, but keep in mind that if the process crashes for any reason before the writing is complete, you will lose all of the data since it was in memory...

Yes, there is backup; thank you for confirming.

> # ----------------------
> # way 2 - does NOT work
> # ----------------------
> print('way 2 - start')
> ddf = dd.read_parquet('error.parquet')
>
> # compute and resave
> ddf.drop(cols[0:11], axis=1)
> dd.to_parquet(
> 	ddf, 'error.parquet', engine='auto', compression='default',
> 	write_index=True, overwrite=True, append=False)
> print('way 2 - end')
>

This is IMHO a common dask antipattern. You should put yourself in a position to let dask re-run either any one node of the graph or the whole graph without warning. This is regardless of eagerness or laziness of the implementation.
I would suggest that you write to "error.parquet.new" or similar, and then rename and overwrite client-side after the computation is done, e.g. using distributed.Future.add_done_callback

@crusaderky Thank you for the heads up - will try to implement what you suggested. Would you be able to provide a code snippet on how you would practically modify the above? Thank you!

I wonder if @rjzamora or @crusaderky have ideas about what the behavior should be here.

pseudocode. Didn't actually check how the graph for read_parquet looks like.
def to_parquet(self, name):
    for layer_name, layer in self.dask.layers.items():
        if layer.startswith("read-parquet-"):
            for k, v in layer.items():
                if k[0].startswith("read-parquet-"):
                    prev_name = v[1]
                    if prev_name == name:  # TODO add logic for absolute/relative paths?
                        raise ValueError("read and write on the same file within the same graph")

@crusaderky Thank you for the heads up - will try to implement what you suggested. Would you be able to provide a code snippet on how you would practically modify the above? Thank you!

dd.to_parquet(ddf, 'error.parquet.new', ...)
os.remove('error.parquet')
os.rename('error.parquet.new', 'error.parquet')

or
d = dd.to_parquet(ddf, 'error.parquet.new', compute=False, ...)
fut, = client.compute(d)
def rename(fut):
   os.remove('error.parquet')
   os.rename('error.parquet.new', 'error.parquet')
fut.add_done_callback(rename)
fut.compute()

@crusaderky  Thank you!

I'd like to pick this up!


Rebranding of Ferdium Server

Rebrands to Ferdium logo. Related with https://github.com/ferdium/ferdium-app/issues/39

@all-contributors please add @SpecialAro for design

@all-contributors please add @mydarkstar for review


Timeouts in GenServer calls leak to the calling application

Describe the bug
Occasionally, a call to ConfigCat.get_value will time out. The timeout is not handled and therefore leaks to the calling application which can crash a process if the caller doesn't handle the timeout either.
Erlang error: {:timeout, {GenServer, :call, [ConfigCat.Client, {:get_value, "myFeatureFlag", false, nil}, 10000]}}

To reproduce
It is not easy to reproduce this, as it only happens occasionally.
Expected behavior
Any ConfigCat API call that has a default value should rescue the timeout error and return the default value rather than leaking the timeout back to the calling application. This includes get_value and get_variation_id.
It is less clear what should happen for calls like get_all_keys, get_all_variation_ids, and get_key_and_value, but probably returning empty lists for the first two and nil for the last would be best.
force_refresh should return {:error, :timeout} or similar.
My intention when I implemented this library is that get_value and similar calls shouldn't block on HTTP requests which is what seems to be happening here, so it would also be worth figuring out how these timeouts are happening in the first place to see if we can fix the root cause. I'll look into that as well.
Screenshots
If applicable, add screenshots to help explain your problem.

In the latest instance of this issue, I see this sequence of events:
May 28 03:54:54 [info] Fetching configuration from ConfigCat
May 28 03:54:55 ** (stop) exited in: GenServer.call(ConfigCat.Client, {:get_value, "quartzHttpRetriesEnabled", false, nil}, 10000)
May 28 03:54:56 [info] ConfigCat configuration json fetch response code: 304 Cached: "60b036b7-14054"

So it looks like the fetch isn't the root cause of the timeout, as it started only 1 second before the timeout happened and completed in two seconds. By design, the fetch happens in a separate task.
It is possible that our app is making enough calls to get_value that it backs up the process's message queue. We'll look at that on our end.

Hey Randy, is this issue still valid?

Yes, but it's pretty rare. I haven't had the bandwidth to dig in and fix it yet, sorry.

The Elixir SDK has been re-architecting since this issue was opened. We close this GitHub issue and see what happens with the new architecture. We can re-open later if the same issue arises again.


process.return should have return type of Nothing

As stated. Right now the type checker's definite return checks break if a function exits rather than returning.

I agree that process.exit should return Nothing, but this wouldn’t actually affect definite return analysis until ceylon/ceylon-spec#1179 is added.

Well you can write return process.exit();


"MD010 - Hard tabs" - ignore for code fences?

Hi,
If I'm having a code fence in markdown for a language that may contain tabs, markdownlink shouldn't mark this with a MD010 warning? Because it's not markdown, it's a "quote" of another programming language that perhaps may contain tabs.
E.g. this .gitattributes content have tabs and that's fine:
*               text=auto
*.txt		text
*.vcproj	text eol=crlf
*.sh		text eol=lf
*.jpg		-text

You want to set the code_blocks parameter of MD010 to false: https://github.com/DavidAnson/markdownlint/blob/master/doc/Rules.md#md010

Ah sorry, didn't know how I overlooked that. Curious, why is the default true?

In my experience, tabs are handled differently by many tools. So I would prefer not to have any in my document which should lead to the most predictable behavior. But for scenarios where tabs are wanted in code, this option lets you allow them.

I see. Thanks for the elaboration.

On 1 Aug 2018, at 02.53, David Anson<EMAIL_ADDRESS>wrote:
In my experience, tabs are handled differently by many tools. So I would prefer not to have any in my document which should lead to the most predictable behavior. But for scenarios where tabs are wanted in code, this option lets you allow them.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Support swift 5.0

@LowKostKustomz
I think this is correct. Won't you merge?

Thanks a lot! Will publish version with this change soon. Stay tuned:)


Add vue-material-design-icons

Adds the vue-material-design-icons library to the Icons section. This is a collection of inline svg Material Design Icons packaged as single file components.
See https://gitlab.com/robcresswell/vue-material-design-icons or https://www.npmjs.com/package/vue-material-design-icons for more information.

Can you please include a link to a list of icons in your README.md? I couldn't find one.

Done. 0.6.3 notes the source repo more explicitly at the top of the README, and also has a section titled "Icons" that explains how to find more, as well as the naming structure. Thanks for the quick response!


Can the parsing performance be improved by using a precomputed index?

Hi. I use your parser to parse large (in fact not very large, from 200 MiB to 2 GiB each file is) CSV datasets consisting mostly of strings. No DB functionality is needed, so I feel like moving the data into a real DB as a preprocessing step will not bring any benefit.
Also I cache intermediate results into plain text files, one record per a line, but lines are not necessarily CSV.
I have designed a file format (mostly a prefix tree with additional data in each non-leaf node, just not to waste space) allowing one to save the offsets of line breaks for getting a record by its line number fast.
I have also extended it to hold arbitrary shapes. So we can save not only line breaks, but other separators, like commas (and no separators at all). The format seems to be very versatile and multipurpose.
But all of that scannjng code is currently coded in python. Intel Hyperscan (a fast large scale jitted AVX-accelerated regex matching engine used in DPIs) is not useful, since it cannot match subgroups.
I wonder if we can accelerate parsing by first precomputing the index file (it is very likely it can even be done on GPU, though I am not sure if it will bring any benefit) and saving it (done once), then by using the offsets from it (can be done multiple times) to avoid scanning and tokenizing entirely.

If you can create an index, then my recommendation is to work with binary files and read-only mmap. Nothing beats a simple array lookup. This library is meant to parse CSV files that you see for the first time.
I have experimented with SIMD code in the past but was not able to get a consistent significant speedup. For this reason, I stuck with the simpler approach. GPU makes IMO no sense as just the data copying from GPU to CPU will be slower than than the CPU parsing.

If you can create an index, then my recommendation is to work with binary files and read-only mmap.

Yeah, it's the way I use it, but is the for plain index. I haven't yet finished a trie-based index, because after I added the data for fast arbitrary lookup by index the trie-based index is only twice smaller than the flat one. But applying the trie compression recursively looks promising, but I need to tinker bit more around it.

But all of that scanning code is currently coded in python.

is no longer true, I have rewritten the scanner in C++ and paralleled it aggressively and have also implemented JIT for x86_64. https://github.com/ScanBytes/ScanBytes.cpp Very surprisingly, std::thread-based impl turned out to be the fastest, and Intel TBB-based one turned out to be the slowest and OpenMP in between.

GPU makes IMO no sense as just the data copying from GPU to CPU will be slower than than the CPU parsing.

It depends. Some devices have a GPU and a CPU within a single chip and use the same memory.


@Info annotation is not picked up on my Swagger.json file (Scala)

With a code like this in Scala, Info object is not picked up on my Swagger.json:
@SwaggerDefinition(
  info = new Info(
    description = "Gets the weather",
    version = "V12.0.12",
    title = "The Weather API",
    termsOfService = "http://theweatherapi.io/terms.html",
    contact = new Contact(
      name = "Rain Moore",
      email =<EMAIL_ADDRESS>      url = "http://theweatherapi.io"
    ),
    license = new License(
      name = "Apache 2.0",
      url = "http://www.apache.org/licenses/LICENSE-2.0"
    )
  ),
  consumes = Array("application/json"),
  produces = Array("application/json"),
  schemes = Array(SwaggerDefinition.Scheme.HTTP, SwaggerDefinition.Scheme.HTTPS)
)

However, properties like consumes, produces and schemes are picked up.
I got the reference from this Java example: https://github.com/swagger-api/swagger-samples/blob/master/java/java-servlet/src/main/java/io/swagger/sample/servlet/SampleServlet.java#L27-L40
What can I check to solve this?
Thanks!

Same issue here: https://github.com/swagger-api/swagger-play/issues/134


when creating a new resource, the UI is converting double to int

To replicate, simply try to create a voucher resource with:
{
  "owner": "user/<uuid>",
  "target-audience"<EMAIL_ADDRESS>  "amount": 50.0,
  "currency": "EUR",
  "code": "CEODDOECXYZ01",
  "state": "NEW"
}

and you'll see a schema error due to 50.0 being converted to 50.

With 50.1 it works. Strange.

There is no such thing as an integer in JavaScript. Numbers in JavaScript are "double-precision 64-bit format IEEE 754 values".
Open up your web browser's console and type 1.0. You'll see 1 printed out. All numbers in JavaScript are floating point numbers, so your serializer simply chose to leave off unnecessary precision.
ref:  https://stackoverflow.com/questions/16662031/json-stringify-conversion-of-float-to-int