Builder unit tests refactoring

- What I did
I created couple unit test helpers and applied them to builder unit tests.
- How I did it
By extracting common functionalities to helper functions.
- How to verify it
Check builder/utils_test.go. Remaining changes are done by applying functions from this file to builder unit tests.
- A picture of a cute animal (not mandatory but encouraged)
here you are:

Signed-off-by: Tomasz Kopczynski<EMAIL_ADDRESS>

@tkopczynski there is trivial vet error in CI

@LK4D4 Yes, somehow it slipped through... It should be ok now

LGTM

LGTM 🐰


Coverage doesn't show files with 0% coverage

nyc --reporter=json npm test
would not generate coverage for all files.
For javascript files with no tests, (therefore no coverage), it will not show in the test reports.
Please use the template provided below, when reporting bugs:
Expected Behavior
nyc --reporter=json npm test should show coverage for all files, even if test doesn't exist.
Observed Behavior
nyc --reporter=json npm test should show coverage for files with no test.
Bonus Points! Code (or Repository) that Reproduces Issue
create some js code in your project with no tests (no coverage)
then do nyc --reporter=json npm test
and nyc report
The files with no test will not show
Forensic Information
Operating System: the operating system you observed the issue on.
On a mac
node --version
v7.0.0
npm --version
3.10.8
Thanks for your time.

Hey @jennylia, you can get this behavior by adding --all to your command:
nyc --all --reporter=json npm test
If this doesn't do the trick, please feel free to re-open this ticket 👍

@bcoe it does the trick, but in this case, it will also generate coverage for various .spec.js file, even with this in my .npmrc:
"exclude": [
    "src/**/*.spec.js",
    "node_modules"
  ]

@jennylia make sure you're on the newest version of nyc (9.0.1) and give this a shot:
"exclude": [
    "**/src/**/*.spec.js"
  ]

All the test files are still included in the nyc report.

@bcoe Using "nyc": "^10.3.2" and it still doesn't work.
  "nyc": {
    "check-coverage": true,
    "lines": 99,
    "statements": 99,
    "functions": 99,
    "branches": 99,
    "sourceMap": false,
    "instrument": false,
    "cache": true,
    "reporter": [
      "html",
      "text-summary"
    ],
    "all": true,
    "exclude": [
      "**/*.test.js",
      "**/node_modules/**"
    ]
  }

@mrchief --all works for most common cases, my guess would be their's an edge-case in your configuration. Could you please open a new issue, with a link to code that reproduces the issue if possible.

@bcoe I found #594. I'll continue discussion there.

I had this error when I converted to ES6 modules. Reverting to CommonJS fixed the issue for me.


Tensorflow estimator script mode not working if we pass HyperParameters as part of TrainingStep

Hello,
I was trying to use Tensorflow estimator in script mode and I was trying to pass the hyper-parameters as part of stepfunctions.steps.TrainingStep as follows:

But, the training job failed with the following error

Upon investigation I found out that other hyper-parameters that the estimator attaches on its own were not going (ex - 'sagemaker_program', 'sagemaker_submit_directory' etc.)
For example, the following image is the set of hyper-parameters for the job when we give it as part of TrainingStep:

But, if we give the hyper-parameters as part of the estimator:

The reason why I'd want to give hyper-parameters as part of the TrainingStep is that I'd be able to use stepfunctions.inputs.ExecutionInput in order to provide inputs dynamically that might be useful in the script.

fixed through #144 which enables the capability to specify hyperparameters in the TrainingStep constructor and merges them with hyperparameters established in the estimator.


Add trigger type to CodeActionContext

Part of #33555
Problems
Extensions currently do not know how a code actions request was triggered. Code actions may be requested automatically by vscode or when the user requests them
Proposal
Add a trigger type to CodeActionContext:
export interface CodeActionContext {
    readonly diagnostics: Diagnostic[];
    readonly trigger: 'auto' | 'manual';
}

auto: Code actions requested by VS Code automatically
manual: Code actions requested by user / command

Extensions could use this to determine which code actions they want to compute / show. For example, an extension may decide to only compute refactorings when a user requests them
// @jrieken

or example, an extension may decide to only compute refactorings when a user requests them

Not yet sure about that because one user might like that and another might not like that. I would solve the "noise" problem by richer code action information


Limit list of authorized characters in basic secret

Currently we can specify the characters we want to be used in a secret of type token, the example below is from the doc:
secret: {
    password: {
        type: "token"
        params: {
            length: 16
            characters: "abcdedfhifj01234567890"
        }
        data: token: ""
    }
}

That should be great to have the same behavior for secret of type basic as well. Basic secret are great to create database username/password on the fly but because the chars list does not prevent special chars it cannot be used to generate postgres username/password for instance.
I think that could be great if the chars restriction is available for basic secret as well.

Just an additional note ;)
Could the list of characters be also represented with a-z, A-Z, 0-9 so we can use "a-zA-Z0-9" for aphanumerical characters ?

Merged the change to parameterize basic auth secrets, will be adding the "a-zA-Z0-9" enhancement in a follow-up!

Tested with acorn version - v0.9.0-rc1-31-gbdd8d407+bdd8d407
Able to limit list of authorized character for basic secret's username and password.

Deployed secrets with type basic:

secrets: {
    password: {
        type: "basic"
        params: {
            usernameLength: 16
            usernameCharacters: "abcdedfhifj01234567890"
            passwordLength: 16
            passwordCharacters: "abcdedfhifj01234567890"
        }
    }
}

This results in secrets being created with the provided size and character set for both username and password fields.
acorn secrets reveal testnew.password
NAME               TYPE      KEY        VALUE
testnew.password   basic     password   0eb0fa93fehhf7e8
testnew.password   basic     username   04cadd2d5dd6fead

Another example with more restricted character set and different ones for username and password fields also work as expected.

secrets: {
    password: {
        type: "basic"
        params: {
            usernameLength: 10
            usernameCharacters: "xyz123"
            passwordLength: 7
            passwordCharacters: "atyu9"
        }
    }
}

acorn secret reveal testres.password
NAME               TYPE      KEY        VALUE
testres.password   basic     password   a9tytyy
testres.password   basic     username   313z2323yz

Just merged support for "character range" syntax sugar: #2250

Tested with Acorn version - v0.9.0-rc1-37-g2be8a309+2be8a309
Able to restrict character set for username and password by specifying  value like -A-Za-z0-9 and they work as expected.
@njhale When specifying format like a-Z or Z-a we only get secrets generated with a and Z . Wanted to confirm that this is as intended . I am able to get all alphabets (lower and upper case) when specifying A-z which is as expected.

When specifying format like a-Z or Z-a we only get secrets generated with a and Z . Wanted to confirm that this is as intended . I am able to get all alphabets (lower and upper case) when specifying A-z of z-A

yup -- looks like you ran into an edge-case that I didn't account for. Working on a fix now.

Here's the fix: https://github.com/acorn-io/runtime/pull/2255

For reference, that fix does the following:

Treat mixed case and backwards ranges (e.g. A-z, 0-A, z-a) as
individual characters in the given set. This simplifies the UX and
makes ranges easier to reason about.

Tested with acorn version - v0.9.0-rc1-43-gbe8054c6+be8054c6
Mixed case and backwards ranges are now considered as individual characters in the given set.
Using z-A , Z-a,a-Z,A-z for usernameCharacters and passwordCharacters when creating secrets results in individual characters in the given set as being allowed characters.
This is true for  backwards ranges for numbers as well like 9-0 , 6-3.

a-z,A-Z, 0-9 syntax is not yet implemented for the token type secrets.

Hi,
I have created the following secret:
secrets: {
    user: {
        type: "basic"
        params: {
            usernameLength: 7
            usernameCharacters: "a-z"
            passwordLength: 10
            passwordCharacters: "a-Z0-9"
        }
        data: {
            username: ""
            password: ""
        }
    }
}

Looking at the generated password, I get the following:
$ acorn secret reveal mgo.user mgo.admin
NAME        TYPE      KEY        VALUE
mgo.user    basic     password   6Z4670Z-94
mgo.user    basic     username   pobaahs

On another secret I got this content: "a524278-48"
Is there anything I'm doing wrong ?

@lucj a-Z will only translate to "a", "-","Z" since this range has mixed case which is not supported. Instead of using "a-Z0-9" using "A-Za-z0-9" will result in all lower case/uppercase  alphabets and numbers.

My bad I misread the above comment. Thanks @sangee2004


Cannot read property 'createTransformer' of undefined

🐛 Bug Report
After upgrading from ts-jest version 26.5.6 to version 27.0.3 on nom run test I'm getting the below error:
ERROR: Cannot read property 'createTransformer' of undefined
STACK: TypeError: Cannot read property 'createTransformer' of undefined
    at ConfigSet._setupConfigSet

To Reproduce
Upgrading from ts-jest version 26.5.6 to version 27.0.3
Expected behavior
To be able to run tests.
Debug log:

# content of ts-jest.log :
0 verbose cli [
0 verbose cli   'C:\\Program Files\\nodejs\\node.exe',
0 verbose cli   'C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
0 verbose cli   'run',
0 verbose cli   'test'
0 verbose cli ]
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 timing config:load:defaults Completed in 1ms
4 timing config:load:file:C:\Program Files\nodejs\node_modules\npm\npmrc Completed in 1ms
5 timing config:load:builtin Completed in 1ms
6 timing config:load:cli Completed in 2ms
7 timing config:load:env Completed in 0ms
8 timing config:load:file:C:\Users\user\Development\project\.npmrc Completed in 0ms
9 timing config:load:project Completed in 1ms
10 timing config:load:file:C:\Users\user\.npmrc Completed in 0ms
11 timing config:load:user Completed in 0ms
12 timing config:load:file:C:\Users\user\AppData\Roaming\npm\etc\npmrc Completed in 0ms
13 timing config:load:global Completed in 1ms
14 timing config:load:cafile Completed in 0ms
15 timing config:load:validate Completed in 0ms
16 timing config:load:setUserAgent Completed in 1ms
17 timing config:load:setEnvs Completed in 0ms
18 timing config:load Completed in 7ms
19 verbose npm-session 9b21760416926f29
20 timing npm:load Completed in 14ms
21 timing command:run-script Completed in 79278ms
22 verbose stack Error: command failed
22 verbose stack     at ChildProcess.<anonymous> (C:\Program Files\nodejs\node_modules\npm\node_modules\@npmcli\promise-spawn\index.js:64:27)
22 verbose stack     at ChildProcess.emit (node:events:378:20)
22 verbose stack     at maybeClose (node:internal/child_process:1067:16)
22 verbose stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:301:5)
23 verbose pkgid<EMAIL_ADDRESS>24 verbose cwd C:\Users\user\Development\project
25 verbose Windows_NT 10.0.19042
26 verbose argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "test"
27 verbose node v15.8.0
28 verbose npm  v7.5.1
29 error code 1
30 error path C:\Users\user\Development\project
31 error command failed
32 error command C:\windows\system32\cmd.exe /d /s /c jest --config jest.config.js
33 verbose exit 1

envinfo
System:
    OS: Windows
    Node version: `15.0.0`

