Decorated methods disappear from class definition in 1.16.0 experimentalDecorators

Decorated methods are no longer defined in output lua when using experimentalDecorators in 1.16.0. Probably due to the refactor of how decorators work.
Exmple code:
function decorator<Class>(
  target: Class,
  propertyKey: keyof Class,
): void {}

class Foo {
    @decorator
    public method2() { return 4; }
}

export const result = new Foo().method2();

Leads to output:
local Foo = __TS__Class()
Foo.name = "Foo"
function Foo.prototype.____constructor(self)
end
-- Expecting method 2 implementation here??
__TS__DecorateLegacy({decorator}, Foo.prototype, "method2", true)
____exports.result = __TS__New(Foo):method2()

Environment

  "devDependencies": {
    "lua-types": "^2.13.1",
    "typescript": "^5.1.3",
    "typescript-to-lua": "^1.16.2"
  }

TS still compile error.

while output seems right.


Problem with Beast and Samsung Internet (websockets, unencrypted)

I'm using Beast (from Boost 1.73; Beast 292), in synchronous mode, as a server for websockets (without encryption, just ws://), on Windows. The client application (written in Javascript) can communicate just fine from everything we've tried (Safari, Chrome, Firefox, Edge, etc.), with the exception of "Samsung Internet", apparently the default browser on some Android phones, where Beast complains about bad_method before aborting the connection. While not a major deal (we can tell our users to just use Chrome or Firefox instead), I was still wondering if this is a known problem in either Samsung Internet or Beast, and if there is something we can do to make it work.
Our code is an only mildly adjusted version of the Beast synchronous websocket server example.

Are you able to post an example of a request header that Beast rejects?
Beast aims to be strictly standards compliant. If the client is sending a malformed request, we may be less forgiving than other libraries.
However, it is possible that we have an error. I'd like to check.

I can try. What's the best way to obtain such a header?

log_stream << received_request;

log_stream << received_request;

What is received_request in this context? I'm guessing it must be something related to the acceptor?

log_stream << received_request;

What is received_request in this context? I'm guessing it must be something related to the acceptor?

It's the request http request you received:
https://www.boost.org/doc/libs/1_78_0/libs/beast/doc/html/beast/ref/boost__beast__http__request.html
https://www.boost.org/doc/libs/1_78_0/libs/beast/doc/html/beast/ref/boost__beast__websocket__stream/async_accept/overload3.html

is this resolved?


refactor!(updater): migrate run updater using powershell to ShellExecute

Fix flashing powershell window on update

Note: I have not tested this yet

I doubt I did it right on the windows-sys version, felt like I was just cheating on the compiler 🤣


UI - Not able to launch "Edit LoadBalancer" page

Server version - v0.47.0-rc1
Steps to reproduce the prolem:
Create a LB service.
Activate service.
Try to Edit this service.
UI is not able to to launch "Edit LoadBalancer" page.

@deniseschannon can you check this on master? I followed your steps and was able to edit the service. It is worth noting it took some time for the edit service modal come up, it was loading for quite a while, but  this is a side effect of the sheer amount of services we have. The UI loads those services so we can populate the drop down. Maybe this was the issue you were experiencing?

I did see this was working on master yesterday, but the listener ports weren't displaying. I can check all the detailed fields to see what is missing.

@deniseschannon sorry i meant to mention @sangeethah. i had like 10 issues open this morning and was responding to a few

Listener ports is still broken because it used part of the LB API that was removed.  Working on it.

Able to edit load balancers with latest UI (evening 11/24)


Issue3 travis ci

The Travis CI and Coverage reports are here (you can also see them in the bottom of this page):
https://travis-ci.org/github/c-martinez/FAIRDataPoint
https://coveralls.io/github/c-martinez/FAIRDataPoint
@c-martinez  Could you add Codacy to this repo (also add the badge)?  In this way we could track the code quality. The default test_lint.py from the python project template was deleted because it's too strict to make the Travis building always fail.

I've also added codacy -- see #8


Generation with Prefix-cache are slower than the ones without it ?

I'm running the tutorial vllm/offline_inference_with_prefix.py and measuring the generation times, again below is the same code with generation times
`
import argparse
from typing import List, Tuple
from transformers import AutoModelForCausalLM, AutoTokenizer
from vllm import EngineArgs, LLMEngine, RequestOutput, SamplingParams
import time
from vllm import LLM, SamplingParams
prefix = (
"You are an expert school principal, skilled in effectively managing "
"faculty and staff. Draft 10-15 questions for a potential first grade "
"Head Teacher for my K-12, all-girls', independent school that emphasizes "
"community, joyful discovery, and life-long learning. The candidate is "
"coming in for a first-round panel interview for a 8th grade Math "
"teaching role. They have 5 years of previous teaching experience "
"as an assistant teacher at a co-ed, public school with experience "
"in middle school math teaching. Based on these information, fulfill "
"the following paragraph: ")
Sample prompts.
prompts = [
"Hello, my name is",
"The president of the United States is",
"The capital of France is",
"The future of AI is",
]
Create a sampling params object.
sampling_params = SamplingParams(temperature=0.0)
if name == 'main':
# Create an LLM.
llm = LLM(model="facebook/opt-125m")
generating_prompts = [prefix + prompt for prompt in prompts]

# Generate texts from the prompts. The output is a list of RequestOutput objects
# that contain the prompt, generated text, and other information.
st = time.perf_counter()
outputs = llm.generate(generating_prompts, sampling_params)
end = time.perf_counter()
print(f"without caching time:{end-st}")

# Print the outputs.
for output in outputs:
    prompt = output.prompt
    generated_text = output.outputs[0].text
    print(f"Prompt: {prompt!r}, Generated text: {generated_text!r}")

print("-" * 80)

# -1 since the last token can change when concatenating prompts.
prefix_pos = len(llm.llm_engine.tokenizer.encode(prefix)) - 1

# The llm.generate call will batch all prompts and send the batch at once if resources allow.
# The prefix will only be cached after the first batch is processed, so we need to call generate once
# to calculate the prefix and cache it.
outputs = llm.generate(generating_prompts[0],
                    sampling_params,
                    prefix_pos=[prefix_pos])

# Subsequent batches can leverage the cached prefix
st = time.perf_counter()
outputs = llm.generate(generating_prompts,
                    sampling_params,
                    prefix_pos=[prefix_pos] * len(generating_prompts))
end = time.perf_counter()
print(f"with caching time:{end-st}")

# Print the outputs. You should see the same outputs as before
for output in outputs:
    prompt = output.prompt
    generated_text = output.outputs[0].text
    print(f"Prompt: {prompt!r}, Generated text: {generated_text!r}")

`
Output:
with caching time:1.9611055543646216
without caching time:0.07439832389354706
VLLM: vllm==0.3.3

The automatic prefix caching commit seems merged very recent and labeled as 0.3.4 release. So I assume some changes are not available on 0.3.3

Although I have not tested original prefix cache, I am seeing something strange that may point to the benchmark data being skewed.
One would assume like @vin136 that the second request on-ward would be fastest (due to caching). In my simple tests, the second request is always the slowest by a significant margin. At least for the master vllm checkout I did today with auto-prefix caching enabled.
So my suggestion is to collect data from request #3 forward and not #2 and see if new implementation is slower than old.

Yup - we are working this week on optimizing the performance. Original PR focused on correctness

@robertgshaw2-neuralmagic  Do you have any idea why with prefix caching on, the second request is actually slower by a significant margin? I am repeating there request 3 times serially, with several second delay between each request. The order of speed from fastest to slowest is 3, 1, 2 where 2 is slowest by a huge margin. This seems counter-intuitive to most new users expecting how normal caching works.

@robertgshaw2-neuralmagic Do you have any idea why with prefix caching on, the second request is actually slower by a significant margin? I am repeating the request 3 times serially, with several second delay between each request. The order of speed from fastest to slowest is 3, 1, 2 where 2 is slowest by a huge margin. This seems counter-intuitive to most new users expecting how normal caching works.

It's hard to tell without a bit more info about the request pattern
In general, you should not expect to see any speedup at current. Its still experimental and we are working on performance

@robertgshaw2-neuralmagic  thanks, we're really looking forward for the optimization!
Also, could you clarify on the behavior of this feature:

in the same batch, first N tokens of the requests will be shared.
in the second batch, first N tokens of the rests will be shared with requests in the first batch.
Which of the above is the expected behavior? Mainly the difference is that, do we need to let the vllm engine completes prompt processing phase once for 1 request, and then sending the rest common-prefx requests?

Just to confirm, not only it isn't optimized, but also not enabled by default right?
If I run:
model_id = 'meta-llama/Llama-2-7b-hf'
llm = vllm.LLM(model=model_id)
cache_config = llm.llm_engine.cache_config
print(cache_config.__dict__.keys())

I get
dict_keys(['block_size', 'gpu_memory_utilization', 'swap_space_bytes', 'cache_dtype', 'sliding_window', 'num_gpu_blocks', 'num_cpu_blocks'])

The enable_prefix_caching argument is not there... and when I try to initialize an LLM object with the parameter, I get:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Input [In [21]] llm2 = vllm.LLM(model=model_id, enable_prefix_caching=True)
...
TypeError: __init__() got an unexpected keyword argument 'enable_prefix_caching'

With the latest version(0.4.0), it seems we cannot enable prefix caching with Mistral-type models(sliding window attention).

@vin136 i think i saw this mentioned in a different issue. but for now you can go into your model config and manually change the sliding window size to null

Do you have any idea why with prefix caching on, the second request is actually slower by a significant margin? I am repeating the request 3 times serially, with several second delay between each request. The order of speed from fastest to slowest is 3, 1, 2 where 2 is slowest by a huge margin. This seems counter-intuitive to most new users expecting how normal caching works.

In my test, even though I used the same request three times, the response time was about the same, what does your test code look like?@Qubitium

I also met the same condition, do you have any idea 🤔?


Domain name doubled in Containers Hostname

Ive created a mail docker container with these domain name settings (set in the .env) file:

HOSTNAME=mail
DOMAINNAME=mytestmail.com
OVERRIDE_HOSTNAME=mail.mytestmail.com

When i login with bash into the new created container and execute the hostname command, i receive:

mail.mytestmail.commytestmail.com

As my hostname. So the Domain Name is doubled.
Context
Checking the mail server security with mxtoolbox.com responds that my SMTP Banner differs from the expected hostname. Some mail servers would assume this as spam.
Expected Behavior
The hostname of the container should be mail.mytestmail.com.
Actual Behavior
The domain name is doubled.
Your Environment

