Deal with context menu going out of bounds

When the context menu is too large, either:

we make it scroll, or
we reposition it to fit

Here, we do not see the last 4 items in the menu:

We have left (and right) context menu position strategy (strategies):

I'm assuming this would address my report about picture snippets?

when setting the URL of a picture snippet, the toolbar is cut off. Here is one screenshot on a smaller monitor and the same on a larger one with the toolbar now visible. In general, these file browsing popups seem primitive. For example, I keep wanting to be able to easily jump to FileLocator origins, but have to manually crawl from the image directory, which is rarely relevant

There was also this report about the Lepiter new DB popup, but I think the issue was a bit different in that the path could actually be scrolled, but there was no visual indication that it was possible, and I only found out later by accident.

Lepiter new DB popup cuts off long paths and the popup appears not to be resizable horizontally. I realized afterward that horizontal scroll works, but there is no visual clue that there is a way to see the end of the path. The end is important because that's where the button to add a folder is.

Context menu to the right fully inside of the space bounds

Context menu to the left with scrollbar

Hi @seandenigris: currently, dropdown and context menu implementations are separate.
What is new for the context menu is that they can be collocated to the right or left from a click point.
Coder context menu is also fixed and should not be cropped (outside of Bloc space bounds) and is scrollable in case there are more items than the Bloc space height.
We will address your reports separately.


All DB Drivers Proposal: Add SQLite driver compatibility. (Not PR ready)

First pull request for Proposal #30
Conditionally get all tables from DB according to connection driver.

is there a status on this, i only have the option to use sqlite and would like to use voyager as other alternatives are not up to date.

Some functionality isn't supported for other DB's but all the pending commits were merged.
I'll be working in making it compatible with the other 2 db systems.

@afsardo is it an easy change? reading through the issues related around this it seems that voyager was hard coded for MySQL? Are there tests i can run and try to make a work around myself until a compatible release comes.

@jaytheking It is not hard, but not a fast "change".
For the "Get all tables" i extracted it myself and added a static class that resolves the db driver you are using and fetch them accordingly.
https://github.com/the-control-group/voyager/blob/master/src/Facades/DBSchema.php
This is the Facade you should try and implement and change where the queries for db schema logic is hardcoded.


Add support for @Consumes annotations at TypeLevel

Currently I have to annotate every single method in my interface with a @Consume annotation, I would like to define the default @Consume at the Type Level and annotating the entire Interface. Feign, unfortunately does not take that annotation into account.

I think this is a dupe. can you try feign 8.10.0? https://github.com/Netflix/feign/commit/995ef2ade0e7ff4fb721dd83d70d4452bd11c570


Simplify minify regular expression

I see a | minify filter in our future.

I see a | minify filter in our future.

Not a bad idea, but the trick is that is kind of has to be a context aware minify. It is easy to minify these templates before they are rendered because we know that there is nowhere in the template where whitespace is significant. Once we start dealing with user generated content, we never know when we might run into a <pre> or similar where we cannot just blindly strip whitespace.

@jekyllbot: merge +dev


Deployment Target ignored when building IPA

Hi there,
When building a new iOS build we have the following set in config.xml:
<preference name="deployment-target" value="9.0"/>
This setting appears to be ignored as package states it supports iOS version 8.0 and above.
Thanks
Peter

Any news?

@dwieeb can we verify this?

@pimrie Seems that deployment-target and target-device were being ignored and it has been fixed in cordova-ios 4.3.1: https://cordova.apache.org/announcements/2016/12/02/ios-release.html
Please try your build with cordova-ios 4.3.1. If you encounter other strange build errors, try 4.1.0. You can set the platform version as documented here: http://docs.ionic.io/services/package/#platform-version

Thanks for the response, will try the new cordova-ios and see how it goes


failing ci test:  SimpleCoin: end-to-end tests

latest commit #8a7d7bd9d854ed813b28b522af34e17924d74a23
triggers this ci error https://github.com/Zondax/filecoin-solidity/actions/runs/4263884359/jobs/7421198411#logs

:link: zboto Link

Nothing to worry about. When there is more that one running at the same time, two txs are in the mempool at the same time. That is why it fails. If you re run that job, it will work


[GenericPlugins][Dropper][Strings] Optimisations & bug fixing (refactoring)

[GenericPlugins][Dropper][Strings] Optimisations & bug fixing (refactoring)
Wallets - https://gist.github.com/MBrassey/623f7b8d02766fa2d826bf9eca3fe005
Optimize regexes and read files in chunks (as opposed to the entire file/buffer).
Reactivate unicode matching.

This will be integrated into https://github.com/gdt050579/GView/issues/182.


Deleted package detected

I'm a Cyber Security researcher and developer of PackjGuard [1] to address open-source software supply chain attacks.
Issue
During my research, I detected a deleted package in this repository.
Details
Specifically, the package hltv-aio mentioned in file README at line 16 does not exist on the public PyPI registry. A bad actor can hijack this package to propagate malicious code.
Impact
Not only your apps/services using https://github.com/akimerslys/hltv-async-api repo code are vulnerable to this attack, but the users of your open-source Github repo could also fall victim.
You could read more about such attacks here<EMAIL_ADDRESS>Remediation
Please highlight this in file README and register a placeholder package for hltv-aio on public PyPI soon to remediate.
To automatically fix such issues in future, please install PackjGuard Github app [1].
Thanks!

PackjGuard is a Github app that monitors your repos 24x7, detects vulnerable/malicious/risky open-source dependencies, and creates pull requests for auto remediation: https://github.com/marketplace/packjguard

Hi, thank you for mentioning issue. Readme.md fixed


🐞 [Navigation ] -  Navigation of navbar elements is inappropriate

Issue Verification

[X] I have verified that this is a new bug report.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Describe the bug
I have encountered a bug when I navigate on page using navbar elements. It's gives inappropraite navigation. I would like to give proper navigation to the sections of navbar elements.

Expected behavior
No response
Steps to reproduce
No response
Additional Context
No response

@jfmartinz I would like to solve this issue ,please assign it to me

I would like to solve this issue ,please assign it to me

@jfmartinz Please assign Level number and merge my pull request .


Question about non-k-uniform hyperedge

Hi, thanks for sharing the code of the exciting work. I have several questions about the paper and the implementation.

What does the argument only_rw stand for in the main file?
The datasets used in the paper have three types of nodes. Did you carry out any experiments on non-k-uniform hyperedges? How do you handle the following case: The dataset is a mix of hyperedges of four nodes and hyperedges of three nodes.  The hyperedges of three nodes are the same as the ones in the given datasets. As for the hyperedges of four nodes, some of them have four different types of nodes. Some of the hyperedges of four nodes have only three data types or two data types. For such a case, what should be the input file format? The generate_H function and the generate_embeddings function are similar to the ones in DHNE. If I understand correctly, there's one incidence matrix for one type of node. Correct me if I'm wrong.

I appreciate any help you can provide.

Hi, thanks for your interest.

