[Vite] Noisy error “Module level directives cause errors when bundled, "use client" was ignored.” when running remix vite:build on a project that includes a dependency file which includes the "use client" directive.

Reproduction
If I got more time, I may submit a PR with a failing test, but for now, here’s an issue.
To reproduce:
git clone https://github.com/dtinth/fewshot
cd fewshot
git checkout 82d6437be436f23c506294998d85a5222501a22d
corepack enable
pnpm install
pnpm run build

System Info
System:
    OS: macOS 14.2.1
    CPU: (8) arm64 Apple M1
    Memory: 53.09 MB / 16.00 GB
    Shell: 3.6.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 21.5.0 - ~/.asdf/installs/nodejs/21.5.0/bin/node
    npm: 10.2.4 - ~/.asdf/plugins/nodejs/shims/npm
    pnpm: 8.15.4 - /opt/homebrew/bin/pnpm
    bun: 1.0.29 - ~/.bun/bin/bun
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 121.0.6167.184
    Edge: 122.0.2365.52
    Safari: 17.2.1
  npmPackages:
    @remix-run/dev: ^2.7.2 => 2.7.2
    @remix-run/node: ^2.7.2 => 2.7.2
    @remix-run/react: ^2.7.2 => 2.7.2
    vite: ^5.1.0 => 5.1.4

Used Package Manager
pnpm
Expected Behavior
Build output should be short. Vite should not complain about "use client" directive.
Note that there is a similar issue in another project.

https://github.com/TanStack/query/issues/5175

Based on the above issue, @vitejs/plugin-react should suppress this error for us. However, it seems that Remix’s vitePlugin does not come with Vite’s React plugin and implements its own React support.
Perhaps Remix’s Vite plugin should add the warning silencer just like @vitejs/plugin-react? The warning silencer seems to implemented here: https://github.com/vitejs/vite-plugin-react/blob/814ed8043d321f4b4679a9f4a781d1ed14f185e4/packages/plugin-react/src/index.ts#L303-L319
Actual Behavior
I get a wall of warning in the build output that looks like this:

It would be nice if it was handled automatically, but I've been doing this in the meantime:
// vite.config.ts
import { defineConfig } from "vite";

export default defineConfig({
  build: {
    rollupOptions: {
      onwarn(warning, warn) {
        // Suppress "Module level directives cause errors when bundled" warnings
        if (warning.code === "MODULE_LEVEL_DIRECTIVE") {
          return;
        }
        warn(warning);
      },
    },
  },
  // your other Vite config stuff...
});

You probably already found something similar, but dropping it here just in case it's helpful for somebody.

Thanks for raising this. Fixed by #8897.

It would be nice if it was handled automatically, but I've been doing this in the meantime:
// vite.config.ts
import { defineConfig } from "vite";

export default defineConfig({
  build: {
    rollupOptions: {
      onwarn(warning, warn) {
        // Suppress "Module level directives cause errors when bundled" warnings
        if (warning.code === "MODULE_LEVEL_DIRECTIVE") {
          return;
        }
        warn(warning);
      },
    },
  },
  // your other Vite config stuff...
});

You probably already found something similar, but dropping it here just in case it's helpful for somebody.

awesome it's works , thanks bro 👍


[BUG] - CPU spikes during epoch boundaries and at 48h time mark causing important gaps in block production

External
Area
Other:  Interruption in Blocks Production for SPO's pools at epoch transition and 48h time mark
Summary
Both during epoch transitions and at 48h time mark, most of [ALL] nodes experience a significant CPU spike, causing the network to become stagnant. If we have a slot scheduled during these timeframes, that will most likely be a missed block
My pool was scheduled to mint a block at slot #30 (21:45:21 UTC) into epoch 250 and according to the BP node's log, it did not even attempt to produce the block. All in all, during this transition, 2:30 min elapsed without a block produced by SPO's pools
Steps to reproduce
Calculate pool's leaderlogs and then be unlucky to have a slot assignment during these time frames
Expected behavior
A smooth epoch transition, and same for 48h mark
System info (please complete the following information):

OS: Ubuntu
Version: 20.04
Node version: 1.25.1

Screenshots and attachments

Additional context
Leaderlogs schedule according to Andrew's cncli leaderlogs

This is a serious flaw. It will affect rewards in many pools. It should be fixed as a top priority.

The 48 hour bug is being discussed here: https://github.com/input-output-hk/cardano-node/issues/2205

Closing in favor of #2205 .


Incompatibility between this page and the .appinstaller schema docs

In the docs the UpdateSettings element can contain both OnLaunch and AutomaticBackgroundTask. The problem is Visual Studio allows you to select either one of the two.
So if you select check every hour, the .appinstaller will contain the OnLaunch element. If you later add the AutomaticBackgroundTask to the published .appinstaller, no user will be able to install your app. They will receive this error:
Deployment Add operation with target volume C: on Package [packageNameAndVersion]neutral~_vqex3v0c1enw0 from:  ([packagename].appxbundle)  failed with error 0x80070005. See http://go.microsoft.com/fwlink/?LinkId=235160 for help diagnosing app deployment issues.

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

ID: 930536bd-243e-3024-1c8d-b352384a2718
Version Independent ID: 518c2651-df80-73fb-f28c-a48e8dde863d
Content: Create an App Installer file with Visual Studio - MSIX
Content Source: msix-src/app-installer/create-appinstallerfile-vs.md
Product: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

Hey Julien,
I tried to reproduce the issue you mentioned but wasn't able to, my app installed fine when I had both the OnLaunch and AutomaticBackgroundTask elements in
This is what I did:

I created an App Package in VS
I selected the "Check For Updates every 1 Hour" Option
VS created a .appinstaller file
I modified it and added  to the Update Settings.
I installed the package using the .appinstallerfile which worked fine

My app package has MinVersion="10.0.17763.0" and MaxVersionTested="10.0.18362.0". What are the values in your package and what Windows version were you trying to install on?
Here is what my .appinstaller file looks like:
<?xml version="1.0" encoding="utf-8"?> <AppInstaller 	Uri="<AppInstallerFileLocation>\AppInstaller_TestApp\AppInstaller_TestPkg\AppPackages\AppInstaller_TestPkg_x64.appinstaller" 	Version="<IP_ADDRESS>" xmlns="http://schemas.microsoft.com/appx/appinstaller/2017/2"> 	<MainPackage 		Name="MyApp" 		Version="<IP_ADDRESS>" 		Publisher="CN=MyPublisher" 		Uri="<PackageLocation>/AppInstaller_TestPkg_<IP_ADDRESS>_x64_Debug_Test/AppInstaller_TestPkg_<IP_ADDRESS>_x64_Debug.msix" 		ProcessorArchitecture="x64" /> 	<UpdateSettings> 		<OnLaunch 			HoursBetweenUpdateChecks="1" /> 			<AutomaticBackgroundTask/> 	</UpdateSettings> </AppInstaller>

@Huios Can you check if the update works as well?

Host the appinstaller on an webserver (and open it with HTTP).
Install the default version of the app (with appinstaller)
Generate a new version of the app (with appinstaller version number +1)
Then go to the http page and install the update

Does this test work as well for you?

Hey Julien,
I tried to reproduce the issue you mentioned but wasn't able to, my app installed fine when I had both the OnLaunch and AutomaticBackgroundTask elements in
This is what I did:
* I created an App Package in VS

* I selected the "Check For Updates every 1 Hour" Option

* VS created a .appinstaller file

* I modified it and added  to the Update Settings.

* I installed the package using the .appinstallerfile which worked fine

My app package has MinVersion="10.0.17763.0" and MaxVersionTested="10.0.18362.0". What are the values in your package and what Windows version were you trying to install on?
Here is what my .appinstaller file looks like:
<?xml version="1.0" encoding="utf-8"?> <AppInstaller Uri="<AppInstallerFileLocation>\AppInstaller_TestApp\AppInstaller_TestPkg\AppPackages\AppInstaller_TestPkg_x64.appinstaller" Version="<IP_ADDRESS>" xmlns="http://schemas.microsoft.com/appx/appinstaller/2017/2"> <MainPackage Name="MyApp" Version="<IP_ADDRESS>" Publisher="CN=MyPublisher" Uri="<PackageLocation>/AppInstaller_TestPkg_<IP_ADDRESS>_x64_Debug_Test/AppInstaller_TestPkg_<IP_ADDRESS>_x64_Debug.msix" ProcessorArchitecture="x64" /> <UpdateSettings> <OnLaunch HoursBetweenUpdateChecks="1" /> <AutomaticBackgroundTask/> </UpdateSettings> </AppInstaller>
Tanaka MSFT

My AppInstaller has this:
<?xml version="1.0" encoding="utf-8"?>
<AppInstaller xmlns="http://schemas.microsoft.com/appx/appinstaller/2017"
              Version="<IP_ADDRESS>"
              Uri="MyPrivateUri">

  <MainBundle Name="MyPrivatePackageName"
	      Publisher="CN=CompanyName, O=CompanyName, L=CompanyCity, C=CompanyCountryCode"
	      Version="2019.909.1235.0"
              Uri="MyPrivateAppxBundleUri" />

  <UpdateSettings>
      <OnLaunch />
      <AutomaticBackgroundTask />
  </UpdateSettings>
</AppInstaller>

The app targets 1803, the minimum target is also 1803.

@julienshepherd - Did you try an update of a higher version? Or just an install?

@julienshepherd - Did you try an update of a higher version? Or just an install?

I tried both.

@julienshepherd
Thanks for posting your AppInstallerfile. I think the issue is due to the AppInstaller schema you're referencing in your file.
According to the docs here AutomaticBackgroundTask requires the xmlns="http://schemas.microsoft.com/appx/appinstaller/2017/2" namespace. The one in your current AppInstallerfile is  xmlns="http://schemas.microsoft.com/appx/appinstaller/2017"
We have AppInstallerFileBuilder tool that can help with creating a file with the right namespace schema for different update settings. You can check it out here.
I hope this helps.
Tanaka MSFT


