Add $header modifier support

I think it makes sense to be able to target requests depending on the HTTP headers they have.
Basic idea is to be able to write something like this: ||example.org^$header=/regex/ where regex matches headername: value string.
We'll need to come up with a normal syntax spec for that before implementing it.

[ ] Syntax spec
[ ] Implementation
[ ] Update Knowledge base
[ ] Port to tsurlfilter

uBO (https://github.com/uBlockOrigin/uBlock-issues/wiki/Static-filter-syntax#header) and ABP (https://help.eyeo.com/adblockplus/how-to-write-filters#header-filter) already have such a modifier. So it seems like a rule converter update is required as well.
Here is the proposed spec (basically, it's an adapted uBO's one):
Basic modifiers
header
The $header modifier allows matching the HTTP response having a specific header with a specific value.
Syntax
$header "=" h_name [":" h_value]
h_value = string / regex

where

h_name — an HTTP header name. It is matched case insesitively.
h_value — the HTTP header value matching expression, and may be one of the following:

string — a sequence of characters. It is matched against the header value lexicographically.
regex — a regular expression, starts and ends with /. The pattern works the same way as in the basic URL rules, but the characters /, $, , and | must be escaped with \.

The modifier ":" h_value part may be omitted. In that case the modifier matches the header name only.

Compatibility with different versions of AdGuard. Rules with the $header modifier are supported by AdGuard for Windows, Mac, and Android, running CoreLibs version 1.11 or later.

Examples
||example.com^$header=set-cookie:foo — blocks the request which response has the Set-Cookie header with value matching foo literally.
||example.com^$header=set-cookie — blocks the request which response has a Set-Cookie header with any value.
@@||example.com^$header=set-cookie:/foo\, bar\$/ — unblocks the request which response has the Set-Cookie header with value matching the foo, bar$ regular expression.
@@||example.com^$header=set-cookie — blocks the request which response has a Set-Cookie header with any value.

Works for me

@ameshkov
What about extending matching ability of  $header modifier to HTTP request header?
https://listauthorschat.slack.com/archives/C010N14G4TF/p1662830803562149

@piquark6046 generally, I am not against that, but it should be implemented in a safe manner (i.e. like we do with the set-constant scriptlet).
Could you please come up with a few examples where we would need to change the header instead of removing it?

https://github.com/AdguardTeam/CoreLibs/issues/1433#issuecomment-1243361370

Could you please come up with a few examples where we would need to change the header instead of removing it?

How many you need to prove this issue?

Well, as many as possible:)
@AdguardTeam/filters-maintainers do you have any examples too?

If I remember correctly, we had some issues, which could be resolved by changing header(ads=enabled/disabled parameter).
May be it can be useful for anti-tracking purpose, when a parameter cannot be removed.

Well I need concrete examples in order to come up with the implementation details.
I'd like to avoid adding a modifier that would allow setting an arbitrary value as in this case we'll have to limit it's usage to AdGuard filters only.


CO - Production Error

Problem Statement
submission error over the weekend
What you need to know
file name was 0ff8a2e4-4cb2-4e44-b712-af687c06a7b7
Acceptance Criteria
resubmission and confirmation of resubmission, will confirm submission with state

Looks like it is resent sucessfully!


[AutoDiff] Implement AnyDerivative.

Implement AnyDerivative, a type-erased derivative.
AnyDerivative forwards its operations to an arbitrary underlying base
derivative conforming to Differentiable and AdditiveArithmetic, hiding
the specifics of the underlying derivative.
Add tests.

@swift-ci Please test tensorflow


vim.error: Vim:E315: ml_get: invalid lnum: 18

Describe the bug
I encounter an issue https://github.com/Yggdroot/LeaderF/issues/725
To Reproduce
Detailed steps to reproduce the behavior:
suppose there is a file https://github.com/open-mmlab/mmdetection/blob/master/tools/train.py under current directory(~).
suppose there is another file a.txt at the same directory.
a.txt
docs/tutorials/data_pipeline.md
resources/data_pipeline.png
mmdet/datasets/pipelines/compose.py
mmdet/datasets/pipelines/loading.py
mmdet/datasets/pipelines/__init__.py
mmdet/datasets/pipelines/formating.py
mmdet/datasets/pipelines/transforms.py
mmdet/datasets/pipelines/instaboost.py
mmdet/datasets/pipelines/auto_augment.py
mmdet/datasets/pipelines/test_time_aug.py
mmdet/datasets/samplers/__init__.py
configs/_base_/datasets/cityscapes_instance.py
mmdet/datasets/deepfashion.py
configs/_base_/datasets/deepfashion.py
configs/_base_/datasets/cityscapes_detection.py
mmdet/datasets/samplers/distributed_sampler.py
configs/guided_anchoring/ga_retinanet_r101_caffe_fpn_mstrain_2x.py

cd && git clone https://github.com/tmhedberg/SimpylFold.git

vim train.py -Nu <(cat <<'EOF'
    filetype plugin on
    set rtp-=~/.vim
    set rtp-=~/.vim/after
    set rtp^=~/SimpylFold
EOF
)

execute the following commands in cmdline-mode.

:sp a.txt
:py3 import vim
:py3 b = vim.current.buffer
:py3 aaa = b[:]
:hide
:py3 b[:] = aaa

the error occurs

PS. The main job SimpylFold plugin does is
setlocal foldexpr=SimpylFold#FoldExpr(v:lnum)
setlocal foldmethod=expr

Expected behavior
no error
Screenshots
Environment (please complete the following information):

Vim version  8.2.1891
OS: [e.g. Ubuntu 18.04, Windows 10 1809, macOS 10.14]
Terminal: [e.g. GNOME Terminal, mintty, iTerm2, tmux, GNU screen] (Use GUI if you use the GUI.)

I reduced the issue to this (no third-party plugin or files required):
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    set fdm=expr fde=Fde(v:lnum)
    b:regex = '^'
    fu Fde(lnum)
        let ld = [{}]
        let lines = bufnr('%')->getbufline(1, '$')
        let was_import = 0
        for lnum in range(1, len(lines))
            let line = lines[lnum]
            call add(ld, {'a': b:regex})
            let ld[lnum].foldexpr = was_import ? 1 : '>1'
            let was_import = 1
        endfor
        return ld[a:lnum].foldexpr
    endfu
    setline(1, repeat([''], 15) + repeat(['from'], 3))
    repeat(['x'], 17)->writefile('/tmp/a.txt')
    sp /tmp/a.txt
    py3 import vim
    py3 b = vim.current.buffer
    py3 aaa = b[:]
    hide
    py3 b[:] = aaa
EOF
)

Which gives this stack trace:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
vim.error: Vim:E315: ml_get: invalid lnum: 18

Backtrace.

Interestingly enough, no internal error is raised when the folding function is refactored in Vim9 script:
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    b:regex = '^'
    set fdm=expr fde=Fde(v:lnum)
    set debug=throw
    def Fde(lnum: any): any
        var ld = [{}]
        var lines = bufnr('%')->getbufline(1, '$')
        var was_import = 0
        for _lnum in [1, 18]
            var line = lines[_lnum]
            call add(ld, {'a': b:regex})
            ld[_lnum].foldexpr = was_import ? 1 : '>1'
            was_import = 1
        endfor
        return ld[lnum].foldexpr
    enddef
    setline(1, repeat([''], 15) + repeat(['from'], 3))
    repeat(['x'], 17)->writefile('/tmp/a.txt')
    sp /tmp/a.txt
    py3 import vim
    py3 b = vim.current.buffer
    py3 aaa = b[:]
    hide
    py3 b[:] = aaa
EOF
)

Instead, it correctly throws regular errors (visible if 'debug' is set to throw):
E684: list index out of range: 18
...
E121: Undefined variable: b:regex
...

I reduced the issue to this (no third-party plugin or files required):
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    set fdm=expr fde=Fde(v:lnum)
    b:regex = '^'
    fu Fde(lnum)
        let ld = [{}]
        let lines = bufnr('%')->getbufline(1, '$')
        let was_import = 0
        for lnum in range(1, len(lines))
            let line = lines[lnum]
            call add(ld, {'a': b:regex})
            let ld[lnum].foldexpr = was_import ? 1 : '>1'
            let was_import = 1
        endfor
        return ld[a:lnum].foldexpr
    endfu
    setline(1, repeat([''], 15) + repeat(['from'], 3))
    repeat(['x'], 17)->writefile('/tmp/a.txt')
    sp /tmp/a.txt
    py3 import vim
    py3 b = vim.current.buffer
    py3 aaa = b[:]
    hide
    py3 b[:] = aaa
EOF
)

Which gives this stack trace:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
vim.error: Vim:E315: ml_get: invalid lnum: 18

Backtrace.

Interestingly enough, no internal error is raised when the folding function is refactored in Vim9 script:
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    b:regex = '^'
    set fdm=expr fde=Fde(v:lnum)
    set debug=throw
    def Fde(lnum: any): any
        var ld = [{}]
        var lines = bufnr('%')->getbufline(1, '$')
        var was_import = 0
        for _lnum in [1, 18]
            var line = lines[_lnum]
            call add(ld, {'a': b:regex})
            ld[_lnum].foldexpr = was_import ? 1 : '>1'
            was_import = 1
        endfor
        return ld[lnum].foldexpr
    enddef
    setline(1, repeat([''], 15) + repeat(['from'], 3))
    repeat(['x'], 17)->writefile('/tmp/a.txt')
    sp /tmp/a.txt
    py3 import vim
    py3 b = vim.current.buffer
    py3 aaa = b[:]
    hide
    py3 b[:] = aaa
EOF
)

Instead, it correctly throws regular errors (visible if 'debug' is set to throw):
E684: list index out of range: 18
...
E121: Undefined variable: b:regex
...

I reduced the issue to this (no third-party plugin or files required):
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    set fdm=expr fde=Fde(v:lnum)
    b:regex = '^'
    fu Fde(lnum)
        let ld = [{}]
        let lines = bufnr('%')->getbufline(1, '$')
        let was_import = 0
        for lnum in range(1, len(lines))
            let line = lines[lnum]
            call add(ld, {'a': b:regex})
            let ld[lnum].foldexpr = was_import ? 1 : '>1'
            let was_import = 1
        endfor
        return ld[a:lnum].foldexpr
    endfu
    setline(1, repeat([''], 15) + repeat(['from'], 3))
    repeat(['x'], 17)->writefile('/tmp/a.txt')
    sp /tmp/a.txt
    py3 import vim
    py3 b = vim.current.buffer
    py3 aaa = b[:]
    hide
    py3 b[:] = aaa
