Showing "No space left on device" even though clearly having enough space

Describe the bug
Showing "No space left on device" even though clearly having enough space.
This probably happened when i actually had no space while i was building wluma package for uploading new version.
Which is nothing major, usually i clear the /home or / directory depending upon which is necessary, in this case being /home directory..
I cleared up storage
and built and uploaded the wluma package (nothing out of unusual)
After which i wanted to
sudo nixos-rebuild switch --upgrade-all
and this error showcased
sudo nixos-rebuild switch --upgrade-all
[sudo] password for Kihsir:
error: creating directory '/nix/var/nix/temproots': No space left on device

which i again assumed being the storage of / directory being full
(In case i wasn't clear before, i have kept 2 different partitions for root and home)
So i checked the root partition and it had 4 Gb remaining
which in itself confused me
I tried various other things like rollback and doing
sudo nix-collect-garbage
Which i got a similar error but not quite the same
sudo nix-collect-garbage
[sudo] password for Kihsir:
error: creating directory '/nix/var/nix/temproots': No space left on device

I just gave up and tried clearing it out regardless
Which failed as well
sudo nix-collect-garbage
error: creating directory '/nix/var/nix/temproots': No space left on device

and tried
sudo nix-store --optimise
[sudo] password for Kihsir:
error: creating directory '/nix/var/nix/temproots': No space left on device

Which again failed

Steps To Reproduce
Steps to reproduce the behavior:

sudo nix-store --optimise
sudo nix-collect-garbage
sudo nixos-rebuild switch --upgrade-all
etc

Expected behavior
Do i really have to write this?

Screenshots
Directly terminal commands only, no GUI..

Additional context
Okay, this is important info,
I am not on a stable build as of right now
I am using nixos unstable build which is probably what caused it,
Hopefully, it would be fixed when changing to the current channel?
But still wanted to let the devs know about this issue,
assuming they want me to do some testing while I am stuck here

Metadata

nix-shell -p nix-info --run "nix-info -m"
error: creating directory '/tmp/nix-shell-35597-0': No space left on device

Lmao this was a bit unexpected, but Ig I should have expected it..
Notify maintainers

I genuinely don't know now on who to notify and talk with..

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Use df -i to check if you have run out of inodes (not bytes) on your filesystem.

df -i
Filesystem       Inodes IUsed   IFree IUse% Mounted on
devtmpfs        1965152<PHONE_NUMBER>    1% /dev
tmpfs           1967532     9 1967523    1% /dev/shm
tmpfs           1967532  2055 1965477    1% /run
/dev/nvme0n1p9        0     0       0     - /
efivarfs              0     0       0     - /sys/firmware/efi/efivars
tmpfs              1024     1    1023    1% /run/credentials/systemd-journald.service
tmpfs              1024     1    1023    1% /run/credentials/systemd-tmpfiles-setup-dev-early.service
tmpfs              1024     1    1023    1% /run/credentials/systemd-sysctl.service
tmpfs              1024     1    1023    1% /run/credentials/systemd-tmpfiles-setup-dev.service
tmpfs              1024     1    1023    1% /run/credentials/systemd-vconsole-setup.service
tmpfs           1967532    20 1967512    1% /run/wrappers
/dev/nvme0n1p4        0     0       0     - /boot
/dev/nvme0n1p10       0     0       0     - /home
/dev/nvme0n1p5        0     0       0     - /media/Arch-home
tmpfs              1024     1    1023    1% /run/credentials/systemd-tmpfiles-setup.service
tmpfs              1024     1    1023    1%<EMAIL_ADDRESS>tmpfs            393506    56  393450    1% /run/user/1000

I also ran these
sudo btrfs filesystem df /home
Data, single: total=56.01GiB, used=53.21GiB
System, DUP: total=8.00MiB, used=16.00KiB
Metadata, DUP: total=1.56GiB, used=620.66MiB
GlobalReserve, single: total=92.19MiB, used=0.00B

sudo btrfs device stats /home
[/dev/nvme0n1p10].write_io_errs    0
[/dev/nvme0n1p10].read_io_errs     0
[/dev/nvme0n1p10].flush_io_errs    0
[/dev/nvme0n1p10].corruption_errs  0
[/dev/nvme0n1p10].generation_errs  0

But no error seems to appear

OK, if it is BtrFS, then: the number of inodes available should be 0, you should not be able to run out of inodes as such ever, and also the notion of free space is more complicated than df can show.
Run btrfs filesystem df /

Hm, maybe try doing a balance? Indeed you should have some metadata space left, but BtrFS allocation is finicky ans mysterious when usage is high enough.

Umm this is again unexpected but maybe expected i suppose
sudo btrfs balance start /
[sudo] password for Kihsir:
WARNING:

	Full balance without filters requested. This operation is very
	intense and takes potentially very long. It is recommended to
	use the balance filters to narrow down the scope of balance.
	Use 'btrfs balance start --full-balance' option to skip this
	warning. The operation will start in 10 seconds.
	Use Ctrl-C to stop it.
10 9 8 7 6 5 4 3 2 1
Starting balance without any filters.
ERROR: error during balancing '/': No space left on device
There may be more info in syslog - try dmesg | tail

If i remember correctly this is the command..
Am i doing it wrongly?

Oh!
However for my home partition it worked
sudo btrfs balance start /home
WARNING:

	Full balance without filters requested. This operation is very
	intense and takes potentially very long. It is recommended to
	use the balance filters to narrow down the scope of balance.
	Use 'btrfs balance start --full-balance' option to skip this
	warning. The operation will start in 10 seconds.
	Use Ctrl-C to stop it.
10 9 8 7 6 5 4 3 2 1
Starting balance without any filters.
Done, had to relocate 62 out of 62 chunks

Umm this is again unexpected but maybe expected i suppose
sudo btrfs balance start /
[sudo] password for Kihsir:
WARNING:

	Full balance without filters requested. This operation is very
	intense and takes potentially very long. It is recommended to
	use the balance filters to narrow down the scope of balance.
	Use 'btrfs balance start --full-balance' option to skip this
	warning. The operation will start in 10 seconds.
	Use Ctrl-C to stop it.
10 9 8 7 6 5 4 3 2 1
Starting balance without any filters.
ERROR: error during balancing '/': No space left on device
There may be more info in syslog - try dmesg | tail

If i remember correctly this is the command.. Am i doing it wrongly?

If incase this helps:
sudo dmesg | tail
[53392.970361] BTRFS info (device nvme0n1p10): found 7895 extents, stage: move data extents
[53393.534229] BTRFS info (device nvme0n1p10): relocating block group 22020096 flags system|dup
[53393.546039] BTRFS info (device nvme0n1p10): found 1 extents, stage: move data extents
[53393.556088] BTRFS info (device nvme0n1p10): relocating block group 13631488 flags data
[53393.588371] BTRFS info (device nvme0n1p10): found 7 extents, stage: move data extents
[53393.643833] BTRFS info (device nvme0n1p10): found 7 extents, stage: update data pointers
[53393.687454] BTRFS info (device nvme0n1p10): balance: ended with status: 0
[53480.128003] BTRFS info (device nvme0n1p9): balance: start -d -m -s
[53480.658869] BTRFS info (device nvme0n1p9): 28 enospc errors during balance
[53480.658875] BTRFS info (device nvme0n1p9): balance: ended with status: -28

Also
I do have found this Reddit post with exactly similar issue (except in his case is for /home and mine is probably /)
Here.

Full balance doesn't work on near-full filesystem, that's BtrFS…
Try something like btrfs balance start -dusage=5 -musage=5 / and maybe slowly increase the percentage (this tells to relocate records from mostly empty chunks, if any, so hopefully it manages to put them into another chunk without needing to allocate a new one)

umm.. 😅
sudo btrfs balance start -dusage=5 -musage=5 /
[sudo] password for Kihsir:
ERROR: error during balancing '/': No space left on device
There may be more info in syslog - try dmesg | tail

sudo btrfs balance start -dusage=1 -musage=1 /
ERROR: error during balancing '/': No space left on device
There may be more info in syslog - try dmesg | tail

Here is the log
sudo dmesg | tail
[70767.519748] BTRFS info (device nvme0n1p9): balance: start -dusage=1 -musage=1 -susage=1
[70767.530661] BTRFS info (device nvme0n1p9): 1 enospc errors during balance
[70767.530670] BTRFS info (device nvme0n1p9): balance: ended with status: -28

Why dmesg shows -susage=1 I wonder

A priori, you could try setting all three to 0 (this should succeed but do nothing), then try one to 1 and two to 0 and see if any of the attempts succeeds while doing anything at all

Ok as per your instruction this works
sudo btrfs balance start -dusage=0 -musage=0 /
[sudo] password for Kihsir:
Done, had to relocate 0 out of 28 chunks

sudo btrfs balance start -dusage=1 -musage=0 /
Done, had to relocate 0 out of 28 chunks

However this doesn't
sudo btrfs balance start -dusage=0 -musage=1 /
ERROR: error during balancing '/': No space left on device
There may be more info in syslog - try dmesg | tail

Here is the dmesg | tail
[72575.792459] BTRFS info (device nvme0n1p9): balance: start -dusage=0 -musage=0 -susage=0
[72575.792493] BTRFS info (device nvme0n1p9): balance: ended with status: 0
[72593.936698] BTRFS info (device nvme0n1p9): balance: start -dusage=0 -musage=1 -susage=1
[72593.945665] BTRFS info (device nvme0n1p9): 1 enospc errors during balance
[72593.945669] BTRFS info (device nvme0n1p9): balance: ended with status: -28
[72600.110044] BTRFS info (device nvme0n1p9): balance: start -dusage=1 -musage=0 -susage=0
[72600.110081] BTRFS info (device nvme0n1p9): balance: ended with status: 0

Ok, -dusage=5 -musage=0 and increase -dusage until failure or something being done, whichever comes first?

Yep, here you go!
sudo btrfs balance start -dusage=5 -musage=0 /
[sudo] password for Kihsir:
Done, had to relocate 0 out of 28 chunks

[Kihsir@NixOS:~]$ sudo btrfs balance start -dusage=10 -musage=0 /
Done, had to relocate 0 out of 28 chunks

[Kihsir@NixOS:~]$ sudo btrfs balance start -dusage=15 -musage=0 /
ERROR: error during balancing '/': No space left on device
There may be more info in syslog - try dmesg | tail

[Kihsir@NixOS:~]$ sudo btrfs balance start -dusage=14 -musage=0 /
ERROR: error during balancing '/': No space left on device
There may be more info in syslog - try dmesg | tail

[Kihsir@NixOS:~]$ sudo btrfs balance start -dusage=13 -musage=0 /
ERROR: error during balancing '/': No space left on device
There may be more info in syslog - try dmesg | tail

[Kihsir@NixOS:~]$ sudo btrfs balance start -dusage=12 -musage=0 /
Done, had to relocate 0 out of 28 chunks

Hmm, pity.
The thing people recommend for a stuck BtrFS like that, is to add an extra device to the filesystem (a 8GB USB drive lying around would be fine), clean up some stuff on the FS, then do balance, then ask BtrFS to remove the extra device.

Yeah figured that would be whats required,
But the question is
was this a bug or something i did on my part for the FS to end up this way?

I would say this is an often-observed BtrFS behaviour corner case when you have high utilisation percentage. In the narrow circles of people with a specific kind of experience it is relatively well-known. My BtrFS /nix/ is one GiB smaller than its LVM2 volume for this specific reason (to make adding space for GC easier).
Nothing reasonable that I would expect NixOS to be doing better here, so probably not a NixOS bug. Arguably a BtrFS … misfeature.
(There are reasons I use Ext4 from /home/ and BtrFS for /nix/ and not going to unify in either direction…)

Hmm..
I see now
Anyway Thanks very much for your time..
I don't think Most Member of Nix organisation would have given me this much explanation as you have 🤗

Most Member of GH/NixOS org do not indeed have the specific experience that makes the responses possible to write almost without thinking!
Yeah, I close the issue as not actionable on our side / answered as a question.


GROOVY-8243: support closure for functional interface that extends trait

https://issues.apache.org/jira/browse/GROOVY-8243

Merged, thanks!


Function node editor not supporting valid switch syntax (multi-case - chained operations)

Hi,
I tried writing a function node with valid javascript switch syntax for multi-case chained operations (as per last example at: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/switch) and the editor shows errors (red x box) with the text: Expected a 'break' statement before 'case'.  Trying to deploy the function node fails with "The workspace contains some nodes that are not properly configured..." ... "Are you sure you want to deploy?"... and, of course, the node doesn't run.
To be clear, what I'm trying to do is the same as the mozilla example:
var foo = 1;
var output = 'Output: ';
switch (foo) {
  case 0:
    output += 'So ';
  case 1:
    output += 'What ';
    output += 'Is ';
  case 2:
    output += 'Your ';
  case 3:
    output += 'Name';
  case 4:
    output += '?';
    console.log(output);
    break;
  case 5:
    output += '!';
    console.log(output);
    break;
  default:
    console.log('Please pick a number from 0 to 5!');
}

With the expected result:
      foo is NaN or not 1, 2, 3, 4, 5 or 0:    Please pick a number from 0 to 5!
                                         0:    Output: So What Is Your Name?
                                         1:    Output: What Is Your Name?
                                         2:    Output: Your Name?
                                         3:    Output: Name?
                                         4:    Output: ?
                                         5:    Output: !

I thought this has been supported for years?

Hi, please provide the extra information that was in the issue template you've deleted:
- [ ] Node-RED version:
- [ ] node.js version:
- [ ] npm version:
- [ ] Platform/OS:
- [ ] Browser:

When I paste your code into a Function node, I get yellow warning triangles (as forgetting the break statement is a common cause of bugs), but they are not marked as errors.

As the code you've shared is the moz example code, could you provide the node-red specific code you're trying to use?

My apologies!!

Node-RED version: v0.19.4
Node.js  version: v8.12.0
NPM: 6.4.1
Linux 4.14.70-v7+ arm LE (Dietpi distro on Raspberry Pi)
Browser: Google Chrome Version 69.0.3497.100 (Official Build) (64-bit) on Windows 10

Soooo... this is interesting.
I had exactly that problem with the Moz code -- and it blew my mind that you copy/pasted same and only got warnings.
Thinking I'm going nuts, I again created a function node and copy / pasted the Moz example in.
YELLOW TRIANGLES this time.  WTF???
...and then I typed in right at the end
return msg;

BANG.  RED TRIANGLES...
Whaaa???

Huh - I see what you mean.
We embed the ace editor and are completely reliant on the language support they provide. I've seen some other quirks like this around some of the newer es6 syntax - warnings that become errors if you do certain things.
We can try raising this upstream with ace - but I would not expect a quick turnaround.

That would be awesome -- thank you!!!!

Figured it out - not an upstream issue, but one for us to fix which I have done for the 0.20 release - whenever that arrives. Happily this addresses all the quirks I mentioned before.
In short, the code you type into the Function node gets wrapped in a Function itself before being run. To properly validate the javascript code, it needs to be examined in its wrapped form - not the raw text you type in. To do that, I've created a custom edit mode for the Function node to use that does the necessary work.


Regarding releasing training code

Hello Authors,
Thanks for publishing this great work!
Do you have any plan to release the training code?

Sorry for the slow progress of organizing the code. If you just want to fine-tune it, we recommend using the xtuner framework for fine-tuning. Thank you for your support and we will let you know if there's any progress.


Cannot create additional Azure Privatelink

What happened?
While trying to setup an additional Azure PrivateLink connection for a service an error was triggered during the connection approval :
Error: number of privatelink connections != 1 (2

  with module.aiven_redis_privatelink.aiven_azure_privatelink_connection_approval.approval,
  on network/aiven_privatelink/main.tf line 28, in resource "aiven_azure_privatelink_connection_approval" "approval":
  28: resource "aiven_azure_privatelink_connection_approval" "approval" {

What did you expect to happen?
I would expect the creation of the additional Azure Privatelink
What else do we need to know?
I could overcome this using the Aiven Cli with:
avn service privatelink azure connection approve
avn service privatelink azure connection update

hey, @leandrorosa! 👋
thank you for raising this issue, I'm going to check this now!

hey again @leandrorosa! 👋
I have created a PR to add this feature, it will be released soon, but in case of urgency, you are welcome to build the provider manually!

Wow, that was fast!
No worries, I will wait for the next release.
Many thanks @Serpentiel!

hey, @leandrorosa! 👋
after performing additional QA evaluation, we have noticed that Aiven public API currently does not supports multiple Privatelink connections to a single service
may we know exactly (step-by-step) how you create these?
we have tried to reproduce it with your provided description, however for us it results in the following error:
aiven_azure_privatelink.c1: Creation complete after 36s [id=aiven-ci-terraform-provider/ivans-pg2]
╷
│ Error: 409: {"errors":[{"message":"Service already has Privatelink configured","status":409}],"message":"Service already has Privatelink configured"} -
│
│   with aiven_azure_privatelink.c2,
│   on getting-started.tf line 50, in resource "aiven_azure_privatelink" "c2":
│   50: resource "aiven_azure_privatelink" "c2" {
│
╵

hey, @leandrorosa! 👋
since it's been a week, and we haven't heard back from you, I'm going to close this for now
we were unable to confirm that this is supported by Aiven API, and is supported in general, and therefore cannot merge the PR
please let us know if you have a way to reproduce it, so that the service ends up having multiple Privatelink configured


Make TimelineProcess a Step

This corrects TimelineProcess to apply updates at the time(s) indicated in the timeline. Before, specifying an update at t=0, for example, only results in an updated simulation state after a timestep has elapsed.

By creating this pull request, I agree to the Contributor License
Agreement, which is available in CLA.md at the top level of this
repository.

That's true. Leaving this as is also makes perfect sense if I just think of the timeline as a set of times when actions will occur. In that case, the state should only reflect the results of these actions at the beginning of the following time steps.
We'll just have to keep this delay in mind when using TimelineProcesses.


About Extend Tier 2

Hi, I am reading your paper recently published online in Nature Methods. You provided a wonderful job on hard-to-call SV detection with a novel framework. However, I got some confusions about the Extend Tier 2 benchmarking.

NIST_SVs_Integration_v0.6 FTP only provides a bed file for Tier 2 (HG002_SVs_Tier2_v0.6.bed). There is no Tier 2 VCF file so that Truvari is not applicable (You mentioned Truvari command in page xvi of supplementary file. Truvari requires both VCF file and included bed file).  So you used another comparison tool (maybe bedtools intersection) for Tier 2?

Could you please provide the details of Extend Tier 2? I didn't find the Extend Tier 2 file in your supplementary files and githubs. Extend Tier 2 played a key role in your benchmarking section and was much helpful for readers to run some tests or futher comparison.

Thanks

Hi, thanks!
For extended tier 2, you are right, there is no .bed in the FTP server but we built it from Tier1 .bed using bedtools: extended Tier 2 is everything outside Tier 1, so we computed it by complementing the Tier1 .bed.
You can find all the steps to reproduce our exps here: https://github.com/ldenti/svdss-experiments:

here for hg38: https://github.com/ldenti/svdss-experiments#prerequisites

wget https://ftp-trace.ncbi.nlm.nih.gov/giab/ftp/data/AshkenazimTrio/analysis/NIST_SVs_Integration_v0.6/HG002_SVs_Tier1_v0.6.bed
sed -e 's/^/chr/' HG002_SVs_Tier1_v0.6.bed > Tier1_v0.6.hg19.bed
wget http://hgdownload.soe.ucsc.edu/admin/exe/linux.x86_64/liftOver
chmod +x liftOver
wget http://hgdownload.soe.ucsc.edu/goldenPath/hg19/liftOver/hg19ToHg38.over.chain.gz
gunzip hg19ToHg38.over.chain.gz
./liftOver Tier1_v0.6.hg19.bed hg19ToHg38.over.chain Tier1_v0.6.hg38.bed Tier1_v0.6.unlifted.bed
for c in $(seq 1 22) X Y ; do grep -P "^chr${c}\t" Tier1_v0.6.hg38.bed ; done | sort -k1,1 -k2,2n > Tier1_v0.6.hg38.noalt.bed
bedtools complement -i Tier1_v0.6.hg38.noalt.bed -g hg38.chroms.genome > Tier23_v0.6.hg38.noalt.bed

here for hg19: https://github.com/ldenti/svdss-experiments#giab-vs-dipcall-analysis

wget https://ftp-trace.ncbi.nlm.nih.gov/giab/ftp/data/AshkenazimTrio/analysis/NIST_SVs_Integration_v0.6/HG002_SVs_Tier1_v0.6.bed
bedtools complement -i HG002_SVs_Tier1_v0.6.bed -g hs37d5.chroms.fa.fai > HG002_SVs_Tier23_v0.6.bed

But I just realized that this is a bit hidden in the README.. And moreover I named the Extended Tier 2 .bed file as Tier23.. I'll write a section for it to better explain everything.
Let me know if this helps you out.
Best,

I tried to make it clearer in the README of that repo (here and here).

oh, that really helps. I will do more tests. Thanks

I'm closing this issue for now, if you have any other doubt/question on this, please reopen it.


Crash when launching the main Activity with a PendingIntent when it is already running

When starting the main Activity with a PendingIntent (for example when taping on a push notification), this latter goes through the normal lifecycle (onCreate -> onStart ...).  If the app is running when the intent is received, the Navigator object still exists in the Activity and therefore can't be set using mNavigator = ActivityArchitector.onCreateNavigator(...) in onCreate method because this method calls navigator.delegate().onCreate(...) which sends an exception if the container view is not null.
As a temp fix, we modified the code in the Activity's onCreate method to:
mNavigator = Navigator.find(this);
if (mNavigator == null) {
    mNavigator = ActivityArchitector.onCreateNavigator(...);
} else {
    mNavigator.backToRoot();
}

However, in onStart method mNavigator.delegate().onStart() is called which throws an exception is the dispatching callback is not null, which is the case in that situation. Not sure what would be the best fix for that, because I don't see why this check is necessary. If it is, there should be a way to manually deactivate the dispatcher.
Also, interesting stuff, we tried to temporary hack it using this code:
mNavigator = Navigator.find(this);
if (mNavigator != null) {
    mNavigator.delegate().onStop();
    mNavigator.delegate().onDestroy();
    // simulate history clearing
    mNavigator.backToRoot();
    mNavigator.back();
}
mNavigator = ActivityArchitector.onCreateNavigator(...);

For some reason we can't explain (we would have to dig deeper into the code), it does not work. It seems like the Mortar scope gets cleared. Any ideas? Also, I guess a method to clear the history (that would call this.history.killAll(true)) would be useful in Navigator.

Ok, fixed it without any changes to the library.
Simply added android:launchMode="singleInstance" (not sure if singleTask is better...) and removed all the flags in the Intent created when receiving the push notification. The Activity is not recreated if already running. This can be closed.

android:launchMode="singleTop" :)


AssignChiralTypesFromBondDirs assigns chirality for achiral molecules?

It might be a feature, but who knows.
Presently I am integrating rdkit into BKChem. Initially I am going to use it for SMILES generation and SMILES reading. The easiest way to achieve chirality is to use proper bonddir parameter for dashed/wedged bonds and calculate later proper configuration. During unitesting I realized that when you do MolToSmiles for achiral molecules created in that way, @ appears in the SMILES. The smallest possible example below screenshot (replaceExistingTags=True does not help):

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
from rdkit import Chem
from rdkit.Chem import AllChem
mol_smiles = Chem.MolFromSmiles('CC(C)(F)Cl')
for bond_index in range(len(mol_smiles.GetBonds())):
    print(bond_index, mol_smiles.GetBondWithIdx(bond_index).GetBeginAtom().GetSymbol(), mol_smiles.GetBondWithIdx(bond_index).GetEndAtom().GetSymbol())

0 C C
1 C C
2 C F
3 C Cl
mol_smiles.GetBondWithIdx(2).SetBondDir(Chem.rdchem.BondDir.BEGINWEDGE)
mol_smiles.GetBondWithIdx(3).SetBondDir(Chem.rdchem.BondDir.BEGINDASH)
Chem.rdmolops.SanitizeMol(mol_smiles)
AllChem.Compute2DCoords(mol_smiles)
Chem.rdmolops.AssignChiralTypesFromBondDirs(mol_smiles)
Chem.MolToSmiles(mol_smiles)

CC@@(F)Cl
Chem.MolToSmiles(Chem.MolFromSmiles('C[C@@](C)(F)Cl'))
CC(C)(F)Cl
Expected behavior
In my opinion results of both actions should be the same - smiles without @ should be generated.
Configuration (please complete the following information):

RDKit version: 2019.09.1 (presently I am not able to check it on the newest one, it might have been fixed already, is so, then ignore it)
OS: [e.g. Ubuntu 20.04] Ubuntu linux
Python version (if relevant): 3.8.10
Are you using conda? no
If you are using conda, which channel did you install the rdkit from?
If you are not using conda: how did you install the RDKit? system packages, dpgk

Hi @wopozka : I agree that it seems a bit odd, but the code is behaving as intended.
AssignChiralTypesFromBondDirs() just sets the appropriate @ or @@ flags on atoms based on the bond wedging provided. If you then want to remove the chiral markers for atoms which cannot be stereocenters you should call AssignStereoChemistry() with the force and cleanIt arguments both set to True


dataframe.plot accepts undocumented 'by=' keyword but ignores it.

Maybe this is a request for enhancement, or documentation of a bug - I don't know.
A tutorial showed that DataFrame.boxplot() can take a by= keyword to produced plot stratified by the values in one column.   I naively assumed I could apply that to other plots as well.  DataFrame.plot() accepts by= but seems to silently ignores it.    This seems to be an undocumented keyword for DataFrame.plot().
Is this intentional?  Why not include the groupby functionality from boxplot into plot so it can do something reasonable with all the other plot types as well?  If not, then why does plot accept the by= keyword?
Example code below,
import pandas as pd
pd.show_versions()

plt.rcParams.update(pd.tools.plotting.mpl_stylesheet)

# taken from http://pandas.pydata.org/pandas-docs/stable/visualization.html#box-plots
df = pd.DataFrame(rand(10,2), columns=['Col1', 'Col2'] )
df['X'] = pd.Series(['A','A','A','A','A','B','B','B','B','B'])
plt.figure();
print(df)

# boxplot produces two subplots segregated by te values in B
df.boxplot(by='X')

# Other kinds of plots seem to silently ignore 'by='
df.plot(kind='box', by='X')

# Other kinds of plots seem to silently ignore 'by='
df.plot(kind='scatter', x=0, y=1, by='X')

# Other kinds of plots seem to silently ignore 'by='
df.plot(kind='line', by='X')

INSTALLED VERSIONS
------------------
commit: None
python: 2.7.6.final.0
python-bits: 64
OS: Linux
OS-release: 3.13.0-32-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8

pandas: 0.15.2
nose: 1.3.4
Cython: 0.20.1post0
numpy: 1.9.1
scipy: 0.13.3
statsmodels: 0.6.0
IPython: 2.3.1
sphinx: 1.2.3
patsy: 0.3.0
dateutil: 2.4.0
pytz: 2014.10
bottleneck: None
tables: 3.1.1
numexpr: 2.2.2
matplotlib: 1.4.2
openpyxl: 1.8.6
xlrd: 0.9.3
xlwt: 0.7.5
xlsxwriter: None
lxml: 3.3.3
bs4: 4.3.2
html5lib: 0.999
httplib2: 0.9
apiclient: None
rpy2: 2.5.2
sqlalchemy: 0.8.4
pymysql: None
psycopg2: None
       Col1      Col2  X
0  0.561433  0.329668  A
1  0.502967  0.111894  A
2  0.607194  0.565945  A
3  0.006764  0.617442  A
4  0.912123  0.790524  A
5  0.992081  0.958802  B
6  0.791964  0.285251  B
7  0.624917  0.478094  B
8  0.195675  0.382317  B
9  0.053874  0.451648  B

DataFrame.boxplot was united under DataFrame.plot() with the kind='boxplot' keyword argument a while back. boxplot was the only plot type to ever support faceting (with the by keyword).
There aren't any plans to add faceting for other plot types right now. Other libraries like seaborn or ggplot do it and handle DataFrames nicely.
A pull request to document the by keyword, noting that it only has an effect with kind=boxplot would be welcome! There's an issue somewhere about validating keyword arguments to df.plot and warning / raising when things are unused.

This is a hazard of our current API design, which uses a single plot method to dispatch to many different underlying plot types. The ultimate fix is probably to encourage more specific methods like df.plot.scatter() rather than df.plot(). See https://github.com/pydata/pandas/issues/9124 for more discussion of that design.

Thanks Tom, I'll dig into the code to see how it's used and document the 'by=' keyword.


rendering profile face

hi, patrik, how  can I  render face with different pose.
I tried yaw pitch roll in RenderingParameters, but no lucky.

solved. I should extract texture first .


Check for irreversibality once the transaction has been sent through api.transact?

How do I check if the transaction has reached its irreversible status?
What does this do?
useLastIrreversible : True
Does this mean that the api.transact will return only after the irreversibality has been confirmed??
I could not find it in doc, so hopefully I can get some help.

No, as stated in v21.0.2 release notes: adding the useLastIrreversible argument to transaciton methods will use the last irreversible block number for the transaction TAPOS headers.


Can't install phantomjs on Ubuntu 14.04

Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.8/README.Bugs> for instructions.
make[2]: *** [.obj/qxml.o] Error 4
make[2]: Leaving directory `/home/sofish/phantomjs-2.0.0/src/qt/qtbase/src/tools/bootstrap'
make[1]: *** [sub-bootstrap-make_first] Error 2
make[1]: Leaving directory `/home/sofish/phantomjs-2.0.0/src/qt/qtbase/src'
make: *** [sub-src-make_first] Error 2

Thank you for submitting this ticket!
This issue tracker primarily serves as the tool to report and track bugs, defects, and development tasks.
For general question, code review request, or other related discussion, please post it to either:

the official PhantomJS users mailing-list
Stack Overflow (with phantomjs tag)

If you have not read it yet, please check also a few tips on Effective Q&A.


Fix testing scripts

Multiple changes to HydroPowerSimulations.jl to bring back the testing
- Removed unused exports,
- Migrated most of the methods to use PSI,
- Added missing packages,
- Added solvers to testing .toml,
- Added missing parameters.

Current status: 219 test out of 220.

I will merge to fix the last test in a different PR and restore this capability


kudu 1.4.0 Hive 1.1.0 support

Does this version support kudu 1.4.0 and hive 1.1.0 ?
I have this issue.
AILED: Execution Error, return code 1 from org.apache.hadoop.hive.ql.exec.DDLTask. MetaException(message:Error creating Kudu table: test_alias_rut_hivekudu:org.kududb.client.NonRecoverableException: Too many attempts: KuduRpc(method=CreateTable, tablet=null, attempt=6, DeadlineTracker(timeout=10000, elapsed=8044), Deferred@1146919769(state=PENDING, result=null, callback=(continuation of Deferred@2091976938 after retry RPC after error@1976728735) -> (continuation of Deferred@81113478 after retry RPC after error@611542038) -> (continuation of Deferred@1358024234 after retry RPC after error@908162607) -> (continuation of Deferred@533838060 after retry RPC after error@540884180) -> (continuation of Deferred@2120431207 after retry RPC after error@1129228170), errback=(continuation of Deferred@2091976938 after retry RPC after error@1976728735) -> (continuation of Deferred@81113478 after retry RPC after error@611542038) -> (continuation of Deferred@1358024234 after retry RPC after error@908162607) -> (continuation of Deferred@533838060 after retry RPC after error@540884180) -> (continuation of Deferred@2120431207 after retry RPC after error@1129228170))))
Thank you
Regards,
Cristian

supports hive 1.1.0
https://github.com/BimalTandel/HiveKudu-Handler/blob/master/pom.xml#L30
check the pull requests for more recent version


Meet Up Website

Summary
This is the open issue that will capture discussion and workflow for the Meet Up's Website.

What needs to be done?

Design ✍

[ ] finalize wireframe design
[ ] finalize page content
[ ] decide on colour scheme ( 🟥 COMMUNITY LED DECISION)

Develop 💻

[ ] create static version of the page
[ ] add in active links (QA check)
[ ] add in updating functioning for links section
[ ] add in updating functionality for time zones

Who can help?
Technical

web developers (python/django)
web designers
python for automation

Logistical

anyone involved in the project

Updates

Should we put this into the Wiki instead as well? 😄

Yes!

I've made a pretty empty page here: https://github.com/alan-turing-institute/AutSPACEs/wiki/Meetups
If you have a link of links somewhere I'd be happy to add all the past information!

Could be closing this as we just need to keep adding the old notes link to the Wiki. @BrainonSilicon do you want to keep this around and close once the notes are there or just close this? :)

Could be closing this as we just need to keep adding the old notes link to the Wiki. @BrainonSilicon do you want to keep this around and close once the notes are there or just close this? :)

Let's close this so that there's not a weird paper trail. We can open a new one about the wiki.


How to set this up on web hosting without composer

title says all

Install composer on your local machine, composer require tumblr/tumblr, and then gather all the files and dependencies it created and upload them to your web host. You can repeat this as necessary to update the packages well.
There are some stackoverflow questions that may help you as well.


light-client: Optimize voting power calculation by breaking the loop when we have enough voting power

Fixes #235

[x] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[x] Updated all code comments where relevant
[ ] Wrote tests
[x] Added entry in .changelog/

Thanks @i-m-aditya! 💐


Can I change size of a container after creating an environment?

Similar to hub #2560.

Yes you go to Environments. Edit existing environment. Its containers appear on the page.
You can change size by clicking plus icon:


[Bug]: Undo of Transfer Shipment

Describe the issue
We experience the same issue as mentioned in reference issue https://github.com/microsoft/ALAppExtensions/issues/24386
BC 24.1
Also when we do undo of a transfer shipment of a item with a UOM different than the base UOM, then an error occurs:
Invoiced Quantity must have the same value as Quantity in Item Journal Line Journal Template Name='',Journal Batch Name='',Line No.='0'.
AL-call stack:
"Item Jnl.-Check Line"(CodeUnit 21).RunCheck line 86 - Base Application by Microsoft
"Item Jnl.-Post Line"(CodeUnit 22).Code line 14 - Base Application by Microsoft
"Item Jnl.-Post Line"(CodeUnit 22).PostSplitJnlLine line 12 - Base Application by Microsoft
"Item Jnl.-Post Line"(CodeUnit 22).RunWithCheck line 16 - Base Application by Microsoft
"Item Jnl.-Post Line"(CodeUnit 22).OnRun(Trigger) line 3 - Base Application by Microsoft
"Undo Posting Management"(CodeUnit 5817).PostItemJnlLine line 16 - Base Application by Microsoft
"Undo Posting Management"(CodeUnit 5817).PostItemJnlLineAppliedToList line 44 - Base Application by Microsoft
"Undo Transfer Shipment"(CodeUnit 9030).PostItemJnlLine line 63 - Base Application by Microsoft
"Undo Transfer Shipment"(CodeUnit 9030).Code line 56 - Base Application by Microsoft
"Undo Transfer Shipment"(CodeUnit 9030).OnRun(Trigger) line 15 - Base Application by Microsoft
"Posted Transfer Shpt. Subform"(Page 5744).UndoShipmentPosting line 6 - Base Application by Microsoft
"Posted Transfer Shpt. Subform"(Page 5744)."UndoShipment - OnAction"(Trigger) line 2 - Base Application by Microsoft
Expected behavior
Compare the base quantities, or a modified check so the posted transfer shipment can be undone, for an item that has been registered in another unit of measure than the base unit of measure.
Steps to reproduce
Create a transfer (warehouse) shipment for an item that is registered with a different UOM than the base UOM. Try to undo the posted shipment
Additional context
No response
I will provide a fix for a bug

[ ] I will provide a fix for a bug

Hi @JeoffreyAxians
I can see you unchecked the "I will provide a fix for a bug" box. I assume that's because you want to report a bug for Microsoft to fix? If that's the case, please report the bug through one of these channels instead: https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/help-and-support#bugs
This repository is not the right place to report product defects with customer impact to Microsoft. Issues reported in this repository do not fall under SLAs (Service Level Agreements) and hence have no guaranteed time to mitigation.
If you would like to provide a fix for this bug, you will need to fix it in the BaseApp contribution pilot. Do you want me to transfer this issue to the BaseApp pilot repository? To get access to that repository, please fill out the following form: https://forms.office.com/r/E9wyE5a58f.


Customizing 401 HTML code

Hey, is there any way we can possibly customize the error response in case of an error ? Instead of "Unauthorized" , i need to use this with swagger and integrate the default swagger error code but i have no way of doing it.

Correct me if I'm mistaken, but I think you can define the status code in the response of your node application, e.g., res.status(401) if you're using express.

@sdotson You'll need to create a custom callback/middleware for this. Check out: http://passportjs.org/docs/authenticate under custom callback.

@MocanuMarius I think what @ebaghern provided should answer your question. Let me know if not and we can reopen this.


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information.
##Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, GIT, and JavaScript with associated notes listed

I think the issue is resolved. Visit


added lookup helpers needed for data scrub

@irfansharif
new and improved #1

we can refactor this with ruby's magic functions later, if that's okay with you

un private the helper methods and add small tests for each one otherwise LGTM

what you have in the comments should suffice for tests tbh


Fix exception message for missing ModelContainer annotation

The old exception message was using outdated class names. It took me quite some time to figure out what was wrong. The new message should be safe for any future class name changes.

@panzerfahrer Looks like I merged a different PR for this same error message just before you sent this one in. If you can clean up the conflicts I can get this in, the message is more descriptive here.

@mlebarron Sure thing.


Added new app - AcadeMeData for AcadeMe Database

includes all files generated by startapp command
Does not include models
modified AcadeMe/settings.py file to add our app to the list of apps Django knows about.

Usually, I would ask for the same thing Yariv asked, but I see you have other open PRs about that.
so LGTM, but you need to rebase first.

New app AcadeMeData --- conflict resolved #33


QueryBuilder can build a subquery for where() but not select()

I noticed the following when looking to subquery an aggregate into the select list.
I found I could do it with knex's builder but not with objection.
Product.
     query().
     select('*',
// works
 select(function(builder){
 	builder.
 	avg('rating').
 	from('productReviews').
 	whereRef('productReviews.productId', '=', 'products.id')
 }),
// doesn't work as not wrapped with brackets
 select(ProductReview.
   query().
   avg('rating').
   whereRef('productReviews.practiceId', '=', 'products.id')).

I know you can pass a QueryBuilder instance to where() so thinking this might not be by design?

Weird, that should work. I'll write some tests and get back to you. I'm quite busy with other stuff right now, so it may take some days.

This works for me if you give the selection a name using as method. You probably want to do that anyway (?). Could you try something like this:
Product
  .query()
  .select('Product.*', ProductReview.query()
    .avg('rating')
    .whereRef('productReviews.practiceId', '=', 'products.id')
    .as('avgRating')
  )
  .then(products => {
    console.log(products)
  });

Confirmed working. Thanks!


Re-Test all recipes

There are recipes that are quite old and should be re-tested against current websites the updated/deleted were appropriate.
Note: Deletes are where nothing works, updates are where there are only a few minor glitches

[x] abgeordnetenwatch_de - Del
[x] aljazeera -Del
[x] androidpolice_com
[x] ap - Unable to find test feed
[x] arstechnica
[x] backchannel_com -Del No longer exists part of Wired.com
[x] bangkokpost - Del
[x] bbc.com - New
[x] bgr - Del
[x] breitbart.com - New
[x] buenosairesherald - Del
[x] chefkoch_de - Del
[x] cnet - Needs Update
[x] csmonitor - Del
[x] dailymail.co.uk - New
[ ] dailytech
[x] der-farang_com
[ ] der-postillon_com
[x] dilbert_com
[ ] discovermagazine
[ ] dorkly.com
[ ] epitition_bundestag_de
[ ] feedblitz
[ ] foxnews.com
[x] fredericksburg.com - New
[ ] geek
[x] github - Del
[x] github_releases - Needs update
[ ] golem_de
[ ] heise_de
[ ] heise_de_developer
[ ] heise_de_newsticker
[ ] heise_de_technology_review
[ ] heise_de_telepolis
[ ] heise_de_tr
[x] hiveworks - New
[ ] japantimes
[ ] japantoday
[ ] jsonline
[ ] koreaherald
[ ] lesjoiesducode.fr-image
[ ] linux - Del
[ ] linuxinsider - Needs update
[ ] managewp.org
[ ] massivelyop_com
[ ] merkur_de
[ ] moderntokyotimes
[ ] muckrock
[ ] nba
[x] nbc12.com - New
[ ] networkworld
[ ] npr
[ ] opensource.com
[ ] postillon_de
[ ] recode
[ ] sankakucomplex
[ ] scienexx_de
[ ] securityfocus
[ ] smashingmagazine.com
[ ] soundandvision
[ ] spiegel_de
[ ] stereophile.com
[ ] sz_de
[ ] tagesspiegel
[ ] theatlantic
[x] theguardian.com - New
[x] thehill.com - New
[ ] thehindu
[ ] themoscowtimes
[ ] thenewslens
[ ] theoatmeal_com
[ ] thestandard
[ ] time.com
[ ] titanic-magazin_de
[ ] toonpool_com
[ ] upi
[ ] wired
[ ] wonderhowto.com
[x] xkcd_com
[ ] yahoo.com
[x] zdnet
[ ] zeit_de

I'll take a stab at working on some of them

@dugite-code looks like AP is dead, http://www.hosted.ap.org routes to apnews.com.
I'll have to turn off my other AP plugin to test it, but ill write a new config in a bit


set type base on whether an optional property has been implemented

Search Terms
optional property type
Suggestion
set type base on whether an optional property has been implemented
Use Cases
writing type definitions on old api that take multiple param type is very hard, this is the current solution which needs to add an extra property to work
Examples
when paramB exists, paramA must be a string
interface a {
    paramA: any;
}

interface b {
    paramA: string;
    paramB: any;
}

type c = a | b;
const exampleA: c = {paramA: 2};
const exampleB: c = {paramA: 2, paramB: 1};
if (exampleA.paramB) {
  doSomething();
}
if (exampleB.paramB) {
  doSomething();
}

above code doesn't work
Checklist
My suggestion meets these guidelines:

[✔] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[✔] This wouldn't change the runtime behavior of existing JavaScript code
[✔] This could be implemented without emitting different JS based on the types of the expressions
[✔] This isn't a runtime feature (e.g. new expression-level syntax)

if..in statement can solve the problem


Split listing and identity contract migrations

We currently deploy nearly all smart contracts in 2_deploy_contracts.js.
Because we may not always want to also deploy user smart contracts, and for greater modularity, we should break smart contract "clusters" into separate migrations.

initial_migration
deploy_marketplace_contracts
create_sample_listings
deploy_identity_contracts
add_sample_issuers (Consider renaming to "create_sample_issuers" for naming consistancy.)

Resolved by https://github.com/OriginProtocol/origin-js/pull/352


Provide From<*> for lucet_runtime::UntypedRetVal

What do you think about the idea of adding these instances for representable primitives? It would make certain tests much easier to write (since I could construct a simulated return value from lucet).

Sure, should be no problem. Want to submit a PR for this? Otherwise I'll try to get to it when I return from traveling next week.


Add migration induced by aldryn-apphooks-config==0.3.0.

This commit updates the namespace and type fields of BlogConfig due
to aldryn-apphooks-config==0.3.0. Fixes:
https://github.com/nephila/djangocms-blog/issues/345

Thanks for this. Apparently there are some packaging errors in released 0.3.1 version which makes this PR to fail. I'll check with paulo and angelo the current state of aldryn-apphooks-config

Did i get the version wrong?

@electroniceagle nope. I think there is something wrong with 0.3.1 release

@electroniceagle I'm going to push two different PRs (for 0.8 and 0.9 releases) and I will pin the aldryn-apphooks version to make the tests pass
Thanks for raising this

Fixed by #348 and 7de807e7419cbf1503a834c22c54c5e764ed654b


Remove res.send(status, body) signature

Fixed the History.md mistake.

@tunniclm coverage has decreased by a fraction, otherwise looks good to me.

As mentioned in the other similar pull request (see https://github.com/expressjs/express/pull/2939#issuecomment-216882000), the slight reduction in coverage is due to the reduced number of lines in the changed file (response.js).

ma ma ma merged into 5.0 branch 🎉


[+] Allow caller to specify compression level via setter.

For context, refer to this issue

Compression field added to existing Info struct, since that seems to be it's purpose.
Default to deflate::Compression::Fast to maintain backward compatible output.
Compression enum is defined by deflate crate.

Implemented as a generic setter through the Parameter trait.
This simplifies the usage and conforms to the established
pattern.
One caveat with this approach is requiring the caller to import
deflate and png::HasParameters.
Example use of the api:
use std::io::prelude::*;
use png::{self, HasParameters};
use deflate;

/// Encodes the image ```image```
/// that has dimensions ```width``` and ```height```
/// and ```ColorType``` ```c```
pub fn encode<W: Write>(
    w: W,
    data: &[u8],
    width: u32,
    height: u32,
    ct: png::ColorType,
    bits: png::BitDepth,
) -> io::Result<()> {
    let mut encoder = png::Encoder::new(w, width, height);
    encoder
        .set(ct)
        .set(bits)
        // Note that we can now set the compression level.
        // If we don't manually set it, it defaults to
        // `deflate::Compression::Fast` which was the previously
        // hard-coded value.
        .set(deflate::CompressionOptions::default());
    let mut writer = encoder.write_header()?;
    writer.write_image_data(data).map_err(|e| e.into())
}

Ignore the force-push. I undid a commit that was me attempting to figure out how to import a crate from a fork.

Is there anything blocking this from being merged? :)


ci: add ionice idle level to cluster-sync

What this PR does
Before this PR:
After this PR:

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer
/cc none

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

/test pull-kubevirt-e2e-k8s-1.30-sig-storage
/test pull-kubevirt-e2e-k8s-1.30-sig-operator
/test pull-kubevirt-e2e-k8s-1.30-sig-compute
/test pull-kubevirt-e2e-k8s-1.30-sig-network
/test pull-kubevirt-e2e-k8s-1.30-sig-compute-migrations

/test pull-kubevirt-e2e-k8s-1.30-sig-storage
/test pull-kubevirt-e2e-k8s-1.30-sig-operator
/test pull-kubevirt-e2e-k8s-1.30-sig-compute
/test pull-kubevirt-e2e-k8s-1.30-sig-network
/test pull-kubevirt-e2e-k8s-1.30-sig-compute-migrations

/test pull-kubevirt-e2e-k8s-1.30-sig-compute

/test pull-kubevirt-e2e-k8s-1.30-sig-storage
/test pull-kubevirt-e2e-k8s-1.30-sig-operator
/test pull-kubevirt-e2e-k8s-1.30-sig-compute
/test pull-kubevirt-e2e-k8s-1.30-sig-network
/test pull-kubevirt-e2e-k8s-1.30-sig-compute-migrations

/cc @xpivarc @akalenyu @fossedihelm

@xpivarc @akalenyu I still think that it is worth applying this change as it will reduce competition on the disks in the CI nodes.

This has also chance to improve the Podman startup problem.

This has also chance to improve the Podman startup problem.

And potentially help with - https://github.com/kubevirt/kubevirt/issues/12412

This has also chance to improve the Podman startup problem.

And potentially help with - #12412

Good point, lets see. We need to check if this is actually still an issue

/lgtm
Just noting again that I did catch the etcd error with this PR in https://github.com/kubevirt/kubevirt/pull/12511#discussion_r1705441045

/cherry-pick release-1.3


Latency Compensation and Caching

I like what you're doing here -- Relay has a ton of awesome features but they're so opinionated, its hard to work with on your own. But two really hard features are going to be latency compensation and caching -- I'm curious what your angle of attack is going to be on these?
After finding the babel-graphql-plugin, I didnt realize that graphql parses to a simple JSON tree:
https://github.com/ooflorent/babel-plugin-graphql
I suppose diffing wouldnt be too hard, but then theres a question of how to organize this graph... Its not going to be easy... :/

You don't need babel-plugin for that. There are many ways to get the GraphQL AST. It's a tough to implement the cache locally. In this version, I just want to delegate it to some other layer Redux. Later on we can think of an well organized solution.


Title mangled on IOS Safari

FYI, The title "The perfect dark mode" comes out as "The perfect" in IOS Safari. There is a large black space where the second line of the title should be. This is the latest IOS/Safari 14.2 on an iPad air. I noticed this in dark mode, but changing to light mode had no effect.

Sorry, could you please provide a screenshot? Thanks so much for letting me know! 😄

It does the same thing on MacOS Big Sur Safari. It might be the font you used. After upgrading to Big Sur, I had to change fonts in VSCode because it came out black on black (the problem font was "source code pro").

I updated Safari and looks like this is a Safari 14 issue, I'll look around for ways to fix this. Thanks again for bringing this up! 🙌

I've deployed a temp fix: -webkit-box-decoration-break: clone; which shows two "rows" of text gradient like so:


feat: Allow calling of the UserInfo endpoint to be configurable

Allows the calling of the UserInfo endpoint to be configurable.
As mentioned in #148, ADFS does not support the email or profile claims, and calling the UserInfo endpoint is pointless, as it only returns the subject.
As the UserInfo endpoint is not required, this PR adds a new config item - call_userinfo_endpoint (default true), which allows the skipping of this endpoint.
Resolves #145

@stanhu @bufferoverflow - Would it be possible to get some feedback on this?


createGeohashesBoundingBox

Added createGeohashesBoundingBox that returns geohashes based on bounding box #10

Sorry it took this super long :) Looks fine!


How long to train the model from scratch

i train the model nealy 24 hours，but only get one tenth of the train stage  t = 108800 / 1000000，my device is one TITAN Xp.

Yes, it will take about 160 hours to train in a single GPU(in my case, 2080).
BTW, you can try to adjust image_print_every to reduce training time.


Pin dependency versions in Dockerfile.tpu

Pin the dependencies in Dockerfile.tpu to versions that work for Trillium

Hi @youkaichao it looks like we have a failing tpu test because of depyf not supporting torch nightly (breaking change with check_fn). Any tips on what to do?

thanks for the heads-up! let me fix it really quickly.

@bvrockwell the depyf issue should be solved by https://github.com/thuml/depyf/pull/64 . I think there's still an torch xla error from the ci results.

Closing as superseded by #10010


Upgrade Poetry installation

Type of changes

[x] Bug fix

Checklist

[ ] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Version 1.1.6 of the install-poetry action is now compatible with the Python 3.10 pre-releases, so upgrading the version here should fix the issue referenced in  https://github.com/willmcgugan/rich/issues/1216 🙂
Extra context
Unfortunately, while the original issue there is fixed, there seems to be another unrelated issue blocking the project from actually adding the 3.10 beta release into the matrix.
See this run, which outputs:
Installing dependencies from lock file

  SolverProblemError

  The current project's Python requirement (3.10.0b1) is not compatible with some of the required packages Python requirement:
    - dataclasses requires Python >=3.6, <3.7, so it will not be satisfied for Python 3.10.0b1

  Because no versions of black match >20.8b1,<21.0
   and black (20.8b1) depends on dataclasses (>=0.6), black (>=20.8b1,<21.0) requires dataclasses (>=0.6).
  Because dataclasses (0.8) requires Python >=3.6, <3.7
   and no versions of dataclasses match >=0.6,<0.8 || >0.8, dataclasses is forbidden.
  Thus, black is forbidden.
  So, because rich depends on black (^20.8b1), version solving failed.

I might be missing something obvious here, but this seems like an already resolved issue (see https://github.com/python-poetry/poetry/issues/1413), and the fix seems to be using notation to tell Poetry that the dataclasses dependency is only relevant on certain Python versions (3.6), which the current pyproject.toml already seems to be doing.
If this really was a problem with the Python version I would expect version 3.7-3.9 to also fail this validation, so I wonder if Poetry has a bug tied to versioning of pre-releases somehow.

Thanks!


Moving Sarifer code from SARIF VS Viewer repositoy

Description

There is no source code change compared to original files.
Updated the build related scripts to build only Sarifer extensions.
Removed the submodule references to Sarif-PatternMatcher repo, use nuget package instead.
Added the submodule to Sarif-VisualStudio-Extension to reuse shared libraies.

I see a file called "sarif-visualstudio-extension" in the src folder, what is it?

Description

There is no source code change compared to original files.
Updated the build related scripts to build only Sarifer extensions.
Removed the submodule references to Sarif-PatternMatcher repo, use nuget package instead.
Added the submodule to Sarif-VisualStudio-Extension to reuse shared libraies.

Its the sarif-visualstudio-extension submodule. Sarifer references to the Sarif.Viewer.VisualStudio.Interop project and maybe the shared Sarif.Viewer.VisualStudio.Shell/2022 projects in the future.

In reply to:<PHONE_NUMBER>

Ah, right

In reply to:<PHONE_NUMBER>


Setup LITL pipeline for issues with respect to conflicting dates

The data integration issues int1-int3 concern authorities with conflicting birth/death dates or publications with conflicting publication dates.
https://github.com/kbrbe/beltrans-data-integration/blob/27f0d058af040042b71f03f877dc3fff363dcb07/data-quality/integrated-issues.csv#L2-L4
The issues can be detected by using a Python postprocessing on the dataprofile CSV output.
Afterwards, the root cause should be identified by a human, before a human can fix the issue and document the corrected data (in such a way it can be used to take the corrections and root cause into account)

The script added in the commit above resulted in the following numbers for one of the last internal corpus versions (input were the not yet postprocessed files contributors-person-all-info.csv and integrated-data.csv:

186 conflicting publication years
58 conflicting birth dates (for actual corpus contributors)
18 conflicting death dates (for actual corpus contributors)


Changing ContentTypes' schema configured for navigation results in error

Enable a previously created contentType for navigation, configuring the models as described in the README (in this example it is named 'topic-items').
From AdminUI -> Plugin -> Content-Types Builder try to modify the schema of the contentType (adding a field for example)
Press the green SAVE Button => It notices "An error occurred"

Looking on NetworkTools I found that:
REQUEST URL
http://<HOST>:1337/content-type-builder/content-types/application::topic-items.topic-items
RESPONSE:
{"error":{"contentType.attributes.navigation.nature":["must be one of the following values: oneWay, manyWay, oneToOne, oneToMany, manyToOne, manyToMany"]}}
package.json
"dependencies": {
    "knex": "0.21.18",
    "mysql": "^2.18.1",
    "sqlite3": "5.0.0",
    "strapi": "3.6.2",
    "strapi-admin": "3.6.2",
    "strapi-connector-bookshelf": "3.6.2",
    "strapi-plugin-content-manager": "3.6.2",
    "strapi-plugin-content-type-builder": "3.6.2",
    "strapi-plugin-email": "3.6.2",
    "strapi-plugin-i18n": "3.6.2",
    "strapi-plugin-navigation": "^1.0.1",
    "strapi-plugin-upload": "3.6.2",
    "strapi-plugin-users-permissions": "3.6.2",
    "strapi-utils": "3.6.2"
  },

Closing. It is already tracked on
https://github.com/VirtusLab-Open-Source/strapi-plugin-navigation/issues/47


Support finding spec definitions & checking types based on behaviours

A common pattern in Elixir is to define a function spec in a behaviour via a @callback attribute, and then modules implementing that behaviour put @impl true on the implementing functions rather than reproducing the full spec.
It would be awesome if TypeCheck could work with these attributes to generate runtime type checks the same way it does for @spec! and @type!. E.g. replace the attributes with probably something like @behaviour! and @callback!, and generate runtime checks for functions with @impl!.
Ideally this feature would allow for handling behaviours defined without TypeCheck. I'm guessing that would happen in a similar way to the existing overrides functionality, but probably would need some tweaks to play nicely with the callback logic.
Hypothetical example:
defmodule MyBehaviour do
  use TypeCheck
  @callback! my_func(integer(), atom()) :: :ok | {:error, integer()}
end

defmodule MyModule do
  use TypeCheck
  @behaviour! MyBehaviour

  @impl! true
  # Gets a runtime-generated type check as if it had the spec from the callback, e.g.
  # @spec! my_func(integer(), atom()) :: :ok | {:error, integer()}
  def my_func(the_integer, the_atom) do
    :ok
  end
end

Hi there!
This is a great suggestion. I'm not sure yet how it could be implemented (but I'll give it some thought).
For now, I first want to focus on finishing the basic functionality first (which mainly means: ensuring that all common typespec map syntaxes are properly supported).
However, this definitely is a cool suggestion for the future!
:green_heart:

@orsinium This might be the prime example of something which we can support using the main spec parser.
That way, we could use it without having to add TypeCheck to the Behaviour's own module at all (and e.g. use it to check custom implementations for pre-existing behaviours).
@impl! true / @impl! SomeBehaviourName could then invoke, once the next function is defined, the parser's lookup/convert functionality to wrap the implementation using the spec from the behaviour module.
What do you think?

Closing this for now.
I still think it is an interesting idea, but I believe it might be better as an extra library rather than adding this logic to the core library necessarily. This will keep the core library simpler and more focused.


[Program]: Write a Julia program to find sum of ap series

Description
Write a Julia program to find sum of ap series
An arithmetic progression (AP) is a sequence of numbers such that the difference of any two consecutive terms is a constant. The nth term of an AP is given by the formula an = a1 + (n-1)d where a1 is the first term and d is the common difference.
a = 2
n = 5
d = 3

Output : 40

How to contribute

Save the solution in program/find-sum-of-ap-series.jl file
Add find-sum-of-ap-series.jl file in find-sum-of-ap-series folder

!assign


configurable highlight

Specifically this is useful for dark console backgrounds where the default
for 'Blue' for log level INFO is difficult to read.
Most of the changes are in src/encode/patter/mod.rs. I added default
parameters for deserializing. I added a function to describe the default
pattern string, and defined that at the top of the source file.
I added a color_map to the Encoder structs which is a HashMap that
defines the LogLovel->Color mapping. The color_map is of type
HashMap<Level, Option<Color>>, if the Option is None, then no styling
is applied.
TODO: Adding a feature means adding documentation, which isn't complete.
I can do this and add to the PR, however before i continue i want to
make sure this is the direction that the maintiner wants to take.
TODO: Other testing may need to be done. I was testing with cargo test --features file
with a modified config option in the yaml test config. As i am not familiar
with this code base there may be other cases that need to be tested.
TODO/Future: This patch only allows foreground color to be specified.
There are other aspects of styling like bold,italics,background color.
Does it make sense to allow for more detailed styling? That involves
more work and more work for the maintiner. I think forground color will
probably cover most everyone's cases.
To use, the relevant config change is as follows:
      pattern: "%m"
      color_map:
        INFO: Blue
        TRACE: Black

Related: issue:#17, PR:#186

I love the feature!
I added a couple of comments and was hoping you might add some tests.

Also, this should target devel

Closing in favor of PR:#204 which targets the devel branch rather than master.


Flutter Doctor insists on having Android Studio when I already have IDEA

It's kind of annoying to have to install an additional 500MB of IDE that I'll never use and which is essentially a copy of what I already have (Intellij IDEA) - is there no way that Flutter could use IDEA instead if present?

This is likely a dupe of https://github.com/flutter/flutter/issues/9747.

Closing as dupe of #9747.


add: signature command in iovnscli

Two notes dave, check the signature model if it's fine, I put the text message as bytes so it will be encode in base64 in json, this is to make sure the message stays intact and cycles of marshalling and unmarshalling do not mess with ordering and hence make the sig invalid.

Antoine invigorated Alessio to revisit https://github.com/cosmos/cosmos-sdk/issues/4581, so let's see how that develops before we proceed.

Closing this for now, eventually we can rivis it in the future

The future arrived quickly. :)

Obviated by https://github.com/iov-one/iovns/pull/356.


[gha] non-relative paths

Motivation
Fix the ci-test-complete workflow.
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
CI
Fix verified-ish here... https://github.com/diem/actions/pull/27
Related PRs
None
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

/land

:broken_heart: Test Failed - ci-test

/land


Fix screenreader Question reading order

What is the context of this PR?
Fixes issue where question descriptions are being read out by screenreaders before all questions. Descriptions need to be read out first for relationships where it makes more sense to read the description first grammatically.
This will be a BREAKING CHANGE because now the relationship macro calls the question macro inside of it rather than it be configured the other way round.
How to review

Check questions read out first for non relationship questions
Check question descriptions read out first for relationship questions

 HIGH: D.O.B question - hint text question description above h1 (pg.29) >> Investigate issue in runner - was done for relationships context

Can we move the 'anatomy of a question' pattern under the 'Page' sub-category, just rename it 'Question', and remove 'Questionnaires' category?


Keeps getting error

when i do buycraft secret then the code it just keeps on giving me an error on game and console and it does have the secret code in the config

This error is being looked into.

Hi Falkirks!Please answer my Questions: 1.:Can I write you in a E-mail my Buycraft E-mail and Password? After you look if it works on your Server I can change my secret key and I can change my datas! Can you do this???
2.: Can you make a Tutorial how I configure Buycraft with VIP Slots???
3.: When a player buy anything,he pay money! But where I can confige at which Bank Account the money should be go????
4.: Please write back when you read this! And sorry for my badly English! I speak German!thanks

It might work now cuz I 4got to pay for it so it might work now if I set it
up again we will just see and if it doesn't I will tell you what you want
to know.
On Sunday, March 8, 2015, Drumpit<EMAIL_ADDRESS>wrote:

Hi Falkirks!Please answer my Questions: 1.:Can I write you in a E-mail my
Buycraft E-mail and Password? After you look if it works on your Server I
can change my secret key and I can change my datas! Can you do this???
2.: Can you make a Tutorial how I configure Buycraft with VIP Slots???
3.: When a player buy anything,he pay money! But where I can confige at
which Bank Account the money should be go????
4.: Please write back when you read this! And sorry for my badly English!
I speak German!thanks
—
Reply to this email directly or view it on GitHub
https://github.com/Falkirks/BuyCraft/issues/5#issuecomment-77766010.

Thanks but can you answer all Questions Falkirks????

Don't know what you mean
yes
I haven't got to it cuz of the error

On Sunday, March 8, 2015, Drumpit<EMAIL_ADDRESS>wrote:

Thanks but can you answer all Questions Falkirks????
—
Reply to this email directly or view it on GitHub
https://github.com/Falkirks/BuyCraft/issues/5#issuecomment-77768605.

@Drumpit

No. I can test on my own account.
Probably not. I don't have much time on my hands.
That is something that buycraft.net handles, you can configure a bunch of payment gateways.

On Monday, March 9, 2015, Drumpit<EMAIL_ADDRESS>wrote:

And this Bug.... have you fond it???
—
Reply to this email directly or view it on GitHub
https://github.com/Falkirks/BuyCraft/issues/5#issuecomment-77864653.

Yes thanks!
How I connect Buycraft with VIPSlots and a BankAccount???@Falkirks

Do either of those plugins have commands for management?

@Drumpit VIPSlots has "/vips <add\remove>", watch https://youtu.be/vydmBue9rmk for tutorial of how to add a command.


Update versioning metadata

Changes the reference point after the v0.1 tag was applied.

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   94.71%   94.71%
=======================================
  Files          23       23
  Lines        1759     1759
=======================================
  Hits         1666     1666
  Misses         93       93

Continue to review full report at Codecov.

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


LOGTOOL-33 Failing testcase

This is a failing test case for LOGTOOL-33, which I just opened.

Merged. Thanks Sanne.


app credentials format

preliminary agreement between @MHBauer @swetharepakula @nimakaviani 2018-10-03 about 4PM pacific

one "external_address" field for access from outside of the cluster/vm/bosh deployment (useful for application access to the delpoyed contracts/eth nodes)
one "internal_address" field for access by in-cluster/local resources to avoid a hop to outside (for kubernetes this is the cluster-ip or service-name which resolves to cluster ip)
a mapping of internal ports the geth-node/blockchain-client is listening on and used to be connected to.

mhb: I'm not sure we should be using the ports as the key for lookup. I'm thinking it should probably be an array of objects, and each object should be {"internalPort":80, "externalPort":8080}

ports as strings is still weird, but I consider this
closed by #65


java.lang.IllegalStateException: Invalid pool element found

Server version: CraftBukkit version 3161-Spigot-f773da8-f81f499 (MC: 1.17) (Implementing API version 1.17-R0.1-SNAPSHOT)

Full output of /mv version -p: https://j.mp/3wUF9lQ

Server log: https://gist.github.com/zhro/f9c8f517a34dd14e64532e4206963561

Details
I was <able / not able> to reproduce my issue on a freshly setup and up-to-date server with the latest version of Multiverse plugins with no other plugins and with no kinds of other server or client mods.
Description
Tested on latest Multiverse-Core 4.3.1 snapshot with Spigot 1.17
There is an unknown error that is occurring when the server starts up since updating to 1.17. This error does not appear when creating a new world. It only appeared when updating an existing wold.
Steps to reproduce
Occurs in the log when the server starts up.

I suggest reporting this to spigot or paper, doesn't look like something caused by mv.

Can this issue be reproduced in 1.16?

https://bugs.mojang.com/browse/MC-227945


Fix auto prefix bug

Resolves #3193
Fixes a bug that occurs when the entire prompt has already been computed. If the entire prompt is marked as computed, the model runner will attempt to create zero-sized tensors for non-computed prompt tokens. This throws an exception in torch.arrange.
This fix is somewhat of a band-aid because it just makes sure that the last block is never considered computed. This will result in a small amount of unnecessary computation when running the same prompt multiple times.
A more comprehensive fix would update the model runner to behave correctly when it detects that all blocks coming in have already been computed. This can be done in a subsequent change.

@ElizaWszola , note that yapf failing

@zhuohan123 I ran benchmark_throughput.py with and without auto prefix caching enabled, and it looks like the code is slightly faster after the fix:
old version with auto prefix caching
Throughput: 8.61 requests/s, 4166.47 tokens/s
new version with auto prefix caching
Throughput: 8.92 requests/s, 4317.72 tokens/s
old version without auto prefix caching
Throughput: 8.98 requests/s, 4347.19 tokens/s
new version without auto prefix caching
Throughput: 9.10 requests/s, 4401.76 tokens/s


This port is used by Apache Tomcat.

Can be a problem in some systems. You can look for unassigned ports there: https://en.wikipedia.org/wiki/List_of_TCP_and_UDP_port_numbers
https://github.com/amateomi/BSUIR-labs/blob/ad14fe12881559fb67b90869ea477cd4813baafd/SPOLKS/lab01-Basic-TCP-server/src/main.rs#L10

I guess you still need to fix it.....


Error with Instructions for Counting Cards - 196

Challenge Counting Cards has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
The issue isn't so much with getting the proper code, but more-so with the wording of the task. The task tells you that they want a specific output with a "Space" in-between the Count and "Belt" or "Hold" and then they give you the following Example:
Example Output
"-3 Hold"
"5 Bet"
So, when making this (and you can see my code below) I added the "" around my output, thinking that this is what was needed (since I followed the Example). However, after about 30 minutes of re-doing my code and changing methods, I found it was the "" surrounding my answer.
I might be nitpicking here, but I believe if you show an Example Output, it should be of the actual output, not the output being quoted (throws people like myself off, especially when I thought they were testing to make sure we remember how to use "escapes")
My code:

var count = 0;

function cc(card) {
  // Only change code below this line
switch(card){
    case 2:
    case 3:
    case 4:
    case 5:
    case 6:
      count++;
      break;
    case 7:
    case 8:
    case 9:
      count = count;
      break;
    case 10:
    case "J":
    case "Q":
    case "K":
    case "A":
      count--;
      break;
  }

  if (count > 0){
    return "\"" + count + " Bet\"";
  }else
    return "\"" + count + " Hold\"";
  // Only change code above this line
}

// Add/remove calls to test your function.
// Note: Only the last will display
cc(10); cc('J'); cc('Q'); cc('K'); cc('A');

@Diglio thanks for the issue. Your suggestion has been implemented in the beta site. So I'll be closing this as resolved in the development branch. The update will be pushed to production in the future. Happy coding!


Enhancement for the Chinese readme:Are pr recommended?

I'm willing to write a Chinese version of the readme and other docs for this project. As a native Chinese speaker, I believe I can contribute significantly to the project. Currently, I am a high school student, yet I am eager to dedicate some of my time to this project.
Thank you for considering my offer to help. I look forward to the opportunity to contribute to your team.
Best regards,
Sam

yeah of course we welcome your contribution.


Submit Media Grid snippet for approval

I've addressed all PR feedback. Let me know what you think.
Thank you!

Awesome work! Thanks for contributing :smile:


Cannot read property 'waveCtx' of undefined

Wavesurfer.js version(s): 2.1.2
Browser version(s): Chrome Version 71.0.3578.98 (Official Build) (64-bit)
OS: Windows 10 x64
Issue:
I have a user load a video from local storage and wavesurfer.js creates a waveform of the audio. However, when pressing play on the html5 video player the following error is logged in the console:

Uncaught TypeError: Cannot read property 'waveCtx' of undefined
at n.value (drawer.multicanvas.js:486)
at drawer.multicanvas.js:297
at drawer.multicanvas.js:561
at frame.js:12

When I pause the video and play again it display the waveform and everything is great again! Just looking for some insight. Thanks!
Code needed to reproduce the issue:
window.wavesurfer = WaveSurfer.create({
              container: '#waveform',
              barHeight: 1.5,
              barWidth: 1,
              fillParent: false,
              minPxPerSec: 1,
              responsive: false,
              hideScrollbar: true,
              interact: false,
              waveColor: "rgba(1, 1, 1,0.7)",
              normalize: false
            });
            wavesurfer.load(settings.srcFile);

@Latentimagedesign can you create a codepen that re-creates this issue?

no feedback, closing.


[workloadmeta] Refactor source filtering

What does this PR do?
Previously, each collector pretty much owned its own Source (as in,
the kubelet collector used a kubelet source, the docker collector
used a docker source, and so on). This implies that components that
filter by container runtimes would need to update their filters for
every new container runtime the workloadmeta supports, which is less
than ideal.
To solve that, the sources have been reworked. Now we have the following
sources:

SourceRuntime: entities detected by the container runtime running on
the node, collecting lower level information about containers.
docker, containerd, podman and ecs_fargate use this source.
SourceNodeOrchestrator: entities detected by the node agent from an
orchestrator. kubelet and ecs use this.
SourceClusterOrchestrator: entities detected by calling the central
component of an orchestrator, or the Datadog Cluster Agent.
kube_metadata and cloudfoundry use this.

This commit also removes the ability to filter by more than one source,
as that's no longer necessary, simplifying the code a lot.
Describe how to test/QA your changes

Check that agent workload-list --verbose shows a reasonable output of the workloads running on the node.
Check that agent tagger-list seems plausibly close to the above
Check that autodiscovery still works.

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The appropriate team/.. label has been applied, if known.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

@juliogreff did you want to rebase or merge from master?

@djmitche I'll rebase once master is unfrozen

Should we get this rebased and merged?


Processing models with large number of voxels

I am trying to process large models using small voxels and I usually have memory problems. I guess there are two things that can be done to improve this.

Using Minimum Volume Bounding Box to align the voxels model with input data, and have less empty voxels.

This will reduce to some extend the problem.

Create patch-based voxel system, where the model would be subdivided in many smaller same size voxel models. For example, 32x32x32 size patches which relative position are known compared to the other patches. You can store the patches in some files externally, and use them when you need. This would enable to create a lot of voxels.

Hola @MitkoAle ! Thanks for reporting and the suggestions :)
As many things in PyntCloud (actually everithing xD), the VoxelGrid implementation is not optimized at all.
In my opinion, the best way to reduce the memmory usage is to use sparse arrays to store the VoxelGrid information. This requires a little bit of refactoring of the current VoxelGrid code and I really hope to find the time and energy some day to do this.
Regarding your suggestions:

