New channel due to missing replaces(postgresql community)

Signed-off-by: Martin Vala<EMAIL_ADDRESS>Hi @jkatz,
there are two options, maybe you forgot to set replaces for the latest version or it was intended.
We added a version to separate channel as it actually fall off the update graph and are orphaned and not installable / upgradable anymore due to missing replaces in the path. After this PR, it will be installable from this channel.
The main reason for this update is that we are currently preparing to enable bundle based catalogs which involves background conversion, it’s currently failing for the reason stated above.
Our PR is just technically correct workaround, but only you know if everything can be replaced by higher version or not. If replaces is missing somewhere, we have 2 options. Create new channel or add replaces where it is missing. The last option is better, but only possible for someone who knows operator very well, so you as an author. Feel free to open a new PR if you have different opinion, we can communicate and get it merged.
Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

New Submissions

[ ] Does your operator have nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Is your submission signed?

Updates to existing Operators

[ ] Is your new CSV pointing to the previous version with the replaces property?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Submit operators to both upstream-community-operators and community-operators at once
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@mvalahtv Per the comment by @J0zi in #2042 (https://github.com/operator-framework/community-operators/pull/2042/files#r458846365)

Hi @jkatz. We are in the replaces mode. You can set replaces in specific CSV and point to previous version, so update is possible. If you don't specify replaces, it means version X has big changes and does not allow replacing version Y. So you can start new channel.
Maybe you just forgot to put replaces to versions 4.0.1 and 4.1.0. Just put it there and all versions stay in stable channel. So we don't need to create new channels.
Our PR is just technically correct workaround, but only you know if everything can be replaced by higher version or not. If replaces is missing somewhere, we have 2 options. Create new channel or add replaces where it is missing. The last option is better, but only possible from someone who knows operator better than me:)

Would it be better if I just corrected the replaces chain in the older CSVs? That way we wouldn't need all the channels?

@jkatz, i just fixed replaces as you requested. Should i do the same for upstream?

@mvalahtv Yes please, that would be awesome. Thank you!


Override labels in [label=x] syntax

Hi, when I try to override the labels used in [label=x] it just uses the string values specified behind label=:
raw-render(
  ```
  digraph {
    rankdir=LR;
    size="8,5"

    node [shape=circle]
    s_1 -> s_2 [label=a]
    s_1 -> s_1 [label=b]
    s_2 -> s_1 [label=b]
    s_2 -> s_2 [label=a]
  }
  ```,
  labels: (
    s_1: $s_1$,
    s_2: $s_2$,
    a: $a$,
    b: $b$
  )
)

This looks like a bug to me or is there any other way to do this?

It's not a bug. Graphviz in its current state doesn't allow us to use the method we use for the node labels in edge labels. It's inconvenient but unfortunately we didn't find a reliable solution. Maybe in future version it will be possible but not now.

Ah, that's unfortunate.


Multi-language support？

Hi, I like your project, but I am wondering if you plan to make the resume support multiple languages or not.
like a switch between English and Chinese
i tried to do so in my resume, but the code looks ugly in the end

Hi Shyu, thank you for your request.
In our future plans, I have taken into consideration the support for internationalization (i18n). I am considering using next-intl  to achieve this and allow for multiple versions of the resume to be deployed together.
However, this feature will not be integrated into the current main branch due to the following reasons:

The main branch is currently in a feature-frozen state and will undergo significant changes in future versions.
next-intl is currently on beta for server components, so I prefer to wait for their 3.0 release.

If you really require internationalization now, you can refer to the next-intl documentation or the official documentation on internationalization in Next.js.

Btw, I noticed that for our project, when we switch languages from English to Chinese, the length of the content changes, requiring us to adjust the CSS, such as the line height, for each resume. We may need to find a convenient solution for this.

thanks for your answer， let me close it


PHPCS-24 Added a sniff to force the sorting T_USE, T_CONST, T_VARIABLE, T_FUNCTION

Please delete feature branch after merge.

Updated. Thx @migo315 .


JPA Autoconfiguration ignores existing instance of EventBus

When an EventBus is explicitly configured in the Spring ApplicationContext, the JPA Autoconfiguration should not configure an EventStorage Engine.
In cases where people explicitly disable Event Sourcing, this generates unnecessary confusion and requires a table to be present for the StorageEngine, even though it remains unused.

Closing this issue as it has been resolved in pull request #1403


Performance of pow function

Hi Mike,
I've noticed the performance of the pow() function to be significantly slower in Big.js compared to BigNumber.js.
A calculation such as new Big("1.00096153846153846154").pow(2500) can take up to 6s to calculate vs new BigNumber("1.00096153846153846154").pow(2500) which will return in under a second.
Is this expected behaviour in Big.js?

Yes.
It will be slower because the result of that calculation is over 50,000 digits long and a Big stores a number as an array of single digits e.g. [1, 2, 3, 4, 5, 6, 7, 8, 9] while a BigNumber stores the same number as [123456789] which makes multiplication much quicker when the number of digits gets high because there are far fewer array elements to operate on.
For normal sized numbers though big.js is often faster and the code is simpler and smaller because of how the values are stored.
Also, with BigNumbers you can set the POW_PRECISION to limit the number of significant digits calculated which canl make the operation faster still.

Thanks for the explanation.


css variables should use !default so they can be overridden

E.g. at least in _variables.css.scss and _gradients.css.scss
Anywhere you define variables that are meant to support being overridden please tack !default on the end so they can actually be overridden.

I've never heard of !default before, could you expand on this?

http://sass-lang.com/docs/yardoc/file.SASS_REFERENCE.html#variable_defaults_
Since your 'globalish' variables are buried I set the values I want before including the default active_admin sccs:
$primary-color: $color-sky-blue;

// Active Admin CSS Styles
@import "active_admin/mixins";
@import "active_admin/base";

But this only works if you use the !default syntax
Other option is to pull all of your globals into a separate file so it can be included first, then overrides set, then the rest of active_admin scss:
// Active Admin CSS Styles
@import "active_admin/variables";

$primary-color: $color-sky-blue;

@import "active_admin/mixins";
@import "active_admin/base";


libreoffice-still: Disable more failing tests

NOTE
Haven't tested this build locally, I'm using ofborg for that. In the meantime, libreoffice-fresh builds, use it.
Description of changes
libreoffice-still fails to build now:
https://hydra.nixos.org/log/pyc6gmsvgb998flailn189znkc63nlfz-libreoffice-<IP_ADDRESS>.drv
This is an attempt to fix it.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 225116 run on x86_64-linux 1


K8up is unable to restore a single file snapshot

Description
I've created an instance from ghcr.io/k8up-io/k8up:v2 and set the followings:
  - args:
    - -restore
    - -restoreSnap
    - 207d6a74
    - -restoreType
    - folder

The Snapshot id points to a single .sql file which has been created via K8up application aware backup from a PostgreSQL instance. It fails with with the following error: panic: runtime error: slice bounds out of range [:3] with capacity 2.
The snapshot above can be restored manually via the Restic binary packed in the same image.
Additional Context
My goal was to restore to a RWO volume thus used podAffinity spec to get my pod scheduled beside the target pod.
Tried with the operator as well, pointing to a RWX volume, it fails too.
Logs
1.6745719441801126e+09    INFO    k8up.restic    initializing
1.674571944180123e+09    INFO    k8up.restic    setting up a signal handler
1.6745719441801894e+09    INFO    k8up.restic.restic    using the following restic options    {"options": [""]}
1.6745719441802113e+09    INFO    k8up.restic.restic.RepoInit.command    restic command    {"path": "/usr/local/bin/restic", "args": ["init", "--option", ""]}
1.6745719441802182e+09    INFO    k8up.restic.restic.RepoInit.command    Defining RESTIC_PROGRESS_FPS    {"frequency": 0.016666666666666666}
1.6745719442500741e+09    INFO    k8up.restic.restic.unlock    unlocking repository    {"all": false}
1.6745719442501287e+09    INFO    k8up.restic.restic.unlock.command    restic command    {"path": "/usr/local/bin/restic", "args": ["unlock", "--option", ""]}
1.6745719442501488e+09    INFO    k8up.restic.restic.unlock.command    Defining RESTIC_PROGRESS_FPS    {"frequency": 0.016666666666666666}
1.6745719448515744e+09    INFO    k8up.restic.restic.unlock.restic.stderr    successfully removed locks
1.6745719448527825e+09    INFO    k8up.restic.restic.snapshots    getting list of snapshots
1.6745719448528092e+09    INFO    k8up.restic.restic.snapshots.command    restic command    {"path": "/usr/local/bin/restic", "args": ["snapshots", "--option", "", "--json"]}
1.6745719448528461e+09    INFO    k8up.restic.restic.snapshots.command    Defining RESTIC_PROGRESS_FPS    {"frequency": 0.016666666666666666}
1.674571945789223e+09    INFO    k8up.restic.restic.restore    restore initialised
1.674571945789251e+09    INFO    k8up.restic.restic.restore    loading all snapshots from repositoy
1.6745719457892542e+09    INFO    k8up.restic.restic.snapshots    getting list of snapshots
1.6745719457892606e+09    INFO    k8up.restic.restic.snapshots.command    restic command    {"path": "/usr/local/bin/restic", "args": ["snapshots", "--option", "", "--json"]}
1.674571945789275e+09    INFO    k8up.restic.restic.snapshots.command    Defining RESTIC_PROGRESS_FPS    {"frequency": 0.016666666666666666}
panic: runtime error: slice bounds out of range [:3] with capacity 2

goroutine 1 [running]:
github.com/k8up-io/k8up/v2/restic/cli.(*Restic).linkRestorePaths(0x14a05f9?, {{0xc000054700, 0x40}, {0x230ca086, 0xedb60a0fa, 0x0}, {0xc0000546c0, 0x40}, {0xc000698380, 0x1, ...}, ...}, ...)
    /home/runner/work/k8up/k8up/restic/cli/restore.go:200 +0x265
