Support of ruby 2.4.0

Opentok gem v2.3.4 depends httparty (~> 0.13.1),
which depends on json (~> 1.8),
which cannot build native extensions with ruby 2.4.0

Try upgrading to json 1.8.6. Worked for me.

Thank you, json 1.8.6 works just fine


Ability to list permissions for a User or set of Users

I'd like to be able to generate a list of assigned permissions for a given user or set of users.
Additionally, it would be good if the effective/inherited permissions could be generated, although this is not immediately required.
Currently the first part of this requirement is performed using an SQL query

Can this be closed or are you after something in the UI to list all this?


Handle a dot character in project name correctly

A dot is not interpreted corrected in project names. For a project name like 'abc.def', the generated dll is 'abc.dll' instead of 'abc.def.dll'. Affects the output of both static and shared libraries. Haven't been confirmed for applications.

I think you are saying these methods 'getSharedLibraryName(String libraryPath)' , 'getExecutableName(String executablePath)' and others like it need to have 'withExtension' method removed.   These methods are all in 'org.gradle.internal.os.OperatingSystem', and are what is getting used at execution time for the VisualCppPlatformToolProvider on windows, and not the ones in 'org.gradle.nativeplatform.toolchain.internal.UnavailablePlatformToolProvider'.  I have found the only scenario that uses the 'withExtension' method is for Windows/Vs variant.  Other tool chains, and OS's aren't affected by the 'withExtension' method to my knowledge.
This easiest solution is to remove calls to 'withExtension' altogether coming form the OperatingSystem class, and just return what is passed into it, /libname + ".some-extension", because that is essentially what 'withExtension' did previously, only it returned a different result if you had a '.' in the path passed in.  I can put together a PR on this, and you guys can either take it as is, or massage it into a more strongly modeled approach.  If I change the abstract method signatures on 'OperatingSystem', this results in a much bigger change having to have the signatures changed in every method upstream that calls it, so to minimize that, I can just drop the 'withExtension' method which really only impact the Windows/Vs variant.

https://github.com/gradle/gradle/commit/b64d51b4fe6797b2ea20db75b9c20503bf2e4139 wasn't merged, after all?


Could not find files for the given pattern(s)

I'm getting the following when I run the snatcher;

<EMAIL_ADDRESS>start C:\nvidia-snatcher
npm run build && node build/index.js

<EMAIL_ADDRESS>build C:\nvidia-snatcher
rimraf ./build && tsc

INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
[7:07:38 PM] info :: ✖ [nvidia] still out of stock: nvidia founders edition
This is my .env setup;
DESKTOP_NOTIFICATIONS="true"
EMAIL_USERNAME=REDACTED
EMAIL_PASSWORD=REDACTED
HEADLESS="true"
IN_STOCK_WAIT_TIME="10"
LOG_LEVEL="info"
OPEN_BROWSER="true"
PAGE_TIMEOUT="30000"
PHONE_NUMBER=REDACTED
PHONE_CARRIER="google"
PLAY_SOUND="notification.mp3"
PAGE_SLEEP_MIN="5000"
PAGE_SLEEP_MAX="10000"
SHOW_ONLY_SERIES="3080"
STORES="bestbuy,bandh,nvidia,newegg,evga,asus"
SCREENSHOT="true"
USER_AGENT=REDACTED
I'm like 90% sure I installed everything correctly as the instructions are pretty clear. Any insight?

Make sure to rename the .env-example to .env
I was getting the same error and realized I forgot to do that. After changing it, it gave the INFO:"" error but then began working like it should.

You must have a command line audio player supported by the library that's being used to play the sound. Others have reported success with mplayer and mpg123. You can place it in the same folder as the program.
Duplicate of #116

Hm, even without the sound its still not providing updates for any of the websites other than Nvidia itself.

This is an issue with your .env. If the bot thinks it's missing, it'll default to nvidia only. Check the extensions of your files; make sure .env isn't really .env.txt or .env.env.

The issue is that your file named '.env' has a .txt extension to it (you can tell based off of the file type). Click on View at the top of the explorer window and ensure you have 'File Name Extensions' checked off under the show/hide section - once that's done you should see the .txt extension that's appended to the .env file. Simply delete the .txt off the end and that should do it.

The issue is that your file named '.env' has a .txt extension

We really need to write up something in the wiki.
Thanks for the help y'all. Going to close.


Solves the problem of Flask-User missing actual runtime information

when working behind a reverse proxy (i.e. nginx serving static content
or handling HTTPS traffic).

by adding a USER_APP_ROOT_URL customization value
by generating USER_APP_ROOT_URL + url_for(, _external=True) in
links when the customization value is available
by adding USER_APP_ROOT_URL  + url_for(, _external=True) in
emails when the customization value is available

Hi @jcborras! Thank you for your contribution! I'm sorry you had to wait for an answer for so long. Could you please rebase your branch over actual master?


capm_returns function should not amend the prices dataframe

The capm_returns function should not amend the prices dataframe when the returns_data is True.
Currently, if returns_data is True, the returns dataframe is equal to the prices dataframe. A new column is added for the market returns as "mkt" which impacts as well the prices dataframe.
This behavior is not expected and the prices dataframe should be copied before.

Good spot, will push this with the next patch


Support for nginx websocket proxying via environment variable

Similarly to setting up static routes it should be possible to set up websocket proxying via an environment variable.
https://www.nginx.com/blog/websocket-nginx/
upstream websocket {
    server <IP_ADDRESS>:8010;
}

location /wsurl/ {
    proxy_pass http://websocket;
    proxy_http_version 1.1;
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection "Upgrade";
}

I think it's possible these days to have an app serve both HTTP and WS with the same server & port in which case the URL can default to / and the port & IP will be the same as the default http config (so the setting could be on/off for websockets).
@rcarmo I frequently need this for projects, so I will probably take a shot at implementation at some point in the future if nobody else beats me to it.

@rcarmo it turns out you already have all of these websocket upgrade directives in the default nginx config. I've just tested a nodejs server running under piku with websockets enabled and they work great so I'll close this. I'll also add a nodejs example once I've fixed a couple of bugs.


Windows Installer missing

I cant find the windows installer. Neither in Ripple nor Ripple-V2. I think it's missing?

Hi @janimango , please check this comment.
Hope it may helps

Thank you very much! I already found this comment, but unfortunately this mail is no longer available.

Here is the link for the compiled binaries and some examples that I have received back then.
https://onedrive.live.com/?cid=fb403fd52c470b65&id=FB403FD52C470B65!6609&authkey=!AKE-DNbbEmaCc20
Good luck...


Experiment integration tests should limit query to qiskit_test only

Information

Qiskit IBM Provider version:
Python version:
Operating system:

What is the current behavior?
A follow up to #112. Tests in TestExperimentServerIntegration should always use experiment_type='qiskit_test' as a filter whenever applicable.
Steps to reproduce the problem
What is the expected behavior?
Suggested solutions

For this I'd probably write a helper function which takes whatever args/kwargs should be passed through to list experiments and it would check if experiment_type isn't in kwargs it'd be added and filter on qiskit_test. The same might be done for list analysis results (would have to check to see how tests are creating and listing analysis results).


feat: Filter Query Browser

Closes #158. (Its not really interactive tho)
Notes:

I'd appreciate if someone can implement the search component, if that is needed. As @roziscoding have offered to help, I guess he is ready to implement one 😄?
@KnorpelSenf should this be ignored algolia? If so, can you update the configuration?
I'm not sure how the generated files should be ignored by git, if they have to be.
Consider helping improving the docs part please.
To the translators, you can see the translations/en.ts file to know how the translations should go. And they need to be imported in translations/mod.ts. That's it.
The prefixDocs in en.ts translation is only a demonstration. I think its nice. Can be removed if its not really useful. If it is useful, consider adding more to the list. @KnorpelSenf, if you could choose a right set of filter queries that needs additional explanation, that'll be useful.
This is not actually a plugin. Can be moved to another directory (tools?) if needed.

Feedbacks welcome.

Please be transparent about why this was closed. DMs aren't known publicly.

Oh, sorry. I did it again.

Closed because we decided to move it to a separate webpage because its a tool, and not a part of the actual documentation. This repository should be only for documentation.


Can this plugin be used without TypeScript?

Hello, it says in the readme that "Most rules can be used both for JavaScript and TypeScript". I assumed rules like no-let should work fine,  but I get a Cannot find module 'typescript' error. Please advise!

Which version of this plugin are you using? This plugin should be usable without TypeScript from version 0.2.1 onwards.
I haven't personally checked yet to see if it is in fact working as intended. If your using the latest version and receiving that error, let me know and I'll look into the issue.

I just tested this with v0.4 and I got that same error. I tried again with the master branch and didn't get any issues. I'm about to make a new release. Could you try with that and see If you have any issues?

Hey, thanks for checking! I installed 0.5 and till getting Error: Failed to load plugin 'functional' declared in '.eslintrc.js': Cannot find module 'typescript' unfortunately. It works fine if I install typescript.

