Pull in Pip fixes.

Some Pex users are encountering resolve issues fixed in newer versions of Pip. Pex has been stuck on the last version of Pip to support Python 2.7 (20.3.4) in order to continue to support Python 2.7 customers. Either Pex has to cut those customers loose or it has to support both old and new Pips.
The basic scenarios:

Cut loose - fend for yourself: Pex simply bumps its major version and moves on; Python 2.7 users get no bug fixes or features and are stuck on the last version of Pex to vendor Pip 20.3.4.
Bifurcate with support: Like 1, except Pex cuts a long-term branch where fixes and releases for the Pex 2.1.x / Pip 20.3.4 series go on in parallel to Pex 3.x / Pip ~latest development and releases off of main.
Support two versions of Pip in the same Pex release: This could keep the 20.3.4 Pip vendoring in the released Pex but just-in-time resolve a newer version of Pip using the older vendored Pip. The use of the newer Pip could either be controlled via flag or via auto-detect from interpreter constraints / interpreters used / platforms.

Obviously, 1 is easiest from a maintainer perspective, but it violates an implicit contract with users. It would be better to phase out Python 2.7 / 3.5 / 3.6 / etc. support in a controlled way independent of needing to upgrade Pip to get Pip bug-fixes. As the current ~lone maintainer, I'm partial to 3 to both meet users needs and have a compact codebase to maintain. I fear 2 would lead to more complicated maintenance and missed / partial fixes.

The motivating slow lock resolves were reported variously through #1798 on behalf of Pants users.

This upgrade looks like it will close #1526, #1527 and #1528.

The only apparent differences found in Pip 22.2.2 are:

An extra Wheel model method that needs to be runtime patched for --style universal locks. This patch is a noop against 20.3.4-patched; so this works out well.
Pip now attempts to use the json API defined in https://peps.python.org/pep-0691/. This API is implemented by PyPI; so Pip logs from PyPI downloads no longer contain hashes, they are hidden in a fetch of https://pypi.org/simple/<project name>/ with Accept: application/vnd.pypi.simple.v1+json.

1 is patched and working, and 2 just leads to less efficient locking. The fall-back download-and-hash code is now used always. So 2 needs an optimization fix to 1st try to use PEP-691 json API for all locked urls without a hash before falling back to the much more expensive, download all those URLS to hash their contents.

Ok, wiring up the PEP 691 JSON API was straight forward. This will fail for custom indexes with Bearer auth, but then we just fall back to downloading and hashing. That's slower, but still correct.
This leaves 1 unit and 3 IT failing using the 22.2.2 resolver in existing tests. The unit failure is in bdist_pex and still mysterious and the 3 ITs appear to center around local project / VCS hashing and those are still otherwise mysterious as well.

Alright, sorted. Supporting Pip 20.3.4-patched + vendored setuptools and wheel by default and Pip 22.2.2 + setuptools 65.0.0 + wheel 0.37.1 (all latest as of today), when --pip-version is requested turns out to be viable. There is 1 added line of runtime patching code to Pip and a new utility to get hashes via PEP-691 that is only engaged for new log lines emitted by new Pip. It should be the case that Pex can just add to the PipVersion enum every now and again to expand to newer Pips while retaining support for 20.3.4-patched ~indefinitely at the User's prerogative. Pex3 can just flip the default from vendored to latest supported Pip, but can continue to support users until Redhat 2.7 support goes away or some similar proxy for truly EOL.

Noting I hit this bug using 22.2.2 in ITs sporadically: https://github.com/pypa/pip/issues/11340 Users will too and that's fine. The answer will be to add a new --pip-version once they release the fix. It does beg the question though of how many Pip versions Pex will end up having in the choices for --pip-version. That set could grow pretty big - 100s. Maybe that's fine. Alternatively Pex could get fancy and allow for implicit latest in that set when version components are left off. --pip-version 22 would snap to 22.2.3 when Pip releases that and Pex then adds / verifies support for 22.2.3.


readme + visual changes

This PR fixes some tiny issues when rendering in the cli the results.
I've also added some documentation to the readme. 👓

Codecov Report

Merging #7 (7d56cd4) into main (3febb0b) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines           99        99
  Branches         2         2
=========================================
  Hits            99        99

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/commands/diff.ts
100.00% <100.00%> (ø)

src/commands/regression.ts
100.00% <100.00%> (ø)

src/commands/results.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[Bug]: Inkscape-Dev fails to create shims but otherwise installs correctly

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.
[X] I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name
inkscape-dev
Expected/Current Behaviour
Experiencing a failure when installing inkscape-dev using scoop. It should create shims to /bin/inkscape.exe and /bin/inkview.exe, but it looks like it is trying to install shims for bin/inkscape.com and bin/inkview.com.
Steps to Reproduce
PS > scoop install inkscape-dev
Installing 'inkscape-dev' (1.3-dev_2023-03-28) [64bit] from versions bucket
inkscape-1.3-dev_2023-03-28_d93531fe2-x64.7z (91.9 MB) [======================================================] 100%
Checking hash of inkscape-1.3-dev_2023-03-28_d93531fe2-x64.7z ... ok.
Extracting inkscape-1.3-dev_2023-03-28_d93531fe2-x64.7z ... done.
Linking ~\scoop\apps\inkscape-dev\current => ~\scoop\apps\inkscape-dev\1.3-dev_2023-03-28
Creating shim for 'inkscape'.
Can't shim 'bin\inkscape.com': File doesn't exist.

Possible Solution
By changing the .json file "\scoop\buckets\versions\bucket\inkscape-dev.json", to change
17    "bin": [
18           "bin\\inkscape.com",
19           "bin\\inkview.com"
20     ],

to
17    "bin": [
18           "bin\\inkscape.exe",
19           "bin\\inkview.exe"
20     ],

allowed me to complete the install.
Output:
PS > scoop install inkscape-dev
Installing 'inkscape-dev' (1.3-dev_2023-03-28) [64bit] from versions bucket
Loading inkscape-1.3-dev_2023-03-28_d93531fe2-x64.7z from cache
Checking hash of inkscape-1.3-dev_2023-03-28_d93531fe2-x64.7z ... ok.
Extracting inkscape-1.3-dev_2023-03-28_d93531fe2-x64.7z ... done.
Linking ~\scoop\apps\inkscape-dev\current => ~\scoop\apps\inkscape-dev\1.3-dev_2023-03-28
Creating shim for 'inkscape'.
Creating shim for 'inkview'.
Creating shortcut for Inkscape Dev (inkscape.exe)
'inkscape-dev' (1.3-dev_2023-03-28) was installed successfully!

Scoop and Buckets Version
PS > scoop --version
Current Scoop version:
v0.3.1 - Released at 2022-11-15

'extras' bucket:
5277529d5 (HEAD -> master, origin/master, origin/HEAD) zoom: Update to version <IP_ADDRESS>87

'main' bucket:
9caaf1c9d (HEAD -> master, origin/master, origin/HEAD) mysql: Update to version 8.0.33

'versions' bucket:
aa8fe5461 (HEAD -> master, origin/master, origin/HEAD) tomcat9: Update to version 9.0.74

Scoop Config
PS > scoop config

last_update                       scoop_branch scoop_repo
-----------                       ------------ ----------
2023-04-18T08:41:11.5986111-07:00 master       https://github.com/ScoopInstaller/Scoop

PowerShell Version
PS > $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.22621.963
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.22621.963
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Additional Softwares
No response

Extracting inkscape-1.4-dev_2023-06-09_83ab8c0da-x64.7z ... done.
Linking ~\scoop\apps\inkscape-dev\current => ~\scoop\apps\inkscape-dev\1.4-dev_2023-06-09_83ab8c0da
Creating shim for 'inkscapecom'.
Creating shim for 'inkviewcom'.
Creating shortcut for Inkscape Dev (inkscape.exe)
'inkscape-dev' (1.4-dev_2023-06-09_83ab8c0da) was installed successfully!

Fixed in the latest version of the manifest!


Default Non-Analog Elements specified directions to Output, remove fixups and Unspecified handling. Fixes #2794

Contributor Checklist

[x] Did you add Scaladoc to every public function/method?
[x] Did you add at least one test demonstrating the PR?
[x] Did you delete any extraneous printlns/debugging code?
[x] Did you specify the type of improvement?
[ ] Did you add appropriate documentation in docs/src?
[x] Did you state the API impact?
[x] Did you specify the code generation impact?
[x] Did you request a desired merge strategy?
[x] Did you add text to be included in the Release Notes for this change?

Type of Improvement

bug fix

code refactoring

API Impact

The main thing this does is that, now that we agree that "Output" and "Unspecified" are basically the same thing aka "Aligned", and are going through hoops to fix this up on ports, is to just make that done at the leaves themselves for non-Analog Elements. This avoids the need to do fixups and confusion/errors around cloneTypeFull which does not do the same recursion
This is an alternative to #2792 . Fixes https://github.com/chipsalliance/chisel3/issues/2794
Backend Code Generation Impact

This shouldn't change generated verilog unless there were bugs.
Desired Merge Strategy

