GDS=>PNG render improvement for GitHub Actions and local

This is introduces the --create-png-preview option that could replace/upgrade the method I contributed to the TT04 gds action.
It is intended to address https://github.com/TinyTapeout/tt-gds-action/issues/8, avoiding a failed harden due to PNG rendering failure.
NOTE: --create-svg and --create-png retain their existing behaviour, for backwards-compatibility. I've tested this a lot -- using GDS from 25 TT04 submissions with a mix of 1x1, 2x2, and 4x2, and most-importantly the previously-failing 8x2 in petersn/tt04 -- and in any case it should be safe to merge. If you'd like to run your own test, make sure you've done sudo apt install librsvg2-bin pngquant
This also brings these improvements:

Tries harder to find a way to succeed using the 'fast' methods, logging outcomes.
Falls back to cairosvg (slow method) if all else fails.
Removes wasted grey border area.
Uses even more compression if the design is 8x2 (to keep large and complex design PNGs under 5MB).
Easier/faster for users to run locally.

NOTE: If this PR is accepted, then I'm also ready with the corresponding update of the PNG steps of tt-gds-action.
Let me know too if you'd like me to add suggested changes to the PNG instructions of the "Hardening Tiny Tapeout Projects Locally" guide.

Thanks! I don't think backward compatibility is a huge concern here - AFAIK we don't really use --create-png anymore (it was previously used by the tt-gds-action).
Any reason not to replace --create-png with the new method?

NOTE: If this PR is accepted, then I'm also ready with the corresponding update of the PNG steps of tt-gds-action.

Excellent

Let me know too if you'd like me to add suggested changes to the PNG instructions of the "Hardening Tiny Tapeout Projects Locally" guide.

Yes please

Sounds good. I’ll replace —create-png and update the doco :)

OK, changes made: --create-png-preview option is deleted and --create-png has been replaced with this new method. Ping me if you merge this PR and I'll do a PR for tt-gds-action.


Potential bug in Markdown Editor sample

this.input is updated by the text typed in the textarea in debounce.
This means that if the component needs to be rendered in the debounce timeout for whatever reason, the input changes are erased as the textarea is rerendered with the "old" value of this.input (the current one in fact, but not updated yet as debounce timeout is not finished).
this.input should be directly linked to the value of the textarea (potentially by v-model) and the debounce should be applied to the conversion of the markdown only.

this.input is updated by the text typed in the textarea in debounce.
This means that if the component needs to be rendered in the debounce timeout for whatever reason, the input changes are erased as the textarea is rerendered.
this.input should be directly linked to the value of the textarea (potentially by v-model) and the debounce should be applied to the conversion of the markdown only.

cc @chrisvfritz


Improve documentation.

Made the install instructions more apparent since we're not installing clients but installing HTTPie "via" several clients.

My bad. The copy was actually correct. Closing the PR now.


create a page for trying to merge toscaIDE in the orchestrator

Let us try to see if we manage to have a page visible only to service provider to try to place the toscaIDE inside the orchestrator.
I need a page much like the "button" page at the moment so I can try to take it and port it there.
If you can also remember me where I have to place javascript that has to be applyed just to that page.
Thanks

We have to define where to put the link to access it.

at the moment choose a link in the SP dashboard, it is something that just SP and admin can see, we need the SP landing page so there is the place I leave to you the decision, I just need an empty page and you can see in the IDE code the javascript libraries needed (mind the versions, but once you manage to place it we will check that together or I will)

A link may change, so I do not know if it is a good idea

place in the SP dashboard a button or link that goes to "generate tosca file" that will be the IDE page

Can you create an empty page in the UI (like the "press the button").
With the following JS libraries?
tv4.js
cytoscape.js (2.4.9 not higher)
ObjectPath.js
and an empty JS file where I can place the directive I have created?

Call it "ide"

You can see the button if you signed up as a "Publisher".
The files to modify are in the folder:
cloudopting-manager\rest-component\src\main\webapp\scripts\app\service\toscaide
TOUCHABLE:

toscaide.controller.js
toscaide.directive.js
toscaide.html

NOT TOUCHABLE:

toscaide.js

@gioppoluca I saw a commit with a comment that the IDE is working. Could this issue be closed?


enhancement: Converting model registry file to json file

Converting model registry file to json file

Closing as done.


COMP: Fix build removing nonexistent resource reference

This removes the reference to Resources/CTK~vtkModalityWidget_LANG.ts originally introduced in 2facb49 ("Fix Slicer crash when using localized DICOM module", 2023-04-12) and partially removed in a76855c ("Mark translatable strings", 2024-01-03).
Issue was discovered while testing changes introduced in:

https://github.com/commontk/CTK/pull/1179

Thank you, good catch!


Use cadquery.Solid instead of FreeCADPart.Solid in threads/base.py

This ticks one of the boxes on https://github.com/cqparts/cqparts/issues/129 and fixes a test.

I just noticed the segfault being triggered by the badly formatted STEP file.
Not the best way for pythonocc to deal with a bad format, it may be a performance trade-off thing... as for cqparts testing, I've just set it to skip.

The segfault should have been fixed in https://github.com/CadQuery/cadquery/pull/4. It looks like travis used an older version of cadquery-occ from conda that didn't include the fix, but hopefully It will start using the new build soon.

OK, that's better:
----------------------------------------------------------------------
Ran 188 tests in 0.617s

FAILED (failures=13, errors=50, skipped=4)

Thanks again for another contribution @justbuchanan !


Fix readdir regression while merging

Ref https://github.com/roboll/helmfile/pull/2058#issuecomment-1019439394

@mumoshu Thanks for being so fast to respond. From my local tests, everything seems to be working now. :tada:

@nlueb Thanks for your confirmation ☺


Trouble with installation

I'm still having trouble getting your tool up and running. You mentioned that I may need to reference the FancyTextRendering assembly in my code, but according to the documentation this is done automatically for all assemblies inside the Assets folder, right?

With this I'm assuming the other dependencies you mentioned are installed correctly, but I don't understand why this assembly file isn't being recognized.

A few things to test:

Are there any compiler errors?
Can you add the FTR monobehaviors to gameobjects?
Are you able to reference the scripts if you put your code inside an asmdef, and give it an explicit reference to FTR?

I'm not really sure how auto-referenced asmdefs work, I always use explicit references.

(I appreciate the lightning fast reply <3)
1st

2nd
Yes! And they work like a charm

3rd
Never really used asmdef's, kind of shooting in the dark here
Created this assembly definition in the folder with my script, and added a dependency to FTR. Still no luck

Does ProjectGetter have using FancyTextRendering; at the top of the file?

I removed it before creating the asmdef and upon adding it again, it now doesn't even recognize the assembly (it didnt give this error before creating APIStuff)

What the hell. I am so confused.
If you clone the entire FTR project does it work properly? If so, you might be able to figure out the problem by checking the differences between the FTR project and your new project with FTR added.

 (It will re-generate the Library folder)

Sometimes Unity does a widdle fucko boingo when compiling scripts. This resets all its cached information and can fix weird errors.

Throughout this whole process I've been hitting Reimport all after almost every change. There doesn't seem to be a library folder in my project shrug. I'm reading online about vscode having trouble with assemblies, Im going to investigate that a little further and then will clone your demo project

There... doesn't seem to be a library folder? If you're correct about that, Unity shouldn't be working at all. This is what a project folder is supposed to look like:

I have occasionally encountered issues that "reimport all" does not fix, but deleting the Library folder does.

Ok I wrongly assumed it could be seen from the explorer in the Unity editor. I deleted it and still no luck. Im going to clone your project now

Taking a look at your code I saw you used using LogicUI.FancyTextRendering;
After reimporting, the editor shows no more errors!

Aaaaah, yes, that's the problem! My apologies -- I originally considered changing all the namespaces from LogicUI.FancyTextRendering to just FancyTextRendering, but I eventually decided not to in order to make cross-development easier between this repo and my main LogicUI repo, as well as to enable forward compatibility for FTR projects when I release LogicUI. I forgot that I'd decided not to do it when talking with you however.
I'm glad you've got it working! Best of luck with your project :D


Update version components and repository K8s

Pull Request template
Please, go through these steps before you submit a PR.
Why is this PR required? What issue does it fix?:
Update version components and change to new k8s repository as recommended by Kubernetes
What this PR does?:
Does this PR require any upgrade changes?:
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Fixes #
[ ] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

PLEASE REMOVE BELOW INFORMATION BEFORE SUBMITTING
The PR title message must follow convention:
<type>(<scope>): <subject>.
Where:

type is defining if release will be triggering after merging submitted changes, details in CONTRIBUTING.md.
Most common types are:

feat      - for new features, not a new feature for build script
fix       - for bug fixes or improvements, not a fix for build script
chore     - changes not related to production code
docs      - changes related to documentation
style     - formatting, missing semi colons, linting fix etc; no significant production code changes
test      - adding missing tests, refactoring tests; no production code change
refactor  - refactoring production code, eg. renaming a variable or function name, there should not be any significant production code changes

scope is a single word that best describes where the changes fit.
Most common scopes are like:

data engine (localpv, jiva, cstor)
feature (provisioning, backup, restore, exporter)
code component (api, webhook, cast, upgrade)
test (tests, bdd)
chores (version, build, log, travis)

subject is a single line brief description of the changes made in the pull request.

Closing this PR. These changes have been included in PRs:

https://github.com/openebs/lvm-localpv/pull/233
https://github.com/openebs/lvm-localpv/pull/212
https://github.com/openebs/lvm-localpv/pull/243


IPROTO-100 Add test to demonstrate the issue is fixed

this issue was happening in 4.3.0.Alpha6 and was fixed since then due to the many refactorings required by IPROTO-94

https://issues.jboss.org/browse/IPROTO-100

Merged in master (4.3.x)


[Candidate 1v1 Rainforest Reservoir ZH v1] Performance drops by around 80% while a dam breaks

Observed behavior
Performance drops by around 80% while a dam breaks.
Expected behavior
Performance does not drop much while a dam breaks.
Screenshots

new version was sent on discord a few days ago. We tested 3 people on this map now and no FPS issues.

Please test more hardware. On my machine 80% performance drop.

Destroyed both dams
FPS droped from 63 down to 54 for a few seconds and then went back up to 63 again.
Its something you cant rid of without removing the dam.
In my opinion its ok. The frame drop is not gamebreaking and only happens for a few seconds

Windows 10, AMD Ryzen 7 5600G with Radeon Graphics
3.80 GHz
NVIDIA GeForce GTX 1650

It perhaps depends on hardware. On my machine drop is significant. Note that other maps also had this drop and it was fixed on all of them except this one.

Newgate says this is technically as good as we can make it.

Tanso managed to improve it regardless. Performance drop on my machine was reduced from around 60 FPS low to 170 FPS low.


