Devise a means for testing on real hardware

Right now for the pc_x86_64 platform I'm using an MSI B85M board and I'm flashing a usb stick for each test, we need something more automated. It should eventually support testing from the cloud (CircleCi) but in the short term maybe PXE booting could be used.
Eventually we'll probably need a testing server which controls other test devices. The test server will probably need to be connected to a test devices reset and power switches so as tests can be totally controlled something like this. The most common option are boxes that are remote power switches, here is a google search.

Here is what osdev.org has to say and here is pixiecore which might be helpful.


dont remake aggregated graphs every attempt

while short-term this seems unproblematic, long term that could cause to history loss (e.g., ix retires/deletes a port which is now no longer considered in the aggregation)

Codecov Report

Merging #177 (63c0b98) into prep-release (8cd6186) will increase coverage by 0.00%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@              Coverage Diff              @@
##           prep-release     #177   +/-   ##
=============================================
  Coverage         48.02%   48.03%
=============================================
  Files               140      140
  Lines              5120     5119    -1
=============================================
  Hits               2459     2459
+ Misses             2661     2660    -1

Files Changed
Coverage Δ

src/fullctl/graph/mrtg/rrd.py
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Docs] "satrt" should be "start"

Current State
typo in documentation : mesheryctl system satrt
Desired State
should be mesheryctl system start
Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation
Layer5 Community Slack

I am making PR :)


.env.example file does not exist

The file is not present in the repository as an example to be renamed

Thanks for pointing out. It will be fixed soon.


Release tracking

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#812 8ce76569 Thanks @danielguillan! - Ensure accessible content order in PricingOptions markup

#815 7345cc9d Thanks @rezrah! - Fixed corner background overflow bug in Card component on Safari browsers

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Smoke test results

Fixed background overflow bug in Card 🟢

Verified to have solved the bug in Safari browsers in github/github.

Pricing options markup order 🟢

Verified to render as expected in githubuniverse.com

No regressions observed.
Release testing status: Ready to release 🟢


Create an app under gcloud-java-examples that uses gcloud-java-resourcemanager

It should be similar in style to DatastoreExample and StorageExample.  This should be done when gcloud-java-resourcemanager is complete and ready for use.
Also be sure to update the global README and gcloud-java-examples' README.

Fixed by #487


pls try and add these pull rq did not work

pls add umlimited deamon keys and insta complelte do not give any

Suggestions should go to the Discord server.


Routing and Constraint

Two failing tests:
FAILED test_router.py::test_buf_1 - AssertionError: ckt_buf_1 (CKT_BUF_1_0):Number of DRC errorrs: 4
FAILED test_router.py::test_ro_1 - AssertionError: Where is the order constraint???

@854768750 , @Lastdayends , test_buf_1 has opens and DRC violations.
@kkunal1408 , test_ro_1 the constraints for the subcircuit is not propagated to PnR. Has there been a change in behavior?

@soneryaldiz
After changing the h_skip_factor and v_skip_factor to 3 and 4, this case passes the test.

@kkunal1408 , could you please review?


Roll engine to aa9793 (3 commits)

flutter/engine@aa9793f Roll src/third_party/skia a2187bf7629a..281224020a57 (5 commits)
flutter/engine@3978f07 Keep a copy of each engine's description that can be accessed outside the engine's UI thread
flutter/engine@0870e37 Roll src/third_party/skia 3b155a77c370..a2187bf7629a (11 commits)

This won't roll due to failing test introduced by flutter/engine@3978f07 (see https://github.com/flutter/flutter/pull/24458).
But @jason-simmons has another fix for it here: https://github.com/flutter/engine/pull/6888

Gotcha.  I'll wait for that PR to land.


Unable to extract stream URL (openload)

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.08.02
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

$ youtube-dl --verbose "http://openload.co/f/2s172zXFKLI"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'http://openload.co/f/2s172zXFKLI']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.08.02
[debug] Python version 2.7.15+ (CPython) - Linux-4.15.0-55-generic-x86_64-with-LinuxMint-19-tara
[debug] exe versions: ffmpeg 3.4.6, ffprobe 3.4.6, phantomjs 5
[debug] Proxy map: {}
[Openload] 2s172zXFKLI: Downloading embed webpage
[Openload] 2s172zXFKLI: Executing JS on webpage
ERROR: Unable to extract stream URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/openload.py", line 432, in _real_extract
    'stream URL')
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1005, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract stream URL; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Unable to extract stream URL

cant-reproduce? Meaning you can download the video? I don't think so!
This bug exists many months. I hope someone will finally fix it despite the "cant-reproduce" responses.

> py -3.7 .\youtube_dl\__main__.py http://openload.co/f/2s172zXFKLI
[Openload] 2s172zXFKLI: Downloading embed webpage
[Openload] 2s172zXFKLI: Executing JS on webpage
[download] Destination: 1967 GAMPROS AP' TO LONDINO DVDRIP.mp4.mp4
[download]   0.1% of 459.02MiB at 247.12KiB/s ETA 31:41
ERROR: Interrupted by user

There is no bug.

Thanks for the useful recommendations!


TRSS: add forever service creation to TRSS playbook

Ref https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1679
Ive added the steps to create the forever services for the TRSS frontend and backend to the trss playbook as per https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1679#issuecomment-729766341
Will test on a ubuntu 18 vm shortly

@llxia
(as I cant seem to add you as a reviewer)

Runs fine on a ubuntu 18 vm 👍🏻, using a blank file for trssConf.json

trssConf.json stores credentials for connecting to DB and CI servers. If we do not have DB password set, an empty trssConf.json is fine/correct. Personally, I would recommend adding the password into DB for security reasons at a later stage.


[GHSA-68xg-gqqm-vgj8] Puma HTTP Request/Response Smuggling vulnerability

Updates

Affected products

Comments
https://github.com/puma/puma

Hi there @nateberkopec! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

