Changing the colors of the highlight types

Feature Requested
The main highlight and 2 alternative color options are really nice additions. It would be nice to adjust their colors from the style settings so that they could be used in more contexts.
Relevant Screenshots

There is no UI change, only settings options to update the highlight colors

Checklist

[x] The feature would be useful to more users than just me.
[x] I have checked that the feature is not already available via the Style Settings plugin.

done in the latest release

As a tiny nitpick, would it be possible to add a color scale beneath the thing that you move around? That way you don't have to "blindly" move it (not that it's a huge spectrum, just as a thought) :)

uff, yeah it would be possible but it's would be quite tricky to implement, so I think I'll invest the time rather somewhere else :P

Absolutely fine! That was just a thought, and not important. Thanks anyway :)

Nevermind, I was procrastinating and figured it out actually 😆

Oh wow, that looks awesome, thank you!! This isn't showing up for me yet, though?

try again? maybe it takes a few minutes for GitHub to update the file? 🤔

Yup, there it is, thanks!! Tangentially related, I have a question on a snippet I had that no longer seems to be working.
.theme-dark
{ --selection-color: #90482a;

}

.theme-light
{ --selection-color: #F07847;

}

.theme-light .cm-strong,
.theme-light .markdown-preview-view strong {
	color: var(--bold-in-lightmode);
}

.vertical-tab-nav-item:last-child {
	color: var(--bold-in-lightmode);
}

It's supposed to cause the selection color to be different, i.e. brighter to make it easier to see (orange-ish). I'm not sure exactly when this "broke" but it no longer does that. First I thought it had to do with your theme, but it also no longer works in the default theme either. Any ideas why that might be? Not that important either, just something I noticed.

ah yes, that's cause I have changed a lot how colors work in preparation for a bigger color customization update. You can use this snippet to change the colors anyway:
.theme-dark {
	--text-selection: #90482a;
}

.theme-light {
	--text-selection: #F07847;
}

Alternatively, you can use this CSS snippet to "unlock" some accent color style settings I am still working on. Changing the Primary Accent to Orange or blue already works.
/* "unlock" under-development-color options in the style settings
(note that some unlocked settings do not do anything yet, since it
is still work-in-progress.) */
body .style-settings-container .setting-item[data-id*="-accent-"] {
	display: flex;
}

oh never mind the snippet. I had the better idea to simply make it a style setting toggle under Misc :P

Thanks :)


Add guided_pool_executor

Extends #3415 to supply a guided_pool_executor that can pass hints from user supplied task functions through to the scheduler.
Initial work has focused on NUMA aware scheduling of tasks.

Removed the WIP moniker from the PR title because it is now passing all tests locally and it might be feasible to get this into the next release if tests on other machines are ok.

if #3192 is reviewed and merged, then I will update this branch to use it. I'm hoping to get this branch into 1.2 release too.

@biddisco looks like this is passing all tests now, nice! Are you planning on adding anything still? Maybe you could do a bit of squashing a few of the commits?

I will not add anything new to the branch, but tonight I can spend some time cleaning it up. There are a few commits that I feel need to be squashed.

I've hopefully fixed the c++11 issues and I squashed a couple of commits, but not many. As far as I'm concerned, if this passes tests, then it is good to go.

If this isn't merged soon, I'm sending hit men round for the kittens.

Puuuh! Thank god, the kittens are safe!


Model instantiation is very slow

While trying out the example provided in the Readme, I noticed that this line executes very slowly:
model = RNN(layers=layers, cost='BinaryCrossEntropy')

Profiling shows that this line takes around 20 seconds to execute on both CPU as well as GPU. I am observing similar performance for the load function in utils.py as well. Am I missing something here? If not, can this be sped up somehow? ( Such performance poses a serious obstacle in loading trained models in real time applications. )

@adithyarao, I have a hunch that the slow execution when building and loading models has more to do with Theano than Passage itself.  Theano takes time to compile graphs of computation, both on CPU and GPU.  For real world applications, the best bet is to keep a server continuously running with the Theano graph already compiled.


Add OAuthUrl constants and ensure forwarded activities contain a recipient value

Fixes #1816
Fixes #1997
Description
If the forwarded activity to the skill is missing a recipient, certain methods can throw ReferenceErrors (or in dotnet, NullReferenceExceptions) as Activtiy.recipient is expected (#1997 This PR fixes that and adds a test for this scenario.
The OAuthUrl constants in dotnet were missing from the AuthenticationConstants and GovernmentAuthenticationConstants in botframework-connector. These values have been added in the PR.
Specific Changes

BotframeworkHttpClient.postActivity adds an empty recipient ({}) to all forwarded activities if the field is missing
AuthenticationConstants and GovernmentAuthenticationConstants now have OAuthUrl constants

Testing
Added tests

Pull Request Test Coverage Report for Build 122045

4 of 4 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 67.346%

Totals

Change from base Build 121972:
0.01%

Covered Lines:
10348

Relevant Lines:
14541

💛  - Coveralls


Add support for dbt-core>=1.4.2

When running data-diff --dbt I get a bunch of these errors:
[23:42:37] ERROR - 90 validation errors for ManifestV8
nodes -> seed.demo.org_created -> resource_type
  value is not a valid enumeration member; permitted: 'analysis' (type=type_error.enum; enum_values=[<ResourceType.analysis: 'analysis'>])

The output from dbt --version:
Core:
  - installed: 1.4.4
  - latest:    1.4.4 - Up to date!

Plugins:
  - snowflake: 1.4.1 - Up to date!

If I downgrade dbt with pip install "dbt-core<1.4.2" -U then it works.

Duplicate of https://github.com/datafold/data-diff/issues/433


KNOX-1827 Knox Fails to Rewrite WebFonts

(It is very important that you created an Apache Knox JIRA for this change and that the PR title/commit message includes the Apache Knox JIRA ID!)
Update rewrite rule for ZeppelinUI. Current service.xml is sending all the js and webfont files to the rule ZEPPELINUI/zeppelin/inbound/rootAppScript which is looking for a file pattern starting with app. This removes that restriction allow the webfonts to work again.
(Please fill in changes proposed in this fix)
Setup Zeppelin 0.8.0 and Knox and watch the Knox logs for failure to rewrite errors for *.woff files.
(Please explain how this patch was tested. For instance: running automated unit/integration tests, manual tests. Please write down your test steps as detailed as possible)
Please review Knox Contributing Process before opening a pull request.

Feel free to reject if you'd prefer this being resolved another way. With this change and KNOX-1828 I haven't been able to cause any errors in Zeppelin and everything seems to be working.

@moresandeep would be good to review this.

LGTM


As a user, I'd like to log in to secure openeew-dashboard

The current dev/test deployment does not have https enabled. So we'll need to figure out how to modify our LoadBalancer service to enable navigating to the dashboard with an https:// url.

Created a certificate for the dashboard. I imported the cert and private key into IBM Certificate Mgr and attached the CRN to a Kubernetes Secret. The secret is then referenced by the Ingress yaml for the dashboard.


WG metrics and OKRs

Do we have a space where we could start to collect some suggestion on how to introduce some of our metrics in OKRs?
As often opensource is perceived as a "positive"  externality but without a specific measurable impact on OKRs all the CHAOSS efforts on metrics could really help to increase the impact on opensource software hosted by large companies (or any other kind of company that use an OKRs like approach).
If we could a contact point (checklist, extras section in metric, etc..) between metrics and OKRs I suppose it could have a great impact.
I would also be really nice if we could enforce a subset of this measurable goals directly in the contribution covenant.

I also hope that if we could start to work in this domain we are going to also analyze how we could mitigate Goodhart law/Campbell's law effects when converting CHAOSS metrics in targets.

See also:
https://arxiv.org/abs/1803.04585
https://news.ycombinator.com/item?id=17320640

Do you think it is better to move this in WG-risk?

I like the idea of the metrics models wg as this (bias concerns) could be part of the narrative that we discussion around each specific metric in a model. An example is here: https://docs.google.com/document/d/1qGxLizy5iybFsVbCqXdy1A3I8P622oiiWv7iCgCL1iI/edit?usp=sharing

Just to and a nice short reading:
https://towardsdatascience.com/unintended-consequences-and-goodharts-law-68d60a94705c

A gentle introduction to OKRs vs performance:
https://www.koan.co/blog/okrs-vs-performance-management

Thanks for sharing these resources @bhack. I really think that a best approach for these is going to be inclusion in our metrics models. I'm not sure we can easily/immediately refactor all metrics.

@germonprez I have a related ticket in https://github.com/chaoss/wg-metrics-models/issues/17
So I don't know if you want to nove this there on just to close It here.

I think if the related ticket in the MM WG is tracking the same conversation, it's okay to close this issue.


Add AZAddOwner relationship creation

Add AZAddOwner relationship between users with Hybrid Identity Administrator, Partner Tier1 Support, Partner Tier2 Support, or Directory Synchronization Accounts roles and all Service Principals and Apps within the tenant

Description
Add logic to analysis/azure/post to perform post-processing analysis for AZAddOwner edges in the UserRoleAssignments function. This was completed by adding a new addOwners function that fetches all ServicePrincipals and Apps within the tenant and proceeds to create the edge between users with the following roles:

Hybrid Identity Administrator
Partner Tier1 Support
Partner Tier2 Support
Directory Synchronization Accounts

Motivation and Context
This PR addresses: Issue #686
Fully implements documented behavior for AZAddOwner relationships per documentation.
How Has This Been Tested?
Verifying issue:

Collect Azurehound data for tenant with users assigned with at least one of the above role.s
Upload data to bloodhound server and wait for ingestion.
Run the following cypher command against data:

MATCH p=()-[r:AZAddOwner]->() RETURN p LIMIT 25

verify no relationships exist.

Verifying fix.

Rerun steps 1-3 above.
Verify expected relationships between known users with role and all apps/service principals within the same tenant.

Screenshots (optional):
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] I have met the contributing prerequisites

Assigned myself to this PR
Added the appropriate labels
Associated an issue: https://github.com/SpecterOps/BloodHound/issues/672
Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing

[x] I have ensured that related documentation is up-to-date

Open API docs
Code comments (GoDocs / JSDocs)

[ ] I have followed proper test practices

Added/updated tests to cover my changes
All new and existing tests passed

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

recheck

Hey, @cntC0d3 - thanks for the PR! We're prepping a release right now and won't be able to get a full review before this one, but we'll be sure to get this reviewed here in the next couple of weeks!

Hey @cntC0d3,
Thanks for the PR! It looks good - great job 🙌
We have however started to model the permissions of roles directly out of the AZRole nodes instead of out of the principals with the role assignment. See AZResetPassword as example:

In that way, we reduce the total number of edges created (better performance) and it becomes clear how a given principal has a permission through a role assignment.
Are you up for modifying your AZAddOwner implementation to match that pattern?
The documentation for the AZAddOwner edge has to be updated to match the new pattern. I'll take care of that.

@JonasBK - yeah I can knock that out in the next couple days. 😃

