KDE notification sounds

I'm trying to display a notification in KDE Plasma 5.22.5 (X11) and cannot seem to trigger notification sounds because some basic searching revealed that KDE may not implement the XDG sound naming specification / sound theme specification. I looked at the source code, but it looks like the library is just sending the sound name as a message hint over DBus.
Are there any more examples or documentation available for triggering sounds in KDE? Specifically, is it possible to trigger a sound as defined in System Settings > Notifications > Plasma Workspace > Configure Events? It appears that specifying an absolute path (SoundFile hint) does not work, so I can't currently do that either. It's unclear to me if this is a KDE issue, configuration issue, etc.
Thanks!

Hiho, thanks for looking into this, I have to be honest I kinda never "saw" sound working on KDE either. I always assumed it's just not implemented (being a fairly annoying feature if you think about it).
So notify-rust sends the sound hint as described in the spec and that's it.
But I haven't spent a whole lot of time trying to make sound work either, so maybe there is a way to make it work that I just overlooked.

Wow! Thank you very much for the extensive documentation. I'd love to support KDE better, as I'm a KDE user myself.
I suppose as long as we're just adding another DBus endpoint that needs to be targeted this should be fine. We could technically even send it redundantly if we're not on KDE. I'd becurious if gnome does something similar.
So from a "is there room" it should be fine, since we're not gonna bloat the library much. But I'd prefer if we could keep the API nice and simple. For images we actually query the version of the service to make a decision at the start. Maybe we can do something similar for KDE and then hide all the gritty details from the public API altogether.

Are there any news related to this?

As of knotifications v6.4.0, it appears that KNotifications does not support notification-specific sounds.  Here is the result of GetCapabilities, with "sound" notably missing:
{
  "body",
  "body-hyperlinks",
  "body-markup",
  "body-images",
  "icon-static",
  "actions",
  "persistence",
  "inline-reply",
  "x-kde-urls",
  "x-kde-origin-name",
  "x-kde-display-appname",
  "inhibitions"
}


Update TileDB-VCF to 0.29.2

Recipe looks good. Unclear why all the jobs are failing due to an error while building libtiledbvcf

I'm seeing spdlog attempting to use libfmt for a C++20 build, when it should be using std::fmt.

I figured out the problem.
The last successful build was from ~2 weeks ago from commit 2ec4de49ca4ea81ce68e7d39130d7d037307239d (when I added pyarrow-hotfix to the requirements)
The linux-64 job installed fmt as an external project
-- Could NOT find spdlog (missing: spdlog_DIR)
-- Adding spdlog as an external project
-- Found TileDB: /home/conda/feedstock_root/build_artifacts/tiledb-vcf_1710174242588/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_/lib/libtiledb.so.2.20

Because it installed tiledb 2.20.1-h7c1ff67_3, which doesn't pin fmt or spdlog
wget https://anaconda.org/conda-forge/tiledb/2.20.1/download/linux-64/tiledb-2.20.1-h7c1ff67_3.conda
conda inspect hash-inputs tiledb-2.20.1-h7c1ff67_3.conda
## {'tiledb-2.20.1-h7c1ff67_3.conda': {'recipe': {'aws_crt_cpp': '0.26.2',
##                                                'aws_sdk_cpp': '1.11.267',
##                                                'bzip2': '1',
##                                                'channel_targets': 'conda-forge '
##                                                                   'main',
##                                                'curl': '8',
##                                                'cxx_compiler': 'gxx',
##                                                'cxx_compiler_version': '12',
##                                                'libabseil': '20240116',
##                                                'libcurl': '8',
##                                                'libgoogle_cloud_devel': '2.22',
##                                                'libgoogle_cloud_storage_devel': '2.22',
##                                                'libxml2': '2',
##                                                'lz4_c': '1.9.3',
##                                                'openssl': '3',
##                                                'target_platform': 'linux-64',
##                                                'zlib': '1.2',
##                                                'zstd': '1.5'}}}

In contrast, the latest linux-64 build from this PR installed tiledb 2.20.1-h99d63bd_4, which pins fmt 10 and spdlog 1.12
wget https://anaconda.org/conda-forge/tiledb/2.20.1/download/linux-64/tiledb-2.20.1-h99d63bd_4.conda
conda inspect hash-inputs tiledb-2.20.1-h99d63bd_4.conda
## {'tiledb-2.20.1-h99d63bd_4.conda': {'recipe': {'aws_crt_cpp': '0.26.3',
##                                                'aws_sdk_cpp': '1.11.267',
##                                                'bzip2': '1',
##                                                'channel_targets': 'conda-forge '
##                                                                   'main',
##                                                'curl': '8',
##                                                'cxx_compiler': 'gxx',
##                                                'cxx_compiler_version': '12',
##                                                'fmt': '10',
##                                                'libabseil': '20240116',
##                                                'libcurl': '8',
##                                                'libgoogle_cloud_devel': '2.22',
##                                                'libgoogle_cloud_storage_devel': '2.22',
##                                                'libxml2': '2',
##                                                'lz4_c': '1.9.3',
##                                                'openssl': '3',
##                                                'spdlog': '1.12',
##                                                'target_platform': 'linux-64',
##                                                'zlib': '1.2',
##                                                'zstd': '1.5'}}}

