tcmulib: delete device blocked (kernel v4.13)

There is no userspace process listening the netlink socket, genlmsg_multicast_allns would return -3, and then would block in tcmu_wait_genl_cmd_reply until timeout.
Sep  6 09:14:40 ubuntu17 kernel: [  363.469072] INFO: task test:2044 blocked for more than 120 seconds.
Sep  6 09:14:40 ubuntu17 kernel: [  363.469098]       Tainted: G           OE   4.13.0 #1
Sep  6 09:14:40 ubuntu17 kernel: [  363.469118] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
Sep  6 09:14:40 ubuntu17 kernel: [  363.469146] test          D    0  2044   1928 0x00000004
Sep  6 09:14:40 ubuntu17 kernel: [  363.469147] Call Trace:
Sep  6 09:14:40 ubuntu17 kernel: [  363.469150]  __schedule+0x3cc/0x860
Sep  6 09:14:40 ubuntu17 kernel: [  363.469152]  ? log_store+0x1b5/0x270
Sep  6 09:14:40 ubuntu17 kernel: [  363.469154]  schedule+0x36/0x80
Sep  6 09:14:40 ubuntu17 kernel: [  363.469155]  schedule_timeout+0x1da/0x350
Sep  6 09:14:40 ubuntu17 kernel: [  363.469157]  ? vprintk_emit+0x308/0x3c0
Sep  6 09:14:40 ubuntu17 kernel: [  363.469161]  wait_for_completion+0xb4/0x140
Sep  6 09:14:40 ubuntu17 kernel: [  363.469162]  ? wait_for_completion+0xb4/0x140
Sep  6 09:14:40 ubuntu17 kernel: [  363.469164]  ? wake_up_q+0x80/0x80
Sep  6 09:14:40 ubuntu17 kernel: [  363.469167]  tcmu_netlink_event+0x480/0x4e0 [target_core_user]
Sep  6 09:14:40 ubuntu17 kernel: [  363.469168]  ? vprintk_func+0x27/0x60
Sep  6 09:14:40 ubuntu17 kernel: [  363.469169]  ? printk+0x52/0x6e
Sep  6 09:14:40 ubuntu17 kernel: [  363.469172]  ? wait_woken+0x80/0x80
Sep  6 09:14:40 ubuntu17 kernel: [  363.469174]  tcmu_destroy_device+0x1bd/0x234 [target_core_user]
Sep  6 09:14:40 ubuntu17 kernel: [  363.469190]  target_free_device+0xa4/0x100 [target_core_mod]
Sep  6 09:14:40 ubuntu17 kernel: [  363.469195]  target_core_dev_release+0x15/0x20 [target_core_mod]
Sep  6 09:14:40 ubuntu17 kernel: [  363.469200]  config_item_put+0x7c/0xa0 [configfs]
Sep  6 09:14:40 ubuntu17 kernel: [  363.469202]  configfs_rmdir+0x1f8/0x320 [configfs]
Sep  6 09:14:40 ubuntu17 kernel: [  363.469204]  vfs_rmdir+0xb7/0x130
Sep  6 09:14:40 ubuntu17 kernel: [  363.469206]  do_rmdir+0x219/0x230
Sep  6 09:14:40 ubuntu17 kernel: [  363.469208]  SyS_unlinkat+0x22/0x30
Sep  6 09:14:40 ubuntu17 kernel: [  363.469209]  entry_SYSCALL_64_fastpath+0x1e/0xa9
Sep  6 09:14:40 ubuntu17 kernel: [  363.469210] RIP: 0033:0x482164
Sep  6 09:14:40 ubuntu17 kernel: [  363.469211] RSP: 002b:000000c420274df8 EFLAGS: 00000246 ORIG_RAX:<PHONE_NUMBER>000107
Sep  6 09:14:40 ubuntu17 kernel: [  363.469212] RAX: ffffffffffffffda RBX:<PHONE_NUMBER>000000 RCX:<PHONE_NUMBER>482164
Sep  6 09:14:40 ubuntu17 kernel: [  363.469212] RDX:<PHONE_NUMBER>000200 RSI: 000000c420222630 RDI: ffffffffffffff9c
Sep  6 09:14:40 ubuntu17 kernel: [  363.469213] RBP: 00007fc2ce7fcbf8 R08:<PHONE_NUMBER>000000 R09:<PHONE_NUMBER>000000
Sep  6 09:14:40 ubuntu17 kernel: [  363.469213] R10:<PHONE_NUMBER>000000 R11:<PHONE_NUMBER>000246 R12:<PHONE_NUMBER>0000c2
Sep  6 09:14:40 ubuntu17 kernel: [  363.469214] R13: 00000000ffffffee R14:<PHONE_NUMBER>000e40 R15:<PHONE_NUMBER>000055

https://patchwork.kernel.org/patch/9805757/
@@ -1224,12 +1388,15 @@  static int tcmu_netlink_event(enum tcmu_genl_cmd cmd, const char *name,

 	genlmsg_end(skb, msg_header);

+	tcmu_init_genl_cmd_reply(udev, cmd);
+
 	ret = genlmsg_multicast_allns(&tcmu_genl_family, skb, 0,
 				TCMU_MCGRP_CONFIG, GFP_KERNEL);
-
 	/* We don't care if no one is listening */
 	if (ret == -ESRCH)
 		ret = 0;
+	if (!ret)
+		ret = tcmu_wait_genl_cmd_reply(udev);

 	return ret;
 free_skb:

Stopping tcmu-runner while devices are open is not supported. There are a lot of bugs in it still. Why are you doing it? Is it just for testing right now?
This is a known issue and on my TODO. The problem was that you can get -ESRCH when the daemon is not up and also if the netlink socket does not have enough space to send the event. If the daemon is not up, then we can just finish the removal procedure since there is nothing in user space to cleanup. If the nl socket is just congested, I thought we needed some code to to retry but I have not had time to add that.

I want to implement one daemon(using tcmulib) for one device.  When the device is removed, the daemon will exit. But until remove_device return，would reply the netlink event to kernel. I don't know how to wait the netlink event complete.  If i can reply the event in callback handler->removed or there is a remove_done callback, i think it will ok.
switch (cmd->c_id) {
	case TCMU_CMD_ADDED_DEVICE:
		reply_cmd = TCMU_CMD_ADDED_DEVICE_DONE;
		ret = add_device(ctx, buf,
				 nla_get_string(info->attrs[TCMU_ATTR_DEVICE]));
		break;
	case TCMU_CMD_REMOVED_DEVICE:
		reply_cmd = TCMU_CMD_REMOVED_DEVICE_DONE;
		remove_device(ctx, buf,
			      nla_get_string(info->attrs[TCMU_ATTR_DEVICE]));
		ret = 0;
		break;
	case TCMU_CMD_RECONFIG_DEVICE:
		reply_cmd = TCMU_CMD_RECONFIG_DEVICE_DONE;
		ret = reconfig_device(ctx, buf, info);
		break;
	default:
		tcmu_err("Unknown netlink command %d. Netlink header received version %d. libtcmu supports %d\n",
			 cmd->c_id, version, TCMU_NL_VERSION);
		return -EOPNOTSUPP;
	}

	if (version > 1)
		ret = send_netlink_reply(ctx, reply_cmd,
				nla_get_u32(info->attrs[TCMU_ATTR_DEVICE_ID]),
				ret);

Just a FYI, libtcmu as it is now is being depreciated. We are going to start working on a new one that just does the low level kernel operations like netlink, uio, etc and not handle scsi stuff like in api.c. We have not even started on it yet though, so it will not help you now :) If you want to work on it let me know and I can send links to some of the discussions about what is needed.
I am not sure what you are doing in your daemon. It sounds like maybe you want to just tell the kernel you are doing the old style interface that does not wait for user space to reply, but it is not clear to me. Is your daemon modeled after tcmu-runner's style, or consumer.c or tcmu-synthesizer.c? Could you post some of it? Code is always easier to understand :)

