体验反馈

View使用【.cs+style】的方式，无法在设计视图中实时查看效果，在我看来，这会造成极大的不方便……尽管可以热重载。
对于WPF，我认为设计视图真的非常方便、有用、高效。
还有一点小问题，tab切换较为卡顿，是加载了大量图片的原因？

@LazyCuteLion Thank you for your feedback. The DesignView is much more convenient than hot reload.
This repository may present learning challenges for those unfamiliar with CustomControl-based project design. However, CustomControl-based design offers many advantages beyond the limitation of not being able to use the designer view.
For example, it enables distributed project design, distributed resource management based on CustomControl, and non-hierarchical design optimized for dependency injection. While it may sacrifice design convenience, it's highly effective from an architectural perspective. Furthermore, by properly utilizing ContentControl and ItemsControl features of the CustomControl approach, you can easily implement complex functionality with sophisticated design patterns. CustomControl is not only one of the most crucial elements of WPF but also a core concept that applies across all XAML-based platforms.
For more detailed information, please refer to these articles:

https://jamesnset.dev/articles/38
@vickyqu115's article: https://www.bilibili.com/opus/997272572701704198

Regarding the tab switching lag, there could be several causes:

PC environment
Asynchronous processing during menu transitions

Screens are instantiated and registered for dependency injection when menus are called. This dynamic screen creation during menu transitions can cause lag. If you want to address this, try implementing asynchronous processing in the menu selection process.
You can also contribute to our project. Feel free to submit Pull Requests with your good ideas. Note that this repository is designed as a single framework to work across WPF, Uno Platform, WinUI3, and UWP, with cross-platform compatibility being our top priority.
We'll work on adding the guide improvements and Pull Request participation instructions you mentioned.
Thank you.

使用这样的方式可以支持设计视图（我的疑问点在于，为何要定义资源字典而不是这样）：
<james:WPFView x:Class="Leagueoflegends.Main.UI.Views.MainContent"
xmlns:james="clr-namespace:Jamesnet.Windows;assembly=Jamesnet.Windows"
……>

</james:WPFView>
cs 文件：
public partial class MainContent : WPFView // 使用“partial”
{
public MainContent()
{
DefaultStyleKey = typeof(MainContent);
this.InitializeComponent();//调用该方法
}
}

Yes, that approach works fine.
However, the main reason for using CustomControl in views like MainContent is to enable more flexible individual implementation of ControlTemplates and independent resource management.
If you prefer using UserControl instead of CustomControl, that's perfectly acceptable. It's entirely a matter of developer preference.


CI: Docker build image is >400 mb

Description

Docker builds image is whooping 400mb.
Expected Behavior

Reduce the docker image size by at least 8 times.

@pranav-bhatt will you confirm whether this issue remains?

One way to make image thinner is to use [distroless image] for istio (https://hub.docker.com/layers/proxyv2/istio/proxyv2/1.5.0-distroless/images/sha256-29292a1689e8ee5bec7efb6437b012b5dc8d04cea6da6059088a96ff1da7373f?context=explore)
Most significant size reduce would be to change go image into alpine version https://hub.docker.com/layers/golang/library/golang/1.13.5-alpine3.11/images/sha256-53dd41aad34f015cd945e924847ce5f7ce6f5114ecf5c284a2b7242b9464f046?context=explore


Give typesafety for data-*

We don't need to add this to our package, but we can tell our users how to ensure they have typesafety and autocomplete for our data attributes.
declare global {
  namespace React.JSX {
    interface IntrinsicAttributes {
      'data-ds-color-mode'?: 'dark' | 'light'
    }
  }
  namespace React {
    interface HTMLAttributes<T> {
      'data-ds-color-mode'?: 'dark' | 'light';
    }
  }
}

We already do this for popover (popovertarget), and this is just something we need to add to our docs
Same goes for size:
Since all components (and HTML elements) will support data-size and data-color, we should expose this as props also in React and thus avoiding confusion on when size is available and when it is not.
We can however provide type safety by creating a shared props definition, which all components can extend:
/** Recommended sizes */
'data-size'?: 'sm' | 'md' | 'xl';

and those components with more sizes (Avatar, Spinner, Heading) can extend with more sizes in data-size

My suggestion:

Add a file types.d.ts with any types that are not dependent on tokens/themes to @digdir/designsystemet-react which can be imported in the consuming library
Output a types.d.ts from the CLI build command with any types that depend on token values / modes etc, which then becomes a part of @digdir/designsystemet-theme or an organization's own theme package
Add documentation on how/where to import these, and why, to README.md

Can be imported in an entry point file, or added to "types" array in tsconfig.json

I'm handling this as part of #2703.


Provide a new page to view other user details when logged in as super user

I believe this has been addressed.
I like the new SU login mechanism and that I can go to user details to see their name / email etc.


Added changes to CONTRIBUTING.md

Updated the CONTRIBUTING.md file for better organization and clarity

My goal here was to clear up the errors in the contributing.md file.


Tutorial cleanup: next/previous chapter should be routerlinks

And should route to the chapter heading of the next chapter with an anchor link (on small screens, the table of contents is rendered above the content instead of to the left)


New Merge Join planner

This should fix https://github.com/dolthub/dolt/issues/6020 once finished, and then some.
The killer new feature in this new join planner is "Multi-Column Merge Joins", that is, merges where the comparison used for merges incorporates multiple filter conditions. This allows us to, in some cases, choose a much more selective index for merge joins. This improves both memory usage and performance because there will be fewer cases where the join iterator needs to keep multiple secondary rows in memory and cross-join them with multiple primary rows.
The algorithm goes like this:

For each index on the left table:

Compute the max set of filter expressions that match that index
Check to see if any indexes on the right table match that same set of filters in the same order.

If so, use this set of filter expressions to generate a Merge Join plan. If there are multiple expressions, we combine them into a comparison on tuples.

Remove the last filter expression and check again; repeating until the "matched filters" list is empty.

I added a test in join_planning_tests that demonstrates the potential of this new algorithm, allowing us to select a better index that otherwise allowed.
However, there still some remaining work before this can be merged.
Costing
Currently the choice of index and comparison expression does not influence costing: all merge join plans on two given tables will have the same cost. This needs to be fixed to ensure we actually use these improved indexes once we find them.
The costing function for MergeJoins also seems suspiciously low: we may currently be favoring MergeJoins even when another join type might be preferable. Attempts to improve the cost function should take this into account.
Correctness
Looking at the changed tests, there are three main ways that plan tests were affected:
Tests that previously generated MergeJoins but don't anymore
Since this PR doesn't touch costing, tests that no longer produce MergeJoins are a red flag, because it means that there may be some plans that the old implementation generates that the new one doesn't. These should be investigated before merging.
Tests that now generate MergeJoins but didn't before
Since this PR doesn't touch costing, tests that produce MergeJoins but didn't before imply that these new plans weren't being generated before. This means either the old planner was missing these plans, or these plans may not be correct and may be a bug in the new planner. These should be inspected to make sure the new plan is correct.
Tests that changed from one MergeJoin plan to another, equally useful MergeJoin plan
This was likely a consequence of the new algorithm generating the same set of plans but in a different order. Given that we currently cost all these plans equally, it's not surprising that small changes in the planner will cause churn here. These are fine.

Okay, this is finally out of draft.
The only changes since @max-hoffman last looked at it are the new coster and the updated tests.
There's one new test suite, named "merge join large and small table"
There's a lot of changes to brittle tests, but having inspected them I believe these are all acceptable changes.
Of the modified tests, they fall into a couple of categories:

The new coster typically increases the cost for merge joins, because the previous cost was too low. This is especially apparent in cases where neither indexed lookup is provably unique (max one row per key), because in those cases we may have O(N*M) behavior, where N and M are the sizes of the child tables.
The new coster allows merges that weren't previously possible, so some tests use those new merges.
The old coster ignored things like the choice of index for the merge, instead just choosing whichever merge plan it happened to encounter first. The new merge planner may generate plans in a different order, changing which plan is selected. Or, because we now take things like index selection into account, one choice of index is these tests gets replaced with a better one.


LATERAL subquery docs

Fixes #5990.
@chrisseto Adding you for technical review

Added a section on LATERAL subqueries to the "Subqueries" documentation page.
Added a section on LATERAL subqueries to the "Join Expressions" documentation page. This section really only links to the main section, on the "Subqueries" page.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/7b7fcb72469b15778b5618fc70e6e9c58e3b3207/
Edited pages:

v20.1/joins.md
v20.1/subqueries.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/954c5aaeaa7b675db38573e2930578f12260b367/
Edited pages:

v20.1/joins.md
v20.1/subqueries.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/56035acc9fcec10b4392f06d706540a12886ca55/
Edited pages:

v20.1/joins.md
v20.1/subqueries.md

@ericharmeling should the JOIN syntax chart include the LATERAL keyword?
This is awesome! The explanation is extremely understandable and properly motivates the feature.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/ccb215fda4d66b1d9c5ea04ce3a0298030d2ee18/
Edited pages:

v20.1/joins.md
v20.1/subqueries.md


CLN: use str instead of os.fsdecode for converting Path

See https://github.com/pydata/pydata-sphinx-theme/pull/565#discussion_r786310474
This is a bit cleaner IMO, and avoids anyone in the future to wonder why this is using os.fsdecode

as long as this doesn't introduce a significant downside

As far as I understand now, this is exactly equivalent in our case (os.fsdecode will call __fspath__ on the object, and if that's a string just return it, and pathlib.Path.__fspath__ is exactly the same as calling str(..) on it (see the link in https://github.com/pydata/pydata-sphinx-theme/pull/565#discussion_r786310474)


Add PIP package creation to the CMake build.

Since the setup.py is mostly identical, is it possible to somehow reuse the existing one?

@martinwicke I'd rather keep this simplified one as a temporary measure (until the two PIP packages are at parity) than refactor the main setup.py with a bunch of conditional flags that we're eventually not going to need. Does that make sense?

Makes sense to me. Just wanted to check that this is deliberate.
On Sat, Sep 10, 2016 at 09:21 Derek Murray<EMAIL_ADDRESS>wrote:

@martinwicke https://github.com/martinwicke I'd rather keep this
simplified one as a temporary measure (until the two PIP packages are at
parity) than refactor the main setup.py with a bunch of conditional flags
that we're eventually not going to need. Does that make sense?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/4310#issuecomment-246120673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjO_Tpyl6t0I4jWCXd5z6AEyjL0pV4Cks5qotkIgaJpZM4J5ju4
.

I think we need a new build for this in jenkins, the existing pull-requests-cmake would probably not cut it.
Do we have any scripts for building the pip package for this build?
What is the command to build it?

Found it, we will first merge this, and refer to this one through a script similar to what we have here:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/pip_package/build_pip_package.sh


New package: SparksCommunications.Relative version 0.95

Description

Resolve https://github.com/microsoft/winget-pkgs/issues/127258

Checks

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Notes
This InnoSetup installer wizard does not add the installation path into $PATH environment variables. So it can't be used directly as Command line usage.

Command line usage
If you put the installation directory on your PATH, you can use the program from the command line:
relative targetPath shortcutPath

For example:
relative a/b/c/someTarget.txt a/b/d/myshortcut

The relative shortcut path in this case would turn out to be:
../c/someTarget.txt

According to Usage, validation process can only be done manually and graphically.

Usage
Right-click on any file or folder. From the context menu, select "Create relative shortcut...". A file browsing window will appear. Navigate to the location where you want to create your shortcut and click "Ok".

@wingetbot waivers Add Validation-Executable-Error


Add docs

I managed to set this stuff up, but it still isn't complete. The iptables rules aren't persistant and I didn't compile JDK/JVM from source. Add instructions?

I agree, there should be more documentation on how to set this up

Oh yeah, you should probably also make the restart every 24h optional, and instead just run stop.sh and start.sh.

I'd say just allow people to change the restart period altogether.

Bump @kaboombot


Home Assistant: Setup failed for custom integration tesla_custom

Logger: homeassistant.setup
Source: setup.py:213
First occurred: 10:04:48 (1 occurrences)
Last logged: 10:04:48
Setup failed for custom integration tesla_custom: Unable to import component: cannot import name 'Controller' from 'teslajsonpy' (/usr/local/lib/python3.10/site-packages/teslajsonpy/init.py)

You may have to manually install teslajsonpy. It appears damaged. You can try readding the integration. This may force HA to reinstall it.

Fixed in version 2023.2.1


SD1.5 models problem

When i run with as follows:
layerdiffuser_method = (SD1.5) From Background to Foreground (need batch size 2),
base model is sd1.5,
I met such errors:
'BaseModel' object has no attribute 'diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1'

same, even with the realisticVision model

I successfully solved this problem because the author updated the SD1.5 model while also updating Forge. After merging the new commit of Forge, I can use the SD1.5 model.

I successfully solved this problem because the author updated the SD1.5 model while also updating Forge. After merging the new commit of Forge, I can use the SD1.5 model.

My repo Already up to date.

I successfully solved this problem because the author updated the SD1.5 model while also updating Forge. After merging the new commit of Forge, I can use the SD1.5 model.

My repo already up to date.
The commits I can see is different with you .

I successfully solved this problem because the author updated the SD1.5 model while also updating Forge. After merging the new commit of Forge, I can use the SD1.5 model.

My repo already up to date.
The commits I can see is different with you .

I mean the stable-diffusion-webui-forge repo

I successfully solved this problem because the author updated the SD1.5 model while also updating Forge. After merging the new commit of Forge, I can use the SD1.5 model.

My repo already up to date.
The commits I can see is different with you .

I mean the stable-diffusion-webui-forge repo

Thank you!!! This problem solved, but I meet another problem:
RuntimeError: CUDA error: invalid configuration argument Compile withTORCH_USE_CUDA_DSAto enable device-side assertions.

i have a similar error message, though independent of the model.
but dependent of lora. if i activate one (any, really), i get this message:

Warning: field infotext in API payload not found in <modules.processing.StableDiffusionProcessingTxt2Img object at 0x00000275ECA339D0>.
[LORA] Loaded E:\ai_gh_repos\sd.webui\webui\models\Lora\arch\analogFilmPhotography_10.safetensors for BaseModel-UNet with 192 keys at weight 1.0 (skipped 0 keys)
[LORA] Loaded E:\ai_gh_repos\sd.webui\webui\models\Lora\arch\analogFilmPhotography_10.safetensors for BaseModel-CLIP with 72 keys at weight 1.0 (skipped 0 keys)
To load target model SD1ClipModel
Begin to load 1 model
Reuse 1 loaded models
[Memory Management] Current Free GPU Memory (MB) =  5977.4732303619385
[Memory Management] Model Memory (MB) =  0.0
[Memory Management] Minimal Inference Memory (MB) =  1024.0
[Memory Management] Estimated Remaining GPU Memory (MB) =  4953.4732303619385
Moving model(s) has taken 0.64 seconds
[Layer Diffusion] LayerMethod.FG_ONLY_ATTN_SD15
Reuse 1 loaded models
To load target model BaseModel
Begin to load 1 model
[Memory Management] Current Free GPU Memory (MB) =  6271.980878829956
[Memory Management] Model Memory (MB) =<PHONE_NUMBER>21875
[Memory Management] Minimal Inference Memory (MB) =  1024.0
[Memory Management] Estimated Remaining GPU Memory (MB) =  5069.573896408081
could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1.to_k.weight
could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn2.to_q.weight
could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn2.to_out.0.weight
could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn2.to_k.weight
could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1.to_q.weight
could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1.to_v.weight
could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn2.to_v.weight
could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1.to_out.0.weight
...
it repeats that lines a few times and then renders the usual image as if i hadn't activated any lora.

I have a similar error message, if activate LoRA.

i have a similar error message, though independent of the model. but dependent of lora. if i activate one (any, really), i get this message:
Warning: field infotext in API payload not found in <modules.processing.StableDiffusionProcessingTxt2Img object at 0x00000275ECA339D0>. [LORA] Loaded E:\ai_gh_repos\sd.webui\webui\models\Lora\arch\analogFilmPhotography_10.safetensors for BaseModel-UNet with 192 keys at weight 1.0 (skipped 0 keys) [LORA] Loaded E:\ai_gh_repos\sd.webui\webui\models\Lora\arch\analogFilmPhotography_10.safetensors for BaseModel-CLIP with 72 keys at weight 1.0 (skipped 0 keys) To load target model SD1ClipModel Begin to load 1 model Reuse 1 loaded models [Memory Management] Current Free GPU Memory (MB) = 5977.4732303619385 [Memory Management] Model Memory (MB) = 0.0 [Memory Management] Minimal Inference Memory (MB) = 1024.0 [Memory Management] Estimated Remaining GPU Memory (MB) = 4953.4732303619385 Moving model(s) has taken 0.64 seconds [Layer Diffusion] LayerMethod.FG_ONLY_ATTN_SD15 Reuse 1 loaded models To load target model BaseModel Begin to load 1 model [Memory Management] Current Free GPU Memory (MB) = 6271.980878829956 [Memory Management] Model Memory (MB) =<PHONE_NUMBER>21875 [Memory Management] Minimal Inference Memory (MB) = 1024.0 [Memory Management] Estimated Remaining GPU Memory (MB) = 5069.573896408081 could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1.to_k.weight could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn2.to_q.weight could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn2.to_out.0.weight could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn2.to_k.weight could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1.to_q.weight could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1.to_v.weight could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn2.to_v.weight could not patch. key doesn't exist in model: diffusion_model.input_blocks.1.1.transformer_blocks.0.attn1.to_out.0.weight ... it repeats that lines a few times and then renders the usual image as if i hadn't activated any lora.


Use case dg

Add use cases in dg

Codecov Report

Merging #266 (f6dca33) into master (e1a1546) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #266   +/-   ##
=========================================
  Coverage     65.21%   65.21%
  Complexity      604      604
=========================================
  Files            93       93
  Lines          2199     2199
  Branches        271      271
=========================================
  Hits           1434     1434
  Misses          687      687
  Partials         78       78

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


feat: Use seperate auth lambda database user (#718)

For #718 (at least partially). Changes to existing API lambda user are planned to be kept in a separate PR.

I'm planning to use it on my next branch and will clean it up then.


[BUG] GraphQL types not being generated

Using a minimum configration, I can't get the type to generate pydantic classes in the output file.
graphql.config.yml
projects:
  default:
    schema: ./schema.graphql
    extensions:
      turms:
        out_dir: api
        stylers:
          - type: turms.stylers.capitalize.CapitalizeStyler
          - type: turms.stylers.snake_case.SnakeCaseStyler
        plugins:
          - type: turms.plugins.enums.EnumsPlugin
          - type: turms.plugins.inputs.InputsPlugin
        processors:
          - type: turms.processors.black.BlackProcessor
          - type: turms.processors.isort.IsortProcessor
        scalar_definitions:
          ID: str

schema.graphql
type TestType {
  id: ID
  name: String
}

input TestInput {
  name: String
}

api/schema.py
from enum import Enum
from typing import Optional

from pydantic import BaseModel

class TestInput(BaseModel):
    name: Optional[str]

As you can see, I get enums and inputs, but no types.
More info:
    Hardware Overview:

      Model Name: MacBook Pro
      Model Identifier: MacBookPro18,2
      Model Number: Z14X000EUX/A
      Chip: Apple M1 Max
      Total Number of Cores: 10 (8 performance and 2 efficiency)
      Memory: 64 GB
      System Firmware Version: 10151.101.3
      OS Loader Version: 10151.101.3
      Serial Number (system): H62X42Y6FC
      Hardware UUID: 479FB683-C073-5D5F-BE74-F93305B60B53
      Provisioning UDID: 00006001-000240691144801E
      Activation Lock Status: Enabled

Python 3.12.3
turms 0.5.0
pydantic 1.10.16
pydantic_core 2.18.2
graphql-core 3.2.3
black 24.4.2
isort 5.13.2

@jhnnsrs I figured it would be default for the type to be generated, am I missing something?

@jakevollkommer hi there! Thanks for submitting the issue!
Generally there is an 'Objects plugin' that generates the types from the schema that you could add and that would generate models from that. M
However the main idea if you want to use turms on the client side is to go through the 'documents' driven API Generation (utilizing the fragments, operations and funcs) plugins.
Not sure if this is what you are looking for though? What would you like to achieve with the objects?
However for most usecases I would recommend checking

Thanks for the response! I actually managed to figure out to use the plugin. I had to dig through the source to eventually figure it out.
Our use case is essentially type sharing between the typescript and python parts of our backend, specifically where the python bits contain LLM operations that use pydantic classes to define LLM tools
I will check out the documents idiom as well!

Hey sorry for the long wait for the reply! Would you care for explaining where you got stuck? I am planning to "update" / "finally freaking write" the documentation in the next couple of weeks and would love some feedback, as well as your use case! :)


Tammann nb -- update to include wave color coding

This updates the Tammann notebook to be compatible with changes introduced in https://github.com/clawpack/riemann_book/pull/36.

@maojrs I believe that if you merge this the tests will pass.  Also, the exact solver and notebook should work exactly as before, except that the solver now returns the wave types (see https://github.com/clawpack/riemann_book/pull/36).

I'm not sure why Travis doesn't test this PR.

Closing this since I already merged these changes into maser.


Integrate performance enhancement changes in core API.

Requires lockstep merge with purescript-concur-core.

Thanks @ajbarber, I have merged your changes into main.
It still needs changes to match the changes I made on concur-core, so it won't compile yet, but I wanted to get the merge out of the way right now.


Why support reconnects?

Hi, i'm asking about why on any message sent or read to the Discord IPC, drpc will attempt to reconnect upon failure? If the socket has experienced an error, it is most likely Discord quit. This behavior is present in the popular rich-go library, in which i'm finding an alternative for (this project is quite nice).
There is no discussions on this project, so i am asking via an issue.

Additionally, i would like to submit a few features to this project, and i want your thoughts on them as well:

Support Flatpak and Snap
Provide a default IPC Client similar to how net/http provides a default global top-level HTTP Client.
Remove reconnects

Hello! I'm pretty surprised that someone found my project :D
The decision to try reconnecting was exactly due to Discord's quits. For me, it seems quite a common scenario to quit Discord during the application lifecycle (https://github.com/altfoxie/am-rich-presence, for which DRPC was developed).
Thanks, your suggestions look great! The only thing, I just would prefer having the option to disable automatic reconnection (maybe drpc.New(id, disableReconnect ...bool)), rather than removing it entirely. Feel free to send a pull request =)

The decision to try reconnecting was exactly due to Discord's quits.

Could this be due to a missing connection DRPC is supposed to send? that be it a heartbeat or so?

The decision to try reconnecting was exactly due to Discord's quits.

Could this be due to a missing connection DRPC is supposed to send? that be it a heartbeat or so?

No, I mean situations where a user quits Discord themselves. Overall, connection is stable.

No, I mean situations where a user quits Discord themselves.

If this is the case, and this behavior is default in the official DiscordSDK, then i see no reason for drpc to have the ability to enable or disable it, as it is most likely a good thing to reconnect rich presence.

No, I mean situations where a user quits Discord themselves.

If this is the case, and this behavior is default in the official DiscordSDK, then i see no reason for drpc to have the ability to enable or disable it, as it is most likely a good thing to reconnect rich presence.

I don't know if DiscordSDK behaves the same. Since there are no active users of DRPC, I also can't determine if autoreconnect is useful or harmful.
Your other changes look good and I'm happy to merge them when you submit a pull request.

I've looked at the following Discord Rich Presence API implementations:

github.com/jewlexx/discord-presence
github.com/hugolgst/rich-go
github.com/sardonicism-04/discord-rich-presence
https://github.com/qwertyquerty/pypresence
https://github.com/Lachee/discord-rpc-csharp

and i've noticed (atleast as far as i know and have seen, read with basic language assumptions) that none of them attempt to reconnect, going as far as to return an 'connection closed' error.
So... maybe no reconnect?

So... maybe no reconnect?

Since reconnecting was the reason to start this project, I'm only considering the option to disable it. Otherwise, there is no point from DRPC.

there is no point from DRPC.

I am currently using the rich-go library in my software, but unfortunately it doesn't support Flatpak and doesn't seem to be updated or accept contributions that much.
This project on the other hand is designed pretty good, and i'm willing to contribute that support.


ENH: labels_near_sensors function

Another thing I'm having to do multiple times in fNIRS analyses is say which anatomical region a channel corresponds to, basically a label-mapping analogous to stc_near_sensors.
The easiest solution is just to take each channel and do a cdist-like operation to the pial surface of fsaverage to say which region is closest. However, I like the idea better of taking a stc_near_sensors approach, where for the Evoked data we use np.eye(n_chan) to obtain a (n_dense_vertices, n_channels) array. Then the "easiest" solution just gets vertex numbers from np.argmax(..., axis=0) of this array when mode='nearest', but using other modes like sum or weighted is more like a np.argsort(..., axis=0)-type operation where you get a varying number of labels per channel that it maps into, along with the weights in those labels.
That way, the labels_near_sensors can:

Have an API that matches stc_near_sensors very closely, with the addition of a labels argument (which would typically be obtained from read_labels_from_annot for example).
Have an output that also mirrors whatever stc_near_sensors does to project activation onto the cortical surface.

This is another one that could in theory go in MNE-NIRS (cc @rob-luke), but to me MNE-Python is probably a better scope since stc_near_sensors was actually designed for ECoG originally, and it's nice to have labels_near_sensors mirror stc_near_sensors in terms of namespace. To avoid crowding the mne. namespace where stc_near_sensors lives, we could have this new function in mne.label.labels_near_sensors.

The correct way to determine what anatomical structure your fNIRS channels are sensitive too is to use a photon migration simulation. There are several tools available to do this and it's usually done when designing your experiment/montage.
The tool I tend to use is the FOLD toolbox. And there are convenience functions in MNE-nirs to read the fold data and to determine what structures standard positions are sensitive to and the sensitivity to each structure. https://mne.tools/mne-nirs/generated/mne_nirs.io.fold_channel_specificity.html#mne_nirs.io.fold_channel_specificity
Your suggestion would work with non standard (10-05) positions, a large benefit. But I don't think the approach you propose would be easily accepted in a fNIRS journal, as there are established methods for doing this.
But until we have our own photon migration integration that would allow for non standard 10-05 optode positions https://github.com/mne-tools/mne-nirs/issues/405 your suggestion would be practically useful. But I think it's better suited to MNE-Python than MNE-NIRS.

Nice, I'll try that instead! We can reopen this later if someone is interested. Just for posterity, this is code I wrote to approximately accomplish this task:

    ch_weights = list()
    labels = mne.read_labels_from_annot(
        'fsaverage', parc, subjects_dir=subjects_dir)
    offsets = dict(lh=0, rh=len(stc.vertices[0]))
    label_map = -np.ones(stc.data.shape[0], int)
    for li, label in enumerate(labels):
        assert label.hemi in ('lh', 'rh')  # need to handle "both" eventually
        hi = 0 if label.hemi == 'lh' else 1
        idx = offsets[label.hemi] + np.where(
            np.in1d(stc.vertices[hi], label.vertices))[0]
        assert (label_map[idx] == -1).all()
        label_map[idx] = li
    assert (label_map >= 0).all()  # complete (enough) parcellations
    # go through channels
    for ci, d in enumerate(stc_eye.data.T):
        ch_weights.append(list())
        assert d.shape == label_map.shape
        w = np.bincount(label_map, weights=d, minlength=len(labels))
        assert np.isclose(w.sum(), d.sum())  # all mapped somewhere
        w /= w.sum()  # normalize
        order = np.argsort(w)[::-1]
        for idx in order:
            perc = w[idx] * 100
            if perc == 0:
                break
            ch_weights[-1].append(f'{labels[idx].name}({perc:0.3f}%)')

    with open(op.join(results_path, f'ch_mapping_{parc}.csv'), 'w') as fid:
        fid.write('channel,labels\n')
        for ci in range(info['nchan']):
            fid.write(f'{ci + 1},')
            fid.write(':'.join(ch_weights[ci]) + '\n')

The fold data is precomputed for standard 10-20 etc positions only. So I think we will be revisiting this discussion soon 😉

Also https://mne.tools/mne-nirs/auto_examples/general/plot_12_group_glm.html#relating-responses-to-brain-landmarks


Priority of unary - relative to **

What aspect of the language would you like to see improved?
Change operator precedence, so ** will have higher priority then unary - (and perhaps over other unary operators).
What are the reasons?
In most languages I've checked (Ruby, Python, Julia, Octave, VB6) -2**2 gives -4 (operation ** have higher priority than unary -). This is IMHO consistent with mathematical notation (-22 = 4). Excel (and VBScript\Google Sheets) returns 4 though, so there is no universal consensus.
Include practical examples to illustrate your points.
More complete list of languages (compiled from here and my own checks):

-2 power 2 gives -4: Ada, AWK, Haskell, J, Julia, langur, Lua, Maple, Mathematica / Wolfram, ExtendedPascal, R, Raku, Ruby, Python, Octave/MATLAB, BASIC(most, but not all dialects), DLang, PHP, VB.NET
-2 power 2 gives 4: ALGOL 68, F#, Factor, Nim, REXX, Crystal, VBScript/Excel, Smalltalk (but it hasn't unary operators actually)
explicitly require parenthesis in -2**2:  JavaScript

Optionally add one (or more) proposals to improve the current situation.
In Crystal, -2**2 gives 4, this is documented, but i wonder if there is any motivation behind this decision or just coincidence.
JavaScript approach looks appealing to me, but I'm not sure how it fits the rest of language.
Of course, changing it at this stage would be a breaking change for minimal profit, but i haven't found corresponding issue, so I think there should be one. If consensus is to keep current behavior, maybe this worth noting in a Differences from Ruby.

This was already discussed in the past: https://github.com/crystal-lang/crystal/pull/9684
We made some changes back then so it's unlikely we'll make them again.
There's more discussion here: https://forum.crystal-lang.org/t/exponentiation-operator-has-unexpected-behaviour/2457

Or maybe it was a separate discussion. I think given that 1.0 is out this is unlikely to change.

AFAIK there were concerns about the parser complexity because number literals like -2 can't directly be converted to a token.

Until at least Crystal 2.0 (and probably even afterwards), this should stay as-is. Changing the operator precedence post-1.0 will introduce lots of subtle breaking changes, without any useful compiler messages.

JavaScript's solution sounds great - always require parentheses. It is not subtle. To me it seems perfectly fitting for 2.0. And maybe even the parser can have a retroactive check in a way that doesn't harm the main path.

A much more softer measure we can introduce without any hassel: The formatter could apply parenethesis for this.
That would clearly express associativity in the code.

Both is best 🙂
They complement each other

I'd like to avoid having the compiler spit out excessive warnings. There's not much point in adding warnings for anything the formatter would reformat.
So whether this should trigger a warning is debatable. The code is technically not ambiguous, the behaviour is clearly specified. So why should the compiler warn about anything?


Photography Section in CoC

As its a small change and nothing but content change I should be fine doing it myself


Add required arguments for torchmetrics Accuracy

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Necessary to avoid
---> 50         self.val_accuracy = Accuracy()
     51         self.test_accuracy = Accuracy()
     52

TypeError: __new__() missing 1 required positional argument: 'task'

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Kinda!

@kampelmuehler seems we have finally fixed the ci/cd and so we are fine to proceed... :rabbit:


Fixes broken emit with useDefineForClassFields + private field

Add check for whether the class property is originated from a parameter.
Fixes #35590

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: hikarinotomadoi sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the fix @hikarinotomadoi !


Update scroll-controls.mdx

I believe the example here was missing this import. Otherwise I get
Cannot find name 'Scroll'. Did you mean 'scroll'?ts(2552)

Thanks! The import was missing but it looks good now.


Bowtie error

Hi, on running miRge on one sample, I get the following error -
`bowtie version: 1.2.3
cutadapt version: 3.1
Samtools error!: samtools, command not found. Type "samtools --version" and check the version
Use argument -psam
NOTE: Will not exit unless you encounter this error otherwise
Collecting and validating input files...
miRge3.0 will process 1 out of 1 input file(s).
Cutadapt finished for file QD1065-01_S1_L003_R1_001 in 229.2512 second(s)
Collapsing finished for file QD1065-01_S1_L003_R1_001 in 2.7011 second(s)
Matrix creation finished in 3.4465 second(s)
Data pre-processing completed in 247.1582 second(s)`
After this I get traceback calls like so -
Alignment in progress ... Traceback (most recent call last):   File "/sw/eb/sw/miRge3/0.0.9-foss-2020a-Python-3.8.2/bin/miRge3.0", line 8, in <module>     sys.exit(main())   File "/sw/eb/sw/miRge3/0.0.9-foss-2020a-Python-3.8.2/lib/python3.8/site-packages/mirge/__main__.py", line 105, in main     pdDataFrame = bwtAlign(args,pdDataFrame,workDir,ref_db)   File "/sw/eb/sw/miRge3/0.0.9-foss-2020a-Python-3.8.2/lib/python3.8/site-packages/mirge/libs/manifoldAlign.py", line 100, in bwtAlign     alignPlusParse(bwtExec, bwt_iter, pdDataFrame, args, workDir)   File "/sw/eb/sw/miRge3/0.0.9-foss-2020a-Python-3.8.2/lib/python3.8/site-packages/mirge/libs/manifoldAlign.py", line 19, in alignPlusParse     bowtie = subprocess.run(str(bwtExec), shell=True, check=True, stdout=subprocess.PIPE, text=True, stderr=subprocess.PIPE, universal_newlines=True)   File "/sw/eb/sw/Python/3.8.2-GCCcore-9.3.0/lib/python3.8/subprocess.py", line 512, in run     raise CalledProcessError(retcode, process.args, subprocess.CalledProcessError: Command 'bowtie /scratch/user/sanatbhadsavle/spermRNAseq/mouse/mouse/index.Libs/mouse_mirna_miRBase -n 0 -f --norc -S --threads 48 output_dir/miRge.2022-08-04_15-02-15/bwtInput.fasta' returned non-zero exit status 1.
What could be the problem? Thanks.

Hi @sanatbhadsavle ,
It seems like the library is not accessible, can you check the path? On the other hand, try executing the following command (taken from the error above):
bowtie /scratch/user/sanatbhadsavle/spermRNAseq/mouse/mouse/index.Libs/mouse_mirna_miRBase -n 0 -f --norc -S --threads 48 output_dir/miRge.2022-08-04_15-02-15/bwtInput.fasta > test_output.sam
Basically, the following path should exist and if you say ls, it should list several files one of which should be mouse_mirna_miRBase with ebwt extensions:
ls /scratch/user/sanatbhadsavle/spermRNAseq/mouse/mouse/index.Libs/
Let me know if this helps.
Thanks,
Arun.

Hi Arun,
My bowtie is working fine, bowtie -h spits the usual help message.
I also ran the command you mentioned and it gave out this -
`# reads processed: 757333
reads with at least one reported alignment: 3210 (0.42%)
reads that failed to align: 754123 (99.58%)
Reported 3210 alignments`
I should mention that I am running miRge with the Qiagen UMI mode, my original input command was -
miRge3.0 -s QD1065-01_S1_L003_R1_001.fastq.gz -lib /scratch/user/sanatbhadsavle/spermRNAseq/mouse/ -db miRBase -on mouse -a AACTGTAGGCACCATCAAT --qiagenumi -umi 0,12 -o output_dir -gff -udd

@sanatbhadsavle,
The command looks fine, the path to -lib seems incorrect.
For example in my laptop:
ls /mnt/d/Halushka_lab/Arun/GTF_Repeats_miRge2to3/miRge3_Lib
I should see the folder named mouse and then
ls /mnt/d/Halushka_lab/Arun/GTF_Repeats_miRge2to3/miRge3_Lib/mouse
Should show the following sub-folders:
annotation.Libs ,  fasta.Libs , index.Libs
Similarly, when you list the folder by the following command:
ls /scratch/user/sanatbhadsavle/spermRNAseq/mouse/
you should see the folder named  mouse and then
ls /scratch/user/sanatbhadsavle/spermRNAseq/mouse/mouse/
Should show the following sub-folders:
annotation.Libs ,  fasta.Libs , index.Libs
Please let me know if the above paths exist. In case if you have doubts, then try to execute the following command:
miRge3.0 -s QD1065-01_S1_L003_R1_001.fastq.gz -lib /scratch/user/sanatbhadsavle/spermRNAseq  -db miRBase -on mouse -a AACTGTAGGCACCATCAAT --qiagenumi -umi 0,12 -o output_dir -gff -udd
Here I have tweaked your absolute path.
Thanks,
Arun

@arunhpatil
Hey,
Thanks for the help, that worked. I just has to take out /mouse/ from my path.

@sanatbhadsavle,
Yay. I am glad it worked out for you.
Thanks,
Arun.


Enhance pysa datetimes processing (#50)

Describe the changes
Adding some logics into pysa.py to try to process the datetime better.
Also, exception handling has been added to avoid crash of the script.
Related issue(s)
#50
How was it tested?
Before: scrapping failed at some point if pysa was defined in the yaml config file (see related issue).
Now:

[x] scrapping works
[x] dates look good (although as we don't know what is the true value, we can only admit it's relevant)
[x] the script does not crash any longer because of the try/catch instructions.

Perfect, thank you!


nodePackages.postcss-cli: fix execution of binary

Description of changes
Without this change, executing the postcss binary results in the following error:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'postcss'
Thanks to https://github.com/NixOS/nixpkgs/issues/145618#issuecomment-1051347961 I finally managed to fix this. For some reason wrapProgram is still needed though.
The test currently doesn't pass with or without the changes in this PR. I'll fix it some other time
Fixes https://github.com/NixOS/nixpkgs/issues/145618.
Also add myself as a maintainer.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Do we need to link autoprefixer, too?

Yes, if I remove it, the parts of the test after the code where it currently fails, also fail.

Yes, if I remove it, the parts of the test after the code where it currently fails, also fail.

But you don't currently link autoprefixer? You only link postcss, but the wrapProgram call adds both postcss and autoprefixer to NODE_PATH, so shouldn't we also link autoprefixer, in addition to postcss?

I don't understand nodejs packaging well enough to understand why but removing the line with autoprefixer (while not forgetting to remove the trailing backslash in the previous line) and instead adding ln -s '${self.autoprefixer}/lib/node_modules/autoprefixer' "$out/lib/node_modules/autoprefixer" doesn't work. Apply the following patch to remove the failing part of the test and try for yourself if you like and test with nix-build -A nodePackages.postcss-cli.tests:
--- a/pkgs/development/node-packages/package-tests/postcss-cli.nix
+++ b/pkgs/development/node-packages/package-tests/postcss-cli.nix
@@ -6,13 +6,6 @@ in

 runCommand "${packageName}-tests" { meta.timeout = 60; }
   ''
-    # get version of installed program and compare with package version
-    claimed_version="$(${postcss-cli}/bin/postcss --version)"
-    if [[ "$claimed_version" != "${version}" ]]; then
-      echo "Error: program version does not match package version ($claimed_version != ${version})"
-      exit 1
-    fi
-
     # run basic help command
     ${postcss-cli}/bin/postcss --help > /dev/null

/status needs_merger

/status needs_merger


Mic permission prompt blocks automatic testing

I have an application which first requests mic permission and then starts.
A problem of testing it with webgfx-tests is mic permission prompt blocks automatic testing. I needed to manually click the prompt to grant the permission.
One of the solutions would be to allow the mic in browser-level or domain-level via browser settings. But this solution has a problem with Chrome. webgfx-tests opens Chrome in private (Incognito) mode and private mode seems to reset the settings.
I found --deny-permission-prompts command line option in https://peter.sh/experiments/chromium-command-line-switches/ but it didn't seem to work.
Another solution would be booting it up without private mode by removing --incognito command line option. It works for my case so far.
So I'd like to suggest booting Chrome up without private mode. (I don't remember what the root reason why it does with private mode is.)

If I had to guess, incognito mode prevents caching to disk across different sessions. Without incognito, you're trusting Chrome to keep a consistent, favorable caching behavior for testing through major & minor updates. Could have been a simple preventative measure. However, the server can just tell browsers to not cache any of the files if that was ever a concern.

I don't remember exactly how it works without the incognito mode, I guess it will just run your default profile right? If so, we should at least create a new profile, I believe that was the initial approach, as otherwise your profile could affect the benchmark, for example if you have extensions installed or so

I guess it will just run your default profile right?

I guess so.

we should at least create a new profile, I believe that was the initial approach, as otherwise your profile could affect the benchmark, for example if you have extensions installed or so

webgfx-tests doesn't use private mode for other browsers. The other browsers have the same problem?


Set vhost in apache for advanced app

I want to setup an advanced app.
Say I have one domain e.g. www.example.com
I use virtuell hosts on my apache 2.4
I want that users can see my frontend by www.example.com and my backend by www.example.com/backend
Must I setup two different sites-avaiable with different document roots.
To simulate a domain
I create a new host in my windows host file (domain): <IP_ADDRESS>	fewo_yii2_advanced
I create a new host in my windows host file (subdomain): <IP_ADDRESS>	backend.fewo_yii2_advanced
then I create my vhosts under apache conf
<VirtualHost <IP_ADDRESS>:80>
	ServerName fewo_yii2_advanced
	ServerAlias <IP_ADDRESS>
	DocumentRoot c:/www/fewo_yii2_advanced/frontend/web
</VirtualHost>

<VirtualHost <IP_ADDRESS>:80>
	ServerName backend.fewo_yii2_advanced
	ServerAlias <IP_ADDRESS>
	DocumentRoot c:/www/fewo_yii2_advanced/backend/web
</VirtualHost>

This worked well as expected for my front- and backend, but I don't know what to do when I want to reach my backend without a subdomain under fewo_yii_advanced/backend.

https://github.com/yiisoft/yii2-app-advanced/commit/6f638632ffe309f57fc15075547ecf9afaba2101


Setting the backgroundColor of an ARTSurfaceView crashes

Setting the backgroundColor of and ARTSurfaceView crashes with the following message when     compileSdkVersion=25 and targetSdkVersion=25
java.lang.UnsupportedOperationException: TextureView doesn't support displaying a background drawable
In Nougat, you can't change the background of TextureViews it seems. This will be important for #12929 as anyone using ART will encounter problems.

@robclouth do you have any work around so far?

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.


Add Card tooltip

Summary

Added tooltip prop to card

:tada: This PR is included in version 1.55.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.55.0 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


Clear Console

There doesn't seem to be a shortcut to clear the console.
(eg: ctrl + l in chrome)

I have added this functionality.


undefined method `call' for nil:NilClass

I'm trying to run neo4j on a rails app in development and I followed all the steps that were written here http://neo4jrb.readthedocs.io/en/8.0.x/Setup.html but for some reason, after I set options in the application.rb file for a session, I get this error:
undefined method `call' for nil:NilClass
Additional information which could be helpful if relevent to your issue:
Code example (inline, gist, or repo)
end
Neo4j::Core::CypherSession::Adaptors::Base.subscribe_to_query(&logger_proc)
TYPE_SUBSCRIBERS[session_type.to_sym].call(&logger_proc)
  @neo4j_cypher_logging_registered = true
end

Runtime information:
NoMethodError (undefined method call' for nil:NilClass):   neo4j (8.0.2) lib/neo4j/railtie.rb:139:in register_neo4j_cypher_logging'
neo4j (8.0.2) lib/neo4j/railtie.rb:66:in setup!'   neo4j (8.0.2) lib/neo4j/railtie.rb:55:in block (2 levels) in class:Railtie'
neo4j (8.0.2) lib/neo4j/active_base.rb:18:in call'   neo4j (8.0.2) lib/neo4j/active_base.rb:18:in establish_session'
neo4j (8.0.2) lib/neo4j/active_base.rb:8:in current_session'   neo4j (8.0.2) lib/neo4j/shared.rb:17:in neo4j_session'
neo4j (8.0.2) lib/neo4j/active_node/labels.rb:143:in mapped_label'   neo4j (8.0.2) lib/neo4j/migrations/runner.rb:16:in initialize'
neo4j (8.0.2) lib/neo4j/migrations.rb:13:in new'   neo4j (8.0.2) lib/neo4j/migrations.rb:13:in check_for_pending_migrations!'
neo4j (8.0.2) lib/neo4j/migrations/check_pending.rb:13:in call'   actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/callbacks.rb:29:in block in call'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:88:in __run_callbacks__'   activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:778:in _run_call_callbacks'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:81:in run_callbacks'   actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/callbacks.rb:27:in call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/reloader.rb:73:in call'   actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/remote_ip.rb:78:in call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/debug_exceptions.rb:17:in call'   web-console (2.3.0) lib/web_console/middleware.rb:28:in block in call'
web-console (2.3.0) lib/web_console/middleware.rb:18:in catch'   web-console (2.3.0) lib/web_console/middleware.rb:18:in call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/show_exceptions.rb:30:in call'   railties (<IP_ADDRESS>) lib/rails/rack/logger.rb:38:in call_app'
railties (<IP_ADDRESS>) lib/rails/rack/logger.rb:20:in block in call'   activesupport (<IP_ADDRESS>) lib/active_support/tagged_logging.rb:68:in block in tagged'
activesupport (<IP_ADDRESS>) lib/active_support/tagged_logging.rb:26:in tagged'   activesupport (<IP_ADDRESS>) lib/active_support/tagged_logging.rb:68:in tagged'
railties (<IP_ADDRESS>) lib/rails/rack/logger.rb:20:in call'   actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/request_id.rb:21:in call'
rack (1.6.5) lib/rack/methodoverride.rb:22:in call'   rack (1.6.5) lib/rack/runtime.rb:18:in call'
activesupport (<IP_ADDRESS>) lib/active_support/cache/strategy/local_cache_middleware.rb:28:in call'   rack (1.6.5) lib/rack/lock.rb:17:in call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/static.rb:116:in call'   rack (1.6.5) lib/rack/sendfile.rb:113:in call'
railties (<IP_ADDRESS>) lib/rails/engine.rb:518:in call'   railties (<IP_ADDRESS>) lib/rails/application.rb:165:in call'
rack (1.6.5) lib/rack/lock.rb:17:in call'   rack (1.6.5) lib/rack/content_length.rb:15:in call'
rack (1.6.5) lib/rack/handler/webrick.rb:88:in service'   /Users/farmanp/.rbenv/versions/2.2.1/lib/ruby/2.2.0/webrick/httpserver.rb:138:in service'
/Users/farmanp/.rbenv/versions/2.2.1/lib/ruby/2.2.0/webrick/httpserver.rb:94:in run'   /Users/farmanp/.rbenv/versions/2.2.1/lib/ruby/2.2.0/webrick/server.rb:294:in block in start_thread'
neo4j gem version: 8.0.0

Have you updated to the latest versions of the gems (8.0.0 for neo4j and 7.0.2 for neo4j-core)?  This might already be fixed

Did a bundle update from neo4j-core 6.1.6 , neo4j 7.2.3, and rails 5.0.1.rc1 to neo4j-core 7.0.2, neo4j 8.0.2, and rails 5.0.1.
Now getting the same error when doing Model.all (specifically, render Model.all and Model.all.first):
irb(main):002:0> Artefact.all.first
NoMethodError: undefined method `call' for nil:NilClass
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/railtie.rb:139:in `register_neo4j_cypher_logging'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/railtie.rb:66:in `setup!'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/railtie.rb:55:in `block (2 levels) in <class:Railtie>'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_base.rb:18:in `establish_session'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_base.rb:8:in `current_session'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/model_schema.rb:35:in `model_constraints'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/model_schema.rb:131:in `each_schema_element'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/model_schema.rb:94:in `validate_model_schema!'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_base.rb:78:in `validate_model_schema!'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_base.rb:50:in `new_query'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_node/query/query_proxy.rb:305:in `_query'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_node/query/query_proxy.rb:290:in `_query_model_as'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_node/query/query_proxy.rb:113:in `base_query'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_node/query/query_proxy.rb:98:in `query_as'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_node/query/query_proxy.rb:87:in `query'
	from /usr/local/bundle/gems/neo4j-8.0.2/lib/neo4j/active_node/query/query_proxy_methods.rb:253:in `first_and_last'
... 12 levels...
	from /usr/local/bundle/gems/activesupport-5.0.1/lib/active_support/dependencies.rb:287:in `load'
	from /usr/local/bundle/gems/activesupport-5.0.1/lib/active_support/dependencies.rb:287:in `block in load'
	from /usr/local/bundle/gems/activesupport-5.0.1/lib/active_support/dependencies.rb:259:in `load_dependency'
	from /usr/local/bundle/gems/activesupport-5.0.1/lib/active_support/dependencies.rb:287:in `load'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/commands/rails.rb:6:in `call'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/command_wrapper.rb:38:in `call'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/application.rb:191:in `block in serve'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/application.rb:161:in `fork'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/application.rb:161:in `serve'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/application.rb:131:in `block in run'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/application.rb:125:in `loop'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/application.rb:125:in `run'
	from /usr/local/bundle/gems/spring-2.0.0/lib/spring/application/boot.rb:19:in `<top (required)>'
	from /usr/local/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
	from /usr/local/lib/ruby/site_ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
	from -e:1:in `<main>'

Another interesting tidbit is this:
railstest_1           | => Booting Puma
railstest_1           | => Rails 5.0.1 application starting in development on http://<IP_ADDRESS>:3000
railstest_1           | => Run `rails server -h` for more startup options
railstest_1           | Puma starting in single mode...
railstest_1           | * Version 3.6.2 (ruby 2.3.3-p222), codename: Sleepy Sunday Serenity
railstest_1           | * Min threads: 5, max threads: 5
railstest_1           | * Environment: development
railstest_1           | * Listening on tcp://<IP_ADDRESS>:3000
railstest_1           | Use Ctrl-C to stop
railstest_1           | Started GET "/" for <IP_ADDRESS> at 2017-01-03 23:31:30 +0000
railstest_1           |   ActiveRecord::SchemaMigration Load (2.3ms)  SELECT "schema_migrations".* FROM "schema_migrations"
railstest_1           | Processing by Rails::WelcomeController#index as HTML
...

It's failing trying to setup the logging.  Specifically it's choking because it's (I think) not given a valid session type  How is your app configured for Neo4j in config/application.rb, config/environments/*.rb, config/neo4j.yml, and/or using the NEO4J_URL environment variable?
Also, if you've upgraded from 7.0/6.0 to 8.0/7.0 of neo4j/neo4j-core you might need to consult this upgrade guide:
http://neo4jrb.readthedocs.io/en/8.0.x/UpgradeGuide.html

Just released 8.0.5 to make this a bit easier

Excellent. Just tested it out, and the error was this:
ArgumentError (Invalid session type: "server_db" (expected one of [:http, :bolt, :embedded])):
Changed config/neo4j.yml from
  type: server_db
  url: http://neo4j:experimental@neo4j-experimental:7474

to
  type: http
  url: http://neo4j:experimental@neo4j-experimental:7474

(And for kicks, even to type: bolt, url: bolt://neo4j:experimental@neo4j-experimental:7687)
Initial testing passed (for both http and bolt).
Hopefully this helps anyone else experiencing the same, and thanks @cheerfulstoic


Kubernetes CSI support

Investigate the use of CSI drivers for better volume stability and support across a Kubernetes cluster.

I've created a new branch to spike out a quick and dirty way to integrate Rook + Ceph with kube-rclone.
See branch for updates: 4-wip-rook-ceph-csi


aurchain/aursift: pkgbase/pkgname versioning

Some people completely ignore AUR guidelines and upload packages where pkgname deliberately matches pkgbase of an existing repository package, i.e python-xlib. That said this is a good excuse to clean up the pkgbase code in aurchain and aursift.

aurchain and aursift now solely use pkgname. This means they're less self-contained, but more flexible. It also allows me to keep the split packages code in one place, i.e. aursync. Latter is now more complex but in the end, it's a net win.


Tweak pull request branch delete ui

Move delete button to right and remove red color on it.
Remove CLI instructions when PR has been merged.

Before:

After:

After (deleted):

Closed state box is now handled similarily as well. I also changed the SVG icon from grey to red, matching the state button:

You still need to display the branch name in the button. Some users would like reuse the fork repository's branch name. It may result in a mistake deletion.

You still need to display the branch name in the button. Some users would like reuse the fork repository's branch name. It may result in a mistake deletion.

Hmm, I do prefer the short button. Maybe it is sufficient if the branch name shows only in the following deletion modal?

You still need to display the branch name in the button. Some users would like reuse the fork repository's branch name. It may result in a mistake deletion.

I do prefer the button without the branch name. Maybe it is sufficient if the branch name shows only in the following deletion modal?
Is PR Branch deletion on Gitea is more "dangerous" than on GitHub? I think GH keeps a copy of all PR branches on the target repo even after deletion, not sure about Gitea. Also, Gitea does not support undelete.

Hm, GH has a following head bar in pull request UI but Gitea hasn't. So when deleting a branch in GH, you can take a look at the branch name from the following head bar but with Gitea, the following head bar isn't implemented.

GH does show the branch name in in the dialog text. I think we could do too, but it will require a new localization string.

There is another case of "wrong" item color:

In this case, I would use yellow icon, but it is currently red.

New merged box design:

The previous message distinguished between "manually merged" and "merged" but I think this is a useless detail, so I removed it.

Ready again.

All resolved. @delvh still blocking?

.


Fellowships

It contains all the information about the fellowships.

There are lots of fellowships. I need to know what type of fellowship you want?


Make imageView public so subclasses can use it

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@deb2cb3). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #167   +/-   ##
=========================================
  Coverage          ?   93.42%
=========================================
  Files             ?       39
  Lines             ?     4107
  Branches          ?      333
=========================================
  Hits              ?     3837
  Misses            ?      269
  Partials          ?        1

Impacted Files
Coverage Δ

Source/Bricks/Image/ImageBrick.swift
78.32% <ø> (ø)

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 deb2cb3...6cabbb7. Read the comment docs.


Block message does not show !

The block message feature does not work im using chrome and it only shows a blocked page nothing else. Please help !

also the url redirect does not work too!


Mutually recursive functions

Morel should allow functions to refer to each other when declared using and. The following expression succeeds in Standard ML:
let
   fun half x = if x > 3 then (prev x) div 2 else x
   and prev x = if x > 10 then half (x - 3) else x
in
  half 14
end;                                                                                =
val it = 1 : int

but in Morel it currently gives
unbound variable or constructor: prev

Fixed in 68b52219c6bbcaf7cf211b5890e0671934ec129e.


Fix Error Parsing to Support Chain Overrides

Changes

Here is the problem being solved.

Try relaying concurrent transactions signed by the same wallet while on the same non-commercial chain, the UNPREDICTABLE_GAS Error gets triggered, and when parsing Error data the getChainByChainIdAsync gets called but it doesn't have info about custom chain override it errors which results in infinite looping of all transactions trying to process and never do

Here are the changes made.

errors.ts added check for chainOverrides

How this PR will be tested

[ ] Open the dashboard and click X. Result: Try relaying concurrent transactions signed by the same wallet on custom chain override

Output
(Example: Screenshot/GIF for UI changes, cURL output for API changes)

PR-Codex overview
The focus of this PR is to enhance error handling and configuration retrieval in the errors.ts file.
Detailed summary

Added Chain import
Added getConfig import
Updated error handling logic to use configuration overrides
Improved error message readability

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Hey there and thank you for the contribution! You're absolutely right that we aren't accounting for custom chain overrides in this error checker here.
Ideally we can abstract the chain resolver to a helper function instead since we'll likely want this "get chain from overrides or SDK" logic everywhere.
Is this actively blocking you right now? Are you able to delete the bad txs from the DB manually to unblock yourself?

Hey,
Not not acively after the fix I haven’t had any more problems m, and yes I
had to cancel manually the bad txns from DB, now everything that I am using
seems to be working fine,
Best,
Marko
On Tue, 18 Jun 2024 at 04:30, Phillip Ho @.***> wrote:

Hey there and thank you for the contribution! You're absolutely right that
we aren't accounting for custom chain overrides in this error checker here.
Ideally we can abstract the chain resolver to a helper function instead
since we'll likely want this "get chain from overrides or SDK" logic
everywhere.
Is this actively blocking you right now? Are you able to delete the bad
txs from the DB manually to unblock yourself?
—
Reply to this email directly, view it on GitHub
https://github.com/thirdweb-dev/engine/pull/546#issuecomment-2174834952,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APQG3NZIYS7HADAR3YIL4FLZH6LVXAVCNFSM6AAAAABJJ647DCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUHAZTIOJVGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Issue-77 Add support for shorhand/implicit case in augment statement …

Add support for shorhand/implicit case in augment statement when augment target is a choice

https://datatracker.ietf.org/doc/html/rfc7950#section-7.9.2

As a shorthand, the "case" statement can be omitted if the branch
contains a single "anydata", "anyxml", "choice", "container", "leaf",
"list", or "leaf-list" statement. In this case, the case node still
exists in the schema tree, and its identifier is the same as the
identifier of the child node. Schema node identifiers (Section 6.5)
MUST always explicitly include case node identifiers.

So it would be only .../y/q/q according to spec if case statement was missing.

Infact the fix i have provided in this pull request is for alllowing a shorhand case ( or implicit case) under a choice  if the augment target was a choice.   It was not about  the validity of a  augment target string  which represents a case.
As i understand https://datatracker.ietf.org/doc/html/rfc7950#section-7.9.2,  in relation with our context, it may apply as  augment target  should be explicitly have additional  case node identifier  (if the target was a case) (.../y/q/q) , since augment targets as represented as Schema node identifiers ( as per section-6.5  and section-14).
i hope you are also in sync with my above undertanding,
Do you have an any suggestion/fix  on how we can  restrict/reject  (.../y/q)  for augment target if the target is a case,  and only allow  .../y/q/q  .

I see now, your solution looks good.  It can be simplified a bit
_, targetIsChoice := target.(*Choice)

and then 2nd part:
if _, isCase := d.(*ChoiceCase); !isCase && targetIsChoice {
... rest is the same as you have it

Normally I would request a unit test as well but I'll add one after but i'll add one to TestParseSamples after this gets into master

Thank you for the response and the suggestions.
I have done suggested changes and pushed. Please check.


Which dataparser should I choose for mip-nerf 360 datasets? under the method of gaussian-splatting.

Which dataparser should I choose for mip-nerf 360 datasets? under the method of gaussian-splatting.
Thank you!

mip-nerf 360 datasets came as colmap format, you can directly load them using colmap parser.


Evaluation of SuperPoint on datasets

Sorry to bother you,
I have evaluated the pertrained models on HPatches and RDNIM datasets already.
If I want to compare the performance of SuperPoint on these datasets, how can I do it? Could you give me some details?
Thanks!

Hi, similarly as for LISRD and its export_features.py, you can do the same with the official repo of SuperPoint. You just need to adapt export_features.py to take the SuperPoint model and weights instead of the one of LISRD, and then export the descriptors and compute the metrics with the same notebooks hpatches_evaluation.ipynb and rdnim_evaluation.ipynb.

I did not provide the source code to train on aachen in this repo, because I used the code originally used in R2D2. and I did not want additional dependencies. But you can use their code to create your own Aachen dataloader.
You can then fine-tune your existing model by adding the option --resume_training_from <path to the pre-trained checkpoint> in the training command.

I did not provide the source code to train on aachen in this repo, because I used the code originally used in R2D2. and I did not want additional dependencies. But you can use their code to create your own Aachen dataloader.
You can then fine-tune your existing model by adding the option --resume_training_from <path to the pre-trained checkpoint> in the training command.

ok, thanks for your reply!

I am closing this now, feel free to reopen it if you have more questions.


Accept globs as excludes

Based on #65.
BREAKING CHANGE: Excludes of directories and filenames including globbing characters (+, @, *, ?, [, ], !, {, }) will now be interpreted as globs. This shouldn't be that relevant, except for maybe + and @

Nice!  I am super busy today, but I'll have a look at this ASAP and try to merge it by tomorrow.  (Pester me if I don't. ;)

Will do! ;)

Ok, published as v1.0.0.  Thanks again!  This is a much requested feature.


Prompted slow startup issue

VSCode Version: Code - Insiders 1.11.0-insider (4db944bc08e34fb15c5805927d9cb67470776606, 2017-03-02T07:10:18.838Z)
OS Version: Windows_NT ia32 10.0.14393
CPUs: Intel(R) Xeon(R) CPU E3-1245 v3 @ 3.40GHz (8 x 3392)
Memory (System): 7.91GB (3.76GB free)
Memory (Process): 119.72MB working set (149.63MB peak, 94.53MB private, 27.44MB shared)
Load (avg): 0, 0, 0
VM: 0%
Initial Startup: yes
Screen Reader: no
Empty Workspace: yes
Timings:

Component
Task
Time (ms)

main
start => app.isReady
30

main
app.isReady => window.loadUrl()
523

renderer
window.loadUrl() => begin to require(workbench.main.js)
266

renderer
require(workbench.main.js)
1159

renderer
create extension host => extensions onReady()
12436

renderer
restore viewlet
0

renderer
restore editor view state
1

renderer
overall workbench load
2495

main + renderer
start => extensions ready
15511

main + renderer
start => workbench ready
4488

8fc6_renderer.cpuprofile.txt
8fc6_main.cpuprofile.txt

Thanks. Looking at the traces I can spot these issues with will make sure to get fixed soon: #21963, https://github.com/Microsoft/vscode/issues/22086, and https://github.com/Microsoft/vscode/issues/22090

Thanks!


Bug-fix-Sidbar-scrollbar-auto-hides-incorrrectly

Bug : #6469 Sidbar scrollbar auto hides incorrrectly
Issue happening at : https://react.dev/reference/react/useState
Every time in the of sidebar whenver scroll comes from any Nav Link section.

Screencast from 10-12-23 01:13:41 AM IST.webm
Solution :
Added a margin -left attribute in Page.tsx file.

Screencast from 10-12-23 01:14:41 AM IST.webm
Issue Resolved :+1:
Browser and device :
Google-chrome : Version 120.0.6099.71
OS : Ubuntu 22.04.3 LTS

Fixed by https://github.com/reactjs/react.dev/pull/6586


Disable Integration with Hub in CircleCI

Since the integration with hub test fails pretty consistently I think we should just remove it from circleCI now.

I think it's worth doing this. For example, of the 6 PRs open at the moment, 2 are failing because of this test.
If we want to keep running this and working on getting it to pass reliably, I'd suggest setting it up on a schedule, to run once a day or something, so that it doesn't interrupt our dev workflow.


Introduced CLI options for both file path and language code.

Hello,
I found your script very useful. However, I found it boring to add every single file by hand, and selecting more files at once did not work.
So, I thought that it would be useful if the app could take as CLI input also the name of the video file and the language code.
This way, you can automatize the work of getting subs for many files at once:
#!/bin/bash
for entry in ./*
do
  python sbox.py --file_path "$entry" --language_code "en"
done

If you do NOT specify the file path and the language code, the application should work identically as before (showing the GUI), at least so it does on my platform.
A bit of work should be done to treat the exceptions gracefully. For example, if you specify a wrong file name (not existent) or a wrong language code, or if the language is not available for that particular file.
Also, the script could be made less verbose if used in CLI.
Thank you!

Hello!
I introduced the suggestions you proposed. I also made a small modify to the available_languages variable, to not use a global one and to make the script more clearer.
Let me know if it okay.
Thank you!

Thanks for the contribution ♥

No problem :)


installed packages don't list at "Packages" OSX

Hello guys,
I just noticed that I have lots of packages that are installed on my Atom that don't show up on the "Packages" Tab. They only appear at "Install" section after you search for them.

Hi @mamartins, usually when this happens one of your packages is throwing an error and preventing the rest of the packages from displaying.  Please see atom/settings-view#591 for more information.  Thanks!


I have to change static /js/css path after - npm run build

Its not a issue. Im looking for a idea.
I build a react app using CRA. In deployment, we have different static servers to maintain js/css files. So I have to change static URLs after the build(npm run build).
For example,
in .env.staging I set,
PUBLIC_URL=http://api-testing.example.com/
when,
env-cmd -f .env.staging npm run build
I got "http://api-testing.example.com/" this in my js/css path.
After testing I have to change this url to "http://api-prod.example.com/"
Please help me. Post your ideas.

If all sites are served from root, you could just not set PUBLIC_URL at all and everything would work fine.
If you don't use client side routing, you could set homepage to '.'. Read here.
You may be able to use webpack's __webpack_public_path__ to change this at runtime, but we don't explicitly support it so you're on your own for setting it up.


core: add validation if resources passed during cli exists

Adding validation method which checks if resources
passed during cli exists.
for example:
sourceStorageClass, destinationStorageClass, etc.
Signed-off-by: subhamkrai<EMAIL_ADDRESS>

CC @travisn


Flutter app still shows leading password character on Android even after disabling feature in settings (Security Vulnerability)

Steps to Reproduce

Run flutter create bug.
Update the files as follows:

// lib/main.dart
import 'package:flutter/material.dart';

void main() => runApp(App());

class App extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: TextField(obscureText: true),
        ),
      ),
    );
  }
}

On Android, go to settings.
Go to 'Security' or 'Security & Location'
Uncheck 'Show Passwords' (Display characters briefly as you type)
Open Flutter app bug and type in any text

Expected results:
There should be no characters displayed for any time period as password
Actual results:
Characters are displayed briefly as you type

Logs
[√] Flutter (Channel stable, 1.20.3, on Microsoft Windows [Version 10.0.10240], locale en-US)
    • Flutter version 1.20.3 at C:\flutter
    • Framework revision 216dee60c0 (6 days ago), 2020-09-01 12:24:47 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at C:\Users\Joseph\AppData\Local\Android\Sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_SDK_ROOT = C:\Users\Joseph\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[!] Android Studio (version 4.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] VS Code, 64-bit edition (version 1.48.2)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.14.0

[√] Connected device (1 available)
    • Infinix X610B (mobile) •<PHONE_NUMBER>000854 • android-arm64 • Android 9 (API 28)

! Doctor found issues in 1 category.

Hi @jjmutumi I just tried with some application from play store (native and hybrid) and I don't see any changes. So I don't think it's Flutter issue

@hamdikahloun try with password input in browser or any native Android application. The setting makes it so that when you are typing password no character is shown. Specifically Flutter apps will not conform to it.

Google chrome :

Setting:

Reproducible on the latest master 1.22.0-10.0.pre.88.

flutter doctor -v
[√] Flutter (Channel master, 1.22.0-10.0.pre.88, on Microsoft Windows [Version 10.0.19041.450], locale en-US)
    • Flutter version 1.22.0-10.0.pre.88 at C:\Development\flutter_master
    • Framework revision e10bdbbd9d (11 hours ago), 2020-09-07 11:40:03 -0700
    • Engine revision ac8b9c4c52
    • Dart version 2.10.0 (build 2.10.0-86.0.dev)

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Users\marku\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.6.5)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.6.30320.27
    • Windows 10 SDK version 10.0.18362.0

[√] Android Studio (version 4.0)
    • Android Studio at C:\Users\marku\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\193.6626763
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[√] Connected device (6 available)
    • SM G950F (mobile)           • ce12171c51cc001c03 • android-arm64  • Android 9 (API 28)
    • sdk gphone x86 arm (mobile) • emulator-5554      • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)           • windows            • windows-x64    • Microsoft Windows [Version 10.0.19041.450]
    • Web Server (web)            • web-server         • web-javascript • Flutter Tools
    • Chrome (web)                • chrome             • web-javascript • Google Chrome 85.0.4183.83
    • Edge (web)                  • edge               • web-javascript • Microsoft Edge 85.0.564.44

• No issues found!

The setting on my SM G950F

Expected vs actual

cc @AndroidNils
This was already fixed for web and desktop in https://github.com/flutter/flutter/pull/56794. That was obviously just a platform issue but here it seems like the native setting isn't implemented yet.

CC @justinmc

That's right, this setting has not been implemented in Flutter.  The final character is revealed only on iOS, Android, and Fuchsia apps (excluding browser) regardless of any setting:
https://github.com/flutter/flutter/blob/0bb1e573fce0a32c28d1bd3caf26e53405604f48/packages/flutter/lib/src/widgets/editable_text.dart#L2487-L2491
We'll have to pipe this setting through the engine and then use it in that if statement.  It seems that this setting doesn't exist on iOS.

Any news? We have multiple reports from customers for bug in our applications. But it's hard to explain that the bug is in the Flutter framework :D.

This is on my list of issues and I hope to get to it soon.  Meanwhile, if this issue is still unassigned, it's available if anyone else wants to submit a PR.
I think we have a pretty clear path forward by looking at my previous comment, as long as it's easy to read this setting in the Android engine.

Still no update on this topic?

status update from @LongCatIsLooong: just merged the engine pull request and it's one small framework pull request away

This was reverted in https://github.com/flutter/flutter/pull/98089


Mgrudzinska/transformation get capi

CAPI added, includes #400

https://github.com/Samsung/thorvg/pull/410


Product Dashboard - Bugs

Github merges do not move through the pipeline in Product dashboard
Reason: Product dashboard matches up github commits with jenkins build change set. The change set do not contain merge commits, hence the merge commits never move.
Solution: (a) Mark the merge commits as such (b) Do not put merge commits in the pipeline.

Timing issue: When github collectors run slower, but Jenkins plugin pushes build data in Hygieia, in some situation when the build data shows up earlier than the github commits, those commits do not move via the pipeline.
Solution: Create skeleton commit data using the Jenkins change sets and update them when github commits show up.

See this https://github.com/capitalone/Hygieia/pull/789

Latest code is in master.


Trigger function names collide

@tomage I just encountered an issue with django-pgtrigger that opens the possibility of some bad bugs.
Trigger names are unique for every table, but trigger function names are globally unique. Currently django-pgtrigger uses the same function and trigger name. If a user uses the same trigger name across models, they will run into an issue. This is an issue that has been present since the beginning of django-pgtrigger.
I'm going to resolve this by creating a globally unique function name for the trigger based on the model. The django-pgtrigger registry will also help ensure it is globally unique and throw an error if names collide.

@tomage closing this issue since it has been resolved


Enable or disable crafting recipes for coins

Solves #28 Setting for craftable coins

Merged manually with https://github.com/minetest-mods/maptools/commit/9915ec18528385d174a61f268a630983b34fba2c.


adjust tests

you might hate this but there are a few advantages:

no longer need to store a test file in git
tests can be run from any directory (don't need an absolute path to a test file)
added support for test coverage with pytest-cov. run pytest --cov=waifuvault tests/

leaving this in draft mode. if you like the idea, it may be good to test for yourself.

git clone
cd to directory
git checkout tests

EDIT: for some reason I'm having to use git checkout -b tests origin/tests in order to switch to the branch...

python -m venv venv
source venv/bin/activate
pip install -e .[dev]
pytest --cov=waifuvault tests/

Honestly, I'm happy with the tests the way they are now.  I want them in the repo, they go along with the code.
I also really have no interest in coverage reports.
Thanks for the work, but I want them the way they are now.

Honestly, I'm happy with the tests the way they are now. I want them in the repo, they go along with the code. I also really have no interest in coverage reports. Thanks for the work, but I want them the way they are now.

@nakedmcse  No problem! It just bothers me that the tests will fail if you don't run them from a specific directory. This was one (somewhat convoluted) solution to that. Should've opened an issue first. Closing the PR, will delete the branch


Plugin not responding over 4G

I have been using the Garadget plugin for a while, but a few months ago I noticed that my automation stopped working, shows as disabled.. After a little digging I also noticed that while outside of my home network, such as 4g or on a friends LAN, the device shows as "not responding". I have tried an update of both homebridge and the plugin as well as a clean install.
Expected/Current Behavior
Below if an output of a reboot and action of the plugin. Between the reboot completion and the output of the "test" was a test over 4g. No output appeared, only over the wifi. The 4g output should be identical.
Jun 12 20:35:38 MasterServer2 homebridge[13657]: [12/6/2018, 8:35:38 PM] Homebridge is running on port 51826.
Jun 12 20:36:30 MasterServer2 homebridge[13657]: [12/6/2018, 8:36:30 PM] [Main Garage Door] Getting current state...
Jun 12 20:36:30 MasterServer2 homebridge[13657]: [12/6/2018, 8:36:30 PM] [Main Garage Door] Get ObstructionDetected...
Jun 12 20:36:30 MasterServer2 homebridge[13657]: [12/6/2018, 8:36:30 PM] [Main Garage Door] Getting current state...
Jun 12 20:36:31 MasterServer2 homebridge[13657]: [12/6/2018, 8:36:31 PM] [Main Garage Door] Door state is closed
Jun 12 20:36:31 MasterServer2 homebridge[13657]: [12/6/2018, 8:36:31 PM] [Main Garage Door] Door state is closed
Jun 12 20:36:32 MasterServer2 homebridge[13657]: [12/6/2018, 8:36:32 PM] [Main Garage Door] Access Key good...

Versions

Homebridge: homebridge -V 0.4.43
Node:       node --version v4.3.2
OS:        Ubuntu VM on KVM/Unriad - 16.04.4 LTS - Auto starts using systemctl

Configuration
"bridge": {
        "name": "Alucards Homebridge",
        "username": "####",
        "port": 51826,
        "pin": "####"
    },

"description": "Garadget as an accessory.",
"accessories": [
    {
        "accessory": "Garadget",
        "name": "Main Garage Door",
        "cloudURL": "https://api.particle.io/v1/devices/",
        "deviceID": "######",
        "access_token": "######",
        "bypass": "0",
        "args": "{STATE}"
    }
],
"platforms": [
    {
      #########
    }
  ]
}

Link to issue report on the plugin git
[]https://github.com/xNinjasx/homebridge-garadget/issues/2

Same issue with home bridge-fibaro-hc2.


Directly modifying the URL path & search without any side effects.

Is it possible to directly modify the URL's path and search without any sideeffects in terms of reloading, refreshing, reinitialising, or affecting AngularJS in anyway. Basically a pure cosmetic effect.
I need this sometimes when I need to correct the URL without doing anything else.
This could be useful for overlays that need a directly accessible URL. Or when someone enters the wrong title for a particular ID. See Stackoverflow, when you change the title but enter just the ID, it will correct the title portion of the URL to be correct.
I know history.pushState can do this, but when I try this inside AngularJS it results in 10 digest errors.

now you should use the $locationProvider.html5Mode(mode) to enable html5 mode, then you can use history.pushState|replaceState

+1, still not working; history.replaceState for me causes the route to try to reload

I still don't see why this would be useful (assuming you rely on client side routing). Why change the URL to something that doesn't have any meaning (or a different meaning) in Angular? Wouldn't that break routing (e.g. deep-linking, forward/backword actions etc)?
Off the top of my head, it should be possible (famous last words :smile:) to implement something in $location for "cosmetically" changing the URL, but I haven't seen a compelling usecase yet (and we'll need a pretty compelling one if we are to add more complexity to one of our most complex services :wink:).

In my case, I have a temporary param that tells me where to scroll when the page loads. After it's done scrolling, I want to remove that param, without refreshing/reloading etc. transitionTo ended up being a solution for this case, though it didn't work for all cases for me.

This doesn't sounds like a good candidate for a hash, not part of the path. Also, removing it means that if the user copy-pastes the URL, they won't be taken to the same spot as they were taken by the initial URL (before the extra param was removed).
(Not sure what transitionTo refers to btw :confused: )

Sorry, meant transitionTo in UI router (I guess I mistakenly found this issue thinking it's directly related), and hash is no good because it has the same effects when I try to remove it

Hash is different in that ngRoute (and I think ui.router too) does support updating the search/hash without reloading the route.
Btw, I have submitted a PR that let's you configure a route to not reload when the URL changes (#15002).

@gkalpak Here's my use case where I need to update the url (to allow deep linking) but NOT update the browser history:
A table that stores the current sorting, filtering, and paging, so that if a user clicks a row on the table or leaves the page, but comes back to the page, they can keep their place in the table (e.g. go directly back to page 3). I've got that working fine. HOWEVER, the problem is that by updating the url on each table state change, the back button now just cycles through these different states, and that is NOT the desired behavior. The back button should go back to the previous page, not cycle through table states.
Basically we need a way to update the url for deep linking but do nothing else (e.g. do NOT update the browser history).
Here's a demo: https://apps.presencetest.com/components/tables
How can we achieve this?
Thanks!

@lukemadera, your usecase doesn't sounds related to this issue. I think $location.replace() might be what you are looking for.

Thanks for the reply @gkalpak and yes, I was able to use history.replaceState() directly to change the url while also preventing the back button from cycling through the table states.
However, here is a related use case that I believe is this issue - a set of tabs, each with their own children routes, that refer to different sub-pages (in this case steps). Again, the desire is to be able to deep link directly to a specific (sub)route (step) but going "back" should NOT cycle through tabs, it should go to the previous page (top level route). However, as before with the table example, if you click through the steps, then click the back button, instead of coming directly back to this github issue, you cycle through the steps first.
https://apps.presencetest.com/components/other/step-2
{path: 'other', component: OtherDemoComponent,
                children: [
                    { path: '', redirectTo: 'step-1', pathMatch: 'full' },
                    { path: 'step-1', component: Step1Component },
                    { path: 'step-2', component: Step2Component },
                    { path: 'step-3', component: Step3Component },
                ],
            },

Ah, whoops, I'm in the wrong repo; I just googled to get here and didn't realize this was Angular 1; I'm working with Angular 2.

PArtially addressed by #15002 (in addition to the already supported reloadOnSearch config option, as mentioned in https://github.com/angular/angular.js/issues/3789#issuecomment-25579303).


Change access request form to onboarding form

@Jimmyj30 please apply this change in #52

@Jimmyj30 please apply this change in #52

You may or may not have pinged the other Jimmy LOL

unless you really want me to...

Oops you are right
@jimmyygit


When month is changed thru swiping in either direction, I want to handle certain action.

Nice calendar library.
I want to have a method that handles the event to swiping a month in either direction.

Okay, solved it.
func didShowNextMonthView(date: NSDate) method executed as soon as the month is changed through swipe in right direction.
func didShowPreviousMonthView(date: NSDate) method executed as soon as the month is changed through swipe in left direction.
Anyway, Thank you.


created javelexi animation app

Description
Created app that displays the animated Javelexi gif
Copilot

🤖 Generated by Copilot at fe7fa43
Summary
📝📦🚀

This pull request adds documentation and metadata for the javelexi applet, which is a word game for the Tidbyt device. It includes a docstring for the applet script and a manifest file for the app store.

javelexi applet
docstring and manifest file
autumn leaves of code

Walkthrough

Add a manifest file manifest.yaml to specify the applet metadata and file name (link)
Add a docstring to the applet script javelexi.star to provide a summary, description, and author information (link)

I have read the CLA Document and I hereby sign the CLA

recheck

recheck


.gradle.kts script in a source set doesn't get accessors

Expected Behavior

When a user types a function name in a .gradle.kts file, e.g. kotlin {} IDEA is able to resolve it thanks to s.c. accessors.
These are files (script dependencies) got through Gradle-IDEA integration. Accessors contain declarations for Kotlin DSL, e.g.
public fun Project.kotlin(configure: Action<KotlinJvmProjectExtension>)).

Mentioned files are reported via org.gradle.tooling.BuildAction (org.jetbrains.plugins.gradle.model.ProjectImportAction implementation) during a project import phase.

https://github.com/adamko-dev/kafkatorio and its build file gradle-plugins/settings-plugins/src/main/kotlin/kafkatorio/conventions/kotlin-dsl.gradle.kts are expected to get the accessor declaration described above.
Current Behavior

Mentioned accessor is unavailable for the file. As a result, the declaration cannot be resolved.
And IDEA reports kotlin {} as an unknown call whereas CLI mode works as expected.

Context

Gradle output doesn't contain any helpful information (--console=verbose --warning-mode all).
Script accessors generation doesn't seem to fail.

systemProp.org.gradle.kotlin.dsl.precompiled.accessors.strict=true
:generatePrecompiledScriptPluginAccessors (--rerun-tasks) doesn't fail.

Steps to Reproduce
The issue is stably reproduced with Gradle 7.6 and IDEA 2022.2/3. One needs to:

Clone https://github.com/adamko-dev/kafkatorio
Open the project in the IDEA and wait for import to finish.
Open file gradle-plugins/settings-plugins/src/main/kotlin/kafkatorio/conventions/kotlin-dsl.gradle.kts
Highlighting is broken (kotlin function isn't resolved)

Environment
The issue is stably reproduced with Gradle 7.6 and IDEA 2022.1/2/3.

Hi Andrei,
Thank you for providing a valid reproducer.
I can confirm the reproducer with Gradle 7.6 or 8.0-rc-3 and IDEA 2023.1 EAP.
The issue is in the backlog of the relevant team and is prioritized by them.