Awesome! 🙌

@JonasBK  - since Microsoft decided to fix many of the issues associated with this edge I haven't had much motivation to continue working this. Closing PR.

Hey @cntC0d3,
Talked with Andy about it. It is only Directory Synchronization Accounts that will loose the ability to add owners to SPs and Apps to our understanding. So your PR is still relevant for the Hybrid Identity Administrator, Partner Tier1 Support, and Partner Tier2 Support roles.
Let me know if you are still interested in fixing up the PR. All cool if not - then we will get it fixed :)


Support non-FloatQuantity and non-ArrayQuantity types

I expected this to work:
from openff.models.models import DefaultModel
from openff.models.types import ArrayQuantity, FloatQuantity
from openff.units import unit

class Atom(DefaultModel):
    mass: FloatQuantity["atomic_mass_constant"]
    charge: FloatQuantity["elementary_charge"]
    some_array: ArrayQuantity["nanometer"]
    baz: int

atom = Atom(
    mass=12.011 * unit.atomic_mass_constant,
    charge=0.0 * unit.elementary_charge,
    some_array=unit.Quantity([4, -1, 0], unit.nanometer),
    baz = 2,
)

Atom.parse_raw(atom.json())

But if fails on the round trip since it can't parse baz properly:
{'baz': 2,
 'charge': '{"val": 0.0, "unit": "elementary_charge"}',
 'mass': '{"val": 12.011, "unit": "atomic_mass_constant"}',
 'some_array': '{"val": [4, -1, 0], "unit": "nanometer"}'}

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
File ~/miniconda3/envs/gufe/lib/python3.10/site-packages/pydantic/main.py:534, in pydantic.main.BaseModel.parse_raw()

File ~/miniconda3/envs/gufe/lib/python3.10/site-packages/pydantic/parse.py:37, in pydantic.parse.load_str_bytes()

File ~/miniconda3/envs/gufe/lib/python3.10/site-packages/openff/models/types.py:135, in json_loader(data)
    132 try:
    133     # Directly look for an encoded FloatQuantity/ArrayQuantity,
    134     # which is itself a dict
--> 135     v = json.loads(val)
    136 except json.JSONDecodeError:
    137     # Handles some cases of the val being a primitive type

File ~/miniconda3/envs/gufe/lib/python3.10/json/__init__.py:339, in loads(s, cls, object_hook, parse_float, parse_int, parse_constant, object_pairs_hook, **kw)
    338 if not isinstance(s, (bytes, bytearray)):
--> 339     raise TypeError(f'the JSON object must be str, bytes or bytearray, '
    340                     f'not {s.__class__.__name__}')
    341 s = s.decode(detect_encoding(s), 'surrogatepass')

TypeError: the JSON object must be str, bytes or bytearray, not int

During handling of the above exception, another exception occurred:

ValidationError                           Traceback (most recent call last)
Input In [4], in <cell line: 20>()
     10     baz: int
     13 atom = Atom(
     14     mass=12.011 * unit.atomic_mass_constant,
     15     charge=0.0 * unit.elementary_charge,
     16     some_array=unit.Quantity([4, -1, 0], unit.nanometer),
     17     baz = 2,
     18 )
---> 20 Atom.parse_raw(atom.json())

File ~/miniconda3/envs/gufe/lib/python3.10/site-packages/pydantic/main.py:543, in pydantic.main.BaseModel.parse_raw()

ValidationError: 1 validation error for Atom
__root__
  the JSON object must be str, bytes or bytearray, not int (type=type_error)

This bit https://github.com/mattwthompson/openff-models/blob/61260ec1d2746a0431671498dfe1540b006354db/openff/models/types.py#L137
doesn't actually seem to handle the case where the type isn't a float or array quantity.

Weird - that's definitely not good. I'll have a look!

Slight clarification: I'm pretty sure the issue is isolated to the JSON parser forgetting that primitives exist.

Closed #4 - thanks Mike!


Column names should be in quotes

It doesn't seem possible to use sqlite keywords as column names because they aren't being quoted as instructed by the sqlite docs.
If I have the following entity:
@Entity
public abstract class DbModel {

	@Key
	@Generated
	int id;

	@Column(nullable = false, unique = true)
	int order;
}

I get this exception
java.sql.SQLException: android.database.sqlite.SQLiteException:
near "order": syntax error (code 1): , while compiling:
create table DbModel (id integer primary key autoincrement, order integer not null unique)

order should be "order" in the create table statement

In order to quote column names and be able to use reserved keywords as columns you just need to do the following:
DatabaseSource source = new DatabaseSource(this, Models.DEFAULT, 1) {
                @Override
                protected void onConfigure(ConfigurationBuilder builder) {
                    super.onConfigure(builder);
                    builder.setQuoteColumnNames(true);
                }
            };

It's not enabled by default since it may not be desirable to have column names that are reserved in certain cases, although for SQLite it may be ok.

Cool, good to know!
Is it possible to throw a compile error or warning when a reserved keyword is used? It took me awhile to go through my logs to figure out the column name was the issue.

It should be emitted as a warning per #102 should see something to the effect of 'Column or index name <name> may need to be escaped

I didn't get that warning, but I have gotten other warnings about foreign keys before so it seems like that specific warning might not be working.
Just curious, what's the reason for not quoting column names by default? Does it cause any issues if they're just always quoted?

The warning wasn't being printed for plain field that is fixed, thanks. As for always quoting need to see if there is any performance impact from doing that in the sqlite parser if not will enable it.

Sounds good, thanks!


BondingCurve account not found

Im am trying to launch a token with several buys, but it does not work, because bundingcurve account is not found
Bonding curve after create and buy: null
9atFnGwHDDFeA7Y1r24iV39mTpafXchx6mtXHtQNNYfe: No Account Found
Buy code:
`async createAndBuyWithAdditionalWallets(
creator: Keypair,
mint: Keypair,
createTokenMetadata: CreateTokenMetadata,
buyAmountSol: bigint,
wallet1: { wallet: Keypair; buyAmountSol: bigint },  // First additional wallet for buying
buyWallets: { wallet: Keypair; buyAmountSol: bigint }[], // Additional wallets for buying in parallel
slippageBasisPoints: bigint = 500n,
priorityFees?: PriorityFee,
commitment: Commitment = DEFAULT_COMMITMENT,
finality: Finality = DEFAULT_FINALITY
): Promise<TransactionResult[]> {
try {
// First Transaction: Create and mint the token, and buy with one additional wallet
console.log("Creating token metadata...");
const tokenMetadata = await this.createTokenMetadata(createTokenMetadata);
console.log("Token metadata created.");
console.log("Getting create and mint instructions...");
const createMintTx = await this.getCreateInstructions(
  creator.publicKey,
  createTokenMetadata.name,
  createTokenMetadata.symbol,
  tokenMetadata.metadataUri,
  mint
);
console.log("Create and mint instructions obtained.");

let transaction1 = new Transaction();

// Increase compute budget
console.log("Setting compute budget...");
const computeBudgetIx1 = ComputeBudgetProgram.setComputeUnitLimit({ units: 500000 });
transaction1.add(computeBudgetIx1);

// Add priority fees if provided
if (priorityFees) {
  console.log("Adding priority fees...");
  const priorityFeeIx1 = ComputeBudgetProgram.setComputeUnitPrice({ microLamports: priorityFees.unitPrice });
  transaction1.add(priorityFeeIx1);
}

transaction1.add(createMintTx);

if (buyAmountSol > 0) {
  console.log("Adding buy instructions for the first wallet...");
  const globalAccount = await this.getGlobalAccount(commitment);
  const buyAmount = globalAccount.getInitialBuyPrice(buyAmountSol);
  const buyAmountWithSlippage = calculateWithSlippageBuy(buyAmountSol, slippageBasisPoints);

  console.log(`First wallet buyAmount: ${buyAmount.toString()}, buyAmountWithSlippage: ${buyAmountWithSlippage.toString()}`);

  const buyTx = await this.getBuyInstructions(
    wallet1.wallet.publicKey,
    mint.publicKey,
    globalAccount.feeRecipient,
    buyAmount,
    buyAmountWithSlippage
  );

  transaction1.add(buyTx);
}

// Fetch recent blockhash
console.log("Fetching recent blockhash for first transaction...");
const { blockhash: blockhash1 } = await this.connection.getRecentBlockhash(commitment);

// Set recent blockhash
transaction1.recentBlockhash = blockhash1;

// Set fee payer
transaction1.feePayer = creator.publicKey;

// Sign transaction
console.log("Signing first transaction...");
transaction1.partialSign(creator, mint, wallet1.wallet);

console.log("Sending first transaction...");
const signature1 = await this.connection.sendTransaction(transaction1, [creator, mint, wallet1.wallet], { skipPreflight: false, preflightCommitment: commitment });

console.log("Confirming first transaction...");
await this.connection.confirmTransaction(signature1, finality);

// Fetch the full transaction response
console.log("Fetching first transaction details...");
const transactionResponse1 = await this.connection.getTransaction(signature1, { commitment: finality });

if (transactionResponse1 === null) {
  throw new Error('Failed to fetch first transaction response.');
}

const transactionResult1: TransactionResult = {
  signature: signature1,
  success: true,
  results: transactionResponse1 as VersionedTransactionResponse,
};

// Second Step: Send multiple buy transactions in parallel
console.log("Sending multiple buy transactions in parallel...");
const buyTransactions = buyWallets.map(async ({ wallet, buyAmountSol }) => {
  if (buyAmountSol > 0) {
    let transaction = new Transaction();

    const globalAccount = await this.getGlobalAccount(commitment);
    const buyAmount = globalAccount.getInitialBuyPrice(buyAmountSol);
    const buyAmountWithSlippage = calculateWithSlippageBuy(buyAmountSol, slippageBasisPoints);

    console.log(`Wallet ${wallet.publicKey.toBase58()} buyAmount: ${buyAmount.toString()}, buyAmountWithSlippage: ${buyAmountWithSlippage.toString()}`);

    const buyTx = await this.getBuyInstructions(
      wallet.publicKey,
      mint.publicKey,
      globalAccount.feeRecipient,
      buyAmount,
      buyAmountWithSlippage
    );

    transaction.add(buyTx);

    // Fetch recent blockhash
    const { blockhash } = await this.connection.getRecentBlockhash(commitment);

    // Set recent blockhash and fee payer
    transaction.recentBlockhash = blockhash;
    transaction.feePayer = wallet.publicKey;

    // Sign transaction
    transaction.partialSign(wallet);

    console.log(`Sending buy transaction for wallet ${wallet.publicKey.toBase58()}...`);
    const signature = await this.connection.sendTransaction(transaction, [wallet], { skipPreflight: false, preflightCommitment: commitment });

    console.log(`Confirming buy transaction for wallet ${wallet.publicKey.toBase58()}...`);
    await this.connection.confirmTransaction(signature, finality);

    // Fetch the transaction response
    const transactionResponse = await this.connection.getTransaction(signature, { commitment: finality });

    if (transactionResponse === null) {
      throw new Error(`Failed to fetch transaction response for wallet ${wallet.publicKey.toBase58()}.`);
    }

    return {
      signature: signature,
      success: true,
      results: transactionResponse as VersionedTransactionResponse,
    } as TransactionResult;
  }
  return undefined;
});

// Wait for all transactions to complete and filter out undefined values
const transactionResults2 = (await Promise.all(buyTransactions)).filter((result): result is TransactionResult => result !== undefined);

// Combine the results
return [transactionResult1, ...transactionResults2];

} catch (error) {
console.error("Error during transaction creation and submission:", error);
return [{
success: false,
error: error,
}];
}
}
`

