About 0.6-dev

@icefapper I noticed the effort you put into splitting the codebase into smaller components but unfortunately, in my opinion, it's not completely on the mark yet.
For now you should only think about node's environment and focus on making the best use of its module system. A practical example of this would be what I've done in the parser folder where:

Prototype functions are defined one-per-file in parser/parse and parser/util.
Shorthands that are meant to be used by only a function (like #asArrowFuncArgList and #asArrowFuncArg being called only by Parser#parseArrow) are now bound to that function module's scope.
The constructor is a function defined in parser/constructor.js.
The parser's entry point is parser/index.js which automatically binds the prototype to its constructor.

Pros & Cons
Pros

Hierarchical organization of modules
No global space pollution (like with the _class)
Better scope management thanks to modules (you expose only what you need)
Natively supported by node (no need for compilation/transpilation of any sort)

Cons

The browser version requires to be generated via browserify

Additional notes
While the use of browserify might look a bit restrictive, it is very advantageous instead. Why? It can generate UMD builds!. This way jsRube would work as intended independently of how it is loaded in the browser (be it a src tag, an AMD module loader a CommonJS module loader or whatever).
By the way, if you want to discuss privately, I'm always avaiable at<EMAIL_ADDRESS>

Hello
Thinking again, I guess I should give up on my strict abstinence from dependencies. I reached this conslusion after viewing 0.6-dev; even though they have not (yet) posed a radical change on the structure, the changes look great. I just wonder why you did not do it directly on the master branch. It deserves to be there.
tl;dr
Your code looks much more manageable than mine, and, if it is passing the tests, it deserves to be directly on the master branch.
Thanks again for all your efforts!

Just one note about chosing the right node vesion to run jsRube on:
in all versions of node before 6x, it abrubtly ran into crashes while benchmarking "real-world" cases, like jquery1.4.2-mobile, etc.
Please take this issue into account while changing the node version. Thanks again!

@icefapper Didn't want to push to the master branch because of my poor git skills and because this is just a proposal (only a few functions have been moved and I've been very rough while doing so).
About the node version I thought the bug had been already fixed, I should've paid more attention there!

I've actually been sold by your proposal and am now refactoring accordingly.
Could you tell me how should I adjust the test files (quick-test.js and build-and-test.js)? I have never used browserify and I'm clueless how I can make the ./dist/lube.js now that we are going to depend on browserify :\

@zenekron also, what is your idea about having an IRC channel for our discussion? I've created a #jsrube channel, on freenode but I'm not sure how of value it would be to development speed

@icefapper I will be happy to demonstrate how to build and test the library, unfortunately that can't be done until all the code has been refactored. Anyway I can assure you that it's a pretty simple process that wont require too much effort.
I proposed the mail only because I'm not really used to open issues only to discuss about some features, but it's not a problem for me.
IMO if in the future you feel the need to create some sort of chatroom for the project, you might want to take a look at gitter before creating IRC channels.

Thanks for mentioning gitter; looks great; I will give it a try!
As for the refactoring thing, I've done some; in fact, it's almost done, except that I have no clue how to make modules in src (which are the "global" modules) accessible to those under src/parser/ :\

@icefapper I'll step in to sort a couple things out :)

Thanks a lot!

@zenekron
I just created the gitter "channel"!

@icefapper The porting has been completed, the modules behave correctly but, as I said in f10a774df68b044fa232c6e59a8e3fa6f053594c there must have been some minor issue during the porting that makes the parser fail.
Let me hear what you think about it.

@zenekron
Wow!
Thanks for all the efforts, the code base looks much cleaner now; I just can't believe it!
As for the errors, we had both made a few :wink:
Mine was the unforgivable one though; it was the aftermath of a careless replace (vim's :%s/ cough/g) of 'function [something]' with 'module.exports.[something] = function'. I hope I have corrected it.
Yours was a minor one indeed. A lesson I dearly paid for back when I started this project was:

This in an object literal does not refer to the object literal it is contained in.

Also for some real-world cases (notably angular) the parser gets into infinite loops; strange, because that does not happen in the master branch's version of the parser. I have no immediate clues as to where this is, and I'd be glad should you help me browserify it so I can inspect it in the venerable chrome environment :wink:
cleaning and refactoring the mess into something this modular and readable is something I couldn't have done all by myself. Thanks again for the time :)
P.S. refactoring for this particular code base might have a couple minor performance issues to beware of, but that would be something that can be taken care of after we get the tests passing.


Add Activity SOPAAN

Under SSSVV activities we have 5 activities – We now add a 6th one called SOPAAN. This will be another sub page just as we have for the other 5 activities. The write up is given in the link below

please suggest suitable text for SOPAAN on home page.
(I would suggest something like
SOPAAN stands for S??? O??? P?? etc which is a bilingual, foundational learning program for children aimed at developing their passion and confidence for reading and lifelong learning. SOPAAN is currently active in . To know more about SSSVV's SOPAAN, click here. )

TODO - add sopaan to all footers as well as headers

reduce padding in home page so that SOPAAN button also comes in the first line instead of wrapping.

finished this with sopaan button on home page.

add links which are mentioned in the Word Doc


Convert Groovy test classes to Java

Convert the Groovy acceptance test classes back to Java and remove the dependency on Groovy.
This will help support newer JDK versions sooner (like Java 22).

@ascopes I would like to work on this task.
Could you please assign it to me and provide a bit more context about this task.

@VarunEnishetty sure thing. Assigned!
 Many are implemented in Groovy rather than Java. These tests try to compile some sample code in the resources/ directory. This sample code will relate to a commonly used annotation processor library, and is basically a way of me ensuring the compiler configuration is correct.
What we want to do is rewrite those tests in Java instead (will be a bit more verbose due to how reflection works in Java versus Groovy). You'll need to make use of the java.lang.reflect APIs to translate some of the logic that compiles code and then loads the classes dynamically.
Some of the tests already use Java (I vaguely recall the dogfood test already does this, which is a test that compiles the code in the java-compiler-testing module as a proof that complex code can be compiled via this library).
I'd suggest doing one module at a time to keep it easy to review and easier for you to work on.
Each module will need to pass the build process (./mvnw clean verify -am -pl <module-name> should be all you need to run to do this, e.g. ./mvnw clean verify -am -pl acceptance-tests).
You'll also need to make sure the license headers are in place for that build to work. You can do this automatically by running ./mvnw license:format.
Let me know if you need any help or have any questions!

@VarunEnishetty how are you getting on? Is there anything else you need clarification on?

If you're still interested in working on this, please let me know. Happy to keep it assigned but if you aren't interested then it enables me or someone else to pick it up when we have time :-)

Going to unassign this, just as I haven't heard back... if you find you have time to pick this up in the future, please let me know.
Thanks.

Going to pick this up as I have some changes for this on my laptop at home that I will push when I get the time.


⚠ Bofrost.de has degraded performance

In 25a6b36, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1046 ms

Resolved: Bofrost.de performance has improved in ad17ee4.


Change LogEntry.get_for_objects to distinguish between number and text keys.

When using LogEntry.get_for_objects with a model that has an automatic (numeric) primary key, the following error occurs:
operator does not exist: text = integer LINE 1: ...ction_id" = 1) OR "auditlog_logentry"."object_pk" IN (SELECT... ^ HINT: No operator matches the given name and argument type(s). You might need to add explicit type casts.

I suspect this is a Postgres-specific error.

Thanks for your contribution!

Just wanted to point out that a new version, including this fix, has just been released!

Great timing and much appreciated.
On Sat, Jan 23, 2016 at 5:01 PM, Jan-Jelle Kester<EMAIL_ADDRESS>wrote:

Just wanted to point out that a new version, including this fix, has just
been released!
—
Reply to this email directly or view it on GitHub
https://github.com/jjkester/django-auditlog/pull/47#issuecomment-174225733
.


Ventilation system schedules

We have implemented ventilation system with schedules.

@avneet006 I took the liberty to commit the required input dictionary changes directly into your PR. I will be approving and merging the request once the tests run


Interpret highway=pedestrian and bus-onl roads properly

https://www.openstreetmap.org/way/353926198
There should be filters on these by default

Thanks, came here to open that issue :)

This can't be done in a straightforward way of one (or two) filters per road segment with the tag, or this monstrosity occurs. The filters need to only get placed where these special case roads meet normal roads.

Not perfect, but the Manchester example is looking more tame. Need to rethink the red disconnected cells -- this was meant to show user error originally, but they're pedestrianized areas here.

Thanks ! That works fine for my use-cases based on my quick tests.
The red areas should ideally show streets that are only reachable through a street with a different road restriction than they currently have (making them "disconnected"/"islands").


Complete of import does not work

Hi! My English is poor) I use the google-translate.
I use vim8.0, linux.  My pom.xml:
...  11     <dependencies>  ...  17         <dependency>  18             <groupId>commons-digester</groupId>  19             <artifactId>commons-digester</artifactId>  20             <version>2.0</version>  21         </dependency> ...  22     </dependencies> ...
jar exist in ~/.m2/repository. But complete of import not work:
http://storage7.static.itmages.ru/i/16/1117/h_1479362939_2623733_c52de645ac.jpg
But complete of object work:
http://storage4.static.itmages.ru/i/16/1117/h_1479362830_1917880_f55a6d2cf3.jpg
I did "let g:JavaComplete_LibsPath", but it does not help.

It works! Thank you!


Unexpected column_names processing when incorporated with line skip

I would have expected when skipping lines before starting to read a sheet when column_names is set TRUE, the first line read would be the assumed column_names.  If it is always expected that column_names appear in first row, then it would be good feature add have mechanism to specify the starting row for read rather than having it always assumed equal first row when case is that the header is not in first row.

The behaviour of skip is about to align much more with expectations, when #240 gets merged.
Please open a new issue with a reprex if you have continuing difficulty after that.


feat: add B512 support

closes: https://github.com/FuelLabs/fuels-rs/issues/688
What was done:

add the B512 type and excluded it from the Abigen
add B512::from((Bits253, Bits256)) to construct a B512 ( this is how it is done in the std lib)
add test and docs
change name of bits_256.rs to bits.rs - we have there Bits256, EvmAddress and B512

I have updated all manual Tokenizable implementations for our structs. Now we make sure that they receive the right token and the right number of tokens. By chance it turned out that all need only one token e.g. have one field.


How do I tell people I made a new bucket!?

It'd be great that we get a website to share our bucket, like npm &  npmjs.com !

You can add it to buckets.json in this repo. At the moment there's no way to list what buckets are available to install, but that would be an easy fix if someone wanted to do it.


Color blind accessibility

Avoid duplicates

[ X] Feature request is not mentioned in the FAQ
[ ] Feature request is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[ X] Feature request is not already reported in another issue