I approve.
(I think that's how I do this?)
Thanks @kwansuddoungjai

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.


ectosymbiotic parasitoidism == facultative ectosymbiotic parasitoidism

http://purl.obolibrary.org/obo/APOLLO_SV_00000345
http://purl.obolibrary.org/obo/APOLLO_SV_00000346
have the exact same logical definition

@hoganwr is ok with dropping the logical definitions from both as long as classification is still the same.


Wrong fields on Item Search

Noticed due to a mistake while using the filter functionality, at the item level, it provides the fields from the catalog level.
What fields are available at item level? Would expect the same as when searching the collection for an item.

Thanks for the updates


Rename two items under "Info" menu: Publications (from Publication) and Data Sources (from Source)

@Bibiko, could you make the above changes? I think it will help clarify. Thanks.

Closed in  311d415


Add test for compilation database extraction

Hmm, there appear to be some missing data deps maybe.

Forgot to remove the obsolete shell-script equivalent from which I took the test; fixed.


Audit Fee handling of BitAssets managed by delegates.

[x] If the collateral is CORE make sure the fee pool ratio is updated via the price feed.

[x] If the collateral is another BitAsset then recursively apply the fee pool ratios until a price per CORE is found

[ ] Using the fee pool price ratio, calculate the value of all fees in the BitAsset in terms of core, if the value of the collected fees is higher than a threshold defined by chain parameters, then liquidate the collected BitAsset fees for the collateral.   If the collateral is another BitAsset then add the proceeds to that BitAssets fees and repeat until the fees have been converted to CORE.

[ ] Generally speaking the delegates shouldn't be creating any BitAssets that are not ultimately backed by CORE which means this process will work for all network based BitAssets.   This process should not be applied for assets issued by others.

[ ] Only perform this fee conversion step on maintenance intervals.

First two components are completed by #94.

Actually, we're going to restrict the core_exchange_rate updates to the maintenance interval, to keep fees predictable.


Selected Date Drawable

Thank you for creating this amazing library. Its very useful.
I want to assign selected date to drawable. I want to assign circle border. But all I can do is assign color. How can I assign circle border to selected date?
All I can do is:
		materialCalendarView.setSelectionColor( /*Some Color Here*/ );

You will have to use a decorator to supply a custom selector. Please look at the BasicActivityDecorated in the sample application. Specifically the MySelectorDecorator is what you want. Make sure the drawable you use responds to pressed and checked.


how to pretrain llama2 in custom data?

how to pretrain llama2 in custom data? And how to load custom data？

I recommend the section on pretraining in custom data here, which I think should address your question: https://github.com/Lightning-AI/litgpt/blob/main/tutorials/pretrain.md#pretrain-on-custom-data


Review eprints repos that were not parsed

Some repositories could not be completely parsed (see #22 for details). Find out why and rerun analysis.

usir.salford.ac.uk: cannot export, some rerouting happens and we download the wrong file format.

The failed parsings hapened due to unexpected formats that would need to be accounted for for any future reruns wanting to incorporate info from those repositories. It's not something that needs "fixing" though.


system_fingerprint: 0.7.0-1 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository system_fingerprint to 0.7.0-1:

upstream repository: https://github.com/MetroRobots/ros_system_fingerprint.git
release repository: https://github.com/MetroRobots/ros_system_fingerprint-release.git
distro file: foxy/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.5.0-1

Holding for Foxy sync


feat(FDS-375): [Cascara] - Switch DOM node to Role instead

feat(FDS-375): [Cascara] - Switch over our components that render a DOM node at their root to use abstract components from Reakit.
We replace :

div by Role,
when we have a native HTML tag that has an onClick action, we use Clickable instead.

No changes in snapshots.
New Component Checklist

[x] Includes unit tests for ALL required FDS use cases
[x] Does not mute any top level API props in React
[x] Includes an explanation for any changes to Jest snapshots (should be a PR tag automatically added if this happens)
[x] Component uses React.forwardRef() when a single DOM node is returned, AND it makes React-sense to access that DOM node

@rociobaena I see that we are using conventional commits 👍, although we are still missing the changeset, I am approving this since it won't be merged yet, in the meantime, let's add the changeset. This is awesome!


EditorSearch => scroll top on page change

containerRef.scrollTop = 0 when not working, considering it's already on top, while it isn't


Add usage

Addresses #68 by making a markdown version of the usage (about.md) and adds pointers to that and to the original html version in the README.
I used pandoc to convert about.xhtml to about.md and then did some manual editing of about.md.
The markdown version has the benefit of being directly viewable in github browser.
The main drawback of this PR is that the .md and .xhtml versions may get out of sync in the future.
BTW, I am a huge fan of nostalgy and I am still using TB60 because (if I recall) nostalgy did not work with TB68 when TB68 first came out.  It would be nice if the README would specify what is the current version of TB that is supported by nostalgy.

Current versión of Nostalgy works fine with Thunderbird 68.0 - *
https://addons.thunderbird.net/en-US/thunderbird/addon/nostalgy/


test_crypt does not function as a test

Follow-up to my comment in #1350. test_crypt has multiple issues.

Tests fail if the backing cryptography library does not support the algorithm. For example, currently (our) MbedTLS does not support HMAC-SHA3 which is reported as a failure rather than skipped.
Some of the tests do not return a status on pass / fail, rendering them useless in CI/CD. For example : https://github.com/DMTF/libspdm/blob/beee7bd85a4f0961a5773aefc4dcab0408d538d1/unit_test/test_crypt/hash_verify.c#L420-L439
Return status confusion. In particular https://github.com/DMTF/libspdm/blob/3337da5d52c32d4bf3585506c67d8c92b587b256/unit_test/test_crypt/test_crypt.c#L149-L158 is incorrect. libspdm_cryptest_main returns true (1) if all the tests pass and false (0) if one of them fails. main should return 0 if all tests pass and != 0 if one of them fails.

Due to 3, test_crypt has failing tests but it is being reported as a pass to CI/CD. We should disable test_crypt in CI/CD until these issues are resolved.

once you fix it issue, please re-enable test_crypt in CI.


Update travis rubies

:muscle:

Coverage remained the same at 91.096% when pulling 4c5d8adfad9d76d5ad5f1a7f797dd11bb1263027 on sanemat:chore/travis-rubies into 986ab1e3d63c9217170a76c829bb3dfa05f11f28 on bitpay:master.

Coverage remained the same at 91.096% when pulling 4c5d8adfad9d76d5ad5f1a7f797dd11bb1263027 on sanemat:chore/travis-rubies into 986ab1e3d63c9217170a76c829bb3dfa05f11f28 on bitpay:master.

oops

Fix Travis failure on nil ENV['BITPAYPEM'] #52


chore: upgrade sqlalchemy to 2.x

Reason for Change

upgrades to the latest major version of sqlalchemy
this remove deprecation warning

Changes

modified functions in the persistence layer to comply with the new version syntax
updated requirements of sqlalchemy, psycopg2-binary, and alembic to match the latest major version
adopting the x.* syntax for pinning to major version in python requirements files

Testing steps

verified all persistence layer tests pass.

@nayib-jose-gloria I think this should be done as needed. If we end up splitting the persistence layer we will be touch a lot of this code anyway and can refactor as we go.


buildDependencies?

By using prebuild-install I've found mostly is the only one dependency I need, except when it can resolve to a prebuild image and needs to be compiled, for example nan and similar ones that are useless for prebuild images. I could set them as devDependencies, but in that case they would not be available if the prebuild is not available, so that dependencies are in a no-man-land status.
For that situation, I think a solution would be a buildDependencies where to set the dependencies needed only when the prebuild image could not be fetch (because it's not available or due to download failure), that would be installed by prebuild-install itself. This could be done being that buildDependencies an independent field object with no relationship with the other dependencies, or it could be an array with the name of the devDependencies that are in fact build ones, so this way it could be integrated with current workflows and being compatible with tools like node-check-updates and similar ones.
Thoughts?

Interesting idea! 👍
Would the following install script work for you?
"scripts": {
    "install": "prebuild-install || (npm install nan && node-gyp rebuild)"
  }

This way prebuild-install will remain lightweight and will only handle the downloading of the prebuilts.

Would the following install script work for you?
"scripts": {
"install": "prebuild-install || (npm install nan && node-gyp rebuild)"
}

Yes, you got the idea ;-) In fact I was thinking about doing it like this in a first time, but when you have several buildDependencies the install line gets too long and complex, and also there's the problem that doing it that way these dependencies get unversioned and will always install the latests ones, that's not always desirable (or set them as package@version, making the install line more long and complex...).

npm also supports a syntax like npm install<EMAIL_ADDRESS><EMAIL_ADDRESS>which is versioned and for me not much bulkier than implementing buildDependencies.

When I said "versioned" I mean "automatic versioned", it's said, dependencies could be check and updated using node-check-updates and similar tools.

When I said "versioned" I mean "automatic versioned", it's said, dependencies could be check and updated using node-check-updates and similar tools.

Ooh I see, that wouldn't work.
Though one could still add them as a devDependency and and make use of update notifications via node-check-updates. This way one could update the install command with minimal effort.

Though one could still add them as a devDependency and and make use of update notifications via node-check-updates. This way one could update the install command with minimal effort.

Yeah, but this way you need to have them defined in two places that would get unsync.
I've giving a look at your idea of the install script and I've created the buildDependencies module :-P It's just a simple command that look for the buildDependencies and devDependencies fields and exec npm install with them, and I've give it the possiblity to define the name of the field with the list of dependencies instead of the default buildDependencies. With that, now there's no need to add this functionality to prebuild-install, although the module is importable so we could add it as a dependency and done it automatically too. If not, I think it would a good idea at least add a reference on the docs about it in case others are in the same situation... What do you think?


Fix Scala 2.13.0-RC1 compilation, fix a number of simple bugs / enhancements

If you're fine with these, we can then tag the version. build.sbt already has version set to 2.1.1

The build changes LGTM

I pushed the corresponding tag: https://github.com/scala/scala-swing/releases/tag/v2.1.1

🎉

I think someone needs to release that on sonatype, AFAICS the artifacts have been uploaded.

@SethTisue could you do the honors?

sorry for the delay. artifacts should appear soon at https://repo1.maven.org/maven2/org/scala-lang/modules/scala-swing_2.13.0-RC1/2.1.1

and there they are!
scala/make-release-notes@2d559a65b940d7c90ed2b732fdc935594d0579e8


Flag to allow the game to run with EAC

Hello All,
This Feature adds a flag to the script that allow the game to start with EAC enabled, by making Steam execute the "start_protected_game.exe" instead of the "eldenring.exe".
At the time of writing, I've been playing the game online with all fixes active and with EAC enabled for the better part of a week. I'm yet to encounter any problems with EAC/Online Playing due to the hex edits, but good-judgment must be made by the player, as to whether use or not this feature.
As evidence that it does in fact work, here is a screenshot of me playing the game online with all fixes active:

Best Regards,
João Pedro Braz

Hello, thanks for this pull request! Did you intentionally not add documentation to the readme or is this intended to be a "hidden" feature? To know this feature exists currently one would have to effectively read the source code.
I think if we add this feature it would be best to document it in the readme and put a bold warning in the flag description.

Hello!
Yeah, I forgot to do that... 🙂
I'll fix it in a gist, thanks for pointing it out!
Best Regards,
João Pedro Braz

Hello!
Just committed an update to the README, adding the flag to the Features table.
Best Regards,
João Pedro Braz

Hello!
I updated the README to make it clearer that it's possible to run the game with EAC enabled.
Don't mind the reverted commit, had a brain fart.
Best Regards,
João Pedro Braz

Thanks for the update! It looks fine now :)


Swagger Documentation

Is your feature request related to a problem? Please describe.
We need to be build a better experience for developers by providing documentations that can be used to programmatically integrate delta.
Describe the solution you'd like
We can use swagger and start documenting the REST API handlers.
We have done this on Estuary and we can apply the same process for Delta.
### Tasks
- [ ] Generate swagger docs for deal api
- [ ] Generate swagger docs for open stats api
- [ ] Generate swagger docs for open stats info api
- [ ] Generate swagger docs for repair and retry api

Scaffolding/Initial commit: https://github.com/application-research/delta/tree/swagger-docs
To run the swagger generator
make generate-swagger

The above command will do the following:

scan the ./api directory for all the *.go files starting with router.go
scan all the annotations on the handlers
generate the swagger documentation in ./docs/swagger/swagger.yaml

Check swagger docs using /swagger endpoint.

To check the generate swagger doc (json), go to http://localhost:1414/swagger/doc.json
To check the swagger ui, go to http://localhost:1414/swagger/index.html

First attempt at annotations for the deal.go API file Swagger docs #143
Will need review, I have some active TODO items that I need to clarify. Requested code review with Alvin for next week when I get back from ETH Waterloo.


types: add Stringer implementation for List and Tuple

@matteo-grella PTAL
this provides nice(r) display of *types.List and *types.Tuple.

Very cool, thank you!


any way to automate install on mac?

Could I install this with homebrew or similar, so I could save myself a few steps when I want to update to add the latest plugins?

Updating in home brew is a manual process. We do it for surge, but since the cadence here is weekly I didn’t set up a homebrew package for awcon


Fix ngnix redirects to localhost.

Verified the fix by creating a test image and deploying it to a cloud instance. This URL now correctly redirects to /browser/: https://dc-dev-kqb7thiuka-uc.a.run.app/browser
Also created a script to easily build and push test images in the cloud.

Thanks Dan.


Backend Response Format Change

This PR creates a minor change in how the backend will send the response. It makes it easier for the frontend to print the statement, regardless of whether the response is a string or a nested JSON.

create a struct called Response that will be mapped to the response by the OAuth2l wrapper and the credentialsToken (if necessary). This struct will then be encoded to a JSON response.
changes the TokenValidation part to fit the format of the JSON response.

@googlebot I consent.


the video is restarted when the device is rotated

Reported by @Vincentlp:matrix.org
When playing video in a room, it is going back at the beginning when I change the screen orientation or if I touch the screen during the playback
https://riot.im/bugreports/listing/2017-05-03/105112/details.log.gz

https://github.com/vector-im/riot-android/issues/1189


ScyllaDB: Enable the CI

This should be as simple as adding the docker run --name my_scylla_container -d -p 9042:9042 scylladb/scylla entry in the CI.
However, how it would be organized is an interesting question.

This has been resolved by PR #953


dpkg complains about rm missing from path

Issue description
The NixOS cheatsheet claims I can install .deb files through a Nix-installed dpkg, but when trying this I encounter some errors.
Steps to reproduce
Example using the .deb for Sogou Pinyin, a popular proprietary Chinese IME:
$ sudo dpkg -i sogoupinyin_<IP_ADDRESS>8_amd64.deb
dpkg: error: cannot scan updates directory '/var/lib/dpkg/updates/': No such file or directory
# sure, I'll make you some directories.
$ sudo mkdir -p /var/lib/dpkg/updates/

$ sudo dpkg -i sogoupinyin_<IP_ADDRESS>8_amd64.deb
dpkg: error: failed to open package info file '/var/lib/dpkg/status' for reading: No such file or directory
# sure, I can make that.
$ sudo touch /var/lib/dpkg/status

$ sudo dpkg -i sogoupinyin_<IP_ADDRESS>8_amd64.deb
dpkg: warning: '/nix/store/dm20hrdk6s4jzfmk1197p2nya0p8fy3a-coreutils-8.29/bin/rm' not found in PATH or not executable
dpkg: warning: '/nix/store/dm20hrdk6s4jzfmk1197p2nya0p8fy3a-coreutils-8.29/bin/diff' not found in PATH or not executable
dpkg: warning: '/nix/store/143saabh5yjfbpwwadvc8hm31khxxrb6-dpkg-<IP_ADDRESS>/bin/dpkg-deb' not found in PATH or not executable
dpkg: error: 3 expected programs not found in PATH or not executable
Note: root's PATH should usually contain /usr/local/sbin, /usr/sbin and /sbin

# so, something's wrong with `rm` and the like involving $PATH and those locations? let's check them out.
$ which rm
/run/current-system/sw/bin/rm
# so there's our `rm`. is this in $PATH?
$  echo $PATH
/home/tycho/bin /run/wrappers/bin /home/tycho/.nix-profile/bin /nix/var/nix/profiles/default/bin /run/current-system/sw/bin /etc/profiles/per-user/tycho/bin
# looks like its folder is! let's check it there?
$  ll /run/current-system/sw/bin/rm
lrwxrwxrwx 15 root root 65 Jan  1  1970 /run/current-system/sw/bin/rm -> /nix/store/dm20hrdk6s4jzfmk1197p2nya0p8fy3a-coreutils-8.29/bin/rm
# we know that place from the error trace. let's check that?
$ ll /nix/store/dm20hrdk6s4jzfmk1197p2nya0p8fy3a-coreutils-8.29/bin/rm
lrwxrwxrwx 1 root root 9 Jan  1  1970 /nix/store/dm20hrdk6s4jzfmk1197p2nya0p8fy3a-coreutils-8.29/bin/rm -> coreutils
# it points to coreutils? I'm confused how that works.
$ coreutils
Try 'coreutils --help' for more information.
$ coreutils --help
Usage: coreutils --coreutils-prog=PROGRAM_NAME [PARAMETERS]...
...

At this point I'm feeling confused what's going on here and what the errors might be going on about, as well as why I haven't seen similar reports here despite dpkg used apparently having been tried.
Technical details

system: "x86_64-linux"
host os: Linux 4.18.0, NixOS, 18.09.1799.b9fa31cea0e (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-18.09.1799.b9fa31cea0e, nixos-unstable-small-19.03pre164396.24cdaa7b488"
channels(tycho): "nixos-unstable-small-19.03pre164396.24cdaa7b488"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

It's probably not what you want to hear, but if I were you I would write a proper derivation for this package and then use dpkg -x to unpack it and shift the various parts into place.
It does look like dpkg in nix is broken - you do have a valid point, but I doubt this particular use case sees a lot of real life action so it is unlikely to be well maintained.

Thanks for the suggestion -- that does actually seem helpful.
As this particular package is proprietary, I feared making a native package would be a problem.
Now that you mention it though, it does seem like other packages pulled this off as well.
I'll look into this route. Thank you!
(If any Chinese NixOS users are reading this and have suggestions on how to deal with Chinese IMEs in NixOS, I'd love to hear them!)

Even if the dpkg -i ./some_file.deb part worked as planned, the binary would be unlikely to work unless you use patchelf on it, so then we're back to "might as well do a proper derivation" territory...

As this particular package is proprietary, I feared making a native package would be a problem.

We have lots of proprietary stuff in nixpkgs - that shouldn't be an issue. If the source .deb isn't publicly available on the internet for downloading, you can take a look at how making the source package available for the citrix receiver is handled in nixpkgs .

I have updated the wiki.


Map Block Names to IDs

Block IDs used by the standalone viewer do not match the deprecated id numbers or their position in the block list. For instance oak saplings have an id of 6, and they are the 19th block in the block list, but the ID in prismarine is 21. Ideally you should be able to use their names, as that is their only official identifiers, "oak_sapling".

This leads to the same problem, perhaps the ids are misaligned. mcData.blocksByName['oak_sapling'].id returns 19, as it should, but when you call viewer.world.setBlockStateId(new Vec3(0, 1, 0), mcData.blocksByName['oak_sapling'].id); a acacia plank appears.

ah yeah, sorry the generator function returns a stateId not a type, so mcData.blocksByName['oak_sapling'].minStateId

Yes, this appears to be working, thanks! I can make another issue about this if you want but is there a way to access the three.js scene from the standalone viewer? For instance if I wanted to add new models or edit the controls? Without needing to modify the original package.

No it is not possible at the moment without modifying the packages. Some people have made it work by copying bits of the code in their project, but thats not a good long term solution. There is an open issue related to this https://github.com/PrismarineJS/prismarine-viewer/issues/69


venv isolation broken when debugging

Type: Bug

Behaviour
Expected vs. Actual
When I activate a virtual environment I expect python to use packages from this virtual environment over globally installed packages.
If run the debugger with the vscode python extension on Ubuntu sys.path contains /usr/lib/python3/dist-packages in front of the venv packages. This causes global packages to be loaded.
If you add export PYTHONPATH="" to .zshrc this is not added to sys.path during debug. But if you modify the PYTHONPATH eg to add a src folder inside your launch.json
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python Debugger: Current File with Arguments",
            "type": "debugpy",
            "request": "launch",
            "program": "${file}",
            "console": "integratedTerminal",
            "env": {
                "PYTHONPATH": "src"
            }
        }
    ]
}

