AI does not respond

I've got everything installed on Windows with WSL2. UI work, but the eternal loading of the answer

same issue

Can you check the logs of the api container, see if it's maybe converting the model and hanging because of that ?

I have an i5-7400, but the important thing is that I used alpaca with llama without ui and it responded in 10 seconds on the same model

i use wsl 2, win 11, intel i7 11800

I'm on a Ubuntu Cloud Image running on Proxmox with Host CPU.
Trying to figure out how to run the Straight API on a different node to see if there is a difference.
File weights/ggml-alpaca-7B-q4_0.bin already converted INFO:     <IP_ADDRESS>:56482 - "GET /models HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:56498 - "GET /chats HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:46222 - "GET /chat/1db1f49e-ca45-4a13-8aee-f6a82bde7b76 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:46224 - "GET /chat/8a32fda3-cc0e-46d0-afed-e7c079a588e4 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:46236 - "GET /chat/1db1f49e-ca45-4a13-8aee-f6a82bde7b76 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:50102 - "GET /models HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:47664 - "GET /chat/8a32fda3-cc0e-46d0-afed-e7c079a588e4 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:47680 - "GET /chat/1db1f49e-ca45-4a13-8aee-f6a82bde7b76 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:55744 - "GET /chat/1db1f49e-ca45-4a13-8aee-f6a82bde7b76 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:48880 - "GET /chat/8a32fda3-cc0e-46d0-afed-e7c079a588e4 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:35214 - "GET /models HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:35224 - "GET /chats HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:48390 - "POST /chat?temp=0.1&top_k=50&max_length=256&top_p=0.95&model=ggml-alpaca-13B-q4_0.bin&repeat_last_n=64&repeat_penalty=1.3&preprompt=Below+is+an+instruction+that+describes+a+task.+Write+a+response+that+appropriately+completes+the+request.+The+response+must+be+accurate%2C+concise+and+evidence-based+whenever+possible.+A+complete+answer+is+always+ended+by+%5Bend+of+text%5D. HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:48390 - "GET /chats HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:48402 - "GET /chat/25b1b3b5-226c-4143-9491-07a8f7acfd51 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:54912 - "GET /chat/8a32fda3-cc0e-46d0-afed-e7c079a588e4 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:54920 - "GET /chat/1db1f49e-ca45-4a13-8aee-f6a82bde7b76 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:54926 - "GET /chat/8a32fda3-cc0e-46d0-afed-e7c079a588e4 HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:46472 - "GET /chat/25b1b3b5-226c-4143-9491-07a8f7acfd51/question?prompt=[PROMPT] HTTP/1.1" 200 OK INFO:     <IP_ADDRESS>:44188 - "GET /models HTTP/1.1" 200 OK

Yeah, using https://github.com/ggerganov/llama.cpp with
./main -m ./models/ggml-alpaca-7b-q4.bin --color -f ./prompts/alpaca.txt -ins
Is outrageously faster than serge.

I'll have a look at this tonight, thanks for reporting!

Thank you for making this.
If there's anything I can do to help, please let me know.

Same here, running on macos, on a macbook pro M2, with podman.
�INFO:     <IP_ADDRESS>:45776 - "GET /chat/637636b2-7e17-4508-a5a9-be04d1c6e894/question?prompt=What+is+a+bridge%3F HTTP/1.1" 200 OK
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/uvicorn/protocols/http/httptools_impl.py", line 436, in run_asgi
    result = await app(  # type: ignore[func-returns-value]
  File "/usr/local/lib/python3.10/dist-packages/uvicorn/middleware/proxy_headers.py", line 78, in __call__
    return await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/dist-packages/fastapi/applications.py", line 276, in __call__
    await super().__call__(scope, receive, send)
  File "/usr/local/lib/python3.10/dist-packages/starlette/applications.py", line 122, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/usr/local/lib/python3.10/dist-packages/starlette/middleware/errors.py", line 184, in __call__
    raise exc
  File "/usr/local/lib/python3.10/dist-packages/starlette/middleware/errors.py", line 162, in __call__
    await self.app(scope, receive, _send)
  File "/usr/local/lib/python3.10/dist-packages/starlette/middleware/cors.py", line 84, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/dist-packages/starlette/middleware/exceptions.py", line 79, in __call__
    raise exc
  File "/usr/local/lib/python3.10/dist-packages/starlette/middleware/exceptions.py", line 68, in __call__
    await self.app(scope, receive, sender)
  File "/usr/local/lib/python3.10/dist-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
    raise e
  File "/usr/local/lib/python3.10/dist-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/dist-packages/starlette/routing.py", line 718, in __call__
    await route.handle(scope, receive, send)
  File "/usr/local/lib/python3.10/dist-packages/starlette/routing.py", line 276, in handle
    await self.app(scope, receive, send)
  File "/usr/local/lib/python3.10/dist-packages/starlette/routing.py", line 69, in app
    await response(scope, receive, send)
  File "/usr/local/lib/python3.10/dist-packages/sse_starlette/sse.py", line 227, in __call__
    async with anyio.create_task_group() as task_group:
  File "/usr/local/lib/python3.10/dist-packages/anyio/_backends/_asyncio.py", line 662, in __aexit__
    raise exceptions[0]
  File "/usr/local/lib/python3.10/dist-packages/sse_starlette/sse.py", line 230, in wrap
    await func()
  File "/usr/local/lib/python3.10/dist-packages/sse_starlette/sse.py", line 219, in stream_response
    async for data in self.body_iterator:
  File "/usr/src/app/main.py", line 161, in event_generator
    async for output in generate(
  File "/usr/src/app/utils/generate.py", line 65, in generate
    raise ValueError(error_output.decode("utf-8"))
ValueError: main: seed =<PHONE_NUMBER>
llama_model_load: loading model from '/usr/src/app/weights/ggml-alpaca-7B-q4_0.bin' - please wait ...
llama_model_load: n_vocab = 32000
llama_model_load: n_ctx   = 512
llama_model_load: n_embd  = 4096
llama_model_load: n_mult  = 256
llama_model_load: n_head  = 32
llama_model_load: n_layer = 32
llama_model_load: n_rot   = 128
llama_model_load: f16     = 2
llama_model_load: n_ff    = 11008
llama_model_load: n_parts = 1
llama_model_load: ggml ctx size = 4529.34 MB

Hey everyone! Can you try to grab the latest main, rebuild the docker container and tell me if it's still happening?
In the case where it's still happening, can you run the following:
docker compose up -d
docker compose exec api bash
llama -m weights/your_model.bin

see if it loads & starts outputting stuff. If it doesn't please send me the traceback.

In the case where it's still happening, can you run the following:
docker compose up -d
docker compose exec api bash
llama -m weights/your_model.bin

see if it loads & starts outputting stuff. If it doesn't please send me the traceback.

7B model works for me, but 13B is not happy:
root@8730b2d5f0fd:/usr/src/app# llama -m weights/ggml-alpaca-13B-q4_0.bin
main: seed =<PHONE_NUMBER>
llama_model_load: loading model from 'weights/ggml-alpaca-13B-q4_0.bin' - please wait ...
llama_model_load: n_vocab = 32000
llama_model_load: n_ctx   = 512
llama_model_load: n_embd  = 5120
llama_model_load: n_mult  = 256
llama_model_load: n_head  = 40
llama_model_load: n_layer = 40
llama_model_load: n_rot   = 128
llama_model_load: f16     = 2
llama_model_load: n_ff    = 13824
llama_model_load: n_parts = 2
llama_model_load: ggml ctx size = 8559.49 MB
llama_model_load: memory_size =   800.00 MB, n_mem = 20480
llama_model_load: loading model part 1/2 from 'weights/ggml-alpaca-13B-q4_0.bin'
llama_model_load: llama_model_load: tensor 'tok_embeddings.weight' has wrong size in model file
llama_init_from_file: failed to load model
main: error: failed to load model 'weights/ggml-alpaca-13B-q4_0.bin'

(MacBook Pro M1 Max, 32 GB RAM)

Similar issues here as with @panicsteve.
7B model works well for me.
13B and 30B are failing (with different errors):
/usr/src/app# llama -m weights/ggml-alpaca-13B-q4_0.bin main: seed =<PHONE_NUMBER> llama_model_load: loading model from 'weights/ggml-alpaca-13B-q4_0.bin' - please wait ... llama_model_load: n_vocab = 32000 llama_model_load: n_ctx   = 512 llama_model_load: n_embd  = 5120 llama_model_load: n_mult  = 256 llama_model_load: n_head  = 40 llama_model_load: n_layer = 40 llama_model_load: n_rot   = 128 llama_model_load: f16     = 2 llama_model_load: n_ff    = 13824 llama_model_load: n_parts = 2 llama_model_load: ggml ctx size = 8559.49 MB llama_model_load: memory_size =   800.00 MB, n_mem = 20480 llama_model_load: loading model part 1/2 from 'weights/ggml-alpaca-13B-q4_0.bin' llama_model_load: llama_model_load: tensor 'tok_embeddings.weight' has wrong size in model file llama_init_from_file: failed to load model main: error: failed to load model 'weights/ggml-alpaca-13B-q4_0.bin'
/usr/src/app# llama -m weights/ggml-alpaca-30B-q4_0.bin main: seed =<PHONE_NUMBER> llama_model_load: loading model from 'weights/ggml-alpaca-30B-q4_0.bin' - please wait ... llama_model_load: n_vocab = 32000 llama_model_load: n_ctx   = 512 llama_model_load: n_embd  = 6656 llama_model_load: n_mult  = 256 llama_model_load: n_head  = 52 llama_model_load: n_layer = 60 llama_model_load: n_rot   = 128 llama_model_load: f16     = 2 llama_model_load: n_ff    = 17920 llama_model_load: n_parts = 4 llama_model_load: ggml ctx size = 20951.50 MB Segmentation fault

llama -m weights/ggml-alpaca-7B-q4_0.bin main: seed =<PHONE_NUMBER> llama_model_load: loading model from 'weights/ggml-alpaca-7B-q4_0.bin' - please wait ... llama_model_load: n_vocab = 32000 llama_model_load: n_ctx   = 512 llama_model_load: n_embd  = 4096 llama_model_load: n_mult  = 256 llama_model_load: n_head  = 32 llama_model_load: n_layer = 32 llama_model_load: n_rot   = 128 llama_model_load: f16     = 2 llama_model_load: n_ff    = 11008 llama_model_load: n_parts = 1 llama_model_load: ggml ctx size = 4529.34 MB llama_model_load: memory_size =   512.00 MB, n_mem = 16384 llama_model_load: loading model part 1/1 from 'weights/ggml-alpaca-7B-q4_0.bin' llama_model_load: ..............................Killed
llama -m weights/ggml-alpaca-13B-q4_0.bin main: seed =<PHONE_NUMBER> llama_model_load: loading model from 'weights/ggml-alpaca-13B-q4_0.bin' - please wait ... llama_model_load: n_vocab = 32000 llama_model_load: n_ctx   = 512 llama_model_load: n_embd  = 5120 llama_model_load: n_mult  = 256 llama_model_load: n_head  = 40 llama_model_load: n_layer = 40 llama_model_load: n_rot   = 128 llama_model_load: f16     = 2 llama_model_load: n_ff    = 13824 llama_model_load: n_parts = 2 llama_model_load: ggml ctx size = 8559.49 MB llama_model_load: memory_size =   800.00 MB, n_mem = 20480 llama_model_load: loading model part 1/2 from 'weights/ggml-alpaca-13B-q4_0.bin' llama_model_load: llama_model_load: tensor 'tok_embeddings.weight' has wrong size in model file llama_init_from_file: failed to load model main: error: failed to load model 'weights/ggml-alpaca-13B-q4_0.bin'
I'm using docker on Mac mini 2018 (i5 Series)

Stumbled accros the issue, too. Could be you are running out of memory...
On Windows/WSL2 it seems your dockerimages are restricted to memory usage. here https://learn.microsoft.com/en-us/windows/wsl/wsl-config you find information to increase memory and CPU for you WSL2 containers.
Not shure if that helps anyone, but fixed the issue with the output like https://github.com/nsarrazin/serge/issues/27#issuecomment-1481279246 for me.

llama_model_load: llama_model_load: tensor 'tok_embeddings.weight' has wrong size in model file
For every 13B and 30B, didn't tried 7B yet

llama -m weights/your_model.bin

I tried what you suggested after getting the latest build....
root@69f29a3e3123:/usr/src/app# llama -m weights/your_model.bin
main: seed =<PHONE_NUMBER>
llama_model_load: loading model from 'weights/your_model.bin' - please wait ...
llama_model_load: failed to open 'weights/your_model.bin'
llama_init_from_file: failed to load model
main: error: failed to load model 'weights/your_model.bin'

Hey everyone! Can you try to grab the latest main, rebuild the docker container and tell me if it's still happening?
In the case where it's still happening, can you run the following:
docker compose up -d
docker compose exec api bash
llama -m weights/your_model.bin

see if it loads & starts outputting stuff. If it doesn't please send me the traceback.

root@69f29a3e3123:/usr/src/app# llama -m weights/ggml-alpaca-7B-q4_0.bin
main: seed =<PHONE_NUMBER>
llama_model_load: loading model from 'weights/ggml-alpaca-7B-q4_0.bin' - please wait ...
llama_model_load: n_vocab = 32000
llama_model_load: n_ctx   = 512
llama_model_load: n_embd  = 4096
llama_model_load: n_mult  = 256
llama_model_load: n_head  = 32
llama_model_load: n_layer = 32
llama_model_load: n_rot   = 128
llama_model_load: f16     = 2
llama_model_load: n_ff    = 11008
llama_model_load: n_parts = 1
llama_model_load: ggml ctx size = 4529.34 MB
llama_model_load: memory_size =   512.00 MB, n_mem = 16384
llama_model_load: loading model part 1/1 from 'weights/ggml-alpaca-7B-q4_0.bin'
llama_model_load: ...................................Killed

main: seed =<PHONE_NUMBER>
llama_model_load: loading model from 'weights/ggml-alpaca-7B-q4_0.bin' - please wait ...
llama_model_load: n_vocab = 32000
llama_model_load: n_ctx   = 512
llama_model_load: n_embd  = 4096
llama_model_load: n_mult  = 256
llama_model_load: n_head  = 32
llama_model_load: n_layer = 32
llama_model_load: n_rot   = 128
llama_model_load: f16     = 2
llama_model_load: n_ff    = 11008
llama_model_load: n_parts = 1
llama_model_load: ggml ctx size = 4529.34 MB
llama_model_load: memory_size =   512.00 MB, n_mem = 16384
llama_model_load: loading model part 1/1 from 'weights/ggml-alpaca-7B-q4_0.bin'
llama_model_load: ...................Killed

Most of the time, the error is caused by low RAM or VRAM.
Using a research graphics card seems to be the answer.

Indeed low RAM is often the issue, but no need for graphics card on this repo. I updated the RAM requirements in the README, make sure you have enough free !

I'm getting this:
root@personal-gpt-tasks-546548ffbb-69c85:/app# llama -m /mnt/data/weights/ggml-alpaca-7B-q4_0.bin --n_parts 1
main: seed =<PHONE_NUMBER>
llama_model_load: loading model from '/mnt/data/weights/ggml-alpaca-7B-q4_0.bin' - please wait ...
llama_model_load: n_vocab = 32000
llama_model_load: n_ctx   = 512
llama_model_load: n_embd  = 4096
llama_model_load: n_mult  = 256
llama_model_load: n_head  = 32
llama_model_load: n_layer = 32
llama_model_load: n_rot   = 128
llama_model_load: f16     = 2
llama_model_load: n_ff    = 11008
llama_model_load: n_parts = 1
llama_model_load: type    = 1
Illegal instruction (core dumped)


Add hashes to bsindex file path

This makes use of xxhash's hash instead of creating recursive folders to determine where to put bsindex in. It might also fix a niche use case when converting a url to filename (e.g. < and > can be interchangeable)

I'm fairly content with recursive folders. Makes it easy to spot where something was from and do partial cleanups if desired.

Alright, thanks for your feedback.


__ow_path is blank in API created with response type http

We have created 2 APIs in same base path as shown below.

ibmcloud fn api create /v1/abc/ka /assets/upload/{aaid}/name/{name} get 'serverless-files/presign' --response-type http

ibmcloud fn api create /v1/abc/ka /metadata post 'ams-cloudant/validate_seq' --response-type http

For the 1st API with presign I am able to read __ow_path and its returning /v1/abc/ka/assets/upload/1234/name/test.pdf .
The same code I am using in 2nd API, but ow_path is blank..it should return /v1/abc/ka/metadata right?
May I know why ow_path is blank in 2nd case.

Did you use the serverless framework openwhisk plugin to create these APIs? From the description above, it looks like you are using the CLI. Support for OpenWhisk related issues, rather than the serverless framework plugin, needs to use one of the support channels for that project...

Stack Overflow
Slack
IBM Cloud Support

@PallaviPatil2
This should be fixed now, also fixed if you IBM Cloud Functions.
https://github.com/apache/incubator-openwhisk/issues/4055
Just update your apis with wsk api create or ibmcloud fn api create


Accept configuration file path as an argument

Request
It seems /etc/alloy/config.alloy is hardcoded and there is no way to pass a custom location.
Use case
I want to put configuration file in the same directory as Alloy instance as I don't have access to /etc/ directory.

Can you explain how you're running Alloy? The configuration is provided to the run command in the first argument, like the following.
./alloy run /another/directory/my_config.alloy

Just running the binary like ./alloy-linux-amd64


Fix: Reduce index HTML rendering lag

There is a really inefficient loop somewhere, I think around the token list that was hacked together, adds at least 1-2 seconds to page load times. Remove the delay.

Might need to make more stuff async.

A large chunk will be improved when we AJAX-ify the token search, instead of rendering all tokens on load. Tracking the work in SRM-69


add themeswitch in blog theme

Closes #115

Hey, this is a very naive implementation so I can get used to the packages and contrib workflow.
I'm clearly not a designer, so I'll need some help on defining some basic but solid variables for the blog "dark theme". In the last commit, I suggested some changes + extracted some .dark classes from the docs theme.
Happy to discuss all of this! Cheers ;)

well, that was quick :) A bit unfortunate, that the vercel / gitlab preview integration points to the site theme in this case.
But keep up the good work 💪

