[macOS] Application's "Quit" keyboard shortcut does not work

Command-Q shortcut does not work, it displays a message in status bar: "(Cmd-@) was pressed. Waiting for second key of chord." After pressing it again it display a second message: "The key combination (Cmd-Q, Cmd-Q) is not a command."

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:18:32.269Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author
Version

vscode-markdownlint
DavidAnson
0.7.2

vscode-devskim
MS-DevSkim
0.1.2

material-icon-theme
PKief
2.0.5

vscode-caniuse
akamud
0.5.3

vscode-svgviewer
cssho
1.4.0

phpunit
emallin
1.0.0

json-tools
eriklynd
1.0.2

php-debug
felixfbecker
1.10.0

php-intellisense
felixfbecker
1.2.1

live-html-previewer
hdg
0.0.1

php-symbols
linyang95
2.1.0

vscode-todo-parser
minhthai
1.8.9

debugger-for-chrome
msjsdiag
3.1.1

Ruby
rebornix
0.12.1

vsc-nwjs
ruakr
1.0.5

Align
steve8708
0.2.0

html-preview-vscode
tht13
0.1.1

Steps to Reproduce:

Press Command-Q.
Press again the shortcut.

Command-Q shortcut does not work, it displays a message in status bar: "(Cmd-@) was pressed

@IDDemon are you using a special keyboard layout? If yes, can you please help us to track down the issue following this guide https://github.com/Microsoft/vscode/wiki/Keybindings

I'm using the standard hungarian (quertz) keyboard layout that comes with MacOS and never altered any of the keyboard shortcuts of VSC. I think it should be the same keycode for Q as on US keyboard layout.
Another detail that i didn't notice until now: the shortcut stops working only when the editor has the focus and caret inside it. When i click the titlebar of window, the editor loses focus and caret disappears, and everything fine again. But when i go back to editor and caret reappears, it stops working again. Strange.

@IDDemon Extensions can contribute keybindings that can overwrite built in keybindings in certain contexts.
Can you reproduce the issue when running code --disable-extensions ?
Which commands are bound to the desired key combination?

@alexandrudima Yep. It was the Live HTML Previewer extension. I've never tought that anyone (extension developer) wants  to use the OS default shortcuts for own purposes. It's so nonsense to me. Sorry, my bad. :)
Thank you!

@IDDemon Happy to help. Perhaps you can create an issue on the HTML Previewer extension to let them know to avoid using cmd+q on OSX.


Outdated cask: safe-launcher

Outdated cask: safe-launcher.
Info:

version: 0.8.1.
appcast url: https://github.com/maidsafe/safe_launcher/releases.atom.

Please do not run find_outdated_appcasts unless you are a maintainer.


Preload test result summaries and params when getting list of builds

[x] I've added a new note in the CHANGELOG.md file, according to docs:
https://iver-wharf.github.io/#/development/changelogs/writing-changelogs

Summary
Added preloading of test result summaries and params when getting a list of builds.
Motivation
Required to be able to show the test result list summary (currently generated from test result summaries when getting) in the builds list view in iver-wharf/wharf-web/pull/53 without making an extra request per build.
Params preloaded as well to match getting a single build, and so it can be wrapped in a function.
Alternative solution
Keep the preloading, but empty the TestResultSummaries field after generating the TestResultListSummary to save bandwidth when there are many tests. Thoughts?
Notes
I feel this is more of a bugfix, as I believe the idea was to have testResultListSummary be available especially when fetching multiple builds. Hence patch-version bump.

Keep the preloading, but empty the TestResultSummaries field after generating the TestResultListSummary to save bandwidth when there are many tests. Thoughts?

Sounds like future improvements if we notice it's an issue. "Premature optimizations=evil" and all that.
But to theorize: The TestResultSummaries model only correlates to a single test result file, yes? And the biggest use case we have right now has about 10 files uploaded per build, and the builds are paginated by default at a chunk of 10 builds. So worst case we download 100 test result summaries from the database on each build listing. And then the TestResultSummaries model only includes mostly integers (as you're not preloading their associated artifact or build metadata) and a string for the file name, so we're talking <1k bytes or something. Not that heavy on neither the bandwidth, the database, nor the wharf-api.
For now, let's talk optimizations when this becomes an issue. Such as if a user wants to download all builds, then we could have a query parameter for omitting the test result summaries or something. But again, let's consider that later.


20240311 sl document review

Readme.md review

When creating an acronym for a plural, see if there's a way to pluralise the acronym, e.g. Financial Service Providers (FSPs).

Modern grammatical style is to append an "s" to an acronym for a plural without an apostrophe, e.g. FSPs and not FSP's, which would denote possession by the FSP instead.

There is some confusion in our use of DFPS vs FSP. Initially, we borrowed the DFSP (Digital Financial Service Provider) from Mojaloop since they were largely and almost exclusively focused on the Digital part of the ecosystem. We have since decided to remove the "Digital" and the "D" because we felt our focus was more universal.

Should all references to DFSP be changed to FSP?

Should all references to DFSP be changed to FSP?

Yes, eventually. We'll update as we go.


Cannot estimate gas on flash loan

I have the following script to request a USD flash loan on Polygon blockchain:
import * as api from "@protocolink/api";
import * as common from "@protocolink/common";
import ethers from "ethers";
import dotenv from "dotenv";

// Config
dotenv.config();
let chainId = common.ChainId.polygon;
let loanToken = "USDC";
let amount = 1;
let network = "matic";
let account = process.env.ACCOUNT;
let mnemonic = process.env.MNEMONIC;

try {
  // Setup wallet

  let wallet = ethers.Wallet.fromMnemonic(mnemonic);
  let provider = new ethers.providers.InfuraProvider(network, process.env.INFURA_KEY);
  let signer = wallet.connect(provider);

  console.log("Start bot");

  // Get token list for loan
  const tokenList = await api.protocols.aavev3.getFlashLoanTokenList(chainId);
  let underlyingToken;
  for (let token of tokenList) {
    if (token.symbol == loanToken) {
      underlyingToken = token;
    }
  }

  const loans = [
    {
      token: underlyingToken,
      amount: amount.toString(),
    },
  ];

  // Create loan
  const flashLoanQuotation = await api.protocols.aavev3.getFlashLoanQuotation(chainId, {
    loans,
  });

  let i = underlyingToken.address;
  console.log(
    "Amount to repay: ",
    flashLoanQuotation.repays.tokenAmountMap[i].amount + " " + flashLoanQuotation.repays.tokenAmountMap[i].token.symbol
  );

  const [flashLoanLoanLogic, flashLoanRepayLogic] = api.protocols.aavev3.newFlashLoanLogicPair(loans);
  const logics = [flashLoanLoanLogic, flashLoanRepayLogic];

  // Build router data
  const routerData: api.RouterData = {
    chainId,
    account: account,
    logics: logics,
  };
  let estimateResult = await api.estimateRouterData(routerData, { permit2Type: "permit" });

  console.log(
    "FUNDS: ",
    estimateResult.funds.tokenAmountMap[i].amount + " " + estimateResult.funds.tokenAmountMap[i].token.symbol
  );
  console.log("FEE 1: ", estimateResult.fees[0].feeAmount.amount + " " + estimateResult.fees[0].feeAmount.token.symbol);
  console.log("FEE 2: ", estimateResult.fees[1].feeAmount.amount + " " + estimateResult.fees[0].feeAmount.token.symbol);

  // Increase fee
  estimateResult.fees[0].feeAmount.amount = (Number(estimateResult.fees[0].feeAmount.amount) * 20).toString();
  estimateResult.fees[1].feeAmount.amount = (Number(estimateResult.fees[1].feeAmount.amount) * 20).toString();

  // Approve
  for (const approval of estimateResult.approvals) {
    console.log("Approve ...");
    const txApprove = await signer.sendTransaction(approval);
    txApprove.wait();
    console.log("Gas limit: ", Number(txApprove.gasLimit));
  }

  // Permit
  if (estimateResult.permitData) {
    console.log("Sign ...");
    const permitSig = await signer._signTypedData(
      estimateResult.permitData.domain,
      estimateResult.permitData.types,
      estimateResult.permitData.values
    );
    // console.log("PermitSig", permitSig);
    routerData.permitData = estimateResult.permitData;
    routerData.permitSig = permitSig;
  } else {
    console.log("NOTHING TO SIGN");
  }

  // Execute
  console.log("Execute ...");
  const transactionRequest = await api.buildRouterTransactionRequest(routerData);

  console.log("Send ...");
  const tx = await signer.sendTransaction(transactionRequest);
  console.log(tx);
} catch (e) {
  console.error("ERROR: ", e.reason || e);
}

This is the output log:
Start bot
Amount to repay:  1.0005 USDC
FUNDS:  0.0005 USDC
FEE 1:  0.000667898340426478 MATIC
FEE 2:  0.000001335796680852 MATIC
Approve ...
Gas limit:  56107
Sign ...
Set fees
Execute ...
Send ...
ERROR:  cannot estimate gas; transaction may fail or may require manual gas limit

Why is it trowing this error on gas fees?

Also with the balancer flash loan I get the 2 objects in approval and the permitData in the estimateResult. Is it normal?
I keep getting the same error.
How can I set a manual gas limit in protocolink?

This is unusual. Did you implement newFlashLoanLogicPair similar to the example? If so, your routerData should look like this:"

{
  "chainId": 137,
  "account": "0xaAaAaAaaAaAaAaaAaAAAAAAAAaaaAaAaAaaAaaAa",
  "logics": [
    {
      "rid": "balancer-v2:flash-loan",
      "fields": {
        "id": "6f6cc4d7-001f-4be8-a444-3f98abb9ba24",
        "loans": [
          {
            "token": {
              "chainId": 137,
              "address": "0x0d500B1d8E8eF31E21C99d1Db9A6444d3ADf1270",
              "decimals": 18,
              "symbol": "WMATIC",
              "name": "Wrapped Matic"
            },
            "amount": "10000"
          }
        ],
        "isLoan": true
      }
    },
    {
      "rid": "balancer-v2:flash-loan",
      "fields": {
        "id": "6f6cc4d7-001f-4be8-a444-3f98abb9ba24",
        "loans": [
          {
            "token": {
              "chainId": 137,
              "address": "0x0d500B1d8E8eF31E21C99d1Db9A6444d3ADf1270",
              "decimals": 18,
              "symbol": "WMATIC",
              "name": "Wrapped Matic"
            },
            "amount": "10000"
          }
        ],
        "isLoan": false
      }
    }
  ]
}

Ethers.js’s signer.sendTransaction() allows to set both gasLimit and gasPrice.

The txApprove.gasLimit is typically insufficient for a flash loan. Consider using Ethers.js’s estimateGas for a more accurate gas estimate.
Additionally, simulate the transaction using tools like Tenderly to ensure it is ready for execution. You can then share the simulation results with us for more effective debugging.
Also, note that the nonce is set to 100, but it appears that the nonce for 0x53e09Dd70E20a0b5CeaF2447484063dcCbE3f219 on Polygon is not as high.

Could you clarify hot to get the gasLimit with Ethers.js's estimateGas?

I see I can get it with provider.getGasPrice() or provider.getFeeData() but both don't works.
Then I tried with the contract.estimateGas but I miss some info, here what I tried:
  // Is this correct? I took it from the transactionRequest
  let protolinkContractAddress="0xDec80E988F4baF43be69c13711453013c212feA8";
  let contract = new ethers.Contract(protolinkContractAddress, ABIprotolink, provider);
  // here I miss the parameters to execute this function
  let estimategas = await contract.estimateGas.executeWithSignerFee();

If this is a standard procedure to get the gas limit it would be helpful to add this to the docs.

How do I set the nonce? It seems retrieved from api.buildRouterTransactionRequest
Here is the simulation results of Tenderly:

This Tenderly simulation indicates that 0x53e09dd70e20a0b5ceaf2447484063dccbe3f219 has not yet approved USDC for its agent 0x6E544A364fb3412E25809aA882EfD5b5d5ea36D1 (calcAgent).
If you are unfamiliar with how the Uniswap off-chain permit2 sign works, I recommend using permit2Type: approve initially (see documentation). First, execute the two approvals obtained from await api.estimateRouterData(routerData, { permit2Type: 'approve'}) on-chain. Afterwards, use Ethers.js’s estimateGas to estimate the gas limit and verify whether the transaction reverts.

@lucarducci Is there anything else we can help you with? And feel free to join our Discord we're here and happy to help!

So are flash loans working? Using furucombo I noticed it just fails the transactions.
Anyone know of alternative that works or have code that can be forked?


Helm template Standardization

Is your feature request related to a problem? Please describe.
There are a few non-standard parameters in the current helm chart as well as a few parameters that would be desired by users
(being able to set a more specific image for instance).
Describe the solution you'd like
Modify some of the current helm chart parameters to align more with standard sets of parameters as well as modify
the templates slightly to allow users greater control in specifying things, while making their default values follow
what is currently hard coded.
Additional context
I know this is still in alpha and am willing to take it on myself if the current dev team is open to some changes.

Hey @jtcarnes, we have now a chart on a single location: here
We're open to any suggestions or PR's :)


