Fixes streaming for anthropic tool call.

Fixes #80
Replaces #81

Any timelines on when we could merge this? The pr looks good to me, and the problem is blocking usage of Bedrock sonnet on our end.

Just tried it in langgraph, but I had to adjust a little bit how to use it.
Here are the docs. I'm not sure if part of this PR or not. But I thought it would work seamlessly, and it didn't.
https://langchain-ai.github.io/langgraph/how-tos/streaming-tokens/#streaming-llm-tokens
langgraph==0.1.14
sonnet3.5 through Bedrock

Just ran into this same problem with Claude with ChatBedrock, good to see it close to be addressed!  Thanks!

@3coins what work remains on this? Can we push it through?

I am still getting the same error using ChatBedrock with streaming=True.
File "/home/ec2-user/miniconda3/envs/intrepid/lib/python3.11/site-packages/langchain_aws/llms/bedrock.py", line 594, in _get_provider
if self.model_id.startswith("arn"):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RecursionError: maximum recursion depth exceeded while calling a Python object

I am still getting the same error using ChatBedrock with streaming=True.
File "/home/ec2-user/miniconda3/envs/intrepid/lib/python3.11/site-packages/langchain_aws/llms/bedrock.py", line 594, in _get_provider if self.model_id.startswith("arn"): ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ RecursionError: maximum recursion depth exceeded while calling a Python object
It works without issue with streaming not set.

Could you try again? This wasn't released at the time of your comment. Just released in langchain-aws==0.1.14.

Gave it another shot after doing pip3 langchain_aws --upgrade.  Still getting this?
File "/home/ec2-user/miniconda3/envs/intrepid/lib/python3.11/site-packages/langchain_aws/llms/bedrock.py", line 594, in _get_provider
client_params["endpoint_url"] = values["endpoint_url"]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RecursionError: maximum recursion depth exceeded while calling a Python object
Do you want the full trace?  This is an anthropic llm calling tools, which then takes multiple data sets as context from the tool.  so not sure if that might be part of the issue or not?


The collision shape of the ViewNode isn't updated

If the ViewNode is moved to another location, it still responds to touches where it has been before.

Is the collision system still needed?
It's quite consuming and will be replaced by the Filament Pick in coming next release (#38). On which @StevenMohr is doing a great work.
Do you see any other reason to keep it in place?
I'm also actually doing a quite big work on moving renderable to model (Filament asset) and this part will be moved since it was inside the Renderer (RIP)

Is the collision system still needed?
It's quite consuming and will be replaced by the Filament Pick in coming next release

Will it be possible to interact with Views shown by the ViewNode?

I think, if we can project the screen coordinates at the plane of the picked renderable, we should be able to replace the old collision system.
https://github.com/google/filament/blob/df6a100fcba66d9c99328a49d41fe3adecc0165d/android/filament-android/src/main/java/com/google/android/filament/View.java#L1722-L1729

Great. I'm currently working on the Kotlin-Math project and unproject equation from the Camera so it won't be difficult.
I just hope not making any errors in the equation between the view matrix, the projection and the model one.

https://github.com/google/filament/blob/df6a100fcba66d9c99328a49d41fe3adecc0165d/android/filament-android/src/main/java/com/google/android/filament/Camera.java#L470-L485
https://github.com/google/filament/blob/df6a100fcba66d9c99328a49d41fe3adecc0165d/android/filament-android/src/main/java/com/google/android/filament/Camera.java#L517-L531
https://github.com/google/filament/blob/df6a100fcba66d9c99328a49d41fe3adecc0165d/android/filament-android/src/main/java/com/google/android/filament/Camera.java#L549-L562

I have to admit that I'm a little confused between each one usage.

If I'm not wrong, it should be:
/**
 * ### Convert a point from world space into screen space.
 *
 * The X value is negative when the point is left of the viewport, between 0 and the width of
 * the [SceneView] when the point is within the viewport, and greater than the width when
 * the point is to the right of the viewport.
 *
 * The Y value is negative when the point is below the viewport, between 0 and the height of
 * the [SceneView] when the point is within the viewport, and greater than the height when
 * the point is above the viewport.
 *
 * The Z value is always 0 since the return value is a 2D coordinate.
 *
 * @param position the point in world space to convert
 * @return a new vector that represents the point in screen-space.
 */
fun worldToScreenPoint(position: Position): Float2 {
    val m = projectionMatrix * viewMatrix
    return (m * Float4(v = position, w = (1.0f / (m * Float4(v = position))).w)).xy
}

I have to admit that I'm a little confused between each one usage.

The projection matrix transforms the points from the world coordinate system to the OpenGL clip space.
The view matrix transforms the points in the world coordinate system so that they appear to be viewed from the camera point. That means that instead of moving the camera the world is moved in the opposite direction.
The model matrix represents the camera transform similarly to other nodes.

If I'm not wrong, it should be:

I think it should be:
val clipSpacePosition = m * Float4(position, 1f)
clipSpacePosition.xy = clipSpacePosition.xy / clipSpacePosition.w
val screenPosition = Float2(
    (clipSpacePosition.x + 1) / 2 * sceneView.width,
    (1 - clipSpacePosition.y) / 2 * sceneView.height
)

I've solved the original issue by calling the Node.onTransformChanged() after the ViewNode has been moved.


Terraform modules as HTTP

Add modules.tf lambda which creates small versioned archives from git repos to have smaller modules to download. See, slides 33-36 - https://speakerdeck.com/so0k/terraform-at-honestbee?slide=33
Thanks https://github.com/so0k for tips.

irrelevant :-/


whitelist flashloans

flashloans are currently commented out on LoanTokenLogicStandard.sol, but we had them audited. we'd like to enable flashloans for whitelisted addresses.
this also means, we need to remove some of the reentrancy guards (locations are already marked with a comment)

do whitelisted addresses objects/logic exist in the protocol or needs to be created?
if it doesn't, i need more details on it.
the locations commented to uncomment for flashloans?

do whitelisted addresses objects/logic exist in the protocol or needs to be created?
if it doesn't, i need more details on it.
the locations commented to uncomment for flashloans?


Add cargo:rustc-link-arg to pass custom linker arguments

It is useful to produce correct cdylibs on platforms such as Linux and
MacOS.
Groundwork to address https://github.com/rust-lang/cargo/issues/5045 in the future.

Thanks for the PR! I'm personally a bit wary to add this though without more protections and/or safeguards. In addition to the soft feature freeze this is a feature we have intentionally not added historically as it can be tricky to do so.
I would personally prefer at least that this takes the route of a more formal feature proposal such as a discussion on internals or a mini-rfc.

If you have time to guide me on that I'm more than willing to :)
Keep in mind that we have an env key that already let you pass those values, but it does to all the rustc invocations and my actual need is to pass the right linker flags so the soname is set for cdylibs.
We can restrict it to just cdylibs or just make cargo come up with the os-specific incantations at least for the tier1 platforms.

For posterity, there's more info on this in https://internals.rust-lang.org/t/pre-rfc-complete-cdylib-support-in-cargo/8843/12

