Element with index attribute has broken attrs field when compiled

compiles to:
({
	tag: "div",
	attrs: vtext,
	dom: null
});

This was using the latest spike branch.

index="text" is invalid html.  Did you mean data-index="text"?

It also happens when passing a parameter called index to a component:
 <Component index="text"/>

@jpdutoit - Can you confirm that #14 fixes the issue?

Why keep the behaviour for number values? I couldn't see this documented
anywhere.
Will test it as soon as i get a chance.
On Wednesday, 23 March 2016, Justin Wilaby<EMAIL_ADDRESS>wrote:

@jpdutoit https://github.com/jpdutoit - Can you confirm that #14
https://github.com/trueadm/babel-plugin-inferno/pull/14 fixes the issue?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/trueadm/babel-plugin-inferno/issues/13#issuecomment-200461919

If you look at how Inferno Works, the schema is created with arguments whose name corresponds to the index of the template function prepended with "v" I don't think this is necessarily relevant for the spike branch since it seems there are changes that may obviate this but it makes sense in the master and dev branches of inferno.

Transpiles to:
tpl3295379617 = Inferno.createTemplate(function (v0) {
    return {
        tag: 'div',
        attrs: {
            selected: v0
        }
    };
});

function render(){
    return tpl3295379617(this.shouldBeSelected);
}

Where v0 represents the first argument in the template's function.

I should add that adding the index on an attribute's schema descriptor is here

Yeah, createTemplate does not exist anymore in spike, so I thought perhaps
that part was a remnant of earlier code that has not been removed yet.
Unless it has some purpose...
On Wednesday, 23 March 2016, Justin Wilaby<EMAIL_ADDRESS>wrote:

I should add that adding the index on an attribute's schema descriptor is
here
https://github.com/trueadm/babel-plugin-inferno/blob/master/lib/index.js#L66
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/trueadm/babel-plugin-inferno/issues/13#issuecomment-200521662

createTemplate and schemas have been removed in the spike branch. They are clunky in behaviour and made the codebase twice the size it should be with no gain in performance in most use-cases. I'm hoping to promote spike to master by Friday (in time for Easter) for both Inferno and this Babel plugin.
I've just got to sort out a few issues. If anyone is not on Slack, let me know so I can send you an invite. I'd love as much interaction before promoting to master, so we can squash the last few big bugs.

Closing as this is no longer an issue.


v0.3.0 release

Prepare to release v0.3.0

Pull Request Test Coverage Report for Build e2ad366eb68d4939b53d814336c2daaa16031296-PR-47

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 43776d5f07ac698054bfcffecf49143af429e5c9:
0.0%

Covered Lines:
302

Relevant Lines:
302

💛  - Coveralls


Bug in testing live cluster bug

Added a delay after deploying resources in testing live cluster, to make sure all pods are up and running

Test_docker fails because gcc is missing - I remember you mentioned it in the project meeting, but I don't know how to workaround it.

I fixed the test_docker failure, Should be ok after you merge the latest changes from master.

Also, maybe it's time to sync tests/fw_rules_tests/podlist/kubernetes-manifests.yaml with its origin, as it got a little bit old.
For example, our version specifies using adservice version v0.1.3 - but there is no longer such version on Google's registry, so the pod keeps failing.


RATIS-1516. Remove non-hadoop modules from ratis-hadoop-projects.git

See https://issues.apache.org/jira/browse/RATIS-1516

I think we should also update scm URLs in pom.xml.

Sure.  Will update the URLs.

sonar check and corresponding sonar.sh will need to be updated, too, but this requires registering the new repo at sonarcloud.io. I think that can be done in a follow-up task.

I see.  Thanks for pointing it out.


vscode extension needs to perform an analysis

Fixed with https://github.com/konveyor/editor-extensions/pull/54


Mapped types relations inside function body

I have been struggling to write a simple map function that return different types, and couldn't find any good examples in Google for that, but shouldn't this simple example work?
interface A {
    a: string;
}

interface B {
    b: string;
}

interface TypeMap {
    a: A;
    b: B;
}

function map<K extends keyof TypeMap>(key: K): TypeMap[K] {
    if (key === 'a') {
        return {
            a: 'a',
        } as TypeMap['a'];
    }
    return { b: 'b' } as TypeMap['b'];
}

It complains that the Type A is not assignable to A & B. Imo A & B is the wrong type to compare with inside the function. I should at least be able to assert the type inside the map function? Why not just let it be A | B instead?
Playground Link:
http://www.typescriptlang.org/play/?ssl=1&ssc=14&pln=21&pc=2#code/JYOwLgpgTgZghgYwgAgILIN4Chm+XALmQGcwpQBzAbiwF8stRJZEUAhTHPAIyNPJDU6DJtHhJkAFQCeABwgBZOLM558RVDTW9kbGvSwwAriARhgAexDIAtsoA8AaWQQAHpBAATYsgDWEaQsYKTlFZQA+AAp-aSJHAEoiGXklWQBtRwBdVTxgYOiA5ABeEuQAcjgy+Jy1XCgIMCMoa2xatvVyyoAaLjbafB9ksPSKssytPHo1esbmzGQdMu4y5H64QdDUtKWx-SA
Related Issues:

See #33014


Openstack infra adding filesystems and custom attributes to drift

Only last commit is relevant:
Adding filesystems and their custom attributes into drift
Making possible to collection of records to have include.
So we can show custom attributes of the filesystems.

@blomquisg @Fryguy

