Can't combine query string with URI string ($_GET not work)

My previous application use $_GET in many feature, but the link detected as it has disallowed characters. My example link like this:
http://localhost/myapp/panel/pages/edit?page=lipsum/subpage
and it shows error "The URI you submitted has disallowed characters."

This is a support issue, not a bug. It should be raised on the forum, not here.
See http://www.codeigniter.com/userguide3/general/urls.html#enabling-query-strings
You might be able to use http://www.codeigniter.com/userguide3/libraries/uri.html#CI_URI::uri_string
Or you might have to use the appropriate PHP superglobal.
Your previous usage may have worked, but it is not supported.
Your specific error message comes from http://www.codeigniter.com/userguide3/libraries/security.html?highlight=disallowed
You could have searched the user guide too :-/

So then in v3 we cannot use query string and uri string together anymore?

You might be able to, but you will need to deal with the security, perhaps turning it off. Ask on the forum for suggestions - I could be misinterpreting things :-/

So then in v3 we cannot use query string and uri string together anymore?

The valid characters are configurable.


How to use?

How to use?
How to use?
How to use?

try example
https://github.com/39ff/google-unlimited-search-scrape/blob/master/example/00-google-scrape.php


IP not printing in serial monitor

Hello,
I'm using web_en = true in the code.
I just uploaded the code to my arduino but during the upload process there was no IP printed out in the serial monitor.
How do I get the webpage IP exactly?
**UPDATE:
I changed the baud to 115200 which fixed the problem so now it displays:
ready!
waiting for LAN connection...
But still, how do I get the IP address?**
Much appreciated.

The IP adress will be printed out as soon as the device got a connection to your LAN.

Thank you and also, does the IP change if it is connected to a different LAN or will it always be the same?

no it gets an IP by your DHCP so you have to look that up every time.

Thank you, much appreciated.


Moved client-side dependencies to NPM

Currently using browserify, majority of client-side dependencies are now on NPM

So, a couple of questions:

Does this work as is?
If not, what is the error?

The jquery stuff should be on npm as wlel, isn't it?
The handlebars templates can just live in the repository, as they are not libraries (I've never understood why they were in the libs/ directory, they are custom stuff for the app itself).

@YaManicKill No, it doesn't work, this is the progress I made and I'm currently not sure how to go about finishing it. I didn't look into the Jquery libs yet, and I plan to leave the Handlebars as they are now. I think they should be moved in a separate PR; enough refactoring will occur when this is finished

Looking good so far!
Did you forget to add socket.io-client?

@xPaw : according to the socket.io-client npm page:

A standalone build of socket.io-client is exposed automatically by the socket.io server as

I think its supposed to work now, I'm just not sure how to setup grunt/browserify to compile everything correctly. Any ideas?

Update: afaik, it should now work (but doesn't, and no errors are shown). By running npm run build, you grunt-browserify all of the files into 'libs.js', and minify/uglify them with grunt-uglify.

Current status: Jquery plugins inside libs.js aren't working in lounge.js

Current status: It works! Just need to update to be even with master and squish commits, then it should be good to go :)

Right, so, in general this looks pretty good. If you could do the 1 suggested change and rebase, I'll then have a play to make sure it works for me as well, and then I'll give my +.
Great work, @MaxLeiter I'm really happy this is going to be done.

Alright, @MaxLeiter sorry to do this to you, but we might end up not using browserify.
I'd suggest pausing any work on this as of just now until we get through discussion of what module system to use, etc. We have to make sure that it'll work with not only this, but packages. And one of the key things about packages is that users should be able to install client packages and have them available in the browser without restarting the server. Browserify wouldn't allow us to do this (without having a watch running, which isn't good for production).
@astorije Where do you want to have the discussion about this? I know you are investigating a couple of things but it would be good to have a central place for deciding this, and I'd really like to have us decide this in the next few days as a lot of things rely on it (this PR, the package system, #66 all require us to have the module loader in place, and they are all things that we want to get done reasonably soon).

And one of the key things about packages is that users should be able to install client packages and have them available in the browser without restarting the server. Browserify wouldn't allow us to do this (without having a watch running, which isn't good for production).

There's nothing preventing the user to run browserify seperately without stopping the server. Actually, even with the current setup, if all the changes are client-side and don't depend on the server's assistance, you can even do a git pull + npm install over a running lounge instance and it will update just fine. So I still think the solution is valid. The only reason we talked about watchify was for people that don't want to run the build command after doing changes :)
As the client grows, eventually it just won't fit on a single file. It's already getting rather large at the moment, so we will need a build system sooner or later.

Alternatively, later we could also possibly split the server and client components into two seperate repos to handle this kind of thing.
This way,

We could use a client-side dependency manager without it being a mess
We won't have to worry about the server part
Probably won't even need npm at all.

The only downside of this solution is client/server version mismatch. But that shouldn't be a big problem if we structure things properly (have the client say what version it supports?) and establish a proper client/server protocol instead of just dumping whatever feature we need here.

So, what exactly is blocking this from getting merged? The client module handling right now is atrocious with no package manager at all, just a bunch of files in a folder.
This is a big step on the way towards a better structure, and while the module structure is being reworked over in #77, contributors can work on the project while keeping their sanity intact.

@lindskogen some people are against the idea of having to restart the server in order to install client plugins

Alright. Let's not block this with the debate around packages, as this is happening, and packages aren't currently happening.
I'm happy to go with browserify for this, and we can work out if we want to deal with something else for packages later. But yeah, I'm currently getting to the point of just wanting to use browserify for it all. If we have to restart the server, that's not the end of the world, and we should really implement a database to store the scrollback to fix that issue.
@MaxLeiter What state are we in for this? You happy to continue doing this? Could you rebase and squash commits, and I'll take another look at the code afterwards and try and get this in. It'd be really good to get the dependencies out of the repo.
Any objections, @thelounge/maintainers ?

My objection is that it generates a single js file. I'm strongly of opinion that lounge.js should be separate file.

It previously made 3 files, that's an easy fix. How many and what files would you like to see, @xPaw? compiled/libs.js, compiled/lounge.js, and compiled/lounge.templates.js?
The reason for one file, iirc, is because templates requires a required Handlebars object, but you can't add a require() into lounge.template.js because it gets rewritten every 'npm run build'
@YaManicKill: it works as-is, will rebase and squash tonight.

compiled/libs.js, compiled/lounge.js, and compiled/lounge.templates.js?

Ideally yes, because that would keep the existing structure.

Any idea on how to fix the Handlebars in templates? Previously I had a hack of appending to the file using grunt, which was an effective but dirty fix.

Fine, fine. If you feel that strongly about it, @xPaw then we can go for the existing structure and debate what we want to do in the future.

Any idea on how to fix the Handlebars in templates? Previously I had a hack of appending to the file using grunt, which was an effective but dirty fix.

What's the issue with it, @MaxLeiter ?

What was the issue with require? Isn't this exactly the problem that browserify solves?

Templates requires a require('handlebars'), the issue is that whenever you run npm run build, lounge.templates.js gets overwritten and the require is replaced.

Templates requires a require('handlebars'), the issue is that whenever you run npm run build, lounge.templates.js gets overwritten and the require is replaced.

I'm not sure I understand, were you telling browserify to overwrite lounge.templates.js with the compiled version? If so, that's your problem, just don't do that, have a compiled file which is different.
If that's not what the problem is, then I'm not sure I understand, why is it overwriting the file?

https://github.com/thelounge/lounge/blob/master/package.json#L15
The templates file gets replaced, so any changes made are overwritten.

So why not compile templates before running grunt then?

Ah right, I see your issue, @MaxLeiter that makes sense.
I think having Handlebars as a global is fine then, just don't run the templates through browserify if the only thing they require is having handlebars.

Any luck with this, @MaxLeiter ?

Not trying to rush you, @MaxLeiter, just interested in the state of this, you had any time to work on it? Would be good to get this through.

@YaManicKill: There are a huge amount of merge errors which I'm trying to figure out, next time I have time I'll hop on IRC

As discussed on IRC, at the point things are and how lounge evolved it's probably better to just start over fresh for this.


flush vpd every cycle for debugging

Makes VPD dumps are flushed every cycle. This is important for debugging when we encounter any exceptions in tester, but can be much slower.

I don't the performance impact yet, but controlling it by a flag is a good idea.


First new release with AnnubesEnv functionalities and improved CI

 @gcroci2 cloned issue ANNUBS/neurogym#48 on 2024-09-13:

Once the CI will be properly set (hopefully in the original repository), and once we will have added tests and finished to add the remaining main functionalities to the AnnubesEnv class, do a first release via PyPI and on GitHub. Add the necessary documentation on the readme for then use the released package.
Archive the old annubes repository and add a disclaimer about the neurogym one.
Remember to check and update the CHANGELOG.md file

Blocked by all the issues in the columns "Todo_pkg"

🥳


Options of processing ROIs in parallel within wells.

The current vision for ROI processing (see https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/27) is running all the ROIs in a well sequentially. This will be a very useful first implementation. Many operations inherently parallelize and we can run many wells in parallel.
Nevertheless, we may eventually want to parallelize some ROI processing within a well. This becomes hard when ROIs need to write to the same chunk of the zarr array, which would not be safe. But we can think of ways to handle this. I think this roughly goes in the following order:

Run all ROIs sequentially (current plan to implement)
Run ROIs in parallel if they are all saved to independent chunks of the zarr array. Basically, when we have a grid of field of views, we want to run some processing by field of view at level 0 (e.g. illumination correction) and we saved the field of views as chunks in the zarr file. If we can verify that this is the case, we can run the tasks in parallel.
Run batches of ROIs that write to independent chunks in parallel. We may have some ROIs that need to write to the same chunk, but most ROIs don't overlap. In that case, we could check which ROIs need to write to independent zarr chunks and batch them in a clever way that groups ROIs that can be processed independently. These can then be processed in parallel.

We will implement 1 now. 2 should be fairly doable and useful. 3 is more of a potential thing we could eventually do, so we don't forget we have that option

