Observable.from('string') browser compatibility

Observable.from can take string and iterate it as
Rx.Observable.from('foo').subscribe(console.log);
// => Next: f
// => Next: o
// => Next: o
// => Completed

but fails to do it on some browsers does not support string iterator (IE, for example)
in RxJS4, seems it has own iterator for string object (https://github.com/Reactive-Extensions/RxJS/blob/master/src/core/linq/observable/from.js#L3-L23) to support this behavior - does RxJS5 need to port same behavior?
I've used  https://jsfiddle.net/v0w220v6/ to test out between different browsers.

related to https://github.com/ReactiveX/RxJS/issues/998 .

I believe some of the infrastructure for this is already in place. So there might be a bug.

I assume it's StringIteratror in IteratorObservable, will try to look into what's happening. Marking it as bug meanwhile.


python 2.7 and openblas incompatible on windows

Issue:
I'm getting this error:
conda.exceptions.UnsatisfiableError: The following specifications were found to be in conflict:
  - openblas=0.3.3
  - python=2.7
Use "conda search <package> --info" to see the dependencies for each package.

on a feedstock build here:
https://github.com/conda-forge/slycot-feedstock/pull/18
Also if if try conda create -n test python=2.7 openblas on a windows computer it fails to resolve the dependencies with an incompatibility.
Is openblas + py27 supported?

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Is openblas + py27 supported?

No. See https://conda-forge.org/docs/maintainer/knowledge_base.html#blas on how to link against blas.


Add API calls for bytecode dump/load

Add API calls to dump a function into bytecode (a buffer value) and to load a function from bytecode. The bytecode format should be platform neutral (but can be version specific) so that cross compilation works as expected.
Tasks:

[x] Implement API calls
[x] Feature option for compilation (footprint)
[x] API testcase
[x] Internal documentation for bytecode format, including rationale
[x] API documentation
[x] Website changes, mention bytecode dump/load support and basic features/limitations
[x] Bytecode dumper script for diagnosing bytecode issues
[x] Command line tool to compile a .js file into bytecode, perhaps add to duk
[x] Command line tool to execute a bytecode file, perhaps add to duk
[x] Cleanup branch merge (resolve FIXMEs)
[x] Resolve unused variable warnings
[x] Resolve testcase FIXMEs
[ ] Fix DUK_ALWAYS_INLINE issues with pre-C99 compilers, maybe enable DUK_ALWAYS_INLINE only by default with C99/C++11 or above (this is now in duk_config.h so user can tweak it anyway)

Original issue #27.

What are you doing about the problem mentioned in the original issue of verification?  A header specifying the Duktape version?
Actually a better solution might be to not use the Duktape version directly, but a bytecode revision number.  This way if the format doesn't change between releases existing bytecode files won't break.

Current prototype is here:

https://github.com/svaarala/duktape/compare/bytecode-dump-load

The testcase is functional but the bytecode format is missing function flags (e.g. "strict", "has name binding", "create Arguments object") and properties ("length", "name", "_Varmap", etc), so some development is still needed.
Code footprint impact is currently ~2kB over master so probably needs a feature option to disable support (for low footprint devices).

Not sure what to do about verification. For now I'm using a 0xFF leading byte followed by a one-byte version number. The 0xFF byte ensures that no ordinary string can be loaded as bytecode as 0xFF prefix never occurs for (extended) UTF-8.
Bytecode format byte could be bumped for every Duktape minor release and kept the same for a patch release.
But a signature is far from being able to validate bytecode - if the signature matches, the current implementation will do memory unsafe things if the data that follows is invalid. Fixing this is very difficult because one would need to validate also the bytecode by simulating all operations, their register indices etc (which is very complicated because there are also indirect operations etc).
So I'm not going to implement bytecode validation as such, i.e. it's not supported to load bytecode from an untrusted source.

Yeah, I wouldn't expect it to have that level of verification, just to make sure the case is handled where trying to execute the wrong version of otherwise valid code fails fast.

Wow, you are on fire.  This is a really important feature.  The main issue I ran into with Lua bytecode was undefined endian, it depended on where the bytecode was dumped.

@JoshEngebretson I've run into that issue myself too. We solved it by using a separate compiler that was able to flip number endianness. But it's a bit annoying that the bytecode is almost portable but that one issue makes it awkward to work with. I want to avoid this issue (and alignment requirements) because the cost of doing so is in the end quite small.

Since the serialized functions are going to need a bunch of properties to work correctly, I'll have to rewrite the dumping approach to use a dynamic buffer rather than to precompute the final size in advance (which becomes progressively more awkward the more properties with variable size there are).
The loading part can be stripped of out-of-bounds checks: that may seem dangerous, but on the other hand broken bytecode is memory unsafe anyway because bytecode instructions may index values outside the executor value stack. That should speed up and simplify loading a little bit.

There are a few limitations I can see with bytecode dumping.  A big one is that modules can't be precompiled, since Duktape.modSearch is required to return source code.  This is a shame, as this would be one of the big use cases for it, I think.
And of course anything using eval() or the equivalent C API (e.g. the RequireScript function in my game engine) needs to stay in source code form as well.

One problem for precompiling modules is that if you just compile the source file it won't work correctly: modules need a specific environment with certain bindings which is now provided using a function wrapper. If precompiled bytecode were used, it'd have to be for the module already wrapped inside the temporary function.
But if you were to do that, I don't wee why you shouldn't be able to load the module directly in modSearch() and then just return undefined (= no source code to load), similarly to how native modules are loaded. It's not intuitive though, so this is certainly something to keep in mind when the module mechanism is reworked.

Force pushed updated branch which should be functionally close to complete. It'll need some cleanup work before it's merge ready though.  See the API testcase for details on what works and what doesn't:

https://github.com/svaarala/duktape/blob/bytecode-dump-load/api-testcases/test-dump-load-basic.c

Pushed an updated branch which is close to merge ready: the branch has documentation, API testcase covering also current limitations, website updates, etc. It's still missing a command line tool to offline compile bytecode, it'd be nice to include such a tool in the distributable.

Implemented bytecode compilation and execution to "duk" command line for convenience (program code only, not eval/function), rebased to master.

I'll be merging this later today - you've been warned :) There's still chance to fix issues and work on details until 1.3 release.

There's still a few tasks unchecked... :wink:

Yup, I'll do them before the merge.

As a fun little test, tested bytecode dump/load with "emscripten inception test". This is the testcase where Duktape and a hello world program (both written in C) are Emscripten compiled, and the resulting Javascript is then executed using Duktape:
# Run the test
$ make emscripteninceptiontest
[...]
emscripten/emcc -O2 -std=c99 -Wall --memory-init-file 0 -Idist/src dist/src/duktape.c dist/examples/hello/hello.c -o /tmp/duk-emcc-test.js
[...]
./duk /tmp/duk-emcc-test-fixed.js
Hello world!
2+3=5

# Compile into bytecode
$ ./duk -c /tmp/duk-emcc-test-fixed.bin /tmp/duk-emcc-test-fixed.js
$ ls -l /tmp/duk-emcc-test-fixed.*
-rw-rw-r-- 1 sva sva 2852105 Jul 28 06:01 /tmp/duk-emcc-test-fixed.bin
-rw-rw-r-- 1 sva sva 1497643 Jul 28 06:01 /tmp/duk-emcc-test-fixed.js

# Execute from source
$ time ./duk /tmp/duk-emcc-test-fixed.js
Hello world!
2+3=5

real  0m0.930s
user  0m0.907s
sys   0m0.016s

# Execute from bytecode
$ time ./duk /tmp/duk-emcc-test-fixed.bin
Hello world!
2+3=5

real  0m0.281s
user  0m0.268s
sys   0m0.012s

So: bytecode is larger than source (which is know, and also described in doc/bytecode.rst) but it loads a lot faster than compilation.

Ok, this is now in master.
I noticed a small issue (which was already in master) for pre-C99 compilers: when GCC 3+ is used, DUK_ALWAYS_INLINE should not be defined (or rather should be defined as empty). Added a bullet to clean that up.

There is an error when it compile duk_api_bytecode.c, because the MACRO DUK_USE_BYTECODE_DUMP_SUPPORT used before the INCLUDE file.

@haosu1987 I think I fixed that in the merge?

https://github.com/svaarala/duktape/blob/8fbd81e597d31e5b0ec85ddf6dfe32322090dc6d/src/duk_api_bytecode.c#L13-L15

Hmm, maybe there's another place than what I fixed.

