无法覆盖同名的内置插件

在IPublicTypeEditorView的ctx中覆盖同名的内置插件没有生效是什么原因？lowcode-engine版本是1.2.0

需要先 remove 掉原来的插件面板
skeleton.remove({
  area: 'leftArea',
  name: 'outline-master-pane',
  type: 'PanelDock',
});

skeleton.remove({
  area: 'rightArea',
  name: 'outline-backup-pane',
  type: 'Panel',
});

好的，我试试

忘了补充，是在workspace模式下

那就需要在对应的试图里面通过插件进行 remove 和注册。

比如我要覆盖的是outline插件，能否具体给一段代码参考一下

await plugins.delete('OutlinePlugin');
我是这样写的，没有生效

你需要移除的是内部注册的面板，而不是插件。

我应该在什么时机对这个内部注册的面板进行remove

remove了然后add，没有生效

提供可复现的 demo 吧。

https://codesandbox.io/p/github/shiwofeiwo/editor-demo/main?layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522clo51xovk0007356jxde2w61d%2522%252C%2522sizes%2522%253A%255B100%252C0%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522clo51xovj0003356jyeczfbq5%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522clo51xovj0005356js68o7xqn%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522clo51xovj0006356jj3qyybc0%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B100%252C0%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522clo51xovj0003356jyeczfbq5%2522%253A%257B%2522id%2522%253A%2522clo51xovj0003356jyeczfbq5%2522%252C%2522tabs%2522%253A%255B%257B%2522type%2522%253A%2522FILE%2522%252C%2522filepath%2522%253A%2522%252Fsrc%252Fplugins%252Fplugin-outline-pane%252Findex.tsx%2522%252C%2522id%2522%253A%2522clo52d6k600c7356j646vfy8g%2522%252C%2522mode%2522%253A%2522temporary%2522%257D%255D%252C%2522activeTabId%2522%253A%2522clo52d6k600c7356j646vfy8g%2522%257D%252C%2522clo51xovj0006356jj3qyybc0%2522%253A%257B%2522id%2522%253A%2522clo51xovj0006356jj3qyybc0%2522%252C%2522activeTabId%2522%253A%2522clo525qw900of356jundauxr8%2522%252C%2522tabs%2522%253A%255B%257B%2522type%2522%253A%2522ENV_SETUP%2522%252C%2522id%2522%253A%2522clo51xp0i000p356jtkpb0otn%2522%252C%2522mode%2522%253A%2522permanent%2522%257D%252C%257B%2522type%2522%253A%2522TASK_PORT%2522%252C%2522taskId%2522%253A%2522start%2522%252C%2522port%2522%253A5556%252C%2522id%2522%253A%2522clo525qw900of356jundauxr8%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522path%2522%253A%2522%252F%2522%257D%255D%257D%252C%2522clo51xovj0005356js68o7xqn%2522%253A%257B%2522id%2522%253A%2522clo51xovj0005356js68o7xqn%2522%252C%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clo51xovj0004356jl5oim4fr%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TERMINAL%2522%252C%2522shellId%2522%253A%2522clo51xpll002xegcndscm71c2%2522%257D%255D%252C%2522activeTabId%2522%253A%2522clo51xovj0004356jl5oim4fr%2522%257D%257D%252C%2522showDevtools%2522%253Afalse%252C%2522showShells%2522%253Afalse%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D
这是复现的demo

后来怎么解决的？

同问，是否已解决？怎么解决的？


 /usr/bin/ld: cannot find -lz when building the backend

I'd like to try the compile bundle from there https://github.com/intel/intel-xpu-backend-for-triton/pull/83 but the very last step (building triton) fails with
 /usr/bin/ld: cannot find -lz
a solution is suggested here https://github.com/intel/intel-xpu-backend-for-triton/wiki/Possible-Build-Bugs#1-usrbinld-cannot-find--lz but I'm not root on the system so I can't install this package.
I also have zlib install in the conda environment, or I tried to extract the package and use LD_LIBRARY_PATH to point to other locations, it doesn't help.
Is there a way to do without system zlib ?

You can explicitly set zlib path with patch.
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 7dd62d1a3..d71a33362 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -203,6 +203,8 @@ set(TRITON_BINARY_DIR "${CMAKE_CURRENT_BINARY_DIR}")
 get_property(dialect_libs GLOBAL PROPERTY MLIR_DIALECT_LIBS)
 get_property(conversion_libs GLOBAL PROPERTY MLIR_CONVERSION_LIBS)

+find_package(ZLIB)
+
 if(TRITON_BUILD_PYTHON_MODULE)
   add_library(triton SHARED ${PYTHON_SRC})
   set(TRITON_LIBRARIES
@@ -235,11 +237,11 @@ if(TRITON_BUILD_PYTHON_MODULE)
       ${TRITON_LIBRARIES}
     )
   elseif(APPLE)