same question


Tests failing after building

I am running these commands:
git clone --recurse-submodules git@github.com:jmbuhr/tree-sitter-pandoc-markdown.git
cd tree-sitter-pandoc-markdown
npm install
npm run build
npm run test

But it seems that the tests are failing. If I read it correctly, this suggests that tree-sitter-pandoc-markdown parser is missing the implementation of external scanner functions.
❯ npm run test

><EMAIL_ADDRESS>test
> node scripts/test.js

testing tree-sitter-pandoc-markdown
Missing required functions in the external scanner, parsing won't work without these!

  `tree_sitter_pandoc_markdown_external_scanner_create`
  `tree_sitter_pandoc_markdown_external_scanner_destroy`
  `tree_sitter_pandoc_markdown_external_scanner_serialize`
  `tree_sitter_pandoc_markdown_external_scanner_deserialize`
  `tree_sitter_pandoc_markdown_external_scanner_scan`

You can read more about this at https://tree-sitter.github.io/tree-sitter/creating-parsers#external-scanners

testing tree-sitter-pandoc-markdown-inline
Missing required functions in the external scanner, parsing won't work without these!

  `tree_sitter_pandoc_markdown_inline_external_scanner_create`
  `tree_sitter_pandoc_markdown_inline_external_scanner_destroy`
  `tree_sitter_pandoc_markdown_inline_external_scanner_serialize`
  `tree_sitter_pandoc_markdown_inline_external_scanner_deserialize`
  `tree_sitter_pandoc_markdown_inline_external_scanner_scan`

You can read more about this at https://tree-sitter.github.io/tree-sitter/creating-parsers#external-scanners

If I try to run the test a second time, I get:
❯ npm run test

><EMAIL_ADDRESS>test
> node scripts/test.js

testing tree-sitter-pandoc-markdown
Error opening dynamic library "/home/filoche/.cache/tree-sitter/lib/pandoc_markdown.so"

Caused by:
    /home/filoche/.cache/tree-sitter/lib/pandoc_markdown.so: undefined symbol: tree_sitter_pandoc_markdown_external_scanner_create
testing tree-sitter-pandoc-markdown-inline
Error opening dynamic library "/home/filoche/.cache/tree-sitter/lib/pandoc_markdown_inline.so"

Caused by:
    /home/filoche/.cache/tree-sitter/lib/pandoc_markdown_inline.so: undefined symbol: tree_sitter_pandoc_markdown_inline_external_scanner_create

Any ideas on what is going on?

If I look in tree-sitter-markdown-inline/src/scanner.c I can see it is implemented
void *tree_sitter_markdown_inline_external_scanner_create() {
    Scanner *s = (Scanner *)malloc(sizeof(Scanner));
    deserialize(s, NULL, 0);
    return s;
}

oh, looks like I lost some changes during making all the necessary renames. Yes, all _markdown in the function names of both scanner.c have to be replaced with pandoc_markdown.

Closed in 84db6e154efbd97a49f2516f38f4f379195aef98


Token is not generating