No, separate sources compile seems fine for me:
$ gcc -o /tmp/test -Isrc-separate src-separate/*.c examples/cmdline/duk_cmdline.c -lm
$ /tmp/test
((o) Duktape [no readline] 1.2.99 (v1.2.0-216-ga9423c5)
duk>

How is the bytecode larger than source?  Aren't all instructions 4 bytes?  I guess the average JS statement needs a lot of instructions...

Bytecode consists of both code and constants. Consider for example:
a=0.1;

That's 6 bytes, but produces one 4-byte instruction and one 8-byte constant (12 bytes).

In fact nearly all numeric and string constants will be larger in bytecode. Generic numbers are 8 bytes in IEEE format and mostly less than that as source code (e.g. "1.234"). Strings will also be larger in general, e.g. "foo" is 5 bytes but the corresponding serialized variant is 7 bytes (4-byte length + 3 bytes data).

Huh, that makes sense.  It's intuitive to think of binary data as more compact than the equivalent text representation in general, but as proven in this case, it's often not true!

Do you deduplicate strings/numeric constants in the bytecode format? ie a
string pool or something similar? Bytecode size isn't a huge deal but code
is often very repetitive in its use thereof... I see this feature very
commonly across bytecode formats so it's probably worth a brief
consideration.
On Mon, Jul 27, 2015 at 10:30 PM, Bruce Pascoe<EMAIL_ADDRESS>wrote:

Huh, that makes sense. It's intuitive to think of binary data as more
compact than the equivalent text representation in general, but as proven
in this case, it's often not true!
—
Reply to this email directly or view it on GitHub
https://github.com/svaarala/duktape/issues/208#issuecomment-125453963.

@bengarney The bytecode dump/load just dumps the in-memory representation without doing much - but the compiler dedplicates strings and number constants during compilation.

@bengarney In a bit more detail, Duktape in-memory representation uses a constant table indexed by bytecode, and that constant table is used for number and string constants, except for trivial integers that can be loaded with a single opcode anyway.

Ah OK - so for "real" programs the actual cost of a constant will be
amortized and likely work out fine. But if I have e.g. just a big table of
numbers  like [1.2, 3.4, 5.6, 1.2] it will be much larger.
On Mon, Jul 27, 2015 at 10:33 PM, Sami Vaarala<EMAIL_ADDRESS>wrote:

@bengarney https://github.com/bengarney In a bit more detail, Duktape
in-memory representation uses a constant table indexed by bytecode, and
that constant table is used for number and string constants, except for
trivial integers that can be loaded with a single opcode anyway.
—
Reply to this email directly or view it on GitHub
https://github.com/svaarala/duktape/issues/208#issuecomment-125454532.

The bytecode will be larger even for stuff like [x,x,x,x,x,x] because there will be one or more opcodes (~4 bytes) for each member and each member is ~2 characters.
In general bytecode is about 2x larger than a minified source. That's fine because the bytecode format is not intended to be small - but fast to load into memory.
Here's a small example: https://github.com/svaarala/duktape/blob/master/doc/bytecode.rst#minimizing-code-size

Cool. Great explanation, and I think it serves its goals very well. I
imagine it also compresses well with even basic zlib if you wanted to
strike a different balance between speed and size.
On Mon, Jul 27, 2015 at 10:39 PM, Sami Vaarala<EMAIL_ADDRESS>wrote:

The bytecode will be larger even for stuff like [x,x,x,x,x,x] because
there will be one or more opcodes (~4 bytes) for each member and each
member is ~2 characters.
In general bytecode is about 2x larger than a minified source. That's fine
because the bytecode format is not intended to be small - but fast to load
into memory.
Here's a small example:
https://github.com/svaarala/duktape/blob/master/doc/bytecode.rst#minimizing-code-size
—
Reply to this email directly or view it on GitHub
https://github.com/svaarala/duktape/issues/208#issuecomment-125455213.

What I find most interesting of all is that even with gzip compression, the source is still smaller.  I would have expected phenomenal compression due to all the redundancy in the bytecode, but minification still beats it out.
Getting back on topic, I don't understand the DUK_ALWAYS_INLINE issue.  What exactly is wrong?

Didn't have time to look into it, but I'm guessing GCC doesn't like:
#define DUK_ALWAYS_INLINE inline __attribute__((always_inline))

with -std=c89.

That makes some sense, I know MSVC (which has abysmal C99 support) chokes on the inline keyword in general and makes you replace it with __inline.

Inlining is now restricted to gcc/clang and C99/C+11 only so there are no longer build failures with pre-C99 compilers related to this.


[bug] setFileHandler() gives invalid http response

setFileHandler() appears to serve some sort of invalid http response, making it nonfunctional.

// Show the window
myWindow.show(myHtml); // Or myWindow.show('./myFile.html');

myWindow.setFileHandler((myUrl: URL) => {
  if (myUrl.pathname === '/app.js') return "console.log('hello from client')";
  throw new Error(`Unknown request "${myUrl.pathname}""`)
});

// Wait until all windows get closed
await WebUI.wait();

console.log("Thank you.");

I am unfortunately unable to debug why the response is invalid, curl -v was not able to give anything useful.
I am on Windows 11.

Aha! That is a good change, I approve.
However, when I try your example, I get one more byte of output than I am supposed to get. I checked with curl and it the extra byte is 0x00, I assume this is the termination of a C-style string?


Allow attachment of worker to Dead-letter queue

References Issue #188
Changes aimed at allowing a worker to be assigned as a handler to the dead letter queue. Also publishing the dlq ARN to the yaml file - similar to how the main queue ARN
Reasons:

It was difficult to add a worker to a DLQ compared to how easy it was to add one to the main queue
It would have required manually inserting the ARN into the YAML after having created the queue
Allowing additional manipulation of the DLQ due to exposure of it's ARN in the YAML file

Hi, I'm curious why you would want to have a worker on the DLQ?

Hi @mnapoli - In my specific scenario, I'm using SQS to handle making api calls to a third party provider. If their API is unavailable for some reason, the messages will get pushed into the DLQ, and I need to be able to retry these failed messages at a second provider. This means changing up the content of the message.
Here's another example from the AWS blog, written by one of their Senior solution architects.
Essentially, there exists use cases where simply pushing the failed messages back into the main queue isn't viable, and additional processing is required.

Thanks for the explanation.
I think that falls outside of the scope of the construct, as it is right now. I don't see that feature being merged at the moment.
Maybe #128 is an alternative that has more chance of happening. Would that work for you?

As far as the base framework goes, exposing the URL allows you to send messages into the queue, but in order to consume messages from it I believe the ARN needs to be exposed?
Sadly I don't think this would be a viable solution - unless it's modified to expose the DLQ ARN too, similar to what I've done in this PR.

@mnapoli Updated to remove the dlqWorker construct, now only exposes dead letter queue URL & ARN

LGTM :) Merging and issuing release now ;) thanks @whambulance for the contribution !


Don’t forget the web.config

Here’s the template for Node: https://a-k-apart.com/webconfigs/web.config.node.txt

@aarongustafson thanks.
Sorry I'm not familiar with Azure, I've literally just plonked that in the root and renamed to web.config. That's enough right? Should I be using something like forever in my npm start script or does iisnode handle all that?

I think IIS will handle that, but I will deploy and we can see where we’re
at.
On Mon, Sep 26, 2016 at 4:14 PM, Luke Childs<EMAIL_ADDRESS>wrote:

@aarongustafson https://github.com/aarongustafson thanks.
Sorry I'm not familiar with Azure, I've literally just plonked that in the
root and renamed to web.config. That's enough right? Should I be using
something like forever in my npm start script or does iisnode handle all
that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lukechilds/tor-explorer/issues/1#issuecomment-249683822,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEn2J9UwlbqHLuadbJq5he7RhZk0Kl3ks5quCePgaJpZM4KGzRk
.

@aarongustafson Do you need me to do anything else?

I think we’re good.


Plugin triggering GitHub Rate Limits

Exception:
Sep 29, 2015 1:19:26 PM org.jenkinsci.plugins.ghprb.GhprbRepository initGhRepository
SEVERE: Error while accessing rate limit API
java.io.IOException: Server returned HTTP response code: 401 for URL: https://api.github.com/rate_limit
        at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
        at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
        at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
        at java.lang.reflect.Constructor.newInstance(Constructor.java:422)
        at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1889)
        at sun.net.www.protocol.http.HttpURLConnection$10.run(HttpURLConnection.java:1884)
        at java.security.AccessController.doPrivileged(Native Method)
        at sun.net.www.protocol.http.HttpURLConnection.getChainedException(HttpURLConnection.java:1883)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1456)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1440)
        at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:254)
        at org.kohsuke.github.Requester.parse(Requester.java:458)
        at org.kohsuke.github.Requester._to(Requester.java:227)
        at org.kohsuke.github.Requester.to(Requester.java:194)
        at org.kohsuke.github.GitHub.getRateLimit(GitHub.java:245)
        at org.jenkinsci.plugins.ghprb.GhprbRepository.initGhRepository(GhprbRepository.java:71)
        at org.jenkinsci.plugins.ghprb.GhprbRepository.init(GhprbRepository.java:47)
        at org.jenkinsci.plugins.ghprb.Ghprb.init(Ghprb.java:86)
        at org.jenkinsci.plugins.ghprb.GhprbTrigger.start(GhprbTrigger.java:205)
        at org.jenkinsci.plugins.ghprb.GhprbTrigger.start(GhprbTrigger.java:52)
        at hudson.model.AbstractProject.onLoad(AbstractProject.java:331)
        at hudson.model.Project.onLoad(Project.java:98)
        at hudson.model.Items.load(Items.java:327)
        at hudson.model.ItemGroupMixIn.loadChildren(ItemGroupMixIn.java:109)
        at com.cloudbees.hudson.plugins.folder.Folder.onLoad(Folder.java:240)
        at hudson.model.Items.load(Items.java:327)
        at jenkins.model.Jenkins$17.run(Jenkins.java:2685)
        at org.jvnet.hudson.reactor.TaskGraphBuilder$TaskImpl.run(TaskGraphBuilder.java:169)
        at org.jvnet.hudson.reactor.Reactor.runTask(Reactor.java:282)
        at jenkins.model.Jenkins$7.runTask(Jenkins.java:899)
        at org.jvnet.hudson.reactor.Reactor$2.run(Reactor.java:210)
        at org.jvnet.hudson.reactor.Reactor$Node.run(Reactor.java:117)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
        at java.lang.Thread.run(Thread.java:745)
Caused by: java.io.IOException: Server returned HTTP response code: 401 for URL: https://api.github.com/rate_limit
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1839)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1440)
        at java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:480)
        at sun.net.www.protocol.https.HttpsURLConnectionImpl.getResponseCode(HttpsURLConnectionImpl.java:338)
        at org.kohsuke.github.Requester.parse(Requester.java:454)
        ... 23 more

Hi thanks for the report. What is your plugin version?

Plugin Version: 1.29
Jenkins Version: 1.631

The exception occurred during Jenkins start-up, it seems it prevent Jenkins from correctly finishing the start-up procedures so therefore it appeared to have hung. I had to disable the job in order to start Jenkins

I presume you have an ungodly amount of GHPRB usage history, right?

Are you running GHPRB anonymously or in an authenticated fashion?

@arcivanov do you know the reason for why the init code is calling getRateLimit, and can we remove it?

            if (gitHub.getRateLimit().remaining == 0) {
                return false;
            }

Why? Beats me, probably there was a good reason in the past, but right now it's completely useless: the false returned by  initGhRepository is only checked in check of GhprbRepository, but check's returned value is not checked anywhere so we can easily remove this.

Afraid first time user!
I've found the reason for the problem, it appears I misconfigured the secret key. After fixing that and waiting for the next period of rate limiting it works as expected.
Although I solved my problem, ideally the plugin would stop sending API requests when either the rate limit or unauthorised status code is received with the appropriate log messaging.

@JBodkin That would make sense, but the only time the rate limit is checked is when initializing the repo.

It appears there are two scenarios whereby the error occurs, during startup and switch a build state from disabled to enabled.

If the project is disabled, we probably shouldn't run the init code then.

Once the fixes for the github-api go in, combined with various fixes made in the ghprb itself, this should no longer happen.

These are the related issues with the github-api and rate_limit:

https://github.com/kohsuke/github-api/issues/273
Although it seems it was fixed in:
https://github.com/kohsuke/github-api/issues/78

But i still get the same issues

This is the critical PR for this: https://github.com/kohsuke/github-api/pull/283
Without that, User info requests can cause huge numbers of simultaneous API calls.

We are running the plugin in a very large setup and not hitting the limits, also the mentioned PRs are now already included.
I therefore close the issue.


Not working with mongoose 4.0.4 when in embedded array item

Getting: [Error: Unable to invalidate a subdocument that has not been added to an array.]
'use strict';
var co = require('co'),
    mongoose = require('mongoose'),
    mongooseLong = require('mongoose-long');

mongooseLong(mongoose);
mongoose.connect('mongodb://localhost/dummy');

var dummySchema = new mongoose.Schema({
        foo: mongoose.Schema.Types.Long
    }, {_id: false}),
    schema = new mongoose.Schema({
        dummies: [dummySchema]
    }),
    DummyModel = mongoose.model('Dummy', schema);

co(function *() {
    let dummy = new DummyModel({});
    yield dummy.save();

    yield DummyModel.update(
        {_id: dummy._id},
        {$push: {dummies: {foo: 1}}}
    ).lean().exec();

    process.exit(0);
}).catch(function (err) {
    console.error(err);
    process.exit(1);
});

So is this project dead or is this issue ignored?

Not ignored, just got lost in the shuffle. Our practice for handling issues across multiple repos was much worse in 2015 :( Going to take a look at this.


Travis build for windows failed

The template (with necessary changes) fails on travis with the following error
Running `wine /target/x86_64-pc-windows-gnu/debug/heaven-on-earth.exe`
err:module:import_dll Library ncrypt.dll (which is needed by L"Z:\\target\\x86_64-pc-windows-gnu\\debug\\heaven-on-earth.exe") not found
err:module:LdrInitializeThunk Main exe initialization for L"Z:\\target\\x86_64-pc-windows-gnu\\debug\\heaven-on-earth.exe" failed, status c0000135

Is there a way to fix this or to disable testing for a windows target?

Is there a way to fix this or to disable testing for a windows target?

You can just comment out the lines related to Windows targets in .travis.yml.

I have the same problem. Did someone find a solution ?


binary ninja 2.0 can not analyse switch jump table correctly!

there is supposed a switch table.

Could you possibly share this binary?

lelfei_fix.exe.zip

IDA can identify switch table correctly.

Are you sure the image came from that binary? or did you rebase? I was not able to find the raw bytes or the code from the image in the file provided...

the big function is at 0x401630.I update the bn,now it can identify the switch jump table, but i maybe crash.because the function is too big.

Just to follow up -- can you please double check the binary that you're having trouble with and make sure you shared the right one? The screenshot you showed is at 0x400a50, but in the binary you included, there's nothing at that address! See:

There also have been several bug-fixes to the 2.0 beta over the last week so if you can make sure you're on the latest dev and try again, please let us know if you're still experiencing any problems.

As we've not gotten a binary which demonstrates this issue we can not fix. Please reopen if you have a binary you can share or additional details.


validClassName doesn't like camelCase class names.

https://github.com/observablehq/plot/blob/e2749e8bd3f28305f02c7aa6e7f0d9c0f99e93cb/src/style.js#L417
validClassName.test('myNewClassName') fails.

CSS class name selectors are case-insensitive, so consider using mynewclassname instead. But we should make this validation case-insensitive since it is a valid name.


39 give the ability to switch between deployed backend and localhost

Pull Request Summary
Fixed the docker file to include all subdirectories in both the front end and backend
Added the ability to switch between the deployed backend and the localhost backend for development purposes
Description
Modified docker file
Added dotenv to the front end to allow for environmental variables
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that aren't relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Creating tests
[ ] Frontend/UI change

Has This Change Been Tested?

[ ] Yes
[x] No

If "yes", please...
**Describe the tests that you ran to verify your changes: **
**Provide instructions so we can reproduce: **
**List any relevant details for your test configuration: **
Any questions, comments, concerns, or next steps?

@Ajinkya-B I fixed that problem we discussed over discord with the data formatter, I did it in a separate branch it was a one line chance which I merged into this branch prior to the merge of development. So that change will also be included in this pull request just wanted to document it

@Ajinkya-B I fixed that problem we discussed over discord with the data formatter, I did it in a separate branch it was a one line chance which I merged into this branch prior to the merge of development. So that change will also be included in this pull request just wanted to document it

Ok perfect, I reviewed the files and they look good!


MuiButton can't Test. Enzyme Internal Error: unknown node with tag 14

when i use enzyme mount/shallow button, will failed. Enzyme Internal Error: unknown node with tag 14(v4.2.1 can work, but v4.9.7 can't work)

[x] The issue is present in the latest release.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Enzyme Internal Error: unknown node with tag 14

Expected Behavior 🤔
can test

Steps to Reproduce 🕹

Steps:
import React from 'react';
import { ReactWrapper, mount } from 'enzyme';
import { Button } from '@material-ui/core';

describe('Test MuiButton', () => {
  it('test mount', () => {
    const wrapper: ReactWrapper = mount(
      <Button variant="contained">Default</Button>,
    );
    console.info(wrapper.debug());
  });
});

Context 🔦

Your Environment 🌎
localhost

Tech
Version

Material-UI
v4.9.7

React
16.11.0

TypeScript
3.6.4

enzyme
3.9.0

enzyme-adapter-react-16
1.9.1

What was the fix for this?  I am encountering the same issue.

@smaye81 It's likely related to the version of enzyme and React you are using (they need to match)

Turns out the fix for me was to upgrade my version of enzyme-adapter-react-16 from 1.9.1 to 1.15.6


Memory leak with observer

There is a memory leak when using an observer, this appears to be caused by mixinLifecycleEvents not patching componentWillUnmount resulting in the reaction never being disposed.
After adding patch(target, 'componentWillUnmount'); the memory leak went away.
Is there any reason it was omitted, or can it be added in?

@jsaunderson Looks like that is missing indeed! Feel free to open PR


Load URL with POST data

I am trying to load a URL with POST data.
Is there any way we can do with it

If you create your own request object GuzzleHttp\Psr7\Request you can pass it to the loadFromUrl method as the fourth parameter. You can set up the request object any way you want, including changing it to a POST request and using a custom url.
Hope that helps.


Default value when no initial params

Heya,
When adding an association through for example Ecto.Changeset.put_assoc/3, the params attribute in the changeset is nil. Then when you want to use a polymorphic embed with inputs_for/3, it throws the following error:
** (BadMapError) expected a map, got: nil
    (elixir 1.10.4) lib/map.ex:450: Map.get(nil, "parser", %{})
    (polymorphic_embed 0.4.0) lib/html/form.ex:25: PolymorphicEmbed.HTML.Form.to_form/5
    (polymorphic_embed 0.4.0) lib/html/form.ex:12: PolymorphicEmbed.HTML.Form.polymorphic_embed_inputs_for/4

This is a fix for the issue.

👍


slightly different performance result of un-conditional generation on ImageNet

Hi, my reproduction result (using the very eval code and ckpt in this repo) of un-conditional image generation is slightly different from your official results. As for ViT-B, the FID and IS scores are 13.74 and 81.16 (1.76), respectively. Is such performance fluctuation normal and acceptable?

This should not be the case, as the random seed for the generation is fixed in the script. Do you use the script provided in this repo to generate the imagenet validation set?

Thanks for your advice! We later found that the bug is caused by the saving format of images, we saved the imagenet validation set as '.jpeg' format rather than '.png', which caused this difference.
The corrected result of our reproduction is FID 11.18 & IS 80.57 (1.22), nearly the same as your official results.


Limitations using angular-cli.json instead of webpack.config.js?

Hello, why is the webpack file missing? Was it replace by angular-cli.json? if so, does it imply that there's less flexibility using angular-cli.json then using only the webpack file in case you want to use a custom project structure? I also haven't found any documentation about the angular-cli.json and how it works.

some config in angular-cli.json can pass to the angular-cli when you exec 'ng serve' or 'ng build', about "apps" in the angular-cli.json. you can clone the code of angular-cli, this section about webpack config can  understand with little diffcult .

I thought the whole point of using angular-cli was so that you could have an opinionated, standardised build setup - with the actual details of it abstracted away from you (with the important thing being consistent and replicable builds, despite the underlying technology).
That aside, perhaps it would be useful to be able to do something similar to create-react-app, and offer an "eject" command to dump all of the build files?
In regards to documentation, that may well be an ongoing effort, as this tool is still in beta.

I think a webpack config file should be provided which can be imported and merged with the main webpack config file of the cli. Then we can use this webpack file for our custom task for example copying any files other than the assets folder or using fontgen-loader to generate & load icon fonts from svg icons etc.

Yeah, there should be a webpack config file that we custom to provide more options.

I agree, I have a build that currently won't support the chunkhash generated by cli's prod build. I have to hack the cli to remove it. However, this is probably a duplicate to #1656 which @hansl closed another issue regarding webpack config and stated this comment as cli's position.

Aah, that does sound very interesting. An addon system would probably solve the problem. For the time being you could potentially write your own scripts the run alongside angular-cli. The final solution I suppose could be to write your own webpack configuration, as hansl has said in that other issue, if you are at the point where you need to customise your build that much, then you may already be at the point where you're comfortable enough setting up your own build anyway.

But that's just the rub. It was actually quite easy to change in angular-cli and it's basically the only change I need right now. I'd rather not move away from a rubust scaffold, test and build tool and spend my own time writing a whole custom build or rely on a  less well-known/supported starter. So it seems hacking is the way to go right now if you only need a few tweaks. But I agree, I'd love to see the addon system takeoff because I do think it will alleviate/solve this and other problems.

Well, I've finally had an issue with this. I'm using Handlebars for some on-the-fly, simple templating coming from a web service backend. I've had to hack around getting Handlebars to load without issuing warnings because I can't use the handlebars-loader.
I'd love to make some kind of suggestion about how an add-on system would work, but it'd be a pretty mammoth task. Maybe it's already being thought about?

I'm going to have to close this one as a dupe of https://github.com/angular/angular-cli/issues/1656. I know the discussion there is closed so you can't discuss there, and that is unfortunate, but on the other hand most of what there is to discuss and be said around this topic has already been said and discussed there.
@t0f3rchr15 you're not the only one wanting to disable the hashing, but I can't say it's a big priority. See https://github.com/angular/angular-cli/issues/2774 for more info.

@filipesilva Can you give us some updates on the progress of the addon system mentioned in #1656? For my project this addon system is crucial if I want to use this awesome CLI.
I really need handlebars in my project. Now I'm not sure if I should start without handlebars and wait for the addon system or not.


Setting hotkeys through .kwmrc

It would be useful to just be able to set hotkeys through the .kwmrc file instead of modifying hotkeys.cpp and recompiling. Something like
kwmc bind ctrl+alt+cmd-t focus -t toggle
kwmc bind ctrl+cmd+alt-q quit
kwmc bind cmd+alt-t space -t toggle

Currently I have no plans of changing the hotkeys system.
While I do agree that this seems like a much more user friendly way to approach this kind of functionality, it will put pretty strict limits on what users can and cannot do.
The current system allows the user to extend by writing C code to write conditions and what not, and so a given hotkey can actually do many tasks depending on things happening on the users system.

Something like bspwm might be interesting to see.
I agree that having the hotkey-system being code is powerful, but it makes the config far less portable. If one wanted to use the same hotkey config on different systems they would either have to fork this project or have a special compilation step during install. This makes using package managers like Homebrew a little awkward.
Using some sort of configuration file or simple scripting language that is interpreted at run time would allow for people to easily maintain a single config across systems.

I'll see if I can implement something similar to kwmc bind keys command that can be placed in .kwmrc, while still keeping the live-coded hotkey system intact.

You can now create binds using the command kwmc bind mod+mod+mod-key command and kwmc unbind mod+mod+mod-key.
Example: kwmc bind cmd+alt-l window -f next
You can also bind keys to perform a system command by using the 'sys' prefix as previously.
Example to launch iTerm: kwmc bind cmd+alt+ctrl-return sys open -na /Applications/iTerm.app


SparkApplication properties - .spec.sparkConfigMap, .spec.driver.securityContext, .spec.executor.securityContext, spec.driver.nodeAffinity not working as expected.

Hi,
We are facing following issues on using the above mentioned properties :-

.spec.sparkConfigMap
When  Spark configuration is added by ConfigMap storing Spark configuration files (spark-defaults.conf, spark-env.sh, log4j.properties) using the optional field .spec.sparkConfigMap ,SPARK_CONF_DIR is being set to point to the mount path of the ConfigMap , i.e, /etc/spark/conf/ but properties mentioned in spark-defaults.conf are not being considered and environment variables set in spark-env.sh are not getting reflected in driver.
We observed that properties mentioned in spark-defaults.conf are not getting appended in /opt/spark/conf/spark.properties so it is not being taken into affect.
log4j.properties is taking effect as expected.
Configuration Used :-
sparkConfigMap: "spark-config"

.spec.driver.securityContext/ .spec.executor.securityContext:
When PodSecurityContext is specified for the driver and executor pod by setting .spec.driver.securityContext.runAsUser to '1000’, the pods are still coming up with default root user.
Configuration Used :-
spec:
driver:
securityContext:
runAsUser: 1000
executor:
securityContext:
runAsUser: 1000

spec.driver.affinity:
When nodeAffinity for driver is set to run on a node with label ‘<key=value>’,the driver is coming up on different node ignoring the affinity set.
Configuration Used :-
driver
affinity:
podAffinity:
requiredDuringSchedulingIgnoredDuringExecution:
nodeSelectorTerms:
- matchExpressions:
- key: spark-affinity-test
operator: In
values:
- true
executor
affinity:
podAntiAffinity:
requiredDuringSchedulingIgnoredDuringExecution:
nodeSelectorTerms:
- matchExpressions:
- key: spark-affinity-test
operator: In
values:
- true
We, also tried specifying podAntiAffinity for executor, but that also didn't work.

Also, please be informed that webhook is enabled for all of the above scenarios.
Can you please verify if we are missing any configuration or is it a known issue?
Thanks!!
Breeta

hi @liyinan926
Writing on behalf of @breetasinha1109
We do see that OwnerReference is added to the driver pod that references the owning SparkApplication in metadata.
[root@vm-10-15-173-143 programs]# kubectl get pod spark-security-context-driver -ncspk -oyaml
apiVersion: v1
kind: Pod
metadata:
annotations:
kubernetes.io/psp: restricted
seccomp.security.alpha.kubernetes.io/pod: docker/default
creationTimestamp: "2019-06-03T06:52:49Z"
labels:
spark-app-selector: spark-7f97fd6ae3e042faaa6b659b87c107f2
spark-role: driver
sparkoperator.k8s.io/app-name: spark-security-context
sparkoperator.k8s.io/launched-by-spark-operator: "true"
sparkoperator.k8s.io/submission-id: bb961944-a799-4113-9706-e3cc08cd518e
version: 2.4.0
name: spark-security-context-driver
namespace: cspk
**ownerReferences:

apiVersion: sparkoperator.k8s.io/v1beta1
controller: true
kind: SparkApplication
name: spark-security-context**
uid: 30d7af7d-85cc-11e9-9094-fa163e610ac2
resourceVersion: "15064848"
selfLink: /api/v1/namespaces/cspk/pods/spark-security-context-driver
uid: 3389150d-85cc-11e9-9094-fa163e610ac2

I am hitting the same on Openshift,I cannot override the default security context.
I think its the check here: https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/blob/master/pkg/webhook/patch.go#L76-L81 where it assumes the pod context to be nil. Why is that?

@SachinRavadi and @breetasinha1109 which version of Spark operator are you running?

@liyinan926 on Openshift you get a default policy if you dont specify anything:
 oc describe scc restricted
Name:	restricted
Priority:	<none>
Access:
  Users:	<none>
  Groups:	system:authenticated
Settings:
  Allow Privileged:	false
  Allow Privilege Escalation:	true
  Default Add Capabilities:	<none>
  Required Drop Capabilities:	KILL,MKNOD,SETUID,SETGID
  Allowed Capabilities:	<none>
  Allowed Seccomp Profiles:	<none>
  Allowed Volume Types:	configMap,downwardAPI,emptyDir,persistentVolumeClaim,projected,secret
  Allowed Flexvolumes:	<all>
  Allowed Unsafe Sysctls:	<none>
  Forbidden Sysctls:	<none>
  Allow Host Network:	false
  Allow Host Ports:	false
  Allow Host PID:	false
  Allow Host IPC:	false
  Read Only Root Filesystem:	false
  Run As User Strategy: MustRunAsRange
    UID:	<none>
    UID Range Min:	<none>
    UID Range Max:	<none>
  SELinux Context Strategy: MustRunAs
    User:	<none>
    Role:	<none>
    Type:	<none>
    Level:	<none>
  FSGroup Strategy: MustRunAs
    Ranges:	<none>
  Supplemental Groups Strategy: RunAsAny

https://docs.okd.io/3.6/install_config/persistent_storage/pod_security_context.html
I am planning to make overriding options configurable is that ok?

@liyinan926  We are using v2.4.0-v1beta1-0.8.2 version of spark-operator.

Hi,
Facing the same issue here about .spec.driver.securityContext and .spec.executor.securityContext which not change the user context inside the container.
Our use case is to run Spark jobs which send requests to Hive througth LLAP. LLAP does not support impersonation so we have to be able to run our job as the Hive UID if we want to be authorized by the Hive metastore, pretty anoying and no workaround found on the job itself for now.
Are we agree about the fact that this specs allow jobs to be run on a different context inside the container ? For example, running with user test or something ? Or maybe I missunderstand something.
Driver logs :
`
++ id -u

myuid=0
++ id -g
mygid=0
set +e
++ getent passwd 0
uidentry=root:x:0:0:root:/root:/bin/ash
set -e
'[' -z root:x:0:0:root:/root:/bin/ash ']'
SPARK_K8S_CMD=driver
case "$SPARK_K8S_CMD" in
shift 1
SPARK_CLASSPATH=':/opt/spark/jars/*'
env
grep SPARK_JAVA_OPT_
sort -t_ -k4 -n
sed 's/[^=]=(.)/\1/g'
readarray -t SPARK_EXECUTOR_JAVA_OPTS
'[' -n '' ']'
'[' -n '' ']'
PYSPARK_ARGS=
'[' -n '' ']'
R_ARGS=
'[' -n '' ']'
'[' '' == 2 ']'
'[' '' == 3 ']'
case "$SPARK_K8S_CMD" in
CMD=("$SPARK_HOME/bin/spark-submit" --conf "spark.driver.bindAddress=$SPARK_DRIVER_BIND_ADDRESS" --deploy-mode client "$@")
`
And next, we've got a permission denied exception from hive saying specificly USER=root.
As you can see the user context remain root.

I have deploy spark-operator with the latest helm charts (v0.5.0) deploying the latest img (v1beta2-1.0.1-2.4.4). I have just enabled metrics and webhook and set sparkJobNamespace as "spark".
I run my spark's jobs with a YAML static file realy closed to this one : https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/blob/master/examples/spark-pi-prometheus.yaml with .spec.*.securityContext of course
Thanks !!


Improve transaction list in account detail

Improve Transaction List:

Change transaction list UI on account detail ( remove pages and show only 20 transactions (default) and add view all transaction list button which display all transaction based on the account)

PR: https://github.com/proximax-foundry/web-explorer-vuejs/pull/235


Translationfr

Test git for translation. Add doc/about/about.bb  to the dev.

There is something wrong with this PR. It seems to be against an older version. Please make sure to create PR's against recent dev branch (if you look at the changes tab you should only see the changes you made).


Update quit-applications extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Works as expected for me 🔥 Maybe we should consider a preference to poptoroot/clear search bar after quit? What do you think @codesjedi
@mackopes do you want to give it a go too?

Hey @pernielsentikaer, thanks for the suggestion. I've added it and updated the changelog.md file accordingly. 🙂

Nice! 👩🏼🚒


Add information about Arch Linux package.

I'm currently maintaining the kurly Arch User Repository package.

Thank you so much both for the PR and for maintaining the kurly for Arch.  That's awesome. 👍
I'm going to merge this and then make a OS-dist section for the readme.
https://aur.archlinux.org/packages/?O=0&K=kurly
:-)


Bad renders after last ControlNet update --

i have been using controlnet since a while without any problem. After recently updating all i am getting is "half-renders", like if SD was not completing the steps. I tested it with all the controlnet model but the result is the same
I haven't touched anything in the models folder, just added the new models to the one i already had and always worked.
I am just guessing but I think it has to do with yaml file. Problem is that i am also not sure which yaml file could be casuing the problem, since there a few.

Any help would be very appreciated. Thx a lot!

I am getting horrible renders too and I did a fresh install of stable diffusion webui as a friend instructed me, only changed controlnet model cache size to 2 and added an extra controlnet slot and I have this type of quality no matter what. I even followed this tutorial exactly the same and mine looks awful.. What the hell😔 I need a solution ASAP for this extension because I need to use it😔 https://youtu.be/wDM8iDK-yng

ok let us look into it

Thank you for looking into it

can you check your setting and see if you mistakenly checked "cfg-based controlnet"?

When I reinstalled webui for the fresh install, that setting was off. The only setting checked in controlnet is use mid-control on highres pass. My other install of webui previously before the fresh install is in this picture below. Either way I get bad looking renders that don't represent the style at all. Even if I am using canny or depth mode

l, it all looks horrible

uncheck the cfg-based  guidance and restart your webui completely, is the problem solved?
does this option checked by default when fresh install?

No, and I used this on the previous install of webui
On the fresh install of webui, this isn't checked and I still get the same bad quality

it seems that the "cfg-based guidance" is broken. we will further look into it

Thank you so much once again 🙏

are u willing to share the input image that you tested?

here's mine

i just saw the updates of the issue.
I also followed your instruction but the problem stays

Second image is for the style

I also tried using an image that wasn't at 512 x 512, used innerfit in controlnet too.

it seems that the building image is a difficult case for ControlNet and the model fail to recognize it. it is not a bug.
The girl case is likely to be caused by t2i adaper, which is not a part of ControlNet. let me know if you can reproduce the problem without using t2i adaper

Sadly on looks I am getting and I got the style adapter from here https://huggingface.co/TencentARC/T2I-Adapter/tree/main/models
The canny, and depth models etc are from your huggingface

Can you get similar quality in webui?

can you send me your screenshot with exactly same setting with me?

For the girl image case, please let me know if the problem exist without using t2i adapter. If the problem exist with pure text prompts, we will look into it deeper.

All below are with having the weight in controlnet range from 0.5-1. Also, using your models instead and no adapter. If I have the weight low like 0.15, of course I see my prompt but this defeats the point of me using your extension :(

From the result it seems that the model works as intended. If no additional bugs are reported (especially thay building image), we will close this soon.
The t2i-adapter is from a 3rd team (Tencent-ARC), and the webui plugin's support for t2i-adapter is experimental - because we are not sure if that model is robust enough for a production environment.
All standard controlnets are well-tested and should be robust.
If you want to transfer the prompt contents into your wanted coarse shape, a better choice could be using the depth model and tune the parameters a bit.
Thank you for your supports as always.

Well I appreciate you taking the time looking into this for us at least, thank you for trying, I really wish this would be totally different.

Besides, you are encouraged to direct your issue to Tencent-ARC
https://github.com/TencentARC/T2I-Adapter
Nevertheless, please note that they are an independent team and may not have the same way as us to interact with end users.

I closed the bug issue in webui https://github.com/Mikubill/sd-webui-controlnet/issues/555
let me know if the problem still exist with that building image and I will reopen it

And before you report to Tencent-ARC you are encouraged to see if your problem can be reproduced by their official demo https://huggingface.co/spaces/Adapter/T2I-Adapter

Thanks for that recommendation. I tried it just now and used a different input image and the issue still proceeds so there is something going on. Its weird because this is a fresh install of webui. SMH..

Unfortunately, this is because that model is not robust enough. The problem even exist in their official demos.
We are not able to do anything more about this. This model belongs to Tencent-ARC

Thank God it isn't just me, THAT is good news, so within the two days of people making videos about this tool, something got broken. I reached out to them, and even though this will be closed, can I still send them this thread so they have deeper info and more proof of what I am dealing with?

I am going to close this issue. feel free to let me know if real addional bugs are reported and I will reopen it.

Thank God it isn't just me, THAT is good news, so within the two days of people making videos about this tool, something got broken. I reached out to them, and even though this will be closed, can I still send them this thread so they have deeper info and more proof of what I am dealing with?

Again, any problem caused by other projects should be directly posted to that project, like https://github.com/TencentARC/T2I-Adapter/issues

so followed your settings and it seems like its working now for me. I will test it with more random settings and prompt and let you know how is it going

For me everything seems fixed. Thank you guys!


Assert future or past Mock Oracle calls

Problem
OracleMock is currently useful for replacing an oracle and injecting values into code under test, but it doesn't let us test how the circuit calls the oracle. This will be quite important in some scenarios, e.g. in Aztec when using oracles to perform public storage writes, where we want to know the storage slot and value passed to the call.
Happy Case
There's multiple ways to do this. Some options off the top of my head:

have the oracle call a closure in which the user might perform some assertion, or store the value somewhere for later inspection
provide a way to query past oracle calls
provide a way to perform an assertion over the next oracle call

The past assertion over past and future call approches might be problematic in functions where the oracle is called multiple times - how would we handle this? Foundry follows this approach for events and calls, as as long as one of them in the transaction matches the assertion the test is considered as passing.
Project Impact
Blocker
Impact Context
There's some design patterns that are simply untestable without this feature.
Workaround
None

Related to https://github.com/noir-lang/noir/issues/4371, though I believe this one is a bit broader.


New entry into superset user

SUMMARY
I have added a new company that extensively uses apache superset for the dashboards and visualization purpose.

Could you please add the company in abc order? Thanks!

Done


Can't play songs from the MusicTime menu in VS Code

Describe the bug
A clear and concise description of what the bug is.
After logging in to spotify and selecting either web player or desktop player, I am not able to play any songs from the MusicTime menu in VS Code.

ISSUE TYPE

Bug Report

Steps to reproduce

Go to MusicTime menu in VS Code
Verify that you are logged into Spotify and that you have either a Web or Dekstop player open
Make sure that either player is selected in MusicTime
Click any song in playlists or recommended.
Music doesn't play and there is no error thrown.

Expected behavior
A clear and concise description of what you expected to happen.
Music plays when a song is selected.

ACTUAL RESULTS

Nothing plays when selected from the MusicTime menu. When a song is selected in Spotify the status bar displays correctly in VS Code and I'm able to stop/play or skip/go back. But selecting new songs still doesn't work.

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS Mojave v.10.14.6
Plugin version v.1.8.1

Additional context
Add any other context about the problem here.

@RickCoxDev this happens on both the web player and desktop? Also do you have a premium or non-premium account? Think we might have an idea of what's going on. Will send an update soon.

I have a non-premium account and yes it happens for both web player and desktop.

@RickCoxDev we're working on a fix for this now that should be ready in the next release tomorrow.

@RickCoxDev This is fixed in 1.8.4. With a non-premium account on Mac, the player will only work with the desktop app installed (since we use applescript instead of Spotify APIs to control the player). Let me know if everything works on your end.

Everything seems to be working great now. Thanks!


Added submariner operator to catalog

Added required manifests for the submariner operator

@james-cccc , one minor issue with the README.  Once that's fixed, I'll merge this PR.
Thanks!

@pittar - fixed the typo issue so we should be good now :)

Thanks for the contribution!


Inner license

You have: MPC-HC is free software;
in your license file area. In your version.sh to be specific...
Just letting you know!
Thank you for keeping this alive as I prefer it over notepad++

Yup, good catch! :+1:
3 occurrences here: https://github.com/XhmikosR/notepad2-mod/blob/master/version.sh#L4-L11

This is by design. I'm the author of that file and I used it here too.


tests/drivers/i2s_api: test_i2s_transfer_tx_underrun fails on sam_e70_xplained in user mode

Describe the bug
A particular test case for i2s fails on sam_e70_xplained when run in user mode, the output just hangs forever.
To Reproduce
Pull #11420
Remove the #if 0 block from tests/drivers/i2s/i2s_api/src/main.c so that the test case test_i2s_transfer_tx_underrun is executed in user mode
Observe that the test starts and hangs with no console output at all
Expected behavior
The test to complete successfully
Impact
Unsure. Might be an issue with the test design, the driver, or excessive latency imposed by running from user mode

@ManojSubbarao do you have anyone who can look into this?


Flow:

Spark logged in as user
Join camp page
List of enabled and open camps
Request to join sent by email to camp manager
Save the state in user model, camp_id = -1 (for example)
Camp manager will approve/decline in pending requests page

Mail tested with:
https://mailtrap.io/

Tradeoffs

Testing Done

Cobditions:

user role include 'camp_manager'
user camp_id not null


Add contains support to NSSet and NSOrderedSet

Currently only NSArray supports the [[container should] contain:object] assertion. This commit adds the same support to NSSet and NSOrderedSet.

Pull requests are amazing.


add a test case for a build with 2 build vols

Fixes #119

Local CI status: passed
=== 61 passed in 702.48 seconds ===


New Feature: Search data-filterTableKeywords as well as text

I had the requirement to filter based on extra text so I implemented code where the data-filterTableKeywords attribute on any tags is also checked for search string.

existing code modified to use a function isMatch for all matching code which was repeated in several places
added check of data-filterTableKeywords attribute on each match.
added new example page filtertable-datatag.html  and added to index.

Obviously i have only updated jquery.filtertable.js and not jquery.filtertable.min.js
I also wondered about adding a new config option to allow user to specify name of data attribute.  Can look to do that if this pull request is accepted.
This is different to pullrequest #17 as it adds extra keywords to be searched on also rather than searching on a differnt value.

@sunnywalker  Is there some changes I should make to this pull request for it to be merged?

@sunnywalker I have been using this change quite well on my own systems, is this something you are willing to merge in do you think?


fix bash

Coverage remained the same at 98.895% when pulling 7fd5d2ffe91e2f3e54354c80f1b244f31037d349 on emosbaugh:master into 41885ce6fb83b7e7cf1951128f2636100205a7fe on replicatedhq:master.

Coverage remained the same at 98.895% when pulling 7fd5d2ffe91e2f3e54354c80f1b244f31037d349 on emosbaugh:master into 41885ce6fb83b7e7cf1951128f2636100205a7fe on replicatedhq:master.

lgtm


bug: opentrons_execute does not exit cleanly

Overview
Protocols executed directly on the robot with the opentrons_execute utility perform (most of) the expected robot motions, but throw an error instead of killing the process cleanly.
Current behavior
When used as described in the docs, otherwise valid protocols executed directly on the robot with opentrons_execute produce the following error in the terminal:
Unhandled error in exception handler
context: {'message': 'Task was destroyed but it is pending!', 'task': <Task pending coro=<Controller.watch_modules() running at usr/lib/python3.7/site-packages/opentrons/hardware_control/controller.py:185> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x76194fb0>()]>>}
Traceback (most recent call last):
  File "usr/lib/python3.7/asyncio/base_events.py", line 1632, in call_exception_handler
  File "usr/lib/python3.7/site-packages/opentrons/hardware_control/util.py", line 14, in _handle_loop_exception
KeyError: 'exception'

Additionally, by monitoring journalctl -f in a separate terminal, it appears as if API and serial system logs are not being created when calling opentrons_execute protocol.py with the default logging configuration.
Finally, it appears as if the robot does not successfully complete the homing motion upon finishing protocol execution.
Steps to reproduce
Execute an otherwise valid protocol (Run App throws no errors) with opentrons_execute on the robot with 3.18.0
from opentrons import protocol_api
metadata = {"apiLevel": "2.0"}

def run(protocol: protocol_api.ProtocolContext):

    # labware
    plate = protocol.load_labware('corning_96_wellplate_360ul_flat', '2')
    plate2 = protocol.load_labware('biorad_96_wellplate_200ul_pcr', '3')

    tiprack = protocol.load_labware('opentrons_96_tiprack_300ul', '1')

    # pipettes
    left_pipette = protocol.load_instrument(
        'p1000_single', 'right', tip_racks=[tiprack] )

    protocol.comment("Loaded")

    left_pipette.transfer(100, plate["A1"], plate2["A2"])

Expected behavior
Protocols that can be executed through the Run App should have the same behavior as when executed via opentrons_execute on the robot's command line.

The message appears to be 1. caused by improper teardown of some of the hardware controller's backing tasks in a way that doesn't really harm much and 2. a bug in the function that prints the error, which turns it into a scarier message.
The home behavior doesn't happen for an unrelated reason - that's implemented in the protoocl session machinery rather than the machinery used for opentrons.execute.

Should we still expect to see journalctl logs created when running with opentrons.execute ?


Catalog-backend- need for optimizing the ingestion loop

In service catalog backend, there is auto-refresh of locations happens every 1.6 min . Even using postgre DB for Production data(around 13K entities) from an organization, It takes around  ~16 min to complete refresh.
Background - Service catalog backend is integrated with our organization 's internal metadata API (Fetching around 13K entities)-following external-integrations approach (https://backstage.io/docs/features/software-catalog/external-integrations)
Issue - This latency will have a great effect in time taken to display an app in backstage catalog, once it is created by scaffolder.
a) on every refresh there is a comparison of existing entities and new entities for all the entities and then entities are split into three groups {toAdd, toUpdate, toIgnore}. Though there will be lot of entities endup in 'toIgnore' - they are updated as well with setRelations()- meaning there is an DB update operation happens for all entities.
https://github.com/backstage/backstage/blob/f531d307c4b9c8fb98f0d4b1b3006dd6023a8961/plugins/catalog-backend/src/catalog/DatabaseEntitiesCatalog.ts#L190-L212
b) batch _size == 100 (having 12K entities) this may slow down things further
When I increase the batch size -5000 (may not be realistic and may lead to race condition) its takes around 10 min to finish - not a significant improvement.

Feature Suggestion

Possible Implementation

Context
To decrease the time to display an app in service catalog, once it is create by scaffolder.

Thank you!
I tried with a fairly large input dataset (Spotify LDAP) on latest master.
First with sqlite in memory:
2020-12-02T20:30:15.674Z catalog info Read 17857 entities from location bootstrap:bootstrap in 8.5s
2020-12-02T20:30:39.361Z catalog info Posting update success markers
2020-12-02T20:30:55.500Z catalog info Wrote 17857 entities from location bootstrap:bootstrap in 39.8s
2020-12-02T20:30:55.502Z catalog info Completed locations refresh in 48.4s

Not too shabby. Then with a local postgres 12.3 running in docker:
2020-12-02T20:34:38.006Z catalog info Read 17857 entities from location bootstrap:bootstrap in 9.9s
...

... I stopped waiting 15 minutes later while the Docker daemon was under very high CPU usage.
This is a regression; when I did the same test with the same source a few months ago the times were comparable between the two databases (although the inmemory one was a bit faster).

If you run the backend with LOG_LEVEL=debug, you will see it process individual batches as they are being written. You'll notice that the batches get very quickly slower and slower the more data that's stored.
2020-12-02T20:51:37.917Z catalog debug Found 100 entities to add, 0 entities to update in 12ms
2020-12-02T20:51:37.927Z catalog debug Found 100 entities to add, 0 entities to update in 22ms
2020-12-02T20:51:37.935Z catalog debug Found 100 entities to add, 0 entities to update in 30ms
2020-12-02T20:51:39.370Z catalog debug Added 100 entities in 1.5s
2020-12-02T20:51:39.371Z catalog debug Considering batch [redacted] (100 entries)
2020-12-02T20:51:40.388Z catalog debug Found 100 entities to add, 0 entities to update in 1.0s
2020-12-02T20:51:40.395Z catalog debug Added 100 entities in 2.5s
2020-12-02T20:51:40.395Z catalog debug Considering batch [redacted] (100 entries)
2020-12-02T20:51:41.467Z catalog debug Found 100 entities to add, 0 entities to update in 1.1s
2020-12-02T20:51:41.474Z catalog debug Added 100 entities in 3.5s
2020-12-02T20:51:41.474Z catalog debug Considering batch [redacted] (100 entries)
2020-12-02T20:51:42.506Z catalog debug Found 100 entities to add, 0 entities to update in 1.0s
2020-12-02T20:51:43.959Z catalog debug Added 100 entities in 3.6s
2020-12-02T20:51:43.959Z catalog debug Considering batch [redacted] (100 entries)
2020-12-02T20:51:46.352Z catalog debug Found 100 entities to add, 0 entities to update in 2.4s
2020-12-02T20:51:46.360Z catalog debug Added 100 entities in 4.9s
2020-12-02T20:51:46.360Z catalog debug Considering batch [redacted] (100 entries)
2020-12-02T20:51:48.747Z catalog debug Found 100 entities to add, 0 entities to update in 2.4s
2020-12-02T20:51:48.753Z catalog debug Added 100 entities in 6.2s
2020-12-02T20:51:48.753Z catalog debug Considering batch [redacted] (100 entries)
2020-12-02T20:51:51.210Z catalog debug Found 100 entities to add, 0 entities to update in 2.5s
2020-12-02T20:51:52.690Z catalog debug Added 100 entities in 6.3s

The logs are there in this form because I've seen this behaviour before and fixed it. I'll look into again why it has resurfaced.

@freben Thanks for checking on the issue.
Was it consistent for subsequent refreshes?
Because the first time it just has to add all the entities, so it is faster for me as well refresh completed below 1 min. But, in the following refreshes it took around 15  min, as entities are compared with old version of entities and then updated.

About using sqlite in-memory for Production - In our organization there is an existing DB  for storing all entity information and an API to fetch entities from. So, Catalog backend DB merely acts as a cache layer in this case.
Do you see any potential reason for not using sqlite for production in this case?

I didn't follow through since even the first run (starting from an empty database) just took too long. I expect the next run would have been almost equally slow!
Current theory is that it's actually reads that are slowing it down. Removing this step (replacing it with an empty array) makes it run to completion pretty quickly.
https://github.com/backstage/backstage/blob/master/plugins/catalog-backend/src/catalog/DatabaseEntitiesCatalog.ts#L285
The core of the issue would then be the query bilder here.
https://github.com/backstage/backstage/blob/master/plugins/catalog-backend/src/database/CommonDatabase.ts#L242
I rewrote it some time ago to use WHERE...IN instead of JOIN. This made sense from a code standpoint but maybe it hurt the pg query planner. Or maybe some other fix around that time removed an index. I'll have to look closer when I get some time to do so.
Using sqlite in prod - sorry that won't work well. Even if it's cache-like here, you will get other ill  effects and other plugins that want actual persistence - including the catalog itself which stores locations that users have registered - will lose their data on each deploy.

Created PR - https://github.com/backstage/backstage/pull/3540
Why?
During location refresh in catalog backend, there is a comparison between old entities and new entities in entityHasChanges() function. In that method, relations in the existing entity from DB (e1) is not removed causing lodash.IsEqual() to always returns false and entities always end up in toUpdate list instead of ideally going to toIgnore list. Due to this, there is a high latency especially if the entities count is large though those entities exist in DB already.
Note :
relations don't exist in incoming entity, but exist only in existing entity in DB
For 13K entities, it took around 60 seconds for first write to DB and successive updates took 1000 seconds.
What?
Removed relations in the existing entity from DB (e1) before performing lodash.IsEqual(), and this helps successive updates to complete within 100 seconds.

I see a few related issues like #3592, #3666 - is this resolved, or still an issue?

Yes, we had some massive speedups through batching and other fixes. And especially given that large parts of the engine that drives refreshes is being rewritten as we speak, we should close this ticket and open a new more targeted one if performance issues happen again.


Further config fixes

Codecov Report

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

@@           Coverage Diff           @@
##           master     #365   +/-   ##
=======================================
  Coverage   64.38%   64.38%
=======================================
  Files          14       14
  Lines       29496    29496
  Branches      139      139
=======================================
  Hits        18990    18990
  Misses      10431    10431
  Partials       75       75

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 078208d...e3ba041. Read the comment docs.


Error 429

I have constantly en error 429 limit exceeded. no matter what I do it gives me the same error. I tried to make a number of API connections lower, but it didn't work

Pretty sure this is duplicate of this issue:
https://github.com/TumblThreeApp/TumblThree/issues/46


main.py always closing

i dont know why but main.py closes at the 3rd step at the proxy

give me the error when the program is closed

, fixed the bug, don't forget to give a star


fix: use commands in environment variables instead of absolute paths

But using this change solves the problem of adaptation to different platforms. You cannot assume that the command exists in the specified directory, especially for the Windows platform.

IMO issuing errors early (executable not found) is way more intuitive (and debuggable) than getting ModuleNotFoundError: No module named '<random-package-name>'. b/c there're still many users who don't use Python (or are new to nvim) but still need python_host_prog support. FWIW simply changing the file path is easier than debugging an unfamiliar language.
Say if a user gets ExecutableNotFoundError first and then ModuleNotFoundError again, then things are very clear - that specific Python distribution used doesn't have corresponding dependencies installed.

I don't have any doubts, but I still think that it would be better to use environment variables, one is more general, it can be cross-platform, and the other is that users should be responsible for their own environment variables, after all, setting environment variables is a basic operation for developers.

I don't have any doubts, but I still think that it would be better to use environment variables, one is more general, it can be cross-platform, and the other is that users should be responsible for their own environment variables, after all, setting environment variables is a basic operation for developers.

Agree with you on this. We shall wait for other reviewers to check this PR, and if most of us are happy with this idea, we can also make more changes and remove several unnecessary conditionals 👍

For versatility, just use executable name is better because we provides cross-platform experience in fact. Approve these changes.

side note: i hard coded the path just to get a faster startup time.


Build Error

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
When I tried to deploy this program on an Android phone using termux, a build error occurred after running for some time after using the npm run setup command. When I looked at the console log, I found that it had requested a file that didn't exist, causing subsequent builds to fail. The following is the error log output at the time of the request:
ERR! install response status 404 Not Found on https://github.com/louislam/node-sqlite3/releases/download/v15.1.6/napi-v6-android-arm-unknown.tar.gz
node-pre-gyp WARN Pre-built binaries not installable for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v115 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp WARN Hit error response status 404 Not Found on https://github.com/louislam/node-sqlite3/releases/download/v15.1.6/napi-v6-android-arm-unknown.tar.gz

In order to avoid inaccurate expression in the translation process, the following is the original Chinese text:
当我在使用termux在安卓手机上尝试部署此程序时，使用npm run setup命令之后，运行一段时间之后发生构建错误。我在查看控制台日志时，发现它请求了一个不存在的文件，导致后续的构建出错。
📝 Error Message(s) or Log
This is the full log
🐻 Uptime-Kuma Version
latest
💻 Operating System and Arch
Android10 Termux
🌐 Browser
No need to provide
🐋 Docker Version
No response
🟩 NodeJS Version
v20.2.0

It is not official supported.
It is just for fun.
You can read this: https://github.com/louislam/uptime-kuma/issues/423

Thanks for your help, now it can run successfully.

 and? It's not working

@Murchik0v Termux is an experiment and might not be working correctly.
It is not official supported, It is just for fun.
You should read this: https://github.com/louislam/uptime-kuma/issues/423


Classes cannot be used as InputJsonValue

Bug description
We use NestJS, so we have DTOs defined as classes and used in request bodies.
When we put a class value to a json field, the follow typescript error occurs
Type 'MyDto' is not assignable to type 'InputJsonValue'.
  Type 'MyDto' is not assignable to type 'InputJsonObject'.
    Index signature for type 'string' is missing in type 'MyDto'.ts(2322)

How to reproduce
Schema:
model Model {
  id @id
  field Json
}

Code:
class MyDto {
  something: string
}
const dto = new MyDto();
prisma.model.create({
  data: {
    id: "id",
    field: dto // <-- error here
/*
Type 'MyDto' is not assignable to type 'InputJsonValue'.
  Type 'MyDto' is not assignable to type 'InputJsonObject'.
    Index signature for type 'string' is missing in type 'MyDto'.ts(2322)
*/
  }
})