EOF
)

Which gives this stack trace:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
vim.error: Vim:E315: ml_get: invalid lnum: 18

Thanks for figuring this out.  I can use it for a test.
It happens because Python changes a hidden buffer using the current
window, which has side effects.  I'll make a simple solution that fixes
this case, but there might be other problems.
--
hundred-and-one symptoms of being an internet addict:
91. It's Saturday afternoon in the middle of May and you
are on computer.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I reduced the issue to this (no third-party plugin or files required):
vim -Nu NONE -S <(cat <<'EOF'
    vim9
    set fdm=expr fde=Fde(v:lnum)
    b:regex = '^'
    fu Fde(lnum)
        let ld = [{}]
        let lines = bufnr('%')->getbufline(1, '$')
        let was_import = 0
        for lnum in range(1, len(lines))
            let line = lines[lnum]
            call add(ld, {'a': b:regex})
            let ld[lnum].foldexpr = was_import ? 1 : '>1'
            let was_import = 1
        endfor
        return ld[a:lnum].foldexpr
    endfu
    setline(1, repeat([''], 15) + repeat(['from'], 3))
    repeat(['x'], 17)->writefile('/tmp/a.txt')
    sp /tmp/a.txt
    py3 import vim
    py3 b = vim.current.buffer
    py3 aaa = b[:]
    hide
    py3 b[:] = aaa
EOF
)

Which gives this stack trace:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
vim.error: Vim:E315: ml_get: invalid lnum: 18

Thanks for figuring this out.  I can use it for a test.
It happens because Python changes a hidden buffer using the current
window, which has side effects.  I'll make a simple solution that fixes
this case, but there might be other problems.
--
hundred-and-one symptoms of being an internet addict:
91. It's Saturday afternoon in the middle of May and you
are on computer.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

@brammool thanks a lot!

@brammool thanks a lot!


Gradle build failed

Expected Behavior

Current Behavior

Context

Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

Please follow our issue template.

Sorry about that. I didn't realize it submitted. I had second thoughts on submitting the issue as I realized it might be a better fit in the IntelliJ IDEA community. The issue is the following:
I was looking for advice on the ideal way to resolve the Project Build Failure issue using Gradle within the IntelliJ environment. The error message I get is:
Cause: startup failed:
initialization script 'C:\Users\JenniferD'Souza\AppData\Local\Temp\ijinit.gradle': 5: unexpected char: '\' @ line 5, column 45.
le('C:\\Users\\JenniferD'Souza\\AppData\
                          ^

1 error

I understand the single-quote (apostrophe) character in the directory name is breaking the build file path. Is there a way to just keep the apostrophe and try to make the build work correctly which could involve escaping the apostrophe with a backslash character in any paths?
Relevant to the issue, please find the error screenshot attached

Many thanks for any suggestions for resolving this.

Thanks for the follow-up :) This will have to be fixed on the IntelliJ side. IntelliJ generates an init script that refers to its own installation path and it'll need to quote the single-quote character so it becomes a valid String literal.

Is there a fix for this

I am also running into this issue cause my user is a money sign. $


Implement per host HTTP client

The problem
StyxHttpClient is a HTTP client for communicating to backend service apps. It load balances, retries, in addition to many other things. Some complexity has crept in over the time. To reduce this complexity it makes sense to refactor the act of sending a HTTP request out into a separate "per-host" HTTP client.
As an added benefit, having a separate "per-host" HTTP client allows us to open it up in the Styx routing object model. That is, the routing configuration would be able to route to individual hosts, instead of just applications (which are clusters of individual hosts). This would be a separate increment though.
Detailed description
Implement a new HostHttpClient with following functionality:

Connection pooling
Release content buffers on error
Run the HttpRequestOperation, which is to

Add/remove relevant Netty handlers
Write HTTP request to origin
Request/response logging
Finalise response processing - return borrowed connection back to pool, or close the connection in face of failure.

This is, the HostHttpClient will encapsulate Transport and HttpRequestOperation functionality.
The Styx HTTP Client, or the backend service client will then be responsible of:

Apply rewrites
Apply origins restriction feature
Sticky session handling
Run the load balancer
Retry handling
Metrics recording
Adds response ID

Acceptance criteria

HostHttpClient implements the HttpClient interface
sendRequest returns an observable of HttpResponse
A subscription to the returned observable triggers the request to be sent.
Unsubscribing from the observable triggers request cancellation.
When request is cancelled, the underlying TCP connection is terminated.
When response (and its body) is fully consumed, the client returns the underlying TCP connection back to the connection pool.
if errors occur, the underlying TCP connection is returned back to pool.
Client supports optional *flow-control to prevent hogging up memory.

Was implemented by #88 and #96.


Add an option to ignore postcss warnings

Description
I want to disable unnecessary postcss warning in the console, like this:
[vite:css] C:\projects\my-site\src\styles\mixins.pcss is empty

Suggested solution
I propose to add a new option similar to onwarn in rollup, in which it will be possible to filter postcss warnings. I think the warnings filter should be used here
Alternative
Put this comment into mixins.pcss file: /* The file is empty now */
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

This is possible by customLogger option.
import { createLogger, defineConfig } from 'vite';

const logger = createLogger();
const originalWarning = logger.warn;
logger.warn = (msg, options) => {
  if (msg.includes('vite:css') && msg.includes(' is empty')) return;
  originalWarning(msg, options);
};

export default defineConfig({
  customLogger: logger,
});

https://stackblitz.com/edit/vitejs-vite-mjupek?file=vite.config.js&terminal=dev

I feel like customLogger here is the best option as we would have a single point to handle all warnings of a Vite app. It looks like the option isn't documented though, so we could also do that.


Optionally call AMD define() to register module

This issue has come up before, in Issue #287. This pull request is different in these two aspects:

The patch itself is different, doing it optionally as part of the global registration branch.
I can provide more context on why this is useful to do.

Patch
It is done as part of the "set global" branch, and it does not interfere with that branch, just optionally calls define if it is available. So this should not cause issues with code that still wants to use underscore as a global even when an AMD loader is on the page.
Context
In the previous pull/issue there were concerns that a global define not being part of too many useful JS runtimes.
define() is part of the AMD API which is being adopted by a few toolkits like Dojo, MooTools and EmbedJS. jQuery 1.7 will include a very similar opt-in define() call as specified in this patch. define()-based code can run in Node via the requirejs package, and define() can be used in jetpack/add-on SDK add-ons for Firefox. Firebug 1.8+ uses define().
More reasoning and context for AMD.

I would like to see this pulled in as well. AMD is more about namespace protection than asynchronous loading.

If Underscore is already being exported globally in the same conditional body ... then what use case will this patch enable, where Underscore isn't already usable?

To shine some light on why this would be really cool, here is how I currently use underscore in my AMD-based projects:
myfile.js:
require(['underscore'], function() {
// Do stuff using the global "" reference, hoping no other part of the system has defined "" globally
});
But with this pull request, AMD loaders can use the "define" factory method to create an underscore instance that isn't globally available:
myfile.js:
require(['underscore-with-amd'], function() {
// Do stuff using the function-local "" reference
});
You can of course, like you say, inject underscore into the same conditional body using scripts, but then you'd need to include the same source in multiple places if you use underscore in different namespaces and you would need a custom build system to assemble your application (vs. requirejs that just reads your dependency tree and can both load files individually in the browser, making debugging super-simple, as well as create concatenated single-file apps for production).

In reply to the question from @jashkenas about use cases:
The original patch in this pull request was to minimize the impact on behavior for underscore, but the main use case is to allow not bleeding globals and being able to build contained libraries that use underscore that have their own version of underscore that does not interfere with the global on the page.
This is possible with AMD because the requirejs optimizer can be used to rename the AMD calls to namespace-specific items, or if a single file library is being generated and even the AMD API should not be globally available since there will be no dynamic loading, the almond AMD API shim can be used to completely encapsulate the library without using an AMD loader.
Using _.noConflict does not give 100% coverage in this case -- if the library dynamically loads underscore, in IE, the script onload callback (the first opportunity for the library to call noConflict) can actually fire after other dynamic scripts load, and if one of those dynamically loaded scripts wants a different version of underscore, they could bind to the wrong version, since IE can execute a few scripts before calling the first script's onload handler.
Furthermore, by using string names for dependencies as AMD does, it allows swapping out implementations without the sub-library having to export a specific global name. This is really useful for creating mocks for testing, and in the case of libraries that use jQuery, they can swap in a lighter library that exports jQuery's API/behavior without having to code for a bunch of specific global names.
The original patch I put up required the library developer that wants this case to manually call _.noConflict() in their code if they want these use cases. Not the ideal for the library developer, and it had the noConflict edge case, but the patch caused the minimal amount of behavior change for underscore. In other libraries, having a global particularly for their unit tests was important.
However for underscore, since a new test page that uses an AMD loader is not being included, this is not an issue. I have tested it in an AMD project, and I am willing to own any issues reported to underscore with this registration.
It is actually best if underscore does not export a global if define() is available.
Therefore, I have updated this pull request to not export a global if define() is available. However, if you are uncomfortable with the latest version of the pull request, I can revert it back to the original one.

Thanks for the fix -- putting it in its own conditional makes much more sense.

Should
define('underscore', function() {
  return _;
});

because the location of the file doesn't match the moduleId. I use alias in require configuration right now.
Am I missing something?
instead be
define(function() {
  return _;
});

Having that "underscore" there means that I can't do stuff like
define(["vendors/underscore"], function (_) {});