Which resulted in fmt 10.2.1-h00ab1b0_0 and spdlog 1.12.0-hd2e6256_2 getting installed from conda-forge and causing the failure.
So the change was the PR https://github.com/conda-forge/tiledb-feedstock/pull/254, also from ~2 weeks ago, to properly depend on fmt and spdlog from conda-forge
Note that this same change also recently caused problems for tiledbsoma-feedstock (https://github.com/TileDB-Inc/tiledbsoma-feedstock/issues/101#issuecomment-2004788942, https://github.com/TileDB-Inc/tiledbsoma-feedstock/pull/106)
There are various ways around this. We could pin the recipe to the version of tiledb 2.20 prior to its installing fmt/spdlog from conda-forge (this would be tedious to maintain and wouldn't work long-term, but it would allow us to get 0.29.1 shipped). Another option would be to do something similar to https://github.com/TileDB-Inc/tiledbsoma-feedstock/pull/106 and depend on the dependency-free "for-cloud" tiledb binary.
However, if it's possible to update TileDB-VCF to avoid these fmt/spdlog issues entirely (ie https://github.com/TileDB-Inc/TileDB-VCF/pull/682), that would be ideal.

Another option would be to do something similar to https://github.com/TileDB-Inc/tiledbsoma-feedstock/pull/106 and depend on the dependency-free "for-cloud" tiledb binary.

+1, IMO let's keep it simple to ship this ASAP without a dependency chain/overhead.

Summary from discussion with @ihnorton:

Plan A is to migrate TileDB-VCF to C++20 (https://github.com/TileDB-Inc/TileDB-VCF/pull/682) and release 0.29.2, which should hopefully fix the build error related to the version of fmt
Plan B is to build against the "for-cloud" tiledb binary, which doesn't depend on conda-forge::fmt

Hopefully one of the two above plans will work. There is the possibility that libtiledbvcf will have trouble at runtime if it is built against fmt 10 and spdlog 1.12 in this feedstock but then deployed to TileDB Cloud which has fmt 9 and spdlog 1.11 installed. If that happens, then:

Plan C is to build separate variants aimed at conda-forge versus TileDB Cloud like I did for SOMA in https://github.com/TileDB-Inc/tiledbsoma-feedstock/pull/106

From the errors, it looks like it is still having trouble properly linking against fmt:
[100%] Linking CXX executable tiledbvcf
/home/conda/feedstock_root/build_artifacts/tiledb-vcf_1711152722224/_build_env/bin/../lib/gcc/x86_64-conda-linux-gnu/12.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: CMakeFiles/tiledbvcf-bin.dir/cli/tiledbvcf.cc.o: in function `spdlog::details::fmt_helper::pad2(int, fmt::v10::basic_memory_buffer<char, 250ul, std::allocator<char> >&)':
tiledbvcf.cc:(.text._ZN6spdlog7details10fmt_helper4pad2EiRN3fmt3v1019basic_memory_bufferIcLm250ESaIcEEE[_ZN6spdlog7details10fmt_helper4pad2EiRN3fmt3v1019basic_memory_bufferIcLm250ESaIcEEE]+0xf7): undefined reference to `void fmt::v10::detail::vformat_to<char>(fmt::v10::detail::buffer<char>&, fmt::v10::basic_string_view<char>, fmt::v10::detail::vformat_args<char>::type, fmt::v10::detail::locale_ref)'
/home/conda/feedstock_root/build_artifacts/tiledb-vcf_1711152722224/_build_env/bin/../lib/gcc/x86_64-conda-linux-gnu/12.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: CMakeFiles/tiledbvcf-bin.dir/cli/tiledbvcf.cc.o: in function `spdlog::logger::sink_it_(spdlog::details::log_msg const&)':
tiledbvcf.cc:(.text._ZN6spdlog6logger8sink_it_ERKNS_7details7log_msgE[_ZN6spdlog6logger8sink_it_ERKNS_7details7log_msgE]+0x180): undefined reference to `fmt::v10::vformat[abi:cxx11](fmt::v10::basic_string_view<char>, fmt::v10::basic_format_args<fmt::v10::basic_format_context<fmt::v10::appender, char> >)'

From the errors, it looks like it is still having trouble properly linking against fmt:

One possibility is that a template is being substituted with a std::string_view argument, but not instantiated, resulting in a symbol being expected but not generated. I'm trying a forced update of spdlog; if that doesn't have the desired effect, I can look for reasons why the templates may be used in a manner the authors hadn't anticipated.

The latest errors show improvement after (https://github.com/TileDB-Inc/TileDB-VCF/pull/684) and (https://github.com/TileDB-Inc/TileDB-VCF/pull/686).
One linker error remains for Linux:
/home/conda/feedstock_root/build_artifacts/tiledb-vcf_1711384793823/_build_env/bin/../lib/gcc/x86_64-conda-linux-gnu/12.3.0/../../../../x86_64-conda-linux-gnu/bin/ld: tiledbvcf: hidden symbol `_ZN6spdlog6logger12err_handler_ERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE' in CMakeFiles/tiledbvcf-bin.dir/cli/tiledbvcf.cc.o is referenced by DSO

@ihnorton, @jdblischak any ideas on how to resolve this error? Should we fall back to Plan B?

any ideas on how to resolve this error? Should we fall back to Plan B?

Something weird is going on. spdlog and fmt are installed in the conda env, and ideally libtiledbvcf would link against these (since these are also what libtiledb linked against). That is what I thought 0.29.1 was doing. However, now with 0.29.2, it is back to installing spdlog as an external project.
-- Could NOT find spdlog (missing: spdlog_DIR)
-- Adding spdlog as an external project
-- Found TileDB: /home/conda/feedstock_root/build_artifacts/tiledb-vcf_1711384793823/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_/lib/libtiledb.so.2.20

Can either the source file FindSpdlog.cmake or the build script be updated so that libtiledbvcf builds against the conda-installed spdlog and fmt?

There's a new cmake option to control how spdlog is handled, similar to htslib.
This change to the build script should use the conda installed spdlog, if that's what we need:
cmake \
  -DCMAKE_INSTALL_PREFIX:PATH="${PREFIX}" \
  -DOVERRIDE_INSTALL_PREFIX=OFF \
  -DCMAKE_BUILD_TYPE=Release \
  -DFORCE_EXTERNAL_HTSLIB=OFF \
  -DFORCE_EXTERNAL_SPDLOG=OFF \
  ../libtiledbvcf

We have a different problem now. CMake can't find spdlog in the conda environment, which is causing the build to fail on every platform.
https://dev.azure.com/TileDB-Inc/CI/_build/results?buildId=38681&view=results
CMake Error at cmake/Modules/FindSpdlog.cmake:93 (message):
  Unable to find spdlog
Call Stack (most recent call first):
  src/CMakeLists.txt:9 (find_package)

-- Could NOT find spdlog (missing: spdlog_DIR)
-- Configuring incomplete, errors occurred!
make[2]: *** [CMakeFiles/libtiledbvcf.dir/build.make:93: libtiledbvcf-prefix/src/libtiledbvcf-stamp/libtiledbvcf-configure] Error 1
make[1]: *** [CMakeFiles/Makefile2:141: CMakeFiles/libtiledbvcf.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

0.29.1 was able to find spdlog in the conda environment, so I assume some recent change to FindSpdlog.cmake caused the change in behavior
https://github.com/TileDB-Inc/TileDB-VCF/compare/0.29.1...release-0.29

We need to figure out how to remove the arg -Wl,--allow-shlib-undefined from the final linker command for the tiledbvcf executable.

Now I'm running into a wheel/packaging problem:
2024-03-28T02:28:58.6026304Z LookupError: setuptools-scm was unable to detect version for '/home/conda/feedstock_root/build_artifacts/tiledb-vcf_1711592366035/work'.
2024-03-28T02:28:58.6026660Z
2024-03-28T02:28:58.6027180Z Make sure you're either building from a fully intact git repository or PyPI tarballs. Most other sources (such as GitHub's tarballs, a git checkout without the .git folder) don't contain the necessary metadata and will not work.

How did this ever work, given that we always use tarballs? I'll try creating an archive with a numeric filename to see whether that fixes it.

@ihnorton can you bypass the Windows CI after the remaining checks pass?


Renders [?] rather than advanced characters when using non-standard fonts

When rendering a PDF and including fonts with limited character sets (i.e. only part of the entire Unicode character set), the rendered PDF shows [?] instead. This is because of the unicode_char for different types of spaces in the converter.rb file. Not all fonts support:
NarrowSpace = unicode_char 0x2009
NarrowNoBreakSpace = unicode_char 0x202f
ZeroWidthSpace = unicode_char 0x200b
HairSpace = unicode_char 0x200a
If there could either be a check or a replacement with a standard font (Arial, etc.) for these invisible characters, it would help when using fonts from foundries like MonoType or others. As written, when using a font like Avenir, the button declarations and the table of contents become unreadable.

This is what the fallback font is supposed to handle. I do agree, however, that in certain circumstances we should check the font first before using certain characters. I think the solution here is both documentation and perhaps a change to the code. I'll explore the possible solutions.

It's now far easier to define a theme with custom fonts (including a fallback font) and we've thoroughly documented all the non-typical glyphs that Asciidoctor PDF relies on. See https://github.com/asciidoctor/asciidoctor-pdf/blob/master/docs/theming-guide.adoc#preparing-a-custom-font. Therefore, I'm considering this issue resolved.


Python 3 compatibility

A lot of Python code inside CloudStack is still Python 2 and we have just 10 months left before Python 2 is no longer supported: https://pythonclock.org/
Various items need to be looked at and checked for Python 3 compatibility. A list, but probably not complete:

[x] KVM Security Groups
[x] Xen Security Groups
[ ] Virtual Router scripts (addressed in #3730)
[x] Various setup tools
[x] build tools (addressed in #4030)
[x] marvin framework and smoke tests (#4727)
[x] marvin component test sets (#5082)
[x] packaging (cloud.spec) (#4513 is asking for it)

These are isolated projects and can usually be taken care of separately and also properly tested.
Creating this ticket so it can be referred to and also this is out there.

Is there an argument to say, these may be implemented purely in Java (if possible)? Currently, many of these scripts esp on KVM are forked via the cloudstack-agent and called.
For VR, I played with GRPC and I think it can be secured against the current CA-framework introduced in 4.11; here's a PoC: https://github.com/shapeblue/cloudstack/pull/44
I'll propose more details on dev@ once I can put together design/idea details.

@rhytd @wido I would love to help with moving over to Python 3, if that's still the plan.

Although I agree with @rhtyd that a lot can be done in Java, it's not always the easiest. Python is sometimes a lot easier to develop and talk directly with the system for executing commands.
We haven't started on this @falcon78921 , but any help is welcome! Just pick a Python script and start working on it.

@wido changing the env dependency is a big step, python3.x is not available on all distros by default. I think it's better to discuss on dev@ for future steps. In my proposal, for the VR for example I've proposed that we move to a lightweight agent and as PoC I've shared a Go-Java approach based on grpc.
I think the best time would be to discuss/start doing things after 4.13, as we've decided to deprecate CentOS6 after 4.13.

@rhtyd Python 3 is available on ALL distros. It has been for a very long time. It's on RHEL, CentOS, Debian and Ubuntu. No need to install anything :)
We can always talk about this, but the fact is that per January Python 2 is EOL and it's not so super difficult to write things in Python 3.
I'm not against an Agent in the VR for example, sounds like a good thing!

There are also knock-on effects, ie I suspect that the chances of running Python3 on XenServer 6.x is near zero.

@PaulAngus Python 2 is EOL in 2020, so any plans on what Xen is going to do?
But this doesn't stop us to fix other Python code before that time.

I think we'll need to deprecate all the XS 6.x, which we can discuss and aim to do in 4.14 and onwards. XS7 or xcp-ng 7.x with CentOS7 based dom0 may be more easy to support python3.

guys, any update on this one? I don't see it coming to 4.14 due to a needed effort?

Given the time it won't be possible to completely do it and test within 4.14, let's do it against 4.15.

@GabrielBrascher Has done work on this and he can comment on how far he got.

@DaanHoogland can you review that's remaining and being targetted for 4.16; I suppose except for component tests and VR scripts all are 100% complete? Even component tests are all migrated to py3? cc @nvazquez @sureshanaparti

yes, component tests are migrated and #5082 is open for fixing (some of) them. The VR is not done and there are some obscure tests in test/integration/testpaths and some in test/integration/plugins
nothing else that I know of.

Thanks @DaanHoogland - PR 5082 is merged but there are still some component tests to migrate?
@GabrielBrascher are you still working on this PR for VR code? https://github.com/apache/cloudstack/pull/4807

@nvazquez I need to roll a couple more tests.
Let's have it in the next release, for 4.16 it might be risky to have it.
Thanks!

Thanks @DaanHoogland - PR 5082 is merged but there are still some component tests to migrate?
@GabrielBrascher are you still working on this PR for VR code? #4807

not to migrate but to fix. component tests have been neglected way too long.

as there are no plans for doing this in 4.18 I'm moving this to 'unplanned'.

I think at some point our systemvmtemplate based OS (Debian 11) will EOL, so our runway may be limited by that. For this reason we may need to migrate the VR scripts to py3 or rework the VR agent approach.
Thanks @wido all, I think most of the items are done except migration of the VR scripts for which I've created a new ticket - https://github.com/apache/cloudstack/issues/7500


[2.1.3] Use structural equality for concurrency check in in-memory database

Fixes #12214
Fix is to use structural equality all the time for concurrency checks. This matches more closely the way the check works on real database systems.

Superseded by #12483


SDSTOR-11576 test_vdev is using same file name as test_data_service which cause it to fail

when test_vdev runs slower, the test_data_service runs into recovery mode by mistake, because same file name is used by test_vdev which is not intended.

Codecov Report

Merging #140 (8597185) into master (73ab6ac) will decrease coverage by 6.41%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master     #140      +/-   ##
==========================================
- Coverage   44.21%   37.80%   -6.41%
==========================================
  Files          83       74       -9
  Lines        9024     7538    -1486
  Branches     1156      996     -160
==========================================
- Hits         3990     2850    -1140
+ Misses       4656     4375     -281
+ Partials      378      313      -65

see 15 files with indirect coverage changes


New term request

Hello,
Could you look at the term motif 2 in Figure 2E in PMID: 38763332 please?
https://pubmed.ncbi.nlm.nih.gov/38763332/
The PDF is attached.
Does this structure exist in ChEBI already? If so, could you tell me its ChEBI ID?
If not, would you be able to create it?
This is needed for an IEDB curation.
Thanks,
Catriona
PMID_38763332.pdf
@ rvita

Hi,
A new entry (CHEBI:233000) has now been created for the above motif.


Upgrade front-end libs

Upgrade any possible libs

For now, can't do it... One of the project libs are incompatible with the newer version of Vue, might try to update again before releasing the first alpha of webBudget v4


Prevent files from being required programatically

I am using browserify like this:
  var b = browserify();
  b.require('/path/to/app/root.js', {basedir: __dirname, expose: '__app'});

...etc.
I'm trying to make sure browserify doesn't bundle in certain .js files which are supposed to live on the server side, and should never make their way onto the client. To do so, I've tried to pass them in as opts to browserify(opts) with the following keys: exclude, ignore, noParse.
For example:
var b = browserify({
  noParse: ['path/of/file/to/never/include/in/bundle.js']
});

All to no avail.
Is there any way to do this?
Extra context which might or might not be useful:
These .js files which I'd like to ignore should never accidentally make their way into the client bundle, even if unwittingly required. I don't care what happens instead, I just don't want the modules to silently sneak in. It's fine if a blank object, for example, was imported instead.

 As clearly depicted here, it's just:
var b = browserify();
b.exclude('path/of/file/to/never/include/in/bundle.js');


pin the base images versions

fixes: https://github.com/GoogleContainerTools/kpt/issues/1247

Do we have tracker for base image updating?

Yes, we are tracking it at b/179503280 internally. It will remind us at the beginning of next quarter.


Merge reorder into refactor-assets-hierarchy

@mikitfreek please double-check me if everything's fine. We can merge whenever you're ready and push the changes to main right after.

Reopening the PR. I got it wrong - the hotfix commits I mentioned in the last comment are already pushed to refactor-assets-hierarchy.


Added missing french translations

Added the missing french translations for batch actions, browse field and select2 field.

Great! Thank you @7ute - tagged and released, available with a composer update.
Cheers!


detect leiningen projects on the fly

should be picking up on new leiningen projects as they are initiated. Makes workflow less annoying.

closed as of e04b04991a09602e17b9f06accdb2f629a08187a


tiny change to make Guice happy

Currently use of the inject annotation on a named constructor works in CDI, but not in Guice.  Unfortunately the inner "tag" class we create for named constructors has a private constructor. I think it would be ok to make that constructor default, just especially for Guice.

Fine by me.

Done what you asked @gavinking, but you might want to test that Guice is now happy.

Thanks!!


Grammar update

'want archive' updated to: want to archive

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Er

Permiso para acceder a memoria SD storage o emulate de Huawei honor 9xde Manuel😎

@goncharova7714 @Manuel13dpl `

Automatically generated comment ℹ
This comment is automatically generated and will be overwritten every time changes are committed to this branch.
The table contains an overview of files in the content directory that have been changed in this pull request. It's provided to make it easy to review your changes on the staging site. Please note that changes to the data directory will not show up in this table.

Content directory changes
You may find it useful to copy this table into the pull request summary. There you can edit it to share links to important articles or changes and to give a high-level overview of how the changes in your pull request support the overall goals of the pull request.

Source
Staging
Production
What Changed

content/issues/organizing-your-work-with-project-boards/tracking-work-with-project-boards/archiving-cards-on-a-project-board.md
Modified
Original


Fixed BUILD_ONLY example in README.md. Modified CMakeLists.txt for better integration with IDE.

Assigned BUILD_ONLY, STATIC_LINKING, and CUSTOM_MEMORY_MANAGEMENT to their default values.
Some IDEs, qtcreator in particular allow to edit cmake build options in a nice GUI form. However if an option is not set to some value in CMakeLists.txt then the nice GUI form does not recognize the option.
BUILD_ONLY accepts project names without "aws-cpp-sdk-" prefix.
But In the README example BUILD_ONLY uses the prefix.

STATIC_LINKING only exists for backwards compatibility and should stay hidden; BUILD_SHARED_LIBS is the preferred option now.
Defaulting CUSTOM_MEMORY_MANAGEMENT to 0 is actually a breaking change given the way it's used in core (gives a different result if you build shared libraries and don't specify anything memory-wise on the command line).  This was a mistake on my part as I should have defaulted it to off unconditionally.  We may go ahead and make the change in the near future, but if we do it will be a transition to ON/OFF and not the awkward "0"/"1" that we used early on.

Removed  STATIC_LINKING and CUSTOM_MEMORY_MANAGEMENT default assignments.
Put BUILD_SHARED_LIBS instead of STATIC_LINKING in README.md.


⚠ Bofrost.at has degraded performance

In 5660ba5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1372 ms

Resolved: Bofrost.at performance has improved in cd0808f.


TC.3.3 The game will be closed

a05f8557073cfce42e22af36cb4982408a5ce1b3

This task has been finished on the day with posted SHA comment, in the interest of correctly closing the issue (and us having to first learn how to), it has been closed a while after the finishing of the task.

We have in the meantime figured out, that we can not correctly close it once it is done (otherwise the commit is kinda useless).


adapter: NebulaGraph

What are the key elements of this solution?
Add support of NebulaGraph
Why did you design your solution this way? Did you assess any alternatives? Are there tradeoffs?
n/a
Does this PR require a change to Harlequin's docs?

[ ] No.
[x] Yes, and I have opened a PR at tconbeer/harlequin-web. https://github.com/tconbeer/harlequin-web/pull/91
[ ] Yes; I haven't opened a PR, but the gist of the change is: ...

Did you add or update tests for this change?

[ ] Yes.
[x] No, I believe tests aren't necessary, adaptor tests are in https://github.com/wey-gu/harlequin-nebulagraph
[ ] No, I need help with testing this change.

Please complete the following checklist:

[x] I have added an entry to CHANGELOG.md, under the [Unreleased] section heading. That entry references the issue closed by this PR.
[x] I acknowledge Harlequin's MIT license. I do not own my contribution.

A graph database! Cool! Thanks so much!


[Do not merge] example of using LibriSpeech + Lhotse Shar

This PR shows how to use Lhotse Shar data format to train models in Icefall. I only added the necessary adaptation to the basic TDNN LSTM recipe as I don't have a free GPU at hand to run anything bigger. If it's useful, feel free to adapt this to more recent recipes (very little change is required as most of the changes come inside the datamodule).

We will try to apply this to other recipes and see what performance improvements we get.


Date pickers lose time zone when using the calendar after values are cleared

Provide a general summary of the issue here
When a date picker is supplied a ZonedDateTime as a value or defaultValue, if the user clears out the date and/or time value (by either backspacing over the field values or by clicking a "clear dates" button), then choses a new date from the popup calendar, the internal state value looses the time zone information and becomes a CalendarDateTime.
https://github.com/adobe/react-spectrum/assets/157808128/9f586135-17ca-4920-97e9-40f1aeec16f5
https://github.com/adobe/react-spectrum/assets/157808128/e887b9fc-0ca7-476f-a9b9-b0ea4f416835
🤔 Expected Behavior?
The internal state value type should persist regardless of editing (If it starts out as a ZonedDateTime, it should stay a ZonedDateTime)
😯 Current Behavior
See general summary. There is a work-around: If you provide a placeholderValue, the time zone will persist
https://github.com/adobe/react-spectrum/assets/157808128/e1d4adbb-0eba-4e61-a412-9482f81df230
💁 Possible Solution
Maybe a similar issue that was fixed in useTimeFieldState here:
https://github.com/adobe/react-spectrum/pull/4715/files
🔦 Context
No response
🖥 Steps to Reproduce
You can use this fork of the useDatePicker Tailwind CSS example:
https://codesandbox.io/p/sandbox/gifted-vaughan-9n2wfm?file=%2Fsrc%2FApp.js
The only thing that was changed was to swap the DatePicker's defaultValue for an absolute date

Backspace over the hours and minutes

Open the calendar dialog and choose a new date
time zone is lost, value becomes a CalendarDateTime

Backspace over the year, month and day

Open the calendar dialog and choose a new date
time zone is lost, value becomes a CalendarDateTime

Workaround:

Add a placeholderValue that is a ZonedDateTime
Repeat the steps above
time zone persists as expected

Version
"react-stately": "^3.30.1", "react-aria": "^3.32.1",
What browsers are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge
If other, please specify.
No response
What operating system are you using?
MacOS 14.3.1
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Thanks for reporting this, reproduced locally. This was partially fixed via https://github.com/adobe/react-spectrum/pull/4715, that PR might help point to where we would want to investigate further.

Hi @LFDanLu I will work on this.

Sure, thanks! Let us know if you need any help or guidance.

@LFDanLu Could you please check https://github.com/adobe/react-spectrum/pull/7069


Problem with gestures

No matter what I try, I can't make the PanGestureRecognizer work and I don't know whether I'm doing something wrong, or the engine is failing, or Flutter is having some weird fun.
When I touch the screen, onDown is called (twice, actually, for whatever reason). Then if I start moving my finger, nothing happens even though I expect the onStart / onUpdate to be called. After moving my finger some distance from the initial touch location, onCancel is called, then onStart / onUpdate / onEnd as it should.
Here's the sample app that produces this behavior:
import 'package:flame/components/component.dart';
import 'package:flame/components/events/gestures.dart';
import 'package:flame/flame.dart';
import 'package:flame/game.dart';
import 'package:flutter/gestures.dart';
import 'package:flutter/services.dart';
import 'package:flutter/widgets.dart';

void main() {
  SystemChrome.setPreferredOrientations(
          [DeviceOrientation.landscapeLeft, DeviceOrientation.landscapeRight])
      .then((_) {
    MainGame game = new MainGame();
    runApp(game.widget);
  });
}

class MainGame extends BaseGame {
  PanGestureRecognizer _createPanGestureRecognizer() => PanGestureRecognizer()
    ..onDown = (evt) {
      debugPrint("[PAN] onDown: $evt");
    }
    ..onStart = (evt) {
      debugPrint("[PAN] onStart: $evt");
    }
    ..onUpdate = (evt) {
      debugPrint("[PAN] onUpdate: $evt");
    }
    ..onEnd = (evt) {
      debugPrint("[PAN] onEnd: $evt");
    }
    ..onCancel = () {
      debugPrint("[PAN] onCancel");
    };

  PanGestureRecognizer _panGestureRecognizer;

  @override
  bool debugMode() => true;

  @override
  void onAttach() {
    super.onAttach();

    if (_panGestureRecognizer != null) {
      Flame.util.removeGestureRecognizer(_panGestureRecognizer);
    }
    _panGestureRecognizer = _createPanGestureRecognizer();
    Flame.util.addGestureRecognizer(_panGestureRecognizer);

    Flame.util.fullScreen();
  }

  @override
  void onDetach() {
    super.onDetach();

    if (_panGestureRecognizer != null) {
      Flame.util.removeGestureRecognizer(_panGestureRecognizer);
    }
  }
}

I am not very expert on Flutter gestures, but I think Pan has some delay indeed, I suggest you trying this recognizer: https://api.flutter.dev/flutter/gestures/ImmediateMultiDragGestureRecognizer-class.html
I have used it and it seems to work very well, and without delays.

@renancaraujo knows a lot about advanced flutter gesture recognizers

@erickzanardo The problem with that is that I need to detect the end of drag (i.e. when the users lift their fingers). To better understand the problem, here's the use case:
There are 2 objects on the screen and the user needs to connect them with a line. This means that the user must tap and hold on object 1, move the finger to object 2, and lift it up.
I assumed PanGestureRecognizer would solve the problem as, well, it's a panning move, but any recommendations are welcome.

We have added a new API for gesture detection on version 0.17.0, this should fix this, gonna close this, but if there is still problems, feel free to open a new issue


[JENKINS-56794] Support saveOutput parameter in the pipeline step

See JENKINS-56794
See https://groups.google.com/forum/#!topic/jenkinsci-users/sk1jroBxNbo

How can I set the name of the output file?

@BarelElbaz, I guess your question is not related to this PR itself, I'd say you can use GitHub issues or Jenkins issues to request new functionality, or contribute if possible in the code, or ask these kind of questions in https://www.jenkins.io/mailing-lists/.
https://github.com/jenkinsci/email-ext-plugin/blob/3f620604bcf8ca0aec4226236674daefd5e4262e/src/main/webapp/help/projectConfig/saveOutput.html#L3 provides some insights about the existing behaviour.
Cheers


Can not run in FSI?

I have this class for benchmarking:
type MapComparison () =
    let mutable fsmap : Map<int64,int64> = Map.empty<_,_>
    let mutable hashmap = HashMap<int64,int64>()

    [<Params (100, 1000, 10000, 100000)>]
    member val public DictSize = 0L with get, set

    [<Setup>]
    member self.SetupData() =
        for i in 0L .. self.DictSize do
           fsmap <- Map.add i i fsmap
           hashmap <- hashmap.Add(i,i )

    [<Benchmark>]
    member self.HashMapLookup () =
      for i in 0L .. self.DictSize do
        let b = hashmap.Get i
        ()

    [<Benchmark>]
    member self.FSharpMapLookup () =
      for i in 0L .. self.DictSize do
        let b = fsmap.[i]
        ()
BenchmarkDotNet.Running.BenchmarkRunner.Run<MapComparison>()

For each of the test sizes, I'll get this error:

// Validating benchmarks:
// **************************
// Benchmark: MapComparison_HashMapLookup__DictSize-100
// *** Generate ***
// Result = Failure
// Exception: The invoked member is not supported in a dynamic assembly.

Am I doing something I'm not supposed to?
This is on Windows 10, 64-bit. All the code that is being tested is built in Release, but the code above is in an F# script which I send to interactive.

Hello @rojepp !
I am afraid that this scenario is not supported. We will also rather not add such feature because we encourage our users to close all applications (including IDE) before running benchmarks, to reduce the chance for results corruption.
However we support classic F#, you can find sample project here. We also support F# on .NET Core, samples here

Ok! Thanks for answering!


Preview of a Scene by showing frames as thumbnails images.

For building up complex scenes, It might be useful to not render the whole video but to only have a preview at some frames of it.
To give an example, this Video:
https://user-images.githubusercontent.com/44469195/108001842-ec75af80-6fed-11eb-95c7-4e19b3c2c211.mp4
Could also be displayed as this image:

I can think of a lot of use cases where this could be helpful. Also with this method, we could have video-tests without building a video-test infrastructure.
This could be implemented as a new flag, e.g. -thumbnails

Sounds like a great idea to me

Okay, here is a first idea to do so:
First, rendering a scene to pngs like this:
with "manim ExampleScene -qm -v WARNING --disable_caching  -g" in the console:
class ExampleScene(Scene):
    def construct(self):
        circle = Circle()
        square = Square()
        square.flip(RIGHT)
        square.rotate(-3 * TAU / 8)
        circle.set_fill(PINK, opacity=0.5)

        self.play(ShowCreation(square))
        self.play(Transform(square, circle))

and then adding the preview scene like this:
class PreviewWindows(Group):
    def __init__(self):
        Group.__init__(self)
        height= 9/4
        width = 16/4
        frames = [Rectangle(height=height,width=width).set_color(BLUE) for i in range(0,6)]
        a=VGroup(*frames[0:3]).set_x(0).arrange(buff=0.5).shift(UP)
        b=VGroup(*frames[3:6]).set_x(0).arrange(buff=0.5).shift(2 * DOWN)
        self.add(a,b)
        imgs= Group()
        total_frames= 60
        previews= len(frames)
        for i, frame in enumerate(frames):
            position = int(total_frames/previews*i)
            img= ImageMobject(f"media/images/first_point/ExampleScene{position}.png").move_to(frame).scale(0.42)
            imgs.add(img)
        self.add(imgs)

class FameIdeaNo1(Scene):
    def construct(self):
        pw= PreviewWindows()
        self.add(pw)

gives this output:

Maybe we should wait with this feature until the click pr is merged:  #1013


Inconsistent braces in example on the website for metadata filtering

The last example for nested filters on the website at : https://haystack.deepset.ai/guides/metadata-filtering doesn't run due to inconsistent curly braces.
Specifically this bit:
    {"$and": {
        "Type": "News Paper",
        "Date": {"$lt": "2019-01-01"}
        }
    },
    "$and": {
        "Type": "Blog post",
        "Date": {"$gte": "2019-01-01"}
        }
    }]
}