it still shows up in sys.path
Steps to reproduce:

Create venv
Select venv in vscode
Create launch.json file as above
debug a python file and print sys.path
Verify that /usr/lib/python3/dist-packages is in path before venv folders

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.9.18
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Venv
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

User Settings

languageServer: "Pylance"

terminal
• activateEnvironment: false

Extension version: 2024.0.1
VS Code version: Code 1.86.2 (903b1e9d8990623e3d7da1df3d33db3e42d80eda, 2024-02-13T19:41:37.860Z)
OS version: Linux x64 6.2.0-39-generic
Modes:

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscoreces:30445986
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
vscrpc:30673769
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
edj9j583:30958191
fegfb526:30961430
7j2b6412:30964151
ccp1r6:30958156

I am having a similar issue with conda environments (https://github.com/microsoft/vscode-python/issues/22918).
It seems like something is botched in the Python plugin.

This is particularly annoying as I cannot debug properly unless I install the packages in the base environment.


Colors Showing up Differently on Different machines

Idk if this is for you @Mofiqul or something else, but here's what I'm seeing:
I have my minimial neovim setup on my remote development environments, using Packer as the package manager.
On my daily driver, I have a very similar setup, only using Lazy, and more plugins. But they came from the same original setup.
The color scheme renders differently for the same file. Here's a screenshot - the left is my daily driver machine, the right is my remote server. I love the look of the right environment so much more, so I'm hoping for it to show up the same way on my daily machine; I just can't seem to figure out why!

If you have any suggestions, that would be helpful! Thanks again for the great color scheme.

@Normanras - I guess your terminal does not support truecolor or need to enable it.

@Mofiqul Well that's the weird thing. Those screenshots are from the exact same terminal app on my local machine, the only difference is the right is ssh/mosh into a remote machine and the left is the same exact setup on my local machine. Versions of neovim & plugins are all equal. That's why it seems like a head scratcher. I'm sure I'm config-ing something incorrectly, just not sure what.

@Normanras A bit late, but might be worth checking this: https://wiki.webevaluation.nl/bashrc

I am having a very similar issue... At least the color palette is similar to that of the left picture. Here is a comparison between using the mofiqul dracula theme (first image), and the https://github.com/maxmx03/dracula.nvim theme (second image):

So the Mofiqull color palette appears wrong in a similar manner to @Normanras issue. Both of the above are running on the same remote linux machine accessed from an iTerm2 terminal on an OSX machine, over SSH. I don't believe its a truecolor issue as some suggest here, as it appears to work for the maxmx03 plugin, because my terminal shoud support it and because I have it enabled in my .zshrc.
Before it is suggested, I'd prefer not to use the Mofiqul theme, as it appears to work with likas-reineke/indent blankline plugin.
Further, it looks like on the original post, that indent blankline (or similar) is running in the left image, but not in the right. I tried to uninstall mine, to see if this was the cause of the error, but with no luck.

Okay so just to add to the above: I just created a new user on my Ubuntu system. No tmux, no oh-my-zsh (it's bash per default anyways) - completely clean user.
I then set up a completely virgin nvim config, and the problem persists. My init.lua is as follows:
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "https://github.com/folke/lazy.nvim.git",
    "--branch=stable", -- latest stable release
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)

vim.g.mapleader = " " -- Make sure to set `mapleader` before lazy so your mappings are correct

require("lazy").setup({
  "folke/which-key.nvim",
  { "folke/neoconf.nvim", cmd = "Neoconf" },
  {"folke/neodev.nvim"},
  {"Mofiqul/dracula.nvim"},
})
vim.cmd[[colorscheme dracula]]

So basically lazy plugin manager, and the dracula theme. The result is the same, so I guess we can rule out any influence from any plugins other than potentially the pluginmanager itself.
My .bashrc is then a completely standard ubuntu one, so for completion it reads:
# ~/.bashrc: executed by bash(1) for non-login shells.
# see /usr/share/doc/bash/examples/startup-files (in the package bash-doc)
# for examples

# If not running interactively, don't do anything
case $- in
    *i*) ;;
      *) return;;