github.com/k8up-io/k8up/v2/restic/cli.(*Restic).folderRestore(0xc000056c80, {0xc00004806c, 0x8}, {{0xc000054700, 0x40}, {0x230ca086, 0xedb60a0fa, 0x0}, {0xc0000546c0, 0x40}, ...}, ...)
    /home/runner/work/k8up/k8up/restic/cli/restore.go:144 +0xf8
github.com/k8up-io/k8up/v2/restic/cli.(*Restic).Restore(0xc000056c80, {0x7fff01c76681, 0x8}, {{0x7fff01c76697, 0x6}, {0xc00004806c, 0x8}, {0x0, 0x0}, 0x0, ...}, ...)
    /home/runner/work/k8up/k8up/restic/cli/restore.go:89 +0x4aa
github.com/k8up-io/k8up/v2/cmd/restic.doRestore(0x1?)
    /home/runner/work/k8up/k8up/cmd/restic/main.go:189 +0x1c7
github.com/k8up-io/k8up/v2/cmd/restic.doNonBackupTasks(0x0?)
    /home/runner/work/k8up/k8up/cmd/restic/main.go:159 +0x45
github.com/k8up-io/k8up/v2/cmd/restic.run({0x1a56560, 0xc00004c030}, 0x1a58f50?, {{0x1a58f50?, 0xc000200c60?}, 0x1a452b0?})
    /home/runner/work/k8up/k8up/cmd/restic/main.go:118 +0x92
github.com/k8up-io/k8up/v2/cmd/restic.resticMain(0xc0006991c0)
    /home/runner/work/k8up/k8up/cmd/restic/main.go:105 +0x25e
github.com/urfave/cli/v2.(*Command).Run(0x25c6f40, 0xc0006991c0, {0xc0000a9500, 0x6, 0x6})
    /home/runner/go/pkg/mod/github.com/urfave/cli/v2@v2.20.2/command.go:271 +0xa8f
github.com/urfave/cli/v2.(*Command).Run(0xc00031e140, 0xc000698b80, {0xc00003e070, 0x7, 0x7})
    /home/runner/go/pkg/mod/github.com/urfave/cli/v2@v2.20.2/command.go:264 +0xced
github.com/urfave/cli/v2.(*App).RunContext(0xc0001e88c0, {0x1a56560?, 0xc00004c030}, {0xc00003e070, 0x7, 0x7})
    /home/runner/go/pkg/mod/github.com/urfave/cli/v2@v2.20.2/app.go:305 +0x658
github.com/urfave/cli/v2.(*App).Run(...)
    /home/runner/go/pkg/mod/github.com/urfave/cli/v2@v2.20.2/app.go:282
main.main()
    /home/runner/work/k8up/k8up/cmd/k8up/main.go:28 +0x45

Expected Behavior
K8up should be able to restore a single file snapshot.
Steps To Reproduce
spec:
  affinity:
    podAffinity:
      requiredDuringSchedulingIgnoredDuringExecution:
      - labelSelector:
         matchExpressions:
         - key: statefulset.kubernetes.io/pod-name
           operator: In
           values:
            - podname
        topologyKey: kubernetes.io/hostname
  containers:
  - args:
    - -restore
    - -restoreSnap
    - 207d6a74
    - -restoreType
    - folder
    command:
    - /usr/local/bin/k8up
    - restic

Version of K8up
v2.5
Version of Kubernetes
v1.24.6
Distribution of Kubernetes
AKS

Hi @bfeher-kx
Thanks for reporting this! I thought we're already tracking that, but was not able to find an issue for that.

Hi @Kidswiss
We have checked the code and it seems its expecting 3 sections for the path. In this case the last section is missing as we don't have a directory.

hi @bfeher-kx
Yes that is correct, the current restore code to PVC expects that the backup also came from a PVC and not from a pre-backup command. So it's a use-case that has not yet been implemented as such.
The main issue is, that in most cases a dump file in a PVC isn't of much use to someone. They then still have to get the dump out of the PVC and into the DB instance somehow. But it would make for a better user-experience than just crashing.
There's the idea of restoreCommands: https://github.com/k8up-io/k8up/issues/637

BTW: just found in the docs that we have a note about this case: https://k8up.io/k8up/2.5/how-tos/restore.html#_restore_from_s3_to_pvc
I knew we had written down something about this exact issue.

Hi @Kidswiss
Does that mean we should not wait for a fix? Is there a reason why its done like that?

Hi @bfeher-kx
I think to implement a fix would be reasonable.
As to your second question, basically what I wrote above:

The main issue is, that in most cases a dump file in a PVC isn't of much use to someone. They then still have to get the dump out of the PVC and into the DB instance somehow.

That's why we have the descriptions how to do manual restores.


Recommend to use the name argument in the pytest.fixture decorator

Add a tip about using the name argument in the pytest.fixture decorator to avoid naming conflicts.

Can you add a description to the PR, I know the title is self explaining, but let's be consistent in adding descriptions

I have the filling that this convention could be wider, or more generic.
Something like "always set the name argument" and convention towards fixture function naming "always _fixture suffix" and eventually a "scope" prefix
What will happen if we have the following:
conftest.py
@pytest.fixture
def app():
  yield "app"

test_charm.py
@pytest.fixture(name="app")
def app_fixture(ops_test):
   yield ops_test.model.app

Can you add a description to the PR, I know the title is self explaining, but let's be consistent in adding descriptions

Added, thanks!

I have the filling that this convention could be wider, or more generic. Something like "always set the name argument" and convention towards fixture function naming "always _fixture suffix" and eventually a "scope" prefix
What will happen if we have the following: conftest.py
@pytest.fixture
def app():
  yield "app"

test_charm.py
@pytest.fixture(name="app")
def app_fixture(ops_test):
   yield ops_test.model.app

def test_charm(app): None
 ...

I actually view this PR as a recommendation or tip, the original form of fixture still have its benefit of being short and easy to write.
In terms of two fixtures have the same name, pytest will choice one without raising an error. This is a known issue in pytest. pytest-dev/pytest#3966

I'm with Weii here, this standard is more to prevent needing to have clashing names with function and arguments which is a bit messy in Python. When possible, name="app" shouldn't be used

I have the filling that this convention could be wider, or more generic. Something like "always set the name argument" and convention towards fixture function naming "always _fixture suffix" and eventually a "scope" prefix
What will happen if we have the following: conftest.py
@pytest.fixture
def app():
  yield "app"

test_charm.py
@pytest.fixture(name="app")
def app_fixture(ops_test):
   yield ops_test.model.app

def test_charm(app): None
 ...

I actually view this PR as a recommendation or tip, the original form of fixture still have its benefit of being short and easy to write.
In terms of two fixtures have the same name, pytest will choice one without raising an error. This is a known issue in pytest. pytest-dev/pytest#3966

I'm with Weii here, this standard is more to prevent needing to have clashing names with function and arguments which is a bit messy in Python. When there is no clash, name="app" shouldn't be used since it has the drawback of being longer

Ok 👌


[FEATURE] Metrics/Model Confidence Evaluation

It would be useful to have some more quantitative measurements of the model's confidence or a comparable version of AlphaFold (i)pTM metrics to evaluate the model's confidence. I saw that the Colab notebook has a "pred_error" in the visualization but would be useful for me to have a number to compare against as well.

Hi Deniz,
Thanks for your interest in ImmuneBuilder. Error estimation in the ImmuneBuilder models is measured in a different way to AF2.  So don't expect to be able to directly compare them. If you want a number, you can get it from the b-factor column of the output PDB files.
Hope this helps!!
All the best,
Brennan


It seems that FSDP's cpu_ram_efficient_loading setting doesn't work?

I run my FSDP scripts with the following configs:
compute_environment: LOCAL_MACHINE
debug: false
distributed_type: FSDP
downcast_bf16: 'no'
fsdp_config:
  fsdp_auto_wrap_policy: TRANSFORMER_BASED_WRAP
  fsdp_backward_prefetch_policy: BACKWARD_PRE
  fsdp_cpu_ram_efficient_loading: true
  fsdp_forward_prefetch: false
  fsdp_offload_params: false
  fsdp_sharding_strategy: 1
  fsdp_state_dict_type: FULL_STATE_DICT
  fsdp_sync_module_states: true
  fsdp_use_orig_params: false
machine_rank: 0
main_training_function: main
mixed_precision: 'bf16'
num_machines: 1
num_processes: 4
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

But it seems that there are still 4 processes loading the full model checkpoint and transferring data from CPU to GPU?  Does fsdp_cpu_ram_efficient_loading: true still work?

Seems that the fsdp_cpu_ram_efficient_loading config is loaded through
https://github.com/huggingface/accelerate/blob/55747318a0f47cdfbc281e11269ba96214e4092d/src/accelerate/utils/launch.py#L106
and finally transferred into torch.distributed.run API:
https://github.com/huggingface/accelerate/blob/55747318a0f47cdfbc281e11269ba96214e4092d/src/accelerate/commands/launch.py#L654
Am I getting this right?
But I couldn't find any handling code for fsdp_cpu_ram_efficient_loading in Pytorch, does that mean the fsdp_cpu_ram_efficient_loading can't change anything?
Thanks for your reply.