1

This migth be a good solution but it's efectiveness is conditioned by the shape of your point cloud and I'm not sure (at first glance) if adding this logic to VoxelGrid in a generalizable way would be possible.
Regardless, you can use the rotations available in https://github.com/daavoo/pyntcloud/blob/master/pyntcloud/geometry/rotations.py to manually aling the point cloud before computing the VoxelGrid.

2

If I understand correctly the thing you are describing is some sort of Octree.
This is a widely use solution and I think that it should not be part of / extend the VoxelGrid structure but rather should be implemented as an independent structure.
The only reason of not having an Octree structure so far is that I haven't been able to implement it in "pure" Python (+numpy) in a satisfactory way. I might try to wrap some existing external implementations.

Hola @daavoo ! :)
I've not worked with sparse and dense arrays. Probably, the computational time would be better using them.
Regarding the suggestions:

1

I agree with you that a manual rotation can be a solution.

2

I'm not talking about octree structures. I think Minecraft uses such kind of structure that I suggest, keeping huge models in smaller voxel models as separate objects. In this way, you can have a lot of small voxel models knowing their position, which represent the main voxel model instead. I guess if you store them in some files during the voxel creation you can think of working with very small voxels.


Create CloudFormation Wait Condition

Create a wait condition in the CF template so that it will turn green when the LB url is ready and available