DACPAC Versioning Slow

Azure DevOps Extensions
Manifest Versioning
Platform
Azure DevOps Services
Azure DevOps Server (TFS) Version
No response
Extension Version
DACPAC 3.7.2
Describe the bug
The discovery of the path to the location of the required tools is very slow on some agent, in this example it took over 6minutes
2024-03-04T18:44:31.5982842Z No user provided ToolPath, so searching default locations
2024-03-04T18:44:31.5989613Z Scan standard Visual Studio locations (2017 and later)
2024-03-04T18:44:31.6014688Z ##[debug]Found a VS2022 SKU in 'C:\Program Files\Microsoft Visual Studio\2022'
2024-03-04T18:51:10.9244247Z ##[debug]Found 1 SDK(s)
2024-03-04T18:51:10.9277486Z ##[debug]Considering 'C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\SQLDB\DAC\Microsoft.SqlServer.Dac.Extensions.dll'
2024-03-04T18:51:10.9304294Z Found the newest SDK in 'C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\Extensions\Microsoft\SQLDB\DAC'

Repo Steps
Run the task, on sone agent the time is slow, on others it is not
Expected Behavior
Fast discovery of the path
Logging Information
No response

This it appears is due to the way get-childitem -filter works.
See https://devblogs.microsoft.com/powershell/why-is-get-childitem-so-slow/


[WIP] implement inner product proof

following annotations & equations here https://doc-internal.dalek.rs/bulletproofs/inner_product_proof/index.html
code reference https://doc-internal.dalek.rs/src/bulletproofs/inner_product_proof.rs.html#1-534

It makes sense to make a separate folder/module in halo2-ecc/src for this. The kzg PR made a commitments folder, you could either also do that and put IPA inside, or just make a separate folder.
Also inner product is used to mean various things, so better to say IPA whenever possible since that's less ambiguous.

will create a new pr based on updated ce branch


type of element object => string

If one of element's type is object, it can not get the expected result.

Thanks for the PR, but since this library uses strict equality, it does return the expected results. This is because different objects in JavaScript are not equal even if they contain the same properties with the same values. But if the same instance of an object is in both arrays, those will be equal.


[bug] Don't possible to wrap Stop element with react context's consumer

AR:
There is no possibility to use render-prop pattern because LinearGradient requires Stop as a child. see file where it probably should be resolved. Make attention, that importing Stop component in that file is already unnecessary.
ER:
Stop element can be wrapped with render-prop component.
Example:
<Svg.LinearGradient x1="50%" y1="8.373%" x2="50%" y2="97.857%" id="_lg" key={key}>
  <OffsetContext.Consumer>
    {(offset) => (
      <Svg.Stop stopColor="white" offset={offset} />
    )}
  </ColorContext.Consumer>
  <Svg.Stop stopColor="blue" offset="80%" />
</Svg.LinearGradient>

Have you tried wrapping the Svg root with the consumer instead? I think that should work fine at least.


Add unit test for IP.resolve_hostname

This should reproduce the bug we found here:
#80727

I just realized this test was meant to reproduce a bug that's apparently not fixed yet. So if it actually succeeds in triggering a bug, it means we'd have a broken test suite now.
So far it doesn't seem to have triggered it but this should be looked into.


Fix reference errors

The _formData reference was causing a reference error that prevented my errors from being posted, as well as causing infinite errors to be generated (since every time it failed it creates a new uncaught error, and recur...), making the whole npm module useless.
Needed to do this._formData instead.

hi @obermillerk! thanks for the PR, I will merge it shortly. In the meantime, curious what runtime you're using? We use this package in some of our internal tools and we never ran into this so I'm surprised to hear it was so broken for you!

Using it in electron 8.2.5, running on top of node 12 (lts). I've also got typescript and webpack on top, not sure if maybe they're tweaking behavior somehow.

@obermillerk thank you! webpack definitely could have been the culprit here. additionally we'll get some typings for you shortly. thanks again for the PR!

Thanks for the quick responses! Types would be great too!

Hey @obermillerk, version 5.0.0 has just been released and it includes a type declaration file. Here's an example project that uses the new version. Please try it out and let us know if we should make any changes!


[BUG] Run State and Pre State not update

After install the new version the wascher Pre State and Run state stopped to work, they are keep showing stand by indepentent of the wasker real state
Environment details:

Environment (HASSIO, Raspbian, etc): HASSION
Home Assistant version installed: core-2021.3.4
Component version installed: v0.6.1
Last know working version: v0.5.16
LG device type and model with issue: Washer CV9011WC4A
LG devices connected (list):

Output of HA logs
Paste the relavant output of the HA log here.

Additional context
Add any other context about the problem here.

The attributes name was changed. Please refer to release note

I saw,  I've also activate the new entities but the run state at the device properties and the run state at the new entities that you created still with the stand by status (Em Espera in portuguese). The washer now is Washing but still showing stand by

Look at this screenshot

This is my washer model

Only this 2 states are not updated? They never changed or at least they was updated once?

these ones is the ones that I use and can affirm that not changing.
What I noted is, the stated change from undefined to Stand by when I power on the washer but after I start to wash they not update to the real state of the washer
looking for the current status of each new entity that you created, looks like only the state when the machine was porwered on was pulled and after that nothing was update anymore
current course is showing only a dash - , the same thing for dry level, spin speed, water temp,

tube clean count is show 25, this is the count when I power of the washer, I will monitor when the washer finish if it change to 26 or only change when I power off and power on again
I'm locate in Brazil is 18:30 for me now. Thi is a washer in my office. I will be back tomorrow at 09:00 my time and can update here.

This is really strange, I have a washer and everything works fine. Please try also installing previous version from HACS to be sure that is a component issue.
I'll wait for your feedback

Hi, after I rebook reboot the host everything started to work.
I apply the last update and still working. following the image with all states with information


while hosting the app changes to ts files are not being reflected

I don't know if I am just being dumb or if this is the way the live-reload works but while I am hosting the app if I make changes to .ts files the changes are being recognized but the browser itself doesn't reflect the changes. Is there a setting I didn't enable or is this just the way the package is suppose to work?

I'm not sure if this is the same problem but since I upgraded to the latest version my ts files are no longer being copied to the dist folder.

The ts files being copied were a bug; you shouldn't need the source files in your dist folder, we removed those.
Using ng serve and changing the ts files has been fixed by https://github.com/angular/angular-cli/pull/367. Closing this as fixed.

When the ts files were being copied I could set breakpoints in the ts files and debug the ts code. I'd suggest that the ts are copped to the dist folder during dev builds / serves and not copied for production releases

I believe you don't need source files in the dist folder, the compiler gives you a nice error output when something goes wrong.

It wasn't for the compiler error point of view, it was for using the debugging tools in chrome


CommandLine::getUsageMessage returns message with carrage returns

When trying to send the usage message to the player, by writing:
CommandLine cmd = commandSpec.commandLine();
if (cmd.isUsageHelpRequested()) {
	context.getPlayer().sendMessage(cmd.getUsageMessage());
}

The resulting message contains carrage returns (\r) that are displayed in minecraft as a square with letters inside in minecraft. It would be nice if getUserMessage were to create the message without the carrage returns, so that they dont have to be filtered out manually before sending the message to the player.
(I am using Windows 10 with Minecraft java edition 1.13.2 on the client side and spigot 1.13.2 with mcspring-boot 0.10.0 on the server side)

Have you tried cmd.getUsageMessage(Ansi.OFF)? It disallows some customizations in the output, should work for your case

Thanks, I'll try that.

Keep me updated ;)

This is with Ansi.OFF added.

Can you provide me a more complete code? I want to know the source of your commandSpec

It is those same code samples as from https://github.com/Alan-Gomes/mcspring-boot/issues/7#issuecomment-494018111

This problem has been fixed here and it will arrive as soon as possible on 0.11.1
Thanks :)


compile the SPAdes and COG tools executables： fail

I ran " ./build_cog_tools.sh " , the following error appeared. Is this step necessary?
/****/
/  preparing debug /
//
CMake Warning (dev) in CMakeLists.txt:
No project() command is present.  The top-level CMakeLists.txt file must
contain a literal, direct call to the project() command.  Add a line of
code such as
project(ProjectName)

near the top of the file, but after cmake_minimum_required().
CMake is pretending there is a "project(Project)" command on the first
line.
This warning is for project developers.  Use -Wno-dev to suppress it.
-- Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES)
-- Could NOT find OpenMP_CXX (missing: OpenMP_CXX_FLAGS OpenMP_CXX_LIB_NAMES)
-- Could NOT find OpenMP (missing: OpenMP_C_FOUND OpenMP_CXX_FOUND)
CMake Warning (dev) at /ldfssz1/ST_META/share/User/zhujie/.conda/envs/bioenv3.7/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:424 (message):
The package name passed to find_package_handle_standard_args (readline)
does not match the name of the calling package (Readline).  This can lead
to problems in calling code that expects find_package result variables
(e.g., _FOUND) to follow a certain pattern.
Call Stack (most recent call first):
cmake/Modules/FindReadline.cmake:32 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
cmake/deps.cmake:19 (find_package)
CMakeLists.txt:52 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.
-- Found Boost 1.70.0 at /ldfssz1/ST_META/share/User/zhujie/.conda/envs/bioenv/lib/cmake/Boost-1.70.0
--   Requested configuration: QUIET REQUIRED
-- Found boost_headers 1.70.0 at /ldfssz1/ST_META/share/User/zhujie/.conda/envs/bioenv/lib/cmake/boost_headers-1.70.0
CMake Error at cmake/flags.cmake:14 (message):
SPAdes requires OpenMP to be available
Call Stack (most recent call first):
CMakeLists.txt:55 (include)
-- Configuring incomplete, errors occurred!
See also "/ldfssz1/ST_META/P18Z10200N0127_GRJ/liangweiting/software/stronginstall/STRONG/SPAdes/assembler/build/debug/CMakeFiles/CMakeOutput.log".
See also "/ldfssz1/ST_META/P18Z10200N0127_GRJ/liangweiting/software/stronginstall/STRONG/SPAdes/assembler/build/debug/CMakeFiles/CMakeError.log".
↓
on /ldfssz1/ST_META/P18Z10200N0127_GRJ/liangweiting/software/stronginstall/STRONG/SPAdes/assembler/build/debug/CMakeFiles/CMakeError.log:
Detecting C OpenMP failed with the following output:
Change Dir: /ldfssz1/ST_META/P18Z10200N0127_GRJ/liangweiting/software/stronginstall/STRONG/SPAdes/assembler/build/debug/CMakeFiles/CMakeTmp
…………
…………
…………