synth_ice40 -flowmap terminate called after throwing an instance of 'std::length_error'   what():  vector::reserve

Steps to reproduce the issue
read_verilog e.v
synth_ice40 -flowmap

e.v:
module test
  (
   input wire [11:0] x,
   input wire [1:0] y,
   output wire [11:0] z,
   );

assign z = x >> y;

endmodule

Expected behavior
No crash.
Actual behavior
[...]
2.41. Executing FLOWMAP pass (pack LUTs with FlowMap).
Labeling cells.
Mapped to 24 LUTs with maximum depth 2.

Packing cells.
Packing test.$techmap$shr$/home/corecode/tmp/e.v:8$1.$auto$simplemap.cc:278:simplemap_mux$584.\Y (\z [11]).
  Packing test.$techmap$shr$/home/corecode/tmp/e.v:8$1.$auto$simplemap.cc:278:simplemap_mux$620.\Y ($techmap$shr$/home/corecode/tmp/e.v:8$1.$1\buffer[11:0] [11]).
terminate called after throwing an instance of 'std::length_error'
  what():  vector::reserve
zsh: IOT instruction (core dumped)  ./yosys

I have experienced the same issue. It seems to trace back to flowmap.cc:1409, where minlut - input_nodes.size() can return a negative number.
A workaround is to change the minlut parameter to 4 to equal the maxlut parameter, but quite frankly that's not ideal. I'm thinking maybe the expression should be something like maxlut - input_node.size, but I'm not familiar enough with the code of yosys to actually know if this will fix it, and whether it would actually be correct.


Added a new question about git (git fetch command)

Question that allow to download objects without having to commit to the main branch. A great feature to consider when you are working on projects using different sources.
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

Closes #XXXXX

Thank you so much for your support!!


Support creation of contracts with CREATE2 in scripts

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (2457e5f 2022-06-16T00:07:46.767196942Z)
What command(s) is the bug in?
forge script
Operating System
(tested on Ubuntu Linux and Mac)
Describe the bug
When running contract creation with CREATE2 scripts don't deploy contracts as expected.
Reproducible example:
pragma solidity ^0.8.13;

import "forge-std/Script.sol";

contract D {
  uint256 public x;

  constructor(uint256 a) {
    x = a;
  }
}

contract MyScript is Script {
  uint256 create2Salt = 5;

  function run() external {
    vm.startBroadcast();

    uint256 arg = 654654654;

    address predictedAddress = address(
      uint160(
        uint256(
          keccak256(
            abi.encodePacked(
              bytes1(0xff),
              msg.sender,
              create2Salt,
              keccak256(abi.encodePacked(type(D).creationCode, abi.encode(arg)))
            )
          )
        )
      )
    );

    D d = new D{salt: bytes32(create2Salt)}(arg);
    console.log("The below 2 addresses should be the same: <address(d), predictedAddress>");
    console.log(address(d), predictedAddress);

    vm.stopBroadcast();
  }
}

When running this script we get this output:
$ forge script ./contracts/scripts/Create2.s.sol:MyScript --rpc-url $MUMBAI_RPC_URL  --private-key $PRIVATE_KEY --broadcast --verify --etherscan-api-key $ETHERSCAN_KEY -vvvv
[⠰] Compiling...
[⠊] Compiling 1 files with 0.8.13
[⠒] Solc 0.8.13 finished in 549.83ms
Compiler run successful
Traces:
  [90323] MyScript::run()
    ├─ [0] VM::startBroadcast()
    │   └─ ← ()
    ├─ [47315] → new D@0x3e0dde95fe6889388e7b5202e87a651b5f1a0f4e
    │   └─ ← 125 bytes of code
    ├─ [0] console::log("The below 2 addresses should be the same: <address(d), predictedAddress>") [staticcall]
    │   └─ ← ()
    ├─ [0] console::log(D: [0x3e0dde95fe6889388e7b5202e87a651b5f1a0f4e], 0xfe0e52391686289fc281350bce83bde14d235421) [staticcall]
    │   └─ ← ()
    ├─ [0] VM::stopBroadcast()
    │   └─ ← ()
    └─ ← ()

Script ran successfully.
Gas used: 90323

== Logs ==
  The below 2 addresses should be the same: <address(d), predictedAddress>
  0x3e0dde95fe6889388e7b5202e87a651b5f1a0f4e, 0xfe0e52391686289fc281350bce83bde14d235421
==========================
Simulated On-chain Traces:

  [104072] Create2Deployer::create2()
    ├─ [47315] → new D@0x3e0dde95fe6889388e7b5202e87a651b5f1a0f4e
    │   └─ ← 125 bytes of code
    └─ ← 0x3e0dde95fe6889388e7b5202e87a651b5f1a0f4e

==========================

Estimated total gas used for script: 135293

Amount required: 0.00040587900270586 ETH

==========================