Fixed it in #388

Thank you so much for picking this up!


BUG: Make NumExprFilters window_safe.

Coverage increased (+0.002%) to 86.736% when pulling 5fe79e8fcea324f0bf07a6593f0949186217b84e on numexpr-filter-bug into e3f328da1ff25ff70bfdb2ce40dffdd87da3f1f5 on master.


fix: advanced_dhcp_server : file Create /etc/dhcp

Hello,
during the installation of the advanced DHCP role, I encountered this error:
TASK [advanced_dhcp_server : file █ Create /etc/dhcp] ***********************************************************************************************************************************************************************************
Monday 30 January 2023  10:04:43 +0100 (0:00:01.636)       0:00:04.166 ********
fatal: [mngt-01]: FAILED! =>
  msg: |-
    The task includes an option with an undefined variable. The error was: 'dhcp_server_conf_dir' is undefined

    The error appears to be in '/root/bluebanquise/roles/advanced_core/advanced_dhcp_server/tasks/main.yml': line 49, column 3, but may
    be elsewhere in the file depending on the exact syntax problem.

    The offending line appears to be:

    - name: "file █ Create {{ advanced_dhcp_server_conf_dir }}"
      ^ here
    We could be wrong, but this one looks like it might be an issue with
    missing quotes. Always quote template expression brackets when they
    start a value. For instance:

        with_items:
          - {{ foo }}

    Should be written as:

        with_items:
          - "{{ foo }}"