As of ongoing discussions with @jluethi and @mfranzon (related to #72, #27 and #75), we are now only implementing strategy 1, where all ROIs are computed sequentially (as in "define computation for a certain ROI, execute it, free up memory, move on to the next ROI").
This allows us to simplify the function that gets mapped onto the well array, and to make it such that its I/O are both numpy arrays (rather than delayed arrays).
Moving towards strategies 2 or 3 will clearly require a refactor of the relevant tasks, because within the in-progress work (see https://github.com/fractal-analytics-platform/fractal-tasks-core/commit/5b61cd90a469e615acf56e99eaeda21fd70d31ef and #75) each ROI computation is blocking - and nothing else happens until it's over.

Fully agree, thanks for the summary Tommaso.
When we want to tackle this eventually, we'll have to find a way to call the functions with numpy arrays, but somehow remaining delayed in this call. At the moment, the conversion of the dask region to a numpy array forces computation.
I think this sequential per well approach should be fine for quite a while, because we parallelize over the wells. I see 3 reasons when we may need to reconsider this trade-off:

We want to process large OME-Zarr images, especially things that aren't in the HCS spec (=> no wells to parallelize over). That's not on our current roadmap, but eventually interesting
We want to parallelize more for the GPU operations: The current implementation also means only one thing runs on a GPU at a time. If we e.g. do segmentation at level 2, we could often fit multiple jobs in a typical GPU at the same time. But the current implementation likely won't allow for that. (because sending multiple well jobs to the same GPU can become tricky)
We want to optimize for processing HCS datasets with very few wells (similar to 1) => eventually interesting, but not the immediate roadmap

Another thing to consider: I've started processing the 23 well dataset again and the parsing to OME-Zarr now seems to take about 10 hours. Looks like that is a bit slower than before. I think the biggest bottleneck is parallel IO performance, so that's not something Fractal can optimize. But given that it seems to have slowed down a bit (I remember this being in the 6 hour range before), there may be a bit of optimization potential.
One thing we could consider: Currently we're parsing all the channels sequentially. An potentially easy way to get more parallelization without having to process multiple ROIs in parallel would be to process the different channels in parallel for a given FOV.

cc @lorenzocerrone on this issue. Will be something that we eventually cover in the OME-Zarr reader/writer class :)


Core cvSeqRemoveSlice: do not proceed with removing zero-length slice

Fixing valgrind reported issue: call memcpy with source equals destination since slice length is 0.
[ RUN ] Core_DS_Seq.basic_operations
==974== Source and destination overlap in memcpy(0x504b4b0, 0x504b4b0, 28)
==974== at 0x48495F4: memcpy (/home/nvidia/ilya/valgrind/memcheck/../shared/vg_replace_strmem.c:1016)
==974== by 0x4AC07D3: cvSeqRemoveSlice (/modules/core/src/datastructs.cpp:1725)
==974== by 0x5496F3: Core_SeqBaseTest::test_seq_ops(int) (/modules/core/test/test_ds.cpp:911)
==974== by 0x54A1EB: Core_SeqBaseTest::run(int) (/modules/core/test/test_ds.cpp:1026)
==974== by 0x5DE0F3: cvtest::BaseTest::safe_run(int) (/modules/ts/src/ts.cpp:214)
==974== by 0x54FE47: Core_DS_Seq_basic_operations_Test::TestBody() (in /home/nvidia/build_opencv/bin/opencv_test_core)
==974== by 0x6051D7: void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::)(), char const) (/modules/ts/src/ts_gtest.cpp:3578)
==974== by 0x5FFADF: void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::)(), char const) (/modules/ts/src/ts_gtest.cpp:3614)
==974== by 0x5EB907: testing::Test::Run() (/modules/ts/src/ts_gtest.cpp:3651)
==974== by 0x5EC183: testing::TestInfo::Run() (/modules/ts/src/ts_gtest.cpp:3826)
==974== by 0x5EC847: testing::TestCase::Run() (/modules/ts/src/ts_gtest.cpp:3944)
==974== by 0x5F361F: testing::internal::UnitTestImpl::RunAllTests() (/modules/ts/src/ts_gtest.cpp:5823)
==974==
==974== Source and destination overlap in memcpy(0x5072fe8, 0x5072fe8, 1)
==974== at 0x48495F4: memcpy (/home/nvidia/ilya/valgrind/memcheck/../shared/vg_replace_strmem.c:1016)
==974== by 0x4AC066B: cvSeqRemoveSlice (/modules/core/src/datastructs.cpp:1707)
==974== by 0x5496F3: Core_SeqBaseTest::test_seq_ops(int) (/modules/core/test/test_ds.cpp:911)
==974== by 0x54A1EB: Core_SeqBaseTest::run(int) (/modules/core/test/test_ds.cpp:1026)
==974== by 0x5DE0F3: cvtest::BaseTest::safe_run(int) (/modules/ts/src/ts.cpp:214)
==974== by 0x54FE47: Core_DS_Seq_basic_operations_Test::TestBody() (in /home/nvidia/build_opencv/bin/opencv_test_core)
==974== by 0x6051D7: void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::)(), char const) (/modules/ts/src/ts_gtest.cpp:3578)
==974== by 0x5FFADF: void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::)(), char const) (/modules/ts/src/ts_gtest.cpp:3614)
==974== by 0x5EB907: testing::Test::Run() (/modules/ts/src/ts_gtest.cpp:3651)
==974== by 0x5EC183: testing::TestInfo::Run() (/modules/ts/src/ts_gtest.cpp:3826)
==974== by 0x5EC847: testing::TestCase::Run() (/modules/ts/src/ts_gtest.cpp:3944)
==974== by 0x5F361F: testing::internal::UnitTestImpl::RunAllTests() (/modules/ts/src/ts_gtest.cpp:5823)
==974==

:+1:


Use span styled as table instead of line break

fixes #33

Yeah, this seems to work! I tested with Voiceover on a few browsers.


Hyphenate ISBN

Hey. Just curious if you guys want to include additional feature of formatting ISBN. If so, you can use beautify-isbn, which I crafted for my library software project. So the ISBN agency ranges are up-to-date all the time. :)

This appears to be out of scope of this library.
Thanks anyway.


when I use the version of svg.js V2, the latest version of svg.topath.js can't work well and report error;

That's how I import the plugin :
import<EMAIL_ADDRESS>and that's the errors:
1.DOMException: Failed to execute 'setAttribute' on 'Element': '0' is not a valid attribute name.
2. Error in mounted hook: "InvalidCharacterError: Failed to execute 'setAttribute' on 'Element': '0' is not a valid attribute name."

The latest version ot svg.topath.js depends on svg.js v3.0.10. Even the latest commit messages states that:

update plugin for use with svg.js v3

So please use the correct version


Remove support of skip-git option

There is no reason or need to roll a new app without Git.
Doing so breaks Heroku setup as we need to setup Git remotes.
This commit removes the support of this option until we really need it.

:+1:


Quadratic time complexity of group

The ⊕ group operator runs in O(n²) time.
From the implementation I would expect it to be O(n) when indices are unique.
Tested on the current main branch:
$ cat main.ua
;⊕∘.⇡ 100
;⊕∘.⇡ 1000
;⊕∘.⇡ 10000
;⊕∘.⇡ 100000

$ uiua run --time-instrs  | grep ⊕
  ⏲  0.02ms - ⊕
  ⏲  0.21ms - ⊕
  ⏲  14.25ms - ⊕
  ⏲  1320.36ms - ⊕

For comparison, putting everything into the same index scales fine:
$ cat main.ua
;⊕∘.↯ 1000 0
;⊕∘.↯ 10000 0
;⊕∘.↯ 100000 0
$ uiua run --time-instrs  | grep ⊕
  ⏲  0.01ms - ⊕
  ⏲  0.00ms - ⊕
  ⏲  0.00ms - ⊕

Here is the code for group.
I can't figure out where it would be quadratic.

They're both quadratic, it's just that for ;⊕∘.⇡ 30000 the slow part is timed as ⊕, but for ;⊕∘.↯ 30000 0 it isn't.
I bet something is copying instead of reusing a CoW when building a large value. Maybe an accidental refcount increase, maybe something else. (When matching shapes / fills?)
My profile trace shows that these functions are hot, descending the call stack, for those two executions:
Primitive::run
loops::group
loops::collapse_groups

# only in ;⊕∘.⇡ 30000
    Value::from_row_values
    Value::append

# only in ;⊕∘.↯ 30000 0
    Value::group_groups
    Array::from_row_arrays_infallible

Array::append
<CowSlice as Extend>::extend
CowSlice::modify
<EcoVec as Extend>::extend

(Don't take that as gospel, I wrote it out by hand.)
Notably Value::group_groups is hot when collecting a lot of 0s into a single uiua::Array<_>, and Value::from_row_values is hot when collecting a lot of unique rows into the output uiua::Value.
I might be able to help more but frankly I think I need to leave the rest to someone who knows the interpreter.
This might also be a bug in the time reporting and I have no idea where to even start with that.


Use an RW lock to checkpoint fingerprint mappings.

This has to be backported to 0.13.x.
@fabxc @juliusv

:+1:

I have no created a new branch release-0.13, backported this fix to that branch, and cut 0.13.4 there. Do you think that is a sane approach? @fabxc @juliusv

Yes, that's the right way to go :+1:
On Sat, May 23, 2015 at 2:53 PM Björn Rabenstein<EMAIL_ADDRESS>wrote:

I have no created a new branch release-0.13, backported this fix to that
branch, and cut 0.13.4 there. Do you think that is a sane approach? @fabxc
https://github.com/fabxc @juliusv https://github.com/juliusv
—
Reply to this email directly or view it on GitHub
https://github.com/prometheus/prometheus/pull/727#issuecomment-104889630
.

:+1:
@fabxc @beorn7 - please remember to delete branches after merging (via the "Delete branch" button that appears immediately after pressing the merge button). Otherwise we'll continue accumulating old PR branches...

OK, but the release-0.13 branch is special as we will never merge it into master. (Or will we?) OTOH, the commit and the tag is there, so we would not need that branch anymore, and we could recreate it if we ever need to cut 0.13.5...
Anyway, building the binary now.

@beorn7 Yeah, that one should be fine. I was talking about the beorn7/fingerprint branch. Just have seen a lot of not-deleted branches after PR merges these days :)

I see... I deleted all my merged branches now. @juliusv There are a number of very old stale branches listed at https://github.com/prometheus/prometheus/branches/stale . Perhaps you can check them out and delete as you see fit.


datastore: insert example has busted Get and datastore.ErrNoSuchEntity check

The Go version of the docs for inserting  (in contrast to upserting) point to this code sample which has a bug.
if err := tx.Get(taskKey, nil); err != datastore.ErrNoSuchEntity {
        log.Fatalf("tx.Get returned err %v, want ErrNoSuchEntity", err)
}

ErrNoSuchEntity will never be returned because calling tx.Get with a nil message is always returning ErrInvalidEntityType.
The error occurs inside newStructPLS. The call chain is Transaction.Get to Client.get to loadEntity to LoadStruct to newStructPLS.
I'm not sure where along in this change we should allow the nil. Client.get seems the most reasonable place to check for it? Maybe only allowed in the case of a single key query coming straight from a Get and not MultiGet call?
Or maybe there should be a Client.GetKey?