Expected behavior
Any objects can be put inside JSON field
Prisma information
See above
Environment & setup

OS: MacOS 12.1
Database: MySQL 8.0
Node.js version: v16.13.0

Prisma Version
prisma                  : 3.8.1
@prisma/client          : 3.8.1
Current platform        : darwin
Query Engine (Node-API) : libquery-engine 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine    : introspection-core 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary           : prisma-fmt 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f (at node_modules/@prisma/engines/prisma-fmt-darwin)
Default Engines Hash    : 34df67547cf5598f5a6cd3eb45f14ee70c3fb86f
Studio                  : 0.452.0

Hmm, I'm not so sure this should be allowed because not all classes can be serialized (e.g. classes with methods).
I did just test it in JS and you can stringify classes:
class A {}
JSON.stringify(new A()) // {}

Any reason you believe classes should be allowed?

After some research, I think instanceToPlain from class-transformer should be the way to go. It transform a class to a plain JS object (should be similar to serializing then deserializing it)
export declare function instanceToPlain<T>(object: T, options?: ClassTransformOptions): Record<string, any>;

I believed classes should be allowed as inputs simply because when we use DTO classes we are defining the object shape, and it is similar to an interface.
You're right that methods cannot be serialized therefore should not be used as an object.
I will close this issue as this is not a bug. Perhaps there will be some improvements to help people using DTOs in prisma but I'm currently fine with the instanceToPlain approach.
Thanks for your time!