Npm packages:
    jest: 27.0.4
    ts-jest: 27.0.3
    typescript: 4.3.2
    babel: 6.23.0

jest 26 goes with ts-jest 26, jest 27 goes with ts-jest 27. Please keep the major version of those 2 together :)

Also don’t forget to clear Jest cache :)

jest 26 goes with ts-jest 26, jest 27 goes with ts-jest 27. Please keep the major version of those 2 together :)

My bad, as I wrote on the begging of the issue, the error came up after we upgraded to ts-jest version 27.0.3

remove node_modules, reinstall deps and clear Jest cache should help.

Same thing, we initially got the error on the upgrading dependency pr (to v 27.0.3)

that means something in your project still refers to ts-jest 26, or something creates the mismatch. If you simply create a plain project, it definitely works.

that means something in your project still refers to ts-jest 26, or something creates the mismatch. If you simply create a plain project, it definitely works.

We run lots of dependencies prs every day, they all work, I think I'm missing something here
Could be it's related to our jest.config configurations?
module.exports = {
  preset: 'ts-jest',
  cacheDirectory: 'node_modules/.cache/jest',
  collectCoverage: true,
  collectCoverageFrom: ['websrc/**'],
  coveragePathIgnorePatterns: ['<rootDir>/node_modules/'],
  coverageReporters: ['text-summary', 'html'],
  testRegex: 'tests/.*',
  testPathIgnorePatterns: [
    '<rootDir>/webtests/studies/',
    '<rootDir>/webtests/fixtures/'
  ],
  modulePaths: ['<rootDir>'],
  testEnvironment: 'node',
  transform: {
    '^.+\\.tsx?$': 'ts-jest',
    '^.+\\.[tj]sx?$': 'babel-jest'
  },
  globals: {
    'ts-jest': {
      babelConfig: true
    }
  }
}

Jest config is correct.
The error is typically that Jest doesnt load the correct ts-jest version. It can be that in your node_modules, there is another ts-jest which is loaded by Jest.

You can try to delete node_modules/.cache/jest which is the Jest cache folder.
Do you use any test libraries?
Are you using any frameworks like Vue or React?

Jest config is correct.
The error is typically that Jest doesnt load the correct ts-jest version. It can be that in your node_modules, there is another ts-jest which is loaded by Jest.

You can try to delete node_modules/.cache/jest which is the Jest cache folder.
Do you use any test libraries?
Are you using any frameworks like Vue or React?

I'll try
Other than Jest? No
Yes, we use React

I suspect the problem might come from React itself.

I suspect the problem might come from React itself.

What on the upgrade may trigger that?

I see you are using babel-jest too, make sure it is also v27
27.0.2

I have an example project here, hope it helps https://github.com/ahnpnl/react-ts-jest-babel

Thank you.
I have that: "@types/jest": "26.0.23" might that be an issue?

Hmm, that is just a typing package, the error you got should not be related to that

I encountered this error today with the addition of babelConfig: true. When I did not have that setting, ts-jest works fine.


Update version of Slim in composer.json

Unable to use your package because the slim requirement is outdated.

The requirement is fine because this code won't work with Slim 3. There is apparently a working fork for Slim 3 here.


No support for MARS (backup management type) in az backup container/item list

This is autogenerated. Please review and update as needed.
Describe the bug
Result is empty.
Command Name
az backup container list
Errors:
To Reproduce:

az backup container list -g {} -v {} --debug
msrest.http_logger : Request URL: 'https://management.azure.com/Subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.RecoveryServices/vaults/xxx/backupProtectionContainers?api-version=2016-12-01&$filter=backupManagementType%20eq%20%27AzureIaasVM%27%20and%20status%20eq%20%27Registered%27'

Expected Behavior
Two containers as result
Environment Summary
Windows-10-10.0.14393-SP0
Python 3.6.6
Shell: cmd.exe

azure-cli 2.0.64

Additional Context

Currently, CLI for Azure Backup is only supported for Azure VMs. We are focussing on adding support for other Azure workloads like Azure File Share, SQL in Azure VM backups. Support for on-premise agents like MARS is currently not in THIS semester's roadmap i.e., until CY, 2019 end. Given that we have the support in PS, is this a blocker? If valid business justification is provided, I will see if we adjust the priority.

I've found another way with REST API.
In PS this also doesn't work.

That's correct @Wand3rvogel PS is not supported either because Central management for MARS is not a GA product. (i.e. syncing Backup items, jobs, alerts, etc. to Azure portal) We will add support for MARS Central management in CLI and PS when these features get to GA. Until then, can you please work around this using REST API?

@TAdityaAnirudh I do it already.

@Wand3rvogel: Thanks for the update. You cannot get backup items for MARS in PS but you can get containers for MARS. We do not have plans to onboard MARS to CLI in the current CY. I will re-visit this next year and since you are unblocked using REST API, I will close this issue.
Added to backlog and closing this since this is not taken up in this CY, 2019


Add Swift Package Manager support

Hi,
This PR adds Swift Package Manager support to this library.
Close #23

@anatoliyv @woxtu Is this something that can be merged?

@anatoliyv @woxtu Is this something that can be merged?

please merge this PR when you can. Is one of the few missing from my project

Do you guys hear that? It's a crowd chanting!
"Let's Go AssistantKit Developers; Let's Go!"
thump
thump
thump thump
thump
✺◟( ͡° ͜ʖ ͡°)◞✺ ✺◟( ͡ ͜ ʖ ͡ )◞✺ ✺◟(◕‿◕✿)◞✺
✺◟( ͡ ͜ ʖ ͡ )◞✺ ✺◟( ͡° ͜ʖ ͡°)◞✺ ✺◟( ͡ ͜ ʖ ͡ )◞✺
✺◟(◕‿◕✿)◞✺ ✺◟( ͡ ͜ ʖ ͡ )◞✺ ✺◟( ͡° ͜ʖ ͡°)◞✺
We got this!!! Let's bring it home! Let's merge this PR :D


Add GD Menu Theme (Beatbox)   by Abdul

Song Name
GD Menu Theme (Beatbox)
Artist Name
Abdul
Source
Youtube
Youtube Link
0EDWuur2TTQ
Direct File Link
No response
Song ID
584131
Start Offset [ms]
100
Extra
Created from Auto Nong.
Created for level: 12927241
Downloaded: Yes

bNndndms


[EnhanceTextSelection] Make expandTextDivs more efficient by updating all styles at once instead of piecewise

I intended to provide proper benchmarking results here, as outlined in https://github.com/mozilla/pdf.js/wiki/Benchmarking-your-changes, but after wasting a couple of hours over the weekend getting weird results I gave up.
It appears that there's a lot of, i.e. way too much, variance between subsequent runs of text tests for the results to be meaningful.
(Previously I've only benchmarked eq tests, so I don't know if the text tests has never worked well or if it's a newer problem. For reference, please see the results of back-to-back benchmark runs on the current master with a very simple manifest file: https://gist.github.com/Snuffleupagus/eec3acc22f519eddce6404be883b6960.)
Instead I used console.time/timeEnd in appendText and expandTextDivs to be able to compare the performance with/without the patch. The entire viewer was (skip-cache) reloaded between measurements, and the result are available here: https://gist.github.com/Snuffleupagus/33fc971b653e6524bc889216ff95499c.
Given the troubles I've had with benchmarking, I've not yet computed any statistics on the results (e.g. mean, variance, confidence intervals, and so on).
However, just by looking at the data I think it's safe to say that this patch first of all doesn't seem to regress the current performance. Secondly it certainly looks very likely that this patch actually improves the performance, especially for the one-glyph-per-text-div case (cf. issue 7224).
Re: issue #7584.
/cc @timvandermeij

This change is

/botio test

Comments have been addressed.
/botio test

/botio-linux preview

/botio makeref

Awesome work!


fix(cli): delete the obsolete function "gas config(v1.9.0)"

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.086%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5266

Relevant Lines:
6338

💛  - Coveralls


New QC tests

will be pushing attempts at the more straightforward of the new QC tests to this PR (more sophisticated ones will get their own feature branch).

3f2b658:
Implements the EN_constant_value test from #49; testing validates that this implementation:

flags every element in a profile where 9/10 temperature measurements are the same
does not flag anything in a profile where 8/10 measurements are the same
does not flag anything in a profile where 9/10 measurements are the same, but those measurements are spread over less than 100 m.

Note only temperature (and not salinity) is examined here, per the other qc tests so far.

Hi @BillMills - looks good! I'm not sure what would happen if there were missing data values, though. It may just work but I think it would be good to include the check that there are valid temperature and depths at the levels as done e.g. in Argo gradient check just in case it doesn't.

@s-good - agreed, implemented in b7be6f6; also, generalized testing profiles to support missing values; should include missing value validation for all qc tests. More to come shortly.

Argo_impossible_date doesn't have it's time check implemented yet, need to resolve #62 first.

This PR implements solutions for #49, #51, #52 and #53, including validation checks for all of the above. The only outstanding task is resolving #62, so we can finish implementing the time check in the impossible date test (#51) - otherwise this is mergable pending feedback.


Stores/Checkout Page embedded in iFrames

Description
Problem statement: when the PayPal checkout jsv4 tries to render the checkout experience from within an iFrame, it throws a "unable to delegate rendering" error and closes the pop-up.
There are 2 use cases to consider:
Use Case 1 being the merchant has ownership of the parent page. eg. for security reasons the merchant hosts the payment page on another domain. (included for completeness; aware of existing solve by including js in parent)
Use Case 2 being the merchant doesn't have ownership of the parent page. eg. Facebook embedded store. (unable to include js in parent)
The following is quick demo: https://borma6.wixsite.com/testpp
Affected browsers

[x ] Chrome
[x ] Safari
[x ] Firefox
[x ] Edge
[x ] IE
[x ] Chrome Mobile/Tablet
[x ] Safari Mobile/Tablet
[x ] Web View / Safari ViewController
[ ] Other

This is now resolved.


Add -p and -l to cue_consolidated_files

What
This adds -p to cue def to allow naming non-CUE files with a package name
Testing
Test yaml
$ cat test/test.yaml
here: there
outer: inner
build:
  spec:
    metadata:
      labels:
        this: thing
    containers:
    - name: test
      image: thatimage

Test Bazel
$ cat test/BUILD.bazel
cue_consolidated_files(
    name = "test",
    srcs = [
        "test.yaml"
    ],
    package_name = "environmentbuild",
    module = "//cue.mod",
)

Building my test
$ bazel build //test:test
INFO: Analyzed target //test:test (25 packages loaded, 190 targets configured).
INFO: Found 1 target...
Target //test:test up-to-date:
  bazel-bin/test/test.cue
INFO: Elapsed time: 0.296s, Critical Path: 0.11s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action

$ cat bazel-bin/test/test.cue
package environmentbuild

here:  "there"
outer: "inner"
build: spec: {
	metadata: labels: this: "thing"
	containers: [{
		name:  "test"
		image: "thatimage"
	}]
}

The repo builds
$ bazel-5.2.0 build //...
INFO: Analyzed 46 targets (22 packages loaded, 127 targets configured).
INFO: Found 46 targets...
INFO: Elapsed time: 0.281s, Critical Path: 0.01s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action

An empty -p acts the same as not passing a package name as far as i can tell in cue

Similarly for -l, if you specify an empty -l '' it will not be used and the resulting build with -l '#Env: produces
package environmentbuild

#Env: {
	here:  "there"
	outer: "inner"
	build: spec: {
		metadata: labels: this: "thing"
		containers: [{
			name:  "test"
			image: "thatimage"
		}]
	}
}

