[Bug]: 模组加载器无法初始化/MelonLoader failed to initialize

All of the following criteria must be met

[X] All Requirements must be installed.
[ ] Full Latest.log file included. If no file exists then leave this unchecked and state so.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[X] Tried restarting PC.
[ ] Was able to see the Start Screen.

Describe the issue.
游戏开始运行时直接弹出Internal Failure窗口，窗口内显示Failed to initialize MelonLoader:Load LibraryExW failed。
使用手动安装方法仍未解决，已多次尝试检查游戏文件完整性、重新下载MelonLoader、LimbusLocalize以及字体文件。
已排查网络问题：已尝试更换加速工具（Watt Toolkit、Steamcommunity 302，CFW未尝试，但我想不是这个能解决的），已检查防火墙设置，已尝试暂时关闭杀软，已证实主机能够连接上GitHub。
已尝试重新启动电脑，已尝试将企业版系统更换为专业版（密钥方式）。
已尝试更换汉化模组及附属文件版本，均未解决。
游戏本体在完全删除汉化模组文件后可以正常启动。
两个log（Latest(框架日志).log，Player(游戏日志).log）均未生成，UserLibs未生成，Plugins未生成。
错误截图见下，文件夹内文件（启动游戏后）见下。
When the game starts running, an Internal Failure window will pop up directly, displaying Failed to initialize MelonLoader: Load LibraryExW failed.
The manual installation method did not solve the problem,Multiple attempts have been made to check the integrity of game files,re-download MelonLoader,LimbusLocalize and font file.
Possibility of the Internet connection causing problems has been checked and excluded: the accelerator has been replaced many times(Watt Toolkit,Steamcommunity 302)(I haven't tried CFW or such ladders yet,but I think it neither can sovle the problem of fail to init the melonlodaer),Firewall settings have been checked and it has been confirmed that the host can connect to GitHub.
The computer has already been restarted,the system version has been changed from business version to professional version(by changing the key).
The version of the mod and the modloader has been changed,neither of them has solved the problem.
The game itself can start normally after completely removing the mod files and melonloader.
Two logs (Latest(框架日志). log, Player(游戏日志). log), UserLibs, and Plugins were not generated even afer running the game.
Please see the screenshot of the error and the files in the folder (after starting the game) below.

Did you attach your log file?

[ ] Yes, I attached my log file to the text box above.
[X] No, I could not find a log file at {Game_Directory}\MelonLoader\Latest.log

It is very likely that this is caused by Chinese/other unicode symbols in the path to the game.

This is not a problem that occurred to me, so I'm sorry that  I couldn't provide an exact answer immediately. However, in my impression, it was placed in the default installation path of Steam under C:\ Program Files (x86).In my opinionaAlthough the system language is Chinese,this path does not contain Chinese characters, right?I have never figured this out.
Also, I'm sorry, I'm not sure if I should first publish these in the issue of that mod, but I have done it

This is not a problem that occurred to me, so I'm sorry that I couldn't provide an exact answer immediately. However, in my impression, it was placed in the default installation path of Steam under C:\ Program Files (x86).In my opinion although the system language is Chinese,this path does not contain Chinese characters, right?I have never figured this out.
Also, I'm sorry, I'm not sure if I should first publish these in the issue of that mod, but I have done it.

This is not a problem that occurred to myself, so I'm sorry that I couldn't provide an exact answer immediately. However, in my impression, it was placed in the default installation path of Steam under C:\ Program Files (x86).In my opinionaAlthough the system language is Chinese,this path does not contain Chinese characters, right?I have never figured this out.
Also, I'm sorry, I'm not sure if I should first publish these in the issue of that mod, but I have done it

This is not a problem that occurred to myself, so I'm sorry that I couldn't provide an exact answer immediately. However, in my impression, it was placed in the default installation path of Steam under C:\ Program Files (x86).In my opinion although the system language is Chinese,this path does not contain Chinese characters, right?I have never figured this out.
Also, I'm sorry, I'm not sure if I should first publish these in the issue of that mod, but I have done it.

I just found out that if the installation path contains non ASCII characters,,Steam cannot running at all.


Update python script link

Update python script link to point to the raw link so that when Users copy the link and use say wget it just works!

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: dpkshetty
To complete the pull request process, please assign yussufsh after the PR has been reviewed.
You can assign the PR to them by writing /assign @yussufsh in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @dpkshetty! It looks like this is your first PR to ocp-power-automation/ocp4-upi-powervs 🎉

Hi @dpkshetty. Thanks for your PR.
I'm waiting for a ocp-power-automation member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mkumatag @bpradipt Can you take a look at this simple PR and help approve it ?

/ok-to-test
/lgtm
/hold for others to review

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt, dpkshetty
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [bpradipt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/unhold

/assign @mkumatag

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt, dpkshetty, mkumatag
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [bpradipt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/retest

Lately I have trouble running the pods in the IKS, image pull is failing and fails to start or complete that pod execution, so the est is failing because of that. Need to again raise a service ticket and see what is going wrong!

/retest

@dpkshetty can you please rebase..

New changes are detected. LGTM label has been removed.

@dpkshetty: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@bpradipt rebase done


Problem with resizing JDialog

I try to use your MaterialDesign LookAndFeel in my project and I have a problem with resizing JDialog modal window. You can see this bug if change JFrame to JDialog in your example form and move mouse to right down corner. You don't see change mouse cursor to cursor for resizing. Also if you move mouse cursor to down edge of form you may see change cursor for resizing but resize don't work correctly. Please fix this bug and thank you for your work. It's very important to develop swing direction in Java

Hi @izemskov.
Thanks for using this l&f.
I know this bug and I'm working now to optimizing it, but the solution already existed with this PR.
If you use your app for simple cases and want to try the solution, you can compile this vincenzopalazzo/material-ui-swing/development.
After the build, you should have a material-ui-swing-1.1.1_0.X-dev.
You can see the actual bug status inside this issue
I hope to help with your issue and also, I suggest you see inside this repository vincenzopalazzo/material-ui-swing. it contains the actual status of this l&f.

Fixed in material-ui-swing-1.1.1-rc1, look here https://github.com/vincenzopalazzo/material-ui-swing/releases

It's really works! Thank you!

Hi @izemskov,
Happy to help :)


[ENH] implement getstate

Is your feature request related to a problem? Please describe.
In my code, I would like to do nothing different.
Describe the solution you'd like.
Right now __getstate__ returns nothing, while __setstate__ correctly sets values. I think it'd be nice for __getstate__ to return what can be passed to __setstate__. This information is already provided by __reduce__()[1][2], but I'd like to be able to simply do:
obj = MyCythonObj(...)
obj_state = obj.__getstate__()
new_obj = type(obj).__new__(type(obj))
new_obj.__setstate__(obj_state)

This is going to help us easily support cython objects in https://github.com/skops-dev/skops, which persists / loads objects w/o using pickle.
Describe alternatives you've considered.
No response
Additional context
No response

Right now __getstate__ returns nothing

That's probably the default implementation of Python 3.11+ that strikes here. If you try it under Py3.10 or earlier, I don't think you'll find a __getstate__ method.

Isn't __reduce__ the more generic interface from the consumer's point of view? If you target __reduce__, then it should work with all objects that support the pickle protocol, whereas if you require __getstate__, then that's not necessarily the case, AFAIU.

Yes it is, but it also opens the door to easily creating malicious pickle objects. The point in skops is that we don't rely on objects' __reduce__. It means we can't pickle everything, but we already support pretty much the whole sklearn objects. For instance, for numpy, we rely on numpy.save/load instead of pickling them.


start writing tests

I think we reached a point that this is absolutely necessary

I have configured the environment to run tests, so far I've only written two primitive tests. More substantial ones will be written soon


Change bs-platform from master to current release

Sorry, I forgot about this. I've pinned bs-platform in the latest release, now published as bs-webapi. Master is still pinned for the moment, but is prone to track master again. I hope that'll do for you.


iPhone-Aadarsh

My artwork is an iPhone and each time the program is run the size of the phone changes along with the size and position of the secondary camera. There is a slider in the code which can be manipulated to change the height of the camera bar. I am relatively new to coding so I haven't made anything too complex.

Okay sure. Thanks for the feedback.
On Tue, May 28, 2024 at 8:18 AM leomcelroy @.***> wrote:

Thanks for the submission. I think this piece may be a bit too simple. Try
to add some more control flow into the code (loops and if statements). A
lot of your code is hardcoded at the moment. While not a perfect metric aim
for at least 150 lines of code.
I'm going to close this for housekeeping but feel free to resubmit in the
future.
—
Reply to this email directly, view it on GitHub
https://github.com/hackclub/blot/pull/487#issuecomment-2135496549, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BIC5PBHQ4AUAIPO6LCOHA4DZESN3XAVCNFSM6AAAAABIJDFCJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZVGQ4TMNJUHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix for attn_softmax_bf16 when generation_config is None

This addresses an error from the attn_softmax_bf16 change in cases where self.model.generation_config exists but is 'None'.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Improve fairness of initial allocation of segments

Problem description
When the Flink job starts, the readers acquire segments as they start up.   The first reader to start is being overly greedy by acquiring all segments and starving the other readers.   Rebalancing eventually occurs as other readers come online.
This is an optimization issue not a functional problem.
Problem location
FlinkPravegaReader
Suggestions for an improvement
The problem is basically a race between reader initialization and segment acquisition.   Since the number of subtasks (reader instances) is known when the reader group is created, their names could be pre-registered with the reader group during its initialization.  This implies that the names be made stable (issue #16).
Here's a log showing that reader 1 (of 4) acquires all segments before readers 2..4 come online.  Also shown is subsequent rebalancing.
2017-05-08 16:58:46,501 28409 [Source: Custom Source (2/4)] INFO  i.p.c.flink.FlinkPravegaReader - Starting Pravega reader 'Source: Custom Source (2/4)' for controller URI tcp://localhost:32770
2017-05-08 16:58:46,506 28414 [flink-akka.actor.default-dispatcher-7] INFO  o.a.f.r.e.ExecutionGraph - Source: Custom Source (1/4) (4d19a3c2c16c6f6086f6b235f4335dbc) switched from DEPLOYING to RUNNING.
2017-05-08 16:58:46,509 28417 [flink-akka.actor.default-dispatcher-2] INFO  o.a.f.r.e.ExecutionGraph - Source: Custom Source (2/4)<PHONE_NUMBER>8624a6918951a22530243d) switched from DEPLOYING to RUNNING.
2017-05-08 16:58:46,511 28419 [Source: Custom Source (2/4)] INFO  i.p.c.s.impl.EventStreamReaderImpl - EventStreamReaderImpl( id=Source: Custom Source (2/4)) acquiring segments {Segment(scope=scope, streamName=ShlxqXWdXvXzjGdpvZKJ, segmentNumber=2)=10236, Segment(scope=scope, streamName=ShlxqXWdXvXzjGdpvZKJ, segmentNumber=3)=9996, Segment(scope=scope, streamName=ShlxqXWdXvXzjGdpvZKJ, segmentNumber=0)=10308, Segment(scope=scope, streamName=ShlxqXWdXvXzjGdpvZKJ, segmentNumber=1)=10308}
2017-05-08 16:58:46,518 28426 [Source: Custom Source (3/4)] INFO  i.p.c.flink.FlinkPravegaReader - Starting Pravega reader 'Source: Custom Source (3/4)' for controller URI tcp://localhost:32770
2017-05-08 16:58:46,547 28455 [flink-akka.actor.default-dispatcher-6] INFO  o.a.f.r.e.ExecutionGraph - Source: Custom Source (3/4) (30fd38ce163e51e765d3b236b522c811) switched from DEPLOYING to RUNNING.
2017-05-08 16:58:46,556 28464 [Source: Custom Source (1/4)] INFO  i.p.c.flink.FlinkPravegaReader - Starting Pravega reader 'Source: Custom Source (1/4)' for controller URI tcp://localhost:32770
2017-05-08 16:58:46,558 28466 [Source: Custom Source (4/4)] INFO  i.p.c.flink.FlinkPravegaReader - Starting Pravega reader 'Source: Custom Source (4/4)' for controller URI tcp://localhost:32770
2017-05-08 16:58:46,608 28516 [flink-akka.actor.default-dispatcher-6] INFO  o.a.f.r.e.ExecutionGraph - Source: Custom Source (4/4) (03d62d7c81dae731e685adeca1175177) switched from DEPLOYING to RUNNING.
2017-05-08 16:58:46,610 28518 [flink-akka.actor.default-dispatcher-6] INFO  o.a.f.r.e.ExecutionGraph - Map -> Sink: Unnamed (1/1) (9aff8650dc230d80764275ab6a5e2dc2) switched from DEPLOYING to RUNNING.
...
2017-05-08 16:58:53,142 25485 [Source: Custom Source (4/4)] DEBUG i.p.c.flink.FlinkPravegaReader - Emitting: 3858
2017-05-08 16:58:53,144 25487 [Source: Custom Source (4/4)] DEBUG i.p.c.flink.FlinkPravegaReader - Emitting: 3859
2017-05-08 16:58:54,723 28631 [Map -> Sink: Unnamed (1/1)] DEBUG i.p.c.f.u.IntSequenceExactlyOnceValidator - IntSequenceExactlyOnceValidator - received element: 3858
2017-05-08 16:58:54,723 28631 [Map -> Sink: Unnamed (1/1)] DEBUG i.p.c.f.u.IntSequenceExactlyOnceValidator - IntSequenceExactlyOnceValidator - received element: 3859
...
2017-05-08 16:59:23,314 25657 [Source: Custom Source (4/4)] INFO  i.p.c.s.impl.EventStreamReaderImpl - EventStreamReaderImpl( id=Source: Custom Source (4/4)) releasing segment Segment(scope=scope, streamName=OwKBLcwfDwfhNkhIqXuK, segmentNumber=2)
2017-05-08 16:59:23,332 25675 [Source: Custom Source (1/4)] INFO  i.p.c.s.impl.EventStreamReaderImpl - EventStreamReaderImpl( id=Source: Custom Source (1/4)) acquiring segments {Segment(scope=scope, streamName=OwKBLcwfDwfhNkhIqXuK, segmentNumber=2)=4572}
...
2017-05-08 16:59:56,142 25485 [Source: Custom Source (1/4)] DEBUG i.p.c.flink.FlinkPravegaReader - Emitting: 4002
2017-05-08 16:59:56,144 25487 [Source: Custom Source (4/4)] DEBUG i.p.c.flink.FlinkPravegaReader - Emitting: 4021
...

We can easily add a delay to this acquisition. I think it is currently one second, but we can make that a config parameter.

I'd like to make it deterministic using an optional parameter to reader group initialization for 'initial readers'.

Those aren't mutually exclusive. But is is normal that there would be a master process that would know the IDs of all the readers? It seems like the IDs would often be generated at process startup.

It is normal in Flink's case that all the readers are known upfront.


NETOBSERV-972 Make DISABLED auth mode not restricted to admins

@jpinsonneau @OlivierCazade - this might not be a definitive answer to the problem. The "CheckAdmin" mode is too strict as it checks for belonging to system:cluster-admins group, which is NOT the same thing as checking if a user is a cluster admin.
Ideally, we would like to check if the user has a binding to the "cluster-admin" role, but afaik this isn't trivial to do.

an alternative could be to reintroduce my previous "hack", that was checking if the user can list namespaces

/approve


Added few missing requirements

Hi!
I noticed that these requirements are needed to run sparse/dense reconstruction. I think these might as well be added to the requirements.txt?

Thanks for adding them!


doc lint for errors and aggregations

aggregations is brand new, so nothing to do :D #1232

Thanks!


Make submit buy green

When a user clicks on BUY, do BUY.
Merge logic of button Submit BUY into BUY so user doesn't need to click twice.

@yangwao I'll do this

In commit of 49a623512af2f108c778ba0a543a7c63dcec6913 , the buy logic was broken

This missing on main-nuxt

@yangwao sure, I'll do this in nuxt


Add custom-static-files option

Just a draft proposal to toy around with. Probably not a good idea, but it's useful in my usecase: wanting one less component in my setup for a customer.

Adds the --custom-static-files CLI option that allows users to use their own static files, instead of the dashboard. Makes it easier to host a custom front-end.
Adds the actix_files crate as dependency. The impact of this is pretty low, as by far most of the sub-dependencies are already used.

See https://github.com/meilisearch/product/discussions/610

Hey, I like the idea.
That means, if you have a static website, it's super easy to host it with meilisearch with https and without a reverse proxy.
Not sure it's something we want to merge product-wise (pretty sure it's not going to land for the v1.1.0).
What do you think of it, @gmourier?
Personally, I don't think it's too risky to merge it as an experimental feature to see if people use it for now before making a real decision.

Hey, I like the idea. That means, if you have a static website, it's super easy to host it with meilisearch with https and without a reverse proxy.

Mostly yes, but if we want to get rid of the reverse proxy and still have HTTPS with plain meilisearch, we'll also need some form of acme / TLS support.  (May be off-topic for this PR, but still good to share some thoughts as this too relates to the usecase of using Meili as a one-stop-shop for hosting a static site with powerful search.) I added automated acme / TLS setup in atomic-server using acme_lib and later using instant-acme (which uses Rustls instead of OpenSSL, removing an important runtime dependency). Feel free to use my implementation, which also is built on actix-web.

Thanks again for the work @joepio
Since the PR is a little bite old and it's not a priority for the team now, I close the PR
However, thanks again for your work, it's definitely valuable and will help us if decide to go with it. Let's see if the discussion on the product side has more attractiveness in the future.


Drop unmapped columns in presence of mapping

Description

Include a summary of the changes and the related issue.
Include relevant motivation and context.

Resolves #(issue)

@fivegrant is this still relevant? Can you provide an update on it.

@brandomr I'm really not sure what we want the behavior to be here so I'm going to close this for now, @Tom-Szendrey mentioned during the hackathon that it would be nice if unmpaped dataset columns would be dropped. Currently though, this would also drop the timestep columns so this change cannot be made preceding a change to a change around the time column in the sim spec


PHP 8.1.9 deprecation: "Symfony\Component\Config\Resource\FileResource implements the Serializable interface, which is deprecated"

PHPMD version: 2.12.0
PHP Version: 8.1.9
Installation type: via brew install and downloading the phar file from the website
Operating System / Distribution & Version: macOS Monterrey (12.2.1)

Current Behavior
Getting deprecation warnings related with symfony/config package.
PHP Deprecated:  Symfony\Component\Config\Resource\FileResource implements the Serializable interface, which is deprecated. Implement __serialize() and __unserialize() instead (or in addition, if support for old PHP versions is necessary) in phar:///usr/local/bin/phpmd/vendor/symfony/config/Resource/FileResource.php on line 21

Expected Behavior
Not having these warnings.
Steps To Reproduce:

Download and install the phar phpmd version 2.12.0 from the website.
Run phpmd with php 8.1.9 and the warnings will appear.

Checks before submitting

[x] Be sure that there isn't already an issue about this. See: Issues list
[x] Be sure that there isn't already a pull request about this. See: Pull requests
[x] I have added every step to reproduce the bug.
[ ] If possible I added relevant code examples.
[x] This issue is about 1 bug and nothing more.
[x] The issue has a descriptive title. For example:  "JSON rendering failed on Windows for filenames with space".

The phar file is made to be compatible with PHP 5.3 which require this version. If you need to run it on a later version of PHP and do not want warnings then please install via composer.


LTO doesn't appear to affect llvm intrinsics.

The readme suggests that linker-plugin-lto is required for llvm intrinsics to be faster than native intrinsics.

Additionally, Rust usually optimizes when compiling and linking Rust code (and not externally linked code), this means that these intrinsics cannot be inlined adding an extra function call overhead in some cases which make it perform worse than the native Rust implementation despite having more optimized assembly. Luckily, Rust currently has support for linker plugin LTO to enable optimizations during the link step, enabling optimizations with Clang-compiled LLVM IR.

However, every way I tried to build it, the llvm intrinsics are never inlined and seem opaque to the optimizer. The optimizer doesn't even realize they don't have side effects, inhibiting dead code elimination.
The instructions say in bold "Note that the clang version must match the rustc LLVM version." I am using the same major version, 17, but not the same minor version. I am not sure if that has anything to do with it, because the LTO works well between rust crates when llvm-intrinsics is disabled.

version information

rustc 1.73.0 (cc66ad468 2023-10-03)
binary: rustc
commit-hash: cc66ad468955717ab92600c770da8c1601a4ff33
commit-date: 2023-10-03
host: x86_64-unknown-linux-gnu
release: 1.73.0
LLVM version: 17.0.2

Debian clang version 17.0.4 (++20231031083056+309d55140c46-1~exp1~20231031083156.63)
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

According to the rustc documentation, -C lto should be irrelevant when -C linker-plugin-lto is set. However, when C lto is "off" or "false", no LTO happens what so ever. Therefor, I am probably doing it incorrectly. I will still keep this issue open until I solve it because this could be an opportunity for documentation improvement.

I found one thing I was doing wrong: I did not set export CLANG=clang-17.  Unfortunately, it still isn't working, and I have yet to see linker-plugin-lto make any difference whatsoever. I know the environment variable is being respected because export CLANG=clang-18 causes the build to fail. (I don't have 18 installed).
To makes sure I wasn't just seeing the assembly output before the LTO took place, I used objdump to view the final linked executable. To make sure the issue wasn't the difference between 17.0.2 and 17.0.4, I switched to rust nightly. However, clang automatically updated to 17.0.5, so it still wasn't an exact match. I couldn't revert clang, but I marked it as hold to give rust nightly time to catch up.
What I will try next:

Follow the macOS instructions as closely as possible on Debian and see if that works.
Compile another library with external dependencies and see if I have the same issue.

OK - I think its an issue with the intrinsics build script. I see inlining when I comment out this:
diff --git a/intrinsics/build/main.rs b/intrinsics/build/main.rs
index bb063cf..6361fec 100644
--- a/intrinsics/build/main.rs
+++ b/intrinsics/build/main.rs
@@ -52,9 +52,9 @@ fn main() -> Result<(), Box<dyn Error>> {

     let linker_plugin_lto =
         matches!(env::var("RUSTFLAGS"), Ok(flags) if flags.contains("-Clinker-plugin-lto"));
-    if linker_plugin_lto {
+    //if linker_plugin_lto {
         build.flag("-flto=thin");
-    }
+    //}

     build.try_compile("ethnum")?;
     Ok(())


Discussion: How about make list node "blockable" like notion?

At present, we treat each list node as an entire block, similar to this.

However, a more refined approach would be to consider each indented list node as an individual block, akin to how it is done in Notion.

If my suggestion is accepted, I would be more than happy to contribute to the code.

If my suggestion is accepted, I would be more than happy to contribute to the code.

Awesome, thank you!
I think one thing we'd need to think about here is what exactly makes this more "refined"? I know that block-based editors are trendy right now, and I do see some benefits, but I'd like to understand what specific advantages adding the concept of more granular block to Lexical might confer?
If we can get past that to a place where we can see clear advantages that are worth the implementation trade-offs, I'm not opposed to this.

The only advantage I can think of is that such a function is essential for a Notion-like editor. If Lexical aims to create a general editing framework, then this kind of modification will only enhance Lexical's versatility and applicability.

I think the concept of "block" may evoke in many a rigid architecture with little flexibility, which is used by other editors like https://editorjs.io/
It is difficult in these editors to achieve an arbitrary HTML structure. Also, I think many make the mistake of reinventing arbitrary selection with CSS.
I think this is not what @aquarius-wing is proposing or what @acywatson is concerned about.
I suggest changing the title of this discussion to something like: "Making list-items draggable instead of list-nodes".
That's plausible without sacrificing anything in Lexical, just modify "DraggableBlockPlugin".
In fact, I've almost done it by using a function to split lists. The problem is that after dragging and dropping several list-items they start to fragment into contiguous lists. The only thing I would be missing is a way to combine lists. I'm experimenting with different strategies, meanwhile I've opened a thread here:
https://github.com/facebook/lexical/issues/4618

I think the concept of "block" may evoke in many a rigid architecture with little flexibility, which is used by other editors like https://editorjs.io/
It is difficult in these editors to achieve an arbitrary HTML structure. Also, I think many make the mistake of reinventing arbitrary selection with CSS.
I think this is not what @aquarius-wing is proposing or what @acywatson is concerned about.
I suggest changing the title of this discussion to something like: "Making list-items draggable instead of list-nodes".
That's plausible without sacrificing anything in Lexical, just modify "DraggableBlockPlugin".
In fact, I've almost done it by using a function to split lists. The problem is that after dragging and dropping several list-items they start to fragment into contiguous lists. The only thing I would be missing is a way to combine lists. I'm experimenting with different strategies, meanwhile I've opened a thread here: #4618

I cann't agree with you any more. May be we don't need to take such big steps, just modify the list-node is fine.

Hi @GermanJablo
Have you solved this problem yet? I want to drag and drop list-item instead of a list

What I did was not use @lexical/list and use my own list nodes which follows a flat model using indentation

Can you share in more detail how you handled it?
It would be great if you could show me your code


Creative and Newer Format for "Algorithms / Domain" Detail Tables, in ReadMe

Is your feature request related to a problem? Please describe.
As a beginner, while witnessing the ReadMe Section, it should be pretty clear about Algorithms / Tools.
Describe the solution you'd like
As more models would be added in the future, Models/Tools should not be only mentioned but categorized according to application field, with 1-2 line description and link to the class/function of the same.
Describe alternatives you've considered
More Creative and Neat, better will be ReadMe :)