-    target_link_libraries(triton ${LLVM_LIBRARIES} z
+    target_link_libraries(triton ${LLVM_LIBRARIES} ZLIB::ZLIB
       ${TRITON_LIBRARIES}
     )
   else()
-    target_link_libraries(triton ${LLVM_LIBRARIES} z
+    target_link_libraries(triton ${LLVM_LIBRARIES} ZLIB::ZLIB
       ${TRITON_LIBRARIES}
     )
     # TODO: Figure out which target is sufficient to fix errors; triton is

Also export CMAKE_PREFIX_PATH=${CONDA_PREFIX:-"$(dirname $(which conda))/../"}

@fcharras can we close this ticket?

Yes, to be honest I gave up trying to build myself as a curious external early tester. but I'm looking forward binary releases (see https://github.com/intel/intel-xpu-backend-for-triton/issues/275 ) ;-) .


Standardize Header Handling when creating new nibabel object

Checklist
GitHub

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[ ] I've applied the relevant labels to this PR
[x] I've assigned a reviewer

PR contents

[ ] I've consulted ivadomed's internal developer documentation to ensure my contribution is in line with any relevant design decisions
[ ] I've added relevant tests for my contribution
[ ] I've updated the relevant documentation for my changes, including argparse descriptions, docstrings, and ReadTheDocs tutorial pages

Description

In PR https://github.com/ivadomed/ivadomed/pull/714 the way nibabel objects are created was changed, so we want to update the object creation in the following places as well

uncertainty
preprocessing
postprocessing
visualize
loader/adaptive
evaluation

Linked issues

Timeline of the related issues:

Original Issue raised #711 and also summarized at https://github.com/spinalcordtoolbox/spinalcordtoolbox/issues/3283
Subsequent PR fix #714.
(which created issue: #720 and that test needs to be eventually re-enabled)
Recent issue #775 discovered when using Affine=None when creating new NifTI image, meaning upstream issue resolution needs to reach consensus among IvadoMed/Neuropoly devs.
The likely proposal is to either implement:

Nifti1Image(data, img.header.get_best_affine(), img.header)  OR
Nifti1Image(data, img.affine, img.header)

Resolve https://github.com/ivadomed/ivadomed/issues/715
Resolve https://github.com/ivadomed/ivadomed/issues/775

all changed except for
https://github.com/ivadomed/ivadomed/blob/fb3c913ddc3fb1942884bdbb16a7796e94d20192/ivadomed/loader/adaptative.py#L753

Pull Request Test Coverage Report for Build 752289338

2 of 8 (25.0%)  changed or added relevant lines in 5 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage remained the same at 76.389%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ivadomed/evaluation.py
0
1
0.0%

ivadomed/visualize.py
0
2
0.0%

ivadomed/uncertainty.py
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

ivadomed/evaluation.py
1
11.71%

ivadomed/visualize.py
1
37.76%

ivadomed/uncertainty.py
2
11.83%

Totals

Change from base Build 745629067:
0.0%

Covered Lines:
5555

Relevant Lines:
7272

💛  - Coveralls

Going to need a bit more time to review this. Converting to draft for now.

I think you just missed a few of the places when the NifTI images are instantiated, the header didn't get copied over.

please see https://github.com/spinalcordtoolbox/spinalcordtoolbox/issues/3295#issuecomment-833966224. I don't think it is the best strategy to implement the suggestion of https://github.com/ivadomed/ivadomed/issues/715#issue-831306354 without strategizing on the potential consequences (e.g. https://github.com/nipy/nibabel/issues/1012).

@jcohenadad acknowledged. Will low prioritize this while monitoring that upstream thread. Thanks for the heads up. I somehow thought it had been resolved. Will update PR with the missing contextual details.

someone approve this https://github.com/ivadomed/data_functional_testing/pull/6

was passing but now failing after https://github.com/ivadomed/ivadomed/pull/818 got in T.T

note this test is very long compare to all other tests...

note this test is very long compare to all other tests...

what test are you referring to? how long is it compare to other tests?

note this test is very long compare to all other tests...

what test are you referring to? how long is it compare to other tests?

testing/functional_tests/test_cascaded_architecture_tutorial.py, around 23 min for this test...

note once https://github.com/ivadomed/ivadomed/pull/865 this got in, we don't need cascade tutorial test here anymore

This PR's change has been empirically validated in yd/tutorial_testing as of 2021-11-05T194421EST.  This PR will be merged first as part of the Python 3.9/Torch 1.8 adoption. This addressed the issue in Tutorial 2 crash that is currently happening in the master branch.


No UI or terminal

name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20171230152552~master:de1361af
release: xenial
architecture: amd64
xmethod: xorg
targets: xorg,xiwi,extension,keyboard,touch,unity-desktop
host: version 10176.76.0 (Official Build) stable-channel lulu
kernel: Linux localhost 3.14.0 #1 SMP PREEMPT Sat Feb 24 13:28:49 PST 2018 x86_64 x86_64 x86_64 GNU/Linux
freon: yes

I've just removed my old trusty chroot and installed a new xenial one to see if it would fix previous issues.
I can enter the chroot via sudo startunity but beyond there I can't progress. I'm presented with the chroot's desktop but no menu or navigation bars. I can right-click, and from that dialogue I can access the settings menu and I can create new documents/folders onto the desktop. Clicking into each opens gedit and nautilus respectively, and both seem to work as expected.
I cannot close any windows once opened, minimise/maximise, or drag them around on the desktop. The buttons to do so do not appear.
Alt+Tab allows me to switch between any opened windows, but not the desktop.
I cannot open the terminal from the right-click menu (despite the option being presented) or via Ctrl+Alt+T.
Running from ChromeOS sudo enter-chroot xiwi -T gnome-terminal outputs Error constructing proxy for org.gnome.Terminal:/org/gnome/Terminal/Factory0: Error calling StartServiceByName for org.gnome.Terminal: GDBus.Error:org.freedesktop.DBus.Error.Spawn.ChildExited: Process org.gnome.Terminal exited with status 8
Running from ChromeOS sudo enter-chroot xiwi -T xterm works as expected.
I have updated the chroot using sudo sh ~/Downloads/crouton -n xenial -u, and problems persist.
Any assistance is much appreciated.

I fixed this by reinstalling with "unity" as a target instead of "unity-desktop", then updating the chroot with the "unity-desktop" target.

I Have the same problem with you someone help?

I Reset my Chrome OS nothing works then i install ubuntu again and enter the same username and password .
The Question › did the data was saved when i enter an existing username and password then it would load all setting i made and data or etc?
Sorry for bad english.

@Geraldalingod,
It depends on what you mean by 'Reset my Chrome OS', if you did a 'powerwash' or 'full recovery' then no, nothing is saved, it's gone.
A Chrome OS update and/or a crouton update should leave all of your user data and settings intact but, apart from that, it will get removed.
Hope this helps,
-DennisLfromGA

@rossjrw,
Glad you figured a way around the issue and got it fixed.
It's a bit strange since installing the unity-desktop target requires the installation of the unity target as a prerequisite but I guess stranger things have happened.
-DennisLfromGA


ugit (via docker) does not resolve that it is in a git repo.

What happened?
Attempted invoking ugit via

docker pull bhupeshimself/ugit
docker run --rm -it -v $(pwd):/app bhupeshimself/ugit

Even in the ugit repo that has been cloned   (or any other git repo), always get the following message:

Ummm, you are not inside a Git repo  😟

Test Repo

/home/auser/Projects/TestGitRepo
total 58
drwxrwxr-x   3 auser auser   4 Feb  5 15:45 .
drwxrwxr-x 300 auser auser 305 Feb  5 15:58 ..
drwxrwxr-x   8 auser auser  14 Feb  5 15:45 .git
-rwxrwxr-x   1 auser auser 147 Feb  5 15:44 ungitInvoke

Using the script directly works fine.
Maybe a missing environment variable or volume mapping that is unaccounted for?
Ugit Version
v5.1 (Latest)
Bash Version
Bash >= 5.0
Git Version
Whatever versions that are in the current docker image for all of the above.
❯  docker images

REPOSITORY
TAG
IMAGE ID
CREATED
SIZE

bhupeshimself/ugit
latest
6c57f819c864
2 days ago
14.9MB

What OS are you seeing the problem on?
Linux Based
Terminal Name
SSH'ed into Ubuntu 22.04 host.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@fargly Thanks for reporting,
Can you try this variation of sharing file permission while starting a container
docker run --rm -it -u $(id -u):$(id -g) -v $PWD:/app bhupeshimself/ugit

Worked like a charm

❯  docker run --rm -it -u $(id -u):$(id -g) -v $PWD:/app bhupeshimself/ugit
Undo your last oopsie in Git 🙈
Press ctrl+c to exit anytime

Thanks for your attention on this matter.
Also, that write up for a minimal bash docker image was very interesting.
Cheers


Query can end up doing a POST!

Using latest alpha (master) I have the following query:
Parse::Query.new('QuizUserGrade').tap do |q|
  q.value_in('groupMembership', group_memberships.map(&:pointer))
end

Which works fine until you have too many group_memberships. (breaks with 25, works with 10). Once you have too many it does a POST instead of a GET and you end up creating a record when you are trying to execute a query. Still digging but this is a bad bug.

I think it's related to the Faraday::GetMethodOverride option. Parse isn't handling this and is instead creating an object with a where attribute.

@crayment can you contribute a patch or at least provide more detail about the issue? Thank you

I think technically this is a bug with open source parse not respecting the X-Http-Method-Override header. This leads to the nasty behaviour of it just seeing a POST with a giant where attribute and so it creates an object.
I resolved this for now by disabling the GetMethodOverride middleware by doing client.session.builder.delete Faraday::GetMethodOverride.
I would suggest removing this middleware (delete this line) until it's confirmed that parse is correctly handling the header. This will likely bring back issues with large GET requests but is much better than the alternative at this point.

@crayment have a look at this PR, let me know if it's satisfactory for you. Instead of outright deleting the middleware I just made it so you can disable it.

@rhymes #211 Looks great and gives users a good way to resolve the issue, thanks! The only thing I would change is defaulting to false (at least when a custom host is specified?) but your judgement is better here on what users would expect.

@crayment I agree with you, it should default to false, the issue though is that it might break existing user's code that's relying on the, now, implicit override switch.
@xavdid what do you think about this issue?

Nice!
I think we should default it to true because we don't want to be changing people's code out from under them. If they're having an issue (like you were) we've given them an avenue to solve it, but their exiting code won't change unexpectedly.
Related, we should bump the version number when we make changes. Even though we're not releasing to rubygems, bundler will still respect the version number and when there are actual changes like this, it'd be good to signify that somehow. I would think this would be a minor release.
Otherwise, looks awesome.

@xavdid you mean false then ? false would be the current behavior (with the potential problem of override ending up a post for large GETs which would be fixed by setting the flag as true).
I agree wth the version bump, I'll add it to the PR

@rhymes hmm? If I'm reading the change correctly, c.use Faraday::GetMethodOverride currently runs each time now. Functionally, c.use Faraday::GetMethodOverride if true is identical, so true should be the default so we don't change the code that relies on this bug.
The option to fix it is good, and we can default it the other way if we do a major release (and really in most cases, it shouldn't matter).

@xavdid yeah sorry, tired eyes! thanks for the patience.
I'll merge the PR


My ionic 2 tab template  www folder is empty...

From @jackryu on January 20, 2017 22:27
Hello Team,
I am new with ionic based mobile application development .

I am try everything  but my ionic 2 application tab template WWW folder is   empty and ionic 1 is working.

ionic start --v2 myApp tabs (this Command is create blank www folder)
ionic start  myApp tabs (and this command is working fine)
but i need to working with ionic 2
_# Already try below things are not working _

all dependency install again after uninstalling.
cordova and ionic install one by one
update node version and degrade node version.
degrade  cordova  version 4.2.2
add sudo in every command

----Configuration-----
OS
Windows 10
GIT
git version 2.9.2.windows.1
IONIC
2.2.1

Dependency warning - for the CLI to run correctly,
it is highly recommended to install/upgrade the following:
Please install your Cordova CLI to version  >=4.2.0 npm install -g cordova

2.2.1
NODE
v6.9.4
CORDOVA
You have been opted out of telemetry. To change this, run: cordova telemetry on.
6.4.0
Regard's
Abhishek Tomar
email<EMAIL_ADDRESS>Copied from original issue: driftyco/ionic#10117

From @chrisgriffith on January 20, 2017 22:35
Yes, that is the correct behavior for an Ionic 2 app until you build the app using the ionic build  command. If you use ionic serve command, the www folder will be populated as well.

From @jackryu on January 20, 2017 22:46
but i am running my server but this is not working -

From @chrisgriffith on January 21, 2017 0:37
But did you run either ionic build or ionic serve? The new ionic app scripts will take the contents of the src directory and then generate the contents of the www directory.
If nothing is being generated, then review the terminal logs to see if there is an error during the build.

@jackryu,
Please explain more of your issue.
Thanks,
Dan

Due to lack of response, we're going to close this issue.
Thanks,
Dan


Add LeetCode-1 (different approach) and LeetCode-167 in C++

resolves #42

HI,
Thank you for your feedback. I have addressed the requested changes, please review.

Hi @Tahanima
I am kind of lost now, could you please help me out  and point me to the right direction? As your recent requested changes are suggesting there are indentation issues in my code but in my forked repo the code looks okay. What am I missing here?
What I did so far:

Opened a pull request.
Some changes were requested by you.
Addressed the requested changes in my local git repo, updated my repo with the original upstream repo by fetching and merging, then pushed them to my forked repo.
And now as you addressed some indentation issues, as I visited my fork repo they seems fine there!!

So did I miss any steps after pushing the first requested changes to my forked repo?

Thank You :)


Add 1.0.5 to mimaPreviousArtifacts, & backfill

Remove the file exclude files before (a) compiler-bridge's exclude
file is redundant since we no longer mima-check it, and (b) it interacts
badly with multiple versions.

To validate this pull request against other sbt modules, please visit this link.


Is there any way to make internal global variable readonly?

eg: Math, Array, encodeURI, ...
The reason is that I want to reuse runtime for improving performance.
Now, user can change global variable like this:
Math = 1;
Math.a = 1;

This is dangerous when reusing runtime.
My suggestion is that add a flag in Runtime struct, like this:
type Runtime struct {
    global          global
    globalObject    *Object
    globalReadonly  bool // maybe put this flag in global struct better
    // ...
}

and add options for New()  to set global readonly:
type Option func (*Runtime)
func New(options ...Option) *Runtime {}

and change the internal libs dependent on globalReadOnly flag.

There is already a mechanism for it: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/freeze. To be 100% sure you could apply it recursively to all keys of globalThis, however I think (although not 100% sure) you can get away with this:
for (const k of Reflect.ownKeys(globalThis)) {
    const v = globalThis[k];
    if (v) {
        Object.freeze(v);
        v.prototype && Object.freeze(v.prototype);
        v.__proto__ && Object.freeze(v.__proto__);
    }
}

Note that because globalThis contains itself (i.e. globalThis.globalThis === globalThis), this will also freeze it.
After this defining global variables or functions won't be possible (which is probably good for reusable Runtimes anyway), however it still allows global lexical declarations (i.e. let and const). To prevent this you should wrap the code in { ... } so that it is run in its own block context.
Again, I'm not 100% sure this can't be bypassed. It is also not future-proof. If at some point the standard built-ins include nested object properties, this won't prevent them from being changed.

@dop251 thanks

What would be the equivalent of doing this from Go?


createSecurePair() is deprecated, use TLSSocket instead

This doesn't actually appear to be deprecated. http://nodejs.org/api/tls.html#tls_tls_createsecurepair_context_isserver_requestcert_rejectunauthorized
There is a note which says:

NOTE: cleartext has the same APIs as tls.TLSSocket

But I don't see anything about deprecation.
Upon further digging, it looks like it was once deprecated, but the deprecation notice was removed in 0.11.15.

This looks like it has since been deprecated according to the
https://nodejs.org/api/tls.html#tls_tls_createsecurepair_context_isserver_requestcert_rejectunauthorized_options
Referenced in this commit in node 6.0 ( so not LTS yet ) - the favoured tls.tlsSocket is present in 4.x
https://github.com/nodejs/node/pull/6063
And referenced in these meeting minutes from May 2016
https://github.com/nodejs/node/blob/7f1111b1bb3f1af777e062cd7ca62dd0dc0fc8e0/doc/ctc-meetings/2016-05-04.md


Can't run .NET Core console app

Steps:

Create an F# project from the command line via dotnet new console --language F#

Open it in VS Code, go to the F# tab, next to the project name click the "Run" symbol

Witness the following error message:
The terminal process command '"C:\Program Files\dotnet\dotnet.exe" run -p
c:\Users\carlp\OneDrive\Desktop\RedditModTools\RedditModTools.fsproj' failed to launch (exit code: 2)

... but cutting and pasting into cmd.exe and running this command works just fine

Ionide-fsharp 3.30.0
.NET Core SDK 2.1.500
F# 4.5
Tested on Visual Studio Code 1.28.2 and 1.29.1
Windows Insiders, version 10.0.17763

Works for me when I remove the extra quotes from here:
https://github.com/ionide/ionide-vscode-fsharp/blob/ff7b5ae6684b022d02b0f0056927dc71986936fc/src/Core/Project.fs#L526

Any updates on this?

@devblah PRs welcome :)))