Squash: The PR will be squashed and merged (choose this if you have no preference.

Release Notes

Assumption that elements with Unspecified specified Direction == Output is now done in the Element constructor itself rather than as a fixup step. Fixes #2794 to ensure that D/I works with modules with IO with Bundles with Unspecified directions.
Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (Bug fix: 3.4.x, [small] API extension: 3.5.x, API modification or big change: 3.6.0)?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you do one of the following when ready to merge:

[ ] Squash: You/ the contributor Enable auto-merge (squash), clean up the commit message, and label with Please Merge.
[ ] Merge: Ensure that contributor has cleaned up their commit history, then merge with Create a merge commit.

i think this is the right fix for master, but for 3.5.x we probably should just do the appropriate version of https://github.com/chipsalliance/chisel3/pull/2792

Chatted a bit with @azidar on this, and we are thinking it might be worth doing more of #2634 tasks (e.g. add a word Aligned and use it consistently instead of Output and Unspecified to mean "aligned") first before tackling this bug. In the meantime it can be logged as a known issue that cloneTypeFull does NOT copy specifiedDirections recursively so anywhere the fixup is relied upon (any ports) is going to give the wrong answer if we do e.g. val io =  IO(chiselTypeOf(someOtherIO)).
Another "fix" here would be to make cloneTypeFull copy specified direction recursively but that could be a big performance hit.

closing in favor of https://github.com/chipsalliance/chisel3/pull/2792.


Enable both sliding and absolute expiration for cookie authentication

The current Microsoft.Owin.Security.Cookies middleware lets you select from either sliding or absolute expiration. It would be nice to allow for both - a short sliding expiration to handle timing out inactive users, but also an absolute expiration to handle maximum allowed session length before requiring re-authentication.
The only way to handle this currently is outlined in this blog article - modifying the cookie authentication provider to add custom data for the absolute expiration and having it enforced there.
It would be nice to have first-class support for this rather than having to hack it in every time.

This would be great to have.


Need update on limitations?

I believe this one is not: Export using Underlying data won't work if The data source is an Analysis Services live connection.

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

ID: bc572957-04b2-9512-afb5-11bc0393cf66
Version Independent ID: 70c74633-44df-466b-879c-4ae5c1524fc1
Content: Export data from a Power BI visualization - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualization-export-data.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @mihart
Microsoft Alias: mihart

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

Hi @MichaelDotEarly ,
Wanted to let you know that the limitation has been removed from the documentation.  Thank you for letting us know.


Combining ingredients using config

Hello authors! Thank you for great project, I was excited when I first tried it for my workflow and got easily automated experiments. But I have some issues with ingredients.
In my setup I have several configurations of model. I implemented it using ingredients for each separate part of model. Now for each part I have several ingredients, and want to combine them in whole model. I cannot use config scope for getting ingredient list, so I need to change my source code for each different combination of ingredients. Is there a better way to do this?

To clarify with a small example.
Suppose I have ingredients:
part1_1 = Ingredient('part1.1')
part1_2 = Ingredient('part1.2')

part2_1 = Ingredient('part2.1')
part2_2 = Ingredient('part2.2')

And I need to combine part1 and part2. For this I use:
model = Experiment('experiment', ingredients=[part1_1, part2_1])

Now if I want to change ingredient list for the model, I need to change this line in source file. It's looking a bit ugly for me, using both config scope and source file changes for setting up the experiment.
It would be better if I somehow could do like this:
# source file
model = Experiment('experiment', ingredients=model_parts)

# config file
model_parts:
  part1.1
  part2.1

See technique that I use with success in similar scenario, I described it here https://github.com/IDSIA/sacred/issues/295 TL;DR include all ingredients to experiment, but use additional config to select which one is in use.

Thank you, it's a solution, but I don't like the idea to store list of all ingredients somewhere. Because when you add something new, you need not only to add source file for implementation but also add name of the ingredient to another source file.
I wonder if there some other way.

You can use importlib and import Python module based on some config, but it (propably) won't be an  Ingredient anymore, so you'll lose Sacred config handling.

Ok, I'll look into it.

@sebastian-df Thanks for stepping in!
@vuvko : Unfortunately, the ingredients by themselves are not currently not flexible enough for your usecase. You can workaround that problem, but it requires some manual work (like suggested by sebastian-df). See also #260 for an example on how to dynamically change ingredients based on commandline. That solution could probably be adapted to read the ingredients list from a config file.


Danielhickman master

Close #46. Simplified some of the styles in it.

Ok, seems good to me.

Closed #46 .

@razetime whenever a branch is merged, we can also delete it. ;) Click the »Delete branch« button here next to the merge message.

Thanks!


Modules Not Loading In Nethunter

So I'm trying to load the driver modules manually in Nethunter (v2019.2) and it is not loading at all. I flashed the Nethunter klte kernel nougat. My device I'm using is GS5, I installed Nethunter through Magisk Module.
When I type find /lib/modules/3.4.114-nethunter-klte-spr-3.0 -type f -name athk9k* in Kali I get a error saying no such file or directory.
When in type modprobe -rtv ath9k I get a error saying modprobe: ERROR: ../libkmod/libkmod.c:586 kmod_search_moddep() could not open moddep file '/lib/modules/3.4.113-nethunter-klte-spr-3.0/modules.dep.bin
So trying to load this in kali just isn't working.
So I tried running find /lib/modules/3.4.114-nethunter-klte-spr-3.0 -type f -name athk9k* in Android su. It finds the files with no problem, it lists them. So I decided to run these commands in Android su:
modprobe -rtv ath9k_htc.ko
rfkill unblock all modprobe -v ath9k_htc.ko

It errors out saying modprobe: module 'ath9k_htc.ko*' not found. Insmod brings up a no such file or directory.
I get the same error when i replace ath9k_htc.ko with ath or ath9k  What's going on!!!! I've never had issues loading ath9k drivers through Nethunter. I need help. My wifi adapter is TP-LINK WN722N v1.

Update the NetHunter app to v2019.3-rc1 -- fixes the modules autoloading feature so you don't need to load modules in Android SU before entering NetHunter chroot

And for even easier control of the modules, download the "KoControl" app

@kimocoder thank you for the quick response. I will give it a try.

@kimocoder is the KoControl app in Magisk?

No problem :+1:
https://gitlab.com/kalilinux/nethunter/apps/kali-nethunter-app/releases

No it's not, we're working to integrate it into NetHunter, the developer sent the sources today.
Until then, use https://m.apkpure.com/kocontrol/com.faust93.kocontrol

@kimocoder What's with this error "modprobe: ERROR: ../libkmod/libkmod.c:586 kmod_search_moddep() could not open moddep file '/lib/modules/3.4.113-nethunter-klte-spr-3.0/modules.dep.bin" what is it from?

@kimocoder still not loading up modules and it's running the latest nethunter apk.

I have this issue too. No modules and updated to latest. KO control is blank.

@kimocoder still not loading up modules and it's running the latest nethunter apk.

Edit: all is well. Drivers are loading now.

I have the same problem, how did you fixed it?

me too

@kimocoder still not loading up modules and it's running the latest nethunter apk.

Edit: all is well. Drivers are loading now.

I have the same problem, how did you fixed it?

me too i already install kernel nethunter for my poco x3..but no modules..cant use externel wifi..and hid..howto load modules ?

I am facing right now the same issue with no modules on lsmod on my Nexus 5x with everything updated... Anyone can help with that?

I am facing right now the same issue with no modules on lsmod on my Nexus 5x with everything updated... Anyone can help with that?

did you fixed that?


Curve, Points, and Scatter plots broken on Safari for Bokeh backend

On the Safari browser on Mac OS X, if you use the Bokeh backend, some plot types fail to show any data:

The plot types affected appear to be those handled by WebGL, and indeed the curves come back if we turn off Bokeh's WebGL support:

This appears to be a known issue in Bokeh.
We should presumably either turn WebGL off by default in Safari, or emit a warning.

Just double checked, this has been fixed in bokeh. Closing.


AVRO-2963: Add docker configurability from env

When testing and building locally, I frequently find myself fiddling with the Avro docker image to try and reproduce what is happening on Travis.
This PR will permit us to do things like:
# Rebuild the docker image without using the build cache.
DOCKER_BUILD_XTRA_ARGS=--no-cache ./build.sh docker

# Build a docker image with a specific tag (for an RC or poc, for example)
DOCKER_IMAGE_NAME=avro-build:1.10.1-rc1 ./build.sh docker

# Run a command and return.
DOCKER_RUN_ENTRYPOINT="mvn --version" ./build.sh docker

Jira

[ ] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-2963
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

What do you think about adding some of your examples from the PR description to BUILD.md?


Implement Connection DisconnectedErr Callback

Feature Request see:  https://github.com/nats-io/nats.go/pull/462

Error can be retrieved by calling getLastError on the Connection


journald logging driver is skipping lines

Description
Docker daemon configured with --log-driver=journald is skipping random log-entries.
The following is a stripped-down example for debugging this issue.
Steps to reproduce the issue:

run test with journald driver:
 docker run -d --name=ml_jd --log-driver=journald nilsotto/massivelog; sleep 60; docker stop ml_jd
check logs
docker logs ml_jd
As you will see, the container prints equal-signs in bathces of 75 lines increasing in length
With the journald log driver, random parts of the output is not logged.
This can be verified by
docker logs ml_jd | cut -d " " -f 1 | uniq -c | grep -v "75"
wich prints iteration numbers not having 75 lines.
When a "skip" occurs, the log lacks several hundred lines, this happens two times in a one minute run on my laptop
Describe the results you received:
LInes are skipped in the journald log

Describe the results you expected:
All lines should be logged,
The same experiment with the json-file log driver is logging every line
Additional information you deem important (e.g. issue happens only occasionally):
Output of docker version:
Client:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 489
 Running: 0
 Paused: 0
 Stopped: 489
Images: 154
Server Version: 1.12.1
Storage Driver: overlay
 Backing Filesystem: extfs
Logging Driver: journald
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host overlay null bridge
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.7.2-101.fc23.x86_64
Operating System: Fedora 23 (Workstation Edition)
OSType: linux
Architecture: x86_64
CPUs: 8
Total Memory: 6.75 GiB
Name: localhost.localdomain
ID: SUN6:43M3:MT3R:S2KS:JUYK:HHS5:3DTN:RFK5:55SX:BB6G:3FUV:A57Y
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Username: nilsotto
Registry: https://index.docker.io/v1/
Insecure Registries:
 <IP_ADDRESS>/8

Additional environment details (AWS, VirtualBox, physical, etc.):
I have tried on our test-hosts as well with the same result