The workaround, of course, is to use an actual pointer to a struct for this and just ignore whatever gets put inside it.

The example seems to be misleading, we need to pass the task's pointer.

Yes, we should fix this.
The example is obviously wrong, but I feel like we should accept nil for Get (in the same way that we do already for the query iterator's Next method) for cases like this when all you want is an existence check.

If we should support nil on Get, maybe keeping the example as-it-is is a better idea.

I'd vote to fix the example now, and support Get(key, nil) later (since it's more of a feature request anyway).

Thanks!


feat: support TypeScript in --test flag

fixes https://github.com/privatenumber/tsx/issues/257
This adds TypeScript file detection to Node v21 and higher. Older versions are not supported because it requires a glob file-traverser and it sounds like quite a bit of implementation & maintenance work when users can just upgrade Node.

:tada: This issue has been resolved in v4.4.0
If you appreciate this project, please consider supporting this project by sponsoring :heart: :pray:


Update melcloud stable version to v1.3.5

Hotfix needed because of memory leak in current stable version (see https://github.com/Black-Thunder/ioBroker.melcloud/issues/542)

1.3.5 created 01.08.2023
22 user (2,7%)
Hotfix
OK


Add support for optional function arguments

VimL supports optional function arguments.
This change doesn't care much about the valid order of arguments, it
just enumerates the possible arguments.
See h: optional-function-arguments in vim for docs on the language
construct.
Fixes  #54

I could not run tree-sitter generate with the --abi=latest flag, it won't accept it. The parser changes do seem to work for me with ABI=13 though.

I could not run tree-sitter generate with the --abi=latest flag, it won't accept it. The parser changes do seem to work for me with ABI=13 though.

In order to use the latest version of treesitter, you can

on archlinux, tree-sitter-git in the AUR provides the latest version of tree-sitter
build from source:

git clone https://github.com/tree-sitter/tree-sitter
cd tree-sitter
cargo install --path cli
make
sudo make install

the latest cli should be located in target/release/tree-sitter

I have tested the pr locally with the latest ABI version and works nicely ! 👍

I'll regenerate the parser and all, but it all seems great !

Thanks @vigoux !

@vigoux this is interesting, it seems like it doesn't work for me with the latest ABI, only with 13...
I'm using the latest release of tree-sitter (0.20.2).


Patch 4

Folders for the rest of the year

@Kamhorn In general, when you use dates in file or folder names (called a file path), you want to use the ISO8601 format for dates. That is the form of YYYY-MM-DD, for instance, 2023-09-22 for today. I've renamed the folders for you. Likewise, from a general structural point of view, since the epi stats are sessions based on date, I would use only the date as the folder, and the file name inside either be called the session or have that information somewhere in a file in that folder. So I've created some plain text Markdown files for you instead. If you want to keep a folder in Git, a recommended approach is to include an empty plain text file like .md, and not to use .pdf since PDF are not as easy to work with from a programming/automation point of view.


multi-world-patches in global config works incorrectly.

https://github.com/SpongePowered/SpongeCommon/blob/851bff2a36f910feaa3f501b257bdc42317f6486/src/main/java/org/spongepowered/common/mixin/plugin/MultiWorldCommandPlugin.java#L56
The code here does not change after the Mixin classes renaming.

Follow the template like everyone else has to do and explain exactly what is going on and why it's a problem. This is a lazy attempt at an issue and, as someone who has been trying to prioritise issues, I honestly have no idea why this is an issue.
Just posting a code snippet and saying "this does not work" or "this no longer does what it should" is not helpful to those of us who haven't touched this part of the system and therefore might not necesarily know why this is a problem.

@dualspiral I'm sorry. edited.

And I find an old global.conf file whit this part
        multi-world-patches {
            defaultgamemode=true
            difficulty=true
            gamerule=true
            seed=true
            setdefaultspawnpoint=true
            time=true
            toggledownfall=true
            weather=true
            worldborder=true
        }


Extend Controller to BaseController by default

Description
Create a default extended controller at app/Controllers/BaseController.php and updates Home controller and the User Guide to recommend using this for new Controllers. From a development standpoint, it makes sense to start with this extension even if it doesn't have any modifications since making the change down the road involves going back and editing every controller.
There was a fair amount of discussion on this idea in the forums already. See: https://forum.codeigniter.com/thread-73095.html
Checklist:

[X] Securely signed commits
[X] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[X] User guide updated
[X] Conforms to style guide

(:any) is normally a segment placeholder, not "part" of a segment.
Having said that, the "$routes->cli('ci(:any)',..." seems very odd, and makes no sense to me. WHy would one map ci1 or ci4 to a CommandRunner index method? I could be misinterpreting.
My comment about the slash in the basecontroller rule still stands, and I suugested it be moved above the cli rules in case one of them kicked in before it, sending the request down a deep, dark path that we don't want :-/
I think your rule will fire a 404 for /basecontroller, but not for /basecontroller/anythingelse.

I've tested and it definitely matches as I have said: /basecontroller, /basecontroller/, basecontroller/test, etc.
I will move the rule above CLI - let me know if you want me to changes anything else with it.

Good to know about the (:any) - I learned something new :)
We still need to determine why the CommandsTest::testRoutesCommand() is failing, across the board.
The base controller and the additional routing rule are the only two things I see different, and I don't offhand see a reason why they would break that test.

Can you tell if it is testing with https://github.com/codeigniter4/CodeIgniter4/pull/1842 in place? If not then forPageNotFound() without a parameter will throw a syntax error.

1842 has been merged, so will be part of the tests.

(:any) is normally a segment placeholder, not "part" of a segment.

Yup. In CI4 :any was changed to mean, literally, anything. :segment is available if you want just a single segment.

The test is failing because the CLI command routes tries to parse the callback as a string. I sent PR https://github.com/codeigniter4/CodeIgniter4/pull/1859 for that command to filter on strings only, though at some point someone might want to figure out a way to have it display something helpful when a function is detected. I initially had it noting the callback but it made for a very messy output in a vanilla setup with no user-defined routes:
+----------------------------+---------+------------------------------------------------+
| Route                      | Method  | Command                                        |
+----------------------------+---------+------------------------------------------------+
| /                          | get     | {callback}                                     |
| basecontroller(.*)         | get     | {callback}                                     |
| (.*)/initController        | get     | {callback}                                     |
| basecontroller(.*)         | head    | {callback}                                     |
| (.*)/initController        | head    | {callback}                                     |
| basecontroller(.*)         | post    | {callback}                                     |
| (.*)/initController        | post    | {callback}                                     |
| basecontroller(.*)         | put     | {callback}                                     |
| (.*)/initController        | put     | {callback}                                     |
| basecontroller(.*)         | delete  | {callback}                                     |
| (.*)/initController        | delete  | {callback}                                     |
| basecontroller(.*)         | options | {callback}                                     |
| (.*)/initController        | options | {callback}                                     |
| basecontroller(.*)         | trace   | {callback}                                     |
| (.*)/initController        | trace   | {callback}                                     |
| basecontroller(.*)         | connect | {callback}                                     |
| (.*)/initController        | connect | {callback}                                     |
| migrations/([^/]+)/([^/]+) | cli     | \CodeIgniter\Commands\MigrationsCommand::$1/$2 |
| migrations/([^/]+)         | cli     | \CodeIgniter\Commands\MigrationsCommand::$1    |
| migrations                 | cli     | \CodeIgniter\Commands\MigrationsCommand::index |
| ci(.*)                     | cli     | \CodeIgniter\CLI\CommandRunner::index/$1       |
| basecontroller(.*)         | cli     | {callback}                                     |
| (.*)/initController        | cli     | {callback}                                     |
+----------------------------+---------+------------------------------------------------+

Is there a way to rerun the runner now that the other PR is merged?

Is there a way to rerun the runner now that the other PR is merged?


Test-Json doesn't take in count the dependencies in the json schema.

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I am working in a script to validate if a version defined in a JSON file is valid (major or equal). And to do this I choose the follow items to define the version:

Major
Minor
Patch

But the thing is that the patch property can only be define when the minor property exists. To do that I use the "dependencies" definition in the JSON schema.
Schema:
{
    "$schema": "https://json-schema.org/draft/2019-09/schema",
    "required": [
    	"major"
    ],
    "properies": {
    	"major": {
        	"type": "integer"
        },
    	"minor": {
        	"type": "integer"
        },
    	"patch": {
        	"type": "integer"
        }
    },
	"dependencies": {
		"patch": ["minor"]
	}
}

{
    "major": 1,
    "patch": 1
}

In this case, the result of Test-Json must be False, but this function is not taking in count this property and return true, because "major" exists. I tested this example in this web page and it shows the expected result:
https://www.jsonschemavalidator.net/
Dependencies in json-schema.org:
https://json-schema.org/understanding-json-schema/reference/conditionals.html#dependencies
Expected behavior
PS> Test-Json -Json $(Get-Content -Path .\test.json | Out-String) -SchemaFile .\test.schema.json;
False

Actual behavior
Test-Json -Json $(Get-Content -Path .\test.json | Out-String) -SchemaFile .\test.schema.json;
True

Error details
No response
Environment data
PS> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.1.3
PSEdition                      Core
GitCommitId                    7.1.3
OS                             Microsoft Windows 10.0.19043
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

PowerShell uses https://github.com/RicoSuter/NJsonSchema so please report the issue there.


Adding decorative SVG attributes

This fixes https://github.com/CovidShield/portal/issues/10

@dfmcphee Missed one in the modal window close control.
<svg class="modal__close-icon"><use xlink:href="#close"></use></svg>

Good catch @svinkle. I added the attributes to the modal close button as well.


Fixes as discussed f2f.

Removes form attribute test from hasSpacingAttributes
Removes  setting of texclass=none from setTeXclass method.
Comments out incorrect range setting in setInheritedAttributes

Thanks, Volker!


Wrong columns rendered after reload of gridOptions.data

If you have a wide table with a lot of columns and a horizontal scrollbar, and you scroll all of the way to the right of that table and update the data backing the grid, the wrong columns get rendered. Example: http://plnkr.co/m9dWODm9cqqzscPY09L1. It appears to chop off the last few columns until you adjust the scrollbar to move slightly to the left.
I think it's because GridRenderContainer miscalculates the horizontal scroll percentage in adjustColumns, which leads to the range of viewable columns being off by a few. In the scrollHorizontal function the horizScrollPercentage is calculated using both the canvas width and the viewport width, but in adjustColumns only the canvas width is used. Should both functions be using the same method for determining horizontal scroll percentage?