Hello, this only works with Transformers Models as is mentioned when setting that config parameter. It is used in Transformers when doing from_pretrained. See these:
https://github.com/huggingface/transformers/blob/c5be38cd27bee92be73c73ba09aec8bedf841423/src/transformers/modeling_utils.py#L131
https://github.com/huggingface/transformers/blob/c5be38cd27bee92be73c73ba09aec8bedf841423/src/transformers/modeling_utils.py#L476-L483
https://github.com/huggingface/transformers/blob/c5be38cd27bee92be73c73ba09aec8bedf841423/src/transformers/modeling_utils.py#L2598-L2599
https://github.com/huggingface/transformers/blob/c5be38cd27bee92be73c73ba09aec8bedf841423/src/transformers/modeling_utils.py#L3907-L3935

But it seems that there are still 4 processes loading the full model checkpoint and transferring data from CPU to GPU?

On non-main processes, the weights are torch.empty weights.
Hope this resolves your question!


[Documentation] Node tainting and tolerations

Docs to include node tainting on master and worker nodes. Also include tolerations.
--node-taint value                         (agent/node) Registering kubelet with set of taints

Just to chime in here with a very lowly & recent new user's experiences, the docs made me think at first that I needed a multi-node cluster. I was surprised when I started a single node k3s server that I could run workloads on it. I was expecting there to be some tainting/tolerations issues that would have prevented this.
I haven't tried moving my k3s's towards multi-node clusters yet but I'm not sure whether taint/tolerations are going to kick in & remove workloads from the first node/server or not. And I'm not sure what I should be looking at to understand these systems.
Please forgive me if these comments to not belong in this ticket. It seemed similar & I was surprised I made it as far as I did (having only one node), & I'm not sure how to view or understand what goes on here. Maybe nothing will happen as my cluster goes multi-node. That would actually be fine for my very preliminary low-use use case, but I'd like to better understand this topic & what if anything k3s does or doesn't do here & how to understand it. Thanks @ShylajaDevadiga &al.

Need to review this with @ShylajaDevadiga determine what we need clearly in the docs.

@ShylajaDevadiga can you post what you are looking for? We call out the option in docs - do you want it expaned on? What is needed?
Reference:
https://rancher.com/docs/k3s/latest/en/installation/install-options/server-config/

and -
https://rancher.com/docs/k3s/latest/en/installation/install-options/agent-config/#node-labels-and-taints-for-agents

@rektide this is expected behavior. A master is an agent as well, so you can run workloads on it. Thats what you are seeing. To avoid being able to run workloads you can apply taints, but agent components, kubelet kubeproxy will still be running on it.

Based on discussion sounds like what we have should be sufficient in the documentation. If there are objections we can reopen the issue.


the tags input is autofocus.can you add a param to setting when don't need focus this input

Hi, I've published a new version which makes this configurable. Also, the latest version of ReactTags has a breaking changes in the API (from 0.x.x version), so make sure you read the usage section carefully - https://github.com/prakhar1989/react-tags#usage


Chapter 14: MultiRNNCell

This code fails in the latest tf
multi_layer_cell = tf.nn.rnn_cell.MultiRNNCell(cells)
outputs, states = tf.nn.dynamic_rnn(multi_layer_cell, X, dtype=tf.float32)

Error: "TypeError: cell must be an instance of RNNCell"

Oops, my bad, it was an issue with DeviceCellWrapper


Add windows timestamp shortcut like F5 in notepad

Description of the new feature / enhancement
Add Windows timestamp shortcut like F5 in Notepad.
Scenario when this would be used?
Is pastes a configurable timestamp like (YYYY-MM-DD HH-MI-SS.MS). Suggested shortcut: "Win + Shift + .(Dot)" / "Alt + Shift + =".
Supporting information
No response

This seems too small to be its own utility, however it could get wrapped into a #671 or smart #1758. I will say PowerToys Run has a plugin to get the current time by typing now. Would improvements to that plugin be a solution to this issue? /needinfo


perf(semantic): recycle unresolved references hash maps

Closes #4169.
Reduce allocations while building Semantic by recycling hash maps used for tracking unresolved references, rather than creating a new one for every scope.

#4213  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite

@lucab FYI this is what I was getting at when I suggested re-using hash maps. Sorry I explained it so badly!


[Improvement] Should show an understandable error message to non technical users when try to add user without providing email

Description:
[Improvement] Should show an understandable error message to non-technical users when try to add user without providing email
Suggested Labels:
Priority/Low
Severity/Minor
Suggested Assignees:
Affected Product Version:
API 210
OS, DB, other environment details and versions:
Steps to reproduce:
Go to the management console https://openbanking.staging.wso2.com:9443/carbon
Go to "Users and Roles" --> Add -->Add New User
Fill the details on step 1 as below
User Name Asanka Password Asanka Password Repeat Asanka
4 Select the Roles and click finish

Issue:
Getting issue as in attached image which is not understandable

Related Issues:
https://github.com/wso2/financial-open-banking/issues/328

closing since an answer is provided


Why is your idea so similar to masked attention mask transformer https://arxiv.org/abs/2112.01527 .

Why is your idea so similar to masked attention mask transformer https://arxiv.org/abs/2112.01527

.

Since this work is motivated by Max-DeepLab, MaskFormer, Mask2Former, and PanopticFCN, this line of research in segmentation. We demonstrate a similar double-stream structure can work well (seems very well) in depth estimation. Based on this, it is interesting to associate these two scene understanding tasks in a unified structure.


[Package request ] s4cmd

Package request for s4cmd: https://github.com/bloomreach/s4cmd

looks like it would be a simple package using buildPythonApplication,
as buildInputs:  boto3 and pytz from pythonPackage has to be passed.

Thanks for your request. If you like to get a package in, then the best approach would be to roll up your sleeves and open a PR! :) People on this tracker and the NixOS IRC channel are typically quite willing to help out (as shown here by @Mic92), and especially when discussing a PR.


Camera awesome package not working

I found this package last week, and I used it in my project. It was working fine. But, in this week it stopped working. I got following errors for it.
Launching lib\main.dart on sdk gphone64 x86 64 in debug mode...
e: C:\Users\Chmain\AppData\Roaming\Pub\Cache\hosted\pub.dev\camerawesome-1.2.0\android\src\main\kotlin\com\apparence\camerawesome\cameraX\CameraPermissions.kt: (175, 22): Not enough information to infer type variable T
e: C:\Users\Chmain\AppData\Roaming\Pub\Cache\hosted\pub.dev\camerawesome-1.2.0\android\src\main\kotlin\com\apparence\camerawesome\cameraX\CameraPermissions.kt: (175, 41): Cannot infer a type for this parameter. Please specify it explicitly.
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':camerawesome:compileDebugKotlin'.

Compilation error. See log for more details

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 40s
Exception: Gradle task assembleDebug failed with exit code 1
Exited (sigterm)

I think it's because we updated kotlin and gradle version.
It might be improved in a future PR, but in the meantime you can resolve your issue by updating them as well.
See #231

Upgrade in build.gradle kotlin version to ext.kotlin_version = '1.7.10'
And classpath 'com.android.tools.build:gradle:7.2.0'


Schema at Microsoft.Insights/components/analyticsItems@2015-05-01 is impossible

Bicep version
Bicep CLI version 0.4.1124 (66c84c8ee5)
Describe the bug
The schema to save a kusto query as a function can not work. A working payload would look like this:
/sub/123/rg/abc/providers/Microsoft.Insights/components/insights_name/analyticsItems/item
{
    "Name": "query",
    "Content": "dependencies",
    "Properties": {
        "functionAlias": "query_alias"
    },
    "Scope": "Shared",
    "Type": "Function"
}

Issue 1: "item" is the hardcoded required name. This prevents it from working in a loop in bicep, because the name property MUST reference the copyIndex() loop.  )

Issue 2
The Type and Name properties conflict with name and type. Bicep will request Name to be of correct length: "name/subname", but that is not applicable for "Name"; only for "name".
"Type" is impossible to pass as well; it does not match any schema, etc. When I omit "Type" property, the request fails as well:

The request content was invalid and could not be deserialized: 'Could not find member 'Content' on object of type 'TemplateResource'. Path 'properties.template.resources[0].Content

To Reproduce
resource appInsights 'Microsoft.Insights/components@2020-02-02' = {
  name: 'app_insights_for_app'
  location: resourceGroup().location
  kind: 'web'
  properties: {
    Application_Type: 'web'
    publicNetworkAccessForIngestion: 'Enabled'
    publicNetworkAccessForQuery: 'Enabled'
    IngestionMode: 'LogAnalytics'
    RetentionInDays: 90
    Flow_Type: 'Bluefield'
    WorkspaceResourceId: '/subscriptions/123/resourcegroups/123/providers/microsoft.operationalinsights/workspaces/log_analytics_workspace_name'
  }

  resource insights_query_function 'analyticsItems@2015-05-01' = {
    name: 'item' // hardcoded, must be "item"; this is the ARM resource name
    Name: 'important_function' // This is the user-defined name of the item;
      // Intellisense error after bicep build: a member with the name "Name" already exists;
      // Additionally: I got error with Name property because "Name" needs equal parts to the type, as in: name/subname.
    Content: 'dependencies' // For some reason "Content" also yields intellisense errors after build.
      // Template validation failed: Could not find member 'Content' on object of type 'TemplateResource'. Path 'resources[0].Content', line 17, position 16.arm-template (validation)
      // This is strange because Content is explicitly mentioned as required in the schema for both arm and bicep: https://docs.microsoft.com/en-us/azure/templates/microsoft.insights/components/analyticsitems?tabs=json
    Scope: 'shared'
    Properties: {
      functionAlias: 'important_function'
    }
  }
}
// {
//   "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
//   "contentVersion": "<IP_ADDRESS>",
//   "metadata": {
//     "_generator": {
//       "name": "bicep",
//       "version": "0.4.1124.51302",
//       "templateHash": "6262658167201736790"
//     }
//   },
//   "resources": [
//     {
//       "type": "microsoft.insights/components/analyticsItems",
//       "apiVersion": "2015-05-01",
//       "name": "[format('{0}/{1}', 'app_insights_for_app', 'item')]",
//       "Name": "important_function",
//       "Content": "dependencies",
//       "Scope": "shared",
//       "Properties": {
//         "functionAlias": "important_function"
//       },
//       "dependsOn": [
//         "[resourceId('Microsoft.Insights/components', 'app_insights_for_app')]"
//       ]
//     },
//     {
//       "type": "Microsoft.Insights/components",
//       "apiVersion": "2020-02-02",
//       "name": "app_insights_for_app",
//       "location": "[resourceGroup().location]",
//       "kind": "web",
//       "properties": {
//         "Application_Type": "web",
//         "publicNetworkAccessForIngestion": "Enabled",
//         "publicNetworkAccessForQuery": "Enabled",
//         "IngestionMode": "LogAnalytics",
//         "RetentionInDays": 90,
//         "Flow_Type": "Bluefield",
//         "WorkspaceResourceId": "/subscriptions/123/resourcegroups/123/providers/microsoft.operationalinsights/workspaces/log_analytics_workspace_name"
//       }
//     }
//   ]
// }

