Failed to load library libortextensions.so

Hi, I wanted to try this awesome project and write an article about it, but I encountered an issue.
I compiled the onnxruntime-extensions on Linux and copied the library (actually everything, as it wasn't working) to the sd4j directory which contains your git repository:
sd4j$ ls -1
CONTRIBUTING.md
images
libnoexcep_operators.a
libocos_operators.a
libortcustomops.a
libortextensions.so
libortextensions.so.0
libortextensions.so.0.10.0
LICENSE.txt
pom.xml
README.md
SECURITY.md
src
target
text_tokenizer
THIRD_PARTY_LICENSES.txt

Then I run:
sd4j$ mvn package exec:exec -DmodelPath=../stable-diffusion-v1-5/
And I get the following error:
[INFO] --- exec-maven-plugin:3.1.0:exec (default-cli) @ sd4j ---
Gtk-Message: 21:05:49.827: Failed to load module "canberra-gtk-module"
Exception in thread "main" java.lang.IllegalStateException: Failed to instantiate SD4J pipeline
	at com.oracle.labs.mlrg.sd4j.SD4J.factory(SD4J.java:356)
	at com.oracle.labs.mlrg.sd4j.SD4JApp.<init>(SD4JApp.java:89)
	at com.oracle.labs.mlrg.sd4j.SD4JApp.main(SD4JApp.java:421)
Caused by: ai.onnxruntime.OrtException: Error code - ORT_FAIL - message: Failed to load library libortextensions.so with error: libortextensions.so: kan gedeeld objectbestand niet openen: Bestand of map bestaat niet
	at ai.onnxruntime.OrtSession$SessionOptions.registerCustomOpLibrary(Native Method)
	at ai.onnxruntime.OrtSession$SessionOptions.registerCustomOpLibrary(OrtSession.java:700)
	at com.oracle.labs.mlrg.sd4j.TextEmbedder.<init>(TextEmbedder.java:113)
	at com.oracle.labs.mlrg.sd4j.SD4J.factory(SD4J.java:340)
	... 2 more
[ERROR] Command execution failed.

I looked at the code and the TextEmbedder seems to not even load the libortextensions but ortextensions:
        this.tokenizerOpts.registerCustomOpLibrary(System.mapLibraryName("ortextensions"));
Probably I'm doing something stupid here, but I couldn't quickly figure out what, so any help/tips would be really appreciated.

The mapLibraryName call does the right thing, you can see it pulled out libortextensions.so which is the correct form for Linux (e.g. on Windows it'll turn ortextensions into ortextensions.dll). How big is the libortextensions.so file (or whatever it's linked to)?
The other thing to check is that this currently targets ORT 1.14 and ORT-extensions 0.7.0 (because I forgot to put the ORT-extensions version number in the docs), and I think you compiled the head of ORT-extensions, so try compiling the v0.7.0 tag. I wouldn't expect that to result in a file not found (which is what Google translate said that error was, my Dutch is not great so if it means something else let me know), unless there were linker errors. Could you run ldd libortextensions.so and see if it found all the libraries?

I managed to generate an image using ORT 1.14 and the head of ORT-extensions main branch on an Ubuntu 22.04 desktop so it's not that. I get the following output for the libraries it needs:
$ ldd libortextensions.so
	linux-vdso.so.1 (0x00007ffdb533e000)
	libgtk3-nocsd.so.0 => /lib/x86_64-linux-gnu/libgtk3-nocsd.so.0 (0x00007f012ea00000)
	libopenblas.so.0 => /opt/OpenBLAS/lib/libopenblas.so.0 (0x00007f012da00000)
	libz.so.1 => /lib/x86_64-linux-gnu/libz.so.1 (0x00007f012edbe000)
	libstdc++.so.6 => /lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007f012d600000)
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f012ecd7000)
	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f012ecb7000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f012d200000)
	/lib64/ld-linux-x86-64.so.2 (0x00007f012f696000)
	libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007f012ecb2000)
	libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f012ecad000)
	libgfortran.so.3 => /lib/x86_64-linux-gnu/libgfortran.so.3 (0x00007f012d8ce000)
	libgomp.so.1 => /lib/x86_64-linux-gnu/libgomp.so.1 (0x00007f012ec63000)
	libquadmath.so.0 => /lib/x86_64-linux-gnu/libquadmath.so.0 (0x00007f012ec19000)

Thanks for the quick reply, it looks indeed like I'm missing some libraries:
sd4j$ ldd libortextensions.so
	linux-vdso.so.1 (0x00007fffc951b000)
	libz.so.1 => /lib/x86_64-linux-gnu/libz.so.1 (0x00007f5eb5a9e000)
	libstdc++.so.6 => /lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007f5eb4c00000)
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f5eb59b5000)
	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f5eb5991000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f5eb4800000)
	/lib64/ld-linux-x86-64.so.2 (0x00007f5eb5acf000)

So I used v0.7.0:
onnxruntime-extensions$ git status
HEAD detached at v0.7.0
nothing to commit, working tree clean

And build it again:
./build_lib.sh --config Release --update --build --parallel

[100%] Linking CXX shared library lib/libortextensions.so
[100%] Built target extensions_shared
2023-12-19 23:02:23,960 util.run [DEBUG] - Subprocess completed. Return code: 0
2023-12-19 23:02:23,960 build [INFO] - Build complete

However it gives the same result:
onnxruntime-extensions/build/Linux/Release/lib$ ldd libortextensions.so
	linux-vdso.so.1 (0x00007ffdaa7e7000)
	libz.so.1 => /lib/x86_64-linux-gnu/libz.so.1 (0x00007f2af5d44000)
	libstdc++.so.6 => /lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007f2af5000000)
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f2af5317000)
	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007f2af5d20000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f2af4c00000)
	/lib64/ld-linux-x86-64.so.2 (0x00007f2af5d75000)

I checked the output and it complains about some things it can't find (see below), but as shown above the build still succeeds.
2023-12-19 23:05:15,253 build [DEBUG] - Command line arguments:
  --config Release --update --build --parallel
2023-12-19 23:05:15,254 build [INFO] - Build started
2023-12-19 23:05:15,254 build [INFO] - Generating CMake build tree
2023-12-19 23:05:15,254 util.run [INFO] - Running subprocess in 'build/Linux/Release'
  /usr/bin/cmake /home/johan/InfoQ/SD4J/onnxruntime-extensions -DPython_EXECUTABLE=/usr/bin/python3 -DOCOS_ENABLE_SELECTED_OPLIST=OFF -G 'Unix Makefiles' -DCMAKE_BUILD_TYPE=Release
-- ONNX Runtime URL suffix: v1.10.0/onnxruntime-linux-x64-1.10.0.tgz
-- Fetch googlere2
-- Fetch opencv
-- Detected processor: x86_64
-- Looking for ccache - not found
-- libjpeg-turbo: VERSION = 2.1.0, BUILD = opencv-4.5.4-libjpeg-turbo
-- libva: missing va.h header (VA_INCLUDE_DIR)
-- Could not find OpenBLAS include. Turning OpenBLAS_FOUND off
-- Could not find OpenBLAS lib. Turning OpenBLAS_FOUND off
-- Could NOT find Atlas (missing: Atlas_CBLAS_INCLUDE_DIR Atlas_CLAPACK_INCLUDE_DIR Atlas_CBLAS_LIBRARY Atlas_BLAS_LIBRARY Atlas_LAPACK_LIBRARY)
-- Could NOT find BLAS (missing: BLAS_LIBRARIES)
-- Could NOT find LAPACK (missing: LAPACK_LIBRARIES)
    Reason given by package: LAPACK could not be found because dependency BLAS could not be found.

-- VTK is not found. Please set -DVTK_DIR in CMake to VTK build directory, or to VTK install subdirectory with

Then I looked at the libraries that where missing from my libortextensions.so and installed some of the missing ones:
sudo apt-get install gtk3-nocsd
sudo apt-get install libopenblas64-dev

However that didn't seem to help:
onnxruntime-extensions/build/Linux/Release/lib$ ldd libortextensions.so
	linux-vdso.so.1 (0x00007ffd755c3000)
	libz.so.1 => /lib/x86_64-linux-gnu/libz.so.1 (0x00007fc181824000)
	libstdc++.so.6 => /lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007fc180a00000)
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007fc18173b000)
	libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007fc181717000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007fc180600000)
	/lib64/ld-linux-x86-64.so.2 (0x00007fc181855000)

Do you know how I can get those libraries in the libortextensions.so?

The different set of libraries shouldn't make much difference, it's more to check that none of them say <foo> => not found as that would cause a linking error. We only use the tokenizer from onnxruntime-extensions so it doesn't matter that yours isn't linking against BLAS or other math stuff like mine did.
This then suggests that it's some kind of path issue, which is weird because it worked fine on my Ubuntu box earlier. As you're running the exec in Maven then it shouldn't be a working directory issue (as it looks for that library in the working directory), so maybe there's something odd in the path resolution logic inside ONNX Runtime. Could you try adding the absolute path into line 113 in TextEmbedder (e.g. this.tokenizerOpts.registerCustomOpLibrary("/<path>/<from>/<root>/<to>/sd4j/"+System.mapLibraryName("ortextensions"));?

ORT's library loader bottoms out here - https://github.com/microsoft/onnxruntime/blob/main/onnxruntime/core/platform/posix/env.cc#L537, and it might be that I should modify ORT's Java code to convert that String into a path, get the absolute path and then pass it in. Either that or prepend ./ to it on Linux as it seems like dlopen might not always search the working directory first.
If the absolute path works for you then could you also try using "./"+System.mapLibraryName("ortextensions") and see if that works?

Thanks a lot for the pointers, I managed to get it working on my machine. Apparently it prepends "lib" to "ortexensions", but somehow that doesn't work on my machine. I managed to get it working by placing the libortexensions.so file in the sd4j/lib folder and then adding the bold part of the code as follows:
this.tokenizerOpts.registerCustomOpLibrary(System.mapLibraryName("/libortextensions"));

Indeed the workaround you proposed also works:
this.tokenizerOpts.registerCustomOpLibrary("./"+System.mapLibraryName("ortextensions"));

Ok, I'll check that that behaves itself on macOS and Windows then update the code.

Great, thanks for the quick support! You can expect an article about it in the near future on InfoQ

Yes it now works on my machine.

I checked on Windows, macOS and my Linux box and the fix didn't affect anything on those systems, so I've pushed it to main.


[BUG] origins:name and origins:type Damage Condition Types seem to be swapped

Describe the bug
Pretty simple issue; The origins:name Damage Condition Type is trying to use the Damage Type IDs, while the origins:type Damage Condition Type is trying to use the old Damage Source Name system, when it should be the other way around.
To Reproduce
Steps to reproduce the behavior:

Make any power that needs a Damage Condition.
Make a Damage Condition with the type origins:name
Type a Damage Source Name in the name field (i.e. onFire)
See VSCode complain

For the opposite behavior:

Make any power that needs a Damage Condition.
Make a Damage Condition with the type origins:type
Try to type the ID of a Damage Type in the damage_type field (i.e. minecraft:on_fire)
The autocompletion will show Damage Source Names instead.

Expected behavior
origins:type should use Damage Type IDs, and origins:name should use Damage Source Names.
Screenshots

Fixed


[Monitor][Query] Update CHANGELOG for release

This updates the changelog for the azure-monitor-query 1.2.0 release.

API change check
API changes are not detected in this pull request.


PLT-7815 - Made RunIndexerConfig a lensy record

RunIndexerConfig is now a record, and lenses are generated for it.
Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[x] Commit sequence broadly makes sense and have useful messages
[ ] Important changes are reflected in changelog.d of the affected packages
[x] Relevant tickets are mentioned in commit messages

PR

[ ] (For external contributions) Corresponding issue exists and is linked in the description
[x] Targeting main unless this is a cherry-pick backport
[x] Self-reviewed the diff
[x] Useful pull request description
[ ] If relevant, reference the ADR in the PR and reference the PR in the ADR
[x] Reviewer requested

There are warnings because the lenses aren't currently used (and using them in the places I'm already accessing the data would degrade readability).
If we're happy with the warnings for the time being I'll leave it as is, otherwise I can ignore the unused-top-bindings warning in that file, if we want.