The new @tailwindcss/typography or tailwindcss is causing issue for me locally:

Seems all prose styles are missing.

Closing as completed https://github.com/shuding/nextra/pull/242


Multiple methods named 'initWithValue:' found with mismatched result, parameter type or attributes v10.50.1

How frequently does the bug occur?
-- select --
Description
Xcode show Multiple methods named 'initWithValue:' found with mismatched result, parameter type or attributes when built
Stacktrace & log output
No response
Can you reproduce the bug?
-- select --
Reproduction Steps
No response
Version
10.50.1
What Atlas Services are you using?
-- select --
Are you using encryption?
-- select --
Platform OS and version(s)
Xcode 14.1 ,iOS 16.1
Build environment
Xcode version: ...
Dependency manager and version: ...

I downgraded to 10.49.0


the ability to create access controls with the module

Is your request related to a problem?
By using create_origin_access_identity you could create access identities to use with your origin, but there is no way to create access control using the module.
Since Access identities have become legacy, I think we need a way to create access controls as well using the module
Describe the solution you'd like.
something like that would make sense
create_origin_access_control = true
origin_access_controls = {
    s3_bucket_one = {
      name                              = "example"
      description                       = "Example Policy"
      origin_access_control_origin_type = "s3"
      signing_behavior                  = "always"
      signing_protocol                  = "sigv4"
    }
 }

Seems the pull request #97 should address this issue.


[BUG] sc_install_defaults finds defunct Apache instances

Describe the bug
Continuation discussion from #100.
Namely:

Out of interest, why adopt this method instead of stepping through the QUSRSYS/QATMHINSTC file which will show only apache instances that are current? I often see a lot of **** in the www directory that is no longer used or relevant.

This issue tracks discussion around trying to fix this and process the QATMHINSTC entries (cc @AndyYouens )

A hint at how this can be done in bash...
cat AJSP.MBR | /usr/bin/iconv -f `attr AJSP.MBR ccsid` -t 819 | sed 's|.*-d ||g' | sed 's| .*||g'

According to @worksofbarry, something could also be done with rfile, like so:
/usr/bin/Rfile AJSP.MBR | sed 's|.*-d ||g' | sed 's| .*||g'


Verify renaming works with version control

Files need to be checked out and marked dirty, including metafiles in the case of Spritesheets with sprites.
If a file is locked or can't be checked out it should have the warning indicator.

This works for normal assets, which use Unity's Rename API, but I'm not sure it works on spritesheets, which use file IO to rename the meta file.


Update MaxQuant

This PR adds the "Match between runs" parameters as well as the quantification method for "Reporter ion MS3".

Thank you so much @npinter!!! It looks good to me.

Gracias!

This PR makes maxquant_mqpar fail, because it uses the same macros, e.g. NameMapper.NotFound: cannot find 'search_opts' . This became apparent in https://github.com/galaxyproteomics/tools-galaxyp/pull/575

@bernt-matthias the TS was not updated for the mqpar wrapper: https://toolshed.g2.bx.psu.edu/view/galaxyp/maxquant_mqpar/9cb7dcc07dae
And this is like it should be. The symlinking is not optimal, we should move them in one folder so test gets not triggered when we update only one tool. But at least the second tool is not updated in the TS - so it can not break. What do I miss?

TS is fine. Its just that planemo test fails for maxquant_mqpar since the change .. you can see this in the weekly CI and also in my PR.
+1 for moving the tools in one dir .. or https://github.com/galaxyproject/planemo/issues/1150


Update badges

Use SVG for build status badge.
Add a badge for gem version. Helpful for knowing what the latest version is, without having to go to rubygems.

Probably should have used a [skip ci] on these commits.


suggestion : <context.is_critical> for swords hits in "on player damaged entity"

All is in the title ^^
It might be very nice 😊

I don't believe there's any API for that, and I don't feel comfortable adding a manual check for it. You might be able to implement it somewhat easily in scripting, though. See this link for a general reference on how to accomplish it: https://www.spigotmc.org/threads/check-if-a-hit-is-a-critical-hit.187105/


updating to always run vitest with coverage

Please check if the PR fulfills these requirements

[x] Followed the Contributing guidelines.
[x] Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
updating to always run vitest with coverage

Codecov Report

Merging #605 (cf6d17c) into master (2eca5a0) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##            master      #605   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          292       292
  Branches        75        75
=========================================
  Hits           292       292


Extension doesn't work

When trying to open a file using Nano I get the following error:

Any help with this?

@marek-stoj Thanks for the feedback!
That's because I couldn't tested the extension on Windows. I've just updated it to support PowerShell syntax. Could you please upgrade to v0.0.3 and check if things are working? Note that you had to install Vi/Vim/Nano beforehand.

@marek-stoj Any updates on this?

I've tested the new version. There is still an issue on Windows but I know
how to fix it. I will create a PR tomorrow.
On Tue, Oct 25, 2022 at 2:38 PM Babak K. Shandiz @.***>
wrote:

@marek-stoj https://github.com/marek-stoj Any updates on this?
—
Reply to this email directly, view it on GitHub
https://github.com/babakks/vscode-open-with-vim/issues/1#issuecomment-1290483067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPMISQCDDCVO7PWJF7A6LWE7IFBANCNFSM6AAAAAARE42VYQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Just published the new version 0.0.4. A few minutes from now it'll be accessible for the upgrade through the extensions sidebar.


Rename schema file to web-features.schema.json

For lack of a better name.
The --id defs bit isn't needed now that there's a single schema file.

The idea is that this would help with https://github.com/web-platform-dx/web-features/issues/715. We can publish web-features.schema.json in both the NPM package and the GitHub release. The name of the data files are different (index.json vs. data.json) so matching either of those names would still leave a mismatch in the other place.

I've sent https://github.com/web-platform-dx/web-features/pull/1470, and if we do that I think we should call this data.schema.json instead.

I've documented the import as import schema from "web-features/data.schema.json" with { type: "json" } and will merge this now.


[551] Add programme end date to programme detail section

Context
Add the programme end date field within the programme details page.
Changes proposed in this pull request

Added a model, view and controller for the programme_end_date
Added a feature test, and unit tests

Guidance to review

Pull down the branch and boot up the server
Create a trainee
Under programme details, fill in all the sections apart from programme end date
Click continue and check that it throws an error
Enter a valid date and click submit
Check that the date you entered is surfaced on the confirmation page

 Add programme detail end date to programme detail section

Nice one @avinhurry for getting this hooked up and working with very little input from me!
I haven't reviewed the code yet, but there might be places we can refactor/tidy up together - I'll give you a nudge and we can talk through on a call.
Great perseverance 💪


[5.1] Allow to use . inside session key when explicit = true

If you want to set as session key string with dot, Laravel automatically assumes it's dot notation and it will save key as array. When Session::all() is used, you won't get valid key for value you put into session. Adding 3rd parameter $explicit will allow to manually force to save key as string key without using dot notation

Holding off on this for now.

@taylorotwell Any change it will be added in future? I was inspired by this thread http://stackoverflow.com/questions/34233640/trouble-with-laravels-dot-notation-when-using-sessions


Style models update

Related Ticket(s)
#1384
Description

Simplification of style models from 9 to 3
Reduced content from 2 pages (2 tabs) to just 1 page no tab

Changelog
New

Guidelines / style-models

Changed

Data / nav-items.yaml

Removed

Guidelines / Style models / overview
Guidelines / Style models / style-model-list
And associated images

Ready for another look @janchild


setting serial speed resets target

the Vendor2 command (0x82) which is used to set the line speed also sends a break:
https://github.com/microbit-foundation/DAPLink/blob/master/source/daplink/cmsis-dap/DAP_vendor.c#L93
this resets the target:
https://github.com/microbit-foundation/DAPLink/blob/master/source/daplink/usb2uart/usbd_user_cdc_acm.c#L130
I don't thinks v1 was doing this, and I think this is why makecode resets the board twice while flashing over WebUSB.
cc @jaustin

