Text Remains After Ending Timeline

Describe the bug
The last displayed text remains visible after ending a timeline. When starting a new timeline, I'm having characters joining while the last displayed text remains visible, until the first character begins speaking. This is a problem for the 2nd conversation and onward. This may not be noticeable with the default theme.
To Reproduce
Steps to reproduce the behavior:

Layout some portrait containers and a Dialogic text box
Start a timeline with "Dialogic.start_timeline(timelineName)"
Continue until a "timeline_ended" event has been fired.
If Dialogic textbox is still visible, you should still see the last line of the timeline in the text field.
Checking "Dialogic.current_state_info.text", the last line of the previous timeline appears to still be set.

I've attached a sample project.
Expected behavior
I'm expecting the text box to be empty after a timeline has ended. And to be empty as a new timeline is started.
System (please complete the following information):

OS: OSX
Godot Version: 4.1.2
Dialogic Version: 2.0-Alpha-11

Solutions
Workaround
It seems that calling the Dialogic.Text.clear_game_state() will clear the text. For some reason I'm not seeing it being fired from DialogicGameHandler.clear() unless I remove the if statement completely.
This seems to work but I understand there's consequences to clearing everything at the end of every timeline.
Possible fixes
Looking into the code a bit, it seems that Dialogic.Text.clear_game_state() doesn't appear to be fired at the end of a timeline, except when initializing Dialogic.
In DialogicGameHandler.clear(),
	if !clear_flags & ClearFlags.TIMLEINE_INFO_ONLY:
		for subsystem in get_children():
			subsystem.clear_game_state(clear_flags)

It appears that clear() for the Text module doesn't seem to be called after ending a timeline. When removing the 'if,' the text appears to clear after a timeline ends (but of course that causes other issues).
I've attached a project demonstrating this.
DialogicTest_TextClear.zip
If this is intended behavior, a work around would be appreciated. Thanks!

FYI, my current workaround is to just force text to clear at the every "timeline_end" event.
In DialogicGameHandler.clear(), I'm executing this after the existing "clear_game_state()" call.
# force text to always clear
for subsystem in get_children():
	if subsystem.get_name() == "Text":
		subsystem.clear_game_state(clear_flags)

If you want to clear stuff, then the "Clear" event is your friend.
In case it doesn't do what you want Dialogic.Text.update_text("") would be an easier way of removing the text.


Codegeneration for Inner classes extending from MvpView

I have such interface:
public interface Contract {
    interface View extends MvpView {
        void setLoadingIndicator(boolean active);
    }
    interface Presenter {
        ...
    }
}

Also, i have a presenter
@InjectViewState
public class Presenter
        extends AbstractPresenter<Contract.View> implements Contract.Presenter {

When i  build project, the error occurs: it can not find variable "View" in the generated code for Contract$View$$ViewState class.
If i refactor Contract.View to simple ContractView interface, then everything works fine.
Is this my mistake or moxy does not support code generation for nested interfaces?

Hi, moxy doesn't support this approach. But it seems ok to do this. Will be done in some near release ;)

That's great, because it is very nice to have the class, defining the view/presenter contract (as adviced by google android architecture samples).
Thanks! I will leave this issue open, if you don't mind.

This is published in version 1.4.6.

That is just great, thank you very much!

Hi, this works in 1.4.6, but fails in 1.5.5. Would you mind to fix this? Thanks!

Copy paste^) Hi, this works in 1.4.6, but fails in 1.5.5. Would you mind to fix this? Thanks!
    implementation 'tech.schoolhelper:moxy-x:1.7.0'
    annotationProcessor 'tech.schoolhelper:moxy-x-compiler:1.7.0'
    implementation 'tech.schoolhelper:moxy-x-androidx:1.7.0'


Emulate WinterCG false by default

Related to #37

I left some comments in the linked issue a while ago. I disagree that this should be the default, but during npm create edgespec I'm all for fully explaining the tradeoffs and letting the user decide.


Lack part of emoji sheet

I executed php build_image.php.  But sheet_apple_64.png lacked part.
Would you please tell me solution?
I using ImageMagick 7.0.7-5.
 ~  montage -version
Version: ImageMagick 7.0.7-5 Q16 x86_64 2017-10-03 http://www.imagemagick.org
Copyright: © 1999-2017 ImageMagick Studio LLC
License: http://www.imagemagick.org/script/license.php
Features: Cipher DPC HDRI Modules
Delegates (built-in): bzlib freetype jng jpeg ltdl lzma png tiff xml zlib

it looks like the process died part way through, perhaps? did you run our of disk space during processing


docs/phenom: minor add some missing semicolons and variable declaration

Thanks!
Applied via hub am https://github.com/facebook/libphenom/pull/74 and pushed to the gh-pages branch


Fix CodeQL CLI download

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[ ] For content changes, I have completed the self-review checklist.

@oliviertassinari Thanks for opening a PR 👍 Please fill out the pull request template in the top comment in this pull request, then we'll get this up for review!

Let's close, no time for this, right now.


Governance Proposal

Here is a PR with an initial governance proposal for Perses. I've kept this very simple and short with the idea that we can build on it as the project grows. We can make changes to any of this - please let me know what you do or do not like about this proposal.
I've also suggested that we prune the list of "maintainers" or people who have access to make changes within the Perses org, since based on issue https://github.com/perses/perses/issues/429, a lot of people have access who are no longer active in the project. I've proposed the following 5 people as maintainers based on the feedback from issue https://github.com/perses/perses/issues/429 and my review of existing contributions:

Antoine Thebaud (@AntoineThebaud)
Augustin Husson (@Nexucis)
Luke Tillman (@LukeTillman)
Steven Cobb (@sjcobb)
Julius Volz (@juliusv)

If any of you do not want to be maintainers, please let me know, and I'll remove you.
The governance process for adding new maintainers is very simple and just requires endorsement from 2 existing maintainers, so we can easily add maintainers as the project grows.

Looks good to me @geekygirldawn. Thanks !
For the moment the current maintainers are part of the team too. Maybe it will change later if we have some "external" contributor :).

@Nexucis Sounds good! I just pushed commits to add the current maintainers to the "Team Members" list on the governance page. I think this is ready to merge unless anyone else has further suggestions.

Cool, thanks @geekygirldawn ! I agree this is ready to merge.
I think I will merge it in a couple of days to let others the time to comment if necessary.

thanks for your help @geekygirldawn. I will merge this PR now as there are no more comments from others.


[ISSUE] Inconsistency in APP_NAME values in IDN_OAUTH_CONSUMER_APPS

Describe the issue:
In IS side OAuth application data are stored in IDN_OAUTH_CONSUMER_APPS. This APP_NAME is not consistent.
When an OAuth App is created via REST API, the APP_NAME is a random uuid.
When an OAUth App is created via management console, the the APP_NAME is service provider's name.
And if we update the app created from the REST_API using management console, the APP_NAME changes from uuid to sp_name.
How to reproduce:

Create a oauth sp(spname: testy) using Application Management Rest API.
The APP_NAME in IDN_OAUTH_CONSUMER_APPS  table will be a uuid.

Update the sp(spname: testy) using management console (please note that I didn't update spname. ).
Now the APP_NAME in IDN_OAUTH_CONSUMER_APPS  table will be a testy.

Now if we update the sp using management console, the app_name will change from uuid to sp_name. This will make issues.
Suggested labels:

@mefarazath @emswbandara Please give your thoughts

@piraveena Does the consumer app name change to SP's name only when you change SP name or do any update to the OAuth app configurations?

@mefarazath APP_NAME changes to sp's name from uuid with any update to OAuth app configurations

@piraveena Does the consumer app name change to SP's name only when you change SP name or do any update to the OAuth app configurations?

@mefarazath APP_NAME changes to sp's name from uuid with any update to OAuth app configurations

Okay, that looks wrong to me :)
Management console shouldn't override the existing app name. What's the behavior when you change the SP name. I assume it will not change the OAuth client name?

Updating the SP name (testapp -> testapp1) from management console changes the APP_NAME . It changes into uuid to the new app name (uuid -> testapp1).

Okay, that looks wrong to me :)
Management console shouldn't override the existing app name. What's the behavior when you change the SP name. I assume it will not change the OAuth client name?

Created a sp testapp.
Updating the SP name testapp1 (testapp -> testapp1) from management console changes the APP_NAME . It changes into uuid to the new app name (uuid -> testapp1).

Created a sp testapp1

Updated a sp using restapi.
curl --location --request PATCH 'https://localhost:9443/t/carbon.super/api/server/v1/applications/e6586b7e-7e47-40a4-bf01-aed6b9bb60f8' \
--header 'Content-Type: application/json' \
--header 'Authorization: Basic YWRtaW46YWRtaW4=' \
--header 'Content-Type: application/json' \
--data-raw '{
 "name": "newapp"
}'

APP_NAME changes to sp_name

Okay considering the complexity, I think the best way would be to fix the REST API to be consistent with other APIs (SOAP and DCR).


Lazily iterate over tests

This should make @brockelmore 's branch brock/par_evm ready to go for #141

Hold on this, a couple of tests need to be fixed


Create easier to analyse exam template

Provide SVG graphic with exam template similar to one provided by supervisor.
Goals:

Keep field structure, order and position
Avoid "soft" edges for bounding boxes and exam checkboxes
Be readable and clear for students
Contain "removable" letters (constant grayscale)

Resolved in https://github.com/cvlab-ai/splinter/pull/41


Basic profile login/register and homepage implemented.

User authentication added.

+1 from me - the markup will change when I get to it in all likelihood


Epic: pg_jsonschema extension

Motivation
pg_jsonschema is an extension built by Supabase. Github repo.
It provides the ability to validate the integrity of json columns with deep structure.
DoD
Implementation ideas
Tasks

[ ]

Other related tasks and Epics

Created a bug tcdi/pgx#1032

@nileshtrivedi. The extension is available in all regions

Thank you @vadim2404 for driving this to closure. I can see that it was not an easy fix because of hardcoded Postgres ABI in PGX etc. 👏


What is the difference between bpf_obj_get() in libbpf.h and BPF_OBJ_GET() in init_tables()

I notice that there are both functions to get map descriptor and update in the control plane of user_xdp5.c and the header of xdp5.h .
In user_xdp5.c , it's
fd = bpf_obj_get(MAP_PATH TABLE);
[...]
ret = bpf_update_elem(fd, &key, &value, BPF_ANY);

In xdp5.h, it's
int tableFileDescriptor = BPF_OBJ_GET(MAP_PATH "/Ingress_dstmactable_defaultAction");
[...]
int ok = BPF_USER_MAP_UPDATE_ELEM(tableFileDescriptor, &ebpf_zero, &value, BPF_ANY);