Yes it is completely necessary. These are some supplementary tools builds using SPAdes library for kmer coverage/ subgraph extraction/ high resolution graph processing....
It looks like you're missing OpenMP. If you didn't do it yet I advise you run :
sudo apt-get -y install libbz2-dev libreadline-dev cmake g++ zlib1g zlib1g-dev
I think OpenMP is installed as part of these. Otherwise It seems OpenMP is available as part of gcc > 4.2.0.

Thank you for your help! I'm sorry for the late reply. I connected the administrator to add the OpenMP, and now it seems could run, but the new error appeared.
I have tried several times and the program "./build_cog_tools.sh" stoped in different places.
##############################################################
[ 73%] Linking CXX static library libspades-stages.a
[ 73%] Built target spades-stages
make: *** [Makefile:171: all] Error 2
[ 73%] Built target hmmer
[ 74%] Linking CXX static library libspades-stages.a
[ 74%] Built target spades-stages
[ 74%] Linking CXX static library libpath_extend.a
[ 74%] Built target path_extend
make: *** [Makefile:171: all] Error 2
[ 67%] Built target jemalloc-syms-internal
[ 68%] Linking CXX static library libspades-stages.a
[ 68%] Built target spades-stages
make: *** [Makefile:171: all] Error 2
[ 70%] Built target path_extend
make: *** [Makefile:171: all] Error 2
###############################################################
I don't kown what went wrong, so I pick up the log's highlight.
-- Found readline: /usr/include
-- Looking for wsyncup in /usr/lib64/libncurses.so
-- Looking for wsyncup in /usr/lib64/libncurses.so - not found
-- Looking for wsyncup in /usr/lib64/libcurses.so
-- Looking for wsyncup in /usr/lib64/libcurses.so - not found
-- Found Boost 1.70.0 at /ldfssz1/ST_META/share/User/zhujie/.conda/envs/bioenv/lib/cmake/Boost-1.70.0
--   Requested configuration: QUIET REQUIRED
-- Found boost_headers 1.70.0 at /ldfssz1/ST_META/share/User/zhujie/.conda/envs/bioenv/lib/cmake/boost_headers-1.70.0
-- Found Boost: /ldfssz1/ST_META/share/User/zhujie/.conda/envs/bioenv/lib/cmake/Boost-1.70.0/BoostConfig.cmake (found version "1.70.0")
………………
-- Performing Test malloc_usable_size_const
-- Performing Test malloc_usable_size_const - Failed
………………
-- Performing Test JEMALLOC_HAVE_MACH_ABSOLUTE_TIME
-- Performing Test JEMALLOC_HAVE_MACH_ABSOLUTE_TIME - Failed
………………
-- Looking for memalign
-- Looking for memalign - not found
………………
-- Looking for issetugid
-- Looking for issetugid - not found
-- Looking for _malloc_thread_cleanup
-- Looking for _malloc_thread_cleanup - not found
-- Looking for _pthread_mutex_init_calloc_cb
-- Looking for _pthread_mutex_init_calloc_cb - not found
………………
-- Performing Test HAS_MADVISE_MADV_FREE
-- Performing Test HAS_MADVISE_MADV_FREE - Failed
………………
-- Performing Test JEMALLOC_OS_UNFAIR_LOCK
-- Performing Test JEMALLOC_OS_UNFAIR_LOCK - Failed
………………
-- Performing Test JEMALLOC_OSATOMIC
-- Performing Test JEMALLOC_OSATOMIC - Failed
………………
-- Performing Test _Wshorten_64_to_32
-- Performing Test _Wshorten_64_to_32 - Failed
………………
-- Looking for include file cxxabi.h
-- Looking for include file cxxabi.h - not found
………………
-- Looking for include file mach/mach.h
-- Looking for include file mach/mach.h - not found
………………
-- Looking for shm_open
-- Looking for shm_open - not found
………………
-- Looking for erfc
-- Looking for erfc - not found
………………


chore: New feature flag

💔 Breaking Changes
🏆 Enhancements
📜 Documentation
🐛 Bug Fix
🏠 Internal
Changing the name to better reflect what the feature flag is doing.

@suddjian is this a breaking change we change the name of a feature flag?

IMO this should be considered a breaking change (see SIP 57). It could be made a non-breaking change if measures are taken to make it so that the old name still works.

@suddjian currently only one place in superset is using this featureFlag, and it is the database connection ui.

@AAfghahi I would recommend adding the new flag instead of changing the name. You can remove the old flag later- mark that it is deprecated.

The codebase on this repo has been moved to the main Apache Superset repo, and consequently the repo is in the process of being archived. See the Superset Improvement Proposal for details: https://github.com/apache/superset/issues/13013 . While all currently open issues and PRs will be closed, we encourage you to reopen this PR on the main repo, which should be as simple as moving over any code changes as follows:

core packages: packages/** (this repo) -> superset-frontend/packages/** (main repo)
plugins: plugins/** (this repo) -> superset-frontend/plugins/** (main repo)

If you need help with the migration, please post a message on the SIP or reach out on the community Slack.


Adding a sort confirmation box

I'm trying to add a confirmation box to my sorting table to avoid sloppy clicking.  When a user sorts and item a bootstrap modal opens up and asks if they're sure.  If they say "OK" the sort stays.  If they cancel the sort reverts.
The problem I'm running into is that when I use the update method and fire the modal, the code continues to execute and I can't call ui.item.sortable.cancel() because it's undefined by the time there's a user click.  Here's my code:
var SortController = function (vm, $modal) {
vm.sortableOptions = {
    placeholder: 'ui-state-highlight',
    containment: 'parent',
    cursor: 'move',
    handle: '.dragger',
    tolerance: 'pointer',
    start: function (e,ui) {
        ui.placeholder.height(ui.item.height());
    },
    update: function (e, ui) {
        var modalInstance = $modal.open({
            templateUrl: 'confirmSort.html',
            controller: 'ModalInstanceCtrl',
            size: 'sm',
            resolve: {
                objText: function() {
                    return vm.objText;
                }
            }
        });

        modalInstance.result.then(function () {
            //sort confirmed
        }, function () {
            //revert sort
            ui.item.sortable.cancel();
        });
    }
};
};

Thanks for any help.

I have added a Promised Reverting example to README.
You might find it useful.


Dev

Updated the needed information for both clusters.

oops..I need to merge with the forked branch


Added link to CodeIgnitor documentation.

Unique design and navigation. Easy to read typography.

Awesome. Thanks.


[SPARK-45562][SQL][FOLLOWUP] Restore test case not require rowTag option.

What changes were proposed in this pull request?
https://github.com/apache/spark/pull/43389 makes rowTag a required option. But the xml API (please see https://github.com/apache/spark/blob/7057952f6bc2c5cf97dd408effd1b18bee1cb8f4/sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala#L579C1-L579C1) is unrelated to rowTag.
This PR also improves some code and remove one line of unused code.
Why are the changes needed?
Restore test case not require rowTag option.
Does this PR introduce any user-facing change?
'No'.
How was this patch tested?
Exists test cases.
Was this patch authored or co-authored using generative AI tooling?
'No'.

cc @sandip-db

Merged to master.

@HyukjinKwon @sandip-db Thank you!


Move sources from cli-ng to common

Signed-off-by: David Greven<EMAIL_ADDRESS>

@sbckr Blocked until discussed internally.

https://github.com/carbynestack/common/blob/4224fd439a52443ab3457c0d9161e5caeb684c9d/.pre-commit-config.yaml#L8-L12
Needs to be removed because of JDK 17 (see carbynestack/cli-ng#18).

@sbckr @strieflin Test coverage can be ignored in this case, as the values cannot be accurately calculated due to the division of classes between common-testing and common-types.


Fails to connect.

Earlier today, YakYak was working, then it just said "Not connected" all the time and refuses to reconnect no matter what I do. Including reinstalling it.
I tried updating to 1.3.2 (from https://github.com/yakyak/yakyak/releases/tag/v1.3.2), and it still says the same thing.
Using YakYak 1.3.2 on Windows 10 x64.
(Also, in 1.3.2's yakyak-win32-x64.zip, ./version says "v1.2.5" for some reason? I even tried downloading and extracting it twice just to be sure.)

You need to update to v1.4. Google has changed their hangouts protocol and earlier versions will no longer connect.

@PSiAU I am Mac user and there's no 1.4 OS X prebuilt version ;)

There is, use the darwin zip for 1.4 beta 4 which is the same as the final 1.4 version which according to Andre should be available tomorrow. I'm using it myself on a Mac and it's working great.

built v1.4.0 and put it here
https://bintray.com/drewwells/generic/download_file?file_path=yakyak-osx.app.zip
sha256: 4598c4a318e46cd5aee4b8aff93a994aa6f790543f845f2b425a4dac654f56f5

Anybody know why there isn't a build from source version of yakyak in homebrew? All I found was a cask which depends on prebuilt os x binaries: https://github.com/caskroom/homebrew-cask/tree/master/Casks

PR open for a homebrew formula https://github.com/Homebrew/homebrew-core/pull/7477

thanks @PSiAU.

officially released 1.4.0

anybody know why the homebrew formula is emitting a vanilla Electron app?
    system "npm", "run", "deploy:darwin-x64"
    prefix.install "dist/darwin-x64/YakYak.app"
    bin.install_symlink prefix/"Yakyak.app/Contents/MacOS/YakYak"

This works fine when I do it manually, but I'm getting some kind of electron.app via homebrew.

the arch targets do not run gulp


Debugging AppleScript?

Is there any way this plugin does, or can, do AppleScript debugging (perhaps similar to Late Night Software's Script Debugger)?
Two things here:

I can't seem to find any way that it does it already, but want to check I'm not missing anything.
If that's right and it doesn't currently do it, what would it take to get it to do it?

To really clarify, in case there's any ambiguity:  I'm looking/hoping for an experience that's similar to VSCode's built in Node debugging (but with AppleScript of course).
If it's not already possible, I imagine it's a lot of work, but... I'll bet I'm not the only one who would really like this.
Does this somehow do this already and I'm missing it?  Or if not, is it possible to add this?
Thanks!

I suppose no answers to this question in the past 17 days means this extension does not provide any AppleScript debugging?
I was hoping for the developer response to this question.

I'm not aware of possibilities, e.g. CLI tools, to debug AppleScript files


Fix typo in Makefile

Codecov Report

Merging #3977 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3977      +/-   ##
==========================================
+ Coverage   78.99%   78.99%   +<.01%
==========================================
  Files         106      106
  Lines      142244   142244
==========================================
+ Hits       112369   112372       +3
+ Misses      29875    29872       -3

Impacted Files
Coverage Δ

src/gui_gtk_x11.c
48.37% <0%> (-0.2%)
:arrow_down:

src/window.c
83.61% <0%> (+0.09%)
:arrow_up:

src/gui.c
58.2% <0%> (+0.2%)
:arrow_up:

Continue to review full report at Codecov.

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


Support returning a native call stack asynchronously through plugins

I didn't find an existing bug for this.
Internal issue b/215360895.
Currently, Flutter has the ability to return native stack traces through a plugin synchronously. For an Android plugin implementation, it even does so if an exemption is thrown synchronously while inside the MethodCallHandler#onMethodCall.
However, if the Java method call handler goes async and later returns an error through the Result object, there's no way to pass the native stack trace back through. This should be a supported use case.
Let asynchronous errors detected in the plugin implementation have feature parity (in terms of returning a stack) as synchronously detected errors.

Trying to understand the ask a bit better here: It seems that you could put any string you want in the errorDetails field on the Result object. Am I misunderstanding? Can you sketch the API you are seeking?

MethodChannel currently implements onMessage as
      final MethodCall call = codec.decodeMethodCall(message);
      try {
        handler.onMethodCall(
            call,
            new Result() {
              @Override
              public void success(Object result) {
                reply.reply(codec.encodeSuccessEnvelope(result));
              }

              @Override
              public void error(String errorCode, String errorMessage, Object errorDetails) {
                reply.reply(codec.encodeErrorEnvelope(errorCode, errorMessage, errorDetails));
              }

              @Override
              public void notImplemented() {
                reply.reply(null);
              }
            });
      } catch (RuntimeException e) {
        Log.e(TAG + name, "Failed to handle method call", e);
        reply.reply(
            codec.encodeErrorEnvelopeWithStacktrace(
                "error", e.getMessage(), null, getStackTrace(e)));
      }

In the case of a synchronous unechecked exception, the exception is passed to codec.encodeErrorEnvelopeWithStacktrace(). The Result interface should support the same call through. Maybe adding an overload to Result.error() that can be implemented as
              @Override
              public void error(String errorCode, String errorDetails, Exception e) {
                reply.reply(codec.encodeErrorEnvelopeWithStacktrace(
                    errorCode, e.getMessage(), errorDetails, getStackTrace(e)));
              }


Extension extremely slow when running with Ocp sandbox

I'm opening this issue just to share my findings. Maybe someone will find it helpful when using the search bar.
Or we could update the doc mentioning it.
When I start the extension on ocp sandbox it takes a lot to load. After investigating a bit it looks like my old kubectl version causes it.
Kubectl 1.22

Kubectl 1.26

Solution - use an updated version of kubectl.

@mohitsuman related to this... I was thinking why we have a specified version range here for kubectl https://github.com/redhat-developer/vscode-tekton/blob/main/src/tools.json#L7
What's the point? For the tkn cli we always target the latest release as we assume users want to use the latest operator but for kubectl this is not true. I could use an older cluster with the latest operator.
And what's the point of checking the version range if the user can face a similar issue as above? Or we download/ask the user to use a kubectl version in line with the cluster version or having those static numbers have no sense. WDYT?

I think the idea is to have the kubectl version in the first place to make sure we support the latest release, in sync with the security fixes (CVEs if any). I agree we do not need version range check anymore as the specific version makes more sense.
We can remove the version range check accordingly.


README updates

I think the new image is missing though, I get a 404 on https://cdn.rawgit.com/cilium/cilium/master/Documentation/images/cilium_overview.png

Coverage decreased (-0.02%) to 37.106% when pulling d82e981ba9a3dfba1ca5617e4c1f87687dcdd675 on pr/tgraf/readme into df367832502efffea998a9325be1d4a55cd5db1a on master.


Update framework version

$subject

Skipping pr builder since this includes only one change from framework repo, and that change already succeeded with pr builder.


Research of creation custom service extension for CLI <--> app test communication

Idea was sparked by @jakubfijalkowski in this comment.

Currently, I'm registering the service extension in the test and calling from the driver, but I want it the reverse.
Will poke around with this tomorrow.

Works. Wow.
Caveats:

gotta find a way to find test. Should take a look at how Flutter does it
HostAutomator.takeScreenshot() will crash when more than single device is attached


Invoke the dialog a second time to change the tracking choice

I can't invoke the dialog a second time to change the tracking choice.
I would like to make the dialog visible again, without forcing the user to go into the app settings.
How can I do?

In iOS, once the user has made their decision on whether to allow or deny app tracking, the app tracking authorization dialog cannot be shown again for that specific app. The user can still change their decision from the Settings app, but the app cannot request authorization again.


NFC - Transform strategies cleanups

Address remaining comments from #11597 re. trimming unnecessary dependencies
NFC - Rename ReductionStrategyThreadDistribution to ReductionStrategy3StageThreadDistribution and make space for another strategy
Move errs spew to dbg to avoid polluting non-debug runs

Commits to main probably need a better description

Commits to main probably need a better description

Don't these automatically get populated with the bulletpoints from the underlying commits on merge?
I thought I saw that occur a bunch of times, doing it manually ..

Git does this if you manually squash commits before pushing. Not sure if Github does this automatically, judging by the recent https://github.com/iree-org/iree/commit/4d85a79a1b7f81370404a3e8eeb390ade25feceb it does not.


Enable zoom for Speaker segmentation task

I am performing Speaker segmentation task in Label Studio (LS) for audio data. The demo as presented here shows that we can zoom the audio waveform in x-axis i.e. time axis as well. But even when I copied the same configuration to my use case, the feature does not work. The audio clips I am dealing with are more than 10mins long; when loaded to LS the waveform looks very dense making the annotation virtually impossible and also making (time/ x-axis) zoom a must have feature for my use case.
Here are my configurations:
<View>
  <Labels name="label" toName="audio" zoom="true" hotkey="ctrl+enter">

  <Label value="spk_0" background="#0eb4f1"/>
  <Label value="spk_1" background="#D4380D"/>
  <Label value="spk_2" background="#aef12b"/>
  <Label value="spk_3" background="#7f1ac3"/>
  </Labels>

  <Audio name="audio" value="$audio"/>

  <Header value="Confirmation"></Header>
  <Text name="text" value="Are the annotations reviewed?"></Text>
  <Choices name="choices" toName="text" choice="single" showInLine="true">
    <Choice value="Yes" />
    <Choice value="No" />
    <Choice value="Pending" />
  </Choices>
</View>

I also tried using AudioPlus tag (link) instead of Audio but the issue still persists.
Attaching UI Preview for reference.

OS: MacOS
LS version: 1.7.0
Is there anything I am missing out to bring the zooming feature for audio waveform? Kindly guide me on this.

@makseq kindly help!

Hey @Ayush-TCIN 👋. Zoom in the x-axis is controlled by either cmd + scroll wheel (if using mac os and a mouse), ctrl + scroll wheel (if using windows, or linux and a mouse). If using a laptop with a multitouch trackpad you can also pinch to zoom as well. Hopefully that works for you, let me know if it doesn't.

Yes, it worked!
Thanks @bmartel


Updates and improvements (GSI-719)

Use Python 3.12 as the default Python version
Update gh-action-common to v6 which uses Python 3.12
Update some dependencies
Remove the Python VARIANT in docker-compose (was not used)
Remove the Python version in pre-commit config (uses the default Python version then)
Allow overriding settings in the pyproject.toml template via custom file
Do not mangle values with newlines in pyproject.toml (tox.ini became unreadable)
Added htmlcov as exclusion to the license checker
Renamed license checker to check_license since update_...` commands also start with a verb

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.667%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
30

Relevant Lines:
72

💛  - Coveralls


Problem running on Windows PC

Problems with running code on windows. Getting the following error see screenshot. Path issues model_output files (log_return works) but no files are generated for raw, scaled ... Conversely, the calculations for RMSE do not work either. Thanks for the help in advance

@Alex1996-alt It seems that the column 'close' is missing in your prediction data. Can you show me what is in your /pred_{period}.csv?

Hey thanks for your help. I will upload my output files maybe that is more helpfull to solve it:)
output.zip
src.zip

I accidently put the raw-to-log model in experiment/utils.py in line 82, you can delete this line and it should work then.

Hi you mean log_to_raw in line 82? I dont have raw_to_log  here (see screenshot) Thanks for your time :)

@Alex1996-alt Yeah my bad I meant that one, it's deleted with the latest commit: https://github.com/StephanAkkerman/crypto-forecasting-benchmark/commit/ffe49db486560b176ccb50f9d012cd2f75a1eb2c

Hi thanks in advance for the help that is really very nice of you. I have at least made some progress and now I get the following error message. See screenshot. All rmse files could be generated except for scaled_to_raw. Thank you again hopefully you can help me with the next point :)

src.zip

For this you can remove the scaled_to_raw in the list too, that should fix it.

Hi, everything seems to be working fine for now thank you very much. A general question of understanding. What exactly are the output files such as log_to_raw, raw_to_log etc. relevant for and what exactly are we training the models for? As I understand it, only the raw, scaled and log returns are of interest to us, aren't they?

What exactly are the output files such as log_to_raw, raw_to_log etc. relevant for
This is to see if the transformation of the data had any impact on the performance, for instance log_to_raw uses the model's forecast that was trained with logistic regression data and then the output is converted back into raw (close price). I did this to see if there would be big differences in transforming the output in comparison to transforming the input data.
What exactly are we training the models for?
To predict one step ahead, for instance what the price of Bitcoin will be tomorrow given the last x days of data.
only the raw, scaled and log returns are of interest to us, aren't they?
Yes, this is what we need as output and then all the other formats can be converted afterwards. If you are purely interested in getting the best performance you could drop the raw and maybe even the scaled (depending on the models you want to use).

Unfortunately, the website of the university is having difficulties right now it seems. I think if you're purely interested in finding out the best model, then reading through the paper would be the easiest.
Would you mind sharing what your goal is?

Hey yes, among other things I want to see how the models perform and what relevance data properties have. I might look at a few other time frames and a few less models and cryptocurrencies. I also wanted to implement the Holt Winters model and see how much worse or maybe half as good the model performs. I'm currently writing my master's thesis and I'm trying to see if the somewhat simpler models like ARIMA and Holt Winters still perform well by implementing Holt Winters.

I've added the PDF since the website that's hosting my paper is currently down. Hopefully that helps you with a bit of the data properties. If you are looking for performance, then I would suggest looking into models that are built for cryptocurrency / stocks forecasting.

Thank you very much since I am an economist and not the best at programming your code and work is helping me really a lot :)

Glad to hear, let me know if you need any help!

Hi, I've run into another error (see the screenshot). It’s trying to access the file "Ljung-Box_log_return" in the Statistics folder, but unfortunately, that file doesn’t exist on my end. Could you please take a look at this or let me know in which script and function the file is generated? The screenshot also shows my Statistics folder with all the files created by the analysis.data_analysis_tests() function. If you need my current code for this, just let me know. Thanks a lot for your time!

Good find, I improved that function but apparently there is still some old code left. I will try to fix it today. The data can now be found in auto_correlation_results_log_returns excel file. So the column for the Ljung-Box is located in there, instead of a separate file for this data.

All right, thanks. Please let me know when the error has been fixed. And thanks again for your time

Think I am missing a lot more files here

@Alex1996-alt it should be fixed now, you can use the new data_properties.py file

@Alex1996-alt Thanks for reporting it once again, I updated volatility to automatically create the files if they do not exist

Hi I think we are getting close :) That is the next missing file and error

@Alex1996-alt I once did a large overhaul near the end of the thesis, which is why the code that is part of the beginning has some errors. Any missing file errors in data_properties should be fixed now.

Yeah no worries. I still thank you for helping me so much to get everything running. Unfortunately, I'm getting the following error now, any ideas?

Should be fixed now!

Hey thanks. I am getting the next error :D Thank you so much for your time

I'll look into it in a bit and will do a full run locally to (hopefully) prevent future bugs

Thank you very much, please let me know as soon as everything is up and running. :)

@Alex1996-alt I improved both analysis.data_analysis_tests() and analysis.forecast_statistical_tests() and should work now without any issues. Let me know if there's anything else you find.

Hi yes everything is working well so far, thank you very much for your help. The only thing that is not yet saved are the comparison files in the output folder. Any idea what I can do best there?

@Alex1996-alt I have added the call to that function to main, so if your config is set up correctly it should run and create all comparisons

Nice thank you very much comparison files creation is working :) Getting one last error  for the analysis.market_factors_impact() function. Thanks for your time

It has been fixed and will now output both test results, I set group_tf to False but you can also set it to True to see both results

hey thank you so much for your time everything is working fine so far :)

hey how are you


Fix bug with scene collection tiles and runtime updates

Fixes #75800

Added logic to clear cached tile data for a given coordinate if the new tile placed in that location is not of type TileSetAtlasSource. This prevents crashes later as we try to clean up data.

Updated to not call the gdscript method unless it's a TileSetAtlasSource and reverted documentation changes.

Sooooo I checked the code, and I am not sure this is the right way to fix the bug. I think the current code is mode or less correct, and that the cache is simply not cleaned up correctly.
Each runtime TimeData* is memdeleted, but the runtime_tile_data_cache itself is not. So it may contain entries that would point to junk. I think the correct fix would be to change the cleanup to this (around line 922):
		// Clear the list
		while (dirty_quadrant_list.first()) {
			// Clear the runtime tile data.
			for (const KeyValue<Vector2i, TileData *> &kv : dirty_quadrant_list.first()->self()->runtime_tile_data_cache) {
				memdelete(kv.value);
			}
			dirty_quadrant_list.first()->self()->runtime_tile_data_cache.clear() // <- I added this line
			dirty_quadrant_list.remove(dirty_quadrant_list.first());
		}

Could you try that maybe?


Testing in Rapidminer

Using jdbc Release 3.0.1 on Neo4j 3.0.5, using the (GUI) Read Database operator within Rapidminer.
Connection string is
jdbc:neo4j:bolt://localhost:7687/?debug=false,noSsl,flatten=1000
The resultset for this simple cypher statement looks promising:
Match (n:mynode)
WHERE NOT n.aninteger is NULL
RETURN n.aninteger
..however it fails when I comment out the second line:
org.neo4j.driver.v1.exceptions.value.Uncoercible: Cannot coerce NULL to Java double
However, it can handle null values of a sring field just fine in the output.
Also, if I try just this:
Match (n:mynode)
RETURN n
Then I get:
org.neo4j.driver.v1.exceptions.value.LossyCoercion: Cannot coerce FLOAT to Java float without losing precision
Although, I'm not yet sure which field it's referring to here.
Also, where the Neo4j browser would happily return arrays, the driver doesn't (the field is just null).

Hi @whitfa99, I'll investigate and ping you back soon. It'd greatly simplify our work if you could create a test graph that certifies the problem.

Also could you please send us the Java code so that we can see the way you're using the driver?

I can reproduce the first error using the domain created by this:
UNWIND [{name:'A', aninteger:1}, {name:'B',aninteger:2}, {name:'C',aninteger:null}, {name:'D',aninteger:4}] AS properties CREATE (n:mynode) SET n = properties
RETURN n
Interestingly, while reproducing this for you I first only created the 'aninteger' property, which produces the expected result when using:
Match (n:mynode)
WHERE NOT n.aninteger is NULL
RETURN n.aninteger
,but when you exclude the 2nd line here the result set is
n.aninteger
null
null
null
null
If instead you create the nodes with both 'name' and 'aninteger' properties and retry the same then it throws the Cannot coerce NULL to Java double error.
As far as the java code is concerned I'm not entirely sure because I've only added this project's jar file (neo4j-jdbc-driver-3.0.1.jar) to the classpath of the Rapidminer application and utilised it's GUI to create the jdbc connection as described here.
Full Disclosure:  I am using Rapidminer 5.31, which was the last fully open source version (imho).  I don't imagine it's changed much though.  Also, I ended up deleting  com.fasterxml.* from the jar because it was meddling with a v2.4.0 of the same package elsewhere in the classpath.
If that last bit has caused the anomaly I will gladly apologise and go crawl back under my rock.

Thanks for the additional info @whitfa99.
com.fasterxml.* is used for the HTTP protocol. If you just need to use the BOLT one (which is faster) then you could bring the protocol-specific jar (please have a look at the documentation here). The jar is downloadable here. This can save you from loading the all-protocols driver neo4j-jdbc-driver-3.0.1.jar and removing unwanted pieces inside it.
I'm keen to think that your config should be ok, but I still need to have a look at Rapidminer (I have never used it). In the meantime, if it sounds reasonable for you, you can keep the ResultSet clean either by adding the WHERE NOT n.aninteger is NULL or by coalescing the result like this RETURN coalesce(n.aninteger, 0).
Lorenzo

Thanks for the tip Lorenzo.  The domain I'm using in this thread is just for reproducing the error.  I can't do without those nulls :)  Perhaps if it's feasible, I'll hold out for a fix, assuming it's also something that affects other people.  I can see that Knime is also mentioned elsewhere here and Rapidminer is in the same space.
I'm looking to replace my existing neo4j>rapidminer method, which currently involves me using the http REST interface and then a rather slow XML deserialiser.  Hoping to replace with something less convoluted and more "Bolty-er" :)

I think the problem is that the "int" type is primitive and doesn't allow for nulls at all.
We would have to coerce it to 0 then instead of null, but not sure how common that case is.
Not sure what rapidminer uses internally to access the fields.
perhaps if you can get any access to stack traces / logs of rapidminer that would be helpful.
Not sure how that's solved in SQL.
Did you try return toString(n.aninteger)

I configured (for the very first time Rapidminer) like that:

Defined neo4j driver params + uploaded the jar file:

Defined a connection to a specific instance of neo4j (localhost):

Defined a very basic Rapidminer process that executes a cypher query and automatically print the result:

Defined the query to be executed match (n:mynode) return coalesce(n.aninteger, 0):

Run the process:

So coalescing could be a temporary workaround...
If I remove the coalesce like this match (n:mynode) return n.aninteger, I get the above mentioned exception:

For completeness, we have to implement (as it's in the roadmap) some metadata methods for schema, because I'm observing this exception also:

@whitfa99 I can rapidly provide you a patch for that, but a new 3.1 release is planned for half if October. This mean I'll give you a snapshot jar.

That would be fantastic Lorenzo.  Thanks for your suggestions guys.

Hey @whitfa99,
can you try to replace the 3.0.1 driver jar with this 3.1-SNAPSHOT version and see if it sounds good for you? https://github.com/neo4j-contrib/neo4j-jdbc/releases/download/3.1-SNAPSHOT/neo4j-jdbc-bolt-3.1.0-SNAPSHOT.jar
Let me know!
Thanks,
Lorenzo

Thank you - this looks promising! I no longer get the Cannot coerce NULL to Java double error.  The fields/attributes/columns are being typed/cast correctly as far as I can see, with the exception of any column which has all null values for the returned rows, whereas in my version of Rapidminer at least, it then defaults(?) to nominal (string).
I am however getting a NPE when the recordset should otherwise return empty rows such as:
Match (n:mynode)
WHERE n.aninteger = 5
RETURN n.aninteger, n.name
Also, I've run some other queries and I'm seeing some odd behaviour.  Is there perhaps some code that samples X rows and decides that if a column has been null then it should be for the whole dataset?  That's what I'm guessing.  I have simplified one of my queries like this (you don't have the domain for this):
Match (n:mynode)-[r]-()
WHERE NOT r.adouble is NULL
RETURN r.adouble
Which returns 299 rows of populated values with Rapidminer typing it as Real.  If I comment out the WHERE statement, then I get 536 rows of nulls (299 of them should not be!), with the Type now being Nominal.  If I run the same statement in the Neo4j Browser, then I get all 536 rows including 299 populated values, the rest are null, the first 190 of these nulls are in the first rows.
If I then change the order in the statement in Rapidminer like this:
Match (n:mynode)-[r]-()
RETURN r.adouble
ORDER BY id(r)
Then I get the 299 populated values, plus the 237 nulls = 536 records as expected (the same as the output in Neo4j Browser).  Presumably, this is because the first 190 field values are no longer all null.
Apart from that, all looks good and runs very quickly.
Also, are arrays in plan to be handled in a similar way as the browser?  I don't currently get any output of them using the driver in this way.
Adrian

@whitfa99 if you could share a minimal dataset to work on together then it'd be great.

This dataset also works:
WITH REDUCE (Build=[], n in range(1,500)|Build+{NodeNo:n,adouble:CASE WHEN n>200 THEN n*0.01 END}) as mapcollection
UNWIND mapcollection AS map CREATE (n:mynode) SET n = map
RETURN n
If you then run this in Rapidminer:
Match (n:mynode)
RETURN n.NodeNo,n.adouble
ORDER BY n.NodeNo DESC
Then you get what you expect, however if you run this:
Match (n:mynode)
RETURN n.NodeNo,n.adouble
ORDER BY n.NodeNo ASC
Then you only get null values for n.adouble

Great, thank you. Let me try.

This works correctly in version 6, see
try (var result = stmt.executeQuery("MATCH (n:mynode) RETURN n.aninteger ORDER BY n.order_prop ASC")) {
	assertThat(result.next()).isTrue();
	assertThat(result.getInt(1)).isEqualTo(42);
	assertThat(result.next()).isTrue();
	assertThat(result.getInt(1)).isZero();
	assertThat(result.wasNull()).isTrue();
	assertThat(result.next()).isFalse();
}

I cannot test in RapidMiner, as the latest version of the product only works with a quite dated JDK 11, therefor I keep this issue open.


Connection Error 20191030

error type: ConnectionError: HTTPSConnectionPool(host='raw.githubusercontent.com', port=443): Max retries exceeded with url: /houtianze/bypy/master/update/update.json (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x000002965877F448>: Failed to establish a new connection: [WinError 10060] 由于连接方在一段时间后没有正确答复或连接的主机没有反应，连接尝试失败。'))
Issue:
by = ByPy()
then, error comes out like pics above.
I try different accounts on different computers, but the results are same. Is there anything wrong with your sever? or any suggestions for solving this?
Thank you.

It's a temp solution.
edit lib/python3.7/site-packages/bypy/bypy.py

It's a temp solution.
edit lib/python3.7/site-packages/bypy/bypy.py

This is really helpful! Thanks a lot :)


Adding language to tweet metadata

This adds the language of the tweet (pulled from the html) to the tweet metadata by adding the attribute t.lang to the Tweet object, as well as updating write_meta.py and format.py to allow for saving.

Great! Would also be possible to update the schema for Elasticsearch, Database and Panda? The concept is the same.
For Elasticsearch

edit createIndex function (lang would need to be keyword, see hashtag as an example)
edit Tweet function

For Database

edit the tweets' table schema in the init function
edit tweets function, don't forget to add a ?

For Pandas

edit update function

Will do.    I knew there were probably other places in the code that needed this added but wasn't sure where!

I apologize, I have been insanely busy with work and trying not to code in my free time...  I will try to get to this this week though.

Okay Francesco,  I THINK I've done all this successfully.    You may want to look at the elasticsearch and database changes just to make sure I didn't screw that up.

Seems nice to me


Feature request: Create OpenFAST input files from the WindIO format [openfast_io]

Is your feature request related to a problem? Please describe.
No, It's a new feature request for the upcoming openfast_io library.
Describe the solution you'd like
The ability to create OpenFAST input files from the 'WindIO' format using openfast_io library
Describe alternatives you've considered
Using WEIS to generate the files, but this requires a larger space and memory overhead.
Additional context
WindIO is a framework defining the inputs and outputs for systems engineering MDAO of wind turbine and plants. The framework was developed by the IEA Wind Task 37 team within Work Package 1. https://github.com/IEAWindTask37/windIO

I would say that it would be great to also develop the inverse of this capability, i.e., the ability to generate WindIO format from a given OpenFAST model.  Two-way conversions between WindIO format and various aero-elastic models will allow us to use WindIO as a mediator for code-to-code converters.
Best regards,


[KEP 0025] Template toYaml function

What this PR does / why we need it:
This KEP describes addition of a function called toYaml for use in the operator templates.
Signed-off-by: Marcin Owsiany<EMAIL_ADDRESS>

Do we need a KEP for this? IMO, the motivation is already covered in https://github.com/kudobuilder/kudo/issues/1221. And adding a toYAML function that marshals an interface{} into YAML is maybe 10 lines of code. The only thing needed is an implementation of KEP-24, that allows parameter values that are more complex that string.

Do we need a KEP for this?

So, your comment on KEP-24 that toYaml should not be a part of that KEP somehow made me think that it's big enough to warrant its own KEP. Then the way I read the guidelines I thought that it might be borderline, but does not hurt. So I just did it :-P
If you think it's a no-brainer please just hit the approve button :tada:  :-)

@porridge You're right and I don't want to discourage anyone from creating new KEPs. Just be aware that a KEP has a bit more overhead :)


Custom IParameterConsumer causes native-image to fail

Using a file with a custom parameter consumer:
import picocli.CommandLine.*;
import picocli.CommandLine.Model.*;
import java.util.Stack;
import java.util.concurrent.Callable;
import picocli.CommandLine;

@Command(name = "checksum", mixinStandardHelpOptions = true, version = "checksum 4.0", description = "Prints the checksum (MD5 by default) of a file to STDOUT.")
public class Boom implements Callable<Integer> {
	@Parameters(index = "0", description = "Ignored", parameterConsumer = LocalPC.class)
	private String file;
	public static void main(String... args) {
		System.exit(new CommandLine(new Boom()).execute(args));
	}
	@Override public Integer call() throws Exception {return 0;}
}
class LocalPC implements IParameterConsumer {
	@Override public void consumeParameters(Stack<String> args, ArgSpec argSpec, CommandSpec commandSpec) {return;}
}

$ java -cp boom.jar:picocli-codegen-4.0.3.jar picocli.codegen.aot.graalvm.ReflectionConfigGenerator Boom  > test.json
$ native-image -jar boom.jar -H:ReflectionConfigurationFiles=test.json -H:+ReportUnsupportedElementsAtRuntime --no-server
$ ./boom --help                                                                                ] EP_BAD 4:54 PM
Exception in thread "main" picocli.CommandLine$InitializationException: Could not instantiate class LocalPC: java.lang.NoSuchMethodException: LocalPC.<init>
	at picocli.CommandLine$DefaultFactory.create(CommandLine.java:4749)
	at picocli.CommandLine$DefaultFactory.createParameterConsumer(CommandLine.java:4745)
	at picocli.CommandLine$Model$ArgSpec$Builder.<init>(CommandLine.java:7662)
	at picocli.CommandLine$Model$PositionalParamSpec$Builder.<init>(CommandLine.java:8271)
	at picocli.CommandLine$Model$PositionalParamSpec$Builder.<init>(CommandLine.java:8261)
	at picocli.CommandLine$Model$PositionalParamSpec.builder(CommandLine.java:8220)
	at picocli.CommandLine$Model$CommandReflection.buildArgForMember(CommandLine.java:9773)
	at picocli.CommandLine$Model$CommandReflection.initFromAnnotatedTypedMembers(CommandLine.java:9688)
	at picocli.CommandLine$Model$CommandReflection.initFromAnnotatedFields(CommandLine.java:9641)
	at picocli.CommandLine$Model$CommandReflection.extractCommandSpec(CommandLine.java:9546)
	at picocli.CommandLine$Model$CommandSpec.forAnnotatedObject(CommandLine.java:5118)
	at picocli.CommandLine.<init>(CommandLine.java:223)
	at picocli.CommandLine.<init>(CommandLine.java:196)
	at Boom.main(Boom.java:14)
Caused by: java.lang.NoSuchMethodException: LocalPC.<init>
	at java.lang.Class.getConstructor0(DynamicHub.java:3082)
	at java.lang.Class.getDeclaredConstructor(DynamicHub.java:2178)
	at picocli.CommandLine$DefaultFactory.create(CommandLine.java:4725)
	at picocli.CommandLine$DefaultFactory.create(CommandLine.java:4748)
	... 13 more

The issue seems to be that the LocalPC class is not put into the reflection output by the ReflectionConfigGenerator, and manually putting it in I get it to build and show help successfully

Thank you for raising this issue and for the pull request!
I merged the PR but it would be good to add a test for this in picocli-annotation-processing-tests. Do you think you’ll be able to provide this?

I added a commit to my branch that should, in theory, test for this fix, but I can't seem to get it to work, and I'm not sure if it's my weak gradle skills, or what.

@byteit101, it could be because I already merged that PR. Maybe create another branch?

I might have misread. Did you mean you have trouble getting the test to work? If you can share the code we can look at it together...

Yes, the latter. I can't seem to get them to work. Works fine when I do a command line generation, but through gradle test it fails. Code is https://github.com/byteit101/picocli/commit/1e7b2f573dc802e9a1ab8f69269dc23e74b26cbf and I'm kinda stuck. Looks to me like it should work, but I'm clearly missing something, either in gradle (gradle n00b here), or in your test framework, or something else.

Code looks fine to me too. Can you put it in a pull request?

I went ahead and released picocli 4.0.4 with your PR and the additional changes to the annotation processor to correctly handle parameterConsumer attribute in the @Option or @Parameters annotations.
Please verify that this solves the issue.
Thanks again for the contribution!

This happens to my code with picocli 4.6.1 running on OpenJDK 11.

This happens to my code with picocli 4.6.1 running on OpenJDK 11.

Hi @pedrolamarao
Thanks for raising this.
Can you open a new ticket with some code to reproduce the issue? That would help me a lot!

Hi @pedrolamarao
Thanks for raising this.
Can you open a new ticket with some code to reproduce the issue? That would help me a lot!

I'll see to it next week. We discovered that this happens only if the parameter consumer is an inner non static class. An inner static class works. (If this is a known limitation, we totally missed it from the documentation.)

I'll see to it next week. We discovered that this happens only if the parameter consumer is an inner non static class. An inner static class works. (If this is a known limitation, we totally missed it from the documentation.)

@pedrolamarao Thank you for the clarification.
Yes, that is a know limitation (for more details, see this ticket: https://github.com/remkop/picocli/issues/1297),
and perhaps this is an opportunity to improve the documentation!  😄
Any suggestions (or even better, PRs) would be much appreciated.
Thanks again for raising this!

@pedrolamarao Thank you for the clarification.
Yes, that is a know limitation (for more details, see this ticket: https://github.com/remkop/picocli/issues/1297),
and perhaps this is an opportunity to improve the documentation!  😄
Any suggestions (or even better, PRs) would be much appreciated.
Thanks again for raising this!


Document the API

With @HubLot  work, the API starts to look like a proper API. Yet, it should be documented or it will be unusable by others.
I suggest to build the doc interactively. First we should have a reference documentation built on the docstrings, then we could add examples in the form of notebooks, finally we should add some narrative doc.
I suggest to use sphinx to generate the doc. The doc can be hosted either as github pages, using the pipy doc hosting service or using read the docs.
The course of action would be:

[ ] set-up sphinx to build a reference doc
[ ] clean the docstrings so they render properly
[ ] choose a hosting service for the doc
[ ] add example notebooks
[ ] include the notebooks in the sphinx doc (see http://sphinx-ipynb.readthedocs.org/en/latest/howto.html)
[ ] write narrative doc

This should be built on top of #80 or wait for #80 to be merged.

At first, I would prefer we write notebook user cases.
I want to be sure the fresh API that has just been release is robust and easy-to-use.

The steps do not have to follow a particular order. I actually started with notebooks as they are funnier to do...

;-)

Being able to install PBxplore using a setup.py script (#21) would ease greatly the notebooks.

It's on my top priority :)


Jimmy/7253 up sender transform constants

This PR involves adding tests to verify/demonstrate that constants are appropriately handled in the FhirTransformer. This was pretty straightforward with the exception of the bundleProperty field. bundleProperty is parsed on our end before passing it over to FhirPathUtils; we need to find the path to the property being updated (including each step along that path in case we can create missing properties), as well as the property name itself, and therefore, we can't depend on the FhirPathUtils to handle constants as we can for the other fields which may make use of the constants. Instead, I re-introduced the ConstantSubstitutor so we can do some pre-processing string substitution on constants that are wrapped in %{ and }.
I also stumbled upon some error cases in the process of writing these tests, so included some changes intended to make certain error cases a bit more graceful, and more clearly logged.
Test Steps:

Fhir Sender Transforms have still not been integrated in the pipeline, so for now, running the FhirTransformerTests and checking code coverage is the best way to test these changes.

Changes

New test cases to cover the acceptance criteria listed in #7253 in FhirTransformerTests.kt
Add ConstantSubsitutor to FhirTransformer.kt (and update comment about ConstantSubstitutor in FhirToHl7Converter.kt), remove currently unneeded property strict from FhirTransformer.kt
Adjust error handling and logging in FhirTransformer.kt and FhirBundleUtils.kt
Add logging checks and adjust existing error testing in FhirTransformerTests.kt

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[x] Added tests?

Linked Issues

Fixes #8652

Updated this with another commit to remove the support for %{constant} and to add some error handling to enforce some of the restrictions around the use of constants within bundleProperty. Ready for review again!

In my latest two commits, I made some refactors to hopefully make functions easier to read. Part of this involved extracting a function for validation of inputs. I also removed the parts of tests that looked for specific logging messages and instead will just verify that errors did happen. I also changed most/all of the warnings in the function to errors. I think as part of the integration into the pipeline, there will be more error handling work to be done, but I don't want to get more into that as part of this PR, since it's not really related to this story.


We need to include "Created" and "Expiration" for AkamaiIamClient Nodes

To properly enable key rotation, we need to know the dates associated with Akamai client keys

#55


feat: add new dex chart

Fixes #2

Is this a new chart or are we porting the one from the old charts repo?

Is this a new chart or are we porting the one from the old charts repo?

This is a new chart. The old one is not really optimal, but this one should already provide more or less the same features.


[Tokyo Mirage Sessions ♯FE] Models with visible textures from the inside

At times, the game will go into a 1st-person camera of sorts, literally putting you inside a character's head to see what it would see. In the real hardware, this would cause said character's model to go transparent, but Cemu keeps rendering the model from the inside, blocking the scene from view and making things look very weird. I suppose that making the backface more transparent would ameliorate that, and introducing backface culling would fix that (but might break something else).
The bug might be replicated in the very first scene of the game. I've attached a screenshot to show what I'm talking about.

I'm using a nVidia GPU, in case that's important.

Yes,I can confirm
I clear the game and it happens on some of the cut scenes
I also see it happening once on Xenogears Chronicles X
I also have nvidia gpu

As this issue can be reproduced on a basic Cemu install without any packs active, this isn't related to graphics packs.
Consider reporting it to http://bugs.cemu.info/issues, closing as not relevant.


Remove motor stub from motor UI picker

          do this

Originally posted by @jbbjarnason in https://github.com/Skaginn3x/framework/pull/458#discussion_r1487411926

Cockpit no longer utilized closing


Investigate RectAreaLightHelper individual color

Might want to tweak the Helper colour independently of the light color

Added in 1.0.1


Interact with the context via the the browser's dev tools (remote procedure call)

Similar to #86
For testing purpose, I would like to make access runtime access to the context object at the heart of a server as convenient as possible. Why not expose it through the browser's dev tools?
Not necessarily direct access. What I'm thinking is we create a proxy context object that runs in the browser, with method signatures matching the context object on the server. When you call context.reticulate('splines') on the browser object, it sends the method name and arguments to the server via a web socket. Counterfact then intercepts that request on the server side and calls context.reticulate('splines') on the server object. The return value is then JSON.stringify()ed and returned to the client.
Two consequences of this remote procedure call (RPC) approach:

It only works on methods that take JSON-serializable objects as arguments and return JSON-serializable values. Maybe we can use TypeScript to enforce that all context methods follow this convention. That feels like a good idea anyway, as it will encourage people to move business logic into the Context class instead of the /paths/**/*.ts files.