well, it seems that this problem stems from system configuration on our docker host.
Systems with massive logging wil probably need to set RateLimitIntervalSec and RateLimitBurst in the docker host's journald.conf file.
I have created PR https://github.com/docker/docker.github.io/pull/670 to try to prevent others from having the same problems.


Resource interpreted as Stylesheet but transferred with MIME type

I'm getting this when i set:
<script src="/min/?f=min/../template/js/jquery2_2_4.js,template/js/bootstrap.js,template/js/jquery.colorpanel.js"></script>
With CSS i have no problems, but with js this is what i'm getting in console log in my browser.

What are the HTTP response headers for that request?

http://i.imgur.com/ePF4LxR.png
text/html

Resource interpreted as Stylesheet but transferred this is a problem with your PHP script response, not Minify. Minify is returning Content-Type: application/x-javascript; charset=utf-8 as expected.


dm: handle the query in metadata from binlog

What problem does this PR solve?

Issue Number: close https://github.com/pingcap/tiflow/issues/11735
What is changed and how it works?
Handle the new format of binlog metadata from mariadb to read the binlog position.
Check List
Tests

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

Questions
Will it cause performance regression or break compatibility?
Not.
Do you need to update user documentation, design documentation or monitoring documentation?
Not.
Release note
None

@joccau please sign CLA

@joccau please sign CLA

Done

/retest

/retest


vcard is not a function

When I using this npm it shows like
vcard is not a function     at Vcardgenerator.generate (/home/jose/VCardApp/vcardgenerator.js:32:30)     at /home/jose/VCardApp/app.js:62:20     at Layer.handle [as handle_request] (/home/jose/VCardApp/node_modules/express/lib/router/layer.js:95:5)     at next (/home/jose/VCardApp/node_modules/express/lib/router/route.js:137:13)     at Route.dispatch (/home/jose/VCardApp/node_modules/express/lib/router/route.js:112:3)     at Layer.handle [as handle_request] (/home/jose/VCardApp/node_modules/express/lib/router/layer.js:95:5)     at /home/jose/VCardApp/node_modules/express/lib/router/index.js:281:22     at Function.process_params (/home/jose/VCardApp/node_modules/express/lib/router/index.js:335:12)     at next (/home/jose/VCardApp/node_modules/express/lib/router/index.js:275:10)     at /home/jose/VCardApp/node_modules/body-parser/lib/read.js:130:5     at invokeCallback (/home/jose/VCardApp/node_modules/raw-body/index.js:262:16)     at done (/home/jose/VCardApp/node_modules/raw-body/index.js:251:7)     at IncomingMessage.onEnd (/home/jose/VCardApp/node_modules/raw-body/index.js:307:7)     at emitNone (events.js:86:13)     at IncomingMessage.emit (events.js:185:7)
The npm is successfully installed..but shows like this...

Hey @joseamt ! I was helping you over on Stack Overflow. I won't, but you may want to consider linking your SO questions to this issue. It will help the developer gain more details without the repetition of re-asking.

@joseamt @trentrand Thanks for reporting the issue. Sorry for delay, I see that I have made a mistake of not compiling sources into lib/. I will fix this soon.

@joseamt @trentrand hmm. I think lib/ was published already, but I've republished anyways under new version v0.0.4 to npm. I have tested and it works for me.

I will close this as fixed. Feel free to open it   again if it doesnt work for you.

@joonhocho The issue still exists. How can I use this npm in to my project. I am using node v6.9.4. How can I use this npm in my project. In node, 'require' is using for including modules. As per the documentation 'import' command is not working. can you please help me

@joseamt I honestly don't know why it's not working for you. It should. It works for me. You simply need to run npm i -S vcard-generator@latest and use var vcard = require('vcard-generator') in your code.

@joseamt OK, now I know the problem. I think without babel or es6, you can't export and import function directly.  I always use babel with es6, so it worked for me.
It seems you don't. So, I wrapped the function inside an object and it should work for you.
Run npm i -S vcard-generator@latest and use var vcard = require('vcard-generator'); vcard.generate({...}).
Let me know if that works.

@joonhocho First of all thank you for your cooperation. Actualy my requirement is set set multiple urls inside a single Vcard. No other npm found for supporting to set multiple ulrs. Can you suggest any other npm available for my use case.

@joseamt Sorry, I don't know.

@joonhocho Ok. Now I changed my node version to 7 and its working. Vcard is generated. But it is not supporting or importing in to my contacts. It shows 'vcard is not supported'.  How can I change the vcard version?

You can use version option.


[BEAM-14004] fix(mongodb): Ignore error Command not supported because of CosmosDB not support splitKeys command

fix(mongodb): Ignore error Command not supported because of CosmosDB not support splitKeys command
Reviewed-by: @jbonofre
Refs: # (pull request) [BEAM-14004] (jira)

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@jbonofre , I'm create pull #16972 with 2.36.1 version.

Can you revert the release version related changes ? Cutting a patch version requires a separate process an I don't think it's worth it just for this change. Beam releases minor versions approximately every six weeks so this should be picked up by the next minor release.

Can you revert the release version related changes ? Cutting a patch version requires a separate process an I don't think it's worth it just for this change. Beam releases minor versions approximately every six weeks so this should be picked up by the next minor release.

Ok, but where do I put this?
Bugfixes

Ignore error Command not supported because of CosmosDB not support splitKeys command in Java SDK (BEAM-14004).

You can put it under the bug fixes for 2.39.0
https://github.com/apache/beam/blob/master/CHANGES.md#2390---unreleased

Run Java PreCommit

What is the next step on this PR?

Could we remove the changes to the changes.md and merge the rest (if that part is reviewed & tested) ?

@chamikaramj @kileys @aaltay , I'm closing this pull and open the new PR #17511 pointing the version 2.39 (master)


Login button/link

I love this template, but it would be nice to be able to have the default login functionality back to provide a login link/button, when the user is not logged in.  I added a login link to our sidebar but it looks kinda janky.

There is an option "Show toolbox even when logged out" in "Configuration Manager" - "template" - "Readthedokus". Please check it; then the Tool Box menu appears under each page even when logged out. There is the "Login" link in the menu.

Perhaps,  if the option
https://github.com/my17560/dokuwiki-template-readthedokus/blob/3b961a9c718a0b6261efe222a602566a121ce574/conf/default.php#L4
was true by default, it would cause less confusion.

Sorry for the confusion. I wanted to make it look like a regular web page, not a wiki page when logged out.

It’s been a while since the last comment, I’m closing the issue. If you still have any problems, please reopen this issue or create a new one.


Find interpreter based on hash in kernelspec of nb metadata

For #13782

Codecov Report

Merging #13856 into main will increase coverage by 0.02%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##             main   #13856      +/-   ##
==========================================
+ Coverage   59.98%   60.00%   +0.02%
==========================================
  Files         685      685
  Lines       38006    38103      +97
  Branches     5474     5492      +18
==========================================
+ Hits        22797    22865      +68
- Misses      14034    14049      +15
- Partials     1175     1189      +14

Impacted Files
Coverage Δ

...rc/client/datascience/notebookStorage/baseModel.ts
63.01% <28.57%> (-1.28%)
:arrow_down:

...ient/datascience/jupyter/kernels/kernelSelector.ts
63.50% <54.54%> (-0.50%)
:arrow_down:

src/client/datascience/crossProcessLock.ts
79.41% <0.00%> (-11.77%)
:arrow_down:

src/client/common/utils/platform.ts
56.00% <0.00%> (-8.00%)
:arrow_down:

src/client/common/utils/async.ts
84.94% <0.00%> (-2.56%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

src/client/datascience/common.ts
58.66% <0.00%> (ø)

...ent/pythonEnvironments/discovery/locators/index.ts
87.93% <0.00%> (+2.21%)
:arrow_up:

Continue to review full report at Codecov.

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


Upgrade USWDS 3.0

Description
Our current version of USWDS (2.13) will no longer be supported beyond May 2023 -
https://designsystem.digital.gov/documentation/developers/#long-term-support-of-v2x
Upgrade to USWDS 3
Might have to deviate from migration guide.
Benefits up upgrading to 3.0:

Modern Sass syntax
Improved performance, reduced size of project CSS
No markup or style changes from USWDS 2.13.3 - "If you already use USWDS 2.13.0 or later, you should be able to update to USWDS 3.0 in a matter of minutes."
Easier to stay up-to-date with USWDS over time
Design team is planning on using most current version as basis for Figma library

Action requested

Determine the best course of action to upgrade and make upgrade

Acceptance criteria

We are on a supported version of USWDS

When the update to the static site is complete, please replace the current external link disclaimer with @include external-link;


fix(userspace/libsinsp): remote data fetching with openssl 1.1.1

Signed-off-by: Federico Di Pierro<EMAIL_ADDRESS>
Co-authored-by: Jason Dellaluce<EMAIL_ADDRESS>Co-authored-by: Leonardo Grasso<EMAIL_ADDRESS>Co-authored-by: lucklypse<EMAIL_ADDRESS>
What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area libsinsp
What this PR does / why we need it:
This PR fixes a bug introduced with the openssl version bump from 1.0.2 to 1.1.1, that broke socket_handler::get_all_data() https path.
Which issue(s) this PR fixes:
Fixes #89
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

build: upgrade openssl to 1.1.1l

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

b9ffc5a Merge branch 'falcosecurity:master' into fix_ssl_1_1_get_all_data

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@FedeDP: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Hi @FedeDP. Thanks for your PR.
I'm waiting for a falcosecurity member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

LGTM label has been added.  Git tree hash: 4d1bb59fa4419a04cc8949c00851e334d945209d

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold
we need to check if this work with civitweb on Falco

New changes are detected. LGTM label has been removed.

Had to force push as falco master was failing to build with old libs.
Will ask for approval once tested successfully.

@FedeDP: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs
ee588ae492c12ad6907362c717e0a775c9c9a098
link
true
/test build-libs

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Note that i added the OPENSSL_LIBRARIES definition in openssl cmake module when built BUNDLED as that variable is set by system openssl module and expected by lots of other cmake modules, like grpc, civetweb and a few more (grepping for OPENSSL_LIBRARIES in libs build folder will reveal all occurrences).

Rebased on libs master.

@FedeDP: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-minimal
d7ffb9972e481c0696bb24ab06cccd02ae09f0fd
link
true
/test build-libs-minimal

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test build-libs-with-chisels

LGTM label has been added.  Git tree hash: 502913639f9baffcc4e23b9a32cfdf42eeaa7e99

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@FedeDP: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-bundled-deps
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs-bundled-deps

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@FedeDP: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-bundled-deps
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs-bundled-deps

build-libs-minimal
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs-minimal

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@FedeDP: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-bundled-deps
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs-bundled-deps

build-libs-minimal
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs-minimal

build-libs
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@FedeDP: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

build-libs-bundled-deps
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs-bundled-deps

build-libs-minimal
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs-minimal

build-libs
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs

build-libs-with-chisels
a77eefdf338facc3b5adc00876f6b1af08096cf8
link
unknown
/test build-libs-with-chisels

Full PR test history. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Whooa it seems like googletest updated their master branch name to main...literally one hour ago :D https://github.com/google/googletest/commit/4848324c5ff3186a7f72a674f6760d0b8da585d7
Is this worth fixing in this PR?

New changes are detected. LGTM label has been removed.

Rebased on master.
Asking for re-approval :) @leogr  @mstemm