Is this related to #3527 perhaps?

Hmm, although it looks similar I think it's probably a different root cause. For this issue it actually helps to scroll horizontally a little so that the correct horizontal scroll position is calculated.


Link to paper from github README and cabal description

It may help people understand, it may help the paper gain more attention.

Done!


IntegerField Validation

IntegerField validation doesn't provides user friendly message, Code for handling that was there already but don't know why It wasn't triggered. I haven't debugged it.

.isnumeric() is not a good way to do this as it does not work with negative numbers.
>>> "123".isnumeric()
True
>>> "-123".isnumeric()
False


Version 2.13.x error with floatingLabel

Hi!
Issue Description
When using input with floatingLabel attribute + error validation, I suddenly got problem after I upgrade to version 2.13.x.
Expo 33 (or 34), Native Base 2.13.x
Expected behaviour
As example on native base doc https://docs.nativebase.io/Components.html#error-textbox-headref
Actual behaviour
Click on the input to type text, after typing, blur out of the input, app not responding.
Steps to reproduce
Check my snack right here https://snack.expo.io/@yanghuyming/a7a874
Don't wait for snack to render the code. It cannot render :) There is an error with the code that I just copied from Native base doc. That's why I open this.
Is the bug present in both iOS and Android or in any one of them?
Android (iOS not tested)
Note:
Roll back to native base version 2.12, it works normally.
Thanks.

@minhdang-idev Can you please share a screenshot of the error you're getting?

It is the same issue as #2786 Here is the minimum code for a crash
      <Content padder>
        <Form>
	<Item floatingLabel>
          <Icon name='mail' style={{ color: 'gray' }} />
	  Email
	  <Input />
	</Item>
	<Item floatingLabel last>
		 <Icon name='mail' style={{ color: 'gray' }} />
	  Password
	  <Input />
	</Item>
</Form>
      </Content>

Hi @tosbaha, can you try cleaning your cache and checking again?

@minhdang-idev Can you please share a screenshot of the error you're getting?

It's just not responding then 5 sec later the app crashes. I tried to run expo  r -c many times but i does not work. One more thing I need to inform is: if you roll back to version 2.12.0 then it works normally.

@sankhadeeproy007 Yes I tried multiple times. It is crashing. Please try the minimal code I posted and see the crash yourself. Last working version for me is 2.12.1

Possibly related. I can't confirm the crash but I am getting a related issue on both android and iOS. The focus seems to stick to the first field. It will not release to the next field, hard to explain. On iOS you can force the cursor to move but I don't the state updates. On android the keyboard refocus on the first one. Also broke with an update to latest.
<Item floatingLabel error={error} success={success}>
        {i18n.t('user')}
        <Input
          keyboardType='email-address'
          textContentType="emailAddress"
          onChangeText={this.setValue}
          value={value}
          autoCapitalize='none'
        />
        {icon}
      </Item>
...
<Item floatingLabel error={error} success={success}>
        {i18n.t('password')}
        <Input
          secureTextEntry
          textContentType="password"
          onChangeText={this.setValue}
          value={value}
          autoCapitalize='none'
        />
        {icon}
      </Item>

So... the crash was caused by the code in native-base:
for (let i = 0; i < this.props.children.length; i + 1) {
I don't know what to say :) Btw thank you for the fix.


notify 0-Orchestrator when a hot-reloaded config is invalid

Whenever a config which is hot reloaded because of an update trigged by the 0-Orchestrator, results in an invalid config for the vdisk in question, we ignore that config for usage, as it would break our mounted vdisk.
While this is fine, we do also need to notify the 0-Orchestrator, so its aware of this failure and can fix it with a correction (if possible). Make sure to add to the failure the key of the config, and any other useful information, that might allow the 0-Orchestrator to auto-fix the issue or handle it in some way.
This issue can be resolved together with #300.

If I'm not wrong, you should be able to just add notification code at https://github.com/zero-os/0-Disk/blob/2224b3ac24e3382e82167d419e310ca8136ac0d8/config/source_etcd.go#L135-L138, as that function is (supposed to be) called from all locations in the config code where an invalid config prevented it from being read/updated.

Did another pass on the config code base, included some improvements, and I'm pretty sure that you can just add a single notification at the location I linked in my previous comment. Again be critical, and make sure so check the config codebase yourself to see if I didn't miss a situation in which the config couldn't be read or updated because of an invalid config key (due to the config not existing our not valid), feel free to fix that as well while you're working on this issue @chrisvdg.


Add File Batching to file_input operator

Description of Changes
Adds a max_concurrent_files field to file_input configuration. If the number of matching files exceeds this value, then files will be consumed in batches. Each poll interval will consume the next batch of files. nce each file has been read to the end, only then will the operator query for matching files again.
Please check that the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Add a changelog entry (for non-trivial bug fixes / features)
[ ] CI passes

Log Files
Logs / Second
CPU Avg (%)
CPU Avg Δ (%)
Memory Avg (MB)
Memory Avg Δ (MB)

1
1000
1.620741
-0.017223597
127.79633
+5.7013702

1
5000
5.810442
-0.50001574
136.53731
-0.67645264

1
10000
12.3794985
+0.39645672
143.74542
+6.564789

1
50000
58.983936
-1.8989296
178.05846
+1.7809753

1
100000
115.37855
-1.0358887
234.58917
+7.9026184

10
100
2.2414095
-0.24145341
130.19908
+1.4722443

10
500
6.8449
-0.2414546
139.42241
+7.5664062

10
1000
13.896886
-0.20710278
146.63551
+3.0374603

10
5000
66.320114
+0.87319946
187.50606
+4.180359

10
10000
125.80203
-1.4598999
236.59052
+11.452454

Log Files
Logs / Second
CPU Avg (%)
CPU Avg Δ (%)
Memory Avg (MB)
Memory Avg Δ (MB)

1
1000
1.8448653
+0.13793302
125.10601
+0.5554962

1
5000
5.7587256
-0.2586298
131.3878
-2.2233276

1
10000
12.086572
-0.20649719
137.61072
-1.1601562

1
50000
66.470665
+8.677155
178.13605
+5.9884186

1
100000
115.60671
-16.424797
228.84604
-626.7567

10
100
2.3449526
+0.15521407
128.72145
+0.018997192

10
500
6.7761383
-0.10345745
132.5854
+0.14575195

10
1000
13.207303
+0.17261505
141.81546
-1.8642273

10
5000
64.1144
-0.5424881
183.06183
-2.713501

10
10000
125.22474
-3.3818283
217.3257
-44.836075


Pin API-stub-generator version to stable version

@weshaggard Can you please review and approve this PR?


React native Range slider or query between two fields

My scenario is I would like to get results between two fields. I have experience_from and experience_to attributes. I want to get results between these two attributes. How would I implement a custom filter refinement to accomplish this.
In algolia we do something like below
res = index.search("query term", { "filters" => "experience_from >=1 AND experience_to <=3"})
I would like the above implementation in React native algolia instant search.

@ramachandranv, you should be able to use connectRange for this, feel free to open a new issue in https://github.com/algolia/react-instantsearch if you have more issues about this!


[Bug]: Remove Textarea bottom shadow when at bottom of messages

Is there an existing issue for the same bug?

[X] I have checked the troubleshooting document at https://opendevin.github.io/OpenDevin/modules/usage/troubleshooting
[X] I have checked the existing issues.

Describe the bug
There is a shadow effect at the bottom of the messages list, which is quite distracting/annoying when it is still visible at the bottom of the list. Just visually I'm then triggered to scroll down further! 🙄 Habits!

It's a tiny thing, but I can't unsee it each and every time. 😬
Current OpenDevin version
main

Installation and Configuration
WSL

Model and Agent
No response
Operating System
No response
Reproduction Steps
No response
Logs, Errors, Screenshots, and Additional Context
No response

Ok, I found what is responsible, I think, in frontend/src/components/chat/ChatInterface.tsx:
        {/* Fade between messages and input */}
        <div
          className={twMerge(
            "absolute bottom-0 left-0 right-0",
            curAgentState === AgentState.AWAITING_USER_INPUT ? "h-10" : "h-4",
            "bg-gradient-to-b from-transparent to-neutral-800",
          )}
        />

Did you change and confirm?


Don't work for public users, just for intranet

Describe the bug
The website can be loaded, but if you want to join, there is an interruption
Bug Report Checklist

[X] I have granted camera/microphone access permissions while testing
[x] This bug is reproducible on https://peercalls.com

To Reproduce
Steps to reproduce the behavior:

Go to https://peer.crashlog.ch
Click on "start session"
Click on "join call"
See error

Expected behavior
if I access from the local net via domain, then it works with several participants. As long as all participants are in the intranet it works well. If I come from outside, then it does not work.
Screenshots
not available.
Desktop (please complete the following information):

OS: macOS 10.15.3
Browser safari
Version: latest

Smartphone (please complete the following information):

Device: iPhone 11
OS: iOS 13.3.1
Browser stock browser, safari
Version: latest

Additional context
Server runs on port 3000 and is routed with HAProxy with SSL.
I'm not a such "pro", but I couldn't find the error on my side, so I suppose, the error must be on your side.

If you cannot reproduce this on peercalls.com, then it is not a bug. I assume you are testing this within a corporate network? If so, perhaps the corporate firewall is blocking access to devices on the internal network. Peercalls.com is configured to use a TURN server (coturn), which will relay traffic when a direct connection cannot be established. The default configuration uses only a public STUN server, which is not enough in some situations.

Ok. I'm behind a Firewall.
Can you please tell me which ports and protocols except tcp/udp on port 443 must be open?

AbortError: The operation was aborted.
Peer connection closed
A peer connection error occurred
Connecting to peer...
Connected users: 2
Connected users: 1
Ready for connections
Connected to server socket

From Wikipedia:

The standard listening port number for a STUN server is 3478 for UDP and TCP, and 5349 for TLS.

However, the default STUN server configured is stun.l.google.com:19302, since Google's STUN servers use 19302 instead of 3478.

I've to reopen this case.
Now peercalls doesn't work for me in any way.
Also not on your homepage peercalls.com
I've tried different scenarios.
UPC to UPC -> not working
Swisscom to UPC-> not working
Sunrise to Salt-> not working
It works if both users are behind the same Firewall, else it doesn't work

Sorry to hear that it doesn't work. I want to debug this. Can you join for a chat at Gitter: https://gitter.im/peer-calls/troubleshooting ?

I have get the same this issues.
I deployed on aws, and run coturn.

passed with: the same internet service provider
Failed if participants are come other internet service provider
please help me to explain/ resolve this issues !.
Thanks you!

Hi @robinpham,
Can you test your coturn server at: https://test.webrtc.org ?
You don't need to share your mic & camera there, I'm most interested in results of Network and Connectivity.
Make sure you enter your TURN settings here and leave the STUN server blank:

There are two ways of figuring out the configuration:
a) Open the call page on PeerCalls and run this in the developer console:
JSON.parse(document.getElementById('iceServers').value)