Additional context
Add any other context about the problem here.

Same problem here, using latest version of az and bicep.
resource myFunction 'microsoft.insights/components/analyticsItems@2015-05-01' = {
name: 'item'  // must be set to item
Name: 'Foo'   // There is another Name property (?) with uppercase?
parent: azAppInsights
Scope: 'user'
Id: 'b401c7de-0c50-4c07-91c4-e975694fdd32'
Content: loadTextContent('some_function.kql')   // throws the error
Properties: {
functionAlias: 'some_function'
}
}
{"code": "InvalidRequestContent", "message": "The request content was invalid and could not be deserialized: 'Could not find member 'Content' on object of type 'TemplateResource'. Path 'properties.template.resources[0].Content', line 21, position 16.'."}

I'm having the same issue. Were you able to find any workaround to deploy functions using ARM or REST calls?

You should be able to use the mentioned ARM payload in an az rest command.

In case somebody else will hit this issue, there is a workaround: for log analytics workspace-based AppInsights instances, you can switch to Log Analytics functions, which have proper Bicep support and can accept parameters as well.


Provide a simple way of removing scroll effects

Use case
I don't what my ListView to stretch the content on Android.
Proposal
Right now to achieve this, I need to wrap the ListView in a ScrollConfiguration, and provide a custom override class to change the behavior.
This is way too much clutter for a "configuration" of a very basic behavior. I shouldn't have to create descendants classes of the child of a configuration class to just change the behavior of a ListView.
Please provide a much simpler alternative to the following code :
class NormalScrollBehavior extends ScrollBehavior {
  @override
  Widget buildOverscrollIndicator(BuildContext context, Widget child, ScrollableDetails details) => child;
}

class MyListView extends StatelessWidget {
  const MyListView({super.key});

  @override
  Widget build(BuildContext context) {
    return ScrollConfiguration(
      behavior: NormalScrollBehavior(),
      child: ListView(
        children: [
          // children
        ],
      ),
    );
  }
}

Why not have something as simple as the physics argument to the ListView constructor ? This makes things infinitely more readable.
class MyListView extends StatelessWidget {
  const MyListView({super.key});

  @override
  Widget build(BuildContext context) {
    return ListView(
      behavior: NormalScrollBehavior(),
      children: [
        // children
      ],
    );
  }
}

Edit : NestedScrollView already has this.

Hi @TheCarpetMerchant you do not need to create a subclass NormalScrollBehavior, ScrollBehavior and its subclasses MaterialScrollBehavior and CupertinoScrollBehavior already have a convenience method for disabling different effects. You can also set the scroll behavior for the entire app rather than adding ScrollConfigurations above individual Scrollables.
class MyListView extends StatelessWidget {
  const MyListView({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      behavior: MaterialScrollBehavior().copyWith(overscroll: false),
      home: ListView(
        children: [
          // children
        ],
      ),
    );
  }
}

Edit : NestedScrollView already has this.

This is because NestedScrollView overrides some aspects of the inherited ScrollBehavior by default internally, so it is exposed on NestedScrollView to give users control that would not be possible with ScrollConfiguration. ListView and others do not override the behavior internally so do not have it exposed.


Using translate_url on 404 page

If you try to call translate_url in your 404 template this results in 500 error.
The problem comes from this line https://github.com/stephenmcd/mezzanine/blob/master/mezzanine/core/templatetags/mezzanine_tags.py#L678
Unfortunately I am not sure which ones is the best solutions but here are some options:

fail silently and return empty string
fail silently and return language url pointing to the root of the website e.g. /en/
accept a default URL to be translated in such cases e.g. {% translate_url 'de' '/de/my-page' %}
something else?

It should return the URL to redirect to for the language selector right? If that's the case, homepage makes sense, no point in redirecting back to a translated 404.

It should return the URL for a different language version of the current page.
E.g. if you are on page '/en/about-us/' it should return something like '/de/about-us/'

It should return the URL for a different language version of the current page.

Um, but we're talking about 404s - the page doesn't exist. Please re-read my first comment.

Surry, but I got a bit confused so I will try to explain my self better:
Using {% translate_url  %} we normally get URLs like '//about-us/'
However if the user tries to access /non-existing-url/ then {% translate_url  %} will raise an exception - which for me is a bug because the 404 page turns into 500.
So I think a better behaviour will be something like:

fails silently and returns empty string e.g. ''
fails silently and return pointer to the URL root e.g. '/en/'
accept an extra argument that is used as a fallback URL e.g.
{% translate_url 'de' '/de/my-page' %} which returns '/de/my-page' only if the current URL does not exists.

Can't reproduce this - 404 should resolve fine to the catch-all page urlpattern, which is what happens for me.


The AuthorizeCommand attribute requires projects to take unnecessary dependencies

Sequin, Newtonsoft.Json and Microsoft.Owin should not be needed in order to flag a command as required authentication.  E.g. if commands live in a domain project with no concept of the web layer we should not be forced to take these packages as dependencies.


Add basic support for auctex package

Hi, I like this theme. Thanks. I took a shot at fixing some of the main issues with the auctex faces.
Before:

After:

BeforeAfter

🚢 Shipped with MELPA package release version 🏷 0.5.0.


fix: use resource_bundles for privacy manifest

This is done to avoid getting build issues with multiple commands trying to produce xcprivacy file in application bundle.
For a more detailed explanation see: https://github.com/google/promises/pull/226#discussion_r1447871477

Hi @maikelvdh, thank you for your contribution. I am addressing this in PR #584 for the Location and LocationAndBeacons pods.


Research and Design for PV/PVC

In GitLab by @luotian-github on Oct 10, 2017, 15:20
updata service and node api

In GitLab by @yhua123 on Nov 8, 2019, 07:39
10.30 review meeting

In GitLab by @yhua123 on Nov 8, 2019, 07:39
please add pv/pvc design

In GitLab by @yhua123 on Nov 8, 2019, 07:39
open #614 follow this issue

In GitLab by @sokril on Nov 8, 2019, 08:12
closed

In GitLab by @sokril on Nov 8, 2019, 08:12
assigned to @luotian-github


Fa data protection

Locally host fontawesome files instead of using their online-service, to comply with dsgvo / for data protection

deploy preview was ok, let's try this in production :-)


Minor website fix

We have more and more contributions to the content and now there is a pagination for custodians, with only 10 per page:
.
Could you change this number to 50? Is there any way to show the taxonomy other than buttons?

I like Wikipedia alphabetical taxonomy - not critical now, but we will have much more custodians in the future, so this maybe the way to go. and this is our general reference for this project.

ok


Storing raw JSON

I've been using this library with great success, but up until now I've been serializing Java POJOs into JSON and storing those.
However now I want to cache JSON API responses directly in a jsonb field in a column.
I have the json payload in a String variable and I've declared the property type like so:
    @Type(type = "Json", parameters = {
            @Parameter(name = "type", value = "Object")
    })
    protected Object payload;

The problem is that my string-containing-json is being serialized as a Json String in the column, rather than a Json Object. This makes querying and such against the json structure not possible.
Is it possible to workaround this with the library?

If you manipulate json with its serialized form in your code, you dont need the lib to do that. Just declare a @Column(columnDefinition = "jsonb") and hibernate should transfer the string directly and pg will structure it as json.

Perfect, thanks.


Adds Mindtagger integration for error analysis

which downloads Mindtagger (in latest DeepDive release), then creates a
task, and launches it in a user-specific port, and displays the GUI in
an iframe.

rels.open_mindtagger(num_sample=20, width='100%', height=600)
rels.launch_mindtagger(task_name, item_generator, ...)
rels.get_mindtagger_tags()

As long as the GUI is running in the background, users can freely
add/update the tags, and get the tags to postprocess them from the rest
of the notebook cells.
Also updates early notebook cells to skip NLP markup steps by default if
there's cached result, and show some examples to give a sense of what
the data looks like.
Known issue: Mindtagger UI is not responsive enough for small screens,
so needs some touches to fit nicely into typical notebook widths and
heights.  However, this can be done independently from DDlite.
Here's a screenshot:

Fixes #16

SO AWESOME!!!
On Fri, Mar 11, 2016 at 4:02 AM Jaeho Shin<EMAIL_ADDRESS>wrote:

which downloads Mindtagger (in latest DeepDive release), then creates a
task, and launches it in a user-specific port, and displays the GUI in
an iframe.

rels.open_mindtagger(num_sample=20, width='100%', height=600)
rels.launch_mindtagger(task_name, item_generator, ...)
rels.get_mindtagger_tags()