LGTM label has been added.  Git tree hash: 0834ea52d0ba9057ccfdc62bc3027462e750000d

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Logging System

Logging system that moderates some of the moderation commands.

in development... not prefect as you have to provide the logging channel id in each moderation command.


Can I prefetch / preload data before the route is processed?

Does this router support loading of data before a route is handled? I would like to do something like
async function fetchMyData(props){
    props.viewData = await ajax(`/server/${props.id}`);
}

<Router>
    <MyComponent path="/view/:id" preload={fetchMyData}>
</Router>

class MyComponent extends React.Component {
    render(){
        return(
            <Fragment>
                {this.props.viewData}
            </Fragment>
        );
    }
}

The route for MyComponent calls fetchMyData before proceeding to mount MyComponent, at which point the data is already bound to props. This spares my component the mess of having to maintain a "busy" and a "done" view state, and lets me cleanly separate data and view logic. I've been trying for hours to get this to work in react-router-dom, but it doesn't look like they support it.

It doesn't support loading data before the URL navigation happens because it relies on the Browser to resolve the navigation path.
However I made a demo that showcases data prefetching using a method "getInitialProps" like Next.js does plus lazy loading and animating transitions between routes. You can find it here:
https://github.com/jvitela/reach-router-lazy-transitions

@jvitela thanks for the reply, closing this ticket then. I'll give your solution a try too.


Fix documentation error

Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[x] Did you read the contributor guideline?
[x] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes error in documentation, method test_end should refer to test_loss rather than  val_loss
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

personally I would wait with any documentation updates/fixes until #521 is resolved, otherwise, these changes will be almost impossible to track back and apply so they will be lost... :cry:

@ctlaltdefeat could you update your fix to the actual master

Is there a way to rebase to head from Github's web interface?

I have never tried it from web, but note that the documentation moved to the code...

@ctlaltdefeat mind rebasing?


Add format method to Date

For consistency with Time and to make Date easier to use can you add a format method to Date.

0.6.0


[BUG]  Issue when i use functions that use R functions

Hello !
I am working on a Causal Learning projectCausal Learning project, I would like to use the CausalDiscoveryToolbox (cdt) package to find the causal graph that fits my problem.
I'm working on Windows on a Python(version: 3.7.6) anaconda notebook. The package installs correctly (version : 0.5.18). But when I use functions in the package that use R functions, I get an error (e.g. ImportError: R Package pcalg is not available). However, I have installed all the necessary R packages on R. Rscipt is well intalls and i can execute some scripts with. So I tried to change the path of the following variable  cdt.SETTINGS.rpath by "C:\Program Files\R\R-3.6.3\bin\Rscript" but it doesn't work.
Thank you in advance for your help!

Even I am facing the same issue. Hoping for help on this! Thanks!

I'm not really using Windows to work on CDT, so this issue may take a bit more time to fix ; what happens if you run this small code snippet ? (Run the python code)
#testlib.R
result = library("pcalg")

#test.py
import subprocess
output = subprocess.call("{} --vanilla {}".format("C:\Program Files\R\R-3.6.3\bin\Rscript", "testlib.R"), shell=True,
                                     stdout=subprocess.DEVNULL, stderr=subprocess.DEVNULL)
print(output)

A quick and easy solution would be to use the Docker images to run your scripts, in the meantime.
I'll try took look into it  !
Best,
Diviyan

Hello, thanks for the response:)
The testlib.R works fine
the test.py gives the output as 1.
Thanks for the advice, I will try to use with a docker image meanwhile!

Hello again,
1 means that the script has errored (0 means no error) ; is your path to the Rscript executable C:\Program Files\R\R-3.6.3\bin\Rscript ? If not, can you run it with the correct path ?
Best regards,

Hello
My path is C:\Program Files\R\R-4.0.2\bin\Rscript
This gives error again.
Is it issue with the R version?

Also very kind of you to take time to look into this :)

Hello again,
No problem :)
I'm trying on a windows platform and encountered small bugs myself, but solved them ;  can you try :
#test.py
import subprocess
output = subprocess.call(r"C:\Program Files\R\R-4.0.2\bin\Rscript --vanilla testlib.R", shell=True)
print(output)

This may sound silly, but can you import "pcalg" correctly from an R console ?
Best,
Diviyan

Hello
When i ran it -
'C:\Program' is not recognized as an internal or external command,
operable program or batch file.
1

Process finished with exit code 0

Yeah I am able to import pcalg from R console without an issue

Hello,
Thank you for your response!
I can import correctly "pcalg" from R console
I tried the code you give and I get 1.

I tried the following code and I get 0.
import subprocess
command = "C:/Program Files/R/R-3.6.3/bin/Rscript"
arg = '--vanilla'
path2script = "C:/Users/UT/Documents/margaux/testlib.R"
output = subprocess.call([command, arg, path2script], shell=True)
print (output)

Hello
This works for my configuration too!

Thank you for your feedback ! The issue comes from the path management that differs from Linux and Windows ; I'll try to patch this asap

Thanks a lot:)

It should be fixed in the dev branch, can you check if everything works on your side ?

Hello
I am running into this issue
Connected to pydev debugger (build 201.7846.77)
Detecting 1 CUDA device(s).
Traceback (most recent call last):
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\utils\R.py", line 207, in launch_R_script
    raise e
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\utils\R.py", line 192, in launch_R_script
    output = subprocess.call([rpath, "--vanilla", scriptpath])
  File "C:\Users\DELL\Anaconda3\lib\subprocess.py", line 339, in call
    with Popen(*popenargs, **kwargs) as p:
  File "C:\Users\DELL\Anaconda3\lib\subprocess.py", line 800, in __init__
    restore_signals, start_new_session)
  File "C:\Users\DELL\Anaconda3\lib\subprocess.py", line 1148, in _execute_child
    args = list2cmdline(args)
  File "C:\Users\DELL\Anaconda3\lib\subprocess.py", line 555, in list2cmdline
    needquote = (" " in arg) or ("\t" in arg) or not arg
TypeError: argument of type 'WindowsPath' is not iterable

when I was running
import networkx as nx
from cdt.causality.graph import PC
from cdt.data import load_dataset
data, graph = load_dataset("sachs")
obj = PC()

Can you try setting manually the rpath before calling PC?
import cdt
import networkx as nx
from cdt.causality.graph import PC
from cdt.data import load_dataset
data, graph = load_dataset("sachs")
cdt.SETTINGS.rpath= r'C:\Program Files\R\R-4.0.2\bin\Rscript'
obj = PC()

It seems to be working that way on my windows machine...

I tried manually setting the path. Same error coming up
Traceback (most recent call last):
  File "C:/PML Code/Causal discovery/CausalDiscoveryToolbox/trial.py", line 7, in <module>
    obj = PC()
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\causality\graph\PC.py", line 175, in __init__
    if not (RPackages.pcalg and RPackages.kpcalg and RPackages.RCIT):
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\utils\R.py", line 132, in __getattribute__
    availability = self.check_R_package(name)
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\utils\R.py", line 147, in check_R_package
    {"{package}": package}, verbose=True))
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\utils\R.py", line 207, in launch_R_script
    raise e
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\utils\R.py", line 192, in launch_R_script
    output = subprocess.call([rpath, "--vanilla", scriptpath])
  File "C:\Users\DELL\Anaconda3\lib\subprocess.py", line 339, in call
    with Popen(*popenargs, **kwargs) as p:
  File "C:\Users\DELL\Anaconda3\lib\subprocess.py", line 800, in __init__
    restore_signals, start_new_session)
  File "C:\Users\DELL\Anaconda3\lib\subprocess.py", line 1148, in _execute_child
    args = list2cmdline(args)
  File "C:\Users\DELL\Anaconda3\lib\subprocess.py", line 555, in list2cmdline
    needquote = (" " in arg) or ("\t" in arg) or not arg
TypeError: argument of type 'WindowsPath' is not iterable

Process finished with exit code 1

I converted the paths to str, can you pull dev and try again ?
I think that this issue is linked to package versions : your version of subprocess doesn't seem to support pathlib.Path objects

Hello
Awesome, thanks a lot, that works!
But running  output = obj.predict(data) throws error

Connected to pydev debugger (build 201.7846.77)
Detecting 1 CUDA device(s).
Loading required package: momentchi2
Loading required package: MASS
Error: '\d' is an unrecognized escape in character string starting "'C:\\Users\\DELL\\AppData\\Local\\Temp\\cdt_pc_4a26313d-831b-402f-a21d-e0f5b3ac137c\d"
Execution halted
Traceback (most recent call last):
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\causality\graph\PC.py", line 286, in _run_pc
    raise e
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\causality\graph\PC.py", line 282, in _run_pc
    self.arguments, output_function=retrieve_result, verbose=verbose)
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\utils\R.py", line 196, in launch_R_script
    raise e
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\utils\R.py", line 190, in launch_R_script
    output = output_function()
  File "C:\PML Code\Causal discovery\CausalDiscoveryToolbox\cdt\causality\graph\PC.py", line 265, in retrieve_result
    return read_csv(Path('{}/result.csv'.format(run_dir)), delimiter=',').values
  File "C:\Users\DELL\Anaconda3\lib\site-packages\pandas\io\parsers.py", line 676, in parser_f
    return _read(filepath_or_buffer, kwds)
  File "C:\Users\DELL\Anaconda3\lib\site-packages\pandas\io\parsers.py", line 448, in _read
    parser = TextFileReader(fp_or_buf, **kwds)
  File "C:\Users\DELL\Anaconda3\lib\site-packages\pandas\io\parsers.py", line 880, in __init__
    self._make_engine(self.engine)
  File "C:\Users\DELL\Anaconda3\lib\site-packages\pandas\io\parsers.py", line 1114, in _make_engine
    self._engine = CParserWrapper(self.f, **self.options)
  File "C:\Users\DELL\Anaconda3\lib\site-packages\pandas\io\parsers.py", line 1891, in __init__
    self._reader = parsers.TextReader(src, **kwds)
  File "pandas\_libs\parsers.pyx", line 374, in pandas._libs.parsers.TextReader.__cinit__
  File "pandas\_libs\parsers.pyx", line 674, in pandas._libs.parsers.TextReader._setup_parser_source
FileNotFoundError: [Errno 2] File C:\Users\DELL\AppData\Local\Temp\cdt_pc_4a26313d-831b-402f-a21d-e0f5b3ac137c\result.csv does not exist: 'C:\\Users\\DELL\\AppData\\Local\\Temp\\cdt_pc_4a26313d-831b-402f-a21d-e0f5b3ac137c\\result.csv'

Process finished with exit code 1

Hello !
I tried the following code but I still have the error : ImportError: R Package (k)pcalg/RCIT is not available. RCIT has to be installed from https://github.com/Diviyan-Kalainathan/RCIT.
import cdt
import networkx as nx
from cdt.causality.graph import PC
from cdt.data import load_dataset
data, graph = load_dataset("sachs")
cdt.SETTINGS.rpath= r"C:/Program Files/R/R-3.6.3/bin/Rscript"
obj = PC()
I used pip install cdt --upgrade to have the changement of the code that you made but I think it doesn't work.

Have you installed RCIT from https://github.com/Diviyan-Kalainathan/RCIT.?? @Margaux-Chevallier

@Margaux-Chevallier You need to install those two packages :
from R:
install.packages('kpcalg')
install.packages('devtools')
library('devtools')
install_github("Diviyan-Kalainathan/RCIT")

@nishtha82 Strange, I'll look into it

Hello again,
Awesome, looking forward to that!
Thanks much for all the help so far:)

I checked, the package RCIT and kpcalg are well install on R.

@Margaux-Chevallier Using pip install cdt --upgrade doen't work, only master is pushed to pypi.
One thing you could do is :
pip uninstall cdt
pip install git+https://github.com/FenTechSolutions/CausalDiscoveryToolbox.git@1dcd97a34e73cfaf7e6820ab06760f7d8dded4b2

1dcd97a34e73cfaf7e6820ab06760f7d8dded4b2 refers to the latest commit as of now.

Could you try again ?
pip install git+https://github.com/FenTechSolutions/CausalDiscoveryToolbox.git@1e553f49b4adb7ccc19f7d8fc473ec875b554ab4

Thanks !
I run this :
pip uninstall cdt --yes
pip install git+https://github.com/FenTechSolutions/CausalDiscoveryToolbox.git@1e553f49b4adb7ccc19f7d8fc473ec875b554ab4
import cdt
import networkx as nx
from cdt.causality.graph import PC
from cdt.data import load_dataset
data, graph = load_dataset("sachs")
cdt.SETTINGS.rpath= r'C:/Program Files/R/R-3.6.3/bin/Rscript'
obj = PC()
And I still have the same issue.

Sorry, I didn't restart the kernel...
It's working now !!! Thanks a lot for your help and your reactivity !

@Diviyan-Kalainathan  I found this issue https://github.com/FenTechSolutions/CausalDiscoveryToolbox/issues/50 which is similar to mine, but couldn't understand the workaround suggested?

I thik I have a similar issue when I use the CAM method.
import cdt
import networkx as nx
from cdt.causality.graph import CAM
from cdt.data import load_dataset
data, graph = load_dataset("sachs")
cdt.SETTINGS.rpath= r'C:/Program Files/R/R-3.6.3/bin/Rscript'
obj = CAM()
output = obj.predict(data)
Running the last line give the following error :
FileNotFoundError: [Errno 2] File C:\Users\UT\AppData\Local\Temp\cdt_cam_67b55cc5-05a5-4373-8650-44c9f2e4206b\result.csv does not exist: 'C:\Users\UT\AppData\Local\Temp\cdt_cam_67b55cc5-05a5-4373-8650-44c9f2e4206b\result.csv'
The detailed error is :
FileNotFoundError                         Traceback (most recent call last)
 in