Add jquery dependency to package.json

The new require('jquery') is great for using browserify, but it doesn't work because jquery is not listed as a dependency in package.json.

Comments from those who know about this?

Correct me if I'm wrong but if you add jquery: ^2.0, backbone: ^1.2 in your projects package.json the import should work

Eh? If your main app depends on jQuery, browserify should pick it up with no issues.

In my case my app does not depend on jquery, but it does depend on Backbone. I have no require('jquery') statements in my code, but if I pull in require('backbone') it fails in browserify because it can't find jquery.
In my opinion if the backbone code includes require('jquery') then it should be included as a dependency. I mean, it is a dependency, even if it is an optional one.
But I guess @megawac's solution does work. It just seems to go against how dependencies work - should I be declaring dependencies of sub-modules in my app?

Ok, seeing the discussion in #3421 now. I guess that makes the most sense for now.

If you're not using jQuery in your app, you have to tell Browserify to exclude it. See the wiki.
Sorry if any confusion. Is there a way we can make this clearer to folks?


updates for websummit

This update is to the new content that @kacole2 has created for Web Summit.

will only need to make a few changes but LGTM


Added support for deviceStatuses page

This adds support for the deviceStatuses page from organisation,
I copied the getOrganizationDeviceStatuses method and changed the name, documentation URL (FYI, none of the old ones work because you need to add "/v0" before the "#") and the resource