esac

# don't put duplicate lines or lines starting with space in the history.
# See bash(1) for more options
HISTCONTROL=ignoreboth

# append to the history file, don't overwrite it
shopt -s histappend

# for setting history length see HISTSIZE and HISTFILESIZE in bash(1)
HISTSIZE=1000
HISTFILESIZE=2000

# check the window size after each command and, if necessary,
# update the values of LINES and COLUMNS.
shopt -s checkwinsize

# If set, the pattern "**" used in a pathname expansion context will
# match all files and zero or more directories and subdirectories.
#shopt -s globstar

# make less more friendly for non-text input files, see lesspipe(1)
[ -x /usr/bin/lesspipe ] && eval "$(SHELL=/bin/sh lesspipe)"

# set variable identifying the chroot you work in (used in the prompt below)
if [ -z "${debian_chroot:-}" ] && [ -r /etc/debian_chroot ]; then
    debian_chroot=$(cat /etc/debian_chroot)
fi

# set a fancy prompt (non-color, unless we know we "want" color)
case "$TERM" in
    xterm-color|*-256color) color_prompt=yes;;
esac

# uncomment for a colored prompt, if the terminal has the capability; turned
# off by default to not distract the user: the focus in a terminal window
# should be on the output of commands, not on the prompt
#force_color_prompt=yes

if [ -n "$force_color_prompt" ]; then
    if [ -x /usr/bin/tput ] && tput setaf 1 >&/dev/null; then
        # We have color support; assume it's compliant with Ecma-48
        # (ISO/IEC-6429). (Lack of such support is extremely rare, and such
        # a case would tend to support setf rather than setaf.)
        color_prompt=yes
    else
        color_prompt=
    fi
fi

if [ "$color_prompt" = yes ]; then
    PS1='${debian_chroot:+($debian_chroot)}\[\033[01;32m\]\u@\h\[\033[00m\]:\[\033[01;34m\]\w\[\033[00m\]\$ '
else
    PS1='${debian_chroot:+($debian_chroot)}\u@\h:\w\$ '
fi
unset color_prompt force_color_prompt
# If this is an xterm set the title to user@host:dir
case "$TERM" in
xterm*|rxvt*)
    PS1="\[\e]0;${debian_chroot:+($debian_chroot)}\u@\h: \w\a\]$PS1"
    ;;
*)
    ;;
esac

# enable color support of ls and also add handy aliases
if [ -x /usr/bin/dircolors ]; then
    test -r ~/.dircolors && eval "$(dircolors -b ~/.dircolors)" || eval "$(dircolors -b)"
    alias ls='ls --color=auto'
    #alias dir='dir --color=auto'
    #alias vdir='vdir --color=auto'

    alias grep='grep --color=auto'
    alias fgrep='fgrep --color=auto'
    alias egrep='egrep --color=auto'
fi

# colored GCC warnings and errors
#export GCC_COLORS='error=01;31:warning=01;35:note=01;36:caret=01;32:locus=01:quote=01'

# some more ls aliases
alias ll='ls -alF'
alias la='ls -A'
alias l='ls -CF'

# Add an "alert" alias for long running commands.  Use like so:
#   sleep 10; alert
alias alert='notify-send --urgency=low -i "$([ $? = 0 ] && echo terminal || echo error)" "$(history|tail -n1|sed -e '\''s/^\s*[0-9]\+\s*//;s/[;&|]\s*alert$//'\'')"'

# Alias definitions.
# You may want to put all your additions into a separate file like
# ~/.bash_aliases, instead of adding them here directly.
# See /usr/share/doc/bash-doc/examples in the bash-doc package.

if [ -f ~/.bash_aliases ]; then
    . ~/.bash_aliases
fi

# enable programmable completion features (you don't need to enable
# this, if it's already enabled in /etc/bash.bashrc and /etc/profile
# sources /etc/bash.bashrc).
if ! shopt -oq posix; then
  if [ -f /usr/share/bash-completion/bash_completion ]; then
    . /usr/share/bash-completion/bash_completion
  elif [ -f /etc/bash_completion ]; then
    . /etc/bash_completion
  fi
fi

Note that I am doing all this over SSH (ssh -Y servername) from an OSX terminal iTerm2. Don't know if that makes any difference.


Stripe merge

This is a cleaner version of #70.
Add communication with the Stripe API to generate invoices for data deposits. Addresses issue  https://github.com/CDL-Dryad/dryad-product-roadmap/issues/190.
An invoice is generated whenever an dataset's curation status is set to "Published" or "Embargoed", and the dataset's Identifier has not yet been invoiced.
This PR generates an invoice for all datasets. It does not include the logic for when a dataset is being paid by a journal or institution. That logic will be part of https://github.com/CDL-Dryad/dryad-product-roadmap/issues/191.
To deploy:

This PR must be deployed in conjunction with https://github.com/CDL-Dryad/dryad/pull/48 and https://github.com/cdlib/dryad-config/pull/19.
Also note that there is a database migration required.

To test:

Change any dataset to curation status "Published" or "Embargoed".
Look in the database and verify that the submitter's entry in stash_engine_users now has a Stripe customer_id, and the dataset's entry in stash_engine_identifiers now has a Stripe invoice_id.
If you have a Stripe account, you can view the invoice there. (The invoice won't actually be sent to the user, since Stripe is in test mode.)

Hi Ryan,
I haven't looked at the tests all that much, but you were asking me why to use a test double for Stripe.

Those places (where submitting to stripe) isn't your core activity and it breaks the tests you'll probably need to add something like allow_any_instance_of(CurationActivity).to receive(:submit_to_stripe).and_return(true) in order to ignore all those calls that are generated when they're not wanted for the core of testing some other component or method.
If you're using the stripe object in the method that submits to stripe you can do something like mocking a stripe object and mocking the methods you call with return values or whatever you're doing.  That way you can ensure that the method that does things with stripe doesn't fail in a breaking way in the future if you change the method that works with stripe since it stands in for methods you know work on the stripe object.
You don't need to test stripe itself if you're using a stripe gem and they presumably already have tests on that gem to ensure they keep their bargain and their gem doesn't break.


Fix:  warnings during build process

Description:
Fixed the <img> tag warning issue during the build process by replacing the normal <img> element with the Nextjs <Image> element.
Resources:

NextJS  element
NextJS image responsive

@bekossy can you please have a look if the images are looking similar to what they used to be ?


How to create a streak type achievement?

I'm trying to understand how to create this kind of achievement:
Login every day for x Days
Commit every day for x days...

i have some problems when run the project ? could you help me ?


Changing the size of preview window.

Is there a way to change the size of a preview window in VScode. I'm working with this on a laptop and it would be nice if I can resize the preview window to be bigger. Sometimt it's really hard to see snall text, object. Thank you for your work.

Can you please provide some screenshots as to what you mean?


Fix setRepeated on non power of two textures

Should fix #13

@Nathan-M-code @Scorbutics if you'd like to test here is an example:
#include <SFML/Graphics.hpp>

int main()
{
    sf::Image img;
    img.create(17, 17);
    for (int i = 0; i < 17; i++)
        for (int j = 0; j < 17; j++)
            img.setPixel(i, j, sf::Color(rand() % 256, 0, 0));
    sf::RenderWindow window(sf::VideoMode::getDesktopMode(), "SFML works!");

    sf::Texture tex;
    tex.loadFromImage(img);
    tex.setRepeated(true);
    sf::RectangleShape shape;
    shape.setSize(sf::Vector2f(256, 256));
    shape.setTexture(&tex);
    shape.setTextureRect(sf::IntRect(0, 0, 68, 68));

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

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

    return 0;
}

Before:

Should now look like this:


Build depends on a full download of IntelliJ

When building detekt now that the detekt-intellij-plugin is merged Gradle will download a full installation of IntelliJ during the build phase.
Expected Behavior

We don't depend on a 350+MB download to build the project.

Current Behavior
Gradle downloads ideaIC-2017.3.5.zip
Steps to Reproduce (for bugs)
./gradlew assemble

Yes, that's right. It's huge, but what to do about it?
gradle and travis caches the dependency. After the first download it is not a big deal, is it?
Well, ofc we could open a new repository for it.

I agree with @Mauin
I think there are a number of people out there who don't want to use detekt with the IntelliJ plugin. Especially those who use detekt on their CI server.
Adding a new repository is a good approach in my opinion.

#826 removes the plugin. It has now a new home here


Race condition in updating ProtocolParameters

Currently, there is no way of know which ProtocolParameters belong to which Milestone.
Once #1623 is fixed in Hornet, we should fix this in Chronicle as well.

This has been fixed by the recent changes to the INX protocol.


pipeline parameters

Hi, I'm trying to setup a release and it seems to deploy. However, the IIS Web App Manage task has Website name fixed as 'Default Web Site'. The link thing states "This setting is linked to the 'Website name' (Parameters.WebsiteName) pipeline parameter."
I just cannot figure out how to access this Parameters.WebsiteName setting and deploy to a different website.
I notice that here https://github.com/MicrosoftDocs/vsts-docs/issues/2102#issuecomment-521165342
it is stated "Please note, setting process parameters is supported only in build pipelines and not in release pipelines. ". I cannot see the value of setting Website name in a build pipeline.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ca1c5394-611b-b119-dbf3-b88404e351d5
Version Independent ID: fcc23259-21ec-f9fa-4e25-f587e1e0f41b
Content: Deploy to IIS on a Windows VM - Azure Pipelines
Content Source: docs/pipelines/apps/cd/deploy-webdeploy-iis-deploygroups.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

I tried with the template you suggested and I'm able to edit.

Click on the stage 1 and you can edit the text "Default Web Site" to whatever the website name is..

Thank you @andrewrandsbluestone for the workaround. I've used the full template in the past and it allowed me to enter a web site name. Now it doesn't. The only way I could get there was by adapting your solution, except I had to begin by adding via the IIS (without SQL) template since it was the only way I could find to reference my Deployment Group. Then I deleted the template's Manage and Deploy IIS tasks, then I added them back in manually as you indicate. Finally I could specify my web site name.
Absolutely agree: this issue should not be closed!  It is still very much an active problem with hard-coded dependency on a parameter, Parameters.WebsiteName, and no indication of how to set it or where it comes from. Super frustrating.


A couple of PHP 8.2 fixes

Avoid passing null to str_pad() and declare the missing fromHeader property in SpanContext.php

Tests are failing because they rely on phpspec/prophecy and that isn't in your composer.json but that is not related to my changes.


Backport "HBASE-29005 Cannot split split system tables when quota enforcement is enabled (#6501)" to branch-2

Backport of #6501

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  3s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m  9s

branch-2 passed

+1 :green_heart:
compile
4m  1s

branch-2 passed

+1 :green_heart:
checkstyle
1m  5s

branch-2 passed

+1 :green_heart:
spotbugs
2m 33s

branch-2 passed

+1 :green_heart:
spotless
1m 22s

branch has no errors when running spotless:check.

-0 :warning:
patch
1m 31s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 51s

the patch passed

+1 :green_heart:
compile
3m 28s

the patch passed

+1 :green_heart:
javac
3m 28s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 54s

the patch passed

+1 :green_heart:
spotbugs
2m 43s

the patch passed

+1 :green_heart:
hadoopcheck
23m  7s

Patch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.0.

+1 :green_heart:
spotless
1m  5s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 22s

The patch does not generate ASF License warnings.

52m 48s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6512

Optional Tests
dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless

uname
Linux 33845be75a70 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 17b1825b5d96a03e2bf564235d042e16a108199b

Default Java
Eclipse Adoptium-11.0.23+9

Max. process+thread count
77 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/console

versions
git=2.34.1 maven=3.9.8 spotbugs=4.7.3

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 59s

Docker mode activated.

-0 :warning:
yetus
0m  5s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 14s

branch-2 passed

+1 :green_heart:
compile
1m  7s

branch-2 passed

+1 :green_heart:
javadoc
0m 30s

branch-2 passed

+1 :green_heart:
shadedjars
6m 24s

branch has no errors when building our shaded downstream artifacts.

-0 :warning:
patch
6m 37s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 45s

the patch passed

+1 :green_heart:
compile
1m  3s

the patch passed

+1 :green_heart:
javac
1m  3s

the patch passed

+1 :green_heart:
javadoc
0m 29s

the patch passed

+1 :green_heart:
shadedjars
6m  5s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
246m 10s

hbase-server in the patch passed.

275m 37s

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6512

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 877332fb9674 5.4.0-195-generic #215-Ubuntu SMP Fri Aug 2 18:28:05 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 17b1825b5d96a03e2bf564235d042e16a108199b

Default Java
Eclipse Adoptium-11.0.23+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/testReport/

Max. process+thread count
4511 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  9s

Docker mode activated.

-0 :warning:
yetus
0m  4s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 56s

branch-2 passed

+1 :green_heart:
compile
1m 33s

branch-2 passed

+1 :green_heart:
javadoc
0m 44s

branch-2 passed

+1 :green_heart:
shadedjars
7m 50s

branch has no errors when building our shaded downstream artifacts.

-0 :warning:
patch
8m  4s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 24s

the patch passed

+1 :green_heart:
compile
1m 15s

the patch passed

+1 :green_heart:
javac
1m 15s

the patch passed

+1 :green_heart:
javadoc
0m 35s

the patch passed

+1 :green_heart:
shadedjars
8m 11s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

-1 :x:
unit
313m 38s
/patch-unit-hbase-server.txt
hbase-server in the patch failed.

349m 57s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6512

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 519afa678f28 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 17b1825b5d96a03e2bf564235d042e16a108199b

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/testReport/

Max. process+thread count
4470 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 10s

Docker mode activated.

-0 :warning:
yetus
0m  5s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 25s

branch-2 passed

+1 :green_heart:
compile
1m  1s

branch-2 passed

+1 :green_heart:
javadoc
0m 45s

branch-2 passed

+1 :green_heart:
shadedjars
6m 48s

branch has no errors when building our shaded downstream artifacts.

-0 :warning:
patch
6m 58s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 23s

the patch passed

+1 :green_heart:
compile
1m  8s

the patch passed

+1 :green_heart:
javac
1m  8s

the patch passed

+1 :green_heart:
javadoc
0m 45s

the patch passed

+1 :green_heart:
shadedjars
6m 37s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

-1 :x:
unit
330m 29s
/patch-unit-hbase-server.txt
hbase-server in the patch failed.

361m 54s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6512

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux cda0278a8049 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 17b1825b5d96a03e2bf564235d042e16a108199b

Default Java
Temurin-1.8.0_412-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/testReport/

Max. process+thread count
4239 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  6s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
2m 46s

branch-2 passed

+1 :green_heart:
compile
2m 30s

branch-2 passed

+1 :green_heart:
checkstyle
0m 35s

branch-2 passed

+1 :green_heart:
spotbugs
1m 26s

branch-2 passed

+1 :green_heart:
spotless
0m 42s

branch has no errors when running spotless:check.

-0 :warning:
patch
0m 49s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 40s

the patch passed

+1 :green_heart:
compile
2m 24s

the patch passed

+1 :green_heart:
javac
2m 24s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 34s

the patch passed

+1 :green_heart:
spotbugs
1m 32s

the patch passed

+1 :green_heart:
hadoopcheck
13m 36s

Patch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 40s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 12s

The patch does not generate ASF License warnings.

33m 25s

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6512

Optional Tests
dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless

uname
Linux 306eea989bc7 5.4.0-195-generic #215-Ubuntu SMP Fri Aug 2 18:28:05 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 17b1825b5d96a03e2bf564235d042e16a108199b

Default Java
Eclipse Adoptium-11.0.23+9

Max. process+thread count
79 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/console

versions
git=2.34.1 maven=3.9.8 spotbugs=4.7.3

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 48s

Docker mode activated.

-0 :warning:
yetus
0m  4s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 10s

branch-2 passed

+1 :green_heart:
compile
0m 59s

branch-2 passed

+1 :green_heart:
javadoc
0m 47s

branch-2 passed

+1 :green_heart:
shadedjars
8m 10s

branch has no errors when building our shaded downstream artifacts.

-0 :warning:
patch
8m 19s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  5s

the patch passed

+1 :green_heart:
compile
0m 58s

the patch passed

+1 :green_heart:
javac
0m 58s

the patch passed

+1 :green_heart:
javadoc
0m 35s

the patch passed

+1 :green_heart:
shadedjars
6m 11s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
206m 56s

hbase-server in the patch passed.

235m 39s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6512

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 5db4a07cbdce 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 17b1825b5d96a03e2bf564235d042e16a108199b

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/testReport/

Max. process+thread count
4722 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 10s

Docker mode activated.

-0 :warning:
yetus
0m  5s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 48s

branch-2 passed

+1 :green_heart:
compile
0m 56s

branch-2 passed

+1 :green_heart:
javadoc
0m 29s

branch-2 passed

+1 :green_heart:
shadedjars
6m 30s

branch has no errors when building our shaded downstream artifacts.

-0 :warning:
patch
6m 41s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 22s

the patch passed

+1 :green_heart:
compile
0m 57s

the patch passed

+1 :green_heart:
javac
0m 57s

the patch passed

+1 :green_heart:
javadoc
0m 29s

the patch passed

+1 :green_heart:
shadedjars
7m 42s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

-1 :x:
unit
302m 55s
/patch-unit-hbase-server.txt
hbase-server in the patch failed.

334m 40s

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6512

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 5e96700b1839 5.4.0-195-generic #215-Ubuntu SMP Fri Aug 2 18:28:05 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 17b1825b5d96a03e2bf564235d042e16a108199b

Default Java
Eclipse Adoptium-11.0.23+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/testReport/

Max. process+thread count
4358 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 55s

Docker mode activated.

-0 :warning:
yetus
0m  5s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
2m 27s

branch-2 passed

+1 :green_heart:
compile
1m  6s

branch-2 passed

+1 :green_heart:
javadoc
0m 33s

branch-2 passed

+1 :green_heart:
shadedjars
5m 40s

branch has no errors when building our shaded downstream artifacts.

-0 :warning:
patch
5m 48s

Used diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 32s

the patch passed

+1 :green_heart:
compile
0m 48s

the patch passed

+1 :green_heart:
javac
0m 48s

the patch passed

+1 :green_heart:
javadoc
0m 25s

the patch passed

+1 :green_heart:
shadedjars
5m 54s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

-1 :x:
unit
311m 35s
/patch-unit-hbase-server.txt
hbase-server in the patch failed.

336m 50s

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6512

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux 1719702fc68c 5.4.0-195-generic #215-Ubuntu SMP Fri Aug 2 18:28:05 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 17b1825b5d96a03e2bf564235d042e16a108199b

Default Java
Temurin-1.8.0_412-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/testReport/

Max. process+thread count
4172 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6512/2/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.


Lock down distribution to a specific Xcode version

Ensures reproducible release archives.

Depends on https://github.com/MobileNativeFoundation/rules_xcodeproj/pull/2955.


wrong characters on response of stored procedures

When I execute stored procedures with node-mssql (tedious driver) the acentuation of characters are wrong..
Example:
The return of stored procedures are:
{ id: 1, "name": "OPERA€ÇO" }
But I want this return (the same of db):
{ id: 1, "name": "OPERAÇÃO" }
I am using SQL Server 2008 with collation SQL_Latin1_General_CP850_CI_AI and the wrong fields are varchar, not nvarchar.
I try search any encoding setting of node-mssql or tedious without success.
Can anybody help me to fix this?

Fix is pending to be relased here: https://github.com/pekim/tedious/pull/376
Until then you can use this workaround: select convert(nvarchar(max), sometextcolumn) from mytable.

Thanks @patriksimek!
The fix was merged and works like a charm. :rocket:


Remove redundant remote exchanges due to missing unaliasing

Removes redundant remote exchange from TPCH/q04

This also makes planning more deterministic as unaliasing after reorder joins behave differently depending if there are more predicates from table scan layouts that could be pushed down.

Benchmarks. One can see some overhead of join using dictionaries (q23_1, q23_2)
Benchmarks comparison-unalias.pdf

https://github.com/prestosql/presto/pull/2862 fixes most of the observed CPU perf penalty.


Difficulty using AsyncWrite

I have a compiler error when trying to use AsyncWrite. I reduced the code to the snippet below. What am I doing wrong?
error[E0599]: no method named `poll_write` found for type `IO` in the current scope
  --> src/main.rs:15:17
   |
15 |         self.io.poll_write(cx, buf)
   |                 ^^^^^^^^^^ method not found in `IO`
   |
   = help: items from traits can only be used if the type parameter is bounded by the trait
help: the following trait defines an item `poll_write`, perhaps you need to restrict type parameter `IO` with it:
   |
9  | impl<IO: futures_io::if_std::AsyncWrite + AsyncWrite> AsyncWrite for A<IO> {
   |      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

use futures::io::AsyncWrite;
use std::pin::Pin;
use std::task::{Context, Poll};

struct A<IO: AsyncWrite> {
    io: IO,
}

impl<IO: AsyncWrite> AsyncWrite for A<IO> {
    fn poll_write(
        self: Pin<&mut Self>,
        cx: &mut Context,
        buf: &[u8],
    ) -> Poll<futures::io::Result<usize>> {
        self.io.poll_write(cx, buf)
    }
    fn poll_flush(self: Pin<&mut Self>, cx: &mut Context) -> Poll<futures::io::Result<()>> {
        self.io.poll_flush()
    }
    fn poll_close(self: Pin<&mut Self>, cx: &mut Context) -> Poll<futures::io::Result<()>> {
        self.io.poll_close()
    }
}
fn main() {
    println!("Hello, world!");
}

Solved it! The missing piece was:
impl<IO: AsyncWrite> A<IO> {
    unsafe_pinned!(io: IO);
}


Propagate VF MTU to it's parent PF

Describe the bug
Setting of MTU over virtual function of sr-iov device may fail if MTU of the physical function is lower. It would be beneficial if the PF would be configured also with the max of the VFs MTUs
Nmstate version

nmstate 0.2.10 or greater.

NetworkManager version

1.22.10 or greater

Platform

All.

How reproducible

100%

Steps to Reproduce

Enable VF
Configure MTU of the VF to be higher than PF

Nmstate output

Expected output:
Fails with nmstate verification error

Current output:
It should pass and configure the MTU also for PF

Additional context
Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1856256

As this is a specific NIC/driver issue, I am closing this issue because Nmstate should not handle driver/NIC limitations. As I said above, this is working well with other NICs. Please, reopen it if you think we should handle this issue. Thanks!


Publish over NPM

Would please consider publishing this over NPM?
Lots of developers do use webpack or rollup to bundle their web apps and are not using <script src="whatever"></script> anymore :)

PHP doesn't work like that but there's composer 🥇

@nsuchy I have no idea what you're talking about.
Anyway I'm referring to this piece of code a developer has to include:
<script src='https://www.google.com/recaptcha/api.js'></script>

Which ships just the following:
/* PLEASE DO NOT COPY AND PASTE THIS CODE. */(function() {if (!window['___grecaptcha_cfg']) { window['___grecaptcha_cfg'] = {}; };if (!window['___grecaptcha_cfg']['render']) { window['___grecaptcha_cfg']['render'] = 'onload'; };window['__google_recaptcha_client'] = true;var po = document.createElement('script'); po.type = 'text/javascript'; po.async = true;po.src = 'https://www.gstatic.com/recaptcha/api2/v1517207450225/recaptcha__en.js'; var elem = document.querySelector('script[nonce]');var nonce = elem && (elem['nonce'] || elem.getAttribute('nonce'));if (nonce) { po.setAttribute('nonce', nonce); }var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(po, s);})();

Which would be nice to be included with a simple:
npm add recaptcha

The repository is the PHP Client for validating responses. As for the client side stuff, there is an npm package for verifying the captchas in a node.js app (See: https://www.npmjs.com/package/recaptcha), there's also https://www.npmjs.com/package/react-recaptcha for client side.

Oh I see: thanks for clarifying.
By the way neither of them is packaging the official script, they seem to both require you to include that manually.
Do you have any idea who is mantaining this hosted file https://www.google.com/recaptcha/api.js?
Or which repo is hosting it?

The reCaptcha Team at Google Maintains it, as for repo it's a client side API, I don't think that part is open sourced, I may be wrong.

Who may I ask to have the script being "officially" published over NPM? Any contact? 😅

@damianobarbati I'm not a Google employee nor do I know any, I'm not sure if there's an official contact for this. The issue is probably the only thing you can do.

Thanks for the help anyway @nsuchy , closing this :)


Server Side Blazor support

Hi Chris,
Does this component work with server side blazor and if so can you update your sample.
Love your work.

Hey, it should work with server-side but will need to be updated first. I do have it on my todo list but just haven’t quite got there yet.
Always happy to take a PR if you want to have a crack at it.

Just upgraded the package to Preview 7 so this should all work on server-side now.


Run Rasa X on HTTPS

Hi,
I enable https of rasa core with the below command and rasa core is now running on HTTPS.
rasa run --ssl-certificate <.crt> --ssl-keyfile <.key> --enable-api -m models/
https://localhost:5005
But when I am trying to do the same stuff with rasa X
rasa x --ssl-certificate <.crt> --ssl-keyfile <.key>
then still its showing running on http://localhost:5002 not on HTTPS.

hi @kapoorh, running rasa x on SSL is currently only supported on Rasa X EE. https://rasa.com/docs/rasa-x/rasa-x-changelog/#id28

@kapoorh  check out step 9 here for how to put everything behind https https://rasa.com/docs/rasa-x/deploy/ - you just need to provide some certs. What @erohmensing says is right but that's only if you want SSL termination inside the container. In a normal Rasa X setup you have nginx acting as a proxy and that can hold your ssl certs

Hi @amn41 , @erohmensing ,
Thanks for the quick reply.  Rasa X CE doesn't handle SSL certs

Thanks Alan!!
I think this feature should be in Rasa X CE version as well :)


