backend openDAL Backend cannot be loaded: scheme is not enabled or supported

I am getting this error when I try to use onedrive and gdrive. I assume it is the case for others too..
backend openDAL Backend cannot be loaded: Unsupported (permanent) at , context: { scheme: onedrive } => scheme is not enabled or supported
repository = "opendal:onedrive"
password = ""

[repository.options]
access_token = "token"
root = "/root/path"

Here's what OpenDAL says about onedrive
https://docs.rs/opendal/latest/opendal/services/struct.Onedrive.html
When I change it to opendal:gdrive i get the same thing.
But if I try b2 or s3, it just gives me an error regarding the bucket being invalid (since I didnt provide any credentials for it). That suggests to me that onedrive, gdrive and others are simply broken - either within OpenDAL itself or, more likely, how it was integrated into Rustic

I tried patching in OpenDAL OneDrive backend and using it. But there was no documentation about how to get the access_token value needed for it. After prompting them, here is some guidance how to do it, but it requires using some other library:

https://github.com/apache/opendal/issues/4715#issuecomment-2157753501

Then I tried using GNOME's built-in OneDrive support via FUSE. This failed because there are memory leaks :laughing: reported here: https://gitlab.gnome.org/GNOME/gvfs/-/issues/742

The saga continues. I eventually got FUSE/GVFS/OneDrive backup done, by doing it chunk by chunk and restarting GVFS daemon before it leaked too much. But now I have repository corruption that cannot be resolved with rustic repair 😆
TL;DR: I found that rclone is a much better way to work with OneDrive, it is wayy faster than GVFS was. Although I think I have to redo the backup from scratch due to pre-existing corruption.
The documentation for rclone was a bit scarce, but I needed to run rclone config and add repository = "rclone:<rclone-remote-name>:<relative-path>"
More info on rclone OneDrive support here: https://rclone.org/onedrive/

@nickchomey We are trying to prepare a patch version 0.8.1 soon and I think we can integrate the fix there.

Yes, rclone works great.

@nickchomey Thanks for your suggestion.
In fact, out of the 60 services, there are many which most likely will never be interesting as backup storage. And for those we actually don't want to compile support as that would just bloat the binary sizes without value
Moreover, as opendal designed the services as opt-in, there is no easy way to just just use all or maybe only opt-out non-needed services. Acuatally even in their self-provided CLI oli they specify each service directly, see https://github.com/apache/opendal/blob/main/bin/oli/Cargo.toml


Rearrange the scripts in the form of a module

@smaranjitghose Can you elaborate a little bit on this issue?

@smaranjitghose Can you elaborate a little bit on this issue?

@smaranjitghose I'll like to work on this, kindly assign this to me.

@smaranjitghose I'll like to work on this, kindly assign this to me.

@smaranjitghose Can you elaborate a little bit about this issue? Means do you mean that we have to make a folder and then add all the scripts in that?

Please check out how to build python modules and libraries

@smaranjitghose Can you elaborate a little bit about this issue? Means do you mean that we have to make a folder and then add all the scripts in that?

Please check out how to build python modules and libraries


new check for STAT table in static fonts

Added to the Universal Profile
com.google.fonts/check/STAT_in_statics
Static fonts with more than a single entry per design axis cause trouble on Windows
(issue #4149)

@felipesanches going forward, can you please request for someone's review before merging?

@felipesanches going forward, can you please request for someone's review before merging?

As a maintainer I never felt the need to do so. But if you feel like reviewing my PRs of new universal profile checks, I can start requesting it. And it sounds reasonable as the universal profile is our common ground.
For vendor-specific new checks, though, I feel it would be reasonable for me to have my own code-review policy among my Google Fonts peers (and you with the Adobe folks similarly).

The Ms Spec has a few examples at the end of the STAT chapter. In example 2, it looks like a single font can actually have more than 1 axisValue, https://learn.microsoft.com/en-us/typography/opentype/spec/stat#example-2-family-using-non-variable-fonts-for-different-weight-values-plus-italic.
I agree with reviews on hard stuff (this STAT stuff imo isn't trivial).

@felipesanches Thank you.
It's fine to merge without a review if you're fixing a typo, updating release notes or any code that doesn't have a broad impact on the codebase. But I definitely would like to be able to chime-in when the changes affect any of the code that is not specific to Google Fonts.

For vendor-specific new checks, though, I feel it would be reasonable for me to have my own code-review policy among my Google Fonts peers (and you with the Adobe folks similarly).

Agreed.

It's even more complicated than the above, if you have a family which has weight, width and italic. I imagine you need 3 axisValues per font.

That's what my implementation does.

Sorry ignore me.

No worries, @m4rc1e. Indeed, when I first read the original issue, I had interpreted it as you though I had implemented the check. But during implementation I noticed that broader case you just mentioned, and included that into my implementation.


Scaling costs

Is your feature request related to a problem? Please describe.
Some effects (namely krelics and pandora) can be stocked an used repeatedly, and with enough viewers can prevent a run from finishing
Describe the solution you'd like
A possible option to make effects (toggleable per effect) scale based on # of times used (i.e., +10% per use, or +X per use)
Describe alternatives you've considered
Cooldowns work, but lock things out for the duration, which can potentially cause a softlock/reset situation.
Additional context

I planned some kind of scaling system from the start, but then it turned out that the bot was the most straight forward way to handle currency. For now cooldowns are the recommended solution. Heavy help can prevent softlocks. In the future there may be a script for the chat bot that accompanies the tool for Khaos.

Command cost scaling has been introduced in 1.0.0


Can not show the colorbar

Requirement already satisfied: legendkit in d:\miniconda3\envs\dataanalysis\lib\site-packages (0.3.4)
Requirement already satisfied: matplotlib in d:\miniconda3\envs\dataanalysis\lib\site-packages (from legendkit) (3.9.2)
Requirement already satisfied: contourpy>=1.0.1 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (1.2.1)
Requirement already satisfied: cycler>=0.10 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (0.12.1)
Requirement already satisfied: fonttools>=4.22.0 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (4.53.1)
Requirement already satisfied: kiwisolver>=1.3.1 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (1.4.5)
Requirement already satisfied: numpy>=1.23 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (1.26.4)
Requirement already satisfied: packaging>=20.0 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (24.1)
Requirement already satisfied: pillow>=8 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (10.4.0)
Requirement already satisfied: pyparsing>=2.3.1 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (3.1.2)
Requirement already satisfied: python-dateutil>=2.7 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from matplotlib->legendkit) (2.9.0)
Requirement already satisfied: six>=1.5 in d:\miniconda3\envs\dataanalysis\lib\site-packages (from python-dateutil>=2.7->matplotlib->legendkit) (1.16.0)
but the code and others like this can not show the colorbar:
from legendkit import colorart
from legendkit import cat_legend, vstack, hstack
_, ax = plt.subplots(figsize=(1, 3)); ax.set_axis_off()
m = np.random.randn(10, 2)
mappable = ax.pcolormesh(m, cmap="cool")
cbar = colorart(mappable, ax=ax, title="Colorart")
legend = cat_legend(**args, title="Legend")
vstack([legend, cbar], spacing=10, title="Stack Legend and Colorart",
alignment="left", loc="out right center", frameon=False, ax=ax)
and
from legendkit import colorart
_, ax = plt.subplots()
m = np.random.randn(10, 10)
mappable = ax.pcolormesh(m, cmap="cool")
colorart(mappable, ax=ax, loc="out right center")
The marsilea bag will also have this problem。

This is the same issue as mentioned in Marsilea-viz/marsilea#42. This dark mode will change matplotlib styles. You can either submit an issue to the dark mode maintainer or change the styles yourself.

This is the same issue as mentioned in Marsilea-viz/marsilea#42. This dark mode will change matplotlib styles. You can either submit an issue to the dark mode maintainer or change the styles yourself.

Hi, I have found a solution. Using %matplotlib tk in Jupyter instead of %matplotlib notebook and %matplotlib inline can effectively avoid this problem.

这与Marsilea-viz/marsilea#42中提到的问题相同。此暗模式将更改 matplotlib 样式。您可以向暗模式维护者提交问题，也可以自行更改样式。

These problems only occur when using PyCharm 2024.1.4, which is caused by DIE


Add ConditionalOrders Service

Instead of buying/selling at current price, users should be able to CRUD a conditional trade order, that can also execute when they are away.
The semantics can be like following:
Service ConditionalOrders

CreateConditionalOrder (returns id)
ListConditionalOrders (that only belong to user)
DeleteConditionalOrder (takes id)
CheckOrders(only called from server)

The Condition will be the price of the crypto and the amount will be the asset you are giving(checked with a oneof field). For example:
$ ./client conditional create buy btc 37000 50k
will buy 50k worth of bitcoin when its price is equal/below 37000.

$ ./client conditional create sell btc 38000 1
will sell 1 btc when the price hits 38000.

To prevent over-subscription, as soon as the conditional order is placed, the resources will be taken from the user's account and preserved in the new table, tied to the order. Cancelling a pending order will return the unit back to user's bank.
Every time the ticker updates the current price, it will send an rpc to CheckOrders, which will search for orders that satisfy the new price, and execute them. (If every time is too much, a different frequency can be used).
Orders will always execute with the price in their contract and not from the current price. E.g. if you give sell order at 38k, but the new price was 40k in the next ticker update, it will still sell from 38. Tough luck...
For the leaderboard, the assets that are tied to a conditional order will also be taken into account while calculating the final worth.
Since this is a learning game , a different metric can show the total asset value tied to pending conditional orders that the exchange has at any time, which will teach traders who the actual winner is :)

Cloud Run can't run things in the background so probably wontfix for now. Clients can implement this using the API.

Yes, clients can implement this for sure.
But which part of this is a background job ? Are you referring to the time, when there is no traffic to the server so it turns off ?
Otherwise this is a running service that watches the ticker endpoint in a goroutine and sends some rpcs to trade service.

But which part of this is a background job ? Are you referring to the time, when there is no traffic to the server so it turns off ?

Yes. Executing the conditional order, as I understand it, requires a trigger listening on the prices, which we'd need to do run in the background outside the context of any requests.


11 configure cicd

Resolves #11
Merging #9, #10 and #12 will make this PR easier to review.

Docker image fails due to some frontend dep. failing to build (babel). That should be fixed with, eeeh, some time passed

I think it's better to squash all those awkward commits on merge....

Looks good, but should be rechecked when all previous requests will be merged

Current dependencies on/for this PR:

main

PR #9

PR #10

PR #12

PR #13   👈

This comment was auto-generated by Graphite.

Please, resolve merge conflicts

There are 3 warning during docker build. Example:
#26 53.19 warning: function `main` is never used
#26 53.20  --> cli/src/lib.rs:1:4
#26 53.20   |
#26 53.20 1 | fn main() {println!("if you see this, the build broke")}
#26 53.20   |    ^^^^
#26 53.20   |
#26 53.20   = note: `#[warn(dead_code)]` on by default

Looks like lib.rs file was not copied before the actual build phase. Please, find out the reason