Thanks for your reply. My Daemon modeled like consumer.c , and  do these things:

create/remove a tcmu device.
get scsi command from libtcmu,  handle it and get data, offset.
send/recv the data to/from our own storage system by TCP/IP.

I am trying to use libtcmu and expect it to be used on our products. And i also want to improve libtcmu, please let me join your discussions about it.

Sorry for the late reply.
One question I had for you is why not just use tcmu-runner and then just make handler for it that does the send/recv to your storage system? Does tcmu-runner not provide something you need?
For the issue you are hitting with the consumer.c based design, I am not sure what the issue is. Are you exiting the daemon from the tcmulib_handler->removed function? If that is the issue then go ahead and send a patch to add a tcmulib_handler->removed_done callback so you can exit your daemon from there.
Here is a discussion about some of the requirements for the next libtcmu:
https://github.com/open-iscsi/tcmu-runner/pull/235
The new lib sounds like it will be what you need in that it will handle the lower level kernel stuff like uio and netlink similar to what consumer.c's main() is doing, so feel free to drive the process so it handles what you need in your daemon. Since there is not code, you have a lot of freedom.

Thanks for your reply again.  I find my problem, i used to call tcmu_close and ignore remove netlink event. tcmu_close will call close_device and handler->removed.
I want each device to be processed by a separate daemon, so each device would reduce mutual influence. For the new kernel， I must do these:

I start waiting netlink event(tcmulib_get_master_fd) before write create information to /sys, and then ignore the follow netlink events until i want to remove device.
I start waiting netlink event again before write delete information to /sys.
Each device create or delete must hold a global lock.

We are also adding this patch to the upstream kernel
https://www.spinics.net/lists/target-devel/msg15983.html
You can just set this for each device your daemon is managing and it will give you the old behavior back.


Handle projects where the user cannot read resources

If the user does not have resources/read permission on a project, we should indicate visually that they cannot list resources for the project. On the project page (think direct link), display a specific error message in this case.

I think if a user has read access on project but no read access on resources, permissions have been setup wrong


ERROR Install Qt 6.4.0 Mac OS 13.0

error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/install_name_tool: no LC_RPATH load command with path: /Users/mirrol/Documents/A_TEMP_DIR/lib found in: /usr/local/./Quick.app/Contents/MacOS/Quick (for architecture x86_64), required for specified option "-delete_rpath /Users/mirrol/Documents/A_TEMP_DIR/lib"
error: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/install_name_tool: no LC_RPATH load command with path: /Users/mirrol/Applications/Qt/6.4.0/macos/lib found in: /usr/local/./Quick.app/Contents/MacOS/Quick (for architecture x86_64), required for specified option "-delete_rpath /Users/mirrol/Applications/Qt/6.4.0/macos/lib"
CMake Error at /Users/mirrol/Applications/Qt/6.4.0/macos/lib/cmake/Qt6Qml/Qt6QmlDeploySupport.cmake:50 (message):
No QML imports information recorded for target Quick.  The target must be
an executable and qt_add_qml_module() must have been called with it.  If
using a CMake version lower than 3.19, ensure that the executable is
manually finalized with qt_finalize_target().  Missing file:
  /Users/mirrol/Documents/A_TEMP_DIR/.qt/deploy_qml_imports/Quick.cmake

Call Stack (most recent call first):
.qt/deploy_qml_app_Quick_57a92c478e:3 (qt_deploy_qml_imports)
examples/quick/cmake_install.cmake:52 (include)
examples/cmake_install.cmake:62 (include)
cmake_install.cmake:54 (include)

For now you can use -DFRAMELESSHELPER_EXAMPLES_DEPLOYQT =OFF to fix this error temporarily.

Thank.
When can you fix the quick example?  The QML engine failed to create component: QUrl("qrc:/Demo/HomePage.qml")

Why can't the QML engine create the component? Can you provide more debug log?

QML debugging is enabled. Only use this in a safe environment.
[2022/10/31 12:35:04.632]  QQmlApplicationEngine failed to load component
[2022/10/31 12:35:04.633]  qrc:/Demo/HomePage.qml: module "QtQml" version 1.0 is not installed
[2022/10/31 12:35:04.633]  qrc:/Demo/HomePage.qml: module "QtQml" version 1.0 is not installed
[2022/10/31 12:35:04.633]  qrc:/Demo/HomePage.qml: Failed to load dependent import "QtQml" version 1.0
[2022/10/31 12:35:04.633]  qrc:/Demo/HomePage.qml: Failed to load dependencies for module "org.wangwenx190.FramelessHelper" version 1.0
[2022/10/31 12:35:04.633]  qrc:/Demo/HomePage.qml: Failed to load dependent import "org.wangwenx190.FramelessHelper" version 1.0
[2022/10/31 12:35:04.643]  The QML engine failed to create component: QUrl("qrc:/Demo/HomePage.qml")

You need to use the macdeployqt tool to copy the Qt dependencies to the binary folder.

Or you can just drop the binary file to Qt's bin directory.

Thank!

ninja: build stopped: subcommand failed.
Log:
PS C:\Users\Alexe\Documents\A_TEMP_DIR> C:\Qt\Tools\CMake_64\bin\cmake.exe -DCMAKE_INSTALL_PREFIX:PATH="C:/Program Files (x86)\FramelessHelper" -DCMAKE_PREFIX_PATH:PATH="C:/Qt/6.4.0/msvc2019_64" -DCMAKE_BUILD_TYPE:STRING=Release -DCMAKE_CXX_COMPILER:FILEPATH="C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.33.31629/bin/HostX64/x64/cl.exe" -DCMAKE_MAKE_PROGRAM:FILEPATG="C:\Qt\Tools\Ninja\ninja" -GNinja C:\Users\Alexe\Documents\framelesshelper
-- The CXX compiler identification is MSVC 19.33.31630.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - failed
-- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.33.31629/bin/HostX64/x64/cl.exe
-- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.33.31629/bin/HostX64/x64/cl.exe - broken
CMake Error at C:/Qt/Tools/CMake_64/share/cmake-3.24/Modules/CMakeTestCXXCompiler.cmake:62 (message):
The C++ compiler
"C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.33.31629/bin/HostX64/x64/cl.exe"

