Mistral model requires flash attention v2

Hi
I got the following error:
NotImplementedError: Mistral model requires flash attention v2

I tried use model=TheBloke/Mistral-7B-Instruct-v0.1-GPTQ
Thanks

Hey @pribadihcr, thanks for trying out LoRAX!
This error message can sometimes be misleading, was there any other error above in the logs that you can share? Also, are you running in our pre-built Docker images?

Hi @tgaddair , here is the full logs
model=TheBloke/Mistral-7B-Instruct-v0.1-GPTQ

docker run --gpus all --shm-size 1g -p 8080:80 -v $volume:/data -e HF_HUB_ENABLE_HF_TRANSFER=0 ghcr.io/predibase/lorax:latest --model-id $model

2023-11-17T02:38:15.469298Z  INFO lorax_launcher: Args { model_id: "TheBloke/Mistral-7B-Instruct-v0.1-GPTQ", adapter_id: "", source: "hub", adapter_source: "hub", revision: None, validation_workers: 2, sharded: None, num_shard: None, quantize: None, dtype: None, trust_remote_code: false, max_concurrent_requests: 128, max_best_of: 2, max_stop_sequences: 4, max_input_length: 1024, max_total_tokens: 2048, waiting_served_ratio: 1.2, max_batch_prefill_tokens: 4096, max_batch_total_tokens: None, max_waiting_tokens: 20, hostname: "f5ab61aeb3c5", port: 80, shard_uds_path: "/tmp/lorax-server", master_addr: "localhost", master_port: 29500, huggingface_hub_cache: Some("/data"), weights_cache_override: None, disable_custom_kernels: false, cuda_memory_fraction: 1.0, json_output: false, otlp_endpoint: None, cors_allow_origin: [], watermark_gamma: None, watermark_delta: None, ngrok: false, ngrok_authtoken: None, ngrok_edge: None, env: false, download_only: false }
2023-11-17T02:38:15.469394Z  INFO download: lorax_launcher: Starting download process.
2023-11-17T02:38:18.625603Z  INFO lorax_launcher: hub.py:123 Download file: model.safetensors

2023-11-17T02:44:40.962134Z  INFO lorax_launcher: hub.py:131 Downloaded /data/models--TheBloke--Mistral-7B-Instruct-v0.1-GPTQ/snapshots/6ae1e4ae2cfbaf107c705ed722ec243b4f88014d/model.safetensors in 0:06:22.

2023-11-17T02:44:40.962245Z  INFO lorax_launcher: hub.py:153 Download: [1/1] -- ETA: 0

2023-11-17T02:44:41.505805Z  INFO download: lorax_launcher: Successfully downloaded weights.
2023-11-17T02:44:41.505949Z  INFO shard-manager: lorax_launcher: Starting shard rank=0
2023-11-17T02:44:43.908316Z  WARN lorax_launcher: flash_attn.py:48 Unable to use Flash Attention V2: GPU with CUDA capability 7 5 is not supported for Flash Attention V2

2023-11-17T02:44:43.911488Z  WARN lorax_launcher: __init__.py:73 Could not import Mistral model: Mistral model requires flash attn v2