As long as the GUI is running in the background, users can freely
add/update the tags, and get the tags to postprocess them from the rest
of the notebook cells.
Also updates early notebook cells to skip NLP markup steps by default if
there's cached result, and show some examples to give a sense of what
the data looks like.
Known issue: Mindtagger UI is not responsive enough for small screens,
so needs some touches to fit nicely into typical notebook widths and
heights. However, this can be done independently from DDlite.
Here's a screenshot:
[image: 2016-03-11 03 57 32]
https://cloud.githubusercontent.com/assets/53634/13701625/057d7dfc-e73e-11e5-8ebc-661c0985e017.png
You can view, comment on, or merge this pull request online at:
https://github.com/HazyResearch/ddlite/pull/37
Commit Summary

Adds Mindtagger integration for error analysis (fixes #16)

File Changes

M .gitignore
https://github.com/HazyResearch/ddlite/pull/37/files#diff-0 (6)
M DeepDiveLite.ipynb
https://github.com/HazyResearch/ddlite/pull/37/files#diff-1 (249)
M ddlite.py
https://github.com/HazyResearch/ddlite/pull/37/files#diff-2 (144)

Patch Links:

https://github.com/HazyResearch/ddlite/pull/37.patch
https://github.com/HazyResearch/ddlite/pull/37.diff

—
Reply to this email directly or view it on GitHub
https://github.com/HazyResearch/ddlite/pull/37.

This rocks. I'll play with it in the new example

@ajratner @netj
Looks like the mindtagger task for evaluating Relations is set up in the Entities class. Give me a few to make it play nice with the Extractions class

@netj Cleaned up the integration and the notebook a bit in the latest push to this branch, so everything should work smoothly (no more "it works on my machine"). I think there's an issue interfacing between ddlite's tagged indexes and the ones in MindTagger. See Relations.get_mindtagger_items(). For the example you gave, ddlite gives the index of "BMPR-II" with e1_idxs and the indices of "pulmonary hypertension" with e2_idxs. But when e2_idxs are loaded into MindTagger, they get translated to "BMPR-II" and "not". Any guess as to what's going on?

@henryre Thanks for restructuring this!  Looks so much better :)
There apparently was a bug in the MT template I was generating and made it explicit that we're laying down the indexes as JSON arrays, which now fixes the highlights as below.

@netj Great! @ajratner I'll get Entities to work nicely then this looks good to go

LGTM now. @ajratner all you. We should add utilities for analyzing precision, recall, etc. once we set up a good framework for doing that with ddlite.

Working (on my Mac)!  Thanks @netj!!
One very minor thing: I had issues for a second because I was loading the pickled dump of the parsed sentences from a different branch... this issue should quickly go away, but just noting in case anyone else runs across this
Merging now, excited to start using!

I had the same issue, but was hoping it wouldn't corrupt systems worldwide based on the ... exclusivity of our user base. Easy fix is to give unique names for the sentence dumps we're doing in the examples (should have already been doing this)

Nbd either way.  Maintaining brand exclusivity trumps all
On Sat, Mar 12, 2016 at 9:09 PM Henry Ehrenberg<EMAIL_ADDRESS>wrote:

I had the same issue, but was hoping it wouldn't corrupt systems worldwide
based on the ... exclusivity of our user base. Easy fix is to give unique
names for the sentence dumps we're doing in the examples (should have
already been doing this)
—
Reply to this email directly or view it on GitHub
https://github.com/HazyResearch/ddlite/pull/37#issuecomment-195879668.


websocket upgrade on a per route basis

Title: websocket upgrades cannot be done on a per route basis
Description:
https://github.com/envoyproxy/envoy/pull/4745 removed use_websocket from the route config. This means that now websocket upgrades are now only configurable on a http conn man basis. In order to preserve functionality envoy would need to allow upgrades on a per route basis.

@alyssawilk are you able to take this one? IMO we should have capability here to what we had previously?

@alyssawilk @mattklein123 opening this issue to track what I was talking to both of you today. For Lyft, the current state is ok. However, I am not sure if anyone else in the community was dependent on this feature. This deprecation removed existing functionality, rather than just deprecating the field.

Happy to add it if we think it's worthwhile.

@alyssawilk yeah I think definitely useful. Some folks do not want to expose WS upgrades globally, but want to allow it on a per-route basis.

Oh man, so I managed to slog through splitting the hcm proto in to multiple files to avoid dependency loop issues hcm->rcs->route->hcm but looking at the similar problem with the includes directory I think we're either have to violate style and do a bunch of pre-declares and file splitting, or we could just say you can enable websocket on a per-hcm basis and override-disable on a per-route basis.
Lyft folks, you OK with this as a workaround or should I keep fighting the good fight in include/?


K8s fixes

Description of Changes
A couple of small fixes that came up from the k8s custom plugin development.

The docker container needs to be built with up-to-date root certificates
The Google Cloud output was missing one more case for supported types. We should consider doing a full audit of this, and falling back to either reflection or json marshalling/unmarshalling.

Please check that the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Add a changelog entry (for non-trivial bug fixes / features)
[ ] CI passes

Log Files
Logs / Second
CPU Average (%)
Memory Average (MB)

1
1000
2.6207426
27.74623

1
5000
5.896688
35.089035

1
10000
9.482955
41.222927

1
50000
36.535244
120.17376

1
100000
62.122044
214.18561

10
100
4.206961
28.407192

10
500
6.8967338
36.89507

10
1000
13.293449
42.12352

10
5000
41.759605
93.243

10
10000
74.58331
186.94316


Origin/default manifest

This replaces  remove local hc and nodepool creation. I took Ian's changes, and fixed a few bugs I saw and also included CLUSTERID generation which is something we need to do until ManifestWork can Merge the spec's.

Incorporate Ian's changes to remove the local HC and NP code.
Removed an extra Pull Secret create that was being done that isn't needed
Add ClusterID support.
Make sure the delete flow for ManifestWork (default flow and InfraOnly still work.

/test pr-image-mirror

/cc @zhujian7 @philipwu08

/test unit-tests
The previous failure is a network issue pulling build data

/lgtm


export function parquet_to_array_schema_and_fields

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

We want to use some low-level APIs to read parquet files by rowgroup (such as build_array_reader), but this function need field: Option<&ParquetField> that can only constructed by parquet_to_array_schema_and_fields.
Describe the solution you'd like

Export function parquet_to_array_schema_and_fields
Describe alternatives you've considered

Additional context

label_issue.py automatically added labels {'parquet'} from #4307


Second notification inside the app besides system notification possible?

Documentation File:
https://github.com/corona-warn-app/cwa-documentation/blob/master/scoping_document.md#exposure-contact-with-an-infected-person
Question:
Is there a way in the exposure notification API that allows the cwa app to notify me inside the app as well so that I get notified regardless of my notification settings? (send all to notification center on iOS for example)

Thanks for your question, @benekuehn .
Do you have something like a modal alert in mind, or are there any other other properties that the notification should have?

@tkowark yeah, guess a modal alert seems fine. But I can also think of directing the user directly to the page she gets to after pressing on the system notification. Just to eliminate the possibility that a user does not notice that she has been exposed

Thanks for the clarification! Unfortunately, I do not (yet) have a concrete answer for this and your other questions regarding the UX.
Can we kindly ask you to bear with us for a couple of more days and check your questions again, once UX documentation and the real implementations are available? We'll give you a short notice in this issue, once this information is available.

@tkowark okay, thanks for letting me know and please do ping me :)

@benekuehn as visible in the ui screens and the current app implementation the risk level is prominently displayed on the app's home screen in case there were any recent exposure events


What is the version/commit that was used to produce the paper results?

For help in troubleshooting getting it up and running.

In private correspondence, Peder Lyngby mentioned:

I am using commit: 65c98702416a60c35a809943a325bbe75de6bc0f (from Dec 14, 2021)

As far as I remember I did not make any significant changes to the code. I had to change a few package names, as it was fixed in the commit 9b70abe7ad8bb90c6b45daf0c8428b5034f503cd (from Dec 18, 2021)
I am running CDVAE on a HPC server running CentOS 7.9 linux and using a RTX 3090 GPU.


Serialized json can't be loaded in different machine

Currently, the function to_json exports the JSON file with an absolute path to the hdf5 file containing the time series. If the files are shared or moved to different files PowerSystem throws an incomprehensible error and require manual modification of the JSON file.
cc. @rodrigomha

The code does not specifically create an absolute path to the HDF5 file in the serialized JSON.  It uses the directory that you provide.
Example 1:
to_json(sys, "sys.json")

produces sys.json in the current directory with an HDF5 file called sys_time_series_storage.h5 in the current directory.  Paths are relative.
Example 2:
to_json(sys, "/tmp/sys.json")

produces /tmp/sys.json and /tmp/sys_time_series_storage.h5.  The JSON file has an absolute path to the HDF5 file.
It's possible that the code could always serialize a path to the .h5 file that is relative to the requested system JSON file.  It would require some refactoring.
We could certainly make the error more user-friendly.
Does this explain what you saw?  You didn't list any steps, so I am not entirely certain.

Yes, this is the problem. We should implement a better relative path referencing for the serialization process.