Specular Textures [BlueEvil's Addon]

Specular getting set to 0.5 even tho it meant to be 0 when i prep the world with blueevil's addon.

principled
Go to the addon and add under that line in the meantime.
4.0+

bsdf_node.inputs["Specular IOR Level"].default_value = 0
3.6
bsdf_node.inputs["Specular"].default_value = 0

Specular getting set to 0.5 even tho it meant to be 0 when i prep the world with blueevil's addon.

That's the default for Principled BSDF

MiEx now has a new Blender addon that fixes this issue. The new Blender addon imports in the materials exported out from MiEx, which themselves are generated from the material templates. The repository also contains an example resource pack specifically for Blender which has specular turned off by default.
You can read more about the new Blender addon here: https://github.com/BramStoutProductions/MiEx/wiki/8.-Importing-into-DCCs#blender


UnicodeDecodeError: 'charmap' codec can't decode byte 0x81

I'm trying to read a database produced by an ancient version of the ACDsee photo manager program (don't ask).
When I try to read it simply as:
table = DBF('asset.dbf')
for record in table:
    print(record)

I get ValueError: Unknown field type: '7'.
I followed the advice in another issue and created a field parser as:
class TestFieldParser(FieldParser):
    def parse7(self, field, data):
        return data

table = DBF('asset.dbf', parserclass=TestFieldParser)
for record in table:
    print(record)