send PR https://github.com/ionide/ionide-vscode-fsharp/pull/1071

Closing, fixed by https://github.com/ionide/ionide-vscode-fsharp/pull/1071

Getting the exact same issue following the exact same reproducing steps.
Ionide-fsharp 3.35.0 (The latest currently. I update my plugins religiously).

The fix has not been released yet


NLTK fails to find the Java file

when i use stanford word segmenter ,  here is code :
import nltk
from nltk.tokenize.stanford_segmenter import StanfordSegmenter
segmenter = StanfordSegmenter(
path_to_jar = "./stanford_segmenter/stanford-segmenter-3.6.0.jar",
path_to_sihan_corpora_dict = "./stanford_segmenter/data",
path_to_model = "./stanford_segmenter/data/pku.gz",
path_to_dict = "./stanford_segmenter/data/dict-chris6.ser.gz")
sentence = u"这里是斯坦福分词器测试"
result = segmenter.segment(sentence)
print result
the error as follows :
NLTK was unable to find the java file!
Use software specific configuration paramaters or set the JAVAHOME environment variable.
i have set the javehome enviroment , but it also doesn't work

You have to set JAVA_HOME, not JAVAHOME
JAVA_HOME=/opt/java/jre1.8.0_45/bin/java

If your code cannot find the Stanford files, then you are using the wrong relative path. Try using the absolute paths.

Thanks @yooper for the instructions to setup JAVA.
@AlexYoung757 , did @yooper's answer resolve your problem?

This should be resolved in our new interface to the Stanford Tools. If not, please reopen.


Fixes #948 - makes specs flag accept array

Fixes #948 - makes specs flag accept array.
Changes

Added ability for acceptance --specs test flag to accept list of test files.

Testing

Add a comma-separated list to the --specs flag value. For example, to run two tests: gulp build && gulp test:acceptance --sauce=false --specs=about-us.js,contact-us.js

Review

@sebworks
@jimmynotjim
@KimberlyMunoz

Runs perfectly. :+1:


"Recommended Content" lists same item twice

Hase two identical entries to the Az.Sql powershell module.

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

ID: 052ba85d-cc93-5de7-4aa6-1d1df4b9eda7
Version Independent ID: 2a6edae1-13c7-ef95-bf43-6de8a0cb5197
Content: Azure PowerShell script examples - Azure SQL Database
Content Source: articles/azure-sql/database/powershell-script-content-guide.md
Service: sql-db-mi
Sub-service: development
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@OranguTech Could you please specify what the request is ? Thanks

Remove the duplicate entry...?

Hi @orangutech, thanks for your feedback! My apologies, it's unclear to us what you're referring to that is duplicated. Can you please provide a screenshot of where you see duplication as it's not immediately evident to us?
Thanks!
Masha from the SQL Docs team

Oh, are you referring to this?

This is automatically generated, unfortunately we have no control over the behavior here. I will pass this feedback along to our team. However, since this is not a doc bug, I'll proceed with closing out this Git Issue.
Please feel free to comment should you need anything further, and thanks so much for reaching out!
Hope you have a wonderful rest of your day :)
Masha from the SQL Docs team
#reassign:MashaMSFT
#please-close


[SPARK-17880][DOC] The url linking to AccumulatorV2 in the document is incorrect.

What changes were proposed in this pull request?
In programming-guide.md, the url which links to AccumulatorV2 says api/scala/index.html#org.apache.spark.AccumulatorV2 but api/scala/index.html#org.apache.spark.util.AccumulatorV2 is correct.
How was this patch tested?
manual test.

Test build #66769 has started for PR 15439 at commit 6231206.

Test build #66769 has finished for PR 15439 at commit 6231206.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66769/
Test PASSed.

Thanks - merging in master/2.0.


audio desync in davinci resolve

hi, I have an audio desync in davinci resolve 16. I checked video with VLC video player - audio is ok, but when I importing it to davinci resolve to edit and cut - there is an audio desync, what should I do to fix it?

I don't know, actually. Maybe try the Resolve forum for further info. I cannot provide reasonable fixes for every software that does not work properly. I only know that many programs have trouble with downloaded streams from a m3u8 playlist (e.g. Twitch streams).

I don't know, actually. Maybe try the Resolve forum for further info. I cannot provide reasonable fixes for every software that does not work properly. I only know that many programs have trouble with downloaded streams from a m3u8 playlist (e.g. Twitch streams).

What could you recommend to solve this?

Did you read my comment?

I don't know, actually. Maybe try the Resolve forum for further info.

This issue is not a Twitch Leecher problem, but a Davinci Resolve problem. I don't know how to fix issues with Davinci Resolve, unfortunately.


Simplify NameConverter and make it more generic

We should remove
libraryDependencies += "org.scalaj" %% "scalaj-http" % "2.4.2" dependency as we removed webnamedatabase and we donot rely on this library anymore

Done

Stick with WebNameDatabase, it seems quite useful


Allow ignoring/skipping img tags in RichText

I have a list of items, and for each of which I'm using RichText to display a summary of the item's HTML description. the HTML of course can contain some img tags but I'm only interested in displaying a text description.
Currently, with the release of 1.0-rc06 that added support to image loading I'm not able to use RichText composable anymore as it breaks my item layout, and I can't find a way to tell it to ignore any img tags it encounters and just renders text the same as before, So, I'm suggesting to add a new parameter or a flag to RichTextState or the RichText composable that would allow the caller to explicitly ignore/skip images if he wants to.

Totally makes sense, I'll work on it.


Allow svg import without width/height attributes

This is possible in an .svg file, as indicated by this stackexchange post [1]. I also have created them and they render just fine. Further, your code seems to already support having a None value for self._height and self._width so it seems that not much has to change.
Without this check, the code raises an exception.
...
  File "/home/jonnyjack7/.local/lib/python3.9/site-packages/svgutils/compose.py", line 122, in __init__
    if svg.width.endswith("%"):
AttributeError: 'NoneType' object has no attribute 'endswith'

[1] https://graphicdesign.stackexchange.com/a/71574

@hartwork I run into this exact error today. I try to generalize https://github.com/RoepStoep/lidraughts-boardimage to support chess variants playable on pychess.org and it has lots of piece images with no width/height svg attribs.
It would be nice to have a new version with this PR merged. Maybe @btel is busy with real life stuff in past 3 years.

@gbtami I agree that svg_utils is not getting the love it needs. I see no current use of svg_utils in https://github.com/RoepStoep/lidraughts-boardimage — am I missing something?
PS: While I seem to be able to close pull requests here, I have no write access to https://pypi.org/project/svgutils/ , if you're wondering.

@hartwork I find the idea to use svg_utils to load and scale piece images in your https://github.com/hartwork/xiangqi-setup :)
Btw I faced another minor issue:
# This can read "viewBox" but can't do scale()
svg = svgutils.transform.fromfile(orig_file)

# This can't read "viewBox" but can do scale()
svg = svgutils.compose.SVG(orig_file)

Maybe I should create a github fork of this repo and use it until this upstream repo will be active again. Idk.


Make jest-matcher-utils use ESM exports

Summary
Might help with intermittent  'default' is not exported by packages/jest-matcher-utils/build/index.js

Test plan
jest

I can not reproduce the windows CI failure locally...

The AppVeyor build is still failing for some reason, but now the travis build is passing again!!


Auto (Output) Preselection