Here is my suggested fix.
best regards,

Dear @hmeNLE ,
Thank you for this fix 😊
You can try latest role in collections.
Ok for me, merging.


Adding Scroller/ScrollViewer ViewportWidth/Height and ScrollableWidth/Height read-only properties.

Issue #264
These variables were already used internally by the controls, we are just making them public, to match the old WUX ScrollViewer properties.

If you use one of the keywords (like fixes #264), it will close the issue automatically when you complete the PR : )
https://help.github.com/articles/closing-issues-using-keywords/

Thanks @kaiguo, issue #264 is a tasklist with many remaining tasks to complete still.


Updating only selected fields of immutable

Coming from the JavaScript world, seeing something like this in a reducer is common:
return {
    ...state,
    foo: "bar"
};

This is basically creating a new object, but replacing (not setting) the foo field, making it "immutable" (doesn't change the original, just creates a new with the updated field). This is usually compiled down to use Object.assign when using Babel.
In Dart and redux.dart, when making a new state, I found no way to "copy" the old object and replace a value. The only way I found was to fully recreate the object, reassigning all the fields from the old. This quickly becomes a problem when scaling, because if you creating a new field, you must check and add every single reducer branch. By failing to do so, you'll have an empty property when running that branch/action.
Is there any solution for this problem?

The best solution (which I think could be considered right/correct) is to use an intermediary class for mutations. Like so:
class AppState {
  final int counter;
  final String name;

  AppState({this.counter = 0, this.name = "John"});
}

class AppStateBuilder {
  int counter;
  String name;

  AppStateBuilder({this.counter, this.name});

  AppStateBuilder.fromAppState(AppState state) {
    counter = state.counter;
    name = state.name;
  }

  AppState toAppState() {
    return new AppState(counter: counter, name: name);
  }
}

class IncrementCounterAction {}

class SetNameAction {
  final String name;

  SetNameAction(this.name);
}

AppState counterReducer(state, action) {
  var builder = new AppStateBuilder.fromAppState(state);

  if (action is IncrementCounterAction) {
    builder.counter++;
    return builder.toAppState();
  } else if (action is SetNameAction) {
    builder.name = action.name;
    return builder.toAppState();
  }

  return state;
}

Hey @Cretezy! There are a couple patterns I've used to solve this problem.
1. Future Dream
I really hope Dart supports data classes or case classes in the Future! This would give us the Object.assign functionality in a type-safe way!
2. copyWith method
You can write your own copyWith method. It will take only the params you provide it, and default back to the old values if you don't.
https://github.com/brianegan/flutter_architecture_samples/blob/master/example/redux/lib/models/app_state.dart#L23
3. built_value
You can generate all of the copyWith code using a package called built_value. https://pub.dartlang.org/packages/built_value
(built_value will generate a method called rebuild that does the same thing as copyWith).
How do these options work for ya?

The copyWith method is very clean! It's basically a simpler version of my solution, I like it very much, and avoids code generation. I hadn't thought about using it that way. Thank you very much!

Cool, I'll close this issue for now! Let us know if you run into anything else :)

Just as a final remark, this is what I've ended up with as a result, very clean!
class AppState {
  final int counter;
  final String name;

  AppState({this.counter = 0, this.name = "John"});

  AppState copyWith({counter, name}) {
    return new AppState(
        counter: counter ?? this.counter,
        name: name ?? this.name
    );
  }
}

class IncrementCounterAction {}

class SetNameAction {
  final String name;

  SetNameAction(this.name);
}

AppState counterReducer(state, action) {
  switch (action.runtimeType) {
    case IncrementCounterAction:
      return state.copyWith(counter: state.counter + 1);
    case SetNameAction:
      return state.copyWith(name: action.name);
    default:
      // No change
      return state;
  }
}

The only small problem is you can't set a value to null using this solution. You'd need to set it to an empty object ("", 0), to "reset".

I used Symbol.empty as placeholder for "not defined" value, allowing to set value to null also.
For example:
class AppState {
  final int counter;
  final String name;

  AppState({this.counter = 0, this.name = "John"});

  AppState copyWith({counter = Symbol.empty, name = Symbol.empty}) {
    return new AppState(
        counter: counter != Symbol.empty ? counter : this.counter,
        name: name != Symbol.empty ? name : this.name,
    );
  }
}

This could be helpful if you want to update without giving all parameters:
class AppState {
  final int counter;
  final String name;

  AppState({this.counter = 0, this.name = "John"});

  AppState.rebuild(AppState state, {counter, name})
          : counter = counter ?? state.counter,
            name = name ?? state.name;
}

var updatedState = new AppState.rebuild(state, counter = 1);

A clean way to set a value to null in the copyWith() method is to use an Optional.
`
class Optional {
T value;
Optional(this.value);
}
class AppState {
final int counter;
final String name;
AppState({this.counter = 0, this.name = "John"});
AppState copyWith({Optional counter, Optional name}) {
return new AppState(
counter: (counter == null) ? this.counter : counter.value,
name: (name == null) ? this.name : name.value,
);
}
}
class IncrementCounterAction {}
class SetNameAction {
final String name;
SetNameAction(this.name);
}
AppState counterReducer(state, action) {
switch (action.runtimeType) {
case IncrementCounterAction:
return state.copyWith(counter: Optional(state.counter + 1));
case SetNameAction:
return state.copyWith(name: Optional(action.name));
default:
// No change
return state;
}
}
`


UX issue: wallet address split onto two lines is potentially confusing

I just tried to withdraw my raiblocks from bitgrail to my desktop wallet. When I input my account address on the receive I thought only the top line was the address as, when you full screen the wallet, it still condenses the address down to two lines even though there is plenty of room to fit the whole address into one line.
This was my fault and a noob error, luckily I only did a test withdrawal of 1 rai, however it is a reasonably easy fix.
My suggestion is that you put the address in a box to make it more obvious that you should use the entire string. Additionally, when the window is enlarged or full screened the address could reformat to go on one line (perhaps do it as a 2 stage reformatting, so it's either 2 equal strings or one full string, so you never have a big string on top and a little string below).
Alternatively the label could be un-selectable, to force the user to press the copy button and remove the change of a highlighting error.

@cryptocode has verified this is done.  Closing this issue.


Cannot create gemset

So I successfuly use the cookbook to do user_install. It installs ruby-1.9.3-p194 and sets up the environment.
Then I am trying to multiple gemset and I get the error: https://gist.github.com/3726237
Even creating a single gemset like: rvm_gemset "ruby-1.9.3-p194" fails with the same chomp error.

@psykidellic Do you still need help with this?

Closing due to inactivity, please reopen if this is still an issue


rename Index -> ProverIndex

we have a VerifierIndex, so we should have a ProverIndex (which is called Index atm)
relevant code: https://github.com/o1-labs/proof-systems/blob/master/kimchi/src/prover_index.rs#L29

will do!


feat(generator): add polling policy sample

https://github.com/googleapis/google-cloud-cpp/issues/13455

This change is

I'm waiting to generate the rest of the files, until I get a review. I based it off the retry policy and took from the spanner sample that set a polling operation
To generate the second commit, I used: GENERATE_GOLDEN_ONLY=1 ci/cloudbuild/build.sh -t generate-libraries-pr

I'm waiting to generate the rest of the files, until I get a review.

That is not necessary. We don't tend to look at the generated code anyway.
FWIW, it LGTM so far.

Part of the problem is to include the new samples in the docs. e.g. this is what we do for retry policies:
https://github.com/googleapis/google-cloud-cpp/blob/393e0f39d087377806c9efcc0b4179ce1d1934be/google/cloud/iap/doc/override-retry-policies.dox#L73

Part of the problem is to include the new samples in the docs. e.g. this is what we do for retry policies:
https://github.com/googleapis/google-cloud-cpp/blob/393e0f39d087377806c9efcc0b4179ce1d1934be/google/cloud/iap/doc/override-retry-policies.dox#L73

Most of the work is in the scripts to update markdown:
https://github.com/googleapis/google-cloud-cpp/blob/393e0f39d087377806c9efcc0b4179ce1d1934be/ci/generate-markdown/update-library-landing-dox.sh#L131-L150

I'm waiting to generate the rest of the files, until I get a review.

That is not necessary. We don't tend to look at the generated code anyway.
FWIW, it LGTM so far.
Part of the problem is to include the new samples in the docs. e.g. this is what we do for retry policies:
https://github.com/googleapis/google-cloud-cpp/blob/393e0f39d087377806c9efcc0b4179ce1d1934be/google/cloud/iap/doc/override-retry-policies.dox#L73

ack i am planning on doing that in a separate PR


fonts not rendering

The Montserrat and Arial fonts are being correctly bundled into the build folder but are not being found by the style sheets / styled components.

Paired on this 29/08/24. It looks like we were referring to the fonts via src: url(...) but when running from the Vite build in https://github.com/rcpch/digital-growth-charts-react-client it doesn't know to serve the fonts from the library so ends up returning the SPA build by default.
We need the library to load the fonts. We discussed:

Loading Montserrat from the Google CDN and not trying to embed any fonts at all
Don't embed Arial, reference it directly with a standard sans-serif fallback chain for non Windows machines
Embed the fonts but use data urls so we can embed them within the library bundle
Use the Font Loading API to load them from a bundle within the library

@dmc-cambric Dan I think finally the easiest way to fix your font problem is to put this in your client:
<link rel="preconnect" href="https://fonts.googleapis.com">
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
<link href="https://fonts.googleapis.com/css2?family=Montserrat:ital,wght@0,100..900;1,100..900&display=swap" rel="stylesheet">

The fonts are bundled into the client too but serving them in a react app is not that easy. Let me know if this is possible.

@eatyourpeas This would be ideal, however our app needs to be able to function offline, so we can't always guarantee access to the google font sites. Not entirely sure what the best way forward is here, if these embedded fonts can't be accessed or if they are loaded from a url in the rcpch package, we would probably need to serve those same static font files, just from within our app.

This now fixed with #104
Note that montserrat only is bundled into the library, not Arial that is standard on most windows computers. If there specific requests for fonts those will need to be added to the bundle as b64 files. Best for people to raise and issue with this request in future.
@dmc-cambric do please let me know that this fixes your fonts issue


Add support for the Xenon resource

Fix #1
This is the same PR that #10 , but with the new packet ID. Tested on my controller.

Looks good to me! Thanks for the work. I was wondering why my attempt at this did not work - turns out looking at your code I had the AddToDeviceHandler and setting the channel variables flipped in order.


Rendering directly to a DOM element

I wonder why it is that render() accepts a VNode and a parent, rather than a target element?
It's inconsistent with how the diff() function works internally, and in some situations, this means you're forced to create a dummy parent wrapper-element, just to control where in the DOM your rendered elements end up.
While, in the case of simply adding an view to document.body, it's perhaps convenient to have render() automatically append to an empty container, it's not very practical, because the parent/child relationship of the arguments assumes a 1:1 relationship between the root of the component view and it's parent/wrapper element.
It would be much simpler and more elegant to have the external render() function work just like the internal diff() function, in terms of accepting a target DOM node, and the VNode state you'd like it to have.
It's not like the typical case of appending to document.body gets a whole lot more complex that way:
render(<App/>, document.body); // now

document.body.appendChild(render(<App/>)); // proposed

This would free us the need for dummy wrapper elements, and provides a lot more flexibility in cases where you have an existing element and want to update it directly.
For example, in this prototype I'm currently re-rendering all of the draggable panels, which are actually unrelated user-interfaces. I'll eventually change that, so that the panels themselves get created and managed individually, at which point the current API will force me to add a dummy wrapper element around every panel.
That's no disaster, obviously - it's just a bit silly, and it's not how Preact itself works internally in diff(), so it's a bit frustrating that convenience has to get in the way of correctness. The fact that the API ends up dictating HTML structure leads to real-world problems when designers deliver HTML and CSS, and a developer ends up having to change the HTML and CSS (and possibly breaking the design) in order to accommodate the framework.
This issue was widely debated here and ultimately lead to a similar API change in that library.
Thoughts?

That's and interesting question!. I guess render acts like a shorthand and keeps consistency. Most of times you only have to use once to set the root of your preact app. And you don't want to append a child, you want to replace all contents within the target container. Dealing with diff directly would be, somehow, hard to provide all needed parameters, too much effor and driving confusion for new comers..

We have it this way, because react also does it the same way
Why do you want the panels to be managed outside of preact?
( btw: you should really use transform: translate(Xpx, Ypx) + will-change: transform instead of top and left)

Why do you want the panels to be managed outside of preact?

Fair question.
All of these panels will be movable by drag-and-drop. They will also have connectors, forming a graph, likely with connections in the thousands - all of the connectors will be drag-and-drop too.
During these drag-operations, only the moving connector or panel needs to be redrawn. Preact will be important, since, while you're dragging connectors between panels, the panels themselves may need updates, and Preact will do the hard work when panel states change.
But simply having a list of all panels and (worse) a list of all connectors, and expecting this to perform well, when every single panel and connector's view gets computed and diffed with every mouse-move event, does not seem like a realistic strategy. It's going to get sluggish. Even if it isn't horribly sluggish, it'll be substantial enough to interfere with the web-audio stream that needs as much available CPU and as little latency as possible.
I know about shouldComponentUpdate of course - but querying thousands of components for every mouse-move event to find out if they should update is bad enough, and is also redundant.
So I figured I'll have a more optimized strategy that accurately updates connectors and panels individually, while Preact can be in charge of the contents of the panels.
Unless you think you can propose a better strategy that leverages Preact for more of the hard work? Of course that would be ideal.
But Preact works with trees, like the DOM, and these are graphs, so computing necessary updates, as far as I can figure, is only possibly by querying long flat lists of connectors and panels - as opposed to some kind of pub/sub strategy, where the individual nodes in the graph can notify neighboring nodes of changes, which should be much more efficient.

Most of times you only have to use once to set the root of your preact app.

Not every use-case is a single app. (see above)

Dealing with diff directly would be, somehow, hard to provide all needed parameters

Agree, but that's not what I'm suggesting - only that you'd be able to target the element you want to update, directly, instead of having to target it's parent, which forces a 1:1 parent/child relationship.

We can't render without a parent, because mounting will happen prior to the tree being appended to the DOM.
To render into an empty tree, you can use a DocumentFragment:
let frag = document.createDocumentFragment()
render(<App />, frag)

There are also ways to change the 1:1 parent-child relationship too:
https://github.com/developit/preact-slots
https://github.com/developit/preact-portal

Well, you can create component and insert it where you want:
var myDom = preact.render(h(MyComponent));
document.body.appendChild(myDom);

P.S. like you have proposed

I do think @developit has provided a fair answer here
any other thoughts or safe to close? thanks <3


Do you really need GMS Services?

Do you really need GMS Services?
For maps you could use OpenStreetMap, that would be great.

We got several requests about this topic. Eventually we're going to implement this in the future.


@conda-forge-admin,please add bot automerge

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/ibm-platform-services-feedstock#4.


Fixed hardcoded "vector" and replaced with vector_query_field variable

Description: In the max_marginal_relevance_search function of the ElasticsearchStore vector store, the name of the field corresponding to the vector embedding of the document is hard coded in the delete statement that drops the field from the document metadata. This results in an exception if the vector embedding field is customized. This PR changes the hard-coded "vector" into the vector_query_field variable.
Issue: None
Dependencies: None
Tag maintainer: @hwchase17

makes sense to me, cc @joemcelroy


Feature request: clear the compilation status in the console after every compilation

Operating System: Windows 10
Node Version: 10.3.0
NPM Version: 6.1.0
webpack Version: 4.12.0
webpack-serve Version: 1.0.4

This issue is for a:

[ ] bug
[x] feature request
[ ] modification request

Code
webpack.config.js
module.exports = {
 serve: {
  dev: {
   stats: 'none',
   clearConsole: true // proposed option: clears console after every rebuilt
  }
 }
}

Expected Behavior
It should clear previous status in console so the new one is still under the URL.
When compiling:
｢hot｣: WebSocket Server Listening at localhost:8081
｢serve｣: Project is running at http://localhost:8080
｢serve｣: Server URI copied to clipboard
｢wdm｣: Compiling...

When compilation is done:
｢hot｣: WebSocket Server Listening at localhost:8081
｢serve｣: Project is running at http://localhost:8080
｢serve｣: Server URI copied to clipboard
｢wdm｣: Compiled successfully. (11:30:11) # no new lines it just replaces the previous, time would be an interesting enhancement

Actual Behavior
Now it looks like this:

After few rebuilts it's just a huge mess in console and you cannot easily find the URL which is the probably the most important information in console.
New Feature Use Case
Add an option to clear the console after every compilation.
Sorry for the previous issue without template.

Thanks for the issue :beer:
I guess I'm a little confused as to why you would need the URL printed to the console again. It won't have changed since the process started, so it should be in the address bar of the browser you're using, and it will have been copied to the clipboard as well, unless that feature has been turned off. If you'd rather not see the output from wdm and hot then why not just set their respective logLevel settings to 'warn'?

The browser can be closed (relativaly normal in practice) and copy to clipboard is only a temporary solution.
When you have multiple dev servers then you need to find the right URL.
And I would like to keep actual messages in the console because only then I know if compilation is done. But I don't want to see in my console the previous messages which are not relevant anymore.


feat: Change service method format to split on ' ' instead of '/'

Description
As a developer, I would like to type my service methods format like -> 'String myMethod' instead of 'String/myMethod'.
Desired Solution
Instead of splitting at '/' we would need to split at ' '.
Additional Context
Very easy change :)

Done and in version 0.0.4 of service brick: https://brickhub.dev/bricks/service/0.0.4 🎉


Asynchronous loading of locales and overriding specific methods

Why this PR?
Because of my issue described in #60 and following problems in #65 that were caused by my PR (f3da9d1
What did I change?

added an asynchronous implementation in cronstrue-async.ts
kept the rest of the library nearly untouched (no breaking changes but I needed to change the way how the static variables are used in the ExpressionDescriptor)
added unit tests
added usage information to README.md

I had build errors because of console.warn, so I've added dom to the lib array in tsconfig.json. Another solution would be to use @types/nodemaybe. I also added the cronstrue-async.ts to the files array in tsconfig.json so it will be part of the build output.

Thanks for the PR @seveves but this is quite a bit of changes to the library so I'd rather not merge it.  Feel free to fork and carry on your work on your own.  I appreciate the effort, though.


Latest 3.0.0 version fails on android

Latest version is throwing BiometricStoragePlugin.kt: (265, 21): Unresolved reference: androidBiometricOnly, which is probably  just caused by this commit 48a0182f3bfea2a7e9c96bbdb1d027371c3449f1

oops, thanks for the report. Seems like my last-minute find/replace was not successful 🤦♂
just fixed it, can you please try 3.0.0-rc.2


Added width/height properties to the Script Component docs

Fixes:

Added width/height properties to the Script Component documentation.

I confirm I have signed the Contributor License Agreement.

This PR, changes style of the file which very much does not comply with https://github.com/playcanvas/engine/blob/master/.github/CONTRIBUTING.md#general
Do not use auto indentation in IDEs and try to write additions within same style as in original file and documented in link above. This will avoid unnecessary line changes. And is easier to understand what actually was changed.

Hey @vkalpias - can you confirm your cool with exposing these properties as public API?


Question: Is there any way to force RepoDb to cast enums to integers?

Hi,
in my company we use Dapper+PostgreSQL for many years but we are considering switching to use RepoDb. In our initial tests using RepoDb we hit a problem with the treatment of enums by npgsql that does not support writing enums as their underlying integer types.
With Dapper we didn't have this problem since it always casts the enums to the underlying integer.
Is there any way to have this behaviour applied by default to all parameter enums with RepoDb?
If not, would you consider this an interesting feature for RepoDb? With a little bit of guidance I could implement this and open a PR.

Have you tried enumeration Defaut Conversion? Or, the ConversionType?

Have you tried enumeration Defaut Conversion? Or, the ConversionType?

Yes, we tried both methods that but the result are the same:
System.InvalidCastException : Can't write CLR type Inoa.Db.Test.CanExecuteCmd+TestEnum with handler type Int32Handler

  Rastreamento de Pilha:
    Int32Handler.ValidateObjectAndGetLength(Object value, NpgsqlLengthCache& lengthCache, NpgsqlParameter parameter)
    NpgsqlParameter.ValidateAndGetLength()
    NpgsqlParameterCollection.ProcessParameters(TypeMapper typeMapper, Boolean validateValues, CommandType commandType)
    NpgsqlCommand.ExecuteReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken)
    NpgsqlCommand.ExecuteReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken)
    NpgsqlCommand.ExecuteNonQuery(Boolean async, CancellationToken cancellationToken)
    DbConnectionExtension.ExecuteNonQueryAsyncInternal(IDbConnection connection, String commandText, Object param, Nullable`1 commandType, Nullable`1 commandTimeout, String traceKey, IDbTransaction transaction, ITrace trace, CancellationToken cancellationToken, Type entityType, DbFieldCollection dbFields, Boolean skipCommandArrayParametersCheck) linha 1686