:umbrella: The latest upstream changes (presumably #6328) made this pull request unmergeable. Please resolve the merge conflicts.

Here an attempt on restricting, not sure where is the best place to error out if somebody mixes cdylib and dylib.

Thanks, although I think there's still a number of open questions here:
* Currently this is ignoring link-args on non-cdylib crates (silently), is that the behavior that we want?

The use-case I'm focusing is to pair it with a build.rs providing the link lines. So non-cdylib or non-cdylib+staticlib could be enjoy an hard error.

* What happens if there's mutliple crate types, where do the link args go?

from what I'm seeing for the cdylib+staticlib case it works as expected even if rustc is called only once.

* Do we want to read this from `.cargo/config`? Is there a use case for that?

For my usecase there isn't.
On the other hand this can be reused by cargo-deb/rpm/ebuild later for their distro-specific purposes (e.g. custom rpaths), but I'd rather not conflate it.

:umbrella: The latest upstream changes (presumably #6352) made this pull request unmergeable. Please resolve the merge conflicts.

Sorry for having this fall by the wayside, this fell off my radar by accident!
@lu-zero would you still be willing to help drive this PR forward?
Following up with some of my questions above, for cases where the link-args aren't used I don't really know what the best answer is. I think it's fine for now though to swallow them and document that we swallow them, and if it becomes an issue we can issue warnings or something like that later.
For the many crate types, we probably want to either:

Pass the link args to all linked artifacts (except maybe tests)
Rename these to rustc-cdylib-link-arg or somethig like that (to make it clear it only applies to cdylib and leave space to customize them in the future)

For .cargo/config what I meant was that you can specify build script outputs in .cargo/config in native library overrides. I think I missed though where this was already handled.
So all in all, the remaining items needed for this I think are:

Documentation for this new feature
Decision on where to route link args for multiple artifact (aka ignore for all but cdylib or rename to say it's only going to cdylib)

I think I'd personally lean towards renaming this rustc-cdylib-link-arg which leaves us room to add more flavorful configuration for other output artifacts in the future. For example we could have a mode where you specify binary link args but only for one binary and not others.

@lu-zero would you still be willing to help drive this PR forward?

Sure. I'll update the PR during the weekend :)

Ok sounds great, and sorry again for the delay!

I renamed the key, regarding the documentation to update, where I should I find it?

@rfcbot fcp merge
Ok! This looks great to me, so I'd like to propose that this be merged, but would also like to get a signoff from the rest of the team.

I personally would like this (and would use this) for binaries, not just cdylib, but cdylib certainly helps.

@bors: r+

:pushpin: Commit f608ed6197feb8b21fba0db8ee24227f40548ed3 has been approved by alexcrichton

:hourglass: Testing commit f608ed6197feb8b21fba0db8ee24227f40548ed3 with merge 5f3f2811280a5e59e5db31821635c15b920acf65...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: alexcrichton
Pushing 5f3f2811280a5e59e5db31821635c15b920acf65 to master...


Centralize building of the aws.Config object

When running the ENI IPAM an operator does not need to pass any region env var flag for the AWS SDK to configure it's client because we read it from the ec2 metadata endpoint. If the same operator wants to run with --update-ec2-apdater-limit-via-api=true then the cilium-operator will fail to construct the ec2 client because a valid region was not passed.
Centralize building of the aws.Config object

test-me-please

test-me-please


Closed connection 1006

tried access my server and friend server with both the official HTTP link and my own setup fails with connection error 1006
Trying to administrate a gmod server (not sure if this works for gmod as it doesn't say its wont)

We are having the same bug here

still active bug

Same issue, tested in another rcon client and I was able to login just fine. I try the same IP, port and password in webrcon and it's just Connection was closed - Error 1006


Rke cluster.yaml examples have a broken link in comments

yaml references at https://rancher.com/docs/rke/latest/en/example-yamls/ include https://github.com/rancher/types/blob/master/apis/management.cattle.io/v3/k8s_defaults.go to lookup valid kubernetes_version and system_version information, but that file appears to be gone.

Realizing that the file probably only exists in the release branches? We may want to clarify that

These are the two locations which have the file:
content/rke/latest/en/example-yamls/_index.md
content/rke/latest/en/config-options/system-images/_index.md```

For RKE v0.2.x and below, the files exist at:

https://github.com/rancher/types/blob/release/v2.2/apis/management.cattle.io/v3/k8s_defaults.go

For RKE v0.3.0 and above, the files exist at:

https://github.com/rancher/kontainer-driver-metadata/blob/master/rke/k8s_defaults.go

Closing this issue because it was addressed by http://github.com/rancher/docs/pull/1739


Speed up 4x using numba

This PR addresses the following issues.

Accelerated the seam calculation by reducing memory allocation and using numba jit (#1). It gives a 4x end-to-end speedup with forward energy and 2x with backward energy. This is inspired by @keurcien 's POC for forward seam in #2.
Supported multi-step expansion. Previously the image cannot be expanded to over 2x its width. Now resizing the image to any target size is supported.
Fixed some bugs and CI failures. Upgraded the supported python versions. Formatted the codes with black.

Codecov Report

Merging #4 (c06cd01) into master (2711ecd) will decrease coverage by 22.71%.
The diff coverage is 78.78%.

@@             Coverage Diff              @@
##            master       #4       +/-   ##
============================================
- Coverage   100.00%   77.28%   -22.72%
============================================
  Files            1        1
  Lines          234      273       +39
============================================
- Hits           234      211       -23
- Misses           0       62       +62

Flag
Coverage Δ

unittests
77.28% <78.78%> (-22.72%)
:arrow_down:

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

Impacted Files
Coverage Δ

seam_carving/carve.py
77.28% <78.78%> (-22.72%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[YUNIKORN-2755] Install private copy of pnpm and @angular/cli

What is this PR for?
Ensure that we use private copies of both pnpm and @angular/cli so that package and lock files remain consistent. Also upgraded to the latest current pnpm.
What type of PR is it?

[x] - Bug Fix
[ ] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2755
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 38.77%. Comparing base (246c802) to head (3c1f89c).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #196   +/-   ##
=======================================
  Coverage   38.77%   38.77%
=======================================
  Files           2        2
  Lines          49       49
=======================================
  Hits           19       19
  Misses         27       27
  Partials        3        3

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


Add support for Careem

Are the only apps that will open the ones listed on the readme page?
For example, I don't see "careem" listed - could I find the right id for careem and add it?

@ismaelrumzan Yes please! I've created a quick document to outline the steps involved in adding support for a new app to the library:
https://github.com/includable/react-native-map-link/blob/master/docs/add-app.md

Thanks for that
I've been trying to figure out what the url scheme would be.
I was able to find the manifest.plist file from the backup and looked the the CFBundleIdentifier
However, these are different from what you've used as URL schemes
For example, google maps is com.google.Maps and you use comgooglemaps://
Is it basically the same but all lowercase and without dots?
But Uber is com.ubercab.UberClient and you have uber://

CFBundleIdentifier is not the URL scheme. There should be one or more keys called CFBundleURLSchemes which will contain the URL scheme:
Example:
<key>CFBundleURLTypes</key>
<array>
	<dict>
		<key>CFBundleURLName</key>
		<string>nl.hoyapp.hoyapp</string>
		<key>CFBundleURLSchemes</key>
		<array>
			<string>hoyapp</string>
		</array>
	</dict>
</array>

In this example the URL scheme is hoyapp://.

Thanks for that, I'm need to download the ipa to find that data
I got google maps to load now - however, the co-ordinates are not being passed when I click on it.
It works fine for uber and apple maps. In google maps, it seems to try to search on the name passed.
Any ideas?

Just the IPA won't be enough, because you need to know the rest of the URL scheme (how the parameters work) as well.
They do seem to have online documentation (click Deep Linking top right), but the Developer portal requires signing in, and the Sign Up form doesn't seem to work, like these people also found out.

Great, I found the url scheme and added all the info in a forked version of your source code.
How can I test it with the new source and make sure things work on both android and ios, without requesting a pull request to push back to your code?

Thanks,
Got it to work - it opens the careem app - now I need to troubleshoot how to pass the co-ordinates - any tips on finding this or are there a set of "limited" conventions?

Not really, it's might be similar to the way the Uber deep linking works, but you would really need the documentation I linked to in a previous comment to know for sure.
I've already been looking around if I could find any other resources for Careem, but I couldn't find any...

Hi, any updates on this?

Still waiting for someone to find out info about their URL scheme. I've tried signing up for their developer portal multiple times, but it don't seem to work.
As soon as we have the URL scheme data it should be really simple to build the actual integration.

thank you I'll also try to register and help

Hi @baselbj have you been able to access Careem's developer docs? Otherwise I might close this issue for now.

to tell you the truth I totally forgot about this ... I will start working on that now... sorry for that

after some tries and playing with my Careem accounts I registered a developer account the response is:

You are almost there!
We have received your request to become a developer. A Careem representative will get in touch with you shortly.

waiting for there response

I think I also got to that point, and then never got a response :)
We'll see, thanks for your quick reply though!

I will keep trying to contact them and trying to find some one that can help the information needed is the CFBundleURLSchemes , right?

Yep, just some documentation on how their URL scheme works, like this from Uber: https://developer.uber.com/docs/riders/ride-requests/tutorials/deep-links/introduction

Closing this for now until we have documentation for Careem.

sorry for that I will find some way to contact them ... I will try contacting their branch in Jordan ... Hopping to remember doing this


[SPARK-16950] [PySpark] fromOffsets parameter support in KafkaUtils.createDirectStream for python3

What changes were proposed in this pull request?
Ability to use KafkaUtils.createDirectStream with starting offsets in python 3 by using java.lang.Number instead of Long during param mapping in scala helper. This allows py4j to pass Integer or Long to the map and resolves ClassCastException problems.
How was this patch tested?
unit tests
@jerryshao  - could you please look at this PR?

Can one of the admins verify this patch?

ok to test

Test build #63363 has started for PR 14540 at commit d2a4027.

Test build #63363 has finished for PR 14540 at commit d2a4027.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63363/
Test PASSed.

Also cc @tdas who might be interested

I wonder why we don't see this issue crop up in the Python MLlib API because we use java longs for things like seeds.

Looks good to me, we should also change the unit tests accordingly. Currently several related tests are excluded from python3 test.

@holdenk I've checked setSeed methods in MLlib and it seems py4j handles them well. If function gets simple arguments (strings, numerric, bool), py4j applies conversion between types (see https://github.com/bartdag/py4j/blob/master/py4j-java/src/main/java/py4j/reflection/MethodInvoker.java#L99).
For setSeed(Long), if argument is mapped to Integer, it goes through toString and Long.parseLong (see https://github.com/bartdag/py4j/blob/master/py4j-java/src/main/java/py4j/reflection/TypeConverter.java#L88)
Apparently, this conversion does not work for complex types like fromOffsets map.

Ah interesting, we might want to report the bug upstream with Py4J - but this change looks good to me :) Thanks for getting this working in Python 3 :) cc @davies who can maybe take a look as well?

LGTM, merging this into master, thanks!


Website unresponsive after Upgrade

Following is the email with error Code ( at bottom ) I have recieved from Wordpress after recent Upgrade to the latest virsion of WP-Stateless
Howdy!
WordPress has a built-in feature that detects when a plugin or theme causes a fatal error on your site, and notifies you with this automated email.
In this case, WordPress caught an error with one of your plugins, WP-Stateless.
First, visit your website (https://journals.stmjournals.com/) and check for any visible issues. Next, visit the page where the error was caught (https://journals.stmjournals.com/wp-admin/) and check for any visible issues.
Please contact your host for assistance with investigating this issue further.
If your site appears broken and you can't access your dashboard normally, WordPress now has a special "recovery mode". This lets you safely login to your dashboard and investigate further.
https://journals.stmjournals.com/wp-login.php?action=enter_recovery_mode&rm_token=PZ6G2sh4PEG5Z4py0IGCfs&rm_key=QNCsjbTT6Kwly5tU2pgTpX
To keep your site safe, this link will expire in 1 day. Don't worry about that, though: a new link will be emailed to you if the error occurs again after it expires.
When seeking help with this issue, you may be asked for some of the following information:
WordPress version 6.2.2
Active theme: Thrive Theme Builder (version 3.19.1)
Current plugin: WP-Stateless (version 3.2.3)
PHP version 7.4.33
Error Details
An error of type E_COMPILE_ERROR was caused in line 313 of the file /home/p2387928/stmjournals.com_wpress/wp-content/plugins/wp-stateless/lib/Google/vendor/ramsey/uuid/src/Uuid.php. Error message: Declaration of Ramsey\Uuid\Uuid::unserialize(string $data): void must be compatible with Serializable::unserialize($serialized)

Pls let me know how to rectify the issue asap.
Regards
Puneet

Unfortunately this error took several of our client websites offline this morning, with only this PHP error returned:
Parse error: syntax error, unexpected '|', expecting variable (T_VARIABLE) in /home/customer/www/[DOMAIN]/public_html/wp-content/plugins/wp-stateless/lib/Google/vendor/psr/cache/src/CacheItemInterface.php on line 104

With no access to the frontend or WordPress dashboard, we took the following steps to resolve:

Log in to hosting panel / FTP in to server
Rename the /public_html/wp-content/plugins/wp-stateless/ directory (to deactivate the plugin)
Download previous version 3.2.2 [link]
Login via WordPress dashboard, manually upload and activate v3.2.2 plugin
Delete v3.2.3 plugin

@Dishoom007 @darrenbarklie
We have just released new version 3.2.4, these issues should be resolved there.
Please let us know if you still have any issues


How to check whether the pdf is compressed or not?

I need to check whether the pdf is compressed or not.
I tried with a compressed pdf file but got isCompressed: false as in the output.
CMapReaderFactory: DOMCMapReaderFactory { baseUrl: null, isCompressed: false },
Is there any way to find this?
Thanks in advance

I need to check whether the pdf is compressed or not.

What do you actually mean by "compressed" here?
In PDF documents streams may use different filters (thus implying some form of data compression), but the filters used usually varies considerably for different streams even in a single document.

CMapReaderFactory: DOMCMapReaderFactory { baseUrl: null, isCompressed: false },

Most likely CMaps have nothing to do with this issue...

Finally, as-is this issue isn't really actionable; when opening an issue, please provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md

OS: Ubuntu 18.04.4 LTS
PDF.js version: "pdfjs-dist": "^2.0.943"
I'm using pdfdist npm to identify the properties of a pdf in backend nodejs.
I got a compressed pdf file of size 10mb but its actual size is more than 120MB. I need to find whether the pdf is compressed or not before I need to convert the pdf to jpg.


custom CUDA op example returns random values

What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
Nothing
Environment info
Operating System:
$ uname -a
Linux n-62-18-47 2.6.32-642.6.1.el6.x86_64 #1 SMP Wed Oct 5 08:48:31 CDT 2016 x86_64 x86_64 x86_64 GNU/Linux

Installed version of CUDA and cuDNN: CUDA: 8, cuDNN 5.1
(please attach the output of ls -l /path/to/cuda/lib/libcud*):
$ ls -l /appl/cuda/8.0/lib64/libcud*
-rw-r--r-- 1 sebo root 560184 Sep  1 14:31 /appl/cuda/8.0/lib64/libcudadevrt.a
lrwxrwxrwx 1 sebo root     16 Sep  1 14:31 /appl/cuda/8.0/lib64/libcudart.so -> libcudart.so.8.0
lrwxrwxrwx 1 sebo root     19 Sep  1 14:31 /appl/cuda/8.0/lib64/libcudart.so.8.0 -> libcudart.so.8.0.27
-rwxr-xr-x 1 sebo root 394472 Sep  1 14:31 /appl/cuda/8.0/lib64/libcudart.so.8.0.27
-rw-r--r-- 1 sebo root 737516 Sep  1 14:31 /appl/cuda/8.0/lib64/libcudart_static.a

If installed from source, provide

The commit hash (git rev-parse HEAD) 5a5a25ea3ebef623e07fb9a46419a9df377a37a5
The output of bazel version

$ bazel version
Build label: 0.3.2- (@non-git)
Build target: bazel-out/local-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Fri Oct 21 15:09:04 2016<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
Using the CUDA example from: https://github.com/tensorflow/tensorflow/tree/r0.11/tensorflow/g3doc/how_tos/adding_an_op

compile example

export TF_INC=/zhome/ff/2/77654/stdpy3/lib/python3.5/site-packages/tensorflow/include

nvcc -std=c++11 -c -o cuda_op_kernel.cu.o cuda_op_kernel.cu.cc \
-I $TF_INC -D GOOGLE_CUDA=1 -x cu -Xcompiler -fPIC

g++ -std=c++11 -shared -o cuda_op_kernel.so cuda_op_kernel.cc \
cuda_op_kernel.cu.o -I $TF_INC -fPIC -L /appl/cuda/8.0/lib64 -L /appl/cudnn/v5.1-prod/lib64 -lcudart

edit tensorflow.g3doc.how_tos.adding_an_op import cuda_op to import cuda_op in cuda_op_test.py.

What other attempted solutions have you tried?

I tried a non CUDA example, worked fine.
I tried a diffrent cuda kernel (square operator) also failed.
I added printf to the kernel launcher and made sure it was executed.

Logs or other output that would be helpful
(If logs are large, please upload as attachment or provide link).
CUDA_VISIBLE_DEVICES=3 python3 cuda_op_test.py
I tensorflow/stream_executor/dso_loader.cc:125] successfully opened CUDA library libcurand.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:125] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:125] successfully opened CUDA library libcufft.so.8.0 locally
I tensorflow/stream_executor/dso_loader.cc:125] successfully opened CUDA library libcudnn.so.5 locally
I tensorflow/stream_executor/dso_loader.cc:125] successfully opened CUDA library libcublas.so.8.0 locally
I tensorflow/core/common_runtime/gpu/gpu_device.cc:944] Found device 0 with properties:
name: Tesla K40c
major: 3 minor: 5 memoryClockRate (GHz) 0.745
pciBusID 0000:02:00.0
Total memory: 11.25GiB
Free memory: 11.15GiB
I tensorflow/core/common_runtime/gpu/gpu_device.cc:965] DMA: 0
I tensorflow/core/common_runtime/gpu/gpu_device.cc:975] 0:   Y
I tensorflow/core/common_runtime/gpu/gpu_device.cc:1034] Creating TensorFlow device (/gpu:0) -> (device: 0, name: Tesla K40c, pci bus id: 0000:02:00.0)
Failed to get the number of CUDA devices: CUDA driver version is insufficient for CUDA runtime version
not equal where =  (array([0, 1, 2, 3, 4]),)
not equal lhs =  [ 280541332  143397048<PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER>]
not equal rhs =  [6 5 4 3 2]
F.
======================================================================
FAIL: test (__main__.AddOneTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "cuda_op_test.py", line 31, in test
    self.assertAllEqual(result.eval(), [6, 5, 4, 3, 2])
  File "/zhome/ff/2/77654/stdpy3/lib/python3.5/site-packages/tensorflow/python/framework/test_util.py", line 499, in assertAllEqual
    np.testing.assert_array_equal(a, b)
  File "/zhome/ff/2/77654/stdpy3/lib/python3.5/site-packages/numpy/testing/utils.py", line 813, in assert_array_equal
    verbose=verbose, header='Arrays are not equal')
  File "/zhome/ff/2/77654/stdpy3/lib/python3.5/site-packages/numpy/testing/utils.py", line 739, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not equal

(mismatch 100.0%)
 x: array([ 280541332,  143397048,<PHONE_NUMBER>,<PHONE_NUMBER>,<PHONE_NUMBER>], dtype=int32)
 y: array([6, 5, 4, 3, 2])

----------------------------------------------------------------------
Ran 2 tests in 0.213s

FAILED (failures=1)

It looks like the output just contains random memory. Perhaps the GPU memory isn't copied back to the host memory.

@AndreasMadsen : That does seem fishy, but I happened to notice this line in your pasted output:
Failed to get the number of CUDA devices: CUDA driver version is insufficient for CUDA runtime version

Could that indicate some problem with the driver installation?

@asimshankar I don't know, it is not my setup. Is there some Nvidia tool I can use to diagnose it?

Here is the nvidia-smi output:
Sun Oct 23 09:17:43 2016
+------------------------------------------------------+
| NVIDIA-SMI 352.39     Driver Version: 352.39         |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Tesla K40c          On   | 0000:02:00.0     Off |                    0 |
| 23%   24C    P8    19W / 235W |     22MiB / 11519MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   1  Tesla K20c          On   | 0000:03:00.0     Off |                    0 |
| 30%   20C    P8    16W / 225W |     13MiB /  4799MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   2  Tesla K20c          On   | 0000:83:00.0     Off |                    0 |
| 30%   22C    P8    16W / 225W |     13MiB /  4799MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+
|   3  GeForce GTX TIT...  On   | 0000:84:00.0     Off |                  N/A |
| 22%   31C    P8    29W / 250W |    161MiB / 12287MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+

@AndreasMadsen , do let us know if the newer driver works out.
Threads like this one do suggest that an updated driver is needed.

@asimshankar The driver is now updated and that did help with the issue. There are still some other issues but they are unrelated to this one.
Thanks for your help.


BTRFS build fails with RaspiKey

rPi 4 with 32GB USB and RaspiKey eMMC module for OS fails the status check for BTRFS.
https://www.uugear.com/product/raspikey-plug-and-play-emmc-module-for-raspberry-pi/
lsblk lists 2 additional boot devices causing raidCandidates=3
mmcblk0boot0 179:32   0     4M  1 disk
mmcblk0boot1 179:64   0     4M  1 disk
These pass the criteria set for raid candidates in blitz.datadrive.sh line 397 as they are listed as "disk" and do not contain "/" (couldn't this just search for sd* that is not main hdd?).
Failing section:
    for disk in $(lsblk -o NAME,TYPE | grep "disk" | awk '$1=$1' | cut -d " " -f 1)
    do
      devMounted=$(lsblk -o MOUNTPOINT,NAME | grep "$disk" | grep -c "^/")
      # is raid candidate when not mounted and not the data drive cadidate (hdd/ssd)
      if [ ${devMounted} -eq 0 ] && [ "${disk}" != "${hdd}" ] && [ "${hdd}" != "" ]; then
An additional check is needed to ignore them:
    for disk in $(lsblk -o NAME,TYPE | grep "disk" | awk '$1=$1' | cut -d " " -f 1)
    do
      devMounted=$(lsblk -o MOUNTPOINT,NAME | grep "$disk" | grep -c "^/")
      devBoot=$(lsblk -o NAME | grep "$disk" | grep -c boot)
      # is raid candidate when not mounted and not the data drive cadidate (hdd/ssd)
      if [ ${devMounted} -eq 0 ] && [ "${disk}" != "${hdd}" ] && [ "${hdd}" != "" ] && [ ${devBoot} -eq 0 ]; then

Just out of curiosity, why did you choose BTFRS?
For the other curious folks, this summarizes BTFRS: https://wiki.archlinux.org/title/Btrfs

Just out of curiosity, why did you choose BTRFS?
For the other curious folks, this summarizes BTRFS: https://wiki.archlinux.org/title/Btrfs

It was already in the build, just didn't work with the emmc modules.

Just for quick background why BTRFS and not ZFS (for now): If you search the issues (also closed) for ZFS you should find some discussions on that already. The one that at let me keep my hands off ZFS for RaspiBlitz was the following:
Yes ZFS on RaspberryPi even with low memory configs are possible - you can find tutorials that show ZFS on RaspberryPi. But ZFS for ARM is not common - thats why for a TruNAS you normally need a x86. If we would switch to ZFS with RaspiBlitz we would most prob become one of the biggest projects on ZFS build for ARM processors. And thats a very shaky ground to stand on - if we run into a bug down that lane there is a big change to get stuck.
The chapter on ZFS is not closed ... if there is another big RaspberryPi project running ZFS with no problems for a good long time we might reconsider.

Also BTRFS might get removed from RaspiBlitz now - https://github.com/raspiblitz/raspiblitz/issues/2045#issuecomment-2471845122
The idea is when you want to use BTFS or ZFS you should consider to run RaspiBlitz in a VM like with Proxmox.


Concrete simd mask operations

Concretize additional SIMDMask operations:

.&=, .|=, .^=, .==, .!=, replace|ing

Also reflect documentation improvements back to generic implementations.

@swift-ci test


More search fields, case-insensitive keyword fields.

At the moment to find a Contact by country you have to type country with correct case, for example United Kingdom, otherwise it will not bring any results. This PR fixes that.
It also adds more fields for each entity to be searchable.

Codecov Report

Merging #389 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #389   +/-   ##
========================================
  Coverage    95.85%   95.85%
========================================
  Files           70       70
  Lines         2705     2705
  Branches       243      243
========================================
  Hits          2593     2593
  Misses          79       79
  Partials        33       33

Continue to review full report at Codecov.

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


DirectLine 'event trigger' makes the last message to be disappeared/deleted

I come across a strange issue. I trigger an event using DirectLine and based on the event name I am posting back the message to the user.
But after triggering the message my last message which is already sent got deleted/disappeared. I am not sure why this is happening. Please refer the screenshot.

Only the last message got deleted/disappeared.
Here is the code I am using.

Here is the Payload

@selvaduraiece, the Direct Line channel doesn't support updating or deleting activities. While there could have been a hiccup in the service, it is more probable that the issue you are experiencing is rooted in your Web Chat code and how it's interacting with your bot's code.
To start, can you supply your Web Chat code as well as your bot's dialog that handles sending the initial activity / receives the response from the user?

Hi Stevkan,
Attached webchat code. For dialog code since I am using composer sharing screenshot

and then resume dialog

WebChat-github.zip

One more thing I am using ES5 version webchat. Majorly our client want IE11 support, that's y

@selvaduraiece, thank you for your files/details. I'll take a look at them and see if I can repro your issue.

Hi Steven, Any update on this? it quiet urgent requirement. I am suspecting it might be because of the payload I am using. It would be great help if you could help me as early as possible.

Hi Steven, Any update on this?

@stevkan @peterinnesmsft - Team  any update on this? If needed, I can even raise a support ticket for the same. It would be great help if I could get some resolution. UAT is about to happen.

@selvaduraiece, thank you for your patience. Unfortunately, I've had encountered a separate issue in Composer that is hindering me from assisting you properly. I'm assigning this to a colleague of mine who  will continue to provide assistance.

@selvaduraiece, thank you for your patience. Unfortunately, I've had encountered a separate issue in Composer that is hindering me from assisting you properly. I'm assigning this to a colleague of mine who  will continue to provide assistance.

Hi @selvaduraiece I have attempted to setup a working repro, and I think i have the bot setup correctly and perhaps most of the webchat. the one thing I am unclear on is how the function sendProactiveMsg() is being fired? How is that being triggered?

Hi @selvaduraiece I have attempted to setup a working repro, and I think i have the bot setup correctly and perhaps most of the webchat. the one thing I am unclear on is how the function sendProactiveMsg() is being fired? How is that being triggered?

Hi,
Find  the attached Front end JavaScript which triggers the sendProactiveMsg(). If the user is Idle for 3 minutes then I am calling the function.(Payload also included in the script)
chat.txt

Hi,
Find  the attached Front end JavaScript which triggers the sendProactiveMsg(). If the user is Idle for 3 minutes then I am calling the function.(Payload also included in the script)
chat.txt

Hi Dana,
Any update on this?

Hi Dana,
Any update on this?

Hi @selvaduraiece I am not able to get that event firing. I am continuing to work on it.

Hi @selvaduraiece I am not able to get that event firing. I am continuing to work on it.

Hi Dhana, Was that helpful? hope the event will automatically triggered after 1minute.  Let me know if anything required from my end

Hi Dhana, Was that helpful? hope the event will automatically triggered after 1minute.  Let me know if anything required from my end

Hi Dana,
I am sharing the complete code. Just replace DirectLine secret with yours.

Extract the .zip and open WebChat.js and replace your Directline Secret as shown below

Open Assistant.html and start conversation as below

Once conversation is initiated wait for 2 minute, after that automatically it should trigger a msg as shown below. As I told earlier last msg will get disappeared automatically as shown below.

I hope this helps. Let me know if this doesn't help you out.
It would be grt if I get some assistance at the earliest.
chatbot.zip

Hi Dana,
I am sharing the complete code. Just replace DirectLine secret with yours.

Extract the .zip and open WebChat.js and replace your Directline Secret as shown below

Open Assistant.html and start conversation as below

Once conversation is initiated wait for 2 minute, after that automatically it should trigger a msg as shown below. As I told earlier last msg will get disappeared automatically as shown below.

I hope this helps. Let me know if this doesn't help you out.
It would be grt if I get some assistance at the earliest.
chatbot.zip

Thank you. I am testing this out.

Thank you. I am testing this out.

I am able to reproduce and investigating the cause.

I am able to reproduce and investigating the cause.

Still not sure why this is happening. I've isolated it to confirm it has nothing to do with whether it is ES5 or not, whether it is a composer bot or simple echo bot. I'm still investigating and troubleshooting the code to figure it out.

Still not sure why this is happening. I've isolated it to confirm it has nothing to do with whether it is ES5 or not, whether it is a composer bot or simple echo bot. I'm still investigating and troubleshooting the code to figure it out.

Okay, I tested with emulator using postman. But with Emulator it is not happening. I thought tht is because of Payload. Stil no luck. Will wait for your update.

Okay, I tested with emulator using postman. But with Emulator it is not happening. I thought tht is because of Payload. Stil no luck. Will wait for your update.

Hi, Any update on this?

Hi, Any update on this?

Hi @selvaduraiece My apologies. I am still investigating. Hope to have some info to share with you soon.

Hi @selvaduraiece I am still struggling to find out what the cause is with your approach. Would you be interested in other (timer) approaches that will work?

I am fine with other approaches as wel. can u help me with the approach
?

If the activity come in with the same ID or same channelData.clientActivityID, the previous one will be replaced.
Do you think if it's the case?

Thanks @compulim It is the clientActivityID.
With a quick comment out of this line the issue no longer reproduces:
  // payload.channelData.clientActivityID = updateConversation.clientActivityID;

@selvaduraiece Can you give the above a try and let me know if that works for you?

hi @selvaduraiece Just checking in. Does that resolve the issue for you?

Closing as solution/fix given. Please let us know if you need further assistance.

Closing, per request.


Finish Code-Docs

Summary:
Final changes for the docs inside the code including:

Event Documentation and Info
Openhiven.py Types Documentation and JSON-Data Reference documentation
Finish of Function Docs and Parameter docs

@AroLeaf
Kudo thought you said you'd do this

how would I ever write docs for something I didn't make and definitely don't understand??

but if you mean docs on hiven api stuff in general, I'm not going to make it any better looking than TODO.md, you know you can take anything from there

Prioritise. Docs. Over. The. Rest. @Nicolas-Klatzer.

how would I ever write docs for something I didn't make and definitely don't understand??

bitch m a g i c

how would I ever write docs for something I didn't make and definitely don't understand??

bitch m a g i c

shut

Assigning important - @Nicolas-Klatzer This really needs doing when possible

Assigning important - @Nicolas-Klatzer This really needs doing when possible

Assigning important - @Nicolas-Klatzer This really needs doing when possible

yes

Assigning important - @Nicolas-Klatzer This really needs doing when possible

yes

Assigning important - @Nicolas-Klatzer This really needs doing when possible

yes

and I can't do it because I'm focusing on Frost v1.4, so unless you can force leafeon to write docs for something he doesn't know..

Assigning important - @Nicolas-Klatzer This really needs doing when possible

yes

and I can't do it because I'm focusing on Frost v1.4, so unless you can force leafeon to write docs for something he doesn't know..

Replaced with #36

Replaced with #36


Platform flag is being ignored when scanning a remote image

What happened:
I used the following command to scan an linux/arm64 image
syft -v registry:idanfrim/multi-arch:only-arm64 --platform=linux/amd64

What you expected to happen:
The scan should have failed, there is no idanfrim/multi-arch:only-arm64 linux/amd64 image
Steps to reproduce the issue:
Run syft -v registry:idanfrim/multi-arch:only-arm64 --platform=linux/amd64
Anything else we need to know?:
Scanning the same image with the docker scheme failed as expected.
$ syft -v docker:idanfrim/multi-arch:only-arm64 --platform=linux/amd64
[0000]  INFO syft version: 0.77.0
[0000]  INFO new version of syft is available: 0.80.0 (current version is 0.77.0)
[0002]  WARN scheme "docker" specified, but it coincides with a common image name; re-examining user input "docker:idanfrim/multi-arch:only-arm64" without scheme parsing because image retrieval using scheme parsing was unsuccessful: unable to use DockerDaemon source: image has unexpected architecture "arm64", which differs from the user specified architecture "amd64"

Environment:

Output of syft version:

syft version
Application:        syft
Version:            0.77.0
JsonSchemaVersion:  7.1.1
BuildDate:          2023-04-11T14:32:58Z
GitCommit:          dd30c99bc2439cb91e3d084eb21e1040dd5a54dc
GitDescription:     v0.77.0
Platform:           darwin/amd64
GoVersion:          go1.19.7
Compiler:           gc

OS (e.g: cat /etc/os-release or similar): MacOS

@wagoodman I see  the platform support in #866 was added by you, do you mind take a look at that issue to see if that is a real bug or expected behavior? I would assume that adding platform to a remote scan will only scan the relevant image.

Hi @FrimIdan, thank you for filing the issue and sorry for the slow response! I think we have been able to reproduce this on our side. We'll take a look and see what we can come up with.


Characters URL-encoded twice in runInstalledQuery()

Describe the bug
When passing params as dict into TigerGraphConnection.runInstalledQuery() where any of the values contains a character subject to URL-encoding, the encoding happens twice, which, for example, in the case of space character, produces %2520 instead of expected %20.
1st encoding: https://github.com/pyTigerGraph/pyTigerGraph/blob/master/pyTigerGraph/pyTigerGraph.py#L1287
2nd encoding: when execution reaches requests.models.PreparedRequest.prepare_url()
To Reproduce
from unittest import TestCase
from unittest.mock import patch
from requests import Session, Response

from pyTigerGraph import TigerGraphConnection

class TestPyTigerGraph(TestCase):

    @patch.object(Session, 'send')
    def test_urlencode_space(self, send):
        res = Response()
        res.status_code = 200
        res._content = b'{"results":{}}'
        send.return_value = res

        conn = TigerGraphConnection()
        conn.runInstalledQuery("whatever", params={"param": "has space"})
        actual_url = send.call_args.args[0].url

        expected_url = "http://localhost:9000/query/MyGraph/whatever?param=has%20space"
        self.assertEquals(expected_url, actual_url)

Results in:

Expected behavior
The URL-escaping for the params behave correctly
Environment
Python 3.8.6 (default, Oct 27 2020, 08:57:44)
[Clang 12.0.0 (clang-1<IP_ADDRESS>)] on darwin

hey @maciejmrug
thanks for the bug report please note that this issue is fixed now the version is v<IP_ADDRESS>.3
please do :
pip install pyTigerGraph --upgrade
Thanks!

@zrougamed Thank you for a quick response and fix! :)


Added simple inference for record declarations

Still a lot of TODOs:

decide whether it is a class or a struct
maybe make it optional to not break things
where to place the additional declaration in the AST?

How should we tag the inferred declarations?
Is this a dummy or implicit or something else?
If I see it correctly, the nodes are currently tagged as implicit.

How should we tag the inferred declarations?
Is this a dummy or implicit or something else?
If I see it correctly, the nodes are currently tagged as implicit.

There was a lot of inconsistencies between those two and this PR might be a chance to rectify some mistakes here. An implicit node should be something which is definitely there, but not part of the AST, for example an implicit this. The dummy property originally also included these implicit definitions, but also the inferences. I would propose removing the dummy one (it is already deprecated) and introduce an inferred property for those.


"return over experience" / "chatty" file

tilted iced buoy
buoy coming back when ice melted
others? ...

datawell in icing conditions...


tracks path includes albums in place of artists

Hey there - I dunno if I'm missing something here but I am not sure this is working as intended...
What'd I'm trying to do is to pass a track ID to the catalogue API and return the title and artist in the same call. Here's what I have been testing with:
await catalogueClient.GET('/tracks/{id}', {
  params: {
    path: {id},
    query: {countryCode: 'NO', include: 'artists'},
  },
});

using ID 343267475, the data.relationships object looks like this:
"relationships": {
  "albums": {
    "links": {
      "self": "/tracks/330476412/relationships/albums?countryCode=NO"
    }
  },
  "artists": {
    "data": [
      {
        "id": "330476411",
        "type": "albums"
      }
    ],
    "links": {
      "self": "/tracks/330476412/relationships/albums?countryCode=NO"
    }
  },
  "providers": {
    "links": {
      "self": "/tracks/330476412/relationships/providers?countryCode=NO"
    }
  },
  "similarTracks": {
    "links": {
      "self": "/tracks/330476412/relationships/similarTracks?countryCode=NO"
    }
  }
}

note that artists has returned a relationship of type albums. The included object is also the album (full response is here). If I don't pass the include param, it works fine:
"relationships": {
  "albums": {
    "links": {
      "self": "/tracks/343267475/relationships/albums?countryCode=NO"
    }
  },
  "artists": {
    "links": {
      "self": "/tracks/343267475/relationships/artists?countryCode=NO"
    }
  },
  "providers": {
    "links": {
      "self": "/tracks/343267475/relationships/providers?countryCode=NO"
    }
  },
  "similarTracks": {
    "links": {
      "self": "/tracks/343267475/relationships/similarTracks?countryCode=NO"
    }
  }
}

but I'd then need to make an additional call to fetch the artist details.
The component seems correct:
https://github.com/tidal-music/tidal-sdk-web/blob/9ee48fc0c42bc02143920d0ef9ddb5f75734fc9f/packages/catalogue/src/catalogueAPI.d.ts#L554
so I'm guessing this is an upstream problem?
As an aside, probably not related as it pertains to similarAlbums but I also spotted some oddities in paths:
https://github.com/tidal-music/tidal-sdk-web/blob/9ee48fc0c42bc02143920d0ef9ddb5f75734fc9f/packages/catalogue/src/catalogueAPI.d.ts#L58-L64
which points to:
https://github.com/tidal-music/tidal-sdk-web/blob/9ee48fc0c42bc02143920d0ef9ddb5f75734fc9f/packages/catalogue/src/catalogueAPI.d.ts#L2433-L2460
which means the similarAlbums path gets a _1:
https://github.com/tidal-music/tidal-sdk-web/blob/9ee48fc0c42bc02143920d0ef9ddb5f75734fc9f/packages/catalogue/src/catalogueAPI.d.ts#L156-L162
Lemme know if I can provide anything 👍

Just a quick follow up - forgot to mention that shortly after I posted this, I tested with a regular non-SDK HTTP request and the results are the same (see below).
Seeing as this is likely an upstream issue, is there a better place for me to post it?

Yes this definitely looks like a bug at our API level!
@aliaksei-taliuk can you take a look? (I can reproduce the problem here when using track id 343267475 and including artists)
Thanks for reporting!
(As for the issue with similarAlbums, we have fixed that in the OpenAPI definition, but still need to publish a new version of the SDK, will try to get that out shortly :-) )

@jade-doggerel The fix has been deployed. Thanks for reporting it!

And the SDK has also been published with a fix for the similarTracks / similarAlbums now :-)


ES Creates Redundant Sys Log Entries When Creating ER Log Entries (GSFC DCR 22768)

For most of the reset cases, ES will create an entry in the Exception and Reset Log ( ER Log ) and it will also create a Syslog entry. Both the ER Log and Syslog are stored in the preserved RAM on a platform, so it is probably not necessary to create the redundant Syslog entries.

Determine the impact to unit and build tests, and remove the duplicate Syslog entires for resets.

Imported from trac issue 102.  Created by sstrege on 2015-08-28T14:18:50, last modified: 2019-03-21T16:57:33

Trac comment by glimes on 2015-09-02 11:52:27:
Suggestion: allow a tunable $THRESHOLD for duplicates before the
deduplication kicks in.
Concern: is the syslog entry actually redundant? Timestamp?

Trac comment by glimes on 2016-10-18 15:07:03:
Remove my name from many tickets (where it was applied by default), so others can tell that there are tickets that need some attention.


update kmeans.py

Hi,thank you so much for your work! It helps me a lot.
I fixed a small mistake.
it may not a big step,but I do hope that by our joint effort could make it better.
good luck:)

Hi, it seems that this repo is inactive for a while... (more than a year :worried:)
Would you consider to pass your changes to this fork https://github.com/Borda/keras-yolo3 ?


question about irpe

Hi, thanks for your work. Just a quick question.
line194 of irpe.py:
def _rp_2d_product(diff, **kwargs):
beta_int = int(kwargs['beta'])
S = 2 * beta_int + 1
# the output of piecewise index function is in [-beta_int, beta_int]
r = piecewise_index(diff[:, :, 0], **kwargs) + beta_int  # [0, 2 * beta_int]
c = piecewise_index(diff[:, :, 1], **kwargs) + beta_int  # [0, 2 * beta_int]
pid = r * S + c
return pid
r and c supposed to be piecewise index for row and column respectively. But I can't understand why times r with the buckets number(or square root of buckets number in product method)?
Thank you!

Thanks for your attention to our work : )
Let F denote the function piecewise_index, x denote the relative offset.
The range of F(x) is [-beta, beta], namely F(x) \in [-beta, beta]. (F(x) + beta) \in [0, 2 * beta].
Therefore r, c \in [0, 2 * beta], and the number of integers in [0, 2 * beta] is 2 * beta + 1, denoted by the constant value S (square root of buckets number in product method).
In order to get a unique bucket for different pair (r, c), (r * S + c) \in [0, S * S) is a unique ID of the bucket for the pair (r, c), where S * S is the number of buckets.

I get it. Thanks very much!

I get it. Thanks very much!


Wrong terminal tab is selected after reload window

Repro:

Open several terminals
Change their color
Show active terminal always via setting
Reload, notice how the active terminal (red below) is not selected in the tabs list

This is working for me, I think you fixed it with https://github.com/microsoft/vscode/commit/8ab8d2ae9515bfdb563d809d9be2a0cd41e462aa

Still repros on today's build:

Yep happens for me in today's build but not yesterday's, so the change that caused this was very recent

Confirmed it's the inline tab that's wrong


Fix: corrected byte count in shift

Description of changes
Fixes #80:
The count of bytes shifted out was incorrect. It did not affect operation because the excess byte was on the tail end and was discarded, however it unnecessarily increased timing and might cause problems in the future.
Checklist
If your change touches anything under src/ or the README.md file
these items must be done:

[v] Add a user friendly description of the change to the unreleased section of CHANGELOG.md

Can you also open a matching issue for this? Thanks!

Sure. I hadn't thought about it because it had no apparent malfunction, but no problem.

Il 03/11/2021 16:33 Neil Enns<EMAIL_ADDRESS>ha scritto:

Can you also open a matching issue for this? Thanks!

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/MobiFlight/MobiFlight-FirmwareSource/pull/79#issuecomment-959463583 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGD2RTR6FPVZ2SV4IU5UG2TUKFP5ZANCNFSM5HJDLJMQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .

It's always good to have an issue matching each pull request. Makes it easier to see the history of work in the project. Thanks!

From: GioCC @.>
Sent: Wednesday, November 3, 2021 8:38:10 AM
To: MobiFlight/MobiFlight-FirmwareSource @.>
Cc: Neil Enns @.>; Comment @.>
Subject: Re: [MobiFlight/MobiFlight-FirmwareSource] Fix: corrected byte count in shift (PR #79)
Sure. I hadn't thought about it because it had no apparent malfunction, but no problem.

Il 03/11/2021 16:33 Neil Enns @.***> ha scritto:
Can you also open a matching issue for this? Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/MobiFlight/MobiFlight-FirmwareSource/pull/79#issuecomment-959463583 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGD2RTR6FPVZ2SV4IU5UG2TUKFP5ZANCNFSM5HJDLJMQ .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMobiFlight%2FMobiFlight-FirmwareSource%2Fpull%2F79%23issuecomment-959479318&data=04|01||88dd0e857e7346e7c67008d99edff15e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637715506927839996|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2FHnAdl41Mf77RGsvnni%2B1M9khWNCzK1Z7Qa8WUb6H1c%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACIVHFUAFBDJWXUAOCHQUO3UKFJOFANCNFSM5HJDLJMQ&data=04|01||88dd0e857e7346e7c67008d99edff15e|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637715506927839996|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=CzibkF7FNdq28dT4U4Hek9l4ZNEWk%2FGS476HgQ6%2Fm2s%3D&reserved=0.

The change makes sense to me. I tried testing it locally but unfortunately my test setup was having issues even with the old firmware. I'll try again tomorrow (I had this problem before and can't remember what was causing it. I suspect a bad wire on the breadboard).

Took me forever but I wired a board up with two 74HC595s and managed to get the wiring correct and confirmed this change works as expected.

As I understand this pull request ,all changed files for the "handler" changes are still part of this one which are now in a separate pull request.

As I understand this pull request ,all changed files for the "handler" changes are still part of this one which are now in a separate pull request.

You're right, and that's unintended! My bad, I noticed this after @DocMoebiuz pointed it out yesterday. I will have to be more careful next time, for now it's probably better to leave things this way (or otherwise suggestions are welcome on how to proceed).

@GioCC Since this is such a small change the easiest way to fix this is to just do it again. Close this pull request, delete your branch, make a new one, apply the change, and open a new PR.
There are other ways to fix this with git revert but it's honestly easier to just make a new branch/PR for a chance this size.
Make sure to fetch from upstream/main first and merge to your main branch so the new branch is up-to-date with recent changes in the upstream repo.

@neilenns , sensible advice. I will definitely do that this evening back from work.


The loading dots on the visualization stayed visible on the full screen grid view

Discord username
No response
What type of issue is this?
Transient – Occurring only once
Is this issue blocking you from using Enso?

[ ] Yes, I can't use Enso because of this issue.

Is this a regression?

[ ] Yes, previous version of Enso did not have this issue.

What issue are you facing?
On freshly reopened project the full screen vis was opened. The loading dots stand visible together with the data.
https://github.com/enso-org/enso/assets/12892578/743da53c-d235-473e-961c-65aa85afd4f1
Expected behaviour
User should see either the dots or the data
How we can reproduce it?
No response
Screenshots or screencasts
No response
Enso Version
develop 17.05
Browser or standalone distribution
Standalone distribution
Browser Version or standalone distribution
standalone
Operating System
Windows
Operating System Version
No response
Hardware you are using
No response

Seems to be resolved - will close and reopen if occurs again.


Use version v2.2.0 as default helm version

Bump to the latest version (v2.2.0) would allow us to use the latest features of this helm.
Also, it seems this IAM policy would need to be added: https://kubernetes-sigs.github.io/aws-load-balancer-controller/v2.0/install/iam_policy_v1_to_v2_additional.json
More info about the migration: https://kubernetes-sigs.github.io/aws-load-balancer-controller/v2.0/guide/upgrade/migrate_v1_v2/

https://raw.githubusercontent.com/kubernetes-sigs/aws-load-balancer-controller/v2.2.0/docs/install/iam_policy.json


Add UDIM example to unit tests

There is a small example linked here.
I think it would be good to include this, or some other similarly small example,  directly in the repo, for use in unit tests.

Yes, a multi-UDIM example material would be a great improvement, if anyone has the bandwidth to contribute one!

Hi sorry for the late request, I would love to work on this!
I'm very new to open source software but this sounds like a very interesting project!

Welcome @Arin1604, and it's all yours!

@Arin1604 has let us know that school commitments will prevent him from tackling this project, so I'll mark this as free for others to work on.  Thanks for taking a look, Arin, and feel free to join other ASWF Dev Days events in the future!


CLIENT.fileRequest fails, same request works with standard http request

Yop,
This works:
  var request = require('superagent');

  request
    .post('https://api.recast.ai/v2/request')
    .attach('voice', file)
    .send({
      language: 'en'
    })
    .set('Authorization', 'Token '+process.env.RECAST_CLIENT)
    .set('Content-Type', '')
    .end(function(err, res) {
      console.log(err, res);
    });

This doesn't work with exactly the same file (it returns Error: Request failed with status code 400 and nothing more, so it's hard to guess what's wrong as it's the same file):
var recast = require("recastai");
var CLIENT = new recast.Client(process.env.RECAST_CLIENT, "en");

  CLIENT.fileRequest(file)
    .then(function (data) {
      console.log(data.source);
    })
    .catch(function(err) {
      console.log(err);
    });

Here is a file I'm sending. It's in French and I set recast to English but that's not the issue here (it returns 400, not some misunderstood content).
9cb212b7-067d-4f94-83c0-5f22c4e9cd44.wav.zip
Thanks for having a look! In the meantime, I'll stick with superagent ;-)

Hi Francois,
It's has been fixed in the latest version of the SDK, It should now work as intended!
We switched back to superagent ;-)
Thanks for reporting this issue!


Splitting apart code which contains pipe

I am documenting code. I frequently have cells containing {backtick} foo | bar {backtick}.
Doing alt + shift + f is splitting this into two cells.
Example:
| Parameter | Type                                   | Default     | Required | Description                                                    |
|-----------|----------------------------------------|-------------|----------|----------------------------------------------------------------|
| title     | `string | null | void`                 |             | Yes      | Array of strings that are offered as autocomplete suggestions. |
| content   | `string | null | void`                 |             | Yes      | A callback that gets                                           |

Awesome!!! THank you!


REST transport: callback is not a function

Reported in https://github.com/firebase/firebase-admin-node/pull/1901:
TypeError: callback is not a function
at /workspace/node_modules/google-gax/build/src/fallbackServiceStub.js:119:33
Function execution took 564 ms, finished with status: 'connection error'

We recently had to start using preferRest after having issues without it where firebase-admin would crash very often.
We made the changes on February 1st and have not crashed since. However, tonight we received reports of crashing happening again, and upon looking through the logs it seems like we have the same error as posted by @alexander-fenster:
Mar 08 22:38:35 /app/node_modules/google-gax/build/src/fallbackServiceStub.js:119
Mar 08 22:38:35 .catch((err) => callback(err));
Mar 08 22:38:35 ^
Mar 08 22:38:35
Mar 08 22:38:35 TypeError: callback is not a function
Mar 08 22:38:35 at /app/node_modules/google-gax/build/src/fallbackServiceStub.js:119:33
Mar 08 22:38:35 at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Mar 08 22:38:35
Mar 08 22:38:35 Node.js v18.14.2
Mar 08 22:38:37 running on port 3002

This is super crucial and crashing often, please advise ASAP

I'm looking.

Hey @alexander-fenster,👋
Thank you for the prompt reply.
A few follow up questions on that:
A) What is the normal turn around time for minor issues to be merged and released?
B) Although you state that we'll now see the real error behind the crash, that error itself shouldn't crash the backend, right? It's only crashing now because of the nonexistent callback?
We'll monitor this ASAP to see if it's an error we can prevent, once we see what it is. Considering our code has not changed, it must have been a non crucial error that normally passed unattended, and since a recent release of the SDK, it's only now causing a crash.

A) I expect it to go out today as a patch version, so whenever you redeploy / rerun npm install after it's released, you'll get it. I'll post an update in this issue.
B) Correct, the error will be passed up to the caller level. At this point I believe that it can only happen with the streaming call; since you're using Firestore, it might handle the streaming error in different ways (I'm not sure without looking at their code), ultimately, it might throw, but you will see it (and you'll be able to catch it).

@iAMkVIN-S I just published v3.5.8, please reinstall your dependencies. If it starts showing another error and the reason is not obvious, please come back here (feel free to submit a new issue here or in Firestore library, depending on what type of error you get :) )
I receive a lot of GitHub notifications (more than I can realistically process), so please feel free to ping me by email if you need faster response.


Add account type field to item, item details and search bar.

This PR adds the Account Type field in three places:

The item_component_table (item table on main screen).
Item details (appears in sidebar)
Search Bar.  Autocomplete is also supported.

This change is meant as UI only; no logic changes have been made as part of this PR.

Coverage decreased (-0.07%) to 52.645% when pulling 9627be90f5bc9a45ce67834f15744969c7bc885a on supertom:acct_type into 9f9f7f8ce18bf05e596144161e6ec73093a7172a on Netflix:develop.

Closed in #540.


nz-sider trigger disappeared after build.

Version
7.0.0-rc.3
Environment
OS:macOS 10.14.3  node:v10.9.0  npm:6.2.0  browser:Chrome 72.0.3626.109
Reproduction link
https://github.com/Sioxas/antd-ui-bug
Steps to reproduce
nz-sider trigger disappeared after build
Bug to reproduce
git clone https://github.com/Sioxas/antd-ui-bug
cd antd-ui-bug
npm install
ng serve

The trigger shows as normal.

In the production mode the trigger disappeared.
ng build --prod

or
ng serve --prod

What is expected?
Trigger shows after build.
What is actually happening?
Trigger disappeared after build.
Other?

this is resolved in the latest version, closed


Add ability for checkboxes to be used with arrays

Currently the checkbox component does not work as expected. Normally the input element of type checkbox can work with arrays referenced in v-model by adding and removing its value from the array, gb-checkbox just coerces whatever is in the value property to a boolean by using the ! operator and following that, it overwrites the array.

I don't understand what you are trying to do. Normally, a checkbox should return a Boolean value (true when checked, false when unchecked).
Do you have an example in mind?

Yes, I do, I would like to be able to use the checkboxes as shown in the example in Vue.js documentation: https://vuejs.org/v2/guide/forms.html#Checkbox

I see it now. Thank you. Unfortunately, I do not plan on adding this behavior as you could use the field-toggle to get the same.


Velbus auto-discovery

Description:
This pull requests simplifies velbus platforms by moving most of the logic to python-velbus. It now supports auto-discovery of modules.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#5132
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Happy to provide more info on this one if somebody requires it. Would be great to get some feedback on what is required to get this merged.

That would be great! @pvizeli : this is the change you suggested back then around platform discovery.

@MartinHjelmare changed based on your latest comments. Could you take another look and merge if okay for you?

Please add a paragraph in the description that describes the breaking change for the release notes.

Added a paragraph with breaking changes

Can be merged when build passes.

Thanks!

@MartinHjelmare this is now ready for merging imho

@MartinHjelmare : I have made change you requested but now build fails so I am at loss at what to do ...

It seems that the frontend requirement was bumped today without updating the requirements file. Now all builds are failing.
We hold here until we fix this.

Ok, dev branch should be fixed now. Please rebase on latest dev. That should hopefully be enough.

@MartinHjelmare it's working now.

Great! Thanks for your help @MartinHjelmare


Whats wrong with lexicon?

I've upgraded to 6.2.1 and now lexicon doesn't work ... any idea?.
My previous code:
let lexicon = Compromise.lexicon();
lexicon['ava', 'AVA', 'Ava'] = 'Person';

hey Javi, here's the instructions about how to add words to the lexicon
https://github.com/nlp-compromise/nlp_compromise/wiki/Lexicon#adding-words
it's a hash,

Ey guy, don't close it,
I'm so sorry but you have a bug, this is my complete code:
import Compromise from 'nlp_compromise';

const sentence = 'Ava, Do you know if tomorrow will rain in Bangkok?';
let lexicon = Compromise.lexicon();
lexicon['ava', 'AVA', 'Ava'] = 'Person';

const tags = Compromise.sentence(sentence, {lexicon}).tags();
>  [ 'Noun',
  'Infinitive',
  'Person',
  'Infinitive',
  'Condition',
  'Date',
  'Modal',
  'Noun',
  'Preposition',
  'Place' ]

hi Javi, please look more carefully at the docs.
lexicon['ava', 'AVA', 'Ava'] = 'Person'
is invalid javascript!
lexicon['ava']='Person'

@spencermountain It is valid, it just uses the last item in this car "Ava". Just tested it in node repl.

yeah, the lexicon needs lowerCase
var Compromise =require('nlp_compromise')
const sentence = 'Ava, Do you know if tomorrow will rain in Bangkok?';
let lexicon = Compromise.lexicon();
lexicon['ava'] = 'Person';
const tags = Compromise.sentence(sentence, {lexicon}).tags();
// [ Person, ...

Advance but if you use terms:
Person {
  whitespace: { preceding: '', trailing: ' ' },
  text: 'Ava, Do you',
  normal: 'ava do you',
...

The text is ava do you???

fixed in v6.3.1
(this was a pretty good one)
thanks

I love this library and I'm using it in hard-mode :) ... for now the only way I can help you is reporting issues :)


Remove @babel/preset-typescript defaults

Summary
Fixes #267
I was thinking of doing something similar to SWC rules https://github.com/shakacode/shakapacker/blob/master/package/swc/index.js#L28-L33 but in the end decided it brings no value realistically.
The setting in SWC to figure out config from the file was added to go around the limitations there, Babel should be already correctly figuring out file types and behaviour from the extensions with the default preset settings.
Pull Request checklist
_Remove this line after checking all the items here. If the item is not applicable to the PR, both check it out and wrap it by

[x] Update CHANGELOG file
Add the CHANGELOG entry at the top of the file.

Thanks,
My only concern is related to the Shakapacker version for this change to be applied.
@justin808 Should we have this for version 7 or version 6? As per your comment, I think it is better to have this for version 6.

With this being potentially breaking, I would keep it for v7 probably. Have we got any other stuff to go into v6?


[Feature Request] will it support Tensorrt?

🚀 Feature
Tensorrt support
Motivation
accelerate the inference with Nvidia Gpu

I think if TensorRT could meaningfully accelerate some subgraph, integrating it is not a problem in terms of engineering effort, but so far, according to the data we have, it doesn't yet. Definitely good to keep this option open in the long run


Attempting to split pane through command palette crashes Windows Terminal when in settings.

Windows Terminal version
1.13.11431.0
Windows build number
10.0.19044.1766
Other Software
No response
Steps to reproduce

Open Settings
Open command palette with keyboard shortcut ([ctrl + shift + p] in my case)
Search "split"
Select "Split pane, split: "

Expected Behavior
A new settings to be opened in parallel to the existing settings.
This way I can edit similar settings at the same time.
Actual Behavior
Windows Terminal crashes

Expected Behavior
A new settings to be opened in parallel to the existing settings. This way I can edit similar settings at the same time.

Alas, that's probably not something which will land for quite some time now - we'd have to land #997 first

As far as the crash is concerned, I can't repro this on 1.15 or 1.14. Can you check out one of those builds off our Releases page/? We may have coincidentally fixed this.

I fixed this in #13172 - this is a dupe of #13166


Is an SDL2 warning to be expected?

I'm just about all set up with my build and running the software for the first time shows the following warning.
fluidsynth: warning: SDL2 not initialized, SDL2 audio driver won't be usable
fluidsynth: warning: Failed to set thread to high priority
I get that sometimes it's ok to ignore warnings, but don't know enough about the nature of this one to make that call. Is the above warning an issue for me to resolve or should I disregard?

Hi @RyanParsley ,
The first one is generated by libfluidsynth and seems to be benign.  I don't like it, but have not been able to resolve it.
The second one should disappear when you run the program as root.


What about the implementation of sdxl inpaint pipeline?

Based on the great work, I modified some code to implement inpaint pipeline. I found it lose the ID face feature totally, and sometime failed to align mask region with the referenced face.  Here are results below:
original image:

mask image:

results:

Any suggetions would be appreciated. thank you in advance.

I solved it by myself, and the reuslts is pretty well.

can you show some your result?

@shoutOutYangJie .Here are test results below. As it shows, The aberration around face is obvious. Furthermore, the size of reference image influence the results a lot.
original image:

results:

It need a further  improvement and i think img2img pipeline is in need.

@shoutOutYangJie .Here are test results below. As it shows, The aberration around face is obvious. Furthermore, the size of reference image influence the results a lot.
original image:

results:

It need a further  improvement and i think img2img pipeline is in need.

@shoutOutYangJie .Here are test results below. As it shows, The aberration around face is obvious. Furthermore, the size of reference image influence the results a lot. original image:  results:
It need a further improvement and i think img2img pipeline is in need.

That's a cool result. Could you please share the code of the inpainting pipelines?

I'm sorry I response this so late, because I didnt satisfied with my implement of this, and I finally use this code to reach my goal;
Furthermore, If use sdxl-inpainting-1.0 model,  the  more smooth the results you got.
Here are my results below:


Add the TODO comment to support for email notification

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Thank you.
/approve


Remove the extra delay in storage integration test on mobile.

This delay broke the Storage integration test on mobile, though it helps with flakes on desktop.

Is there a ticket associated with this that we can reference?

No ticket, should I add one retroactively?


Configurable Prompts: Starting with Commit Message

If we want commit messages to follow a certain format, there is no way to configure it. We could add configuration / arguments to aider for customising the prompt for generating commit messages.
For example, a team may choose to commit with a short word descriptor (fix/feature/refactor/chore).
refactor: add type definitions to orphan functions
Currently this requires editing the source. A yaml configuration could be very helpful!
I'd like to help out to contribute this, but not sure if is appropriate as command-line argument?

Here's a basic PR to possibly work off of: https://github.com/paul-gauthier/aider/pull/494

This is possible with the --commit-prompt switch now.
I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


Pretrained=True does not work in Wav2vec.

Here is another error in wav2vec.
https://github.com/shangeth/wavencoder/blob/6874c6738f37bf04c8105bf6839e495848196a44/wavencoder/models/wav2vec.py#L217-L221
In the above code, pretrained_dict becomes an empty dict {} because there is no intersection between pretrained_dict.state_dict() and feature_extractor.state_dict().
How about this:
pretrained_dict = cp['model']
model_dict = self.state_dict()
pretrained_dict = {k: v for k, v in pretrained_dict.items() if k in model_dict}
model_dict.update(pretrained_dict)
self.load_state_dict(model_dict)

Thanks!

Hi @Harry24k
Check the latest push.
Hope the issue is solved.
Thanks for pointing the issue.


Generating a dark mode version

Hi folks! Thanks for this awesome project!! It's really really good!!
I do have a mobile app where I'm showing sheet music as a PDF file, they look beautiful on the device. However, the app supports dark mode and I'd like to show a dark version of the PDF. I'm thinking of generating two sets of files, one in a light color and another one in dark.
Is it possible to change the background of each file from white to black? Is it possible to change the content of the document from black to white?
This is the original PDF I have:
1.pdf
It looks like the background color can be changed like this: https://github.com/Hopding/pdf-lib/issues/446
But what about the content? Texts and notes?
Thanks!

I don't think you're going to have much luck changing that amount content in an existing file. I think the best approach would be to generate a light and dark version up front, when the sheet music PDF is first being generated. If that can't be done, then you could try applying some sort of graphical filter on top of the rendered PDF to modify the colors.


upgrade libuv to 1.41.0

@costanic , I think you were the last person to touch greasego.  Can you take a look at this?  It's related to: https://go.arm.com/OZJrBNB

@ryannowarm this change needs to be made directly in the maestro repo in the vendor folder.  This repo is not used and needs to be deprecated.

This repo is not used and needs to be deprecated.

Ah!  Thanks.  I'll move this PR. Can you mark this repo as "archived"?  I think have have admin rights.


[Feature Request] Ability to minimize to system tray

It would be nice to be able to have option to minimize Insomnia to system tray
When that option would be enabled it could work like that on Windows:

Click on close button (×): minimize to tray
Click on minimize button (–): minimize to taskbar (standard behavior)
To fully quit Insomnia: right click on tray icon and click something like "Quite Insomnia"

Having ability to minimize applications to tray is pretty useful for decluttering when working with a lot of open applications that take the taskbar real estate

https://electronjs.org/docs/api/tray

Hi @DominikSerafin,
Thanks for the suggestion. In my opinion, a tray icon should only be implemented if it adds additional functionality that requires it. The problem you describe seems like it should be solved in user-space, not by Insomnia.

@gschier how can it be solved by user-space? Is there a possibility to write plugin that would solve it?

👀

Hey @gschier, it seems this issue can't be solved by plugins (if that's what you meant by user-space).
From reading the docs, it doesn't seem there is any support for hooking into Electron app events like ready, etc. which to my knowledge are needed to add tray functionality.

I mean it should be solved by tools external to insomnia. It seems like you have a window management problem in general which is not specific to Insomnia. Why not minimize the app instead?
I would consider adding a tray icon if it provided useful functionality but not it it is simply to save space.


Quick question about bullseye and pi zero 2w

Hi again @oandrew,
I'm back again with some free time to once again try and get this to work on the boars Clarion MD6 headunit.
I had a question I wondered if you would have time to test for me... being that raspbian bullseye is now compatible with both the zero most if not all raspberry pi variants - I wondered if you would be able to do an install of your ipod software with that os as the base test it working for yourself (Being its your project, you'd know for sure if it was working as expected) and from there create an image file of the pi's sd card that I could then attempt to deploy onto my own pi zero 2w.
The reason I ask for this route, is I simply cannot seem to work out where things are failing.... And being the head unit does nothing but tell me reading usb - and even with swap_configs enabled - and setting the apple_id (not the correct configstring I know) to be that of an ipod nano 5th gen which the cmd6 supports - Still get no joy.
If this is too much of an ask I totally understand, and I also understand this isn't an issue with the software so feel free to close this off (I couldn't find any other way to get in contact with you to try and progress forward with advancing things).
Hope this message finds you well.
Cheers
James

Hi, I hope to find some time to look into this in the following weeks.
Thanks!
Andrew


Raise min PHP version to PHP 7.3, check compatibility PHP 8.3.

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

This PR introduces the following changes:

Raises the minimum PHP version to 7.4.
Update PHPUNIT to version 9.6.
Removes the cgwans path for PHPUNIT.
Add tests for PHP version 8.3.
Add tests for PGSQL version 16.

These changes could potentially enable us to introduce type hinting in the code in the future, prevent unintended errors, incorporate static analysis tools such as PHPSTAN, and implement code styling using StyleCI.

Why PHP 7.4? PHPUnit 9 should work fine with PHP 7.3.

Why PHP 7.4? PHPUnit 9 should work fine with PHP 7.3.

I thought about the latest version of PHP 7, the most recent, 7.4, but if everyone agrees, i can lower it to 7.3, the tests will pass anyway.

I think we can safely remove https://github.com/yiisoft/yii2/blob/81f3033087b5568442ca4013ce2b6149336722bb/framework/base/Object.php, since it is unusable since PHP 7.2.

Why PHP 7.4? PHPUnit 9 should work fine with PHP 7.3.

I thought about the latest version of PHP 7, the most recent, 7.4, but if everyone agrees, i can lower it to 7.3, the tests will pass anyway.

Done PHP 7.3.


Instructions frame not hidden if there are no instructions for programming exercise

Describe the bug

Instructions frame not hidden if there are no instructions for programming exercise. Instead empty space is shown, which can be resized and behaves like an instruction frame.
To Reproduce

Go to 'Overview'
Click on course that has programming exercise without instructions
Select 'Open code editor' on the programming exercise
See empty space on the right

Expected behavior

Instruction frame should be minimized and not use extra space of the code editor.
Screenshots

It's also not possible to change the instructions to an empty String --> The PATCH request returns a 400


feat: direct indexer query path

Add an API path /api/deployments/id/:deployment/indexers/id/:indexer (alternatively, add an optional graph-indexer header to the existing deployments path). When using this feature, the request bypasses indexer-selection and goes directly to the specified indexer without being rewritten. This is to facilitate client-side cross-checking efforts.

implemented by #867


licensing

Hi, the code does not clearly states the license, except for atmel code that is something like mit + atmel hardware only clause

Thanks! added MIT license.