Thanks to @Siddharth-Singh27 and @devyani-code, most of the Update part been done. Only Description Column is left. To be Done after Code Review.


Immutable JS Data from apollo

Hi,
We are trying to implement react-apollo in our react app :tada:  one of the blocker that we came across is:
How can I tell the apollo-client that I want the data stored into redux-store as an immutable object?
So that i can access it in this way:
const MyGqlComponent = ({data}) {
   return
data.myquery.get('myField')
;
}

I remember in the previous versions there was a prop called immutable available in the ApolloProvider that allowed me to do exactly that!
it doesn't seem to be there anymore, any help please

➤ stale[bot] commented:
This issue has been automatically labled because it has not had recent activity. If you have not received a response from anyone, please mention the repository maintainer (most likely @jbaxleyiii). It will be closed if no further activity occurs. Thank you for your contributions to React Apollo!

➤ Fabrizio commented:
Please can someone help with this

@jbaxleyiii did you ever find a solution?

@peggyrayzis any updates here?

I'd like to know if its possible to use immutable.js in apollo.

This should be more of an issue with apollo cache rather than apollo link state I imagine.


Add RealmFriend for java-binding and fix other build issues

Ideally Java should use the async query logic from OS, but since the
behaviours are not exact the same right now, Java will keep the current
async implementation for a bit while.
Expose SharedGroup and Group pointer by RealmFriend to make java's async
query work.