There are 3 warning during docker build. Example:
#26 53.19 warning: function `main` is never used
#26 53.20  --> cli/src/lib.rs:1:4
#26 53.20   |
#26 53.20 1 | fn main() {println!("if you see this, the build broke")}
#26 53.20   |    ^^^^
#26 53.20   |
#26 53.20   = note: `#[warn(dead_code)]` on by default

Looks like lib.rs file was not copied before the actual build phase. Please, find out the reason

I think 2 out of 3 warnings are part of the expected behavior, since the warnings appear before the project files are copied.

Or at least, this is the same behaviour that in the main bob repo....

(better message to avoid confusion perhaps?...)
The 3rd warnings was about some dead code in release build (fixed)

Ok, didn't notice, that the error is on the first phase. Then it is better to make lib.rs files empty on the first phase, because they don't need main function inside. Here (https://github.com/qoollo/bob-management/blob/main/dockerfiles/alpine/Dockerfile#L36) and here (https://github.com/qoollo/bob-management/blob/main/dockerfiles/alpine/Dockerfile#L39) you can write, for example, just a comment: echo "// if you see this, the build broke" > backend/src/lib.rs

I think it would be good to add linter checks for frontend. This can be done by running yarn lint in frontend directory


TLS support

Hiya, does this support using TLS to connect to scylla? If not, can we add a feature to enable that functionality?

For the main crate session is passed from the app, so there we don't need to do anything. However, within charybdis-migrate we create session in crate itself, and currently it doesn't support TLS. If you are interested, PR would be welcomed.

Will PR this in if we need it for what I'm working on.

completed with https://github.com/nodecosmos/charybdis/releases/tag/v0.7.2


MM-21153 Count mentions in DM channels using mention_count

See https://github.com/mattermost/mattermost-redux/pull/1013
Ticket Link
https://mattermost.atlassian.net/browse/MM-21153

Updated commit of mattermost-redux to https://github.com/mattermost/mattermost-redux/commit/c52a797f5e7865c6628d7fcd0aaec3da8c27e3d0.
Will push separate commit for 1.26 - https://github.com/mattermost/mattermost-redux/commit/3fac9150fac5f8157817215173db827334a94c9b


tb_counter  = len([log for log in os.listdir(os.path.expanduser('~/logs/')) if 'coco_' in log]) + 1

Could someone explain what the above code mean in the Python code? I got an FileNotFoundError:
[WinError 3] The system cannot find the path specified: 'C:\Users\Yufeng2/logs/'

it count how many directories with "coco_1" prefix in the name, this prevent to overwrite the logs from older trainings.

You need to create a folder named logs to save the log files.

Thx.


New version: vim.vim.nightly version 9.1.0675

Pull request has been created with WinGet Releaser :rocket:
Microsoft Reviewers: Open in CodeFlow

/azp run


Move „add money“ below products

… or better, make it configurable.

I close this for now until the open questions are resolved.

Whoops, forgot about this one. We're using a tiny 7" 16:9 screen with a relatively low resolution, so the „add money“ thingie uses almost a third of the screen area.

I came to the conclusion that the section is just too large for what it does. I already experimented with putting small buttons below the credit information on the left. I wasn't happy with it. My decision right now is to keep it on the right above the products but give it a redesign to make it slim and handy.

The new add money section needs to feature a way to add and remove custom amounts of money.

I hope the new condensed version introduced in v2.2.0 helps already. I keep this issue open, because further improvements need to be made.


Add a linkedin icon + link in the socials bar (where the insta, discord, GitHub, icons are at)

oh no ! there seems to be a missing social thing here plus icon.
This would be a good task for one of the newly onboarded developers to take on to briefly learn.
Huly®: UTRP-308

Link: https://www.linkedin.com/company/longhorn-developers


vc4graphics.cfg: build DRM_VC4 as a module

Somewhere inside the raspberry code this is expected to be built as a module.
Since i don't know better i assume that is for a reason. But if not at least this misleading error gets removed from the log:
You have a RaspberryPi, but the vc4 Module is not loaded! Set 'dtoverlay=vc4-kms-v3d' in /boot/config.txt and reboot.

This is sway output? Found something there and it looks like checking for vc4 support by checking module list. Probably raspbian is using vc4 as module. I think this is some kind shortcut for developers and should be done properly by querying device.
Anyway I don't have any objections to make it as module. But I'm hoping that screen is early initialized.

https://github.com/swaywm/sway/issues/6241

@MarkusVolk You can try to apply this patch on sway.

I've seen that your change already was accepted upstream. I'll build sway from current code


Bug Report: when playing a song with speed lower then one it randomly stops the song

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86_64)
What version of the game are you using? Look in the bottom left corner of the main menu.
<IP_ADDRESS>
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
having the speed rate in freeplay lower than one
for this example I used milf at 0.7

then a little bit past the halfway mark of the song it just stops
Please describe your issue. Provide extensive detail and images if possible.
basically, this will work on ANY song I just used milf as an example, but the issue is, you can't fully complete the song when the speed is lower then 1.0 due to the song stopping where the song would stop if it had 1x speed
if this doesn't help, I'm sorry. I seriously am not the best at explaining

Already did a pull request to fix that. If you want you can copy the code. I'm not sure if it works in 1.7 but in 1.8+ it does!
https://github.com/KadeDev/Kade-Engine/pull/2858


param tests failing

The tests at test_GtagMP_internal_param_functions.py were failing, this  PR fixes them by adding the necessary functions
didn't wanted to delete anything in case it was going to be used in the future

Hey, @rptomey I think you added this, could you take a look?


RuntimeError: dictionary changed size during iteration

This happens when you have model in config file but later deleted model from Unet-trt folder
*** Error loading script: trt.py
    Traceback (most recent call last):
      File "G:\auto_quick\stable-diffusion-webui\modules\scripts.py", line 382, in load_scripts
        script_module = script_loading.load_module(scriptfile.path)
      File "G:\auto_quick\stable-diffusion-webui\modules\script_loading.py", line 10, in load_module
        module_spec.loader.exec_module(module)
      File "<frozen importlib._bootstrap_external>", line 883, in exec_module
      File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
      File "G:\auto_quick\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\scripts\trt.py", line 10, in <module>
        import ui_trt
      File "G:\auto_quick\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\ui_trt.py", line 16, in <module>
        from model_manager import modelmanager, cc_major, TRT_MODEL_DIR
      File "G:\auto_quick\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\model_manager.py", line 219, in <module>
        modelmanager = ModelManager()
      File "G:\auto_quick\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\model_manager.py", line 35, in __init__
        self.update()
      File "G:\auto_quick\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\model_manager.py", line 67, in update
        for base_model, models in base_models.items():
    RuntimeError: dictionary changed size during iteration

---
Exception ignored in: <function ModelManager.__del__ at 0x000001799BA029E0>
Traceback (most recent call last):
  File "G:\auto_quick\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\model_manager.py", line 87, in __del__
    self.update()
  File "G:\auto_quick\stable-diffusion-webui\extensions\Stable-Diffusion-WebUI-TensorRT\model_manager.py", line 67, in update
    for base_model, models in base_models.items():
RuntimeError: dictionary changed size during iteration

This happened to me as well. The extension worked the first time but when i tried to reinstall it this is the issue that kept appearing each time i launch stable diffusion

i found my error. it was about missing tensor rt from config mismatch file naming
i fixed myself

I found the issue causing this. I added a fix for this in hot_fix branch.

actually i just did a pull request haha
https://github.com/NVIDIA/Stable-Diffusion-WebUI-TensorRT/pull/44


Loading into VRAM

Is there a way to load model into vram and use it on-demand?
For example i download file and it instantly gets sent into loaded model so that processing time is not spent into loading model every time
In regular whisper it whisper.load_model

You load the model, and if you don't unload it, you can still use model.transcribe.


Getting BPM value in Serial MOnitor

Is there any way to print the BPM values in Serial Monitor??

Just download and install pulseSensor playground library in Arduino Ide and oper example section under file then select Getting_BPM_to_Monitor program to display bpm values in serial monitor

Please use the Pulse Sensor Playground library. This code is deprecated.
https://github.com/WorldFamousElectronics/PulseSensorPlayground
In the doc below, you can find useful commands to access data in the library
getBeatsPerMinute() is a function you can call that returns the most recent BPM value.
https://github.com/WorldFamousElectronics/PulseSensorPlayground/blob/master/resources/PulseSensor Playground Tools.md


ReliableOrdered bug

Here should used DateTime. UtcNow . Otherwise, it will cause the time to be calculated incorrectly, and it will not send data again.
if (packetFound)
            {
                currentPacket.TimeStamp = DateTime.Now;
                _peer.SendRawData(currentPacket.Packet);
                NetUtils.DebugWrite("[RR]Sended");
            }

Oops)


切换IOS平台编译报错

系统：macOS Mojave 10.14.4
描述：在Unity 2019.1及2018.4上切换IOS平台有编译错误
Assets/LoxodonFramework/Scripts/Framework/Binding/Reflection/ProxyFieldInfo.cs(134,33): error CS0103: The name 'Expression' does not exist in the current context

Assets/LoxodonFramework/Scripts/Framework/Binding/Reflection/ProxyFieldInfo.cs(135,32): error CS0103: The name 'Expression' does not exist in the current context

Assets/LoxodonFramework/Scripts/Framework/Binding/Reflection/ProxyFieldInfo.cs(136,32): error CS0103: The name 'Expression' does not exist in the current context

Assets/LoxodonFramework/Scripts/Framework/Binding/Reflection/ProxyFieldInfo.cs(137,33): error CS0103: The name 'Expression' does not exist in the current context

Assets/LoxodonFramework/Scripts/Framework/Binding/Reflection/ProxyFieldInfo.cs(138,30): error CS0103: The name 'Expression' does not exist in the current context

错误原因应该是 ENABLE_IL2CPP 没生效 手动配置ENABLE_IL2CPP后编译正常
运行Databinding Tutorials示例 后出现错误
2019-08-16 11:38:22.420 [ERROR] Binding - An exception occurs when the target property is updated.Please check the binding "Text{binding text Expression:vm => Window (Loxodon.Framework.Tutorials.DatabindingExample).localization.GetFormattedText("databinding.tutorials.description", new [] {vm.Account.Username, vm.Username}) Mode:OneWay }" in the view "DatabindingExample[Window]".exception: System.NotSupportedException: Expressions of type System.Object[] are not supported.
  at Loxodon.Framework.Binding.Expressions.ExpressionVisitor.Visit (System.Linq.Expressions.Expression expr) [0x00179] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Expressions/ExpressionVisitor.cs:110
  at Loxodon.Framework.Binding.Expressions.EvaluatingVisitor.<InvokeMethod>b__7_0 (System.Linq.Expressions.Expression a) [0x00000] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Expressions/EvaluatingVisitor.cs:891
  at System.Linq.Enumerable+SelectIListIterator`2[TSource,TResult].ToArray () [0x00034] in <1b13ba6391c74847bbc3eddc86df7eee>:0
  at System.Linq.Enumerable.ToArray[TSource] (System.Collections.Generic.IEnumerable`1[T] source) [0x0001f] in <1b13ba6391c74847bbc3eddc86df7eee>:0
  at Loxodon.Framework.Binding.Expressions.EvaluatingVisitor.InvokeMethod (System.Func`2[T,TResult] invoke, System.Collections.Generic.IEnumerable`1[T] arguments) [0x00001] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Expressions/EvaluatingVisitor.cs:891
  at Loxodon.Framework.Binding.Expressions.EvaluatingVisitor.VisitMethodCall (System.Linq.Expressions.MethodCallExpression expr) [0x0009f] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Expressions/EvaluatingVisitor.cs:951
  at Loxodon.Framework.Binding.Expressions.ExpressionVisitor.Visit (System.Linq.Expressions.Expression expr) [0x000fe] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Expressions/ExpressionVisitor.cs:77
  at Loxodon.Framework.Binding.Expressions.EvaluatingVisitor.Evaluate (System.Linq.Expressions.LambdaExpression expr, Loxodon.Framework.Binding.Expressions.Scope scope, System.Object[] args) [0x0005c] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Expressions/EvaluatingVisitor.cs:996
  at Loxodon.Framework.Binding.Expressions.EvaluatingVisitor+<>c__DisplayClass13_0.<VisitLambda>b__0 (System.Object[] args) [0x00000] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Expressions/EvaluatingVisitor.cs:971
  at Loxodon.Framework.Binding.Proxy.Sources.Expressions.ExpressionSourceProxy.GetValue () [0x0000f] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Proxy/Sources/Expressions/ExpressionSourceProxy.cs:59
  at Loxodon.Framework.Binding.Proxy.Sources.Expressions.ExpressionSourceProxy.GetValue[TValue] () [0x00001] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Proxy/Sources/Expressions/ExpressionSourceProxy.cs:69
  at Loxodon.Framework.Binding.Binding.<UpdateTargetFromSource>b__25_0 () [0x001c5] in /Users/xxxxx/Documents/workspace_unity/loxodon-framework/Assets/LoxodonFramework/Scripts/Framework/Binding/Binding.cs:302
UnityEngine.Debug:LogError(Object)
Loxodon.Log.LogImpl:Error(Object) (at Assets/LoxodonFramework/Scripts/Log/DefaultLogFactory.cs:160)
Loxodon.Log.LogImpl:ErrorFormat(String, Object[]) (at Assets/LoxodonFramework/Scripts/Log/DefaultLogFactory.cs:174)
Loxodon.Framework.Binding.Binding:<UpdateTargetFromSource>b__25_0() (at Assets/LoxodonFramework/Scripts/Framework/Binding/Binding.cs:379)
Loxodon.Framework.Execution.Executors:RunOnMainThread(Action, Boolean) (at Assets/LoxodonFramework/Scripts/Framework/Execution/Executors.cs:164)
Loxodon.Framework.Binding.Binding:UpdateTargetFromSource() (at Assets/LoxodonFramework/Scripts/Framework/Binding/Binding.cs:209)
Loxodon.Framework.Binding.Binding:UpdateDataOnBind() (at Assets/LoxodonFramework/Scripts/Framework/Binding/Binding.cs:126)
Loxodon.Framework.Binding.Binding:.ctor(IBindingContext, Object, Object, BindingDescription, ISourceProxyFactory, ITargetProxyFactory) (at Assets/LoxodonFramework/Scripts/Framework/Binding/Binding.cs:76)
Loxodon.Framework.Binding.BindingFactory:Create(IBindingContext, Object, Object, BindingDescription) (at Assets/LoxodonFramework/Scripts/Framework/Binding/BindingFactory.cs:56)
Loxodon.Framework.Binding.Binders.StandardBinder:Bind(IBindingContext, Object, Object, BindingDescription) (at Assets/LoxodonFramework/Scripts/Framework/Binding/Binders/StandardBinder.cs:42)
Loxodon.Framework.Binding.Contexts.BindingContext:Add(Object, BindingDescription, Object) (at Assets/LoxodonFramework/Scripts/Framework/Binding/Contexts/BindingContext.cs:181)
Loxodon.Framework.Binding.Builder.BindingBuilderBase:Build() (at Assets/LoxodonFramework/Scripts/Framework/Binding/Builder/BindingBuilderBase.cs:218)
Loxodon.Framework.Binding.Builder.BindingSetBase:Build() (at Assets/LoxodonFramework/Scripts/Framework/Binding/Builder/BindingSet.cs:50)
Loxodon.Framework.Tutorials.DatabindingExample:Start() (at Assets/LoxodonFramework/Tutorials/Scripts/DatabindingExample.cs:230)Assets/LoxodonFramework/Scripts/Framework/Binding/Reflection/ProxyFieldInfo.cs(138,30): error CS0103: The name 'Expression' does not exist in the current context

在Unity2018版本，在IOS平台，默认使用IL2CPP时，ENABLE_IL2CPP是无效的，这应该是Unity3d的一个bug，已经改用UNITY_IOS宏来替换，bug已经修复。


Find staple with given sequence

A useful feature when working both from the source json .sc and the interface would be, in the interface, to be able to highlight a strand which has a given sequence.
Example usecase: in a design I receive from a colleague with a few hundres of staples, 2 of them are of length 31. I can easily get their sequence in the .sc file but really hard to figure out where they are exactly on the screen in the interface.

Repeat of #529


How to set num_training_steps in lr_scheduler properly

Usually I call something like this to set the scheduler
from transformers import get_linear_schedule_with_warmup
scheduler = get_linear_schedule_with_warmup(
        optimizer, num_warmup_steps=warmup_step, num_training_steps=num_training_steps
    )

And the num_training_steps usually equals to
t_total = int(len(train_dataloader) * num_epochs) ## len(train_dataloader) = num_batches