Thanks, but sorry this is not correct, as the getOrganizationDeviceStatuses function is already part of the organizations tag, and not devices. Furthermore, it does not use or require pagination.
As for the documentation URLs, those will be changed as part of the April release next week. (General updates are pushed around the first full week of every month).


Will you update for 2024?

Hey @annaken
yes, absolutely! I'm waiting for the IFSC to post the proper schedules. I'm parsing the event pages to obtain the info, but there's nothing yet. See https://www.ifsc-climbing.org/index.php/component/ifsc/?view=event&WetId=1353 for example.
This should hopefully change soon.

Amazing, thank you!

Thanks! Yeah, I'm aware. The problem with these is that they're not very detailed. I want the calendar to show proper times (not just dates) and which kind of event is being held (discipline, semis, finals, etc).
I might add some functionality later that shows the date range if nothing else is available, and then the details once they're published.

Gotcha. Thanks again!

 Thanks for your support ❤

❤

Perfect! Added to my calendar- thank you 😄


For dataset having proto schema , the raw format schema is not visible

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Navigate to any dataset(which have protobuff schema)  and schema tab
Click on Raw button on left
Page redirect to to a black page
See error in console

Screenshots

Closed via https://github.com/linkedin/datahub/pull/2566


spark-mongodb (0.8.0) connector is not compatible with spark-sql 1.3