Hi,
I am trying to get token for airbnb and it returning null in response. Here is the code I am using
airbnb.login({ username: 'xxxxx', password: 'xxxxx }).then(function(data) {     console.log('data: ', data);   })
It returning null in console.
airbnb.newAccessToken({ username: 'xxxxxx', password: 'xxxxxx' }).then(function(data) {   console.log('data: ', data); })
Above code returning error:-
data:  {   error: {     error_code: 420,     error: 'unknown_error',     error_message: 'Unable to perform action. Please try again through the website or contact support if you need immediate assistance.'   } }
Please help me and let me know where I am doing wrong.

I'm also getting this with a new airbnb account

i've started using this lib https://github.com/drawrowfly/airbnb-private-api . its does not have many endpoint support as this one but it works great at least for the token part

Yes there is another way to get token via v2/authentications a bit different json post
body: {
authenticationParams: {
...(type === 'email'
? {
email: {
email: this.email,
password: this.password,
},
}
: {
phone: {
deliveryMethod: 'TEXT',
isCombinedFlow: false,
isGlobal: true,
otp: code,
phoneNumber: this.phone,
},
}),
},
metadata: {
sxsMode: 'OFF',
},
},


home-assistant: add missing setuptools dependency

Motivation for this change
Fixes currently broken nixos hass test: https://hydra.nixos.org/build/100923199
home-assistant currently doesn't start up without the extra dependency (which used to be implicit).
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Related: #68314 (cc @FRidh)


Upgrade to prometheus 29.2.0

What
Upgrade to prometheus 29.2.0.
We are using the 'attic' setting for now. There is a separate ticket to migrate to the latest firehose.
How to review
I deployed to dev03 in dev05.
Please deploy into a different dev environment and check it deploys okay.
Issue
I saw an issue when deploying that was hard to reproduce. Namely the error:
"Database locked, sleeping then retrying"
In the grafana log. My suspicion is that this problem is related to the monit change to use bpm in this release. There might be a timing issue where we end up with two grafana's running for a short amount of time. I resolved this when I saw it with a "bosh recreate".

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

So.. you think this is a glitch we'll only see the first time we migrate from a non-bpm-using grafana to a bpm-using grafana?

Shouldn't we be pointing to https://github.com/bosh-prometheus/prometheus-boshrelease/commit/173e7df0a3a3730b4ef4ff2f317dffd6165344c4 ? This appears to point to its parent.


feat: internal table support (contributors)

close https://github.com/risinglightdb/risinglight/issues/99
> \dt
+---+----------+---+------------+---+--------------+
| 0 | postgres | 1 | pg_catalog | 0 | contributors |
+---+----------+---+------------+---+--------------+
> select * from pg_catalog.contributors
+--------------+
| name         |
+--------------+
| Alex Chi     |
| Alissa Tung  |
| ArkBriar     |
| Chojan Shang |
| Frank Ma     |
| Lu Di        |
| Mingj        |
| MingjiHan    |
| Runji Wang   |
| Tab Zhang    |
| TennyZhuang  |
| Ting Sun     |
| Yingjun Wu   |
| Yuhui Liu    |
| Yujia Qiao   |
| Zehao Wei    |
| kutori       |
| likg227      |
| nanderstabel |
| stonepage    |
| wzl          |
| xxchan       |
| xyz          |
| 庄天翼       |
|              |
+--------------+

> select count(*) from pg_catalog.contributors
+-------+
| count |
+-------+
| 25    |
+-------+

Btw, the generate contributors list contains duplicate names 😅

Now every time we checkout to a new branch or add a new commit, we will trigger a full rebuild...

Yes I also have this concern😓😓😓

I think it's better to store contributor list to a JSON, and update it on every release.

Maybe generated by a pre-release script? (and now we can use this api https://api.github.com/repos/risinglightdb/risinglight/contributors)
Is there something like this? (I don't know much about how to release)

Maybe we can take a look at how https://github.com/datafuselabs/databend update their contributor table?

I actually looked. They use build.rs 😸


forward compatibility with xgboost 0.90

change positional arguments to keyword arguments for the forward compatibility with xgboost 0.90

When I used the xgboostExtension on xgboost 0.90, I found the XgbRanker class was not properly initialized with correct value in each argument. So I updated the arguments to keyword style for better compatibility.

When using XGBoost 0.90 or up I would propose using the XGBRanker object that has been made available as part of the scikit-learn api there instead of relying on this module. So maybe it would be better to make users aware that this is not meant to be used with that version?

@FlorisHoogenboom  I'm going to have a try and make the decision.


Relocating

The relocating as mentioned did not work.
Specifying the relocatingConfig.txt with 'move all by 0 16 0' when converting the second 'floor'.
But i found another way.
First i convert the second floor that goes from 256 to 512. Anvil -> Cubic Chunks Default option selected in the menu. Not using the 'move all by 0 16 0' or whatever.
Then i convert the created word to a new world with the Cubic Chunks -> Cubic Chunks (Relocating). Using the 'move all by 0 16 0' in relocatingConfig.txt. In the new create world (second) its moves 16 chunks up. (256 blocks). So, ok.
THEN i take the floor 0 region file, and convert it into the second created world.
THEN they are merged.
Maybe there are other ways, but like mentioned before just overwrites the whole 0-255 range, and doesn't moves anything 16 chunks (256 blocks) up..... I assume its only when option Cubic Chunks -> Cubic Chunks (Relocating) option is selected.
Anyway. Its working....

This is the way you were supposed to do it, more or less. I probably should have explained it better.

Thanks.

https://youtu.be/MY9EL2yRO0E
It Worked.
Op vr 9 okt. 2020 om 00:56 schreef Bartosz Skrzypczak <
<EMAIL_ADDRESS>
Closed #6
https://github.com/OpenCubicChunks/CubicChunksConverter/issues/6.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenCubicChunks/CubicChunksConverter/issues/6#event-3857683543,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APX6XFBNEGNZANMMKJETYJDSJY7RJANCNFSM4SJLLNRA
.


qwt: 6.1 -> 6.2

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[X] Fits CONTRIBUTING.md.

Would it be reasonable to consider renaming 'qwt' on top-level/all-packages.nix to be qwt5, and moving it to 5.nix, to help discourage usage of that in favor of the libsForQt versions?


Improve CI and packaging

This pull request improves the general packaging of pyavb and also improves the CI.
Packaging improvements:

Wheels produced when PYAVB_BUILD_EXT is set to 0 are now marked as pure and universal and compatible with both Python 2 and 3. This allows to have a single wheel with the idea that it's unnecessary to have python and platform specific wheels when the content is just pure python code. See the bdist_wheel section in the setup.cfg file to see how I did this.
python_requires metadata is now set to let pip (and any other installers) which Python version the library supports. This is extremely important when a library starts to drop support for Python 2 and any other versions of Python.
Wheels are now built using cibuildwheel. This allows to have known working wheels. This is because it takes care of compiling in vetted environments:

on Linux it uses docker images while on mac and Windows it builds on the host (because of their lack of docker support).
Wheels are "repaired", which means it creates wheels, then inspects them to make sure that it is self-contained and doesn't rely on any system level libraries that could cause problems between RedHat and Ubuntu for example.

Build dependencies are now provided via a pyproject.toml file. This means that if you pip install ., it will create a virtualenv with the dependencies set in pyproject.toml to build the package. Again, this is important for reproducibility.

CI improvements:

Wheels (for both compiled and non-compiled avb) are first created, then uploaded as artifacts, then the tests jobs download the artifacts and install the wheels. The tests are run against the installed wheels. This allows to verify that the wheels are working as expected (for example, if some files are missing, the tests would probably fail). It gives extra confidence that wheels can be released.
I changed the layout of the code to use what's called a src layout. That is the code lives under the src directory. This avoids CI to accidentally tests the source code instead of the installed code. You can see some explanations at https://setuptools.pypa.io/en/latest/userguide/package_discovery.html#src-layout and https://hynek.me/articles/testing-packaging/.
Pure and non-pure (compiled) wheels are tested. This ensures that pure (non-compiled) wheels continue to work (since the default is to use Cython if it's available).
Python 2.7 on Windows: The MSVC compiler is no more downloaded from the Internet Archive. Wheels are now compiled using what's available in the GitHub Actions windows runners. To achieve this, I had to set the /MT compiler flag. This method has been extensively battle tested by OTIO.

I manually tested the Python 2.7 wheel on Windows using a clean VM. Testing on a clean VM was required in this case because I wanted to be 100% that the latest runtime libraries were not present to ensure that the /MT flag did its job. I also confirmed the that pyd only linked to Python 2.7 DLL and nothing else, using dumpbin /dependents C:\path\to'_ext.pyd (dumpbin can be somewhat seen as the equivalent of ldd but for Windows).
But I still encourage you to test the wheels, just in case I missed something.
I hope this little wall of text isn't too much to digest... If there is anything, please let me know and I'll be happy to walk you through the changes or answer any questions or concerns.

BTW, you will have to approve the workflow so that it can run.

Thanks for being so detailed! I see the latest-release step failed, but that shouldn't be a issue as far a I can tell? I'll test it out locally and merge it if I don't find any issues. publish.yml will also need to be updated with the new artifacts names but I can do that.

Yeah, I was expecting the latest-release to fail. It fails because PRs coming from forks don't have write access to the target repo (for quite obvious reasons... You don't want random people to create PRs that would damage the repo).
As for the release pipeline, I completely forgot about it. I'll push a commit to use the new artifact name.

do the pure-wheels need to uploaded to pypi too?

In theory no. I'd have to do some testing and reading to see if it's common to have both pure and non pure wheels in PyPI. So I went with just uploading the Cython wheels for now.

So if you want, give me some time and I'll come back to tell if we can upload both the pure and non pure wheels.

I think it will be fine for now as long as we upload the sdist.

After looking at what Cython do themselves, they publish compiled and non compiled wheels + sdist:
$ curl -s https://pypi.org/pypi/cython/0.29.30/json | jq '.releases["0.29.30"][].filename' -r
Cython-0.29.30-cp27-cp27m-manylinux_2_5_i686.manylinux1_i686.whl
Cython-0.29.30-cp27-cp27m-manylinux_2_5_x86_64.manylinux1_x86_64.whl
Cython-0.29.30-cp27-cp27mu-manylinux_2_5_i686.manylinux1_i686.whl
Cython-0.29.30-cp27-cp27mu-manylinux_2_5_x86_64.manylinux1_x86_64.whl
Cython-0.29.30-cp310-cp310-manylinux_2_17_aarch64.manylinux2014_aarch64.manylinux_2_24_aarch64.whl
Cython-0.29.30-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl
Cython-0.29.30-cp310-cp310-manylinux_2_5_i686.manylinux1_i686.manylinux_2_24_i686.whl
Cython-0.29.30-cp310-cp310-musllinux_1_1_x86_64.whl
Cython-0.29.30-cp311-cp311-manylinux_2_17_aarch64.manylinux2014_aarch64.manylinux_2_24_aarch64.whl
Cython-0.29.30-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl
Cython-0.29.30-cp311-cp311-manylinux_2_5_i686.manylinux1_i686.manylinux_2_24_i686.whl
Cython-0.29.30-cp311-cp311-musllinux_1_1_x86_64.whl
Cython-0.29.30-cp35-cp35m-manylinux_2_5_i686.manylinux1_i686.whl
Cython-0.29.30-cp35-cp35m-manylinux_2_5_x86_64.manylinux1_x86_64.whl
Cython-0.29.30-cp36-cp36m-manylinux_2_17_aarch64.manylinux2014_aarch64.manylinux_2_24_aarch64.whl
Cython-0.29.30-cp36-cp36m-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl
Cython-0.29.30-cp36-cp36m-manylinux_2_5_i686.manylinux1_i686.manylinux_2_24_i686.whl
Cython-0.29.30-cp36-cp36m-manylinux_2_5_i686.manylinux1_i686.whl
Cython-0.29.30-cp36-cp36m-manylinux_2_5_x86_64.manylinux1_x86_64.whl
Cython-0.29.30-cp36-cp36m-musllinux_1_1_x86_64.whl
Cython-0.29.30-cp37-cp37m-manylinux_2_17_aarch64.manylinux2014_aarch64.manylinux_2_24_aarch64.whl
Cython-0.29.30-cp37-cp37m-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl
Cython-0.29.30-cp37-cp37m-manylinux_2_5_i686.manylinux1_i686.manylinux_2_24_i686.whl
Cython-0.29.30-cp37-cp37m-manylinux_2_5_i686.manylinux1_i686.whl
Cython-0.29.30-cp37-cp37m-manylinux_2_5_x86_64.manylinux1_x86_64.whl
Cython-0.29.30-cp37-cp37m-musllinux_1_1_x86_64.whl
Cython-0.29.30-cp38-cp38-manylinux_2_17_aarch64.manylinux2014_aarch64.manylinux_2_24_aarch64.whl
Cython-0.29.30-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl
Cython-0.29.30-cp38-cp38-manylinux_2_5_i686.manylinux1_i686.manylinux_2_24_i686.whl
Cython-0.29.30-cp38-cp38-manylinux_2_5_i686.manylinux1_i686.whl
Cython-0.29.30-cp38-cp38-manylinux_2_5_x86_64.manylinux1_x86_64.whl
Cython-0.29.30-cp38-cp38-musllinux_1_1_x86_64.whl
Cython-0.29.30-cp39-cp39-manylinux_2_17_aarch64.manylinux2014_aarch64.manylinux_2_24_aarch64.whl
Cython-0.29.30-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.manylinux_2_24_x86_64.whl
Cython-0.29.30-cp39-cp39-manylinux_2_5_i686.manylinux1_i686.manylinux_2_24_i686.whl
Cython-0.29.30-cp39-cp39-manylinux_2_5_i686.manylinux1_i686.whl
Cython-0.29.30-cp39-cp39-manylinux_2_5_x86_64.manylinux1_x86_64.whl
Cython-0.29.30-cp39-cp39-musllinux_1_1_x86_64.whl
Cython-0.29.30-py2.py3-none-any.whl
Cython-0.29.30.tar.gz

So I pushed another commit to do that too. Thanks for asking the question!


Update Package.md and Module.md with new tooling changes

Purpose
This PR is to update documentation for the new changes made to the bal persist tooling.
Examples
Checklist

[ ] Linked to an issue
[ ] Updated the specification
[ ] Updated the changelog
[ ] Added tests
[ ] Checked native-image compatibility

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: hasathcharu:x: daneshkYou have signed the CLA already but the status is still pending? Let us recheck it.


Auth fixes

I noticed some issues with the auth flow when developing core-app. This should fix things and make developing a bit easier. I still thing hot reloading isn't working as it should though.

Token refresh was broken as we weren't actually passing through the refresh token
Implement the session token cache that we had on the pwa for web only

/lgtm


Migrate to PF4 based react-console

Depends on https://github.com/oVirt/ovirt-web-ui/pull/1564
Re-implement console based on newest react-console
Changes:

drop portaled toolbars - react-console provides AccessConsoles
component with built-in grid layout that includes toolbar
drop support for native consoles on the console page:
a) to enter the screen user needs to first chose BrowserVnc console.
Only after disconnecting native consoles can be chosen
b) react-console by default generates the *.vv files - more research
is needed how to merge it with current flow (downloading)
c) react-console uses one common 'Connect' button for all native
consoles and internally picks the preferred connection type
d) react-consoles prefers SPICE over VNC when both are available
adapt VncConsole component from @patternfly/react-console
to fix following issues:
a) use 'binary' as default 'wsProtocols' to support Qemu < 5.0.0
b) infinite re-connect loop in case of connection problem - problem
caused by useEffect dependencies changing with every render and
triggering 'Maximum update depth exceeded' warning
c) add onConnect callback removed in newer version - required for
handling focus

Reference-Url: https://github.com/novnc/noVNC/commit/c912230309806aacbae4295faf7ad6406da97617
Reference-Url: https://github.com/oVirt/ovirt-web-ui/pull/1525#issuecomment-976329142
Port VncConsole from TypeScript to plain JS
Source:      @patternfly/react-console/src/components/VncConsole/VncConsole.tsx
Commit:     https://github.com/patternfly/patternfly-react/commit/e0240737eb6abb091c3ac83c3de8fb68420d7578

Some tests failed

Some tests failed

Some tests failed

Screenshots

Some tests failed

Some tests failed

All tests passed

All tests passed

All tests passed

All tests passed

All tests passed

All tests passed

All tests passed

All tests passed

/ost

All tests passed

L&F Notes:
VM Details:

On VM Details, if the VM is Off, the Console button doesn't do anything but it is not grayed out like the other disabled action buttons:

VM Details, Spice console via console.vv launches successfully

VM Details, native VNC via console.vv launches successfully

Browser console pages launches successfully

Browser Console:

Functions work as expected:

disconnect,
reconnect,
send ctrl-alt-del,
fullscreen

Directly opening console page for a VM works as expected

Changing screen resolution on the VM works!

Toolbar on the browser page has different padding than the VM Details toolbar.  Ideally they'd match.

@sjd78
The dropdown uses the default PF4 styling. The font color for the disabled state is the same as for active state.As you correctly pointed this is different then styling for regular buttons. However I'm not sure if we should change it - it might be a feature :)

All tests passed


Rbac users table

Created a simple sorting table as a par of rbac user editing components.
Strybook is available on https://5b31fa4967610c6fac9c340d--confident-banach-ee0a7a.netlify.com/?selectedKind=Rbac forms&selectedStory=Rbac users table&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

@karelhala component was renamed and moved.


Add sandbox unit tests

Hey, this PR contains two intended commits: The first one adds a network namespace interface NetNsIface to provide a mock-able abstraction.
In the second commit, I added the unit tests to the sandbox package.

I rebased on top of the latest master branch.

/ok-to-test

/approve
One question, otherwise changes LGTM

/test e2e_fedora

@saschagrunert needs a rebase.

Codecov Report

Merging #1996 into master will increase coverage by 51.62%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #1996       +/-   ##
==========================================
+ Coverage   29.18%   80.8%   +51.62%
==========================================
  Files          72      12       -60
  Lines        6530     865     -5665
