Repo-map can't include file and doesn't ignore the file

Issue
I'm getting an error for "largefolder". I exluded it in the .aiderignore and I tried --subtree-only both didn't work and give me the following output. Anyone an idea how to fix it?
Repo-map can't include /Users/user/dev/project-a/largefolder, it is not a normal file

Unexpected error: litellm.APIConnectionError: b'{"type":"error","error":{"type":"overloaded_error","message":"Overloaded"}}'
Traceback (most recent call last):
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 10080, in __next__
    self.fetch_sync_stream()
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 10175, in fetch_sync_stream
    self.completion_stream = self.make_call(client=litellm.module_level_client)
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/llms/anthropic.py", line 577, in make_sync_call
    raise AnthropicError(status_code=response.status_code, message=response.read())
litellm.llms.anthropic.AnthropicError: b'{"type":"error","error":{"type":"overloaded_error","message":"Overloaded"}}'

Traceback (most recent call last):
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 10080, in __next__
    self.fetch_sync_stream()
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 10175, in fetch_sync_stream
    self.completion_stream = self.make_call(client=litellm.module_level_client)
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/llms/anthropic.py", line 577, in make_sync_call
    raise AnthropicError(status_code=response.status_code, message=response.read())
litellm.llms.anthropic.AnthropicError: b'{"type":"error","error":{"type":"overloaded_error","message":"Overloaded"}}'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/aider/coders/base_coder.py", line 900, in send_new_user_message
    yield from self.send(messages, functions=self.functions)
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/aider/coders/base_coder.py", line 1176, in send
    yield from self.show_send_output_stream(completion)
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/aider/coders/base_coder.py", line 1250, in show_send_output_stream
    for chunk in completion:
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 10166, in __next__
    raise exception_type(
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 8139, in exception_type
    raise e
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 8103, in exception_type
    raise APIConnectionError(
litellm.exceptions.APIConnectionError: litellm.APIConnectionError: b'{"type":"error","error":{"type":"overloaded_error","message":"Overloaded"}}'
Traceback (most recent call last):
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 10080, in __next__
    self.fetch_sync_stream()
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/utils.py", line 10175, in fetch_sync_stream
    self.completion_stream = self.make_call(client=litellm.module_level_client)
  File "/Users/user/opt/anaconda3/lib/python3.9/site-packages/litellm/llms/anthropic.py", line 577, in make_sync_call
    raise AnthropicError(status_code=response.status_code, message=response.read())
litellm.llms.anthropic.AnthropicError: b'{"type":"error","error":{"type":"overloaded_error","message":"Overloaded"}}'

Version and model info
Aider v0.48.1
Models: claude-3-5-sonnet-20240620 with diff edit format, weak model claude-3-haiku-20240307
Git repo: .git with 170 files
Repo-map: using 1024 tokens

Happens to me also since updated to 0.48.1 this morning.
Worked perfectly yesterday in the same repository.

Same here!!:

Happens to me also since updated to 0.48.1 this morning. Worked perfectly yesterday in the same repository.

Hah, it seems to be a problem with the Claude Sonnet Model when used via API. Wanted to use sonnet with fabric and get also the error
Error: Fabric is working fine, but claude is overloaded. Please try again later.
Error: Error code: 529 - {'type': 'error', 'error': {'type': 'overloaded_error', 'message': 'Overloaded'}}

Seems to be unrelated to aider.chat.

Sorry, the logs include two issues. One is the Claude api issue, this is unrelated to aider. The other issue is the large folder issue. I did mean the folder issue.

experiencing the same issue since the update.

These errors are because Anthropic's API servers are overloaded/down. Aider is simply reporting that the server is broken.
Anthropic has been having serious issues since yesterday. You can directly check their server status here:
https://status.anthropic.com/
The file is not being included in the repo map because it is not a normal file. Is it a broken symlink perhaps?

@paul-gauthier
Thanks a lot for your reply.

I get a similar error, not because of API overload, but because I use a lot of symlinks in my repo.
Is there a way (command-line argument or env variable) to suppress the:
Repo-map can't include x
message?
Thanks


More of a usability question.  I am displaying to bubble data side by side with | and want green color for left and red for right.  How to do that?

You can use styles to do this:
data('sample:US States.csv') bubble label(abbr) size(population) style("fill:green") | bubble size(income) style("fill:red")
For docs, see the "Guides & Style" section in the tutorial:  http://brunel.mybluemix.net/docs/


V4:  Fast Refresh is inconsistent

Describe the bug
Adding new utility classes during development does not consistently reflect changes.  Reloading does not correct it either.  I have to kill the development server and reopen the app.
Is there a configuration option to address this?  Is this supposed to be the behavior?
Occurs on both Expo Go & Expo custom development client.
The correct styling is reflected after closing the development server and restarting the server via npx expo start -c and reopening the dev client.
Expected behavior
Adding utility classes in the classNames field of a component should "Fast Refresh" and reflect the changes without requiring the developer to close the
Screenshots
e.g. of usage:
<View className={'flex max-w-screen-lg flex-1 items-center justify-center gap-3'}>
     ...
</View>

Screen shown when dev client first started and running:

Changing gap-3 to gap-5 while development server is running:

You can see the gap class utility is completely ignored now.
Stopping the development server.  Re-run npx expo start -c --dev-client

gap-5 is now accurately reflected.
Conclusion?
There seems to be an issue with the Live Reloading feature when it comes to the tailwind class utilities.  This may just be a configuration issue on my part but reading through the docs I can't find anything specifically that would indicate why that would be.

Related to #643 and #653 . It is a bug in Metro cache and will be fixed in a future release of Expo

Is there any rough ETA on when this will occur?

This might be fixed in 4.0.14. If you'd like an ETA can you please provide a production, as it maybe a configuration issue or something else. Screenshots provide very minimal value when troubleshooting code

@marklawlor I'll try upgrading to 4.0.14 first.  If I am still having issues I'll create a public project with the symptoms.  I would have done that sooner but I am in a project I can't share publicly.  But I'll remove the pieces I can't share and create a reproducible example if I continue to run into issues.

@marklawlor Here is a templated project I have created that shows the above issue after upgrading to 4.0.14.   It occurs both in Expo Go and Custom Development Client.
You can see this by going to app/(aut)/index.tsx and changing the following:
        <Text className={'text-6xl font-bold text-lightTextTheme dark:text-darkTextTheme'}>
          Examples:
        </Text>

to
        <Text className={'text-6xl font-bold bg-red-500'}>
          Examples:
        </Text>

Change is not reflected even though the client shows the app has "Refreshing..."

I'm running into a related issue where if I change my theme's colours (located in a colors.js that is imported into tailwind.config.js), I have to restart the metro bundler + restart the app to see the changed color. However, the colour in the autocomplete would still show the old colour, and I'm not sure how to get the autocomplete to "refresh" with my updated theme colour.

@marklawlor Here is a templated project I have created that shows the above issue after upgrading to 4.0.14. It occurs both in Expo Go and Custom Development Client.
You can see this by going to app/(auth)/index.tsx and changing the following:
        <Text className={'text-6xl font-bold text-lightTextTheme dark:text-darkTextTheme'}>
          Examples:
        </Text>

to
        <Text className={'text-6xl font-bold bg-red-500'}>
          Examples:
        </Text>

Change is not reflected even though the client shows the app has "Refreshing..."

@marklawlor Just realizing that I didn't ever link the actual repo that I created as a template:  https://github.com/rally2/exporouter-nativewind .
Happy Holidays!

I hope this is fixed soon. I also experience this issue with EAS and on Expo SDK 48 and using nativewind v4

It's even worse using react-native-windows. Even tho the generated css file in the cache folder is updated correctly always, the change is not reflected 99% of the time. A full metro server restart is required.
Steps to reproduce
npx react-native init myproject
npx react-native-windows-init --overwrite
# install NativeWind v4
npx react-native run-windows

Then try to create a component using bg-red-400 for example, which won't work, until you re-save global.css, and restart the metro server.
The experience is terrible.

I have to agree with @josetr. A fix is really necessary.

Any updates on this? It's a really bad DX

It seems to be fixed with latest nativewind 4.0.23.

@minhnc For me its still not working in version 4.0.23

Version 4.0.23 didn't fixed it for me either.

Working fine for me. Im with MacOS btw. Do you mind sharing a sample project?

@minhnc Sample project linked above in my comment.  When I add !bg-red-500 as an override, there is no refresh to reflect the changes.

@ShaneKeney hope this helps ( cloned your sample project )

@minhnc Thanks.  I took a look and noticed your running this on MacOS.  I think this is an issue isolated to when leveraging WSL2.  I have my Macbook where this issue does not happen but on my PC running WSL2 it does.

Please try 4.0.26. The dev server was reworked to hook into the existing FastRefresh HTTP server.

@marklawlor Thank you!  It is looking like the issue has resolve itself with the update to 4.0.26.  I'll continue to test it out but the initial example using !bg-red-500 that was not working is now fixed which is a sign in the right direction.
Thanks for all the hard work!

Closing issue, if your still experiencing this issue please open a new ticket


Fail to build when using Python 3.10.x

System information

OS Platform and Distribution (e.g., Linux Ubuntu 20.04, WSL2): Windows 10 Pro
Target (e.g. desktop cpu, android arm64): desktop cpu
Bazel version: bazel 3.7.2
Python version: Python 3.10.0
GCC/G++ version: -
Unity version: Unity 2020.3.13f1
Android NDK version (if building for Android):
Xcode version (if building for iOS):

Describe the problem
I am getting error while running "python build.py build --desktop cpu" command.
ERROR: An error occurred during the fetch of repository 'local_execution_config_python':
Traceback (most recent call last):
File "C:/_bzl/dolx24fw/external/org_tensorflow/third_party/py/python_configure.bzl", line 212, column 41, in _create_local_python_repository
python_include = _get_python_include(repository_ctx, python_bin)
File "C:/_bzl/dolx24fw/external/org_tensorflow/third_party/py/python_configure.bzl", line 152, column 21, in _get_python_include
result = execute(
File "C:/_bzl/dolx24fw/external/org_tensorflow/third_party/remote_config/common.bzl", line 230, column 13, in execute
fail(
Error in fail: Problem getting python include path.
:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
Is the Python binary path set up right? (See ./configure or PYTHON_BIN_PATH.) Is distutils installed?
Steps to reproduce the issue
Provide the exact sequence of commands / steps that you executed before running into the problem.
I am following steps from the github page / readme.md file.
Full logs
INFO (build.py): Building protobuf sources...
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_absl' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_protobuf' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_google_googletest' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'com_github_gflags_gflags' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'rules_python' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_rules_apple' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_rules_swift' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'build_bazel_apple_support' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'bazel_skylib' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/org_tensorflow/third_party/repo.bzl:108:14:
Warning: skipping import of repository 'pybind11' because it already exists.
DEBUG: C:/_bzl/dolx24fw/external/tf_runtime/third_party/cuda/dependencies.bzl:51:10: The following command will download NVIDIA proprietary software. By using the software you agree to comply with the terms of the license agreement that accompanies the software. If you do not agree to the terms of the license agreement, do not use the software.
INFO: Repository local_execution_config_python instantiated at:
D:/unity/mediapipeunityplugin-master/mediapipeunityplugin-master/WORKSPACE:465:14: in
C:/_bzl/dolx24fw/external/org_tensorflow/tensorflow/workspace2.bzl:1087:19: in workspace
C:/_bzl/dolx24fw/external/org_tensorflow/tensorflow/workspace2.bzl:85:27: in _tf_toolchains
C:/_bzl/dolx24fw/external/tf_toolchains/toolchains/remote_config/configs.bzl:6:28: in initialize_rbe_configs
C:/_bzl/dolx24fw/external/tf_toolchains/toolchains/remote_config/rbe_config.bzl:158:27: in _tensorflow_local_config
Repository rule local_python_configure defined at:
C:/_bzl/dolx24fw/external/org_tensorflow/third_party/py/python_configure.bzl:275:41: in
ERROR: An error occurred during the fetch of repository 'local_execution_config_python':
Traceback (most recent call last):
File "C:/_bzl/dolx24fw/external/org_tensorflow/third_party/py/python_configure.bzl", line 212, column 41, in _create_local_python_repository
python_include = _get_python_include(repository_ctx, python_bin)
File "C:/_bzl/dolx24fw/external/org_tensorflow/third_party/py/python_configure.bzl", line 152, column 21, in _get_python_include
result = execute(
File "C:/_bzl/dolx24fw/external/org_tensorflow/third_party/remote_config/common.bzl", line 230, column 13, in execute
fail(
Error in fail: Problem getting python include path.
:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
Is the Python binary path set up right? (See ./configure or PYTHON_BIN_PATH.) Is distutils installed?
INFO: Repository com_google_protobuf instantiated at:
D:/unity/mediapipeunityplugin-master/mediapipeunityplugin-master/WORKSPACE:184:13: in
Repository rule http_archive defined at:
C:/_bzl/dolx24fw/external/bazel_tools/tools/build_defs/repo/http.bzl:336:31: in
ERROR: Analysis of target '//mediapipe_api:mediapipe_proto_srcs' failed; build aborted: Problem getting python include path.
:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead
Is the Python binary path set up right? (See ./configure or PYTHON_BIN_PATH.) Is distutils installed?
INFO: Elapsed time: 0.910s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded, 0 targets configured)
currently loading: @com_google_mediapipe//mediapipe/framework ... (7 packages)
Traceback (most recent call last):
File "D:\Unity\MediaPipeUnityPlugin-master\MediaPipeUnityPlugin-master\build.py", line 444, in
Argument().command().run()
File "D:\Unity\MediaPipeUnityPlugin-master\MediaPipeUnityPlugin-master\build.py", line 112, in run
self._run_command(self._build_proto_srcs_commands())
File "D:\Unity\MediaPipeUnityPlugin-master\MediaPipeUnityPlugin-master\build.py", line 52, in _run_command
return subprocess.run(' '.join(command_list), check=True, shell=shell)
File "C:\Users\Desire\AppData\Local\Programs\Python\Python310\lib\subprocess.py", line 524, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'bazel --output_user_root C:/_bzl build -c opt --action_env PYTHON_BIN_PATH="C://Users//Desire//AppData//Local//Programs//Python//Python310//python.exe" --action_env ProgramData --action_env PROCESSOR_ARCHITECTURE --action_env PROCESSOR_IDENTIFIER --action_env PROCESSOR_LEVEL --action_env PROCESSOR_REVISION //mediapipe_api:mediapipe_proto_srcs' returned non-zero exit status 1.
Additional context
Add any other context about the problem here.

@Arccus-games How did you solve the problem?


Provide Conversion from url::Url to hyper::Uri

Is your feature request related to a problem? Please describe.
I'm always frustrated when I have a url::Url and I need a hyper::Uri
Describe the solution you'd like
Provide an impl From<url::Url> for hyper::Uri
Describe alternatives you've considered
Doing it myself every time.
let uri: Uri = url.as_str().parse()?;

Additional context
Previously requested by someone else as well a few years ago but then closed. https://github.com/hyperium/hyper/issues/1219

AFAIK, hyper uses http::uri and does not use the url crate as a dependency. I would take this on, if the maintainers are not opposed to it.

The issue you linked to has good context, as well as this one: https://github.com/hyperium/http/issues/73
In short, we don't want to tie the stability of url to hyper.


quote function not random "enough"

math.random(#quotes) isn't truly random.

Will soon implement my slightly better random function, however this isn't important as there's no security necessary for this plugin.


Crashes on Nougat

Crashs for me under android 7.0 (nougat)…
my phone specs:
Qualcomm Snapdragon 1.4 GHz Octa-Core
armeabi-v7a
1920x1080 pixels (386ppi)

Could you

describe what you were doing
specify the version you are using
try with the most recent version available (1.31, or 1.32 when it becomes available)?

Version 1.34…
When resume game…
Play selected level.…
Still crashes under Android Nougat with armeabi-v7a but seems to work on my Arm64-v8a tablet with Nougat and a 800 × 1280 screen size.

version 1.35
Android Nougat - armeabi-v7a
Got up to level 6 but when I try to hire a hero on level 7 it crashes.
I have disabled the background, have show attackers in range swiched on and I m using large objects.

I suppose that has been fixed. I'm closing the issue. Please open a new one if there are more crashes.


all nodes are under target utilization, nothing to do here

Given the following policy:
apiVersion: "descheduler/v1alpha1"
kind: "DeschedulerPolicy"
strategies:
  "LowNodeUtilization":
     enabled: true
     params:
       nodeResourceUtilizationThresholds:
         thresholds:
           "cpu" : 50
           "memory": 50
           "pods": 10
         targetThresholds:
           "cpu" : 50
           "memory": 50
           "pods": 50

I am confused by this output:
./_output/bin/descheduler --kubeconfig ~/.kube/config --policy-config-file policy.yaml --node-selector beta.kubernetes.io/instance-type=n1-highmem-4 -v 4
I0814 11:56:02.699491   27948 reflector.go:202] Starting reflector *v1.Node (1h0m0s) from github.com/kubernetes-incubator/descheduler/pkg/descheduler/node/node.go:84
I0814 11:56:02.699629   27948 reflector.go:240] Listing and watching *v1.Node from github.com/kubernetes-incubator/descheduler/pkg/descheduler/node/node.go:84
I0814 11:56:02.799560   27948 node.go:51] node lister returned empty list, now fetch directly
I0814 11:56:04.839125   27948 request.go:480] Throttling request took 122.384366ms, request: GET:https://x.x.x.x.x/api/v1/pods?fieldSelector=spec.nodeName%3Dgke-node-bf2a5a1e-mr9b%2Cstatus.phase%21%3DFailed%2Cstatus.phase%21%3DSucceeded
I0814 11:56:05.039313   27948 request.go:480] Throttling request took 82.857788ms, request: GET:https://x.x.x.x.x/api/v1/pods?fieldSelector=spec.nodeName%3Dgke-node-bf2a5a1e-ps8k%2Cstatus.phase%21%3DFailed%2Cstatus.phase%21%3DSucceeded
I0814 11:56:05.239153   27948 request.go:480] Throttling request took 65.339548ms, request: GET:https://x.x.x.x.x/api/v1/pods?fieldSelector=spec.nodeName%3Dgke-node-bf2a5a1e-qcpq%2Cstatus.phase%21%3DFailed%2Cstatus.phase%21%3DSucceeded
I0814 11:56:05.439252   27948 request.go:480] Throttling request took 126.223138ms, request: GET:https://x.x.x.x.x/api/v1/pods?fieldSelector=spec.nodeName%3Dgke-node-bf2a5a1e-qg9g%2Cstatus.phase%21%3DFailed%2Cstatus.phase%21%3DSucceeded
I0814 11:56:05.639253   27948 request.go:480] Throttling request took 128.039815ms, request: GET:https://x.x.x.x.x/api/v1/pods?fieldSelector=spec.nodeName%3Dgke-node-bf2a5a1e-tw62%2Cstatus.phase%21%3DFailed%2Cstatus.phase%21%3DSucceeded
I0814 11:56:05.839299   27948 request.go:480] Throttling request took 111.435987ms, request: GET:https://x.x.x.x.x/api/v1/pods?fieldSelector=spec.nodeName%3Dgke-node-bf2a5a1e-w7n7%2Cstatus.phase%21%3DFailed%2Cstatus.phase%21%3DSucceeded
I0814 11:56:05.912036   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-kfh8" is under utilized with usage: api.ResourceThresholds{"cpu":21.428571428571427, "memory":7.093371019678181, "pods":7.2727272727272725}
I0814 11:56:05.912091   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:6, bePods:0, bPods:2, gPods:0
I0814 11:56:05.912148   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-mr9b" is under utilized with usage: api.ResourceThresholds{"pods":9.090909090909092, "cpu":33.92857142857143, "memory":3.773439230136872}
I0814 11:56:05.912160   27948 lownodeutilization.go:149] allPods:10, nonRemovablePods:7, bePods:0, bPods:3, gPods:0
I0814 11:56:05.912219   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-qcpq" is under utilized with usage: api.ResourceThresholds{"cpu":15.561224489795919, "memory":7.322094578473096, "pods":9.090909090909092}
I0814 11:56:05.912230   27948 lownodeutilization.go:149] allPods:10, nonRemovablePods:6, bePods:0, bPods:4, gPods:0
I0814 11:56:05.912264   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-qg9g" is under utilized with usage: api.ResourceThresholds{"cpu":27.551020408163264, "memory":7.681984428891371, "pods":7.2727272727272725}
I0814 11:56:05.912273   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:6, bePods:0, bPods:2, gPods:0
I0814 11:56:05.912513   27948 lownodeutilization.go:147] Node "gke-node-bf2a5a1e-9b15" is appropriately utilized with usage: api.ResourceThresholds{"memory":13.213583436348788, "pods":10.909090909090908, "cpu":22.372448979591837}
I0814 11:56:05.912537   27948 lownodeutilization.go:149] allPods:12, nonRemovablePods:6, bePods:0, bPods:6, gPods:0
I0814 11:56:05.912613   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-cs2l" is under utilized with usage: api.ResourceThresholds{"memory":2.9337443495765223, "pods":7.2727272727272725, "cpu":27.551020408163264}
I0814 11:56:05.912631   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:6, bePods:0, bPods:2, gPods:0
I0814 11:56:05.912749   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-dk7j" is under utilized with usage: api.ResourceThresholds{"cpu":15.051020408163266, "memory":6.403485637657102, "pods":7.2727272727272725}
I0814 11:56:05.912770   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:6, bePods:1, bPods:1, gPods:0
I0814 11:56:05.912828   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-ggdw" is under utilized with usage: api.ResourceThresholds{"cpu":19.387755102040817, "memory":1.9656336753240788, "pods":7.2727272727272725}
I0814 11:56:05.912855   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:6, bePods:0, bPods:2, gPods:0
I0814 11:56:05.912897   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-tw62" is under utilized with usage: api.ResourceThresholds{"cpu":20.918367346938776, "memory":2.4510629446719583, "pods":6.363636363636363}
I0814 11:56:05.912909   27948 lownodeutilization.go:149] allPods:7, nonRemovablePods:5, bePods:0, bPods:2, gPods:0
I0814 11:56:05.920135   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-3rt5" is under utilized with usage: api.ResourceThresholds{"cpu":27.551020408163264, "memory":7.68198179540342, "pods":7.2727272727272725}
I0814 11:56:05.920172   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:6, bePods:0, bPods:2, gPods:0
I0814 11:56:05.920269   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-50rb" is under utilized with usage: api.ResourceThresholds{"cpu":28.316326530612244, "memory":8.43523410990875, "pods":10}
I0814 11:56:05.920288   27948 lownodeutilization.go:149] allPods:11, nonRemovablePods:6, bePods:0, bPods:5, gPods:0
I0814 11:56:05.920354   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-hmq1" is under utilized with usage: api.ResourceThresholds{"cpu":27.806122448979593, "memory":7.93306590023853, "pods":8.181818181818182}
I0814 11:56:05.920370   27948 lownodeutilization.go:149] allPods:9, nonRemovablePods:6, bePods:0, bPods:3, gPods:0
I0814 11:56:05.920444   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-ps8k" is under utilized with usage: api.ResourceThresholds{"cpu":27.040816326530614, "memory":7.1798135857332, "pods":5.454545454545454}
I0814 11:56:05.920467   27948 lownodeutilization.go:149] allPods:6, nonRemovablePods:6, bePods:0, bPods:0, gPods:0
I0814 11:56:05.920580   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-w7n7" is under utilized with usage: api.ResourceThresholds{"cpu":23.724489795918366, "memory":4.948809588699222, "pods":8.181818181818182}
I0814 11:56:05.920632   27948 lownodeutilization.go:149] allPods:9, nonRemovablePods:5, bePods:0, bPods:4, gPods:0
I0814 11:56:05.920674   27948 lownodeutilization.go:147] Node "gke-node-bf2a5a1e-1t8l" is appropriately utilized with usage: api.ResourceThresholds{"memory":0.8776025543719349, "pods":3.6363636363636362, "cpu":7.653061224489796}
I0814 11:56:05.920690   27948 lownodeutilization.go:149] allPods:4, nonRemovablePods:4, bePods:0, bPods:0, gPods:0
I0814 11:56:05.920733   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-fjbd" is under utilized with usage: api.ResourceThresholds{"pods":5.454545454545454, "cpu":27.040816326530614, "memory":7.1798135857332}
I0814 11:56:05.920745   27948 lownodeutilization.go:149] allPods:6, nonRemovablePods:6, bePods:0, bPods:0, gPods:0
I0814 11:56:05.921125   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-kxb8" is under utilized with usage: api.ResourceThresholds{"cpu":27.29591836734694, "memory":7.43089769056831, "pods":6.363636363636363}
I0814 11:56:05.921145   27948 lownodeutilization.go:149] allPods:7, nonRemovablePods:6, bePods:0, bPods:1, gPods:0
I0814 11:56:05.921205   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-l4xf" is under utilized with usage: api.ResourceThresholds{"memory":2.8898642218579256, "pods":7.2727272727272725, "cpu":22.193877551020407}
I0814 11:56:05.921220   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:6, bePods:0, bPods:2, gPods:0
I0814 11:56:05.921279   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-cvsc" is under utilized with usage: api.ResourceThresholds{"cpu":8.418367346938776, "memory":1.4256836686734968, "pods":7.2727272727272725}
I0814 11:56:05.921297   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:5, bePods:1, bPods:1, gPods:1
I0814 11:56:05.921388   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-kntr" is under utilized with usage: api.ResourceThresholds{"cpu":12.525510204081632, "memory":3.879292539212722, "pods":6.363636363636363}
I0814 11:56:05.921407   27948 lownodeutilization.go:149] allPods:7, nonRemovablePods:5, bePods:0, bPods:2, gPods:0
I0814 11:56:05.921478   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-5415" is under utilized with usage: api.ResourceThresholds{"cpu":21.428571428571427, "memory":7.093371019678181, "pods":7.2727272727272725}
I0814 11:56:05.921495   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:6, bePods:0, bPods:2, gPods:0
I0814 11:56:05.921562   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-5dd0" is under utilized with usage: api.ResourceThresholds{"cpu":27.806122448979593, "memory":7.93306590023853, "pods":8.181818181818182}
I0814 11:56:05.921580   27948 lownodeutilization.go:149] allPods:9, nonRemovablePods:6, bePods:0, bPods:3, gPods:0
I0814 11:56:05.921636   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-74z1" is under utilized with usage: api.ResourceThresholds{"cpu":21.1734693877551, "memory":2.7021479758398677, "pods":7.2727272727272725}
I0814 11:56:05.921652   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:5, bePods:0, bPods:3, gPods:0
I0814 11:56:05.923369   27948 lownodeutilization.go:141] Node "gke-node-bf2a5a1e-bvsh" is under utilized with usage: api.ResourceThresholds{"cpu":21.1734693877551, "memory":2.7021470495070683, "pods":7.2727272727272725}
I0814 11:56:05.923431   27948 lownodeutilization.go:149] allPods:8, nonRemovablePods:5, bePods:0, bPods:3, gPods:0
I0814 11:56:05.923442   27948 lownodeutilization.go:65] Criteria for a node under utilization: CPU: 50, Mem: 50, Pods: 10
I0814 11:56:05.923478   27948 lownodeutilization.go:72] Total number of underutilized nodes: 22
I0814 11:56:05.923493   27948 lownodeutilization.go:85] all nodes are under target utilization, nothing to do here