https://github.com/neo4j-partners/amazon-cloud-formation-neo4j/pull/44


Enhancement: automatic copying of url

Main use of this project is to quickly share some text to other people. So wouldn't it be better if the url to share is automatically copied to clipboard when we click save button? (and also show a toast notification)
If so Please assign to me

@ThejasKiranPS Good one, I guess this makes total sense. If we need to implement this, I think we might need the Copy button also next to the save button and save can be replace with share.
@alceil What are your thoughts on this

Ya, it can be a good enhancement @ThejasKiranPS  go ahead with it

need some help.
I'm having a dilemma on from where to copy the url to clipboard (typing page or share page)
Typing page
This should be the correct page to copy the url (immediately after clicking save/share), but since it immediately redirects to share page there wont be time to notify the user that the url had been copied (toast wont have time to display)
But copying works perfectly
So I tried to delay the history.push() command by 1.5 second so that the toast have enough time to display
(i will link a pr so that you can view the netlify build)
Share page
Option 1
copy to clipboard after loading the component
but that would cause all the users viewing the page( when we share the link) to get the toast
it could be solved by keeping an isOwner variable in the authors browser which then can be read and determine if this is the author and if so copy the url
But that would overcomplicate things
Option 2
copy to clipboard when clicking on the share button (in share page)
What to do?

