First Ring Suppressor...UPDATE!

Hello,
I noticed this Call Attendant works well, but first ring still happens.  This annoys me when I am woken up at 4am to a tele marketer single ring...lol.  Sometimes two.
  BUT I did a ton of digging and found this from a man who worked for a company back in the 90s, and he designed this.  Since they no longer deal with this technology in his part of the world(all digital and voip) he just gave this to me hoping I could run with it.  BUT, I have not attempted it yet.  Thought one of the people in here might be able to put it to use?
It is a BOM and schematic for a first ring supressor.  I sent this to PCB way, they said it would cost less than 4 bucks a piece in small batches if anyone wanted to make them.  If you do credit Brad Peeters would be great...especially if you sell them.
I think it may need updated because voltages I do not believe are the same today on POTS as they were in the 90s.
Anyway.  enjoy.
FRS_BOM.xlsx
FRS Schematic.PDF

I fond this :
https://www.airplgri.com/frs22100-first-ring-suppressor-silencer-for-robocall-blocking-services-p-449385.html?products_id=449385:c05cf5a1c0745d85af67593931ac8314
3.25$ USD. Look nice... but in my case, I'm from Canada and it will cost me 20$ USD shipping...

Hi! Did you have a chance to try it? I belive the voltages will be still the same as was in the 90s.

No, I cannot source one of the components.  Looks like it was deprecated many many years ago.  That is, I can find the part, but it is expensive...40 bucks.  So I did not look any further.  HOWEVER, I did need a new phone so I just got one with a option to block the first ring. Which works perfectly.  This in conjunction with Call Attendant...I have had zero actual bull crap phone calls and only once in a while do I hear a single ring.  AND since the caller id still fires even though it stops the ring audibly I can see if there is a number accidentally getting swept aside by the system.  I do still get phone calls from people in my area, but they are not actually the ones calling.    BUT after adjusting the rules and black list thing I seem to catch 90-93% of the bs.
It has been a god send.
As to the device you found.  That is what I was looking for originally.  I could not find it in stock.  If you can get it, the price is great.
Let us know how it goes.  Not everyone can drop 100 bucks on a new 3 phone system...and I would not have except my other phone's "main base" took a dump on me.  I had it working in my home for 15 years so it was probably time.
Good luck.

Just a warning for anyone that tries to build this. The first ring suppressor listed relies on a microcontroller (PIC12C508) to operate. It'd be easy to substitute an in-production part for that. However, the microcontroller needs firmware to operate. Without the firmware, the schematic isn't going to do a lot sadly.

Hello. Has anyone removed the first ring with a software or hardware solution?
In the README, it says that intercepts robocallers and blocked numbers at or before the first ring, how is this possible?

As I understand it.  The data that contains number and name(id caller) is transmitted before the first ring...but based on the ticks of the switching mechanics(whatever hardware makes phones work) sometimes that information is sent before the first ring actually makes a sound and sometimes during the ringing.  If things aren't working right on my phone line sometimes it doesn't send at all or until a 2nd or 3rd ring.
BUT typically the packet of data or whatever that is sent is sent during that first or before the first ring.  The software in this attendant software intercepts that data just like an id caller and if it matches white or black listed numbers it handles it accordingly.
Does that make sense?
I am uncertain this software will work on voip, but it does work on pots and digital phone...which is different than voip.
Also has anyone blocked the first ring?  YES.  Some folks buy the el-cheapo robo blockers from amazon that will suppress the ring.  ID info is still sent normally but the noise is silenced.  Some modern phones have the ability to silence the first ring as well.
I found a piece of hardware(design) from the 90s from a man who gave it to us to use to create our own suppressors but when I talked to PCB way about it, we could not source 1 of the components, and that component would need to be programmed with a IC programmer that is beyond my skills.
BUT, it is possible to silence all your phone rings, then add a software ringer that rings on a speaker attached to the GPIO of the pi that runs this software.  Perhaps a possible solution for those with only one phone  or no need to hear it in multiple places.
It would be fairly easy to also attach other Micro Controllers like an ESP32 to ring remotely.  BUT it is a longer process than I am willing to do.
I use this software, it works great.  I did not add in anything other than a few custom messages, and a couple number rules.  And a light to flash when ringing.  Other than that, it is stock...and just works.
alright, this was longer than I expected.  I had coffee.  :)
Regards.


[WIP] add more vep plugins

Starting work to add GeneSplicer,MaxEntScan and dbscSNV plugins to VEP
work to be done:

[x] bump vep version to 86 to include all needed plugins
[x] bump vep version in cloudbiolinux
[x] add ggd recipe for dbscSNV to Cloudbiolinux
[x] add function to call dbscSNV plugin
[x] review genesplicer recipe in Bioconda
[ ] add genesplicer training sets to data repo for download
[x] add ggd recipe for genesplicer training sets to Cloudbiolinux
[x] add function to call genesplicer plugin
[x] add MaxEntScan recipe in Bioconda
[x] add function to call MaxEntScan plugin
[x] add option to select which plugins vep uses
[ ] check for compatibility with GRCh37/hg19/hg38

I've also added some comments in the code where I could use some help to further this effort.

cloudbiolinux related work in chapmanb/cloudbiolinux#234

Hi @chapmanb
Just a quick question,
Is there a reason you use the --fields argument in VEP? I don't really see any benefits from it.
Thanks
M

Bioconda PR's
bioconda/bioconda-recipes#3123
bioconda/bioconda-recipes#3089

Hi @chapmanb,
I think the work for this is almost finished. All that's left is a function call to get the absolute path of an executable, could you help me with that?
I was also wondering if you could proofread the changes I made and see if everything is up to standard.
Thanks!
M

Matthias;
Thanks for all this brilliant work. Things are looking great and happy to merge this when ready in preparation for a 1.0.1 release with the new VEP. This will get you the shared directory where genesplicer is located for getting files:
genesplicer_dir = os.path.dirname(os.path.realpath(config_utils.get_program("genesplicer", data["config"])))

You use config_utils to get the path inside anaconda, then realpath to resolve the symlinks.
My only other suggestion is to refactor all of the "plugins" support to be less hard coded with ifs. I'd add a dictionary mapping plugins to the functions you call and build up two general arguments plugin_args and plugin_fields based on if each plugin is enabled and available.  I can handle that post-merge to be a bit cleaner as well so no big deal there.
To answer your earlier question: the purpose of the fields argument is to ensure a consistent set of outputs in the VEP annotations.
Thanks again for all this work.

Hi Brad,
Thanks for going through it all, I really hope I didn't mess anything up. I just pushed another commit that adds the installation path of the plugins, per your suggestion.
Do I understand correctly that you will handle the hardcoded if statements post merge? I agree the hard coded if's aren't really a nice way to check for active plugins, but I didn't really know how to handle it otherwise.
Glad to be of service!
M

Matthias;
Definitely, I'm happy to do the cleanup. I'm also working on generalizing transcript support to fix some other current issues so can do this as part of this. Thank you again for this contribution and all the hard work putting this together.

Matthias -- I checked in updates and cleanups for this so hopefully all will work as expected for your pipeline. I'm hoping to release a new bcbio version early next week with this included, so would appreciate any testing you can do before then. Thanks again.

Hi Brad,
Thanks for merging! I'll run some tests today and get back to you with the results.
M


Add com.fasterxml.jackson.datatype.jdk8 in parent first packages v0.10.31

embulk-core has not have com.fasterxml.jackson.datatype.jdk8 in parent_first_packages.properties while :

parent_first_packages.properties has had other com.fasterxml.jackson.???.
embulk-core has had jackson-datatype-jdk8 in dependencies until v0.10.31.

It has caused no problems when all "v0.10-ready" plugins are using only Jackson 2.6.7. But, embulk-output-td needed to use Jackson 2.8 and 2.9.
embulk-output-td has caught up with Embulk v0.10 API/SPI in its 0.8.0, and it needed to have jackson-datatype-jdk8 in addition.

https://github.com/treasure-data/embulk-output-td/pull/111

By this addition, Embulk v0.9 - v0.10.31 loads jackson-databind:2.6.7 from embulk-core while embulk-output-td tries to use its embedded jackson-datatype-jdk8:2.9.10. We observed that the mismatch caused NoSuchMethodError like :
Caused by: java.lang.NoSuchMethodError: com.fasterxml.jackson.databind.type.ReferenceType.upgradeFrom(Lcom/fasterxml/jackson/databind/JavaType;Lcom/fasterxml/jackson/databind/JavaType;)Lcom/fasterxml/jackson/databind/type/ReferenceType;
	at com.fasterxml.jackson.datatype.jdk8.Jdk8TypeModifier.modifyType(Jdk8TypeModifier.java:40)
        ...

A "caught-up" should basically work with the latest Embulk v0.9 and recent v0.10s so that users/plugin-developers do not need a one-way jump.
This hotfix is to make it work with v0.10.31(.2).
We release only Maven artifacts (embulk-core, embulk-deps, ...) for v<IP_ADDRESS>.

Thanks!


Issue with Unichart Generating Redundant Words in Non-4.28.1 Transformers

I read your interesting research with great interest. Thank you for conducting such excellent work. I have a minor question to ask. I noticed in the paper that it's mentioned, "Unichart will generate meaningless redundant words when the output is too long." I've found that Unichart functions correctly only with the Transformers version 4.28.1, and when using versions other than this, it tends to produce redundant words. I'm curious to know if the Unichart version was set to 4.28.1 in your setup.

Directly use the demo provided could verify the phenomenon.
As I remember, I use transformers==4.31.0. But I found in most cases the output is meaningful.
And what do the word "correctly" mean? Does this mean Unichart will work totally wrong when the transformers are in a different version? I do not observe such a phenomenon in my machine.


src: error-less ctors

Constructors do only necessary initialization with no possible error.
Errors are deferred until some actual operation is being done.
This allows to initialize all parts of fn.Client at once with no error.
We need this to be able to create a shared factory function of fn.Client for multiple commands.
Without this such a factory is not possible because the function could fail while initializing something that won't be actually used.
For example: func create doesn't need k8s configuration set up, but the command would still fail because of it.

@zroubalik I updated description.

This allows to initialize all parts of fn.Client at once with no error.
We need this to be able to create a shared factory function of fn.Client for multiple commands.