According to the descheduler, Total number of underutilized nodes: 22 and all nodes are under target utilization, yet nothing to do here. None of my underutilized nodes get drained.
How can I instruct the descheduler to drain the underutilized nodes?

@huguesalary descheduler evicts pods from overutilized nodes (decided by thresholds) to move those pods to less utilized nodes so that cluster can get more balanced. Since based on your thresholds (50,50,50), all nodes are under threshold, so there is no need to evict any pods.

Thanks, I re-read the documentation and understand better.
What I was trying to achieve was to effectively rebalance my cluster such that some underutilized nodes would get fully drained, triggering pods to be rescheduled on some other underutilized nodes.
My end goal is to kill some underutilized nodes, better utilizing other nodes and saving money.

@huguesalary the cluster-autoscaler might work for your use case (better utilizing other nodes and saving money).

--scale-down-utilization-threshold float                             Sum of cpu or memory of all pods running on the node divided by node's corresponding allocatable resource, below which a node can be considered for scale down (default 0.5)

While cluster-autoscaler is able to concentrate the load in fewer nodes, I'd say that descheduler does the opposite: spread the load among them.


[SPARK-37987][SS] Fix flaky test StreamingAggregationSuite.changing schema of state when restarting query

What changes were proposed in this pull request?
This PR fixes a flaky test StreamingAggregationSuite.changing schema of state when restarting query, via adjusting the number of shuffle partition to 1.
The flakiness was due to the optimization on schema verification - we only verify it in partition 0 since it is costly and redundant to verify the schema for all partitions. Other partitions are still possible to provide other errors which are considered as unexpected.
Why are the changes needed?
This PR fixes a flaky test.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Ran test suite 10 times locally.