Feature description
In the main screen, there exists three status indicators:

This leads color blind people (like me) to not be aware of the colors and cannot differentiate the various statuses.
Problem and motivation
As a color blind user, I want the application to have ability to switch between colorblindness accessibility and switch it off with a button so that I can see and differentiate the various statuses.
For the start I’d like to sugges statuses as follows:
⚫ Higher Risk
🟡 Low Risk
🔵 Risk unknown
Is this something you're interested in working on
No

Internal Tracking ID: EXPOSUREAPP-2179

Hello @skayikci,
I have transferred the issue to our development team.
Thanks,
LMM
Corona-Warn-App Open Source Team


request for scripts

Great job!
When will u provide the scripts/dist_train.sh?

We have added the script in the latest version. Please check it out.


Search feature

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/8f4403698450ddd/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/8f4403698450ddd/output.txt
Total script time: 2.34 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global


fixing bug with getting frames of video

This fixes a bug I ran into on Ubuntu 18.04 with the default ffmpeg, and updated to 4.2, and I traced it down to imageio not correctly reading the number of frames (and throwing an error when it got to the last section and reporting 0). This comes down to changing:
driving_video = imageio.mimread(opt.driving_video, memtest=False)

to
driving_video = []
try:
    for im in reader:
        driving_video.append(im)
except RuntimeError:
    pass

I also:

added likely folders to the .gitignore
ensured we use the yaml.FullLoader for safe yaml loading

And I just want to say I'm absolutely in love with this work! I generated several videos using paintings over night, and am going to build a Docker -> Singularity container today to run on our cluster with GPU (and try out some of the other models!) Thank you so kindly for this work (and the spot on documentation for reproducing it!)
Signed-off-by: vsoch<EMAIL_ADDRESS>

Thank you for your help.

Changing yaml loader not working with colab.
video, img, driving folders probably not relevant.

okay!

I've removed adding the extra files to the .dockerignore
I've removed the safe yaml loader (and I'd be curious to know why it doesn't work in colab, maybe a version conflict?)

The main change to fix the bug that I originally meant to address is still present.

Thanks.


Please invite me to the GitHub Community Organization

Name
Jyoti prakash Rout
Discord Username (if applicable)
Jyoti#0398
Additional Context
I'm an Web Dev Enthusiasts & a junior pursuing my Bachelors in Computer Science & Engineering. I'm interested in Web Devlopment, Android Devlopment, IOS Devlpoment, Machine Learning & Devops & I enjoy learning new things & stuff as well as I'm a Frontend Devloper with experience building Dynamic & Static Web pages. I am always interested in learning the answers to how, why and what relating to Technologies around me.
I would like to grow my programming skills by doing lot's of  Open Source Contribution. Looking forward to learn, collabrate and grow as a community.

Oh not sure why the bot didn't work. Please could you close the issue and try again

oh it worked, I guess there was a queue/delay

Thank you!! ;) @eddiejaoude


Consolidate Tape logic and scope logic.

This PR:

Moves all of the scope logic into the backendEngine. We do not break math.scope() users as we just push those requests into the backendEngine. This can be renamed in a follow up PR, but is not renamed here for ease of review.
Removes the Tape class. Now, Tape is simply an array of TapeNodes. This is so that serialization is as easy as JSON-ifying the Tape (with some exceptions like the gradient function).
Consolidates the stack logic for scope and tape. When a new scope is created, now we create a subtape.
Calls scope in the gradientWrt method while backpropogating gradients. This scopes all of the backprop code into a subtape element. This subtape element will have its inputs automatically computed (so the subtape tape node operates as any other node on the tape, having inputs, outputs, and a gradient method).
Adds multi-output functionality to TapeNode logic, including filtering and backpropagation. This is necessary because gradientWrt outputs multiple nodes. For now, we leave KernelNode to only have a single output, but we will easily be able to convert it to multi-output if we ever want to.
Adds gradients for strict multiply, subtract, pow. We will flesh out broadcasted in a follow up PR.
Add lots of unit tests for utility methods.

This change is

 Nice job on the tests! Especially tape_util_test which helped me understand what is going on :)

Reviewed 23 of 30 files at r1, 4 of 8 files at r2, 5 of 6 files at r3, 1 of 1 files at r4.
Review status: all files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

src/util.ts, line 25 at r4 (raw file):

NameArrayMap

slightly prefer NamedArrayMap more, but up to you

src/math/element_wise_arithmetic_test.ts, line 243 at r4 (raw file):

      expect(() => math.gradientWrt(result, a)).toThrowError();
      expect(() => math.gradientWrt(result, b)).toThrowError();

a bit ambigious why this test will throw. Could be that result is not a Scalar, not necessarily because broadcast should throw. Maybe call result = math.sum(result)
Also, is the idea to delete this test in the next PR , once we have broadcasing support?