Every method on the client-side context object is asynchronous. If we care about the return value we need to use await x = context.reticulate('splines') even if the reticulate() method is synchronous on the server side.

Re the first point, this isn't absolutely necessary, but here's as far as I've gotten in constraining the methods to only receive and return serializable objects.

type primitive = string

class AbstractContext  {
    [k: string]: (...args: primitive[]) => primitive;
}

class Context extends AbstractContext {
  foo(a: primitive, b: primitive) {
    return "a string"
  }

  bar(foo: primitive) {
    return "also a string"
  }
}

Closing because I don't think this will be very useful. The REPL in the terminal is fine.


Add linux aarch64 wheel build support

Add linux aarch64 wheel build support.
Related to https://github.com/mattandahalfew/Levenshtein_search/issues/24 @mattandahalfew Could you please review this PR?

Sure- I'm reviewing now

I'm getting the error on travis-ci:
Uploading Levenshtein_search-1.4.5-cp36-cp36m-manylinux_2_17_aarch64.manylinux2014_aarch64.whl 100%|██████████| 164k/164k [00:01<00:00, 125kB/s]  Uploading Levenshtein_search-1.4.5-cp37-cp37m-manylinux_2_17_aarch64.manylinux2014_aarch64.whl 100%|██████████| 165k/165k [00:00<00:00, 190kB/s] Uploading Levenshtein_search-1.4.5-cp38-cp38-manylinux_2_17_aarch64.manylinux2014_aarch64.whl 100%|██████████| 164k/164k [00:00<00:00, 169kB/s] View at: https://pypi.org/project/Levenshtein-search/1.4.5/ $ exit 0 No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself. Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received The build has been terminated


