[WFCORE-590]  SSL secured communication between master and slave in domain mode test cases added

Windows Build 1020 is now running using a merge of 90358de7c595acb0ce8a21b3d278c47c0f989445

Core - Full Integration Build 600 is now running using a merge of 90358de7c595acb0ce8a21b3d278c47c0f989445

Linux Build 1348 is now running using a merge of 90358de7c595acb0ce8a21b3d278c47c0f989445

Linux Build 1348 outcome was SUCCESS using a merge of 90358de7c595acb0ce8a21b3d278c47c0f989445
Summary: Tests passed: 3049, ignored: 64 Build time: 0:19:38

Windows Build 1020 outcome was SUCCESS using a merge of 90358de7c595acb0ce8a21b3d278c47c0f989445
Summary: Tests passed: 3049, ignored: 64 Build time: 0:19:48

Core - Full Integration Build 600 outcome was FAILURE using a merge of 90358de7c595acb0ce8a21b3d278c47c0f989445
Summary: Tests failed: 1 (1 new), passed: 2831, ignored: 415 Build time: 0:48:23
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptMultipleClientsTestCase.encryptedAndSignedRequestFromAlice: javax.xml.ws.WebServiceException: Could not send Message.
	at java.net.SocketInputStream.socketRead0(Native Method)
	at java.net.SocketInputStream.read(SocketInputStream.java:150)

The two test classes are doing a lot of duplicative setup/teardown stuff that could be handled in static methods in the parent class.

Thank you very much for feedback. Next commit tries to provide fixes.

Windows Build 1028 is now running using a merge of f8f64a24111e65ed9d5d2ab0757f821999a00e6e

Core - Full Integration Build 608 is now running using a merge of f8f64a24111e65ed9d5d2ab0757f821999a00e6e

Linux Build 1356 is now running using a merge of f8f64a24111e65ed9d5d2ab0757f821999a00e6e

Linux Build 1356 outcome was SUCCESS using a merge of f8f64a24111e65ed9d5d2ab0757f821999a00e6e
Summary: Tests passed: 3049, ignored: 64 Build time: 0:19:01

Windows Build 1028 outcome was SUCCESS using a merge of f8f64a24111e65ed9d5d2ab0757f821999a00e6e
Summary: Tests passed: 3049, ignored: 64 Build time: 0:19:19

Core - Full Integration Build 608 outcome was FAILURE using a merge of f8f64a24111e65ed9d5d2ab0757f821999a00e6e
Summary: Tests failed: 1 (1 new), passed: 2831, ignored: 415 Build time: 0:52:55
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.stateful.passivation.PassivationTestCase.testPassivationMaxSize: java.lang.reflect.UndeclaredThrowableException: null
	at com.sun.proxy.$Proxy62.runTestMethod(Unknown Source)
	at org.jboss.arquillian.protocol.jmx.JMXMethodExecutor.invoke(JMXMethodExecutor.java:77)

Thanks again for your feedback.

Core - Full Integration Build 650 is now running using a merge of 44dac929140bfcd7d7cb1705fcff9c28b3ad87d9

Linux Build 1402 is now running using a merge of 44dac929140bfcd7d7cb1705fcff9c28b3ad87d9

Windows Build 1069 is now running using a merge of 44dac929140bfcd7d7cb1705fcff9c28b3ad87d9

Linux Build 1402 outcome was SUCCESS using a merge of 44dac929140bfcd7d7cb1705fcff9c28b3ad87d9
Summary: Tests passed: 3090, ignored: 64 Build time: 0:18:41

Windows Build 1069 outcome was SUCCESS using a merge of 44dac929140bfcd7d7cb1705fcff9c28b3ad87d9
Summary: Tests passed: 3090, ignored: 64 Build time: 0:19:53

Core - Full Integration Build 650 outcome was FAILURE using a merge of 44dac929140bfcd7d7cb1705fcff9c28b3ad87d9
Summary: Tests failed: 39 (39 new), passed: 2789, ignored: 412 Build time: 0:55:24
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).failoverOnStop: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).nestedBeanFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).connectionFactoryFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).jmsConnectionFactoryFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).noFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulFailoverTestCase(SYNC-tcp).simpleFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.stateful.StatefulTimeoutTestCase(SYNC-tcp).timeout: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.xpc.StatefulWithXPCFailoverTestCase(SYNC-tcp).testBasicXPC: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.xpc.StatefulWithXPCFailoverTestCase(SYNC-tcp).testSecondLevelCache: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.remote.RemoteFailoverTestCase(SYNC-tcp).testSecureStatelessFailover: <no details avaliable>

org.jboss.as.test.clustering.cluster.ejb.remote.RemoteFailoverTestCase(SYNC-tcp).testStatelessFailover: <no details avaliable>

##### there are 29 more failed tests, see build details

Linux Build 1427 is now running using a merge of c1d401f46ac34f71224b95903ac97905e0a51de1

Core - Full Integration Build 690 is now running using a merge of c1d401f46ac34f71224b95903ac97905e0a51de1

Windows Build 1092 is now running using a merge of c1d401f46ac34f71224b95903ac97905e0a51de1

Linux Build 1427 outcome was SUCCESS using a merge of c1d401f46ac34f71224b95903ac97905e0a51de1
Summary: Tests passed: 3077, ignored: 65 Build time: 0:18:26

Windows Build 1092 outcome was SUCCESS using a merge of c1d401f46ac34f71224b95903ac97905e0a51de1
Summary: Tests passed: 3077, ignored: 65 Build time: 0:19:46

Core - Full Integration Build 690 outcome was SUCCESS using a merge of c1d401f46ac34f71224b95903ac97905e0a51de1
Summary: Tests passed: 2826, ignored: 412, muted: 1 Build time: 0:48:24

Windows Build 1104 is now running using a merge of ebf18cee3ab87cbb9e35baaa2bd26f9eab08f44a

Linux Build 1444 is now running using a merge of ebf18cee3ab87cbb9e35baaa2bd26f9eab08f44a

Core - Full Integration Build 704 is now running using a merge of ebf18cee3ab87cbb9e35baaa2bd26f9eab08f44a

Linux Build 1444 outcome was FAILURE using a merge of ebf18cee3ab87cbb9e35baaa2bd26f9eab08f44a
Summary: Tests failed: 20 (20 new), passed: 2769, ignored: 52 Build time: 0:11:20
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.SSLMasterSlaveTwoWayTestCase.testReadSlaveStatusFromMaster: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.SSLMasterSlaveTwoWayTestCase: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.ServerManagementTestCase: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.suspendresume.DomainSuspendResumeTestCase: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.suspendresume.DomainGracefulShutdownTestCase: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.AdminOnlyPolicyTestCase: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.DefaultInterfaceOveridingDomainTestCase: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.AdminOnlyModeTestCase: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.suites.CLITestSuite: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.suites.FullRbacProviderRunAsTestSuite: <no details avaliable>

org.jboss.as.test.integration.domain.suites.FullRbacProviderPropertiesRoleMappingTestSuite: org.jboss.as.test.integration.domain.suites.DomainTestSuite: <no details avaliable>

##### there are 10 more failed tests, see build details

Windows Build 1104 outcome was SUCCESS using a merge of ebf18cee3ab87cbb9e35baaa2bd26f9eab08f44a
Summary: Tests passed: 3078, ignored: 65 Build time: 0:20:33

Core - Full Integration Build 704 outcome was SUCCESS using a merge of ebf18cee3ab87cbb9e35baaa2bd26f9eab08f44a
Summary: Tests passed: 2828, ignored: 412 Build time: 0:52:04


[MODULE] - Price extractor

Description
I want to find prices in texts, e.g. "This notebook costs 2$". This module will output "2$". We make use of the spacy entity labelling to find entities that are labelled as MONEY.
Implementation
from typing import Dict, List, Tuple, Any
import spacy

def price_extractor(record: Dict[str, Any]) -> List[Tuple[str, int, int]]:
    text = record["text"]
    nlp = spacy.load("en_core_web_lg")
    doc = nlp(text)

    for entity in doc.ents:
        if entity.label_ == 'MONEY':
            yield "price", entity.start, entity.end

Additional information

Make sure that spacy is installed if running on the local machine. This can be done by running pip install -U spacy.
The module uses tokenizer to convert the text into a sequence of tokens. We strongly recommend to use the large models (e.g. en_core_web_lg). This can be installed by running python -m spacy download en_core_web_lg on the terminal.

I was building an extractor recently, and this brick can definitely be extended.
E.g., see this code:
import re
import json

def extract_currency_values(text):
    # Regex patterns for currency symbols and ISO codes
    currency_symbols = {
        "usd": r"\$[ ]*([\d,]+\.?\d*)",
        "eur": r"€[ ]*([\d,]+\.?\d*)",
        "cad": r"C\$[ ]*([\d,]+\.?\d*)",
    }

    currency_codes = {
        "usd": r"USD[ ]*([\d,]+\.?\d*)",
        "eur": r"EUR[ ]*([\d,]+\.?\d*)",
        "cad": r"CAD[ ]*([\d,]+\.?\d*)",
    }

    results = []

    # Check for matches using currency symbols
    for currency, pattern in currency_symbols.items():
        matches = re.findall(pattern, text, re.IGNORECASE)
        for match in matches:
            results.append({
                "currency": currency.upper(),
                "value": match.replace(",", "")
            })

    # Check for matches using currency codes
    for currency, pattern in currency_codes.items():
        matches = re.findall(pattern, text, re.IGNORECASE)
        for match in matches:
            results.append({
                "currency": currency.upper(),
                "value": match.replace(",", "")
            })

    return json.dumps(results, indent=4)

# testing the function
print(extract_currency_values("USD 100,000, CAD100000, EUR100.000, 45€"))

In this case, i also grab the USD, CAD, EUR values as well.
@LeonardPuettmannKern


Cannot export public key to PEM

Hello, I'm trying to use the method to_pem_pkcs1 to export my public key and send it over the network. However I have an error telling me that it does not exist. Is this functionality available over crates.io? I'm using the version 0.3.0.
Here is my code:
use rsa::{PublicKey, RSAPrivateKey, RSAPublicKey, PaddingScheme, PublicKeyPemEncoding};
use rand::rngs::OsRng;

pub struct PloufClient {
    pub public_key: RSAPublicKey,
    pub pem: String,
    private_key: RSAPrivateKey,
}

pub fn init_client_session() -> PloufClient {
    let bits = 2048;
    let private_key = RSAPrivateKey::new(&mut OsRng, bits).expect("Failed to generate a key");
    let public_key = RSAPublicKey::from(&private_key);
    let pem = public_key.to_pem_pkcs1();

    PloufClient {
        private_key: private_key,
        pem: pem,
        public_key: public_key,
    }
}

And here is the error when compiling:
error[E0432]: unresolved import `rsa::PublicKeyPemEncoding`
 --> client/src/plouf_client.rs:1:66
  |