cc. @HyukjinKwon

Thanks man!

Thanks! Merging to master/3.2.


An error occurs when throwing a potion in large quantities

Spigot 1.12.2
https://pastebin.com/sUfDxEhS

Does this happen on the latest version of Panilla? 1.4.8-fix

YES

On the latest version, the exact same error

https://pastebin.com/PennMpds

Is there an easy way to replicate this with potions like you said? I briefly tried what you described with spamming a lot of potions with no luck.

Is there an easy way to replicate this with potions like you said? I briefly tried what you described with spamming a lot of potions with no luck.

Is there an easy way to replicate this with potions like you said? I briefly tried what you described with spamming a lot of potions with no luck.


syn = "2" support

Syn has a new version 2. However the syn-error (and syn) features bring in syn v1.
https://github.com/CreepySkeleton/proc-macro-error/blob/75f7d521387b58e20960fdf79010ea23d753d56d/Cargo.toml#L27-L30

syn2 support is available at https://crates.io/crates/proc-macro-error2


Ensure that crypto allocations are using BufferSource

... ensures that all ArrayBuffer instances are using BufferSource rather than external allocations.

A handful of the remaining uses of kj::heapArray could actually be converted to KJ_STACK_ARRAY but will handle that cleanup in a separate PR.


fix: check that calculated gas price is a number

What it solves
Resolves down oracles
How this PR fixes it
Gas price oracles that are down, namely Rinkeby (atm), do not throw when there is an issue but return a string. This was not breaking out of the oracle estimation loop we use.
The fetced gas price is first checked (!isNaN()), throwing if so, before returning it.
How to test it
This is very hard to test because manually switching to another oracle in Django will likely throw. You would likely have to mock the response of an oracle, returning a string (that is not a number).
This is the error returned:

Hopefully this PR is reviewed whilst there is an Etherscan issue.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 33.569%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/logic/wallets/ethTransactions.ts
0
4
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
3167