b) Running this code snippet from root of peer calls config directory.
node -e 'console.log(require("./lib/server/turn").processServers(require("./lib/server/config").config.iceServers))'

If peercalls is installed as as node module, replace ./lib with  ./node_modules/peer-calls/lib`.
When using docker, run:
docker exec -it CONTAINER_ID node -e 'console.log(require("./lib/server/turn").processServers(require("./lib/server/config).config.iceServers))'

replace CONTAINER_ID with the correct container ID.

Thanks you for  your help full.
As your solution, i tested, and failed.

I allowed port 3478 / udp, 1000: 32768 / udp.
Up to now, I have not successfully configured, failed tested: https: //test.webrtc.org.
Because I don't have much experience deploying on aws.
Currently, the application I have deployed only runs on the same  internet service provider.
I would like you to explain/ resolve this issues !.

Try also enabling 3478 tcp, and 5349 for tcp and udp, and rerun the tests (assuming you're using the defaults for coturn):
From https://github.com/coturn/coturn/wiki/turnserver:

-p, --listening-port  TURN listener port for UDP and TCP listeners (Default: 3478). Note: actually, TLS & DTLS sessions can connect to the "plain" TCP & UDP port(s), too - if allowed by configuration. For example, the default listening port can be set to 80 or 443, to go around some strict NATs. Remember that ports < 1024 may require superuser privileges to start the turnserver process.
--tls-listening-port  TURN listener port for TLS and DTLS listeners (Default: 5349). Note: actually, "plain" TCP & UDP sessions can connect to the TLS & DTLS port(s), too - if allowed by configuration. For example, the default tls listening port can be set to 443, to go around some strict NATs. Remember that ports < 1024 may require superuser privileges to start the turnserver process. For secure TCP connections, we currently support SSL version 3 and TLS versions 1.0, 1.1, 1.2. For secure UDP connections, we support DTLS version 1.

Thanks you!.
Based your solution, i reviewed my app coturn.
I detected an error when i start "turnserver"

Im trying to resolve.
I would like you to explain/ resolve this issues !.

Hi @robinpham, please contact the coturn team for support for coturn.

Why closed? Problem still exists...

Because I discusses this issue with OP in Gitter and the problem was resolved. Are you experiencing this issue on peercalls.com, or on a locally hosted instance?

Hey @jeremija I was reading through this issue and have run into similar problems in the past with some (only some) users:
Peer connection closed
A peer connection error occurred
Peer connection established
Connecting to peer...
Connected users: 2
Connected users: 1
Dialling...
Connected users: 1
Dialling...
Connected to server socket

This is what i see on my end, and I believe my peers see a similar thing on their end too.
But NB my PeerCalls instance (https://meet.mills.io) works for most folks I use it with just fine.
So my question is this:

What are the possible reasons this might fail to connect in general?
Failure from the other peer's side to contact Google's STUN servers? (I use the defaults)
Do I need a Coturn server at all? (this is something I've never understood)

I'd like to basically figure out how to make my instance work with some other folks that it doesn't work for. I have one friend that lives closeby where it consistently fails for him, so I can at least have a reproducer.

Hi @prologic, I see you're using peer calls in mesh (peer to peer) mode, perhaps you'll have better success running it in SFU mode and exposing the ephemeral UDP as well as TCP ports on your server.
For example:
network:
  type: sfu
  sfu:
    interfaces:
    - eth0
    protocols:
    - udp4
    - udp6
    - tcp4
    - tcp6
    tcp_listen_port: 8443
    udp:
      port_min: 9000
      port_max: 9010

Then make sure incoming connections to TCP port 8443 and UDP port range 9000-9010 are allowed on your server.
This should cover 99% of the connection issues, but the traffic will be routed through the server rather than peer to peer.
Unfortunately different ISPs across the world have different rules and sometimes network firewalls or routers don't allow peer to peer connections. The coturn server helps in those cases. The coturn server can be configured with both mesh and SFU network types, but a SFU is better suited for conferences with more than two people because each user uploads their video only once. Also, since the SFU server is already publicly exposed, users have a better chance of connecting successfully, even without the coturn server.
Hope this helps!

I see Thank you for this explanation @jeremija 🙇♂ This helps clear things up a lot! It's a shame I can't run the instance in both modes and have a way to pick one over the other or even fallback if p2p doesn't work? I might be just smoking crack though 😅

In any case I'll have a go at reconfiguring my instance to use SFU mode for better experience for folks I video conf with around the globe 👌

@jeremija One final question on this (hopefully it also serves as good documentation, maybe we can put this in a Wiki somewhere? -- Q: Is an alternative solution that can cater for both p2p/mech and sfu is to configure PeerCalls to also use a TURN server such as coturn?
Basically what I've found in practise is setting my PeerCalls instance to use SFU networking also exhibits some problems with some people and has this really nasty side-effect I don't really like of centralising all the traffic to one point, increasing the CPU, Memory and Network I/O of my instance unnecessarily (I have graphs to show this!)

That's a little tricky. Some services enable P2P when there's only two users and then switch to SFU as soon as the third user joins, but it would be tricky to handle this gracefully, at least with the current state.
One thing that we can do to enable a third mode that would enable the user to pick SFU or P2P when they are creating a room. I realize it's not an ideal solution, but could help alleviate some of the issues.

Is an alternative solution that can cater for both p2p/mech and sfu is to configure PeerCalls to also use a TURN server such as coturn?

Yeah the coturn server can relay traffic and it allows those users that can't use the SFU directly to get around their NATs. But all the traffic is then relayed through that TURN server.

But if you had your instance configured in mesh networking with a coturn server some connected peers would go via the coturn SFU while others that can p2p would   directly rite between themselves?

You can have a TURN server configured with SFU (coturn is not an SFU) or P2P, but the Peer Calls server itself cannot combine peers connected via P2P and SFU, it's all or nothing because of the way connections are initialized.


Parse TwineScript in Attribute Directives

SugarCube has Attribute Directives that can act as TwineScript expressions. The extension needs to parse them as such to capture variable references.

Fixed in 569ded32a5841c5b37812228a316a3826d2abc26.


Import uuid.v4 member

Another attempt at https://github.com/airbnb/enzyme/issues/1020

This works.

This doesn't achieve the desired effect, which is a smaller bundle size. Relying on treeshaking doesn't count :-)

Ha of course : ) Our concern was it just working at all because this is a devDependency for us so size was not critical.

Thanks for the help debugging @ljharb : )

We ended up now using<EMAIL_ADDRESS>and npm install<EMAIL_ADDRESS>--save-dev in the depending projects to fix this for now and posted about the issue in the uuid project.


Remove duplicate value in version details on application list page

What this PR does / why we need it:
Before

After

Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice!
/lgtm

Nice.
/approve


I want to know  import pattern to use yorkie-js-sdk

Description:
I want to know  import pattern to use yorkie-js-sdk.
There is no separate text for the syntax below in the manual.
import yorkie from "yorkie-js-sdk"

Thank you for reporting. It would be nice to explain it after npm install.
https://yorkie.dev/docs/getting-started/with-js-sdk#installation
Even in the case of <script>, it may be necessary to explain that yorkie is created globally.


policy_out dimension

Tell me, how the dimension size was determined in the Dense output layer (None, 1968) in the policy_out header?

That's the number of combinations of (starting square, ending square) for legal chess moves (in a rank, file, diagonal, or knight), plus some extra indicators for promotions. See the original DeepMind paper for the detailed explanation.


Update Composer dependencies (2020-07-09-00-14)

Visual regression test passed!
View the visual regression test report


Log curl calls on debug

Fix #280
No diff in the tests since we only use dry-mode to not submit pull requests and so on.

Sorry about the double review, github kind of failed me here!


test

test

test

recheck

test

test

recheck

I have read Develocity Terms of Service and I hereby accept the Terms


Docker integration using Github CI Actions

Docker integration using Github CI Actions

CI/CD for docker integration completed and working.
Docker image built, and pushed successfully to dockerhub as evidenced by the checks on this PR.
Latest image in Dockerhub repository https://hub.docker.com/r/initd1/hackalert:

❕  Future:
Potentially change the tag of the image being pushed to follow Semantic Versioning standard

I think changing it to something like CI or Docker Integration might be a better name. thoughts?

Do you mean changing the name of the Github Action to Docker integration? or the docker image tag?

I was referring to the name of the GitHub action, as I thought that it might be more comprehensive.

I agree..I will change it to Docker integration


[WIP] building the header

cdf3796f26d918977b339e822963c03363a9d084 fixes the bug on validating Pages (which was blocking your rake db:drop db:migrate db:seed.


ui: limit shown chart versions to latest 5

adds a link to show all versions

Codecov Report

Merging #312 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #312   +/-   ##
=======================================
  Coverage   88.96%   88.96%
=======================================
  Files          15       15
  Lines         616      616
=======================================
  Hits          548      548
  Misses         45       45
  Partials       23       23

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 d74f065...dc24b5b. Read the comment docs.


"Next Question" box appears after completing Part A

"Next Question" box appears after completing Part A, B, C etc, instead of just when all parts have been completed.
Happened for this question:
https://staging.isaacphysics.org/questions/basic_flux_linkage?board=ff86c1ec-707c-4824-90c7-06cc04186a4b

I can't reproduce this. Is it still happening?

It's not happening any more; we seem to have disabled the question (but not board) completion pop up.


Update create-self-hosted-integration-runtime.md

Learn Build status updates of commit 0ff5511:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/data-factory/create-self-hosted-integration-runtime.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@lrtoyou1223
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I merged this change now.  Thank you for helping us improve the docs! #please-close


ndgrid

I couldn't write directly, so I forked to make this PR.

Doh! I just realized I hadn't accepted the invite.
I will go ahead and merge this PR so that there is the bare bones there an dev can work.
But feel free to change any of it.

👍 I'd suggest copy-and-pasting .github/workflows from LazyArrays.jl to get the basic CI support


Calcite Maps App lab improvements

Removed reference to beta
Removed QA for JS API 4.0 (it is released now)

Changed 4.x to 4.0

Hey guys, just hold off please. I'm re-writing the whole lab now. We are also moving it.
@nixta


Fix oracle select for update of output and PG SET issue

Changes:

PG: If there is no "FROM" clause, avoid adding DUAL table as PG does not have it
Fix Oracle, PG SELECT FOR UPDATE OF output: should be SELECT ... FOR UPDATE OF col1, col2; output from  SELECT ... FOR UPDATE OF(col1, col2);
Restructure visit(SQLSetStatement x) for PG
For SQLSetStatement output in PG, use SET only if the original query is using SET, related to issue #5663

"SELECT rbi.rma_id, rbi.last_update_time\n" +
                        "FROM ktv_epm.ktv_eq_rprboarditem rbi\n" +
                        "JOIN ktv_epm.ktv_eq_rma_header rma ON rma.rma_id = rbi.rma_id \n" +
                        "\tJOIN ktv_epm.ktv_eq_rprsenditem rsi ON rsi.e_rprboardid = rbi.e_rprboardid\n" +
                        "AND rsi.e_boardid = rbi.e_boardid \n" +
                        "WHERE rbi.rma_id > ?\n" +
                        "\tAND rsi.e_rprsendid = ?\n" +
                        "\tAND rsi.e_boardid IN (?)\n" +
                        "FOR UPDATE rbi.rma_id\n";

Druid wrongly parses this kind of query to 2 statements, the last one is rbi.rma_id, the rest is the 1st one, that's why some tests are failing...

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

Comparison is base (7c5ca9b) 58.75% compared to head (1fa1d04) 58.74%.
Report is 23 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #5725      +/-   ##
============================================
- Coverage     58.75%   58.74%   -0.02%
- Complexity    27796    27801       +5
============================================
  Files          1333     1333
  Lines        127689   127719      +30
  Branches      24061    24065       +4
============================================
+ Hits          75024    75028       +4
- Misses        43332    43352      +20
- Partials       9333     9339       +6

Files
Coverage Δ

...ibaba/druid/sql/ast/statement/SQLSetStatement.java
68.25% <100.00%> (+1.58%)
:arrow_up:

...l/dialect/oracle/parser/OracleStatementParser.java
79.70% <100.00%> (-0.19%)
:arrow_down:

...ql/dialect/oracle/visitor/OracleOutputVisitor.java
71.31% <100.00%> (-0.13%)
:arrow_down:

...ialect/postgresql/parser/PGSQLStatementParser.java
71.14% <100.00%> (+0.04%)
:arrow_up:

...ql/dialect/postgresql/visitor/PGOutputVisitor.java
29.59% <70.83%> (+0.02%)
:arrow_up:

... and 18 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


reformate Y axis returned infinity value with null value

When I try to format  Y axis label value it returns infinity with zero Value
My js code
                   yaxis: {                         show: true,                         labels: {                             formatter: function (val, index) {                                 return Math.abs(Number(val)) >= 1.0e+9                                     ? Math.abs(Number(val)) / 1.0e+9 + "B"                                     : Math.abs(Number(val)) >= 1.0e+6                                         ? Math.abs(Number(val)) / 1.0e+6 + "M"                                         : Math.abs(Number(val)) >= 1.0e+3                                             ? Math.abs(Number(val)) / 1.0e+3 + "K"                                             : Math.abs(Number(val)) < 1.0e+3 ? Math.abs(Number(val))                                                 : Math.abs(Number(val));}                         },
it worked with data but when all data equal zero chart be like this img

before add formatter function all Chart appear like this

I am also facing similar issue.
I have kept y axis max value as 100. And when data is not available, y axis  labels shows infinity

I am also facing similar issue.
I have kept y axis max value as 100. And when data is not available, y axis labels shows infinity

I Don't prefer to set max value because may be value be more than 100

@manonaSamir it is part of my requirement. Y value range cannot exceed 100

@manonaSamir it is part of my requirement. Y value range cannot exceed 100

So put min value your problem will solve


cephadm_adopt: fetch and write ceph minimal config

This commit makes the playbook fetch the minimal current ceph
configuration and write it later on monitoring nodes so cephadm can
proceed with the adoption.
When a monitoring stack was deployed on a dedicated node, it means no
ceph.conf file was written, cephadm requires a ceph.conf in order
to adopt the daemon present on the node.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1939887
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

jenkins test centos-container-cephadm_adopt

jenkins test centos-container-cephadm_adopt

jenkins test centos-container-cephadm_adopt


Please merge r=44876 into 1.9.x stable

E.g.: This fixes issue #23084, which causes transformers to fail to load under some circumstances. This is particularly important for getting the new test runner working with projects using transformers, including projects using Polymer and Angular.
Please merge r44876.

This issue has been moved to dart-lang/pub#6.


int.class and Integer.TYPE in txn.queryIndex() statements (etc)

java.lang.IllegalArgumentException: invalid value type int for index query on field `id' in class de.protubero.ajs.Person: should be a super-type of java.lang.Integer

I was trying to work through some issues and I flipped Integer.class to Integer.TYPE.
txn.queryIndex(Person.class, "id", Integer.TYPE);
// vs
txn.queryIndex(Person.class, "id", Integer.class);

Exception messages could be more helpful (add 'do not use .TYPE' words of advice etc), or the wiki documentation could contain the same information in the broad area of primitives, boxing etc.

Updated wiki page in 89e590d.


Infinite loop in Elasticsearch consumer when user deletes an asset

When a user deletes an asset, the Elasticsearch consumer will forever try to delete that asset over and over. It repeatedly logs statements like these:
  Attempting the following action: REMOVE
  Operator type not present in payload, this must be a request to delete the entire asset
  Decoded payload for asset: c91f9926-668b-49af-b977-1f75dcc19a9b

I'm not sure when exactly this was resolved, but it has been resolved.


Catch all pull exceptions

pull_data is failing due to regular API oddities in the middle of the night, this is an error catch-all so that it doesn't terminate. Not expecting this to have any adverse effects on data consistency or day-to-day trip turnover.

So it just retries every minute until it works again? Should we have a limit for when it stop it if it's failed x times? Also, I have been meaning to ask..what is the 'once' param for? I don't see it being set to true ever (am I missing something?)

There should probably be a limit—I agree and thought about this too a bit. What exactly the logic should be behind that is perhaps "coming soon" since it's unclear to me exactly how long these glitches last. If they're short-lived then we don't really need to back off, otherwise we should. In theory would merge this in and then do some log file analysis to investigate.
Re: the --once argument, it is set by the command line input with pull_data.py:34. Allows you to pass --once in the CLI so you can get one set of data getting in the db, and no more! 👍

Ok! I understand. I think fine to merge for now then and then make an issue to figure out the limit.


Enhancement

Change the names of the shock protocols from "shock scenario 1" to something meaningful such as "shock in dark" or "bright shock"

This issue has been resolved. The two shock scenarios are called Bright surface shock and Dark surface shock.


Some punctuation marks are HTML-encoded when posted to Bluesky

Version: Release 1.2.0
Posts to Mastodon that have certain punctuation characters appear on Bluesky HTML-encoded:

Test post to Mastodon:
Testing some punctuation , . " ? < > ' !
Appearance when it is cross-posted to Bsky:
Testing some punctuation , . " ? < > ' !

yep, noticed this on my posts as well, this combined with the lack of line breaks/spaces makes some posts very difficult to read.


Extend test coverage for aws-secrets-manager

Follows https://github.com/apache/camel-quarkus/issues/3741.
Refactor (to use LocalStack) does not cover all usecases from the documentation.
There are 2 items, which are not covered:

SING AWS SECRETS MANAGER PROPERTY FUNCTION
operation replicateSecretToRegions

Fix back in https://github.com/apache/camel-quarkus/commit/cf21d5a0ba1204f44253f3d6fcf5352b27c8f9a9.


⚠ ClamAV has degraded performance

In 573cfed, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 93 ms

Resolved: ClamAV performance has improved in 3732484.


Null pointer error in method changePoll. socket->nodeData->async == NULL

This bug reproduce when send binary data with high frequency and socket is busy. message add to queue and start poll for send. And when changePoll i have error.
void changePoll(Socket *socket) {
     if (!threadSafeChange(nodeData->loop, this, socket->getPoll())) {
         if (socket->nodeData->tid != pthread_self()) {
             socket->nodeData->asyncMutex->lock();
             socket->nodeData->changePollQueue.push_back(socket);
             socket->nodeData->asyncMutex->unlock();
// async is null, i am fix this call socket->nodeData->addAsync();
             socket->nodeData->async->send();
         } else {
             change(socket->nodeData->loop, socket, socket->getPoll());
         }
     }
 }
// example send binary data
webSocekt->send(message, length, uWS::BINARY);

This is not a bug -> you need to addAsync if you are doing threading. Yes it is a terrible behavior but it is as designed (this has been written about before).


How to train on multiple GPU (w/ small vRAM)

Thanks for the great work.
Do you have any recommended solutions for running finetune.py code on multiple small memory GPUs (such like finetune lamma-30b on 8xV100)?
Is it possible for pipeline parallelism in deepspeed to be easily integrated into current code?

Have you tried using torchrun?

I tried torchrun, but the DDP can not reduce the vRAM.

I am interested in this as well, bumping for visibility


Styles insertion order with server-side rendering

I realise this issue has been mentioned previously but, I wanted to understand something.
I recently had an issue with the insertion order of styles, the base MUI styles were being inserted after the TSS styles and so, took precedence over the TSS styles. This caused issues where I was setting the display to be flex but the MUI Container styles default to block and so rendered in a top-down approach.

As you can see from the screenshot, the MUI Container styles are taking precedence. I tried one of the solutions recommended on another similar issue (to use &&) and that worked. But I then realised that the insertion order was possibly off due to the TSS emotion server coming before the MUI emotion server in the emotionServers array (from the docs):
const createMUICache = () =>
  createCache({
    key: "mui",
    prepend: true,
  });

....

const muiCache = createMUICache();
// sets TSS server first and then the MUI server after, meaning the MUI styles would take precedence?
const emotionServers = [getTssDefaultEmotionCache({ doReset: true }), muiCache].map(createEmotionServer);

.... // renders app

const css = emotionServers
    .map(({ constructStyleTagsFromChunks, extractCriticalToChunks }) => {
      const chunks = extractCriticalToChunks(html);
      return constructStyleTagsFromChunks(chunks);
    })
    .join(""); // creates the styles but the TSS ones first and then the MUI ones

Noticing this, I tried it again by swapping around the two servers like below, rather than using the && approach:
const emotionServers = [muiCache, getTssDefaultEmotionCache({ doReset: true })].map(createEmotionServer);

and that then fixes the insertion order (see the image below).