Make TypeScript the default when running create-solid (#421)

tested manually by running a bin file produced by pnpm --filter create-solid build

Thank you


x/text: "gotext rewrite -w" returns "go format failed: invalid argument"

What version of Go are you using (go version)?
go version go1.10.3 darwin/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/erakhimberdin/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/erakhimberdin/go"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.10.3/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.10.3/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/tq/qs59s2zx57n1hw6m566m81r00000gp/T/go-build616589450=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
go get -u golang.org/x/text
cd $GOPATH/src/golang.org/x/text/cmd/gotext/examples/rewrite/
go run ../../*.go rewrite -w

What did you expect to see?
Rewritten file $GOPATH/src/golang.org/x/text/cmd/gotext/examples/rewrite/main.go
What did you see instead?
$GOPATH/src/golang.org/x/text/cmd/gotext/examples/rewrite/main.go wasn't changed.
gotext: go format failed: invalid argument
exit status 1

CC @mpvl

I've fixed it here https://go-review.googlesource.com/c/text/+/128795


Added Google Advanced Search and Google Image search Clone

🛠 Fixes Issue #552
👨💻 Changes proposed

✅ Check List (Check all the applicable boxes)

[X] My code doesn't break any part of the project (Zero Octave-Javascript-Projects).
[X] This PR does not contain plagiarized content.
[X] My Addition/Changes works properly and matches the overall repo pattern.
[X] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
Google Clone
This website meets the following features.
Pages
Website have three pages: One for Google Search, One for Google Image Search, and One for Google Advanced Search.
On the Google Search page, there are links in the upper-right of the page to go to Image Search or Advanced Search. On each of the other two pages,there is a link in the upper-right to go back to Google Search.
Query Text
On the Google Search page, the user is able to type in a query, click “Google Search”, and be taken to the Google search results for that page.
Like Google’s own, search bar is centered with rounded corners. The search button is also be centered, and is beneath the search bar
Query Images
 On the Google Image Search page, the user will be able to type in a query, click a search button, and be taken to the Google Image search results for that page.
Query Advanced
On the Google Advanced Search page, the user is able to provide input for the following four fields.
Find pages with… “all these words:”
Find pages with… “this exact word or phrase:”
Find pages with… “any of these words:”
Find pages with… “none of these words:”
Appearance
 Like Google’s own Advanced Search page, the four options are stacked vertically, and all of the text fields are left aligned.
Consistent with Google’s own CSS, the “Advanced Search” button is blue with white text. When the “Advanced Search” button is clicked, the user is taken to search results page for their given query.
Lucky
It has an “I’m Feeling Lucky” button to the main Google Search page. Consistent with Google’s own behavior, clicking this link take users directly to the first
Google search result for the query, bypassing the normal results page.
Aesthetics
 The CSS you write should match Google’s own aesthetics as best as possible.

 By- Pranjay Poddar

📷 Screenshots

@NOiR-07 pls review this PR

As you can see projects with multiple files of the same stack are being kept inside a folder, Please do that for this project, Name the folder as 'Google Clone1' inside Public.  Please move the Js file inside Js from CSS. Please update README with only 'Google Clone1' and add a link of only 'googleclone.html' as your change is not being properly reflected.
This project is
-Google clone

Advance search
Image search
together as 1 project having multiple pages.


[Suggestion] Minimize to System Tray

Is there any chance this feature could be added? I want to leave TraXile open but it bothers me having it on my taskbar (I know, lol) and would prefer if it just sat in my system tray. I know there's programs that would allow me to do this but if it was added natively I think many people would benefit.

Will be included in next release "1.1.5"

Resolved with Release 1.1.5 :-)


Replace references to the delhi docker-compose with edinburgh

Replace references to the delhi docker-compose file with references to the edinburgh docker-compose file. Fixes #1514
Signed-off-by: Michael Hall<EMAIL_ADDRESS>

Codecov Report

Merging #1515 into edinburgh will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           edinburgh    #1515   +/-   ##
==========================================
  Coverage      17.05%   17.05%
==========================================
  Files             78       78
  Lines           8790     8790
==========================================
  Hits            1499     1499
  Misses          7141     7141
  Partials         150      150

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 1c2449f...e7d6e7e. Read the comment docs.


Dress everyone up as clowns

About the PR

The station staff will be dressed as clowns or partially
Why / Balance

On the first of April
Last attempt: #26245
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: Litogin

tweak: everyone jobs clown clothes

I think the clown should be the only person dressed as a regular passenger on top of this lol

dont make the mask unremovable how are you gonna wear an oxygen mask

dont make the mask unremovable how are you gonna wear an oxygen mask

clown masks act as oxygen masks

they do?

they do?

Who?

they do?

nvm

I can almost hear it....

Thank you for your PR, unfortunately, the maintainer's conscious was to deny it for the reason that it just makes the game more annoying to play due to the constant clown shoe squeaking even if it's just for April fools.  It's also not funny.


Fix - Incorrect member count in report details after commenting or inviting another member

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/46660
PROPOSAL: https://github.com/Expensify/App/issues/46660#issuecomment-2263034139
Tests

Submit an expense to User B
Click on the expense preview in the main chat to go to expense report
Send a user mention containing User C
Click Invite from the whisper
Click on the report header
Note Members count
click on Members and check that the number indicated in (6) is agree with the member's list

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/b9c999fc-53e8-478c-9ce0-a9ae4af069fa

Android: mWeb Chrome

https://github.com/user-attachments/assets/96d7a212-14ce-4343-bb48-506725647a1f

iOS: Native

iOS: mWeb Safari

https://github.com/user-attachments/assets/a690ca0a-f55a-4e78-8400-4d21b10b8cdb

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/5a406fda-bfd9-4c5b-8839-7a665a3b92e7

MacOS: Desktop

https://github.com/user-attachments/assets/9b3f8145-a3d1-4485-afef-7a4724b98047

@FitseTLT Could you please resolve a conflict?

Resolved

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/techievivek in version: 9.0.21-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 9.0.21-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
failure ❌

🚀 Deployed to production by https://github.com/Beamanator in version: 9.0.21-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


krita: 4.0.1 -> 4.0.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/krita/versions.
These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 4.0.2 with grep in /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2
directory tree listing: https://gist.github.com/dc5ab15c03d296eeef4134cf33666c30

cc @abbradar for testing

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: krita
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Failure on aarch64-linux (full log)
Attempted: krita
Partial log (click to expand)
-- Installing: /build/OpenColorIO-1.0.9/build/ext/dist/include/yaml-cpp/binary.h
-- Installing: /build/OpenColorIO-1.0.9/build/ext/dist/include/yaml-cpp/anchor.h
-- Installing: /build/OpenColorIO-1.0.9/build/ext/dist/include/yaml-cpp/aliasmanager.h
-- Installing: /build/OpenColorIO-1.0.9/build/ext/dist/lib/pkgconfig/yaml-cpp.pc
[ 15%] Completed 'YAML_CPP_LOCAL'
[ 15%] Built target YAML_CPP_LOCAL
make: *** [Makefile:163: all] Error 2
builder for '/nix/store/xs3qykhs8nhxmrvm6n6lwpip687dwisi-opencolorio-1.0.9.drv' failed with exit code 2
cannot build derivation '/nix/store/nh4g0zjyqjnrn1x22gfyy27zmfda9vzc-krita-4.0.2.drv': 1 dependencies couldn't be built
error: build of '/nix/store/nh4g0zjyqjnrn1x22gfyy27zmfda9vzc-krita-4.0.2.drv' failed

Success on x86_64-linux (full log)
Attempted: krita
Partial log (click to expand)
shrinking /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2/lib/libkritatextlayout.so.16.0.0
shrinking /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2/lib/libkritatext.so.16.0.0
shrinking /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2/lib/libkritalibkra.so.16.0.0
shrinking /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2/lib/libkritalibpaintop.so.16.0.0
shrinking /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2/bin/krita
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2/lib  /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2/bin
patching script interpreter paths in /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2
checking for references to /build in /nix/store/frfrgq3pkda5hjpgjzm6bw9b1zp8c575-krita-4.0.2...
postPatchMkspecs


Document choices for number of Poseidon rounds

Could the choices for the number of rounds used in Poseidon be documented?
I was looking at https://github.com/CodaProtocol/coda/blob/develop/src/lib/random_oracle/random_oracle.ml#L35, and the number of rounds is notably less than the number of rounds the paper describes for alpha=5.
I presume this is due to using alpha=11, and the paper only describes parameterization up to alpha=5. Needing fewer rounds makes sense to me intuitively, since the interpolation attack becomes prohibitive much more quickly. (I assume something similar holds for the Grobner basis attack)
As a second question, this also seems like something interesting to optimize over if true. Could you further raise alpha, and lower the number of rounds (and potentially lower overall #constraints in Marlin / amount of time due to fewer MDS's). I think that it should work, and roughly estimating, I think it would be useful to switch from alpha=11 to alpha=17.
This keeps the same number of multiplications per S-Box, but saves 5 or 6 rounds.

I was looking at the equations in Poseidon with higher alpha, and I got a different answer for soundness at 128 bits of security. I also get that this parameterization has a soundness of 119 bits.
I am looking at soundness against the interpolation attack, equation (1) of section 3 (https://eprint.iacr.org/2019/458.pdf). I am assuming that this implementation is doing the recommended round padding suggested in the paper.
Coda uses 8 full rounds, and 33 partial rounds. Accounting for the suggested padding, this has R_f = floor(8 / 1.25) = 6, and R_p = floor(33 / 1.075) = 30.
To be sound against interpolation attacks per the paper, the following must be true:
R_f + R_p > log_11(2) * min(M, N) + log_2(t)
For 128 bits of soundness, and state size 3 (the minimum possible for a 2->1 hash)
min(M, N) = min(128, log_2(field_size)) = 128.
log_11(2) * 128 + log_2(3) = 38.585260269410796. Since the number of rounds must be an integer, we need R_f + R_p = 39. Since R_f is set to 6, we need 33 partial rounds. This means we need ceil(R_f * 1.25) = 8 full rounds, and ceil(R_p * 1.075) = 36 partial rounds.
I get that at R_p = 30, you have security parameter 119.

Closing this issue as it is stale for a long time.


Incorrect movement of child items

Hello everybody! How to fix the strange behavior of children?
https://www.youtube.com/watch?v=vld5YJyYXD0

I solved the problem. getChildId returning the same id for all children


Create call list clearing rules for between shifts

From a conversation between @mebates and the CM directors:

So what came up tonight was the many patients who are 'dormant' for days, weeks, or forever. We don't want to encourage CMs to keep calling PTs who don't give DCAF a call back, and leaving them on the list for the whole shift might do that. And what if you come back a month later and they're still there? Or 9 months?

@NerdyGirl537 seconded this.
Two things here:
a) finding some way to discourage people from calling patients over and over again when they don't hear back. There may or may not be a programmatic way to do this -- I suspect that the solution here is training.
b) starting with a fresh call list when you hop on a fresh shift. This I think is doable -- as part of the cleanup task, clear users' call lists entirely if they haven't logged in for two weeks or so. I don't imagine there are that many cases where people take multiple shifts like that.
@NerdyGirl537 thoughts?
Spun out from #118 .

I think clearing the call lists after a shift (while leaving the option to
mark pts as urgent if they need to retain them in the list) will
effectively take care of this. And don't urgent marks expire after 4 days?
On Sun, Oct 9, 2016 at 3:39 PM, Colin<EMAIL_ADDRESS>wrote:

From a conversation between @mebates https://github.com/mebates and the
CM directors
https://github.com/colinxfleming/dcaf_case_management/issues/118#issuecomment-246886012:
So what came up tonight was the many patients who are 'dormant' for days,
weeks, or forever. We don't want to encourage CMs to keep calling PTs who
don't give DCAF a call back, and leaving them on the list for the whole
shift might do that. And what if you come back a month later and they're
still there? Or 9 months?
@NerdyGirl537 https://github.com/NerdyGirl537 seconded this.
Two things here:
a) finding some way to discourage people from calling patients over and
over again when they don't hear back. There may or may not be a
programmatic way to do this -- I suspect that the solution here is training.
b) starting with a fresh call list when you hop on a fresh shift. This I
think is doable -- as part of the cleanup task, clear users' call lists
entirely if they haven't logged in for two weeks or so. I don't imagine
there are that many cases where people take multiple shifts like that.
@NerdyGirl537 https://github.com/NerdyGirl537 thoughts?
Spun out from #118
https://github.com/colinxfleming/dcaf_case_management/issues/118 .
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/colinxfleming/dcaf_case_management/issues/666, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKFqsIHlqj63e8GmWWDeOEyApJvBeJdFks5qyUMOgaJpZM4KSFqi
.

Yeah, urgents patients are de-flagged after a couple of days of no action.
The task here is to add something to the User#clear_call_list method that will completely wipe a user's call list if User#last_sign_in_at was over two weeks ago. This, in turn, will cascade up to the rake task that we run nightly. (And to add a test for the method you write that does this, if you can!)
Marking backend/hacktoberfest.


Refactor model abstraction

This abstracts the commonalities between the Post and Term models into a unified ActiveRecord base class.  Eventually the upcoming User model will extend this as well.  The common database related actions have been abstracted into their own dedicated classes as well, which makes this possible.

Coverage remained the same at 100.0% when pulling f7adb334a078b95467ceb69a430a98c46553e94e on refactor/model-abstraction into ed9f5f62d43ff4443396a4661d70901c0a7556d6 on master.

Coverage remained the same at 100.0% when pulling 21d9b9312a51e0aafe5b457bf06bc211411a7d6a on refactor/model-abstraction into ed9f5f62d43ff4443396a4661d70901c0a7556d6 on master.


Feature: Bills Included in Budget

Include the rough bills cost estimate in the budget.
e.g.

Broadband
Gas
Power
Water

Hmm. This might be harder to estimate, as it would have to depend on where the person is looking. Maybe needs a search area/address in the settings popup to get a better rough estimate?


[Question] Getting Free Energy results and plots

Hello - I've been trying to obtain results in the format shown in the Espaloma paper in Figure 6. That is, the free energy values and the plots using the pretrained Espaloma model (which I imagine is the one located at http://data.wangyq.net/espaloma_model.pt). In the last couple of days, I haven't been able to piece together the code examples, but I've been looking through Espaloma files, Perses examples, as well as Openmmtools examples.
My particular use case has a PDB file and 1 or more ligand SDFs.
Any help or orientation with this would be appreciated.

Figure 6 was generated using the scripts in this directory, where we provide instructions on installing the appropriate conda environment and then scripts to run and analyze the data with both espaloma-0.2.2 and openff-1.2.0 on a cluster using an LSF batch queue system.
Can you go step-by-step through where you ran into trouble? That will help us clarify the instructions to make it easier for folks to repeat the calculation.


The Broker's Java threads hang on certain SQL queries

The Broker Java Threads responsible for running queries will permanently hang on certain SQL queries.
Affected Version
Druid 0.22.1
Description
When certain SQL queries are made to Druid the Broker's Java Threads can hang. This causes the thread to permanently take up CPU and cause high Garbage Collection until the Broker is restarted.
A detailed description of the Java Thread hang issue can be found in this Druid Forum thread.
https://www.druidforum.org/t/observed-qtp-threads-hanging-on-broker-with-druid-0-22-1/7206
The Brokers are running on an r5.4xlarge footprint with 16 CPU, 128GB of Memory, and 80G Heap.
The Broker runtime.properties config:
druid.service=druid/broker
druid.plaintextPort=8082
druid.tlsPort=8282

druid.server.http.numThreads=40
druid.server.http.gracefulShutdownTimeout=PT1M
druid.server.http.defaultQueryTimeout=60000

druid.broker.http.numConnections=30
druid.broker.http.maxQueuedBytes=10000000

druid.processing.buffer.sizeBytes=500000000
druid.processing.numMergeBuffers=16
druid.processing.numThreads=15

druid.query.scheduler.prioritization.strategy=threshold
druid.query.scheduler.prioritization.durationThreshold=PT24H
druid.query.scheduler.prioritization.adjustment=-1
druid.sql.planner.requireTimeCondition=false
druid.sql.planner.metadataSegmentCacheEnable=true
druid.sql.planner.metadataSegmentPollPeriod=60000

druid.broker.cache.useCache=false
druid.broker.cache.populateCache=false

druid.monitoring.monitors=["org.apache.druid.java.util.metrics.SysMonitor","org.apache.druid.java.util.metrics.JvmMonitor","org.apache.druid.java.util.metrics.JvmCpuMonitor","org.apache.druid.java.util.metrics.JvmThreadsMonitor","org.apache.druid.client.cache.CacheMonitor","org.apache.druid.server.metrics.QueryCountStatsMonitor"]

The Broker runtime arguments:
java -Xms12G -Xmx12G -Duser.timezone=UTC -Dfile.encoding=UTF-8 -Djava.util.logging.manager=org.apache.logging.log4j.jul.LogManager -Dorg.jboss.logging.provider=slf4j -Dnet.spy.log.LoggerImpl=net.spy.memcached.compat.log.SLF4JLogger -Dlog4j.shutdownCallbackRegistry=org.apache.druid.common.config.Log4jShutdown -Dlog4j.shutdownHookEnabled=true -XX:+PrintGCDetails -XX:+PrintGCDateStamps -XX:+PrintGCTimeStamps -XX:+PrintGCApplicationStoppedTime -XX:+PrintGCApplicationConcurrentTime -Xloggc:/var/log/druid.gc.log -XX:+UseGCLogFileRotation -XX:NumberOfGCLogFiles=50 -XX:GCLogFileSize=10m -XX:+ExitOnOutOfMemoryError -XX:+HeapDumpOnOutOfMemoryError -XX:+UseG1GC -XX:+UseBiasedLocking -Xms80G -Xmx80G -XX:MaxDirectMemorySize=40G -Dcom.sun.management.jmxremote -Dcom.sun.magement.jmxremote.authenticate=true -Dcom.sun.management.jmxremote.password.file=/opt/druid/conf/druid/_common/jmxremote.password -Dcom.sun.management.jmxremote.ssl=false -Dcom.sun.management.jmxremote.port=1099 -Dcom.sun.management.jmxremote.rmi.port=1099 -Djava.rmi.server.hostname=<IP_ADDRESS> -cp /opt/druid/conf/druid/_common:/opt/druid/conf/druid/broker:/opt/druid/lib/* org.apache.druid.cli.Main server broker

We have recently been able to determine the type of SQL queries which are causing this problem, and can reproduce the issue on any of our clusters.
The query must:

Use time bucketing.
Filter on a numeric field with an empty string.

Example query:
SELECT AVG("value"), TIME_FLOOR("__time", 'PT1m')
FROM "testing:selfmonitor.metrics"
WHERE __time >= CURRENT_TIMESTAMP - INTERVAL '10' MINUTE
AND "value" = ''
GROUP BY TIME_FLOOR("__time", 'PT1m')

The query does not cause a thread hang if:

Time bucketing is removed.
The WHERE clause is changed to filter on a string field. (example: "metric" = '')
The empty string contains some value. (example: "value" = '1')

This has been an issue for us because of queries coming from Grafana. If a dashboard's variable does not get set properly then it has no value, and the SQL queries sent to Druid contain these empty string comparisons. If the comparison is done on a numeric field then the Broker's CPU will increase permanently until we reboot it.

Interesting. I am debugging another similar-sounding issue with TIME_FLOOR causing hung queries when run on inline data. It's related to materialization of excessive numbers of time buckets. I wonder if something similar is happening here.
If you have a reproducible test case that someone else can use (including both dataset + query, or a unit test in CalciteQueryTest) that would be helpful. Otherwise we'll try to come up with a unit test for this.

I think https://github.com/apache/druid/pull/12944/files should have fixed this. @rohangarg can you confirm?

@sh-vs : Are you using null-compatible mode in your cluster? (setting druid.generic.useDefaultValueForNull to false enables null-compatible mode). If so, then the query you mentioned would get fixed by https://github.com/apache/druid/pull/12944/files which is present as a part of 24.0 release.
@gianm : Yes while looking at the issue and changes above, I had observed that the time buckets generation for time-granularity queries could be improved. One crude way of stopping time-grain generation going out of hand is to limit the amount of grains allowed to be generated by the query.
A better but effort requiring way could be to derive the time-grains from data as we read it, instead of generating the time buckets first and then passing data through it.

Just to add a bit more colour here, the mode matters here because in SQL-compatible mode, the "value" = '' condition becomes an impossible condition for a numeric column. In non SQL-compatible mode, that is a valid condition and is translated to "valud" = 0.
When one of the conditions is impossible, the time filter after the AND clause gets removed by the planner and the query becomes an ETERNITY query. This is when the bug fixed by Rohan can surface.

Yes while looking at the issue and changes above, I had observed that the time buckets generation for time-granularity queries could be improved. One crude way of stopping time-grain generation going out of hand is to limit the amount of grains allowed to be generated by the query.
A better but effort requiring way could be to derive the time-grains from data as we read it, instead of generating the time buckets first and then passing data through it.

A similar problem happens on the ingestion side as well, when someone accidentally selects the wrong column for timestamp and kaboom :) I think your crude solution makes sense. The limit could be high enough that we know something is wrong with the query if a query hits that limit. And the limit is low enough that it stops an OOM. like 100,000 seems like a good number. what do you think?

The crude is fine but the problem with that is - incase the actual data produces much lesser time buckets, then we'll reject the query based on incorrect calculation of time buckets from the time intervals. And it might be confusing for users to understand the reason for failure. But if we see this problem more frequently, then probably adding a limit can be a stop gap.

Just raised #13206, which makes queries like this succeed instead of throwing an error (what we get with only #12944) or hanging (what we get before #12944).

Are you using null-compatible mode in your cluster?

Yes we are.
As a test I switched this option. With the option set to true the query does not hang, but returns invalid data (a bunch of rows of zeros). I changed the filter to "value" IS NOT NULL, and that caused the thread to hang. I switched the setting back to false, and then the IS NOT NULL ran and returned no data, while the previous "value" = '' caused the thread to hang.
Thanks for all the feedback. We do have plans to upgrade to Druid 24 soon.


hotkey to move a window

Description of the new feature / enhancement
with more and more apps using the upper most part of a window, its harder and harder to find a spot to grab a window to move it.
firefox for example, the tabs now take the top space, only a small amount of space exists between the tabs and the minimise button so that the window can be clicked and moved.
a hotkey combination + left click could be a perfect way to quickly move windows.
this would also solve those issues when the header of the window is off screen
Scenario when this would be used?
to quickly move windows without trying to find a spot on the window header to move it.
sometimes applications also remove the move option from the task bar
Supporting information
No response

/dup #269

@magiva try to use this?
https://github.com/zoo-3d/D-R/releases/tag/v1.1


Update to run on port 5001

For development we will want to run multiple apps, so they should each bind to a different port number.

Rebased to remove README changes.


Dynamic suggestions of threats

Static way of creating new threats.
Solution:
Suggesting new threats naturally upon creation depending on their diagram type.
Specifications:
suggesting the least frequent threat type every time we create a new threat. (works for all diagram types except the generic)
considered an extension to the implementation of the issue #985

#988 Closes this issue!


How to avoid using the OpenAM interface from OpenIDM

Currently the node-openam-agent redirects to OpenAM login screen and once the login is done, the user is redirected back to the demo page.
Very briefly, what might be a best practice if we just want to login to our site without being redirected to OpenAM. There are many REST API calls available and we could probably call them to login to OpenAM -- now how would that integrate with Cookieshield?
Just some hints will be useful. If this question is out of scope for the node-openam-agent -- please go ahead and close the ticket!

see the noRedirect option for CookieShield at https://github.com/zoltantarcsay/node-openam-agent/wiki/Shield#CookieShield
Alternatively, you can extend CookieShield to use a built in login page.
Another option would be to make the login URL configurable, by overriding https://github.com/zoltantarcsay/node-openam-agent/wiki/PolicyAgent#PolicyAgent+getLoginUrl

Thanks for the hints. This is what I have done and its working:

Created a custom form for login in my express app at /login
Override the getLoginUrl function to simply return /login custom login URL at all times
After user POSTs to the login form, get the session tokenId by authenticating via REST api endpoint openam/json/authenticate
Set the cookie iPlanetDirectoryPro with the tokenId

Since I set the cookie, cookie shield is able to work seamlessly.
Let me know if there are some other things I need to keep in mind. Thanks again for your help!


Using callbacks within decorate

Hi,
First and foremost, FactoryGirl is simply as great as useful.
I just stumbled on this (basically a callback within decorate):
 trait :admin do
  groups [:admins]
end

factory :user do
  sequence(:email) {|n|<EMAIL_ADDRESS>}
  password               "password"
  password_confirmation  "password"
  activities             ["none"]

  decorate :active do
    after_create {|u| a.author_profile.status = :active; a.author_profile.save }
  end

  factory :admin,        :traits => [:admin ]
end

And here are the relevant line of the stacktrace:
in `add_attribute': Both value and block given (FactoryGirl::AttributeDefinitionError)
.../factory_girl/definition_proxy.rb:89:in `method_missing'

Am I missing something?

decorate was never actually implemented. I'm assuming that you read about that syntax here but I mentioned that it's a method we're considering, not something that's actually been implemented. Sorry about that!
I decided to hold off its implementation because I think it'll end up causing confusion by basically building factory names. I think it'd be really easy to glance at with maybe one nesting, but more than one and it'll be hard to mentally map the names out based on nesting.
Something we're thinking about doing instead would be being able to apply multiple traits to a factory dynamically. So, instead of having to create an admin factory when you have an admin trait (and also assuming that your active is actually a trait as well), you could just do FactoryGirl.create(:user, :admin, :active) to build an active admin user instead of having to name all the different permutations manually. This is also not implemented yet but on the table for 2.3 probably.

Damn I overread, sorry for this useful report then...
Anyway, I like the decorator idea ;)
Thanks for this short notice answer!


CLI[Test]: Improve test coverage for core/shell-runner module

:tada: This issue has been resolved in version 0.15.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sema: Allow protocols with 'Self' constraints again

This sort of worked in 4.2, then I banned it but it broke source comaptibility.
Now that protocol superclass constraints are fully supported, let's allow this again, with some minor refactoring to the request evaluator.
So both of these are equivalent and should work correctly:
protocol P : SomeClass {}
protocol P where Self : SomeClass {}

Fixes rdar://problem/43028442.

@swift-ci Please test

@swift-ci Please test source compatibility

Build failed
Swift Test OS X Platform
Git Sha - 495873fb83ea21f3344794f466cb08d66264d388

Build failed
Swift Test Linux Platform
Git Sha - 495873fb83ea21f3344794f466cb08d66264d388

@swift-ci Please smoke test


Don't stop looking for symbols if we can't load a native pdb symbol provider. Fixes #56808.

We might run into native pdbs, but we might not include support for native
pdbs, in which case we shouldn't stop looking for other debug symbols we might
be able to load (mdbs).
https://bugzilla.xamarin.com/show_bug.cgi?id=56808

I've opened an identical PR for jbevain/cecil#419

@marek-safar I've created a mono-2017-04 branch based on the current cecil hash mono/2017-04 uses, and changed the merge branch in this PR to the mono-2017-04 branch.
Also JB merged the upstream fix.


Can't resolve './../$$_gendir/ClientApp/app/app.server.module.ngfactory'

I've been battling this error (supposedly related to AOT) even using the newer templates that don't have the include: '/ClientApp'  in the webpack.config.js. Here are my steps for reproducing it:

Start with a new template dotnet new angular
Remove jQuery and bootstrap references from the package.json, webpack.config.vendor.js, and boot.browser.ts
Run the following: npm install webpack --config .\webpack.config.vendor.js --env.prod webpack --env.prod
start updating the remaining packages in the package.json to the latest versions
run the commands in step 3 again, and you'll get get the following error:

ERROR in ./ClientApp/boot.server.ts
Module not found: Error: Can't resolve './../$$_gendir/ClientApp/app/app.server.module.ngfactory' in 'C:\Users\xxxxx\Documents\Visual Studio 2017\Projects\testapp2\ClientApp'
@ ./ClientApp/boot.server.ts 8:0-94
ERROR in Error: Please update @angular/cli. Angular 5+ requires at least Angular CLI 1.5+
    at throwNotSupportedError (C:\Users\xxx\Documents\Visual Studio 2017\Projects\testapp2\packages\compiler-cli\src\ngtools_api.ts:132:9)
    at Function.NgTools_InternalApi_NG_2.codeGen (C:\Users\xxx\Documents\Visual Studio 2017\Projects\testapp2\packages\compiler-cli\src\ngtools_api.ts:80:11)
    at _donePromise.Promise.resolve.then (C:\Users\xxx\Documents\Visual Studio 2017\Projects\testapp2\node_modules\@ngtools\webpack\src\plugin.js:428:58)
    at process._tickCallback (internal/process/next_tick.js:109:7)
    at Module.runMain (module.js:606:11)
    at run (bootstrap_node.js:393:7)
    at startup (bootstrap_node.js:150:9)
    at bootstrap_node.js:508:3

Once the error starts, even reverting changes won't make it go away.

I believe I found what corrects this. In webpack.config.js, change line 4 from:
const AotPlugin = require('@ngtools/webpack').AotPlugin;
to
const AotPlugin = require('@ngtools/webpack').AngularCompilerPlugin;
I can verify that this resolved one issue I was having and if you're upgrading to Angular 5, it appears it's a necessity.

Here is a reference https://github.com/NativeScript/nativescript-dev-webpack/issues/312

End result to fix everything was to disable server side rendering. :-(