I know the lowercase function in user_xdp5.c actually is from libbpf.h which finally calls the syscall:
return syscall(__NR_bpf, BPF_OBJ_GET, &attr, sizeof(attr));
# or
return syscall(__NR_bpf, BPF_MAP_UPDATE_ELEM, &attr, sizeof(attr));

But where is the captital function BPF_OBJ_GET() definition ?  Why use this function in xdp.h ?
And I did some test that I use lowercase function bpf_obj_get() in xdp5.h to replace BPF_OBG_GET(). It seems work
After google it , I didn't find any source code in linux kernel matches this . And just know that BPF_OBJ_GET is a type of bpf_cmd in /linux/bpf.h

Here is some documentation for BPF_OBJ_GET.
https://ferrisellis.com/content/ebpf_syscall_and_maps/#bpf-obj-pin-and-bpf-obj-get
The Linux kernel keeps changing, I am not sure whether our programs work with the latest kernels and header files.

I think the uppercase BPF_OBJ_GET is used to replaced by pre-processor to replace to different implementation. In the case of kernel, BPF_OBJ_GET = bpf_obj_get. When we're doing userspace
testing, it gets replaced to the userspace implementation.
The bpf interfaces used in this project now was copied long time ago from kernel.
Now the community is shifting to use libbpf
https://github.com/libbpf/libbpf
This is a mirror of bpf-next linux tree's tools/lib/bpf directory
We should remove the p4c-xdp/lib/ and link to libbpf.so

Thanks for the information.  :smile:


Occasionally cuda stops working

Describe the bug
I have  nix-shell with pytorch installed in the following way:
  buildInputs = with pkgs; [
    python311
    python311Packages.pip
    python311Packages.pytorch-bin
    stdenv.cc.cc.lib
    pam
  ];

  shellHook = ''
        # Tells pip to put packages into $PIP_PREFIX instead of the usual locations.
        # See https://pip.pypa.io/en/stable/user_guide/#environment-variables.
        export PIP_PREFIX=$(pwd)/_build/pip_packages
        export PYTHONPATH="$PIP_PREFIX/${pkgs.python311.sitePackages}:$PYTHONPATH"
        export PATH="$PIP_PREFIX/bin:$PATH"
        export CUDA_PATH=${pkgs.cudatoolkit}
        LD_LIBRARY_PATH=/run/opengl-driver/lib:$LD_LIBRARY_PATH
        echo path $CUDA_PATH
        unset SOURCE_DATE_EPOCH
  '';

Occasionally, especially after waking the laptop up from sleep, running python
> import torch; torch.cuda.is_available()

Results in this:
pip_packages/lib/python3.11/site-packages/torch/cuda/__init__.py:128: UserWarning: CUDA initialization: CUDA unknown error - this may be due to an incorrectly set up environment, e.g. changing env variable CUDA_VISIBLE_DEVICES after program start. Setting the available devices to be zero. (Triggered internally at ../c10/cuda/CUDAFunctions.cpp:108.)
  return torch._C._cuda_getDeviceCount() > 0
False

When this happens, I see no way to bring my pytorch cuda support back except to reboot the machine. After rebooting it works for a while (the above call returns true) and then at some point (usually after days) it stops working again.
Steps To Reproduce
Steps to reproduce the behavior:

Run the above nix shell
Use it for a while
Watch it break