Looks like this is an upstream issue with @typescript-eslint/typescript-estree.
I've created an issue for this.
As a work around for now, I'd recommend installing typescript as a devDependency.
npm install typescript --save-dev

@yosevukilonzo, I've managed to modify the build process so that @typescript-eslint is no longer a dependency. In doing so, typescript will no longer be a peer dependency.
From the next release, things should now just work for vanilla JS out of the box.


examples: add blecent and bleprph similar to mynewt-nimble apps

Summary
These example apps create a BLE central and peripheral based on mynewt-nimBLE and work in companion. The apps are mostly based on their equivalents, which can be found at https://github.com/apache/mynewt-nimble/tree/master/apps
Impact
Testing
These were successfully tested on two nrf52840-dk boards.

why not modify the current nimble peripheral example (https://github.com/apache/nuttx-apps/blob/master/examples/nimble/nimble_main.c)? do we need 2 different examples for this?

@maxikrie Mynewt The mynewt coding standard is far from the NuttX coding standard, so the code will be a lot different anyway. Keeping 2 examples demonstrating the same functionality seems pointless to me. The easiest thing to do would be to change the name of existing examples/nimble to examples/nimble_bleprph (or nimble_prph) and add all your changes there.
Or you can delete examples/nimble but then you will have more work to adapt the Mynewt code to the NuttX coding standard.

CI checkpatch is now fixed, CI restarted :-)

Source code still needs reformat, @maxikrie  please take a look at:

https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md
https://nuttx.apache.org/docs/latest/contributing/coding_style.html