is not able to compile a simple test program.
It fails with the following output:
Change Dir: C:/Users/Alexe/Documents/A_TEMP_DIR/CMakeFiles/CMakeTmp

Run Build Command(s):C:\Qt\Tools\Ninja\ninja cmTC_c460f && [1/2] Building CXX object CMakeFiles\cmTC_c460f.dir\testCXXCompiler.cxx.obj
[2/2] Linking CXX executable cmTC_c460f.exe
FAILED: cmTC_c460f.exe
cmd.exe /C "cd . && C:\Qt\Tools\CMake_64\bin\cmake.exe -E vs_link_exe --intdir=CMakeFiles\cmTC_c460f.dir --rc=rc --mt=CMAKE_MT-NOTFOUND --manifests  -- C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1433~1.316\bin\Hostx64\x64\link.exe /nologo CMakeFiles\cmTC_c460f.dir\testCXXCompiler.cxx.obj  /out:cmTC_c460f.exe /implib:cmTC_c460f.lib /pdb:cmTC_c460f.pdb /version:0.0 /machine:x64  /debug /INCREMENTAL /subsystem:console  kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib && cd ."
RC Pass 1: command "rc /fo CMakeFiles\cmTC_c460f.dir/manifest.res CMakeFiles\cmTC_c460f.dir/manifest.rc" failed (exit code 0) with the following output:
Не удается найти указанный файл
ninja: build stopped: subcommand failed.

CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
CMakeLists.txt:27 (project)
-- Configuring incomplete, errors occurred!
See also "C:/Users/Alexe/Documents/A_TEMP_DIR/CMakeFiles/CMakeOutput.log".
See also "C:/Users/Alexe/Documents/A_TEMP_DIR/CMakeFiles/CMakeError.log".

From your CMake log, I can see CMake failed to compile a simple test program, so it stopped running. It even has not began to parse this project yet. You should check what's wrong with your toolchain.

Thank!


Created SER tutorial

Description
Creates a tutorial using Senselab for SER.
Related Issue(s)
#197
Motivation and Context
We were lacking a tutorial using the functionality for audio classification, which currently has one specific implementation, speech emotion recognition, and so now novice users can better understand how to Senselab might be useful for this task.
How Has This Been Tested?
Through Colab

Thanks @wilke0818 for the tutorial! It’s helpful and does a good job of addressing real challenges that users might face. I like it, and I’m curious to see how actual users respond.
A few things are still missing to make this more complete:

Senselab installation: We should install senselab only when running in Colab. If the user is running elsewhere, we can assume they’ve already set it up. You can use a function like this:

def is_colab():
    try:
        import google.colab
        return True
    except ImportError:
        return False

if is_colab():
    %pip install senselab
else:
    print("Not running on Colab. Skipping installation.")

API for classification and SER: Add an API for audio classification and speech emotion recognition (SER). For now, this API should call the Hugging Face-based function only if the model uses Hugging Face. If other types of models are used, raise a NotImplementedError. This will make this task more in line with the others in senselab and more easily maintainable.

Documentation for classification and SER: Add documentation for these tasks. It should explain what the tasks are and link to the tutorial. You can use resources like this to get started: https://huggingface.co/tasks/audio-classification.

This branch is out-of-date with the base branch: Please, update the branch before requesting a new review.

Let me know if you need help with any of these points!

hi, @wilke0818 ! Did you have any time to work on this?

Nope. Added the tutorial change that you gave (might not have updated). Need to refactor for the API per the other issue on this topic. It is unclear to me what you would want for documentation. The functionalities themselves are documented already and this tutorial provides the information about the task a user might need (it is pretty akin to the link you sent).


How do you make sure that async fires callbacks every time?

Version: 1.x

Platform: UNIX
I use the following method at window,However, UNIX did not find a good solution.
struct uv_req_s *req = (void *)malloc(sizeof(*req));
UV_REQ_INIT(req, UV_WAKEUP);
req->data = handle;
//if (!uv__atomic_exchange_set(&handle->async_sent)) {
POST_COMPLETION_FOR_REQ(loop, req);
//}

check uv_callback

Please post questions to https://github.com/libuv/help next time, that is what it is for.


self signed certificate error when trying webdriver-manager update

This is what I get:
Updating selenium standalone
downloading https://selenium-release.storage.googleapis.com/2.44/selenium-server-standalone-2.44.0.jar...
Updating chromedriver
downloading https://chromedriver.storage.googleapis.com/2.14/chromedriver_mac32.zip...
Error: Got error Error: self signed certificate in certificate chain from https://selenium-release.storage.googleapis.com/2.44/selenium-server-standalone-2.44.0.jar
Error: Got error Error: self signed certificate in certificate chain from https://chromedriver.storage.googleapis.com/2.14/chromedriver_mac32.zip

This probably isn't a Protractor problem, but a configuration issue on your host.  You should be able to reproduce this by just visiting the URL directly:
curl -O "https://chromedriver.storage.googleapis.com/2.14/chromedriver_mac32.zip"

(Or try it in a local web browser from the same host.)  You should see the same error.
I ran into a similar problem recently and the issue was resolved by updating the trusted CA certificates on my host (the "ca-certificates" package on Debian Linux).  But, I suspect there are lots of ways to break SSL trust chains, so the problem could be different in your case.  (I'm pretty confident that googleapis.com isn't using a self-signed certificate.)

@tullmann that sounds about right, I don't think webdriver-manager can fix all configuration issues. Closing as not a general protractor issue.

It turned out to be a Kaspersky Antivirus HTTPS protection issue. Thanks for your help!

I found this can be overcome by using the --ignore_ssl option:
https://github.com/angular/protractor/issues/1477
I had previously tried manually doing a curl of the three libraries with the -k option, which did download the libraries, but that wasn't enough to get it installed as a start still wouldn't work. Judging by the console output when I ran update with the -ignore_ssl option, it looks like I simply forgot to unzip the files (and it also sets file permissions).

@esoyke I recently only used set strict-ssl false  but now it is not working. Not sure why
I was trying ignore-ssl instead of ignore_ssl
Thanks

I had same issue there. Our machine is working on windows server. I have opened cmd with as administrator. it is working like a charm now.


[HELP!!!] Any way to set 'sakoe_chiba_radius' in metric_param for tslearn.clustering.TimeSeriesKMeans

any way to set 'sakoe_chiba_radius' in metric_param for tslearn.clustering.TimeSeriesKMeans when metric is 'dtw'?
the following does not work!!! this makes me crazy for all night today!!!Help!!!
TimeSeriesKMeans(n_clusters=3, metric="dtw", max_iter=5,
...                            max_iter_barycenter=5,
...                            metric_params={"sakoe_chiba_radius": .5},
...                            random_state=0)

After closely inspecting the code, I may have found what is going on.
cdist_dtw is called within TimeSeriesKMeans. The definition is:
def cdist_dtw(dataset1, dataset2=None, global_constraint=None,
              sakoe_chiba_radius=1, itakura_max_slope=2., n_jobs=None):

Now when looking at the comments at sakoe_chiba_radius, I see:
Radius to be used for Sakoe-Chiba band global constraint. Used only if
        ``global_constraint="sakoe_chiba"``.