@jrburke
Read the "Unserscore is so important we will use named define(" monologue at https://github.com/jrburke/requirejs/wiki/Updating-existing-libraries
Still did not see a good-looking example of loading underscore from a file-name that is not "[root]/underscore[.js]"
Is there a good example somewhere on how to require underscore that is actually located in a file "js/libs/underscore-1.2.3.min.js" and getting a ref for it in require( callback?
do I have to double-require it?
require(["js/libs/underscore-1.2.3.min.js"], function(_){
    // Whoa! WTF? underscore is null!?
    require(['underscore','myapp'], function(_, myapp){
        // finally, do usefull things with _
    })
})

frankly, i would rather see you go back to anonymous define in unserscore and say to users:
// You want consistent, named module for 'underscore'?  insert the following into your "app_loader.js"
define(
    'underscore'
    , ["js/libs/underscore-1.2.3.min.js"]
    , function(_) {return _}
)

You get the preloading bonus and it works as expected.

@dvdotsenko all AMD loaders allow mapping a module ID to a partial path, usually the configuration is called 'paths', so to do what you want:
requirejs.config({
    paths:
        underscore: 'js/libs/underscore-1.2.3.min'
    }
});

require(['underscore'], function () {});

Since underscore is used by other higher-level modules, like backbone, a common dependency name needs to be used to communicate a common dependency on underscore, and it makes sense to call that dependency 'underscore'. The paths config gives a way to do the mapping to a specific URL you want to use for that dependency.

@jrburke
Sorry to bother. Never mind the comment above. Switched to curl.js and underscore.js just started working with custom paths in config. All good on underscore.js front.

@jrburke
Yes, indeed. path helps. My, albeit subjective, two issues were:

I was trying to pass config to inline require deeper in the code (not in index.html) and, somehow, that config did not stick for require( nested deeper in that tree. Deeper calls resolved 'jquery' to a [root]/jquery[.js]. Learned my lesson - need to use global config.
Another edge issue - I wanted to have 'jquery' and 'jquery_other' (for funky CDN > local failover at first, then, to play with different versions of jQuery because some of the plugins i use were born before the .args .prop split)

I admit, my use-cases are not mainstream, but was seriously disenchanted with how brittle RequireJS was with modules that define themselved named inside the module. I am happy to inline my own named define(, but, obviously, with hardcoded names in modules that is not going anywhere unless i change the module (not a proposition for CDN-hosted common modules).

@dvdotsenko If a config value is not applied until after the target resource has been initially mentioned I can see there being problems. For loading multiple versions of jQuery in a page, requirejs has multiversion support. Although it sounds like you wanted different versions for timeout recovery, so you may be interested in this recent thread on timeout recovery.
If you are interested in talking through it or working out the use case a bit more, please feel free to start a thread on the requirejs list since we are getting outside the concerns of underscore.

@jrburke
Indeed conversation digressed towards peculiarities of RequireJS, but, the issue of hard-coding a name INTO an AMD module is still nagging me in respect to Unserscore. Where can read up on definitive explanation for why Unserscore MUST be named AMD module, so I would stop nagging you?

@dvdotsenko The updating existing libraries page, anon section attempts to give the reasoning for the named module guidance for libraries like jQuery and underscore.
If you would prefer to discuss the relative merits of this guidance, it would be great if you can start a thread on the amd-implement list. This guidance is not specific to underscore, and we will have the opportunity to engage people who are very motivated to get this right in general for AMD. AMD loader implementers, like John Hann who develops the curl loader, is also on that list.
We can post back any summary that changes the guidance to this bug, or better yet if it results in a possible code change, just follow it up with a targeted pull request citing the new common understanding worked out in the other group.


Restore cacheability of ScalaCompile

This was broken by adding the current JVM's absolute path to CompileOptions.executable
I can't tell why this change was made, as all the tests are passing without this line.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

@bmuskalla FYI since you introduced this change

This used to be required as the compiler option had no default anymore with the introduction of toolchains. But I think we changed that around once we fixed all other places. Thanks for catching.

Would you mind adding a test case for that as well?

A real integration test of this is hard, since it would need two Java installations of the same version, but in different directories. The unit test on the other hand feels almost too trivial (asserting that options.executable==null by default).

Yep, that works and ensures Java home relocatability for all tasks.

Okay, finally got it working in all tests - Turns out our copy function didn't work with the symlinked Java homes on Windows.
This is ready for another look.

Ping :)

@bot-gradle test and merge

Your PR is queued because #17053 is being built.

OK, I've already triggered a build for you.


I need x64RetSpoof

I need x64RetSpoof. Can you provide an x64 architecture RetSpoof

机缘巧合下看到这个问题，希望对你有帮助
CXiaoshaoFuncCall.h
https://www.aliyundrive.com/s/1SX1oMB5Q6d
点击链接保存，或者复制本段内容，打开「阿里云盘」APP ，无需下载极速在线查看，视频原画倍速播放。

@AiLulu999, Hi bro, i cant download from this site, because i havent Chinese number wich need for register. Can you send this code to pastebin, or github?

@AiLulu999, Hi bro, i cant download from this site, because i havent Chinese number wich need for register. Can you send this code to pastebin, or github?

In fact, CXiaoshaoFuncCall.h is the code I wrote by myself, which may not be perfect. So I suggest you read this article, I also get inspiration from it.
https://zhuanlan.zhihu.com/p/362173165
Second, you can also look at BlackBone, which is very well-written code and can be used as a textbook.  It has the code you want
https://github.com/DarthTon/Blackbone

@AiLulu999, Hi bro, i cant download from this site, because i havent Chinese number wich need for register. Can you send this code to pastebin, or github?

事实上，CXiaoshaoFuncCall.h 是我自己写的代码，它可能并不完善。所以我建议你看这篇文章，我也是从中得到启发。
https://zhuanlan.zhihu.com/p/362173165
其次你也可以参考 BlackBone，它的代码写的很好，完全可以作为教科书来用，其中有你想要的代码
https://github.com/DarthTon/Blackbone

I need x64RetSpoof. Can you provide an x64 architecture RetSpoof

x64没有栈帧概念，你可以根据x86的自己修改


Support for osx/arm build

Comment:
Given that apple has been supplying ARM chips for many years. Might be good to have arm build for osx? Thanks.
@jan-janssen Do you mind add this repo to the migrator https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/5192, please? Thanks.

Thanks for the pin, but I do not maintain this package. You can try to reach out to @clonker, @franknoe and @marscher the maintainers of this package.  https://github.com/conda-forge/pyemma-feedstock#feedstock-maintainers

@jan-janssen Thanks. Do you mind approve the PR https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/5192, please? Then regro-cf-autotick-bot shall open a PR for the migration.

@xiki-tempula I have the feeling you overestimate my privileges. While I maintain a series of conda-forge packages, I am not part of the conda-forge core team, so I cannot merge pull requests on conda-forge-pinning-feedstock.

@jan-janssen Sorry, I see you everywhere so I kind of assume that you could merge PR.


Intellisense not working

Issue Type: Bug
Ctrl + Space
VS Code version: Code 1.40.1 (8795a9889db74563ddd43eb0a897a2384129a619, 2019-11-13T16:49:35.976Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8565U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.78GB (5.76GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (12)

Extension
Author (truncated)
Version

vscode-css-formatter
aes
1.0.1

simple-react-snippets
bur
1.2.2

npm-intellisense
chr
1.3.0

vscode-eslint
dba
1.9.1

react-native-snippet
jun
0.5.5

mssql
ms-
1.7.1

python
ms-
2019.11.50794

powershell
ms-
2019.11.0

vscode-typescript-next
ms-
3.8.20191123

vscode-react-native
msj
0.13.1

JavaScriptSnippets
Tem
0.5.0

JavaScriptSnippets
xab
1.7.2

/needsMoreInfo
where exactly does Intellisense not working, please provide detailed steps an provide a sample.

Hello,
I also can not use InteIliSence.
I'm using vscode(1.40.1) on linux(Lubuntu19.10) with PlatformIO.
When I install PlarformIO, the system  installs 'c/c++' extension automatically.
IntelliSence works fine with 'c/c++' extension Version 0.26.1: October 28, 2019
IntelliSence  can not work with Version 0.26.2-insiders2.
I downgrade the extension by using its preference, but the system update it automatically.......
I should mange the version of extension every 5 min.

@mikionakajima this looks like another issue, the original reporter doesn't have this extension installed. Please file a new issue with reproducible steps against the C++ extension.


IdentityServer4 Swagger documentation

Hello,
How can Swashbuckle pickup IdentityServer4 endpoints and display it as part of the swagger docs

It can't - we have not instrumented the endpoint as we don't see a benefit here.
The endpoints are documented in the spec.


fix: used double quotes for env vars in .env files

A certain DIAL user experienced issues while running the appication quick-start docker-compose in the following environment:
$ uname -a
Linux dial 6.8.0-31-generic #31-Ubuntu SMP PREEMPT_DYNAMIC Sat Apr 20 00:40:06 UTC 2024 x86_64 x86_64 x86_64
$ cat /etc/os-release | grep NAME
PRETTY_NAME="Ubuntu 24.04 LTS"
NAME="Ubuntu"
$ echo $BASH_VERSION
5.2.21(1)-release
$ cat /etc/passwd | grep dial
dial:x:1000:1000:dial:/home/dial:/bin/bash

The following sequence of commands:
git clone https://github.com/epam/ai-dial.git
cd ai-dial/dial-docker-compose/application
docker compose up

led to DIAL Core failing with error complaining about missing Core config file.
It turned out that docker compose didn't pick up $DIAL_DIR variable from .env. And thus the variable was unset when the common.yml was interpolated. As a result aidial.config.files was pointing to a wrong directory which was missing DIAL core config.json.
The env file looked like this:
DIAL_DIR=./application

After many hours of debugging, it turned out that the following env file with double-quotes actually works:
DIAL_DIR="./application"

It's still unclear why the original env var wasn't picked up properly in the given environment.

That was a red herring actually.
See https://github.com/epam/ai-dial/pull/105 for the actual fix.


apksigcopier: 1.1.0 -> 1.1.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for apksigcopier is: Copy/extract/patch android apk signatures & compare APKs
meta.homepage for apksigcopier is: https://github.com/obfusk/apksigcopier
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.1 with grep in /nix/store/ps1wfsz6jvf12nnzn68k9kai26rac9vl-apksigcopier-1.1.1
found 1.1.1 in filename of file in /nix/store/ps1wfsz6jvf12nnzn68k9kai26rac9vl-apksigcopier-1.1.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
apksigcopier
diffoscope

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ps1wfsz6jvf12nnzn68k9kai26rac9vl-apksigcopier-1.1.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A apksigcopier https://github.com/r-ryantm/nixpkgs/archive/5442ecbb129c715912c94d53a0c12901825de210.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ps1wfsz6jvf12nnzn68k9kai26rac9vl-apksigcopier-1.1.1
ls -la /nix/store/ps1wfsz6jvf12nnzn68k9kai26rac9vl-apksigcopier-1.1.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

apksigcopier
diffoscope

Maintainer pings
cc @obfusk for testing.

@linsui could you test this please?


Routes with domains are serving the wrong controller

Issue:
Because Laravel processes routes on a first-come-first-serve basis, if you have routes with the same URI on different domains, it doesn't matter which domain you are trying to request, Laravel with serve you the first route that matches that URI.
Demo:
With this route definition if you're trying to request admin.localhost/test Laravel will serve you localhost/test.
Route::get('/test', function () {
    //
});

Route::group(['domain' => 'admin.localhost'], function () {
    Route::get('/test', function () {
        //
    });
});

That's why we will have to define the subdomain group first.
With this route definition, Laravel will serve correct routes.
Route::group(['domain' => 'admin.localhost'], function () {
    Route::get('/test', function () {
        //
    });
});

Route::get('/test', function () {
    //
});

Conclusion:
Back to why I'm posting this issue... I know this is not an issue with this package per se, but using it, we are losing the ability to control the order of defined routes.
We could solve this by sorting the collected routes by ones that have a domain defined first, and then registering them.
Currently, I'm unable to fully utilize this great tool, because I have subdomains in my project, but if I find some spare time I'll try to PR a "fix".
Regards.

The problem is not domain related it's the registration route order that causes the issue
In a classic route file you will switch routes from explicit to generic to resolve that kind of problem
But with this package, it's not possible cause it register as parsing file.
I'm working on a fork to add priority to route attribute, hope I can help you soon!

@mmoutih Where are we on route attribute priority, how can I help?

Hi @Parables i had to put the project to standby for a few weeks cause i had some issus with an other project, i thing i will be releasing something by the end of the month.

The order of the methods is already preserved when defining routes, the order of classes is also preserved alphabetically, seeing how you declare your routes would be nice to get some thoughts on the issue
It would be possible to add the domain routes in a different folder and put this folder in your configuration before the other controller folder


Remove backward compatibility

Fixes #70

:ok_hand:

Go for it :wink:.

:ship: ing

Published :tada:


Change project structure to reflect Edge Cloud API 'family'

Per recent calls, the Edge Cloud APIs will be divided into four groups:
Discovery
Resource Management
Workload Orchestration
Traffic Influence
Each would work on APIs that deliver a set of intents: so Discovery would provide APIs that meet the Discovery intents, and Traffic Influence would contain the Traffic Influence API.   “Resource Management” and “Workload Orchestration” would include intents that are today called ‘runtime’ and ‘provisioning’ and are included in the EdgeXR/GSMA/5GFF contributions.
This  means we can more quickly publish the first API from the Discovery group (‘Simple Edge Discovery’) with a route to extend it to the additional Discovery functionality (‘MEC Exposure and Experience Management’). Both ‘self-build’ and ‘hyperscaler partner’ operators can utilise these APIs.
All operators can support the discovery APIs.
Operators supporting the ‘self-build’ MEC model (i.e. no hyperscaler) can utilise the ‘Resource Management’  and ‘workload orchestration’ groups.
Operators supporting the ‘hyperscaler partner’ MEC model can utilise the Resource Management’  and ‘workload orchestration’ groups if their hyperscaler partner(s) support an integration to do so.
That can help operators start to expose the APIs that support their MEC implementation. Otherwise we will find it very difficult to arrive at one single API that everybody can implement.
Each group would be a sub-project containing members that can work on that particular API. All APIs would need to align with the Commonalities API Design Guidelines.

PR #126


Cannot inject dependency which is a subclass of a subscripted type

Running the example code below,
from typing import Sequence
from injector import inject, Injector

class FibonacciSequence(Sequence[int]):
    def __init__(self, n_terms: int = 10) -> None:
        self._n_terms = n_terms

    def __getitem__(self, i: int) -> int:
        if i >= self._n_terms: raise IndexError

        if i == 0: return 0
        if i == 1: return 1
        if i >= 2: return self[i-2] + self[i-1]

    def __len__(self) -> int:
        return self._n_terms

# Test injecting FibonacciSequence dependency

@inject
def main(sequence: FibonacciSequence) -> None:
    print(tuple(sequence))

injector = Injector()
injector.call_with_injection(main)

I get the following error,
Traceback (most recent call last):
  ...
  File ".../python3.5/site-packages/injector/__init__.py", line 1072, in _infer_injected_bindings
    v, metadata = v.__origin__, v.__metadata__
AttributeError: type object 'FibonacciSequence' has no attribute '__metadata__'

I ran this on Python 3.5.7.
I think the cause of this issue is _is_specialization(v, Annotated) incorrectly returns True when v is FibonacciSequence.
https://github.com/alecthomas/injector/blob/a461f722adcf8e23f0fdd4e122f101029a74877c/injector/__init__.py#L1071-L1073
The current implementation of _is_specialization() is
https://github.com/alecthomas/injector/blob/a461f722adcf8e23f0fdd4e122f101029a74877c/injector/__init__.py#L596-L618
The value of FibonacciSequence.__origin__ is None. I'm not sure where exactly the __origin__ attribute is set but it seems (I'm guessing) that it is set to None for unsubscripted generics or subclasses of subscripted generics. So the not hasattr(cls, '__origin__') condition in line 608 fails as FibonacciSequence has an __origin__ attribute set to None. Finally on line 618, the comparison origin is generic_class succeeds because here origin is None, and generic_class is also None because I am running this on a version of Python lower than 3.7.0 so HAVE_ANNOTATED is False and so Annotated has also been set to None.
I think modifying line 608 to
if getattr(cls, '__origin__', None) is None:

should fix this issue.

Hey, a fix has just landed in the master branch. The patch is different than yours as I needed to resolve a new issue discovered by new mypy (0.750) anyway but this doesn't take away from your analysis of the bug – it was thorough and detailed, an exemplary report in my book. Thank you!

The fix for this has just been released as part of the version 0.18.1.


Fix failing integration test on Windws with Chrome

It fixes #18775.

/botio integrationtest

/botio integrationtest

/botio integrationtest

/botio-windows integrationtest

/botio-windows integrationtest

/botio integrationtest

/botio integrationtest

/botio integrationtest

/botio integrationtest

@timvandermeij The issue is fixed now: the root cause was about not having integer coordinates. I don't know if some was selected but for sure the selectionchange event wasn't triggered because of that.

/botio integrationtest

/botio integrationtest


Fail to run test scenarios in docker

Describe the bug
In the tutorial, Run test scenarios in docker does not work.
First of all,  command  $ rocker --x11 --oyr-mount $PWD/scenarios/UC-001-0001-Kashiwa.yaml -- tier4/scenario_simulator_v2 ros2 launch scenario_test_runner scenario_test_runner.launch.py  is not valid now.
Instead, command $ rocker --x11 --oyr-mount $PWD/scenarios/UC-001-0001-Kashiwa.yaml -- tier4/scenario_simulator_v2:galactic ros2 launch scenario_test_runner scenario_test_runner.launch.py
However, it fails to launch and  the error says that the lifetimecycle is unconfigured.
[scenario_test_runner-1] [ERROR] [1650413557.603908415] [simulation.scenario_test_runner]: Failed to configure interpreter
[scenario_test_runner-1] [ERROR] [1650413557.604781543] [simulation.scenario_test_runner]: Interpreter is unconfigured now, but inactive expected.

I check UC-001-0001-Kashiwa.yaml and it executes $(find-pkg-share cargo_delivery)
but I cannot found cargo_delivery
$ ros2 pkg prefix cargo_delivery
Package not found

So it seems we have to install cargo_delivery in advance? or the yaml is out of date?
The whole output is as follows:
$ rocker --x11 --oyr-mount $PWD/scenarios/UC-001-0001-Kashiwa.yaml -- tier4/scenario_simulator_v2:galactic ros2 launch scenario_test_runner scenario_test_runner.launch.py scenario:=/home/shumpeiwakabayashi/scenario_simulator_ws/src/scenario_simulator_v2/scenarios/UC-001-0001-Kashiwa.yaml launch_rviz:=True
Extension oyr_cap_add doesn't support default arguments. Please extend it.
Extension oyr_cap_drop doesn't support default arguments. Please extend it.
Extension oyr_colcon doesn't support default arguments. Please extend it.
Extension oyr_mount doesn't support default arguments. Please extend it.
Extension oyr_run_arg doesn't support default arguments. Please extend it.
Extension oyr_spacenav doesn't support default arguments. Please extend it.
Extension volume doesn't support default arguments. Please extend it.
Active extensions ['oyr_mount', 'x11']
Writing dockerfile to /tmp/tmpc41ljbfk/Dockerfile
vvvvvv
# Preamble from extension [oyr_mount]

# Preamble from extension [x11]

FROM tier4/scenario_simulator_v2:galactic
USER root
# Snippet from extension [oyr_mount]

# Snippet from extension [x11]

^^^^^^
Building docker file with arguments:  {'path': '/tmp/tmpc41ljbfk', 'rm': True, 'nocache': False, 'pull': False}
building > Step 1/2 : FROM tier4/scenario_simulator_v2:galactic
building >  ---> 5959b93f732d
building > Step 2/2 : USER root
building >  ---> Using cache
building >  ---> befb4b2cfefc
building > Successfully built befb4b2cfefc
Executing command:
docker run --rm -it -v /home/shumpeiwakabayashi/scenario_simulator_ws/src/scenario_simulator_v2/scenarios/UC-001-0001-Kashiwa.yaml:/home/shumpeiwakabayashi/scenario_simulator_ws/src/scenario_simulator_v2/scenarios/UC-001-0001-Kashiwa.yaml  -e DISPLAY -e TERM   -e QT_X11_NO_MITSHM=1   -e XAUTHORITY=/tmp/.docker81mxnvmp.xauth -v /tmp/.docker81mxnvmp.xauth:/tmp/.docker81mxnvmp.xauth   -v /tmp/.X11-unix:/tmp/.X11-unix   -v /etc/localtime:/etc/localtime:ro  befb4b2cfefc ros2 launch scenario_test_runner scenario_test_runner.launch.py scenario:=/home/shumpeiwakabayashi/scenario_simulator_ws/src/scenario_simulator_v2/scenarios/UC-001-0001-Kashiwa.yaml launch_rviz:=True
[INFO] [launch]: All log files can be found below /root/.ros/log/2022-04-20-09-12-35-406001-6f5c18362c68-1
[INFO] [launch]: Default logging verbosity is set to INFO
architecture_type       := awf/universe
autoware_launch_file    := planning_simulator.launch.xml
autoware_launch_package := autoware_launch
global_frame_rate       := 30.0
global_real_time_factor := 1.0
global_timeout          := 180
initialize_duration     := 30
launch_autoware         := True
launch_rviz             := True
output_directory        := /tmp
port                    := 8080
record                  := True
scenario                := /home/shumpeiwakabayashi/scenario_simulator_ws/src/scenario_simulator_v2/scenarios/UC-001-0001-Kashiwa.yaml
sensor_model            :=
vehicle_model           :=
workflow                := /dev/null
sigterm_timeout         := 8
[INFO] [scenario_test_runner-1]: process started with pid [64]
[INFO] [simple_sensor_simulator_node-2]: process started with pid [66]
[INFO] [openscenario_interpreter_node-3]: process started with pid [68]
[INFO] [openscenario_visualization_node-4]: process started with pid [70]
[INFO] [rviz2-5]: process started with pid [72]
[scenario_test_runner-1] [INFO] [1650413556.600185803] [simulation.scenario_test_runner]: Run UC-001-0001-Kashiwa.xosc (1 of 1)
[scenario_test_runner-1] [ERROR] [1650413557.603908415] [simulation.scenario_test_runner]: Failed to configure interpreter
[scenario_test_runner-1] [ERROR] [1650413557.604781543] [simulation.scenario_test_runner]: Interpreter is unconfigured now, but inactive expected.
[scenario_test_runner-1] [INFO] [1650413557.605071644] [simulation.scenario_test_runner]: Shutdown interpreter.
[scenario_test_runner-1] /home/shumpeiwakabayashi/scenario_simulator_ws/src/scenario_simulator_v2/scenarios/UC-001-0001-Kashiwa.yaml
[INFO] [scenario_test_runner-1]: process has finished cleanly [pid 64]
[INFO] [launch]: process[scenario_test_runner-1] was required: shutting down launched system
[INFO] [rviz2-5]: sending signal 'SIGINT' to process[rviz2-5]
[INFO] [openscenario_visualization_node-4]: sending signal 'SIGINT' to process[openscenario_visualization_node-4]
[INFO] [openscenario_interpreter_node-3]: sending signal 'SIGINT' to process[openscenario_interpreter_node-3]
[INFO] [simple_sensor_simulator_node-2]: sending signal 'SIGINT' to process[simple_sensor_simulator_node-2]
[openscenario_visualization_node-4] [INFO] [1650413558.812376422] [rclcpp]: signal_handler(signal_value=2)
[openscenario_interpreter_node-3] [INFO] [1650413558.812806248] [rclcpp]: signal_handler(signal_value=2)
[simple_sensor_simulator_node-2] [INFO] [1650413558.813335045] [rclcpp]: signal_handler(signal_value=2)
[INFO] [openscenario_visualization_node-4]: process has finished cleanly [pid 70]
[INFO] [simple_sensor_simulator_node-2]: process has finished cleanly [pid 66]
[INFO] [openscenario_interpreter_node-3]: process has finished cleanly [pid 68]
[INFO] [rviz2-5]: process has finished cleanly [pid 72]

To Reproduce
Steps to reproduce the behavior:

Follow Run test scenarios in docker

Expected behavior
Succeed to run test scenario and to show it in rviz.
Screenshots
Desktop (please complete the following information):

OS: Ubuntu20.04
ROS2 galactic

@shmpwk Oh... cargo delivery package was already removed and I forget to update this document.
Thank you for finding my mistake.

@shmpwk I think this problem was fixed by this PR.
https://github.com/tier4/scenario_simulator_v2/pull/769

Thank you!


FEATURE - Add TipKit

Describe the feature you want the app to have that it currently doesn't

Implement a TipKit tip that shows a user on the SettingsView that this an app that's open source and that anyone can contribute to this project if they're interested in iOS development with Swift

Describe alternatives you've considered

N/A

Additional context

I haven't used TipKit yet, but I did not want to make a tip pointing to the + button because that feels obvious and self explanatory. If you have any other ideas for a tip, comment below what you want to implement it and @mikaelacaron will approve it (if it's different than what's described above)

Hey @mikaelacaron  can I implement this feature? How do you find this representation of the tip?

yeah! Can you "anchor" it to the GitHub repo row? rather than the heading


Fixes basic a-frame example

It looks like the marker used in the couch image no longer works, and I can't find the pattern to the marker.   This PR makes it so that it sets the preset to the hiro marker used in all the other examples.
Also bumps the build, a-frame version and fixes some whitespace.

@jeromeetienne review?  The aframe example wasn't working at all, so this fixes that. Please let me know if there are changes you would like to see :)

sorry for hte delay i will check today :)

Merged! thanks!

thanks!


Bump haskell.nix

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Formatting, PNG optimization, etc. are updated
[ ] Important changes are reflected in changelog.d of the affected packages

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reference the ADR in the PR and reference the PR in the ADR (if revelant)
[ ] Reviewer requested

I think rebasing will fix the build failures.

Everything seems to work :+1:


Add city field to City of Piedmont, CA

Frustrating that it doesn't create a log 😞 . Not sure where it went.


os matches more files in Obsidian vault than oss

Assume that we have the following notes:

Test A.md (inside Obsidian vault)
Test B.md (inside Obsidian vault)
Test C.md (outside Obsidian vault)

The following tests are done with obsidian-alfred version 0.2.1.
If I search for the keyword "Test" with os Test, all three notes would be listed. However, if I search for the keyword "Test" with oss Test, only Test A.md and Test C.md would be listed.
Also, is it possible to open a note outside Obsidian vault, i.e. Test C.md in this case? Apparently one cannot open this note in Obsidian, but is it possible to open it with some default editor for Markdown files?

Also, is it possible to open a note outside Obsidian vault, i.e. Test C.md in this case? Apparently one cannot open this note in Obsidian, but is it possible to open it with some default editor for Markdown files?

Yes, this should be relatively easy to implement. It's just a matter of replacing obsidian's uri (obsidian://open?path=) with file:/// if your file is outside the vault. I'll work on it in the next few days.

I don't fully understand the answer to the first question. Since both Test A.md and Test B.md are within Obsidian vaults. Shouldn't os and oss both list them when searching with the keyword "Test"?
After I switch on the option "with space" for os. Searching with oss tries to implement fallback search, as can be seen from the figure below. However, it would try to search for "oss Test" in my vault, instead of "Test". Also, I would have to hit Enter to select a specific vault to search the keyword in. Is it possible to search for the keyword in all vaults?

Oh, if you read the new documentation, it says you should press enter after typing oss (and the other two). That is, oss, press enter/return, then perform your search.
As stated in #9, there isn't a better solution than this for now (as far as I know). To restrict the search scope to your vaults, it requires an extra step. If you want to change its behavior slightly, set it to Argument Optional. In that case, it won't default to fallback search, but matching files also won't be listed until you press enter. There's no quick solution to this (unless the you/the user manually specifies the scopes in the File Filter os, or, ot). I can look into solutions with mdfind, but that would involve making significant changes to the current script/solution.

Ah, I see now. Thanks for the detailed explanation. I realize that I need to change the default keyword in the left part of following blocks from os to oss. Now it works for me. I was looking for the keyword oss in the workflow blocks but didn't find any...

Glad it's working for you!


Chat - Emoji suggestions are not displayed when user searches "animals"

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go https://staging.new.expensify.com/
Tap on a report
Tap on emoji picker
Search for "animal" Or "animals"
Search "flower"
Search " Rose"

Expected Result:
When user searches for "flower" in general or "rose" in specific, emoji suggestions are displayed. So, when user searches for "animals", emoji suggestions related to animals must be displayed
Actual Result:
When user searches for "flower" in general or "rose" in specific, emoji suggestions are displayed. But, when user searches for "animals", emoji suggestions related to animals are not displayed. No result found message only shown
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.66-3
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/12af1c1e-d9ae-41cd-97ba-a470ba026dc8
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

I don't think this is a bug

Yea, this doesn't feel worth fixing.


Build errors should not break watch script

Currently, a build error will interrupt the watch cycle.
Reproduction steps:

mkdir test && cd test && mm-plugin init to setup
mm-plugin watch to use watch script
Edit index.js to have some invalid JS.
Observe watch script breaks.

Desired behavior:
Watch script should report errors, but be ready to retry on file change still.

Fixed in 0.3.4


Async debug

Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

[Enter positive test case here]

@alvacouch - please approve again (fixed merge conflicts) and we'll get this out in the next release.

@alvacouch - if you approve again, I'll get this merged.


👌 IMPROVE: Set the default endpoint name to the function name

This will set the default endpoint name according to the view function name, if the value of the url_name parameter is not set.

This relates to https://github.com/vitalik/django-ninja/issues/66#issuecomment-826259777


Lua support?

There seems to be to mention of or support for lua configuration.  Neovim appears to be moving everything to lua and as a *vim newbie, the split between init.vim and init.lua are killing me.  I see that there are issues filed, but nothing in the README or FAQ.
I'd like there to be a section under https://github.com/junegunn/vim-plug#example that includes a working lua example.

Type:

[ ] Bug
[X] Enhancement
[X] Feature Request
[X] Question

OS:

[ ] All/Other
[X] Linux
[ ] macOS
[ ] Windows

Vim:

[ ] Terminal Vim
[ ] GVim
[X] Neovim

Something like this would help: https://gitlab.com/sultanahamer/dotfiles/-/blob/master/nvim/lua/plugins.lua?ref_type=heads

Thanks for the guidance. I added a Lua example section to the README. I'm not actually a Neovim/Lua user, so please let me know if I did something wrong.
https://github.com/junegunn/vim-plug?tab=readme-ov-file#example-lua-configuration-file-for-neovim


Better error pages

Custom error pages can be placed in resources/views/errors/

Fixed 404 in 470c341bd3b083f505658a093e6dc7857ea242c3
Do you have other errors in mind?

Also add a generic error page with a contact email that can be configured.

Seperate pages for 404, 500 and bad query, generic page for all other exceptions.
Email report is being sent every time an exception occurs.
Must configure project email


fix(manual-span-example): add example in the README to create span manually

Codecov Report

Merging #97 (b791403) into main (4e93d39) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #97   +/-   ##
=======================================
  Coverage   82.81%   82.81%
=======================================
  Files          18       18
  Lines         745      745
  Branches      147      147
=======================================
  Hits          617      617
  Misses        128      128


fix: raise error for invalid environment variables file

Issue #, if available: #626
Description of changes:
Introduces an exception to be thrown when an environment variable file is invalid.
SAM still swallows regular malformed env files, but will throw an error if the json structure doesn't match as expected.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jfuss I think this is about done now, unless you have other changes


Enable test for persistent storage

We no longer run tests using minikube in a VM on the CI infrastructure
but with GCP
Fixes #292
Depends on #302

Needs a rebase.

The test makes me wonder if we should validate that the pods are actually created. The creation of a pod can fail if the persistent volume claim can't be fulfilled. And I'm not sure if the tests actually tests it. What do you think?
@krnowak I am not sure right now, let me think about it for a bit.

@krnowak The test actually waits for the persisted pod to be in a running state here https://github.com/habitat-sh/habitat-operator/pull/303/files#diff-2e3b6c005e3fc97b3b2a73f5cdeb0543R262

Oh, I overlooked that. In that case, LFAD.


Schema for v1.26.10 is missing

The schema for v1.26.10 is missing. Is it intended?

Should be there https://github.com/yannh/kubernetes-json-schema/tree/master/v1.26.10  ? Does it work with other versions? :bow:

Apparently I'm blind, forgot to drink my coffee today and for sure forgot how github sorts folder lexicographically.  🙇


chore(types): make ElementArrayFinder more generic

Add some generics to ElementArrayFinder methods.

I signed it!


change moonbeam to secp256k1

Changes the ss58-registry.json so that moonbeam networks use secp256k1
closes https://github.com/paritytech/substrate/issues/9318

ready for review

bot merge


Torrentio RD not working on Android but Media Fusion works

Torrentio is not working for me on my android device. No matter what I try to play, it always give "Some usage limit exceeded, check your debrid account for more info" error.
Media Fusion with RD is working on my android device without any issue.
I don't have any pending downloads in RD.
Torrentio RD in web Stremio works fine.
Since Torrentio is working fine on web version of Stremio and Media Fusion is working fine on Android, the issue shouldn't be with RD, right?

The message is clear enough, you're exceeding some usage limits - Too many active downloads;Traffic exhausted;Upload too big;Fair Usage Limit.


resolves #85 enable web fonts in asciidoctor theme

I think we need to find another way to load the fonts because Chrome extension CSP is too restrictive.
The same goes for embeded YouTube video...

Can we just add it to permissions as described in this SO post? http://stackoverflow.com/questions/19210451/packaging-a-font-with-a-google-chrome-extension

Even with the following permission
 "permissions":[
    "http://fonts.googleapis.com/",
    "https://fonts.googleapis.com/"
  ],

I still get:
Refused to load the stylesheet 'https://fonts.googleapis.com/css?family=Open+Sans:300,300italic,400,400ital…0,600italic%7CNoto+Serif:400,400italic,700,700italic%7CDroid+Sans+Mono:400' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'style-src' was not explicitly set, so 'default-src' is used as a fallback.

And if I append the following <link> to the <head> instead of the @import:
<link href='https://fonts.googleapis.com/css?family=Open+Sans:300,300italic,400,400italic,600,600italic%7CNoto+Serif:400,400italic,700,700italic%7CDroid+Sans+Mono:400' rel='stylesheet' type='text/css'>

I get:
Refused to load the font 'https://fonts.gstatic.com/s/droidsansmono/v7/ns-m2xQYezAtqh7ai59hJYdJ2JT0J65PSe7wdxAnx_I.woff2' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'font-src' was not explicitly set, so 'default-src' is used as a fallback.
Refused to load the font 'https://fonts.gstatic.com/s/notoserif/v4/Q47Ro23nlKqZrOLipd3-SyEAvth_LlrfE80CYdSH47w.woff2' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'font-src' was not explicitly set, so 'default-src' is used as a fallback.
Refused to load the font 'https://fonts.gstatic.com/s/notoserif/v4/qkE6YsKPRiYUugBb1_QwHCEAvth_LlrfE80CYdSH47w.woff2' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'font-src' was not explicitly set, so 'default-src' is used as a fallback.
Refused to load the font 'https://fonts.gstatic.com/s/notoserif/v4/N2U74xxQEyaTBF6QLZRr1CEAvth_LlrfE80CYdSH47w.woff2' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'font-src' was not explicitly set, so 'default-src' is used as a fallback.
Refused to load the font 'https://fonts.gstatic.com/s/notoserif/v4/1_daFS3X6gkNOcmGmHl7UiEAvth_LlrfE80CYdSH47w.woff2' because it violates the following Content Security Policy directive: "default-src 'none'". Note that 'font-src' was not explicitly set, so 'default-src' is used as a fallback.

It works for me when using an unpacked extension. Are you testing it packed
or unpacked when you get the error?

I am testing with packed extension and remote document hosted on GitHub.
No error is thrown when viewing local document.
But even with the console full of errors I can't see the difference in the
rendered document...
Are you experiencing the same behavior ?
Le 10 janv. 2015 23:43, "Dan Allen"<EMAIL_ADDRESS>a écrit :

It works for me when using an unpacked extension. Are you testing it packed
or unpacked when you get the error?
—
Reply to this email directly or view it on GitHub
https://github.com/asciidoctor/asciidoctor-chrome-extension/pull/88#issuecomment-69475226
.

Aha, I forgot to test the remote URLs. That explains why I wasn't seeing the error.
I have searched and searched and I can't seem to find any confirmation that loading Google Fonts in a Google Chrome extension works, even with permissions or the content_security_policy set (I tried all sorts of combinations). An engineer in Google Fonts needs to have a stroll down the hall to visit an engineer on the Chrome team and figure out what the heck is going on over there. This has to be possible.
I'm going to ping Twitter for some answers. I guess we could also look for another extension that uses web fonts.
Note that the individual fonts are loaded from fonts.gstatic.com rather than fonts.googleapis.com, so we are dealing with two base URLs.

I'm going to ping Twitter for some answers. I guess we could also look for another extension that uses web fonts.

Any news ?

Note that the individual fonts are loaded from fonts.gstatic.com rather than fonts.googleapis.com, so we are dealing with two base URLs.

Yes I tried every combination but still not working...

No news. Quite frustrating that it doesn't just work.
I'm ready at this point to resort to using JavaScript. Surely there must be some way to force fonts to resolve after the page loads. There's a Google Web Font preview extension that must be doing something like this. Perhaps it's time to study that source code.

Here's code that loads fonts dynamically:
https://github.com/pamelafox/font-previewer-extension/blob/master/popup.js#L257
We have to know which fonts are loaded per theme, but for now we can just do this for the default theme...or we maintain a map of fonts that need to be loaded per theme.


Add more features to .interactive()?

What is your suggestion?
I'm wondering if it would be helpful if .interactive() did more by default. I'm primarily thinking about adding a tooltip (of the plotted encodings), and an interactive legend (based on opacity). There should of course be some way to turn these on/off, so that they don't interfere with manually set up bindings (maybe chart.interactive(legend=False).
Have you considered any alternative solutions?
No response

User feedback
Cross-referencing some feedback I've seen in polars:
https://github.com/pola-rs/polars/pull/17995#issuecomment-2316605863

One advantage of hvplot / bokeh is the free interactivity.
Altair does interactive too but I think only in a subset.
Please consider having a config for interactivity.

https://github.com/pola-rs/polars/pull/17995#issuecomment-2319700036

@joelostblom Thanks for the links.
Yup, whenever it makes sense for it to be the default, at least I would prefer interactive becoming the default.
It may not be acceptable for everyone, so whenever .interactive() is as mature as you'd like, bringing this up a poll/discussion in altair's repo will help understand what users want.

https://github.com/pola-rs/polars/pull/17995#issuecomment-2439144285

The new default plots from Altair have reduced the interactivity of plots. I've gone back to using hvplot because I struggled to get useful images out of Altair.
Was this a premature move considering Altair is missing essential features:

zoom to selection
resize plot
hover to show datapoint

Was the main intent to deliver a basic plotting experience without adding many dependencies?


Work around unsupported RegExp lookbehind in WebKit.

As reported on Zulip.

Cool, thanks.
This will however not be rolled out on mathlib4_docs until the CI is fixed again but I'll probably get to that today, I'll merge for now.


Make servo smoothing optional

I want to be able to let the servo move a switch back. The servo smoothing prevents this.
[Issue created by DocMoebiuz: 2016-12-11]
[Last updated on bitbucket: 2019-10-01]

has not been asked for during the last 24 months


Code | Alphabetic Pattern 33

Hey @aryashah2k, please assign me
-> Code | Alphabetic Pattern 33 #359
-> Code | Alphabetic Pattern 34 #360
-> Code | Alphabetic Pattern 35 #361
-> Code | Alphabetic Pattern 36 #362
-> Code | Alphabetic Pattern 37 #363
-> Code | Alphabetic Pattern 39 #364
-> Code | Alphabetic Pattern 40 #366
-> Code | Alphabetic Pattern 41 #367
-> Code | Alphabetic Pattern 42 #368
-> Code | Alphabetic Pattern 43 #369
so that it can be qualify for  beginner tag(10 points).

Or, I have to comment on all the issues separately?

It's okay, you could do it separately to make my work easy assigning them directly!


Making Modin more efficient with smaller DataFrames

What do these changes do?
This is not comprehensive, though it is a solid start.

Initial partitioning can also be determined by size now
Fixed some internal bugs around having 1x1 partitions (previously an
extreme edge case -- 1x1 DataFrame)
Starting with 4KB blocks, will adjust after some testing
Still encountering some edge case bugss

Related issue number

[x] passes git diff upstream/master -u -- "*.py" | flake8 --diff
[x] passes black --check modin/

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-Performance-PRB/127/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-Performance-PRB/128/
Test PASSed.

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-Performance-PRB/153/
Test PASSed.

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-Performance-PRB/154/
Test PASSed.

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

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-Performance-PRB/155/
Test PASSed.

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-Performance-PRB/156/
Test PASSed.

Merged build finished. Test PASSed.

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


It breaks on Markdown links

i.e.
[Some link](https://github.com/philippotto/Sublime-BracketGuard#installation)

And thank you for the plugin! :wink:

Thanks, should be fixed now!


Fixed stylesheet generation

Only difference I can see now is the newline break between each file 'block'.
I can look into getting rid of that too when I have some time.

Also tested with some NPM dependency css files and fixed issues around that.

Thanks Ben.
Could you please revert the changes on README.md? There are lines in there marked "<<<<<<HEAD" etc. - I am guessing due to merge conflicts with angular-seed. That's one file that I would like to maintain independent of the angular2-seed project.

Ahh yep, must have missed that file! Should be sorted now.

Thanks, Ben.


feat(issue): provide the submitted ticket url to caller

closes #351

@ostridm closing this PR, it is not relevant anymore

But it looked so good to me! 😂


release:tag attempts to push

For some reason when using 2.2.0 release:tag attempts to push and I get the error:
# cow release:tag 4.2.0 -vvv
[publish] Running release of all modules
[publish] Releasing library silverstripe/assets at version 1.2.0

In Library.php line 308:

  [Exception]
  Module silverstripe/assets cannot push without a current branch

Exception trace:
 SilverStripe\Cow\Model\Modules\Library->pushTo() at /root/.composer/vendor/silverstripe/cow/src/Steps/Release/PublishRelease.php:78
 SilverStripe\Cow\Steps\Release\PublishRelease->releaseLibrary() at /root/.composer/vendor/silverstripe/cow/src/Steps/Release/PublishRelease.php:59
 SilverStripe\Cow\Steps\Release\PublishRelease->releaseRecursive() at /root/.composer/vendor/silverstripe/cow/src/Steps/Release/PublishRelease.php:55
 SilverStripe\Cow\Steps\Release\PublishRelease->releaseRecursive() at /root/.composer/vendor/silverstripe/cow/src/Steps/Release/PublishRelease.php:55
 SilverStripe\Cow\Steps\Release\PublishRelease->releaseRecursive() at /root/.composer/vendor/silverstripe/cow/src/Steps/Release/PublishRelease.php:55
 SilverStripe\Cow\Steps\Release\PublishRelease->releaseRecursive() at /root/.composer/vendor/silverstripe/cow/src/Steps/Release/PublishRelease.php:35
 SilverStripe\Cow\Steps\Release\PublishRelease->run() at /root/.composer/vendor/silverstripe/cow/src/Commands/Release/Tag.php:27
 SilverStripe\Cow\Commands\Release\Tag->fire() at /root/.composer/vendor/silverstripe/cow/src/Commands/Command.php:58
 SilverStripe\Cow\Commands\Command->execute() at /root/.composer/vendor/symfony/console/Command/Command.php:251
 Symfony\Component\Console\Command\Command->run() at /root/.composer/vendor/symfony/console/Application.php:946
 Symfony\Component\Console\Application->doRunCommand() at /root/.composer/vendor/symfony/console/Application.php:248
 Symfony\Component\Console\Application->doRun() at /root/.composer/vendor/symfony/console/Application.php:148
 Symfony\Component\Console\Application->run() at /root/.composer/vendor/silverstripe/cow/bin/cow:31

release:tag [-r|--repository REPOSITORY] [-d|--directory DIRECTORY] [--skip-tests] [--skip-i18n] [-b|--branching BRANCHING] [--skip-archive-upload] [--skip-upload] [--aws-profile AWS-PROFILE] [--] <version> [<recipe>]

I note that the help out put states:

Tag modules and push

but the docs https://docs.silverstripe.org/en/4/contributing/making_a_silverstripe_core_release/ don't state a push will happen and that's what the release:push command is for

Yeah I've come to rely on this haha but it is a bug. Tags shouldn't be pushed until the publish step


Update to the latest wasi-libc.

This adds the __errno_location function and the change to readdir to call stat if d_ino is 0.

It also contains the change to handle zero inode values in fd_readdir.

This is just updating to the latest wasi-libc.


Migrate old issues from Jira (https://issues.carrot2.org/projects/HPPC/)

I'll port an archive of old jira issues to gh.

Done.
https://github.com/carrotsearch/hppc/issues?q=is%3Aissue+"migrated+from+JIRA"
Not all authors are correctly resolved to github ids, but this does not seem to be crucial. Thank you, @mocobeta for the jira-github migration scripts!


fix(congregation): override parts not saved in backup

:tada: This PR is included in version 2.53.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Marketplace

Coverage remained the same at 45.8% when pulling 84bb84e10725e884a89edc1951143be96522841c on marketplace into 91b631daa209967f6adf224cbc16344e0d0f6ea1 on chenqi/marketplace.


[SR-8824] Users have no way to customize target's Info.plist in SwiftPM

Previous ID
SR-8824

Radar
None

Original Reporter
@dduan

Type
Improvement

Additional Detail from JIRA

Votes
0

Component/s
Package Manager

Labels
Improvement, Screened

Assignee
None

Priority
Medium

md5: 51e69fa30efb16d6a53f4ee10c7e6b15

Issue Description:
When using `swift package generate-xcodeproj`, there's no convenient way to include a Info.plist for an Xcode target.

generate-xcodeproj command doesn't exist anymore. Closing.


usage about the symbol RNN

I'm trying to use the built-in symbol mx.sym.RNN, I encountered a shape issue, any help is appreciated.
Here's a part of my script:
data = mx.sym.Variable('data')
gru = mx.sym.RNN(data=data, state_size=1, num_layers=1, mode='gru')
sm = mx.sym.SoftmaxOutput(data=gru, name='softmax')
init = mx.initializer.Xavier()
model = mx.model.FeedForward(ctx=mx.context.gpu(), num_epoch=1, symbol=sm)
train_data = mx.nd.zeros((3, 1, 10))
y = np.zeros((3, 1))
model.fit(X=train_data, y=y)

It complains the shape is not consistent, does that mean I have to reshape the label all the time?

MXNetError: InferShape Error in softmax's label argument
Corresponding keyword of symbol: softmax_label
Shape inconsistent, Provided =(3,), inferred shape=(3,1)

okay, I should specify the shape of label

Hi, I meet the same problem and have no idea on how to fix it. Excuse me, how do you "specify the shape of label"?


fix validation for duplicate keys

Coverage increased (+2.2%) to 96.203% when pulling 2dd82ceb8bd3c7066924128a67df3f0a2e4e5b21 on fix/validation into d38c2c9ad947ecbd6dd7fc38fd6f6c9cb1fed258 on main.


Support Python 3.11

This change vendors the Authorize.net SDK and its PyXB dependency to fix its imports. It then switches the Python version from 3.9 to 3.11.
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Superseded by #15357 as we're still blocked by Django 4.x adoption


[DIAG] Page feed load error

*** Auto diag report from user. Use id for reference.
lang: nl:nl

Internal reference: Uploaded diag data https://chrome-facebook-hide-ads-and-reels.mrincops.net/diag/41.html

Internal reference: Diag data deleted from server

Fixed in v1.26.4


Check the current state before executing anything

As reviewed in #7, it's very common to run the dac factory multiple times on trial and error. It should be great if the factory script could check if the accounts are created before trying to create them again, and so on.

Deprecated


Replace beams with confidence scores

From vali: https://spacy.io/api/scorer

Scorer doesn't actually apply - closing this because it's not possible

Scorer doesn't actually apply - closing this because it's not possible


feat: allow specifying "github" dependencies

dependencies.txt files can be used by scripts to specify necessary dependencies. At the moment, these dependencies are specified implicitly including the tool necessary to install them:
https://github.com/Loathing-Associates-Scripting-Society/combo/branches/release/ (SVN)
https://github.com/Loathing-Associates-Scripting-Society/combo.git release (git)

This PR introduces some new syntax
github Loathing-Associates-Scripting-Society/combo release

Dependencies specified with the new syntax will check the user's local install to see if the dependency is installed using either SVN or git and not install it if so (and otherwise install it using git). This means that it doesn't matter how the user initially installed the script, or what other scripts are using in their dependencies files.
The aim of this is to reduce the number of duplicate copies of scripts the user may install.
Scripts that use the folder install method are not supported. I figure that either:

there is not a manifest file, so git install won't work, and SVN install should be used explicitly
there is a manifest file, so the author intends the dep to be installed using git, and git install should be used explicitly

The main target of this PR is scripts that are on github, but still use the SVN install. There are competing philosophies for install:

some users prefer to use git wherever possible
some users prefer to use whatever the author recommends in the README

I hope that the new syntax will satisfy both.

When I initially designed the git scripts feature, I was expecting it to only be used for those github scripts which had issues with SVN, and authors with affected scripts would be motivated to change to recommending git for download to avoid support requests around failure to update (i.e. the very problem git was introduced to fix).
However, it's been adopted more generally. Script duplication has been an issue since the start, but "delete your SVN script and adopt the git one" is an easy fix for those scripts that adopt git. The issue that arises is that script preferences for direct downloads and script preferences for their dependencies may differ. I'm on the "whatever the author recommends in the README" side of the debate, so I didn't see this one coming ;)


Cyclic dependency when importing library nodes

Describe the bug
When importing DaCe, there's a cyclic dependency in dace.libraries.blas -> dace.library -> dace -> dace.frontend.python.newast -> dace.libraries.blas. This is causing some weird errors in unexpected places, and we need to find a resolution to this.
To Reproduce
No minimal test case has been produced yet.

@definelicht Fixed, it was the nodes.py package mis-importing things in its namespace.
If you can verify that this is not fixed, please reopen.


Add Debre Tabor University

Add Debre Tabor University
URL = https://dtu.edu.et/

@Tibebu-ES First stage of verification passed, official domain is facebook.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.facebook.com/www.dtu.edu.et/


PEP8 & autoformat; import fixes (since you moved to src/ folder);

added missing import

Thanks for this! I notice the Travis build is failing because of the import path being incorrect in bin/*.py. If you remove those two lines then I believe it should work. :+1:

Yaaa also the directory structure really needs to be conformed to python standards.
Setuptools/disttools style with proper setup.py and all that. Then maybe move to a python -m vandal -c disassemble -s <filename> syntax also

@SamuelMarks Yep! I think it would be best to do that in a separate PR though. Are you intending to work on packaging (so we don't double-up)?

Spent about 5 minutes playing around: https://github.com/SamuelMarks/vandal/tree/setuptools
Yeah, I can probably concentrate on packaging. But note that my changes will effect essentially every file in the repo…

Cool, thanks! That's fine, provided you can merge it into master again afterwards ;)
(that's also the reason I suggested using a separate PR)


Remove hardcoded version from pilot assistant, add avc

Hoping this works, some .version files seem less worky than others. If worst comes to worse I'll change to a hardcoded 1.0.

#rebuild


Cannot select MeshInstance2D by clicking on it

Godot version:
3.1-alpha5
OS/device including version:
Any
Steps to reproduce:

Create a Sprite
Assign a texture
Sprite → Convert to 2D Mesh
Try to click on the newly created MeshInstance2D

Still valid in 3.2.2
You need to click the gizmo.


Delete webhooks from GitHub if the train is deleted

So that we don't fill up the webhook limits, GitHub (20), GitLab (50) etc. This would be applicable to all VCS integrations in Tramline – GitHub, GitLab, and Bitbucket.
The best time to do this is when a train is destroyed.

Can you assign this to me?

Can you assign this issue to me?

Can you assign this issue to me?


Not able to chroot into gobox

The default list of applets don't output a static binary, and are not able to be chrooted into using only /bin/gobox and symlinks created with -install
And the gobox -install . command installs correctly if not using a chroot, but it is not a relative path (such as the example file tree below) and therefore links to the pre-chroot path of gobox.
Is there a way to output a static binary with networking features? :smile:
A quick ldd returns the following dependencies:
bash-4.3$ ldd /git/gopath/bin/gobox
        linux-vdso.so.1 (0x00007ffe14764000)
        libpthread.so.0 => /lib64/libpthread.so.0 (0x00007fa94151a000)
        libc.so.6 => /lib64/libc.so.6 (0x00007fa941151000)
        /lib64/ld-linux-x86-64.so.2 (0x000055e35e78f000)

I am using go version go1.6.2 linux/amd64
Here is the filesystem that is able to be chrooted into, after removing the network related applets (telnetd,httpd,wget) and manually creating the correct symbolic links:
.
└── bin
    ├── bash -> sh
    ├── cat -> gobox
    ├── echo -> gobox
    ├── gobox
    ├── grep -> gobox
    ├── gunzip -> gobox
    ├── gzip -> gobox
    ├── head -> gobox
    ├── kill -> gobox
    ├── ls -> gobox
    ├── mkdir -> gobox
    ├── mknod -> gobox
    ├── mount -> gobox
    ├── ps -> gobox
    ├── rm -> gobox
    ├── sh -> gobox
    ├── shell -> gobox
    ├── umount -> gobox
    └── zcat -> gobox

Here is the size difference in case you were curious:
-rwxr-xr-x 1 aerth root  3966408 May 24 00:39 gobox-nonet
-rwxr-xr-x 1 aerth root  8966744 May 24 00:48 gobox-net
# and human form
-rwxr-xr-x 1 aerth root  3.8M May 24 00:39 gobox-nonet
-rwxr-xr-x 1 aerth root  8.6M May 24 00:48 gobox-net

and ldd difference:
bash-4.3$ ldd gobox-net
        linux-vdso.so.1 (0x00007ffe1abef000)
        libpthread.so.0 => /lib64/libpthread.so.0 (0x00007f33b17aa000)
        libc.so.6 => /lib64/libc.so.6 (0x00007f33b13e1000)
        /lib64/ld-linux-x86-64.so.2 (0x0000556a98aba000)
bash-4.3$ ldd gobox-nonet
        not a dynamic executable

You can enable static linking in Go by setting CGO_ENABLED=0.
$ CGO_ENABLED=0 go build -o gobox main.go gobox.go applets.go
$ file gobox
gobox: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, not stripped

Thanks!


Is it possible to set the random seed for tf.feature_column.categorical_column_with_hash_bucket on operation-level？

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 1.13.rc1
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
API:tf.feature_column.categorical_column_with_hash_bucket can not set random seed
Will this change the current api? How?
Yes, It will add a new parametre to set the random seed.
Who will benefit with this feature?
The one who want to get two different hash bucket on the same field.
Any Other info.

I think it's a reasonable request. Rohan, what do you think?

@xielm12,
The tf.feature_column.categorical_column_with_hash_bucket api was deprecated. Please take a look at this official doc link and image for the reference.

tf.feature_column is not recommended for new code. Instead, feature preprocessing can be done directly using either Keras preprocessing layers or through the one-stop utility tf.keras.utils.FeatureSpace built on top of them. Kindly check the migration guide for details. Thank you!


Introduction should have the explanation of what is a meta framework. Not pointing to another tool so we go to their documentation

Which scope/s are relevant/related to the feature request?
create-analog
Information
The introduction should have an explanation of what is a meta-framework. Not pointing to another tool so we go to their documentation
I think this is a bad practice in several layers because if I need to go to another documentation to understand the most basic concept of the tool, that implies that the documentation will be really bad or null. This implies that I should check other tools.
Describe any alternatives/workarounds you're currently using
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

Good point @titusfx. The initial page was done to show the two working together. The plan is to refresh the page with the Analog logo and a link to the Analog docs


Upgrade k8s api, client to v0.30

Description of your changes

Fixes #
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

https://github.ibm.com/IBMPrivateCloud/roadmap/issues/63182


xbib/marc should escape controlfields to make sure the resulting JSON stays valid

included controlfields (e.g. tabs) aren't escaped. e.g. if you transform the following record to a json, jq can't read it: "parse error: Invalid string: control characters from U+0000 through U+001F must be escaped at line 1, column 1442"
beispiel.mrc.zip

I guess the unicode escaping in MarcJsonWriter in marc4j might be good starting point/or impression.
re. line-delimited JSON it's not only important to deliver JSON valid content but also line-delimited JSON valid content, e.g., escaped line breaks.

implementing a custom MarcValueTransformer won't really lead to the expected result, since MarcJsonWriter#escape will escape backslashes again and e.g. "\n" will result in "\n", i.e., I think that all these other special characters also need to be processed via MarcJsonWriter#escape instead of a custom MarcValueTransformer (that will be applied early in the processing chain).

Shoud be fixed in MarcJsonWriter


enums: GPIO, TIMG

I'll return to this after array/cluster/strip oriented PRs. First need to focus on breaking changes.

Closing this due to its age, feel free to re-open or to open a new PR if you feel like revisiting this.


MySQL: unsigned int

I have a weird case I cannot make sense of:
I am trying to insert an unsigned int value into a MySQL table into a column of type unsigned int.
commands.execute(query, schemaModel)
where schemaModel is a marshmallow Schema and the field in question is defined like this:
flags = fields.Integer(required=False, allow_none=True)
Everything works fine unless the sign bit is set. Setting the flags value to 4.294.967.295 will result in an out of range error.
What is happening, am I doing something wrong?

What's the trace back? Does MySQL allow inserting a signed int into an unsigned column? Are you expecting it to convert implicitly?

Hi Zach, thank you for your reply. When looking up the trace I found the error elsewhere.. the incoming value from the API was already wrong, I got on the wrong track. Sorry about that. Closing this now.
brgs, Daniel


No visible path/directory of project

Issue Type: Feature Request
Hey! can I get to see project path/ directory without going somewhere or clicking anywhere. When i m working on two VS code windows with similar name projects and switching between them, it becomes difficult to know.
Right now i have to hover on a file to know the path. A quick view of path will be really handy.
                                                                                Thank You.

VS Code version: Code 1.47.2 (17299e413d5590b14ab0340ea477cdd86ff13daf, 2020-07-15T18:22:06.216Z)
OS version: Windows_NT x64 6.1.7601

See https://code.visualstudio.com/updates/v1_10#_configurable-window-title

In addition to ability to customize window titles you might also check Peacock extension which allows to quickly generate different colors for each workspace and differentiate them by colors.

Both good suggestions, thanks

See https://code.visualstudio.com/updates/v1_10#_configurable-window-title

Thank you for help


clean: logging snapshot for debug

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 57.076%

Files with Coverage Reduction
New Missed Lines
%

../../../.cargo/registry/src/index.crates.io-6f17d22bba15001f/rust-format-0.3.4/src/lib.rs
1
62.86%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
3412

Relevant Lines:
5978

💛  - Coveralls


jacoco test

【修复issue】#issue号
【修改内容】增加/修改/删除了xxx
【用例描述】是否需要修改用例？原因？/ 不增加用例的原因?
【自测情况】1、本地静态检查清理情况；2、xxx门槛用例通过；3、UT覆盖；4、如何测试的？
【影响范围】1、对用户的使用有没有影响（比如涉及api修改，启动参数修改）2、是不是需要更新一些文档（比如用户手册）

What type of PR is this?
/feat
Which issue(s) this PR fixes:
Fixes #
Use case description: xxxx
Does this PR introduce a user-facing change?
No
Additional documentation e.g., usage docs, etc.:
No

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@daec8c9). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #732   +/-   ##
==========================================
  Coverage           ?   26.44%
  Complexity         ?     1574
==========================================
  Files              ?      617
  Lines              ?    13438
  Branches           ?     1482
==========================================
  Hits               ?     3554
  Misses             ?     9438
  Partials           ?      446

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


boolean field rewrite

semantic markup - label is now an actual, clickable label
removed some js - not necessary anymore
changed CSS

moved switch into :before/:after
minor changes (em instead of px)
added :focus for visibility when using tab key

Clicking on the boolean field label is possible again after I changed the switch from onclick to onchange in #1007.
I addressed this before in #1036, but reverted it later. Now I changed the CSS properly.

Thanks!