It appears to be a limitation of npgsql, that dows not treat the value as an integer even when the DbType is set to Int32. ConvertionType  seens exactly what I want, but I might be using wrong.
If this is something that should be working, I more than happy to work on fix. Tomorrow I will dive deep into the code and open a draft PR for you to take a look.


CustomMsg trait

We already have a CustomQuery trait, this takes a similar approach for CustomMsg, which was T: Clone + fmt::Debug + PartialEq + JsonSchema all over the code.
Even if we will need it fewer places now that the struct bound is gone, I figured it is nice to have a helper trait to combine all these requirements.
Furthermore, since almost all structs (messages, query, state) fulfil those trait bounds, it is nice to explicitly mark one as a CustomMsg. These occur very rarely. Outside of cosmwasm tests, there will be one such type per blockchain that makes extensions. I am sure one line to implement this trait for that type is worth the type-safety and reduced verbosity other places.

I like it. Anything that makes types clearer is :ok_hand: in my book.

I am also happy to revisit if CustomMsg really needs all of : Clone + fmt::Debug + PartialEq + JsonSchema.
But I would do that as another PR if desired, and having this one trait/alias will make it easier to change if we don't need them all.

I guess I need to fix tests when the CI is working again...

Okay, this passes CI now.
This PR is just about unifying the types, not changing them. I think we all agree here.
If we change the trait bounds for CustomMsg, we should open a new issue to discuss just that. It will now be much easier to change (as it is defined in one central place), and I think it may be useful to adjust them. But I am happy just to have a simpler alias to start with, that also locks it to very few types that will ever be used as CustomMsg