Relevant Lines:
8465

💛  - Coveralls


fix!: use plural names for options

Consistently uses singular and plural names for options in config and CLI
BREAKING CHANGE: This requires to adapt CLI calls and config keys.

Added to https://github.com/rustic-rs/docs/pull/88

Can we still add the plural for:
filter-host = ["host2"] # Default: []
filter-label = ["label1"] # Default: []

Can we still add the plural for:
filter-host = ["host2"] # Default: []
filter-label = ["label1"] # Default: []

Yes, but this is a change in rustic-rs, not in rustic_core, see the latest commit in https://github.com/rustic-rs/rustic/pull/1240.


Update webhooks log count

Only 20 webhook request and responses can be viewed in the UI. Updating the docs to match reality.

Hi @nick-f, thanks for finding this and fixing it! Sorry, it's taken so long for us to get to but I'll help get it resolved. First, I'll get someone from the team to verify the information is correct, then I'll run the linting and merge for you.
@dabarrell, could you please check this change is technically correct?

Hey @mbelton-buildkite, on behalf of @dabarrell I can say this change is technically correct 🙌


Doc landing page doesn't mention React

Hello!
I was perusing your docs and noticed the components section -- however when visiting https://primer.style/components it took me a minute to figure out that these were specifically React components, since the word React is not mentioned anywhere on this page!
I think adding some introductory text explaining what kind of components these are might help newcomers better understand the project when visiting https://primer.style/components. There are quite a lot of different flavors of components out there 😄
Thank you for all your hard work!

Hey Nate! Great catch 😂 The library was originally called primer-react and we never got around to updating the homepage content afterwards it looks like!


Modal verbal questions don't scroll into view when focused

Discovered an accessibility 🐛 where some verbal modal questions don't scroll into view when tabbing through.
Radio and checkboxes that don't scroll into view when focused 🚫

Test questionnaire
Symptoms
Pregnancy

Form inputs that scroll into view when focused  ✅

All text inputs
Test History (radio)

radio buttons have a horizontal display

Steps to reproduce

Click “Test questionnaire” on the patient card / queue item
Select “Complete questionnaire verbally” radio button
Tab around, including unchecking “No Symptoms” if it’s checked so you can see the full list of checkboxes

Is there a React way to do scrollIntoView?
let form_inputs = document.querySelectAll('.ReactModal__Content input');
form_inputs.forEach(function(elem){
elem.addEventListener('focusin', function(elem2){
elem2.closest(".prime-formgroup").scrollIntoView();
})
})

Yes, it still uses the native scrollIntoView under the hood: https://www.carlrippon.com/scrolling-a-react-element-into-view/

wont do


Join task

uses the new parse_uri_scheme function from PR #51. so that should be merged first
fixes: #49
fixes: #52

I have address all the review comments. @klwhaley can we cut a new beta release when you get a chance (after this PR is merged.)


Update typespec init links for azure

Links where using  https://aka.ms/typespec/core-init which is not our aka.ms link instead of https://aka.ms/typespec/azure-init

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/67/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/67/


Recordings cannot be played in the audio playground in an air-gapped environment

Describe the bug

Environment

GPUStack version: main dbf71dd
OS: macOS 14.6.1
GPU: Apple M2

Additional context
Add any other context about the problem here.

Verified in UI version e74cda3


Motion blur of normals and tangents in MeshObject

Currently in MeshObject, position of vertices can be specified at different times but not normals or tangents.

Fixed in 25039779a758cb14a06a649c32d279f51c6a711c


In a mult-screen environment, the menu is shown only on the main screen...

See the menu on the screen where the Beeftext window is
The menu opens but on the "principal" screen
Just move the Beeftext window on a another screen than the main one a pull a menu
Windows 10

Hi.
Thanks for reporting. I could reproduce it. It's a bug in the Qt framework, probably too minor to fix.


fix GetWallets method

There was a bug in WalletProxy::GetWallets - that is fixed (yay for unit tests!)
@KevinOnFrontEnd let me know if this looks right. My cert paths are in the testingappsettings.json file now so we have to figure that out yet.
I also applied a bunch of style cop rules (hope you don't mind) and started on FarmerProxy tests

LGTM.


Filters on associations

Is there a way to use filters on associations?
I'm stranded in a place where I'd like to include associated records like this:
/books?include=authors,authors.books&filter=[authors.books.that_my_friends_like_from_the_same_author]
If there isn't a way to use make this happen with filters, does anybody have another idea of how i can make it happen?

In the master branch there is a new way of filtering using the :apply callable on the filter. Using that you could do:
 class BookResource < JSONAPI::Resource
  filter :that_my_friends_like_from_the_same_author, apply: ->(records, value, _options) {
    # records.where(friends like this author)
}
end

You'll need to provide the actual ActiveResource functions to perform the filtering.

@mintcore did you figure out a solution? The one @lgebhardt mentioned did not work for me


Interface with the extension type from cython

As an experiment, I was trying to implement something in cython using the extension type / object array holding them (a simple vectorized contains; not to replace the contains ufunc, but as simple example of how to use cython with an array of pygeos objects, for potentially a custom non-ufunc algorithm).
So I succeeded in interfacing with the GeometryObject struct (to get the pointer out of the struct), without Python interaction.
But what I didn't get to work yet is to access a single pygeos.Geometry object out of the object array (which needs to happen in the loop).
And the result of this is that I also can't put this for loop in a nogil block, which would be needed to run it in a parallel context.
See experiment here: https://nbviewer.jupyter.org/gist/jorisvandenbossche/c90181a330972d5f4564d7c73b70a114

Great that you got it to work with Cython!
I think the numpy-accessing bit is an issue unrelated to PyGEOS. Maybe you need to type the index correctly?

It's not the index I think (that one is typed, and you can also see it in the output of the annotated code). As far as I can see, the problem is that you get a pygeos.Geometry object out of the array, and cast that to a GeometryObject. That casting to a GeometryObject is fine, as that is something that cython knows about, but getting the object out as pygeos.Geometry is the problem.
The annotated code shows (the first line is the cython line, the lines below that the C code that is generated from it):
+56:         pygeom2 = <GeometryObject *> array[idx]

    __pyx_t_10 = __pyx_v_idx;
    __pyx_t_6 = (PyObject *) *__Pyx_BufPtrStrided1d(PyObject **, __pyx_pybuffernd_array.rcbuffer->pybuffer.buf, __pyx_t_10, __pyx_pybuffernd_array.diminfo[0].strides);
    __Pyx_INCREF((PyObject*)__pyx_t_6);
    __pyx_v_pygeom2 = ((GeometryObject *)__pyx_t_6);
    __Pyx_DECREF(__pyx_t_6); __pyx_t_6 = 0;

So to cast the array[idx] element to a GeometryObject and assign it to pygeom2 variable, cython creates a temporary PyObject.
And I am not sure how to tell cython (or whether this is possible) that the object array holds all GeometryObjects, so that it can avoid this temporary python object.

Right, would this mean that Cython casts any object from an object-typed bumpy array?
Btw the GeometryObject is the C symbol I use for an instance of pygeos.Geometry.

This issue is probably the core problem of #226 .


information added

more information added about hacktoberfest

In 2019, we changed the rules. We want to be awesome. More information on awesome at https://github.com/sindresorhus/awesome/blob/master/awesome.md


Mocking a node_module through mock/ fails in vanilla React-Native projects

🐛 Bug Report
My first bug report, I remain at your demand to provide further details. I didn't find any other reports showing the same characteristics.
On React Native, automatic mocking of a node_module  (by adding a file of the same name in __mock__/) does not work. Jest tests import the original, unmocked module. I'm using react-native as a Jest preset.
React Native Environment Info:
    System:
      OS: Linux 5.0 Arch Linux undefined
      CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
      Memory: 1.58 GB / 7.69 GB
      Shell: 5.7.1 - /usr/bin/zsh
    Binaries:
      Node: 11.6.0 - ~/.nvm/versions/node/v11.6.0/bin/node
      Yarn: 1.15.0 - /usr/bin/yarn
      npm: 6.5.0-next.0 - ~/.nvm/versions/node/v11.6.0/bin/npm
      Watchman: 4.9.0 - /usr/bin/watchman
    SDKs:
      Android SDK:
        API Levels: 23, 24, 27, 28
        Build Tools: 28.0.3
    IDEs:
      Android Studio: 3.3
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.1 => 0.59.1
    npmGlobalPackages:
      create-react-native-app: 2.0.2

To Reproduce
Steps to reproduce the behavior:

react-native init JestBugDemo
cd JestBugDemo

#JestBugDemo/__tests__/lodash.js
const lodash = jest.genMockFromModule('lodash');
lodash.foo = jest.fn(() => 5); // Mock extra function to lodash
export default lodash

#JestBugDemo/__mock__/App-test.js
import 'react-native';
import React from 'react';
import App from '../App';

// Note: test renderer must be required after react-native.
import renderer from 'react-test-renderer';

import lodash  from "lodash";
it('mocks lodash correctly', () => {
  expect(lodash.foo()).toEqual(5);
});

Running yarn test outputs:
expect(received).toEqual(expected)
    Expected: 5
    Received: undefined

So it seems the automocking of lodash failed.
Expected behavior
Test should pass as lodash.foo() returns 5, as specified in __mock__/lodash.js
Link to repl or repo (highly encouraged)
Repo highlighting the bug here
Run npx envinfo --preset jest
Paste the results here:
System:
    OS: Linux 5.0 Arch Linux undefined
    CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
  Binaries:
    Node: 11.6.0 - ~/.nvm/versions/node/v11.6.0/bin/node
    Yarn: 1.15.0 - /usr/bin/yarn
    npm: 6.5.0-next.0 - ~/.nvm/versions/node/v11.6.0/bin/npm
  npmPackages:
    jest: ^24.5.0 => 24.5.0

Thank you for your time and this great framework :+1:

It's __mocks__, not __mock__ ;).

... of course. ;)
Thanks for the quick follow-up, and sorry for the inconvenience. It's working now, guess I need a new pair of glasses.


Errors when using 'cache' property on client config options

Version
v4.0.0-beta.5
Reproduction link
https://github.com/papertokyo/apollo-module-error
Steps to reproduce
On project with apollo module, add 'cache' property to default client using result from apollo-cache-inmemory generator
Run dev
Open index page in browser
What is expected ?
Apollo client is configured with custom cache object
What is actually happening?
Build fails with TypeError
"client.cache.extract is not a function"
Additional comments?
I have a graphQL schema with multiple sub field fragments, which apparently requires the IntrospectionFragmentMatcher from apollo.
https://www.apollographql.com/docs/react/advanced/fragments.html#fragment-matcher