RKNN Model Zoo

Hi friends.
Do you know whether there is any model zoo or converted model for RKNN or not?
for example yolov3.rknn

Hello!
1.For yolov3, it's converted from darknet model, trained by using https://github.com/pjreddie/darknet.
2.Yeah, we had got a plan to push a respository, aiming to claim how to train/test/eval/deploy the common model on rknn-devices. The yolov5 would be the first one to support. It's supposed to come out in the middle of this month.

Hello! 1.For yolov3, it's converted from darknet model, trained by using https://github.com/pjreddie/darknet. 2.Yeah, we had got a plan to push a respository, aiming to claim how to train/test/eval/deploy the common model on rknn-devices. The yolov5 would be the first one to support. It's supposed to come out in the middle of this month.

we will be very thankful dear @zen-xingle
I have another idea and deeply appreciate if you consider.
unfortunately there is no straightforward document to run the models with NPU on development board.
for example at first I thought installing the RKNN_Toolkit is enough to use NPU but spent two days to know the driver plays an important role.
I'm just wondering would it be possible to enrich the Installation part of User Guide's document for beginners and help the users step by step after OS installation?

Sorry for the confusion in using RKNN devices.  The new respository will also contains the deployment code as C++ demo. And the EN doc will also be updating.
For driver,
1.The driver is deeply binded to the system, meaning that different third party development boards may got imcompatible. And the best way is to ask the seller(third party) for updating driver.
2.If you are using development board mamufactured directly from RK, updating driver version(for rk1808/rv1109/rv1126) refer to here https://github.com/rockchip-linux/rknpu.
For deployment example, refer to here https://github.com/rockchip-linux/rknpu/tree/master/rknn/rknn_api/examples. This should work well with all boards.

Sorry for the confusion in using RKNN devices. The new respository will also contains the deployment code as C++ demo. And the EN doc will also be updating.
For driver, 1.The driver is deeply binded to the system, meaning that different third party development boards may got imcompatible. And the best way is to ask the seller(third party) for updating driver. 2.If you are using development board mamufactured directly from RK, updating driver version(for rk1808/rv1109/rv1126) refer to here https://github.com/rockchip-linux/rknpu.
For deployment example, refer to here https://github.com/rockchip-linux/rknpu/tree/master/rknn/rknn_api/examples. This should work well with all boards.

thanks for for your responsibility to answer the comments. your comment for the driver installation would be very worthy.
we are being on watch to get updated document and C++ api


Register Package

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/90784
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.0.1 -m "<description of version>" c4f327083234965a57b0f340db0f425c71b41534
git push origin v0.0.1

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/90907
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.0.1 -m "<description of version>" e0d5488576228e701f900e40b1d69849ab25037f
git push origin v0.0.1


.Net Core Console connect signalR 401 under windows authentication

Describe the bug
When .net core signalR console app connect signalR server with Windows Authentication, receive 401.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core netcoreapp2.1 with Windows Authentication enabled.

    [Authorize]
    public class TimeHub: Hub
    {
        public async Task UpdateTime(string message)
        {
            if (Clients != null)
            {
                await Clients?.All.SendAsync("ReceiveMessage", message);
            }
        }
    }

.Net Core Console App

    class Program
    {
        static Action<string> OnReceivedAction = OnReceived;
        static void Main(string[] args)
        {
            Console.WriteLine(WindowsIdentity.GetCurrent().Name);
            //WindowsIdentity.RunImpersonated(WindowsIdentity.GetCurrent().AccessToken, Connect);
            Connect();
            Console.WriteLine("Hello World!");
            Console.ReadLine();
        }

        private static async void Connect()
        {
            var hubConnectionBuilder = new HubConnectionBuilder();
            var hubConnection = hubConnectionBuilder.WithUrl("http://localhost:61045/timeHub",options => {
                options.UseDefaultCredentials = true;
            }).Build();
            await hubConnection.StartAsync();
            await hubConnection.SendAsync("UpdateTime", $"From Client");
            var on = hubConnection.On("ReceiveMessage", OnReceivedAction);
            Console.WriteLine($"Client is Start");
            Console.ReadLine();
            on.Dispose();
            await hubConnection.StopAsync();
        }

        static void OnReceived(string message)
        {
            Console.WriteLine($"{ message }");
        }
    }

Expected behavior
Console App connect signalR server correctly.
Current behavior
Receive error 401 at console app.
Update:
After making a test with https and http url in client signalr, it works with https url, it seems  app.UseHttpsRedirection(); at server side fail to redirect the client credential while redirecting http to https.

Is app.UseHttpsRedirection() at the beginning of your middleware pipeline? Can you share your Startup.cs? Can you capture the server logs at Debug level?

Take a look at our docs for windows auth https://docs.microsoft.com/aspnet/core/security/authentication/windowsauth?view=aspnetcore-2.2
Once I added the services.AddAuthentication(IISDefaults.AuthenticationScheme); because the apps logs said "No authenticationScheme was specified, and there was no DefaultChallengeScheme found." then the app started working. I hosted it behind IIS, I'm assuming that is what you're doing, otherwise you'll need to use Http.Sys and configure it based on the docs.

Is there any difference between hosting in IIS and IIS Express?
Currently, I am developing with VS and IIS Express, I did not receive any error related with No authenticationScheme was specified. Even through, I add code services.AddAuthentication(IISDefaults.AuthenticationScheme);, it still throw 401.
Do you make it work with my demo project?

I made your demo project work with hosting in IIS, didn't try with IIS Express, but the docs I linked also talk about IIS Express.

How to make my project work with IIS Express?

var connection = new HubConnectionBuilder()
    .WithUrl("<SOME ENDPOINT>", options =>
    {
        var handler = new HttpClientHandler()
        {
            Credentials = /*CONFIGURE YOUR CREDENTIAL CACHE*/
        };

        options.HttpMessageHandlerFactory = (_) => handler;
    }).Build();