Simply export the lens records in the module. CI doesn't accept code with warnings.


Update the manifest of serving to 0.14.0

Fixes #
Proposed Changes

This PR updates the manifest of serving to 0.14 version.

Release Note

/lgtm


feat(tiller): add --net-host flag to 'helm init'

When 'helm init --net-host=true' is run, Tiller will be installed with access to the host network (net=host in Docker)
Closes #2222

The documentation needs a rebase after the last merge. Sorry about that.

We (collectively) need to figure out a good test pattern to verify init changes to the manifests. But I think that's out of scope for this PR.


Events from previous page are still running

What i am trying to do ?
I have setup two track players, in one page I want the duration of audio to be 10 seconds so I wrote this piece of code
when this page loads by default value of stopMusic.current is set to true
const stopMusic = useRef(true)
useEffect(() => {
if (stopMusic.current == true) {
progress.duration = 10;
logOnConsole('dur', progress.duration)
}
logOnConsole('Progress.duration', progress.duration)
// progress.duration
    if (progress.position >= progress.duration) {
        TrackPlayer.stop();
        progress.position = 0;
        setPlaying(false)
    }

}, [progress.position])

The above code runs fine in the first page but when I navigate to next page I set stopMusic.current to false and then navigate . Somehow this piece of code still runs on the next page.
Update when running code in next page , the value of stopMusic is being printed once false and true like
stopMusic.current is false
stopMusic.current is true

@vishangj123 this isn't a bug but a question about how to use the library. Please ask the community on Discord.
Thanks!


[Question]: I think it's better to move the samples to root project?

General info

[X] I checked the troubleshooting page for similar problems

What is your question?
HI,
I think it's better to move samples to the root project, it's easier for testing and debugging. You can directly run the sample with the latest changes without the need to publish the library. Also, when you clone the project, the IDE will only recognize the root project and you need to open the samples as separate projects.
Relevant log output (optional)
No response

Yea, I think that should be fine. Originally there were some problems with Compose in the same class path, but that has been fixed a while ago.

Handling this currently, unsure if this as easy as it sounds. Each sample has their own modules & settings, which makes it hard to include them all without a lot of boilerplate. Looking at the KotlinX dependencies, most of them have samples with their own gradle project.

@MohamedRejeb any idea on this?

I think that the library needs convention plugins, it will make this much easier.
The build.gradle files will look like this
https://github.com/MohamedRejeb/Calf/blob/main/calf-file-picker/build.gradle.kts
I can work on this if it makes sense to you.
For modules with different targets we can have different function to setup targets depending on the case.

If we mean the same thing, we already have some methods to make the configuration easier:
https://github.com/supabase-community/supabase-kt/blob/master/Storage/build.gradle.kts
But yea, we can probably continue improving on these, if we include all samples. I'm wondering, how should we handle the project build (when changes get merged into master)? We could try to build & compile all samples in the GH actions, but that is probably going to take ages since building the root project already takes 40 min. Same with the PR actions.

root project already takes 40 min.
Probably a bit less if the Gradle files are cached, but its still ~25-30min.

We can conditionally exclude samples from build when needed. Something like this should work:
if (System.getProperty("LibrariesOnly") != "true") {
    include(":sample:kmp")
}

./gradlew -PLibrariesOnly=true build

We can close this issue now, all non-deprecated samples have been migrated.


feat(docker-node): update version of undici manually

#415

Codecov Report

Merging #421 (e9fe0e4) into develop (641a23a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #421   +/-   ##
========================================
  Coverage    62.21%   62.21%
========================================
  Files           94       94
  Lines         3104     3104
  Branches       690      690
========================================
  Hits          1931     1931
  Misses        1063     1063
  Partials       110      110

Continue to review full report in Codecov by Sentry.

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

fetch has been natively supported in Node.js LTS.
Ref:

fetch: https://developer.mozilla.org/en-US/docs/Web/API/Fetch_API#browser_compatibility


Add test for features

add test for ValidDataFractionPredicate and MaxNDVI

Codecov Report
Base: 82.10% // Head: 82.16% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (f34049f) compared to base (ead7d09).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #179      +/-   ##
===========================================
+ Coverage    82.10%   82.16%   +0.05%
===========================================
  Files           50       50
  Lines         3750     3750
===========================================
+ Hits          3079     3081       +2
+ Misses         671      669       -2

Impacted Files
Coverage Δ

eogrow/core/logging.py
98.63% <0.00%> (+1.36%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


fix: Added fromBlock and toBlock parameters to getLogs and fixed typo.

Added toBlock and fromBlock parameters
Fixes #2464

fixed in this PR https://github.com/hashgraph/hedera-json-rpc-relay/pull/2521


feat(RealitioProxy): new contract with appeal support

I believe removing the IForeignArbitrationProxy interface will break the cross-chain interaction. I didn't think of that in my previous comment, sorry about that.
Is there any specific reason to make it implement IDisputeResolver... if yes, then we should change the IForeignArbitrationProxy interface and make the required changes in both RealitioForeignArbitrationProxy and RealitioHomeArbitrationProxy.
Notice that the above would require updates from the Realitio team in their interfaces as well.
If this is meant to be used with the Dispute Resolver, I think the best approach is to comply with the already existing interfaces here and then write an adapter contract which will wrap this one and is compatible with the Dispute Resolver.

I answered on Slack


Automated cherry pick of #4728: fix: eip duplicate name

Cherry pick of #4728 on release/2.10.0.
#4728: fix: eip duplicate name

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@0b5d223). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #4734   +/-   ##
================================================
  Coverage                  ?   7.45%
================================================
  Files                     ?     418
  Lines                     ?   66795
  Branches                  ?       0
================================================
  Hits                      ?    4982
  Misses                    ?   61386
  Partials                  ?     427

Flag
Coverage Δ

#aFlag
7.45% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/elasticips.go
1.39% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve

/approve


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #15.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #15

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.

For the host section of the recipe, you should usually use python {{ python_min }} for the python entry.
For the run section of the recipe, you should usually use python >={{ python_min }} for the python entry.
For the test.requires section of the recipe, you should usually use python {{ python_min }} for the python entry.
If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12200562548. Examine the logs at this URL for more detail.


Electric field due to dipole moments

Hi Andy,
I have a question regarding reciprocal-space dipole contributions to the electric field (computePRec method). It's the last term in the Eq (49) https://doi.org/10.1063/1.3511713 for electric field due to dipole moments (I guess it's the field due to interaction the dipole's images in other cells?). I'm not an expert in the Ewald summation to judge if the articles cited in the helPME docs have or not the term but I did run some comparison between helPME and another (non-PME) Ewald implementation where this term can be switched off. The results without the term were identical to helPME, which makes me think that this term might be missing. Am I missing something or is there a mistake in the paper by Sala et. al?
Kind regards,
Alexei

Hi, Alexei,
Thanks for reaching out.  I have read the paper you linked, as well as the paper they cite with the original definition, and here is what I think the source of the discrepancy is.
For standard (non-PME) sums, increasing the cutoff to a massive value results in an conditionally convergent sum; that is, the energy will depend on the order of summation of the terms.  Another way of thinking about it is that the crystal has a macroscopic shape and that shape determines the order of summation, which in turn defines the energy.
In Ewald summation (and PME) the standard energy terms are absolutely convergent and there is no ambiguity in the energy.  This can be reconciled with the "brute force" long cutoff answer by realizing that the h=0 term in reciprocal space is ill defined.  There's a term related to the sum of the charges, which vanishes for a neutral system.  Then there's a "surface" term that is related to the macroscopic crystal shape, which is the term you pointed out in Eq (49).
That term is related to the square dipole moment of the whole unit cell, and is usually neglected.  The force resulting from the point dipole contribution is zero, as you point out.  However, if point charges are present, they contribute to the cell's dipole moment and the corresponding force term is proportional to the unit cell dipole.  If a charge goes out of the top of the cell and back into the bottom, due to the periodic boundaries, I think this makes for a discontinuous force and I think it is why this surface term is usually neglected.  The neglect of the surface term is usually motivated by "metallic" or "tinfoil" boundary conditions where we assume an infinite dielectric surrounding the system.  The term that is derived in the paper you referenced looks like the result one gets by assuming cubic boxes arranged in a spherical shape.
I don't think either answer is wrong, it's just a different choice of boundary conditions.  If you'd like the code to compute and add surface terms, I think it will be easy to code.  Choosing the API to be something reasonable might take a little thought, but that is the hardest part.  Please let me know if you agree with my analysis and how you would like to move forwards,
Best regards,
Andy.

Hi, Alexei,
Thanks for reaching out.  I have read the paper you linked, as well as the paper they cite with the original definition, and here is what I think the source of the discrepancy is.
For standard (non-PME) sums, increasing the cutoff to a massive value results in an conditionally convergent sum; that is, the energy will depend on the order of summation of the terms.  Another way of thinking about it is that the crystal has a macroscopic shape and that shape determines the order of summation, which in turn defines the energy.
In Ewald summation (and PME) the standard energy terms are absolutely convergent and there is no ambiguity in the energy.  This can be reconciled with the "brute force" long cutoff answer by realizing that the h=0 term in reciprocal space is ill defined.  There's a term related to the sum of the charges, which vanishes for a neutral system.  Then there's a "surface" term that is related to the macroscopic crystal shape, which is the term you pointed out in Eq (49).
That term is related to the square dipole moment of the whole unit cell, and is usually neglected.  The force resulting from the point dipole contribution is zero, as you point out.  However, if point charges are present, they contribute to the cell's dipole moment and the corresponding force term is proportional to the unit cell dipole.  If a charge goes out of the top of the cell and back into the bottom, due to the periodic boundaries, I think this makes for a discontinuous force and I think it is why this surface term is usually neglected.  The neglect of the surface term is usually motivated by "metallic" or "tinfoil" boundary conditions where we assume an infinite dielectric surrounding the system.  The term that is derived in the paper you referenced looks like the result one gets by assuming cubic boxes arranged in a spherical shape.
I don't think either answer is wrong, it's just a different choice of boundary conditions.  If you'd like the code to compute and add surface terms, I think it will be easy to code.  Choosing the API to be something reasonable might take a little thought, but that is the hardest part.  Please let me know if you agree with my analysis and how you would like to move forwards,
Best regards,
Andy.