This bug report is available on Nuxt community (#c104)

duplicate of #120


[Suggestion] Enhance utils.py to Support Additional Tool/Agent Folders

I am interested in using your project and would like to deploy it on my Kubernetes (k8s) cluster. However, I encountered some issues related to the maintenance of tools and agents during deployment.
It would be very helpful if there could be an additional folder to store new tools and agents. Could you enhance utils.py to load tools and agents from multiple folders, specified via a .env file?

Can the problem be solved by rebuilding the images?

I’d like to provide more details about my suggestion. I have built an image and deployed it in Kubernetes. It works perfectly when I use the command kubectl exec -it -- python agent_kernel.py.
However, when I want to add new tools or agents, I need to place them in the existing tools/agents folders, which requires rebuilding the image each time. To improve this process, I suggest separating the tools into two folders: one for the original tools and one for custom tools. This way, we can add or update custom tools without needing to rebuild the entire image.
I have already forked the project and am testing these changes. This is why I am raising this suggestion to make the project more flexible and easier to maintain.

Based on my understanding of your requirements, you seem to want to add tools and agents without the need to rebuild the project, and to use new features within the current container.
I have a new idea: using volume to map the tool and agents folders to the local environment, and to automatically reload when there are changes, which might solve your problem.
However, more details need to be tested and adjusted by you.
I hope to receive your further feedback, thank you.
The content above is to be translated by Kimi.

This is an interesting one. It was an intentional design decision not to separate out the core agents and tools as special cases so that the system can eventually modify/upgrade them just as it would with any other tool. You're experiencing one of the downsides of that. Another that I considered at the time is that not splitting out the core components also makes it easier for AgentK to brick itself.
For the sake of argument, let's say that we intend to stick with that original design decision for now, is there a reason you can't mount a volume containing the full repo just as the docker-compose scheduler is doing?

Thank you for your detailed response. I understand the reasoning behind the design decision to keep the core agents and tools together. I appreciate the insight into the potential downsides of separating them.
Regarding your suggestion, I will explore the option of mounting a volume containing the full repo, similar to the docker-compose scheduler. This approach might indeed address the issue without needing to rebuild the image each time.


Update npx and yarn getting started commands

👋 hello! I went to use this template today and the npx and yarn commands failed with the following error:
Could not locate an example named "saas-starter". Please check your spelling and try again.

I'm not sure how a template gets to use a nice name like that, but I was able to get it to work by pointing to the template directly.
Feel free to edit this PR if there's a better way to handle this. Thanks for the great template! ✨

@kmcq thanks for pointing this out, I added that as a placeholder before pushing up to git and then forgot to update it.
https://github.com/supabase/supabase/tree/master/examples/nextjs-with-supabase-auth is actually a separate example. I'm updating the README here: https://github.com/thorwebdev/nextjs-saas-starter/pull/11/commits/dbf40042698f8fc7f26284e60ee5cd8189dda0fc
Quick note: this repository will be moved to the Vercel Github org next week, so it well change then again.

Cool thanks!


Working Group Setup Instructions

First pass at working group setup instructions. Would love your feedback Angel!

Thanks for filling out the setup instructions! They look great, I just caught some minor typos but that was it.
One question: do we want to revise our "Guidelines for sharing scripts" and mention that under the "Edit README" section?


local identity server invalid client error

when i use local identity server invalid client error. can you check
https://ibb.co/yYhRBjY

This should be fixed now. I have updated readme to run and point to local identity server instead of hosted one. Give it a try. Happy to reopen if you encounter another issue.


add pinger and counter examples

progress towards https://github.com/appcelerator/amp/issues/385
Adds pinger and counter examples
Verify with instructions contained in their READMEs

I pushed a few minor updates. Overall, very nice, thanks @generalhenry. 👍


Conan Recipe

I created a simple Conan recipe for cnl. Example usage, from root of cnl repository: conan create . cnl/0.0.1@bloomlife/testing.
If cnl starts using GitHub releases with version numbers we can do something fancier.

Thanks! I can never figure out how to get CI tattoo run automatically for MRs from forks so bear with me while I kick that off. Also, would you like a CLion licence? I have five to use on my open source projects but never remember to share them.

@johnmcfarlane thank you for the offer, but I already have a CLion license through work :) Also, thank you for the invite. I'm just getting started with CNL, but I have a lot of use-cases for fixed-point math on microcontrollers.


Add Install Requirements

Uses setuptools' install-requires to check for numpy and install if necessary during installs of big_O package. Resolves #18.

Thank you!


feat(forc): add manifest formatter

This PR fixes issue #422 by implementing a format checker for the forc manifest
file (Forc.toml) in the forc fmt tool.

fix: add core dependency in sway example project and fix formatting
docs: warn about the formatter needing code that compiles
refactor: put the diff display inside its own function
feat: format manifest with forc fmt command

Should be good now!


Update main.scss

Cleanup:

Cleaned up CSS keyframe animations
Added prefixes where possible

Style changes:

Added a light gray background color

Thanks for submitting a PR, I’ll review your changes shortly.


helm-docs: 1.5.0 -> 1.7.0

Signed-off-by: Mark Sagi-Kazar<EMAIL_ADDRESS>
Motivation for this change
Update to the lastest version
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg eval


[Installation]: failed installation with pip

Your current environment
Collecting environment information...
PyTorch version: N/A
Is debug build: N/A
CUDA used to build PyTorch: N/A
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.6 LTS (x86_64)
GCC version: (Ubuntu 8.4.0-3ubuntu2) 8.4.0
Clang version: Could not collect
CMake version: version 3.16.3
Libc version: glibc-2.31

