What's the proper workflow for compiling this?

Hello.
I'm extremely new to C++, but I was able nonetheless  to modify this plugin for my current needs.
And I'm able to compile it by running
cd godot-cpp
scons platform=windows generate_bindings=yes
cd ..
scons platform=windows

And it runs fine on my machine (Windows 10 PC)
But as soon as I open the project on my another machine (Windows 11 laptop) I get an error that weirds me the hell out
Can't open dynamic library: C:/.../project/addons/opus/bin/win64/libgodotopus.dll, error: Error 126
 modules/gdnative/gdnative.cpp:510 - No valid library handle, can't get symbol from GDNative object
 modules/gdnative/nativescript/nativescript.cpp:1503 - No nativescript_init in "res://addons/opus/bin/win64/libgodotopus.dll" found

Your original asset from https://github.com/Godot-Opus/libopus-gdnative-asset runs perfectly however, and I'm unable to figure out what could be the difference between the two :/
If you could guide me throught the process (or point out some specific technique you used to overcome this problem) I would really appreciate it.

Not to mention the resulting .dll is 3.4+ megabytes, instead of 0.5 that can be found in your asset


Cannot use custom context and connect (as es7 decorator) together

When connecting a react component using es7 decorators, you cannot make custom context properties available:
@connect((store) => {
    return store
})
class MyComp extends React.Component {
  // ... component definition here
}

// BELOW DOES NOT WORK
MyComp.contextTypes = {
  // Keep contextTypes defined by connect, otherwise react will crash because connect can't access its context
  ...MyComp.contextTypes,
  prop: PropTypes.string
}

Everything seems to work fine, but this.context stays empty in MyComp. There's no issue when you use connect as a normal function (not a decorator):
class MyComp {...}
MyComp.contextTypes = {
  prop: PropTypes.string
}
connect(...)(MyComp)