An 'auto'/quite-mode option (in other words, specify the output csv/json, then press go and it'll output those files and close the app)

HH Team

Although this was the case with a previous GUI and people were unsure if it had crashed or completed. Maybe keep some UI showing completion.

Closing as no longer necessary due to change of internal workflow.


The monitor signin - audit log query does not work. Very poorly written

[Enter feedback here]

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

ID: fff5de5d-9a97-73ff-4694-79027720b5c3
Version Independent ID: 566a8387-bbdd-8e35-13cc-f2d891bd5608
Content: Manage emergency access admin accounts - Azure AD - Microsoft Entra
Content Source: articles/active-directory/roles/security-emergency-access.md
Service: active-directory
Sub-service: roles
GitHub Login: @markwahl-msft
Microsoft Alias: rolyon

@ltjones129
Thanks for your feedback! We will investigate and update as appropriate.

@ltjones129
Thank you for your time and patience!

Can you share what issue you're specifically running in to?
Are you receiving any error messages that you can share?

Any additional information or screenshots would be greatly appreciated.
Thank you!

@ltjones129
I just wanted to check in and see if you had any other questions or if you were able to review my previous comment?


Support annotation parsing for Python

Need to check if this is not already supported by now

Can be closed for now with #1057 and #1751. In general annotations have problem, but this is not python specific.


DateWidget render() gives empty string when handling DateTime

I'm trying to export to xlsx, change some dates in Excel and then import again using django-import-export. I'm running into some issues and it's probably an underlying issue as I end up getting DateTime instances rather than Date instances from tablib when re-importing.
This might not be considered a concrete bug as DateWidget maybe should not handle DateTimes but I feel like the code is slightly off either way:
For DateWidget clean() we call _ParseDateTimeMixin _parse_value() which does the following check:
    def _parse_value(self, value, value_type):
        """Attempt to parse the value using the provided formats.
        Raise ValueError if parsing fails."""
        if not value:
            return None
        if isinstance(value, value_type):
            return value
        # ...etc

the isinstance(<datetime>, date) check passes since datetime is a subclass of date. So the import works.
However for the render() call in admin it returns empty string since the render code uses type() instead of isinstance():
    def render(self, value, obj=None):
        self._obj_deprecation_warning(obj)
        if self.coerce_to_string is False:
            return value
        if not value or not type(value) is date:
            return ""
        return format_datetime(value, self.formats[0])

Changing the above check to if not value or not isinstance(value, date): fixes the rendering and all tests passes when I tried changing it in the lib. Either way it feels strange for me that the import works but not the rendering.
If you think this is a reasonable change I'd be happy to submit a PR.
Versions:
django-import-export==4.1.0
tablib==3.5.0
openpyxl==3.1.5
Django==4.2.13

Python 3.11.9

Thanks for reporting.  Feel free to submit a PR.  Contribution guidelines are here.


Responsive style corrections

Addresses Issue #90 - fix display issues on smaller screens

Coverage remained the same at 96.36% when pulling 314354e3c6b38f4e4294d92ea958fe3cd1db7458 on responsive-style-corrections into 1528a90985fb421ce934f285994f0fdba9fcce9a on master.


remove, add and fix a few rules

some rules need to be updated, here are a few so far:

cloudtrail-enabled rule name was wrong
codebuild-project-environment-privileged-check has been removed from all security standards since Apr. 10th, 2024.
lambda-concurrency-check rule should be added back

Checking this out now. I think we need to update the rule pack info generation automation to include some of the conventions from the config rules automation. I'll create a new PR with those changes and link it here when I'm done. Thanks for looking into this, Emily!

This should be resolved in #44
@yangx17 Is it ok to close this PR?

yeah, I will close this one.


Update README

Hello!
Few small (cases,spellings,etc) changes done to README which makes it look better.
Please add hacktoberfest if you like it and I'll request a pull .
Regards.

Hey, i can help you if you want.
If you want, asign me to the repo, and i'll make fork and merge request :)


Way to ignore smell highlights after reviewing?

This plugin sometimes marks code as smelly, when it isn't. Is there a way to mark a smell as not smelly?

Any update on this?


[1.10.2] Rendering crash with bin and solar generator

So this appears to be a very specific crash. I had 212 Advanced Solar Generators in an Advanced Bin, I then used the Configurator in wrench mode in order to pick up the bin, and move it from the houses basement, to outdoors where I could more easily place the generators. As soon as the bin dropped in item form however, me, and the other player on the server, crashed. The only way to get back on was killing the item drop via command in console.
http://pastebin.com/SQvpzDU6

Had this same issue when I tried to configurate/wrench a basic bin with 16 basic energy cubes in it. Caused anyone whose client attempted to render it to crash.  Same error as in your pastebin.

No longer should be an issue.


ability to pass in array of video names and have them played one after another

This is a great idea! We should definitely include this as functionality in 3.0.0.
PS: if you need this functionality rn, one hack-y way I've used before is using ffmpeg to directly concatenate multiple mp4's into a single file. Here is my commit for on how I did this.


Update configuration of chart by passing a new "ChartConfiguration" object does not update the title

I am currently using Chartjs v2.7.2
I am trying to update the title of a doughnut chart dynamically.
The following code works fine.

chart.config.options.title.text = "Replace the old title with the New title!!!";
chart.update();

I have created a configuration object which implements ChartConfiguration. My new code is as follows:

var customConfigObject = new CustomConfigObject();
chart.config = customConfigObject;
chart.update();

The CustomConfigObject implements the official chartjs interface ChartConfiguration. I have defined the new title inside that class under the options.title.text property.
All the other properties including the data gets updated except the options.title.text.

@manittrivedi can you please provide a jsfiddle that reproduces your issue?

Please see this jsfiddle
https://jsfiddle.net/efwct0an/

@manittrivedi updating the whole config is not supported, options should be updated using Chart.options and data this way. If you want to rebuild the whole chart with a new config (type, data, options, plugins), it's better to destroy() and recreate it.


[CreateTearsheet]: including steps dynamically with includeStep not working after v2.36

Package
Carbon for IBM Products
Description
We are using CreateTearsheet component with a couple of steps being included/excluded based on an option the user selects on a previous step.
It was working fine until v2.35, and then it broke after we upgraded to v2.45. After going through the changelogs, I noticed a change on v2.36  that includes a state change inside an useEffect that resets the tearsheet step to the first step, which I think is causing the problem.
I added a stackblitz reproducing the issue. If you downgrade ibm-products to v2.35 you can see the form works as expected.
Component(s) impacted
CreateTearsheet
Browser
Chrome, Firefox
@carbon/ibm-products (previously @carbon/ibm-cloud-cognitive) version
v2.45
Suggested Severity
Severity 2 = Aspects of design is broken, and impedes users in a significant way, but there is a way to complete their tasks. Affects major functionality, has a workaround.
Product/offering
IBM RPA
CodeSandbox or Stackblitz example
https://stackblitz.com/edit/github-kndrwa?file=src%2FApp.jsx
Steps to reproduce the issue (if applicable)
1 - Create a form using CreateTearsheet and set a couple of steps to be included dynamically
2 - Include an intro step and a step with an option selection that is saved in a state
3 - The following steps should be included based on the option selected (using includeStep prop)
4 - Open the tearsheet and try to select the option
Expected result: user should be able to go to the next step
Actual result: tearsheet goes back to the first step
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

@Tomahaawk i think the problem here is that the flow is expected to be the introStep which is where typically you would establish any of the dynamic steps. i believe this is the intended behavior, but i don't see anything that explicitly forbids this in the create flow documentation. i will get design input on this and confirm whether or not we want to specifically allow steps to be dynamically set outside of the designated introStep
i did test it locally and confirmed that adding introStep to the second step does alleviate this problem. if you need a temporary solution i believe that will work.

i think the problem here is that the flow is expected to be the introStep which is where typically you would establish any of the dynamic steps.

Just to give more context, in our use case we actually have the intro step where the user selects a flow, then in the middle of the flow there is another step where the user selects an option to decide what the following step will be. We have 2 steps that can change the flow of the form.

@davidmenendez We are using the CreateTearsheet component in IBM Guardium Insights to dynamically add steps based on options a user selects. The issue is due to the addition of the else statement in v2.36 the CreateTearsheet component resets after a step is added through the includeStep prop and the tearsheet goes to the first step. This is blocking our efforts of updating Guardium Insights to the latest Carbon.

@1pharaxh @Tomahaawk i have a fix for this and will submit shortly.

Thanks for the fix @davidmenendez is this going to be in the next release ?

i'm hoping it will. still waiting for it to be reviewed and merged. will you you posted.


"feature1": "abc.feature:10 - failed to construct class by name: karate-http.properties not found, aborting

I am trying to execute some feature file using karate runner from REST controller class call(spring-boot app and bundled as jar running on port 8080). In service class I am using karate runner to run the tests.
public Results executeTest(String featureFile, List<String> featureTags) {    return Runner.path(featureFile).tags(featureTags).parallel(10); }
passing featureFile=/abc.feature to this method
kept the feature file in root folder of OS as /abc.feature , I am getting error: "feature1": "abc.feature:10 - failed to construct class by name: karate-http.properties not found, aborting
I am using Java 11 with dependencies:
<dependency> <groupId>com.intuit.karate</groupId> <artifactId>karate-apache</artifactId> <version>0.9.6</version> </dependency> <dependency> <groupId>com.intuit.karate</groupId> <artifactId>karate-junit5</artifactId> <version>1.3.0.RC2</version> </dependency>

follow this process to reopen please: https://github.com/karatelabs/karate/wiki/How-to-Submit-an-Issue


CSS Static page linking error in either install documentation or in the /opt/nautobot/contrib file.

Environment

Python version:3.8.5
Nautobot version: 1.0.0b1

Steps to Reproduce

Follow the installation docs for http server: https://nautobot.readthedocs.io/en/latest/installation/5-http-server/
When you get to the step of Confirm Connectivity, it will fail.  A static error for CSS errors will come up when you try to browse to the server page.

Expected Behavior

Expect to see the home page of nautobot.

Observed Behavior

You get a Static error CSS not found page.

Root Cause

The root cause for this error is because the static page linked in the /opt/nautobot/contrib/apache.conf (or nginx.conf) links to a non-existent directory.

When you git clone the repo, you get a /opt/nautobot/nautobot/project-static directory, and the .conf files references to /opt/nautobot/static instead.  I'd open a PR for this, but i'm not sure whether this is a documentation error, or a contrib config file error, so not sure which file to put the PR for.  Either way, should be a super easy fix.

Thank you for opening this issue! We definitely need to clarify the docs here.
Just as a general note to hopefully make sure you're not blocked in the meantime, you generally shouldn't be pointing to /opt/nautobot/nautobot/project-static, but should instead be pointing to wherever nautobot-server collectstatic is copying the static files to. This is the STATIC_ROOT value in settings.py/nautobot_config.py, and if you haven't overridden it explicitly or set the $NAUTOBOT_BASE_STORAGE_DIR environment variable, it defaults to ~/.nautobot/static.
We'll get the docs updated, thanks!

Thanks for the report @syyna! Should have some totally new install docs in the coming days.

Will be fixed in #62


Not able to login into azuredevops demo

Hi @bluemountains1979,
Is there really an issue with login? could you please explain your repro steps?

Duplicate #261 Hence closing the issue.


Shift states incomplete when potentially empty following rule present

A grammar like:
token a;
token comma;
Start -> As comma?;
As -> a As2?;
As2 -> comma a As2?;

Will not properly terminate due to Propane not including the shift states for the potentially empty 'comma?' rule.

It turns out that this grammar is not actually unambiguous LALR(1).


Create stubbed implementation of API

Create API that

Loads settings from config file (default to settings already in arclib.py if not provided)
Provides methods for actions listed in #2

Update the ceda-arc-app site to use the new interface.
Just create stubbed method implementations for the time being (until certificates etc are in place for me to interact with ARC for real)

Done in 6733810fdd05df8db27da2fe53d7e3b38775d4c0 and 7c91fb9e8629d4f49c88ba9b0cc1bd17984ecd87


Can't use custom templates for receivers

I tried adding custom templates, using alertmanager_template_files, which works. What doesn't work is when I configure the receiver:
- name: "receiver"
  pagerduty_configs:
    - routing_key: "xxx"
      send_resolved: true
      description: '{{ template "pagerduty.custom.description" . }}'

The job failes on preflight.yml, Fail when there are no receivers defined and Ansible says
Error was a <class 'ansible.errors.AnsibleError'>, original message: template error while templating string: expected token 'end of print statement', got 'string'. String: {{ template \"pagerduty.custom.description\" . }}

What am I expected to write to use custom templates?

You need to wrap every '{{ ... }}' into `{% raw %}{{ ... }}{% endraw %}' blocks to prevent ansible from treating those as jinja and passing further.
Example of that approach: https://github.com/cloudalchemy/ansible-prometheus/blob/master/defaults/main.yml#L174-L181

Thank you!


Largest Common SubSequence in Java

Programming language

[ ] C
[ ] C++
[x] Java
[ ] Python
I want to work on this issue

Can I do in Java? I'm a participant of LGM Soc21.


[#98091344] Create Influxdb ansible role

Add metrics capture to core components
What
In preparation for load testing of our environment, we would like to capture metrics to evaluate how our systems are impacted.
The purpose of this PR is to create an InfluxDB time series database to store our metrics data.
How this PR should be reviewed
This PR has been created to be reviewed with the following narrative with non-breaking changes:

I want to create a Vagrant environment in order to test my code
I want to create a best practice ansible skeleton so I can write well structured ansible code
I want to ensure that the ansible-galaxy command can correctly parse my role's metadata
I want to setup some default variables for use in this role
I want to create some basic tasks in order to configure the influxdb service with a database for storing metrics
I want to create an administrative database account so that http based authentication can be enabled
I want to enable security so only admins can create and delete databases and users only have access to a single database
I want to tell people how to use this ansible role
I want to let people know that I have used a license that promotes code reuse

How to test this PR
The repository contains a Vagrant machine that can be used for testing by doing the following:
git clone git@github.com:alphagov/ansible-playbook-influxdb.git
cd ansible-playbook-influxdb
git checkout 98091344_create_influxdb_role
echo "influxdb_http_auth_enabled: 'true'" >> defaults/main.yml
vagrant up

Point your browser at http://localhost:8083/

Login as root with password root
confirm that a database called testdb exists and that someuser has been granted access to it

Known issues
All users appear to have admin permissions even when they are granted with limited access rights
Who can review this PR

Anyone on the core team can review this PR with the exception of @jimconner who I paired with :p

Tested successfully, merging.
Confirmed the security issue. Should be monitored closely.


Logo update

OSL logo must be included instead of OSC logo

Logo size should be 191 X 80 px. @madhurish

logo has been updated


Dask-expr - Extremely slow with using .compute

I installed the basic dask version using "pip install dask". When running, I receive a FutureWarning:

Dask dataframe query planning is disabled because dask-expr is not installed. You can install it with 'pip install dask[dataframe]' or 'conda install dask'. This will raise in a future version.

I proceeded to install using the provided pip command. However, upon doing so, my .compute() went from less than 1 second to run, to 22 seconds (sometimes longer in later parts of the code). These are small slices/excerpts from the entire dataframe so I can create a summary of certain events and do some calculations. The slices are only about 15 rows so I am just trying to extract them to pandas to get rid of any partitioning and do some calculations.
Code Excerpt (part of a for loop) --> I split the excerpt of the dataframe assignment and the compute steps up to figure out where the problem lies. For background, ddf is a dask dataframe that, in its final state, will consist of about 250,000 partitions, one for each CSV file read in.
    event_a = ddf.loc[start_index:end_index]
    print('Partial dataframe into event_i.', '---- Process Time: %s ----' % (datetime.now() - time_start_j))
    event_i = event_a.compute()
    print('Computed event_i to pandas dataframe.', '---- Process Time: %s ----' % (datetime.now() - time_start_j))
    event_i['Event ID'] = eventid
    print('Finished collecting Event DataFrame.', '---- Process Time: %s ----' % (datetime.now() - time_start_j))

Output (without dask-expr):

Starting ID 1
Partial dataframe into event_i. ---- Process Time: 0:00:00.017307 ----
Computed event_i to pandas dataframe. ---- Process Time: 0:00:00.133990 ----
Finished collecting Event DataFrame. ---- Process Time: 0:00:00.134989 ----

Output (with dask-expr):

Starting ID 1
Partial dataframe into event_i. ---- Process Time: 0:00:00.041680 ----
Computed event_i to pandas dataframe. ---- Process Time: 0:00:21.894825 ----
Finished collecting Event DataFrame. ---- Process Time: 0:00:21.894825 ----

Version: Dask 2024.7.1
I have attached a zip file containing a generic python script and two example CSV files. For this example, all I did was copy the two files repeatedly to get the full 80 files using shutil the first time. I didn't send all 80 files in order to keep the file size down.
The problem does seem to scale with the size of the dask dataframe though, even though I am only querying it once. I tried it with only two CSV files and it had very little time impact. As I increased the number of files, the problem balloons. This would be a huge impact for my full dataset. I am not sure if that is because it has to compute the entire Dask dataframe even if you are only asking for part of it?
TS_DaskGeneric.zip

Could you create an example that doesn't rely on external files? I.e. something that we can copy paste. See https://matthewrocklin.com/minimal-bug-reports for a more detailed description

K, so I deleted my previous post based upon your most recent request and am providing the information requested below.
I am not sure if that has something to do with this latest version all being internalized code (i.e. not referencing CSV files) or the fact that my previously supplied example had lots of different data types and intermixed datatypes within the columns (i.e. there are explicit "NaN" strings present across rows when data is missing, and some cells are blank.
Version: Dask 2024.7.1
The problem does seem to scale with the size of the dask dataframe though, even though I am only querying it once.  The results I received with and without dask-expr for this particular code is shown below. What is interesting about the supplied code below versus the code I previously supplied (which referenced example CSV files):

There is still a significant process time increase with the presence of dask-expr
However, the process time increase switched from the .compute() step to the earlier extracting/slicing of the data with .loc in this latest version.

Output (without dask-expr):

Starting Program.
Finished Reading ---- Process Time: 0:00:08.307470 ----
Start Add continuous index.
Finished adding continuous index. ---- Process Time: 0:00:03.352517 ----
Starting event slicing.
Partial dataframe into event_i. ---- Process Time: 0:00:00.017952 ----
Computed event_i to pandas dataframe. ---- Process Time: 0:00:00.029939 ----
Finished collecting Event DataFrame. ---- Process Time: 0:00:00 ----

Output (with dask-expr):

Starting Program.
Finished Reading ---- Process Time: 0:00:05.408973 ----
Start Add continuous index.
Finished adding continuous index. ---- Process Time: 0:00:00.037657 ----
Starting event slicing.
Partial dataframe into event_i. ---- Process Time: 0:00:10.001254 ----
Computed event_i to pandas dataframe. ---- Process Time: 0:00:01.642147 ----
Finished collecting Event DataFrame. ---- Process Time: 0:00:00 ----

#!/usr/bin/env python3
from datetime import datetime
import pandas as pd
import dask.dataframe as dd
import numpy as np
# ---------------------------------------------------------
# Input
# ---------------------------------------------------------
lst_colnames = []
for i in range(0, 47): lst_colnames.append('Rand ' + str(i))
lst_dfs = [] # List of pandas dataframes to be populated
cnt_dfs = 160 # Number of dataframes to be generated
# ---------------------------------------------------------
# Read Files into Dask DataFrame
# ---------------------------------------------------------
time_start = datetime.now()
print('Starting Program.')
for i in range(0, cnt_dfs): #Populate the dataframe list
    cnt_rows = np.random.randint(8000, 50000)
    df_i = pd.DataFrame(np.random.random(size=(cnt_rows, 47)), columns=lst_colnames)
    lst_dfs.append(dd.from_pandas(df_i, npartitions=1))
ddf = dd.concat(lst_dfs)
print('Finished Reading', '---- Process Time: %s ----' % (datetime.now() - time_start))
# ---------------------------------------------------------
# Add Continuous Index across all partitions (important for later)
# ---------------------------------------------------------
time_start = datetime.now()
print('Start Add continuous index.')
ddf['x'] = 1
ddf['x'] = ddf.x.cumsum()
ddf = ddf.set_index('x', drop=True)
print('Finished adding continuous index.', '---- Process Time: %s ----' % (datetime.now() - time_start))
# ---------------------------------------------------------
# Process a random chunk
# ---------------------------------------------------------
start_index = 8651
end_index = 8665
eventid = 7
print('Starting event slicing.')
time_start = datetime.now()
event_a = ddf.loc[start_index:end_index]
print('Partial dataframe into event_i.', '---- Process Time: %s ----' % (datetime.now() - time_start))
time_start = datetime.now()
event_i = event_a.compute()
print('Computed event_i to pandas dataframe.', '---- Process Time: %s ----' % (datetime.now() - time_start))
time_start = datetime.now()
event_i['Event ID'] = eventid
print('Finished collecting Event DataFrame.', '---- Process Time: %s ----' % (datetime.now() - time_start))


trailing_table_separator when strings is last row

I think I found wrong behavior.
When trailing_table_separator is always,  I think it should insert comma here
local t = {
  a = 'a' -- <- here
}

So it should become
-- expected behavior
local t = {
  a = 'a',
}

But it doesn't happen
-- actual behavior
local t = {
  a = 'a'
}

NOTE: it always works except when the last element is string and is surrounded with ' or "
local t = {
  a = {},
}

local t1 = {
  a = 0,
}

local t2 = {
  a = true,
}

-- it even works here
local t3 = {
  a = [[here]],
}

I have the same problem in https://github.com/LuaLS/lua-language-server/issues/2045#issue-1649652364
I found out that the problem is caused by the quote_style:
[*.lua]
trailing_table_separator = smart
quote_style = double

local foo = {
    "bar"
}

It works without this option:
[*.lua]
trailing_table_separator = smart

local foo = {
    "bar",
}


Added support for OCCURS String => Int mappings

This PR implements the feature request on #151.  It adds a new option occurs_mappings to the spark reader.

@yruslan -- if you think this looks good, I'll add something to the docs before merging.

Awesome! Looking at it now... will let you know soon.


#882 - AE - update or switch of Background is more intelligent

Background update/switch supports layers:

update
add
remove

Works on comparison of file names!
Requires redeploy of extension.
Closes Freshdesk:507
Closes pypeclub/pype#882

can we please have 3.0 corresponding PR @kalisp

can we please have 3.0 corresponding PR @kalisp


Fixes broken compatibility with recent MongoDB versions

MongoDB no longer support sorts on tailable cursors. Removes sorts from code

Fixed in https://github.com/scttnlsn/mubsub/pull/39


Fix ESLint Errors

[x] You have read the Odin Bot README/Contributing Guide.

1.Describe the changes made:
As the title suggests, all of the ESLint Errors have been fixed. Most of these were trivial semicolon/trailing comma errors. Other, less trivial errors have been amended to remain as true to the original code as possible.
I have been made aware that some commands will need changes; and this commit has not made any substantial changes, outside of ESLint compliance.
2. If this PR modifies a command, please provide a screenshot of the passing tests below. NOTE: Your PR will not be merged if you have any failing test cases.

3. If this PR is related to an open issue please reference it with the # operator and the issue number below:
#116

Hey @JonColman94, the .gitignore file and .env.sample are still showing up in the files changed in this PR.
If you've changed files in a branch unintentionally and want to get them back to the state they were in the master/main branch its easier to checkout the master version of those files:
git checkout master .gitignore
git checkout master .env.sample
Then commit those and push them up.
If you have any questions don't hesitate to let me know.

Okay, that should be it. Neither file is in the changed files anymore from what I can see.

The latest fix as of this should pass all checks.


Using bulk actions causes container/service to look un-checked even thou it's not

Rancher versions: master 8/16
Steps to Reproduce:

Check a container
Use the bulk actions to stop the container

Results:  The container un-checks, but it's actually still checked.  The bulk actions are still visible and you can use them on the un-checked container.  I have to click on the checkbox for the container again to actually get it to uncheck.  This is happening in all places that have "bulk actions"

Version - master 9/11
Verified fixed


Version 1.15.2

Hi guys, I'm having problems with glitz graphics in the game Tenchu Z.

Invalid issue. Please create a proper issue following the template.


CPU: RET instruction is not entirely correct

https://www.felixcloutier.com/x86/ret
The 0xC2 / 0xCA param is 16 bits, not 8.
It's uncommon to have more than 255 POP after a RET, but you never know...

Fixed: https://github.com/OpenRakis/Spice86/commit/009120dc0b6f4a95819cb24d88d7bde0d8b90d07


UnknownServiceError on HyperparameterTuningJobAnalytics

In the HPO_Analyze_TuningJob_Results.ipynb, the constructor of tuner returns UnknownServiceError. Based on the stack strace, it's related to the Session() constructor. Please investigate. Thank you!
tuner = sagemaker.analytics.HyperparameterTuningJobAnalytics(tuning_job_name)
UnknownServiceError                       Traceback (most recent call last)
<ipython-input-26-f5dcb168cae5> in <module>
----> 1 print (sagemaker.Session())

~/miniconda3/lib/python3.6/site-packages/sagemaker/session.py in __init__(self, boto_session, sagemaker_client, sagemaker_runtime_client)
     88             self.config = None
     89
---> 90         self._initialize(boto_session, sagemaker_client, sagemaker_runtime_client)
     91
     92     def _initialize(self, boto_session, sagemaker_client, sagemaker_runtime_client):

~/miniconda3/lib/python3.6/site-packages/sagemaker/session.py in _initialize(self, boto_session, sagemaker_client, sagemaker_runtime_client)
    109         else:
    110             config = botocore.config.Config(read_timeout=80)
--> 111             self.sagemaker_runtime_client = self.boto_session.client('runtime.sagemaker', config=config)
    112
    113         prepend_user_agent(self.sagemaker_runtime_client)

~/miniconda3/lib/python3.6/site-packages/boto3/session.py in client(self, service_name, region_name, api_version, use_ssl, verify, endpoint_url, aws_access_key_id, aws_secret_access_key, aws_session_token, config)
    261             aws_access_key_id=aws_access_key_id,
    262             aws_secret_access_key=aws_secret_access_key,
--> 263             aws_session_token=aws_session_token, config=config)
    264
    265     def resource(self, service_name, region_name=None, api_version=None,

~/miniconda3/lib/python3.6/site-packages/botocore/session.py in create_client(self, service_name, region_name, api_version, use_ssl, verify, endpoint_url, aws_access_key_id, aws_secret_access_key, aws_session_token, config)
    834         client = client_creator.create_client(
    835             service_name=service_name, region_name=region_name,
--> 836             is_secure=use_ssl, endpoint_url=endpoint_url, verify=verify,
    837             credentials=credentials, scoped_config=self.get_scoped_config(),
    838             client_config=config, api_version=api_version)

~/miniconda3/lib/python3.6/site-packages/botocore/client.py in create_client(self, service_name, region_name, is_secure, endpoint_url, verify, credentials, scoped_config, api_version, client_config)
     62         self._retry_config_translator = retry_config_translator
     63         self._response_parser_factory = response_parser_factory
---> 64         self._exceptions_factory = exceptions_factory
     65         # TODO: Migrate things away from scoped_config in favor of the
     66         # config_store.  The config store can pull things from both the scoped

~/miniconda3/lib/python3.6/site-packages/botocore/client.py in _load_service_model(self, service_name, api_version)
     94         )
     95         return service_client
---> 96
     97     def create_client_class(self, service_name, api_version=None):
     98         service_model = self._load_service_model(service_name, api_version)

~/miniconda3/lib/python3.6/site-packages/botocore/loaders.py in _wrapper(self, *args, **kwargs)
    130         if key in self._cache:
    131             return self._cache[key]
--> 132         data = func(self, *args, **kwargs)
    133         self._cache[key] = data
    134         return data

~/miniconda3/lib/python3.6/site-packages/botocore/loaders.py in load_service_model(self, service_name, type_name, api_version)
    376             raise UnknownServiceError(
    377                 service_name=service_name,
--> 378                 known_service_names=', '.join(sorted(known_services)))
    379         if api_version is None:
    380             api_version = self.determine_latest_version(

UnknownServiceError: Unknown service: 'runtime.sagemaker'.
If you are using a SageMaker notebook instance, restarting the instance will also solve this problem. If you are using it in your local setup, then please re-install or update the installation of Boto3. Let us know if you still face the same issue.


update multivariate related traits in statistics module'

fixed the last one here, was just DVector used in Multinomial


feat: bot detecting + go_router integration for rendering otherwise flutter web

Description
Right now Flutter is horrible for SEO because even if you were to use some sort of pre-rendering tool, it is generating canvas.  Meanwhile Jaspr provides a great rendering system that is super friendly for SEO.
If Jaspr integrated with standard flutter and provided tooling to take go_router and source generate the same urls for jaspr so that we could share all of the same code, and generate simple versions of all of the routes for the search engine bots, we could have the best of both worlds where the bots got their SEO and everyone else got the rich Flutter client.
If there was a way to create a hybrid project that generated both in the same project and setup the rules so that the auto-rendering occured, this would be the best of all worlds.
Additional Context
The intent here is to provide a side-by-side rendering for each page that you care about from your flutter app that would render using jaspr only for the SEO bots, and every other case would use the Flutter SPA.

You can certainly set this up for yourself. Ideally even make a package that does this.
I wouldn't see this as a feature for Jaspr itself though.

Any guidance on how to integrate it with go_router?


Cannot use Spring Boot 2.x

I am using Spring Boot 2.6.2 in my project. The org.hibernate.MappingException is thrown when I use Association  type. I tried this also with the jmolecules-spring-data-jpa in jmolecules-examples, which is using Spring Boot version 3.6.1 as I write this message. The test is completed successfully. But when I change the Spring Boot version to 2.6.2, it's complaining:
javax.persistence.PersistenceException: [PersistenceUnit: default] Unable to build Hibernate SessionFactory; nested exception is org.hibernate.MappingException: No Dialect mapping for JDBC type: 821576394

This is the start of the stack trace.

Does JMolecules support Spring Boot 2.x?

If you can provide a reproducer project, I can take a look. While we have general integration tests against Spring Boot back until 2.6, note that the latest 2.6 depends on Hibernate 5.6.15.Final. Both that Boot and Hibernate versions have been effectively out of support for quite a while. I can only recommend moving to a newer Spring Boot version, or, at least, upgrade to 2.7.15, which is the version we build against.

Closing due to lack of feedback and the Boot version this is reported as broken against being severely outdated.


ARROW-7532: [CI] Unskip brew test after Homebrew fixes it upstream

@ursabot crossbow submit homebrew-cpp

This is not yet ready: Homebrew/brew needs a new release. That happens pretty regularly so I'll check back.


An actual ui for the front page

I can design the ui for the front page of the tas management website with html and css.
The buttons, the background and the cards could be improved alongside a better font.
Please assign this to me under hacktober fest tag

@khumanshusingh you can do the needful

hello ,
i would like to request you to assign me this task under hacktoberfest tag,
i would create the best ui for the task manager page using html , css and simple javascript

@AditiBanik you also can contribute based on your both PRs we will merge accordingly


Missing mail section in development mode of confi.example.js

Just a minor nitpick, would it be okay to update the comments to mention ghost will crash out if mail section is missing or to add an empty one i.e. mail{},?
If it's okay I'll raise the PR.

This should not be happening, so if it is its a bug that needs fixing :)

Hi @booyaa, I responded to you in slack as well, like so:

sounds like an issue with your environment
if transport isn't configured, nodemailer falls back to direct mail, which might go boom for you: https://github.com/andris9/Nodemailer#use-direct-transport (edited)

So, a PR to fix things is welcome if it turns out to not be specific to your environment, but I don't see any need to modify default config in config.example.js

so in the absence of mail section it fallbacks to nodemail?

Hi @booyaa. No, if there is no configuration for mail in config.js, Ghost should use direct mail method which is part of the nodemailer version Ghost is using.
What error messages do you get when you experience it crashing?

Here's the stack trace (happens just after I hit OK LET'S DO THIS button) during setup phase:
Running in development mode and config.js (can attach if you want is a copy of config.example.js)
/usr/src/ghost/core/server/mail.js:95
                if (data.error.errno === 'ENOTFOUND') {
                              ^
TypeError: Cannot read property 'errno' of null
    at EventEmitter.<anonymous> (/usr/src/ghost/core/server/mail.js:95:31)
    at EventEmitter.g (events.js:180:16)
    at EventEmitter.emit (events.js:95:17)
    at DirectMailer.<anonymous> (/usr/src/ghost/node_modules/nodemailer/node_modules/directmail/lib/mailer.js:138:30)
    at asyncCallback (dns.js:68:16)
    at Object.onanswer [as oncomplete] (dns.js:147:9)

npm ERR! Linux 3.18.11-tinycore64
npm ERR! argv "node" "/usr/local/bin/npm" "start"
npm ERR! node v0.10.38
npm ERR! npm  v2.7.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node index`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node index'.
npm ERR! This is most likely a problem with the ghost package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node index
npm ERR! You can get their info via:
npm ERR!     npm owner ls ghost
npm ERR! There is likely additional logging output above.
npm ERR! Linux 3.18.11-tinycore64
npm ERR! argv "node" "/usr/local/bin/npm" "start"
npm ERR! node v0.10.38
npm ERR! npm  v2.7.3

npm ERR! Uncaught, unspecified "error" event.
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /usr/src/ghost/npm-debug.log

I'm assuming direct mail means it's looking for the sendmail or something similar (a bit like how php's mailer works?)

Thanks for that. No, direct mail is just that. Direct. It sends the message to the MX server of the recipient. We've removed sendmail a while ago.

I think I've shed some ligh on this problemt, and apologies for not mentioning this upfront. I'm using docker, furthermore my dev docker engine is using the virtual box boot2docker image (a super light weight linux distro). I was able to avoid a crash by explicitly declaring a dns server using --dns <IP_ADDRESS>
Ghost is running in development...

 Unable to send welcome email, your blog will continue to function.
 Please see http://support.ghost.org/mail/ for instructions on configuring email

Investigating the crashing out exception on nodemailer issues confirms this is a DNS issue: andris9/Nodemailer/issues/409
Is it worth trying raising a bug report for the crashing version of the exception? I don't mind if it's deemed specific to my environment, hopefully this issue will help others trouble.

This is sounding more and more like it is the same as this bug: #5095
Are we perhaps just not handing errors from nodemailer quite right? cc @javorszky :wink:

That may be the case. I'll look into it over the weekend. Also backporting things. And allowing fancy domains. :)

And picking people up from airports.
I'd also fancy a boat, if you wouldn't mind.
On Fri, Apr 24, 2015 at 6:59 PM, Gabor Javorszky<EMAIL_ADDRESS>wrote:

That may be the case. I'll look into it over the weekend. Also backporting
things. And allowing fancy domains. :)
—
Reply to this email directly or view it on GitHub
https://github.com/TryGhost/Ghost/issues/5188#issuecomment-95992232.

That's not the weekend :P

I'm going to close this against #5243.
As discussed above, this isn't really an error specific to sending mail, but rather a DNS-related error that propagates through nodemailer and then our error handling code incorrectly assumes that an error object exists when it may not.


RC Test: Number Line: Operations 1.0.0-rc.1

@amanda-phet, @KatieWoe, and @kathy-phet - number-line-operations/1.0.0-rc.1 is ready for RC testing. This is the first
RC for this sim, though two dev tests have already been done, so a thorough test on all supported platforms is necessary.
Please log issues in https://github.com/phetsims/number-line-operations/issues.
Assigning to @kathy-phet and @KatieWoe for prioritization.

General RC Test

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test all query parameters on all platforms. (See QA Book
for a list of query parameters.)
Download HTML on Chrome and iOS.
Make sure the iFrame version of the simulation is working as intended on all platforms.
Make sure the XHTML version of the simulation is working as intended on all platforms.
Complete the test matrix.
Don't forget to make sure the sim works with Legends of Learning.
Test the Game Up harness on at least one platform.
Check this LoL
spreadsheet and notify AR or AM if it not there.
If this is rc.1 please do a memory test.
When making an issue, check to see if it was in a previously published version
Try to include version numbers for browsers
If there is a console available, check for errors and include them in the Problem Description.
As an RC begins and ends, check the sim repo. If there is a maintenance issue, check it and notify developers if
there is a problem.

Focus and Special Instructions
[Provide further instructions here. If you have any further tests you want done or specific platforms you want tested,
list them here. Any behaviors you want QA to pay special attention to should be listed here.]

Issues to Verify

[x] Operation off scale message and text box behave oddly with long strings
[x] Alignment of ring and value looks off (Net Worth screen)
[x] Screenshots take longer on last two screens

These issues should have the "status:ready-for-qa" label. Check these issues off and close them if they are fixed.
Otherwise, post a comment in the issue saying that it wasn't fixed and link back to this issue. If the label is
"status:ready-for-review" or "status:fixed-pending-deploy" then assign back to the developer when done, even if fixed.

Link(s)

Simulation
iFrame
XHTML
Test Matrix
Legends of Learning Harness

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

Memory test looks ok:

Memory test looks ok:

Some screenshots look a bit fuzzy, but I'm not sure it's issue worthy. Let me know if you think it is @jbphet

Some screenshots look a bit fuzzy, but I'm not sure it's issue worthy. Let me know if you think it is @jbphet

@KatieWoe - It doesn't look particularly fuzzy to me.  Are you referring to the bag just under the number line, the screenshot in general, or something else?

@KatieWoe - It doesn't look particularly fuzzy to me.  Are you referring to the bag just under the number line, the screenshot in general, or something else?

The bag, and the chips in the icon image

The bag, and the chips in the icon image

QA is done

QA is done

Thanks team.  I'll close this and follow up on the individual issues.

Thanks team.  I'll close this and follow up on the individual issues.


Added Send All

This method should allow users to send all owned omni assets in 1 transaction.
More detailed description here
https://github.com/OmniLayer/omnicore/issues/153

@marv-engine: is this an all or nothing transaction - transfers of all available balances succeed or none of them succeed if at least 1 fails?

Current behavior of https://github.com/OmniLayer/omnicore/pull/155 is that the transaction is only valid, if at least one token is transferred. This is in line with send to owners transactions, which require at least one receiver to be valid.

certain sends may be blocked in the future, e.g. a non-transferable SP (maybe that's just not in the available balance)

I think we'd most likely create a new "tally" type, such as "reserved for sell offers", which is not part of the "available" balance.

side effects of transfers

The dilemma started with the "grant tokens" implementation (see https://github.com/mastercoin-MSC/mastercore/issues/227), which simply calls the simple send logic. I'm not sure, if the passive effect (i.e. crowdsale participation) was intended, but now we have to live with it, I guess. It would be great, if this would be specified (of course not part of this PR).
I'd be fine, if the send all transaction does not trigger crowdsale participations, and thanks for pointing out that send to owners is without passiv effect. This serves as great justification, why send all may not trigger the participation either.

In this context: if the transaction has no receiver, then we assume the sender is the receiver, and the transaction may be a valid "send all to self" transaction. This mirrors the behavior of "simple send".

Send to owners is a great example, thanks!
In this context then it seems feasible to make crowdsale participation
limited to an explicit simple send only.
I'll take a look at whether any grants have ever triggered crowdsale
participation and if they have not, it may be a viable option to say that
as of X consensus-affecting feature activation that grants will no longer
have a secondary affect of participating in a crowdsale.  As long as such
an action doesn't occur before then we could remove current logic in a
future release.
On Jul 31, 2015 2:07 AM, "dexX7"<EMAIL_ADDRESS>wrote:

@marv-engine https://github.com/marv-engine: is this an all or nothing
transaction - transfers of all available balances succeed or none of them
succeed if at least 1 fails?
Current behavior of OmniLayer/omnicore#155
https://github.com/OmniLayer/omnicore/pull/155 is that the transaction
is only valid, if at least one token is transferred. This is in line with
send to owners transactions, which require at least one receiver to be
valid.
certain sends may be blocked in the future, e.g. a non-transferable SP
(maybe that's just not in the available balance)
I think we'd most likely create a new "tally" type, such as "reserved for
sell offers", which is not part of the "available" balance.
side effects of transfers
The dilemma started with the "grant tokens" implementation (see
mastercoin-MSC/mastercore#227
https://github.com/mastercoin-MSC/mastercore/issues/227), which simply
calls the simple send logic. I'm not sure, if the passive effect (i.e.
crowdsale participation) was intended, but now we have to live with it, I
guess. It would be great, if this would be specified (of course not part of
this PR).
I'd be fine, if the send all transaction does not trigger crowdsale
participations, and thanks for pointing out that send to owners is without
passiv effect. This serves as great justification, why send all may not
trigger the participation either.
—
Reply to this email directly or view it on GitHub
https://github.com/OmniLayer/spec/pull/313#issuecomment-126384580.

I've run through a full parse with some extra code to look for crowdsale participation from a grant and I can't see any, so I'm fairly confident no grants have ever triggered crowdsale participation to date.

In the context of an unrelated discussion about the mutability of transactions with test properties on mainnet:
The "send all" transaction currently violates that the main and test ecosystem should be strictly seperated. This has no direct effect on the global state, and the resulting main ecosystem balances [...] are exactly the same in any case, even if we assume the test ecosystem history may not be immutable.
However, if we assume the test ecosystem history is mutable, then it may have an impact on transaction validity. Right now a "send all" transaction is considered as valid, if at least one token was transferred, which may be in the test or main ecosystem.
To get to the point: I was wondering, if the "send all" transaction should come with an ecosystem parameter, to specify, whether test or main properties should be transferred.

I was wondering, if the "send all" transaction should come with an ecosystem parameter, to specify, whether test or main properties should be transferred.

@dexX7 Yes, all transactions should specify the ecosystem.
Also,

Right now a "send all" transaction is considered as valid, if at least one token was transferred

In this case, it's not necessarily one whole divisible token, but any fractional amount qualifies.

@marvgmail: Yes, all transactions should specify the ecosystem.

Alright, so the new transaction format would be:
[version: 0] [type: 4] [ecosystem]

In this case, it's not necessarily one whole divisible token, but any fractional amount qualifies.

Yup, sure. I was referring to base units.

What about MetaDEx cancels?  Is that not the same kind of scenario?  Where we allow either ecosystem, or both?
Q - why invalidate a 'send all' transaction that does not transfer any tokens.  Other "no-ops" are not invalidated:

We don't invalidate a MetaDEx cancel that doesn't match any trades.
We don't invalidate sends to self that don't move any tokens.
etc etc

What about MetaDEx cancels? Is that not the same kind of scenario? Where we allow either ecosystem, or both?

It would probably best to allow only valid ecosystem values, i.e. no wildcard for both. It was initially done this way (with wildcard) due to the transaction message format (with property/property desired).

Q - why invalidate a 'send all' transaction that does not transfer any tokens.

Hmm.. I think we should get rid of those no-ops altogether. Send-to-self should be allowed, if the receiver is explicitly specified as such, but the current fallback to send-to-self, if no receiver was provided, just seems unnecessary.
I'd argue against no-ops with a counter question: do you think we should remove all checks, whether amounts to be transferred/created/... are not zero, and allow sends with zero tokens?

A few words to my motivation in this context: the no-op question could become more relevant in the future. Let's assume we extend the methology of consensus hashes, and not only include balances [...], but also hash a list of valid transactions.
To ensure (historical) consensus hashes/checkpoints don't change, this requires that the validity of historical transactions is immutable.
Based on your earlier statement, that test property transactions may not be immutable, it follows that the validity of historical transactions may not be immutable either, if transactions without strict main/test ecosystem seperation are involved, thus introducing the chance of changing (historical) consensus hashes.

It would probably best to allow only valid ecosystem values, i.e. no wildcard for both. It was initially done this way (with wildcard) due to the transaction message format (with property/property desired).

I agree here - I think we should have explicit separation of test and main ecosystems.  If MetaDEx is the only place we allow a single payload to impact both test and main, that's close that behaviour now before live.

I'd argue against no-ops with a counter question: do you think we should remove all checks, whether amounts to be transferred/created/... are not zero, and allow sends with zero tokens?

Nope.  But there is a big difference here that needs to be considered, explicit versus implicit actions.
Explicit actions, eg "Send X from A to B" should be invalidated if that explicit action cannot be performed, either because X doesn't exist or is insufficient value.
But implicit actions, eg "Send {any} from A to B" should not be invalidated if {any} does not have any matches, because no matches is not the same as an invalid value.  Just the same as "Cancel {any} trades from A" is not invalid if {any} does not have any matches.
In both cases A and B are explicit values and should invalidate if they are not acceptable values.
I hope I've explained that right hehe :)

Based on your earlier statement, that test property transactions may not be immutable, it follows that the validity of historical transactions may not be immutable either, if transactions without strict main/test ecosystem seperation are involved, thus introducing the chance of changing (historical) consensus hashes.

I agree here 100%, which is why I think there should be a "ban" if you will on any transactions that can cross ecosystems (ie single payload = single ecosystem only).

But implicit actions, eg "Send {any} from A to B" should not be invalidated if {any} does not have any matches, because no matches is not the same as an invalid value. Just the same as "Cancel {any} trades from A" is not invalid if {any} does not have any matches.

This sounds reasonable. How about the other MetaDEx cancel operations then? The cancel operations are currently only considered as valid, if trades are really cancelled.

Oh, my mistake - MetaDEx cancels shouldn't be invalid if they don't match
anything imo.
On Aug 4, 2015 12:37 PM, "dexX7"<EMAIL_ADDRESS>wrote:

But implicit actions, eg "Send {any} from A to B" should not be
invalidated if {any} does not have any matches, because no matches is not
the same as an invalid value. Just the same as "Cancel {any} trades from A"
is not invalid if {any} does not have any matches.
This sounds reasonable. How about the other MetaDEx cancel operations
then? The cancel operations are currently only considered as valid, if
trades are really cancelled.
—
Reply to this email directly or view it on GitHub
https://github.com/OmniLayer/spec/pull/313#issuecomment-127458600.

But implicit actions, eg "Send {any} from A to B" should not be invalidated if {any} does not have any matches, because no matches is not the same as an invalid value. Just the same as "Cancel {any} trades from A" is not invalid if {any} does not have any matches.

This sounds reasonable. How about the other MetaDEx cancel operations then? The cancel operations are currently only considered as valid, if trades are really cancelled.

I know this is not directly related to Send All, but I think both Send All and MetaDEx cancels should not be invalidated if they do not have any matches.  Since MetaDEx isn't live yet I feel we have room to make that change without affecting consensus.  Thoughts?

Since MetaDEx isn't live yet I feel we have room to make that change without affecting consensus. Thoughts?

Supported.

Supported

What do you think to an approach like https://github.com/zathras-crypto/omnicore/commit/a751befa977e9c870f4d1e809a06d7dc248e1b79

It's about a month and a 1/2 since the last comment. Where do we stand on this PR?

The "send-all" command is part of Omni Core 0.0.10.
The transaction format is as follows:
[version: 0] [type: 4] [ecosystem: 1 (for main) or 2 (for test)]

All available (i.e. not reserved) tokens in the given ecosystem are transferred from the sender to the receiver.
The transaction is considered as invalid, if no tokens were actually moved, i.e. the sender must have tokens in the given ecosystem.
See also the RPC documentation for "omni_sendall".

The transaction "send all" is part of the current consensus rules, so we should build upon this pull request and eventually integrate it.


[1608] Sync courses when updating courses and site_statuses

Context
The frontend doesn't necessarily know when a course is due to be synced with search-and-compare, the concern for that knowledge should be in the backend.
Changes proposed in this pull request
Call ManageCoursesAPIService in update actions for both courses and site statuses.
Guidance to review
There is a regression here: the frontend only called sync once after performing all the site_status saves. This calls sync for each site_status.
We think the majority of courses only have one site, so this shouldn't  impact current functionality. The syncing is idempotent and can be called multiple times safely.
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Sync a course to search-and-compare when changing vacancies or locations

We think the majority of courses only have one site
But some will have 30.


Update Minimal TLS Version enforcement for new Azure Cosmos DB Accounts

Cosmos DB - Minimal TLS Enforcement to TLS 1.2
This update changes the documentation for the MinimalTlsVersion Property in the API Version 2023-04-15. This will only change the default value of the Minimal Tls Version when a Cosmos DB Account is created.
The enforcement used to be Tls 1.0, now it is Tls 1.2.
This behavior in TLS was already notified to the customers in https://learn.microsoft.com/en-us/azure/cosmos-db/self-serve-minimum-tls-enforcement#how-to-set-the-minimum-tls-version-for-my-cosmos-db-database-account

Hey @rkmanda just as a FYI: This PR only updates documentation, so it shouldn't be considered as a breaking change. The documentation guides to an already existing property, but with a new default value.

@rkmanda can we merge this PR? This is just a documentation update, and we have got approvals as well.

@rkmanda ping on this.


[TRACKER] Movement System

Goals
Provide a relatively easy to use, adaptable, and customizable system for pathfinding and calculating an objects available moves.
Write-Up
Adaptable, replaceable, and extensible.
Provide a basic movement system and allow it to be replaced as needed.
Features for Release

[x] Multiple units in tile selection
[x] Movement Component with basic movement information
[x] Backend to handle the movement system events and update resources with support for any implementation easily
[x] Default movement system implementations - advance wars style
[x] Option to override a tiles movement cost over it's terrain cost and support it in the backend. Either through objects or through the actually tile. Ex. Building a road on the tile and updating the movement cost through that //-- Tiles currently use a unique component for their move costs. Nothing stops anything from updating this and changing it. It wont get automatically changed back to the default terrain cost if its changed though.
[x] Option on objects to override their allowed terrain types for specific other objects. Ex. A bridge building object that allows land units to go over the water and prevents water units from going onto it. // Implemented optional ObjectMovementTypeRules. If assigned to an object it checks the tiles during the TileMoveCheck phase to see if there is an object on the tile that has a rule related to one of the rules in the objects ObjectMovementTypeRules component

Notes

Finished as of #31 - Movement system is not complete but the mvp is done enough to work on other stuff. See #33 for tracking the next stage of the movement system


Start testing runtime execution of wasm32 simd intrinsics

This commit starts to add support to CI to not only compile all wasm32 SIMD intrinsics but also actually execute some of them at runtime. The master branch of the nodejs/node repository has recently merged an update of v8 which aligns internal opcode definitions with the most recent version of the wasm SIMD spec. As a result we can start actually executing some WebAssembly code with SIMD instructions in it!
Unfortunately not all SIMD instructions are implemented in v8 right now. In LLVM this is known as the +simd128 feature (what v8 implements) and the +unsupported-simd128 (everything v8 doesn't implement but is spec'd). We continue to compile the library with both features (to make sure it works) but this PR then also adds a mdoe where it compiles with only +simd128 and executes the resulting binary in node.js. Unfortunately compiling with just +simd128 results in lots of LLVM errors, so we also compile with --cfg only_node_compatible to statically remove non-node-compatible functions.
The end result is that we're successfully executing all #[assert_instr] annotations! Additionally I've started uncommenting some tests in mod tests and updating them to the new intrinsic names. There's still a lot more work to be done there, but I figured it'd be good to post this for initial feedback.
In general, the changes here are:

Compile a version of Node.js with wasm support (can download binaries once node 12 is released)
Update CI to dynamically download the right version of wasm-bindgen instead of pinning
Add #[cfg] to optionally remove SIMD which isn't implemented in node.js
Add CI configuration to execute select SIMD tests in node.js

I should also point out that building node on CI may just take too long, so we may want to just hold off on this until Node 12 is release. (I don't know when that will be, but I presume not too far in the future)

Looks like the build job is getting very close to the 50min travis time limit (49min ..). The error is:
Removing intermediate container ec77fe5cf34f
Successfully built 97372c202902
Successfully tagged stdsimd:latest
+mkdir -p target
+echo Running docker
Running docker
+id -u
+id -g
+rustc --print sysroot
+pwd
+pwd
+docker run --user 2000:2000 --rm --init --volume /home/travis/.cargo:/cargo-h --env CARGO_HOME=/cargo-h --volume /home/travis/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu:/rust:ro --env TARGET=wasm32-unknown-unknown --env STDSIMD_TEST_EVERYTHING --env STDSIMD_ASSERT_INSTR_IGNORE --env STDSIMD_DISABLE_ASSERT_INSTR --env NOSTD --env NORUN --env RUSTFLAGS --env STDSIMD_TEST_NORUN --volume /home/travis/build/rust-lang-nursery/stdsimd:/checkout:ro --volume /home/travis/build/rust-lang-nursery/stdsimd/target:/checkout/target --workdir /checkout --privileged stdsimd bash -c PATH=/rust/bin:$PATH exec ci/run.sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100     9    0     9    0     0     44      0 --:--:-- --:--:-- --:--:--    44

gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
travis_time:end:1a5d3d37:start=1555012192491503409,finish=1555015069636286444,duration=2877144783035
The command "if [ "${NO_DOCKER}" = "1" ]; then
    ci/run.sh $TARGET
else
    ci/run-docker.sh $TARGET
fi
" exited with 2.

Done. Your build exited with 1.

so it appears that node does indeed build in that timeframe, but tar failed.

Alas! That was probably a bug around the wasm-bindgen release but the 50m build time is a deal breaker for sure.
It looks like Node 12 may be released by the end of the month though, so we can try again around then!

If you want to merge this in the meantime, a quick "fix" would be to just compile node 12 from source on Linux, tar it, and upload the result somewhere (e.g. a github repo_, and pull that here.

It's possible yeah but there's not necessarily a burning need to get this in, so I'm fine waiting


Can we perform Named Entity Extraction Using Datumbox

I want to know does datumbox help us in Named Entity Extraction or Named Entity  recognition?__

Unfortunately at the moment Datumbox does not provide an out of the box Named Entity Extraction algorithm. You can build one using the algorithms provided in the framework but this is not a trivial task. If you do, please consider contributing the code by sending a pull-request.
Here are some references on how you could formulate it as a classification problem and use the Multinomial Naive Bayes algorithm:
https://arxiv.org/pdf/1509.04385.pdf
https://web.stanford.edu/class/cs124/lec/Information_Extraction_and_Named_Entity_Recognition.pdf

yes.I will try it from my side :) thanks


feat: add arc_lock feature and typedefs

This PR has two commits:

Adding the arc_lock feature and corresponding typedefs.
Configuring docs.rs to

document all features
use doc_auto_cfg to annotate required features on types
enable --generate-link-to-definition for rustdoc sources

You can test this with the following command:
RUSTDOCFLAGS="-Zunstable-options --cfg=docsrs --generate-link-to-definition" cargo +nightly doc --open --all-features

Closes https://github.com/rust-osdev/spinning_top/issues/24

Thanks a lot!
I don't have any more wishes currently, so as far as I am concerned, we are ready for a release. :)

Ok great! I opened https://github.com/rust-osdev/spinning_top/pull/26.


new release with async iterators

[ ] fix remaining tests
[ ] new function reduce (that you can curry on)
[ ] documentation
[ ] perf tests on batch
[ ] typescript update

Opened a ticket for the bug on async/execute https://github.com/babel/babel/issues/7912


Added JMESPath sort documentation. Solves #2536

Added JMESPath sort documentation. Solves #2536

Thanks for this @appieschot ! I have updated some minor changes like typo and punctuations but rest looks great. Merged manually.