SklearnClassifier does not support sklearn-like estimators (e.g. ThunderSVM)

Is your feature request related to a problem? Please describe.
Scikitlearn SVMs can be slow on some datasets. GPU accelerated SVMs, such as the one in ThunderSVM, can perform better.
Currently if you pass thundersvm.SVC into SklearnClassifier an error is raised:
  File "...\python\python38\lib\site-packages\art\estimators\classification\scikitlearn.py", line 76, in SklearnClassifier
    raise TypeError("Model is not an sklearn model. Received '%s'" % model.__class__)
TypeError: Model is not an sklearn model. Received '<class 'thundersvm.thundersvmScikit.SVC'>'

Describe the solution you'd like
I'd like SklearnClassifier to support sklearn-like classifiers.
Describe alternatives you've considered
Specific support for thundersvm could be added.

I'm not sure I'm happy with this, but the only simple solution I can think of is:
diff --git a/art/estimators/classification/scikitlearn.py b/art/estimators/classification/scikitlearn.py
index 3c94b5d7..79fa3350 100644
--- a/art/estimators/classification/scikitlearn.py
+++ b/art/estimators/classification/scikitlearn.py
@@ -1086,7 +1086,12 @@ class ScikitlearnSVC(ClassGradientsMixin, LossGradientsMixin, ScikitlearnClassif
         # pylint: disable=E0001
         import sklearn  # lgtm [py/repeated-import]

-        if not isinstance(model, sklearn.svm.SVC) and not isinstance(model, sklearn.svm.LinearSVC):
+        try:
+            from thundersvm import SvmModel as ThunderSvmModel
+        except Exception:
+            class ThunderSvmModel: pass
+
+        if not isinstance(model, sklearn.svm.SVC) and not isinstance(model, sklearn.svm.LinearSVC) and not isinstance(model, ThunderSvmModel):
             raise TypeError(
                 "Model must be of type sklearn.svm.SVC or sklearn.svm.LinearSVC. Found type {}".format(type(model))
             )

I think the 'right' solution would be to use the new typing.Protocol with runtime_checkable to define an interface. However, this is only available in 3.8+.

I'm not sure I'm happy with this, but the only simple solution I can think of is:
diff --git a/art/estimators/classification/scikitlearn.py b/art/estimators/classification/scikitlearn.py
index 3c94b5d7..79fa3350 100644
--- a/art/estimators/classification/scikitlearn.py
+++ b/art/estimators/classification/scikitlearn.py
@@ -1086,7 +1086,12 @@ class ScikitlearnSVC(ClassGradientsMixin, LossGradientsMixin, ScikitlearnClassif
         # pylint: disable=E0001
         import sklearn  # lgtm [py/repeated-import]

-        if not isinstance(model, sklearn.svm.SVC) and not isinstance(model, sklearn.svm.LinearSVC):
+        try:
+            from thundersvm import SvmModel as ThunderSvmModel
+        except Exception:
+            class ThunderSvmModel: pass
+
+        if not isinstance(model, sklearn.svm.SVC) and not isinstance(model, sklearn.svm.LinearSVC) and not isinstance(model, ThunderSvmModel):
             raise TypeError(
                 "Model must be of type sklearn.svm.SVC or sklearn.svm.LinearSVC. Found type {}".format(type(model))
             )

I think the 'right' solution would be to use the new typing.Protocol with runtime_checkable to define an interface. However, this is only available in 3.8+.

Hi @zacps Thank you very much for sharing this idea for new features! I hadn't thought of it yet, but I think it could be useful in many applications to support scikitlearn-like libraries. In case you continue exploring thundersvm with ART, please let us know.

Hi @zacps Thank you very much for sharing this idea for new features! I hadn't thought of it yet, but I think it could be useful in many applications to support scikitlearn-like libraries. In case you continue exploring thundersvm with ART, please let us know.


build container image only when related source change

use dorny paths filter to check file changes.

fixed in c56082860ba4a60e77947123f7dc04c91c271c61


Prevent double code execution for SquiggleChart

In Squiggle Hub, on view model pages, it seems like SquiggleChart runs Squiggle code twice: once on server during SSR, and second time during hydration.

we don't have view model pages anymore
there's no problem on edit pages

Closing for now as stale.


Bug: Can't scan from the second time

I build the MobileScanner on initState.
After the first scan I'm doing setState, but when trying to scan the next time I'm getting this error:
'Unhandled Exception: Bad state: Cannot add new events after calling close"
Why the stream is closed after the first screen, how to keep it open?
The controller is set like this:
MobileScannerController(detectionSpeed: DetectionSpeed.unrestricted);

I also have this same problem. Did you get anything?
I check if the qrcode was read correctly. If it returns status 400, it displays the error message. The camera is still enabled, but it is not possible to read a new qrcode.
final scannerController = MobileScannerController(
formats: [BarcodeFormat.qrCode],
autoStart: true,
detectionSpeed: DetectionSpeed.normal,
);

I'm getting similar but no error message. I'm using a standard barcode. The scanner reads for the first time, then when I stop and restart, it doesn't read any further

Any news about this problem?
This happens to me too! I can't read a qrcode sequentially

I faced that problem too. I was calling the stop() method. I had to remove it

@arok can you please create a pull request for the fix?

I have the same issue

+1


Can't call query() in Electron: childProcess.spawn is not a function

If i run my code in node it works just fine, but i can't call the  query() function if it runs in electron.
Is this a limitation of electron or am i missing something here?
I changed this line in engine.js because in Vue the env variables where not set correctly:
const sysroot = process.env['systemroot'] || process.env['windir'] || "C:\\WINDOWS";

I import and open the connection like this:
const ADODB = require("node-adodb")

const connection = ADODB.open("Driver={Microsoft Access Driver (*.mdb, *.accdb)};Dbq=C:\\Users\\XXXX\\YYYY\\ZZZZ.accdb;Uid=Admin;Pwd=;");

but when i call
console.log(connection.query("SELECT * FROM MSysObjects WHERE Type=1 AND Flags=0"));

it spits out the following error:
log.js?1afd:24 [HMR] Waiting for update signal from WDS...
Home.vue?76f2:16 Promise
spawn.js?d1a4:87 Uncaught (in promise) TypeError: childProcess.spawn is not a function
    at eval (webpack-internal:///./node_modules/node-adodb/lib/spawn.js:87)
    at new Promise (<anonymous>)
    at spawn (webpack-internal:///./node_modules/node-adodb/lib/spawn.js:74)
    at Proxy.exec (webpack-internal:///./node_modules/node-adodb/lib/proxy.js:40)
    at ADODB.query (webpack-internal:///./node_modules/node-adodb/index.js:64)
    at VueComponent.created (webpack-internal:///./node_modules/cache-loader/dist/cjs.js?!./node_modules/babel-loader/lib/index.js!./node_modules/cache-loader/dist/cjs.js?!./node_modules/vue-loader/lib/index.js?!./src/views/Home.vue?vue&type=script&lang=js&:17)
    at invokeWithErrorHandling (webpack-internal:///./node_modules/vue/dist/vue.runtime.esm.js:1862)
    at callHook (webpack-internal:///./node_modules/vue/dist/vue.runtime.esm.js:4229)
    at VueComponent.Vue._init (webpack-internal:///./node_modules/vue/dist/vue.runtime.esm.js:5012)
    at new VueComponent (webpack-internal:///./node_modules/vue/dist/vue.runtime.esm.js:5158)

I just found out that this only happens with
javascript
console.log(connection.query("SELECT * FROM MSysObjects WHERE Type=1 AND Flags=0"));

this query works fine:
javascript
console.log(connection.query("SELECT * FROM Resistors"));


Order of series doesn't match data order

Steps To Reproduce
Sample data: sample-data.csv
SQL:
select run, coalesce(failure, kind, cast(level as varchar)) as category, round(sum(duration) / 1000) as duration
from tasks
where duration > 0
group by run, category
order by run, category

Environment

Node version (node -v): 20.11.0
npm version (npm -v): 10.2.4
OS: macOS 14.2.1 (23C71)
Browser: Chrome 120.0.6099.216 (Official Build) (arm64).
Database: DuckDB

Expected Behavior
The order of categories on the X-axis doesn't match the data order.
17 (1),  17 (2), 17 (3), 21 (1), 21 (2), 21 (3)
Actual Behaviour
21 (3),  21 (2), 21 (1), 17 (3), 17 (2), 17 (1)

Workarounds
I am sure I can find a workaround using the ECharts custom options :) Thanks for using this library.

By default series are ordered by size on the y axis. If you want to disable this behaviour, use
<BarChart
     ...
     sort=false
/>

If you want to disable this behaviour, use

It is false by default, according to docs :)

Also, if the documentation is incorrect and true is the default, I would consider it an unusual default. Metabase uses data order, which is also the expected behavior. In my case, run represents a scenario name (with a suffix such as (N)). Consequently, I expect all my charts to maintain a consistent order that aligns with the data domain.

You are correct that the sort is true by default, and the docs are wrong.

I would consider it an unusual default

Thanks for this feedback, and I do agree - I think we have gone back and forth on this a bit

https://github.com/evidence-dev/evidence/pull/542
https://github.com/evidence-dev/evidence/issues/1134

I think it's slightly difficult to find a default that works in all cases


Cloudwatch integration test

Description of your changes:
Adds an integration test that install Cloudwatch container insights and verifies the fluentbit log groups are available.

Testing:

[x] Unit tests pass
[x] e2e tests pass
Details about new tests (If this PR adds a new feature)
Details about any manual tests performed

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Also we should update documentation for CW integration added in this PR. While thats a high level documentation we can be a bit more specific.
It's important we assume that our users are beginners.
Lets break into it two parts.

Pre-requisite: We can be specific with instruction (preferably command)  saying that you need to add CloudWatchAgentServerPolicy in worker node.
QuickStart: We can also just copy paste the command from this page , which is basically

ClusterName=<my-cluster-name>
RegionName=<my-cluster-region>
FluentBitHttpPort='2020'
FluentBitReadFromHead='Off'
[[ ${FluentBitReadFromHead} = 'On' ]] && FluentBitReadFromTail='Off'|| FluentBitReadFromTail='On'
[[ -z ${FluentBitHttpPort} ]] && FluentBitHttpServer='Off' || FluentBitHttpServer='On'
curl https://raw.githubusercontent.com/aws-samples/amazon-cloudwatch-container-insights/latest/k8s-deployment-manifest-templates/deployment-mode/daemonset/container-insights-monitoring/quickstart/cwagent-fluent-bit-quickstart.yaml | sed 's/{{cluster_name}}/'${ClusterName}'/;s/{{region_name}}/'${RegionName}'/;s/{{http_server_toggle}}/"'${FluentBitHttpServer}'"/;s/{{http_server_port}}/"'${FluentBitHttpPort}'"/;s/{{read_from_head}}/"'${FluentBitReadFromHead}'"/;s/{{read_from_tail}}/"'${FluentBitReadFromTail}'"/' | kubectl apply -f -

I know we will pay price of maintaining it, but as long as our tests are consistent with these instruction, we should be good and we will be able to catch up if something changes.


add LayerGroup

Can it be added LayerGroup，Convenience to layer group operation

@treasureMoment Yes, this sounds like a good idea. Can you provide a bit more detail about what you would like to see?

the objects has the same attributes,only to operation layer group
like leaflet

A more advanced layer grouping concept is being considered as part of #2085


Train use saved checkpoint

Thank you for sharing the excellent code!
How can I train Hivt use saved checkpoint?
example, I want to finetune the model

Hi,
This link may be helpful.


Export "just" HTML code, rather than CSS

Sorry if this has been answered, but I'm a Sublime Text newbie and don't really know how to configure the settings to remove the CSS that's generated when I "Copy to Clipboard" in MarkdownPreview.
Is there a way to do this? I'd like similar output to what I get when I "Export CSS in Sublime Text", only to my clipboard; there, I only get the HTML equivalent of the Markdown I highlighted, rather than the associated CSS.

@facelessuser @revolunet would you accept a pull-request if I were to implement this? I'm also interested in implementing something that allows selecting a custom css file.

You can already use a custom CSS: https://github.com/revolunet/sublimetext-markdown-preview/blob/master/MarkdownPreview.sublime-settings#L150-L158.

@facelessuser hmm okay. I was imagining implementing it as an option in the dropdown menu when you choose the various 'preview/export' options in the command palette.
how do you feel about the 'no css' option?

@facelessuser Looking at this, it doesn't really seem to answer my question. I don't want to specify my own CSS; I want no CSS and "naked" HTML for posting into things like blog CMS.

@MattDemers does the embed_css_for_sublime_output flag in the settings do what you want?

@MattDemers I was addressing @cmyr, that is why it didn't answer your question.
In answer to your question, what about: https://github.com/revolunet/sublimetext-markdown-preview/blob/master/MarkdownPreview.sublime-settings#L219-L226.  Would that option help?

As there has been no response on this for about a year, I'm going to close this. Reopen if there are still questions on this that need to be addressed.


Remove Makefile in favor of npm scripts. See hapijs/contrib#14

Might as well fix licenses while in there as well as remove the root index.js file.

Was it just the year that needed fixing?

I meant the package.json licenses key.

Woops, sorted now. I'd like to be the maintainer too if this is still up for grabs?

@mtharrison start by documenting it...


Release ext reg

Description
required for https://github.com/adobe/aio-cli-plugin-app/pull/417/
depends on https://github.com/adobe/aio-lib-console/pull/22
Before merge, revert these commits

[x] re-enable unit tests (git revert 4d160aa)
[ ]  update dependencies version of @adobe/aio-lib-console to the released version (remove next tag)

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

re-enabled npm test -- all unit tests pass and code coverage is 100%


[Feature Request] support video clip, video editing, video motion effect by movis or moviepy

https://github.com/rezoo/movis
support video clip, video editing, video motion effect by movis

https://github.com/Zulko/moviepy

I don't have the time right now to take these nodes on, but I am definitely open to a PR if anyone wants to give it a go.


How set object array as options instead if normal array

I have checked the example and found that we can set the object as options but not sure how to implement it. Can you please explain step by step how we can achieve it?
A quick reply will be much appreciated.

How I did it was by passing array of an objects in the options and using renderRow function-
`renderRow={(option,index,isSelected)=>{
                                    return(
                                        <View style={{paddingHorizontal:5,paddingVertical:5}}>
                                        <Text style={{fontFamily:theme.textFont,fontSize:14,color:option.value==this.state.selectedPreset.value?theme.textColor:'#cfcfcf'}}>{option.label}</Text>
                                        </View>
                                    )

                            }}`

here "option.label" is my object key.
Sorry if answer was unformatted. I am new here


3d extent of slice

currently assume we are simulating a 2d slice of a 3d volume with unit thickness in 3rd dimension

todo

add option for user to specify size of third dimension
doesn't affect simulation, but does affect automatic determination of compartment volume from geometry
in particular, changing units changing compartment volume, since unit thickness changes with the units

resolved by #582


GitHub is HTTPS by default

This reduces unnecessary redirection from http://... to https://... when someone visits this gem's homepage via https://rubygems.org/gems/rufus-scheduler or some tools or APIs that use the gem's metadata.

Thanks!


tests pass, incorporates new storage

@mdavezac This incorporates the changes of https://github.com/JuliaMatrices/BlockBandedMatrices.jl/pull/12
and tests now pass.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a158e88). Click here to learn what that means.
The diff coverage is 73.46%.

@@           Coverage Diff            @@
##             master     #18   +/-   ##
========================================
  Coverage          ?   48.4%
========================================
  Files             ?       8
  Lines             ?     849
  Branches          ?       0
========================================
  Hits              ?     411
  Misses            ?     438
  Partials          ?       0

Impacted Files
Coverage Δ

src/interfaceimpl.jl
62.5% <100%> (ø)

src/AbstractBlockBandedMatrix.jl
48.38% <100%> (ø)

src/triblockbanded.jl
52.38% <42.85%> (ø)

src/BandedBlockBandedMatrix.jl
60.37% <70.96%> (ø)

src/linalg.jl
32.66% <72.72%> (ø)

src/broadcast.jl
51.97% <81.81%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a158e88...4aae550. Read the comment docs.


Support package version comparison breaks with versions > b9

When determining the support package to use (in briefcase/app.py, function find_support_pkg()), the code looks to S3, finds all matching support packages, reverse sorts them, and then returns the first item.
This is an alphabetical sort, based on filename.
A support package has a name matching the pattern Python-3.7-macOS-support.b1.tar.gz (for Python 3.7, build 1)
Alphabetical sorts will work fine for builds 1-9... but build 10 will appear alphabetically between b1 and b2.
We need to split the filename to extract the build number, and then sort numerically.

I read somewhere i'm supposed to comment if i'm going to try to fix it. Is that accurate? Anyway, i'm looking into it!

I claim this issue.


Working with COM types, such as BSTR and VARIANT should reference the wil library

this is a dupe of #1707 but linked via the doc page for reference.
This section should be updated to not mention ATL or the compiler support for COM.
instead use the windows implementation library (wil) https://github.com/Microsoft/wil
it has wil::unique_variant, wil::unique_bstr for managing resource lifetimes.

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

ID: 5a5b38aa-8df6-251c-6da9-3c2ce888c866
Version Independent ID: 08f6669a-4dc7-ab0d-04b1-7125e809cd60
Content: Consume COM components with C++/WinRT - Windows UWP applications
Content Source: windows-apps-src/cpp-and-winrt-apis/consume-com.md
Product: uwp
Technology: platform
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Thanks, Chris, fix is live now! ~Steve


Neofetch display image

Running with Arch Linux and bspwm
with st, so I have tried anything but it's doesn't work with me

st doesn't support w3m images. While I believe this build is already compatible with ueberzug images, I have no intention of supporting w3m since it's effectively deprecated and inferior. I'm not sure if neofetch supports ueberzug, but you might ask the maintainer of that project.


VueJS Hello World example.

What does the program do ?
It displays Hello World and Hacktoberfest 2018 using a VueJS component.
In what programming language it is written?
VueJS

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Question, why ecma_create_error_reference are needed, as error are also an object.

When I am using iot.js NAPI, the NAPI treat error as object, but in jerryscript,
error is not an object but a error reference to object. That's inconsistence with n-api and
getting iot.js napi doesn't support for external errors.

The simple reason is that the current API's error handling differs from the internal. Internally only 1 pedinding expection is allowed. These execptions are transformed into error_references, which "saves" the thrown error. Therefore you can hold several previous exceptions in the C stack wrapped into error references. However, there are plans to change it for the next release, so stay focused.

The simple reason is that the current API's error handling differs from the internal. Internally only 1 pedinding expection is allowed. These execptions are transformed into error_references, which "saves" the thrown error. Therefore you can hold several previous exceptions in the C stack wrapped into error references. However, there are plans to change it for the next release, so stay focused.

I faced some sorts of memory leak problems when migrating sqlite napi to iotjs. and found once we raise a exception in javascript
and through to the C caller(this caller is an external native envet(file system io)), and the error  would cause memory in JERRY_SYSTEM_MEMORY=ON options(JERRY_SYSTEM_MEMORY=OFF won't raise memory leak). The memory leak detected by multiple tools( valgrind and drmemory), I have no idea how to fixes, because I don't know which object are leaked.

Just a side note, NAPI is not part of JerryScript, it belongs to IoT.js. Beside that's not the right place to discuss I don't really understand what kind of help/answers do you expect. You faced with a problem, okay but whats the testcase, build environtment, expected behaviour, how to reproduce it? However, if you really meant to report it to JerryScript, could you show a reduced expamle of this problem which only uses Jerry API (without IoT.js and NAPI) and reproduces the same memory leak?

Just a side note, NAPI is not part of JerryScript, it belongs to IoT.js. Beside that's not the right place to discuss I don't really understand what kind of help/answers do you expect. You faced with a problem, okay but whats the testcase, build environtment, expected behaviour, how to reproduce it? However, if you really meant to report it to JerryScript, could you show a reduced expamle of this problem which only uses Jerry API (without IoT.js and NAPI) and reproduces the same memory leak?

Hi, thanks for you help, I spend a week to tracing this memory leak problem, and now it's figured out that's a iot.js napi wrapping bug. but also I am facing that the error object are hard to debugging with.
As in Javascript, error can be any value of
undefined/boolean/number/object/string and so on, but in C world, that's another things, at least the node_api wanna setting property on error object. I create a issue here mainly not about tracking the memory leak problem, but what's the design improvement about the Error object. As you said However, there are plans to change it for the next release, so stay focused.
What's the plan you talking about?

Does https://github.com/jerryscript-project/jerryscript/pull/4829 fixed this?


Does not work well with next.js 12 and 13 versions in production build

Hello! When I using next.js in develpoment ENV everything is ok, useWallet working well, I got right statuses.
Let's make that code:
  const wallet = useWallet();
  useEffect(() => {
    console.log(wallet?.status)
  }, [wallet?.status])

Everything is OK.
BUT I caught the bug, when I make next.js production build yarn build thent yarn start. Sometimes wallet status freeze in connecting status.
I thought maybe that some bugs on blockchain. But I try to investigate it, and I thought it bug in the your package.
I understand how to reproduce that bug always:

Take my new repository https://github.com/ansbr/suiet-bugs.  It is fresh next.js app with one commit https://github.com/ansbr/suiet-bugs/commit/125785b8af458c2e06df7458ce6250ee832f4568
Clone it, yarn install -> yarn build -> yarn start or use npm
Open http://localhost:3000
Connect to wallet with your component
Then open developer console, go to Network tab, and choose Fast 3G

Refresh page without reset cache.
You got result, button is blue, status connecting is freezed

If you will try connect one more time, you got error in console

That bug is very annoying for me, I and my team saw it every day (without throttling). Can you fix it?
Thank you.

@ansbr Hi, can you help us to confirm if the issue still exist?

@bruceeewong Hi, I update packages.json
<EMAIL_ADDRESS>"^0.30.0",
    "@suiet/wallet-kit": "0.2.3",

But modal did not see suiet extension

I update suiet extension too, but problem is not gone. Now I have extension version 0.2.0

@ansbr Yeah the wallet kit has upgraded to new wallet standard, but our wallet's new version 0.2.1 is still reviewing. In these two days this issue would be gone naturally.
You can play with Sui Wallet for now.

@bruceeewong As I remember for Sui Wallet everything was ok. That Bug appears only for Suiet Wallet, so I need to wait new version of Suiet Wallet for check it.
Anyway, now Sui Wallet working well too.

@bruceeewong Unfortunately, this bug still remains unresolved. I have "@suiet/wallet-kit": "^0.2.4", and I'm using Suiet Wallet version 0.2.3 in Chrome.

@ansbr things should work now, help us check it again?


Atom freezes when installing packages (and 25% CPU load)

When I'm installing any package with Atom it freezes a short time while it's installing.
Additionally all 5 processes of Atom are causing 25% CPU load.
After the package is installed (so a few seconds) everything is normal again.
Atom 1.0.7
Windows 7

Thanks for the report! This looks like a duplicate of https://github.com/atom/atom/issues/7275 though - closing in favor of that issue.


Remove PseudoUrl in favour of glob patterns

move PseudoUrl implementation to new package in apify-shared-js

add also purlToRegExp(purl: string): RegExp function that will convert string pseudo url to regexp instance

use this package in enqueueLinks (via purlToRegExp function), mark its usage as deprecated and suggets using globs instead
allow using glob patterns (via minimatch) and regexes directly

enqueueLinks({
    globs: [
        'https://example.com/**',
        'https://example.com/?*/?*',
    ],
});

// or

enqueueLinks({
    regexps: [
        /https:\/\/example\.com.*/,
        /https:\/\/example\.com\/.+\/.+/,
    ],
});

// or

enqueueLinks({
    pseudoUrls: [
        'https://example.com/*',
        'https://example.com/*/*',
    ],
}),

In future (v4) we will remove the pseudoUrls usage and allow only explict purlToRegExp via regexps

Partially solved by https://github.com/apify/apify-shared-js/pull/288


Consider tagging releases on github

Hi, packager of http://pkgsrc.se/net/py-gcs-oauth2-boto-plugin here.
It would be easier to follow new releases if they were tagged, with release notes, in Github. Or is there a different recommended way to keep up with upstream releases?

Retroactively added commit tags when I pushed out v2.2.  Marking as closed.


Waybar intersects Sway workspace by a few pixels.

There seems to be a slight overlap between waybar and the sway workspace. I think it's most apparent in the spacing from the top of the screen with the b in genbyte as seen in the screenshots below. I don't know if this is an issue with my configuration, with sway, or with waybar. Here are my config files for sway and waybar at the time of the screenshot.

When I commented out the border in my configuration it did not work but when I tried the configuration in your dotfiles repo it seems to be fixed. Seems to be my configuration and not a bug. Thanks for the quick response :)

Nice to know thx !

I keep this issue in case I find a way to force the height

I have been investigating this, it appears that for whatever reason a default min-height value is set for all css objects at around 22 pixels. I cannot find where, but setting "min-height: 0;" for the wildcard section within the css file fixes it and allows smaller settings for the height.
perhaps a workable solution would be to either auto-shrink this setting or at least throw a warning about graphic defects if the user has conflicting settings

I'm sorry but I don't understand: I still have the overlapping issue.
Isn't it confusing if you have to config the height in two files?
Is it possible to adjust the height of the font (downsizing) in regard of height setting of waybar (which should override?).
It seems like a bug: Setting height to 0 fills the whole screen.
Setting height to 1 leads to applications overlapping to: I guess 1 pixel.
Do I misunderstand the setting? I would like to shrink the waybar, i would like to shrink the spaces above and under the writing, its too bulky for my taste.
Can I help?


Plus sign "+" is not handled properly when passed in the URLs in iOS

Describe the bug
Plus sign "+" is not handled properly when passed in the URLs.
I noticed that when I pass the plus sign in a URL, if it is not escaped, if it is escaped a single time or if it is escaped twice,  the plugin is passing it to the upstream as unescaped: "+"
This is causing problem on  upstream, where it is processed as an empty space.
If I escape the plus sign 3 times, the plugin will pass it over as double escaped: "%252B" which again is not OK as I will read "%2B" on my server side.
Any idea what is going on here?
Thanks
To Reproduce
Pass "%2B", "%252B" and "%25252B" as params in a URL through cordova-plugin-webview-proxy in iOS.
Expected behavior
the proxy should allow passing the "%2B" as a parameter to upstream servers.
Logs
not available

If i remove the following 2 lines from the ios code, and pass in an escaped url, everything works ok
startPath = [startPath stringByRemovingPercentEncoding];
startPath = [startPath stringByAddingPercentEscapesUsingEncoding:NSUTF8StringEncoding];

Something is getting messed up during re-encoding


Utils#playlist: filter out null songs correctly

Previously, the code was .filtering on the array of promises, so because all objects are truthy the filter wasn't actually doing anything. This moves the filter call to after the await Promise.all(...) so it properly filters the Song | null returned from the promises.
This commit also removes some unnecessary type assertions.

You could do filter(Boolean) to remove all falsey

Thank you for the improvement.

@slienlaffa You could, but it works better with TypeScript to use a custom type guard (returning V is Song) so you don't need to do a as Song[] after the filter. I could have made this its own function:
const isNotNull = <T>(x: T): x is Exclude<T, null> => x !== null

declare const arrayOfSongOrNulls: (Song | null)[]
const songs = arrayOfSongOrNulls.filter(isNotNull) // type is Song[]

Oh! thanks for the explanation. I hate typescript now. (Not that I liked to begin with)

Hey, the update is live! Thanks once again.
https://www.npmjs.com/package/discord-music-player/v/8.3.1


Create amount filter

Details

This PR adds an amount filter.
Fixed Issues

$ https://github.com/Expensify/App/issues/46036
PROPOSAL:
Tests

Go to advanced filters tab

Try setting up amount filter and make sure the inputs work correctly

Save and view results and verify that values are filtered by amount.

Offline tests

QA Steps

Same as tests
PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/30def578-47e7-499a-bcd3-7ec78234f41f

Android: mWeb Chrome
https://github.com/user-attachments/assets/dc9947eb-f0bf-4e32-a61b-8b2dd7ad7b97

iOS: Native
https://github.com/user-attachments/assets/d7f849a6-ab81-4155-8d85-dbb6b192336c

iOS: mWeb Safari
https://github.com/user-attachments/assets/bec8126a-f686-4677-8117-e18dea3ad331

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/311d0ced-fe21-41e2-8c91-3b0c91961ccb

MacOS: Desktop
https://github.com/user-attachments/assets/60abebc7-4ffc-4108-87b9-c4266f81ebad

@luacmartins I noticed that when we set url with param amount>2000 the BE treats 2000 as if this value was 20.00.
I know that this approach was is widely used on BE, but I think that in this case we should treat 2000 as 2000.00.
We don't specify currency and there are some currencies with no digits after decimal point(like JPY), so when somebody would want to filter out 20 JPY he would get values under 0.20 JPY and this doesn't make sense.
Can we change it on BE side 🙏 ?

I'll OOO the whole next week but I asked @289Adam289 to take care of merging this one 😄

This is looking good to me. Could we see an example of where we have just Greater than filled in and one with just Less than filled in?

@rayane-djouah is that bug specific to this filter or does it happen in other text inputs?

@SzymczakJ could you please take a look at https://github.com/Expensify/App/pull/47343#issuecomment-2291467068?

@SzymczakJ is OOO this week. I am currently working on a fix.

Looking great from a visual perspective too

@luacmartins This bug appears to be bigger than I originally thought. It is connected to these issues: https://github.com/facebook/react-native/issues/38155, https://github.com/facebook/react-native/issues/24585, and how the native TextInput component works. A similar problem has already been encountered in the Expensify App https://github.com/Expensify/App/issues/41961. I propose we deal with this in a follow-up issue.

That sounds good to me. @rayane-djouah wanna do a final review? We'll address the input issue in a follow up

bump @rayane-djouah for final review 🙇♂

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/luacmartins in version: 9.0.24-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/luacmartins in version: 9.0.24-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/marcaaron in version: 9.0.24-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Prepare TypeScript interfaces for API.Category

Context
Acceptance criteria

TypeScript interface for GET /sales-channel-api/v1/category endpoint is created.
All the fields from GET /sales-channel-api/v1/category endpoint are included.
TypeScript interface for /sales-channel-api/v1/category/{categoryID} endpoint is created.
All the fields from GET /sales-channel-api/v1/category{categoryID} endpoint are included.

Scope moved to #43 issue.


Using Fabric with something other than the latest version of MC?

I'm trying to use TYPE=FABRIC (recently added by @crazymanjinn, thanks!).
Unfortunately specifying:
TYPE=FABRIC
VERSION=1.14.2

Doesn't work. It's still trying to use MC 1.14.4 for some reason. These are snippets of the container startup logs:
Installing Fabric <IP_ADDRESS> using /tmp/fabric-installer-<IP_ADDRESS>.jar
Loading Fabric Installer: <IP_ADDRESS>
Using latest loader version
Using latest game version
...

[05:56:29] [main/INFO]: Loading for game Minecraft 1.14.4

Since my /data volume and world already exist for MC 1.14.2 this is causing startup to crash.
Is this line using the wrong value for -version or something?: https://github.com/itzg/docker-minecraft-server/blob/539f7ada706724fb38becb7d527ac73ae198ed13/start-deployFabric#L53

FWIW the flag is actually documented as mcversion and not version.

Yeah I think that was the problem. Manually installing with -mcversion fixed it for me.

Fixed by #352


GH-3523 Cache mechanism for MemStatementIterator(s)

GitHub issue resolved: #3523
Briefly describe the changes proposed in this PR:

functionality to measure when an MemStatementIterator should be cached
cache for MemStatementIterator(s)

PR Author Checklist (see the contributor guidelines for more details):

[ ] my pull request is self-contained
[ ] I've added tests for the changes I made
[ ] I've applied code formatting (you can use mvn process-resources to format from the command line)
[ ] I've squashed my commits where necessary
[ ] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

After
Benchmark                            Mode  Cnt   Score   Error  Units
QueryBenchmark.complexQuery          avgt    5   6.617 ± 0.020  ms/op
QueryBenchmark.groupByQuery          avgt    5   2.026 ± 0.016  ms/op
QueryBenchmark.pathExpressionQuery1  avgt    5  25.105 ± 0.418  ms/op
QueryBenchmark.pathExpressionQuery2  avgt    5   1.443 ± 0.060  ms/op

Also need to benchmark what cost this has for write operations.

What happens if to queries run in parallel and access the same iterator?

It's not the actual iterator that is cached, but an ArrayList with the results from the iterator. So if two threads both hit the cache then they will each get a new iterator backed by the list.

Before
Benchmark                                                     Mode  Cnt      Score     Error  Units
QueryBenchmark.complexQuery                                   avgt    5     47.596 ±   0.488  ms/op
QueryBenchmark.different_datasets_with_similar_distributions  avgt    5      3.232 ±   0.090  ms/op
QueryBenchmark.groupByQuery                                   avgt    5      1.983 ±   0.072  ms/op
QueryBenchmark.long_chain                                     avgt    5   1055.210 ±  32.642  ms/op
QueryBenchmark.lots_of_optional                               avgt    5    271.959 ±   8.758  ms/op
QueryBenchmark.minus                                          avgt    5  12103.994 ± 100.582  ms/op
QueryBenchmark.nested_optionals                               avgt    5    262.800 ±   5.527  ms/op
QueryBenchmark.pathExpressionQuery1                           avgt    5     23.529 ±   0.894  ms/op
QueryBenchmark.pathExpressionQuery2                           avgt    5      1.596 ±   0.102  ms/op
QueryBenchmark.query_distinct_predicates                      avgt    5    194.093 ±   9.729  ms/op
QueryBenchmark.simple_filter_not                              avgt    5     10.962 ±   0.539  ms/op

After
Benchmark                                                     Mode  Cnt      Score     Error  Units
QueryBenchmark.complexQuery                                   avgt    5      6.758 ±   0.084  ms/op
QueryBenchmark.different_datasets_with_similar_distributions  avgt    5      3.324 ±   0.128  ms/op
QueryBenchmark.groupByQuery                                   avgt    5      2.056 ±   0.077  ms/op
QueryBenchmark.long_chain                                     avgt    5   1044.142 ±  31.316  ms/op
QueryBenchmark.lots_of_optional                               avgt    5    282.401 ±   7.528  ms/op
QueryBenchmark.minus                                          avgt    5  12203.275 ± 410.164  ms/op
QueryBenchmark.nested_optionals                               avgt    5    268.526 ±   6.198  ms/op
QueryBenchmark.pathExpressionQuery1                           avgt    5     23.477 ±   0.752  ms/op
QueryBenchmark.pathExpressionQuery2                           avgt    5      1.553 ±   0.123  ms/op
QueryBenchmark.query_distinct_predicates                      avgt    5    169.904 ±   1.297  ms/op
QueryBenchmark.simple_filter_not                              avgt    5     10.334 ±   0.221  ms/op

So this doesn't really benefit the other queries much at the moment.

I'll go over the naming a bit. Maybe something like CacheAwareIteration would be more informative. Also need to check how much this impacts data insertion.

@jeenbroekstra Could you take another look? I've moved around some of the code and renamed things to make it all a bit more obvious what the code is supposed to do.


Save models from xgb.cv

#793 Please note that this change have not been tested. I could not install xgboost from the most recent repository due to some compilation errors - missing logging.h.

try this to update your repository
git submodule init
git submodule update

Hi! Thanks for kind words. I can't figure out what is wrong. I did
git submodule init git submodule update
before the pull request but after commit. and then again after I got the appveyor error message/.
How do I fix it now?

Do you compile on your computer now ?
Have you seen that:
https://github.com/dmlc/xgboost/blob/5d9b80cd8bef05e6f3fd7d4b7624d9a17c255978/doc/build.md#trouble-shooting

regarding AppVeyor, I don't think it is related to you.

Ah, sorry I thought you were talking about AppVeyor issue, the compilation issue is different - I am getting the exact same message as
https://github.com/dmlc/xgboost/issues/797
I don't think it could be related to submodules. I am getting it when installing from github from my repo and as well from dmlc.

Hi Guys,
I am fairly new to the process of contributing to XGB, or to any other SW to be honest.
I fail to re-install xgb from my own git to test my change, due to some problems with compilation.
Is there any other way to test?
Otherwise, may be someone would be so kind and test my change?
With respect, Andrey

@terrytangyuan can you look into this change and see if it can be merged? The Travis tests already passed

@vykhand sorry for the slow action on this. Can you also add an example on how to use this feature, as well as a testcase?
Thanks

Hi @tqchen,
Something like this should work :
`library(xgboost)
data(agaricus.train, package='xgboost')
dtrain <- xgb.DMatrix(agaricus.train$data, label = agaricus.train$label)
history <- xgb.cv(data = dtrain, nround=3, nthread = 2, nfold = 5, metrics=list("rmse","auc"),
max.depth =3, eta = 1, objective = "binary:logistic", prediction = TRUE)
#in structure, there must be a list named "models" with 5 elements, model1 .. model5
str(history)
#test that we can save all 5 models for later reuse
models <-  history$models
for (i in 1:length(models) ) xgb.save(models[[i]],paste0("model",i,".xgb"))
`

@vykhand  Thanks. Maybe let users specify how many top models to save? They probably don't want to save all of them.

Hi @tqchen,
You are right, I was thinking more about the separate parameter called save_models with default to 0, and if is other than 0, the xgb.cv would save x top models based on the metric.
I did it this way because it was the simplest/quickest way. I would probably do it in a more complicated way, but the stopper is that I can't compile from my own git (and from this git also). I get the missing logging.h error above. I just tested it again after fetching new changes - no luck. It appears on 3 machines - my windows laptop, centos VM and ubuntu VM.

Hi @tqchen, I managed to check this change and sadly it is not working. The models list is not a list of xgb.Boosters, but instead is a list of Booster.handles. As I am now able to test this, I will try to figure out a better way.

closing this for now, see https://github.com/dmlc/xgboost/issues/892 for new guideline in enhancing training API


format indentation for Results sections

Overview of the Pull Request:

Created component for content sections
Created component for content subsections with horizontal padding per design specifications
Refactored existing content sections to nest under new content sections and subsections
Created header components for each subsection
Refactored Introduction section to fit into standard subsection components
Created JSON file with design specs so all components can refer to a single source of truth.

link to Trello card or Github issue
How Has This Been Tested?

Please describe the tests that you ran to verify your changes.
Provide instructions so we can reproduce.
If necessary, please also list any relevant details for your test setup/configuration

The Results page content sections should have indentations on the right as shown:

Dotted lines serve as a guide only, and have been removed from submitted code
Page has not been optimised for smaller screen sizes in this PR
Keeping commented code in for possible use in subsequent PRs

Pull Request Checklist:
Make sure the following checkboxes[ ] are checked with x before sending your PR -- thank you!

[ ] Have you included a link Trello card / Github issue and written sufficient description to help others review your work?
[X] Is your pull request up-to-date with main branch?
[X] Have you checked That code is well formatted? Did npx prettier --check . return no warnings/errors?
[ ] Have you written instructions on how to test that your changes work as expected?
[ ] Have you tested your work thoroughly on your local machine to ensure you are not breaking anything?

Hi @RCode-Blue 👋🏽
As per our conversation this afternoon with @OlaShabalina regarding this PR, we won't be merging this PR before next week's release.
We will organise a workshop with you and the dev team after the release, so that we can walkthrough and carefully review the changes in this PR with you.
Thx so much for your efforts! I'm looking forward to the workshop 🙇🏽
We will leave this PR open till then, but I'll convert it to a draft PR in the meantime 👍🏽


Compilation errors: bad register name

Hi,
I am attempting to build openblas with spack. I have tried two different compilers gcc8.2 and gcc7.3 and each time have ended up with errors concerning bad register name . For example,
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S: Assembler messages:
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm0'
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm2'
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm3'
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm2'
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm3'
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm7'
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm9'
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm7'
../kernel/x86_64/sgemm_kernel_16x4_skylakex.S:1311: Error: bad register name `%zmm9'

The detailed errors are in error.txt.
I would greatly appreciate advice on how to deal with them.
Thanks,
Manav

To me this looks as if your assembler may be too old, or perhaps you need to set up AS to point to the spack version of gas like you did with CC ?

You need binutils 2.25 or better and gcc 5 or better
$ as --version
GNU assembler (GNU Binutils; openSUSE Leap 15.0) *2.31*.1.20180828-lp150.5.3
$ gcc -march=skylake-avx512 --version
gcc (SUSE Linux) *7*.3.1 20180323 [gcc-7-branch revision 258812]

@manavbhatia probably it is worth bringing this issue to spack developers that gcc is not functional with archaic host binutils.
@martin-frbg i think I adressed at least popular part of particular problem (Ubuntu14 has old compiler but fresh binutils):
https://github.com/xianyi/OpenBLAS/wiki/Faq#binutils

@brada4 I do not think this is a problem with spack in particular - just that sometimes updating one package is not sufficient to receive some particular new functionality. I think this assembler problem came up before, probably when the Haswell kernels brought AVX2 mnemonics (and one could not say that rpm or apt-get were at fault back then either)

Old systems do not have right tools to do the job. I wonder how this slipped past initial AVX512 compiler detection which invokes as before proceeding.


Implement TokenScript signature verification locally instead of using web API

Replacing #1308

Duplicate of https://github.com/AlphaWallet/alpha-wallet-ios/issues/1612


options to change chalk color?

I am using Iterm 3 with default theme.
the chalk.white makes the error message invisible.

#50

Option for dark theme - #50
Related to flow-reporter Support for dark/bright color themes


spec: activation bars for sequence diagram

Purpose for this PR

Sequence diagrams usually have activation bars to denote the lifespan of activities (operations), this change adds them using the activate & deactivate PlantUML keywords.
The other option is to use ++ and -- shorthand, which would have a different affect on the reading of the code (also with that shorthand, it only applies to the receiver of the function, making it difficult to get an activation bar on Alice).
Let me know what you think (I suspect the extra lines reduce readability 🤔)

activations reaching into the optional box, which makes it look like snapshot is activated through the pass and into the fail conditionals

That's down to the snapshot actor not being deactivated after it's call.
There is a rendering issue (imprecision) with PlantUML and the activation / termination of activities. You can't reliably re-start an activity bar in the sub-option and the beginning of it's sub-box.
There are three option

An activity bar running through the options, terminating on the last call at the bottom else clause.
Wonk start positions for every following else option.
No activity bar when you enter into the group element.

Which of those options look nicer? ...either option one or three (don't mind which)

some activations extend past an arrow which makes it look like the object continues to do something

Interestingly the ordering of activate deactivate was causing those visual artefacts, fixed.

The fixes look good.
suggestion: I prefer option 3.

As there's no other preferences state @shahankhatch option 3 it is 😁


Azure pipelines

Per #7 I get the pipeline up and running. The only thing to to now is add the tests.

Thanks a lot!


Rationale for prod server API gateway

@ltetrel I did my best to list the reasons why having an API gateway on the prod server can be of value after discussing with @emdupre.
1.1 Serving books
Generic: For quite some time I was under the impression that the final book artifacts were going to be served through GitHub pages from a centralized repository and persist there. We were aware that this solution was suboptimal because the repo would get larger and larger over time and we would not have the all bundle of resources in one place that (physically) belongs to NeuroLibre.
Feasibility checkbox:

[x] Serve static html from prod server

1.2 Cutting versions & Crossref
Specific: Upon acceptance, we'll need to cut version for all the components of the submission using Zenodo API. We cannot do this on Heroku workers (highly limited memory) and from what @ltetrel said regarding file transfers, this would be heavy on the test server as well.
The PDF of the submission lives on GitHub, although heroku workers would pull off crossref for a single PDF, to mint and collect DOIs (metadata) for other artifacts we need something more powerful.
Feasibility checkbox:

[x] API functionality to use Zenodo API for NeuroLibre artifacts

1.3 Moving review workload to prod  server
This way:

We have the artifacts already available on the prod
Advantage for data management given that that's where we have the storage
We can expect higher performance and have a feeling of how the final product will behave when published

Feasibility checkbox:

[x] API functionality similar to that on the test server for Binder & Book build

1.4 Providing the version-cut repo for BinderHub
Following a complete build, I thought that BinderHub would use repository information (user/repo+commit) to find the right Docker image to spawn the right user instance, but not to consume what exists on the repo for that specific (user/repo+commit, assume that there are no changes, which will be the case upon publication). Is this correct?
In any case, to build a Docker image on prod server, we need that repo to live under neurolibre organization for the time being. Questions regarding this:

Is there a way to grant build access to a repo on the fly?
Is there a more customized way for BinderHub to consume repository files from prod server's filesystem?

Appreciation
I am aware that @ltetrel is dealing with a lot of backend components on 2 different cloud infrastructures. We spent months together getting ourselves familiar with all those things under the hood, and I know at first hand that these things are not necessarily a piece of cake. I just wanted to acknowledge this not to give you the impression that we are unreasonably requesting a lot of things from you without knowing what it takes :)
Validation
I hope that it was clear. I would like to wait for sanity check by @emdupre before we proceed to discussion.

Can we consider this resolved since the API has been added to the production server, or should we keep it open until the production API has been integrated into the workflow ?

+1 for closing this and open other issues for the development ✌


Fix minor editorial issues in VTKFileFormats.md.

Did a bit of careful reading. Also added in a link to Figure 1.
Figure 4 looks really nice, I also like the link to the source code.

Thanks!


Integration of OCI delivery in gardenlinux build pipelines

Integrate oci delivery gear into Garden Linux build pipelines.
Requirements:

gardenlinux/python-gardenlinux-lib#26
gardenlinux/python-gardenlinux-lib#16

Tasks:

[ ] Check and set defaults for gardenlinux/gardenlinux use case (path to pub/priv keys)
[ ] Deploy gl-oci in pipelines (check what method is best)

Can you please align if we can have in the final version the "standard" gardenlinux manifest so we can do like podman/docker run/pull but also pull the relevant layers in the same label?
CC @nkraetzschmar


the future of cocos2d-js

i noticed that cocos2d-js will be merged back to cocos2d-x. in list 4.0.
what dose it mean?
the cocos2d-js will be removed?
see the Merge back cocos2d-js into cocos2d-x. in list 4.0 (2015 apr) from
https://trello.com/c/17tdK3TN/144-merge-back-cocos2d-js-into-cocos2d-x
@pandamicro

Cocos2d-JS won't be removed, it will be merged into Cocos2d-x in v4.0. But we will keep the current workflow, API design, etc. If you only use JavaScript, you can see it as a simple folder structure rearrangement.
The goal to merge Cocos2d-JS into Cocos2d-x is to make the maintenance easier.
What I want to ensure you is :

Cocos2d-JS brand will continue to exist.
The workflow will remain the same as v3 (for all platforms, including web).
JavaScript style API will get improved in v4.
We won't abandon things we achieved in v3.

thanks a lot


Change watchDelay default value

Fixed multiple calls applyPlugins("done", stats) by changing a default value of watchDelay to 1. May be need to change watchDelay to 200 as said in the documentation  (http://webpack.github.io/docs/configuration.html#watchdelay).

watchDelay was replaced with watchOptions.


[core-http] Reintroduce ServiceClientCredentials as an accepted credential type

This change reintroduces the ServiceClientCredentials interface as an accepted credential type for ServiceClient classes.  This is necessary because libraries like @azure/app-configuration and the various Cognitive Services libraries need to use their own custom (non-AAD) authentication schemes.  Without this change, users of those libraries would have to construct their own RequestPolicy pipeline just so that the signingPolicy can be used.
Since this affects a bunch of libraries I went ahead and bumped the core dependency versions of our various libraries so that everything is up to date (@chradek, this will affect your tracing PR).

Also added a fix that removes some unneeded methods in core-http's AbortSignalLike interface so that it doesn't conflict with @azure/abort-controller's interface.


Error on setup

First of all, thanks so much for sharing your work!
I tried to setup the plugin via the default Packer snippet:
use {
  'boltlessengineer/bufterm.nvim',
  config = function()
    require('bufterm').setup()
  end,
}

Installation was successful, but I get a Packer error on setup:
packer.nvim: Error running config for bufterm.nvim: vim/shared.lua:0: after the second argument: expected table, got nil

@bronzehedwick, can you use this?
use {
  'boltlessengineer/bufterm.nvim',
  config = function()
    require('bufterm').setup({})
  end,
}

Thanks for opening the issue. Now fixed. setup() can be called with no argument.

Downloaded the update, and this is fixed. Thanks!


160 character short description is not always enforced

The 160 character limit is not enforced on some stores. Below is the short_description lengths for all the   stores that Duo Search has on record. As can be seen, some stores have short_descriptions greater than 160 characters.

[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 2, 2, 2, 2, 2, 3, 3, 3, 3, 3, 3, 3, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 5, 5, 5, 5, 5, 5, 5, 5, 5, 6, 6, 6, 6, 6, 6, 6, 7, 7, 7, 7, 8, 8, 8, 8, 8, 8, 8, 8, 8, 8, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 9, 10, 10, 10, 10, 10, 10, 10, 10, 10, 10, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 11, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 12, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 13, 14, 14, 14, 14, 14, 14, 14, 14, 14, 14, 14, 14, 14, 15, 15, 15, 15, 15, 15, 15, 15, 15, 15, 15, 15, 15, 15, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 16, 17, 17, 17, 17, 17, 17, 17, 17, 17, 17, 17, 17, 17, 17, 17, 18, 18, 18, 18, 18, 18, 18, 18, 18, 18, 18, 18, 18, 18, 18, 18, 18, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 19, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 21, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 22, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 23, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 25, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 26, 27, 27, 27, 27, 27, 27, 27, 27, 27, 27, 27, 27, 27, 27, 27, 27, 28, 28, 28, 28, 28, 28, 28, 28, 28, 28, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 29, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 30, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 31, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 32, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 33, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 34, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 35, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 36, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 37, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 38, 39, 39, 39, 39, 39, 39, 39, 39, 39, 39, 39, 39, 39, 39, 39, 40, 40, 40, 40, 40, 40, 40, 40, 40, 40, 40, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 41, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 42, 43, 43, 43, 43, 43, 43, 43, 43, 43, 43, 43, 43, 43, 43, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 44, 45, 45, 45, 45, 45, 45, 45, 45, 45, 45, 45, 45, 45, 45, 45, 46, 46, 46, 46, 46, 46, 46, 46, 46, 46, 46, 46, 46, 46, 46, 46, 47, 47, 47, 47, 47, 47, 47, 47, 47, 47, 47, 47, 48, 48, 48, 48, 48, 48, 48, 48, 48, 48, 48, 48, 48, 49, 49, 49, 49, 49, 49, 49, 49, 49, 49, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 51, 51, 51, 51, 51, 51, 51, 51, 51, 51, 51, 51, 51, 51, 51, 51, 52, 52, 52, 52, 52, 52, 52, 52, 52, 52, 52, 52, 52, 53, 53, 53, 53, 53, 53, 53, 53, 53, 53, 53, 53, 54, 54, 54, 54, 54, 54, 54, 54, 54, 55, 55, 55, 55, 55, 55, 55, 55, 55, 55, 55, 55, 56, 56, 56, 56, 56, 56, 56, 56, 56, 56, 56, 56, 56, 56, 56, 56, 56, 57, 57, 57, 57, 57, 57, 57, 57, 57, 57, 57, 57, 58, 58, 58, 58, 58, 59, 59, 59, 59, 59, 59, 59, 60, 60, 60, 60, 60, 60, 60, 60, 60, 60, 60, 60, 61, 61, 61, 61, 61, 61, 61, 61, 62, 62, 62, 62, 62, 62, 62, 62, 62, 63, 63, 63, 63, 63, 63, 64, 64, 64, 64, 64, 64, 64, 64, 64, 64, 65, 65, 65, 65, 65, 65, 65, 66, 66, 66, 66, 66, 66, 66, 66, 66, 66, 67, 67, 67, 67, 67, 67, 67, 67, 67, 68, 68, 68, 68, 68, 68, 68, 68, 69, 69, 69, 69, 69, 69, 70, 70, 70, 70, 70, 70, 70, 71, 71, 71, 71, 71, 71, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 73, 73, 73, 73, 73, 73, 73, 73, 73, 73, 73, 74, 74, 74, 74, 74, 74, 74, 74, 75, 75, 75, 75, 75, 75, 75, 75, 76, 76, 76, 76, 76, 76, 76, 76, 76, 77, 77, 77, 77, 77, 77, 77, 77, 77, 77, 77, 78, 78, 78, 78, 78, 78, 79, 80, 80, 80, 80, 80, 80, 80, 80, 80, 80, 81, 81, 81, 81, 81, 81, 81, 82, 82, 82, 82, 82, 82, 82, 82, 82, 82, 82, 83, 83, 83, 83, 83, 84, 84, 84, 84, 85, 85, 85, 85, 85, 85, 85, 86, 86, 86, 86, 86, 86, 86, 87, 87, 87, 87, 87, 87, 88, 88, 88, 88, 88, 88, 88, 88, 88, 88, 88, 89, 89, 89, 89, 89, 89, 89, 90, 90, 90, 90, 90, 90, 90, 91, 91, 91, 91, 91, 91, 91, 91, 91, 92, 92, 92, 92, 92, 93, 93, 93, 93, 93, 93, 94, 94, 94, 94, 95, 95, 95, 95, 95, 95, 95, 95, 96, 96, 96, 96, 96, 96, 97, 97, 97, 97, 97, 98, 98, 98, 99, 99, 99, 99, 99, 99, 99, 99, 100, 100, 100, 101, 101, 101, 101, 102, 102, 102, 102, 103, 103, 103, 103, 103, 104, 105, 105, 105, 105, 106, 106, 106, 107, 107, 107, 107, 107, 108, 108, 110, 110, 110, 110, 110, 111, 111, 111, 111, 113, 113, 113, 113, 113, 113, 114, 114, 114, 114, 115, 116, 117, 117, 117, 117, 118, 119, 119, 119, 119, 120, 120, 120, 120, 121, 121, 122, 122, 122, 122, 122, 122, 122, 122, 122, 123, 123, 123, 124, 124, 124, 125, 125, 125, 126, 126, 126, 126, 126, 126, 127, 127, 127, 128, 128, 128, 128, 128, 129, 129, 129, 129, 130, 130, 130, 130, 130, 131, 131, 131, 132, 132, 132, 133, 133, 133, 134, 134, 134, 134, 134, 135, 135, 135, 137, 137, 137, 137, 138, 139, 139, 139, 140, 140, 141, 142, 143, 143, 143, 143, 143, 144, 144, 144, 145, 145, 145, 146, 146, 146, 146, 146, 146, 147, 147, 147, 147, 148, 148, 148, 148, 148, 148, 149, 149, 149, 149, 150, 150, 150, 150, 150, 150, 150, 151, 151, 151, 152, 152, 152, 152, 153, 153, 153, 153, 153, 153, 153, 153, 154, 154, 154, 154, 154, 154, 154, 155, 155, 155, 155, 155, 155, 156, 156, 156, 156, 156, 156, 157, 157, 157, 157, 157, 158, 158, 158, 158, 158, 159, 159, 159, 159, 159, 159, 159, 159, 159, 159, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 160, 161, 161, 164, 166, 171, 180, 181, 186, 187, 188, 194, 194, 198, 200, 208, 209, 211, 211, 213, 216, 222, 223, 223, 229, 234, 250, 251, 253, 256, 261, 263, 264, 280, 302, 329, 342, 342, 374, 420, 1148, 1519, 1823]

The limit should be enforced on the server, both when saving and when getting the description from other nodes.
I'll move this issue to the server repo.

Issue moved to OpenBazaar/OpenBazaar-Server #443 via ZenHub


Inception_v3 with FPGMPruner

I got my compression config as below:
_compression_algo_cfg = [{
'sparsity': self.sparsity,
'op_types': ['Conv2d']
#'op_names': pruned_list
}, {
'exclude': True,
#'op_names': exclued_list#['fc']
'op_types':['Linear']
}]
and the inception_v3 model is loaded from torchvision with normal forat as:
from torchvision.models import inception_v3
input_model = inception_v3(pretrained=False).cuda()
when I tried to fintune the model after ModelSpeedup, we got:
Traceback (most recent call last):
File "/tmp/codehub/juicer/juicer_slim/juicing/base_juicing.py", line 115, in run
self.juice_slim()
File "/tmp/codehub/juicer/juicer_slim/juicing/pytorch_prune_distill_juicing.py", line 571, in juice_slim
self.model_wrapper.train(self._input_model, optimizer)
File "/tmp/codehub/juicer/juicer_slim/pytorch_jobs/common/trainer.py", line 46, in train
output, aux_output = model(data)#for inception_v3
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/torchvision/models/inception.py", line 204, in forward
x, aux = self._forward(x)
File "/usr/local/lib/python3.7/dist-packages/torchvision/models/inception.py", line 176, in _forward
aux = self.AuxLogits(x)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/torchvision/models/inception.py", line 449, in forward
x = self.conv0(x)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/torchvision/models/inception.py", line 476, in forward
x = self.conv(x)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1051, in _call_impl
return forward_call(*input, **kwargs)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/conv.py", line 443, in forward
return self._conv_forward(input, self.weight, self.bias)
File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/conv.py", line 440, in _conv_forward
self.padding, self.dilation, self.groups)
RuntimeError: Given groups=1, weight of size [128, 768, 1, 1], expected input[16, 384, 5, 5] to have 768 channels, but got 384 channels instead
How can I get rid of these problem?

hello @moonlightian , we are working on a new speedup version, please have a try on this branch https://github.com/microsoft/nni/pull/5403
import torch
from torchvision.models import inception_v3

from nni.contrib.compression.pruning import FPGMPruner
from nni.contrib.compression.utils import auto_set_denpendency_group_ids
from nni.compression.pytorch.speedup.v2 import ModelSpeedup

model = inception_v3(pretrained=False)

model(torch.rand([4, 3, 299, 299]))

config_list = [{
    'op_types': ['Conv2d'],
    'sparse_ratio': 0.5,
}]
config_list = auto_set_denpendency_group_ids(model, config_list, torch.rand([4, 3, 299, 299]))

pruner = FPGMPruner(model, config_list)
_, masks = pruner.compress()
pruner.unwrap_model()

ModelSpeedup(model, torch.rand([4, 3, 299, 299]), masks).speedup_model()

model(torch.rand([4, 3, 299, 299]))

Please close this issue if you don't have any other question by using latest master branch, thanks~ @moonlightian


Enhancing application of derivatives by adding more stuff

Right now the application of derivatives page has only documentation and no image for visualizing the points written over there.
I would like to design images for the applications mentioned and add Rolle's theorem and Lagrange's mean value theorem along with proper images in this section.
Please assign this issue to me.

@sairish2001 Can I work on this issue?

Go ahead
Deadline 18 April,2021 11:59 PM IST

I have made the PR #1039. Kindly review it


feat: add vite-plugin-pwa

The test in pwa plugin requires 1 build for the lib and a few builds per framework (check PR linked below) inside examples subfolder, we need to update a few dependencies.
https://github.com/vite-pwa/vite-plugin-pwa/pull/516
Cannot test on my Windows laptop: I need to remove node_modules and workspaces between tests.
I need to fix pnpm in pwa plugin repo and merge the PR, once done we only need to change here the branch to main.
I should also include react, svelte and vue plugins builds here. What about solid?

Excluded all fw tests: we only test vite-plugin-pwa with TypeScript since we only need to test the service worker is generated properly using workbox or custom service worker.

test passes locally, we still have to update the target branch before merging


Add python.app to the test matrix

🐛 Bug
@joshmoore pointed out in #1597 that our command line interface was failing when used with pythonw/python.app. We should add tests with and without python.app in our testing matrix to prevent a regression.

Agreed, would be nice to add this

@tlambert03 should this be added to #2051 too? Maybe as it's own thing? I'm a little wary of test explosion, but curious what you think

I can do that, let's address #2051 as is though, and it will be easier to play around with and merge afterwards

Waiting till after #2051 sounds good, no rush on this.

I took a look at this and how to potentially implement it. Since we are using tox, the tox-conda plugin looked like the obvious choice, but right now it's either all-in or nothing - you can't select which tox entries use conda. If this issue (https://github.com/tox-dev/tox-conda/issues/36) gets solved, then we could add a new entry to tox.ini.
The other option is to turn all tox envs into conda envs with tox-conda (deps still installed from PyPI), but that might be too much of a change (and potentially slower?) for the CI.
We can also implement it without tox, adding some duplication. What would the preferred approach?

Wait for tox-conda to be more granular
Use tox-conda for all envs
Implement outside the tox workflow

I think options 1 and 2 would be fine, outside the tox workflow would add a bit too much complexity I think.


Transition from Dockerfile to Nix Flake

I have replaced the Dockerfile used for building the book with Nix Flake. This change offers significant improvements in the development process. Previously, we had to download a tarball containing all the necessary utilities and unpack it into the /bin directory. Now, by simply running nix develop in the shell, we can set up the development environment for the book effortlessly.

Thanks for the contribution. It looks like a good idea to use nix for dev setup. I don't use it myself yet so I'll need some time  to review/test. Will this work for all major platforms?

Will this work for all major platforms?

Currently Nix supports only Linux and MacOS, but can work on Windows under WSL 2

@AlexSherbinin Thanks again for the contribution. I like it a lot. I did some modifications:

Instead of official mdbook I use my fork with applied PR for the problem with include indentations. Rustemo docs uses mdbook #include a lot and this issue breaks the docs severely.
Added dependency to plantuml - used by mdbook-plantuml
Added dependency to tex with qtree package and poppler utils - required to run build-latex-images.sh script which creates tree images from .tex files in the docs.
Added info on docs setup with Nix


next() not fired if number of items doesn't overflow the container

See demo on https://codesandbox.io/s/upbeat-payne-hw269
If the number of loaded items does not overflow the parent container, the next function is not called because there won't be any scroll event.

This is the behaviour.
On Fri, Apr 24, 2020 at 1:43 PM Rhuan<EMAIL_ADDRESS>wrote:

See demo on https://codesandbox.io/s/upbeat-payne-hw269
If the number of loaded items does not overflow the parent container, the
next function is not called because there won't be any scroll event.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ankeetmaini/react-infinite-scroll-component/issues/185,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABSYHF65FDFNYGFJLWZJ5NTROFC4HANCNFSM4MP4N4UA
.

--
Ankeet Maini
"Just when the caterpillar thought the world was over, it became a
butterfly!"

But do you think there is a way to trigger next if hasMore is true and the height of the div is less than the parent element?

This one triggers automatically and has almost the same functionality: https://github.com/danbovey/react-infinite-scroller

Good! I'm moving to it now. Closing the issue.


Method of transferring app to server causes error in grpc_csharp_ext.x64.dll

Describe the bug
I am using npm, grunt, and dotnet to built and deploy a dotnet new react project on osx. When this pipeline runs, the project fails to start with a 502.5 error. If I go to the server and execute dotnet app.dll I get a strange error.
Stack trace
Unhandled Exception: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.IO.IOException: Error loading native library "C:\...\runtimes/win/native\grpc_csharp_ext.x64.dll".
   at Grpc.Core.Internal.UnmanagedLibrary..ctor(String[] libraryPathAlternatives)
   at Grpc.Core.Internal.NativeExtension.LoadUnmanagedLibrary()
   at Grpc.Core.Internal.NativeExtension.LoadNativeMethods()
   at Grpc.Core.Internal.NativeExtension..ctor()
   at Grpc.Core.Internal.NativeExtension.Get()
   at Grpc.Core.GrpcEnvironment.GrpcNativeInit()
   at Grpc.Core.GrpcEnvironment..ctor()
   at Grpc.Core.GrpcEnvironment.AddRef()
   at Grpc.Core.Channel..ctor(String target, ChannelCredentials credentials, IEnumerable`1 options)
   at Google.Api.Gax.Grpc.ChannelPool.GetChannel(ServiceEndpoint endpoint, ChannelCredentials credentials)
   at Serilog.Sinks.GoogleCloudLogging.GoogleCloudLoggingSink..ctor(GoogleCloudLoggingSinkOptions sinkOptions, MessageTemplateTextFormatter messageTemplateTextFormatter, Int32 batchSizeLimit, TimeSpan period)
   at Serilog.Sinks.GoogleCloudLogging.GoogleCloudLoggingSinkExtensions.GoogleCloudLogging(LoggerSinkConfiguration loggerConfiguration, GoogleCloudLoggingSinkOptions sinkOptions, Nullable`1 batchSizeLimit, Nullable`1 period, String outputTemplate)
   at Serilog.Sinks.GoogleCloudLogging.GoogleCloudLoggingSinkExtensions.GoogleCloudLogging(LoggerSinkConfiguration loggerConfiguration, String projectId, String resourceType, String logName, Dictionary`2 labels, Dictionary`2 resourceLabels, Boolean useSourceContextAsLogName, Boolean useJsonOutput, String googleCredentialJson, String serviceName, String serviceVersion, Nullable`1 batchSizeLimit, Nullable`1 period, String outputTemplate)
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Serilog.Settings.Configuration.ConfigurationReader.CallConfigurationMethods(ILookup`2 methods, IList`1 configurationMethods, Object receiver)
   at Serilog.Settings.Configuration.ConfigurationReader.Configure(LoggerConfiguration loggerConfiguration)
   at Serilog.Configuration.LoggerSettingsConfiguration.Settings(ILoggerSettings settings)
   at app.Program.Main(String[] args) in /Users/.../src/Program.cs:line 17

Somewhat related issues?
https://github.com/grpc/grpc/issues/11914
https://github.com/grpc/grpc/issues/12570
Reproduce
Currently, I use npm scripts to run

dotnet publish -c Release
npm build to build the react app
grunt-contrib-compress to compress the bin/release/netcoreapp2.2/publish/ folder
grunt-ssh then pushes an app_offline.htm the file to a windows 2016 server running openssh to stop the app.
grunt-ssh sends the publish zip and unzips the project overwriting the old app.

The strangest part is if I RDP into the server and copy the files from the same publish folder the application starts as expected.
What could possibly be causing the above error in the compression, file transfer, and uncompression process?

This isn't related to ASP.NET Core or GRPC. It looks like an issue with the serilog GCP logging sink which happens to be using the native interop C# grpc implementation.  The issue should be moved to this repository https://github.com/manigandham/serilog-sinks-googlecloudlogging

Thanks for looking.


Add latest checksums for Python 3.7, 3.8 and 3.9

This builds off of @tcumby's PR #312 which added the latest checksum for Python 3.8.

@jcfr I can issue a follow-up like https://github.com/python-cmake-buildsystem/python-cmake-buildsystem/pull/307 that would update the CI to use these latest versions and to update the default to 3.9.12 or I can do it here in this PR. Your choice.

Resolved by #327

Closing. This was superseded by:

https://github.com/python-cmake-buildsystem/python-cmake-buildsystem/pull/327

I can issue a follow-up like https://github.com/python-cmake-buildsystem/python-cmake-buildsystem/pull/307 that would update the CI

@jamesobutler If you have the bandwidth, we still need to update the CI to test the latest version


float32 & float64 Error

I just run this code, and it occurs the problem that

found the dtype double but expect float

and I found this code may have some problems:
target = batch["val_acc"]
predict = net(batch)
loss = criterion(predict, target)

The dtype of target is float64 but the predict is float32.
Have I made some mistakes?

I got the same error when running the python train.py


issue at step9ts. I tried to change testnet to mainnet for step9 and always get error: wallet is not deployed. All steps before works, whats the deal

I have the same issue. Were you able to resolve?
All the steps before 9th worked, even getting correct balance of the wallet on the testnet. I thought my wallet was deployed when I created it on my phone via Tonkeeper.

You should first make a send transaction from the wallet to get it mounted and deployed to the blockchain else you would face this issue.

I have implemented this on a project even went to the point of getting bulk transactions and also single transactions more like transaction history, I don't mind sharing my code


ci: Should test transitioning towards official tarballs on Ubuntu Jammy

From meeting notes, should ensure we have the following under CI to de-risk the infra work towards using Ubuntu Jammy:

[ ] Jammy x Official Humble tarball
[ ] Jammy x Official Rolling tarball
Still needs determination on hosting resources

[ ] Jammy x Unofficial Humble tarball
[ ] Jammy x Unofficial Rolling tarball

@cottsay This is what I remember; please let me know if I missed anything.
\cc @IanTheEngineer

The official publication will be enabled by ros2/ros_buildfarm_config#260.

Scott to confirm/fix to ensure Focal CI has been removed.

Everything looks good, closing this out.


Add ability to calculate sha256 hashes for CSP

:+1: for this. I might investigate this myself if I have time this week.


Fix Issue #77

changed size of data array from 21 to 20

This is for 0.8.7, so I close as current is 0.9.3

merged


Posibility to merge Ignore mapping rules?

Is there a way to combine (Or add) new Ignore mapping rules?
Like config.IgnoreNullValues(true)
&& config.IgnoreIf (src == DateTime.MinValue)
There are situations when DateTime properties are initialized with default, like DateTime.MinValue, which is not a valid SQL DateTime value.

No, Mapster can't ignore default value. I think we should make DTO nullable, if it was integer then we cannot know user input 0 or not input.


rz does not work

Version: 1.0.106
Platform: darwin 19.4.0
Plugins: save-output

Duplicate of #2352


Adding TLS to RPC

I'm feeling a little bit dumb, I don't suppose you have an example of adding TLS to a RPC service?
	security.Setup(log, security.Vault())
	tlsConfig := &tls.Config{
		GetCertificate: security.Vault().CertificateGetter(),
	}

	// initialise service
	service.Init(
		micro.Action(func(c *cli.Context) {
			if c.Bool("debug") {
				log.Level = logrus.DebugLevel
				log.Debug("Debug log level enabled")
			}
		}),
// transport.TLSConfig(tlsConfig),
	)

	countryHandler, err := handler.NewCountryHandler()
	if err != nil {
		log.Fatal(err)
	}

	country.RegisterCountriesHandler(service.Server(), countryHandler)

	if err := service.Run(); err != nil {
		log.Fatal(err)
	}

I tried adding transport options to the service.Init but it's clearly not that simple?

So I got it to compile and work with this
	// initialise service
	service.Init(
		micro.Action(func(c *cli.Context) {
			if c.Bool("debug") {
				log.Level = logrus.DebugLevel
				log.Debug("Debug log level enabled")
			}
		}),
		micro.Transport(transport.NewTransport(
			transport.TLSConfig(tlsConfig),
		)),
	)

But now the micro client strokes out with
{"id":"go.micro.client.transport","code":500,"detail":"malformed HTTP response \"\\x15\\x03\\x01\\x00\\x02\\x02\\x16\"","status":"Internal Server Error"}

Is the client you actually using tls?

I'm pretty sure no, I'm just using the micro cli tool
I've tried
micro --enable_tls query srv.mapper Country.List

And
micro --enable_tls=true query srv.mapper Country.List

The micro cli does not currently support tls. The enable_tls flag is used to enable it for the http entrypoints; api, sidecar, web ui. You will need to setup tls yourself by switching out the default transport in go-micro/transport and compiling with that. The micro cli will make use of the default transport.

Might be awesome to figure it out with meta data - the service registers as using tls then the client should use tls. Less chance of cross channels.

Yea might be something to add but requires some work to do it without it being a hack. Also has the potential to be exploited in reverse where untrustworthy services are registered with no tls in a secure environment causing data to leak. There's no reason why a client could not be written to check metadata and do this but not sure it will be the default behaviour.

Hmm yes, that's a concern, but perhaps that can be fulfilled with a "requiretls" option
I know that sounds backwards, but I'm thinking from a dev perspective where they'll probably have a mix of tls and non tls while developing but they'll deploy to tls (although, I'm currently working on that being not the case with dev vault instances that just work)

Likely to spend more time thinking about it before actually implementing anything. A secure by default config would be nice but it's not quite simple.
Please close the issue when satisfied that tls works.

That doesn't seem to be working, looks like it's still trying to http transport.

Try setting the transport in the client after setting it up.
import "github.com/micro/go-micro/client"

func init() {
    client.Init(client.Transport(t))
}

package main

import (
	"crypto/tls"

	"github.com/micro/go-micro/client"
	"github.com/micro/go-micro/transport"
)

func init() {
	// initialise a tls config
	t := &tls.Config{InsecureSkipVerify: false}
	tr := transport.NewTransport(transport.TLSConfig(t))
	client.DefaultClient.Init(client.Transport(tr))
}

This works

Decided to test it myself.
When creating a service I set:
service := micro.NewService(
    micro.Transport(transport.NewTransport(transport.Secure(true))),
)

For the micro client I set:
package main

import (
	"github.com/micro/go-micro/client"
	"github.com/micro/go-micro/transport"
)

func init() {
	client.DefaultClient.Init(
		client.Transport(
			transport.NewTransport(transport.Secure(true)),
		),
	)
}

This works.

@asim : I tried what you mentioned, but the client and server still communicates even though they are set to true/false.
I also tried with certificates:
Server.go
service.Init( 		micro.Action(func(c *cli.Context) { 			config.PackageFilePath = c.String("filepath") 		}), 		micro.Transport(transport.NewTransport( 			transport.TLSConfig(tlsConfig), 		)), 	)
Client.go
service.Init( 		micro.Action(func(c *cli.Context) { 			Server1 = c.String("server") 		}), 		micro.Transport(transport.NewTransport( 			transport.TLSConfig(tlsConfig), 		)), 	)
Here, even if the certificates are different on both sides, they still communicate.
This is strange as I am facing this issue only in case of go-micro.
I tried with normal grpc, and there it works perfectly.

Are you using go-micro with grpc? If so it ignores the transport and you need to set this in the client and server.

@asim : Yes, I am using go-micro with grpc.

Can you tell me how to set the transport in the client and server ??

Also, another query:
I have tried using "google.golang.org/grpc/credentials"
Server:
creds, err := credentials.NewServerTLSFromFile("server.crt", "server.key")
opts := []grpc.ServerOption{grpc.Creds(creds)}

Client:
creds, err := credentials.NewClientTLSFromFile("server.crt", "")
conn, err = grpc.Dial("<IP_ADDRESS>:8090", grpc.WithTransportCredentials(creds))
Using the "grpc" api works properly. However, using the "go-micro/grpc" doesn't.

Please don't comment on closed issues. Feel free to join the slack to discuss resolving this. Alternatively for direct support please ask Riverbed to pay for this.
If you're using go-grpc please pay attention to the options https://godoc.org/github.com/micro/go-grpc#WithTLS

{"id":"go.micro.client","code":500,"detail":"connection error: x509: certificate signed by unknown authority","status":"Internal Server Error"}
Getting error in my client to hit sample.com:443, Kindly please help me in creating client


Update idiff to correct image test output dir

Point idiff at the correct image test output directory.

?


Documentation: List of technologies used in project

Based off an idea from @nikoheikkila and myself from #166
As cool things are added to this project, we should document what they are for future reference.
Here's what I'm thinking.

Let's start a new header on README for Technologies.
Under that, we can list each technology used with a link to their documentation, site, repo, etc
Optional, a short description explaining what the technology is and how it was used.
Optional, include a reference to this in the contributing doc, pull request template, etc
Optional, include a code sample for so that copy and pasting and adding new links is easy peasy

This is an easy issue I'd like to reserve for an eager first timer
Comment below if you have questions and before starting work on this issue

Hi! So, as far as I can tell, you're using the following technologies:

JavaScript

JQuery
Moment.js

Bootstrap

Where are you hosting the website (and could it be hosted by GitHub Pages)?

I'm hosting the site on the server I use for my blog/portfolio site.
I guess it could be hosted on GitHub pages. What's the pro / cons for traditional hosting vs Github Pages

Pages is convenient (code on GitHub), quick (just edit, push, and changes are live) and free! Great if you don't have your own server.

I've created a pull request with a proposed solution!


Build REST API samples with Scala 2.13 when Gatling moves to 2.13

This is a spin-off from #32.
REST API samples can't build with 2.13 for the moment.

Fixed in #36

#36 wasn't merged

This was closed by #37 and #38.


PUT image without multipart with query token

I'm replicating the behavior of a signed put request with Cordova (for example to amazon s3, in this case a local PUT request).
The upload request is not multipart and only sends the image (I save it with createWriteStream(..).pipe(this.req).
I got this to work with manual tests, now I want to see this automated.
Problems:

The implementation of .write() in Superagent v.1.3.0 loses the query params (including an important token)
Supertest .pipe() implementation has an inheritance flaw/bug

When will supertest update superagent to v.1.6?
How to use .pipe() correctly?

we are now updated to the latest version of superagent.


loosing alignment on  expand click

I have a page which has  3 <DIV> in a row. All the Div's are having collapse link (collapse-link) within that DIV.
If i expand first DIV, its height increases, which causes the miss alignment of the subsequent DIV's.
Please refer below images.
https://i.stack.imgur.com/JKfJv.jpg
https://i.stack.imgur.com/xJwuI.jpg
If i expand third collapse link, then its working fine.
https://i.stack.imgur.com/rH8jL.jpg
Stackoverflow Issue: https://stackoverflow.com/questions/49819454/jquery-ui-sortable-loosing-alignment-on-div-expand-click

That's how float works. If that's not working for you, of course you can find another way to align the blocks but they are working just fine.


Can data packages be stored on a private conda repository instead of on Anaconda cloud?

I want to use GDD to manage our data in an enterprise environment, but cannot share our data on Anaconda cloud. Does GGD work also on a private conda repository? What should I do to make it work?
Cheers,
Zhen

@zhangzhen, can you explain a bit more about what you are trying to do?

@zhangzhen, I am closing this issue because I haven't heard back from you. If you are interested in private data packages you can read about it on the GGD docs: https://gogetdata.github.io/private_recipes.html. Feel free to contact us if you have any other questions.

I've read the private_recipes part of the GGD docs, but the private recipes are not what I want. I want the bzip file created by ggd check-recipe to upload our private channel located at http://172.16.x.x:5000/ggd-genomics. ggd install -c http://172.16.x.x:5000/ggd-genomics can pull the bzip file and install this data package into an environment.