AF-593: Decouple kie-wb-common-datamodel from Drools

Related:
https://github.com/kiegroup/droolsjbpm-build-bootstrap/pull/541
https://github.com/kiegroup/droolsjbpm-knowledge/pull/263
https://github.com/kiegroup/drools/pull/1444
https://github.com/kiegroup/jbpm/pull/973
https://github.com/kiegroup/droolsjbpm-integration/pull/1163
https://github.com/kiegroup/kie-wb-common/pull/1133
https://github.com/kiegroup/drools-wb/pull/596
https://github.com/kiegroup/jbpm-designer/pull/654
https://github.com/kiegroup/jbpm-wb/pull/864
https://github.com/kiegroup/optaplanner-wb/pull/218

jenkins please execute full downstream build

Jenkins, please execute full downstream build.

@manstis Thanks for catching those. Working on them now.

@manstis Issues fixed.
@ederign There are test failures in some of the downstream pull requests. I will retry the full build after those are fixed.

Jenkins, please execute full downstream build.

@manstis @ederign Please do not merge this yet. There is a missing PR I need to send to kie-wb-distributions.

Jenkins, please execute full downstream build.

Jenkins, please execute full downstream build.

Build finished. No test results found.

@ederign There appear to be GitHub API issues so the downstream build is not working.
AFAICT, the only repo I have not built fully (with tests) that does not have a PR sent to it is optaplanner, so I'm building that locally now to make sure everything is okay there. If that works, can we approve this?

(Optaplanner builds locally with the changes.)

@ederign Turns out you can run the IntelliJ formatter headless, so I fixed the formatting in that file.

@ederign Fixed the formatting of the ClassTypeResolver and removed the unnecessary change to the Export-Package of uberfire-project-datamodel-commons. Everything good now?

Jenkins, please retest this.


adding nbQA support for example notebooks

Description
nbQA now includes support for ruff to format Jupyter notebooks and already has integration for black, so we can add it for both (or even justruff would be fine)
Motivation
We already use ruff as a pre-commit hook
Possible Implementation
in .pre-commit-config.yaml
Additional context
From "Does Ruff support Jupyter notebooks? -- https://beta.ruff.rs/docs/faq/#does-ruff-support-jupyter-notebooks"

I went through the resources you have attached & I want to give it a try.
Furthermore I feel that I might have doubts while working on it which I'll be clearing here itself.

@arjxn-py this is more of a discussion at this moment. We want to see how the notebooks will evolve during GSoC and then decide if we want these pre-commits or not.

Got it @Saransh-cpp.

@agriyakhetarpal I think this aligns with your GSoC project? Feel free to pick this up. @arjxn-py can also pick this up as it relates to the infrastructure project as well.

Happy to take this up.

That's nice. I discussed embedding the example notebooks on readthedocs during today's meeting (which is why I was waiting on this), so I will be restructuring some parts of the documentation in a PR soon — @arjxn-py I would recommend starting with this after that gets merged

@arjxn-py it would be nice to add some pre-commits for notebooks once #3043 is merged. You can maybe pick this up once the option dependencies PR is merged.


RaspiBlitz Sync

I updated the RaspiBlitz img to the latest version. The screen I am seeing has been displaying for days saying "Preparing Blockchain sync please wait..." I asked a telegram group for RaspiBlitz and someone said I'm fine. It's just not showing any percentage of the blockchain sync and wanted to see if anyone can help me out or if this takes days. I've looked around and some people say Raspi is slow so it's taken some weeks to sync.

Is your SSD in good shape?

It got delivered from Germany recently. SSD looks fine
I’ll replace when I can to see if that’s the issue, gratitude! I shutdown,
disconnected SSD reconnected and at least now I’m seeing this screen. At
least some of the restarts I want to say are from me tinkering with it.
On Wed, Jan 11, 2023 at 8:59 PM tysecure @.***> wrote:

Is your SSD in good shape?
—
Reply to this email directly, view it on GitHub
https://github.com/rootzoll/raspiblitz/issues/3604#issuecomment-1379780554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXEGMXWBTX2GR7JNPILABKDWR56TJANCNFSM6AAAAAATYPZN4Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm seeing the same screen. The first installation I try stuck at blockchain sync at around 40% after 5 days of sync. Then I waited another 2 days and got nothing. An error I already forgot about. (something about a couple of thousands of tries).
I decided to flash the SD card again and did a fresh install with the existing blockchain on SSD. And I'm here where you are. The web view says Waitsync "Provision done"
I was hoping for a smoother installation but now I know it's a pain in the butt

Have you tried joining the telegram group. I wasn't able to get into the english speaking one but got on the spanish speaking RaspiBlitz telegram group.

