adldap2 + api_token authentication fails

Laravel Version: 5.4
Adldap2-Laravel Version: 3.0
PHP Version: 7.0

Description:
When I try to authenticate users using the stored api_token using the default Laravel TokenGuard from Illuminate/Auth/TokenGuard.php it fails with unknown index 'email' once I want to retrieve the authenticated user.
I tracked down the reason. In Resolver.php (see here) in public function byCredentials it always tries to authenticated with the ldapUsername and eloquentUsername.
The credentials array at this point only holds one key-value pair, namely the api_token, thus it throws an unknown index exception.
public function byCredentials(array $credentials = [])
{
    if (empty($credentials)) {
        return;
    }
    return $this->query()
        ->whereEquals($this->getLdapUsername(), $credentials[$this->getEloquentUsername()])
        ->first();
}

Is there an easy way to fix this? Can I subclass the Resolver class and tell Adldap2 to use my custom resolver instead of the standard one?
Steps To Reproduce:

Setup default Laravel project
Set up Adldap2 Laravel as described
Add a api_token column to the user migration
Set the api_token each time the user logs in
Add a route in the API Group, using the middleware('auth:api') and return Auth::guard('api')->user()
Call the route with a valid access token in the Authorization header with Bearer {valid_api_token}

Update:
Long Story Short, subclassing the Adldap2 Resolver class would be an overkill. The entire issue can be fixed by updating the User Provider and the Authentication Guard in config/auth.php as follows:
**1. Add a new entry to the providers array. **
( I called it 'api-users')
'providers' => [
    'users' => [
        'driver' => 'adldap', // Was 'eloquent'.
        'model'  => App\User::class,
    ],
    'api-users' => [
        'driver' => 'eloquent',
        'model'  => App\User::class,
    ]
],

**2. Update the 'api' entry in theguards array. **
'guards' => [
    'web' => [
        'driver' => 'session',
        'provider'  => 'users',
    ],
    'api' => [
        'driver' => 'token',
        'provider'  => 'api-users'
    ]
],

Thanks for following up and closing this issue @FroeMic.

I think I'm having a similar issue in trying to implement Basic Auth on some API routes.
If I revert to eloquent driver, all is fine of course. But I'm little out of my depth now. Here's the error;
ErrorException in Resolver.php line 46: Undefined index: username

    in Resolver.php line 46
    at HandleExceptions->handleError(8, 'Undefined index: username', '/var/www/sites/ththgove1701/current/vendor/adldap2/adldap2-laravel/src/Auth/Resolver.php', 46, array('credentials' => array('email' => null, 'password' => null))) in Resolver.php line 46
    at Resolver->byCredentials(array('email' => null, 'password' => null)) in DatabaseUserProvider.php line 105
    at DatabaseUserProvider->retrieveByCredentials(array('email' => null, 'password' => null)) in SessionGuard.php line 255
    at SessionGuard->validate(array('email' => null, 'password' => null)) in SessionGuard.php line 221
    at SessionGuard->once(array('email' => null, 'password' => null)) in SessionGuard.php line 294
    at SessionGuard->onceBasic() in AuthManager.php line 294
    at AuthManager->__call('onceBasic', array()) in Facade.php line 221
    at AuthManager->onceBasic() in Facade.php line 221
    at Facade::__callStatic('onceBasic', array()) in AuthenticateOnceWithBasicAuth.php line 19
    at Auth::onceBasic() in AuthenticateOnceWithBasicAuth.php line 19
    at AuthenticateOnceWithBasicAuth->handle(object(Request), object(Closure)) in Pipeline.php line 148
    at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
    at Pipeline->Illuminate\Routing\{closure}(object(Request)) in SubstituteBindings.php line 41
    at SubstituteBindings->handle(object(Request), object(Closure)) in Pipeline.php line 148
    at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
    at Pipeline->Illuminate\Routing\{closure}(object(Request)) in ThrottleRequests.php line 49
    at ThrottleRequests->handle(object(Request), object(Closure), '60', '1') in Pipeline.php line 148
    at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
    at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 102
    at Pipeline->then(object(Closure)) in Router.php line 561
    at Router->runRouteWithinStack(object(Route), object(Request)) in Router.php line 520
    at Router->dispatchToRoute(object(Request)) in Router.php line 498
    at Router->dispatch(object(Request)) in Kernel.php line 174
    at Kernel->Illuminate\Foundation\Http\{closure}(object(Request)) in Pipeline.php line 30
    at Pipeline->Illuminate\Routing\{closure}(object(Request)) in TransformsRequest.php line 30
    at TransformsRequest->handle(object(Request), object(Closure)) in Pipeline.php line 148
    at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
    at Pipeline->Illuminate\Routing\{closure}(object(Request)) in TransformsRequest.php line 30
    at TransformsRequest->handle(object(Request), object(Closure)) in Pipeline.php line 148
    at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
    at Pipeline->Illuminate\Routing\{closure}(object(Request)) in ValidatePostSize.php line 27
    at ValidatePostSize->handle(object(Request), object(Closure)) in Pipeline.php line 148
    at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
    at Pipeline->Illuminate\Routing\{closure}(object(Request)) in CheckForMaintenanceMode.php line 46
    at CheckForMaintenanceMode->handle(object(Request), object(Closure)) in Pipeline.php line 148
    at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
    at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 102
    at Pipeline->then(object(Closure)) in Kernel.php line 149
    at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 116
    at Kernel->handle(object(Request)) in index.php line 54


[Model] Add support for 'gte-Qwen2' embedding models

FIX #6015
FIX #5827
FIX #5611
FIX #5600
This should work for Alibaba-NLP/gte-Qwen2-7B-instruct and Alibaba-NLP/gte-Qwen2-1.5B-instruct
You can serve OpenAI compatible API with:
python -m vllm.entrypoints.openai.api_server \
  --served-model-name gte-Qwen2-7B-instruct \
  --model Alibaba-NLP/gte-Qwen2-7B-instruct \
  --dtype bfloat16 \
  --trust-remote-code

However, the current version has a consistency issue of embeddings, which means it can not pass the following test. It should be fixed before merging.
pytest tests/models/test_embedding.py

# FAILED tests/models/test_embedding.py::test_models[half-Alibaba-NLP/gte-Qwen2-7B-instruct] - AssertionError: Not all values are within 0.01 of 1.0

FIX #6015 FIX #5827 FIX #5611 FIX #5600
This should work for Alibaba-NLP/gte-Qwen2-7B-instruct and Alibaba-NLP/gte-Qwen2-1.5B-instruct
You can serve OpenAI compatible API with:
python -m vllm.entrypoints.openai.api_server \
  --served-model-name gte-Qwen2-7B-instruct \
  --model Alibaba-NLP/gte-Qwen2-7B-instruct \
  --dtype bfloat16 \
  --trust-remote-code

Is trust-remote-code required?

@zifeitong I set it according to official example. You can further check whether it is necessary.
from sentence_transformers import SentenceTransformer

model = SentenceTransformer("Alibaba-NLP/gte-Qwen2-1.5B-instruct", trust_remote_code=True)

any chance this will land in the next version?

The below issues need to be resolved. I am not very familiar with vllm and can only provide limited assistance. You can invite someone who is familiar with the field to help.

Ensure that vllm correctly recognizes embedding models, especially when there are LLMs with the same architecture (e.g. Qwen2ForCausalLM).
Pass pytest tests/models/test_embedding.py

Can this feature be merged?

Can this feature be merged?

I dont think anyone is actively working on this. current state output wrong embedding value so no

The gte-Qwen2 model extends its attention to bi-directional. I think we need to pass causal=False to attn_func like what they did when evaluating the model.
https://huggingface.co/Alibaba-NLP/gte-Qwen2-1.5B-instruct/blob/main/scripts/eval_mteb.py#L553

The gte-Qwen2 model extends its attention to bi-directional. I think we need to pass causal=False to attn_func like what they did when evaluating the model.
https://huggingface.co/Alibaba-NLP/gte-Qwen2-1.5B-instruct/blob/main/scripts/eval_mteb.py#L553

It may cause flash_attn NotImplementedError.

@Nickydusk In a word, oo enable bi-directional attention of gte-qwen2, we need to pass causal=False from vllm to flash_attention. And currently, there is no name_arg for causal, all is hardcoded as causal=True for the decoding-only model.
vllm  caulse=True is hard coded in here:
https://github.com/vllm-project/vllm/blob/82a1b1a82b1fbb454c82a9ef95730b929c9b270c/vllm/attention/backends/flash_attn.py#L528
flash attention has causal=False as default though:
https://github.com/Dao-AILab/flash-attention/blob/3f6ff1c1c52fa3d148b502e465ffb7bc88f7a50e/hopper/flash_attn_interface.py#L259

@Nickydusk Hey Nick. I think that you should not use Qwen2ForCausalLM in the embedding model. Since:
In [2]: model
Out[2]:
SentenceTransformer(
  (0): Transformer({'max_seq_length': 8192, 'do_lower_case': False}) with Transformer model: Qwen2Model
  (1): Pooling({'word_embedding_dimension': 3584, 'pooling_mode_cls_token': False, 'pooling_mode_mean_tokens': False, 'pooling_mode_max_tokens': False, 'pooling_mode_mean_sqrt_len_tokens': False, 'pooling_mode_weightedmean_tokens': False, 'pooling_mode_lasttoken': True, 'include_prompt': True})
  (2): Normalize()
)

The embedding model should be Qwen2Model.

Sorry for the previous wrong statement. You could check my PR on how I am supporting the gte-qwen2 in SGLang:
https://github.com/sgl-project/sglang/pull/1186

how to run with gguf ？
gte-qwen2-7b-instruct-q5_k_m.gguf


Added import for upstream package change

PR to be merged shortly after a change to upstream neo4j, so that apoc continues to compile after a package rename.
Proposed Changes
Renames an import in DbmsRule as the class in question has been moved in upstream neo4j.

@nadja-muller and @ncordon reopened with a branch on the main repo. However this will not go green until an upstream merge is made (remote aliases). How would you like to proceed?


add nlp.relation

It's good to add all the annotators or api from CoreNLP. If possible, add nlp.relation.
Other python wrapper doesn't support relation extractor either.

Currently i m looking same with this model do we have other possibilities for extract relationship? @suisuiwudi


Please run afl-fuzz against yosys

This is a placeholder for me or someone else to run afl-fuzz on yosys and report problems found.
@q3k has it on his todo list, but maybe someone else gets there first?