This behavior is probably not even under react-redux' control (es7, maybe I don't understand decorators fully), but I thought I'd post anyway, so it is at least documented.
(Off topic: Some of you may cringe that I use custom context in addition to redux, but I think I've actually found a legitimate use case; I use context with custom table components so that every part of the table is aware of its row, column and kind of data being displayed. :) )

Yep, this is likely the same general issue as trying to propTypes or defaultProps.  When you use @connect() as a decorator, the "plain" component is never exposed, and those items are applied to the wrapper component definition instead.
This is one of several reasons why the Redux team recommends that you do not use connect as a decorator, but rather use it as a standard function call instead.

@markerikson what i should to if i want to use decorator, it worked fine with<EMAIL_ADDRESS>before


Change Apply3DSecure default to 0 to reduce potential liability for c…

…hargebacks
Set the Apply3DSecure to match the SagePay default value of 0. Having 2 as the default bypasses 3D secure verification and leaves the site owner liable for any chargebacks.
SagePay recommend you configure 3D-Secure verification to protect yourself against chargebacks
3D-Secure provides protection from chargebacks - the bank takes responsibility if 3D-Secure is used

Thanks @TwoMoreThings I wasn't aware this was the case so no sure why it was coded to go against the default (unless the default changed at some point). I'll get this merged in, but it'll probably be next week when I can get the update rolled out.
Sorry to hear this caused a problem for you client. That's obviously not the experience we want customers to have with out product.

@TwoMoreThings just to let you know 3.4.4 is now out with this PR merged in. Thanks again for letting us know and submitting the PR 👍


A consultation on your used GPU

Dear author:
Could you please tell me if two 2080Ti are enough for reproducing your results? Thank you in advance!!! PS. If the answer is yes, could you please tell how much time I need to spend on it?

Yes, they are enough.
I need around two weeks to train everything from scratch (propagation, scribble to mask, and fusion) with two 1080Tis. With the faster 2080Tis, it is probably going to take around 10 days.


[AutoDiff upstream] more IRGen for @differentiable functions

Handle applies of differentiable functions.
Handle convert_escape_to_no_escape of differentiable functions.
Adds filecheck covering this.
Also I took the opportunity to add filecheck tests covering IRGen of differentiable_function and differentiable_function_extract, because we don't have those in master yet.

This is one of the blockers for getting differential operators upstreamed.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


Fix tokenizer for reuters dataset

Need to remove a few characters ( like ?, ! ) from sentences. In other words, add a few relevant delimiters.

Take a look at datasets/reuters.py. Removing the special characters from the regular expression should do what you want.
def clean_string(string):
    """
    Performs tokenization and string cleaning for the Reuters dataset
    """
    string = re.sub(r"[^A-Za-z0-9(),!?\'`]", " ", string)
    string = re.sub(r"\s{2,}", " ", string)
    return string.lower().strip().split()


arcgisbinding installed but arctoolbox not recognizing

R 3.5.0 and desktop 10.5.1.  R integration python scripts install/version scripts output suggest it installed correctly but my arctoolbox tool gives:
*** Please install ArcGIS R integration package
I've got Rstudio referencing 32-bit version of R and arc.check_product suggests it's working correctly:

arc.check_product()
product: ArcGIS Desktop ( <IP_ADDRESS>33 )
license: Advanced
version: <IP_ADDRESS>

full output of R installation details:
Executing: RInstallDetails
Start Time: Sat Aug 25 09:52:29 2018
Running script RInstallDetails...
R (version 3.5.0), installed in: C:\Program Files\R\R-3.5.0
R packages will be installed into: C:\Users\strobrid\R\win-library\3.5
All R package libraries detected: C:\Users\strobrid\R\win-library\3.5;C:\Program Files\R\R-3.5.0\library
The ArcGIS R package (version <IP_ADDRESS>) is installed at: C:\Users\strobrid\R\win-library\3.5\arcgisbinding
Completed script RInstallDetails...
Succeeded at Sat Aug 25 09:52:29 2018 (Elapsed Time: 0.03 seconds)

I installed R 3.5.1 and installed the package there but no luck.  However, I solved my issue by copying the arcgisbinding package from the C:\Users<username>\R\win-library location to the C:\Program Files\R\R-3.5.1\library directory.

@DanStrobridge Glad you were able to resolve this yourself -- I think the issue is desktop checks a few standard locations, and the default per-user library directory is %USERPROFILE%\Documents\R\win-library\3.5, on your machine it looks like its configured to be directly in your home folder instead of within Documents. Is this set up via R_LIBS_USER? Thanks, Shaun


Add Tests for learning.py

Added tests for the following learners:

DecisionTreeLearner
NeuralNetLearner
PerceptronLearner

Also, updated test for PluralityLearner.

(closed and reopened PR to restart Travis Build Test)


Some posts not recording feedback? (resulting in a 'feedback recorded' post loop via SD)

https://metasmoke.erwaysoftware.com/post/149859 as an example, has gotten multiple feedback submissions for it today, and none of the feedback seems to be being kept/logged as feedback.
https://chat.stackexchange.com/transcript/message/54895296#54895296 is where you can see the chat logs begin for it with attempting to put feedback on it.

@ArtOfCode- dug and found some redis errors triggering for this one.  (Adding the redis tag)

For some reason, there was a sorted set where there should've been a set. No idea why. But it was a one off thing, since that particular post was in my redis dump from 6+ months ago, so I don't think this bug should occur again.


TargetStorage fails to store pages because of too long file name

TargetCBORRepository and TargetFileRepository eventualy fail to store pages because the file name is too long. A shorter name scheme should be used to avoid this errors. Example:
[2015-09-17 19:36:23,046]ERROR [dispatcher-22] (TargetCBORRepository.java:63) - Failed to store object in repository.
java.io.FileNotFoundException: ./data/data_target/www.trust.org/http%3A%2F%2Fwww.trust.org%2Fspotlight%2FEbola%2F%3Futm_medium%3Demail%26utm_campaign%3DAlertNet%2520Expresso%252028%2520Jan%25202015%26utm_content%3DAlertNet%2520Expresso%252028%2520Jan%25202015%2520CID_f5d471ee9a77c8bdd3ade118bd02acbe%26utm_source%3DCampaign%2520Monitor%26utm_term%3DEbola%2520outbreak%2520in%2520West%2520Africa (File name too long)
        at java.io.FileOutputStream.open0(Native Method) ~[na:1.8.0_60]
        at java.io.FileOutputStream.open(FileOutputStream.java:270) ~[na:1.8.0_60]
        at java.io.FileOutputStream.<init>(FileOutputStream.java:213) ~[na:1.8.0_60]
        at java.io.FileOutputStream.<init>(FileOutputStream.java:162) ~[na:1.8.0_60]
        at com.fasterxml.jackson.core.JsonFactory.createGenerator(JsonFactory.java:1072) ~[jackson-core-2.5.4.jar:2.5.4]
        at com.fasterxml.jackson.databind.ObjectMapper.writeValue(ObjectMapper.java:2730) ~[jackson-databind-2.5.4.jar:2.5.4]
        at focusedCrawler.target.TargetCBORRepository.insert(TargetCBORRepository.java:60) ~[ache-0.3.1.jar:0.3.1]
        at focusedCrawler.target.TargetStorage.insert(TargetStorage.java:91) [ache-0.3.1.jar:0.3.1]
        at [...]

Added a new option for data format FILESYSTEM_* that hashes the names of files to avoid this error in commit 58d6209c7d29b1c9c89194d951f7f37a0a08409b.


LinkObject::linkd: Unable to communicate with closed link

I've installed version 0.0.10 in VSCode 1.68, and I am using Wolfram Engine 13.0.1 on Windows 10. Attempting to connect to a kernel, using the default wolframscript configuration, fails with the following log output:
[Sun, 12 Jun 2022 16:19:28 GMT] kernelIsRemote = false, kernelPort = 59322
[Sun, 12 Jun 2022 16:19:28 GMT] kernelInitPath = c:\Users\dylan\.vscode\extensions\njpipeorgan.wolfram-language-notebook-0.0.10\resources\init-compressed.txt
[Sun, 12 Jun 2022 16:19:28 GMT] kernelRenderInitPath = c:\Users\dylan\.vscode\extensions\njpipeorgan.wolfram-language-notebook-0.0.10\resources\render-html.wl
[Sun, 12 Jun 2022 16:19:28 GMT] launchCommand = wolframscript
[Sun, 12 Jun 2022 16:19:28 GMT] launchArguments = -code,ToExpression["zmqPort=59322;"<>Uncompress["1:eJy9PA1X27iy96e4gsMBrkJJ+7bvHSB54fOWu0ApSXfvWdcLJhHEi2OnttNCQ/afvf/2ZvRhS7IdUtpddimJJY1Go9F8yy+u44vu/7n/+MfqurO1de737/xbtrXlrK993Gy+wl/e0GX9LIijvGF1/
[Sun, 12 Jun 2022 16:19:30 GMT] Received the following data from kernel:
[Sun, 12 Jun 2022 16:19:30 GMT] <INITIALIZATION STARTS>
[Sun, 12 Jun 2022 16:19:33 GMT] Received the following data from kernel:
[Sun, 12 Jun 2022 16:19:33 GMT] $CommandLine={C:/Program Files/Wolfram Research/Wolfram Engine/13.0/wolfram.exe, -runfirst, Unprotect[$EvaluationEnvironment];$EvaluationEnvironment="Script";Protect[$EvaluationEnvironment];, -linkmode, Connect, -
[Sun, 12 Jun 2022 16:19:33 GMT] Received the following data from kernel:
[Sun, 12 Jun 2022 16:19:33 GMT] $ScriptCommandLine={wolframscript, -code, ToExpression["zmqPort=59322;"<>Uncompress["1:eJy9PA1X27iy96e4gsMBrkJJ+7bvHSB54fOWu0ApSXfvWdcLJhHEi2OnttNCQ/afvf/2ZvRhS7IdUtpddimJJY1Go9F8yy+u44vu/7n/+MfqurO1de737/xbtrXlrK993Gy+
[Sun, 12 Jun 2022 16:19:33 GMT] Received the following data from kernel:
[Sun, 12 Jun 2022 16:19:33 GMT] Using the following command to launch the subkernel process:
[Sun, 12 Jun 2022 16:19:33 GMT] Received the following data from kernel:
[Sun, 12 Jun 2022 16:19:33 GMT] C:/Program Files/Wolfram Research/Wolfram Engine/13.0/wolfram.exe -noicon -wstp -linkprotocol SharedMemory -linkmode connect -linkname qnuuu1jsqc0r55ie
[Sun, 12 Jun 2022 16:19:35 GMT] Received the following data from kernel:
[Sun, 12 Jun 2022 16:19:35 GMT]
LinkObject::linkd: Unable to communicate with closed link LinkObject[qnuuu1jsqc0r55ie, 192, 2].

[Sun, 12 Jun 2022 16:19:35 GMT] Killing kernel process, pid = 27420
[Sun, 12 Jun 2022 16:19:35 GMT] Process exited with code 0 and signal null.

Note that I have put C:\Program Files\Wolfram Research\Wolfram Engine\13.0 and C:\Program Files\Wolfram Research\WolframScript on PATH.

Is there anything more I can do to help diagnose this problem?

Sorry for the late reply.

This problem is still occurring. And it occurs in container development in VS Code.
$Version
13.3.0 for Linux x86 (64-bit) (June 12, 2023)

Would like to support container development so that Docker can be used.

Unfortunately, I had a same problem when I try to use it in VS code. This problem persists.
[Sun, 29 Sep 2024 14:40:58 GMT] kernelIsRemote = false, kernelPort = 52892
[Sun, 29 Sep 2024 14:40:58 GMT] kernelInitPath = c:\Users\Tieti_Tage\.vscode\extensions\njpipeorgan.wolfram-language-notebook-0.1.1\resources\init-compressed.txt
[Sun, 29 Sep 2024 14:40:58 GMT] kernelRenderInitPath = c:\Users\Tieti_Tage\.vscode\extensions\njpipeorgan.wolfram-language-notebook-0.1.1\resources\render-html.wl
[Sun, 29 Sep 2024 14:40:58 GMT] launchCommand = wolframscript
[Sun, 29 Sep 2024 14:40:58 GMT] launchArguments = -code,ToExpression["zmqPort=52892;"<>Uncompress["1:eJy9PA1X27iy96e4gsMBrkJJ+7bvHSB54fOWu0ApSXfvWdcLJhHEi2OnttNCQ/afvf/2ZvRhS7IdUtpddimJJY1Go9F8yy+u44vu/7n/+MfqurO1de737/xbtrXlrK993Gy+wl/e0GX9LIijvGF1/
[Sun, 29 Sep 2024 14:41:00 GMT] Received the following data from kernel:
[Sun, 29 Sep 2024 14:41:00 GMT] <INITIALIZATION STARTS>
[Sun, 29 Sep 2024 14:41:01 GMT] Received the following data from kernel:
[Sun, 29 Sep 2024 14:41:01 GMT] $CommandLine={C:/Program Files/Wolfram Research/Wolfram Engine/14.0/wolfram.exe, -runfirst, Unprotect[$EvaluationEnvironment];$EvaluationEnvironment="Script";Protect[$EvaluationEnvironment];, -linkmode, Connect, -
[Sun, 29 Sep 2024 14:41:01 GMT] Received the following data from kernel:
[Sun, 29 Sep 2024 14:41:01 GMT] $ScriptCommandLine={wolframscript, -code, ToExpression["zmqPort=52892;"<>Uncompress["1:eJy9PA1X27iy96e4gsMBrkJJ+7bvHSB54fOWu0ApSXfvWdcLJhHEi2OnttNCQ/afvf/2ZvRhS7IdUtpddimJJY1Go9F8yy+u44vu/7n/+MfqurO1de737/xbtrXlrK993Gy+
[Sun, 29 Sep 2024 14:41:01 GMT] Received the following data from kernel:
[Sun, 29 Sep 2024 14:41:01 GMT] Using the following command to launch the subkernel process:
[Sun, 29 Sep 2024 14:41:01 GMT] Received the following data from kernel:
[Sun, 29 Sep 2024 14:41:01 GMT] C:/Program Files/Wolfram Research/Wolfram Engine/14.0/wolfram.exe -noicon -wstp -linkprotocol SharedMemory -linkmode connect -linkname 3b112vys2c6cxarc
[Sun, 29 Sep 2024 14:41:01 GMT] Received the following data from kernel:
[Sun, 29 Sep 2024 14:41:01 GMT]
LinkObject::linkd: Unable to communicate with closed link LinkObject[3b112vys2c6cxarc, 205, 3].

[Sun, 29 Sep 2024 14:41:01 GMT] Killing kernel process, pid = 5732
[Sun, 29 Sep 2024 14:41:01 GMT] Process exited with code 0 and signal null.

Here are some outputs.
I tried restarting my PC and VS Code. I definitely believe that I killed all the Wolfram processes before. However, it seems that it did not work at all. This occurs in Wolfram Engine version 14.1 on Windows 11. Have you encountered this bug? Would you like to fix it in a future version?


Have You added the Ability to Disconnect() ?

This API is really great and I actually won't mind financially supporting you. Where can I donate?
I would like to know if you have or plan to add a Disconnect() feature.

Add this function to BleWinrtDll.cpp
void Disconnect(wchar_t* deviceId) {
	for (auto device : cache) {
		if (hsh(deviceId) == device.first) {
			device.second.device.Close();
			for (auto service : device.second.services) {
				service.second.service.Close();
			}
			cache.erase(hsh(deviceId));
			return;
		}
	}
}

Add this function to BleWinrtDll.h
__declspec(dllexport) void Disconnect(wchar_t* deviceId);

Add this function to BLE.cs
        [DllImport("BleWinrtDll.dll", EntryPoint = "Disconnect", CharSet = CharSet.Unicode)]
        public static extern void Disconnect(string deviceId);

@thangdhz Thank you, this is great!
Do I have to recompile a new .dll if i add void Disconnect(wchar_t* deviceId) to the .dll file?

yes, need recompile .dll file

yes, need recompile .dll file

Excellent, Thanks

Hi @Ugobyte , thanks for the offer! I probably can't work on this project for like the next ten months. You can feel free to hire freelancers to fix some of the most popular open bugs 😉 or maybe sponsor one of the forks. I would gladly redirect to it if it gets more features.

@thangdhz This code works, but do not work well. After disconnect from device, you can not find the same device by scan function.Is there something wrong with the scan codes?

Thanks, this code worked for me


Bug: Contact Us under Help/Support throws an Undefined Error

Issue:
When a user wishes to submit a query using the Contact Us section on the Help/Support page, it displays the Success Message and throws an Undefined Error. This need to be fixed so users are able to submit the requests successfully
Testing
Tester: @JanellC

Bug resolved. The system displays an success message without an error


Terminal widget - does it exist? How can I find it?

Hi,
I recall reading that there is a terminal widget where you can type comments conveniently.
How can I find/enable said widget?
If it does not exist would it be a good enhancement?
Cheers!

If you're on Linux and run with hb-service you can add the "Terminal" widget to the status screen, and also access the terminal from the option in the upper-right-corner drop down menu.
Terminal is disabled on macOS and Windows for various non-technical reasons.

How about Raspbian OS?

That is Linux, so yes.

If you setup using the Homebridge Wiki Guide for Raspbian,  then you should have this already.
https://github.com/homebridge/homebridge-raspbian-image/wiki/Getting-Started

I think that's what I did like a year ago, but terminal does not appear (see attached). I guess I used another way to set-up homebridge as a service. Perhaps I will just re-install if I find some time :)
WDYT?


react-native-meteor not working on android versions 9.0 and above

Hi, I have noticed that react-native-meteor doesn't run on android versions 9.0 and above when the app is run via apk, though it works properly on local(using dev server).
i.e
In my code nothing is executed after this line when using v9.0 and above via apk
Meteor.call('Account.checkEmailVerification', { email }, (err, ok) => {
All other versions execute perfectly.
Note: react-native-meteor works properly on all versions when run in development
Any help will be highly appreciated 🙏🏽

I fixed this by adding android:usesCleartextTraffic="true" to android manifest file. Read more about it here.
It occurs because android versions 9+ don't allow insecure connections in release builds. For my case I was connecting to ws:// which would have also worked if it were wss://


Testing with lnbits v1.0.0-rc5

I see that many of the extensions have been updated to the upcoming v1.0.0 release, but not the usermanager.
This is the error I get when trying to install usermanager v0.4.1:
 2024-11-14 12:08:30.00 | INFO | lnbits.core.extensions.models:download_archive:455 | Downloading extension usermanager (0.4.1).
 2024-11-14 12:08:30.84 | INFO | lnbits.core.extensions.models:extract_archive:488 | Extracting extension usermanager (0.4.1).
 2024-11-14 12:08:30.94 | INFO | lnbits.core.extensions.models:extract_archive:526 | Extension User Manager (0.4.1) extracted.
 2024-11-14 12:08:31.00 | ERROR | lnbits.core.helpers:migrate_extension_database:30 | cannot import name 'get_key_type' from 'lnbits.decorators' (/app/lnbits/decorators.py)
 2024-11-14 12:08:31.00 | WARNING | lnbits.core.views.extension_api:api_install_extension:88 | Cannot import module for extension 'usermanager'.
 Traceback (most recent call last):
   File "/app/lnbits/core/helpers.py", line 27, in migrate_extension_database
     ext_migrations = importlib.import_module(f"{ext.module_name}.migrations")
   File "/usr/local/lib/python3.10/importlib/__init__.py", line 126, in import_module
     return _bootstrap._gcd_import(name[level:], package, level)
   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
   File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
   File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
   File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
   File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
   File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
   File "<frozen importlib._bootstrap_external>", line 883, in exec_module
   File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
   File "/app/lnbits/extensions/usermanager/__init__.py", line 23, in <module>
     from .views_api import *  # noqa
   File "/app/lnbits/extensions/usermanager/views_api.py", line 11, in <module>
     from lnbits.decorators import (
 ImportError: cannot import name 'get_key_type' from 'lnbits.decorators' (/app/lnbits/decorators.py)

 The above exception was the direct cause of the following exception:

 Traceback (most recent call last):
   File "/app/lnbits/core/views/extension_api.py", line 85, in api_install_extension
     extension = await install_extension(ext_info)
   File "/app/lnbits/core/extensions/extension_manager.py", line 32, in install_extension
     await migrate_extension_database(ext_info, db_version)
   File "/app/lnbits/core/helpers.py", line 31, in migrate_extension_database
     raise ImportError(f"Cannot import module for extension '{ext.id}'.") from exc
 ImportError: Cannot import module for extension 'usermanager'.
 2024-11-14 12:08:31.13 | ERROR | lnbits.exceptions:http_exception_handler:103 | HTTPException 500: Failed to install extension 'usermanager'.(0.4.1).

i think usermanager will be phased out, @motorina0 whats the consensus on that topic, do you know?

Core covers all the functionality that used to be in usermanager extension.
See: https://github.com/lnbits/lnbits/pull/2777
usermanager will be retired


Deploy app to a staging environment to share with stakeholders

Transitionary step towards real production deployment

Done. Currently/temporarily available at https://inkind-admin.herokuapp.com/ .


Do not attempt to rename the “type” of a block.

Do not attempt to rename the type of a block when it’s name is
changed. Doing so can corrupt projects is the oldname of a block is a
substring of a type name (say component_set_get). If you create a
component named “et”. Make sure it has blocks in the workspace and
then rename it to “st” the project will become corrupted because
occurances of the string “component_set_get” will turn into
“component_sst_get.”
Change-Id: I2dbcb308a6f81f67a2ebdef645e70c4dfee93f7a

@halatmit This fixes the corruption...

@halatmit Note: I'm going to push up another commit that completely removes the commented out code (and the snide comment...)

@halatmit @afmckinney One of the early XML representation of components had the component name as part of its type. At some point Paul rewrote how components were represented and the names are no longer part of the component type. The code I removed was a vestige of the previous code and is no longer needed, or wanted!
-Jeff

LGTM
Great catch in figuring this out.


It seems the forbidden apis check do not work

The build log:
Classes directory does not exist, forbidden apis check skipped

The forbiddenapis plugin works on compiled classes. Currently, the Mojo runs in the validate lifecycle phase, there may be no classes for it to check forbidden apis if we have cleared all classes. To make it work, we have to run a compile command first, then the maven command may like this mvn clean compile compile which is not elegant. So modify the execution phase to compile may better.


Added support for neo4j connection scheme

required by Neo4j cluster setup

Thanks for the pull request @DirkMahler!
2 questions:

I assume this is scheme works with Neo4j 3.x as well, is that correct?
what's the fastest to manually validate this change in your opinion? Docker?

Thanks again!

As far as I can see the neo4j connection scheme is supported by Neo4j 3.x as well
Regarding manual validation I think docker is the way to go (already checked it)

Merged as 2110b398be16f59eecf84f3d69c4ce804c3ccb7d, thanks!
neo4j scheme is only for 4.x, it replaces bolt+routing.


Remove uneeded feature dependencies

Following #2735, these are no longer required.

https://githubstatus.com doesn't seem to know that GitHub Actions are down...


Add some better documentation around the intended structure of the ap…

The Pull Request is ready

[x] all github actions are passing
[ ] only a single issue was worked on
[x] fixes #106, #62
[x] the branch follows the naming schema issue-123-enable-x-does-not-disable-y
[x] the pull request has a sensible title

Intention
With this change I intend to add more info around the design of the application
Review Points
Please take extra care reviewing...

The code follows best practices

[x] duplicate code has been extracted where possible
[x] issues for follow-up tasks have been created
[x] tests have been written for any new functionality
[x] there is no any type used
[x] texts have been checked for grammar and spelling issues

Notes

This is super helpful!
If requests are intended to go through the Wakatime CLI, that means that our users would need to also sign up over there and have an API key, right?

This is super helpful!
If requests are intended to go through the Wakatime CLI, that means that our users would need to also sign up over there and have an API key, right?

The api key is only required if you are sending the heartbeats to Wakatime's central server. Right now, our cli is writing an api key to the .wakatime.cfg, but only to stop the extensions from complaining about one being missing.
The way they have their application setup, all users data goes to a centralized repository and you view your data through their website. In the future, maybe there's a mod we make that allows our CLI to proxy the information you have locally to the wakatime server if you'd like to have it there as well. Right now, the requests stop at our CLI so we don't need their api key


Fix subcommand 'list' help message

Currently we have: https://github.com/foresterre/cargo-msrv/blob/main/src/cli.rs#L167
However, this actually replaces the help text, i.e.:
$ cargo msrv list --help
List the MSRV's specified by dependency crate authors

Instead, we wanted to have it be the text printed on running cargo msrv --help, which currently shows:
SUBCOMMANDS:
    help
            Prints this message or the help of the given subcommand(s)

    list

But should show up as:
SUBCOMMANDS:
    help
            Prints this message or the help of the given subcommand(s)

    list
            List the MSRV's specified by dependency crate authors

Probably we need to replace the .help(...) call with .about(...)

maybe release as 0.12.1?


Add COPY GRANTS to #Snowflake table materialisation

Describe the bug
Currently, doing a full table refresh on snowflake is done via create or replace table. Which works fine for the data side as the operation is atomic. However, existing grants are not copied when replacing the table, requiring a pre-hook to set them right. This can be corrected by adding COPY GRANTS to the CTAS command.
See https://docs.snowflake.com/en/sql-reference/sql/create-table.html
Steps To Reproduce
Running a job in Snowflake with the table materialisation
Expected behavior
Pre-existing grants being copied over when replacing a table
System information
Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[X] snowflake
[ ] other (specify: ____________)

Right on, @bashyroger. dbt has supported copy grants in Snowflake materializations since 0.15.0 (changelog):
{{ config(
    materialized = 'table',
    copy_grants = true
}}

select 1 as fun

Can you give that a go and see if it does the trick? I'm going to close this issue, I encourage you to reopen if you run into any issues.

@jtcohen6 can you PR a change to the docs that adds this config to the list of Snowflake profile configs?

Right on, @bashyroger. dbt has supported copy grants in Snowflake materializations since 0.15.0 (changelog):
{{ config(
    materialized = 'table',
    copy_grants = true
}}

select 1 as fun

Can you give that a go and see if it does the trick? I'm going to close this issue, I encourage you to reopen if you run into any issues.

Ah, totally missed this @jtcohen6 ! I see it in the docs now (https://docs.getdbt.com/reference/resource-configs/snowflake-configs/), my bad!


Fix VSP secret to have extra layer of base64 encoding of the keys

For the record, when the sealed secrets controller unsealed this it correctly assumed type: Opaque, and also the change of ciphertext for the environment seems to be irrelevant - the value has not changed and this has been checked.


Fix date time convention for all dates (not just Calendar, Interval, …

Make sure dates are properly converted for all chart types

Looks good.

tag v1.3


Allowing for other event handlers

I ran into an issue with the current version of jQuery-contextMenu not being able to handle objects that I wanted to associate draggable or resizable to. Basically the current code stops propagation of left mouse events, thereby breaking certain types of events/actions that you would want to associate with an element that has a context menu associated with it. The fix is simple, after line 34 in jquery.contextMenu.js add
if( evt.button == 0) return;
If your line numbers are different you will want to do that after $(this).mousdown(functino(e){ .... and before evt.stopPropagation();
Thanks for writing the code and making it open.
esse

which trigger type are you using?

I am using the default, which is right click trigger.
$(".media_object").contextMenu( { menu: 'myMenu' }, function(action, el, pos) { ...
I tried setting the trigger option at initialization also and that did not fix the issue for me and what I was trying to do.
Basically I have images that I want a user to be able to "right" click on to add to a containing div. Once added to the div I want the image to be resizable and draggable and still have the context-menu associated with it. The only why I could get that functionality was be the minor code tweak I described. Though I may have been doing something wrong from the first initialization of the media objects with the context-menu, but I tried lots of different approaches.

I'll look into this (weekend)


Update to latest logger

Problem
The Alephant Broker is used by Newbeat which is using an updated CloudWatch logger (the CloudWatch logger now has an updated interface).
Solution
We need to modify Alephant Broker to use the latest Alephant Logger (as that's what the updated Alephant CloudWatch Logger uses). Then update the metric calls to match the new interface.
Release Plan
I believe this will be a patch release rather than a minor or major.
Existing projects will need their Gemfiles updated at some point soon to point to the older version of the broker (as I don't believe we anchor gems to specific versions in all projects), so when older projects get deployed they just go ahead and grab the latest version of the broker gem which will use a different logger than probably expected.

:cake:


FileUtils.cp_r possible bug when dest_path exists

FileUtils.cp_r fails with,
Unable to create directory '.': File exists (Errno)
when dest_path exists. By looking at the code in file_utils.cr, I think what is needed is a test before attempting to create dest_path
 Dir.mkdir(dest_path) unless Dir.exists?(dest_path)

instead of,
 Dir.mkdir(dest_path)

Crystal 0.24.2 [4f9ed8d03] (2018-03-08)

Resolved by #9175


JSON mode support

🌟 Feature Description

Currently, the RD-Agent workflow relies heavily on JSON mode to simplify implementation.
However, many LLMs do not natively support the JSON model.
RD-Agent should offer a fallback solution, even though it might result in a higher error rate.

https://github.com/microsoft/RD-Agent/pull/419  is related

There are also some related tools for json repair in the output of LLM, such as this repo.

Whether this package is useful for the project？https://github.com/dottxt-ai/outlines


Add Intel AMX-FP16 detection

https://cdrdv2-public.intel.com/774990/architecture-instruction-set-extensions-programming-reference.pdf
this instruction set will be in future Granite Rapids (~2024-2025)

llvm https://github.com/llvm/llvm-project/blob/9f3e3efd98a29eb8df9e3ad43a573c9141d1ace2/clang/lib/Basic/Targets/X86.cpp#L834

I assume we don't yet have a CPUID dump at that point correct?


choosing ASP.Net Core version 2.2 produce no content in .NET API Browser

Describe the bug
no content is showing if we choose version 2.2 of asp.net core
To Reproduce
Steps to reproduce the behavior:

Go to https://docs.microsoft.com/en-us/dotnet/api/?view=aspnetcore-2.2
See error "No namespaces"
Type "usermanager" in search box
See error "No results"

Expected behavior
A list of apis should show
Desktop (please complete the following information):

OS: windows 10
Browser: firefox
Version: 63

@dend Can you take a look at this issue?

ASP .NET Core 2.2 has been live for two weeks now, and there's still no public API documentation for it yet.

This is now resolved, and the documentation shows up correctly.

hehe


unsatisfiable ontology classes causing Jenkins GAF build failure

I just committed an updated GAF for PomBase, and its build has failed in Jenkins because of unsatisfiable classes in the ontology. I hope it's obvious that this isn't something we can fix at the GAF level ;)
I'm attaching a list of the errors (copied from the console output for PomBase GAF build 121 at http://build.berkeleybop.org/view/GAF/job/gaf-check-pombase/). The offending terms aren't even ones we would ever use for fission yeast.

I don't see this in the editor's version of the ontology. When I load go-edit.obo into Protege and run the reasoner, there are no 'nothing' classes.
I spot checked some of the terms on the list in the Jenkin's report and they seem fine. Could this be related to the switch to the CARO anatomical structure?
@cmungall any ideas?

Oh goody, a sneaky one! On the off-chance that it helps any: I also get similar results if I run owltools locally, but it doesn't flag all of the same terms (misses a handful, perhaps because of an update lag somewhere?).
oh, and here's the file I meant to attach earlier ...
pb_gaf_build_121_errors.txt

From the GO build console output (build-go-ontology/733):
go-edit.obo-check.tmp go-edit.obo-check
:: ERROR: Undeclared relation: RO:0002015
:: ERROR: Undeclared relation: RO:0002013
Makefile:133: recipe for target 'go-edit.obo-check' failed
make: *** [go-edit.obo-check] Error 2
Build step 'Execute shell' marked build as failure

RO:0002013= 'has regulatory component activity'
RO:0002015= 'has positive regulatory component activity'
This might be unrelated to the above error.

We don't do a full integration test with the travis checks, just against the local cl module, which may be out of sync. We should probably do both.
Currently this job tests go-gaf, which is the importer ontology that uses all of cl and is used by owltools for gaf checks. It is consistently passing so need to explore more what is happening here

I debugged using the unsat module core dump that gets created:

Same as noted here and subsequently fixed: https://github.com/geneontology/go-ontology/issues/13898
So this is a file sync issue, investigating...

Can/should we put this kind of debugging on the agenda for August? Where is the unsat core dump file that you loaded into Protege?

Can certainly do that but this particular issue is caused by the fact
https://github.com/geneontology/go-site/issues/414

OK, turned out the cause was really dumb
The correct PURL is
http://purl.obolibrary.org/obo/go/imports/x-disjoint.owl
go-gaf was importing
http://purl.obolibrary.org/obo/go/extensions/x-disjoint.owl
which was the old PURL. There was a stale copy there.
The snapshot ontology should be fixed tonight. Then the subsequent gaf build should succeed.


additionalProperties helm lint failure when using opentelemetry-operator as subchart

During this issue https://github.com/open-telemetry/opentelemetry-helm-charts/pull/1065 additionalProperties was set to False for opentelemetry-operator creating a disruptive behavior for helm lint in the situation of using opentelemetry-operator as a subchart.
As the current consensus from helm community based on the long discussed issue https://github.com/helm/helm/issues/10392 is to discuss this matters with chart originator I'd like to present the scenario when the operator is consumed as a subchart by a customer.
In our scenario we created under the umbrella chart a set of templates that ease end-user usability of the chart including the otel-operator.
For example created a template for collectors that allow configuration only from values.yaml.
Therefore in our values.schema.json under the umbrella chart we will have under opentelemetry-operator this field called collectors (but could be anything else like there was a scenario sometime ago with enabled flag).
Using the chart as a subchart is a pretty common scenario therefore I would suggest reverting the additionalProperties to its previous value (default to True).
I can open a PR if allowed.

@ifalex is there a specific property you're adding to the subchart that is causing a failure?

Hi @TylerHelmuth, as specified in initial message we use opentelemetry-operator inside an umbrella chart. In order to ease the utilization process by our developers we created a collector dictionary in values.yaml which does a wrapping of OpenTelemetryCollector CR and can be modified by client directly from values.yaml.
Is another pattern to consume OpenTelemetryCollector that we found to be quite useful for our clients. More on that in an argocd setup this will mean only configuration without any need to create CRs.
On top of helm lint issue failing in our pipelines, also argocd uses a helm lint mechanism which is also causing failure due to the missing property, as additionalProperties is set on False.
By not leaving additionalProperties set to True we enforce against typos which consumer should detect on their own, limiting scenarios of usability for opentelemetry-operator as a subchart for example.
Even if we add collector: as an object to the parent values.schema.json this will not prevent other additions to fail in the future.
My proposal in this case would be to rever to the additionalProperties: True that the values.schema.json had until a few months ago.

@ifalex does your values.yaml file look like:
# top level chart details
...
opentelemetry-operator:
  collectors:
    # custom section nested under opentelemetry-operator
    ...
  # Other operator options
  ...

We also use additionalProperties: false in the collector and demo chart to help users write their values.yaml, but also to help with field deprecations (see https://github.com/open-telemetry/opentelemetry-helm-charts/issues/91).
Although the OTel Operator manages the OpenTelemetryCollector Custom Resource, the opentelemetry-operator does not. Is their a strict requirement that the collectors section be nested under opentelemetry-operator?

@TylerHelmuth Indeed, will manage it on our side. Thanks for the nice explanation! We can close the issue.


RFC: Create a Steeltoe Reference application

Build a reference application that utilizes many/most of the components our libraries provide.
This will allow the team and users a way to observe our recommended way to utilize our features, testing of new features and a central area for referencing our features.
We would love to get ideas for this application from the community.

A possible candidate would be a compatible port of PetClinic

Work is being done here: https://github.com/steeltoeoss-incubator/spring-petclinic-microservices/issues/3

Work is being done here: https://github.com/steeltoeoss-incubator/spring-petclinic-microservices/issues/3


Is this project alive ?

Simply following README fails.
No commit for 6months, plenty of bugs PR left unattended ...
hence the question

Yep, seems like it
I warned all my people involved in Sol to backout.
Bots for RcE injection points on it during scans over the last 4-5 months.


Add servo as browser option

Split off #80 with https://github.com/ErichDonGubler/moz-webgpu-cts/pull/80#issuecomment-1967398153 taken into account. This PR only contains minimal work to acknowledge servo's existence.
It works on servo (if also used with https://github.com/sagudev/moz-webgpu-cts/commit/dddc19ded5ff7b400c4cf4a26989ab6a605b2970), but I do not have m-c clone to test with.

@sagudev: Just so you know where I'm at: I'm in favor of this PR, with a few nitpicks that I'm trying to address as fixup! changes to facilitate. I'm hoping to get you feedback sometime in the next week; this has been a challenge to get to with the other work I'm doing on Firefox's WebGPU team. 😅

@sagudev: Just so you know where I'm at: I'm in favor of this PR, with a few nitpicks that I'm trying to address as fixup! changes to facilitate. I'm hoping to get you feedback sometime in the next week; this has been a challenge to get to with the other work I'm doing on Firefox's WebGPU team. 😅

No rush, I am currently using #80 myself to update expectations in servo, because it has all I need.

Okay! Whew, I think I flushed my queue of feedback here. 😮💨 Hope it's not too much!
I actually think that this feedback is sufficient, since I'm punting making things "perfect" to https://github.com/ErichDonGubler/moz-webgpu-cts/pull/103. If you think this feedback is fine, then feel free to get history back down to the few commits we intend with git rebase -i main --autosquash && git push --force-with-lease. 🙂

I used Conventional Comments in this review! I hope they help with clarity and tone.

For better or worse, you are the only person who can actually do this, since the tip branch of this PR isn't part of this repo.

Actually you should be able to to that to, even if the branch is in my fork, because I have Allowing edits by maintainers enabled (I think it is by default):

Allowing edits by maintainers
If checked, users with write access to ErichDonGubler/moz-webgpu-cts can add new commits to your servo-init branch.
You can always change this setting later.

@sagudev: Ahhh, I see now:

I'm so used to everybody not checking that, thanks!

@sagudev: FYI: Just piggybacked a fix to CI with 062b2b9, so this could auto-merge. 🙂


Tree view

Eio encourages structured programming, notably using cancellation contexts.
It could be useful to be able to inspect that tree structure.

The tree view would in particular contain switches: https://github.com/patricoferris/meio/issues/26


NHEK (2)

Functions providing the parametrizations and properties of Kerr near-horizon geodesics added.

@nielsw2, @duetosymmetry, where does this error comes from ? The file Kernel/NearHorizonGeoOrbit.m seems to work correctly on my MMA installation...

I've now rewound the master branch. Please don't merge this NHEK (2) branch until we've had a chance to review it.
I'm not sure why the tests failed. I looked at the log and it seemed the tests were timing out. Might be an issue with the test server so we'll check that and get back to you.

Thanks, @nielsw2 ! I don't do anything until you ask me

Hi @nielsw2, @barrywardell ! Is there any news regarding the addition of our NHEK code to the main branch of KerrGeodesics ? Thanks for your answer !

Hi @addruart, apologies for the delay. It's been a busy few months so I've not had time to look at all the great work that needs to be merged. I hope to have more time in the next couple of weeks and will be in touch then.

No problem @nielsw2 ! Thoses times are not simple for anyone... I'll be glad to hear from you soon !


Feature/deploy

I can't actually test the stuff that is failing for codecov - I think what we are going to have to do is put a few things in the codecove ignore file so that it only checks components and related code on the fronend and endpoints on the backend.


Frozen notebook

Hi,
I am trying to run a LightGBM quantile regression on a Toy example but the notebook seems to be stuck and cannot finish.
import pandas as pd
import numpy as np

import os
import pyspark.sql.functions as F

from pyspark.sql import SparkSession, DataFrame
from pyspark import SparkContext
from pyspark.sql.types import DoubleType

spark_context = spark.sparkContext

sql_context = SQLContext(spark_context)
import mmlspark

import pandas as pd
import numpy as np

def xcos(x):
    return x * np.cos(x)

def xsin(x):
    return x * np.sin(x)

def ad_dataset(samples=4000):
    ts =<PHONE_NUMBER>
    delta_time_seconds = 600

    f1 = np.atleast_2d(np.random.uniform(0, 10.0, size=samples)).T
    f2 = xcos(f1)
    y = xsin(f1)

    dy = 1.5 + 1.0 * np.random.random(y.shape)
    noise = np.random.normal(0, dy)
    y += noise

    dates = np.linspace(ts, ts + (samples-1) * delta_time_seconds, num=samples).reshape(-1, 1)

    concatenate_array = np.concatenate((dates, f1, f2, y), axis=1)
    concatenate_array = concatenate_array.astype(np.float32)

    df_pd = pd.DataFrame(concatenate_array, columns=["timestamp", "f1", "f2", "y"])

    return df_pd

df = sql_context.createDataFrame(ad_dataset())

features = ["f1", "f2"]
from mmlspark import FastVectorAssembler, SelectColumns
from pyspark.ml import Transformer, Estimator, Pipeline
from pyspark.ml.feature import VectorAssembler

outputFeaturesName = "features"
assembler = FastVectorAssembler(inputCols = features, outputCol=outputFeaturesName)
assembledData = assembler.transform(df)

model = mmlspark.LightGBMRegressor(application='quantile',
                                   alpha = 0.9,
                                   featuresCol=outputFeaturesName,
                                   labelCol="y",
                                   learningRate=0.1,
                                   numIterations=10,
                                   numLeaves=31,
                                   predictionCol='prediction').fit(assembledData)

I already contacted @imatiach-msft, this issue is just for tracking purpose.

@imatiach-msft I am hitting this as well, and can reproduce with the above notebook.
After a bit of debugging, I have observed:

This only happens for quantile application. Other regression losses work fine.
The call hanging is Networking::GlobalSum in the serial tree learner when doing RenewTreeOutput: https://github.com/Microsoft/LightGBM/blob/master/src/treelearner/serial_tree_learner.cpp#L800
The "small" optimization path is taken: (I have 8 executors, input_size less than 100) https://github.com/Microsoft/LightGBM/blob/master/src/network/network.cpp#L71
Recursive doubling is used:

The first iterations of the loop work fine, up to the point where we get to send_need_len = recv_need_len = 224, where it starts to hang on SendRecv
There, the non-blocking call to Recv is the one hanging: https://github.com/Microsoft/LightGBM/blob/master/src/network/linkers.h#L247

@imatiach-msft @mhamilton723 any ideas on what could be going on? Does this information help?

@peay @ezerhoun I've created a PR to fix this issue here:
https://github.com/Microsoft/LightGBM/pull/1593
I will update LightGBM to latest version in MMLSpark as soon as the other PR goes through code review and is merged

@peay getting a different error on an assert inside lightgbm in the distributed case but only when running on a cluster, the notebook runs for me locally, I created an issue here to ask whether the check in the code is needed:
https://github.com/Microsoft/LightGBM/issues/1604

@peay @ezerhoun there was another fix checked in here:
https://github.com/Microsoft/LightGBM/pull/1605
will upload another version of lightgbm to maven central

PR has been merged, closing issue - will be in 0.14


Button disabled attribute does not exist in type

Describe the bug
The attribute has the effect.
Object literal may only specify known properties, and '"disabled"' does not exist in type 'ButtonPropsWithoutHTML & HTMLAnchorAttributes & { href?: string | null | undefined; type?: undefined; } & { el?: HTMLAnchorElement | undefined; } & { ...; }'

Reproduction
Add a disabled attribute to a Button
Logs
No response
System Info
System:
  OS: macOS 14.5
  CPU: (8) arm64 Apple M2
  Memory: 1.00 MB / 16.00 GB
  Shell: 3.7.1 - /opt/homebrew/bin/fish
Binaries:
  pnpm: 9.1.2 - ~/Library/pnpm/pnpm
  bun: 1.1.7 - /private/tmp/bun-node-b0b7db5c0/bun
Browsers:
  Chrome: 125.0.6422.141
  Safari: 17.5
npmPackages:
  @sveltejs/kit: ^2.0.0 => 2.5.10
  bits-ui: ^0.21.10 => 0.21.10
  svelte: ^4.2.17 => 4.2.17

Severity
annoyance

Do you have an href on the button? If so, then it's a link, and links don't have disabled attributes.

Yeah. It makes sense. Sorry for the trouble.

I'm getting this without an href on a Form.Button, maybe because it also doesn't have an on:click handler?


Fixing https://github.com/flyway/flyway/issues/687 - added code that can remove Flashback tables in Oracle and a test case for it. Also fixed one incorrect Oracle test configuration method.

@awal11 Could you get in touch with me via email? (axel at boxfuse.com) Thanks!


[ChannelTabs] TypingIndicator Integration + UI Adjustments

Added:

TypingIndicator Integration
The Ability to Show Badges even when the Tab is Selected

Tweaked:

UI Adjustments

I just quickly skimmed it and it looks fairly good. It seemed to me like you did not really know JS or at least how to work with discord so I am impressed how good it looks. There are some formatting issues like a few lines not being indented correctly, you putting the opening brackets on the next instead of the same line (would not be an issue on its own, but now those two styles are mixed. And I prefer them on the same line so I'd like that to be the case) or channelTabs-selectedbadgeshide not being camelCase but those are just small issues that I could just fix myself.


Only use GCM if IS_GCM_ENABLED is true in the google plist

Description
Added a check for IS_GCM_ENABLED from the GoogleService-Info.plist If the value is false, APNS will be used instead of FCM.
Related Issue
https://github.com/phonegap/phonegap-plugin-push/issues/1770
Motivation and Context
This prevents FCM being used when it has been disabled in the plist.
How Has This Been Tested?
This has been tested locally in an existing Ionic project. All unit tests pass.
Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@geo242 hahaha, I was just looking at doing the same thing. Thanks for the PR.

This information needs to go into the documentation somewhere. This broke my push notifications moving from 1.x to 2.x


Fix presubmit.yml

https://buildkite.com/bazel/rules-graalvm/builds/8#018d2952-59ec-4171-8b75-a6c99ba4aac9

@sgammon Sorry for the breakage, if we can set up presubmit, we will be able to verify those changes!


Android: Meshes assigned by SurfaceTool.commit not "materializing"

Godot version:
Official 3.1.1-RC1
OS/device including version:
Development:
    Hardware: OpenGL ES 2.0 Renderer: Gallium 0.4 on AMD POLARIS11 (DRM 3.27.0 / 4.19.0-0.bpo.2-amd64, LLVM 3.9.1)
    OS: Debian Stretch
    uname -a: Linux caladan 4.19.0-0.bpo.2-amd64 #1 SMP Debian 4.19.16-1~bpo9+1 (2019-02-07) x86_64 GNU/Linux

Deployment:
    Hardware: Samsung Galaxy S9
    OS: LineageOS 15.1 (= Android 8.1)

Issue description:
I have a MeshInstance whose actual mesh data I manipulate using both SurfaceTool and MeshDataTool. When I finally commit the manipulated vertices back to the MeshInstance (line 30 in the Ground.gd script included in the supplied test project), the manipulated mesh shows up as expected in Godot's development output window. However, when running the project after deployment to Android, the manipulated mesh is no longer visible.
When exporting the project with line 30 being commented out, the (now, of course, unmanipulated) mesh shows up in Android as expected.
I don't believe this is just a visibility issue; rather, the committed mesh data seem to be really missing in Android. I initially encountered this issue when I was creating a collision sibling from a MeshInstance manipulated as described above and an object which was supposed to interact with the ground mesh just dropped down into negative Y nirvana...
Steps to reproduce:
Deploy the supplied test project to an Android device

as is
after commenting out line 30 in Ground.gd

Minimal reproduction project:
vrtest.zip

Seems to be working in 3.2 (tried out the official alpha2) -- hence I suggest to close this bug report.
Cheers --
Torsten

Not sure if this a bad habit when a bug report is closed a long time ago.
This seem to happen on the mobile renderer on Samsung S9+ (Android 10)
I tried both Godot 4.1.2.stable and 4.2b3.

This is indeed a bad habit :) This bug was fixed 4 years ago, the one you experience is a different issue, even if the symptoms are similar. Please open a new bug report with a reproduction project.


improved songs searching

Instead of searching only for videos in YouTube music, search for everything but get only videos and songs. We can't specify more than 1 filter when retrieving data from YouTube api.
Another solution would be to concat two lists (songs, videos) but that would increase number of requests because we have to send 2 POST requests to yt api instead of one. This would increase the chance of getting rate limited
searchSongs = ytmApiClient.search(searchTerm, filter='songs')
searchVideos = ytmApiClient.search(searchTerm, filter='videos')
searchResult = searchSongs + searchVideos

The code looks good, but please fix the CI checks.

Regression test are failing because of ffmpeg (fixed in #1200).

And I have no clue why flake8 fails, it completes successfully on my pc

Huh, interesting - the versions seem to be the same. What happens if you fix the error from the CI? Will it fail locally?

Huh, interesting - the versions seem to be the same. What happens if you fix the error from the CI? Will it fail locally?

Yup, it failed

It's funny because I didn't even change this file.

Can you run tox --recreate -e flake8? Does that change anything?

Can you run tox --recreate -e flake8? Does that change anything?

Nope

In this case, I would add ignore = E301 to [flake8] section in setup.cfg. There's no point in fighting with the linter if it goes wild. This particular error is not very important anyway.

I guess the #1200 should be merged first. The flake8 error there seems reasonable, regardless that the file wasn't changed.

Can you approve changes in #1200

Just wanted to say, very excited for this! I just made this change locally and it makes things so much better!

@xnetcat please update branch & resolve conflicts.

@Silverarmor resolved commits


[Linux] localeResolutionCallback returns null

On master channel localeResolutionCallback on Linux returns null.
MaterialApp(
           localeResolutionCallback: (locale, supportedLocales) {
                return locale;
           },
...
);

I have en_US locale on my Linux:
[✓] Flutter (Channel master, v1.15.19, on Linux, locale en_US.UTF-8)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 3.6)
[✓] Connected device (4 available)

Hi @aliyazdi75
I see there's an open issue addressing the case you described.
Please follow up on https://github.com/flutter/flutter/issues/45343,
I'm closing the current one as duplicate.
If you disagree please write in the comments
and I will reopen it.
Thank you


debouncing updateField when using mapMultiRowFields (or any other)

Hi,
I'm using the plugin currently with great success, but as we get to more complex forms I'm finding myself wanting to debounce the store events so we reduce the commits and subsequent actions, but cannot currently see a way to do this.
I already have a store.subscribe which listens to the specific store module updating, by checking for mutation type 'some/store/module/updateField', which is fine, but I'd like to debounce them for longer user input, or constant user input.
Generally debouncing via lodash requires it to be done on the input side, rather than the store side, so I don't want to do this in the store.subscribe, but I'm wondering if this is something you have considered for updateField, and if not if it could be?
Thanks!

Hello,
I thought about this but for me it never was a problem.
You can also use the .lazy modifier https://vuejs.org/v2/guide/forms.html#lazy if you're concerned about performance.
I want to keep the code of this package as minimal as possible so there won't be debouncing built in any time soon. Sorry.


Adding a script to test sqlsrv connection pooling on Linux and Mac

Coverage increased (+0.02%) to 55.447% when pulling 14c8cc83c810ba5d225b338079094af5c7a4031e on ulvii:sqlsrvConnPoolTest into 9ebc69993934bb6f6a5359acb1d38a54fdd7dcac on Microsoft:dev.

Coverage increased (+0.2%) to 55.593% when pulling 800a0eb6b2c5fd5afe1c21bcf34e2ec3b1ca9e76 on ulvii:sqlsrvConnPoolTest into 9ebc69993934bb6f6a5359acb1d38a54fdd7dcac on Microsoft:dev.

Coverage increased (+0.2%) to 55.593% when pulling 70ad77f9ce70dd7a4b9538f41b810c3644c0c121 on ulvii:sqlsrvConnPoolTest into 9ebc69993934bb6f6a5359acb1d38a54fdd7dcac on Microsoft:dev.

Coverage increased (+0.2%) to 55.593% when pulling 70ad77f9ce70dd7a4b9538f41b810c3644c0c121 on ulvii:sqlsrvConnPoolTest into 9ebc69993934bb6f6a5359acb1d38a54fdd7dcac on Microsoft:dev.

Coverage increased (+0.2%) to 55.593% when pulling 70ad77f9ce70dd7a4b9538f41b810c3644c0c121 on ulvii:sqlsrvConnPoolTest into 9ebc69993934bb6f6a5359acb1d38a54fdd7dcac on Microsoft:dev.


feat: Add distribution information for rpm

In general, the name of the rpm artifact often contains the distribution of the supported OS. For example, openresty-<IP_ADDRESS>-1.el8.x86_64.rpm shows that it's for CentOS 8. While the rpm files for apisix do not contain the distribution information and users may get confused to know which OS the apisix-2.8-0.x86_64.rpm is for.
So it would be great if we could make use of the --rpm-dist flag of fpm to add the distribution of rpm, such as: apisix-2.8-0.el8.x86_64.rpm. Furthermore, this improvement may also allow us to make one rpm artifact for each version of one distribution.
BTW: the 1 is often used as the first release version of one rpm. So do we need to change the current 0 to 1, like: apisix-2.8-1.el8.x86_64.rpm?
Any feedback is welcome. Thanks.

LGTM


Add query execution time to returned QueryResult struct

Currently the only way to "time" a query is to sit there and do it yourself, but when using HTTP, its hard to account for the network latency. If the query result struct returned the query latency along with the results, then that could just be pulled out by the client if it cared.

https://github.com/gtfierro/hod/commit/c9ec39a83cc5309f23ba87bcb660bcf75a4daa69


Consider more name change fallout from 'if' --> 'assert' rename

#80 changed the keyword from if to assert.  There are two other name changes to consider as follow-ups:

Rename the proposal from "Import Conditions" to "Import Assertions"
Rename ModuleRecord's [[Conditions]] fields to [[Assertions]]

Thoughts/objections/any other changes we're missing?

We might want to update Mozilla's position

Other parts of spec text also reference "conditions" (ConditionEntries, ConditionKey, AssertClauseToConditions...), so I'm renaming those as well in #88.

I landed #88 and posted an update to https://github.com/mozilla/standards-positions/issues/373 (if they want to change it I'll offer to submit a PR; I'm not sure if creating a PR against mozilla/standards-position as a non-Mozilla employee presumes too much 😄).  So, closing this.

@dandclark I think it's ok to do it, like I did https://github.com/mozilla/standards-positions/pull/392. I'm happy to send a PR

@xtuc Oh, I didn't notice it was done that way last time.  Looks like dbaron beat us to it though: https://github.com/mozilla/standards-positions/commit/da631527be5346a8993496e9d9f1cf36219bde12


[Bigtable] Added checkConsistency() and generateConsistencyToken() to Grpc.php

@jdpedrie @dwsupplee PTAL. Thanks!


Add Vietnamese translation

Hi @parksb, I closed https://github.com/parksb/handmade-blog/pull/47 and reopen this PR.

@parksb I updated missing translations. Can you review it?


STOR-858: Bump github.com/openshift/* and k8s.io/*

Bump OpenShift and Kubernetes libraries. @openshift/storage

/retest

/retest

/lgtm

/label docs-approved
/label px-approved
This is not user-facing change

/label qe-approved
CI is mostly green


Create dummy dataset

We need a publicly available dataset for people to import/use for testing. We want to invite everyone to work on this project, but some folks haven't quite reached the level of trust needed to be given unfettered access to member details.
Ideally, this would take the form of a set of sql dumps that could exist in HACKING/sample_data. Perhaps we could also set up a .dev instance that's 100% open (probably nuked nightly) for people to play with.

#262 allows you to scrub an existing database of all real member data. We still need a "generator" of sorts to fully synthesize data with a single command.

What if we set up the import box to produce an anonymized dataset each day, and upload it somewhere semi-public?

@WillNilges remaining work here is to anonymize the notes fields everywhere (and probably address too?)


Make API URL static to api.byteshare.io

Sub Task of
Issue: #212
Describe the solution you'd like
Make API URL to be api.byteshare.io as static URL

Changed Env variables and Configured domain in API gateway


Chore: Cleanup monorepo scripts

Adds back the beloved dev scripts for the playgrounds (among some other unrelated stuff).

Ready to merge 👍


Api request: PUBLIC/PRIVATE KEY MATCH.

Currently looking for an API to match both the private key and public key.
This is a key needed feature for the chain.

I am currently working on getting tests ready.  I will add this when I am finished.

Implemented


Added German Translation

Added German, tho i think you have to fix the duplicates.
Might also add my Other Mother Language Icelandic soon.
https://hosted.weblate.org/projects/aonsoku/translation/de/

If German translation's needed, hit me up, I can help you out there.


Remote List Retrieval

Enhancement Description

One-line enhancement description (can be used as a release note): Remote List Retrieval
Keylime Enhancement Proposal: #16
Primary contact (assignee): @mpeters
Enhancement target (which target equals to which milestone):

Alpha release target (x.y)
Beta release target (x.y)
Stable release target (x.y)

Please to keep this description up to date.

This enhancement has been implemented


Fix doc comment for ShaderModule::GetSourceFile

This PR fixes the doc comment for ShaderModule::GetSourceFile

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@flinti thanks for making the patch!


[Online-DPO] fixes to the training scripts and setup.py

What does this PR do?
update the script and run the online dpo tests

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


CASM-4821 removing WLM backup

Summary and Scope
The WLM backup step is not required during CSM upgrade process. So the hook is being removed which was implemented as a part of CASM-4548 epic.
The backups are already taken care of by the product hooks.
Issues and Related PRs

Resolves CASM-4821
CASM-4548
https://github.com/Cray-HPE/csm/pull/3384

Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[x] Target branch correct
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

The PR makes sense to me but I have one question. Even if the WLM is not being upgraded, I think we want to take a backup in case something goes wrong during the upgrade. If we are just upgrading CSM itself, we backup the wlm here. Since we are removing this hook and expecting the product (wlm) hook to take care of this, I think we are missing taking the wlm backup during upgrades where the wlm itself is not being upgraded. Do other people have thoughts about this?

We also have periodic database backups with PXC and spool directory backups with velero, so I think we're covered.


Missing trend_precompute in some trends

only two files categorical.py and statistical.py have trend_precompute

@Shine226 what info would help as a the precomputed info for regression? otherwise I'll just make it empty for that one

@brownsarahm Make it empty for now.


Add prototool

This adds prototool as an available .proto linter.
https://github.com/uber/prototool
Protoc-gen-lint is a fine linter, and I don't want to disturb the work going on there, prototool has a bunch of extra functionality.
Prototool is a toolbelt for working with Protocol Buffers - it takes care of downloading and caching protoc from https://github.com/google/protobuf/releases for you so you don't have to worry about where or whether it is installed, or what version of protoc you have, and prototool can compile, lint, and format your files on the fly. A standard prototool.yaml config file describes your build, however for most .proto files this won't be needed. It's also extremely fast - I'm able to compile and lint around 200 files in about half a second.
Here is a list of the currently available linters:
$ prototool list-all-linters
COMMENTS_NO_C_STYLE                                   Verifies that there are no /* c-style */ comments.
ENUMS_HAVE_COMMENTS                                   Verifies that all enums have a comment of the form "// EnumName ...".
ENUM_FIELD_NAMES_UPPERCASE                            Verifies that all enum field names are UPPERCASE.
ENUM_FIELD_NAMES_UPPER_SNAKE_CASE                     Verifies that all enum field names are UPPER_SNAKE_CASE.
ENUM_FIELD_PREFIXES                                   Verifies that all enum fields are prefixed with [NESTED_MESSAGE_NAME_]ENUM_NAME_.
ENUM_NAMES_CAMEL_CASE                                 Verifies that all enum names are CamelCase.
ENUM_NAMES_CAPITALIZED                                Verifies that all enum names are Capitalized.
ENUM_ZERO_VALUES_INVALID                              Verifies that all enum zero value names are [NESTED_MESSAGE_NAME_]ENUM_NAME_INVALID.
FILE_OPTIONS_EQUAL_GO_PACKAGE_PB_SUFFIX               Verifies that the file option "go_package" is equal to $(basename PACKAGE)pb.
FILE_OPTIONS_EQUAL_JAVA_MULTIPLE_FILES_TRUE           Verifies that the file option "java_multiple_files" is equal to true.
FILE_OPTIONS_EQUAL_JAVA_PACKAGE_COM_PB                Verifies that the file option "java_package" is equal to com.PACKAGE.pb.
FILE_OPTIONS_REQUIRE_GO_PACKAGE                       Verifies that the file option "go_package" is set.
FILE_OPTIONS_REQUIRE_JAVA_MULTIPLE_FILES              Verifies that the file option "java_multiple_files" is set.
FILE_OPTIONS_REQUIRE_JAVA_PACKAGE                     Verifies that the file option "java_package" is set.
MESSAGES_HAVE_COMMENTS                                Verifies that all non-extended messages have a comment of the form "// MessageName ...".
MESSAGES_HAVE_COMMENTS_EXCEPT_REQUEST_RESPONSE_TYPES  Verifies that all non-extended messages except for request and response types have a comment of the form "// MessageName ...".
MESSAGE_FIELDS_NOT_FLOATS                             Verifies that all message fields are not floats or doubles.
MESSAGE_FIELD_NAMES_LOWERCASE                         Verifies that all message field names are lowercase.
MESSAGE_FIELD_NAMES_LOWER_SNAKE_CASE                  Verifies that all message field names are lower_snake_case.
MESSAGE_NAMES_CAMEL_CASE                              Verifies that all non-extended message names are CamelCase.
MESSAGE_NAMES_CAPITALIZED                             Verifies that all non-extended message names are Capitalized.
ONEOF_NAMES_LOWER_SNAKE_CASE                          Verifies that all oneof names are lower_snake_case.
PACKAGE_LOWER_SNAKE_CASE                              Verifies that the package is lower_snake.case.
REQUEST_RESPONSE_NAMES_MATCH_RPC                      Verifies that all request names are RpcNameRequest and all response names are RpcNameResponse.
REQUEST_RESPONSE_TYPES_IN_SAME_FILE                   Verifies that all request and response types are in the same file as their corresponding service and are not nested messages.
REQUEST_RESPONSE_TYPES_UNIQUE                         Verifies that all request and response types are unique to each rpc.
RPCS_HAVE_COMMENTS                                    Verifies that all rpcs have a comment of the form "// RPCName ...".
RPC_NAMES_CAMEL_CASE                                  Verifies that all RPC names are CamelCase.
RPC_NAMES_CAPITALIZED                                 Verifies that all RPC names are Capitalized.
SERVICES_HAVE_COMMENTS                                Verifies that all services have a comment of the form "// ServiceName ...".
SERVICE_NAMES_CAMEL_CASE                              Verifies that all service names are CamelCase.
SERVICE_NAMES_CAPITALIZED                             Verifies that all service names are Capitalized.
SYNTAX_PROTO3                                         Verifies that the syntax is proto3.
WKT_DIRECTLY_IMPORTED                                 Verifies that the Well-Known Types are directly imported using "google/protobuf/" as the base of the import.

I've been effectively using this plugin for a while now https://github.com/uber/prototool#vim-integration but would love to get it on mainline!
Prototool is in extreme alpha, but I've abstracted the options away so that future changes can be easily made without breaking e.g. ale_proto_prototool_command. However any early adopters will understand this and I'll be sure to push any updates.
The formatting I'm not sure how to integrate, and would love any pointers. The general issue is that the formatter needs to be able to read the file from disk, and then write back to disk, which I'm not sure if this fits into the ALE fixer paradigm. I've included the functions I use here, but they surely can't be merged as-is.
Let me know if you have any pointers or other questions, I love your plugin and want others to be able to take advantage of what we're doing.

Closing due to inactivity.


No visibility of Vertical (horizontal) stack cards

Couldn't see vertical (horizontal) stack cards on dApp.
Dashboard in HA:

Same view on dApp (Chrome Desktop version):

And on dApp (mobile version via Nova wallet):

Same here. Neither do mushroom cards

scheduled on July milestone


Fix throttling

Throttling was always set to 1.0 because the max function was used
instead of min

Thanks for your PR! I'm no longer using DynamoDB and these utils, so it's not going to be easy for me to test this (the tests also against DynamoDB Local) - even though it's a quite small change I always want to be sure :) Have you confirmed this fix with dumps you've run, and how? Thanks :)

I still haven't been able to make throttle to run properly (did it ever work for you?), but in this case it's easy to see that "min" is the function you want to use.
If you leave it as "max", it will always return 1.0 because the allowed value is always between 1.0 and 0.01 (see lines 80 and 81):
    capacity_consumption = max(0.01, capacity_consumption)
    capacity_consumption = min(1.0, capacity_consumption)

I don't remember whether or not it worked for me.
I see that the change to min is at least required, but maybe there are other things to fix? Will look into testing it better.

I think the throttling code is right before this change. Yes capacity_consumption is constrained between 0.01 and 1.0 but that is a percentage that is mutliplied out in (capacity_consumption * total_capacity) / float(parallelism).
Running some numbers by in ipython with some test code we get, e.g.:
In [1]: # imaginary commandline parameters

In [2]: capacity_consumption = 0.5

In [3]: parallelism = 4

In [4]: # adapted from main()

In [5]: capacity_consumption = max(0.01, capacity_consumption)

In [6]: capacity_consumption = min(1.0, capacity_consumption)

In [7]: total_capacity = 1000

In [8]: capacity_per_process = max(
   ...:     1.0,
   ...:     (capacity_consumption * total_capacity) / float(parallelism)
   ...: )

In [9]: print(capacity_per_process)
125.0

125.0 read units per each of the 4 processes seems right to me for consuming 0.5 = 50% of the 1000 read units. Swapping for min would only restrict each process to 1.0 read units. The reason this max exists is so there is at least 1 read unit per process.

Ok, I understand it now. Thanks for taking some time investigating and sorry for the wrong push.


SpeechRecognizer - changing languages

I would like to implement speech-to-text for a case where there will be multiple spoken languages - one person will speak, and then afterwards someone else will speak in a different language. I want to manually change the input language when the 2nd person starts speaking. Currently, I am doing this using the following code:
speechRecognizer.Properties.SetProperty(PropertyId.SpeechServiceConnection_RecoLanguage, spokenLanguage);
speechRecognizer.Properties.SetProperty(PropertyId.SpeechServiceConnection_EndpointId, endpointId);

It generally works, however, I have noticed that when the SpeechRecognizer is initialised using a custom endpoint (setting the EndpointId on the SpeechConfig before creating the SpeechRecognizer), changing the language this way no longer works, it continues with the previous language. The language returned on the Recognizer changes, but the text returned is in the previous language. Is this a bug? Or is this not a recommended approach? Would it be better to recreate the SpeechRecognizer when changing the language? I know I could potentially use auto-detect to detect the new language, but it isn't really needed here as I already know ahead of time that the language will be changed and what it will be changed to.

Hi, what do you mean by "The language returned on the Recognizer changes"? If you read PropertyId.SpeechServiceConnection_RecoLanguage then it's just the value last written by your application. Or are you using language identification?
Using a custom endpoint, you have trained a custom speech model, right? There may be an inherent limitation on changing languages on the fly but we will need to check from the service side. You could first try if it makes a difference that you stop recognition, change the language, then start recognition again. Also, creating a new SpeechRecognizer is guaranteed to re-set the configuration and start anew.

Hi @pankopon, thanks for your reply! When I said the language returned on the Recognizer changes, I meant the value of the ((SpeechRecognizer)sender).SpeechRecognitionLanguage property on the Recognized event, that changed, and same with the RecoLanguage property. All the language-related properties return the new language, but yet it's still recognising in the original language, only when using a custom endpoint, and yes it's using a custom speech model. I have a custom endpoint for each language so I'm stopping it, changing both the language and endpoint, and then starting it again, but it stays recognising on the old language. I've just done another quick test using session ID d391602051fe4faea8ea1518830849be incase you can see anything. I changed the language from Spanish to Russian, stopped and started the recognizer, and it stayed recognizing in Spanish.
I'm not using language identification, and I could create a new SpeechRecognizer, it just seems more efficient to reuse it and just change the language and endpoint, especially if I might need to change it a few times.

Hi, SpeechRecognitionLanguage actually returns the value of PropertyId.SpeechServiceConnection_RecoLanguage so it's just the latest value you wrote and not updated based on the service response. The service only returns a language ID when language identification is enabled.
Can you record and attach a Speech SDK log from the case where you change the language? Set the log filename in SpeechConfig before you create a recognizer. As you are using a custom endpoint, even if it's done by setting EndpointId on SpeechConfig it's possible that the same limitations apply as when using FromEndpoint i.e. the recognition language may not be changed after a connection is established. With a log file we can check this.

Sure, attached is the log file.
log.txt

Thanks for the log. Looks like the language setting is sent to the service in each case but apparently the change from the initial value has no effect, presuming the response speech phrases (starting at 35100ms) after the ru-RU setting are not as expected.
[778413]: 84ms SPX_DBG_TRACE_FUNCTION:  audio_stream_session.cpp:1116 CSpxAudioStreamSession::StartRecognitionAsync
[162010]: 96ms SPX_TRACE_INFO:  usp_connection.cpp:552 connectionUrl=wss://uksouth.stt.speech.microsoft.com/speech/universal/v2?cid=2ed64348-6e92-4281-85fc-c2274ed8272d
[162010]: 174ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1250 speech.context='{"phraseDetection":{"mode":"CONVERSATION","language":"es-ES", ...
[162010]: 319ms SPX_TRACE_VERBOSE:  web_socket.cpp:540 [0x00000304E9DFB790] Web socket sending message. Time: 2024-07-26T21:28:27.9988687Z, TimeInQueue: 145ms, IsBinary: 0, Path: speech.context, Size:613 B
[162010]: 320ms SPX_TRACE_VERBOSE:  web_socket.cpp:649 [0x00000304E9DFB790] Web socket send message completed. Result: 0, SendTime: 0ms, IsBinary: 0, Path: speech.context, Size:613 B
[162010]: 366ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1881 Response: Turn.Start message. Context.ServiceTag: a9e819e6df5d44fb9a7e5030bc516023
[162010]: 8157ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1569 Response: Speech.Phrase message. Status: 0, Text: ¿Hola, cómo estás, me escuchas bien?, starts at 16500000, with duration 18800000 (100ns).
[162010]: 21178ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1569 Response: Speech.Phrase message. Status: 0, Text: Ahora puedes cambiar de tema a español, a ruso., starts at 171900000, with duration 31600000 (100ns).
[345411]: 31320ms SPX_DBG_TRACE_FUNCTION:  audio_stream_session.cpp:1138 CSpxAudioStreamSession::StopRecognitionAsync
[162010]: 31478ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1569 Response: Speech.Phrase message. Status: 5, Text: , starts at 326000000, with duration 0 (100ns).
[162010]: 31481ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1912 Response: Turn.End message.
[778413]: 31511ms SPX_DBG_TRACE_FUNCTION:  audio_stream_session.cpp:1116 CSpxAudioStreamSession::StartRecognitionAsync
[162010]: 31526ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1250 speech.context='{"phraseDetection":{"mode":"CONVERSATION","language":"ru-RU", ...
[162010]: 31530ms SPX_TRACE_VERBOSE:  web_socket.cpp:540 [0x00000304E9DFB790] Web socket sending message. Time: 2024-07-26T21:28:59.2093768Z, TimeInQueue: 3ms, IsBinary: 0, Path: speech.context, Size:647 B
[162010]: 31530ms SPX_TRACE_VERBOSE:  web_socket.cpp:649 [0x00000304E9DFB790] Web socket send message completed. Result: 0, SendTime: 0ms, IsBinary: 0, Path: speech.context, Size:647 B
[162010]: 31568ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1881 Response: Turn.Start message. Context.ServiceTag: c14fa4c7bbb34d09b32166fcedd7809a
[162010]: 35100ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1569 Response: Speech.Phrase message. Status: 0, Text: Pasiva., starts at 347200000, with duration 6000000 (100ns).
[162010]: 47103ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1569 Response: Speech.Phrase message. Status: 0, Text: Si pasita., starts at 455100000, with duration 6800000 (100ns).
[285090]: 51965ms SPX_DBG_TRACE_FUNCTION:  audio_stream_session.cpp:1138 CSpxAudioStreamSession::StopRecognitionAsync
[162010]: 52123ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1569 Response: Speech.Phrase message. Status: 5, Text: , starts at 532700000, with duration 0 (100ns).
[162010]: 52125ms SPX_DBG_TRACE_VERBOSE:  usp_reco_engine_adapter.cpp:1912 Response: Turn.End message.

However, I think the cause is what you wrote

I have a custom endpoint for each language so I'm stopping it, changing both the language and endpoint, and then starting it again

The endpoint cannot be changed on the fly after the recognizer is created and connection established (updating PropertyId.SpeechServiceConnection_EndpointId has no effect). The endpoint can be only set before a recognizer is created and this endpoint is then used for the service connection (connectionUrl shown in the log) as long as the same recognizer instance is used.
So if you indeed have language specific endpoints and thus want to change the endpoint at runtime then the only way is to create a new SpeechRecognizer. Please note that C# API documentation has the following remarks about both PropertyId.SpeechServiceConnection_RecoLanguage

Under normal circumstances, you shouldn't have to use this property directly. Instead, use SpeechRecognitionLanguage.

and PropertyId.SpeechServiceConnection_EndpointId

Under normal circumstances, you shouldn't have to use this property directly. Instead use FromEndpoint(Uri, String), or FromEndpoint(Uri).

Thanks, and yes the change had no effect, the subsequent responses were still in Spanish. In some cases, I see the language change, but 5-10 minutes later which is strange. Maybe it later reconnects and that then updates the connection url.
I saw the remarks in the documentation regarding the RecoLanguage and EndpointId properties, but I didn't think it was saying it wouldn't work, just that normally you would set the values when instantiating and when testing, changing the RecoLanguage works, just not if you are using a custom endpoint it seems.
I'll switch it to create a new SpeechRecognizer. Out of interest, the language identification feature supports using custom endpoints per language. Is this because it uses multiple SpeechRecognizers? I want to do something similar to what it supports in terms of having multiple spoken languages, but switching the language myself as it doesn't support all the languages you offer or using multiple locales for the same language.

the language identification feature supports using custom endpoints per language. Is this because it uses multiple SpeechRecognizers

It's as described in the example

var sourceLanguageConfigs = new SourceLanguageConfig[]
{
SourceLanguageConfig.FromLanguage("en-US"),
SourceLanguageConfig.FromLanguage("fr-FR", "The Endpoint Id for custom model of fr-FR")
};

If the detected language is en-US, the example uses the default model. If the detected language is fr-FR, the example uses the custom model endpoint.

You only need one recognizer instance, the rest is automatic based on the configuration as in the example.

Thanks, I know it's based on the configuration, but it supports multiple custom endpoints right? I could do something like this:
var sourceLanguageConfigs = new SourceLanguageConfig[]
{
SourceLanguageConfig.FromLanguage("en-US", "The Endpoint Id for custom model of en-US"),
SourceLanguageConfig.FromLanguage("fr-FR", "The Endpoint Id for custom model of fr-FR")
};

I'm wondering how it can switch the endpoint on a single recognizer, or it's doing it in a way that isn't available directly using the SDK?

In case of language identification this is managed by the service, not the SDK.

Thanks for your explanations!!


Conform input files to PEP8 style guide

This is necessary to avoid miniogre breaking when extracting imports from the input source code. An example of an error that is shown when miniogre process a non-PEP8 source code is:
TabError: inconsistent use of tabs and spaces in indentation
Function extract_requirements_from_code uses python ast library, which requires the input code to conform to PEP8.
Here is the command that needs to be run before calling the extract_requirements_from_code:
autopep8 --in-place --aggressive <filename>

Fixed with merge https://github.com/ogre-run/miniogre/pull/7


Failing to fetch changesets

When trying to fetch changesets it fails:
$ RAILS_ENV=production rake redmine_git_hosting:fetch_changesets --trace
** Invoke redmine_git_hosting:fetch_changesets (first_time)
** Invoke environment (first_time)
** Execute environment
** Execute redmine_git_hosting:fetch_changesets
Performing manual fetch_changesets operation...
fatal: Not a git repository: 'repositories/example.git'
Done!

Can it be that somehow the git users home directory is missing. Should it be /home/git/repositories/example.git? If so, how is the home directory being set?
Strangely enough removing stuff from the Recycle Bin works.
The Config Test page is completely green and all the repositories are shown there. The repositories exist in /home/git/repositories. In those directories I can also run git log, etc.

Strangely enough stuff like removing the repo, creating a new one, etc. appears to work. So it seems to be related to upgrading from an earlier version. However I followed the migration instruction.

Mh I think If you remove the example.git repository, you should remove it in the configfile too.

Should it be /home/git/repositories/example.git?

Nope, this path is always relative

how is the home directory being set?

From /etc/passwd file

Btw does clone work on this repo repositories/example.git?


Search by video length

Perhaps this would make sense as a pivot facet that only shows up for video items
Maybe display as a range of minutes (e.g. <10 minutes, 10min-30min, 30mins-50mins, Over 50mins)


arch/arc: Detect arcv2 TLS register automatically

The compiler defines ARC_TLS_REGNO as the number of the register used for TLS variables. Use that instead of hard-coding a specific register.
Signed-off-by: Keith Packard<EMAIL_ADDRESS>I couldn't see any other dependencies on the thread register number in the tree; with this, we should be able to switch which register is used.

It looks like this works for ARCv2 when using GNU, but why isn't the ARCv3 + GNU case changed to use the same mechanism (line 130).

ARCv3 defines register 30 for this purpose and doesn't have (or need) a symbol defined in the compiler.

Furthermore, this solution will (probably) not work for other toolchains like MetaWare, when TLS support for Metaware is added. It's not clear what 'problem' this PR solves.

When that is merged in, whatever mechanism is used to declare the TLS register can be added here.
The goal of this PR is to allow the toolchain to migrate from register 26 (the current Zephyr value) to register 25 (as used by Linux), or whatever register the toolchain reserves for this purpose without having to change this code for each toolchain.

@keith-packard thank you so much for this nice improvement. Looks really nice and elegant to me.


Watchlisted rule is missing.

Describe the bug
In the rules glossary in the documentation there is a rule called "Watchlisted by (username)", but that rule does not show up in the app.
To Reproduce
Steps to reproduce the behavior:

Go to a rule
Select first value
No "Watchlisted by (username)" or anything about watchlists in the options

Expected behavior
Available rules should match documentation
Device (please complete the following information):

OS: Ubuntu
Version 2.0.4

That rule is currently only available in the next develop image, it’s unreleased yet. Since the documentation doesn’t work with branches, the rule is already mentioned there.


Allow to deprecate enums values

Description
We should be able to deprecate enum fields :) as per spec, GraphQL allows to deprecate enum values by passing a directive: https://spec.graphql.org/June2018/#sec-Enums
Types of Changes

[ ] Core
[ ] Bugfix
[x] New feature
[ ] Enhancement/optimization
[x] Documentation

Issues Fixed or Closed by This PR

Closes https://github.com/strawberry-graphql/strawberry/issues/537

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to Mateusz Sobas for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

Thanks for contributing to Strawberry! 🎉 You've been invited to join
the Strawberry GraphQL organisation 😊
You can also request a free sticker by filling this form: https://forms.gle/dmnfQUPoY5gZbVT67
And don't forget to join our discord server: https://strawberry.rocks/discord 🔥


Update quantum version on standalone

Currently warp10-standalone is shipped with an outdated quantum, 2.6.7.
Would it be possible to update the embedded quantum to 3.x?
Thanks!

Done as part of the 1.2.13 release


define real64 constants more consistently

define real64 with real64 values. Hopefully it won't cause non-zero diff
Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Trivial change (affects only documentation or cleanup)

Checklist:

[x] I have tested this change with a run of GEOSgcm (if non-trivial)
[x] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[x] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

Well, as far as I can tell this is zero-diff. Actually sort of surprises me. I guess I figured something would get tiny different with this PR...

Well, as far as I can tell this is zero-diff. Actually sort of surprises me. I guess I figured something would get tiny different with this PR...

I am a little bit surprised too. I found the inconsistency here when I chased the in-accuracy of lat-lon corners and centers in cubed-sphere grid. maybe we should put on a label "possible non-zero-diff"?

Maybe I should try C720? I suppose if that works, then it's probably as zero-diff as we can test?


replacing occurences of 'developer' resources with 'development'

That's an interesting and important distinction, thanks. :-)


added tree sort

It takes input from user and then sorts it using tree sort and displays the sorted list.

Pls review this program.

#3

@bhaveshlohana kindly add "hactober accepted" label in my merged PR #753


Windows tools compatability issues.

Opened this in lieu of https://github.com/cloudfoundry/windows-tools-release/pull/4 which was a tad out of date (rebase issues and such).


CVEs reported 8/16/23

Address the CVEs found on the 8/16/23 Dependabot scan.

PR merged. Waiting for dev branch scan to close this out.


Use a UUID string as identifier for NSToolbar

Using a UUID string as identifier for NSToolbar, so it won't mess up toolbar if there are multiple windows.
Description

Add identifier to NSToolbar

Releated Issue

Fixes #236
Checklist (for drafts):

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] Review requested

Screenshots (if appropriate):

Yeah, I agree, it needs to go in the sidebar and animate to the content area when clicked. We've struggled to know how to do that in this custom sidebar.

One question though: do you know how to move the sidebar toggle button above the sidebar?

I've tried to move it above the sidebar, I guess when the project just started, but I hadn't figured this out. I can try to figure this out now

Yeah, I agree, it needs to go in the sidebar and animate to the content area when clicked. We've struggled to know how to do that in this custom sidebar.

Actually I don't think it has to do with the sidebar itself since it is the native implementation. (NavigationView)

One question though: do you know how to move the sidebar toggle button above the sidebar?

I've tried to move it above the sidebar, I guess when the project just started, but I hadn't figured this out. I can try to figure this out now

Just a hint though:
This doesn't work though it is how it should.

Just a hint though: This doesn't work though it is how it should be according to Apples docs.

I know, so I'll try to figure this out, and I think we will have it in this PR too.

Seems like it would be better for someone else to handle moving sidebar above the sidebar itself. I'll merge fix for #236 here and create a new issue for the sidebar toggle location


getDepositHistory added to api and readme

thanks for the input!

No problem. Do you think you could push this to npm. I have an app running on heroku.

it's published on npm.


Can't get (partial) union type on extended generics

Bug Report
🔎 Search Terms
Generic Inference
Type Narrowing
🕗 Version & Regression Information
Seems to occur in every version I tried. I reviewed the FAQ about 'Generic Inference'
⏯ Playground Link
Playground link with relevant code
💻 Code

export interface Schema {
  response?: unknown;
}

export interface SchemaError extends Schema {
  response: {
    401: string;
    403: number;
  };
}

type ResponseUnion<S extends Schema> = S['response'][keyof S['response']];

const test = <T extends SchemaError>(schema: T) => {
  // Typescript can get T shape and 401 type
  const a401 = schema.response['401'];

  // Works as expected
  type A = ResponseUnion<SchemaError>; // string | number
  const works: A = 'abc';

  // T extends SchemaError so the type should return string | number | unknown (since we can't deduce [types of T['response']] here)
  const error: ResponseUnion<T> = 'abc';

  // Intersection does not work either
  const error2: ResponseUnion<T & SchemaError> = 'abc';
};

🙁 Actual behavior
Gives the following error Type 'string' is not assignable to type 'ResponseUnion<T>'.
Can't get a union of a extended generic, even if it's guaranteed (at least I think) to be of a partial extended type.
🙂 Expected behavior
Given a extended generic return a narrowed type like: ResponseUnion<T> = string | number | unknown

This is a correct error.
const test = <T extends SchemaError>(schema: T): ResponseUnion<T> => {
  // Assertion: this is OK
  const error: ResponseUnion<T> = 'abc';
  return error;
};

// Unsound: value 'abc' inhabits binding of 0 | 'def'
const m: 0 | 'def' = test({response: { 401: 'def', 403: 0 }} as const);

Sorry, I couldn't follow that example. ResponseUnion<T> returns the types of the keys of response of a given T. Your example seems to return the keys. I think test will never return 0 | 'def', it will return types, something like string | number.
const test = <T extends SchemaError>(schema: T): ResponseUnion<T> => {
  const error: ResponseUnion<T> = 'abc'; // error is a String, since T extends SchemaError
  return error; // I am returning a string, it complies to string | number
};

// Since I am supplying a T that extends SchemaError, it shouldn't be always true that ResponseUnion<T> returns a union of string | number?
const m: string | number = test({response: { 401: 'def', 403: 0 }} as const); // Supplying a T that extends SchemaError