meaning instead of newCreateClient newBuildClient newDeployClient newDeleteClient newInfoClient newListClient newInvokeClient newRepositoryClient newRunClient which are partially initializing fn.Client,
there would be just single factory function.

/unhold


Cannot read property 'compileOptions' of undefined

Hi @davidrekow,
When executing the example provided on the documentation of this project as described below:
var sass = require('sass-stream'),
    fs = require('fs');

// As an intermediary step in a flow
fs.createReadStream('sprites')
    .pipe(sass())
    .pipe(fs.createWriteStream('css/stylesheet1.css'));

// As the terminus in a flow
fs.createReadStream('sprites')
    .pipe(sass({
        output: 'css/stylesheet1.css'
    }));

I'm stuck with the the following error:
$ node sass-stream.js
/.../node_modules/sass-stream/index.js:64
        compileOpts = opts.compileOptions || {},
                          ^
TypeError: Cannot read property 'compileOptions' of undefined
    at module.exports (/.../node_modules/sass-stream/index.js:64:27)
    at Object.<anonymous> (/.../sass-stream.js:6:11)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)
    at node.js:814:3

I'm not sure how to deal with this error, do you have any suggestions ?
Thank you in advance.

Although it may seen obvious that the error is due to no params passed to the function saas(), when I do it, eg: .pipe(sass({})) I'm then stuck with another meanless error:
$ node sass-stream.js
events.js:85
      throw er; // Unhandled 'error' event
            ^
Error: EISDIR, read
    at Error (native)

Not sure, is this because he can't find the scss ?
Thank you.

When I correct all this errors (at least it looks like the absolute paths are correct)
I get into another error:
var sass = require('sass-stream'),
    fs = require('fs'),
    path = require('path');

var dir = path.join(__dirname, '/sprites/scss/_sprites.scss')
console.log(dir)

var dest = path.join(__dirname, '/css/style.css');
console.log(dest);

// As an intermediary step in a flow
fs.createReadStream(dir)
    .pipe(sass({}))
    .pipe(fs.createWriteStream( dest));

error:
$ node sass-stream.js
.../sprites/scss/_sprites.scss
.../css/karai.css
node(99192,0x7fff7023acc0) malloc: *** error for object 0x10296b040: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug
Abort trap


Wrong border on small screen for activity and wallet pages

1: On activity and wallet pages there should not be right border on small screen here:

2: The top right corner should not have white background

@bonustrack Can I look at this? Also there is a border-radius, that probably shouldn't be there for small devices too.

Sure @alokpant you can look on it, and you are right, small screen should not have border radius good catch :)

@bonustrack Is there a bypass authentication with Steemconnect? I do not really use Steemconnect services. So to fix this one issue, I do not want to locked in Steemconnect services as it asks for Phone authentication.

Also would be great to include where to visit in browser after:
yarn dev

Like visit 'http://localhost:3000' or
print in console: App is available at 'http://localhost:3000' or any other port if 3000 is taken.

My mistake, I should have asked more question before trying to solve the problem. Looks like Wallet and Activity pages are independent of user-login.

+1 for structure of CSS. It is really nice.


Root level files get not path-translated for Cygwin (and msys) in the generated shell scripts

The generated shell scripts in 7.1.1 and also the revamped ones in 7.2-20210722005157+0000 only translate paths from Linux notation to Windows notation if they are absolute and starting with an existing directory.
While it might be questionable why only existing directories are translated, it is probably the most common case.
But if you have a root level file as argument like /LICENSE.txt, this is not translated to a windows path.
On msys it works nontheless if you didn't disable the automatic path translation msys already does, but the Gradle generated script also there does not do its work.
The reason in the 7.2 snapshot is, that [ -d "$t" ] is tested and thus only directories are considered.
If this check would be changed to test for existence [ -e "$t" ] it would also work properly for files.
Your Environment
Gradle 7.2-20210722005157+0000

@big-guy could you please take a look at this as well, as you were touching the relevant bits recently?

I believe this is fixed in https://github.com/gradle/gradle/pull/17854


apm: Update to v2.6.5-atomic.1.1

Requirements for Contributing a Bug Fix (click to expand):

Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
The pull request must only fix an existing bug. To contribute other changes, you must use a different template. You can see all templates at https://github.com/atom/.github/tree/master/.github/PULL_REQUEST_TEMPLATE.
The pull request must update the test suite to demonstrate the changed functionality. For guidance, please see https://flight-manual.atom.io/hacking-atom/sections/writing-specs/.
After you create the pull request, all status checks must be pass before a maintainer reviews your contribution. For more details, please see https://github.com/atom/.github/tree/master/CONTRIBUTING.md#pull-requests.

Identify the Bug

Includes this fix (https://github.com/atom-community/apm/pull/124) so apm's bundled copy of node-gyp can download the 'headers' tarball needed to compile native C/C++ addons.
See another PR that did the same thing at an apm fork for more explanation: https://github.com/pulsar-edit/ppm/pull/43
Description of the Change

Bump apm to the latest version.

Includes https://github.com/atom-community/apm/pull/124 that helps apm compile native C/C++ addons after the sunset.
Had to get apm off of GitHub, since I don't have credentials to publish the apm package to the npm registry. Prepared a commit (https://github.com/atom-community/apm/commit/3952d49a1c505f79f6e9cd52d69aecb99c76a1d0) at apm's git repo where there are no un-needed files, and specified this commit as a GitHub.com tarball URL, so there should be no practical difference from getting the package tarball implicitly from the npm package registry vs this tarball from apm's GitHub repo.
Also bumps apm's dependencies, such as npm to the latest 6.14.18, since any time you change any given dependency, npm install automatically fetches the latest sub-dependencies under that dependency.

Alternate Designs

Could do this same apm update but keep all the sub-dependencies unchanged (for a more minimalist change).
I ran the Atom repo's CI tests, and there was no difference in terms of passing or failing Atom tests, so I don't see the point in staying on the older sub-dependencies. Might as well get the latest npm 6.x, in my opinion.
Possible Drawbacks

N/A, nothing I am aware of. It's a one-line fix in apm, from a fully dead URL (atom.io has been sunset) to a live one.
Verification Process

CI passes 100% at the apm repo, and no new tests fail here.
In fact, the only failing test here ( one failing assert in the settings-view package tests) happens also on master branch without this fix (if you can get master branch to even build, that is! https://github.com/atom-community/apm/pull/124 fixes an issue that can prevent you building Atom if you don't know how to work around it!).
I was even able to build upstream Atom, and reproduce the failing CI test with that built version of Atom, and with official Atom 1.60.0 (the last version with Electron 9, which I consider the most stable upstream Atom release).
So, the CI failure is unrelated. Everything else passes in CI.
Release Notes

N/A
Commit Message
Commit message with more notes (click to expand):
Includes an updated Electron headers dist URL, as the atom.io one is now down as a result of the Atom sunset.
Should allow apm to build native C/C++ packages again.
NOTE:
Fetching apm from its GitHub repo directly, instead of from the npm package registry as before, as I do not have access to publish the '@atom-ide-community/atom-package-manager' package to the npm package registry.
This is the next best option. The commit this is pointing to has been prepared equivalently to the process for uploading to the npm package registry, by running npm install and npm pack, extracting the resulting tarball, and committing (only) the result of those operations.
Essentially pre-compiles the .coffee src/ files into .js lib/ files, and prunes out a lot of un-needed files for Atom's purposes, such as all the specs. Again, this is equivalent to how apm would be packaged if I had been able to upload it to the npm package registry.
NOTE:
The commit this points to at the 'atom-community/apm' repo is the commit at the tip of the '2.6.5-atomic.1.1-dist' branch at that repo, as of the time this commit is being authored (28th January 2023).
NOTE:
Using a full git SHA so that this is content-addressable, and the file content cannot change. Either this SHA is available at the git repo, and the exact same file content can be downloaded from there, or the npm install will fail. (Failing to install would be better than surreptitiously downloading/running different/unexpected code.)
This is a best practice, in my opinion, when specifying dependencies from a remote git repo, as the lockfile 'integrity' field is easier to overlook when it changes, whereas this needs the exact SHA available, or it will outright fail. Branches and tags are not immutable. SHAs are immutable (barring some rather exotic hash collision attack).

Welp... The checksum of the tarball isn't matching today what I recorded yesterday.
$ npm install --global-style
npm ERR! code EINTEGRITY
npm ERR! sha512-rYJV54BFzKoTMAyYZbOfphayYqmsSYnBw3ybyzqd6eBs1e3yBoVK5Vyh5J1cj5mbMiXJy6iCx0SmUtuL8JlEkA== integrity checksum failed when using sha512: wanted sha512-rYJV54BFzKoTMAyYZbOfphayYqmsSYnBw3ybyzqd6eBs1e3yBoVK5Vyh5J1cj5mbMiXJy6iCx0SmUtuL8JlEkA== but got sha512-Mew2vEnzev+921/loUmHBw3U28h7Af3eVAMbzOyHMijq/vQlHE33bI9KNrKY/HOsMErXn1U5bm0Yu7tfZq0Byw==. (65172 bytes)

Obscure details about the npm cache (click to expand):
Error shows if you never cached this tarball before today, or else if you purge your npm cache.
(The npm cache is located at... Linux/macOS: ~/.npm/_cacache by default. Windows: C:\Users\[User]\AppData\Local\npm-cache\_cacache by default.)

I will try again tomorrow and see if the checksum changes again... Maybe GitHub.com's tarballs aren't content-stable somehow? Maybe the modification date is being changed every day, or something like that?
I'll be disappointed if I can't get the tarball this way, but I would be willing to chuck a stable tarball onto the atom-community/apm repo's release binaries and use that, I guess. It's just unfortunate not to be able to rely on the git repo SHA for an additional layer of verification, so I will seek to do that if at all possible.

It would be nice if there was an auto-publish bot whenever some release commit or issue comment or smth happens

So, yesterday I wrote  this:

I will try again tomorrow and see if the checksum changes again...

Aaaaand today the tarball integrity (sha512 sum)  is back to matching what I recorded in package-lock.json in the first place.
(❓ ⁉ ❗ ❓ ⁉ ❗ GitHub, what the hey? This is supposed to just be a stable tarball, right?)
I may or may not wait to see that this continues to be stable for several days in a row before marking this ready for review. I really don't want to have to babysit that this tarball stays stable for months going forward, I want to commit it once and be sure it's gonna stay stable. (And frankly, stability is what I expected when I used this GitHub tarball URL thing.)
If I don't see such stability, I will upload a tarball to the GitHub Release page for apm v2.6.5-atomic.1.1, and use that tarball instead. At that point, the tarball wouldn't be programatically generated by some automated GitHub API server, it'd be a one-and-done operation by me, a human, and saved to an exact sequence of binary bits once and only once, and can be reasonably expected to not change unless GitHub's binary servers go on the fritz, or someone gains malicious access to the apm repo, at which point at least the integrity recorded in this repo's package-lock.json would not match the (hypothetical) malicious or modified tarball, and people's copy of apm in this repo wouldn't install, due to incorrect SHA512SUM hash.

Fetching with git+https:// URL scheme only applies if you're git cloneing a git repo.
I'm telling npm to fetch a .tar.gz archive file and extract that as the package. (GitHub.com happens to serve the tarball over HTTPS. But git software isn't a part of it.)
I dunno what happened that made the tarball come back with a different SHA, but it's back to normal now.


Tree-sitter grammars must maintain compatibility with specific Tree-sitter

tree-sitter is a direct dependency of Topiary. Each Tree-sitter grammar (e.g., tree-sitter-json, tree-sitter-rust, etc.) has a transitive dependency on tree-sitter. The direct dependency and transitive dependencies must maintain API compatibility, otherwise Topiary won't build.
Currently, we are using Tree-sitter 0.20. v0.21 is now available and some grammars allow this, which causes Cargo to build both 0.20 and 0.21 for Topiary and subsequently fail. We can use the [patch] directive in our Cargo.toml to pin the transitive dependency (e.g., e91534b0aa).
This might be more of a "Cargo problem", than a Topiary one, but it would be nice if there were some tooling around this. Perhaps, at least, a CI step that checks there's only one version of tree-sitter in the Cargo.lock...
(#4 will definitely be impacted by this.)

Would pinning the git dependencies on those grammars help, or even with a pinned dep, cargo would still somehow fetch a newer version of tree-sitter if available?

Would pinning the git dependencies on those grammars help, or even with a pinned dep, cargo would still somehow fetch a newer version of tree-sitter if available?

Pinning both the grammar and its transitive dependency on Tree-sitter appears to work in practice. It's a bit verbose, mind you:
[dependencies]
tree-sitter = "= X"
tree-sitter-foo = { git = "https://github.com/tree-sitter-foo/tree-sitter-foo.git", rev = "Y" }

[patch."https://github.com/tree-sitter-foo/tree-sitter-foo"]
tree-sitter = "= X"

I see, you still need to patch part, which is a bit unfortunate.

Since tree-sitter 0.23 this is no longer the case. Grammars should depend on tree-sitter-language, with only a dev dependency on tree-sitter itself.

Looks like it's not as straightforward as bumping to the latest tree-sitter, 0.24.4:
modified   Cargo.toml
@@ -75,12 +75,12 @@ test-log = "0.2"
 tokio = "1.32"
 tokio-test = "0.4"
 toml = "0.8"
-tree-sitter = "0.22.6"
+tree-sitter = "0.24"
 # tree-sitter-bash = { git = "https://github.com/tree-sitter/tree-sitter-bash", rev = "1b0321ee85701d5036c334a6f04761cdc672e64c" }
 # tree-sitter-css = { git = "https://github.com/tree-sitter/tree-sitter-css.git", rev = "02b4ee757654b7d54fe35352fd8e53a8a4385d42" }
-tree-sitter-json = { git = "https://github.com/tree-sitter/tree-sitter-json.git", rev = "94f5c527b2965465956c2000ed6134dd24daf2a7" }
+tree-sitter-json = { git = "https://github.com/tree-sitter/tree-sitter-json.git", rev = "4d770d31f732d50d3ec373865822fbe659e47c75" }
 # tree-sitter-nickel = { git = "https://github.com/nickel-lang/tree-sitter-nickel", rev = "43433d8477b24cd13acaac20a66deda49b7e2547" }
-tree-sitter-ocaml = { git = "https://github.com/tree-sitter/tree-sitter-ocaml.git", rev = "036226e5edb410aec004cc7ac0f4b2014dd04a0e" }
+tree-sitter-ocaml = { git = "https://github.com/tree-sitter/tree-sitter-ocaml.git", rev = "98c2130c59ca7553b47086f91c5d22180151ad55" }
 # tree-sitter-ocamllex = { git = "https://github.com/314eter/tree-sitter-ocamllex.git", rev = "4b9898ccbf198602bb0dec9cd67cc1d2c0a4fad2" }
 # tree-sitter-query = { git = "https://github.com/nvim-treesitter/tree-sitter-query", rev = "a0ccc351e5e868ec1f8135e97aa3b53c663cf2df" }
 # tree-sitter-rust = { git = "https://github.com/tree-sitter/tree-sitter-rust.git", rev = "e0e8b6de6e4aa354749c794f5f36a906dcccda74" }
@@ -96,8 +96,3 @@ topiary-tree-sitter-facade = { version = "0.5.1", path = "./topiary-tree-sitter-
 topiary-core = { version = "0.5.1", path = "./topiary-core" }
 topiary-config = { version = "0.5.1", path = "./topiary-config" }
 topiary-queries = { version = "0.5.1", path = "./topiary-queries" }
-
-# tree-sitter-json's dependency on Tree-sitter is looser than ours, so
-# we have to pin its version to maintain API compatibility
-[patch."https://github.com/tree-sitter/tree-sitter-json"]
-tree-sitter = "0.22.6"

gives build failures in topiary-tree-sitter-facade parser.rs and query.rs.
The parser is an easy fix:
modified   topiary-tree-sitter-facade/src/parser.rs
@@ -4,7 +4,7 @@
 mod native {
     use crate::{
         error::{IncludedRangesError, LanguageError, ParserError},
-        language::Language,
+        language::{Language, LanguageRef},
         logger::{Logger, LoggerReturn},
         point::Point,
         range::Range,
@@ -31,7 +31,7 @@ mod native {
         }

         #[inline]
-        pub fn language(&self) -> Option<Language> {
+        pub fn language(&self) -> Option<LanguageRef> {
             self.inner.language().map(Into::into)
         }

seems to work, but tree_sitter::QueryMatches now implements StreamingIterator instead of Iterator, so this requires further changes

I've had a stab at updating Query to use tree-sitter 0.24, but I'm struggling with lifetime issues and I'm afraid my rust just isn't good enough.
The streaming iterator returns references to the items, which means QueryMatch needs to store a reference to tree_sitter::QueryMatch, so in Query::matches we would now have:
        pub fn matches<
            'query,
            'cursor: 'query,
            'tree: 'query,
            T: tree_sitter::TextProvider<I> + 'query,
            I: AsRef<[u8]> + 'query,
        >(
            &'query self,
            node: &Node<'tree>,
            source: T,
            cursor: &'cursor mut QueryCursor,
        ) -> impl StreamingIterator<Item = QueryMatch<'query>> + 'query + use<'query, 'tree, T, I> {
            cursor
                .inner
                .matches(&self.inner, node.inner, source)
                .map(Into::into)
        }

and I don't know how to convince the .map(Into::into) that its argument will live as long as self. This is as far as I can get:
error: lifetime may not live long enough
  --> topiary-tree-sitter-facade/src/query.rs:36:23
   |
22 |             'query,
   |             ------ lifetime `'query` defined here
...
36 |                 .map(|q: &'query tree_sitter::QueryMatch<'query, 'tree>| -> QueryMatch<'query> {Into::into(q)})
   |                       ^  - let's call the lifetime of this reference `'1`
   |                       |
   |                       requires that `'1` must outlive `'query`

My rust is not so great and I don't understand the error -- I thought that the &'query on the closure argument is saying precisely that its lifetime will be 'query
Any help would be appreciated!


Small screen navbar button not working with widget

Hi guys, love the widget, I came across a small issue; I'm using react and the responsive Navbar. I noticed that when I reduce the screen size, and the navbar shrinks, the navbar button is just not clickable when the Widget is active.
Any way around this?
thanks

Sorry, i made a duplicate


Add example on how to collect logs from custom host paths

Add a configuration example using extraFileLogs option added in this PR with config parameter for mounted volumes.
Discussion reference: https://github.com/signalfx/splunk-otel-collector-chart/pull/300#discussion_r752483728

Would love if we provided example for EmptyDir and persistent Volume Claims.
I have tested with AzureFile mount successfully with a customer, however admins seem to be keen on using EmptyDir and in cluster storage classes to avoid latency and extra costs.
Still waiting to hear back on whether AKS even supports collection from EmptyDir or if there are special config consideration/security considerations...

Yes, I'd also like to see an example (maybe in the rendered manifests) of how to forward logs from an EmptyDir volume mount using the Otel Collector as an agent daemonset.
Even better would be to include a way to automatically create a filelog receiver for pods as they are instantiated on the host, which configures the filelog "include" based on an annotation on the pod as to the directory/location of the log file.

Thank you for your recommendations. Please feel free to open Splunk Ideas to follow up if this is still relevant. Closing this issue at this time.


All elm apps rebuild on each change

This might be a peculiarity of my setup but I have 4 separate elm apps being built by webpack and when ever I make a change all 4 rebuild. Even when the change is to a file that is only used by one of them.
The rebuilding can be quite quick for 3 of the 4 apps but it is still slows down the experience.
I see output like this on a typical file change:
[1354] ./node_modules/moment/locale ^\.\/.*$ 2.86 kB {27} [optional] [built]
    + 1495 hidden modules
 46% building modules 6/10 modules 4 active ...erver/apps/business/src/Locations.elmStarted compiling Elm..
 52% building modules 7/10 modules 3 active ...erver/apps/business/src/Locations.elmStarted compiling Elm..
 57% building modules 8/10 modules 2 active ...erver/apps/business/src/Locations.elmStarted compiling Elm..
 64% building modules 9/10 modules 1 active ...erver/apps/business/src/Locations.elmStarted compiling Elm..
 94% asset optimization

Though interestingly, src/Locations.elm is only one of the 4 elm entry files. Not sure why that name is printed out each time.
My set up is that I have a single folder (server/apps/business) which has a elm-package.json and 4 typescript files. Each of those typescript files requires a different .elm file from the src subdirectory in server/apps/business.
My webpack elm rule looks like:
                {
                    test: /\.elm$/,
                    exclude: [/elm-stuff/, /node_modules/],
                    use: [
                        'elm-css-modules-loader',
                        {
                            loader: 'elm-webpack-loader',
                            options: {
                                debug,
                                cwd: path.join(__dirname, '..', 'server', 'apps', 'business'),
                                pathToMake
                            }
                        }
                    ]
                },

I've tried looking at the code. I get the feeling that it is watching all *.elm files and rebuilds everything if any of them changes rather than following imports from each entry .elm file. Understandable if that is the case but I wanted to check in case the behaviour I'm seeing is avoidable with some configuration changes.
Thanks for the project. It has made it much easier to add elm into my stack at work.

Is it possible it's connected to https://github.com/elm-community/elm-webpack-loader/issues/123? I'm curious if commenting this line https://github.com/elm-community/elm-webpack-loader/blob/master/index.js#L135 would fix it for you the same way it does for me.

I am having the same issue with what sounds like a similar file structure. Tried commenting out various lines but nothing seemed to help. Not a deal breaker but would be nice to have it fixed

We also just managed to get this working by removing the cwd option as well.
We did run into an issue were some files were not compiling when file dependencies changed but this was due to some files not having the proper namespaced module name in the main elm file that was initialised in the Javascript. These files shouldn't have been compiling from the start because of the module name but because the cwd adds all of these file as dependencies of each other they compile regardless.

Interesting to read. I've just tried removing the cwd option but then I get errors saying it cannot find elm.json. I am using the elm 0.19.1 beta but I assume the issue would be the same. I run webpack from the top of my project whilst the elm.json is in an apps folder along with the elm code (things have been moved around since I opened this issue.) I'll report back if I try moving the elm.json to the top level and running things from there.

@gavin667 Thank you for sharing this experience! You saved my day yesterday. :-) I ran exactly into the same issue and because of reading your comment in the morning it took very short to fix it.
@michaeljones I am 99% sure that this problem is because elm.json is not in the same directory where you run Webpack. I also had to move it to the same folder where Webpack is to make the loader happy.


changelog / release notes

Obviously related to #756 and #621.
There's the breaking changes changelog in docs but it would be nice to have a proper CHANGELOG.md in the root or notes on the releases page (since there hasn't been any notes attached to releases since v10.1.8).
Having to look at the commit titles to figure out what has changed is less than ideal.

I agree, this would be useful. I'll try to document the changes better.

The other issues proposes to use If  https://github.com/TypeStrong/atom-typescript/releases.
If we intend to use that, we should be diligent to convey the changes in a more visible way.
Our beloved editor does a good job with that: https://github.com/atom/atom/releases

Changelog.md has been added. Can we trust that it will be maintained and close this issue ;)

We can 😁


E2e updates

Hi @alexeigo can you fix the conflicts and rebase?

@SethDavenport conflicts were resolved.


Delete kubernetes.github.io site

It still seems like there's a website out there called https://kubernetes.github.io/ingress-nginx/user-guide/multiple-ingress/ serving very old content.
Can we please wipe this site out?

My bad, it's coming from github.com/kubernetes/nginx-ingress repo! It really looked like kubernetes documentation.
Looks like nginx-ingress repo hosts its own documentation site. Feel  free to close.

Thanks for noticing and clarifying 🎉
I'll close this one as it is correct behavior.
/close


chore: build lean4checker with current toolchain

After leanprover/lean4#2766 lands, the git hash of will be embedded in every olean. lean4checker will thus reject oleans unless it was compiled on the same toolchain as they were!
We may want to relax this later, but for now, since we are building lean4checker in CI anyway, let's just make sure it is on the same toolchain as mathlib.

bors r+


Configuring role ARN in ~/.aws/config

Thank you for the tool, it work great. One minor annoyance I found when working with multiple roles, is that you always have to select the right one. I would prefer to have a role preconfigured by profile, for example:
[profile Administrator]
azure_tenant_id=...
azure_app_id_uri=...
azure_default_username=...
azure_role_arn=arn:aws:iam::123456789012:role/Administrator

[profile Developer]
azure_tenant_id=...
azure_app_id_uri=...
azure_default_username=...
azure_role_arn=arn:aws:iam::123456789012:role/Developer

Thanks!. Just published v0.5.0 with your PR. I did make a minor tweak to the prompt.
One quick question if you don't mind. How do you configure multiple roles for an app in Azure AD? (I'm interested in doing this myself.)

You need to edit Azure application manifest (better using https://apps.dev.microsoft.com, editing from https://portal.azure.com has some issues). There you can add as many appRoles as you want, just be sure to generate an unique id for each role.


fix infinite loop in the parser

bors r+
🤖

bors r+
On Fri, 20 Sep 2019 at 16:55, bors[bot]<EMAIL_ADDRESS>wrote:

Merge conflict
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/pull/1882?email_source=notifications&email_token=AANB3MYVMPTZY62KFOUM52TQKTI5RA5CNFSM4IYXMEF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7GYV6Q#issuecomment-533564154,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANB3M66OAYSYKEIRV2BJ7LQKTI5RANCNFSM4IYXMEFQ
.


[Discuss] Is the conventional commit format working?

I've found that it's easy to typo the 'component' and that it's not clear what to use for the component. (For instance: is a cross-language change fix(c,python)?) Maybe we should align with the Arrow project and just use the language as the 'component' (so c, python, go, etc.)? Or, we could improve the validation to check that the 'component' really is a subdirectory of the repo (that way we won't typo go/adbc/flightsql when we mean go/adbc/driver/flightsql).
It doesn't help that GitHub defaults to the commit message, not the PR title/message, when merging - so we'll fix it in the PR, only to have GitHub merge using the original message. We can ask INFRA to change the setting to merge using the PR title/description, if people agree.

We can ask INFRA to change the setting to merge using the PR title/description, if people agree.

I like the setting!

I'll send in a request to adjust the merge settings shortly.
To Dewey's point: I think we can keep the format, and I'll see about setting up the CI check to catch typos in the 'component' part.

The default commit message: https://issues.apache.org/jira/browse/INFRA-24133

Ah, thank you for filing that, I forgot to come back to this.

INFRA made the change and it sounds like we can keep conventional-commits - I'll file a follow up to improve the validation


Cannot use Hugo with ASDF

What version of Hugo are you using (hugo version)?
hugo v0.97.0-c07f3626e7c8160943591f4d209977efa02c3dca+extended darwin/arm64 BuildDate=2022-04-14T08:45:07Z VendorInfo=gohugoio
(extended)
Does this issue reproduce with the latest release?
Yes
Issue
When running with ASDF, Hugo errors trying to run npx. This has to do with how it sanitizes or modifies the environment to invoke out to npx.
> cat .tool-versions
nodejs 16.13.1
hugo extended_0.97.0
> npx --version
8.1.2
> which npx
/Users/rob/.asdf/shims/npx
> asdf which npx
/Users/rob/.asdf/installs/nodejs/16.13.1/bin/npx
> hugo serve
Start building sites …
hugo v0.97.0-c07f3626e7c8160943591f4d209977efa02c3dca+extended darwin/arm64 BuildDate=2022-04-14T08:45:07Z VendorInfo=gohugoio
ERROR 2022/04/21 14:13:22 POSTCSS: failed to transform "sass/styleguide.css" (text/css): unknown command: npx. Perhaps you have to reshim?

And to fix I was able to do:
PATH=`asdf which npx | xargs dirname`:${PATH} make up-ui

I suspect this is related to #9800 but am not certain. Other than fixing this in Hugo, or working around as given, is there any way to customize the exec behavior?

Let’s see if an environment variable is being blocked. In your site configuration:
[security.exec]
osEnv = ['.*']

Yup, that fixed it @jmooring. It's a reasonable workaround to find the offending environment variable and include it there?

The default config is:
[security.exec]
  osEnv = ['(?i)^(PATH|PATHEXT|APPDATA|TMP|TEMP|TERM)$']

Try:
[security.exec]
  osEnv = ['(?i)^(PATH|PATHEXT|APPDATA|TMP|TEMP|TERM)$','(?i)^ASDF']

could the error message hint towards this solution

I think it did:

Perhaps you have to reshim?

This smelled like an ASDF message, because it certainly isn't a Hugo message. From there I read about ASDF, learned that it depends on env vars, and there you go.

I knew it had to do with ASDF due to the reshim message, but I had to look at the Hugo source to understand it was due to env vars. In other words: you knew about Hugo exec's use of env vars and had to read about ASDF; most Hugo users that hit an error like this will know about their environment but have to read about Hugo exec.
It seems like, if the exec fails, it'd be nice to include a link to some help- the issues due to missing env vars could be a lot more obscure than this, I think. LMK if you're open to it. I understand these sorts of UX edge cases are usually annoying for maintainers to fix, but it's hard to see how much time is saved in the aggregate by handling them.

Hugo has no idea why npx failed. Perhaps the asdf framework needs to emit more explicit messages.

I meant (and wrote) that Hugo should add a link to some docs any time the exec fails, nothing to do with ASDF. But if you're not interested you should close the issue.

For the sake of anyone stumbling upon this, the final set of env vars needed was:
security:
  exec:
    osEnv:
      - '(?i)^(PATH|PATHEXT|APPDATA|TMP|TEMP|TERM)$'
      - ASDF_DIR
      - HOME

Hugo should add a link to some docs any time the exec fails

I'll let @bep make that call.
To me, the more important question is, does the value provided by this security mechanism outweigh its cost? We've had a number of issues since this was implemented.

https://discourse.gohugo.io/t/38303/
https://discourse.gohugo.io/t/37397/
https://discourse.gohugo.io/t/37090/
https://discourse.gohugo.io/t/36180/
https://discourse.gohugo.io/t/36146/
https://github.com/gohugoio/hugo/issues/9741
https://github.com/gohugoio/hugo/issues/9732
https://github.com/gohugoio/hugo/issues/9333

When I create a site that I expect to be cloned, I can include the security settings in the config file, in which case you are only protected if you have set the corresponding env vars. For example, in .bashrc:
export HUGO_SECURITY_FUNCS_GETENV="^HUGO_"
export HUGO_SECURITY_EXEC_OSENV="(?i)^(PATH|PATHEXT|APPDATA|TMP|TEMP|TERM)$"
export HUGO_SECURITY_EXEC_ALLOW="'^dart-sass-embedded$', '^go$', '^npx$', '^postcss$'"

So, perhaps we could set the default to osEnv = ['.*'], and you would still be protected if you have set your local env. Just a thought...

To me, the more important question is, does the value provided by this security mechanism outweigh its cost?

Without this security mechanism, I'm not using Hugo, but that's my 50 cents.

When I create a site that I expect to be cloned, I can include the security settings in the config file.

Sure you can, but that's also fairly easy for me to reason about, looking at the top level config.

in which case you are only protected if you have set the corresponding env vars to be more restrictive.

Which I have. Which makes me not only protected, but always protected from all sites.

So, perhaps we could set the default to osEnv = ['.*'], and you would still be protected if you have set your local env to be more restrictive. Just a thought...

We cannot have a default security setting set to its most insecure setting. Especially considering that 95% or all Hugo sites does not need these external executables (number taken out of a hat), and 4.99 of the remaining are perfectly fine with the defaults (again numbers out of a hat).
This recent news story should be a good read:
https://github.blog/2022-04-15-security-alert-stolen-oauth-user-tokens/
I'm not saying that Hugo's security setup is perfect and bullet proof and all, but at least we're trying. Having the full OS environment leak via some random theme, that is not trying. This situation isn't going to get any easier in the future, and it's a little bit scary to think of all the build tools out there with access to the full OS.

You mean ADFS in the title, meaning Active Directory Federation Services.
Yesterday I spent hours before finding out that I had to set osEnv = ['.*'] in the security section of toml configuration!
Yeah at least a message hint would help next one.

This was a security configuration setting, resolved by adding ASDF to security.exes.osenv in site configuration. See https://gohugo.io/about/security-model/.


Failed to extract videos

Steps to reproduce
1.open the app
2.go to anime library
3.open up an anime ep
Expected behavior
Instead of opening the episode and letting me watch.
Actual behavior
Instead it says Failed to extract episode and does let me watch and makes I have to open and close the app multiple times to get it working again.
Crash logs
No response
Aniyomi version
App version: 0.14.6-6691 (standard, 5fd00d4a1, 105, 2023-11-02T23:11Z)
Android version
Android 13
Device
Samsung s23
Other details
No response
Acknowledgements

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

is it on a specific source/extension?

How did you get to fix it?

I fixed it using another extension this only happened to me while using aniwatch

I fixed it using another extension this only happened to me while using aniwatch

Yeah did the same, can I know which extension you use now?

I fixed it using another extension this only happened to me while using aniwatch

Yeah did the same, can I know which extension you use now?

I used gogoanime

Do not recommend any source extensions. This is to prevent any single one becoming too popular and getting Tachiyomi or Aniyomi blacklisted from using it.

Do not recommend any source extensions. This is to prevent any single one becoming too popular and getting Tachiyomi or Aniyomi blacklisted from using it.

Oh ok that's my bad


CodeLabs Write your first Flutter App part 4

Steps to Reproduce
import 'package:flutter/material.dart';
import 'package:english_words/english_words.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final wordPair = WordPair.random();
    return MaterialApp(
      title: 'Welcome to Flutter',
      home: Scaffold(
        appBar: AppBar(
          title: const Text('Welcome to Flutter'),
        ),
        body: const Center(
			child: Text(wordPair.asPascalCase),
        ),
      ),
    );
  }
}

Expected results:
in Chrome browser an English Word
$ flutter run -d chrome
Launching lib\main.dart on Chrome in debug mode...
Waiting for connection from debug service on Chrome...
lib/main.dart:21:16: Error: Not a constant expression.
                        child: Text(wordPair.asPascalCase),
                                    ^^^^^^^^
Waiting for connection from debug service on Chrome...             26,1s
Failed to compile application.

Logs

Hi @AngeMauriciolAntolin, Thanks for filing the issue. The issue in your code is you are using const for Center removing that should help fix the issue. However, looking at the code lab https://codelabs.developers.google.com/codelabs/first-flutter-app-pt1#4 The code seems correct to me.

I was getting the same error during Part 3, and it looked to me like a problem with wordPair, because after removing const from the Body declaration I was still getting a (more verbose) error about wordPair:
lib/main.dart:20:23: Error: The getter 'wordPair' isn't defined for the class 'MyApp'.
 - 'MyApp' is from 'package:startup_namer/main.dart' ('lib/main.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'wordPair'.
          child: Text(wordPair.asPascalCase),

The real issue was that I was missing this line: final wordPair = WordPair.random();. Revisiting this now I simiply didn't pick up on the use of the pink comment lines which denoted each change.


Update azure-subscription-limits-azure-resource-manager.md

Adding subscription tag limits

@rjmax,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@rjmax : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@rothja Hi Jason Please review these proposed changes. Thanks.

#sign-off


Turn Off Console Debugging

Describe the solution you'd like
A way to disable the mouse events being logged to console. Such as the "onMouseDown check"
I had thought it was just a me thing or because it had yet to be built. But then I saw it on the live site which makes me confused why you want constant logging on the live version of your product with no way to disable it.

@Lagicrus
Thanks for reporting.
Could you review https://github.com/brainhubeu/react-carousel/pull/236 ?
Excluding the building scripts, the only other file with debugging code is https://github.com/brainhubeu/react-carousel/blob/master/docs-www/src/globalReferences.js which is used in demo.

Hey @piotr-s-brainhub , that looks to be it. Thanks :)

was what I was getting

@Lagicrus so https://github.com/brainhubeu/react-carousel/pull/236 should be enough - could you review?

Added an approval review

@Lagicrus
Thanks, the fix should be in 1.10.56 version

@Lagicrus
Can I ask you to fill in this form https://docs.google.com/forms/d/1tfRs1Io49tJBwz-06aDnz8hrJ3WySJTi45y4k6hHGaQ ?


Runtime Data Sync using Spectrum Fusion

Create process for runtime data sync facilitated by Spectrum Fusion
This pr creates template restore CRs and template recipe CRs that can be used to sync CS DB and Zen runtime data across clusters.
There are also several changes to scripts like deploy-br-resources.sh and deployment files to facilitate these processes.
Assumption:

Working spectrum fusion hub and spoke clusters
existing backups from hub
full identical deployment (preferably restored or the original) to one of the backups

Process:

edit the values of the recipe CR to match expectations (anything within <>)
apply the recipe CR to the target cluster
edit the values of the restore CR to match expectations (anything within <>)
apply the restore CR to the hub cluster
Wait for restore to complete
validate restore ran properly

check cpfs utils pod in operator namespace, look for relevant cleanup logs in /tmp/logs
check cs-db-backup or zen5-backup pods in services namespace, look for logs in cs-db/cs-db-backup/logs or zen5/zen-backup/logs for relevant restore logs (there should only be one)

validate services are working and data

This PR is ready for review


Master layout: resizing the last window doesn't work

Hyprland Version
Hyprland, built from branch  at commit<PHONE_NUMBER>117f7f150c6f2a2ce4dede6e8cbb25 dirty (). Tag: v0.28.0  flags: (if any)
Bug or Regression?
Bug
Description
Master layout: resizing the last window vertically doesn't work.
How to reproduce

Configure Hyprland:

master {
    mfact = 0.5
    new_is_master = false
}
general {
    #...
    layout = master
}
binde = SUPER ALT, H, resizeactive, -20 0
binde = SUPER ALT, J, resizeactive, 0 20
binde = SUPER ALT, K, resizeactive, 0 -20
binde = SUPER ALT, L, resizeactive, 20 0

Open at least 3 windows in a single workspace, i. e. 1 master and at least 2 non-master/slave windows.
Focus the last non-master window and try to resize it vertically with SUPER+ALT+J or SUPER+ALT+K with no effect.

It's interesting that the last window in similar dwindle layout can be resized vertically without any problems.
Crash reports, logs, images, videos
No response

Fixed in v0.30.0


[Snyk] Upgrade undici from 5.22.0 to 5.22.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade undici from 5.22.0 to 5.22.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-05-11.

Release notes

Package name: undici

5.22.1 - 2023-05-11What's Changed

Cache storage by @ KhafraDev in #2076
test: skip content-disposition test in node 18 by @ KhafraDev in #2081
Cache storage cleanup by @ KhafraDev in #2082
Cache storage fixes by @ KhafraDev in #2083
test: improve test coverage for ErrorEvent and MessageEvent by @ KhafraDev in #2085
test: remove --experimental-wasm-simd by @ KhafraDev in #2087
websocket: add websocketinit by @ KhafraDev in #2088
feat(websocket): allow setting custom headers by @ KhafraDev in #2089
test: fix tests failing only on node v20 by @ KhafraDev in #2096
fix: skip set content-length when FormData value is stream by @ fengmk2 in #2091
doc: update outdated command in contributing.md by @ jazelly in #2099
cache: fix most failing WPTs by @ KhafraDev in #2100
feat: allow build:wasm to auto detect platform by @ jazelly in #2102
docs: updated Error documentation (fixes #2090) by @ titanism in #2092
mimesniff: fix many broken tests by @ KhafraDev in #2103
test: fix failing tests by @ KhafraDev in #2097
build(deps): bump github/codeql-action from 2.2.9 to 2.3.2 by @ dependabot in #2105
fix: more informative error message to tell that the server doesn't match http/1.1 protocol by @ Songkeys in #2055
Fix bug in 16-bit frame length when buffer is a subarray by @ jawj in #2106
update wpts by @ KhafraDev in #2108
fix: update error definitions by @ dfilatov in #2112
fix: make assertion a noop by @ ronag in #2111

New Contributors

@ jazelly made their first contribution in #2099
@ titanism made their first contribution in #2092
@ Songkeys made their first contribution in #2055
@ jawj made their first contribution in #2106
@ dfilatov made their first contribution in #2112

Full Changelog: v5.22.0...v5.22.1

5.22.0 - 2023-04-20 Read more

    from undici GitHub release notes

Commit messages

Package name: undici

9d30456 5.22.1
241dfaf fix: make assertion a noop (#2111)
4cda7d5 fix: update error definitions (#2112)
790b35d update wpts (#2108)
4688da2 Fix bug in 16-bit frame length when buffer is a subarray (#2106)
ab2e0ce fix: more informative error message to tell that the server doesn't match http/1.1 protocol (#2055)
c1c8d83 build(deps): bump github/codeql-action from 2.2.9 to 2.3.2 (#2105)
72ca86a test: fix failing tests (#2097)
56e7624 mimesniff: fix many broken tests (#2103)
48f2078 docs: updated Error documentation (fixes #2090) (#2092)
3c514d8 feat: allow build:wasm to auto detect platform (#2102)
ba18823 cache: fix most failing WPTs (#2100)
73c59dd doc: update outdated command in contributing.md (#2099)
8ffad9d fix: skip set content-length when FormData value is stream (#2091)
699d3e8 test: fix tests failing only on node v20 (#2096)
a3efc98 feat(websocket): allow setting custom headers (#2089)
7fe5a0d websocket: add websocketinit (#2088)
286d5ec test: remove --experimental-wasm-simd (#2087)
1b3fed9 test: improve test coverage for ErrorEvent and MessageEvent (#2085)
eeaa073 Cache storage fixes (#2083)
674e2b6 Cache storage cleanup (#2082)
27569e9 test: skip content-disposition test in node 18 (#2081)
6197e93 Cache storage (#2076)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 100.0%. remained the same when pulling f1e004b1bb49b5db8e79b4ff05db0de3dfedb01e on snyk-upgrade-8acdeec0279a5c69fb0f827f8786538c into e443fe99e88ae824a03826a6bfb1613d260cdf51 on main.


⚠ Bofrost.lu has degraded performance

In d0a5c93, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1925 ms

Resolved: Bofrost.lu performance has improved in c56bc6f after 10 minutes.


Ineffective for Sonic & All-Stars Racing Transformed

Hello. Thank you for your great work. I have seen this work for the XInputTest.exe program that can be found in the x360ce repository but unfortunately it seems ineffective for the one game I'm trying to play. I also tried your new and wine branches but they didn't help.
To cut a long story short, without XInput it's like one of the axis is permanently stuck, even under real Windows 10. x360ce works under Windows but it also has no effect under Wine for this particular game. I tried to get more information from the x360ce team in order to approach the Wine team but despite giving some hints to another user earlier this year, they decided they weren't going to help me today. I'd rather use your solution anyway.
This game requires x360ce's COM HookMask. After comparing your code against theirs, it looks as though you're doing something similar but not hooking as many functions. Perhaps CoGetClassObject is significant? I captured logging from starting Steam, starting the game, quitting the game, and then quitting Steam. I can see CoGetClassObject mentioned just before it outputs controller messages at line 1081 that you only see when it has fallen back to DirectInput.
One more thing, when using this project with Launcher.exe, the game's configurator, it crashes out on startup with a StackOverflowException after printing this over and over.
koku-xinput-wine: CoSetProxyBlanket(...);

This doesn't matter much as you generally don't need to use the configurator and this doesn't happen to the game itself. Just thought I should make you aware of it.
Please let me know if you need any more information.

The whole COM thing is pretty ugly and stupid...
With https://github.com/KoKuToru/koku-xinput-wine/pull/9 I started to cleanup some stuff..
But the COM-Part is still pretty ugly. It's maybe much easier to add the "gamepads" to the wine devices..
I tried to find the code where the gamepad gets returned but couldn't find it..
The COM-Hack does nothing else then changing the Name/ID (or whatever) so the game thinks it's a XInput Controller.
Maybe I have time this holidays (~20.December) to check the wine code again.. if I don't forget..

Thanks for the quick reply. I know that Wine's DirectInput code adds the devices here for js and here for event but I suspect that's not the code you meant. xboxdrv allows me to name the device whatever I like but if I try VID_3ED9&PID_9E57&IG_00, it doesn't make any difference, probably because this is only exposed through DirectInput. You're talking about WMI?

well I am searching for the code which returns the PnP devices.. (and/or register the devices)
the thing I hook

Maybe I have time this holidays (~20.December) to check the wine code again.. if I don't forget..

So about that code... :wink: In your own time, man, have a good Christmas.

The alternative project, wine-xinput, explains that this game doesn't work because it mixes DirectInput with XInput. I have heard that Wine's own XInput support is slowly progressing so hopefully something can be done to make this work.

The same issue seem to affect Pro Evolution Soccer 2016/2017 as well, for the same reason. It does DInput enumeration before XInput, the same as Sonic ASRT.

Sonic Racing Transformed now works perfectly, although you can't load the Launcher (the config program) if Koku is loaded as it'll crash.
On the other hand, Pro Evo 2017 (and its config program) crash if you load Koku:
=>0 0xf7f7b763 _ZN4koku16CoCreateInstanceERK5_GUIDP8IUnknownjS2_PPv+0xb3() in koku-xinput-wine.so (0x0032f328)

Another one is Castle Crashers. With Koku loaded it sits at 100% CPU usage and never actually loads the game. Loads fine without Koku (but obviously the controllers then don't work).

Sonic Racing Transformed now works perfectly. Its config program (Launcher.exe) will crash if Koku is loaded, but you don't need to use that program when using XInput anyway.

I found this hard to believe given how much trouble I'd had with it before but you are… almost right! It's not immediately apparent but I'm quite sure that you don't get full acceleration. When using the keyboard, I came 4th without trying that hard. When using the controller, I struggled to keep out of last place and the car felt a bit sluggish. I'm not sure why as XInputTest.exe shows what I believe is the full range of values for the trigger, 0-255. I usually use Thrustmaster controllers with xboxdrv. I tried to use with a real 360 controller but the wireless adapter wasn't playing ball today.

Odd, I came second in B class after having not played for probably more than a year and repeatedly cocking up, so I wonder if that's not a universal problem.
Worth noting that Castle Crashers works with one of the DLL based XInput emulators so it's not universally broken.

Here's an update for Wine 3.5. As you know, we can't build this project any more. If I run the game with xboxdrv but without any additional hacks, it seems to use XInput (the button icons are green A, red B, etc) and the menus work but I can't accelerate or reverse. I guess Wine didn't put it the extra stuff needed to make this one work yet.


Chain cause in ComputeException.translateAndThrow, add tests

This fixes #946 for the compute module

Changes Unknown when pulling b0446baedf4b3c6c2be3dfa5f3b651d64e34834c on mziccard:compute-add-cause into ** on GoogleCloudPlatform:compute-alpha**.

LGTM

@aozarov I just noticed that we do very have different access modifiers for exceptions' constructors. In dns they are private while in other services they're public. I suggest we make the package scoped for the sake of testing. What do you think?

Yes, I think consistency here is good and package scope would be a good choice.

Changes Unknown when pulling 678f04950a3083b4273b725b2b803764517c846e on mziccard:compute-add-cause into ** on GoogleCloudPlatform:compute-alpha**.


Enhancement for denoting "key" : value pairs

Hi ZSerge,
I have added a functionality to help make sure two tokens have a ':' between them by adding a "valueof" field to the token structure. In the token after the ':' the valueof will point to the token before the ':', and for all other tokens it will be -1 (in the same way a token will point to its parent when this is enabled).
.c file is attached, changes in .h are:
typedef struct {
jsmntype_t type;
int start;
int end;
int size;
int valueof;
#ifdef JSMN_PARENT_LINKS
int parent;
#endif
} jsmntok_t;
/**

JSON parser. Contains an array of token blocks available. Also stores
the string being parsed now and current position in that string
/
typedef struct {
unsigned int pos; / offset in the JSON string /
int toknext; / next token to allocate /
int toksuper; / superior token node, e.g parent object or array /
int nextisval; / flag: next token is value of previous */
} jsmn_parser;

Hope you find this useful!

Bitbucket: https://bitbucket.org/zserge/jsmn/issue/11
Originally reported by: Anonymous
Originally created at: 2013-01-15T17:44:39.337

Issue #15 was marked as a duplicate of this issue.

Original comment by: Serge Zaitsev

Issue #12 was marked as a duplicate of this issue.

Original comment by: Serge Zaitsev

Issue #23 was marked as a duplicate of this issue.

Original comment by: Serge Zaitsev

It's finally implemented! Now some string tokens have size of 1 which means they have a single child node. That's how you can tell key from a string value. It also simplifies parsing a lot. See commit 84cb5790d973

Original comment by: Serge Zaitsev

Fixed in 84cb5790d973

Original comment by: Serge Zaitsev


Download links generated incorrectly

Example (valid) download link:
https://github.com/atc0005/check-vmware/releases/download/v0.20.0/check_vmware_alarms-v0.20.0-0-g5bb7e11-linux-amd64
The literal v0.20.0 portion of the download subdirectory path is the tag for that release.
Download links are currently generated like so:
https://github.com/atc0005/check-vmware/releases/download/v0.20.0-0-g5bb7e11/check_vmware_alarms-v0.20.0-0-g5bb7e11-linux-amd64
The v0.20.0-0-g5bb7e11 literal portion of the download subdirectory path is not the tag for the release, but the internal version string. I'll need to fix the Makefile so that it uses the tag for the release and not the version string.
References:

#330
#200

Note: For the v0.20.0 release, I manually updated the download links text file and then re-uploaded it.


Add a README section referencing companies using ConsoleMe

Adds a README section referencing companies using ConsoleMe, mentions IAM managed policy support, and a third party blog post

CC @rms1000watt  okay to mention Calm here?

@castrapel heck yea! Can we add a large Calm logo also? 😅😅😅 And how can I get us to the top of the list? 🤣🤣🤣
jkjkjk, this looks perfect! Thank you!!

@castrapel heck yea! Can we add a large Calm logo also? 😅😅😅 And how can I get us to the top of the list? 🤣🤣🤣
jkjkjk, this looks perfect! Thank you!!

@rms1000watt -  Haha!  It's alphabetically sorted. Naturally, we can be bribed with caffeine, but the legitimate route is to change your company name. Here are suggestions that will put you on the top most of the time! Please let straight to work with your Legal team, register trademark and domains, and get back to me!  🤣
Alpha Calm
AI Calm
Amazing Calmness
1Calm
But in all seriousness, logos aren't a bad idea.

Just wanted to add that NCBI, the "National Center for Biotechnology Information", part of the National Institutes of Health (NIH), is a happy user of ConsoleMe - https://www.ncbi.nlm.nih.gov/


use identity hash for memo

@irmen memo uses Map<Object, Integer> and hence memo lookup calls Object.hashCode. The implementation of hashCode may be quite expensive, e.g., String.hashCode scans all characters. It may be good to change the logic here. In this PR, I use System.identityHashCode in the map and store the object in the value. So we only re-use serialized bytes if the same object (instead of objects with the same content) appears multiple times. Please let me know whether you think this is a good change. I will fix the code style (I used spaces instead of tabs).

Looks good, but yeah, please adapt the code style.
Have you done any performance measurements to see what we gain with this change?

I profiled it in PySpark. Please see the screenshot in https://issues.apache.org/jira/browse/SPARK-6288 (https://issues.apache.org/jira/secure/attachment/12704465/Screen Shot 2015-03-13 at 10.45.35 AM.png)

Okay, all tabs now:)

16% is quite significant indeed. How does the version with identityHashcode perform?

It seems that System.identityHashCode has a bug: http://bugs.java.com/view_bug.do?bug_id=6378256. But this is still faster than object.hashCode:

this has been released in 4.4


Custom commands for HomeKit

Hey,
Is it possible that you can insert your own commands that aren't related to HomeKit but for Siri with this library?

There's nothing stopping you from adding "Accessories" that aren't actually accessories. However the language to use them may be awkward, as Siri only knows phrases related to controlling a small set of connected-home devices. So "Turn on the XYZ" for instance.

Ok thank you, how can I close this issue?

Thank you!


HI there from Rails

How i can use this new in my rails addiction i try user Installation from CDN but it does not work with rails thank advance
please let know if something i m missing thank s

everything is working perfect thanks


txn: Move Command's read or write process to their own file

Signed-off-by: longfangsong<EMAIL_ADDRESS>What problem does this PR solve?
Problem Summary:
All read and write code are in process.rs, which makes the file very long and hard to find the part you want.
What is changed and how it works?
What's Changed:
Move the handling of commands into their own files.
Check List
Tests

Unit test

Side effects
There are a few places I have to add a clone, which might cause performance regression, a benchmark might be necessary.

Performance regression

Consumes more CPU
Consumes more MEM

/bench

@nrc PTAL

/run-all-tests

/bench+tpcc

/bench

/bench +tpch +sysbench

/bench

/bench

\run-all-tests

/run-all-tests

Maybe this pr should be merged after #8310, so the conflit doesn't need to be handled in it.

@nrc @sticnarf PTAL

/bench +sysbench +tpcc

/run-all-tests

I will look at it tomorrow

/bench +sysbench +tpcc

/lgtm

/merge


Change let module to mod keyword. Change mod to %.

Fix #26

Now that we have proper identifier swapping systems in place, I'd like to see us bring back % as the infix mod operator. We could always just have a prelude that defines the function %, but then it is not as seamless of a conversion back/forth between ocaml/reason and would require runtime support. We can swap the operators for (mod) and % perhaps.


Add ESLint to Frontend Package

Added ESLint to the frontend project and enforce the recommended rules, also added:

npm run lint: Fix the fixable lint issues (including prettier)

npm run lint:test: Just check if the code is well-formatted (used in the CI)

Note : Multiple files have changed after the linitng is done.
Issue: https://github.com/aerabi/events/issues/10

Hey, sorry doing the change had to close the PR because it got submitted from my other account, doing the change and adding

No worries, take your time. 👍


🔧 MAINTAIN: Update links to reflect migration to EBP

This PR updates most links to reflect the new destination under the Executable Books Project.
@choldgraf @chrisjsewell I would appreciate if you could advise on the best way to update Netlify and Codecov so I can update both accordingly.

I was able to setup Netlify :tada:


Map printer name and description instead of just description

https://github.com/boxture/oms/issues/2315

Manually merged into main


removing non-supported versions

Removed non supported doc links from API reference guide (links were also broke).

👍
/approve


--noImplicitAny is an error, not a warning

The help says it's a warning, but it actually fails the build with an error:
--noImplicitAny    Warn on expressions and declarations with an implied 'any' type.

I believe this should say something like:
--noImplicitAny    Raise an error on expressions and declarations with an implied 'any' type.

Fair point. PR would be appreciated :)


[Bug] New request instance mutating the original request when in phpunit

Laravel Version: 5.8.35
PHP Version: 7.2.11
Database Driver & Version:

Description:
I tried a simple code that will clone a request and change some params in the duplicate request. In the browser it is working fine but in phpunit the original request is also displaying the same value same with the duplicate request.
public function index(Request $request)
    {
        $request->merge(['foo' => '100']);

        $dup = clone $request;
        $dup->merge(['foo' => '200']);

       // this is both 200 in phpunit but in browser it is correct, 100 and 200
        return [
            $request->foo,
            $dup->foo,
        ];

    }

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? We'll help you out and re-open this issue if so.
Thanks!


Implementation of FasterIVA

Summary
I implemented FasterIVA, but the separation quality seems to be limited.

Codecov Report

Merging #78 (3712dee) into develop (8dbfd3c) will increase coverage by 0.16%.
The diff coverage is 97.01%.

@@             Coverage Diff             @@
##           develop      #78      +/-   ##
===========================================
+ Coverage    92.20%   92.36%   +0.16%
===========================================
  Files           18       18
  Lines         1834     1900      +66
===========================================
+ Hits          1691     1755      +64
- Misses         143      145       +2

Impacted Files
Coverage Δ

ssspy/bss/iva.py
94.46% <96.87%> (+0.31%)
:arrow_up:

ssspy/linalg/eigh.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Is this really a valid way of creating a class method? - "React 26: Bind 'this' with an ES6 Arrow Function"

Given the following experiment,

is the following really a valid ES6 class method?
class MyComponent extends React.Component {
  /* ... */
  setMessage = () => this.setState({ message: 'Goodbye!' })
  /* ... */
}

I thought methods only could be function declarations with the methodName () {} syntax.

I guess this could be some kind of syntactical sugar of the React library?

hmmm.......... interesting so this is now more clear after I've investigated: apparently the use of arrow functions as class methods is still an experimental feature, which can be enabled with babel stage-2 preset.
Here is some (not complete) info and some more.
I have seen this used for React component methods elsewhere.
@no-stack-dub-sack and @Greenheart what do you think we should do about this?

@bonham000 @Greenheart It's a good catch, but it's incredibly common these days, valid in CodePen and CreateReactApp both, which are the 2 most common places our campers will be writing React code. I think as long as we note it, we can prob keep it - it's valid in the challenges, its supported by the React team, and is a common way of writing react methods that's valid where our campers will likely be coding.
A  prominent note would probably be helpful

@bonham000 @no-stack-dub-sack Thanks for the info! If it's a common practice, I'd say we should go with it.
I'm just too used to vanilla JS I guess :smile:

Leaning toward providing a note at this point since doing otherwise would change several challenges, and, realistically, using arrow functions is a lot easier than binding them all explicitly in the constructor (i.e. it's very useful to have so worth showing people).

@bonham000 I like the idea of having a note.
If this gets accepted as a standard JS feature, it could be removed.

@Greenheart Agreed

Great I've made a note and revised the challenge description. Would you all @Greenheart @no-stack-dub-sack mind taking a quick look and seeing if this looks sufficient?

@bonham000 Excellent! :smile:


App.tsx ESLint/Prettier problem

What's going on?
New generated project have file app.tsx error line 53.
I have to add "no-extra-semi": "off", "@typescript-eslint/no-extra-semi": 0 to package.json to mute it, looking for another solution.

Steps to reproduce

Generate new project with --expo
In app.tsx file line 53: ;(async () => { have error by @typescript-eslint/no-extra-semi

ignite doctor results:
System
  platform           darwin
  arch               x64
  cpu                4 cores          Intel(R) Core(TM) i5-4278U CPU @ 2.60GHz
  directory          MyApp

JavaScript (and globally-installed packages)
  node               14.2.0       /usr/local/bin/node
  npm                6.14.4       /usr/local/bin/npm
    expo-cli         4.3.0
    ignite-cli       6.5.0
    ios-deploy       1.10.0
    npm              6.14.4
  yarn               1.22.4       /usr/local/bin/yarn
    eslint           7.0.0
    react-devtools   3.6.3

Ignite
  ignite-cli         6.5.0        /usr/local/bin/ignite
  ignite src         build        /usr/local/lib/node_modules/ignite-cli/build

Android
  java               12.0.2       /usr/bin/java
  android home       -            undefined

iOS
  xcode              12.4
  cocoapods          1.9.3        /usr/local/bin/pod

I can confirm this.

@nonameolsson thank you for providing the solution. I can confirm that both solutions fixed the issue–I personally prefer Solution 2. To answer your question, I don't see any reason why some functionality would be lost by this refactor.
@Tj3n can you confirm that either of these solutions fixed the issue for you without having to mute the error?

It's fixed, I'm closing this. Thanks!


Post/why write

Proof reading new blog post

I think you want to make the commit to your fork and not the main repo 🙂

@aadibajpai HAHAHA!!!! Last time I make a PR with a UI :). Back to terminal only for me :)


⚠ Bofrost.lu has degraded performance

In 8ce7044, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2008 ms

Resolved: Bofrost.lu performance has improved in 906f17d after 24 minutes.


Warn users when it's unable to assign average slot

Description
Fixes #2255
Type of change

[x] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[x] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[x] I've properly formatted my code according to the guidelines

[ ] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
New tests with preprocessing layer.

@fsx950223 Hi, can you take a look at the PR when time allows? Thank you!


Unix socket support

Currently, the given API in index.d.ts listens listen(host: string, port: number) and listen(port: number which implies to me there's no official unix socket support yet. Is this correct, or is there a way around it? Do you plan to support unix sockets?

https://github.com/uNetworking/uSockets/issues/15
This is doable at the lower levels, not prioritized right now but could be

From my experience this would lead to a substantial performance improvement when server is behind of a reverse proxy or when both client and server are running on the same machine. Would be great if possible 👍

Sure, Unix sockets are simpler than TCP sockets but a Linux kernel can do way more TCP than your proxy software can handle in any case. It's a low priority, esp if you plan on doing SSL

More interesting would be to write zeromq plugin to uWebSockets.. And use something like IPC socket type. I am doing that with JS bindings, and keeping uWebSocket.js node process away of any my businesses logic ...

This thread is not about zeromq, I don't see how that relates to anything here. "IPC socket type" is literally Unix sockets. This thread is about Unix sockets

This is already tracked in usockets

Unclear is this feature is implemented or not and how we can use it if yes?

It's not implemented. It could be, but given how incredibly efficient localhost TCP is on Linux, far far more efficient than your Nginx proxy process can handle - esp when you consider SSL, it's super low priority and quite pointless. Also, why would you put a proxy on the same very machine?

Also, why would you put a proxy on the same very machine?

@alexhultman It's convenient way for SSL termination and app separation.
In my case nginx decode HTTPS, serve static (by himself) and forwards non-static into UWS and auth server (packaged express app).

On more usecase for this - is if you don't want to have several ports open, like I could use a single 8080 port of Nginx to host the www app and use 8080/ws path for websocket connections with /ws being proxied to a local Unix stream.

https://lists.freebsd.org/pipermail/freebsd-performance/2005-February/001143.html
https://momjian.us/main/blogs/pgblog/2012.html#June_6_2012
I don't know what had changed with IP stack in linux since 2012, but I think not much in this case.

https://lists.freebsd.org/pipermail/freebsd-performance/2005-February/001143.html
https://momjian.us/main/blogs/pgblog/2012.html#June_6_2012
I don't know what had changed with IP stack in linux since 2012, but I think not much in this case.

The argument being Unix sockets are 60% faster than localhost is not relevant. Yes, Unix sockets are faster than TCP sockets - but Nginx modules are faster than Unix sockets, and not having Nginx at all is faster still.
Adding Unix sockets is a feature, yes, but it is low priority right now because you don't critically need it. It doesn't really matter because your proxy will bottleneck long before localhost will, anyways.

The argument being Unix sockets are 60% faster than localhost is not relevant. Yes, Unix sockets are faster than TCP sockets - but Nginx modules are faster than Unix sockets, and not having Nginx at all is faster still.
Adding Unix sockets is a feature, yes, but it is low priority right now because you don't critically need it. It doesn't really matter because your proxy will bottleneck long before localhost will, anyways.

It's kind of bizarre to use Node.js but still argue about the need for kernel space optimizations. If you care that much for performance, first step is to eliminate Node.js because it is by far the most immediate bottleneck. Node.js I/O is 800% slower than what you can get without it.

It's kind of bizarre to use Node.js but still argue about the need for kernel space optimizations. If you care that much for performance, first step is to eliminate Node.js because it is by far the most immediate bottleneck. Node.js I/O is 800% slower than what you can get without it.

Unix sockets will allow to establish inter-process communiction between docker containers more easily and secure. They will allow to run isolated docker containers and share sockets using file system instead of run containers in host network and use random ports (very unhandful).

Unix sockets will allow to establish inter-process communiction between docker containers more easily and secure. They will allow to run isolated docker containers and share sockets using file system instead of run containers in host network and use random ports (very unhandful).

Postgresq core developer measurements: tcp/ip, no ssl vs UNIX = +31%

If used pg client, you increase performance by ~300% using a new postgres client
https://github.com/porsager/postgres

Postgresq core developer measurements: tcp/ip, no ssl vs UNIX = +31%

If used pg client, you increase performance by ~300% using a new postgres client
https://github.com/porsager/postgres

For me it is not abut performance, but more that I don't have (and in my cases I just can't) open another network port.

For me it is not abut performance, but more that I don't have (and in my cases I just can't) open another network port.


7th: Customize FFNx Flag setting behavior

This PR is a follow up of #29 and introduces the behavior we discussed the last time, allowing modders to decide when to enable a specific flag in FFNx, based on what the user configured inside of their own mod panel.
Through this new patch we now do support two possible scenarios.
Scenario 1
A modder wants to set always this flag, no matter what. The syntax to achieve it remains the same as specified in #34
<?xml version="1.0"?>
<ModInfo>
  <Author>Julian Xhokaxhiu</Author>
  <Version>0.1</Version>
  <Description>FFNx Config override example</Description>
  <Link>https://github.com/julianxhokaxhiu/FFNx</Link>
  <FFNxConfig>
    <ff7_footsteps>true</ff7_footsteps>
  </FFNxConfig>
</ModInfo>

Scenario 2
A modder wants to set a flag, but only when a given option has been enabled. The syntax to achieve this scenario is the following:
<?xml version="1.0"?>
<ModInfo>
  <Author>Julian Xhokaxhiu</Author>
  <Version>0.1</Version>
  <Description>FFNx Config override example</Description>
  <Link>https://github.com/julianxhokaxhiu/FFNx</Link>
  <ConfigOption>
    <Type>Bool</Type>
    <Default>0</Default>
    <ID>Footsteps</ID>
    <Name>Footsteps</Name>
    <Description>Enable Footstep audio.</Description>
    <Option Value="0" Name=""/>
    <Option Value="1" Name=""/>
  </ConfigOption>
  <FFNxConfig>
    <ff7_footsteps Footsteps="1">true</ff7_footsteps>
  </FFNxConfig>
</ModInfo>

Please note that ff7_footsteps now has a new attribute, which is the Option -> ID tag name ( in this case Footsteps ) and its expected value ( the 7th code treats possible option values always as an int, so you can put basically only numbers in the attribute value ), in this case 1.
The code will then check, for each flag specified, if ALL of the written attributes do match. If at least one of them will not, the flag will NOT be set and the code will move on. There's no limit on how many Attributes ( Options ) you can specify, although it matters the way you write it: attributes are Case-Sensitive ( so if your option ID is written footSteps, the attribute in the flag node has to be written as well as footSteps ).
I've tested this code already on my own end and it seems to work fine, although feel free to check it also on your own side and feel free to merge when you think it's good enough to pass.

Hey,
Sorry i just want to make sure this is clear (for documentation purposes) and that I have got it correctly,
<ff7_footsteps Footsteps="1">true</ff7_footsteps>

Means set to true if Footsteps=1, and if I was to add another option E.G
<ff7_footsteps Footsteps="1" Lang="1">true</ff7_footsteps>

Means set to true if Footsteps equal to 1 & Lang equal to 1.

Means set to true if Footsteps equal to 1 & Lang equal to 1.

Exactly.


adding new tests for 160m benchmarking

As discussed with @achalddave

this has local paths and will not be runable on other setups. what is the rational for checking this script into tests?

Sunny, Achal, and I already talked about the PR on Slack. Sunny is going to re-work it.

Member


Turn MainApp inits into static methods

We could possibly construct instances like ModelManager via static method calls from the same class, instead of having them as MainApp methods.
This may also make it easier to be more efficient, where constructors don't need to needlessly construct new copies of objects.

We are close to implementing final features, going into the bugfixing phase, and wrapping up the project, so closing this idea.


Installation

How can i install this in my Project(Like cocoapods)..?
I have installed but the error showing like this 'Cannot invoke initializer for type 'Hedwig' with an argument list of type '(hostName: String, user: String, password: String)'

@abdulmalikaman
Have you found any way for this ?

Not yet Bro..
They r not providing a proper document or sample project..

they are not proper document for installation


integrate with travis-ci and coveralls

integrate with travis-ci and coveralls

greate work :+1:

greate work :+1:


Implement bootstrap for Windows

At the moment bootstrap script does not work on Windows.
There should be an easy way to make a conveniently setup Windows machine.

The initial working bootstrap implementation for Windows was merged at commit c088a560be425d52a730f791c4f96a7e4274fe02.
To close this task, there are still items to complete (like checking network, configuring hostname, adding user, etc.) which implemented for Linux but not for Windows.

The bootstrap and default highstate (including setting up SSH on bootstrap) for practical purposes has been merged at 34e8bcdf748052dd60b046df9146c2a15823920c - this task is closed.
Some networking (adding default route and pinging the router) is not done, but can be completed along rising needs.

Commit 919ec28e56691fba0d76edceecb96b84c734579b completes all steps except link_sources and link_resources which are only needed for standalone minion (and this has to be refactored with generic approach to system_resources).


Dashboard-style layout

Move the frontend to a style like these:
http://marketblog.envato.com/resources/bootstrap-admin-templates/

Base Features:
Share jobs with other users
Share files with other users
Future Enhancements:
Job file output management via something like:
File manager:
http://django-media-tree.readthedocs.org/en/latest/extending.html

The work on the update interface is in full swing in the interface-updates branch. At present it's looking much like flask-wooey and the navigation is broken but this should be resolved soon.
Once all the base views are complete I'll start work on building the dashboard interface.

Sounds good, want to create a PR so we can discuss the branch and comment on code changes in there/etc?

Just tried, but it's creating the pull-request against your repo at django-djangui. Is it possible to create a PR within the same repo (branch to master)?
I could always fork this to a private repo and open a PR from there.

If you change the base repo to here it'll make it against this codebase.

Ah, kind of obvious now you mention it. PR open here https://github.com/wooey/django-djangui/pull/16

PR #16 merged as ede20d9 although it doesn't include all the features listed here. Should we move this to a new issue on the other repo?

I think a new issue is warranted, we can work on these features there.

Moved to https://github.com/wooey/Wooey/issues/21


JPetStatistic update

THxF -> THxD (x = 1, 2, 3) requested change: Every file that uses histograms the old way with THxF needs to be adapted to ThxD
TH3D histograms handling
Defining histogram with axis names, not returning object to user to stop crashes. Returning only Errors in terminal and in log to notify user about errors
Filling histograms not by returning histogram to the user, also to stop crashes, only notifications are written
Square histograms defined as a square TCanvas. Histogram is filled separately, square histogram is drawn on the TCanvas with the same name as histogram, with addition "_Square". There is default option to draw 2D histos in colz style, because after saving there were some problems with using drawing options in TBrowser fro TCanvas

It seems that your PR contains changes related to JPetLogger which are already on develop (likely due to the merge commit in the history of this PR). Please try to apply your commits on top of a branch initially consistent with develop. You can do it e.g. by fetching develop from this (upstream) repository, making a new branch from it and using git cherry-pick to move your three commits to this new branch, so that there are only your changes on top of the current develop.
Otherwise we cannot merge this PR without making a mess in the Git history.

I made a mess in a history for my master branch, so I close this PR, and open a new from clean branch. Sorry for inconvinience.