Expected behavior
I expect that if my torch works, it should keep working.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
[nix-shell:/run/media/alexander/BigLinuxData/AI]$ nix run -f "<nixpkgs>" cudaPackages.saxpy
trace: warning: cudaPackages.autoAddDriverRunpath is deprecated, use pkgs.autoAddDriverRunpath instead
Start
Runtime version: 12020
Driver version: 12040
Host memory initialized, copying to the device
CUDA error at cudaMalloc(&xDevice, N * sizeof(float)): unknown error

Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
path /nix/store/gdph9rnz9b46c73rm7d20hxkk3m394kq-cuda-merged-12.2
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.54, NixOS, 24.05 (Uakari), 24.05.5480.6e6b3dd395c3`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.8`
 - channels(root): `"nixos-24.05"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

Ok, huge progress. Found this thread:
https://discuss.pytorch.org/t/userwarning-cuda-initialization-cuda-unknown-error-this-may-be-due-to-an-incorrectly-set-up-environment-e-g-changing-env-variable-cuda-visible-devices-after-program-start-setting-the-available-devices-to-be-zero/129335/2
They recommend this:
sudo rmmod nvidia_uvm
sudo modprobe nvidia_uvm

This did work for me!
[nix-shell:/run/opengl-driver/bin]$ python
Python 3.11.9 (main, Apr  2 2024, 08:25:04) [GCC 13.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch; torch.cuda.is_available()
True
>>>

Potentially related (same workaround): https://github.com/NixOS/nixpkgs/issues/319303

The issue does not seem to be directly related to nixpkgs -- as the link above shows, the same problem happens on Ubuntu. I hope maybe there's a nix-specific way to fix/workaround it, or maybe it can be addressed by the CUDA/pytorch people. However, if nixpkgs maintainers consider it to be some else's problem, I can close the issue.


Added Event page

[x] Compile Event page and make that available in Zip.
[x] Write Markup and styling for Event page
[x] Async load image at top
[x] Add event Name, Place , Time and date on top of image same as orga.
[x] Add speaker section
@championswimmer @mariobehling I will carry out more changes in next PR. Please check and merge this.

Current coverage is 51.75% (diff: 33.33%)

Merging #538 into development will decrease coverage by 0.12%

@@           development       #538   diff @@
=============================================
  Files                4          4
  Lines              453        456     +3
  Methods             45         45
  Messages             0          0
  Branches            62         62
=============================================
+ Hits               235        236     +1
- Misses             218        220     +2
  Partials             0          0

Powered by Codecov. Last update 8c26bc0...5c5b4b1

@championswimmer Merge this I will carry out More UI/UX in next PR
Image in Header

Speakers Section

@mariobehling should we show all speakers on front page ?

@championswimmer I have made this because I think orga UI follows the same

Thanks. The transparency is too dark. It is also to dark on the orga server.


Can you add Nuxt 3 support?

We may add it in the future. But AFAIK you can already use the control in Nuxt (Vue). See https://docs.maptiler.com/vuejs/maplibre-gl-js/how-to-use-maplibre-gl-js/ how to add a MapLibre GL map and under new Map(...) you could add the control with the following code (not tried):
const gc = markRaw(new GeocodingControl({ apiKey, maplibregl }));

map.value.addControl(gc);


⬆ Bump tested SSL options

@Thalhammer may you please remove the required checks for the older versions of LibreSSL? In a future PR I am working on they'll get renamed might not be worth updating them

@Thalhammer may you please remove the required checks for the older versions of LibreSSL?

I think I will just remove all of them, since its only us two that can merge anyway.


What's the core difference between the selective_scan package v1 and v2? About the original mamba_ssm, would v2 be faster or something else?

Yes, you are right. And we're still updating our code, which will be faster.
:rocket: The History of Speed Up
Time is tested on 1xA100 with batch_size 128 for training; the config file is vssm1/vssm_tiny_224_0220.yaml. GPU memory is adopted from the log.
The experiments (arXiv 2401.10166) done before #20240119 used mamba-ssm + group-parallel.
The experiments done since #20240201 use sscore + fused cross scan + fused cross merge. We plan to use ssoflex + fused cross scan + fused cross merge + input16output32 in the future.

name
GPU Memory
time (s/iter)

mamba-ssm + sequence scan
25927M
0.6585s

mamba-ssm + group parallel
25672M
0.4860s

mamba-ssm + float16
20439M
0.4195s

mamba-ssm + fused cross scan
25675M
0.4820s

mamba-ssm + fused csm
25596M
0.4020s

sscore + fused csm
24984M
0.3930s

sscore + fused csm + forward nrow
24984M
0.4090s

sscore + fused csm + backward nrow
24984M
0.4490s

sscore + fused csm + forward nrow + backward nrow
24984M
0.4640s

ssoflex + fused csm
24986M
0.3940s

ssoflex + fused csm + i16o32
19842M
0.3650s

ssoflex + csm in triton + i16o32
19888M
0.3610s

ssoflex + csm in triton + i16o32 + v4
19500M
0.2970s

mamba-ssm: mamba_ssm-1.1.3.post1+cu122torch2.2cxx11abiFALSE-cp310-cp310-linux_x86_64.whl
sscore: selective_scan_cuda_core
ssoflex: selective_scan_cuda_oflex, oflex means output flexible
csm: cross scan and cross merge
i16o32: input fp16 + output fp32

Thanks for your detailed reply!


fix: check error first before accessing the result during translation

Fixes #1917

it seems that the e2e-test is a bit flaky. It succeeded on v1.28, but failed on v1.26.6, am I missing something here?
/cc @arkodg


Review webpack-validator

Hi,
i wrote the tool webpack-validator that validates webpack configs with the validation library joi.
The supported properties are 95% dependent on webpack config documentation. I know that there are quite some undocumented properties. The tool disallows unknown keys, so it would be nice to add those undocumented properties in order to not throw false-positives.
Would anyone be willing to audit the schema and maybe offer some advice if there's some way to find those undocumented properties without reading the whole webpack source? :)

Do you think we should close this? I'm not sure if a lot of people read this tracker. I guess the key here would be to encourage usage through certain other means. Do you have a gitter for the project?
Tobias would likely know the best but I'm sure he is super busy these days. :smile:

Ok, lets close it. I'll do as you suggest. No gitter yet, will create one.


failed to find match for field 'intensity' 'ring'

@toams789. I obtained one bag file with the 'kitti2bag'. When this bag file was employed to LOAM_velodyne(https://github.com/laboshinl/loam_velodyne), the program failed with an error "failed to find match for field 'intensity' 'failed to find match for field 'ring'". Is there the intensity and ring field in the sensor_msgs::PointCloud2 in the bag file generated by 'kitti2bag'?

Hi, the field 'intensity' is treated as the field 'i'.
You could replace PointField('i', 12, PointField.FLOAT32, 1)  with PointField('intensity', 12, PointField.FLOAT32, 1) by yourself to fix the problem.
As for the field 'ring', there is no 'ring' info in the origin KITTI bin file. So the bag file don't contain the field 'ring' as velodyne driver does.

A ring channel has been added in a custom version of kitti2bag in the LIO-SAM repository. Here is a link to the relevant chunk of code.
Although there is a casting issue in the current code which I have fixed in my pull request (which may be implemented by the time you read this)

Check https://github.com/Taeyoung96/kitti2bag/commit/a77c6dffe28fc110dca8f7be6167b5b43cfedbff for finding the ring in a angle based approach
Check https://github.com/VincentCheungM/Run_based_segmentation/issues/3#issuecomment-452686184 for finding the ring in a non-angle based approach


SAK-40211: resources > improve accuracy of new/updated resources email notification

https://jira.sakaiproject.org/browse/SAK-40211
The opening line of the current emails sent (when new resources are added or existing resources are updated) is always plural regardless of how many resources were added or updated:
Several new resources have been added to ...

Several resources have been updated in ...

This PR changes the behaviour to be plural or singular based on how many resources were added/updated:
anewresDnD = A new resource has been added to

anewresDnD.plural = {0} new resources have been added to

anewres2DnD = A resource has been updated in

anewres2DnD.plural = {0} resources have been updated in

Travis build failure unrelated:
[ERROR] Failed to execute goal com.github.eirslett:frontend-maven-plugin:1.3:gulp (gulp build) on project rubrics-service: Failed to run task: 'gulp.js build' failed. (error code 1) -> [Help 1]

Please rebase to include the changes of #5732


Hide FPS Text using lua

What is your question?
This might make me look like a dumbass, but how do i hide the fps text with lua?

setPropertyFromClass("Main","fpsVar.visible",false)

Thank you


This guide should show how to create Django/Postgres project with Django waiting for postgres to start.

File: compose/django.md
This example has very little use as nobody creates new django project inside running container but rather build container with earlier created django application. This tutorial would be perfect to present compose start-up order with bash script waiting for postgres container and only after that starting django app.

I agree. I saw a person who was new to docker use this tutorial and add their whole project to the container, because that's what the tutorial says. And then they were wondering why Docker is so slow for them.
There should be a mention of ADD not being required.

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!

I definetely agree to this point.


Collapsed feature does  not work

@davideas Cannot be collapsed when Item is selected.

Clearly a wanted feature as stated everywhere.

I've just updated from v5.0.0-b7 to v5.0.5 and collapsing an expandable header item (or clicking any item for that matter) is not working anymore. Is this issue related?


Vert.x4.2.2  (io.vertx.core.http.impl.Http1xClientConnection#createRequest) is not supported

Vert.x 4.2.2 的 io.vertx.core.http.impl.Http1xClientConnection.createRequest(io.vertx.core.http.HttpMethod method, java.lang.String uri, io.vertx.core.MultiMap headerMap, java.lang.String authority, boolean chunked, io.netty.buffer.ByteBuf buf, boolean end) createRequest()需要单独加上，这个方法的传参跟3.7.0不一致，拦截器里面还需要微调

Please use English.
We don't understand Chinese.

This place in Vert.x 3.7 createRequestMethod is createRequest(
"io.vertx.core.http.HttpMethod", "java.lang.String", "java.lang.String", "io.vertx.core.MultiMap", "java.lang.String", "boolean"
)
in Vert.x 4.2.2 createRequestMethod is createRequest(
"io.vertx.core.http.HttpMethod", "java.lang.String","io.vertx.core.MultiMap", "java.lang.String", "boolean",
"io.netty.buffer.ByteBuf", "boolean"
)
they are different. you should add this and add a new class
Http1xClientConnectionCreateRequestInterceptor and in this class ,you should Change a little. by the way, your pinpoint is very good! thx

@zhic0223
We will reflect it in the next patch version.
thank you

@jaehong-kim You're welcome.

#9164


Use discriminator in json-schemas (not openAPI) for better error messaging?

I know that discriminator is only part of openAPI and not base json-schema, but there is no particular reason someone couldn't include a discriminator inside their json_schema file. And if this library just happened to check for that and made better error messages, I personally at least wouldn't complain.

If I understand version 2.0 correctly you can set the metaschema to openapi and have json_schemer check all openapi keywords, including discriminator.
JSONSchemer.schema(myschema_that_uses_discriminator, meta_schema: JSONSchemer.openapi31)

Hmm when I try that it kind of works (certainly, recognizes the discriminator) but it also does strange stuff, such as when the discriminator value is something completely unknown, it tries to load it as a ref, and also the error messages aren't that specific. At least the first issue is different from what's found in https://github.com/davishmcclurg/json_schemer/blob/6c88530c594a953a11061c6abf642e451b1ad522/test/open_api_test.rb , so I suspect that the drop-in doesn't just work as-is. (and I don't want to convert everything to openapi to be able to do JSONSchemer.openapi(...), which might behave more like the linked test does)

Hmm when I try that it kind of works (certainly, recognizes the discriminator) but it also does strange stuff, such as when the discriminator value is something completely unknown, it tries to load it as a ref, and also the error messages aren't that specific.

Can you provide examples (schema and data) of the strange behavior? discriminator support is new and it's not well defined in the openapi spec, so I had to make some behavior guesses. You may also be hitting bugs.
Also, if you could provide the current error messages you're seeing as well as messages that you think would be more helpful, that would be great.

so I suspect that the drop-in doesn't just work as-is. (and I don't want to convert everything to openapi to be able to do JSONSchemer.openapi(...), which might behave more like the linked test does)

@ahx's example should work the same way that JSONSchemer.openapi does. You can provide meta_schema: JSONSchemer.openapi31 or set "$schema": "https://spec.openapis.org/oas/3.1/dialect/base" in your schema.

So to be clear, one can use a json-schema (without components, or an openapi version, or anything in the schema), and by passing the openapi meta schema, it should work?
As one example of puzzling behavior adapted from a test (actually using openAPI functions)

      openapi = {
        'openapi' => '3.1.0',
        'components' => {
          'schemas' => {
            'MyResponseType' => {
              'oneOf' => [
                { '$ref' => '#/components/schemas/Cat' },
                { '$ref' => '#/components/schemas/Dog' }
              ],
              'discriminator' => {
                'propertyName' => 'petType',
                'mapping' => {
                  'c' => '#/components/schemas/Cat',
                  'd' => 'Dog',
                  'dog' => 'Dog'
                }
              },
            },
            'Cat' => {
              'type' => 'object',
              'properties' => {
                'name' => {
                  'type' => 'string'
                }
              }
            },
            'Dog' => {
              'properties' => {
                'bark' => {
                  'type' => 'string'
                }
              }
            }
          }
        }
      }

      schemer = JSONSchemer.openapi(openapi).schema('MyResponseType')
      puts(schemer.validate({ 'petType' => 'foo' }).to_json)

When running this I get
     JSONSchemer::UnknownRef:
       json-schemer://schema/foo

To be clear, I know very little about how discriminators work, but if someone presents an invalid schema that has a discriminator field that is not valid, I feel like it shouldn't try to guess a ref, especially if there is an explicit mapping, since then validating anything suddenly becomes much trickier. I think the line from the spec if the discriminator value does not match an implicit or explicit mapping, no schema can be determined and validation SHOULD fail suggests that the consequence should be failed validation rather than an exception being thrown.

So to be clear, one can use a json-schema (without components, or an openapi version, or anything in the schema), and by passing the openapi meta schema, it should work?

Yep that's right.

To be clear, I know very little about how discriminators work, but if someone presents an invalid schema that has a discriminator field that is not valid, I feel like it shouldn't try to guess a ref, especially if there is an explicit mapping, since then validating anything suddenly becomes much trickier. I think the line from the spec if the discriminator value does not match an implicit or explicit mapping, no schema can be determined and validation SHOULD fail suggests that the consequence should be failed validation rather than an exception being thrown.

I think you're right. The spec says:

In both the oneOf and anyOf use cases, all possible schemas MUST be listed explicitly.

Your example uses oneOf, so it shouldn't be looking for schemas other than the ones listed. The allOf use case is where it gets tricky because the possible schemas aren't listed in the same way:

Specification `allOf` example
components:
  schemas:
    Pet:
      type: object
      required:
      - petType
      properties:
        petType:
          type: string
      discriminator:
        propertyName: petType
        mapping:
          dog: Dog
    Cat:
      allOf:
      - $ref: '#/components/schemas/Pet'
      - type: object
        # all other properties specific to a `Cat`
        properties:
          name:
            type: string
    Dog:
      allOf:
      - $ref: '#/components/schemas/Pet'
      - type: object
        # all other properties specific to a `Dog`
        properties:
          bark:
            type: string
    Lizard:
      allOf:
      - $ref: '#/components/schemas/Pet'
      - type: object
        # all other properties specific to a `Lizard`
        properties:
          lovesRocks:
            type: boolean

And they describe mapping as potentially being a reference:

An object to hold mappings between payload values and schema names or references.

Which I think means ref resolution is necessary at some point. I'll have to think about the best way to handle it. At a minimum, oneOf and anyOf should only look in the provided list. For allOf, it might make sense to catch UnknownRef (and InvalidRefResolution) and return a validation error instead.

EDIT: In this example, it is also a bit surprising to me that even if you provide an explicit mapping, then assert(schemer.valid?(MISTY.merge('petType' => 'Cat'))) should still work - it seems like if you provide an explicit mapping, then you shouldn't listen to any implicit mapping. Though I can't tell if the spec says to still do the implicit mapping as well anyways.

It is a little unclear (from the spec again):

Here the discriminator value of dog will map to the schema #/components/schemas/Dog, rather than the default (implicit) value of Dog.

"rather than" seems to indicate the implicit mapping no longer applies.

If you'd like a workaround for your original JSONSchemer::UnknownRef problem, I think you could use a ref resolver that always returns an error:
JSONSchemer.openapi(openapi, :ref_resolver => proc { { 'not' => true } })

I just opened a PR that I think fixes the issues you're running into: https://github.com/davishmcclurg/json_schemer/pull/145
Do you think you could test it out?

Thanks! I can confirm that when I change a test to purposely use a wrong discriminator value, and also remove the hack in the ref_resolver, it fails with the missing schema issue. Then, when using that branch instead, it now fails with a validation error as one would hope, value at `<path>` does not match `discriminator` schema. (It would be nice I feel like if the data_pointer field pointed to the field itself (the one specified by discriminator) instead of the whole object, but I guess there's reasons for the current behavior. But nevertheless it works.

Fix went out in 2.1.0.


[Bug]  babel-jest throws if only in babel.config.js excludes current file

I am upgrading jest 23.6.0 to jest 24.0.0.
They have deprecated setupTestFrameworkScriptFile , see: https://jestjs.io/docs/en/configuration#setupfilesafterenv-array
We used to have in our package.json:
"setupTestFrameworkScriptFile": "<rootDir>/internals/testing/test-bundler.js",

We have tried to update to :
"setupFilesAfterEnv": ["<rootDir>/internals/testing/test-bundler.js"],

We now have the following error in our test npm run test:
FAIL  src/Table/tests/Tbody.test.js
● Test suite failed to run

TypeError: Cannot read property 'config' of null

  at Object.getCacheKey (node_modules/babel-jest/build/index.js:127:22)

This is how our test-bundler.js:
const Enzyme = require('enzyme');
const Adapter = require('enzyme-adapter-react-16');
Enzyme.configure({ adapter: new Adapter() });

It fails when doing:
const babelOptions = loadBabelConfig("/home/dka/workspace/github.com/bootstrap-styled/v4", "/home/dka/workspace/github.com/bootstrap-styled/v4/internals/testing/test-bundler.js");
babelOptions is null
This is happening in babel-jest package after upgrading from 23.6.0 to 24.0.0
It seems that setupFilesAfterEnv behave differently, does anybody know how to fix it, I see many breaking change on their changlog since our two version but none says how to fix this.
reproduction
git clone git@github.com:bootstrap-styled/v4.git
cd v4
git checkout dev-jest24
npm i
npm test

There's also a detailled issue on stackoverflow
Travis build failure: https://travis-ci.org/bootstrap-styled/v4/jobs/486846842#L614

babel.transform() returns null if a file is ignored, and loadPartialConfig does the same because we stop processing the config ASAP to avoid doing extra work for ignored files.

This also fails when .test.js files are excluded with ignore in the .babelrc file.

Yeah, we need to handle Babel saying "nope". I'll put together a PR

Not sure how to handle code coverage. We currently do that by adding a babel plugin. Should we still do that even though a file is ignored by babel config?

I think I'll just throw an error saying that you need to also ignore the file in transformIgnorePatterns if babel wants to ignore the file

PR: #7797


Resolve 1.14 regressions

The following PRs have merged and introduced regressions. Creating a tracker issue with release blocker priority.

[x] https://github.com/istio/istio/pull/38499
[x] https://github.com/istio/istio/pull/38659. REVERT: https://github.com/istio/istio/pull/38863
[x] https://github.com/istio/istio/issues/38676

As per our offline discussion, summarized here for others, #38499 could be excluded from the above list. I am working on adding integration tests as well.

@vikaschoudhary16 As https://github.com/istio/istio/pull/38499 could be excluded from the above list, can you cross it off by clicking the mark on the left?

#38499 is reverted from the release branch, [release-1.14] Revert #38499 and #38702 . I am not able to click the box in the description.

Thank you, Vikas! I have clicked the box in the description.


Security update

Hi
nodemail is outdated and this package nodemailer before 6.4.16 use a crafted recipient email addresses and may result in arbitrary command flag injection in sendmail transport for sending mails.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: whitesource-bolt-for-github[bot]:x: c0d3x27You have signed the CLA already but the status is still pending? Let us recheck it.

Hi,
Thanks for your contribution. However, this PR cannot be merged in its state as-is. Check out the contributing guidlines: https://github.com/camunda/camunda-modeler/blob/691f5c400850ae3d3c06b93402546ccc2cfdd62f/.github/CONTRIBUTING.md
What needs to be done:

Commit history should be cleaned (in some commits you added files which were closed afterwards).
Commit messages should follow the Conventional Commits guidlines.
There is a problem with CLA.

Also, we are not subject to the vulnerability you mentioned because we only use nodemailer in one place in our pipeline. There is simply no place for injecting a crafted recipient address unless you take over the CI environment.

ok, i will just a whole new clean pull. im closing this one


Have a video in the enterprise pages of the mission control explaining the feature

The problem faced currently?
Currently we have some enterprise pages in mission control which has a Upgrade to avail feature kind of a message when SC is running on the open source version. We are not telling the user what the feature really is and how will she will be able to use it or apply it to her deployment.
How can we solve it?
Lets add a video explaining the feature in the alongside that boring message. This could perhaps be the same video we use in learn.

If you want this feature to be implemented, give it a thumbs up reaction, so that we can determine which features are important to you.
👍

No enterprise edition exists for this tool


Xml feature

What is this PR for?
To save note in Xml file.
What type of PR is it?
Feature
Todos

[ ] - To convert xml file to Java Object i.e unmarshalling
[ ] - To map config member in Paragraph to xml.

What is the Jira issue?
How should this be tested?
Screenshots (if appropriate)
Questions:

Does the licenses files need update?
Is there breaking changes for older versions?
Does this needs documentation?

@onkarshedge the goal of this small project is to get though all stages of contributing a change to the upstream project, using git workflow.
First requirement is that code in the PR should be "mergable" to the parent branch without a conflict. As you can see from github message above - right now it is no possible to merge your changes, as it will result in a conflict (you can see that doing locally "git merge master" from your branch, to see the conflicts).
Best thing to do in such case - pull latest parent from the upstream (this repo) and rebase your branch on top of that parent branch, and then force-push it again - this PR then gets updated.

@bzz I have forked the main apache/incubator-zeppelin repository created my xml-feature branch added 5 commits and then I sent the PR to your repo. I didn't clone your repository.
So then I added a remote upstreambzz refering this repo. created a new branch xml-feature-bzz, pulled from upstreambzz master and then rebased xml-feature.
So I am still getting conflicts from zeppelin-666 and zeppelin-689. conflicts from parts that I haven't modified also. I am thinking of creating a fresh PR from fork of this repo. is there other way around ?

@onkarshedge great!
Generally - you did a great job with initial implementation. One thing is that XmlNotebookRepo looks very much like copy-paste from JSON one (it even has note.json references in text). Can you please come up with the improvement that re-uese existing code from VFSNotebookRepo.java ? There might be small changes to VFSNotebookRepo.java impl to make it possible, but try to keep them really minimal.
Another thing is (after finishing with the above) - having that many commits in the Git history might be not the best idea, why do not you squash them to 1-2 really meaningful one? (git rebase -i is a blast, and is quite easy to use)


Siad needs to be running

when i run
Sia-UI
I get an error saying that siad needs to be running.
This should be started automatically

siad is started automatically, though it can take a while to startup. Personally, I prefer to run siad separately from the UI.

I do think that the UI should not close 'siad'. People were originally confused by the behavior that 'siad' would not shut down when the UI was closed, but it's important to the hostdb that 'siad' be running most of the time otherwise it won't know which hosts have good uptime.

This behavior has been specified and implemented as of #220 and #222.  I am therefore closing this issue.


Configure CI

[ ] Canary test to ensure the app starts up on an emulator
[ ] Ensure downloadable APKs are being generated

@timcheadle you will have to talk to bitrise.io and figure out if they will give us a special plan for being an open source project. If they won't, you will have to setup payment on bitrise -- our trial ends in 7 days.

I think this is all done.


Most feeds are missing from the ini files

The live Fedora Planet instance claims that there are 806 feeds: http://fedoraplanet.org/heads.html, whereas there are only a handful in people.ini.

FYI, the current planet setup runs on fedorapeople.org, it just looks for a ".planet" file in every users homedirectory.
Ideally however, we would use the new account systems "Website or Blog URL" field via fasjson.
We may also be able to change the account system so it requires https...

We still didnt retire the old planet thats why its showing +800 feeds there, but new planet also had missing feeds because it only allows HTTPS, that was fixed enforcing HTTPS in noggin and that PR was merged this week, so Im closing this issue.


Fix merging experimental options

Summary
It was reported that using the experimental to pass in nested options didn't always work. For example:
options = {
    "optimization_level": 0,
    "execution": {
        "shots": 1,
        "max_circuits": 40,
    },
    "twirling": {"strategy": "active-circuit", "gates": True, "measure": True},
}

Estimator(backend).run([(circuit, observable, parameter_values)]).result()

fails with
main() got an unexpected keyword argument 'max_circuits'
It turns out this could happen if the dictionary containing the experimental option (in this case execution) is the last one to be merged in, because in _update_options, the matched dictionary is not cleared after each iteration and the residual value can be reused.
Details and comments
Fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 79.985%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
5263

Relevant Lines:
6580

💛  - Coveralls


Updated repository URL from solidui to solidjs

Updating the URLs in this package with the new namespace.

Thank you


The necessary change for the release of the latest version

Summary
These are the necessary changes to release version 0.2.1 of qiskit-cold-atom.
Details and comments
The upload of 0.2.0 was sadly an old version that was locally stored. However, it was not possible to remove it again and hence a minor relase update was necessary.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Where parameters do not have relationship typings

https://github.com/themetalfleece/neogma/blob/da28eccb3221476d5d7693be65f84297d8ea0532/src/Queries/Where/Where.ts#L109-L116
The key of WhereParamsI is a string type. This can cause errors that are difficult to debug. For example:
user.findRelationships({
  alias: 'Group',
  where: { target: { grpName: 'Administrators' } },
});

If the model of user groups has a property called groupName then this will fail only at runtime when findRelationships returns zero results.
It would be better if the key of WhereParamsI were typed to be the keys of the related node properties.

That's true, but it's a bit difficult to type properly for nested relationships with where. I'll have a fresh look though.
Until then, you can use the satisfies keyword to add some inline type safety, like:
where: {
  name: 'Jason'
} satisfies Partial<UserProperties>


Add Cocoapods static_framework support

I see Cocoapods RC (1.4.0.rc.1) supports swift static frameworks. Added that to avoid creating a dynamic library.
The latest Cocoapods RC can be installed using this command sudo gem install cocoapods --pre

Enabling this will cause the binary to be linked statically.
If the pod consumer has multiple targets using LayoutKit, LayoutKit will be liked multiple times.
Agree with  @staguer it might be more appropriate for the consumers to decide how it will be linked.
Cocoapods 1.5.0 is adding support for this use case, FYI.


Replicate Fig.4-Right Experiment, Fix bug in Binary Leg F1-Score calculation

This pull request does two main things:
Makes changes to facilitate generating Fig.4-Right by replicating the contact experiment.

README.md was updated with specific instructions to ensure that the environment is properly setup and that python imports all work.
conda_env.yml updated to remove conflicting libraries.
Edited "contact_final_model_comparison.py" to have the proper experiment path.

Fix a bug in the Binary Leg F1-Score calculation to ensure accurate comparisons.
According to Wikipedia, F1-score should be calculated as follows:

However, the previous code calculated it as such:

Therefore:

Corrects the Binary Leg F1-Score Calculation (and by extension, the Legs Avg Binary F1-Score).
Wrote a test case to verify that the Binary Leg F1-Score was calculated correctly.

This is to partially deal with Issue #9, in addition to all the work that was already done by Danfoa.


Mod updates not working properly

I have had this same issue before.  I run my servers for months with no issue.  Everything updates properly for a while and then one day mod downloads aren't successful..  I have to use uninstall mods and removemods commands and sometimes it will fix it.  Sometimes I have to delete my Steam folder and then it starts working again.  Once I delete it I can issue the update command and it works perfectly...In the past I gave up trying to fix it and reinstalled.  It then worked for a while and the same happened again.  The past week I am constantly finding that mods aren't updated.   I don't change anything to cause it.  I set the servers up and haven't touched them since..  is there anything I can do to stop this from happening?

@KGM6911
I had a similar issue. My setup is different then yours in that I have only one staging directory for all three instances. This is what I did to get all my servers to update the mods.
First I did upgrade-tools to 1.6.38. Then I purged all "instance" related commands from the arkmanager.cfg file and placed them in the appropriate instance.cfg file.
Check out the Global configuration section and it will tell you what options cannot be overridden in the instance config files.
Look at it like this,
The following options cannot be overridden in the instance config files: (configures ark-server-tools)
The following options can be overridden on a per-instance basis: (configures ark survival servers)
I separated options based on what they control, making sure there was no cross duplication.
I removed my cron for "update @all", added individual cron jobs for each instance that looks like this:
0 0,4,8,12,16,20 * * * /home//bin/arkmanager update --warn --saveworld --backup --update-mods @scorched 2>&1 >> /home//cronlogs/arkupdtSE.log 2>&1
20 1,5,9,13,17,21 * * * /home//bin/arkmanager update --warn --saveworld --backup --update-mods @center 2>&1 >> /home//cronlogs/arkupdtcntr.log 2>&1
40 2,6,10,14,18,22 * * * /home//bin/arkmanager update --warn --saveworld --backup --update-mods @island 2>&1 >> /home//cronlogs/arkupdtisl.log 2>&1
Servers AND mods update every four hours but never closer then 1:20:00 to each other to control load.
This has worked for me. :)

I also use ark_GameModIds= in the instance.cfg

Thanks for the reply!   I think the main difference with what you wrote is that yours are running at different times.  My cron jobs are all running every 30 minutes.  Maybe it's just messing up now and then because it's trying to do it multiple times at the same time... I'll give it a shot your way.  Thanks again!

Not sure my way is the best way but it works for me :) Steam really messed things up when they took out manifest info on mod packages and went to date format. My logs show consistent updates on all 3 servers. There is usually no more then 8 hours difference between a triggered mod-update and all being updated. Now I can actually play on the weekends and not worry about whether I need to issue installmods and server restart commands.
No mod updates for me this past weekend but 6 server updates.

@KGM6911 An update on the cron jobs
I went back to 0 0,4,8,12,16,20 * * * /home//bin/arkmanager update --warn --saveworld --backup --update-mods @all because I was getting "Update download unsuccessful" when the cron starts the update for each instance and does not finish before the next cron job.
As of 2/11, 2 server patches and 1 mod update, all three of my servers are updating flawlessly.
Seems that running with the @all argument updates the servers successively and not concurrently like the cron jobs potentially could. BTW, I have not yet upgrade the server-tools to https://github.com/FezVrasta/ark-server-tools/releases/tag/v1.6.39 because I was not experiencing the issue in #923 but I did however make use of @klightspeed request to use --verbose and the output was very useful.


SingleTestContainer only renders if there is TestContent to render

TestContainer component only renders SingleTestContainer if there is TestContent on the response object

LGTM


Error reporting needs work

See https://twitter.com/gustavorobertux/status/1143545880601079808 for an example
An empty response from cert spotter results in an umarshal error, and it's reported badly.
Given it's perfectly legitimate for there to be no results it might be a good idea just to swallow these errors.

assetfinder 0.1.0 enum the false domin

is this issue up for grabs?


luci-app-package-manager: apk update: fetch of package.adb index is shown in Errors output

Is there an existing issue for this?

[X] I have searched the existing issues

screenshots or captures

Actual behaviour

Go to https://<IP_ADDRESS>/cgi-bin/luci/admin/system/package-manager
Click on Update lists... action button
Once the apk update finishes it shows the output in the Errors section, which is misleading as its not an error

Errors
fetch https://downloads.openwrt.org/snapshots/targets/x86/64/packages/packages.adb
fetch https://downloads.openwrt.org/snapshots/packages/x86_64/base/packages.adb
fetch https://downloads.openwrt.org/snapshots/targets/x86/64/kmods/6.6.60-1-2e46b6878177e1dea330c0dbbbcfb6be/packages.adb
fetch https://downloads.openwrt.org/snapshots/packages/x86_64/luci/packages.adb
fetch https://downloads.openwrt.org/snapshots/packages/x86_64/packages/packages.adb
fetch https://downloads.openwrt.org/snapshots/packages/x86_64/routing/packages.adb
fetch https://downloads.openwrt.org/snapshots/packages/x86_64/telephony/packages.adb

Expected behaviour
Only errors/issues are shown in the Errors section.
Steps to reproduce

Go to https://<IP_ADDRESS>/cgi-bin/luci/admin/system/package-manager
Click on Update lists... action button

Additional Information
{
	"kernel": "6.6.60",
	"hostname": "OpenWrt",
	"system": "13th Gen Intel(R) Core(TM) i9-13900HX",
	"model": "QEMU Standard PC (Q35 + ICH9, 2009)",
	"board_name": "qemu-standard-pc-q35-ich9-2009",
	"rootfs_type": "squashfs",
	"release": {
		"distribution": "OpenWrt",
		"version": "SNAPSHOT",
		"revision": "r28056-40b8fbaa97",
		"target": "x86/64",
		"description": "OpenWrt SNAPSHOT r28056-40b8fbaa97",
		"builddate": "1731280190"
	}
}

What browsers do you see the problem on?
Firefox
Relevant log output
No response

Hi @ynezz
At a cursory glance at the source, it appears that a script captures stderr and stdout. So anything landing in the error output is what is, in fact, output by apk to stderr? ( I've not yet verified or run any new apk builds )
https://github.com/openwrt/luci/blob/d6506880edb0cfbd8e94790911a353a57b23f5c4/applications/luci-app-package-manager/root/usr/libexec/package-manager-call#L90-L99
Could you test on your installation, comment out the later line which does
https://github.com/openwrt/luci/blob/d6506880edb0cfbd8e94790911a353a57b23f5c4/applications/luci-app-package-manager/root/usr/libexec/package-manager-call#L109
And double-check the contents of those files?

Yes, its the case
{
	"code": 0,
	"pkmcmd": "apk update ",
	"stdout": " [https://downloads.openwrt.org/snapshots/targets/x86/64/packages/packages.adb]\n [https://downloads.openwrt.org/snapshots/packages/x86_64/base/packages.adb]\n [https://downloads.openwrt.org/snapshots/targets/x86/64/kmods/6.6.60-1-2e46b6878177e1dea330c0dbbbcfb6be/packages.adb]\n [https://downloads.openwrt.org/snapshots/packages/x86_64/luci/packages.adb]\n [https://downloads.openwrt.org/snapshots/packages/x86_64/packages/packages.adb]\n [https://downloads.openwrt.org/snapshots/packages/x86_64/routing/packages.adb]\n [https://downloads.openwrt.org/snapshots/packages/x86_64/telephony/packages.adb]\nOK: 10505 distinct packages available",
	"stderr": "fetch https://downloads.openwrt.org/snapshots/targets/x86/64/packages/packages.adb\nfetch https://downloads.openwrt.org/snapshots/packages/x86_64/base/packages.adb\nfetch https://downloads.openwrt.org/snapshots/targets/x86/64/kmods/6.6.60-1-2e46b6878177e1dea330c0dbbbcfb6be/packages.adb\nfetch https://downloads.openwrt.org/snapshots/packages/x86_64/luci/packages.adb\nfetch https://downloads.openwrt.org/snapshots/packages/x86_64/packages/packages.adb\nfetch https://downloads.openwrt.org/snapshots/packages/x86_64/routing/packages.adb\nfetch https://downloads.openwrt.org/snapshots/packages/x86_64/telephony/packages.adb"
}

@systemcrash magical way of handling apk... But it's probably not intended that those fetch command goes in stderr... I will propose a fix upstream

Just to be sure we aren't on a wild goose chase, @ynezz can you test an equivalent apk command and try redirecting the relevant outputs? To see if it really is apk to blame?

Just to be sure we aren't on a wild goose chase, @ynezz can you test an equivalent apk command and try redirecting the relevant outputs? To see if it really is apk to blame?

https://gitlab.alpinelinux.org/alpine/apk-tools/-/blob/master/libfetch/common.c?ref_type=heads#L176

Wow.

Well i checked the usage of fetch_info and it's really mixed for both error condition and real info so it just need some love :D

BTW it's not even there... fetch comes from database.c

https://gitlab.alpinelinux.org/alpine/apk-tools/-/merge_requests/243
lets see how they get this... Eventually we can experiment with the -q option and see if stdout is printed anyway....

The reason for the duplicate output is that apk update shows a list of the repository description followed by the url/path. This is how it is meant to look:
~$ doas apk update
fetch https://mirror.postmarketos.org/postmarketos/master/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/edge/community/x86_64/APKINDEX.tar.gz
fetch https://dl-cdn.alpinelinux.org/alpine/edge/testing/x86_64/APKINDEX.tar.gz
main v20240923-4045-g4edeca90fc3 [/home/sertonix/.local/share/abuild/main]
community v20240923-4094-gb9c0be0959f [/home/sertonix/.local/share/abuild/community]
testing v20240923-4156-g4a3a35b9c4f [/home/sertonix/.local/share/abuild/testing]
2024-11-13 08:01:54.161666+00:00 [https://mirror.postmarketos.org/postmarketos/master]
v20240923-4215-gd1699ddc803 [https://dl-cdn.alpinelinux.org/alpine/edge/main]
v20240923-4235-gdf4f1f21465 [https://dl-cdn.alpinelinux.org/alpine/edge/community]
v20240923-4234-gf00c9daaa6c [https://dl-cdn.alpinelinux.org/alpine/edge/testing]
OK: 33164 distinct packages available

It looks like your repositories ether don't have a description or apk is not handling descriptions of adb repositories correctly.

@sertonix are you sure that output only comes from stdout and it's not mixed stderr and stdout?

It is mixed stderr and stdout. I agree that this doesn't make sense. I am uncertain how to fix this. Just wanted to mention that the proposed fix introduces a regression.


ipfs object patch not supported

Looking at the ipfs object methods, it seems that ipfs object patch is not supported.
This would be super-useful for partial updates to IPFS hashes.

Hey @bergie, @haadcode PR'ed this yesterday on https://github.com/ipfs/js-ipfs-api/pull/103
Please make sure to send a hi5 :)


Fixes and and tests for issues #23, merging code from PR #19

Will follow up with fixes later, verifying the tests find the problem first

Hi @TravisEz13, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Travis Plunk). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Merged PR #19

Fixes #23

Reviewed. Looks good to me.


strtolower on array doesn't work

I think there is a big bug in your module
At GravatarLib/Gravatar.php is on line 136 a type mismatch.
You're trying to cast the config at laravel-gravatar/config/gravatar.php to a string, but its an array.

Is this speculation of do you have a test case or stack trace where you are hitting the error?
This is the line you reference, which is in a method to set the default avatar.
https://github.com/thomaswelton/gravatarlib/blob/master/Gravatar.php#L136
This method is only ever called with a string from inside this lib
https://github.com/thomaswelton/laravel-gravatar/search?utf8=✓&q=setDefaultImage&type=

Hello @thomaswelton
I have a test case and a stack trace for you. I dont know why it happends.
(1/4) ErrorException
strtolower() expects parameter 1 to be string, array given
in Gravatar.php (line 136)
at HandleExceptions->handleError(2, 'strtolower() expects parameter 1 to be string, array given', '/home/philipp/Workspace/tpd/vendor/thomaswelton/gravatarlib/thomaswelton/GravatarLib/Gravatar.php', 136, array('image' => array('size' => 80, 'fallback' => 'mm', 'secure' => false, 'maximumRating' => 'g', 'forceDefault' => false, 'forceExtension' => 'jpg'), 'this' => object(Gravatar)))
at strtolower(array('size' => 80, 'fallback' => 'mm', 'secure' => false, 'maximumRating' => 'g', 'forceDefault' => false, 'forceExtension' => 'jpg'))
in Gravatar.php (line 136)
at Gravatar->setDefaultImage(array('size' => 80, 'fallback' => 'mm', 'secure' => false, 'maximumRating' => 'g', 'forceDefault' => false, 'forceExtension' => 'jpg'))
in Gravatar.php (line 25)
at Gravatar->__construct(object(Repository))
in LaravelGravatarServiceProvider.php (line 33)
at LaravelGravatarServiceProvider->Thomaswelton\LaravelGravatar\{closure}(object(Application), array())
in Container.php (line 726)
at Container->build(object(Closure))
in Container.php (line 608)
at Container->resolve('gravatar')
in Container.php (line 575)
at Container->make('gravatar')
in Application.php (line 728)
at Application->make('gravatar')
in Container.php (line 1172)
at Container->offsetGet('gravatar')
in Facade.php (line 159)
at Facade::resolveFacadeInstance('gravatar')
in Facade.php (line 128)
at Facade::getFacadeRoot()
in Facade.php (line 215)
at Facade::__callStatic('src', array('test123@mail.ru'))
in 86f3ba339f4890af663214000bba76e279bf6e30.php (line 115)
at<EMAIL_ADDRESS>in 86f3ba339f4890af663214000bba76e279bf6e30.php (line 115)
at include('/home/philipp/Workspace/tpd/storage/framework/views/86f3ba339f4890af663214000bba76e279bf6e30.php')
in PhpEngine.php (line 42)
at PhpEngine->evaluatePath('/home/philipp/Workspace/tpd/storage/framework/views/86f3ba339f4890af663214000bba76e279bf6e30.php', array('__env' => object(Factory), 'app' => object(Application), 'errors' => object(ViewErrorBag), 'signedIn' => true, 'user' => object(User), 'obLevel' => 2))
in CompilerEngine.php (line 59)
at CompilerEngine->get('/home/philipp/Workspace/tpd/resources/views/vendor/adminlte/layouts/partials/mainheader.blade.php', array('__env' => object(Factory), 'app' => object(Application), 'errors' => object(ViewErrorBag), 'signedIn' => true, 'user' => object(User), 'obLevel' => 2))
in View.php (line 137)
at View->getContents()
in View.php (line 120)
at View->renderContents()
in View.php (line 85)
at View->render()
in a66e92ec166d3006a311dd3df9b810334cdda9e4.php (line 36)
at include('/home/philipp/Workspace/tpd/storage/framework/views/a66e92ec166d3006a311dd3df9b810334cdda9e4.php')
in PhpEngine.php (line 42)
at PhpEngine->evaluatePath('/home/philipp/Workspace/tpd/storage/framework/views/a66e92ec166d3006a311dd3df9b810334cdda9e4.php', array('__env' => object(Factory), 'app' => object(Application), 'errors' => object(ViewErrorBag), 'signedIn' => true, 'user' => object(User), 'obLevel' => 1))
in CompilerEngine.php (line 59)
at CompilerEngine->get('/home/philipp/Workspace/tpd/resources/views/vendor/adminlte/layouts/app.blade.php', array('__env' => object(Factory), 'app' => object(Application), 'errors' => object(ViewErrorBag), 'signedIn' => true, 'user' => object(User), 'obLevel' => 1))
in View.php (line 137)
at View->getContents()
in View.php (line 120)
at View->renderContents()
in View.php (line 85)
at View->render()
in 666bbd5e4ae80e5419a8a7f070e80d7c6da65b7f.php (line 36)
at include('/home/philipp/Workspace/tpd/storage/framework/views/666bbd5e4ae80e5419a8a7f070e80d7c6da65b7f.php')
in PhpEngine.php (line 42)
at PhpEngine->evaluatePath('/home/philipp/Workspace/tpd/storage/framework/views/666bbd5e4ae80e5419a8a7f070e80d7c6da65b7f.php', array('__env' => object(Factory), 'app' => object(Application), 'errors' => object(ViewErrorBag), 'signedIn' => true, 'user' => object(User)))
in CompilerEngine.php (line 59)
at CompilerEngine->get('/home/philipp/Workspace/tpd/resources/views/vendor/adminlte/home.blade.php', array('__env' => object(Factory), 'app' => object(Application), 'errors' => object(ViewErrorBag), 'signedIn' => true, 'user' => object(User)))
in View.php (line 137)
at View->getContents()
in View.php (line 120)
at View->renderContents()
in View.php (line 85)
at View->render()
in Response.php (line 38)
at Response->setContent(object(View))
in Response.php (line 201)
at Response->__construct(object(View))
in Router.php (line 617)
at Router->prepareResponse(object(Request), object(View))
in Router.php (line 575)
at Router->Illuminate\Routing\{closure}(object(Request))
in Pipeline.php (line 30)
at Pipeline->Illuminate\Routing\{closure}(object(Request))
in GuestUser.php (line 24)
at GuestUser->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 Authenticate.php (line 43)
at Authenticate->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 VerifyCsrfToken.php (line 65)
at VerifyCsrfToken->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 ShareErrorsFromSession.php (line 49)
at ShareErrorsFromSession->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 StartSession.php (line 64)
at StartSession->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 AddQueuedCookiesToResponse.php (line 37)
at AddQueuedCookiesToResponse->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 EncryptCookies.php (line 59)
at EncryptCookies->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 Router.php (line 576)
at Router->runRouteWithinStack(object(Route), object(Request))
in Router.php (line 535)
at Router->dispatchToRoute(object(Request))
in Router.php (line 513)
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 Debugbar.php (line 51)
at Debugbar->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 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)

But the config is like yours:
<?php

return array(
	// --- The default avatar size
	'size' => 80,

	// --- The default avatar to display if we have no results
	// (bool)   false
	// (string) 404
	// (string) mm: (mystery-man) a simple, cartoon-style silhouetted outline of a person (does not vary by email hash).
	// (string) identicon: a geometric pattern based on an email hash.
	// (string) monsterid: a generated 'monster' with different colors, faces, etc.
	// (string) wavatar: generated faces with differing features and backgrounds.
	// (string) retro: awesome generated, 8-bit arcade-style pixelated faces.
	'default' => 'identicon',

	// --- Set the type of avatars we allow to show
	// - g: suitable for display on all websites with any audience type.
	// - pg: may contain rude gestures, provocatively dressed individuals, the lesser swear words, or mild violence.
	// - r: may contain such things as harsh profanity, intense violence, nudity, or hard drug use.
	// - x: may contain hardcore sexual imagery or extremely disturbing violence.
	'maxRating' => 'g'
);

And I didn't changed the constructor:
public function __construct(Config $config)
    {
        // Set default configuration values
        $this->setDefaultImage($config->get('gravatar.default'));
        $this->defaultSize = $config->get('gravatar.size');
        $this->setMaxRating($config->get('gravatar.maxRating', 'g'));

        $this->enableSecureImages();
    }

With this being over 2 years old, it can probably be closed out.


Skip empty binary name to be searched on Windows (#993)

Without the change, if PATH environment variable contains a component
where the component name + ".exe" exists, the exe is used as well as gpg.
I'm not sure bin is ever non empty though.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1018   +/-   ##
=======================================
  Coverage   63.57%   63.57%
=======================================
  Files         173      173
  Lines        9965     9965
=======================================
  Hits         6335     6335
  Misses       2842     2842
  Partials      788      788

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 27bd082...43e23ae. Read the comment docs.


[BUG] Publisher Error PessimisticConcurrencyConflict - Operation on the API is in progress

Release version
v4.10.0
Describe the bug
Publisher version 4.10.0 is throwing PessimisticConcurrencyConflict error:
Error Logs:
crit: Publisher[0]       System.Net.Http.HttpRequestException: HTTP request to URI https://management.azure.com/subscriptions/***/resourceGroups/<RG>/providers/Microsoft.ApiManagement/service/<APIM>/apis/data-lookup-apiv1?api-version=2022-04-01-preview failed with status code 409. Content is '{"error":{"code":"PessimisticConcurrencyConflict","message":"Operation on the API is in progress","details":null}}'.          at common.HttpPipelineExtensions.Validate(Response response, Uri requestUri)          at common.HttpPipelineExtensions.PutResource(HttpPipeline pipeline, Uri uri, JsonObject resource, CancellationToken cancellationToken)          at publisher.Program.<>c__DisplayClass12_0.<<GetPutRestResource>b__0>d.MoveNext()       --- End of stack trace from previous location ---          at publisher.Api.PutApi(ApiName apiName, ApiInformationFile apiInformationFile, ApiSpecificationFile specificationFile, JsonObject configurationApiJson, ServiceUri serviceUri, PutRestResource putRestResource, ILogger logger, CancellationToken cancellationToken)          at publisher.Api.<>c__DisplayClass23_0.<<ProcessArtifactsToPut>b__4>d.MoveNext()       --- End of stack trace from previous location ---          at System.Threading.Tasks.Parallel.<>c__501.<b__50_0>d.MoveNext()
--- End of stack trace from previous location ---
at common.IEnumerableExtensions.ForEachParallel[T](IEnumerable1 enumerable, Func2 action, CancellationToken cancellationToken)
at publisher.Api.ProcessArtifactsToPut(IReadOnlyCollection1 files, JsonObject configurationJson, ServiceDirectory serviceDirectory, ServiceUri serviceUri, PutRestResource putRestResource, ILogger logger, CancellationToken cancellationToken)          at publisher.Service.ProcessArtifactsToPut(IReadOnlyCollection1 files, JsonObject configurationJson, ServiceDirectory serviceDirectory, ServiceUri serviceUri, ListRestResources listRestResources, PutRestResource putRestResource, DeleteRestResource deleteRestResource, ILogger logger, CancellationToken cancellationToken)
at publisher.Publisher.RunWithoutCommitId(CancellationToken cancellationToken)
at publisher.Publisher.Run(CancellationToken cancellationToken)
at publisher.Publisher.ExecuteAsync(CancellationToken cancellationToken)
info: Microsoft.Hosting.Lifetime[0]
Application is shutting down...
fail: Microsoft.Extensions.Hosting.Internal.Host[9]
BackgroundService failed
`
Expected behavior
All artifacts should be published without errors
Actual behavior
After running the publisher using publish-all-artifacts, its throwing PessimisticConcurrencyConflict error whenever executing a PUT operation to an existing APIs in APIM:
System.Net.Http.HttpRequestException: HTTP request to URI https://management.azure.com/subscriptions/***/resourceGroups/<RG>/providers/Microsoft.ApiManagement/service/<APIM>/apis/data-lookup-apiv1?api-version=2022-04-01-preview failed with status code 409. Content is '{"error":{"code":"PessimisticConcurrencyConflict","message":"Operation on the API is in progress","details":null}}'.
Reproduction Steps
STEP:

Make sure you have existing APIs in APIM.
Run the Publisher pipeline with publish-all-artifacts
Check the pipeline logs

duplicate #141


Feature/tin/implement all animation states

I have implemented all animation states.
Right now I can't figure out how you would use to switch the player's state, I tested what I did to make sure it was done properly by changing this->state at line 9 of character.cpp .
I also finished putting the origin of each animation in place, animations have been aligned with the player's hitbox very well.

Note that the origin of the animation is not always width/2, height/2 (for example, in the case of skeleton attack, we have to put the center of the animation in the center of the skeleton's chest, not width/2, height/2)
We have many animations with different sizes so animate manually in the Skeleton constructor is necessary. I'll shorten the code by writing an animation constructor with a string, I'll find a way to parse the string as discussed earlier.

Close due to new implementation at https://github.com/nitro2/BeatThemAll/pull/18


Adding guestbook, minio and mongodb-shards yamls for K8S documentation

YAMLs for

GuestBook
minio stateful set distributed
minio standalone
mongodb-shards - 4 node deployment

I have tested deploying these YAMLs on Kubernetes v1.7.0-beta.2
I am not authorized to push to this branch - "kube-examples", as it is protected branch.
cc:  @luomiao @BaluDontu @rohitjogvmw @tusharnt

/approve

/lgtm

/approve


Implement Named argument rule

Related to https://github.com/twigphp/Twig/pull/4474

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.91%. Comparing base (d9fb981) to head (bc38e06).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##               main     #333   +/-   ##
=========================================
  Coverage     99.91%   99.91%
- Complexity      764      769    +5
=========================================
  Files            66       67    +1
  Lines          2358     2381   +23
=========================================
+ Hits           2356     2379   +23
  Misses            2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Done

Description
Please describe the changes in your pull request in few words here.
Changes
List the changes done to fix a bug or introducing a new feature.
How to test
Describe the steps required to test the changes proposed in the pull request.

@Bharadwajshivam28 - use this tutorial https://www.geeksforgeeks.org/how-to-add-gifs-on-readme-md-file-in-a-github-repository/ to embedded gifs in a readme.

@Bharadwajshivam28 - use this tutorial https://www.geeksforgeeks.org/how-to-add-gifs-on-readme-md-file-in-a-github-repository/ to embedded gifs in a readme.

Ok sir thanks for your reply it will be great for me learing under you.
Will do this work

@Mr-Sunglasses Updated it sir


#.junit.xml: No such file or directory

I can't seem to get this working. When I run a test, I get an error saying that a file 3.junit.xml doesn't exist.
I searched for the issue and saw that someone reported an issue with it but they didn't have cargo-nextest installed. I do, v0.9.48.
Did I misconfigure something?

Same issue here

I can't reproduce. Does this happen with a minimal example or only with some files?

I can't reproduce. Does this happen with a minimal example or only with some files?

I tried to make minimal example, Is this same error?
https://github.com/ShortArrow/PoC_neotest_rust_issue_20

I can't reproduce. Does this happen with a minimal example or only with some files?

I tried to make minimal example, Is this same error? https://github.com/ShortArrow/PoC_neotest_rust_issue_20

Thanks, I can reproduce with that! I'm not sure when I'll have time to look into this more, but this is a huge help!

I can't reproduce. Does this happen with a minimal example or only with some files?

I tried to make minimal example, Is this same error? https://github.com/ShortArrow/PoC_neotest_rust_issue_20

Oh, this issue here is that the build fails, not the test, so there is not result file. I hadn't considered that.

Maybe the same issue? #22

I may have just found a clue:

or, in lay and text terms:
error: failed to parse nextest config at C:\Users\sias.mey\AppData\Local\Temp\nvim.0\WFp1tH\1.nextest.tomlCaused by:
invalid hex escape character in string: s at line 2 column 13 in ................\Temp\nvim.0\WFp1tH\1.nextest.toml
]0;C:\WINDOWS\SYSTEM32\cmd.exe