spark-mongodb connector is not compatible with spark-sql 1.3.0.
Gives me the following error:
java.lang.IncompatibleClassChangeError: class com.stratio.deep.mongodb.MongodbRelation has interface org.apache.spark.sql.sources.PrunedFilteredScan as super class.
It works with spark-sql 1.2.
Is there a fix available for the same?

Hi!
We are working on it. By the moment, you can download the github project and use spark-mongodb-0.8.0.jar generated in /target  after you execute "mvn clean install" in the project root directory. This should work.
Anyway, will be in central maven very soon.
Thanks!

Yes, that got resolved, but has resulted in another:
java.lang.NoClassDefFoundError: com/mongodb/casbah/Imports
at com.stratio.deep.mongodb.partitioner.MongodbPartitioner.isShardedCollection(MongodbPartitioner.scala:62).
Seems like incompatibility with com.stratio.deep:deep-mongodb:0.7.0:jar.
I will try downloading this project as well from github and build it.
But do let me know if there is a solution already available.

That worked, Thank you.

It's already updated in central maven!
Thank you very much for your feedback!


sgx target_env

How the #[cfg(target_env = "sgx")] in libstd, libc etc. affects this SDK?
is this the dream of SGX being integrated into the SDK?
will this SDK someday leverage that to natively compile rust code into sgx?
i.e. https://github.com/rust-lang/rust/blob/f843ad60ef5a78d8b4da85c5007d3ecac229f1a8/src/libstd/sys/mod.rs#L44