I am in the EN telegram group but Telegram with more than 100 members is too much for me. Can't follow the noise.
here's the link. Try again and wait for shieldy bot to ask you. Confirm that you are human. https://t.me/raspiblitz


Multi line suggestion bug

Fixed a bug concerning the placement of the suggestion dropdown in the case where a user is typing into a multi-line paragraph. In the process I performed a pretty massive overhaul on the suggestion-plugin code, bringing the styling inline with our current code-quality standards, adding comments where I could, and updating the code in a few other places. As a result, a few other bugs have been addressed in the process.

Original Bug: Shortcuts that require two selections (e.g. '@condition' prompts users for a second selection containing all the patients current conditions) would not close the first suggestion menu. This menu now closes.
Original Bug: Backspacing into a suggestion (e.g. '@age' -> '@ag') used to remove the menu/disable the insertion of a suggestion. This is no longer the case, and the expected element is now inserted.
Original Bug: The nil case for suggestions (i.e. just '@') used to always insert the previously typed shortcut, regardless of what the suggestion menu indicated. This has now been fixed.

Hopefully this, along with the work Julia has been doing, should prepare our for the demo and make it easier to make changes to the editor in the long term.
It's also worth noting that the changes we've made to the suggestion plugin have become pretty drastic. A result of this is that I suggest we use my fork of the original repo (https://github.com/Dtphelan1/slate-suggestions/) as the canonical version of the tool. The previous repo is largely a ghosttown, with PR's in limbo for months now (one of them being ours from August!)
Pull requests into Flux require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable (N/A) and :white_check_mark:.
Submitter:
Running the Application

[x] Manually tested in Chrome
[x] Manually tested in IE

Documentation

[x] This pull request describes why these changes were made
[x] Recognizes any potential shortcomings/bugs in the description
N/A Cheat sheet is updated
N/A Demo script is updated
N/A Documentation on Wiki has been updated
N/A Additional technical components and libraries are documented and added to wiki as needed

NoteParser

N/A  Note parser has been updated if structured phrases change

Code Quality

[x] 4-space indents - convert any tabs to spaces
[x] Use public class field syntax for binding functions - ex. handleChange = () => { ... };
[x] Code is commented

Tests

[x] Existing tests passed
N/A  Added UI tests for slim mode
N/A  Added UI tests for full mode
N/A  Added backend tests for fluxNotes code
N/A  Added note parser examples to test new functionality

Reviewer 1: Julia

[x] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
The tests appropriately test the new code, including edge cases
[x] You have tried to break the code

Reviewer 2: Name

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

This looks really awesome! All these changes really help the editor work more intuitively. The original bug, along with the three you listed, all are fixed. I also texted out typing a shortcut that would span two lines (if you type @condition 4 characters from the end of the line), and the portal follows the cursor and gets repositioned in the correct place. All the tests pass.

[JA1] If you open the condition selection portal, the page seems like it shifts. Dylan and I already talking about this one, but figured I'd note it here. He's looking into it.

[DP1] When users click on muli-value suggestion (e.g. @medication) the portal for options either won't pop up, will pop up above the cursor instead of above where the editor-caret is, or will pop up with the original dropdown containing @medication

JA1 is techincally fixed, but DP1 is a subset of this bigger issue. I'm not going to be able to fix it before I leave, as it has to do with weird interactions between concept of a Context Portal and the suggestion portal. I'd ask the people if they think it'd be worth merging (assuming there aren't other issues) and fixing the existing bugs, while recognizing that there's a new bug that we're introducing.

I'll try to take a look at it, but what do you think @jafeltra? Is the app functionally better for demos despite the new bug? I'm assuming you're saying it is better for demos @Dtphelan1 ?

AFter talking with Julia, I don't think merging this without addressing the new bugs makes a lot of sense. While there are fewer bugs, they are certainly more jarring to the user than the previous ones.

Ok. thanks! Julia, is this something that you can look at after Dylan has left for vacation?

Yeah I think so. Since the other bugs this fixes are really useful, I think it would be good to try to figure out the remaining bug and I will need a new task soon anyway.

I've pushed updates that I think fix the remaining bugs. The approach I took to fixing the bugs required passing in new props to the suggestions portal. I thought since we have such significant changes to the portal, it would be okay. If we don't think this is a good thing to do, let me know and we can maybe revisit how the changes are made.
Right now, the portal aligns with the cursor's position in the editor. The second portal, the context portal with the conditions list, gets positioned in the same location when it opens based on using the enter key to select @condition, clicking condition, and autocompleting the word condition. The first portal successfully close when this is opened.
I was also able to close the portal with the escape key as part of this task. The Jira's that I think have been fixed by this PR are below. I don't think this list covers all the bugs Dylan listed at the top of this PR, which should still all be fixed.