@ThejasKiranPS
I guess you can try the following approach

The user types the content in the editor
In that page he get's two options - share and copy
On clicking share - Call the api to generate unique link, copy that url to the clipboard and show a toast conveying that the share url has been generated and copied to your clipboard.
On clicking copy -  The text will be copied to the clipboard.

Here we are trying to avoid the redirection to another page and doing all things in one single route.

oooh that's pretty good. Working on it

Hey, can you do a git pull and resolve conflicts new changes have been made!


Backup file size limitation

Is there any limitation for the size of the backup file or for the size of a site collection that can be backed up using this cmdlet?

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

ID: c3624b67-49a9-3920-fa38-853e8590e75c
Version Independent ID: 944a6907-a47b-1ae6-d7c0-cdf8b3d1425d
Content: Back up site collections in SharePoint Server
Content Source: SharePoint/SharePointServer/administration/back-up-site-collections.md
Product: sharepoint-server-itpro
GitHub Login: @bentoncity
Microsoft Alias: stevhord

@scanum please close.


how to make the graph draggable?

i try to make the graph like your example ,but it seems the graph is fixed

This is a library that I don't maintain anymore, the dragging functionality should come out of the box, could you please use these two scripts instead?
<script src="https://cdnjs.cloudflare.com/ajax/libs/d3/3.5.6/d3.js"></script>
<script src="https://mauriciopoppe.github.io/greuler/scripts/lib/cola.v3.js"></script>