This has been indeed on my backlog for ages now, but I'm still not exactly sure how to approach fuzzing Yosys.
The main problem comes from the fact that AFL (and all other fuzzing tools that I've seen) expects to manipulate just one stream of data, while the typical Yosys job will be composed of at least one script file, and one or more sources of possibly different types.
One approach would be to fuzz the script input and frontends independently, but this would surely not be exhaustive when it comes to complex bugs.
An alternative would be to 'encode' a script and multiple sources into a single stream (delimited somehow, with their file names encoded as well). If we seed this properly, this might work.
I'm thinking of a combination of the two: first independently fuzz the script interpreter, then then Verilog frontend, and then maybe the JSON/BLIF frontends. After that, proceed with the encoded-multi-file approach.
I'd love to hear other opinions on this.

@jeremysalwen did that in 2015 and found a series of parser bugs and even some things much deeper in Yosys, for example #95 and #96. I agree it would be a good idea to repeat that, maybe even make it part of CI in some way.

Should this be switched from afl to afl++?
There seem to be a large number of improvements with coverage and speed, especially with the clang++ LTO backend.


Program terminates abnormally with no error message only in JIT mode, but not on AOT/exe

Dart SDK Version (dart --version):
Dart SDK version: 2.19.0-374.1.beta (beta) (Tue Nov 15 07:44:30 2022 -0800) on "windows_x64"
I encontoured a rather confusing issue when running a Dart command line program. When running with dart run in the terminal the program just stops prematurely without any error message. I then tried running in debug mode using Android Studio, and while there is still no error nor exception, the following message emitted in the middle of the execution:
Process finished with exit code -2147483645
At first, I was unable to find out where the program ends with breakpoints (the program finishes successfully). Then I tried removing the breakpoints and insert various print messages here and there. After that I am quite sure there's something wrong with the execution because the program can stop at a very strange place, like after the if statement and before the for statement. The program processes a set of data in a loop, but it isn't any particular data that causes the program to stop - the data that fails runs fine if I move it to the first of the loop.
Some additional observations:

I sometimes cannot reproduce the issue when running line by line using breakpoint.
Sometimes the termination can be reproduced when breakpoint is in effect but the line of the termination point is different from when running without the breakpoint. Still no error message, just a Process finished with exit code -2147483645.
The program runs normally when it is compiled to an executable, or to an aot snapshot and run it with dartaotruntime. But it fails similarly when compiled to jit and execute.

I am willing to contribute the code privately for reproducting this issue, but I need some time to remove the unnecessary noises (it's still under development). The code is about detecting overlapping curves by using the intersectionsWithCurve() function in the beizer package. The function will be called multiple times in a loop. While I suppose this is a relatively heavy computation, memory should not be a program (I have over 30GB free memory), and the program stops very early.
I know there isn't very much useful information provided, please see if you can provide any suggestion on the issue I encountered. Thank you!

Loosely triaging this to VM...

Here is an update to the issue I have reported. The code that reproduces this issue can be found at https://github.com/tamcy/dart-issue-50622.
When running on Docker, I can see the following crash message emitted in the middle:
===== CRASH =====
si_signo=Trace/breakpoint trap(5), si_code=128, si_addr=(nil)
version=2.19.0-374.2.beta (beta) (Wed Nov 23 12:15:38 2022 +0000) on "linux_x64"
pid=1, thread=16, isolate_group=main(0x55b312372000), isolate=main(0x55b312465000)
os=linux, arch=x64, comp=no, sim=no
isolate_instructions=55b30fb16360, vm_instructions=55b30fb16360
  pc 0x00007fde15b4d947 fp 0x00007fde1137e050 Unknown symbol
  pc 0x00007fde15b38ad8 fp 0x00007fde1137e0e8 Unknown symbol
  pc 0x00007fde15b37e5b fp 0x00007fde1137e178 Unknown symbol
  pc 0x00007fde15b2d9af fp 0x00007fde1137e1e8 Unknown symbol
  pc 0x00007fde15b2d7f7 fp 0x00007fde1137e260 Unknown symbol
  pc 0x00007fde15b2d34c fp 0x00007fde1137e2c0 Unknown symbol
  pc 0x00007fde15b2c32b fp 0x00007fde1137e310 Unknown symbol
  pc 0x00007fde15b23dd2 fp 0x00007fde1137e350 Unknown symbol
  pc 0x00007fde15b23c0c fp 0x00007fde1137e380 Unknown symbol
  pc 0x00007fde15b22d9f fp 0x00007fde1137e3e8 Unknown symbol
  pc 0x00007fde15b2303b fp 0x00007fde1137e428 Unknown symbol
  pc 0x00007fde15b22d9f fp 0x00007fde1137e490 Unknown symbol
  pc 0x00007fde15b225e1 fp 0x00007fde1137e4e8 Unknown symbol
  pc 0x00007fde1c402ffc fp 0x00007fde1137e560 Unknown symbol
  pc 0x000055b30fc8a949 fp 0x00007fde1137e600 dart::DartEntry::InvokeCode(dart::Code const&, unsigned long, dart::Array const&, dart::Array const&, dart::Thread*)+0x139
  pc 0x000055b30fc8a7c5 fp 0x00007fde1137e660 dart::DartEntry::InvokeFunction(dart::Function const&, dart::Array const&, dart::Array const&, unsigned long)+0x145
  pc 0x000055b30fc8cb24 fp 0x00007fde1137e6a0 dart::DartLibraryCalls::HandleMessage(long, dart::Instance const&)+0x144
  pc 0x000055b30fcafe58 fp 0x00007fde1137ec30 dart::IsolateMessageHandler::HandleMessage(std::__2::unique_ptr<dart::Message, std::__2::default_delete<dart::Message>>)+0x348
  pc 0x000055b30fcd808a fp 0x00007fde1137ecb0 dart::MessageHandler::HandleMessages(dart::MonitorLocker*, bool, bool)+0x15a
  pc 0x000055b30fcd87ab fp 0x00007fde1137ed00 dart::MessageHandler::TaskCallback()+0x1db
  pc 0x000055b30fdfdb9b fp 0x00007fde1137ed80 dart::ThreadPool::WorkerLoop(dart::ThreadPool::Worker*)+0x13b
  pc 0x000055b30fdfdfe8 fp 0x00007fde1137edb0 dart::ThreadPool::Worker::Main(unsigned long)+0x78
  pc 0x000055b30fd74906 fp 0x00007fde1137ee70 dart+0x21b4906
-- End of DumpStackTrace

When running on Windows in debug mode, the code terminates with exit code -2147483645. No error
message is printed. But occasionally a message similar to the following is emitted (yet I counter the message just once in tens of the executions):
Error: Failed to send request: {"jsonrpc":"2.0","id":"9","method":"setExceptionPauseMode","params":{"isolateId":"isolates/3021199231267163","mode":"Unhandled"}}

I have tested the code against several versions on Docker. The code crashes on dart:2.19.0-374.2.beta, dart:2.19.0-374.1.beta, dart:2.19.0-255.2.beta and dart:2.19.0-146.1.beta. It runs fine on dart:2.18.5.

@tamcy please add some open-source license of your choice to the repo, otherwise we can't look at it. (e.g. MIT, BSD, GPL, LGPL, Apache, etc.).

@mraleph No problem, MIT license file added to the repo.

Would you have some free cycles to take a look at this @aam?

dart --optimization-counter-threshold=2 bin/demo.dart seems to reliably reproduce the breakpoint trap which is inserted by https://github.com/dart-lang/sdk/blob/main/runtime/vm/compiler/backend/flow_graph.cc#L1950.
There somehow seems to be a  confusion whlie optimizing/inlining package:bezier/src/bezier.dart_Bezier__nonOverlappingNormalVectorAt where UnboxedFloat is attempted to be converted to UnboxedDouble

Problem seems to be with https://github.com/dart-lang/sdk/blob/main/runtime/vm/compiler/backend/flow_graph.cc#L2056 where PhiInstruction gets unboxed double representation even if incoming values are unboxed floats.
Something like https://dart-review.googlesource.com/c/sdk/+/275180 seems to fix the crash.

Makes sense. The fix should be fixed-point based, because you can have float phis flowing into other phis. So it should likely follow the same template that HasUnboxedIncomingValue has.


Non-working Link in section "Update Existing Apps"

In this page, section "Update Existing Apps", the bullet that begins with the link "Install the Windows App SDK NuGet package" leads to a 404-Page Not found when that link is clicked.

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

ID: 25fcf332-0c1b-7c10-406d-bfd5f6eef952
Version Independent ID: d2b12b06-f936-204b-1be7-9ca844ceff43
Content: Overview of app development options - Windows apps
Content Source: hub/apps/get-started/index.md
Product: uwp
Technology: apps
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Thanks for letting us know. This link has been fixed - it should point to https://docs.microsoft.com/en-us/windows/apps/windows-app-sdk/use-windows-app-sdk-in-existing-project.


Support \nonumber and \notag

Is your feature request related to a problem? Please describe.
Now that we have equation numbering, it would be nice to add \nonumber (or the AMS equivalent, \notag), so that we can only number some rows of an {align} environment (for example).
Describe the solution you'd like:
Hopefully not too hard: \nonumber just prevents the eqn-num generation for that row.
Link to or name of a (La)TeX package that provides the same feature:
It's in AMSMath, as well as base LaTeX (for eqnarray, which we don't yet support).

Hmm, perhaps I should try to implement \tag-per-row at the same time; this should be doable with a similar (if not the same) mechanism.


Add support for 3840x2560 3:2 monitor

Similar to https://github.com/utmapp/UTM/issues/4668 I want to add an additional monitor resolution to the Apple Hypervisor.
In file: https://github.com/utmapp/UTM/blob/main/Platform/macOS/VMConfigAppleDisplayView.swift
Just add the additional NamedResolution to resolutions list
NamedResolution(name: "3840 × 2560 — 4K+ 3:2", resolution: Resolution(width: 3840, height: 2560))

Added support for custom resolutions in https://github.com/utmapp/UTM/commit/3a79d06f729fff7b55d58e145a93a936530128e6


Feature Request: Support http://search.bsky.social/search/profiles?q=bluesky

1. Problem to solve

2. Proposal

3. More information

To be implemented when Lexicon is completed.


Pre-loading quicker to avoid user seeing the loading icon at the bottom

I thought at first the setting scrollThreshold might help but it doesn't seem to make much difference for me.
I thought also this suggestion may be useful but it doesn'ts eem to work on the current codebase.
Anyone have any suggestions for making the library load more results earlier?

Didn't see that solution there, thanks. I had a similar question and implemented a timer to trigger nextLoads to front load a couple more pages. Super hacky and dirty solution, but working.
There are some issues with browser back though. For ex. if the user browses back before the requests being made are finished on returning to the page infinite scroll won't fetch more requests (seems to be a safari issue only related to bfcache).
Logic: Initial load is 10 posts, then another trigger in js after page load every x seconds. Timer ends on last page. Works in my case since we have a finite amount of posts that will never be too large (100 etc.)
`  // Auto load using timed throttle
$container.on( 'append.infiniteScroll', tttjAutoLoad );
function tttjAutoLoad() {
let myInterval = setInterval(myTimer, 250);

function myTimer() {
  // document.getElementById('view-more-button').click();
  $container.infiniteScroll('loadNextPage');
  // document.getElementById("demo").innerHTML += "Hello";

}

function myStopFunction() {
  clearInterval(myInterval);
}

$container.on( 'last.infiniteScroll', myStopFunction );

};
});`


unread notification even after there is no new notification

Describe the bug
the notification icon on the home page always shows new notification, even if there is no new notification.
To Reproduce

[1] Yes

[x] No

Other:

Describe the bug
the notification icon on the home page always shows new notification, even if there is no new notification.
To Reproduce

[1] Yes
[x]  No
Other:

can you please describe this issue more like attach a screenshot or something


Filling search/hidden inputs

Thanks @s-belichenko-sold.

Hi @s-belichenko-sold.
This feature is available in:
.

Fix navigation to invalid pages.
Filling search/hidden inputs.
Search and filter case insensitive.
Support for Bootstrap 3.3.7.
Update license to MIT.

Regards.


Fix editable install of meson-python for pip 23.2.1 and python3.11

With python 3.11 and pip 23.2.1, an editable install of meson-python itself fails with:
pip._internal.exceptions.InstallationSubprocessError: Checking if build backend supports build_editable exited with 1
This is due to lacking an implementation of PEP660's prepare_metadata_for_build_editable.

Still getting an issue in my environment that _meson_python_editable_loader.py is created with the ninja execitable set to the one installed in the temporary build enviroment created by pip.
Not sure why I'm hitting this but the tests aren't!

How did you determine that the lack of the prepare_metadata_for_build_editable() hook is the source of your problems?
Everything works as expected for me:
$ python --version
Python 3.11.4
$ python -m pip --version
pip 23.2.1 from /Users/daniele/src/meson-python/venv/lib/python3.11/site-packages/pip (python 3.11)
$ python -m pip install --no-build-isolation -e .
Obtaining file:///Users/daniele/src/meson-python
  Checking if build backend supports build_editable ... done
  Preparing editable metadata (pyproject.toml) ... done
Requirement already satisfied: meson>=0.63.3 in ./venv/lib/python3.11/site-packages (from meson-python==0.14.0.dev0) (1.2.1)
Requirement already satisfied: pyproject-metadata>=0.7.1 in ./venv/lib/python3.11/site-packages (from meson-python==0.14.0.dev0) (0.7.1)
Requirement already satisfied: packaging>=19.0 in ./venv/lib/python3.11/site-packages (from pyproject-metadata>=0.7.1->meson-python==0.14.0.dev0) (23.1)
Building wheels for collected packages: meson-python
  Building editable for meson-python (pyproject.toml) ... done
  Created wheel for meson-python: filename=meson_python-0.14.0.dev0-py3-none-any.whl size=17632 sha256=43e997401297472d0d9edb03577b91f7f40637b2c3e100f2bb9f475020a4e599
  Stored in directory: /private/var/folders/6f/4l4d5n9x1y71cxyrprf_x35c0000gn/T/pip-ephem-wheel-cache-s0rqdxog/wheels/91/8a/31/237de59d176b9ce63c1dcd55038958a1238f2fd5f1c0cd7506
Successfully built meson-python
Installing collected packages: meson-python
  Attempting uninstall: meson-python
    Found existing installation: meson-python 0.14.0.dev0
    Uninstalling meson-python-0.14.0.dev0:
      Successfully uninstalled meson-python-0.14.0.dev0
Successfully installed meson-python-0.14.0.dev0

Still getting an issue in my environment that _meson_python_editable_loader.py is created with the ninja execitable set to the one installed in the temporary build enviroment created by pip.
Not sure why I'm hitting this but the tests aren't!

As I already answered here https://github.com/mesonbuild/meson-python/discussions/425#discussioncomment-6897683, you are installing in editable mode without disabling build isolation, and you don't have a suitable ninja installed.

How did you determine that the lack of the prepare_metadata_for_build_editable() hook is the source of your problems?
Everything works as expected for me:
$ python --version
Python 3.11.4
$ python -m pip --version
pip 23.2.1 from /Users/daniele/src/meson-python/venv/lib/python3.11/site-packages/pip (python 3.11)
$ python -m pip install --no-build-isolation -e .
Obtaining file:///Users/daniele/src/meson-python
  Checking if build backend supports build_editable ... done
  Preparing editable metadata (pyproject.toml) ... done
Requirement already satisfied: meson>=0.63.3 in ./venv/lib/python3.11/site-packages (from meson-python==0.14.0.dev0) (1.2.1)
Requirement already satisfied: pyproject-metadata>=0.7.1 in ./venv/lib/python3.11/site-packages (from meson-python==0.14.0.dev0) (0.7.1)
Requirement already satisfied: packaging>=19.0 in ./venv/lib/python3.11/site-packages (from pyproject-metadata>=0.7.1->meson-python==0.14.0.dev0) (23.1)
Building wheels for collected packages: meson-python
  Building editable for meson-python (pyproject.toml) ... done
  Created wheel for meson-python: filename=meson_python-0.14.0.dev0-py3-none-any.whl size=17632 sha256=43e997401297472d0d9edb03577b91f7f40637b2c3e100f2bb9f475020a4e599
  Stored in directory: /private/var/folders/6f/4l4d5n9x1y71cxyrprf_x35c0000gn/T/pip-ephem-wheel-cache-s0rqdxog/wheels/91/8a/31/237de59d176b9ce63c1dcd55038958a1238f2fd5f1c0cd7506
Successfully built meson-python
Installing collected packages: meson-python
  Attempting uninstall: meson-python
    Found existing installation: meson-python 0.14.0.dev0
    Uninstalling meson-python-0.14.0.dev0:
      Successfully uninstalled meson-python-0.14.0.dev0
Successfully installed meson-python-0.14.0.dev0

using pip install -vvv
If you try running the test changes in this patchset without the fixes, you'll see the issues...

I just tried python -m pip install -vvv -e . in the meson-python source directory and everything still works as expected, just with more output. Can you please be more specific about how that fails for you?

Still getting an issue in my environment that _meson_python_editable_loader.py is created with the ninja execitable set to the one installed in the temporary build enviroment created by pip.
Not sure why I'm hitting this but the tests aren't!

As I already answered here #425 (reply in thread), you are installing in editable mode without disabling build isolation, and you don't have a suitable ninja installed.

Ahh, ok I had definitely missed how-to-guides-editable-installs.
The patch here seems to fix the need for this in projects using meson-python - at least that's what the test set shows, maybe there's a corner case i'm missing?
Still need --no-build-isolation for meson-python itself, as the pip environment results in the installed ninja failing to work (the stub fails on from ninja import ninja). I wonder if we can do something so folk don't get caught out, and add a note in contributing?

The tests work just because ninja is provided by the system on the test runners. It would fail the same with or without these patches if it would not be present, as yourself pointed out happens on your system. How to install an editable wheel for a project using meson-python is documented. I don't think we need to repeat that. Also, an editable (or otherwise) installation of meson-python is not required to work, debug, or test meson-python.

Can you please describe what this PR achieves and why we need it? PEP 660 states that the prepare_metadata_for_build_editable is optional and the front end should fall back to build_editable if it is missing. In the case of meson-python, providing accurate metadata requires building the project, thus there is no advantage in providing prepare_metadata_for_build_editable over build_editable as the former would need to do all the work the latter does, except packaging the build artifacts in a zip file, which is negligible work compared to the rest.

I was actually going to close that issue as wontfix over the weekend. To produce metadata we need to know if the wheel is pure or not (ie if it included platform or python version specific content) and for doing that we need to configure, and in some occasions, actually build the project.
PEP 660 is not clear on what prepare_metadata_for_build_editable is used for by the front-end. I fear that some front-ends may execute prepare_metadata_for_build_editable before build_editable, just in case. This would result in meson-python doing work twice for no good reason. Lacking an use case requiring prepare_metadata_for_build_editable (or prepare_metadata_for_build_wheel as requested in #236), I much prefer not to implement these methods.

The tests work just because ninja is provided by the system on the test runners. It would fail the same with or without these patches if it would not be present, as yourself pointed out happens on your system. How to install an editable wheel for a project using meson-python is documented. I don't think we need to repeat that. Also, an editable (or otherwise) installation of meson-python is not required to work, debug, or test meson-python.

To check this, i've added the test set on top of main:
https://github.com/robtaylor/meson-python/actions/runs/6074445627
CF: test set with the two other commits:
https://github.com/robtaylor/meson-python/actions/runs/6071530369
So I do feel like I'm still not understanding the objection overall here - does feel like i've stepped in a hornets nest, and I'm not attached to whether this is closed DONTFIX! I could just be sleep deprived though. ><

No problem. I'm happy that we can agree that there is nothing to fix.
However, for the future, I recommend you to be mindful of the fact that the project maintainers have most likely spend much more time than you thinking about an issue and almost certainly have a better understanding of the context in which a project is developed. Providing a concise and punctual description of the issue you have encountered and of the reason why you are proposing a patch. Having to argue the position of the maintainers, especially when the opposing position is not well defined, is very tiring. This talk does a very good job at describing the issue https://www.youtube.com/watch?v=o_4EX4dPppA

The patches that are useful here are the patch for breakage on 3.7/3.8 and
the test set for meson-python editable install

We have tests for the editable wheels functionality, for example https://github.com/mesonbuild/meson-python/blob/main/tests/test_editable.py#L170 They run also on Python 3.7 and 3.8 thus I doubt there is anything to fix there.

Thanks @robtaylor, glad there was no problem in the end.

I was actually going to close that issue as wontfix over the weekend. [...]

Thanks for those thoughts @dnicolodi - I'll follow up on that issue now with a reply to your concerns.


Backporting http parser upgrade?

Given https://github.com/httprb/http/pull/489/files is in 5.0, and wanting to hold 5.0 until there is more (https://github.com/httprb/http/issues/564). Is there a reason to not backport the HTTP Parser upgrade?
The changes look simple enough I don't see a reason we can't just do a 4.1.2 to unblock fixing the Upgrade issue, without having to require moving to a pre version.
I'm happy to do  the port, just checking ahead of time. I'll be going on vacation later today so I can do it when I'm back in ~10 days.

@zanker-stripe Sure. I guess we should release backport as minor bump though. I'll try to tackle that during next week, but I'll be fine if anybody will beat me to it. :D

Hello!  Just a friendly bump and +1 so this doesn't get lost. 🙂

Done (#570) and released as v4.2.0

This "upgrade" appears to pull in a different gem entirely, and the http-parser gem doesn't appear to work on JRuby. We just pulled in http v4.2.0 and got this error:

Resolving dependencies.............
Fetching rake 13.0.0
Installing rake 13.0.0
Fetching public_suffix 4.0.1
Installing public_suffix 4.0.1
Fetching addressable 2.7.0
Installing addressable 2.7.0
Using ast 2.4.0
Using bundler 1.17.3
Fetching childprocess 3.0.0
Installing childprocess 3.0.0
Using safe_yaml 1.0.5
Using crack 0.4.3
Using diff-lcs 1.3
Using docile 1.3.2
Using unf 0.1.4 (java)
Using domain_name 0.5.20190701
Using fakefs 0.20.1
Fetching ffi 1.11.1 (java)
Installing ffi 1.11.1 (java)
Fetching ffi-compiler 1.0.1
Installing ffi-compiler 1.0.1
Using hashdiff 1.0.0
Using http-cookie 1.0.3
Using http-form_data 2.1.1
Fetching http-parser 1.2.1
Installing http-parser 1.2.1 with native extensions
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.
current directory:
/opt/atlassian/pipelines/agent/build/.gems/gems/http-parser-1.2.1/ext
/usr/lib/jvm/java-11-openjdk-amd64/bin/java -cp
:/root/.m2/repository/org/jruby/jruby-complete/<IP_ADDRESS>/jruby-complete-<IP_ADDRESS>.jar
org.jruby.Main -rrubygems
/opt/atlassian/pipelines/agent/build/.gems/gems/rake-13.0.0/exe/rake
RUBYARCHDIR=/opt/atlassian/pipelines/agent/build/.gems/extensions/universal-java-11/2.5.0/http-parser-1.2.1
RUBYLIBDIR=/opt/atlassian/pipelines/agent/build/.gems/extensions/universal-java-11/2.5.0/http-parser-1.2.1
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by org.jruby.util.SecurityHelper
(file:/root/.m2/repository/org/jruby/jruby-complete/<IP_ADDRESS>/jruby-complete-<IP_ADDRESS>.jar)
to field java.lang.reflect.Field.modifiers
WARNING: Please consider reporting this to the maintainers of
org.jruby.util.SecurityHelper
WARNING: Use --illegal-access=warn to enable warnings of further illegal
reflective access operations
WARNING: All illegal access operations will be denied in a future release
mkdir -p x86_64-linux/http-parser
cc -fexceptions -O -fno-omit-frame-pointer -fno-strict-aliasing -Wall -Wextra
-O3 -fPIC   -o x86_64-linux/http-parser/http_parser.o -c
./http-parser/http_parser.c
rake aborted!
Command failed with status (127): [cc -fexceptions -O
-fno-omit-frame-pointer...]
/opt/atlassian/pipelines/agent/build/.gems/gems/ffi-compiler-1.0.1/lib/ffi-compiler/compile_task.rb:133:in
block in define_task!' /opt/atlassian/pipelines/agent/build/.gems/gems/rake-13.0.0/exe/rake:27:in '
Tasks: TOP => default => x86_64-linux/libhttp-parser-ext.so =>
x86_64-linux/http-parser/http_parser.o
(See full trace by running task with --trace)
rake failed, exit code 1
Gem files will remain installed in
/opt/atlassian/pipelines/agent/build/.gems/gems/http-parser-1.2.1 for
inspection.
Results logged to
/opt/atlassian/pipelines/agent/build/.gems/extensions/universal-java-11/2.5.0/http-parser-1.2.1/gem_make.out
An error occurred while installing http-parser (1.2.1), and Bundler cannot
continue.
Make sure that gem install http-parser -v '1.2.1 succeeds before bundling.
In Gemfile:
http was resolved to 4.2.0, which depends on
http-parser

@jessicaspeir oof, that's unfortunate, and seems like a semver breakage
@ixti what do you think we should do?

Yes, we changed underlying http parser, but it's implementation details and should not be considered breaking change IMO. But this change is big enough that's why we bumped minor version. New parser is fully compatible and tested with JRuby (both parser and http gem are).
I believe it might be related to system gems (similar to #572).

The new http-parser works on the jruby images I've tried, but it needs a c compiler and rake. apt install build-essential or yum install gcc and gem install rake makes it compile just fine

Its good to know that it is working for others. I have limited knowledge of how jruby works, but it seems very odd that to install a gem with jruby I would need a C compiler. I've never encountered that before in ~4 years of development on a jruby platform.

@jessicaspeir previous backend had java version, so that it was compiled for java, but its development was stalled, and thus was not working correctly with Upgrade: h2,h2c headers. We switched to use different parser library that is written as FFI binding to underlying C HTTP parser. Naturally it requires C compiler now.
The only difference in HTTP 4.2.0 from previous version is exactly this parser change. If you prefer not to have C compiler, you may safely stay at previous version of http gem.

Thanks for all the help! I was able to get the build working in our Linux build container, so that's great. Unfortunately we also have a requirement to be able to build on Windows, and I'm getting the same "Failed to build native extensions" error. Any thoughts?

@jessicaspeir sorry took me a while to respond. Unfortunately I don't have Windows machine to reproduce, so I can't help here. Sorry.


About Semantic Attacks Against Vicuna

Hi!
Thanks for your nice work!
One small question is that when I visited your demo site, and I choose "Vicuna" + "MNLI" + “Semantic” + “zero-shot task”, the site return nothing. Therefore, I am confused and  could you please replenish these adversarial prompts? Many thanks!
Best wishes

Hi,
Thank you for bringing this error to our attention! We will update the attack results shortly.

Hi!
Thank you very much!
I also noticed that when I selected model as Vicuna, and selected dataset squad_v2，UN Multi, and math, there are also not adversarial prompts.

Yes, Vicuna performs much worse on these datasets, we choose not to evaluate Vicuna on these datasets.


QSL sends a bunch of login plugin packets all at once instead of doing proper handshakes

Apparently QSL inherits the ServerLoginConnectionEvents.QUERY_START stuff from FAPI. What it should be doing is sending requests and responses one at a time instead of all at once.

So the problem that's being referenced here is in a blanketcon-specific optimization where they use zstd compression on the networking stack. With the current behavior, the server can prepare uncompressed packets before the client's reply is handled. This means the client expects zstd-compressed packets and receives the uncompressed packets instead.
In other words,
The QSL networking API is based on the assumption that login query packets do not have side-effects. Is that a reasonable assumption, (and the blanketcon usecase a one-in-a-million exception), or should we change the QSL networking API to send and handle login query packets one at a time?


Filters to work in iframe

Filters to work in iframe (such as severe accidents)

Actually it turns out that this is already working. It's probably implemented here. I still need to figure out why didn't it work for rsa, but that's another issue.

Thanks! I updated Gal we can work with Or Yarok.


Repo sync for protected CLA branch

The pull request is created from main638086267031544795sync_temp to main to fix git push error for protected CLA branch

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


[RUNTIME][API] Graph runtime API enahncement to support NDArray

get_output, get_input : Second arg is optional and returns NDArray if not passed.
get_output_count, get_input_count : New API to query number of outputs and inputs to graph.

Thanks for contributing to TVM!   Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from others in the community.

@tqchen  ready for review.

get_output, get_input api made compatible to return NDArray of second argument is not passed.
get_input_count, get_output_count new API added.
Added strict shape check for empty array passed for get_input and get_output.

Above check revealed few instances listed below which require fix.

reduction test cases doing a null array comparison instead of scalar - fixed
Keras test cases receiving NCHW layout from TVM and comparing with NHWC output ref data - fixed
LSTM and stridedSlice has similar issue with shape in compatibility - #1662 to follow up and disabled now.

@Huyuwei , @masahi & @joyalbin welcome to review and comment.

@srkreddy1238 I think it is better to implement GetInputAsNDArray using GetInputAsDLTensor or vice versa. The same goes for GetOutput. Is that possible?

@masahi  "GetInputAsNDArray using GetInputAsDLTensor" ??
Can you details it ?

I mean, implement GetInputAsNDArray by redirecting to GetInputAsDLTensor, since two functions are similar.

I'm not familiar with NDArray, so if that is not a good idea then that's fine.

Common code here only entry id finding from index. One returns an NDArray and other is return value by argument.

The current set of changes is not what we intended. Currently, most of the data are backed by DLTensor and NDArray is copied and returned during the query function. What we really want, is to actually use NDArray in most places that DLTensor is getting used, and directly return NDArray of the corresponding value.
Useful API

https://github.com/dmlc/tvm/blob/master/include/tvm/runtime/ndarray.h#L133
https://github.com/dmlc/tvm/blob/master/include/tvm/runtime/ndarray.h#L91

I think NDArray version of GetOutput and GetInput should return views. Current PR returns copy. If user wants a copy, they can use another APIs to do copy themselves.

@masahi  & @yzhliu Please review.

@tqchen  review pls.

@yzhliu  please https://docs.tvm.ai/contribute/code_review.html#approve-and-request-changes-explicitly

@yzhliu  & @tqchen can we conclude on this PR ?
We have too many open PR on graph_runtime (heterogeneous execution and tvmdbg)


Split parallel tests and recycle workers to tame memory usage

Tests are chunked to reduce memory pressure during parallel testing.
Seems to speed up TravisCI testing, indicating that our tests are swapping in the build env.

Codecov Report

Merging #2291 into master will decrease coverage by -0.01%.
The diff coverage is 62.5%.

@@            Coverage Diff             @@
##           master    #2291      +/-   ##
==========================================
- Coverage   86.63%   86.63%   -0.01%
==========================================
  Files         314      314
  Lines       58166    58170       +4
  Branches     6012     6014       +2
==========================================
+ Hits        50394    50397       +3
- Misses       6813     6814       +1
  Partials      959      959

Continue to review full report at Codecov.

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

This is revealing some access violation on windows.  I'm checking.

Ok, it's really coming from MCJIT removeModule leak.  Disabling it fixes the access violations.

This patch requires #2294 to run properly on windows.  The access violation can be observed on other platforms but with different test combinations.


🥳 added posix support

Added support for other os [posix based]

@areed1192 Please do review it and lemme know.


[BUG] Bloburibuilder does not populate versionid or snapshot

Describe the bug
Providing a url of a blobs version or shapshot, e.g.
passing url into constructor of bloburibuilder https://storageaccount.blob.core.windows.net/testcontainer/testfile.txt?versionId=2020-11-17T01:12:35.3895766Z
Expected behavior
The properties of bloburibuilder, versionid, should be populated with 2020-11-17T01:12:35.3895766Z
Actual behavior (include Exception or Stack Trace)
What is the actual behavior?
the property is simply null
To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)

Instantiate bloburibuilder with url https://storageaccount.blob.core.windows.net/testcontainer/testfile.txt?versionId=2020-11-17T01:12:35.3895766Z
Look at property versionid

Environment:

package Azure.Storage.Blobs 12.8.0
Windows 10, .Net Core 3.1
Visual studio 2017

Thank you for your feedback.  Tagging and routing to the team best able to assist.

@xgithubtriage

Hi, I believe that parameter is case sensitive. If the parameter was "versionid" instead of "versionId" as you have it in your issue, the BlobUriBuilder should be able to parse the value.
What are you doing for snapshot?

Then it still looks wrong because https://storageaccount.blob.core.windows.net/testcontainer/testfile.txt?versionId=2020-11-17T01:12:35.3895766Z is as it appears from Azure portal.
Can you confirm from an api perspective what a versioned url would look like?
The snapshot urls actually work though.

@amnguye can you check with other SDKs on what they do, portal is generating versionId (https://ashishwithversioning.blob.core.windows.net/test/text.txt?versionId=2021-11-30T20:32:49.6056147Z)

@amnguye did you get a chance to check?


Dots and Arrows not showing

Hi All,
I'm running the basic HTML (with Slick included) in a PHP file and see that the "stock versions" of the dots and arrows are not coming in - rather it looks more like "invalid characters". They still work, but just wondering what the issue could be? Thanks for your help in advance!
Previously posted: https://github.com/kenwheeler/slick/issues/1115
Image attached.

Didn't see this issue before so I posted a reply to #1115. As I said there:
I had a similar issue when I was testing my page from the file system without having it on a server. After I put it on a server and loaded it from there it looked like it should.

Hmm - I already have that page on a server and previously tried it on a file system - none worked :(.

Can you recreate this in a JS Fiddle, please :) - along with an indication of which browser and OS you're using.
Regards, Si.

Lol - can't win with this one. Put together a JSFiddle, and...it works fine.
https://jsfiddle.net/xtvdjphe/3/
Here's my raw code that's leading to the screenshot I showed above - thoughts?

    <title>My Now Amazing Webpage</title>
    <link rel="stylesheet" type="text/css" href="https://cdn.jsdelivr.net/jquery.slick/1.5.0/slick.css"/>
    <link rel="stylesheet" type="text/css" href="https://cdn.jsdelivr.net/jquery.slick/1.5.0/slick-theme.css"/>

your content

your content

your content

Don't forget to define the charset in the :
<meta charset="UTF-8">
This should fix your issue.

@levymetal that was it!! Thank you for your help!

No worries :)

Ah the good old UTF-8 gotcha.
Usually the meta tag should work fine but it would be best practice to let your server handle this by setting the right encoding in the http-header.

Yes , It's "UTF-8" .. Thanks

I've also added <meta charset='utf-8'> but still having same issue.
Can anyone help?

@mojsho I'm having this problem right now. I opened the page in Firefox as someone above suggested, and I saw some errors that were not displayed in Chrome:
downloadable font: rejected by sanitizer (font-family: "slick" style:normal weight:normal stretch:normal src index:1) source: https://mywebsite.com/styles/fonts/slick.woff

downloadable font: rejected by sanitizer (font-family: "slick" style:normal weight:normal stretch:normal src index:2) source: https://mywebsite.com/styles/fonts/slick.ttf

Then I noticed that in the network request for slick.ttf and slick.woff, the response headers had Content-Type: "text/html" when they really should be "application/font-woff" and "application/x-font-ttf". Seems like the server just needs to be configured to return the right MIME types for these font files. I'll be trying this tomorrow.
Hope this helps you somewhat!

i have the same issue here and i did't find the way to show the arrows

@amine-louni Be sure to load the CSS files of slick (slick.css , slick-theme.css) in your project

Hi everybody , I have the same problem . And it is related with the UTF-8, with the <meta charset='utf-8'> is not enought but if you can put the sentence in php header('Content-Type: text/html; charset=utf-8')   the problem is solved , but , changing the charset is not a minor impact in a website, You need to analyze how this impact all the infomation that you have in databases or if you only have code html you need to convert it. to UTF-8  . I'll try to find another solution to the arrows. I'll be back if i find . regards

Hi everybody , I have the same problem . And it is related with the UTF-8, with the <meta charset='utf-8'> is not enought but if you can put the sentence in php header('Content-Type: text/html; charset=utf-8')  the problem is solved , but , changing the charset is not a minor impact in a website, You need to analyze how this impact all the infomation that you have in databases or if you only have code html you need to convert it. to UTF-8 . I'll try to find another solution to the arrows. I'll be back if i find . regards

I just solved it .
As I mentionned previously I cant put UTF-8 in all my site so I found that in the file slick-theme.css row 92 was defined
.slick-prev:before {     content: 'â†'; } [dir='rtl'] .slick-prev:before {     content: 'â†’'; }
which is the same that i saw in my web
so in another file of the library I found
 $slick-prev-character: "\2190" !default;
so I just tryed to bring the numer of the character to the css definition
.slick-prev:before {     content: '\2190'; } [dir='rtl'] .slick-prev:before {     content: '\2190'; }
Which in some way solve the UTF-8 problem which was to convert
content: 'â†';
on
content: '\2190';
Hope helps others .
Regards
Sergio

Same issue. Tried all suggestions. UTF-8 is in place, css and js are loading, everything but the dots are working as expected.
Changed the font-family to something more common like Arial, Helvetica still didn't do the trick. :/


fix build on arm64

Needed for arm Linux architectures
cc @nrabinowitz

Pull Request Test Coverage Report for Build #456
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.817%

Totals

Change from base Build #451:
0.0%

Covered Lines:
501

Relevant Lines:
507

💛  - Coveralls


required props in react-i18next

library react-i18next use HOC to inject prop function t to use this function to get traslations.
//@flow
import type {IssueTasks} from '../../types';

import React, {Component, PropTypes} from 'react';
import {translate} from 'react-i18next';

class IssueTaskPanel extends Component {
  static propTypes = {
    type: PropTypes.string.isRequired,
    issueTasks: PropTypes.array.isRequired,
  };
  props: {
    type: string,
    issueTasks: IssueTasks,
  };
  render() {
    const {t, type}  = this.props;
    return (

 {t(type)}

    );
  }
}

export default translate('issue_task_panel')(IssueTaskPanel);

But flow still complains to t prop. How to avoid that?

You need to add t to the type of your props:
props: {
    t: (x: string) => string
    type: string,
    issueTasks: IssueTasks,
  };

It's strange because I already tried this and flow starts to complains to render this component without passing required t prop but now it works.

@abonec FYI, I just added a libdef for react-i18next in flow-typed. It will allow you to type-check across the boundary of the translate HOC. https://github.com/flowtype/flow-typed/pull/505
e.g.
// @flow
import { translate } from 'react-i18next'
import type { Translator, TFunction } from 'react-i18next'

type OwnProps = { s: string }
type Props = OwnProps & { t: TFunction }

const Comp = ({ t }: Props) =>
{ t('whatever') }

const translator: Translator<OwnProps, Props> = translate('whatever')

const WrappedComp = translator(Comp)

const OtherComp = () => <Comp s={ 2 } /> // TypeError


Modify the CoRegister for Com Activation so if there are not notification handlers registered a new process will be launched every time

When registering an activator for Toast check if there are handlers registered with the Toast Notification Manager, if so then use the multi use flag in CoRegister otherwise use the single use flag to launch a new process each time.

/azp run


Not found

Every time I search for a song, I get this:
2020-10-28T19:29:01.958415+00:00 app[web.1]: 2020-10-28 19:29:01.957  INFO 32 --- [ader-2-thread-1] lavalink.server.player.AudioLoader       : No matches found
2020-10-28T19:29:01.986346+00:00 heroku[router]: at=info method=GET path="/loadtracks?identifier=ytsearch:chill%20gaming%20beats" host=lavalinkkookies.herokuapp.com request_id=867b3305-23c4-4a60-a625-9c8373f4d800 fwd="" dyno=web.1 connect=1ms service=1938ms status=200 bytes=221 protocol=http
No matches found basically
Also, you said in the README to edit the application file, how and what do you do with it?
Also, uptime monitor is down because of 401 Unauthorized, how do I fix that?

im having same issue too

having the same issue here. as far as i am aware, the 401 error is normal, it's being caused by uptimerobot having no password for authentication into the lavalink server


Test Agent Deployment Fails using v2 2017 Agents

Switched an existing on-prem Automation UI build to use 2017.1 v2 agent deployment step. Was previously working on 2017 using v1 step.
It errors saying "##[error]Error occured on 'ORBUS-AGENT-04:5985'. Details : 'The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: Cannot find path 'C:\TestAgent2017Setup\vstf_testagent.exe' because it does not exist."
The build step is set to download and upgrade the agent when it runs. I've logged in and uninstalled the 2015 agent from the target machine incase there was a conflict. I can confirm that the 2017 agent is in the "C:\TestAgent" folder as expected, which is being populated by this step.
Confused as to why it then complains about this totally different folder. Logs for this step are attached.
10_Deploy Test Agents.txt

@cytron201 are you using TFS OnPrem or VSTS?
If you are using TFS 2017 U1, This issue is fixed in Update 2 which will be released soon.
Alternatively, you can patch the task in your TFS from latest master using tfx-cli [We can guide you through the steps].
PR for ref: https://github.com/Microsoft/vsts-tasks/pull/3707/files
Sorry for the trouble.

@nigurr hi, yes it's 2017.1 on prem.
Thanks that's good news; I'd like to be able to build the latest build step from the code if you can take me through the steps please?

@nigurr The new issue seems to be the same as the one described here:
https://github.com/Microsoft/vsts-tasks/issues/3837
The work around is to RDP the machine while the agent is being installed. We have a farm of 20 UI Automation VMs, and this is therefore not really feasible. Is there a better fix available yet please?

@nigurr Reducing the target agents to one machine, and using the RDP workaround it is able to pass this step. It however fails on the Run Tests step, reporting that test files have not been copied across. Logs are attached:
11_Run Automated UI Tests.zip
I'm struggling to understand how this rolled out like this in a Production Build, is this not a common use of these steps or is VSTS the only environment tested comprehensively at this point? I understand the build step is in preview, but it's completely non functional at the moment.

@cytron201 Its inexcusable. We will consider this feedback seriously and will make sure we ship with good quality.
If you patch the task from latest master, even the second issue gets addressed automatically.

@nigurr
Yes we are trying to migrate to VS2017 CodedUI.
Thanks for the assistance - to run the latest version of the Tasks straight from Master, do i need to follow the instructions on this site: https://www.visualstudio.com/en-us/docs/integrate/extensions/develop/add-build-task?
How can i push these into TFS to replace the pre-installed versions?

Here are the steps:

Install latest version of NodeJS and install https://github.com/Microsoft/tfs-cli
Clone the Tasks repository https://github.com/Microsoft/vsts-tasks - master branch
Build the tasks by executing node make.js build --task DeployVisualStudioTestAgent from C:\Vsts-tasks\
Once the build is complete, you will notice the following file structure C:\Vsts-tasks\_build\Tasks\DeployVisualStudioTestAgent\
Then patch the tasks by following executin tfx build tasks upload --task-path C:\Vsts-tasks\_build\Tasks\DeployVisualStudioTestAgent\ --overwrite --service-url http://YourTfsServer/tfs/DefaultCollection and it will ask you for Personal Access Token which you can generate from your TFS My Account -> Security page

Closing this issue as it will be fixed and supported in TFS 2017 U2.

@nigurr
How can you close a critical case that has not been solved?
Two months later there is still not released a fix for such a critical error!

@pi3k14 For On-Prem TFS, the fix will be available via 2017 Update 2.
For VSTS customers, the fix is already rolled out.
You can continue to use 2015 Test agent as we don't have any feature changes from 2015 TA to 2017 Test agent.
Please let us know if you have any critical issue blocking. We will happy to resolve

2015 Test agent has this problem https://github.com/Microsoft/vsts-tasks/issues/4101, or are your referring to the Deploy test agent v1.0 task?

@pi3k14 can you please describe the issue you are facing?
1.using TA 2017 or TA 2015?
2.using DTA v1 or v2 task?
3.what's the exact issue you are facing?

@nigurr, I have a VS2017 testproject that I'm trying to run remotely. We have an On-Prem TFS, where I'm using the Deploy Test Agent v2 (preview) task. If I select the Visual Studio 2017 test agent for deployment it crash with this issue (#3922), if I select Visual Studio 2015 test agent for deployment it crash with issue #4101.
The Deploy Test Agent v2 (preview) task seems dead in the water.

@pi3k14, #4101 is related to Test Agent 2017. Can you please attach DTAExecutionHost log from C:\TestAgent\ folder?
You can continue to run VS 2017 tests using Test Agent 2015. You can just drop the mstest v2 adapter and the tests will run out of the box.

@nigurr
DTAExecutionHost.exe.txt version 15.111.26109.0 of DTAExecutionHost.exe

@pi3k14 thanks for the logs. Is Test Agent 2017 and Test Agent 2015 both are located in the same machine?

@nigurr  Yes, but quite frankly I don't understand the question. It is Test Agent Deploy that deploys the test agents, and yes I don't have different machines for running the tests on.

@pi3k14 Sorry for the trouble. We don't support side by side installation of TA 2015 and TA 2017.
[Temp Workaround till Update 2] If you have single test agent machine, You can manually install TA 2017 and use this machine for running the tests. The bug was there in installation of Test Agent. This can be bypassed using manual installation.
If you want to use TA 2015 and run your tests, need to uninstall TA 2017 and cleanup registry (I can help you with this) so that you can continue to run both VS 2017/2015 tests using the TA 2015.

@nigurr, trying to manually install TA 2017 from https://aka.ms/vs/15/release/vs_TestAgent.exe result in an install with zero progress.

A guess you have to give my instruction to cleanup registry for this.

@pi3k14 is this machine connected to internet? If not we need to prepare the layout for the test agent from internet connected machine

@nigurr this machine is connected to the internet, but not for inbound traffic (it's a internal test machine). Are you implying that manual test agent installation is not possible?

@pi3k14 it's possible. we published blog for the same https://blogs.msdn.microsoft.com/visualstudioalm/2017/05/05/using-visual-studio-agent-deployment-task-on-machines-not-connected-to-the-internet/

From the internet connected machine

vs_testagent.exe --layout C:\TA2017Setup --lang en-US

Copy the entire folder C:\TS2017Setup to Test Agent machine and run the setup

C:\TA2017Setup\vs_testagent.exe --norestart --installPath --wait C:\TestAgent2017\

@nigurr sorry, but this give the same problem https://github.com/Microsoft/vsts-tasks/issues/3922#issuecomment-301699137 (no progress in installation)

@pi3k14 can you drop your skype to devops_tools at microsoft dot com
We will assist you with this.

@nigurr have to get back on this, end of day and tomorrow is a public holiday

@pi3k14 I installed into one machine which is not connected to internet and it worked.
Followed the same step which I shared with you. Before installing, in the C:\Ta2017Setup\certificates -> there are couple of signed certificates. These must be installed [right click -> install] before starting installation.

@nigurr manually installing the certificates did the trick, thank you :)
But then I stumbled into the next problem, it doesn't seem like the Run Functional Tests Task support net core tests "Error : No tests were discovered from the specified test sources".
I am using the NUnit3 TestAdapter v 3.8.0 (alpha1) http://www.alteridem.net/2017/05/04/test-net-core-nunit-vs2017/ , any pointers on how to proceed?

@pi3k14 please copy the "Nunit adapaters" along with your Test Assemblies to remote machine. The engine will auto-load the adapters if it finds within test folder.

@nigurr yes I know. Both the NUnit3.TestAdapter.dll and the nunit.engine.netstandard.dll are copied. I even overrode the testrun parameter TestAdaptersPaths to point to the test folder, but it didn't change anything.
Is there any "hidden" parameters to get trace information from the Run Functional Tests Task?

@pi3k14 There's no hidden information. We print everything to "C:\TestAgent\dtaexecutionhost.log". You can try running the tests with "vstest.console.exe" from test agent machine and see if tests are getting discovered. use "vstest.console.exe /?" for help. We use the same engine to execute the tests.

@nigurr found the problem in the vstest.console diagnostics
I, 4300, 1, 2017/05/18, 10:03:10.895,<PHONE_NUMBER>693, vstest.console, AssemblyResolver: System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a: Resolving assembly.
V, 4300, 1, 2017/05/18, 10:03:10.895,<PHONE_NUMBER>028, vstest.console, CurrentDomain_AssemblyResolve: Resolving assembly 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
V, 4300, 1, 2017/05/18, 10:03:10.895,<PHONE_NUMBER>319, vstest.console, CurrentDomain_AssemblyResolve: Failed to resolve assembly 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.

found several references to this problem at StackOverflow (eg. http://stackoverflow.com/a/41414568/2294065), but not sure how to proceed from here. Is this something that has to be fixed in vstest.console?
Both NUnit (3.6.1) and Microsoft.NET.Test.Sdk (15.0) are bound to version 4.3 of System.Runtime

Hi Experts,
I got struck with Release error in vsts + BizTalk Server 2016 with Feature pack installed.
While creating release definition in VSTS I am getting below release error message, please help me to error message.
when iam trying to multiple projects into the same solution for Automatically deploy BizTalk Server applications using Visual Studio Team Services and getting the below release error message.
"Cannot find path 'C:\agent_work\r3\a_btsDeploy\unzipped\SampleDemoApplication\bin\Binding.xml' because it does not exist.".


Using accents in hieroglyph-quickstart fails with UnicodeError

Welcome to the Hieroglyph 1.0.0 quickstart utility.
> Presentation title: Qualité du code
[snip]
Traceback (most recent call last):
  File "/home/arthur/.virtualenvs/formation/lib/python3.5/encodings/idna.py", line 181, in encode
    result.extend(ToASCII(label))
  File "/home/arthur/.virtualenvs/formation/lib/python3.5/encodings/idna.py", line 103, in ToASCII
    raise UnicodeError("label empty or too long")
UnicodeError: label empty or too long

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/arthur/.virtualenvs/formation/bin/hieroglyph-quickstart", line 11, in <module>
    sys.exit(quickstart())
  File "/home/arthur/.virtualenvs/formation/lib/python3.5/site-packages/hieroglyph/quickstart.py", line 94, in quickstart
    sphinx.quickstart.generate(d, templatedir=templatedir)
  File "/home/arthur/.virtualenvs/formation/lib/python3.5/site-packages/sphinx/quickstart.py", line 412, in generate
    d['project_url'] = urlquote(d['project'].encode('idna'))
UnicodeError: encoding with 'idna' codec failed (UnicodeError: label empty or too long)

$ pip show hieroglyph
Name: hieroglyph
Version: 1.0.0
Summary: Generate HTML presentations from plain text sources with all the power of Sphinx.
Home-page: https://github.com/nyergler/hieroglyph
Author: Nathan Yergler
Author-email<EMAIL_ADDRESS>License: BSD
Location: /home/arthur/.virtualenvs/formation/lib/python3.5/site-packages
Requires: setuptools, Sphinx, six

I haven't dug into this deeply, but this appears to be a Sphinx issue. To test that you'd need to try running sphinx-quickstart with the same input (although it calls it "project title" not "presentation title") and see if it succeeds. I believe it's a Sphinx issue because we're using Sphinx's prompt support, as well as quickstart generation. Looking at how they prompt for the title, it doesn't appear that Sphinx does any additional processing.

Just tested this with the latest Sphinx + Hieroglyph master and it appears to be working. Despite the delay in closing, thank you for reporting!


Rework OfflineVirtualMachine status

The ovm was too much focused on conditions and not enough on the acutal
important states to track. The condition for tracking the vm state got
removed now. Instead "created" and "ready" booleans are tracking the vm
lifecycle, like agreed on in the original design document.

@pkliczewski FYI: This implements missing parts of the OfflineVirtualMachine. It is now much closer to what was agreed on. However that means that the "Running" condition, which should not have been there at all will be gone. I think that will affect the ManageIQ implementation. Since we are still in alpha, I would prefer to simply remove the condition.


Group crashes by location of crash in Objective C code

Currently, all Objective-C objc_MsgSend SIGSEV crashes are all grouped together, even though they occur in different places in the code.
Would it be possible to group crashes more intelligently, for example by the location of the last line in symbolicated code in the stack?

Hi @shimfish,
Sorry for the delay, but this is a duplicate of #348, so let's track the status of it there


[BUG] integrated mode does consume double the power than nvidia mode

Hey there.
I have a Thinkpad W530. It has a pre Turing GPU, the Quadro K2000M, i use NVIDA Driver Version 470 on Arch Linux.
When i use envycontrol to set integrated mode, powertop does not show the NVidia Device in device statistics any more, but with my display backlight  set to the darkest mode possible, all pm modes in powertop set to enabled and wifi switched off with the hardware switch, the computer consumes still 18-20W power.
But when i switch to nvidia mode, with the same setting, the power usage is around 10W, half of integrated mode. But the nvidia card is acitve and usable. Even now, while i am writing this text with around 50% backlight and Wifi on, it still uses only 15W of power, still less than integrated mode.
I once had a OpenSuSE Tumbleweed setup, and it brings its own Optimus Switcher, SUSE-PRIME, which uses bbswitch, if it is installed. When i used integrated mode (intel) there, the power usage was around 7W. So i guess SUSE-PRIME was able to turn off the NVIDIA GPU successfully using bbswitch and that value is my reference. On Nvidia Mode, Power Usage was around 12W, so comparable to arch. In fact, that was the only setup, where i was able to successfully turn off the nvidia gpu since the bumblebee times when i bought that laptop.
So i guess there must be something wrong with envycontrol, because it seems as if it does really not turn off the nvidia gpu. Maybe i missed something. I really do not know. Does someone have the same problem?

Hi @grimba, EnvyControl relies on udev rules for turning off your GPU so without lspci output I wouldn't be able to tell what's different on your device.

Hey, thanks for the really quick answer. No problem, here the outputs:
integrated:
00:00.0 Host bridge: Intel Corporation 3rd Gen Core processor DRAM Controller (rev 09) 00:01.0 PCI bridge: Intel Corporation Xeon E3-1200 v2/3rd Gen Core processor PCI Express Root Port (rev 09) 00:02.0 VGA compatible controller: Intel Corporation 3rd Gen Core processor Graphics Controller (rev 09) 00:14.0 USB controller: Intel Corporation 7 Series/C210 Series Chipset Family USB xHCI Host Controller (rev 04) 00:19.0 Ethernet controller: Intel Corporation 82579LM Gigabit Network Connection (Lewisville) (rev 04) 00:1a.0 USB controller: Intel Corporation 7 Series/C216 Chipset Family USB Enhanced Host Controller #2 (rev 04) 00:1b.0 Audio device: Intel Corporation 7 Series/C216 Chipset Family High Definition Audio Controller (rev 04) 00:1c.0 PCI bridge: Intel Corporation 7 Series/C216 Chipset Family PCI Express Root Port 1 (rev c4) 00:1c.1 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 2 (rev c4) 00:1c.2 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 3 (rev c4) 00:1d.0 USB controller: Intel Corporation 7 Series/C216 Chipset Family USB Enhanced Host Controller #1 (rev 04) 00:1f.0 ISA bridge: Intel Corporation QM77 Express Chipset LPC Controller (rev 04) 00:1f.2 SATA controller: Intel Corporation 7 Series Chipset Family 6-port SATA Controller [AHCI mode] (rev 04) 00:1f.3 SMBus: Intel Corporation 7 Series/C216 Chipset Family SMBus Controller (rev 04) 02:00.0 System peripheral: Ricoh Co Ltd MMC/SD Host Controller (rev 08) 02:00.3 FireWire (IEEE 1394): Ricoh Co Ltd R5C832 PCIe IEEE 1394 Controller (rev 04) 03:00.0 Network controller: Intel Corporation Wi-Fi 6 AX200 (rev 1a)
nvidia:
00:00.0 Host bridge: Intel Corporation 3rd Gen Core processor DRAM Controller (rev 09) 00:01.0 PCI bridge: Intel Corporation Xeon E3-1200 v2/3rd Gen Core processor PCI Express Root Port (rev 09) 00:02.0 VGA compatible controller: Intel Corporation 3rd Gen Core processor Graphics Controller (rev 09) 00:14.0 USB controller: Intel Corporation 7 Series/C210 Series Chipset Family USB xHCI Host Controller (rev 04) 00:19.0 Ethernet controller: Intel Corporation 82579LM Gigabit Network Connection (Lewisville) (rev 04) 00:1a.0 USB controller: Intel Corporation 7 Series/C216 Chipset Family USB Enhanced Host Controller #2 (rev 04) 00:1b.0 Audio device: Intel Corporation 7 Series/C216 Chipset Family High Definition Audio Controller (rev 04) 00:1c.0 PCI bridge: Intel Corporation 7 Series/C216 Chipset Family PCI Express Root Port 1 (rev c4) 00:1c.1 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 2 (rev c4) 00:1c.2 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 3 (rev c4) 00:1d.0 USB controller: Intel Corporation 7 Series/C216 Chipset Family USB Enhanced Host Controller #1 (rev 04) 00:1f.0 ISA bridge: Intel Corporation QM77 Express Chipset LPC Controller (rev 04) 00:1f.2 SATA controller: Intel Corporation 7 Series Chipset Family 6-port SATA Controller [AHCI mode] (rev 04) 00:1f.3 SMBus: Intel Corporation 7 Series/C216 Chipset Family SMBus Controller (rev 04) 01:00.0 VGA compatible controller: NVIDIA Corporation GK107GLM [Quadro K2000M] (rev a1) 01:00.1 Audio device: NVIDIA Corporation GK107 HDMI Audio Controller (rev a1) 02:00.0 System peripheral: Ricoh Co Ltd MMC/SD Host Controller (rev 08) 02:00.3 FireWire (IEEE 1394): Ricoh Co Ltd R5C832 PCIe IEEE 1394 Controller (rev 04) 03:00.0 Network controller: Intel Corporation Wi-Fi 6 AX200 (rev 1a)
hybrid:
00:00.0 Host bridge: Intel Corporation 3rd Gen Core processor DRAM Controller (rev 09) 00:01.0 PCI bridge: Intel Corporation Xeon E3-1200 v2/3rd Gen Core processor PCI Express Root Port (rev 09) 00:02.0 VGA compatible controller: Intel Corporation 3rd Gen Core processor Graphics Controller (rev 09) 00:14.0 USB controller: Intel Corporation 7 Series/C210 Series Chipset Family USB xHCI Host Controller (rev 04) 00:19.0 Ethernet controller: Intel Corporation 82579LM Gigabit Network Connection (Lewisville) (rev 04) 00:1a.0 USB controller: Intel Corporation 7 Series/C216 Chipset Family USB Enhanced Host Controller #2 (rev 04) 00:1b.0 Audio device: Intel Corporation 7 Series/C216 Chipset Family High Definition Audio Controller (rev 04) 00:1c.0 PCI bridge: Intel Corporation 7 Series/C216 Chipset Family PCI Express Root Port 1 (rev c4) 00:1c.1 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 2 (rev c4) 00:1c.2 PCI bridge: Intel Corporation 7 Series/C210 Series Chipset Family PCI Express Root Port 3 (rev c4) 00:1d.0 USB controller: Intel Corporation 7 Series/C216 Chipset Family USB Enhanced Host Controller #1 (rev 04) 00:1f.0 ISA bridge: Intel Corporation QM77 Express Chipset LPC Controller (rev 04) 00:1f.2 SATA controller: Intel Corporation 7 Series Chipset Family 6-port SATA Controller [AHCI mode] (rev 04) 00:1f.3 SMBus: Intel Corporation 7 Series/C216 Chipset Family SMBus Controller (rev 04) 01:00.0 VGA compatible controller: NVIDIA Corporation GK107GLM [Quadro K2000M] (rev a1) 01:00.1 Audio device: NVIDIA Corporation GK107 HDMI Audio Controller (rev a1) 02:00.0 System peripheral: Ricoh Co Ltd MMC/SD Host Controller (rev 08) 02:00.3 FireWire (IEEE 1394): Ricoh Co Ltd R5C832 PCIe IEEE 1394 Controller (rev 04) 03:00.0 Network controller: Intel Corporation Wi-Fi 6 AX200 (rev 1a)

That's strange since the Nvidia GPU is correctly removed from the PCI bus and the kernel shall take care of the power management. In the other hand, bbswitch works by doing a call to the Nvidia GPU telling it to turn off (old method used in the pre Windows 10 era).

I also found out (using your other guide to turn of nvidia gpu), that _SB.PCI0.LPC.EC.PUBS._OFF is the right ACPI call to turn off the GPU. Would it be possible to add a function to turn the GPU off by acpi_call for Devices, where the udev rules do not work? Similar like Suse does with SUSE-PRIME and the optional use of bbswitch?

I also found out (using your other guide to turn of nvidia gpu), that _SB.PCI0.LPC.EC.PUBS._OFF is the right ACPI call to turn off the GPU. Would it be possible to add a function to turn the GPU off by acpi_call for Devices, where the udev rules do not work? Similar like Suse does with SUSE-PRIME and the optional use of bbswitch?

The problem with acpi_call is that it's an out of the tree module so it won't work across distros.

Sup folks. I'm having basically the same problem with envycontrol.
When setting to integrated mode, udev-worker hits hard the CPU, making one of the cores to be pinned on 100% usage, depleting the battery reaaaaaally fast.
It does not reproduce on hybrid mode, but, with that mode it keeps nvidia powered up consuming constant 8W since it is a gaming laptop and having nvidia completely powered down makes battery life when laptop is not docked way better.
Not trying to compare software here or bash anyone, just adding some more data: With optimus-manager I was able to completely shut down Nvidia, but that software also is not being actively maintained lately and had other problems which led me to stop using it.

I had the same problem, basically the gpu is woken up at startup and since the nvidia driver is not loaded it remains in the full power state.
I was able to solve the problem by removing the file: usr/share/glvnd/egl_vendor.d/10_nvidia.json as stated [here] (https://github.com/SnipeXandrej/zephyrus-g15-tweaks?tab=readme-ov-file#nvidia-gpu-d3-power-state-on-arch-linux) in the third point.


Feat: Add icon to filename component

Requested feature
An option to filename component that allows displaying the filetype icon along with the name. With the current components, the best that I could achieve was the following:

It is a reasonable result, but the file name and the type icon are kind of distant... Furthermore, I had to disable the file status option, otherwise it would be in the middle...

Motivation
In my humble opinion, filename and filetype carry very close meanings and could be displayed together, as I tried to do with the available components.

setup
local is_status_ok, lualine = pcall(require, "lualine")
if not is_status_ok then
  return
end

lualine.setup {
  options = {
    icons_enabled = true,
    theme = 'codedark',
    component_separators = { left = '', right = ''},
    section_separators = { left = '', right = ''},
    disabled_filetypes = {},
    always_divide_middle = true,
    globalstatus = false,
  },
  sections = {
    lualine_a = {'mode'},
    lualine_b = {'branch', 'diff', 'diagnostics'},
    lualine_c = {{'filename', file_status = false, separator = ''}, {'filetype', icon_only = true}},
    lualine_x = {'encoding', 'fileformat'},
    lualine_y = {'progress'},
    lualine_z = {'location'}
  },
  inactive_sections = {
    lualine_a = {},
    lualine_b = {},
    lualine_c = {'filename'},
    lualine_x = {'location'},
    lualine_y = {},
    lualine_z = {}
  },
  tabline = {},
  extensions = {}
}

You can put the fitletype icon infront of filename and also control the spacing with padding option.
You're quite close actually .
-    lualine_c = {{'filename', file_status = false, separator = ''}, {'filetype', icon_only = true}},
+   lualine_c = {{'filetype', icon_only = true, separator = '', padding={right=0, left=1}}, 'filename' },

I thought just setting the padding to zero wouldn't be enough. Now it looks fine. Thanks.


Adapt UI to new json-rpc functions

Gets discussions when getting into a Realm for the first time after logging in
Fetches NPCs backstories
Changed townhall to discussions, left some explicitely as town hall fit more for the user sometimes

Added a commit to remove some misplaced elements from the NpcStore. Added a DiscussionContext instead.
Removed selectedNpc and function to set the state from NpcStore and use them as props now


Google

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Is your feature request related to a problem? Please describe.
What the problem is. Example: I'm always frustrated when [...]
Describe the solution you'd like
What you want to happen.
Describe alternatives you've considered
Any alternative solutions or features you've considered.
Additional context
Any other context or screenshots about the feature request.

com.google.apis
google-api-services-androidmanagement
v1-rev20241022-2.0.0

Closing as this issue do not have sufficient detail.
Please open a new issue if needed.


google-cloud-logging-logback: Allow user to specify custom LoggingOptions

Allow users to subclass com.google.cloud.logging.logback.LoggingAppender to specify custom LoggingOptions, such as custom credentials.
Fixes #3215

WDYT about adding setter method for credential file so user can configure from xml rather than writing code to subclass?

WDYT about adding setter method for credential file so user can configure from xml rather than writing code to subclass?

Yes, you are absolutely right. I'll change it to support that.


Add Warnings As Errors

Description
Adds warnings as errors.
Adds vkb__register_component to guarantee consistent library creation. Warnings are set at a target level which should make sure third party warnings are ignored. If this is not the case then we will need to create warning wrappers similar to the master branch.
Fixes #478

@asuessenbach if this merges first I can fix the folder issue that you found in the other PR in a new PR


Add React Native Helpers

Opening this to discuss the possibility of adding helpers for React Native percentages based on Height/Width. The API could look something like this:
{
  'font-size': deviceWidth('10%'),
  'height': deviceHeight('100%'),
}

which would replace something like this:
{
  'font-size': (Dimensions.get('window').width * .18),
  'height': (Dimensions.get('window').height),
}

See: https://facebook.github.io/react-native/docs/dimensions.html
We could potentially get really fancy and offer the ability to determine values based off measurements other than percentage.
Additional context: https://github.com/styled-components/polished/issues/129#issuecomment-291002816
@mxstbr @nikgraf thoughts on this idea?

Nice, does it make sense to have a separate package or part of the package dedicated to React Native?

That is a good question. In addition to percentage helpers, there is the simple difference of a lot of the FB subset of CSS for React Native being camel-cased instead of hyphenated. There also seem to be some challenges around responsive design in general: https://medium.com/@elieslama/responsive-design-in-react-native-876ea9cd72a8
Seems like there would be

React Native specific helpers (like outlined above).
React Native versions of other polished mixins/helpers.

Why React Native is a different beast, my worry with a separate package is in the case of 2, where there would be a lot of duplicated code with just the casing of the properties being different.
Since we encourage (force) individual pieces to be imported, we could have a stand-alone section for React Native specific mixins/helpers. Then on the ones with shared code we could (theoretically) detect if it is a RN app and adjust the property names automatically, keeping the file size the same with dead code elimination.

styled-components works on ReactNative with standard polished (we take care of camel casing etc.), so I'm wary of introducing another package for these. Why not just add them to the package, but make it clear in the docs they're RN only?

@mxstbr I think that is the proposition. The only part up in the air was the camel casing, need to research how other libraries that support RN. Hopefully it is handled the same way styled components does it.

This has been open for over a year with no real interest.


change how delete and provision signature are accepted and verified

After this commit, user that are requested in the delete or provision
consensus are only able to push a signature once.
If they already have a signature registered and try to put a new one, an
400 bad request error will be returned by the API.
This change allow to simplify the way we deal with signature array
internally and should prevent issue like #141
fixes #141

@muhamadazmy I think I've covered all the cases that could lead to a nil array.
We might still need to manually ensure there are not already such a null array  in the db and update that is needed.


Optional elements

I need to parse X12 segments with a variable number of elements., Such as element 8 in the R4 segment of this example:
R4*R*UN*USCHI*Chicago Ramp*US***IL~
DTM*140*20220709*13000000*LT~
R4*L*UN*USNYC*Port of New York*US***NY~
DTM*140*20220710*21000000*LT~
R4*D*UN*BHBAH*Bahrain Port*BH~
DTM*139*20220711*13000000*LT~

Since the third occurrence of R4 is only 5 elements, we receive the error: "unable to find element x on segment x"
What is the correct way to configure such a case in the schema?
I was hoping for something like:
{"name": "portStateCode","index": 8, "min": 0, "max": 1}

Have you tried default when declaring your element? See element.default in https://github.com/jf-tech/omniparser/blob/master/doc%2Fedi_in_depth.md

@sivan-koren closing due to lack of activity. reopen if you have any further questions.


Add version: JetBrains.Kotlin.Compiler version 1.1.2-5

Pull request has been created with Komac v1.9.1 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116288&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


Add scanCombine

scanCombine([
  [prepend, (item, list) => [item].concat(list)],
  [removeKey, (keys, list) => list.filter(item => !contains(itemToKey(item), keys))],
], initial);

Another way I've enjoyed doing this type of thing lately is to make the updater streams into a single stream of functions that I scan over:
const updaters = mergeAll([
  map(prepend, prependElemStream)
, map(filterOut, removeKeysStream)
])
const elems = scan((acc, fn) => fn(acc), initial, updaters)

const prepend = item => list => [item].concat(list)
const filterOut = keys => list => list.filter(item => !contains(itemToKey(item), keys))

Interesting. Why do you prefer that approach?

I don't think I really prefer it, was just generally thinking of ideas around aggregating state by using streams of updater functions. However I think scanCombine should be included, as I find it immediately useful in maybe 95%+ of UI components. Just a tangential comment, apologies

Ok. Thank you for explaining.

I don't think I really prefer it, was just generally thinking of ideas around aggregating state by using streams of updater functions.

On a related note. In Reactive Banana they have a function accumE that takes a stream of updater functions and uses them to accumulate state. The function has the type MonadMoment m => a -> Event (a -> a) -> m (Event a). In Hareactive it whould be accumS: <A>(intial: A, s: Stream<(a: A) => A>) => Stream<A>.

We've added scanCombine :smile:


com.apple.vm.networking entitlement is restricted to developers of virtualization software?

Bridged Network require com.apple.vm.networking entitilment,  while as the apple develop doc says that
This entitlement is restricted to developers of virtualization software. To request this entitlement, contact your Apple representative.
any ideas?
How did you test the bridged network feature?
Doc

@aoxn Sorry, I don't know as well.
@AkihiroSuda Do you have any ideas?

The entitlement is not needed when you run it as the root user.
In Lima (https://lima-vm.io), https://github.com/lima-vm/socket_vmnet is used to run the bridged networking stack as the root while keeping the VM itself non-root.
https://lima-vm.io/docs/config/network/

I've asked Apple recently about the process to follow to get this entitlement: https://developer.apple.com/forums/thread/769896

@aoxn Is it likely to resolve the problem?


Searching for name from belongsToMany relationship

Summary of problem or feature request
Let me start by explaining my case. I have a large database of users to display, so I am using server-side filtering. In my table, i have a column called Membership Types this is a concatenation  of membership_type.names.
// I can reach the value I want to put to the table like this.
$value = $user->currentMembership->membershipTypes->implode('name', ' / ');

I was able to display the table as I wanted, but it would be great to let the client filter through the names.
For example:
// User1 has the types `Boxer`, `Official` and `Coach`
// User2 has the types `Boxer` and `Coach`
// User3 has the types `Boxer`

If the client were to type Coach he should be able to get User1 and User2.

Laravel Version 5.2
Laravel-Datatables Version 6.18

Regards,
Ozzy

Thanks for reporting, I haven't really tried doing a belongsToMany with the package. But I will try to make a demo for this to test if it's supported. If not, then will try to add a support for it. Will try to do it as soon as I can. Thanks!

@yajra Is there chance for me to add some stuff to your documentation website? I read your whole repository and have some real-life use cases. I could help. :)

@OzanKurt , yes. Would really appreciate it if you can contribute to the docs. Just send a PR on the demo repo. Thanks!

Did either one of you figure out belongsToMany searching?

@barryvdh I did a work around.

if ($this->request->get('producer') != '') {
            if (is_numeric($this->request->get('producer'))) {
                $producer = $producer->where('producer.producerid', '=', (int)$this->request->get('producer'));
            } else {
                $producer = $producer->where('upper(producername)', 'LIKE',
                    '%' . strtoupper($this->request->get('producer')) . '%');
                $producer = $producer->orWhere('upper(produceralias)', 'LIKE',
                    '%' . strtoupper($this->request->get('producer')) . '%');
            }
        }

@barryvdh I just added a demo for belongs-to-many and the approach is just like the has-many demo but please take note of the distinct query added to avoid duplicate data since it's many to many relationships.


Prefix attributes to avoid name collisions with other libraries

Prefix attributes to avoid name collisions with other libraries like ConstraintLayout
https://github.com/miguelhincapie/CustomBottomSheetBehavior/issues/85

The last two commits should perhaps not be merged in but was required to get around a build.gradle error with correct placement of the plugins block, to make it possible to directly include this pullrequest like this:
implementation 'com.github.mortenholmgaard:CustomBottomSheetBehavior:master-SNAPSHOT'

ty for the PR! I near future I will have free time to merge and update this library. I'm thinking on moving it to kotlin using motionLayout.


StackOverflowError when creating infinite recursion

How to reproduce:

Create infinite recursion with gates
Hook an output into it
See console

After thinking about this issue for a while I might have come up with a solution:

Add the method Set<Node> getLastNodes to the ConnectorHolder interface
Implement the method in a way that it returns the nodes at the end of the connections
When connecting a node (A) to another node (B) check if the last nodes of A contain B; If so, abort connecting

This is just an idea. I'll try to fix this when I get home from work as long as no one else claims this issue. It's still up for grabs!

Turns out my idea didn't really work

I would like to see it, can you assign it to me and also, please provide the steps to run the program.

Sure, I'll assign this issue to you. Every modern IDE should be able to run the main class directly but these are the manual steps:

Clone this repository git clone https://github.com/RealCerus/logic-builder.git
Change directory cd logic-builder
Build the project with Maven mvn package
Run the jar file java -jar target/logic-builder-1.0.0.jar

@kaggrwal Any progress so far?

@RealCerus Hi, I didn't get the time, i'll try it soon and let you know about the progress.

@RealCerus Can we discuss it over IRC/chat ?

You can add me on Discord Cerus#5149


datev2 make function nullable DEPEND_ON_ARGUMENT

Proposed changes
Set all functions' nullable mode for datev2 to DEPEND_ON_ARGUMENT
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.51 seconds
load time: 524 seconds
storage size:<PHONE_NUMBER>4 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230128132528_clickbench_pr_85693.html


store discovery files under $HOME so they're not cleaned up by the OS

A recent PR wrote discovery JSON files to /tmp but telemetry is reporting that sometimes an expected file is missing.  This appears in the logs as:
Errno::ENOENT: No such file or directory @ rb_sysopen - /tmp/.dependabot/discovery.1.json

Ultimately, I don't know exactly what's happening, but here's a guess.

An update job consists of two processes: bin/run fetch_files and bin/run update_files.
The recent PR (#11026) writes discovery JSON files during the fetch_files step to /tmp/.dependabot/dependabot*.
That same PR then reads those discovery files during the update_files step.

According to the logs, all instances of discovery.1.json not being found all happen during update_files, but the log also confirms that the files were written during fetch_files.  The explanation is that the OS/container is cleaning files in /tmp after the fetch_files process ends and before the update_files process starts.
The fix is to write these files to $HOME/.dependabot/dependabot* so they're not marked for cleanup by the OS/container on process exit.  There are other instances (e.g., during analysis and checking for updates) where we still write to /tmp, but we're not crossing a process lifetime boundary, so those are safe.

Has a change gone into Dependabot core in the last 24 hours that will impact or fix this scenario soon.  Just I'm seeing this in our repository for dependabot : Errno::ENOENT: No such file or directory @ rb_sysopen - /tmp/.dependabot/discovery.1.json and but I am unsure if this is fixed by this change, and when it may go in? It's changed to the home directory in the tests I've carried out this afternoon, but the file discover.1.json still cannot be found.
updater | 2024/12/04 20:31:45 INFO <job_927637624> Discovery JSON path for workspace path [/] created for file [/home/dependabot/.dependabot/discovery_map.json] at location [/home/dependabot/.dependabot/discovery.1.json]
updater | 2024/12/04 20:31:45 ERROR <job_927637624> No such file or directory @ rb_sysopen - /home/dependabot/.dependabot/discovery.1.json
updater | 2024/12/04 20:31:45 ERROR <job_927637624> /home/dependabot/nuget/lib/dependabot/nuget/native_discovery/native_discovery_json_reader.rb:166:in `read'

updater | Dependabot encountered '1' error(s) during execution, please check the logs for more details.
updater | +--------------------+
updater | |       Errors       |
updater | +--------------------+
updater | | update_files_error |

Dependabot updates in the last 7 hours are failing because of this kind of error appearing in the logs.