src/math/math.ts, line 953 at r4 (raw file):

                     });
                   },
                   b: () => {

this is a bit hard to read.
split into variables, here and for math.sub as well
const config = {inputs: {a, b}};
const derA = () => {...}
const derB = () => {...}
const der = (dy, y) => {a: derA, b:derB}

executeKernel('Pow', config, der);

src/math/math.ts, line 2286 at r4 (raw file):

   */
  gradientWrt<T extends NDArray|NameArrayMap>(y: Scalar, x: T): T {
    const xs = util.flattenNameArrayMap(x);

missed from previous PR. Can you explicitly check that y.rank == 1 at run-time

src/math/matmul_test.ts, line 332 at r4 (raw file):

onst expectedDa = math.matMul(
Array2D.ones([2, 2]), b, MatrixOrientation.REGULAR,

better to hard-code the values of expectedDa and expectedDb so this test is not linked to the impl of matmul

src/math/unaryop_test.ts, line 398 at r4 (raw file):

1e-1

curious why this and following tests need such a loose bound 1e-1 and not the previous test?

Comments from Reviewable

Thanks for the review!

Reviewed 28 of 30 files at r1, 6 of 8 files at r2, 5 of 6 files at r3.
Review status: 6 of 33 files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

src/util.ts, line 25 at r4 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…

NameArrayMap

slightly prefer NamedArrayMap more, but up to you

Done.

src/math/element_wise_arithmetic_test.ts, line 243 at r4 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
a bit ambigious why this test will throw. Could be that result is not a Scalar, not necessarily because broadcast should throw. Maybe call result = math.sum(result)
Also, is the idea to delete this test in the next PR , once we have broadcasing support?

Good point. I used sum, and yes that's the idea.

src/math/math.ts, line 953 at r4 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
this is a bit hard to read.
split into variables, here and for math.sub as well
const config = {inputs: {a, b}};
const derA = () => {...}
const derB = () => {...}
const der = (dy, y) => {a: derA, b:derB}

executeKernel('Pow', config, der);

Unfortunately I can't break it up like you're suggesting because of the dependency on dy, but I pulled "der" out to a separate variable.

src/math/math.ts, line 2286 at r4 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
missed from previous PR. Can you explicitly check that y.rank == 1 at run-time

Done. And added a unit test.

src/math/matmul_test.ts, line 332 at r4 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…

onst expectedDa = math.matMul(
Array2D.ones([2, 2]), b, MatrixOrientation.REGULAR,

better to hard-code the values of expectedDa and expectedDb so this test is not linked to the impl of matmul

Done.

src/math/unaryop_test.ts, line 398 at r4 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…

1e-1

curious why this and following tests need such a loose bound 1e-1 and not the previous test?

Ah, great question. The problem is that the error accumulates because of the sum :/

Comments from Reviewable

Review status: 6 of 34 files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

src/math/math.ts, line 953 at r4 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
Unfortunately I can't break it up like you're suggesting because of the dependency on dy, but I pulled "der" out to a separate variable.

Okay, pulled derA and derB out inside of the der function itself. Much better.

Comments from Reviewable


[Bug]: Incorrect info shown to user on user having execute but not having view permissions on datasources

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When user is only given execute and not given view rights to datasources, on logging in as that user, the Data CTA comes with the yellow exclamation 'create a datasource to write your first query', and asking user to connect to a datasource to write the first query.
This is incorrect and misleading, as the user does not have access to create new datasources, but he has permissions to run queries on existing datasources
Steps To Reproduce

Create an application with few datasources and associated queries in the enterprise edition of appsmith
In the Roles section of admin settings, for the above app, give all the application resources permission for all pages
In the datasources and env tab, give only execute permissions on a datasource, and give Production environment access.
Add this role to a group and add a user to that group.
Login with this user and open the application.
Note that the Data CTA comes with the yellow exclamation 'create a datasource to write your first query', and asking user to connect to a datasource to write the first query.
Also on clicking the Data CTA, the right panel comes up with Invalid URL text

https://github.com/appsmithorg/appsmith/assets/101863839/3da7b1ea-96eb-4ab7-81d8-c81a17f93c71
Public Sample App
No response
Environment
Production
Severity
Medium (Frustrating UX)
Issue video log
No response
Version
Cloud

known limitation


[python] Support X-free AnnData files

This is from user feedback.

#1672 is merged and released in 1.5.0rc1.

[sc-35075]


[BUG] app delay

Describe the bug
sound delay (1 second).
Is there a way to remove delay?

Please fill details in the bug issue template.


Crypto payment issue

hey @RamRamez @yussdev
For some reason Coinbase Commerce gives the error below

Can you look into this and see what the error is? It may have to do with our account, so I submitted a request to Coinbase. However, can you check to make sure there are no technical reasons why it's not working?
Also, can you look into alternative apps we can use for Shopify Crypto payments besides Coinbase?
Thanks

Hey @ahmadabugosh I checked this one there are no technical issues on the front side we just redirect the user to the checkout page, all the things are handled by Shopify checkout page
For alternative crypto payments there are more choice that we can integrate like this one:
https://developer.bitpay.com/docs/shopify-plugin

Thanks @yussdev , I signed up for the new KYC system for Coinbase commerce and Bitpay. I'll keep you posted once I hear back from either of them.

This has been done, we're Coinbase Commerce is back online


[WIP] Adding a metric to show advance cluster feature use

Added a new metric which shows whether any advance feature is being used or
not.
Signed-off-by: Arun Kumar Mohan<EMAIL_ADDRESS>

/retest

@aruniiird This PR has a lint error, please check the logs

What exactly is an advance feature? Is it something like a tech preview or new feature? Will a feature in one release be removed in a later release?

What exactly is an advance feature? Is it something like a tech preview or new feature? Will a feature in one release be removed in a later release?

@nbalacha ,
Mainly focus on creating a (single) metric, which indicates whether customers are using any of our advance features (like external cluster mode, PV/KMS encryption etc). Later this metric will be transferred to telemetric client and will be further populated to the telemetry server.
For more details here is the epic link: https://issues.redhat.com/browse/RHSTOR-3173

Ready for an initial review... @umangachapagain , @jarrpa , please take a look...

What exactly is an advance feature? Is it something like a tech preview or new feature? Will a feature in one release be removed in a later release?

@nbalacha , Mainly focus on creating a (single) metric per ceph-cluster, which indicates whether customers are using any of our advance features (like external cluster mode, PV/KMS encryption etc). Later this metric will be transferred to telemetric client and will be further populated to the telemetry server. For more details here is the epic link: https://issues.redhat.com/browse/RHSTOR-3173

Thanks. I'd recommend renaming it to "advanced" in that case.

Submitted the changes... @umangachapagain , can you please take a look...


Remake for better chunk generation.

The update is being made on the Efficient/ branch of my fork: https://github.com/somePythonProgrammer/PyCraft

@1upCommunity/everyone the multiple VBO lag issue was fixed!


refactor: remove unnecessary Box from into_inner

mod symphonia is private to rodio, so this change is possible without changing rodio's API.
Could you also remove the Box in the caller?

Sure! Is 371d963 what you're looking for?

perfect!


Changes parsing of disable-limit to treat as bool

While debugging why the ansible-disable-limit property (resources.source.{n}.config.ansible-disable-limit)  does not resolve in a workflow step, I came across this line, which seemed to be parsing the boolean property as a string.

Thanks! I added you as a collaborator. :)
See CONTRIBUTING on how to do releases!


Use netip.AddrPort for FiveTuple

This commit refactors FiveTuple to use netip.AddrPort instead of net.Addr to store the source and the destination address in order to improve performance on frequent GetAllocations.
Description
The GetAllocation() routine calls Fivetuple.FingerPrint() per each received packet to find the allocation in the allocation map, effectively calling UDPAddr.String() per each packet multiple times, adding 4-5% CPU overhead per packet.  This commit refactors FiveTuple to use netip.AddrPort instead of net.Addr to store the source and the destination address: since netip.AddrPort is comparable we can use it directly as a map key instead of having to stringify the addresses first. As the FiveTuple.SrcAddr (being now a netip.AddrPort) can no longer be passed directly to net.WriteTo() (as netip.AddrPort does not implement Network() but even if it did transport.VNet and company still require a net.UDPAddr), the client address is now directly stored in the allocation.
This commit should cause no user-visible change.
Benchmark
Benchmarks (use go test -cpuprofile cpu.prof -bench=. -run=XXX -benchtime=10s) are somewhat inconclusive; see the attached CPU profiles against the current master and the refactored TURN server:
go tool pprof -http=":8001" ./cpu-master &
go tool pprof -http=":8002" ./cpu-refactor &

On the one hand, current master spends about >8sec in GetAllocation, this all but disappears (<.2 sec) with the refactored version. We get some new overhead as netip.AddrPort seems to perform a malloc in the hot path but this does not seem like a massive loss. At the same time, the overall benchmark results do not seem to show significant gains, but this may be due to that we benchmark the client (that this PR didn't touch) and the server (which it did) jointly.
The bad news is that UDP.String() still appears in the CPU profile: it seems that the spend quite some time to stringify the client and server addresses for Log.Debug here and here, just to drop the result  anyway unless we run at the debug loglevel in production (which is not really useful); see also this pion/logging issue.
Reference issue
Fixes #299
cpu-master.prof.gz
cpu-refactor.prof.gz

Hi @rg0now,
Lets close the PR for now. It seems like Pion, will keep compatability with Go < 1.18 for now until major Distros have updated their Go compiler to a newer version. Hence, we can not utilize netip (yet).
Lets keep #299 open for now and revisit it later once we switch to Go 1.18 and newer.


Restrict ConveRTTokenizer to work only if model_url was specified.

Since ConveRT was taken offline by the original authors, the tokenizers and featurizers cannot download the model from the original URL of the model.
However, some users may already have a copy of the model and they can host the model on their own if they wish to and use it inside the components. To do so, they can specify the URL of the model or path to a local directory as the value to the parameter model_url in the configuration of ConveRTTokenizer. ConveRTTokenizer should throw an error if the model_url is not specified in the configuration or it is the same as the URL of the original model which the authors hosted.
Currently, without this check in place the components error out with a model not found error which is not helpful for the user.
👉  Slack Thread

Bumping up the priority as it affects a lot of users.

Hi Team,
Even I was facing the ConveRTTokenizer unavailibility issue before so i upgraded rasa to '1.10.14' (using the command "pip3 install rasa[convert] ==1.10.14") and made the following changes to the config.yml file --
pipeline:
name: "ConveRTTokenizer"
"model_url": https://github.com/connorbrinton/polyai-models/releases/download/v1.0/model.tar.gz
name: "ConveRTFeaturizer"
"model_url": https://github.com/connorbrinton/polyai-models/releases/download/v1.0/model.tar.gz
But ever since I made the above two mentioned changes, my model has become extremely erratic (unable to identify entities properly, misclassifying intensions etc.).
My data has not changed and only above mentioned changes are made from my side. Is there anything i am missing or need to do here?
Any suggestion or pointer would be of great help. Thanks community.


remove buggy url encoding

Fixes https://github.com/zmxv/react-native-sound/issues/500

How much will it take to merge this fix? it worked for me locally but i would love if it's available in the package.

Could you please give some example of context working after and before? Is it still working on with filename with spaces?

This look like to be the opposite of this fix https://github.com/zmxv/react-native-sound/commit/3fe5480fce935e888d5089d94a191c7c7e3aa190
@KnowYourLines  @Reda1337

This look like to be the opposite of this fix 3fe5480
@KnowYourLines @Reda1337

Yes that fix causes this problem. https://github.com/zmxv/react-native-sound/pull/628#issuecomment-1179426688 from that PR points this out.
Frankly, I don't believe URI encoding should be done by this library to begin with as there are very likely more effective libraries that accomplish URI encoding and would allow encoded URIs to be passed to this library.
It would make more sense for the users of this library to be responsible for passing an appropriate URI to this library rather than have URI encoding in this library that enables some inputs to work but prevents others from working. The current URI encoding in this library seems like a premature optimization since it arguably causes more problems than it solves.

Can we merge it?

@RomualdPercereau I had to revert the change made in this 3fe5480. and now it works perfectly fine. So Could we merge please

Sorry for the delay, it's merged!


fix: ndcg evaluator

Plus, remember to bump version

No I see no need, I guess no proper built happened maybe?

@JoanFM okay, I removed the empty requirements file


Update ListingTable to use StatisticsConverter

Is your feature request related to a problem or challenge?
Thanks to https://github.com/apache/datafusion/issues/10453, we now have the very nice StatisticsConverter API (code link) that extracts statistics from parquet row groups and is well tested.
However, there is a different code path that extracts and summarizes statistics in ListingTable used for pruning files https://github.com/apache/datafusion/blob/9ab597b2511b23dc951c9f35d00d38c2071e9fea/datafusion/core/src/datasource/file_format/parquet.rs#L310-L373
In addition to being more code, this also seems like it doesn't properly convert the data types
Which is used https://github.com/apache/datafusion/blob/ea21b08e477cc48f458917c132f79c4980c957c1/datafusion/core/src/datasource/file_format/parquet.rs#L511-L535

@NGA-TRAN  recently made this API public in https://github.com/apache/datafusion/pull/10880 but I think we can do better

Describe the solution you'd like
I would like to update the code to use StatisticsConverter and rather than converting row/column at a time
Describe alternatives you've considered
No response
Additional context
No response

I did not notice there was a PR already. Feel free to close mine.


Dev fee not added to API response if payment processing is not enabled.

Description
Disable payment processing and the dev fee is no longer returned in the api.
Steps to Reproduce

Set paymentProcessing - enabled: false
Query API for poolfee on a coin that has a dev fee and it will be missing

Expected behavior
To include the dev fee so api responses are consistent.
Actual behavior
Dev fee is not included.
Reproduces how often
100%
Version
v42
I have one stratum with payments disabled and one with them enabled but both are part of my api VIP and I noticed I was getting inconsistent responses for the fee.  There's probably a better way to architect my stratums to avoid this.

This is by design. In a clustering scenario, there should be a single pool master that does all the payment processing and serves API requests.


Fix issue with get repo info #130

Description
Issue when the sub group has the same name as the repository.
The getRepoInfo function does not parse it out correctly.
This fixes this issue.
Example url: https://my-gitlab-instance.com/top-group/sub-group/repo-name/repo-name
getRepoInfo returned before:
host: my-gitlab-instance.com
namespace: top-group/sub-group
repo: repo-name
err: %!s(<nil>)

it now returns:
host: my-gitlab-instance.com
namespace: top-group/sub-group/repo-name
repo: repo-name
err: %!s(<nil>)

Related issues

Close #130
Might close:
Close #80

Checklist

[X] I've read the guidelines for contributing to this repository.
[X] I've followed the conventions in the PR title.
[X] I've added tests that prove my fix is effective or that my feature works.
[N/A] I've updated the readme with the relevant information (if needed).
[N/A] I've added usage information (if the PR introduces new options)
[X] I've included a "before" and "after" example to the description (if the PR is a user interface change).

mb, updated the test with correct BaseURL.

LGTM, accepted the changes

LGTM, accepted the changes, added for github as well since It was the same problem their if a org had the same name as repo.

Hi, I don't see the tested you added for github

LGTM, accepted the changes, added for github as well since It was the same problem their if a org had the same name as repo.

Hi, I don't see the tested you added for github

You are right, your changes did override them. Do you want me to add them again or should we ignore them?


Authentication error: Get accesstoken failed. Username or password error!

Describe the bug
After start is the error displayed. nothing changed in the system or username/pw
all devices are not functial. username and PW checked in the app... is ok.

Steps To Reproduce
after the error... this line/error displayed
Cannot acquire token, waiting 185 seconds.

Plugin Version
latest official

Homebridge Config
{
"name": "TuyaWebPlatform",
"options": {
"username": "Steffen.leicht@xxx",
"password": "xxxxxxxxx",
"countryCode": "49",
"platform": "smart_life"
},
"defaults": [
{
"id": "74457336cc50e3d39579",
"device_type": "outlet"
},
{
"id": "50203372dc4f229d0f7f_7",
"device_type": "outlet"
},
{
"id": "50203372dc4f229d20ff_7",
"device_type": "outlet"
},
{
"id": "50203372dc4f229d0f7f_1",
"device_type": "outlet"
},
{
"id": "50203372dc4f229d20ff_1",
"device_type": "outlet"
}
],
"scenes": false,
"_bridge": {
"username": "0E:ED:A8:42:47:38",
"port": 33824
},
"platform": "TuyaWebPlatform"
}

Device Config

Additional Context

having the same issue.
[3/22/2024, 3:20:07 PM] [TuyaWebPlatform] Initializing TuyaWebPlatform platform...
[3/22/2024, 3:20:07 PM] Loading 2 accessories...
[3/22/2024, 3:20:08 PM] [TuyaWebPlatform] Cannot acquire token, waiting 185 seconds.

same issue. maybe due to their extra verification?

Same, here, let me know if there is anything I can share that would help.

Hi @doomedramen @tibtiby @Scirocco3
Good news. I found the root cause:
https://github.com/tuya/tuya-homebridge#:~:text=The TuyaSmart or SmartLife app account which registered using Google or Apple ID email need to link your phone number and use the linked phone number as username to login.
TLDR: Register another Tuya account and DO NOT use SSO login (like sign-in with Google/Apple). Just use your email to receive a code to activate the account.
Then repeat the previous steps.


Unicode error token reporting

When encountering errors at multi-byte/unicode tokens, the ojg parser reports an error message with the wrong character.
$ echo '→' | oj
*-*-* unexpected character 'â' at 1:1

I'd like to either display the correct unicode character, or omit it:
*-*-* unexpected character '→' at 1:1

Note: this same bug shows up in the "encoding/json" library and in Firefox JS, but is correct in Chrome JS.
Here is Chrome's error message:
> JSON.parse('→')

VM83:1 Uncaught SyntaxError: Unexpected token → in JSON at position 0
    at JSON.parse (<anonymous>)
    at <anonymous>:1:6

I'll take a look. If the unicode can be preserved I'll do that but failing that maybe leave it off.

Fixed in branch bug/unicode-in-error. Please give it a try and then I'll release. It was easier to fix than expected.

Thanks Peter for the quick fix.
Confirmed:
*-*-* unexpected character '→' at 1:1

Looks like the "escape" code path needs the same fix:
$ echo '"\→"' | ./main
*-*-* invalid JSON escape character '\â' at 1:3

Considered that but chose wrong. On it.

Ok, pushed that change too.

Yep, looks good.

Released


Signal difference error in tutorial example

Difference code from the introduction results in bounds error. When this line
diff = lift(x->x[1], foldl(difference, 0.0, signal))

is modified by providing a tuple as the initial fold value instead
diff = lift(x->x[1], foldl(difference, (0.0, 0.0), s))

then everything seems to work.

Oh thanks! Yes, tuple the initial value must be a tuple. I will fix this in the docs soon.

After lift API change from this commit on there are even more changes to be made. Took me some time to trace how lift changed - would really help to update the documentation examples.


init_template is not exposed correctly

init_template is exposed in init.pp but its never used anywhere else.
in instance.pp we should pull the variable in and ensure we have the sane default in init.pp
Reported on IRC by acidrainfall

+1 Thanks


ButtonBehavior.on_touch_up dispatches on_release immediately

ButtonBehavior.MIN_STATE_TIME defines a minimum time the button stays in down state. But the on_release event gets dispatched immediately. Imo it would be better to defer the event dispatching as well to ButtonBehavior._do_release, at least on demand.
Any objections?
-r

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Here is the code from button.py, written later in 2015, that implements this.
        touchtime = time() - self.__touch_time
        if touchtime < self.min_state_time:
            self.__state_event = Clock.schedule_once(
                self._do_release, self.min_state_time - touchtime)
        else:
            self._do_release()
        self.dispatch('on_release')
        return True

Closing as fixed, but if the problem persists, let us know.


Re-order labels in legend

Hi,
Thank you so much for this great package! Is it possible to re-order labels within a legend created by anno_simple?
For example, in this quick example, how would one re-order the labels in the "CD" annotation from being "C", "D", "G" to, say, "G", "D", "C" in the legend?
Thank you so much!
Best,
Matt

Hi @mbernste ,
Thanks for your question.
Actually, for this example, it's very simple, just change the code from
CD=anno_simple(df.CD,add_text=True,colors={'C':'tomato','D':'yellowgreen','G':'skyblue'},
                                            legend_kws={'frameon':False}),

to
CD=anno_simple(df.CD,add_text=True,colors={'G':'skyblue','D':'yellowgreen','C':'tomato'},
                                            legend_kws={'frameon':False}),

If colors is not given and one would like to use cmap to indicate colors. Then you can change the order of df.CD and the order of the labels in the legend would be changed too.


CI build to run tests on pull requests

Should run tests in Node and in at least one browser.  (Ideally the CI build would run in all supported browsers.)
The CI build should also load Frozen Moment with all three supported module systems.

Closing since we have a CI build.  Moving browser CI to another issue.


README update and documentation of repos.json

This does not need code review since it only impacts markdown files. I'd appreciate a quick check of the changes for readability/clarity before merging.
Note - the link in the README to repos_json_properties.md should work after merge; the file just doesn't exist on master yet, so it will be a 404 error for now.

Nothing is jumping out at me as an issue with these changes.


GH-36634: [Dev] Ensure merge script goes over all pages when requesting info from GitHub

Rationale for this change
We currently were missing maintenance branches due to pagination on GH API.
What changes are included in this PR?
Check whether the API is returning a paginated view and extend the list returned.
Are these changes tested?
I have tested locally:
(Pdb) pr.maintenance_branches
['maint-0.11.x', 'maint-0.12.x', 'maint-0.14.x', 'maint-0.15.x', 'maint-0.17.x', 'maint-1.0.x', 'maint-3.0.x', 'maint-4.0.x', 'maint-6.0.x', 'maint-7.0.x', 'maint-7.0.1', 'maint-8.0.x', 'maint-9.0.0', 'maint-10.0.x', 'maint-10.0.0', 'maint-10.0.1', 'maint-11.0.0', 'maint-12.0.x', 'maint-12.0.0', 'maint-12.0.1', 'maint-13.0.0']
(Pdb) c
Enter fix version [14.0.0]:

Are there any user-facing changes?
No

Closes: #36634

There are currently no tests on the merge_pr script around neither GH issues or any interaction with GH. We should add them but this is tracked on this PR: https://github.com/apache/arrow/issues/14802

Just for the record, other libraries like pygithub use the link header for the pagination and do a similar parsing: https://github.com/PyGithub/PyGithub/blob/de80ff4b914b9ffd47347d60b14cbf270e07f346/github/PaginatedList.py#L241

Is this something we need to add to archery somewhere too? Or not as that is using an api wrapper that handles it?

The pygithub and github3.py libraries used on archery handle pagination. I am not sure if there's any place where we should explicitly call to get next pages, etcetera but I don't think  so as  from what I've seen they just return iterators and handle that for us. Would be good though to try and refactor archery to use a single library so we are sure around those: https://github.com/apache/arrow/issues/33241


Remove RELEASING.md requirement

I'm not that convinced about MUST NOT, but let's start with stricter requirements and relax them if it's needed.

Should add language that all tests/CI are public and maybe a note that release a private to support signing of packages.

Done.
I've added a short "Exception: Go" note but I'd like to leave the actual explanation why it's okay to somebody that works with Go.

Projects can have a RELEASING.md file and document how to do it publicly when applicable (e.g, Go libs), have it link to internal docs, simply document that release jobs are private or skip adding the file altogether.

Good point - done.

@pellared @flands PTAL


Support zero input subset for raw profile

Now the raw plugin isn't supported Zero Input Subsetting.
So, I updated the raw plugin to support Zero Input Subsetting.
e.g)
$ launchable subset --target 30% --get-tests-from-previous-sessions --output-exclusion-rules > ignore-list.txt
$ pytest $(cat ignore-list.txt | grep file= | awk -F# '{print $1}' |  awk -F= '{print $2}' | sort | uniq | xargs -I% echo '--ignore %' | tr '\n' ' ')

Same as the other PR. I see some formatting changes, and it seems to me that it's reverting to the previous format.

@draftcode  Sorry, I fixed code format. Please review, again.


[tests-only][full-ci]Add api test for propfind

Description

Related Issue

Part of: https://github.com/owncloud/ocis/issues/4937

Motivation and Context

How Has This Been Tested?

locally
CI

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)
[ ] Technical debt
[x] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[x] Acceptance tests added
[ ] Documentation ticket raised:

This is a good start, to check the API response given to the user who created the space.
IMO we should also have some scenario(s) where an "ordinary" user who has a role for the space (Viewer, Editor) but is not the creator does the PROPFIND. It will be good to know that the PROPFIND works correctly for anyone with access to the space.
@ScharfViktor how much do we want to expand the Test Scenario coverage? Do it now, or later, or are we trying to keep the number of test scenarios to the minimum?

This is a good start, to check the API response given to the user who created the space. IMO we should also have some scenario(s) where an "ordinary" user who has a role for the space (Viewer, Editor) but is not the creator does the PROPFIND. It will be good to know that the PROPFIND works correctly for anyone with access to the space.
@ScharfViktor how much do we want to expand the Test Scenario coverage? Do it now, or later, or are we trying to keep the number of test scenarios to the minimum?

good suggestion. Thanks Phil. I added task to #4937


Add a Gitter chat badge to README.md

shelljs/shelljs now has a Chat Room on Gitter
@arturadib has just created a chat room. You can visit it here: https://gitter.im/shelljs/shelljs.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

done already


feat: add logging hook, rm logging from evaluation

Implements: https://github.com/open-feature/spec/pull/269
This PR removes logging from the main evaluation code path, with a couple exceptions:

unhandled errors in before/finally/error hooks (after hooks may throw in order to do validation)
unhandled errors in event handlers

(IMO these represent bugs in code, since they should be handled, and not transient errors).
It also adds a LoggingHook as per the spec link above, which adds SLF4J structured logs for evaluation objects.

Please add a section that describes the logging hook to the readme.

Please check out my addition.


do i paste all code or line by line?

do i paste all this code
git clone https://github.com/ncoop57/code-clippy-vscode
cd code-clippy-vscode
npm install
code-insiders .
or line by line?


Delay on clicking outgoing links

Thing that bugs me slightly: when you click an outgoing link on an etherpad doc you very briefly see a screen that starts "For the next trick" and then something about javascript being enabled.

This is probably as good as we can get without a lot more work. Closing for now.


custom leap function: detecting user cancellation

Well, leap is looking great :) A little issue: I wrote a custom leap function, but I don't find how to detect whether the user cancelled the jump (pressed escape) or not.
I offer a custom list of jumps to the user, and once the jump is done, I do an extra action (insert some text at that location).
However if the user cancels leap, I don't want to insert the text. At first I implemented this by checking if the cursor moved, but in corner cases the cursor may already be in the right spot...
I noticed the action option. I see that it's not called if the user cancelled. If I could invoke the jump there, or get the coordinates to jump to there, I could know whether to perform my action or not.
What would be the best way to achieve this? Thank you!

sorry, I didn't see in the documentation that I get location info for the target in the action callback... That's the solution.
Otherwise, I actually want to move the cursor (if the user didn't cancel), so I probably don't need spooky.


Revert the change to the load paths in the specs

This reverts commit de8d3a3299a18d4bfd2b419cb1303258441a862a.
This code works fine within Workstation, but it fails hard with DK. We
need to keep the DK verify checks running with Chef 15 until both of
these codebases go away. For now just revert this and roll forward with
life in master where we have the new paths

@phiggins It seems to be no impact on workstation so I think we can just go on with our lives here. This branch is a dead end


Readme entrypoint hints

Inspired by conversation from here: https://discord.com/channels/799672011265015819/1161765008347254915/1298706112605978795

Somehow your branch got out of wack with main and cant be merged. I just manually copied over your changes into main, thanks!


Hello

is there a way to check if one swipe is already open then before opening 2nd swipe first swipe gets close

SwipeLayout is just a ViewGroup. So it shouldn't know about other views.
You can resolve that logic by yourself
For example, if you use RececlerView with many items(viewed by SwipeLayout) you can ask each SwipeLayout is it opened or closed and do your logic:
var closed = 0
for (i in 0..recyclerView.childCount) {
    val view = recyclerView.getChildAt(i)
    if (view is SwipeLayout) {
        if (view.openStatus == SwipeLayout.Status.Open)  {
            closed++
            view.close()
        }
    }
}
Toast.makeText(context, "We closed $closed items", Toast.LENGTH_SHORT).show()

I'm not sure about how resolve situation with Status.Middle in right way


Weird initial building quirk

So from a clean repo clone, running pnpm install is fine up until it does npm i --prefix docs, at which point, it errors as follows

When I go into package.json and change the postinstall script from npm i to pnpm install, it gets stuck in a recursive post-install. When I then revert the package.json postinstall script, it successfully runs

so assuming JQ is installed, the bash replication would be as follows
git clone https://github.com/vitebook/vitebook.git ~/projects/vitebook_unique_new_folder
cd ~/projects/vitebook_unique_new_folder
pnpm install # fail
mv package.json package.json.old
jq '.scripts.postinstall="pnpm install --prefix docs"' package.json.old > package.json
pnpm install # CTRL+C after a couple of recursions
rm package.json && mv package.json.old package.json
pnpm install # success

When setting the postinstall to pnpm --prefix ./docs install ./docs. Could someone with a linux environment try and replicate and see if this new postinstall works on linux?

Definitely strange, not sure if it's an environment issue but I just tested a fresh clone/install on Mac and it worked.

What strikes me as odd is the fact that the postinstall script would ever work. Since pnpm keeps track of state separately from npm, then running npm i, regardless of if --prefix is used, surely should run install on the top-level package json anyway before doing the prefix?

What's the intended behaviour for running npm i --prefix docs? Is it just a case of executing npm i from within the docs folder? Or is it meant to link the docs workspace to the vitebook workspace? Because it could be worth reimplementing it to avoid any unnecessary nested npm calls. Here's a stack overflow answer that mentions about the node lifecycles


Treating the current CI failure with tests

Describe the bug
Since #144, the CI has been failing. Not sure why (because I didn't make any breaking change), but that failure is a serious blocker.
To Reproduce
Check out the affected PRs (see below). Please check the tests CI (e.g. Tests (ubuntu-latest, 3.10))
Expected behavior
The CI (to be precise, the tests) should not fail like that.
Additional context
Current list of affected PRs:

#144
#145
#146
#147
#148
#149
#150
#152

I think I've gotta remove the error handler to investigate this issue. Of course, I won't do that globally :P

Here comes a detailed explanation of my debugging:

First, I removed the following line (to get a clear message instead of a "weird error"):
https://github.com/DiddiLeija/diddiparser2/blob/2ade780009a157662a5d7c692820c1265805a287/diddiparser2/parser.py#L270-L271
Then I ran nox -s tests. The crash came from test-dsgp3-example.diddi:
nox > diddiparser2 tests/test-dsgp3-example.diddi
Traceback (most recent call last):
  File "C:\Users\Diego Ramirez\AppData\Local\Programs\Python\Python39\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\Diego Ramirez\AppData\Local\Programs\Python\Python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\Diego Ramirez\diddiparser2\.nox\tests\Scripts\diddiparser2.EXE\__main__.py", line 7, in <module>
  File "C:\Users\Diego Ramirez\diddiparser2\diddiparser2\cli.py", line 50, in main
    script.runfile()
  File "C:\Users\Diego Ramirez\diddiparser2\diddiparser2\parser.py", line 157, in runfile
    self.executeline(line)
  File "C:\Users\Diego Ramirez\diddiparser2\diddiparser2\parser.py", line 174, in executeline
    self.execute_func(line)
  File "C:\Users\Diego Ramirez\diddiparser2\diddiparser2\parser.py", line 260, in execute_func
    value = func(*fixed_args)
  File "C:\Users\Diego Ramirez\diddiparser2\diddiparser2\lib\_builtin.py", line 66, in load_extension
    ext = importlib.import_module(f"{arg}")
  File "C:\Users\Diego Ramirez\AppData\Local\Programs\Python\Python39\lib\importlib\__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'tests'
nox > Command diddiparser2 tests/test-dsgp3-example.diddi failed with exit code 1
nox > Session tests failed.

Looking at the test file, the breaking line is:
https://github.com/DiddiLeija/diddiparser2/blob/2ade780009a157662a5d7c692820c1265805a287/tests/test-dsgp3-example.diddi#L10

DiddiParser will look for a tests local package, which doesn't exist!

I hope we could fix this without changing a lot of stuff :)

I finally found a fix. I will open a PR soon.

Ok, now that this issue's fixed, let's merge the affected PRs! :tada:


Adds setting to delay start

The Adjust iOS SDK has a configuration to set a delay for their SDK https://github.com/adjust/ios_sdk#delay-start
The Adjust integration sometimes does not track the install attribution properly, and after contacting the Adjust support they suggested configuring a delay for their reporting to ensure all session parameters have been loaded properly (including the ones from Segment).
This fix will allow users to set the delay if settings.setDelay is enabled and the delay start time does not exceed 10 seconds.

The behavior now will set the delay if setDelay is enabled.
I will add a validator in the UI to ensure the delay time does not exceed 10.

After testing, saw that the Adjust SDK checks the max value for us. If the delay time exceeds the 10 it just defaults it to 10 :
[Adjust]w: Delay start of 11.0 seconds bigger than max allowed value of 10.0 seconds
2017-10-23 13:45:06.973 Segment-Adjust_Example[29963:964638]
[Adjust]i: Waiting 10.0 seconds before starting first session

Our UI will have a validation to check that nothing greater than 10 is set, but to be sure that our UI validation does not get out of sync with the code, and since Adjust defaults to 10, we remove the check here.

This should do it:


Create Lead space pattern page for website

User Story

As a [user role below]:
IBM.com Library adopter

I need to:
Understand what Lead space patterns are and where they are used on ibm.com

so that I can:
ensure my designs adhere to best practices

Additional information

This is one of the most popular patterns seen on ibm.com (and other web/digital platforms)
There is MVP WIP content available here that was created for future state prototype.

Acceptance criteria

[ ] Lara creates the In-page navigation pattern MVP page
[ ] Jenny reviews the content
[ ] Wonil reviews and approves the content before publishing

This is dependent on Lead space research over on the Ideation board. The outcome of the research story (focusing on things like height, user mindsets) will help to inform the guidelines of the pattern we publish on the site.

Draft content can be found here: https://ibm.ent.box.com/notes/725421390623

Draft mockup can be found here: https://ibm.box.com/s/mukebu21clvkkz3wofnaw06iqpo9u8t3

@RobertaJHahn The Lead space: variation research #127 on the ideation board needs to be done before the publishing can start, as discussed in the Sprint Planning meeting today. Let's move this to Sprint 20-23 to allow the research work to be done in Sprint 20-22 (starting next Monday).

Updated Sketch file with the latest thinking and the dimensions that define lead spaces here:
https://ibm.box.com/s/27nfxovqcoktv49li79vpka2h7dgh2xi
The dimensions

Where the page is on the journey, rough segments (early, mid, checkout, post-purchase, etc)
Which page type within each segment (product listing, solution, home page, etc)
Expressive VS productive lead spaces
Centered VS left-aligned

Potential MVP content suggestion

Section 1: The dimensions of a lead space
Section 2: Lead space size standards
Section 3: Content alignment standards (left, centered)

@wonilsuhibm The draft MVP version of the Lead Space patterns page content is in markdown and ready for edits as discussed.
See PR 729 for detail.
The initial design shared with you via Slack yesterday is included below.

Issue moved to carbon-design-system/carbon-for-ibm-dotcom-website #733 via ZenHub


Problems following "get-started" guide - part6

File: get-started/part6.md, CC @johndmulhausen
I have been following the "get-started" guide and am currently trying to follow part6. But one of the steps I ran has now buggered up my whole docker getstarted environment and no commands I run are now able to run as before. I keep  getting the same error,

C:\Program Files\Docker Toolbox\docker.exe: error during connect: Post
https://<IP_ADDRESS>:2376/v1.33/containers/create: x509: cannot validate certificate for
<IP_ADDRESS> because it doesn't contain any IP SANs.
See 'C:\Program Files\Docker Toolbox\docker.exe run --help'.

I was following the steps on Docker Cloud to bring your own swarm thinking that the swarm I'd created in step5 I could bring into the Docker Cloud. The last good step I was able to run was,

$ docker run -ti --rm -v /var/run/docker.sock:/var/run/docker.sock dockercloud/registration
Unable to find image 'dockercloud/registration:latest' locally
latest: Pulling from dockercloud/registration
90f4dba627d6: Pull complete
49ab6bb20a7f: Pull complete
d185660c194b: Pull complete
Digest: sha256:11dbb757fc07870824d22030cb1034c4376cf9670e272ef8b0f393e6232d315d
Status: Downloaded newer image for dockercloud/registration:latest
Use your Docker ID credentials to authenticate:
Username: robbie70
Password:
Available namespaces:

robbie70
Enter name for the new cluster [robbie70/npsxowoz8ks4vhy5bgu5gb3w7]: robbie70/getstartedlab
You can now access this cluster using the following command in any Docker Engine:
docker run --rm -ti -v /var/run/docker.sock:/var/run/docker.sock -e DOCKER_HOST dockercloud/client robbie70/getstartedlab

I then tried to connect to the Docker Cloud using the generated command above,

$ docker run --rm -ti -v /var/run/docker.sock:/var/run/docker.sock -e DOCKER_HOST dockercloud/client robbie70/getstartedlab

but get the error below,

C:\Program Files\Docker Toolbox\docker.exe: error during connect: Post https://<IP_ADDRESS>:2376/v1.33/containers/create: x509: cannot validate certificate for <IP_ADDRESS> because it doesn't contain any IP SANs.
See 'C:\Program Files\Docker Toolbox\docker.exe run --help'.

And now any command I try to run, e.g.

docker node ls,
docker services ls,
docker images ls,

gives me the same error message.
Can you please advise and maybe update the instructions to better take a new user through this process....
Kind Regards,
Robert Stannard

I have exactly the same problem following tutorial on OSX and using Docker Community Edition Version 17.12.0-ce-mac49 (21995).

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


Do not throw exceptions on iframe origin mismatch

Currently DigiTrustCommunication.js throws Error when it receives a postMessage event from a non-matching origin.
DigiTrustCommunication._messageHandler = function (evt) {
    if (evt.origin !== configGeneral.iframe.postMessageOrigin) {
        throw new Error(configErrors.en.postMessageOrigin);
    }

Instead it should probably silently fail (at least in production).

released in 1.3.1


[SPARK-31333][SQL][DOCS] Document Join Hints

What changes were proposed in this pull request?
Document Join Hints
Why are the changes needed?
To make SQL Reference complete
Does this PR introduce any user-facing change?
Yes

How was this patch tested?
Manually build and check

Test build #120783 has started for PR 28113 at commit 0d2e92e.

Merged build finished. Test PASSed.

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

cc @maryannxue

It would be worth mentioning the conflict resolving policy in case multiple hints are specified in a query. Even in case a hint can be matched against two different relations with same table name under different namespaces.

Test build #120783 has finished for PR 28113 at commit 0d2e92e.

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

Merged build finished. Test PASSed.

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

Test build #120785 has started for PR 28113 at commit 1ae78d8.

Merged build finished. Test PASSed.

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

Test build #120785 has finished for PR 28113 at commit 1ae78d8.

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

Merged build finished. Test PASSed.

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

@srowen Could you take a look of this one too? Thank you!

Merged to master/3.0

Thanks all!


KAFKA-14132: Replace PowerMock and EasyMock with Mockito in streams tests

Batch 1 of the tests detailed in https://issues.apache.org/jira/browse/KAFKA-14132 which use PowerMock/EasyMock and need to be moved to Mockito.

Hey @cadonna 👋! Adding you for visibility as these are Streams-related tests.

Thank you for the review @cadonna. I will aim to address it today!

@clolov please fix the checkstyle failures in this PR!

Hello @cadonna and thanks for the review! I will aim to address the comments tomorrow.

Build failures are unrelated:
Build / JDK 11 and Scala 2.13 / org.apache.kafka.clients.consumer.KafkaConsumerTest.testReturnRecordsDuringRebalance()
Build / JDK 11 and Scala 2.13 / org.apache.kafka.connect.integration.BlockingConnectorTest.testBlockInSinkTaskStart
Build / JDK 11 and Scala 2.13 / org.apache.kafka.connect.integration.ConnectWorkerIntegrationTest.testBrokerCoordinator
Build / JDK 17 and Scala 2.13 / org.apache.kafka.trogdor.coordinator.CoordinatorTest.testTaskRequestWithOldStartMsGetsUpdated()
Build / JDK 17 and Scala 2.13 / org.apache.kafka.trogdor.coordinator.CoordinatorTest.testTaskRequestWithOldStartMsGetsUpdated()

Thank you very much for the review and merge!


Different Target color for superior slayer monsters

Would be a nice addition to the slayer plugin especially for abyssal demons where its sometimes difficult to spot the superior in the slayer tower.

I personally don't use notifications so I can also understand why it would be useful if utilizing the slayer plugin overlays on NPC's.

This is 100% necessary IMO. I just nearly lost my Hardcore status (12 hp before I panic ectophiald) due to not being able to distinguish a Greater Abyssal Demon from a regular one and not putting Protect from Melee up. For now, I'll manually highlight all superiors with NPC Indicators, but I think this should be added as a default.

Duplicate of #1960


React 16

[x] update peer deps
[x] update dev setup – blocked
[x] with-lifecycle: support componentDidCatch

🎉


Error handler

Codecov Report

Merging #1243 (8281368) into master (55c441a) will increase coverage by 56.87%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #1243       +/-   ##
===========================================
+ Coverage   37.06%   93.93%   +56.87%
===========================================
  Files         422        7      -415
  Lines        9893       66     -9827
  Branches     1794        4     -1790
===========================================
- Hits         3667       62     -3605
+ Misses       5929        4     -5925
+ Partials      297        0      -297

Impacted Files
Coverage Δ

src/index.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

src/txmeta.ts
100.00% <0.00%> (ø)

src/txhistory.ts
100.00% <0.00%> (ø)

src/addressbook.ts
100.00% <0.00%> (ø)

src/create-account/SelectHDPath.tsx

src/app/vault/SetupVault/SetupVault.tsx

src/create-wallet/SaveMnemonic.tsx

src/ledger/WaitDialog/index.ts

src/components/common/ErrorDialog/index.ts

...c/transaction/CreateTx/ToField/AddressBookMenu.tsx

... and 411 more

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


"Produlești", "Broșteni", "Costeștii din Deal" are listed in the drop-down menu

Description:
When user writes ” Produlești”, "Broșteni" and "Costeștii din Deal" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Produlești"

Expected results
The location appears in the drop-down menu as "Sat Produlești, DÂMBOVIȚA (Produlești)".

Step 3
Write in the search bar "Broșteni"

Expected results
The location appears in the drop-down menu as "Sat Broșteni, DÂMBOVIȚA (Produlești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Costeștii din Deal"

Expected results
The location appears in the drop-down menu as "Sat Costeștii din Deal, DÂMBOVIȚA (Produlești)".


Highlight legend of active curve

Closes #177 (?)
Background color of active curve is light yellow and font is bold.

To test it, run python bootstrap.py examples/fftPlotAction.py

Closed by https://github.com/silx-kit/silx/pull/1031


Facebook sign-in feature

Thank you! I'll work on it as soon as possible. Could I also get access to the firebase project?

Thank you! I'll work on it as soon as possible. Could I also get access to the firebase project?

Yes, I can give you access. Can you please share your email id?

Thank you! It's<EMAIL_ADDRESS>

Thank you! It's<EMAIL_ADDRESS>
Now you have access to firebase.


Error code line 16

It shows an error in line 16 of the code that you provide and it says this"JSON integer<PHONE_NUMBER>6 is too large or small for an Int32. Path 'quoteSummary.result[0].defaultKeyStatistics.impliedSharesOutstanding.raw', line 1, position 1063.", already send you and email and I would really appriciate if you find a way to fix it thank you

@Lilxxchrisxx Just did an update and new version on nuget is 1.5 which will fix this error for you. Please get the latest version and confirm that the issue is resolved.

so I try and now when I try running it just open the terminal and it closes it

I don't know if is just me doing something wrong or what and thank you for your help.

According to your screenshot, you have the unit test project set as your
start up project. If you want to use this library in your existing project
then you would want to install it via nuget package but if you just want to
run this for testing then make sure the TestConsoleApp is set as your start
up project
Franklin
On Thu, Jun 29, 2023 at 2:13 AM Lilxxchrisxx @.***>
wrote:

so I try and now when I try running it just open the terminal and it
closes it
[image: Screen Shot 2023-06-29 at 2 10 55 AM]
https://user-images.githubusercontent.com/137856345/249687668-fa02053d-cadf-466d-85b2-be2d03eec41f.png
I don't know if is just me doing something wrong or what and thank you for
your help.
—
Reply to this email directly, view it on GitHub
https://github.com/ooples/OoplesFinance.YahooFinanceAPI/issues/19#issuecomment-1612483965,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXO4ZL45OZOGAHP4NAY2SLXNUMKHANCNFSM6AAAAAAZWOHPRY
.
You are receiving this because you commented.Message ID:
@.***>

alright, now it shows anothe error it says this "System.InvalidOperationException has been thrown'' its for line 49 and thank you a lot for the help

Yeah that means what it says. What symbol are you trying to use? Which
method are you using?
On Thu, Jun 29, 2023, 10:58 AM Lilxxchrisxx @.***>
wrote:

it just says this "Requested Information Not Available On Yahoo Finance"
—
Reply to this email directly, view it on GitHub
https://github.com/ooples/OoplesFinance.YahooFinanceAPI/issues/19#issuecomment-1613333482,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXO4ZM5BZBSHTIXKDZLVI3XNWJYVANCNFSM6AAAAAAZWOHPRY
.
You are receiving this because you commented.Message ID:
@.***>

I trying to use the same code you have, You can look at it in one of the comments that as the screenshot

@Lilxxchrisxx  line 49 is this:
//var chartInfoList = await yahooClient.GetChartInfoAsync("GOOG", TimeRange._1Year, TimeInterval._1Day);
When I uncomment it, it works just fine without errors so can you please give me more info so I can replicate what you are doing?


Define how to deal with options bag defined in cadl

Cadl wants to be able to define an options bag which can have parameters from different parts of the http request.  Typically we always received these as separate path parameters vs body parameters vs query parameters etc.
We will need to create special handling for this as our serialization writers etc will not know which properties are which.

This is brough out to typespec team, and this is a GA-block for typespec.


JSONB. Character with value 0x10 must be escaped

I have field with postgres type jsonb and DataType.Jsonb in map IPgBulkInsert
when I insert data I have Exception:
Caused by: org.postgresql.util.PSQLException: ERROR: invalid input syntax for type json
Detail: Character with value 0x10 must be escaped.
May be we can do it into PgBulkInsert?
Or provide util for escape this characters

I did some test.
As result it is not problem of this library. It is not valid json.
Went to correct error in generating json


wav_len dose not work in TransformerASR tesing when batch size > 1.

https://github.com/speechbrain/speechbrain/blob/97e027ce5234e49e3f4969b218f0e12ee104f2cc/speechbrain/lobes/models/transformer/TransformerASR.py#L215
My Code is like:
enc = self.tm.encode(feat, wav_len=wav_len)
preds, score = beam_search(enc , wav_len=wav_len)

In my code, wav_len does not work in testing when batch size > 1. How to run a mini-batch when testing?  Thanks.

I don't really understand. Could you give us an example of what you are trying to achieve? batch_decoding should work (it does for other recipes).

For example:
feat = torch.randn(2, 100, 512)
wav_len = torch.Tensor([0.9, 1.0])
enc = tm.encode(feat, wav_len=wav_len)
preds, score = beam_search(enc , wav_len=wav_len)

however, wav_len is ignored when self.tm.encode is called
because of:
https://github.com/speechbrain/speechbrain/blob/97e027ce5234e49e3f4969b218f0e12ee104f2cc/speechbrain/lobes/models/transformer/TransformerASR.py#L215
Can you provide a correct sample for TransformerASR  when batch_decoding?

@JianyuanZhong could you let us know when a fix can be done for this ?

Hi @Fengdalu is the issue still up?


Thank you so much for making a great plugin

I haven't gotten around to using it yet, but thank you so much for making such a great plugin!


Double check testing/building assumptions

I think the test setup I have now isn't going to actually test how the code runs in the browser. Need to make sure tests are set up correctly so that the code actually runs in the right environment.
Introduce whatever build/compile stuff will have to be done :(

The testing itself could run its own api server if we want end-to-end testing.  I don't think it needs to run in an actual browser since we're not doing anything dom-centric.

Don't want end to end testing. I was thinking the same as you, but I'm pretty sure none of the code I've written in the current PRs will actually work if you require() them into a project running in the browser since I don't have a compile step set up.
Am I wrong? I never remember how this works

ok - i'm gonna copy how they do builds from the redux guys

#15 takes care of my concerns here. Not doing browser testing, just doing a simple babel build


Created SVC algorithm to train model

Using supervised vector classification to train our ML model

We'll need to update this to try classifiers apart from svm next sprint

Got it. I'll keep that in mind for the next sprint.


how to disable update for a specific plugin

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Not a user config issue

[X] Confirm

Neovim version
NVIM v0.10.0
Operating system/version
arm64 armbian
Terminal name/version
xterm 3.44
$TERM environment variable
No response
Branch info
0.10 (Nightly)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue
i use arm64 to run nvim0.10.0 and nvimdots, good experience, thk to dev, but sniprun default edition is x86, it can't run in arm64 linux, so i compiled it and run ok，i hope use pr never update this plugin, only manual compiled update, what should i do,any idea ?
Expected behavior
if i use pr , sniprun don't update
Actual behavior
No response
Additional information
No response

add pin = true for sniprun: https://lazy.folke.io/spec#spec-versioning

add pin = true for sniprun: https://lazy.folke.io/spec#spec-versioning

thk. it works


#932 Download qualifying test report

Also includes drilling down into each score

I don't kno if this is expected, but the percentage is NaN

@lloback I've removed old data from our develop database, which was responsible for the issues experienced above.


Initial fixups for Regex

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: anyone, since build is broken
CC: @eljefe6a
Jesse, here is just fixups for checkstyle, which has broken the build, and I don't think we need the Transform suffix.

LGTM

Jenkins is onto the example integration tests, which don't touch this, so I'll take that LGTM and commit.


Allow to config no-password truststore

Motivation
Supprot to config truststore which password is empty ""
Modifications

when password is null, load as ""

Verifying this change

add test method to verify tls success

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[x] no-need-doc

That's a little enhance, doesn't need doc

@codelipenghui @lhotari @michaeljmarshall @merlimat @315157973 @hangc0276 PTAL, thanks

I don't believe we're allowed to add binary files to our repo.

Assuming it is not possible to create this test file from source then to me this test case is reasonable.
It would be good to have a description of how these two keystores are created. Then we can determine if it is relatively simple to build.

I don't believe we're allowed to add binary files to our repo. @dave2wave and @eolivelli - can you confirm this for me?

We do have binaries when it make sense. eg: PNG image for diagrams. I believe that if you want to test with a keystore and be able to read it, there's no other way than to include it.
Or if you want, just encode in base64.. which kind of demonstrate that bytes are bytes..

@codelipenghui @lhotari @merlimat @315157973 @hangc0276 PTAL, thanks
@michaeljmarshall  PTAL agian, thanks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

ping @michaeljmarshall

Hi, just a tip for new committers. When you merge a PR, it's better to copy and paste the PR description like

Otherwise, the commit log would only contain all the single commits like https://github.com/apache/pulsar/commit/53ad936b28035a7ae81e80c12e924b3f03a573e9
See a correct commit like: https://github.com/apache/pulsar/commit/40ac7934ed567b2b9f16dbd7f937f7973dcddc21

Hi, just a tip for new committers. When you merge a PR, it's better to copy and paste the PR description like

Otherwise, the commit log would only contain all the single commits like 53ad936
See a correct commit like: 40ac793

Thank you, I haven't known that before, sorry for my fault.

Never mind. There was a PR that adds the guideline for merging a PR: https://github.com/apache/pulsar/pull/12988. It will be included in Pulsar 2.10.0 website.


feat(website): create website

Hey everyone,
I'm currently working on a nice little website for the project so we can promote it a little bit better and also because, awesome projects deserve awesome websites. It's really just about making a nice landing page with all the needed information to get started and some links to the GitHub repo and the docs (which we will publish on gh-pages through this repo).
Here's a screenshot of the work-in-progress:

Would love to hear your thoughts! Feedback is highly welcome. Also, it'd be nice if we could brainstorm a little bit about the content of the site. Maybe you have some cool ideas.
In addition to that, @Nevraeka is so kind to help us out and works on a very nice logo too, based on this scribble:

Here's what it looks like on mobile btw:

I think it looks awesome! :+1:
I'd be really cool to see some changelog images (not just links to changelogs) in the design as well, or just as examples. I'm not a design guy though, so take my ideas with a grain of salt! :P

@kbknapp Yes, I totally agree. We should somehow get some changelog examples in there so people can actually see what they get.

Just wanted to let you know that I put together a "use clog directly from the browser"-demo site. It's build with nickel.rs and uses clog as a library. It's not shiny and pretty at this point but we can iterate on it to have it nicely integrated into the planned clog project site.
I'm going to start writing a blog post soon that leverages the work that I've put into this. Once that's done I'll be pushing the code forward to be useful for a proper project site. E.g. let the user configure the various options of clog through the interface.
I put a preview up here: https://nickel-test.herokuapp.com/
The code is here: https://github.com/thoughtram/clog.thoughtram.io
@kbknapp I know your Rust is much better than mine, if have some time to spare to look through the code, give feedback or send PRs to improve it, I'm always grateful to learn from you.

@cburgdorf That's super cool! I've just looked briefly though the repo and it looks good! Something that's really impressive to me is how short and concise the code is to do something like this using clog and nickel.rs! I'm looking forward to this blog and final product!
I'll keep an eye on the issues section of that repo and help out where I can of course! :+1:
Also, I think incorporating something like this into the clog web page would be perfect! I'd put up a very short somewhat curated list of repositories known to use conventional commit messages though (perhaps just 3 or so), since running it on a repo without those would be of limited use :P And it would also add a few different changelogs to see versus the default angular one.

Love your collaborations here :heart:

@Nevraeka Hey Erik I might be wrong, but it seems like the logo is not entirely centered. Also, do you think we could squeeze it a bit vertically? Would love to hear your thoughts on that!

 Lost the whole repository while upgrading my machine (because I didn't make a proper back up of it). So I'll have to rebuild the site again  😒

The new repository for the website is here: https://github.com/clog-tool/clog-tool.github.io
Pushing to master should serve the files of this repo statically at clog-tool.github.io


Where to get labels of evaluation dataset?

As the competition has finished, where can I find the labels of evaluation dataset?
I hope to use this dataset in my research work.
Thanks a lot !

Which task you are referring to?
Tasks 2, 4 and 5 have released reference annotations for evaluation dataset. For task 1, there is no plan of releasing the reference labels for evaluation or public leaderboard datasets (see http://dcase.community/challenge2018/task-acoustic-scene-classification#reference-labels).

FYI task 3 does not currently plan to release the eval labels.

Thanks for your reply.


Node Responds but isn't recognized

After changing to ZWaveZJ2MQTT, I had more failed devices than not. I cleared everything out and started fresh. I'm unable to get my  Dome Z-wave Water Valve Shut Off model DMWV1 to add back in again. It was one of the working devices after conversion, too. It gets included and then very shortly after it goes dead. I've tried restarting it, resetting it, included and excluded it 3 times, healing it, interviewing it, etc. It keeps coming back in and remaining only partially setup.
Here's the log showing a ping and re-interview
2022-08-14 14:56:36.501 INFO ZWAVE: Calling api pingNode with args: [ 10, [length]: 1 ] 2022-08-14T18:56:36.504Z CNTRLR » [Node 010] pinging the node... 2022-08-14T18:56:40.635Z CNTRLR [Node 010] The node did not respond after 1 attempts, it is presumed dead 2022-08-14T18:56:40.653Z CNTRLR [Node 010] ping failed: Failed to send the command after 1 attempts (Status No Ack) (ZW0204) 2022-08-14 14:56:40.656 INFO ZWAVE: Success zwave api call pingNode false 2022-08-14 14:56:58.777 INFO APP: GET /health/zwave 200 3.051 ms - 1876 2022-08-14 14:57:17.201 INFO ZWAVE: Calling api refreshInfo with args: [ 10, { resetSecurityClasses: false }, [length]: 2 ] 2022-08-14T18:57:17.225Z CNTRLR [Node 010] Beginning interview - last completed stage: None 2022-08-14T18:57:17.226Z CNTRLR [Node 010] new node, doing a full interview... 2022-08-14 14:57:17.228 INFO ZWAVE: Node 10: interview started 2022-08-14T18:57:17.231Z CNTRLR » [Node 010] querying protocol info... 2022-08-14 14:57:17.254 INFO ZWAVE: Success zwave api call refreshInfo undefined 2022-08-14T18:57:17.313Z CNTRLR « [Node 010] received response for protocol info: basic device class: Routing Slave generic device class: Binary Switch specific device class: Valve node type: End Node is always listening: true is frequent listening: false can route messages: true supports security: false supports beaming: true maximum data rate: 100000 kbps protocol version: 3 2022-08-14 14:57:17.318 INFO ZWAVE: Node 10: interview stage PROTOCOLINFO completed 2022-08-14T18:57:17.321Z CNTRLR [Node 010] Interview stage completed: ProtocolInfo 2022-08-14T18:57:17.322Z CNTRLR » [Node 010] pinging the node... 2022-08-14T18:57:21.457Z CNTRLR [Node 010] The node did not respond after 1 attempts, it is presumed dead 2022-08-14T18:57:21.466Z CNTRLR [Node 010] ping failed: Failed to send the command after 1 attempts (Status No Ack) (ZW0204) 2022-08-14T18:57:21.467Z CNTRLR » [Node 010] querying node info... 2022-08-14T18:57:21.468Z CNTRLR » [Node 010] pinging the node... 2022-08-14T18:57:25.631Z CNTRLR [Node 010] The node did not respond after 1 attempts, it is presumed dead 2022-08-14T18:57:25.638Z CNTRLR [Node 010] ping failed: Failed to send the command after 1 attempts (Status No Ack) (ZW0204) 2022-08-14T18:57:25.640Z CNTRLR [Node 010] Interview attempt (1/5) failed, node is dead. 2022-08-14 14:57:25.642 ERROR ZWAVE: Interview of node 10 has failed: The node is dead 2022-08-14 14:58:58.809 INFO APP: GET /health/zwave 200 2.707 ms - 1876

So this may not be the best place to post this issue.  The Addon-ZwaveJS2MQTT is just a wrapper and Docker setup for ZWaveJS2MQTT which relies upon ZWave-JS.
Your particular issue is likely related to the Z-WaveJS Driver, thus you will likely get better resolution from here.
https://github.com/zwave-js/node-zwave-js/issues
However, note that Z-Wave-JS JUST released a major version (version 10.0.x).  This major release has not yet trickled into this add-on as of the time of your posting.  So when posting for help there, you might want to specify that you are still on 9.x.x.  It would be helpful for them to know exactly which version.  To get this, go into Z-Wavejs2MQTT webpage, at the top right, hover over the blue Info button.

and that will tell you your version.

You will also likely be asked for additional logs so the developers can help troubleshoot your issue, but keep in mind, that specifying that you are on 9.x vs 10.x will be very helpful right now since they are dealing with mostly influx of issues around the recent 10.x release.


[macOS]  remove blackhole 2ch

blackhole-2ch upstream site is now blocked as malicious and ssl got expired which makes it uninstallable.

/azp run macos12


Updated sql_database documentation for resource usage

Description
Updated sql_database documentation for resource usage

Related Issues

Closes #2069

very good Aman! I just noticed a moment (not related to your PR), add please extra line between subtitle and text body, otherwise they are on the same line.


[string_view] Use member-types instead of hardcoding char type

This change replaces uses of char with string_view::value_type.
This may allow making string_view a template and support different
value types.
Updates #124, #734

We do not plan to make string_view a template and making these changes sends wrong message. C++ 17's string_view is a template if you need one.


driver name org.gluster.glusterfs not found in the list of registered CSI drivers

I can create a new PVC, it works without any issue:
kubectl describe pvc test2 -n test
Name:          test2
Namespace:     test
StorageClass:  glusterfs-csi
Status:        Bound
Volume:        pvc-46e094b7-2ddf-11e9-a318-002590927952
Labels:        cattle.io/creator=norman
Annotations:   field.cattle.io/creatorId=u-jtvds
               pv.kubernetes.io/bind-completed=yes
               pv.kubernetes.io/bound-by-controller=yes
               volume.beta.kubernetes.io/storage-provisioner=org.gluster.glusterfs
Capacity:      1Gi
Access Modes:  RWX
Events:        <none>

But when i want to use it on a container i get this issue:
Events:
  Type     Reason       Age                From                  Message
  ----     ------       ----               ----                  -------
  Warning  FailedMount  14m (x27 over 1h)  kubelet, myhostname  Unable to mount volumes for pod "test-56b7889987-wklxs_test(ec1d4cd0-2de6-11e9-877e-00259092799a)": timeout expired waiting for volumes to attach or mount for pod "test"/"test-56b7889987-wklxs". list of unmounted volumes=[test]. list of unattached volumes=[test default-token-6k4fj]
  Warning  FailedMount  3m (x43 over 1h)   kubelet, myhostname  MountVolume.MountDevice failed for volume "pvc-46e094b7-2ddf-11e9-a318-002590927952" : driver name org.gluster.glusterfs not found in the list of registered CSI drivers

I use the kubernetes cluster under rancher.
Kubernets version is  v1.12.3
I hope someone can help me.

@muecahit94 current GCS support kube 1.13+ version
to deploy in kube  1.12 may be you need to update  driver register path in gcs deployment
from var/lib/kubelet/plugins_registry to var/lib/kubelet/plugins in https://github.com/gluster/gcs/blob/master/deploy/templates/gcs-manifests/gcs-fs-csi.yml.j2

there is plugins_registry at three places:
cat  templates/gcs-manifests/gcs-fs-csi.yml.j2 | grep var/lib/kubelet/plugins_registry
              value: /var/lib/kubelet/plugins_registry/org.gluster.glusterfs/csi.sock
            path: /var/lib/kubelet/plugins_registry/org.gluster.glusterfs
            path: /var/lib/kubelet/plugins_registry/

when I change this part:
        - name: registration-dir
          hostPath:
            path: /var/lib/kubelet/plugins/
            type: Directory

I got this error on csi-nodeplugin:
I0212 17:14:04.271438 1 main.go:111] Version: v1.0.1-0-g27703026
I0212 17:14:04.271552 1 main.go:118] Attempting to open a gRPC connection with: "/plugin/csi.sock"
I0212 17:14:04.271570 1 connection.go:69] Connecting to /plugin/csi.sock
I0212 17:14:04.272055 1 connection.go:96] Still trying, connection is CONNECTING
I0212 17:14:04.272383 1 connection.go:93] Connected
I0212 17:14:04.272401 1 main.go:126] Calling CSI driver to discover driver name.

