l2ethers: Reporting failing deploy transaction as successful

When deploying a contract with l2ethers to a local optimism-integration OVM node, a failing deployment is reported as successful.
Test code excerpt:
const ideaTokenExchange = await IdeaTokenExchange.deploy()
await ideaTokenExchange.deployed()

const receipt = await ideaTokenExchange.deployTransaction.wait()
console.log('tx', await ethers.provider.getTransaction(receipt.transactionHash))
console.log('receipt', receipt)
console.log('code', await ethers.provider.getCode(ideaTokenExchange.address))
await ideaTokenExchange.getOwner()

Deploy transaction:
{
  hash: '0xfa35a0f2dd0b126aff75fa09c6c2704918bb37a4a5baa3f8e0492bb64c96ba5e',
  blockHash: '0xf7b70395fc9756f9a7fc4a319eb0c781b551d9f3cbbee3359ca6a6d1020190e2',
  blockNumber: 42,
  transactionIndex: 0,
  confirmations: 1,
  from: '0xBc5058C4F77AC924584CE63cEBB754Ea14775a70',
  gasPrice: BigNumber { _hex: '0x00', _isBigNumber: true },
  gasLimit: BigNumber { _hex: '0x89543f', _isBigNumber: true },
  to: null,
  value: BigNumber { _hex: '0x00', _isBigNumber: true },
  nonce: 41,
  data: '0x608060405234801562..........', // contract bytecode
  r: '0xda67d4a11637d9c8bb0f56cb63947de409933b7b0148e902939adad695c88334',
  s: '0x4032027920305e81fba37e633643dd86194930c8638ff2471a59c711375ff576',
  v: 876,
  creates: '0x065afE7ADe5F05E4Bd69F00Edd53DC57Ea09774d',
  chainId: 420,
  wait: [Function (anonymous)]
}

Receipt:
{
  to: null,
  from: '0xBc5058C4F77AC924584CE63cEBB754Ea14775a70',
  contractAddress: '0x065afE7ADe5F05E4Bd69F00Edd53DC57Ea09774d',
  transactionIndex: 0,
  gasUsed: BigNumber { _hex: '0x7b89b5', _isBigNumber: true },
  logsBloom: '0x00000000004000000000000000000000000000000000000000040000000000008000000000000000000000100000000000000000000000000000000000000000000000000000000000000008000000000000000010000000000000000000000400000000000000000000000000008000000000000000000000000010000000000000000000000000000000000000000000000000002000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000002000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000800000000000000000000000000',
  blockHash: '0xf7b70395fc9756f9a7fc4a319eb0c781b551d9f3cbbee3359ca6a6d1020190e2',
  transactionHash: '0xfa35a0f2dd0b126aff75fa09c6c2704918bb37a4a5baa3f8e0492bb64c96ba5e',
  logs: [
    {
      transactionIndex: 0,
      blockNumber: 42,
      transactionHash: '0xfa35a0f2dd0b126aff75fa09c6c2704918bb37a4a5baa3f8e0492bb64c96ba5e',
      address: '0x4200000000000000000000000000000000000006',
      topics: [Array],
      data: '0x0000000000000000000000000000000000000000000000000000000000000000',
      logIndex: 0,
      blockHash: '0xf7b70395fc9756f9a7fc4a319eb0c781b551d9f3cbbee3359ca6a6d1020190e2'
    }
  ],
  blockNumber: 42,
  confirmations: 1,
  cumulativeGasUsed: BigNumber { _hex: '0x7b89b5', _isBigNumber: true },
  status: 1, // <------------- Transaction successful
  byzantium: true
}

Checking the deployed bytecode:
0x

At the end, the await ideaTokenExchange.getOwner() call fails because there is no bytecode at that address.
Also, the deployment works when running the test with the built-in Hardhat network with l2ethers.
Output from optimism-integration:
geth_l2_1               | DEBUG[03-16|11:01:40.239] Calling Known Contract                   ID=50d5fab9 Name=OVM_SafetyChecker        Method=isBytecodeSafe
geth_l2_1               | DEBUG[03-16|11:01:40.242] Reached the end of an OVM execution      ID=50d5fab9 Return Data=0x0000000000000000000000000000000000000000000000000000000000000000 Error=nil

Does this mean the contract did not pass the bytecode safety check?

Reproduce
git clone https://github.com/Ideamarket/ideamarket-contracts
cd ideamarket-contracts
git checkout cb55038
npm i
./optimism-integration/up.sh &
# Wait a moment for up.sh to boot
npx hardhat test --network ovm_local test/contracts/ovm/core/IdeaTokenExchange.test.js

@Bobface can you run ./optimism-integration/pull.sh and try again? We recently made a change to our contracts that should fix this problem.

And thank you again for opening issues here :-) it means a lot that you're willing to take the time

Sure, no need to thank me :) I'm glad for every bug we can get fixed this way.
I ran docker-compose pull to get the latest images, but the bug is still happening. Here is the output of docker image ls --digests:
REPOSITORY                              TAG       DIGEST                                                                    IMAGE ID       CREATED         SIZE
ethereumoptimism/deployer               latest    sha256:2bda1e5f9d6b67fcd05d0950e9f5f9b5dda42ba3d8e6b89fe58119ec7235a712   8321f87e3158   17 hours ago    1.52GB
ethereumoptimism/batch-submitter        latest    sha256:8f6ef6f78b075be85951fb1f7666c838c4a4b39862d8c990833e4f0e8a084375   4426dcbe370d   32 hours ago    1.49GB
ethereumoptimism/data-transport-layer   latest    sha256:ef5895a30ef6c8a8a4f7b73657a8141235f2bf2630c048ee032c8da3884ed2b0   be612ac6f0ae   3 days ago      1.39GB
ethereumoptimism/deployer               <none>    sha256:1173f0b6d754783dad0a65da948d13a1937a9d889520c0af96e6529e2b25d354   2a04a9dec939   4 days ago      1.52GB
ethereumoptimism/batch-submitter        <none>    sha256:256b5bb86492051fee93978a2a6c467e70770c4c5b833b4bb8e94b35765aa468   481353b77c59   4 days ago      1.51GB
ethereumoptimism/integration-tests      latest    sha256:76a9cbecfd8850ecdc295e3d5b1f28c627ee25585d1cb29136bc7f7790265cbe   96f18e75b096   5 days ago      2.25GB
ethereumoptimism/go-ethereum            latest    sha256:7f08d062ab6aa3a5d1854cbb614237d83771e875f292f9bcdc7d2a567c40e3bc   779ab29798eb   5 days ago      869MB
ethereumoptimism/message-relayer        latest    sha256:e817a730b6857ec2ba35db7fb09fd2838239381d267b5be00b5b1204c8a02087   af40f86ed090   7 days ago      1.25GB
ethereumoptimism/hardhat                latest    sha256:225b659975ce950427e853df0044aa0c003141f8bfb8a61210f53233ee2ed394   7da463caaa32   10 days ago     1.53GB

@smartcontracts I was able to reproduce this on the offical Kovan testnet:
Deploy Tx with Receipt Status 0x1
Deployed contract with bytecode 0x.

Hmmm I would expect this to be happening on Kovan, but not on the latest images. Will give this a shot locally myself, one moment please!