2023-11-17T02:44:44.480708Z ERROR lorax_launcher: server.py:218 Error when initializing model
Traceback (most recent call last):
  File "/opt/conda/bin/lorax-server", line 8, in <module>
    sys.exit(app())
  File "/opt/conda/lib/python3.9/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/opt/conda/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/lib/python3.9/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/opt/conda/lib/python3.9/site-packages/typer/core.py", line 216, in _main
    rv = self.invoke(ctx)
  File "/opt/conda/lib/python3.9/site-packages/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/conda/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/lib/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/opt/conda/lib/python3.9/site-packages/typer/main.py", line 683, in wrapper
    return callback(**use_params)  # type: ignore
  File "/opt/conda/lib/python3.9/site-packages/lorax_server/cli.py", line 81, in serve
    server.serve(
  File "/opt/conda/lib/python3.9/site-packages/lorax_server/server.py", line 262, in serve
    asyncio.run(
  File "/opt/conda/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/opt/conda/lib/python3.9/asyncio/base_events.py", line 634, in run_until_complete
    self.run_forever()
  File "/opt/conda/lib/python3.9/asyncio/base_events.py", line 601, in run_forever
    self._run_once()
  File "/opt/conda/lib/python3.9/asyncio/base_events.py", line 1905, in _run_once
    handle._run()
  File "/opt/conda/lib/python3.9/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
> File "/opt/conda/lib/python3.9/site-packages/lorax_server/server.py", line 214, in serve_inner
    model = get_model(
  File "/opt/conda/lib/python3.9/site-packages/lorax_server/models/__init__.py", line 283, in get_model
    raise NotImplementedError("Mistral model requires flash attention v2")
NotImplementedError: Mistral model requires flash attention v2

2023-11-17T02:44:44.910114Z ERROR shard-manager: lorax_launcher: Shard complete standard error output:

Downloading (…)lve/main/config.json: 100%|██████████| 963/963 [00:00<00:00, 46.9kB/s]
Traceback (most recent call last):

  File "/opt/conda/bin/lorax-server", line 8, in <module>
    sys.exit(app())

  File "/opt/conda/lib/python3.9/site-packages/lorax_server/cli.py", line 81, in serve
    server.serve(

  File "/opt/conda/lib/python3.9/site-packages/lorax_server/server.py", line 262, in serve
    asyncio.run(

  File "/opt/conda/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)

  File "/opt/conda/lib/python3.9/asyncio/base_events.py", line 647, in run_until_complete
    return future.result()

  File "/opt/conda/lib/python3.9/site-packages/lorax_server/server.py", line 214, in serve_inner
    model = get_model(

  File "/opt/conda/lib/python3.9/site-packages/lorax_server/models/__init__.py", line 283, in get_model
    raise NotImplementedError("Mistral model requires flash attention v2")

NotImplementedError: Mistral model requires flash attention v2
 rank=0
Error: ShardCannotStart
2023-11-17T02:44:45.009297Z ERROR lorax_launcher: Shard 0 failed to start
2023-11-17T02:44:45.009346Z  INFO lorax_launcher: Shutting down shards

Thanks @pribadihcr, the key line is this one:
2023-11-17T02:44:43.908316Z  WARN lorax_launcher: flash_attn.py:48 Unable to use Flash Attention V2: GPU with CUDA capability 7 5 is not supported for Flash Attention V2

What type of GPU are you running on? Looks lik flash attention v2 requires Ampere (A10G, A100, etc.) and above.

Hey @pribadihcr, just wanted to follow-up on this.

HI @tgaddair, Sorry. My GPU is poor ( GTX 2070 )  :(

Getting the same error with mistral 7b when deploying on A10G, base lorax container. Any workaround?

Hey @joaopcm1996, can you share the nvidia driver version you're using (output from nvidia-smi)? We haven't updated min driver version in some time, so this shouldn't be due to a change in the LoRAX image. I just put up PR #328 which should hopefully make the true error more apparent, however.

Turns out this was indeed a driver version issue, not on LoRAX. Thanks !
NVIDIA-SMI 470.182.03 Driver Version: 470.182.03 CUDA Version: 12.1

Thanks @joaopcm1996 for closing the loop!

Hi @joaopcm1996 I followed your tutorial that you posted on medium and I got stuck with the same problem. I'm trying to spin up a AWS Sagemaker Async endpoint and I get the same error:
ImportError: CUDA is not available
Shard complete standard error output:
The NVIDIA driver on your system is too old (found version 11080). Please update your GPU driver by downloading and installing a new version from the URL .........
The installed version of bitsandbytes was compiled without GPU support. 8-bit optimizers, 8-bit

I'm also using an A10g.
My question is, how can you update the drivers on sagemaker? Or do I just build the docker image using a more recent cuda version?

Backing NVIDIA drivers into the container will not work, and it's not possible for a user to update the drivers on SageMaker endpoint instances. The solution until they are updated is to revert to LoRAX 0.8.0, worked for me.


quick connection shortcut for bluetooth device

Description of the new feature / enhancement
can a shortcut be made to connect and disconnect a bluetooth device when I assign a shortcut from the keyboard?
Scenario when this would be used?
when I press right ctrl+right shift on the keyboard.
Supporting information
I use my airpods on both my phone and my computer. when connecting the airpods to the computer, it is difficult to enter the settings every time. instead, I want to assign a shortcut from the keyboard.

/feedbackhub


ajax request 404 not found

Hi Dave, how are you? Here again with questions.
I am running the application on a windows server (http://sd-1965379-w.ferozo.com/)
The root of the server is pointed to the public folder of the project.
So far everything works fine, but when I try to do an ajax to a declared route (ej: http://sd-1965379-w.ferozo.com/login) the server returns 404 because it is trying to access that /login directory
Can you think that could be what happens?
As always, thank you very much for your time and your wisdom.
Greetings

I think I found a way that the rewrite always reaches the index.php but it is not matching any route.

Hi Dave, how are you? Here again with questions.
I am running the application on a windows server (http://sd-1965379-w.ferozo.com/)
The root of the server is pointed to the public folder of the project.
So far everything works fine, but when I try to do an ajax to a declared route (ej: http://sd-1965379-w.ferozo.com/login) the server returns 404 because it is trying to access that /login directory
Can you think that could be what happens?
As always, thank you very much for your time and your wisdom.
Greetings

I already have the rewrite in IIS to get to the index.php always, but $ _SERVER ['QUERY_STRING'] is always empty.

$ _SERVER ['QUERY_STRING']

thats my rewrite in IIS, i cant get de routes in QUERY_STRING
<?xml version="1.0"?>
<configuration>
  <system.webServer>
    <defaultDocument>
      <files>
        <clear />
        <add value="index.php" />
      </files>
    </defaultDocument>
    <rewrite>
      <rules>
        <rule name="Imported Rule 2" stopProcessing="true">
          <match url="^(.*)$" ignoreCase="false" />
          <conditions>
            <add input="{REQUEST_FILENAME}" matchType="IsFile" ignoreCase="false" negate="true" />
            <add input="{REQUEST_FILENAME}" matchType="IsDirectory" ignoreCase="false" negate="true" />
            <add input="{URL}" pattern="^/favicon.ico$" ignoreCase="false" negate="true" />
          </conditions>
          <action type="Rewrite" url="index.php?q={R:1}" appendQueryString="true" />
        </rule>
      </rules>
    </rewrite>
  </system.webServer>
</configuration>

Solve, just a litlle modification ja
 <action type="Rewrite" url="index.php?q={R:1}" appendQueryString="true" />


add Yearn EURS strategy

EURS Jar is bricked because an initially faulty v1 wrapper contract will not allow for withdrawAll() to run properly, in order to migrate the strategy to a working on. Closing PR


The big clean up

WIP

Let scrutiny begin
@richardlitt @noffle @dignifiedquire can you CR the code please?
After finishing here, I'll bring https://github.com/diasdavid/registry-static to be part of the project instead of a dependency that gets run as a shell command, so that I can get more interesting stats and control of what it is doing.

There is a note on the README important to be able to run the tests https://github.com/diasdavid/registry-mirror/tree/refactor/the-big-clean-up#development

Will check!

I don't understand; what does this PR do?

@diasdavid left some preliminary comments. Also travis is unhappy

@diasdavid: could we get some more context? What are the high-level goals of this refactor?

@dignifiedquire thank you for the CR, I've pushed changes following you comments :)
@RichardLitt @noffle Apologies for not being clear, this PR is the transformation of registry-mirror from a collection of hacks and glue to it all work to a more structured code base, where there is order, making contribution and debugging easier

@dignifiedquire travis is unhappy due to the missing 0.4.0 binary :)

Will move ahead and merge this one, to be able to go to the next step, merge registry-static into the mix :)


changed port and added example

Changed port from 8888 to 8898 to not conflict with Jupyter port.
Removed label column from test example
Added a second test example that produces a response of 1.0
Leaving output cells in notebook, so folks can see what the notebook is supposed to produce.

Thanks a ton for adding this @inchiosa. DO you think you could remove the notbook output for consistency's sake. The general idea of keeping notbook ouputs for legibility is a good one, but we will need to do a bit of refactoring work to make this work for all notebooks.  If you need I can do that before the merge as well!


(1.16.5, version 1.2.0) Uncraftable Forges

I am unable to craft any of the default alloy forge controllers. Attempting to craft them in a crafting table does nothing and Roughly Enough Items doesn't display a way to craft them.
Mod list:
Alloy Forgery
Backslot
YUNG's Better Caves
Better Third Person
Biome Makeover
Cave Biomes
Charm
Clean Cut
Cloth Config
Companion Bats
Durability Viewer
Dynamic FPS
Enhanced Attack Indicator
INMIS
Iris+Sodium
Item Scroller
Lithium
malilib
Mod Menu
Mo' Structures
Mythic Metals
Paxi
Phosphor
Roughly Enough Items
Sodium Extra
Spark
Terrestria
Tool Stats
Traverse
YUNG's API

Quick update, after reading through the past issues, I've found out Charm is the mod that's causing things to screw up.


Plus-minus (±) Quick Accent

Description of the new feature / enhancement
Can you please add support for the plus-minus symbol (±) in the Quick Accent feature?
When holding down '=', it would be great to see ± and possibly some more relevant symbols such as approximately equal to (≈).
It would act similar to holding down '-' for different dashes (e.g., en and em dash).
Scenario when this would be used?
When writing scientific results, I often use ±.
E.g., "mean ± SEM".
Supporting information
No response

This feature already exists. If you hold down ,, you should see it there. Note that I'm running v0.74.0, so it's possibly a recent addition.

Agreed this is under the , but maybe it should be under the = or + as well.

Agreed.
While the , key contains ±, the - and = keys both contains ∓ ∓. I think this can be streamlined, be removed from the comma key (which already houses a few other duplicates) and be brought into the - or = keys.
I'm no developer, but I wouldn't mind helping out in streamlining the special characters. For the most part, they work very well (the Pound sign can be found in P, which I can imagine, but it would make sense if it was available in E, because £ looks like...) and it's nice when they're there, but sometimes having to hunt around for the relevant symbol can be troublesome (I had to google where the ± was located after I copy/pasted it from another webpage).

Come on folks, make this happen. Put ± on + where it belongs, and let ∓ stay on -


Temporarily add on_exit_shutdown to ros_gz humble branch until backport

The ros2 branch on ros_gz has moved passed us so we need to be on the humble branch. I just noticed that this PR hasn't been backported to humble and notified @mjcarroll who is working on it now. The issue is that the batch script won't run more than one sim because it doesn't shutdown the current launch when gz sim exists.
In the meantime (until the backport):
in the ros_gz clone:
$ git checkout ros2
$ git pull
$ git checkout humble
$ git pull
$ git cherry-pick 63e85a952668b824d266740606a245c2a1db4d0c

and then colcon build the workspace again

Backports are in, I'll try to get a release out ASAP

thanks a ton for the quick turnaround!


Add Riemann Quantum Classifier

This PR implements #23.
The helper classifier which is implemented here will be used for CVGridSearch.
A couple of tests were added to test_classification.py and required to refactorize some of the methods into a dataset/utils module.
(WIP: missing documentation of the dataset module, and error in the pipeline)

Thanks for the review!
There is still a problem with the pipeline, using python 3.8. To be honest I am not sure where it comes from.
Tests are ok on my local computer.

Ok, I compared the package version with older builds and it seems that the scipy version changed 2 days ago.
@toncho11 FYI

Thanks for the review :)


--record feature is freakin' awesome.

I sure love lowering the bar to communicating new issues unambiguously. That is all.
Feel free to close this bug as "working as intended".

Let me second @jonhnet's comment. It's already been really useful and it promises to be very helpful in the future.
Thank you @ahuoguo, @elaustell, and @jaybosamiya.
@elaustell, you are the assignee, feel free to close as "working as intended" once you see this :) Thanks!


Cluster Explorer Nav - Update so that the jump frame can be wider to accommodate CRDs

On master-head - commit id: 16616de9c

Logged in as a standard user
Click on Jump to search field
Some of the options are NOT displayed fully.
The nav bar CANNOT be resized horizontally.

Expected
The nav bar should be resizable
OR
On Hovering over the resource, the name should be displayed fully.

We're not going to do resizing.  There is a separate wireframe for enhancing the Jump menu that will be done eventually.

This is no longer an issue with the 2.6 UI


[FLINK-23228][docs-zh] Translate "Stateful Stream Processing" page into Chinese

What is the purpose of the change
(This pull request is to translate "Stateful Stream Processing" page into Chinese.)
Brief change log

Translate "Stateful Stream Processing" page into Chinese, the page url is https://issues.apache.org/jira/projects/FLINK/issues/FLINK-23228?filter=allopenissues

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no )
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

English to Chinese key word in this translation:

English
中文

overtaken
超过

barrier
屏障

in-flight
运行中的

Keyed State
键控状态

Key Group
键组

@ranqiqiang 重新更新了一下

Hi @hackergin , @ranqiqiang , updated

@ranqiqiang @hackergin ，I submitted some changes, please have a review if you are free, thanks!

Hi @hackergin , thanks for your review and following up my PR, I make some changes as you suggested, aslo I review my whole translation and do some update.
My update as below(exclude you suggested):

有状态操作的一些示例：  change to 一些有状态操作的示例：
Keyed state 被维护在一个可以认为是键/值存储的地方。 change to Keyed state 被维护在一个键/值存储的地方。
在执行期间，每个有键算子的并行实例和一个或多个键组的键一起使用。 change to 在执行期间，每个 keyed operator 的并行实例和一个或多个键组的键一起使用。
例如在 Apache Kafka 中，这意味着告诉消费者（consumer）从 offset Sk 开始获取。  change to 例如在 Apache Kafka 中，这意味着告诉消费者（consumer）从 offset Sk 开始获取。

Thanks @RollsBean  for the great job. LGTM.

Hi @hackergin  , if nothing need to be modified, can you merge this PR? Thanks

@RollsBean  sorry , I don't the the merge permission, @wuchong  please have a look at this pr.

Hi @wuchong  can you have a look at this PR

Hi @xccui  @klion26 , can you have a look at this PR? I have finished translation and hackergin reviewed this PR but no one merge it.


Use jndi-dns to obtain default name servers

Motivation:
Using reflection to obtain the default name servers may fail in Java9 and also in previous Java versions if a SecurityManager is present.
Modifications:
Try using jndi-dns to obtain default name servers and only try using reflection if this fails.
Result:
Be able to detect default name servers in all cases. Fixes [#6347].

Works well on JDK 9 on Mac for me without extra settings.
One question: Why do this in a static block if unused and also log "Default DNS servers:" even when clients provide alternatives?

The google server definitions would be useful as DnsServerAddressStreamProvider.google

This change conflicts with #6592, building off that PR would be good.
Also generally I think it should be one line to do any of

use platform configuration or fallback to google
use platform configuration or fail
use google
use a specific configured host list

@yschimke I just rebased and will merge it as it is once the CI pass. We can do another PR as a followup if needed.

Cherry-picked into 4.1 (a0fcb72e5d)


Fix broken links in docs and comments #740

Links to uber.github.io/ludwig are broken and should be replaced with ludwig-ai.github.io/ludwig-docs

Thank you for this! We are transitioning to a new org an these things ore the side product of it, thank for helping fixing them!


report playback

Now you can launch the scope app with something like
./prog/scope --mode=app --weave=false --app.collector=file:///tmp/reports
and if the specified dir contains reports with filenames in the form <timestamp>.{msgpack|json}[.gz]
e.g. "1488557088545489008.msgpack.gz", then these reports are replayed in a loop at a sequence and speed determined by the timestamps.

This is really handy for benchmarking of the standalone app.
@awh grabbed 30s of Weave Cloud production reports for me, which I fed to a scope app running on my laptop and restricted to a single core (i.e. GOMAXPROCS=1). I then ran
for i in $(seq 1 100); do echo $i $(date); curl -o /dev/null -s http://localhost:4040/api/topology; done

to see at what rate I can grab the complete topology - something the UI tries to do every 5s. It's taking an average of 9s, with the CPU pegged. Ouch. And bear in mind that this isn't even reading & parsing reports - that's only done once, at startup.

@rade Have you explicitly tested that the demo service doesn't break with these changes? (i.e. that collecting a single file didn't break).
For extra points you could supply a unit test for it (using https://github.com/weaveworks/common/blob/master/fs/fs.go for mocking)

Have you explicitly tested that the demo service doesn't break with these changes? (i.e. that collecting a single file didn't break).

Yes.

For extra points you could supply a unit test for it

I thought about it, but the pain/gain equation doesn't work out, especially considering that this really isn't a mainstream user-facing feature.

I thought about it, but the pain/gain equation doesn't work out, especially considering that this really isn't a mainstream user-facing feature.

It may very well end up used for https://github.com/weaveworks/scope/issues/2286 but, fair enough.


All tiny_mce fields in Builder have the same id et_pb_content, broken conditional logic.

Problem Description
All tiny_mce fields in Builder have the same id et_pb_content
Hence they don't support conditional logic (original field id just missing)
Steps To Reproduce

Add tiny_mce filed my_awesome_editor_field, add depends_show_if with some value
Add other field with affects = my_awesome_editor_field
Display condition doesn't work

Screenshot and/or gif
https://i.imgur.com/SIbYUrV.png

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[x] Detailed description
[x] Steps to reproduce included
[ ] Support Ticket URL included
[x] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.


BUG: Leverage BOT JSON RPC error VM102:1 MetaMask - RPC Error: JsonRpcEngine: Response has no error or result for request:

VM102:1 MetaMask - RPC Error: JsonRpcEngine: Response has no error or result for request:
{
"method": "net_version",
"jsonrpc": "2.0",
"id": 286587432,
"origin": "https://www.curvebot.fi",
"networkClientId": "de1f9b7a-3dc6-4a19-a7b4-7a81bd70feab",
"tabId": 900248136
} {code: -32603, message: 'JsonRpcEngine: Response has no error or result for…6-4a19-a7b4-7a81bd70feab",\n  "tabId": 900248136\n}', data: {…}}

@taariq Are you getting this error right now? When creating a bot or loading the UI?
Because no issue for me.

Closing as rollback to Liquify


Guidelines for PR Related to Developer Doc

Include some high-level guidelines to be followed when raising PRs for the developer documentation.

I felt it will be helpful to have the steps with commands documented.
Even add the commands or steps when a developer has merge issues.
Same is the case for commit message template. Alternatively, if we can set a commit template to our projects that will be even better.

@prateekpandey14 @AmitKumarDas
It is a good idea to include actual commands. Will work on it.


Better error messages when composing remote functions

The error messages that we currently give for tasks that fail and whose outputs are passed into subsequent functions are pretty uninformative.
Consider the code below.
import ray

ray.init(start_ray_local=True, num_workers=1)

@ray.remote([int], [int])
def f(x):
  raise Exception("Function f failed.")

@ray.remote([int], [int])
def g(x):
  return x

ray.get(g.remote(f.remote(0)))

It fails with
Exception                                 Traceback (most recent call last)
<ipython-input-1-b17bf94ecad7> in <module>()
     11   return x
     12
---> 13 ray.get(g.remote(f.remote(0)))

/Users/rkn/Workspace/ray/lib/python/ray/worker.pyc in get(objectid, worker)
    705   value = worker.get_object(objectid)
    706   if isinstance(value, RayFailedObject):
--> 707     raise Exception("The task that created this object ID failed with error message:\n{}".format(value.error_message))
    708   return value
    709

Exception: The task that created this object ID failed with error message:
          Traceback (most recent call last):
            File "/Users/rkn/Workspace/ray/lib/python/ray/worker.py", line 1040, in typecheck_arg
              raise Exception("Argument {} for function {} has type {} but an argument of type {} was expected.".format(i, name, type(arg), expected_type))
          Exception: Argument 0 for function <function __main__.g at 0x103828cf8> has type <class 'ray.worker.RayFailedObject'> but an argument of type <type 'int'> was expected.

The problem with this error message is that it doesn't contain the exception thrown by f.

The following examples are good test cases for checking whether error messages are helpful or not.
In each case, check the error message given by ray.get(f.remote()) as well as f.remote() followed by ray.put(0).
@ray.remote([], [int])
def f():
  raise Exception("Something went wrong.")

ray.get(f.remote())


Decals can't be removed visually while mapping

As per title. Deleting a decal does actually work though, and when you save/reload the map your efforts are visualized properly again. It feels like this function broke a few days after the decal system was added. Probably the same issue as #6874 #6851 and #6610.
Description

Screenshots

Additional context

Unrelated to #6851 and #6610

Pretty sure this is quite related to if not the same: #6873. When decals were first merged the remove by right click did work. Now it only updates once you reload the map.

This was fixed, but the decal curse continues via other demonic pathways


Nuget package update?

Is it possible to get the nuget package updated? We'd like to use the .NET Standard-compatible library in osu!.

Hello,
Sorry about that - I thought I'd pushed it at the time. I've just packaged it fresh and pushed it.
Note that as I'm not hugely familiar with .NET Core yet, and the fact it was such a PITA to get VS2017 to build the darn thing, I've marked this package as pre-release just in case I've screwed something up. Assuming I haven't and it works for you, I'll upversion again without the pre-release suffix as I have found to my displeasure that depending on pre-release packages is another PITA.
Again, apologies for the delay - let me know if you run into any issues with the new package.
Regards;
Richard Moss

Hiya!
I've confirmed that the updated package is working perfectly, thanks!
Closing issue.


Disallow running br/sig as root, check for required caps.

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Update node version to 16 (build-in npm 8) npm to 7+

Is your feature request related to a problem? Please describe.

annotation-app
npm install

annotation-service
npm install

Describe the solution you'd like
update npm to 7+ to suit package-lock.json file to version 2
Describe alternatives you've considered
No response
Additional context
No response

lockfileVersion to 2  has some change: https://docs.npmjs.com/cli/v8/configuring-npm/package-lock-json

need update:

GitHub actions - smoke-test
app-docker file
server-docker file
docker-compose
docker-compose-host


Weird error when loading a dynamic property of a type

Hello,
I'm trying to migrate my project MrScrooge from nestjs with graphql into swift with vapor, graphqlkit (which uses graphiti), And seems I've got some issue, not sure if it's my error or is an error on graphiti.
When I add a dynamic field into a type, to load only when I've got the request, I do it as following:
Type(MyProfile.self){
   Field("group", MyProfile.groups
}
MyProfile{
   func groups(request: Request, Parent: MyProfile){}
}

You can see this here:
https://github.com/Dracks/mr-scrooge/commit/27862abb0d6008fb3e6b8e545565bf4afc85a2f5#diff-86900d9223aa878afd6fd2f942cdd076fa63c7a147fae5e22db15769e0d2ec95R63
and here:
https://github.com/Dracks/mr-scrooge/commit/27862abb0d6008fb3e6b8e545565bf4afc85a2f5#diff-86900d9223aa878afd6fd2f942cdd076fa63c7a147fae5e22db15769e0d2ec95R136-R141
it ends crashing in some weird error, the error is the following:
{
  "errors": [
    {
      "message": "keyNotFound(CodingKeys(stringValue: \"id\", intValue: nil), Swift.DecodingError.Context(codingPath: [], debugDescription: \"No value associated with key CodingKeys(stringValue: \\\"id\\\", intValue: nil) (\\\"id\\\").\", underlyingError: nil))",
      "locations": [
        {
          "line": 6,
          "column": 7
        }
      ],
      "path": [
        "me",
        "groups"
      ]
    }
  ]
}```

I look with xcode, it seem to be some error parsing the session. Any Idea of what it can be? It happens in multiple places I've got the dynamic field.

Do you need some small example? I can try to create it.

Thanks,
Dracks

Hi, this is a pretty typical Swift error when you try to decode a type that doesn't have a required field. I'd double check that the data coming back from your database has the id field populated so it matches your codable model definitions.
I might be able to give use some more direct guidance if you provide the GraphQL query that failed

Hey, thanks for your answord.
The query was the following one:
query {
  me{
    __typename
    ... on MyProfile {
      username
      groups{
        name
        id
      }
    }
  }
}

But the debugger with breakpoints never reached the dynamic load function. And when I don't require the groups, everything work fine
Thanks,
Jaume

If everything worked okay when you didn't include groups, I'd try dropping a breakpoint in the groups resolver before you load the object from the database or whatever data source you're using. Since this is a decoding error, it is likely thrown when you load the group object from the underlying data source.

I'm pretty sure, I already tried and never reach the group resolver. But I will try it again.

Hey, I tested, it never reach the resolver of the subElement, I generated a small repo that it's a minimum reproducible error (I think)
You can see it here: https://github.com/Dracks/GraphqlVaporSample
Simply run the tests, there is 2 tests, one includes the call to the groups, the other doesn't, the first one fails the other works fine.


remove require autoload.php

copy of #145 but to dev branch
While implementing the PasswordService into my (Laravel 5) project I got a fatal error:

Fatal error: main(): Failed opening required '/path/to/project/vendor/jgrossi/corcel/src/Password/../../vendor/autoload.php' (include_path='.:/usr/share/php:') in /path/to/project/vendor/jgrossi/corcel/src/Password/PasswordService.php on line 7

Since Laravel already includes /vendor/autoload.php, it's not needed to do this again.

made a mistake by merging from the wrong branch ;)


Add more details about tuner, assessor design.

What would you like to be added:
For below tuners, assessors and advisors in docs folder, they have only overview usage, but not introduce how they are implemented.
evolutionTuner.md, gridsearchTuner.md, hyperoptTuner.md, medianstopAssessor.md, smacTuner.md
Why is this needed:
Those information is helpful for who want to write customized tuner, assessor, and advisor.
Without this feature, how does current nni work：
it works without document changes..
Components that may involve changes:
document
Brief description of your proposal if any:
as above..

We had made lots of improvement on documentation since Feb, the fundmental problem raised here should already addressed. Assigning to @rabbit008 to quickly go through the doc and list out what is left to be improved.


chore: dependency update

Description
Suppose to resolve dependency on crates time and tokio, could not update lower level dependency on failure.
Issue link (if applicable)
https://github.com/graphops/graphcast-sdk/security/dependabot

we should automate dependapot PRs 🤔


Update Index.json

Added information about Devang Gandhi.

Hey it looks like the PR is not accepted in my Hacktoberfest profile, can you please look into this?

Thanks its resolved.


Assigning Conference Bridge

This page really needs screen shots!
Step 4: Assign a service number to the conferencing bridge:
Microsoft 365 admin center > Admin centers > Teams > Legacy portal.
is this still valid?  I don't have this option.

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

ID: 8ed33488-1a24-f443-dd56-c5327188b5a4
Version Independent ID: 8da32180-396d-c7a8-6410-472beabff9ce
Content: Set up Audio Conferencing for Microsoft Teams - Microsoft Teams
Content Source: Teams/set-up-audio-conferencing-in-teams.md
Service: msteams
GitHub Login: @tonysmit
Microsoft Alias: tonysmit

@bjstigall Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Expose the commit dates

I need this for SCM API stuff

@kohsuke did you review it or blindly merged and released?

@stephenc ?

Good catch! I did look at the change but I missed it.
I'll fix this and push out a new release.

I'll leave the getAuthoredDate as is. I trust Stephen's English than mine and yours.

Good catch on the copy and paste.
So I went with "commitDate" rather than "committerDate" and "authoredDate" vs "authorDate" as these are the dates of the respective actions. In a sense GitHub conflates the action with the person, and it just sounded wrong to my ear to say "getCommitterDate"... what's the committer date... does that method arrange for flowers a movie and a meal with the committer... ok, perhaps the committer deserves that as a reward for their work... that is more the date of the commit... closest analogy for author I could find was authored

Question not in english, it not a novel. When you know API you can use library easily, otherwise you should scroll code/methods to get idea what they return. Also why you can't call getter().field yourself and need this proxy method?


OCI support

It would be fairly trivial to produce OCI images based on the available metadata.

Not going to do this, at least not implementing a packager that produces an OCI image. There is too much required in the way of customization that would involve including too much OCI-specific info in the package.


Initial implicit waiting work

This begins the work to migrate over implicit waits to being enabled by default. To start with I've just looked at doing a couple of things

https://github.com/natritmeyer/site_prism/commit/66861dfecff23995ded8db527ec7ab76b29627d5 -> Remove the 2 waiter meta-programmed methods as these are no longer required. And fix up all spec files
https://github.com/natritmeyer/site_prism/commit/2fd49314213162502d87be129189f9cadc48b027 -> Migrate over all features that used the old meta-programmed methods, either to use a visibility waiter or using the wait key assignment. I've added a separate ticket to re-visit these and ensure we're testing the key assignment in all methods. There are also other breakout tickets for reviewing the features to ensure they're all in the right place, and another to standardise the API

ping @tgaff @ineverov - This "should" be ready to review now. I believe I've fixed everything that should make it work out of the box and configure implicit waits by default. But I've possibly missed something.
Also shrunk the helper DSL by 2 (wait_for_*), as these will now be used with regular locators (And failure to locate will revert to the Capybara DSL error codes, which we test for)

For the travis re-work @mnohai-mdsol feel free to raise a PR and do it there. As it's probably not relevant to this work.

Is master now 3.x?   It seems like we're still going to want one more 2.xx build, right?

Will make a 2.17 cut with deprecations as a siloed branch.


Fix bug with simd_bits +=.

Caught this when trying to address #598.

Windows test is failing to build cirq?

Looks like a hash failed to match. Probably transient, and if not then it's matplotlib's problem not yours.

For the fuzz tests I ran 10000 times initially. I also checked with the following python code:
from typing import List, Union
import numpy as np
from fixedint import UInt64

def join_words(words: List[int]) -> int:
    """Join a list of 64bit words into a single python integer"""
    out_str = ''
    for x in words[::-1]:
        out_str += f"{x:064b}"
    return int(out_str, 2)

def split_into_words(x: str) -> List[int]:
    """Split a binary representation of a python int into a list of 64 bit words"""
    strings = [x[i:i+64] for i in range(0, len(x), 64)]
    return [int(i, 2) for i in strings[::-1]]

def adder(lhs: List[UInt64], rhs: List[UInt64]) -> List[UInt64]:
    """Multi-word adder"""
    carry = 0
    for i, _ in enumerate(lhs):
        vbf = lhs[i]
        lhs[i] = lhs[i] + rhs[i] + carry
        carry = lhs[i] < vbf or (carry & (lhs[i] == vbf))
    return lhs

def to_fixed(x: int, width=192):
    """Fixed width string rep of python integer"""
    return f"{x:0{width}b}"[-width:]

def to_cpp_vec(x: List[Union[int, UInt64]], label="x"):
    """Write as a vector of integers"""
    out_str = f"std::vector<uint64_t> {label}{{"
    for i, w in enumerate(x):
        comma = "," if i < len(x) - 1 else ""
        out_str += f"{w}ULL{comma}"
    out_str += "};"
    print(out_str)

def gen_random_integer(num_bits):
    num_words = (num_bits - 1) // 64 + 1
    words = []
    for iw in range(num_words):
        r = np.random.random()
        b = 0
        if r < 1/3:
            b = 0
        elif r > 2/3:
            if iw == num_words - 1:
                for k in range(num_bits - 64 * iw):
                    b |= (1<< k)
            else:
                b = 0xFFFFFFFFFFFFFFFF
        else:
            for k in range(64):
                if np.random.random() > 0.5:
                    b |= (1<< k)
        words.append(b)
    return words

if __name__ == "__main__":
    np.random.seed(7)
    #for i in range(10_000):
    for i in range(4):
        #num_bits = np.random.randint(10, 1200)
        num_bits = 512
        x = gen_random_integer(num_bits)
        y = gen_random_integer(num_bits)
        a = join_words(x)
        b = join_words(y)
        xpy = to_fixed(a+b, num_bits)
        xu = [UInt64(i) for i in x]
        yu = [UInt64(i) for i in y]
        z = adder(xu, yu)
        z_str = to_fixed(join_words(z), num_bits)
        xpy_str = xpy
        if i == 3:
            to_cpp_vec(x, "x")
            to_cpp_vec(y, "y")
            to_cpp_vec(split_into_words(xpy), "z")
        assert to_fixed(join_words(z), num_bits) == xpy
        assert xpy_str == z_str

The tests appear to be crashing with buffer overflows.

The perils of developing on my mac


[WIP] Change some csharp_namespace of protobuf

close #1487

:white_check_mark: Build AElf <IP_ADDRESS>31 completed (commit https://github.com/AElfProject/AElf/commit/3696cadfae by @jason-hoopox)

:white_check_mark: Build AElf <IP_ADDRESS>78 completed (commit https://github.com/AElfProject/AElf/commit/ef214e826c by @jason-hoopox)


Spacing issue

After running the WAC create command there were some issues in the strings.ts file from a dash in the resjason file. It was resolved by removing the "-" in the company title. The "-" didn't need to be removed from the section title ("//:" "Test-Feature" was fine).
ex. "Test-Feature_title" => "TestFeature_title"

Thanks for pointing this out.  Fixed in 0.1.9.


Multiple actions can be started at once, but pio kill <action>  will only kill the last one started

Actions don't have a uniqueness constraint, since they don't check pio_jobs_running on start up. But they do write their PID to pio jobs running. Currently pio kill will look at the PID in pio_jobs_running.
This is a problem for experiment profiles.

Fixed with the new version of JobManager


Release 1.0.0

Should we use this opportunity to mention it's Diffix Elm more prominently, e.g. in the abstract, description etc?

Kind of late now, I already made the release.


Support Phlex DSL on scenarios

I'm doing some tests on my local repo, and I forgot to change to my repository on Github UI. Sorry!


bad gateway Login to your account

hola, he instalado mariadb configurado en configuration.yaml (recorder:
db_url: mysql://homeassistant:password@core-mariadb/homeassistant?charset=utf8mb4, he abierto los puertor 80 y 443, he instalado ngnix, pero cuando intento ingresar me da error error bad gateway, ¿alguna solución?, gracias

o se esto ayuda
2021/05/15 07:25:23 [error] 574#574: *3588 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: nginxproxymanager, request: "GET /api/ HTTP/1.1", upstream: "http://<IP_ADDRESS>:3000/", host: "<IP_ADDRESS>:81", referrer: "http://<IP_ADDRESS>:81/"

I have the same issue after upgrading to 2.9.3, looked at the logs, but haven't figured it out yet

. I have enabled on the router nat loopback. Thanks
El vie., 21 may. 2021 3:43, Rafael @.***> escribió:

I have the same issue after upgrading to 2.9.3, looked at the logs, but
haven't figured it out yet
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jc21/nginx-proxy-manager/issues/1098#issuecomment-845592096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHZEP4IOFIYQW7ETYMOLOLTOW3FZANCNFSM445QMAEQ
.

docker logs name_of_npm_container
and look for obvious problems.

@jc21 i'm running npm on a alpine linux LXC container (inside proxmox), I was digging on /var/log, but didn't find anything. have you seen this error before?

I'm sorry, I don't know what it can be
El vie., 21 may. 2021 14:59, Rafael @.***> escribió:

@jc21 https://github.com/jc21 i'm running npm on a alpine linux LXC
container (inside proxmox), I was digging on /var/log, but didn't find
anything. have you seen this error before?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jc21/nginx-proxy-manager/issues/1098#issuecomment-845932728,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHZEP6WHHMSF34ABWU6FFDTOZKL3ANCNFSM445QMAEQ
.

I'm having the same issue.  It seems to be a problem connecting to the database
error     getaddrinfo ENOTFOUND db

fresh install and same issue

ok so I have tried multiple versions and have the same error, so I dont think its to do with this version and more to do with the db

I too am having this issue

I made no progress and ended up moving to Traefik2 instead as my reverse proxy.  It seemed much more complicated but I seem to have got it working in the end.  I followed this guide for anyone interested.  No matter what I tried I just couldn't get this working, either with jlsage or jc21.

Are you using Docker Compose v2? According to the last post of #1057, this appears to cause this issue. I'm currently trying to figure out how to fix it using v2 (because I use Portainer on arm, and that Portainer still doesn't support v3 sadly).


Feature/cosmetic updates

Updated main to show 1 from each of our functions.
Added comments to city class for consistency.

Added Report Comment Convention on app Report - 1.3, 2.4, 3.3 etc


============= DO NOT USE THIS LIBRARY ==============

As per #158 please do not use this library as it is obsolete. Several upstream packages rely on this repo when it is no longer maintained and DOES NOT HAVE py3k compatibility.
This issue has been posted here as it is currently the only way to put up a banner until @urbanairship can find someone who has access. In the meantime @joestump @markunsworth and myself are working together to deliberate on the next best course of action for the PyPi repo holding this package.
PLEASE DO NOT COMMENT HERE: All comments should be placed in #158.

Just noting here that this issue can now be ignored for anyone who sees this in the future.


Did CalendarDatePicker support INotifyDataErrorInfo

Describe the bug
According to this PR https://github.com/AvaloniaUI/Avalonia/pull/7308 validation with Avalonia is handled correctly when using CalendarDatePicker.SelectedDate property (didn't tested). Now Im using the last version of FluentAvalonia but I cant make it to work, not sure if it is an Avalonia or this library bug. The validation is running correctly against a TextBox but not against CalendarDatePicker, it didnt show any visual indication. Maybe Im missing something. If I set the same validation against a property binding to a textbox I can see the validation error.
Maybe you can provide a workaround.
Screenshots
Failed validation state. In the Textbox the error appears but the CalendarDatePicker dont show any date validation error.

Desktop/Platform (please complete the following information):

OS: Windows 10
FluentAvalonia Version 1.3.4
Avalonia Version 0.10.14

Additional context
Code sample:
using System;
using DataModel;
using ReactiveUI;
using ReactiveUI.Fody.Helpers;
using ReactiveUI.Validation.Extensions;

namespace ViewModels.Modules.Competitions
{
    public class CompetitionProxy : ValidatableViewModelBase
    {
        public CompetitionProxy(Competition competition)
        {
            Competition = competition ?? throw new ArgumentNullException(nameof(competition));

            InitializeProperties(competition);

            InitializeValidations();

            UpdateSkaterOnValueChange();
        }

        public Competition Competition { get; }

        [Reactive] public DateTime StartDate { get; set; }

        [Reactive] public DateTime EndDate { get; set; }

        private void InitializeProperties(Competition competition)
        {
            StartDate = competition.StartDate;
            EndDate = competition.EndDate;
        }

        private void InitializeValidations()
        {
            // Start date
            // End date
            var checkDateRangeObservable = this.WhenAnyValue(
                viewModel => viewModel.StartDate,
                viewModel => viewModel.EndDate,
                (startDate, endDate) => startDate < endDate);

            this.ValidationRule(
                viewModel => viewModel.EndDate,
                checkDateRangeObservable,
                "Competition end date must after start date"); // Date validation
        }
    }
}

<UserControl xmlns="https://github.com/avaloniaui"
             xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
             xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
             xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
             mc:Ignorable="d" d:DesignWidth="800" d:DesignHeight="450"
             x:Class="PatiTournApp.Modules.Competitions.CompetitionProxyEditorView">
    <Grid RowDefinitions="* * * * * * * *">
        <TextBlock Grid.Row="4" Text="Start date:" />
        <CalendarDatePicker Grid.Row="5"  SelectedDate="{Binding StartDate}" />

        <TextBlock Grid.Row="6" Text="End date:" />
        <CalendarDatePicker Grid.Row="7" SelectedDate="{Binding EndDate}" />
    </Grid>
</UserControl>

It's a styling issue here...
https://github.com/amwx/FluentAvalonia/blob/172ab14f0c4ddb8b4025a9699e3ad35d174bb08a/FluentAvalonia/Styling/BasicControls/CalendarDatePickerStyles.axaml#L25-L122
I didn't add a separate DataValidationErrors item as the root of the control template, instead template binded it to the TextBox. Theoretically that should work (and should be the way to do it) but for some reason it doesn't


auto-update Docker instance after source edits

Hello, great work!  I too prefer to deploy via Docker.
One minor pet peeve was having to stop, rebuild and run the Docker image after any change in code (or git pull).
Here is my new way to launching, after the initial docker build:

$ cd ~/www/robinhood-on-rails
$  docker run --name=RobinhoodOnRails  -v $HOME/www/robinhood-on-rails:$HOME/robinhood-on-rails  -dt -p 3222:3000 robinhood-on-rails

Wherein:

--name since I don't expect to run multiple instances on the same server. This prevents Docker from assigning random aliases, and simplifies instance management.

-p 3222 externalizes to a port other than 3000, which on my server was already taken.

-v The main point of this issue.  Binds a dir from host to guest, and since Rails is already launched in dev mode, reloads new code on the fly.

I am unsure how to best adapt into the Dockerfile, etc.

Thanks for posting! As I don't use docker I'll let someone with more experience handle this


Add JVMenu

Project URL
https://github.com/JV17/JVMenu
Category
UI->Menu
Description
Added GitHub link to JVMenu and a small description.
Why it should be included to awesome-ios (mandatory)
Because it's a cool menu that I think many developers would love to use.
Checklist

[ ] Has 50 GitHub stargazers or more
[x ] Only one project/change is in this pull request
[ x] Isn't an archived project
[ ] Has more than one contributor
[ ] Has unit tests, integration tests or UI tests
[x ] Addition in chronological order (bottom of category)
[ ] Supports iOS 9 / tvOS 10 or later
[ x] Supports Swift 4 or later
[ x] Has a commit from less than 2 years ago
[x ] Has a clear README in English

@JV17 We cannot merge your PR yet until you fill all of the contribution guideline requirements. Feel free to open PR once it is ready.
Please check our CONTRIBUTION GUIDELINES


Bartholomew template

My thought here is to provide an "add component" version of this so that people can easily add a content-managed component to their Spin applications.  This will hopefully make a nice demo of how Spin facilitates composing applications out of Wasm modules, whether custom or off-the-shelf.
Known issues:

Does not play nicely with HTTP bases other than root.
The static fileserver URL currently points to a test release in my fork.

Open question:

Does this even belong in the Spin default template set, or should it go in the Bartholomew repo and require separate installation?  Or should we avoid having a Spin template that would need to be maintained alongside the GitHub template?  (If we don't want this here or at all, I would at least like to offer a static file server component in the default set - it's a widely useful thing.)

Signed-off-by: itowlson<EMAIL_ADDRESS>

Closing as stale: the GitHub template repository does everything this does that people are likely to care about, and has the huge advantage of working correctly.


Add a utility to get a cell value efficiently.

See here for more context: https://stackoverflow.com/questions/49520198/google-bigtable-hbase-getvaluearray-vs-cellutil-clonevalue

PR #1799  is created & merged for the same hence closing this issue.


Accessing nonexistent properties is not checked in JS files

TypeScript Version: 2.8.0-dev.20180302
Search Terms:

checkJs
noImplicitAny
object
undefined

Code
const fruits = {
    PINEAPPLE: 'tasty'
};

// Name of the property is mistyped and running this code in JS engines
// will result in a TypeError.
fruits.PINAPPLE.toUpperCase();

Expected behavior:
Following error should be raised if checkJs and noImplicitAny flags are enabled:
Property 'PINAPPLE' does not exist on type '{ PINEAPPLE: string; }'. Did you mean 'PINEAPPLE'?
Actual behavior:
No error is shown.
Related Issues: #16137 - this one describes a similar problem, but for some reason it was closed by reporter.

Probably a bug since JS objects created by object literals allow assignment of new properties -- but a property access should still be an error.

Objects in a .js file are assumed to be open-ended. in other words they behave as if they had  { [x:string]: any; } on them.


[Feature][Connector] update flink elasticsearch sink version

Purpose of this pull request
Update flink elasticsearch sink version to 7.x
This close #1597

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in you PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Have you checked, whether the corresponding license or notice has changed, can you list your checklist?

I checked elasticsearch series's license, both are Apache license 2

Have you checked, whether the corresponding license or notice has changed, can you list your checklist?

I checked elasticsearch series's license, both are Apache license 2

I mean, in addition to this, you also need to check whether the corresponding License file and the corresponding NOTICE have changed.

Have you checked, whether the corresponding license or notice has changed, can you list your checklist?

I checked elasticsearch series's license, both are Apache license 2

I mean, in addition to this, you also need to check whether the corresponding License file and the corresponding NOTICE have changed.

Done


Why add document only support text and json files when I have unstructured API?

Hi there,
since I have added the Unstructured API which should support all type of file formats like DOCX and HTML why the Add document just allow text and json files?
Cheers.

I think I have to install the full package: https://unstructured-io.github.io/unstructured/installation/full_installation.html
Testing now!

Nope that didn't affect the UI at all.

Agree! We will add more data types for ingestion in the next update

The newest version should now allow any Unstructured input through the API


Add Send To Toys

Send To Toys
Enhancement of the Send To system menu, replacement for the Microsoft PoswerToy.

Thanks for the recommendation, if you would like it to be added to the list, please create an Pull Request Instead. Thanks for Understanding!

Thanks for the suggestion, but:

It's your list, not mine.
I'm not currently setup to do GitHub Pull Requests, which is too much hassle just for this.
Since you've not that interested in adding it, I'll just write it off as a waste of my time.

https://help.github.com/en/github/collaborating-with-issues-and-pull-requests/creating-a-pull-request

As I said, that's too much hassle.


Domain model improvements

Domain prioritising and other improvements

Can we rebase and merge these commits into one as well.

Do verify the changes made in the pull request. Upon completed verification, a new pull request will be opened from a different branch discarding this pull request which will contain only a single commit


pure firebase usage which is not supported by angularfire2 such as Storage or Messaging

Version info
Angular:
"@angular/common": "^4.2.0-rc.2",
"@angular/compiler": "^4.2.0-rc.2",
"@angular/core": "^4.2.0-rc.2",
"@angular/forms": "^4.2.0-rc.2",
"@angular/http": "^4.2.0-rc.2",
"@angular/platform-browser": "^4.2.0-rc.2",
"@angular/platform-browser-dynamic": "^4.2.0-rc.2",
"@angular/router": "^4.2.0-rc.2",
Firebase:
"firebase": "^4.1.1",

AngularFire:
"angularfire2": "^4.0.0-rc.1",

Other
"@angular/cli": "^1.1.0",
"@angular/compiler-cli": "^4.1.3",
Failing code
import * as messaging from 'firebase/messaging';

export class XXXXX{
    constructor(){
        this.messaging = messaging();
        this.messaging.onTokenRefresh(()
    }

Debug output
** Errors in the JavaScript console **
core.es5.js:1020 ERROR Error: Uncaught (in promise): TypeError: __WEBPACK_IMPORTED_MODULE_2_firebase_messaging__ is not a function
TypeError: __WEBPACK_IMPORTED_MODULE_2_firebase_messaging__ is not a function
    at new CloudMessagingService (cloud-messaging.service.ts:36)
    at _createClass (core.es5.js:9605)
    at _createProviderInstance$1 (core.es5.js:9573)
    at resolveNgModuleDep (core.es5.js:9558)
    at NgModuleRef_.get (core.es5.js:10644)
    at resolveNgModuleDep (core.es5.js:9562)
    at NgModuleRef_.get (core.es5.js:10644)
    at resolveDep (core.es5.js:11147)
    at createClass (core.es5.js:11011)
    at createDirectiveInstance (core.es5.js:10831)
    at new CloudMessagingService (cloud-messaging.service.ts:36)
    at _createClass (core.es5.js:9605)
    at _createProviderInstance$1 (core.es5.js:9573)
    at resolveNgModuleDep (core.es5.js:9558)
    at NgModuleRef_.get (core.es5.js:10644)
    at resolveNgModuleDep (core.es5.js:9562)
    at NgModuleRef_.get (core.es5.js:10644)
    at resolveDep (core.es5.js:11147)
    at createClass (core.es5.js:11011)
    at createDirectiveInstance (core.es5.js:10831)
    at resolvePromise (zone.js:769)
    at resolvePromise (zone.js:740)
    at zone.js:817
    at ZoneDelegate.webpackJsonp../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:424)
    at Object.onInvokeTask (core.es5.js:3924)
    at ZoneDelegate.webpackJsonp../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:423)
    at Zone.webpackJsonp../node_modules/zone.js/dist/zone.js.Zone.runTask (zone.js:191)
    at drainMicroTaskQueue (zone.js:584)
    at HTMLButtonElement.ZoneTask.invoke (zone.js:490)

What has happening
When I am using firebase 3.9.0 and angularfire2 4.0.0-rc0, messaging and storage is available by  following code
import * as messaging from 'firebase/messaging';

export class XXXXX{
    constructor(){
        this.messaging = messaging();
        this.messaging.onTokenRefresh(()
    }

import * as storage from 'firebase/storage';
export class XXXXX{
    constructor(){
       this.storage_ref = storage().ref();
    }

However after firebase major upgrade to 4.x, I cannot access either messaging nor storage with the above code.

Hi @morninng! The import syntax you have it a bit off.
Instead of:
import * as messaging from 'firebase/messaging';

You first import firebase/app and then naked import the feature needed.
import * as firebase from 'firebase/app'; // brings in app in typings
import 'firebase/messaging'; // naked import to bring in feature

Thank you! it works


some markets missing their estimated start times

I'm unable to reproduce this, do we have any specific examples?

looks fixed

Example was in the screenshot in the first comment. Made this exact market again and it's fixed

Reopening this as the issue has come up again

I have yet to be able to reproduce this. are you consistently seeing it on a specific market maybe? @matt-bullock

Not seeing it either now. I'll close but lets keep an eye on it @JohnDanz


[Dapr] Disable applying CRDs during a downgrade

Signed-off-by: Shubham Sharma<EMAIL_ADDRESS>This PR checks for a version downgrade and sets hooks.applyCRDs to false accordingly.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

@shubham1172 , I am approving the PR.  Pl. make sure to test these 3 cases - upgrade, downgrade and no change, in case you have not done that yet.

@shubham1172 , I am approving the PR. Pl. make sure to test these 3 cases - upgrade, downgrade and no change, in case you have not done that yet.

Thanks @NarayanThiru, yes, all 3 scenarios were tested. Here are some examples:
Scenario 1 - Upgrade from 1.9.0 to 1.9.4
az k8s-extension create --cluster-type managedClusters --cluster-name dapr-ms --resource-group dapr-rg --name dapr --extension-type microsoft.dapr --auto-upgrade false --version 1.9.0 --configuration-settings "skipExistingDaprCheck=true" --debug
az k8s-extension update --cluster-type managedClusters --cluster-name dapr-ms --resource-group dapr-rg --name dapr --auto-upgrade false --version 1.9.4 --configuration-settings hooks.hookDeletePolicy="before-hook-creation" --debug
From logs:

DEBUG: cli.azext_k8s_extension.partner_extensions.Dapr: No downgrade detected. Setting hooks.applyCrds to true.

Scenario 2 - Downgrade 1.9.4 to 1.9.0
az k8s-extension update --cluster-type managedClusters --cluster-name dapr-ms --resource-group dapr-rg --name dapr --auto-upgrade false --version 1.9.0 --configuration-settings hooks.hookDeletePolicy="before-hook-creation" --debug
From logs:

DEBUG: cli.azext_k8s_extension.partner_extensions.Dapr: Downgrade detected from 1.9.4 to 1.9.0. Setting hooks.applyCrds to false.

Scenario 3 - Set auto-upgrade to true at 1.9.0 (and upgrade to 1.9.4)
az k8s-extension update --cluster-type managedClusters --cluster-name dapr-ms --resource-group dapr-rg --name dapr --auto-upgrade true --configuration-settings hooks.hookDeletePolicy="before-hook-creation" --debug
From logs:

DEBUG: cli.azext_k8s_extension.partner_extensions.Dapr: No downgrade detected. Setting hooks.applyCrds to true.

Scenario 4 - Disable auto-upgrade and downgrade to 1.9.0
az k8s-extension update --cluster-type managedClusters --cluster-name dapr-ms --resource-group dapr-rg --name dapr --auto-upgrade false --version 1.9.0 --configuration-settings hooks.hookDeletePolicy="before-hook-creation" --debug
From logs:

cli.azext_k8s_extension.partner_extensions.Dapr: Downgrade detected from 1.9.4 to 1.9.0. Setting hooks.applyCrds to false.


I just damaged my new roboclaw 2x45A controller by launching this file

It is useless now and here is what I did,
I have opened the windows app and limited the max I to 50 for protection.
Checked P I D parameters. They were all 0.
Then cloned the repo.
Connected the computer via usb.
Started roslaunch roboclaw_node roboclaw.launch
Firstly motor 1 started to run in full pwm,
Then motor 2 started to run in full pwm and instantly changed direction
Because of the inertia, the controller died even Motors were in neutral.
It was the weirdest thing that I have experience. No idea what the launch file did.
There was not encoders only motors without touching ground.
Anyway, does anybody know why? I have just buy a 2x60A roboclaw and I certainly do not want to damage this one too.

Wooooooo your PID was set to 0 ?!

Yep, I was trying to connections without any move. So I thought 0 pid gives no motion. What do you think, what was wrong?

You have several problems:

You have to have the encoders hooked up or the roboclaw has no way to know how fast the wheels are spinning. For the ROS driver to work that is needed.

PID does not work that way, read this for more information. PID

YOU MUST tune the PID loop using the software as I stated in the README for the ROS code to work or things like you described will happen.

I know how PID works. What do you expect when you give zero to P I D coefficients? Well, I expect no motion. As I said before, I set that parameters to zero for not to damage controller in the first launch.

The ROS driver does not do anything but set speeds for the roboclaw controller to meet.  If it is not properly configured it will not work. This means you must have the encoders attached.  Also if you are just testing I would recommend to disconnect the motors from the power terminals and measure the voltage to see if things are operating properly.

agreed with Sonyccd, I have tested with all the proser connections to the encoders and setup the right values on the Windows application provided by ionmc, when you setup the correct values ans test the node it does work.
I had followed all the recommendations and also did a auto tune, but in my case I did setup and test with low revolutions and thermo fused connections to the motors
the node work very well and does not burn your motor controller :), the only isseu is the odom data
but it works :)


The plugin causes the Neovim start screen to flicker.

Thanks for making a good Go plugin for neovim, but here i found before loading the plugin, when opening neovim, the start screen is fixed in place. However, if the plugin is loaded, the start screen flickers and becomes a blank screen. How to fix ?
start screen:

blank screen

Here is config
require('go').setup()

vim.api.nvim_exec([[ autocmd BufWritePre *.go :silent! lua require('go.format').goimport() ]], false)

I have not see this before. Could you provide more context? e.g. your OS, nvim version. a minium config.
Also the plugin has a default minium config in playground folder. Is the issue reproducible with  https://github.com/ray-x/go.nvim/blob/master/playground/init_lazy.lua ?

@ray-x thanks for your reply, this is my environment:

OS: macOS 13.3.1
nvim: 0.9.0
plugin config:

require('go').setup()

vim.api.nvim_exec([[ autocmd BufWritePre *.go :silent! lua require('go.format').goimport() ]], false)

my full config: https://github.com/MrZhang123/dotfiles/tree/master/nvim

maybe lazy load go.nvim for filetype go, gomod intead of all filetypes? I think one reason might be gopls starup time and startup check slow down the nvim.

In my case i had a similar issue with this plugin. Similar to what ray-x mentioned, the workaround for me that worked for me was lazy loading go.nvim and my gopls lsp config, although I'm pretty sure this isn't ideal
{
		"ray-x/go.nvim",
		ft = "go",
		config = function()
			require("go").setup({
				lsp_cfg = true,
				goimport = "gopls", -- goimport command, can be gopls[default] or goimport
				lsp_gofumpt = true, -- true: set default gofmt in gopls format to gofumpt
				lsp_on_attach = require("lsp.on_attach").setup, -- use on_attach from go.nvim
				lsp_diag_virtual_text = false, -- DO NOT overwrite my virtual text settings
				luasnip = true,
				lsp_inlay_hints = {
					parameter_hints_prefix = "󰊕",
				},
			})
			local cfg = require("go.lsp").config() -- config() return the go.nvim gopls setup
			require("lspconfig").gopls.setup(cfg)
			local format_sync_grp = vim.api.nvim_create_augroup("GoImport", {})
			vim.api.nvim_create_autocmd("BufWritePre", {
				pattern = "*.go",
				callback = function()
					require("go.format").goimport()
				end,
				group = format_sync_grp,
			})
		end,
		lazy = true,
	}```


Feature Request: Multipart Response

I'm currently using JavaMail to parse multipart responses since I can't see a way to do that with OkHttp.  However, it appears that JavaMail parses the response as a single payload rather than allowing streaming.  I say this because javax.mail.internet.Multipart only offers getCount() and getBodyPart(int index) to walk through the parts, rather than an iterator or async API.  I'd like a way to work on parts as they come in, to support streaming response processing.  That would allow processing very large responses without requiring outrageous amounts of JVM memory.  Ideally, each part would of course be available once its headers are available, so we could also stream each part as it comes in (in case one part is too large to fit in memory).  So my feature request is that OkHttp support Multipart Responses, and do it in a way to support streaming (processing each part as it arrives).

Yep, being able to decode a multipart body would be really nice.
We’ve got this plan for a thing “OkHttp 4” which would have a single type for RequestBody and ResponseBody. If we do that, it would be handy for our MultipartBody class to be able to decode its content. That’s also useful for testing clients!

Its been almost 3 years but it’s done!

Hello @swankjesse! Took me a lot of digging to find this thread :) ... do you have any code samples on how to parse multipart responses (basically I’m using HTTP range requests with OkHttp3 and I get multiple ranges in one response)?

@sayan-chaliha try this: https://square.github.io/okhttp/4.x/okhttp/okhttp3/-multipart-reader/

Hi @swankjesse is it possible to create http responses of Content-Type: Multipart/Related;, by using OkHttp ?


Detection of package name

The detection of a package name gives wrong results in case we have:
package Mail::SpamAssassin::Bayes::CombineChi; 1;

This gives a warning:
warning: the name '1' supplied as the argument of the \class, \struct, \union, or \include command is not an input file

we will now stop searching after the first ';'

Thanks for merging all the proposed pull requests.
Do you think it will be possible to create a new version on sourceforge?

I saw the release. perfect!!!

A release was made on Github. I copied that to Sourceforge and over to
CPAN.  Thanks for all of your contributions and for moving the
project forward.
Bret
On Fri, Nov 15, 2019 at 2:54 AM albert-github<EMAIL_ADDRESS>wrote:

Thanks for merging all the proposed pull requests.
Do you think it will be possible to create a new version on sourceforge?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jordan2175/doxygen-filter-perl/pull/20?email_source=notifications&email_token=ACMTMGEFMAWFPR5JMXZ5QRLQTZWVTA5CNFSM4JLNOQK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEE5N6A#issuecomment-554292984,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMTMGC3CPF43IWWGSTJKHTQTZWVTANCNFSM4JLNOQKQ
.

--
Sent from my TI-99/4A
PGP Fingerprint: 63B4 FC53 680A 6B7D 1447  F2C0 74F8 ACAE 7415 0050


.bashrc syntax error

When logging in the VM, bash throws this error :
Welcome to Ubuntu 14.04.3 LTS (GNU/Linux 3.16.0-55-generic x86_64)

 * Documentation:  https://help.ubuntu.com/
Development Environment
Last login: Thu May 12 03:14:58 2016 from <IP_ADDRESS>
-bash: $'\r': command not found
-bash: /home/vagrant/.bashrc: line 6: syntax error near unexpected token `$'in\r''
'bash: /home/vagrant/.bashrc: line 6: `case $- in
vagrant@trusty64:~$

wich correspond at this case in the .bashrc file :
# If not running interactively, don't do anything
case $- in
    *i*) ;;
      *) return;;
esac

Thanks @mtancoigne for informing us. I will push a fix soon.

No issues for me (Ubuntu + Virtualbox). @alayek @byteknacker can you test this?

@mtancoigne thanks for creating this issue and letting us know about this bug. Would you like to join our slack channel where we actively exchange development details about this project?

@koustuvsinha testing now. I am doing it with zeus.py. But will also try another one just with vagrant up directly from the git clone of this repo.

Guys, the issue appeared with a windows 10 host with vagrant 1.8.1 and virtualBox 5.0.18
Then I tested on a kubuntu 16.04 with vagrant 1.8.1 and virtualBox 5.0.18.
I had to install virtualbox-ext-utils to get USB2 support, but it went fine, no errors.

@mtancoigne If you can, please uninstall Virtualbox from Windows. Then install version 5.0.20 and latest extension pack. Then latest vagrant and try again.

The issue remains on windows with vagrant 1.8.1 and Virtualbox 5.0.20

I see. Let me see what I can do.

@alayek  and @mtancoigne I got a similar issue at the very beginning of the project when vagrant up failed for minimal/trusty64. The reason was old version of virtualbox and most importantly the lack of the Extension Pack. @mtancoigne did you download the extension pack? I think we should add this to the dependencies list on README.md.

Without extension pack, the error would be different. But, yes, it might be older version of extension pack.
@mtancoigne please give another try with

latest Virtualbox
latest extension pack.
latest vagrant

@alayek but how come it is a syntax error? I don't understand this shouldn't be happening. The box is the same for everyone. @mtancoigne can you get inside the vagrant ssh session and do some stuff normally there with terminal?

@mtancoigne please sign up here if you like to join our slack channel to collaborate in real time: https://johnwu.typeform.com/to/ifhehu

Closed by #14


Remove WebStore text in appearance settings

This PR attempts to fix #2197.  The issue only occurs on non-linux builds.
I've marked this as a draft since I am only able to test linux builds at the moment.
If you're able to test this out please comment with the results!

Thanks for testing that out, I guess I'll mark this as ready then!

Thanks!


python312Packages.django-modelcluster: refactor

add changelog to meta
update disabled
enable all tests
remove superfluous commets

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 351595

x86_64-linux

:white_check_mark: 24 packages built:

python311Packages.django-modelcluster
python311Packages.django-modelcluster.dist
python311Packages.permissionedforms
python311Packages.permissionedforms.dist
python311Packages.wagtail
python311Packages.wagtail-factories
python311Packages.wagtail-factories.dist
python311Packages.wagtail-localize
python311Packages.wagtail-localize.dist
python311Packages.wagtail-modeladmin
python311Packages.wagtail-modeladmin.dist
python311Packages.wagtail.dist
python312Packages.django-modelcluster
python312Packages.django-modelcluster.dist
python312Packages.permissionedforms
python312Packages.permissionedforms.dist
python312Packages.wagtail
python312Packages.wagtail-factories
python312Packages.wagtail-factories.dist
python312Packages.wagtail-localize
python312Packages.wagtail-localize.dist
python312Packages.wagtail-modeladmin
python312Packages.wagtail-modeladmin.dist
python312Packages.wagtail.dist


Error: Found transformation with invalid path: components/schemas/metrics/properties/time_series

I cloned the repo and ran the example command, then got this:
> npm run convert -- --output openapi/hcloud.json

><EMAIL_ADDRESS>convert
> ts-node ./src/index.ts --output openapi/hcloud.json

Loading JSON from https://docs.hetzner.cloud/spec.json
Overwriting existing schema description label_selector ("Configuration for type LabelSelector, required if type is `label_selector` | Configuration for type label_selector, required if type is `label_selector` | Label selector used to determine targets. Only present for target type "label_selector". | Configuration for label selector targets, required if type is `label_selector`" -> "Configuration for type LabelSelector, required if type is `label_selector`)"
Overwriting existing schema description protection ("Protection configuration for the Resource." -> "Protection configuration for the Resource)"
Overwriting existing schema description ip_type ("Type of the Floating IP | Floating IP type | The type of the Floating IP | The type of the Primary IP | Type of the Primary IP | Primary IP type" -> "The type of the IP)"
Overwriting existing schema description load_balancer_service ("Request for POST https://api.hetzner.cloud/v1/load_balancers/{id}/actions/add_service" -> "A service for a Load Balancer.)"
Overwriting existing schema description update_load_balancer_service ("Request for POST https://api.hetzner.cloud/v1/load_balancers/{id}/actions/update_service" -> "An update to a service for a Load Balancer.)"
Overwriting existing schema description subnet ("Request for POST https://api.hetzner.cloud/v1/networks/{id}/actions/add_subnet" -> "Subnets divide the ip_range from the parent Network object into multiple Subnetworks that you can use for different specific purposes.)"
Found 17 component entries which only occur once
Error: Found transformation with invalid path: components/schemas/metrics/properties/time_series
    at applyTransformation (/Users/aaa/src/hcloud-openapi/src/document/transformation.ts:20:11)
    at /Users/aaa/src/hcloud-openapi/src/document/transformation.ts:60:5
    at Array.forEach (<anonymous>)
    at Object.<anonymous> (/Users/aaa/src/hcloud-openapi/src/document/transformation.ts:59:19)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/aaa/src/hcloud-openapi/src/document/transformation.ts:5:58)

Is this a bug?

Hello @egze , this project is following the upstream documentation/spec file at https://docs.hetzner.cloud/spec.json .
Sometimes Hetzner performs big changes on their side which break the very manual processing done by the scripts in this repo.
I unfortunately fell a bit behind in the recent months due to time constraints on my side and Hetzner has done some substantial refactoring on their side. I should be able to update and fix the scripts in the upcoming weeks though.
Regarding a more long-term perspective, Hetzner is now introducing components into their own spec file and they also maintain a proper change log now. So I foresee that my project will get deprecated in the near future as it does not bring major benefits over the upstream spec. How long I keep this repo alive depends on how long we need it for hcloud-rust, to my knowledge the only downstream project that uses my spec and the reason why I started building an OpenAPI document for the Hetzner API in the first place.

I updated the spec to the current version from Hetzner, all should be working now again.


enhancement: head function within page route files could return JSX head elements (e.g. meta, title and link tags) instead of an array of objects.

This is a subjective opinion, but if it is possible (I'm not sure what the internal reason for the current API is), then I think that it would be more ergonomic and intuitive to use a similar return as the usual page or component.
e.g.
instead of:
export const head = () => {
  return [
    {
      title: "Hello world!"
    }
  ];
}

we could use:
export const head = () => {
  return <title>Hello world!</title>
}

or instead of:
export const head = () => {
  return [
    {
      tag: "link",
      rel: "stylesheet",
      href: "/fonts/..."
    },
   {
      tag: "meta",
      name: "description",
      content: "This is an application!"
    }
  ];
}

we could use:
export const head = () => {
  return <>
    <link rel="stylesheet" href="/fonts/..." />
    <meta name="description" content="This is an application!" />
  </>
}

Let me know what you think @sjc5! :)

So the reasoning IIRC is because the various nested routes need to "battle" to see which headers will win. Sometimes you'll have duplicates, and the children routes need to take priority in those situations. That's really trivial to do with an array-of-objects pattern (what Remix does as well) and becomes quite a hassle if you need to do it with JSX. Whereas right now, Hwy doesn't do any JSX parsing at all (that's all handled by Hono, or whatever other JSX renderer you may choose such as preact-render-to-string), to do this Hwy would have to start concerning itself with "understanding" JSX, if that makes sense.

So the reasoning IIRC is because the various nested routes need to "battle" to see which headers will win. Sometimes you'll have duplicates, and the children routes need to take priority in those situations. That's really trivial to do with an array-of-objects pattern (what Remix does as well) and becomes quite a hassle if you need to do it with JSX. Whereas right now, Hwy doesn't do any JSX parsing at all (that's all handled by Hono, or whatever other JSX renderer you may choose such as preact-render-to-string), to do this Hwy would have to start concerning itself with "understanding" JSX, if that makes sense.

Ahhh, that's fair enough - it has internal reasons similar to Remix. In that case, it's fine the way it is. I just wasn't sure if it had reasons for it's design internally, and JSX is definitely easier to read and write from an ergonomic point of view. However, it would be much more difficult to parse and understand behind the scenes.
Shall I close this issue then?

Yeah we can close this one -- while I totally agree from a dev perspective, it's not worth the squeeze. And another counter-argument for it is that using standard arrays/objects is a bit easier to do programmatically if you want to as well.
Can you point me to the Remix docs on loader merging @jaymanmdev? I'm not sure what you mean by that. Maybe it's a newer feature.

Yeah we can close this one -- while I totally agree from a dev perspective, it's not worth the squeeze. And another counter-argument for it is that using standard arrays/objects is a bit easier to do programmatically if you want to as well.
Can you point me to the Remix docs on loader merging @jaymanmdev? I'm not sure what you mean by that. Maybe it's a newer feature.

I agree 100%, you can do much more programmatically and there is more flexibility in using object-related data structures for this type of thing.
I might be wrong, but I thought that Remix attempts to merge loader data together into a single object to be retrieved by the 'useLoader' hook. Granted, I might be entirely wrong here as I haven't used Remix all that much. I know that they do the same thing as Hwy in that they run nested loaders in parallel to avoid waterfalls and things of that nature, but I couldn't find any docs regarding merging loader data so I might be wrong haha
Also, since Hwy supports and builds upon Hono's experimental HTML streaming support (server components), which are so freakin' cool and have played a huge part in the development of my Pokémon app, what is the usefulness of loaders? Is it more as an API route interface (same as actions for POST requests and form submissions)? and an alternative way to load data? I know that it is heavily inspired by Remix's patterns. I know that Remix is going to support React Server Components quite soon. I feel like in a lot of cases, loaders wouldn't be the most efficient way to load data, as it blocks the entire page from showing any content at all, which is fixed with streaming.

Loaders will almost always be the most efficient way to load data because it will parallelize all the nested routes loaders.
When you fetch inside of the components, you'll create waterfalls for anything that's nested.
My opinion is that your goal should be for your underlying data fetching to be fast enough that your first page load doesn't feel slow without an intermediate loading state, but you can always also manually parallelize in your parent component with Suspense and then either prop drill or add the results to the Hono context to get parallelized Suspense on that initial page load if that's your goal. IMO usually not the right choice or worth it, but definitely an option.

Yeah, I guess it is really up to personal preference. I like being able to fetch inside of components and choosing where to break up my boundaries. Yes, it can cause layout shift issues with waterfalls, but if you design things correctly, then I think that there are benefits to such a design. There are auto-parallelisation benefits to using loaders too as you said, so I guess it is a per use case-basis type of thing. :)
For example, in my Pokémon app, I was initially fetching the Pokémon data inside of the loader. This would cause the data to take up to a few seconds to load, and if there were fetch errors (which happen occasionally as you know), then it would take even longer. With Suspense I was able to speed up the load time of the header and footer to half a second or so, and then use a loading spinner component to show that more data is loading until successful Pokémon data comes in - I feel like this is much better UX, but that is all up to personal interpretation too.

Yeah several seconds is quite bad, and I agree you need some sort of loading state in that situation.

@sjc5 Just messaging here as I'm not sure where else to contact you - do you have a Discord or something of the sort that you could share for private DMs if needed.
I might spend some time this afternoon tightening up the documentation and working on reworking it if that is okay? If you give me your discord, we can talk more about the design etc. :)


Create group fails

Context
Versions:
cartodb: 4.33.0
Postgres 12 PostGIS 3
cartodb-postgresql 0.35
When trying to create a group an exception is thrown with the follow error:
{"error": {
"spiexceptions.ExternalRoutineException: spiexceptions.ExternalRoutineException: spiexceptions.ExternalRoutineException: TypeError: expected bytes-like object, not str"} }

Screenshot:

Steps to Reproduce
Please break down here below all the needed steps to reproduce the issue
Test via rails console
irb(main):005:0>  group = @organization.create_group(display_name)
[pool] Creating a new connection for localhost:cartodb_user_e6d6f832-7528-4261-96f6-a28e4f78dd8a_db:5432:cartodb_user_e6d6f832-7528-4261-96f6-a28e4f78dd8a:ar (1)
ActiveRecord::StatementInvalid: PG::ExternalRoutineException: ERROR:  spiexceptions.ExternalRoutineException: spiexceptions.ExternalRoutineException: spiexceptions.ExternalRoutineException: TypeError: expected bytes-like object, not str
CONTEXT:  Traceback (most recent call last):
  PL/Python function "_cdb_group_creategroup_api", line 7, in <module>
    plpy.execute(query)
PL/Python function "_cdb_group_creategroup_api"
SQL statement "SELECT cartodb._CDB_Group_CreateGroup_API(group_name, group_role)"
PL/pgSQL function cdb_group_creategroup(text) line 7 at PERFORM
:  select cartodb.CDB_Group_CreateGroup('thedudes')
	from /var/lib/gems/2.4.0/gems/activerecord-4.2.11/lib/active_record/connection_adapters/postgresql/database_statements.rb:155:in `async_exec'

@CartoDB/rt-managers

@gonzaloriestra unfortunately RT doesn't provide support for open source installations. @sabman you might want to try posting your issue in CARTO's Google Group, support for Open Source happens there: https://groups.google.com/forum/#!forum/cartodb

This is a bug report not a support ticket with version numbers close to bleeding edge. If it's not a bug would you care to elaborate on your message as to why?

@sabman sorry, but in carto.com it's working fine and we are not able to reproduce it, so it's probably an installation/configuration issue.

I think is a bug because it happens also to me on more than one instance!
always same errors. So or it's a bug or there is some error in the guide to install carto.

Ok, I'll add this to our backlog to have a look when we can. Thanks!

ClubHouse story: https://app.clubhouse.io/cartoteam/story/83570/create-group-fails


[BugFix] Fix null pointer exception in sigterm_handler function

Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[ ] 2.4

@mergify backport branch-3.1 branch-3.0 branch-2.5

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 3 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/common/daemon.cpp
0
3
00.00%
[216, 217, 219]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


Filter out geometries with a small area

Refs mapzen/vector-datasource#151

This is based on the branch from #26.
@zerebubuth review?


Add integration test for KafkaConnectSource

In PR #2791, @sijie suggested to add an integration test for debezium.
KafkaConnectSource is a little not compatible with current source test framework.
In current existing source test framework, we call produceSourceMessages: Map<String, String> kvs = tester.produceSourceMessages(numMessages) and return expected kvs to verify later,
But in this KafkaConnectSource, the SourceMessages is created by debezium (convert mysql binlog into KafkaConnectSource), And it seems not easy to save the content as explicit Kvs.
To do this, we may need move some of the methods(such as  prepareSource and validateSourceResult) from PulsarFunctionsTest into SourceTester to make the SourceTester do the things more customized.
This is only an initial thought,  this issue is to track the integration test for KafkaConnectSource

@jiazhai any updates on this?


修复首次安装报错

@all-contributors please add @chengzhongxue for infrastructure, tests and code

@all-contributors please add @chengzhongxue for infrastructure, tests and code


CDN URL was analyze as a local path.

Describe the bug
I try to use my own animation engine to init the CanvasKit-WASM, when I run pnpm dev, it's all OK. But when I try to build it, it throws a Error:
RuntimeError: Aborted(Error: ENOENT: no such file or directory, open 'C:\Users\acbox\Desktop\projects\newcar-docs\https:\unpkg.com\canvaskit-wasm@latest\bin\canvaskit.wasm'). Build with -sASSERTIONS for more info.

Is that I use it with a wrong way or it's a bug?
Reproduction
https://github.com/Bug-Duck/newcar-docs/blob/main/docs/examples/Demo1.vue
Steps to reproduce
No response
System Info
System: Windows11
Node: v20.12.1

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I wasn't able to reproduce. The steps I tried are:

pnpm i
pnpm docs:build
pnpm docs:preview
open http://localhost:4173/examples

Emm...I asked GPT and find the answer. It must be mounted so that it can analyze directly


rememberedUserSessionDuration of false causes crash

The craftCms config value of rememberedUserSessionDuration being set to false instead of a time (eg. 'P1D') causes the following php error.
Exception: DateInterval::__construct(): Unknown or bad format () in /var/www/craft/plugins/twofactorauthentication/services/TwoFactorAuthentication_VerifyService.php:41
(see source)
Perhaps a sane default value for $sessionDuration when rememberedUserSessionDuration is false would be good? A few hours or a day perhaps?

@brybeecher thanks, i'll look into this.

fixed in 8571b5e8117760ceea606938c40c95215ecfd354


Multiple TypeConveters ?

Hi, wondering if it would be possible to have multiple type converter attributes applied to a property, and have them executed in order ?
For example, may be useful to have a more generic type converter first: saved value to IEnumerable<IPublishedContent> and then a more specific type converter to be executed after converting the IEnumerable<IPublishedContent> to IEnumerable<ProjectSpecificPOCO>
[RelationsTypeConverter("exampleRelationTypeAlias")] // (supplies param to RelationsTypeConverter)
[TypeConverter(typeof(RelationsTypeConverter)]
[TypeConverter(typeof(ProjectSpecificTypeConverter))]
public IEnumerable<MyPoco> MyPocos { get; set; }

Wouldn't you just have your upper typeconverter inherit from the base one?
Will have to check what type converters are meant to do as standard,
whether that is supported, but I would think inheritance and a single
attribute would be the way.
On 15 Mar 2015 12:58, "Hendy Racher"<EMAIL_ADDRESS>wrote:

Hi, wondering if it would be possible to have multiple type converter
attributes applied to a property, and have them executed in order ?
For example, may be useful to have a more generic type converter first,
value -> IPublishedContent and then a more specific type converter to be
executed after IPublishedContent -> projectSpecific POCO
[RelationsTypeConverter("exampleRelationTypeAlias")] // (supplies param to
RelationsTypeConverter)
[TypeConverter(typeof(RelationsTypeConverter)]
[TypeConverter(typeof(ProjectSpecificTypeConverter))]
public IEnumerable MyPocos { get; set; }
—
Reply to this email directly or view it on GitHub
https://github.com/leekelleher/umbraco-ditto/issues/50.

@mattbrailsford yup, that makes sense, although multiple TCs would mean that the 2nd TC doesn't need to know anything about the 1st, so potentially less combinations of specific all-in-one-TCs might be required.
I wonder if forcing the use of only the first TC would result in more verbose, but cleaner code ?

I'm afraid you can't add multiple TypeConverter attributes to a class or property. It won't compile.

arh ha, well that resolves that idea :)

@Hendy An alternative would be to look at using ConvertedType event hook (or Func delegate)?
Then you can add in any project-specific logic in there.


⚠ Guya.moe Proxied has degraded performance

In 7985884, Guya.moe Proxied (https://ice.guya.moe/) experienced degraded performance:

HTTP code: 200
Response time: 13081 ms

Resolved: Guya.moe Proxied performance has improved in 9f7b8a7 after 10 days, 23 hours, 58 minutes.


chore(main): release 1.3.3

:robot: I have created a release beep boop
1.3.3 (2024-06-05)
Bug Fixes

Pipeline: Try..catch around the pipeline execution to catch all errors that can occur and exit the program with an error (fc0da2a)
The pipeline job is not stopped in case of error (#27) (1ee95b8)

Miscellaneous

Add a wildcard for the patch number of openhexa.toolbox (3d13ef9)
Removing the fuse unmount (done by the system) (9c57eb6)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-docker-images/releases/tag/1.3.3 :sunflower: