Not booting, infinite loading on startup

Describe the bug
stuck loading with yggdrasil in the "void". waited up to 30 min and no main menu or music played
To Reproduce
Steps to reproduce the behavior:

Launch game through SteamVR and/or MO2
Wait for skyrim VR to start
Infinite loading

Expected behavior
expected to load in after 10min
Screenshots
None
Save
None
Crash log
No crashing
Desktop (please complete the following information):

OS version: [e.g. Windows 11, Windows 10] Windows 11
RAM: [e.g. 16gb, 32gb] 16gb
CPU: [e.g. Intel i7-9750H]
GPU: [e.g. Nvidia RTX 2060 Mobile] 2070 Super
Storage type Yggdrasil VR is installed on: [e.g. SSD, HDD] HDD 1TB
Antivirus [e.g. Windows Defender, Bitdefender] Windows Defender

Additional context
tried to launch from both SteamVR and MO2, always gets stuck with yggdrasil at my feet and nothing changes.

I reinstalled and gonna boot up again soon, ill keep updated. Thank you for response. So you advise loading game on my SSD? HDD will make skyrim unbootable later on HDD?


upgrade to v1.6 of neo4j javascript driver is throwing an exception

related to https://github.com/neo4j-contrib/neovis.js/issues/17


Metadata module names dont match class names

The value of 'name' in metadata.json should correspond to the class name in init.pp.
Example of issue is: https://github.com/weareact/wso2puppet-modules/commit/2b97d7076051fdf3dc91d1885d7cb96b2b1af686
Suggest retaining the 'wso2' class prefix, as 'base' could likely conflict with local 'base' modules that people use.

Thanks @fatmcgav for reporting this! We have now fixed this with following commits:
https://github.com/wso2/puppet-modules/commit/8ac3d8466215f12be9b5484a923300cf296392b1
https://github.com/wso2/puppet-modules/commit/c3dc7291a327c6a43fa6f02b8415ffcd81083c82


Fix to include the most recent commit in the commit list

@junha1 I just found out that there is list_descendants method how about just using for building the commit list?


chore: fix kwok offerings to use new requirements

Fixes #N/A
Description
We changed offerings to use requirements rather than hard-coded strings for capacity type and zone. This made kwok offerings incompatible.
How was this change tested?
make apply and scale up and scale down
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 25 (0.0%)  changed or added relevant lines in 2 files are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 78.196%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kwok/tools/gen_instance_types.go
0
10
0.0%

kwok/cloudprovider/helpers.go
0
15
0.0%

Files with Coverage Reduction
New Missed Lines
%

kwok/cloudprovider/helpers.go
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
8270

Relevant Lines:
10576

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 25 (0.0%)  changed or added relevant lines in 2 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.06%) to 78.177%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kwok/tools/gen_instance_types.go
0
10
0.0%

kwok/cloudprovider/helpers.go
0
15
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/disruption/expiration.go
2
89.66%

kwok/cloudprovider/helpers.go
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
8268

Relevant Lines:
10576

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 25 (0.0%)  changed or added relevant lines in 2 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.06%) to 78.177%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kwok/tools/gen_instance_types.go
0
10
0.0%

kwok/cloudprovider/helpers.go
0
15
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/test/expectations/expectations.go
2
93.61%

kwok/cloudprovider/helpers.go
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
8268

Relevant Lines:
10576

💛  - Coveralls


not uploading backup to S3

Summary
Error on upload backup to S3
Steps to reproduce
Use S3 option
What is the expected correct behavior?
Uplaod the file
Relevant logs and/or screenshots

tiredofit/db-backup:3.9.10 12:52:46 /] $ aws
Traceback (most recent call last):
  File "/usr/bin/aws", line 4, in <module>
    __import__('pkg_resources').run_script('awscli==1.25.97', 'aws')
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named 'pkg_resources'

Environment

Image version / tag:  3.3.10
Host OS: Arch Linux

Possible fixes
When i do apk add py3-setuptools into the container, was binary works again

Sorry about this - We started compiling AWS CLI in a recent release and this must have broke it.
Going to investigate.

Not sure what I was thinking when I compiled aws-cli but I have now updated it to 2.13.9 which in my tests now is not throwing the python errors. Please, if you could move up to tiredofit/db-backup:3.9.11 and see if things are working correctly?

Hi! sure, but you still not updated the docker images, I only see 3.9.10

Looks like builds are failing on Github which is why they aren't appearing. Looking into..

@BakasuraRCE FYI the current latest tag seems to be 3.9.11, so could test with that 🙂

Rebuild the 3.9.11 tag...

thanks guys! Its working now


Epinio UI test - Split tests into small ones to improve reliability

Right now, our e2e tests are two big scenarios with all the tests done after each other.
But the reliability decreases as long as we add more tests, that's why we should try to split them into small ones. It might not be the final solution but it worth to give it a try.

Finally, the tests failed due to https://github.com/epinio/epinio/pull/1317
It was not an issue with the increasing of tests number.
Now that it's fixed, they are green again and should be more reliable.
Let's close the issue and I think we will have to revamp them anyway when the standalone UI tests will be integrated.


[String] LIKE properly escapes regexp-characters

Closes #149

Codecov Report

Merging #156 (125e08c) into main (961ce06) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #156      +/-   ##
==========================================
- Coverage   46.39%   46.34%   -0.05%
==========================================
  Files          47       47
  Lines       17828    17828
==========================================
- Hits         8271     8263       -8
- Misses       8111     8122      +11
+ Partials     1446     1443       -3


Update ITKv4 commitid to pull in MINC/libnifti confusion fix

Enabling MINC support was breaking nifti loading/writing in ANTs due to MINC support including it's own libnifti. Update to latest MINC resync on ITK to fix.
This fixes #270

Hi, just wondering if there's an issue with the fix? Are there concerns with bumping the ITK rev?


SPFx 1.17.1 - pdf.js

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version

Chrome 112.0.5615.86

SPFx version

1.17.1

Node.js version

v16.14.1

Issue description
Following on from expanding the custom dialog box, I would like to display the pdf using pdf-js:

For repair dialog:

npm i<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>--save-exact

Install pdf-js:

npm install pdfjs-dist --save-dev

Create compo

import { useEffect, useRef, useState, useCallback } from "react";
import * as React from "react";
// import pdfjsWorker from "pdfjs-dist/build/pdf.worker.entry";
// import * as pdfjsLib from "pdfjs-dist";
import * as pdfjsLib from "pdfjs-dist/legacy/build/pdf.js";
export default function CustomPdfReader() {
    const url = "https://cors-anywhere.herokuapp.com/http://www.pdf995.com/samples/pdf.pdf";
    const canvasRef = useRef();
    const [pdfRef, setPdfRef] = useState("");
    const currentPage = 1;
    const zoomScale = 1;
    const rotateAngle = 0;
    useEffect(() => { pdfjsLib.GlobalWorkerOptions.workerSrc =<EMAIL_ADDRESS>});
    const renderPage = useCallback(
        (pageNum, pdf = pdfRef) => {
            pdf &&
                pdf.getPage(pageNum).then(function (page: any) {
                    const viewport = page.getViewport({ scale: zoomScale, rotation: rotateAngle });
                    const outputScale = window.devicePixelRatio || 1;
                    const canvas = document.getElementById("pdf-doc") as HTMLCanvasElement;
                    const context = canvas.getContext("2d");
                    canvas.width = Math.floor(viewport.width * outputScale);
                    canvas.height = Math.floor(viewport.height * outputScale);
                    canvas.style.width = Math.floor(viewport.width) + "px";
                    canvas.style.height = Math.floor(viewport.height) + "px";
                    const transform = outputScale !== 1
                        ? [outputScale, 0, 0, outputScale, 0, 0]
                        : null;
                    const renderContext = {
                        canvasContext: context,
                        transform: transform,
                        viewport: viewport,
                        textContent: pdfRef
                    };
                    page.render(renderContext);
                });
        },
        [pdfRef, url]
    );
    useEffect(() => {
        if (url.slice(-4).toLowerCase() === ".pdf") {
            renderPage(currentPage, pdfRef);
        } else {
            setPdfRef("");
        }
    }, [pdfRef, renderPage, url]);
    useEffect(() => {
        const loadingTask = pdfjsLib.getDocument(url);
        loadingTask.promise.then(
            (loadedPdf: any) => {
                setPdfRef(loadedPdf);
            },
            function (reason: any) {
                console.error(reason);
            }
        );
    }, [url]);
    return (
        <div>
            <canvas id="pdf-doc" ref={canvasRef} />
        </div>
    );
}

Add pdf reader to customDialog.tsx:
But then i got error:

[11:17:50] Error - [webpack] 'dist':
./node_modules/pdfjs-dist/legacy/build/pdf.js 4380:33
Module parse failed: Unexpected token (4380:33)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|   const httpHeaders = src.httpHeaders || null;
|   const withCredentials = src.withCredentials === true;
`>`   const password = src.password ?? null;
|   const rangeTransport = src.range instanceof PDFDataRangeTransport ? src.range : null;
|   const rangeChunkSize = Number.isInteger(src.rangeChunkSize) && src.rangeChunkSize > 0 ? src.rangeChunkSize : DEFAULT_RANGE_CHUNK_SIZE;
 @ ./lib/components/CustomPdfReader.js 5:0-59 13:28-56 47:26-46
 @ ./lib/components/CustomDialog.js
 @ ./lib/extensions/helloWorld/HelloWorldCommandSet.js

So i found this:

Issue1
Issue2
Everywhere is result: Update to webpack 5 it should solve this issue. or workaround change webpack:

So i go spfx :

Extending Webpack in the SharePoint Framework toolchain
gulpfile.js

build.configureWebpack.mergeConfig({
  additionalConfiguration: (generatedConfiguration) => {
    generatedConfiguration.module.rules.push(
      {
        test: /\.js$/,
        exclude: /node_modules(?!(\/|\\)pdfjs-dist)/,
        loader: 'babel-loader',
        options: {
          'presets': ['@babel/preset-env'],
          'plugins': ['@babel/plugin-proposal-optional-chaining']
        }
      }
    );
    return generatedConfiguration;
  }
});