So, my question is just, is there a particular reason as to why the MUI emotion server should come after the TSS one, or is it fine to have the TSS one last? If so, should the docs be updated for this?
Let me know if you would like any more information, this is the only issue I have come across with TSS and I think it is an awesome package and nice to see there is still support out there for JSS approaches with MUI! :)

Hi @dvprrsh,
Sorry you had to go through this.
This is my bad, the mui caches should absolutely comes first!
Thank for reporting the error, I updated the documentation.

@garronej no problem at all, thanks for the quick reply :)


Add sj

Thanks for the PR, but does sj project is still active?

Whether they still work on this? Doesn't look like it.
And I think there are more succesful things out there.
However in my opinion it's still interesting to look at the code for some people. Or to see what projects didn't succeed/were canceled. Or to revive them if one is inclined to do so.
Actually I can see many reasons why I personally would include it.

Fair enough, thank you for the merge!


Re-enabling Windows tests

Fix tests around Windows' handling (or not handling) of : and @ in filesystem paths.
To address this, I did a few things.

If a URI has an authority (that is a username or a port in the format username@host:port), then we URL encode that to get a "safe" path.

Make sure that we are passing hasAuthority = true for all of our URIs that use https.  This may be a behavior that we want to change in gz-common eventually, but is sufficient for here.

Fixes UniqueName to generate valid paths for both ModelIdentifier and WorldIdentifier.  I also choose to make UniqueName always return unix-delimited paths, which cleans up some windows-specific test logic.

I believe that this resolves the test failures in FuelClientTest.CachedWorld and FuelClientTest.PatchModelFail as described in #373.
When forward ported to main, this should resolve #231
This (finally) resolves #106

Additionally, with bumping gz-common, it may be that we can just fix it there and most of this goes away in a forward port to main?

Okay, I've added to the list of things to consider working on for Ionic. I think this is good to merge!


Pausing VelocityTransitionGroup enter / leave animation

Hello!  I'm a bit new to the library, but am enjoying it a lot thus far.
Is it possible to "pause" an animation during the leave or enter animation phase when using VelocityTransitionGroup?  I was working through examples/scrolling-group.jsx demo that was included and was attempting to figure out how to pause the animation.  Any insight or direction would be appreciated on how to approach this.

What effect are you trying to achieve?

I have an automatic scrolling slide show of images.  It's currently modeled after the demo scrolling-group.jsx because the behavior is nearly identical with some minor tweaks.
However, I would like the end user to be able to pause / play the slide show.  I can "pause" the adding of additional items once the animation finishes entering/leaving, but actually pausing during the animation is proving a challenge.

That would be tricky. Calling Velocity.stop on the animated elements would stop them from animating. But, it would also cause the TransitionGroup to keep them around because stop keeps the complete callback from firing, which is how VelocityTransitionGroup can tell TransitionGroup to finally remove the element.
You might have better luck wrapping each image in a <VelocityComponent>, which can have its animation stopped (by setting its animation prop to null).


Cannot download APK-Files with DDG - it will be removed after download

Describe the bug
Downloading any APK-File - once completed, the file will be deleted automatically
How to Reproduce
https://github.com/user-attachments/assets/bc7a63be-2ac5-4e71-8290-e0086c0bd724
Expected behavior
APK-File should stay in Download-Folder
Environment
- DDG App Version:5.220.0
- Device: Samsung S23 Ultra
- OS: Android 14

Hi @kangaroo72! Thanks for opening the issue. I might have misunderstood the description, but the .apk file appears to be downloaded and is visible in the Downloads folder in your video. Could you clarify the bug you're reporting and steps to reproduce it?

Hi @nalcalag. At the end of the video, you will see that the file is disappearing without any  command from my side. It will be deleted automatically. When I start the download with Chrome, the file remains in the folder....
Hope this helps.
As mentioned, any APK-File is affected.
Best regards

@kangaroo72 A couple of questions:
1/ Is the name and location of the downloaded file the same when you download the APK using Chrome compared to when using DuckDuckGo?
2/ Have you enabled Samsung Storage Booster and defined any rules for storage cleanup?


Wrong VGMdb image types

MB cover art types are set from VGMdb image comments.
But it wrongly sets back each time it sees back, it should not:

Booklet back, obi back.

In fact I saw also several other issues, and editors don't necessarily notice and change types.
I'm not sure that settings types automatically is good at all... 🤔
https://musicbrainz.org/search/edits?auto_edit_filter=&order=desc&negation=0&combinator=and&conditions.0.field=type&conditions.0.operator=%3D&conditions.0.args=314&conditions.1.field=edit_note_author&conditions.1.operator=%3D&conditions.1.name=jesus2099&conditions.1.args.0=285909&conditions.2.field=editor&conditions.2.operator=!%3D&conditions.2.name=jesus2099&conditions.2.args.0=285909&conditions.3.field=open_time&conditions.3.operator=BETWEEN&conditions.3.args.0=2023-05-01&conditions.3.args.1=2023-05-20

I'm not charging the script, it is normal that editors should have a look at what they are about to submit.


No message when user has no right to delete attachment

Describe the Bug
A user complained that he edited a page and wanted to delete an attachment. He clicked on the delete button for the attachment, confirm it but nothing happen. It turned out that the permission setting in the group was "own". Changing to "all" and he could delete the attachment in question. I guess the attachment was uploaded by another person.
Steps to Reproduce
Check that the group setting for attachment is "onw".
Edit a page as user A and upload an attachment
Save it
Edit the page as user B and try to delete the attachment. You can confirm the deletion but the attachment is still there.
Expected Behaviour
Bookstack should provide a message why that attachment can't be deleted.
Screenshots or Additional Context
No response
Browser Details
No response
Exact BookStack Version
v24.10.2

Thanks @JtheBAB for reporting.
Ultimately, the user shouldn't be provided the option in the first place if they don't have the permissions to perform the action.
I've assigned to be addressed for the next feature release.

This has now been addressed in fcf0bf79a98c9b9b545d36ef728ee9cbe054048e, so that controls are not shown if the user does not have permission to use them. This will be part of the next feature release.
Thanks again for reporting @JtheBAB.


CSV export: in file name hh is in 12 hour

Hi!
I got confused with csv exports because I took them at "2242" but file name read "1042". Maybe change to 24h?

I can confirm Version 1.5.2 has this problem

QA passed

No way! Wait for 1.5.5 to test

So this is default filename when exporting not inside logs?

Yes, it's about csv filename

Filename now has 24 hours. QA passed.


RDD tests round 2

Adding tests for the not implemented methods.

Coverage increased (+9.2%) to 78.417% when pulling 209ff659adfc3b49aa5437cda318babfd52bc415 on htssouza:rdd-tests-2 into 9fb2320f85bc3813e0722e556e29fa837152323a on wdm0006:master.

Coverage increased (+9.2%) to 78.417% when pulling 209ff659adfc3b49aa5437cda318babfd52bc415 on htssouza:rdd-tests-2 into 9fb2320f85bc3813e0722e556e29fa837152323a on wdm0006:master.


[WIP] Added GetUsage for filter by account ID, principal ID, start date and limit

Proposed changes

Added GetUsage for filter by account ID, principal ID, start date and limit
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (changes to code, which do not change application behavior)

Checklist

[x] I have filled out this PR template
[x] I have read the CONTRIBUTING doc
[x] I have added automated tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (README.md, inline comments, etc.)
[x] I have updated the CHANGELOG.md under a ## next release, with a short summary of my changes

Relevant Links

Further comments

run dce#destory

/azp run dce#destroy


[add][webmail] next prev action

Coverage increased (+0.005%) to 83.995% when pulling 69981864ed8551a4accfa78eb3e967905929780b on pr-webmail-prev-next into 8417c83337d054605477dce4d49f5f307bb3edef on master.


[Crash]: Crash after updating to 5.4.4

What happened?
I got the following crash after updating Linkart to 5.4.4:
Logs
https://gist.github.com/vico93/12a7ddf48eeede31e691b9102bbbaad9
Mod Version
5.4.4
Minecraft Version
1.20.4

Update RBIP.


Generate Testnet SegWit Addresses

I want to support SegWit, but I cannot use them in testnet using the ones generated by coinb.in. In testnet, I generate a bech32 address, but it starts with bc1 (production moede), not tb1.
How can I generate a testnet SegWit address?

You can add this at line 1924 of coinbin.js:
    else if (coinjs.pub == 0x6f){        // BITCOIN TESTNET
        explorer_addr = "https://chain.so/address/BTCTEST/";
        coinjs.bech32.hrp = "tb";
    }