I0212 17:14:04.277943 1 node_register.go:55] Calling CSI driver to discover node ID.
I0212 17:14:04.277976 1 connection.go:137] GRPC call: /csi.v1.Node/NodeGetInfo
I0212 17:14:04.277984 1 connection.go:138] GRPC request: {}
I0212 17:14:04.279040 1 connection.go:140] GRPC response: {"node_id":"hostname"}
I0212 17:14:04.279609 1 connection.go:141] GRPC error: <nil>
I0212 17:14:04.279616 1 node_register.go:63] CSI driver node ID: "hostname"
I0212 17:14:04.279696 1 node_register.go:86] Starting Registration Server at: /registration/org.gluster.glusterfs-reg.sock
I0212 17:14:04.279778 1 node_register.go:93] Registration Server started at: /registration/org.gluster.glusterfs-reg.sock
I0212 17:14:04.280374 1 main.go:84] Received GetInfo call: &InfoRequest{}
I0212 17:14:04.282779 1 main.go:94] Received NotifyRegistrationStatus call: &RegistrationStatus{PluginRegistered:false,Error:plugin registration failed with err: error updating CSI node info in the cluster: error adding CSI driver node info: driverNodeID must not be empty,}
E0212 17:14:04.282897 1 main.go:96] Registration process failed with error: plugin registration failed with err: error updating CSI node info in the cluster: error adding CSI driver node info: driverNodeID must not be empty, restarting registration container.