not help so i got next error:
[11:50:14] Error - [webpack] 'dist':
./lib/extensions/helloWorld/HelloWorldCommandSet.js
Module build failed (from ./node_modules/babel-loader/lib/index.js):
TypeError: this.getOptions is not a function
    at Object.loader (C:\github\spfx-pdfjs\spfx-pdfjs\node_modules\babel-loader\lib\index.js:46:28)
    at Object.<anonymous> (C:\github\spfx-pdfjs\spfx-pdfjs\node_modules\babel-loader\lib\index.js:41:12)

and i am again back:
next issue
You need webpack v5 to use the latest version of style-loader, please read changelog
<EMAIL_ADDRESS>C:\github\spfx-pdfjs\spfx-pdfjs
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └─┬<EMAIL_ADDRESS>│ │   └─┬<EMAIL_ADDRESS>│ │     └──<EMAIL_ADDRESS>deduped
│ ├─┬<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └─┬<EMAIL_ADDRESS>│ │ │ │   └─┬<EMAIL_ADDRESS>│ │ │ │     └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ │ └──<EMAIL_ADDRESS>deduped
│ │ │ └─┬<EMAIL_ADDRESS>│ │ │   └─┬<EMAIL_ADDRESS>│ │ │     └──<EMAIL_ADDRESS>deduped
│ │ └─┬<EMAIL_ADDRESS>│ │   └─┬<EMAIL_ADDRESS>│ │     └──<EMAIL_ADDRESS>deduped
│ └─┬<EMAIL_ADDRESS>│   └─┬<EMAIL_ADDRESS>│     └──<EMAIL_ADDRESS>deduped
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      └──<EMAIL_ADDRESS>deduped

Code on branch: https://github.com/necik11/spfx-pdfjs/tree/pdfjs
I already try almost every possible solutions:

Again, any tips would be very appreciated... webpack is giving me a headache... wouldn't it be easier to upgrade it to version 5? 😍

@necik11 - seems like the problem is with nullish coalescing operator (??).
There are two options I can think of:

try to load the library as external: https://learn.microsoft.com/en-us/sharepoint/dev/spfx/web-parts/basics/add-an-external-library
use custom webpack rule with babel-loader for this specific module/path. Something like include: /node_modules\/path-to-pdfjs\/blah.js/

I finally figured it out...:
There was no problem in loading the library, because I tried different library imports..evenetuall try add some async function, change loading url or even adding module to types.d.ts -> nothing helped.. still same error.
From issue 1 + instructions in spfx, I tried adding a rule to webpack also didn't help, it started giving me the error from issue2
...
But YES the error was probably in WEBPACK (problem in loading modules from those libraries):
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └─┬<EMAIL_ADDRESS>Webpack 4... and as it is written, you need to have v5 and more everywhere..
So the procedure for right working is as follows:

Add Custom rule to Webpack - what I have already stated here
install babel libraries:

npm and @babel/preset-env --save-dev
npm and @babel/plugin-proposal-optional-chaining --save-dev

and the most important babel-loader - but version 8.3.0 the latest version no longer supports webpack smaller than 5.
npm and<EMAIL_ADDRESS>--save-dev

thanks again for the tips and reply.


week in Vietnam is not correct

I use locale in Vietnam, but the "week name" is not correct

Monday is thứ Hai
Tuesday is thứ Ba
etc..
But it looks like only show previous 2 chracters

The picker uses default JS and slices names to 2 letters, in your case, it might be better to take a look at day-names prop


Maximize build spaces of all workflows by manually removing unnecessary resources

Description
While running tests, all indices of elasticsearch become read-only because of limited disk space.
This change maximize the build disk space by removing unnecessary built-in tools.
Related issue and scope