Okay I can reproduce this locally. I think this is very likely to be a problem with ./up.sh and not with @eth-optimism/plugins. We recently made a change that should have fixed this issue (specifically here: https://github.com/ethereum-optimism/contracts/pull/308).

Going to leave this open until the issue is fixed in Geth.


Nullpointer at BasicEntityIdGenerator.getDatacenterId()

Hello,
The latest version on Linux breaks because of a null pointer.
At line 81 getHardwareAddress() returns null which is then thrown at line 83 as mac has a null value.
81: byte[] mac = network.getHardwareAddress();
82: //System.out.println(DatatypeConverter.printHexBinary(mac));
83: long id = ((0x000000FF & (long)mac[mac.length-1]) | (0x0000FF00 & (((long)mac[mac.length-2])<<8)))>>6;

@chriswareham Can be used offline clients generate id?app online used server generate id,app offline used clients generated id.


feat: implement coinbase trading

Relates to:
Coinbase support

Risks

Risk Level: Medium
Potential Risks:

Integration issues with the Coinbase SDK could impact trading functionality.
Possible edge cases in wallet initialization might cause unexpected errors.
Dependencies updated in package.json may cause version conflicts.

Background
What does this PR do?
This PR implements the Coinbase trading functionality using the Coinbase SDK. It adds the ability to:

Execute trades between assets (e.g., ETH to USDC).
Log trades and export trade details to a CSV file.
Dynamically initialize wallets and handle on-chain trading operations.

What kind of change is this?

Feature: Non-breaking change adding new functionality.
Adds Coinbase trading plugin to the project, enabling automated trading.
Updates dependencies and introduces new helper methods for wallet initialization.

Documentation changes needed?

[x] Yes: The trading functionality and wallet setup process need documentation.

Added a section in the README on how to configure Coinbase API keys.

[ ] No: N/A

Testing
Where should a reviewer start?
Begin with the following files:

packages/plugin-coinbase/src/plugins/trade.ts: Contains the trading logic and integration.
packages/plugin-coinbase/src/utils.ts: Handles wallet initialization.

Detailed testing steps:

Wallet Initialization:

Use the initializeWallet function to create a new wallet with the Coinbase API keys.
Verify wallet details are correctly saved to the environment or character files.

Execute a Trade:

Trigger a trade using the executeTradeAction in the tradePlugin.
Example: Swap 0.01 ETH for USDC on the "base" network.

Verify Trade Logging:

Check the trades.csv file for accurate logging of the executed trade.
Validate fields like network, source asset, and target asset.

Negative Testing:

Attempt a trade with an invalid network or unsupported asset.
Confirm appropriate error messages are displayed.

Screenshots (if applicable):
Before:
N/A (New feature).
After:

https://drive.google.com/file/d/1xeUW9sbQ74DinzFrXjqps7o2wqFWRSNv/view?usp=sharing
Successful Trade:
Trade executed successfully:
- Network: base
- Amount: 0.01
- From: ETH
- To: USDC

Error Handling:
Error: Invalid network. Supported networks are: base, sol, eth, arb, pol.

Deploy Notes:

Ensure Coinbase API keys are correctly configured in the environment variables before deployment.

Database Changes:
None.

Deployment Instructions:

Add the following environment variables to .env:
COINBASE_API_KEY=<your-api-key>
COINBASE_PRIVATE_KEY=<your-private-key>

Run pnpm install to update dependencies.

Test all features in a staging environment before deploying to production.

@monilpat
We have conflicts
.env.example
agent/src/index.ts
docs/docs/packages/plugins.md
package.json
pnpm-lock.yaml

plugins.md

Yup aware and resolving them :)


CI Job Status hidden behind commits

open a commit stream which contains greyed-out commits, like

hover the first commit to get it into the foreground and click the "X" for the CI status:

the checks panel is hidden behind the greyed-out commits
occured on google chrome canary 85.0.4150.0 with refined github 20.5.21

Why are the greyed-out merge commits in your case collapsed. We don't collapse them, we only change their opacity (source).
Disable other extensions to see if they might be interfering with commits listing.
(Example URL to test: https://github.com/substack/tape/commits/master)

Why are the greyed-out merge commits in your case collapsed. We don't collapse them, we only change their opacity (source).

tried disabling other extensions, but it seems only refined github is the cause.

collapsing might be a github native feature, when commit messages are "long enough"

We do collapse them in the “dim bots” feature. Disable it if you use bots

But yeah we need to drop overflow:hidden once it uncollapses to avoid the CI panel being clipped

https://github.com/sindresorhus/refined-github/blob/38b48a560b339610461b4acdcf8192ca3cd349e9/source/features/dim-bots.css#L33-L38
/* Dont hide the CI Job Status on focus*/
.rgh-dim-bot.Box-row.navigation-focus {
	overflow: inherit;
}

Adding this after that seems to work

The overflow is required during the animation or else the content will be outside the box. That code simply drops the overflow: hidden

@fregante Would a possible solution to this issue be to listen for the end of the transition, catch that event and just remove the overflow attribute with JavaScript?
If so, I would like to try to implement that in this case :)

I think this can be done with transition delays, no js needed

I can try to fix that if no one else suddenly volunteers :)


Error : Not able to connect thing and companion app

How to solve this error :
10-24 06:55:38.922 3975-3975/com.example.androidthings.robocar I/FirebaseInitProvider: FirebaseApp initialization unsuccessful
10-24 06:55:38.925 3975-3975/com.example.androidthings.robocar I/InstantRun: starting instant run server: is main process
10-24 06:55:39.910 3975-3975/com.example.androidthings.robocar D/vndksupport: Loading<EMAIL_ADDRESS>from current namespace instead of sphal namespace.
10-24 06:55:39.928 3975-3975/com.example.androidthings.robocar E/RobocarActivity: Google API connection failed: ConnectionResult{statusCode=UNKNOWN_ERROR_CODE(8050), resolution=null, message=null}

I had tried with Android Things 1.0.0. Exception not occur in version 1.0.0.


Update event example

Current coverage is 100% (diff: 100%)

Merging #131 into master will not change coverage

@@           master   #131   diff @@
====================================
  Files           1      1
  Lines         379    379
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits          379    379
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update a73e938...104349f


Flaky ActiveRepairServiceTest.testRejectWhenPoolFullStrategy

patch by Berenguer Blasi; reviewed by  Andres de la Peña and Michael Semb Wever for CASSANDRA-16685

CI j11
CI j8


modify pool-incentives submit proposal cli

Closes: #1888
Brief Changelog

add proposal-specific flags to NewCmdSubmitUpdatePoolIncentivesProposal and NewCmdSubmitReplacePoolIncentivesProposal
disable poolincetives tx commands since these should be only used with a gov prefix
register NewCmdSubmitReplacePoolIncentivesProposal handler on the gov module

Testing and Verifying

Manually verified

Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? yes
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? yes
How is the feature or change documented?  not documented

Sorry, I'm actually generally confused about whats going on in these governance proposal routing.
My understanding is that in the today code, replace incentives proposal works on mainnet. Looking into this, looks like the handler is registered here? https://github.com/osmosis-labs/osmosis/blob/main/app/keepers/keepers.go#L366
Which then splits execution based on type https://github.com/osmosis-labs/osmosis/blob/fd23552f94caf1df9dbeb6fb4784e9d0efded2a4/x/pool-incentives/handler.go#L12-L24
So the routes being equivalent may be correct?

(this is also needlessly complex, and we should simplify it)

Sorry, I'm actually generally confused about whats going on in these governance proposal routing.
My understanding is that in the today code, replace incentives proposal works on mainnet. Looking into this, looks like the handler is registered here? https://github.com/osmosis-labs/osmosis/blob/main/app/keepers/keepers.go#L366
Which then splits execution based on type
https://github.com/osmosis-labs/osmosis/blob/fd23552f94caf1df9dbeb6fb4784e9d0efded2a4/x/pool-incentives/handler.go#L12-L24
So the routes being equivalent may be correct?

I think that the linked handler registration is focused on routing. On the other hand, this PR is primarily about fixing the client bootstrapping / cli.
Currently, pool-incentives gov cli is broken due to incorrectly setting up the cli logic and registering some flags twice. I've verified that this should help fix the original problem in #1888.
We expose 2 ways to submit pool-incentives proposals:

osmosisd tx poolincentives update-pool-incentives
osmosisd tx gov submit-proposal update-pool-incentives

Only the latter is correct because it properly gets registered on the gov module. The original bug is stemming from attempting to support both ways and, as a result, registering some CLI flags twice which breaks both commands.
The change related to the proposal type seems to be a pass-through change. However, It does seem like a valid change that we should keep because we were returning an invalid proposal type.
The claim that ProposalType() is used in routing does seem to be correct. Although this is client-side (rest) routing.
Based on my investigation, the changes in this PR do not affect the state-breaking routing we are concerned about.
I think we need the changes in this PR to fix the CLI and client (REST) routing. However, it might make sense to spend more time manually testing to ensure that everything is sound

Hi @catShaark . I just would like to confirm that you've gotten a chance to manually test that both pool incentives proposals are functioning as expected.
Please let me know

I think it would be great to have an e2e test for submitting these pool-incentive proposals. Would you be interested in adding it?


Fix/KFS-1609 multibyte chars

I fixed the util functions that we use and will create a ticket to fix the other ones. More details in the ticket.

LGTM since all my comments resolved themselves in the next commit. Personally, seems overkill to add Godash (Lodash for Go, get it? Why the heck did they not use that name?!) for the one remaining usage (afaict), though.

Coming from the frontend world, I totally agree that loading a lib like loadash only for a one func is totally using a sledgehammer to crack a nut.
However, the beauty in go is that unused stuff from a package will be dropped by the linker and won't clutter the binary, so a library like this makes much more sense in go.
Also, I think at least in all repos I worked in our team we informally try to use this (quite nice) lib when we can to reduce the bloat from go.


Error in workloop Quorum check failed HostUnreachable