is there no way to deploy gcs on a kubernetes version 1.12.x ?
Pls give me some tips, what i could try out.


Docs latest

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

For latest version of cilium there is no warning specifying in the documentation regarding it as a unstable/development version. The issue focuses on fixes it for the latest version by adding warning at the top of documentation.
Fixes:  #29969

@joestringer am I on the right direction? Can we implement the similar done in index to all the doc files?


Issue 245 fix

Motivation
Fix the issue. In mobile mode, instead of display: none, do display: flex; flex-direction: column on sitemap. Also added margin
Test Plan
Tested in local build
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@richardzcode Thanks for this PR. It looks reasonable to me.

For those that actually want to not have the footer on mobile, can use an override in a custom CSS file (e.g.,static/css/custom.css)

fixes #245


Add link to wallet for succesful withdrawals and deposits page

Is your feature request related to a problem? Please describe.
It feels natural to want to see your balance after making a wallet transaction, so it would be convenient to make it easy navigate to your wallet after successful deposits and withdrawals.
Describe the solution you'd like
Have link for Go to wallet to the right of BOLT11 information on successful deposit or withdrawal.
Describe alternatives you've considered
Going to the menu options to check the wallet seems like a longer step
Additional context

The image is not working. I'm not sure if that clarifies things but I'm not 100% certain what you're referring to. Do you mean attached wallet?