I think that the URL of the cola library is no longer compatible with this library, the one I have on the website is a snapshot of it

This is a library that I don't maintain anymore, the dragging functionality should come out of the box, could you please use these two scripts instead?
<script src="https://cdnjs.cloudflare.com/ajax/libs/d3/3.5.6/d3.js"></script>
<script src="https://mauriciopoppe.github.io/greuler/scripts/lib/cola.v3.js"></script>

I think that the URL of the cola library is no longer compatible with this library, the one I have on the website is a snapshot of it

i've already use the new url http://marvl.infotech.monash.edu/webcola/cola.js  but it seems not work.
part of my code is
https://paste.ubuntu.com/p/nF4ZCMjq65/
thank you for your kindness

i've already use the new url http://marvl.infotech.monash.edu/webcola/cola.js  but it seems not work.
part of my code is
https://paste.ubuntu.com/p/nF4ZCMjq65/
thank you for your kindness

I've created a small working example here, sorry if the links on the main page are different:
https://codesandbox.io/s/cocky-sammet-e6sl6?file=/index.html

I've created a small working example here, sorry if the links on the main page are different:
https://codesandbox.io/s/cocky-sammet-e6sl6?file=/index.html

greuler is now available as an npm module, check out the homepage:
npm install greuler@alpha