Fixed:
I will do a PR tomorrow, I cant test on other OS, but this makes it work for me in windows.

Just swapping " for ' in the config toml
Line 135 of lua/init.lua

That seems to fix it for me too. I was able to run a test with that change in place!

I’d love a PR with the fix and ideally a new test!

I should have some time this weekend; I will try and do a PR if SiasMay doesn't get to it first.

My week exploded, sorry.. But I am very glad that it seems I got to the root cause of the issue for a few folks.
We should probably report upstream, since I am not sure the intent of config parsing should be to allow escape sequences like that. At least, for the moment we know that ' is much safer than " in situations like this.

No worries, I totally understand.
I submitted a PR (https://github.com/rouge8/neotest-rust/pull/24)
@rouge8 I didn't add tests because I'm unfamiliar with the process in lua and couldn't get make test to run properly in an unmodified repo (details in the PR).

On Arch linux.
I was tried inject this.
writer:write("[profile.neotest.junit]\npath = '" .. junit_path .. "'")
print("[profile.neotest.junit]\npath = '" .. junit_path .. "'")

result was here.
[profile.neotest.junit]
path = "/tmp/nvim.who/2yVsIM/10.junit.xml"

Error executing vim.schedule lua callback: ...share/nvim/lazy/plenary.nvim/lua/plenary/async/async.lua:18: The coroutine failed with this message: ...e/nvim/lazy/plenary.nvim/lua/plenary/context_manager.lua:47: /tmp/nvim.who/2yVsIM/10.junit.xml: No such file or directory
stack traceback:
[C]: in function 'error'
...share/nvim/lazy/plenary.nvim/lua/plenary/async/async.lua:18: in function 'callback_or_next'
...share/nvim/lazy/plenary.nvim/lua/plenary/async/async.lua:45: in function <...share/nvim/lazy/plenary.nvim/lua/plenary/async/async.lua:44>

Therefore, it seems that the escape character is not the cause of not being able to find a file in Linux.
I have two hypotheses.

There is a problem in cooperation with Upstream with Plenary, and it has disappeared before reading the .junit.xml file.
No .junit.xml file is generated

If "no file is generated" has expected behavior then, skip .junit.xml and need to make load feature 11 file or 2,5,8.
I want to make a PR if possible. But don't expect me. I have just started using Lua language. Of course I will make an effort.

I can't reproduce. Does this happen with a minimal example or only with some files?

I tried to make minimal example, Is this same error? https://github.com/ShortArrow/PoC_neotest_rust_issue_20

Oh, this issue here is that the build fails, not the test, so there is not result file. I hadn't considered that.

Failing builds will now no longer crash and will now display the build output.
The single vs double quote thing might be an issue with Windows paths? If someone can reproduce it on non-Windows, I might take a look.


Update README.md

correction of typos in readme.md
thank you :)

@ObsidianMaximus Thanks for the minor corrections!

No problem ma'am 😄


Feat: first-pass index.html

Closes #2
Summary
Adds a basic index.html file which mirrors our business cards with Compiler branding. Currently not utilizing any Jekyll features yet.

I might put this back into draft so that I can add media queries for mobile/tablet breakpoints in our CSS.

@thekaveman we actually have a favicon!
google link

Invert colors to mirror future website designs

[ ] Background color changed to #262626
[ ] Change font-color: #ffffff
[ ] Logo changed to white version with teal #00B3CA line "_." https://drive.google.com/open?id=1M0kuSPcwZkHP1pc2Fl_O6rP4z5u2oZoi&authuser=anthony%40compiler.la&usp=drive_fs

I am not finding the white logo with the teal line in the branding folders, it should be the one in the drive link above, but it appears to be the wrong file.

[ ] Icon inverted to https://drive.google.com/open?id=1k4OaFR6LG5gMIapGe8h55Zot25t__5V8&authuser=anthony%40compiler.la&usp=drive_fs

Change email link<EMAIL_ADDRESS>color and text decoration

[ ] Change on-hover transition to green #92B252
[ ] Add text-decoration: underline

Adding a underline is important to accessibility

@AnthonyRollins Thank you for the detailed list of changes! I was wondering where you found the values for #00B3CA as teal and #92B252 as green.
I was referencing a PDF in Google Drive named Compiler Style Guide V2.pdf, but I'm not sure if that's the "official"/finalized resource. It has #01B4CB as teal and #88B440 as green.
Here's a screenshot of the page I was looking at... interestingly though, some colors in the PDF don't exactly match what's listed (see teal as an example).

Task linked: CU-31era6j k

@AnthonyRollins here's a test comment to see if it pings you in Slack

@AnthonyRollins here's a test comment to see if it pings you in Slack

I pulled the colors from the Figma designs: https://www.figma.com/file/fkRyBpWGDyaBh1kzaXIljo/Compiler-website?node-id=0%3A1. I would imagine the PDF brand guide has the color codes we should be using.

We're self-hosting the fonts since there isn't any performance gain to using CDN versions
https://wicki.io/posts/2020-11-goodbye-google-fonts/


lyr.stac.bands disappears

http://localhost:8080/test/stac-item/band-info.html test passes, but when I click on the lyr, the callback data doesn't appear to include bands.

Was this already the case in 0.11.0 with Leaflet 1.7?

Yes, I assume so.

I checked the implementation and currently stac-layer just passes the eo:bands completely and not the bands that are actually shown/used. As such the implementation can be removed and replaced with stac-js functionality to retrieve bands. Long-term we should actually return the bands used, but that's not even the case right now.

Sounds like a good plan


[Krest] [peaq] Add Price ID for KREST

Please support displaying the Krest token's price in SubWallet.
KREST has been listed on Coingecko at: https://www.coingecko.com/en/coins/krest

Validate in https://github.com/Koniverse/SubWallet-Extension/issues/2198


Set up a "slow" marker for pytest

xref #114 #463

Add pytest support for a custom marker named "slow"
github action for pull requests will not run slow tests
set up nightly(?) CI that runs all tests including "slow".
What should the default be for running tests on a local workstation? probably skip them, right?

This is done


Add awesome-cl

For Common Lisp: https://github.com/CodyReichert/awesome-cl
Awesome-cl was added (https://github.com/sindresorhus/awesome/commit/8236f1a0feda0aa69273f1088234e69b28362a04), then removed (https://github.com/sindresorhus/awesome/commit/c498a389169e49f049f44f97cc0cabec3621dc5c), and now hopefully re-added to the list.
I've taken over maintenance on the repo and moved it back to GitHub.

Cool :)
Missing a contributing guideline. Read https://github.com/sindresorhus/awesome/blob/master/awesome.md

Would be awesome if you add the awesome badge to the list.
https://github.com/sindresorhus/awesome/blob/master/awesome.md#awesome-badge

Done and done. Thanks


Improved consistency

Changed "or" to "and" because all the other sentence sections use "and".

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove not important keywords from generated JSON schemas

They are redundant, but still may be processed by hypothesis-jsonschema and affect performance negatively. Also, it is extra data that may be copied (e.g., in Open API links implementation), which can cost some time as well since it can be performed many times during a test run.
Example:

in
name

So instead of this:
{
    "properties": {"code": {"in": "query", "name": "code", "type": "integer"}},
    "additionalProperties": False,
    "type": "object",
    "required": ["code"],
}

We'll have this:
{
    "properties": {"code": {"type": "integer"}},
    "additionalProperties": False,
    "type": "object",
    "required": ["code"],
}

NOTE. On small schemas, it is almost not noticeable, but the larger the schema is, the more significant the effect - internally, schemas are passed to json.dumps many times, especially in upcoming recursive schemas. The result is visible in such cases, but on the Schemathesis level, it is a one-time operation and worth the efforts.
Implementation notes

Update the schemathesis.specs.openapi.converter.to_json_schema function and drop all irrelevant keywords
Relevant keywords could be found in the JSON Schema spec

Resolved via #862


Update machine-learning-algorithm-choice.md

Updated author info

Hi @brohrer-ms, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (brohrer). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@garyericson
Please review and approve this PR here, thanks.

This looks good. Please accept.


Add another workflow to allow running regressions on-demand

Supports running Windows and Linux regressions on demand.

Codecov Report

Merging #1898 (5781a24) into master (b3e061e) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1898      +/-   ##
==========================================
- Coverage   24.19%   24.19%   -0.01%
==========================================
  Files         191      191
  Lines       31359    31360       +1
==========================================
  Hits         7586     7586
- Misses      23773    23774       +1

Impacted Files
Coverage Δ

src/DesignCompile/NetlistElaboration.cpp
28.59% <0.00%> (-0.02%)
:arrow_down:

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 b3e061e...5781a24. Read the comment docs.


Add a funtion to get image id

I added this function because add_image requires mut and sometimes it inconvenient because you can't use it inside show.

That's a good addition, thanks!


Remove "but" to avoid link with previous statement

SSH from private VNet still requires SSH to be enabled. Otherwise, Cx will not have SSH user / key details.

@sdgilley

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


vscode cannot view image

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:Version: 1.71.0 (system setup)
OS Version:windows10

Steps to Reproduce:

I cannot view image jpg,png etc. though I uninstall vscode and then install it again

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Revert "python3Packages.smartypants: fix src url"

This reverts commit acc3b5ad904804755d33a44792ea24d8f707a35c.

Motivation for this change
with 5001+ rebuilds this should have gone to staging.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Going to skip ofborg here to minimize the damage done by the rebuilds.


[UBS-User. Orders tab view] Order status should be "Узгоджено" not "Узгодження"

Environment: Chrome (latest)
Reproducible: always
Steps to reproduce

Logged in to GreenCity as a User https://ita-social-projects.github.io/GreenCityClient/#
The tab "Замовлення" in the personal cabinet is opened
The client has a few orders
Go to DB greencity-ubs
Change Order status "Сформовано" to "Узгоджено" for the record № 1

Actual result
Order status became  "Узгодження"

Expected result
Order status became  "Узгоджено"
User story and test case links
E.g.: "User story #3477
Test case
Labels to be added
"Bug", Priority ("pri: high"), Severity ("severity: trial"), Type ("UI")

Order status is still 'Узгодження'

Hi. Yes 'Узгодження'. Sorry, my mistake. Thank you)
чт, 23 черв. 2022 р. о 09:55 NataliaHlazova @.***> пише:

Should it really be "Узгоджено"? In the user story
https://github.com/ita-social-projects/GreenCity/issues/3477 the status
is stated as "Узгодження".
[image: image]
https://user-images.githubusercontent.com/87066582/175234779-73357bd5-dbe6-455c-9564-8134d9b5d538.png
—
Reply to this email directly, view it on GitHub
https://github.com/ita-social-projects/GreenCity/issues/4114#issuecomment-1164024444,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZEQJ64Q3XBV6FYZTENPW2LVQQC5JANCNFSM5VRJ7LYA
.
You are receiving this because you commented.Message ID:
@.***>


[stable/stolon] external secret, auto cluster spec update, pdb

Signed-off-by: Taehyun Kim<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR add

Support for existing secret usage in stolon chart
Auto cluster spec update
Pod Disruption budget

Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @davidkarlsen

/assing @unguiculus

/assign @unguiculus

cc @rtluckie @lwolf

/ok-to-test

also added auto cluster update using helm hook

Also added failkeeper hook before keeper pod termination option.


Failing CI: Unittests with sanitizers on Ubuntu 18.04 with gcc

Please see https://github.com/tesseract-ocr/tesseract/runs/2283272332?check_suite_focus=true
PASSES - ubuntu-20.04-clang-unittest - https://github.com/tesseract-ocr/tesseract/runs/2283272355?check_suite_focus=true#step:12:6
FAILS - ubuntu-18.04-gcc-unittest - https://github.com/tesseract-ocr/tesseract/runs/2283272332?check_suite_focus=true#step:12:6

FAIL: baseapi_test
==================

Running main() from googletest/googletest/src/gtest_main.cc
[==========] Running 10 tests from 2 test suites.
[----------] Global test environment set-up.
[----------] 8 tests from TesseractTest
[ RUN      ] TesseractTest.BasicTesseractTest
=================================================================
==30594==ERROR: AddressSanitizer: new-delete-type-mismatch on 0x6030001a62f0 in thread T0:

FAIL: layout_test
=================

Running main() from googletest/googletest/src/gtest_main.cc
[==========] Running 3 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 3 tests from LayoutTest
[ RUN      ] LayoutTest.ArraySizeTest
[       OK ] LayoutTest.ArraySizeTest (1 ms)
[ RUN      ] LayoutTest.UNLV8087_054
=================================================================
==30898==ERROR: AddressSanitizer: new-delete-type-mismatch on 0x60300060b0b0 in thread T0:
  object passed to delete has wrong type:

FAIL: pagesegmode_test
======================

Running main() from googletest/googletest/src/gtest_main.cc
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from PageSegModeTest
[ RUN      ] PageSegModeTest.WordTest
=================================================================
==1038==ERROR: AddressSanitizer: new-delete-type-mismatch on 0x603000193d80 in thread T0:
  object passed to delete has wrong type:

FAIL: resultiterator_test
=========================

Running main() from googletest/googletest/src/gtest_main.cc
[==========] Running 9 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 9 tests from ResultIteratorTest

==1131==ERROR: AddressSanitizer: new-delete-type-mismatch on 0x6030001ba7e0 in thread T0:
  object passed to delete has wrong type:

SUMMARY: AddressSanitizer: new-delete-type-mismatch (/usr/lib/x86_64-linux-gnu/libasan.so.4+0xe19c8) in operator delete(void*, unsigned long)
==1131==HINT: if you don't care about these errors you may set ASAN_OPTIONS=new_delete_type_mismatch=0
==1131==ABORTING
FAIL resultiterator_test (exit status: 1)

FAIL: textlineprojection_test
=============================

Running main() from googletest/googletest/src/gtest_main.cc
[==========] Running 2 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 2 tests from TextlineProjectionTest

==1218==ERROR: AddressSanitizer: new-delete-type-mismatch on 0x6030002fc770 in thread T0:
  object passed to delete has wrong type:
  size of the allocated type:   32 bytes;
  size of the deallocated type: 16 bytes.

SUMMARY: AddressSanitizer: new-delete-type-mismatch (/usr/lib/x86_64-linux-gnu/libasan.so.4+0xe19c8) in operator delete(void*, unsigned long)
==1218==HINT: if you don't care about these errors you may set ASAN_OPTIONS=new_delete_type_mismatch=0
==1218==ABORTING
FAIL textlineprojection_test (exit status: 1)

g++ (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
Copyright (C) 2017 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Last version that passed was 9 days ago.
https://github.com/tesseract-ocr/tesseract/actions/runs/696227374

Commit 10255d013a012603d2e865d5da3943538d7be84c should fix that.
The failures were caused by a new / delete mismatch: new as for TEMP_PROTO_STRUCT, but delete got a pointer to list_rec from oldlist.h (which should be eliminated).

Thanks @stweil
https://github.com/tesseract-ocr/tesseract/actions/runs/727822691