I'm aware of that. Currently, Fortanix's solution occupies that.
I have several discussions with jethrob on github, where crates are merging commits to support their solutions. Currently the rust-lang/rust can checks target_env = "sgx" and target_vendor = "fortanix" at the same time to be compatible with our solutions.
Fortanix and my solution differ from the foundation -- whether or not use the Intel's codes, and, whether or not use the current design of libstd. The current design of libstd assumes that every target platform should provide time/net/env/thread/process/fs but sgx does not have that. So current targets in libstd only provide runtime panic. This happens when you write "std::fs::File::open" in the wasm target as well -- it panics but compiles. This is exactly what I'm trying to prevent. So Fortanix and I choose different ways for the tradeoff between usability and safety/security. Their codes are easy to use -- but hides too much info to become trustworthy. See my gif below until it ends.
https://dingelish.com/record.gif
The rustc compile, or the combination of "cargo + libstd" has to change to help embedded systems generating compile-time warning/faults instead of making unsupported APIs runtime panics. I proposed an pre-rfc for it at here. But I don't have time for it now.

Yes, I know about the different assumptions which is why we chose your sdk. For now I prefer it to be intel bindings than pure rust, at least as it's not a big well maintained well reviewed project.
I was just cirious if you could leverage their work to help prevent the need to fork dependencies some day.
Thanks for your work!