greuler is now available as an npm module, check out the homepage:
npm install greuler@alpha


Bindings are incorrect for multiple queues with empty name after recovering

The spring boot 2.7.2 is used with spring-boot-starter-amqp.
Bindings are incorrect after recovery when multiple queues with an empty name (server-named) are used with the same RabbitAdmin.
The first queue has binding to exchange with the "#" routing key. The second queue has another binding to the same exchange, but differ routing key like "*.*.5". There are two queues as expected after losing and establishing a connection to rabbitmq. However, one of the queues has no bindings, but the second one has two bindings to exchange with both routing keys.
The expected result is two queues with the same bindings as before the lost connection.
I think the problem is in org.springframework.amqp.rabbit.core.RabbitAdmin#initialize that declares bindings by queue name. However, in this situation, both queues have the same name.
Steps to reproduce:

Make configuration as in the sample below.
Simulate losing connection with rabbitmq by disabling the internet or stopping rabbitmq service.
Restore connection with rabbitmq (enable internet or run rabbitmq service).
Check recovered queues and their bindings.

Configuration sample:
    @Bean
    public Queue queue1(RabbitAdmin rabbitAdmin) {
        Queue queue = new Queue("", false, true, true);
        queue.setAdminsThatShouldDeclare(rabbitAdmin);
        rabbitAdmin.declareQueue(queue);
        return queue;
    }

    @Bean
    public Binding binding1(RabbitAdmin rabbitAdmin, Queue queue1) {
        Binding binding = new Binding(queue1.getName(), Binding.DestinationType.QUEUE, "exchange", "*.*.5", new HashMap<>());
        binding.setAdminsThatShouldDeclare(rabbitAdmin);
        rabbitAdmin.declareBinding(binding);
        return binding;
    }

    @Bean
    public Queue queue2(RabbitAdmin rabbitAdmin) {
        Queue queue = new Queue("", false, true, true);
        queue.setAdminsThatShouldDeclare(rabbitAdmin);
        rabbitAdmin.declareBinding(binding);
        return queue;
    }

    @Bean
    public Binding binding2(RabbitAdmin rabbitAdmin, Queue queue2) {
        Binding binding = new Binding(queue2.getName(), DestinationType.QUEUE, "exchange", "#", new HashMap());
        binding.setAdminsThatShouldDeclare(rabbitAdmin);
        rabbitAdmin.declareBinding(binding);
        return binding;
    }