###
Finding wallets for all the necessary addresses...
##
Sending transactions [0 - 0].
⠁ [00:00:00] [################################################################################] 1/1 txes (0.0s)
Transactions saved to: broadcast/Create2.s.sol/80001/run-latest.json

##
Waiting for receipts.
⠉ [00:00:07] [############################################################################] 1/1 receipts (0.0s)
#####
✅ Hash: 0xdb21c3da66739ac53d8a2a06843ebd7964187beecf4fe592a5957afa56a0e967
Block: 26773410
Paid: 0.00031221600104072 ETH (104072 gas * 3.00000001 gwei)

Transactions saved to: broadcast/Create2.s.sol/80001/run-latest.json

==========================

ONCHAIN EXECUTION COMPLETE & SUCCESSFUL. Transaction receipts written to "broadcast/Create2.s.sol/80001/run-latest.json"
##
Start Contract Verification

Transactions saved to: broadcast/Create2.s.sol/80001/run-latest.json

And on-chain the transaction that is created is: https://mumbai.polygonscan.com/tx/0xdb21c3da66739ac53d8a2a06843ebd7964187beecf4fe592a5957afa56a0e967 - notice that it doesn't deploy any contract, and it just calls out to this contract.
I notice that the address it calls out to is mentioned in the foundry codebase, eg here.

Example contract used is taken from here: https://docs.soliditylang.org/en/latest/control-structures.html#salted-contract-creations-create2 , but the same issue happens with other contracts I've tested.

It does work—I've been predicting/verifying create2 address in scripts. I think the issue is that when you predict the address that msg.sender is different than the one broadcasting the transaction

The 0x4e59b44847b379578588920ca78fbf26c0b4956c address is a create2 deterministic deployer that is deployed across many chains. You need that contract at the same address on all supported chains. Create2 cannot be called by an EOA, so to use it you send the contract bytecode to a deployer contract

closing as there is not a bug here

It does work—I've been predicting/verifying create2 address in scripts. I think the issue is that when you predict the address that msg.sender is different than the one broadcasting the transaction

Thanks, can you please share what you use instead of msg.sender? Logging msg.sender in the scripts does display the correct address afaik.
I did try very carefully to get this to work. So I would really appreciate it if you could share a snippit from your scripts @mds1

On mobile right now so can't send a snippet, but looking again it seems I misspoke earlier. When predicting the address, you're using msg.sender as the deployer address, but it should be 0x4e59b44847b379578588920ca78fbf26c0b4956c

Great - that is working, thanks @mds1 .
The contract however isn't being automatically verified by the script currently.

Cool! let's follow up on the separate issue

I have a problem at present; When I use ganache to fork, create2 is created successfully. When using anvi fork l, create2 returns 0x00000000000000000000000000000000000000000000000000000000000
ganache -f https://bsc-mainnet.nodereal.io/v1/a9d5920ea85349c298ee543d3d4922ac -e 100
00 --chain.chainId 56 -l<PHONE_NUMBER> -g<PHONE_NUMBER>000 --fork.blockNumber 22070910
anvil -f https://bsc-mainnet.nodereal.io/v1/a9d5920ea85349c298ee543d3d4922ac  --chain-id  56 --gas-limit<PHONE_NUMBER>0000000000 --gas-price<PHONE_NUMBER>000000000   --fork-block-number 22070910
`
// SPDX-License-Identifier: Unlicensed
pragma solidity 0.8.15;
contract Deployer {
bytes32 public salt = keccak256(abi.encodePacked(0x0000000000000000000000000000000000000002));
function deploy(bytes memory data) public returns(address Addr){
bytes32 _salt = salt;
address addr;
assembly {
addr := create2(0, add(data, 0x20), mload(data), _salt)
}
Addr = address(addr);
return Addr;
}
}
`


--user-data-dir no longer works for extension test runs

Using --user-data-dir for extension test runs used to work (I used it so my test runs aren't affected by my local settings), but it seems to have become broken somewhere in the last few months.

Create a new extension with yo code
Edit the launch.json to pass --user-data-dir in the extension test run config:"args": [
	"--extensionDevelopmentPath=${workspaceFolder}",
	"--extensionTestsPath=${workspaceFolder}/out/test/suite/index",
      "--user-data-dir=${workspaceFolder}/user-data-dir" // Add this
],

Create that user-data-dir folder (I'm not sure if this is required)
Update the sample test so it never completes:test('Sample test', async function () {
	assert.strictEqual(-1, [1, 2, 3].indexOf(5));
	assert.strictEqual(-1, [1, 2, 3].indexOf(0));

	this.timeout(999999);
	await new Promise(function () { });
});

Select the Extension Tests debug config and launch it
Once the tests start, open the User Settings for the spawned VS Code extension development host
Note that the opened settings file is your standard user settings and not the one supplied in --user-data-dir (and that the one you created is empty)

@DanTup I'm surprised that you've seen --user-data-dir work with extension debugging since we have this long-standing issue: https://github.com/microsoft/vscode/issues/44734

Is that issue talking about passing the main VS Code flags through to the extension dev host? Mine are specifically in the launch.json.
I'm certain it was working 4 months ago, as I had to shorten the paths being used in it to fix some other issues:
https://github.com/Dart-Code/Dart-Code/commit/e5bce778ba8d545f013d814fb88565a26b1905a2

VS Code debugging never supported extension debugging across VS Code instances with different "user-data-dir".
This was confirmed by the recent investigation on https://github.com/microsoft/vscode/issues/130504.


DOCS Adding GitHub labels

As per my proposal to change GH labels.

+1 Nice addition @dhensby :+1:

I'm not sure this is the best place for them to go but couldn't find anywhere more suitable.
It may be worth adding a section about guidelines for core commiters somewhere?
@camfindlay / @wilr

Dan, can you confirm that you'll rename the existing labels once this is merged?

can you confirm that you'll rename the existing labels once this is merged

Yep :)

Addressed those comments

Thanks Dan!

Ok, I've added the labels now (to framework) renaming where appropriate - Colours were chosen by GitHub - I'm not great with colours and such so maybe someone could propose or amend them to be more suitable?


A little suggestion

Hello, your library looks great. I have used it in my project, but there is actually a small problem. Users don't have to make a selection every time. They can cancel and keep the last selected time, but I can't add it to the default title for now.
I hope to add a feature that allows users to abandon this selection.
Now my experience is: clicking outside the screen will return null, which is equivalent to resetting.
Logically:
Click reset: return null
Click confirm: return date
Click outside the screen: return nothing (user still keeps the previously selected date)

Similar to how I can add a reset or cancel here

Similar to how I can add a reset or cancel here

I read the source code carefully and found that single selection supports TopWidget customization, but multiple selection does not support TopWidget and Controller. I hope that subsequent versions can add

@ilovexy2tom
You’re right; it might not have supported multiple selections. I will fix it along with bug #58. Please wait a moment.


Clear cache on save

On Clicking the save button the cache will be cleared

need to wait for this PR to merge because the cache is already been cleared on save


[IM] Add Fabric Filtered Write Support

Problem
Fabric filtered write is not well supported.
Fixes #12053
Change overview
Add support to fabric filtered write

Updated Decode() function (accepts a optional fabric index, for overwriting the fabric index in TLV)
Add DecodeWithoutFabricIndex() for client side decoding.
Update ACL cluster server

Testing
WIP, please review the logic first

Fast tracking, given > 3 business days for review.


Dont add librt to the MIOpen's interface

librt is not used in miopen's public header so it should only be linked privately and for the tests.
This causes an issue when using miopen on ubuntu 22.04, because it is named librt.so.1 instead of librt.so. We really shouldn't ever use absolute paths in the target interface since libraries can be located in other locations.

@pfultz2 it's outdated but the PUBLIC link is explicitly changed in https://github.com/ROCmSoftwarePlatform/MIOpen/commit/ec3fdbe6b907f2b6599fc8de366d41a67eeaaf09 to resolve a problem. Is there a way to track what was the linking problem?

it's outdated but the PUBLIC link is explicitly changed in https://github.com/ROCmSoftwarePlatform/MIOpen/commit/ec3fdbe6b907f2b6599fc8de366d41a67eeaaf09 to resolve a problem.

My assumption is that it was made PUBLIC for the tests, which use miopen's internal headers. Thats why I used target_internal_library so it will still link for the tests.
However, miopen's public header doesn't include Boost.Interprocess(nor any headers from librt), so users of miopen shouldn't need to link with librt explicitly. The runtime loader will automatically load librt with miopen automatically.
For the static library, cmake will propagate the dependency, which means it would still use the absolute path. Maybe it would be better to use -lrt instead and let the linker find the library.
Either way, this at least fixes the issue for shared library case, and we could try to fix static library case in the future.

@junliume @pfultz2 @JehandadKhan I totally agree with Paul.


Equivalent to Haskell's Enum typeclass?

I recently pulled in bs-deriving to generate some enum helpers for a polyvariant type I was using to model a multi select.
Which was fine till I upgraded bs-platform to 7.2.2 which the ppx doesn't work for which got me wondering why Bastet doesn't have an implementation for the Enum typeclass?

It would be good to get Enum and BoundedEnum in, it looks like it would be pretty straightforward to implement -- just add succ, pred and the Verify laws which are straightfoward:
https://pursuit.purescript.org/packages/purescript-enums/4.0.1/docs/Data.Enum#t:Enum
I probably won't have the bandwidth to work on this any time soon though. PR are very welcome though!

Turns out it's in Relude so I think we can just wait for them to upstream their Interfaces work apparently? @mlms13 correct me if I'm misquoting.

If you're open to a PR, we have ENUM, BOUNDED_ENUM, LOWER_BOUNDED, and UPPER_BOUNDED in Relude, and I've been meaning to send them your way. I'll put together a PR in just a bit.

Nice! yeah sure thing


Driving Smarts 2023 environments

Added new driving-smarts-v2023 benchmark
Added new platoon-v0 env
Modified naming of benchmark used in NeurIPS 2022 from driving-smarts-competition-env to driving-smarts-v2022

since we havent officially announced this, would it better if we name the PR and the env name differently until we launch this later?

since we havent officially announced this, would it better if we name the PR and the env name differently until we launch this later?

This is simply a new set of enviroments which we are adding into SMARTS. It could be used for any purpose. I guess it is better to simply add it in now rather than later. Users are free to explore and start early.


[WIP] Fix flacky docker login

What does this PR do?

Fix flacky docker login
Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

/azp list

/azp run PR Changes


Cloud Logging Client: Exception Stack Trace is not recognized as such and not sent to Google Error Reporting

Using "google/cloud-logging": "^1.21"  /   PHP 7.1.33   /  Symfony 3.4
Docs says that Error Reporting can be fed from Cloud Logging.
I had a hard time trying to make it work. Docs re not very helpful in regards that for example here: https://cloud.google.com/error-reporting/docs/formatting-error-messages#json_representation they show the format but they don't mention that for PHP and PSR Logger, the information should go inside $context.
There's an exception coming in my service and I saw that the Google library has the following:
       if (isset($context['exception'])
            && ($context['exception'] instanceof \Exception || $context['exception'] instanceof \Throwable)) {
            $context['exception'] = (string) $context['exception'];
        }

Docs state that:
Log entries in Cloud Logging that contain stack traces or exceptions, or that are formatted like ReportedErrorEvent, generate errors in Error Reporting.
However this turned out to be false.
As currently is, the Cloud Logging logger would generate the following for an Exception:
{
  "insertId": "4zyvczg1dwcyas",
  "jsonPayload": {
    "message": "Uncaught PHP Exception Symfony\\Component\\Debug\\Exception\\ContextErrorException: \"Notice: Undefined index: track\" at /var/www/siembro_api_ar/src/AppBundle/Handler/Api/RiskHandler.php line 822",
    "exception": "Symfony\\Component\\Debug\\Exception\\ContextErrorException: Notice: Undefined index: track in /var/www/siembro_api_ar/src/AppBundle/Handler/Api/RiskHandler.php:822\nStack trace:\n#0 /var/www/siembro_api_ar/src/AppBundle/Controller/ApiFront/v2/RiskController.php(154): AppBundle\\Handler\\Api\\RiskHandler->getTrackByAttributes(Array)\n#1 /var/www/siembro_api_ar/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/HttpKernel.php(151): AppBundle\\Controller\\ApiFront\\v2\\RiskController->getTrackByAttributesAction(Object(Symfony\\Component\\HttpFoundation\\Request))\n#2 /var/www/siembro_api_ar/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/HttpKernel.php(68): Symfony\\Component\\HttpKernel\\HttpKernel->handleRaw(Object(Symfony\\Component\\HttpFoundation\\Request), 1)\n#3 /var/www/siembro_api_ar/vendor/symfony/symfony/src/Symfony/Component/HttpKernel/Kernel.php(200): Symfony\\Component\\HttpKernel\\HttpKernel->handle(Object(Symfony\\Component\\HttpFoundation\\Request), 1, true)\n#4 /var/www/siembro_api_ar/web/app_stage.php(29): Symfony\\Component\\HttpKernel\\Kernel->handle(Object(Symfony\\Component\\HttpFoundation\\Request))\n#5 {main}"
  },
  "resource": {
    "type": "global",
    "labels": {
      "project_id": "stage-siembro"
    }
  },
  "timestamp": "2021-07-23T03:14:25.832531Z",
  "severity": "CRITICAL",
  "labels": {
    "context": "request"
  },
  "logName": "projects/stage-siembro/logs/panel-ar",
  "receiveTimestamp": "2021-07-23T03:14:25.878451489Z"
}

However this code, even if having a trace available, is not recognized and not sent to Error Reporting.
Even adding the following inside jsonPayload didn't work:
serviceContext: {
  service: "panel-ar"
  version: 1
}

In the end what did work was to set the @type inside jsonPayload
@type: "type.googleapis.com/google.devtools.clouderrorreporting.v1beta1.ReportedErrorEvent"
Obviously having the serviceContext helped group the traces by service name.
To summarize: The library is not generating stack traces for Cloud Logging that Cloud Logging recognizes as such.

@JorgeSivil its already very late for us to respond on this issue and some improvements have happened since.
Pls consider looking at https://github.com/vishwarajanand/php-docs-samples/blob/e5ddf59ee4a6a8f9ff78c3b582eb6e78d106769a/error_reporting/src/report_error.php#L41 for the usecase. I ran it ina script and it worked.


Emit Sendable diagnostics in actor-like deinits only in 'complete' checking mode

The flow-isolation pass was not respecting the new strict-concurrency checking mode.
Since the Sendable diagnostics in these deinits are very noisy, I'm moving them to only
be emitted in 'complete' mode. The reason why they're so noisy is that any class that
inherits from a @MainActor-constrained class will have these diagnostics emitted when
trying to access its own @MainActor-isolated members.
This is needed, even during the deinit, because multiple instances of a @MainActor-isolated
class might have stored properties that refer to the same state.
This change specifically avoids emitting these diagnostics even in 'targeted' mode because
I'd like to take more time to reconsider the ergonomics of these deinits.
resolves rdar://94699928

@swift-ci please test

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge


(Reporter) Send klay back to delegator from early stage reporters (when it was part of oraklnode)

done
https://klaytnscope.com/tx/0xe33e0f9b493e16422cca6d1c2aa5f098ada19d9855cac5b2e61a3fb6efaddc17?tabId=internalTx
https://klaytnscope.com/tx/0x527d753d62daa5936dc0e01954d0b7e79276ca4c4ef40263c7fe527ef9db8778?tabId=internalTx


Support certPath, tlsVerify, and apiVersion when using Docker Remote API

Have the docker task support inputs certPath and tlsVerify. These are useful for using, for example, minikube's Docker daemon as the remote, which would normally have these environment variables set:
DOCKER_TLS_VERIFY=1
DOCKER_HOST=tcp://<IP_ADDRESS>:2376
DOCKER_CERT_PATH=/home/me/.minikube/certs
DOCKER_API_VERSION=1.23

docker {
    useApi true
    tlsVerify '1' // <<<<<<<
    hostUrl 'tcp://<IP_ADDRESS>:2376'
    certPath '/home/me/.minikube/certs' // <<<<<<<
    apiVersion '1.23' // <<<<<<<
    apiUsername 'user'
    apiPassword 'password'
    apiEmail<EMAIL_ADDRESS>}

Thanks for reporting this.  This has been an long time issue,  and should definitely be fixed


Mike

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

hi there @mmassare -- it looks like you removed the pixel you were trying to contribute, in your most recent commit. Would you mind fixing that and then I'll take another look? Thanks!

I moved it to row 19, as row 40 threw an error...
On Thu, Oct 10, 2019 at 8:21 PM Tilde Ann Thurium<EMAIL_ADDRESS>wrote:

hi there @mmassare https://github.com/mmassare -- it looks like you
removed the pixel you were trying to contribute, in your most recent
commit. Would you mind fixing that and then I'll take another look? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/twilio-labs/open-pixel-art/pull/989?email_source=notifications&email_token=ANMUDGFD3WGSSJBMR4DBXMTQN7BG3A5CNFSM4I7TIKCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA6LM2Y#issuecomment-540849771,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANMUDGCCV3NDQDECBGIBSB3QN7BG3ANCNFSM4I7TIKCA
.


Wrap MLIR SparseUtils.cpp with Cython

Currently minimally wrapped.

btw @seibert and @paul-tqh-nguyen, I updated build_sparse_tensor to be able to build any combination of types for the pointers, index, and values arrays.  It still simply returns an integer pointer.  Let me know if it would be helpful to have a proper Cython class for the SparseTensorStorage object.


No error when trying to leave comment on DAO you aren't a memeber of

I think it would be better if the comment section was greyed out completely, and it shows a message "add {dao_name} tokens to leave a comment"

Lets make it more verbose at the API level
"You are not a member of this DAO. Add {dao_name} tokens to your wallet to participate"

Also it let me make a comment without having dao tokens


Hifive1 RevB defmt hello.rs

I'm getting linking errors during cargo build --bin hello
error: linking with `rust-lld` failed: exit code: 1
  |
  = note: "rust-lld" "-flavor" "gnu" "-L" "/home/dkhayes117/.rustup/toolchains/beta-x86_64-unknown-linux-gnu/lib/rustlib/riscv32imac-unknown-none-elf/lib" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o" "-o" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350" "--gc-sections" "-L" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps" "-L" "/home/dkhayes117/IdeaProjects/defmt_test/target/debug/deps" "-L" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/build/defmt-786b78cb5fb56c90/out" "-L" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/build/hifive1-12e6de30a7c42179/out" "-L" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/build/e310x-c9d6b8bcdebcec36/out" "-L" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/build/riscv-b96e58c41469ec83/out" "-L" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/build/riscv-rt-7fa43864cbb5bfaa/out" "-L" "/home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/build/riscv-rt-7fa43864cbb5bfaa/out" "-L" "/home/dkhayes117/.rustup/toolchains/beta-x86_64-unknown-linux-gnu/lib/rustlib/riscv32imac-unknown-none-elf/lib" "--start-group" "-Bstatic" "/tmp/rustcQiAIxg/libriscv_rt-c7d2a202c972997c.rlib" "/tmp/rustcQiAIxg/libriscv-6fb334952367127a.rlib" "--end-group" "/home/dkhayes117/.rustup/toolchains/beta-x86_64-unknown-linux-gnu/lib/rustlib/riscv32imac-unknown-none-elf/lib/libcompiler_builtins-c1fdd1f60c58bc16.rlib" "-Bdynamic"
  = note: rust-lld: error: undefined symbol: _mp_hook
          >>> referenced by lib.rs:301 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/riscv-rt-0.6.2/src/lib.rs:301)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)

          rust-lld: error: undefined symbol: __pre_init
          >>> referenced by lib.rs:302 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/riscv-rt-0.6.2/src/lib.rs:302)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)

          rust-lld: error: undefined symbol: _sbss
          >>> referenced by lib.rs:302 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/riscv-rt-0.6.2/src/lib.rs:302)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)
          >>> referenced by lib.rs:302 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/riscv-rt-0.6.2/src/lib.rs:302)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)

          rust-lld: error: undefined symbol: _ebss
          >>> referenced by lib.rs:302 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/riscv-rt-0.6.2/src/lib.rs:302)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)
          >>> referenced by lib.rs:302 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/riscv-rt-0.6.2/src/lib.rs:302)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)

          rust-lld: error: undefined symbol: _sdata
          >>> referenced by lib.rs:0 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/r0-0.2.2/src/lib.rs:0)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)
          >>> referenced by lib.rs:0 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/r0-0.2.2/src/lib.rs:0)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)

          rust-lld: error: undefined symbol: _sidata
          >>> referenced by lib.rs:0 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/r0-0.2.2/src/lib.rs:0)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)
          >>> referenced by lib.rs:0 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/r0-0.2.2/src/lib.rs:0)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_rust)

          rust-lld: error: undefined symbol: trap_handler
          >>> referenced by lib.rs:329 (/home/dkhayes117/.cargo/registry/src/github.com-1ecc6299db9ec823/riscv-rt-0.6.2/src/lib.rs:329)
          >>>               /home/dkhayes117/IdeaProjects/defmt_test/target/riscv32imac-unknown-none-elf/debug/deps/hello-a9b45fbc99b62350.hello.1n7pvg5w-cgu.0.rcgu.o:(_start_trap_rust)

          rust-lld: error: undefined symbol: _max_hart_id
          >>> referenced by riscv-rt.o:(.init+0x52) in archive /tmp/rustcQiAIxg/libriscv_rt-c7d2a202c972997c.rlib
          >>> referenced by riscv-rt.o:(.init+0x56) in archive /tmp/rustcQiAIxg/libriscv_rt-c7d2a202c972997c.rlib

          rust-lld: error: undefined symbol: _stack_start
          >>> referenced by riscv-rt.o:(.init+0x5E) in archive /tmp/rustcQiAIxg/libriscv_rt-c7d2a202c972997c.rlib

          rust-lld: error: undefined symbol: _hart_stack_size
          >>> referenced by riscv-rt.o:(.init+0x66) in archive /tmp/rustcQiAIxg/libriscv_rt-c7d2a202c972997c.rlib
          >>> referenced by riscv-rt.o:(.init+0x6A) in archive /tmp/rustcQiAIxg/libriscv_rt-c7d2a202c972997c.rlib