:-)
I do have the plan to reorg this and push this to the upstream, as long as I can generate compile-time warnings for unsupported functions!


Use experimental ASM 7 mode

For full Java 11 compatibility

Unfortunately this is still not enough, because cglib uses ASM6, so a lot of our Spock tests fail on Java 11. We won't be able to announce compatibility in this state. It's a bit upsetting that such major changes come so late, when it's supposed to be in "rampdown".

On the upside at least we found out about this issue before announcing support for Java 11. Sounds like it will have to wait until 5.0, doesn't it?

Yes, I'll ping the Spock team about a fix.

Out of curiosity, gradle is not affected by https://github.com/cglib/cglib/pull/130 ?

@oehme I changed the milestone to 5.0 RC1.

As an aside, ASM 6.2.1 has been released https://mail.ow2.org/wws/arc/asm/2018-08/msg00000.html


New feature: bleeding effect

When triggered, target entity will get hurt <N> times, dealt <X> damage (ignore any other factor, like realdamage) each hurt, with interval <I>.
The effect can be applied to both self and other entities.

用属性插件做这种东西比较好


Register nick and perform ident at login

Register nick and perform ident at login so bot can be auto op'd
https://github.com/TeamPeggle/ppp-helpdesk/blob/ppp-helpdesk/docs/Configuration.md

It does this now


Add note for initial releases

Thank you for this PR.
Wonder how it may can be combined with the notes highlighted in the Configuration section: https://github.com/mikepenz/release-changelog-builder-action#configuration

In the rarest cases there would be no checkout, as it would be also required to retrieve the configuration.json

@mikepenz please have another look

I'd actually propose to make option 2 the preferred one, as it is anyways suggested to checkout so the action has access to the configuration.json :)

I'm not quite sure what you mean. I didn't mean to provide preferences for either option, I was just listing that I see 2 different approaches. Did you mean to omit the option to use a tag like v0.0.1 ?


Scroll behavior not working when hash is UTF-8 string

Versions

nuxt: 2.15.8
node: v16.15.1

Reproduction
<nuxt-link to="#按紙質效果選購">按紙質效果選購</nuxt-link>

Steps to reproduce

Have a link to the hash 按紙質效果選購
Click on it, it will be encoded to #%E6%8C%89%E7%B4%99%E8%B3%AA%E6%95%88%E6%9E%9C%E9%81%B8%E8%B3%BC
Scroll behavior is not working due to encoded URI.

What is Expected?
Scroll behavior is working as usual
What is actually happening?
No scroll at all.
Solution
https://github.com/nuxt/nuxt.js/blob/4ae7c089cb12fae5ed128ffad2c308da8e44c22e/packages/vue-app/template/router.scrollBehavior.js#L63
change let hash = to.hash to let hash = decodeURIComponent(to.hash)

We are approaching the Nuxt 2 EOL date (June 30, 2024) - see this article for more information. This is advance warning that I'm going to close this issue then, as it's currently marked as a Nuxt 2 related bug.
If it's a critical or security issue, please do comment and let me know, in case it is possible to address it before the EOL date.
If it's a an issue you think is relevant for Nuxt 3, please feel free to open a fresh issue (or just comment here so I can update labels, etc.). 🙏
Thank you for your understanding and for your contribution to Nuxt. ❤

It's the day, at last. Nuxt 2 is now marked end-of-life.
My apologies we never got round to resolving this bug in the 2.x branch, but we do have to draw a line somewhere. Again, if you think this is still relevant for Nuxt going forward, please feel free to open a fresh issue (or just comment here so I can update labels, etc.). 🙏


fly: enter should honor uid/gid/supp-gid

Following up on https://github.com/coreos/rkt/pull/3377#discussion_r88637292. Whenever we enter into a running fly chroot, we should honor user and group details as specified when running the app.

It appears that the user and group stored in the stage2 manifest are not correct.
After this:
# /usr/bin/rkt --insecure-options=image run --interactive --set-env HOME=/home/guestsi quay.io/coreos/alpine-sh --user=511 --group=511 --exec /bin/sh

The manifest is as follows:
# cat /var/lib/rkt/pods/run/a58f3371-8aca-44bd-b495-73ecb9f42d81/stage1/rootfs/opt/stage2/alpine-sh/manifest
{"acKind":"ImageManifest","acVersion":"0.6.1","name":"quay.io/coreos/alpine-sh","labels":[{"name":"version","value":"latest"},{"name":"arch","value":"amd64"},{"name":"os","value":"linux"}],"app":{"exec":["/bin/sh","-c","/bin/sh"],"user":"root","group":"root","workingDirectory":"/","environment":[{"name":"PATH","value":"/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"}]}}

I expected to see user 511 (or guestsi, which is the username on the host), and group 511.
Same problem with stage1-fly (which is of course what I'm really interested in):
# /usr/bin/rkt --insecure-options=image run --interactive --set-env HOME=/home/guestsi quay.io/coreos/alpine-sh --user=511 --group=511 --stage1-name=coreos.com/rkt/stage1-fly:1.26.0 --exec /bin/sh

Results in:
invention# cat /var/lib/rkt/pods/run/a5c15ff9-ac99-4724-a15b-fbc461242fb4/stage1/rootfs/opt/stage2/alpine-sh/manifest
{"acKind":"ImageManifest","acVersion":"0.6.1","name":"quay.io/coreos/alpine-sh","labels":[{"name":"version","value":"latest"},{"name":"arch","value":"amd64"},{"name":"os","value":"linux"}],"app":{"exec":["/bin/sh","-c","/bin/sh"],"user":"root","group":"root","workingDirectory":"/","environment":[{"name":"PATH","value":"/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"}]}}

Is this a bug, or am I looking in the wrong place?
rkt Version: 1.26.0
appc Version: 0.8.10
Go Version: go1.7.4
Go OS/Arch: linux/amd64
Features: -TPM +SDJOURNAL
--
Linux 4.4.65-1.el7.elrepo.x86_64 x86_64
--
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"
--
systemd 219
+PAM +AUDIT +SELINUX +IMA -APPARMOR +SMACK +SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT +GNUTLS +ACL +XZ -LZ4 -SECCOMP +BLKID +ELFUTILS +KMOD +IDN

Ah, I discovered I was looking in the wrong place.  I found the manifest in a file called pod:
invention# cat /var/lib/rkt/pods/run/f59ce510-057e-4b40-ade0-42a561be77ac/pod
{"acVersion":"1.26.0+git5f42e6bb-dirty","acKind":"PodManifest","apps":[{"name":"alpine-sh","image":{"name":"quay.io/coreos/alpine-sh","id":"sha512-2222d0a867084bc1c6f47f86982867dcc013a1bcfab15128a1065108808fc998","labels":[{"name":"version","value":"latest"},{"name":"arch","value":"amd64"},{"name":"os","value":"linux"}]},"app":{"exec":["/bin/sh"],"user":"511","group":"511","workingDirectory":"/","environment":[{"name":"PATH","value":"/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"},{"name":"HOME","value":"/home/guestsi"}]}}],"volumes":null,"isolators":null,"annotations":[{"name":"coreos.com/rkt/stage1/mutable","value":"false"}],"ports":[]}

Now I found it, I can see I hadn't read the Stage1 ACI implementor's guide carefully enough.  It is indeed documented clearly.  Sorry about that.
Making progress ...

For reference, the manifest in the first comment is the image one (describing app configuration as defined at buildtime by the entity who generated the image) while the one in the followups is the app section in the runtime pod manifest (describing app configuration as defined at runtime by the entity running the pod).