Code is truncated

When the code file is long, it will make changes to the file and in places put comments like:
// Rest of the code remains the same
This essentially renders the code file useless.

+1

Apply edits is something I've realized is a pretty hard problem to solve. Currently Claude Dev asks Sonnet 3.5 for the whole file in its response, even if it only has to change one line of code. Not efficient I know, and I planned on optimizing this after the hackathon.
LLMs like Claude have a maximum # of output tokens they can respond with each API request, and so you'll likely see it get "lazy" when writing large amounts of code i.e. with "rest of the code remains the same".
Cursor uses a fine tuned model trained on applying edits from a partial response to a whole file change, along with “speculative decoding” to speed up its responses (instead of inferring the next single token, it infers multiple tokens in parallel). This way it can take a lazy suggestion from a foundational (expensive) model like Sonnet 3.5 and use their lighter ft model to apply those suggestions to a large file much more affordably. (They also use a pretty interesting concept of 'Shadow Workspace' where they essentially use a hidden second instance of vscode to catch and fix any linting errors before applying the edits.)
Aider uses a diff strategy where it asks the LLM to respond with a unified diff format (like how you see with patches), which works much better than other diff strategies considering these models were likely trained on unified diffs at some point. However  asking for a whole file results in higher quality responses because 1) their training data likely includes way more whole files than unified diffs, and 2) it is forced to think with more tokens.
I want to build Claude Dev with the assumption that scaling laws will persist and LLMs will get better and cheaper, so I'd prefer not to incorporate hacks that would reduce its quality. Fortunately, Anthropic recently doubled the max output token limit for Claude 3.5 Sonnet from 4096 to 8192 in their API, and I will be updating Claude Dev with this change once the hackathon is over. This means way less lazy coding and we can maintain the quality of Claude's edits without compromise.

I agree with your assessment to not go with the unified diff approach aider uses. I get way more diff errors messing up my code with aider than I get code errors occurring with this. One thing I do like with aider is that I can /undo changes and rollback as many changes as I want to. Its use of git for this is helpful as well. I think that kind of ability either utilizing git or some other mechanism would be a good feature for the roadmap.

v1.0.4 now uses 8192 output tokens, so there should be way less lazy coding. Please feel free to re-open if you continue to see this issue.

Could we automate this with some blacklisted phrases that automatically reply to claude"provide full updated code" or the like?
Also, I've read that adding to the system prompt is sometimes less effective than appending to user prompt.

Wouldn't it be simpler to ask Claude to always generate diff files? It would probably cost much less in terms of output tokens.

Thanks so much, saoudrizwan,  for this great api  to streamline coding and make bug fixing and brainstorming CODING IS A JOY WITH CLAUDE.DEV

I am having this issue as well. The main reason to use in vscode sidebar is to integrate directly into code with a diff and not have to copy and paste.

But if the file is truncated with  "rest of the code remains unchanged" then look for that message and dont go into diff mode, only show those lines as diffed.
go into diff mode but automatically add the truncated parts of files. should be easy to match for the end and starts.
always put into the original prompt: Always print all functions in the file and do not ever print a message that rest of the code remains unchanged to replace functions.
let us customize the sysprompt

Wouldn't it be simpler to ask Claude to always generate diff files? It would probably cost much less in terms of output tokens.

You can write a custom instruction to achieve this with unified diffing (YMMV, especially for Ruby like languages):
When editing files, please use the unified diff format to specify your changes. Follow these guidelines:

1. Start each diff with the file name, using '--- a/filename' for the original file and '+++ b/filename' for the modified file.
2. Use the standard unified diff syntax, including line numbers at the start of each hunk. Begin each hunk with @@ -start,count +start,count @@, where:
   - 'start' is the starting line number in the original file
   - 'count' is the number of lines in the hunk (including context lines)
   Use '1' for the count if you're adding new lines at the beginning or end of the file.

3. Provide coherent diffs that show the entire function or logical block of code being modified, including sufficient context.
4. Use '-' to indicate lines being removed, '+' to indicate lines being added, and ' ' (space) for unchanged context lines.
5. Include at least 3 lines of unchanged context before and after the changes to clearly locate the modification in the file.
6. For multiple changes in a file, use separate hunks divided by the @@ ... @@ marker.
7. Ensure that your diffs are complete and don't elide code with comments.
8. When editing a function or block, replace the entire block if possible.
9. To move code within a file, use 2 hunks: 1 to delete it from its current location, 1 to insert it in the new location.
10. Pay close attention to indentation and whitespace. They matter in the diffs!

Here's an example of a correctly formatted hunk:

\```diff
@@ -1,7 +1,9 @@
 defmodule Memo.WatchRun do
+  @moduledoc """
+  A GenServer that watches for file changes in a specified vault path and triggers
+  export scripts when changes are detected.
+  """
+
   use GenServer
   require Logger

-  # delay in milliseconds
-  @unlock_delay 2000
+  @unlock_delay Application.get_env(:memo, :unlock_delay, 2000)
\```

After providing the diff, explicitly state the commands to write the diff to a file in the .claude.dev folder and apply the changes:

Run the `write_to_file` tool for the diff file:

\```
write_to_file .claude.dev/watch_run.patch "
--- a/lib/memo/watch_run.ex
+++ b/lib/memo/watch_run.ex
@@ -1,7 +1,9 @@
 defmodule Memo.WatchRun do
+  @moduledoc """
+  A GenServer that watches for file changes in a specified vault path and triggers
+  export scripts when changes are detected.
+  """
+
   use GenServer
   require Logger

-  # delay in milliseconds
-  @unlock_delay 2000
+  @unlock_delay Application.get_env(:memo, :unlock_delay, 2000)
"
\```

Then run the following command with `execute_command`:

\```bash
patch -p1 < .claude.dev/watch_run.patch
\```

Ensure that the patch applies cleanly against the current contents of the file. Double-check line numbers, indentation, and whitespace.

Some parts are taken from aider's unified prompts. YMMV - 700+ LOC will output malformed patches, so feel free to tune the prompt or use it as a point to help Claude refer back to. For these cases, it is better to rely on aider, so I'm hoping for a unified_diff_write tool that we can deprecate once limits on output tokens disappear.

https://github.com/saoudrizwan/claude-dev/releases/tag/v1.5.6
Does this attempt to fix the issue?

@vtempest yes! pls see the update at the bottom of this comment.

Rather than asking Claude to provide me with the entire contents of the file back or just the parts of the code that changed, I'll specifically ask for the entire function to be returned for any modified functions. This helped substantially with token usage and my ability to quickly know where code is supposed to go when I'm doing it manually. I feel like it's one zoom level out from "just the code that's changed".

How is everyone feeling about the published fix?  Are we back in business?  Any gotchas?
I had moved over the Cursor but that has its own set of problems!  :)

I still see the code getting overwritten with placeholders :(

Can confirm. I am able to reproduce by asking it to modify claude-dev.

Feature Idea: Have an input in the settings for custom system level instructions, which is a great feature of Cursor Editor your lead competitor.

"Generate complete, untruncated code without omissions. Include all necessary details, imports, and full function/class implementations. Do not use ellipses (...) or comments like 'rest of code here' to abbreviate. Provide the entire requested code in full."

For anybody that faces this issue, Anthropic will be releasing a new fast edit mode that should fix this–it will help claude easily output tokens he's already read, in other words editing files will be much faster and less //rest of code here (+ hopefully cheaper).
In the meantime, I've created a wiki to help work around this issue: https://github.com/saoudrizwan/claude-dev/wiki/Troubleshooting-‐-Claude-Deleting-Code-with-"rest-of-code-here"-Comments

@saoudrizwan first off, thank you for this incredible tool.
In the latest version, these issues still happen quite often. It would be really useful if you could have "// rest of code here" sections automatically get swapped for the contents. Often there is a lot of copy/pasting required since // rest of code here gets combined as an deletion + insertion in the UI.
Alternatively, I don't know if this is possible but perhaps the diff can be modified so that any "// rest of code here" sections are always stand alone deletions and never combined as a single change with additions.
If "// rest of code here" could be automatically swapped then it might actually be better to request Claude output those and reduce useless output tokens. Maybe it could be prompted to include a special output for these deletions including line numbers.

I'm having a hard time working with Claude and cline. It's almost unworkable currently because of the truncations. This was better before :-(

I'm having a hard time working with Claude and cline. It's almost unworkable currently because of the truncations. This was better before :-(

Try adding to each prompt: Print out all lines of the file even if not modified. Do not omit any code and do not say "rest of code here"

I don't think this one should be marked as Closed. It is a huge bug, regardless of the cause.  Suggested workarounds don't seem to work consistently, if at all.

I am finding myself in endless truncation loops as well e.g. you tell it to restore truncation and it truncates somewhere else. Having an easy option to revert "rest of the code" blocks would be a good workaround.

Yes we should vectorize the lines to see which ones from the original to use to fill in the "rest of the code here" blocks. You can code this with regex, hf transformers, and airesearch.js.org


Request:  intelligent reactions to MQTT diagnostic commands

There are occasions when I'd like to force the add-on to try and refresh diagnostic data.  For example, when I park my car in my garage and plug it in, I'd like refreshed diagnostics for things like "last trip" data (which will get cleared if the car finishes charging before the next 30 minute polling interval.)
I can forcibly send an MQTT message with a topic of "homeassistant/VIN_NUMBER/command" and payload of '{"command":"diagnostics"}', and that does cause the plugin to forward the request to onstar, but unfortunately, the add-on doesn't seem to recognize the response data from onstar and/or parse it into MQTT sensor/binary_sensor MQTT topics/messages.
So, my feature request is that either the add-on responds to a pre-determined MQTT topic to refresh diagnostic data (bypassing the timer) or that the add-on recognizes the response data from a "diagnostics" MQTT command and parses it to MQTT sensor/binary_sensor messages.
For completeness, this is what I see in the add-on log when I try to send a 'diagnostics' command:
info: Command sent {"command":"diagnostics","timestamp":"2023-12-05 21:49:30"} info: Command completed {"command":"diagnostics","timestamp":"2023-12-05 21:49:45"} info: Command response data {"responseData":{"commandResponse":{"body":{"diagnosticResponse":[{"diagnosticElement":[{"message":"na","name":"AMBIENT AIR TEMPERATURE","status":"NA","unit":"Cel","value":"11"}],"name":"AMBIENT AIR TEMPERATURE"},{"diagnosticElement":[{"message":"na","name":"LAST TRIP TOTAL DISTANCE","status":"NA","unit":"KM","value":"0.015625"}],"name":"LAST TRIP DISTANCE"},{"diagnosticElement":[{"message":"na","name":"ODOMETER","status":"NA","unit":"KM","value":"13972.5"}],"name":"ODOMETER"},{"diagnosticElement":[{"message":"GREEN","name":"TIRE PRESSURE LF","status":"NA","unit":"KPa","value":"264.0"},{"message":"GREEN","name":"TIRE PRESSURE LR","status":"NA","unit":"KPa","value":"260.0"},{"message":"na","name":"TIRE PRESSURE PLACARD FRONT","status":"NA","unit":"KPa","value":"260.0"},{"message":"na","name":"TIRE PRESSURE PLACARD REAR","status":"NA","unit":"KPa","value":"260.0"},{"message":"GREEN","name":"TIRE PRESSURE RF","status":"NA","unit":"KPa","value":"268.0"},{"message":"GREEN","name":"TIRE PRESSURE RR","status":"NA","unit":"KPa","value":"264.0"}],"name":"TIRE PRESSURE"}]},"completionTime":"2023-12-06T02:49:43.980Z","requestTime":"2023-12-06T02:49:30.890Z","status":"success","type":"diagnostics","url":"https://api.gm.com/api/v1/account/vehicles/VIN_NUMBER/requests/45849256997"}},"timestamp":"2023-12-05 21:49:45"}

Thank you for the response.  I'll (ab)use the add-on restart service for now, and keep an eye on that upstream issue.

Please beware that restarting too often will result in 429 and 500 errors akin to having the refresh threshold set to too low of a value as I have noted in the documentation.

For now, I'm only restarting twice a day.  I hope that wouldn't result in rate-limiting.
Please leave this issue open.  I'm going to (try and) spend some time getting myself familiar enough with node.js to modify the project to refactor the diagnostics parsing into a function (or whatever node.js calls it) that can be used by both the timer interval, and the manual command code.    I'm not sure if I'll PR here or upstream.  Looking over the upstream github, I have the impression that the owner is no longer able to test anything, so a PR to them wouldn't get a good review.

Looking at the update today, it occurred to me that the main (timer) loop probably shouldn't be doing any diagnostic parsing.
Instead:
The block of code that subscribes the command topic (near the top of configureMQTT in index.js) should handle the parsing of any diagnostic data (as well as location data.)  Then, the loop runs every refreshInterval would do nothing but form new command topics that it would publish (and the previously mentioned subscription code would parse.)
Why?  First, this normalizes the parsing of diagnostic code to a single place AND the sending of ANY commands to one place.  Second, the intervalTimer loop could be changed to allow an option of never looping, thereby allowing home assistant automations to control the polling of data.
Why allow HA to control polling?  Assuming a limit of 5 "polls" while a car is turned off, I don't really care what my car is doing while I'm home and sleeping, but would much rather that a poll is "available" when I wake up in the morning so I can get a notification that (for example) my tire pressure is low.  I could see HA automations that run at different intervals with various conditions to allow this flexibility.
In addition, any response data that isn't recognized (for example, the response for a "lock" command) should be shoved into a topic and published.  This would allow easily using any new (or previously unknown) commands without changing the add-on code.  Instead, any data could be parsed using HA template sensors.

@BigThunderSR see https://github.com/michaelwoods/onstar2mqtt/pull/338
The only thing I didn't do was understand what I was doing. lol.  It works the way I expect, though.  One thing I didn't do was to figure out a way to disable the timerInterval loop.  I guess setting it to a 5 hour interval or something is sufficient for my planned usage.

I can't just make simple mods.  Now I'm planning to completely re-write the availability code to properly integrate with MQTT with proper availability topics that don't require users to muck with things.  Also to publish "button" configs for things like start/cancel start/lock/unlock.  I suppose I'm going to have to create a proper dev environment for doing this stuff instead of what I was doing (a simple linux container and "nano")

You might consider holding off on grabbing that PR...  It works just fine, but I suspect I'm going to have a really busy weekend with this code.  There are a few diagnostic elements that aren't being passed along, there are some that are being passed in odd ways (why convert the tire pressure to PSI, but not the recommended pressure?) and some device strings that are annoyingly misrepresented (lifetime_mpge and lifetime_mpge_mpge.)
At this point, I might just run with my fork without sending a PR back to the origin.  I'll send him a note asking him if he wants the PR if I do large changes...


Clean up Jetty test server

Removes the outdated GeoExt examples from version 2
Updates the index.html file that you see when opening http://localhost:9876/e2egeoserver

Closes #295

:+1:

Thanks!


[Bug]: Publishing config doesn't work

What happened?
When using command
php artisan vendor:publish --tag="laravel-optimize-init-db-connection-config"
result is
INFO  No publishable resources for tag [laravel-optimize-init-db-connection-config].
How to reproduce the bug
Use command
php artisan vendor:publish --tag="laravel-optimize-init-db-connection-config"
Package Version
1.0
PHP Version
8.1.13
Laravel Version
10.44.0
Which operating systems does with happen with?
Windows, Linux
Notes
No response

Same here

I updated the command to: php artisan vendor:publish --tag="optimize-init-db-connection-config". Sorry for typo


Set up a CI build (eg. travis)

Happy to set it up code-wise, but someone in the KhronosGroup group will need to enable it.

Odd... Now the CI doesn't show up anymore.

I think everything's working, it was just that your .travis.yml change resulted in an invalid file.
Closing again :+1:


How to run on the GPU?

Using latest imagededup. I uninstall tensorflow and installed tensorflow-gpu==2.0.
Any ideas why? Anything more to do? I've already run successfully apps on tensorflow with GPUs.
Thanks
Output of imagededup.
2020-02-16 16:50:46,167: INFO Start: Calculating hashes...
100%|█████████████████████████████████████████████████████████████████████████████████████| 2350/2350 [00:03<00:00, 689.70it/s]
2020-02-16 16:50:49,706: INFO End: Calculating hashes!
2020-02-16 16:50:49,708: INFO Start: Evaluating hamming distances for getting duplicates
2020-02-16 16:50:49,708: INFO Start: Retrieving duplicates using Cython Brute force algorithm
100%|███████████████████████████████████████████████████████████████████████████████████████| 14/14 [00:00<00:00, 48690.10it/s]
2020-02-16 16:50:49,848: INFO End: Retrieving duplicates using Cython Brute force algorithm
2020-02-16 16:50:49,848: INFO End: Evaluating hamming distances for getting duplicates

Pip list
Package              Version
-------------------- ----------
absl-py              0.9.0
astor                0.8.1
cachetools           4.0.0
certifi              2019.11.28
chardet              3.0.4
cycler               0.10.0
Cython               0.29.15
gast                 0.2.2
google-auth          1.11.2
google-auth-oauthlib 0.4.1
google-pasta         0.1.8
grpcio               1.27.2
h5py                 2.10.0
idna                 2.8
imagededup           0.2.2
joblib               0.14.1
Keras-Applications   1.0.8
Keras-Preprocessing  1.1.0
kiwisolver           1.1.0
Markdown             3.2.1
matplotlib           3.1.3
numpy                1.16.6
oauthlib             3.1.0
opt-einsum           3.1.0
Pillow               6.2.2
pip                  20.0.2
protobuf             3.11.3
pyasn1               0.4.8
pyasn1-modules       0.2.8
pyparsing            2.4.6
python-dateutil      2.8.1
PyWavelets           1.0.3
requests             2.22.0
requests-oauthlib    1.3.0
rsa                  4.0
scikit-learn         0.22.1
scipy                1.4.1
setuptools           45.2.0
six                  1.14.0
tensorboard          2.0.2
tensorflow-estimator 2.0.1
tensorflow-gpu       2.0.0
termcolor            1.1.0
tqdm                 4.42.1
urllib3              1.25.8
Werkzeug             1.0.0
wheel                0.34.2
wrapt                1.11.2

NVIDIA SMI output
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 410.48                 Driver Version: 410.48                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  GeForce GTX 1070    On   | 00000000:01:00.0 Off |                  N/A |
|  0%   50C    P8    15W / 200W |      1MiB /  8117MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+

When I run
python -c 'import tensorflow as tf; print(tf.test.is_gpu_available())'
I get
2020-02-16 16:57:57.319386: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2020-02-16 16:57:57.324196: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2020-02-16 16:57:57.324669: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x4b05070 executing computations on platform Host. Devices:
2020-02-16 16:57:57.324693: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): Host, Default Version
2020-02-16 16:57:57.326121: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcuda.so.1
2020-02-16 16:57:57.442858: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1006] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-16 16:57:57.447548: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x4bea150 executing computations on platform CUDA. Devices:
2020-02-16 16:57:57.447562: I tensorflow/compiler/xla/service/service.cc:175]   StreamExecutor device (0): GeForce GTX 1070, Compute Capability 6.1
2020-02-16 16:57:57.447694: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1006] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-16 16:57:57.448190: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1618] Found device 0 with properties:
name: GeForce GTX 1070 major: 6 minor: 1 memoryClockRate(GHz): 1.7465
pciBusID: 0000:01:00.0
2020-02-16 16:57:57.448388: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.0
2020-02-16 16:57:57.449189: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10.0
2020-02-16 16:57:57.449862: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10.0
2020-02-16 16:57:57.450007: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10.0
2020-02-16 16:57:57.450828: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10.0
2020-02-16 16:57:57.451509: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10.0
2020-02-16 16:57:57.453533: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-02-16 16:57:57.453597: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1006] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-16 16:57:57.454020: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1006] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-16 16:57:57.454393: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1746] Adding visible gpu devices: 0
2020-02-16 16:57:57.454416: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.0
2020-02-16 16:57:57.455013: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1159] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-02-16 16:57:57.455023: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1165]      0
2020-02-16 16:57:57.455029: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1178] 0:   N
2020-02-16 16:57:57.455144: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1006] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-16 16:57:57.455575: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1006] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-02-16 16:57:57.455966: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1304] Created TensorFlow device (/device:GPU:0 with 7619 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1070, pci bus id: 0000:01:00.0, compute capability: 6.1)
True

Which shows that my GPU is recognized by tensorflow.

Hi
Looking at your code, you are running a hashing algorithm which can not make use of GPU.  You will need to use CNN method to do so. Have a look at the documentation: https://idealo.github.io/imagededup/methods/cnn/

Ok fair enough!


Latent support for VS 2019

VS2019 not supported officially.
Right now Ghidra team does not support VS2019 and does not have capacity to do that. At the same time community regularly asking for that.
Describe the solution you'd like
I would like small change in the build process, specifically in the vsconfig.gradle in the manner similar to https://github.com/NationalSecurityAgency/ghidra/pull/730 . More specifically to have function for look for VS locations based on version and edition. In official version you will have only VS2017 enabled, and who want to maintain fork (or small customaization, if you like) can do that with less effort.
Also the build PDB project using devenv does not allow to specify parameter, I would like that to be switched to MSBuild if possible.
Describe alternatives you've considered
Other way, which I look which is viable is to have ability to specify VS version and edition and Windows SDK as gradle parameters, to support overriding of VS version and Windows SDK version
Additional context
I want to ask you for solution which would be less burden for you, at the same time, allow for better configuration to work in different environment then yours.
I also willing to do all legwork with implementation, since this is me who need this.

You might want to consider to use clang-cl - it's compatible with MSVC, and recently both Google and Mozilla switched to use it for building Chrome and Firefox respectively. Moreover, you will get ASAN, UBSAN, whatever for free (at Windows platform):

http://blog.llvm.org/2018/03/clang-is-now-used-to-build-chrome-for.html
http://blog.llvm.org/2018/11/30-faster-windows-builds-with-clang-cl_14.html
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Windows_Prerequisites

While this is an interesting idea, I would think that it does not help with the SDK version and path concerns which would still need to be handled.  In addition, there is the integration with Gradle for the Windows platform.

@kant2002, can you give the latest changes a try?

@ryanmkurtz working without anything from my side

Big thanks!

Great, thanks for your contributions and helping to drive it forward!


Include MOM5 code changes for required input files

Commit da3ba2a to the access-esm1.5 branch of MOM5 stops the model from requiring  unused biogeochemistry input files (related issue here). These changes should be included in the next version of the ESM1.5 MOM5 executable.

Can we close this now @blimlim?

Can we close this now @blimlim?

No - we hadn't updated to a version of MOM5 that includes these changes. I've just tagged a new version of MOM5 and submitted a PR to update ACCESS-ESM1.5 to use it.


Some questions of batch messages consumption

Which Component
spring-cloud-stream-binder-rocketmq
Describe the solution you'd like
Support batch messages consumption.
Describe alternatives you've considered
When we would like to use batch messages consumption feature, we can set consumeMessageBatchMaxSize property to consumer, then consumer will pull a batch of messages on method consumeMessage of DefaultMessageListenerConcurrently or DefaultMessageListenerOrderly if message accumulation occurs. If one of this batch of messages throw an exception, the batch of messages would be sent to %RETRY% topic, it would cause Repeat Purchases. I have some ideas about this question.

Set a header to message which have been consumed successfully, and filter messages by the property.
Sign the index of the message which throw exception, and set the index to context by context.setAckIndex(index). It needs to modify method processConsumeResult of class ConsumeMessageConcurrentlyService to set the number of retransmitted messages.
Hand over the processing logic to the users.

please try the beta version #2029 to see if it solves your problem, or provide us more information to help find the latent bug.


KML import has corrupted degree symbol rendering for info

I exported a track file from Places tab on the Layer menu, and on re-import, it has corrupted rendering of the degree symbol.

The exported file is attached
Cookbook track.kml.txt
This is also reproducible as a standard file open, and is also visible in the "Show Features" grid.

jschardet is reporting the detected character encoding as EUC-KR, which is definitely incorrect (should be UTF-8).

This is detecting correctly for me today locally on master. @bradh can you check it out again?

Still happening on https://master-branch-opensphere-ngageoint.surge.sh/ with the original file.

Submitted ticket with jschardet.
https://github.com/aadsm/jschardet/issues/50
Probably nothing will come of it so we may have to add some additional handlers of our own.

@glennkitchellcaci did you try upgrading jschardet to 2.1.0?

Can no longer reproduce. I didn't try to find the specific fix, but likely associated with the new chardet work.


Font-Awesome v6.2.0

Thank you @owenvoke for the great package.
I saw "resources/svg" already updated to v6.2.0 but when I fresh install via composer it still using v.6.0.0.
How to use the latest version?

Looks like I've forgotten to tag new versions, I'll create a new release shortly. 👍🏻

@suretarget, just released v2.1.0 with Font Awesome v6.2.0 👍🏻


No stars are visible if container was hidden on mount

Thanks for this components! Looks promising for my use case. However, I have a small issue
I'm integrating this component with Preline Tabs and whenever the stars component is mounted on a default hidden tab, the stars are invisible when the tab gets selected. I guess there must be some size calculations that get broken if the container is hidden. If I move the rating component to a default visible tab, everything works as expected.
Not sure how to work around this issue.

Hi @JavoByte, you're totally right. getBBox method is not able to compute the viewport if display: none is applied.
I will try to release a non-breaking fix in the next minor release. Thank you for reporting the issue.
In the meanwhile, you can overcome the issue by mounting/unmounting the rating component on tab change. You should be able to do that by listening to this event.
{isActiveTab ? <Rating /> : null}

i have the same issue when i try to implement the rating component on a modal the stars disappear. Works on local but in productions it fails

same

same!

I'm sorry for this, as it has been caused by some bad architectural decisions I took almost two years ago when I first started developing this package.
I refused to make compromises on various features, especially on how to handle custom SVGs. This resulted in a over-engineered Rating Item API (all those techniques intended to render and style the stars).
For this reason, I found myself not able to fix this bug "the right-way" without introducing breaking changes. I'm planning in the near future to fully rewrite of this package with a lightweight API which enables proper SSR support and minimum waste of unused code.
That said, I've released v1.4.0 that fixes the issue. Some important considerations:
The fix detects whether a parent element has display: none, if yes, it will observe its attributes until the display value is not set to none anymore. When this condition is met, it will recompute the SVG size which will be calculated properly as the parent is now visible.
There will be only one scenario where this won't work as expected:
❌  Won't work
.SomeOtherParent .RatingParent {
  display: hidden
}

✅ Works
.RatingParent {
  display: hidden
}

Basically, the display property should be toggled directly on the hidden element and not delegated by another parent element.
While this is an edge case, this can be worked around by conditionally mount/unmount the rating component as explained in the first reply of this issue.
Thank you,
Simone

I'm sorry for this, as it has been caused by some bad architectural decisions I took almost two years ago when I first started developing this package.
I refused to make compromises on various features, especially on how to handle custom SVGs. This resulted in a over-engineered Rating Item API (all those techniques intended to render and style the stars).
For this reason, I am not able to fix this bug the "right-way" without introducing breaking changes. I'm planning in the near future to fully rewrite this package with a lightweight API with proper rendering logic and SSR support.
That said, I've released React Rating v1.4.0 that should fix the issue for 99% of the users that are facing it.
Some important considerations:
The fix detects whether a parent element has display: none, if yes, it will observe its attributes until the display value is not set to none anymore. When this condition is met, it will recompute the SVG size which will be calculated properly as the parent is now visible.
There will be only one scenario where this won't work as expected:
❌  Won't work
.SomeOtherParent .RatingParent {
  display: hidden
}

✅ Works
.RatingParent {
  display: hidden
}

Basically, the display property should be toggled directly on the hidden element and not delegated by another parent element.
While this is an edge case, this can be worked around by conditionally mount/unmount the rating component as explained in the first reply of this issue.
Thank you,
Simone

I'm sorry for this, as it has been caused by some bad architectural decisions I took almost two years ago when I first started developing this package.
I refused to make compromises on various features, especially on how to handle custom SVGs. This resulted in a over-engineered internal Rating Item API (all those patterns intended to render and style the stars).
For this reason, I am not able to fix this bug the "right-way" without introducing breaking changes. In the near future, I'm planning to fully rewrite this package with a lightweight API with a streamlined rendering logic and proper RSC support.
That said, I've released React Rating v1.4.0 that should fix the issue for 99% of the users that are facing it.
Some important considerations:
The fix detects whether a parent element has display: none, if yes, it will observe its attributes until the display value is not set to none anymore. When this condition is met, it will recompute the SVG size which will be calculated properly as the parent is now visible.
There will be only one scenario where this won't work as expected:
❌  Won't work
.SomeOtherParent .RatingParent {
  display: hidden
}

✅ Works
.RatingParent {
  display: hidden
}

Basically, the display property should be toggled directly on the hidden element and not delegated by another parent element.
While this is an edge case, this can be worked around by conditionally mounting/unmounting the rating component as explained in the first reply of this issue.
Thank you,
Simone


ApprovalDAO reputObj context update

the branch relies on https://github.com/kgrgreer/foam3/pull/2587. Please review the change in approvalDAO

@adamfox-nanopay the subject in the context has fraud-ops as real user and business user as user, the subject userpath has all the fraud-ops user, individual user and business user so if you need to access if you need the individual user.


Create code coverage during the CI builds

The code coverage is only created for one PHP version (the highest version)
per build as there is no point to have multiple code coverage uploads
for the same code change.

Note: This needs the secret CODACY_PROJECT_TOKEN set in the GitHub repository configuration to work.


Flag color validation fails on some existing transactions fetched through the API

This might have been caused by a change in data format, combined with new validations added in recent versions of the gem.
I'm trying to fetch all transactions in a budget using the API:
ynab_api = YNAB::API.new(YNAB_ACCESS_TOKEN)
budget = ynab_api.budgets.get_budgets.data.budgets.find { _1.name == YNAB_BUDGET_NAME }
transactions = ynab_api.transactions.get_transactions(budget.id)

And I get the following error:
.../ynab-3.4.0/lib/ynab/models/transaction_flag_color.rb:39:in `build_from_hash': Invalid ENUM value  for class #TransactionFlagColor (RuntimeError)
        from .../ynab-3.4.0/lib/ynab/models/transaction_flag_color.rb:31:in `build_from_hash'
        from .../ynab-3.4.0/lib/ynab/models/transaction_detail.rb:431:in `_deserialize'

(continued...)
        from .../ynab-3.4.0/lib/ynab/models/transaction_detail.rb:386:in `block in build_from_hash'
        from .../ynab-3.4.0/lib/ynab/models/transaction_detail.rb:376:in `each_pair'
        from .../ynab-3.4.0/lib/ynab/models/transaction_detail.rb:376:in `build_from_hash'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response_data.rb:171:in `_deserialize'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response_data.rb:123:in `block (2 levels) in build_from_hash'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response_data.rb:123:in `map'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response_data.rb:123:in `block in build_from_hash'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response_data.rb:116:in `each_pair'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response_data.rb:116:in `build_from_hash'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response.rb:158:in `_deserialize'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response.rb:113:in `block in build_from_hash'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response.rb:103:in `each_pair'
        from .../ynab-3.4.0/lib/ynab/models/transactions_response.rb:103:in `build_from_hash'
        from .../ynab-3.4.0/lib/ynab/api_client.rb:282:in `convert_to_type'
        from .../ynab-3.4.0/lib/ynab/api_client.rb:242:in `deserialize'
        from .../ynab-3.4.0/lib/ynab/api_client.rb:73:in `call_api'
        from .../ynab-3.4.0/lib/ynab/api/transactions_api.rb:296:in `get_transactions_with_http_info'
        from .../ynab-3.4.0/lib/ynab/api/transactions_api.rb:240:in `get_transactions'

I've also confirmed that the error started happening in the version 3.0.0 of the gem.
Hacking into the gem a little bit, I was able to find that my budget contains a transaction which flag is an empty string, when returned from the API, then I confirmed it by also querying the API directly:
$ curl --no-progress-meter -X 'GET' \
  'https://api.ynab.com/v1/budgets/0a08ffe7-0121-43c9-a9c6-7484e9138bd5/transactions/9aced28e-a240-47ff-b108-2d672872536c' \
  -H 'accept: application/json' \
  -H "Authorization: Bearer $YNAB_ACCESS_TOKEN" | \
jq '.data.transaction'

{
  "id": "9aced28e-a240-47ff-b108-2d672872536c",
  "date": "2022-11-15",
  "flag_color": "", // 💥
  "flag_name": null,
  "account_id": "48d5416c-2351-49ab-ab4e-1d367b9756c7",
  "category_id": "98a88345-80e8-4015-9f4c-8b53858c84b3",
  // ...
}

A transaction with an empty string for flag color is invalid, according to the OpenAPI spec:
https://github.com/ynab/ynab-sdk-ruby/blob/825a49af31fa8bd10ee7406fc40bf72955a1b406/open_api_spec.yaml#L3276-L3287
I can picture three ways to fix this problem:

update the OpenAPI spec (and rebuild the gem), to mark the empty string as a valid value for flag_color
update the YNAB API server implementation to return null values when it meets an empty string from the database
run a migration on the YNAB user data to update all transactions with an invalid flag color

To me, the real fix would be number 3.

Thanks @davidstosik  - I have a PR here to address this.  I'd prefer to gracefully handle this situation in the client for now.


Socket.io websocket getting stucked after a while

Hey guys.
I have a weird behavior with my socket.io implementation, it works well a while (30min - 1 hour uptime), but then it seems connections can't be made any further, look at the print screen of what  Network Inspector shows:

The client keeps receiving these "2" and "3" data forever, that I have no idea what it means.
If I restart the webserver, things get back working:

I'm out of ideas how to debug this, the logs doesn't show anything specific on disconnect, I usually get generic messages like "transport close", "server namespace disconnect" and "client namespace disconnect".
Can anyone please help me debug this further?

Not sure how to debug this, but I would check my nginx / haproxy / apache settings, to check if they're ready for socket connections.

@peteruithoven I don't think it's a proxy issue, otherwise it wouldn't even work to begin with.
I'm more interested in those "3" and "2" sent from the server...they look like some kind of status code, but I can't figure it out.

Those 2 and 3 look more like requests to me. If you look at the request strings, they conten 2 or 3 at the end. Maybe you do something with the requests, as is seen in your later responses ([42, "percentage", 2] like them.

@maxzeroedge I still don't understand. I'm not making those '2' and '3' requests, not directly at least... The ["percentage", 2] is my json-encoded data, but the "42" inserted in the beginning is inserted by socket.io 42["percentage", 2]

Then that 42 is probably the size of data transmitted. 4x10 + 2
That makes me wonder, if those 2 and 3 are also sizes of data transmitted, in which situation, it must mean that there is some problen with requests at server. Can you log the requests you get at server to see what happens when such data starts to appear?

I don't think it's the size of data. Why 4x10? It's a 17-length string, shouldn't it be 17 bytes then? Since the chars used are within the ascii table, each one should occupy 1 byte only. Also, when the data is bigger, it always sends "42", independently of data size. Hence the why I think it's some kind of status code.

About logging the requests,
That is an interesting point, I've tried that in the past. What happens is the client connects to the server, but that's it. It doesn't receive any messages or sends them.

I'm closing this, since I don't need it anymore.


Handles custom YAML tag constructors properly

Constructor implementation:

[x] !cite
[x] !color
[x] !ref
[x] !metadata handling Metadata
[ ] !metadata handling MetadataColumn
[x] Artifacts passed as Metadata are included as inputs/parents in provenance
[x] !no-provenance
[x] !set

Also:

[x] build ProvNodes for v0 archives to support Union

MetadataColumns seem to be indistinguishable from Metadata objects as implemented by the framework. Both are stored as .tsv, and represented in memory as pandas DataFrames. (Historically, columns were stored as pd.Series, but this interfered with saving row labels.)
I'm going to avoid dealing with this directly for now, and indefinitely if possible. :crossed_fingers: All of this metadata will get read into memory and made available to the node. Worst-case scenario, we create a new representation of this data in provenance, and bump the archive version to support replay.


Methods for work with marketing

API has method https://api.sendgrid.com/v3/contactdb/lists
And this method always return in Postman (i used API key)
{     "errors": [         {             "field": null,             "message": "access forbidden"         }     ] }
And the bundle does the same. May be  it is because it's legacy endpoint.
Also API has method https://api.sendgrid.com/v3/marketing/lists
In Postman it works well.
But bundle don't have a method for work with it.
How I can manage marketings with this bundle?
Technical details:

sendgrid-php Version: "sendgrid/sendgrid": "^7.3",
PHP Version: 7.3

This can be an issue with API key permissions.
Maybe it has selected option 'Restricted access' and 'Marketing Campaigns' isn't enabled.
Tried myself. After creating a new API key without the permission enabled, I got the same error. After enabling I got {"lists":[]} with HTTP status 200.
The requests for /marketing/lists don't work for me. Also when having 'Full access'.

Its definitely because of legacy. I checked and I have no access to legacy endpoints that haven't been carried forward. What I don't understand is why SendGrid is still on v3/ even though v3 is technically a completely different API. There are no GitHub docs for the current API.
Worse still. there appears to be a tonne of stuff missing from the new marketing endpoints. I have no way of accessing all contacts by a given list. I want to retrieve all contacts within list A, but I cannot do that, I have to retrieve ALL contacts, then just filter through them manually.
And ideas why this is such a mess?

omg I found it: https://sendgrid.com/docs/API_Reference/api_v3.html
They want you to run a query yourself:
{
  "query": "email LIKE 'ENTER_COMPLETE_OR_PARTIAL_EMAIL_ADDRESS_HERE%' AND CONTAINS(list_ids, 'YOUR_LIST_IDs')"
}

Thank you. I did not guess to build requests on the fly, even if they are not in the class Client.
$sg->client->marketing()->lists()->get() work well.

If its any help. Here is what I did to get a list by it's name, followed by all contacts in that list:
$name = 'App Users';

// get lists, search by name, and use the index to fetch the list item.
$lists = $sg->marketing()->lists()->get();
$index = array_search($name, array_column($lists, 'name'));
$list = $lists[$index];

// get the list item ID and use it to generate a query for all contacts within that list.
$query = "CONTAINS(list_ids, '" . $list['id'] . "')";
$contacts = $sg->marketing()->contacts()->search()->post(compact('query'));

Note, this does not include formatting the response.


Add Turbolinks

Turbolinks makes navigating Chalk even faster. For only 33 KB, you get an SPA that renders additional page views ~2x faster. There's a small learning curve to Turbolinks, but I find the benefits far outweigh the costs. This PR will manually add Turbolinks and restructure the existing JS for compatibility.
Check it out and let me know what you think. I'm also planning to strip jQuery, load all JS async, and  slightly enhance the page transition (by adding a fade out on link clicks). If any of this sounds interesting, I can send additional pull requests.

@bruncun I like the idea of using turbolinks for sure. I'm just a bit hesitant of ditching jQuery as this is a template for everyone and people tend to know jQuery very well ;-)
I would love to see everything ported over to turbolinks with the async js for sure! Can you continue on this PR? Also try importing turbolinks with bower eventually?
Afterwards I can add documentation on how to add jQuery if they wish to.

@nielsenramon You can see all my performance changes live on my blog. Includes:

Turbolinks
jQuery removal (replaced fluidbox with baguetteBox.js
Inline CSS (cumbersome manual process)
Webfonts removed
Icon set replaced with necessary inline SVG icons

Loading the JS async not only added no obvious added performance, but it also caused some funky/tricky behavior from both Turbolinks and Google Analytics, so I just opted to load it normally in the header. Like dropping jQuery, async would be more trouble than its worth for most.
All in all, I got the initial page load down to 2 requests and like ~20 kb, gzipped. Really pleased with the results.
I hadn't considered jQuery's convenience. Did you know that adding Turbolinks would require that developers stop installing behavior with jQuery's ready event, and instead opt for listening to "turbo links:load"? It's a nuance, but more than enough to cause headaches for anyone expecting the typical jQuery workflow.
🤔

@bruncun Great to see your version life! Looking good.
The main issue I am having with your solution is that the JavaScript is loaded in the head. Which PageSpeed punishes a lot. If that would be fixed it would be a no-brainer for me to get turbolinks in.

I really like the fluidbox transition here (mainly a visual choice) so not really into replacing it with yours.
Inline CSS is something I though about to, maybe this is something for the build process. Will look into it.
I am also planning on replacing the webfonts with a custom icon set for Chalk.

Is it really such a hassle to load all the JS Async? I guess that would fix the PageSpeed punishment.

@bruncun, I've made a few of these changes to my blog as well - local webfonts, no jQuery, and svg icons. I don't understand, however, why you would inline css given that you lose the caching benefit on multiple page loads.
All in all I welcome the changes, especially since turbolinks degrades nicely with no-js.

@migueldemoura There are tradeoffs to be sure. In addition to 1 less request, by inlining CSS we avoid blocking the critical rendering path. It's worth noting that, on subsequent visits, Turbolinks merges the new page's content into the existing window, eliminating the need to reload stylesheets. Also, our inline CSS still gets cached inside our HTML assets. I don't expect mine to change often, and my gzipped total payload is ~25kb so I'm personally okay with this caching strategy. YMMV.
Its awesome to see some excitement about Turbolinks! I should be wrapping this branch up this weekend.

Turbolinks merges the new page's content into the existing window

Didn't dive deep into Turbolink's spec, but if that's the case then we should be fine.

@bruncun, can you add rel="noopener noreferrer" to all </a > with target=_blank?
Issue POC: lcamtuf.coredump.cx/switch/

@nielsenramon Branch updates!

Use Bower to install Turbolinks. Had to create a "turbolinks5" package because the original is taken and author seems to have gone AWOL.
As requested, return Javascript include to end of body tag to avoid PageSpeed penalties. From a Turbolinks perspective, this should be fine. I believe including Turbolinks this way will cause Turbolinks to attempt load on every visit, but everything should work just fine.
I encountered some issues with the social share links. The onClick handlers were breaking with Turbolinks. Rather than preventing Turbolinks on these links, I figured this could be a good refactor.

@migueldemoura Nice catch. https://github.com/nielsenramon/chalk/pull/40/commits/4097166ae65dc31cefb6827dd3471126047ccb81

@bruncun Nice work! I like the changes. Gonna test out all the basic features before merging this one (pagespeed, google analytics, ...).
Would there be any way to open a window when clicking the social sharing buttons instead of a new tab? I made them that way so they opened the correct size of the share block :-)

@nielsenramon Done! Whatcha think?

@bruncun Sorry for taking so long. Big thanks!

No problem, happy to give back!

@bruncun, are you planning on submitting further pull requests to remove webfonts and replace ionicons with svg icons?

Not at this time, sorry.


Support multi-instance plugins

In some cases, it'd be ok to let some plugins register multiple instances. That's the case for e.g. our SwaggerPlugin:

https://github.com/javalin/javalin-openapi/issues/161 (and some usages reported earlier)

Such usage is currently disallowed:
https://github.com/javalin/javalin/blob/c77e5802b4d296fe65f76610ca025edca897b306/javalin/src/main/java/io/javalin/plugin/PluginManager.kt#L12
As a workaround, users can extend plugin and register sth like class SwaggerPlugin2 : SwaggerPlugin, but it's slightly inconvenient. It might be a good idea to provide an interface like MultiInstancePlugin/RepeatablePlugin that'd bypass this check.

RepeatablePlugin

I'm sold.

Let's provisionally add this to our 6.x backlog.

🤔
Why can't it be part of 5.x ?

Why can't it be part of 5.x ?

It would not be a breaking change, but imo it might be time to start work on 6.x since there are a few issues on backlog already.

It's better if we get these features out ASAP though, we want to make the major release as little as possible, only containing breaking changes.


Add some debugging flags

Codecov Report
Attention: 24 lines in your changes are missing coverage. Please review.

Comparison is base (5f58031) 64.66% compared to head (0559f72) 63.61%.
Report is 11 commits behind head on main.

Files
Patch %
Lines

pydatatask/task.py
50.00%
19 Missing :warning:

pydatatask/main.py
50.00%
4 Missing :warning:

pydatatask/pipeline.py
93.75%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
- Coverage   64.66%   63.61%   -1.06%
==========================================
  Files          34       34
  Lines        5570     5744     +174
==========================================
+ Hits         3602     3654      +52
- Misses       1968     2090     +122

Flag
Coverage Δ

unittests
63.61% <61.29%> (-1.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


QGIS Archive Automatic Signing Key (2019) expired [2020-08-08]

Hi all,
As of today the key used to sign packages (at least on the Debian/Ubuntu) repository is expired, so the installation/update fails.
Please renew or update the key, and if necessary modify the documentation accordingly.
Regards :)

Duplicate of qgis/QGIS#38204


The speed of using multi gpus with tf as backend.

I use Tensorflow as the backend. And I use code here https://github.com/kuza55/keras-extras/blob/master/utils/multi_gpu.py to achieve multi-gpus training.  However, I find that the speed of using two gpus is almost same with using one gpu. Besides, I find when I use one gpu, the usage of gpu is almost 100%; but using two gpus, the usage og each gpu is about 40%-60%.  How can I solve the problem?
My environment:
CPU: 40x Intel E5-2630 v4
Mem: 384GB
GPU: 4x NVIDIA GTX 1080 Ti

Maybe your batch size isn't large enough?

Using a larger batch means that the model need more epochs to achieve the best performance, that is, it would cost more time...

I have the same situation


championify download stuck

Hi i try to update the builds for the patch 7.22 and the app got stuck at 74% i tought that probably the problem could be my pc but y try it in a different one and is the same result, also i reinstalled championify but it was pretty much the same result.

Do you have op.gg selected as a source?

Stale, and I believe was fixed in the last few patches. Closing.


Remove I386_ONLY support

Should we just remove I386 support altogether?
Checklist
Thank you for your contribution. Please answer the following:

[X] I acknowledge that I am authorized to submit this code under
the terms of the Apache License
[ ] All new public APIs are documented
[ ] All new public functions have tests
[ ] All new public functions have fuzzing tests

@botovq addressed your concerns, please take a look if you get a chance.


No reverb FX

In downloaded version reverb not working on chrome ?

Only localy. On server it is ok


REL: 2024.10 (pathogenome) take 2

second attempt at 2024.10 pathogenome release - this time with python 3.10 & including the viromics patch that resolves an upstream dependency that is incompatible with python 3.9/3.10 (relevant PR here)

Test failures in RESCRIPt are due to a SILVA maintenance-related outage:


[ Home Page ] Update the percent bar

Instead of the current Circular percentage bar,

Create this linear percentage bar:

Hey, I would like to try my hand at updating this to a linear progress bar. Can you please assign me?


gget setup alphafold

hello,
I installed gget using the command:
pip install --upgrade gget
then when i tried to install alphafold using the command:
gget setup alphafold
I got the following error:
gget setup alphafold
Traceback (most recent call last):
  File "/home/najib/miniconda3/bin/gget", line 5, in <module>
    from gget.main import main
  File "/home/najib/miniconda3/lib/python3.9/site-packages/gget/__init__.py", line 10, in <module>
    from .gget_alphafold import alphafold
  File "/home/najib/miniconda3/lib/python3.9/site-packages/gget/gget_alphafold.py", line 31, in <module>
    from ipywidgets import GridspecLayout
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipywidgets/__init__.py", line 25, in <module>
    from .widgets import *
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipywidgets/widgets/__init__.py", line 4, in <module>
    from .widget import Widget, CallbackDispatcher, register, widget_serialization
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipywidgets/widgets/widget.py", line 13, in <module>
    from ipykernel.comm import Comm
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipykernel/comm/__init__.py", line 3, in <module>
    from .comm import Comm
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipykernel/comm/comm.py", line 14, in <module>
    from ipykernel.kernelbase import Kernel
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipykernel/kernelbase.py", line 70, in <module>
    class Kernel(SingletonConfigurable):
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipykernel/kernelbase.py", line 79, in Kernel
    processes: t.Dict[str, psutil.Process] = {}
AttributeError: module 'psutil' has no attribute 'Process'

i uninstalled the psutil and installed again, i got the same error.
what is the possible solution?
thank you.

Hi, thanks for reaching out. I have not encountered this error before. My first recommendation would indeed be to reinstall psutil using something like pip3 install --ignore-installed psutil. Could you please tell me which version of gget and psutil you currently have installed?

thank you for your reply.
i checked the version for gget and it is 0.27.2 and for psutill and it is 5.9.4.
i tried to reinstall psutil using
pip3 install --ignore-installed psutil
and then run:
gget setup alphafold
i got the following error:
Traceback (most recent call last):
  File "/home/najib/miniconda3/bin/gget", line 5, in <module>
    from gget.main import main
  File "/home/najib/miniconda3/lib/python3.9/site-packages/gget/__init__.py", line 10, in <module>
    from .gget_alphafold import alphafold
  File "/home/najib/miniconda3/lib/python3.9/site-packages/gget/gget_alphafold.py", line 31, in <module>
    from ipywidgets import GridspecLayout
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipywidgets/__init__.py", line 25, in <module>
    from .widgets import *
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipywidgets/widgets/__init__.py", line 4, in <module>
    from .widget import Widget, CallbackDispatcher, register, widget_serialization
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipywidgets/widgets/widget.py", line 13, in <module>
    from ipykernel.comm import Comm
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipykernel/comm/__init__.py", line 3, in <module>
    from .comm import Comm
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipykernel/comm/comm.py", line 14, in <module>
    from ipykernel.kernelbase import Kernel
  File "/home/najib/miniconda3/lib/python3.9/site-packages/ipykernel/kernelbase.py", line 35, in <module>
    import psutil
  File "/home/najib/miniconda3/lib/python3.9/site-packages/psutil/__init__.py", line 241, in <module>
    raise ImportError(msg)
ImportError: version conflict: '/home/najib/miniconda3/lib/python3.9/site-packages/psutil/_psutil_linux.cpython-39-x86_64-linux-gnu.so' C extension module was built for another version of psutil (5.9.1 instead of 5.9.4); you may try to 'pip uninstall psutil', manually remove /home/najib/miniconda3/lib/python3.9/site-packages/psutil/_psutil_linux.cpython-39-x86_64-linux-gnu.so or clean the virtual env somehow, then reinstall

It looks like the psutil issue was fixed. It seems like there are some issues with other packages in your environment. I would recommend trying the following:
First, reinstall/update numpy since the issue might be coming from numpy (pip install -U numpy).
If that doesn’t fix the issue, I would recommend restarting in a fresh Python environment (conda environments manual).
Alternatively, check out this solution. It might help fix the issue in the current environment.

It looks like the psutil issue was fixed. It seems like there are some issues with other packages in your environment. I would recommend trying the following: First, reinstall/update numpy since the issue might be coming from numpy (pip install -U numpy).
If that doesn’t fix the issue, I would recommend restarting in a fresh Python environment (conda environments manual).
Alternatively, check out this solution. It might help fix the issue in the current environment.

hello,
thank you for your suggestion.
i did as your suggestion and reinstall numpy and it is wokring fine.
thanks a lot.

Great! :)


HttpClient.Post sends separate TCP packets for Header and Body

This was a general issue with .net I am assuming. I noticed it when I started looking at .net core source code.
Using Microsoft Message Analyzer, I can see that post data using the HttpClient is being sent in two tcp packets. One for the header, then one for the post data. This data could easily fit into one packet, however it is being split into two. I have explicitly turned on nagling and expect 100 continue off using the ServicePointManager, though, it doesn't seem to help.
public void TestRequest()
    {
        var uri = new Uri("http://www.webscantest.com/");
        ServicePointManager.Expect100Continue = false;
        ServicePointManager.UseNagleAlgorithm = true;
        var p = ServicePointManager.FindServicePoint(uri);
        p.Expect100Continue = false;
        p.UseNagleAlgorithm = true;
        HttpClient client = new HttpClient();
        client.DefaultRequestHeaders.Add("Connection", "close");

        var values = new Dictionary<string, string>
        {
            { "thing1", "hello" },
            { "thing2", "world" }
        };

        var content = new FormUrlEncodedContent(values);

        var response = client.PostAsync("http://www.webscantest.com/", content, CancellationToken.None).Result;
    }

https://stackoverflow.com/questions/49727720/c-sharp-httpclient-post-force-single-packet/49788340#49788340

moved to https://github.com/dotnet/corefx/issues/29080


Integration Test - Release - 2020/08/11 15:08:47

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest2 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Hotstar clone project is not loading thumbnails

Describe the bug
In the Hotstar clone project the website is not loading the images & video thumbnails
Expected behavior
The thumbnails should have been properly rendered
Screenshots

Additional context
I can try to fix the bug if needed

@dhwanish-3 please go through contributing.md
& thanks for raising an issue


Translate Wiki reference to Brazilian Portuguese

to Translate files:
aragonos.md
devplan.md
index.md
\i18n\pt-Br..

ok, Qui postremus abit, redeuntibus ostia claudet

Sorry, we don't have the bandwidth to maintain those yet


CANNOT LINK EXECUTABLE

When i run brew install gcc into termux i get this error and it advises me to install clang or gcc. Clang is up to date using both pkg install clang and apt install clang. Any advice would be awesome.
I am simply trying to get brew running and see that all ia good. Obviously its not all good.

Please complete the issue template. Please try installing gcc. clang should work, but it's less tested. Please report the exact output of brew config and brew install hello.


Update SECURITY.md

The correct way to submit non-public security issues as per https://apache.org/security/

@iamamoose thanks


Automate documentation builds with nox

Description
I find the developer documentation for this theme to be pretty complex, especially as someone without a lot of background in npm / webpack / yarn workflows. As noted in #468 , I have a lot of weird behavior when I run yarn build:dev and I have no idea how to resolve this. It makes developing this theme really cumbersome for me, and I suspect that other potential developers may have similar issues since many Python devs who come to this theme are not familiar with JS workflows either.
I think it'd be helpful if we provided a one-command method of installing development dependencies, generating asset bundles, and building / previewing the documentation.
I've found that the nox package is a really nice / lightweight way to do things like this. For example, we could turn most of our "Set up a development environment" documentation into two commands:
pip install nox
nox -s docs

Benefits of this approach
I think the main benefits to this are:

nox is more python-friendly than manually installing and running yarn
nox is more flexible and we could also use it for running our tests
nox handles the virtual environment for you, so you're more certain that your dev environment is what you expect.

Implementation details
I think that this could be accomplished with a nox config like this:
@nox.session(venv_backend='conda')
def docs(session):
    session.conda_install("--channel=conda-forge", "yarn")
    session.install("-r", "docs/requirements.txt")
    session.install("-e", ".")

    session.run("yarn", "build")
	session.run("sphinx-build", "docs", "docs/_build/html")

or for docs previews, it might be the following (and invoked with nox -s docs-live:
@nox.session(name="docs-live", venv_backend='conda')
def docs_live(session):
    session.conda_install("--channel=conda-forge", "yarn")
    session.install("-r", "docs/requirements.txt")
    session.install("-e", ".")

    session.run("yarn", "build:dev")

+100.  I don't know if nox is the best fix, but a fix is definitely needed.  Currently, contributing to the theme requires:

conda install -c conda-forge --file docs/requirements.txt
installing nodejs --- specifically version 14 (this is not documented on the contributing page; conda install yarn will pull in version 16, which has lots of version conflicts with other required packages, cf #467)
editable pip install of the repo clone
installing and running yarn
installing and running pre-commit

In addition, I think y'all need to pin some of your yarn dependencies.  I literally just did the above steps in a new conda environment, and running yarn yields over 800 changed lines in yarn.lock versus current master.  That means every time a new contributor sets up a dev environment in order to contribute, they're going to change the package versions for everyone.

As an aside - what do we use webpack for? Is it just for bundling our Javascript and SCSS as assets in the package? If that is the case, I'd also suggest we consider using web-compile, a little pure-python tool that does the same thing without adopting a whole different toolchain.

As an aside - what do we use webpack for? Is it just for bundling our Javascript and SCSS as assets in the package? If that is the case, I'd also suggest we consider using web-compile, a little pure-python tool that does the same thing without adopting a whole different toolchain.

Indeed, it's being used for building the SCSS sources and bundling them. If you have been happily using web-compile with the executable-books projects, it sounds good to switch here as well

I've found web-compile to be much simpler and faster than the whole webpack toolchain. I think our problem is that we're using webpack for a lot of other stuff too, like vendoring fontawesome + bootstrap. In my opinion this is hard to maintain and follow since none of us are JS devs. I am curious what you think about just manually bundling all assets with this repository, removing webpack entirely, and then using web-compile to compile our theme-specific CSS and JS


Import plot_sbc from sbi.analysis

Current import is from sbi.analysis.plot import sbc_rank_plot. Why not from sbi.analysis import sbc_rank_plot?

agree, but this should be possible: https://github.com/mackelab/sbi/blob/d1b840304498e9e424fe38d2e00d0de0e82cac7a/sbi/analysis/init.py#L12

it's indeed possible to from sbi.analysis import sbc_rank_plot, see #661


[REQ] Guide / Tutorial On How To Make Your Own FastFlags For Bloxstrap In The Menu. Like A Doc

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
i feel like people to want make their own bloxstrap fastflags (i know how) but you should make a doc on how to make your own fastflags

We can't make FastFlags. Only Roblox engineers can create them and we have no control over them, such as when one is released, changed, or deleted.

yeah i just found that out sadly i know how to make fastflags but cant test them fully  to its max

yeah i just found that out sadly i know how to make fastflags but cant test them fully to its max

Unfortunately yeah.
@bluepilledgreat Please close.
(Unresolved, unplanned)

yeah i just found that out sadly i know how to make fastflags  but cant test them fully  to its max

@luastatee How???


GCDS Component - Feasibility check

Description:
Review GCDS Server Side Rendering to provide feedback on components.
Acceptance Criteria:

[ ] (Once the DS provides a dev contact) Check if the SSR works on our end and if using the components would be viable.

Ran into issue compiling
https://github.com/cds-snc/platform-forms-client/pull/4281

Having done some initial investigation into feasibility we have 2 initial blockers.
Content security policy https://github.com/cds-snc/platform-forms-client/pull/4281#issuecomment-2358313268
i18N - browser detect script issue https://github.com/cds-snc/platform-forms-client/pull/4281#issuecomment-2355728595

@Abi-Nada @timarney thanks for looking into this! My apologies, I didn't see my github notifications in time. If you need a dev contact, please feel free to ping me directly on Slack as well.
I'll take these two blockers and put it on our project board so we can look into it further.

Added a note on the CSP / nonce blocker here https://github.com/cds-snc/platform-forms-client/pull/4281#issuecomment-2377369468.

Adding some further notes.
Looking at other options i.e. just use the tokens / CSS
Given the Design system CSS relies on the web components existing in the markup we can't just grab the styles and use HTML elements + css classes etc...   makes sense as far as css specificity etc... goes just doesn't play well with our existing setup.
We will need manually update the CSS to match our HTML markup -- if GCDS gets updated the updates won't carry over to out end.

Looks like some others i.e. selects will port over easier
We can grab the css vars from the GCDS repo

Side note the CSS is nice :)


import error

after installing ember serve doesn't run anymore. Any idea what this can be?
while import vendor/showdown.js: found an unknown transformation name amd. Available transformNames are:

Error: while import vendor/showdown.js: found an unknown transformation name amd. Available transformNames are:
    at options.using.forEach.entry (/Users/bas/Documents/Projects/PLHW.ProjectHealthyFeet/applications/hf-app-lab/node_modules/ember-cli/lib/broccoli/ember-app.js:1688:19)
    at Array.forEach (native)
    at EmberApp._import (/Users/bas/Documents/Projects/PLHW.ProjectHealthyFeet/applications/hf-app-lab/node_modules/ember-cli/lib/broccoli/ember-app.js:1679:23)
    at EmberApp.import (/Users/bas/Documents/Projects/PLHW.ProjectHealthyFeet/applications/hf-app-lab/node_modules/ember-cli/lib/broccoli/ember-app.js:1652:10)
    at Class.import (/Users/bas/Documents/Projects/PLHW.ProjectHealthyFeet/applications/hf-app-lab/node_modules/ember-cli/lib/models/addon.js:677:15)
    at Class.included (/Users/bas/Documents/Projects/PLHW.ProjectHealthyFeet/applications/hf-app-lab/node_modules/ember-cli-showdown/index.js:15:16)
    at Class.superWrapper [as included] (/Users/bas/Documents/Projects/PLHW.ProjectHealthyFeet/applications/hf-app-lab/node_modules/ember-cli/node_modules/core-object/lib/assign-properties.js:34:20)
    at project.addons.project.addons.filter.addon (/Users/bas/Documents/Projects/PLHW.ProjectHealthyFeet/applications/hf-app-lab/node_modules/ember-cli/lib/broccoli/ember-app.js:545:17)
    at Array.filter (native)
    at EmberApp._notifyAddonIncluded (/Users/bas/Documents/Projects/PLHW.ProjectHealthyFeet/applications/hf-app-lab/node_modules/ember-cli/lib/broccoli/ember-app.js:542:47)
rattletrap:hf-app-lab bas$

somewhere here.
https://github.com/gcollazo/ember-cli-showdown/blob/master/index.js#L15-L17
the transformations map empty, see https://github.com/ember-cli/ember-cli/blob/f46be84f38a0c790f864d5b982e675c206f59a45/lib/broccoli/ember-app.js#L1708-L1711
This feels like I am missing something here...

What version of ember-cli?

Where your demo app? could not find it...
{
  "name": "plhw-client",
  "description": "Client hf-app-lab application to hf-app-api",
  "private": true,
  "directories": {
    "doc": "doc",
    "test": "tests"
  },
  "scripts": {
    "build": "ember build",
    "start": "ember server",
    "test": "ember test"
  },
  "repository": "",
  "engines": {
    "node": "^4.5 || 6.* || >= 7.*"
  },
  "author": "Bas Kamer <xxxxx>",
  "license": "LicenseRef-LICENSE.md",
  "devDependencies": {
    "bootstrap": "^4.0.0-beta",
    "broccoli-asset-rev": "^2.6.0",
    "broccoli-funnel": "^2.0.1",
    "broccoli-merge-trees": "^2.0.0",
    "ember-ajax": "^3.0.0",
    "ember-array-contains-helper": "^1.3.2",
    "ember-awesome-macros": "^0.40.0",
    "ember-best-language": "^0.0.3",
    "ember-body-class": "^0.4.0",
    "ember-bootstrap": "kaliber5/ember-bootstrap#bs4-beta",
    "ember-bootstrap-cp-validations": "^1.0.0-alpha.0",
    "ember-browserify": "^1.2.0",
    "ember-buffered-proxy": "^0.7.0",
    "ember-can": "^0.8.4",
    "ember-cli": "^2.16.0-beta.1",
    "ember-cli-app-version": "^3.1.0",
    "ember-cli-autoprefixer": "^0.8.0",
    "ember-cli-babel": "^6.8.2",
    "ember-cli-bootstrap-datepicker": "^0.5.6",
    "ember-cli-content-security-policy": "^1.0.0",
    "ember-cli-copyable": "^0.9.6",
    "ember-cli-dependency-checker": "^2.0.1",
    "ember-cli-deploy": "^1.0.1",
    "ember-cli-deploy-build": "^1.1.0",
    "ember-cli-deploy-display-revisions": "^1.0.0",
    "ember-cli-deploy-json-config": "^1.0.0",
    "ember-cli-deploy-revision-data": "^1.0.0",
    "ember-cli-deploy-rsync-assets": "^0.2.1",
    "ember-cli-deploy-slack": "^1.0.0",
    "ember-cli-deploy-ssh-index": "^0.2.3",
    "ember-cli-eslint": "^4.0.0",
    "ember-cli-fastboot": "^1.0.5",
    "ember-cli-gravatar": "^3.9.0",
    "ember-cli-htmlbars": "^2.0.3",
    "ember-cli-htmlbars-inline-precompile": "^1.0.2",
    "ember-cli-inject-live-reload": "^1.7.0",
    "ember-cli-moment-shim": "^3.5.0",
    "ember-cli-qunit": "^4.0.0",
    "ember-cli-release": "^1.0.0-beta.2",
    "ember-cli-sass": "^7.0.0",
    "ember-cli-shims": "^1.2.0-beta.2",
    "ember-cli-sri": "^2.1.1",
    "ember-cli-uuid": "^0.4.1",
    "ember-cookies": "^0.0.13",
    "ember-cp-validations": "^3.4.1",
    "ember-crumbly": "^1.0.7",
    "ember-data": "~2.15.0",
    "ember-data-model-fragments": "^2.14.0",
    "ember-data-url-templates": "^0.4.0",
    "ember-deferred-content": "^0.2.0",
    "ember-element-resize-detector": "^0.1.5",
    "ember-exif-shim": "^0.1.2",
    "ember-export-application-global": "^2.0.0",
    "ember-feature-flags": "^3.0.0",
    "ember-fetch": "^3.4.0",
    "ember-file-upload": "^2.1.1",
    "ember-flex-menu": "^0.0.16",
    "ember-font-awesome": "^4.0.0-alpha.4",
    "ember-g-map": "^0.0.25",
    "ember-i18n": "^5.0.2",
    "ember-i18n-cp-validations": "^3.0.2",
    "ember-light-table": "^1.10.0",
    "ember-line-clamp": "^0.1.3",
    "ember-link-action": "^0.0.36",
    "ember-load-initializers": "^1.0.0",
    "ember-local-storage": "^1.4.0",
    "ember-moment": "^7.4.1",
    "ember-multiselect-checkboxes": "^0.10.3",
    "ember-notify": "^5.2.1",
    "ember-progress-bar": "^0.0.7",
    "ember-qrcode": "^0.0.4",
    "ember-radio-button": "^1.1.1",
    "ember-resize": "^0.0.17",
    "ember-resolver": "^4.0.0",
    "ember-responsive": "^2.0.4",
    "ember-route-action-helper": "^2.0.6",
    "ember-scroll-to": "^0.6.4",
    "ember-shepherd": "^2.7.0",
    "ember-simple-auth": "^1.4.0",
    "ember-source": "~2.15.0",
    "ember-state-services": "^4.0.0",
    "ember-tether": "^1.0.0-beta.0",
    "ember-truth-helpers": "^1.3.0",
    "ember-watson": "^0.9.1",
    "emberx-select": "^3.1.0",
    "eslint-plugin-ember-suave": "^1.0.0",
    "liquid-fire": "^0.28.0",
    "loader.js": "^4.6.0",
    "node-zopfli": "^2.0.2",
    "outdated": "^0.1.1",
    "sockjs-client": "^1.1.4",
    "ui-navigator-service": "^0.3.11",
    "webstomp-client": "^1.0.7"
  },
  "dependencies": {
    "ember-cli-showdown": "^4.1.0"
  },
  "browserslist": {
    "production": [
      "last 2 version",
      "ie 10"
    ],
    "testing": [
      "last 2 versions"
    ],
    "development": [
      "last 2 versions"
    ]
  },
  "version": "0.22.2"
}

@basz unsure what you mean, the dummy app within the addon is the demo app.  But if you can reproduce this bug in a demo project and push it to github that would be helpful.  The dependencies you have listed out likely won't help me much.

ok right... :-) Turns out I was using ember-cli v 2.16.0-beta1. Downgrading to 2.15.1 solves this issue

Thanks you, i'll close this one for you.
If you're interested, this is what I'm using it for... https://gist.github.com/basz/49369483fcda33481eec24398e8a734f, localised markdown...
One more question you might be able to answer; if i use links inside the markdown which point towards routes inside the application, the whole application obviously reloads. Would you know of a (recommended) to prevent such behaviour? I've looked at the link-to helper which adds a 'one' event listener to its rendered html and a stopPropagation. Did you ever deal with anchors inside an ember app coming from showdown?

I've looked at the link-to helper which adds a 'one' event listener to its rendered html and a stopPropagation.

That's how I would achieve this behavior.  This is also how ember-href-to is implemented which might be a decent reference https://github.com/intercom/ember-href-to/blob/master/app/instance-initializers/ember-href-to.js

Did you ever deal with anchors inside an ember app coming from showdown?

I haven't personally.

great tip, thanks! that href-to addons works on any anchor that isn't an ember view in itself. I needed to change nothing. +1

Looks like this is still an issue in 2.16 stable.

confirmed to be solved in 2.16.2


Scaled regions mess up the algorithm

When a scaling transformation is applied to a region (or one of its ancestors), this messes up the polyfill's algorithm in deciding where to cut off that region's content exactly. In effect it will cut it off too late when the region is scaled down, and too early when it is scaled up.
This can be easily verified by applying the following CSS to a region (or one of its ancestors):
transform: scale(2);. Its content will be cut roughly in half.
Why this happens
The reason for this is that the algorithm compares the "intrinsic" height of the region (its offsetHeight) with the space a Range takes up relative to the viewport (via its getBoundingClientRect-method). When the scale is 1, this works just fine. But once a different scale is applied, the intrinsic height of the region remains unchanged, whereas the space the Range takes up in the viewport will change.
The code that retrieves the offsetHeight and offsetWidth can be found here on lines 250 & 251:
https://github.com/FremyCompany/css-regions-polyfill/blob/f508484fbe5d4f208c3592a4e71e729e06582169/src/css-regions/polyfill.js#L249-L253
Possible fix
My suggestion to fix this, is to also switch to the getBoundingClientRect-method when determining the width & height of the region. This would make sure we are comparing apples with apples.
Both an element's offsetHeight and its boundingClientRect take the element's padding and border into account, so no other change seems necessary.
There is however one tiny difference between both: the offsetHeight is always rounded to an integer, whereas the boundingClientRect returns a float. I think this is inconsequential, though if I am wrong the outcome could simply be rounded to an integer afterwards.
IE 8?
Interestingly, the element's boundingClientRect is already requested -as can be seen on line 252- but only to get the coordinates of the element's edges. I expect this is because in IE8 this method does not return a width and height, and so it could not be used to retrieve the actual dimensions. But we could be simply calculate them using the edge-information that is supplied.

This sounds like a good fix, thanks. IE8 support isn't nearly as important as it used to be :)
Hopefully the math you wrote does check out, but even if it doesn't, this shouldn't matter much.


Reviewed the code and plots for this project

@datalorax

Thanks Cassie - in your actual commit, I don't see any edits to the code for the actual project. However, I appreciate the summary above.


Status of this project?

After seashell, the company, seems to be in liquidation, what is the status of this project?

The project could deserve a wave of freshness but without active owner it feels hard for incoming contributions to be merged. What is the status of this project, again? Is there an active fork, or can we fork it? :)

@edufschmidt I'd be interested in taking over the project and moving it to a different Org.
Is there anyway left that has access to Seashell org, or someone that can transfer the project?

hey @gaby thanks for your interest. I believe creating a fork would be more appropriate than moving it to a different org, but feel free to contact me on linkedin to discuss the alternatives.


[Snyk] Security upgrade @kubernetes/client-node from 0.18.1 to 1.0.0

Snyk has created this PR to fix 2 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

plugins/kubernetes/package.json

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

⚠ Warning
Failed to update the yarn.lock, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Server-side Request Forgery (SSRF) SNYK-JS-REQUEST-3361831
  185

Prototype Pollution SNYK-JS-TOUGHCOOKIE-5672873
  112

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Server-side Request Forgery (SSRF)
🦉 Prototype Pollution

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: code/snyk check is complete. No issues have been found. (View Details)


Development

Tests fail.

Reran after textfeatures stopped acting up, we're now good to go.

Dang. I always delete the branch.


Question about usage of embed instead of query_embed in query method in QdrantFastembedMixin

https://github.com/qdrant/qdrant-client/blob/e2ad0661037fe40478ca2e3300f2b3c275183b5e/qdrant_client/qdrant_fastembed.py#L698
Hello, I have a question.
For the Fastembed mixin, why is embed used for sparse embeddings instead of query_embed? From what I read, for bm25 specifically at least, is that it doesn't use term frequency and length, but instead sets a uniform 1.0 for each token. What is the effect of that difference here? Is 'query_embed' for sparse embeddings not meant to be used with Qdrant?

Hey @martinkozle
We have fixed it in the latest release (1.12.0), query method should now use the correct embed function


Data missing

I found an "frames2" folder in the data of iPhone-lego, but my own data captured with Spectacular Rec App does not have it. How is this "frames2" folder generated?

Does your iPhone have a ToF sensor? ("LiDAR") The folder only exists in that case

Does your iPhone have a ToF sensor? ("LiDAR") The folder only exists in that case

Thank you very much for your explanation. I think I need a phone equipped with LiDAR.


⚠ Bofrost.at has degraded performance

In 9a38087, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1351 ms

Resolved: Bofrost.at performance has improved in 42ef2d9.


RuntimeError: The expanded size of the tensor (8) must match the existing size (10) at non-singleton dimension 1.  Target sizes: [128, 8].  Tensor sizes: [128, 10]

when I train PDV on waymo dataset, I meet this error.
The yaml file is "cfgs/waymo_models/pdv.yaml". I have cost much time on this.
error details:
Traceback (most recent call last):
File "train.py", line 232, in
main()
File "train.py", line 176, in main
train_model(
File "/home/rcvlab/lly/PDV/tools/train_utils/train_utils.py", line 86, in train_model
accumulated_iter = train_one_epoch(
File "/home/rcvlab/lly/PDV/tools/train_utils/train_utils.py", line 38, in train_one_epoch
loss, tb_dict, disp_dict = model_func(model, batch)
File "/home/rcvlab/lly/PDV/pcdet/models/init.py", line 42, in model_func
ret_dict, tb_dict, disp_dict = model(batch_dict)
File "/home/rcvlab/anaconda3/envs/lly_torch17/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/home/rcvlab/lly/PDV/pcdet/models/detectors/pdv.py", line 11, in forward
batch_dict = cur_module(batch_dict)
File "/home/rcvlab/anaconda3/envs/lly_torch17/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/home/rcvlab/lly/PDV/pcdet/models/roi_heads/pdv_head.py", line 239, in forward
targets_dict = self.assign_targets(batch_dict)
File "/home/rcvlab/lly/PDV/pcdet/models/roi_heads/roi_head_template.py", line 110, in assign_targets
targets_dict = self.proposal_target_layer.forward(batch_dict)
File "/home/rcvlab/lly/PDV/pcdet/models/roi_heads/target_assigner/proposal_target_layer.py", line 32, in forward
batch_rois, batch_gt_of_rois, batch_roi_ious, batch_roi_scores, batch_roi_labels = self.sample_rois_for_rcnn(
File "/home/rcvlab/lly/PDV/pcdet/models/roi_heads/target_assigner/proposal_target_layer.py", line 113, in sample_rois_for_rcnn
batch_gt_of_rois[index] = cur_gt[gt_assignment[sampled_inds]]
RuntimeError: The expanded size of the tensor (8) must match the existing size (10) at non-singleton dimension 1.  Target sizes: [128, 8].  Tensor sizes: [128, 10]

Any update? Same question

@KyleYueye solved here


Src Files Added

additions

Client: contains persistenceJs client
Helpers: helper functions related to multisend, send, proposal and vote. more to be added in the future(v2), or comment if some more helpers are needed in v1. The name can be changed to utils.
Contracts: Generated scripts for contracts on testnet.
Config: config json for local or default chain

use pretter to add newline for all files at once @vishrutsingh


上海市地图数据 崇明区数据缺失

https://github.com/ecomfe/echarts/issues/6373

https://github.com/chfw/jupyter-echarts/commit/800f068069fcd2d18db5676dbbd2b8bec5d73ce6


pausation::emplace BOOST_ASSERT(! base_);

Hello, Can I call ws.async_read and ws.async_write methods from different threads? Because I have error:
pausation.hpp:149
void beast::websocket::detail::pausation::emplace(F&&) [with F = beast::websocket::stream<boost::asio::basic_stream_socketboost::asio::ip::tcp&>::write_frame_op<beast::buffer_prefix_view<beast::consuming_buffersboost::asio::const_buffers_1 >, beast::websocket::stream<boost::asio::basic_stream_socketboost::asio::ip::tcp&>::write_op<boost::asio::const_buffers_1, easynet::connection::send(const string&)::<lambda(const boost::system::error_code&)> > >]: Assertion `! base_' failed.

You cannot call any members from different threads concurrently. You have to make all your calls from the same implicit or explicit strand. Beast websocket streams work just like Asio streams - they are not thread-safe (even though you are using threads in your io_service).

Thank you, but it  does not help me, I use one thread for my 'async_read' and 'async_write' operations now but I have error still.
pausation.hpp:149
void beast::websocket::detail::pausation::emplace(F&&) [with F = beast::websocket::streamboost::asio::basic_stream_socketboost::asio::ip::tcp&::write_frame_opbeast::buffer_prefix_view<beast::consuming_buffers<boost::asio::const_buffers_1 >, beast::websocket::streamboost::asio::basic_stream_socketboost::asio::ip::tcp&::write_op<boost::asio::const_buffers_1, easynet::connection::send(const string&)::<lambda(const boost::system::error_code&)> > >]: Assertion `! base_' failed.
What I can do? Thank you.

The other reason that this assert comes up is if you try to call async_read or async_write twice without an intermediate completion. Is your code on GitHub where I can see it? Or can you post a gist?

Thank you very much, it help me. I call "async_write" several without an intermediate completion.

Sounds like this is resolved, so I'll go ahead and close the issue. If you have another problem feel free to open a new issue any time, thanks!


Make the docker image smaller

Currently everything from api and client is in the docker image (9 GB). This have to be reduced

du -a | sort -n -r | head -n 5


feat: add save file task example code

I have added some type parsing as Python types and written some tasks.
I will add test code and refactor the codebase.
If you have any suggestions to improve this project, please advise me. Your input would be greatly appreciated.

And, add Test code.


IONIC APP

hey, How can I use the SDK in hybrid app like IONIC applications?

@Fleker some help please

I don't have any experience or familiarity with IONIC, but I don't imagine it would necessarily be any different than calling the Node.js functions as they exist now.


IllegalAccessError: GitlabURLHandler tried to access protected method

When sbt-gitlab tries to fetch a package from GitLab, I get the following error:
java.lang.IllegalAccessError: class org.apache.ivy.util.url.GitlabURLHandler tried to access protected
  method 'void org.apache.ivy.util.url.URLHandler$URLInfo.<init>(boolean, long, long, java.lang.String)'
  (org.apache.ivy.util.url.GitlabURLHandler is in unnamed module of loader
  sbt.internal.PluginManagement$PluginClassLoader @75c9e76b;
  org.apache.ivy.util.url.URLHandler$URLInfo is in unnamed module of
  loader sbt.internal.MetaBuildLoader @57a3af25)
 	at org.apache.ivy.util.url.GitlabURLHandler.getURLInfo(GitlabURLHandler.java:105)
 	at org.apache.ivy.util.url.GitlabURLHandler.getURLInfo(GitlabURLHandler.java:82)
 	at org.apache.ivy.util.url.URLHandlerDispatcher.getURLInfo(URLHandlerDispatcher.java:66)
 	...

It's not a credentials issue, but rather it appears to be a classloader issue similar to https://github.com/sbt/sbt/issues/3498.  sbt gives each plugin a unique classloader that must have the sbt metabuild classloader as its parent, and URLHandler$URLInfo is not loaded in the plugin classloader.

@dpdearing Thanks for the issue report! Which sbt version are you using?

Which sbt version are you using?

@azolotko This is with sbt version 1.7.2
I am trying to migrate from gilcloud/sbt-gitlab for support on 1.7.x

had the same issue, it's the ThisBuild / useCoursier := false flag that seems to trigger this issue and this version seems to be compatible with coursier

it's the ThisBuild / useCoursier := false flag that seems to trigger this issue

Thank you @vaslabs!  I did have this in my build definition and removing it does appear to fix the problem.  I'll close this issue as resolved.

Sorry for the back-and-forth, but it's not clear that this actually resolved the issue after all so I'm reopening after prematurely closing.

Thanks @dpdearing and @vaslabs ! I've just pushed v0.1.2 with a fix addressing this issue. It should soon appear at Maven Central. Please give it a try and let me know if there're any issues still.