@beeender, it'd be preferable to separate build fixes from changes of substance in the future so build fixes can be landed without delay.
It'd be great if you could look at @JadenGeller's work on exposing a handover API from object store (#116) to see if it'll fit Java's needs. If it looks like it'll be a good fit it may be preferable to keep the change to exposeRealm methods on a Java branch of the object store until you've been able to  adopt enough of the object store API to use the new handover API.

See #128 and #127
Others stay in java branch for now.


SequenceFilter WIP

SequenceFilter by @danemadsen
refactored code for reuseablity

@danemadsen can you please review, does it do what you need?

Im working on a fix but its not ready yet, i'm trying to follow what your doing here though by abstracting out the class


Update logo and beta tag for Workflows

@parth-appsmith  - TBD

Hey team! Please add your planning poker estimate with Zenhub @ayushpahwa @nsarupr @Parthvi12


[KBM]Change UI strings to mention remap to text

Summary of the Pull Request
Keyboard Manager will now allow remapping to text as well. This PR tweaks the strings in the UI to mention the remap to text as well.

Validation Steps Performed
Ran Settings and KBM Editor and verified the strings were changed.

Thanks for the review and suggestions @htcfreek . I've incorporated those.


Support StringFormat without Converter in MultiBinding

This PR adds support for using StringFormat in MultiBinding even when Converter isn't specified, instead of throwing an exception. This feature is present in WPF.
Implementation is now closer to how it was done in Binding.
Side-effect of this change: Specifying neither StringFormat nor Converter will result in an object[] value, instead of throwing an exception.
Fixes #2590

[x] Updated unit tests
[x] Added XML documentation to any related classes

@jhofinger could you take a look and see if this is expected behavior now?

Thanks for the quick fix! In theory, yes, however, I can't get it to work for the AutoCompleteBox with MultiBinding in the ControlCatalog. When I specify the following:
var binding = new MultiBinding { StringFormat = "{0} ({1})" };
binding.Bindings.Add(new Binding("Name"));
binding.Bindings.Add(new Binding("Abbreviation"));
var multibindingBox = this.FindControl<AutoCompleteBox>("MultiBindingBox");
multibindingBox.ValueMemberBinding = binding;

I get a System.FormatException in the Convert method of StringFormatMultiValueConverter, and the values parameter shows two results of type Avalonia.BindingNotification with Error = Null value in expression "." You might want to add a handler for this.
Instead, I tried to set the multi binding from XAML (as in WPF). So I specified
<AutoCompleteBox.ValueMemberBinding>
      <MultiBinding StringFormat="\{0\} (\{1\})">
            <Binding Path="Name"></Binding>
            <Binding Path="Abbreviation"></Binding>
      </MultiBinding>
</AutoCompleteBox.ValueMemberBinding>

which of course conflicts with the correct format for format strings, but unescaped braces are interpreted as markup extension (see also #2592) so neither works for me.

@jhofinger I've fixed it now, sorry for the delay.
(The Converter was not "instanced", so since AutoCompleteBox causes Initiate to be called multiple times, the string format converter was being applied multiple times.)

@jhofinger does the recent change work for you?

@Grokys yes, it does - but only when setting it from code behind, and there I don't see a clear advantage over using FuncMultiValueConverter. The idea of StringFormat with MultiBinding would be to set it from XAML, which is still not possible, see my last answer. At least the user should be presented an error when trying to instantiate a MultiBinding with StringFormat in XAML.

Ok, I'll take a look at this.

Yep, confirmed that this is a problem with the XAML compiler: https://github.com/AvaloniaUI/Avalonia/issues/2592#issuecomment-505972730

Going to merge this and then we'll fix the XAML compiler.


Demo page update

@Demotive Hi Mat could you have a quick look at this when you get a moment. It's all of the work done to implement the property page demo for Ian, hoping the commits make sense, just want to check I haven't made any mistake/bad practice etc.
Thanks!

:+1: Nice commit messages :)

Going to merge, but going forwards it's worth making sure html and css indentation is tidy - it aids readability.
General style is two spaces for an indent - you can set this in your editor's preferences.


Algorithms

Implement different sorting algorithms with detailed documentation explaining them

I would like to work on it can i do it ?

Sure @Abhishek3223

Please assign this to me @Kunal-Kumar-Sahoo


Tracking site-packages in the venv

I realized after putting the Fx-signal copier files in the root of my venv and hitting 'git add . ' all the files under Lib are tracked too. they seem to be a lot, is it a must i track them too for the app to function properly on heroku?

Not entirely sure if I understand the context of this issue, but I do have a demonstration video showcasing step by step the installation from GitHub to deploy onto Heroku along with the steps in README. Here is a link to the video, let me know if this able to help you.
https://www.youtube.com/watch?v=QTCco30M3BY&t=5s


Changing enum which variables are exported silently breaks resources and scenes

Godot version
4.0.dev (1d14c054a12dacdc193b589e4afb0ef319ee2aae)
System information
Windows 11
Issue description
If user has exported enum that is serialized in scenes or resources and which values are frequently changed. all resources using this enum become silently broken if user changes numeration of existing enum values.
Example: I have simple object tags system, that are stored in enum and can tag objects in my game via editor fields
enum ETags {
  TagA,  # has implicitly set value 0
}

I have exported fields like this
@export
var DemoTag:ETags
@export
var Tags:Array[ETags] = []

When I set these values to TagA through editor they are shown in Properties View by their names and serialized to .tres/.tscn files like this
Tags = [0]
DemoTag = 0

Notice they are serialized not by name but by value. Now I want to add another tag to my tag collection before existing tag.
enum ETags {
  NewTag, # becomes set to 0
  TagA,  # now has value of 1
}

Property View value stays the same TagA (which is great) as serialized value stays the same old value of 0. And this breaks all saved resources with tags. Note: this will happen also with explicitly set enum values if numeration changes.
There are 2 possible ways of dealing with this problem:

Serialization of enums stays the same. After saving exported enum type, editor will have to recheck and resave all dependant resources. This will lead to avalanche of unwanted diffs on every new enum value.
Serialization of enums changes to saving names. This breaks compatibility with existing resources, but won't lead to their break. The only case when resources will break will be "User renamed some enum value". And this can be simply caught on resource load or export. This way is preferred for data driven developments pipelines.

Steps to reproduce
Explained in previous section
Minimal reproduction project
n/a

I think this will be nice to have but hard to implement. Because what you get with a enum is just an integer. Enum names are only shown in the Inspector.

Written further is written about solution that changes serialization to saving names.
I have very shallow knowledge about godot code but during serialization we deal only with exported properties, so:

we can safely assume that rule const bool isEnum = prop.type == Variant::Type::INT && prop.hint == PROPERTY_HINT_ENUM; is enough to check if property is of enum type.
If it's enum:
2.1. Parse hint_string for enum names, and then populate file from them. Bad for obvious reasons: enum names are made readable for user (spaces, capitalization, etc), also I don't know if hint_string is retained in non-editor build.
2.2. Enrich hint_string with enum type name. This one makes hint_string larger.
2.3. Or set property class_name with enum type name (if it's applicable)
In case 2-2, 2-3 we can get enum names from ClassDB via existing or new helper function.

@timothyqiu  I think topic should be core, not editor. It's serialization of core types issue.

Would be nice to have them saved as string by default, but here is a some sort of workaround (as stupid as it seems to be, it does to work):
Class GameConstants contains enum:
enum SurfaceType {
    DEFAULT,
    WATER,
    STONE,
}

In the script that should export this variable (normally with @export var type : GameConstants.SurfaceType):

Create a property with _get_property_list(), using values TYPE_STRING & PROPERTY_HINT_ENUM and parsing enums keys() into to hint_string (needs to be comma separated like "a,b,c")
override get/set and convert the string value to the enum value (untested as how would this behave if one defines enum key with manual index, like: SOME_ENUM_KEY = 21):

var surface_property: GameConstants.SurfaceType = GameConstants.SurfaceType.DEFAULT

func _get(property):
    if property == "/type":
        return GameConstants.SurfaceType.keys()[surface_property]

func _set(property, value):
    if value:
        if property == "/type":
            for i in GameConstants.SurfaceType.keys().size():
                if GameConstants.SurfaceType.keys()[i] == value:
                    surface_property = i

func _get_property_list():
    var property_list = []
    var types = ""
    for i in GameConstants.SurfaceType.keys().size():
        types += ("," if i > 0 else "") + GameConstants.SurfaceType.keys()[i]
    property_list.append({
        "type": TYPE_STRING,
        "name": "/type",
        "hint": PROPERTY_HINT_ENUM,
        "hint_string": types,
        "usage": PROPERTY_USAGE_DEFAULT
    })
    return property_list

then the property shown as below, and is saved to the file as: /type = "WATER"

Nice. Could be exploitable with macros in gdextensions or some wrapper lib


Infinite loop using AndroidKeyStore authentication

Saw this issue on phones:

Xiaomi Mi 5X
Samsung J3
LG Nexus 5

https://issuetracker.google.com/issues/119944680
https://stackoverflow.com/questions/53468190/android-keystore-usernotauthenticatedexception-infinite-loop
https://stackoverflow.com/questions/47920995/infinite-loop-using-androidkeystore-authentication

no longer reproducible, closing!


fix sprockets configuration

Thanks @masarakki - this PR fixes the problem with Sprockets for me.
:+1: for getting the PR merged into the gem

Could someone provide an explanation of the issue and how this fixes it? I haven't use jQuery templates in forever, and I'm pretty rusty on Rails these days, for that matter.

@jimmycuadra I'm not the author, but I ran into this today while upgrading a legacy Rails app. Sprockets has made significant changes to how registration of new mime type handlers are registered.
This is the best example I found today:
https://github.com/yasaichi/grease
This PR looks like it should work, but it doesn't for me. I tried some tweaking, using the Sprockets 4 syntax. But no luck, quite possibly because the templates I'm working with were written in haml

A further note -- the preferred way to set this up is no longer using an initializer, but putting something in config/application.rb. I think that's what this PR is attempting to fix.

That all being said, it looks like these changes now are calling for more than the current PR.

Am having a crack at committing the deprecated changes, if they pass our tests I'll file a PR :)


Fix swift 6 concurrency warnings

This will wait until Swift 6 is released.

This will result in a 2.0.0 release.

[ ] Remove .unsafeFlags(["-warnings-as-errors"]) before merge


NOTICE: The Uitzending Gemist tvOS App has reached its End of Life

I don't see any new programs appear since last Sunday (Jan 28). The live views work, the existing programs from the past weeks work, but the "By Day" view shows nothing, and the "Program" view shows no new episodes. Did NPO change something?

FYI, it looks like part of the old API is still available. I do a weekly update on all series, starting at http://apps-api.uitzendinggemist.nl/series.json, which seems still available.
Only the daily updates have indeed gone away (https://apps-api.uitzendinggemist.nl/broadcasts/2018-01-30.json)

@MKaterbarg Sure, the old API is still working, but there's less and less data added to it.
The series.json you mention does contain data, but (besides new series) you need details from:
../series/{series-id}.json requests, and those have no episode info (yet) of 30-01-2018 and 31-01-2018.
The latest episodes that were added in the last few days are from 28-01-2018 and 29-01-2018. I know this because I create RSS feeds from this old API.  My feeling is that NPO will slowly kill it by not feeding it new data.  That way users will seek refuge to new apps and the NPO website.

Ah thanks @MatrixView . My script hadn't run for a few days, so it still bound about 2000 new eps :)

Hah, nice work @dyve, that's a different API Key than I have :) That means I can publish the code I am working on using that publicly available API Key which makes it a lot easier :)
@Reino17, that tt888 endpoint is indeed the old one, the new media API (e.g.  https://start-api.npo.nl/media/AT_2083054) provides references to the SRT subtitles.
Like @MatrixView says, the old API is in part still functioning, but the content is lagging behind (if updated at all). When I have some time I'll create a new repo to publish the working code I have right now.

@4np Ah, that'd be great!
NPO seems to have a github channel too... with a different API.
https://github.com/npo-poms
...Unfortunately they didn't slip any APIkey and/or credentials in with that source. :-(

I have just published NPOKit. A Swift 4 Framework for interfacing with the NPO 2.0 APIs. Note though that this is still very much alpha and method signatures might change without notice. I have a working tvOS app as well, but I need to clean the code up a bit more before I publish it to GitHub.

@4np Thx for the NPOKit! I will look into it. Although I don't use Swift or even a Mac...  But I'm interested in the code and see what I can use for my shell scripts.
BTW.  the old API seems indeed lagging, as I've seen "new" episodes of January 30th and 31st roll in. And even some of yesterday. Let's wait and see...

As Swift is available on Linux, in theory it should be possible to use it in Swift based shell scripts (or compiled programs) as well... It doesn't have any dependencies other than Foundation, so perhaps it might already work?

@MatrixView: You can use it from the command line as well. And as Swift if available on Linux that might work as well (I haven't tested that use case yet). Please take a look at the HOWTO for a command line example...

@4np O, thx for the tips but please don't bother!  I'm using Powershell and Xidel under Windows. I've taken a quick look at your NPOKit and I'll probably manage to convert your http requests to a script when I have the time to do so.
In the meantime I'm using code from Reino17's BatchGemist to get my TVguide. (This scrapes the NPO-Gids webpage).
Thanx again for your work!  If I have any questions I'll know where to find you!
--MV

No worries, I just wanted to make sure that use case was covered as well :) That allows for people to write command line tools as well as by using a Swift server side framework (Vapor, Perfect, Kitura, etcetera) :)

I have just pushed the first working version of TV Gemist. It currently supports paging through programs, filtering programs, paging through episodes and playing episodes. Please take a look :)

@4np Jeroen, ook al heb ik geen AppleTV, bedankt voor het coden van TV Gemist! In dit soort projecten zitten altijd leerzame dingen voor me.

I'm just getting started with this wonderful app and I can report that this works for me.


Human readable output of models

We can support human readable output of models so user can evaluate what features are important or not. With #2727 it could be quite useful: one would be able to port the trained model to some simple thing (say put learnt linear SVM to your excel spreadsheet).
Suggested by kingsley_ in irc.
@karlnapf @lambday @sonney2k @vigsterkr @kevinhughes27 tell me what you think

It sounds quite nice. I see it very useful to use for the output of feature selection algorithms (like LMNN).

could be cool!

Like it, but this is not easily interpreted for example for SVMs.
Good for feature selectors as @iglesias said.
But, this is again something that is super easily done in a target language and increases complexity of our codes. So not sure whether its worth it.
We should rather use some data-storing backend to do such things. Like reading features of a HDF5 frame aka python-pandas. That would be a more sophisticated solution which also has the advantage that its done properly. To me, such ideas often seam to be a quick temporary hack that are not used later on but make maintaining harder. But also not against it

@karlnapf I don't see that much complexity in this - all we need is to make features aware of what they store which sounds like a good design anyway


Make @pattern work with function definitions in assignment form

@pattern currently only works with traditional function declaration syntax:
@pattern function f(:one)
    return 1
end

Implement support for assignment form:
@pattern f(:one) = 1

Fixed by #4.


Influxdb2 token based auth

Will it work with tokenbased authorization of influxdb2 ?

Hello do you get any solution for that


Update dApp and main readme without mainnet changes

Thank you for submitting this pull request :)
Partly fixes #1759
Short description
Updates the following readmes:

Main readme
dApp readme

But without the changes for mainnet. They reside within #2489
Definition of Done

[ ] Steps to manually test the change have been documented
[ ] Acceptance criteria are met
[ ] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

@agatsoh This file needs to be checked: https://github.com/raiden-network/light-client/blob/christianbrb-remove-mainnet/raiden-ts/README.md
At least the architecture diagram needs to get updated with the new persistance layer: https://github.com/raiden-network/light-client/blob/christianbrb-remove-mainnet/raiden-ts/README.md#architecture-diagram
Please check, if there is anything else to change and ask @andrevmatos if there is anything to change or adapt.

@agatsoh This file needs to be checked: https://github.com/raiden-network/light-client/blob/christianbrb-remove-mainnet/raiden-ts/README.md
At least the architecture diagram needs to get updated with the new persistance layer: https://github.com/raiden-network/light-client/blob/christianbrb-remove-mainnet/raiden-ts/README.md#architecture-diagram
Please check, if there is anything else to change and ask @andrevmatos if there is anything to change or adapt.

@christianbrb this line here needs to change
"Redux state is optionally persisted on localStorage or emitted to be persisted somewhere else. `
also needs to change.
According to me it should be
"Redux state is optionally persisted on the indexedDB on the browser if you use the raiden-dapp or if used with raiden-cli is persisted on the file system with leveldown db or emitted to be persisted somewhere else.
About the architecture diagram here
https://github.com/raiden-network/light-client/blob/christianbrb-remove-mainnet/raiden-ts/README.md#architecture-diagram
Instead of localStorage Andre told we can just write database.
If you want i could make the change here.

@christianbrb this line here needs to change
"Redux state is optionally persisted on localStorage or emitted to be persisted somewhere else. `
also needs to change.
According to me it should be
"Redux state is optionally persisted on the indexedDB on the browser if you use the raiden-dapp or if used with raiden-cli is persisted on the file system with leveldown db or emitted to be persisted somewhere else.
About the architecture diagram here
https://github.com/raiden-network/light-client/blob/christianbrb-remove-mainnet/raiden-ts/README.md#architecture-diagram
Instead of localStorage Andre told we can just write database.
If you want i could make the change here.


Brew install on OS X 10.12.3 having issues with serf version

Summary
Installing Kong 0.10 on OS X 10.12.3 using Homebrew:
brew tap mashape/kong
brew install kong
And then starting Kong with `kong start' I keep getting this error:
Error: /usr/local/share/lua/5.1/kong/cmd/start.lua:22: incompatible serf found. Kong requires version 0.7.0 to 0.8.0, got 0.8.1 Run with --v (verbose) or --vv (debug) for more details
I thought maybe I could just remove serf all together and let kong install the version it wanted, so I tried
brew uninstall --force serf
And then re-installing Kong but got the same output
I was able to get it working using the .pkg installer. After it worked properly, when I check the serf version, I see 8.0
Steps To Reproduce

Be on a computer with OS X 10.12.3
brew tap mashape/kong
brew install kong
kong start

Additional Details & Logs

Kong version ($ kong version)
0.10.0

(Belongs to Mashape/homebrew-kong)

(also, dup of https://github.com/Mashape/kong/issues/2070?)


TypeError: Cannot read property 'replace' of undefined (DocBuilder.js)

I am unable to run "esdoc -c myconfig.json" on my project due to an error that occurs in DocBuilder (we are using ES 6 & 7 code--with JSX--so I am using the esdoc-es7-plugin, but this issue happens without configuring the plugin). Output is:
badge.svg
E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\DocBuilder.js:483
            throw _iteratorError2;
            ^

TypeError: Cannot read property 'replace' of undefined
    at E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\DocBuilder.js:955:32
    at Array.map (native)
    at IdentifiersDocBuilder._buildTypeDocLinkHTML (E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\DocBuilder.js:950:43)
    at IdentifiersDocBuilder._buildSignatureHTML (E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\DocBuilder.js:1231:38)
    at E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\DocBuilder.js:512:38
    at IceCap.loop (E:\Source\GIMLI\web\orion\orion-prep\node_modules\ice-cap\out\src\IceCap.js:261:9)
    at IdentifiersDocBuilder._buildSummaryDoc (E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\DocBuilder.js:509:11)
    at IdentifiersDocBuilder._buildSummaryHTML (E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\DocBuilder.js:468:29)
    at IdentifiersDocBuilder._buildIdentifierDoc (E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\IdentifiersDocBuilder.js:78:40)
    at IdentifiersDocBuilder.exec (E:\Source\GIMLI\web\orion\orion-prep\node_modules\esdoc\out\src\Publisher\Builder\IdentifiersDocBuilder.js:52:32)

If I insert this code before line 955:
if(typeof tmp[1]==="undefined") {
     console.log("tmp[1] is undefined");
} else { console.log(tmp[1]); }

I see output like:
string
Array
ReduxActionHandler
string
Array
tmp[1] is undefined

esdoc-es7-plugin is old and defunct plugin which doesn't support ES6+, but simply tries to avoid / reduce parsing errors.
A future update to ESDoc will support Babylon, but I have a fork of ESDoc now that has full Babylon support with all plugins enabled including JSX. You can read more about it in #292. Give my fork a try and perhaps drop a note in #292 if it works for your project.

Thanks for the suggestion. I tried your fork, but am still hitting the same issue.

If I look at the value prior to splitting on the colon (again with println debugging), I see:
Value of 'v' prior to split: mutationString: string
Value of 'v' prior to split:  mutationObjects: Array
Value of 'v' prior to split: reduxUpdate: ReduxActionHandler
Value of 'v' prior to split: queryString: string
Value of 'v' prior to split:  queryObjects: Array
Value of 'v' prior to split: *

I assume that it is not likely comments like the following:
/**
 * Get server path which is dependant on whether or not aliases
 * are specified for the namespace and operation.
 * @param parsedNamespace
 * @param parsedOperation
 * @returns {*}
 * @private
 */

It looks like the @param tags are not formatted correctly to include type information.
https://esdoc.org/tags.html#-param
It should be @param {object} parsedNamespace - description. Of course fill in {object} with whatever type it should be. If you don't provide the type ESDoc will try and infer it (not as completely as possible).
That is not likely the problem, but worth trying out. In the code that is failing both sections where tmp[1] is preceded by the comment: this is a bad hack. It looks like you might have an object or function type information in docs incorrectly formatted somewhere.
As things go there are some "spartan" areas of ESDoc which can fail if the input doesn't match the expected formatting. The maintainer hasn't allowed outside collaborators, so movement is slow with ESDoc and it's a bummer more helpful error information isn't generated.
While my babylon fork indeed works better than the current master ESDoc branch I haven't fixed problem areas that are inherent in ESDoc itself.

Well the code here is matching expressions like { .* } and expects that the .* here is in the form paramName:type.  If you see the typeName variable before the split:
typeName: {*}
Value of 'v' prior to split: *

OK, got rid of those errors by removing all the @returns {*} that we had in our code. Would be nice to have a simple regex that validates if a tag is in the right format, though...

Better error reporting in general when ESDoc fails is pertinent. As things go something like that is a widespread change across the codebase versus a more contained PR and the single maintainer of ESDoc hasn't been too keen on making these kinds of changes. I suppose you can close this issue if you feel your problem has been resolved.


Publish latest version to registry

Hi,
I was encountering security issues when auditing because of the old Mocha version as a productive dependency.
I saw that you recently updated the Mocha version used and in 2020 also changed it to a dev dependency, but it seems like you did not publish it to the registry since then.
May I ask you to publish it so I can receive your patch via NPM?

The mocha version has no bearing on the security of actual use because it
is only used to test this module, but it would make sense for us to publish
yes.
On Thu, Jan 20, 2022 at 8:30 AM Marius Korte @.***>
wrote:

Hi,
I was encountering security issues when auditing because of the old Mocha
version.
I saw that you recently updated the Mocha version used, but it seems like
you did not publish it to the registry (last update 2 years ago).
May I ask you to publish it so I can receive your patch via NPM?
—
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/random-words/issues/25, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27JOR7MZ5DRTAKHML63UXAE6LANCNFSM5MMYFCXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

Yeah I know that, but as it is listed as a prod dependency in the published version from April 2020, the audit throws a warning.

Quite right. Would you like to submit a PR to make this a devDependency?
On Thu, Jan 20, 2022 at 12:11 PM Marius Korte @.***>
wrote:

Yeah I know that, but as it is listed as a prod dependency in the
published version from April 2020, the audit throws a warning.
—
Reply to this email directly, view it on GitHub
https://github.com/apostrophecms/random-words/issues/25#issuecomment-1017728386,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27JZ3JII6ELR5EBT35LUXA63FANCNFSM5MMYFCXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

This has already been done in #18. Unfortunately this change is also not in the registry yet.

This was published yesterday.


HDDS-7199. Implement new mix workload Read/Write Freon command which meets specific test requirements

What changes were proposed in this pull request?
In Cisco/Intel cluster, measure r/w performance when there is a very large of amount of metadata in rocksDB, & different amount of working sets whose size are larger than cache available.
Pure read
Pure write
Mixed workload, Read + Write
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7199
How was this patch tested?
Robot tests, manual tests.

cc. @kerneltime


Button Modification

When issues button is clicked, it shows the comments for the project.
Instead we can rename it to "Comments" and additionally add "Issues" button in navigation bar which redirects to issues in GitHub, this will be easy to use without any confusion.

Thanks for your interest, but I don't intend to add the GitHub issues link directly in the navbar. Also, comments are also named Issues, so having two issues buttons will create confusion. Additionally, your code has a syntax error in the enum.
I am closing this PR as I don't intend to keep this change.


TLS handshake error with Let's encrypt cert

With camlistored version: 2018-03-14-72ee5d1c93 and Go version: go1.9.4 (linux/amd64) when I try to connect to the UI my browser (Safari ) seams to time out.  In the log i get TLS handshake error from <IP_ADDRESS>:44646: tls: client offered an unsupported, maximum protocol version of 301.
curl tells me : curl: (35) Unknown SSL protocol error in connection to my.domain.example.com.eu:-9838.
config
{
    "auth": "userpass:example:secret:+localhost",
    "identity": "I_WONT_TELL",
    "identitySecretRing": "REDACTED/.config/camlistore/identity-secring.gpg",
    "blobPath": "REDACTED/camlistore/blobs",
    "packRelated": true,
    "levelDB": "REDACTED/camlistore/index.leveldb",
    "baseURL": "https://my.domain.example.com",
    "https": true,
    "listen": "<IP_ADDRESS>:443"
}

log
Mar 26 13:25:42 my-host systemd[1]: Started perkeep Daemon.

Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Index integrity check done.
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 [SyncHandler /bs/ -> /index/] Added 0 pending blobs from sync queue to pending list
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 ui: serving Closure from embedded resources
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index...
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index... (1/9: meta rows)
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "meta": 1 rows, 34.573µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "signerkeyid": 0 rows, 9.261µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index... (2/9: prefix "claim")
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index... (3/9: prefix "fileinfo")
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index... (4/9: prefix "filetimes")
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index... (5/9: prefix "imagesize")
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index... (6/9: prefix "wholetofile")
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index... (7/9: prefix "exifgps")
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Slurping corpus to memory from index... (8/9: prefix "mediatag")
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "mediatag": 0 rows, 23.302µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "claim": 0 rows, 14.275µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "fileinfo": 0 rows, 12.71µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "filetimes": 0 rows, 11.869µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "imagesize": 0 rows, 12.2µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "wholetofile": 0 rows, 10.295µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Scanned prefix "exifgps": 0 rows, 11.798µs
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Corpus stats: 0.006 MiB mem: 1 blobs (0.000 GiB) (0 schema (0 permanode, 0 file (0 image), ...)
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Corpus scanning CPU usage: 0s
Mar 26 13:25:42 my-host camlistored[23461]: 2018/03/26 13:25:42 Available on https://my.domain.example.com/ui/
Mar 26 13:26:08 my-host camlistored[23461]: 2018/03/26 13:26:08 http: TLS handshake error from <IP_ADDRESS>:42465: acme/autocert: unable to authorize "my.domain.example.com"; tried ["tls-sni-02" "tls-sni-01" "http-01"]
Mar 26 13:26:09 my-host camlistored[23461]: 2018/03/26 13:26:09 http: TLS handshake error from <IP_ADDRESS>:56988: acme/autocert: missing certificate
Mar 26 13:26:09 my-host camlistored[23461]: 2018/03/26 13:26:09 http: TLS handshake error from <IP_ADDRESS>:47217: tls: client offered an unsupported, maximum protocol version of 301
Mar 26 13:26:09 my-host camlistored[23461]: 2018/03/26 13:26:09 http: TLS handshake error from <IP_ADDRESS>:44646: tls: client offered an unsupported, maximum protocol version of 301
Mar 26 13:26:13 my-host camlistored[23461]: 2018/03/26 13:26:13 http: TLS handshake error from <IP_ADDRESS>:34259: acme/autocert: missing certificate

@zoranzaric is the baseURL value something you have redacted out as well, or is it actually https://my.domain.example.com ? And do you actually own my.domain.example.com?
I'm just making sure you know that you can't get a Let's Encrypt cert for a domain that you do not control.

Could it be the same problem as in https://groups.google.com/d/msg/camlistore/TbCyqN_DfHk/ahhx2ubyAAAJ ?

No, netstat shows me that camlistored listens on both 80 and 443.  So I guess this isnt't the problem.
on port 80 I get 302 redirect to https.

The problem was a wrong AAAA (IPv6) DNS record.
camlistored bound to :::80 for the Let's Encrypt http-01 challange.  It didn't bind to a IPv4 port at all and the IPV6 DNS record was wrong.
A paramter similar to "listen" for the http-01 challange would be great so one could tell camlistored where to bind to.

Discussed over IRC.

As mentioned on IRC, I think http.ListenAndServe (which is what the code calls), uses "tcp" as the default network to listen on. Which means (as explained in https://golang.org/pkg/net/#Dial) that it should work for both IPv4 and IPv6, if they are available.
So I don't think that the fact you are seeing it listening only on IPv6 is imposed by the code. It's probably another routing or OS configuration problem. Maybe you're not allowed to listen on low ports for IPv4 as regular user? or something like that.

I‘ll try to run as root later again.

Am 27.03.2018 um 16:53 schrieb mpl<EMAIL_ADDRESS>As mentioned on IRC, I think http.ListenAndServe (which is what the code calls), uses "tcp" as the default network to listen on. Which means (as explained in https://golang.org/pkg/net/#Dial) that it should work for both IPv4 and IPv6, if they are available.
So I don't think that the fact you are seeing it listening only on IPv6 is imposed by the code. It's probably another routing or OS configuration problem. Maybe you're not allowed to listen on low ports for IPv4 as regular user? or something like that.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

But yeah, I'll try to get to https://github.com/perkeep/perkeep/issues/1078 soon anyhow.


FixForEmptyStoreResponseStatisticsList

Fix for https://github.com/Azure/azure-sdk-for-java/issues/18126
Rootcause:
Currently we do verifyCanContinueOnException when creating storeResult, for PartitionKeyRangeGoneException, PartitionKeyRangeIsSplittingException, PartitionIsMigratingException we throw the exception before record it.
Fix:
First record the response, then do verifyCanContinueOnException
Before change: 	"responseStatisticsList": [],
After change:
"responseStatisticsList": [{
"storeResult": {
"storePhysicalAddress": "rntbd://cdb-ms-prod-westus2-fd28.documents.azure.com:14120/apps/436c3df8-cdfa-42b7-8f4c-0488f02160c6/services/fbfedf76-fde1-4642-b2bb-d15100160538/partitions/a6039e37-1a27-4d83-8345-de02217c84c0/replicas/132524124018886985s/",
"lsn": 0,
"globalCommittedLsn": -1,
"partitionKeyRangeId": null,
"isValid": false,
"statusCode": 410,
"subStatusCode": 1007,
"isGone": true,
"isNotFound": false,
"isInvalidPartition": false,
"requestCharge": 0.0,
"itemLSN": -1,
"sessionToken": null,
"exception": "The requested resource is no longer available at the server.",
"transportRequestTimeline": [{
"eventName": "created",
"startTimeUTC": "2020-12-16T19:52:03.400859900Z",
"durationInMicroSec": 49028
}, {
"eventName": "queued",
"startTimeUTC": "2020-12-16T19:52:03.449888500Z",
"durationInMicroSec": 0
}, {
"eventName": "channelAcquisitionStarted",
"startTimeUTC": "2020-12-16T19:52:03.449888500Z",
"durationInMicroSec": 274977
}, {
"eventName": "pipelined",
"startTimeUTC": "2020-12-16T19:52:03.724866Z",
"durationInMicroSec": 113352
}, {
"eventName": "transitTime",
"startTimeUTC": "2020-12-16T19:52:03.838218800Z",
"durationInMicroSec": 14998
}, {
"eventName": "received",
"startTimeUTC": "2020-12-16T19:52:03.853217200Z",
"durationInMicroSec": 32984
}, {
"eventName": "completed",
"startTimeUTC": "2020-12-16T19:52:03.886201400Z",
"durationInMicroSec": 2028
},
...
]

/azp run java - cosmos - tests

/azp run java - cosmos - tests

Please add CTL coverage. Any missing should be treated as bugs.

I remember around the time initial diagnostics in the consistency stack were implemented, DotNet SDK diagnostics for consistency stack evolved and hence due to not porting those changes from DotNet, diagnostics support for scenarios like PartitionSplit, etc are broken in the Java SDK.
I would recommend you digg in StoreReader.createStoreResult and go down you will find some discrepancies between how diagnostics in consistency stack is done comparing to DotNet. and that's what the source of PartitionSplit not captured in diagnostics

@moderakh Hi Mo, I checked the .net SDK code, and i see the logic there are still the same as we have today, so i think the issue could also exists in .Net as well

/azp run java - cosmos - tests

/azp run java - cosmos - tests


Does nothing

When I run private_gem new test nothing happens... no files are created and no output is displayed.
I'm using rbenv with Ruby 2.1.4 and Private Gem 1.1.0.

Fixed in #5


adding KERNEL_VERSION macro for linux.

Could we have a test like sigrt?

Thanks :)
@bors r+

:pushpin: Commit 88d6a1fd9b4b62729bd55a76e727f5f1d0aeef20 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 88d6a1fd9b4b62729bd55a76e727f5f1d0aeef20 with merge c59ca735304319dd4face349f49fb77f2164b544...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing c59ca735304319dd4face349f49fb77f2164b544 to master...


Upgrade vlucas/phpdotenv to ^4.0.1 & fix breaking change

Ref #484
vlucas/phpdotenv > 4.0.0 has a breaking change as the initialization method has been split up and the old method has been removed.
The new methods require you to specify if the env definitions are immutable or mutable.
This PR addresses this incompatibility by implementing the new Dotenv::createImmutable method. I don't think Bedrock is doing any overloading, so this is probably the better practice.

@kellymears Do you want to add this too?: https://github.com/roots/bedrock/issues/474#issuecomment-544714647
I was gonna create a PR (per swalk's request), but it would conflict with this PR, so might as well combine them.

no combine pls: keep these pr topics separate


TPH with re-used column names not supported

I have a TPH structure similar to the following where an inheriting class adds a property with the same name / column mapping as another inheriting class.
abstract class Mamal
{
}

class Person : Mamal
{
    public string Name { get; set; }
    internal new class Configuration : IEntityTypeConfiguration<Person>
    {
        public void Configure(EntityTypeBuilder<Person> builder)
        {
            builder.Property(x => x.Name).HasColumnName(nameof(Name));
        }
    }
}

class Pet : Mamal
{
    public string Name { get; set; }
    internal new class Configuration : IEntityTypeConfiguration<Pet>
    {
        public void Configure(EntityTypeBuilder<Pet> builder)
        {
            builder.Property(x => x.Name).HasColumnName(nameof(Name));
        }
    }
}

Using BulkInsert this yields the following exception:

System.ArgumentException: An item with the same key has already been added. Key: Name

Not currently supported.

This could be a very nice feature.


Failure with minizinc steiner triples example

minizinc -G or-tools -f ~/or-tools/bin/fz minizinc-2.0.10/doc/examples/steiner-triples.mzn
Unknown predicate: set_intersect

Sorry, set variables and constraints are not supported.


Fix #1564: Allow filtering of JS dependency manifests

This allows to resolve relative path conflicts in manifests inherited
from dependencies as the test shows. The typical example (for which we
provide a convenience filter) is a dependency that does not declare
specific enough relative paths.
Binary compatibility of PartialClasspath.resolve is a bit funky here:
The default parameter for the one argument version is now provided by
the parameter of the two argument version. However, since the values
equal, this is fine. Also, argument names need not be deprecated, since
the overloading resolution works properly, if filter is used.

I am not very sure about the general architecture and naming of this thing and would appreciate comments and suggestions of more general nature than mere code review.

Review by @sjrd

I have manually tested that the test does fail, if the filter is not provided.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1574/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2050/
Test FAILed.

I think the architecture is fine. It's about as general as it can get, without getting in the way either.

That's all.

Updated (except for the duplicate types)

Updated

LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1575/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2051/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1576/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2052/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1577/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2053/
Test PASSed.


Role verification for actions

Checks user's role for each action

Globally you should use the PHPDoc for all of your methods


cannot publish to pypi

version 1.1.4
Omicron depends module called bottleneck, to achieve better performance. However, this module doesn't have official apple m1 release. A private release has been made and been put on pypi.jieyu.ai.
However, pypi doesn't allow package which source isn't on pypi
When we try to release omicron 1.1.4 to pypi, the exception raise:
Error during upload. Retry with the --verbose option for more details.
HTTPError: 400 Bad Request from https://upload.pypi.org/legacy/
Invalid value for requires_dist. Error: Can't have direct dependency: 'Bottleneck @ https://pypi.jieyu.ai/simple/bottleneck/Bottleneck-1.4.0.dev0+117.gf2bc792.dirty-cp39-cp39-macosx_11_0_arm64.whl ; platform_machine == "arm64" and sys_platform == "darwin"

withdraw support for apple m1


✨ feat: luajit replace lua

Luajit replace lua
rm test case "cmsgpack can pack and unpack circular references". Becase luajit's core code diffrent from lua. Also, different compilers produce different code execution effects (GCC&clang on Ubuntu).

In order to use sanitizers, you not only need to add compile options, but also link options. (which is why CI is currently failing)
Also, don't forget to remove these sanitizer options when the bug is fixed.

This is  mem leak on my ubuntu

Many thanks for @xiaobiaozhao hard try

As I commented in discussion, Redis did some changes to lua codebases, not only lua extensions. You can check here to see these commits. I haven't investigated carefully what all these changes affect, and this may need some work.
Other comments are inline.

here The part of the lua code is modified for Redis 7.0, mainly the function function. I don't think this part needs to be ported to KVROCKS at this time

@tisonkun and @ShooterIT can take a look if you have time. For https://github.com/apache/incubator-kvrocks/pull/697#discussion_r910672053, I also prefer providing an option to let user fallback if they have any problem.

Maybe you need to add *.hpp to line 42 in lua.cmake to pass the "Ubuntu GCC without luajit" job.

Maybe you need to add *.hpp to line 42 in lua.cmake to pass the "Ubuntu GCC without luajit" job.

done

There are some memory leaks reported by ASan which are steadily reproduced and need some investigation.
The rest is good for me. 👍

There are some memory leaks reported by ASan which are steadily reproduced and need some investigation.
The rest is good for me. 👍

Those memory leaks looks not related to this PR.

Those memory leaks looks not related to this PR.

Seems these reported leaks are steadily reproduced for more than 10 times, and do not appear in other PRs and the unstable branch, so I think it is hard to say that it is not related to this PR.
I'll take a deeper look when I have time : )

Those memory leaks looks not related to this PR.

Seems these reported leaks are steadily (100%) reproduced for more than 10 times, and do not appear in other PRs and the unstable branch, so I think it is hard to say that it is not related to this PR.
I'll take a deeper look when I have time : )

I try to fix those issues in #714, looks cases like CommandFetchFile and CommandBPop are disappear, but still got a new leak warning and I'm investigating it. Guys can also have a look if you have time.

@tisonkun @PragmaTwice @ShooterIT I think we can merge this PR if no objections.

@tisonkun Agree, I also think this is a big change and we can release it on the next release.

Thanks for @xiaobiaozhao great contribution again.


feature: rendering table-like object in spreadsheet format

As sqlopsstudio displays the query result in spreadsheet format, It can be great to make table-like object (e.g, pandas.DataFrame, numpy.ndarray) rendered in similar way. AFAIK, Some IDEs already supported this feature, which makes inspecting data more efficient.

The Python Interactive window lets you view data in Pandas dataframes and numpy arrays. See more details on the blog post: https://blogs.msdn.microsoft.com/pythonengineering/2018/11/08/data-science-with-python-in-visual-studio-code/.


Abandoned?

Hi,
This API seems like the perfect fit for a new project of mine, but I'm wondering if it has been abandoned? No official 2.x support yet nor any commits in the main branch for 2 years. Should I assume that if we decide to use it we'll have to maintain it ourselves?
Thanks!

Maybe ask Django if they want contribution

Hi, I finally released the official Django 2 support in swapper 1.1.1.  I admit that there has been little activity here, primarily for two reasons:

The swapper API is relatively stable and has not needed any changes for over 2 years.  Most of the heavy lifting is done by Django.
The issues that do exist are related to the underlying Django implementation, so fixing them would require updates to Django.

In particular, there are a number of problems related to migrations (#10, #12, #14) that will be hard to address without a deep understanding of Django's internal model code.  While these issues may not be an dealbreaker for most common uses of swapper, they have become so for me.
I would say that swapper makes the most sense when there is at most one (or maybe two) models in your reusable application that you want to allow users to swap.  Once you have more than that, it gets to be a real mess for your users to keep track of migrations and remember where fields are defined.  For that reason, I am moving my own projects away from this approach (e.g. see wq/vera#7).
It is clear that swapper is useful for many people, and I will continue to accept PRs and make releases as I am able.  That said, I would be open to transferring ownership to someone with a more compelling ongoing interest in the project.

In case it helps, in @openwisp we're using this a lot, it is helping us to provide a standardized way to extend our modules and reduce boilerplate code.
Without swappable, if we wanted to allow creating customized versions of one of our modules, we had to separate the generic views from the concrete views, abstract admin from concrete admin, base tests from actual tests. With swappable we can avoid doing all that which makes our life easier and so far we have not encountered blockers.
So if you need help with maintenance, let us know.

@nemesisdesign, would you be interested in taking this project over (and presumably moving it to @openwisp)?  I can grant access to the PyPI package as well.  If you would like, I can make one last release incorporating #23 and potentially #21 (cc @pandafy).

@nemesisdesign, would you be interested in taking this project over (and presumably moving it to @openwisp)?  I can grant access to the PyPI package as well.  If you would like, I can make one last release incorporating #23 and potentially #21 (cc @pandafy).

@sheppard, yes, I am available for this, since we use this module extensively now, we'll take care of its maintenance.

@sheppard, yes, I am available for this, since we use this module extensively now, we'll take care of its maintenance.

Thanks @sheppard for you collaboration :+1: :+1:.
This project is now maintained by OpenWISP and @sheppard will help us during the transition.
No big changes are planned, only maintenance and bugfixes.

Thanks @sheppard for you collaboration :+1: :+1:.
This project is now maintained by OpenWISP and @sheppard will help us during the transition.
No big changes are planned, only maintenance and bugfixes.


Should button-variant function allow a manually-specified color?

In Bootstrap 3, the button-variant function/mixin had this signature:
@mixin button-variant($color, $background, $border)

You specified the (text) color, background color, and border. Other values (such as focus/hover colors) were computed automatically derived from these.
In Bootstrap 4, the button-variant function/mixi has this signature instead:
@mixin button-variant($background, $border, $hover-background: darken($background, 7.5%), $hover-border: darken($border, 10%), $active-background: darken($background, 10%), $active-border: darken($border, 12.5%)) {

You specify, at least, the background and border color. All other values are computed derived from those. You can optionally specify your hover and active border/backgrounds instead. The built-in uses of button-variant only specify $background and $border (and actually specify $border to be the same value as $background), and allow the default computation for the rest.
There is no way to manually specify the actual (text/foreground) color as an option though -- for default, hover, or active. You are stuck with the computed/derived value based on contrasting colors with specified/default backgrounds.
If you wanted to manually override this to something that worked better, for your particular palette or branding stylesheet, than the automatically computed value, you could do it by applying button-variant mixin and then providing CSS overrides -- but the need to deal with all the pseudo-selectors and nesting (which can be complex and non-obvious ) makes this kind of challenging to do in a reliable way.
Should button-variant function allow optional arguments with manually specified color values? Which if not provided, would still default to the automatically computed value with color-yiq and relevant background color.

Should button-variant function allow optional arguments with manually specified color values? Which if not provided, would still default to the automatically computed value with color-yiq and relevant background color.

That would seem like a nice enhancement.

For accessibility purposes it would be nice to define a unified style for the focus/hover states.
I would think overriding the following variables would override any other focus colors

$input-btn-focus-color
$input-focus-border-color
$input-focus-color

But it does not because of the dynamic color generation of https://github.com/twbs/bootstrap/blob/f1e6e6e4a19dac0fa08bc235c701cbdc99969eaf/scss/mixins/_buttons.scss#L24-L28


[feature] Ability to rename package names.

Currenly, Jadx is able to rename classes and field names, but not packages.
Usage cases:

With deobfuscation enabled, sometimes Jadx wrongly renames some packages, like android.support.v4 to android.support.p000v4, so we could be able to revert it back to v4.
Sometimes, package names are obfuscated, but its inner classes not. This feature could help to manually rename the obfuscated package name to a more descriptive name.

@alissonlauffer I have added rename for packages.
Although if package consist of serveral parts you can rename only one at a time:

Not sure how to implement it better.

I've been testing it for a while and found a regression: When renaming some packages, the renaming will happen in some of its subpackages instead of the package itself.
Try to rename android.support.v7 to something else in the following APK: https://drive.google.com/file/d/1RQesxeywrcHQveXEZGFiv9EfIczF2Q3X/view

@alissonlauffer I have fixed several issues. Please check :)

Okay, it's now working fine. Thanks!
One last thing, I think would be great if we could rename class files by right clicking their name (just like for packages) instead of opening the file and renaming main class name.

I implemented the rename function。https://github.com/pkilller/super-jadx/raw/super-jadx/demo.gif

I implemented the rename function

@pkilller I see you created a JADX fork (apparently based on version 1.0.0) with renaming feature, but that feature is already present in dev version and will be released in v1.2.0.

Looks like this issue can be closed now

Sorry if this is not the right place for such questions.
I have been trying this feature but I see that this only allows renaming that conforms the existing structure.  That is if a class is located under a.b.c I cannot rename it to a.b.c.d.
May I ask if there are any existing method that can allow me to do so?

My previous comment can be ignored, sorry for bothering.
I think my issue is described in #1590.
Not sure why it didn't show up before when I searched for it.


Error calling php artisan migrate --seed

This used to work! See my comment here https://github.com/guillaumebriday/todolist-backend-laravel/issues/5
Note: I am on Mysql version 8
mysql  Ver 8.0.16 for osx10.13 on x86_64 (Homebrew)

Commands I ran:
$ git clone https://github.com/guillaumebriday/todolist-backend-laravel.git
$ cd todolist-backend-laravel
$ cp .env.example .env
$ composer install
$ php artisan key:generate
$ php artisan jwt:secret
$ php artisan migrate --seed

Error:
   Illuminate\Database\QueryException  : SQLSTATE[HY000] [2002] php_network_getaddresses: getaddrinfo failed: nodename nor servname provided, or not known (SQL: select * from information_schema.tables where table_schema = laravel-todolist and table_name = migrations and table_type = 'BASE TABLE')

  at /Users/kunalpunjabi/Code/LARAVEL_WORKING_CODE/NUXT/todolist-backend-laravel/vendor/laravel/framework/src/Illuminate/Database/Connection.php:664
    660|         // If an exception occurs when attempting to run a query, we'll format the error
    661|         // message to include the bindings with SQL, which will make this exception a
    662|         // lot more helpful to the developer instead of just the database's errors.
    663|         catch (Exception $e) {
  > 664|             throw new QueryException(
    665|                 $query, $this->prepareBindings($bindings), $e
    666|             );
    667|         }
    668|

  Exception trace:

  1   PDOException::("PDO::__construct(): php_network_getaddresses: getaddrinfo failed: nodename nor servname provided, or not known")
      /Users/kunalpunjabi/Code/LARAVEL_WORKING_CODE/NUXT/todolist-backend-laravel/vendor/laravel/framework/src/Illuminate/Database/Connectors/Connector.php:70

  2   PDO::__construct("mysql:host=mysql;port=3306;dbname=laravel-todolist", "root", "root", [])
      /Users/kunalpunjabi/Code/LARAVEL_WORKING_CODE/NUXT/todolist-backend-laravel/vendor/laravel/framework/src/Illuminate/Database/Connectors/Connector.php:70

Did you run a mysql server in docker or on your host ?

I have always hated docker and consider it an unnecessary complexity.
I host mysql on my localhost. It works 100% of the time with other codebases, so I'm not sure what's going on here...

Ok if you want to use mysql on your host you need to change the .env like so:
DB_CONNECTION=mysql
- DB_HOST=mysql
+ DB_HOST=<IP_ADDRESS>
DB_DATABASE=laravel-todolist
DB_USERNAME=root
DB_PASSWORD=secret

it should work

Ok that worked...thanks! Not sure why DB_HOST was not set to localhost...typically is

It's not because in this definition: DB_HOST=mysql, mysql refer to the docker-composer service named mysql
And Docker use its internal DNS to resolve mysql in other containers to the mysql container ip

Ah I see...so it had to do with the docker / non-docker environments..I'll make a note of this for next time. Not everyone is using docker :)


added support for xzing library, removed qr-encode

changed code to use libzxing removing the need to compile zxing on the target system.
moved back to Berkeley db 4.8 as db 5.1 is not available as a package for some Linux distributions
moved to new autogen.sh
adapted .gitignore to new filenames (e.g. bitcoin-qt -> feathercoin-qt)

Just checking we can move back on database versions, seems we can. At one stage I remember you couldn't, at least with a straight "upgrade", without re-downloading the blockchain?
https://bitcointalk.org/index.php?topic=502482.0
"The main reason it is discouraged is because it breaks compatibility with the pre-built binaries. If you're building from source you may not care.
 Use the bdb tools (packages db4.8-util db5.1-util on Ubuntu):"

The problem , that db 5.1 is not available as a standard package on all linux distributions.
This breaks the automatic solving of dependencies done by the different package managers.
Reloading the BC is the worst case and a backup of the wallet data or a dump of the wallet private keys should be advised to the user anyway.
We need to decide which version of the DB to use.
Lizhi took a lonesome decision here....


Fix creation of the default uniform

The original instantiation is incorrect.

Lol :) Thank you!

That's why I'm writing all new plugins on typescript.


Create an OrtResult abstraction

Create an abstraction for the OrtResult to decouple the ORT tools from the way that ORT data is stored.
Currently, we access the data stored in OrtResult directly in many places. This has the downside that the tools are tightly coupled to the OrtResult class, while the OrtResult class has to be optimized for serialization at the same time, because it is used to write the result files. Creating a facade (e.g. an interface of a collection of interfaces) would have several benefits:

We could provide different means to store ORT result data, for example a local database, and it would be easier to experiment with alternative storage formats like Protobuf.
Breaking changes in the ORT result file format would not necessarily require changes in the interface.
Tests often require to build large parts of an OrtResult or even a complete OrtResult, when they actually only use a much smaller part of it. In those cases mocks could be used to reduce the size of the tests.

This does not only affect the reading of result data but also the creation. One possible approach could be based on the builder pattern: Instead of creating result objects directly, ORT components are passed a builder they can use to propagate their results.

FYI, there's a similar topic for LicenseClassifications. Maybe it makes sense to try out a new API approach first in the context of such a smaller / isolated class.


Fix CI test discovery

Reopened from upstream branch to accept changes from entire team

@marstr at this point the only thing blocking this should be the azure-mgmt-reservations issue.

Small fix to get azure-mgmt-reservations patched quickly in 77f533d, and issue #9781 opened to follow up.


Can't find the functionality

The functionality described in this article is exactly what we require at a client but we can't seem to find any of the forms or functionality described. For example Warehouse management > Loads > Transport loads or the mobile device menu item for transport loading. Is this due to the version we are using or are we missing something here? The environments we have checked are mostly Update12 (7.0.4709.41190)

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

ID: 91ab4dcf-7c32-a35a-c5d4-768aa13b3fac
Version Independent ID: 7b993e54-f072-ff8a-dbf3-a713a3a1fd30
Content: Partial shipment of a transport load - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/supply-chain/warehousing/partial-shipping-of-transport-loads.md
Service: dynamics-ax-applications
Product: unspecified
GitHub Login: @Mirzaab
Microsoft Alias: mirzaab

Dear ,
We are very happy that you are interested in our partial shipping functionality.
We do have this functionality in our latest version 8.0.
So it is due to the version you are using.
Best Regards
Mirza Abdic'
Senior Program Manager

Hi Mirza
Thank you for the feedback. We provisioned a new demo environment of version 8 and found the fields and functionality you documented.
I just wondered how this functionality works with a packing station process. Is a manual packing process supported with the transport loading functionality?
Thanks in advance
Emily
From: Mirzaab<EMAIL_ADDRESS>Sent: Tuesday, 05 June 2018 06:32
To: MicrosoftDocs/dynamics-365-unified-operations-public<EMAIL_ADDRESS>Cc: emily<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics-365-unified-operations-public] Can't find the functionality (#151)
Dear ,
We are very happy that you are interested in our partial shipping functionality.
We do have this functionality in our latest version 8.0.
So it is due to the version you are using.
Best Regards
Mirza Abdic'
Senior Program Manager
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/dynamics-365-unified-operations-public/issues/151#issuecomment-394578214, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Al84rK0LtftWOkCHEZUKB91ug2C0U7pBks5t5gnIgaJpZM4UUt-B.

Hi Emily, to discuss product functionality, rather than the documentation, please use the Finance and Operations forum:
https://community.dynamics.com/365/financeandoperations/f/765


Extend accessibilityLabel support to aria-label prop alias

Feature description
The idea would be to use aria-label the way accessibilityLabel is for interaction tracking
Background
React Native 0.71.3 added support for "web inspired" prop aliases [docs]

This release includes a number of new props inspired by web standards to align React Native’s APIs across many platforms. These new props are purely additive so there are no expected migrations or change of behavior for equivalent accessibility, behavior, or style props.
For any new prop alias introduced, if there is an existing prop with a different name and both are specified, the new alias prop value will take precedence. For example, this release adds a src prop alias for source on the Image component to align with the src prop on web. If both src and source are provided, the new src prop will be used.

This includes support for aria-label as a prop alias of accessibilityLabel

[!NOTE]
The RFC and discussion, linked from the docs, may be useful, for additional context

Proposed solution
Extend the existing user interactions tracking feature, to support aria-label in addition to accessibilityLabel
https://docs.datadoghq.com/real_user_monitoring/mobile_and_tv_monitoring/setup/reactnative/#user-interactions-tracking
Other relevant information
It may be useful to consider parity with browser RUM, which also includes support for label and placeholder
I think in theory it would be possible but support for these in RN Is not a standard

label - aria-labelledby /  accessibilityLabelledBy
placeholder - aria-placeholder* placeholder

*not yet supported

Hi @leggomuhgreggo 👋
Thank you for your detailed suggestion, it is definitely something we want to introduce to align our SDK with this new React Native standard.
We have included this in our roadmap :)