Make sure deduplication has no false-positives.

Bug: T207329
cc @wikimedia/services

Coverage increased (+0.1%) to 88.99% when pulling d6cc6329b7a9d0659d8ca6e9aa60aae332dbeca5 on Pchelolo:dedupe_delay into eb1d7c9902a22917b98190fc1f9e80667b59fdf5 on wikimedia:master.

Coverage increased (+0.1%) to 88.99% when pulling d6cc6329b7a9d0659d8ca6e9aa60aae332dbeca5 on Pchelolo:dedupe_delay into eb1d7c9902a22917b98190fc1f9e80667b59fdf5 on wikimedia:master.


Readme instructions for building on macOS still don't work

This is the error I get:
While building package haskell-lsp-<IP_ADDRESS> using:
      /Users/ismailmustafa/.stack/setup-exe-cache/x86_64-osx/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.0.2 --builddir=.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS> build lib:haskell-lsp exe:lsp-hello --ghc-options " -ddump-hi -ddump-to-file"
    Process exited with code: ExitFailure 1
    Logs have been written to: /Users/ismailmustafa/Desktop/projects/haskell-ide-engine/.stack-work/logs/haskell-lsp-<IP_ADDRESS>.log

    Configuring haskell-lsp-<IP_ADDRESS>...
    Preprocessing library haskell-lsp-<IP_ADDRESS>...
    [ 1 of 10] Compiling Language.Haskell.LSP.TH.Constants ( src/Language/Haskell/LSP/TH/Constants.hs, .stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS>/build/Language/Haskell/LSP/TH/Constants.o )
    [ 2 of 10] Compiling Language.Haskell.LSP.TH.ClientCapabilities ( src/Language/Haskell/LSP/TH/ClientCapabilities.hs, .stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS>/build/Language/Haskell/LSP/TH/ClientCapabilities.o )
    <command line>: can't load .so/.DLL for: /Users/ismailmustafa/.stack/snapshots/x86_64-osx/nightly-2017-06-16/8.0.2/lib/x86_64-osx-ghc-8.0.2/libHStext-icu-<IP_ADDRESS>-EllpGYmNw3bJ3FIJMPisuJ-ghc8.0.2.dylib (dlopen(/Users/ismailmustafa/.stack/snapshots/x86_64-osx/nightly-2017-06-16/8.0.2/lib/x86_64-osx-ghc-8.0.2/libHStext-icu-<IP_ADDRESS>-EllpGYmNw3bJ3FIJMPisuJ-ghc8.0.2.dylib, 5): Library not loaded: /usr/local/opt/icu4c/lib/libicuuc.58.dylib
      Referenced from: /Users/ismailmustafa/.stack/snapshots/x86_64-osx/nightly-2017-06-16/8.0.2/lib/x86_64-osx-ghc-8.0.2/libHStext-icu-<IP_ADDRESS>-EllpGYmNw3bJ3FIJMPisuJ-ghc8.0.2.dylib
      Reason: image not found)