So global_constraint: "sakoe_chiba" must also be passed with metric_params!
@rtavenar this is pretty unclear from the docs I'd say and I had to look into the code to figure this out. It would be more user-friendly that this global_constraint is inferred whenever needed (e.g. when either sakoe_chiba_radius or itakura_max_slope is passed in the metric_params dictionary.

Hi @chinghoo
Thanks @GillesVandewiele for your prompt reply. Indeed, it is expected that global_constraint is set to "sakoe_chiba" for sakoe_chiba_radius to be used.
The thing is, as it is implemented now, distances to centroids are computed using this constrained DTW, but DBA algorithm does not have a sakoe_chiba_radius or itakura_max_slope parameter, so this is not satisfactory.
And this is also true for other estimators that use DTW, but for kNN for example, the only problem is the one raised by @chinghoo since only cdist_dtw is used there.
Finally, I am not sure how to fix things, since sakoe_chiba_radius has a default value that cannot be easily differentiated from an user-input value (no None by default).
Maybe the fix would be to set the default for these parameters as None, but document the fact that this default corresponds to specific values (the current default ones) such that we do not introduce any backward incompatibility.
If you can work on that PR, that would be cool (branch from dev if you do so).

A quick fix on master could be tu just add an example with constrained DTW to the docs for kNN estimators (not fork means since the implementation is not satisfactory, so we should not advertise for that at the moment).
And the constrained DBA should be available asap, ideally in v0.3, I can take care of that, I guess.

One other way would be, wherever metric_params is used, to automatically add the global_constraint key whenever sakoe_chiba_radius or itakura_max_slope. It's not ideal, since it would require similar code on different places, so I will think about a better solution for this and I will get back to you!

@GillesVandewiele
Thank you so much for your reply, my problem solved. Inspecting the source code is important!!

I just created a related PR. @GillesVandewiele, could you give your opinion on the choices made there?

Great @rtavenar I will do this right now!

Thanks @GillesVandewiele
This now in dev branch on github, and will be integrated (automatic constraint setting + TimeSeriesKMeans with constrained DBA) in tslearn v0.3.


Create symbolic link only work on the first layer of directory and all sub directories are repeat the first layer and again again

A brief description
I try to create a soft link of directory by command line "ln", and it is only works for the first layer of dirctory.
Expected results
for instance I have directory F:/work/test

after I use ln -s /mnt/f/work work
I expect the I can enter work and test directory.
but Actually, I can not go inside the sub directories,

Actual results (with terminal output if applicable)

(dpe) root@DESKTOP-6GTI9HG:~/virtualenvs/dpe/PayMate/work# ls
ExpressPayment  manage.py  PayMate  startup.sh  templates  work
(dpe) root@DESKTOP-6GTI9HG:~/virtualenvs/dpe/PayMate/work# cd work
(dpe) root@DESKTOP-6GTI9HG:~/virtualenvs/dpe/PayMate/work/work# ls
ExpressPayment  manage.py  PayMate  startup.sh  templates  work
(dpe) root@DESKTOP-6GTI9HG:~/virtualenvs/dpe/PayMate/work/work# ls
ExpressPayment  manage.py  PayMate  startup.sh  templates  work
(dpe) root@DESKTOP-6GTI9HG:~/virtualenvs/dpe/PayMate/work/work# cd work
(dpe) root@DESKTOP-6GTI9HG:~/virtualenvs/dpe/PayMate/work/work/work# ls
ExpressPayment  manage.py  PayMate  startup.sh  templates  work
(dpe) root@DESKTOP-6GTI9HG:~/virtualenvs/dpe/PayMate/work/work/work#

you can see the is loop, when I came in the sub directory there is same directory as it above.

Your Windows build number
OS Build 14393.187
Steps / All commands required to reproduce the error from a brand new installation
use the ln -s an directory in windows and try to go the sub directory
Strace of the failing command
the command do not failing
Required packages and commands to install
nothing

See our contributing instructions for assistance.

Are you sure you created it exactly as you wrote ("ln -s /mnt/f/work work")? Or was it maybe with a trailing slash after work like this "ln -s /mnt/f/work/ work", in linux the effect is the same, but here it's still different.

Had the same issue - the problem is indeed in that trailing slash: "ln -s /mnt/f/work work" - works well, while "ln -s /mnt/f/work/ work" creates that loop

I can confirm both the bug and the grasshoppergn's solution. The loop also messes up the TAB completiton.

I too can confirm the behavior and that it seems related to the trailing slash in the symlink. Strangely, if I cd into the new directory pwd seems to see the subdirectory correctly, but if I use touch to create a new file, it's created in the parent directory.

For what it's worth, there's a known issue with symlinks containing trailing slashes:  #650 .  It is fixed in Insider builds.  The fix will be in the next major Windows update, which is rumored to maybe be coming out within the next few months.

Duplicate of #650 which has been fixed for some time.


Replace external artist links with links to the relevant creator search on Openverse

Problem

Rather than link to the external resource for the creator, which may or may not give the user quick access to the artist/creator's content, let's replace the links and mentions of creator names everywhere with links to the relevant Openverse search.
Description

According to @zackkrida we can use the following format to build the search link:
/search/audio?searchBy=creator&q=<creator name>&source=<creator source>