This produces the stack trace below. Googling for the error suggests that maybe the file is being read with the wrong encoding. Is there an easy way to try reading e.g. with UTF-8?
Traceback (most recent call last):
  File "/mnt/acdsee/ACDsee/./dumpdb.py", line 10, in <module>
    for record in table:
  File "/mnt/acdsee/ACDsee/venv/lib/python3.9/site-packages/dbfread/dbf.py", line 314, in _iter_records
    items = [(field.name,
  File "/mnt/acdsee/ACDsee/venv/lib/python3.9/site-packages/dbfread/dbf.py", line 315, in <listcomp>
    parse(field, read(field.length))) \
  File "/mnt/acdsee/ACDsee/venv/lib/python3.9/site-packages/dbfread/field_parser.py", line 79, in parse
    return func(field, data)
  File "/mnt/acdsee/ACDsee/venv/lib/python3.9/site-packages/dbfread/field_parser.py", line 87, in parseC
    return self.decode_text(data.rstrip(b'\0 '))
  File "/mnt/acdsee/ACDsee/venv/lib/python3.9/site-packages/dbfread/field_parser.py", line 45, in decode_text
    return decode_text(text, self.encoding, errors=self.char_decode_errors)
  File<EMAIL_ADDRESS>line 15, in decode
    return codecs.charmap_decode(input,errors,decoding_table)
UnicodeDecodeError: 'charmap' codec can't decode byte 0x81 in position 35: character maps to <undefined>

I'm not sure if I have an answer for you but have you tried specifying the encoding?
If you know what encoding you have, you might be able to get it to work this way. The DBF() constructor's optional second argument is for an encoding:
table = DBF('asset.dbf', 'UTF-8')
for record in table:
    print(record)

table = DBF('asset.dbf', 'Latin-1')
for record in table:
    print(record)

Another option is to set the char_decode_errors handler. The argument defaults to 'strict' when unspecified.
So this...
table = DBF('asset.dbf')

Is the same as...
table = DBF('asset.dbf', char_decode_errors='strict')

But you could relax this requirement by specifying a more forgiving error handler (see Python's Error Handlers docs for more options):
table = DBF('asset.dbf', char_decode_errors='replace')

You might settle on some combination of the two... defining an expected encoding and loosening the error handling behavior:
table = DBF('asset.dbf', 'Latin-1', char_decode_errors='replace')
for record in table:
    print(record)


Breadcrumb that is not a real breadcrumb

Context 📋
In the page-header,
I want to add buttons like Create new,

and still have the breadcrumb: Items > Actions.

Current state 👀
Page header buttons are at the position of Breadcrumb.
https://github.com/kevinpapst/TablerBundle/blob/014debb4a80785a99d070259b62cc34d4be2ad90/templates/layout.html.twig#L96-L109
Problem ❌
With that current state,

If I want to add my buttons, I'll put them INSIDE breadcrumb block with custom HTML to allow to have Buttons + Breadcrumb.
Plus, we are declaring a block about Breadcrumb with buttons on default values. In that case, it's not really a "Breadcrumb".

Possible solution ✔
Split current {% block breadcrumb %} into 3 blocks
{% block page_header_right %}

        {% block page_header_actions %}

                    New view

                    Create new report

        {% endblock %}
        {% block breadcrumb_container %}

                {% block breadcrumb %}
                    Entity > Action > ...
                {% endblock %}

        {% endblock %}

{% endblock %}

Other solution could be to use page_stats as breadcrumb.
And rename breadcrumb into page_header_actions

Hm, I would always try to keep the menu level as flat as possible.
And then you could switch page title and pre title and add a link to the pre title - in your example: Articles would be the pre_title with a link and Create article would be the title.
The other solution I would prefer are be the 3 blocks.
But I would keep the breadcrumb on the left side (start for RTL) and followed by the buttons on right side (end for RTL).

Hm, I would always try to keep the menu level as flat as possible.

Ok

And then you could switch page title and pre title and add a link to the pre title - in your example: Articles would be the pre_title with a link and Create article would be the title.

Ok that is about #19

The other solution I would prefer are be the 3 blocks.

Ok

keep the breadcrumb on the left side

followed by the buttons on right side

Left side of the header buttons ?

@kevinpapst up 😉

What is missing? You mean that:

Left side of the header buttons ? Like that ?

Then I would say yes, just make it "start" and not "left". Quote from bootstrap docs:

For example, instead of .ml-3 for margin-left, use .ms-3.

Yeah,I was waiting an answer about the breadcrumb position VS buttons position.
I was OK with the 3 blocks, but I don' know on which order you want the breadcrumb.

Position
Demo

Top

Bottom

Left

Right

In Left mode, Breadcrumb will move depending on the action buttons.
I don't know if it is good for the end user (not dev) to move the breadcrumb on different cases.
Breadcrumb is a landmark/reference, If we change his position it could be messing up with user.
In Right mode, buttons will move depending of the Breadcrumb.

@kevinpapst IMO Top mode is the better choice

Hmmmm, I don' really like any of these. Too many clickable elements next to each other.
Why not replacing "subtitle" with the breadcrumb? The information is partially duplicated there anyways.
I find the top position the least intrusive one (with a little bit of added bottom margin).

I don' really like any of these

What about the page_stats then ? https://github.com/kevinpapst/TablerBundle/issues/21#issuecomment-993451128

I would prefer if the original Tabler bundle had a showcase for the breadcrumb... so I posted the question upstream.
Honestly: for now I would prefer if you would add the breadcrumb in your custom layout. Then you can use any block that you want. Overwrite page_stats or any other block, add the breadcrumb and call {{ parent() }}.
And then, after collecting some user feedback => share it in the upstream discussion. Maybe we can get an example in the official demo.

Waiting for an answer from Tabler then. 👍

add the breadcrumb in your custom layout

I'm OK with that. But breadcrumb is already taken, and it will contain buttons..
https://github.com/kevinpapst/TablerBundle/blob/058fd31189cfcc1dc249a5fd0f359aed13f1ccbd/templates/layout.html.twig#L97-L106
I can manage to create an page_header_actions block INSIDE the current breadcrumb block.
And create a new block breadcrumb_custom INSIDE page_stats.
Bit tricky, but should do the job.

Then let's rename the breadcrumb block to page_actions? And add it to the migration guide.

Have you seen the answers?

Yes,
It will update #19 again 😄
How can we place subtitle block and breadcrumb block in the template.
According to their answer, there are at the same place.
Not that simple with block template pre-created 🤔

Why? Can't you simply put {% block breadcrumb %} into {% block pretitle %}?

ATM we have to place.

page_pretitle
page_subtitle (from AdminLTE)
page_title
page_stats
breadcrumb (also used in AdminLTE)

It can be like:
{% block page_pretitle%}{% block breadcrumb %}{% endblock %}{% endblock %}
{% block page_title%}{% endblock %}
{% block page_stats %}{% block page_subtitle %}{% endblock %}{% endblock %}

Merged in https://github.com/kevinpapst/TablerBundle/pull/48


We need LM STUDIO Support

I believe LM Studio would be a great addition to OpenDevin. I feel this is a gray area currently which I'm sure is being worked on in the background. Hopefully some good documentation on how to get it working will come out with the release of the next updates. It would be nice to be able to utilize more powerful LLMs on smaller machines.

This may be a duplicate of documenting LM studio support: https://github.com/OpenDevin/OpenDevin/issues/419

Going to close this as a duplicate of the issue I mentioned above which has a lot of discussion on it already.


idea: vocabulary of different kind of timelines

We've been collecting on some timelines the draw_timeline_dir which has enabled some discussion around different forms of layer files in the wild. We could start to collect, name and document these, as it would help to make sure we have tests for such timelines.
This would of course require the draw_timeline_dir to be easy to use (#3693, neondatabase/cloud#4041).
Links
Internal slack threads on #storage--image-heuristics.
Related: #3602, #3667
Semi-requires: #3693

No interest in this.


glide.yaml: Update to terraform v0.9.2 and vendor

Using terraform 0.9.2 you'll see,
Incompatible API version with plugin. Plugin version: 2, Ours: 4

Update vendored terraform dependency to v0.9.2 fixes this
Fix the revendor so project compiles and matches CoreOS projects

Sure. @bison this works for you? Have you updated to v0.9.2 yet? I use my fork for all my own infrastructure and its been alright.

I must have missed this, sorry. Anyway, this is fine with me. :+1:


Move flutter_gallery...transition_perf family of benchmarks to staging from prod

The family of flutter_gallery...transition_perf has been flaky for a while, and this tracks the process to migrate them from prod to staging.
The set of tests:
Linux_android flutter_gallery__transition_perf_e2e
Linux_android flutter_gallery_sksl_warmup__transition_perf_e2e
Mac_ios flutter_gallery__transition_perf_e2e_ios
Mac_ios flutter_gallery_ios__transition_perf
Mac_ios flutter_gallery_ios_sksl_warmup__transition_perf
Linux_android flutter_gallery__transition_perf_with_semantics
Linux_android flutter_gallery__transition_perf
Linux_android flutter_gallery__transition_perf_hybrid
Linux_android flutter_gallery_sksl_warmup__transition_perf
Mac_ios32 flutter_gallery__transition_perf_e2e_ios32 (will leave it in prod due to capacity limitation).

To keep track of these tests benchmarks, we need to support uploading metrics to Skia for them.
More context: https://github.com/flutter/flutter/issues/83298

Since we have been running ios32 tests as flaky all the time, shall we move them to staging as well? @zanderso @jmagman

/cc @chinmaygarde

Not sure that the new_gallery... ones need to be in there. Have they been flaky?

Not sure that the new_gallery... ones need to be in there. Have they been flaky?

Confirmed they are not. Updated to skip them.

The updated list looks good to me too.

Since we have been running ios32 tests as flaky all the time, shall we move them to staging as well?

Does anyone ever look at failing staging tests?

will those tests still be in the ci.yaml? Do we want to make the automation script to ignore these target?

Another part of this request is providing a dashboard so that we can monitor the status of things running in staging.

Does anyone ever look at failing staging tests?

Infra team regularly look at the staging status, especially when infra changes land.
But as for test issues/flakes, unless looking here intensionally for specific tests.
Another reason we may want to migrate ios32 to staging is to avoid affecting metrics/SLOs, like the flake ones. As of now, we track flaky metrics including those tests marked as flaky. We may want ios32 ones to affect that.
Please let me know if the staging pool data, together with the Skia dashboard benchmarks are good enough. If not, we may leave the ios32 ones in prod as flaky for now.

will those tests still be in the ci.yaml? Do we want to make the automation script to ignore these targets?

.ci.yaml is not supporting staging configs yet. I will remove those when ready, so no need changes from the automation script for now.

Another reason we may want to migrate ios32 to staging is to avoid affecting metrics/SLOs, like the flake ones.

This issue is only about moving the flutter_gallery...transition_perf family of benchmarks. Let's not conflate it with ios32 (unless the ios32 tests are part of the flutter_gallery...transition_perf family).

Another reason we may want to migrate ios32 to staging is to avoid affecting metrics/SLOs, like the flake ones.

This issue is only about moving the flutter_gallery...transition_perf family of benchmarks. Let's not conflate it with ios32 (unless the ios32 tests are part of the flutter_gallery...transition_perf family).

Mac_ios32 flutter_gallery__transition_perf_e2e_ios32 belongs to this family, but we have another one Mac_ios32 native_ui_tests_ios32 doesn't. Since we have only one ios32 bot/device, we have to keep them in a single pool as of now.
If no strong opinion, I will just leave both ios32 in prod for now. (updated the description).

Couple of things to implement:

[ ] Support upload staging metrics to Skia from cocoon update-task-status API.
[ ] Upload staging metrics to Skia from recipe devicelab_drone.py.
[ ] Enable tests in staging with upload_metrics:true
[ ] Remove tests from .ci.yaml

Let's also include the web variants of this test. transition_perf is the flakiest test in the web_long_running_tests shard.

All flutter_gallery...transistion_perf tests in devicelab are running in staging not blocking the tree but keep uploading benchmarks.
Leave this issue open to keep track of ones in web_long_running_tests shards.

Decreasing priority to P3 as the devicelab family has been migrated to staging.

Closing because the bug has not been updated for more than one year.
Feel free to reopen any bugs that you believe are important but please provide updates on how to resource the fix of the bug and what can be done to prevent the issue from not getting updates for another year.


Use the default settings for Logging{Client,Service}'s default fact…

…ory methods
Motivation:
So far, Logging{Client,Service}.newDecorator() had different settings
than Logging{Client,Service}.builder().newDecorator() due to backward
compatibility. It's time to change this in favor of consistency.
Modifications:

Change Logging{Client,Service}.newDecorator() to use its builder's
default settings.

TRACE level logging for successful requests and responses.
WARN level logging for failed requests and responses`.

Result:

Consistency
Fixes #2696

By the way, is TRACE level the right default for successful requests and responses? I was rather expecting DEBUG. Do you remember why we lowered to TRACE? /cc @imasahiro @kojilin @anuraaga

I don't remember but DEBUG is fine with me.

Actually, I'm leaning towards INFO.😆 Because users can easily turn it off using logging framewrok xml configuration.

Related work: The default log level of Spring's CommonsRequestLoggingFilter is DEBUG.
https://github.com/spring-projects/spring-framework/blob/master/spring-web/src/main/java/org/springframework/web/filter/CommonsRequestLoggingFilter.java#L39

Let me change the default level from TRACE to DEBUG tomorrow. Please feel free to let me know if you have any concern.


Change the default cilium configMap value

If we want L7 Cilium Network Policies to use the sidecar proxy when
istio is present, sidecar-istio-proxy-image: must point to istio-proxy
instead of cilium/istio_proxy because our proxy image name is
istio-proxy
https://github.com/cilium/cilium/blob/master/pkg/k8s/labels.go#L71-L78
https://github.com/cilium/cilium/blob/master/pkg/k8s/labels.go#L46
Signed-off-by: Manuel Buil<EMAIL_ADDRESS>(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

Shouldn't this name be cilium/istio_proxy as referred by https://github.com/cilium/cilium/blob/master/pkg/k8s/labels.go#L46

That's the default value if nothing is provided as a flag of cilium-agent or in the ConfigMap. By changing the configMap, we overwrite that value

Q: without bumping the manifest version, skuba addon upgrade apply will not migrate an existing configMap with this change only, is that fine?

Q: without bumping the manifest version, skuba addon upgrade apply will not migrate an existing configMap with this change only, is that fine?

Istio has not been released in any stable version of CaaSP so far. So ideally, this will still be the initial version. Should we still be concerned about upgrade ?

Istio has not been released in any stable version of CaaSP so far. So ideally, this will still be the initial version. Should we still be concerned about upgrade ?

I see, not necessarily, but when the time comes we will have to remember to bump the manifest version so it gets applied on upgrades -- there's room to forget about it. If we don't forget it shouldn't be a problem though.


Conda badge

Description
Since there is a conda package of TeachOpenCADD :)
Taken from here https://anaconda.org/conda-forge/teachopencadd/badges
Todos

[x] Add conda badge to readme.

Status

[x] Ready to go

@t-kimber thanks a lot, this PR is ready to be merged!


where is the create_model,create_dataloader, create_dataset

Hi,thanks for your great work.In the train.py,where is the create_model,create_dataloader, create_dataset?

from data import create_dataloader, create_dataset
from models import create_model

They are in the data or models folder, __init__.py

Oh,I see.Thanks a lot.


1823825 | Message fixes | Address validation issues | 17

This PR is for user story 1823825.
Fixed an issue with missing table headers.
If these changes are not appropriate, please let me know. I'm only trying to clean up validation issues in this repo. So I can easily close this PR or modify it in whatever way is best for the content.

Docs Build status updates of commit 4517751:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/message-type-is-not-allowed-as-part-of-the-agreement.md
:white_check_mark:Succeeded
View

biztalk/core/message-types-without-parts-are-not-supported.md
:white_check_mark:Succeeded
View

biztalk/core/metadata-errors.md
:white_check_mark:Succeeded
View

biztalk/core/missing-or-invalid-or-duplicate-transaction-set-identifier.md
:white_check_mark:Succeeded
View

biztalk/core/missing-or-invalid-transaction-set-control-number.md
:white_check_mark:Succeeded
View

biztalk/core/missing-or-invalid-transaction-set-identifier.md
:white_check_mark:Succeeded
View

biztalk/core/mqscorrelationsetorchestrationwithsolicitresponse-biztalk-server-sample.md
:white_check_mark:Succeeded
View

biztalk/core/no-matching-transform-found-for-doctype-in-send-port.md
:white_check_mark:Succeeded
View

biztalk/core/number-of-included-groups-do-not-match.md
:white_check_mark:Succeeded
View

biztalk/core/number-of-included-segments-do-not-match.md
:white_check_mark:Succeeded
View

biztalk/core/number-of-included-transaction-sets-do-not-match.md
:white_check_mark:Succeeded
View

biztalk/core/one-or-more-segments-in-error.md
:white_check_mark:Succeeded
View

biztalk/core/orchestration-variable-types.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit cac052c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/message-type-is-not-allowed-as-part-of-the-agreement.md
:white_check_mark:Succeeded
View

biztalk/core/message-types-without-parts-are-not-supported.md
:white_check_mark:Succeeded
View

biztalk/core/metadata-errors.md
:white_check_mark:Succeeded
View

biztalk/core/missing-or-invalid-or-duplicate-transaction-set-identifier.md
:white_check_mark:Succeeded
View

biztalk/core/missing-or-invalid-transaction-set-control-number.md
:white_check_mark:Succeeded
View

biztalk/core/missing-or-invalid-transaction-set-identifier.md
:white_check_mark:Succeeded
View

biztalk/core/mqscorrelationsetorchestrationwithsolicitresponse-biztalk-server-sample.md
:white_check_mark:Succeeded
View

biztalk/core/no-matching-transform-found-for-doctype-in-send-port.md
:white_check_mark:Succeeded
View

biztalk/core/number-of-included-groups-do-not-match.md
:white_check_mark:Succeeded
View

biztalk/core/number-of-included-segments-do-not-match.md
:white_check_mark:Succeeded
View

biztalk/core/number-of-included-transaction-sets-do-not-match.md
:white_check_mark:Succeeded
View

biztalk/core/one-or-more-segments-in-error.md
:white_check_mark:Succeeded
View

biztalk/core/orchestration-variable-types.md
:white_check_mark:Succeeded
View

biztalk/core/outboundcustomheaders-does-not-have-correct-format.md
:white_check_mark:Succeeded
View

biztalk/core/port-configuration-errors.md
:white_check_mark:Succeeded
View

biztalk/core/query-not-allowed-in-target-namespace.md
:white_check_mark:Succeeded
View

biztalk/core/reading-batch-descriptions-from-database-failed.md
:white_check_mark:Succeeded
View

biztalk/core/receipt-delivery-option-value-is-invalid.md
:white_check_mark:Succeeded
View

biztalk/core/receive-location-for-address-not-found.md
:white_check_mark:Succeeded
View

biztalk/core/receive-location-for-metadata-not-found.md
:white_check_mark:Succeeded
View

biztalk/core/registry-errors.md
:white_check_mark:Succeeded
View

biztalk/core/repeats-less-than-required.md
:white_check_mark:Succeeded
View

biztalk/core/repeats-more-than-required.md
:white_check_mark:Succeeded
View

biztalk/core/required-affiliate-application-name-not-specified.md
:white_check_mark:Succeeded
View

biztalk/core/required-client-certificate-thumbprint-not-specified.md
:white_check_mark:Succeeded
View

biztalk/core/required-property-binding-type-not-specified-r2.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@MandiOhlinger This PR is ready for merge.

@damabe I see you sneaking in some other improvements. Thank you!!


Cannot install Big Sur, supposedly incompatible

I tried installing Big Sur, due to Monterey not booting and when it did boot, it wouldn't detect any hard drives. When I select a disk to install to in Big Sur, I cannot select it because it says: This update cannot be installed on this computer. Thank you for any help.

Big Sur 11.3 or newer is requird so the board-id skip works.

Big Sur 11.3 or newer is requird so the board-id skip works.

I'm using the latest version.

Which EFI are you using?

Big Sur 11.3 or newer is requird so the board-id skip works.

I'm using the latest version.

Which EFI are you using?

Opencore one. I modified it with airportitwlm.

As I said: for the board-id skip to work, macOS Big Sur 11.3 has to be be INSTALLED and RUNNING already since it requires virtualization technology which is only present in 11.3 and newer. So you can't UPGRADE from macOS Catalina or older with the MacBookPro10,1 SMBIOS since it is not supported by Big Sur and newer. The skip only works when upgrading from Big Sur to Monterey and newer.
So you have to temporarily switch the SMBIOS for the installation/upgrade:

Disbale Wifi/LAN (flip the physical switch for the AirPlane mode on the left near the USB ports which disables all communication). Otherwise you have to generate new Serials, etc.
Change SystemProductName to MacBookPro11,4 - leave the rest as is.
Reboot. Install Big Sur.
Once macOS Big Sur is installed, change SystemProductName back to MacBookPro10,1
Enbale WifI/LAN again


Not working properly at android mobile chrome

I am facing an issue when I am using the website on mobile device android google chrome. The card number input, expiry date, CVV all input fields neither formatting the inputs not validating them allow to the user to input string also at the place of digits.

@devqualwebs
Care to share some code of how you have this setup so we could offer some assistance in getting it to work for you?

Just ran into this bug @devqualwebs.
I believe the problem lies in QJ. It is not properly binding the key events in android chrome. Any ideas on why that is @BallisticPain @jessepollak ?

Getting quite a few failed payments because of this intermittent error with Android devices 😢  @jessepollak
Here's the user agents that are getting it - essentially, they can't enter the slash "/", and then our payment fails from Expiry Validation error:
ANDROID USER AGENTS NOT WORKING
Mozilla/5.0 (Linux; Android 6.0.1; SAMSUNG SM-G930F Build/MMB29K) AppleWebKit/537.36 (KHTML, like Gecko) SamsungBrowser/4.0 Chrome/44.0.2403.133 Mobile Safari/537.36
Mozilla/5.0 (Linux; Android 6.0.1; SAMSUNG SM-T355Y Build/MMB29M) AppleWebKit/537.36 (KHTML, like Gecko) SamsungBrowser/4.0 Chrome/44.0.2403.133 Safari/537.36
Mozilla/5.0 (Linux; Android 6.0.1; SAMSUNG SM-N920I Build/MMB29K) AppleWebKit/537.36 (KHTML, like Gecko) SamsungBrowser/4.0 Chrome/44.0.2403.133 Mobile Safari/537.36
Mozilla/5.0 (Linux; Android 6.0.1; SAMSUNG SM-G900I Build/MMB29M) AppleWebKit/537.36 (KHTML, like Gecko) SamsungBrowser/4.0 Chrome/44.0.2403.133 Mobile Safari/537.36
CONFIRMED ANDROID AGENTS WORKING OK
Mozilla/5.0 (Linux; Android 4.4.2; Tab2A7-10F Build/KOT49H) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.132 Safari/537.36
Mozilla/5.0 (Linux; Android 6.0.1; SM-G900I Build/MMB29M) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.132 Mobile Safari/537.36
Also, I can confirm, when this occurs, there are no console errors occurring (can confirm that from client-side error reporting script), they are just unable to type in the expiry date fully on these devices.
Looks like this might be something to do with either the Chrome Version?

Related?
https://github.com/braintree/braintree-web/issues/245
Seems like it might be something to with the SamsungBrowser/4.0

The problem is keypress event is not fired on most of chrome android devices. Changing this to keydown doesn't work because these browsers send event.which = 229 (combined characters).
Changing this to keyup doesn't work too, because in this moment, the input already have the value and event.preventDefault() doesn't matter.
To change to keyup requires change all the lib's logic.

Any insights on this one @luishmcmoreno as to how it could be monkey patched or fixed otherwise? It's costing a lot of conversions for us with Android phones just because a simple "/" isn't being entered like it does normally.

Hey @nickooolas ... Try this https://gist.github.com/luishmcmoreno/45839565828ae3597816b98fa83bf754 . I've made some changes on the lib and it worked for me. I didn't all the necessary tests to create a pull request but to my needs, it worked.

@luishmcmoreno awesome work, thanks for the quick reply and patch on this one! :)

it worked to you @nickooolas ? The problem seems to be in the payments lib, not here.

it worked for me but expiry date input (length) not work so it able to take 1000 digit :)

@mostafa-saqr , you are right!
Any solution?

I think this is the same issue - when I type in a credit card number on my S7 Edge - Chrome 64 - after about the 5th digit, it repeats all 5 digits (so "45140" turns into "4514045140"). I can paste okay, just not type.

duplicate of #199 you can..

working on android devices with input type="tel" instead of input type="text"


Rewrite Interaction Manager

I think it becoming aparant that the interaction manager is due for an update. There are many bugs and feature requests that need to be taken care of:
https://github.com/pixijs/pixi.js/issues?q=is%3Aopen+is%3Aissue+label%3A"Interaction+Manager"
This issue is to track completion of a new interaction manager that fixes all (or at least most) of these issues.

I'm pretty busy with work right now, but I have opened a bounty on this issue. If we reach the goal then I will make time to complete this feature. You can post to the bounty on Bountysource.

Just in case it's of potential use for the InteractionManager incl. Pointer Events incl. "pointerup":
Perhaps one of these libs could be used / forked / built on / looked at?
https://github.com/Deltakosh/handjs
https://twitter.com/Inateno/status/641677865038315520
https://github.com/Rapid-Application-Development-JS/Pointer

Where's this work being done? Fork, feature branch, privately?

It isn't being done, we haven't reached the goal I set or gotten close enough for me to have the time to start :)

Also another thing we might need to add to this.
enabled=false
Our project has HTML interaction capabilities as well as PIXI interaction capabilities.
Sometimes I would like to disable PIXI interaction and re-enable it later.
The way I have implemented it is by simply adding a this.enabled=true field in InteractionManager and then checking for it in the update method as well as the processInteractive method.


refactor(admin): rename from proto/vmspb to proto/admpb

What this PR does
Renamed the package from proto/vmspb to proto/admpb.

Current dependencies on/for this PR:

main

PR #313   👈

PR #314

This comment was auto-generated by Graphite.

Codecov Report
Base: 63.21% // Head: 63.31% // Increases project coverage by +0.09% :tada:

Coverage data is based on head (eb3de97) compared to base (00c0c42).
Patch coverage: 52.06% of modified lines in pull request are covered.

:exclamation: Current head eb3de97 differs from pull request most recent head 9e7878a. Consider uploading reports for the commit 9e7878a to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #313      +/-   ##
==========================================
+ Coverage   63.21%   63.31%   +0.09%
==========================================
  Files         130      130
  Lines       17455    17455
==========================================
+ Hits        11035    11051      +16
+ Misses       5869     5854      -15
+ Partials      551      550       -1

Impacted Files
Coverage Δ

internal/admin/testing.go
44.68% <0.00%> (ø)

pkg/varlog/admin.go
6.19% <8.57%> (ø)

pkg/varlogtest/admin.go
42.03% <46.15%> (ø)

internal/admin/stats/repository.go
82.08% <57.14%> (ø)

internal/admin/server.go
67.44% <75.47%> (ø)

internal/admin/admin.go
61.82% <87.50%> (ø)

internal/admin/snmanager/manager.go
82.31% <100.00%> (ø)

internal/storagenode/storagenode.go
70.95% <0.00%> (-0.34%)
:arrow_down:

internal/metarepos/report_collector.go
77.42% <0.00%> (+0.42%)
:arrow_up:

internal/metarepos/storage.go
80.30% <0.00%> (+0.58%)
:arrow_up:

... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Evaluate and add ephemeral infrastructure to conceptual docs, blogs and videos

Evaluate doc sets, blogs and videos where there's an opportunity to highlight Pulumi's capabilities for the various use cases for ephemeral infrastructure, and ensure we have SEO optimization in place.

Hey @interurban I marked this as in-progress because of the added milestone but feel free to update!


Upgrade Boxcryptor.app to v2.1.467 (Build: 718)

Merged as 549f1208c955f700db7bba3c7a287ebb2f9598b4.


Creating multiple directories with multiple intermediate subdirectories

I think somewhere I read this feature exists but I can't seem to find it.
Allow creating multiple directories with multiple intermediate directories in one go.
For example, allow creating this structure in one go:
src
   main
      java
      resources
   test
      java
      resources

with a syntax like this:
src/{main,test}/{java,resources}

from the explorer.

This is a fair feature request, however we do not have it on our immediate plan. Thus assigning to backlog and we might add this in the future. Thanks


[Issue860] Addresses a translator issue with Mono and Project reading.

Microsoft.Builder.Evaluation.Project(string) overload has an issue in Mono
where it does not read the file at all.
The bug has been detailed in:
https://bugzilla.xamarin.com/show_bug.cgi?id=38224
As an approach that works on both Windows and Linux, the correct overload
for the project is chosen just by changing the call from:
Project(fileName)
to:
Project(XmlReader.Create(fileName))
Which is anyway the chosen overload when just the first form is chosen.
The issue in Bridge is triggered by
Bridge.Translator.ProjectReader.GetSourceFiles()
Called when Bridge builder is about to read the list of source files inside
the .csproj project file.
Current Mono version always returns an empty list of files without the fix
this commit introduces.
This issue has been introduced with #531, released on Bridge 1.10.0, and
reported on #860, which may be released to 1.11.0 or 1.10.5.

Exception details thrown when building Bridge.sln (with test projects enabled) in Windows:
9>C:\NET\BridgeNew\Bridge\Testing\tests\clientOne\Bridge.Builder.Console : error 2016-02-02T22: 35:56 Error: exception level: 0 - Value cannot be null.
9>  Parameter name: key
9>  Stack trace:
9>     at System.Collections.Generic.Dictionary`2.FindEntry(TKey key)
9>     at System.Collections.Generic.Dictionary`2.TryGetValue(TKey key, TValue& value)
9>     at Microsoft.Build.Evaluation.ProjectCollection.LoadedProjectCollection.RemoveProject(String projectFullPath, Project project)
9>     at Microsoft.Build.Evaluation.ProjectCollection.UnloadProject(Project project)
9>     at Bridge.Translator.Translator.GetSourceFiles(XDocument doc) in C:\NET\BridgeNew\Bridge\Compiler\Translator\Translator\Translator.ProjectReader.cs:line 152
9>     at Bridge.Translator.Translator.ReadProjectFile() in C:\NET\BridgeNew\Bridge\Compiler\Translator\Translator\Translator.ProjectReader.cs:line 19
9>     at Bridge.Translator.Translator.Translate() in C:\NET\BridgeNew\Bridge\Compiler\Translator\Translator\Translator.cs:line 99

The current code after 3f9e07b

fully works on Windows
Works on Mono but #531 broken (#562)

@fabricio, we need to check if the current changes in the code work on Mono and whether #531 broken or not (on Mono also). To check that, Bridge.sln should be compiler with the Testing projects loaded, the client tests run and then checking server-side NUnit tests (especially test 18).


font-size setting confusion on MacOS

If you create a custom config file here ~/.config/ghostty/config, the config file in the Library directory overrides it and resets the font-size back to the default.
This is fixed when removing the line from the ~/Library/Application\ Support/com.mitchellh.ghostty/config file. In my case, the font setting was the only setting in that file.
Also note that using cmd+, opens the custom file, not the Library version making this a very difficult setting to find.
It's very confusing to have all the settings change as expected except for the font-size.

As per the documentation, its stated in the following section that this will occur if both files exist: https://ghostty.org/docs/config#macos-specific-path-(macos-only):
However Ghostty opening the lower priority config file is a bug, will open that against Ghostty.


The readme says "stale detection"?

The project readme says "stale detection", but I would expect that to check some kind of timestamp in/on the file which does not appear to be the case. Any idea what that note is based on?

I believe the 'stale detection' is based upon the the _inner_check method.


Favorite button doesn't return original scale

When I press the favorite button and slide to outside of CardView, it does not return original scale.
I tried to fix it, sorry, but I couldn't. It seems to not call ACTION_UP touch event！

:eyes:

It seems complex issue. it might be LikeButton, UI library issue.
Anyway, thanks for your report!


Route params

How do I use Celebrate to validate route params?
For example http://localhost/verify/12445
I need to validate 12445. How can I do this?

Check out the first example in the README. Instead of body or query, use params.

Do I need to specify keys for Joi object? There is no key usually in params.
Secondly, if I would like to validate multiple parts of the route e.g /:user/:Id, how do I tell celebrate which parts of the route params I'm interested in.
An example to cover above scenario would help.

It works exactly like body and query in the README example. In your case, the keys would be named user and id.


Investigate Potential Bug in MCMC Tree Moves

During the visualization issue #42 some trees in an outdated, hence now irreproducible dataset, outputted by the MCMC sampler were not convertible to the TreeNode format. The reason was, that they seemed to lack a root. - Or the last node in the adjacency matrix at least was not the root. The exact issue is not clear. The function tree_inference_tree._get_root_label got an empty array in their line
# find row which has all ones in ancestor_matrix     root_idx = jnp.where(jnp.all(ancestor_matrix == 1, axis=1))[0]

[x] Consider adding debug logging to the tree moved to discover what went wrong the next time this happens.
[x] Add tests for the individual tree moves and check if it is a valid tree
[x] identify the issue
[ ] fix the issue
[ ] rerun mcmc mark00
[ ] check if all trees can be viszualized
[ ] check speed of mcmc

Conjectures:

root is assumed to be the last in the tree, and was swapped to be not the last
confusion if to include the root in moves or not

Write a test now to check the nested subtree case as Figure 17 in the supplement of the SCITE paper. As expected it is failed.
There is just some logic missing here. Next:

[ ] consider what the proposal has to check for - if they are of the same lineage? or just weather to swap the nodes, if one is a descendant of the other?
[ ] add logic to the swap_subtree_move

Implemented first not-working version of the complete logic of swap_subtrees
Next: debug with a print statement -already worked in.

New runtime per sample: 269ms at $n_{cells} = 2000$ and $n_{mutations}=50$


Use a common error struct to throw errors

https://github.com/deis/workflow-cli/pull/109#discussion_r69937420

This issue was moved to teamhephy/workflow-cli#26


Error in Example

When running the sample web application an exception is thrown
An unhandled exception occurred while processing the request.
InvalidOperationException: No service for type 'reCAPTCHA.AspNetCore.RecaptchaSettings' has been registered.
Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)
InvalidOperationException: No service for type 'reCAPTCHA.AspNetCore.RecaptchaSettings' has been registered.

error occurs in @inject RecaptchaSettings RecaptchaSettings

Can't currently reproduce this, example works just fine for me.

Perhaps @utech626 forgot to pull the latest version.

The example fails on 2 different development machine, plus if I try in production it fails:
.NET Core SDK (reflecting any global.json):
Version:   3.1.301
Commit:    7feb845744
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19041
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.301\
Host (useful for support):
Version: 3.1.5
Commit:  65cd789777
.NET Core SDKs installed:
3.0.100 [C:\Program Files\dotnet\sdk]
3.1.100 [C:\Program Files\dotnet\sdk]
3.1.101 [C:\Program Files\dotnet\sdk]
3.1.301 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.19 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.19 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.0.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.19 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.0.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.0.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.1 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

Add to Startup.cs
foreach (var _item in services)
{
if (_item.ServiceType.FullName.Contains("reCAPTCHA", StringComparison.OrdinalIgnoreCase))
{
Console.WriteLine($"Service: {_item.ServiceType.FullName}");
Console.WriteLine($"Lifetime: {_item.Lifetime}\n Instance:");
Console.WriteLine($"{_item.ImplementationType?.FullName}");
}
}
output shows:
Service: Microsoft.Extensions.Options.IConfigureOptions`1[[reCAPTCHA.AspNetCore.RecaptchaSettings, reCAPTCHA.AspNetCore, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]
Lifetime: Singleton
Instance:
Service: reCAPTCHA.AspNetCore.IRecaptchaService
Lifetime: Transient
Instance:
registered instance is a IConfigureOptions
Add to HomeController.cs constructor
public HomeController(IRecaptchaService recaptcha, IConfigureOptions settings)
{
recaptchaSettings = new RecaptchaSettings();
settings.Configure(recaptchaSettings);
return;
}
and now you can get an instance of RecaptchaSettings, this would need to be set in model

I just pulled a fresh copy of the repo and was able to reproduce this. It looks like my local copy had a stashed version of the Contact view which had changes for IOptions.
RecaptchaSettings does indeed no longer exist it's IOptions.


Failed builed on new arch Fabric on iOS (Did not find header RCTImagePrimitivesConversions.h)

Question
/node_modules/react-native-svg/apple/Elements/RNSVGImage.mm:36:9: 'react/renderer/imagemanager/RCTImagePrimitivesConversions.h' file not found

/node_modules/react-native-svg/apple/Elements/RNSVGImage.mm:36:9: Did not find header 'renderer/imagemanager/RCTImagePrimitivesConversions.h' in framework 'react' (loaded from '/Users/vladimir.evstratov/Library/Developer/Xcode/DerivedData/ShoppingList-cxwetcgtjsmprfcosczisbbsibkw/Build/Products/Debug-iphonesimulator/React-Core')

I have a problem with building project on iOS with new arch enabled.
System:
OS: macOS 13.5.2
CPU: (12) x64 Apple M2 Max
Memory: 382.69 MB / 32.00 GB
Shell:
version: "5.9"
path: /bin/zsh
Binaries:
Node:
version: 16.19.1
path: ~/.nvm/versions/node/v16.19.1/bin/node
Yarn:
version: 1.22.19
path: ~/.nvm/versions/node/v16.19.1/bin/yarn
npm:
version: 9.8.1
path: ~/.nvm/versions/node/v16.19.1/bin/npm
Watchman: Not Found
Managers:
CocoaPods:
version: 1.13.0
path: /Users/vladimir.evstratov/.rbenv/shims/pod
SDKs:
iOS SDK:
Platforms:
- DriverKit 23.0
- iOS 17.0
- macOS 14.0
- tvOS 17.0
- watchOS 10.0
Android SDK:
Android NDK: 21.4.7075529
IDEs:
Android Studio: 2022.3 AI-223.8836.35.2231.10671973
Xcode:
version: 15.0/15A240d
path: /usr/bin/xcodebuild
Languages:
Java:
version: 11.0.2
path: /usr/bin/javac
Ruby:
version: 2.7.6
path: /Users/vladimir.evstratov/.rbenv/shims/ruby
npmPackages:
"@react-native-community/cli": Not Found
react:
installed: 18.2.0
wanted: 18.2.0
react-native:
installed: 0.72.6
wanted: 0.72.6
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: true
newArchEnabled: false
iOS:
hermesEnabled: true
newArchEnabled: true

Any solution?

Can you provide a small repo with reproduction so we can work on it?

hello, I'm having the exact issue and it's currently a blocker for supporting the new architecture.

Attached is a sample project, that has nothing in it, just the result of npx<EMAIL_ADDRESS>init AwesomeProject and then added react-native-svg to it. You just need to do a yarn install and RCT_NEW_ARCH_ENABELD=1 bundle exec pod install and then try to build for iOS and you can see the issue.
The main thing here is that in the Podfile, I'm using use_frameworks! :linkage => :static and the new architecture is enabled.
AwesomeProject.zip

Can you check if #2175 solves the issue?


Add tint to tiling sprites in canvas renderer

Nice! We should also add it to WebGL so that they are consistent.

It's already in WebGL renderer because it use the same backend function that Sprite

It's already in WebGL renderer because it use the same backend function that Sprite

Well, no; but I do see that the color attribute already exists in the TilingSprite shader so it is supported.


Storages: Fix obtaining incorrect column information when there are virtual columns in the query (release-7.1)

This is an manual cherry-pick of #9189
What problem does this PR solve?
Issue Number: close #9188
Problem Summary:
When parsing some column expressions, it may need  to use column index to obtain column information, but since virtual columns are filtered out before being sent to the storage layer, the original columns and the columns for storage to read are inconsistency.
What is changed and how it works?
Use the original columns  in `query_info.dag_query` instead of `columns_to_read` when building `RSOperator`.

Check List
Tests

[ ] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

Fix obtaining incorrect column information when there are virtual columns in the query.

/run-all-tests

unstable test WriteLimiterTest.Rate


Make address screen match UK address format better

This tool is only for the UK, but we can use the standard address picker. states_list() will show the country within the UK, e.g., England, Wales, Scotland, Northern Ireland.
https://suffolklitlab.org/docassemble-AssemblyLine-documentation/docs/question_library/addresses
We should also use address autocomplete: True

Here's an easy way to do it:
states_list(country_code="gb")
Note that the country code is lowercase


capi by example

Something comparable to https://github.com/denoland/denobyexample

@tjjfvi shall we close, or do we still want to pursue rendering the examples as a by-example type site?

Closing, as it's preferable to view in code form / with LS-supported highlighting.


docs: add permissions info to FAQ

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[ ] Have you successfully run brew man locally and committed any changes?

Add some info about fixing permissions to the FAQ.
Inspired by Claudia's answer to a similar question on twitter.

Ah, okay. Didn't realize that. No need for this PR, then


Dolphin - Unable to set game path

The directory appears empty in Dolphin and clicking "open" does nothing.
The folder is on the internal drive and I have run this command to give access:
flatpak override org.DolphinEmu.dolphin-emu --filesystem=/home/deck/roms/ --user

Are you able to manually enter in the path instead of using the directory picker?

Didn't work, but got to it to work by long pressing the folder to highlight the folder name and then clicking Open instead of trying to open it from within.
Issue can be closed.


error when referencing same task in multiple chains of a workflow

I am trying to do...
var checkCanWrite = require('../tasks/checkCanWrite')
var checkExists = require('../tasks/checkExists')
var transcodeFile = require('../tasks/transcodeFile')
var killPid = require('../tasks/killPid')

module.exports = {
    name: 'video transcode',
    chain: [checkExists,checkCanWrite,transcodeFile],
    timeout: 7200,
    onError: [killPid],
    onCancel: [killPid]
}

When I try to insert this workflow into the database using factory.workflow() I get the error Task "body" must be a function.
I am using killPid task for both onError and onCancel.  If I remove the onCancel line from my workflow, no error occurs.

I have worked around this by cloning the required task like...
var checkCanWrite = require('../tasks/checkCanWrite')
var checkExists = require('../tasks/checkExists')
var transcodeFile = require('../tasks/transcodeFile')
var killPid = require('../tasks/killPid')
var killPid2 = Object.assign({},killPid)

module.exports = {
    name: 'video transcode',
    chain: [checkExists,checkCanWrite,transcodeFile],
    timeout: 7200,
    onError: [killPid],
    onCancel: [killPid2]
}


Game Crashes

1.3.5
44.1.23
Game Crashes immediately since the new update
Game just up and crashes as soon as I updated to the new version of the mod.
Updating to the latest version
Updated the mod.
Screenshots
No screenshots, but will provide logs.
crash-2023-07-12_14.54.45-client.txt

Can you confirm that this happens in 1.19.4 too? as its build using 1.19.4 so there is no gurantee it works on 1.19.3 (and as of my testings 1.19.4 works as intended)
may i ask why you are using 1.19.3 at all tho? you probably shouldnt use 1.19.3 and upgrade to 1.19.4, most of your mods should exist for 1.19.4 as it introduced rather minor changes

Unsure if it happens on 1.19.4 but I'm not trying to upgrade my game, that new update for the mod came up on the forge mod loader as 'for 1.19.3'. Not sure why

You can try to use 1.3.4 or older if they work, if not i need to create a build for 1.19.3 specifically
there shouldnt be major differences between those versions anyways

Will do, will also lyk if it continues to crash after downgrading

Yes, now after downgrading it works. Unsure why, but it shows as a compatible update.

I am currently working on unifying all versions in one project, however i started now to import a 1.19.3 versions so i can quickly release a build manually for 1.19.3 which should work properly

Epic! Hoping you also add some more blocks to be able to be compressed soon :)

this should work for now
(extract the archive, github doesnt allow to upload jars)
Compressed Blocks-forge-1.19.3.zip

I forgot to mention the reason why the 1.19.4 version doesn't work on 1.19.3:
In 1.19.3 cherry logs etc don't exist yet which causes the game to crash and because of a mapping difference


Allow reading the input state of pins in any function mode

On the actual chip, all input pins are always wired to the SIO block for inputs, no matter what mode the device is in. This means that no matter the function selected, reading the input state would be valid. It would be very nice if the InputPin trait could be implemented for all function types, as this doesn't break any of rust's safety rules.
I actually ran into a case where I needed to work around this. I was using a PWM Input, and needed to know when the falling edge was triggered. However, for the pin to be readable by PWM, it has to be in FunctionPwm mode, which doesn't allow reading the input. I actually had to use into_unchecked to turn the pin back into an input just so I could read it.

I see one trade-off to consider: While we technically are able to implement InputPin for all modes, it does decrease the amount of help the type system provides. In the current implementation, if you think you configured something as input pin, but in fact you configured the pin differently, you get a compile error. This can be really useful in case you mix up some pin numbers.
If all pins impl InputPin, there are a lot more cases that can no longer be caught by the compiler.
Perhaps we shouldn't just impl InputPin for everything, but instead provide some manual way to enable it? For example some wrapper type could be used. That way, accidental mix-ups of pins will still be caught, while special use-cases are covered.

Perhaps we shouldn't just impl InputPin for everything, but instead provide some manual way to enable it? For example some wrapper type could be used.

The wrapper could be accessible via an as_input() method on pins, like:
let pin = pin.into_function::<FunctionPwm>();
if pin.as_input().is_low() {
    // ...
}

Perhaps we shouldn't just impl InputPin for everything, but instead provide some manual way to enable it? For example some wrapper type could be used.

The wrapper could be accessible via an as_input() method on pins, like:
let pin = pin.into_function::<FunctionPwm>();
if pin.as_input().is_low() {
    // ...
}

That’d be fine, and makes sense to me. Keeps the type safety, but still allows safely reading the state when necessary.

I just noticed that we already have impl<I, P> embedded_hal::digital::v2::InputPin for Pin<I, FunctionSio<SioOutput>, P>, ie. we implement InputPin for pins configured as output. That weakens my argument regarding type safety. We might as well just implement InputPin for all Pin.
If we still want the as_input variant, https://github.com/rp-rs/rp-hal/commit/e4f1a0b114fff5648a3f0d92649f79faf7cea8be is a possible implementation.

For some reasons I cannot point yet, implementing InputPin for any function of a pin (effectively for Pin<_,_,_> seems off.
On the other hand, the datasheet explicitly says:

Software control of GPIO, from the single-cycle IO (SIO) block. The SIO function (F5) must be selected for the processors to drive a GPIO, but the input is always connected, so software can check the state of GPIOs at any time.

So I'm not strongly against it.
Alternatively, one can use Sio::read_bank0 to get a raw version of the pins status.

For some reasons I cannot point yet, implementing InputPin for any function of a pin (effectively for Pin<_,_,_> seems off.

I share that feeling. I think it's because we use the type-system to reach several goals, and some of those have divergent requirements:

Avoid undefined behavior. (This is not optional, as UB isn't localized. A program with UB is buggy.)
Steer developers towards working programs. "If it compiles, it works". Try to make buggy programs fail to compile.
Feed information to the compiler. Eg. if the compiler knows the desired pin function, a call like .reconfigure() can automatically do the right thing, without any parameters specifying what should be configured.
Of course, it should allow as many valid programs as possible. If it prevents the user from doing something the hardware is capable of, there usually is an option to use unsafe instead, but that should be a last resort, and not a common pattern.

In this case, implementing InputPin for all pin modes is fine with the first and the third goal, but it weakens the second goal. Meanwhile, it helps reaching the fourth goal.
In my opinion, there are two valid approaches regarding InputPin:

either implement it only for Pin<_, FunctionSio<SioInput>, _> (and not, as it's currently done, also for Pin<_, FunctionSio<SioOutput>, _>
or implement it for all Pin<_, _, _>

I don't see a good reasoning for the current middle ground.
I prefer the first option, with the addition of something like asInput() as an escape hatch. But removing the impl for SioOutput would be a breaking change.


bug: App crashes when trying to add a account.

My device is a Samsung Galaxy S7 Edge (Exynos)
Model: G935FD
Region: XSG
Android: 7.0
Going into the MicroG settings, then into Google Account and trying to log in crashes MicroG.
Same goes if you try to log in from any other app (i.e: Youtube, Youtube Music).
logcat.txt

The attached logs don't contain any crash logs.

Apologies, it's my first time using adb's logcat, perhaps these do?
logcat.txt
Here's a screenshot of what happened after i clicked log in:

The issue is not caused by ReVanced or even upstream. I think this is an issue with your device/Chrome installation.
06-02 15:18:17.077 13603 13603 E AndroidRuntime: Caused by: java.lang.reflect.InvocationTargetException                                                 06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at java.lang.reflect.Constructor.newInstance0(Native Method)                                    06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at java.lang.reflect.Constructor.newInstance(Constructor.java:430)                              06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at android.webkit.WebViewFactory.getProvider(WebViewFactory.java:198)                           06-02 15:18:17.077 13603 13603 E AndroidRuntime:        ... 26 more         06-02 15:18:17.077 13603 13603 E AndroidRuntime: Caused by: org.chromium.base.library_loader.ProcessInitException: errorCode=2                          06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at org.chromium.base.library_loader.b.i(chromium-Monochrome.aab-stable-604519420:172)           06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at org.chromium.base.library_loader.b.j(chromium-Monochrome.aab-stable-604519420:28)            06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at org.chromium.android_webview.AwBrowserProcess.i(chromium-Monochrome.aab-stable-604519420:34) 06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at com.android.webview.chromium.WebViewChromiumFactoryProvider.<init>(chromium-Monochrome.aab-stable-604519420:451)                                                         06-02 15:18:17.077 13603 13603 E AndroidRuntime:        ... 29 more         06-02 15:18:17.077 13603 13603 E AndroidRuntime: Caused by: java.lang.UnsatisfiedLinkError: dlopen failed: "/data/app/com.android.chrome-1/lib/arm/libmonochrome.so" is 32-bit instead of 64-bit                                    06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at java.lang.Runtime.loadLibrary0(Runtime.java:977)                                             06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at java.lang.System.loadLibrary(System.java:1567)                                               06-02 15:18:17.077 13603 13603 E AndroidRuntime:        at org.chromium.base.library_loader.b.i(chromium-Monochrome.aab-stable-604519420:52)            06-02 15:18:17.077 13603 13603 E AndroidRuntime:        ... 32 more


3-legged OAuth 1.0A

Seems that this is needed for twitter api requests that need a user context, like the user streaming api - httpie seemed great for that, but without 3-legged oauth, not so much.

Right. Is this for OAuth 2 or Oauth 1 or .... ?

I'm trying to use it to authenticate to tumblr and having no luck.


Odd offset when displayed as a modal

Not sure if this is a bug or a known tradeoff, but is there a way to adjust the offset of the header when the screen is being displayed as a popup? Wanting to allow users to drag the screen down. Have tried a few different things but can't quite crack it

Hey @dwnz, sorry for the late answer, but I'm happy to let you know I added an adjustment for cases like yours, please give version 0.1.1 a go. Have a marvellous day!

@f3dm76 Amazing, works perfectly - that has made my day. Thanks heaps for the fix (and the epic plugin)


Add option to retrieve ExplainXKCD entry

XKCD can get pretty meta. I'd like to get an explanation.
This would require some web-scraping, which can get kludgy really quickly.

I found out MediaWiki has an option to retrieve the wiki page text in raw form. A request to https://explainxkcd.com/wiki/index.php/614:_Woodpecker?action=raw returns
[comic
...

I could iterate through each line and cut out everything outside the Transcript.


Fix kernel issue

updates jupyter meta data to python3
ensures ipykernel in docs dependencies
ensure upstream is python kernel

OK, after some trial and error I got the tutorial so it looks more visually appealing and gets rid of a lot of the noise. Ready to merge.


Parenthesize multi-line jsx

Is it already possible or would this plugin be the right place to automatically parenthesize multi-line jsx?
In other words:
return

  Hello World

;

should become:
return (

    Hello World

);

it would be possible to be added as an option. And yes it can be done in this plugin

that said, pull requests welcome!

Hello! I've managed to wrap multi-line JSX into parens. But the problem is the closing parenthesis, there's no any ident or whitespace token before it which I could remove. How can I do this?
return (

    Hello World

  );

+1 for this

@roman01la do you still have the source for this?

@chenglou I think it's here https://github.com/roman01la/html-to-react-components/blob/master/lib/format.js#L226
I'm using in a tool to convert HTML to React components. You can find how to run it in readme.

Very nice @roman01la. I will try to see if I find time to add this to esformatter-jsx directly.

@royriojas Hey! Any updates on this issue? :)

So sorry @roman01la I haven't had the time yet. Will try to see if I can get something done over the weekend. On the other hand, since you do have an idea of how to fix this, please don't hesitate to make a PR. Will be more than thankful for your help.

Also, just in case you're also using eslint, the eslint-plugin-react has this rule https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/jsx-wrap-multilines.md that might help with this issue.

I'm now using eslint & eslint-plugin-react with auto-fix for all my formatting needs.

@DaleJefferson it seems like the auto fix simply wraps it in parens, so it ends up looking like this:
  render() {
    return (

      {initials(this.props.user)}

)
  }

is that what you're seeing too?

@aaronjensen Yes it's not perfect but I find ESLint --fix does 90% of the formatting, having a separate formatter seems redundant, I don't think there is an ESLint rule for new line after parens but i'm sure it would be fairly simple to write.

@DaleJefferson me too, I tend to agree with that. I can see how eslint can be converted to a formatter as well in the future at least.

@DaleJefferson - sorry if this is off topic...
Do you know if it is possible to use auto-fix for ESLint to auto format in Sublime 3 ?

@swinston1000 I execute eslint --fix from the command line, not from Sublime. But probably there is an option for that.

@royriojas - many thanks!


Fix supplier barcode order numbers

Seems like the logic for retrieving a purchase order from an order number from a supplier barcode got broken when refactoring. This fixes the logic and also some supplier specific details.

(I'll add some test cases later to confirm this keeps working in later versions.)

@30350n can you please provide some more specific information about how the current implementation is "broken"?

Sure!

The current implementation always checks for both the PO and CPO matching, if both are present in the barcode. Having a match on one of them should be sufficient.
LCSCs and TMEs barcode POs were incorrectly labeled as CPOs. TME actually has a CPO field though, which hasn't been used.
Mouser uses the ecia barcode 'K' field for both, POs and CPOs interchangeably (yeah I know, don't ask me why).

Actuall the implementation I came up is kind of convoluted I guess, I'll try to refactor it a bit.

@30350n if you have the data on hand to test that this works, I'm satisfied with that. please keep an eye on whether this continues to work

Thanks, will definitely do! I actually also wanted to confirm everythings actually working correctly with some tests but didn't find time to implement them yet. I'll try to get that down the coming days and create another PR 👍


Added abesit,ghaziabad

http://abesit.in/
http://abesit.in/computer-science-engineering/

@Vaibhavav Pull request merged. Thank you!


Minor fixes in the blog's sidebar

Moved the Published by Microsoft at the bottom of the sidebar.
Made the sidebar black all the way.
Fixed the blank link.

Related issue : https://github.com/Microsoft/pxt/issues/2789

@veler,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Tested on the microbit docs site? They share the same code path.

Yup, works fine :)


Add left-click-bank-actions plugin

Adds configurable item equipping, eating, and drinking directly from the bank interface using only the mouse.

Wouldn't it be easier to tell people to use the normal menu swapper?

Apologies, I was unaware of Runelite having this functionality.


create 2D mesh data member for crystal class

[ ] populate with dummy values for excited state density (sigma)
[ ] query incoming wave front (received via propagate method) for it's mesh size and dimensions
[ ] interpolate the sigma values from the crystal mesh to a new mesh that matches the wavefront

It looks like propagator names have changed and the use of variable name phE might lead to failures.
Address these issues as well as the boxes defined in the original comment.

The transverse extent of the two meshes [m] needs to be considered, as well as the number of cells along each axis.


Add support for header conditions

Adds header conditions to HTTPProxy if specified in Knative Ingress.
Fixes #54

@googlebot I signed it!

/assign @tcnghia @shashwathi @andrew-su

/approve


Make it optional to globally test the phone numbers

There are scenarios when the SimpleRegex tests will be run multiple times, for example, testing phone numbers during typing, after a user blurs input and just before sending an HTTP request to the server.
In these scenarios, the regex test return alternating results for the exact same phone number. After doing some digging (thanks to stack overflow), I realized that the global regex search starts from the previous match for each invocation of .test and that causes subsequent tests to have a different result even with the same input.
Example: Running SimpleRegex.MobileNumber.Econet.test(phoneNumber) twice, gives you different results
I would like to have an option to remove the global matching of the regexes.

Thank you @kudapara. I really ought to have considered this in terms of the documentation bit. So that behaviour is becuase I added the global flag on the Regex pattern. The flag is useful for cases where you want to search for multiple occurrences of a pattern within a large string. More on that here
So my thoughts are that we probably forgo the global flag altogether and allow package users to be able to add it if they are doing search on a paragraph. Maybe something like this:
const regexWithGlobalFlag = RegExp( SimpleRegex.MobileNumber +'/gm' )....  the new regexWithGlobalFlag could then be used to do searches where all occurrences of the string need to be found.
I think this would keep the package small and simple while providing convenience for users but I am open to hearing from people using the package and see what they prefer so I will invite people to discuss this.


Add BOHB Advisor

Add BOHB Advisor

Add new advisor with new algorithm BOHB, reference paper: https://arxiv.org/abs/1807.01774
Add MNIST example of BOHB usage
Add related docs and links

@PurityFan  I have one simple suggestion that you can change the _KEY = 'TRIAL_BUDGET' to _KEY = 'STEPS'.

@PurityFan I have one simple suggestion that you can change the _KEY = 'TRIAL_BUDGET' to _KEY = 'STEPS'(Hyperband uses this name).

We decided to change the “STEPS” uniform to “TRIAL_BUDGET” at the previous meeting.

I can run it successfully. Hope for approval.

I suggest that more people to review this big PR.


Improve kitchen sink

Please describe the changes this PR makes and why it should be merged:
Most of the projects are not mobile-responsive. Some texts are only visible in dark mode. Some needed UI enhancement. I have updated/cross-checked around 15 to 20 of them. I will continue the rest in a new PR.
Status

[x] Code changes have been tested against prettier, or there are no code changes
[x] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[ ] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1257

Relevant Lines:
5082

💛  - Coveralls

@tobySolutions check it out.

Awesome! Thanks it's under review.


#155570415 add gravatar image for users

What does this PR do?
add gravatar image for users
Description of Task to be completed?

install react-gravatar
display gravatar image

How should this be manually tested?
https://purpose-more-recipes.herokuapp.com/
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
#155570415 add gravatar image for users
Screenshots(is any)

Questions:
N/A

Coverage remained the same at 84.342% when pulling 2318b5b26ec636d858cfcf1f96c173536d96bbe8 on feature/155570415/gravatar-image-for-users into bcda8ccad7a20e967a775d4753616918221b4eb7 on develope.


Hungarian translation

Based on the English language file downloaded from GitHub today

Hi, i can't merge that because it would break compatibility.
Please check out the wiki (https://github.com/martinet101/WingetUI/wiki#translating-wingetui) in order to see how to translate WingetUI.
Thank you!


PSAOAI - add new Mode

new mode: "VeryPrecise" - temp 0,1 and topp 0,7

new modes:
Balanced:
Temp: 0.5
Topp: 0.85 (Balances creativity and precision)
More Creative:
Temp: 0.8
Topp: 0.9 (Higher chance of surprising or unusual outputs)

Other modes that could be added are:

'fast': temp 0.9 and topp 0.5 - this mode would prioritize speed over accuracy.
'balanced': temp 0.5 and topp 0.5 - this mode would strike a balance between speed and accuracy.
'random': temp 0.9 and topp 0.9 - this mode would introduce more randomness in the output, resulting in more creative and unpredictable responses.
'emotional': temp 0.5 and topp 0.9 - this mode would prioritize generating emotionally charged responses.
'logical': temp 0.5 and topp 0.1 - this mode would prioritize generating logical and rational responses.

Use Case (Your Words)
Temperature
Top_p
Description

Creative
0.7
0.8
Explores creative text formats, less constrained.

Surreal
1.0
0.1
Highly imaginative and random text formats, often nonsensical.

Ultra Precise
0.1
0.95
Deterministic code adhering to strictest conventions. Prioritizes correctness over anything else.

Focused
0.2
0.8
Ensures correctness and emphasizes clarity, likely for code or technical writing.

Balanced
0.5
0.5
Natural conversation with balance between coherence and some creativity.

Informative
0.4
0.7
Well-structured and informative text formats (reports, articles).


 Maybe there is an easy way to disable this behavior?
P.S.: It's weird that I am the only one having an issue with this, because I could not find anything related to this on google. Maybe I am missing something and the switch is already implemented in game and everybody knows about it, or i'm just using wrong keywords. -_-
Thank you.

I have never heard of something like this, this most certainly is not a feature of the game. You must be experiencing some kind of game bug, have you tried asking for help in BSMG?


WARNING: update already refers to: #'clojure.core/update in namespace: leiningen.core.project, being replaced by: #'useful.map/update

This warning appears to be from lein-cucumber depending on an old version of leiningen-core.

This warning appears to be from lein-cucumber depending on an old version of leiningen-core.

You can create a pull request to fix it.


0.14.3 - no method named clear found for struct HashMap in the current scope

I'm trying to understand if 0.14.3 has a compile time bug or if it's just being used incorrectly.
I opened up this issue https://github.com/lightningdevkit/ldk-node/issues/221
error[E0599]: no method named `clear` found for struct `HashMap` in the current scope
   --> /...../.cargo/registry/src/index.crates.io-6f17d22bba15001f/hashbrown-0.14.3/src/set.rs:432:18
    |
432 |         self.map.clear();
    |                  ^^^^^
    |
   ::: /..../.cargo/registry/src/index.crates.io-6f17d22bba15001f/hashbrown-0.14.3/src/map.rs:190:1
    |
190 | pub struct HashMap<K, V, S = DefaultHashBuilder, A: Allocator = Global> {
    | ----------------------------------------------------------------------- method `clear` not found for this struct
    |
help: one of the expressions' fields has a method of the same name
    |
432 |         self.map.table.clear();
    |                  ++++++
help: there is a method with a similar name
    |
432 |         self.map.waclear();
    |

This leads me to think there is an issue with v0.14.3 of hashbrown.

help: there is a method with a similar name
|
432 |         self.map.waclear();

Did you accidentally edit your copy of hashbrown in your .cargo cache? There is no waclear method in hashbrown, it looks like a typo on your end.

@Amanieu, just running the https://github.com/lightningdevkit/ldk-node readme example. The error makes it look like it's suggesting .waclear instead of clear. I'm a little unsure of how to debug what's going on.

Try deleting ~/.cargo/registry to force re-downloading all crates.

i just built this project for the first time today a bit ago, i don't think anything got downloaded wrong?

deleting ~/.cargo/registry/src and rebuilding fixed my issue. Unsure why this was needed or what happened though.


Missing parameter names in outline.entry definition in documentation

https://typst.app/docs/reference/meta/outline/#outline-entry-parameters:

crates/typst-library/src/meta/outline.rs
Should have parameter names:

I believe this is because the parameters all positional and required, so that is actually how you call the outline.entry function (you use outline.entry(a, b, c, d) instead of outline.entry(level: 5, element: ..., ....)).
I agree this isn't ideal though. Not sure but we could try to find a way to have the named parameters be required.

this is because the parameters all positional and required

The fact is that you only see data types, but in order for the documentation to be readable, reader have to know what actually they need to pass to the function with the shown data types.
So, if the parameters are provided without names (in a source file), then it's better to say about this right there in the documentation, but names of the "things" that the function (outline.entry) needs should be present (in the definition in the documentation).

The names are present down below in the explanations (which are in the same order).
Ideally, if we switched from positional/named to required/optional arguments, that issue would go away.

Hacking in required named parameters isn't the way to go in my opinion. We should rather concretize how we can switch to required/optional as the distinction.

Valid. That's still gonna require a bit of discussion, but I don't think this issue is the appropriate place for it. We should perhaps just leave a link here to an actual issue on the subject instead.

The names are present down below in the explanations (which are in the same order).

Yes, but that's the thing. You see function definition with no parameter names (only their types), but below you can see all the missing names. Usually (in the same documentation), you can visually map each parameter in definition and its description below. In this case, you can't.
I'm just speaking from the perspective of the reader: this function breaks the usual assumption of the Typst doc page (even on the same page! outline/outline.entry), if you know what I mean.

About "positional/named" and "required/optional", I don't even want to touch it. It's kinda makes wonders with super slick (IMO) func(a: 234, b: dfsa, content)/func(content, a: 234, b: dfsa)/func(a: 234, b: dfsa)[any content], but it also feels strange when creating functions with default values (you have to name parameter, but you don't if there is no default value). It's very unique and has pros and cons, so it's up to you to make all the heave thinking.

Just found another example: https://typst.app/docs/reference/layout/enum/#enum-item-parameters.

It is always for positional parameters, so also e.g. for calc functions. The idea is that you can see from the signature whether it is positional or named.

I would classify this as "working as intended" because the section at the top shows how you would call the function.


Selected Azure profile 'v1.0' does not exist for module 'Microsoft.Graph.Identity.Protection'

When importing the module I get the following message:
WARNING: Selected Azure profile 'v1.0' does not exist for module 'Microsoft.Graph.Identity.Protection'. No cmdlets were loaded.
WARNING: Selected Azure profile 'v1.0' does not exist for module 'Microsoft.Graph.Identity.Protection'. No cmdlets were loaded.
What am I missing?
//Jasper

Never mind, it's working now. Don't really know what happened. :)


Undefined services causes exception

Version:
0.45.0

Severity:
Low
Description:
Creating a libp2p instance with services that are undefined creates a null pointer exception even if it is allowed by the types
createLibp2p({
...,
    services: {
         abc: undefined
    }
})

To mitigate this I think there needs to be a null check before this
https://github.com/libp2p/js-libp2p/blob/710da918a89b6f6ee5d03c0d14a311efd40a1b8f/src/libp2p.ts#L167
to check if createService is a function.
I ended up with this error because I am generating different configs based on conditions, and this leads me to assign "undefined" to services relay occasionally

Related: https://github.com/libp2p/js-libp2p/issues/1573

A runtime check makes sense but this really shouldn't be allowed by the types.

Will be closed by #2133 when that lands.

JS Colo 2024 discussion
still waiting on config validation PR..

no new updates


Update doc conf version match regex

closes https://github.com/ROCm/ROCmValidationSuite/issues/827
fixes failing doc build due to mismatch

fixed in https://github.com/ROCm/ROCmValidationSuite/pull/823


Live Preview not working - no selected (blue) borders

I don't know what caused it, possibly installing Windows 10, as brackets still worked for me on friend's computer on Win 7; including with the same page I'm coding - only html and css.
What doesn't work?
Live Preview starts up and is live, though it doesn't mark what I've selected in the code with those blue borders. Also, Live Preview crashes much more often ("Live Preview was cancelled because the page was closed in the browser"), though I can (sometimes) get it to work.
What didn't help?
Anything in the FAQ
Reinstalling brackets and chrome
Brackets v1.3
Chrome disconnected account

Cheers :)

Just to sort the simple things out, could you just check that "View > Live Preview Highlight" is still checked?

Thank you :D
What solved the problem was toggling it on and off (but a reinstall didn't...)

I also got this problem on Windows 10. and toggling it off and on not work for me.
sad...


feat: added uuid validation rule

Proposed changes
Add the ability to validate UUIDs
Types of changes
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation (if appropriate)

Further comments
Added an additional rule and corresponding tests. Shouldn't interfere with existing rules etc. Just stuck with the available options in validator so it won't validate UUID v1 or v2 or nil UUIDs.

Closed and re-opened to kick in circleci tests

Thanks for contributing. The PR was great 😄


Vendor deps

Pretty low priority IMO, in practice this isn't a huge problem for something with infrequent releases like Apex. Go 1.6 is coming soon so we'll have the vendor lookup.
One thing I'm not sure of is if you can disable the GOPATH lookup or not, the docs didn't make it clear if ./vendor lookup was basically prepending GOPATH or what, if so I think that's a mistake on their part, having it lookup ./vendor and GOPATH is not good. So if we can have it explicitly use ./vendor only that would be great.

TJ, 1.6 looks in both places by default. I believe 1.5 checks GOPATH and ./vendor if the environmental variable is present. You can explicitly use ./vendor by rewriting imports...is that something we would want to do in the future?

This might be re-looked at later, closing for now.

Would like to reopen this and investigate what we can use. I'd be happy to submit a PR. Trying to contribute is somewhat of an uphill battle, tracking down what dependencies apex needs. It'd be much better if I could just use godep or glide and install everything.

go get -u ./... should be fine, I was using https://github.com/dpw/vendetta for this PR but doing all the hacks to omit ./vendor is pretty lame

(facepalm). Thanks!

/me adds to CONTRIBUTING.md