I think the text-icu tracker would be a better place to report this issue, and you would be able to get more help there.

@wz1000 I was about to post an issue on the text-icu tracker however I thought I would try building it first. I was successfully able to build it so I'm not convinced it's an issue with text-icu.

@ismailmustafa Small but maybe not good solution, some problem with icu4c
cd /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core
git checkout e1b518a22 Formula/icu4c.rb
brew reinstall icu4c

@xgrommx Solution is not ideal but it works!!! Thank you.

brew switch icu4c 58.2
brew pin icu4c

is as unsatisfactory, but doesn't involve picking homebrew's guts.

I removed text-icu dependency from yi-rope, so this should work now.


could you please give more  example image in the readme or your website about the document of this project

so that i could get more comfortable develop experence when use your library,and i think this project is  awesome .

Thank you
I will do this but for now i am a little busy, you can see demo app, or my personel projects such as Al-Anvar, TvTime and ArtWorks.
I use most of features in my projects

is it based on wpf?or win32?

i updated docs (internally) with more images and i also updated Demo app and i will upload it in MS Store soon.
i am renaming WinUICommunity to WindowUI or Something else, so i am working on it and changes will be made public soon.

done


Update simple service and client tutorial with synchronous example

The current tutorial for writing a simple ROS2 service and client only has an example of the client using the call_async() API in rclpy. This change adds an example using the synchronous call() API.

@maryaB-osr, can you please take a look?

I'll try to give a fuller review when I have more time tomorrow, but for now I can say I'm a little conflicted. We deliberately left out any mention of synchronous clients for a couple reasons, but obviously this is important content that needs to be documented somewhere so it's good that you've done this.
First, this is a beginner tutorial. There is no reason to introduce someone who's just starting to learn about ROS to both of these options. A tutorial is supposed to show one easy way to do something, not every way and not even necessarily the best way. I think async is generally more efficient or the more common solution so that's why we went with it.
If we do decide to keep both in this tutorial (sync and async), I've noticed a few things that need to be changed. Most important is that there's no narrative around using both of them. There needs to be some talking the user through the fact that they'll be doing the same thing twice, and some indication of where where the fork is.
Also, if we're going to make the tutorial about sync and async, it probably needs to show users the difference between the two when running them. There needs to be some indication of cause and effect for each type. And if that requires reworking the content or the outcome to be a little more complex, then this becomes an intermediate tutorial about "sync and async", not a beginner one about "services and clients". If you have any ideas in that direction that'd be great.