I have tested all variations for the two -l and -p and they seem to work appropriately

@seh can you take a look at this please

This is implemented in commit fdc58098c632600de83e05939740bac581e27fb1.


Undeclared variable "attribute_type"

I ran into an issue testing with Jina/DocArray and found a variable that is not declared.
Here is the error:
ERROR  weaviate/rep-0@67192 UnboundLocalError("local variable 'attribute_type' referenced before assignment")    [10/24/22 16:49:34]
        add "--quiet-error" to suppress the exception details
       Traceback (most recent call last):
         File ".local/lib/python3.8/site-packages/jina/serve/runtimes/worker/__init__.py", line
       182, in process_data
           result = await self._data_request_handler.handle(requests=requests)
         File
       ".local/lib/python3.8/site-packages/jina/serve/runtimes/request_handlers/data_request_han…
       line 164, in handle
           return_data = await self._executor.__acall__(
         File ".local/lib/python3.8/site-packages/jina/serve/executors/__init__.py", line 291, in
       __acall__
           return await self.__acall_endpoint__(req_endpoint, **kwargs)
         File ".local/lib/python3.8/site-packages/jina/serve/executors/__init__.py", line 314, in
       __acall_endpoint__
           return func(self, **kwargs)
         File ".local/lib/python3.8/site-packages/jina/serve/executors/decorators.py", line 162,
       in arg_wrapper
           return fn(executor_instance, *args, **kwargs)
         File "test_executor.py", line
       193, in index
           return DocumentArray(Document(dc.Video(
         File ".local/lib/python3.8/site-packages/docarray/document/__init__.py", line 147, in
       __init__
           super().__init__(*args, **kwargs)
         File ".local/lib/python3.8/site-packages/docarray/base.py", line 38, in __init__
           self._data = type(self)._from_dataclass(_obj)._data
         File ".local/lib/python3.8/site-packages/docarray/document/mixins/multimodal.py", line
       86, in _from_dataclass
           doc, attribute_type = cls._from_obj(attribute, field.type, field)
         File ".local/lib/python3.8/site-packages/docarray/document/mixins/multimodal.py", line
       164, in _from_obj
           doc = cls(obj)
         File ".local/lib/python3.8/site-packages/docarray/document/__init__.py", line 147, in
       __init__
           super().__init__(*args, **kwargs)
         File ".local/lib/python3.8/site-packages/docarray/base.py", line 38, in __init__
           self._data = type(self)._from_dataclass(_obj)._data
         File ".local/lib/python3.8/site-packages/docarray/document/mixins/multimodal.py", line
       76, in _from_dataclass
           'attribute_type': attribute_type,
       UnboundLocalError: local variable 'attribute_type' referenced before assignment

I looked at the file .local/lib/python3.8/site-packages/docarray/document/mixins/multimodal.py and found on line 76 that it was not defined.

Im not sure what the default value should be.

Indeed it is a bug on our side,
Here is the minimal code for reproduce it :
from docarray import *
from docarray.typing import *

from typing import List

@dataclass()
class A:
    img: List[bytes] = None

Document(A(img = []))

>>>
Traceback (most recent call last):
  File "/home/sami/.config/JetBrains/PyCharmCE2022.2/scratches/scratch_8.py", line 11, in <module>
    Document(A(img = []))
  File "/home/sami/Documents/workspace/Jina/docarray/docarray/document/__init__.py", line 147, in __init__
    super().__init__(*args, **kwargs)
  File "/home/sami/Documents/workspace/Jina/docarray/docarray/base.py", line 38, in __init__
    self._data = type(self)._from_dataclass(_obj)._data
  File "/home/sami/Documents/workspace/Jina/docarray/docarray/document/mixins/multimodal.py", line 76, in _from_dataclass
    'attribute_type': attribute_type,
UnboundLocalError: local variable 'attribute_type' referenced before assignment


[Feature][Manager] Add  group Real-time metrics

Description
Add  group Real-time metrics in manager-web
Use case
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This feature has been supported by https://github.com/apache/inlong/issues/3856


As a user, I want to be able to filter for desired values and visualize that data

Comments from @DTAIEB:

I think it would be really beneficial to the user if we were to add a “filter” functionality, where the user can specify fine grain queries e.g. age > 21, description contains XXXX, etc… This filter needs to run against the whole datasets, not only the 100 sample rows, which means it will take more time to complete

I see the filter as a not associated only to the preview pane, but rather to all the visualizations. e.g. the user specifies the filter and then can visualize the results with the pane preview, the charts, etc..

Finally, similar to charts that hides matplotlib and spark complexity, the filter is a nice way to hide Spark complexity for filtering information

important to let user zoom in on the data that’s interesting to them

The last thought is a good summary of what I think a user would expect from a filter.
Appearance and Functionality
I imagine the filter could appear within the display() API.  There is also the question of whether the filter would coexist with the search bar.
Right now, I can think of a couple of ways a user could use the filter within the display() API:

Excel-style: a user would type a filter into the search bar (more typing)
Looker-style: a user would click on a field to add to a list of filters (and then specify the filter) and continually add desired filters (more clicking)

The latter would depend upon the datatype of the field and have some predefined operations: e.g. if it is an int then you could filter "equal to" or "greater than," etc. and if it is a string then you could filter "contains" or "matches."
Applying the filter would result in the classic preview table as if you entered display(filteredData) in the cell.  Then you could proceed to visualize as usual or continue to mess around with the filters (with the option to reset to original data without filters).

duplicate #127
resolved in PR #578


Opensource LA and histogram kernels

No build procedure currently
Signed-off-by: Artem Shaporenko<EMAIL_ADDRESS>

@artem-shaporenko : tests have failed. Can you, please, look into that?

@artem-shaporenko : tests have failed. Can you, please, look into that?

Not related to this commit - broken before, need just rebase - will do


Add support for duplicate fieldnames in dom.Value#Struct

Description of changes:
This PR is to support duplicate fields in a Struct according to the Ion Spec.
Tasks:

Added changes in the dom.Value#Struct for supporting duplicate fields. (All the Struct field will now be stored in (fieldname, values) -> [string, Value[]] form)
Changes in test suite for duplicate fields.
add tests for duplicate fields.

Completed Tasks:

added changes for getAll() and allFields() methods.
changed ionEquals() method for checking unordered duplicate fields.
Tests suite changes for all the above tasks.

Questions:
Could you explain with an example on how the fields return value would look like? Also for the iterator?
e.g.
{foo: 1, baz: true, foo: 2}

For above Struct value what should be the return value for fields() as well as iterator ?
Currently it will return something like this:
[['foo',[1,2]] , ['baz' , true]]

Changes:
{foo: 1, baz: true, foo: 2}

fields() would return Struct fields as below with only the last value:
[['foo', 2] , ['baz' , true]]

To get all the field values allFields() can be used.
toString method returns with all the values of Struct fields.


Create the image using your generalized OS VHD.

The powershell commands in the section 'Create a managed image from the uploaded VHD' don't seem to be correct for a VHD that has been uploaded straight into a managed disk.
Specifically:
-BlobUri $urlOfUploadedImageVhd `
The vhd in the managed disk doesn't have a URL.
How do i create an image from a generalized VHD uploaded directly into a managed disk?
Thanks

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

ID: 0665b390-9b97-e799-0837-d03b86544afc
Version Independent ID: 5bb16d0e-d274-6df9-db4f-ebd414f9bba1
Content: Create a managed Azure VM from a generalized on-premises VHD
Content Source: articles/virtual-machines/windows/upload-generalized-managed.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

@andyturn31 Please take a look at this doc and let me know if you have any questions.
https://docs.microsoft.com/en-us/azure/virtual-machines/windows/capture-image-resource#create-an-image-from-a-managed-disk-using-powershell

@andyturn31 Is there any update?

HI
Am i correct in thinking that you can't create an Image directly from an uploaded VHD?  So you have to create a VM first and then reference $diskID = $vm.StorageProfile.OsDisk.ManagedDisk.Id when creating the image?

@andyturn31 Sorry it has taken me awhile to get back to this issue. If you follow the instructions here:  for uploading straight to a manage disk, you should already have a variable called $disk the is for the managed disk. If that is the case, you can simply assign the ID to the $diskID variable by doing: `$diskID = $disk.id'.
If you don't still have that variable available, you can Get the disk again and do the same variable assignment by doing"
$disk = Get-AzDisk -ResourceGroupName 'myResourceGroup' -DiskName 'myDiskName'
$diskId = $disk.Id
From there, you should be able to make an image from the uploaded managed disk.
I'll try to get the article updated ASAP, but hopefully this will help in the meantime :)

Thanks @cynthn :)
@andyturn31 Hope that helped answer your question.
Thanks for bringing this to our attention. We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


Change Molap to CarbonData for all modules.

As per new project name of CarbonData to update all modules' name.

closing this issue as this activity is complered


Source spans assume ASCII-only text and treat each byte as 1 character

I use miette in rulex, where I discovered that the source span is at the wrong position if the same line contains non-ASCII characters

Things to take into account

Unicode code points sometimes span multiple bytes:
assert_eq!("ä".len(), 2);                   // 2 bytes
assert_eq!("ä".chars().count(), 1);         // 1 `char`

What we perceive as "characters" are actually extended grapheme clusters, which can consist of multiple code points:
use unicode_segmentation::UnicodeSegmentation;

assert_eq!("🤦🏼♂".len(), 17);                   // 17 bytes
assert_eq!("🤦🏼♂".chars().count(), 5);          // 5 `char`s
assert_eq!("🤦🏼♂".graphemes(true).count(), 1);  // 1 extended grapheme cluster

A character usually occupies exactly 1 column in a terminal, but not always. For example, emojis and many East Asian characters occupy 2 columns:

All of these can be addressed with the unicode-width crate.

This is a duplicate of https://github.com/zkat/miette/issues/97. I'd love some help with this, since it's a pretty long-standing issue!


Replace Color 'Library' With Third Party Package

The current color code is messy, using a third party package would make sense and also mean less to test

It appears not be used so I have removed it for now


[BUG] Basic client: 'Syncer' has no attribute 'instance'

i was able to setup the server and successfully ran all the code in https://github.com/mitdbg/modeldb/tree/master/client/python/samples/sklearn.
However when i tried the basic client, all the samples code in here https://github.com/mitdbg/modeldb/tree/master/client/python/samples/basic yields this error Syncer.instance.add_to_buffer(fit_event)
AttributeError: type object 'Syncer' has no attribute 'instance'

I also seem to be to having this issue. I'm trying to run a slightly modified version of the BasicWorkflow.py. My file dies on the line "syncer_obj.sync_model("train", model_config1, mdb_model1)" with the same error as reported above.

Everything else works just fine; I'm able to sync and view the model.
I'm running ModelDB on an Ubuntu 16 VM. I have attached the file I'm trying to run below.
from modeldb.basic.Structs import (
    Model, ModelConfig, ModelMetrics, Dataset)
from modeldb.basic.ModelDbSyncerBase import *
import pyspark

# Create a syncer using a convenience API
# syncer_obj = Syncer.create_syncer("gensim test", "test_user", \
#     "using modeldb light logging")

# Example: Create a syncer from a config file
syncer_obj = Syncer.create_syncer_from_config("/home/testuser/modeldb/client/syncer.json")

# Create a syncer using a convenience API
#syncer_obj = Syncer.create_syncer("Sample Project", "test_user", "sample description")
# or
# Create a syncer explicitly
#syncer_obj = Syncer(
#    NewOrExistingProject("PySpark_test", "Andrew", "pyspark test using modeldb light logging"),
#    DefaultExperiment(),
#    NewExperimentRun("", "sha_A1B2C3D4"))

# Example: Create a syncer from an existing experiment run
# experiment_run_id = int(sys.argv[len(sys.argv) - 1])
# syncer_obj = Syncer.create_syncer_for_experiment_run(experiment_run_id)

print "I'm training some model"

# create Datasets by specifying their filepaths and optional metadata
# associate a tag (key) for each Dataset (value)
datasets = {
    "train": Dataset("/home/testuser/modeldb/data/titanic_train.csv", {"num_cols": 12,}),
    "test": Dataset("/home/testuser/modeldb/data/titanic_test.csv", {"num_cols": 12,})
}

# create the Model, ModelConfig, and ModelMetrics instances
model = "model_obj"
mdb_model1 = Model("LogReg", model, "client/python/samples/sklearn/Titanic-LogisticRegression.py")
model_config1 = ModelConfig("NN", {"l1": 10})
model_metrics1 = ModelMetrics({"accuracy": 0.8})

#mdb_model2 = Model("NN", model, "/path/to/model2")
#model_config2 = ModelConfig("NN", {"l1": 20})
#model_metrics2 = ModelMetrics({"accuracy": 0.9})

syncer_obj.sync_datasets(datasets)

syncer_obj.sync_model("train", model_config1, mdb_model1)
syncer_obj.sync_metrics("test", mdb_model1, model_metrics1)

#syncer_obj.sync_model("train", model_config2, mdb_model2)
#syncer_obj.sync_metrics("test", mdb_model2, model_metrics2)

syncer_obj.sync()

I was able to fix this problem by modifying the 'sync_model' and 'sync_metrics' methods in /usr/local/lib/python2.7/dist-packages/modeldb/basic/ModelDbSyncerBase.py. I changed Syncer.instance.add_to_buffer() to self.add_to_buffer() and the script ran.
I don't know what other issues this fix may cause, but so far I've not noticed any.

It looks like this is a problem with the pip distribution of modeldb. If you build from source it works without modification.
I'll see about updating the PyPi distribution.

Resolved by updating the PyPi distribution. New version is 0.0.1a29.


[Runtime Security-RN] O'Neal Update 3 PCCE

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--prisma-cloud-docs--hlxsites.hlx.page/
After: https://--prisma-cloud-docs--hlxsites.hlx.page/
Worker: https://prisma-cloud-docs-production.adobeaem.workers.dev/?branch=

@rcaballeromx - Links are updated.
cc: @manukumar6


add baudrate to actisense2

This replaces #669
difference: fixed Indentation

@sbender9 any comments on this?

Looks good


[BUG]version issue of Pillow while installing iopaint

Describe the bug
While installing iopaint with pip install iopaint, I get a Could not build wheels for Pillow, which is required to install pyproject.toml-based projects which probably could be an incompatibility with Pillow and other packages. I tried installing iopaint==1.0.0 but still same error.
System Info
Software version used

iopaint: 1.2.0
python: 3.12.1
conda: 23.3.1
pip(within conda): 23.3.1

Maybe it's because your Python version is too new, can you try 3.10?

Python version 3.10 is working.


EN-603 Footer

Footer component taken from designs in the docs site and the Jira ticket.
Uses the Links component just added. By default produces the footer example shown in the ticket but also allows links to be overridden the same way as Nav and the main footer message can be overridden by placing content within the tag <Footer>This is some content</Footer>
Did cross browser testing on Browserstack to check Chrome 38, only issue I saw was that the copyright message falls below the main message on IE 10 due to issues with flexbox but I figured this is acceptable.
Desktop

Mobile

Chrome 38

The only other thing i'll say is maybe it could do with a couple of simple tests? Just a snapshot and then something to assert that when given certain props you get the expected output.

Gonna withdraw the PR after all as I need to move this to the doc site and add tests


Combined server gui

This change ign gazebo to run the server and GUI in the same process. The goal is to improve performance (less protobuf serialization), and help with video recording.

I don't think we should change the default behavior on Citadel. This is a behavior change that could affect logic that's counting on having separate processes.
How about making it an option?

Do you know what things are dependent on separate processes? I would like to know what all would break.

Do you know what things are dependent on separate processes?

The main thing that comes to mind is the rendering singleton. The current implementation relies on having 2 separate scenes that are synced through the state topic, and they have separate rendering threads. If the render engine treats these 2 scenes as the same one, it could crash if 2 threads are writing to the scene at the same time.
Using different scenes for GUI and server means that they can be slightly different. For example, right now they can have different background colors. I don't think this is very important though. On the flip side, if we can remove the reliance on the SceneBroadcaster for the GUI 3D scene, that system can be significantly throttled, because widgets like the entity tree and component inspector don't need 60Hz.
I can't think of anything else right now. In any case, we never know what users may be doing in their plugins.

Mentioning rendering brings to mind that this probably also breaks sensor generation.

Doesn't solve the intended purpose.


Livestreamer doesn't accept any command in the CLI

Using Livestreamer 1.11.1 on OS X 10.9.5, I can't seem to use any command in terminal. Specifically I am trying to specify config file location using:
livestreamer --config [FILE PATH]
but it simply returns a list of the available commands.
I might be a complete idiot and have missed an integral instruction on how to properly do this, but I can't figure it out.

Livestreamer will print the help if you do not specify a stream and quality to watch. See the tutorial for more information.

I'm also having the same issue, I just want to set the default player as MPV instead of VLC using a config file.

I'm also having the same issue, I just want to set the default player as MPV instead of VLC using a config file.

You don't need to use the --config option to use a config, it's only if you want to use a custom location. See configuration file for more information.

@mtlam7 you need to specify the full path!


Scheduling: Topology Spread Constraints

From the Kubernetes docs:

You can use topology spread constraints to control how Pods are spread across your cluster among failure-domains such as regions, zones, nodes, and other user-defined topology domains. This can help to achieve high availability as well as efficient resource utilization.

Picking this up


LDAP groups from Active Directory

It seems that the LDAP connector can't currently pull group memberships from Active Directory LDAP servers because it assumes "member" fields to be populated with Uids.
In order for this to work, I must be able to set this to be the user DN instead.
It looks like someone has already made these changes but the code was never made available (#950)
I'm not very experienced in Go, but it seems like this would be a pretty straightforward change.
From what I can determine, on line 480 in ldap.go:
for _, attr := range getAttrs(user, c.GroupSearch.UserAttr) {
should be
for _, attr := range getAttrs(user, c.GroupSearch.BaseDN) {
This doesn't seem to work, however.
Could someone point me in the right direction?

+1

You can already do this. E.g. the following config would match any entry with member=<user DN>.
    # Group search queries for groups given a user entry.
    groupSearch:
      # BaseDN to start the search from. It will translate to the query
      # "(&(objectClass=group)(member=<user DN>))".
      baseDN: cn=groups,dc=freeipa,dc=example,dc=com
      # Optional filter to apply when searching the directory.
      filter: "(objectClass=group)"

      userAttr: DN # Use "DN" here not "uid"
      groupAttr: member

      nameAttr: name

Closing. Please let me know if I've misunderstood.

Thank you! That worked!


Create Kura CloudService component (into camel-kura)

We could use the CloudService provided by Kura Platform to publish and subscribe some msg.
This component doesn't need any configuration physical configuration, it uses the Kura cloud service stack
from("timer://heartbeat").setBody(constant("Hello"))
	.to("kura-cloud:appId/appTopic?qos=1&retain=false&priority=5");

    from("kura-cloud:appId/appTopic?qos=1&retain=false&priority=5").to(...)

producer and consumer are ok, we can close it


Change Dockerfile to use smaller base image

Motivation and Context
Python Docker Images are ~1GB.  This is because of the base image.
What has changed
Starting using python:3.6-slim instead
How to test?
System should behave as before, acceptance tests should workfinr
Links
https://trello.com/c/x0swxIs4/358-tbl358-review-base-docker-images

 TBL358 - review base docker images


Add a loader while dictionary loads

Subject of the issue
After selecting a word, when we click on the Get Definition button there is a delay due to API call. Try to add a small loader or an appropriate waiting message while the meaning is being fetched.

I would like to work on this issue.

Sure! Go ahead.

Closed via #65


Printing trace when NUID initialization takes long

Finding that one took a while ...
Signed-off-by: Matthias Hanel<EMAIL_ADDRESS>

I wonder if we should add this to the connect code when TLS is used, that is the other place people run into this.

@sasbury would you agree with this: Since the nuid code is going to be executed in either case, I assume adding a trace around ssl would basically cause the /dev/random delay to be logged twice. So maybe this trace already covers it?
Does the delay for ssl happen in upgradeToSecure or during ssl context creation?

I think the delay only happens the first time, i haven't traced it, but i think either can happen first so it may be worth logging on both. Upgrade to secure would be the place.

I'll add a trace for timing around upgradeToSecure as well and update this pr

For the TLS upgrade it seemed appropriate to trace all the time.


Is compatibility with the Windows Runtime a goal of the project?

I'm mostly asking about the way language projections work. For example if I write a tool that generates Rust code akin to the C++/Python ones, will it work with today's Windows Runtime APIs?

Compatibility was something that we orginally hoped to preserve. We eventually dropped it and chose to diverge so that we could make a few key breaking changes that should make the system more usable from a larger set of languages and platforms. One of the key motivators was supporting UTF-8 strings across the ABI, though there are a few other differences as well.
Having said that, we are staying closely aligned in ways that will help both component authors and consumers.
You should find it easy to adapt libraries and projection tools from one to the other. Most concepts will map 1:1. In theory, we also expect it to be possible to surface both sets of interfaces from a single object. However, the type systems are separate, type lookup will work differently, and the base C APIs used support projections are separate.
From the perspective of someone consuming the two systems, projections for winrt and xlang will mostly hide the differences. E.g. C++ /WinRT and C++ xlang types will both be created using C++ new and methods will be called through a dot '.' operator.
Ben


test: create utility function to infer rule types

During #174 and #175 I noticed that in test files rule options types are not inferred inside test functions.
To avoid this problem I added createRuleTestCaseFunction that returns a function that can be used to generate both valid and invalid cases.
This function accept only 1 type parameter which is used to infer rule Options and MessageIds which are not exported from the rule source file.
Pros
The immediate advantage is that now types are properly inferred from the rule declaration and a type error is reported if the rule Options type doesn't match them:

Intellisense on rule options screenshot

Additionally also rule MessageIds type can be inferred and provided as autocomplete suggestions:

Intellisense on rule message ids screenshot

Cons
The only problem is that Invalid test case errors property now use the messageId and data structure rather than simple strings: you can see the impact of these changes on the two rules test files changed in this PR.
I'm willing to convert all test to this pattern but I understand that this will also increase maintenance efforts while pulling changes from the upstream repository, so no problem if you don't want to proceed with this change 😉.

After what discussed in typescript-eslint/typescript-eslint#10321 I would consider to provide plain objects to the RuleTester#run method: test helper function is used to provide a default set of languagesOptions to each test case but I think that it can be done once when creating a RuleTester instance:

test/utils could export a TEST_RULE_DEFAULT_LANGUAGE_OPTIONS or a getTestRuleDefaultLanguageOptions function that returns the very same object, which then is passed when creating RuleTester instance (if required)
test function will be deprecated
createRuleTestCaseFunction can be removed since the plain objects provided to RuleTester#run are already type-checked.

What do you think?

There are many variations of lanaguageOptions (at least 5?) passed to the test function. We can't create a rule tester instance for each lanaguageOptions. IMHO let's stick with one rule tester instance here.
RuleTester#run is already type-checked, but if you can further improve test DX, feel free to go ahead!

Ok, I'll update the two rules I already edited in this PR and then if you like the result we can move on!


Gradle 4.x build fails with Groovy version 2.5.0-beta-1

Behavior
Gradle 4.x (i.e. master branch) project build fails after Groovy version is being updated from 2.4.11 to 2.5.0-beta-1

Can't have an abstract method in a non-abstract class. The class '<class name>' must be declared abstract or the method '<method name>' must be implemented.

Context

Note: I guess that Groovy version upgrade (from 2.4.x to 2.5.y) is/will be scheduled for Gradle 5.0
Related to / blocks this issue: #2487 No support for Java8 parameter name reflection
Possibly related Groovy 2.5.0 tickets/changes (I did some research):
- GROOVY-6617 joint compiler fails to compile groovy class implemented java interface
- GROOVY-6263 Accessing private methods from public ones using categories and inheritance causes MissingMethodException
Groovy 2.5 changes (so far):
- http://groovy-lang.org/changelogs.html#changelog2.5
- http://groovy-lang.org/releasenotes/groovy-2.5.html#Groovy2.5releasenotes-Breakingchanges

Steps to Reproduce:

change gradle/dependencies.gradle property versions.groovy from "2.4.11" to "2.5.0-beta-1"
execute ./gradlew build -q

dejans@BEG-IT-09 MINGW64 /c/Work/IDEA-2017.2/projects/gradle (master)
$ ./gradlew build -q
CodeNarc completed: (p1=0; p2=0; p3=0) 1446ms
CodeNarc completed: (p1=0; p2=0; p3=0) 1430ms
CodeNarc completed: (p1=0; p2=0; p3=0) 141ms
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
startup failed:
C:\Work\IDEA-2017.2\projects\gradle\subprojects\internal-testing\src\main\groovy\org\gradle\integtests\fixtures\DefaultTestExecutionResult.groovy: 22: Can't have an abstract method in a non-abstract class. The class 'org.gradle.integtests.fixtures.DefaultTestExecutionResult' must be declared abstract or the method 'org.gradle.integtests.fixtures.TestClassExecutionResult testClass(java.lang.String)' must be implemented.
 @ line 22, column 1.
   class DefaultTestExecutionResult implements TestExecutionResult {
   ^

C:\Work\IDEA-2017.2\projects\gradle\subprojects\internal-testing\src\main\groovy\org\gradle\integtests\fixtures\DefaultTestExecutionResult.groovy: 22: Can't have an abstract method in a non-abstract class. The class 'org.gradle.integtests.fixtures.DefaultTestExecutionResult' must be declared abstract or the method 'org.gradle.integtests.fixtures.TestClassExecutionResult testClassStartsWith(java.lang.String)' must be implemented.
 @ line 22, column 1.
   class DefaultTestExecutionResult implements TestExecutionResult {
   ^

C:\Work\IDEA-2017.2\projects\gradle\subprojects\internal-testing\src\main\groovy\org\gradle\integtests\fixtures\DefaultTestExecutionResult.groovy: 22: Can't have an abstract method in a non-abstract class. The class 'org.gradle.integtests.fixtures.DefaultTestExecutionResult' must be declared abstract or the method 'org.gradle.integtests.fixtures.TestExecutionResult assertTestClassesExecuted(java.lang.String[])' must be implemented.
 @ line 22, column 1.
   class DefaultTestExecutionResult implements TestExecutionResult {
   ^

3 errors

Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: C:\Work\IDEA-2017.2\projects\gradle\subprojects\reporting\src\main\java\org\gradle\api\reporting\internal\SimpleReport.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: C:\Work\IDEA-2017.2\projects\gradle\subprojects\reporting\src\main\java\org\gradle\api\reporting\plugins\BuildDashboardPlugin.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':internalTesting:compileGroovy'.
> Compilation failed; see the compiler error output for details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

* Get more help at https://help.gradle.org

BUILD FAILED in 45s

dejans@BEG-IT-09 MINGW64 /c/Work/IDEA-2017.2/projects/gradle (master)

Environment

Windows 10
Oracle Java, JDK 1.8.0_144

I guess that Groovy version upgrade (from 2.4.x to 2.5.y) is/will be scheduled for Gradle 5.0

That would very likely be the case.

Gradle 4.x (i.e. master branch) project build fails after Groovy version is being updated from 2.4.11 to 2.5.0-beta-1

I don't think we are planning to integrate a beta. We'd wait for the final.

change gradle/dependencies.gradle property versions.groovy from "2.4.11" to "2.5.0-beta-1"

The classes that fail with a compilation error are internal test classes only. They wouldn't even affect the distribution of Gradle.

See #6324


How do I sort by score?

DO NOT REMOVE OR SKIP THE TEMPLATE

[X] I understand that my issue may get closed without notice if I intentionally remove or skip any mandatory* field

Provide a well enough worded question
On rule34 you can place sort:score in tags and it will sort by score.
Ruxx doesn't seem to work with sort:score tag. So how do I do that?
Provide verbose output that demonstrates the problem (use -verbose | Edit -> Verbose log)
No response

You can't override sorting unfortunatelly, only default sorting by ID enables date filter, so it's forced.
Use score tag instead: score:>=500 score:<1000 and so on. Use check button to see the number of posts matching the tags before proceeding if you want. Note that RN uses score tag without colon score>=500.

Maybe I could add a check at launch so it's possible to use either date filter or sort tag but not both

what are the "modules" - rn rs rx?

These are just abbreviations. Website names are only displayed at github home page.

RN: rule34hentai.net
RX: rule34.xxx
RS: rule34.us

Okay, thanks

Maybe I could add a check at launch so it's possible to use either date filter or sort tag but not both
Yeah that would be okay, I'll be looking forward to this feature

There's no new release so I need to compile it manually - but there's no build instructions in README.md
Please consider adding them

Okay, sort:score works automatically. If you change the date then it wouldn't pass the check.
Looks good to me, thanks :+1:

Wait, I checked and actually the files are not being downloaded in score order

There's no new release so I need to compile it manually - but there's no build instructions in README.md Please consider adding them
upd: forgot that it's a Python project, sorry :-) so how does it work right now? is there any button/option or what?

I use my own build scripts but I'm not very experienced at python project distribution & maintainance so I don't share them. Also contain local paths and custom composed versioninfo for Windows, so - not very good.
But if you want to just build install PyInstaller package it use it on app_gui.py from root folder:

python -m PyInstaller src/app_gui.py -F --noupx -c -y -n Ruxx

Note that it will be missing proper icon and revision in title

Wait, I checked and actually the files are not being downloaded in score order

Tell me what kind of query you're trying to run so it is more clear what changes are requried to make it work

Cmd: metal_gear_solid -gay -futanari -1animal -horse sort:score -module rx -path '/home/user/Downloads' -threads 8 -timeout 10 -retries 10 -prefix -dump_tags -dump_sources -dump_comments -append_info -warn_nonempty

Do I understand correctly that idea here is to cancel downloading process at some point, when you decide you've downloaded enough? Is that the purpose of sort:score here?

Yep

I always imagined such approach a problem since you risk getting unfinished files and Ruxx tries to delete those when download is cancelled.
Anyway, this change enables download order override using sort tag https://github.com/trickerer01/Ruxx/commit/6ceaa8d3eaad3311a1a3355ef4f936f40a2dd112
Also, if you really want the exact number of files downloaded there is actually a max downloads setting which is only exposed in debug mode or if ran using console. Set __RUXX_DEBUG__ flag in app_revision.py to True to enable Debug features menu in GUI or use -dlimit flag in console.

https://github.com/trickerer01/Ruxx/releases/tag/2023.12.19


Minor code readability/style refactor

Some minor changes while reading through code for another PR.

Old compilers :( Will hold off on this until we upgrade them someday.

I was going to just close this, but we have new compilers! I figured I'd rebase and see if the conflicts weren't too bad. @rainersigwald @cdmihai if you want to take another look.

@dotnet-bot test Windows_NT Build for Full please


SPFx PnP Controls depends on MGT

Category
[x] Enhancement
[x] Bug
[ ] Question
Version
PnP controls in version 3.1
Observed Behavior
PnP Controls package has a dependency on @microsoft/mgt (Graph Toolkit) which causes problems in case 2 web parts using controls are loaded on the page:

Expected / Desired Behavior / Question
The solution could be to switch to mgt-spfx (https://www.npmjs.com/package/@microsoft/mgt-spfx), but it brings additional problems to always install the latest version of the solution of relaying on PnP controls along with mgt-spfx.sppkg to provide all the functionalities.
Could the controls relaying on mgt be moved to separate project??
Thanks!

@joelfmrodrigues @AJIXuMuK
I think this issue was fixed with "@microsoft/mgt": "^2.2.0".
with incorporation of npm install @microsoft/mgt-spfx
mgt-spfx

I'll check that later. We've already have v2.2 in beta.

@siata13 just to double-check if this is not a scenario similar to what I experienced this week at work, are you using the PnP controls package directly in your web part solution, or via a SPFx library component solution?

I was able to repro it on beta version as well. But only if you use mat-dependent control (MyTeams).
If you use any other control from the library - everything works fine.
I will check what's the best approach here. Maybe the best thing is to remove mgt dependency at all.

@joelfmrodrigues  I use it in web part solutions.
@AJIXuMuK I am not using MyTeams control. If needed I can try to get deep understanding why it happens on my end.
But I believe that moving control relying on MGT to separate library would make it much easier.
Thanks a lot for checking!

@siata13 please check why it's happening.
My only assumption is non direct import from library.
Like import { WebPartTitle } from @pnp/spfx-react-controls instead of import { WebPartTitle } from @pnp/spfx-react-controls/lib/WebPartTitle

@AJIXuMuK I have found the problematic part of the code which wasn't related to pnp controls as I initially thought.
The import strategy you suggested would be sufficient to avoid the problem, but nevertheless I think mgt-spfx is the only way to go.
Thanks for your time and support!

Just pushed some changes originally done by @joaojmendes on a PR that should fix this issue by using mgt-spfx.
I have also updated the documentation of MyTeams (the control that uses mgt) to include the instruction for installing the mgt-spfx library. Tested on my tenant and the error was resolved after the changes were applied.
Please note this is relevant for the MyTeams control. For anything else, use direct imports as suggested by @AJIXuMuK
import { WebPartTitle } from @pnp/spfx-react-controls/lib/WebPartTitle


Feature Request: Macro command should work with key repeat

I would like to use a macro command like official runCommands with KEY REPEAT (long pressing a shortcut key).
For this purpose, it seems the runCommands works almost perfectly but fails when calling extension commands.
Suppose we have an extension command that contains await for some reason:
        vscode.commands.registerCommand(
            'some-extension.command',
            async function() {
                // some processing that includes await
                await new Promise(resolve => setTimeout(resolve, 300));
            }
        );

And we use it along with other commands through a shortcut key.
        {
            "key": "ctrl+p",
            "command": "runCommands",
            "args": {
                "commands": [
                    { "command": "type", "args": { "text": "BEGIN " } },
                    { "command": "some-extension.command" },
                    { "command": "type", "args": { "text": "END\n" } }
                ]
            }
        }

Now we can get the following text by this shortcut.
BEGIN END

This is perfect.
But when I long press the key, I get a broken result.

I know the vscode nature that every command possibly runs concurrently and a shortcut key executes the assigned command immediately even if some commands are under execution. So I think this feature request might need some new mechanism to handle this kind of exceptional exclusive execution.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


added external stage support to fud

Frontends may want to hook into the fud infrastructure without actually having their Stage source be a part of upstream fud code. This PR adds a simple mechanism to allow fud to load stages at arbitrary paths.
This command:
fud register my_private_stage --path /some/path/to/stage.py

adds the following section to the config.toml file:
[external-stages.my_private_stage]
location = "/some/path/to/stage.py"

and
fud register my_private_stage --delete

will remove it.
There is some new registry logic that looks for these stages in the config file and loads them. The specified python module
needs to have a __STAGES__ variable that specifies the classes for stages that should be loaded.

The code looks good. Can you add some documentation about registering external stages in the main Calyx documentation page under the Fud category: https://capra.cs.cornell.edu/docs/calyx/fud/index.html
Merge whenever done.


Images not displaying

Hi,
When I debug the app in browser (acting like an iPad) everything is fine, however when I run the app either through the emulator or by using the PhoneGap Developer app on iPad none of the images that are in the HTML load.
On initial load the image on the home page is shown but when you navigate away and come back to the home page it no longer shows.
I am using Angular routing to display different templates based on the route and referencing the images by images/... or /images/... and neither work.
If you could help shed some light on this it would be amazing.
Thanks
Harry

Hi Harry, your issue does not seem strictly related to mobile-angular-ui, maybe you can report this on forum.
Feel free to reopen if you find the cause to be mobile-angular-ui


Generic External Bitstream Storage Based on Apache JClouds

References
Add references/links to any related issues or PRs. These may include:

Fixes Fixes  https://github.com/DSpace/DSpace/issues/8115
Related to https://github.com/DSpace/DSpace/issues/7325

Description
Provides DSpace with a Bistore based on the Apache JClouds framework to support storing content in multiple cloud vendors. JCloudStore which can be configured inside of bitstore.xml. This store can be connected to any storage provider supported here https://jclouds.apache.org/ and allows saving bitstreams to the connected Cloud container. This allows for DSpace to support a multitude of storage options, for example, Microsoft Azure, Google Cloud Storage, or independent S3 providers such as MinIO.
This functionality was made possible through the support of the Flemish Government - Departement Omgeving, who generously sponsored its development for the benefit of the open-source community.
Instructions for Reviewers
To enabled setup the following in bitstore.xml with some kind of local object store configured. For the example provided:

<bean name="org.dspace.storage.bitstore.BitstreamStorageService" class="org.dspace.storage.bitstore.BitstreamStorageServiceImpl">
        <property name="incoming" value="${assetstore.index.primary}"/>
        <property name="stores">
            <map>
                <entry key="0" value-ref="localStore"/>
                <entry key="1" value-ref="s3Store"/>
                <entry key="2" value-ref="jcloudStore"/>
            </map>
        </property>
    </bean>

    <bean name="jcloudStore" class="org.dspace.storage.bitstore.JCloudBitStoreService" scope="singleton" lazy-init="true">
        <property name="enabled" value="${assetstore.s3.generic.enabled:false}"/>
        <property name="useRelativePath" value="${assetstore.s3.generic.useRelativePath:false}"/>
        <property name="container" value="${assetstore.s3.generic.container}"/>
        <property name="endpoint" value="${assetstore.s3.endpoint}"/>

        <property name="providerOrApi" value="${assetstore.s3.generic.provider}"/>
        <property name="identity" value="${assetstore.s3.generic.awsAccessKey}"/>
        <property name="credentials" value="${assetstore.s3.generic.awsSecretKey}"/>
        <property name="maxCounter" value="${assetstore.s3.maxCounter:100}"/>

        Subfolder to organize assets within the bucket, in case this bucket is shared
        Optional, default is root level of bucket
        <property name="subFolder" value="${assetstore.s3.generic.subfolder}"/>
    </bean>

Specific connection properties may then be set in the dspace configuration
Then all assetstore.s3.generic must be defined in shared.cfg

# Enabled the assetstore [true|false]
assetstore.s3.generic.enabled=true

# The storage container/bucket of the object store that this service will use. [bucketname]
assetstore.s3.generic.container=bucket_name

# Endpoint of the storage service [http://endpoint]
assetstore.s3.endpoint=

# https://jclouds.apache.org/reference/providers/#blobstore  Which provider we are using from the list of supported ones.
assetstore.s3.provider

# Private key or username for the store
assetstore.s3.generic.awsAccessKey

# Private password for the store
assetstore.s3.generic.awsSecretKey

# How many times the store will connection will be used before reconnection
assetstore.s3.maxCounter

# Subfolder all files will be stored in inside of the container/bucket.
assetstore.s3.generic.subfolder

Then attempt to create new items with bitstreams, delete items with bitstreams and replace item bitstreams.
List of changes in this PR:

Inclusion of Apache JClouds libraries via Maven depdnencies
Assignment of compatable Google Guava, Guise and GSon versions
Addition of new method to BitstreamStorageSertvice to support setting the incoming external store for unit tests
JCloudBitstore implementation
JCloudBitTStoreServiceTest based on Mock and local JClouds file storage.

Checklist

[x] My PR is created against the main branch of code (unless it is a backport or is fixing an issue specific to an older branch).
[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] My PR includes details on how to test it. I've provided clear instructions to reviewers on how to successfully test this fix or feature.
[x] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation
[x] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[x] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[x] If my PR fixes an issue ticket, I've linked them together.

@mdiggory what value should i set in providerOrApi (assetstore.s3.generic.provider) for a self-hosted S3 instance like MinIO? Is aws-s3 compatible, or do I just set s3 (as in, the API)?

@mdiggory what value should i set in providerOrApi (assetstore.s3.generic.provider) for a self-hosted S3 instance like MinIO? Is aws-s3 compatible, or do I just set s3 (as in, the API)?

@kshepherd the providers that are supported are available in the JClouds site/documentation https://jclouds.apache.org/reference/providers/#blobstore
Here is a sample of the settings that one might set for MinIO

property
default
Description

assetstore.s3.generic.enabled
true
Enables the JClouds configuration

assetstore.s3.generic.provider
s3
Enables the generic s3 provider in the JCouldsBitstore

assetstore.s3.generic.subfolder
assetstore
if enabled, a subfolder will be added to the container

assetstore.s3.generic.useRelativePath
true
if enabled, will use relative path hashing for assetstore file assetstore/12/34/56/1234567890... otherwise it will be stored in the subfolder assetstore/1234567890...`

assetstore.s3.generic.container
my-assetstore-bucket
The Container (S3 Bucket) that objects should be stored within.

assetstore.s3.endpoint
http://localhost:9000
Provide the appropriate URL to the Minio S3 Endpoint

private.s3.generic.awsAccessKey

Access Key for S3 Repository

private.s3.generic.awsSecretKey

Access Credential for S3 Repository


Centralised PyTorch 2.0 issues

Describe the bug
This issue serves as a centralized platform for documenting any bugs or issues encountered when utilizing SpeechBrain in conjunction with PyTorch 2.0.
Expected behaviour
x
To Reproduce
No response
Versions
No response
Relevant log output
No response
Additional context
No response

Describe the bug
A small bug: when I use torch 2.0 instead of 1.13.1, DDP training does not work properly.
In general (in version 1.13.1 of torch), running a similar script can help train the model using DDP.
CUDA_VISIBLE_DEVICES=2,3 python -m torch.distributed.launch --nproc_per_node=2 train_hifigan.py hparams/train_hifigan.yaml --distributed_launch --distributed_backend=nccl
But in torch 2.0, this does not seem to be available. It seems to be caused by an update of torch, which can be referred to this.
The content in this commit is to change underscores to hyphens in some parameter names in the parser. Although this commit has little impact on the native use of torch because it also supports underscore parameters, in SpeechBrain, this will cause an error because speechbrain.parse_arguments() can only recognize the --local_rank provided by the previous version of torch.distributed.launch, but not the --local-rank in version 2.0. Therefore, parse_arguments() will recognize --local-rank as an override and attempt to overwrite a non-existent local-ralocal-ranknk parameter in the hparams.
Expected behaviour
xxxx
To Reproduce
In my experiment, it will prompt an error that looks like this.
Traceback (most recent call last):
    File "/home/sturjw/Code/TTSS/train_hifigan.py", line 336, in <module>
        hparams = load_hyperpyyaml(fin, overrides)
    File "/home/sturjw/.conda/envs/SB/lib/python3.9/site-packages/hyperpyyaml/core.py", line 157, in load_hyperpyyaml
        yaml_stream = resolve_references(yaml_stream, overrides, overrides_must_match)
    File "/home/sturjw/.conda/envs/SB/lib/python3.9/site-packages/hyperpyyaml/core.py", line 316, in resolve_references
        recursive_update(preview, overrides, must_match=overrides_must_match)
    File "/home/sturjw/.conda/envs/SB/lib/python3.9/site-packages/hyperpyyaml/core.py", line 776, in recursive_update
        raise KeyError(f"Override '{k}' not found in: {[key for key in d.keys()]}")
KeyError: "Override 'local-rank' not found in: ['seed', '__set_seed', 'output_folder', 'save_folder', 'train_log', 'progress_sample_path', 'epochs', 'keep_checkpoint_interval', 'use_tensorboard', 'data_folder', 'libritts_subsets', 'json_folder', 'train_json', 'valid_json', 'test_json', 'skip_prep', 'splits', 'sheckpoint_interval', 'use_tensorboard', 'data_folder', 'libritts_subsets', 'json_folder', 'train_json', 'valid_json', 'test_json', 'skip_prep', 'splits', 'split_ratio', 'segment_size', 'sample_rate', 'hop_length', 'win_length', 'n_mel_channels', 'n_fft', 'mel_fmin', 'mel_fmax', 'mel_normalized', 'power', 'norm'.....]

As I mentioned above, the error can be traced back to attempting to overwrite a parameter that does not exist in the YAML file.
Therefore, I believe that this is not a specific use case, and this bug should be reproducible in any attempt to use DDP to train models in Torch 2.0.
So this situation should be easily reproducible by simply installing Torch 2.0.
Versions
torch 2.0
speechbrain 0.5.14
Relevant log output
No response
Additional context
I ultimately resolved this issue by rolling back the Torch version.
But I believe that this bug can be easily fixed and made compatible both with Torch 1.x and 2.0 by changing parser.add_argument("--local_rank", type=int, help="Rank on local machine") to parser.add_argument("--local_rank", "--local-rank", type=int, help="Rank on local machine") in speechbrain.parse_arguments().
Also, I am not a native English speaker, so if my expressions confuse you, please ask me. Finally, I hope this will help you make SB better.

Hi @jiaweiru, thanks for reporting this issue.
This has been fixed in #1951.
Many thanks for spotting this issue and making SB better. (PS: your English is really good don't worry :))

When I tried upgrading PyTorch to 2.0,  I started getting the following error from Checkpointer while moving from the training to validation stage (first time it attempts to save the checkpoint):
File "C:.......\speechbrain\speechbrain\utils\checkpoints.py", line 572, in save_checkpoint
raise RuntimeError(MSG)
RuntimeError: Don't know how to save <class 'torch.optim.lr_scheduler.ExponentialLR'>. Register default hook or add custom hook for this object.


infinite loop when exporting (Dependency cycle)

Current Behavior
using version 1.3.3 with Blender 3.3 or 3.6 LTS I get an infinite loop when exporting the model "dependency cycle detected".

Expected Behavior
it works fine for me with Blender 3.1 and version 1.3.0 that I used before. Do you know what could be the reason?
Steps To Reproduce
Blender 3.3 or 3.6 LTS and exporter version 1.3.3
Environment
- OS: Windows 11
- Blender: 3.3 LTS or 3.6 LTS
- glTF-Blender-IO-MSFS: 1.3.3

Anything else?
No response

Same here, never got export to work since 3.3.

Hello, this issue seems to be an issue with the official blender exporter, it should be reported here : https://github.com/KhronosGroup/glTF-Blender-IO/issues
Thank you for your report


Add pdbfixer dependency to openff-toolkit-examples

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)

This PR adds the pdbfixer dependency to the openff-toolkit-examples package.

@conda-forge-admin, please rerender


Tests can't run after putting this project as a subproject

Hi
I'm trying to use this template for building a plugin.
My problem is that when I use this project, it all works just fine, but when I add this project as a part of another project using includeBuild, I can't run tests anymore (at least in IDEA).
I created a repository that reproduces this behaviour https://github.com/jvmusin/kotlin-gradle-plugin-subproject-issue
Do you have any ideas on how to include this project as a part of another one with an ability to run tests and do all other stuff?

I says Plugin [id: 'org.jetbrains.packages.dht.plugin'] was not found in any of the following sources: and an empty line below

It says Plugin [id: 'com.ncorti.kotlin.gradle.template.plugin'] was not found in any of the following sources: and a blank line below.

but when I add this project as a part of another project using includeBuild, I can't run tests anymore (at least in IDEA).

I believe you can't do that. Or at least not so easily.
This template already has an included build here:
https://github.com/cortinico/kotlin-gradle-plugin-template/blob/e0a0534d51b31ec433412a11e0c071efafebe134/settings.gradle.kts#L12
If you include this whole template in your build you will probably ended up with a nested included build. See https://github.com/gradle/gradle/issues/1909 for more context.

It says Plugin [id: 'com.ncorti.kotlin.gradle.template.plugin'] was not found in any of the following sources: and a blank line below.

The reason is because Gradle fails to find the plugin-build included build and cannot expose the Gradle plugin. You probably need to remove one level of nesting if you want to use includeBuild in your project.

Thanks for your response.
I actually resolved my problem by putting these projects as subprojects of my root project and I added the plugin using includeBuild in pluginManagement section according to https://docs.gradle.org/current/userguide/composite_builds.html#included_plugin_builds
Issue can be closed now

Great thanks for sharing you approach 🙏


Switch summary list markup to govuk-component

What problem does the pull request solve?
govuk-component gem provides us with a summary list component.
Theres a couple of benefits here in that the govuk-component version
should always match the implementation of govuk-frontend nunjucks version.
This means that we don't have to worry about manually updating our version
if govuk-frontend updates their components and ends up breaking our one.
Apart for the key values being made bold and the actions being on one line,
I think govuk-component implementation does a better job with the actions
because now the screenreader would alert the user to the fact that there
is a list of actions rather than not realising that there is more actions
without navigation through the page.
Before

After

Checklist

[x] I've used the pull request template
[ ] I've linked this PR to the relevant issue (if mission work)
[ ] I've written unit tests for these changes (if code change)
[ ] I've updated the documentation in (If any documentation requires updating)

[ ] README.md
[ ] Elsewhere (please link)

Nice one - just the one comment from me about the change of case on these action links, but otherwise looks really good and seems like a sensible change!


expectDeprecation*(): Document custom extension + events approach for PHPUnit 10

PHPUnit 10 removed the expect*() methods, including expectDeprecation(), expectDeprecationMessage(), and expectDeprecationMessageMatches().
As PHPUnit Polyfills is forward compatible, these removed expectations will not be polyfilled in 2.x.
As it is possible to polyfill these removed expectations, @jrfnl and I discussed documenting a custom approach in the 2.x README's FAQ.
The custom approach:

takes inspiration from PHPUnit 11: refer to the Error Handling > Expecting Deprecations docs and the PR that implemented it.
adds an extension.
uses the PHPUnit\Event\Test\ events.

Closing as duplicate (or close ) of #186.


e-Consent

Brief description:
This is a framework used to take consent of a patient electronically.
User spec:
https://github.com/CCTC-team/redcap_cypress/blob/redcap_val/user_requirement_specification/core/24_e_consent_framework.spec
Feature link:
https://github.com/CCTC-team/redcap_rsvc/blob/redcap_val/Feature Tests/C/e-Consent framework/C.3.24.100 - eConsent enable.feature
Pre review:

[ ] Script(s) targets the correct feature sufficiently
[ ] The user requirements are met

Post review:

[ ] Script(s) targets the correct feature sufficiently
[ ] Script(s) appropriately tests the feature

Scripts Pre-reviewed:

C.3.24.100 - eConsent enable

C.3.24.100 - eConsent enable
•	When selecting ‘Auto-Archiver enabled’, Cypress is selecting 'Auto-Archiver + e-Consent Framework' for some reason.
•	Some steps are not visible when 'Auto-Archiver enabled' is selected. This is not present in the original file in Theresa Baker's repository
•	Script modified and pushed to MX_v13.1.37


Unknown/unhandled server invoke

Please help me !!!. when i use url (rtmp://<IP_ADDRESS>:1935/live_portrait/stream_id) it's working, but i use url (rtmp://<IP_ADDRESS>:1935/live1/stream_id) it got error >_<. this is me Logcat

11-28 17:17:20.545 16355-16355/jp.stagegroup.livestream I/VideoEncoder: videoEncoder OMX.qcom.video.encoder.avc type supported: video/avc
11-28 17:17:20.545 16355-16355/jp.stagegroup.livestream W/VideoCapabilities: Unrecognized profile<PHONE_NUMBER> for video/avc
11-28 17:17:20.547 16355-16355/jp.stagegroup.livestream I/VideoEncoder: Color supported:<PHONE_NUMBER>
11-28 17:17:20.547 16355-16355/jp.stagegroup.livestream I/VideoEncoder: Color supported:<PHONE_NUMBER>
11-28 17:17:20.548 16355-16355/jp.stagegroup.livestream I/VideoEncoder: Color supported:<PHONE_NUMBER>
11-28 17:17:20.548 16355-16355/jp.stagegroup.livestream I/VideoEncoder: Color supported: 21
11-28 17:17:20.627 16355-16355/jp.stagegroup.livestream W/VideoCapabilities: Unrecognized profile<PHONE_NUMBER> for video/avc
11-28 17:17:20.632 16355-25371/jp.stagegroup.livestream W/OMXUtils: do not know color format 0x7fa30c04 =<PHONE_NUMBER>
11-28 17:17:20.633 16355-25371/jp.stagegroup.livestream W/OMXUtils: do not know color format 0x7f000789 =<PHONE_NUMBER>
11-28 17:17:20.639 16355-25371/jp.stagegroup.livestream I/ACodec: setupAVCEncoderParameters with [profile: Baseline] [level: Level3]
11-28 17:17:20.640 16355-25371/jp.stagegroup.livestream I/ACodec: [OMX.qcom.video.encoder.avc] cannot encode color aspects. Ignoring.
11-28 17:17:20.641 16355-25371/jp.stagegroup.livestream I/ACodec: [OMX.qcom.video.encoder.avc] cannot encode HDR static metadata. Ignoring.
11-28 17:17:20.641 16355-25371/jp.stagegroup.livestream I/ACodec: setupVideoEncoder succeeded
11-28 17:17:20.679 16355-16355/jp.stagegroup.livestream I/MicrophoneManager: Microphone created, 44100hz, Stereo
11-28 17:17:20.693 16355-25387/jp.stagegroup.livestream I/SrsFlvMuxer: worker: connecting to RTMP server by url=rtmp://<IP_ADDRESS>:1935/live1/6457a7a4009ce3ea
11-28 17:17:20.694 16355-25387/jp.stagegroup.livestream D/RtmpConnection: connect() called. Host: <IP_ADDRESS>, port: 1935, appName: live1, publishPath: 6457a7a4009ce3ea
11-28 17:17:20.721 16355-16355/jp.stagegroup.livestream I/AudioEncoder: AudioEncoder started
11-28 17:17:20.816 16355-25387/jp.stagegroup.livestream D/RtmpConnection: connect(): socket connection established, doing handhake...
11-28 17:17:20.816 16355-25387/jp.stagegroup.livestream D/Handshake: writeC0
11-28 17:17:20.816 16355-25387/jp.stagegroup.livestream D/Handshake: writeC1
11-28 17:17:20.816 16355-25387/jp.stagegroup.livestream D/Handshake: writeC1(): Calculating digest offset
11-28 17:17:20.817 16355-25387/jp.stagegroup.livestream D/Handshake: writeC1(): (real value of) digestOffset: 421
11-28 17:17:20.817 16355-25387/jp.stagegroup.livestream D/Handshake: writeC1(): recalculated digestOffset: 1197
11-28 17:17:20.817 16355-25387/jp.stagegroup.livestream D/Handshake: writeC1(): digestOffsetBytes: 0000A6FF
11-28 17:17:20.817 16355-25387/jp.stagegroup.livestream D/Handshake: partBeforeDigest(): size: 1197
11-28 17:17:20.819 16355-25387/jp.stagegroup.livestream D/Handshake: writeC1(): Writing timestamp and Flash Player version
11-28 17:17:20.819 16355-25387/jp.stagegroup.livestream D/Handshake: partAfterDigest(): size: 307
11-28 17:17:20.819 16355-25387/jp.stagegroup.livestream D/Handshake: copying digest offset bytes in partBeforeDigest
11-28 17:17:20.820 16355-25387/jp.stagegroup.livestream D/Handshake: writeC1(): Calculating digest
11-28 17:17:20.820 16355-25387/jp.stagegroup.livestream D/Handshake: writeC1(): writing C1 packet
11-28 17:17:20.822 16355-25387/jp.stagegroup.livestream D/Handshake: readS0
11-28 17:17:20.854 16355-16355/jp.stagegroup.livestream I/Camera1ApiManager: 640X480
11-28 17:17:20.859 16355-16355/jp.stagegroup.livestream I/MicrophoneManager: Microphone started
11-28 17:17:20.861 16355-16355/jp.stagegroup.livestream I/Choreographer: Skipped 45 frames!  The application may be doing too much work on its main thread.
11-28 17:17:20.890 16355-25387/jp.stagegroup.livestream D/Handshake: readS1
11-28 17:17:20.890 16355-25387/jp.stagegroup.livestream D/Handshake: readS1(): S1 total bytes read OK
11-28 17:17:20.890 16355-25387/jp.stagegroup.livestream D/Handshake: readC2
11-28 17:17:20.890 16355-25387/jp.stagegroup.livestream D/Handshake: readS2
11-28 17:17:20.890 16355-25387/jp.stagegroup.livestream D/Handshake: readS2(): S2 total bytes read OK
11-28 17:17:20.890 16355-25387/jp.stagegroup.livestream D/RtmpConnection: connect(): handshake done
11-28 17:17:20.894 16355-25387/jp.stagegroup.livestream D/RtmpConnection: rtmpConnect(): Building 'connect' invoke packet
11-28 17:17:20.896 16355-25387/jp.stagegroup.livestream D/RtmpConnection: wrote packet: RTMP Command (command: connect, transaction ID: 1), size: 273
11-28 17:17:20.898 16355-25443/jp.stagegroup.livestream D/RtmpConnection: starting main rx handler loop
11-28 17:17:20.904 16355-16361/jp.stagegroup.livestream I/zygote64: Do partial code cache collection, code=247KB, data=189KB
11-28 17:17:20.904 16355-16361/jp.stagegroup.livestream I/zygote64: After code cache collection, code=246KB, data=189KB
11-28 17:17:20.904 16355-16361/jp.stagegroup.livestream I/zygote64: Increasing code cache capacity to 1024KB
11-28 17:17:20.982 16355-25443/jp.stagegroup.livestream D/RtmpConnection: handleRxPacketLoop(): Setting acknowledgement window size: 2500000
11-28 17:17:20.982 16355-25443/jp.stagegroup.livestream D/RtmpConnection: handleRxPacketLoop(): Send acknowledgement window size: 2500000
11-28 17:17:20.982 16355-25443/jp.stagegroup.livestream D/RtmpConnection: wrote packet: RTMP Window Acknowledgment Size, size: 4
11-28 17:17:20.983 16355-25443/jp.stagegroup.livestream D/RtmpDecoder: readPacket(): Setting chunk size to: 4096
11-28 17:17:20.985 16355-25443/jp.stagegroup.livestream I/RtmpConnection: Connection failed: Application rejected connection.
11-28 17:17:20.986 16355-25443/jp.stagegroup.livestream E/RtmpConnection: handleRxInvoke(): Unknown/unhandled server invoke: RTMP Command (command: close, transaction ID: 0)
11-28 17:17:20.988 16355-25387/jp.stagegroup.livestream D/RtmpConnection: socket closed
11-28 17:17:21.061 16355-16355/jp.stagegroup.livestream I/MicrophoneManager: Microphone stopped
11-28 17:17:21.062 16355-16355/jp.stagegroup.livestream I/SrsFlvMuxer: SrsFlvMuxer closed
11-28 17:17:21.063 16355-25459/jp.stagegroup.livestream I/SrsFlvMuxer: worker: disconnect ok.
11-28 17:17:21.092 16355-16355/jp.stagegroup.livestream I/AudioEncoder: AudioEncoder stopped

live1: landscape
live_portrait: portrait
Thanks you so much !!!

This look a bad configuration in live1 application on server side.
11-28 17:17:20.985 16355-25443/jp.stagegroup.livestream I/RtmpConnection: Connection failed: Application rejected connection.

Did you tried with ffmpeg commands?

sorry, i just learn android on 2 week. i will try with ffmpeg (research by google) as you tell. i too think server side got bad configuration in live1 application. Thanks for your help !!!

ffmpeg on desktop not on android.
https://github.com/fluent-ffmpeg/node-fluent-ffmpeg/wiki/Installing-ffmpeg-on-Mac-OS-X
With ffmpeg installed use this command in console:
ffmpeg -i pathtomp4file -f flv rtmp://ip:port/appname/streamname

Where pathtomp4file is a path to a mp4 file in your disk that will be streamed and rtmp://ip:port/appname/streamname is the endpoint used in this library.
Ffmpeg should give you more info about the connection fail.
What media server are you using? can you post your live1 configuration?

What media server are you using? can you post your live1 configuration?
(rtmp://<IP_ADDRESS>:1935/) this server of customer i can't touch it
Where pathtomp4file is a path to a mp4 file
i stream by live camera, not mp4 file
On IOS my friend use (https://github.com/shogo4405/HaishinKit.swift) and it work, i use same library (https://github.com/shogo4405/HaishinKit.kt) but it not work and got error like when i use your lirary. I asked admin of server, he talk "i can't see any error in log of server, I see the log of Wowza (distribution) server". ^^ i don't know what happen.

If i do stream to:
rtmp://<IP_ADDRESS>:1935/live1/pedro

Server response me with auth error. I think, you need user and password to do stream, ask to the admin for it and use the method rtmpCamera1.setAuthorization(); before rtmpCamera1.startStream() method. Other option is disable auth requeriments like in live_portrait application.

you are correct, finaly i found solution. Thank you so much


ARROW_GANDIVA_JAVA flag missing in instructions

Thanks for this beautiful project!
The build instructions
https://github.com/apache/arrow/blob/master/docs/source/developers/cpp.rst#optional-components
seem to be missing the new -DARROW_GANDIVA_JAVA=ON (in addition to DARROW_GANDIVA=ON).
See https://github.com/apache/arrow/pull/2817 for background.

@daslu could you file a JIRA issue on this?  Would you have time to contribute a patch?

@emkornfield sure, thanks.

@daslu can you post a link to the JIRA issue?

Hi, tomorrow finally I'll be able to create a ticket and a pull request, and post the links here.

Jira issue: https://issues.apache.org/jira/browse/ARROW-5373
Pull request: https://github.com/apache/arrow/pull/4345
Thanks

Thank you.


ISPN-15336 Non-understandable column names in case of running count(*) on aggregation queries

https://issues.redhat.com/browse/ISPN-15336
Draft => regression should have been fixed, but the labels for the aggregations are still not good done
Based on top of => https://github.com/infinispan/infinispan/pull/12764

I rebased the contribution on top of https://github.com/infinispan/infinispan/pull/12784


Seems like this is not working for Angular 5

if anyone got same error?

I have used this for my new angular 5 projects, it's not working, basically not auto unsubscribing,
But the same library I have used for Angular 4 project, it's working.
That's why I wanted to know, this issue having to others also or not

We are working with Angular 5 and all good. Can you show me the code, please?

Indeed! The code is working absolutely fine with angular 5 application without any room for issues.


IPTC data written as UTF8 incorrectly read

The IPTC value CodedCharacterSet is ignored, when set to UTF8 (or ESC % G).
The attached image has Keywords, City, Sublocation and State with special characters. Command line exiftool returns them correctly, but exifr does not.

Keywords return by exifr:
  iptc: {
    ApplicationRecordVersion: "\u0000\u0004",
    Keywords: [
      "Ã¦ÃÃ¸ÃÃ¥ÃÃ©ÃÃ¼ÃÃ¤ÃÃ¶ÃÃ¯ÃÃ±Ã",
      "abcdefghijklmnopqrstuvwxyz",
    ],
    City: "JÃ¶nkÃ¶ping",
    Sublocation: "JÃ¶nkÃ¶ping",
    State: "JÃ¶nkÃ¶ping",
    CountryCode: "SWE",
    Country: "Sverige",
  },

Exiftool:
$ exiftool -Keywords -City -Sub-location -Location -State -CodedCharacterSet Chars.jpg
Keywords                        : æÆøØåÅéÉüÜäÄöÖïÏñÑ, abcdefghijklmnopqrstuvwxyz
City                            : Jönköping
Sub-location                    : Jönköping
Location                        : Jönköping
State                           : Jönköping
Coded Character Set             : UTF8

Could the problem be line 86 here?
https://github.com/MikeKovarik/exifr/blob/master/src/segment-parsers/iptc.mjs
getLatin1String sounds non-UTF8

Closing because it's a duplicate of https://github.com/MikeKovarik/exifr/issues/82


[BUG] Improper Error Handling in index creation API

Describe the bug
Sending malformed JSON to index creation API returns 500 Internal Server Error response. Ideally, it should be 400 Bad Request.

Looks like a duplicate of #2756.


Cannot find module 'graphql/validation/rules/PossibleTypeExtensions'

Error: Cannot find module 'graphql/validation/rules/PossibleTypeExtensions'
Error: Cannot find module 'graphql/validation/rules/PossibleTypeExtensions'
graphql  : 0.13.2
<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

@Karthickgreets99 it looks like you're using a very old version of https://github.com/graphql/graphql-js, you might ask for help in that repo!
The code here is not used in your setup.


Adding social media Icons with active links.

Please assign me this issue under Hacktoberffest2024.
I can add various social media icons like Linkedin, Github etc with active links that takes user to the respective pages.

Hello @divyanshudhruv ,  I would like to work on it, can you please assign this to me?

Please refer to #116


add Freighter instructions

Preview is available here:http://soroban-docs-pr390.previews.kube001.services.stellar-ops.com/


Proposal: Add a cidfile flag to docker-build command

Proposed in #7727 by @virtuald

docker run has a --cidfile argument. It's a bit strange that docker build doesn't have a similar thing,
[...]  a --cidfile option is something that would really only be useful for scripting

Providing this argument can be a nice alternative to the docker-build's quiet mode (read more here).
Thoughts?
Thanks.

#17428 was merged. A cidfile flag doesn't make sense at all for docker build for a few reasons. docker build starts and stops multiple containers and there's no single container to have in the cidfile. docker build also has --rm and --force-rm flags to ensure containers are always removed, no matter what happens with the build. There's no point in adding such a flag.
The builder code will also be moved out of the Docker daemon. This means it'll be possible to do all kinds of things. This will most likely allow doing quiet builds and the retrieval of the logs of the container which has halted the build.
I'll close this now since it doesn't make sense to implement this now. Please feel free to comment.

@unclejack I think the intent here was to write the image-id to a file (--imgid-file?). But agreed to wait for the builder to be moved out

Yes, the intent was to write the image id to a file, that way you don't have to create unique container names or parse the output of docker build, etc.