If I use accelerator, should I change the num_training_steps to something like this? And how to understand this:
I believe the len(train_dataloader) = batch size at each device
t_total = int(len(train_dataloader) * num_epochs // accelerator.num_processes)

All the above operations are before accelerator.prepare

No, Accelerate will take into account the number of processes for you, as long as you do send your lr_scheduler to the Accelerator.prepare method. You can thus use:
t_total = int(len(train_dataloader) * num_epochs) ## len(train_dataloader) = num_batches

then maybe update needed here: https://github.com/huggingface/accelerate/blob/b0f8189d34fa42821ca041e2cba161db864c76b5/examples/complete_nlp_example.py#L141

then maybe update needed here:
https://github.com/huggingface/accelerate/blob/b0f8189d34fa42821ca041e2cba161db864c76b5/examples/complete_nlp_example.py#L141

I guess that means we don't need to update that

Yep! Will do so today, thanks for the heads up!

Hi @muellerzr , I don't quite understand, I thought the sample sent by @takiholadi  has no accelerator.num_processed. Why we should revise that

maybe also need to update: https://huggingface.co/docs/accelerate/main/en/concept_guides/gradient_synchronization

Hi @muellerzr , I don't quite understand, I thought the sample sent by @takiholadi has no accelerator.num_processed. Why we should revise that

Since:
https://github.com/huggingface/accelerate/blob/main/src/accelerate/scheduler.py#L31-L32

@allanj correct, everything and anything that has to do with gradient accumulation accelerate will now handle for you. Just pass in the gradient_accumulation_steps arg and make no changes to your code, as if you weren't using gradient accumulation at all 😄

Thanks. I was actually also trying that. But somehow run into an issue when I combine this functionality (https://huggingface.co/docs/accelerate/usage_guides/gradient_accumulation) with clip_grad_norm
with torch.cuda.amp.autocast(enabled=bool(args.fp16)), accelerator.accumulate(model):
    output = model(**batch)
    loss = output.loss
    total_loss += loss.item()
    accelerator.backward(loss)
    accelerator.clip_grad_norm_(model.parameters(), args.max_grad_norm)
    optimizer.step()
    scheduler.step()
    optimizer.zero_grad()

I didn't copy the error. But the error happens in the line of gradient clipping, showing something similar to below
unscale() has already been called...

@allanj everything and anything that has to do with gradient accumulation accelerate will now handle for you. Just pass in the gradient_accumulation_steps arg and make no changes to your code, as if you weren't using gradient accumulation at all 😄

@allanj  I had the same problem, did you solve it ?

Sorry. I don't quite have the impression about how I fixed it.


Use docker v.2 instead of docker-py (v.1), first attempt

Really first attempt, based on changes from docker-py (v 1.x) to docker (v. 2.x) specified in issue #274.
Seems to work, but I made this change without deep understanding of source code in dockworker.py. Now I am testing it on server with pool of 20 workers, each of us can see it on jupymike server.Will leave it running for at least 24 hours.

Not so important :-)


Add tx-decoding to safe-apps review modal.

Split safe-apps review modal in several files.
Add steps to the modal so the user can navigate each tx.
Add stand-alone components so they can be reused later contract-interaction

Single tx unsuccessfully decoded.

Single tx successfully decoded.
a.

b.

Multiple txs unsuccessfully decoded.

Multiple txs successfully decoded.

🤔

Whatever was edited in the "Advanced options" is lost if you visit the details of a contract interaction:
https://www.loom.com/share/ef1ed4894c6a4b1ab7920ef9394082f7

For the Idle app, trying to make a deposit:
One of the methods doesn't have a name. I cannot be sure if it simply doesn't have a name or we cannot decode it.

In the first instance we should check if we are failing to decode the name of the method
If the issue is not in our side and the name of the method is coming empty or undefined we should probably define a placeholder name for these cases

| One of the methods doesn't have a name
That's because the backend was not able to decode it, there is not design for empty names, I think that's fine leaving it empty.

| Whatever was edited in the "Advanced options" is lost if you visit the details of a contract interaction:
Contract interaction was not part of this issue, I didn't change anything there, so it should be reported in a different ticket.

@nicosampler a couple of questions regarding the feature behavior:

Is it intentional to re-request the decoded data every time we go back from the details to the review modal?

From my perspective, it's a bit counterintuitive that the X in the details modal closes the whole tx modal. WDYT?

@fernandomg @fernandomg can you check again, those problems should be solved now.

The re render of the modal was fixed
The values in the advance settings are being stored so you don't lose them when you enter into the actions' details
Looks good to me


[OPENJDK-2722] Split the tests inside openjdk_s2i.feature to couple them with appropriate modules

Could you please review the changes?
As the hsperfdata is related to jvm performance log I have moved it to modules/jvm and rest other to modules/s2i/core/

This PR is for ubi9 but OPENJDK-2722 is for ubi8. The parent issue, OPENJDK-2177, is for ubi9 but is closed -- we closed it in the last release. Ive created a new sub-task for UBI9 to track the follow-on work here, OPENJDK-2837: please add that issue name to the title and URI to PR description. Thanks!


Fix a build error when using Ruby 2.6 runtime

The master brach of RuboCop has been dropped Ruby 2.6 runtime support. https://github.com/rubocop/rubocop/pull/11791
This PR fixes the following build error:
Run cd ../rubocop && bundle install --jobs 3 --retry 3
Your RubyGems version (<IP_ADDRESS>) has a bug that prevents `required_ruby_version` from working for Bundler.
Any scripts that use `gem install bundler` will break as soon as Bundler drops support for your Ruby version.
Please upgrade RubyGems to avoid future breakage and silence this warning by running `gem update --system 3.2.3`
Fetching gem metadata from https://rubygems.org/..........
Resolving dependencies...
Could not find compatible versions

Because every version of rubocop depends on Ruby >= 2.7.0
and Gemfile depends on rubocop >= 0,
Ruby >= 2.7.0 is required.
So, because current Ruby version is = 2.6.10,
version solving has failed.

https://github.com/rubocop/rubocop-ast/actions/runs/4857086508/jobs/8657259864

Thank you


Orphaned console rows in Set table

Hi
I'm using a global filter .UseFilter(new AutomaticRetryAttribute { Attempts = 0, LogEvents = true }) on my jobs.
When a job fails, it goes to the "Failed" section on the dashboard. At this time, the console:xxxx rows in the Set sql table have a null ExpireAt value.
When I delete the job from the dashboard, the ExpireAt is still null.
It seems that this results in the rows never getting cleaned up? I had 2 million console:xxx rows in the table.
If the job fishes succesfully, then the ExpireAt gets a value.
Do I need to do something differently to get the rows cleaned up automatically? Or should I make my own job that cleans them on a regular basis?

In the ConsoleApplyStateFilter this code is in the OnStateApplied:
        if (context.OldStateName != ProcessingState.StateName)
        {
            // Job was not in 'Processing' state, ignore.
            return;
        }

But that does not take into account the flow for Processing => Failed => Deleted.
I think that means that if a job is failed, and then deleted from the dashboard, then the ExpireAt is never set for the Set and Hash table rows? Is that correct?

Yeah, you're right. Fixed in 1.3.10.

Thank you. That fixed the issue.


Update operation_utils.py - Added docstrings

Hi @fchollet @chenmoneygithub ,
I have Added docstrings to the required functions. Kindly review and let me know if there are any modifications to be made.

Hi @fchollet ,
I have formatted the docstring as per your input and it has cleared all the checks. Kindly review and let me know if there are any changes to be made.

Hi @fchollet ,
Thanks for being patient with me. I have updated the changes. Kindly review and advise.

Hi @AakashKumarNain ,
Thanks for helping me out. Can you please check if the docstring is correct now?


[Not an issue] Very confused and clueless newbie developer wondering how session data is maintained with RemGlk

Hi there,
I apologise for asking such a trivial and possibly completely stupid (a. k. a. clueless) question.
I've been reading all information I could find on the whole Glk ecosystem in search of what I thought that would be a reasonably simple question: in environments with multiple users playing a remotely stored game (each playing by themselves, of course, not simultaneously), how does RemGlk know which data to send to each?
Essentially what I wish to do is to launch RemGlk with a game file, wrap it into some middleware that communicates via stdin/stdout, and present a front-end for that middleware, where, say, users would log in, have their own profiles, who knows, a forum to discuss progress on the game — and an area where they can continue to play their current game. There is just one RemGlk process running. What information must the middleware pass to RemGlk so that it knows it's talking with 'Player X' (possibly at generation N) and not 'Player Y' (possibly at generation M)?
When looking at all the available information, I'm scratching my head in confusion. Apparently, deep inside the GlkOte code, there are references being made to a sessionID, which could be what I'm looking for. However, these are just mentioned in passing and not used by anything that I could find.
The best source for some clues on how to achieve this seems to be the remote-if-demo, which profusely scatters sessionIDs all over the place and even kindly lets you know which one is yours (when you launch the demo game). While I currently am unable to set this environment up, as well as being quite unfamiliar with Python (and not really making the effort of becoming proficient in it...) in any case, the Remote IF Demo clearly states (or doesn't it?) that for each player a new game is launched until memory is exhausted. In other words, as far as I can understand the code, the sessionID exists only at the middleware side of things (the Python code); it is not passed to RemGlk at all (or is it?), but rather for each player a different instance is forked; the Python code assigns each randomly generated sessionID to a specific player and, in turn, assigns that sessionID to a specific instance.
At least in theory, it should be possible to pre-launch, say, a dozen instances of RemGlk, and allow 12 different players to be playing at the same time; then it would be a matter of juggling these instances around, as new players come in, others quit or abandon the game, and so forth. Such code has not been developed but is certainly a theoretical possibility. However, even taking into account that RemGlk has a very small footprint (merely 450K or so on macOS Big Sur) which would be page-shared in memory across multiple identical forks anyway, it seems a rather messy way to set up such an environment.
I suppose I was expecting to have some kind of 'current state' pointer (into what theoretically can be seen as being a finite state machine), and, somehow, assign each player to the current state they're in; that way, every time RemGlk is contacted, you'd pass that pointer to it, and the VM would continue to run from that point onwards. Different players would just be on different states, but knowing the pointer to their state should be enough to configure the state machine accordingly; thus, the Glk engine would not really have a concept of 'different players', just of different states, and have an internal graph depicting state changes.
Now I'm aware that you can load RemGlk with a save file every time it runs, thus, for all purposes, 'initialising' the whole game for a specific player until a certain state is reached, wait for some input, switch to the next state, and save the whole game again. That's a possibility, but it seems to me that it's not a good idea for a system that is supposed to simultaneously serve several dozens (thousands?) of simultaneous users. I'm pretty sure it has its use in very specific areas, but I don't think it applies to my personal case.
So... all I'm left with seems to be the 'multiple instance forking' solution, and — possibly — worry about a system allocating a pool of RemGlk-enabled VMs according to demand. Is that correct? Am I seeing the whole picture here, or overseeing a simple and obvious solution?
(Note that an alternative would be embedding the whole C code inside my own code, and running it on multiple threads, as opposed to fully-blown, independent processes. That would at least save a lot of resources. However, I'm afraid that this will be quite a bit beyond my own meagre abilities.)
Thank you in advance for any insights!

Hi! Sorry that this is messy. RemGlk is the result of thinking of a lot of possible use cases, most of which never got beyond the prototype stage.
I think what you're tripping over is the idea of different users having different games but the same RemGlk process. This isn't going to work because the underlying VM is already written with the assumption of "one player per process". That is, if you look at the Glulxe source code, it keeps the VM state in global variables. So when you link that against RemGlk, there's no point in having a way for multiple players to connect.
(You might want multiple players to connect to the same game. That is, a shared display state that everybody can see and anybody can send a command to. This is workable if you assume that every viewer is up to date -- that is, they're all on the current generation -- except that if someone joins, they've got nothing and require a full update.)

Now I'm aware that you can load RemGlk with a save file every time it runs, thus, for all purposes, 'initialising' the whole game for a specific player until a certain state is reached, wait for some input, switch to the next state, and save the whole game again. That's a possibility, but it seems to me that it's not a good idea for a system that is supposed to simultaneously serve several dozens (thousands?) of simultaneous users.

On the contrary, if I understand your situation, it's meant for exactly that. The idea would be to have a web app that maintains a session for each player, and then launch RemGlk+VM every time a command comes in, using the -singleturn argument. That is, RemGlk always runs as a momentary handler -- it's never blocked waiting for input.
With this setup, the scaling is by the number of commands received per second, not by the number of simultaneous users. In theory, this should be much better. But, as I said, it hasn't been tried in real life.

Hm! 🤔
/me *scratches head* 🤨
O-kay. I think I've grokked it; as said, I haven't delved too deep into the Glulxe code — enough to think that it might only support a single player (1 player = 1 VM), but I wasn't sure if that was really the case or if I was just missing something very obvious. Thanks so much for the clarification! 🙏
In a way, it makes my middleware code so much easier, since instead of worrying about handling multiple connections to one instance of RemGlk+VM, I just fork a new process and connect stdin/stdout to it. Essentially I just need to track down which savefile is associated with each player. I'm aware that's a very rough way of describing it, but I think it should work. Sort of quick & dirty, but it should be fine.
A slight variant that ought to be a little more performant is to keep persistent connections with each game instance. I don't need to worry about blocking issues — technically speaking, there will be a lightweight thread communicating with each game, and there's no problem in having them all blocked for input. The idea is that keeping a persistent, blocking connection on a long-term process via a lightweight thread is far cheaper in terms of operating system resources than constantly forking new processes and killing them immediately afterwards. It's not just CPU + memory, which, granted, may not be significant (unless there are millions of players, of course... not my use case). It's also constantly dumping savefiles and loading them again — that's a lot of wasteful disk I/O, IMHO... and while you avoid the communications block, you'll have blocking at the disk I/O level, so it's basically a similar issue (it might be argued that contemporary file systems are far superior at dealing with such situations than whatever crude hack can be made by us newbies in the vague attempt to squeeze a bit more performance out our already-overloaded servers...).
Granted, the savefiles could get saved to a memory-based temporary filesystem... everything would work fine until the server reboots. Or, alternatively, I guess that it could be possible to store the savefiles on a Redis or memcached server instead.
Lots to think about!
I'm closing this not-issue because I consider myself answered 😃

Another option would be to use one of the JavaScript Glulx interpreters, which happily run side by side.

Another option would be to use one of the JavaScript Glulx interpreters, which happily run side by side.

Aye, I also thought about that approach. It's a possibility, although, mostly due to my irrational beliefs, I still trust an application natively compiled in good, old C more than anything written in JavaScript... I'm still stuck with the mental image that JavaScript is something you use to make web pages look nicer, while C, well, that's what you use to write mission-critical operating systems...
I know it's irrational and does not correspond to the reality of what can be done with JavaScript in 2021!
Nevertheless, at this phase, I'm not completely discarding your suggestion. If my endpoints were web browsers, I think I'd give it a try; but they aren't — my middleware will connect Glk to a virtual world, and, although it uses HTTP(s) for communications with the 'outside world', it has several limitations (mostly to prevent abuse), namely, even sending JSON back and forth is 'too expensive' (so to speak), and an even terser format is usually preferred. The virtual world does not use HTML or JavaScript; it has its own scripting language (vaguely inspired in JS, but fully event-oriented, even more so than JS). Anyway, it's just another layer of complexity to place on top of Glk 😁


[Bug] Getting rid of the "update" hover message

Describe the Bug
I can't get rid of this since some Discord updates.
To Reproduce
No idea sincerely
Expected Behavior
This hover message to disappear
Screenshots

Discord Version
Stable 94010 (5b2a0b4)
BetterDiscord 1.2.1

what to do to make it work better discord?

Manually ensure your plugins are up to date, this is not caused by BD.,


add rwkv support

This is ready for review.

Just a heads up, you can probably remove support for rwkv_pre_ffn and rwkv_mishglu as they aren't really used in any production RWKV models.

done


session schema needs a list of active devices

I don't think this is the same issue for imaging, but I can look at it

closed by #486


Fix destandardization error in spatialstats gallery example

Actually there was no mistake in the formulation.
Closing! :smile:


Deleting dataMigration.md

The Data-Migration_tool repository will not be maintained in future and has been removed. Thus, deleting the reference of the tool from this main repository.

hi @iamarnavgarg, why delete dataMigration.md? We updated it in master w/ a new section on migrating data from Crate to Timescale...we definitely need that info to be somewhere. Are you suggesting we should move those sections somewhere else? If so where?

Hi @c0c0n3 I would like to inform you that @pooja1pathak would not be maintaining the GitHub repository any further. Also, the repository no longer exists. So, would kindly request you to please merge this PR asap.
Thanks

@iamarnavgarg thanks for clarifying. Like I said, we can't merge this into master since there's a new section about Timescale in dataMigration.md which we'd like to keep. So rather than deleting this file, I suggest you:

undo the commit to delete the file
rebase iamarnavgarg:iamarnavgarg-patch-1 on top of  smartsdk:master
edit the file to remove the whole section: "Migrating STH Comet data". Also please take care of removing any unused links at the bottom of the page.

Alternatively, I can do this myself, but then the easiest would be to close this PR without merging. Give me a shout if you'd rather go for this second option.

Hi @c0c0n3,
I have reverted the commit and incorporated your suggested comments by just removing the "Migrating STH Comet data" sub-section. Now, would request you to please merge this PR asap.
Thanks

can you rebase you branch on the remote master?

can you rebase you branch on the remote master?

Hi @chicco785, I have updated the branch.
Thanks

@iamarnavgarg just merged into master, thank you so much for updating the docs! :-)


Potential bug: the results from TestProject are different from the ones from MavenFilterTestOutputHandler

01:26:15.070 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler - Results :
01:26:15.070 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -
01:26:15.070 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler - Failed tests:
01:26:15.070 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -   LookupSnapshotTakerTest.testIOExceptionDuringLookupPersist Expected test to throw (an instance of io.druid.java.util.common.ISE and exception with message a string containing "Exception during serialization of lookups")
01:26:15.070 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -   LocalDataSegmentPusherTest.testPushCannotCreateDirectory Expected test to throw (an instance of java.io.IOException and exception with message a string containing "Unable to create directory")
01:26:15.070 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -   SegmentLoaderLocalCacheManagerTest.testRetryAllFail:307
01:26:15.070 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -   SegmentLoaderLocalCacheManagerTest.testRetryAllFail:307
01:26:15.071 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -   SegmentLoaderLocalCacheManagerTest.testRetrySuccessAtSecondLocation:252
01:26:15.071 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -   SegmentLoaderLocalCacheManagerTest.testRetrySuccessAtSecondLocation:252
01:26:15.071 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler - Tests in error:
01:26:15.071 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -   SQLMetadataRuleManagerTest.cleanup:204->dropTable:209 » CallbackFailed org.ski...
01:26:15.071 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -   LookupReferencesManagerTest.testRealModeWithMainThread:472 »  test timed out a...
01:26:15.071 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -
01:26:15.071 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler - Tests run: 1220, Failures: 6, Errors: 2, Skipped: 18
01:26:15.071 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -

[...]
01:26:16.634 [main] DEBUG f.i.s.r.process.step.TestProject - 12 tests failed, 4 tests have error.
It is probably being doubled.

@surli, could you please explain to me a thing on MavenFilterTestOutputHandler?
In the test TestMavenFilterTestOutputHandler.testFilterLogWillWork2(), why you multiply by 2 the number of running tests and skipping tests?
assertEquals(77351*2, filterTest.getRunningTests());
assertEquals(0, filterTest.getFailingTests());
assertEquals(0, filterTest.getErroringTests());
assertEquals((52+6+1)*2, filterTest.getSkippingTests());

If I remember well the purpose of MavenFilterTestOutputHandler is primarily to detect a test failure from the log. So I'm catching both intermediate results and sum-up results from the logs. Then in:
05:15:11.926 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler - Running io.druid.concurrent.ExecsTest
05:15:13.454 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler - Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 1.526 sec - in io.druid.concurrent.ExecsTest
05:15:13.807 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -
05:15:13.808 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler - Results :
05:15:13.808 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler -
05:15:13.808 [Thread-8] DEBUG f.i.s.r.p.m.o.MavenFilterTestOutputHandler - Tests run: 324, Failures: 0, Errors: 0, Skipped: 1

I will get both the info about the 3 running test and the final sum-up about 324 tests: in the end all my results are multiplied by 2.
On the global vision I did not really care at this step to get the right exact numbers: I mostly want to know if a test was failing or not, the right results should come from the surefire xml files.

I'm catching both intermediate results and sum-up results from the logs [...] in the end all my results are multiplied by 2.

I imagined that, but it's strange because in some projects that I test the final results are not multiplied by 2, I don't understand why, but anyway...

On the global vision I did not really care at this step to get the right exact numbers: I mostly want to know if a test was failing or not

If the numbers are really not important for this step, the MVN_GOAL_FAILED_WITH_TEST_FAILURE = "There are test failures" by itself does the job you want.
My point is that I don't really think nice a log message with wrong info, so I see three options for me:

To remove the computation of numbers from MavenFilterTestOutputHandler, just keep the thing I just mentioned;
Or to remove the log messages that show wrong info;
Or to skip the log messages when it's running for Bears, so nothing would affect for you.

WDYT?

the right results should come from the surefire xml files

This is what I think too.

In fact, I'm thinking at putting that log message I'm discussing here in the step GatherTestInformation instead of keeping it in TestProject. There is no such log message in GatherTestInformation, and there the numbers would be precise, and I also think it makes sense this info be logged from there.

Actually I think we should get rid of MavenFilterTestOutputHandler and to rely only on GatherTestInformation by making TestProject a not blocking step. WDYT?

Actually I think we should get rid of MavenFilterTestOutputHandler and to rely only on GatherTestInformation by making TestProject a not blocking step. WDYT?

Hum, ok, let me think. In fact, let me check a think first with you before I agree with you. I'm checking the entire code of TestProject, and there is a piece that makes me believe that such step should be a blocking one; it's this piece:
int result = MavenHelper.MAVEN_ERROR;
        try {
            result = helper.run();
        } catch (InterruptedException e) {
            this.addStepError("Error while executing maven goal", e);
        }

Because if the step fails because of this part of the code, I think the pipeline should be stopped in TestProject. Please, tell me what you think about this piece of code, maybe I'm thinking wrong.
BUT, for the rest of the code in TestProject, I already agree with you: TestProject doesn't need to stop the pipeline, because the results of the tests would be analyzed in GatherTestInformation.

In simple words, what I'm thinking is: TestProject is to execute the tests, so if something wrong happens during the execution of the tests, the pipeline should stop; but the pipeline should not stop depending on the test results, this is the job of GatherTestInformation. Do you agree with that?

In my vision now, I think we could replace this:
        int result = MavenHelper.MAVEN_ERROR;
        try {
            result = helper.run();
        } catch (InterruptedException e) {
            this.addStepError("Error while executing maven goal", e);
        }

        if (result == MavenHelper.MAVEN_SUCCESS) {
            if (outputTestFilter.getRunningTests() > 0) {
                this.getLogger().debug(outputTestFilter.getRunningTests() + " tests has been launched but none failed.");
            } else {
                this.addStepError("No test recorded.");
            }
            return StepStatus.buildSuccess(this);
        } else {
            if (outputTestFilter.isFailingWithTest()) {
                this.getLogger().debug(outputTestFilter.getFailingTests() + " tests failed, "
                        + outputTestFilter.getErroringTests() + " tests have error.");
                return StepStatus.buildSuccess(this);
            } else {
                this.addStepError("Error while testing the project.");
                return StepStatus.buildError(this, PipelineState.NOTTESTABLE);
            }
        }

by only this:
        try {
            helper.run();
        } catch (InterruptedException e) {
            this.addStepError("Error while executing maven goal", e);
            return StepStatus.buildError(this, PipelineState.NOTTESTABLE);
        }
        return StepStatus.buildSuccess(this);

And then to log numbers in GatherTestInformation.
WDYT?

ok with that.

Deal then, I hope this works.
I will not touch in MavenFilterTestOutputHandler by now, but I will open another issue for a review of this class.

fixed in #656


stop reopening stdout and stderr

Some (newer) versions of passenger have startup issues if the app reopens stdout or stderr
Attached patch changes the deployinator logger and app to use sync rather than reopen.
for reference:

https://github.com/phusion/passenger/wiki/Debugging-application-startup-problems
https://github.com/phusion/passenger/issues/1013
http://stackoverflow.com/a/15350326/973744
http://stackoverflow.com/questions/6683328/difference-between-reassignment-and-reopen-with-ruby-io-streams

ruby-2.1.2
passenger-4.0.56

While this allows me to start the server, this might cause stuff not to be logged to the file.

Added a few patches to this and tested. This should now be good for passenger 3 and 4.


Is 'in menus' a good title, when starting the game? I think not

https://github.com/ThiaudioTT/hoi4-presence/blob/bd92a27a6247460e39c6820db68dbc779630f9b8/src/discordRPC/hoi4RPC.py#L21
I think it should be "Playing hoi4" or something.
Maybe a good idea is to add some intelligence and display other things as well.

"Preparing for war..." 👀


fix: no blank line at the end of file for git 2.39

OS: Darwin 22.5.0 arm64
Git: git version 2.39.2 (Apple Git-143)
diff file in diffview got the following error:
[ERROR 2023-06-19 10:47:27.415 +0800] ...re/nvim/lazy/diffview.nvim/lua/diffview/scene/window.lua:102: ...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:179: The coroutine failed with this message:
	context: cur_thread=<thread 0x0105a85f38> co_thread=<thread 0x0105a86b70> co_func=.../share/nvim/lazy/diffview.nvim/lua/diffview/vcs/file.lua:193
...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:179: The coroutine failed with this message:
	context: cur_thread=<thread 0x0105a86b70> co_thread=<thread 0x0105a94888> co_func=.../share/nvim/lazy/diffview.nvim/lua/diffview/vcs/file.lua:174
...re/nvim/lazy/neogit/lua/neogit/integrations/diffview.lua:20: Git show did not end with a blankline
stack traceback:
	[C]: in function 'error'
	...re/nvim/lazy/neogit/lua/neogit/integrations/diffview.lua:20: in function 'remove_trailing_blankline'
	...re/nvim/lazy/neogit/lua/neogit/integrations/diffview.lua:129: in function 'get_data'
	.../share/nvim/lazy/diffview.nvim/lua/diffview/vcs/file.lua:177: in function <.../share/nvim/lazy/diffview.nvim/lua/diffview/vcs/file.lua:174>
	[C]: in function 'xpcall'
	...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:354: in function <...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:351>
stack traceback:
	[C]: in function 'error'
	...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:362: in function <...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:351>
stack traceback:
	[C]: in function 'error'
	...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:179: in function 'raise'
	...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:207: in function 'step'
	...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:405: in function 'produce_data'
	.../share/nvim/lazy/diffview.nvim/lua/diffview/vcs/file.lua:245: in function <.../share/nvim/lazy/diffview.nvim/lua/diffview/vcs/file.lua:193>
	[C]: in function 'xpcall'
	...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:354: in function <...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:351>
stack traceback:
	[C]: in function 'error'
	...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:362: in function <...cal/share/nvim/lazy/diffview.nvim/lua/diffview/async.lua:351>
[ERROR 2023-06-19 10:47:27.415 +0800] ...cal/share/nvim/lazy/diffview.nvim/lua/diffview/utils.lua:92: Failed to create diff buffer: ':0:src/main/...File.java'

seems this version of git dosen't output a blank line at the end of the file

If its an issue with diffview, it might make more sense to open an issue on their repo ;)

I think the issue could be on our end, since in neogit we are responsible for reading and feeding the files into diffview, rather than having diffview load and parse a revision

problem solved by upgrading my git version to 2.41.0


Fix bottom-bar position on loading

Fixes #173
This PR is for [Anime Name].

[x] This issue is for enhancement/bug fixing.
[ ] This issue is an addition to JSON database.

[ ] Pull Request Title is of form Added [Anime Name].

[ ] This addition is already on ongaku.

[ ] This PR is an improvement to current addition.

[x] I have checked my addition locally and it is working and won't break existing entries.
[x] I have gone through Contribution Guidelines.
[x] This pull request will be closed if I failed to update it even once in a week.

Looks amazing !

@aashutoshrathi can you check why travis build is failing ?

@anshumanv I saw that you have a bad URL in your data.json line 1162 (on the img):
{
	"link" : "http://<IP_ADDRESS>/anime_ost/fairy-tail-original-soundtrack-vol-1/tixatygzwn/08%20-%20Nigiyaka%20na%20Machi.mp3",
	"name" : "Fairy Tail OST - Nigiyaka na Machi",
	"img" : "https://i.ytimg.com/vi/n8a4f3QSowk/maxresdefault.jpgg"
}

Not sure, but may be the root of the failing build problem.

yeah @Sn0wFox is correct, it is causing the build to fail.

Sorry, my bad!
Thanks for the follow up @Sn0wFox
@Monal5031 please do if @Sn0wFox is okay, since he is the one who picked that :stuck_out_tongue:

Yeah sure, finders keepers 👍 do tell @Sn0wFox if you are not doing it.

You guys can go ahead and fix it 👍


Rework

Apologies, I mistakenly targeted this repo for my PR. I was just running some experiments on my fork.


Incompatible install in (summarize your environment)

Hi @yeolep this issue is currently very minimalist and difficult to understand, so I will go ahead and close it. Hope that is fine.
If you are still facing problems with setuptools please go ahead and open another issue using the bug template. The information provided via that form is very valuable to understand what is going wrong.


tried to import hammerspoon config

2017-01-17 22:10:03: -- Lazy extension loading enabled
2017-01-17 22:10:03: -- Loading ~/.hammerspoon/init.lua
2017-01-17 22:10:03: *** ERROR: ...app/Contents/Resources/extensions/hs/_coresetup/init.lua:449: module 'hs._asm.axuielement' not found:
	no field package.preload['hs._asm.axuielement']
	no file '/Users/slifin/.hammerspoon/hs/_asm/axuielement.lua'
	no file '/Users/slifin/.hammerspoon/hs/_asm/axuielement/init.lua'
	no file '/usr/local/share/lua/5.3/hs/_asm/axuielement.lua'
	no file '/usr/local/share/lua/5.3/hs/_asm/axuielement/init.lua'
	no file '/usr/local/lib/lua/5.3/hs/_asm/axuielement.lua'
	no file '/usr/local/lib/lua/5.3/hs/_asm/axuielement/init.lua'
	no file './hs/_asm/axuielement.lua'
	no file './hs/_asm/axuielement/init.lua'
	no file '/Applications/Hammerspoon.app/Contents/Resources/extensions/hs/_asm/axuielement.lua'
	no file '/Applications/Hammerspoon.app/Contents/Resources/extensions/hs/_asm/axuielement/init.lua'
	no file '/Users/slifin/.hammerspoon/hs/_asm/axuielement.so'
	no file '/usr/local/lib/lua/5.3/hs/_asm/axuielement.so'
	no file '/usr/local/lib/lua/5.3/loadall.so'
	no file './hs/_asm/axuielement.so'
	no file '/Applications/Hammerspoon.app/Contents/Resources/extensions/hs/_asm/axuielement.so'
	no file '/Users/slifin/.hammerspoon/hs.so'
	no file '/usr/local/lib/lua/5.3/hs.so'
	no file '/usr/local/lib/lua/5.3/loadall.so'
	no file './hs.so'
	no file '/Applications/Hammerspoon.app/Contents/Resources/extensions/hs.so'
stack traceback:
	[C]: in function 'rawrequire'
	...app/Contents/Resources/extensions/hs/_coresetup/init.lua:449: in function 'require'
	/Users/slifin/.hammerspoon/overrides.lua:1: in main chunk
	[C]: in function 'rawrequire'
	...app/Contents/Resources/extensions/hs/_coresetup/init.lua:449: in function 'require'
	/Users/slifin/.hammerspoon/init.lua:2: in main chunk
	[C]: in function 'xpcall'
	...app/Contents/Resources/extensions/hs/_coresetup/init.lua:481: in function 'hs._coresetup.setup'
	(...tail calls...)

Yes, you need some additional extensions:

https://github.com/asmagill/hs._asm.axuielement
https://github.com/asmagill/hammerspoon_asm.undocumented/tree/630e499acf7264739294feb898e95ee4d2b2638a/bluetooth
https://github.com/asmagill/hs._asm.undocumented.spaces/tree/c59bb374a325ab57967c9c129612d3fde2366517

Also, I strongly advise not to straight import anything from my dotfiles, there's a ton of custom code that's specific to how/what I do and my machine (for example some things will work only right if you have the same hostname, etc...).
It would be much better to treat this repo as inspiration, not ready-to-use distribution :)