Amount of RAM available: 2GB
Mailserver version used: latest (May 14, 2019 10:17 PM)
Docker version used: 18.09.7, build 2d0083d

@n73671 Always leave a comment when closing an issue. Even it was e.g. your own mis-configuration - it will help others.


Implement a way to search for an image by manifest, config or layer digest

Usage:
  zot images [config-name] [flags]

Flags:
  -d, --digest string   List images containing a specific manifest, config, or layer digest
[...]

Resolves #175

@andaaron, couple of comments:

the original images-by-name search now returns config, layers, etc
I would keep the original output but add a -vvv type option for more details

what is the expected argument format for -d option? the full digest like "sha256:abcd...."? We should also allow partial matches

@andaaron, couple of comments:

the original images-by-name search now returns config, layers, etc
I would keep the original output but add a -vvv type option for more details
what is the expected argument format for -d option? the full digest like "sha256:abcd...."? We should also allow partial matches

Sounds good
Currently it's any substring that matches the digest, including the 'sha256:' part.


Fixing typos in README.md

Cool project. I'll take a closer look at the code later on and maybe contribute something if I can. For now, I fixed up a couple typos in the README.md file so it's a bit easier to read.

Wow!  Could I make any more mistakes in such a small amount of time? :D
Thanks for that!! I appreciate it!
Just one thing, you made a mistake too!

still provids

I'm glad to see I'm not the only one here!  I guess we have to leave some room for improvement. :)
See pull request #2


Linux permisson issue

Hi,
I am using linux machine, I have setup the files as the instruction in the readme.
Now when I run docker-compose run --rm artisan make:controller TestController command then I am not able to edit that file from the local file system.
I have tried the following but it didn't fix the issue:

Can you advice me on this?

Check out some of what's been talked about in #49, but basically you'll need to ensure that the user created during the Docker image build has the same uid and gid as the user that owns the files in your local environment. You can try chown'ing the src folder after spinning up the containers and seeing if that helps as well!