Isn't this related somehow to this one: https://github.com/spring-projects/spring-amqp/issues/2439 ?

With the fix, you would use
    @Bean
    public Binding binding2(RabbitAdmin rabbitAdmin, Queue queue2) {
        Binding binding = new Binding(queue2, DestinationType.QUEUE, "exchange", "#", new HashMap());
        binding.setAdminsThatShouldDeclare(rabbitAdmin);
//        rabbitAdmin.declareBinding(binding);
        return binding;
    }

By the way, you should NOT call the admin from a bean definition; the framework will do that for you at the correct time.

And that one is probably wrong.
It feels like an error must be thrown from the broker to notify the client that there is no way to determine the target queue for binding...

With the fix, you would use
    @Bean
    public Binding binding2(RabbitAdmin rabbitAdmin, Queue queue2) {
        Binding binding = new Binding(queue2, null, DestinationType.QUEUE, "exchange", "#", new HashMap());
        binding.setAdminsThatShouldDeclare(rabbitAdmin);
//        rabbitAdmin.declareBinding(binding);
        return binding;
    }

By the way, you should NOT call the admin from a bean definition; the framework will do that for you at the correct time.

What to do if there are multiple RabbitAdmin beans? Does it work correctly?

Work around here: https://github.com/spring-projects/spring-amqp/issues/2439#issuecomment-1481638789