I'm having an issue getting the sidecar to see my two MongoDB replicas.

Error in workloop { MongoError: Quorum check failed because not enough voting nodes responded; required 3 but only the following 1 voting nodes responded: mongodb-0:27017; the following nodes did not respond affirmatively: mongodb-0.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable, mongodb-1.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable, mongodb-2.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable
at Function.MongoError.create (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/error.js:31:11)
at /opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/connection/pool.js:497:72
at authenticateStragglers (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/connection/pool.js:443:16)
at Connection.messageHandler (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/connection/pool.js:477:5)
at Socket. (/opt/cvallance/mongo-k8s-sidecar/node_modules/mongodb-core/lib/connection/connection.js:331:22)
at Socket.emit (events.js:159:13)
at addChunk (_stream_readable.js:265:12)
at readableAddChunk (_stream_readable.js:252:11)
at Socket.Readable.push (_stream_readable.js:209:10)
at TCP.onread (net.js:598:20)
name: 'MongoError',
message: 'Quorum check failed because not enough voting nodes responded; required 3 but only the following 1 voting nodes responded: mongodb-0:27017; the following nodes did not respond affirmatively: mongodb-0.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable, mongodb-1.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable, mongodb-2.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable',
ok: 0,
errmsg: 'Quorum check failed because not enough voting nodes responded; required 3 but only the following 1 voting nodes responded: mongodb-0:27017; the following nodes did not respond affirmatively: mongodb-0.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable, mongodb-1.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable, mongodb-2.mongodb.stage-sprint.svc.cluster.local:27017 failed with HostUnreachable',
code: 74,
codeName: 'NodeNotFound',

Here's my Kubernetes yaml configuration that defines my resources.
# Based off of http://blog.kubernetes.io/2017/01/running-mongodb-on-kubernetes-with-statefulsets.html
# and https://github.com/cvallance/mongo-k8s-sidecar/tree/master/example/StatefulSet
#
# Learn more about StatefulSets, PersistentVolumes and PersistentVolumeClaims:
# https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/
#

# Based off of https://github.com/cvallance/mongo-k8s-sidecar/blob/master/example/StatefulSet/azure_ssd.yaml

# Define the Azure volume storage class for use by our customer MongoDB nodes
apiVersion: storage.k8s.io/v1beta1
kind: StorageClass
metadata:
  name: mongodb-storage
  namespace: stage-sprint
  labels:
    app: mongodb
    tier: storage
provisioner: kubernetes.io/azure-disk
parameters:
  skuName: Premium_LRS
  location: centralus
  storageAccount: <storage_acct>
---

# Create a headless service so each MongoDB pod will get a stable hostname (note: this does NOT load balance)
apiVersion: v1
kind: Service
metadata:
  name: mongodb
  namespace: stage-sprint
  labels:
    app: mongodb
    tier: svc
spec:
  ports:
  - port: 27017
    targetPort: 27017
  clusterIP: None
  selector:
    app: mongodb
    tier: db
---

# Create the stateful set for the MongoDB nodes
apiVersion: apps/v1beta1
kind: StatefulSet
metadata:
  name: mongodb
  namespace: stage-sprint
spec:
  serviceName: "mongodb"
  replicas: 3
  template:
    metadata:
      labels:
        app: mongodb
        tier: db
    spec:
      terminationGracePeriodSeconds: 10
      containers:
        - name: mongo
          image: mongo:latest
          command:
            - mongod
            - "--replSet"
            - rs0
            - "--smallfiles"
            - "--noprealloc"
            - "--bind_ip_all"
          ports:
            - containerPort: 27017
          #imagePullPolicy: Always
          volumeMounts:
            - name: mongo-persistent-storage
              mountPath: /data/db
        - name: mongo-sidecar
          image: cvallance/mongo-k8s-sidecar
          env:
            - name: MONGO_SIDECAR_POD_LABELS
              value: "app=mongodb,tier=db"
            - name: KUBE_NAMESPACE
              value: "stage-sprint"
            - name: KUBERNETES_MONGO_SERVICE_NAME
              value: "mongodb"
      imagePullSecrets:
        - name: regsecret
  volumeClaimTemplates:
  - metadata:
      name: mongo-persistent-storage
      annotations:
        volume.beta.kubernetes.io/storage-class: "mongodb-storage"
    spec:
      accessModes: [ "ReadWriteOnce" ]
      resources:
        requests:
          storage: 5Gi

Update: I tried to ping the other pods from within the first instance of the mongo pod and I was unsuccessful... What am I missing that allows the Mongo pods to connect to each other?

Same thing here. This error appears when serviceName in StatefulSet differs from the Headless Service name. And the Headless Service must be created before the StatefulSet.


WSL 2 distro won't start if resource specified in /etc/fstab is not available

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [Version 10.0.19035.1]
What you're doing and what's happening:
add \\<IP_ADDRESS>\Seagate1TB /media/keenetic cifs  rw,uid=0,credentials=/home/archil/.smb,iocharset=utf8 0 0 in /etc/fstab
Starting wsl distro.
What's wrong / what should be happening instead:
When I'm on a different network the distribution won't start.
One workaround I found is adding noauto parameter.

Adding nofail doesn't seem to help either, which is how you would address this on Linux


Clean API for discovering IDE

Some exotic build configurations are effectively unworkable in IDEs.  For example, having an Android test launcher that also builds test paths from other modules.
Having a clean API to detect when running in an IDE and make decisions would help.
Expected Behavior
Either support an API, or publish clear documentation on the System properties to reliably detect IDE.
Current Behavior
https://github.com/square/okio/commit/f14edb72f70b7e0be51dab444e1f2a2c0d1b4fa3
andrdoi-test.gradle
 def isIDE = properties.containsKey('android.injected.invoked.from.ide') ||
   (System.getenv("XPC_SERVICE_NAME") ?: "").contains("intellij") ||
   System.getenv("IDEA_INITIAL_DIRECTORY") != null
...
  if (!isIDE) {
     sourceSets {
       named("androidTest") {
         it.java.srcDirs += [
           project.file("../okio-testing/src/commonMain/kotlin"),
           project.file("../okio/src/commonMain/kotlin"),
           project.file("../okio/src/commonTest/java"),
           project.file("../okio/src/commonTest/kotlin"),
           project.file("../okio/src/hashFunctions/kotlin"),
           project.file("../okio/src/jvmMain/kotlin"),
           project.file("../okio/src/jvmTest/java"),
           project.file("../okio/src/jvmTest/kotlin"),
         ]
       }
     }
   }

settings.gradle
// The Android test module doesn't work in IntelliJ. Use Android Studio or the command line.
 if (properties.containsKey('android.injected.invoked.from.ide') ||
   System.getenv('ANDROID_SDK_ROOT') != null) {
   include ':android-test'
 }

Context

Thank you for your interest in Gradle!
This issue will be closed as a duplicate of #17275. Please see the reasoning there.


[MLU] fix cross_entropy and add bmm

fix cross_entropy and add bmm

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.tanhaojue seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Public initializers for payloads in multipart message

What?
In order to create messages locally outside the library we need to have public initializers. Otherwise they cannot be initialized due to internal accessibilty of those initializers
Why?
The ability to setup and send messages internally without having to wait until the message is pushed through the server means we can have a "message in transit". If these initializers are internal, the entire messaging system needs to replicate the message structure. With these public initializers we create custom internal messages.

Sorry for the late response! This looks good! Thank you for the PR!


Datagrid header row won't stick to the top with fixed height

Describe the bug
When styling the datagrid with a fixed height and selecting page size of e.g. 100, the header row scrolls along after a while.
How to reproduce

Go to clarity's datagrid component with pagination
Select page size 100
Inspect the datagrid DOM element and give it a fixed height of 500px
Scroll down

Expected behavior
Header row stays on top
Versions
App

Clarity: 3.1.5

Device:

Type: MacBook
OS: macOs
Browser: Chrome
Version 84.0.4147.89

Additional notes
I noticed on inspecting that div.datagrid-table-wrapper and div.datagrid-table don't have the full height als the row items, which is why the sticky header row stops sticking. Compare to older datagrid verions, you seem to have added new wrapper elements, maybe it's a styling bug?

Hello @aavelyn, thanks for opening the issue, we will investigate it and get back to you.

@bdryanovski any update on this issue?

No sorry, @edgas316 there is no fix for that for now. We gonna investigate it soon.
I think I have an idea why this is happening but have to test it.


Conditional index in a migration

Is it possible to specify a condition index when creating table in a migration? Something that would result in the following index:
create index category_score_id on products(category, score desc, id desc) where active;

Thank you.

No partial index support right now (you need raws)

So looks like I need to write out the whole thing via knex.raw, right?
knex.raw('create index category_score_id on products(category, score desc, id desc) where active');

Or is there a way to offload some of it to the table.index() and then the rest to the knex.raw?

Full raw unfortunately

I'm interested in taking a stab at this feature since it seems (at first glance) easy enough and it would benefit my team.  I'm not familiar with the knex code base, so I can't commit to getting it done nor commit to a timeframe, but I'll see what I can do.
Not all RDBMS's support partial indexes.  I know PostgreSQL does and MySQL does not.  My focus is PostgreSQL, but if I can make it re-usable by others, I'll try to do that.
@tgriesser If there are any particular design considerations or approaches you'd like me to take, please let me know.  Especially given this feature will not work on all RDBMS's.  I figure there is already an established pattern for handling this constraint.
I'll submit a PR once it's complete and the tests pass.  I am reviewing the CONTRIBUTING.md file.


Powertoys update 5th June 2020 stops PC restarting

Environment
Windows build number: [run "ver" at a command prompt]
PowerToys version:
PowerToy module for which you are reporting the bug (if applicable):

Steps to reproduce

Expected behavior

Actual behavior

Screenshots

Just updated powertoys, pop up message said update ready to install....then PC refused to start up received message a device not connected, pc needs to be repaired......repair failed, so restoring a Macrium reflect image from yesterday.......don’t know what I would do with regular MR backups 👌

Please give us more details:

Windows version
PT version
Detailed error logs if you have some

Without this information we can't help you.

the diff between 18.1 and 18.2 is minimal. Nothing PT does should cause this type of regression


Added create new branch button

Added "Create new branch" Button. Lots of Bugs to fix.

Added "Create new branch" Button. Lots of Bugs to fix.

@alwin48 Thanks! You plan to fix the bugs in this PR itself, right? Also, please have different branches for different PRs.
Also as I suggested earlier, you can try to go with React's newer functional componentes with hooks which will help you in the future and also more beginner-friendly for other contributors. Feel free to ask me for a review after the bugs are fixed! :)

Added "Create new branch" Button. Lots of Bugs to fix.

@alwin48 Thanks! You plan to fix the bugs in this PR itself, right? Also, please have different branches for different PRs.
Also as I suggested earlier, you can try to go with React's newer functional componentes with hooks which will help you in the future and also more beginner-friendly for other contributors. Feel free to ask me for a review after the bugs are fixed! :)

I am working on the bugs. And I am just learning hooks and I will implement them soon. Also I will use new branches for pull requests from now on. It will take some time for the code to be organized though 😅

Closing in favour of #6


Model Binding in controller methods is case insensitive.

Platform: ASP.NET Core 2.2.0
Matching the parameters case insensitively in controller methods, leads to unexpected results, especially in POST requests. Let's have example model as following
public class Customer
{
	public int ID { get; set; }
	public string Name { get; set; }
}

In order to simplify the example, if we scaffold CustomersController, we will get the following content for Edit[Post] method
// POST: Customers/Edit/5
[HttpPost]
[ValidateAntiForgeryToken]
public async Task<IActionResult> Edit(int id, [Bind("ID,Name")] Customer customer)
{
	if (id != customer.ID)
	{
		return NotFound();
	}

	if (ModelState.IsValid)
	{
		// Some code here
	}
	return View(customer);
}

Presumably  the "id" value should be filled from "Route data", while "ID" from "Form fields". But both values are filled with the value from "Form fields" (I assume there is a match since it's case insensitive). In this scenario the following code portion has no meaning.
if (id != customer.ID)
{
	return NotFound();
}

Quick workaround would be never to name properties in models as "ID", or to change the "id" variable name in route mapping definition  "{controller=Home}/{action=Index}/{id?}"
PS. If this is by design (for whatever reasons), I apologize for my ignorance.

The docs for model binding briefly states this: https://docs.microsoft.com/en-us/aspnet/core/mvc/models/model-binding?view=aspnetcore-2.2#sources. In the absence of any indication from the user, properties and fields are bound in case-insensitive manner over the following sources in order:

Form POST
Route
Query string

You can restrict the sources a property or field is bound from using From* attributes. In this case, applying [FromRoute] on the int id parameter should get it to bind from the route as you'd expected.

Thank you @pranavkm for your answer.
I'm aware of the corresponding rules, but the above code is not production code. I just posted it as an example, and it's generated/scaffolded code.
I totally agree, bounding properties in case-insensitive manner is probably the right choice, otherwise we would have even more issues with API requests (I assume in .net Core model binding infrastructure is common for MVC and API controllers).
I just think it would be great if we are more explicit in the documentation regarding the possible downsides of this rule in MVC controllers. There are cases where this can lead to over-posting issues. Let's imagine the following case
public class Order
{
	public int ID { get; set; }
	public decimal Price { get; set; }
	public decimal price { get; set; }
}

public async Task<IActionResult> Edit([Bind("ID,Price,price")] Order order)
{
}

Even though I can't think of a reason why somebody would name the properties this way, in the context of c# language there is no any constraint for not doing so. If we test it, we would get really surprising results.

You can use view models to bind a subset of properties. It's certainly our recommendation to avoid over-posting.

I just think it would be great if we are more explicit in the documentation regarding the possible downsides of this rule in MVC controllers.

I'm not entirely sure what you mean by this. The documentation already points out that model binding in MVC is case-insensitive. But if you'd like to add to the document, we'd be happy to accept it.


How to make tab content stateful ?

Suppose we have a couple of components that are stateful, now we mount them to tab content.
I know I can change the state of one component, when the tab that contains it is activated. But after switching to another tab and back, the component is re-mounted and the modified state of it is gone.
Maybe it is too much for react-tabtab, but is there a way to maintain the states of all tab content ?

That's a good point.
The reason why react-tabtab is designed with re-mount mechanism is hoped to handle as many tabs as possible. If it keeps all the panel mounted result, it will cause performance issue.
But yes, I think that's a possible situation when creating tab UI. Maybe it's possible to add a cache props on the tab, it means this panel won't be unmounted. (Of course, it means developers need to handle the performance issue by themselves.
The possible api:
<Tabs>
  <TabList>
    <Tab>Tab1</Tab>
    <Tab cached>Tab2</Tab> // this tab wont be unmount
  </TabList>
  <PanelList>
    <Panel>
      Accusamus enim nisi itaque voluptas nesciunt repudiandae velit.

      Ad molestiae magni quidem saepe et quia voluptatibus minima.

      Omnis autem distinctio tempore. Qui omnis eum illum adipisci ab.
    </Panel>
    <Panel>
      Officiis commodi facilis optio eum aliquam.

      Tempore libero sit est architecto voluptate. Harum dolor modi deleniti animi qui similique facilis. Sit delectus voluptatem praesentium recusandae neque quo quod.
    </Panel>
  </PanelList>
</Tabs>

What do you think?

Yeah, I think it's a great solution when the effects of the performance issue are limited.
Some tabs with heavy component and lots of middle states can be set with cache prop, and other tabs without the prop.

About how to handle performance issues, I think it deserves detailed discussion.
Many small but useful usage tips can be brought out.

Hi @ctxhou , is there any plan to add a feature like cached prop to this module, in near future???

Hi @dineshnamburi, I will try to add this feature in recent. Maybe next month.
If you have any idea of this feature, any help and PR is welcome.

@ctxhou Any plan on implementing the feature? Thanks.

Hi @qqdaiyu55
I am a little busy recently. Are you interested in implementing this issue?

@ctxhou Sorry I'm also busy recently, I'll take a look if I have time. If you have any idea on implementing this feature, you can post it here : )

Any update on it? or any possible solution or hack for now?


Several Questions

Hi,
Thank you for your great work, I am new to this area, I have the following questions:
1、How much improvements can be made by the branch of Graph-based contrastive learning? I can not see that ablation experiments in the paper. I'm interested in this part and would also like to know the performance improvements that might be useful for my task.
2、Does the Co-match method fit for multi-label  classification tasks？
Once again, thank you for your nice work and clean code! Looking forward to your reply.
Best regards!
Tan

Thanks for your questions!

The graph-based contrastive loss gives a huge improvement. As shown in Figure 4b, reducing the contrastive loss weight from 10 to 1 leads to a big drop, and the performance further degrades if the weight is 0.
Yes the method should be able to easily extend to multi-label classification.

Ok. Thank you for your quick reply. I will try your method, and hope you can offer guidence if needed.
Once again, I really appreciate your works.
Best regards!


Add files via upload

@dylancodes24 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.


Azure Sphere - Tenant Registration Issues on Ubuntu

Background :

I am Mac developer. I thought of trying Azure Sphere and Bought one in 2019 (Azure Sphere MT3620 Starter Kit)
At that time the only Supported OS is Windows 10. So, i installed Virtual Box and successfully able to connect the Starter Kit and Registered the tenant.
Then i wanted to run few programmes and i thought i will wait for support for Ubuntu and then start actively using this.

Later this month, I opened the Azure Sphere and connected to a Ubuntu and i was getting a weird error. Tenant not found. If i try to create one its saying the device is already claimed in a different tenant.
I was spending long hours and i could not find any relevant information other than the similar thread.  https://docs.microsoft.com/en-us/answers/questions/57338/azure-sphere-tenant-not-found.html
Then i used device recover command. After this, the device is no more recognized.
 I mean i want to know whether i corrupted the device or its the SDK Version conflict between the Windows 10 and Ubuntu SDK.
Can someone help me on this?

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

ID: ad1dfe8d-03d1-28ec-6155-6560a12fabce
Version Independent ID: 25def8b1-7c70-e84c-b262-ec55a9c3244e
Content: Create an Azure Sphere tenant
Content Source: AzureSphere/articles/deployment/create-tenant.md
Product: azure-sphere
GitHub Login: @pennyo
Microsoft Alias: azurespheredocs

You may have more luck getting a response from the Microsoft Q&A Platform:  https://docs.microsoft.com/en-us/answers/index.html
Make sure you tag your post with Azure Sphere.

Did you migrate your tenant?  There were changes to the tenant last year that required older Azure Sphere Tenants to be migrated.  Documentation on migration is here:  https://docs.microsoft.com/en-us/answers/index.html.  Since you claimed your device before, it can't be used to create a new Azure Sphere tenant.

In my experience you likely have some kind of a driver issue where you're not able to connect to the device.  Sorry, I'm a Windows 10 user, so I can't help with the Ubuntu issue.  I'm working on building a Ubuntu box now so I can learn this path.

I would check any connectivity issues on the system.  By the way, the recover command uses a different method to communicate to the device, it's a straight serial connection and uses a protocol like zmodem to transfer the files.  But normal communications is ethernet over serial, and this will trip you up if it's not working.  On windows it uses a TAP driver.
The other thing to try is make sure you're not connected to a VPN when you try to connect to your device.  Some VPN's disable "split tunneling" and that's required for the TAP driver to work (on windows).
Good luck

Thanks @bawilless
@AshokPeddakotla-MSFT , I think you may be aware of this problem.
I suspect the issue is because of the SDK Version which i used in Windows 10 vs SDK which is used in Ubuntu.
Should i need to use the same old SDK for migrating the tenant?. If thats the case, do we have old SDK for Ubuntu to migrate as i don't have a Windows 10 anymore.
Please help to resolve this.

@nk-gears Could you please check this doc About migration and let us know if you still need help.

@nk-gears Yes you need ti use the old SDK to migrate.
As mentioned in the doc,

Continue using the 19.09 SDK until you are ready to perform the migration. If you have already installed the 19.10 SDK but are not ready to migrate, uninstall it, then download the 19.09 SDK and reinstall it.
The 19.10 Azure Sphere SDK supports only the new user authentication and cloud management environment. After you install the 19.10 SDK, the only cloud-dependent azsphere command that you can use starts the migration process for a tenant. All other cloud-dependent commands will fail.

I would suggest you, Please file a support request @ https://aka.ms/azsupt? If you do not have access to a support plan, please reach out<EMAIL_ADDRESS>with a link to this Issue as well as your subscription ID and we can help get the support ticket opened for this issue.

@AshokPeddakotla-MSFT Thanks for the detailed instructions.
Can you point me where to download the Azure Sphere 19.09 SDK for Ubuntu ?

@nk-gears Sorry for the confusion around this. You should be able to perform migration with the current 20.07 Linux SDK (or any SDK from 19.10).  Indeed, the first Linux SDK wasn’t until 19.11!
https://docs.microsoft.com/en-us/azure-sphere/install/install-sdk-linux?pivots=vs-code-linux
Please try with the current SDK and update. If you are still having the issue, as suggested, you can raise a request with support and the team will be able to help address the issue and provide the resolution accordingly.

When i try to migrate it prompts to login and after signing in using the same credentials.
AADSTS50020: User account<EMAIL_ADDRESS>from identity provider 'live.com' does not exist in tenant 'OsTechnology' and cannot access the application '0b1c8f7e-28d2-4378-97e2-7d7d63f7c87f'(Azure Sphere Utility) in that tenant. The account needs to be added as an external user in the tenant first. Sign out and sign in again with a different Azure Active Directory user account. Wondering who is "OsTechnology" ?
I will send mail to support. You can close this ticket. The above information is just  FYI.

@nk-gears To provide more context, looks like you are using the personal MSA to login when attempting the migration.  But if you previously used Azure Sphere before the 19.10 migration, you must have signed up with an AAD organizational (work/school) account.  You should use that account to drive the migration.  The CLI prompts should instruct further on this. if that doesn't help and you are still struggling, please share get-support-data logs when you contact support.
Since you are contacting support for further assistance, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Is there a way to identify to which Tenant my device is registered. Also i am seeing a tenant with the following name. Is this created by microsoft by default for all users?

I have added a TXT Entry to the DNS for hopechurch site. Please ask the client to test now.

Hello @nk-gears Just checking back, could you please let us know if you need further help in this matter.

Yes. I need few clarifications.
How do I know which tenant the device was claimed ?.
I am a individual developer and I really don't remember the tenant which I claimed the device last year.

Hi Nirmal,@nk-gears
Did you try the below commands from an AzSphere Dev CLI?
https://docs.microsoft.com/en-us/azure-sphere/reference/azsphere-device#list
azsphere device list
https://docs.microsoft.com/en-us/azure-sphere/reference/azsphere-tenant#list
azsphere tenant list
ID                                   Name

d343c263-4aa3-4558-adbb-d3fc34631800 Microsoft

One of our Microsoft MVP @matsujirushi [profile]  developed this tool: Azure Sphere Explorer
Please give it a try and let us know if it helps.

We will update the outcome of the enabled support discussions soon.

@AshokPeddakotla-MSFT , Closing this Issue with resolved.
@SatishBoddu-MSFT , Thanks for working closely to troubleshoot the root cause and resolving them. I Appreciate the Team behind for the efforts and co-ordination.
Later i will share a detailed blog post on the scenario and how what are the things which i missed while migrating the device.

@AshokPeddakotla-MSFT , Closing this Issue with resolved.
@SatishBoddu-MSFT , Thanks for working closely to troubleshoot the root cause and resolving them. I Appreciate the Team behind for the efforts and co-ordination.
Later i will share a detailed blog post on the scenario and how what are the things which i missed while migrating the device.

Hi, I am having the same issue, can you please tell how to resolve? I can't migrate to the previous tenant, as it was 2 years ago.

Hi @arkesray
The Issue happened to me when i tried to Login using a different domain.
We need use similar to this<EMAIL_ADDRESS>Then the issues resolved.
Please check what Directory you are trying to Login.


Update present navigationBar.translucent No

fix FLEXGlobalsTableViewController && FLEXNetworkSettingsTableViewController display stype

Could you clarify what this is trying to accomplish, maybe with before/after screenshots? Thanks!

@Vanson Could you provide screenshots of what you mean? I'm not seeing an issue on iOS 11; if it was an iOS 10 bug then we should make this fix do a dynamic check for iOS 10

Closing for lack of activity and inability to reproduce the bug at this time. We can reopen this if @Vanson  responds with more information.


Issue with trying NetDump sample

Hello!
I already used WinDivert in C++ and Python and I wanted to try your Divert.Net, sadly I having issues using it. I am not really familiar with wrappers so it's possible I miss something really obvious, please help me if you can.
For example I wanted to try out your NetDump example but after opening the solution in VS2015 it can't find the already added by default Divert.Net reference (yellow triangle) and having issues with "using Divert.Net;" too, missing a using directive or an assembly reference.

I've read the wiki of Divert.Net and as the wiki says I created the necessary directory structure in the root directory of the NetDump example.
The root directory of my project:

The deps\windivert:

The deps\windivert\x64:

The deps\windivert\x86:

Should I make any other changes in VS2015 too? I am thinking about things which are not written in the wiki because they are too obvious, but not for me.
Thank you in advance!

Hey @maczbal, so I made this wrapper a long time ago, and I am interested in solving any issues with it. However, as you've experienced, it's a mess to get the project off the ground. I'll see if I can make a nuget package. You're going to have to follow the Wiki, build Divert.NET, then manually add the reference again if you want to get it up and running right now. Alternatively you can try my Pinvoke class (it's under my repositories list) but this requires the "unsafe" flag because it uses raw pointers.

Thank you for your fast reply!
Can I ask you to link a sample code for your Pinvoke class?

@maczbal I don't have an example published. I'll see if I can get one together.

@maczbal You can find an example in one of my other projects, which is a filtering, local, transparent proxy server here.

Oops, still need to investigate this.


Uncaught TypeError: Cannot read properties of undefined (reading 'holder') bug fixed #179

Hello @miguelnietoa has this been added to the main codebase? I am seeing this same issue when I try to use this plugin.

@ARH3 I was unable not reproduce it... I think it's not an issue. Thank you.
Hey @khanakia, could you describe better how the error occurs? Please!

It's related to editor.js default state and i could find this reference https://github.com/codex-team/editor.js/issues/2084 the same kind of issue was reported by many other users also but i could find this one.
So when you keep undoing and there is nothing to undo then it actually gives the error because undo tries to set the default state it was something like that i cannot remember it completely.
Also, there were many other bugs with undo the undo-redo does not work perfectly with components. So i forked the repo of editorjs-undo and customized the code to make it work because our core product is fully dependent of Editor.js so we could not wait to create a pull request and wait it to merge.


Add latest static Ulster Scots translations

What is the context of this PR?
Adds latest static Ulster Scots translations.
How to review
Check the translations match crowdin.
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Load PO Messages file - Ulster Scots


Use logInfo to output more index information for OAP users

Three categories of information are provided:

which filters are supported by OAP index
available indexes and which best index will be used
how long index, stats and iterator are loaded or constructed

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

So far I haven't found the appropriate place to get the updated value of accumulator variable, since the accumulator value will be finally updated after actions are finished on the executors. The accumulator value may not be deterministic in the transformation body.

Any suggestions or comments are really appreciated. Thanks.

The information required is which data will be using index or not. Some data (or tables) may use index. Some may not. Right now it can be reported easily on the executor side.
Currently there are two approaches to get the above information on the driver side:

It requires a call back or some place on driver side to be informed that the actions (queries) are finished on the executor side. In the call back, we can use the accumulator's final value to decide which data was used index or not on the executor side.
On the executor side, during building file scanner in the initialization of OapDataReader, it will be known if index will be used or not. I am not sure if there is any approach to immediately return this kind of information back to the driver.

Close this PR. Please refer to the PR#318. The link is https://github.com/Intel-bigdata/OAP/pull/318.


Fix races

This PR fixes some races in Traefik.
The Docker provider still needs to be fixed /cc @vdemeester

LGTM 🐯


Null safety

Hi ! I'm hoping this issue can be used to follow updates on nullsafety. Currently, there are still several dependencies that are not migrated and might be considered blocking.
dart pub outdated --mode=null-safety

Showing dependencies that are currently not opted in to null-safety.
[✗] indicates versions without null safety support.
[✓] indicates versions opting in to null safety.

Package Name                 Current    Upgradable  Resolvable           Latest

direct dependencies:
file_picker                  ✗2.1.6     ✗2.1.6      ✓3.0.0-nullsafety.2  ✓3.0.0-nullsafety.2
flutter_colorpicker          ✗0.3.5     ✗0.3.5      ✓0.4.0-nullsafety.0  ✓0.4.0-nullsafety.0
flutter_keyboard_visibility  ✗4.0.4     ✗4.0.4      ✓5.0.0-nullsafety.2  ✓5.0.0-nullsafety.2
image_picker                 ✗0.6.7+22  ✗0.6.7+22   ✓0.7.0               ✓0.7.0
photo_view                   ✗0.10.3    ✗0.10.3     ✗0.10.3              ✗0.10.3
quill_delta                  ✗2.0.0     ✗2.0.0      ✗2.0.0               ✗2.0.0
quiver_hashcode              ✗2.0.0     ✗2.0.0      ✗2.0.0               ✗2.0.0
string_validator             ✗0.1.4     ✗0.1.4      ✓0.2.0-nullsafety.0  ✓0.2.0-nullsafety.0
tuple                        ✗1.0.3     ✗1.0.3      ✓2.0.0-nullsafety.0  ✓2.0.0-nullsafety.0
universal_html               ✗1.2.4     ✗1.2.4      ✗1.2.4               ✗1.2.4
url_launcher                 ✗5.7.10    ✗5.7.10     ✓6.0.0               ✓6.0.0

7  dependencies are constrained to versions that are older than a resolvable version.
To update these dependencies, edit pubspec.yaml, or run `dart pub upgrade --null-safety`.

quill_delta has a active pull request for null safety here. photo_view also has an active pr for null safety. As for quiver_hashcode, here is a comment that says people should instead rely on quiver instead which is already migrated to null safety. This just leaves universal_html without an active PR.

We are on channel stable.
We cannot upgrade package image_picker to 0.7.0  for example

This would be referring to the master branch. Also do you think there could be a pre-release version that always follows master as you already have that branch? I remember zefyr had this conversation a few times as well and it always caused people headaches. If there's a pre-release version on pub, people can opt into that without having to file weird errors about something not working that was designed to work for stable. pre-releases, as far as I understand, don't have to be specifically about null safety

Default branch master is on channel master. To use channel stable, switch to branch stable. Branch master on channel master supports web.

Our pub is published based on stable branch, not master branch

Sorry, I just meant to ask why not do this:
  flutter_quill: 0.3.3  # This version follows stable branch

and then also always have a published pre-released version like this:
  flutter_quill: 0.3.3.dev-0  # This version follows master branch

If someone installs flutter_quill even without a version number they'll install the stable version. However, if they specify the dev version, they'll get the master branch one. If we also note this in the readme, I think it'll stop people from filing errors in the readme about SDK errors

@pengw00 could you work on this? And whatever @Nolence needs to update
@Nolence thanks for the suggestion

@Nolence can you check https://github.com/singerdmx/flutter-quill/commit/2122f9b7cdf75cb64e50c1c65accd1cf1fe209a6

@Nolence also check out https://github.com/singerdmx/flutter-quill/commit/08734a8ec36309a8e4cfb7f046c5ed3654543dfd

It looks good to me :)
See comments on 2122f9b though


az command not having a similar result like in the docs

Dear page editors,
I landed on the given page due to CVE-2021-27075 .
The page tells me I should check my Kubernetes Node version using this command:
'
az aks nodepool get-upgrades \
    --nodepool-name mynodepool \
    --cluster-name myAKSCluster \
    --resource-group myResourceGroup

In the output you can see the latestNodeImageVersion like on the example below:
'
However, when I issue this comand, i do not get a latestNodeImageVersion attribute as result, I only get:
{
  "id": "/subscriptions/...",
  "kubernetesVersion": "1.19.7",
  "name": "default",
  "osType": "Linux",
  "resourceGroup": "kubernetes-production",
  "type": "Microsoft.ContainerService/managedClusters/agentPools/upgradeProfiles",
  "upgrades": null
}

which renders me clueless on how to proceed. Maybe you can tell me what is missing here on my behalf?
Thanks in advance,
Armin

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

ID: 8092825b-b867-d947-9842-f44eba91db00
Version Independent ID: 5b9b865c-0d51-9955-564c-1c898d3150a7
Content: Upgrade Azure Kubernetes Service (AKS) node images - Azure Kubernetes Service
Content Source: articles/aks/node-image-upgrade.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: jpalma

Thanks for brining this to our attention. We will investigate it further and update you shortly.

Hi @ArminEberle could you update your CLI or CLI extension version to the latest. It's likely your version doesn't have that output yet.

Hi @palma21 thanks for your reply, this has fixed the issue. My az cli version was  2.11.1 before, after upgrade to 2.20.0 the "latestNodeImageVersion" attribute is returned properly.


feat: add information on how to merge a PR on the mobile app

Why:
To help users learn how to merge PRs on GitHub's mobile app without having to do experiments.
Closes:
#26617

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[X] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[X] For content changes, I have completed the self-review checklist.

Note
I need a bit more assistance on how to create a Mobile section in the sidebar.

Yes


sftp

looks good. DO you know if it supports sftp ?

It's not yet.

Ok. I am looking for a client and server that implements it with an admin
GUI
On Mon, 29 Feb 2016, 12:33 Lunny Xiao<EMAIL_ADDRESS>wrote:

It's not yet.
—
Reply to this email directly or view it on GitHub
https://github.com/goftp/server/issues/5#issuecomment-190166528.

Github.com/goftp/ftpd maybe is what you want.

thanks @lunny.
Yes i had a look at it and its pretty good, and am working with that.
GUI i will have to add myself but thats not too bad.


please add Python-3.8 in AppVeyor

I fail to compile it myself

targeting GDAL-3.0.0 along the way may be nice

needed to get latest ipyleaflet working in Python-3.8

@stonebig Fiona currently does not (yet) support gdal 3.0, see https://github.com/Toblerity/Fiona/issues/745
On travis / linux, fiona builds with Python 3.8.0b1+ with cython 0.29.10 (however, it fails to compile with version 0.28.3 specified in requirements-dev.txt). Regarding appveyor, I'm not sure if Python 3.8 is already available on Appveyor.

ok, I workarounded the problem by downgrading from ipyleaflet-0.10.4 to ipyleaflet-0.10.2 that doesn't requires geopandas and so, fiona.


OpenLogic images do not support SSH access

I started trying to build centOS images from packer-azure, and it's going great. Everything gets saved properly. However, when I try to create a new viritual machine from the the created image, even if I only give it a password for SSH authentication, I get denied from logging in on the basis of:
Permission denied (publickey,gssapi-keyex,gssapi-with-mic).
Is this a packer-azure issue? Or should I just close this issue, and talk to Microsoft Azure about this?

The previous maintainer made a note about this here: https://github.com/MSOpenTech/packer-azure#known-issues
I'm not sure what the status of this issue is or if it's fixable. I can certainly (help to) look into it...

@ldanielw1 did you find that this issue still exists when building CentOS images?

Yes, this bug still exists.

Thanks for confirming!

Hi there! I am hitting this issue too. Is there a fix or workaround? Thanks!

I think the problem might be in CreateVirtualMachineDeployment.go ? Not sure why it wouldn't affect Ubuntu VM's too ...
buff.WriteString("<DisableSshPasswordAuthentication>true</DisableSshPasswordAuthentication>")

Hi @tomconte
AFAIK, the azure agent is supposed to handle the value of DisableSshPasswordAuthentication in the VM deployment xml and configure SSH daemon accordingly. It may have a bug there for the case of CentOS...?
Also not sure if waagent --deprov is supposed to clean that up...

Ok, I looked it up... The Azure agent only takes action if it needs to disable password auth:
https://github.com/Azure/WALinuxAgent/blob/ddfe7821580842f6f2de1a9ee2c68629bd26a5bf/waagent#L4441-L4447
So there is no way to undo this. I'll change the code to not disable password auth because it can't be undone by a re-deploy.


Ticket bulk import?

I'm trying to access the ticket bulk import feature. I've tried numerous variations on the following without much luck:
client.tickets.create_many!(my_tickets, :path => 'imports/tickets')

Thanks in advance for any suggestions.

I got this to work:
client.tickets.request(tickets: my_tickets, :verb => :post, :path => 'imports/tickets/create_many').fetch

If there's a more canonical/elegant way to do it, please let me know!

I'm trying the same with update_many, but can't make it work

just got my issue.. I was using a pretty old version of the sdk. You should probably have access to ZendeskAPI::Ticket.create_many, which is defined here https://github.com/zendesk/zendesk_api_client_rb/blob/e9ddd8f504477fa62c0e1cd5a3b79f9adc84d626/lib/zendesk_api/actions.rb#L168-L182

Hi @ericsaboia, ZendeskAPI::Ticket.create_many calls the tickets/create_many endpoint, but I'm specifically trying to access the imports/tickets/create_many endpoint. If you'd read above you'd find my explanation of what I'm trying to do, and you'll see that my current solution is, in fact, using create_many! on a collection of ZendeskAPI::Ticket objects.

@mwpastore sorry for the late reply. Our client does not support importing many tickets yet, and we don't have an ETA about it either. I've created  an internal note of the request and we will review it in the future.


Add alias for resource type: directory

The recommended way to specify a directory (e.g, of training images) as a run resource is like this:
#| requires:
#|   - file: celeba_gan
#|     target-type: link

Two notes:

It would be nice to accept dir or path instead of file here, to make clear that the resource is a directory.
On Windows, this resource type may need some investigation. In the case where celeba_gan is a large directory of images, guild run has a very long startup time (80 seconds on a modern desktop just to stage 1 script + a symlink). I haven't investigated the cause yet, but it might due to sourcecode rules attempting to index the resource dir unnecessarily.

For testing, here is an easy way to get a large directory of images locally:
pip install gdown
gdown 1O7m1010EJjLE5QxLZiM9Fpjs7Oj6e684

This will download a 'img_align_celeba.zip' file, which you can unzip to create 'celeba_gan'.


GPU benchmark for moist case

@szy21 is there a config we should use?

We can use this one: https://github.com/CliMA/ClimaAtmos.jl/blob/main/config/longrun_configs/longrun_hs_rhoe_equilmoist_nz63_0M_55km_rs35km.yml, but change the resolution to the targeted one.

Currently failing here: https://buildkite.com/clima/climaatmos-target-gpu-simulations/builds/82#018ad87d-0c3a-49d1-8335-e88c6f4a525f/138-201

Can we also add to the CI pipeline as a soft fail for now?


Matthew

fixes #82
Thinking about this pattern for different storms - we create a branch from upstream named {storm_name}-{NOAA_storm_key}, then PR to that w/ storm-specific code.
The tricky part w/ this pattern is pulling in changes from upstream master that we want to apply to storm branches. I think we probably want to delete all data, .ind files, .remake dir, and the build/status dir locally (on the storm-specific branch), then pull in changes from upstream master, then delete ind and build/status again. Or is there a better way? Otherwise we are going to have conflicts w/ .inds and build.yamls.
Creates:

I didn't do much for formatting and text work, and didn't spend any time selecting sites, so we don't have any for FL, but see #97 discussion.

GIF looks good. Are you able to build the intro sequence as well?
Good point that the differences in build/status and .ind files will be an issue among branches.

The tricky part w/ this pattern is pulling in changes from upstream master that we want to apply to storm branches. I think we probably want to delete all data, .ind files, .remake dir, and the build/status dir locally (on the storm-specific branch), then pull in changes from upstream master, then delete ind and build/status again. Or is there a better way? Otherwise we are going to have conflicts w/ .inds and build.yamls.

I'm wondering about the deletion of everything twice...does the above approach mean you have to rebuild all the data each time you pull changes from master?
Some options, not mutually exclusive:

Build each vizzy branch in a different local folder - that way your .remake, build/status, and data files can all be unique to each storm without needing to overwrite other storms.
Delete all storm-specific files from the master branch permanently: build/status/*.yml files, *.ind files, and lib/cfg/gd_config.yml. This would make the master branch impossible to test on its own, but you could test the code in the context of a storm, and there wouldn't be conflicts between master and any storm-specific branches.

I like  {storm_name}-{NOAA_storm_key} as the branch naming convention. Could we safely go even simpler to {storm_name}-{2_digit_year}, e.g., matthew-16? All storm names are unique within a year, right?

GIF looks good. Are you able to build the intro sequence as well?

no...and I've had a mix of results with adding the intro. Would you mind sharing your workflow when you add the intro to the build? I end up getting errors related to missing png files lately, but I haven't dug into it.

I'm wondering about the deletion of everything twice...does the above approach mean you have to rebuild all the data each time you pull changes from master?

yes, I have to do big rebuilds when I switch branches or pull from master. I don't think this is ideal at all, but was thinking my current use-case is kind of temporary (hopping between storms and testing things out).
out of your two options above, option two seems to be the easiest to implement in our current set up, and seems like a clever idea to me. Did you have a preference?

I don't think we are yet in a place where we know how to spawn a new storm in the repo. I consider this PR to be a test case or an example of one pattern. I think we should probably do something different after reading Alison's comments:

Build each vizzy branch in a different local folder - that way your .remake, build/status, and data files can all be unique to each storm without needing to overwrite other storms.
Delete all storm-specific files from the master branch permanently: build/status/*.yml files, *.ind files, and lib/cfg/gd_config.yml. This would make the master branch impossible to test on its own, but you could test the code in the context of a storm, and there wouldn't be conflicts between master and any storm-specific branches.

I am going to close this and focus on getting the good fixes in here onto a PR to master, and then we'll figure out the pattern of how we build a new storm.

Sounds good - you've made good progress regardless. I think those two options can both be used together, or option 1 can be optional on a person-by-person basis. If you like it, too, then I think we should definitely do option 2.

For rebuilding the intro, I wonder if you need to call one or all of the following:
scmake('6_visualize/log/6_storm_gif_tasks.ind', force=TRUE) # shouldn't need force, but might help
scmake('final_intro_frames', force=TRUE) # more likely to need force here
scmake('6_visualize/out/animation_a.gif', force=TRUE)

The second line is only needed if you're using final_intro_frames as input to the combine_animation_frames command under 6_visualize/out/animation_a.gif. You may also need to call scmake('final_intro_frames', force=TRUE) periodically in the same conditions and for the same reason: neither function explicitly relies on the contents of the /tmp directory, but the results of existing_png_tasks describe those contents and so need to be rebuilt whenever the list of actual .png files within /tmp/ changes.
Until you merged my PR this morning, I don't think we had shared working code for the intro frames because we needed to switch to the dynamic version of the legend function. So that may also have played a role in your troubles.

I think scmake('final_intro_frames', force=TRUE) is what I was missing here.
What I'd like to do is progress far enough into the "base storm" and then move into option 2 for this master branch. Sound good?

I think that sounds good but am not clear on your precise plan. What's the "base storm" - Matthew, or storm rather than intro animation, or something else? What are your criteria/goals for "progress far enough into" the base storm?

I was thinking the base storm is Harvey, which is what the master branch is on. I don't have a precise plan, but it seems there are some open issues that should be wrapped up before we delete all of the .ind and build yamls from the master. I was planning on making some progress there.
With the instructions above, when do you build from the 6_intro_gif_tasks.yml? Should that first command be scmake('6_visualize/log/6_intro_gif_tasks.ind', force = TRUE) instead? either way, I am getting a Error in mutate_impl(.data, dots) :    Column y must be length 1 (the number of rows), not 3 error on the spark plot plotting function when I build the final .ind file from 6_intro_gif_tasks.yml

Got it. Sounds good - will indeed be easier to test and debug while there are still .ind and build/status files in the repo.
Oops, yes, I should have written scmake('6_visualize/log/6_intro_gif_tasks.ind', force = TRUE) above.


Makefile: do not gen error code before build

Signed-off-by: 5kbpers<EMAIL_ADDRESS>
What problem does this PR solve?
Problem Summary:
Previously we introduced an error-code pre-condition of every build command in Makefile, which makes the build process slower.
What is changed and how it works?
Proposal: xxx
What's Changed:
Remove the error-code pre-condition in Makefile.
Related changes

Need to cherry-pick to the release branch

Check List
Tests

No code

Release note

N/A

/run-build-image

/merge


Hello gentleman,download checkpoint to where?

Hello gentleman,download checkpoint to where?When I run the playground it report no model.Where should I pull checkpoint to?Help!Thanks

You don't need to download the model. It should grab it from hugging face.
riffusion/riffusion-model-v1 is the repository on hugging face.


Feature/22 jwt support

Pull Request Information
Relates to Github Issue: #22
What is the PR Type:
feature
Describe the added/fixed behaviour:

Allows users to login. If the username and password combination matches an existing user, will create and send a jwt token to the client.
JWT token can be sent in the authorization header of subsequent requests to successfully enter [authorize] endpoints (as the user is logged in)

Checklist

[x] Pull request has suitable title, adequately described and associated with a github issue
[x] Branch name follows correct convention (e.g. feature/[id]-some-feature)
[x] Branch builds and executes with no errors
[ ] Test cases has been generated for the feature and test cases pass
[x] Updated relevant wiki/documentation

Final Reminders
Please remember to squash the commits

Will there be a quick wiki thing showing how to use/test this?

Will there be a quick wiki thing showing how to use/test this?

I've created a rough wiki explaining how to use it


Constant Error Pop-up

After the Winter '24 release in the sandbox orgs, we are receiving an error when trying to use or upload files onto the "Lightning File Explorer" installed package. This error is only happening on Sandboxes but not on our production org - I believe because the Winter '24 release has not happened on production yet.
We can technically close the error pop-up and proceed but it keeps popping up every time we try to do something (e.g. open the file explorer, select a file template, move files, etc). A screenshot of the error pop-up is attached.

We are also experiencing this issue. Any news on this?

We are also experiencing this issue. Any news on this?

I am also receiving an error on File Explorer, but only on one specific record so far:

Update on my issue: Somehow I ended up with files with one linked entity id looking up folders with a different linked entity id. Cleaning up that data fixed my issue!

Same issue. It has a weird behaviour as in some orgs works and some others (same type) doesn't.

I cloned this repo and deployed the source code into a sandbox and found the error.  In qsydFileExplorer.js, lines 98 should be changed from this:
this.addEventListener(CONSTANTS.CUSTOM_DOM_EVENT_TYPES.DATA_LOADED,this.handleDataLoaded);
to this:
this.addEventListener(CONSTANTS.CUSTOM_DOM_EVENT_TYPES.DATA_LOADED,this.handleDataLoaded.bind(this));
I will be deploying the unpackaged code and changing all references to the package, after which I will uninstall the package.

Hey,
I am also facing the same issue. Was anyone able to resolve it?
Please let me know


fix typo in install-w-machine.md

Signed-off-by: mqliang<EMAIL_ADDRESS>

LGTM.
Thanks @mqliang.


Problems with workspace configuration - Go 1.26.1 + VS Code

Hello! When I try to set up a working environment in the vs code editor to comfortably run and debug simple tests connected to the library module I am developing, I constantly get BrokenImport errors ("could not import libs.com/wclib (cannot find package "libs.com/wclib" in GOROOT or GOPATH)"). I run VS Code in the directory of the test module (projects/myproject/test/simple/ - see below). The GO111MOLUDE environment variable is set to auto. At the same time, the go build command works fine without errors. The executable file is linked and runs without problems.
After configuring the multi-root workspace in vs code, the BrokenImport error disappears for a time when switching to the library module (projects/myproject/lib/wclib.go). But the notification appears: "Error loading workspace: You are outside of a module and outside of $GOPATH/src. If you are using modules, please open your editor to a directory in your module. If you believe this warning is incorrect, please file an issue: https://github.com/golang/go/issues/new.". When switching to the test module (projects/myproject/test/simple/go.mod), BrokenImport appears again.
Here is the project structure i have:
projects/myproject
   +lib
     |wclib.go
     |go.mod
   +test
     +simple
       |main.go
       |go.mod
       |go.work

here is the content of the header and import section in the projects/myproject/lib/wclib.go
package wclib

import (
	"bytes"
	"container/list"
	"context"
	"crypto/tls"
	"encoding/json"
	"fmt"
	"io"
	"net/http"
	"reflect"
	"sync"
	"time"
)

here is the content of the projects/myproject/lib/go.mod
module libs.com/wclib

go 1.21.6

here is the content of the header and import section in the projects/myproject/test/simple/go.mod
package main

import (
	"fmt"
	"time"

	"libs.com/wclib"
)

here is the content of the projects/myproject/test/simple/go.mod
module example.com/ilya2ik/wclib/simple

go 1.21.6

replace libs.com/wclib => ../../lib

require libs.com/wclib v0.0.0-00010101000000-000000000000

here is the content of the projects/myproject/test/simple/go.work
go 1.21.6

use (
	.
	../../lib
)

Please help me. Maybe I'm missing something.


Add SM

Official announcement:https://twitter.com/Minesweeper_SM/status/1480796206452273152
Team background：
The SuperMinesweeper team is located in Australia, and the core members have more than 10 years of Internet product development experience on average，Greg is an experienced CEO with an amazing history of contributing significant value to Fortune 500 companies. Greg has expertise in developing business strategies and a strong interest in the blockchain space. As an early adopter of NFT and a fan of multiple fields including games, art and anime, Greg is striving to define the NFT ecosystem with cutting-edge innovative solutions.
Project Basic Information：
The biggest difference between "Super Minesweeper" and Microsoft Minesweeper is that it empowers the game map to make a perfectly replicated decentralized + NFT + P2E + DeFi game for the vast number of cryptocurrency lovers and Minesweeper lovers. The game map that users play is not generated by the system, but a unique game map created by users, uploading the map parameters to the block chain to obtain NFT credentials, Minesweeper starts by referring to the NFT parameters on the chain to obtain the game map. At the same time, the user creates and generates the game map NFT can be stored and traded.
Website：https://sm.tt
Twitter: https://twitter.com/Minesweeper_SM
Media reports:
AP NEWS https://apnews.com/press-release/kisspr/technology-blockchain-2f1c91832b02eac815f95f154873605b
Recommended gas limit: 60000
Exchange：
https://pancakeswap.finance/swap?outputCurrency=0x119f095a45df2a77337042bcc4f2e52fa6e7e881

hi, we have received your information, but the json file has some bug, please upload the right file.

It also says Line 15 has parsing error.