First, this is a beginner tutorial. There is no reason to introduce someone who's just starting to learn about ROS to both of these options. A tutorial is supposed to show one easy way to do something, not every way and not even necessarily the best way. I think async is generally more efficient or the more common solution so that's why we went with it.

I agree with this; I think that we should keep this tutorial simple and just show one way to do it.  We can also have a second, longer, advanced tutorial that discusses both options and their pros and cons.  And then we can link to that second article from this one, in a "More information" section or something.

Okay, those are reasonable concerns.
Should we do a synchronous vs asynchronous client tutorial that goes under the "Advanced" section?

@clalancette @maryaB-osr The goal of this PR is to warn new users that calling services synchronously from some part of the code (callbacks) will result in a deadlock. While a technical solution would be better (e.g. throwing an exception or failing the call when the sync service API call is invoked), we thought that updating the documentation would be a good first step.
FYI - we saw real cases of experienced software developers being confused by this problem in our organization, so this is a real problem for new ROS 2 users.
What do you think?
If this is not the right place to notify users, please kindly explain where, and whether the documentation should be updated, and we'll rework this PR accordingly.

I think @mm318's idea for a new tutorial is the best course of action (I'd put it under a new "Intermediate" section, "Advanced" seems like too much). That tutorial can be linked to from the original, so I think you can keep the same PR open since there'll still be an edit in the client service tutorial.
@thomas-moulard the purpose of the edit is definitely warranted. However, to benefit readers and maintainers (who are concerned with the overall structure and consistency of the docs), it's important to keep the purpose of each piece of documentation as clear, succinct, and granular as possible. Mixing up the goal you described with the client service tutorial, which has a very different goal, does both docs a disservice.
I'm happy to work on this as much as @mm318 is interested; I can take over or just provide a tone/structure edit at the end. Let me know!

Hi @maryaB-osr, it would be great if you can take over! Please let me know if you need help creating/testing example code, if the ones provided in this pull request are not already sufficient. Thanks!

@mm318 would you be able to create the same sync example and explanation in C++? So the C++ equivalent of lines 173-255. Let me know if you're not able, or if you are but you won't be able to get to it this week.

@maryaB-osr I can get to it by end of next week. Also, there is actually no equivalent synchronous API in rclcpp for C++ yet, so the sample will need to look something like this.

@mm318 I didn't realize that. In that case no worries, we'll keep the new doc focused on the existing python API.

Thanks for closing this out @maryaB-osr! :tada:


Fixes #3143 No completions on multi line strings

Fixes #3143 No completions on multi line strings
Updates the classifier to produce a single coherent token for multiline strings.

Note that #3171 means that you don't get signature info after a multiline string yet. I didn't try to fix it right away.


TypeError: cannot concatenate object of type '<class 'str'>'; only Series and DataFrame objs are valid

calling
$ boldigger-cline ie_coi <username> <password> ./test03_20220407/results-2022-04-07-16\:26/barcode03_nonchimeras.fasta output

results in
Login successfull
00:11:05: Requesting BOLD. This will take a while.
Downloading results: 100%|██████████████████████| 10/10 [00:50<00:00,  5.04s/it]
00:14:31: Parsing html.
Requesting BOLD:   0%|                                   | 0/19 [03:25<?, ?it/s]
Traceback (most recent call last):
  File "/PATH/bin/boldigger-cline", line 8, in <module>
    sys.exit(main())
  File "/PATH/lib/python3.8/site-packages/boldigger_cline/__main__.py", line 42, in main
    boldblast_coi.main(args.username, args.password, args.fasta_path, args.output_folder, args.batch_size)
  File "/PATH/lib/python3.8/site-packages/boldigger_cline/boldblast_coi.py", line 67, in main
    dataframes = save_as_df(html_list, sequences_names[querys.index(query)])
  File "/PATH/lib/python3.8/site-packages/boldigger/boldblast_coi.py", line 106, in save_as_df
    result = pd.concat(tables, axis = 0)
  File "/PATH/lib/python3.8/site-packages/pandas/util/_decorators.py", line 311, in wrapper
    return func(*args, **kwargs)
  File "/PATH/lib/python3.8/site-packages/pandas/core/reshape/concat.py", line 347, in concat
    op = _Concatenator(
  File "/PATH/lib/python3.8/site-packages/pandas/core/reshape/concat.py", line 437, in __init__
    raise TypeError(msg)
TypeError: cannot concatenate object of type '<class 'str'>'; only Series and DataFrame objs are valid

It seems that the script wants to concatenate objects of type 'str'. Unfortunately I could not find out what exactly caused this issue. Can someone help?

Will work on this asap. Did not maintain boldigger-cline for a while.


Make choco package provider's installed_packages public

Description

This converts the chocolatey package provider's installed_packages method into a class method and makes it available publicly.  There are various reasons one might want access to a list of packages installed and managed by choco, such as when the ohai plugin is not available.
As part of the change several supporting methods that interact with the choco executible have also been converted to class methods.
The spec was also updated to accomodate the changes.  All of the cases pass on my test machine except the one marked as pending.
Unfortunately choco_command lost its returns: annotation because there didn't seem to be a good way to preserve it.  I feel bad about that, particularly as it comes to the unit tests.  I welcome comments that might suggest a better approach.
At this point I'm posting for comments, but I will also update the docs if this overall approach is judged to be sound.  If there are additional unit tests desired, I am also open to writing them.
If I've done everything right, nothing should break or change, the package provider just gains a new public class method.
Signed-off-by: Eric Logan<EMAIL_ADDRESS>Related Issue

chef/chef#11687
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

This feels like we're digging ourselves an API hole without any clear design principles behind it.  I really deeply dislike the idea that people would be calling Chef::Provider::Package::Chocolatey.installed_packages because of the tight coupling to the implementation reflected by the use of that class symbol.  While that hasn't changed in a long time, you should consider the fact that I would really like to at some point convert all the package resources to custom resources.  They are all 1:1 resources right now and would be prime candidates to convert other than the issue with the inheritance from the base package provider.  But what needs to happen there is just splitting out the base class into a mixin / resource partial (and ideally having separate partials for multipackage and non-multipackage and de-grossifying all that code).  If that got done then the class disappears.
To retain the flexibility to do those kinds of refactorings you should not be using class names like that directly (and it think there's some ancillary class names that you're using under the Yum/Dnf providers that should really go away as well).
So, the way to do this is to introduce DSL methods, but those really need to get designed a bit better than solving a single problem and propose a DSL API for managing package information.  And this would be very, very good thing to do.  I've wanted it for years.  Things like installed_package?("lsof") or candidate_version?("lsof") directly in the DSL utilizing the package provider which is wired up through the ResourceResolver/ProviderResolver mechanism by default (probably installed_package?(:chocolatey_package, "WinSock") if you need the non-default package manager).
And we have a DSL mechanism to get at the provider already but its just moderately messy:
provider = build_resource(:chocolatey_package, "whatever").provider_for_action([:anything])
provider.load_current_resource
provider.installed_packages

The mess of getting at the provider after load_current_resource'ing it could be setup as a method on the Resource class:
class Chef
  class Resource
    def current_provider(actions = nil)
      # XXX: provider_for_action really needs to take nil or no arguments and DTRT, we've never utilized individual providers for actions,
      # and probably never will
      provider_for_action(actions).tap do |provider|
        provider.load_current_resource
      end
    end
  end
end

Then something like this as a DSL method:
def installed_packaged(package_resource = :package)
  build_resource(package_resource, "whatever").current_provider.installed_packages
end

Then helpers like candidate_version? and installed_package? could also be written (and put into chef-utils).

I wanted to start by acknowledging that this is a very generous review.  Thank you!
I like your DSL proposal a lot...with the small hitch that I'm not sure I'll have time to get to work on it myself, as much as I would like to.  I'm new to hacking on chef internals and I'm not sure how long this would take me.  I am in conversations with colleagues about whether someone can take this on at all.
Based on the overall review I agree that the overall approach is not great and so I'll close this PR.


Enable layout to be configured dynamically within shaders

Currently layout is set up in the shader, it would be possible to explore having the algorithm configure the layout of the shader itself dynamically, so it can be set and configured on the host side even if this is something that was initially set up in the shader.

Done in 0.5.0


Add SRI attributes to templates

For: https://trello.com/c/SVfWxYYs/205-enable-subresource-integrity-sri-on-static-s
We don't actually add the SRI attributes, but instead add hooks to let the template processors themselves use their own api's to add the SRI attributes.  Currently we only expect the rails version of these to work as we our shims for the other processors ignores the integrity options, but they can be filled in once we know how generate integrity values with them.
Resurrects the work from #301 which was removed in #308 when we realised there was a bug in firefox.

@h-lame how can this be tested?

Does this vary much from https://github.com/alphagov/govuk_template/pull/301?

@fofr - the implementation is mildly different (pulling in less of the rails helper methods and inlining some code), but part from that it's the same.


React to changes in Microsoft.NETCore.App packaging structure

Part of https://github.com/aspnet/AspNetCore/issues/9720
cc @dagood

@natemcmaster please let us know once you've resolved conflicts and the build cleans up

I'm confused about how my proposed changes resulted in this error:

src\Caching\Abstractions\src\Microsoft.Extensions.Caching.Abstractions.csproj(0,0): error NU1101: Unable to find package Microsoft.DotNet.GenAPI. No packages exist with this id in source(s): https://dotnet.myget.org/F/roslyn/api/v3/index.json, https://dotnet.myget.org/F/sourcelink/api/v3/index.json, https://www.myget.org/F/xunit/api/v3/index.json, nuget.org

I didn't touch the list of restore sources or the package ref to Microsoft.DotNet.GenAPI.

Updating the SDK might have changed the implicit restore feeds. Adding dotnetfeed back to restore sources solved the issue.


How to setup?

I have ZHA installed in Hassio using xbee module.  Trying to pair with aqara motion sensor...it appears i need a device handler for these sensors..but how do i add these handlers to hassio.   I dont see any instructins on this?

They are installed with ZHA automatically. There is nothing you need to do. If there is an issue related to a particular device open a new issue and fill out the entire template please.


✨ Mobile | Disable autocorrect for the quiz answer input box

Pain
Many Quiz questions have technical terms which are picked up by the autocorrect on iOS/Android.
Suggested Solution
Disable autocorrect for the input box where people type their answers.
Tasks

[ ] Investigate if it's possible
[ ] Implement

Acceptance Criteria

[ ] Meets Definition of Done - https://github.com/SSWConsulting/SSW.Rewards.Mobile/blob/main/_docs/Definition-of-Done.md

Thanks!

Fixed in #934