error: aborting due to previous error

error: could not compile `defmt_test`.

To learn more, run the command again with --verbose.

defmt currently doesn't support risc-v, but let's keep this issue open to see if/when we want to change that :)

The template currently does not provide risc-v support. keeping this issue open as a reminder if/when we want to change that. :)


Add exception to prevent invalid configuration

manifest and css path(s) cannot/should not be both defined.
Addresses #45

Doesn't solve the issue - just tried with this specific commit : ( - but still getting this:
Error: ENOENT: no such file or directory, open '/assets/application-fe612b1419b5a8d61dfc3dbc0fe5e5e58683baab1701e8e5870e475209314419.css'
Failed to get CSS for route /
  with options={"url"=>"http://localhost:3000/", "css"=>"/assets/application-fe612b1419b5a8d61dfc3dbc0fe5e5e58683baab1701e8e5870e475209314419.css", "width"=>1300, "height"=>900, "timeout"=>30000, "forceInclude"=>[], "strict"=>false, "maxEmbeddedBase64Length"=>1000, "userAgent"=>"Penthouse Critical Path CSS Generator", "renderWaitTime"=>100, "blockJSRequests"=>true, "customPageHeaders"=>{"Accept-Encoding"=>"identity"}}

Doesn't solve the issue - just tried with this specific commit : ( - but still getting this:
Error: ENOENT: no such file or directory, open '/assets/application-fe612b1419b5a8d61dfc3dbc0fe5e5e58683baab1701e8e5870e475209314419.css'
Failed to get CSS for route /
  with options={"url"=>"http://localhost:3000/", "css"=>"/assets/application-fe612b1419b5a8d61dfc3dbc0fe5e5e58683baab1701e8e5870e475209314419.css", "width"=>1300, "height"=>900, "timeout"=>30000, "forceInclude"=>[], "strict"=>false, "maxEmbeddedBase64Length"=>1000, "userAgent"=>"Penthouse Critical Path CSS Generator", "renderWaitTime"=>100, "blockJSRequests"=>true, "customPageHeaders"=>{"Accept-Encoding"=>"identity"}}

Same here. This is merged to 4.0.1, right?


Error when disconnected activity inside a FlowChart node

I found that when we have a workflow that act as Activity and is disconnected from all activities, we cannot see the instance in the Instance view.
It throw an "The node must be of type 'JsonObject'."

For more info:

I tried to reproduce the problem as you described, but for me it seems to work just fine. Its possible that this got fixed since you reported it, so I will close it as "cannot reproduce". If you are still able to reproduce this issue with the latest version, then please feel free to reopen it. Thank you.


