Enabling Geospatial causes a white screen

When you enable Geospatial the screen turns white. I've attached a screenshot where I've just set sceneView.geospatialEnabled = true in the sample-ar-model-viewer sample. I'm using v0.9.0 / the current main branch. I'm testing on a Pixel 6 with Android 13, which could be the culprit.

On my Motorola One Action (Android 11) the screen weirdly goes complete black instead:

On another phone (Oneplus 6T, Android 11) the screen glitches out completely with some weird textures:

Maybe you forget the location permission

Good idea, but I have granted ACCESS_FINE_LOCATION and ACCESS_COARSE_LOCATION to the app and still get the same result.

@eschoenawa
I also had the same problem.
One of the following will solve the problem.

Declaration in AndroidManifest<uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
<uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />

Declaration in AndroidManifest      <meta-data
          android:name="com.google.android.ar.API_KEY"
          android:value="@string/GoogleCloudApiKey" />

      <meta-data
          android:name="com.google.ar.core"
          android:value="required"
          tools:replace="android:value" />

requestPermissions ACCESS_FINE_LOCATION and ACCESS_COARSE_LOCATION

Not fixed, facing same problem in Redmi note 10 pro


Mechanism to discard a document

If a document has a key discard set to true, irrespective of what the doc contains it should be discarded. This can be helpful in cases where although the object creation is successful and schema validation suceeds, the doc might not have enough data thats useful.

https://review.gerrithub.io/#/c/redhat-performance/scribe/+/427932/ brings this functionality


Feature: Toggle option(ignore_all_space, word_diff) for diff view

closes 173

It is not ready to get merged. We need some syntax file for word-diff