1 | use rsa::{PublicKey, RSAPrivateKey, RSAPublicKey, PaddingScheme, PublicKeyPemEncoding};
  |                                                                  ^^^^^^^^^^^^^^^^^^^^ no `PublicKeyPemEncoding` in the root

error[E0599]: no method named `to_pem_pkcs1` found for struct `RSAPublicKey` in the current scope
  --> client/src/main.rs:16:91
   |
16 |     let connection_message = format!("connect:{}:{:?}", username, plouf_client.public_key.to_pem_pkcs1());
   |                                                                                ^^^^^^^^^^^^ method not found in `RSAPublicKey`

Thanks!

Well, it turns that I needed to add a cfg parameter with the pem feature:
#[cfg(feature="pem")]
fn init_client_session() -> PloufClient {
    let mut rng = OsRng;
    let bits = 2048;
    let private_key = RSAPrivateKey::new(&mut rng, bits).expect("failed to generate a key");
    let public_key = RSAPublicKey::from(&private_key);
    let pem = public_key.to_pem_pkcs1();

    PloufClient {
        private_key: private_key,
        pem: pem,
        public_key: public_key,
    }
}

It would be nice to have a little bit of documentation on this part (if I haven't missed it).

It'd probably be good to add doc_cfg ala the other RustCrypto crates to document which features are needed for certain methods to be available


Inconsistent HTML -> LaTeX conversion

[x] I have verified in the Zotero addons screen that I have the latest release of BBT (https://github.com/retorquere/zotero-better-bibtex/releases/latest) and Zotero installed
[x] I am posting a single question, bug or feature request
[x] I have included a descriptive subject of the problem
[x] I am available for follow-up questions and testing
[x] I have included the error-report ID here after selecting the problematic reference(s), right-clicking, and submitting an BBT error report from that popup menu

Report ID: TT3NAPUJ-euc
Exporter used: Better BibLaTeX
Expected behavior: I think that if the title in Zotero has HTML formatting like ... it should be converted to LaTeX in the exported .bib file.
Example title 1: "Spectroscopic and laser characteristics of Ti:Al2O3"
Example title 2: "Vibrational energy transfer in OH X 2Πi, v=2 and 1"
Actual behavior: For some references the output passes through the HTML tags, but for other references it does a fine job at converting to LaTeX.
Example 1 output: title = {Spectroscopic and laser characteristics of Ti:Al2O3}
Example 2 output: title = {Vibrational Energy Transfer in {{OH X}} {\textsuperscript{2}}{{Π}}{\textsubscript{{\emph{i}}}}, {\emph{V}}=2 and 1}
I'm not sure what explains the different behavior. Am I missing something, or is this a bug? Thanks!

Example 1 has tag #LaTeX which puts bbt in raw latex export mode - with that tag on an item, bbt assumes you have raw latex in names and titles and won't escape anything.

Thanks, that fixes it. I didn't know about that behavior.
Out of curiosity, do you happen to know how the #LaTeX tag gets added? Does Zotero decide to add it automatically when some references are imported?

#LaTeX is BBT specific, it's not something Zotero would do. At one point BBT could be configured (off by default) to always add that tag when importing bibtex, but that was a long time ago, and to turn it on you'd have to go into hidden preferences - it's not something you could do by accident.
In other words, I have no idea how that tag was added to that reference.

Huh, interesting. But thanks again, really glad I learned about the tag. Feel free to close.


EIDSCA.CP03: Default Settings - Consent Policy Settings - Block user consent for risky apps

Issue appears to be checking the wrong setting.
Issue text describes the check as https://graph.microsoft.com/beta/settings
.values | where-object name -eq 'BlockUserConsentForRiskyApps' | select-object -expand value = 'true'
Correct check seems to be under https://graph.microsoft.com/v1.0/policies/authorizationPolicy and should be allowUserConsentForRiskyApps =  'false'

Both API endpoints should already been implemented in EIDSCA:
EIDSCA.CP03 = https://graph.microsoft.com/beta/settings
EIDSCA.AP09 = https://graph.microsoft.com/v1.0/policies/authorizationPolicy
Do you have different results of both checks or invalid results?
In case of endpoints with identical/similar checks, we have decided to include both.
See related issue: https://github.com/maester365/maester/issues/72

I'm working in a new tenant (less than a week old).
https://graph.microsoft.com/v1.0/policies/authorizationPolicy returns
{
<EMAIL_ADDRESS>"https://graph.microsoft.com/v1.0/$metadata#policies/authorizationPolicy/$entity",
<EMAIL_ADDRESS>"Use $select to choose only the properties your app needs, as this can lead to performance improvements. For example: GET policies/authorizationPolicy?$select=allowedToSignUpEmailBasedSubscriptions,allowedToUseSSPR",
"id": "authorizationPolicy",
"allowInvitesFrom": "none",
"allowedToSignUpEmailBasedSubscriptions": false,
"allowedToUseSSPR": false,
"allowEmailVerifiedUsersToJoinOrganization": false,
"allowUserConsentForRiskyApps": false,
"blockMsolPowerShell": true,
"displayName": "Authorization Policy",
"description": "Used to manage authorization related settings across the company.",
"guestUserRoleId": "2af84b1e-32c8-42b7-82bc-daa82404023b",
"defaultUserRolePermissions": {
"allowedToCreateApps": false,
"allowedToCreateSecurityGroups": false,
"allowedToCreateTenants": false,
"allowedToReadBitlockerKeysForOwnedDevice": true,
"allowedToReadOtherUsers": true,
"permissionGrantPoliciesAssigned": []
}
}
https://graph.microsoft.com/beta/settings returns no values ...
{
<EMAIL_ADDRESS>"https://graph.microsoft.com/beta/$metadata#settings",
<EMAIL_ADDRESS>"Use $select to choose only the properties your app needs, as this can lead to performance improvements. For example: GET settings?$select=displayName,templateId",
"value": []
}
I suspect "settings" have moved somewhere else. For example, the docs pages now say that Group Consent is gone and RSC is the replacement (which has an entirely different API).
An older tenant I have is well populated under settings.

I've started to implement a discovery option if a setting value is not available under this beta endpoint. Do you have noticed any other check with relation on "settings" endpoint that seems to be affected?

Failures I seem to get for this group of settings are
EIDSCA.CP01: Default Settings - Consent Policy Settings - Group owner consent for apps accessing data.
EIDSCA.CP04: Default Settings - Consent Policy Settings - Users can request admin consent to apps they are unable to consent to.
EIDSCA.PR01: Default Settings - Password Rule Settings - Password Protection - Mode.
EIDSCA.PR03: Default Settings - Password Rule Settings - Enforce custom list.
Might be worth trying this in another new tenant for comparison.

This issues should be fixed now by the following PR:
https://github.com/maester365/maester/pull/318


JSON-RPC: Add eth_createAccessList RPC call

WIPPPPP! @timbeiko :D, replaces https://github.com/ethereum/eth1.0-specs/pull/196

@lightclient asked

Is this field optional? If an error is returned, is accessList and gasUsed omitted?

No, this field is not optional. If an error is returned the accessList and gasUsed are still returned. Only if the method creates a real error (state not available, block in the future, ...) then nothing is returned.
Returning the vmErr and the  accessList and gasUsed together is still useful in some cases.

The implementation in geth for this call can be found here:
https://github.com/ethereum/go-ethereum/pull/22550
(Interesting part: https://github.com/ethereum/go-ethereum/pull/22550/files#diff-c426ecd2f7d247753b9ea8c1cc003f21fa412661c1f967d203d4edf8163da344R1332)
Basically we implement the following algorithm:
al <- tx.AccessList // Take the current accesslist
for {
   al2 <- execute(tx, al) // execute the transaction
   if al == al2 {
      return al // nothing changed between runs
   }
   al = al2
}

This feels inconsistent with the result of debug_traceTransaction, which has the failed and returnValue properties. Would it make sense to use those properties instead?

@MariusVanDerWijden is this something you're still working on or can it be closed?

@lightclient I'm not sure, did someone port this to the new spec already? If not I would like to keep it open as a reminder that this RPC call exists

@MariusVanDerWijden I see this was supposed to replace #196, but that was merged. So I'm not sure this is still a WIP or if #196 adequately describes the method?
I will port any new methods merged to this repo into the eth1-apis repo after London.

Yeah Tim merged #196 pretty quickly. but I wasn't really finished with the discussions yet, so he reverted the change. The codebase doesn't contain the eth_createAccessList RPC yet.

@MariusVanDerWijden ah right, I remember now. Should this PR be merged then (i.e. does it accurately describe the endpoint)? I can port it to the new spec once it's merged here.

Should be migrated to https://github.com/ethereum/eth1.0-apis


Redis version bump

Updating the versions of redis used by CI for integration testing. Changes were made only to the github actions, there are no code changes.

Codecov Report
Patch coverage has no change and project coverage change: -2.48% :warning:

Comparison is base (78be574) 93.82% compared to head (a765325) 91.34%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #163      +/-   ##
==========================================
- Coverage   93.82%   91.34%   -2.48%
==========================================
  Files          82       82
  Lines        5082     5082
  Branches      476      476
==========================================
- Hits         4768     4642     -126
- Misses        185      313     +128
+ Partials      129      127       -2

see 8 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@chayim the CI failed, do you want to fix it?


Accept empty publish elements in RRDP.

Resolves #137.

It wasn't entirely clear to me looking at the code that parsing an empty element as base64 would work. But given the tests you have added I see it does.

There’s an unwrap_or_else(Vec::new) in ObjectReader::process that injects an empty vec if the XML decoder returns no text.

Yeah, I saw that but I was confused whether that is valid base64 and translates to an empty body. I guess it makes sense that it does :)


Object Ids are Cached after Deletion

Describe the bug
After deleting a file using the checkbox and delete button at the top and subsequently trying to download or delete using the top buttons cause errors as BCBox is trying to download/delete the original object id.
To Reproduce
Steps to reproduce the behavior:

Go to a bucket

Upload a file

Select the checkbox to the left of the file name

Use the Delete button at the top (next to the Upload and Download buttons)

File is deleted

Use the Download or Delete buttons at the top

Error is encountered

Trying to download a file after deleting it:
Trying to delete a file after deleting it:

Expected behavior
The Download and Delete buttons should not retain previous Object Ids after they have been deleted.
Screenshots
Desktop (please complete the following information):

OS: Windows 10
Browser: Edge
Version: 117.0.2045.31

Jira issue 3364

Resolved in #136


chore: change to vitest and remove cjs bundling.

closes #1072, closes #1055
This is a change that is part of a bigger goal to improve our confidence in browser testing and update the codebase to adapt to node/browser environment without throwing errors:

adds Vitest as the test framework that has browser testing.
drops karma. eventually drop add-ins like Sinon used for mocking and completely build on Vitest.
updates test imports to use Vitest API.
adds browser testing for the packages.
adds utility function to check the environment(node/browser) the code is running in.
Fixes browser incompatible code in the codebase.
Removes common js bundling tooling and support.

Status

[x] abstractions package is updated to vitest and has browser tests
[x] authentication/azure package is updated to vitest and has a trigger for browser tests that pass
[x] authentication/spfx is updated
[x] http is updated.
[x] serialization/* is updated.

@baywet I'm having a blocker with completing this. When running the steps to build and test, there's an error with building that comes from having build instructions in tsconfig that outputs commonjs. I think it would be a good idea to make a decision on #1055 to unblock this. What do you think?

Agreed, let's get confirmation on the PM side.
If Seb doesn't reply within a few days, please reach out to him directly

Hi @musale,
Can you please give instructions on how to run the build and tests locally. this is what I get:
npm run build

npm run test

Hi @musale, Can you please give instructions on how to run the build and tests locally. this is what I get:
npm run build

npm run test

Hi @koros this CI/CD pipeline and build is adapted for a Linux environment. I see you're using PowerShell and this will fail for commands like clean and path resolutions. Please use WSL or a Linux environment to test this.

Hi @musale, Can you please give instructions on how to run the build and tests locally. this is what I get:
npm run build

npm run test

Hi @koros this CI/CD pipeline and build is adapted for a Linux environment. I see you're using PowerShell and this will fail for commands like clean and tsconfig file path resolutions. Please use WSL or a Linux environment to test this.

@musale this seems to resolve the issue on Win OS
    "build:@microsoft/*": "lerna run build --scope \"@microsoft/*\"",
    "clean": "lerna run --parallel --stream --scope \"@microsoft/*\" clean",
    "test:browser": "lerna run --scope \"@microsoft/*\" test:browser",
    "test:node": "lerna run --scope \"@microsoft/*\" test:node",


ARROW-9546: [Python] Clean up Pandas Metadata Conversion test

Cosmetic improvement to a pyarrow Pandas test. I rewrote the metadata conversion test using _check_pandas_roundtrip.

Thanks!


Added APIs and included install headers

This is to help support ibm-acf upload feature.
Added isTimeExpired() and decodeAndVerifySignature()
as library calls for support within BMC

corequisites:
bmcweb/130
ibm-acf/1
phosphor-dbus-interfaces/27
phosphor-certificate-manager/9

PR approved in GHE: https://github.ibm.com/openbmc/ibm-acf/pull/13


Update cats version

Current coverage is 100.00%

Merging #51 into master will not affect coverage as of d319afe

@@            master     #51   diff @@
======================================
  Files           17      17
  Stmts          447     447
  Branches        37      37
  Methods          0       0
======================================
  Hit            447     447
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of d319afe

Powered by Codecov. Updated on successful CI builds.


Fix high CPU usage by the async-io thread

@Byron wanna check if this fixes the issue reported in #6?
The async-io thread is supposed to only "help" push things forward when Parkers are not actually parking or when there are no Parkers at all. Previously, the async-io thread would be too aggressive, competing with Parkers to grab the lock on the Reactor too often, rather than giving priority to Parkers and only waking up when the reactor is idling.
This could be further improved. On Unix systems, we could use kqueue timers or timerfds to not wake up the async-io thread at all unless the reactor hasn't been polled in X milliseconds. But I hope this PR is good enough for now.

Fantastic - that works! prodash is back to the usual CPU usage when idle, which is dominated by the TUI (unfortunately so, but that's for another day and another repository ;)).

Thank you for testing! :)


Create Community Standards [Technical Writer]

GitHub Community standards : https://github.com/briangormanly/agora/community
Create documents to define code of conduct, contributing, issue and pull request templates
Make sure to work with project management personal on issue template, engineering leads for pull request templates. Research best standards for all documents using established existing open source examples.

Likes added


Mention the :extend face attribute for Emacs 27+.

In the documentation string of the user option
selectrum-extend-current-candidate-highlight
In the documentation string of the face
selectrum-current-candidate
In README.md

This was maybe relevant in #591, and I didn't see it stated explicitly anywhere.


Cargo.toml: Add docs.rs documentation link

ping :-)

Sorry, I'm not sure that it is correct, because link to docs.rs is just generated documentation, while the most important part is written in README.md.

I've added a link to readme as documentation: 58e454b0e5579710d02805251b1baf1d5f4b7551. Thanks for the pointing to the issue.


Proposal: Add inject function/struct ability to go test for mock

When I use go test for unit test, I found it is hard (maybe no way) to mock a function or a struct object inside a struct.
---------------------------
//download.go
---------------------------
func get(url string) string {
    resp, err := http.Get(url)
    if err != nil { return "" }
    defer resp.Body.Close()

    contents, err := ioutil.ReadAll(resp.Body)
    if err != nil { return "" }
    return string(contents)
}

func downloader() string {
    content := get(BASE_URL)
    res := processString(content)
    return res
}

I think we could give a more powerful ability to go test (xxx_test.go), that we can create a mock function or a mock struct, and inject it and replace original symbol at compile phase. So we have a more flexible way to write the codes, not worry about "hardwire" function or struct in codes. Otherwise, we need to interface everything, just for testing and mock the object.
--------------------------------------------
download_test.go
--------------------------------------------
//inject/mock get
func get(url string) string {
    return string("hello world!")
}

func TestDownloader(t *testing.T){
    res := downloader()
    if res != "hello world!" {
       t.Error("fail")
    }
}

Sorry, I'm not a native speaker. Hope I explained my idea clearly.

Sounds like an xy problem. Please show some code illustrating the problem you want to solve. Point out what you need but cannot do. Thanks.

Yes. See this question in link. https://stackoverflow.com/questions/19167970/mock-functions-in-go.
The solutions in that page, are not straight. Need to introduce another thing, like a interface or intermediate variable.

I edit my proposal, hope it is easy to understand now.

The testing package is deliberately minimal and avoids magic code rewriting like this. For complex cases there is the gomock tool, although it is a bit heavy-weight to use. In this specific case, where you want to override code you wrote, you can add to it (in the real copy of the code):
var getHook func(string) string

func get(url string) string {
    if getHook != nil { return getHook(url) }
    ... real code
}

and then in your test you can have your specific test case set and restore getHook:
func Test(t *testing.T) {
    getHook = myGet
    defer func() { getHook = nil }()
    ... your test ...
}


IndexError: map::at when using tl.dot

I am working on my first triton kernel and I am running into the following error at runtime.
  File "/opt/conda/lib/python3.10/site-packages/triton/runtime/jit.py", line 161, in <lambda>
    return lambda *args, **kwargs: self.run(grid=grid, warmup=False, *args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/triton/runtime/jit.py", line 395, in run
    self.cache[device][key] = compile(
  File "/opt/conda/lib/python3.10/site-packages/triton/compiler/compiler.py", line 225, in compile
    next_module = compile_ir(module, metadata)
  File "/opt/conda/lib/python3.10/site-packages/triton/backends/cuda/compiler.py", line 560, in <lambda>
    stages["llir"] = lambda src, metadata: self.make_llir(src, metadata, options, self.capability)
  File "/opt/conda/lib/python3.10/site-packages/triton/backends/cuda/compiler.py", line 477, in make_llir
    pm.run(mod)
IndexError: map::at

Here's a minimal reproduction

import torch
import triton
import triton.language as tl
import numpy as np

@triton.jit
def my_kernel(output_ptr, input_ptr, h_ptr, n_cols, stride, BLOCK_SIZE: tl.constexpr):
    row_idx = tl.program_id(0)

    input_start_ptr = input_ptr + row_idx * stride
    output_start_ptr = output_ptr + row_idx * stride

    col_offsets = tl.arange(0, BLOCK_SIZE)
    input_ptrs = input_start_ptr + col_offsets
    output_ptrs = output_start_ptr + col_offsets
    h_ptrs = h_ptr + col_offsets

    mask = col_offsets < n_cols
    x = tl.load(input_ptrs, mask=mask, other=0.0)
    h = tl.load(h_ptrs)
    assert(BLOCK_SIZE == 256)
    H = tl.reshape(h, (16, 16)) # will make this more general once BLOCK_SIZE == 256 works
    X = tl.reshape(x, (16, 16))
    #HX = H + X
    HX = tl.dot(H, X)
    HXflat = tl.ravel(HX)
    tl.store(output_ptrs, HXflat, mask=mask)

def makeH():
    H2 = np.array([[1, -1], [1, 1]])
    H4 = np.kron(H2, H2)
    H16 = np.kron(H4, H4)
    return torch.tensor(H16.ravel(), dtype=torch.float32).cuda()

Hcuda = makeH()

def foo(x):
    n_rows, n_cols = x.shape
    BLOCK_SIZE = triton.next_power_of_2(n_cols)
    print(f'{BLOCK_SIZE=}')
    y = torch.empty_like(x)
    assert x.stride(0) == y.stride(0)
    my_kernel[(n_rows, )](
        y,
        x,
        Hcuda,
        n_cols,
        x.stride(0),
        BLOCK_SIZE=BLOCK_SIZE
    )
    return y

x = torch.randn(size=(32, 256)).cuda()
output = foo(x)
print(output)

My environment is this Dockerfile
FROM pytorch/pytorch:2.0.1-cuda11.7-cudnn8-runtime

RUN apt update && apt install -y git wget clang gcc
RUN pip install notebook matplotlib pandas
RUN pip install -U --index-url https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/Triton-Nightly/pypi/simple/ triton-nightly

using the latest triton nightly package
My GPU has a compute capability of 7.5
The code runs without the above error if HX = tl.dot(H, X) is replaced with the commented line above HX = H + X.

Can anyone confirm whether they can repro this?

I can reproduce this error on an RTX 2070 with CUDA 12.1 and a driver version of 530 but not on an A100 or T4. Setting allow_tf32 to False resolves the problem, although other solutions not involving disabling TF32 would be more ideal.

Thanks @BobMcDear for trying all these different setups.
I can also verify on my setup that
tl.dot(H, X, allow_tf32=False)

resolves the problem and looking forward to a better solution from the team.

Just ran into this while using JAX's pallas on a Quadro RTX 6000, CUDA 12.3.

It is common for people to develop and debug on a lower end GPU without support for TensorFloat32, then deploy the same code on a GPU that supports it. I think a good default argument should be None rather than True/False and it should mean that behavior is determined by whether support for TensorFloat32 has been detected.

@n17s I believe the intended behaviour of allow_tf32=True is what you describe: It signals to Triton to utilize TF32 if and only if it is available, similar to activating TF32 on PyTorch. Indeed, in versions 2.0.0 and 2.1.0, this error does not occur even with the said argument set to True, and it is only by upgrading to 2.2.0 that I am able to reproduce it. Also, regarding my previous comment, I had 2.1.0 installed on the T4 and that is why I did not encounter the error.

Can confirm I get this error on a T4 with triton 2.2.0, and I can resolve it by setting allow_tf32=False (in your case: HX = tl.dot(H, X, allow_tf32=False))


The user should be able to disable the confirmation popup

Is your feature request related to a problem? Please describe.
The user always has to confirm when deleting a file, some people might not want to do this.
This should've been done in the issue #316, but it has been forgotten.
Describe the solution you'd like

[ ] Add a setting to disable the confirmation popup. This setting should go under assistance, probably.
[ ] We should also add a button to "Never show this popup again" that toggles the same setting.

Setting: 1u
Never show again: 1u
Todal ~2u

:tada: This issue has been resolved in version 2.19.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update jenkins

@JimLau49 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@JimLau49 Thank you for signing the Contributor License Agreement!

@JimLau49 can you check with the teacher if the instructions be amended please? We've had a number of unexpected PRs submitted here for this and this creates noise for no good reason.

Yeah for sure, ill send him an email. Sorry about the trouble we're causing :(


Refactor InfluxDB scaler config

Refactor the scaler to match the new metadata guidelines.
Checklist

[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Tests have been added
[x] Changelog has been updated and is aligned with our changelog requirements
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #

Related: https://github.com/kedacore/keda/pull/5797

/run-e2e influx

/run-e2e influx


Third part file name will be added hash if file name is duplicated local file.

Bug report

What is the current behavior?
I use facebook sdk for live video and import fb sdk,  i also have a file named sdk.js, when building with webpack,  the fb sdk file will be add hash.

Before

After

If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?

Other relevant information:
webpack version:    v4.41.0
Node.js version:      v12.18.2
Operating System:   mac os
Additional tools:

Thank you for creating this issue. However, issues need to follow one of our templates so that we can clearly understand your particular circumstances.
Please help us help you by recreating the issue using one of our templates.
Please fill out all fields in the issue template


Incorrect precedence for the range operator.

var n = 10
System.print(1..n-1)

In the code above, the AST generated is wrong. It is being parsed as (1..n) instead of 1..(n-1)

Fixed in commit ea4f55f.


Support definition files.

Fixes #15.

N.b. this needs more testing. I'm getting all sorts of weird errors, and I'm not sure whether they're related to this or not.

Talk to me, Chis.

I'm going to test this later; I suspect – though I do not yet have any proof – that fixing the tsconfig may have resolved this.

NOPE. Resolved by #22.


chore!: upgrade ethers & roles sdk

This PR upgrades ethers from v5 to v6 and to the latest Roles SDK. Breaking changes are documented in the ethers migration guide: https://docs.ethers.org/v6/migrating/

Can we please change the ci to run on self-hosted?

Can we please change the ci to run on self-hosted?

How exactly does the available self-hosted runner look like? Can we review this somewhere? And what would be the advantage using it over the standard runners?
Worth noting that GitHub warns about using self-hosted runners for public repos:

https://docs.github.com/en/actions/hosting-your-own-runners/managing-self-hosted-runners/adding-self-hosted-runners


I can't get force feeding to work

This doesnt work...
    $node.insertBefore(next)
      .velocity {translateX: ['0%', '100%']},
        duration: 500
        easing: 'ease-in-out'
        queue: false

It only works when I use a hook:
    $.Velocity.hook($node, "translateX", "100%");
    $node.insertBefore(next)
      .velocity {translateX: '0%'},
        duration: 500
        easing: 'ease-in-out'
        queue: false

Am I missing something?

fiddle :)

lol.
What should I fiddle with? Is this your way of saying that it should would?

most errors are due to users' own code. as is stated in the contributing faq, submitting q/bug with fiddle helps remove that uncertainty so that people helping out can be extra certain they're not wasting their time.
not saying that reflects in any way on your particular question

And assuming you're familiar with http://jsfiddle.net/

Ok. So I've fiddled around a little more ;)
Force-feeding works. But on certain browsers (every time with firefox, sometimes with Safari, never with Chrome), you'll see a brief flash when the element is inserted before velocity catches it and moves it.
For example, check out the slide left button on this demo: http://transition-demo.meteor.com/
P.S. I'm using velocity to make a much needed page transitioning package for meteor: https://github.com/ccorcos/meteor-transitioner
And great job with this framework.

Ok. Here we go:
Here's the code:
https://gist.github.com/anonymous/23f0f73678bc81354908
and here it is on CodePen:
http://codepen.io/ccorcos/pen/gbxGpZ
You'll notice there is a flash of red when you press the button

@ccorcos that's because how you're inserting the second page into the DOM dynamically on click. Try giving us a clean example where the elements are present in the DOM before you move them around with Velocity.
To be more specific, it's probably due to the fact you're inserting an element that's initially in view when inserted.

Yeah, I see. The problem for me is that this is the way meteor works. I'm given and insert and remove function and I need to to insert it into the dom and do this animation. The odd thing is that any of the built-in effects don't have this flickering problem:
https://github.com/julianshapiro/velocity/blob/master/velocity.ui.js#L535
But the ones I wrote do have this problem:
$.Velocity.RegisterEffect 'transition.pushRightIn',
  defaultDuration: 500,
  calls: [
    [{translateX: ['0%', '100%'], translateZ: 0, easing: "ease-in-out"}]
  ]

$.Velocity.RegisterEffect 'transition.pushLeftOut',
  defaultDuration: 500,
  calls: [
    [{translateX: ['-100%', '0%'], translateZ: 0, easing: "ease-in-out"}]
  ]

Notice the difference between slideLeft and slideRight in this demo: http://transition-demo.meteor.com
Any ideas?

Taking a look now.

The UI pack addresses the flashing by immediately setting opacity upfront: https://github.com/julianshapiro/velocity/blob/master/velocity.ui.js#L134.
I would try doing that with your code as well. Let me know how that works out.

Interesting. I see whats going on here. Every effect animates opacity. Is it possible to immediately animate the opacity to prevent the flash?
Here's what I have going right now...
$.Velocity.RegisterEffect('transition.pushRightIn', {
  defaultDuration: 500,
  calls: [
    [
      {
        translateX: ['0%', '100%'],
        translateZ: 0,
        easing: "ease-in-out"
      }
    ]
  ]
});

oh shit, you're right! it's because you have to explicitly animate opacity in there. either throw it in as a dummy property (E.g. opacity: [ 1, 1]) or manually set opacity to 0 before the animation begins

Much better! That works. Thanks.

i suggest to remove the style attribute after the transition complete
 var start = '100%';       $.Velocity.hook(node, 'translateX', start);       $(node)       .insertBefore(next)       .velocity({translateX: [0, start]}, {         duration: ANIMATION_DURATION,         easing: 'ease-in-out',         queue: false,         complete: function() {           $(node).removeAttr("style");         }       });


Can't get default printer under Apple sandbox

Scenario:
try to get default printer in sandboxed app
cupsGetNamedDest(CUPS_HTTP_DEFAULT, 0, 0)
when specific default printer is set in System Preferences - Printers.
Expected:
default printer
Actual:
IPP_STATUS_ERROR_NOT_FOUND
What happens:
libcups tries to access ~/.cups/loptions to get default printer
Apple does provide entitlement com.apple.security.print which does allow that
BUT
cups uses getenv("HOME") to get home folder pathwhich returns ~/Containers/{bundle id}/Data under sandbox
while it should use real home folder (getpwuid(getuid())->pw_dir)

Hmm, this seems to be working in Safari (also sandboxed). What version of macOS and CUPS are you using?

That's on macOS 10.15.1 / CUPS 2.3.0
Safari uses PrintCore which talks to cups via printtool daemon.
I.e. PMSessionGetCurrentPrinter works fine, but we would like to use cups directly as well.

OK, I've created a short test program to reproduce this, and confirmed that ~/.cups can be safely read in a sandboxed application.
Changes will be a little more involved (need to replace all uses of getenv("HOME")) but will see what I can do...

[master e2eb28cfc] Sandboxed applications were not able to get the default printer (Issue #5676)
Unfortunately, that doesn't do much to help you now - you'll need to wait until this fix makes it into a macOS release...

Thanks, we can work with that.


Staking Amount Entered Is Wrong in the Verify Transaction Pop-Up Window

After I have entered the amount of EOS I want to stake for the CPU and Network, I have noticed that in the verify transaction pop-up window that appears on the screen the staking amount shown is different than what I specified.
For Example: I wanted to stake 27.3750 EOS for the CPU and 27.3750 for the Network.

On the "Verify Transaction" screen, the staking amount shown for the CPU and Network is a different amount than what I specified. I specified 27.3750 EOS for the CPU and 27.3750 for the Network, but the amount that is shown on the "Verify Transaction" Screen is 27.2750  for the CPU and 27.2750 for the Network.

EDIT: I figured out what is happening. Because I already had 0.2000 staked, 0.2000 EOS was being deducted from the total staking amount (54.7500) I specified I wanted to stake. So the total staking amount (54.550) that is shown in the "Verify Transactions" pop-up window is correct. 54.550 + the 0.2000 I already had staked makes the total staking amount 54.7500 (which is what I wanted).

You are right, thanks for noticing it.


HoldFixed causes value(<fixed_variable>) to return 0.0

A minor issue, but I thought I would report it in case it can be easily fixed or others stumble upon it.
To reproduce:
using JuMP
using GAMS

model = Model(GAMS.Optimizer)
set_optimizer_attribute(model, "HoldFixed", 1)
set_optimizer_attribute(model, GAMS.ModelType(), "CNS")

@variables model begin
    x
    y
end

@constraints model begin
    y == x
end

fix(x, 1.0)
optimize!(model)

@assert value(x) == value(y) "$(value(x)) == $(value(y))"

AssertionError: 0.0 == 1.0
set_optimizer_attribute(model, "HoldFixed", 0)
optimize!(model)
"$(value(x)) == $(value(y))"

"1.0 == 1.0"

Turns out this is not directly due to HoldFixed, but rather it is the case for any variable not actually included in the model:
using JuMP, GAMS
model = Model(GAMS.Optimizer)

@variables model begin
    x
    y
end
fix(x, 1)

@constraints model begin
    y == 1
end

optimize!(model)
@assert value(x) == value(y) "$(value(x)) == $(value(y))"

AssertionError: 0.0 == 1.0

I can confirm the bug. I'll look into it. Thanks very much for reporting!

Fixed with v0.3.2, will be available in a moment.


Remove SSL bypass.

It seems the code to accept all certificates is an unnecessary security risk. These kinds of calls are sometimes needed, for example in development environments, but can be made from client code and do not need to be included in the SDK.

It looks like this is no longer an issue in the API client so I am closing this PR. Thank you.


Import of the bindings module failed after fresh install

System Info
-CPU : x86_64
-GPU : RTX 3090
-TensorRT_LLM 0.9.0
-OS : Windows 10
Who can help?
@byshiue
Information

[X] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
1)Fresh install of everything using the powershell script from the installation guide.
2)pip install tensorrt_llm --extra-index-url https://pypi.nvidia.com --extra-index-url https://download.pytorch.org/whl/cu121
3)python -c "import tensorrt_llm; print(tensorrt_llm._utils.trt_version())"
Expected behavior
Tesorrt_LLM working fine
actual behavior
Traceback (most recent call last):
File "C:\Users\cedri\Desktop\Concours Nvidia\venv\lib\site-packages\tensorrt_llm_init_.py", line 39, in
import tensorrt_llm.bindings  # NOQA
ImportError: DLL load failed while importing bindings: Le module spécifié est introuvable.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\cedri\Desktop\Concours Nvidia\venv\lib\site-packages\tensorrt_llm_init_.py", line 41, in
raise ImportError(
ImportError: Import of the bindings module failed. Please check the package integrity. If you are attempting to use the pip development mode (editable installation), please execute build_wheels.py first, and then run pip install -e .
additional notes
i have also tried to use the version 0.8.0 of Tesnorrt_llm but got the same error. Note that i've no error message during any phases of the installation

Went on Ubuntu and installed without trouble ! Guess the problem come from windows


cannot import numpy.core.multiarray in fb.python

I successfully installed fb.python. However, in loading fb.python I got the following error
th> local py = require('fb.python')
ImportError: numpy.core.multiarray failed to import
/home/fs/ylu/torch/install/share/lua/5.1/trepl/init.lua:363: /home/fs/ylu/torch/install/share/lua/5.1/trepl/init.lua:363: Python error: import numpy array module
ImportError: numpy.core.multiarray failed to import
stack traceback:
[C]: in function 'error'
/home/fs/ylu/torch/install/share/lua/5.1/trepl/init.lua:363: in function 'require'
[string "local py = require('fb.python')"]:1: in main chunk
[C]: in function 'xpcall'
/home/fs/ylu/torch/install/share/lua/5.1/trepl/init.lua:648: in function 'repl'
.../ylu/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:185: in main chunk
[C]: at 0x00405d30
While in python, I can import numpy.core.multiarray with no problem.
>>> import numpy.core.multiarray
>>>
I wonder what the problem is and how to solve it.

The problem is solved by updating numpy to the newest version.

In case anyone else has this issue and updating Numpy doesn't resolve it: check that the version numbers kicked out by cmake when installing fb.python line up with what you expect. For me, the problem turned out to be that some of the Python variables were pointing at an old version of Python, as indicated by
-- Found PythonLibs: /usr/lib64/libpython2.7.so (found version "2.7.8")
-- Found PythonInterp: /home/ajones/miniconda3/bin/python (found version "3.5.1")
-- Found NumPy: version "1.10.2"

when I expected to be working with 2.7.11. Anyway, the quick-and-dirty fix for this is to dive into CMakeLists.txt and set the correct paths:
FIND_PACKAGE(Folly REQUIRED)
FIND_PACKAGE(Glog REQUIRED)
FIND_PACKAGE(Torch REQUIRED)
FIND_PACKAGE(THPP REQUIRED)
FIND_PACKAGE(FBLuaLib REQUIRED)
FIND_PACKAGE(PythonLibs REQUIRED)
FIND_PACKAGE(NumPy REQUIRED)

set(PYTHON_LIBRARIES "/home/ajones/miniconda3/envs/aflow/lib/libpython2.7.so")
set(PYTHON_INCLUDE_DIRS "/home/ajones/miniconda3/envs/aflow/include/python2.7/")

Updating the numpy with sudo pip install -U numpy can solve the problem. Thank you @yaolubrain

Updaing numpy with
sudo pip install -U numpy
doesn't solve it for me.


English support?

Do we have English support? Something like:
model = Model(model_name="vosk-model-tts-en-0.6-multi")

TIA for any help!

Not for now, sorry, we are still in R&D stage. You can use piper meanwhile


Dcat terms in the metadata

We should make sure that we have places in the metadata for DC/Dcat terms, and have the API return proper JSON-LD responses perhaps?

Part of #56 should be solved by this.


Fix: Oak Inn is not accessible if game is updating #84

First approach, tap on the left

Fixes #84.


How to visualize huge bin file (over 2TB CPU Memory)

Is there an existing proposal for this?

[X] I have searched the existing proposals

Is your feature request related to a problem?
I have a huge bin file (over 2TB), How to visualize or extract information (especially large memory consume block or functions). I have limitation physical memory 512GB memory. I execute stats 3
hours.
Describe the solution you'd like
Visualizing or extracting information memory efficient, Not all data put on memory.
Alternatives you considered
No response

@qawnaoya have you considered using https://bloomberg.github.io/memray/run.html#aggregated-capture-files ?

What have you tried? I'd expect that memray stats would be the most likely reporter to work on a capture file that large, followed by memray summary and perhaps memray tree. I think there's virtually no chance of memray flamegraph --temporal working, though a non-temporal memray flamegraph might.
Whether most of these wind up working depends less on the size of the capture file, and more on the number of distinct stacks from which allocations were performed.

Without more info on what has been tried, there's nothing more we can do here.


Added label for 'managedappschemabuilder'

Install log:
$ sudo zsh Installomator.sh managedappschemabuilder DEBUG=0

2022-01-13 18:52:55 managedappschemabuilder setting variable from argument DEBUG=0
2022-01-13 18:52:55 managedappschemabuilder ################## Start Installomator v. 8.0
2022-01-13 18:52:55 managedappschemabuilder ################## managedappschemabuilder
2022-01-13 18:52:55 managedappschemabuilder BLOCKING_PROCESS_ACTION=tell_user
2022-01-13 18:52:55 managedappschemabuilder NOTIFY=success
2022-01-13 18:52:55 managedappschemabuilder LOGO=/System/Applications/App Store.app/Contents/Resources/AppIcon.icns
2022-01-13 18:52:55 managedappschemabuilder no blocking processes defined, using Managed App Schema Builder as default
2022-01-13 18:52:55 managedappschemabuilder Changing directory to /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/tmp.i5daAYvs
2022-01-13 18:52:56 managedappschemabuilder App(s) found:
2022-01-13 18:52:56 managedappschemabuilder could not find Managed App Schema Builder.app
2022-01-13 18:52:56 managedappschemabuilder appversion:
2022-01-13 18:52:56 managedappschemabuilder Latest version of Managed App Schema Builder is 2022-01-13 18:52:55 managedappschemabuilder could not retrieve version number for BIG-RAT/Managed-App-Schema-Builder
2022-01-13 18:52:56 managedappschemabuilder Downloading https://github.com/BIG-RAT/Managed-App-Schema-Builder/releases/download/current/Managed.App.Schema.Builder.zip to Managed App Schema Builder.zip
2022-01-13 18:52:57 managedappschemabuilder no more blocking processes, continue with update
2022-01-13 18:52:57 managedappschemabuilder Installing Managed App Schema Builder
2022-01-13 18:52:57 managedappschemabuilder Unzipping Managed App Schema Builder.zip
2022-01-13 18:52:57 managedappschemabuilder Verifying: /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/tmp.i5daAYvs/Managed App Schema Builder.app
2022-01-13 18:52:57 managedappschemabuilder Team ID matching: PS2F6S478M (expected: PS2F6S478M )
2022-01-13 18:52:57 managedappschemabuilder Downloaded version of Managed App Schema Builder is 0.9.7 (replacing version ).
2022-01-13 18:52:57 managedappschemabuilder Copy /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/tmp.i5daAYvs/Managed App Schema Builder.app to /Applications
2022-01-13 18:52:57 managedappschemabuilder Changing owner to tonystark
2022-01-13 18:52:57 managedappschemabuilder Finishing...
2022-01-13 18:53:07 managedappschemabuilder App(s) found: /Applications/Managed App Schema Builder.app
2022-01-13 18:53:07 managedappschemabuilder found app at /Applications/Managed App Schema Builder.app, version 0.9.7
2022-01-13 18:53:07 managedappschemabuilder Installed Managed App Schema Builder, version 0.9.7
2022-01-13 18:53:07 managedappschemabuilder notifying
2022-01-13 18:53:08 managedappschemabuilder Deleting /var/folders/zz/zyxvpxvq6csfxvn_n0000000000000/T/tmp.i5daAYvs
2022-01-13 18:53:08 managedappschemabuilder App not closed, so no reopen.
2022-01-13 18:53:08 managedappschemabuilder ################## End Installomator, exit code 0

Thank you.
In your output there is a problem with version of the software.
2022-01-13 18:52:56 managedappschemabuilder Latest version of Managed App Schema Builder is 2022-01-13 18:52:55 managedappschemabuilder could not retrieve version number for BIG-RAT/Managed-App-Schema-Builder
…
2022-01-13 18:53:07 managedappschemabuilder Installed Managed App Schema Builder, version 0.9.7

These should match to work, but looks like this software was incorrectly filled out in the version number on the site.

Hey @Theile,
Yes, I noticed that too.
I decided to leave it there in case the developer fixes it in the future. But I'm OK with commenting that line out if it bothers too much :-)

After som consideration I think you should use valuesfromarguments until the software is complete.


time-picker does not accept 24:00 as a valid value, and does thus not conform to ISO8601 5.3.2

This is somewhat mentioned in issue 43:
Issue 43
In Europe (and other military time / 24 hour clock using areas) 24:00 is a valid value so as to distinguish between the start of the day and the end of the day.
I'm aware that 24:00 wouldn't work when saved in a db. (ex. Python datetime.time() accepts only up to 23:59), but from a UX POV. it makes sense for the user to be able to choose 24:00.
As an example, lets take a Shift at a bar a friday night(Our current usecase).
Some shifts start friday at 24:00 and ends saturday morning at 04:00. (Technically the shift is saturday from 00:00 to 04:00 but thats not how the user thinks about it).
What would be nice is some way to make 24:00 a submittable value(and then do the right things in backend). Would that be possible perhabs?
Context:
http://forum.jquery.com/topic/timepickers-should-conform-to-iso8601-5-3-2-on-midnight-representations-accepting-24-00-as-a-valid-value

any update?
moment support k/kk format tokens from version 2.13.0: https://github.com/moment/moment/pull/3102, if time-picker change all format string from h/HH to k/kk, 24:xx will be supported?

Any Updates?

Any Updates ?

Any Updates?

Any Updates?

Hello, are there any updates?


Implement is_monotonic_*

I chose not to implement the plain is_monotonic since this is already deprecated in dask/dask in favor of is_monotonic_increasing

test fails due to https://github.com/dask/dask/pull/10671

thx


Add new points engines for Hebda Cup and Wy-Hi Regatta

Nicely done!  Thanks for the contribution.  Do you have any suggestions for the README to make it easier for someone new to make a contribution?  It'll be active shortly.


Clone a Voice. How to improve?

import os
import torch
from InferenceInterfaces.ToucanTTSInterface import ToucanTTSInterface

if __name__ == '__main__':
    tts = ToucanTTSInterface(device="cuda" if torch.cuda.is_available() else "cpu", tts_model_path="Meta", language="it")

    input_text = "my text to say"

    # Loop through the speaker reference audio files in the folder
    speaker_reference_folder = "input/folder"
    for file_name in os.listdir(speaker_reference_folder):
        if file_name.endswith('.wav'):
            speaker_reference = os.path.join(speaker_reference_folder, file_name)

            # Set the speaker embedding to clone the voice
            tts.set_utterance_embedding(speaker_reference)

            # Synthesize speech with the cloned voice
            output_file_name = "audios/cloned_voice.wav"
            tts.read_to_file(text_list=[input_text], file_location=output_file_name)

    del tts

I used this method to clone a voice, the result is something similar to the original voice but i guess it can improve.
There's a different/better approach to do this?
How big my dataset should be?
Now I've used like 7 sample of 1 minute each
Thanks

Actually look like it only take in cosidaration one file even if I give more as input

If set_utterance_embedding does not give you satisfaction I think you have to fine tune the model (Meta) properly on your dataset.
If you want to only use speaker reference 6 to 12 seconds is enough (a single one per speaker as you guessed already).

I am currently writing one but it takes too much time ! Yet if you carefuly follow this project readme file "quietly" 😉 https://github.com/DigitalPhonetics/IMS-Toucan#build-a-toucantts-pipeline you will be able to fine tune Meta model on your dataset (you'll probably need more data than 7 minutes of audio). Please mind that you need to create your dataset with the transcription of each audio sample (10 sec max). Ask ChatGPT how to generate a dataset for tts training it will give you advice if you need some. You'll also need a GPU for the training.

@vikolaz I have not officially published this yet, but:

it's a good example of specifically what's required
it includes a training set, ish
you can ignore the OpenShift/Jupyter parts. All the code/content would work locally on a system with Python and CUDA

https://github.com/OpenShiftDemos/ToucanTTS-RHODS-voice-cloning

A small update on this: Zero-shot voice cloning is being worked on right now. It does not sound good yet and I've already put multiple months into this. But hopefully with the next version, the model can be used to speak in an unseen voice much better even without finetuning and everything will be a bit simpler.

A small update on this: Zero-shot voice cloning is being worked on right now. It does not sound good yet and I've already put multiple months into this. But hopefully with the next version, the model can be used to speak in an unseen voice much better even without finetuning and everything will be a bit simpler.

Is it fixed in new release?

In the new release, voice cloning is definitely much better than it was, but there is still plenty of room to improve. I'll make an English-Only checkpoint in the next few weeks that's going to be focussed on speaker adaptation.

In the new release, voice cloning is definitely much better than it was, but there is still plenty of room to improve. I'll make an English-Only checkpoint in the next few weeks that's going to be focussed on speaker adaptation.

Can you also share training info afterwards, so would like to train for other languages (voice cloning)


Missing methods on 1.1.0 release.

I was looking through the 1.1.0 source code for what methods are available and I found the uploadFile method. But when trying to use it from the jar 1.1.0, the method was not there. Same with a few other methods. Is it just me, or was there some mistake in the release process?

@psamsotha Thanks for letting us know! We'll take a closer look. In the meantime, please feel free to use the source if that assists you with development.


invalid MIME type 'multipart/form-data' in 'consumes

invalid MIME type 'multipart/form-data' in 'consumes
getting that as part of verify although this is a valid MIME

and another small question : Can I set "SemanticErrors" not to fail on errors, but DefenitionSemanticErrors to fail?
Thanks a lot

Thanks,
will there be a new version release for that soon?

@ellavish89 hopefully yes, though it depends when the pull request will be merged.
In the meantime, can you give my branch a try and verify that the issue described here is fixed?

and another small question : Can I set "SemanticErrors" not to fail on errors, but DefenitionSemanticErrors to fail?
Thanks a lot

@ellavish89 about your question, currently this kind of granularity in behavior is not supported.
Personally I think that the spec, as a whole, is either valid or not valid: in the latter case, the only available options should be to fail or just raise warning messages.

Thanks for the answer. I will check now and update

I am still getting the failures below with the SNAPSHOT from branch you mentioned above:
[ERROR] SemanticError  'consumes' cannot be empty
[ERROR] SemanticError  'consumes' must be equal to 'consumes: []'
[ERROR] SemanticError  'produces' cannot be empty
[ERROR] SemanticError invalid MIME type 'multipart/form-data' in 'consumes'

[ERROR] Failed to execute goal com.github.sylvainlaurent.maven:swagger-validator-maven-plugin:1.2.7-SNAPSHOT:validate (validate) on project test: Some files are not valid, see previous logs. Encountered 139 errors. -> [Help 1]


fix: typo in description

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix: Fixed an issue where Run with PowerShell doesn't show a window

Resolved / Related Issues
To prevent extra work, all changes to the Files codebase must link to an approved issue marked as Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.

Closes #15826

Steps used to test these changes
Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.

Open Files
Select a Powershell Script (.ps1)
Cilck the "Run with PowerShell" button on the toolbar.
Before: The script executes in the background, without displaying a window (the script will stuck if it need to interact with the user).
After:  The script executes in a new window.

@yaira2 That's a good idea. Should I place it in Files.App.Data.Enums?

Yes


CIP-0006 schema is not strictly compliant

Using ajv to validate MetadataExtended.json throws following warnings suggesting the schema is not strictly compliant:
strict mode: missing type "string" for keyword "maxLength" at "https://raw.githubusercontent.com/cardano-foundation/CIPs/master/CIP-0006/schema.json#" (strictTypes)
strict mode: missing type "string" for keyword "maxLength" at "https://raw.githubusercontent.com/cardano-foundation/CIPs/master/CIP-0006/schema.json#/properties/serial" (strictTypes)
strict mode: missing type "string" for keyword "minLength" at "https://raw.githubusercontent.com/cardano-foundation/CIPs/master/CIP-0006/schema.json#/properties/serial" (strictTypes)
strict mode: missing type "object" for keyword "required" at "https://raw.githubusercontent.com/cardano-foundation/CIPs/master/CIP-0006/schema.json#/properties/serial" (strictTypes)

CC: @gufmar, @papacarp


Using the recwatch package

Being able to watch directories recursively was a very useful feature, and I used your code in Algernon. Thanks for writing it!
Now, this pull request is only a suggestion. Accept, reject or ask me to make modifications at will.
I think splitting out the recursive watch functionality into a separate package is a good idea.
If you should happen to want to host recwatch or a similar package on github somewhere, I would be happy to use that one instead of the fork of your code.
Best regards,
Alexander F Rødseth

Thanks Alexander. I think I'll just keep with the current implementation until fsnotify adds support for recursive watches. https://github.com/go-fsnotify/fsnotify/issues/18

OK, understandable.


[BUG] - Input component unresponsive to updates to value via ref

NextUI Version
2.3.6
Describe the bug
When modifying an input's value via its ref (i.e. how react-hook-form passes updates to components when the imperative setValue function is used), the input doesn't respond to the change.
I guess that RHF does something like ref.current.value = "new value", so I have also provided an example in the StackBlitz project.
Without digging into the internals of the NextUI component, I'm guessing it creates a controlled input, making it incompatible with an uncontrolled form library.
Your Example Website or App
https://stackblitz.com/edit/vitejs-vite-vnrckd?file=src%2FApp.tsx
Steps to Reproduce the Bug or Issue

Go to https://stackblitz.com/edit/vitejs-vite-vnrckd?file=src%2FApp.tsx
Observe the value in input and the printed value above and below the button.
Click the button.
Observe the printed value updates but the input value remains unchanged.
(Change the input value to observe the original value restored)the

Expected behavior
I expect the value of the input to change to track the form state.
Screenshots or Videos
No response
Operating System Version
Linux
Browser
Chrome

I am interested in this issue.
If it's okay, could you assign to me?

@wingkwong
I am interested in this issue.
If it's okay, could you assign to me?

@jijiseong assigned

Reopening this issue. The linked PR has been rollbacked.

Any update? I'm facing the same issue.

It's funny that your repro also shows this bug with disappearing value: https://github.com/nextui-org/nextui/issues/3678
https://github.com/user-attachments/assets/23d57879-bb35-41cd-b7f1-1f9ac5bcc1bc


chore: Replaced lerna by workspaces

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
497

Relevant Lines:
497

💛  - Coveralls


ChoiceWidget titles not pulling through

Hi there
The ChoiceWidget does not appear to make use of the titles passed in through the Liform schema. The problem is that it is looking within the overall schema array and not within the liform element of the schema array.
This is what the Liform generated array looks like:
{
   "title":"updaterbundle_announcement",
   "type":"object",
   "properties":{
      "announcement":{
         "type":"string",
         "title":"announcement",
         "attr":{
            "placeholder":"Enter line update ..."
         },
         "default":"Enter line update ...",
         "required":[
            "announcement"
         ],
         "widget":"textarea",
         "propertyOrder":1
      },
      "announcementType":{
         "enum":[
            "",
            "1",
            "2",
            "3"
         ],
         "liform":{
            "enum_titles":[
               "-- Choose --",
               "One",
               "Two",
               "Three"
            ]
         },
         "type":"string",
         "title":"Category",
         "required":[
            "announcementType"
         ],
         "propertyOrder":2
      },
      "lineId":{
         "type":"string",
         "title":"lineId",
         "attr":{
            "maxlength":255
         },
         "maxLength":255,
         "propertyOrder":3
      }
   },
   "required":[
      "announcement",
      "announcementType"
   ]
}

As you can see from the above, the "enum_titles" element is a child of "liform" and not the overall field schema ("announcementType" in this example).
I believe that by changing this line within the ChoiceWidget
var optionNames = field.schema.enum_titles || options;

to
var optionNames = field.schema.liform.enum_titles || options;

will fix the issue.
Thanks!

Thanks for the report!
Hehe, I did again the fix in the PHP repo :). The reason is that at the beginning I started to put all the non standard json schema properties in a liform subarray, but then I decided not to do so anymore, because... for starters liform itself was non standard. Also, json schema validators validate correctly schemas with additional properties, and the standard itself says that anything a json schema parser cannot understand should be ignored.
But it looks like I forgot about this one. So thank you for finding it :) I will make a new release of liform in minutes, but it is fixed in master.

There we go, v0.9.0 https://github.com/Limenius/Liform/releases/tag/v0.9.0

Apologies for delay in responding!
Fix works perfectly. Really appreciate the great support.


backport some code

I removed the loggers from the main PR, but there’s no reason to remove them here.


Global: OpenPype fails to launch if environment has value with { } tokens

Running version
3.15.2-nightly.6
but the issue has been present for a long time
Describe the bug
Launching OpenPype with an environment variable in your local environment containing a value with { } tokens will fail to launch OpenPype.
The error (from an older version):
~ ./openpype_gui
>>> run disk mapping command ...
>>> Logging to server is turned ON
*** Cannot get OpenPype path from database.
>>> Finding latest version with [ 3.14.6 ]
  - booting version: 3.14.6
>>> Injecting OpenPype version to running environment  ...
>>> Validating version [ /mnt/apps/software/openpype/openpype-3.14.6 ]
--- version is valid
>>> loading environments ...
  - for Avalon ...
  - global OpenPype ...
Traceback (most recent call last):
  File "/mnt/apps/software/openpype/openpype-3.14.6/dependencies/acre/lib.py", line 48, in partial_format
    f = formatter.vformat(s, (), mapping)
  File "/root/.pyenv/versions/3.7.12/lib/python3.7/string.py", line 190, in vformat
  File "/root/.pyenv/versions/3.7.12/lib/python3.7/string.py", line 200, in _vformat
ValueError: unexpected '{' in field name

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/openpype/.venv/lib/python3.7/site-packages/cx_Freeze/initscripts/__startup__.py", line 113, in run
  File "/opt/openpype/.venv/lib/python3.7/site-packages/cx_Freeze/initscripts/Console.py", line 15, in run
  File "start.py", line 1214, in <module>
  File "start.py", line 1122, in boot
  File "start.py", line 290, in set_openpype_global_environments
  File "/mnt/apps/software/openpype/openpype-3.14.6/dependencies/acre/core.py", line 76, in compute
    env[key] = lib.partial_format(env[key], data=data)
  File "/mnt/apps/software/openpype/openpype-3.14.6/dependencies/acre/lib.py", line 55, in partial_format
    f = re.sub(m, m.format(**data), f)
ValueError: Sign not allowed in string format specifier

Linux
This is something which can quite trivially happen on Linux due to storing BASH functions in the environment being explained in this OP discord thread.
E.g. a function could be exported like:
function restartworker { /opt/Thinkbox/Deadline10/bin/deadlinecommand RemoteControl `hostname -i` LaunchSlave `hostname -s`;}
export -f restartworker

Resulting in environment variable value of:
BASH_FUNC_restartworker%%=() {  /opt/Thinkbox/Deadline10/bin/deadlinecommand RemoteControl `hostname -i` LaunchSlave `hostname -s`;}

A workaround for this Linux issue could be clearing the environment of bash functions as you launch OpenPype:
for func in `declare -x -F | sed 's/declare -fx//'`;do unset -f $func;done && ./openpype_gui

To Reproduce
Steps to reproduce the behavior:

Set an environment variable like myvar = hello {world} or export a bash function in Linux like above.
Try and launch OpenPype and DCCs.

Expected behavior
It should be able to launch fine.
The local environment value should be ignored and not re-formatted. We should expect the value is written in a way that it was intended to be and if possible be left untouched.
Additional context

1st Report on Discord Thread by @Minkiu

Also mentioned here

2nd Report on Discord Thread

[cuID:85zrrd366]

Based on this SO answer

This feature is bash-specific, so a test for an exported shell function needs to do what Bash does. Experimentation and source code show that Bash recognizes an environment variable as a shell function at startup by the presence of a () { prefix in its value — if the prefix is missing, or even slightly altered, the variable is treated as an ordinary data variable.

Maybe we can clean up the env dict when starting the compute function in acre, something like:
env = env.copy()
env = {
    key: value,
    for key, value in env.items()
     if not value.startswith('() {')
}

This bug was hit again recently, reported here on discord.

This has now been merged in the ynput/acre fork, so unless you want to keep it for acre (upstream) we can close this issue.

Would anyone be able to reproduce this still in Linux with AYON? @iLLiCiTiT do you know?

Would anyone be able to reproduce this still in Linux with AYON? @iLLiCiTiT do you know?

I did see that env on one of our linux test machines, and I don't think it caused errors. Should be easy to reproduce on windows, just set some env variable to () {  /opt/Thinkbox/Deadline10/bin/deadlinecommand RemoteControl hostname -iLaunchSlavehostname -s;}

Would anyone be able to reproduce this still in Linux with AYON? @iLLiCiTiT do you know?

I did see similar env on one of our linux test machines, and I don't think it caused errors. Should be easy to reproduce on windows, just set some env variable to () { /opt/Thinkbox/Deadline10/bin/deadlinecommand RemoteControl `hostname -i` LaunchSlave `hostname -s`;}

With this env var set I ran launcher (which you can see being present here in console as well)...
It launches e.g. Maya and Fusion just fine.
Seems to work. Closing.


PloidyConflict & Comprehensive Ploidy Investigation

One major curiosity about the new DRAGEN callset is the fact that it imputes the biological sex of samples and makes their sex chromosome calls haploid as appropriate, or marks them with a 'PloidyConflict' flag as appropriate. However, we 1) do not see this 'PloidyConflict' flag in the delivered callset and 2) do not know quite as much as we'd like about how these imputations are performed 3) aren't returned the (apparently existing) output metrics for these. This ticket exists for the investigation of all three of these, as it relates to sex and ploidy and imputation in the newly delivered callset.

Some further discussion has found that , once we CONFIRM that where we should be seeing 'PloidyConflict' flags in the FT field are there in the DRAGEN gVCFs , can can load the data as-is. We don't have to set XX-individual diploid variants in ChrY as NA or any fixes like that. Still we want to confirm WHY we are seeing these.

This is on hold... this was not delivered with the gVCFs or single-sample VCFs , and we can't find it anywhere in the DRAGEN pipeline.


getProperty and setProperty do not work properly for classes in playstate

Describe your bug here.
getProperty and setPropertydo not work for classes within playstate. While this works for non-classes, it may not work for classes. Here's an example:

According to the wiki, cameraX is an alias to getProperty(camFollowPos.x).
Note: in debug print, the latest messages are printed on the top.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No modifications. Done in psych engine 0.5.1

it does work, but onCreate happens before the camera is properly loaded
onCreate

onSongStart

getProperty and setPropertydo not work for classes within playstate. While this works for non-classes, it may not work for classes.

You're just not inputing the variable names as a string, that's all.

Oh right
except for scoreTxt.text, it doesn't seem to work at all

scoreTxt is a bit of a bitch because it's under onUpdate, so you pretty much have to set it back every time it updates through the whole song

Anyways this is not a bug so you should close this

DISPUTE
Whether onCreate and onSongStart was called first has nothing to do with this issue. I've updated this issue for increased clarification.

What?? It's literally the reason you're not getting the value you want, because there is no value
Getters and setters work as they're supposed to, this is an error from your part

oop, found the problem. has to be done on onUpdatePost. my bad.


Update readme.md with website link

Related Issue
Closes  #917

Description

Screenshots

Checklist

[ ] My code adheres to the established style guidelines of the project.
[ ] I have included comments in areas that may be difficult to understand.
[ ] My changes have not introduced any new warnings.
[ ] I have conducted a self-review of my code.

I did it.


context: propagate RPC tags as log tags

This lets the service print KBFS log tags that are passed via RPC.
Issue: KBFS-2035

r? @maxtaco @mmaxim @jzila

If someone calls WithLogTag(WithLogTag(ctx, "LU"), "ID2"), does this wind up making duplicated copies of all of the rpc tags?

No, it shouldn't change the existing behavior at all.  It only adds RPC tags to the context, if they're present.  And once it adds them, it deletes them from the context, so they won't get added a second time.  It should have no effects on "LU" or "ID2", which are not RPC tags.

Ok :+1:
On Mon, Mar 20, 2017 at 4:03 PM Jeremy Stribling<EMAIL_ADDRESS>wrote:

If someone calls WithLogTag(WithLogTag(ctx, "LU"), "ID2"), does this wind
up making duplicated copies of all of the rpc tags?
No, it shouldn't change the existing behavior at all. It only adds RPC
tags to the context, if they're present. And once it adds them, it deletes
them from the context, so they won't get added a second time. It should
have no effects on "LU" or "ID2", which are not RPC tags.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/pull/6308#issuecomment-287925308, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA05_001K_SlMo6P--TsXduMB8jzJFS1ks5rnwWxgaJpZM4MjHBz
.


Unified account - Bug related to the Buy token feature

Describe the task
Bug related to the Buy token feature

[x] Button Buy token is disable on the Token Group screen

Screenshot:

Version test: https://drive.google.com/file/d/1A8Qd0AuAukcLomXHHGwnWsRksv7akJsF/view?usp=drive_link
Please check this issue:

[ ] Invalid API key

Version test: https://drive.google.com/file/d/1nuWeryGaUd1jg8ZQaiz_IvndgZsLpV_f/view?usp=sharing
Validation passed for cases:

Show token list to buy for Unified account
Show token list to buy for Solo account
Show token list to buy for External account
Check information when buy with Transak, Banxa

Follow-up: Check information when buy with Coinbase


RowAction without Text

Feature Request: Just showing an image without underlying text.

Hmm, it could be done.
Do you want to add it? Feel free to submit a PR.

My problem is that I don't know how to generate an empty/invisible string with a certain width (the image width). Think thats needed to generate the UITableRowAction. Any ideas?

Hmm, I see what you mean. Have you tried using the "fitted width" methods that @klauslanza contributed? I would think you could just put nil or a blank string for the text, but I'm not sure about the vertical alignment of the image as I don't think the current version of the pod/library handles that case.
Perhaps the methods could be modified to not add a vertical offset/padding for the image if a nil string is passed.

Sorted out by #14.


Failed to compile flutter engine on Windows when Python path contains space

Steps to reproduce
I make it short.
I followed the instructions to build flutter engine without success. I got always errors like

...
ninja: fatal: CreateProcess: %1 is not a valid Win32 application.

and alike.
Then I changed all paths inside

.\build\toolchain\win\BUILD.gn

For example

tool("copy") {
command = "$python_path $tool_wrapper_path recursive-mirror {{source}} {{output}}"
description = "COPY {{source}} {{output}}"
}

into

tool("copy") {
command = ""$python_path" $tool_wrapper_path recursive-mirror {{source}} {{output}}"
description = "COPY {{source}} {{output}}"
}

That because my user name on windows 10 is Name[space]Surname.
Changed that, I could continue the compilation but it didn't success.
Again I got erros like

[0/7379] ACTION //flutter/sky/packages/sky_engine:copy_dart_ui_web(//build/toolchain/win:clang_x64)
...
ninja: fatal: CreateProcess: %1 is not a valid Win32 application.

Unfortunately I couldn't find where those files/commands for ACTION are generated.
toolchain.ninja and build.ninja are recreated every time.
So my questions:

How could I fix the errors?
Where can I find  help on building flutter engine on windows?

I tried on discord but got no answers.
Thanks
Expected results
Pls see above
Actual results
Pls see above
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Hi @CodeVisio, please share the information below:

Are you compiling for Windows?
Which step did this error occur? (at ninja -C step?)
Did you make any modifications to the engine that you built or use the default engine code on master channel? Please share the output of flutter doctor -v as well.
Which engine hash are you building at?
Please share the entire error stack trace if any. Thanks!

I don't think I can run "flutter..", I don't have that command.

Does this mean you don't have Flutter SDK on your machine?

The latest on head.

Can you share the hash?

Yes, ninja -C .\out\host_debug_unopt

Please try re-running the command with verbose to see if it prints any logs that can give hints: ninja -v -C .\out\host_debug_unopt

Thanks for sharing your thoughts. I'm not sure about that as well. Labeling this for other's input.

@CodeVisio thanks for investigating this. I've landed a couple patches just now; let me know if that resolves your issue.

@cbracken
Thanks for your time.
I think the main problem is still there.

H:\src\code\web\engine\src>ninja -C .\out\host_debug_unopt

ninja: Entering directory `.\out\host_debug_unopt'
[0/7145] ACTION //flutter/build/dart:copy_dart_sdk(//build/toolchain/win:clang_x64)
CreateProcess failed. Command attempted:
"C:/Users/Name Surname/AppData/Local/.vpython-root/store/python_venv-6vj8bjfokgtv35i1s7oatiq278/contents/Scripts/python3.exe ../../third_party/dart/tools/copy_tree.py --from H:/src/code/web/engine/src/flutter/prebuilts/windows-x64/dart-sdk --to H:/src/code/web/engine/src/out/host_debug_unopt/dart-sdk"
ninja: fatal: CreateProcess: %1 is not a valid Win32 application.

I'm not familiar with all those tools but I think the problem is not caused at compiling/building time but when you launch:

python .\flutter\tools\gn --unoptimized

It is at that time that the toolchain.ninja file is created and content is wrong and as consequence the above ninja command fails.
I attached my toolchain.ninja file (renamed as txt file) for you reference.
Le me know if you need more.
Thanks

@cbracken.

@CodeVisio thanks for investigating this. I've landed a couple patches just now; let me know if syncing to tip of tree and trying again resolves your issue.

Thanks for your time.
I tried to build it again based on the last repo without success.
H:\src\code\web\engine\src>ninja -C .\out\host_debug_unopt
ninja: Entering directory `.\out\host_debug_unopt'
[0/7145] ACTION //flutter/build/dart:copy_dart_sdk(//build/toolchain/win:clang_x64)
CreateProcess failed. Command attempted:
"C:/Users/Name Surname/AppData/Local/.vpython-root/store/python_venv-6vj8bjfokgtv35i1s7oatiq278/contents/Scripts/python3.exe ../../third_party/dart/tools/copy_tree.py --from H:/src/code/web/engine/src/flutter/prebuilts/windows-x64/dart-sdk --to H:/src/code/web/engine/src/out/host_debug_unopt/dart-sdk"
ninja: fatal: CreateProcess: %1 is not a valid Win32 application.
 #```

The problem is inside `toolchain.ninja` and is generated when you run
`python .\flutter\tools\gn --unoptimized`

I attached my toolchain.ninja file.

Let me know if you need more.

Thanks

@CodeVisio did you re-run flutter\tools\gn --unopt [any other params you use] to regenerate your ninja files before attempting a rebuild?

Looking at toolchain.ninja most of the python invocations look correct and are quoted, but a few are definitely wrong/unquoted -- the ones with the dollar signs in them.
I'll do some poking around to see if I can spot other invocations of python than the ones I fixed.

@cbracken

@CodeVisio did you re-run flutter\tools\gn --unopt [any other params you use] to regenerate your ninja files before >attempting a rebuild?

No. I followed the instructions to build it as they are described in the documentation, but I never invoked gn directly.

Looking at toolchain.ninja most of the python invocations look correct and are quoted, but a few are definitely wrong/unquoted -- the ones with the dollar signs in them.
I'll do some poking around to see if I can spot other invocations of python than the ones I fixed.
The copy, rc, link etc. rules all look good -- these are the ones I fixed.
However, a bunch of them such as these, do not:

__flutter_build_archives_archive_win_gen_snapshot___build_toolchain_win_clang_x64__rule
__flutter_build_archives_artifacts___build_toolchain_win_clang_x64__rule
__flutter_build_archives_dart_sdk_archive___build_toolchain_win_clang_x64__rule
__flutter_build_archives_flutter_patched_sdk___build_toolchain_win_clang_x64__rule

Doing a little bit of quick digging.

Yes. As I wrote here in the past, toolchain.ninja is partially written with the content of engine\src\build\toolchain\win\BUILD.gn,
and that was easy to fix (surrounding the variable python_path with double quotes), however the other big part of toolchain.ninja is written taking the content somewhere else and I couldn't find where, since I'm not familiar with those tools.
As far as I've understood, gn has its way to find and invokes automatically the python executable, the path of it is wrong managed.
the python .\flutter\tools\gn --unoptimized command line is translated in my case to:

'H:\src\code\web\engine\src/flutter/third_party/gn/gn.exe',
'gen',
'--check',
'--export-compile-commands',
'--ide=vs',
'--ninja-executable=ninja',
'--export-compile-commands=default',
'out\host_debug_unopt',
'--args=
is_debug=true
use_goma=false
goma_dir="None"
full_dart_sdk=false
skia_enable_flutter_defines=true
skia_use_dng_sdk=false
skia_use_sfntly=false
skia_enable_pdf=false
skia_use_x11=false
skia_use_wuffs=true
skia_use_expat=true
skia_use_fontconfig=false
skia_use_icu=true
is_official_build=true
android_full_debug=false
is_clang=true
skia_gl_standard=""
enable_coverage=false
host_os="win"
host_cpu="x64"
target_os="win"
target_cpu="x64"
dart_target_arch="x64"
flutter_use_fontconfig=false
dart_component_kind="static_library"
embedder_for_target=false
dart_lib_export_symbols=false
flutter_runtime_mode="debug"
dart_version_git_info=true
dart_runtime_mode="develop"
enable_desktop_embeddings=true
enable_backtrace=false
bssl_use_clang_integrated_as=true
dart_use_fallback_root_certificates=true
skia_use_gl=true skia_use_vulkan=true
skia_use_vma=false
shell_enable_vulkan=true
disable_vma_stl_shared_mutex=true
disable_hidden_visibility=false
flutter_prebuilt_dart_sdk=true
dart_sdk_output="built-dart-sdk"
stripped_symbols=true
angle_build_all=false
angle_has_astc_encoder=false
angle_force_context_check_every_call=true
angle_use_x11=false
angle_use_wayland=false
angle_enable_vulkan_validation_layers=false
angle_vulkan_headers_dir="//third_party/vulkan-deps/vulkan-headers/src"
angle_vulkan_loader_dir="//third_party/vulkan-deps/vulkan-loader/src"
angle_vulkan_tools_dir="//third_party/vulkan-deps/vulkan-tools/src"
',
'--tracelog=out\host_debug_unopt/gn_trace.json'

I hope it helps.
Thanks

Re-opening as a complete fix has not landed. Github auto-closing it on @Jasguerrero's personal rebase seems like a GitHub bug.

This bug has not been comletely fixed as I exlained above.

It seems definitely a github's bug, I rebased my own repo from flutter/buildroot, which seems totally unrelated to this bug.


Question: Is there any way to prevent the original menu from popping up on hover?

When I move my mouse to the top of the screen, the original mac menu still pops up, is it possible to have it disappear completely?

To my knowledge there isn't, this is a mac os feature that can't be disabled.


Add a new controller endpoint for segment deletion with a time window

Implement the proposal in the issue #10740
Move the code of segment selection with time window, which is inside getSegments API, to a standalone function. Then, pass the segment list into the existing deleteSegmentsInternal function.

Codecov Report

Merging #10758 (30e5ca2) into master (51bf75e) will decrease coverage by 56.60%.
The diff coverage is 48.38%.

@@              Coverage Diff              @@
##             master   #10758       +/-   ##
=============================================
- Coverage     70.22%   13.62%   -56.60%
+ Complexity     6525      439     -6086
=============================================
  Files          2164     2116       -48
  Lines        116369   114188     -2181
  Branches      17599    17360      -239
=============================================
- Hits          81719    15558    -66161
- Misses        28952    97355    +68403
+ Partials       5698     1275     -4423

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
13.62% <48.38%> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ler/api/resources/PinotSegmentRestletResource.java
12.72% <48.38%> (-25.39%)
:arrow_down:

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

cc @snleee

Thanks @wirybeaver. Let us know when this is ready for review.

@jtao15 @snleee Could you review the PR when you are available?

Thanks for the contribution! Can you help also update the pinot documentation for this new API?

@Jackie-Jiang Do you have an suggestion on which section shall I update?
This one https://docs.pinot.apache.org/users/api/controller-api-reference ?

@wirybeaver Sure, let's add it here. Seems we have a lot of APIs not properly documented, but let's not make them more :-P
Here is the link of how to contribute the doc


Long time no update

Long time no update

Apologies I have been pretty busy this summer. I do have changes to make an update, please wait one more week and I will try to get those out in a release :)

Sorry for the wait, v2.3.0 is now published. Enjoy :)


Remove unreachable! from filter push down rule

Which issue does this PR close?

Part of https://github.com/apache/arrow-datafusion/issues/3314
Rationale for this change

Optimizer can recover from rules that fail with Err but not if they panic
What changes are included in this PR?

Use Err instead of unreachable!
Are there any user-facing changes?

No

cc @avantgardnerio


iOS 10.2

With iOS 10.2 I see an issue with a delay on loading the table cells and also sometimes displaying the text and image I have in the cell don't load correctly either (usually occurs on one cell and the other cells display fine). Works fine on previous iOS versions. I've tried storageNeedsReloading() and reloadItem(item) to force a reload but to no avail.

Hey!
That's interesting, we do not see this behavior currently in our apps. Do you have any specific details on when and why this happens? Also, which version are you using, Swift 2.3 or 3.0?
It would be great to have working sample of this bug, or maybe an Instruments trace.

Thanks for the quick response. I believe I've misunderstood the problem. I had read through previous issues and realised that the problem was caused by not running manager.memoryStorage.addItem(someItem) on the main thread. I've done that now and it has fixed the issue.


NFS Example Dockerfile is invalid

The example here:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/examples/nfs/exporter/Dockerfile
Doesn't look like it'll result in a valid NFS container.
Entrypoint needs to be copied into the container.

Opened pull request: https://github.com/GoogleCloudPlatform/kubernetes/pull/7417


Extend (I had a problem using extend and always give me Failed Prop type error after i re-running the app even though i write the write format

I had a problem using extend and always give me this error after i run the app even though i write the write format

Could you fix it ?


remove add default set label controller

Signed-off-by: ldpliu<EMAIL_ADDRESS>

we should also remove the rbac rules, right?

Removed https://github.com/open-cluster-management-io/registration/pull/250/files#diff-68752c39408458ef5dc35f2232378e0f2848f1ff035af41ab6a5fa57eb5831acL64
And will submit another pr to remove this in registration-oprator after this pr merged.

/approve
/lgtm