Refactor using a base widget class

For now, it's working for the rotational and dial gauges. Dial needs a bit more work.
All gauges should be functional.

@caguero is this ready for review?

Not yet.

@caguero is this ready for review?

Not yet.

Good to go!

In the dial the units are not working
the minimum value in the rotational widge should be -45 and not 0

It should be fixed in d5ebb40

min value in rotational is not working
min and max values not working on bar gauges

fcd44f5

I tweaked the limits in some widgets. Could you check now?


Cannot convert GET to POST in a filter

Currently filters can modify HeaderMap / Buffer in decode{Headers,Body,Trailers}, but the end_stream flag cannot be changed.
The use case for this is for REST-gRPC transcoding, when a gRPC method is mapped to HTTP GET, there is no API to convert it to POST (gRPC is always POST) with a request body in a filter.

Hmm. This is going to be quite complicated to fix. Let me think about this and maybe you and I can chat over VC briefly next week.

Sounds good.

@lizan can you clarify a bit more on the flows you need here? I assume that you need basically this:

Conversion filter receives decodeHeaders(headers, true) // header only get
You want to convert GET to POST, and add a body.
You want router/other filters to see:

decodeHeaders(headers, false)
decodeData(body, true)

Do you need streaming support? Continuation support? Or will you always add a body and then continue iteration.
I've been thinking about and I don't think it will be too bad to add this, but would like to clarify exactly what you need.

Yes that is correct.
For REST-gRPC transcoding use case, there is no need of streaming, continuation support. The request body (gRPC frame) for router/other filters will be generated from headers.

OK sounds good. I will work on this in the next week.

I'm very, very excited for this feature. I think it's actually a killer feature for Envoy/gRPC. :)

@lizan what about the response. What will you do with the trailers? Just send them along? Or do we need a way to zero them out and send an empty data frame or something.

Ah yes, I need to clear trailers.
For non-streaming response, current existing implantation is wait until trailers and send response to upstream, so grpc-status in trailers can be reflected to HTTP status.
For streaming response, sending an empty data frame should be good.

Yes, this is what happens here: https://github.com/lyft/envoy/blob/master/source/common/grpc/http1_bridge_filter.cc
I think for right now, the trailers could just be emptied and it would probably just work. If we need to actually get rid of the trailers I think I can do it fairly easily in a follow up. I'm fixing the primary issue with request body now.

@mattklein123 I would like to re-open this issue, #633 doesn't work in real case:

router returns StopIteration even it was successful.
https://github.com/lyft/envoy/blob/master/source/common/router/router.cc#L252

when ConnectionManager get StopIteration it doesn't continue on data.
https://github.com/lyft/envoy/blob/master/source/common/http/conn_manager_impl.cc#L490