Option for SuperClusterer algorithm bounding box

Hello @jpoehnelt,
Is there a reason why 2 months ago (#60, src/algorithms/supercluster.ts:96), you switched the bounding box for getClusterer() in the SuperClusterer algorithm from a dynamic [west, south, east, north] to a static [-180, -90, 180, 90]?
We were working on a project, and we had some huge performance issues (tested on Ubuntu 20.04 LTS and macOS 12.1 Monterey).
Running on 2.3k markers, going above 100 clusters on the entire map would literally crash the browser.
Digging through the SuperClusterer algorithm, we found the bounding box set on those previously mentioned static numbers. Changing back those values for the dynamic ones, fixed it.
Correct me if I'm wrong, but it seems relevant to only want the algorithm to operate on currently visible clusters (instead of the entire world).
Thank you,
Have a lovely day.
(Related pull request : !135)

There is a trade off here that needs to be analyzed further.

The current implementation only computes clusters when the zoom has changed but for the entire globe.
This implementation computes clusters every time the map bounds changes but for a smaller region.

Which is better? Might depend on the specifics of the data. If markers are concentrated in a small area, the current implementation is probably more performant. If the markers are more uniform, probably the latter implementation is better.

That's an interesting point, we directly assumed it was a bug because of the browser crash and change in src/algorithms/supercluster.ts:96.
We have made another algorithm as suggested.
I'm interested in you mentioning that the current implementation is more optimized for concentrated data, because, that represents our case.
We were confronted with huge performance issues, using 2.3k markers in an approx. area of 30 km².

If you also feel divided by this, and understand the need for a dynamic bounding box in some cases, maybe it would be appealing to introduce it as a SuperClusterOptions?
(This goes hand-in-hand with extendBoundsToPaddedViewport(), so it doesn't feel too far-fetched to add it here.)
Some sort of :
enum bboxStatus {
    WHOLE_GLOBE,
    DYNAMIC,
    CUSTOM
}

function defineBoundingBox(status: bboxStatus = bboxStatus.WHOLE_GLOBE, custom?: number[]): number[] { ... }

If it were to be implemented and added in the docs, it would save lots of time for future people encountering this problem.

Good suggestion on the configuration option.

Hi, i have the same performance issue... it would help if this option existed in the supercluster options...
Do you plan to add this option in the next future version? thank you

Please submit a pr! 😄


POC julia solver

ping @josephsalmon @tomMoral @ngazagna @gowerrobert
just a quick hack to show the way.
please take over and let me know if you see any obvious bug

I just tried:
from lasso_pgd import lasso
ModuleNotFoundError: No module named 'lasso_pgd'
can you send me folder with a script that works for you?

thx @tomMoral
that's a big step forward !


Anerdw: Watch SCAMRECOVERYSITE[\W_]*+FIRM

Anerdw requests the watch of the watch_keyword SCAMRECOVERYSITE[\W_]*+FIRM. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

SCAMRECOVERYSITE[\W_]*+FIRM has been seen in 4 true positives, 0 false positives, and 0 NAAs.

Approved by VLAZ in Charcoal HQ


[Phcode.dev Bot] Publishing Release 0.0.1 to Extension Store

Thank you for contributing to phcode.dev extension store.
Please track extension/theme publish status by clicking this link.
You may close this issue at any time.

Extension successfully published to store. You can close this issue at any time.


nd4j-x86 resulting in errors

I have been trying to create an array, and the build breaks.
Here is the code :
val arr = (1 to 8 by 1).toArray
arr.map(println(_))
val m2 = Nd4j.create((1 to 8 by 1).toArray, Array(2, 2, 2))
Here is the error I am seeing
java.lang.IllegalArgumentException: Illegal index 40320 derived from 1 with offset of 0 and stride of 40320
[info]   at org.nd4j.linalg.api.ndarray.BaseNDArray.linearIndex(BaseNDArray.java:2931)
[info]   at org.nd4j.linalg.api.ndarray.BaseNDArray.getDouble(BaseNDArray.java:3380)
[info]   at org.nd4j.linalg.api.ndarray.BaseNDArray.toString(BaseNDArray.java:4511)
[info]   at org.nd4j.linalg.api.ndarray.BaseNDArray.toString(BaseNDArray.java:4539)
[info]   at org.nd4j.linalg.api.ndarray.BaseNDArray.toString(BaseNDArray.java:4539)
Note that the jblas package didn't present me with these errors.

I believe the indexing issues were fixed in nd4s.


APP-2173 Update fragments endpoints to take in org id

clarification q:

do fragments always belong to an org?
if so, would a user ever access a fragment in an org they don't belong to?

an alternate design is to make these endpoints use the eventual currentOrg field stored on the user object in mongo -- but this will take longer to implement

clarification q:

do fragments always belong to an org?

That is my understanding.

if so, would a user ever access a fragment in an org they don't belong to?

If they were public, but I think only Viam can make public fragments right now.
@nataliajacobowitz are both of the above correct?

an alternate design is to make these endpoints use the eventual currentOrg field stored on the user object in mongo -- but this will take longer to implement

We can ticket this out as a mini project.


Add basic support for building against custom USD library prefixes

This adds support for building against USD libraries with a custom naming prefix via a new USD_LIB_PREFIX build variable.

I'm going to roll these changes into a larger PR that addresses #4, so I'm closing this one.


Dev

Release of 4.3.0

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 12 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/217/rdkcentral/Lightning-SDK

Commit: aca11ca0bb0132c8770d05ea8a421866b1615494

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 12 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/217/rdkcentral/Lightning-SDK

Commit: 190e6c379a1d1d19aabc8fa360c5e51b79b18758

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 13 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/217/rdkcentral/Lightning-SDK

Commit: 01e4df87f4ba95d1302ac5944b03b98e86510cdf

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: Needs PR166 merged to stop this happening (RDK header switched to Metrological); tools now expect Metrological

Commit: 01e4df87f4ba95d1302ac5944b03b98e86510cdf