[x] I opened an issue to propose and discuss this change (#5433)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[x] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #5437 (3c37413) into master (9371d62) will decrease coverage by 47.27%.
The diff coverage is n/a.

:exclamation: Current head 3c37413 differs from pull request most recent head 31b5c2d. Consider uploading reports for the commit 31b5c2d to get more accurate results

@@             Coverage Diff             @@
##           master    #5437       +/-   ##
===========================================
- Coverage   76.69%   29.43%   -47.27%
===========================================
  Files         241      234        -7
  Lines       14632    14368      -264
  Branches      616      612        -4
===========================================
- Hits        11222     4229     -6993
- Misses       3410    10139     +6729

see 187 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


python312Packages.bayeux-ml: init at 0.1.14

Description of changes
Add bayeux-ml, a JAX library to work with probabilistic models.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Fails at import with:
File "/nix/store/f2lbq222i5di5p3kc19fgxsmi0i99sy2-python3.12-dm-tree-0.1.8/lib/python3.12/site-packages/tree/sequence.py", line 19, in <module>
   from tree import _tree
ImportError: /nix/store/f2lbq222i5di5p3kc19fgxsmi0i99sy2-python3.12-dm-tree-0.1.8/lib/python3.12/site-packages/tree/_tree.cpython-312-x86_64-linux-gnu.so: undefined symbol: _ZNK4absl12lts_2024072211string_view5rfindEcm

See: https://github.com/NixOS/nixpkgs/issues/345223#issuecomment-2509809415


Rename toSignedInt to toInt & add support for unsigned integers

Fixes #99.
Followup to #74. toInt also works with unsigned integers. I also cleaned up the implementation a bit and added a few more tests.
I could also change the name to something like toInteger, if that's preferred.
cc @Araq

Fine with me, I will update my code.


Add multiple endpoints in prophet

Create multiple endpoints to handle different kinds of requests with different request objects; different endpoint for 1 repo vs. multi-repo project.

Duplicate of #11


This no longer works because d3.rebind has not been a function since d3v4 was launched

also many of the functions inside the code also do not work with d3v4


Error [INSTALL_PARSE_FAILED_INCONSISTENT_CERTIFICATES]

I'm getting this error and no tests are being run.
:app:spoonDebugAndroidTest
2015-07-23 16:41:50 [SR.runTests] Executing instrumentation suite on 1 device(s).
2015-07-23 16:42:09 [SDR.run] [009ee8a01646b2b6] test apk install failed.  Error [INSTALL_PARSE_FAILED_INCONSISTENT_CERTIFICATES]
:app:spoonDebugAndroidTest FAILED
I tried both ./gradlew spoon and ./gradlew spoonDebugAndroidTest
I've uninstalled the Application beforehand so the device is starting out clean
tried with a moto g and nexus 4 device

Probably you removed only application apk. Tests are bundled into a separate apk and are separately installed.
Try running ./gradlew uninstallAll to remove all the possible variants from your device or ./gradlew uninstallDebugAndroidTest to delete the test apk.

looks like that worked! thanks!


Fix broken JSON-RPC eth_syncing API call

What is wrong?
The functionality got removed in https://github.com/ethereum/trinity/commit/4c87a663a0b426d418045ece74c54cb4d6c1eeb4#diff-8a563f755f9f95bd5dbd1e7b6a0f3c22L160 (probably accidentially ping @carver )
We had no tests for this so that went unnoticed for a while.
How can it be fixed
Add it back in and then expand this test to ensure it doesn't break again.
https://github.com/ethereum/trinity/blob/b35ddb9b90a440ec98d5d141e3fbc6e2f1f35b2b/tests/integration/test_trinity_cli.py#L142-L143

The functionality got removed in 4c87a66#diff-8a563f755f9f95bd5dbd1e7b6a0f3c22L160 (probably accidentially ping @carver )

Yup, almost certainly an accident.

@cburgdorf  Can I work on this issue? Can you explain it a little more>
Thanks!

@Patil2099 sure, you can. You just have to be prepared to explore the exact path forward on your own.
To get you off the ground, here's what's wrong and how you can prove it.

Open a terminal and run trinity
Open a second terminal tab and run trinity attach
The terminal output in the second terminal should look like this:

$ trinity attach
    INFO  09-06 09:52:43               trinity
      ______     _       _ __
     /_  __/____(_)___  (_) /___  __
      / / / ___/ / __ \/ / __/ / / /
     / / / /  / / / / / / /_/ /_/ /
    /_/ /_/  /_/_/ /_/_/\__/\__, /
                           /____/
    INFO  09-06 09:52:43               trinity  Started main process (pid=14010)
    INFO  09-06 09:52:43               trinity  Trinity/0.1.0a27/linux/cpython3.7.3
    INFO  09-06 09:52:43               trinity  Trinity DEBUG log file is created at /home/cburgdorf/.local/share/trinity/mainnet/logs-eth1/trinity.log
Trinity Console
---------------
An instance of Web3 connected to the running chain is available as the `w3` variable
The exposed `rpc` function allows raw RPC API calls (e.g. rpc('net_listening'))

Type rpc('net_listening') and hit enter
Your output so far should look like this.

In [1]: rpc('net_listening')
Out[1]: {'id': 0, 'jsonrpc': '2.0', 'result': True}

:point_up: is just to demonstrate that the RPC API works in general and what correct output would look like.

Now type rpc('eth_syncing') and hit enter
You'll notice that it just hangs for ten seconds until it hits a timeout

Timeout: 10 seconds

This is what's broken

Why is it broken? Because behind the scenes there is a SyncingRequest being fired that is waiting on a SyncingResponse but there's no one responding.

https://github.com/ethereum/trinity/blob/248efac0cc6daf43e49e5739a80f11737dd06161/trinity/rpc/modules/eth.py#L273-L281

Why is no one responding?

Because it was accidentally removed here: https://github.com/ethereum/trinity/commit/4c87a663a0b426d418045ece74c54cb4d6c1eeb4#diff-8a563f755f9f95bd5dbd1e7b6a0f3c22L160

What needs to be done is to figure out how to put back in the code that listens for the SyncingRequest and fires back an appropriate SyncingResponse.

Does that get you started?

hi @cburgdorf ,
I tried to fix this, here is my commit. I haven't merged yet because it always has some error and my code doesn't be executed.
   DEBUG  2019-10-31 13:56:32,862  ETHPeerPoolEventServer  Running ETHPeerPoolEventServer
   ERROR  2019-10-31 13:56:32,866            FullServer  Unexpected error in <trinity.server.FullServer object at 0x1089e95c0>, exiting
Traceback (most recent call last):
  File "uvloop/loop.pyx", line 1109, in uvloop.loop.Loop._create_server
  File "uvloop/handles/streamserver.pyx", line 51, in uvloop.loop.UVStreamServer.listen
  File "uvloop/handles/streamserver.pyx", line 89, in uvloop.loop.UVStreamServer._fatal_error
OSError: [Errno 48] Address already in use

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/kimi/CODE/trinity/p2p/service.py", line 117, in run
    await self._run()
  File "/Users/kimi/CODE/trinity/trinity/server.py", line 125, in _run
    await self._start_tcp_listener()
  File "/Users/kimi/CODE/trinity/trinity/server.py", line 115, in _start_tcp_listener
    port=self.port,
  File "/usr/local/Cellar/python/3.6.5_1/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/streams.py", line 119, in start_server
    return (yield from loop.create_server(factory, host, port, **kwds))
  File "uvloop/loop.pyx", line 1669, in create_server
  File "uvloop/loop.pyx", line 1113, in uvloop.loop.Loop._create_server
OSError: [Errno 48] error while attempting to bind on address ('<IP_ADDRESS>', 30303): address already in use
    INFO  2019-10-31 13:56:32,880            FullServer  Closing server...
   DEBUG  2019-10-31 13:56:32,880            FullServer  <trinity.server.FullServer object at 0x1089e95c0> halted cleanly
   DEBUG  2019-10-31 13:56:32,880              FullNode  <trinity.server.FullServer object at 0x1089e95c0> finished while <trinity.nodes.full.FullNode object at 0x1089bb588> is still running, terminating as well
   DEBUG  2019-10-31 13:56:32,880              FullNode  Cancelling service: <trinity.nodes.full.FullNode object at 0x1089bb588>
   DEBUG  2019-10-31 13:56:32,884              FullNode  <trinity.nodes.full.FullNode object at 0x1089bb588> finished: Cancellation requested by FullNode token
   DEBUG  2019-10-31 13:56:32,884              FullNode  Waiting for child services: [<trinity.protocol.eth.peer.ETHPeerPoolEventServer object at 0x108a0da20>, <trinity.server.FullServer object at 0x1089e95c0>]
   DEBUG  2019-10-31 13:56:32,887       BeamSyncService  <trinity.sync.beam.service.BeamSyncService object at 0x108a0def0> finished: Cancellation requested by FullNode token
   DEBUG  2019-10-31 13:56:32,887       BeamSyncService  <trinity.sync.beam.service.BeamSyncService object at 0x108a0def0> halted cleanly
   ERROR  2019-10-31 13:56:32,888  component(#SyncerComponent)  Sync ended unexpectedly. Shutting down trinity

I cloned the source directly and executed trinity without any arguments and didn't touch any configs. Have any ideas? thanks.

@KimiWu123 the error suggests one of these things:

You have another Trinity instance running
You have something else running taking that port
Maybe a previous Trinity run did not shutdown cleanly.

I'd suggest you try running trinity fix-unclean-shutdown to see if that fixes the error.
However, that particular issue already has an active PR by @voith open so I suggest looking for a different issue that doesn't have an open PR yet.

hey guys, sorry for not posting here before. I wanted this issue fixed to use somewhere else.


GD-337: Start an individual parameterised test from the GdUnit3 tab context menu

Is your feature request related to a problem? Please describe.
May be related to #336
Describe the solution you'd like
In a GdUnit3 inspector tab that is displaying the results of individual parameterised unit tests, I would like to be able to right-click an individual test result and select "Run Test" or "Debug Test" to execute a test run containing only the test with those parameters.
Describe alternatives you've considered
Being able to right-click the test parameter array in the code editor to run/debug individual tests would be an edge case.
Additional context
i.e. add to this context menu

running a single parameterized test by select

🙌


Feature/add jackson lib

Added Jackson library to the circe-benchmark.

Content looks good to me, but do you mind reorganizing the imports to keep Scalastyle happy and fix the build? I think it just wants the databind lines together / combined:
import com.fasterxml.jackson.databind.{ JsonNode, ObjectMapper }

(Also, you can run Scalastyle locally with sbt scalastyle to confirm that it's fixed.)
Thanks again!

Thanks, will merge when green!

Codecov Report

Merging #18 into master will decrease coverage by 4.52%.
The diff coverage is 38.46%.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
- Coverage   90.67%   86.15%   -4.53%
==========================================
  Files          11       12       +1
  Lines         118      130      +12
==========================================
+ Hits          107      112       +5
- Misses         11       18       +7

Impacted Files
Coverage Δ

src/main/scala/io/circe/benchmarks/Benchmark.scala
100% <ø> (ø)
:arrow_up:

...scala/io/circe/benchmarks/JacksonDefinitions.scala
38.46% <38.46%> (ø)

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 714d5c5...62a2e71. Read the comment docs.


Stepping into a built in function in C++ doesn't work on linux

Does this issue occur when all extensions are disabled?: No

VS Code Version: 1.59.0
OS Version: Ubuntu 20.04

Steps to Reproduce:
1.Debug any C++ code using gdb
2.Step into a built in function (like cout)
Stepping into a built in function doesn't do anything it just steps over the next line. But it works fine on windows so maybe vscode can't access usr/include files ?

/extC++


MV3 SuperTokens/browser-tabs-lock libraries expects window context

https://github.com/supertokens/supertokens-web-js/issues/62

the browser-tabs-lock library also requires localStorage


Increase dipow speed

Coverage remained the same at 100.0% when pulling 865eab64268b464a4042dd0048386726319f7eea on dipowspeed into b9ab0efd98c668a21ca3ec33c26e440ac3fab970 on master.


Disable time

Is there a way to completely hide time from output?

See the readme, section "Customize Attributes"


PressableButton fixes

Use local space push distances, so that pressable buttons can be scaled

Add experimental versioning to PressableButton to not make this a breaking change
Adapt plane gizmos for local space unit distances

Overview
Changes

Fixes: # .

The version check is pretty gnarly.
You could do this instead:
public enum ButtonSpaceMode
{
    World
    Local
}

public class PressableButton : MonoBehaviour, IMixedRealityTouchHandler
{
    ...

    [SerializeField]
    private ButtonSpaceMode spaceMode = ButtonSpaceMode.World;

    ...
}

And then execute different press logic based on spaceMode.
This would enable local space distances without breaking existing buttons and without the need for a version check.
I see both modes being useful so having an option would be nice anyway.

One reason we may not need to do such an advanced version upgrade for the buttons is that we are now very clearly visualizing the butotn push planes in the editor. So, as long as we clearly state "after you upgrade, double check the push planes for your custom buttons", we should be fine.
Also, we should update the pressable button prefabs, since I think most people just use the PressablebuttonPlaced prefab anyway.

It would be very useful to have a test scene committed (perhaps in playmode tests) that verifies that after this change we can press buttons whose global scale is at widely diffrent values, like 100x, 10x, 1x, 0.1x, 0.01x

Isn't there a bug that this is actually fixing? It's not mentioned in the PR.

Use local space push distances, so that pressable buttons can be scaled.
I haven't pulled to test, but I'm hoping this fixes a related issue I'm running into where I change the size of the pressable-button collider (not the object scale) and end up with wonky states pretty reliably.

closing this draft PR for now - will reopen a new PR with the changes we want to keep (and addressing the feedback) and link it here soon


Missing 'new' flag in finance/get/stakeinfo result

finance/get/stakeinfo is not returning the new flag that indicates whether trust account is staking Genesis.

Resolved with commit a5b6f71ec10dfbcac0914cecdcd3b625a9f6edb2 thank you for submitting the issue!


[New Port Request]  vJoy

Library name: vJoy
Library description: A Windows library for creating a "fake" joystick that the Windows kernel thinks is a real device, but is actually being fed software-generated values that users can set through a software API (called the vJoy Interface API).
Source repository URL: https://github.com/njz3/vJoy.git
Project homepage (if different from the source repository): http://vjoystick.sourceforge.net/site/
Anything else that is useful to know when adding (such as optional features the library may have that should be included):
So this is a somewhat unique situation.  Porting the vJoy interface API to vcpkg doesn't sound too difficult (the github has a ready to go Visual Studio project).  However, without the corresponding device driver to create the emulated device in Windows, it's not particularly useful.  The source repo has the source code for both the interface API and the device driver.  But, from what I understand, vcpkg is not in the business of building and installing kernel-privileged drivers.
So, if this project were to get ported, would it make sense for that port to download the prebuilt driver installer (available through a release on the project GitHub) and display a helpful message along the lines of "you need to install the driver that was just downloaded to (some path)"?
Is this approach even "allowed" for vcpkg ports?  I don't mind working on porting the interface API to vcpkg.  But if a PR for that port has no chance of getting merged due to the requirement to install an external driver, I am not going to waste my time.

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Update Composer dependencies (2022-01-26-00-09)

Visual regression test passed!
View the visual regression test report


modifications to support running on AWS

This is a work in progress.   In the IMS_PROC repo, instances of ksh need to be changed to bash in makefile.sh and Makefile.

changes also needed in add_jedi_incr (add aws_modules file, update build.sh to use it)

Closing this PR, to re-submit against a feature branch.


HTML5 customisation with chunk attribute

We have the following customisation in-place for HTML5 transformations in the rewrite-rules.xsl file:
<xsl:template match="file[@format = 'dita']/@result">
<xsl:variable name="outDir">topics/</xsl:variable>
<xsl:variable name="fileOnly" select="(tokenize(., '/')[last()])" />
<xsl:variable name="parentDir" select="tokenize(., '/')[last()-1]" />
xsl:choose
<xsl:when test="starts-with($parentDir, 'temp|topic')">
<xsl:attribute name="{local-name()}" select="concat($outDir, $fileOnly)" />
</xsl:when>
</xsl:choose>
</xsl:template>
This customisation creates issues when I include a markdown topic with chunk=”by-topic” in my ditamap.

Only the first chunk that is converted to html is moved to ‘topics’ folder.
Rest of the generated chunks are moved to a different folder

I added an xsl:message inside the xsl:template to see if it gets called for the files which were generated by the chunking, it doesn't call the template for chunked files.
Can this template be called for the chunked files?

DITA-OT version: 3.6.1

Operating system and version: macOS

How did you run DITA-OT? dita` command

Transformation type: HTML5

So you are using this DITA OT extension point: https://www.dita-ot.org/dev/topics/plugin-rewrite-rules.html
Can you also add to the issue a zip containing a very small sample DITA project to test the problem with and a DITA OT plugin which uses the extension point ?


The wave effect does not move with the sound.

The wave effect does not move with the sound. Help?

I also faced the same issue both in android and iOS.

Having the same issue. On the last version of flutter (3.3.8) and audio_waveforms package (0.1.5+1).
The audio is complete and the progress bar is in sync with the recording, but the visualization is wrong.

There are 2 open PRs #126 #123  which might fix the sync issue and wrong visualization.


Refactoring of connectors

Originally connectors were built for CF.  With our move to a more platform agnostic approach to Steeltoe, we need to take a look at how we can better abstract out connector functionality for usage on other platforms.
related ticket: #26

Consider better compatibility with GetConnectionString


Modularize twitch_oauth2 & gh-pages

This tries to do a local doc renderer, doubt it's going to work but let's see!

It didn't work but now it should


Chore/update elixir and otp and dependencies

What does this do on a high level?
This updates the library to be compatible with the most current Elixir and OTP while maintaining backwards compatibility.
Bumps library version from 1.1 to 1.2.
This adds checks to this library's CI for the following versions:

OTP 26.2 and Elixir 1.15.7
OTP 26.2 and Elixir 1.16.0
Bump Elixir 1.14 version from 1.14.2 to 1.14.5

Unfortunately, there conflicts with outdated dependencies. As a result, I updated all of the dependencies to they most current version.
The upgrade to Elixir 1.16 was causing some assertion areas where the test code was a little hand-wavy and so that had to be updated to a less flaky approach.
The PR was co-written with @GeoffreyPS, who added the version updates. The blame for the dependency updates and changes to testing approach lands on me.
How was this change made?
I added new entries to the matrix field in main.yml for the github configuration.
Dependencies in mix.exs were updated.
The approach to assert on the test message was updated.
Why is this change necessary?
This library should keep up with new versions of elixir so as to remain relevant.
How will this be verified?

[ ] The PR includes new tests
[X] Current test suite covers functionality
[ ] This has been tested locally
[ ] This has been tested in dev

Any additional information or special handling required on this PR?

motivational imagery

@lmarlow Never got reports that it wasn't, BUT that's because it's very likely that anyone using it had newer versions of the dependencies as part of their application. So, practically, this was mostly impacting the ability to maintain the library.


NER paper

but it improve F1 in your paper

You need to understand what is the "Oracle scores" https://github.com/jiesutd/NCRFpp/issues/151#issuecomment-613114038. Please add your reply to the original issue https://github.com/jiesutd/NCRFpp/issues/161  rather than open a new issue.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #1480

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Error when trying to install

Hi, I'm encountering an error whenever I try to install the css tweaks:
Traceback (most recent call last):
  File "backend.py", line 847, in write_js_fixes
PermissionError: [Errno 13] Permission denied: 'fixes.temp.txt'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "manager.py", line 55, in install_click
  File "backend.py", line 907, in write_js_fixes
UnboundLocalError: local variable 'line' referenced before assignment

I tried all of the steps in the troubleshooting section and the same error occurs, hope you can help. I only have Install CSS Tweaks selected and have Steam Beta Update enabled.

Try running the program as admin, or move it to a folder that you have write permissions for

Thanks, it finally worked when I moved it inside the Steam folder.


Turn off html ids generated by form_with to improve asset comments

Fixes #747
Fixes #578
Context
We were unable to make private comments on the 2nd or greater instance of assets on a page.
In part this was due to reused ids due to asset caching — the asset was stuffed full of asset.unique_id which ironically is no longer unique when cached.
We rely on the caching, so the answer was to remove ids, as they do not provide a function.
However in Rails 6, this can only be done globally, so there is some amount of risk.
In addition, I had to move around some of the click handling on the custom checkbox code to get the label + checkboxes to function again. I believe this is because the label no longer is auto-associated with the checkbox.
QA

@sudara  – I cruised through these checkboxes, in comment assets and on the playlist edit. They seem to work fine on all platforms i tested, could click labels, correct values sent to the server..

@sudara – I think there might be a difference in how Firefox and Safari are handling the Private checkbox on the playlist edit page.
I think Firefox is updating the database when you click the input, whereas Safari you need to click  save changers.
I also think I've seen a situation in Firefox where I've clicked "save changes" and then hit Reload after that page has loaded, and seen the checkbox flip states, even when the only action was a page reload.

Capybara still isn't clicking the private comment checkbox correctly...


Fix Joint2D/Joint3D node path reset on scene switch

When one of the bodies exited the tree, the corresponding node path was reset instead of just resetting the joint from the physics server. That was causing the node path to be reset on scene switch when one of the bodies is under the joint in the scene tree.
Regression from #44703
Fixes #46571
Tested with case from #44510, still fixed.

The change seems to invalidate most of the rest of what was added in #44703 (connecting signals, then adding code in several places to disconnect the signals). If the proposed change is sufficient, then the previously added code could likely be removed too and replaced by connecting the body's body_exited signal directly to update_joints(true), no?

@akien-mga Let me know if I'm missing something, but it seems still safer to disconnect signals from both bodies when one of them exits the tree to avoid any side effect.
So I would rather keep _body_exit_tree which disconnects signals and disables the joint, but I've just updated the PR to remove the body id that was passed as an argument since it's not used anymore.

Thanks!


Pull request that passes tests on a branch but will fail when merged

Do not merge!
This is a commit that was authored on a version of Master before https://github.com/DFE-Digital/apply-for-postgraduate-teacher-training/pull/939 was merged, and modifies something that will be tested by that branch. The tests on this branch should be green, and it does not produce a merge conflict, but if this is merged, it should blow up!

Tests green! Now pushing a commit that switches to testing the merge commit.

🎉 Yes!


Data Quality Score Algorithm - Potential Issue

The new Data Quality Score algorithm code and documentation appears to have some potential errors. And we should consider reviewing the solution to ensure that all required fields are being score and optional fields are weighted less in point value.
Algorithm Documentation:
https://github.com/GSA/code-gov/blob/master/data_quality_scoring.md
Algorithm Rules:
https://github.com/GSA/code-gov-harvester/blob/master/libs/rules/index.js
The documentation shows the field's point value assignment for the Metadata Schema 2.0.0 required and optional fields. The screenshot below shows the documentation point assignment per field, and the source code list for fields that are being evaluated on reach repo.

The solution is missing the following fields:
  agency
  measurementType
  releases

Keep in mind that "organization" is an optional field nested under "releases".
Also, the algorithm evaluates each repo on all required and all options fields, not just the required fields. Meaning that each repo is graded on a 158 total points scale, not just the  71 required fields total points.
 (Repo Total Points / 158 ) * 10 = repo score.

Perhaps consider scaling the optional fields as a bonus point value to the overall score?
i.e.
 (Repo Required Fields Points / 71 ) * 10 + (Repo Optional Fields Points / 87)*10 = repo score.

Or consider alternative solution in which optional fields are not negatively impacting the required fields point score.  Not all agencies are populating their code.json metadata file with optional fields on their releases/repos and that impacts their overall Data Quality Score.

Metadata Schema 2.0.0 Requirements
https://code.gov/about/compliance/inventory-code
Good Metadata Examples
https://github.com/GSA/code-gov/blob/master/METADATA_EXAMPLES.md


How to set up with app directory and new layout?

How should I set up this with the app directory and layout file?
Can you point me the correct way?
I have this in my layout.tsx:
import type { Metadata } from 'next'
import { Inter } from 'next/font/google'
import './globals.css'

const inter = Inter({ subsets: ['latin'] })

export const metadata: Metadata = {
  title: 'Create Next App',
  description: 'Generated by create next app',
}

export default function RootLayout({
  children,
}: {
  children: React.ReactNode
}) {
  return (
    <html lang="en">
      <body className={inter.className}>{children}</body>
    </html>
  )
}

Did you find a solution?

Is there any information about this? At the moment, I urgently need static generation, next still does not support it in combination with i18n


Register to app and write test calls

As a product owner I would like the developers to register to app and write test calls to open fda as proof of concept.

Test calls are complted as Food Enforements and Drug Events.


Question: is partial column extract possible?

Hello,
I have a question regarding the partial data extraction. Here's my scenario:
I have a huge (1.5 - 2Tb) parquet file in the Azure Blob Container and I am given a stream to that file. The requirement is to extract N rows from that file. It does not have to be same N rows across all columns. For instance, it is acceptable to have top 10 row values for the first column, and then random 10 ones for remaining columns.
Question: is there a way to extract only portion of values (something similar as Spark does) without reading the entire column? Something like DataColumn ReadColumn(DataField field, int start, int count) method.
Thank you in advance.

@aloneguid it still would be useful to read the column in chunks rather than in one huge array, i.e. something like groupReader.ReadColumn(dataField, buffer, count)


Tmp subtitle format support

.tmp format is an old one but can still be found, especially for old movies.
It is a single line format having only a start time:
00:05:22:This is a subtitle text

Hey, thanks for the patch! :) I'll have a closer look and release a new version with it as soon as I have some more time.
In the meantime, if you could also provide a test case for reading and writing the format - pysubs2/test_tmp.py in spirit of pysubs2/test_subrip.py, that would be great!


Final merge of feature/abci++vef into main

Contributes to #10
This is the final merge of feature branch feature/abci++vef. After this merge, the feature branch will cease to exist technically.
We will need to make some adjustment to UPGRADING.md after the merge with extra info on how FinalizeBlock events are indexed, but that shouldn't block this merge.
IMPORTANT: Please do not review the detail of this merge, as this code has already been reviewed on feature/abci++vef.

PR checklist

[x] Tests written/updated
[x] Changelog entry added in .changelog (we use unclog to manage our changelog)
[x] Updated relevant documentation (docs/ or spec/) and code comments

4790ea3e4 represents the original merge commit + all subsequent commits of this PR squashed onto the merge commit


inject dinamic states

Its normal when inject dynamic reducers the other states called again all before states called for example in your example if you clicked in the button 10 times, after you go for example to chat and the clicked state is fired 10 times again all time when you called inject({ chat: reducer }) for example.

I do not understand the problem, is there one?

The state is global to the application (with redux).
You can reset the reducers to the page change in a componentDidMount or @provideHooks.


npm

This package should make its way onto npm. In order to get there, I'd like to do a bit of cleanup and make a proper exports.

Dupe of #8

You might want to hold off on the exports and to straight to ES modules, which will be standard across all JS VMs. This will be much better than using Node's CommonJS because it will not require an extra build step for browser versus Node, and since you've already been waiting a while to do it right, why not wait to really do it right with ES Modules? Those are my thoughts.

You might want to hold off on the exports and go straight to ES modules, which will be standard across all JS VMs. This will be much better than using Node's CommonJS because it will not require an extra build step for browser versus Node, and since you've already been waiting a while to do it right, why not wait to really do it right with ES Modules? Those are my thoughts.

Thanks for the info. Admittedly I'm not a JS guy and it's hard to stay on top of all of this. I do have an exports patch that's 95% done, just need to find the motivation for the last push

@brian-armstrong, is there any update on your progress?

In case there are still people interested, I made this module for a personal project. It is quite dirty but it works (web only) and will probably the job until quiet-js is officially published to npm. Feel free to fork it or send PRs if you have improvements to suggest.


Change TypeNameHandling enum to be None

Change TypeNameHandling enum to be None instead of All, because using All runs the risk of performing insecure Json.Net deserialization, especially since the Deserialization step (line 212) happens without using a custom SerializationBinder, and with content that is downloaded from potentially attacker-controlled content (an attacker's blob storage container).
Description
I simply changed the TypeNameHandling to be None

Hi @Voulnet
Thank you for submitting this PR, and your concern regarding security and serialization in the dotnet sdk.  We do allow a custom serializer parameter, but cannot change the default to .None as that would be a breaking change.
Considering storage is retrieved from a container controlled by the bot developer, there is no risk in that regard.  A bot developer would need to purposefully architect a bot retrieving storage from untrusted sources in order for this to be an issue.
Also, please consider the changes in this PR: https://github.com/microsoft/botbuilder-dotnet/pull/4369/files  Supporting TypeNameHandling.None requires other sdk modifications.
Again, thank you for thinking on these things and working towards contributing improvements.  Unfortunately, we cannot take this change at this time.


Incorrect comment aligning with strange unicode characters

With this code:
fn main() {}

pub struct WhitespaceCharacters {
    pub space: char,
    pub nbsp: char,
    pub tab: char,
    pub newline: char,
}

impl Default for WhitespaceCharacters {
    fn default() -> Self {
        Self {
            space: '·',    // U+00B7
            nbsp: '⍽',    // U+237D
            tab: '→',     // U+2192
            newline: '⏎', // U+23CE
        }
    }
}

rustfmt incorrectly aligns the comments, so that the one next to space is one character past the others.
Note that in utf8 the character next to space is 2 bytes, while the others they are 3 bytes.
Changing the chars to more basic ASCII chars resolves the issue:
impl Default for WhitespaceCharacters {
    fn default() -> Self {
        Self {
            space: 'h',   // U+00B7
            nbsp: 'i',    // U+237D
            tab: 'j',     // U+2192
            newline: 'k', // U+23CE
        }
    }
}

cargo fmt --version
rustfmt 1.5.1-stable (4b91a6e 2022-08-08)

technically this is a bug within buggy behavior (rustfmt isn't supposed to vertically align comments in these positions by default anyway), but the overarching point about unicode still holds true


Add CVSS information

Unfortunately, CVSS information is missing. This information could be very helpful in vulnerability management process.

Yep, I think given this information is available in CVE, we should populate it in the resultant OSV record.

I think this issue belongs better in osv-scanner.
The CVSS score can be added dynamically in the osv-scanner output, based on the grouped matches. We aren't able to modify entries from other sources.

I'm thinking we're talking about two separate uses of CVSS.
I was thinking about including CVSS in the OSV records that are converted from CVEs.
I'm beginning to suspect based on your most recent comment, that you're talking about including CVSS scores in the output from OSV Scanner?


[SPARK-48996][SQL][PYTHON] Allow bare literals for and and or of Column

What changes were proposed in this pull request?
Allows bare literals for __and__ and __or__ of Column API in Spark Classic.
Why are the changes needed?
Currently bare literals are not allowed for __and__ and __or__ of Column API in Spark Classic and need to wrap with lit() function. It should be allowed similar to other similar operators.
>>> from pyspark.sql.functions import *
>>> c = col("c")
>>> c & True
Traceback (most recent call last):
...
py4j.Py4JException: Method and([class java.lang.Boolean]) does not exist

>>> c & lit(True)
Column<'and(c, true)'>

whereas other operators:
>>> c + 1
Column<'`+`(c, 1)'>
>>> c + lit(1)
Column<'`+`(c, 1)'>

Spark Connect allows this.
>>> c & True
Column<'and(c, True)'>
>>> c & lit(True)
Column<'and(c, True)'>

Does this PR introduce any user-facing change?
Yes.
How was this patch tested?
Added the related tests.
Was this patch authored or co-authored using generative AI tooling?
No.

The failure seems not related to this PR.

Thanks! merging to master.


Add support for FusedBatchNormV3

Issue Description
After training an resnet50 and inceptionv3 model from keras.application on using tensorflow 1.15 I was unable to import the model with TFGraphMapper since it is missing the op for FusedBatchNormV3.
java.lang.IllegalStateException: Could not find class for TF Ops: FusedBatchNormV3
  at org.nd4j.base.Preconditions.throwStateEx(Preconditions.java:641)
  at org.nd4j.base.Preconditions.checkState(Preconditions.java:304)
  at org.nd4j.imports.graphmapper.tf.TFGraphMapper.importGraph(TFGraphMapper.java:278)
  at org.nd4j.imports.graphmapper.tf.TFGraphMapper.importGraph(TFGraphMapper.java:141)
  at org.nd4j.imports.graphmapper.tf.TFGraphMapper.importGraph(TFGraphMapper.java:87)
  at org.nd4j.imports.graphmapper.tf.TFGraphMapper.importGraph(TFGraphMapper.java:73)
  ... 45 elided

I've included the pbtxt of the inceptionv3 model.
inceptionv3-docclas-77acc.h5.zip
Version Information
1.0.0-beta6

We appear to support this now:  https://github.com/KonduitAI/deeplearning4j/blob/88d3c4867fb87ec760b445c6b9459ecf353cec47/nd4j/nd4j-backends/nd4j-api-parent/nd4j-api/src/main/java/org/nd4j/linalg/api/ops/custom/FusedBatchNorm.java#L71
Closing this.


Add a new FAQ for containerlog enrichment changes

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

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

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

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

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


Audio on Compositing Reviews

Goal
When publishing from Nuke, similar to Maya, we want audio on the reviews.
Implementation
Nuke does not have any audio node to reference, so the best place to grab the audio from would be by looking in a specific place in the database. Something like shot > audio.

Only audioMain subset is currently supported. Further development is needed to support configurable subsets.

Once we port nuke to 3.0 this will be possible to do in settings

With this PR the subset name can be configured; https://github.com/pypeclub/pype/pull/1073


Unable to find .prisma/client

Hi there! Thanks for the repo.
Anyways I bumped into this issue here, and I'm not sure if its intentional to import from .prisma/client or if its an issue with pnpm linking.
https://github.com/kepelrs/nestjs-prisma-crud/blob/2a89f7b24441c009e2abc45e7ccb52065f0ffb68/src/crud/prisma-crud.service.ts#LL1-L1C47

Thanks, this was probably unintentional IDE autocomplete.
It is fixed in the beta-17. Let me know if you still have issues after upgrade :)


objectident.py getting it to only write once to my file "seen.txt"

This should be fairly easy to check if foundClass is in seen.txt, if not write foundClass into file "seen.txt" if foundClass exists in seen.txt dont do anything. I have it as print already seen in the terminal for testing
def recordFound(recordDex):
foundDex = foundClass
recordDex = os.path.abspath("seen.txt")
with open(recordDex, "a+") as f:
if foundClass in f.read():
print("already seen")
else:
f.write(foundClass + "\n")

Was this resolved?

Yeah I should probably close the issue LMAO


ML-165, ML-166: Non persistent chat

Adds chat without persistency. Things to do in the future:

chat moderation
persistency (possibly)

I updated npm dependencies, thus +5,950 changes.


self position

Ingame, !position should return self-position

Does it not do that currently?

The handler was missing from the handlers list. I fixed that.


Restyle repository to 80 columns.

Simply bumping up the acceptable line width doesn't seem like a good way to tackle the problem of overly nested structures.
This change removes the special settings and restyles everything to 80 columns with exceptions where doing so would tie us in knots.
This change does not attempt to tackle any of the problems such as the powerful odour coming from all the mockery in unit tests.

While I totally agree with the 80 column column width, can we wait with this? We are at least 4 PRs ahead of trunk (and I have been reformatting things to 80 columns :) ), so please let us check what kind of conflicts this creates for us. I assume this is done with black (or a similar tool??), so we could do this on our end as the PR after if we have too many conflicts?

I have no problem with pulling this change if it messes with what you are doing.

It causes over 20 conflicts for us, not sure how much effort it would be to fix them :(  Did you just run black (or any other tool) on the sources? If we can re-run whatever you did, I would prefer that. But if you have done it manually, then we might have to bite the bullet and resolve the conflicts :(
FWIW, we are currently trying to change our fork to point back to this repo, atm we are 'disconnected', and can't even easily submit PRs anymore

This change is being made piecemeal as other changes are made.


kythe tool: add --target_definitions switch to decor subcommand [phab:D825]

Created by schroederc at 2016-04-12 21:52:28:

shahms accepted this revision at 2016-04-12 21:54:19

schroederc closed via commit f6dc9fb84f4e10f46cdb732c50889eff482bb6d6 at 2016-04-12 22:06:58


go-rice: 1.0.0 -> 1.0.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/go.rice/versions.
meta.description for go-rice is: "A Go package that makes working with resources such as html, js, css, images, templates very easy"
meta.homepage for go-rice is: "https://github.com/GeertJohan/go.rice"
meta.changelog for go-rice is: ""
Updates performed

Golang update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/xhvxpi462pa8xpyspf894n7d4hai53h5-go.rice-1.0.2/bin/rice passed the binary check.

1 of 1 passed binary check by having a zero exit code.

0 of 1 passed binary check by having the new version present in output.

found 1.0.2 with grep in /nix/store/xhvxpi462pa8xpyspf894n7d4hai53h5-go.rice-1.0.2

found 1.0.2 in filename of file in /nix/store/xhvxpi462pa8xpyspf894n7d4hai53h5-go.rice-1.0.2

directory tree listing: https://gist.github.com/dfd697957ecb0f0fb3215914dd8d27a1

du listing: https://gist.github.com/03c6ddd491edda615500167c2709651b

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
cfssl
go-rice

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xhvxpi462pa8xpyspf894n7d4hai53h5-go.rice-1.0.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A go-rice https://github.com/r-ryantm/nixpkgs/archive/a19528785e500624ce3b1f3647e30c65cbfa4a5f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xhvxpi462pa8xpyspf894n7d4hai53h5-go.rice-1.0.2
ls -la /nix/store/xhvxpi462pa8xpyspf894n7d4hai53h5-go.rice-1.0.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

cfssl
go-rice

Maintainer pings
cc @blaggacao @kalbasit for testing.

Result of nixpkgs-review pr 127221 at a1952878 run on aarch64-linux 1

2 packages built successfully:

cfssl
go-rice


[Concurrent Segment Search] SignificantTerms agg should not gather bg_count for each slice

Subtask as a part of https://github.com/opensearch-project/OpenSearch/issues/8509
To Reproduce
./gradlew ':server:internalClusterTest' --tests "org.opensearch.search.aggregations.bucket.SignificantTermsSignificanceScoreIT.testXContentResponse" -Dtests.seed=6D7B107FC7821E62 -Dtests.locale=mk -Dtests.timezone=America/Atka -Druntime.java=17

Details
This test is failing due to an unexpected result in the query response:
Expected:
{
    "class": {
        "doc_count_error_upper_bound": 0,
        "sum_other_doc_count": 0,
        "buckets": [{
            "key": "0",
            "doc_count": 4,
            "sig_terms": {
                "doc_count": 4,
                "bg_count": 7,
                "buckets": [{
                    "key": 0,
                    "doc_count": 4,
                    "score": 0.39999999999999997,
                    "bg_count": 5
                }]
            }
        }, {
            "key": "1",
            "doc_count": 3,
            "sig_terms": {
                "doc_count": 3,
                "bg_count": 7,
                "buckets": [{
                    "key": 1,
                    "doc_count": 3,
                    "score": 0.75,
                    "bg_count": 4
                }]
            }
        }]
    }
}

Actual:
{
    "class": {
        "doc_count_error_upper_bound": 0,
        "sum_other_doc_count": 0,
        "buckets": [{
            "key": "0",
            "doc_count": 4,
            "sig_terms": {
                "doc_count": 4,
                "bg_count": 28,
                "buckets": [{
                    "key": 0,
                    "doc_count": 4,
                    "score": 0.39999999999999997,
                    "bg_count": 20
                }]
            }
        }, {
            "key": "1",
            "doc_count": 3,
            "sig_terms": {
                "doc_count": 3,
                "bg_count": 21,
                "buckets": [{
                    "key": 1,
                    "doc_count": 3,
                    "score": 0.75,
                    "bg_count": 12
                }]
            }
        }]
    }
}

Specifically, the bg_count is wrong in the concurrent search case. In a Significant terms aggregation, bg_count is the number of docs with the term in the background set while doc_count is the number of docs with the term in the foreground set.
So at the top level, the background set is the number of docs in the index with the “class” field, so the bg_count should simply be the number of docs in the index. Code ref: https://github.com/opensearch-project/OpenSearch/blob/8eea7b986453271cd227a7b98ee09e70d5e28634/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/SignificanceLookup.java#L100
Background Set == All documents in the index with “class” field
At the inner level, the bg_count therefore refers to all of the documents in the background set with the given key in the “text” field. See how index is created here: https://github.com/opensearch-project/OpenSearch/blob/b9f04405d6dfabcc2b8e87a86d4f83b003b3ffe7/server/src/test/java/org/opensearch/test/search/aggregations/bucket/SharedSignificantTermsTestMethods.java#L94-L115
So for key “0” this is 5, and for key “1” this is 4. This value is retrieved through a querying the index from the search thread. Code ref:

https://github.com/opensearch-project/OpenSearch/blob/8eea7b986453271cd227a7b98ee09e70d5e28634/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/NumericTermsAggregator.java#L647-L656
https://github.com/opensearch-project/OpenSearch/blob/e3542011cd9584c354405ba6cbf9a31ced7bc5ce/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/GlobalOrdinalsStringTermsAggregator.java#L897

Just to crystalize the meanings of background and foreground sets, let’s break down the example in the ElasticSearch docs here: https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-bucket-significantterms-aggregation.html#_single_set_analysis
Here the outer level bg_count refers to the number of documents with the field crime_type in the background set, which is the entire index. The query is for force == British Transport Police, so doc_count represents the number of documents where the force field matches British Transport Police, also known as our foreground set.
Background Set == Number of crimes in the index
Foreground Set == Number of crimes where force == British Transport Police
So now looking at the inner level, bg_count is going to be the number of documents where crime_type == Bicycle theft in the Background set, while doc_count is going to be the number of documents matching the same condition in the Foreground set.
References:

https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-bucket-significantterms-aggregation.html#significantterms-aggregation-parameters
https://stackoverflow.com/questions/49928871/elasticsearch-significant-terms-aggregation-meaning-of-doc-count-and-bg-count
https://javadoc.io/static/org.elasticsearch/elasticsearch/7.16.3/org/elasticsearch/search/aggregations/bucket/terms/InternalSignificantTerms.Bucket.html#getSupersetDf()

During the buildAggregations phase, the bg_count is gathered per collector. In the non-concurrent search case, that means it will only happen once per shard, and then at the coordinator level reduce can just sum up the bg_count across all of the shards to get the total. However, in the concurrent search use case we will have a collector per segment slice, and then at the shard-level reduce these will get summed up, so we will end up with SEGMENT_SLICE_COUNT * BG_COUNT documents.
Small caveat here is that this will only happen for slices that actually have docs, so in our test example, if a slice does not have any documents that have “text” field “0” then that collector will not contribute to the BG_COUNT miscount.
Here is an example to illustrate this problem:
In a test run, if the buckets look like this
0 = {StringTerms@13132} "{"class":{"doc_count_error_upper_bound":0,"sum_other_doc_count":0,"buckets":[{"key":"1","doc_count":2,"sig_terms":{"doc_count":2,"bg_count":7,"buckets":[]}}]}}"
1 = {StringTerms@13133} "{"class":{"doc_count_error_upper_bound":0,"sum_other_doc_count":0,"buckets":[{"key":"0","doc_count":1,"sig_terms":{"doc_count":1,"bg_count":7,"buckets":[]}},{"key":"1","doc_count":1,"sig_terms":{"doc_count":1,"bg_count":7,"buckets":[]}}]}}"
2 = {StringTerms@13134} "{"class":{"doc_count_error_upper_bound":0,"sum_other_doc_count":0,"buckets":[{"key":"0","doc_count":2,"sig_terms":{"doc_count":2,"bg_count":7,"buckets":[]}}]}}"
3 = {StringTerms@13135} "{"class":{"doc_count_error_upper_bound":0,"sum_other_doc_count":0,"buckets":[{"key":"0","doc_count":1,"sig_terms":{"doc_count":1,"bg_count":7,"buckets":[]}}]}}"

Then there are 5 buckets total, 2 for key “1” and 3 for key “0”, so if you look at the results below:
Expected:
{
    "class": {
        "doc_count_error_upper_bound": 0,
        "sum_other_doc_count": 0,
        "buckets": [{
            "key": "0",
            "doc_count": 4,
            "sig_terms": {
                "doc_count": 4,
                "bg_count": 7,
                "buckets": [{
                    "key": 0,
                    "doc_count": 4,
                    "score": 0.39999999999999997,
                    "bg_count": 5
                }]
            }
        }, {
            "key": "1",
            "doc_count": 3,
            "sig_terms": {
                "doc_count": 3,
                "bg_count": 7,
                "buckets": [{
                    "key": 1,
                    "doc_count": 3,
                    "score": 0.75,
                    "bg_count": 4
                }]
            }
        }]
    }
}

Actual:
{
    "class": {
        "doc_count_error_upper_bound": 0,
        "sum_other_doc_count": 0,
        "buckets": [{
            "key": "0",
            "doc_count": 4,
            "sig_terms": {
                "doc_count": 4,
                "bg_count": 21,
                "buckets": [{
                    "key": 0,
                    "doc_count": 4,
                    "score": 0.39999999999999997,
                    "bg_count": 15
                }]
            }
        }, {
            "key": "1",
            "doc_count": 3,
            "sig_terms": {
                "doc_count": 3,
                "bg_count": 14,
                "buckets": [{
                    "key": 1,
                    "doc_count": 3,
                    "score": 0.75,
                    "bg_count": 8
                }]
            }
        }]
    }
}

We see that key “1” has 2x the expected count while key “0” has 3x.
Single set case for reference:
    "aggregations": {
        "sig_terms": {
            "doc_count": 4,
            "bg_count": 21,
            "buckets": [{
                "key": 0,
                "doc_count": 4,
                "score": 0.39999999999999997,
                "bg_count": 15
            }]
        }
    }

Solutions
Without a deeper understanding of the scoring mechanism, it would be best for a solution to fix the scores in the slice level rather than the coordinator level.
I have added a few possible solutions below, with solution 3 looking the most promising to me and I will be working on a PR for that.
1. Divide by the bucket count
Due to the caveat above, in sparse data cases if a bucket is not collected in a segment slice then it won’t contribute to the bg_count. That means we can’t just blindly divide these numbers by the slice count. We would need to figure out some way to get the bucket count per collector to see how to divide the bg_count.
2. Separate out the shard-level vs slice-level reduce logic
Slice level reduce: https://github.com/opensearch-project/OpenSearch/blob/064f265362f6ae8296bfeed09aaaf48be86e6a4d/server/src/main/java/org/opensearch/search/aggregations/AggregationCollectorManager.java#L70-L74
Add a flag to reduce context and then don’t sum up supersetDf here: https://github.com/opensearch-project/OpenSearch/blob/e3542011cd9584c354405ba6cbf9a31ced7bc5ce/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/InternalSignificantTerms.java#L286-L299
Nor supersetSize here: https://github.com/opensearch-project/OpenSearch/blob/e3542011cd9584c354405ba6cbf9a31ced7bc5ce/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/InternalSignificantTerms.java#L231-L236
3. Create a singleton SignificanceLookup that will only return backgroundFrequencies and supersetSize once per bucket
This seems to be the best solution to me because the sum bg_count will be correct going into the aggs level reduce so we won't need to modify that. Moreover, we're actually doing up to 2 * NUM_SLICES * NUM_BUCKETS extra queries to get the bg_count for every single bucket at the slice level, so this singleton solution will eliminate all but up to 2 of these extra queries.
Currently I am thinking we can use searchContext.isConcurrentSegmentSearchEnabled() here:
https://github.com/opensearch-project/OpenSearch/blob/9b9158e4cb692dfe04315225ce2b5f9c6c453307/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/SignificantTermsAggregatorFactory.java#L290-L295
and then further use the queryShardContext to determine when to rest the Singleton.
Note that we will most likely also need to do this in SignificantTextAggregatorFactory here:
https://github.com/opensearch-project/OpenSearch/blob/8eea7b986453271cd227a7b98ee09e70d5e28634/server/src/main/java/org/opensearch/search/aggregations/bucket/terms/SignificantTextAggregatorFactory.java#L157
However I'm seeing assert codec failures in that execution path as mentioned in https://github.com/opensearch-project/OpenSearch/issues/8509#issuecomment-1636306061. Nevertheless the same solution should be applicable in both cases here.
I still need to hash out some of the finer details here, such as how to reset the Singleton class per bucket, however conceptually this solution is both the most intuitive and least intrusive.

Tagging @reta @sohami to see if you have any additional thoughts while I work on a PR for solution 3

After taking a look at this, I think solution 2 is actually much more straightforward. ReduceContext already has a isFinalReduce() method that is true for the coordinator level reduce. We can use that method in the places mentioned above and then we can leave the slice level query optimization for later. Will open a PR with this implementation shortly.


[Bug] robocop is not handling arguments correctly when run with pre-commit

What happened?
When having following pre-commit configuration for robocop:
  - repo: https://github.com/MarketSquare/robotframework-robocop
    rev: 964bef5a7cf5a8ab5a720bd79fe3b9acd1e96a1e
    hooks:
      - id: robocop
        args: [
             "-A=config/robocop.cfg", # Link to config file
             "--ignore ./rf-tests",
             ]

I would expect the robocop to treat the arguments the same way as if they were given from command line.
Instead, I get errors like this for every argument:
Error: File "--ignore rf-tests" does not exist

I'm aware that robocop should be able to read config from .robocop or pyproject.toml but in my case I'm hoping to read it from argument file located in project config folder f.g. config/robocop.cfg.
(I'm comparing robocop to pylint (when running them with pre-commit 😀)
Operating System
Windows
Robocop version
from master

Can you try to separate arguments?
[..., "--ignore", "your path"]

Obviously, that is how to do it.
Works perfectly.
One question though:
have a file under: [currentdir]\resources\taf\sims\pdsimulator\fluke_prosim8.resource
How to use --ignore to ignore everything under resources folder?
(it did work when part of the filename was used: fluke*.resource)
Like, I would not want to lint any files under that folder even if that file would be part of files checked by pre-commit (i.e in stage).
--ignore-default did seem to do the trick, though.

Looking at documentation, --ignore should work with path as well. I did develop it but I don't remember the details, I need to verify it. Looking at implementation the --ignore is less lenient (it is using match instead of search):
   def is_path_ignored(self, path):
        for pattern in self.ignore:
            if path.match(pattern):
                return True
        if self.ignore_default:
            match = self.ignore_default.search(str(path))
            return bool(match and match.group(0))
        return False

But I see possible bug here - it should be pattern.match(path) and not how it is (path.match(pattern)). I will test it.
And there is also additional option - robocop will also read .gitignore files in the directory and ignore files matching those from gitignore. But it's probably not useful for you because any file in the gitignore will be also not stored in the git.

I tested it and I think it works properly.
I tried to run --ignore resources* and it ignored all files below that directory.
@jylind Do you still have this issue? If yes, can you provide how do you execute Robocop and what output do you get?

The initial issue was because of the invalid configuration - args needs array of strings, with every value and option as separate element. It was resolved in later posts. And for the later question, --ignore-default was an answer (it allows to ignore whole directories, however you need to be remember to include the default values if you want to ignore .git directory).

So I guess we can close this issue, right?

Yes

Sorry for late answer.
Can't remember in what context I was having that problem (not the original problem for this issues, though) with --ignore.
--ignore seems to work but it seems to be too eager for what I would have wanted.
So our folder structure:
root/
...
    rf-tests/testsuite
...
    testsuite
...

when running command:

root:> robocop --ignore testsuite

both testsuite folders are ignored but I only want to ignore testsuite from root not rf-tests/testsuite folder.

@jyrlindr Thanks for your answer.
It's an interesting case and I must admit I don't know how to do it using current implementation in Robocop.
Maybe I'm doing something wrong but I tried both --ignore and --ignore-default options with different values and I can't exclude the testsuite from the root.
@bhirsz Do you have an idea how it can be achieved?
I think our --ignore implementation ignores too much and does not align with the common way how glob patterns work.

I don't know how it can be achieved using current configuration - I only know that the matching is done using pathlib match() method in is_path_ignored method. However the pathlib match documentation says:

Maybe using --ignore with absolute path would work (but then we need absolute path which is not convenient). Or introduce part of the path so it filter out different paths: --ignore root/testsuite*
I believe it should work that --ignore only matches filename, not paths and --ignore-default can allow ignoring the paths.


Update "View Job Openings" URL

Fix #322.

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


file: part3/chunking_introduction

[ ] Update obsolete commands
[ ] Update example data set so it follows the story of part3
[ ] Put visibilities of 'Zarr' in table of contents and/or title
[ ] Fix warning on hdf.SingleHdf5ToZarr
[ ] If Xarray backend for kerchunk is ready we will update how we load kerchunk  (https://github.com/fsspec/kerchunk/issues/360)
[ ] specify version of kerchunk and fsspec in enviroment file?

The data we use to show chunking explanation could be better if we can use data from former 3 notebooks.

The title of this section also include 'zarr'.

on pangeo-eosc cluster
fsspec 2022.7.1
kerchunk '0.0.7'
on my Mac-m1 local enviroment
fsspec 2023.3.0
kerchunk 0.1.2
This works, but with my new local enviroment (same version as https://github.com/pangeo-data/pangeo-docker-images/blob/master/pangeo-notebook/packages.txt)
fsspec==2023.10.0
kerchunk==0.2.1
Kerchunk shows warning and does not translate all informations

UserWarning: The following excepion was caught and quashed while traversing HDF5
'SingleHdf5ToZarr' object has no attribute 'decoders'
Traceback (most recent call last):
File "/Users/todaka/micromamba/envs/bids23/lib/python3.11/site-packages/kerchunk/hdf.py", line 262, in _translator
filters = self._decode_filters(h5obj)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/todaka/micromamba/envs/bids23/lib/python3.11/site-packages/kerchunk/hdf.py", line 246, in _decode_filters
f" not supported by kerchunk., supported filters are {self.decoders.keys()}"
^^^^^^^^^^^^^
AttributeError: 'SingleHdf5ToZarr' object has no attribute 'decoders'
warnings.warn(msg)

on pangeo-eosc cluster fsspec 2022.7.1 kerchunk '0.0.7'
on my Mac-m1 local enviroment fsspec 2023.3.0 kerchunk 0.1.2 This works, but with my new local enviroment (same version as https://github.com/pangeo-data/pangeo-docker-images/blob/master/pangeo-notebook/packages.txt) fsspec==2023.10.0 kerchunk==0.2.1
Kerchunk shows warning and does not translate all informations

UserWarning: The following excepion was caught and quashed while traversing HDF5
'SingleHdf5ToZarr' object has no attribute 'decoders'
Traceback (most recent call last):
File "/Users/todaka/micromamba/envs/bids23/lib/python3.11/site-packages/kerchunk/hdf.py", line 262, in _translator
filters = self._decode_filters(h5obj)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/todaka/micromamba/envs/bids23/lib/python3.11/site-packages/kerchunk/hdf.py", line 246, in _decode_filters
f" not supported by kerchunk., supported filters are {self.decoders.keys()}"
^^^^^^^^^^^^^
AttributeError: 'SingleHdf5ToZarr' object has no attribute 'decoders'
warnings.warn(msg)

Fixed with https://github.com/fsspec/kerchunk/issues/382
kerchunk==0.2.2

Update data set, and xarray kerchunk backend will be later

https://github.com/pangeo-data/pangeo-openeo-BiDS-2023/pull/22/

https://github.com/pangeo-data/pangeo-openeo-BiDS-2023/pull/41

we will keep kerchunk, and to do so we will go back to original NVDI example.


Update mediapipe version?

Hello @homuler , In order to upgrade my mediapipe version for this unity project.
If I take a pull on my main repository of mediapipe (the google one). Would that work in my case?
If not, how do you suggest upgrading the mediapipe version?
I'm asking this because in the new update they released world coordinates of landmarks. Try it out here https://codepen.io/mediapipe/full/jOMbvxw

MediaPipe v0.8.6 is supported now.
https://github.com/homuler/MediaPipeUnityPlugin/releases/tag/v0.6.2

I tried to run the build.py commands separately for this release. And I'm getting these logs
My current system is Windows10.


Changes to interpolation for innundation risk calculations.

Interpolation procedure will now ignore data points for which data is unavailable (equal to -9999.0). An additional test has been created to test these cases.

This contribution guideline document explains how to resolve DCO failures: https://github.com/os-climate/OS-Climate-Community-Hub/blob/main/CONTRIBUTING.md


PATCHing an entry with multiple key/value pairs only applies the first pair

Here's what exists in the database:
curl -X GET http://localhost:8082/depositories/eb7c9914cfc21eaf2ba2dbd517a1910a17a2b039 -H 'x-user-id: taylor' -H "Content-Type: application/json"

{"id":"eb7c9914cfc21eaf2ba2dbd517a1910a17a2b039","bankName":"BBT","holder":"My Company,llc","holderType":"individual","type":"checking","routingNumber":"051504597","accountNumber":"0001027028","status":"unverified","metadata":"Payroll","created":"2019-06-05T01:22:33Z","updated":"2019-07-03T22:41:34Z"}

But when trying to update several key/value pairs of the entry, only the first one gets applied:
curl -X PATCH http://localhost:8082/depositories/eb7c9914cfc21eaf2ba2dbd517a1910a17a2b039 -H 'x-user-id: taylor' -H "Content-Type: application/json" --verbose --data '{"bankName":"New bank name","holder":"Taylor","type":"savings"}'

*   Trying ::1:8082...
* TCP_NODELAY set
* Connected to localhost (::1) port 8082 (#0)
> PATCH /depositories/eb7c9914cfc21eaf2ba2dbd517a1910a17a2b039 HTTP/1.1
> Host: localhost:8082
> User-Agent: curl/7.65.1
> Accept: */*
> x-user-id: taylor
> Content-Type: application/json
> Content-Length: 63
>
* upload completely sent off: 63 out of 63 bytes
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Content-Type: application/json; charset=utf-8
< Date: Wed, 03 Jul 2019 22:42:32 GMT
< Content-Length: 328
<

{"id":"eb7c9914cfc21eaf2ba2dbd517a1910a17a2b039","bankName":"New bank name","holder":"My Company,llc","holderType":"individual","type":"checking","routingNumber":"051504597","accountNumber":"0001027028","status":"unverified","metadata":"Payroll","created":"2019-06-05T01:22:33Z","updated":"2019-07-03T22:42:32Z"}

* Connection #0 to host localhost left intact

I'm using the latest docker images

Whoops! I filed a patch to fix that.

v0.5.2 should fix this bug @tgunnoe

Awesome, glad it wasn't just me!


PR: Split all plugins into their own modules

Fixes #4591.
With this PR all our internal plugins will have the same structure as the ones developed outside Spyder (e.g. spyder-terminal, spyder-notebook, etc).
We hope this will help outside contributors to better understand how Spyder works and simplify the development of each plugin too.

@ccordoba12 If you are moving the files anyway, you are harmonizing them all to LF newlines right?

It'll break merging with 3.x, so no, sorry.

Oh, I thought it would do that anyway because of the moves so the LF change wouldn't make things worse, but I guess not. Maybe in the final stages of the 4.0 release process when all the non-trivial PRs are in and the 3.x branch is frozen we can do it for everything, and harmonize the copyright headers at the same time (to be in a standard format, with a beginning copyright year and refined text)? If not, it would have to be for Spyder 5 or after ofc.

Ok, this is going in!


[Feature request] Add feature for combos

Hello,
thanks for all the effort you have put into your great software!
I have an existing KMK configuration for a retro computer keyboard that I am trying to port to POG. The idea is to make it easier for end users to modify the key mapping depending on their use cases. I have a few keys that need special handling due to how the keyboard is designed.
For example, to get Delete on this old keyboard you are supposed to press Shift + Backspace. To get KMK to send a normal Delete key press I hade to use combos - otherwise it would either send a delete key with an extra Shift, or no Shift when I wanted it for other keys.
The code that ended up doing what I want looks like this:
combos.combos = [
    # Add DEL to Backspace
    Sequence((KC.LSFT, KC.BSPC), KC.DEL, fast_reset=False),
    Sequence((KC.RSFT, KC.BSPC), KC.DEL, fast_reset=False),
]

Is there a way to configure the same feature with POG? If it isn't, could it please be added?

Sorry, I did a bad job searching the other issues before creating a new one. Now I found #16 , so this can probably be closed.


Suggest more sources for auto imports

Hi! The feature that allows to import things automatically while typing is super cool! However, in my opinion, there is one case where it can be improved even further.
I experienced the issue when using the feature when using my library (as a third-party module): https://github.com/Yura52/zero
The library contains the zero/all.py module that is designed specifically for importing things without producing many lines. However, the auto-import feature suggests imports only from submodules (such as zero.io), where entities are actually implemented. Instead, I would prefer importing them from zero.all.

currently, in the next release, "from m import *" form is not considered, but "from m import t as t" is considered when choosing what to show in completion for auto import.
but let me play with it and see how much it affects our auto-import experiences.

Note that the new heuristic wouldn't help for this case, as zero.all is not shorter (in length or module dots) than zero.io. Usually re-exports are things like "np.array is actually np.core.some._long_compiled_module", for example.

we should both actual definition and best 1 import alias, so zero.all will at least show up.

I agree with this issue. I think we should certainly improve it. Hope this issue priority could be more urgent.

I agree with this issue. I think we should certainly improve it. Hope this issue priority could be more urgent.

now, "python.analysis.indexing" became a real option. Set "python.analysis.indexing": true to enable indexing which will improve auto-import experience.
Also, new hidden option "python.analysis.packageIndexDepths" is added which can be used as below to control how much get indexed.

"python.analysis.packageIndexDepths": [['sklearn', 2],['matplotlib', 2]],

by default, only 1 level module of third party libraries will be indexed, but now, you can use ["package name", module levels to be scanned] to control that.
if you put "" to package name, then that will affect all packages.
for your example, ["zero", 2] will make all.py to be picked up.
thank you

This issue has been fixed in version 2022.1.3, which we've just released. You can find the changelog here:  CHANGELOG.md


Should the top matches be paired?

Say I'm compare document A with documents A B C D and E; and document B is the most related. I would expect that comparing document B with A B C D and E would result in document A being the most related (I'm assuming the absolute match is somehow excluded). I'm not finding this to be the case. Is this to be expected?

I don't think it is expected with tfidf. Note this library ultimately ends up using https://github.com/NaturalNode/natural/blob/master/lib/natural/tfidf/tfidf.js internally.


Upgrade to 0.6 failed!

Log show :
2015/03/23 15:09:08 [I] Gogs: Go Git Service <IP_ADDRESS>9 Beta
2015/03/23 15:09:08 [I] Log Mode: File(Trace)
2015/03/23 15:09:08 [I] Redis Enabled
2015/03/23 15:09:08 [I] Memcache Enabled
2015/03/23 15:09:08 [I] Cache Service Enabled
2015/03/23 15:09:08 [I] Session Service Enabled
2015/03/23 15:09:08 [I] Migration: refactor access table to use id's
2015/03/23 15:09:08 [install.go:65 GlobalInit()] [E] Fail to initialize ORM engine: migrate: do migrate: drop access table: Error 1051: Unknown table 'access'

Shoud I add 'access' table first?

Hi, how old was your previous version?

It's <IP_ADDRESS>0 Beta

Do these error occur when you first time run 0.6.0? Or you already ran multiple times to then still see this error?

Already ran multiple times!
And collaborators data was lost after degrade to 0.5.

collaborators now uses collaborator table, see if this table exists?

It's not exists, I only see collaboration table.

Oops, sorry for my typo. So is collaboration table empty?

It's not empty. And I see access table . Why log show that it's not exists?

Is access table empty?

Yes. I am sure that the table was not exists after I ran and failed.

Can you dump log/gogs.log file here? There may be other error messages could be helpful.

Sorry, I had dumped . No more data .

OK, please backup your database now, and re-run Gogs(create access table if it does not exist).
BTW, what is your database?

Amazing, I success.
I use MYSQL.

@decker502 cool, please check if your organization, collaboration relations are still correct?

@Unknwon Yes, all is correct.
Thank u very much!

@decker502 no problem!
I haven't found out why MySQL fails because we mostly work on MySQL. But in your case all access data were already regenerable from collaboration.

This is  really  a problem.
Another problem always perplex me. I as admin can't access and manage other private repos.
It should be to my understanding.

Another problem always perplex me. I as admin can't access and manage other private repos.
It should be to my understanding.

Right, someone posted issue about this.

OK.


parameterize restart on change for the main config

This will allow us to restart the agent conditionally when $config_hash changes. This does not affect reloads when service, check and watch configs change.
It defaults to the current behavior of restarting on config change.
In my experience its never a good idea for puppet to restart consul server. More often than not you will run into a situation where the cluster doesn't know who the leader is. Its best to restart the agent on server in a more controlled fashion.

Thanks!


add starbucks website

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes:  #483

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Issue Linked

@piyushsonawane07

Reopen this PR when ready.