bech32 is not supported very well in coinbin right now.  There is a bug generating new bech32 addresses where the redeem script does not generate correctly (missing 0014 at the beginning) so the address generated will be wrong.
Line 193 of coin.js:
/* create a new segwit bech32 encoded address */
coinjs.bech32Address = function(pubkey){
    var program = [0x00,0x14].concat(ripemd160(Crypto.SHA256(Crypto.util.hexToBytes(pubkey), {asBytes: true}), {asBytes: true}));

But fixing the redeem script causes transaction signing to break.  Needs some work.

That sucks, I rely on the transaction signing to broadcast my txs. SegWit has lower Tc_fee's, I wish coinb.in supported it.

@OutCast3k is it possible for you to fix this issue?
right now the Bitcoin testnet doesn´t support correct bech32 addresses, only bc1 addresses when it should be tp1.


Fix autogeneration of DecomposableK.hs

Commit be05072 fixed the llvm build by splitting the big list in DecomposableK.hs into two smaller lists. However, this file is an autogenerated file, generated from the unicode database using the unicode-data/UCD2Haskell.hs script, therefore the fix should be made in the function called here in this script instead and the file should be generated by that script.
cc @brent80 in case you got time to fix this.

https://github.com/harendra-kumar/unicode-transforms/pull/14

Fixed via e5de0f5fb65fa2608fa2f01c074a79b3417310a2. Thanks @brent80 !


This package is compatible for Angular 4

Hi,
I am trying to use these package in angular 4 web application.
I got the error message like below,
Uncaught TypeError: Object(...) is not a function
    at Object.../../../../../src/app/app.config.ts (app.config.ts:39)
    at __webpack_require__ (bootstrap 271169117e7f18058034:54)
    at Object.../../../../../src/app/app.service.ts (app.service.ts:7)
    at __webpack_require__ (bootstrap 271169117e7f18058034:54)
    at Object.../../../../../src/app/app.service.ts (country.ts:22)
    at __webpack_require__ (bootstrap 271169117e7f18058034:54)
    at Object.../../../../../src/app/app.service.ts (app.service.ts:7)
    at __webpack_require__ (bootstrap 271169117e7f18058034:54)
    at Object.../../../../../src/app/app.service.ts (app.service.ts:49)
    at __webpack_require__

When I am trying to add the module in app.module.ts, there is no module available in these package.
Thanks for your help.
Regards,
Sudhakar.

Hi Sudhkar,
I think you might need a polyfill in your app. Since you're using TypeScript, do you use tslib? I think it polyfills some things on Object. I haven't tested with angular but there is not anything specific to any framework in this code.

@cchamberlain
Thanks.


need a way to have visibility into the amount of resources that the AS is consuming

This is re-filed version of #24020 - some comments from that in-line:
Related to dart-atom/dartlang#189. We have situations where the AS process consumes large amounts of RAM, and/or persistent high CPU usage. We'd like a way to be aware of this on the client end. It may be difficult to get precise resource usage (RAM / CPU), but hopefully there are some suitable proxies. Number of contexts, size of each context, ... This could be exposed as a specific API that clients call, or perhaps as a periodic notification.
--
It would still be nice for clients to be able to opt in to a periodic heartbeat from the analysis server. The payload could be metrics that are easy for the AS to calculate and that would act as good proxies for the resource consumption of the AS. I would see this API as generally useful to clients. It would mean that the monitoring code could be written in one place instead of each client writing its own monitoring code.
--
From @lukechurch:
Let me try and help out here.
People develop a 'spider sense' for 'this is a well behaving tool', 'this isn't'. In the physical world we use all sorts of signals to build this 'feeling'. E.g. the vibration in a car steering wheel as you're driving it.
Digital things are on a whole polished glass boxes, which lack a lot of these signals. So people need other ones. When the system gets really reliable people can start to use its behaviour as a signal. E.g. we saw people using code completion stopping working as a signal that they'd got something wrong with their code.
On the whole our tooling isn't there yet. So when people get strange behaviour they don't know whether it's their code being broken or the system misbehaving. They will often assume the former and waste a lot of time. Anything we can do to add signals that people can start using to build a 'this is normal, this isn't' feeling will help a lot in the user knowing when to keep pushing on or when to repair.
So e.g. RAM usage. If the user builds an instinct that 'my project usually takes AS about 1Gb to run', the system starts behaving funny and they notice it's using 4Gb - then they might correctly short cut to the realisation that it was AS. And click the ''Something is dodgy, send a perf report and restart' button.
My instinct for what 3 useful metrics would be are:

RAM usage of AS
Average + Variance of latency Request/Response from Server over the past 30s
CPU load fraction in AS process

I think putting these three numbers somewhere continuously on the UI at low intensity might well help build the 'spider sense'.

@devoncarew how close would we get if we added a Status request that supplied the kinds of stats that we render on the HTML status page?  Given that you could tease out some proxies to surface.

First cut plumbed through as of https://github.com/dart-lang/sdk/commit/82e83bb34d913c1ba171d174e66e60b95bea492a.
Preliminary diagnostics payload looks like:
{
  contexts:  [
    {
      name: /project,
      explicitFileCount: 8,
      implicitFileCount: 0,
      workItemQueueLength: 0,
      cacheEntryExceptions: []
    }
  ]
}

More details in the review.
Note that this experimental API is in spec_input.html but anything flagged experimental will not have docs generated. :+1:

Closing this as much of the work has landed, and we have two open, more specific, issues for the remaining work.

Thanks!


:bug:Fix controller queue bug.

Summary
In controller process queueItem should be used in queue and not key string.
Related issue(s)
Fixes #

/lgtm

/approve


Fixing design doc about DataContractSerializer, and adding the starting code

@mconnew Wanted to double check that this is good as an update to the info on DataContractSerializer in the design doc. Is everything in there accurate?

@cablooshe Could you please rebase away all the code changes from this PR, and keep just the doc update?


Restructure "Create a root CA certificate"

Specifically, in the Create a Root Certificate and self-sign it section, the screenshot should be moved up as it pertains to the generation of the CSR, not the creation of the root certificate.

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

ID: 3b0e716d-54fc-0b48-e2aa-9f5b126072ae
Version Independent ID: cda1facc-d299-a154-e824-a1b695f812d8
Content: Generate self-signed certificate with a custom root CA - Azure Application Gateway
Content Source: articles/application-gateway/self-signed-certificates.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

Hello @simonkurtz-MSFT
I have assigned the issue to the content author for further actions.


Concurrency

Has Langflow considered the issue of multiple concurrent users? How many people can be supported to use it online simultaneously if I deploy it  by custom?

Hello!
The Multi-User functionality is coming up in a few weeks. We're going to be able to support multiple users in the same flow, and, futurely, even cursor tracking.
Thanks for the feedback!


Cannot install Bazel plugin in Clion IDE

setting->plugin->Browse repositories and search Bazel can only find "Bazel build formatter" and no "Bazel" package, how should I install it in another way?
Thanks for the help!

Probably because you're on a version of CLion for which there's no Bazel plugin (either very old, or 2018.2+).
You can see the supported versions here: https://plugins.jetbrains.com/plugin/9554-bazel

Closing as a dupe of https://github.com/bazelbuild/intellij/issues/320 on the assumption you're on CLion 2018.2

thanks a lot!

Hi,
Just found that my clion version is 2017.3. It should be in the range of executable list.
Any other idea? Thanks.

Perhaps you're on a 2017.3 version prior to 173.4301? If not, you might have to manually download a compatible plugin from JetBrains' plugin repo.


Parallel record for animation?

I have looked throughout the manual, and didn't find a way to use multiple thread to render the frames of an animation -- is this something that is possible at all?

It doesn't make sense to set one observable, let's say, to two different values in two different threads at the same time, which is what would happen in a parallel recording of the same Figure. However, if you have a very large amount of frames you need to render and a deterministic splittable collection of frames, then you could in principle run Makie in multiple processes at the same time, with one Figure in each, that might be a bit faster.

Right - this specific case is 100% deterministic, I update the observable within a loop, so I can split the frames as much as I want.
Is there a code example of using multiple threads?

Makie isn't necessarily threadsafe, but you could use the Distributed stdlib to create multiple figures, and render multiple videos (since it spins out multiple worker processes, which might run on different threads.  After that, it's just a matter of stitching the videos together - which you can do with FFMPEG or any other software.
Here's how I did it with Distributed, in concept:

N = 4 # number of processes
# everything outside @everywhere is in thread 1
using Distributed
addprocs()

@everywhere begin
using $(Packages...)
function do_animation_for_values(values, filename)
...
fig = Figure()
# create the figure and modify it
...

record(fig, filename, values) do i
    myobservable[] = i
end
return filename
end # end function
end # end @everywhere

using TiledIteration # (nice for this sort of thing)

indices = 1:100 # (this can be whatever you want)

# N-0.5 assigns less work to the master process which is responsible for scheduling
partitions = collect(SplitAxis(axes(indices, 1), N - 0.5)

futures = [@spawn do_animation_for_values(partitioned_indices, "myanimation_fragment_$i.mp4") for (i, partitioned_indices) in enumerate(partitions)]

filenames = fetch.(futures) # wait until the code executes

# use FFMPEG to stitch filenames together.  Access the executable by:
Makie.FFMPEG.ffmpeg() do ffmpeg
    run(`$ffmpeg ...`)
end


Pasting an URL to 'Original URL' leads to string duplication

Feature
Create / Edit a snapp
Environment
Docker container with image uraniadev/snapp:0.8.1
Utilizing the default database configuration (sqlite)
Steps to reproduce

Log in with a valid user
Navigate to 'DASHBOARD'
Click '+ Shorten an URL'
Paste the valid URL https://example.com to 'Original URL' via:

keyboard shortcut: Ctrl + V
browser context menu: Right click -> Paste

Expected Result
The value of 'Original URL' is https://example.com
Actual Result
The value of 'Original URL' is https://example.comhttps://example.com
Side note
Pasting the URL via the clipboard button in the input field works as expected
Assumption
Pasting with keyboard or context menu additionally triggers the clipboard function,
which leads to a string duplication.

i can fix this and add it to 0.8.2

should be fixed 👍


What do you think about making flatpickr peer dep?

What do you think about making flatpickr peer dep?

I don't think it's necessary, semantic versioning should be enough, npm upgrade should always give you the latest one since we don't provide a maximum version (until 5.0 is released, if ever). The only reason to would be if people wanted to use older versions of it, which I don't see the reason for.

If you want to lock the dependency you can add it to your own dependencies and that's the one that your package manager should give you, or you can use a package manager with specific resolutions like yarn.


docs: promote GitHub discussions

v14.12.1 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


[Docs] Add Intro and acknowledgement to DG

Also fixed spelling and grammar mistakes, and formatting errors.

Removed undo/redo command

Codecov Report
Base: 77.21% // Head: 77.21% // No change to project coverage :thumbsup:

Coverage data is based on head (0c26602) compared to base (8423b9a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #164   +/-   ##
=========================================
  Coverage     77.21%   77.21%
  Complexity      786      786
=========================================
  Files           108      108
  Lines          2199     2199
  Branches        249      249
=========================================
  Hits           1698     1698
  Misses          410      410
  Partials         91       91

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

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


 I don't know why, nothing was changed by me:

I had that the other day. Got corrected automatically when building (maybe a fresh build).

This fixed itself... weird but I'll take it


Error with spec

When installing the resulting RPM's:
chown: invalid spec: `clickhouse:'
warning: %post(clickhouse-compressor-1.1.54198-1.el6.x86_64) scriptlet failed, exit status 1

The problem on centos 6 is with this line in the .spec:
chown -R clickhouse: /etc/compressor

Is the problem still there? I corrected the issue with commit 11 days ago and updated packages in YUM repository?

I've created a PR #7 for this.


Skip linting in cosmos emulator tests

On suggestion of @alzimmermsft, this PR ports the common test options we run in the base live test template into the cosmos emulator test template, which should shave 5-10 minutes. These steps were overridden, but it's not clear if the intent was to enable linting or to do something else (like clean verify) or if the steps just got out of sync.

/azp run java - cosmos


update-alternatives fails in udev_setup.bash when alternative is already set?

On WSL2 if running udev_setup.bash after update-alternatives has been set for usbip, the line
update-alternatives --install /usr/local/bin/usbip usbip /usr/lib/linux-tools/*-generic/usbip 20 fails with
update-alternatives: priority must be an integer
This causes the rest of the setup script to not be run. This error also occurs if the line is ran from the command line. I am speculating its because the priority is already set? 20 is clearly an integer.
Commenting out the line lets the rest of the script run.

Oh man okay. Uh I just ran this on my WSL2 setup so maybe we can diff errors or something

I had followed the WSL2 instructions manually last year but wanted to try to new script so that's why I am guessing it's because it is already set