Hi Andy,
This thread and paper have been super interesting to follow. I just wanted to point out that the surface term/PBC is that there are expressions for some specific unit cell shapes that have been derived (e.g. by Hummer), but that may be going way into the reeds (Tom mentioned it briefly in the PME bible 🙂 ).
Best,
Andrés
G. Andrés Cisneros, Ph.D
Professor
Department of Chemistry
Center for Adv. Scientific Computing and Modeling (CASCaM)
University of North Texas
http://chemistry.unt.edu/~CisnerosResearch
<EMAIL_ADDRESS>he/him/his

From: Andy Simmonett<EMAIL_ADDRESS>Sent: Monday, January 11, 2021 3:39 PM
To: andysim/helpme<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [EXT] Re: [andysim/helpme] Electric field due to dipole moments (#56)
Hi, Alexei,
Thanks for reaching out. I have read the paper you linked, as well as the paper they cite with the original definition, and here is what I think the source of the discrepancy is.
For standard (non-PME) sums, increasing the cutoff to a massive value results in an conditionally convergent sum; that is, the energy will depend on the order of summation of the terms. Another way of thinking about it is that the crystal has a macroscopic shape and that shape determines the order of summation, which in turn defines the energy.
In Ewald summation (and PME) the standard energy terms are absolutely convergent and there is no ambiguity in the energy. This can be reconciled with the "brute force" long cutoff answer by realizing that the h=0 term in reciprocal space is ill defined. There's a term related to the sum of the charges, which vanishes for a neutral system. Then there's a "surface" term that is related to the macroscopic crystal shape, which is the term you pointed out in Eq (49).
That term is related to the square dipole moment of the whole unit cell, and is usually neglected. The force resulting from the point dipole contribution is zero, as you point out. However, if point charges are present, they contribute to the cell's dipole moment and the corresponding force term is proportional to the unit cell dipole. If a charge goes out of the top of the cell and back into the bottom, due to the periodic boundaries, I think this makes for a discontinuous force and I think it is why this surface term is usually neglected. The neglect of the surface term is usually motivated by "metallic" or "tinfoil" boundary conditions where we assume an infinite dielectric surrounding the system. The term that is derived in the paper you referenced looks like the result one gets by assuming cubic boxes arranged in a spherical shape.
I don't think either answer is wrong, it's just a different choice of boundary conditions. If you'd like the code to compute and add surface terms, I think it will be easy to code. Choosing the API to be something reasonable might take a little thought, but that is the hardest part. Please let me know if you agree with my analysis and how you would like to move forwards,
Best regards,
Andy.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fandysim%2Fhelpme%2Fissues%2F56%23issuecomment-758242068&data=04|01|Gerardo.Cisneros%40unt.edu|efa3990352cd4a59ae9708d8b6796953|70de199207c6480fa318a1afcba03983|0|0|637459979866943689|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=BG9a09HUpClE5vfjzrtJhzvN8xBFTO7s3r66weROWVQ%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKNH5GZN45IO57P4N4WO3XTSZNVZ5ANCNFSM4V2MYIBQ&data=04|01|Gerardo.Cisneros%40unt.edu|efa3990352cd4a59ae9708d8b6796953|70de199207c6480fa318a1afcba03983|0|0|637459979866953686|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=x6rlM9VjypA3%2FGi4E6f7FdiKUV0PpRDHAHWWMBRlYmI%3D&reserved=0.

Hi Andy,
This thread and paper have been super interesting to follow. I just wanted to point out that the surface term/PBC is that there are expressions for some specific unit cell shapes that have been derived (e.g. by Hummer), but that may be going way into the reeds (Tom mentioned it briefly in the PME bible 🙂 ).
Best,
Andrés
G. Andrés Cisneros, Ph.D
Professor
Department of Chemistry
Center for Adv. Scientific Computing and Modeling (CASCaM)
University of North Texas
http://chemistry.unt.edu/~CisnerosResearch
<EMAIL_ADDRESS>he/him/his

From: Andy Simmonett<EMAIL_ADDRESS>Sent: Monday, January 11, 2021 3:39 PM
To: andysim/helpme<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [EXT] Re: [andysim/helpme] Electric field due to dipole moments (#56)
Hi, Alexei,
Thanks for reaching out. I have read the paper you linked, as well as the paper they cite with the original definition, and here is what I think the source of the discrepancy is.
For standard (non-PME) sums, increasing the cutoff to a massive value results in an conditionally convergent sum; that is, the energy will depend on the order of summation of the terms. Another way of thinking about it is that the crystal has a macroscopic shape and that shape determines the order of summation, which in turn defines the energy.
In Ewald summation (and PME) the standard energy terms are absolutely convergent and there is no ambiguity in the energy. This can be reconciled with the "brute force" long cutoff answer by realizing that the h=0 term in reciprocal space is ill defined. There's a term related to the sum of the charges, which vanishes for a neutral system. Then there's a "surface" term that is related to the macroscopic crystal shape, which is the term you pointed out in Eq (49).
That term is related to the square dipole moment of the whole unit cell, and is usually neglected. The force resulting from the point dipole contribution is zero, as you point out. However, if point charges are present, they contribute to the cell's dipole moment and the corresponding force term is proportional to the unit cell dipole. If a charge goes out of the top of the cell and back into the bottom, due to the periodic boundaries, I think this makes for a discontinuous force and I think it is why this surface term is usually neglected. The neglect of the surface term is usually motivated by "metallic" or "tinfoil" boundary conditions where we assume an infinite dielectric surrounding the system. The term that is derived in the paper you referenced looks like the result one gets by assuming cubic boxes arranged in a spherical shape.
I don't think either answer is wrong, it's just a different choice of boundary conditions. If you'd like the code to compute and add surface terms, I think it will be easy to code. Choosing the API to be something reasonable might take a little thought, but that is the hardest part. Please let me know if you agree with my analysis and how you would like to move forwards,
Best regards,
Andy.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fandysim%2Fhelpme%2Fissues%2F56%23issuecomment-758242068&data=04|01|Gerardo.Cisneros%40unt.edu|efa3990352cd4a59ae9708d8b6796953|70de199207c6480fa318a1afcba03983|0|0|637459979866943689|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=BG9a09HUpClE5vfjzrtJhzvN8xBFTO7s3r66weROWVQ%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKNH5GZN45IO57P4N4WO3XTSZNVZ5ANCNFSM4V2MYIBQ&data=04|01|Gerardo.Cisneros%40unt.edu|efa3990352cd4a59ae9708d8b6796953|70de199207c6480fa318a1afcba03983|0|0|637459979866953686|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=x6rlM9VjypA3%2FGi4E6f7FdiKUV0PpRDHAHWWMBRlYmI%3D&reserved=0.

Hi Andy,
Thank you for your response. I'll leave it as is for now and focus on other things. I do have a feeling that's not related to the surface term (there's no epsilon_sur anywhere near it) but that it comes from differentiating the exp(ikr)*Q^mu product of the electrostatic potential eq(27) in https://doi.org/10.1063/1.481216 (cited by Sala et al.). Though I must admit I did not try to work out the equations myself.
Cheers,
Alexei

Hi Andy,
Thank you for your response. I'll leave it as is for now and focus on other things. I do have a feeling that's not related to the surface term (there's no epsilon_sur anywhere near it) but that it comes from differentiating the exp(ikr)*Q^mu product of the electrostatic potential eq(27) in https://doi.org/10.1063/1.481216 (cited by Sala et al.). Though I must admit I did not try to work out the equations myself.
Cheers,
Alexei

The electric field due to dipoles seems to be fine. The problem seems to be in the dipole energy.

The electric field due to dipoles seems to be fine. The problem seems to be in the dipole energy.


Crash Line String Relative

I am using Visual Studio 2022.  I had already turn on/off many features, but when I turned the Line String on for the first time it crashed.  I have not been able to recreate the issue.  See below for a screen shot.

I'm not able to reproduce this issue either, and the stack trace in down on VSG. If you see if again, let us know.


change notifacation email conformation can't be seen

Your notification email address has been changed.
can only be seen when mouse is hovered over the message

This is actually also a problem on the "set main wallet" confirmation page, it's just less obvious because there's additional output as well.


Suggesting to whitelist wl.spotify.com - without it password reset links don't work.

I just tried a password reset on Spotify, but the password reset link, wl.spotify.com, was blocked. I suggest that you whitelist this one.

Thank you Rick @Marcoevich!  The wl.spotify.com domains comes to us via Adguard CNAME-cloaked trackers.  See the readme there to understand why this is a problem.
Can you post this in Adguard's issues and see what they say.
I support whatever Adguard decides to do.  By guess: it stays blocked by Adguard, but you're welcome to try.
Keep us informed?

We're all aware by now about spotify and that's why is off the table here.They shouldn't be messing with it neither.

Well, as I've said before, CNAME-cloaked trackers list only shows what domains are actually CNAME's to known tracking domains. It's not a blocklist, it's just a list, it's up to the downstream lists to decide what to do if blocking causes issues somewhere. In our blocklists we selectively unblock problematic domains if the issue is important.

Thank you Andrey @Marcoevich.
I'm inclined to live with this.  Let me think about it @Marcoevich, let's let this one simmer for awhile before deciding one way of another.

Steve @StevenBlack what is the situation with implementing own whitelist here. Is there a option for this?
I don't really remember being discussed in the past.
Andrey @ameshkov thanks btw for the quick response 👍

Pull requests speak loud and carry weight Dan @dnmTX 🙃

Pull requests speak loud and carry weight

Even if the domain in question coming from upstream and it will reappear after the following update?

Dan @dnmTX well for starters we're using Andrey @ameshkov list at cross-purposes.  But we get occasional Type I errors for our specific purpose from all our sources, not just Andrey's.
In our composition, including Adguard CNAME-list is a conscious choice, and that can be revisited.  That said, overall I'm delighted with this list as a source of domains.
I presently imagine a whitelist implementation a potential post-process to cull our amalgamated lists at some point before the optional userland whitelist culling hook fires.
I'm suspicious of implementing a compensatory whitelist here for one simple reason: where does that end?
On balance I trust our curators and that has been wonderful guiding principle since 2012 when this repo's mission was conceived.
I'm leaning towards won't fix for this particular issue.  Maybe that's kicking the can down the road.

I'm suspicious of implementing a compensatory whitelist here for one simple reason: where does that end?

I got that and no arguments there,i was only mentioned whitelist due to the fact that the main curator of that list emphasised numerous times already that the list is only informative and it falls to downstream to do any adjustments.
That's all 👍

I see this issue has been marked as stale and will be closed soon if no one replies. @StevenBlack can you decide what to do with this issue? I'm happy either way as long as there's human thought process behind it.

Hi Rick @Marcoevich thank you for the reminder.
I won't change this.  The CNAME list occasionally yields false positives, and that's unfortunate.

Apparently this is still a thing it also happen when you ask to reset the password


Not correct until newline at the end of output

Whenever I check my solution, I have to put a '\n' to the end of my output which can be hard to remember to do when in competition.

Fixed

I was hoping this could be opened again as now it also gives WA for trailing whitespace characters. However, it does give a warning about them.


Is there a sample app to automatically create a sitemap during deployment?

I believe generating a sitemap should be a standard feature of this great boilerplate. Is there a sample?

Good suggestion! I don't think we'll be adding a sitemap to this boilerplate natively, only because some people will use this bp for an app (which may not require a sitemap) and they take time to setup properly. I suggest integrating (sitemap.js)[https://github.com/ekalinin/sitemap.js] into your project.


Add support for binary operators like +,-,*,/ along with precedence rules

I started thinking about this last week, along with support for parenthesised expressions (do we have a ticket for those?)


node-sass No input file was found.

Does anyone have a touble using node-sass that is given in package.json?
I am new to nodejs and followed various guidelines but now stuck in building css from _scss.
When I ran one of the script that is in package.json in window command prompt,
node-sass --output-style compressed -o assets/css assets/_scss

it gives a following error.
No input file was found.

Could you please tell me anyone had the problem and made it work?
These are what I tried:

Installed different nodejs versions (i.e. v4.4.3 and v0.12.13) multiple times with different packages.
checked package dependancy that is written in package.json (from master branch)
tried with different node-sass syntax

What if you change line 32 in package.json to this?
"scss": "node-sass assets/_scss --output-style compressed -o assets/css",

I think I may have had the input and outputs reversed. It seems to work both ways for me on OS X and Windows.

C:\USERNAME.github.io>node-sass assets/_scss --output-style compressed -o assets/css
No input file was found.
C:\USERNAME.github.io>node -v
v0.12.13
C:\USERNAME.github.io>npm -v
2.15.0
C:\USERNAME.github.io>node-sass -v
node-sass       3.6.0   (Wrapper)       [JavaScript]
libsass         3.3.6   (Sass Compiler) [C/C++]
what version do you use?

Here's the versions on my Windows 7 machine. For Mac OSX they're about the same node v4.x and npm v2.x.
C:\Users\Michael\Documents\minimal-mistakes> node -v
v4.3.2

C:\Users\Michael\Documents\minimal-mistakes> npm -v
2.14.12

C:\Users\Michael\Documents\minimal-mistakes> node-sass -v
The term 'node-sass' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the s
pelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:10
+ node-sass <<<<  -v
    + CategoryInfo          : ObjectNotFound: (node-sass:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

C:\Users\Michael\Documents\minimal-mistakes> npm run build:css

><EMAIL_ADDRESS>build:css C:\Users\Michael\Documents\minimal-mistakes
> npm run scss && npm run autoprefixer

><EMAIL_ADDRESS>scss C:\Users\Michael\Documents\minimal-mistakes
> node-sass assets/_scss --output-style compressed -o assets/css

Rendering Complete, saving .css file...
Wrote CSS to C:\Users\Michael\Documents\minimal-mistakes\assets\css\main.css
Wrote 1 CSS files to C:\Users\Michael\Documents\minimal-mistakes\assets\css

><EMAIL_ADDRESS>autoprefixer C:\Users\Michael\Documents\minimal-mistakes\minimal-mistakes
> postcss -u autoprefixer -b ">, 5%, ie 9" -r assets/css/*

I synchronized those versions to yours but still did not work so that
I reported it to node-sass as well.
https://github.com/sass/node-sass/issues/1505
Wait and see their feedback.

Closing as this appears to be a node-sass issue. Please reopen if that's not the case.


RESTEASY-1684 - Arguments must not be null when sending a null JSON object with ResteasyWebTarget

Issue: https://issues.jboss.org/browse/JBEAP-11860
Upstream Issue: https://issues.jboss.org/browse/RESTEASY-1684
Upstream PR: https://github.com/resteasy/Resteasy/pull/1206
Backporting fix for RESTEASY-1684 to branch 3.0.19.SPX (thus for EAP 7.0.x)

OK this time it ssems to NOT be related to my changes. @asoldano of course if you feel it might be, let me know and I'll dig deeper.

I'm going to merge this. https://github.com/resteasy/Resteasy/pull/1365 has a test for for this change.


Is there any option to fix CVE-2017-12629 without upgrading the elastic search in my project

Issue description
I have elastic search 5.6.9 in my application which uses lucene 6.6.1 which has CVE-2017-12629 I could not upgrade elastic search to 6.x due to various reasons.
How can I solve the problem without upgrading the elastic search
We tried to use the lucene 6.6.2 jars instead of 6.6.1 in our elasticsearch dependencies.
But the elastic search server startup has failed with the version check and there are other dependencies on the lucene version in es code.
I need a way to solve this problem without upgrading the elastic search version
Steps to reproduce
Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.

Closing as this isn't really related to nixpkgs.


kitura: ensure .appsody folder exists in container

Checklist:

[x] Read the Code of Conduct and Contributing Guidelines.

[x] Followed the commit message guidelines.

[x] Stack adheres to Appsody stack structure.

Modifying an existing stack:

[x] Updated the stack version in stack.yaml

Copies /project/deps folder to /project/user-app/.appsody in the container if it doesn't exist in the user's project (because the init script failed)
Related Issues:
Fixes #610

Replacing myself as reviewer with @djones6 because he wrote this :)


fix: handled tos agreement page in splunk cloud

Update login.py file to handle TOS agreement page in Splunk cloud
Validation:
The below scenarios are validated with the ServiceNow addon v7.1.0.

Single Search head (8.2.2106):

When TOS agreement page is present:

Chrome: https://assets.saucelabs.com/jobs/231ecae43f9445db8afcd5f0801dd225/video.mp4
Edge: https://assets.saucelabs.com/jobs/4422961d255b4d77976795858d5a940c/video.mp4
Firefox: https://assets.saucelabs.com/jobs/77e36f5db9c8456da4a1018d5afcaec8/video.mp4
Safari: https://assets.saucelabs.com/jobs/544b4eeeafe14ca5a5c9bd02ca900f68/video.mp4

When TOS agreement page is not present:

Chrome: https://assets.saucelabs.com/jobs/76303b9d57d247778d75c537dff243f9/video.mp4
Edge: https://assets.saucelabs.com/jobs/65da2e20ae4b477c931574395d4b9608/video.mp4
Firefox: https://assets.saucelabs.com/jobs/62759fe664a04de5910a57a1936494d9/video.mp4
Safari: https://assets.saucelabs.com/jobs/57d0bf825fb84974a248635a6117864b/video.mp4

Search Head Cluster (8.2.2106):

When TOS agreement page is present:

Chrome: https://assets.saucelabs.com/jobs/65fb762764f04db6a66c2021e6c59e77/video.mp4
Safari: https://assets.saucelabs.com/jobs/f4a574da1fc9448690e24e7dc355b941/video.mp4
Firefox: https://assets.saucelabs.com/jobs/0677059d7aa848ac8d404cea160873fe/video.mp4
Edge: https://assets.saucelabs.com/jobs/2d026958d7fd416f9c57c14b37367a18/video.mp4

When TOS agreement page is not present:

Firefox: https://assets.saucelabs.com/jobs/775860b2f1874664bc65330654525cb5/video.mp4
Chrome: https://assets.saucelabs.com/jobs/536aeb748f0e4a5abb435e043ca5a4e8/video.mp4
Safari: https://assets.saucelabs.com/jobs/f40debb642824373a834271ad27f6f2f/video.mp4
Edge: https://assets.saucelabs.com/jobs/fff04757a1b54617adb0895be30d4ca3/video.mp4

So1 (8.2.1):

Chrome:  https://assets.saucelabs.com/jobs/615d73a5bf52471c989ced27e61136af/video.mp4
Edge: https://assets.saucelabs.com/jobs/003f665fc9d74ee3bab19431a615002c/video.mp4
Firefox: https://assets.saucelabs.com/jobs/edd987d2be744c9781bb1eaad4bbaa17/video.mp4
Safari: https://assets.saucelabs.com/jobs/62da2aee7bfe4365ba93b23b8bbb814f/video.mp4

I have read the CLA Document and I hereby sign the CLA

:tada: This PR is included in version 2.0.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add mapping for GCP virtual machine sizes

We only have mapping for general purpose compute, we're missing compute optimized and memory optimized for GCP.
Mapping is defined in https://github.com/multycloud/multy/blob/main/resources/common/vm_size.go

Hi, want to work on this next

Perfect, I'll assign it to you!


Incorrect documentation for fixing memory leaks

The "isMounted is an Antipattern" documentation article suggest avoiding memory leaks with makeCancelable, but this doesn't actually solve the issue since the callback is still referenced. The garbage collector does not understand that setting hasCanceled_ means that resolve will never be called, so it cannot release resolve or the references contained within. Even if it could, reject might also reference this, so memory leaks could still exist. Instead, makeCancelable should store the resolve and reject callbacks itself, and it should null them out on cancelation.
Another user pointed this issue out in https://github.com/facebook/react/issues/5465#issuecomment-287113777

There are two pretty reasonable approaches I can think of for modifying this. Thoughts?

Instead of capturing the component directly in the promise, capture a "handle" that stores the component as a property. When the component is unmounted, null out this property to break the reference.

const componentHandle = new Handle(component);
promise.then(() => {
  if (!componentHandle.isReleased()) {
    const component = componentHandle.value();
  }
})

Register all callbacks with the component, wrapping them in functions that call the stored implementation. On unmount, cancel all registered callbacks, which deletes the functions stored within these wrappers.

promise.then(component.registeredCallback(() => {
  // directly reference component
}))

see https://github.com/hjylewis/trashable-react

Dont you just hide the problem like this?

i am attempting to avoid the "Can only update a mounted or mounting component.." warning, believing that React is able to effectively garbage collect if i'm not updating/setting state on an unmounted component.
In the example that i provided, i tried to do the this._isMounted check inside the constructor, but that still does NOT prevent the warning from showing up.


Idea: Create the d.ts Typescript type definitions

Being the best touch-joystick lib available on the market you could maybe publish on DefinitelyTyped the d.ts definitions to support Typescript.
This will help both the users of your library and you by promoting it  :)
Just an idea....

Thanks :relaxed:.
Do you have some hints on this? Or even create a PR for it?
I don't know/use TypeScript at all :disappointed:.

Sure, look at https://github.com/DefinitelyTyped/DefinitelyTyped
That's the biggest repository for type definitions. All well-known libraries are having the definitions there.

Thanks,
I'll look into it once I get some free time...
In the mean time, if you want to do it, you could submit a PR.

This is far from perfect, I will eventually get around to a proper definition but for anyone today who cannot be bothered, this does the job.  Note that NippleInteractiveData interface is what occurs on mouse move.  Such as:
sample
nipple.on("move", this.onNippleMove);
private onNippleMove = (evt: EventData, data: NippleInteractiveData): void => {
    console.log(data.angle.radians);
 }

nipple.d.ts
declare interface ManagerOptions {

    zone?: HTMLElement;
    color?: string;
    size?: number;
    threshold?: number;
    fadeTime?: number;
    multitouch?: boolean;
    maxNumberOfNipples?: number;
    dataOnly?: boolean;
    position?: any;
    mode?: string;
    restOpacity?: number;
    catchDistance?: MimeType

}

declare interface NippleInteractiveData {

    angle: {
        degree: number;
        radian: number;
    },
    direction: {
        angle: string;
        x: string;
        y: string;
    }
    distance: number;
    force: number;
    identifier: number;
    instance: Nipple;
    position: {
        x: number;
        y: number;
    };
    pressure: number;

}

declare interface EventData {
    type: string;
    target: any;
}

declare class Manager {

    static create(options?: ManagerOptions): Manager;

    on(type: string | string[], handler: (evt: EventData, data: any) => void): void;
    off(type: string | string[], handler: (evt: EventData, data: any) => void): void;
    get(identifier: number): Nipple;
    destroy(): void;

    }

declare interface Nipple {

    on(type: string | string[], handler: (evt: EventData, data: any) => void): void;
    off(type: string | string[], handler: (evt: EventData, data: any) => void): void;
    el: HTMLElement;
    show(cb?: () => void): void;
    hide(cb?: () => void): void;
    add(): void;
    remove(): void;
    destroy(): void;
    identifier: number;
    trigger(type: string | string[], handler: (evt: EventData, data: any) => void): void;
    position: {
        x: number;
        y: number;
    };
    frontPosition: {
        x: number;
        y: number;
    };
    ui: {
        el: HTMLElement;
        front: HTMLElement;
        back: HTMLElement;
    };
    options: {
        color: string;
        size: number;
        threshold: number;
        fadeTime: number;
    }

}

declare module "nipplejs" {
    export default Manager;
}

Thanks a lot @clark-stevenson for this.
I might need to investigate a little more on the subject before being able to correctly reviewing it.
Can you create a new PR for this though?
@ThumbGen can you help me reviewing this alos?

Looks good (at least for my small project :+1:  )! Thank you, @clark-stevenson!

Thanks @clark-stevenson for his answer.
Some steps to use it:
0. npm install nipplejs --save

Create a file called index.d.ts inside ../node_modules/nipplejs folder

Copy @clark-stevenson 's code and paste to that index.d.ts file

Import where you have to use it: import Manager from 'nipplejs';

Use like: var joystick = Manager.create({
zone: document.getElementById('zone_joystick'),
mode: 'static',
position: { left: '50%', top: '50%' },
color: 'red'
});
You can use this form for a static joystick or you can use one of the other modes. (check this link: http://yoannmoinet.github.io/nipplejs/#demo)

Any update on this?
Just tried this on Angular 5.1.2, and nipplejs 0.6.8 but after import Manager from 'nipplejs'; and console log to Manager its says undefined

@NarinLab
Do you mind if I ask you where you import this?
This is a generic question of Angular (sorry @yoannmoinet ), where do you put your import? Per file? Per  Module? I am struggling with this and I heard that, if you are not careful, all imports duplicate.  So if I import something into my-crazy.module.ts then import into my-crazy.component.ts I get multiple duplications of the same library?
Not sure.

Hi @clark-stevenson
I put the import statement on pan-controller.component.ts
Like this:
import { Component, OnInit, AfterViewInit } from '@angular/core';
/** THE MAIN SOLUTION **/
import * as nipplejs from 'nipplejs';
/** THE MAIN SOLUTION **/
import { Subscription } from 'rxjs/Subscription';
import { ServerSocketService } from '../server-socket.service';

@Component({
  selector: 'app-pan-controller',
  templateUrl: './pan-controller.component.html',
  styleUrls: ['./pan-controller.component.css']
})

export class PanControllerComponent implements OnInit {
private controller_left = null;
  private controller_right = null;
  progress_pwm = 0;
  public progress_mode = "determinate";
  private options = {
      zone: null,                  // active zone
      color: '#00bcd4',
      size: 150,
      threshold: 0.1,               // before triggering a directional event
      fadeTime: 500,              // transition time
      multitouch: false,
      maxNumberOfNipples: 1,     // when multitouch, what is too many?
      dataOnly: false,              // no dom element whatsoever
      position: {top: '140px', right: '180px'},               // preset position for 'static' mode
      mode: 'static',                   // 'dynamic', 'static' or 'semi'
      restOpacity: 0.5,            // opacity when not 'dynamic' and rested
      catchDistance: 200           // distance to recycle previous joystick in 'semi' mode
  };
  private socketSubscription: Subscription;
  constructor(private socket: ServerSocketService) { }
  ngAfterViewInit() {
    // Component views are initialized
    this.options.zone = document.getElementById('pan-controller-right');
    this.controller_right = nipplejs.create(this.options);;
    this.controller_right.on('move', this.controllerRightOnMove.bind(this));
    this.controller_right.on('end', this.controllerRightOnEnd.bind(this));
  }

 //and so on....

Hope this help

Maybe I'll start with this
https://github.com/Microsoft/dts-gen and see what I get...

Hi @yoannmoinet.
Very fine work on NippleJS and I appreciate you making it available to all.
But, how can it be integrated into an AngularJS project? Are there prescribed steps a beginner in Angular and NippleJS can follow?
I provided the following in app.module.ts: import { JoystickManager } from 'nipplejs';
imports: [
BrowserModule,
AppRoutingModule,
ReactiveFormsModule,
JoystickManager
],
... which resulted in the following error:
ERROR in node_modules/nipplejs/types/index.d.ts:277:14 - error NG6002: Appears in the NgModule.imports of AppModule, but could not be resolved to an NgModule class.
This likely means that the library (nipplejs) which declares JoystickManager has not been processed correctly by ngcc, or is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.
Is there an Ivy compatibility issue ?
I am using<EMAIL_ADDRESS>with Angular 8.

@captsisko I don't think this is an Angular library.
It doesn't belong in the imports or modules, but directly in your components.
You did the right thing, bring it into your individual component where you want to use it.
import { JoystickManager } from 'nipplejs';

afterViewInit(){
    // use JoystickManager
}

onDestroy(){
   // anything you do, be sure to destroy it.
}

I appreciate the input, Clark.
You are right. NippleJS is not built for Angular. I have found a wrapper
for nipple called ngx-joystick which does the trick.
On Sat, 25 Jul 2020 at 21:30, Clark Stevenson<EMAIL_ADDRESS>wrote:

@captsisko https://github.com/captsisko I don't think this is an
Angular library.
It doesn't belong in the imports or modules, but directly in your
components.
You did the right thing, bring it into your individual component where you
want to use it.
import { JoystickManager } from 'nipplejs';
afterViewInit(){
// use JoystickManager
}
onDestroy(){
// anything you do, be sure to destroy it.
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yoannmoinet/nipplejs/issues/21#issuecomment-663902976,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA3BD4ARSQGES3DU2TVH3TDR5M6ETANCNFSM4BVGAKJQ
.

--
Jesus loves you too!


Support For * Route Methods

If there is a route that has * defined for the method, the UI just displays a broken link for route. If a * method is encountered It should generate a rout / section for  GET, PUT, POST, DELETE, and PATCH (?)

Hi I have now added support for for arrays and * wildcards in the  method  property of a  route  i.e.
You can how define * wildcards for  method
server.route({
   method: '*',
   path: '/test',
   handler: defaultHandler
});

This will add GET, PUT, POST, DELETE, and PATCH. It will not support  OPTION or HEAD. OPTION is not supported by Swagger v1.2 and HEAD is not support by HAPI
You can also how define an array of supported methods
server.route({
   method: ['GET', 'POST'],
   path: '/test',
   handler: defaultHandler
});

Any problems reopen the issue


is there any way i can get uses-permissions?

hi，thx for your package.It`s really helpful , but i still have a problem,is there any way i can get uses-permissions of an apk file？

You can get the permissions from Application.Permission.
https://github.com/shogo82148/androidbinary/blob/c5189dc5c59d9d423308e0475d7d72fa2f1687b9/apk/apkxml.go#L71
If it doesn't work, please send me your apk or android manifest.

You can get the permissions from Application.Permission.
https://github.com/shogo82148/androidbinary/blob/c5189dc5c59d9d423308e0475d7d72fa2f1687b9/apk/apkxml.go#L71
If it doesn't work, please send me your apk or android manifest.

thx for your reply,but what i need is 'uses-permission'，and i already fork your repository and find a way out ,thank you

Hi @shogo82148 , I create a pull request #46 for support the uses-permission. If you have time, please help me review it. Hope it could be merged.

closed by #46
thanks @tonywang !


[docs] Add repository/homepage to public package.jsons

Some of these fields were missing or out of date. This will make it easier to reach the source code and relevant readme from each package's respective npm page

@mike-north Thanks!


23Q3 FacDB Update

FacDB Source Data Updates
Like most of our data products, source data must be updated in data library before FacDB is run. As there are are many source datasets with varied update processes, this issue template should be opened to track progress towards updating all source data
All source data listed is to be uploaded as .csv files
Geosupport

[x] Update geosupport version in both Dockerfiles

Version Env

[x] Update previous version for QCQA in version.env file

Scraped by data library

[x] bpl_libraries
Source: Scraped from BPL website
Source url: https://www.bklynlibrary.org/locations/json

[x] nypl_libraries
Source: Scrape from NYPL website
Source url: https://www.nypl.org/locations/list

[x] uscourts_courts
Source: Court locator for NY state
Source url: http://www.uscourts.gov/court-locator/city/New York/state/NY

Source data from OpenData
To see if a dataset needs to be uploaded, check date last updated in open data against version in data library

[x] dca_operatingbusinesses https://data.cityofnewyork.us/Business/Legally-Operating-Businesses/w7w3-xahh

[x] dcp_colp https://www1.nyc.gov/site/planning/data-maps/open-data.page#city_facilities

[x] dcp_facilities_with_unmapped https://www1.nyc.gov/site/planning/data-maps/open-data/dwn-selfac.page

[x] dcla_culturalinstitutions https://data.cityofnewyork.us/Recreation/DCLA-Cultural-Organizations/u35m-9t32

[x] dfta_contracts https://data.cityofnewyork.us/Social-Services/DFTA-Contracts/6j6t-3ixh

[x] doe_busroutesgarages https://data.cityofnewyork.us/Transportation/Routes/8yac-vygm

[x] sca_enrollment_capacity https://data.cityofnewyork.us/Education/Enrollment-Capacity-And-Utilization-Reports-Target/8b9a-pywy

[x] dohmh_daycare https://data.cityofnewyork.us/Health/DOHMH-Childcare-Center-Inspections/dsg6-ifza

[x] dpr_parksproperties https://nycopendata.socrata.com/Recreation/Parks-Properties/enfh-gkve
NOTE: DPR open data table URLs are not consistent. Be sure to double-check before running from the recipes app.

[x] dsny_garages https://data.cityofnewyork.us/Environment/DSNY-Garages/xw3j-2yxf

[x] dsny_specialwastedrop https://data.cityofnewyork.us/Environment/DSNY-Special-Waste-Drop-off-Sites/242c-ru4i

[x] dsny_donatenycdirectory https://data.cityofnewyork.us/Environment/DSNY-DonateNYC-Directory/gkgs-za6m

[x] dsny_leafdrop https://data.cityofnewyork.us/Environment/Leaf-Drop-Off-Locations-in-NYC/8i9k-4gi5

[x] dsny_fooddrop https://data.cityofnewyork.us/Environment/Food-Scrap-Drop-Off-Locations-in-NYC/if26-z6xq

[x] dsny_electronicsdrop https://data.cityofnewyork.us/Environment/Electronics-Drop-Off-Locations-in-NYC/wshr-5vic

[x] dycd_afterschoolprograms https://data.cityofnewyork.us/Education/DYCD-after-school-programs/mbd7-jfnc

[x] fdny_firehouses https://data.cityofnewyork.us/Public-Safety/FDNY-Firehouse-Listing/hc8x-tcnd

[x] hhc_hospitals https://data.cityofnewyork.us/Health/Health-and-Hospitals-Corporation-HHC-Facilities/f7b6-v6v3

[x] hra_jobcenters https://data.cityofnewyork.us/Business/Directory-Of-Job-Centers/9d9t-bmk7

[x] hra_medicaid https://data.cityofnewyork.us/City-Government/Medicaid-Offices/ibs4-k445

[x] hra_snapcenters https://data.cityofnewyork.us/Social-Services/Directory-of-SNAP-Centers/tc6u-8rnp

[x] moeo_socialservicesitelocations https://data.cityofnewyork.us/City-Government/Verified-Locations-for-NYC-City-Funded-Social-Serv/2bvn-ky2h

[x] nycha_communitycenters https://data.cityofnewyork.us/Social-Services/Directory-of-NYCHA-Community-Facilities/crns-fw6u

[x] nycha_policeservice https://data.cityofnewyork.us/Housing-Development/NYCHA-PSA-Police-Service-Areas-/72wx-vdjr

[x] nysdec_solidwaste https://data.ny.gov/Energy-Environment/Solid-Waste-Management-Facilities/2fni-raj8

[x] nysdoh_healthfacilities https://health.data.ny.gov/Health/Health-Facility-General-Information/vn5v-hh5r

[x] nysdoh_nursinghomes https://health.data.ny.gov/Health/Nursing-Home-Weekly-Bed-Census-Last-Submission/izta-vnpq

[x] nysomh_mentalhealth https://data.ny.gov/Human-Services/Local-Mental-Health-Programs/6nvr-tbv8

[x] nysopwdd_providers https://data.ny.gov/Human-Services/Directory-of-Developmental-Disabilities-Service-Pr/ieqx-cqyk

[x] nysparks_historicplaces https://data.ny.gov/Recreation/National-Register-of-Historic-Places/iisn-hnyv

[x] nysparks_parks https://data.ny.gov/Recreation/State-Park-Facility-Points/9uuk-x7vh

[x]  qpl_libraries https://data.cityofnewyork.us/Education/Queens-Library-Branches/kh3d-xhq7

[x] sbs_workforce1 https://data.cityofnewyork.us/dataset/Center-Service-Locations/6smc-7mk6

[x] usdot_airports https://hub.arcgis.com/datasets/usdot::airports
Head to url >> api >> copy url from geojson

[x] usdot_ports https://hub.arcgis.com/datasets/usdot::ports
Head to url >> api >> copy url from geojson

[x] nysdec_lands http://gis.ny.gov/gisdata/inventories/details.cfm?DSID=1114

Manually check data for updates
These don't report date updated as neatly as the open datasets, have to look at data itself

[x] fbop_corrections https://www.bop.gov/locations/list.jsp
When searching by state, there should be 5 NY prisons, 3 of which are in NYC (Brooklyn/New York)

[x] nycdoc_corrections  https://www1.nyc.gov/site/doc/about/facilities-locations.page
Source: NYCDOC locations directory

[x] nycourts_courts http://www.nycourts.gov/courts/nyc/criminal/generalinfo.shtml#BRONX_COUNTY

[x] nysdoccs_corrections https://doccs.ny.gov/find-facility
Hand check for 1 facility in queens, 1 facility in Manhattan, 0 in the other 3 boros. Only look at the correctional facility locations, not the offices.

Manual download

[x] doe_lcgms https://data.cityofnewyork.us/Education/LCGMS-DOE-School-Information-Report/3bkj-34v2
This dataset is updated for CEQR

[x] foodbankny_foodbanks http://www.foodbanknyc.org/get-help/
1. head to http://www.foodbanknyc.org/get-help/
2. navigate to the map and make a copy of the map
3. After making a copy, click on the three dots next to the target layer and click "Export Data" and export as a csv
4. Rename the file (still as a csv) to match Food_Bank_For_NYC_Open_Members_as_of_DATE(YYYYMMDD). You will need to convert the existing date format MMDDYY to YYYYMMDD so that the version matches existing date format standard in data library.
5. place it at the library/tmp folder
6. then run library archive --name foodbankny_foodbanks with the -version flag set to the DATE in the file path
url: "http://www.foodbanknyc.org/get-help/"
dependents: []
5. place it at the library/tmp folder
6. then run library archive --name foodbankny_foodbanks with the -version flag set to the DATE in the file path

[x] nysed_activeinstitutions  https://eservices.nysed.gov/sedreports/list?id=1
Active Institutions with GIS coordinates and OITS Accuracy Code - Select by County__ CSV. Note that .csv data is automatically downloaded without comma delimiter. Exporting to csv from numbers is one way to get around this issue

[x] nysed_nonpublicenrollment http://www.p12.nysed.gov/irs/statistics/nonpublic/
Nonpublic Enrollment by Grade

[x] nysoasas_programs  https://webapps.oasas.ny.gov/providerDirectory/index.cfm?search_type=2
Download all treatment providers
Modify download URL to contain today’s date: https://webapps.oasas.ny.gov/providerDirectory/download/Treatment_Providers_OASAS_Directory_Search_13-Nov-20.csv

[x] usnps_parks https://irma.nps.gov/DataStore/Reference/Profile/2225713
NOTE: the final number in the URL (2225713) is not always stable. If the data is missing, search through the home.

Will receive via email or FTP

[x] dot_bridgehouses
[x] dot_ferryterminals
[x] dot_mannedfacilities
[x] dot_publicparking
[x] dot_pedplazas

Unresolved process
Still waiting to figure out best way to upload these data

[x] doe_universalprek
Source url: https://maps.nyc.gov/prek/data/pka/pka.csv

Last step

[x] dcp_pops
Source: Download from POPs app, available on DCP Commons. Be sure to only take the public version.
Be sure to do this source last, as the OpenData release of POPs needs to be in sync

closing per passing QA by GIS team and starting to publish


ahoy.js does not track click events for a-tags that contain HTML tags. Is it intentional?

As far as I investigate, there are differences in behavior as follows. I want to track click events in all a-tags, is there a way to do that?
# Click events are tracked
click

# Click events are not tracked
click

Hey @ts-3156, this was merged an hour ago if you use master: https://github.com/ankane/ahoy.js/pull/54.

Thanks you your answer!

@ankane When do you plan to release the new version? I can't make up my mind if I should use the master branch or wait for a new release.


chore: release 3.0.0

:robot: I have created a release *beep* *boop*
3.0.0 (2022-06-14)
⚠ BREAKING CHANGES

never support env setting

Code Refactoring

replace mustache with handlebars (a1b381b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/keq-request/keq-cli/releases/tag/v3.0.0 :sunflower:


[NEW] Add rate the app in playstore feature to setting screen

Description
There must be a feature to rate the app on play store inside the setting screen
This feature will surely improve the app ratings at playstore as many user having good experience with the app are too lazy to rate app manually.
Devices and Versions
Your Rocket.Chat.Android version: 3.1.0 Build 2049 - 60b39dad-play
Would you like to work on the issue?

Yes

i fully support this feature.

Sounds good to me! 👍

nice work guys


script repeating output of command run with proc to stdout

i've just started using turtle and i've made this script to count the number of lines in an ebook:
#!/usr/bin/env stack
-- stack script --resolver=lts-16.27 --package turtle --package protolude --package text
{-# LANGUAGE OverloadedStrings, NoImplicitPrelude, TypeApplications, LambdaCase #-}

module Main where

import Protolude hiding (FilePath)
import Data.Maybe (fromJust)
import Turtle

tempFileLocation = "/tmp/pdf_counter_tmp.pdf"

checkPrograms :: Shell ()
checkPrograms = do
    let programs :: IsString s => [s]
        programs = ["ebook-convert", "pdfinfo"]
    locations <- traverse which programs
    when (any isNothing locations) $ do
        print $ "you need to have all these programs on your PATH: " ++ show (programs :: [Text])
        exit (ExitFailure 1)

isSuccess ExitSuccess = True
isSuccess (ExitFailure _) = False

textPath path = toText path & \case
    Left t -> do
        case textToLine t of
            Just t -> echo ("invalid filepath: " <> t)
            Nothing -> echo "filepath contains a newline in it"
        return t
    Right t -> return t

convertFile :: FilePath -> Shell Bool
convertFile path = do
    pathT <- textPath path
    exitCode <-
        if hasExtension path "pdf"
            then proc "cp" [pathT, tempFileLocation] empty
            else proc "ebook-convert" [pathT, tempFileLocation, "--paper-size", "letter"] empty
    return $ isSuccess exitCode

pagesPattern :: Pattern Integer
pagesPattern = prefix "Pages:" *> space *> decimal

countPages :: FilePath -> Shell Integer
countPages path = do
    conversionSuccessful <- convertFile path
    guard conversionSuccessful
    info <- inproc "pdfinfo" [tempFileLocation] empty
    select $ match pagesPattern $ lineToText info

countDirectory :: FilePath -> Shell (Integer, FilePath)
countDirectory dir = do
    file <- lsif (fmap isRegularFile . stat) dir
    count <- countPages file
    return (count, fromJust $ stripPrefix dir file)

main :: IO ()
main = do
    sh checkPrograms
    path <- getArgs >>= \case
            [path] ->
                return $ decodeString path
            _ -> do
                putStrLn ("this program expects exactly one argument of a file or directory" :: Text) :: IO ()
                exitFailure
    isDir <- isDirectory <$> stat path
    if isDir
        then view $ countDirectory path
        else view $ countPages path

i would expect this, when run on a file, to simply print the number of lines in that file
instead, when run on a non-pdf file it returns that and also prints the entire ebook-convert output:
colby@desktop ~/c/p/src> ./Main.hs ~/Documents/books/short/anonymous-desert.epub
1% Converting input to HTML...
InputFormatPlugin: EPUB Input running
on /home/colby/Documents/books/short/anonymous-desert.epub
Parsing all content...
34% Running transforms on e-book...
Merging user specified metadata...
Detecting structure...
Flattening CSS and remapping font sizes...
Source base font size is 12.00000pt
Removing fake margins...
Cleaning up manifest...
Trimming unused files from manifest...
Creating PDF Output...
67% Running PDF Output plugin
68% Parsed all content for markup transformation
70% Completed markup transformation
90% Rendered all HTML as PDF
91% Added links to PDF content
100% Updated metadata in PDF
PDF output written to /tmp/pdf_counter_tmp.pdf
Output saved to   /tmp/pdf_counter_tmp.pdf
111

it's possible that this is just some weird ebook-convert behavior, i haven't really looked into it all that much

a more minimal version of the script:
#!/usr/bin/env stack
-- stack script --resolver=lts-16.27 --package turtle --package protolude --package text
{-# LANGUAGE OverloadedStrings, NoImplicitPrelude, LambdaCase #-}

module Main where

import Protolude hiding (FilePath)
import Turtle

textPath path = toText path & \case
    Left t -> do
        case textToLine t of
            Just t -> echo ("invalid filepath: " <> t)
            Nothing -> echo "filepath contains a newline in it"
        return t
    Right t -> return t

main :: IO ()
main = do
    [file] <- map decodeString <$> getArgs
    fileT <- textPath file
    view (proc "ebook-convert" [fileT, "/tmp/pdf_counter_tmp.pdf", "--paper-size", "letter"] empty)

a more minimal version of the script:
#!/usr/bin/env stack
-- stack script --resolver=lts-16.27 --package turtle --package protolude --package text
{-# LANGUAGE OverloadedStrings, NoImplicitPrelude, LambdaCase #-}

module Main where

import Protolude hiding (FilePath)
import Turtle

textPath path = toText path & \case
    Left t -> do
        case textToLine t of
            Just t -> echo ("invalid filepath: " <> t)
            Nothing -> echo "filepath contains a newline in it"
        return t
    Right t -> return t

main :: IO ()
main = do
    [file] <- map decodeString <$> getArgs
    fileT <- textPath file
    view (proc "ebook-convert" [fileT, "/tmp/pdf_counter_tmp.pdf", "--paper-size", "letter"] empty)

@ikea-shark-official: inproc does not capture stderr, so it will forward anything that the subprocess emits to stderr to the console.  If you use inprocWithErr then it will capture both stdout and stderr.

@ikea-shark-official: inproc does not capture stderr, so it will forward anything that the subprocess emits to stderr to the console.  If you use inprocWithErr then it will capture both stdout and stderr.


Add loader after clicking on login and signup in react

The login and signup page does not have a loader after clicking on login button.
Need to add loader until a response is not shown.

I can work on this issue

Okay


Help?

"I referred to your code and have some questions.
How do I generate the parameters X-Apple-I-MD, X-Apple-I-MD-M, and X-Apple-I-MD-RINFO?"

X-Apple-I-MD:
base64 string encoding 28 bytes, 'one-time password', not sure how to generate this.
X-Apple-I-MD-M:
base64 string encoding 60 bytes, 'machine id', not sure how this is generated either.
X-Apple-I-MD-RINFO:
An integer representing 'router info'. From the source:
A number, either 17106176 or 50660608.

It doesn't seem to matter which one we use.
- 17106176 is used by Sideloadly and Provision (android) based servers.
- 50660608 is used by Windows iCloud based servers.

FindMy.py does not generate these values itself, but uses an external Anisette server to retrieve them. I'm still looking into using local options for this, by utilizing e.g. pyprovision.

Closing due to inactivity, feel free to reopen if your question has not been answered.


! # ^ { } and + will not trigger match with \W

I use:
RegExHotstring("psa(s)?(\W)", "Please see the attached document$1$2", "*C")
! # ^ { } and + will not trigger match when they are the terminal characters. Practically speaking , the only one likely to cause me issue in the day to day is the !.

use SendText with new option T
RegExHotstring("psa(s)?(\W)", "Please see the attached document$1$2", "*CT")


Interactive-dashboard + Voilà

Hi there!
This work is amazing, and I loved you demo at the meeting. Voilà co-author here. I would love to get in touch with you on integrating this as a Voilà template (like e.g. voilà-gridstack). Let me know if you would like to be in touch to discuss how we could collaborate on this!

Jumping in here quickly.
If I got it right, the extension is outstanding for creating layouts on the fly.
What about stand alone deployment? If integration with Voila could work, this would become so much more impacting!
Thanks

Adding here some notes about what has to be done to make the extension compatible with Voià.

[ ] Add a template.
[ ] Have the layout info in the notebook metadata.

Adding here some notes about what has to be done to make the extension compatible with Voià.

[ ] Add a template.
[ ] Have the layout info in the notebook metadata.


Fix callback if migrations fails

To  prevent failed migration, when you insert this gem in existing project and you want to run:
rake db:create && rake db:migrate

Migrations fail because of uid and provider attributes that do not exist yet in the first migrations (past migrations).

Can you explain your change?
I edited my first comment.

Someone wants to merge this PR ?


kie-issues#1298: Decision Services & multiple DRDs: make it possible o add external Decisions to a Decision Service

Closes: https://github.com/apache/incubator-kie-issues/issues/1298
https://github.com/user-attachments/assets/2f14961c-28ea-4d38-a5c9-98f8580c9026

More testes for you, @ljmotta 😄

I will review this tomorrow.

@tiagobento @jomarko Changes applied! Ready for you to continue.

I think Prettier doesn't change the imports... Maybe it's your VS Code "Organize imports"?

It could be. I didn't notice that change in others PR that I did. I'll check it.

Waiting for you @jomarko when you can re-check it. Thank you! 😄

Thank you, @jomarko ! Good points here.
Yep, this is an issue, but it seems to be a runner issue. It only works if the included decision does not have an input node.
Check this. The mmmmm.Pi is an included node and it works fine.
The mmm.Sum is an included node and it fails inside the DS OR outside it.
https://github.com/user-attachments/assets/75f9ad33-1fad-4798-9fd5-b7a4c7d8b26c

Good point about the invocation string @jomarko!

I'm checking if the error about the execution that is not related to this PR is easy to fix. If it is easy, I think we should include it in this PR because it's part of the feature.

https://github.com/user-attachments/assets/58bfce14-b10a-4af2-bd00-dd6652016abe
@tiagobento The test.

The issue found: https://github.com/apache/incubator-kie-issues/issues/1435

https://github.com/user-attachments/assets/58bfce14-b10a-4af2-bd00-dd6652016abe
@tiagobento The test.

Thanks! Yeah, moving a Decision around while keeping it inside a DS is indeed not expected to be working yet... See

https://github.com/apache/incubator-kie-issues/issues/889

About the invocation string, should we handle it in this PR or in another one? It looks like the issue is in the repopulateInputDataAndDecisionsOnAllDecisionServices mutation. @tiagobento

Yes. The issue about decision service string is fixed. The runner issue is in another ticket, @jomarko
It's up to @tiagobento now.

@danielzhe Still reviewing, but please make sure you don't introduce ESLint warnings of missing hook dependencies.

I'll check it. Thank you, @tiagobento

Interesting issue you found, @ljmotta ! I'm working on that.

As we discussed yesterday:

Please incorporate else branches and throw errors on exceptional cases when repopulating the inputDecisions and inputData of Decision Services.
Change the code to accommodate for the fact that ir.requiredDecision["@_href"] and ir.requiredInput["@_href"] may already contain a namespace, thus prefixing it with the external model's namespace may lead to invalid hrefs.

I've found another related to the test that @ljmotta did:

Insert a local Decision Service with an "Included Decision-X"
Insert an included Decision Service that encapsulated the "Included Decision-X"
You'll get the same odd behavior that Luiz found.

https://github.com/user-attachments/assets/e80023f2-3205-473c-bb90-9ad1820ebe33
The both issues are related and it starts in the [computeContainingDecisionServiceHrefsByDecisionHrefs](https://github.com/apache/incubator-kie-tools/blob/main/packages/dmn-editor/src/store/computed/computeContainingDecisionServiceHrefsByDecisionHrefs.ts.ts)
When we insert an external Decision Service, the decisionServiceHrefsByDecisionHrefs we return from computeContainingDecisionServiceHrefsByDecisionHrefs the map is built with the wrong href for local nodes: we're adding the external namespace URI for local nodes, when we shouldn't.

I need a re-review here, @jomarko @ljmotta for this new feature.

This should fix the issue @ljmotta found here

It also fixes the same issue, that I reported here

It fixes an issue with 3rd models not included in the current model. For example:
Local DS -> Decision in Model A -> Input in Model B.

Consider that user only added the "Model A" in the local (the current) model. Now we handle this in this way:
https://github.com/user-attachments/assets/47fc08c6-e56e-46b7-b387-393977df9b2a

Did another round of review. Thanks @danielzhe!
Can you please check if your IDE is correctly displaying ESLint warnings? There are too many hook dependencies missing, and they're really important.

Thank you! I think I lost some of those in the merge. I got some conflicts, because before it was zero hooks errors, but I'm checking again.

@tiagobento Changes applied!

Sorry, I get lost in the conversation, since my approval it seems more code landed in. Should I do a re-reiew?

I've made some comments inline. One tip regarding objects. When the object property and value has the same name you could short it just the property name:

I'm aware of this. If I didn't do it probably I just overlooked it or it was something that was renamed.  😄  @ljmotta
I'm checking the other things you reported. Thank you.

I found some interesting cases doing some manual checkings:

We have here three DRDs, in DRD3 we can observe that the ds2 is in collapsed form, and d3 isn't available to pick from the node palette, as the node was depict.

Dragging a node near the collapse DS will trigger the invalid operation in the DS.

It's possible to hide external Decisions that are part of a Decision Service. Adding them back to the DRD will not remove them from the DS, resulting some weird behaviors

https://github.com/user-attachments/assets/7050aaf7-d263-4e1f-ad2d-aa0182f1c1a1

A DRD that has Decision nodes that are part of a collapsed Decision Service can't be moved:

https://github.com/user-attachments/assets/40de991e-55dc-4268-968c-6ebfbb9de4b0

@ljmotta Can you please discriminate between what's already on main and what's been introduced by this PR? Some of those bugs (E.g., Dragging a node near the collapse DS will trigger the invalid operation in the DS.) you reported seem not to be related to the changes this PR introduces.

@danielzhe I would double-check those too, just to be sure.

https://github.com/user-attachments/assets/a79345b9-aac5-4f9f-8ad5-1f1938d55ab3

https://github.com/user-attachments/assets/09eebb41-595b-4f46-adc3-be793730698f


telegram_bot: verify_ssl configuration doesn't work

Home Assistant release with the issue: 0.91.3
Operating environment (Hass.io/Docker/Windows/etc.): Docker
Component/platform: telegram_bot
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
automation:
- alias: Telegram /photo command
  hide_entity: true
  trigger:
    platform: event
    event_type: telegram_command
    event_data:
      command: /photo
  action:
  - service: telegram_bot.send_photo
    data_template:
      target: '{{ trigger.event.data.user_id }}'
      url: !secret hall_cam_url
      verify_ssl: False
      username: !secret xiaomi_camera_user
      password: !secret xiaomi_camera_pass
      authentication: basic
      caption: Hall camera
~

Linked issue: https://github.com/home-assistant/home-assistant/issues/22067
Traceback (if applicable):
2019-04-16 14:41:48 ERROR (SyncWorker_2) [homeassistant.components.telegram_bot] Can't load data into ByteIO: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /cgi-bin/currentpic.cgi (Caused by SSLError(SSLError("bad handshake: Error([('SSL routines', 'tls_process_server_certificate', 'certificate verify failed')])")))
2019-04-16 14:41:48 ERROR (SyncWorker_2) [homeassistant.components.telegram_bot] Can't send file with kwargs: {'url': 'https://<IP_ADDRESS>/cgi-bin/currentpic.cgi', 'verify_ssl': False, 'username': '***', 'password': '***', 'authentication': 'basic', 'caption': 'Hall camera'}

Additional information:
Changing the condition here fixes bug: https://github.com/home-assistant/home-assistant/blob/51508d69ad91c0bca1a7c326841ccf95b8f03fa7/homeassistant/components/telegram_bot/init.py#L197
From if verify_ssl: to if verify_ssl is not None:
@cgtobi please comment, it was your review https://github.com/home-assistant/home-assistant/pull/22069#pullrequestreview-214981737

This should be fixed. My review was wrong as stated in the followup comment. I'll submit a PR in a moment.

Thanks for reporting.


add more database parameters

Fixes #24

Note that I've tested this with Cloud SQL and Cloud Run and verified that it all works.


[flutter_tools] warn when main library is relocated

Description
Fixes https://github.com/flutter/flutter/issues/62929
If the main entrypoint file is moved/renamed, then changes to it won't be picked up. Since there is no way to signal to the incremental compiler (that I am aware of) that this is changed, warn if detected to prevent users from getting confused when subsequent changes are not surfaced.

(This is a non-fatal error because putting the file back will "fix" it)


Remove Supporters page

This seems redundant now and hard to maintain in any case.

concept ACK. Will review tomorrow.

Concept ACK.
Tested LGTM except that there's some leftover CSS specific to the removed page starting at line 997 in _sass/page.scss

@harding removed, can you check it's OK.

Tested ACK 41a819e6777836de864a91034948517c491a8db3

Tested ACK 41a819e6777836de864a91034948517c491a8db3 (with thanks to everyone who's shown their support :+1: )

Needs rebase.

@harding needs rebase.


fix: deployment fails even if broken job belong to different namespace

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

27 of 28 (96.43%)  changed or added relevant lines in 1 file are covered.
47 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.09%) to 70.895%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

job/service.go
27
28
96.43%

Files with Coverage Reduction
New Missed Lines
%

job/service.go
47
72.13%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
4762

Relevant Lines:
6717

💛  - Coveralls


Additional examples request

Hello
Is it possible to get the following examples added? Specifically looking for "Go"
Go
Rust
C++

Go support released https://pkg.go.dev/github.com/aws/smithy-go/aws-http-auth
Ref: https://github.com/aws/aws-sdk-go-v2/issues/2634


DBZ-7498 Support Oracle 23 "IF [NOT] EXISTS" clauses for DDL

https://issues.redhat.com/browse/DBZ-7498

/packit test --labels oracle

Confirmed test failures are unrelated.


Compile on VS 2012/2013

atomic_bool does not have a constructor on VS 2012/2013 but can be replaced by atomic<bool> which has one. Same for atomic_int
http://stackoverflow.com/a/15751036

If VS isn't C++11 compliant, that's not my problem.

Well, that's not helpful, and the change has no other effects.


download on click of an App

What is this ticket about?

working on the screens and the code behind to be able to upload  files from developer side and download it as an end user.


fix: bump errorreporting to solve groupId issues

Will re-kick CI after new artifact starts showing up in central.

Artifacts have landed: https://repo1.maven.org/maven2/com/google/cloud/google-cloud-errorreporting/0.118.1-beta/


Less implicit panics

Cherry-picked commits from #3129 that are strictly about removing implicit panic call-sites

the "Remove implicit bounds checking from rcc module" commit breaks fdcan tests.
the reason is  unsafe { FOO }.get_mut(0) copies the entire FOO to a temporary, then .get_mut() gets a reference to that. while unsafe { FOO.get_mut(0) } does the right thing. see playground. Fun footgun!

Ha interesting footgun indeed!


implement onRequestMetadata/onResponseMetadata

Hi @mathetake , I'm new to proxy-wasm-go-sdk and trying to fixed https://github.com/tetratelabs/proxy-wasm-go-sdk/issues/91 .

thanks! Would it be possible to add e2e tests?

thanks! Would it be possible to add e2e tests?

I think it's ok, but need a little bit time.
I'm trying to figure out why always fail on my env when run e2e test
go test -v ./e2e -count=1
=== RUN   Test_dispatch_call_on_tick
    main_test.go:63:
                Error Trace:    main_test.go:63
                                                        e2e_test.go:30
                Error:          Condition never satisfied
                Test:           Test_dispatch_call_on_tick
                Messages:       Envoy has not started
--- FAIL: Test_dispatch_call_on_tick (5.00s)

root@istiodev /m/d/g/s/i/proxy-wasm-go-sdk (main) [SIGINT]# which envoy
/usr/local/bin/envoy

maybe you have to do make build.examples before running e2e as it uses example binaries.  https://github.com/tetratelabs/proxy-wasm-go-sdk/blob/main/Makefile#L9-L10

maybe you have to do make build.examples before running e2e as it uses example binaries. https://github.com/tetratelabs/proxy-wasm-go-sdk/blob/main/Makefile#L9-L10

that works fine, maybe we need update doc or makefile?

I would appreciate if you could fix makefile and add file dependency on make tset.e2e

what I want you to add is not only an example but the actual test running on envoy: https://github.com/tetratelabs/proxy-wasm-go-sdk/blob/main/e2e/e2e_test.go

what I want you to add is not only an example but the actual test running on envoy: https://github.com/tetratelabs/proxy-wasm-go-sdk/blob/main/e2e/e2e_test.go

yes, but I got no idea about how to simulate a http request will trigger OnHttpRequestMetadata, any suggestions?

maybe speaking HTTP text directly via tcp connection?

maybe speaking HTTP text directly via tcp connection?

let me try with that.

failed to setup a demo, @mathetake can you share how to do that when you're free, or you can take over the whole PR.


Git Fetch / Rebase

VSCode Version: N/A
OS Version: N/A

Steps to Reproduce:

Select Git tab
Open pulldown
Notice the lack of Fetch / Rebase

Alternatively being able to configure Pull to execute git pull --rebase  would be quite nice and reduce the need for fetch and rebase.
This is also a feature request but I'm not aware if there's a template, so excuse the format 👍

Pull (Rebase) exists already. No need to call fetch, since it is automatically done for you.


[Feature] Syntax Highlighting for Messages

Description

We should add a way to have syntax highlighting for messages so Thream is nicer for developers.
Describe the solution you'd like
For example :
```js
console.log('Hello, world!')
````
Could render like that:
console.log('Hello, world!')

We could use libraries like https://prismjs.com/ or https://highlightjs.org/.

:tada: This issue has been resolved in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add room.send_markdown API

It seems it's not possible to send markdown messages as of today.
Will you consider adding support for it, or is there any workaround (like converting markdown to html automatically then use send_html?

here's my workaround:
from markdown2 import Markdown

MSG_PLAIN = "FooBarBatz"
MSG_MD = "**FooBarBatz**"

response = matrix.send_message_event(
    room_id=MATRIX_ROOM,
    event_type="m.room.message",
    content={
        "msgtype": "m.text",
        "format": "org.matrix.custom.html",
        "body": MSG_PLAIN,
        "formatted_body": Markdown().convert(MSG_MD),
    }
)


problem including image in page_header.html

Hi, first of of all thanks for the great theme! I really like the look and especially its customizability.
I have tried to add a .png image to the _includes/page_header.html so that it will be displayed on every one of the pages calling the layout: page front matter.
However, the image is just being displayed on the home page and not on the about nor the contact page (which I added). Both the about page and the contact page just show the alt-text that was specified in the page_header.html.
See the live website here.
I am now wondering why this is the case. Obviously the code seems to work, as the image is being displayed on home. However, logically any other page with the layout: page should display the image, too as {%- include page_header.html -%} is obviously included in the page.html.
Thanks in advance for any hints/recommendations!

Specifying the base folder of my repository in the page_header.html solved the problem for me. See here.


Fix Bundle-NativeCode in MANIFEST file

Motivation:
We used , but should have used ;
Modifications:
Replace , by ;
Result:
Correct manifest

This came up here https://github.com/netty/netty/commit/197da29257dac0be169b68665cb55ac16fa148d2#commitcomment-45136903


[BUG]Un-handeled rejecetion error for chat.

I am pretty sure this is a bug :)
I have a bot that adds a user to a new group chat once they send a friend request. The error i get is:
Unhandled rejection Error: Send chat message failed
at C:\Users\Admin\Code\Discord\node_modules\noblox.js\lib\chat\sendChatMessage.js:45:15
at tryCatcher (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\util.js:16:23)
at Promise._settlePromiseFromHandler (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\promise.js:547:31)
at Promise._settlePromise (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\promise.js:604:18)
at Promise._settlePromise0 (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\promise.js:649:10)
at Promise._settlePromises (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\promise.js:729:18)
at _drainQueueStep (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\async.js:93:12)
at _drainQueue (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\async.js:86:9)
at Async._drainQueues (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\async.js:102:5)
at Immediate.Async.drainQueues [as _onImmediate] (C:\Users\Admin\Code\Discord\node_modules\bluebird\js\release\async.js:15:14)
at processImmediate (internal/timers.js:461:21)
Part of my code which I believe I am getting the error from is:
        if (sendmessage1) {
            conversationid = noblox.startGroupConversation([data, data], "#### Bot");
            conversations.push(conversationid);
            console.log(conversations);
            console.log(conversationid);
            setTimeout(function() { noblox.sendChatMessage(conversationid, "Hello, I am #### how may I help you?"); }, 1000);
            sendmessage1 = false;
        }

the users is already a friend.
The user id is passed through the data variable.
Sendmessage1 is set o true at beginning of run.
Conversation is a text file and I removed it without fixing the errors.
Noblox is well noblox.
Correct me if I'm wrong, but I think it is an internal error.
Thank you for your time!
Edit
The hashes are not actually in the code. They release info I am not yet ready to release.

conversationid would be an unresolved promise, thus triggering the error in sendChatMessage().
Also make sure you have sufficient permissions between both accounts to initiate a group conversation.

Thank you!


Deploy filter plugins fails on centos7

TASK [cmacrae.sensu : Deploy filter plugins] ***********************************
fatal: [localhost]: FAILED! => {"failed": true, "msg": "the file_name '/etc/sensu/filters' does not exist, or is not readable"}

Added a note to the README that dynamic_data_store needs to be defined: https://github.com/sensu/sensu-ansible#requirements