should I enhance this yaml, or create special yaml for openstack infra host?
I wonder how to expose a name of the file in each custome_attribute (how to write it in the hosts.yaml, also it should not fire the db query for each, so maybe I will just add that field to custom_attributes?

@miq-bot remove_label wip

@Fryguy @matthewd @blomquisg what is the status of this PR?

@blomquisg @Fryguy @dclarizio Please review - specifically the YAML file changes.  Note comment from @Ladas about it here.

Enhancing the base yaml is fine :+1:

@ladas, can you throw in a screenshot for @dclarizio's sanity check?

@blomquisg @dclarizio here it goes
files drift

filesystems custom attrs drift


Update usage.md

I worked through the documentation until the section with example commands.
It's not consistent and there are likely a lot of problems, so let's see this as work in progress. Just add changes to this PR before we decide on merging.

PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/metatdenovo branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

@nf-core-bot fix linting

@nf-core-bot fix linting


Refactors Program, Course and Module View Components to align with the user flow and passed down props

Added CourseView component rendering to ProgramView Component
Added passed down props to CourseView Component and refactored code to utilize this
Refactored Module component to take advantage of passed down props from CourseView component

Truly super impressed with how clean, readable, and welcoming your code is brotha!!


CLI: module options collision

The short option names of the following options are the same:
    -m, --modules [modules]      Module formatter type to use [common]
    -m, --module-ids             Insert module id in modules

Moreover --module-ids description is unclear. Could you enrich its documentation?

Good catch, thanks! The module ids are for enabling ids for SystemJS/UMD/AMD. Odds are if you don't know what they are then you probably don't need them.


Remember last sort

While testing the new sorting functionality, we got this really helpful feedback from an advisor:
My only suggestion is to ask if there is any way for it to keep your last sort setting as you click in and out of individual students.  For example, I sorted by status, and [specific student] was the student at the top of the sort, followed by four others with statuses for [this term].  I checked out [student]'s data, but when I went back to my full list to find the next student to check on, my list wasn't sorted by status any more - it had reverted back to a last name sort.  This is not a real problem, I just clicked on status to sort again, but if my goal was to check statuses on all my students, it was annoying to have to sort again by status every time I clicked in and out of an indiv student record.  Would there be any way to have it keep your last sort setting as you click in an out of indiv students (if others agree that would be more efficient for how they work)?

Hmm, it does look like there's an option.
https://mottie.github.io/tablesorter/docs/example-widget-savesort.html
This would persist only until the local cache is stored and only on that computer but it looks like it would do this. To go beyond we'd have to persist some settings to the server which would be more effort.

I am not sure if I see a need to go beyond local cache on that computer so this seems fine. We may have to remove the default ordering by last name in order for this to work. Is there a setting in tablesorter to set a default order? Right now our default ordering (unless you changed it and I didn't notice) is handled on the backend.


Witgen: Remove sequence cache for block machines

Fixes #1559
The sequence cache is often causing trouble, and I wonder how big the performance benefit even is after #1528 (and given that we don't spend that much time in secondary machines).
We should benchmark this PR and see what the impact is. If it is small, we can through away most of executor/src/witgen/sequence_iterator.rs!

On the Keccak RISC-V example, the block machines do seem to be roughly 2x slower. These are two representative runs:
========== BEFORE
 == Witgen profile (1766802 events)
   39.6% (    4.1s): FixedLookup
   37.7% (    3.9s): Main Machine
   12.7% (    1.3s): Secondary machine 0: main_binary (BlockMachine)
    6.3% ( 652.5ms): witgen (outer code)
    3.1% ( 319.1ms): Secondary machine 2: main_shift (BlockMachine)
    0.6% (  57.7ms): Secondary machine 1: main_memory (DoubleSortedWitnesses)
    0.0% ( 251.6µs): Secondary machine 3: main_split_gl (BlockMachine)
  ---------------------------
    ==> Total: 10.349325541s

========== AFTER
 == Witgen profile (3086124 events)
   35.4% (    4.6s): FixedLookup
   29.8% (    3.8s): Main Machine
   23.3% (    3.0s): Secondary machine 0: main_binary (BlockMachine)
    5.7% ( 738.2ms): Secondary machine 2: main_shift (BlockMachine)
    5.3% ( 677.5ms): witgen (outer code)
    0.4% (  52.3ms): Secondary machine 1: main_memory (DoubleSortedWitnesses)
    0.0% ( 335.5µs): Secondary machine 3: main_split_gl (BlockMachine)
  ---------------------------
    ==> Total: 12.854500542s

#1562 is a better solution.


Bugfix filenames turned into image names

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (3046677) 83.67% compared to head (9e80ed7) 83.73%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
+ Coverage   83.67%   83.73%   +0.06%
==========================================
  Files           5        5
  Lines         533      535       +2
==========================================
+ Hits          446      448       +2
  Misses         87       87

Files
Coverage Δ

src/napari_workflows/_workflow.py
76.83% <80.00%> (+0.13%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[Bug]: useSubmit hook does not post to the route action function

Which Remix packages are impacted?

[ ] remix (Remix core)
[ ] create-remix
[ ] @remix-run/architect
[ ] @remix-run/cloudflare-workers
[ ] @remix-run/dev
[ ] @remix-run/express
[ ] @remix-run/netlify
[ ] @remix-run/node
[X] @remix-run/react
[ ] @remix-run/serve
[ ] @remix-run/server-runtime
[ ] @remix-run/vercel

What version of Remix are you using?
1.0.6
Steps to Reproduce
Clone and run this repo: https://github.com/sbernheim4/remix-bug-demo.
The submit function returned by useSubmit does not hit the route's action function.
Expected Behavior
The action function should be invoked on the post request that the submit function initiates.
Actual Behavior
An error is thrown:
Uncaught (in promise) Error: Route "root" does not have an action, but you are trying to submit to it. To fix this, please add an `action` function to the route

I believe this comes from this line.
I believe this line should instead be something like
  matches = [matches[matches.length - 1]]

but that doesn't solve everything. This only causes another issue (which I don't yet understand) when the final fetch function is made and the remix server returns a 500 error.

Closing this out. Issue was the folder/file structure where I had
app/
  routes/
    my-route/
      index.tsx

when I should have had
app/
  routes/
      my-route.tsx


Allow all origins for presigned URL

One of the biggest advantages of presigned URL for files is to allow uploading files directly from the client / browser. The client could be the Firefly SPA (React spectrum) or a 3rd party site. However, it is currently not possible to make fetch requests from client-side javascript to the presigned URL (on storage CDN: https://firefly.azureedge.net/) due to CORS blocking.
const response = await fetch(
  url,
  {
    method: 'PUT',
    body: file,
    headers: {
      'Content-Type': 'image/png'
    }
  }
)

Expected Behaviour
The file should be uploaded to the presigned URL.
Actual Behaviour
Access to fetch at 'https://firefly.azureedge.net/...' from origin 'https://my-namespace.adobeio-static.net' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Is there any update on this issue? Can this be expedited please? We need it on priority.

@meryllblanchet
The use case is we need to do a file upload to an external api from firefly app and if we use presigned URL to initially upload the file to it , we're getting CORS and completely blocked. There's no other alternative for file upload functionality and so we would like to expedite this.

@shellygoelisobar this is already part of the technical solution to me.
What about uploading the images via an I/O Runtime action directly? This shouldn't be any issue if the files are compatible with Runtime's payload limitations of 1 MB.
If we speak about larger files, I am afraid that you might consider aio-lib-files as a content storage or repository, which is not what we have designed it for - rather for files computation needs, e.g. Asset Compute Microservices.

@meryllblanchet  We get a server error on uploading directly through the runtime action and seems like the file is lost even though smaller than 1MB size. Let me give it a try again and come back to you.

@shellygoelisobar, let's also ping @duynguyen in case you need some more feedback related to the issue you're facing.

@meryllblanchet : we have a different use-case where we want previously uploaded files via aio-lib-files to be retrievable client-side using corresponding presigned URLs. We're running into the same CORS error as mentioned in the original comment.

Same issue as other posters.. What's the latest on this?
Response size of 1 MB is not enough - so we can use presigned urls instead. But CORS prevents us from using presigned urls. So how do I get access to the contenst of my file (if >1MB)? At that point, what is the purpose of presigned urls anyways if you cannot access it outside the app builder ecosystem like within another action?

Hi @amol-anand,
We recently allowed a few domains for CORS to support multiple use cases. Can you share the domains that you want to access pre-signed URLs from?
cc: @pru55e11


Mime-type validation for base64 inline images, template plugin.

Added mime-type validation to the base64 plugin, it will now display an error if you select a file that is not an image.
Added a very basic template-plugin, select a template and it will replace the current editor content with the content the template. Because it is very difficult to find an icon for template, I also made it possible to have textual-buttons in the button-bar.

@Alex-D I've added an example for the template plugin.

Thanks!
I just need to check that when I go back home, then I should merge :)

Cool! It works well!


Teams Task Module - Deserialization Error on Teams mobile app for iOS

Error when loading Task Module on iOS iOS 14.8.1 / MS Teams v3.20.0
Copying the bug here: https://github.com/microsoft/botbuilder-python/issues/1872

@alinealfa - We are trying to repro it our side. Will update the findings soon.
Thanks

We have the repro of this issue and raised a bug Internal team is working on it. I will let you know once it is fixed and rolled out in public
Thanks

@alinealfa - Engineering team is working on this bug and will let you know once this is fixed.

We tested this in iOS 17.4 and this seems to be working now.
Microsoft Teams version:


Portal Aspect of the Rubin Science Platform: no access[BUG]

After the login in GitHub and entering the https://data.lsst.cloud/ link, if I select Portal I get:
"504 Gateway Time-out"
(while the Notebook Aspect works).
This message appears since a couple of days.
Thank you for the support.

Sorry about that. Can you try clearing any cookies you have from data.lsst.cloud in your browser and trying again?

A fix for the cause for this behaviour was deployed in yesterday's Patch Thursday. The workaround described should no longer be necessary for active users.


Add -fmodule-map-file= to RemapClangImporterOptions whitelist

This flag was missed in the RemapClangImporterOptions whitelist. Let me know if I should instead create this PR against a different branch (stable/swift 5?) or if more information is needed.

Friendly ping @adrian-prantl

Sorry I missed this.

@swift-ci test and merge

@adrian-prantl is it possible to merge this into swift-5.0 as well?

Unfortunately, the bar for any changes to swift-5.0 is very high at this point in time. Can you give an estimate for how important this bugfix is to you?

@adrian-prantl At the moment, we can't debug any swift binaries built with Bazel due to this issue.
I'd like to confirm that this fixes it, but I'm unable to properly build swift-lldb:
(building with Xcode 10.2 beta 4, swift-5.0-branch as all active branches for swift repos)
$ lldb/scripts/build-swift-cmake.py
CMake Error at stdlib/public/core/cmake_install.cmake:36 (file):
  file INSTALL cannot find
  "/Users/davg/dev/github/swift-lldb/build/Ninja-DebugAssert/swift-macosx-x86_64/./lib/swift/iphoneos/armv7/Swift.swiftmodule".
Call Stack (most recent call first):
  stdlib/public/cmake_install.cmake:40 (include)
  stdlib/cmake_install.cmake:37 (include)
  cmake_install.cmake:37 (include)

and then during debugging:
 ((Target*)0x7fb5fe03bc00)->GetSwiftASTContext() returning NULL - couldn't load the Swift stdlib
Shared Swift state for GBus could not be initialized.
The REPL and expressions are unavailable.

I also wonder if this remapping will fail to apply for relative paths. e.g. if we want to map a module a /BuildRoot/relative/path/module.modulemap, the -fmodule-map-file=relative/path/module.modulemapmay not be properly remapped to /BuildRoot/relative/path/module.modulemap with a .=/BuildRoot remapping.

Friendly ping, still can't build via the lldb/scripts/build-swift-cmake.py script:
$ lldb/scripts/build-swift-cmake.py
~/dev/swift-lldb
--- Installing swift ---
+ env DESTDIR=/Users/davg/dev/swift-lldb/install/ /Users/davg/.homebrew/bin/cmake --build /Users/davg/dev/github/swift-lldb/build/Ninja-DebugAssert/swift-macosx-x86_64 -- install
[0/1] Install the project...
...
-- Installing: /Users/davg/dev/swift-lldb/install/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift_static/iphoneos/libswiftCore.a
CMake Error at stdlib/public/core/cmake_install.cmake:65 (file):
  file INSTALL cannot find
  "/Users/davg/dev/github/swift-lldb/build/Ninja-DebugAssert/swift-macosx-x86_64/./lib/swift/iphonesimulator/i386/Swift.swiftmodule".
Call Stack (most recent call first):
  stdlib/public/cmake_install.cmake:40 (include)
  stdlib/cmake_install.cmake:37 (include)
  cmake_install.cmake:37 (include)

I am not familiar with that script. How does it invoke swift/utils/build-script? It's complaining about the 32-bit iphone simulator runtime missing, which either means that it itnvoked build-script without -i or that is because the 32-bit iOS SDK isn't present in the SDK that you are building against, which I think may be the case in more recent versions of Xcode. For testing I usually don't bother with installing LLDB, so perhaps you can also just run it from the build directory.
CC: @weliveindetail

It does indeed build lldb but lldb can't find the Swift stdlib modules - any ideas?
 ((Target*)0x7fb5fe03bc00)->GetSwiftASTContext() returning NULL - couldn't load the Swift stdlib
Shared Swift state for GBus could not be initialized.
The REPL and expressions are unavailable.

Two things I would try is

adding -i to build_script_arguments in lldb/scripts/build-swift-cmake.py
see if you can modify the swift cmake code to not expect an i386 runtime

that said, just invoking build-script -r -l -i -- --lldb-build-with-cmake (like the ci.swift.org cmake lldb bot does) should also result in an lldb that can find everything relative to its build location.

Thanks, the -i flag fixed it. I've confirmed that this addition fixes debugging for Bazel-based Swift projects (with dSYMs), can you please merge this into the swift-5.0-branch? Otherwise we won't be able to debug :(

Unfortunately, the bar for any changes to swift-5.0 is very high at this point in time. Can you give an estimate for how important this bugfix is to you?

Hey @adrian-prantl , this bugfix is extremely important to us (Google) encouraging the use of Swift in-house. Without it we are basically 2-3x link times and that is dissuading large teams from adopting Swift. Happy to discuss offline if that is useful, and let me know whom I have to bribe 😉 .

@adrian-prantl apologies.. wrong issue. This one is important to us (Google) because without it we are basically going to be unable to debug Swift 5 apps which would DEFINITELY dissuade large teams from adopting Swift. Still happy to discuss offline if that is useful, and let me know whom I have to bribe 😉 .

Just a quick clarification: Do you need this on Darwin, Linux, or both?

Primarily Darwin, I'd imagine debugging on Linux would run into the same issues though, so I guess both?


Mihon take long to load .cbz offline manga

Steps to reproduce
Read downloaded to .cbz offline manga
Expected behavior
When read offline .cbz the app like to load the website script before check offline file .cbz
But even i turn off wifi n data 4g it still loading and take long time to load the manga. Sometime i have to quit the manga title to library, sometime have ti restart the whole app
Actual behavior
Read fast like immediately when in previous Mihon (or Tachi) version
Crash logs
Noth
Mihon version
0.16.5
Android version
Android 11
Device
Vivo y21 4g ran
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.5.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Do you happen to have a large amount of chapters downloaded?

Sounds like a duplicate of #134, which has been fixed in mihon preview.

See #705

See #705

ok so i dont really get anything about the system but sound like this app not able to deal with 1000+ manga download even in .cbz file or not, u tell me if im wrong someth? if so then i'll wait h dev and close this issue 8))

See #705

ok so i dont really get anything about the system but sound like this app not able to deal with 1000+ manga download even in .cbz file or not, u tell me if im wrong someth? if so then i'll wait h dev and close this issue 8))

You are right, Mihon struggles when there are a lot of downloaded chapters. Whether they are images or in a .cbz doesn't matter for the struggling. For now, I can only recommend checking out the "Download Ahead" setting in Mihon, and deleting chapters after you've read them.

See #705

ok so i dont really get anything about the system but sound like this app not able to deal with 1000+ manga download even in .cbz file or not, u tell me if im wrong someth? if so then i'll wait h dev and close this issue 8))

You are right, Mihon struggles when there are a lot of downloaded chapters. Whether they are images or in a .cbz doesn't matter for the struggling. For now, I can only recommend checking out the "Download Ahead" setting in Mihon, and deleting chapters after you've read them.

i get it, thank you sir.


This is great for Classes, but it omits all routine, (mac), files.

This is great for Classes, but it omits all routine, (mac), files.  Do you have any code to also export the mac files as well?
Many thanks
Mark

You can use cache-torg-git source control for that.

Mark, thanks! Will try to add support for mac routines.

Thanks ev
I have the code to iterate through the routines and check the dates, I just haven't looked into how to export or build it into your management structure, I'd think you'd be able to embed it pretty simply :) !
Eduard, if I can only use one source control per namespace how do I this and cache-torg? This does exactly what I need with classes I don't need all the extra overhead in cache-torg.
Cheers

Hi Mark!
Try the latest version updated today, it supports generation Mac somehow. Please, test first

I tried it, it broke my system...
Firstly my version of cache can't handle * in $P so I've had to change it to
if ..#ExpMode set $p(name,".",$L(name,"."))=($zconvert($piece(InternalName,".",$L(InternalName,".")),"l")"."$p(name,".",$L(name,".")))
Also if I change namespace in studio I get lots of these errors..
^Sources is not setup so hooks will not work
I tried to change the global reference but that didn't help
Parameter cCSSNSpace = "PROGRAMMES";
/// setup control version for namespace and export all classes
ClassMethod SetUp() As %Status
{
/// ^[CSSNSpace]Sources = tempdir path
/// ^[CSSNSpace]Sources(ext,"*") -- subfolder for item type
S CSSNSpace=..#cCSSNSpace
kill ^[CSSNSpace]Sources
set ^[CSSNSpace]Sources=..#Folder_$zconvert($znspace,"l")_"/"
I need to try again with v2 in a new namespace but none of it was working when I first tried :(


UI: deleting buildConfigs

Motivation
https://issues.jboss.org/browse/AEROGEAR-7923
How to verify

Clone this branch and follow instructions from here
Make sure the OpenShift project you are targeting has Mobile CI/CD provisioned
Create Mobile Client and couple of buildconfigs in your OpenShift project
Refresh MDC UI
Select your Client -> Builds and try to delete them
Verify builds have been removed (also check in OpenShift UI) and that there are no errors in developer console

Comments

based on https://github.com/aerogear/mobile-client-service/pull/58

@jhellar thanks for review and suggestion which I've added now.

Pull Request Test Coverage Report for Build 544

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.497%

Totals

Change from base Build 527:
0.0%

Covered Lines:
474

Relevant Lines:
692

💛  - Coveralls


Kubectl docs should mention field validation

The reference documentation on kubectl currently does not mention anything about field validation kubectl --validate.
Specifically, kubectl prior to 1.25 used to detect unknown and duplicate fields via client-side validation built into kubectl.
Since 1.25 kubectl uses server-side validation to detect unknown and duplicate fields and kubectl users have the option to customize the behavior of the server via kubectl --validate

/cc @sftim

/language en
/wg api-expression
/triage accepted

Can I get assigned to this? @sftim

/assign FavourDaniel

I think this issue is resolved by #39716 and hence can be closed now

/close


Add danielfalk/smart-open.nvim

Plugin name
smart-open.nvim
Description
Smart file fuzzy finder that adapts to your usage and improves suggestions over time.
Repo link
https://github.com/danielfalk/smart-open.nvim
Tags
Fuzzy Finder

Tag reference:
Plugin Manager LSP Completion Language Syntax Snippet Register Marks Search Fuzzy Finder File Explorer Project Color Colorscheme Bars and Lines Startup Icon Media Note Taking Utility Terminal Integration Debugging Test Code Runner Neovim Lua Development Fennel Dependency Management Git Motion Keybinding Mouse Scrolling Editing Support Formatting Command Line Session Remote Development Split and Window Game Preconfigured Configuration Version Manager Programming Languages Support

Hey @danielfalk, your plugin is on NeoLand! Check it here https://neoland.dev/plugin/2176


If i run a command, it gets deleted and the command doesnt run.

When i run ANY command, the message i ran the command in gets deleted, and i get no reply from the bot, checked my console, and got nothing, ive removed a few commands that i dont need because my server is based on a family friendly pg streamer, is that an issue?

Hi, can you add me on Discord?
Conner#1386

aight have done im statham.json

i have the same issue

Message me on discord Kappador#1939


[Question] Setting up a classifier with params

Hi,
I am interested in deep-river. Is the project still being incremented?
Also, I am attempting to pass a module with specific params. A possible MWE would be as follows, however, it seems the Classifier class requires as input a non-instantiated nn.Module.
Am I missing something?
MLP code:
class MLP(nn.Module):

    def __init__(self,
                 input_dim: int = 512,
                 hidden_dim: int = 256,
                 output_dim: int = 2):
        super(MLP, self).__init__()
        self.input_dim = input_dim
        self.hidden_dim = hidden_dim
        self.output_dim = output_dim

        self.linear = nn.Linear(self.input_dim, self.hidden_dim)
        self.activation = nn.ReLU()
        self.linear_2 = nn.Linear(self.hidden_dim, self.output_dim)

    def forward(self, x):
        out = self.linear(x)
        out = self.activation(out)
        out = self.linear_2(out)
        return out

Pipeline construction:
    model = MLP(input_dim=dataset.n_features - 1,
                hidden_dim=(dataset.n_features - 1) // 2,
                output_dim=2)
    model_pipeline = compose.Pipeline(
        preprocessing.StandardScaler(),
        classification.Classifier(module=model,
                                  loss_fn='binary_cross_entropy',
                                  optimizer_fn='adam',
                                  )
    )

Hi @jpbarddal,
this project is still being incremented!
You are right the classifier requires a non-instantiated nn.Module.
The reason for that is that in data streams the number of classes to predict or the number of features are might not be known at the beginning of the data stream. For that reason we initialize the model within the first event of the data stream.
However, to pass specific params to the Module, you can still do that by adding it to the Classifiers arguments:
    class MLP(nn.Module):

    def __init__(self,
                 n_features: int = 512,
                 hidden_dim: int = 256,
                 output_dim: int = 2):
        super(MLP, self).__init__()
        self.input_dim = n_features
        self.hidden_dim = hidden_dim
        self.output_dim = output_dim

        self.linear = nn.Linear(self.input_dim, self.hidden_dim)
        self.activation = nn.ReLU()
        self.linear_2 = nn.Linear(self.hidden_dim, self.output_dim)

    def forward(self, x):
        out = self.linear(x)
        out = self.activation(out)
        out = self.linear_2(out)
        return out

    model_pipeline = compose.Pipeline(
        preprocessing.StandardScaler(),
        classification.Classifier(module=MLP,
                                  loss_fn='binary_cross_entropy',
                                  optimizer_fn='adam',
                                  hidden_dim=(dataset.n_features - 1) // 2,
                                  output_dim=2
                                  )
    )

Note, that I removed the input_dim argument and added the n_features field as this variable is set with the first event of the data stream.
Kind regards
Cedric

@jpbarddal


Can Not Activate rust-analyzer: Bootstrap Error

ERROR [7/8/2020, 10:37:58 AM]: [TypeError: node_fetch_1.default is not a function
	at Object.fetchRelease (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:15902:48)
	at bootstrapExtension (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:16118:31)
	at bootstrap (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:16094:11)
	at async tryActivate (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:16022:24)
	at async activate (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:15996:5)
	at async Promise.all (index 0)]
ERROR [7/8/2020, 10:37:58 AM]: Bootstrap error [TypeError: node_fetch_1.default is not a function
	at Object.fetchRelease (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:15902:48)
	at getServer (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:16235:31)
	at async bootstrapServer (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:16145:18)
	at async bootstrap (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:16095:18)
	at async tryActivate (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:16022:24)
	at async activate (/Users/xavier/.vscode-insiders/extensions/matklad.rust-analyzer-0.3.234-nightly/out/src/main.js:15996:5)
	at async Promise.all (index 0)]

Fixed.
Reinstall.

@Veetaha could this be related to the import changes?

@lnicola this is very strange, I do have the same problem when installing the new nightly.
But according to what I see in REPL, everything should work fine.
~/dev/rust-analyzer/editors/code (feat/gzip-binaries) $ node
Welcome to Node.js v14.4.0.
Type ".help" for more information.
> var fetch = require("node-fetch")
undefined
> fetch.default
<ref *1> [Function: fetch] {
  isRedirect: [Function (anonymous)],
  Promise: [Function: Promise],
  default: [Circular *1],
  Headers: [Function: Headers],
  Request: [Function: Request],
  Response: [Function: Response],
  FetchError: [Function: FetchError]
}
> fetch.default()
Promise {
  <rejected> TypeError: Only absolute URLs are supported

Maybe there is some problem with rollup?

But why would reinstalling the extension fix it?

@AurevoirXavier have you switched to stable channel when reinstalling?

@AurevoirXavier have you switched to stable channel when reinstalling?

No, always nightly.


fix typo on Utils.encode_hex deprecation message

encode_hex_short does not exist

lgtm, thanks

I can't merge it, only you have that power


Fixes #58, allows list as query string parameters

Query strings have randomly been a PITA in Alba. Thank you for doing this. I'll have another build (2.2 I think) up by early next week.


[low-power] implement forward tracking series

This PR implements the Forward Tracking Series feature in Link Metrics. Here are a few things to mention in this PR:

Two new apis are added:

otLinkMetricsSendMgmtRequestForwardTrackingSeries, which is used to send an MLE Link Metrics Management Request
otLinkMetricsSendLinkProbe, which is used to send an MLE Link Probe

A new class LinkMetricsSeriesInfo is used to maintain the data of one Series configured by a neighbor. This class inherits LinkedListEntry and each Neighbor has a list of LinkMetricsSeriesInfo. All LinkedListEntrys are allocated and freed by a Pool in LinkMetrics.
To specify SeriesFlags in cli command for Forward Tracking Series, a similar approach with LinkMetricsFlags is used. Another character flags is used to represent SeriesFlags.
Whenever the node receives a frame (including ACK) from a neighbor, it would call Neighbor::AggregateLinkMetrics which goes through each LinkMetricsSeriesInfo entry in the neighbor's list and aggregate the data if the frame type matches the entry.
Two test scripts are added to test this function:

v1_2_LowPower_7_2_01_ForwardTrackingSeries.py. This case in test harness is used to test Forward Tracking Series. So I implemented it in this PR for test purpose. However there's some problems in the test plan.(I proposed in TESTPLAN-485) So I slightly changed the content of the case in this PR. We may not need to be very strict for the content in this PR. I would refine it once this case is discussed by the working group.
v1_2_LowPower_test_forward_tracking_series.py. This case is an extra test as a complement to the above case. It's intended to hit most new code in this PR.

@Irving-cl , size report should be updated to reflect the size changes along with this PR.

I'm observing an invalid value of 0x20 for the Link Metrics Status Sub-TLV after a well formed Link Metrics Management Request.

@edmont
Thanks for finding the problem. Could you provide the steps to reproduce this?

@edmont
I have found the problem: A variable is not initialized before processing: https://github.com/openthread/openthread/commit/6ec22b04b8abeaabe64723a4ccbb1238f3a91974#diff-9efeb98d3167009731304c661cd6f5eb9aab1c6cd027f866f3d13e17c1ac8f94R659  And then in the parsing function, it would return an error. That's not detected on our simulation platform because uninitialized variable have undefined behavior. Thanks very much for finding the issue! Please check again with the latest commit. I have already verified it on dk.


[Candidate 1v1 Drallim Desert v1] China faction has bad access to supply piles

Describe the bug
China faction has bad access to supply piles. Please refer to Relax for more information.
Expected behavior
China faction has proper access to supply piles.

Issue obsolete. Map moved to back log: https://github.com/xezon/GeneralsRankedMaps/commit/6bcc17399631e515e562ee6acec3a3d520a0ff07

Fixed with https://github.com/xezon/GeneralsRankedMaps/commit/19757aa402ad82087d312cf09502d15fecaba13f


Python version used on Archlinux

On Archlinux, the plugin does not run (always gives an error stating that it needs to be recompiled), since the plugin expects the python command to run python2, but arch linux links to python3 instead. This can be fixed by the following config.coffee:
module.exports =
  pythonExecutable:
    type: 'string'
    default: 'python2'
...

but it would be nice if this was automated/mentioned in the README

Thanks for the feedback. I have updated README for that.


Fix bug using Object.keys() in IE9

Object.keys() is supported in IE9, but its behaviour when it receives a String is different compared to Chrom* and FF:
Chrom, FF:*
Object.keys('foo')
["0", "1", "2"]

IE9
Object.keys('foo')
Error

This PR should fix the bug that Philipp Hoschka found in IE.

I don't have IE to test right now but the code looks good.


Is 1.1.2 the correct version for torch-scatter?

Hello. I'm just curious if 1.1.2 is the correct version for the torch-scatter you're using. I'm asking because I notice that you have a from torch_scatter import scatter statement in your message_passing.py module, and as far as I'm aware the scatter wrapper was not introduced until version 2.0+.
I've tried downloading version 1.1.2 and running your code, but I'm getting a
ImportError: cannot import name 'broadcast' from `torch_scatter.utils` (/home/user/anaconda3/envs/userconda/lib/python3.7/site-packages/torch_scatter/utils/__init__.py)

I normally don't like opening Git Issues with these kind of problems, but I was hoping perhaps someone would be kind enough to shed some light on my predicament. Thanks!

Thanks for pointing out the issue. Our original code release was compatible with version 1.1.2 of torch-scatter. A recent pull request updated the scatter calls but missed the changes to the requirements.txt. Please install libraries as per the updated requirements now and reopen the issue if you face further difficulties.


[Sema] Remove obsolete comments

These two comments were added in 2013 and 2012, respectively, most
likely referring to a large patch of code below them. That code was
subsequently deleted in 2014 (0e00f513d).
Remove the obsolete comments, since the performance improvements they
describe can no longer be found in the same file.

@swift-ci Please smoke test and merge

Thanks!


Add image to recent posts for left column | Increase truncate characters

This mod checks if there is a post image and then adds it in the recent posts for the left column

Looks good to me. Merged!
We'll be testing it any further on the default community theme, but it looks good enough to merge already. Thanks!


Update mill-main to 0.12.1

About this PR
📦 Updates com.lihaoyi:mill-main from 0.11.12 to 0.12.1
📜 GitHub Release Notes - Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi", artifactId = "mill-main" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.lihaoyi", artifactId = "mill-main" }
}]

labels: library-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #60.


Upgrade alpine image to 3.15.7

What this PR does / why we need it:
Upgrade alpine version used from v3.15.6 to v3.15.7
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Base alpine image upgraded from `3.15.6` to `3.15.7`

@ishan16696 well, it's not a CVE that's reported. What's reported is just that an outdated OS version is used and a suggestion of the latest patch to be used

/area security


ESPECIALLY IN THE MIDDLE OF THE NIGHT!!

hehe


Add build instructions for x86_64 Linux platform

Issue

Contributes to #620

This PR does not result in breaking changes to upstream dependencies.

Checklist

[ ] This PR contains all the work required to resolve the linked issue.

[X] The work contained has sufficient documentation to describe what it does and how to do it.

[X] The work has sufficient tests and/or testing.

[X] I have committed clear and descriptive commits. Be considerate as somebody else will have to read these.

[ ] I have added the appropriate labels to this PR.

@dcoutts, @erikd, this should now be ready for your review, I squashed all commits into one and it also closes #599.

Thanks @mark-stopka . I have pulled the current version into my branch which is being PRed at #599.

Closing this in favor of #640 .


Translate search command to lua table for windows

Need to test this on windows.
Tested in Powershell and CMD and both seems to work.

@ayooh Thanks but did the branch work otherwise, as being able to run searches in any shell on windows?


1.4.0: Create Map on app-start

Thank you for this great Plugin! I'm using v1.4.0 of your plugin for an Ionic 3 App.
My map is not on the main pages of my app. So the user has to navigate to it. Whenever the user first opens the map (and the googleMaps.create(element); is called the first time) a white fullscreenpage flashes for about 0.5 seconds, which is not hurting anybody, but no good UX.
When the user then goes back and navigate to the page again everything is as it's supposed to be. It's really only when the map is opened for the first time after starting the app.
Hence my question: Is it possible to create the map earlier? Without any markers on it, just initialize the map. For example when the app is started initially? Then the user probably wouldn't even notice this...
By now, I create the app in the ngAfterViewInit() Lifecycle hook on the map page (which is - as said before - none of my main pages).
Thank you very much and best regards.

Since this plugin uses native Google Maps view, it takes a few seconds

Yes, I believe that. But even if I wait on my starting page for 5 minutes and then navigate to the MapPage, I experience this issue.
This is why I'm asking if there is a way to initiate the creation of the map for example from my homePage. If it was created before the user navigates to the MapPage I guess there won't be this "white flashing transition"...

Did you try to create map without div?
Then you can set the map div with map.setDiv().
(If the ionic wrapper is written correctly)

According to the Ionic Documentation, there is a setDiv(domNode) for the ionic wrapper, as well.
But I'm not sure if I fully understand the approach you're suggesting. Creating the map on another page without a div? How can I do this?
Currently on my MapPage I use: this.map = this.googleMaps.create(element); and pass the div (element) to the create function.

this.map = this.googleMaps.create()

That gives me a TS Error:
[ts] Expected 1-2 arguments, but got 0.
(method) GoogleMaps.create(element: string | HTMLElement, options?: any): GoogleMap

In that case, you need to give up.

Why don't you display some message such as loading..., then remove it after the MAP_READY event.

Yes, you're right, I can do that. That at least hides the "flashing page" behind a loading component... Probably our best shot... Thanks.
If you come up with any other idea, please let me know...


Issues with uploading PowerPoint files

I'm not sure if this is a Google issue or an issue with this tool, and I also haven't tested it with any other file formats, but when I save a .pptx file, it syncs to Google Drive as a compressed archive. The workaround that I've found is to drag the .pptx file into the Google Drive web interface, at which point it recognises it as a PowerPoint file. Strange!

How do you save the .pptx? That's because I've copied a .pptx to my Google Drive and it was correctly detected as Power Point (mime type application/vnd.openxmlformats-officedocument.presentationml.presentation). If the program you use to save it, uses a temporary filename without the .pptx extesion, you could experience that issue.


Logging into database

Can we configure it to load the request/response logs into a database.

You can set up a another Sink to send the log to any supported database.
The example uses a appsettings config file to specify the sink(s), you can add more there.


version bump & small Update sparrow-server

What
version bump and note for updating process.
Scope

[ ] significant change to core configuration
[ ] independent bonus guide
[ ] simple bug fix

Closed because the new v2 is ready and updated! Thanks
https://v2.minibolt.info/bonus/bitcoin/sparrow-server


[BACKLOG-13022] Mavenize pentaho-kettle/plugins/aggregate-rows

@smaring aggregate-rows mavenization ready for your review Sir!

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.


trigger workflows on PRs

I believe this should start triggering workflows on PRs with base branch as main.
We can catch the failures and fix it in PR instead of having to fix them separately

https://github.com/facebook/lexical/pull/1750

Workflows have been properly running for internal contributors on PRs. I wonder what accounts for the difference? It would be nice to get to the bottom of it before we make changes, thought I do agree that this should fix it.

Workflows have been properly running for internal contributors on PRs. I wonder what accounts for the difference? It would be nice to get to the bottom of it before we make changes, thought I do agree that this should fix it.

I just realized - it's probably because external contributors are making a PR from a fork, which means they never push a branch to this repo and therefore never trigger the workflows.
Thanks for fixing!

Thanks for the clarification :)

I actually wonder if this will still re-run the CI when internal contributors push to branches with open PRs, or if it will run only when the PR is made.

It will run only when the PR is made 🤔 . We can re trigger workflows when it fails via github actions

It will run only when the PR is made 🤔 . We can re trigger workflows when it fails via github actions

Makes sense - I was thinking about the scenario where the CI passes on one commit, but doesn't re-run on a subsequent commit pushed to the branch before the PR is merged.

Shall we consider using something like this? We use this in reactjs.org site to trigger on commit events, open and re open events.
https://github.com/reactjs/reactjs.org/blob/07dbd86ca421c262157af673a2584a40fd3b2450/.github/workflows/beta_site_lint.yml#L3-L8

Shall we consider using something like this? We use this in reactjs.org site to trigger on commit events, open and re open events.

Yes, "synchronize" sounds like exactly what we need!


Add Test Cases for ListMeeting and Improve Code Quality

Codecov Report
Base: 64.63% // Head: 65.19% // Increases project coverage by +0.55% :tada:

Coverage data is based on head (43f437b) compared to base (ab21519).
Patch coverage: 50.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #326      +/-   ##
============================================
+ Coverage     64.63%   65.19%   +0.55%
- Complexity      672      680       +8
============================================
  Files           112      112
  Lines          2268     2270       +2
  Branches        252      253       +1
============================================
+ Hits           1466     1480      +14
+ Misses          707      693      -14
- Partials         95       97       +2

Impacted Files
Coverage Δ

...in/java/seedu/address/logic/parser/ParserUtil.java
98.18% <ø> (+12.72%)
:arrow_up:

...edu/address/logic/commands/ListMeetingCommand.java
78.57% <25.00%> (-21.43%)
:arrow_down:

...address/logic/parser/ListMeetingCommandParser.java
100.00% <100.00%> (+100.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update security FAQs related to per-service enablement.

These are based on features that already checked in to head, and should be available in v0.3
https://github.com/istio/istio/issues/1232

/assign @wattli @myidpt

/lgtm
/approve

@ldemailly can you approve this? Thanks.


Separate actions into files

we don't want to edit a huge file. huge file causes a lot of conflicts and it makes it tough to review.
after this PR merged, we can start refactoring for actions or we can add new actions easily.

separate Action.h and Action.cc into each Action files.
move constructor implementation to *.cc from *.h
remove pAction. it is used in only Rule.h so we don't need this typedef.
suppress "no virtual destructor" warning in Action class. just I added ~Action(){} because when it is deleted, the runtime will find the vtable for this.

This change is

This is a really nice restructure! I will review this on Friday afternoon / evening.


Add spiderip.py, portcheckers.py, update scanless.py

Added support to :
1.https://spiderip.com
2.http://www.portcheckers.com

Awesome work @Codeshift3r !! Thank you.

Lol, fix your datetime on your system. It looks like those commits were made 2 months ago.


[13996] Failed to execute script 'youtube_music_playlist_downloader' due to unhandled exception!

Hello. I am not knowledgeable with python and in coding in general but I would really love to use your downloader.
Since I posted this I read a bit and managed to install all requirements using Python 3.9.
I also added ffmpeg to PATHs.
After all my troubleshooting, I still have the same exact error:
C:\Users\Shadic\Desktop>youtube_music_playlist_downloader.exe
Traceback (most recent call last):
File "yt_dlp\compat\compat_utils.py", line 36, in getattribute
File "yt_dlp\compat\compat_utils.py", line 56, in getattr
AttributeError: module yt_dlp.dependencies.Cryptodome.Cipher has no attribute AES
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "youtube_music_playlist_downloader.py", line 14, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp_init.py", line 18, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\cookies.py", line 18, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\aes.py", line 5, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\dependencies_init.py", line 76, in
File "yt_dlp\compat\compat_utils.py", line 43, in getattribute
File "yt_dlp\compat\compat_utils.py", line 56, in getattr
AttributeError: module yt_dlp.dependencies.Cryptodome.Cipher has no attribute AES
[13996] Failed to execute script 'youtube_music_playlist_downloader' due to unhandled exception!
I would really love for it to work, can you guys help and know what's going on?
Thank you in advance.

I read a bit on python and was able to install all the requirements from the txt file but I still have the same problem
C:\Users\Shadic\Desktop>youtube_music_playlist_downloader.exe
Traceback (most recent call last):
File "yt_dlp\compat\compat_utils.py", line 36, in getattribute
File "yt_dlp\compat\compat_utils.py", line 56, in getattr
AttributeError: module yt_dlp.dependencies.Cryptodome.Cipher has no attribute AES
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "youtube_music_playlist_downloader.py", line 14, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp_init.py", line 18, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\cookies.py", line 18, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\aes.py", line 5, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\dependencies_init.py", line 76, in
File "yt_dlp\compat\compat_utils.py", line 43, in getattribute
File "yt_dlp\compat\compat_utils.py", line 56, in getattr
AttributeError: module yt_dlp.dependencies.Cryptodome.Cipher has no attribute AES
[14112] Failed to execute script 'youtube_music_playlist_downloader' due to unhandled exception!

Does this fix it?
pip install --uprgade yt-dlp
If not, try running these in succession:
pip install yt-dlp==2023.1.6
pip install yt-dlp==2023.02.17

After all that, it looks like the same error unfortunately. So here what I did since the start, maybe I am just not doing the correct thing because I just started using Python.

Downloaded Python 3.9
Downloaded ffmpeg and added the folder to system PATH
Ran Python 3.9 and opened cmd
Installed the requirements
Added Python, ffmpeg, the executable to my AV Exclusions
Launched youtube_music_playlist_downloader and get those error

Did all with admin elevated

D:\Shadic\Downloads>youtube_music_playlist_downloader.exe
Traceback (most recent call last):
File "yt_dlp\compat\compat_utils.py", line 36, in getattribute
File "yt_dlp\compat\compat_utils.py", line 56, in getattr
AttributeError: module yt_dlp.dependencies.Cryptodome.Cipher has no attribute AES
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "youtube_music_playlist_downloader.py", line 14, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp_init.py", line 18, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\cookies.py", line 18, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\aes.py", line 5, in
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in load_unlocked
File "PyInstaller\loader\pyimod03_importers.py", line 495, in exec_module
File "yt_dlp\dependencies_init.py", line 76, in
File "yt_dlp\compat\compat_utils.py", line 43, in getattribute
File "yt_dlp\compat\compat_utils.py", line 56, in getattr
AttributeError: module yt_dlp.dependencies.Cryptodome.Cipher has no attribute AES
[7172] Failed to execute script 'youtube_music_playlist_downloader' due to unhandled exception!

Are you still using the executable or have you tried using the file in the repo directly?

Ah, I just tried the executable on my end and I'm hitting the issue... I'll try to look into it more but for now avoid using the exe since it seems you have everything installed yourself already

I've also updated the exe and hopefully fixed the issue. There were some changes I needed to make to make it compile yt-dlp properly. If you do decide to use the exe, reminder that it uses its own version of yt-dlp and the requirements and should not need python to run.

So without knowing, all I've done is reproducing what the .exe does by itself... all right. At least I've learned stuff.
It's now working and it's working fine at that, my only concern is it possible for it to download the highest quality possible instead of 124kbps?

Hmm, by default yt-dlp should already hey best quality. You could try setting the audio format to an empty string to try the default or maybe change it to just "bestaudio" but I'm not sure the result will be any different. I think the main reason may be the audio format where I believe opus is the highest quality version YouTube stores from my basic research about it... Perhaps you could set the audio codec to something else to see if you get a higher quality, like opus or vorbis, and then convert it to mp3 manually? I can't experiment with it right now.

It doesn't, it works like Spotify and all files are encrypted. So my guess is that it gets the highest (free) version of the song it can get. Would be nice if I was able to login into my account and it uses the premium quality (if it's really what is going on). Anyway, pitching ideas, I like your project and would like that it gets better.

You can try using ytdlp directly to see if you find any settings that provide better audio. If so do let me know. I'll try to look into it too.

Sorry I'm always editing my comments.

https://github.com/yt-dlp/yt-dlp/issues/1091 It looks like premium audio is indeed a locked feature. You may have to provide your cookies somehow to yt-dlp for it to access the authentication through your account to grab premium audui

From what I understand, the mp3 codec get a low bitrate webm version of the video then converts it to 124kbps mp3... But when I change the codec to flac, I see a webm file for a very short moment before it becomes a flac.. but the flac has really good quality and is arounf 1500kbps, so I guess it takes the highest quality there. The problem there is that it doesn't have the metadata and as a typical flac... really big file.
I am not knowledgable enough in that python thing to make the mp3 320kbps with it's metadata.

Can you provide a link that has that behavior to reproduce it just in case? I'm assuming you got that without YouTube premium cookies

Like I said, looks like I was wrong with the Premium thing, or it takes the cookies with my logged in connexion. I changed the codec to flac in the json settings and it downloads at it's max bitrates (around 1500kbps) and the quality is superb. The problem with that codec is that flac is a huge file and the metadata does not apply.

So would it be possible to download the high quality FLAC then converts it to 320kbps mp3 then it applies the metadata? Like that we would be able to have the best mp3 quality with all it's metadata.

Sorry I haven't had time to take a look yet, will get back when I get a chance

From what I understand, without premium, it should only be offering the quality currently available. When we convert it to other formats, flac/mp3, it still depends on this highest quality option. In this case, I'm inclined to believe the flac file you got is a placebo, and that after converting to mp3 it would be no better than the original mp3. I tried setting the audio format to flac on my end but unfortunately I can't tell a difference versus the normal mp3, and I do not have YouTube premium to compare against an actual confirmed high quality song.
https://www.reddit.com/r/youtubedl/comments/qypms2/are_flac_audio_downloads_truly_flac_quality/

Actually, you can give this a shot. I've updated to include an audio_quality field (adjustable in the config file). By default it is the same as ytdlp's default: 5. If you want the best, set it to 0. Apparently it also supports using custom quality like 128K but I wasn't quite able to verify if that works. You can run with the option Generate default playlist config first, edit the config, then update the playlist to download with the proper settings on first run.

FYI, this update also includes a cookies_from_browser option. You can specify your browser name here such as chrome and it will pull in cookies to use for yt-dlp, such as to get YouTube Premium videos. You can find more info about it here: https://github.com/yt-dlp/yt-dlp

--cookies-from-browser BROWSER[+KEYRING][:PROFILE][::CONTAINER]
The name of the browser to load cookies
from. Currently supported browsers are:
brave, chrome, chromium, edge, firefox,
opera, safari, vivaldi. Optionally, the
KEYRING used for decrypting Chromium cookies
on Linux, the name/path of the PROFILE to
load cookies from, and the CONTAINER name
(if Firefox) ("none" for no container) can
be given with their respective seperators.
By default, all containers of the most
recently accessed profile are used.
Currently supported keyrings are: basictext,
gnomekeyring, kwallet

I’ve been testing it since 2 days now. Using the cookies thing with my Brave browser and it’s been getting automatically. On 90 songs, there’s one that it won’t download but I think that’s a youtube problem.
For the quality, i’ve been using 0 instead of 5 and I can’t find a difference in audio by listening at both youtube music and the downloaded file. Even though it’s not up to 320kbps but the bitrate is variable, I guess there’s no better audio coming from youtube music than that.
Now for the metadata there’s 2 things that’s bothering me a little. There’s no way to turn off the value for the track number so I always have to remove it afterwards. (Youtube music doesn’t have that value for albums anyway I think). The other is that the artwork is kind of low quality. The one on youtube music is higher quality so idk if there’s anything you can do in your code to allow higher quality album covers.
Another things that’s been a little annoying is that it’s always asking if I want to change the settings of the saved playlist. I’d prefer to just select the playlist I want to update the songs and that’s it. It would be nice if you just make another option in the main menu to change the playlist settings for saved playlist instead.
In conclusion. Thank you for the follows up and good job

Thanks for the feedback! For track number you can disable track_num_in_name in the config file for the playlist if you mean the number in the file name. Unless you mean the track number in the mp3 metadata? The reason that originally existed was to maintain the identical ordering from your playlist in YouTube including when updating, but let me know which one you meant. As for a quick update menu option I can look into it. Thanks for the feedback!

I meant the one in the mp3 metadata.

Can you link the video you have thumbnail quality issues with?

I've released a new update with all the changes so far including option to choose png as the image type. If you want to update all your metadata to match your settings, you'll have to pick the Modify previously saved playlist option and choose to force update all names and metadata after adjusting the config. Just in case, you can get the config file to show the new options with proper formatting just by doing a normal update on the playlist.

Closing as assumed resolved


Fund 8 Audit Circle proposal

Fund 8 Audit Circle proposal
https://cardano.ideascale.com/c/idea/399163

[x] Draft
[x] Final Submission
[x] Flag Assessments
[ ] Promote F8 Audit Circle during Voting

Audit Circle Fund 8 Proposal was not funded - https://cardano.ideascale.com/c/idea/399163


Using sqlite3_update_hook

It would be nice to be able to subscribe to events using the sqlite3_update_hook function.
How about adding a setUpdateHook() method to the Database instance ?
I know that this function only fires for updates within the same database connection, but it would still be of use to me (probably).

You can do the same thing (and with more control) by combining triggers and custom functions:
db.prepare('CREATE TABLE data (a, b, c)').run();
db.prepare('INSERT INTO data VALUES (12, 34, 56)').run();

// This is your actual update hook
db.register({ varargs: true }, function logger(...values) {
  console.log(`row updated to (${values.join(', ')})`);
});

db.prepare('CREATE TRIGGER updateHook AFTER UPDATE ON data BEGIN SELECT logger(NEW.a, NEW.b, NEW.c); END').run();

db.prepare('UPDATE data SET a = 78').run();
// => row updated to (78, 34, 56)

Didn't think about it. There is no need for this, then.

😄

Nice! Will this trigger also fire if another connection updates the table?

@wmertens this will only work if the logger() function is registered with the connection that tries to update the table. Otherwise an error will be thrown by SQLite

Aww. I'll just continue polling the DB then 😢. I could probably improve that by using filesystem monitoring and watching the DB file for changes…

Yeah unfortunately that's the only way to read changes made by other connections. Even sqlite3_update_hook() only receives changes made by the same connection

@JoshuaWise No register  function?

The mean function function

The problem with triggers is you'll be called before the transaction commits.
The update hook doesn't call you until the transaction commits and the rows have actually been updated.

@wmertens this will only work if the logger() function is registered with the connection that tries to update the table. Otherwise an error will be thrown by SQLite

Using a temp trigger allows you to create a trigger that invokes functions only installed on the current connection. You still, however, have the problem of not seeing changes made by other connections with this method.

Is there any interest in re-opening this PR? sqlite3_update_hook would be useful in many local first app use cases where reactive updates is a common scenario.
For example, the implementation of useLiveQuery in expo-sqlite uses the update hook:
https://github.com/expo/expo/blob/75172f0339d707a685c60e72e70a7ce4e90c9080/packages/expo-sqlite/ios/SQLiteModule.swift#L433-L457

The trigger approach mentioned in https://github.com/WiseLibs/better-sqlite3/issues/62#issuecomment-325797335 is substantially more flexible and doesn’t have the surprising behavior of sqlite_update_hook (be sure to read all the comments above, there’s good stuff there!)

Triggers are flexible in regards to update individual table update, but how will it handle constantly changing or user created schema, without the need to create trigger on every table?
Also curious to learn what is the surprising behavior of sqlite_update_hook.


docs: use admonition in Testing Asynchronous Code page

Summary
Adds missing admonition in Testing Asynchronous Code page. Tiny change, so I have done all versions in one go.
Test plan
Deploy preview.

@SimenB This one is good now? Or? (;


The Rust crate's version is outdated in https://crates.io/crates/lance

https://crates.io/crates/lance/versions
The latest version is still v0.3.8 which published 27 days ago.

Thanks :)

It seems that the new version 0.4.4 is not published to the crates.io
It will be awesome if we can publish a new version which includes https://github.com/eto-ai/lance/pull/810

Thanks~


Support for Theme Options

Hey ShadeLotus
Is there a way to compile sass files when you include Tumblr meta styles (like {AccentColor} for example) in the sass files? It was an issue I ran into when I tried making my own tumblr template with gulp and sass, and I'm wondering if it's an issue you might know your way around.
Thanks, you did great work here!

I've been injecting the tags in post-meta as inline styles, personally.  Then whatever the Sass builds gets overridden by the inline styles in the header.


Louvain clustering on weighted network?

I'm looking at network based clustering for single cell data, and was wondering about the louvain implementation used here. Is there a reason scanpy doesn't currently allow clustering on a weighted network (like that generated by sc.pp.neighbors(adata, knn=False, method="gauss"))? It's my impression that this would be more similar to the cited Phenograph method.
I'd be interested in adding that functionality (and have a proof of concept on my fork), but wanted to hear if there was a reason this wasn't implemented currently first.
Thanks for any input!

Do you know what are the advantages of using such network in the louvain clusters?

As a quick test, I tried the weighted version of the louvain method and it was able to identify small clusters that are no identified with the non weighted louvain. However, I did not use knn=False as this does not work well with the UMAP representation. Still, I could see differences (eg. cluster 6 and 9 in the top figure):

@fidelram Can you change the resolution of the non-weighted version to reproduce a clustering similar to the weighted case? Weighting can change the scale of the resolution parameter.
I would assume clustering on a weighted, fully connected network would be a lot more computationally expensive. I'm curious if it's worth the additional cost. It may also perform worse as you are putting more emphasis on the euclidean distances between transcriptomes than you may want (are the values more important or only the order of the distances?).

@LuckyMD I changed the resolution of the method and I find that the weighted version produces results that are different. In other words, the difference in the clustering does not seem to come from differences in the resolution parameter.
Here is an example. In both cases, 13 clusters are found; however only the weighted version can identify the small cluster No. 13:

Increasing the resolution of the non-weighted method eventually discerns the small cluster 13
I would vote to add the modifications from @ivirshup

I see no reason why the possibility shouldn't exist to run the weighted version on the full graph. I'm still curious about the quality of the outcome though.
Using protein-protein interaction data, I've noticed that similarity scores perform worse than using network neighbourhoods based on cutoffs to cluster data (this does not have to be the case for scRNA-seq of course). In the latter case you require cells to be each others nearest neighbours to create dense network regions, rather than highly similar transcriptomes based on one calculation of similarity. I would have thought the cutoff approach is more robust to changing similarity metrics as well. It's definitely worth testing this though. Maybe I'm just too skeptical of similarity metrics over all.
@fidelram do you have labels on your data where you could verify the quality of those two partitions?

@LuckyMD The data I am using is from the recent single cell paper about the discovery of the Ionocytes Plasschaert, L. et al.  2018.. Based on the published markers I can label some clusters: the small cluster 11 is the Ionocytes and cluster 10 is labeled a PNEC/Brush.

@fidelram So based on that could you say that the non-weighted method performs better for cluster 10 (PNEC/Brush cluster) as it is identified in this partition, but merged with other cells in cluster 3 in the weighted partition?
I guess it might take a more thorough analysis to make those types of conclusions though.

@LuckyMD I would say that is the other way around:

weighted: identifies small clusters better.
non-weighted: identification of small clusters results in further clustering of larger groups.

However, the differences are not extremely large.

Sorry, I thought you meant the clusters 10 and 11 from the non-weighted version. It would be interesting to see how the labels look for the clusters where the two methods don't agree.

So, I'll go ahead and start a pull request?
Something I think could be useful to include in implementing this is allowing multiple network representations with keyed access (similar to use_rep). This would be useful for the cases where you want to cluster on a network that would be inappropriate to use for UMAP.

@ivirshup I would wait for @falexwolf to get back from his break and see what he says.
As for your second point, I think there are reservations about making the anndata objects too large. But again, @falexwolf will be able to answer that better than me.

Oh, any idea when that is?
As an alternative to allowing more storage, I'd be happy to be able to get a graph returned and later pass it in as an argument, I'm just not sure that fits with the current api. Personally, I feel like it's ultimately up to the user how much they store in an object.

Hi all,
thanks for the nice discussion!
Phenograph is cited not for the specific implementation but for suggesting to use community detection for clustering in single-cell data. This is what the docs state "The Louvain algorithm has been proposed for single-cell analysis by [Levine15]."
My opinion on clustering algorithms: use something that respects the topology of the data (points belonging to one clusters should be connected). Any graph clustering algorithm respects that, even spectral clustering. So, I'm not a big fan of trying 5 clustering algorithms to produce sensible results.   Either a given representation of the data clusters clearly or it doesn't. If it doesn't, Louvain clustering just gives you one possible, representative partitioning of the data. But there are many others that are equally meaningful. Similar for other graph clustering algorithms.
Now, running Louvain clustering on a fully connected graph is prohibitive computationally (memory and CPU time wise).

Intuitively, I'd think having a more complete graph with weighted edges is more representative of the data than an arbitrary k neighbors. Even if you do use a hard cutoff on number of neighbors, I don't see how discounting all distance information would give a more accurate result. I would suspect using a weighted graph could perform better at identifying small subpopulations (where nearest neighbors from other cell types could be common), but that's just conjecture.

That's just speculation to me. I never saw convincing benchmarks. No one claims that "discounting all distance information gives a more accurate result". It's just that it's computationally cheaper.
I acknowledge that a "non-fixed-degree knn graph" varying say, between 5 and 100, would be computationally tractable and would carry information about the sampling density of the data in the  given representation. This information is only indirectly available in the fixed-degree knn graph (more loops etc. in high-density regions). I never investigated this as I never saw fundamental results on such a non-fixed-degree knn graph. As it's also hard to benchmark this, I'd be afraid of getting into this if one doesn't have the time to get the fundamentals right.
I want to note that even in the context of diffusion processes, we managed to obtain meaningful results with kNN graphs in practice. And this clearly contradicts the fundamental results found in all the Coifman papers.
Having said that: if the code is simple, I don't mind at all to have the possibility that you suggest, @ivirshup. Please go ahead with a pull request and I'll see whether the changes are simple enough. The user will still use the default plain knn version, which is also what is done in Seurat.
But my philosophy rests the same: rather than engineering the clustering or any other aspect of the manifold analysis, one should engineer the representation.
Sorry that this got a bit length and confused. I hope I addressed everything but I feel I'm still missing something - I'm currently working through a lot of issues.

Ah, what I missed is the statement about metrics: I know that many people play around with different metrics. But then you mix "representation engineering" (preprocessing or machine learning) with manifold analysis. I'd say the cleanest is to always just use Euclidean distance and all the other work should be done already before.

I'd agree with your statement that engineering the representation is more important than the analysis. I view my goal here as allowing more representations as input.  Would you mind saying more about why you thing using different metics is less clean (simple?)? I would think that would depend on what representation you're calculating the distances on.

I guess we can close with this.
And sorry, I forgot to answer the above:

I view my goal here as allowing more representations as input.

When I say "representation", I mean a feature space representation, which is directly amenable to differentiable mappings, hence optimization and learning. When you say "graph representation" that might be a legit notion, too; and one can definitely think about learning different graph representations (by transforming them back to a vector space). But to me, it appears much more reasonable and straight forward to do all the inference on the feature space representation. And one should do it an a way so that the applied metric used to analyze the arising manifold in the learned representation does make sense.
Of course, if you work directly with raw data, using different metrics can capture a lot of what you'd otherwise need to learn or preprocess (invariance to scales, ...).
Hope this helps a little bit.


Fix for multiple column pickers in a page.

I had a problem with multiple grid on a page, where one column picker would act on another grid. To solve this I added a random value to the id. Alternatively one could use a counter for this to guarantee uniqueness, but SlickGrid uses such random ids as well.

I was under the impression that in "handleHeaderContextMenu" the checkboxes for the different context menus where given identical ids.
I definitely had a problem with this, but can't rule out that it was caused by an error in my setup. When I toggled a checkbox in Firefox the behavior was erratic, sometimes it would work as expected and sometimes it would toggle a column in the other grid. The problem is easy to miss, since it can take a couple of toggle operation in both column pickers until columns from the wrong grid are toggled.
Unfortunately my own example of this problem is embedded in a larger project, but if you can publish your example I can try to compare it and reproduce the error there.

The ids may be the same, but that doesn't matter.  The only place where we "find" elements based on them (i.e. where the elements with the same ids could creep in from the other instance) is in "$menu.find(":checkbox[id^=columnpicker]")" and that call is safe since we scope it to $menu and not search the entire document.
For a test case, I just modified the model example to add another grid and another column picker, but I don't have it handy anymore.
Was is specifically an issue in FF?

(meant to just comment, not close)

Thanks for the quick reply.
Right now I get this error in IE9, but no longer in Firefox 10 or Chrome 17. I'm now using jQuery 1.7.1, so this could also be the reason why I no longer see it in Firefox.
Since duplicate ids are generally not allowed by the HTML standard I thought that this could cause strange errors. It seems that in IE9 it somehow breaks the scoping, maybe because IE is relying on the uniqueness of ids?
It might be a good idea to fix this anyway, to make the generated HTML standards compliant?
If you are not convinced I can try to built a minimal example that reproduces the problem.

There could be a bug in either jQuery or a particular browser's implementation of querySelectorAll.
I suppose I could remove the use of ids in the column picker altogether.

Silly me.  It was the use of "" tag that was interfering.  I never hit on it since I always just click the checkbox.

Right, I didn't notice the difference during my testing, seems like I only clicked the label when I tested in IE9 this time.
Thanks for fixing this properly.


Npm Build issue linked to Uglify-js

Describe the bug
The app runs great, exports pdfs but when I try to run 'npm run build' I get errors that are linked to the Uglify-js plugin that is a dependency of plugins which are dependencies of vue-html2pdf.  So I won't be able to use it in production which is a real shame.
To Reproduce
Steps to reproduce the behavior:

Create app and running with Visual Studio - works fine, exports pdf.
npm run build give error attached:

Unexpected token name «of», expected punc «;» [17.bundle-v=2.0.14-0.js:1184,26]
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>build: `cross-env NODE_ENV=production webpack --progress --hide-modules`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!
Did you use vue-cli or vite?
I tried replicating it on vue 2.6.11. project is generated by vue-cli 4.5.8, I was not having any problems.
I'm guessing you're using vue 3? This package does not support vue 3 yet.

Hi, I found the solution here -
https://forum.vuejs.org/t/error-in-build-js-from-uglifyjs-unexpected-token-punc/33604/6
I had to install the 'uglifyjs-webpack-plugin' and then update the Uglify part of my webpack.config file.
npm i -D<EMAIL_ADDRESS>Then:
Old code:
new webpack.optimize.UglifyJsPlugin({
            compress: {
                warnings: false,
                dead_code: true,
                unused: true,
                drop_console: true,
                drop_debugger: true,
            },
            output: {
                beautify: false,
            }
        })

New code:
const UglifyJsPlugin = require('uglifyjs-webpack-plugin')
....
new UglifyJsPlugin({
            "uglifyOptions":
            {
                compress: {
                    warnings: false,
                    dead_code: true,
                    unused: true,
                    drop_console: true,
                    drop_debugger: true
                },
                output: {
                    beautify: false,
                }
            }
        })```


Performance is very slow.

I used the following script from the root directory (I have an installed instance of tiny-secp256k1 in the folder next to this project)
The Bench results are not good:
13x longer than native JS
281x longer than native C++ bindings
I think the glue code generated by wasm-bindgen is slowing it down. Managing the wasm memory is very expensive.
$ node bench.js
Staring bench for NATIVE 1000 times
Finished 1000 iterations of NATIVE in 51 ms
Staring bench for JS 1000 times
Finished 1000 iterations of JS in 1073 ms
Staring bench for WASM 1000 times
Finished 1000 iterations of WASM in 14356 ms

const { sign } = require('./pkg');
const ecc = require('../tiny-secp256k1/js');
const eccNative = require('../tiny-secp256k1/native');
const crypto = require('crypto');

const ITER = 1e3;

function performBench(_sign, name, iter) {
  console.log(`Staring bench for ${name} ${iter} times`);
  const key = crypto.randomBytes(32);
  const hash = crypto.randomBytes(32);
  const before = Date.now();
  for(let i = 0; i < iter; i++) {
    _sign(hash, key)
  }
  const after = Date.now();
  console.log(`Finished ${iter} iterations of ${name} in ${after - before} ms`);
}

function main(args) {
  const iterations = args[0] || ITER;
  performBench(eccNative.sign, 'NATIVE', iterations);
  performBench(ecc.sign, 'JS', iterations);
  performBench(sign, 'WASM', iterations);
}

main(process.argv.slice(2))

#2

I think so too. The glue code by bindgen is not so fast and flexible. The original glue code also solves #3 .

But there might be some optimization workaround for bindgen.

Resolved in c0fbcac5435653d84952aeb5ca830d481eed107b


studio source code compilation bug

Describe the bug (must be provided)
When I install it, it shows successful login and can receive requests. But the webpage shows no page. What is the situation here?
Your Environments (must be provided)

OS: Linux arm64
Node-version: node --version
Studio-version: 3.3.2（ 3e171a5）

How To Reproduce(must be provided)
Steps to reproduce the behavior:

npm install
npm run build
mv dist server/assets
cd server
vim config/example-config.yaml (modify port)
go build -o server
 ./server( or nohup ./server &)

Expected behavior
I hope I can get the login interface of the studio
Additional context

sorry, step 3 is wrong, please use mv dist/* server/assets


Error: Failed to initialize wallet and storage

Steps to reproduced:

Click the "agree&continue via google" button
Select an account
Pay attention to the error message on the console

Actual result: The "Failed to initialize wallet and storage" error message is displayed on the console and the user is unable to get in G$ account
Expected result: The user should be able to get in G$ account without any errors
Note: sometimes reproduced
Account<EMAIL_ADDRESS> Password: 123Qwe123Qwe
Checked on QA V0.25.6
Desktop: Windows 10 // Google Chrome  83.0.4103.116
Video: https://www.screencast.com/t/hy8MAF0i0Akj

@AnastasiiaOdnoshevna does this still happen?

@sirpy I haven't met that anymore


Document how to generate a left join

from c in db.Customers
join o in db.Orders on c.ID equals o.CustomerID into orders
from o in orders.DefaultIfEmpty()
select new { c, o }

SELECT c.*, o.*
FROM Customers AS c
LEFT JOIN Orders AS o ON c.ID = o.CustomerID


Atom does not detect the language (typescript) if I do not run atom with sudo.

4 days ago everything worked perfectly, and suddenly Atom stopped showing me the colors in the variables of the .ts files
Any suggestions? thanks.
on Ubuntu.
SS Up without sudo, down with sudo

I don't have Kite installed, but I can confirm that the solution from @sirpask fixed the problem for me after an update to 1.45.0 (somehow) caused this bug.

@sli can you confirm if you followed the prerequisites of submitting an issue and trying the solution above?
You can find the prerequisites in the template when reporting a new issue, it might not be the same package but it might still be a community package problem, if you feel it's unrelated it might be worth opening a separate issue with more details to help us reproduce the bug.


Fix page config footer logo welsh

What is the context of this PR?
The welsh footer logo isn't being displayed when using the footer component through page config. This PR fixes that and updates the example to show that.
How to review
Check help users to change language example and see that the welsh ONS logo is now displayed

 Footer not displaying Welsh ONS logo


Game Settings are disabled.

I'm new to this. I'm not sure if this is something that happens with a server but I can't change game settings. Also, are achievements disabled on the Bedrock Server?

I am having trouble also.  I added file ops.txt and admin.txt to the /data directory (with my Xbox username) and that didn't work.
I also tried adding myself to the permissions.json file.
/var/lib/docker/volumes/1d58659676167a387796f294c5b30f3c51ade1604484dbe1aa01236d9bcf199c
I even set the default-player-permission-level in server.properties to operator.  I'm still not operator.

I'm not finding any specific evidence, but observing how creative mode disables achievements I'm thinking Mojang/Microsoft don't want to taint Xbox user achievements due to a possibly hacked server
Server settings can't be changed through the client. On initial world creation settings can be configured via container env vars
Since Mojang's bedrock dedicated server software is still alpha it seems to have bugs about not consulting the properties file after the world has been generated. Instead you will need to docker attach to the server, run the appropriate commands, and detach using Ctrl-p Ctrl-q
All permissions files in bedrock edition use the XUID which is a numerical ID and not the Xbox gamertag. Use something like this to convert a gamertag to an xuid...however, if you use the docker attach approach above you can op "Xbox Gamertag" after the player has joined to convert and persist their permission.


Migrate to Alpine 3.12

Hi Vlad,
Please do let me know if you do not appreciate pull requests for such negligible changes as this.
Kind regards,
Alan

@alderocas  Thank you for your contribution! I appreciate it!

@frol Any time. Happy to help :)


Align API for Mailboxes to how other middleware works

In actors we have ReceiverMiddleware and SenderMiddleware.
In Mailboxes we have MailboxStatistics (which was the very first move towards observability in ProtoActor)
This API is named differently, and structurally different than the middleware.
Should and Could we align these two?
cc @AsynkronIT/developers

Hi, I would like to use protoactor in a lightweight event sourcing client. I could potentially take this on as a task. However, since I am not thoroughly familiar with the project, I would need at least some kind of direction on how it should be implemented. For example would it only have MailboxMiddleware or would there be multiple middle-wares? I know could potentially create a custom queue for this but I feel like filtering out duplicate events would be better as middle-ware.
Reasoning:
I want to avoid orleans because it is definitely not lightweight and over complicated for a front-end client that isn't worried about networking (because it's handled by the azure api). I also have heard many complaints about production implementations with it.
I will most likely be making my own providers for Azure EventHubs, EventGrid, IoT Hub, and Functions (for reading the persisted event hubs data and snapshots as the others are live events only) with multiple actors based on some kind of data partitioning and state hierarchy. Because of this, making sure every actor and provider is idempotent in an at-least-once delivery system would be tedious and repetitive. It would be much easier if I had a mailbox implementation that took care of that itself. It sounds like this could be implemented with middle-ware instead of creating a completely new mailbox implementation.
So um, thoughts? Suggestions?

Idempotency is not something to handle in a middleware. The best way to do it is to keep ids of processed events (a limited number of them) in the actor itself. We do it all the time and it works really well.
I am not sure how do you plan to persist events in EventHub.  After max 7 days, things disappear.
We do have a production system built with Proto.Actor that consumes events from EventHub. However, we used Azure Tables to persist actors, both events and snapshot. That works fine unless you have billions of events per month like we do.
P.S. It might've been a good idea not to go off-topic for the original issue and open a new issue...

I apologize, while I disagree with some of what you said, I did not mean to start an architecture conversion. I still want to take this on so at least extending mailboxes would be clearer, cleaner, and more in line with the rest of the API.
Fyi, EventsHubs can store partition files into Data Lake Storage before clearing via a setting called EventHubs Capture. This can be read much faster than table storage via Stream Analytics but I prefer Functions because it's cheaper with a slight development cost.

I know about EventHub Capture. It shovels all events as Avro files to DataLake, making folders per day.
Event sourcing in Proto.Actor requires the ability to restore a single entity state from events and/or latest snapshot. There's no way it can be done using DataLake.
Clemens Vasters clearly described the aim of each messaging solution in Azure. None of them is considered as persistent storage. EventHub, in particular, serves the purpose of ingestion of events on scale.


Files overwritten when running temporal and spatial analysis in the same output directory

I tend to reuse working & output directories (since I also reuse configuration file) when running temporal and spatial analysis on fMRI.
I have discovered that some output files (mainly nipype graph, mosaic composition and some other files) are identically named in both analysis, thus when running one analysis just after the other in the same destination I got some files overwritten.
Although is not a blocking issue, would be great to have some logic behind to avoid overwriting the existing output files when running both pipelines at once.
Jordi

Hi Jordi,
These issues were addressed in an upcoming release, v1.0.8. However, I will take one more close look to ensure there are no other auxiliary info files like these that are being clobbered. Thank you for the input!

Steve

Good news- the PDF reports no longer get clobbered in 1.0.8.
However, the Nipype graphs still get overwritten with the start of each new group of participants that get run through the pipeline. If I have all of these write out as different filenames to preserve all of them, we can potentially see a very large pool of .dots/.pngs being written especially with large runs where there could be 50+ bundles.
Also, due to the nature of how we build our workflows in QAP, if a lot of participants are included simultaneously, we can see a dependency graph with a large number of parallel workflows all in one graph, which might result in very large png files. Combine this with many individual graphs and it can scale up pretty dramatically, which may not be worth it considering the fact that most of these graphs will be redundant.
However, if all three types of QAP measures are included in one run (anatomical, functional spatial, and functional temporal), the last Nipype graph to be written to (or overwritten into) the .dot/.png file may not include all of these three distinct pipelines. Also, when a pipeline is re-run, and some of the nodes do not run because they are already complete, then the graph changes to only include what happens downstream. Overwriting this into the graph file would remove previous information.
@ccraddock , any thoughts on where to go with this? I was thinking I can put in something that would maybe graph out an example of each type of workflow (as they are available) and stop afterwards.

Great Steven, many thanks!
BTW,when you guys plan to formally release 1.0.8?

Hi @jhuguetn ,
We estimate next week - we will announce it in the support forum when it happens.


Support Astro code blocks in Markdown

Changes
This adds syntax highlighting support for astro fenced codeblocks in all Markdown files.
```astro
---
const title = 'Astro';
---
<title>{title}</title>
```

Testing
This was tested by looking at Astro fenced codeblocks markup in plain Markdown files (i.e. using the default Markdown syntax highlighter built into VSCode).
Docs
bug fix only

lgtm

@matthewp, wonderful! A changeset has been added. With your approval, I will merge.