6 cdt.SETTINGS.rpath= r'C:/Program Files/R/R-3.6.3/bin/Rscript'
7 obj = CAM()
----> 8 output = obj.predict(data)
~\anaconda3\lib\site-packages\cdt\causality\graph\model.py in predict(self, df_data, graph, **kwargs)
61         """
62         if graph is None:
---> 63             return self.create_graph_from_data(df_data, **kwargs)
64         elif isinstance(graph, nx.DiGraph):
65             return self.orient_directed_graph(df_data, graph, **kwargs)
~\anaconda3\lib\site-packages\cdt\causality\graph\CAM.py in create_graph_from_data(self, data, **kwargs)
179         self.arguments['{NJOBS}'] = str(self.njobs)
180         self.arguments['{VERBOSE}'] = str(self.verbose).upper()
--> 181         results = self._run_cam(data, verbose=self.verbose)
182
183         return nx.relabel_nodes(nx.DiGraph(results),
~\anaconda3\lib\site-packages\cdt\causality\graph\CAM.py in _run_cam(self, data, fixedGaps, verbose)
201         except Exception as e:
202             rmtree(run_dir)
--> 203             raise e
204         except KeyboardInterrupt:
205             rmtree(run_dir)
~\anaconda3\lib\site-packages\cdt\causality\graph\CAM.py in _run_cam(self, data, fixedGaps, verbose)
197             data.to_csv(Path('{}/data.csv'.format(run_dir)), header=False, index=False)
198             cam_result = launch_R_script(Path("{}/R_templates/cam.R".format(os.path.dirname(os.path.realpath(file)))),
--> 199                                          self.arguments, output_function=retrieve_result, verbose=verbose)
200         # Cleanup
201         except Exception as e:
~\anaconda3\lib\site-packages\cdt\utils\R.py in launch_R_script(template, arguments, output_function, verbose, debug)
207         if not debug:
208             rmtree(base_dir)
--> 209         raise e
210     except KeyboardInterrupt:
211         if not debug:
~\anaconda3\lib\site-packages\cdt\utils\R.py in launch_R_script(template, arguments, output_function, verbose, debug)
201                                            stderr=subprocess.DEVNULL)
202             process.wait()
--> 203             output = output_function()
204
205     # Cleaning up
~\anaconda3\lib\site-packages\cdt\causality\graph\CAM.py in retrieve_result()
192
193         def retrieve_result():
--> 194             return read_csv(Path('{}/result.csv'.format(run_dir)), delimiter=',').values
195
196         try:
~\anaconda3\lib\site-packages\pandas\io\parsers.py in parser_f(filepath_or_buffer, sep, delimiter, header, names, index_col, usecols, squeeze, prefix, mangle_dupe_cols, dtype, engine, converters, true_values, false_values, skipinitialspace, skiprows, skipfooter, nrows, na_values, keep_default_na, na_filter, verbose, skip_blank_lines, parse_dates, infer_datetime_format, keep_date_col, date_parser, dayfirst, cache_dates, iterator, chunksize, compression, thousands, decimal, lineterminator, quotechar, quoting, doublequote, escapechar, comment, encoding, dialect, error_bad_lines, warn_bad_lines, delim_whitespace, low_memory, memory_map, float_precision)
674         )
675
--> 676         return _read(filepath_or_buffer, kwds)
677
678     parser_f.name = name
~\anaconda3\lib\site-packages\pandas\io\parsers.py in _read(filepath_or_buffer, kwds)
446
447     # Create the parser.
--> 448     parser = TextFileReader(fp_or_buf, **kwds)
449
450     if chunksize or iterator:
~\anaconda3\lib\site-packages\pandas\io\parsers.py in init(self, f, engine, **kwds)
878             self.options["has_index_names"] = kwds["has_index_names"]
879
--> 880         self._make_engine(self.engine)
881
882     def close(self):
~\anaconda3\lib\site-packages\pandas\io\parsers.py in _make_engine(self, engine)
1112     def _make_engine(self, engine="c"):
1113         if engine == "c":
-> 1114             self._engine = CParserWrapper(self.f, **self.options)
1115         else:
1116             if engine == "python":
~\anaconda3\lib\site-packages\pandas\io\parsers.py in init(self, src, **kwds)
1889         kwds["usecols"] = self.usecols
1890
-> 1891         self._reader = parsers.TextReader(src, **kwds)
1892         self.unnamed_cols = self._reader.unnamed_cols
1893
pandas_libs\parsers.pyx in pandas._libs.parsers.TextReader.cinit()
pandas_libs\parsers.pyx in pandas._libs.parsers.TextReader._setup_parser_source()
FileNotFoundError: [Errno 2] File C:\Users\UT\AppData\Local\Temp\cdt_cam_67b55cc5-05a5-4373-8650-44c9f2e4206b\result.csv does not exist: 'C:\Users\UT\AppData\Local\Temp\cdt_cam_67b55cc5-05a5-4373-8650-44c9f2e4206b\result.csv'

Hello,
@Margaux-Chevallier CAM should run correctly now, please take the lastest commit on dev
@nishtha82 I've improved the error message when the R process errors ; can you try again ?
Best,

@Diviyan-Kalainathan  Hello, I ran the latest!!
It works, this is awesome!
Thanks:)
My output was
Detecting 1 CUDA device(s).
C:\Program Files\R\R-4.0.2\bin\Rscript C:\Users\DELL\AppData\Local\Temp\cdt_R_script_28ac4331-5282-4bd2-8cb7-9f9a5741bfbb\instance_test_import.R
C:\Program Files\R\R-4.0.2\bin\Rscript C:\Users\DELL\AppData\Local\Temp\cdt_R_script_793e69d0-aa1c-4f07-acfe-4545efb9fbd5\instance_test_import.R
C:\Program Files\R\R-4.0.2\bin\Rscript C:\Users\DELL\AppData\Local\Temp\cdt_R_script_b2996e20-95c7-47ad-9dad-b0f2b220e9b9\instance_test_import.R

and the graph plot when I ran
import cdt
import networkx as nx
from cdt.causality.graph import PC
from cdt.data import load_dataset
import matplotlib.pyplot as plt
data, graph = load_dataset("sachs")
cdt.SETTINGS.rpath = r'C:\Program Files\R\R-4.0.2\bin\Rscript'
obj = PC()
output = obj.predict(data)
nx.draw_networkx(output, font_size=8)
plt.show()

Hello,
Great ! I may have left a print somewhere, i'll wait for @Margaux-Chevallier and push to master to release the next version
Best,
Diviyan

It's working for me too !
Thanks !

Fantastic, I'll be closing this issue !

HI,
I'm using google colab and getting an error listed below.
ImportError: R Package (k)pcalg/RCIT is not available. RCIT has to be installed from https://github.com/Diviyan-Kalainathan/RCIT
I installed RCIT from github in the previous cell using
%%R
install_github("Diviyan-Kalainathan/RCIT")
--Thank You


Fix to copy schema loader image via bastion

Description
https://scalar-labs.atlassian.net/browse/DLT-8735
https://scalar-labs.atlassian.net/browse/DLT-8757
Errors occur when running taint on a different machine than the one used to build the environment.

Replace node

terraformf taint "module.scalardl.module.scalardl_blue.module.cluster.azurerm_virtual_machine.vm-linux[0]"
terraformf taint "module.scalardl.module.scalardl_blue.module.scalardl_cluster.aws_instance.this[0]"

Error

module.scalardl.module.scalardl_blue.module.scalardl_provision.null_resource.schema_loader_image_load[0] (remote-exec): gzip: /tmp/scalardl-schema-loader-1.3.0.tar.gz: No such file

📝  After merge this PR, we don't need #306
Done

Fix to copy schema loader image via bastion as well as scalardl image.
Add trigger for recreate schema loader image.

@scalar-boney Please check if it works.

@scalar-boney Please check if it works.

Yes, It is working fine.


Ubuntu64 terminal calls

README.md

Updated to have version 2.6.1 of protobuf.

Removed a cd .. that wasn't needed.

Also added instructions for how to get protobuf in IDA on 64 bit Linux. Apparently IDA will not use the system's Python (even after install) on Linux. You can add the modules manually to the zip file though.

disass.py

Added parent dir to sys.path since those imports were failing for me.

 Support code is BBCache (per function) and a 5 function cache of those. Updated is_call to utilize the inst_terminates_bb function.

Added a JUMP_TABLE_EAS set used to filter out addresses within jump tables.

x86.py

Added loop mnemonics.

@douggard, can you fix the CI failures for Linux?

Also, please add your name to the acknowledgements file as part of this PR ;-)

Passed.


UISideMenuNavigationControllerDelegate not work

public func sideMenuDidDisappear(menu: UISideMenuNavigationController, animated: Bool)
not call on ios versiyon 13.2

did you got the answer, i have the same issue in SideMenuNavigationControllerDelegate.

yes !! this is my solution
leftMenuNavigationController.modalPresentationStyle = .overCurrentContext


nixos: Generalise the container tests in stage-2 boot

Motivation for this change
This way, stage-2 behaves correctly also for libvirt-lxc containers.
Some more discussion on this:
https://github.com/NixOS/nixpkgs/commit/a7a08188bf650ababa36300a9a6f34169e2a73bf
https://github.com/NixOS/nixpkgs/commit/bfe46a653ba2f8ff9902128f485cbd87c49cbca7
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Should this be cherry-picked to 16.09?

@shlevy Yeah, it should be in 16.09 too. Thanks!


Remove default signature provider from exports

Some eosjs use cases leverage an alternate signature provider. The default signature provider is the only piece of eosjs that uses eosjs-ecc, and eosjs-ecc is not a light-weight package. By removing the default signature provider from the index.ts exports, developers can avoid having the heavy-weight ecc package bundled.
This also means that environments like React Native, which are not compatible with eosjs-ecc, will not break when using eosjs with alternate signature providers.


Revise or obsolete seeming duplicates for Haemophilus meningitis

haemophilus meningitis (MONDO:0000889) definition: Infections of the nervous system caused by bacteria of the genus haemophilus, and marked by prominent inflammation of the meninges. haemophilus influenzae type B is the most common causative organism. The condition primarily affects children under 6 years of age but may occur in adults
AKA: Haemophilus influenzae bacterial meningitis, Haemophilus influenzae caused bacterial meningitis
haemophilus influenzae meningitis (MONDO:0006775) definition: Infections of the nervous system caused by bacteria of the genus HAEMOPHILUS, and marked by prominent inflammation of the MENINGES. HAEMOPHILUS INFLUENZAE TYPE B is the most common causative organism. The condition primarily affects children under 6 years of age but may occur in adults.
The SNOMEDCT IDs for the two are different, and it looks like MONDO:0000889 gained additional specificity/mentions to haemophilus influenzae, which the corresponding SNOMEDCT item lacks. Recommend either revising MONDO:0000889 to not mention haemophilus influenzae in the definition and AKA and reference MONDO:0006775 as a child of it, OR to merge and obsolete one of them.

These should be merged, but do we need a parent concept for infection-site neutral haemophilus?

we do have MONDO_0006926 'haemophilus infectious disease', and MONDO_0000889 'haemophilus meningitis' is inferred to be a child


Rely Webpage Not loading

Hello Mr. Baldeosinghm,
I was super interested in your project! The concept is amazing! However, I was not able to reach your application using the following url: http://<IP_ADDRESS>/
Could you please take a look at the issue?
Thank you!
Spencer Huang

How am I now seeing this and did you really expect that to work haha.  An inventive way of reaching out my man!


Enable adding repo with proxy

Sometimes the Kubernetes master cannot access the repo tiller but may need a proxy to access, it is better to enable helm can add repo with proxy.

The CLI already support this.


update default server to use production API service

Use api.turnkey.com instead of the old beta service.

D'oh.  That file is autogenerated...


Adding an example for a 3rd party API service that requires basic authentication

While an example of a local API server is good, it will be more helpful to add an example for a 3rd party API service that requires authentication (at least basic login/password).

@mitchell-ua sure! There are soooo many to choose from of course.
Do you have a specific kind of authentication in mind?
I know that OpenAI supports

static tokens as either Bearer Auth Tokens or as Custom Headers
A version of OAuth2 Client Flow

If we flush out a little more detail I can plug it into cursor and implement something

@snaggle-ai Thanks for getting back! My API service supports OAuth2-compatible tokens which are used as Authorization: Bearer. See https://www.oauth.com/oauth2-servers/access-tokens/

No worries. @mitchell-ua how do you get an access token? Is it via an Authorization Code flow ?
I think that supporting auth code flow would mean:

configuring your auth server with aa redirect to localhost for
providing the mcp server with a client secret as part of setup (could go into the claude developer json but I think it's probably better to put it on the file system somewhere like ~/.openapi-mcp-server and equivalent for Windows)
having the mcp server listen on localhost for a redirect from your auth endpoint

There are other options with some auth servers though

@mitchell-ua OK nice and simple!
Would you want to have the user enter the username + password into Claude? Or would you want to put them in a config file or into the claude config json as part of the startup command for the mcp server?

@snaggle-ai I think adding env with credentials to claude_desktop_config.json is a way to go.

Great, just to confirm the interface, something like this:
{
  "mcpServers": {
    "github": {
      "command": "npx",
      "args": ["-y", "openapi-mcp-server", "http://yourdomain.com/openapi.json"],
      "env": {
        "OAUTH_USERNAME": "your_token_here"
        "OAUTH_PASSWORD": "your_token_here"
      }
    }
  }
}

Not sure what the key -y does in this case, but I would say:
{
  "mcpServers": {
    "third-party-api": {
      "command": "npx",
      "args": ["openapi-mcp-server", "http://yourdomain.com/openapi.json"],
      "env": {
        "OAUTH_USERNAME": "your_username_here",
        "OAUTH_PASSWORD": "your_password_here"
      }
    }
  }
}

the -y automatically approves any npx prompts that might arise - the official @modelcontextprotocol node examples all use it, I have an update to the README for this project to push to do the same

So, I've investigated this a bit more and the tricky part is that there's no consistent standard structure of request/response that the token/authorization endpoints have to stick to for their request or response payload.
for example some password flow endpoints use http basic auth (no request body at all) and return access token as text, some need a POST request with form-data fields like username and password, some (the one that you shared @mitchell-ua) need a POST request with json data and the field names vary - the one you shared wants a POST of { login:, password: } but others might use { username:, password: } or {accountId:, key: } or basically it could be anything.
if we could involve the LLM in the login flow a little it could probably navigate this fairly straightforwardly, although it might be a bit of a PITA user experience.
Alternatively I could encode a bit of a trial-and-error/shotgun approach where the server attempts a load of different approaches and sees if one succeeds, but that will make error reporting very difficult/ confusing.

@snaggle-ai Have you gotten any further with this? Seems like involving the LLM would take away the flexibility quite a bit. Probably putting this in the config file would be the best and  depending on the type of authorization, the server manages it accordingly?
For example, one config could look like:
{
  "mcpServers": {
    "mydomainapi": {
      "command": "npx",
      "args": ["-y",
         "openapi-mcp-server",
        "http://yourdomain.com/openapi.json",
        "--username", "<USERNAME>"
        "--password", "<PASSWORD>"
        "--authtype", "OAUTH"
        "],
    }
  }
}

and a different one one
{
  "mcpServers": {
    "mydomainapi": {
      "command": "npx",
      "args": ["-y",
         "openapi-mcp-server",
        "http://yourdomain.com/openapi.json",
        "--appkey", "key"
        "--authtype", "HTTP"
        "],
    }
  }
}

what do you think?

thanks @jscastanoc - I think you're right that this just needs to be user configurable. It needs to be something that can be quite exressive to support a range of cases. I think it has to be namespaced a bit or support some kind of templating.
We have to allow specifying pretty much everything that could go into a fetch call:
url (domain, path, query string)
http method
headers
body
format (json, formdata, xml)
with templating I could skip auto format and just require the provided template is correct for the provided headers.
So I'm thinking of making the open api spec and some arguments available for templating  like

securityScheme..tokenUrl
servers[i].url (from the open api server)
args.[name]

example template file using mustache templates ({{)
{
  url: "{{securityScheme.<name>.tokenUrl}}",
  method: POST,
  headers: {
    "Content-Type": "application/json"
  },
  body: "{ username: "{{ args.username }}", password: "{{ args.password }}" }"
}

or for form encoded you could do:
{
  url: "{{securityScheme.<name>.tokenUrl}}",
  method: POST,
  headers: {
    "Content-Type": "application/x-www-form-urlencoded"
  },
  body: "login={{ args.username }}&key={{ args.key }}"
}

Hello, it seems to be difficult to handle all authentification shemes. An alternative could be to provide some configuration to add custom headers (like Authorization/Cookies).
I don't think that authentification must be done by the MCP server.


Kustomize installation does not work

The kustomize installation described on https://camel.apache.org/camel-k/1.12.x/installation/advanced/kustomize.html does not work
➜  install git:(main) ✗ kubectl apply -k setup-cluster
error: accumulating resources: accumulation err='accumulating resources from '../config/rbac/user-cluster-role.yaml': security; file '/home/jbouska/git/camel-k/install/config/rbac/user-cluster-role.yaml' is not in or below '/home/jbouska/git/camel-k/install/setup-cluster'': must build at directory: '/home/jbouska/git/camel-k/install/config/rbac/user-cluster-role.yaml': file is not directory

➜  install git:(main) ✗ kustomize build setup-cluster
Error: accumulating resources: accumulation err='accumulating resources from '../config/rbac/user-cluster-role.yaml': security; file '/home/jbouska/git/camel-k/install/config/rbac/user-cluster-role.yaml' is not in or below '/home/jbouska/git/camel-k/install/setup-cluster'': got file 'user-cluster-role.yaml', but '/home/jbouska/git/camel-k/install/config/rbac/user-cluster-role.yaml' must be a directory to be a root

Similar as #4135

cc @gansheer


Add a check for empty teams and delete them.

Description
Cause we don't want empty docs with no user to access them.
Issues

I already did that?


Merging bigbang branch into master

Rebooting the project, now with code more close to what we expect to be the real version of this project. Not just a experimentation anymore.

Great work guys. Thanks to everyone involved.


[3.0.11] a little confusion here with the version

If I download either the released zip or js file for 3.0.11, I get a file that claims to be 3.0.10, but has changes against 3.0.10's pixi.js.  If I run gulp build inside the unpacked 3.0.11 zip, I get a somewhat different pixi.js file in the bin directory, which also claims to be 3.0.11.
I'm trying to package 3.0.11 for cljsjs/packages.  I don't think I can run commands like npm install gulp in the boot script that prepares the code for use in ClojureScript.  What should I do?

Hi there! This should be sorted now we have pushed to the master branch 👍


[$500] Android - Line height is increased after pasting code block in composer.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open any report
Copy a code block
Paste in the composer
Observe the text

Expected Result:
The line height should not change.
Actual Result:
The line height is changed.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/4384b6f0-ca9e-4b7a-a5da-97a5d8059494
https://github.com/Expensify/App/assets/93399543/2d6403fe-fc91-415b-a038-c3fa7688102a
Expensify/Expensify Issue URL:
Issue reported by: @krishna2323
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693935913441749
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0127b1c2305c88d4bf
Upwork Job ID:<PHONE_NUMBER>599029760
Last Price Increase: 2023-09-10

This issue is basically in the porps of RNTextInput. The prop textAlightVertical should contain some other value than "center:

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~012eb74f01f4de7f7f

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01bd42ba50c9112a7e

I don't see any bug here. The text adjusts ever so slightly when you paste it in, but to me it just looks like it is loading/rendering, and it's barely perceptible. Closing as this isn't a bug.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01c9910144b0a19a02


chore(deps): update devdependency @types/node to ^20.16.10

This PR contains the following updates:

Package
Type
Update
Change

@types/node (source)
devDependencies
patch
^20.16.9 -> ^20.16.10

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Hywax Assistant.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (cbf6ced) to head (85545ef).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #72   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          146       146
  Branches        26        26
=========================================
  Hits           146       146

Flag
Coverage Δ

100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


[Issue]

Hello, I am having an issue with (deez nuts).
Errors: (sucking).
I am running Windows version (69) and my specs are (i140 140 at 450mz)
I compiled with the build configuration (10) and am using the IDE (1)
(extra information)
In order to reproduce this problem you must, (suck deez nuts).

+1


paths options not working in jsconfig.json

VSCode Version: 1.19.2
OS Version: macOS High Sierra 10.13.2

I have a file structure of this way
shared
|- some-file
|- some-folder
src1
|- jsconfig.js
|- some-other-file
|- some-other-folder

my jsconfig.json is
{
  "compilerOptions": {
    "baseUrl": "./",
    "paths": {
      "*": ["*", "../shared/*"]
    }
  }
}

However, I only get suggestions from the baseUrl which is src1, but none from shared folder

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

Hey @stanleynguyen. I was simply providing another example in which jsconfig.json wasn't working.
Although, I tried setting up an example similar to yours and I was surprised to find your example didn't work (as the documentation seems to indicate it should). However, I experimented a bit and found that changing the baseUrl to go back a directory fixed the problem.
Try doing this instead:
{
  "compilerOptions": {
    "baseUrl": "../",
    "paths": {
      "*": ["*", "./shared/*"]
    },
  },
  "exclude": [
    "node_modules"
  ]
}

@Munksey still doesn't work for me though :( However, looking at this, I think the problem might be with the go back directory in the paths field.

That was one of things I changed. I made it so ../shared/* becomes ./shared/*.  And the value for baseUrl becomes ../.
What's interesting is that you actually made me realize that I can put jsconfig.json in a subdirectory and I was able to come up with a solution for my odd webpack/javascript/typescript project. So my problem is kind of irrelevant now.
If making the two changes I listed above doesn't help, maybe you can try a plugin called "Path Intellisense" (https://marketplace.visualstudio.com/items?itemName=christian-kohler.path-intellisense). There's a field called path-intellisense.mappings that you can use for custom resolver paths.

Thanks @Munksey  for the suggestion. However, the most specific I can go for the plugins is by workspace but I would need different mappings for different sub-folders so this doesn't help either :(

Any advice? @mjbvz

@Munksey I think that if there aren't some special configurations of which the JS files need you can have only the tsconfig.json at the root with "allowJS": true and "checkJS": true

I think it might actually be in the documentation, but I hadn't seen it spelled out, and just realised today that the paths key is just using glob pattern matching.  That means that the "paths" key will only match modules at one level of depth, unless you use a pattern like the following:
  "baseUrl": "src",
  "paths": {
      "**/*": ["**/*"]
    },

Resolved in the newest update of VSCode 👍


Please document your changes properly

"scripts": {
"start": "NODE_ENV=production node index.js",
"dev": "NODE_ENV=development nodemon index.js"
} works on linux not windows,
"scripts": {
"start": "set NODE_ENV=production && node index.js",
"dev": "set NODE_ENV=development && nodemon index.js"
}, is the workaround on windows
Should have specified that dependencies should have been reinstalled, Ran into dotenv and prisma-cli not installed errors, before i realized they were new dependencies
explain the creation of the environment variables better (.env file and any other related file) better, currently poorly explained

Yeah if you notice changes in package.json you should always rerun yarn or yarn install


parameter "force" does not work

Hi,
when using the paramter "force" (this way:
"force":true
)
I would expect this existing data in METS will be overwritten.
But I get an error like this
" Output fileGrp[@USE='OCR-D-DEWARP'] already in METS!"
Could you please check this?

Workaround:
ocrd workspace remove-group -f OCR-D-DEWARP

Workaround:
ocrd workspace remove-group -f OCR-D-DEWARP

Together with -r (recursiv) this workaround works fine.
Many thanks.


Cache-control HTTP response headers

ALISE provides information that describes identity mapping.  Such information is very unlikely to change.  The typical scenario is that the user links their local- and federated identities (as identified by usernames, 'sub' claims, etc) once.  After this, the user's identities (local and federated) will likely never change.  There is, perhaps, the likelihood that new identities are introduced, but that (too) is likely a rare event.
Caching is a widely used tool to reduce latency, reduce load on the server (ALISE) and reduce the impact (on dependent services) of a temporary outage (e.g., restart during an admin intervention).  The information provided by ALISE would seem an ideal candidate to support some form of caching.
RFC 7234 provides a standard mechanisms that allows a server to indicate whether a response should be cached and for how long.
It would be helpful if ALISE were to use these mechanisms to indicate that information should be cached.

Ok; I'll investigate. Also to see if I can inform the client for how long Alise has been caching the linkage (i.e. that may have happened years ago, and the user is gone at one of the home-OPs

Header would currently be:
< cache-control: public
< max-age: 31536000
< x-alise-version: 1.0.5

Anything missing from your perspective?


Dynamic import of small chunks with System.import

As I see actually System.import doesn't work as kind of dynamic one. Say I have I directory:
./templates/
   templateA.js
   templateB.js
   templateC.js
    ...
main.js

I want to have a function that dynamically imports any of this template files by name:
so in main.js I have a function :
function loadTemplate(templateNmae) {
  var templatePath = './templates/' + templateName
  return System.import(templatePath)
}

But I doesn't work webpack says: can not resolve ./templates/templateA
If to do this way:
function loadTemplate(templateNmae) {
  return System.import('./templates/' + templateName)
}

Webpack just break with an error:

ERROR in chunk main [entry]
build.js
Cannot read property 'length' of null
webpack: bundle is now VALID.
But I think I just don't understand how System.import supposed to work in webpack and I wonder if what I want is achievable currently.

I don't know if i am right, but I think webpack does static analysis, so it does not know how to resolve the functions before returning import paths.

Ok I got now, that webpack is about static bundles.

This doc may help: https://webpack.github.io/docs/context.html

@iotch
Yes right thanks I implement it via requiring the whole dir.


Dark mode update

I have updated the dark mode please merge it.

@psykat1116 please share a preview of the changes made.


CDC Tubing Dimensions

I am trying to calculate the head loss through the dose controller that is not associated with the coagulant dosing tubes, namely in the pipe from the constant head tank to the dosing tubes and the flexible tubing from the other end of the dosing tubes to the LFOM.  Where would I be able to find the dimensions currently used for this tubing (diameter and length)?
I am including an image of the tubing I am referring to below:

I checked the technical specifications document and I can't find any specifications. I think that we haven't created the algorithms for this design. So I think that becomes part of your project!


Updated eslint package, migrated to the new eslint config system

In the eslint 9.0.0 version was introduced new configuration system and a lot of breaking changes

Updated eslint package to 9.7.0 and migrated from a deprecated config system to a new one using this Configuration Migration Guide. Old config files .eslintrc.js, .eslintignore were deleted and created eslint.config.mjs instead of them
Added new dev dependencies which are used in the new config file: @eslint/compat, @eslint/eslintrc, @eslint/js, globals
Updated eslint plugins for compatibility with the new eslint version: eslint-config-prettier, eslint-plugin-prettier, eslint-plugin-react, eslint-plugin-react-hooks, eslint-plugin-storybook, eslint-plugin-testing-library, eslint-plugin-vitest-globals

Do we need to think about ignoring of eslint.config.mjs?


K.batch_dot() doc example fails w/ Tensorflow backend

From the batch_dot section of the Backend docs (https://keras.io/backend/):

Shape inference: Let x's shape be (100, 20) and y's shape be (100, 30, 20). If axes is (1, 2), to find the output shape of resultant tensor, loop through each dimension in x's shape and y's shape:
x.shape[0] : 100 : append to output shape
x.shape[1] : 20 : do not append to output shape, dimension 1 of x has been summed over. (dot_axes[0] = 1)
y.shape[0] : 100 : do not append to output shape, always ignore first dimension of y
y.shape[1] : 30 : append to output shape
y.shape[2] : 20 : do not append to output shape, dimension 2 of y has been summed over. (dot_axes[1] = 2) output_shape = (100, 30)

Actually running this (see below code) produces errors when using the newest Tensorflow.  Seems to be ok with Theano.
Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[x] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

from keras import backend as K

x = K.ones(shape=(100,20))
y = K.ones(shape=(100,30,20))
xy = K.batch_dot(x,y,axes=(1,2))

Fails with...
Using TensorFlow backend.
Traceback (most recent call last):
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/framework/common_shapes.py", line 671, in _call_cpp_shape_fn_impl
input_tensors_as_shapes, status)
File "/opt/anaconda/envs/py35/lib/python3.5/contextlib.py", line 66, in exit
next(self.gen)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/framework/errors_impl.py", line 466, in raise_exception_on_not_ok_status
pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.InvalidArgumentError: Shape must be rank 2 but is rank 3 for 'MatMul' (op: 'MatMul') with input shapes: [100,20], [100,30,20].
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "batch_dot_test.py", line 6, in
xy = K.batch_dot(x,y,axes=(1,2))
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/keras/backend/tensorflow_backend.py", line 917, in batch_dot
out = tf.matmul(x, y, adjoint_a=adj_x, adjoint_b=adj_y)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/ops/math_ops.py", line 1801, in matmul
a, b, transpose_a=transpose_a, transpose_b=transpose_b, name=name)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/ops/gen_math_ops.py", line 1263, in _mat_mul
transpose_b=transpose_b, name=name)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/framework/op_def_library.py", line 768, in apply_op
op_def=op_def)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 2338, in create_op
set_shapes_for_outputs(ret)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 1719, in set_shapes_for_outputs
shapes = shape_func(op)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 1669, in call_with_requiring
return call_cpp_shape_fn(op, require_shape_fn=True)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/framework/common_shapes.py", line 610, in call_cpp_shape_fn
debug_python_shape_fn, require_shape_fn)
File "/opt/anaconda/envs/py35/lib/python3.5/site-packages/tensorflow/python/framework/common_shapes.py", line 676, in _call_cpp_shape_fn_impl
raise ValueError(err.message)
ValueError: Shape must be rank 2 but is rank 3 for 'MatMul' (op: 'MatMul') with input shapes: [100,20], [100,30,20].
When using Theano, no output appears at all, indicating success.

just passing x = K.ones(shape=(100,20,1)) resolves this issue.
From keras.backend.tensorflow_backend.py

x_batch = K.ones(shape=(32, 20, 1))
y_batch = K.ones(shape=(32, 30, 20))
xy_batch_dot = K.batch_dot(x_batch, y_batch, axes=[1, 2])
K.int_shape(xy_batch_dot)
(32, 1, 30)

True, that is a viable workaround, however that is not what the documentation says -- as it stands, there's a contradiction.  And it's not what happens with the Theano backend.  Plus your result produces the wrong number of dimensions; it would need a call to reshape() at the end.

Can someone please investigate what a solution would look like?

To me, it is not a bug in code though but somehow inconsistency in the documents. The code example in the official site https://keras.io/backend/, which only appears in the docstring in tensorflow backend, actually implies the fact that x and y should be of the same rank.
Theano backend uses T.batched_tensordot to implement K.batch_dot which happens to help you expand the missing dim if applicable. While in the tensorflow backend, I don't think this is ever supported, no matter using tf.batch_matmul or latest tf.matmul. Specifically, see tf.matmul,
b: Tensor with same type and rank as a.
Solution:
It is possible to manually fix this, possibly add as below to (https://github.com/fchollet/keras/blob/master/keras/backend/tensorflow_backend.py#L911)
        if ndim(x) == 2:
            x = tf.expand_dims(x, axis=ndim(x) if axes[0] == ndim(x) - 1 else ndim(x))
        if ndim(y) == 2:
            y = tf.expand_dims(y, axis=ndim(y) if axes[1] == ndim(y) - 1 else ndim(y))

        if axes is not None:
            adj_x = None if axes[0] == ndim(x) - 1 else True
        ....

Use ndim(x)-1 not hard code1 or 2 because tf.matmul now supports the input to have more than 3 ranks. Or would it be better we raise an error to both backends to notify that the inputs x and y should be of the same rank? Make life easier.
if ndim(x) != ndim(y):
     raise ValueError("Inputs should have same ranks but are " + str(ndim(x)) + " and " + str(ndim(y)) + ".")

#6219


Wasm file size

I am not an expert here but this lib size is about 158Kb (wasm file the browser downloads).
Is there any way this can be optimized? 158Kb can be quite a lot even now-a-days.

Perhaps having specific wasm files per feature.
I'm using pako because I'm able to require only the gzip portion of it, then my final bundle right now is about 30k after webpack.
That's because the work I'm doing I need only to gzip content, I don't need to deflate, inflate or, gunzip;
Having those features within the file would be a big penalty for an old android with a 3G connection across the globe to download it.

@danielsan  that is exactly my use case. Do you have a package for this?
I don't think it's about the 30Kb / 150Kb but those do stack up eventually. I mean Agular can be around 70kb get the point here?

That's exactly what I meant while using only the gzip feature from pako.
My whole deployable script is about 30k with all the dependencies, but if include the whole pako lib with all its functionalities it goes to 47k in my case.
I don't know yet how to export a .wasm file with only the gzip feature but that is my idea

@danielsan oh ok got it. I thought you did the same for the wasm file.
I think for wasm (without being an expert) it depends to towards what compatibility its targeted, not the code per say but I can also be total.y wrong.

pako minification is quite amazing. I've used jsziptools previously and while the unminified source code is similar (~250kB) the minified pako (I'm using just inflate) is ~60kB while jsziptools is ~190kB. Cray cray.