Python version: 3.9.19 (main, May  6 2024, 19:43:03)  [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-5.15.0-106-generic-x86_64-with-glibc2.31
Is CUDA available: N/A
CUDA runtime version: 12.2.140
CUDA_MODULE_LOADING set to: N/A
GPU models and configuration:
GPU 0: Quadro RTX 8000
GPU 1: Quadro RTX 8000

Nvidia driver version: 535.161.08
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: N/A

CPU:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Byte Order:                         Little Endian
Address sizes:                      46 bits physical, 48 bits virtual
CPU(s):                             72
On-line CPU(s) list:                0-71
Thread(s) per core:                 2
Core(s) per socket:                 18
Socket(s):                          2
NUMA node(s):                       2
Vendor ID:                          GenuineIntel
CPU family:                         6
Model:                              85
Model name:                         Intel(R) Xeon(R) Gold 6240 CPU @ 2.60GHz
Stepping:                           7
CPU MHz:                            2600.000
CPU max MHz:                        3900.0000
CPU min MHz:                        1000.0000
BogoMIPS:                           5200.00
L1d cache:                          1.1 MiB
L1i cache:                          1.1 MiB
L2 cache:                           36 MiB
L3 cache:                           49.5 MiB
NUMA node0 CPU(s):                  0-17,36-53
NUMA node1 CPU(s):                  18-35,54-71
...

Versions of relevant libraries:
[pip3] No relevant packages
[conda] No relevant packagesROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: N/A
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0	GPU1	CPU Affinity	NUMA Affinity	GPU NUMA ID
GPU0	 X 	NV2	0-17,36-53	0		N/A
GPU1	NV2	 X 	0-17,36-53	0		N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

How you are installing vllm
conda create -n test python=3.9 -y
conda activate test
pip install vllm

The error:
Defaulting to user installation because normal site-packages is not writeable
Collecting vllm
  Using cached vllm-0.4.2.tar.gz (588 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Collecting cmake>=3.21 (from vllm)
  Using cached cmake-3.29.3-py3-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (6.1 kB)
Collecting ninja (from vllm)
  Using cached ninja-<IP_ADDRESS>-py2.py3-none-manylinux1_x86_64.manylinux_2_5_x86_64.whl.metadata (5.3 kB)
Collecting psutil (from vllm)
  Using cached psutil-5.9.8-cp36-abi3-manylinux_2_12_x86_64.manylinux2010_x86_64.manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (21 kB)
Collecting sentencepiece (from vllm)
  Using cached sentencepiece-0.2.0-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (7.7 kB)
Requirement already satisfied: numpy in /home/adria/.local/lib/python3.12/site-packages (from vllm) (1.26.4)
Requirement already satisfied: requests in /opt/conda/lib/python3.12/site-packages (from vllm) (2.31.0)
Collecting py-cpuinfo (from vllm)
  Using cached py_cpuinfo-9.0.0-py3-none-any.whl.metadata (794 bytes)
Requirement already satisfied: transformers>=4.40.0 in /home/adria/.local/lib/python3.12/site-packages (from vllm) (4.41.0)
Requirement already satisfied: tokenizers>=0.19.1 in /home/adria/.local/lib/python3.12/site-packages (from vllm) (0.19.1)
Collecting fastapi (from vllm)
  Using cached fastapi-0.111.0-py3-none-any.whl.metadata (25 kB)
Requirement already satisfied: openai in /home/adria/.local/lib/python3.12/site-packages (from vllm) (1.30.1)
Collecting uvicorn[standard] (from vllm)
  Using cached uvicorn-0.29.0-py3-none-any.whl.metadata (6.3 kB)
Requirement already satisfied: pydantic>=2.0 in /home/adria/.local/lib/python3.12/site-packages (from vllm) (2.7.1)
Collecting prometheus-client>=0.18.0 (from vllm)
  Using cached prometheus_client-0.20.0-py3-none-any.whl.metadata (1.8 kB)
Collecting prometheus-fastapi-instrumentator>=7.0.0 (from vllm)
  Using cached prometheus_fastapi_instrumentator-7.0.0-py3-none-any.whl.metadata (13 kB)
Collecting tiktoken==0.6.0 (from vllm)
  Using cached tiktoken-0.6.0-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (6.6 kB)
Collecting lm-format-enforcer==0.9.8 (from vllm)
  Using cached lm_format_enforcer-0.9.8-py3-none-any.whl.metadata (14 kB)
Collecting outlines==0.0.34 (from vllm)
  Using cached outlines-0.0.34-py3-none-any.whl.metadata (13 kB)
Requirement already satisfied: typing-extensions in /home/adria/.local/lib/python3.12/site-packages (from vllm) (4.11.0)
Requirement already satisfied: filelock>=3.10.4 in /home/adria/.local/lib/python3.12/site-packages (from vllm) (3.14.0)
INFO: pip is looking at multiple versions of vllm to determine which version is compatible with other requirements. This could take a while.
Collecting vllm
  Using cached vllm-0.4.1.tar.gz (534 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
  Using cached vllm-0.3.3.tar.gz (315 kB)
  Installing build dependencies ... error
  error: subprocess-exited-with-error

  × pip subprocess to install build dependencies did not run successfully.
  │ exit code: 1
  ╰─> [8 lines of output]
      Collecting ninja
        Using cached ninja-<IP_ADDRESS>-py2.py3-none-manylinux1_x86_64.manylinux_2_5_x86_64.whl.metadata (5.3 kB)
      Collecting packaging
        Using cached packaging-24.0-py3-none-any.whl.metadata (3.2 kB)
      Collecting setuptools>=49.4.0
        Using cached setuptools-70.0.0-py3-none-any.whl.metadata (5.9 kB)
      ERROR: Could not find a version that satisfies the requirement torch==2.1.2 (from versions: 2.2.0, 2.2.1, 2.2.2, 2.3.0)
      ERROR: No matching distribution found for torch==2.1.2
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× pip subprocess to install build dependencies did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip

Using cached sentencepiece-0.2.0-cp312-cp312-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (7.7 kB)
Requirement already satisfied: numpy in /home/adria/.local/lib/python3.12/site-packages (from vllm) (1.26.4)

Your pip somehow uses python 3.12's package. You need to figure out what's wrong with it.

Thank you!


@gnosis.pm/safe-apps-wagmi incompatible with wagmi 0.8

Wagmi 0.8 requires connectors be exported as ESM

@matthewlilley I just copied the safeconnector code locally to make it work for now

@matthewlilley @sarinniural I can't reproduce this with wagmi 0.8.7 in the example app from the repo. I also don't see how wagmi and ESM would be connected since wagmi is a library and not a build tool. Could you please share a reproduction example or more information like which error you exactly had?

@mikhailxyz thanks for fixing it, looks like it is working

@mikhailxyz there is still come incompatibility problems.
Property 'onAccountsChanged' is protected but type 'SafeConnector' is not a class derived from 'Connector<Provider, Options, Signer>'.

@mikhailxyz there is still come incompatibility problems.
Property 'onAccountsChanged' is protected but type 'SafeConnector' is not a class derived from 'Connector<Provider, Options, Signer>'.

Is there an example/code snippet where I can reproduce it? I do not run into this error when using the example app

@mikhailxyz this is a typescript error which blocks compilation and can only be ignored by ignoring ts which isn't really a solution. If you enable strict mode you'll see it when trying to add this connector to wagmi client connectors array.

@matthewlilley I did not have this issue with the recent version of wagmi  0.8.7 and safe-apps-wagmi 1.2.0.
I have ts and strict mode enabled

@mikhailxyz any ideas on my issue above, my own work to update the versions is pending.

@mikhailxyz any ideas on my issue above, my own work to update the versions is pending.

could you please do fresh node modules install? we released an update to the transient wagmi wrapper dependency (safe-apps-provider)

@matthewlilley I'm sorry but I can't reproduce it... I can take a look if you give me an example. Also I double checked and the wagmi connector indeed derived from the Connector class:
import { Connector, Chain, ConnectorNotFoundError } from '@wagmi/core';

class SafeConnector extends Connector<SafeAppProvider, SafeOpts | undefined> {

@matthewlilley hello, did you find a solution for the type error? I'm with the same problem when adding the connector to the array

@matthewlilley hello, did you find a solution for the type error? I'm with the same problem when adding the connector to the array

I just owned the code and updated it so it was compatible. I keep nudging wagmi folks to pull this into the core since it's a critical connector for some apps and is annoying when it falls out of sync.

@franm91 @matthewlilley could you please give us a reproducible example of the type error? we're also interested to get this fixed but we cannot reproduce it

@mikhailxyz here is a reproducible example of the error: https://github.com/cruzdanilo/safe-wagmi-bug
i just created a new app with create-wagmi and then added the safe connector.


Don't use StringText for generated files in Visual Studio

Today, we use SourceText instances for generated files in VS that are backed by strings rather than VS text buffers. This results in significant performance problems:

A new StringText gets created for every change to a generated file. This means that a string instance is allocated for the entire file on every key press.
StringText causes Roslyn to skip an optimized path, making completion slower--especially on subsequent edits to the same method body.

Related issue: #8076


keybase_daemon_rpc: oops, we have to register for notifications!

I thought that registering for a specific notification protocol was
enough to receive the notifications, but apparently we also have to
explicitly subscribe to the right notification channels.
Issue: KBFS-1341

lgtm


Refactor overflow checks in intrinsics code

Description of changes:
Refactors the overflow checks added in #1094, #1099 and #1102 into a function that returns the result and the assertions associated with it. When working on this I realized the expected failure for ptr_offset_from (bytes) was actually a failure coming from offset. This was a rabbit hole because it's actually possible to trigger the failure by "wrapping around" a pointer, so I added a comment there to avoid confusion in the future.
Resolved issues:
Resolves #1116
Call-outs:

Testing:

How is this change tested? Updates one test.

Is this a refactor change? No (but it should have been).

Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
[x] Methods or procedures are documented
[x] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

This PR is ready for review again:

Reviewed the overflow check in ptr_offset_from (see discussion above)
Changed the offset-from-bytes-overflow expected output to look for overflows in both offset operations
Added offset-wraps-around to test the "wrapping-around" behavior in CBMC.
Created #1150 and linked in offset-related overflow checks.


Discount over certain amount?

Hi
Is it possible to apply a discount when an order is over a certain amount?
Thanks
Da e

Not at this point, the 'restrictions' part of discounts needs to be extended with further options.


Add Instance::with_loading_function for better cross-platform handling

Right now the Vulkan library is loaded in the loader module. Depending on the operating system the path to the library is different.
When trying to add support for MoltenVK, this became a problem because MoltenVK is a static library that must be linked at compile-time. PR #301 tweaks the loader module to have a totally different behavior on OSX.
However it would be great to not rely on any platform-specific behavior, in case the user wants to use a custom way of accessing Vulkan. Therefore I suggest adding Instance::with_loading_function:
impl Instance {
    pub unsafe fn with_loading_function(f: *const c_void, ... other params ...) -> Result<Instance, ...> {
        ...
    }
}

The function f passed to with_loading_function must be the implementation vkGetInstanceProcAddr, and will be used by vulkano to load everything.
The Instance::new function would be modified to use Instance::with_loading_function under the hood.
With this change Instance::new would try to work transparently on all major platforms, but the user could choose a custom implementation/behavior.

Unfortunately what I didn't think about is that getting the list of layers and extensions also requires loading the function pointers.
Therefore we will need some sort of intermediate object I think.
Example:
let vk_library = unsafe { LoadedLibrary::from_function(...) };
let layers = vk_library.layers_supported_by_core();
let instance = Instance::with_library(vk_library).unwrap();


Some Grammar fixes

1- For Here's a : The singular verb form ’s does not seem to agree with the plural subject a few common cases where you might want to try something else. Consider changing the verb form.
2- Added A before Reasonably : The noun phrase Reasonably good configuration seems to be missing a determiner before it. Consider adding an article. An article (a, an, or the) is a type of determiner. Possessive adjectives (my, his, our), possessive nouns (Joe’s, mother’s), and quantifiers (each, every) are also determiners. Single countable nouns usually require a determiner.

Hey, @gaearon @Timer Could you please check, these are some minor fixes but improves the readability.
Thanks!!

Hi @ianschmitz, this pr is heating the PRs count, please mark it. Thanks!

Thanks!


Quiet AppVeyor builds

--parallel leads to --quiet being ignored in spawned threads

/cc @pranavkm

:shipit:

Closing, didn't remove noise from build. Disentangling output isn't useful with the current tracing level.


Enable APC for PHP via ini file

@M1r1k do we need to disable opcache , because it is enabled by default for recent php versions?

Otherwise apc will start to conflict with opcache

aha
here we go - another problem with using different Ubuntu versions
Vagrant has php and requires 5.4 APC, when CIBOX uses LTS with php 5.5

retest this please


export PyUserWarning

Just as other exceptions are available from pyo3::exceptions.
Perhaps other warnings?
Perhaps from pyo3::warnings?
For know, I just need UserWarning.
In the meantime, I'll import it manually.

I've tried doing the following:
use pyo3::{ffi, impl_exception_boilerplate, pyobject_native_type};
use pyo3::prelude::*;

#[doc = "UserWarning"]
pub struct PyUserWarning(PyAny);

impl_exception_boilerplate!(PyUserWarning);
pyobject_native_type!(PyUserWarning, ffi::PyBaseExceptionObject, *(ffi::PyExc_UserWarning as *mut ffi::PyTypeObject));

But unfortunately I get this error
error[E0603]: module `impl_` is private
   --> /home/samuel/.cargo/registry/src/github.com-1ecc6299db9ec823/pyo3-0.17.3/src/types/mod.rs:247:54
    |
247 |             type PyClassMutability = $crate::pycell::impl_::ImmutableClass;
    |                                                      ^^^^^ private module
    |
note: the module `impl_` is defined here
   --> /home/samuel/.cargo/registry/src/github.com-1ecc6299db9ec823/pyo3-0.17.3/src/pycell.rs:219:1
    |
219 | pub(crate) mod impl_;
    | ^^^^^^^^^^^^^^^^^^^^

For more information about this error, try `rustc --explain E0603`.
error: could not compile `pydantic-core` due to previous error
make: *** [Makefile:21: build-dev] Error 101

I've hacked together a PR to add those classes like the standard exceptions. I also added PyErr::warn_explicit in addition to warn.


Test issue #1

Before opening, please confirm:

[X] I have searched for duplicate or closed issues and discussions.

Language and Async Model
Kotlin
Amplify Categories
Authentication
Gradle script dependencies
test
Environment information
test
Please include any relevant guides or documentation you're referencing
No response
Describe the bug
test
Reproduction steps (if applicable)
No response
Code Snippet
// Put your code below this line.

Log output

// Put your logs below this line

amplifyconfiguration.json
No response
GraphQL Schema

// Put your schema below this line

Additional information and screenshots
No response

test comment.

comment 1

what if I comment on this one?

perfect


2017 Launch List

[ ] Update and send november reminder email
[ ] Run rake task to check status of all existing projects (@andrew)
[ ] Collect ideas for 2017 #2064
[ ] Scale up heroku dynos (@andrew)
[ ] Review current admins and contributors on GitHub (@andrew)
[ ] Update featured projects #2063
[ ] Update sponsors
[ ] Triage existing issues
[ ] Encourage people to run events

:wave: Let me know how I can help.

Would be happy to help Let me know!

I'd also like to help!


Remove rules module?

The rules module in it's current state causes problems with visibility options for blocks and other things in core.
Looking through the code I found out it is used in the custom le_website module, but only for a deactivated rules component.
For what exactly is rules used currently and can it be removed and replaced by using Drupals event system?

Moin! You are completely right, there is no need for rules anymore and we can (should!) remove it. I was once trying to create a rules-based workflow to extend the organic groups module (e.g. "Send a mail to the group owners when a membership is requested"), but I failed as some mandatory tokens were not yet provided by OG.

How good, that I've just removed it today in the develop branch.


Airfoil control surface / hinged geometry not smooth

Bug Description / Observed Behavior
When adding a deflected control surface (flap) to an Airfoil object, the resulting geometry / coordinates have more of an abrupt curvature discontinuity than Xfoil's method of adding a circular arc.  Not necessarily a bug/wrong, but it does end up giving coefficient results that differ from the Xfoil-created flapped geometry by >10% even at modest deflections.  It threw me off for a while when validating the hinged-airfoil results with a pure Xfoil check.
Steps to Reproduce
import aerosandbox as asb
af = asb.Airfoil(name="naca0016").add_control_surface(
    deflection=15.0,
    hinge_point_x=0.7
)
af.draw()

The extended upper-surface "gap" is nearly linear / not filled in smoothly.
!
Expected Behavior
Overlayed against Xfoil's GDES routine (blue/green is aerosandbox's geometry), where Xfoil fills in with a smoother circular arc.

System Information

Operating System: Windows 10 Pro
Python Version: 3.11.2
AeroSandbox Version: 4.2.0

Other Information
See subroutine FLAP in xfoil (6.99) xgdes.f source file for how Xfoil adds a circular arc: was this approach considered for hinged airfoil geometry in asb?

Hi Ben,
Thanks for reporting this, and a good question! Tl;dr is that I agree that the circular-arc method is probably the better way to go these days, but there were older motivations back when this AeroSandbox flap handling code was written that motivated this choice.

The original motivation behind the AeroSandbox flap handling was that it preserves the dimensions of the coordinates array (i.e., no new points are deleted and created). This contrasts with XFoil's approach, which both creates new points on the "expanding" side and deletes points on the "contracting" side. The problem with this is that this makes XFoil's logic inherently $C^1$-discontinuous, which causes issues with gradient-based optimization. (As an illustration, if you plot $C_m$ vs flap deflection in pure-XFoil, and you zoom in far enough, that plot is only $C^0$-continuous - it has "sharp peaks" when points are created or destroyed.)
This also causes issues with automatic differentiation in static-graph backends (e.g., CasADi, JAX), which require array shapes to remain static.
A practical example of where both problems might arise would be iteratively varying the deflection to satisfy a trim condition.

Nowadays, the asb.KulfanAirfoil class is recommended to be used instead of the asb.Airfoil class for most cases where the airfoil shape (or control deflections) are not frozen at runtime. For a lot of reasons, asb.KulfanAirfoil is more optimization-friendly than asb.Airfoil (reasons are detailed in one of the tutorials).
Because of that, it would be reasonable to go back to asb.Airfoil.add_control_surface() and add a new keyword argument to optionally support XFoil-like flap behavior, now that breaking differentiability here is potentially less critical.

I probably don't have time to implement this logic in the near-term, but leaving this issue open in case anyone wants to open a pull request supporting this!

Ah ok that all makes sense, for some reason I thought the Xfoil approach was maintaining c1 continuity, thanks for the correction.
I was coming at it more from wanted to evaluate hinge moments than optimization, and Aerosandbox was just a handier/ready-made way to run those sweeps parametrically in Xfoil vs. Xfoil itself!
I'm fine with closing this but up to you.


Incomplete LUT grid in apply_oe when all z values in loc file are < 0 (when using emulator)

I have a couple flight lines where z values are all slightly below 0.  These all get converted to 0 for 6S, but this results in no range in elevation to build out the LUT grid for GNDALT and it only has one entry.  Maybe this could be fixed with a quick check for this condition, and add a second entry with a little bit of elevation added?
INFO:2023-11-27,17:52:20 ||| Scene contains target lut grid elements < 0 km, and uses 6s (via sRTMnet).  6s does not support targets below sea level in km units.  Setting grid points [-3.8420957e+03 -3.8418457e+03 -3.8415957e+03 ... -5.0370000e-01
 -2.5370000e-01 -3.7000000e-03] to 0.
INFO:2023-11-27,17:52:20 ||| Scene contains a mean target elevation < 0.  6s does not support targets below sea level in km units.  Setting mean elevation to 0.
...
INFO:2023-11-27,17:52:20 ||| Running ISOFIT with full LUT
INFO:2023-11-27,17:52:20 ||| Checking config sections for configuration issues
ERROR:2023-11-27,17:52:20 ||| lut_grid item GNDALT has less than the required 2 elements
Traceback (most recent call last):
  File "/home/nvaughn4/repos/isofit/isofit/utils/apply_oe.py", line 1978, in <module>
    _cli()
  File "/home/nvaughn4/bin/miniconda3/envs/isofit_env/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/nvaughn4/bin/miniconda3/envs/isofit_env/lib/python3.10/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/nvaughn4/bin/miniconda3/envs/isofit_env/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/nvaughn4/bin/miniconda3/envs/isofit_env/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/nvaughn4/repos/isofit/isofit/utils/apply_oe.py", line 1972, in _cli
    apply_oe(SimpleNamespace(**kwargs))
  File "/home/nvaughn4/repos/isofit/isofit/utils/apply_oe.py", line 495, in apply_oe
    retrieval_full = isofit.Isofit(
  File "/home/nvaughn4/repos/isofit/isofit/core/isofit.py", line 68, in __init__
    self.config.get_config_errors()
  File "/home/nvaughn4/repos/isofit/isofit/configs/configs.py", line 115, in get_config_errors
    raise AttributeError("Configuration error(s) found.  See log for details.")
AttributeError: Configuration error(s) found.  See log for details.
isofit returned non-zero code (1) - aborting
srun: error: cg17-15: task 0: Exited with exit code 1

Config has this for lut_grid:
...
        "radiative_transfer": {
            "lut_grid": {
                "AOT550": [
                    0.001,
                    0.1009,
                    0.2008,
                    0.3007,
                    0.4006,
                    0.5005,
                    0.6004,
                    0.7003,
                    0.8002,
                    0.9001,
                    1.0
                ],
                "GNDALT": [
                    0.0
                ],
                "H2OSTR": [
                    0.4332,
                    0.6653,
                    0.8975,
                    1.1296,
                    1.3618,
                    1.5939,
                    1.8261,
                    2.0582,
                    2.2904,
                    2.5225,
                    2.7547,
                    2.9868,
                    3.219,
                    3.4511
                ],
                "OBSZEN": [
                    159.0915,
                    168.1737,
                    177.256
                ],
...

Can you say a bit more about the style of use here?
The current assumed behavior is that LUT grids have at least two dimensions (otherwise it's not really a grid...).  In the case of the above, the simple solution, which I believe is followed by our utils scripts, is to just not include the GNDALT dimension in the config.  In general we prefer this for the LUT to keep the dimensionality low.
We could implement a simple check for this condition in the config that pops out 1-D LUT elements, but that would risk a deviation between the LUT grid and the RTE template file.  We could add additional checks to force these to be the same, but that starts to be a decent amount of checking for a case that's already covered.
If the above solution of just trimming out GNDALT from the config doesn't suffice, maybe you could say a bit more about the desired use case here to help guide ideas?

Ahh, okay, I had misunderstood.
Looks like there's a super simple fix here.  We just need to add a quick check on the elevation grid being length 0, and if it is re-assign for both the template and set the grid to null.  Standby for PR...


Connection from VNC client doesn't work

docker run --shm-size=1gb -it --privileged --name neurodesktop -v C:/neurodesktop-storage:/neurodesktop-storage -p 5901:5901 -p 8080:8080 -h neurodesktop-20220813 vnmd/neurodesktop:20220813 --vnc
Connecting to port 5901 did not work (tested on windows with turbovnc 3.0.1)

Hi @korbinian90. Can you test this build if this fixes the issue? (localhost::5901)
docker run --shm-size=1gb -it --privileged --name neurodesktop -v C:/neurodesktop-storage:/neurodesktop-storage -p 5901:5901 -p 8080:8080 -h neurodesktop-20220813 vnmd/neurodesktop:20221121 --vnc

Hi @aswinnarayanan. It does work now!


Work around a Julia bug

This works around a bug in Julia. I expect that this will be fixed eventually, and at that point we can put this extra call behind a version check.

Do you have a link to the upstream ticket? Could be worth adding as a comment for future reference.

Yes, the upstream bug is https://github.com/JuliaLang/Pkg.jl/issues/1775. Once this is fixed I'll open a new PR that only runs this workaround for Julia versions here this is a problem.

+- 0.06% in code coverage. Seems like "codecov go home, you are drunk" is appropriate here ;)
This looks good to me so merging it.


Cannot insert duplicate key

Why can't we insert a key and overwrite if any existing values in the dictionary?

I guess there would be an ambiguity about the insertion order of overwritten object.

Exactly. You have to go through two different paths to achieve this.
Check whether the key is already present. If that's the case, update the value. If not, insert the key-value pair at the desired index.


[WIP] Fix potential NPEs in BaseMenuManager

Fixes #1215
This PR is not ready for review.
Risk
This PR makes no API changes.
Testing Plan
Try setting the menu to null and make sure the menu cells are getting cleared correctly without crashing the app:
sdlManager.getScreenManager().setMenu(null);

Summary
This PR fixes some potential NPEs in MenuManager when setting the menu to null
CLA

[x] I have signed the CLA

Codecov Report

Merging #1216 into develop will decrease coverage by 0.04%.
The diff coverage is 33.33%.

@@              Coverage Diff              @@
##             develop    #1216      +/-   ##
=============================================
- Coverage       47.1%   47.05%   -0.05%
  Complexity      4327     4327
=============================================
  Files            481      481
  Lines          23475    23479       +4
  Branches        2683     2685       +2
=============================================
- Hits           11057    11049       -8
- Misses         11750    11762      +12
  Partials         668      668

Impacted Files
Coverage Δ
Complexity Δ

...vicelink/managers/screen/menu/BaseMenuManager.java
44.8% <33.33%> (-0.3%)
90 <0> (ø)

...smartdevicelink/encoder/VirtualDisplayEncoder.java
24.43% <0%> (-4.55%)
8% <0%> (ø)

Continue to review full report at Codecov.

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


Add --output_wrapper_file to the Output category in the help message

The flag is useful if the user wants special characters like newline in
the wrapper. See:
https://groups.google.com/forum/#!topic/closure-compiler-discuss/DqxBxukkdPA
for the discussion.

This change is

@ChadKillingsworth


Contribution

Hey laudanum
I have been watching this repo grow and checking all the updates you are making to the website, I would like to contribute if that is okay.
If not I will continue to lurk in the repo 😄

Hey YazidKurdi,
I would be delighted if you are willing to contribute to the project. I have a couple of open points right now which are most "urgent" :-)
I am not that knowledgeable how to best coordinate work on github. If you have any ideas where we can collect and coordinate work on new features please tell me.
Right now the following things are most needed:

Improve the quality of the Named Entity Recognition which is implemented using the Spacy library. Right now it does not seem to work very well as it misses a lot of the NPC names and locations. Maybe we could also tell GPT3 in our initial prompt that it should somehow highlight all NPC names and location names. Some experimentation would be needed I guess
Character Creator for Character Stats and backstory
Location Creator for Location Details

Thanks @laudanum123 !
I suggest writing up each feature in a separate issue, if there are a lot of features, separating them into - high, medium, and low - labels would be best. To avoid duplicating work you can assign tasks you are working on to yourself and I can propose to be assigned certain tasks which you can approve.
As for now, I can work on the Named Entity Recognition feature, I noticed there were duplicated entities and mislabeled objects as entities.
What are you looking to achieve with this feature? An end user would select his saved campaign from db and show a list/table/card of entities to him?
Idea template (if necessary):
Idea Description
Please provide a clear and concise description of the idea.
Problem Solved
What problem does this idea solve or what need does it meet?
Proposed Solution
Describe the solution you are proposing.
Advantages
What are the benefits of this idea? How does it compare to current solutions or workarounds?
Use Cases
Please describe some examples of how this idea could be used in practice.
Additional Context
Add any other relevant information about the idea here. This could include similar ideas or related issues, potential drawbacks, or questions that need to be answered before implementation.

That sounds very good. Let's do this!

Great!
Want to hop on discord for better ideas communication?

sure!

you already got a server?

my user is laudanum#5804


Poss solution to disable popups while dragging

Noticed a new behaviour (FF Linux):
A single click on a marker that has a "pop-down" popup showing temporarily makes it rotate to the "pop-up" position.

Another bug in this branch:
In "Donut Markers->Two Requests" story - the popups stop working after the first drag.
The First Request story isn't broken.
I haven't looked at the code for explanations yet.

I added a fix to the clicking bug. The two-requests bug still exists, but we'll take a look at it later if it ends up being necessary. Otherwise this is a nice, clean solution. Thanks Bob!


how to get event stop scroll on calendar ?

i want to get event stop scroll on calendar please help me sir thank

What u really mean is "disable scrolling"?
if u r, just set
calendar.scrollEnabled = NO;


Allow SQL Connector to insert structs

In the current implementation of SQL connector, we can only insert values as parameters. But it might be a good feature to support inserting/retrieving structs.

Now (0.970.0) instead of using sql:Parameters variables of type int, float, boolean, string or blob can be directly given to the actions as var args.


feat(HMS-1879): Add instance desc table for GCP

This PR adds an instance description table at the end of a successful launch and also adds missing tests.

Shall we remove the DNS at some point? I mean, only AWS supports public DNS by default, maybe showing up SSH is enough.
Other than that, looks good.

Shall we remove the DNS at some point? I mean, only AWS supports public DNS by default, maybe showing up SSH is enough.
Other than that, looks good.

In a followup PR we can show the DNS column only if the DNS(s) exists.

Looks good, just verifying we've got a ticket for the welcome redirect, apart of it looks good :+1:

We have a ticket for it, it is in the backlog :)


Running RobotJS on GlobalMu online - Win 10

I am trying to setup an autoclicker for GlobalMu online, I am on Windows 10, I am using the latest robotjs installation, the game is in window mode.
Expected Behavior
I expect the mouseclick to behave the same on my desktop as it does in the game window
Current Behavior
The mouse click event works outside of the game, and I can even have the mouseclick the tab for the game, but when in game it doesn't recognize any mouse events from robotjs. ?
Possible Solution
Would this be Windows, I'm assuming the game maybe has a blocker?
Steps to Reproduce (for bugs)
1.Here is my Code
`var robot = require("robotjs");
setInterval(function () {
var now = new Date();
robot.mouseClick("right");
console.log(now.getHours() + ":" + now.getMinutes() + ":" + now.getSeconds());
console.log("success");
}, 1000);`
2.I am able to node app.js and start the app, and get the console logs the whole time
3.I am receiving right clicks everywhere on my desktop, however the game doesn't recognize any clicks (windowed mode)
4. Any help is sincerely appreciated, I've searched on other topics and I might be missing something and apologize if I have.
Context
Nothing too terrible, I'm just having fun with botting, not trying to ruin any games or get out of control, I just enjoy automation. any help or response is appreciated, I understand if you don't support specific games.
Your Environment

RobotJS version:  "robotjs": "^0.5.1"
Node.js version: v10.15.0
npm version: 6.4.1
Operating System: Windows 10 Enterprise 64bit

This issue has been closed because it has been inactive for > 180 days.
You may reopen this issue if it has been closed in error.


useSignalEffect should execute after changes have been applied to the DOM

The callback passed to useSignalEffect should be executed after the changes (if any) have been applied to the DOM. Otherwise, it won't be a direct replacement for useEffect.
For example, people expect this to work, even if ref appears when open switches from false to true, because useEffect waits until changes have been applied to the DOM:
useEffect(() => {
  if (open) ref.current.focus();
}, [open]);

But with the current implementation of useSignalEffect, the callback runs before the DOM changes, and therefore ref.current may not exist yet:
useSignalEffect(() => {
  if (open.value) ref.current.focus();
});

Full example in StackBlitz.
Other reactive libraries introduce a tick function (Vue, Svelte, Alpine), but I'd prefer if useSignalEffect could mimic useEffect because the familiarity and ergonomics are better. I.e., access to signals after the tick() call are not tracked.

That's a very good point. I hadn't thought about the interplay with ref's.

Hmm, that makes sense. I guess the sync escape hatch is possible but leveraging the commit options hook would allow us to immitate the timing of a regular useEffect

If you give me some instructions, I can work on the PR.

Ran into this as well, I ended up creating a reactive ref that seems to do the trick and essentially would work like a tick mechanism:
const node = useSignal<HTMLElement | null>(null)
const ref = useCallback((refNode) => (node.value = refNode), [])

useSignalEffect(() => {
  if (open.value) node.value?.focus()
})

If this could somehow work like useEffect out of the box though that'd be awesome!

@JoviDeCroock, @marvinhagemeister: any idea how to implement this? I can work on it.

I would say that it would work like the following, we introduce a second variable an object containing mode/timing as a key with three values

sync (default): current behaviour
layout: behaves like useLayoutEffect - you can probably get that from preact/hooks/src/index.ts but tldr, array that gets processed after RaF
commit: behaves like useEffect gets pushed on _component._commitCallbacks and processes as part of options._commit, can also look at preact/hooks

Would this be sufficient to work on it? Imho we could omit layout as an option for the time being until folks find a use for it.

Thanks @JoviDeCroock.

commit: behaves like useEffect gets pushed on _component._commitCallbacks and processes as part of options._commit, can also look at preact/hooks

I can't find _commitCallbacks in the code. Do you mean _renderCallbacks?

Would this be sufficient to work on it?

Hmmm... I guess the problem is how to delay the execution of the effect callback once it has run for the first time. We can't do it with normal async execution because if we do so, the second time the computation won't pick the dependencies. For example, imagine a setTimeout of 1 second was the right timing:
function useSignalEffect(cb: () => void | (() => void)) {
  const callback = useRef(cb);
  const firstTime = useRef(true);
  callback.current = cb;

  useEffect(() => {
    return effect(() => {
      if (firstTime.current) {
        // First time
        firstTime.current = false;
        callback.current();
      } else {
        // Subsequent times.
        setTimeout(() => callback.current(), 1000);
        // It doesn't work because the callback will run once the computation
        // (`effect`) has finished.
        }
    });
  }, []);
}

So, is there an API to delay the execution of effect to the right moment? Or does this need a change in @preact/signals-core?

we introduce a second variable an object containing mode/timing as a key with three values

About that: so, right now this is the code of useSignalEffect:
function useSignalEffect(cb: () => void | (() => void)) {
  const callback = useRef(cb);
  callback.current = cb;

  useEffect(() => {
    return effect(() => callback.current());
  }, []);
}

And it would accept a second argument, like this:
function useSignalEffect(
  cb: () => void | (() => void),
  mode: "sync" | "layout" | "commit" = "sync"
) {
  // ...
}

Then, the implementation for each mode would be:

"sync"
You say it should be the current behavior, but the current behavior acts as "commit" in the first run and as "sync" later. Should we change it as well for the first run? I guess something like this:
function useSignalEffect(
  cb: () => void | (() => void),
  mode: "sync" | "layout" | "commit" = "sync"
) {
  if (mode === "sync") {
    const dispose = effect(cb);
    useEffect(() => dispose, []);
  }
}

"layout" and "commit"
Again, any direction here would be greatly appreciated as I'm not familiar with the signal's internals yet 🙂
function useSignalEffect(
  cb: () => void | (() => void),
  mode: "sync" | "layout" | "commit" = "sync"
) {
  if (mode === "commit") {
    const callback = useRef(cb);
    callback.current = cb;
    useEffect(() => {
      return effect(() => {
        // The first time it needs to run syncronously.
        callback.current();
        // After that, it needs to wait until `useEffect`/`useLayoutEffect` runs.
        ??
      });
    }, []);
  }
}

Is this still an issue?

We have our own implementation of useSignalEffect in WordPress that has been working well so far.