uu_test binary alias [ causes issues for Zsh completions

Issue
Using the Zsh completions of the coreutils binary created by running cargo run completion coreutils zsh results in an error:
coreutils _coreutils:1418: invalid subscript
_coreutils:1419: bad pattern: _coreutils__[_commands
_coreutils:23: bad pattern: [
_coreutils:23: bad pattern: [

Description
Creating completions using cargo run completion coreutils zsh will create completions for the uu_test create and thus both test and [ binaries as given here:
https://github.com/uutils/coreutils/blob/97d30bd486c8819d630cc864ee49a7edd16e3db1/build.rs#L60-L63
This results in a completions file containing the following:

lines 689-694([)
_arguments "${_arguments_options[@]}" \
'-h[Print help]' \
'--help[Print help]' \
&& ret=0
;;

lines 1177-1180([)
_arguments "${_arguments_options[@]}" \
&& ret=0
;;

lines 1418-1427(( $+functions[_coreutils__[_commands] )) ||
_coreutils__[_commands() {
    local commands; commands=()
    _describe -t commands 'coreutils [ commands' commands "$@"
}
(( $+functions[_coreutils__help__[_commands] )) ||
_coreutils__help__[_commands() {
    local commands; commands=()
    _describe -t commands 'coreutils help [ commands' commands "$@"
}

Each of the snippets above contain invalid opening square brackets, resulting in the shown errors when using the generated completions in Zsh.
Workaround
Removing line 62 in build.rs fixes the issue:
https://github.com/uutils/coreutils/blob/97d30bd486c8819d630cc864ee49a7edd16e3db1/build.rs#L62
Background
I installed the multicall binary from source and created the completions as given above, placing them in $FPATH. I am using Zsh 5.9.4 (with Oh My Zsh) on an Arch-based distro.

We can't remove the [ alias there unfortunately, because we still need to support the alias, but we maybe we can skip generating completions for it? Alternatively, the [ should be an optional feature, since it is a built-in in zsh, so people can decide (based on their shell) whether they want it or not.

or propose a fix in zsh :)

I think it's intentional on zsh's side because [ is a built-in? Maybe not though


Use shields.io instead of template

Since the majority of badges used are made with shields.io, it would be intersting to use it there aswell to make sort of unity between other badges

Thanks, sounds like a great idea. I'll have a closer look at the weekend.

I like the idea of using shields but looks like it's not completed yet (what happens with the curl result?). If you activate the actions in your own fork (it should ask you to do this if you visit https://github.com/MikeDevresse/phpunit-coverage-badge/actions) you can test it as the actions of the project run the action itself to generate the badge.
The tests should also be adjusted accordingly.
Also what happens if the remote request fails? I'd like to stay as independent as possible of external services so maybe we can check the curl call for errors and fall back to the built-in badge generation. Maybe even logging some kind of warning for the user of the action.
I think we might also need to adjust our colors to be used with shields as it seems like they do have their own list of predefined colors.
If you'd like to invest time in this feel free to do so but you'll have to decide for yourself if it's worth the effort.

Closing this as it seems to be stale.


Updated Readme, changed PduLength to PDULength

Just a small update to the readme PduLength seems to be PDULength

Yes, you are right. Can you also change PduRequested to PDURequested and force push?

Aah right completely missed the PDURequested, added it

Would be cool if you can sqash the two commits into one and give it a proper commit message.

Hope this is acceptable ;) thanks for your awesome work so far on this wrapper!

Thank you very much for contributing! I'm very pleased to hear that and I hope the few guys using it are creating awesome stuff with it ;)


Devenv search is slow (comparing to nix search)

Describe the bug
Devenv search is slow even when searching multiple times for the same package. This doesn't happen when using raw nix search.
To reproduce

nix search - first time slow, second time fast

time nix search nixpkgs gdal
21.13s user 6.51s system 58% cpu 46.881 total

time nix search nixpkgs gdal
0.94s user 0.15s system 96% cpu 1.129 total

devenv search - slow all the time

time devenv search gdal
16.62s user 1.66s system 97% cpu 18.691 total

time devenv search gdal
16.27s user 1.65s system 98% cpu 18.284 total

Make sure to include full logs and what you expected to happen.
Version
Paste the output of $ devenv version here.
devenv version
devenv 1.0.1 (x86_64-linux)

Yeah, we could cache the results if we know nixpgks input is the same.

Raw nix search is fast even if I search for other packages.
For example:
time nix search nixpkgs git
0.92s user 0.14s system 98% cpu 1.067 total

time nix search nixpkgs geos
0.92s user 0.12s system 99% cpu 1.044 total

time nix search nixpkgs mc
0.92s user 0.13s system 99% cpu 1.049 total

It looks like nix search is caching evaluation, while devenv is not.

https://blog.replit.com/nixpkgs-search


Changed to py.test instead of nose

#50

@mpatek #reviewmerge

@mpatek that did the trick. #reviewmerge again please.

Thanks @noppanit


Dim screen after timeout

Description of the new feature / enhancement
Dim screen after a timeout. Has been standard on gnome desktops. Switch off is kind of harsh. Why not dim after a timeout
Scenario when this would be used?
After a set time the screen dims to a customized level. Still readable but less glare. Specifically for OLED television screens this might be a live saver
Supporting information
No response

When your monitor supports brightness control you can set this option in the power plan settings under "display"

im afraid that is not true. i can change the brightness but not set an automatic change. unless i didn't understand your comment

https://www.codeproject.com/Articles/47355/Setting-Screen-Brightness-in-C

Interesting idea, I think this is something our team wouldn't do.  Hard for us to verify all the hardware quirks and FW for said hardware.  We learned this hard on video conference mute.
Part of me also wonders if this was a thing, wouldn't the device makers have this just as an idle setting.  I don't know as I have yet to upgrade to a display with OLED, just theorizing.
Windows Feedback Hub for a setting is where i'd recommend this.  I'm also betting a different OSS or app does just this.  Just too hard for our team to support this style of request.


how to have extra parameters for FormControl

Hi
i am trying to set extra parameters that  visible in view  for example here
  final Stop = new FormGroup({
    "location":  FormControl<String>(),
    "status": FormControl(),
    "datetime": FormControl(),
  });

location  controller have  3 more parameters like  coordinates, address,state ... etc    i need only values of these   and in view  will be  only a label contain address of all

you can grouping the form control
final stop=FormGroup({ 'location':FormGroup({ location": FormControl<String>(), "status": FormControl(), "datetime": FormControl(), }),});


[mle] clear messages for sleepy child when role is not router/leader

The commits fix an issue that indirect messages for sleepy child are not correctly cleared in some case.
In my posix-sim tests, this issue together with the issue fixed in PR #4232 can cause sleepy nodes unable to reattach for a long time, or even forever, as described in following steps:

Router R has a sleepy child S which has some indirect messages pending.
R changed role to Detached for any reason
R changed role to Child
R becomes router again with another Router ID
R removes the child S by calling RemoveNeighbor

because now R has mRole == OT_DEVICE_ROLE_CHILD, R does not call ClearAllMessagesForSleepyChild, failed to clear indirect messages

R got another child, which reused S's slot in ChildTable (thus having same ChildIndex)
S tries to reattach to R, sends Child ID Request, and get a different ChildIndex
S tries to send indirect message (Child ID Response) to S, calling IndirectSender::AddMessageForSleepyChild -> SourceMatchController::IncrementMessageCount -> AddEntry
In SourceMatchController::AddEntry, AddAddress returns OT_ERROR_DUPLICATED or OT_ERROR_NO_BUFS, because S's ext address already exists (was not correctly remove due to step 5). AddEntry calls Enable(false) to turn off src match in the code bellow.

VerifyOrExit(AddAddress(aChild) == OT_ERROR_NONE, Enable(false));

Since src match is turned off, issue #4232 causes all frame pending not to be set for data requests, causing sleepy child to go to sleep when ACK of Child ID Request is received, thus unable to receive Child ID Response.

Incorrect sleepy message removing also corrupts the data consistency of
SourceMatchController, causing its behavior unpredictable.
I am still striving to have a better understanding of this issue and have many questions about removing neighbor.

Beside ClearAllMessagesForSleepyChild, do we need to call other functions (listed bellow) too when role is not router/leader?

Signal(OT_NEIGHBOR_TABLE_EVENT_CHILD_REMOVED, aNeighbor);
Get<NetworkData::Leader>().SendServerDataNotification(aNeighbor.GetRloc16());
Get<AddressResolver>().Remove(aNeighbor.GetRloc16());
RemoveStoredChild(aNeighbor.GetRloc16());

Should children be removed when router is detached?

Thoughts?

@simonlingoogle, thanks for identifying this issue and working to resolve it!
I agree that we need to move most of the other code out of the switch statement (see below).

Beside ClearAllMessagesForSleepyChild, do we need to call other functions (listed bellow) too when role is not router/leader?

Signal(OT_NEIGHBOR_TABLE_EVENT_CHILD_REMOVED, aNeighbor);

Yes.

Get<NetworkData::Leader>().SendServerDataNotification(aNeighbor.GetRloc16());

This one is a bit tricky. If R is still a part of the same partition, then yes. If R is no longer a part of the same partition, then no.

Get<AddressResolver>().Remove(aNeighbor.GetRloc16());

Yes.

RemoveStoredChild(aNeighbor.GetRloc16());

Yes.

Should children be removed when router is detached?

No. Thread 1.1.1 Section 5.16.6 indicates that a router should try to keep its children attached, even when migrating to a different partition.

Get<NetworkData::Leader>().SendServerDataNotification(aNeighbor.GetRloc16());

This one is a bit tricky. If R is still a part of the same partition, then yes. If R is no longer a part of the same partition, then no.

@jwhui
I didn't check if the child being removed belongs to the current partition (neither was before), because of following reasons:

Implementing it would requires maintaining a partition id for each Child entry to record which partition the child belongs to, leading to more RAM footprints
SendServerDataNotification will try to lookup the rloc16 in the Network Data, and only send /a/sd  when the rloc16 is found, which I think is highly unlikely if the child was not of the current partition.

@simonlingoogle resolve the conflicts?


Cannot configure the extension

Issue Type: Bug
Open the setting of the extension,then throw an error like that:command 'pr.configurePRViewlet' not found
Extension version: 0.10.0
VS Code version: Code 1.38.0 (3db7e09f3b61f915d03bbfa58e258d6eee843f35, 2019-09-03T21:49:13.459Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz (8 x 1800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.85GB (1.15GB free)

Process Argv

Screen Reader
no

VM
0%

@FrankHuy I haven't been able to reproduce this yet - how are you opening the settings of the extension? Is it from clicking a button the the "GitHub Pull Requests" view, from running a command, or something else?

Also seeing this now. When selecting "GitHub Pull Requests: Configure Remotes" an error alert pops up with "command 'pr.configureRemotes' not found"
Looking at the logs there is the followinng message
[Info] /Users/nschonni/.ssh/config: ENOENT: no such file or directory, open '/Users/nschonni/.ssh/config'
[Info] Looking for git repository

Just started happening, so it would be something in the 0.11 release

I'm having the same issue.

It started working for me again with the 0.11.1 release

yeah, I was also able to repro yesterday on 0.11.0, should have been fixed with https://github.com/microsoft/vscode-pull-request-github/commit/999b7eeb2e6362e36e8294a27d80fdced9ddd3fe
However, the original bug report was for 0.10.0, which didn't have the issue that 0.11.0 did. I'm going to tentatively close this since I can no longer reproduce this in 0.11.1, but please let me know if you encounter this again. Thanks everyone!

facing the same issue on vs code 1.39.2 version

Same issue on Arch, can't open any settings or configs. Just gives me the message as OP described.
Please reopen this

For Arch users, or others using the OSS build of VSCode, @RMacfarlane recently mentioned that you will need to run code with a flag:
code --enable-proposed-api GitHub.vscode-pull-request-github

Alternatively, you can edit your product.json file (code installs this to /usr/lib/code/product.json).
"extensionAllowedProposedApi": [
    "GitHub.vscode-pull-request-github"
]

This issue is documented in the wiki and in other extensions' documentation. I have updated the Arch Wiki with this information and consolidated related errors under a single troubleshooting heading to help users understand the effect of the OSS build missing Microsoft's product.json.

What got this working for me was adding an entry to ~/.vscode/argv.json:
{
    // Allow Github Pull Request to work
    "enable-proposed-api": [
        "GitHub.vscode-pull-request-github"
    ]
}


Plans for future updates

Hi all!
I thought I might spend 10 minutes to write down some ideas on how to evolve the library now.
Deprecation of old methods
We have quite a few methods that are a leftover of the first version of the client, and that are now nothing more than aliases for the base methods.
I would like to reduce the API surface and deprecate those methods for now, and remove them in a later version
V3 support
While I know there is a v3 python client that works decently, it has a couple of issues:

It doesn't have the fault tolerance that python-etcd has
Its API is very different from the one we have here. While I agree that not everything should be 1:1 with the current API, I personally find it very irritating to have to rewrite all of my code to transition to use etcd3.
python-etcd is a standard and it's already packaged in a few linux distros, it would be nice to have support for etcd v 3 for python in the distributions easily.
I am unsure if this should be a small compatibility layer on top of the existing library or a new implementation. I'm honestly thinking of the latter, as I already wrote a semi-complete client implementation some months ago and never found the time to finish it.

What do people think? Other things that would be useful?

It may be as simple as creating a v2 shim—always printing a deprecation message when it's is used—which could be rather trivial, something like:
def _all_get(key, **kwargs):
    return namedtuple('Val', 'value')(self.get(key)) if self.protocol == 3 else self.get(key)

Do we have etcd API v3 support? Kinda confused looking at the comments


CharacterBody velocity slows down if FPS drops below the tick rate

Thank you for this smoothing solution. This smoothing node really helps with removing jitter on my high refresh rate monitor, although I have noticed some extreme player movement slowdown while messing around with low frame rates.
I am able to replicate this using the test project https://github.com/anatolbogun/godot-smoother-node-test-scene
Steps:

In the script level2d.gd for the Level2D node, add the line Engine.max_fps = 10 to the _ready function.

Change the project tick rate to the default value of 60 in Project Settings -> Physics -> Common -> Physics Ticks per Second.

Play the scene. The character now moves very slowly.

Stop playing the scene.

Delete the Smoother node.

Play the scene again.

The character moves at normal speed again.

Is there a workaround for this?

Also happened to me once, although removing and re-adding the Smoother node fixed it, oddly.

There's a connection I found but I don't understand the smoother.gd script enough to know what to change.
My normal conditions:
165 monitor max frame rate
60 physics tick per second
If I lower the the max_fps the physics feel off as we already know
If I follow the formula:
(max_fps / monitor max frame rate) * (physics tick per second) = new physics tick
then at 22 max_fps (because the math works nicely) we get:
(22 / 165) * 60 = 8
Setting the physics tick to 8 with 22 max_fps feels like the physics does under normal conditions with an uncapped frame rate but with a much lower frame rate
This is not a solution but maybe a clue towards what needs fixing in _physics_process

Actually can't this simply be fixed with if not (Engine.max_fps != 0 and Engine.max_fps <= Engine.physics_ticks_per_second): on the first lines of _process and _physics_process? Don't really need interpolation if max_fps is less than or equal to physics tick per second

Hi all. Sorry, I missed the original issue from April for some reason, my apologies. Thanks for the detailed steps and suggestion for a solution.
I'm very busy right now but end of this week I should have some time to look into it.

This is a problem, I'm getting velocity speed drops when the max_fps is lowered, and it's fixed if Smoother is disabled.

Actually can't this simply be fixed with if Engine.max_fps > Engine.physics_ticks_per_second or (Engine.max_fps == 0 and DisplayServer.screen_get_refresh_rate() > Engine.physics_ticks_per_second): on the first lines of _process and _physics_process? Don't really need interpolation if max_fps is less than or equal to physics tick per second

Unfortunately if the framerate drops and returns, the position is saved from when it was at a higher framerate. Maybe it can be set at some other part of the process?

Here's a branch/PR that will hopefully address this issue. For now, it's in the sample project repository at https://github.com/anatolbogun/godot-smoother-node-test-scene/pull/1 . A big thanks to @elvisish for the suggestion, the change is pretty much exactly that code.
I also added a keyboard toggle to the sample project. With CapsLock you can toggle between Engine.max_fps 0 (unlimited) and 10.
It would be great if some people could test this, especially in your own project if you experienced this particular issue.
I'm also curious how you get to low frame rates in your projects. Is this because too much is happening and the FPS can't keep up with the target FPS? I only have this super basic sample project and a few other simple tests. I suppose I could add some code loops that can push the performance over the edge, but so far I haven't experimented with this yet.
@Midrule , regarding your comment:

temp solutions almost working, but still struggling if the fps is going in and out of 60.

Do you mean a potential jump/inconsistency when the smoothing is disabled/re-enabled by the fix? I think I can see a jump when I toggle the max_fps between 0 (physics interpolation happens), 10 (interpolation is skipped and saved properties are cleared) and back to 0. I need to think if that can be avoided, but it's probably tricky because the interpolation requires 2 frames to kick in, so the interpolation can only happen when at least 2 _physics_process property values were saved, like a position A and B.
Maybe this jump can be avoided or reduced if the module continuously keeps track of the property values, even if it doesn't apply the interpolation. I can try to move the new code block elsewhere, maybe it will help. But just to confirm if that's what you're referring to.


With added child view controller, animation can be broken

After adding child view controller navigation bar animation can be broken when switching from transparent bar to untransparent, and looks like this:

But expected this:

The easiest way to replicate it add following code to your example:
   override func viewWillAppear(animated: Bool) {
      super.viewWillAppear(animated)
      navigationController?.setNavigationBarHidden(currentNavigationBarData.prefersHidden, animated: animated)

      dispatch_async(dispatch_get_main_queue()) {
         let child = UIViewController()
         child.view.hidden = true
         child.view.autoresizingMask = [.FlexibleWidth, .FlexibleHeight]
         child.view.frame = self.view.bounds
         self.view.addSubview(child.view)
         self.addChildViewController(child)
      }
   }

Hi, Thanks for your feedback.
You can try to call addChildViewController: before calling addSubview:.

Cool! Thank you! It works


arm64 support for ifuzz

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

Please send smaller changes as separate PR, we can review/merge them faster.

Please take another look. I added pseudo-instruction support and reworked the architecture so that we don't need arm64/internal anymore.

Looks like I was misinterpreting the semantics of fallthrough in Go, these reports should be fixed now.


Add injection iceberg usecase

This first PR create the skeleton and add a first injection use case.
This injection use case:

provides a script to fetch GDELT data as CSV file, ready to be parsed and inserted in the Iceberg table
CreateTable creates a Iceberg table to store GDELT events
DataInjection parses the GDELT CSV file and insert data in the Iceberg table
Analytics provides the first simple queries

NB: when merged, I will create a blog post based on this PR.

@ajantha-bhat can you please take a look ?

I just reviewed high level. I will install and try it out later on.


Fix performance regression of VQE by #7217

Summary
#7217 calls transpilation two times: 1) transpile of parameterized circuits and 2) transpile of bound circuits.
Unless users define their pass for the second pass, we don't need to call the transpile again.
Fixes #7337
Details and comments
from timeit import timeit

from qiskit import Aer
from qiskit.algorithms import VQE
from qiskit.algorithms.optimizers import COBYLA
from qiskit.circuit.library import RealAmplitudes
from qiskit.opflow import PauliSumOp
from qiskit.utils import QuantumInstance

quantum_instance = QuantumInstance(backend=Aer.get_backend('aer_simulator'), shots=1)
vqe = VQE(ansatz=RealAmplitudes(), optimizer=COBYLA(maxiter=1), quantum_instance=quantum_instance)
op = PauliSumOp.from_list([("ZZ"*5, 1), ("IX"*5, 2)])
print(f'{timeit(lambda: vqe.compute_minimum_eigenvalue(op), number=10)} sec')

main c95ec3712
4.612741603000359 sec

this PR
1.943418126999859 sec

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.008%) to 82.962%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/uc.py
2
88.65%

qiskit/pulse/library/waveform.py
3
89.36%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
50772

Relevant Lines:
61199

💛  - Coveralls

I think we need the unbound pass in L299 even if it's None so that we apply the default transpile pass.
The bound pass is newly introduced in #7217 and we can omit it if no bound pass is given.
There is another option to fix this performance regression is to modify QuantumInstance.transpile so that it does not call the default pass if bound_pass_manager is None. Which do you prefer?

I think this solution here makes most sense 👍🏻 thanks for the fix!


Updated documentation for _ in Word style names.

I spent quite some time trying to debug this. Documentation mentions spaces, but even _ are replaced.

Adding in develop branch


Added security rule to controller test

Added the security rule to controller test and reduced diff to solution branch 24 (spring4).

@nenaraab can you merge this?

copied AdvertisementControllerTest


Support arbitrary types?

Or at least avoid explicitly forbidding it
Will be useful for use in Flux, where arbitrary nested structs can be used as parameters

Requires #104

At the moment the policy is: numbers and arrays are thoroughly tested, other stuff may work but you have to sweat for it


Feat/controller implementation

Implementation of the controller

The code is clean, consider merging this part

We still need to separate the "commands" into a dedicated repository. This is in progress, so until it's committed I would rather not merge this branch. Sorry for the wait


gluon-nlp should provide citation information

@mli suggested to provide reference information on github readme for software.

I'd suggest to also add it to the homepage then

need to figure out the bib format for it.

let's write a survey paper and share that instead since this library to be open to all community.


Save user input at FormBuilderTypeAhead

Is there any way to save user input in a FormBuilderTypeAhead without selecting a suggestion? I am not sure how to achieve it.

This still a nice feature to have


Add --dump-core option

This might not be the best way to go about it. Just an idea

@jystic @charleso

this is cool, will take a proper look tomorrow

I really nothing about the flags in question.
Just curious, is it already possible to pass ghc flags though? I guess I'm asking - does this need to be in mafia or is it just a call-site decision?

One can use arguments which are passed to cabal after the --, and --ghc-options=* will shuffle them there.
So I think this is for convenience as there's 4 flags required here.

Yeah it's definitely convenience, I for one can never remember what the flags are called.
I think maybe -dsuppress-all is a little too aggressive, I quite like having the type signatures in the output or I find it harder to decipher what's going on. Would you be ok with using -dsuppress-coercions -dsuppress-idinfo -dsuppress-module-prefixes in place of -dsuppress-all? It's basically the same but with type signatures.
What is the difference between between -ddump-prep and -ddump-simpl? For some reason -ddump-prep always gives me a file with two dumps right after another which appear to be identical?
I think I would prefer to use -ddump-simpl or maybe we should have two options in mafia --dump-simpl and --dump-prep (instead of --dump-core), what do you think? /not sure

maybe we should have two options in mafia --dump-simpl and --dump-prep (instead of --dump-core), what do you think? /not sure

Having said this, I do like the simplicity of having only a single --dump-core option. Part of the benefit of this feature is that it makes it trivial to get started inspecting core, even if you've never done it before.

I'm 👍 for -ddump-core
can also use something from this https://github.com/shachaf/ghc-core/blob/master/ghc-core.hs to make it more readable, if necessary

It's similar to the profiling flag, in that there's scope in mafia for making things which ghc and cabal make hard easy.

Yeah, it really is just a shorthand. Maybe we want a more principled way of doing shortcuts that expand to multiple options?

What is the difference between between -ddump-prep and -ddump-simpl? For some reason -ddump-prep always gives me a file with two dumps right after another which appear to be identical?

It's just cargo-cult muscle memory for me. I think prep is slightly after simpl in the pipeline, but I don't think it makes a real difference. Happy to go with simpl

can also use something from this https://github.com/shachaf/ghc-core/blob/master/ghc-core.hs to make it more readable, if necessary

I just had a look, maybe those regexes are superseded by the -dsuppress options?

:wolf: so good

I just had a look, maybe those regexes are superseded by the -dsuppress options?

@amosr That might be the case. If I remember correctly ghc-core has other features beyond just that. Its a pity its been un-maintained for so long.


Food with both known and unknown diet-incompatible ingredients are displayed as unknown status

Describe the bug
The text about ambiguous ingredients in not in-sync with the diet status.
To Reproduce

Go to a diet-incompatible food item, with both explicit and ambiguous ingredients about the diet.
Click on the diet icon
See error

Expected behavior
The text should say that there are ambiguous ingredients, without saying that the diet-compatibility is unknown.
Screenshots

Smartphone

Version 3.6.8(433)

Additional context
Example food : https://world-fr.openfoodfacts.org/produit/3242272920052/sandwich-jambon-oeuf-sauce-burger-pain-moelleux-viennois-le-mega-sodebo
#3023
#3387

"Some ingredients in this products are ambiguous. This may impact our ability to analyze it" ?


React updates

See individual commits for details. Highlights:

Fixes bad state initializers in tests (closes #3855)
Added: React.ShallowRenderer for shallow rendering
Remove DOM API interfaces (SVGLength, SVGTransformList, etc) from React.SVGAttributes and use number or string where applicable because react stringifies all values before calling node.setAttribute anyway.

cc @pspeter3 @jbrantly

Looks good to me! Thanks for following up!

Yeah, I ran into this problem this weekend. Replacing SVGLength with number | string is definitely needed for these definitions to work.
+1

+1 from me

cc @vvakame @johnnyreilly

Well done!


Fix validation of SoftKeyStore attributes

 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.


detected as botting tool on Easy Anti-Cheat

Why is this treated as a botting tool by Easy Anti-Cheat? Can I do something to avoid it?

I think that might be because it's written in autohotkey which is used a lot to make these botting tools/cheats, so the anti-cheat detected the ahk binary that's running micmute and got triggered by it.
TBH I don't know how you could avoid it, but I'm curious which game were you playing when it was detected? because I've been using this and other AHK stuff with a lot of games that use EAC like fortnite and apex legends and I've never had any issues.
Also what did the anti-cheat say exactly when it was triggered?

That works perfectly! Did you recompile the source code?
Just to know: the game is Albion Online. It works like a charm with the original .exe, but when I enter a new zone, the game kicks me and present an alert from the anti-cheat.

Too early... :/
It disconnect me randomly now, not just when I travel between maps.

I tested the game and it does disconnect me if I'm running this or any other AHK script, and sadly I don't know if I can do anything about it, they seem to have blacklisted anything written in autohotkey example, another example, which is weird because it only happens with this game, I tested 3 other EAC games and they all work fine.

I will revert the commits that I made to use BinMod and ScriptGuard since they made no difference and they broke the github action.
And I will add this issue to the README.

Thanks for the support. I am using Power Toys v36  (wich has global mic mute shortcut written in C# maybe) and Albion Online's Easy Anti-Cheat is not disconnecting me now. But they are showing an persistent floating toast on primary monitor when mic is muted. By the way, I've mentioned this project as an example of how to keep the visual feedback on system tray on a issue there.
Power Toys v36 is not stable yet, so maybe they implement something as sophisticated as your solution.


Cannot pass Inferno 7.2.05 test, get 500 error "export operation failed for reason: Unknown Error"

Hello,
I run the Inferno test for Multi Patient.
The test 7.2.05 is failed

The last API call is get 500 error, more details

Check the log in FHIR proxy, function name "ProxyFunction"

This is unknown error
I check the export storage account and see there is ndjson file in blob container

If you guys has any guidance or troubleshoot direction, please let me know
Thank you

@vohuythao
Have you created a Group resource on the server?
If not please create a group resource and mention 2 patient IDs in the member field.
You need to mention this group resource id in the inferno before executing test 7.
This is my group resource. You can use this just change the Patient ID you want to use for export.
{
"resourceType": "Group",
"id": "abc08f6d-e822-4c80-8237-b52c2e7f515b",
"meta": {
"versionId": "1",
"lastUpdated": "2022-11-23T17:00:39.138+00:00"
},
"type": "person",
"actual": true,
"member": [
{
"entity": {
"reference": "Patient/26b8f4f5-441f-481e-b50a-0cd265df4f08"
},
"period": {
"start": "2014-10-08"
}
},
{
"entity": {
"reference": "Patient/6b0ad32c-eb5d-4db1-81bc-000369b9a1f5"
},
"period": {
"start": "2015-04-02"
}
}
]
}

Hello @Kedar782
Thank you, it would be great if you can send me the batch bundle for above Group, I am not familiar with FHIR data
Thank you very much

There was a configuration step omitted in the documentation which is in this commit:
https://github.com/microsoft/fhir-proxy/commit/d2fc705d8701168a7de215071ab23fadd240dd0c
Please execute the missing step to set the FP-EXPORT-STORAGEACCT this should resolve the 500 error.

Hello @Kedar782
Thank you, it would be great if you can send me the batch bundle for above Group, I am not familiar with FHIR data
Thank you very much

The Inferno Website offers example data and defines the data you need to be on the FHIR Server as the document states this does not cover FHIR Data requirements

There was a configuration step omitted in the documentation which is in this commit: d2fc705 Please execute the missing step to set the FP-EXPORT-STORAGEACCT this should resolve the 500 error.

It is not working even after following this step. I'm still getting the 500 error.

Hello @Kedar782
I can pass 7.3.03

If you follow the new guideline, add FP-EXPORT-STORAGEACCT = connection string of export storage account
in the proxy function app settings

Hope this can help you to pass this

Hello @sordahl-ga @Kedar782
Thank you for the new commit, it fixed all the Goal and Procedure issue.
I don't know why but i still not pass the 4.18.05

With the Provenance issue
"severity": "warning",
"code": "forbidden",
"diagnostics": "Could not find Patient Reference Patient/e91975f5-9445-c11f-cabf-c3c6dae161f2 in reference fields target of resource Provenance/6368d1b9-f765-9495-acfc-4b0c8d11db3e"
I already deploy new version of proxy

Do you have any hint?
Thank you

@vohuythao
Please add a target element in your provenance resource( id 6368d1b9-f765-9495-acfc-4b0c8d11db3e) with a patient reference Patient/e91975f5-9445-c11f-cabf-c3c6dae161f2

@Kedar782 , have you passed the 7.3.03, error 500?


duplicate ids freezes / crashes

I don't have time to dig in I'm afraid, but I was running into an issue with jstree crashing my browser and freezing.  I realized I was duplicating my ids on subsequent create_node, and once fixed, this stopped.   I think it was the root cause.

jsTree has never supported duplicate IDs - I think it is documented in a few places.


Time sync not working in El Capitan?

I just discovered this program yesterday, and was so glad to see that someone had done some work to fix this annoying bug in Docker for Mac! So I ran the automatic install script, and happily went about my day, secure in the knowledge that those frequent S3 timing errors were a thing of the past.
Sadly, when I got into work this morning and woke my machine back up, that turned out not to be the case. The ~/.docker-time-sync-agent.log file says that docker-time-sync-agent ran when my machine woke up, but I'm still getting S3 timing errors. I even ran update-docker-time manually, and it appeared to run successfully, but S3 errors still abound. :(
The issue is definitely with the system time in the VM, as running date within one of my containers gives me a datetime that's over a dozen hours in the past. And strangely, installing ntp and running ntpdate pool.ntp.org manually within my container gives this error, even though I'm root!
24 May 04:31:28 ntpdate[249]: step-systime: Operation not permitted

I'm running on OSX El Capitan version 10.11.6, with the latest updates. My Docker version is Docker 17.03.1-ce-mac12.

Just fyi – latest Docker for Mac beta fixes the time sync issue.

How do I get it? Docker for Mac says I have the latest version when I do "Check for updates".

https://docs.docker.com/docker-for-mac/install/#download-docker-for-mac "Edge channel"

Thanks!

@coredumperror as @CvX mentioned time drift issue is fixed in the latest edge (beta) version of Docker for Mac.
The issue you have mentioned with docker-time-sync-agent seems to be from update-docker-time script. You can debug by running the shell commands in the script individually on you machine. Unfortunately I don't have access to El Capitan.

Closing due to inactivity for a long time.


swc_plugin_proxy-0.44.1 not compatible with swc_common-0.34.4

Describe the bug
I've created a new plugin with the swc cli and could not compile it afterwards.
error[E0308]: mismatched types
  --> .../index.crates.io-6f17d22bba15001f/swc_plugin_proxy-0.44.1/src/comments/plugin_comments_proxy.rs:87:48
   |
87 |             comments.add_leading_comments(pos, comments);
   |                      --------------------      ^^^^^^^^ expected `Vec<Comment>`, found `&Box<dyn Comments>`
   |                      |
   |                      arguments to this method are incorrect
   |
   = note: expected struct `Vec<Comment>`
           found reference `&Box<dyn Comments>`
note: method defined here
  --> .../index.crates.io-6f17d22bba15001f/swc_common-0.34.4/src/comments.rs:33:8
   |
33 |     fn add_leading_comments(&self, pos: BytePos, comments: Vec<Comment>);
   |        ^^^^^^^^^^^^^^^^^^^^

error[E0308]: mismatched types
   --> .../index.crates.io-6f17d22bba15001f/swc_plugin_proxy-0.44.1/src/comments/plugin_comments_proxy.rs:117:49
    |
117 |             comments.add_trailing_comments(pos, comments);
    |                      ---------------------      ^^^^^^^^ expected `Vec<Comment>`, found `&Box<dyn Comments>`
    |                      |
    |                      arguments to this method are incorrect
    |
    = note: expected struct `Vec<Comment>`
            found reference `&Box<dyn Comments>`
note: method defined here
   --> .../index.crates.io-6f17d22bba15001f/swc_common-0.34.4/src/comments.rs:40:8
    |
40  |     fn add_trailing_comments(&self, pos: BytePos, comments: Vec<Comment>);
    |        ^^^^^^^^^^^^^^^^^^^^^

Input code
No response
Config
No response
Playground link (or link to the minimal reproduction)
https://gist.github.com/Mad-Kat/de900587a273deda6753455fa4133867
SWC Info output
No response
Expected behavior
After setup a new plugin, it compiles and the tests run.
Actual behavior
No response
Version
swc_core: 0.96.7
Additional context
No response

Yes worked like a charm. Thank you so much for the speedy fix


use long long for tenmul_dims

Since the linop_matrix_create doubles the dimensions two times (to 64), it is necessary to use a long long for the dimension checking.

Actually unsigned long will suffice


How to use .dump in sqlite ?

export async function DumpDB() {
  await load;
  return db.execute(".dump");
}

Uncaught (in promise) error returned from database: near ".": syntax error

as far as i know, .dump is only accessible via the sqlite3 CLI program which is not used to work with databases programmatically.
So in short: you can't do that in your code. Only manually in the console.


high routing latency when FC is connected via usb

Setup:

Tested on v2 mavlink router
Device=/dev/ttyACM0
Baud=115200

Routing:

QGC > MR over udp
mavlink handler on Companion > MR on tcp
FC > MR on usb

Screened ports using tcpdump:

After disconnecting QGC. QGC heartbeat in the udp stream stops but keeps showing up on tcp.
Latency varies but at least a few minutes depending on how long it's been running.

I can only guess what's going wrong here, but it sounds like there's some re-transmission happening on the TCP link. Either because some packets were lost on the way to the companion or the TCP-ACK was lost somehow. I don't know, at which intervals TCP tries to re-transmit lost packets. But if some kind of exponential backoff is used, the re-transmission "echo" can last some time.
Could you try to run mavlink-router in a shell (instead of via systemd) with the -v verbose flag and check, if the QGC heartbeat is still being transmitted via the TCP companion endpoint after you have disconnected QGC? If it is showing up there, my theory above isn't true and there must be another reason. Otherwise it's in the lower layers of TCP itself and TCP just behaves as it's intended to work.

Any progress on this issue? I'm running a similar build with USB connection to flight controller, but I'm running a UDP connection between MR and Mission Planner and everything appears to be working. I'll try a TCP connection to see if I get similar behavior.

@akkawimo  not sure I understand the setup, sorry. What's "mavlink handler on companion > MR on tcp"? Do you have another sw receiving the mavlink stream and forwarding it to mavlink-router? Is this TCP connection on the same board, using localhost or a different one?

@akkawimo  any update?


Feature Request: setting to open in Safari instead of the in app browser.

This feature will be part of 6.1 and is in the current TestFlight build.


When an attack doesnt have Previous attacks it should not be playable during the recovery frames of other attacks

Change you to assignee.
Think about what you want here.
If you need separate combo time for each attack, then tell me

that does not mean that you can only preform moves after the recovery frames, you frames you can preform attacks on are the recovery and the max combo time. (instant moves can also during the antisipation and active frames of interaptable moves)

Done, check it out. btw, NOT A BUG


macOS is unable to restart while VS Code is open

Type: Bug

Open VS Code
Attempt to restart macOS from the Apple menu
Observe the OS does not reboot until VS Code is manually quit.

VS Code version: Code 1.75.0 (e2816fe719a4026ffa1ee0189dc89bdfdbafb164, 2023-02-01T15:26:31.005Z)
OS version: Darwin arm64 22.2.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
42, 44, 21

Memory (System)
8.00GB (0.16GB free)

Process Argv
--crash-reporter-id 4d50f442-22cc-49b8-b7b1-3de0273dfce9

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

shell-syntax
bma
1.0.3

compareit
in4
0.0.2

dupchecker
jia
0.1.7

prettify-json
moh
0.0.3

vscode-docker
ms-
1.23.3

remote-containers
ms-
0.266.1

powershell
ms-
2023.1.0

hosts
tom
1.2.0

vscode-todo-highlight
way
1.0.5

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
vscccc:30610679
pyindex848:30662994
nodejswelcome1:30587005
2e4cg342:30602488
pyind779:30662992
89544117:30613380
pythonsymbol12:30657548
2i9eh265:30646982
vsccsb:30662443
vscodedisable:30660115

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.75.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Same here. Mac Mini M1, Ventura 13.2.1, clean install of VSCode 1.75.1 with NO extensions & default options.
It seems it just ignores TRM signal or something. Very annoying.

@VSCodeTriageBot the version does not matter. I tested on the latest and it's doing the same thing.

Yeah, I'm also on latest and it's happening for me too (on a Macmini9,1 running Ventura 13.2.1).

I am unable to repro with latest insiders, please check if the issue persists in latest versions.

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!


nested namespace with desc not work

 namespace :order, desc: "order" do

      namespace :product, desc: "product" do
       ....
      end
end

but the swagger_doc:
{
"path": "/order.{format}",
"description": "Operations about orders"
}

Can you write a test that reproduces this? I am pretty sure nested namespaces work fine.

I'm actually having the same problem, all of my base routes are there but the nested ones do not show up. I'm using grape 0.11 and grape swagger 0.10.1 set up pretty standard. Maybe an issue with the latest version of grape?

Sorry this was a confusion on my part on how swagger actually worked. I didn't realize that you had to go to swagger_doc/resource_name.json to get the nested routes, I assumed everything would be in the root json file.
Maybe @TangMonk's issue too?

same here :+1:
I have two mounted APIs with one namespace in each. The first one works, the second don't.

+1


fix compileDTS overlay for raspberry-pi

Due to the recent change in https://github.com/NixOS/nixpkgs/pull/251898
the evaluation of my raspberry-pi-4 config is now broken with the following error:
error:
       … while calling the 'derivationStrict' builtin

         at /builtin/derivation.nix:9:12: (source not available)

       … while evaluating derivation 'nixos-system-kari-23.11.20231007.a1decd5'
         whose name attribute is located at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/pkgs/stdenv/generic/make-derivation.nix:300:7

       … while evaluating attribute 'buildCommand' of derivation 'nixos-system-kari-23.11.20231007.a1decd5'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/nixos/modules/system/activation/top-level.nix:53:5:

           52|     passAsFile = [ "extraDependencies" ];
           53|     buildCommand = systemBuilder;
             |     ^
           54|

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: attribute 'compileDTS' missing

       at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/nixos/modules/hardware/device-tree.nix:91:7:

           90|       in
           91|       pkgs.deviceTree.compileDTS {
             |       ^
           92|         name = "${o.name}-dtbo";

The full stack trace is:
error:
       … while calling the 'derivationStrict' builtin

         at /builtin/derivation.nix:9:12: (source not available)

       … while evaluating derivation 'nixos-system-kari-23.11.20231007.a1decd5'
         whose name attribute is located at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/pkgs/stdenv/generic/make-derivation.nix:300:7

       … while evaluating attribute 'buildCommand' of derivation 'nixos-system-kari-23.11.20231007.a1decd5'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/nixos/modules/system/activation/top-level.nix:53:5:

           52|     passAsFile = [ "extraDependencies" ];
           53|     buildCommand = systemBuilder;
             |     ^
           54|

       … while calling 'g'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/attrsets.nix:599:19:

          598|           g =
          599|             name: value:
             |                   ^
          600|             if isAttrs value && cond value

       … from call site

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/attrsets.nix:602:20:

          601|               then recurse (path ++ [name]) value
          602|               else f (path ++ [name]) value;
             |                    ^
          603|         in mapAttrs g;

       … while calling anonymous lambda

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:242:72:

          241|           # For definitions that have an associated option
          242|           declaredConfig = mapAttrsRecursiveCond (v: ! isOption v) (_: v: v.value) options;
             |                                                                        ^
          243|

       … while evaluating the option `system.systemBuilderCommands':

       … while calling anonymous lambda

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:822:28:

          821|         # Process mkMerge and mkIf properties.
          822|         defs' = concatMap (m:
             |                            ^
          823|           map (value: { inherit (m) file; inherit value; }) (builtins.addErrorContext "while evaluating definitions from `${m.file}':" (dischargeProperties m.value))

       … while evaluating definitions from `/nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/nixos/modules/system/boot/kernel.nix':

       … from call site

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:823:137:

          822|         defs' = concatMap (m:
          823|           map (value: { inherit (m) file; inherit value; }) (builtins.addErrorContext "while evaluating definitions from `${m.file}':" (dischargeProperties m.value))
             |                                                                                ^
          824|         ) defs;

       … while calling 'dischargeProperties'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:894:25:

          893|   */
          894|   dischargeProperties = def:
             |                         ^
          895|     if def._type or "" == "merge" then

       … from call site

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:900:11:

          899|         if def.condition then
          900|           dischargeProperties def.content
             |           ^
          901|         else

       … while calling 'dischargeProperties'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:894:25:

          893|   */
          894|   dischargeProperties = def:
             |                         ^
          895|     if def._type or "" == "merge" then

       … from call site

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/nixos/modules/system/boot/kernel.nix:331:15:

          330|             ln -s ${config.system.modulesTree} $out/kernel-modules
          331|             ${optionalString (config.hardware.deviceTree.package != null) ''
             |               ^
          332|               ln -s ${config.hardware.deviceTree.package} $out/dtbs

       … while calling 'optionalString'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/strings.nix:243:5:

          242|     # String to return if condition is true
          243|     string: if cond then string else "";
             |     ^
          244|

       … while calling 'g'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/attrsets.nix:599:19:

          598|           g =
          599|             name: value:
             |                   ^
          600|             if isAttrs value && cond value

       … from call site

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/attrsets.nix:602:20:

          601|               then recurse (path ++ [name]) value
          602|               else f (path ++ [name]) value;
             |                    ^
          603|         in mapAttrs g;

       … while calling anonymous lambda

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:242:72:

          241|           # For definitions that have an associated option
          242|           declaredConfig = mapAttrsRecursiveCond (v: ! isOption v) (_: v: v.value) options;
             |                                                                        ^
          243|

       … while evaluating the option `hardware.deviceTree.package':

       … while calling anonymous lambda

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:844:17:

          843|       if isDefined then
          844|         if all (def: type.check def.value) defsFinal then type.merge loc defsFinal
             |                 ^
          845|         else let allInvalid = filter (def: ! type.check def.value) defsFinal;

       … from call site

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/modules.nix:844:22:

          843|       if isDefined then
          844|         if all (def: type.check def.value) defsFinal then type.merge loc defsFinal
             |                      ^
          845|         else let allInvalid = filter (def: ! type.check def.value) defsFinal;

       … while calling 'check'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/types.nix:619:15:

          618|       descriptionClass = "conjunction";
          619|       check = x: x == null || elemType.check x;
             |               ^
          620|       merge = loc: defs:

       … from call site

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/types.nix:619:31:

          618|       descriptionClass = "conjunction";
          619|       check = x: x == null || elemType.check x;
             |                               ^
          620|       merge = loc: defs:

       … while calling 'check'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/types.nix:493:15:

          492|       descriptionClass = "noun";
          493|       check = x: isStringLike x && builtins.substring 0 1 (toString x) == "/";
             |               ^
          494|       merge = mergeEqualOption;

       … while evaluating derivation 'device-tree-overlays'
         whose name attribute is located at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/pkgs/stdenv/generic/make-derivation.nix:300:7

       … while evaluating attribute 'buildCommand' of derivation 'device-tree-overlays'

         at /nix/store/b7xl2abnwi94znl0hpqk43z41m9l5spa-source/raspberry-pi/4/apply-overlays-dtmerge.nix:8:3:

            7|   nativeBuildInputs = [ dtc libraspberrypi ];
            8|   buildCommand = let
             |   ^
            9|     overlays = toList overlays';

       … from call site

         at /nix/store/b7xl2abnwi94znl0hpqk43z41m9l5spa-source/raspberry-pi/4/apply-overlays-dtmerge.nix:21:9:

           20|
           21|       ${flip (concatMapStringsSep "\n") overlays (o: ''
             |         ^
           22|       overlayCompat="$(fdtget -t s "${o.dtboFile}" / compatible)"

       … while calling 'flip'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/trivial.nix:138:16:

          137|   */
          138|   flip = f: a: b: f b a;
             |                ^
          139|

       … from call site

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/trivial.nix:138:19:

          137|   */
          138|   flip = f: a: b: f b a;
             |                   ^
          139|

       … while calling 'concatMapStringsSep'

         at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/lib/strings.nix:117:5:

          116|     # List of input strings
          117|     list: concatStringsSep sep (map f list);
             |     ^
          118|

       … while calling anonymous lambda

         at /nix/store/b7xl2abnwi94znl0hpqk43z41m9l5spa-source/raspberry-pi/4/apply-overlays-dtmerge.nix:21:51:

           20|
           21|       ${flip (concatMapStringsSep "\n") overlays (o: ''
             |                                                   ^
           22|       overlayCompat="$(fdtget -t s "${o.dtboFile}" / compatible)"

       error: attribute 'compileDTS' missing

       at /nix/store/6bs7cvmiiv6792bs2mbhigmr0jgaj9ma-source/nixos/modules/hardware/device-tree.nix:91:7:

           90|       in
           91|       pkgs.deviceTree.compileDTS {
             |       ^
           92|         name = "${o.name}-dtbo";

Description of changes
due to https://github.com/NixOS/nixpkgs/pull/251898 the compileDTS function has been moved to device-tree
nixos-hardware has an own overlay for the raspberry-pi which needs the compileDTS function now, too.
This commit adds it.
Things done

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

Please note: This has not yet been tested IRL on my raspberry-pi. But the evaluation and compilation of my NixOS config now doesn't throw an error.
If anyone is willing to test it, please do so and report back.

@Majiir awesome, thanks, I simplified the fix.

Are we sure that any overlay is needed for device-tree? A few fixes have been merged in nixpkgs since this fork was created.

This is a good question. There are subtle differences in there.
Maybe @ipetkov or @carlossless could chime in?
But this is maybe a matter for another PR. This (now small) change at least allows building the config again with the current nixpkgs

@gador @Majiir the main function of that overlay (in the nix sense) is to use dtmerge instead of fdtoverlay for merging the device tree overlays. dtmerge was needed because the overlays (example) that rpi distributes are not compatible with fdtoverlay.
I don't remember the specifics about what exactly is not supported by fdtoverlay and I'm not aware if anything changed.
I won't have time for this myself in the coming weeks, but it could be tested by removing the package overlay, turning on the poe-hat overlay and seeing if it builds.

Let me know when this is good to be merged.

@carlossless thanks for the feedback! I guess one can have a look at removing the custom dtmerge function at a later point?
I just tested this PR in my config on my raspberry pi 4 with POE+ hat. Works just fine


ServerSide - Links added in OnInitialized are not rendered until OnAfterRender

I'm using server side and adding links in OnInitialized, but what ends up happening is the page initially loads with the nodes laid out then shortly after the links are rendered.  I checked the performance sample and do not see it happen in WASM, but it appears to be reproducible using that sample on ServerSide.
Any tips on how to get them rendered with the initial draw of the nodes?

Hello,
That is unfortunately normal if you're using nodes that don't have a known size, which means the links wait for the nodes to be rendered, retrieve their size, then render themselves with the positions received.
It doesn't show up in WASM because there is no latency so it's very fast, but I'm afraid in Server Side it will show unless you maybe use fixed-size nodes

As a test I tried to set the sizes manually, but it doesn't appear to render them sooner, is there a way currently that it's possible to achieve this?

Sorry, I meant the nodes AND ports. Either you set all that information manually, or you can use the new anchors in 3.0 with only the node size set


Suggestion: notifications when user is in the topic

I suggest to make  changes to FBaseMessagingService.java file and to add this code:
    public boolean isScreenOn() {
        PowerManager powerManager = (PowerManager) getSystemService(POWER_SERVICE);
        boolean result= powerManager.isInteractive();
        return result;
    }

And to change the line:
if (visibleTopic != null && visibleTopic.equals(topicName) ) {
to:
boolean CheckScreenOn = isScreenOn();
if (visibleTopic != null && visibleTopic.equals(topicName) && CheckScreenOn) {

I m not an Android developer and probably it's possible to do it in another way but for me this fixes problem with notifications.
The problem appears when user reads a topic and turns phone's screen off without returning to the list of the topics. He stops getting notifications for this topic and I think it's not correct. There is no need to display notifications only when phone's screen is on. Otherwise, the user may not know that he has new messages.

The app detects going into the background and disconnects:
https://github.com/tinode/tindroid/blob/master/app/src/main/java/co/tinode/tindroid/TindroidApp.java#L148
Could you please show the steps to reproduce the problem.

The FBMessaging service does not disconnect and it's a bug, but that has nothing to do with going into the background. The service will disconnect in the next release.

Comment in the following code:
  if (visibleTopic != null && visibleTopic.equals(topicName)) {
                // No need to display a notification if we are in the topic already.
                Log.d(TAG, "Topic is visible, no need to show a notification");
                return;
            }

in the FBaseMessagingService.java file says:
No need to display a notification if we are in the topic already.
and it works. Unfortunately it works even with phone's screen off.
For example, I get a new message from Alice. I tap on the notification, access the topic, write answer, send it to Alice and turn phone's screen off, leaving the topic open. I don't get notifications for this topic anymore.
To get future notifications for this topic, I need to push "back" button after sending message to Alice, and return to the topics list.
I see this problem on 2 Android phone's. For a long time, I couldn't understand why notifications sometimes don't work. Then I checked FBaseMessagingService.java file, made changes that I published in my 1st post and the problem is gone.

The visibleTopic is set to null when onWindowFocusChanged(false) is called. Are you saying onWindowFocusChanged is not called when the screen is turned off? Let me check that.

This should fix the problem you have been observing:
https://github.com/tinode/tindroid/commit/bd8ce597f805ec3ad5f10d2b2d5dc6720af6fd43
Specifically this change:
https://github.com/tinode/tindroid/commit/bd8ce597f805ec3ad5f10d2b2d5dc6720af6fd43#diff-b195deb0e46573e2c9ada2d75e4bb18cR423

Yes, now the problem is fixed. thanks.


Svelte 5 Upgrade

[x] Convert state management to Runes
[x] Remove svelte/legacy
[x] Remove createEventDispatcher and replace on:event -> onevent
[x] Convert code to TypeScript
[x] Update docs
[ ] Simplify Props types, use Stripe's option types directly when possible.
[ ] Add <Elements.Deferred> component for creating payment intent later.
[ ] Manually test all components
[ ] Create changelog of breaking changes

I'd be willing to help test a pre-release of this.

Thanks @niemyjski!


Updated javadocs

Message javadocs:

Switched around the order of Message#clone/copy javadocs to accurately
reflect which is the alias because my ocd is massive
Fixed typo, TexComponent -> TextComponent

CPS javadocs:

Changed plurals to be consistent

KeyBind javadocs:

Changed create to creates in the constructor documentation to be
consistent

OnRenderTrigger javadocs:

fixed kerby's dumb dumb copy paste from OnChatTrigger

Thank you for your contribution :)


Localized content/bn/docs/tutorials/kubernetes-basics/deploy-app/_index.md

Signed-off-by: Noor E Jannat Nafia<EMAIL_ADDRESS>Localizes content/bn/docs/tutorials/kubernetes-basics/deploy-app/_index.md to Bengali
This is an initial PR as may need some improvements!
Tracking issue: #31677

/label tide/merge-method-squash


Fixed tests

Test all passing

Hi @oferns
Not sure on the purpose of this PR - the only significant changes are in the Row - builds a model unit test, changing the row from sparse (i.e. missing column 'C') to contiguous.
Also - the tests pass already.
Could you provide explaination?
Thanks,
Guyon


Block JavaScript is loaded in head instead of footer dist/blocks.build.js

Bug Report
Expected Behavior
When viewing the source on /wp-admin/post-new.php, I expected the script tag for dist/blocks.build.js to appear in the footer.
I also expected the script to have a URL parameter of ?ver=4.9.6 (since I'm running WordPress core 4.9.6).
Actual Behavior
The following line appeared in the <head> of /wp-admin/post-new.php with ?ver=1
<script type='text/javascript' src='http://gutenberg.test/wp-content/plugins/my-gblock/dist/blocks.build.js?ver=1'></script>

Steps to Reproduce

From wp-content/plugins/, generate a block by running $ create-guten-block my-gblock from the command line

Change to the new plugin directory $ cd my-gblock

Activate the plugin ($ wp plugin activate my-gblock)

Run $ npm start

Log into the WordPress site and click Posts > Add New

View the page source

node -v: v8.11.2

npm -v: 5.6.0

yarn --version (if you use Yarn): N/A

npm ls cgb-scripts (if you haven’t ejected):

<EMAIL_ADDRESS>/Users/sal/localsites/gutenberg/app/public/wp-content/plugins/my-gblock
└──<EMAIL_ADDRESS>
Then, specify:

Operating system: MacOS
Browser and version (if relevant): Chrome Version 66.0.3359.181

Possible Solution
Add an empty string ('') as the version number for wp_enqueue_script() in src/init.php
Currently, the version number is commented out
// filemtime( plugin_dir_path( __DIR__ ) . 'dist/blocks.build.js' ), // Version: filemtime — Gets file modification time.

which results in the next parameter, true // Enqueue the script in the footer. being used as the version number, rather than the footer parameter.
Changing the line to
'', // filemtime( plugin_dir_path( __DIR__ ) . 'dist/blocks.build.js' ), // Version: filemtime — Gets file modification time.

results in
<script type='text/javascript' src='http://gutenberg.test/wp-content/plugins/my-gblock/dist/blocks.build.js?ver=4.9.6'></script>

appearing in the footer of the page.
Note: This also appears to have come up in the conversation on PR 52.
Reproducible Demo
https://github.com/salcode/create-guten-block-js-head-bug-report

Oh, I see the problem there. Also, thanks for reporting the best issue ever. All the info I needed is there. I plan to reduce the cgb footprint, use namespaces and update the enqueuing process while at it. This should be covered there unless you can send in a PR to address it.

@boostio funded this issue with $10. Visit this issue on Issuehunt


fix: networks now update and refresh connection for solflare wallet

Breaking Changes
Solving the network mismatch problem for solflare wallet adapter. On change network refreshing a connection with changed chain
Changes

feat:
fix: Solflare network mismatch, doesn't update on switch network
chore:

Associated Issues
https://github.com/WalletConnect/web3modal/pull/2298
closes #...

@glitch-txs This included the last release right?


Rename repository to match ros convention

Summary
We need to match the package naming convention for releasing: maliput_ros
Related to https://github.com/ros/rosdistro/pull/36883#issuecomment-1511879034

Closed as done


added solution for 4sum in C++

It is showing 'Wrong Answer '. Please correct the code.

The solution is provided by another user, before you change the code.


doc: formalize the current loader-layer interface

Expand the section "Discovery of layer entry points" into "Layer Library
Interface".  In general, the new section defines an interface that allows
a loader to discover and insert layers while staying compliant.  Other
sections have additional guidelines for layers to follow.

+1

Cherry picked to master. Thanks!


Whenever using the script I get an error that says "Spotify isn't installed"

Do you have the latest versions of Spicetify, Spotify and Better Bloom?

[X] I'm using the latest versions of Spicetify, Spotify and Better Bloom

Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Is your problem definitely caused by the theme?

[X] I have checked the app behavior without the theme and it is different

Environment & Computer Info
- Spotify version: <IP_ADDRESS>1
- Installed from: flatpak
- Spicetify version: 2.36.8
- Color scheme:
- Extensions:
- Custom apps:
- Better Bloom installation method: Bash Script

Description
Whenever I run the installation script I get this error:
Spotify isn't installed or doesn't exist in your PATH. Better-Bloom is a Spotify theme, and so it's essential. Please install Spotify (https://spotify.com/download) and run this script again to continue. If Spotify is already installed, add it to your PATH variable and rerun this script. Example command for adding to PATH: export PATH=~/spotify:$PATH Abort!
I tried installing other themes and it worked fine. this one however, it didn't work. Also, I tried adding spotify to $PATH but no use.
Screenshots

i will check it on my linux vm and figure out the issue rn !

cant find the issue i will fix it as soon as possible

added new script can you check the script installation @HuangLinBao

It worked, Thank you so much


Allow tuple for geo_field property

I've been asked to create a Django app where users can manage their existing library of KML documents along with other GIS formats.  Unfortunately, it seems that about half of the documents have 2D geometry while the rest are 3D.  This causes issues when saving the geometry model because GeoDjango's GeometryField has a default dim value of 2, which results in errors when uploading a 3D file.
My conundrum is similar to the question here at GIS Stackexchange, just inverted.  However, I would not want to not force a 2D transform, as the height values are a big part of the data.
A short term solution is to include two geometry fields within the model, one 2D and the other 3D.  Depending on the dimension, the geometry will be saved to the appropriate field in the model.  In order to use this within the serializer, a GeometrySerializerMethodField needs to be used, much like
class SegmentSerializer(gis_serializer.GeoFeatureModelSerializer):
    trail = TrailSerializer()
    geometry = gis_serializer.GeometrySerializerMethodField()

    def get_geometry(self, obj):
        print(obj)
        if obj.two_d_geom is None:
            return obj.three_d_geom
        else:
            return obj.two_d_geom

    class Meta:
        model = TrailSegment
        geo_field = 'geometry'

If geo_field were to accept a tuple as a value, then the above could be functionally identical to
class SegmentSerializer(gis_serializer.GeoFeatureModelSerializer):
    trail = TrailSerializer()

    class Meta:
        model = TrailSegment
        geo_field = ('two_d_geom', 'three_d_geom', )

This would also benefit devs who find themselves doing far too many joins that impact performance and use denormalization as a solution.

I don't see the benefit of including such a feature in drf-gis right now, you should be able to extend drf-gis serializers and implement what you need. You might as well publish your solution somewhere so we can better understand the use case in more detail.

This would be awesome


Remove empty hash URI fragments after updates.

Fixes https://jira.duraspace.org/browse/FCREPO-1764

The essence of this PR looks good, pending response to @ajs6f's comments.

@ajs6f I've pushed a new commit that wraps any exceptions in a RepositoryRuntimeException, and casts the JCR PropertyIterator to an Iterator.

Resolved with: https://github.com/fcrepo4/fcrepo4/commit/a71298f4817887538112dccc1979ad40956c8ff3


Error in compiling

When I execute
docker build -t concisions/windscribe-socks-server:latest .
It does all the step, but when it reach Step 5/8, it stops:
Step 5/8 : COPY docker-entrypoint.sh docker-healthcheck.sh /
COPY failed: stat /var/lib/docker/tmp/docker-builder224350060/docker-entrypoint.sh: no such file or directory

I am on MacOS

After cloning the repository, did you change directory to the repository folder first?


Saved svg files are broken

Hey Goeff, you did an amazing work on osmnx.
But I've trouble when I want to save shapes or networks as a svg file.
I've did: import osmnx as ox #matplotlib inline ox.config(log_file=True, log_console=True, use_cache=True) G = ox.graph_from_place('Hamburg, Germany', network_type='drive_service', simplify=False) G_projected = ox.project_graph(G) fig, ax = ox.plot_graph(G_projected, fig_height=30, node_size=0, edge_linewidth=0.5, save=True, file_format='svg', filename='Hamburg')
After running it the last line says „Saved the figure to disk in 48.39 seconds"
And when I'm going to open it, Illustrator prompting an error message that the file seems to be broken.
Am I doing something wrong?
I'm running osmnx on Mac OS El Capitan with Python 3.5

Your code shouldn't work as presented because OSM's first result when geocoding "Hamburg, Germany" is a point, but OSMnx needs a polygon or multipolygon to create a network. You'd need to use the which_result argument to specify that you want the 2nd result from the geocoder. See #16.
Also, you should probably simplify your network after projecting it to reduce the number of nodes and edges to something more topologically correct for a street network.
Finally, it looks like the network for Hamburg has hundreds of thousands of nodes and edges. It would not be a surprise if Illustrator is unable to successfully open an SVG file with hundreds of thousands of elements inside it.
To figure out what's going on, first try the network for a simpler city and verify you can load it in Illustrator:
import osmnx as ox
ox.config(log_console=True, use_cache=True)
G = ox.graph_from_place('Modena, Italy', network_type='drive', simplify=False)
G_projected = ox.project_graph(G)
G_simplified = ox.simplify_graph(G_projected)
fig, ax = ox.plot_graph(G_simplified, fig_height=30, node_size=0, edge_linewidth=0.5,
                        save=True, file_format='svg', filename='mod_simp')

This SVG file works just fine for me in Illustrator. Next, try it for the simplified Hamburg network (using which_result to get the correct administrative boundary geometry):
import osmnx as ox
ox.config(log_console=True, use_cache=True)
G = ox.graph_from_place('Hamburg, Germany', which_result=2, network_type='drive', simplify=False)
G_projected = ox.project_graph(G)
G_simplified = ox.simplify_graph(G_projected)
fig, ax = ox.plot_graph(G_simplified, fig_height=30, node_size=0, edge_linewidth=0.5,
                        save=True, file_format='svg', filename='ham_simp')

Opening this SVG file in Illustrator makes Illustrator hang for me because of the immense number of nodes and edges, but I do not get an error message that the file is broken. If you experience the same as me, we can close this issue as it would be a limitation of Illustrator handling large SVGs with many elements. However, if you get an error message, please post its exact contents as well as full system information for your machine and your Python environment's package versions.

@c9rbjn checking back once again on this open issue... see previous comment.

Thanks Mr. Boeing for your support!


Automatic send the url to the comment ?

I love this tool !
But i don't see the url.
If I want to reproduca a bug?

Do you can import a automatic send url to trello-comment ?


Can't execute notebook cells

Issue Type: Bug
Just updated VSC and I can no longer execute jupyter notebook cells
This happens when working both in local and connected to a SSH host, both with python and markdown cells
I get the following error:
Illegal state: service accessor is only valid during the invocation of its target method
Checked the output every extension and log, there's no errors or no clue as to what is happening.
VS Code version: Code - Insiders 1.68.0-insider (8c882677f947ef7ec748e7f031c7b5470aaf9be1, 2022-05-11T05:17:12.474Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No
Remote OS version: Linux x64 5.4.0-1072-aws

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1035G1 CPU @ 1.00GHz (8 x 1190)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
11.70GB (1.88GB free)

Process Argv
--crash-reporter-id 4285aea8-8024-49ba-97f2-9d60622a70f0

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: AWS

OS
Linux x64 5.4.0-1072-aws

CPUs
Intel(R) Xeon(R) Platinum 8259CL CPU @ 2.50GHz (8 x 2840)

Memory (System)
30.91GB (28.53GB free)

VM
0%

Extensions (11)

Extension
Author (truncated)
Version

jupyter-keymap
ms-
1.0.0

remote-containers
ms-
0.235.0

remote-ssh
ms-
0.80.0

remote-ssh-edit
ms-
0.80.0

remote-wsl
ms-
0.66.2

vscode-remote-extensionpack
ms-
0.21.0

python
ms-
2022.6.1

vscode-pylance
ms-
2022.4.3

jupyter
ms-
2022.5.1001331008

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.6

(2 theme extensions excluded)

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30422396
pythontb:30258533
pythonptprofiler:30281269
vsdfh931:30280409
vshan820:30294714
pythondataviewer:30285072
vscod805cf:30301675
pythonvspyt200:30323110
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30404738
wslgetstarted:30449409
pythonvspyt640:30438691
vscscmwlcmt:30465136
cppdebugcf:30475217
vsclangdc:30480125

Snapshot of message:

I would post snapshots of the outputs of extensions and logs, but there is no output whatsoever generating when attempting to run a cell

Same.

I was able to workaround this issue by using the command "Notebook: Execute Cell".

same

I was able to workaround this issue by using the command "Notebook: Execute Cell".

Via the command palette? (Cmd/Ctrl+shift+p)?

Via the command palette? (Cmd/Ctrl+shift+p)?

Yep, that's how I first found it. You can also invoke the command directly from a cell with ctrl+alt+enter.

Same problem here too, thanks in advance for addressing!

I fixed it by determining the keyboard shortcut again. (ctrl + enter)
However, the run icon still does not work. But the shortcut works.

run icon or regular shortcuts (shift+enter, ctrl+enter) did not work form me, only ctrl+alt+enter did
anyways, seems to be fixed with @roblourens PR, thanks for the quick response

@ghylander How can I activate the PR in Insiders?

fix was already released for the VSC insiders build, update your installation

when I run jupyter notebook，i type three words  in it,the window can't response occur,then the vscode crushed,what should i dot to slove this problem


Throwing error on given view-source: url string

const normalize = require('normalize-url');
const n = normalize('view-source:https://github.com/sindresorhus/normalize-url/blob/master/index.js')

throws the error:
Thrown:
TypeError [ERR_INVALID_URL]: Invalid URL: http:view-source:https://github.com/sindresorhus/normalize-url/blob/master/index.js
    at onParseError (internal/url.js:243:9)
    at new URL (internal/url.js:319:5)
    at normalizeUrl
    [....]
 {
  input: 'http:view-source:https://github.com/sindresorhus/normalize-url/blob/master/index.js',
  code: 'ERR_INVALID_URL'
}

Is this intentionally an error? Seems unintentional to me because it first prepends http to the input, turning it into 'http:view-source:https://... and then leaving it to the URL function to handle it, if I'm correct.
It would be awesome if we could have view-source urls handled by this module itself. Either define a normalisation result for view-source like how it's done for data urls, or return a clear error message.

I agree the error could be better (PR welcome for that), but I'm not interested in supporting lots of random protocols.


Issue from website page Download Das Keyboard Q Software

From URL: https://www.daskeyboard.io/get-started/software.md
I am getting the following errors after downloading the installation .pkg. Please see the screenshot below:

can you please open a ticket at https://daskeyboard.mojohelpdesk.com


Support obfuscation of output code

This is a catch-all issue for work related to allowing Slang to produce output (both serialized IR and final DXIL/SPIR-V) that exposes as little of the original names used in the source program as possible.

[x] Don't emit HLSL/GLSL names using the IR hints when -obfuscate is specified (#1075)
[x] Strip name-hint decorations in front-end when -obfuscate is specified (#1092)
[x] Make sure that emit logic doesn't depend on mangled symbol names in order to compute names for builtin function (#1096)
[ ] Use hashes for mangled symbol export/import names when -obfuscate is specified

This is considered implemented, closing the issue now.


On generating a general waterfall_plot for a multi-class classification

Hi,
I am using Python 3.6.5 on a windows machine with Shap version 0.34.0.
In the case of a multi-class classification, a way to generate a waterfall_plot is by using this command:
shap.waterfall_plot(shap_explainer.expected_value[0], shap_values[0][0], features= pd_test_inputs.iloc[0,:], feature_names= feature_names, show= False)
The produced display is based on the first row of samples in both "shap_values" and "pd_test_inputs"
If I would like to build a general waterfall_plot that includes all samples in both "shap_values" and "pd_test_inputs", what command would be the most indicated:
a) shap.waterfall_plot(shap_explainer.expected_value[0], np.sum(shap_values[0], axis= 0), features= np.sum(pd_test_inputs, axis= 0))
which produced the figure named "waterfall_first_version.png" in zip file
b) shap.waterfall_plot(shap_explainer.expected_value[0], np.sum(np.abs(shap_values[0]), axis= 0), features= np.sum(np.abs(pd_test_inputs), axis= 0))
which produced the figure named "waterfall_second_version.png" in zip file
Also, I have this suggestion: for symbols in the plot that can't have text inside because of their small size, is it possible to post the text to the right of the symbol?
One more thing. Along the x-axis, I noticed this text E[f(X)].  What value is associated with this text?
Many thanks,
Ivan
multi-class_waterfall_plot.zip

Great suggestions! I spent some time improving the plot and now you can see both the value of E[f(X)] (the expected value of the model evaluated on the background dataset), and the values of small effects.
As for you question do (b) because (a) would always lead to values being close to zero because of cancelations. One small change I would do though is use np.mean instead of np.sum. Then you will match what the summary plot does internally.


Refactor audio tests

Codecov Report

Merging #179 (772202e) into main (bcdf9c2) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #179   +/-   ##
=======================================
  Coverage   97.91%   97.91%
=======================================
  Files          11       11
  Lines        1343     1343
  Branches      359      359
=======================================
  Hits         1315     1315
  Misses         24       24
  Partials        4        4

Flag
Coverage Δ

unittests
97.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.