==========================================
- Hits         1906     699     -1207
+ Misses       4520     113     -4407
+ Partials      104      53       -51

I rebased on top of the latest master branch.

/lgtm


Add Utility to Convert Country Code to Country Name

I am in the process of switching my app over from another country code picker library. One feature that library has which would be useful in this one is the ability to convert a country code to the country's name.
For example, in the other library:
CountryPickerUtils.getCountryByIsoCode("GB") -> returns "United Kingdom"

I just added a factory constructor CountryCode.fromCode
eg CountryCode.fromCode('GB')


<CR> keybind is enabled in command mode by default

From this line https://github.com/m4xshen/autoclose.nvim/blob/main/lua/autoclose.lua#L23, I infer that the autoclose.nvim functionality should not be enabled for <CR> in command mode by default. However, it is, which messes with my config.
I am not an expert in lua, but I guess the error may be in this line https://github.com/m4xshen/autoclose.nvim/blob/main/lua/autoclose.lua#L150. Should it not read if not info.disable_command_mode then, because the disable option is set per key and not in the keys table?

Thanks for the feedback! I want to know how the original functionality messes with your config. Could you explain a little bit?

I have the following in my init.lua:
-- fish-like command line completion
vim.opt.wildignorecase = true
vim.opt.wildmode = "longest:full,full"
vim.keymap.set("c", "<cr>", function()
	if vim.fn.pumvisible() == 1 then
		return "<c-y>"
	end
	return "<cr>"
end, { expr = true })

This makes command mode completion behave exactly the same as command line completion in fish, which I also use (enter adds the highlighted entry of the completion menu to the command and closes the menu, but does not execute the command). Thus simply, if any plugin rebinds <CR> in command mode, I notice because my muscle memory messes up command completion.
Now everything works as expected again, thanks a lot for your quick reply and this nice plugin. :)


Wrong docker command for 'make run' in documentation node "Docker Build"

Found out that documentation is not updated with the corresponding 'Makefile'.
'make run' command does not execute 'docker-compose run' it actually runs 'docker-compose up'

I have created a pull request for this now.

Fixed with https://github.com/gothinkster/aspnetcore-realworld-example-app/pull/88


Container nodes webAppestapp_0_2ab2aa8a didn't respond to HTTP pings on port: 3000, failing site start.

Package Name:
Deploying a NodeJS app according this tutorial: https://docs.microsoft.com/en-us/azure/developer/javascript/tutorial/tutorial-vscode-docker-node/tutorial-vscode-docker-node-04
Package Version: Not relevant
Operating system: Every
[x] nodejs

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[x] SDK API docs on https://docs.microsoft.com

Describe the bug
When you deploy the app according to the documentation, you receive the error: Container testapp_0_2ab2aa8a didn't respond to HTTP pings on port: 3000, failing site start. Stopping site testapp because it failed during startup.
To Reproduce
Steps to reproduce the behavior:

Start the tutorial here: https://docs.microsoft.com/en-us/azure/developer/javascript/tutorial/tutorial-vscode-docker-node/tutorial-vscode-docker-node-01
Create docker image: https://docs.microsoft.com/en-us/azure/developer/javascript/tutorial/tutorial-vscode-docker-node/tutorial-vscode-docker-node-04
Deploy app: https://docs.microsoft.com/en-us/azure/developer/javascript/tutorial/tutorial-vscode-docker-node/tutorial-vscode-docker-node-05

Expected behavior
View the web site in a browser (Getting a running app version as in the tutorial)
Screenshots
If applicable, add screenshots to help explain your problem.
Website