mean at link to https://stacker.news/wallet page.

Your wallet balance is in the upper right hand corner. Is that not enough?

Yes, a '"go back to wallet" link/ button.
As it is, it works, nothing to fix per se. This is a suggestion for a more fluid UX.

Are you hkarani on SN? You've earned some sats for this?

Thanks for the sats💛


ArgumentException when parsing a version

Getting the following exception when trying to parse the version of System.IdentityModel.Tokens.Jwt
Using Libyear: 3.0.0
<package id="System.IdentityModel.Tokens.Jwt" version="<IP_ADDRESS>3061554" targetFramework="net461" />
Sorry, an unexpected exception has occurred: System.ArgumentException: '<IP_ADDRESS>3061554' is not a valid version string.
Parameter name: value
   at NuGet.Versioning.SemanticVersion.Parse(String value)
   at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable`1 source, Func`2 keySelector, Func`2 elementSelector, IEqualityComparer`1 comparer)
   at LibYear.Lib.FileTypes.XmlProjectFile..ctor(String filename, String elementName, String[] packageAttributeNames, String versionAttributeName) in D:\Code\LibYear\src\LibYear.Lib\FileTypes\XmlProjectFile.cs:line 29
   at LibYear.Lib.FileTypes.PackagesConfigFile..ctor(String filename) in D:\Code\LibYear\src\LibYear.Lib\FileTypes\PackagesConfigFile.cs:line 7
   at LibYear.Lib.ProjectFileManager.<>c.<FindProjectsInDir>b__3_4(FileInfo f) in D:\Code\LibYear\src\LibYear.Lib\ProjectFileManager.cs:line 36
   at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()
   at System.Linq.Set`1.UnionWith(IEnumerable`1 other)
   at System.Linq.Enumerable.UnionIterator`1.FillSet()
   at System.Linq.Enumerable.UnionIterator`1.ToList()
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at LibYear.App.Runner.Run(IReadOnlyList`1 args) in D:\Code\LibYear\src\LibYear\Runner.cs:line 34
   at LibYear.App.Program.Main(String[] args) in D:\Code\LibYear\src\LibYear\Program.cs:line 24

I tried resolving this by changing the dependency to a newer version, now I'm getting:
Sorry, an unexpected exception has occurred: System.ArgumentException: '<IP_ADDRESS>' is not a valid version string.

Thanks for this bug report -- I believe the official NuGet library being used to parse versions only supports 3 part semantic versioning.
It's on my list to upgrade to the latest version of the NuGet libraries, which may be able to handle this case, but I have the feeling it's more likely going to be a case of no longer parsing semantic versions, which I'm not really inclined to do.
If that's the case, I think maybe just skipping unsupported version formats is probably preferable.

Resolved by #20, will get a new release out shortly -- thanks, @mchughmk!