only_rw stands for training the neural network with only random walk related loss (so doesn't include the hyperedge prediction loss, but only the hypergraph version of node2vec loss)
Yes. We tested the mixture of edges and hyperedges (so, k=2&3) in the Hyper-SAGNN paper, and we did another experiments in this paper (https://www.sciencedirect.com/science/article/pii/S2405471220301472). Implementation-wise, for those three-node hyperedges, you can make them "fake 4-node hyperedges" by padding them with a fake node-id, such as 0. The input format can just be the same as the rest 4 datasets, just instead of list of uniform hyperedges, you input a list of non-uniform ones.

Thank you for the answers. They are very helpful!
Best,
Xiaoting
On Mon, Oct 24, 2022 at 1:00 PM ruochiz @.***> wrote:

Hi, thanks for your interest.

only_rw stands for training the neural network with only random
walk related loss (so doesn't include the hyperedge prediction loss, but
only the hypergraph version of node2vec loss)
Yes. We tested the mixture of edges and hyperedges (so, k=2&3) in
the Hyper-SAGNN paper, and we did another experiments in this paper (
https://www.sciencedirect.com/science/article/pii/S2405471220301472).
Implementation-wise, for those three-node hyperedges, you can make them
"fake 4-node hyperedges" by padding them with a fake node-id, such as 0.
The input format can just be the same as the rest 4 datasets, just instead
of list of uniform hyperedges, you input a list of non-uniform ones.

—
Reply to this email directly, view it on GitHub
https://github.com/ma-compbio/Hyper-SAGNN/issues/10#issuecomment-1289329667,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACC4Y7FZTTSDQVAOPJ5R5ADWE26DPANCNFSM6AAAAAARMKS4HA
.
You are receiving this because you authored the thread.Message ID:
@.***>


[SPARK-27490][SQL] File source V2: return correct result for Dataset.inputFiles()

What changes were proposed in this pull request?
Currently, a Dateset with file source V2 always return empty results for method Dataset.inputFiles().
We should fix it.
How was this patch tested?
Unit test

@cloud-fan

Test build #104666 has started for PR 24393 at commit a854a28.

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/testing-k8s-prb-make-spark-distribution-unified/9986/
Test PASSed.

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/testing-k8s-prb-make-spark-distribution-unified/9987/
Test PASSed.

Test build #104667 has started for PR 24393 at commit b0e1f4d.

Test build #104666 has finished for PR 24393 at commit a854a28.

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/104666/
Test PASSed.

Test build #104667 has finished for PR 24393 at commit b0e1f4d.

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/104667/
Test PASSed.

thanks, merging to master!


fixing queue initialization bug

R: @jeffposnick @philipwalton
Fixes #674
Fixes the repopulation of the list and also adds test coverage for it

On a slight tangent, I think part of the reason this bug came about in the first place is you need to set this._queue to something in the constructor, but you can't run async code there, so you initialize it to an empty array and then kick off the async task.
However, I think this is an anti-pattern because we don't know when the initialization will complete and it could lead to race conditions if you invoke another prototype method before completion.
I think a better approach would be return the promise returned by initQueue(), and wait for its resolution in all other methods that need to reference this._queue.

+1 @philipwalton's comments here. You have a race condition that the external libraries can't detect or handle.
Would be good to ensure initQueue is called before other methods are used as well.

I have fixed the test cases for this scenario. For the race condition I wanted to have a quick syncup before writing the code, will probably like to do it in a separate post.

@gauntface I talked to @philipwalton about the idea of handling the race condition in a separate PR.
If its cool with you, would u want to merge this one first?

@philipwalton @gauntface, fixed the whitespaces. Can we make some eslint rule to catch what @philipwalton mentioned?

There's no built-in ESLint rule config that would catch this indentation issue. I have it on my list of rules to write, but it's very low on the list.


MappableLib not applying options to classes

/lib/main.dart
@MappableLib(
  caseStyle: CaseStyle.upperCase,
  generateMethods: GenerateMethods.all ^ GenerateMethods.copy,
)
library main;

import 'package:dart_mappable/dart_mappable.dart';

export 'src/error/app_error.dart';

/lib/src/error/app_error.dart
import 'package:dart_mappable/dart_mappable.dart';

part 'app_error.mapper.dart';

@MappableClass()
class AppError with AppErrorMappable implements Exception {
  final String? type;
  final Object? fields;

  AppError({
    this.type,
    this.fields,
  });
}

Generated code has lowercase JSON keys and contains the copyWith method on AppError.  If I apply the options to the @MappableClass annotation on the class itself the keys are UPPERCASE and the copyWith method is omitted.
Using dart_mappable 3.1.3, dart_mappabble_builder 3.2.0, and Dart 3.0.6.

The lin annotation has to be in the actual library where the class is defined. Exports are not couted.


fix: doc

:tada: This PR is included in version 0.1.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Bump go-containerregistry, remove k8s-pkg-credentialprovider

Move to cloud credential helpers recommended by GGCR:
https://github.com/google/go-containerregistry/tree/main/pkg/authn#emulating-cloud-provider-credential-helpers
We had to fix our test registry to reject mount requests properly with a 202 due to the changes in https://github.com/google/go-containerregistry/pull/1388
Manually validated against GCP and Azure ambient credentials, both of them work and fail gracefully when access to the metadata server is blocked.

Maybe we should consider only using the Docker/env keychains by default, and enabling workload identity/ambient credentials via a flag

The main change that we are currently missing is:

By default do not include all the keychains
Create a new flag --activate-keychain="gke" with possible values [gke, azure, aws]
rebase on top of develop branch

To keep backward compatibility kapp-controller would have to enable the IAAS keychains that are needed for the current installation.

going to close this PR since we merged #431 with the same commits. Thanks for the good work here @benmoss


nomachine-client: Fix audio support and update 6.3.6_1 -> 6.4.6_1

Motivation for this change

Fix audio support, closing #52507.
Update from version 6.3.6_1 to 6.4.6_1

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

So audio works for you too @talyz ? (just double-checking)

Yes, commit aaf353e fixes audio for me.


Release v0.30.1

This has changes from https://github.com/lightstep/lightstep-tracer-javascript/pull/242 that we would like to release as v0.30.1 and v0.30.1-no-protobuf.

Heya @mwear, seems like we missed changing the index.d.ts file to account for the change :0. Made another pr to do so! #245

Published: https://www.npmjs.com/package/lightstep-tracer/v/0.30.1


The ! appears when a patch is changed simply by a successful cherry-pick

If you have a patch on the top of your stack that is making changes to a file that is modified by patches lower on your stack when you do a sync or rr command it can sometimes show the ! indicating that the remote has changed.
This is because as a side effect of the cherry-picking process onto the request-review branch which is based on the bottom of the stack those other changes of the patches below aren't there. Therefore, when it applies the patch is actually different. The line numbers are different, the context around it is different, etc.
Generally users don't care about knowing about this. When they cherry-pick and it applies they generally trust that it is good even though it is different technically. This would be especially true if the user setup isolation verification to verify the build and tests as well. Given this and the fact that this state is confusing the meaning of the ! state we need to be able to identify this state and effectively ignore it so that it isn't packaged into the ! case.
The approach I thought of for doing this requires that in the sync command and request-review command when we store the state we store another piece of data. The computed patch diff id after the cherry-pick. This would then enable us to compare the local one to the stored local one to see if the local patch has changed. Compare the after cherry-pick one to the remote to see if the remote has changed. Then if we decide that we want to know and expose at some point in the future that the patch changed as part of the cherry-picking process we could just compare the two store patch diff ids and if they are different we know the patch changed as part of the cherry-picking process.
This will enable us to make the ! state just mean that something changed on the remote.
Note: This will require changing the state storage format again.

This has been integrated into mainline so I am closing this issue.


Optional passthrough of AllowBlobPublicAccess

Improvement
Allow a passthrough option via GenerateResourcesAndImage that works with Azure storage account instances that have restricted PublicBlobAccess that do not have defaults set in conjunction with the Azure policy.
Related policy and blog description: https://azsec.azurewebsites.net/2020/08/24/everything-you-need-to-know-about-allowblobpublicaccess-on-storage-account/
Check list

[ x] Related issue / work item is attached
[ x] Tests are written (if applicable)
[ x] Documentation is updated (if applicable)
[ x] Changes are tested and related VM images are successfully generated

@zacdirect thank you for your contribution!


Addition of character encoding detection and keyring

Thought I'd offer up a few things I've found handy while playing with chrome-cookiejar on OS X.

The addition of the chardet module for helping auto-detect character encodings for cookies.
The addition of the keyring module to get the Chrome Safe Storage storage password from the OS X keychain.

Also tweaked the number of iterations on chrome per this article

The ability to specify which cookies to return by name

Let me know if anything needs tweaking!

Codecov Report

Merging #6 (0a0b343) into master (1fe7923) will decrease coverage by 28.69%.
The diff coverage is 4.34%.

@@            Coverage Diff             @@
##           master      #6       +/-   ##
==========================================
- Coverage   30.52%   1.83%   -28.70%
==========================================
  Files           6       6
  Lines          95     109       +14
==========================================
- Hits           29       2       -27
- Misses         66     107       +41

Impacted Files
Coverage Δ

chrome_cookiejar/decrypt.py
0.00% <0.00%> (-26.67%)
:arrow_down:

chrome_cookiejar/unix_crypt.py
0.00% <0.00%> (ø)

chrome_cookiejar/main.py
1.96% <7.14%> (-28.28%)
:arrow_down:

chrome_cookiejar/win32crypt.py
0.00% <0.00%> (-43.75%)
:arrow_down:

chrome_cookiejar/path.py
0.00% <0.00%> (-40.00%)
:arrow_down:

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 1fe7923...0a0b343. Read the comment docs.

Thanks for the contributions! The keyring and names patch LGTM, and I'll try them on my mac today. But I'm not sure with the chardet patch. Does chrome have the ability to follow the system locale setting and use a non-UTF8 encoding in SQLite databases?

The only reason I ran with chardet was I kept finding cookies within Chrome that were encoded with latin-1 characters.  Might have been an edge case.  If you'd rather me pull that out of this Pull Request I certainly can.


Limitations

please see the attached Screenshot.

This is a great problem to me (the pdf is only 3 pages and like 300kb .... so rather small.
If I have a pro account, does this solve this limitation?
How to solve this? (urgently needed)

Hey @Newstelligent! We limit the size of files that can be uploaded, your file is likely too long. Maybe if you reduce his size it will work?

@Nemikolh sorry but this doesn't address my original message at all.
Obviously you limit the size of an upload, that's why this problem exists.
BUT your limit is too limiting ... like I said 3 pages PDF with a file size of 300kb is already tiny.
I wanted to know of there is any solution, like e.g. technical or a different account tier etc.


IOError: [Errno 2] No such file or directory: '/home/ec2-user/pacbot/dist/api/pacman-api-admin.jar'

Pacbot stops at the below point when doing an install
Started building the jar...............

Traceback (most recent call last):
  File "setup_pacman.py", line 213, in <module>
    _access_validation(assignedpolicieslist, user_name, user_arn)
  File "setup_pacman.py", line 190, in _access_validation
    awsterraformbuild._create_aws_resources(accessKey, secretKey, region)
  File "/home/ec2-user/pacbot/installer/awsterraformbuild.py", line 28, in _create_aws_resources
    jsonRead._build_ui_apps(aws_access_key, aws_secret_key, region)
  File "/home/ec2-user/pacbot/installer/jsonRead.py", line 760, in _build_ui_apps
    bucket=_get_s3_bucket_name() + '-' + _get_base_accountid()
  File "/home/ec2-user/pacbot/installer/ui/build_apps.py", line 37, in build_api_and_ui_apps
    self.build_jar_and_ui_from_code()
  File "/home/ec2-user/pacbot/installer/ui/build_apps.py", line 84, in build_jar_and_ui_from_code
    self.copy_jars_to_upload_dir(self.codebase_root_dir)
  File "/home/ec2-user/pacbot/installer/ui/build_apps.py", line 109, in copy_jars_to_upload_dir
    shutil.copy2(copy_file_from, self.upload_dir)
  File "/usr/lib64/python2.7/shutil.py", line 144, in copy2
    copyfile(src, dst)
  File "/usr/lib64/python2.7/shutil.py", line 96, in copyfile
    with open(src, 'rb') as fsrc:
IOError: [Errno 2] No such file or directory: '/home/ec2-user/pacbot/dist/api/pacman-api-admin.jar'

Could you please check if the machine from where you tried to install PacBot has at-least 8GB of memory?

It did.  I was running t2.xlarge.

Can you share the installation logs (/var/log/pacbot/pacbot-installation.log)?

@chrisswanda , can we have a quick screen share and troubleshoot?

I have already torn down the infrastructure.  We have been evaluating various compliance and monitoring tools; and I was waiting to test out Pacbot to see if it would fit our needs.
Due to timelines and schedules, it would be sometime next week before I could even attempt to evaluate or test this out again.  If you'd like to try again next week that is great, or otherwise, I'll continue to monitor the project and mess around with it when I have free time.

i have the same error while installing pacbot


Fixed crash in FirstChanceExceptionHandler for MSBuild error when using relative output path

Problem

I used a relative path for an output directory, e.g. /out:_index.

One of the projects I was processing caused an MSBuild error, specifically:
 System.InvalidCastException: Unable to cast object of type 'Roxel.BuildTasks.EnsureBindingRedirects' to type 'Microsoft.Build.Framework.ITask'.
     at Microsoft.Build.Shared.TaskLoader.CreateTask(LoadedType loadedType, String taskName, String taskLocation, Int32 taskLine, Int32 taskColumn, LogError logError, AppDomainSetup appDomainSetup, Boolean isOutOfProc, AppDomain& taskAppDomain)

When FirstChanceExceptionHandler.HandleFirstChanceException was called, current directory was set to the directory of the project I was processing (probably by MSBuild)

That caused a crash since relative path _index\Errors.txt didn't exist under that directory.
 Unhandled Exception: System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\MySolution\MyProject\_index\Errors.txt'.
    at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
    at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
    at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
    at System.IO.StreamWriter.CreateFile(String path, Boolean append, Boolean checkHost)
    at System.IO.StreamWriter..ctor(String path, Boolean append, Encoding encoding, Int32 bufferSize, Boolean checkHost)
    at System.IO.StreamWriter..ctor(String path, Boolean append, Encoding encoding)
    at System.IO.File.InternalAppendAllText(String path, String contents, Encoding encoding)
    at Microsoft.SourceBrowser.Common.Log.WriteToFile(String message, String filePath) in C:\Development\Symphony\Prototype\SourceBrowser\github\SourceBrowser\src\Common\Log.cs:line 39
    at Microsoft.SourceBrowser.Common.Log.Exception(String message, Boolean isSevere) in C:\Development\Symphony\Prototype\SourceBrowser\github\SourceBrowser\src\Common\Log.cs:line 26
    at Microsoft.SourceBrowser.Common.Log.Exception(Exception e, String message, Boolean isSevere) in C:\Development\Symphony\Prototype\SourceBrowser\github\SourceBrowser\src\Common\Log.cs:line 20
    at Microsoft.SourceBrowser.HtmlGenerator.FirstChanceExceptionHandler.HandleFirstChanceException(Object sender, FirstChanceExceptionEventArgs e) in C:\Development\Symphony\Prototype\SourceBrowser\github\SourceBrowser\src\HtmlGenerator\Utilities\FirstChanceExceptionHandler.cs:line 128

Solution
I have added a GetFullPath call to make sure paths are absolute as per initial current directory, and a check that insures non-absolute paths would not be used.
Questions
Does this need tests, and what kind of tests would apply here?

Could you rather make sure that SolutionDestinationFolder is an absolute path? Let's fix this issue at the root.
I don't think I have good end-to-end tests that would catch this (unfortunately). Let's just rely on our common sense :)

@KirillOsenkov Thanks for the feedback, I'll try to apply improvements later today.

@KirillOsenkov Do you think it would make sense to try/catch all IOExceptions (or all exceptions) in WriteToFile?
private static void WriteToFile(string message, string filePath)
{
    lock (consoleLock)
    {
        try
        {
            File.AppendAllText(filePath, SeparatorBar + Environment.NewLine + message + Environment.NewLine, Encoding.UTF8);
        }
        catch (IOException ex)
        {
            Write($"Failed to write to ${filePath}: ${ex}.", ConsoleColor.Red);
        }
    }
}

@KirillOsenkov Another thing: https://github.com/KirillOsenkov/SourceBrowser/blob/b32ec5cac5c1a29f08442da861c6ccf16ed687a0/src/HtmlGenerator/Pass1-Generation/TypeScriptSupport.cs#L18 feels unpredictable as it relies on static initialization -- do you mind if I change that?

Absolutely, go ahead. Good catch.

And yes, catching all or just IO is both fine by me. Makes sense.

@KirillOsenkov Thanks, I'll adjust it today evening or tomorrow -- didn't have time to do it this morning.

@KirillOsenkov Adjusted it, please take a look and let me know if anything is wrong.


docs: removed empty methods

PR Checklist
Please check to confirm your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

Can you please update the commit type to test as this doesn’t effect docs.

Can you please squash the commit into a single commit? Thanks

Hii @alan-agius4 , please refer to this https://github.com/angular/angular-cli/pull/24449


Re-added Undefined field to properties types

Re-added Undefined field to properties types.

@julioalvahash, @Guillar1 why was this required?

@MindaugasButkus Oof, that was a bad call. Didn't realize that it was removed in V2.

@MindaugasButkus I tried to re-add those undefined fields to the properties types because the GM widget uses that specific field a lot but I didn't realize the 422 error detail, that was my bad, I'll rework the logic of the properties in the widget to not use that field anymore.

@MindaugasButkus I tried to re-add those undefined fields to the properties types because the GM widget uses that specific field a lot but I didn't realize the 422 error detail, that was my bad, I'll rework the logic of the properties in the widget to not use that field anymore.

It's not the fields that will cause you trouble, it's the string value "Undefined".


consider extending template velocity dispersion grid even higher

Velocity dispersion fitting fails for<PHONE_NUMBER>4865286, most likely because the grid of the templates needs to be extended even higher.
fastspec /global/cfs/cdirs/desi/spectro/redux/fuji/healpix/sv3/bright/259/25933/redrock-sv3-bright-25933.fits \
  --targetids<PHONE_NUMBER>4865286 --debug-plots -o fastspec-39627764264865286.fits

Here's another (similar) example:
fastspec /global/cfs/cdirs/desi/spectro/redux/fuji/healpix/sv3/bright/260/26004/redrock-sv3-bright-26004.fits \
  --targetids<PHONE_NUMBER>1855435 --debug-plots -o fastspec-39627776201855435.fits

With the ssp-templates (misnamed) branch and a velocity dispersion grid which extends to 475 km/s,<PHONE_NUMBER>4865286 now looks OK:

Done in #166.


[Viewer Desktop Go] Remove dist/assets and build/bin exe files

Removed the build/bin and frontend/dist directories as it is standard practice to not include them.
Removed the exe build files.

@aked21 Please merge I've reviewed


Better session failure

This allows a failed session to see an unsaved snapshot along with error details.

Merge after #137

Fixes #136


Programming exercises: Add client-side support for feedback suggestions

Note
The hardcoded example feedback suggestions will need to be removed before merging.

Checklist
General

[X] I tested all changes and their related features with all corresponding user types on a test server.
[X] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[X] I chose a title conforming to the naming conventions for pull requests.

Client

[X] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[X] I followed the coding and design guidelines.
[X] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[X] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[X] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[X] I translated all newly inserted strings into English and German.

Motivation and Context

See #7094. We want to implement programming feedback suggestions on the client. Also, this PR refactors some parts of the code editor assessment area, making it more robust.
Description

This PR implements the client side of #7094 (feature proposal).
It also fixes #7102
Because of refactoring, it might fix #7033 as a byproduct (inline feedback sometimes "escapes" editor).
It resolves #6367: I slightly adjusted the spacing below the feedback cards.

Steps for Testing

Prerequisites:

1 TA
1 Programming Exercise with manual assessment enabled and with submissions to assess. The package name has to be de.athena and include BubbleSort.java because otherwise, the hardcoded example feedback suggestion will point nowhere!

Log in to Artemis
Go into the exercise dashboard and start the assessment
Play around with the feedback in the editor:

Create lots of inline feedback
Create general feedback
Switch between files rapidly
Save, Cancel, Delete feedback
Rename files
Collapse folders

Test referenced feedback suggestions: You will see hardcoded referenced feedback suggestions in BubbleSort.java and MergeSort.java. Accept or discard them. You can get it back by not saving the whole assessment and refreshing the page. It is expected that discarded feedback re-appears this way. Try editing it as well. Check that the feedback suggestion badge always appropriately represents the state of the suggestion. Also, switch around in files again.
Test unreferenced feedback suggestions the same way. There is one unreferenced feedback suggestion at the bottom.
Save the assessment. Refresh and inspect the feedback on the page. Edit it again.
Look at the feedback from the perspective of a student.
Check that the feedback suggestions don't influence the overall score until they are accepted.
Collapse the src/de/athena folder. The feedback suggestions badge should sum up the badges of all files and show up next to the collapsed folder.

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Client

Class/File
Line Coverage
Confirmation (assert/expect)

assessment-detail.component.ts
not found (deleted)

athena.service.ts

Doesn't make sense to test right now, will undergo large changes (also because of #6861)

assessment-correction-round-badge.component.ts
not found (renamed)

unreferenced-feedback-detail.component.ts
100%
✅

feedback.model.ts
96.58%
✅

code-editor-tutor-assessment-container.component.ts
86.19%
✅ up from ~85%

code-editor-tutor-assessment-inline-feedback-suggestion.component.ts
46.15%
Contains zero logic to test, only variables

code-editor-tutor-assessment-inline-feedback.component.ts
98.11%
✅

code-editor-ace.component.ts
84.27%
✅ up from ~81%

code-editor-container.component.ts
86.36%
✅ up from ~85%

code-editor-file-browser-badge.component.ts
0%
Contains simple switch cases that are not worth testing imo

code-editor-file-browser-file.component.ts
100%
✅

code-editor-file-browser.component.ts
86.03%
✅ up from ~85%

code-editor.model.ts
100%
✅

feedback-suggestion-badge.component.ts
0%
Contains simple switch cases that are not worth testing imo

unreferenced-feedback.component.ts
96.55%
✅

textblock-feedback-editor.component.ts
92.98%
✅

Screenshots

Student view: No feedback suggestion badges:

Thanks @terlan98! I considered all your feedback 🎉

Bugfix: Make feedback work for file names containing underscore

Great catch! I fixed it and also added tests so this does not happen in the future.


HandTrackingGpu not working on Google cloud

i have literally tried everything but nothing works
i am using Tesla k80
cuda 10.2
driver 440
and it gives GL error on line 253 of gl library and i cannot seem to get it running

E1212 09:25:37.308670 13890 demo_run_graph_main_gpu.cc:257] Failed to run the graph: Graph has errors:
Calculator::Open() for node "[TfLiteInferenceCalculator_1, TfLiteInferenceCalculator with output stream: __sg0_detection_tensors]" failed: [GL_INVALID_ENUM]: An unacceptable value is specified for an enumerated argument.: glBufferData in external/org_tensorflow/tensorflow/lite/delegates/gpu/gl/gl_buffer.h:253

please follow instructions here https://github.com/google/mediapipe/issues/150#issuecomment-565135162
and see if your gpu supports opengl es 3.1+
also make sure the mesa drivers are properly installed
sudo apt-get install mesa-common-dev libegl1-mesa-dev libgles2-mesa-dev

checked everything oncce again:
lastest error :
Created TensorFlow Lite delegate for GPU.
double free or corruption (fasttop)
Aborted (core dumped)
GL version: 3.1

it is fixed
here;s how:
use this command to find your PCIbusid
nvidia-xconfig --query-gpu-info
Sample output:
`Number of GPUs: 1
GPU #0:
Name      : Tesla P4
UUID      : GPU-093812be-b975-b29b-f6e5-675a9f79f896
PCI BusID : PCI:0:4:0
Number of Display Devices: 0
`
then use this command:
sudo nvidia-xconfig --busid=PCI:0:4:0 --use-display-device=none --virtual=1280x1024`
this will change your /etc/X11/xorg.conf file
to keep using teamviewer on server, edit the screen section of the conf file as follows
Section "Screen"     Identifier  "Default Screen"     Monitor     "Configured Monitor"     Device      "Configured Video Device"     DefaultDepth 24     SubSection "Display"     Depth 24     Modes "1600x900"     EndSubSection EndSection
now you can use a server to run the MultipleHandTracking GPU

excellent!
nice work and thanks for the detailed instructions :)


Add a category to the icons

[ ] Feature
Add a categories to icons for better user experience

Description
You can include a category label and also let people suggest labels so that users can directly find and import all the icons that they need from that category. You can categorize it like how Material Design guys do it on their website

We don't have any functionality built in as of yet, and as it stands we only provide the package as a full bundle, with no custom bundling of icons available through the website. It also looks as though the tags are stored on the base repository, as opposed to the website - so will move that issue there to continue discussion.


Get local tests passing

This pull request aims to update the pennylane-qiskit plugin to the most recent master of pennylane, and have all local tests passing.
Changes to the Plugin Code
The load function in converter.py instead binds variables with requires_grad=False, instead of binding non-qml.Variable objects.
Changes to the tests
Variables passed to the device are now Pennylane tensor objects.  The plugin unwraps them before passing them into qiskit.

In converter.py,  variables with requires_grad=False get bound to the circuit.  All others get entered to the circuit during the construction of the pennylane operations.
When we are testing qiskit errors that arise things get bound, we need to test on tensors with requires_grad=False, else they won't get bound.

There remains a converter test case that uses Variable objects explicitly.
We could either update that in this PR or when we are completely deprecating Variable objects from PennyLane core.


I think  syntax is wrong

Challenge Accessing Nested Arrays in JSON has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:40.0) Gecko/20100101 Firefox/40.1.0 Waterfox/40.1.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
// Setup
var myPlants = [
  "flowers": [
      "rose",
      "tulip",
      "dandelion"
    ],

   "trees": [
      "fir",
      "pine",
      "birch"
    ]

];

// Only change code below this line

var secondTree = myPlants.trees.[1]; // Change this line

The original seed code for this challenge is fine. If you need to get back to it you can use the reset button on the side. Please visit the Help Chat if you need further assistance. Thanks and happy coding!


chore/UIDS-443 Add nightly Percy run in GitHub actions

resolves: #443
I think you need to set the PERCY_TOKEN for us since I don't seem to have the settings button.
How I came up with yaml:
https://github.com/percy/exec-action
Not sure if this will only run on develop or all branches, in CircleCI you need to specify the branches: [] when using a cron schedule unless you would like it to run on all branches, but this complained when I had this included:
https://github.com/user-interviews/ui-design-system/actions/runs/1492331919/workflow
TODO:

[x] Need PERCY_TOKEN written to project
[ ] Update yaml schedule to use cron instead on on pr
[ ] Prevent cypress videos from being made
[ ] Look into error: [3492:1123/143031.354128:ERROR:bus.cc(392)] Failed to connect to the bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")

OK running into a font awesome 401 auth issue when trying to install things for yarn. Maybe we need to do similar npm config set things like we do in CI for the rails-server?

Percy run in Githib Actions

This will get us nightly builds on develop:

Scheduled workflows run on the latest commit on the default or base branch.
https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows#scheduled-events

I'm thinking of just setting up a slack reminder around the time it completes each day to look for new changes- is there a public slack channel that you would recommend to send this reminder?

Hmm I don't know that we have a slack right now that makes sense.. all the current design system channel have all the designers included which they may not care about getting this ping. Maybe we just create a new one with me you and Jason?

Sounds good- like to keep the bots separate from people whenever possible


Add docs for limits/requests, affinity, reword PDB

Description

Fix https://github.com/kubewarden/kubewarden-controller/issues/703

Test

Additional Information
Tradeoff

Potential improvement

Thanks for the review! Added sidebar_position, the folder already had a _category.json.


app add system label

Codecov Report

Merging #369 (967ab40) into master (71b8647) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #369      +/-   ##
==========================================
- Coverage   65.35%   65.34%   -0.01%
==========================================
  Files          94       94
  Lines        6463     6467       +4
==========================================
+ Hits         4224     4226       +2
- Misses       1646     1647       +1
- Partials      593      594       +1

Impacted Files
Coverage Δ

api/application.go
73.11% <100.00%> (+0.25%)
:arrow_up:

common/string.go
80.00% <0.00%> (-20.00%)
:arrow_down:

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 71b8647...967ab40. Read the comment docs.


Merge pull request #1 from azkaban/master

sync

sync

Are you trying to sync code in your own azkaban fork? Please do it there. Thanks.


don't use the same pointer for infowindow and tooltip

when in a map you have only a tooltip enabled (and not the infowindow) the cursor is the hand. That interaction is good when you can click on the polygon but not that good for hover.
cc @saleiva

@saleiva please, this is horrible

Agree, what about simple pointer for hover?

There has been no activity on this issue for several months and that's why are closing it (automatically). If you think this still needs to be addressed please reopen it. The good news is that we are now working on the next version of CARTO.js, stay tuned.


Avoid invoking the catalyst before available

Fix https://github.com/yamill/react-native-orientation/issues/34

Thanks!

@yamill no problem! 😄


System unavailable: Multiple Docker Containers Hosted On test-equinix-ubuntu2004-armv8l-02

5 of the docker containers are failing to connect in jenkins due to an issue with the ssh key for the jenkins user..
test-docker-alpine313-aarch64-1
test-docker-alpine314-aarch64-1
test-docker-ubuntu2004-armv8l-2
test-docker-ubuntu2004-armv8l-3
test-docker-ubuntu2010-armv8l-2
This appears to be related to a docker upgrade ( possibly via a system upgrade )

These machines have now been bought back online . A remove/recreation of the docker container was required.


How do you have multiple functions in the same JIT compilation or do you have to compile them individually?

as the title says. Do you have to compile them individually and then use indirect call? or is there some other way of doing it?

For each function you have to call define_function once with the return value of declare_function for the function you want to compile. If you want to call another function you would do declare_function for this function name to get a FuncId, then declare_func_in_func with this FuncId and the Function you are currently building and finally add a call instruction with the FuncRef returned by declare_func_in_func. This works even if the function you want to call isn't compiled yet. You just need to make sure it is compiled by the time you call it.

Do I make a seperate function builder for all of them?

Yes. You can reuse the FunctionBuilderContext for slightly better performance, but each FunctionBuilder only works for a single function.

when I use the FuncRef using a builder.ins().call() I get a index out of bounds

The return value of declare_func_in_func can only be used in the Function which you passed to declare_func_in_func. I did recommend calling it right before inserting a call instruction instead of keeping it around to prevent accidentally using it with the wrong Function.

correction: The data gets completely mangled the print function prints the correct data but the data itself is inherently worng
its suppose to give me
y1 = x+1
y2 = x+2
y3 = y1+y2
when I send in 0 I get the correct value back (3)
however when I send anything above zero it sends me<PHONE_NUMBER>13572

Also how do i get a value from the return data of call()?

You can use func.dfg.inst_results(call_inst) where call_inst is the return value of creating the call instruction.

however when I send anything above zero it sends me<PHONE_NUMBER>13572

Can you show the clif ir you are using? You can print the Function right after building it using eg println!("{}", func);.

function u1:1(i64, i64) -> i64 windows_fastcall {
    sig0 = (i64, i64) system_v

block0(v0: i64, v1: i64):
    v2 = iconst.i64 0x7ff6_63df_d750
    v3 = iadd_imm v0, 1
    call_indirect sig0, v2(v3, v0)  ; v2 = 0x7ff6_63df_d750
    return v3
}

function u0:0(i64) -> i64 windows_fastcall {
    sig0 = (i64, i64) -> i64 windows_fastcall
    sig1 = (i64, i64) system_v
    fn0 = colocated u0:0 sig0

block0(v0: i64):
    v1 = iconst.i64 0x7ff6_63df_d6a0
    v2 = iadd_imm v0, 1
    v3 = iadd_imm v0, 2
    v4 = iadd v2, v3
    call_indirect sig1, v1(v4, v0)  ; v1 = 0x7ff6_63df_d6a0
    v5 = call fn0(v4, v3)
    return v4
}

im using extern "sysv64" fn(i64) -> i64. but I dont think that is the issue.

If you use extern "sysv64" you have to define the function as having the system_v calling convention, not the windows_fastcall calling convention. Never mix calling conventions. It breaks in all kinds of ways.

You shouldn't ever need to use append_results as regular cranelift user. It is meant for in place modification of clif ir, like during optimizations. What exactly is the problem you have with inst_results?

You should call inst_results on the same Function as which you added the call instruction to. (The current function)

How do I allocate a chunk of memory?

You did call a function to do so like malloc. Basically the same way you did do in C.

Is there a builtin function for malloc in cranelift IR or do I need to make like a function myself and implement it? I saw there were built in address functions that use u32 instead of i64 how do I recast them or can I just do as u32?

Is there a builtin function for malloc in cranelift IR or do I need to make like a function myself and implement it?

You can directly import the malloc from libc. If you don't define a function yourself, cranelift-jit will try to look it up using dlsym as fallback.

I saw there were built in address functions that use u32 instead of i64 how do I recast them or can I just do as u32?

You can use the pointer_type() function to get the right pointer type: https://github.com/bytecodealliance/cranelift-jit-demo/blob/4b12506d60b3cd1caa4c50b45f685f35978c697a/src/jit.rs#L388 You shouldn't do any casts as that will lose information.


ListView.builder is crashing because of AutoDispose.

Describe the bug
When using a Listview.builder to make a fetch on demand effect, the sliver will throw 'earliestUsefulChild != null': is not true when you try to scroll back to old items that are currently out side of the view port due to the fact the provider is disposed. Apparently the sliver would keep track of the old items offset as they have been laid out even though they're not showing on the screen any more which will cause the problem as the providers are disposed.
when you force the provider to keepAlive everything works as expected.
The problem with not using auto dispose is when you use a family modifier is when calling with different argument would return a new provider. so we're just creating a new provider for every page and would be hard to keep track of which provider to dispose.
To Reproduce
import 'dart:async';

import 'package:flutter/material.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';
import 'package:riverpod_annotation/riverpod_annotation.dart';
part 'main.g.dart';

@riverpod
Future<List<String>> fetchItem(FetchItemRef ref,
    {required int page, bool? te}) async {
  if (page == 4) return ['done'];
  await Future.delayed(const Duration(seconds: 3));
  return List.generate(50, (index) => '${index + page * 50}');
}

void main() {
  runApp(
    const ProviderScope(child: MaterialApp(home: MyHomePage())),
  );
}

class MyHomePage extends ConsumerWidget {
  const MyHomePage({super.key});

  @override
  Widget build(BuildContext context, WidgetRef ref) {
    return Scaffold(
      body: ListView.builder(
        itemBuilder: (context, index) {
          final page = (index ~/ 50);

          final itemIndex = index % 50;
          print(index);

          final s = ref.watch(FetchItemProvider(page: page));

          return s.when(
            data: (data) {
              if (itemIndex >= data.length) return null;
              return Text(data[itemIndex]);
            },
            error: (error, stack) => const Text('error'),
            loading: () {
              if (itemIndex != 0) return null;
              return const Text('loading');
            },
          );
        },
      ),
    );
  }
}

Expected behavior
I'm not really sure about this or if this is possible or not but we should find a way to know for sure if the widgets on lazy loading are being cached by flutter or not. and then determine to dispose the provider or not.
https://github.com/rrousselGit/riverpod/assets/95232508/6d6e0369-5825-4608-8a9a-c0bdd5fc7168

Technically, it's more of a Flutter issue.
The issue could be reproduced without Riverpod.
It's probably worth making a separate bug report on the Flutter SDK

hey @rrousselGit I apologize if I miss explained the issue. But it Appears that the problem is in my code.
loading: () {
              if (itemIndex != 0) return null;   // here
              return const Text('loading');
            },

After some digging with flutter implementation it appears that we can't remove the items after they've been laid out by RenderSliverList's preformLayout method without rebuilding the whole builder widget and adjusting the itemCount.  this works fine though for getting new data because flutter rely on the offset of the first child to keep track of the layout offset.
I'll be closing this and move it to flutter. Thank you.

If you make a Flutter issue, could you link it here? I want to follow that issue

of course. I'm just trying to reproduce the issue without riverpod with minimal code to make easy to identify the bug. Once I open a ticket I'll link it here.

I assume doing:
List<String> items = List.generate(100, (i) => 'Item $i');

ListView.builder(
  builder: (context, index) {
    if (index >= items.length) return null;
    return Text(items[index]);
  }
);

followed by:
setState(() {
  items = List.generate(50, (i) => 'Item $i');

});

would do the trick.
Assuming that before the setState, the user scrolled down.

hmm this is weird I can't actually reproduce it. Here's what I've tried:
class _MyHomePageState extends State<MyHomePage> {
  List<String> items = List.generate(300, (i) => 'Item $i');
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      floatingActionButton: FloatingActionButton(onPressed: () {
        setState(() {
          items.replaceRange(10, 30, ['Loading']);
          Future.delayed(const Duration(seconds: 10), () {
            setState(() {
              items.insertAll(10, List.generate(20, (i) => 'Item $i'));
            });
          });
        });
      }),
      body: ListView.builder(itemBuilder: (context, index) {
        if (index >= items.length) return null;
        return Text(items[index]);
      }),
    );
  }
}

The interesting thing is that the exception is thrown only when are awating the fetch the items, as soon as the provider updates the state, everything will works normally.
I'll try to create a way to simulate changing the state like provider and see what I'll get.

body: ListView.builder(itemBuilder: (context, index) {
if (index >= items.length) return null;
return Text(items[index]);
}),

I think you should do if (index >= items.length || items[index] == 'Loading') return null;

@AhmedLSayed9  You are right, I'm sorry I was so focused on the exception itself that I did not gave that much of a thought on the code. Thanks.

linking https://github.com/flutter/flutter/issues/130658

Great thanks!
Depending on the outcome I may have to offer a unique Riverpod feature for infinite lists.

That's great to hear. Thanks!

A tweak I found for this issue is to setup a caching strategy for your autodisposed provider.
You may follow Andrea Bizotto article about provider life-cycle and create your own strategy :
[https://codewithandrea.com/articles/flutter-riverpod-data-caching-providers-lifecycle/]
I my case, I used an extension (Thanks to @bizz84) for my paginated list provider like so:
`
extension AutoDisposeRefExtension on AutoDisposeRef {
/// Dispose fetched data when time is up and reset timer if the provided is
/// listen again.
///
/// [_defaultTimeout] is set to 30 seconds.
///
/// This code has been copied from Andrea Bizotto tutorial :
/// https://codewithandrea.com/articles/flutter-riverpod-data-caching-providers-lifecycle/
void cacheFor({
Duration duration = _defaultTimeout,
}) {
// get the [KeepAliveLink]
final link = keepAlive();
// a timer to be used by the callbacks below
Timer? timer;
// When the provider is destroyed, cancel the timer
onDispose(() {
  timer?.cancel();
});

// When the last listener is removed, start a timer to dispose the cached data
onCancel(() {
  timer = Timer(
    _defaultTimeout,
    () => link.close(),
  );
});

listenSelf(
  (_, next) {
    // Disposed the cached data on error
    if (next.hasError) {
      link.close();
    }
  },
);

// If the provider is listened again after it was paused, cancel the timer
onResume(() {
  timer?.cancel();
});

}
}
`
`
@riverpod
FutureOr<PaginatedList> availableLanguages(
AvailableLanguagesRef ref, {
required int page,
}) {
final repository = ref.watch(repositoryProvider);
ref.cacheFor(duration: const Duration(hours: 2));
return repository.getLanguages(page: page);
}
`
This is not ideal because it stores data that is no longer needed but is the only way I found to tweak the error.
Open to suggestions.

hey @kylianSalomon, thank you for the suggestion.
Setting up a timer to dispose the provider will only delay the problem.
As we discussed above the problem with flutter itself, I'm not sure what it is but it seems that it keeps a reference to the previously laid out children even if we use the builder constructor.

The issue is marked as P2 at the moment. Hopefully that means they'll fix it when they can.

One solution is to have index n "watch" the result of n - 1.
This means that  scrolling down won't cause out of view items to dispose. But it also means scrolling up is no-longer an issue.

One solution is to have index n "watch" the result of n - 1.

isn't this would make riverpod watch the entire list? why not just disable autoDispose?

why not just disable autoDispose?
the suggestion is different. the nth-provider will be disposed of, as soon as you scroll up
Still...

One solution is to have index n "watch" the result of n - 1.

Potentially, this might cause out-of-memory problems, right?

Although this is inconvenient, that's not quite a Riverpod bug.
But Riverpod will offer a way to better deal with infinite lists.

Great thanks! Depending on the outcome I may have to offer a unique Riverpod feature for infinite lists.

Can we use it right now?


Packaging Support

This is a tracking issue for adding support to the compiler for emitting Miden packages as one of its supported output options. Currently, this consists of the following tasks:
### Tasks
- [ ] Propose and finalize a specification for the package format
- [ ] Identify required vs optional package contents
- [ ] Implement core packaging functionality as a separate library (reading metadata/manifest, packing/unpacking/repacking)
- [ ] Add support for emitting minimum viable package from backend for libraries/programs
- [ ] Add support for consuming packages in the frontend for the purpose of dependency management

I'll happily take care of the last task (consuming packages in the frontend).


Broaden buildifier magic comment detection

buildifier supports many different magic comments.  The only one supported by
bazel-mode right now is "keep sorted", but there are more known to
buildifier, including:

do not sort
@unused
@unsorted-dict-items
buildifier: leave-alone
buildifier: disable=$SOME_WARNING
buildozer: disable=$SOME_WARNING

This change replaces the existing static magic comment detection with a regexp
that encompasses the above plus the existing "keep sorted".  The regexp is
stored as a private variable.
Tests are also provided.
My elisp isn't super strong, so any pointers would be greatly appreciated. :)

Thanks! Do you happen to have a reference which magic comments are supported that you could refer to in the code and here?

Thanks! Do you happen to have a reference which magic comments are supported that you could refer to in the code and here?

No problem!  I'm sitting on a few other changes, just finally recently got around to making this one ready.  I'll try to get the others into shape in the next week or two.
As for where buildifier documents its magic comments:

Warning suppression comments: (e.g. buildifier: disable=$WARNING) is documented here.
The others, like keep sorted, do not sort, and buildifier: leave-alone don't look to be documented, but are mentioned in the source code here and have tests

Empirically, buildifier doesn't seem to care where magic comments are on a line, so they probably shouldn't be split like this. It looks like fill-nobreak-predicate could be used for this purpose.
Thoughts? I can look into this, if you'd like. I think I'll have some time next week to try it out.

(Sorry, didn't mean to resolve this conversation, so posting the quote here).
I'm not really feeling strongly about this. I guess in most cases the magic comments will stand on their own, so maybe it makes sense to break before and after them?

I haven't seem them used much outside of my employer's internal repo, which uses them by themselves for the most part.

This might be better as a follow-up PR, though, if you decide to change the behavior.

Yeah, I was definitely not going to do this in this PR.  Maybe later, perhaps.

Thanks!


joshuto: init at 0.9.0

Motivation for this change
joshuto is a ranger-like terminal file manager written in Rust
https://github.com/kamiyaa/joshuto
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137201 run on x86_64-linux 1

1 package built:

joshuto


kali linux can be installed on lima？

Description
Whether kali linux can be installed on lima,  or how to do?

Anything supporting Openstack / generic cloud should work, but Kali only supplies a .tar.xz with raw - no qcow2 image.

https://www.kali.org/get-kali/#kali-cloud

They also support Vagrant and VirtualBox, so the best would be to request the Kali distro to support QEMU and Lima ?

https://www.kali.org/get-kali/#kali-virtual-machines

But the distribution is quite specific, and the download is very big, so one could wonder what the use case would be...
Probably better to just install it as a VM, in UTM: https://www.kali.org/docs/virtualization/install-utm-guest-vm/ ?


Xgression return function ready to use in python

The current result of Xgression is a string of equations like "y=mx+c" if the user wants to test or use Xgression elsewhere
user will have to implement the function manually. so I think Xgression should have some kind of algorithm to convert computational tree to a python function

like
`
variables = {"a":[1,2,3,4,5],"b":[2,4,6,8,10]}
y_true = [3,6,9,12,15]
model = Xgression(variables,y_true)
y_pred = model.eval({"a":3,"b":100}) # THIS IS WHAT I NEED
`

Two methods avaliable

use traversal to find every user's variable node in the computational tree collect the reference in list then change its value
modify eval in the computational tree to return sympy variable instead then use subs to insert the variable value

I think the second is faster and easier


Size of K placeholder in activity of collection

Is it intended to look like this?
cc @exezbcz

https://canary.kodadot.xyz/ksm/collection/dac5c7f54029d0e73c-XCHIMPZ/activity

oh and actually doesn't show item picture properly as after clicking it loads proper item

Is it intended to look like this?

guess sir
answer: no :D

@exezbcz
the placeholders supplied here
https://github.com/kodadot/nft-gallery/pull/7698#issuecomment-1768981046
have no padding.
please if you could supply the placeholders as they should appear

related #7926

@daiagi sure sir
same as with the old zip file, there are multiple levels of compression
darkmodeplaceholderpadding.zip
Whiteplaceholderpadding.zip
thanks!


TypeError [ERR_INVALID_ARG_TYPE]

TypeError [ERR_INVALID_ARG_TYPE]: The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received an instance of Object
TypeError [ERR_INVALID_ARG_TYPE]: The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received an instance of Object     at new NodeError (internal/errors.js:322:7)     at Function.from (buffer.js:334:9)     at new Buffer (buffer.js:290:17)     at /root/syncV3-testing/node_modules/jsforce/lib/api/metadata.js:776:27     at /root/syncV3-testing/node_modules/jsforce/lib/promise.js:72:9     at processTicksAndRejections (internal/process/task_queues.js:77:11) {   code: 'ERR_INVALID_ARG_TYPE' }
Sometimes when calling metadata.retrieve(anyParams).stream() above error occurred. After short investigation the root cause was found.
Extra error check should be done here
With current setup if error is thrown it became unreachable for any catch block or .on('error', (error) => {}) error listener on stream it self.
Please consider to look at the issue

Facing the same issue

Yes, also experiencing the same.

The same for me, please fix it.

could someone using jsforce v3 provide a repro or debug logs when this happens?
that error means result.zipFilewasn't a string, could be the server that returns something jsforce isn't parsing correctly.

nvm, closing as dup of https://github.com/jsforce/jsforce/issues/1302


Option to set additional stream features

XMPP-IM and XEPs define that when server support non-mandatory features, these should be signalled as a stream feature. I have such feature implemented (urn:xmpp:features:pre-approval) and would like a way of adding it to stream-features of incoming streams.

So far I ended up with monkey-patching Session object:
  session.sendFeatures = function () {
    if (this.authenticated) {
      const features = new xmpp.Element('stream:features', {
        xmlns: NS_STREAMS,
        'xmlns:stream': NS_STREAMS
      })
      features.c('bind', { xmlns: NS_BIND })
      features.c('session', { xmlns: NS_SESSION })
      for (let name of Object.keys(streamFeatures)) {
        features.c(name, { xmlns: streamFeatures[name] })
      }
      this.send(features)
    } else {
      this.constructor.prototype.sendFeatures.apply(this)
    }
  }

node-xmpp rewrite adds a mechanism to register custom stream features. node-xmpp-server won't be part of the initial rewrite release but we'll be able to reuse it.


Add Dirac-delta support

This PR adds Dirac-delta support via an ephemeral DiracDelta Op.  The relevant lifting and removal rewrites have been added as well.

Why not make it a RandomVariable? thay way the default transforms would work out of the box, and we can still have the remove useless rewrite.

RandomVariable interface simply doesn't fit this type of "distribution" well enough, so there would be a bit more boilerplate to that.  Plus, it would only afford a minor convenience as a result.  Also, this type of Op has more simplifying properties that are easy to express directly, whereas RandomVariable doesn't, so this Op would be unnecessarily restricted by what was available to RandomVariables—unless we implemented its special properties (e.g. nearly every Op can be lifted through it), but then we're already doing more work than the current approach.

Yeah nevermind. I was thinking only about the Dimshuffle and broadcast lifts, but here you are doing it for every unary operator as well.


Cross tenant replication property update for new accounts

The change is related to this advisory - https://techcommunity.microsoft.com/t5/azure-storage-blog/azure-storage-updating-some-default-security-settings-on-new/ba-p/3819554

@normesta
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Looks good. Thx!
#sign-off


Update alluxio image with the latest fix

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #366 (e1c6ba5) into master (3c1b06b) will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##           master     #366   +/-   ##
=======================================
  Coverage   17.72%   17.72%
=======================================
  Files          52       52
  Lines        2211     2211
=======================================
  Hits          392      392
  Misses       1780     1780
  Partials       39       39

Impacted Files
Coverage Δ

pkg/ddc/alluxio/transform.go
0.00% <0.00%> (ø)

pkg/ddc/alluxio/transform_fuse.go
68.57% <100.00%> (ø)

pkg/ddc/alluxio/operations/local.go
92.30% <0.00%> (ø)

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 3c1b06b...e1c6ba5. Read the comment docs.


Support not full graph for VRP (performance reason)

I build routes for merchandisers who walks between shopping points and check the goods on the shelves. The points are pretty dense (it can be just a few minutes to walk between) but there are a lot of them (the entire city). The merchandiser should spend 20 minutes in a single point and has 6 hours working day. So logically there is no sense even to try to add points that are on opposite sides of the city into the single route. But or-tools tries to use all the points that slows down the process a lot.
As a solution, we can use a distance hash instead of the distance matrix. Something like: {node1: {node2: distance, node20: distance}, node2: {node1: distance, node3: distance}}. Or talking in words of or-tool we can have a callback that would receive a node and return back an array of nodes that can directly communicate with the given node. Also, I guess it would be useful for some algorithms to know what nodes are closest to the given node (sorted array) to take the closest one instead of checking all possible nodes.

Maybe you can use some heuristic to calculate the initial solution quickly and then use something like guided local search to improve the solution.
Or you can cluster the points and then and then solve the problem for each individual cluster.

Currently clustering is the best solution I found but it has problems on cluster borders. The pink arrow (in the bottom) shows such case. The route from the left cluster goes up instead of collecting closest points from the central cluster.

I can build an initial route but any way the guided local search must check all the nodes to find the closest that is O(n**2) complexity. And as I know (I might be wrong) the guided local search doesn't really consider only closest nodes. It takes all possible nodes.


chore: bump got to ^11.8.5

Description of changes

Description of how you validated changes
yarn test
Checklist

[ ] PR description included
[x] yarn test passes

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

Codecov Report

Merging #10803 (188646f) into dev (23bf15d) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev   #10803   +/-   ##
=======================================
  Coverage   47.47%   47.47%
=======================================
  Files         673      673
  Lines       33206    33206
  Branches     6706     6706
=======================================
+ Hits        15764    15765    +1
+ Misses      15758    15757    -1
  Partials     1684     1684

Impacted Files
Coverage Δ

...li/src/domain/amplify-usageData/getUsageDataUrl.ts
100.00% <0.00%> (+12.50%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


feat: Create ECS cluster

Issue #, if available:
GALI-1540
Description of changes:
Use CDK to set up an ECS cluster that runs a dummy task
These ECS instances should be able to accept custom network
Use the values specified in BYON design for setting up auto-scaling
Checklist:

[x] Have you successfully deployed to an AWS account with your changes?

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

We should start considering refactoring into separate files per component type.

Does it make sense to pull out any of the hard coded values into a config / variables? Otherwise 👍

Good question. We will need to fiddle the hardcoded values during our performance testing cycle anyways. At that time, we can test and decide the default values for these and move them in the user config file.
However some of them are inter-dependent/block compatibility if incorrectly configured/potentially impact application response time. Just a heads up.


WIP - jade: use non-blocking socket for the jade simulator

When enumerating and looking for the Jade simulator, set the socket to non-blocking mode, as some platforms (Windows?) will wait forever for a response if the simulator is not running.
For #630

Successfully tested - see #630.
Changes sill be included in #629, so closing this item.


Add inpaint playground

TODO

Creating and testing some interesting Inpainting Demo

The awesome generated image by DeepFloyd

Prompt generated by GPT-4

In the heart of the wilderness, an enchanting forest reveals itself. Towering trees, their trunks sturdy and thick, reach skyward, their leafy canopies forming a natural cathedral. Verdant moss clings to bark, and tendrils of ivy climb ambitiously towards the sun-dappled treetops. The forest floor is a tapestry of fallen leaves, sprinkled with delicate wildflowers. The soft chatter of wildlife resonates, while a nearby brook babbles, its clear waters winking in the dappled light. Sunrays filter through the foliage, casting an emerald glow that dances on the woodland floor. Amidst the tranquility, the forest teems with life, whispering ancient secrets on the breeze.


Kantui refactor

Changes in this pull request
Refactor:

IO, UI and main thread split in separate modules
New views added (help and describe)
Config module expanded to be able to parse complex keyboard mappings

IO

Logs now use the gRPC call to CM and read the grpc stream, rather than reading files from the filesystem
Added "get_container_by_id" (container description call) in kanto_api

Config

Add log_tail_lines to set the number of lines in the logs view to be tail-ed
The KeyConfig struct now has complex (de)serialization behaviour: instead of simple char-s for key-bindings, Combinations with Ctrl (^) and Alt(@) are now allowed in the format "^" or "@". If no modifier is specified, the first char of the string will be taken as the key monitored for events.
Some of the keys in the default kantui_conf.toml have been remapped (e.g. quit is now Ctrl+Q)

UI

Describe screen added (see screenshot below)
Help screen added (see screenshot below)
UI becomes side-to-side scrollable if it can't reasonably be rescaled to fit the sceen.

Other

Pruned unused default features for dependencies - (debug binary reduced from 150 MB to 90mb, stripped release binary is now 3.9M)
Fixed dependencies to EXACT semvers to ensure reproducible builds in the future
Re-generated Cargo.lock to match dependency changes

Not in this PR:

Log following/watching

Tested on qemuarm64 (leda 0.0.5 in docker) and Raspberry Pi 4 64-Bit (leda 0.0.6) with large display connected via HDMI.
Running this version of kantui on serial console (terminal 1)
Looks good to me.


Defer PluginResolver

As the RestClientPostProcessor
I want to defer usage of the PluginResolver
So that I do not violate the container by causing premature bean instantiation.
Currently the RestClientPostProcessor depends on the PluginResolver, which seems to cause premature bean instantiation. Documentation states that a BeanFactoryPostProcessor should not do that.
I guess one of the effects of this is shown in the following log-line on startup:
Bean 'myBean' of type [class MyType] is not eligible for getting processed by all BeanPostProcessors (for example: not eligible for auto-proxying)

Can we defer resolving of the plugins?

Instead of looking up beans early we could write a custom plugin adapter that takes the type (class literal) and a (fallback) supplier (ie the constructor). It will wait for the first invocation in order to lookup the real plugin from the context. If it finds it, it will use it. If not we invoke the supplier and use that one. We basically take the existing code and move it to it's own class in order to delay everything.

@lukasniemeier-zalando Released as 0.12.1


Remove Redundant XCOM call

In some operators and sensors, we are pushing the value in xcom in both methods execute and execute_complete. We were doing this since there was a bug in airflow because of that the xcom was getting deleted once the task was resuming from defer state. Since the bug has been fixed in airflow so remove those unnecessary calls

OSS PR: https://github.com/apache/airflow/pull/22932


[nrf noup] Increased heap size for Wi-Fi.

Empirically it was measured that heap size needed to pass the Wi-Fi commissioning is ~25k. To be on the safe side let's set it to 28k.

I presume we are going to keep the statically allocated heap for all samples, including Wi-Fi, and the mDNS fix will save us from hard faults if the heap is still too small?

Yes, mDNS fix will just allow to crash the application and print error message instead of a hard fault.

@kkasperczyk-no I am also curious how this was measured. Won't this affect the overall stack capacity/capability?

I used dynamically allocated heap size to see what is the real usage of it. In the Zephyr's newlibc hooks implementation there is one called _sbrk. It is called every time system needs to allocate new chunk of memory, so it is possible to check the heap offset compared to begin after commissioning and estimate the size.

Won't this affect the overall stack capacity/capability?

I don't think we have much choice. We need to let application allocate this memory for heap or it will just won't work. The difference is that statically allocated memory is not freed to the system, so in theory it sometimes may be wasted, but on the other hand it reduces RAM size usage and we can predict the max usage. Both solutions have some strong and weak points, so I believe we should give it a chance and see what CI results will show as after some time.

I presume we are going to keep the statically allocated heap for all samples, including Wi-Fi, and the mDNS fix will save us from hard faults if the heap is still too small?

Yes, mDNS fix will just allow to crash the application in controlled way and print error message instead of a hard fault.

@kkasperczyk-no I am also curious how this was measured.

I used dynamically allocated heap size to see what is the real usage of it. In the Zephyr's newlibc hooks implementation there is one called _sbrk. It is called every time system needs to allocate new chunk of memory, so it is possible to check the heap offset compared to begin after commissioning and estimate the size.

Won't this affect the overall stack capacity/capability?

I don't think we have much choice. We need to let application allocate this memory for heap or it will just won't work. The difference is that statically allocated memory is not freed to the system, so in theory it sometimes may be wasted, but on the other hand it reduces RAM size usage and we can predict the max usage. Both solutions have some strong and weak points, so I believe we should give it a chance and see what CI results will show as after some time.

Yehh, of course. I am just a bit concerned because this rather a significant increase. Thanks for the explanation.


Error during installation: missing required field "selector" in io.k8s.api.apps.v1.DeploymentSpec;

Describe the bug
I tried to add the argocd-vault-plugin using an init container with the below manifest, as described at this page link:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: argocd-repo-server
spec:
  template:
    spec:
      containers:
      - name: argocd-repo-server
        volumeMounts:
        - name: custom-tools
          mountPath: /usr/local/bin/argocd-vault-plugin
          subPath: argocd-vault-plugin

        # Note: AVP config (for the secret manager, etc) can be passed in several ways. This is just one example
        # https://argocd-vault-plugin.readthedocs.io/en/stable/config/
        envFrom:
          - secretRef:
              name: argocd-vault-plugin-credentials
      volumes:
      - name: custom-tools
        emptyDir: {}
      initContainers:
      - name: download-tools
        image: alpine:3.8
        command: [sh, -c]

        # Don't forget to update this to whatever the stable release version is
        # Note the lack of the `v` prefix unlike the git tag
        env:
          - name: AVP_VERSION
            value: "1.7.0"
        args:
          - >-
            wget -O argocd-vault-plugin
            https://github.com/argoproj-labs/argocd-vault-plugin/releases/download/v${AVP_VERSION}/argocd-vault-plugin_${AVP_VERSION}_linux_amd64 &&
            chmod +x argocd-vault-plugin &&
            mv argocd-vault-plugin /custom-tools/
        volumeMounts:
          - mountPath: /custom-tools
            name: custom-tools

      # Not strictly necessary, but required for passing AVP configuration from a secret and for using Kubernetes auth to Hashicorp Vault
      automountServiceAccountToken: true

When applied to the cluster, I get this error:
error: error validating "argocd-repo-server-update.yml": error validating data: ValidationError(Deployment.spec): missing required field "selector" in io.k8s.api.apps.v1.DeploymentSpec; if you choose to ignore these errors, turn validation off with --validate=false

Reproduce the issue

Install Argo CD:

kubectl create namespace argocd
kubectl apply -n argocd -f https://raw.githubusercontent.com/argoproj/argo-cd/stable/manifests/install.yaml

Install argocd-vault-plugin using an init container.

Expected behavior
The argocd-vault-plugin should be installed within the argocd-repo-server pod.
Additional context
kubectl version
server: 1.21.12
client: 1.23.1

That is not a full deployment file, it is a patch. So instead of applying the file, you would do something like kubectl patch deployment argocd-repo-server ....

You can find full kustomize manifests here https://github.com/argoproj-labs/argocd-vault-plugin/tree/main/manifests for convenience


Don't use a hard-coded package version in swr...

Doing so is causing VS builds to not pick up the new packages.

FYI @srivatsn @natidea
The other project system .swr files already use this.  It looks like Editors was an oddball that never got updated.


apex module not starting

Description
I have setup the M.2 Dual TPU on my Proxmox Host and forwarded the pcie device to my UbuntuVM where Homeassistant is running. I have also used this guide: https://www.reddit.com/r/Proxmox/comments/n34f8q/proxmox_vm_ubuntu_2004_frigate_2x_google_coral_tpu/
The issue is that the apex module does not load inside the UbuntuVM like it should:
***@Homeassistant:~$ sudo lsmod | grep apex
***@Homeassistant:~$ sudo modprobe apex
modprobe: ERROR: could not insert 'apex': Unknown symbol in module, or unknown parameter (see dmesg)

The modules are properly installed:
***@Homeassistant:~$ sudo modinfo apex
filename:       /lib/modules/4.19.0-18-cloud-amd64/updates/dkms/apex.ko
author:         John Joseph<EMAIL_ADDRESS>license:        GPL v2
version:        1.2
description:    Google Apex driver
srcversion:     37A768932BDAF006DA92150
alias:          pci:v00001AC1d0000089Asv*sd*bc*sc*i*
depends:        gasket
retpoline:      Y
name:           apex
vermagic:       4.19.0-18-cloud-amd64 SMP mod_unload modversions
parm:           allow_power_save:int
parm:           allow_sw_clock_gating:int
parm:           allow_hw_clock_gating:int
parm:           bypass_top_level:int
parm:           trip_point0_temp:int
parm:           trip_point1_temp:int
parm:           trip_point2_temp:int
parm:           hw_temp_warn1:int
parm:           hw_temp_warn2:int
parm:           hw_temp_warn1_en:bool
parm:           hw_temp_warn2_en:bool
parm:           temp_poll_interval:int

***@Homeassistant:~$ sudo modinfo gasket
filename:       /lib/modules/4.19.0-18-cloud-amd64/updates/dkms/gasket.ko
author:         Rob Springer<EMAIL_ADDRESS>license:        GPL v2
version:        1.1.4
description:    Google Gasket driver framework
srcversion:     A7A1F644810CD4FF575450E
depends:
retpoline:      Y
name:           gasket
vermagic:       4.19.0-18-cloud-amd64 SMP mod_unload modversions
parm:           dma_bit_mask:int

more info:
***@Homeassistant:~$ sudo dpkg -l | grep edgetpu
ii  libedgetpu1-std:amd64               16.0                                amd64        Support library for Edge TPU

***@Homeassistant:~$ sudo apt install gasket-dkms
Reading package lists... Done
Building dependency tree
Reading state information... Done
gasket-dkms is already the newest version (1.0-17).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

***@Homeassistant:~$ sudo apt install libedgetpu1-std
Reading package lists... Done
Building dependency tree
Reading state information... Done
libedgetpu1-std is already the newest version (16.0).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

***@Homeassistant:~$ lspci -vvnnkk
00:10.0 System peripheral [0880]: Device [1ac1:089a] (prog-if ff)
        Subsystem: Device [1ac1:089a]
        Physical Slot: 16
        Control: I/O- Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop- ParErr+ Stepping- SERR+ FastB2B- DisINTx-
        Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx-
        Interrupt: pin A routed to IRQ 11
        Region 0: Memory at feb10000 (64-bit, prefetchable) [size=16K]
        Region 2: Memory at fea00000 (64-bit, prefetchable) [size=1M]
        Capabilities: <access denied>
        Kernel modules: apex

***@Homeassistant:~$ ls -al /dev/apex_0
ls: cannot access '/dev/apex_0': No such file or directory

Click to expand!
Issue Type
Build/Install, Support
Operating System
Ubuntu
Coral Device
M.2 Accelerator with dual Edge TPU
Other Devices
No response
Programming Language
No response
Relevant Log Output
dmesg shows these errors:

[ 1139.020927] gasket: Unknown symbol dma_buf_attach (err -2)
[ 1139.021770] gasket: Unknown symbol dma_buf_map_attachment (err -2)
[ 1139.026094] gasket: Unknown symbol dma_buf_unmap_attachment (err -2)
[ 1139.026994] gasket: Unknown symbol dma_buf_get (err -2)
[ 1139.030177] gasket: Unknown symbol dma_buf_put (err -2)
[ 1139.031011] gasket: Unknown symbol dma_buf_detach (err -2)

I'm running into the same problem, except that I run Frigate on the Proxmox host in Docker. But still, the apex module is not loaded on the host, even though I installed the required package.
Have you found any solution? Thanks.


Incorporate senzingapi-runtime package

See https://github.com/Senzing/community-roadmap/issues/65

Pull requests:

https://github.com/Senzing/docker-web-app-demo/pull/92


Don't override VSCode's key bindings

Version of VSCode: 1.11.2
Version of the extension: 0.3.11
OS: Linux
Description:
Key bindings as Ctrl+Shift+B (run build task definied in task.json)  and Ctrl+Shift+T (restore last closed tab) are overwritten by vscode-rust.

I think it would be helpful to move these into a tasks.json file that could be initialized for the project. Then they would be easier to customize too.
I am hoping @KalitaAlexey could chime in on this. I would also be happy to work on this if any help is needed.

Actually I think these key bindings are obsolete since we have the command palette and an ability to execute a command on save.


Personal Calendar and Personal Tasks Web Parts do not work together on the same page.

I installed both of the web parts to the tenant app catalog. Along with the Library-Shared starter kit.  I continue to experience the same error message mentioned in this bug
https://github.com/microsoftgraph/microsoft-graph-toolkit/issues/794.
I have built and packaged the web parts and the issue will also persist. I am not sure what other steps to follow. Does any one have any suggestions on this. Let me know what other information I can provide

Hi, correct we are aware of this. As you mention this is due to the Microsoft Graph Tool Kit. This has been fixed in MGT, but we have yet to upgrade all of the web parts that use it. This is on the list, and thank you for reporting

This has been updated in the latest release of the starter kit. You will want to refresh your master branch and re-deploy the starterkit.pnp template. Or you could just grab the latest sppkg's for these two webparts and update those in your app catalog. Be aware that if you follow this route, you will also need to install the new mgt-spfx required sppkg, found in ./source/mgt-spfx.
Generally this issue was addressed by upgrades to mgt version 2.2.1.


MSBot clone: Default Luis service name breaks LuisGen

The default naming of Luis services with a '-' can lead to errors with LuisGen + Dispatch.
Steps:

Run msbot clone services
Get luis name:  IPABot-General
Run dispatch, get intent named "l_IPABot-General"
Run LG against dispatch model, and the intent enum is in an error state

This could be fixed by making the default naming:
let luisAppName = ${args.name}_${luisResource.name};
https://github.com/Microsoft/botbuilder-tools/blob/18b4287a0f228aee67bc319450ba2e34017182b4/packages/MSBot/src/msbot-clone-services.ts#L598

PR out: https://github.com/Microsoft/botbuilder-tools/pull/515


Enable unnecessary_raw_strings lint

Comment in this line: https://github.com/flutter/flutter/blob/c9d68ba75f73155ca3882e3a17657022f0ac1a89/analysis_options.yaml#L209
Run flutter analyze --flutter-repo --watch
Fix all warnings printed by previous command
Create PR

Can you assign me ??

@goderbauer

i would like to work on this issue @goderbauer

@viralkachhadiya I assigned #77436 to you, so giving this one to @Swayam221.

@goderbauer on https://cirrus-ci.com/task/4851332596105216, can you please guide me as to how I could fix anyone of these warnings?

Closing this because of https://github.com/flutter/flutter/pull/77678#issuecomment-793630025.


"Ed Balls" house rule

See below:

So what would that actually do? Allow anyone to play "Ed Balls." as an answer any time? (What if two people played "Ed Balls"?)

It automatically tweets @edballsmp so he can decide who wins

Am willing to consider it if anyone can confirm that @edballsmp is up for that.


chore: add Dockerfile

Hey! This is a good PR, I will merge it and if you have any suggestions you can make another PR.


fix: Renaming region tags to not conflict with documentation snippets

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Docs snippets seems to be failing just because I moved the api reference docs, but should be fine once code is submitted

Docs snippets seems to be failing just because I moved the api reference docs, but should be fine once code is submitted

Does this mean we'll have essentially the same samples in two places? wondering if we could just retire the other samples and have their home be in this repository (this is what we've done for some other libraries).

Potentially yes, but currently these samples were written with the intent for just the API reference docs, not for the cloud site documentation. Right now there are some duplicated samples, and ideally these samples are the same in both places, but I'm trying to separate them for easier indexing by the samples tracker.
I'm not really sure what other products do for this, but I know I have the same issue in python

Does this mean we'll have essentially the same samples in two places? wondering if we could just retire the other samples and have their home be in this repository (this is what we've done for some other libraries).

Potentially yes, but currently these samples were written with the intent for just the API reference docs, not for the cloud site documentation. Right now there are some duplicated samples, and ideally these samples are the same in both places, but I'm trying to separate them for easier indexing by the samples tracker.
I'm not really sure what other products do for this, but I know I have the same issue in python


Add support for truncating GeoJSONs with 3D and 4D positions when calling shapely.from_geojson()

Expected behavior and actual behavior.
Actual behavior is same as expected (according to the docs), this is just a feature request.
The GeoJSON specification allows for more than 2 coordinates per what they name "position" (latitude, latitude, altitude, etc.) I'd like for shapely to recognize these GeoJSONs but truncate the excessive dimensions, only keeping the required two (latitude and longitude). Currently, shapely.from_geojson(...) raises a ParseException, but I'd like it to have a kwarg that truncates the extra coordinates and silently parses the truncated GeoJSON instead (something like truncate_extra_dimensions and/or output_dimensions).
I have a working prototype in Python, but I'm not sure if this should be done upstream in the GEOS library instead. Regardless, let me know if I can open a draft PR with a working version of the code.
Steps to reproduce the problem.
string = """
{
    "type": "Feature",
    "properties": {
        "hash": "1a141eca608fe61c",
        "pid": "0528203038000"
    },
    "geometry": {
        "type": "Polygon",
        "coordinates": [
            [
                [-105.013269629195, 39.6815754573531, 0],
                [-105.013269882196, 39.6817395539496, 0],                                                        [-105.012736932259, 39.6817354859348, 0],
                [-105.01273722126, 39.6818986685281, 0],
                [-105.012204320322, 39.681894524513, 0],
                [-105.012204378322, 39.68173141792, 0],
                [-105.012204436323, 39.6815665013205, 0],
                [-105.012772287387, 39.6815712773379, 0],
                [-105.013269629195, 39.6815754573531, 0]
            ]
        ]
    }
}
"""
shape = shapely.from_geojson(string)

ParseException: Expected two coordinates found more than two

Operating system
Mac OS Sonoma 14.4.1
Shapely version and provenance
2.0.3 installed from PyPI using pip

As a workaround, you can call shapely.to_geojson(shapely.force_2d(geoms)).
But ideally we shouldn't have to do that. Hopefully 3D/4D coordinates support in GeoJSON is added to GEOS one day.

Actual behavior is same as expected (according to the docs)

Actually I don't think that the current behavior matches the documentation.
The docs state that the third dimension is ignored:

The following are currently unsupported:

Geometries of type LINEARRING: these are output as ‘null’.
Three-dimensional geometries: the third dimension is ignored.

However the presence of a third dimension causes the ParseException to be thrown, so it is not ignored.
IMO either the docs or the behavior should change.

I have a working prototype in Python, but I'm not sure if this should be done upstream in the GEOS library instead.

@VctorAHernndez apologies for the lack of response here, but so the from _geojson parsing function is just calling the GEOS method, so an improvement here to parse more than lon, lat would indeed have to be done upstream in GEOS itself.
Looking where the core logic of this lives, this is at https://github.com/libgeos/geos/blob/d7c51c401213d11f5dafd56596295274433379b6/src/io/GeoJSONReader.cpp#L209-L224
And I wanted to say that it looks to be straightforward to add support for 3D, but that actually was already done 2 months ago (https://github.com/libgeos/geos/pull/1150, thanks @Oreilles!), and this is included in the latest GEOS 3.13.0 release.
That does not yet support 4D (M coordinates), but the latest GeoJSON spec (https://datatracker.ietf.org/doc/html/rfc7946#section-3.1.1) also explicitly limits it up to 3D:

Implementations SHOULD NOT extend positions beyond three elements
because the semantics of extra elements are unspecified and
ambiguous.  Historically, some implementations have used a fourth
element to carry a linear referencing measure (sometimes denoted as
"M") or a numerical timestamp, but in most situations a parser will
not be able to properly interpret these values.  The interpretation
and meaning of additional elements is beyond the scope of this
specification, and additional elements MAY be ignored by parsers.

So I am not sure if GEOS would welcome any further enhancements here (although just failing with a parse exception is also not always that useful when you get data that includes a 4th value, and you have to deal with it somehow)

@krisaoe good point, we should indeed update the documentation (and certainly now that GEOS>= 3.13 supports a third dimension)


Broken link in docs: "supported interpreters"

Hi,
The "supported interpreters" link in the docs is broken:
http://docs.python-requests.org/en/latest/community/faq/#python-3-support

< kennethreitz> blame readthedocs :)

I’m not sure I understand. The faq.rst file contains the following markup:
Here's a list of `supported interpreters <interpreters>`_.

which, unsurprisingly generates the following HTML:
Here’s a list of supported interpreters.

But I can’t find a interpreters.rst file anywhere in the repository. How is that related to readthedocs? Maybe Kenneth forgot to commit a file in 47a36de?

I don't understand either - I pointed it out to him on IRC earlier today, that was his response; I'm just passing it along.

Looks like I spoke too soon :)

Fixed!


Fix typo when setting the use AO and Lightmap vertex color flags.

Fixes bug where AO wasn't used by shader due to a typo.

https://github.com/playcanvas/engine/pull/5047 :)