This is great @stoivo - thanks!  I think we should leave the syntax highlighting until a later time - I don't think it'll be necessary to have this to start.
I had a handful of comments, and please fix the alignment issues (in case you're unfamiliar, you can read up on PEP8 guidelines and search for assignment).
Once that is ready, I'll merge this in!

fixed. Ready to get merged


foxglove_bridge: 0.2.0-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository foxglove_bridge to 0.2.0-1:

upstream repository: https://github.com/foxglove/ros-foxglove-bridge.git
release repository: https://github.com/ros2-gbp/foxglove_bridge-release.git
distro file: rolling/distribution.yaml
bloom version: 0.10.7
previous version for package: 0.1.0-1

foxglove_bridge
* Add support for client channels (#66 <https://github.com/foxglove/ros-foxglove-bridge/issues/66>)
* Add smoke tests (#72 <https://github.com/foxglove/ros-foxglove-bridge/issues/72>)
* Update package maintainers (#70 <https://github.com/foxglove/ros-foxglove-bridge/issues/70>)
* [ROS2]: Fix messages not being received anymore after unsubscribing a topic (#92 <https://github.com/foxglove/ros-foxglove-bridge/issues/92>)
* [ROS2]: Refactor node as a component (#63 <https://github.com/foxglove/ros-foxglove-bridge/issues/63>)
* [ROS2]: Fix message definition loading for .msg or .idl files not located in msg/ (#95 <https://github.com/foxglove/ros-foxglove-bridge/issues/95>)
* [ROS1]: Mirror ROS 2 node behavior when /clock` topic is present (#99 <https://github.com/foxglove/ros-foxglove-bridge/issues/99>)
* [ROS1]: Fix topic discovery function not being called frequently at startup (#68 <https://github.com/foxglove/ros-foxglove-bridge/issues/68>)
* Contributors: Hans-Joachim Krauch, Jacob Bandes-Storch, John Hurliman

bloom version: 0.10.7

I just noticed (a moment too late for several other now reverted release PRs) that the bloom version used for these releases is outdated. Can you please update bloom and re-bloom these releases. Thanks!

ok, will do


PacketType.valueOf throws IllegalStateException

Hi,
I'm using socket.io server 1.7.17.
I'm connecting to a namespace /profile-0 from Chrome using

"socket.io-client": "2.0.3"

Client connects normally. When he sends first message to login I see on server this error:

2019-09-06 10:28:47,673 [nioEventLoopGroup-3-1] [ERROR] [c.c.s.handler.InPacketHandler] - Error during data processing. Client sessionId: e79dfa62-a088-4e52-b019-0612656df755, data:
java.lang.IllegalStateException: null
at com.corundumstudio.socketio.protocol.PacketType.valueOf(PacketType.java:48)
at com.corundumstudio.socketio.protocol.PacketDecoder.readType(PacketDecoder.java:86)
at com.corundumstudio.socketio.protocol.PacketDecoder.decode(PacketDecoder.java:160)
at com.corundumstudio.socketio.protocol.PacketDecoder.decodePackets(PacketDecoder.java:142)
at com.corundumstudio.socketio.handler.InPacketHandler.channelRead0(InPacketHandler.java:65)
at com.corundumstudio.socketio.handler.InPacketHandler.channelRead0(InPacketHandler.java:36)
at io.netty.channel.SimpleChannelInboundHandler.channelRead(SimpleChannelInboundHandler.java

I see PacketType supported are
` OPEN(0), CLOSE(1), PING(2), PONG(3), MESSAGE(4), UPGRADE(5), NOOP(6),
CONNECT(0, true), DISCONNECT(1, true), EVENT(2, true), ACK(3, true), ERROR(4, true), BINARY_EVENT(5, true), BINARY_ACK(6, true);`

But type received is 12.
https://i.imgur.com/Q08Vrbn.png
Here is whole msg flow:
https://i.imgur.com/Dfc99YX.png

Also I want to disallow default namespace "".
If I do socketIOServer.removeNamespace(""); I get:

An exception was thrown by com.corundumstudio.socketio.transport.WebSocketTransport$1.operationComplete()
java.lang.NullPointerException: null
at java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:936)
at java.util.concurrent.ConcurrentHashMap.containsKey(ConcurrentHashMap.java:964)
at java.util.concurrent.ConcurrentHashMap$KeySetView.contains(ConcurrentHashMap.java:4558)
at com.corundumstudio.socketio.handler.AuthorizeHandler.connect(AuthorizeHandler.java:255)
at com.corundumstudio.socketio.transport.WebSocketTransport.connectClient(WebSocketTransport.java:195)
at com.corundumstudio.socketio.transport.WebSocketTransport.access$200(WebSocketTransport.java:57)
at com.corundumstudio.socketio.transport.WebSocketTransport$1.operationComplete(WebSocketTransport.java:176)

because
`  Namespace ns = namespacesHub.get(Namespace.DEFAULT_NAME);
    if (!client.getNamespaces().contains(ns)) {`

ns is null.

@MatCuk , I am facing the same issue. I receive packet type 12 from Android client. Did you find anything useful?
Do you know what does packet type 12 means?

I am facing the same issue, when trying to connect to a particular namespace. But the connection is successful even after the exception occurs.

me to,  I get packet type 15, why?

me to,  I get packet type 15, why?

I get same number 12

Seems to happen on binary data, which might be sent in a different packet, without the package type.
if you prepend the binary blob with b'\x04, so it's marked as a message from the client'.
        data=b'\x04' + <rest of binary data>
it should be received ok on the other side.
not sure yet if it's a client issue(though it happens in clients written in python and js) or a server issue(don't have issues on a python server). Did you guys find a workaround? ie. custom InPacketHandler or additional pipe to handle this case?

simple way to reproduce from postman: send binary data, based on endianness, second byte is decoded as the packet type. in attached case 8
using: com.corundumstudio.socketio:netty-socketio:2.0.11


Setting default query string for every request

I am trying to set default query string to be passed for every request but it's not picking up
resty https://api.hipchat.com/v2 -H "Content-Type: application/json" -q "auth_token=ASD"

@jayyvis I retest your query with the new dry run feature and it seams to be working normaly:
> resty https://api.hipchat.com/v2 -H "Content-Type: application/json" -q "auth_token=ASD"
https://api.hipchat.com/v2*

> GET / --dry-run
curl -sLv '' -X GET -b "/Users/abecchis/.resty/c/api.hipchat.com" -c "/Users/abecchis/.resty/c/api.hipchat.com" -H Content-Type:\ application/json -q auth_token=ASD "https://api.hipchat.com/v2/"

> POST / --dry-run
curl -sLv '' -X POST -b "/Users/abecchis/.resty/c/api.hipchat.com" -c "/Users/abecchis/.resty/c/api.hipchat.com" -H Content-Type:\ application/json -q auth_token=ASD "https://api.hipchat.com/v2/"

Have you tried with latest version?
(sorry for the late answer ::/)


Support defining accounts instead of the randomly generated ones

It also adds a command line argument --account=1f1f... which can be passed multiple times to enable more accounts.

Switching Dapple over to use TestRPC as it seems more actively developed. This feature is something we need in order to successfully make the switch.

Great to hear @ryepdx. Looking at it now! And thanks @axic!

Question for both of you (@ryepdx, @axic): If the accounts list were input as an optional configuration file, much like a genesis.json file, instead of command line arguments for each account, would that still satisfy your use cases? We've been throwing that one around and we think a genesis.json would be easier to use.
Here's an example: http://jev.io/genesis_block.json
Either we could strictly adhere to the genesis_block.json format and implement and allow you to set all initial values listed there on startup, or we could create our own format that only deals with accounts. In either case, I agree an optional private key would be great to have TestRPC recognize.

With that specific genesis.json you are losing the ability of signing within testrpc. That can be replicated by provider-engine in many apps, but why lose that capability if it is already written?
I think the

proposed genesis file + commandline option
proposed genesis file + configuration file
an improved genesis file

are all possible options.
In my opinion --accounts is not mutually exclusive with the genesis file. It serves a different purpose as it stands.

Giving a second thought I'm leaning towards:

supporting the official genesis json format for simplicity
having a separate configuration testrpc specific configuration file or command line option
optionally: support keystore files for testrpc managed accounts (you can use ethereumjs-wallet to import/export easily)

an improved genesis file

Definitely agree the improved genesis file is the right answer, which could include the private keys needed for signing.

In my opinion --accounts is not mutually exclusive with the genesis file. It serves a different purpose as it stands.

I'm not necessarily sure I agree with that. The genesis file would include a list of initial accounts and, optionally, their starting balance. If I understand this PR correctly, that's what you're trying to achieve. If the improved genesis file were also included, --account would be another avenue for specifying the same information.
Assuming we go the genesis file route (not set in stone), here's what I imagine it would look like:
{
  // likely won't matter:
  nonce: "0x0000000000000042",

  // won't matter
  difficulty: "0x400000000",

  // specify initial accounts, balances, and private keys
  alloc: {
    "3282791d6fd713f1e94f4bfd565eaa78b3a0599d": {
      balance: "1337000000000000000000"   // Optional. Will be set to default if not specified.
      private_key: "..."                  // Optional. Non-standard.
    },
    ...
  },

  // Could matter in rare cases
  mixhash: "0x0000000000000000000000000000000000000000000000000000000000000000",

  // Easy to set, could matter in rare cases
  coinbase: "0x0000000000000000000000000000000000000000",

  // Likely won't matter
  timestamp: "0x00",

  // Likely won't matter
  parentHash: "0x0000000000000000000000000000000000000000000000000000000000000000",

  // Likely won't matter
  extraData: "0x11bbe8db4e347b4e8c937c1c8370e4b5ed33adb3db69cbdb7a38e1e50b1b82fa",

  // Good for specifying the initial gas limit.
  gasLimit: "0x1388"
}

Alternatively, we could create our own config. Something like:
{
  accounts: {
    "0x1234...": {
      balance: ...
      private_key: ...
    }
  },
  coinbase: "0xabcd...",
  // other options...
}

The good thing about using the genesis.json format is that the TestRPC could share configuration with other Ethereum clients that support it. The bad thing about it is there's extra config. In both cases, however, setting initial accounts and their private keys will be one of the main features.

@axic Looks like we posted at the same time.

supporting the official genesis json format as is
having a separate testrpc specific configuration file or command line option
optionally: support keystore files for testrpc managed accounts (you can use ethereumjs-wallet to import/export easily)

Completely agree. And supporting keystore files would be :boom: :moneybag: :tada:
I'm leaning toward 1 and 3. I suppose the question is: Should we also support --accounts? I could see how it could be useful. There would be many avenues for the same information, though.

@tcoulter if both genesis and the keystore is supported probably there is no need for that tbh. (You can still use bits of the PR - the first two commits.)

being able to set a full genesis is awesome, but my primary usage is just getting ether in an externally managed account. currently i do this by starting the testrpc and sending a tx from one of the managed accounts giving ether to my account. but it adds another setup step to the process.
all im looking for is testrpc --account 0xc25d4cb29b1ce5f7b14be118d8b6b1f0493b664a

@kumavis The jury's still out on whether this should be merged, but you can write a quick script that will do this for you:
var TestRPC = require("ethereumjs-testrpc");
var Web3 = require("web3");

var server = TestRPC.server();

server.listen(8545, function(err, accounts) {
  if (err) throw err;

  var web3 = new Web3();
  web3.setProvider(new Web3.providers.HttpProvider("http://localhost:8545"));

  web3.eth.sendTransaction({
    from: accounts[0],
    to: "0x1234..."     // your address,
    value:<PHONE_NUMBER>00 // a gazillion
  }, function(err) {
    if (err) throw err;
    console.log("Success!");
  });
});

How about having --unlock when:

presented with a small number unlocks the relevant keystore file (as in geth)
presented with an address unlocks the relevant keystore file
presented with a private key loads that key

unlocks the relevant keystore file
@axic there are no keystore filesm, everything is in memory

@kumavis - I think he's suggesting copying over keystore files from geth.
On Mar 11, 2016 8:03 PM, "kumavis"<EMAIL_ADDRESS>wrote:

unlocks the relevant keystore file
@axic https://github.com/axic there are no keystore filesm, everything
is in memory
—
Reply to this email directly or view it on GitHub
https://github.com/ethereumjs/testrpc/pull/24#issuecomment-195655273.

@kumavis it has been suggested a few comments up that it could be possible to support keystore files, too

Just an update regarding the switch to TestRPC: ended up running up against some sort of non-deterministic behavior in TestRPC which led to certain tests in Dapple failing. Ended up being easier to just update EtherSim's dependencies. Will keep an eye on TestRPC going forward, though. If you're interested in poking around yourself, the branch with the sometimes-failing tests is at https://github.com/ryepdx/dapple/tree/testRPC

Will definitely check it out @ryepdx. Can you tell me which tests I should
be running?
On Mar 16, 2016 6:24 PM, "Ryan"<EMAIL_ADDRESS>wrote:

Just an update regarding the switch to TestRPC: ended up running up
against some sort of non-deterministic behavior in TestRPC which led to
certain tests in Dapple failing. Ended up being easier to just update
EtherSim's dependencies. Will keep an eye on TestRPC going forward, though.
If you're interested in poking around yourself, the branch with the
sometimes-failing tests is at
https://github.com/ryepdx/dapple/tree/testRPC
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/ethereumjs/testrpc/pull/24#issuecomment-197601690

npm test should trigger it. IIRC, it was the DSL tests that were being affected.

@ryepdx Can you create a ticket that I can comment on on your dapple branch? I'd like to move this discussion there. If not, can you create a new TestRPC ticket for your specific issue? I have some questions, but don't want to bog down this ticket.
@axic @kumavis - I've been coming around to merging this PR, so the functionality exists at least for the short term. All opposed say "eye".

Issue created.

Update on this. I'm adding a feature that will make the testrpc initial state deterministic, mostly to debug a race condition with the testrpc itself. This PR will be included with that feature. If all goes well, expect something today. (famous last words)

Merged this PR. Thanks @axic!
Along with it I added two other options:

-s/--seed: Seed the random number generator, so accounts created are deterministic. Usage: testrpc -s someseed
-a/--accounts: Specify the number accounts created on initialization. Usage: testrpc -a 2

This is in addition to @axic's --account options.

Also added one more option:

-d/--deterministic: Use a default seed and make addresses deterministic. This is for when you don't care about what the seed is. Usage: testrpc -d.

Great stuff, thanks @tcoulter!
One note on the deterministic feature, it would be still very useful to either:

provide a tool or command line option to print the private keys for the generated ones
or display the private keys of the generated ones upon running

One small risk I see is if in the future some other code is referenced by addAccount which uses the rng, then it won't generate the same 10 accounts.

Good feature requests @axic, totally agree.
Regarding:

One small risk I see is if in the future some other code is referenced by addAccount which uses the rng, then it won't generate the same 10 accounts.

Can you elaborate? I'm not sure I understand.

It will only create the same set of accounts, if the following code is strictly run in succession and nothing else calls it between account creation:
var secretKey = opts.secretKey || random.randomBytes(32, this.rng);
Right now addAccount is executed via async.times(). Make sure nothing else uses the same this.rng instance at least until the accounts are done.

Speaking of account creation, maybe you could use the lightweight HD wallet feature from ethereumjs-wallet and just generate 10 accounts on a specific HD path? 😎

@axic I think that's handled, mostly due to the initialization code in the Manager. Requests won't be  pushed through until the accounts have been created. The only other thing that could use the rng is in that case is more of TestRPC's initialization.
Relevant code:

https://github.com/ethereumjs/testrpc/blob/master/lib/manager.js#L54
https://github.com/ethereumjs/testrpc/blob/master/lib/manager.js#L20

Speaking of account creation, maybe you could use the lightweight HD wallet feature from ethereumjs-wallet and just generate 10 accounts on a specific HD path?

I'm not well versed in the lightwallet, but I will accept a PR. :) What benefit would that provide other than being fancy? This is a mock ethereum client so we don't need to be (cryptographically)secure after all.

Oh, hmm. If we use the lightwallet, that means the private keys are deterministic as well correct?

Yes, it would be fully deterministic and users could easily generate the same set. No magic required from testrpc's part.

Sample code:
const bip39 = require('bip39')
const hdkey = require('ethereumjs-wallet/hdkey')

var mnemonic = 'awake book subject inch gentle blur grant damage process float month clown'
var wallet = hdkey.fromMasterSeed(bip39.mnemonicToSeed(mnemonic)

var account = wallet.derivePath("m/44'/60'/0'/0/" + index) // index is a number
var address = account.getWallet().getAddressString() // 0x prefixed String
var publicKey = account.getWallet().getPublicKey() // Buffer
var secretkey = account.getWallet().getPrivateKey() // Buffer

@axic And you'd do this 10 times like we're doing now? Only issue is mnemonic's are fairly nasty from a UI perspective. Imagine having to input the pneumonic on the command line.
If only there were a less secure, easier to use version. I might opt to simply printing out the private keys.

Although -- thinking out loud at this point -- if the -d option were all that was needed, and we output the pneumonic for HD wallet users, that'd be a fine experience. But I suppose we could input the pneumonic at that point as well.

You do derivePath and the subsequent lines for each account.
On a commandline:
--mnemonic "awake book subject inch gentle blur grant damage process float month clown"
Assuming testrpc has a default mnemonic built in, the user could use import this mnemonic into the client app too.
Printing the private key on top of that I think is a useful feature anyway.

Ya, I'm :+1:, we should support it. I created a new ticket so we can discuss there: https://github.com/ethereumjs/testrpc/issues/42


Help writer improvements

Summary of changes
Version option will show in help even with a default command
Reserve -v and --version as special spectre.console command line arguments (nb. breaking change for spectre.console users who have a default command with a settings class that uses either of these switches)
Help writer correctly determines if trailing commands exist and whether to display them as optional or mandatory in the usage statement
Ability to control the number of indirect commands to display in the help text when the command itself doesn't have any examples of its own. Defaults to 5 (for backward compatibility) but can be set to any integer or zero to disable completely.
Significant increase in unit test coverage for the help writer
Minor grammatical improvements to website documentation

DON'T MERGE THIS PR
I propose to merge the changes in this PR through it's ancestor PR https://github.com/spectreconsole/spectre.console/pull/1259  (once merged, I will delete this PR)

@FrankRay78 I've converted this PR to a draft so it won't be accidentally merged


License not found

Not entirely sure if it's the root cause but when I took a dependency on Paramol 0.0.122 paket install failed with 404 - Not Found (which is correct since in that version Paramol's nuget was pointing to a non existing license url). Is failure expected when the license can not be found?

are you sure that this is the error? Paket should only warn (in yellow) if a license is missing. It should not fail.

I should add that paket was invoked from MSBuild. Might the paket return code 1 cause MSBuild to barf when ContinueOnError="false"?

@forki Correct, from the command line that is the observed behavior.

Might the paket return code 1 cause MSBuild to barf when ContinueOnError="false"?

what's that? can you fix it in https://github.com/fsprojects/Paket/blob/master/.paket/paket.targets ? ;-)

I'm not using auto-restore nor the paket.targets. I'm just using a simple Exec task to invoke paket. So in that case it will be my own code, euh, well, xml build file.


Add watchdog timer to control node

If no new messages have been published to the controller for a certain amount of time (50ms to start), kill all thrusters.

Tajah has completed this and it has been tested on the Pose Control branch. Issue can be closed once pose control merges into master.


Working directory

Description
What steps will reproduce the problem?

my workinfg directory does not browse any txt, png or any other downloaded files.
 I don't understand very well your problem. Could you post a screenshot of what you mean to see what's happening in your case?

Hello, thank you for your response. I am sending two screenshots, one tehm
is the files that downloaded on my computer and the second one is the
working directory of spyder that is not giving an option to browse those
files
On Sun, Mar 28, 2021 at 10:22 PM Carlos Cordoba @.***>
wrote:

Hey @dikodiko https://github.com/dikodiko, thanks for reporting. I
don't understand very well your problem. Could you post a screenshot of
what you mean to see what's happening in your case?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/15041#issuecomment-809021215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATOAMFMMSSKJNZAQ7AUD3E3TF7P67ANCNFSM4Z6IAZIQ
.

I need to browse .txt, .png files, but the working directory is not giving
an option. It is letting me to browse only File Folders
On Mon, Mar 29, 2021 at 8:41 PM Diora Usmanova @.***>
wrote:

Hello, thank you for your response. I am sending two screenshots, one tehm
is the files that downloaded on my computer and the second one is the
working directory of spyder that is not giving an option to browse those
files
On Sun, Mar 28, 2021 at 10:22 PM Carlos Cordoba @.***>
wrote:

Hey @dikodiko https://github.com/dikodiko, thanks for reporting. I
don't understand very well your problem. Could you post a screenshot of
what you mean to see what's happening in your case?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/15041#issuecomment-809021215,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATOAMFMMSSKJNZAQ7AUD3E3TF7P67ANCNFSM4Z6IAZIQ
.

Hey @dikodiko, you can't send images via email because Github doesn't allow it. Instead, you need to drag and drop them here.

Closing because I was unable to reproduce this.


Fix conflict to https://github.com/NixOS/nixpkgs/commit/2a7130d13a032093f5394ef0961842d1e1928789

Description of changes
nixpkgs was updated (https://github.com/NixOS/nixpkgs/commit/2a7130d13a032093f5394ef0961842d1e1928789#diff-28ea4da3a9a595f8bd1a37a555c229f998f3a11b0715ebc5842ba266384eba73R326) and the default value of hardware.nvidia.prime.offload.enable was set to mkDefault reverseSyncCfg.enable;
This conflicted with the value set in prime, so I tried using mkOverride 990 instead. (https://discourse.nixos.org/t/what-does-mkdefault-do-exactly/9028)
Things done

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

bors merge


Excel Data Validation List on a column

Is it possible to filter the available values of a column (Excel -> Data -> Data Validation -> Allow: List) based on a list of values in a hidden sheet (in the image attached the 'LOV' sheet is hidden)?

That's amazing! Great work ;)

Done in v4.2.

addDataValidationRule
setTableColumnValidationRule
Data validation example

Thanks! ;)


[api] Allow to set volume as a relative step count

Fixes https://github.com/librespot-org/librespot-java/issues/272 by allowing a client to increase the volume by 10 steps (or decrease it by 5 steps).
To be discussed:

I don't accept both volume and step being present, but we could change this to give precedence to volume instead.
As the player already has volume clamping in its volumeUp/volumeDown logic, I don't validate that step is in any range.
The player could have a unique changeVolume(int steps) method instead of overloading volumeUp and volumeDown

# Valid requests
$ curl -X POST -d "step=10" localhost:24879/player/set-volume
$ curl -X POST -d "step=-5" localhost:24879/player/set-volume
$ curl -X POST -d "volume=30000" localhost:24879/player/set-volume

# Invalid input: both present
$ curl -X POST -d "step=15&volume=30000" localhost:24879/player/set-volume
{"name":"step","reason":"Cannot be passed alongside volume"}

# Invalid input on existing volume param
$ curl -X POST -d "volume=-30000" localhost:24879/player/set-volume
{"name":"volume","reason":"Must be >= 0 and <= 65536"}

# Invalid input on new step param
$ curl -X POST -d "step=0" localhost:24879/player/set-volume
{"name":"step","reason":"Must be non zero"}
$ curl -X POST -d "step=A" localhost:24879/player/set-volume
{"name":"step","reason":"Not an integer"}

Good
Good
I am keeping the those methods for backward compatibility

I thought you wanted to bypass the step value and step up/down the volume by an arbitrary amount, but it's perfectly fine this way too.


dungeon: extracted rand.Int() functionality into an interface

By abstracting randomness into an interface, it's much easier to test, since a predefined source can be used, and it enables passing in a rand.Rand object initialized with a specific randomness source, which is good for web applications.
It would seem that goimports did some formatting as well.

Thanks for your contribution!


Compiler cannot determine default value for super parameter correctly

The following file is analyzer-clean, but when compiling it via dart compile kernel using Dart 2.17.0-282.0.dev I am getting the error indicated in the comment below. I believe, the code is correct and it should compile without error. I expect showName to copy the default value of "true" from the DiagnosticsNode constructor.
class NumProperty extends DiagnosticsProperty {
  NumProperty({
    super.showName,
  });

  NumProperty.lazy({
    // 🔥 Error: Type 'bool' of the optional super-initializer parameter 'showName' doesn't allow 'null',
    //    but the parameter doesn't have a default value, and the default value can't be copied
    //    from the corresponding parameter of the super constructor.
    super.showName,
  }) : super.lazy();
}

class DiagnosticsProperty extends DiagnosticsNode {
  DiagnosticsProperty({
    super.showName,
  });

  DiagnosticsProperty.lazy({
    super.showName,
  });
}

abstract class DiagnosticsNode {
  DiagnosticsNode({
    this.showName = true,
  });

  final bool showName;
}

void main() {
  print('hello');
}

This is blocking for enabling super parameters in Flutter.

@pq could you route this to the right person?

Maybe @johnniwinther or @stereotype441 have ideas?

Not sure if this is related, but I was seeing a similar issue when rolling some super-parameter enabled code into google3 the other day. The error message was slightly different, though (and thus far I have been unable to reproduce this outside of google3): https://github.com/flutter/flutter/issues/101455

A fix is in progress https://dart-review.googlesource.com/c/sdk/+/240844
It is likely the root cause of https://github.com/flutter/flutter/issues/101455

It is likely the root cause of https://github.com/flutter/flutter/issues/101455

Confirmed that this issue is fixed as well. Thanks @johnniwinther!


Upadte strings.xml

By the way, are you going to release a new build? It has been 8 months since last release.

https://crowdin.com/project/xinstaller
Not sure if there will be any release in future


PR template added

Kindly review the code

LGTM. Approved. Thanks @soujo for contributing to Focusly.


cherry-pick yurthub code of pool-coordinator-dev

What type of PR is this?
/kind feature
What this PR does / why we need it:
Yurthub part of pool-coordinator implementation.
Functions of these code refer to: #778, #779
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
These code is cherry-picked from pool-coordinator-dev, which contains

PR
commit
Note

#1035
0db2b8749013b1e9ff2522a49790da60e0e22cff

#1073
61b6716e856aeb88528060eb4ef9984c14b3678b

#1103
f500dad2af892084129ca2b9687a1a04db95e3cb

#1125
78cdc6206d59b161f5ae3fda42719d3a03f9b699

#1131
a5cd6e33086e7e1311486ea180218d146e85d7b2

#1134
f77c971fa6f4fd93f8ca649e1cf6ddf05601f7da

#1126
07fc159dd2d126e2b60b6dd17b5936f1251704a2

#1137
7fb43e70f8c59e87c906ddcad2fb5db8f52d9f72

#1138
145705333871aa058d9adbc113a36aab020e01e4

#1139
b4a47b9dcdd7f755f2a94683044c2bfa4ca70568

#1140
7964a5ce8bdfeecbe4fac9f2ed1fc44079f406b6

#1141
81aa62b245b63157a0f887d594d5d0d4965f393a
Partial Pick

#1142
094a27351017f7f5ba97de74c87fa30da76c53bb

#1144
d02dac7b7d272822f03e523354e7042a9b21c05a

#1145
abf81f373cf6bdd6f20cf60d4b756ceb7c05e64f
Partial Pick

#1147
9c0df52a050cd79edf2bf643d78f1a91149b5ccd

Note:
Partial Pick means this pr contains other code besides yurthub component.  The code of yurthub part was picked, and other code were not picked in this PR.
Does this PR introduce a user-facing change?

other Note

/hold
Waiting for CI pass

I think it's better to use merge instead of squash and merge. squash and merge will make us cannot distinguish code change among each commit, which may make the code hard to maintain.

/unhold

Fixes #779


Ruby: Add heredoc literals

This is a first pass at adding heredoc string literals for Ruby:
https://ruby-doc.org/core-2.7.0/doc/syntax/literals_rdoc.html#label-Here+Documents+-28heredocs-29
This only adds the basic form:
<<STRING
foo #{42} bar
STRING

There are plenty of other styles with slightly different forms not yet covered by this PR:
<<'STRING'
foo #{42} bar
STRING

<<`STRING`
test 0 -eq #{0}
STRING

def baz
  <<-STRING
  foo #{42} bar
  STRING
end

def baz
  <<~STRING
    foo #{42} bar
  STRING
end

Thanks @RunDevelopment, I've removed the test suite change and expanded the heredoc support to more forms. It's not perfect, but it works well enough.
I wasn't able to run npm ci && npm run build for you though, sorry:
$ npm ci && npm run build

added 932 packages, and audited 933 packages in 7s

12 low severity vulnerabilities

To address issues that do not require attention, run:
  npm audit fix

To address all issues possible (including breaking changes), run:
  npm audit fix --force

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.

><EMAIL_ADDRESS>build
> gulp

[15:09:01] Using gulpfile ~/Developer/prism/gulpfile.js
[15:09:01] Starting 'default'...
[15:09:01] Starting 'minifyComponents'...
[15:09:01] Starting 'componentsJsonToJs'...
[15:09:01] Starting 'build'...
[15:09:01] Starting 'languagePlugins'...

#
# Fatal error in , line 0
# Check failed: allocator->SetPermissions(reinterpret_cast<void*>(region.begin()), region.size(), PageAllocator::kNoAccess).

Yes but don't worry. According to nodejs/node#37061, this issue has been fixed in v15.9.0. You just need to update your NodeJS version.

I have no idea why the tests fail. Everything looks fine to me. I'll investigate this tomorrow.

Thanks @RunDevelopment, I've folded those changes in :+1:

Please rebuild @sj26 and then we can merge this.

Sorry I forgot to rebuild, but done now :+1:

Thank you for contributing @sj26!


Code will always warn that you are offline

https://hesperia.gsfc.nasa.gov/ssw/iris/response is now a 404.
NASA have removed the public facing SSW page due to "hacking" after an attack took the site offline. I will have to find an alternate source for these files or attempt to update them manually.

Small update, it's now a 502. Still looking for alternate source, this is not a top priority right now, however.

Hesperia is back online, Turns out I was not the only one that relied on it being live.


Eslint React Hook report unclear error: React Hook "useABC" is called in function "xYZ" which is neither a React function component or a custom React Hook function

Is this a bug report?
Yes

Did you try recovering your dependencies?

Yes
Which terms did you search for in User Guide?

React Hook is called in function which is neither a React function component or a custom React Hook function
Environment

System:
  OS: macOS 10.14.2
  CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
Binaries:
  Node: 10.14.2 - /usr/local/bin/node
  Yarn: 1.12.3 - /usr/local/bin/yarn
  npm: 6.4.1 - /usr/local/bin/npm
Browsers:
  Chrome: 75.0.3770.100
  Firefox: Not Found
  Safari: 12.0.2
npmPackages:
  react: ^16.8.6 => 16.8.6
  react-dom: ^16.8.6 => 16.8.6
  react-scripts: ^3.0.1 => 3.0.1
npmGlobalPackages:
  create-react-app: Not Found

Steps to Reproduce

Create a function component and name it with first character is lowercase, e.g: iCoolDashboard
Use hook inside that component

  import React, { useState } from 'react';

  const iCoolDashboard = () => {
    const [message, setMessage] = useState('Hello');

    return (
      <div>
        {message}
      </div>
    );
  };

  export default iCoolDashboard;

Runs the app in development mode: yarn start (or npm start)
See bug report in terminal or in browser

Expected Behavior

Bug message should be clearer, something like:
iCoolDashboard is not capitalized. Component name must be start with Capital character.
Consider to show warning message instead of error and stop the compile process.

Actual Behavior

Failed to compile.
Error message is not clear. It makes developer confused and don't know how to resolve.

React Hook "useState" is called in function "iCoolDashboard" which is neither a React function component or a custom React Hook function  react-hooks/rules-of-hooks

Hmm, I think that it's a tough situation because the error message is mostly correct; it's not valid to use hooks in non-component functions. How is the linter supposed to understand that you intended this function to be a component?
I think the current error message gives you just enough info to figure it out. Plus, writing <iCoolDashboard /> won't compile either, though I guess it really depends on the name you import with elsewhere.
Also, I think this really belongs in facebook/react since that's where eslint-plugin-react-hooks lives.

This is indeed an error from the eslint-plugin react-hooks which is a package of facebook/react. So I would suggest to open an issue here if you still feel this error message can be improved. Personally I agree with @heyimalex tho. :)


[PRM-3130] - Added 2 LSP obligation pages + yes routing

AT's: https://github.com/hmrc/penalties-acceptance-tests/pull/292
PT's: https://github.com/hmrc/penalties-performance-tests/pull/108

Prod PR: https://github.com/hmrc/app-config-production/pull/20513


This is not working in Apple tv .

Bug
Getting this error in Apple tv simulator
TypeError: null is not an object (evaluating '_$$_REQUIRE(_dependencyMap[4], "./module").RNImageColors.getColors')
## Environment info
Apple tv-     Not working
iOS mobile-  Working
Android mobile-  Working
Android mobile- Working
Web - Not working
Library
"react-native-image-colors": "^1.5.1",
"react": "17.0.2",
"react-native"<EMAIL_ADDRESS>Reproducible sample code
//get primary color of the given image url
export const imgBGPrimaryColor=async(url)=>{
try {
console.log('--url------',url)
const result = await ImageColors.getColors(url, {
fallback: '#4d5152',
cache: false,
key: url,
})
return result;
} catch (e) {
throw e;
}
}
Please provide the support, Please help i am stuck here

same here

hey, this package does not support Apple TV. If you check the README, you will find that compatible platforms are Android, iOS and Web.

I imagine it cant be that hard to add support for it as its likely very identical to the iOS implementation.
If you would like to submit a PR for that I wouldn't mind reviewing it but I have no experience with Apple TV development.

actually i am getting null from native module in case of apple tv and on other platform working properly

sorry i'm not sure I understand clearly. if you are getting null errors on Apple TV, it is expected as Apple TV is not supported.

can you help me for adding support for apple tv

where we can add support for apple tv in this library, can you suggest ?

Apologies, I have no experience in React Native for Apple TV and can't help you technically.
If you figure it out, I don't mind reviewing and merging a PR to this package and making you the maintainer for it 👍


Unnecessary check for Location in Composition commit body

Steps to reproduce
robot -t "Check Headers with Commit Composition" robot/COMPOSITION_TESTS/HEADERS_CHECKS/"check_headers_xforwarded".robot
Actual result
Dictionary does not contain key 'meta'.
Expected result (Acceptance Criteria)

In test Check Headers with Commit Composition there is a check check that composition body location response has    https   example.com    333 that check for Location in meta attribute.
Attribute meta in body is not necessary in the specification: https://specifications.openehr.org/releases/ITS-REST/latest/ehr.html#composition-composition-post so I think the above check should be modified.
Definition of Done

[ ] The defect is checked by an unit or an integration test (Robot)
[ ] Merge Request approved
[ ] Unit tests passed
[ ] Build without errors
[ ] Release notes prepared
[ ] No additional runtime warnings

@vladislavploaia
I'm ok with checking Location in headers, but check that composition body location response has https example.com 333 checks in body, which is not mandatory (because attribute meta is not specified in the specification: https://specifications.openehr.org/releases/ITS-REST/latest/ehr.html#composition-composition-post).

Indeed @gregorberger. It is not mandatory. However, the xforwarded data needed to be validated as well, inside response of committed composition.
As I already mentioned, it was needed in context of https://github.com/ehrbase/ehrbase/issues/766.
For further details, you can contact @stefanspiska, so he can explain you details (he is on vacation for couple of weeks) or check it with @HolgerReiseVSys.
Thanks!


no right result

i have done it with inference_video.py.
cmd is:
python inference_video.py
--model-type mattingrefine
--model-backbone resnet50
--model-backbone-scale 0.25
--model-refine-mode sampling
--model-refine-sample-pixels 80000
--model-checkpoint ./pytorch_resnet50.pth
--video-src ./test/test.mp4
--video-bgr ./test/bg.png
--video-resize 1920 1080
--output-dir output
--output-type com
i use the er1.mp4 you given and a new bg image.
i use the default seettings --> com = fgr * pha + bgr_green * (1 - pha)
and myself --> com = pha * fgr + (1 - pha) * bgr
And i get the results.
But the two com.mp4 are the same as the input src mp4---video-src ./test/test.mp4
it looks like no different in com.mp4.
the bg  has no changed.
what happened ?

Are you using the provided inference_video.py script without modification? The provided background --video-bgr must be the actual background of the video, not a new background. Our method requires an input video and a pre-captured background of that video without the subject to generate high-quality matting. If you are trying er1.mp4, make sure you provide er1.png as background, which can be found in google drive.

i see it in google drive. i run it in google colab.it is right we need the same bg in mp4 and image.
then i have a question:
how we need to do if  the bg is  dynamic ?

What is the reason you need the background to be dynamic?
It is rare that you need the background to be dynamic. Most of the use cases are that if you have a handheld video, you will want a dynamic background where each frame is aligned with the video frame. We actually provide this functionally in inference_videos.py, you just need to add --preprocess-alignment
If you really want the background to be a video, the model can totally do it, as we process the video frame by frame and you just need to provide the model the corresponding frame of the background. The inference script provided does not have this functionally, you will need to code it up yourself.


Flickering issue on sway 1.6

Hello,
after upgrading sway to 1.6, wlroots to 0.13 and libglvnd to the upstream version (libglvnd-git) I have issue with my external displays. One of them flickers, the other stays black. Laptop's display works fine.
https://streamable.com/gvu25y
I'm on Arch Linux (5.11.15-arch1-2). I don't use any GPU drivers (I only have nvidia-utils since some other package requires it but I guess that does not count as a driver). The current setup worked fine on sway 1.5.1, wlroots 0.12.
This may be connected to #2869.

Your bug report is missing debug logs. Please add a link to the full debug log file.


Improvements: Fix issue 62

@whitlockjc
Bingo!

additionalProperties and properties are containers that themselves do not have a type property.

After thinking about your solution in #64 I tried to fix the issue too.
Most cases relies on which properties can produce a subschema: additionalItems, items, additionalProperties, dependencies, patternProperties and properties.
I'm not sure if this is enough but It seems to cover your tests without issues.
Also coverage was recovered and increased a bit.
Please review the changes I've made, check your tests and add equal assertions as you need.

Looks good.  It's a little more thorough than what I had in #64 so I closed it in favor of this.  Same concept but I like that you broke out the sub schema properties and covered all of them.


h2_h2.c:566:5: error: implicit declaration of function ‘ap_hook_pre_close_connection’ [-Werror=implicit-function-declaration]

/mod_http2-1.8.3# make
Making all in mod_http2
make[1]: Entering directory '/home/ubuntu/mod_http2-1.8.3/mod_http2'
CC       mod_http2_la-h2_conn.lo
CC       mod_http2_la-h2_filter.lo
CC       mod_http2_la-h2_h2.lo
h2_h2.c: In function ‘h2_h2_register_hooks’:
h2_h2.c:566:5: error: implicit declaration of function ‘ap_hook_pre_close_connection’ [-Werror=implicit-function-declaration]
ap_hook_pre_close_connection(h2_h2_pre_close_conn, NULL, mod_ssl, APR_HOOK_LAST);
^
cc1: all warnings being treated as errors
Makefile:705: recipe for target 'mod_http2_la-h2_h2.lo' failed
make[1]: *** [mod_http2_la-h2_h2.lo] Error 1
make[1]: Leaving directory '/home/ubuntu/mod_http2-1.8.3/mod_http2'
Makefile:458: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1

Hmm, something is not correct with the general httpd header files you are using. Which version are you compiling against?. Please run
apxs -q HTTPD_VERSION

The function you are missing should be declared in http_connection.h.

Anyway I can help? Otherwise I close this issue. You mail also get help on the Apache httpd users mailing list.


ParcelJS doesn't parse .ejs files?

Hello,
I just installed as requested the plugin by executing the command:
npm i -S parcel-plugin-ejs

Unfortunately my index.html file is empty.
Here's my script to start ParcelJS:
cross-env NODE_ENV=development parcel ./src/index.ejs

Same here...

the generated dist/index.html return empty :(


It should be possible for integration tests to bring up a drogulus network of N nodes running on localhost

For the integration test suite to work properly and fully exercise all the capabilities of target test-mode (see #22) nodes it should be trivial to bring up a "swarm" of nodes running on localhost.
There should be some convention for allowing the integration test suite (and users) to inspect and check the logs generated by the nodes in order to confirm the expected behaviour has taken place.

 I'm already quite familiar with docker, but less so with rkt (and thanks for the pointer).
It'd be great if it were platform agnostic (which docker sort-of allows). Then again, spinning up a bunch of nodes via many sub-processes (via Popen for example) could also work - although I'm unsure how well this would perform.

rkt is not yet as stable as Docker as far as I know. But it's the better choice in terms of trust and security in my opinion. rkt make cryptografically sure that images are untouched before running them and they are really authored by the person on the cover of the image. Although it works a little different, it's also able to run Docker images with rkt.

It's PITA for the open source projects to get cloud abilities "without" greedy cloud providers. GCE and Amazon have no special discounts for OS projects. On the other hand, running a dedicated server is really hard and not feasible in long term usage. There are some app centric solutions like Heroku and Bluemix but their pricing policy is not perfectly fit when the number of vms/containers grow as project grows. Also it is impossible to manage internal aspects of your application like networking.
I wish we could have a dedicated OS community to construct sustainable open cloud, based on Openstack for OS projects. But for now best option is either dedicated servers or small cheap VM providers like Digitalocean. :8ball:

@kunthar thanks for the heads up.


How to reload Projects rules

When I change the Projects rules.
It often can't normal reload.
If I will  load rules success , have to restart pyspider.

Another user reported this issue, but I have no idea.
What version are you using?
How do you start pyspider?
What database are you using?
Could you paste the logs just before and after you change the scripts?

How to get the pyspider Version?
pyspider --help
Usage: pyspider [OPTIONS] COMMAND [ARGS]...

  A powerful spider system in python.

Options:
  -c, --config FILENAME    a json file with default values for subcommands.
                           {"webui": {"port":5001}}
  --debug                  debug mode
  --queue-maxsize INTEGER  maxsize of queue
  --taskdb TEXT            database url for taskdb, default: sqlite
  --projectdb TEXT         database url for projectdb, default: sqlite
  --resultdb TEXT          database url for resultdb, default: sqlite
  --amqp-url TEXT          amqp url for rabbitmq, default: built-in Queue
  --phantomjs-proxy TEXT   phantomjs proxy ip:port
  --data-path TEXT         data dir path
  --help                   Show this message and exit.

Commands:
  all
  bench
  fetcher
  processor
  result_worker
  scheduler
  webui

If there is no --version option, it's an old version. It's better to upgrade first to see if it's fixed.

I update the pyspider ,but bug still there
➜  ~  pip install pyspider -U
Requirement already up-to-date: pyspider in /Library/Python/2.7/site-packages
...
...
...
Requirement already up-to-date: backports.ssl-match-hostname in /Library/Python/2.7/site-packages (from tornado>=3.2->pyspider)

➜  ~  pyspider --version
pyspider, version 0.3.3

database  => mysql
Strat by
pyspider -c pyspider.json
cat pyspider.json
{
  "taskdb": "mysql+taskdb://user:pass@localhost:3306/pyspider_taskdb",
  "projectdb": "mysql+projectdb://user:pass@localhost:3306/pyspider_projectdb",
  "resultdb": "mysql+resultdb://user_pass@localhost:3306/pyspider_resultdb"
}


Consider allowing to include the markdown cells in the virtual documents

As mentioned in #18, our virtual documents do not include markdown cells. Those could be included in the future, as to allow rename action to cover references in markdown cells. It probably should be a user choice and not a default as it might be difficult to implement well in a way which  is compatible with every kernel. I imagine it as the user installing a language-specific client extension (let's say jupyterlab-lsp-python)  which provides:

sanitize_and_wrap() function to convert markdown to escaped comment or string which can be safely embedded in the source code for given language
unwrap_and_unsanitize() which goes the other way around.
Probably we can think of better names for the methods later on.

This now works in JupyterLab 4 with jupyterlab-lsp 5.0.


Time Line data

Time line command's data mast have ['now','next'], now point dos not saved.

Not sure to understand what you mean by "time line commands data"?

sample is
var basicTimeline = anime.timeline();
basicTimeline
.add({
targets: '#basicTimeline .square.el',
translateX: [0,250],
translateY: [0,100],
rotate: [0,720],
duration: 3000,
easing: 'easeOutExpo'
})
.add({
targets: '#basicTimeline .square.el',
background: ['red','red']
})
.add({
targets: '#basicTimeline .square.el',
translateX: [250,0],
translateY: [100,0],
rotate: [720,0],
duration: 2000,
easing: 'easeOutExpo'
})
.add({
targets: '#basicTimeline .square.el',
translateX: [0,0],
translateY: [0,0],
rotate: [0,0],
background: ['#31FFA6','#31FFA6'],
easing: 'easeOutExpo'
});
https://codepen.io/htakaoka/pen/VpYbem

I see, you're talking about avoiding having to use initial values on a same target in a timeline?
Take a look a this branch https://github.com/juliangarnier/anime/tree/v2.0.2 and let me know if it solves the problem for you.

Thank you for your support.
The ad block did not cooperate.
As for each data, it is separated.
When the number of the data of the property is not same
I understood that a program did not work.
thank you.
version up 2017.3.2
https://codepen.io/htakaoka/pen/VpYbem

It worked on a PC normally wonderfully in v2.0.2.
Thank you.
The problem was settled.
Because CodePen is still v2.0.1, it does not work.
code exp.
https://codepen.io/htakaoka/pen/wJMoLp


Completely Broken, no longer works.

Even after the quick fix listed here #388 this at least gave the menu options back, but nothing else works, such as uploads, even connecting. its completely useless at this time.

open context.cson (.atom/packages/remote-sync/menus) and change the following lines:
line 2: from 'atom-text-editor, .tree-view.full-menu .file':[ to 'atom-text-editor, .tool-panel.tree-view .file':[
line 13: from '.tree-view.full-menu .header.list-item':[ to '.tool-panel.tree-view .header.list-item':[
Restart Atom, done.

Seems to be a dead abandoned project with nobody stepping up to fix it.
There are a few of these types of plugins available but none of them seem to work.  Has anyone jumped ship from this plugin and found one that replaces this?

@nickskitch Yeah same with me. There are a few good ones, esp with file tree access, and none are working anymore. Looks like the death of FTP is on the way....

💀

I've been kicking around the idea of forking this project and updating it up for a while now.  As I said in another thread, this plugin is much too important to my workflow for it to go away... in fact, without this plugin I would most likely have to switch to another editor all together.
Up until this point I've been patching my copy locally to keep things going.
It will be early next week before I can fork it and get things going.  All of the bug fixes will need to be applied, etc.  Just have to figure out what to call the plugin, I wish the owner of this repo would just allow me to take it over, but he seems to be MIA.

Ok - I have forked the project and published a new plugin.  It is called "remote-sync-pro".  To install you should uninstall remote-sync first then search for the remote-sync-pro package and install.  I've added several fixes for the outstanding issues, with more changes to come.  Right now I need people to test it.
There is a known possible issue that I'm seeing randomly, and it has to do with a 3rd party extension (dependency).  I'll keep looking for a fix but it may not be fully fixed until they fix it up stream.  There is a workaround however.  If you install and nothing seems to work, look in the status bar at the bottom of Atom for a red "bug" icon.  Click this icon and it will give you a message about native modules being compiled for an older version of Atom.  At the upper right will be a "Rebuild Packages" button.  Click this button, it will rebuild, then everything should work fine.
If you can test out the new package and file any issues there (not here) - https://github.com/laymance/remote-sync-pro

@laymance I wrote an email to the current owner about adding someone else or transferring this project.  He responded today asking for a suggestion.  Would you be interested?  I know you just forked the project.  Thoughts?

@creategui I would be ok with that.  There are 100k+ users on remote-sync, it would make more sense if we could take it over other than start from scratch.

@laymance I gave him your name a couple days ago.  Has he contacted you?  I emailed him at the address in his github profile.

@creategui No he hasn't contacted me yet.  I'll reach out to him later this evening.

New release https://github.com/yongkangchen/remote-sync/commit/3432d5287fa035dea822e7a7599bab62454cdf69 tested without issue.


#641: Display notification message on successful registration.

After successful registration, user should be presented with a notification message mentioning that an email with activation link has been sent.
This PR let user see the message "Email with an activation link has been sent", when the user is redirected from registration page to login page.

tip: in your comments use text like "closes #641" and after merge the issue will be closed automatically :)

Closes #626 and #641


[sbt0.11.2] dependsOn(x) libraries not included in sbt-idea export where 'x' is multi-module project

Hi Mikko,
As discussed in issue 62, here is an early-stage project of mine which can be used as a reproducible test case:
https://github.com/opyate/grafen (the sbtidea branch!)
It depends on https://github.com/opyate/scala-graph (scala-graph is a multi-module project)
Here is a first run, and you can see the dependent cloning into a staging environment, but it does not get created as a library in the IDEA project:
$ ./sbt
[info] Loading global plugins from /Users/juanuys/.sbt/plugins/project
[info] Loading global plugins from /Users/juanuys/.sbt/plugins
[info] Loading project definition from /Users/juanuys/Documents/src/github/graph-projects/grafen/project
[info] Compiling 1 Scala source to /Users/juanuys/Documents/src/github/graph-projects/grafen/project/target/scala-2.9.1/sbt-0.11.2/classes...

I generate the IDEA project:
> gen-idea no-classifiers
[info] Trying to create an Idea module grafen
[info] Excluding folder target
[info] Created /Users/juanuys/Documents/src/github/graph-projects/grafen/.idea/IdeaProject.iml
[info] Created /Users/juanuys/Documents/src/github/graph-projects/grafen/.idea
[info] Excluding folder /Users/juanuys/Documents/src/github/graph-projects/grafen/target
[info] Created /Users/juanuys/Documents/src/github/graph-projects/grafen/.idea_modules/grafen.iml
[info] Created /Users/juanuys/Documents/src/github/graph-projects/grafen/.idea_modules/project.iml

Also, when opening the project, and trying ⌘N to open, say, Constrained, it does not show one for scalax.collection. I.e. the scala-graph project is clearly not being created.
Thanks,
Juan

You need to add also 'aggregate(scalaGraph)' to get gen-idea run for it too. Anyway, it seems to run only on root project, and only Graph-all.iml -module gets created.

Reproducible case now generates the following projectList, which correctly includes the root grafen, the dependency Graph-all, which in turn has Graph-core and Graph-json modules:
List(
  (ProjectRef(file:/Users/juanuys/Documents/src/github/graph-projects/grafen/,grafen),
  Project(
    id: grafen,
    base: /Users/juanuys/Documents/src/github/graph-projects/grafen,
    aggregate: List(ProjectRef(git://github.com/opyate/scala-graph.git,Graph-all)),
    dependencies: List(
      ResolvedClasspathDependency(ProjectRef(git://github.com/opyate/scala-graph.git,Graph-all),None)),
    delegates: List(),
    configurations: List(compile, runtime, test, provided, optional))),

  (ProjectRef(git://github.com/opyate/scala-graph.git,Graph-all),
  Project(
    id: Graph-all,
    base: /Users/juanuys/.sbt/staging/9c3cb5d06e75e0970bcb,
    aggregate: List(
      ProjectRef(git://github.com/opyate/scala-graph.git,Graph-core),
      ProjectRef(git://github.com/opyate/scala-graph.git,Graph-json)),
    dependencies: List(),
    delegates: List(),
    configurations: List(compile, runtime, test, provided, optional))),

  (ProjectRef(git://github.com/opyate/scala-graph.git,Graph-core),
  Project(
    id: Graph-core,
    base: /Users/juanuys/.sbt/staging/9c3cb5d06e75e0970bcb/core,
    aggregate: List(),
    dependencies: List(),
    delegates: List(),
    configurations: List(compile, runtime, test, provided, optional))),

  (ProjectRef(git://github.com/opyate/scala-graph.git,Graph-json),
  Project(
    id: Graph-json,
    base: /Users/juanuys/.sbt/staging/9c3cb5d06e75e0970bcb/json,
    aggregate: List(),
    dependencies: List(ResolvedClasspathDependency(ProjectRef(git://github.com/opyate/scala-graph.git,Graph-core),None)),
    delegates: List(),
    configurations: List(compile, runtime, test, provided, optional))))

Hi Mikko,
fix #126, now from my branch opyate:issue_126_idiomatic
Same fixes as before:

Recursively add modules of multi-module dependency
Add doc for multi-module project

...but with a more idiomatic approach.
Also, the method have been renamed from getTuplesFromProject to getProjectList, which makes more sense.


Should gzip files be ignored using staticFileGlobsIgnorePatterns property?

[ ] I'm submitting a bug report
[ ] I'm submitting a feature request
[X] I'm submitting a support request

webpack version:
2.6.1
sw-precache-webpack-plugin version:
0.9.1
Please tell us about your environment:
OSX 10.12.5
Browser: [Chrome]
Current behavior:
The service-worker.js file is generated containing among other things the following files in the cache:
    ['/static/css/app.087c2ab96ae39bbe9b78f2c3bb840328.css', '8e020d7c181fa7c866b72fa1db89a60a'],     ['/static/css/app.087c2ab96ae39bbe9b78f2c3bb840328.css.gz', '28e79db6375ef4d1bd36b62e092349fd'],
Expected/desired behavior:
I would like to know if that is correct or if I should just ignore all the .gz files using the staticFileGlobsIgnorePatterns property.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem along with your:

Webpack configuration:

Generated service worker (not minified):

I don't see why you would want to ignore the gzip files. Why do you want to ignore them?

@goldhand we want to ignore gzip files as they are passed from the server when a request is made for a .js file.
Passing [ /\.gz$/ ] to the staticFileGlobsIgnorePatterns property is not preventing files ending in .gz from being added to the service worker.


Readme files organization

Plugin Name
All
🔖 Feature description
Hey,
Currently when you dive into a workspace, let's say : https://github.com/backstage/community-plugins/tree/main/workspaces/sonarqube you land on a not-so-pretty Readme, as the readmes has been moved into a subfolder.
Wondering if we could move some readmes, and huh, i'd be interested in trying to help 😄
🎤 Context
n/a
✌ Possible Implementation
No response
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

As the author of the Azure DevOps and Linguist plugins I'd prefer to keep the structure and create a nice README at this level as it would allow be to create a nice end-to-end flow for installation as they both have frontend and backend plugins. This has been a constant installation problem and this give us the chance to improve it. 🚀

This repo is pretty new and I suspect plugin Owners are busy looking things over and thinking about the best way forward, I know for sure I am.

@awanlin So then the individual plugin readmes could just say "See the main readme" linking to that?
Only caveat is that that will be the contents that people see on npm for the package. Slightly awkward, but not too bad.

Honestly haven't had the chance to think it through fully. My think is that "most" come form the Plugin Directory so from there we could like to the Workspace README easily. For the separate package README they could stay the same but link to the workspace README and then over time perhaps have them focused on more specific concerns of the plugin itself.

I think that is pretty valid
Honestly wanted to discuss and check a bit before opening a random PR 😅
So eventually plugin owners will manage at plugin level?

Yeah, that's why I posted, wanted to share my thoughts 👍


Mention brute force approach in docs

The collection.similar() method currently uses brute-force slow Python code. The docs should mention this.

I can link to this issue for future plans:

#216


Frequent seeking causes "failed to authenticate decrypted block - bad password?" on encrypted ACD

What is your rclone version (eg output from rclone -V)
rclone v1.33-83-g8710741β

Plex scanning on an encrypted Amazon Cloud Drive mount throws ReadFileHandle.Read io.Seeker failed: failed to authenticate decrypted block - bad password? errors.
The file is always different, and reading them from the beginning to the end is working without any problem.
Here is an example with only the relevant file:
file1: Dir.Lookup
file1: Dir.Lookup OK
file1: File.Attr
file1: File.Open
file1: ReadFileHandle.Read size 4096 offset 25944064
file1: ReadFileHandle.seek from 0 to 25944064 (io.Seeker)
file1: ReadFileHandle.Read OK
file1: ReadFileHandle.Read size 16384 offset 0
file1: ReadFileHandle.seek from 25945952 to 0 (io.Seeker)
file1: ReadFileHandle.Read OK
file1: ReadFileHandle.Read size 4096 offset 45056
file1: ReadFileHandle.seek from 16384 to 45056 (io.Seeker)
file1: ReadFileHandle.Read OK
file1: ReadFileHandle.Read size 4096 offset 77824
file1: ReadFileHandle.seek from 49152 to 77824 (io.Seeker)
file1: ReadFileHandle.Read io.Seeker failed: failed to authenticate decrypted block - bad password?
file1: ReadFileHandle.Read size 4096 offset 77824
file1: ReadFileHandle.seek from 49152 to 77824 (io.Seeker)
file1: ReadFileHandle.Read io.Seeker failed: failed to authenticate decrypted block - bad password?
file1: ReadFileHandle.Read size 4096 offset 77824
file1: ReadFileHandle.seek from 49152 to 77824 (io.Seeker)
file1: ReadFileHandle.Read io.Seeker failed: failed to authenticate decrypted block - bad password?
file1: ReadFileHandle.Read size 4096 offset 77824
file1: ReadFileHandle.seek from 49152 to 77824 (io.Seeker)
file1: ReadFileHandle.Read io.Seeker failed: failed to authenticate decrypted block - bad password?
file1: ReadFileHandle.Read size 4096 offset 77824
file1: ReadFileHandle.seek from 49152 to 77824 (io.Seeker)
file1: ReadFileHandle.Read io.Seeker failed: failed to authenticate decrypted block - bad password?
file1: ReadFileHandle.Flush
file1: ReadFileHandle.Flush OK
file1: ReadFileHandle.Release closing
file1: ReadFileHandle.Release OK

I believe the issue is related to the frequent seeking in the file, possibly a missing lock.

As of reproducing it, I'm not entirely sure.
I would upload a moderate sized file (100 megabytes?) to an encrypted amazon cloud folder and try to seek to different positions in it.

Exactly how big is file1 in your log above?
@Coornail

As of reproducing it, I'm not entirely sure.
I would upload a moderate sized file (100 megabytes?) to an encrypted amazon cloud folder and try to seek to different positions in it.

I uploaded a 2 GB file to acd+crypt and ran a seek testing program I wrote which did a random seek, read a random amount of data and compared it against a local copy of the file.  That didn't find any errors 200 times in a row :-(
Can send a complete unmodified log please?  Email it to me at<EMAIL_ADDRESS>if you don't want it to be public.
Thanks
Nick

Sent you an email with the log file.
The file in question is 25 megabytes (25945952 bytes).

Thanks for the log.
Can you send one with --debug-fuse too please?

Edit: I'm not sure, but it's possible that multiple processes opened the file to read. Could that be an issue?

It might be something like that.  How do the multiple processes open the file? Is one forked from the other?

I'll email you a logfile with --debug-fuse.
They are forked from the same process. This is how pstree looks like:
start.sh---start_pms---Plex Media Serv-+-Plex Media Scan---3*[{Plex Media Scan}]
                                       |-Plex Script Hos---12*[{Plex Script Hos}]
                                       |-Plex Script Hos---17*[{Plex Script Hos}]
                                       |-6*[Plex Script Hos---9*[{Plex Script Hos}]]
                                       |-Plex Script Hos---10*[{Plex Script Hos}]
                                       `-26*[{Plex Media Serv}]

Sadly Plex is closed source, so the things I can find out are limited.
My setup is running in a docker container (which is an another things to consider).

@Coornail thanks for the log - I have spent some time studying that but haven't come to a conclusion yet.
Can you send me a log with -v --debug-fuse --dump-headers in please?  This will be quite big and have Authorization: headers in you should strip out before sending eg grep -v Authorization: <dirty-log >clean-log.
I've also posted a version of rclone with the race detector built in.  rclone runs a bit slower like that, but it will produce a traceback if it detects any races which would indicate more missing locking.  If you can run that with -v --debug-fuse and send me the log I would be grateful (the race detector will write to stderr I think so make sure you capture that).
http://pub.rclone.org/rclone-v1.33-85-g6846a1c-linux-amd64-race.gz
Thanks for your help and hopefully we'll get to the bottom of this shortly!

Thanks @ncw!
I'll get to it in a few hours and email you the log.

Thanks for the log.  It looks like there are no races which is good :-)
What appears to be happening is every now and again rclone does a Range: request, but acd replies with a normal 200 OK instead of a 206 Partial Content
2016/10/28 20:51:04 >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
2016/10/28 20:51:04 HTTP REQUEST
2016/10/28 20:51:04 GET /cdproxy/nodes/XXXX/content HTTP/1.1
Host: content-na.drive.amazonaws.com
User-Agent: rclone/v1.33-DEV
Range: bytes=786165168-

2016/10/28 20:51:04 >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>

and the response was
2016/10/28 20:51:06 <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
2016/10/28 20:51:06 HTTP RESPONSE
2016/10/28 20:51:06 HTTP/1.1 200 OK
Content-Length: 359274160
Cache-Control: public, max-age=7200
Connection: keep-alive
Content-Disposition: attachment;   	filename=XXXXX
Content-Type: application/octet-stream
Date: Fri, 28 Oct 2016 20:51:06 GMT
Etag: "3aa5a31aeeca48405be931b18aa754dd-69"
Server: Amazon-Cloud-Drive
X-Amzn-Requestid: 550bfe99-860d-4794-b650-44b0fa256b1c

2016/10/28 20:51:06 <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

So reading from the start of the file not seeked like rclone asked for.
I've put in a work-around for this in this beta.
http://pub.rclone.org/v1.33-92-g881740e-acd-range-fixβ/
Ideally what I'd like to see is Error expecting HTTP response 206 but got 200 - retrying in the logs and it all works properly :-)
Have a go and see what happens - I suggest recording the full log with -v --debug-fuse --dump-headers and sending it to me.
Cross fingers!

Sounds awesome, thank you!
I'll check it out in a few hours and get back to you.

With the hints in your logs I managed to reproduce the problem with a program which did lots of simultaneous seeking.  I managed to get the race detector to fire using go tip and it showed me where the problem was.  Here is the commit message.
crypt: Fix data corruption caused by seeking

The corruption was caused when the file was read to the end thus
setting io.EOF and returning the buffers to the pool.  Seek reset the
EOF and carried on using the buffers that had been returned to the
pool thus causing corruption when other goroutines claimed the buffers
simultaneously.

Fix by resetting the buffer pointers to nil when released and claiming
new ones when seek resets EOF.  Also added locking for Read and Seek
which shouldn't be run concurrently.

I've committed this to a branch and made a beta from it - can you have a go with it please?
http://pub.rclone.org/v1.33-92-g4bcea27-seek-race-fixβ/
If it works for you too I'll merge it to master.
This turned out to be a complicated problem to debug - that is the danger with highly concurrent programs like rclone, but once I could reproduce it locally I knew I'd crack it!
The ACD 200 vs 206 fix was a complete red-herring, alas caused by my mis-reading of the logs!  I made a change to help with this in the future 5c967810ed15def4f5235cef7d58f26516f8bc5d
Thank you for your help debugging this, and I really hope this fixes it for you!
PS You were spot on with your original comment: I believe the issue is related to the frequent seeking in the file, possibly a missing lock. :-)

Thank you, we are very close now!
This fix solved 90% of the errors, which is great news, but it also means that there is still a bug that's 10x harder to find.
I'll email you a log tomorrow, and I'm sure we can figure it out!

OK I've merged the above in: http://beta.rclone.org/v1.33-101-g154e91b/ (ready in 15-30 mins)
I didn't see the logs yet so look forward to seeing them :-)

Thanks for the logs.
I see the problem - looks straight forward for once!
rclone attempted to seek 20 bytes beyond the end of the file to 371155456 however the file was only 371155436 bytes long.  Amazon responsed with 200 OK and the whole file from the beginning (which is technically an error - it should have given an error here).
The decrypted file is 371064828 bytes long
FUSE asked rclone to seek to position 371068928 in the decrypted stream which is 4100 bytes beyond the end of the stream.
This is a legit thing for fuse/plex to do - rclone needs to check it isn't being asked to seek beyond the end of a file.  I'll have to lookup the exact behaviour required though.
I think that is the problem - I'll double check in the morning!

I've fixed this now in: http://beta.rclone.org/v1.33-105-gd1080d5/ (uploaded in 15-30 mins) in changeset 64b5a76bec1a49cf74355f8bc939988649424b91
Can you give this a test please and if it still isn't working re-open the ticket!
Thanks for all your testing


ng command hangs for Node v7.0.0

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

OS X Sierra 10.12.1
Homebrew 1.0.8-133-g84d1661
Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:

v1.0.0-beta.18 installed. Cannot run command, ng command hangs
Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

Type ng command. Nothing happens. Status bar in Terminal window shows: ng --version(node)
Have to Control-C to exit
The log given by the failure.
None.
Mention any other details that might be useful.

Thanks! We'll be in touch soon.

Ok, digging around, looks like it's a problem with watchman. I uninstalled that - after going back to Node 6.9.1 and ng still wasn't working, saw that watchman was trying to run. Uninstalling watchman  seems to have worked.

@jtsom thx, had the same issue 👍

Strange thing is, on another system, having Watchman installed is not causing any problems.
I can run Node 7 just fine with angular-cli without watchman on this system. Add watchman, and it hangs. (For what it's worth, the Ember cli command does the same - not surprising as they're similar)

More testing. I tried installing watchman again, and ng was still hanging.
This time I uninstalled watchman (brew uninstall watchman) and also removed the launchctl item (launchctl remove com.github.facebook.watchman) to make sure it was gone.
ng worked fine. So I reinstalled watchman and started up an angular 2 app - and it seems to be ok now. The launchctl item reinstalled itself - I can only assume something got corrupted?
YMMV


Totally unclear -Quality values

It's totally unclear what would be the result of particular -Quality values.

daily - clear, a raw develop daily snapshot,
signed - something signed, but what?,
validated - another something, though, validated,
preview - quite clear,
GA - huh?

The default value information is also missing.
In the user point of view it would like to choose a develop snapshot, preview/RC, release, LTS release from particular version channel like 2.0, 2.1, 3.1, 6.0.

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

ID: a2e484a9-5e0c-eb3e-1413-7b507c4fa07d
Version Independent ID: 5bc7caa5-f1c2-d7d5-fe02-80c65cca9672
Content: dotnet-install scripts - .NET CLI
Content Source: docs/core/tools/dotnet-install-script.md
Product: dotnet-fundamentals
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

cc @bekir-ozturk

@YuliiaKovalova @mmitche

ping @YuliiaKovalova @mmitche

@MichalPavlik , it's on you.
@tdykstra , sorry for the late response -the previous mention just slipped away from me.

ping @MichalPavlik


Unpredictable Failure to Locate definitions Section at Runtime

We are intermittently receiving the following error:
JSON::Schema::SchemaError:#012 The fragment '/definitions/<definition_name>' does not exist on schema file:<path/to/schema/file>.json#
The schema file has the definitions section in question, and it is being referenced appropriately ({ "$ref": "#/definitions/<definition_name>" }). The schema file contains a few external references, but I don't see how that would make a difference. I am aware of #229.
I have spent some time debugging and have not been able to discover the cause, nor am I able to reproduce the issue on my local machine -- this only happens in production. When I debugged it, I saw the definition was present in memory at the time of loading the schema. My best guess is some kind of memory issue or a race condition in the JSON Schema library itself which only happens at scale -- we currently see this a few times an hour, with no discernible pattern in timing, no apparent clustering of the issue, or anything else that would give me a clear picture of a possible cause.
Notes:

we use OJ as our JSON parser
the schemas do not have public URLs, so we use file: for access

I am not comfortable posting the actual schema file here because it contains some proprietary information, but here's a pared-down example:
lib/schemas/client1_schema.json (modified to mask proprietary information):
{
  "id": "file:lib/schemas/client1_schema.json#",
  "$schema": "http://json-schema.org/draft-04/schema#",
  "title": "Client 1 API - Validation",
  "description": "A request object for validation",
  "definitions": {
    "range_block": {
      "type": "object",
      "properties": {
        "average": {
          "type": "number",
          "minimum": 0
        },
        "top_range": {
          "type": "number",
          "minimum": 0
        },
        "bottom_range": {
          "type": "number",
          "minimum": 0
        }
      }
    }
  },
  "type": "object",
  "required": [
    "identifier",
    "ranges"
  ],
  "properties": {
    "identifier": {
      "$ref": "common/identifier.json"
    },
    "ranges": {
      "type": "object",
        "range1": {
          "$ref": "#/definitions/range_block"
        },
        "range2": {
          "$ref": "#/definitions/range_block"
        }
      }
    }
  }
}

lib/schemas/common/identifier.json (modified to mask proprietary information):
{
  "id": "file:lib/schemas/common/identifier.json#",
  "$schema": "http://json-schema.org/draft-04/schema#",
  "title": "Identifier",
  "description": "An identifier",
  "type": "string",
  "minLength": 1,
  "maxLength": 64
}

We are calling it like so (modified to mask proprietary information):
class Request
  # left out initialize and other irrelevant information

  def invalid_fields # the entry point
    return @invalid_fields if @invalid_fields
    return [] unless request.post?
    return [] unless json_schema

    @raw_request_json = URI.decode_www_form(request.query_string).to_h.with_indifferent_access
    @invalid_fields = JSON::Validator.fully_validate(json_schema, raw_request_json)
  end

  def json_schema
    return @json_schema if @json_schema
    return {} unless File.exist?(schema_filename)

    raw_schema_string = File.read(schema_filename)
    @json_schema = JSON.parse(raw_schema_string)

    @json_schema
  end

  def schema_filename
    @schema_filename ||= "lib/schemas/client1_schema.json"
  end

having the same issue


Let consumers of the gem handle pagination

Currently the gem paginates for you, often leading to the rate limit being hit and difficult-to-handle exceptions being thrown. While I think it's good to introduce an exception for when the rate limit is hit, I also think that pagination should be left up to the consumer of the gem (as mentioned here).
That way, we don't end up with being half-way through iterating through a collection and then getting the RateLimitError thrown. Plus, then it's a more faithful mirroring of the actual RESTful API.
Thoughts?

sure, this can become anoying quickly.
i think a simple solution could be allow an optional block on the method that yields the chunks.
by asking the client about the current rate-limit additional measures could be taken, i guess that only sleeping is likely to help?

Yeah, yielding the block does indeed seem like a good solution.
Yes, indeed what I do now - I wrap calls to the gem in a method that sleeps
when the rate limit is  low.
Op ma 9 dec. 2019 om 08:56 schreef Peter Schröder<EMAIL_ADDRESS>
sure, this can become anoying quickly.
i think a simple solution could be allow an optional block on the method
that yields the chunks.
by asking the client about the current rate-limit
https://github.com/penseo/webflow-ruby/blob/master/lib/webflow/client.rb#L131-L134
additional measures could be taken, i guess that only sleeping is likely to
help?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/penseo/webflow-ruby/issues/9?email_source=notifications&email_token=AAEICGQ72KTANQKNJOT6BDTQXX2ZRA5CNFSM4JYEB5XKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGIGHVI#issuecomment-563110869,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEICGWPYN5JNS4X3THZ7ZDQXX2ZRANCNFSM4JYEB5XA
.

closed by 40b981efbb24691f498e0957b935724738194f8e
@emilesilvis if you like, you could try version 1.1.0 that has just been pushed

Awesome, thanks @phoet ❤


Error on reconnection with resubscribeAllChannels

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[X] I am running the latest version
[X] I checked the documentation and found no answer
[X] I checked to make sure that this issue has not already been filed
[X] I'm reporting the issue to the correct repository

Current Behavior
When I'm running moleculer-channels and I got an error, timeout o whatever (an error how cause a reconnection to amqp) when moleculer-channels try to reconnects calls method resubscribeAllChannels. This method always return an error Already tracking active messages of channel XXXX which is located in the method initChannelActiveMessages.
Expected Behavior
No errors and reconnectins as usual.
Temporal solution
I put a try/catch inside resubscribeAllChannels
async resubscribeAllChannels() {
this.logger.info("Resubscribing to all channels...");
for (const { chan } of Array.from(this.subscriptions.values())) {
try {
await this.subscribe(chan);
} catch (e) {
this.logger.error('[resubscribeAllChannels]', e)
}
}
}
Question
Is necessary call the method resubscribeAllChannels, because I make some tests commenting this line and all works well.
Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Moleculer version: 0.14.17
NodeJS version: 12.22.6

Also I don't undestand this part. ConnectionCount is used to know if there is a reconnection, but always will be reconnect because after the channel created, you sum 1 to connection count, so isReconnect always return true.
`this.logger.info("AMQP channel created.");
this.connectionCount++;
	const isReconnect = this.connectionCount > 1;
	if (isReconnect) {
		await this.resubscribeAllChannels();
	}`

Also I don't undestand this part. ConnectionCount is used to know if there is a reconnection, but always will be reconnect because after the channel created, you sum 1 to connection count, so isReconnect always return true.
The connectionCount initialized to 0, so first time it will be false (1 > 1 == false).

Could you write repro steps and env?


Classic Username + Password auth

Hello can you add the classic email + password into precedent please

Great question – I actually made a template with exactly hat that you can refer to: https://github.com/vercel/nextjs-postgres-auth-starter
Closing this for now! :)


How to use with setUpClass()

From the documentation it is not clear how I can use PyFakeFS in a setUpClass().
setuppyfakefs() only works with a self instance from a setUp() method.
My current solution is to use a Patcher object in setUpClass() like this:
def My(unittest.TestCase):
    @classmethod
    def tearDownClass(cls):
        cls._patcher.tearDown()

    @classmethod
    def setUpClass(cls):
        cls._patcher = pyfakefs_ut.Patcher()
        cls._patcher.setUp()

Maybe you know a better way?
You could improve the "Usage" section of the docu with an situation using setUpClass().
My use-case:
I do create some testfiles in the fake filesystem. Because I only want to read them in the tests I need to create them only once for all tests. Creating them for each test again and again is ressourc inefficient.

No, there is no specific support for that. It is assumed that the setup is fast enough to happen in each test, and so far, this has not been an issue.
Your implementation is fine as is, and I'm not sure if we need more than that, given that this is the first time this use case ever came up. Supporting this specifically would mean just a thin wrapper around that implementation, and I'm not sure if it's worth it.
You are right about the documentation - we could that use case to the usage section.

Actually, on a second thought, it makes probably sense to add a class method for setup. As I recently changed the implementation to allow only one patched instance (this was made for session or module scoped fixtures in pytest), this will allow to access the fake filesystem via self.fs in the tests as usual.

I had another look at this, and while I can add a convenience function for class setup (I did this locally), it may not behave as expected. It would ehave the same way as creating the patcher in the class setup, as shown in the original comment. That would also mean that any changes to the file system in a single test would persist, so the test would influence each other.
Ideally, we would need to capture the state at test begin and reset the file system to that after test end, but I'm not sure how feasible this would be.
@buhtz, what do you think?

That would also mean that any changes to the file system in a single test would persist, so the test would influence each other. Ideally, we would need to capture the state at test begin and reset the file system to that after test end, but I'm not sure how feasible this would be. @buhtz, what do you think?

😄 I think that you think way to far. 😀 I'm aware that the each test can (theoretically) influence each other. But a developer who write unittest should know that. It is his/her responsibility and not yours. You can give a note about that in your documentation but that is IMHO enough.
In my case all unittests only reading from the file system. Thats all. They don't write to it. So I see no problem.

Thanks!

I think that you think way to far

Well, if you say so 😀
Ok, I'll probably just add that convenience class setup method (and a class scoped fixture for pytest) in this case, and make the behavior clear in the documentation.

I added the class method setUpClassPyfakefs that shall do what you want in main - it took me longer than anticipated.
Please check if this works for you!

Dear @mrbean-bremen ,
thanks a lot for investing time into this.
I can confirm that #758 does work. You can see it in real world here.


Allow users to delete synced workspaces from the API

This allows users to remove synced projects from the API, by adding a new item to the sync dropdown menu.

@nijikokun I know you had some thoughts around the wording/messaging, wasn't sure of a good way to communicate that the workspace will be deleted from the API, but not locally.
Please look it over and let me know what wording you'd prefer.
Fixes INS-236
Testing

Enable beta sync in the app
Create a workspace, set up sync, commit and push
Choose "Delete Workspace" from the sync dropdown, and run through the flow.
Delete the workspace locally as well, and verify that it no longer shows up in the workspace dropdown under "Remote projects".

Good catch! I was testing with trailing whitespace to ensure the validation was correct, but hadn't thought about the way it was being displayed.
I've corrected the CSS to not collapse whitespaces, so that's it properly displays when you have more then one whitespace somewhere in the name:

As for being able to select, I don't think there's a good reason to disallow it, so I've gone ahead and made that change as well:

Actually, thinking about this one - are you able to achieve a similar result using the PromptModal that already exists? It contains input validation.
I ask because the UI pattern here is very different to all other modals, and preference would be to keep consistent, unless this is a conscious decision cc: @erickrico

Using a different UI pattern here was a conscious decision made by @nijikokun, as this is a high-impact operation, and none of the existing patterns fitted that well.
I looked at using the PromptModal initially, but it doesn't support text above the input field other than a small label, doesn't provide proper feedback as to when or why a validation is failing, and uses our standard model footer to right-align the submit button (and provides no way to change the state/disable it), which creates a disconnect between the input field and the button that is unwanted for this pattern.
While it would've been possible to modify it to also support this pattern, adding further complexity and special-casing to the PromptModel for this use-case did not feel ideal, which is why I opted to make this a separate component instead.


Fix static power tests on 32-bit platforms

tests were failing due to expected integer overflow

[Current coverage][cc-pull] is 87.26%

Merging [#1978][cc-pull] into [master][cc-base-branch] will decrease coverage by <.01%

@@             master      #1978   diff @@
==========================================
  Files           303        303
  Lines         53993      53988     -5
  Methods           0          0
  Messages          0          0
  Branches       5576       5575     -1
==========================================
- Hits          47119      47113     -6
- Misses         5955       5956     +1
  Partials        919        919

Powered by Codecov. Last updated by [ff54d7b...9f2346c][cc-compare]
[cc-base-branch]: https://codecov.io/gh/numba/numba/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/numba/numba/compare/ff54d7b53c56752603ae676a4a84cc36970565ba...9f2346ccbe4e1f92f22d941525567c236e76517b?src=pr
[cc-pull]: https://codecov.io/gh/numba/numba/pull/1978?src=pr


gitlab is missing Repository::$namespace

I'll try to get some example xml data for easier reproduction, but it seems as if namespace/group is not used on gitlab push events. Or maybe this is a SimpSpector issue concerning old data.

we should refresh the fixtures: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/web_hooks/web_hooks.md

:+1: thx


fix(FMP-331): always show logo in navbar

Current dependencies on/for this PR:

main

PR #36

PR #37   👈

This comment was auto-generated by Graphite.


Switch random generation to use HSV over RGB

joe:

for random duckies I'm tempted to say we should go with colorsys hsv/hsl because it gets nicer colours

I'm personally not familiar at all with HSV, but if someone else wants to implement this to get nicer colour generation be my guest!
If you want an example of how this is done in a way that gets ideal duck colours, check out: https://github.com/python-discord/aaaaAAAA/blob/main/aaaaAAAA/procedural_duckies.py

colorsys is the module to use!

I quite like this implementation from the staff PyWeek submission
def make_color(hue: float, dark_variant: bool) -> tuple[float, float, float]:
    saturation = 1
    lightness = random.uniform(.7, .85)

    # green and blue do not like high lightness, so we adjust this depending on how far from blue-green we are
    # hue_fix is the square of the distance between the hue and cyan (0.5 hue)
    hue_fix = (1 - abs(hue - 0.5))**2
    # magic fudge factors
    lightness -= hue_fix * 0.15
    if dark_variant:
        lightness -= hue_fix * 0.25
    saturation -= hue_fix * 0.1

    return hue, lightness, saturation

It can then be converted in RGB with colorsys.

I will go ahead and do this.


Update to rust-ordered 0.4.0

We just released a version of ordered that makes ArbitraryOrd object safe - use it.
Upgrade to the latest version of rust-ordered - v0.4.0.

ref: https://github.com/rust-bitcoin/rust-ordered/pull/32


Nested bold and italic break rendering

Howdy! Thanks for the great repo - we've found it really useful!
We're running into a bug whenever we have italic nested inside of bold or vice versa.
Any tips for approaching it?

Github markdown seems to work when attempting this:
bold and italic
italic and bold

I think it is time for a rewrite of the project, using the lexer of "marked", instead of the renderer. That will allow for more granular control of how things are rendered :)


Change colors to match IQX White Theme

Summary
Change gate colors to match IQX Light theme.
Change Dictionaries to match font color.
Details and comments
fixes #5032

Can you write test(s) following this style https://github.com/Qiskit/qiskit-terra/pull/4544 ?

@1ucian0 https://mybinder.org/v2/gh/abhobe/qiskit-terra/5038?filepath=%2Ftest%2Fipynb%2Fmpl_tester.ipynb
Is this what you want?

How this relates with the propousal from https://github.com/Qiskit/qiskit-terra/pull/5063#issuecomment-693445283 ?

5063 solves this but does not fully match the style of IQX.  The quantum and classical gates on IQX have white font color and the others have a black font. 5063 makes all fonts white instead. @1ucian0

I disagree with this. Attached is my screenshot of what I tested.

I think these pull requests intend to do the same, looking at #5032 it seems #5063 started working on this before... maybe you can join the efforts but I think we should close this one.


Activate OJS extension on "import"

If no ojs or omd files have been loaded previously (or none are in focus) and the user tries to "import file" it will fail.
FYI @pmunin

@pmunin I think you should revise your review.
Docs clears state how to import OJS notebooks: https://github.com/GordonSmith/vscode-ojs#quick-start-with-observable-javascript--markdown
Also, VS Marketplace reviews are not for logging issues.
Gordon created great extension that makes it possible to run Observable notebooks in vscode, one of a kind.