Changing router to return Continue at that line fixed this (and doesn't break tests!), though I am not sure why router is returning StopIteration at the end of decodeHeaders, and if that is a proper way to fix.

@lizan reopening. I will fix.


151 | Reverse Words in a String | Java

Reverse Words in a String -
I would like to add a solution to this problem Reverse Words in a String
Adding Solution to Problem 151 -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[] Improving Code
[] Improving Documentation
[] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[] Python
[] C++
[x] Java
[] SQL

@SSKale1 please assign me to this task. I am ready to take on the challenge

Could you please assign me this issue?

@SSKale1 could I be assigned this?

@SSKale1 Thanks for the assignment. Will knock it out

👍


Spring6-Scheduled-null

Describe the bug
Spring 6 @Scheduled entpoint trace.
what i see in ui:

端点:SpringScheduled/null
跨度类型:Local
组件:SpringScheduled

Why SpringScheduled/null ?
Steps to reproduce
kotlin: 2.0
spring-boot: 3.2.3
spring: 6.1.4
jdk: 21
skywalking: 9.1.0
sample code:
    @Scheduled(fixedRate = 5, timeUnit = TimeUnit.SECONDS, scheduler = "myScheduler")
    fun method1(){
      ...
    }

Could you provide the agent debug logs for your application?
Normally, @Scheduled transactions should just have the method name as transaction name.


IBM.com Library Services Package

vladislav-saling created the following on Apr 16:
This is the epic that will be capturing the creation of the core services necessary for the IBM.com Library Services package.
The dotcom services includes ES6 services that are specific to the dotcom experience.
Original issue: https://github.ibm.com/webstandards/digital-design/issues/726

On May 21, Jeff-Chew commented:
@hahnrob @vladislav-saling we should have this particular story only focus on the audit of background services, and what we should identify as stories we need to create to migrate these services. Ideally this should be a spreadsheet or airtable.

On May 22, vladislav-saling commented:
@Jeff-Chew we've already did an audit and we've identified services that are needed for Masthead and Footer. See issues that fall under this epic. Other background services (e.g. chat services) in v18 should be migrated only if needed.
cc @james-dow @Kenny-Lam


'oldtitle' does not validate

I'm sorry if I missed this in the docs. I just used the W3C-Validator to check a page and got lots of errors because of the 'oldtitle' attribute. I am not looking for option 'suppress', instead I'd like to suggest to use 'data-oldtitle' instead of 'oldtitle'. (Or allow to set the attribute name to store the old title.)

The attribute is added dynamically to the page and as such isn't really an issue to be valid. Only HTML there on page load is the stuff you should be concerned about imo.

I patched my min.js to use data-oldtitle (there's only one occurance), seems to work perfectly. Maybe you could consider this suggestion for a future release.


feat: added a static quiz model

Added a static quiz model based on an earlier opened 'good-first-issue' request.

@nativekar can you explain what actually does your code do?


Add support for node 12.x

AWS released support for Node 12.x yesterday:
https://aws.amazon.com/blogs/compute/node-js-12-x-runtime-now-available-in-aws-lambda/
This PR was modeled after the Node 10.x upgrade (#50).

Thanks


SyntaxError: missing } after property list

Its working great but i get this error on the console.  What am i missing thanks..
SyntaxError: missing } after property list
here is how i set it up ( i have 5 inputs so i did it 5times with dif id's)
/* color picker */
$("#colorpicker").spectrum({
    color: "#000000"
    showInput: true
});

$("#colorpicker2").spectrum({
    color: "#ffffff"
    showInput: true
});

$("#colorpicker3").spectrum({
    color: "#246893"
    showInput: true
});

$("#colorpicker4").spectrum({
    color: "#000000"
   showInput: true
});

$("#colorpicker5").spectrum({
    color: "#ffffff"
    showInput: true
});

thanks

if i add the comma after the attribute like so
$("#colorpicker5").spectrum({
color: "#ffffff",
showInput: true
});
it fixes the error but now the performance is horrible because it does not show the big color palette window like before.

Hi, you will need the comma - the syntax error is in your example code.  See a working demo with a couple of examples here: http://jsfiddle.net/bgrins/ctkY3/.

Closing the issue, but if you are seeing bad performance, please reopen and link to a test case so I can take a look at it.


Allow range requests to run on separate interface / port

The Issue
Currently, the range requests API runs on the 5279 port. This prevents someone from opening up range requests on a public port (i.e. lbry.tv) without also accepting incoming RPC requests. It should be possible to run it on a separate interface.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

This was completed in 0.37.0


Update documentation according to opts updates (#13694)

With #13694 two small things have changed :

--env and --envfile are now validated according to the following definition : "Environment variables set by the user must have a name consisting solely of alphabetics, numerics, and underscores - the first of which must not be numeric.".
It's possible to use -v containerPath:ro as option for run (and create, …).

I realized I did not think of the documentation when submitting #13694, so this ones try to update the few things it should.

Add a Note on the env validation
Add an example of -v containerPath:ro
Update the link to the PR for volume management (#14214)

Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

@thaJeztah @moxiegirl @jamtur01 @fredlf

@vdemeester added some more notes (sorry, I was called away during reviewing)
Question; https://github.com/docker/docker/pull/13694 is not yet in a release, or is it? If not, I think we should add "impact" labels ("impact/changelog" and "impact/ui"), because the validation-rules changed?

@thaJeztah It will be in 1.8.0 I believe but yeah, it's not in release 1.7.x.
I agree with you on the impact/* label on the PR. The validation rules changed a little bit but changed, so yeah :bow:.

Thanks, LGTM!


URL containing name of the property in defined response returns empty array || object

With giving json-server setup after hitting the route "/api/v1/documents?docId=123" I am getting [] array as response.
Everything works fine when hitting "/doc" to inspect the response from json-server UI.
When I rename docId to something else everything is working fine. Same goes with if I rename url not to contain docId
Version: 0.14.2
db.js
module.exports = () => ({   doc: require('./doc.json'), })
routes.json
{ "/api/v1/documents?docId=:Id": "/doc" }
doc.json
[ { "name": "some name", docId: 123 } ]
Starting command >>> json-server -c json-server.json mockdb/db.js

Also have the same issue with property names inside an array response. For me changing the name inside an existing api just to use it as a fake backend is not an option. Would appreciate an fix or any kind of news if this behaviour is intended.


Help to fix Compile Error

Hi, I'm trying to build the project.
However, there is an syntax error when type make.
Here is the error message.
File "lib/mutable_store.ml", line 71, characters 13-14: 71 |   let module _ = Make (Immutable_store) in                           ^                                     Error: Syntax error                                     make: *** [Makefile:3: build] Error 1
My environment is as follows
OS : Ubuntu 20.04
OCaml Version : 4.09.0
OPAM Version : 2.0.5
Thanks.

It works at 4.10.1.


Allow overriding underlying native component

I'm using react-native-text-input-mask, which provides a native component that's ultimately a wrapper around React Native's TextInput (with a few additional props, and a different onChangeText signature). I want to use it with NativeBase, as well as all my theme config, but I'm finding that i'm having to reimplement the majority of the Input component, including a whole bunch of private APIs that aren't exposed.
I'd be interested in a mechanism that lets us override the underlying native component, something like:
<Input baseComponent={TextInputMask} /* other props */ />

Hi @AndrewIngram, Thanks for reporting the issue. We will look into it.


test data ground truth in the dataset folder

Hi:
I downloaded the dataset from https://xglue.blob.core.windows.net/xglue/xglue_full_dataset.tar.gz.
For question generation task I find files like:
xglue.qg.de.tgt.test
xglue.qg.en.tgt.test
Are these the ground truth of the:
xglue.qg.de.src.test
xglue.qg.en.src.test
I am asking because I wonder if I can do my own evaluation by comparing my result with these files.
Regards!
Wei

Yes, you could do your won evaluation.
We just decide to release to the test data for better results reproducing.

Yes, you could do your won evaluation.
We just decide to release to the test data for better results reproducing.

Thank you!
I asked because these ground truth files are not stated in the Data_Format.txt
I just want to make sure the result evaluated from this ground truth is comparable with the baseline results.
Thank you for the timely response.

Thank you for your information! I will update the data format file later.


New Logo for Juice Shop [$10]

Background Story
Translating "dump" or "useless outfit" into German yields "Saftladen" which can be reverse-translated word by word into "juice shop". Hence the name of this project. Additionally the initials match to "JS" as in "Javascript". So this is what I would like the logo to resemble around:

Juice
Javascript
JS

Design Idea:
My idea is having something similar to one of the inofficial JS-shield logos but make it look a bit like a leaking juice pack/glass/bottle with a puddle of juice beneath it.
This would point out how broken the application is and at the same time keep the resemblance with well-known Javascript logos.

---
There is a **[$10 open bounty](https://www.bountysource.com/issues/7953454-new-logo-for-juice-shop-10?utm_campaign=plugin&utm_content=tracker%2F6283055&utm_medium=issues&utm_source=github)** on this issue. Add to the bounty at [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F6283055&utm_medium=issues&utm_source=github).

Has been implemented with #111. To be closed when original designer claimed bounty.


Add skeleton for hugr dialect

This looks large, and it is, but it's mostly cmake and boilerplate.
I have disabled zlib and terminfo in the LLVM derivation, as an alternative to working out why cmake can't find them when run in the nix shell.
Tests, which now include:

checking the mlir derivation
In both the build and release trees:

Checking hugr-mlir-opt can handle the skeleton dialect
Checking the C api bindings work
Checking the python bindings work

nix linting
C++ linting
are all expected to pass. I've not seen green CI yet, so I may yet have to push small fixes.

Docs, which consist only of generated documentation, should be verified in an install tree.
@aborgna-q Can you review?

Thanks @aborgna-q , I will make the remaining fixes in a followup as discussed


Add Rango Exchange

Revision checks

[x] Used smart contracts were audited.
[x] Your Safe App include a manifest.json file at the root with the required data.
[x] The app can be loaded as a custom Safe App in the Apps section of https://app.safe.global.
[x] The app auto-connects to the Safe.
[x] You are able to trigger and execute one transaction with a Safe.
[x] RPC requests are optimized (not triggering many requests in a very short time period).

Name/Description
Name: Rango Exchange
Description: Cross-Chain Dex & Bridge Aggregator
Type

New addition

Compatible Networks

Mainnet
Arbitrum
Optimism
BSC
Polygon
Avalanche
Linea
Polygon zkEVM

Audit document
Audit by Peckshield
Code for review
https://github.com/rango-exchange/rango-client/
https://github.com/rango-exchange/rango-client/tree/next/wallets/provider-safe
IPFS hash/App URL
https://app.rango.exchange/
Team information

- Project logo: https://rango.exchange/logo.png
- Project landing page: https://rango.exchange/
- Team or discord contact: https://discord.com/invite/q3EngGyTrZ
- Twitter link: https://twitter.com/rangoexchange
- Github link: https://github.com/rango-exchange/

Hey team,
It would be really appreciated if you can provide any update regarding this PR.
Thanks in advance.

Hey @RanGojo, the app has been approved from the product perspective, so we'll be doing a technical review and will likely enable the app later this week. Cheers!

I've tested the app in this Safe: https://safe-wallet-web.dev.5afe.dev/apps/open?safe=eth:0xA77DE01e157f9f57C7c4A326eeE9C4874D0598b6&appUrl=https%3A%2F%2Fapp.rango.exchange

Blocker: It doesn't seem to auto-connect to the safe as a wallet.
Even though the Safe is on mainnet, it offers me to swap BNB for AVAX – is that the intended behavior?

@katspaugh Hi there!
The initial problem has been resolved. It occurred because the safe SDK was restricted to specific domains, a limitation that has now been removed. (https://github.com/rango-exchange/rango-client/commit/366206aa8b5a6796fdd864681946f2e06ba396c8)
Regarding your second point, yes, it was done intentionally.

Can you clarify how such a swap would work in a mainnet Safe? As you might know a Safe is deployed on a specific chains and may not be deployed or have the same address on other chains.

Yeah, sure. @katspaugh
The user experience of the Rango Dapp differs from typical decentralized applications (dapps) where users connect a single wallet and execute a single-step transaction. In the provided gif, you can observe that Rango offers the capability to connect multiple wallets and engage in multi-step transactions. Users have the flexibility to choose the wallet for each blockchain before initiating the swap. For instance, in the presented example, since Safe is exclusively deployed on Polygon, the user can only select it for Polygon, while opting for other wallets for other blockchains.

I see, thanks! I’ll give it another test tomorrow and if everything is fine I’ll enable the app. 👍

@katspaugh Thanks that would be great.

It auto-connects now but I can't seem to confirm a swap even after pasting an EoA receiver.

Also, where do you source the SAFE token value from? It's not transferrable atm and has no price, so this is incorrect.

Closing for inactivity, but please feel free to reopen if you fix the app. Cheers.

Sorry for delay, we were waiting for our new dapp which is just released. So the user experience is more understandable there. (It's live at the same url)
https://app.rango.exchange/
It would be great if you check our app again:

regarding the safe token, it's not removed, it was related to one our our third party token price provides which we where using and we removed that.
regarding the first question, the reason was that the route you selected need another second transaction on destination chain, so providing a custom address for destination blockchain was not enough.

Please let me know if you face any other problems. Thanks in advance. @katspaugh @kirkkonen

@katspaugh Please reopen the issue and proceeda if possible.

Cool, thank you! I’ll test again.

@katspaugh It would be great if you could test our PR. Thanks in advance.


Add Scalingo builds

Sister PR (branch with same name) : https://github.com/tchapgouv/matrix-react-sdk/pull/371 (not needed any more)
Scalingo machine : tchap-web-staging

This PR enables to build and run tchap-web on scalingo, with all deps in package.json (no yarn links). matrix-react-sdk is specified from a github branch, and matrix-js-sdk as a "standard" package.
Out of scope : build with matrix-js-sdk from a github branch. Because this is only needed for v2, and hopefully not v4.
Note : the scalingo env var NPM_CONFIG_PRODUCTION=false is needed so that Scalingo installs the devdependencies. We need the devdependencies in matrix-react-sdk to build it.
We could try to find something smarter where the devdependencies of tchap-web are not installed...
Todo

[x] cleanup : instead of renaming the "build" command and editing the "package.sh" script, make separate commands and configure scalingo to run them (right now "build" is the default so scalingo runs it)
[x] NPM_CONFIG_PRODUCTION, see above. If we can't get rid of it, it would be better to place it in a config file, and document it.
-> replaced by a production=false flag in the appropriate build command.
[x] I also removed the "prepare" script from matrix-react-sdk, because it was automatically run after yarn install, and crashed. It crashed because of the git rev-parse command : fatal: not a git repository (or any of the parent directories): .git. The easiest would be to replace that git rev-parse with a script which fetches the version from package.json, the way it is done in scripts/package.sh.
-> Fixed : prepare is actually not run for the subpackage matrix-react-sdk, juste for tchap-web package. So we dodge that problem :)
[x] if matrix-react-sdk is a standard version instead of a pointer to a git branch, does the whole thing still work ? -> looks like it. We don't use this case anyway since we don't do npm releases (for now)

Ready for review @jdauphant

Hmm, du coup on doit dépendre d'un js-sdk d'une branche github, j'avais oublié ce détail.
Ok, je regarde ca.


Automated cherry pick of #5529: fix: 避免重装系统失败后imageId依然发生改变

Cherry pick of #5529 on release/3.0.
#5529: fix: 避免重装系统失败后imageId依然发生改变

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.0@f51e7d0). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release/3.0   #5531   +/-   ##
=============================================
  Coverage               ?   6.85%
=============================================
  Files                  ?     535
  Lines                  ?   91147
  Branches               ?       0
=============================================
  Hits                   ?    6251
  Misses                 ?   84376
  Partials               ?     520

Flag
Coverage Δ

#aFlag
6.85% <ø> (?)

Continue to review full report at Codecov.

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

/lgtm

/approve


AzureChatOpenAI Streaming causes IndexError: list index out of range

System Info
langchain-0.0.205-py3, macos ventura, python 3.11
Who can help?
@hwchase17 / @agola11
Information

[x] The official example notebooks/scripts
https://python.langchain.com/docs/modules/model_io/models/chat/how_to/streaming

Related Components

[X] LLMs/Chat Models

Reproduction
Reproduction code
# test.py
from langchain.chat_models import AzureChatOpenAI
from langchain.chat_models import ChatOpenAI
from langchain.callbacks.streaming_stdout import StreamingStdOutCallbackHandler
from langchain.schema import (
    HumanMessage,
)

chat_1 = ChatOpenAI(streaming=True,
                  callbacks=[StreamingStdOutCallbackHandler()],
                  openai_api_key="SOME-KEY",
                  model='gpt-3.5-turbo',
                  temperature=0.7,
                  request_timeout=60,
                  max_retries=1)

chat_2 = AzureChatOpenAI(streaming=True,
                       callbacks=[StreamingStdOutCallbackHandler()],
                       openai_api_base="https://some-org-openai.openai.azure.com/",
                       openai_api_version="2023-06-01-preview",
                       openai_api_key="SOME-KEY",
                       deployment_name='gpt-3_5',
                       temperature=0.7,
                       request_timeout=60,
                       max_retries=1)

resp_1 = chat_1([HumanMessage(content="Write me a song about sparkling water.")])
resp_2 = chat_2([HumanMessage(content="Write me a song about sparkling water.")])

python test.py

Output of command 1 (OpenAI)
Verse 1:
Bubbles dancing in my cup
Refreshing taste, can't get enough
Clear and crisp, it's always there
A drink that's beyond compare

Chorus:
Sparkling water, oh how you shine
You make my taste buds come alive
With every sip, I feel so fine
Sparkling water, you're one of a kind

Verse 2:
A drink that's light and calorie-free
A healthier choice, it's plain to see
A perfect thirst quencher, day or night
With sparkling water, everything's right

Chorus:
Sparkling water, oh how you shine
You make my taste buds come alive
With every sip, I feel so fine
Sparkling water, you're one of a kind

Bridge:
From the fizzy sensation to the bubbles popping
You're the drink I never want to stop sipping
Whether at a party or on my own
Sparkling water, you're always in the zone

Chorus:
Sparkling water, oh how you shine
You make my taste buds come alive
With every sip, I feel so fine
Sparkling water, you're one of a kind

Outro:
Sparkling water, you're my go-to
A drink that always feels brand new
With each sip, I'm left in awe
Sparkling water, you're the perfect beverage

Output of command 2 (Azure OpenAI)
raw.Traceback (most recent call last):
  File "/Users/someone/Development/test.py", line 29, in <module>
    resp_2 = chat_2([HumanMessage(content="Write me a song about sparkling water.")])
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/langchain/chat_models/base.py", line 208, in __call__
    generation = self.generate(
                 ^^^^^^^^^^^^^^
  File "/opt/homebrew/lib/python3.11/site-packages/langchain/chat_models/base.py", line 102, in generate
    raise e
  File "/opt/homebrew/lib/python3.11/site-packages/langchain/chat_models/base.py", line 94, in generate
    results = [
              ^
  File "/opt/homebrew/lib/python3.11/site-packages/langchain/chat_models/base.py", line 95, in <listcomp>
    self._generate(m, stop=stop, run_manager=run_manager, **kwargs)
  File "/opt/homebrew/lib/python3.11/site-packages/langchain/chat_models/openai.py", line 334, in _generate
    role = stream_resp["choices"][0]["delta"].get("role", role)
           ~~~~~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range

Expected behavior
I can't find anything in existing issues or documentation stating that there is a known bug in the AzureOpenAI Service Streaming.

@rock-you are you planning to create PR for this? Because I am also facing same issue when using streaming with azure openai.

I am also experiencing the same problem, and having this fix would be great

Does this occur when setting version to 2023-05-15?
It seems there's a change in the preview version - given that it's preview, it might be best to avoid making an unnecessary change here until the mainline version comes out:
https://github.com/Azure/azure-sdk-for-go/issues/21086

Any ideas on a solution here, while I understand the comment about it being preview, it does seem like its here to stay, as far as I can see this is a blocker for being able to test out using OpenAI Functions with streaming responses.

Unfortunately yeah - it's going to require some code changes. If you have time to submit a PR that'd be awesome, otherwise I can look tomorrow.

I had the same problem with version "2023-07-01-preview"

Opened up the fix above as a PR to hopefully help get this unblocked


[FEAT] add WizardCoder 34b

Is your feature request related to a problem? / 你想要的功能和什么问题相关？
This is an example
http://<IP_ADDRESS>:50085/
Describe the solution you'd like. / 你想要的解决方案是什么？
Python
C#
Javascript
Version
Describe alternatives you've considered. / 你考虑过的其他方案是什么？
http://<IP_ADDRESS>:50085/
Additional context / 其他信息
No response

You can try it through Gradio bot with fnIndex as 0

can u upload steps in video or md file or something
tnks

Sorry, I tried. It does not work.

However, I can't believe an IP service will be a stable service. We need to find a stable, long-term service of WizardCoder.


Adding partial object matching support.

Added an additional matcher similar to jasmine.objectContaining or the like that checks for matching of properties by name, but not necessarily all the properties of the object.

Updated README.md with some notes on deep and partial object matching.
Added tests for partial object matching alongside existing "explicit object param values" tests.

Happy to change the name of the matcher.. couldn't think of something better.

Great stuff, thanks. The naming of the new matcher works for me.
I'm going to release shortly a new version with your PR included.

Awesome! :)


Move step logic to helpers along with step refactor

Addresses issues with steps of VM migration details not to be displayed properly and not to be aligned with PipelineSummary.
Refactors Step component to be decoupled from VM status.

Some rebase conflicts here @gildub

@mturley, updated.


Default None for uuid property raises AttributeError (name collision)

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Declaring a pydantic model with a property named uuid and type uuid.UUID | None and a default value of None raises AttributeError: 'NoneType' object has no attribute 'UUID'.
This is probably true for other import/property name collisions as well.
Work arounds:

rename the property
remove the default value
change the import to from uuid import UUID

Example Code
import uuid

import pydantic

class Foo(pydantic.BaseModel):
    uuid: uuid.UUID | None = None

Python, Pydantic & OS Version
pydantic version: 2.0.2
        pydantic-core version: 2.1.2 release build profile
                 install path: /project/path/venv/lib/python3.11/site-packages/pydantic
               python version: 3.11.4 (main, Jul 11 2023, 22:38:37) [GCC 11.3.0]
                     platform: Linux-5.19.0-46-generic-x86_64-with-glibc2.35
     optional deps. installed: ['typing-extensions']

Selected Assignee: @adriangb

This isn't related to Pydantic, it has to do with how Python finds references to the type hints you are using. Foo has its own class scope, and when you are using the type hing uuid.UUID, it tries to resolve uuid in this class scope, which is set to None with the default value.
Either use from uuid import UUID or uuid: "uuid.UUID | None" = None

This isn't related to Pydantic, it has to do with how Python finds references to the type hints you are using. Foo has its own class scope, and when you are using the type hint uuid.UUID, it tries to resolve uuid in this class scope, which is set to None with the default value.
Either use from uuid import UUID or uuid: "uuid.UUID | None" = None

Exactly 👍 .


Performance profiling results

I was testing my own changes&performance and decided to share this as well.

This frame took 981ms GetFeature takes 153 of that. Few things to note;

First of all, I thought we were using clipper for 16+ levels. It seems like it's working at 16 as well? Do we need it at 16- levels?
Yes it's mostly clipper but still there are bunch of easy improvements for slow days
Where & Select, or let's say linq in general, little slow and makes probably unnecessary allocations
List size changes are actually costly, I haven't checked this one but at my side, I started calculating the size and creating them with that size which saved me from all those array copying.
ToList is extremely slow. And it's even following a select statement (for tags). Is ConvertAll  slower? (I couldn't re-test as profiler is actually quite slow in deep-profile mode)

I'll post if I can find anything else!
cc @BergWerkGIS

First of all, I thought we were using clipper for 16+ levels. It seems like it's working at 16 as well?

No, usage of clipper is based on the buffer parameter passed to eg GetFeature() or ToGeoJson().
Passing any not-null value triggers clipper.
We currently pass 0.

Do we need it at 16- levels?

Yes, with the current implementation of the mapbox-unity-sdk (no feature filtering or feature de-duping across tile borders) it is needed to prevent duplicates features.
All vector tiles z0 -> zMax contain data that extends beyond the actual tile borders.

You could speed things up a bit (10-30% depending on the data of the vt) by passing validate = false here or here.
With validate = false some extensive checks if the tile itself and its data is valid are skipped.

@BergWerkGIS is mapbox data always valid?

@brnkhy

is mapbox data always valid?

😄 yes it should always be valid.
I've enabled the validate flag as default mainly for data from other sources that users known what's going and don't think the this library is faulty.

@BergWerkGIS ah I see, it's good to know though as it's small but yet another point of perf improvement when someone asks. I know these are quite small compared to mesh gen performance issues but still it's good to keep track  ;)


PARQUET-1274: Prevent segfault that was occurring when writing a nanosecond timestamp with arrow writer properties set to coerce timestamps and support deprecated int96 timestamps.

The bug was a due to the fact that the physical type was int64 but the WriteTimestamps function was taking a path that assumed the physical type was int96. This caused memory corruption because it was writing past the end of the array. The bug was fixed by checking that coerce timestamps is disabled when writing int96.
A unit test was added for the regression.

Thanks @joshuastorck
I moved the ARROW issue over to the PARQUET tracker and also gave you karma to assign issues to yourself there, too.


Feature 22.05

Description
Upgrades to 22.05 revision ce6aa13369b667ac2542593170993504932eb836 and adds the created = "now";.
Issues Fixed

Related https://github.com/MatrixAI/js-polykey/pull/396

Tasks

[x] 1. Changed pkgs.nix to use the 22.05 revision as are using on our OS
[x] 2. Added created = "now"; that introduces proper timestamps on our docker images, but binary reproducibility (content hashes) will change, input hashing is preserved

Final checklist

[x] Domain specific tests
[x] Full tests
[x] Updated inline-comment documentation
[x] Lint fixed
[x] Squash and rebased
[x] Sanity check the final build

The new gitlab-runner and matching revisions give us this result:
We are now only downloading 68 MiB vs originally 239 MiB on the TS demo lib check:lint job.

The job logs are much shorter
This particular linux job is pretty small, so overall clock time didn't change much, instead... an increased time was spent pulling the container image from the registry

However less bandwidth used, and I'm sure this will scale up to larger projects in the future, plus we have a more comprehensive image with all the relevant configuration with timezones, TLS, iana, locality... and more.


Update purple to match Valadoc

will need to do the CSS and meta theme-colors

will fix #1025

@snwh doesn't like the purple. I leave it to @danrabbit to decide the fate of this PR.

@snwh is indifferent

Can I merge this, or do I need #ux ?


Media Library - Infinite Scroll - Loading Indicator Styling

Feature Description
This is a followup to #23 to add styling.
Design

Note: The requirement here is just a static loading message that is needed and not a button that the user needs to hit to trigger loading of the next set of media. ie. This for when the user is scrolling too fast and it's taking a while to load so show the loading indicator
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

When the user scrolls to the bottom, a "loading" animation is displayed

QA Instructions

@merapi ^


Test against PouchDB Server

because now we can: https://github.com/pouchdb/express-pouchdb/pull/197

fixed


Silence static warnings

No significant changes.

Insignifcant changes

metric
value

change

binary-size-nk3am
1199309
:white_circle:
+0 (+0.00%)

binary-size-nk3am-test
2001375
:white_circle:
+0 (+0.00%)

binary-size-nk3xn
431636
:white_circle:
+0 (+0.00%)

binary-size-nk3xn-test
584620
:white_circle:
+0 (+0.00%)

binary-size-nkpk
736974
:white_circle:
+0 (+0.00%)


Moving dedicated[0].encryption_key to byok_key[0].id does not work as expected

After successfully replacing dedicated[0].encryption_key with byok_key[0].id with version 1.79.0 as described here, Terraform still wants to change the cluster configuration, although nothing has changed in the cluster configuration.
Having dedicated[0].encryption_key and byok_key[0].id specified at the same time leads to a deprecation warning, but Terraform detects no changes.
Could you please take a look at this?
Here is an example from Terraform of the change detected:
Terraform will perform the following actions:
  # confluent_kafka_cluster.my_dedicated_cluster will be updated in-place
  ~ resource "confluent_kafka_cluster" "my_dedicated_cluster" {
        id                 = "lkc-abc123"
        # (9 unchanged attributes hidden)
      ~ dedicated {
          - encryption_key = "arn:aws:kms:my-key" -> null
            # (2 unchanged attributes hidden)
        }
        # (3 unchanged blocks hidden)
    }
Plan: 0 to add, 1 to change, 0 to destroy.

@Ad4m0 thanks for creating the issue!
That's a great call-out! We'll update our documentation since it's currently a bit unclear.
The idea is an in-place update for the existing clusters by removing "encryption_key" from the code and adding the block "byok_key" (that refers to a "confluent_byok_key" resource) is not supported and eventually we'd like users to recreate clusters 😕 (see https://github.com/confluentinc/terraform-provider-confluent/issues/398 for more context).


There's now an extra invalid post in the /posts endpoint

https://jsonplaceholder.typicode.com/posts
There are 100 posts as always but if you scroll down to the end, there's an extra one which has only an id:
{
  "id": 101
}

That's not intended, or is it?

I'm closing this since it seems it's been fixed. If it ever happens again I'll try to open a new one with the goal of finding the root cause.

Thanks for the report, and you're right it's not intended.
I've created a test for this case:
https://github.com/typicode/jsonplaceholder/blob/master/test/app.js#L22
But maybe it's not 100% reliable :/


How to filter  transactions data for one special account with mongodb_filter_plugin ?

I want to save the transactions to mongodb  which their transfer target is one special account, but found no way.Help!

mongo_db_plugin supports a mongodb-filter-on =  setting, which is the same format as history plugin's filter-on setting.  To filter to one account, use mongodb-filter-on = youraccount::.  Note the two trailing colons, which indicate all actions, all actors.


services/horizon: open an endpoint for circulating supply

What problem does your feature solve?
Right now there's not an endpoint that gives an aggregate amount of lumens held by accounts.
An example of when this is needed is for our alert if the total supply of lumens is unexpected, which checks to make sure the amount of lumens in circulation is correct. This calculation can currently be done with a Hubble query, but Hubble atm is updated at a delay, so we can't make this check in real time.
note: we can make the calculation using the below equation from  docs
TotalSupply = circulatingSupply + upgradeReserve + feePool + sdfMandate
Horizon makes every value available except for the circulatingSupply. Because of that, we can't check that that circulatingSupply value is correct using Horizon alone.
What would you like to see?
Having an endpoint that gives us an aggregate amount of lumens held in circulation, ie circulating supply. With this endpoint we can check it against what we expect at a high frequency (at least, close to real time), and ensure the network is operating as we expect.
What alternatives are there?
the endpoint could also be total supply (from the above equation), based off of account balance aggregation. As opposed to totalSupply calculated from 100B + inflation - burned, which is already possible.  With that value one could calculate the circulatingSupply.
see slack convo for more details

@acharb Just confirming from this slack convo that you would still be ok if Hubble performed this check instead of Horizon? Will there be any drawback to a 15-30 min delay in latency?


Add L1 Instruction Cache

Adding an L1 Instruction Cache for fetch as noted in #143

Updated fetch unit so that it requests a block of instructions per cycle from the instruction cache. The block is formed based on the instruction's program counters.
Tidied up some coding style within the L2Cache/MSS.
Added tests for new instruction cache unit.
Changed MSS credit interfaces to be more like those used within the core.

@danbone are you planning to add a documentation describing uarch of the modeled L1 Icache and changes to the fetch unit as a part of this PR?

Yeah that's fine by me, I haven't progressed the branch. Though @arupc-vmicro  did contact me to take ownership of the branch a few months back.

What is the status of this PR? I see it now has conflicts, if those were fixed is this ready to go?
I have a need in BPU evals.

Since Knute and Jeff has reviewed this, I will unassign myself from this review. Please feel free to mege if all comments from other reviewers have been addressed.

Actually, this was merged in #211.  We can safely close this PR as NOT merged.


crypto extension is required to support encryption and don't get blocked

When i try to send massage I got this error
crypto extension is required to support encryption and don't get blocked. See more at https://github.com/mgp25/Chat-API/wiki/Dependencies
How can I setup crypto with xampp ?

Pre-Compiled extensions (Curve25519 and protobuf) for Windows users:
Curve25519
PHP Protobuf
Install and add to modules in your php.ini file

I move files to

C:\xampp\php\ext

and added the entry for the dll in php.ini
extension=php_curve25519_nts.dll extension=php_curve25519_ts.dll extension=php_protobuf_nts.dll extension=php_protobuf_ts.dll
But still i got error

i install it but i got those errors
Warning: PHP Startup: Unable to load dynamic library 'C:\xampp\php\ext\php_curve25519_nts.dll' - The specified module could not be found.
in Unknown on line 0
Warning: PHP Startup: Unable to load dynamic library 'C:\xampp\php\ext\php_protobuf_nts.dll' - The specified module could not be found.
in Unknown on line 0

Im having the same error "curve25519 extension is required to support encryption and don't get blocked" and i have it installed and activated.
PD: im using a docker cluster from a image provided by https://github.com/Shaked/docker-whatsapp.

hi @xaamin how I resolve this issue in ubuntu? please provide me some command to install this extension I'm using php 7.1 on ubuntu 14.04.

I am using mac os how to solve above problem please any one help me out. Thank you in advance


add support for writing mixed type lists and tuples in HDF5Formatter

In this PR three things are done:

The existing HDF5 formatter is improved to support also lists and tuples with mixed type elements and numpy.int32 type elements.
Another formatter is added based on serialization to hdf5 by https://github.com/telegraphic/hickle
In the hdf5 data a special field is written
In the data a special field __format_tag is written to describe the format (e.g. make the distinction between the plan hdf5 formatter and the hickle version)

@WilliamHPNielsen @jenshnielsen @AdriaanRol

@eendebakpt I would like to see typing. Docstring would be nice, for example in the function _read_list_group, but this is not strictly necessary for private methods

On the other hand, a method like "write" needs proper docstrings. I see that you were not the author of that method, but now you are enhancing this module anyway, I will like to see docstrings.

I see that "write" does have proper docstrings. However, "read" does not. At least, it is not formatted properly. We cannot infer the input types for example

@sohailc I improved the docstrings for some of the functions (including all the public functions that have been added).
What do you mean exactly with " infer the input types". Are there some tests from the CI I can check?

All I meant was that the docstring originally did not mention the input type.

I don't have further comments, but as I have not been deeply involved with this module or how writing hdf5 files work, I might have missed something. @WilliamHPNielsen @jenshnielsen, do you have additional comments?

@peendebak you can run mypy to check that all type annotations are consistent. In general, we like having type annotations, but we don't force contributors to use them (yet).
I think we can merge this PR now.

Trailing whitespace...


Error message does not vanish after fix.

After changing the sentence and saving the file, the error message is still displayed. This does not happen for any other LSP instance (Python and Go) although, it could be something I messed up in configuration.
Any idea?

Error between keyboard and floor: asciidoc is not supported so don't try it!
Apology for the noise.


Fix for parser getting confused after faulty row with CRLF line ending.

It looks like the parser is currently not able to fully ignore a broken line because of the regex used for detecting the line ending in the file. The current behaviour will move the parser to the first \r char in the line. But for files using CRLF as line endings that is not enough.
This PR changes the detection to consider the combinations \r\n or \r or \n when ignoring a broken line.

Thanks for the feedback @kou. PR updated accordingly.

Thanks.
I've merged.


Remotehost endpoint to allow using non-root user to connect to remote host

Some hosts don't allow root to ssh-login.  Instead, allow a different user for ssh, then "sudo" to run commands on the remote host.

This was done a while back and was tested on Opemshift worker nodes using user "core" and sudo to root


Plan to incorporate Devtools?

Is your feature request related to a problem? Please describe.
I found the dev tools in vee-validate very helpful. Are there plans to incorporate something similar in Formwerk?
Describe the solution you'd like
Devtools to see active forms/fields and their state
Describe alternatives you've considered
N/A
Additional context

Absolutely, this is in the roadmap. I will be publishing it as soon as I can around the weekend.
I will also create an issue t
detailing the scope.

closing in order to track in #86