2022-04-22T19:15:44  Welcome, you are now connected to log-streaming service.
Starting Log Tail -n 10 of existing logs ----
/home/LogFiles/__lastCheckTime.txt  (https://h21testapp.scm.azurewebsites.net/api/vfs/LogFiles/__lastCheckTime.txt)
4/22/2022 7:14:48 PM
/home/LogFiles/kudu/trace/f694b1344245-f78b4563-37e0-487e-93c3-04f2eed0403f.txt  (https://h21testapp.scm.azurewebsites.net/api/vfs/LogFiles/kudu/trace/f694b1344245-f78b4563-37e0-487e-93c3-04f2eed0403f.txt)
2022-04-22T19:14:29  Startup Request, url: /api/vfs/site/wwwroot/?_=1650654812800, method: GET, type: request, pid: 67,1,6, ScmType: None
/home/LogFiles/2022_04_22_RD0003FF8A4221_default_docker.log  (https://h21testapp.scm.azurewebsites.net/api/vfs/LogFiles/2022_04_22_RD0003FF8A4221_default_docker.log)
2022-04-22T19:14:45.271578119Z standard_init_linux.go:228: exec user process caused: exec format error
2022-04-22T19:15:05.220659958Z standard_init_linux.go:228: exec user process caused: exec format error
/home/LogFiles/2022_04_22_RD0003FF8A4221_docker.log  (https://h21testapp.scm.azurewebsites.net/api/vfs/LogFiles/2022_04_22_RD0003FF8A4221_docker.log)
2022-04-22T19:15:00.651Z INFO  - docker run -d -p 6443:3000 --name h21testapp_0_2ab2aa8a -e WEBSITE_SITE_NAME=h21testapp -e WEBSITE_AUTH_ENABLED=False -e PORT=3000 -e WEBSITE_ROLE_INSTANCE_ID=0 -e WEBSITE_HOSTNAME=h21testapp.azurewebsites.net -e WEBSITE_INSTANCE_ID=faf4dec0e866a78a704d6f50882bec16fc24fe887063009bbdffd2526cac326f h21testreg.azurecr.io/jse2eexpressserver:latest
2022-04-22T19:15:00.653Z INFO  - Logging is not enabled for this container.Please use https://aka.ms/linux-diagnostics to enable logging to see container logs here.
2022-04-22T19:15:05.325Z INFO  - Initiating warmup request to container h21testapp_0_2ab2aa8a_msiProxy for site h21testapp
2022-04-22T19:15:05.335Z INFO  - Container h21testapp_0_2ab2aa8a_msiProxy for site h21testapp initialized successfully and is ready to serve requests.
2022-04-22T19:15:05.341Z INFO  - Initiating warmup request to container h21testapp_0_2ab2aa8a for site h21testapp
2022-04-22T19:15:06.442Z ERROR - Container h21testapp_0_2ab2aa8a for site h21testapp has exited, failing site start
2022-04-22T19:15:06.496Z ERROR - Container h21testapp_0_2ab2aa8a didn't respond to HTTP pings on port: 3000, failing site start. See container logs for debugging.
2022-04-22T19:15:06.535Z INFO  - Stopping site h21testapp because it failed during startup.
Ending Log Tail of existing logs ---
Starting Live Log Stream ---
2022-04-22T19:15:55    Outgoing response, type: response, statusCode: 404, statusText: NotFound
2022-04-22T19:15:53    Outgoing response, type: response, statusCode: 404, statusText: NotFound
2022-04-22T19:17:44  No new trace in the past 1 min(s).

Additional context
There seems to be a problem with the web port exposing. However, also the feedback from another guide didn't help: https://docs.microsoft.com/en-us/answers/questions/168746/container-didn39t-respond-to-http-pings-on-port-80.html

@qiaozha, could you take a look? thanks

I was able to resolve the issue via specifying the correct architecture:
docker buildx build --platform linux/amd64 --rm --pull -f Dockerfile …
Maybe you should mention this in the tutorial.

@diberry Can you help take a look at this issue and see if we need update the document according to the comments above? Thanks

facing same error,deploying django container using docker hub single container public image.
Container smartlib_0_dc086ccb didn't respond to HTTP pings on port: 80, failing site start. See container logs for debugging.

@MoshiulRabbi Does the work around @Kondamon mentioned above work for you ?

Sorry, wrong thread. I should check the python sdk

Close this one as this is document issue not JS SDK issue, new issue created in https://github.com/MicrosoftDocs/azure-dev-docs/issues/851

Close this one as this is document issue not JS SDK issue, new issue created in https://github.com/MicrosoftDocs/azure-dev-docs/issues/851

@Kondamon What operating system were you running this tutorial on?

@diberry MacOS Monterey.

Hi @Kondamon, we can't repro the issue on M1 but we do have the xbuild autoconfig'd in the Docker desktop. Can you please check if you have that configured or what version of docker you are using?

Hi @diberry I was using Docker 4.7.0 (77141). Where did you enable the xbuild autoconfig in Docker?

I ran this procedure on my mac monterey 12.5.1 with docker version 20.10.17 and didn't hit this issue. I'm going to link from the procedure to this issue.
#please-close

@diberry, Azure documentation has already been updated by someone else (see my last comment). There is nothing more to do.

#please-close

Tried using buildx and still getting the same issue, it works locally but not with the aoo :( very sad!

This is seriously frustrating please can MSFT actually provide a turorial that works and that they have tested been wating hours now, going to try on AWS as this basic stuff should not be such a struggle...

@diberry please can you provide a link to a working example, I have tried everying now for hours with these updates and it still does not work. I am running Ubuntu on WSL.

@wpitallo Can you sharemore details on how to repro this issue and if you don't mind, can you open a new issue for us to track  it ? Thank you


Move configuration to TOML

Python seems to be adopting TOML more and more in its ecosystem 1 2, and TOML appears to be an improvement on INI, and simpler than YAML. Some considerations:

Is there reliable backend/client JS TOML parser?
TOML parser is not in the Python std library - but the parser is lightweight.
The benefit of moving to TOML is not necessarily using their unique features, but avoiding error / ambiguity in users entering information.
OTOH, there is some drama around which TOML library to use, and unmaintained libs.

In Python 3.11 (unreleased as of today), there is a built in TOML library.

I don't think we should move to TOML


added domain of sivas university of science and technology

@bl4cktrum First stage of verification passed, official domain is sivas.edu.tr as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@bl4cktrum Thank you for your patience while waiting for a review and processing your request. Pull request merged.


fix(total-balance): Reuse TotalBalance from Home screen in FiatExchange Screen

Description
New Designs

The FiatExchange screen was using a very simplistic and outdated calculation of total value which included just Cusd. It now reflects the value of the homescreen with a slightly different style
New

Old

Other changes
Renamed HomeTotalBalance file to TotalBalance to make it more versatile.
Tested
Visually and with new tests.
How others should test
Verify that the Home screen and Withdraw screen show the same Total Value
Related issues

Fixes [Bug] Add funds current balance in the withdraw flow includes cUSD only

Backwards compatibility
Brief explanation of why these changes are/are not backwards compatible.

Hey @MuckT, We have verified the above issue in Latest Android Internal Release build V:1.26.0 and updated the status in test rails.
Test rail link:
Android 8: https://valoraapp.testrail.io/index.php?/tests/view/9820&group_by=cases:section_id&group_order=asc&group_id=35
Android 10: https://valoraapp.testrail.io/index.php?/tests/view/9946&group_by=cases:section_id&group_order=asc&group_id=35
Device: Samsung Galaxy A8+(8.0), Samsung Galaxy M40(10.0)
Cc: @jh2oman
Thanks..!!

Hey @MuckT, We have verified the above issue on Latest Test flight Release Build V1.26.0 and updated status in Test rails.
Test rail link:
iOS 13: https://valoraapp.testrail.io/index.php?/tests/view/10012&group_by=cases:section_id&group_order=asc&group_id=35
Device: iPhone 7(13.3)
CC: @jh2oman
Thanks..!!


Heroku

Needs success URL (pointing to scheduler) and instructions.

Example URL: https://dashboard.heroku.com/apps/foo-bar-12345
Smallest window I can create with Chrome:

Possibly embed that as an iFrame?


[ENH] Improved error message for MultIndex DataFrames in case level(-1) is not containing a time index

Is your feature request related to a problem? Please describe.
for panel forecasting we use MultiIndex and the inner index level has to be the time index. This is an established convention within the framework and should therefore raise a more informative error message and checks on that. Users might not be aware of this convention.

where would this error message be raised? Via the datatypes conversions called in the base forecaster?

I'm closing this issue as we are moving away from multiindex, feel free to reopen if there is more information


How to exit full screen when using ArFragment

Hello, i've been trying all sorts of things to not make statusbar hide itself when ArFragment loads, but can't seem to successfully do it by setting/clearing various activity window flags. I even tried OnGlobalLayoutListener for sceneView but still no luck. How can this be achieved
This is how i mainly tried it within global listener, scene OnUpdateListener etc.
activity?.window?.clearFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN)
activity?.window?.decorView?.systemUiVisibility = SYSTEM_UI_FLAG_VISIBLE

Navigation bar is also hidden, which i'd like to revert, I assume it is the same fullscreen flag problem

Thanks for the question.  Currently there is no way to directly control this via the fragment configuration.  A work around is to override the onWindowFocusChanged()  methodin a subclass of ArFragment, then you can implement it as needed.

Ok this resolves the problem (if i don't call the super, just to clarify for others). Thanks for your help!

Hi Clayton.
Please, could you explain a little more on this.
How can I use onWindowsFocusChanges from inside a Fragment?
I usually follow this way when an Activity is used:

@Override
    public void onWindowFocusChanged(boolean hasFocus) {
        super.onWindowFocusChanged(hasFocus);
        // Get a support ActionBar corresponding to this toolbar
        ActionBar ab = getSupportActionBar();
        ab.setDisplayHomeAsUpEnabled(true);
        //
        if (hasFocus) {
            showSystemUI();
        }
    }

    private void showSystemUI() {
        // Enables regular immersive mode.
        // For "lean back" mode, remove SYSTEM_UI_FLAG_IMMERSIVE.
        // Or for "sticky immersive," replace it with SYSTEM_UI_FLAG_IMMERSIVE_STICKY
        View decorView = getWindow().getDecorView();

        // Hide both the navigation bar and the status bar.
        // SYSTEM_UI_FLAG_FULLSCREEN is only available on Android 4.1 and higher, but as
        // a general rule, you should design your app to hide the status bar whenever you
        // hide the navigation bar.
        decorView.setSystemUiVisibility(View.SYSTEM_UI_FLAG_LAYOUT_STABLE| View.SYSTEM_UI_FLAG_LAYOUT_HIDE_NAVIGATION| View.SYSTEM_UI_FLAG_LAYOUT_FULLSCREEN);
    }

...but I can´t call getSupportActionBar or getWindow in the case of a Fragment.
Thanks for your time.


Doesn't work with object of derived class

The comparer doesn't work when object of derived class is used.
Ex:
public class BaseClass
{
 public int Id {get; set;}
}

public class DerviedClass
{
 public string Name {get; set;}
}

Comparer works when BaseClass property is different
var obj1 = new DerivedClass { Id = 1, Name = Foo };
var  obj2 = new DerivedClass { Id = 2, Name = Foo };

var _comparer = new ObjectsComparer.Comparer<BaseClass>();
var equal = _comparer.Compare(obj1, obj2, out differencesList);
// equal is false

But when derived class property is different it doesn't
var obj1 = new DerivedClass { Id = 1, Name = Hello };
var  obj2 = new DerivedClass { Id = 1, Name = Foo };

var _comparer = new ObjectsComparer.Comparer<BaseClass>();
var equal = _comparer.Compare(obj1, obj2, out differencesList);
// equal is true

Is this a issue of it's by design?

I think you have to write it like this:
var _comparer = new ObjectsComparer.Comparer<DerivedClass>();

@reponemec is right. When you do Comparer() only members of T will be compared


nodes中的disk信息只获取了"/", 例如挂载的/data数据卷无法显示容量

Given that you it is reporting that you are using 3% of the data and it looks like the graphs are about 3% full I am closing as "working as intended"


bug(cdkDrag): "pointerPosition" property of the "CdkDragMove" event is not correct with "cdkDragBoundary"

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Context
According to the official documentation, the property "pointerPosition" of the "CdkDragMove" event is : "Position of the user's pointer on the page."
See : https://material.angular.io/cdk/drag-drop/api#CdkDragMove
Problem
If I use the "cdkDragBoundary" property, the property "pointerPosition" returns cursor position only between boundaries.
Reproduction
StackBlitz link: https://stackblitz.com/edit/stackblitz-starters-knrvcz?file=src%2Fmain.ts
Steps to reproduce:

Move the red block until the cursor moves out of the blue block
At the bottom of the blue box, you will see that the cursor position is stuck to the boundaries of the blue box.

Expected Behavior
I expect to have the current position of the cursor on screen, wherever it is located.
Actual Behavior
The position displayed of the cursor is limited to the limits of the blue block
Environment

Angular: 18.2.8
CDK/Material: 18.2.8
Browser(s): Chrome 129.0.6668.100
Operating System (e.g. Windows, macOS, Ubuntu): Windows 11

Actually the position you are displaying is of your div which contain "Move" text , that is not the position of cursor

@crisbeto I assume this is working as intended but it might just be confusion around the naming. My understanding is that pointerPosition represents the position of the square that the pointer "grabbed" onto, correct?


Project idea: guides

What do you think about adding a folder like "guides" or something like this and add tutorials or guides for example for setting up a development environment or setting up a server or network correctly etc.
Example:

📁 guides

📑 setup-dev-environment.md
📑 setup-a-server.md
📑 setup-server-docker.md
📑 scheduler-guide.md
📑 commands-guide.md
📑 README.md (Index with links to all guides)

📑 api-repository.md
📑 requirements.md
📑 servers.md

I think that this would be a great addition to this repository!
It looks like you already got some nice ideas. If you have some free time, can you open a second issue that lists your guide ideas with a short description and what the goal is?
However, Spigot already has a wiki section (cf. https://www.spigotmc.org/wiki/) and I don't want to mirror them. So especially, I think that "new" guides like setup-server-docker or setup-dev-env are more interesting.


Add delete component argument method

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 decreased (-87.7%) to 0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-87.7%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

This is all AST related terminology on the Modelica code which we navigate through XPaths. A  Selector finds the object or component that we want to manipulate. ComponentArgumentSelector finds the argument name, then we call make_delete for it to disappear. The unique issue with this one is that we must also delete one of the terminal nodes (i.e., , separator of the argument).


correctly set mon addr when monitor_interface is defined in group_vars

This fixes the ceph.conf template so that it will look for an inventory
defined value for monitor_interface or for monitor_interface defined in
a group_vars file.
Signed-off-by: Andrew Schoen<EMAIL_ADDRESS>

Sorry for all the bugs in this template :(


[BUGFIX] Inconsistency in settings on startup of the app.

It should be ensured that on the startup of the app all default values ideally from Config class are published to homeassistant to ensure consistency with each run.

After further consideration not really an issue anymore after introducing state_locks


OverwriteNewerLocalError raised despite force_overwrite_from_cloud=True in parallel read only scenario

Hi,
In v0.20.0, I'm running into an issue where, when trying to read an S3 file from multiple readers in a parallel scenario (multiple readers, no write access), I am passing force_overwrite_from_cloud as instructed in the error report, but still getting an OverwriteNewerLocalError.
I will try to set different caches for each process, but this could be wasteful as I am only reading and will have to copy already downloaded files.
Related to isses: #283 and #128 , but not exactly.
The error reads like this:
...
fp = file_path.open('rb', force_overwrite_from_cloud=True)
File "/opt/conda/lib/python3.10/site-packages/cloudpathlib/cloudpath.py", line 686, in open
self._refresh_cache(force_overwrite_from_cloud=force_overwrite_from_cloud)
File "/opt/conda/lib/python3.10/site-packages/cloudpathlib/cloudpath.py", line 1303, in _refresh_cache
raise OverwriteNewerLocalError(
cloudpathlib.exceptions.OverwriteNewerLocalError: Local file (/tmp/tmp9h3c_4qk/s3://|bucket|/|ObjectPath|/GT_32.h5) for cloud path (s3://|bucket|/|ObjectPath|/GT_32.h5) is newer on disk, but is being requested for download from cloud. Either (1) push your changes to the cloud, (2) r:emove the local file, or (3) pass force_overwrite_from_cloud=True to overwrite; or set env var CLOUDPATHLIB_FORCE_OVERWRITE_FROM_CLOUD=1.

Thanks for this report. I'm going to close this issue since it is almost certainly the same root cause as #283.
Let's continue the conversation there. I will add some thoughts.


Add empty state for "Projects" panel in VS Code extension

Issue
Currently, "Projects" panel inside the VS Code extension is fully empty if no projects are configured inside the workspace. "Last run" tab in comparison has empty state with a description describing that what to do.

With that "Projects" panel being empty like that, it seems like it's just broken. This is not a bug because I really don't have any projects defined in this workspace, but still it looks weird.
Potential solution
Maybe adding a some text like "No projects found in workspace. Learn how to set up projects --link to docs--" so that extension can point to docs and give some info why it's empty.

Good callout, is easy enough to add.


Enable test and change warning message

This simple test at least passes.

Ah I made a mistake (ran the wrong test!). Getting an error from if "pytket_to_qir" not in locals().
Closing for now...


Added Setting key value in struct based on struct tag

Fixes #21

Updated README to show how to use the new struct tag

Coverage increased (+0.04%) to 84.446% when pulling de91d4127b655d9ef8ba3d638c437abd2c45f305 on develop into 86fabb5a318023405272b7a689a5c1e2a960e4db on master.


Add files via upload

Duran's project 2 code

Putting something into the main branch so we have a place to work from.


Enhancement: Change font size or zoom in/out

The font size in the Task Board is quite large. Although it is good for readability, I am restricted to viewing only a certain number of columns at a time because we can't zoom at all.
To be able to zoom with Cmd + + and Cmd + – would be much appreciated.

Fixed in 2.0


Python's icon

Hi Jorricks!
I want to ask you if you've looked into changing the default Python icon that pops up in the notification on the left and, if you have, could you share how to do it.
Thanks for the code!

Hey @Denkiri21,
First of all, thanks for opening the issue and second, sorry for responding this late. I looked into this quite a bit but the only thing you can do is completely change the Python interpreter image in Macos settings. This means you would change one permanent image for the other. You can however add a small side image, which I usually use for my projects.

I have made this limitation clear in the README here;
https://github.com/Jorricks/macos-notifications/commit/e91cba44a6b153eae5160bf9d90785af8e3544dd


⚠ Mobitopup has degraded performance

In 1cf8fdd, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 24554 ms

Resolved: Mobitopup performance has improved in ae7035b.


Make Condition and ReplicaStatus optional

What this PR does / why we need it:
I made jobStatus.Condition and jobStatus.replicaStatus optional. We shouldn't make those fields necessary since those fields can be null.
/assign @johnugeorge
Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #
Checklist:

[ ] Docs included if any changes are user facing

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
15 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 32.962%

Files with Coverage Reduction
New Missed Lines
%

pkg/controller.v1/mpi/mpijob_controller.go
6
78.98%

pkg/controller.v1/pytorch/pytorchjob_controller.go
9
59.95%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
3194

Relevant Lines:
9690

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
15 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 32.982%

Files with Coverage Reduction
New Missed Lines
%

pkg/controller.v1/mpi/mpijob_controller.go
6
79.17%

pkg/controller.v1/pytorch/pytorchjob_controller.go
9
59.95%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
3196

Relevant Lines:
9690

💛  - Coveralls

/lgtm
/approve


[Hacktoberfest] A function to create a job

Task
Implement a function to create a job in JavaScript SDK.
Follow similar pattern as in the Python SDK.
Deliverables
Create a PR to this repository with the implementation.
:loudspeaker: Additional rewards :trophy:
This issue brings up to :five:  :zero:  points for SWAG and entry into the draw for a  :point_down: :

:three:  :x:   :computer:  Razer Blade 16 Gaming Laptop
:shirt: :bear:  MindsDB Swag

:information_source: For more info check out https://mindsdb.com/hacktoberfest :point_left:

I want to work on this

@martyna-mindsdb Hi, I finished the issue with inserting new items into a table. I would like to work on this one!


Is there any signal or topic can be detected when we reach to goal

As title, when vehicle reach the goal, is there any signal or topic can be detected?
we want to know whether the vehicle is reach the goal or not
thank you
System information

Linux Ubuntu 14.04:
Apollo installed from source:
Apollo version 3.5:

maybe you can check if planning->decision->main_decision->mission_complete?

OK, it works, thank you :-)


Artistic Style Transfer using CycleGAN

#68 Generating Van Gogh style pictures from random normal pictures !!

@smaranjitghose  dada please check the  pr !!

@smaranjitghose  dada please check the  pr !!

I am having a problem in pushing the changed files , thats why i am closing this and i will add a new pr !! Sorry for the inconvenience @smaranjitghose .


Figure out what to do for copyright on maps not through City of Vancouver

We received permission for the maps that weren't scanned on archives, we used the GVRD map since we have permission for other open maps projects, and the rest we are leaving. Evan might upload a picture of the maps for the Vancouver Urban Analysis series


Remove unused socket code, and ioutils::SzMkTemp function.

ioutils::SzMkTemp called mktemp(), which is a security risk. That caused
a compiler warning. The only thing that used ioutils::SzMkTemp was the
test code for CSocket, but all the socket code was unused anyway, so just
remove the whole lot.
I was also getting a warning/error on strict aliasing:
/home/heikki/gpdb/gpos/server/src/unittest/gpos/net/CSocketTest.cpp:549:27: error: dereferencing type-punned pointer will break strict-aliasing rules [-Werror=strict-aliasing]
GPOS_ASSERT((*(ULONG *) szRecv) == pinitUds->m_ulId);
^
So that's another small reason to get rid of this.

this gporca PR: https://github.com/greenplum-db/gporca/pull/85 needs to be merged before this, to remove the last callers of the CSocket stuff.

＋1, glad to see unused code gone.

Bumped version number and pushed, thanks!


Wonky spacing in Hyper term in demo.

Not sure if it's a general issue or specific to the demo. (?)

most probably your terminal is not configured to use a monospace font. try to change its setting or use another terminal app that defaults to a monospace font such as the osx default terminal.

I am using a monospace font, in fact I'm using Meslo LG.
It seems to me that the problem is the way that whitespace is generated.

do you have whitespace in your text for the logger (or other text)? If so does it work when not using white space?

Those are good hints, will check it out later today when I get some time. I saw the source earlier, I don't have a really good grasp on blessed so I'll start out by checking how boxes and whitespace are handled. I already checked if someone is dealing with Hyper term related problems but seems like that's niche. Will report back later.


Merging of dependencies to Pretzel.exe make developing plugins a pain

The way dependencies are currently merged to Pretzel.exe make it painful to develop plugins to Pretzel without compiling together with the Pretzel project. This actually defeats the purpose of plugins.
The problem is caused as all dependent packages are merged to the Pretzel.exe assembly, and are not included as Resouces containing the original Dll-s, and loaded by a custom loading mechanism.
An example I have encountered:

I start writing a plugin project, and need reference to some Pretzel API types.
I add Pretzel.exe as a reference to the project.
I write a class that interfaces a Pretzel class from an 3rd party dll. Example: System.IO.Abstractions.FileSystem is passed on the component boundary.
I want to write a test to the plugin, using System.IO.Abstractions.TestingHelpers
I notice that the strong name of the IFileSystem interface implemented the MockFileSystem from the TestingHelpers is not the same as the one used by SiteContextenerator, for example, as that is an interface with the same FQCN, but from a different assembly.
I cry in pain.

I'm not totally amused by this "merged assembly" exe, as I see it causes more problems than having multiple files around, but probably this requirement is so strong that it must be kept. To solve these problems I think that only the classes in namespaces Pretzel.* should be "merged", other dependencies should be "embedded".
I think someone with more knowledge about this Fody blackmagic can do this trivially, but if the issue stay open for a long time, eventually I'll give it a shot :smile:

Note: this assembly merging stuff fails to work on mono as well. I'd really like if there were pretzel.logic packages in nuget, and I could start writing a plugin in a "clean room" manner, instead of having to hack a lot around.
Maybe simply having Pretzel.API assembly in NuGet, which Pretzel projects would also depend on would be nice. But I'm not sure how many people try to develop plugins to pretzel without docs, and with these hardships.

About the plugins: I know that right now it's a big pain, but with the master branch you have the possibility to use ScriptCs (through Pretzel.ScriptCs) which remove the need to reference Pretzel.exe.
And for the doc, I plan to do it since a while but I haven't got the time...
For the "one and unique exe" merge stuff, I was not here at the start of the project so I don't know why this is important but I think it's to simplify the use/installation of Pretzel.
@JakeGinnivan, @shiftkey do you think that the merge in one exe is still important?
Since it's cause trouble on mono and with the plugins I think that we could stop to do it.
Otherwise, Fody seems to cause trouble in Mono (see #215), but we can try IL-Repack.

I have tried a Fody + ILMerge based hybrid merging on my machine. It worked under windows, I'll try it under mono, and if it works well, I'll submit a PR.
What I did: I Fody.Costura embed every assembly, except for Pretzel.Logic, to the Pretzel.exe.
I ILMerge the Pretzel.Logic.dll to Pretzel.exe
Also I'm not sure if Fody didn't only hinder compilation impossible under mono. I have tried @vikingcode 's branch back then (and have borrowed the Fody related stuff from him in my above experiment), but I don't remember the details.

do you think that the merge in one exe is still important?

Conceptually, this is something I still like. But clearly this isn't great for the extensibility side, so I'm not going to be dogmatically against keeping it this way.


Private/hidden listings

This is related to #183 but there is a use-case for supporting private/hidden listings.
The rationale is that I want to create a listing on my store (or even while not having an official store) that is only accessible to anyone or a specific user provided they have the correct contract hash (ID).
This would also mean that private listings would have their keywords disabled in order to prevent public discovery.

This is definitely a desirable feature. The contracting system lends itself well to invoicing, so you could craft an invoice as an item and then send the link along, but this wouldn't work if it's publicly listed.
Lots of use cases for this. If it's easy to implement it should be a v1 goal.

Updating with some more thoughts:
Right now you can either display an item or delete an item, with nothing in between, but that's going to be a major problem for someone who runs out of stock temporarily.
@morebrownies suggests:

Listings need some type of status field so that vendors can set them to active Out of stock or inactive

They’ll need this for adding new items to their store too. They may want to preload listings and activate all of them on the same day when they’re ready.

[If someone lists as out of stock ] we could hide the buy buttton and display out of stock instead

Private/hidden listings/stores is definitely a good idea and should even be a requirement for many people when they start the creation of a store.


Load Measurement broken

I was working on #108 when I noticed this. Attempting to load a measurement gives a "Loaded 1 measurement" but nothing shows up.

2023-10-12 17:11:18,304 [0x00004db8] enlighten.parser.ColumnFileParser ERROR    unable to convert to datetime: [['2022-09-29 15:59:28.481182']]
2023-10-12 17:11:18,304 [0x00004db8] enlighten.parser.ColumnFileParser DEBUG    active_pixels_horizontal = 1652
2023-10-12 17:11:18,304 [0x00004db8] enlighten.measurement.MeasurementFactory ERROR    failed to parse file C:/Users/sbee/Documents/EnlightenSpectra/New SIG Library/Methanol.csv
Traceback (most recent call last):
  File "Y:\projects\Enlighten-B\enlighten\measurement\MeasurementFactory.py", line 263, in create_from_file
    measurements = [ self.create_from_columnar_file(pathname, encoding=encoding) ]
  File "Y:\projects\Enlighten-B\enlighten\measurement\MeasurementFactory.py", line 410, in create_from_columnar_file
    return parser.parse()
  File "Y:\projects\Enlighten-B\enlighten\parser\ColumnFileParser.py", line 51, in parse
    self.post_process_metadata()
  File "Y:\projects\Enlighten-B\enlighten\parser\ColumnFileParser.py", line 204, in post_process_metadata
    state.boxcar_half_width = int(self.get_safe_float(key))
  File "Y:\projects\Enlighten-B\enlighten\parser\ColumnFileParser.py", line 89, in get_safe_float
    if len(v.strip()) == 0:
AttributeError: 'list' object has no attribute 'strip'
2023-10-12 17:11:18,305 [0x00004db8] enlighten.measurement.Measurements DEBUG    create_from_file: no Measurements parsed from C:/Users/sbee/Documents/EnlightenSpectra/New SIG Library/Methanol.csv
2023-10-12 17:11:18,305 [0x00004db8] enlighten.file_io.FileManager DEBUG    scheduling load of the next file
2023-10-12 17:11:18,306 [0x00004db8] enlighten.device.Multispec DEBUG    Multispec.current_spectrometer: self.device_id is None

This should be corrected and the surrounding try-except should make a msgbox upon failure instead of silently passing the traceback to the enlighten.log.

I ran into this a day or two ago, and am testing a change — not ready to call it a fix — in the ColumnFileParser changes on https://github.com/WasatchPhotonics/ENLIGHTEN/pull/313.

I pulled that specific change into sbee-style and it worked for me. It allowed me to demonstrate the artifact I was looking for. #108 is resolved for the main graph but not for all the small ones.

afaik this is resolved 4.1.0-dev.

afaik this is resolved 4.1.0-dev.

I'm worried about how it happened though.  I haven't tried to trace through history, but I'd like to.

It broke because of me on eea92bb. Really dropped the ball here. Sorry guys.
The commit message "fix broken CSV load" is concerning. I suspect I was testing on some file that needed scalar_metadata=False. I don't know what file that was, and I don't know what error I was getting at that time.
This time, the file I used to trace history, and open this issue, is attached. The error message is above AttributeError: 'list' object has no attribute 'strip'
Methanol.csv - needs scalar_metadata=True

If it breaks again, let's reopen this issue and be sure to attach the file and error message.

Fixed in 4.1.0


Create a load gate for spins

Currently we define the length of the spin as:
 job = backend.run(qc, shots = 1000, spin=Nat/2)

It would be much cleaner to define the spin in a similiar fashion as for the fermions.
circuit1.load_spins(0,Nat)

This will also make it easier to work with wires of different spin length.

Closing this issue, as this is implemented in #17.
Note that this load instruction is not yet compatible with the simulator as this would require quite substantial changes in the numerics.
This, however, is already discussed in another open issue #9.


Code: EndpointUnsupportedAccountFeatures Message: This endpoint does not support BlobStorageEvents or SoftDelete.

Package Name: azure_storage_file_datalake
Package Version: 12.5.0
Operating System: Windows 10
Python Version: 3.9.10

Describe the bug
I have disabled SoftDelete for blobs and containers and have no BlobStorageEvents but when I run the following code:
`       initialize_storage_account()
    file_system_client = service_client.get_file_system_client(file_system="perfectStore")

    i = 0

    LOCALFILENAME_TEMP = LOCALFILENAME + '-temp_' + str(i) + ".csv"

    paths = file_system_client.get_paths(path=blob_dir)

    for path in paths:

        print(path.name + '\n')`

I get:
Code: EndpointUnsupportedAccountFeatures Message: This endpoint does not support BlobStorageEvents or SoftDelete. Please disable these account features if you would like to use this endpoint. RequestId:05bc1ea9-001f-005c-0a56-12102f000000 Time:2022-01-26T01:45:53.3527214Z
at line for path in paths:
To Reproduce
Steps to reproduce the behavior:

create adl2 file system
disabled SoftDelete (Data Protection panel)
Checked in Events panel for any subscription to Events (no subscriptions found) (BlobStorageEvents)
ran code above

Expected behavior
code should loop through paths without throwing errors
Others
I was following the steps detailed here: https://docs.microsoft.com/en-us/azure/storage/blobs/data-lake-storage-directory-file-acl-python

@tasherif-msft do you know who is responsible for Data Lake issues?

@jalauzon-msft

Hi @javiercoh, thanks for reporting the issue. We will investigate and get back to you shortly.

hi @jalauzon-msft any news on this? I tried many changes to the config of the datalake with no success so far... this is a key feature for us

Hi @javiercoh, I am having trouble reproducing this issue.  This will let me look into the service logs for more information. The service logs are only retained for a couple of days so the more recent your request, the better. Thanks.


cmd/go: unused function addModFlag in internal/modcmd/mod.go

In internal/modcmd/mod.go there is a function addModFlags that is never used:
func addModFlags(cmd *base.Command) {
        cmd.Flag.StringVar(&cfg.ModFile, "modfile", "", "")
}

The flag is set in the internal.work.AddModCommonFlags function, so it was probably not removed in https://golang.org/cl/202564.

Thanks. CL 219197 will remove this.

Thanks.
By the way; I find it unusual that no tool reported this problem.  Non even staticcheck -unused.whole-program .


{WIP} Implement registry library

Signed-off-by: jingfu wang<EMAIL_ADDRESS>What does does this PR do / why we need it:
This PR implements the registry library to allow consumers to interact with the registry, currently the registry library supports:

Get the devfile that belongs to a given stack
Get the devfile + the containing stack resource files of a given stack

Which issue(s) this PR fixes:
Fixes https://github.com/devfile/api/issues/224
PR acceptance criteria:

[x] Test (WIP)

[ ] Documentation (WIP)

How to test changes / Special notes to the reviewer:

Have we considered packaging this as part of the devfile library rather than having a separate Go library just for the registry? My thinking is that anything consuming this library will also need to consume the devfile library, and having both libraries together makes things more consistent. CC @elsony

It may not be true that anyone who accesses the registry will need to access the devfile library. Take the Che scenario, my understanding is that they have multiple components who need to access the registry and some of them just need to do the list but do not need to access the actual devfile itself. Having said that, I am open to discuss on if we want to lump them under the same library or not. I just added an item to the next team meeting to discuss.

Have we considered packaging this as part of the devfile library rather than having a separate Go library just for the registry? My thinking is that anything consuming this library will also need to consume the devfile library, and having both libraries together makes things more consistent. CC @elsony

It may not be true that anyone who accesses the registry will need to access the devfile library. Take the Che scenario, my understanding is that they have multiple components who need to access the registry and some of them just need to do the list but do not need to access the actual devfile itself. Having said that, I am open to discuss on if we want to lump them under the same library or not. I just added an item to the next team meeting to discuss.

@elsony Sounds good 👍

@elsony Sounds good 👍

I left some review comments
Some non-specific comments that I had as well:
1. We should look at testing on the OpenShift CI against a devfile registry (post-merging)

I was thinking about the same, we may also need to look at the pre-merging case, like provision a devfile registry so that registry library can be tested.

2. Have we considered packaging this as part of the devfile library rather than having a separate Go library just for the registry? My thinking is that anything consuming this library will also need to consume the devfile library, and having both libraries together makes things more consistent. CC @elsony

IMO, it depends on if some of existing consumers only need to access the registry. If yes we should separate, but if no we can merge the registry library with devfile library. We need to figure out the use cases from consumers perspective.

I left some review comments
Some non-specific comments that I had as well:
1. We should look at testing on the OpenShift CI against a devfile registry (post-merging)

I was thinking about the same, we may also need to look at the pre-merging case, like provision a devfile registry so that registry library can be tested.

2. Have we considered packaging this as part of the devfile library rather than having a separate Go library just for the registry? My thinking is that anything consuming this library will also need to consume the devfile library, and having both libraries together makes things more consistent. CC @elsony

IMO, it depends on if some of existing consumers only need to access the registry. If yes we should separate, but if no we can merge the registry library with devfile library. We need to figure out the use cases from consumers perspective.

I was thinking about the same, we may also need to look at the pre-merging case, like provision a devfile registry so that registry library can be tested.

For that, I just meant it can be looked at after merging this PR, as we'll need to work with QE to get the tests running on openshift CI

I was thinking about the same, we may also need to look at the pre-merging case, like provision a devfile registry so that registry library can be tested.

For that, I just meant it can be looked at after merging this PR, as we'll need to work with QE to get the tests running on openshift CI

@johnmcollier Thanks for reviewing the PR, comments addressed, please review again.

@johnmcollier Thanks for reviewing the PR, comments addressed, please review again.


"except" instead of "expect".

[Enter feedback here]
ABout mid-page, I believe the sentence "3. After restriction, inherited members are removed from the roles expect for collection admin." should contain "except" instead of "expect".

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

ID: 0e51709b-247a-7456-e97f-3cd713872c52
Version Independent ID: 738068ad-2f1c-abba-5189-1ae03736f22e
Content: How to create and manage collections - Microsoft Purview
Content Source: articles/purview/how-to-create-and-manage-collections.md
Service: purview
Sub-service: purview-data-map
GitHub Login: @viseshag
Microsoft Alias: viseshag

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

Hi @mturcottemsft
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.

@mturcottemsft
We are going to close this thread as resolved, Changes reflected in document soon but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Add additional logs for Integration Tests

Signed-off-by: sruti1312<EMAIL_ADDRESS>Is your feature request related to a problem? Please provide an existing Issue # , or describe.
https://github.com/opensearch-project/performance-analyzer/issues/170
Describe the solution you are proposing
Add additional logs for Integration Tests
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #182 (bd3e53d) into main (340f11e) will decrease coverage by 0.08%.
The diff coverage is n/a.

:exclamation: Current head bd3e53d differs from pull request most recent head 81fd4e3. Consider uploading reports for the commit 81fd4e3 to get more accurate results

@@             Coverage Diff              @@
##               main     #182      +/-   ##
============================================
- Coverage     71.77%   71.69%   -0.09%
  Complexity      356      356
============================================
  Files            44       44
  Lines          2459     2459
  Branches        160      160
============================================
- Hits           1765     1763       -2
- Misses          588      589       +1
- Partials        106      107       +1

Impacted Files
Coverage Δ

...ormanceanalyzer/writer/EventLogQueueProcessor.java
55.95% <0.00%> (-4.77%)
:arrow_down:

...analyzer/collectors/MasterServiceEventMetrics.java
73.64% <0.00%> (+1.35%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 340f11e...81fd4e3. Read the comment docs.


Add ignore_certs constructor option (or make it available)

Add ignore_certs constructor option (or make it available) like wsk -i). Currently I get an error during deploy of the boilerplate.
Failed to deploy function (localess-dev-hello_world)
due to error: Error encountered calling OpenWhisk: Error:
self signed certificate

Thanks for reporting this. I will add this in.

HI James,
Thanks it would be great. I already found some other engineers reporting the same issue at the Serverless forum](http://forum.serverless.com). I will communicate this coming fix within the forum. If you need some testing you can contact me.

Thanks for the heads-up, I hadn't been monitoring the forum.

@avwsolutions I've added the code for this, it'll be released in the upcoming 0.5 version.
I'll close this ticket when it's available.


Figure out way to remove this print

Setting `pad_token_id` to `eos_token_id`:50256 for open-end generation.

Happens in during sequence generation in either tokenizer.encode or model.generate

Addressed in #43


the center(right\left) button of the picture does not work  !

the center(right\left) button of the picture does not work unless you change the picture size before clicking the center button

I have got the same issue, have you ever found out how to fix it or do we have to live with it?

Is there any way to trigger an indirect trigger?


add db console synonyms to search

Added synonyms for Admin UI (rebranded to DB Console).

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4ac9650b63dcf6b0994b76da0e3a9c3c57fe25ba/


Implement Final logo screen

The screen will have fade in and fade out, must work correctly and it will be placed after unity logo screen.
We probably should create a few builds to test the screen.
EXPECTED RESULT:
The expected result is LogoScreen.scene on Scenes folder, and it has to be implemented in unity and will fade in and fade out.
Logo screen:

The issue was completed on the following commit: ceb0bcd3e3fd4581ce4016c9781ff301be006a4b