Load jnr-runtime dependencies at initialization

What changes are proposed in this pull request?
Preload  jnr-runtime dependencies for fuse.
Why are the changes needed?
Fix issue.
Does this PR introduce any user facing changes?
no.

Error: 9.964 [ERROR] Failures:
Error: 9.964 [ERROR] alluxio.client.fs.LoadMetadataIntegrationTest.repeatedChildrenSync
Error: 9.964 [ERROR]   Run 1: LoadMetadataIntegrationTest.repeatedChildrenSync:233->checkListStatus:514->checkFunctionCall:538 Expected to be take between 1500 and 1750.0. actual duration (ms): 1752
Error: 9.968 [ERROR]   Run 2: LoadMetadataIntegrationTest.repeatedChildrenSync:233->checkListStatus:514->checkFunctionCall:538 Expected to be take between 1500 and 1750.0. actual duration (ms): 1017

alluxio-bot, merge this please


CI is broken on main

CI broke with the version bump to 2023.8.1. For some reason, some jobs started taking much longer and time out.
Commit: https://github.com/dask/dask/commit/5f456cc899890d11da0bf528412858e19665eccd
Workflow logs: https://github.com/dask/dask/actions/runs/5906801262

Thanks @hendrikmakait. The failures on that commit are from https://github.com/dask/dask/issues/10455, which I believe you've since fixed

@jrbourbeau: Sorry, mixed things up. Those failures are fixed, the new ones originate from https://github.com/dask/dask/commit/709e977b7ca9cdd1941c495056d0f89c93600b7e
CI: https://github.com/dask/dask/actions/runs/5965469810/job/16182976640

cc @crusaderky, though I'm not sure if the CI problems are caused by the PR.

Ah, I see. I'm also not sure if the changes in that PR are cause CI issues, or if it's something else like another library releasing. Seeing if reverting helps in https://github.com/dask/dask/pull/10469

Thanks for checking, I also don't see how the changes could cause issues given that the only failing change was a minor adjustment to the tests.

They're not taking much longer - it looks like they're hanging on 99%. I can't see how #10456 can cause the issue...

Reverting both #10456 and dask/distributed#8127 has no effect.
mamba env export doesn't highlight anything at first glance, but at the moment I don't have a better lead:
@@ -15 +15 @@
- -  argon2-cffi=21.3.0=pyhd8ed1ab_0
+ -  argon2-cffi=23.1.0=pyhd8ed1ab_0
@@ -50 +50 @@
- -  c-blosc2=2.10.0=hb4ffafa_0
+ -  c-blosc2=2.10.2=hb4ffafa_0
@@ -84 +83,0 @@
- -  flit-core=3.9.0=pyhd8ed1ab_0
@@ -115 +114 @@
- -  identify=2.5.26=pyhd8ed1ab_0
+ -  identify=2.5.27=pyhd8ed1ab_0
@@ -141 +140 @@
- -  kiwisolver=1.4.4=py39hf939315_1
+ -  kiwisolver=1.4.5=py39h7633fee_0
@@ -190 +189 @@
- -  libprotobuf=3.21.12=h3eb15da_0
+ -  libprotobuf=3.21.12=hfc55251_1
@@ -238 +237 @@
- -  nss=3.89=he45b914_0
+ -  nss=3.92=h1d7d5a4_0
@@ -337,2 +336,2 @@
- -  websocket-client=1.6.1=pyhd8ed1ab_0
- -  wheel=0.41.1=pyhd8ed1ab_0
+ -  websocket-client=1.6.2=pyhd8ed1ab_0
+ -  wheel=0.41.2=pyhd8ed1ab_0
@@ -372,2 +371,2 @@
- -  dask==2023.8.1
- -  distributed==2023.8.1
+ -  dask==2023.8.1+1.g709e977b7
+ -  distributed==2023.8.1+5.g22eb33a2

The hang is introduced somewhere between distributed 2023.8.1 (works) and dask/distributed#8083 (hangs).
This is before https://github.com/dask/dask/pull/10456 and https://github.com/dask/distributed/pull/8127.
Bisecting...

Bisection done.
dask/dask git tip is:

fully green vs. https://github.com/dask/distributed/pull/8125
hanging on the next PR, https://github.com/dask/distributed/pull/8083

I'm a bit surprised that mindeps-optional hangs, while all other mindeps-* environments work.
This means that the test that causes the whole interpreter to hang runs only if one of these dependencies is installed:
- bokeh=2.4.2
- pyarrow=7.0
- lz4=4.3.2
- cachey=0.1.1
- crick=0.0.3
- cytoolz=0.11.0
- dask-ml=1.4.0
- fastavro=1.1.0
- fastparquet=0.8.2
- h5py=2.10.0
- ipycytoscape=1.0.1
- IPython=7.16.1
- matplotlib=3.4.1
- mimesis=5.3.0
- mmh3=2.5.1
- python-cityhash=0.4.6
- python-graphviz=0.8.4
- python-snappy=0.5.4
- python-xxhash=2.0.0
- s3fs=2021.9.0
- scikit-image=0.17.2
- scipy=1.5.2
- sparse=0.12.0
- sqlalchemy=1.4.16
- tiledb-py=0.8.1
- zarr=2.12.0

https://github.com/dask/distributed/pull/8083 broke #9072.
The lack of pytest-timeout caused the whole test suite to hang instead of a single test to fail.
#10482 adds pytest-timeout.
Currently investigating now if the test for #9072, test_blockwise_cull_allows_numpy_dtype_keys, is still relevant to reproducing the issue - in other words, if https://github.com/dask/distributed/pull/8083 actually breaks compatibility with dask-geopandas - or if it's just a test issue.


Extend wordlist - Geography Terms