We need to include the source because there could be two separate creators by the same name on different sources and they should be distinguished (we don't want to misattribute).
We could also explore making the source clearer on the artist name to help disambiguate multiple sources coming back with the same creator name representing separate people.
We could also continue to include the external link to the creator's page on the source when it exists as a separate link (?).
This probably needs some extra thought from @panchovm on the UX side to make sure we're delivering the correct information and not making things more confusing.
Implementation

[ ] 🙋 I would be interested in implementing this feature.

We could, yup! If you do, please add it to the project thread. We handled an existing issue similarly for the #377 project (specifically https://github.com/WordPress/openverse/issues/750).

I added this issue to the Prior Art section in Project thread #410. Closing this as won't do.


Allow PHP 8.0

The repository could maintain PHP8.0 compatibility, especially in its development version.

Hi,
thank you for your input! Unfortunately, we weren't able to merge your changes before the update to Sylius 1.10, and we've done both updates together because it was easier this way. So we'll have to close this PR. Anyway, thanks again, and keep up the great work!


AAD Auth Support for Managed Disks

Is your feature request related to a problem? Please describe.
We are ready to start the CLI changes for supporting AAD Auth support for exporting and importing managed disks.
Describe the solution you'd like.
Currently for cli command "az disk create" and "az disk update",
add a new option "--data-access-auth-mode", and possible values are AzureActiveDirectory and None
Additional context
We are targeting public preview by the end of April and it will be good to have the CLI support by then.

Compute

@VikasMohandoss Since there are too many contents in the Swagger link in email, could you give us the specific PR link for the change, or give us the code link of Swagger code related to --data-access-auth-mode?
In addition, because the issue was created late, we will launch the release tomorrow, so it can't catch up with the release of the sprint. We plan to release it in the next sprint, is this time acceptable?

This is the swagger link: https://github.com/Azure/azure-rest-api-specs/pull/17709

Hi Xing,
When would the release land if it is a part of the next sprint? We are
looking to launch end of month ideally.
Thanks!
Vikas
When would the next
On Wed, Apr 20, 2022 at 7:28 PM Xing Zhou @.***> wrote:

@VikasMohandoss https://github.com/VikasMohandoss Since there are too
many contents in the Swagger link in email, could you give us the specific
PR link for the change, or give us the code link of Swagger code related to
--data-access-auth-mode?
In addition, because the issue was created late, we will launch the
release tomorrow, so it can't catch up with the release of the sprint. We
plan to release it in the next sprint, is this time acceptable?
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-cli/issues/22134#issuecomment-1104642774,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMUHUPQAFDGYYHQZGJUPQDVGC4M5ANCNFSM5T5NHVQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@VikasMohandoss The release time of the next sprint is 05-24. Before that, we can provide the private package of CLI to customers for installation through edge build, is this acceptable?

Thanks Xing! That works for us.
Regards
Vikas
On Thu, Apr 21, 2022 at 6:08 PM Xing Zhou @.***> wrote:

@VikasMohandoss https://github.com/VikasMohandoss The release time of
the next sprint is 05-24. Before that, we can provide the private
package of CLI to customers for installation through edge build
https://github.com/Azure/azure-cli/blob/dev/doc/try_new_features_before_release.md,
is this acceptable?
—
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-cli/issues/22134#issuecomment-1105904818,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMUHUJCO2VZ6YXLNWWSL43VGH3YNANCNFSM5T5NHVQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Unable to do navigation - Timesync, create3 CPU

Robot Model
Turtlebot4 Standard
ROS distro
Humble
Networking Configuration
Discovery Server
OS
Windows / MAC
Built from source or installed?
Installed
Package version
unsure
Type of issue
Navigation (SLAM, Nav2 etc.)
Expected behaviour
Able to navigate to goal pose
Actual behaviour
After setting the goal pose, I the robot does not move and displays some errors. Also create3 hits 100% usage and is not time synching. (delay too high message). I tried to  reset and restart ntpd but still occurs.
Error messages
1.
Behavior Tree tick rate 100.00 was exceeded!
Timed out while waiting for action server to acknowledge goal request for compute_path_to_pose
[planner_server-3] [INFO] [1725246766.923489017] [global_costmap.global_costmap]: Received request to clear entirely the global_costmap
[bt_navigator-5] [WARN] [1725246766.954822490] [BehaviorTreeEngine]: Behavior Tree tick rate 100.00 was exceeded!
[bt_navigator-5] [WARN] [1725246766.954894165] [bt_navigator_navigate_to_pose_rclcpp_node]: Node timed out while executing service call to global_costmap/clear_entirely_global_costmap.
Message Filter dropping message: frame 'rplidar_link' at time<PHONE_NUMBER>.465 for reason 'the timestamp on the message is earlier than all the data in the transform cache'
2.  Create3 timesync is not occuring despite setting the ntpd and following all the steps to sync. it shows up as a client on the Rpi.

To Reproduce

Setup TB4
Run slam and nav2
set initial pose
set goal pose

Other notes
I am running on a virtual machine on windows. I have also been trying to set up the turtlebot4 for a few weeks not with quite a few issues so would really appreciate any help to move on with my project. Thanks

Hello @swapchal
Thank you for reaching out on the Turtlebot 4 Github page. I am very sorry to hear that you are having issues with your Turtlebot 4.
I have the following recommendations:

In terms of the Create3 not syncing are you restarting the ntpd on the webserver? Have you modified the ntp.conf on the Create3 per the instructions found here https://github.com/turtlebot/turtlebot4/issues/216 If possible, please try flashing/re-flashing the latest firmware onto the Create3 (https://iroboteducation.github.io/create3_docs/releases/overview/) Should this fail please provide the Create3 logs - https://iroboteducation.github.io/create3_docs/webserver/logs/
In terms of platform do note that currently the only officially supported setup is to have a machine running Ubuntu natively. Is this option available to you (i.e. a computer with Linux installed as its OS)?

Looking forward to hearing from you soon.
Best Regards,
Saif

Was able to sort it out thanks


Remove duplicate shuffle utilities

This PR contains the following changes:

remove duplicate shuffle functions and match NVidia implementation more closely
restructure Intel's Targetinfo.cpp and Utility.cpp to match NVidia implementation more closely
remove toGenShuffleMode
change Intel namespace to intel (lowercase)
add some copyright headers

Why is it preferred to rename Intel namespace to intel?
TargetInfo.cpp used mlir::triton::intel while other files used mlir::triton::Intel so we were not consistent to start with.  I prefer the lowercase form because is easier to remember.


Disable attributes from controller?

Hi
I have controller which returns emails, including the body of the email.
Since this attribute is very large I would like it to not return that field when fetching a list of emails.
On the other hand I definitely want i it to return that field when fetching a single email.
Is there any way i can achieve this behavior with Saule.
As an alternative, is there a way I can disable Saule for a specific method in a controller?
Kind regards
Bent Bracke

The way to do this in saule, is to create two ApiResource implementations. They can of course be subclasses of each other.
Example:
public class EmailResource : ApiResource
{
    public EmailResource()
    {
        Attribute("Subject");
        Attribute("To");
        Attribute("CC");
    }
}

public class EmailWithBodyResource : EmailResource
{
    public EmailWithApiResource()
    {
        OfType("Email");
        Attribute("Body");
    }
}

Then on the list endpoint, you use [ReturnsResource(typeof(EmailApiResource))]. On the single-item endpoint you use [ReturnsResource(typeof(EmailWithBodyResource))]. Even when both return the same model type, this will change the API response to include the attribute in one response, and not the other.

Thats awesome!
Thank you!


LeaseAwareVaultPropertySource no longer logs INFO messages when the supplied Vault location is not resolvable: Not found

After migrating to spring-boot-starter-parent 2.4.2 and spring-cloud-vault-config/spring-cloud-starter-vault-config 3.0.0 LeaseAwareVaultPropertySource no longer logs INFO messages if I intentionally add some non-existing Vault path to spring.config.import. I wanted to ask if it's by design or it's a bug.

If you would like us to spend some time helping you to diagnose the problem, please spend some time describing it and, ideally, providing a minimal sample that reproduces the problem.

If you would like us to spend some time helping you to diagnose the problem, please spend some time describing it and, ideally, providing a minimal sample that reproduces the problem.

legacy.zip
modern.zip

legacy.zip
modern.zip

Thanks a lot. The issue is caused by the logging config not being configured yet at the time the Vault interaction is happening.  You can use for now spring.cloud.vault.fail-fast=true to get feedback from the app if the startup of a Vault component fails.

Thanks a lot. The issue is caused by the logging config not being configured yet at the time the Vault interaction is happening.  You can use for now spring.cloud.vault.fail-fast=true to get feedback from the app if the startup of a Vault component fails.

Thanks a lot. The issue is caused by the logging config not being configured yet at the time the Vault interaction is happening. You can use for now spring.cloud.vault.fail-fast=true to get feedback from the app if the startup of a Vault component fails.

spring.cloud.vault.fail-fast=true is already being used but it doesn't provide any useful feedback.

Thanks a lot. The issue is caused by the logging config not being configured yet at the time the Vault interaction is happening. You can use for now spring.cloud.vault.fail-fast=true to get feedback from the app if the startup of a Vault component fails.

spring.cloud.vault.fail-fast=true is already being used but it doesn't provide any useful feedback.

As it looks right now, the lifecycle is controlled from Spring Boot and we don't have any means to do anything useful here. It would make sense to follow up in https://github.com/spring-projects/spring-boot/issues.

As it looks right now, the lifecycle is controlled from Spring Boot and we don't have any means to do anything useful here. It would make sense to follow up in https://github.com/spring-projects/spring-boot/issues.

After consulting with the Boot Team, we could work around the logging issue by obtaining a specific logger that captures startup logs and prints these later on.

After consulting with the Boot Team, we could work around the logging issue by obtaining a specific logger that captures startup logs and prints these later on.

I've opened an issue to help with this on the Boot side: https://github.com/spring-projects/spring-boot/issues/24988. I think that's a prerequisite to fixing this as you need a DeferredLogFactory, not just a Log that is deferred.

I've opened an issue to help with this on the Boot side: https://github.com/spring-projects/spring-boot/issues/24988. I think that's a prerequisite to fixing this as you need a DeferredLogFactory, not just a Log that is deferred.


[F] Add dashboard

Current list of collections is empty. Don't merge until API returns list of collections across communities.

By "collections across communities", do you mean all collections the user has access to? (aka viewer.collections). If so, that'll be out very shortly.

Yep! That's good to hear, thank you Alexa 😄


NPE in AbstractSQLQuery when no connection provided

If I create an SQLQuery without a connection like this
SQLQuery query = new SQLQueryImpl(new PostgresTemplates());
and later forget to clone with a connection I get and NPE when calling uniqueResults. Could you add a null check and throw a more informative exception alerting the user to the problem.
The exception I see is:
Exception in thread "main" java.lang.NullPointerException
at com.mysema.query.sql.AbstractSQLQuery.iterateSingle(AbstractSQLQuery.java:434)
at com.mysema.query.sql.AbstractSQLQuery.iterate(AbstractSQLQuery.java:367)
at com.mysema.query.sql.AbstractSQLQuery.iterate(AbstractSQLQuery.java:360)
at com.mysema.query.sql.AbstractSQLQuery.uniqueResult(AbstractSQLQuery.java:576)
...

Released in 2.2.4


Upgrade systemlink

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Describes the workflows to upgrade or migrate SystemLink Server in single node and multi node configurations in a manner that reduces risk of unplanned downtime.
Why should this Pull Request be merged?
Upgrades go wrong all the time, and we need to encourage our customers to follow practices to minimize risk.
What testing has been done?
At this time all upgrade scenarios expect ones using PostGreSQL and --change-file-store-root argument flag have been completed. I expect I'll have a chance to test both of those before this PR goes in.

Do we still want to include someone from tech comms on these reviews?

@JD-Robertson yes, I've reached out to GitHub admins to get Melissa (tech comm) access. She plans on getting a review in by the end of the week.


Feature Request: Flag to strip  from output

I'm using markdown-toc with markdown-it, and currently markdown-it doesn't support HTML style comments. So, at least when using this from the command line, is there a way to surpress the  and  in the output file when using the -i tag?
I figure your project is easier to change, since markdown-it is nothing but regexes.

Oops githib's markdown stripped these, but i meant to type these:
<!-- toc -->
<!-- tocend -->


Some files seem to miss in<EMAIL_ADDRESS>and 0.4.1

Hi @arqex ,
I'm a member of cdnjs. We are going to host this library.
But there are some files is missing from npm in v0.1.0 and v0.4.1, and I can't get them from GitHub repo too.
So, where can I find the files in v0.1.0 and v0.4.1?
If you have any suggestions, please let me know.
Thank you.
cdnjs/cdnjs#11082

The referenced cdnjs PR has been merged so I assume this can be closed.

Thank you. :)