Jira 933: the first suggestion portal closes if you select something that opens the second (https://jira.mitre.org/browse/ASCODCP-933).
Jira 934: Choosing the initial option after typing @ or # works as expected (https://jira.mitre.org/browse/ASCODCP-934)
Jira 941: Escape key closes the portal (https://jira.mitre.org/browse/ASCODCP-941)

It looks like @jafeltra and @gregquinn2001 have made some great fixes to the bugs before. I'm not finding any new bugs, but I might be a little blind to the testing in virtue of having done it too many times... Any chance we can get a reaffirmation from @gregquinn2001 and @jafeltra that this task is looking good, and at least one new review (maybe from @danlee1025 or @laclark22?) Others should let me know what you think!

Talking with @jafeltra and she's pointed out another bug that exists, though it's not blocking:
[JA(-DP?)2] A portal started on one line doesn't overflow off that line in sync with the text. That is to say, if a shortcut is started on line 1 and overflows over onto line 2, the portal lags behind the shortcut in an off-by-one fashion. Eventually it catches up, but no on that first overflowing character.

[NN1] - When I click on @condition from the context tray, the drop down pops up in the top left corner of the screen. This happens in both IE and chrome

I'd log a JIRA for [JA(-DP?)2]. You definitely win for most complicated bug identifier in a pull request! That's not a challenge though to create more complicated ones!

I'll save further complications for future PR's 😄

@nicoleng12 Does this address your bug sufficiently?

@Dtphelan1 Yes that addresses NN1, thanks! This PR looks good. All the bugs are fixes. All the tests pass. I've tested this manually in IE and chrome as well.
This just needs to be rebased

Rebased the branch


.NET Runtime Requirements missing

The NuGet.exe CLI appears to require some minimum .NET Framework version higher than 4.5.2. I get the following error from any command:
Unhandled Exception: System.TypeLoadException: Could not load type 'System.AppContext' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'.
   at NuGet.CommandLine.Program.Main(String[] args)

But the documentation doesn't clarify what the minimum version is.
Ideally, there would be documentation for each available version of the tool so I can figure out which ones I might be able to run without upgrading.

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

ID: 3460a9b2-3b49-1cda-add2-14d26e217716
Version Independent ID: 95b91106-faff-c7f6-3699-94d643b5cf73
Content: Installing NuGet client tools
Content Source: docs/install-nuget-client-tools.md
Product: nuget
GitHub Login: @karann-msft
Microsoft Alias: karann

@bladeoflight16  I ran into this issue on a brand new Windows Server while trying to execute a cake script as part of my CI process. According to the .NET API, System.AppContext ships with .NET Framework 4.7.2. After I installed .NET Framework 4.7.2 via Choco and then rebooted, nuget.exe ran as expected.

Getting this on Linux simply by:
> sudo apt-get install nuget
> nuget update -self
> nuget

Unhandled Exception:
System.TypeLoadException: Could not load type 'System.AppContext' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'.
[ERROR] FATAL UNHANDLED EXCEPTION: System.TypeLoadException: Could not load type 'System.AppContext' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'.

I just got this on Linux Mint 18.3 with Dotnet Core SDK 2.2

Updating to the latest version of Mono fixed it for me. I was on 4.x.x. Now I have 5.x.x and everything works fine. https://www.mono-project.com/download/stable/#download-lin

@rrelyea - thoughts?

Created https://github.com/NuGet/docs.microsoft.com-nuget/pull/1449 to document .net framework version required.
Unsure of the answer for mono, etc...

merged Rob's changes - https://github.com/NuGet/docs.microsoft.com-nuget/commit/887a44e17384e1c3477c98d9eb09d74135e949ae

@karann-msft Karan, looks like this was fixed in the PR?

Appears to be fixed


forc client check

Closes #145
Small update to fuelup check --verbose to correctly display forc-run and forc-deploy as the same package forc-client.

Sounds good - I'll create an issue for this just so we remember!


Male voice

First thank you, I have solved the issue opened thanks to you support. In my understanding both Melgan (that I have tried) and Waveglow (not run yet) have a female voice. To have a male voice is it necessary to train from scratch the model? Or add support to a specific vocoder?
Thank you.

@loretoparisi  I think that (Universal Vocoding)[https://github.com/bshall/UniversalVocoding] meets your requirement. I have a plan to support multi-speaker TTS, but it is not my top priority now. You are welcomed to fork this repository for a male-TTS  implementation. I think both the f0_min for PyWorld Vocoder and some parameter related to preprocessing have to be modified for male speakers.

HI @ming024
So your suggestion would be to integrate Universal Vocoder into your implementation of FastSpeech2 ?

Hi @ming024,
I am trying what you suggested, to use Universal Vocoder for synthesizing the output of FastSpeech2.
The output of FS2 has size:
torch.Size([1, 80, x])
while UV wants something like this as input:
torch.Size([1, x, 80])
I tryed by swapping the axis but of course didn't work, what I got was just noise or silence.
So this confirms what you suggested, that maybe it is worth retraining UV with the same parameters as FS2.
Another thing I tried was playing with the UV params, using the mel_spec computed from the generated wav file by FS2. I got some interesting changes in pitch, but nothing I can really use for my purpose of changing the speaker.
If you have any other advice I could use please let me know, thanks a lot!

@shoegazerstella Actually I haven't tried Universal Vocoding before so I am not sure where the error come from.
I think the decoder of FastSpeech is similar to the decoder of a voice conversion model. Some VC models use vector quantization or other tricks to learn a discrete embedding space, and find out that phonetic information is contained therein. Maybe there is some way to combine the training of both tasks, no matter jointly training or pretraining, etc.

@ming024 Any idea what specific changes one needs to make for Male voice cloning?

@carankt  any updates about male voice or how to do it, please?

Guys multi-speaker synthesis is supported now.


Create a public repository for easy updating.

Create a public repository for easy updating. Now deb/rpm packages are created, but they need to be installed manually, which is not very convenient. As option there is service cloudsmith.io, which allows you to create free repositories without any restrictions for Open-Source projects (https://cloudsmith.io/f/open_source/). The service has its own API for easy automatic interaction.
I think it would make it easier to install and update your services for systems without docker.

Why not. I will try to do this

You could also use packagecloud.io like the guy of carbonapi is doing: https://packagecloud.io/go-graphite/

Hi @lomik, just saw You released a new version, did you have time to look into this?

I spent some time studying the "ubuntu ppa" repositories. Maintaining such a repository takes too much effort. Probably have to use the packagecloud or something like it :(

Indeed, maintaining a repo completely is waste of time and overkill. Something like packagecloud only requires you to upload the packages and that’s it.

@cloudsmith-io is just as easy btw (thanks to @beliys for the mention), and we're always on hand to give advice and support if needed. :-) If you're open-source you don't need to apply for it, just create an open-source repository and off you go!

Packages were uploaded to packagecloud https://packagecloud.io/go-graphite/stable . graphite-clickhouse packages will be uploaded later

Thanks, updated from the repository


[Site Settings Page] The 'Close' button in the confirmation message is unavailable."

Environment: Windows 10 Home, Google Chrome Version 119.0.6045.200 (Official Build) (64-bit)
Reproducible: always
Build found: N/A
Preconditions

The user is logged into the system as an Administrator.
The 'Site Settings' page is opened.

Steps to reproduce

Click on the 'Save' button.
Click on the 'Close' button represented by an "x" to close the confirmation message.

Actual result
The close button is non-functional.
Expected result
The confirmation window should close upon clicking the 'Close' button.
User story and test case links
E.g.: "User story #426
Labels to be added
"Bug", Priority ("Medium: "), Severity ("Medium:"), Type ("Functional"), "API" (for back-end bugs).

Please see the issue #562


Warn for rules used with multiInstance when no bindingContext

We have a warning for when a user runs detekt without type solving but with type solving rules enabled. This was broken with the introduction of multiple instances for the same rule. This PR fixees that.
Also this PR goes in the line of reduce the usages of name where it is not needed because I think that we can remove it from Rule.

Warnings

:warning:
This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

  Generated by :no_entry_sign: dangerJS against 5af330f5b09b3f9c386cc4eaaade022435b4599f


PaginatedDataTable ignore showCheckboxColumn

Hi,
The PaginatedDataTable has a showCheckboxColumn parameter to display or not the checkbox for selectable rows. But the paremeter is not used ! It should be passed to  the DataTable children where the implementation is written

@bkonyi
was this line removed accidentally or intentionally?
https://github.com/flutter/flutter/commit/d907a2622907df5c059b6bcd12f8bf99b9564b93#diff-243d50d685c9399394efa4693f2ac703L473

That was removed unintentionally (as I had actually just added the showCheckboxColumn around the same time as that PR), so it should be added back.


form states

give styling to the ng-pristine, ng-dirty, ng-untouched?, ng-valid, ng-invalid.

should apply to all forms.
so far, forms include:

login view
search view


 not working anymore using referers in my code

Hello,
using renderers={renderers} the onLinkPress={(evt, href) => Linking.openURL(href)} doesn't work anymore.
Anyone else got this problem?

Totally my fault. I wrote renderers inside the HTML component tag in the render!

If I write the renderers outside the <HTML .... /> the links work, but not the custom tags like table, tbody, tr, td, and so on.
Viceversa everything works except the A HREF

I'm also seeing this, I needed to write the renderer (as used in the readme) so I could work with the images dimensions. Doing so needs to call renderers={renderers} in the tag, but in doing that it disables all links (a tags).
Is there any help on this, even if its just the renderer code to place in the page similar to what was used in the readme for the image example?

Looks like this has been resolved here: #12


Updating for ARM?

Is there a document that covers turning on and off diagnostics for ARM VMs through the Portal? It's odd that the main document for sending diagnostics to storage only covers Cloud Services. I assume this is part of a process of migrating monitoring-related docs here from other places, but otherwise there is only a PowerShell process under the VM extension docs.
https://docs.microsoft.com/en-us/azure/virtual-machines/extensions/diagnostics-windows

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

ID: fc7bc9c3-66f0-7447-f360-8d4f65fa2498
Version Independent ID: 64f4fc80-c87a-a494-7c73-f91a1a878f97
Content: Store and View Diagnostic Data in Azure Storage
Content Source: articles/monitoring-and-diagnostics/azure-diagnostics-storage.md
Service: azure-monitor
GitHub Login: @jpconnock
Microsoft Alias: jeconnoc

test

 Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

@femsulu can you check on the status of this request again? Will this whole capability (saving metrics to storage accounts for VMs) stay the same through the classic metric retirement?

The underlying content has been retired and no longer relevant.
Newer version @ https://docs.microsoft.com/en-us/azure/azure-monitor/platform/data-sources
#Please-close


Bitstream does not run on 5CEFA5F23I7 , missing CMUXVG and CMUXHG statement ?

In current state mistral is able to generate bitstream for 5CEFA5F23I7 / 5CEFA5F23C8. Unfortunately, the fpga is unable to run successfully the bitstream to sram and revert to previously loaded bitstream in flash produced by quartus.
When comparing decompiled bitstream between mistral and quartus there are missing CMUXVG and CMUXHG statement at the beginning of each bitstream for 5CEFA5F23I7 by mistral.
Also, when decompiling quartus generated bistream, mistral-cv output this
Source unknown on rnode SCLK.014.000.0015 ( 6, 01)
Source unknown on rnode SCLK.014.000.0020 ( 6, 04)
Source unknown on rnode SCLK.014.000.0025 ( 6, 08)
Source unknown on rnode SCLK.047.000.0005 ( 6, 08)
Source unknown on rnode SCLK.047.000.0009 ( 6, 02)

I have compared decompiled bistream between mistral and quartus for 3 examples here
In summary decompiled bistream generated by mistral begin like this
m 5CEFA5F23C8
o COMPRESSION_DIS 1
o OPT_B ffffff40.adffffff
r SCLK.047.000.0009 SCLKB<IP_ADDRESS>5
r SCLK.047.000.0009 SCLKB<IP_ADDRESS>4
r SCLK.047.000.0009 SCLKB<IP_ADDRESS>5

and decompiled bistream generated by quartus begin like this
m 5CEFA5F23C8
o COMPRESSION_DIS 1
o IDCODE 22
o JTAG_ID 0078fd4b
o OPT_B ffffff40.adffffff
r CMUXVG.028.000.0:CLKOUT SCLK.014.000.0000
r CMUXVG.028.061.0:CLKOUT SCLK.014.000.0001
r CMUXVG.028.000.3:CLKOUT SCLK.014.000.0005
r CMUXVG.028.061.1:CLKOUT SCLK.014.000.0010
r CMUXVG.028.000.1:CLKOUT SCLK.014.000.0012
r CMUXVG.028.061.0:CLKOUT SCLK.014.000.0013
r CMUXVG.028.061.3:CLKOUT SCLK.014.000.0014
r CMUXHG.000.008.0:CLKOUT SCLK.014.000.0022
r CMUXHG.000.008.3:CLKOUT SCLK.047.000.0007
r CMUXHG.000.008.0:CLKOUT SCLK.047.000.0008
r CMUXHG.068.008.3:CLKOUT SCLK.047.000.0010
r CMUXHG.000.008.3:CLKOUT SCLK.047.000.0020
r CMUXVG.028.000.1:CLKOUT SCLK.047.000.0021
r CMUXHG.068.008.2:CLKOUT SCLK.047.000.0024
r CMUXVG.028.000.2:CLKOUT SCLK.047.035.0027
r SCLK.047.035.0027 SCLKB<IP_ADDRESS>3
r SCLK.047.035.0027 SCLKB<IP_ADDRESS>3

I don't know if I can re-inject those CMUXHG CMUXVG from quartus to mistral, then ajusting numbers like<PHONE_NUMBER> for SCLK. Then recompile to rbf and try if it work on fpga.
Other than that, bitstream generated by mistral is very close to quartus.
Let me know if there are experiments I can do to try make mistral working on Cyclone V 5CEFA5F23 device.
Thanks.

Unfortunately, the fpga is unable to run successfully the bitstream to sram and revert to previously loaded bitstream in flash produced by quartus.

Can you provide a bit of info on how you are generating and loading the bitstream?
The CMUX stuff can't be directly copied across because the two tools might have routed the clock differently, this depends on other placement and routing decisions. Whilst I can't rule out a problem there, I don't expect it would cause the bitstream to be totally rejected either.

In my makefile I have this
$(PROJ).rbf : $(PROJ).json Makefile
	nextpnr-mistral  -r --device $(DEVICE)  --json $< --rbf $@    --freq $(TARGET_FREQUENCY) --qsf $(QSF)

$(PROJ).json : $(FILES)
	yosys -p "read_verilog $(FILES) ; synth_intel_alm  -family cyclonev  -top $(TOP)  ; write_json $(PROJ).json"

I use openFPGALoader to load bitstream for both quartus and mistral.
openFPGALoader -m --bitstream ${PROJ}.rbf

that all looks reasonable, I think I'll have to ask @galibert if they can see anything wrong here. Possibly the missing IDCODE and JTAG_ID writes?

@gatecat you are right . I copied  IDCODE and JTAG ID from quartus  in .bt file of mistral. Then compiled .bt  to .rbf  and it work.
I will test other example to confirm.
So this is not related to CMUXVG.

Very good, it is only IDCODE and JTAG missing in rbf file. The second example work with adding those 2 lines in .bt file
o IDCODE 22
o JTAG_ID 0078fd4b

Definitely one for @Sarayan, but should be easily fixable.
On Tue, 12 Apr 2022, 22:51 infphyny, @.***> wrote:

Very good, it is only IDCODE and JTAG missing in rbf file. The second
example work with adding those 2 lines in .bt file
o IDCODE 22
o JTAG_ID 0078fd4b
—
Reply to this email directly, view it on GitHub
https://github.com/Ravenslofty/mistral/issues/17#issuecomment-1097258057,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALPDW67X2RYCNI3YKS7D6LVEXV4TANCNFSM5TIGXBCQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

To be able to program to flash, I have to put JTAG_ID 2b220dd, the  jtag idcode detected by openFPGALoader for target fpga.  When programming to sram, only "o IDCODE 22" is important.

I guess I have an issue somewhere with the default IDCODE.  E/A5 should
know it's 0x22...
OG.
On Wed, Apr 13, 2022 at 12:31 AM infphyny @.***> wrote:

To be able to program to flash, I have to put JTAG_ID 2b220dd, the jtag
idcode detected by openFPGALoader for target fpga. When programming to
sram, only "o IDCODE 22" is important.
—
Reply to this email directly, view it on GitHub
https://github.com/Ravenslofty/mistral/issues/17#issuecomment-1097294819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACGSF4OO5H4LFG5UKJVJC2DVEX2U5ANCNFSM5TIGXBCQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Fixed 9b0dcbe29697bb803cef4d4f6745c0e3e017e4bb

Perfect, thanks.