#Hacktoberfest 🎃
Extended wordlist to all Geography Terms on https://worldbuildingworkshop.com/2016/02/11/place-naming-word-list/
Issue: #6
Let me know if this is not what you were looking for.

I intended for the words to be added to the slices for speech parts (e.g., adverbs, etc.).


groupby aggregate: class as arg value

Feature request: it would be nice if aggregate allowed classes as an argument, using its constructor as the function to aggregate by. In the code sample it recognises frozenset as an iterable because it happens to have such an instance method.
I'd suggest a check like:
>>> inspect.isclass(frozenset)
True
>>> inspect.isclass(frozenset())
False

(I don't think a sane person would make a class with a @staticmethod __iter__ returning the functions to aggregate by)
Code Sample, a copy-pastable example if possible
>>> pd.DataFrame(np.ones((2,2)), columns=[0,1]).groupby(1)[[0]].agg(frozenset)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/groupby.py", line 3597, in aggregate
    return super(DataFrameGroupBy, self).aggregate(arg, *args, **kwargs)
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/groupby.py", line 3114, in aggregate
    result, how = self._aggregate(arg, _level=_level, *args, **kwargs)
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/base.py", line 564, in _aggregate
    return self._aggregate_multiple_funcs(arg, _level=_level), None
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/base.py", line 616, in _aggregate_multiple_funcs
    return concat(results, keys=keys, axis=1)
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/tools/merge.py", line 845, in concat
    copy=copy)
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/tools/merge.py", line 878, in __init__
    raise ValueError('No objects to concatenate')
ValueError: No objects to concatenate

It can be worked around by using a lambda:
pd.DataFrame(np.ones((2,2)), columns=[0,1]).groupby(1)[[0]].agg(lambda x: frozenset(x))

Expected Output
         0
1
1.0  (1.0)

output of pd.show_versions()
INSTALLED VERSIONS
commit: None
python: 3.5.2.final.0
python-bits: 64
OS: Linux
OS-release: 4.6.4-1-ARCH
machine: x86_64
processor:
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
pandas: 0.18.1
nose: 1.3.7
pip: 8.1.1
setuptools: 20.6.7
Cython: None
numpy: 1.11.1
scipy: 0.18.0
statsmodels: None
xarray: None
IPython: None
sphinx: 1.4.5
patsy: None
dateutil: 2.5.3
pytz: 2016.6.1
blosc: None
bottleneck: 1.0.0
tables: None
numexpr: 2.4.6
matplotlib: 1.5.1
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: None
html5lib: None
httplib2: None
apiclient: None
sqlalchemy: 1.1.0b2
pymysql: None
psycopg2: None
jinja2: 2.8
boto: None
pandas_datareader: None

use .apply; .agg has some guarantees about inference on the return type
In [5]: pd.DataFrame(np.ones((2,2)), columns=[0,1]).groupby(1)[[0]].apply(frozenset)
Out[5]:
1
1.0    (0)
dtype: object

Thanks, though note that when selecting multiple columns, one still has to use agg(lambda):
>>> df = pd.DataFrame([[1,2,1],[3,4,1],[5,6,2],[7,8,2]])
>>> df['irrelevant'] = [1,2,3,4]
>>> df
   0  1  2  irrelevant
0  1  2  1           1
1  3  4  1           2
2  5  6  2           3
3  7  8  2           4
>>> df.groupby(2)[[0,1]].apply(frozenset)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/groupby.py", line 651, in apply
    return self._python_apply_general(f)
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/groupby.py", line 660, in _python_apply_general
    not_indexed_same=mutated or self.mutated)
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/groupby.py", line 3375, in _wrap_applied_output
    return (Series(values, index=key_index, name=self.name)
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/series.py", line 233, in __init__
    self.name = name
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/generic.py", line 2694, in __setattr__
    object.__setattr__(self, name, value)
  File "/home/limyreth/.local/lib/python3.5/site-packages/pandas/core/series.py", line 309, in name
    raise TypeError('Series.name must be a hashable type')
TypeError: Series.name must be a hashable type
>>> df.groupby(2)[[0,1]].agg(lambda x: frozenset(x))
        0       1
2
1  (1, 3)  (2, 4)
2  (5, 7)  (8, 6)


Training on multiple GPUs

I'd like to train with this config: https://github.com/kakaobrain/fast-autoaugment/blob/master/confs/pyramid272_cifar10_b64.yaml
with multiple GPUs (as training on one takes about 600 hours). Is there any parameters adjustment to that config that you recommend to make?

@miriam124578 I trained the pyramidnet with 2 or 4 V100 gpus without changing current configurations.
If you want to leverage more gpus(eg. multiple gpu nodes), I believe that you can increase batch-size and learning-rate linearly. For example, if you increase batch-size from 64 to 128(2x), then you should double learning-rate.
If batch-size is large(more then 256), you might want to use 'warm-up' scheduling. See other configurations like : https://github.com/kakaobrain/fast-autoaugment/blob/master/confs/wresnet28x10_cifar10_b512.yaml#L11


Editing shortcut with display time filter resets date when editing

Describe the bug
When editing a shortcut with a display time filter, the displayed date is set to today, regardless of what the actual date for the filter is.
Steps to reproduce

Create a shortcut with a display time filter
Edit the shortcut
In the edit widget, the display time will be shown as the time you edited the shortcut (instead of the time that it is actually set to).

Screenshots or additional context
No response

Shortcuts have been removed in the recent release.


Implement skipped tests in EditEventCommandHandlerShould1

@tonysurma please label with jump-in

@mgmccarthy  These two asserts make the test fail:
Assert.Equal(-5, data.StartDateTime.Offset.TotalHours);
Assert.Equal(-6, data.EndDateTime.Offset.TotalHours);

Since they are dependent on AdjustToTimeZoneModelBinder I assume that I shouldn't bother with them for now, if you think otherwise, let me know.

hmmm.. not too sure what to do about the model binder. @dpaquette , any feedback here?

Don't test the offset here. That is no longer the responsibility of the command handler


Headings can be questions (MD026)

Thanks for the great library.
The default string for MD026 includes '?' and it should not.
In the documentation there is a link to a page as rationale. This page only mentions full stops, not anything else.
Other style guides also recommend against including a full stop at the end of a heading (e.g. The Economists's Style Guide, but don't mention any other punctuation.
It's perfectly legitimate and common (see this issue) to have a heading that is also a question. The '?' should be removed from the default string for MD026.

A more basic question is, what should a lint be enforcing:

syntactic constraints of a programming / markup language
styling guidelines for the programming / markup language
editorial styling guidelines for the content language

Certainly Yes for the first, maybe for the second, but not the third.
I encountered this when I went to use ! at the end of a heading. I really didn't expect a lint to tell me not to do that because of editorial styling rules.
If there were a syntactic issue, like final punctuation creating havoc with heading links, that would be a different matter.

This project has a mix of all three categories.  It’s slightly opinionated - but it’s also easy to opt out of any rule you don’t agree with.
FWIW, I haven’t used a linter yet where I agreed with every suggestion it made. :)

I fully agree that this ought to be solved. @DavidAnson, it's important for a linter to be correct, not just opioniated. Opinion is fine, as long as is based on some authoritative sources. In addition, a linter should provide value in steering towards best practices, not just steering around. When these are parameters requiring manual tuning, the cost-benefit ratio of using this tool is impacted negatively.
I also agree with @PeterConstable that editorial stylistic rules are beyond the purview of a linter for a formal language.
I did not find authoritative sources that reject terminal question marks in headings or titles in general. See e.g. https://www.thepunctuationguide.com/question-mark.html.
For me, it is important to be able to write question marks for FAQ entries, which occur regularly in the educational materials I create. The current

@sanmai-NL : I just discovered that if the question in a heading is italicized—so wrapped in _ ... _—then that keeps this linter from complaining.
However, I still think a linter shouldn't be functioning like a content-language copy editor or proofing tool.

Thanks everyone for the feedback!
To respond to a couple of points above:

This is not a correctness issue; the rule is doing what it says. Rather, we are discussing the preferred default configuration.
Regarding whether a linter should report style issues also, I suggest the answer is yes, and point to existing rules for enforcing a consistent list marker (-+*) as well as reasonable list numbering (1, 2, 3). Neither is required for correctness, but both are exactly the kinds of things I look to a linter to highlight.

Thinking about whether or not question marks and exclamation points belong in a heading, I would cite the same resource used above:

https://www.thepunctuationguide.com/question-mark.html
https://www.thepunctuationguide.com/exclamation-point.html

Note that headings are not a recommended scenario for either form of punctuation according to that guide. To be fair, it's not called out as being wrong, but none of the examples recommend a question or exclamation as part of a title.
It seems relevant to note that the use of these punctuation marks is so often abused that there are articles about the problem and names for the phenomenon:

https://amp.theatlantic.com/amp/article/563774/
https://en.m.wikipedia.org/wiki/Betteridge's_law_of_headlines

Again, that's not to say that it's always inappropriate to use a question mark or exclamation mark in a heading, but it does provide a certain justification that they are frequently misused and therefore that it's probably worth a second look when they show up in this context.
There is no "right" or "wrong" here, and I do strive to make sure the rules are consistent with general practice. This issue has come up enough that I may remove them in a future release, but I did want to explain a bit more about why they're still included today.

Regarding whether a linter should report style issues also, I suggest the answer is yes, and point to existing rules for enforcing a consistent list marker (-+*) as well as reasonable list numbering (1, 2, 3).

Style enforcement for the markup language is not in question. The issue is about style enforcement in the content language.
Whether or not Betteridge or anybody else with editorial opinions believes that punctuation marks don't belong in headings or titles, there are plenty of content authors that will nonetheless use them in headings or titles. Linting a markup language and proofing/copy editing content are two completely different things and should be handled separately.

No authoritative source categorically rejects question marks in headings. Indeed, the Punctuation Guide only confirms its wide applicability, not only as sentence boundaries (terminal punctuation) but even within sentenced.
¿Cómo abordará las preguntas en español, entonces?

Even if some editorial authority did categorically reject question marks in headings, a tool for enforcing that should be advertised as a tool for enforcing those content requirements, not as a tool for markup-language requirements.

@PeterConstable I fully agree with your angle. Actually, the whole no-terminal-punctuation should be abolished then.

I am looking forward to get this addressed, is really annoying.

@ssbarnea If this detail is bothering you, you can disable the rule or change the list of punctuation it is looking for. There is no need to wait for an update to the library.

David can I just say thank you for maintaining this and engaging as you
have done. 🙏  I remain quite certain the rule is overstepping its bounds
and annoyingly high-lighting something that is 100% not a problem. I'd
still like it to highlight full stops and commas, so it's a useful rule to
me and I don't wish to disable it. But it's your library and I respect your
right to make these decisions. Thanks again.
On Wed, 3 Jun 2020 at 17:12, David Anson<EMAIL_ADDRESS>wrote:

@ssbarnea https://github.com/ssbarnea If this detail is bothering you,
you can disable the rule or change the list of punctuation it is looking
for. There is no need to wait for an update to the library.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DavidAnson/markdownlint/issues/276#issuecomment-638299393,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKWNOOCFCYXQSEBFXBKXREDRUZY6LANCNFSM4MO4URAA
.

I always find myself using the question mark ? in my headers all the time, as many others https://github.com/aip-dev/google.aip.dev/blob/master/README.md#have-an-idea-for-an-aip
For example, one that I use all the time ## What's next? to direct and help people what to do next.
Personally, I find this rule quite annoying, and I agree about not having multiple sentences in the header, but removing these symbols doesn't make sense to me.


Remove firebase_auth dependency from android_alarm_manager library and add initialize step to readme

The Android_Alarm_Manager-Plugin is not working for me because it has firebase_auth as a dependency, but the dependency seems to be not needed. It's only being used in the example, which has its own pubspec.yaml file.
In the readme, there was also the initialization for the plugin missing, which has now been added.

I signed it!

OK I didn't test dry_run locally which failed because the example is depending on firebase_auth.
So what I did now is adding firebase_auth as a dev_dependency as suggested here:

The rule for deciding between a regular or dev dependency is simple: If the dependency is imported from something in your lib or bin directories, it needs to be a regular dependency. If it’s only imported from test, example, etc. it can and should be a dev dependency.
This seems to work at least on my machine.

@collinjackson Thanks for labeling my PR :)
Just want to make sure that you saw that I also made changes in the pubspec file, so it's not only about documentation, but also about removing a (currently) required dependency.