The code would look something like this.
You can look at https://docs.microsoft.com/en-us/dotnet/api/system.net.httpwebrequest.credentials?view=netframework-4.7.2#remarks and https://docs.microsoft.com/en-us/dotnet/api/system.net.httpwebrequest.credentials?view=netframework-4.7.2
For more context on the credentials property.

Closing this as we haven't heard from you in a little while. Please feel free to comment if you're able to get the information we're looking for and we can reopen the issue if we're able to identify the problem.


Refactor code to combine parseMedicalHistory and parseMedical as both…

… share the same functionality
Resolves #79

This refactoring is a good change! LGTM.

The new commit addresses the problem of redundant test cases.

LGTM!


Parameterize testing credentials repo

In build-package.sh, we have the name of the testing credentials folder hardcoded to Kitura-TestingCredentials. Instead, this should be an optional parameterized value coming from the travis.yml file. We should then update the build-package.sh script accordingly. Note that this change will require PRs against the dependent repos.

@rolivieri Shmuel merged my PR.  We can do a code review on the parameterize branch

@BDHernand Just saw this. It looks good. Feel free merge this PR, thanks!


Jonch/algoptminusglv

sorry, I did a hard fork of the wrong parent branch. So it wiped out the ability to compare changes since the review comments. The only comments I don't think have been addressed yet are the criterion based tests. It's a lot of work to overhaul the whole thing so I suggest it be done in a future PR.

sorry, I did a hard fork of the wrong parent branch. So it wiped out the ability to compare changes since the review comments. The only comments I don't think have been addressed yet are the criterion based tests. It's a lot of work to overhaul the whole thing so I suggest it be done in a future PR.
Further, I don't see the issue with benching on nightly, and recall criterion was unsuitable for ops on the order of < 1us due to the long excessively long benching times, without tweaking.

Yeah, we can definitely leave that for a future PR.
Benching with nightly slightly makes me nervous since that's not what we'll be running eventually.

Since this has essentially become "stale" and the GLV branch is the main one, should we close this?


Make the temporary folder where files are saved configurable

This is another thing I needed to fix on my fork.
On AWS I can't write on the filesystem, and the folder ./games is hard coded.
I changed that to /tmp on my fork.
Would /tmp work for your use cases?
Or you want to support windows users?
What about a env var for overriding your default value ? Like TEMP_GAMES_FOLDER?
Or... do you really need to download the file to re-upload it?

This should be easy enough @maurelio1234. I'd actually be pretty happy if we could avoid downloading the file but for some reason I couldn't figure out how to serialize the game properly in memory and upload it as form data, so that was the "get it working" solution. For now I'll go ahead and make this configurable though and could look at a better solution later.

@maurelio1234 went ahead and made this change on main, you can check it out here https://github.com/andrewallred/allez/commit/e0f9602e10a906afe6793b2060104e4033f16aeb


Draft: Copies some of the install no-binary code over to compile (#4064)

Summary
This i still a draft, but it gets some of the work started on issue #4064 , which requests --no-binary functionality similar to uv pip install to exist on uv pip compile.
So far this has moved the command line shape from install and cloned it over to compile. The actual functionality of respecting --no-binary  and generating the resulting line in requirements.txt I could use a hand with.
My understanding is we want to create a requirements.in file such as:
yt

Then when running  cargo run -- pip compile --no-binary yt requirements.in
we want the file to have this line in it:
yt==4.3.1 --no-binary yt

Test Plan
Existing unit tests continue to pass.
No new unit tests have been created yet.
The new command line options do show up when testing with cargo run -- pip compile -h

I think this is actually "done". I'll add a test to demonstrate it.


Using ./designer.sh does not install apps contained in "applications-designer.sh"

Follow instructions up to and including $ ./designer.sh
Let script running, inputting password when prompted
Once workstation setup is complete, check installed applications for those contained in applications-designer.sh
Applications are missing

It might be because you're looking in /Applications and that's not where they're getting installed.
Try running brew linkapps and see if that helps. It makes symlinks in /Applications

Or wait, I'm not sure if brew linkapps affects cask-installed apps, it might only affect regular brew-installed apps.

It was a bad bash string equality check. Eric and I fixed it in the most recent commit.


lecture0 video and lecture 1

Add 2 videos about lecture 0
Add markdown about lecture 1

double checking: the videos are for sessions 2 and 3. there is no session 1, right?

prob we can remove the audio? the audios are too many, and hard to order.  I think video is sufficient. Also maybe we want to refactor the repo a bit


Internal Error: cURL error 6: Could not resolve host: api.telegram.org

Hello!
Since 17th of July have this Error which leads to no notification via telegram. Also it gives error when you're making anything in KB (due to trying to send notification, I believe).
Full error:
Internal Error: cURL error 6: Could not resolve host: api.telegram.org (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://api.telegram.org/bot6485639367:AAESXxjw0UzcNfDvuPinES3U0oDFy_M0We8/sendMessage

Sorry, that was my docker issue. Restarting helped.


404 in README.md

Hello,
I noticed this line [Style guide](http://roykolak.github.com/chrome-bootstrap/) in the README leads to a 404 page.
Just wanted to let you know.

Thanks for bringing this to my attention! Going to use the github alias though in case that domain ends up changing


Create note from opportunity fails

Looks like the CreatorId is missing
{
"errorCode" : -2,
"errors" :
[
{
"descriptor" : "Required value is missing",
"errorCode" : -2,
"path" : "CreatorId",
"scriptName" : "Create Note { data }",
"type" : "JSON.Validate"
}
],
"type" : "JSON.Validate"
}

Looks like the user record wasn't found on login. We should be checking for a user record on login.


Set up cloud dev environment

[x] Select cloud platform for app
[x] Set up build pipeline from GitHub
[x] Configure super-admin for team's use

Just to lay out my plan for this task.
Database
I think I'm going to try and set up mongodb stitch and have our application access that db server. We can use dedicated add-ons with Heroku, but they all cost money per month. The former option is a pay as much as you use option.
Pipeline
Heroku has Heroku pipelines, which allow us to easily setup up multiple application instances together. At the moment, we just need to authorize access to our GH organization to finish this step.

Checklist item number 3 should be complete once helloGov/deploy#9 is merged.

I've enabled automatic deploys on Heroku from the master branch. We should test this out to see if everything is hooked up correctly.

This is happening, closing


sbteclipse-plugin dependency is very old

I'd suggest either updating the dependency to 4.0.0 (current version recommended at https://github.com/typesafehub/sbteclipse as of 25-Oct-2015)...
...or removing it entirely, and just let people add the plugin in their global sbt settings if they are Eclipse users

agreed, this can be added via global plugins for people who prefer eclipse

fixed in c85dcc0cbfa2af53b6c78b0464c068e39c990282


Fix optimistic reports getting cleared by unnecessary OpenReport calls

Details
Fixes issues with failed optimistic reports getting cleared immediately on reconnect. This prevents helpful error messages from being hidden.
Fixed Issues
$ https://github.com/Expensify/App/issues/36928
PROPOSAL: https://github.com/Expensify/App/issues/36928#issuecomment-1984335946
Tests
Web and Desktop only

Log into New Dot on two devices, user A and user B
As user A, go offline and request money from user B

Don't use dev tools. Use the Force offline toggle in Settings > About > Troubleshoot

Verify an offline, optimistic request is created
As user B, go online and request money from user A
Verify the money request is successfully created
As user A, go back online and quickly navigate back to the optimistic report
Verify the report displays an error about a previously existing chat between these users

Offline tests
Same as Tests
QA Steps
Same as Tests
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

This is ready for review however I did have trouble testing on mobile web and native since it's difficult to navigate back to the chat before the network call fails. I skipped those and just tested web and desktop but feel free to try.

Been trying to test this, the offline report that I create goes blank, and hence when I come back I see

If I press back, I can see the chat momentarily but it hides abruptly.

It seems that there's a race condition between the API.write for the money request and dismissing the modal here. We navigate back to the report screen and queue OpenReport before the optimistic report gets merged into Onyx. This is why the report is empty and we can't tell that it's optimistic before calling OpenReport.
I'm not sure why this is suddenly an issue now and not before. Maybe the response from OpenReport for a non-existing report needs to stop being so destructive so that we can still show helpful error messages. I'll circle back on this later and maybe start a discussion about it.

Alright new approach is detailed here. I've updated this PR to handle errorFields.notFound. I'll also leave the shouldFetchReport improvements since those are good regardless.
@mananjadhav this is ready for review. However, this needs to be deployed before the backend changes are deployed so that we do not start setting errorFields.notFound before NewDot can handle it. That means you cannot test the offline money request flow at the moment. Instead Applause will QA when the backend changes are deployed.

Okay thanks for the update. I'll get it reviewed.

There's some conflicts to resolve @arosiclair

@amyevans updated!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/amyevans in version: 1.4.56-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.4.56-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