Error Loading script

the following errors come up when attempting to load Stable Diffusion with your xyz plot script:
Error loading script: vue.min.js
Traceback (most recent call last):
File "C:\dif\stable-diffusion-webui-master\modules\scripts.py", line 69, in load_scripts
compiled = compile(text, path, 'exec')
File "C:\dif\stable-diffusion-webui-master\scripts\vue.min.js", line 1
/*!
^
SyntaxError: invalid syntax
Error loading script: xyz_grid.py
Traceback (most recent call last):
File "C:\dif\stable-diffusion-webui-master\modules\scripts.py", line 71, in load_scripts
exec(compiled, module.dict)
File "C:\dif\stable-diffusion-webui-master\scripts\xyz_grid.py", line 16, in
from modules.hypernetworks import hypernetwork
ModuleNotFoundError: No module named 'modules.hypernetworks'
Error loading script: xyz_grid.template.html
Traceback (most recent call last):
File "C:\dif\stable-diffusion-webui-master\modules\scripts.py", line 69, in load_scripts
compiled = compile(text, path, 'exec')
File "C:\dif\stable-diffusion-webui-master\scripts\xyz_grid.template.html", line 10
--label-col-width: 220px;
^
SyntaxError: invalid decimal literal

So, there are two issues here:

The SD WebUI loads any file as a Python file and tries to run it. This includes the HTML template file. I'm working on a PR to fix this.
You need to update your WebUI to the latest version (git pull), to fix this error: ModuleNotFoundError: No module named 'modules.hypernetworks'

I'll leave this open for now until the script loading is fixed.

This should be resolved if you pull the latest version of AUTOMATIC1111's WebUI - issue fixed in this commit: https://github.com/AUTOMATIC1111/stable-diffusion-webui/commit/6f7b7a3dcdc471ebe63baa8a7731952557859c5b
Closing.


use https instead of http for polygon url

When going through the instructions I noticed the command only worked for the polygon example site if https was used in place of http. This PR updates the url in the README.

Thanks! The build failure is due to an old node version and I'll fix that separately.


Initial commit for injector code (Requires Rust nightly)

Signed-off-by: Jon Doron<EMAIL_ADDRESS>

Hi so I'll just write here in one place the answers to your questions

The reason it was close to the EXE is because in the beginning I got confused and was under the impression TLS callbacks are RVA and not VA, but I figure it's nicer that it's close...
As for the TLS getting called each time, few things I had in mind about this

I was hoping the DLL will "unpatch" or re-patch the TLS to point to it's own implementation, the injection part is already complete (so I really prefer to keep the shellcode stupid like it is now... and let the DLL do whatever it wants at this point like
repatching the TLS if needed... WDYT
Also I was under the impression the EXE is getting unmapped and then mapped, so this TLS patch will be gone soon

Yeah, I was thinking that most likely when everything goes as expected and the real TLS callback unmaps the exe, the hook can be expected to only be ran once. Just that if it for some reason fails and exits, the injected DLL initializing multiple times would cause really nonsensical errors. But I guess we can protect against that by having the DLL having a boolean preventing the initialization function being ran multiple times too.
So this looks fine to me, but I'll leave rest of the work wrt. this to tec if he had something else he wanted to check before merging =)

Thank you for the contribution!


Option to get Certificate directly from Azure Key Vault

With the new certificate capabilities 2.1, it would be very useful if Kestrel could get certificates directly from Azure Key Vault. This is especially true because Key Vault can auto-renew certificates from providers already.
If done with Managed Service Identities, this can be transparent as well.

This is something we should think about being part of https://github.com/aspnet/AzureIntegration/. There's a work item about integrating with KeyVault.


UISeeds - Message + Alert + Toast Components

Transfer the AlertBox component from UIC to UISeeds. (Audit Link) (Notions Docs Link) (Figma)
AlertBox already has V2 styling. We should make a few changes to the component on transfer:
Three use cases:

Message (i.e. "FYI must be signed in")
Alert (i.e. "Invalid form input")
Toast (i.e. "Listing updated")

Set of semantic colors that are paired with a suggested icon if the icon is enabled (so would pass semantic color + icon enabled defaults to suggested based on semantic color + potential custom icon to override the default)

Based on our convo in core sync, it seems like the designer that authored the proposal should QA these component development tickets -- @slowbot in this case I think!

@slowbot oh yeah, my bad. I should have updated the docs once I merged in the PR. Will get on it.


[IOTDB-3656] Loader Tool for TsFile

6498

 Quality gate: Failed
16 Issues

 1 Vulnerabilities
 13 Code issues
 2 Anti-patterns

Details

 Quality gate: Failed
16 Issues

 1 Vulnerabilities
 13 Code issues
 2 Anti-patterns

Details


Refactor RequestHandler

While v4 has been in development, nearly everything has been thrown into the RequestHandler. Now all functionality is embedded, it's time to refactor the calls to everything into their own functions/classes.

This has been nicely refactored now. Can still be improved, but not everything is within the single function.


For Datasets

Hello @ericardomuten can you provide me the datasets link you have used in this repo??Thanks for this wonderful work u have shared in github.

For ECAL and Quark-Gluon datasets, you can contact Prof. Sergei from ML4Sci and ask him for permission. I will immediately share it with you once you get his permission. And for the LArTPC dataset, you should ask the original author of the paper directly as I also got mine from his (the dataset isn't from ML4Sci and we don't have the right to distribute it).
Glad you're interested in this kind of work! Let me know if you have any other questions.


Doesn't build on 8.11

I can't seem to get this repository to build against 8.11.0. I checked out:c33e66c8f2924449c7b98aab108d97b5ee105bab per @Janno 's suggestion but i still get the following.
$ make -j3
CAMLDEP src/munify.ml
coq_makefile -f Make -o Makefile
CAMLOPT -c -for-pack Unicoq src/munify.ml
CAMLOPT -c -for-pack Unicoq src/unitactics.ml
CAMLOPT -pack -o src/unicoq.cmx
CAMLOPT -a -o src/unicoq.cmxa
CAMLOPT -shared -o src/unicoq.cmxs
COQC theories/Unicoq.v
File "./theories/Unicoq.v", line 7, characters 0-27:
Error: while loading
/home/gmalecha/bedrock/mtac-testing/unicoq/src/unicoq.cmxs:
The module `Int_misc' is already loaded (either by the main program or a previously-dynlinked library)

make[1]: *** [Makefile:678: theories/Unicoq.vo] Error 1
make: *** [Makefile:327: all] Error 2

As you'll hopefully see on Zulip I've made an off-by-one error and gave you the commit just before the one I should have given you: d1ff6f05a1740543623374b94a07ca1abfbd4ea9.
However, while that commit will fix universe issues, it will not fix what you have encountered here. I've never seen this error, I think. I would advise cleaning up the directory and building without -j, just in case. If the error persists you'll have to ask the Coq developers. Somebody might have seen similar errors.
I should note that I mostly use OCaml 4.07.1. Which version are you using? 4.08 and up will not work with Mtac2, as far as I know (for reasons that haven't had time to investigate yet). Not sure if the same applies to Unicoq.

There is a tag for 8.11 (v1.3-8.11). However, as Janno pointed out, that won't help you here. There is an long standing issue with certain versions of OCaml that I was never able to grok. In any case, we should fix it, so let me know the details of your setup please.

@gmalecha did you still have this issue? I just run into this issue here, but after a make clean and make it worked :shrug:

Closing it until we have more evidence that the problem is still occurring.

I think (part of?) the problem should be fixed by #47 but I don't remember exactly :(


Roll engine fdd427241b77..f62ab83bae34 (7 commits)

https://github.com/flutter/engine/compare/fdd427241b77...f62ab83bae34
git log fdd427241b7790a1ad86794725b311245ed3b470..f62ab83bae34690998eb2c8de13ec7397294078f --no-merges --oneline
f62ab83ba Roll src/third_party/dart f0da03ca36..bc99f1ca28 (1 commits)
7b38aef03 Roll src/third_party/skia 33906ad1e26f..5155e09d1466 (4 commits) (flutter/engine#8609)
b0dedbd60 Roll src/third_party/dart d9c40462c2..f0da03ca36 (1 commits)
f88ba6e43 Roll src/third_party/skia e88d4382e1cf..33906ad1e26f (1 commits) (flutter/engine#8604)
53c0905b1 Roll src/third_party/skia 652b007a4cbb..e88d4382e1cf (4 commits) (flutter/engine#8603)
4266f8583 Add desktop shell unittests to test script (flutter/engine#8600)
8156268f0 Roll src/third_party/dart 8e73b3c006..d9c40462c2 (4 commits)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (cbracken@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5901837461880832

Commit queue failed; closing this roll.


"https://api.openai.com/v1/chat/completions": x509: certificate

Post "https://api.openai.com/v1/chat/completions": x509: certificate has expired or is not yet valid

Post "https://api.openai.com/v1/chat/completions": x509: certificate has expired or is not yet valid

gfw

Closing this due to lack of comments for an extended period.


Question: Circular log file support

Would it be useful to take in a max_log_file_size_kb that ensures the log file size doesn't grow beyond that and is used in a circular mode?
We are interested in collecting debug logs as those seem to be most useful in problematic situations, but always configuring envoy to collect them is impractical due to the volume.

Which logs are we talking about? Application logs or access logs?

Not sure what the difference is between the 2 you are referring to.
I am referring to the log which is generated by envoy as part of the "log-path" command line argument

One is HTTP/filter access logs, one is the Envoy app log. You are talking about the Envoy app log. @ggreenway @jmarantz any thoughts on this? My feeling is this would be better accomplished with logrotate or similar to keep Envoy simple in this regard?

I agree with using logrotate or similar. There are several unix tools that can be used, with somewhat rich configuration. I don't think we should try to re-implement those in envoy.
Alternately, you could have envoy log to stdout/stderr, and pipe to something that directly manages the data how you want.

+1

Thanks. Wasn't aware of these tools. I'll look into setting them up with envoy output log.


Update Sifchain token logo

Moved in repository. Should we pin this to a specific commit, or prefer that it's kept up to date by that repo assuming it doesn't move again?
Note restake.app dynamically pulls in chain-registry data such as this. I'll keep an eye out for any issues

@tombeynon could we just move it into this repo?  By creating an /images folder inside sifchain folder, and saving the logo here


[SPARK-45926][SQL] Implementing equals and hashCode which takes into account pushed runtime filters , in InMemoryTable related scans

What changes were proposed in this pull request?
Implementing equals and hashCode in the InMemoryBatchScan and  InMemoryV2FilterBatchScan so that the pushed runtime filters are taken into account.
Why are the changes needed?
These test classes need the change so that the issue of re-use of exchange not happening in AQE when runtime filters are pushed to scan levels,  can be reproduced. Currently either the tests are not actually validating the reuse of exchange happening when dynamic pruning expressions are pushed to scan levels in partitioned key based joins, or they are false passing because the equality of the test scans are not taking into account runtime filters.
Does this PR introduce any user-facing change?
No
How was this patch tested?
This is test class change., It is possibe that the existing tests may fail till the dependent fixes are made in source code.
Was this patch authored or co-authored using generative AI tooling?
no

Previous comments:
Do you happen to know why we didn't catch the following? Do you think you can provide a test coverage for that?
If these classes implement equals and hashCode taking into account the pushed runtime filters, we would see that TPCDS Q14b which should ideally be reusing the exchange containing Union , is not happening due to multiple bugs which surface in AQE.
@dongjoon-hyun I think the reason for not catching the issue of reuse of exchange is a mix of multiple things
Spark is not testing with any concrete DataSourceV2 implementation. ( like iceberg)
The simulation of DataSourceV2 impl using InMemoryTableScan is buggy because of equals / hashcode not taking into account pushed runtime filters, as a result any reuse of exchange bug would not be caught ( i.e mismatch of cached exchange plans would not be detected, giving a false assurance of re-use0
If I am not wrong, the tpcds tests are run using Hive as DataSource and not sure if it supports push down of runtime filters.
The bug in AQE only shows in TPCDS if table are partitioned and equi join involves partitioning column. I am not sure if right now various tpcds tests use partitioned table or not.
Yes I have been able to reproduce the issue using InMemoryTableScans as DataSourceV2 impl for tpcds tests. I will checkin a prototype test for reproducing the bug using q14b and if needed all other queries of tpcds can be run.
Though I ought to point out that while running my test I also hit the issue of computeStats being called twice ( which throws error only in testing). I have not debugged that... yet. And not sure if the assertion of computeStats occuring only once is maintainable..
added bug test in PR https://github.com/apache/spark/pull/43824


Add https://heroicons.dev/ to the GitHub description

Hi there, can you add https://heroicons.dev/ to the GitHub description too? Here:

thanks!

Cleaning up here, closing this since it's no longer relevant. Thanks though!


Layout lexer translation from purescript-language-cst-parser

Tested by hand, but the tests run to slow to commit it as the default lexer, we need some refactoring befor that is doable


purchaseUpdatedListener is called multipletimes on ios platform

Please use Discussion board if you want get some help out of it. Please report issue if there is a bug.

using same code as provided in the  docs
Version of react-native-iap
^6.0.2
Version of react-native
0.63.3
Platforms you faced the error (IOS or Android or both?)
ios
Expected behavior
purchaseUpdatedListener should run only after purchase is completed
Actual behavior
purchaseUpdatedListener is called once a new user is navigated to the subscription page with previous purchases
Tested environment (Emulator? Real Device?)
real device
Steps to reproduce the behavior
run the app of ios device , make a purchase , logout and login again with other user

Have you checked if your component is not rerendering?
I am suspecting if this is a similar issue with https://github.com/hyochan/react-native-audio-recorder-player/issues/313

Have you checked if your component is not rerendering?
I am suspecting if this is a similar issue with hyochan/react-native-audio-recorder-player#313

this is not for recorder @hyochan  : )

Have you checked if your component is not rerendering?
I am suspecting if this is a similar issue with hyochan/react-native-audio-recorder-player#313

this is not for recorder @hyochan : )

Of course I know that. My mistake since I have not make it clear enough.
I wanted you to double check if this is not rerendering issue

@hyochan ,I am facing the same issue here (its not related to rendering issue).

Hi,
We are also facing same issue for IOS. Below are full details on the issue:

We purchase a product or subscription using react-native-iap integration. Currently issue seems to be reproducing for subscriptions
We receive the purchase event in purchaseUpdateListner and we send the purchase details to our backend process to make the purchased service available. We create a transaction entry in our DB for the record.
Success is returned to app and we complete the transaction using finishTransaction method
We do receive the subsequent renewable events in purchaseUpdateListener which are processed similarly. DB entries are made for those too.
On next launch of the app, we are again getting the original transaction which was processed and completed successfully.
We are still getting events for transaction which were processed 1-2 months back.

Is there any way to correctly finish those transactions?
Also, for how long we will be receiving old transactions? Is there any expiration time limit on these?

Hi,
I'm also facing same issue for IOS.
when i come to purchase screen purchaseUpdateListner is running multiple times
i'm not getting any solution for the same
if there is any thing that is should know , please let me know,

It seems that I've got the same, even that listeners are initialized only once.

hello. the same issue happens also for auto-renewable subscription products on Android on Sandbox. since 2nd event for the same purchase is triggered right after the first one (2-3 milliseconds later) so it makes the situation more difficult to put a check point on either mobile app code or back-end code. I'm still working on it not to process the same purchase data more than once on back-end.

@Akashdeep312 can you please help us check if this is still an issue on 8.x.x for iOS?
I have found the following as something that is expected from the native SDK: https://stackoverflow.com/a/3633571/570612
If it still happens on 8.x.x and the above doesn't fix. We'll reopen. Thank you


about testing

Hello, thank you very much for your work. I have read your article and tried to reproduce code.This is the screen that appears after I input testing code：

building.npy  was generated

I'm a novice,Can you give me some advice？
1.How do I use building.npy
2.How do I obtain the curve graph during the training process and the charts in your paper
Thank you!

Hi @Shiqi-7-7,
To check the results of the simulation, I recommend you to examine env_infos.
For example, if you want to see the detailed results from the 1st rollout, you can check as follows:
import numpy as np

paths = np.load('../stats/building/long_10.npy', allow_pickle=True)
print(paths[0]['env_infos'][-1])

closing this issue due to inactivity.


ci: enforce shellcheck and shfmt

this doesn't work yet Oren 💀

https://github.com/pacstall/pacstall-programs/commit/89b4ee87c211e0a92afc08c61e20d5f9dbad033c
...

It doesn't work if we remove it. I've tested it.

ok what about this one https://github.com/koalaman/shellcheck-precommit

and https://github.com/pecigonzalo/pre-commit-shfmt
seems like they were just split out

solved with https://github.com/pacstall/pacstall-programs/pull/4965


Handle empty slices correctly

Passing NULL to slice::from_raw_parts is invalid.

@bilelmoussaoui Something broke with the GTK version in the images

@bilelmoussaoui Something broke with the GTK version in the images

It is caused by GDK_EVENT_LAST, see #864

If you rebase, the CI should be happy