Right, so the code was introduced in version 0251 (https://github.com/microbit-foundation/DAPLink-microbit/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc) and we shipped v1.5 with 0249.
However we've been putting the newer versions on the website to upgrade not long after each release, so we have users with these versions.
I can replicate the issue in a v1 with 0254 with the current MakeCode beta:
makecode.microbit.org version:  3.0.28
Microsoft MakeCode version:  6.0.24
microbit runtime version:  v2.2.0-rc6

Do we think it's the right behavior? It seems odd for baud setting to reset board.

Yeah, doesn't sound right to reset the target on a baud rate change. We should raise this issue upstream and we can ask Arm if this was an intentional change.
In the meantime the new factory image shipped contains this, and if we want to update the DAPLink factory image I would prefer to do that with an official 0256 release (we don't want to have two different 0255 images in the wild), which might take time since we need to submit the PR upstream and go through the review process to get it merged. So that's not going to be quick.

Related issue upstream (thanks for the find @gerargz): https://github.com/ARMmbed/DAPLink/issues/613

I think this is orthogonal to the issue that setting baud rate sends break - i don't see why it should


Travis CI macOS Builds are failing

It appears something has changed, possibly external, and all of the macOS builds are failing (including nightly)
./installpsh-osx.sh: line 99: [: missing `]'

\n*** Installing PowerShell Core for osx...

Howebrew is already installed, skipping...

Installing cask...

Updating Homebrew...

==> Downloading https://homebrew.bintray.com/bottles-portable/portable-ruby-2.3.3.leopard_64.bottle.1.tar.gz

######################################################################## 100.0%

==> Pouring portable-ruby-2.3.3.leopard_64.bottle.1.tar.gz

/usr/local/Homebrew/Library/Homebrew/brew.rb:12:in `<main>': Homebrew must be run under Ruby 2.3! (RuntimeError)

ERROR: Cask failed to install! Cannot install powershell...

The command "./install-powershell.sh" failed and exited with 2 during .

examples:

https://travis-ci.org/PowerShell/PowerShell/jobs/285387333
https://travis-ci.org/PowerShell/PowerShell/jobs/285471140
https://travis-ci.org/PowerShell/PowerShell/jobs/285293168
https://travis-ci.org/PowerShell/PowerShell/jobs/285388460 (nightly)

Also, the nightly badge for Travis CI macOS reports success when it clearly failed.

I'm looking into this.  The badge caching is also still an issue.

+1

Hello,
How did you fix this? Our project has same issue.

@benapetr Homebrew always uses the system Ruby, so it likely started with Ruby 2.0.0, installed the vendored Ruby 2.3.3, but then didn't use the vendored Ruby the first time.
In the interim and until this is specifically addressed by Homebrew or Travis CI, I fixed this by passing the no-auto-update flag:
HOMEBREW_NO_AUTO_UPDATE=1 brew install ...

@benapetr the main part of the fix was to move brew update earlier before using brew

oh, I tried this from your patch in this PR, but maybe brew update would be better option: https://github.com/huggle/huggle3-qt-lx/commit/95267bbff361e11212519a609d8c832a704efb56


Broken Lockfile Action

📰 Custom Issue

Lockfile refresher action failed on the last two attempts:
https://github.com/SciTools/iris/actions/workflows/refresh-lockfiles.yml

conda-incubator/conda-lock#202

😬

Update - the following change to refresh-lockfiles.yml fixes the problem:
@@ -52,11 +52,13 @@ jobs:

     steps:
       - uses: actions/checkout@v3
-      - name: install conda-lock
+      - name: install requirements
         run: |
           source $CONDA/bin/activate base
-          conda install -y -c conda-forge conda-lock
+          conda install -y -c conda-forge conda-libmamba-solver conda-lock
       - name: generate lockfile
+        env:
+          CONDA_EXPERIMENTAL_SOLVER: libmamba
         run: |
           $CONDA/bin/conda-lock lock -k explicit -p linux-64 -f requirements/ci/${{matrix.python}}.yml
           mv conda-linux-64.lock ${{matrix.python}}-linux-64.lock

Unfortunately we also need to be able to refresh the lock-files locally (see update_lockfiles.py), and using conda-libmamba-solver is a problem if the Conda base environment is read-only (e.g. for institutional installs). I next plan to investigate whether one can temporarily change what environment is the base environment, but probably not today.


1.11.beta: Fixed Wing Flaps PWM output only on ch6, but not on ch7

On my FW with 1.10.1 I have the flaps on channel 6 & 7, works fine.
Not with 1.11.beta1. Only flap on channel 6 is moving. Flap on Channel 7 is dead.
Reverted back to 1.10.1 on the same pixhawk, both Flaps working again.
QGC is on 4.0.6
HW: pixhawk4
Please let me know If you need futher info
Mixer file is on ./microsd/etc/mixers
### AAERTWF.main.mix
#### Skywalker 2000
##  10.4 16:30
###
#### servo 1 Quer rechts
M: 1
O:      10000  10000      0 -10000  10000
S: 0 0  10000  10000      0 -10000  10000
## S: 0 6  10000  10000      0 -10000  10000
#### servo2 Quer links
M: 1
O:      10000 10000    0 -10000  10000
S: 0 0  10000  10000   0 -10000  10000
### S: 0 6 -10000 -10000     0 -10000  10000
### servo 3 Hoehe
M: 1
O:       10000   10000      0 -10000  10000
S: 0 1  -10000  -10000      0 -10000  10000
### servo4 Seite
M: 1
O:      10000  10000      0 -10000  10000
S: 0 2  10000  10000      0 -10000  10000
### servo 5 Motor
M: 1
O:      10000  10000      0 -10000  10000
S: 0 3      0  20000 -10000 -10000  10000
#####
##### zusaetze
### servo 6 flaps links
M: 1
O:      -10000  -10000      0 -10000  10000
S: 0 4  0  20000  -10000 -10000  10000
###
### servo 7 Flaps rechts
M: 1
O:      10000  10000      0 -10000  10000
S: 0 4  0  20000     -10000 -10000  10000
##### s8
M: 1
O:      10000  10000      0 -10000  10000
S: 0 4  10000  10000      0 -10000  10000
#####
M: 1
O:      10000  10000      0 -10000  10000
S: 0 4  10000  10000      0 -10000  10000

@jstrebel could you share a log from this? Even from some bench testing would be useful, so just arm the vehicle on the ground and apply the flaps. Thanks!

the first is the log 1.10.1
https://review.px4.io/plot_app?log=23e498af-9db1-4794-9464-35c3a2bd529e
this is the log with to 1.11.beta.
Hope this helps. ( I did downgrade to 1.10.1 und upgrade to 1.11.beta several times. NO change with the hardware nor parameter changes

Thanks for the logs. One thing I find weird is that there is also some output on the AUX, even though the aux mixer is set to pass. Not sure if that's related though.

@jstrebel could you maybe run a test with all the other stuff beside the settings for the Flap channels removed in the mixer, so only:
servo 6 flaps links
M: 1
O: -10000 -10000 0 -10000 10000
S: 0 4 0 20000 -10000 -10000 10000
servo 7 Flaps rechts
M: 1
O: 10000 10000 0 -10000 10000
S: 0 4 0 20000 -10000 -10000 10000
?

@jstrebel I would test removing all comment lines. Multi-line comments and alike things are not officially supported in mixers.

@LorenzMeier, I deleted the comment lines. No change in behavior.
@sfuhrer If I put only the Flap entries in the mixer file, then as expected the two ailerons (channel 1 and 2) move as flaps in the correct direction.

If I put the two entries of the ailerons as the first two entries, then the ailerons work correct. The Flaps move now at channel 3 and 4 (yaw and elevon)
In case I go back to the complete mixer File with all channels then the Flaps problem is back.
I do not see any strange character in the mixer file.
Strange to me is the identical mixer file works correct with 1.10.1 but not in 1.11.beta. Should I upload the mixer file?

@sfuhrer, did you get the required info you requested from me?
Is there a reason that this issue is not marked as a bug?

@dagar do you have any idea what could have changed on the mixer level lately?

I made some further experiments. If I put the following mixer file on the sd card then I get:
AAERTWF.main.mix
Skywalker 2000
10.4 16:30
servo 1 Quer rechts
M: 1
O:      10000  10000      0 -10000  10000
S: 0 0  10000  10000      0 -10000  10000
servo2 Quer links
M: 1
O:      10000 10000    0 -10000  10000
S: 0 0  10000  10000   0 -10000  10000
servo 3 Hoehe
M: 1
O:       10000   10000      0 -10000  10000
S: 0 1  -10000  -10000      0 -10000  10000
servo4 Seite
M: 1
O:      10000  10000      0 -10000  10000
S: 0 2  10000  10000      0 -10000  10000
servo 6 flaps links
M: 1
O:      -10000  -10000      0 -10000  10000
S: 0 4  0  20000  -10000 -10000  10000
servo 7 Flaps rechts
M: 1
O:      10000  10000      0 -10000  10000
S: 0 4  0  20000  -10000 -10000  10000
Servo 1&2 roll working correct
Servo 3&4 pitch and yaw woring correct
Servo 5&6 Flaps working correct
Test with the test mixer file:
If I put duplicate now channel 4 (channel 4&5 now the same mixer setting. Servo 5 moves like servo 4, as expected)
Flap on channel 6 work ok but Flap on channel 7 does not move.
AAERTWF.main.mix
Skywalker 2000
10.4 16:30
servo 1 Quer rechts
M: 1
O:      10000  10000      0 -10000  10000
S: 0 0  10000  10000      0 -10000  10000
servo2 Quer links
M: 1
O:      10000 10000    0 -10000  10000
S: 0 0  10000  10000   0 -10000  10000
servo 3 Hoehe
M: 1
O:       10000   10000      0 -10000  10000
S: 0 1  -10000  -10000      0 -10000  10000
servo4 Seite
M: 1
O:      10000  10000      0 -10000  10000
S: 0 2  10000  10000      0 -10000  10000
servo4 Seite
M: 1
O:      10000  10000      0 -10000  10000
S: 0 2  10000  10000      0 -10000  10000
servo 6 flaps links
M: 1
O:      -10000  -10000      0 -10000  10000
S: 0 4  0  20000  -10000 -10000  10000
servo 7 Flaps rechts
M: 1
O:      10000  10000      0 -10000  10000
S: 0 4  0  20000  -10000 -10000  10000

Observation: If I remove the sd card, forcing px4 starts with the built in mixer (Bormatec Maya) channel 4 Yaw and channel 6 Flaps show the same value if I do pwm info.

Observation: If I remove the sd card, forcing px4 starts with the built in mixer (Bormatec Maya) channel 4 Yaw and channel 6  in this case wheel ( show the same value if I do pwm info). There is no pwm output value on channel 7

@jstrebel Really weird. I assume that you are using IO for pwm, correct?

Correct I/O PWM OUT on the pixhawk4. Its really weird. Going back to 1.10.1 everything works again.

@jstrebel Can you try to use the pwm test command on that channel?

I did the following test with the standard Bormatec mixer no SD CARD inserted
pwm test -c 6 -p 1000 and 1800
pwm test -c 7 -p 1000 and 1800
Result: Both Flap Servo move

I’ll load up a fresh plane with 11 beta (and the Albatross standard mixer) and report back the results on the flaps. We have flaps on ch 7 and 8.

Just did another test putting my mixer file back to the SD Card.
pwm test -c 6 -p 1000 (and 1800)
pwm test -c 7 -p 1000 (and 1800)
Result: Both Flap Servos move
But stil no change via RC: only Flap on Ch 6 move

I loaded up 11 beta 1 and tested as well. We have 2 channel flap output, main 7 and 8 but triggered by RC channel 9 when done manually. It also doesn't work in our case. Looking at it a little closer, we have no output on anything from output 6 and up (servo_output_raw) shows this as well.
Taking from above, running pwm test -c 7 -p 1000 and we have movement but otherwise output shows 0 us.

Hi @jstrebel and @ryanjAA.  If you add the following to your airframe config files does it change the behavior?
set PWM_OUT 12345678

@mcsauder : No change

@jstrebel can confirm with his tests, but this change appears to have made this issue show up.
Hi @bkueng, you might be able to explain what the intention of this change was and if there is a better way to address the current behavior observed in this issue.
(@jstrebel , in the meantime, if you want, you can explicitly place set PWM_OUT fmu in your aircraft config file as well and that should take care of things without modifying the code.)

Hi @dagar , you might be able to explain what the intention of this change was and if there is a better way to address the current behavior observed in this issue.

I don't think I'm following the issue here. That change shouldn't have been more than a rename, is that not the case?
I'll take this issue and try to reproduce.

@dagar, happy to help.

@dagar , this issue appears fixed in current master but not in v1.11.0-beta1 as reported back by @jstrebel .... I'll poke around on this a bit more tomorrow.

Just tested it on Beta2. Its ok now


Feature analytics

replaces #19 since the rebase completely screwed it up.
"Adding tool to generate PDF reports with statistics using matplotlib"

Yeah; this was basically just a copy/paste from the original PR. We can improve it - I just wanted to get it in since we’ve had the PR open for over a week and “it works”. 😄
Will add to project board.


Adding exepath field for trace_exec image-based gadget

Adding exepath field for trace_exec image based gadget
How to use
Should specify the --paths flag similar to the built-in gadget

Formatting
Need to run make clang-format
Optimize support for optional big fields
Ideally, we would want to implement #3491.
TestClose (/proc/kallsyms)
This error about leaking the file descriptor to /proc/kallsyms does not seem related to the change:

=== RUN   TestClose
time="2024-11-13T09:59:41Z" level=warning msg="failed to get cgroups info of container a19b50ac744fd448d29b756583c85e52d2166579345d30c76084c26db0aaf389 from /proc/36577/cgroup: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:42Z" level=error msg="cgroup enricher: failed to get cgroup paths on container a19b50ac744fd448d29b756583c85e52d2166579345d30c76084c26db0aaf389: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:42Z" level=error msg="cgroup enricher: failed to get cgroup paths on container 55a90965043e19d63fec32ed480ef1c15a5d8cc66eef85c910f184d346f2f842: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:42Z" level=warning msg="failed to get cgroups info of container a19b50ac744fd448d29b756583c85e52d2166579345d30c76084c26db0aaf389 from /proc/36577/cgroup: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:42Z" level=error msg="cgroup enricher: failed to get cgroup paths on container a19b50ac744fd448d29b756583c85e52d2166579345d30c76084c26db0aaf389: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:43Z" level=warning msg="failed to get cgroups info of container a19b50ac744fd448d29b756583c85e52d2166579345d30c76084c26db0aaf389 from /proc/36577/cgroup: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:43Z" level=error msg="cgroup enricher: failed to get cgroup paths on container a19b50ac744fd448d29b756583c85e52d2166579345d30c76084c26db0aaf389: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:43Z" level=error msg="cgroup enricher: failed to get cgroup paths on container 07b1e660fdb7830628aa9ed0d57d09fcef984b24480d04fce22eaf0434c74ba4: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:44Z" level=warning msg="failed to get cgroups info of container a19b50ac744fd448d29b756583c85e52d2166579345d30c76084c26db0aaf389 from /proc/36577/cgroup: host.Init() must be called before calling isHostNamespace()"
time="2024-11-13T09:59:44Z" level=error msg="cgroup enricher: failed to get cgroup paths on container a19b50ac744fd448d29b756583c85e52d2166579345d30c76084c26db0aaf389: host.Init() must be called before calling isHostNamespace()"
    ig-manager_test.go:119: After 20 attempts, fd leaked:
        0: /dev/null
        1: pipe:[151270]
        2: pipe:[151270]
        3: socket:[152017]
        4: anon_inode:[eventpoll]
        5: anon_inode:[eventfd]
        8: /proc/kallsyms

        0: /dev/null
        1: pipe:[151270]
        2: pipe:[151270]
        3: socket:[152017]
        4: anon_inode:[eventpoll]
        5: anon_inode:[eventfd]
--- FAIL: TestClose (4.88s)

TestSnapshotSocket
This does not seem related to this PR:

=== RUN   TestSnapshotSocket
    utils.go:220: Seed used:<PHONE_NUMBER>562380504
    command.go:92: Run command(Create test namespace):
    command.go:94: Command returned(Create test namespace):

        namespace/test-snapshot-socket-501920[96](https://github.com/inspektor-gadget/inspektor-gadget/actions/runs/11794783154/job/32916881541?pr=3695#step:9:97)01936253251 created
        NAME      SECRETS   AGE
        default   0         1s

    command.go:111: Start command(Run test-snapshot-socket)
    command.go:92: Run command(SleepForSeconds):
    command.go:94: Command returned(SleepForSeconds):

    command.go:92: Run command(WaitForTestPod):
    command.go:94: Command returned(WaitForTestPod):

        pod/test-snapshot-socket condition met

    command.go:128: Stop command(Run test-snapshot-socket)
    command.go:130: Command returned(Run test-snapshot-socket):

        pod/test-snapshot-socket created

    command.go:92: Run command(Run_snapshot_socket):
    command.go:94: Command returned(Run_snapshot_socket):
        time="2024-11-13T10:40:20Z" level=info msg="Experimental features enabled"
        time="2024-11-13T10:40:21Z" level=warning msg="minikube-containerd  | image signature verification is disabled due to using corresponding option"
        time="2024-11-13T10:40:22Z" level=warning msg="minikube-containerd-m02 | image signature verification is disabled due to using corresponding option"

    match.go:175: output doesn't contain the expected entry
        captured:
        expected:
        {"runtime":{"containerId":"1fcade8d9056637be546a7a59d7e3a8e3fe9daab82e582e4c5d9bc501c6e9eb2","containerImageName":"docker.io/library/busybox:latest"},"k8s":{"namespace":"test-snapshot-socket-5019209601936253251","podName":"test-snapshot-socket","containerName":"test-snapshot-socket","owner":{}},"netns_id":1,"ino":1,"src":{"addr":"<IP_ADDRESS>","version":4,"port":9090,"proto":"TCP","k8s":{"kind":"","name":"","namespace":"","labels":""}},"dst":{"addr":"<IP_ADDRESS>","version":4,"port":0,"proto":"TCP","k8s":{"kind":"","name":"","namespace":"","labels":""}},"status":0}


Remove use of deprecated typedef from std::allocator

https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2023/p2868r1.pdf

We uniformly use allocator_traits::is_always_equal::value so no change is needed.


Closure Library in TypeScript

Is there any work or chance of development of Closure library in Typescript(typescript definition file).
Closure library with closure compiler works awesome, but there are some problems

Way too Verbose(JS Doc based typing)
No Intellisense Support.

I believe if typescript is used then

Type check will be handled by Typescript
Intellisense is already supported in TS so probably, this won't be the issue(well I only believe this part is possible.)

So is there any work going on for this or any plan for this port?

I assume that you talk about using the closure library in a project that uses TypeScript and for this you'd like to have this TypeScript definition file so that your code completion tool knows what type parameters are and functions return.
In that case:

I don't get why JSDoc is too verbose since you don't have to write any JSDoc. Furthermore I don't think JSDoc is too verbose since most of it is just the description of each parameter.

Type checking would be handled by TypeScript if you would call a function from the closure library. That would be the case for your project I guess.

I don't know this Intellisense code completion tool but my guess would be that Intellisense supports TypeScript and not the other way around. You can probably write an extension for Intellisense so that it supports JSDoc if you wish.

You can write one if you need one and maybe even make a PR.
It doesn't seem useful to anyone who doesn't write a project in TypeScript and would like to use the closure compiler.
You could probably even write a tool that generates one. Have fun!

Hi,
Thanks for the comments.

Consider the scenario, you are using closure library to write a web app, in one of the function parameter you have passed a parameter, say list of project, but the whole qualified namespace would be xxx.yyy.zzz.aaa.bbb.project. So our param annotation would be

@param {Array.<xxx.yyy.zzz.aaa.bbb.project>}
using typescript it would be something like
import {Project} from xxx.yyy.zzz.aaa.bbb and just use
function(param:project){} and you are good to go and if there are multiple module, we wouldn't need to write so much packages
I just started using closure, so probably it will take me some time(probably a lot), so if some of the older contributors who knows codebase better will probably make it sooner. So, if they probably see it and decide it to be part of next release, probably will make the library a lot easier to use.

Na you can simplify too:
// ==ClosureCompiler==
// @compilation_level ADVANCED_OPTIMIZATIONS
// @output_file_name default.js
// @use_closure_library true
// ==/ClosureCompiler==

goog.module('foo');

const StringBuffer = goog.require('goog.string.StringBuffer');

/**
 * Print the provided buffer.
 * @param {!StringBuffer} buffer
 */
function printBuffer(buffer) {
	console.log(buffer.toString());
}

printBuffer(new StringBuffer('foo', 'bar'));
printBuffer(1);  // This will throw a warning.

As far as I understand writing a TypeScript definition file is all about knowing the types and you know them just by looking at the annotations. You don't need to know the codebase for that.

As @mashedcode points out, if we rewrote the library in TS then non-TS users would have a harder time using it.  On the other hand, if a .d.ts file is sufficient, you might look into Clutz, which (as I understand it) extracts .d.ts files out of closure-annotated .js files.
If you get it working, feel free to contribute any wisdom (in the form of a wiki article or a script, whatever) so that others may benefit from this!

For further reference note that there's teppeis/closure-library.d.ts generated by teppeis/closure-ts.

There are also type definitions in this fork of DefinitelyTyped fivetran/DefinitelyTyped.

For reference, there now is also a active fork of an up-to-date version of the Closure library that has

Typescript definition (.d.ts) files
uses ES6 code instead of the custom goog.require, goog.define etc:

https://github.com/DreierF/ts-closure-library


[Bug]: Game Cards are not shown

Give a brief about the bug ✍
No Game card is visible on home page

What is the expected behavior? 🤔
A List  of games mut be displayed.
Provide step by step information reproduce the bug 📄
Description
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @divyansh-2005 , You can't have another issue before completing the previous one 😀
you already have the following 1 open issues 👀 ! :
#4442

Hey @divyansh-2005 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹


Pane movement breaks showCloseButton=never

Windows Terminal version
1.17.11461.0
Windows build number
Version 10.0.19045.3324
Other Software
No response
Steps to reproduce
Add the following to settings.json (in %LOCALAPPDATA%\Packages\Microsoft.WindowsTerminal_8wekyb3d8bbwe\LocalState\settings.json):
"theme": "foo",
"themes": [
  {
    "name": "foo",
    "tab": {
      "showCloseButton": "never"
    }
  }
]

Run in Command Prompt:
wt -M -p "Ubuntu-20.04" ; ^
	split-pane -p "Ubuntu-20.04" ;

Observe that the first tab has an "X" on the right that you can click to close the tab. But, tab.showCloseButton: never should have hidden this "X". The split-pane line is causing this bug. If you remove it, i.e.
wt -M -p "Ubuntu-20.04" ;

then the tab no longer has the "X", as expected
For related discussion:

https://github.com/microsoft/terminal/issues/3335
https://github.com/microsoft/terminal/pull/13348

Expected Behavior
The "X" icon should not be present on a Windows Terminal tab if tab.showCloseButton: never
Actual Behavior
The "X" icon is present on a tab if using split-pane in wt
Edit (lhecker): Any movement between two panes makes the buttons reappear.

Thanks for filing! We found a more general issue with showCloseButton thanks to you :smile:

Oh well that'll do it:

TerminalPage::_updateAllTabCloseButtons is where we apply TabViewItem().IsClosable based off the theme
Presumably, TerminalTab::_RecalculateAndApplyReadOnly fires later, which then updates the tab's own item's IsClosable based on read-only state only.


Why my code get better f1_score on older versions

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): No
OS Platform and Distribution:  Linux Ubuntu 16.04
TensorFlow installed from binary
TensorFlow version (use command below): 2.10.0 , 2.7.0,
Python version: 3.7.10
GPU model and memory: TITAN V

Describe the problem.
Hi, I adopt different Tensorflow and Keras versions with my code. But it returns a better f_1 Score on older versions.
To avoid the effect of random seeds, I try to set the random seed using this approach.
https://github.com/keras-team/keras/issues/2743#issuecomment-904169662
import random
random.seed(2022)
np.random.seed(2022)
tf.random.set_seed(2022)

I also added shuffle = False in model.fit()
Describe the current behavior.
The reproduction results are as follows:

Tensorflow
Keras
F1 Score

2.10.0
2.10.0
0.5369807497467073

2.7.0
2.7.0
0.29846938775510207

2.4.0
2.4.0
0.7376928728875827

Describe the expected behavior.
Why the new version has worse results? The latest version of the score should be better.
Standalone code to reproduce the issue.
I post the code on colad.
click me to reproduce
click me to download dataset

https://github.com/tensorflow/tensorflow/issues/58431

@DSOTM-pf,
I tried to execute the mentioned code with the dataset attached and was able to fetch the better f1-score than the older versions. Kindly find the gist of it here and let us know if anything required here.

Hi, I noticed that your execution achieved a 1.0 F1 score. But when I execute this code with tf-nightly, it returns 0.5369807497467073. I only uploaded the dataset on colab and modified the folder to '***.csv'.
May I know what version of TensorFlow you are using when running, and whether the lower version also has a 1.0 f1 score?

@DSOTM-pf,
I tried to execute the given code again on tf-nightly and I was still able to get the same f1 1.0. The tf-nightly version which I am using is 2.12.0-dev20221110. Kindly find the gist of it here. Thank you!

@tilakrayal I achieve the same @DSOTM-pf  f1_score:  0.5369807497467073 on your colab gist with today tf-nightly-2.12.0.dev20221111.


enhancement Provide default environment variables if it is unset

Proposed changes
Issue Number: close #19044
Problem summary
BE with java support needs some environment variables to start up and we have set them up in the start scripts (such as start_be.sh). However, if we debug BE directly, these environment variables are not set by default. For the convenience, we should provide default environment variables under this scenario.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.96 seconds
stream load tsv:          430 seconds loaded<PHONE_NUMBER>9 Bytes, about 165 MB/s
stream load json:         25 seconds loaded<PHONE_NUMBER> Bytes, about 89 MB/s
stream load orc:          59 seconds loaded<PHONE_NUMBER> Bytes, about 17 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230425064630_clickbench_pr_134596.html

run p0

run p0

run buildall

run p0


Text editing RTL

Text fields need to have a bidi cursor that shows what direction they are going in.

Specifically:

[ ] It should be possible to tap between LTR text and RTL text on both ends. For example in "3WERBEH english2 1WERBEH", you should be able to tap on either end of the english text.
[ ] On Android, when a cursor is between text of different directions, it should show the cursor in both locations. The side that is near the LTR text should be a half-cursor on the bottom, the part that is near the RTL text should be a half-cursor on the top.

Any updates on RTL support for widgets? I'm writing an app that displays in both arabic & english.

rtl is basically done, this is one of the few remaining issues.

@Hixie Do you know where you got the idea of 2 cursors at a bi-di boundary? Is this still the behavior that we want for Flutter?
@LongCatIsLooong Suggested putting the cursor under the character that would be deleted with a backspace. Maybe that's unconventional, though.

It's what Android did when I filed the bug. I don't know if they still do it.
Our text fields should be identical to the platform text fields (at least by default, but in general I think these are complicated enough that we shouldn't further complicate them by adding our own twist on these behaviours, just implementing all the platforms' default behaviours is confusing enough as it is).

I agree about keeping the same behavior as the platform. I think I can close this issue unless you see anything we're still missing.
It looks like native Android no longer has 2 cursors for bi-di (this is Keep):

Also it seems like it's possible to place the cursor wherever I tap in bi-di text in Flutter now:
Screencast from 2024-03-22 10-49-39.webm

To be clear, Flutter still has problems with RTL text editing, but those are tracked in other issues.


Auth design

Overview

Add color variables (also green-gradient)
Change login and sing up design
Change font to Work Sans (the one we chose initially)

Notes

Button needs to be changed when we have the last version.
I wasn't able to add the icons in the placeholder (check Figma on Login page)

Screenshots

 Style auth pages

Should we make the font for not registered yet, sign up  a bit larger to immediately draw the potential user's attention?


Reshaping trivial dimensions with non-trivial chunks fails

When trying to use reshape on a Dask Array with some trivial (i.e. 0-length) dimensions and non-trivial chunks (i.e. multiple chunks) along non-trivial dimensions, the reshape operation fails. This differs from NumPy in the same situation. It seems to have something to do with how rechunking occurs as a consequence of reshaping. An example is provided in the code snippet below along with environment details.
Code snippet:

In [1]: import numpy as np

In [2]: import dask.array as da

In [3]: a = np.ones((2, 0, 0))

In [4]: d = da.from_array(a, (1, 0, 0))

In [5]: a.reshape((len(a), -1))
Out[5]: array([], shape=(2, 0), dtype=float64)

In [6]: d.reshape((len(d), -1))
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-6-4a136cd07540> in <module>()
----> 1 d.reshape((len(d), -1))

/zopt/conda2/envs/test/lib/python2.7/site-packages/dask/array/core.pyc in reshape(self, *shape)
   1299         if len(shape) == 1 and not isinstance(shape[0], Number):
   1300             shape = shape[0]
-> 1301         return reshape(self, shape)
   1302
   1303     @wraps(topk)

/zopt/conda2/envs/test/lib/python2.7/site-packages/dask/array/reshape.pyc in reshape(x, shape)
    178
    179     # Logic for how to rechunk
--> 180     inchunks, outchunks = reshape_rechunk(x.shape, shape, x.chunks)
    181     x2 = x.rechunk(inchunks)
    182

/zopt/conda2/envs/test/lib/python2.7/site-packages/dask/array/reshape.pyc in reshape_rechunk(inshape, outshape, inchunks)
     40             while ileft >= 0 and reduce(mul, inshape[ileft:ii + 1]) < dout: # 4 < 64, 4*4 < 64, 4*4*4 == 64
     41                 ileft -= 1
---> 42             if reduce(mul, inshape[ileft:ii + 1]) != dout:
     43                 raise ValueError("Shapes not compatible")
     44

TypeError: reduce() of empty sequence with no initial value

Environment:

name: test
channels:
- conda-forge
- defaults
dependencies:
- appnope=0.1.0=py27_0
- asn1crypto=0.22.0=py27_0
- backports.shutil_get_terminal_size=1.0.0=py27_1
- backports_abc=0.5=py27_0
- bkcharts=0.2=py27_0
- blas=1.1=openblas
- bokeh=0.12.6=py27_0
- ca-certificates=2017.4.17=0
- certifi=2017.4.17=py27_0
- cffi=1.10.0=py27_0
- chardet=3.0.2=py27_1
- click=6.7=py27_0
- cloudpickle=0.3.1=py27_0
- cryptography=1.9=py27_0
- cycler=0.10.0=py27_0
- dask=0.15.1=py27_0
- decorator=4.1.2=py27_0
- distributed=1.18.0=py27_0
- enum34=1.1.6=py27_1
- freetype=2.7=1
- functools32=<IP_ADDRESS>=py27_1
- futures=3.0.5=py27_0
- heapdict=1.0.0=py27_0
- idna=2.5=py27_0
- ipaddress=1.0.18=py27_0
- ipython=5.4.1=py27_0
- ipython_genutils=0.2.0=py27_0
- jinja2=2.9.5=py27_0
- libffi=3.2.1=3
- libgfortran=3.0.0=0
- libpng=1.6.28=0
- locket=0.2.0=py27_1
- markupsafe=1.0=py27_0
- matplotlib=2.0.2=np113py27_0
- msgpack-python=0.4.8=py27_0
- ncurses=5.9=10
- numpy=1.13.1=py27_blas_openblas_201
- openblas=0.2.20=1
- openssl=1.0.2l=0
- pandas=0.20.3=py27_1
- partd=0.3.8=py27_0
- pathlib2=2.3.0=py27_0
- pexpect=4.2.1=py27_0
- pickleshare=0.7.3=py27_0
- pip=9.0.1=py27_0
- prompt_toolkit=1.0.14=py27_0
- psutil=5.2.1=py27_0
- ptyprocess=0.5.2=py27_0
- pycparser=2.18=py27_0
- pygments=2.2.0=py27_0
- pyopenssl=16.2.0=py27_0
- pyparsing=2.2.0=py27_0
- pysocks=1.6.7=py27_0
- python=2.7.13=1
- python-dateutil=2.6.1=py27_0
- pytz=2017.2=py27_0
- pyyaml=3.12=py27_1
- readline=6.2=0
- requests=2.18.2=py27_0
- scandir=1.5=py27_1
- setuptools=36.2.2=py27_0
- simplegeneric=0.8.1=py27_0
- singledispatch=<IP_ADDRESS>=py27_0
- six=1.10.0=py27_1
- sortedcontainers=1.5.3=py27_0
- sqlite=3.13.0=1
- ssl_match_hostname=<IP_ADDRESS>=py27_1
- subprocess32=3.2.7=py27_0
- tblib=1.3.2=py27_0
- tk=8.5.19=1
- toolz=0.8.2=py27_0
- tornado=4.5.1=py27_0
- traitlets=4.3.2=py27_0
- urllib3=1.21.1=py27_1
- wcwidth=0.1.7=py27_0
- wheel=0.29.0=py27_0
- yaml=0.1.6=0
- zict=0.1.2=py27_0
- zlib=1.2.11=0
- pip:
  - backports-abc==0.5
  - backports.shutil-get-terminal-size==1.0.0
  - backports.ssl-match-hostname==<IP_ADDRESS>
  - ipython-genutils==0.2.0
  - prompt-toolkit==1.0.14

Another example:
In [1]: import numpy as np

In [2]: import dask.array as da

In [3]: n = np.random.random((2, 3, 4))

In [4]: np.reshape(n, (3, 2, 4))
Out[4]:
array([[[0.17581148, 0.43739375, 0.91312755, 0.10826259],
        [0.27645767, 0.76332953, 0.33347355, 0.3275152 ]],

       [[0.22073202, 0.32309882, 0.89235911, 0.12772585],
        [0.35609819, 0.28493136, 0.52302273, 0.44864522]],

       [[0.98484051, 0.69152642, 0.50832536, 0.55934702],
        [0.22349445, 0.67470517, 0.31865274, 0.11967942]]])

In [5]: dn = da.from_array(n, chunks=(1, 2, 2), asarray=False)

In [6]: da.reshape(dn, (3, 2, 4))
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-6-0a2eadfa678a> in <module>
----> 1 da.reshape(dn, (3, 2, 4))

~/miniconda3/envs/dask-cuda/lib/python3.7/site-packages/dask/array/reshape.py in reshape(x, shape)
    185
    186     # Logic for how to rechunk
--> 187     inchunks, outchunks = reshape_rechunk(x.shape, shape, x.chunks)
    188     x2 = x.rechunk(inchunks)
    189

~/miniconda3/envs/dask-cuda/lib/python3.7/site-packages/dask/array/reshape.py in reshape_rechunk(inshape, outshape, inchunks)
     59                 oleft -= 1
     60             if reduce(mul, outshape[oleft:oi + 1]) != din:
---> 61                 raise ValueError("Shapes not compatible")
     62
     63             # TODO: don't coalesce shapes unnecessarily

ValueError: Shapes not compatible

In [7]: import dask

In [8]: dask.__version__
Out[8]: '1.2.2'

In [9]: da.reshape(dn, (2, 3, 4))
Out[9]: dask.array<array, shape=(2, 3, 4), dtype=float64, chunksize=(1, 2, 2)>

In [10]: da.reshape(dn, (2, 4, 3))
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-10-9c7a17e88c31> in <module>
----> 1 da.reshape(dn, (2, 4, 3))

~/miniconda3/envs/dask-cuda/lib/python3.7/site-packages/dask/array/reshape.py in reshape(x, shape)
    185
    186     # Logic for how to rechunk
--> 187     inchunks, outchunks = reshape_rechunk(x.shape, shape, x.chunks)
    188     x2 = x.rechunk(inchunks)
    189

~/miniconda3/envs/dask-cuda/lib/python3.7/site-packages/dask/array/reshape.py in reshape_rechunk(inshape, outshape, inchunks)
     59                 oleft -= 1
     60             if reduce(mul, outshape[oleft:oi + 1]) != din:
---> 61                 raise ValueError("Shapes not compatible")
     62
     63             # TODO: don't coalesce shapes unnecessarily

ValueError: Shapes not compatible

In [11]: dn = da.from_array(n, chunks=(2, 3, 4), asarray=False)

In [12]: da.reshape(dn, (2, 4, 3))
Out[12]: dask.array<reshape, shape=(2, 4, 3), dtype=float64, chunksize=(2, 4, 3)>

Here again, using trivial chunks functions properly.

Yes, the reshape function only works in certain situations.  See limitations here: https://docs.dask.org/en/latest/array-api.html#dask.array.reshape

Alright, it didn't occur to me checking that before, thanks for the link. Allowing other cases (such as keeping the same number of dimensions) something that we want to tackle in the future? If not, I'd suggest either closing this issue or perhaps adding a few checks in the code to give the user a reason for particular cases.

No strong thoughts from me on this


feat(search): add partial support for query temporal entities with post #420

I pushed a commit to switch back to coroutines, let me know.

I pushed a commit to switch back to coroutines, let me know.

LGTM, I have also updated the PR.

No tests for the new QueryService?

No tests for the new QueryService?

Indeed, i'm adding some tests with kotlinx-coroutines-test.

Anything now useless that you could remove from the TemporalXXXHandlerTests? Now that the query params are checked in a specific function, you should have:

all the query params validation checks in this new function
only one test per handler checking a 400 request is returned if the validation failed


Don't call setup.py directly in AppVeyor

A small change, but we are being discouraged  from calling setup.py directly.  Hence, this PR.
Note, the setup.py build command in appveyor/build_script.cmd wasn't actually needed so I've removed that.
Here's the deprecation warning in the AppVeyor output log:
C:\Python311-x64\Lib\site-packages\setuptools\_distutils\cmd.py:66: SetuptoolsDeprecationWarning: setup.py install is deprecated.

I saw that when I was making some fixes, but didn't take time to figure out what the right thing to do was.  Looks great.  Thanks.

@keitherskine I noticed the Appveyor badge is gone from the front page.  Any idea why?  I'm guessing it was something I did in an earlier commit, but am hoping you see this.

Hi @mkleehammer,
Yes, the AppVeyor badge was removed in this commit:
https://github.com/mkleehammer/pyodbc/commit/c0961ad025f3b1375fbb206caacfdbb6eec78030
I assumed that was deliberate, but perhaps it wasn't.


Update icons

Changes
updates data icons to have background; updates icons in other contexts to be styled appropriately
fixes https://app.asana.com/0/1202105544020589/1204082512322688/f

Fabulous! LGTM! @jlhogan


footer links

Pull Request
Description
Please include a summary of the changes and the related issue.
I made the footer section responsive using media queries. The links in the footer section will be adjusted according to the width.
Closes #1315
Type of change

[ ] Adding a new Resource
[ ] Adding a new Category
[ ] Documentation enhancement or fixes
[X] Website enhancement or fixes

Checklist:

[X] I read carefully CONTRIBUTING.md
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation

Screenshots (if appropriate):
Please add screenshots to help explain your changes.

Additional

Can you resolve the conflict?

Try to merge it know


Introduce SkipOnUnixAttribute and use it to simplify tests

@jpreese this addresses some of your comments on my fix PR. Do you think this goes in the right direction? I'm also very open for feedback on the naming.

Awesome! I think I like this approach better than the categories, a lot easier to run. My only big comment would be, if I recall correctly we have some tests that still check IsUnixPlatform() and add/remove test data based on the environment.
I think it would be a lot better if we werent modifying test data within the test (avoiding logic in tests), and just splitting those out into new tests that we can then just flat out skip for Linux.
Does that make sense?

Looking great! This will be so much better than that if(linux) block everywhere.

Most minor of quibbles, but I would call it WindowsOnly instead, as Windows is the odd one out, and Linux, BSD, macOS, iOS, Android collectively form the norm.
And I feel that rhyming grants this a little extra legitimacy.

That’s actually a good idea @rkoeninger - makes it much clearer IMO! Will change

I'll merge this when the build starts passing. I'm excited for this one!

Woo! Awesome 🎉

Done 🎉 Please squash this crappy commits when merging 🤖


feat:[#732] Implement Gcd and Lcm function to ast_builder

resolves #732
I have implemented  Gcd and Lcm functions to ast_builder.
Many thanks

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

42 of 42 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 97.914%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
29294

Relevant Lines:
29918

💛  - Coveralls


zipWithIndex is broken for org.apache.pekko.stream.scaladsl.GraphDSL.Implicits.PortOps

On the one hand,  org.apache.pekko.stream.scaladsl.GraphDSL.Implicits.PortOps inherits from org.apache.pekko.stream.scaladsl.Flow.
On the other hand, as of https://github.com/apache/pekko/pull/591 the implementation of Flow#zipWithIndex calls Flow#withAttributes
 def zipWithIndex: Repr[(Out, Long)] =
    statefulMap(() => 0L)((index, out) =>
        (index + 1L, (out, index)), _ => None)
      .withAttributes(DefaultAttributes.zipWithIndex)

However PortOpsImpl the implementation of PortOps does not support withAttributes
  override def withAttributes(attr: Attributes): Repr[Out] = throw settingAttrNotSupported

Here is a small program that works with pekko 1.0.2 but crashes with 1.1.1
import org.apache.pekko
import pekko.actor.ActorSystem
import pekko.stream._
import pekko.stream.scaladsl._

object Example extends App {
  import GraphDSL.Implicits._

  implicit val system = ActorSystem("GraphAndShapeExampleSystem")
  implicit val materializer = ActorMaterializer()

  val pickMaxOfThree = GraphDSL.create() { implicit b =>
    import GraphDSL.Implicits._

    val zip1 = b.add(ZipWith[Int, Int, Int](math.max _))
    val zip2 = b.add(ZipWith[Int, Int, Int](math.max _))
    zip1.out ~> zip2.in0

    UniformFanInShape(zip2.out, zip1.in0, zip1.in1, zip2.in1)
  }

  val resultSink = Sink.foreach(println)

  val g = RunnableGraph.fromGraph(GraphDSL.createGraph(resultSink) { implicit b => sink =>
    import GraphDSL.Implicits._

    // importing the partial graph will return its shape (inlets & outlets)
    val pm3 = b.add(pickMaxOfThree)

    Source.single(1) ~> pm3.in(0)
    Source.single(2) ~> pm3.in(1)
    Source.single(3) ~> pm3.in(2)
    pm3.out.zipWithIndex ~> sink.in
    ClosedShape
  })

  g.run()
}

Assigning to @He-Pin since the change was done by him

@remyhaemmerle-da Can you include the compile error or stack trace? It saves the volunteers a lot of time not to have to go and reproduce the issue.

Here is the stack trace.
[error] java.lang.UnsupportedOperationException: Cannot set attributes on chained ops from a junction output port
[error]         at org.apache.pekko.stream.scaladsl.GraphDSL$Implicits$PortOpsImpl.withAttributes(Graph.scala:1854)
[error]         at org.apache.pekko.stream.scaladsl.GraphDSL$Implicits$PortOpsImpl.withAttributes(Graph.scala:1845)
[error]         at org.apache.pekko.stream.scaladsl.FlowOps.zipWithIndex(Flow.scala:3310)
[error]         at org.apache.pekko.stream.scaladsl.FlowOps.zipWithIndex$(Flow.scala:3307)
[error]         at org.apache.pekko.stream.scaladsl.GraphDSL$Implicits$PortOpsImpl.zipWithIndex(Graph.scala:1845)
[error]         at Example$.$anonfun$g$2(Example.scala:33)
[error]         at org.apache.pekko.stream.scaladsl.GraphApply.createGraph(GraphApply.scala:53)
[error]         at org.apache.pekko.stream.scaladsl.GraphApply.createGraph$(GraphApply.scala:50)
[error]         at org.apache.pekko.stream.scaladsl.GraphDSL$.createGraph(Graph.scala:1589)
[error]         at Example$.delayedEndpoint$Example$1(Example.scala:24)
[error]         at Example$delayedInit$body.apply(Example.scala:6)
[error]         at scala.Function0.apply$mcV$sp(Function0.scala:42)
[error]         at scala.Function0.apply$mcV$sp$(Function0.scala:42)
[error]         at scala.runtime.AbstractFunction0.apply$mcV$sp(AbstractFunction0.scala:17)
[error]         at scala.App.$anonfun$main$1(App.scala:98)
[error]         at scala.App.$anonfun$main$1$adapted(App.scala:98)
[error]         at scala.collection.IterableOnceOps.foreach(IterableOnce.scala:619)
[error]         at scala.collection.IterableOnceOps.foreach$(IterableOnce.scala:617)
[error]         at scala.collection.AbstractIterable.foreach(Iterable.scala:935)
[error]         at scala.App.main(App.scala:98)
[error]         at scala.App.main$(App.scala:96)
[error]         at Example$.main(Example.scala:6)
[error]         at Example.main(Example.scala)
[error]         at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
[error]         at java.base/java.lang.reflect.Method.invoke(Method.java:580)
[error] stack trace is suppressed; run last Compile / run for the full output
[error] (Compile / run) java.lang.UnsupportedOperationException: Cannot set attributes on chained ops from a junction output port
[error] Total time: 7 s, completed Oct 9, 2024, 6:41:49 PM

I plan to revert #591. It didn't lead to any major perf gain and it breaks this use case.
We can always revisit.
#591 was one of a few changes of this style made for v1.1.0 and I think we're going to have to check all those changes for similar issues.

I still favour starting by reverting #591 but I think we could consider later following up and seeing if we can make PortOps support attributes instead of throwing UnsupportedOperationException.
https://github.com/apache/pekko/blob/95e8355c4c6f2041587e4d64590764e008d1f2b2/stream/src/main/scala/org/apache/pekko/stream/scaladsl/Graph.scala#L1854

#1526 was merged


validator can mis back edges: must traverse merge edges to see them

Consider this SPIR-V
               OpCapability Shader
               OpMemoryModel Logical GLSL450
               OpEntryPoint GLCompute %unused_entry_point "unused_entry_point"
               OpExecutionMode %unused_entry_point LocalSize 1 1 1
               OpName %unused_entry_point "unused_entry_point"
               OpName %foo "foo"
       %void = OpTypeVoid
          %1 = OpTypeFunction %void
%unused_entry_point = OpFunction %void None %1
          %4 = OpLabel
               OpReturn
               OpFunctionEnd
        %foo = OpFunction %void None %1
          %6 = OpLabel
               OpBranch %7
          %7 = OpLabel
               OpLoopMerge %8 %9 None
               OpBranch %10
         %10 = OpLabel
               OpBranch %9
          %9 = OpLabel
               OpBranch %11
         %11 = OpLabel
               OpLoopMerge %12 %13 None
               OpBranch %14
         %14 = OpLabel
               OpBranch %13
         %13 = OpLabel
               OpBranch %11
         %12 = OpLabel
               OpBranch %7
          %8 = OpLabel
               OpReturn
               OpFunctionEnd

This is the CFG:

The validator says:
error: line 16: Loop header 7[%7] is targeted by 0 back-edge blocks but the standard requires exactly one
%7 = OpLabel
The validator's backedge-finding algorithm traverses control edges, and continue edges, but ignores merge edges.  So if doesn't traverse %11 -> %12, and then doesn't "see" %12 -> %7.
I consider this a bug,  despite that it needs a more expansive definition of "back-edge" than the spec. It's the same expansion that justified following continue edges.

I have a patch to traverse merge edges when finding backedges.
But then the above example is rejected, saying " The continue construct with the continue target 9[%9] is not post dominated by the back-edge block 12[%12]"

The right answer, is that dominance/postdominance has to be adjusted too.


iOS8 requirement is leaving iphone4 without support

well not really a bug , but I wanted to test on an iphone4 stuck with ios7. any way to make it work with ios7 ?

I don't think there is anything requiring iOS 8 actually. However the performance on an iPhone 4 will not be great as the current video rendering mechanism in Bowser is kind of a hack 😏 If you have an Apple Dev account you can modify the build target and install the app through Xcode.

thanks for your answer. If there is no requirement, would be great to update next version so we can test it on iphone4. I guess we might see openCL GPU decoding for VP8 soon which migh help performance on old devices like iphone4

@freechelmi - are you aware of someone working on such a decoder/encoder implementation?
I would say that we should set the requirement as low as possible.


DiscoverClient.getInstances will be blocked after registering to Nacos server

DiscoverClient.getInstances will be blocked after registering to Nacos server

Fixed on nacos 0.3.0


version number on splashscreen not working

Describe the bug
Splashscreen used to show version number (see https://github.com/shabados/presenter/pull/590 for details) but in https://github.com/shabados/presenter/pull/652 electron was updated and remote  method was deprecated. So we need to replace this usage.
https://github.com/shabados/presenter/blob/21efae4a52faa2182eaf63467ab791502fc28634/app/electron/splashscreen/index.html#L21-L23
Screenshots

Specs

Device: MacOS
Version 2.11.0

#674 fixed this


Filters on custom fields are not working as expected

Environment

Nautobot version (Docker tag too if applicable): nautobot:1.5.16-py3.9
Python version: 3.9
Database platform, version: postgres:13
Middleware(s): none

Steps to Reproduce

Create a custom field "foo" of type Integer and assign it to a content type
Fill custom field of several objects with different values
Filter objects by custom field with specific filters (lt, lte, gt, gte ...)

Expected are results according to the used filter, e.g. for cf_foo_lt=100 all objects with custom field value "less than" 100

All results (except for filter the exact value) are not as expected aka wrong.

Example to reproduce the bug on https://main.demo.nautobot.com:
`import pynautobot
def main():
nb = pynautobot.api(
"https://main.demo.nautobot.com",
token="aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa",
)
CustomFields = nb.extras.custom_fields
Racks = nb.dcim.racks
if not CustomFields.filter("height_u"):
    CustomFields.create(
        name="height_u", content_types=["dcim.rack"], type="integer"
    )

    for i, rack in enumerate(Racks.all()):
        rack.u_height = (48, 52, 56)[i % 3]
        rack.custom_fields["height_u"] = rack.u_height
        rack.save()

print(f"filter\t\t\t\t\tcount\n{30*'-'}")
for filter in ("", "__n", "__lte", "__lt", "__gte", "__gt"):
    for field in ("u_height", "cf_height_u"):
        space = (25 - (len(field) + len(filter) + 3)) * " "
        print(
            f"{field}{filter}=52{space}{len(Racks.filter(**{field + filter: 52}))}"
        )
    print()

if name == "main":
main()
`
I've chosen racks just for demonstration, because the sandbox already contains entries and the model includes an integer field, so that the behavior is comparable. But the bug doesn't correlate to the assigned content type - it affects all content_types which a custom_field can be assigned. Here is another example:
`import pynautobot
def main():
nb = pynautobot.api(
"https://main.demo.nautobot.com",
token="aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa",
)
CustomFields = nb.extras.custom_fields
Content = nb.tenancy.tenants
content_type = "tenancy.tenant"
if not CustomFields.filter("foo"):
    CustomFields.create(name="foo", content_types=[content_type], type="integer")

    for i, site in enumerate(Content.all()):
        site.custom_fields["foo"] = i % 3
        site.save()

print(
    f"total objects of {content_type} with custom field foo: {Content.count(cf_foo__n=None)}\n"
)
print(f"filter\t\t\t\t\tcount\n{30*'-'}")
for filter in ("", "__n", "__lte", "__lt", "__gte", "__gt"):
    space = (25 - (9 + len(filter))) * " "
    print(f"cf_foo{filter}=1{space}{len(Content.filter(**{'cf_foo' + filter: 1}))}")

if name == "main":
main()
`
This can be easy changed to dcim.sites and "dcim.site"

Thanks for the report @axelbringenberg!


Mockup HTML layouts

[x] Homepage with header state not logged in
[x] Homepage with header state logged in
[x] User profile page
[x] Registration page
[x] Login page

Use Bootstrap 5.

@DLISM things to fix - https://t.me/c/1882480330/802


It's possible to select multiple party members to get Firebelly

Firebelly is a personal cast, but I can select other party members other than the one that can cast the spell. I could just select the ones that can cast it to receive the buff, but if I don't do it in the right order, one of them will be unable to cast it. Even if I do it in the right order, after a loading screen, it resets the order and one of the members is again unable to cast the spell (it becomes red). I'm not explaining it properly, so here's a video.
https://user-images.githubusercontent.com/49417384/210460976-f36df458-df5e-4483-a78c-5b263ce78e92.mp4

Interesting


Add convenience hash functions

Closes #459

@webmaster128 Do we need unit tests for these functions?

Do we need unit tests for these functions?

Let's have one test with one input each. Then we at least know nothing is broken and the rest is tested in the hasher class.


rebot combining results error

I am using latest Robot Framework version 5.0 for automated tests.
I use rebot tool to aggregate my test result ran in multiple stages.
rebot --nostatusrc --name Integrated -x full_results.xml //output.xml
it usually works fine.
I have a particular scenario where one the test fails with error "Maximum limit of started keywords and control structures exceeded"
when i try to combine this result with other successful test resutls it gives me error.
"failed: Incompatible child element 'errors' for 'test'."
I would like to get the combined report even if the errors are present in one of the it.
please find the attached output.txt (rename it to output.xml
Uploading output.txt…
)

No new info. Closing.


Fix:modify MLUOP to MLU-OPS in log.

Thanks for your contribution and we appreciate it a lot. :rocket::rocket:
1. Motivation
modify MLUOP to MLU-OPS in log.
2. Modification
core/cnlog.cpp
core/context.cpp

LGTM

LGTM


buildSupport: add helpers for verifying code signatures

Motivation for this change
Many projects provide signed source tarballs or signed binaries, which are currently not checked in nixpkgs. While it's not perfect, I believe that checking signatures provides some security advantage our typical "TOFU" workflow of implicitly trusting changed sha256 hashes.
The main reason is that, as opposed to sha256 hashes, public keys are usually long term, so we can invest more time to verify them initially, and after doing this once, malicious changes are more likely to be spotted.
This PR proposes some simple helpers for checking code signatures. It also provides some examples on how to use them.
Things done

fetchpgpkey is an extension of fetchurl that fetches a PGP public key to the nix store and verifies its fingerprint with gpg. It still needs a sha256 because it is based on fetchurl.  (I have experimented with an alternative implementation using builtins.fetchurl and just a fingerprint, no sha256, but that is not strictly a fixed-output derivation).

verifySignatureHook sets up signature verification and adds a preUnpackHook to automatically verify the signature srcSignature of src with signaturePublicKey when present. Verification can also be triggered manually using the shell functions defined in the hook.

Implemented 3 different examples for various use cases of the hook. These examples are for demonstartion and may or may not be merged with this PR.

samba4: the tarball is signed, but the signature is for the uncompressed tar archive. The hook has an option for this.
tor-browser-bundle-bin: provides signed tarballs but there's not standard unpack phase, so the verification must be triggered manually in buildCommand
1password: the tarball contains a signed binary and the signature file, so verification is done in checkPhase here.

Possible future work
Enhance fetchgit to verify signed revisions using git verify-commit.

cc @matthewbauer @jb55 @garbas

Success on x86_64-linux (full log)
Attempted: samba4
Partial log (click to expand)
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/pbrwprlmjdr80hw69vi94liag3jrbxkf-samba-4.7.6-dev/lib
patching script interpreter paths in /nix/store/pbrwprlmjdr80hw69vi94liag3jrbxkf-samba-4.7.6-dev
checking for references to /tmp/nix-build-samba-4.7.6.drv-0 in /nix/store/pbrwprlmjdr80hw69vi94liag3jrbxkf-samba-4.7.6-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/2600n568lxja2xkcd8dlcr5ifw2d1rq9-samba-4.7.6-man
gzipping man pages under /nix/store/2600n568lxja2xkcd8dlcr5ifw2d1rq9-samba-4.7.6-man/share/man/
strip is /nix/store/4qvrxzxa535y8304mk195x50b6p9607d-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/2600n568lxja2xkcd8dlcr5ifw2d1rq9-samba-4.7.6-man
checking for references to /tmp/nix-build-samba-4.7.6.drv-0 in /nix/store/2600n568lxja2xkcd8dlcr5ifw2d1rq9-samba-4.7.6-man...
/nix/store/ls86kj3655g2cy8zspprbxvmlcvjnxkq-samba-4.7.6

Success on x86_64-darwin (full log)
Attempted: samba4
Partial log (click to expand)
/nix/store/pqcxjnqhg4gqsbv3pm7clq54ahz08w4x-samba-4.7.6/bin/pidl: interpreter directive changed from "/usr/bin/env perl" to "/nix/store/206145946rz9hjqyvldq5ka927d4934s-perl-5.24.3/bin/perl"
/nix/store/pqcxjnqhg4gqsbv3pm7clq54ahz08w4x-samba-4.7.6/bin/smbtar: interpreter directive changed from "/bin/sh" to "/nix/store/q2wqq1k20v8kc3vckapqf5nws30brnni-bash-4.4-p23/bin/sh"
moving /nix/store/pqcxjnqhg4gqsbv3pm7clq54ahz08w4x-samba-4.7.6/sbin/* to /nix/store/pqcxjnqhg4gqsbv3pm7clq54ahz08w4x-samba-4.7.6/bin
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/0l9d4yzidj2wvj7iipyvalxf3iplr5qp-samba-4.7.6-dev/lib
patching script interpreter paths in /nix/store/0l9d4yzidj2wvj7iipyvalxf3iplr5qp-samba-4.7.6-dev
gzipping man pages under /nix/store/i76lbanwfwcw1gjs8cxhz1w2dhfnqn48-samba-4.7.6-man/share/man/
strip is /nix/store/7ddbq63v97nk8gkbf7gcsfmby37h6gbl-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/i76lbanwfwcw1gjs8cxhz1w2dhfnqn48-samba-4.7.6-man
/nix/store/pqcxjnqhg4gqsbv3pm7clq54ahz08w4x-samba-4.7.6

Success on aarch64-linux (full log)
Attempted: samba4
Partial log (click to expand)
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/517bwbca7g0ckljr92bgv9sfav6y1462-samba-4.7.6-dev/lib
patching script interpreter paths in /nix/store/517bwbca7g0ckljr92bgv9sfav6y1462-samba-4.7.6-dev
checking for references to /build in /nix/store/517bwbca7g0ckljr92bgv9sfav6y1462-samba-4.7.6-dev...
shrinking RPATHs of ELF executables and libraries in /nix/store/aqfm0zkhgakdp2h6kvdlzn3f1kcjnjcr-samba-4.7.6-man
gzipping man pages under /nix/store/aqfm0zkhgakdp2h6kvdlzn3f1kcjnjcr-samba-4.7.6-man/share/man/
strip is /nix/store/0pjsgkxz0rp5baycq5sp2s72lrr5q9sg-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/aqfm0zkhgakdp2h6kvdlzn3f1kcjnjcr-samba-4.7.6-man
checking for references to /build in /nix/store/aqfm0zkhgakdp2h6kvdlzn3f1kcjnjcr-samba-4.7.6-man...
/nix/store/ixzlxf5bxjsqr1d08nsvjvssnsish60n-samba-4.7.6

It should be worth taking a look at how Triton did it: see https://github.com/triton/triton/commits/master?before=6a1de201948b7755177562105e49181baca2bc2e+35&path[]=pkgs&path[]=build-support&path[]=fetchurl&path[]=builder.sh starting from Mar 30, 2016. (/cc @wkennington in case you are interested or would like to help.)

Okay a few thoughts on this.

I think there is a fairly good case to be made to do this for 1password. We are relying on a binary blob & that's unfortunate. I know a lot of people do use 1password & there is definitely good reason to support it in Nixpkgs because of that. However, we should be encouraging everyone to use free and open source software as much as possible! Nixpkgs just does not work very well with binary blobs.
The setup hook makes a lot more sense to me - it is much more reusable & we now don't have to check in the public key. I don't really understand the need for two layers of integrity checks but enough people have voiced support that maybe it is worth doing. At least I don't think it will hurt to do this.
I would strongly encourage us to drop the tor-browser-bundle-bin & firefox-bin. It makes no sense to support these things when we have a good source-based version of each of them. If someone wants to maintain them please put them in something like NUR or another overlay. Nix has never worked well with binary blobs and this is partly by design. This isn't exactly related to this PR but just something that I had forgotten was still around.

However, we should be encouraging everyone to use free and open source software as much as possible! Nixpkgs just does not work very well with binary blobs.

@matthewbauer fully agree.  I just randomly selected these examples to test and show different use cases of the setup hook.

Marked WIP while I have a look at how Triton does it and try to put part of the features into fetchurl.

It still needs a sha256 because it is based on fetchurl. (I have experimented with an alternative implementation using builtins.fetchurl and just a fingerprint, no sha256, but that is not strictly a fixed-output derivation).

If you move the verification from buildPhase to postFetch of a fixed-output derivation performing the download (one of fetchurl/fetchzip/..), fetchgpgkey could be an impure derivation without sha256.

Would it be hard to support this common use case? Here's are the instructions for verifying bitcoin signatures:

Verify that the checksum of the release file is listed in the checksums
file using the following command:

sha256sum --ignore-missing --check SHA256SUMS.asc

In the output produced by the above command, you can safely ignore any
warnings and failures, but you must ensure the output lists "OK" after
the name of the release file you downloaded. For example:

bitcoin-0.16.2-x86_64-linux-gnu.tar.gz: OK

Obtain a copy of the release signing key by running the following command:

gpg --recv-keys 01EA5486DE18A882D4C2684590C8019E36C2E964

The output of the command above should say that one key was imported,
updated, has new signatures, or remained unchanged.
Verify that the checksums file is PGP signed by the release signing key:

gpg --verify SHA256SUMS.asc

Ooooh I would really like to see this happen.
I don't see it as an extra layer of integrity checking. The SHA checksum ensure integrity of the file but verifying the file signature ensure the identity of the source. There is some overlap in practical but it remains two different goals.
I'm discovering this PR just now and it seems the proposed implementation is easily extensible.
A future improvement could be to support verification of Ed25519 signatures (think minisign or OpenBSD signify). It seems it would be easy enough to write fetched25519key and then extend verifySignatureHook to check whether it's fed with a PGP or Ed25519 key.
TL;DR; I'll now read other posts

A future improvement could be to support verification of Ed25519 signatures

gpg supports ed25519 sigs

Not to get into whether tor-browser-bundle-bin should continue to live, but I want to note that its signature is already verified via the updater.

Please do not make this part of fetchurl.  Placing it as part of fetchurl would enable an attack where and attacker tricks a victim to load a trojan-horse application into their nix-store and then updates the hash of a fetchurl in nixpkgs to reference the trojan horse.
Now I do respect that if the attacker has access to the victim's nixpkgs, there are a lot of bad things they can do, but getting limited "access" to nixpkgs hashes is maybe easier that you might think (see the reference to r-ryantm below).  However my larger point is that putting a GPG check into the derivation "guarantees" that it has been executed during the build, but putting the check in fetchurl does not, because fetchurl is simply bypassed if a file with the listed hash value ever already exists in the store.
As to motivation of why we want this: We'd like to use automatic updates by bots such as r-ryantm, however an attacker can currently attack packages by hijacking domains and serving "updated" trojan-horse applications, and get nixpkgs to be (semi-)automatically updated.  For those packages with signed source releases, it would let us use automatic updates for signed software more safely.

I agree with @roconnor-blockstream we should make this part of our tooling rather than the build process. Once we have our own checksum we no longer need to rely on public key cryptography.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/any-interest-in-checkings-signatures-while-building-packages/8918/7

@Mic92  Correct me if otherwise, but I don't think @roconnor-blockstream was saying that it shouldn't be verified via Nix in a derivation, only that the verification shouldn't be behind a fixed output derivation like fetchurl.  Having a signature check take place in Nix provides assurance of build input authenticity in addition to just integrity.
There is also another alternative I haven't seen mentioned:  Have another derivation that takes the source and signature as input, and outputs a symlink to the original source if and only if it passes a signature check.  The output of that verification derivation could then be passed as the src to mkDerivation.  This seems like more clean separation of concerns, rather than just adding signature checking as an additional setup hook in the stdenv. But, it comes at the cost of an additional derivation.  I have an example use of this idea for android APKs here: https://github.com/danielfullmer/robotnix/pull/59


Mention that the Display Name will be publicly visible if you choose to contribute

What

[ ] Mention that the Display Name will be publicly visible if you choose to contribute

Part of

#1382

Hi @teolemon ,
Should I add it like this?

As an alternative, I was thinking to add the text as onTap on username field. What do you suggest?


HTTP and HTTPS are treated seperately

The URL of the Rick Roll is protocol-dependant.
If http://rickroll.com is registered it won't cover https://rickroll.com.
Solution ideas:

Add a checker that checks if the URL is "close enough" to an existing one.
Do not store the method in the URL.

Fixed


Additional features needed in the snapshot code

Need to support writing and reading of snapshots in a streaming manner so that we don't allocate a buffer upfront for holding teh snapshot.
figure out what we need to do if an object with native fields is serialized (throw exception or serialize a null object).
When writing out a partial snapshot we should restrict ourselves to only sending out objects whose type is in the core-lib
While reading a full snapshot we need to assert that No GC happens even though allocations may happen
we need a way of recompiling a regex when a regex is sent over in the snapshot

@a-siva is any work on this planned short term? maybe we should close this issue.

I am assuming that this is stale.

There is still a comment in the code that references this issue.

I have removed the comment in the code base as it pertained to use of that code in reading full snapshots. We do not use that code any more for reading full snapshots (this is done through the cluster snapshot code).
Closing this bug as obsolete now.