Please could somebody merge this? It is a simple change and is blocking my progress.

@Timmmm For now I would suggest to directly use my GitHub-Repo if this is blocking you using this syntax:
android_alarm_manager:
    git:
      url: git://github.com/Patte1808/plugins.git
      path: packages/android_alarm_manager

Something seems to hold back the Flutter team from merging it, my guess is that it might be something related to their release-cycle.

Thanks for the hint! Worked perfectly :-)


Update renconstruct example in README

renconstruct expects options to come first, before the input and output directory, per its help message:
Usage: renconstruct build [OPTIONS] <INPUT_DIR> <OUTPUT_DIR>
However, the example in the README has it placed after the directories instead, which doesn't seem to work.
This PR updates the example.

Oh yeah, that's definitely an oversight on my part. I typically go through the README to check for this stuff at times, but this must've slipped by. Thanks for catching and fixing it!


echo "42"

when I run echo "42" on my shell it prints 42, just as echo 42 does.
However
assert_cmd!(echo "42").succeeds().prints("\"42\"").unwrap();

Assert::command(&["echo", "\"42\""]) behaves identical.
I would have naively expected the behaviour from my shell. But I don't know how other shell APIs behave. Is there any particular reason for the current behaviour?
PS: I'll get to the code too .. I'm just familiarizing myself with this crate first. :-)

Hm, assert_cmd! uses stringify! internally, maybe that also cause this to be weird? I think I have to rethink that macro, or try to extend it to be a bit more clever (like not stringifying exprs).

I suspect this happens on the expansion side, since Assert::command(&["echo", "\"42\""]) behaves identical.
The question is how Assert::command(&["foo", "bar"]) is supposed to expand. Does it expand to foo bar or does it expand to foo "bar"? Or maybe I have a wrong picture of expansion. I'm so much used to bash, that I am thinking in terms of bash args eval "$1" "$2" (which maybe I should double check for these cases, before making claims..^^).

POSIX shell syntax is basically a lisp where everything is a string. 😅
So, echo 42 (or echo "42" for that matter) is the same as "echo" "42"(try it!). Using quotes allows you to have arguments with white space in them, likecat "file with a fancy name.md". echo is a bad example here, as it, well, echos _all_ arguments (have fun with ["$*"`](http://stackoverflow.com/a/9995322/1254484)!).
But, we are not in a shell here! Rust's std::process::Command spawns a new process without any shell interpolation by just calling execvp with an array of CStrings here.
So, for us, there is a difference between ["echo", "42"] and ["echo", "\"42\""], and I really need to make sure it's either a meaningful one or not a surprising one :)

Hm. Ok I didn't know much about LISP but now I don't want to learn it either. ^^
Regarding the initial "problem" I think it can be solved via documenting the status quo. That is assert_cmd!(foo bar baz) corresponds to the in-shell command "foo" "bar" "baz".
This means however that the assert_cmd macro would be strictly less expressive than the Assert::command function, which is a pity. But you can't express the in-shell command git commit -m "lorem ipsum" with the current macro. The best you'll get is either "git" "commit" "-m" "lorem" "ipsum" or "git" "commit" "-m" "\"lorem ipsum\"". Documenting this as status quo would be an option at least...

I made a comment in a commit.

TL;DR: rustc_serialize::json::Json::from_str(stringify!("foo")); yields "foo". :-)

Ok, we can detect if the stringified output is delimited by ". In that case we now, that we have stringified a string. How do we un-stringify?
That should be the same as to deserialize a string, like in JSON, right? Ok, now phrased that way it sounds solvable.
These crates should be able to handle that job:

rustc-serialize
serde_json
json

I haven't used any of these, but rustc-serialize seems more lightweight than serde_json, so I gave it a try, and voila:
extern crate rustc_serialize;
use rustc_serialize::json::Json;

fn main() {
    let x = stringify!("foo");
    if let Ok(Json::String(y)) = Json::from_str(x) {
        println!("x={:?}, y={:?}", x, y);
        // ^--> x="\"foo\"", y="foo"
    }
}

PS:

String implements FromStr, but that's only a clone. Too bad.
I also found shlex, which sounds related. But I don't get what it does.


SurfaceRef blit functions: Pass destination argument as &mut SurfaceRef

At the moment, the blit() family of functions takes an argument of type S: AsMut<SurfaceRef>. This is obviously supposed to allow code like the following:
let mut dest: Surface = ...;
sprite.blit(None, &mut dest, Some(sprite_rect));

However, this trait isn't implemented for SurfaceRef itself. So while an owned Surface will work, there's no way to pass a SurfaceRef as the destination.
I've changed the type signature to take a &mut SurfaceRef, to which a &mut Surface will automatically coerce (thanks to DerefMut). In other words, code like the above should still compile, though I doubt there's much out there compared to how many applications are using Renderer.

I wanted to add quickly, in case it's not obvious, that one alternative is to go ahead and implement AsMut<SurfaceRef> for SurfaceRef. I'll leave it up to the maintainers to decide which of these is more appropriate.

This looks good. Cheers!


[SPARK-43073][CONNECT] Add proto data types constants

What changes were proposed in this pull request?
Add constants for un-parameterized proto data types
Why are the changes needed?
avoid recreating them
Does this PR introduce any user-facing change?
No, dev-only
How was this patch tested?
existing UT

Thanks @grundprinzip for the reviews, merged into master


Add missing similarity field to KnnQuery and missing fields to KnnSea…

Fix for: https://github.com/elastic/elasticsearch-specification/issues/2149

@ezimuel I signed the Contributor Agreement on Tuesday, but the check still failed. Does it take a while for it to be available? Since then I have signed it 2 more times, just to be sure nothing went wrong.
Could you help me getting the cla-check to pass?

@AlmostFamiliar You have to use the mail that is assigned to your commit. I can see you signed with 2 mails, but none of them equals the commit mail 🙂

Closing because https://github.com/elastic/elasticsearch-specification/pull/2261 got merged


[22] Rollout is giving error on default language-masters pages , livecopy relationship is not correct on default generated content pages

Expected Behaviour
Once the project is created using the archetype 22  and deployed to local AEM, the User can see default content structure under site console. When the user tries to roll out any changes on any page  under "/content/appFolerName/language-masters/en" to "/content/appFolerName/us/en"  this should work.
Actual Behaviour
When the user tries to roll out any changes on any page  under "/content/appFolerName/language-masters/en" to "/content/appFolerName/us/en"   this is giving error.
Reproduce Scenario (including but not limited to)
Steps to Reproduce

Create a sample project using the command at [1]
[1] https://docs.adobe.com/content/help/en/experience-manager-learn/getting-started-wknd-tutorial-develop/project-setup.html#create
Now, build and deploy the code to local AEM
After the project is deployed to local AEM, go-to site console and open /content/appFolerName/language-masters/en page or create any child page under en and open that.
Once the page is open in edit mode, try to roll out page using page information pop up.
Select default live copy from Showing 1 Live Copies window and roll out.
Roll out will not work and it will show error like ERROR 1 Rollout Operations Failed.

Platform and Version
Archetype 22
Root Cause:-
I have added this section myself as I know the root cause.
The default generated content  page /content/appFolderName/us/en/jcr:content missing jcr:mixinTypes = cq:LiveRelationship property.
Sample Code that illustrates the problem
Logs taken while reproducing problem
18.03.2020 16:37:35.967 ERROR [0:0:0:0:0:0:0:1 [1584529655954] POST /bin/wcmcommand HTTP/1.1] com.day.cq.wcm.msm.impl.commands.RolloutCommand Error during roll-out.
java.lang.NullPointerException: null
at com.day.cq.wcm.msm.impl.RolloutManagerImpl$RolloutSettings.fromRelationship(RolloutManagerImpl.java:1224) [com.day.cq.wcm.cq-msm-core:5.12.36]
at com.day.cq.wcm.msm.impl.RolloutManagerImpl.rollout(RolloutManagerImpl.java:537) [com.day.cq.wcm.cq-msm-core:5.12.36]
at com.day.cq.wcm.msm.impl.MoveLiveCopyResourceConflictHandler.handleConflict(MoveLiveCopyResourceConflictHandler.java:139) [com.day.cq.wcm.cq-msm-core:5.12.36]
at com.day.cq.wcm.msm.impl.MoveLiveCopyResourceConflictHandler.handleNameConflict(MoveLiveCopyResourceConflictHandler.java:103) [com.day.cq.wcm.cq-msm-core:5.12.36]

@duchauha Thanks for reporting this, along with the probable root cause. To speed up fixing this bug we would greatly appreciate a contribution in the form of a PR.

@vladbailescu  Sure !! I'll try to fix it asap(maybe tomorrow as right now it is very late here in India) and raise a PR.

@vladbailescu   I have made the fix but I am not able to push it, getting wrong user/password error although the same is working when logging into github.com.
can you please help me with the process to raise a PR.
I have tried a few things : -

Took clone of  https://github.com/adobe/aem-project-archetype.git and create my bugfix branch but not able to push it.
Took clone of https://github.com/duchauha/aem-project-archetype.git and create my bugfix branch but again not able to push it.

can you please help me with the process?
Thanks

@vladbailescu  Ignore my previous comments. After generating the  PAT, I was able to push it.
Although I am a bit confused about the process, I have raised the PR:- https://github.com/adobe/aem-project-archetype/pull/361
so if you see something is not correct about my PR (process or flow vise) let me know.


rightClick in WebDriver is not working

i am trying to do rightClick functionality but it doesn't work and its continuing to execute with other test cases automatically.

The same issue happens for me too with WebDriver. Appreciate any breadcrumbs or pointers to help resolve this.

@PeterNgTr  anyone looking into this issue? I have made a fix using executeScript which is working on firefox and chrome. Should i create a PR for the same?

That would be awesome 😎