@cederom Is there a way to check whether my changes will pass the check before committing? I read the documentation on pre-commi, but it seems to that this only applies to the nuttx repository - at least this repository is missing the required .pre-commit-config.yaml.
I have installed pre-commit in the nuttx repository and tested it on branch nuttx-12.7.0, where everything is fine. I then naively tried from within the nuttx repository:
pre-commit run --files ../apps/*
which gives all sorts of strange errors.

@cederom Is there a way to check whether my changes will pass the check before committing? I read the documentation on pre-commit, but it seems that this only applies to the nuttx repository - at least this repository is missing the required .pre-commit-config.yaml.
I have installed pre-commit in the nuttx repository and tested it on branch nuttx-12.7.0, where everything is fine. I then naively tried from within the nuttx repository: pre-commit run --files ../apps/* which gives all sorts of strange errors.

please run the following command to check the style:
../nuttx/tools/checkpatch.sh -g HEAD~...HEAD

ping @maxikrie


Bumped versions for KeyVault Track 1.

Should #11046 be merged first?

@alzimmermsft Yes, #11046 should be merged first.

Waiting on approval for #11045 before merging this.

/azp run java - core - ci


Greek question mark ;

The Greek question mark is ;  not ?. Although informally (on Internet) ? is commonly used too. Additionally, more emojis would be great.
Btw thank you for this keyboard (:

According to this website the Greek question mark is the same character as the Latin semicolon. I can also see a character called "middle dot" or "dot above the line" is used in Greek, but TT9 does not support it.
Given that ; is already in the list, it means I only need to replace the Latin ? with Greek ; and the Latin ; with Greek ·, right?

Additionally, more emojis would be great.

This is not really on the priority list. However, you can add your own emojis just like you can add new words. Find the one you need on the Internet or send it to yourself from a computer, then hit the Add Word button. See more info in the manual for v31.0.

Btw thank you for this keyboard (:

Enjoy and invite all your friends to the party!

Given that ; is already in the list, it means I only need to replace the Latin ? with Greek ; and the Latin ; with Greek ·, right?

Exactly!

... However, you can add your own emojis just like you can add new words.

Thanks!  Exactly what I needed. A nice feature would also be if via specific words it autocompleted emojis.

Enjoy and invite all your friends to the party!

🥳


Suggest to modify Step 5 with an expression instead of a direct mapping

Hello,
Step 5 suggests to create a direct attribute mapping flow on the outbound to AAD. While this works, it assumes that the on-prem AD attribute is populated with a valid string value. The documentation does say to ensure that it is populated. However, in scenarios where the on-prem attribute is NULL, the export will flow successfully but then fail on subsequent imports from AAD (export confirmation) because AAD does not allow null UserType values. Staging will not catch this because it does not export changes.
I would suggest writing an expression instead where it only flows only if a string value of "Guest" or "Member" are present in the AD attribute of choice. Hope this makes sense.

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

ID: f469d121-1cda-1a60-97ba-6940502cd4cc
Version Independent ID: 03294c05-5c85-24a5-6334-cea158abda0a
Content: Azure AD Connect sync: Make a configuration change in Azure AD Connect sync
Content Source: articles/active-directory/hybrid/how-to-connect-sync-change-the-configuration.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@josuvil
Thanks for your feedback! We will investigate and update as appropriate.

@josuvil
Thank for submitting this. I have created a work item for it and will reviewing this in the coming weeks. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


Logging and constants cleanup

Fixes #112 #53
Cleanup constants and logging modules inside tt_silicon_driver.cpp

LOG1 -> log_debug
LOG2 -> log_trace

Other prints were straight 1-1 mapping to logger prints (log_warning, log_error)
Removed unused constants

I'd rather type UMD_INFO("... or UMD_WARN("... than log_debug(LogSiliconDriver, "....  Whatever LogSiliconDriver represents (module name?) I don't think is useful for a project the size of UMD.

But this way we're using the same logging infra as the other part of our stack? I'd vote for uniformity, rather than each component having their own logging.


build opencv with  tesseract

Tesseract Version: 5.0.0
Platform: windows10 64bit

i builded Tesseract with this method [ Develop Tesseract ] but when try build opencv, cmake can't find Tesseract :

Develop Tesseract :

Download and install Git / CMake / SW and put them in PATH.
sw setup
git clone https://github.com/tesseract-ocr/tesseract tesseract
cd tesseract
mkdir build && cd build
cmake ..
run "tesseract.sln" as administrator -> build "ALL_BUILD / INSTALL" projects as "Debug / Release" mode
add "install folder" to PATH ( Environment Variables )

*) Build OPENCV with cmake but But tesseract is not .... ( Picture above )

https://github.com/tesseract-ocr/tesseract/issues/3636

take a look at https://answers.opencv.org/question/171470/opencv-ocrtesseractcreate-v305/

Opencv does not seem to have a problem with tesseract 4.1.3
According to what I understood from the link above.
How can I use version 4.1.3 in this method [Develop Tesseract]? Instead of the latest version?
This code downloads the latest version [5.0.0]
git clone https://github.com/tesseract-ocr/tesseract tesseract
To download version 4.1.3, I used this code:
git clone --branch 4.1.3 https://github.com/tesseract-ocr/tesseract tesseract
When I run this code "cmake ..", it shows the following error:

CMake Error at C:/Program Files/CMake/share/cmake-3.22/Modules/FindPkgConfig.cmake:592 (message):
A required package was not found
Call Stack (most recent call first):
C:/Program Files/CMake/share/cmake-3.22/Modules/FindPkgConfig.cmake:814 (_pkg_check_modules_internal)
CMakeLists.txt:221 (pkg_check_modules)

cmd full code : cmd.txt


[autopaste] Disallow thread author from deleting the message

This PR changes the delete message view interaction check to disallow the thread author from deleting the paste message and instead allow the help mod to do so (only in help threads).
This is tested.

Can this be changed to allow no deletions in help threads at all?

Can this be changed to allow no deletions in help threads at all?

That would be unfair for users that don't want the annoying message for every codeblock they send in the threads.
I think this should only be for the author since they are usually the ones sending the imported stuff.


[SPIKE] FE + BE Interface

Description
As part of our Tech Modernization initiative, we should look into how to ensure API consistency and design - specifically, how to communicate between backend and frontend by means of structured endpoints with well-defined models.
tRPC has been suggested as an option - we should look into this as a possibility!
Acceptance Criteria

A decision record has been written within ./docs/decisions documenting the decision

@TylerHendrickson  this is marked as "ready" - is it ready for eng to look into further? not clear from the ticket what has been done/is remaining


[Question] Why not releasing v3.6.4 with Laravel 7 support?

Many projects, including our back-ends use this MongoDB package with Laravel. We would like to update to Laravel 7 with the latest features and other dependencies, but this package holds us back.
Is there any change you guys can release v3.6.4 with the already fixed Laravel 7 support (current develop branch?)? Then we can all upgrade to Laravel 7, while you guys work further on version 4 of this package.

Hi,
I'm really sorry but we can't simply do that without completing new version.
There is a huge problem with Laravel itself that we couldn't fix really and I'm still waiting for @Smolevich to check it up.
See https://github.com/laravel/ideas/issues/856
TLDR: there is no point of creating casted class while it doesn't respect on querying a model.
That means  "Auto-casting ObjectID" and "Auto-casting UTCDateTime" movement to casted classes that was introduced in 7.0 just vanished.
Taylor just doesn't care (no plans on adding this feature) and I've just gave up on this, so hopefully @Smolevich will have free time and  will check what could be done.
Might be a solution is adding a "fix" to respect casted objects while querying on model.
Thanks.

Also regarding v3.6.4 this will not happen at all, new version contains a breaking change so next release will be v4 with Laravel 7 support.

But, correct me if I'm wrong, but in all issues i see here, complaining about Laravel 7 support, everyone is saying the current develop branch is working fine. So, why not then?
Now I read it again, you mean the breaking change is the Laravel version support? What about a v4.0.0-alpha version from the current develop branch? That way everyone knows it could break somewhere without support, but we can install it with a fixed version and codebase, without "random" updates on the develop branch. Even the new issue's you could link to this alpha version if it is Laravel v7/pacakge v4 specific.

I want to need investigate question with auto casting as I can find free time, probably in next 1-2 weeks
@hylke94 We can create alpha release but we can check that it works with laravel 7

@Smolevich I already can't wait for this alpha release ;)

Any update for this?

Any update for this?

Please, kindly try reading again, might help hopefully.

Any update for this?

Please, kindly try reading again, might help hopefully.

I appreciate that you guys wanna take your time to check it all. But on the other hand, everyone here is reporting in different issues that develop works without any problems..
Now, I don't want to push you guys, but an alpha release isn't that big of a deal (I guess, didn't have to do it myself yet). It's an alpha, so theoretically not ready for production, but a risk I will take for our new internal applications. But I would want a tag/version to rely on, instead of the develop commits.
No pressure tho ;)

My current practice is to clone the dev version into my project, which I can only do

@hylke94 i released alpha version https://github.com/jenssegers/laravel-mongodb/releases/tag/4.0.0-alpha.1
test and write about bugs please
I continue to work on stable version

@Smolevich I will try it first thing tomorrow! Thanks!

Something new about Laravel 7 support? :)

Something new about Laravel 7 support? :)

The alpha version is compatible with Laravel 7. Have it running with great success here for the last two weeks!


Entities become unavailable before update

Is there an existing issue for this?

[X] I have searched both the existing open issues & recently closed issues and did not find a duplicate of this issue.

Version of the Tesla component
3.24
Version of the Powerwall Gateway software
24.20.0 c128851f
Model
2
Current Behavior
All eblntities show as unavailable for 10 seconds before updating back to normal.
Expected Behavior
Expect old value to stay until new value available
Debug logs
Not required

Anything else?
No response

This is an example of what I see.


Fix onScroll event type

Summary
Fixes type of the event of the onScroll function.
Issue: #1627
Compatibility

OS
Implemented

iOS
✅

Android
✅

Checklist

[X] I have tested this on a device and a simulator
[X] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md (Didn't find CHANGELOG.md in the repo)
[X] I updated the typed files (TS and Flow)
[X] I added a sample use of the API in the example project (example/App.js) (Example already existed)

Still waiting for review and merge 😢

@jamonholmgren @Titozzz This seems like a straightforward types/documentation fix :)

Still waiting for review and merge 😢

Sorry 😭


Make the title more specific when cloning one blob container

Storage Explorer Version: 1.11.2
Build: 20191215.1
Branch: hotfix/1.11.2
Platform/OS: Windows 10/ Linux Ubuntu 18.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one Non-ADLS Gen2 storage account -> Blob Containers.
Select one blob container and right click it -> Click 'Clone with New Name...'.
Check the dialog title.

Expect Experience:
Show 'Clone Blob Container with New Name'.
Actual Experience:
Show 'Clone'.

More Info:

This issue also reproduces for one file share.
This issue also reproduces for one file/ folder under one file share.

Per discussion with Jing, won't fix. We don't want that title to be too long.


Fhircore ANC v0.0.2 and EIR v0.0.9 releases

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #516
Type
Choose one:  Release
Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[x] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

The CHANGELOG.md file will be split into multiple CHANGELOG files corresponding to the different modules as tracked by this ticket #565


Fix for #1272

Proposed additional check for #1272

Not sure what's going on here, it's saying 31 errors in 48 files yet I only changed 1 file.

I pulled a fresh copy from your repo, npm install, grunt test and it's still saying that, something odd is going on there.
I did miss a semicolon on mine though I noticed, damn ASI.

@mattbell87 This fixed #1272 for me, no more errors.

:+1:

@zaggino any idea what's causing the grunt test to fail? Updated linters maybe reporting previously unreported code issues?

@mattbell87 not failing for me:
Zaggi@CO9QUQM MINGW64 ~/AppData/Roaming/Brackets/extensions/user/zaggino.brackets-git (master)
$ grunt test
Running "jshint:files" (jshint) task
>> 48 files lint free.

Running "jscs:src" (jscs) task
>> 48 files without code style errors.

Done, without errors.

Cheers! :)
Re: jshint have you tried doing npm update or npm install --force  to see if it's newer stuff that's causing Travis to fail?

Yes, doing that makes the errors show now. I'll fix them later.


JBPM-5289 - Migrate the docs to a structure that prepares for community and product docs reuse

This PR changes the docs structure so that the downstream docs can be included in this repo.
@psiroky If you wouldn't mind merging today or tomorrow? (Once build completes).
(Contains @tsurdilo's changes from this past friday (aug 25th))

Reviewing this PR now.
With regard to image changes:
...nnersExample/manners_activity_diagram.svg → ...nnersExample/manners_activity_diagram.svg
Is it possible to dump all images into docs/shared/images and then refer to them from source docs using the :imagesdir: attribute that's defined in docs/shared/template/document-attributes? (document-attributes.adoc should be included at the top of master.adoc.)
I know that @tradej had trouble putting all the images into a single dir for the Enterprise versions due to files in subdirs named identically. He was forced to prefix them with chapter/section names.

@silasrh Sure, that sounds like a great next step. However, first I'd like to make sure that this step is good.

Jenkins please rebuild :8ball:
(Failed on an unrelated failure in another repo... )

Build successful! See generated HTML docs (may take a while till they are available):
http://ci.kiegroup.org/job/kie-docs-pullrequests/168/artifact/drools-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/168/artifact/jbpm-docs/target/generated-docs/html_single/index.html
http://ci.kiegroup.org/job/kie-docs-pullrequests/168/artifact/optaplanner-wb-es-docs//target/generated-docs/html_single/index.html

I've quickly reviewed the structure and build the docs locally. Everything seems to be good.
Just one thing I noticed: symlinks are being used(e.g. kie-docs/docs/jbpm-guide/community/topics points to ../topics). Will this work correctly also on Windows?
@silasrh is this ready to be merged from your point of view?

@psiroky Good question. I'm not 100% certain, but I believe that the symlink should also work on Windows. I can't imagine that git saves a textual represenation of the symlink -- it must save a "structural" representation.
Does that make sense?
Thanks for the review!
Re @silasrh -- he confirmed with me that it was good. At the moment, he's on PTO (until next week, I think??)

@mrietveld @psiroky - Yes, I'm afraid he is.
I'll request involving Tomas Radej into it too.

So I've tried to build the docs on Windows 2012 R2 and I am getting these warnings:
asciidoctor: WARNING: index.adoc: line 28: include file not found: C:\....\kie-docs\docs\jbpm-guide\community\topics\Overview-chapter.adoc

When looking at kie-docs\docs\jbpm-guide\community\topics it is actually just an ordinary file (not a Windows link) with text content ../topics. Not sure where the issue is though (if git does not support windows symlinks generally or if there is some configuration issue on my side).

@psiroky Thank you for taking a closer look at this: it appears that I was incorrect. Git does not handle symlinks well. :/
(There's a very good post about it here: http://stackoverflow.com/a/16754068/376390 )
I'll change the PR so that, at least on the community side, the shared symlink is not used.

Hi, all. It looks pretty good, but I have a concern here: I noticed all attributes (like source-dir or project-version) are specified in the pom.xml file. Although (if I recollect properly) it is the standard way of providing arguments to Maven plugins, it will likely pose a problem when building enterprise docs that include community parts—there is no way to automatically set those attributes when building with AsciiDoctor or ccutil.
Imagine, for instance, that we want to overwrite some attributes (like project-version), but keep the value of others (like codeDir or whatever). We would need to somehow keep track of their value somewhere, which only creates room for error.
I suggest you move setting the attributes to a document-attributes.adoc file, and include that file at the beginning of every index.adoc file. That way, this file can be included in the enterprise docs as well, and we can use (or overwrite) any attributes that are set the community docs. From the Maven perspective, nothing changes as you will simply invoke the plugin without any attributes.

@mrietveld, @psiroky: the symbolic links are absolutely critical to enabling single-sourcing topics/content. This may be a workaround for contributors who do want to build the content on Windows machines:
http://stackoverflow.com/questions/5917249/git-symlinks-in-windows
Using cygwin could possibly be an alternative.
For writing content, I use the asciidoctorjs browser plugin, or write in Atom, which automatically does the rendering.
Finally, I agree with @tradej's comment about moving any attributes that should be in document-attributes out of the pom.xml!

I believe we have contributors which are using Windows, so would be nice to make it work also there. I know there is a number of workarounds, but since it worked before...(however I agree that it is not a blocker).
About the document-attributes.adoc - will it be fine to include values like ${some.maven.property}? So that we can use maven-resources-plugin to supply the actual value during the build?

@tradej I have a few additional questions (to better understand where we are heading):

it seems the plan is to store the product content under the directories jbpm-guide and drools-guide, similarly and for community? Where would we for example put the installation guide? Would it be new *-guide module? Or would we put that in the shared dir?

how should we use the document-attributes.adoc? It would be great if we could reuse this for multiple guides, because the content will likely be the same (or very similar). Can we store that directly at the docs dir? Or should this go into the *-guide dirs?

I would like make sure we agree on the structure before we merge this. So that we can avoid further (big) changes in future. (I know its hard to predict the future needs though, so if we are not sure, we can also postpone the decision...)

@psiroky

The product books go on the same level as jbpm-guide and drools-guide, i. e. in /docs. That applies to the installation-guide as well. Any and all shared content will then go to /docs/shared, and be conditionalised the moment somebody wants to share it between books.
Definitely in /docs/shared. It will be included in all the docs. If there is anything specific that needs to be changed in a book, that value can be overwritten directly in that book (or we can have multiple attributes files).

@psiroky - please do let me know if there are any questions around the shared repo workflow or proposed repo structure mentioned in the shared repo doc I shared with you.

Merged as https://github.com/droolsjbpm/kie-docs/commit/b3d63ec4a05c5f52992c6dd33d3bcb040a19465e.

Awesomeness :-) Cheers, Petr!
Kartik and Tomas, let's discuss next steps early next week if your
schedules allow.
-- Silas
On Fri, Oct 14, 2016 at 3:05 PM, Petr Široký<EMAIL_ADDRESS>wrote:

Merged as b3d63ec
https://github.com/droolsjbpm/kie-docs/commit/b3d63ec4a05c5f52992c6dd33d3bcb040a19465e
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/droolsjbpm/kie-docs/pull/152#issuecomment-253793857,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APV109T-TKkFwqFKCl1fOfN4_MvTFbffks5qz34tgaJpZM4Jt_nX
.


nix-info: multi-user? "no" when run as root

Issue description
Does that makes sense?
Steps to reproduce
[root@ethmoid:~]# nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 4.14.108, NixOS, 18.09.2390.40e716b92a7 (Jellyfish)`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.1.3`
 - channels(root): `"nixos-18.09.2390.40e716b92a7, nixos-hardware, nixos-unstable-19.09pre173166.373488e6f4c"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

[davidak@ethmoid:~]$ nix-shell -p nix-info --run "nix-info -m"
/nix/store/a6rf8a474z5y01w4ld2v54d24kyqcxag-stdenv-linux/setup: line 784: /run/user/1000/env-vars: Permission denied
 - system: `"x86_64-linux"`
 - host os: `Linux 4.14.108, NixOS, 18.09.2390.40e716b92a7 (Jellyfish)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.1.3`
 - channels(root): `"nixos-18.09.2390.40e716b92a7, nixos-hardware, nixos-unstable-19.09pre173166.373488e6f4c"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Technical details
See above ;)

Yes, for some values of «yes» — requesting a Nix build as root does not go through the daemon.
Maybe it should be a separate answer. Multi-user? yes/no/root

I interpret the question as "Is Nix installed in multi-user mode?" and that should be "yes" also as root.
Isn't that the information that is usefull in a bugreport?

Hard to say — if you treat that question as «is the build invoked via daemon», the current nix-info answer is more likely to be relevant…

@grahamc what do you think?

Hm. I think the master version indeed checks for nixbld and works the same for user and for root.


use InputImageType as OutputImageType

use InputImageType as OutputImageType
this hopefully resolves issue with pythonic api

Traceback (most recent call last):
File "/Users/lloyd/Code/ITK_Module/ITKDissolve/../example_script.py", line 11, in
modified_labels = itk.dissolve_mask_image_filter(labels, mask_image=mask)
File "/Users/lloyd/Code/ITK_Module/.venv/lib/python3.9/site-packages/itk/support/helpers.py", line 173, in image_filter_wrapper
return image_filter(*args, **kwargs)
File "/Users/lloyd/Code/ITK_Module/.venv/lib/python3.9/site-packages/itk/itkDissolveMaskImageFilterPython.py", line 2070, in dissolve_mask_image_filter
instance = itk.DissolveMaskImageFilter.New(*args, **kwargs)
File "/Users/lloyd/Code/ITK_Module/.venv/lib/python3.9/site-packages/itk/support/template_class.py", line 734, in New
return self[list(keys)[0]].New(*args, **kwargs)
File "/Users/lloyd/Code/ITK_Module/.venv/lib/python3.9/site-packages/itk/itkDissolveMaskImageFilterPython.py", line 2046, in New
template_class.New(obj, *args, **kargs)
File "/Users/lloyd/Code/ITK_Module/.venv/lib/python3.9/site-packages/itk/support/template_class.py", line 800, in New
itk.set_inputs(self, args, kargs)
File "/Users/lloyd/Code/ITK_Module/.venv/lib/python3.9/site-packages/itk/support/extras.py", line 1252, in set_inputs
attrib(itk.output(value))
TypeError: Expecting argument of type itkImageUS3 or itkImageSourceIUS3

The solution to the error message above was to pass the type information, i.e.
ImageType = type(image)
MaskType = type(mask)

modified_image = itk.dissolve_mask_image_filter(image, mask_image=mask, ttype=(ImageType, MaskType))

See: https://discourse.itk.org/t/pythonic-api-issue-calling-filter-with-two-input-types/5280/2


Price per cocktail

It might be nice to be able to add price/ingredient, so that you can view how much each cocktail costs you.

Great idea!
The ingredient prices could differ a lot for which country you are in.
Any ideas on how this could be solved are welcomed 😀

I'd suggest having a field on ingredients where you enter price and unit size from which the app calculates the amount that goes towards the drink. Maybe with a buffer to account for spills and the like.

*insert random anti-stale message *


Add Redis to k8s production

Should we reconsider using redis in staging as well? Would be nice if staging closely mirrors the prod setup.

The redis cache persist after new deployment and needs to be invalidated. Might just use in-memory cache for staging?

Should we reconsider using redis in staging as well? Would be nice if staging closely mirrors the prod setup.

The redis cache persist after new deployment and needs to be invalidated. Might just use in-memory cache for staging?


Corrupted file

Hello,
I tried to download my Deezer playlist with BeatOn, but there's one song that didn't download properly.
Here's the output (found in "temp" file).
It's a song that I added with the upload MP3 feature in Deezer, I think this is why it's corrupted. I hope you can fix this issue.
Adouche

Hello, I tried to download my Deezer playlist with BeatOn, but there's one song that didn't download properly. Here's the output (found in "temp" file). It's a song that I added with the upload MP3 feature in Deezer, I think this is why it's corrupted. I hope you can fix this issue.
Adouche

I don't think that can be fixed at this time. The reason it's probably not downloading correctly is that Deezer doesn't have metadata for it, so the program doesn't know what to do with it. If the file plays and there are no audio problems with it, then it's not corrupt; it just doesn't have metadata.

Thank you. You can just add the original file, so that's not very important.


bug in assignment into compiled model after second model is compiled

It seems inconceivable that this would not have come up before, but I'm getting an error in the following (this is a simplified version of a Paleon user's code):
code1 <- nimbleCode({
  z ~ dnorm(0,1)
})
m1 <- nimbleModel(code1)
cm1 <- compileNimble(m1)
cm1$z <- 4

code2 =nimbleCode({
 y ~ dnorm(0,1)
})
m2 = nimbleModel(code2)
cm2 <- compileNimble(m2)

cm1$z <- 3
## Error in envRefSetField(x, what, refObjectClass(x), selfEnv, value) :
##  ‘z’ is not a field in class “Ccode”

Wow.  Shocking, indeed.  I'm recreating it.


No donate information in the project description

Great tool, thanks for the Open_FSR implementation...
but there's no information where to invite you a drink as a thanks for your work and time.
Pleas add it to the project description, or a link where to find it.

Hey, thank you for appreciating the time and work :) There is a link in the help section of the app, leading you here:
https://sim-site.netlify.app/page/donate
I'm still looking for better alternatives to PayPal. That one donation I received last year had almost 1/3 transaction fees.


Update Electron 1.3.2

What I had trouble with:

bad npm version
this/repo/path ambiguous
npm i -S -E package@ver and commit before running vendor-update
that problem with lots of changes in shrinkwrap

@chromakode

:+1:


Showcase pinned repositories

Hi.
I am new to Jekyll and this project maked it easy for me to understand how stuff works.
I have a question about the repositories showcase
{% for repository in site.github.public_repositories limit: 9 %}

Is it possible to iterate over pinned repositories? As someone with a lot of repositories I would like to showcase 9 one the important ones, not just first 9 as those are mostly forks of other open source projects.
Thanks.

Agree. I do want to be able to exclude forks first of all. Also, I would like to show repositories sorted by stars desc. Or, at least, show only pinned ones.

It appears to be an issue with the plugin: jekyll-github-metadata. It only gives you access to a limited number of options. https://help.github.com/en/articles/repository-metadata-on-github-pages#using-repository-metadata-locally

I was able to exclude forks and sort by stars.
  {% assign public_repositories = site.github.public_repositories | where:'fork', false | sort: 'stargazers_count' | reverse %}
  {% for repository in public_repositories limit: 9 %}

Not sure if I should submit this as PR.

@PavlikPolivka Perfect! Thank you!

It's also possible to manually filter them out by name, which is a bit more tedious than @PavlikPolivka's solution but lets you show repositories that don't necessarily have many stars:
  {% for repository in site.github.public_repositories limit: 99 %}
    {% if repository.name == "myrepo" %}

      {% include repo-card.html %}

    {% endif %}
  {% endfor %}

This would only show repositories named myrepo. Chaining multiple if statements like this lets you choose the exact repositories you want to showcase.

@Ullaakut

Chaining multiple if statements like this lets you choose the exact repositories you want to showcase.

Use case instead of multiple if:
{% case repository.name %}
  {% when "myrepo" %}
    ...
  {% when "myrepo2" %}
    ...
{% endcase %}

But for this case there is something better:
{% if "myrepo, finerepo, goodrepo, ..." contains repository.name %}
  ...
{% endif %}

Of course, the last case will be wrong if repositary.name == "repo".

Ended up adding:
repositories:
  - SonataAnnotationBundle

to _config.yml file and then in projcets.html:
  {% for repository in site.github.public_repositories limit: 9 %}
    {% if site.repositories contains repository.name %}

        {% include repo-card.html %}

    {% endif %}
  {% endfor %}

Should the limit be still there? This way you can only filter from first 9
repositories.
On Fri, Feb 22, 2019 at 10:46 PM Marko Kunic<EMAIL_ADDRESS>wrote:

Ended up adding:
repositories:

SonataAnnotationBundle

to _config.yml file and then in projcets.html:
{% for repository in site.github.public_repositories limit: 9 %}
{% if site.repositories contains repository.name %}

{% include repo-card.html %}

{% endif %}
{% endfor %}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/github/personal-website/issues/14#issuecomment-466559083,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHJtrjmu8jrX2IZseeJyekX9_qe7S6KRks5vQGUkgaJpZM4bFC-7
.

@PavlikPolivka it was edited but I guess you got the old version in the email, the limit should be removed.

My understanding is that forked repositories are often projects that are desirable to share. So I'm initially reluctant to add complexity (and confusion, potentially) by filtering out a subset of a user's public repositories.
A lighter touch, I think, could be adding this as a boolean option in the _config.yml file:
hide_forks: true

What do y'all think?

I still think the idea of filtering projects to ones that you consider most important is a good one. I guess people could make a separate page listing all their github repositories and on the index page only list ones you consider important.

Is there a way to also list external repositories that you have contributed to but do not own? We can list them in pinned repositories but I'd like to list them in my website as well.

I should mention here that we merged a PR today, #61, that supports excluding forks, alongside a handful of other quick customizations to the "My Projects" listing.

I'm investigating ways to access a "pinned" attribute for a user's repositories. 👍

@brandonrosage did you find a way to access "pinned" repositories or even repositories from organizations that a user may be part of?

@brandonrosage did you find a way to access "pinned" repositories or even repositories from organizations that a user may be part of?

I found this https://github.com/SUPERCILEX/personal-website/commit/5367c2b02ffe380190da0f91e9adb7b1fcef12e9 from @SUPERCILEX It might help you add dynamic repos

I've made some improvements since then so I would recommend looking at https://github.com/SUPERCILEX/personal-website/compare/b5de3764ae9d54e5aaae40b7ed62ff5d4f186657...ca987a5fd35a42e1769de8145e1a7019fc402762.


Placeholder shouldn't animate for initially selected values in select

Right now, the placeholder will always animate when it receives a new selected value, even if it is the initial value. It should only animate when there is no initially selected value.

Closed by https://github.com/angular/material2/pull/2054


Get Calendar passing particular month and year.

Hi ,
I want to call particular month and year calendar , is it possible to do that using pignose calendar

@mamtammak Even I was finding the same. But there are no options which can get that job done as of now.Hope contributors address this issue soon!!!!!

Hi @mamtammak
If you want to set each other the initial date to the calendars.
Shortly you can give date option, This option can helps to set the date when they initialized.
Thank you 👍


Incorrect confirmation message shown when users are sent the invite email

Steps
Create a user
From UI select the created user and perform "Invite Selected"
Issue
The confirmation message that is shown is misleading. It again shows " An invitation mail will be sent to the selected user(s)... " This should be omitted.

Labels - 3.1.0-M7, bug, medium

This is fixed in [1].
[1] https://github.com/milanperera/carbon-device-mgt/commit/c605ffbd5817f0dd236cf86d15462578bca22ea0


When row fetch limit size is large, not including a "top N" is 100x slower.

System information:

Windows 10
5.3
Jre shipped with 5.3

Connection specification:

Sql Server Azure
Latest Sql Server
No proxy/tunnel/etc.

Update:  This problem is limited to WAN such as when connecting to Azure
Describe the problem you're observing:
When fetch row limit is large, the runtimes to retrieve results into the datagrid are 100x faster only if a "top N" clause is specified.  Considered unusable otherwise.
Steps to reproduce, if exist:

Set the global fetch size limit to a large value 1000000 (1M),
Create a temp table or find a large table with 1M rows (sorry, I can provide if this is needed).
Note runtime of:  select top 1000000 * from #t
Note runtime of:   select * from #t

For me, (3) was 1s while (4) was >100s
If the fetch row size is set smaller like 1k and then click the "fetch all rows" special button then it is indeed fast.  So I guess that button is the workaround.  The downside that if the query takes a long time to generate the 1k then it will have to be repeated.  So this may be a low priority issue since there is a workaround and also since it is only a WAN issue.
Option "Use Sql to limit fetch size" option did not affect the results.
Include any warning/errors/backtraces from the logs

We'll add TOP in all SELECT queries in the next version.
Please test it in Early Access version (https://dbeaver.io/files/ea)

verified


Handling of KEYCODE_MEDIA_PLAY_PAUSE and other media remote buttons

According to the Android TV Design Guidelines in order to get an application accepted and published into Google Play an Android TV app that plays video or must content must respect the play/pause key during playback. This application doesn't handle any of the PLAY, PAUSE, or PLAY_PAUSE buttons on the PlaybackActivity.java and as a sample of an application that adheres to these guidelines an update should be made to bring this sample back into compliance with these published guidelines.
Please refer to our Media Play/Pause documentation for details:
http://developer.android.com/reference/android/view/KeyEvent.html#KEYCODE_MEDIA_PLAY_PAUSE

I merged your pull request.  Thanks a lot!


Allow as a header action

This is more of a feature request than anything. I would love it if this would allow me to add a header action here instead of just bulk actions.
What are your thoughts?

You mean a Page Action? I haven't had the use for it, but I thought about it and I think it would be easy to add.

Needed

@Shipu @dgood-gdba
What behaviour do you expect from a Table Header action? Export all records? Export current page?

Export all records.

This is supported with v1.0.0


Smart Table external sort not working with object fields

Operating System: Windows 11 - 23H2 (22631.4169)
Browser: Firefox (131.0) and Chrome (129.0.6668.101)
I have a smart-table component in my page which I need to sort (externally). I've set the [columnSorter]="{ external: true }" on the table as well as the [sorterValue] and the (sorterValueChange) to change my query params and refetch the data from the backend.
A specific field is a nested object therefore their key doesn't apply to the object. What I mean is
Structure of an order:
{
   "id": 1,
   "title": "Order 123",
   "customer": {
        "firstName": "...",
        "lastName": "..."
   }
}

The columns that I'm providing to smart table is as follows:
readonly columns: (string | IColumn)[] = [
    {
      key: 'id',
      label: 'ID',
    },
    {
      key: 'title',
      label: 'Title',
    },
    {
      key: 'customerTitle',
      label: 'Customer',
    },
]

In order to display this field I'm using an ng-template inside the smart table:
<ng-template cTemplateId="tableData" let-columnName="columnName" let-item="item" let-tdContent="tdContent">
            <td [cTableActive]="smartTable.getTableDataCellProps(item, columnName)?.['active']"
                [cTableColor]="smartTable.getTableDataCellProps(item, columnName)?.['color']"
                [cAlign]="smartTable.getTableDataCellProps(item, columnName)?.['align']"
                [ngClass]="smartTable.getTableDataCellClass(item, columnName)"
            >
              @switch (columnName) {
                @case ('customerTitle') {
                  {{ item.customer.firstName }} {{ item.customer.lastName }}
                }

However, while the sorting is clickable and changes state when you click the ID and Title columns it doesn't do anything when clicking the Customer column; it doesn't fire the sorterValueChange event, nor does it change visually (the arrows icon next to the title).
I assume there's a condition somewhere inside the smart table column sort click that checks whether this field exists on the objects which makes sense if the sorting was being done automatically. But since I'm making an external sorting, I'd expect for this to work...
Or maybe there's something else different going on that I can't figure out? 🤔

External sort does not work for derived columns at the moment. We are working on a patch to resolve this issue.

Glad to hear. For anyone else facing the same issue, a temporary but also a good fix is to flatten your data structure. You can make a secondary type like
type OrderTableItem = {
    id: number,
    title: string,
    customerTitle: string
}

and when you fetch your data, just map it for the purposes of the smart table
getOrders().pipe(
   map(apiItem => ({
        id: apiItem.id,
        title: apiItem.title,
        customerTitle: apiItem.customer.firstName + " " + apiItem.customer.lastName
   }))
).subscribe(...);

@UltraWelfare fixed in 5.2.20


Connect UI buttons to Patch Library & Asset Library

Add Object hides & shows the Asset Library
Add Patch hides & shows the Patch Library

done but not yet landed to newui.vizor.io


feat: Add a link to the comment at the top of the snapshot file

close #2864

Should this be a breaking change?
The snapshot tests will not fail, when comments are updated in snapshot files.

Should this be a breaking change?
The snapshot tests will not fail, when comments are updated in snapshot files.

Yes, but this will cause every snapshot to be updated.

imho this should not have been added during a regular test run but only when snapshots are being updated via cli flag
Right now this creates a dirty git state after tests have been run which can break tools that test for clean state and also accidentally add these changes to commits where they don't belong if a dev goes git commit -am "added all files because i didn't expect test to modify files under version control"


operator does not exist: uuid ~~ unknown QueryFailedError: operator does not exist: uuid ~~ unknown

Hi, I cannot filter record by uuid in typeorm. I've seen a same issue in adminjs-sequelizejs.
SQL logs:
query: SELECT ... FROM "assessment_session" "AssessmentSession" WHERE "AssessmentSession"."uuid" LIKE $1 ORDER BY "AssessmentSession"."id" ASC LIMIT 10 -- PARAMETERS: ["%ad95965f-0ef2-4496-b583-e43b5de50011%"]
query failed: SELECT ... FROM "assessment_session" "AssessmentSession" WHERE "AssessmentSession"."uuid" LIKE $1 ORDER BY "AssessmentSession"."id" ASC LIMIT 10 -- PARAMETERS: ["%ad95965f-0ef2-4496-b583-e43b5de50011%"]
error: error: operator does not exist: uuid ~~ unknown
    at Parser.parseErrorMessage (/Users/helsonxiao/qingzhou-be/node_modules/pg-protocol/src/parser.ts:369:69)
    at Parser.handlePacket (/Users/helsonxiao/qingzhou-be/node_modules/pg-protocol/src/parser.ts:188:21)
    at Parser.parse (/Users/helsonxiao/qingzhou-be/node_modules/pg-protocol/src/parser.ts:103:30)
    at Socket.<anonymous> (/Users/helsonxiao/qingzhou-be/node_modules/pg-protocol/src/index.ts:7:48)
    at Socket.emit (node:events:394:28)
    at Socket.emit (node:domain:470:12)
    at addChunk (node:internal/streams/readable:312:12)
    at readableAddChunk (node:internal/streams/readable:287:9)
    at Socket.Readable.push (node:internal/streams/readable:226:10)
    at TCP.onStreamRead (node:internal/stream_base_commons:190:23) {
  length: 202,
  severity: 'ERROR',
  code: '42883',
  detail: undefined,
  hint: 'No operator matches the given name and argument types. You might need to add explicit type casts.',
  position: '1026',
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: undefined,
  table: undefined,
  column: undefined,
  dataType: undefined,
  constraint: undefined,
  file: 'parse_oper.c',
  line: '731',
  routine: 'op_error'
}

properties:
properties: {
  uuid: {
    isVisible: {
      show: true,
      filter: true,
    },
    isTitle: true,
  },
}

https://github.com/SoftwareBrothers/adminjs/issues/932

There is still an issue here.
The postgres UUID column allows you to store this value '8b141381-7a98-d85e-7cae-79f9233e0aef'
This is not a valid UUID but rather an MD5 hash that I am generating from a combination of columns in my row and then storing it
Adminjs gives you an error when you try searching by this value

original: error: operator does not exist: uuid ~~ unknown
      at Parser.parseErrorMessage (/Users/vr/Desktop/code/vr/api/node_modules/pg-protocol/dist/parser.js:287:98)
      at Parser.handlePacket (/Users/vr/Desktop/code/vr/api/node_modules/pg-protocol/dist/parser.js:126:29)
      at Parser.parse (/Users/vr/Desktop/code/vr/api/node_modules/pg-protocol/dist/parser.js:39:38)
      at Socket.<anonymous> (/Users/vr/Desktop/code/vr/api/node_modules/pg-protocol/dist/index.js:11:42)
      at Socket.emit (node:events:513:28)
      at addChunk (node:internal/streams/readable:315:12)
      at readableAddChunk (node:internal/streams/readable:289:9)
      at Socket.Readable.push (node:internal/streams/readable:228:10)
      at TCP.onStreamRead (node:internal/stream_base_commons:190:23) {
    length: 201,
    severity: 'ERROR',
    code: '42883',
    detail: undefined,
    hint: 'No operator matches the given name and argument types. You might need to add explicit type casts.',
    position: '167',
    internalPosition: undefined,
    internalQuery: undefined,
    where: undefined,
    schema: undefined,
    table: undefined,
    column: undefined,
    dataType: undefined,
    constraint: undefined,
    file: 'parse_oper.c',
    line: '656',
    routine: 'op_error',
    sql: `SELECT "id", "author", "content", "guid", "link", "pubdate", "summary", "tags", "title", "feed_id" AS "feedId" FROM "feed_items" AS "FeedItem" WHERE ("FeedItem"."id" LIKE '%8b141381-7a98-d85e-7cae-79f9233e0aef%') ORDER BY "FeedItem"."id" ASC LIMIT 10 OFFSET 0;`,
    parameters: undefined
  },
  sql: `SELECT "id", "author", "content", "guid", "link", "pubdate", "summary", "tags", "title", "feed_id" AS "feedId" FROM "feed_items" AS "FeedItem" WHERE ("FeedItem"."id" LIKE '%8b141381-7a98-d85e-7cae-79f9233e0aef%') ORDER BY "FeedItem"."id" ASC LIMIT 10 OFFSET 0;`,
  parameters: {}
}

The reason this is happening is because of THIS LINE
It fails the isUUID check and goes into the else condition and tries to apply an ILIKE operator to what is still considered a UUID. How to resolve this?


Multiple components being loaded

Resources:
Before submitting an issue, please consult our docs.
Stencil version: (run npm list @stencil/core from a terminal/cmd prompt and paste output below):
<EMAIL_ADDRESS>
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or https://stencil-worldwide.slack.com
Current behavior:

Given these routes:
<stencil-route url='/enviroment/new' component='create-enviroment-view' />
<stencil-route url='/enviroment/:id' component='enviroment-details-view' />

And a user visits the URL /enviroment/new the correct view is rendered; however the enviroment-details-view componentWillLoad will load method still gets called.
Expected behavior:

enviroment-details-view should not be instantiated.
Steps to reproduce:

Related code:
I think the above code shows it all.

Other information:

Technically both routes match.  Are you using a stencil-router-switch around this bit of code?  If not you probably should to ensure that the first match is the only one that renders.
https://github.com/ionic-team/stencil-router/wiki/stencil-route-switch

Yea I'm using stencil-router-switch.
<stencil-router>
    <stencil-route-switch scrollTopOffset={0}>
        <stencil-route url='/enviroment/new' component='create-enviroment-view' />
        <stencil-route url='/enviroment/:id' component='enviroment-details-view' />
    </stencil-route-switch>
</stencil-router>

Both routes do match but shouldn't it just be the first route that matches the one that gets loaded and called?

I will add some tests to verify this. Thanks!

Sorry forgot to answer.  Under a switch statment the first match should respond.  In this case environment-details-view should not appear in the dom and an instance should not be created.
I am actually unable to reproduce, do you think you could provide a test repo with this issue. Thanks!

Hey man I made a new repo replicating the bug. https://gitlab.com/rodzzlessa24/stencil-router-bug
As you can see I just created a new component new-profile and added it to the routes.
          <stencil-router>
            <stencil-route-switch scrollTopOffset={0}>
              <stencil-route url='/' component='app-home' exact={true} />
              <stencil-route url='/profile/new' component='new-profile' exact={true} />
              <stencil-route url='/profile/:name' component='app-profile' />
            </stencil-route-switch>
          </stencil-router>

I added a console.log in the componentWillLoad function of the app-profile component. If you visit /profile/new you'll see the log message from app-profile will also be logged.

I think the issue might only occur on page refreshes if I go to the route using a <stencil-route-link> then there's no issue.

Thanks for the repo. I am able to reproduce and have identified a fix. Will be releasing a new version today.

This was fixed in release 0.2.7. Thanks!

Hey man I tried updating to 0.2.7; however its causing issues with the compiler. I opened an issue in the stencil core repo. https://github.com/ionic-team/stencil/issues/1106

There was an issue with 0.2.7. Please use 0.2.8. 🤦♂


Fix tutorial links

The links to the tutorial found on the documentation home page lead to a 404 error.

Fixed in PR #307 .


[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
This is a nice tip I was taught recently: it'd be great to have a bot that congratulates people on their first merged pull request. It can mention...

General happiness and joy, including some happy emojis
Encouragement to try more issues
...is there anything else?

Additional Info
No response

https://github.com/marketplace/actions/first-contribution looks like a good bot here. :)
I bet it could even integrate with options.guide...


Track Maintainer/Track Mentor

Please add me to this track, would like to get it up and in shape.
@kytrinyx

Victor,
Thanks for your willingness to help.  I've PR'd 11 exercises but further development of this track has stalled.  I'm not sure what the next steps ought to be.
Thanks,
Ryan
Sent from my android device.

Yes, I noticed that for sure.  It is why I am wanting access to the track more formally.

Thanks @kotp! I realized we don't have a team for it yet, I'll create that and add both of you to it.

Ok, done! I've added you both as track mentors to the track as well.

Looks like no rights to write bit on this repository for me.

No write bit for me too....


[nRF Connect] Update docker image

Problem
We need recent nRF Connect SDK sources in the docker image to do some changes around nrf_security usage in CHIP.
Summary of Changes
Update chip-build-nrf-platform image. Point to a specific commit in Dockerfile rather than "master" to make sure that the image gets rebuilt even if "--no-cache" flag isn't passed to "docker build ..." command.

@saurabhst @hawk248 @jelderton ?


Edit Project: Save Project

Neither Default Provisioning Location nor Fisma were populated from existing record.  Changing to a selected value from the dropdown and selecting 'Save' results in 'Server Returned Error While Saving' Alert.

JEL-37


Add ability to clear the cache

There are circumstances where this might be nessesary.

/test jvm-build-service-in-repo-apicurio-e2e

/test jvm-build-service-in-repo-apicurio-e2e


how to hide draw turn information

Appreciating your great work firstly.It's best tracker on Mac.
But, is there a way to turn off draw turn information like picture shown above? Sometimes people would like to make UI cleaner and simpler especially to me ^_^.

That would be unchecking: Preferences --> Trackers --> Show opponent in-hand huds

Enjoy!
-TM

Yes, I found that option. But it don't work for me.
I have tested in game. After unchecking "Show opponent in-hand", the huds did not hide as timer option do. Even after restarting HSTracker, this option become checked again. It should be a bug, I think.  Or, this situation just happened to me?
My RMBP information and it is in Chinese.

nope, looks like I messed something with this one. I'll check it


Github actions do not run properly

Without any change that I can see all github worflows seem to be failing on both open PR. The error is always the same:
Run python -m unittest discover -s cornflow.tests.ldap
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/__main__.py", line 18, in <module>
    main(module=None)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/main.py", line 101, in __init__
    self.runTests()
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/main.py", line 271, in runTests
    self.result = testRunner.run(self.test)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/runner.py", line 176, in run
    test(result)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/suite.py", line 84, in __call__
    return self.run(*args, **kwds)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/suite.py", line 122, in run
    test(result)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/suite.py", line 84, in __call__
    return self.run(*args, **kwds)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/suite.py", line 122, in run
    test(result)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/suite.py", line 84, in __call__
    return self.run(*args, **kwds)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/unittest/suite.py", line 122, in run
    test(result)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_testing/utils.py", line 131, in __call__
    self._pre_setup()
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_testing/utils.py", line 144, in _pre_setup
    self.app = self.create_app()
  File "/home/runner/work/corn/corn/cornflow/tests/custom_test_case.py", line 456, in create_app
    app = create_app("testing")
  File "/home/runner/work/corn/corn/cornflow/app.py", line 54, in create_app
    docs.register(target=res["resource"], endpoint=res["endpoint"])
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/extension.py", line 125, in register
    resource_class_args, resource_class_kwargs)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/extension.py", line 72, in _defer
    bound()
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/extension.py", line 147, in _register
    resource_class_kwargs=resource_class_kwargs,
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/apidoc.py", line 39, in convert
    return [self.get_path(rule, target, **kwargs) for rule in rules]
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/apidoc.py", line 39, in <listcomp>
    return [self.get_path(rule, target, **kwargs) for rule in rules]
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/apidoc.py", line 53, in get_path
    for method, view in operations.items()
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/apidoc.py", line 54, in <dictcomp>
    if method.lower() in (set(valid_methods) - excluded_methods)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/apidoc.py", line 66, in get_operation
    'parameters': self.get_parameters(rule, view, docs, parent),
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/flask_apispec/apidoc.py", line 92, in get_parameters
    extra_params += openapi_converter(schema, **options) if args else []
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/apispec/ext/marshmallow/openapi.py", line 135, in schema2parameters
    for field_name, field_obj in fields.items()
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/apispec/ext/marshmallow/openapi.py", line 135, in <listcomp>
    for field_name, field_obj in fields.items()
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/apispec/ext/marshmallow/openapi.py", line 151, in _field2parameter
    prop = self.field2property(field)
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/apispec/ext/marshmallow/field_converter.py", line 172, in field2property
    ret.update(attr_func(field, ret=ret))
  File "/opt/hostedtoolcache/Python/3.7.11/x64/lib/python3.7/site-packages/apispec/ext/marshmallow/field_converter.py", line 219, in field2default
    default = field.load_default
AttributeError: 'Integer' object has no attribute 'load_default'
Error: Process completed with exit code 1.

This issue should be solved now with the update to requirements on PR #78. The origin of the error came from not having an apispec dependency on requirements, as it was installing its latest version (5.1.0 or 5.0.0) that drops support of marshmallow older versions, prior to 3.13.0, and we were using 3.6.0 as enforced by our requirements.
One of the main changes is that now marshmallow does not accept the default option on the schemas, but it has a load_default and dump_default option.
I'm going to open a new issue to review the changes needed to bump from marshmallow 3.6.0 to newer versions and update other libraries as well if the changes introduced in newer marshmallow  seem useful.


Added translations for new texture strings

Added translation for:

Inspector.Texture.Saturation
Inspector.Texture.AdjustSaturation
Inspector.Texture.Value
Inspector.Texture.AdjustValue

(this comment will be updated if more changes will be made before this pull request's merge/closing)

Thank you for the additions!

Thank you for the additions!


[BugFix] From config screen joining call back on network reconnect display recording banner in lobby state

Purpose

Fix bug to not displays recording and transcription banner in lobby call state when teams call is joined from config screen after network disconnect error

Impact
The code changes can have impact on banner display when device screen rotates as well as background mode.
Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

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

How to Test
Scenario 1

Join teams call
Start recording from teams client
UI library will display recording banner on call screen
Disconnect network on mobile device
On network error at setup screen, connect to network and join call
Banner should be displayed when user is admitted

Scenario 2

Join teams call
Start recording from teams client
UI library will display recording banner on call screen
Disconnect network on mobile device
Stop recording from teams call
On network error at setup screen, connect to network and join call
Banner should not be displayed when user is admitted

What to Check
*Verify that the joining call back from setup screen(after network error) is not displaying banner until the call state is not connected
*Verify that banner for recording and transcription is displayed on screen rotation as well as when back from background mode

Tested.


Need to use Microsoft Azure as an authentication provider

As per our need to customize Legend SDLC server, we have an use case where we would to authenticate user through Azure AD ( ADFS) as we have all employees onboarded onto Azure AD. After authentication , we want to share scope like email, user and openid scope to Gitlab as We have all Legend Project created in GitLab.
In a nutshell, we want only Authentication done through Azure AD and would like to keep remaining interaction as provided out of the  box.
Thanks in advance,
Aman

This issue is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This issue was closed because it has been inactive for 35 days. Please re-open if this issue is still relevant.


question on RSA/ECDSA signature endianness in SPDM 1.0/1.1

Current signature endianness is clarified in SPDM 1.2.1.
But SPDM 1.0/1.1 does not include the clarification.
SPDM issue: https://github.com/DMTF/SPDM-WG/issues/2929

According to the SPDM WG, SPDM 1.0/1.1 allows big or little endian, chosen by implementation.  As such, I propose that:

Provide compile-time option for libspdm Requester's signature generation, if negotiated VERSION == 1.0 or 1.1: choose one from: {big endian; little endian}.
Provide compile-time option for libspdm Responder's signature verification, if negotiated VERSION == 1.0 or 1.1: choose one from: {big endian; little endian; flexible - try both and accept signature if one passes}.

Understood this will not intercept libspdm3.0.0, but later.
Please assign Rich Kong to it. @richkong88

I think we need define a set of MACRO to control the different behavior:
LIBSPDM_SPDM_10_RSA_LITTLE_ENDIAN
LIBSPDM_SPDM_10_ECDSA_LITTLE_ENDIAN
LIBSPDM_SPDM_10_RSA_DUAL_VERIFICATION
LIBSPDM_SPDM_10_ECDSA_DUAL_VERIFICATION
LIBSPDM_SPDM_11_RSA_LITTLE_ENDIAN
LIBSPDM_SPDM_11_ECDSA_LITTLE_ENDIAN
LIBSPDM_SPDM_11_RSA_DUAL_VERIFICATION
LIBSPDM_SPDM_11_ECDSA_DUAL_VERIFICATION

I think we need define a set of MACRO to control the different behavior:
LIBSPDM_SPDM_10_RSA_LITTLE_ENDIAN
LIBSPDM_SPDM_10_ECDSA_LITTLE_ENDIAN
LIBSPDM_SPDM_10_RSA_DUAL_VERIFICATION
LIBSPDM_SPDM_10_ECDSA_DUAL_VERIFICATION
LIBSPDM_SPDM_11_RSA_LITTLE_ENDIAN
LIBSPDM_SPDM_11_ECDSA_LITTLE_ENDIAN
LIBSPDM_SPDM_11_RSA_DUAL_VERIFICATION
LIBSPDM_SPDM_11_ECDSA_DUAL_VERIFICATION

@jyao1 These would be defines in spdm_lib_config.h right?
Something like:
#ifndef LIBSPDM_SPDM_10_RSA_LITTLE_ENDIAN
#define LIBSPDM_SPDM_10_RSA_LITTLE_ENDIAN 1
#endif
...

Requesters may want this to be runtime configurable. If signature verification fails with one endianness it may try with another endianness.

Is a good place to put the logic changes in spdm_crypt_lib\libspdm_crypt_asm.c in the functions:
libspdm_asym_sign
libspdm_asym_sign_hash
libspdm_asym_verify
libspdm_asym_verify_hash

@richkong88 , here is discussion result between @steven-bellock and me.

We use different way for signing and verification. The reason is: Signing is NOT controlled by libspdm, but in a customer spdm_device_secret_lib. While Verification is controlled by libspdm.

For signing, it is OK to use MACRO. We can put macro to spdm_device_secret_lib.

For verification, we use runtime switch. If there is any API change is needed, we need add the new API, and keep old API as is for compatibility.  To avoid code duplication, the old API can call new API as default behavior.


pianoroll

Hello,
import symusic
sysmusic.Score(midi_file_path).pianoroll(modes=["frame"], pitch_range=(0, 128),  encode_velocity=False)
gives me the error:
TypeError: pianoroll(): incompatible function arguments. The following argument types are supported:
1. pianoroll(self, modes: collections.abc.Sequence[str], pitch_range: tuple[int, int] = (0, -128), encode_velocity: bool = False) -> numpy.ndarray[dtype=uint8]
Invoked with types: symusic.core.ScoreTick, kwargs = { modes: list, pitch_range: tuple, encode_velocity: bool }
Expected behavior
A clear and concise description of what you expected to happen.
symusic 0.5.1 running on python 3.11 on MacOS (M2 chip)
Same with
sysmusic.Score(midi_file_path).tracks[0].pianoroll(modes=["frame"], pitch_range=(0, 128),  encode_velocity=False)

Thanks for your feedback! It' s a pity our test had not covered pianoroll yet.
Unfortunately, this problem is related to C++ binding. But you may still use this feature with symusic==0.4.8.
Another friendly suggestion is to always use .sample(tpq) before .pianoroll(...) to avoid potential memory explosion.

Thank you very much, this worked. Thank you also for the suggestion, I was doing that already :)

Fixed in v0.5.2.


test runner

Substrate test runner is a new testing kit built specifically for testing runtime upgrades and migrations, It can be used with an existing chain data, to assert that a new runtime upgrade should behave as expected before it even goes live on-chain.
The goal of this project is to have an e2e testing suite that is run on CI whenever runtime code changes, allowing us to test runtime upgrades/migrations in a stress-free manner, previous attempts at this
In order to achieve this goal, pallets need to migrate to using tests written specifically for the test runner, eg see pallet-balances-e2e
These tests are generic and so they can be used to test runtime upgrades/migrations for any runtime 🎊

Any particular reason  why we include the test-runner here rather than just adding the external repo as a dev-dependency where necessary?

Well the test runner depends on substrate master. If we add as an external dependency, there'd would now be two different versions of substrate in use.

For what do we need this test runner in substrate?

For what do we need this test runner in substrate?

This project was originally started with the idea of testing runtime upgrades but in order to get there, we first need a test suite for all pallets. We'd like for these tests to be run in CI as another build step for PR's that touches runtime code and so it doesn't make sense for the tests/runner to live elsewhere.

Please be more specific.
What will these pallet tests do, how will this pallet tests be used later for doing runtime upgrade tests?
How should that work when these tests are written for the Substrate runtime and you want to test a polkadot runtime upgrade?

Pallet tests should cover general use case of pallets.
Runtime upgrades can be done on a testnet that tracks substrate releases.
Post upgrades, we can run the test suite using the test runner and assert that no pallet functionality is broken.
For polkadot runtime upgrades, polkadot specific pallets would also need their own tests which will live in the polkadot repo as well.

We already have tests for all pallets that test their functionality.

Right, yes we do, but not in a way we can use for testing runtime upgrades?

Everything that I built in substrate-debug-kit also has the same issue where:

Ideally I want it to live inside of substrate because I want it to depend on substrate master.
Ideally (from another perspective) is should live outside of substrate because it is external tooling.

Initially my stance was to go through bit of hassle, but keep this stuff outside of substrate, as I have done before.
I am ambivalent here and will review this next week again.

Still unclear of whether this should live here, nor whether this does demonstrate more utility compared to the tests we already have.

Was this answered somewhere? I can not find any answer to this.
I'm still against adding this, as I see no value in having this in Substrate. The argument that this depends on Substrate master, is no real argument, as we have multiple projects which depend on Substrate master. There is also no value in demonstrating a runtime upgrade test here for the Substrate node.
So, it would be really nice to have some explanation why this should be added to Substrate. There is no value in adding code just for the purpose of adding code.
There is also stuff in this pr like some random runtime.wasm file? How is that build? What are we doing when this breaks next week, because something changed?

Still unclear of whether this should live here, nor whether this does demonstrate more utility compared to the tests we already have.

Was this answered somewhere? I can not find any answer to this.
I'm still against adding this, as I see no value in having this in Substrate. The argument that this depends on Substrate master, is no real argument, as we have multiple projects which depend on Substrate master. There is also no value in demonstrating a runtime upgrade test here for the Substrate node.
So, it would be really nice to have some explanation why this should be added to Substrate. There is no value in adding code just for the purpose of adding code.
There is also stuff in this pr like some random runtime.wasm file? How is that build? What are we doing when this breaks next week, because something changed?

@bkchr Your questions were answered here.

This project was originally started with the idea of testing runtime upgrades but in order to get there, we first need a test suite for all pallets. We'd like for these tests to be run in CI as another build step for PR's that touches runtime code and so it doesn't make sense for the tests/runner to live elsewhere.

@bkchr Your questions were answered here.

This project was originally started with the idea of testing runtime upgrades but in order to get there, we first need a test suite for all pallets. We'd like for these tests to be run in CI as another build step for PR's that touches runtime code and so it doesn't make sense for the tests/runner to live elsewhere.

@bkchr please take a look at this once more

Removed my requested change


nextflow id isn't captured

It seems that session_id evaluates to "nextflow":

Because, looking into the database table, it shows "nextflow" for run.reference.

Something is off, here, right, @Zethson? 🤔

.. That's weird and not intended. I'll look into it.

⋊>   nextflow log | awk '/lamin_rnaseq/{print $8}'
22be0b9f-1fdc-499e-b85a-05974464e876

works locally so I'll have to dig into it

And it's also writing it to the db successfully locally?

Trying to fix here: https://github.com/laminlabs/nextflow-lamin-usecases/pull/14

Seems to be fixed with this PR now!


[ANCHOR-867] Create Uber JAR

Create Uber JAR in the build.
Publish Uber JAR to Maven

I think it'd be great to add GH actions for automatic deployment (it's not set up yet)


[2.2.0] Tenant aware commands

Resolves #218
The TenantAwareCommand trait has an abstract method to get an array of the tenants for which the command should be executed. This method is implemented by both HasATenantsOption and the newly introduced HasATenantOption, so you can use these traits like this:
class MyCommand extends Command
{
    use TenantAwareCommand, HasATenantOption;

    public function handle()
    {
        //
    }
}

TODO:

[ ] ~Tests~ #100
[x] Docs docs:39

👏


Fix two cloud-init cases

Update 2 cloud-init cases per test results on ali and openstack:
test_check_metadata - Add ali specific http url
test_cloudinit_no_networkmanager - Add repo for openstack, fix some command issues

LGTM.

Thanks Xiao and Ming, merging now