feat: Add Saved Metrics tab to metrics popover

SUMMARY
This PR adds a possibility of selecting saved metrics (predefined in dataset). Also fixes bugs mentioned in points 1., 2. and 4. in this comment: https://github.com/apache/incubator-superset/pull/12095#issuecomment-747936902
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

CC: @junlincc @villebro @ktmud
@adam-stasiak Can you help with testing please?

Codecov Report

Merging #12123 (ed91b06) into master (1a5f61b) will decrease coverage by 3.98%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12123      +/-   ##
==========================================
- Coverage   67.26%   63.28%   -3.99%
==========================================
  Files         972      481     -491
  Lines       48036    29669   -18367
  Branches     4743        0    -4743
==========================================
- Hits        32312    18776   -13536
+ Misses      15607    10893    -4714
+ Partials      117        0     -117

Flag
Coverage Δ

cypress
?

javascript
?

python
63.28% <ø> (-0.93%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/hive.py
54.61% <0.00%> (-29.62%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/presto.py
69.95% <0.00%> (-12.45%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

superset/views/database/validators.py
78.94% <0.00%> (-5.27%)
:arrow_down:

... and 503 more

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 1a5f61b...ed91b06. Read the comment docs.

Thank you so much for this PR! it's looking great!
minor issue: overflow when resizing

not related to this PR - the styling of the boxes look very different, size and color.. since we are pausing the remaining refractor work until 1.0 is hardened, i wonder if we can style the old ones to match the new antd style?

other than that, it works and look good
@rusackas @kgabryje

@kgabryje I'll provide a PR to fix that little label overflow.


feat(text): toConstantCase() + unicode support + case aware word splitting

per discussion at https://github.com/denoland/deno_std/issues/3125#issuecomment-2180354086

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

Looks good to me. @std/text is now in the process of stabilization #4999. Let's land after 1.0.0 of @std/text published.

The current implementation of splitToWords() only supports Latin letters. This commit adds a unicode-compatible implementation + support for WordSpliting according to case. I can open it in a separate PR if you prefer.

The current implementation of splitToWords() only supports Latin letters. This commit adds a unicode-compatible implementation + support for WordSpliting according to case. I can open it in a separate PR if you prefer.

Nice! Thank you. Yes, could you please open a separate PR?

Nice! Thank you. Yes, could you please open a separate PR?

moved


Click event not triggered

Hi,
I was trying to use d3-zoom to draw a zoomable graph with nodes and edges.
A node can be selected when it is clicked and also can be deselected when the zoomable background is clicked.
The problem here is that a mouse move event is fired along with a mouse down event in chrome
(windows 10, chrome version is 52.0. The following link describes the problem:
https://bugs.chromium.org/p/chromium/issues/detail?id=161464
)
So the click event on the zoomble background will not be triggered as it enables drag event, which results in that the selected status on node cannot be cancelled.

Duplicate of #65.


TextField

TextField
TextField is an input field that users can type into.
Functionality
Rough notes:

Emphasize using the native <input> element.
Cover different states (hover, active, focus, disabled, read-only)
Field label, caption, error

Best practices
Rough notes:

Using <label> along with <input>
Using icons with TextField
Marking a field as optional or required
Field validation feedback
How to use field label or placeholder (maybe some copywriting tips)
Cover naming (a la. Input vs TextField)

Implementation

Cover using <input type="text" /> for numeric inputs

Examples
 If possible, provide links to a few implementations of this component.
Accessibility

Mention relevant accessibility attributes when used with an Autocomplete component.

Resources

Why the GOV.UK Design System team changed the input type for numbers
W3 - Labeling Controls

Visuals
 Any relevant visuals that help describe the component.
Additional context
 Anything that does not fit into the other categories.

Done in #18.


logging-operator - error received after stop sequence was engaged

Hello,
I wanted to install the helm charts logging-operator on my kubernetes cluster, however I encounter the following problem :
{"level":"info","ts":1643188235.7183502,"msg":"Waited for 1.037446427s due to client-side throttling, not priority and fairness, request: GET:https://<IP_ADDRESS>:443/apis/longhorn.io/v1beta1?timeout=32s\n"}
{"level":"error","ts":1643188235.7236505,"logger":"controller-runtime.source","msg":"if kind is a CRD, it should be installed before calling Start","kind":"EventTailer.logging-extensions.banzaicloud.io","error":"no matches for kind \"EventTailer\" in version \"logging-extensions.banzaicloud.io/v1alpha1\"","stacktrace":"sigs.k8s.io/controller-runtime/pkg/source.(*Kind).Start.func1\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/source/source.go:127"}
{"level":"error","ts":1643188235.7239068,"logger":"controller-runtime.manager.controller.eventtailer","msg":"Could not wait for Cache to sync","reconciler group":"logging-extensions.banzaicloud.io","reconciler kind":"EventTailer","error":"failed to wait for eventtailer caches to sync: no matches for kind \"EventTailer\" in version \"logging-extensions.banzaicloud.io/v1alpha1\"","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/internal/controller/controller.go:195\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/internal/controller/controller.go:221\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).startRunnable.func1\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/manager/internal.go:696"}
{"level":"error","ts":1643188235.7243013,"logger":"controller-runtime.manager.controller.hosttailer","msg":"Could not wait for Cache to sync","reconciler group":"logging-extensions.banzaicloud.io","reconciler kind":"HostTailer","error":"failed to wait for hosttailer caches to sync: timed out waiting for cache to be synced","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/internal/controller/controller.go:195\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/internal/controller/controller.go:221\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).startRunnable.func1\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/manager/internal.go:696"}
{"level":"error","ts":1643188235.7242916,"logger":"controller-runtime.manager.controller.logging","msg":"Could not wait for Cache to sync","reconciler group":"logging.banzaicloud.io","reconciler kind":"Logging","error":"failed to wait for logging caches to sync: timed out waiting for cache to be synced","stacktrace":"sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/internal/controller/controller.go:195\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/internal/controller/controller.go:221\nsigs.k8s.io/controller-runtime/pkg/manager.(*controllerManager).startRunnable.func1\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/manager/internal.go:696"}
{"level":"error","ts":1643188235.724392,"logger":"controller-runtime.manager","msg":"error received after stop sequence was engaged","error":"failed to wait for hosttailer caches to sync: timed out waiting for cache to be synced"}
{"level":"error","ts":1643188235.7244577,"logger":"controller-runtime.manager","msg":"error received after stop sequence was engaged","error":"failed to wait for logging caches to sync: timed out waiting for cache to be synced"}
{"level":"error","ts":1643188235.7245076,"logger":"setup","msg":"problem running manager","error":"failed to wait for eventtailer caches to sync: no matches for kind \"EventTailer\" in version \"logging-extensions.banzaicloud.io/v1alpha1\"","stacktrace":"main.main\n\t/workspace/main.go:192\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:225"}
{"level":"error","ts":1643188235.72456,"logger":"controller-runtime.manager","msg":"error received after stop sequence was engaged","error":"leader election lost"}

I installed the latest version of helm charts and here is my configuration file :
replicaCount: 1
image:
  repository: ghcr.io/banzaicloud/logging-operator
  tag: 3.17.0
  pullPolicy: IfNotPresent
extraArgs:
  - -enable-leader-election=true
imagePullSecrets: []
nameOverride: ""
fullnameOverride: ""
namespaceOverride: ""

podLabels: {}
annotations: {}

createCustomResource: false
resources:
  limits:
    cpu: 500m
    memory: 1024Mi
  requests:
    cpu: 100m
    memory: 128Mi

nodeSelector:
  kubernetes.io/hostname: x

tolerations:
  - key: dedicated
    operator: "Exists"
    effect: "NoSchedule"

affinity: {}

http:
  port: 8080
  service:
    type: ClusterIP
    clusterIP: None
    annotations: {}
    labels: {}
rbac:
  enabled: true
  psp:
    enabled: true
    annotations:
      seccomp.security.alpha.kubernetes.io/allowedProfileNames: 'docker/default,runtime/default'
      seccomp.security.alpha.kubernetes.io/defaultProfileName: 'runtime/default'
podSecurityContext: {}
securityContext: {}

priorityClassName: {}

monitoring:
  serviceMonitor:
    enabled: false
    additionalLabels: {}
    metricRelabelings: []
    relabelings: []

Environment details:

Kubernetes version (e.g. v1.15.2): 1.20.5
Cloud-provider/provisioner (e.g. AKS, GKE, EKS, PKE etc): RKE
logging-operator version (e.g. 2.1.1):  3.17.0
Install method (e.g. helm or static manifests): helm

/kind bug

Is this still applicable? Were the CRDs present before the pod started?


Question: is there a reason why indentation is handled differently in different modules?

For example, some modules use shiftwidth to define indentation while others use tabstop. Is there a reason why this is handled in this way? Also, has using indentexpr or the = operator to handle indentation been discused in the past?
I'm sorry if this isn't the best place to ask :p

@TheLeoP Hey! This is a fine place to ask. I know we have tried to centralize this in the past but ran into some issues. If you have any ideas on how to make this easier/centralize the indention configuration (or want to throw up a PR as a POC) we would be more than happy to review.

I made a POC on my local machine using [range]:normal! == to format changed lines on the formatting step, but it caused a bunched of issues because it depends on indentexpr (if equalprg is not set).
Does it make sense to relly on methods outside of this plugin for indenting (i.e. indentexpr) and let users figure that part on their own? Or is the goal to fully manage the indentation of the changes the plugin makes?


Move ADM and SOV command settings into app_settings.py

Move ADM and SOV command settings into app_settings.py so i can properly sanitize them into empty lists if these settings arent used.

fixed in https://github.com/pvyParts/allianceauth-discordbot/pull/38


TypeSchema breaks with TypeScript 5.5.2

When using TypeScript 5.5.2, the imported Schema is typed any. Note that this is with @typeschema/main, @typeschema/zod and zod packages installed, using workspace TypeScript version in VS Code (the current default editor one is 5.4).

.vscode/settings.json (to use workspace TS Version):
{
  "typescript.tsdk": "node_modules/typescript/lib"
}

I'm using pnpm, but I've just tried with npm and got the same result.

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Could be related to:

nonara/ts-patch/issues/159

Now that the ts-patch PR for TS 5.5.2 got merged, is it sufficient to rebuild and publish new TypeSchema packages to fix this issue? @decs


[QTCS] family and missing values

Discuss whether QTCS is capable of supporting missing values (bypassing custom_checks seems to work, but whether results are correct or not is another question).

See discussion in #144


best way to do multiple listeners

Great work on this module, really wraps this process easily.
question:
If I have 2 listeners to implement, do I just create the listener and target group pairs separately and attach to the alb arn?

Same issue here, Im not really seeing how to make this work.
I have 3 autoscaling groups, which all list on HTTPS/8081, and HTTP/80.  SSL Cert will be the same for all the HTTPS ports and access is determined with host based routing.  This appears to want to setup a default listener and target group.

Hey @rojomisin and @jseiser - Thanks for the interest in this project and explaining what you're trying to do. I think the use case here is common enough to find a viable solution and I have an idea in mind for it. For now, you might be able to create some resources outside of the module and connect them via the outputs from this module... though I'm not 100% sure and you might end up with unused resources from within the module. I have some time off this week to play around with some improvements and this is definitely one of them.

Worked through this today and got to the point of successfully defining both listener types (http/tcp, https) as lists of maps to add flexibility. Rereading what you have here, I'm thinking you're also asking for the target groups to be defined in a similar way (as a list of maps that provide config details). I should have that wrapped up in the next day.


what is the meaning of  wd and cl

please tell me what is the meaning of  wd and cl in learning curves,thanks

wd=weight decay, cl=center loss, Kp=keep percentile, 75 => keep only the 75% of the images closest to its class centers


creating 'About' screen

I will be creating 'About' screen with very basic details about the project and about us.

[x] making basic 'About' screen

[x] integrating it with main menu

[x] adding CSS style to it

[x] adding it to the skeleteon of the gui in wiki

Added this screen to the skeleton of Driver GUI in wiki :)
Issue is closed.


Allow https to follow redirection when requesting latest release data

Closes #1505  as it allows https to follow redirections, it checks for status 301 and 302.
I've looked through https docs and github https to see if there was an easier way or option to allow redirections, but I couldn't find anything.

Thanks is great @dan1d , thanks for taking care of it so quickly! Hopefully we can get a patch release with this fix for the old app next week :) (working out some build issues at the moment).

@tzarebczan Do you know how we can test this without renaming?

@seanyesmunt not easily, but could stage something on a local repo - I don't think it's worth it. I'll test this today by temporarily renaming lbry-app to lbry-desktop again.

Tested and looks like it's working as expected!


update Readme.md, neovim: add --lsp argument

The --lsp argument is no longer needed since its now the default: https://github.com/haskell/haskell-ide-engine/pull/706

Ups, sorry, seems like I'm using old hie


Tool 985 use new cache pkg

Caching logic is duplicated throughout several repos. We should change the duplicated code repo by repo to use the centralized package (currently living in here: https://github.com/bitrise-steplib/bitrise-step-android-unit-test/blob/master/cache/cache.go)
What was done:

Godep dropped and migrated to dep
cache logic changed to use aforementioned package
deleted some unused files

@godrei pls merge.


LDD driver initialization err

ldd parameter device address is incorrect in commands, i think it should be 2051 instead of 3040
also example code does not mention how to initialize multiple ldd objects with different addresses (which may cause errors)

No, for TEC controller it is 2051. For laser driver boards (LDD family) it is 3040.
I checked this, 3040 is accepted by LDD and changes also show up in manufacturer's software.
When using 2051 on LDDs, "Parameter is not available" is raised.
What addresses do you refere to? Multiple hosts can be addressed by multiple instances and do NOT require unique settings for the parameter id 3040 (or 2051) e.g.:
device1 = mecom.MeCom('COM1', metype='LDD')
device2 = mecom.MeCom('COM2', metype='LDD')
device3 = mecom.MeCom('COM3', metype='LDD')
If your boards are connected via RS485 they need unique settings for parameter id 3040 (or 2051). You connect to a single host and add the address to the communication, e.g.:
devices = mecom.MeCom('COM1', metype='LDD')
devices.get_parameter(parameter_id=3040, address=1)
devices.get_parameter(parameter_id=3040, address=2)
devices.get_parameter(parameter_id=3040, address=3)

@jscman  thx for the response! That what I tried from the example code given, but it does not execute properly on the ldd1301. I even used the command tool and sent serial to the device and I got a parameter not available response:
https://www.meerstetter.ch/customer-center/downloads/category/76-ldd-130x-communication-protocols?download=631:ldd-130x-communication-protocol-5260d
Let me know if there is something I'm missing here.

Ah, I see. It looks like this is related to some of the newer meerstetter boards having different commands. There was a similar issue with the (LDD-1321) which was solved by a metype = "LDD-1321".
I dont know if the whole LDD-13xx family is identical, but in your case I suggest to check the commandy.py and try to use the LDD-1321 type. But you would have to add the commands you need to the command.py (e.g. device address).


Highlight post layout

When the highlight post are used, for some strange reasons the margin between nav and the post not work.

Fixed


[e2e]: delegator-generic workflow_dispatch branch1 default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/8548687194
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.workflow_dispatch.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.workflow_dispatch.branch1.default.slsa3.yml
Trigger: workflow_dispatch
Branch: branch1
Date: Thu Apr  4 03:08:49 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/8564400139
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-generic.workflow_dispatch.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-generic.workflow_dispatch.branch1.default.slsa3.yml
Trigger: workflow_dispatch
Branch: branch1
Date: Fri Apr  5 03:25:03 UTC 2024
Tests are passing now. Closing this issue.


Bump default server version values to 1.19.0

What was changed
Bumped default server version values to 1.19.0
Why?
To match the latest release version.
Checklist

Closes #344

How was this tested:
Locally.

Any docs updates needed?
Release process update.

@mindaugasrukas please also update the git tag. I just checked out the tag and was very confused when I had 1.18.5 running

@alexshtin maybe you can fix that


Support load splat files

This PR implements loading of .splat file format.
I took @simonbethke 's implementation of deserializeFromSSplat() function from #209 and integrated it as simply as possible.

Nice, thank you!


Issue #1 - Start and End of Night

Closes #1 .
Basics of start and end of night.

Currently night duration set to 10 seconds, in Tunables Data

I think we'd need another issue here, to figure out a good implementation
of stat tracking.
On Wed, Oct 2, 2024 at 10:12 AM NickoLefrench @.***>
wrote:

@.**** commented on this pull request.
On Assets/Scripts/Managers/MenuManager.cs
https://github.com/NickoLefrench/TapperRedone/pull/3#discussion_r1784607535
:
Ok so the score being carried over to the next night. Code is just
implementation of EON.
—
Reply to this email directly, view it on GitHub
https://github.com/NickoLefrench/TapperRedone/pull/3#discussion_r1784607535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEBHK7X2QHQQEB32WBGTUNLZZP5OHAVCNFSM6AAAAABPFZLNLKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNBTGA3TSNBRGY
.
You are receiving this because you were assigned.Message ID:
@.***>


Plan to convert to typescript?

For now the project is written in javascript, is there any plan to convert the project to typescript? So, people who want to contribute can easily knew which function do things and what thing need to be pass on on the function. And the development process can produce less bugs since we knew which stuff that break the code

I try to rewrite on this repo, but looks like the current code with the current tensorflow/tfjs kinda broken

I don't have this plan. Not something particularly interesting to me, I would say. haha :)


Typo in README.md

Describe the bug

In the "Recommended steps" section, there's a sentence that seems to have a typo:
### Recommended Steps
[Fork](https://docs.github.com/en/get-started/quickstart/fork-a-repo) this repo (so that you have your own copy that you can modify) and then installing you can install to your machine using the methods above.

To Reproduce

NA

Desktop

OS: NA
Terminal: NA

Neovim Version

NA

Close because of #731


Finetuning llama2 works but llama2-chat-hf doesn't? Loss goes to zero

Library versions:
Transformers version: 4.31.0
Datasets : 2.13.0
PEFT version: 0.4.0
TRL version : 0.7.1
PyTorch version : 2.0.1
Accelerate : 0.21.0
Bits and Bytes : 0.40.2

Problem:
I used a combination of the above libraries and the steps outlined in this blog by Philipp Schmidd to first instruction fine-tune the llama2 models (not the chat versions) on alpaca data. This works great. And I get decent training curves.

Next, I wanted to use the same script to instruction fine-tune the chat models on alpaca data. For this, the only change should be the name of the model and the way the prompt is formatted (by including the [INST], [\INST], etc tokens) and the rest of the script should work exactly the same way?
Idk why but it doesn't. Here's the training curve I get.

And here is a corresponding minimal code snippet to illustrate my setup

Click to view code
import torch
from transformers import AutoTokenizer, AutoModelForCausalLM, BitsAndBytesConfig
from peft import LoraConfig, prepare_model_for_kbit_training, get_peft_model
from transformers import TrainingArguments
from trl import SFTTrainer, DataCollatorForCompletionOnlyLM
from datasets import load_dataset
import random
import numpy as np

SEED = 42
random.seed(SEED)
np.random.seed(SEED)
torch.manual_seed(SEED)
torch.cuda.manual_seed_all(SEED)

model_id = "meta-llama/Llama-2-7b-chat-hf"

bnb_config = BitsAndBytesConfig(
  load_in_4bit=True,
  bnb_4bit_quant_type="nf4",
  bnb_4bit_use_double_quant=True,
  bnb_4bit_compute_dtype=torch.bfloat16,
)

model = AutoModelForCausalLM.from_pretrained(
  model_id,
  quantization_config=bnb_config,
  use_cache=False,
  device_map="auto",
)

model.config.pretraining_tp = 1

tokenizer = AutoTokenizer.from_pretrained(model_id)
tokenizer.pad_token = tokenizer.eos_token
tokenizer.padding_side = "right"

peft_config = LoraConfig(
  lora_alpha=16,
  lora_dropout=0.1,
  r=64,
  bias="none",
  task_type="CAUSAL_LM",
)

model = prepare_model_for_kbit_training(model)
model = get_peft_model(model, peft_config)

args = TrainingArguments(
  num_train_epochs=1,
  per_device_train_batch_size=4,
  gradient_accumulation_steps=2,
  gradient_checkpointing=True,
  optim="paged_adamw_32bit",
  logging_steps=10,
  save_strategy="epoch",
  learning_rate=3e-4,
  bf16=True,
  tf32=True,
  max_grad_norm=0.3,
  warmup_ratio=0.03,
  lr_scheduler_type="constant",
  disable_tqdm=True,
)

def format_prompt(instruction, sys_prompt=None, input=None, output=None):
  if sys_prompt is None:
      DEFAULT_SYSTEM_PROMPT = """\
          You are a helpful, respectful and honest assistant. Always answer as helpfully as possible, while being safe.  Your answers should not include any harmful, unethical, racist, sexist, toxic, dangerous, or illegal content. Please ensure that your responses are socially unbiased and positive in nature.\n\nIf a question does not make any sense, or is not factually coherent, explain why instead of answering something not correct. If you don't know the answer to a question, please don't share false information.\
          """
  else:
      DEFAULT_SYSTEM_PROMPT = sys_prompt
  PROMPT_DICT = {
      "prompt_input": (
          "<s>[INST] <<SYS>>\n{system_prompt}\n<</SYS>>\n"
          "Below is an instruction that describes a task, paired with an input that provides further context. "
          "Write a response that appropriately completes the request.\n\n"
          "### Instruction:\n{instruction}\n\n### Input:\n{input}\n\n### Response:\n[/INST]\n{output}"
      ),
      "prompt_no_input": (
          "<s>[INST] <<SYS>>\n{system_prompt}\n<</SYS>>\n"
          "Below is an instruction that describes a task. "
          "Write a response that appropriately completes the request.\n\n"
          "### Instruction:\n{instruction}\n\n### Response:\n[/INST]\n{output}"
      ),
  }

  if input is None or input == "":
      return PROMPT_DICT["prompt_no_input"].format_map(
          {"instruction": instruction, "system_prompt": DEFAULT_SYSTEM_PROMPT, "output": output}
      )
  else:
      return PROMPT_DICT["prompt_input"].format_map(
          {"instruction": instruction, "input": input, "system_prompt": DEFAULT_SYSTEM_PROMPT, "output": output}
      )

max_seq_length = 2048

data_path = "yahma/alpaca-cleaned"
dataset = load_dataset(data_path)
dataset = dataset["train"]

dataset = dataset.map(
  lambda example: {
      "text": format_prompt(
          example["instruction"], example["input"], example["output"]
      ),
  },
  remove_columns=["instruction", "input", "output"],
)

splits = dataset.train_test_split(test_size=0.2)
train_dataset = splits["train"]
eval_dataset = splits["test"]

data_collator = DataCollatorForCompletionOnlyLM(
  response_template=tokenizer("[/INST]")['input_ids'][2:],
  tokenizer=tokenizer,
)

trainer = SFTTrainer(
  model=model,
  train_dataset=train_dataset,
  eval_dataset=eval_dataset,
  dataset_text_field="text",
  data_collator=data_collator,
  peft_config=peft_config,
  max_seq_length=max_seq_length,
  tokenizer=tokenizer,
  args=args,
)

trainer.train()

I also tried adding a special PAD token that is not EOS by doing
tokenizer.add_special_tokens({'pad_token': '[PAD]'})
model.resize_token_embeddings(len(tokenizer))

but I get the same issue even then.
Maybe I am missing something here. But it would be very helpful if someone  (Maybe @younesbelkada, @lewtun @lvwerra ?) could help me figure this out! Thanks.

What if you remove the template just for the sake to exclude it as a source of issues?

The issue persists even if I remove the template (INST, SYS, etc) and replace it with just the basic alpaca formatting.

@OneCodeToRuleThemAll Thanks for the pointers. But if you see the code snippet I included in the issue description, I am already doing
tokenizer = AutoTokenizer.from_pretrained(model_id)
tokenizer.pad_token = tokenizer.eos_token
tokenizer.padding_side = "right"

and still have the loss going to zero. I think the rest of the script I have is similar to the one you linked to. Am I missing something else?
The same problem persists for me if I do
tokenizer = AutoTokenizer.from_pretrained(model_id)
tokenizer.pad_token = "[PAD]"
tokenizer.padding_side = "right"
model.resize_token_embeddings(len(tokenizer))

I figured out the problem. It was a mistake on my end.
In the format_prompt function I was using for alpaca, I specified the order of parameters incorrectly, such that the model was only getting the instructions from the dataset, but not the corresponding outputs.
I tried using my script now on another dataset "timdettmers/openassistant-guanaco" as well and it works for that data as well.


Large number of books and personalization.

Hi, I'm quite impressed with what you did with swamisivananda.ai. I want to do something very similar so i wanted to ask based on projects what you did differently other then server deployment frontend and basic stuff. I am more interested if you used something else in your training and if you used some prompts in chat. I've 100 books and i need to chat with them. Any feedback is golden. Also if there is a better way to contact you please lmk so we can talk from there.

Here is the core of the logic, I am yet to opensource this code but this should give you an idea on how to integrate Pinecone with NextJs
import type { NextApiRequest, NextApiResponse } from "next";
import { Configuration, OpenAIApi } from "openai";
import pineconeStore from "@/utils/pineconeStore";

const configuration = new Configuration({
  apiKey: process.env.OPENAI_KEY,
});

const openai = new OpenAIApi(configuration);

export default async function translate(
  req: NextApiRequest,
  res: NextApiResponse
) {
  const { messages, userName } = req.body;

  const translatedText = await askOpenAI({ messages, userName });

  res.setHeader("Content-Type", "application/json");
  res.send(JSON.stringify({translatedText}));
}

async function askOpenAI({
  messages,
  userName,
}: {
  messages: Message[];
  userName: string;
}) {

  const pinecone = await pineconeStore();

  // updated the message content to include context snippets
  if (messages?.length > 0) {
    const lastMsgContent = messages[messages.length - 1].content;

    const data = await pinecone.similaritySearch(lastMsgContent, 3);

    const updatedMsgContent = `
    user question/statement: ${lastMsgContent}
    context snippets:
    ---
    1) ${data?.[0]?.pageContent}
    ---
    2) ${data?.[1]?.pageContent}
    ---
    3) ${data?.[2]?.pageContent}
    `;

    messages[messages.length - 1].content = updatedMsgContent;
  }

  try {
    const response = await openai.createChatCompletion({
      model: "gpt-3.5-turbo-0301",
      messages: [
        {
          role: "system",
          content: `
        Imagine you are Swami Sivananda and give advice to the user you're interacting with that may ask you questions or advice. The user's name is ${userName}. Introduce youself to ${userName}.`,
        },
        ...(messages || [
          {
            role: "user",
            content: "Hi There!",
          },
        ]),
      ],
    });

    return response?.data?.choices?.[0]?.message?.content;
  } catch (e: any) {
    console.log("error in response: ", e.message);
    return "There was an error in processing the ai response.";
  }
}```

@raghavan thanks a lot 🙏


Parallel execution controller for automation farms

Referencing - https://github.com/tebelorg/TagUI/issues/42#issuecomment-320465044
Parallel runs are good to speed up something, provided that process is not tied with a single user account that is meant for single-login at any one time. TagUI already has web-service for Linux/macOS to allow triggering via URL call. This is a capability which deserves looking further into.
To deploy parallel runs, each machine needs to be a server node and a separate controller script can send the execution requirements to the webservice servers accordingly. Alternatively, each slave can connect to a single master controller machine to retrieve inputs for execution and feedback through the controller webservice. Key functionalities - distribute flows, execute, monitor status, collect results..

Was chewing on this for a couple of days. Decided against implementing a parallel execution controller for now. The reason is intentionally pace out how much TagUI can scale.
That is also same reason why the datatable format is in the non-standard data-centric way versus the usual way which columns and rows are swapped from the TagUI datatable format. It takes between 1-9 lines in PHP to transpose an array of rows and columns. The reason for making the default format a non-mainstream format, and also not including a function to transpose is to limit the tool.
I have nothing against data-scrapping use cases or load-testing applications. But making it too easy to scale may end up attracting the wrong users with the wrong expectations of what the tool is designed for. Furthermore, there are established leading open-source tools specifcally for those purposes.
May reopen and revisit in future. At the moment no personal need to implement something like this.

response to a user asking about parallel runs
Yeah TagUI was not designed to run in parallel, although separate folders on the same laptop computers it can run batch automation using separate csv files. Still it does not have a controller to scale the automation in parallel across multiple computers. I didn't add that part into its core features as parallel runs can be easily misused for bad and malicious purposes.


Regenerate the sdk to pick up internal payment field

#30  👈
main

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

Join @jklein24 and the rest of your teammates on  Graphite

Merge activity

Mar 8, 3:33 PM EST: @jklein24 merged this pull request with Graphite.


chunk-QYYA6GZJ.js?v=86847e0a:9145 Uncaught TypeError: Cannot read properties of null (reading 'innerHTML')     at IconPicker2.selectItem (react-quill.js?v=fb924d18:9699:47)     at new IconPicker2 (react-quill.js?v=fb924d18:9691:23)     at react-quill.js?v=fb924d18:7019:30     at Array.map ()     at SnowTheme2.buildPickers (react-quill.js?v=fb924d18:7014:42)     at SnowTheme2.extendToolbar (react-quill.js?v=fb924d18:9928:24)     at SnowTheme2.addModule (react-quill.js?v=fb924d18:6982:26)     at react-quill.js?v=fb924d18:6254:29     at Array.forEach ()     at SnowTheme2.init (react-quill.js?v=fb924d18:6252:53)

Thanks for filing an issue!
If at all possible, please provide a Codesandbox or Codepen to demonstrate the problem you're having with React Quill. Here's a template to get you started.
⚠ Make sure that your bug hasn't already been fixed by ReactQuill v2.0.0-beta.2. See the homepage for instructions on how to upgrade.
Ticket due diligence

[ ] I have verified that the issue persists under ReactQuill v2.0.0-beta.2
[ ] I can't use the beta version for other reasons

ReactQuill version

[ ] master
[ ] v2.0.0-beta.2
[ ] v2.0.0-beta.1
[ ] 1.3.5
[ ] 1.3.4 or older
[ ] Other (fork)

FAQ
Is this a bug in Quill or ReactQuill?
ReactQuill is just a ~thin~ wrapper on top of the Quill editor. Often, what looks like a bug in ReactQuill, is actually a bug in the Quill editor itself. Before opening a ticket, please check the Quill documentation, and the issues page, and see if that answers your question first.
How do I access the wrapped Quill instance?
See the instance methods and API documentation.

I get this when making a custom toolbar and including the align item:
<select className="ql-align" />
My setup is to have both quill and monaco so that I can have a WYSIWYG editor with HTML code editing capabilities.
It works on load, but then errors out when going back to Quill.

Nevermind... Looks like a Quill issue
https://github.com/quilljs/quill/issues/2287


Support Tap to focus

Is your feature request related to a problem? Please describe.
I'd be great if we could support Tap to focus when taking photos with the camera
Describe the solution you'd like
When capturing photos, upon taping in the screen, the camera should focus that area.
Describe alternatives you've considered
https://stackoverflow.com/questions/26682450/set-camera-focus-on-tap-point-with-swift

Closing this since the feature was there but a bug was preventing it's use. PR sent.


Closes #2342 Clean up easy if() and else if() code chunks in datetime/date codebase

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[x] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Review the Cheat Sheet. Make any required updates to it by editing the file inst/cheatsheet/admiral_cheatsheet.pptx and re-upload a PDF version of it to the same folder.
[x] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md under the header # admiral (development version) if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done! Much love to your accomplishment!

How can I address the following warning while running devtools::check() ?

How can I address the following warning while running devtools::check() ?

You don't need to worry about the qpdf stuff. ...but if you want to get rid of it...I believe installing the package will help https://rbasics.org/guides/how-to-use-the-qpdf-package-in-r/
Let me know if it works!!

How can I address the following warning while running devtools::check() ?

You don't need to worry about the qpdf stuff. ...but if you want to get rid of it...I believe installing the package will help https://rbasics.org/guides/how-to-use-the-qpdf-package-in-r/
Let me know if it works!!

I have installed the package, but it does not change anything, I still face the warning. I'll proceed with the PR with this remaining warning.

It looks like derive_date_vars.R does not exist anymore and updates must be implemented in derive_vars_dt_dtm_utils.R instead. Could you please confirm the same and how can I resolve the conflicts with the non-existing file derive_date_vars.R ?

Thanks.

@bms63 Can I simply delete that branch (2342_clean_up_easy_if_and_else_if_code_datetimedate) without proceeding with the PR, as the new file to update is anyway not on that branch ?

@bms63 Can I simply delete that branch (2342_clean_up_easy_if_and_else_if_code_datetimedate) without proceeding with the PR, as the new file to update is anyway not on that branch ?

Yes!! that is easiest and quickest solution!!
Although fixing the merge conflicts at the command line is a nice exercise especially for something so minor if you have not done it before!!  :) Choose your adventure :)

@bms63 Can I simply delete that branch (2342_clean_up_easy_if_and_else_if_code_datetimedate) without proceeding with the PR, as the new file to update is anyway not on that branch ?

Yes!! that is easiest and quickest solution!!
Although fixing the merge conflicts at the command line is a nice exercise especially for something so minor if you have not done it before!! :) Choose your adventure :)

I tried to with the github doc help, but I was expecting to see that there was a conflict...

I am not sure here how to proceed to see the conflicts.

@bms63 thanks for sharing the commands, doing great with your help.

Do I have to commit all these updates ?

Nice!!  Now you can update those files.  Another PR removing and adding files can cause a lot of issues while you are working on the same thing!!
When you have a PR open - it is good to check if there are any latest updates that you can merge into your branch.  Just make sure you pull them down one you click this button


Allow sorting assigned references

Allow sorting assigned attribute values.

Introduce intermediate models between values and Assigned models
Split attribute models into small files
Add mutations for sorting: PageReorderAttributeValues, ProductVariantReorderAttributeValues, ProductReorderAttributeValues

Linked task: SALEOR-1853

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-1853 Allow sorting assigned references


Similar projects

When browsing reddit, I notice a similar project insanitybit/cargo-sandbox, which is intended to be a sandbox for cargo itself and its extensions (the author claims it can support external subcommands like cargo-pgx).

Now that you mention it, I remember seeing that. I considered it unsuitable for my needs because the thing I want to sandbox most is NPM, not Cargo.


Image export should not include temporary state

Describe the bug
When an image export is made, temporary state is included (e.g. highlight).
To Reproduce
Steps to reproduce the behavior:

Go to any result
Click on a node
Export the network image
See the highlight is applied in the image

Expected behavior
The highlight should not be in the image, nor should any other temporary state.  Each state should correspond to Cytoscape CSS classes.  Just remove those classes and restore them after the export.

I believe this is fixed.


rename _improper_ctypes_check functions

rustc seems to have added a warning for dupplicate extern functions with different signatures:
warning: `_improper_ctypes_check` redeclared with a different signature
   --> src/bootinfo/mod.rs:119:5
    |
119 |     fn _improper_ctypes_check(_boot_info: BootInfo);
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ this signature doesn't match the previous declaration
    |
   ::: src/bootinfo/memory_map.rs:235:5
    |
235 |     fn _improper_ctypes_check(_boot_info: MemoryMap);
    |     ------------------------------------------------- `_improper_ctypes_check` previously declared here
    |
    = note: `#[warn(clashing_extern_decl)]` on by default
    = note: expected `unsafe extern "C" fn(bootinfo::memory_map::MemoryMap)`
               found `unsafe extern "C" fn(bootinfo::BootInfo)`

this pr simply renames the functions to get rid of the warning

Thank you!

Published as version 0.9.5.


Review of CONTRIBUTING guidelines

The CONTRIBUTING guidelines have several points that are either incomplete or outdated. These are the things I've noticed:

We use tox, but nowhere it is said that it needs to be installed or how to do it. Indeed, there's no brief discussion about we using tox as the development environment.
Maybe some discussion on how to setup the virtual environment created by tox in VSCode, also working with the debugger.
In the Installation section, it is said that tox -e dev installs all development options, however the Sundials solver is not installed.
About Sundials, there's no mention in that file at all.  I guess it should be installed with tox -e pybamm-requires, right?
If you activate the tox environment, within it there's no tox command so any subsequent tox -e {whatever} will fail.
Travis CI is no longer used as the CI system. It should be removed from here and mention GitHub Actions instead.
In general, the file might benefit form some content reorganisation, but that's just a purely subjective comment.

I'm happy to tackle these issues myself and open a PR with the changes, but wanted to check with you first and gather any other changes that needs to be done.

Agree with all these suggestions, thanks @dalonsoa !

We use tox, but nowhere it is said that it needs to be installed or how to do it. Indeed, there's no brief discussion about we using tox as the development environment

👍

Maybe some discussion on how to setup the virtual environment created by tox in VSCode, also working with the debugger.

Yeah it would be nice to give a full workthrough of how to set up with VS Code for those who have never used python and are used to e.g. the MATLAB IDE. If people prefer something else (vim, spyder, pycharm) then they can ignore that section

In the Installation section, it is said that tox -e dev installs all development options, however the Sundials solver is not installed.
About Sundials, there's no mention in that file at all. I guess it should be installed with tox -e pybamm-requires, right?

Is it possible to install sundials through tox? I thought it required separate steps @tlestang

If you activate the tox environment, within it there's no tox command so any subsequent tox -e {whatever} will fail.

So should tox be a dependency so it gets installed inside the env?

Travis CI is no longer used as the CI system. It should be removed from here and mention GitHub Actions instead.

👍

In general, the file might benefit form some content reorganisation, but that's just a purely subjective comment.

👍

Thanks Diego-  indeed it looks like the guidelines are not up to date with the latest tox workflow..

Is it possible to install sundials through tox?

Yes, with tox -e pybamm-requires which downloads and compiles Sundials. This is intended to be used on GNU/Linux platforms only: on MacOS, the recommended approach is to install sundials through homebrew as that's easier and more reliable. Last time I checked, versions of Sundials available in most GNU/Linux distros were too old so it must be built "manually". I think this is in the installation docs, but should also be in the contributing guidelines - or at the very least they should include a link to the installation docs.

So should tox be a dependency so it gets installed inside the env?

That feels a  bit redundant, maybe the way I designed the Tox workflow isn't great - I'm not sure at this time if installing tox in the development environment is common practice.
@dalonsoa do you have experience whith Tox?

I hadn't heard of tox until you used it here, so I've no idea of what's the normal use. I also think it is redundant, but I was mentioning the issue because approach described in the file is just not working. I think the user should either:

Activate the tox environment created with tox -e dev and then run the tests directly without invoking tox (eg. python run-tests.py --unit --folder all), or
Do not activate the tox environment and then run the tests using tox (eg. tox -e quick).

The later has the limitation that if the user wants to run the examples or run their own calculations, they cannot: they need to create and activate an environment, anyway.

I'd actually go with the second option.
Development work as well as running calculations/examples is done inside the tox environment.
Running the tests and building the documentation is done from outside this environment using tox -e tests, tox -e docs.
It realise it sounds odd to deactivate the environment before running the tests - but an advantage of tox is that it will create a new environment dedicated to the task at hand, isolated from the environment tox was invoked from (for instance only a small number of environment variables are inherited by the test environment). The tests would not have been ran from the development environment anyway. Does this make sense?

It might be a bit confusing when you are using an IDE that automatically activates the environment (VSCode, PyCharm) to get out of the environment to run the tests, though. Being that the case, I think it makes sense to install tox within the environment to benefit from both, being inside an environment for development and running examples and simulations and exploiting tox functionality and compartmentalization of environments for different tasks.
From: Thibault Lestang<EMAIL_ADDRESS>Date: Wednesday, 28 October 2020 at 08:08
To: pybamm-team/PyBaMM<EMAIL_ADDRESS>Cc: Alonso Alvarez, Diego<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [pybamm-team/PyBaMM] Review of CONTRIBUTING guidelines (#1211)
I'd actually go with the second option.
Development work as well as running calculations/examples is done inside the tox environment.
Running the tests and building the documentation is done from outside this environment using tox -e tests, tox -e docs.
It realise it sounds odd to deactivate the environment before running the tests - but an advantage of tox is that it will create a new environment dedicated to the task at hand, isolated from the environment tox was invoked from (for instance only a small number of environment variables are inherited by the test environment). The tests would not have been ran from the development environment anyway. Does this make sense?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/pybamm-team/PyBaMM/issues/1211#issuecomment-717769412, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABOQHLRKAUO4BV2CH6WNPODSM7GQJANCNFSM4S4JKSJQ.

That's a good point. Okay - let's add tox to the dependencies for the dev testenv then.
Just thought of a third approach: we could recommend the use of pipx to install tox. This allows the installation of python applications (tox, black, flake8, pylint...) in separate environments. For instance tox  would be available  from the dev env even if it's not installed in it. haven't tried it yet, but it looks widely used.
Here's a practical introduction to it: https://pycon.switowski.com/02-packages/pipx/

Mmmm… looks like an interesting tool for these kind of things (using it for black and flake8 is quite compelling!). Maybe we can give it a try ourselves and then decide?
There’s going to be an explosion of environments whenever something is run…
From: Thibault Lestang<EMAIL_ADDRESS>Date: Wednesday, 28 October 2020 at 09:36
To: pybamm-team/PyBaMM<EMAIL_ADDRESS>Cc: Alonso Alvarez, Diego<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [pybamm-team/PyBaMM] Review of CONTRIBUTING guidelines (#1211)
That's a good point. Okay - let's add tox to the dependencies for the dev testenv then.
Just thought of a third approach: we could recommend the use of pipxhttps://github.com/pipxproject/pipx to install tox. This allows the installation of python applications (tox, black, flake8, pylint...) in separate environments. For instance tox would be available from the dev env even if it's not installed in it. haven't tried it yet, but it looks widely used.
Here's a practical introduction to it: https://pycon.switowski.com/02-packages/pipx/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/pybamm-team/PyBaMM/issues/1211#issuecomment-717812989, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABOQHLQYHAC3O52NMYIVBYTSM7QXFANCNFSM4S4JKSJQ.

I've checked pipx this and it works quite nicely, to be honest. I would include this as a suggestion, though. In the end, it is yet another tool that is not directly used by PyBaMM, but rather by a tool actually used by PyBaMM.
In summary, I'd include the following options and let the user decide:

Install tox normally with pip (or conda), and then, later on, install tox within the environment.
Install tox with pipx so it is available system wide.

Both options results in the possibility of running things like tox -e quick with the environment activated.
What do you think?

That sounds good to me! If users don't install tox in the dev environment, they can still run the tests with pyhon run-tests anyway.

The CONTRIBUTING guidelines have several points that are either incomplete or outdated. These are the things I've noticed:

We use tox, but nowhere it is said that it needs to be installed or how to do it. Indeed, there's no brief discussion about we using tox as the development environment.
Maybe some discussion on how to setup the virtual environment created by tox in VSCode, also working with the debugger.
In the Installation section, it is said that tox -e dev installs all development options, however the Sundials solver is not installed.
About Sundials, there's no mention in that file at all.  I guess it should be installed with tox -e pybamm-requires, right?
If you activate the tox environment, within it there's no tox command so any subsequent tox -e {whatever} will fail.
Travis CI is no longer used as the CI system. It should be removed from here and mention GitHub Actions instead.
In general, the file might benefit form some content reorganisation, but that's just a purely subjective comment.

I'm happy to tackle these issues myself and open a PR with the changes, but wanted to check with you first and gather any other changes that needs to be done.

Yes I agree, initially I wasn't able to set up the virtual environment because of 'tox' in my commands which I had no idea that I had to install seperately, then I decided to run pybamm without creating a virtual environment and it worked without any problems. If you need any kind of help, I'll be happy to contribute and make the documentation (which is quite good right now) better for complete newbies.

That would be great. Feel free to open issues and/or PRs for anything you find that is missing

I am really sorry I had this abandoned. I was halfway the rewriting but other things came up. If you want to take on this, please go ahead. It will take me at least 2 more weeks to be able to go back to this issue.

Okay! I'll start working on the changes.

I have been experimenting with all the installation steps, I think they should go like this -

Installation without any virtual environment
a) forking and cloning the repository and installing the necessary dependencies.

Creating a virtual environment (Here 2 options as discussed above, using pip and pipx)
a) pip/pipx install tox
b) forking and cloning the repository
c) python -m tox -e windows-dev
d).tox\windows-dev\Scripts\activate.bat
e)pip install tox in the virtual env ( not required with pipx )

And lastly adding the link to your docs which explains installing sundial using homebrew in mac.
I'll update everything around these steps, should I make any changes in the above steps ( this is just an overview, I'll make them more clear in the actual file )?


Cannot access 'watcher' before initialization.  error

Trying the latest beta version 1.0.0-beta.2 to preview docs is not working :sadface:
~> npx @redocly/openapi-cli --version
1.0.0-beta.2
~> npx @redocly/openapi-cli preview-docs server/api/api.yaml
Using Redoc community edition.
Login with openapi-cli login or use an enterprise license key to preview with the premium docs.

  🔎  Preview server running at http://<IP_ADDRESS>:8080
(node:54683) UnhandledPromiseRejectionWarning: TypeError: Non-string provided as watch path: server/api/api.yaml,
    at unifyPaths (/Users/alifarooq/.npm/_npx/54683/lib/node_modules/@redocly/openapi-cli/node_modules/chokidar/index.js:94:11)
    at FSWatcher.add (/Users/alifarooq/.npm/_npx/54683/lib/node_modules/@redocly/openapi-cli/node_modules/chokidar/index.js:395:15)
    at Object.watch (/Users/alifarooq/.npm/_npx/54683/lib/node_modules/@redocly/openapi-cli/node_modules/chokidar/index.js:936:11)
    at Object.<anonymous> (/Users/alifarooq/.npm/_npx/54683/lib/node_modules/@redocly/openapi-cli/lib/cli/preview-docs.js:72:35)
    at Generator.next (<anonymous>)
    at fulfilled (/Users/alifarooq/.npm/_npx/54683/lib/node_modules/@redocly/openapi-cli/lib/cli/preview-docs.js:5:58)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:54683) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:54683) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Bundling...

Something went wrong when processing server/api/api.yaml:

  - Cannot access 'watcher' before initialization.

Version 0.12.16 work fine without any issues.
npx<EMAIL_ADDRESS>preview-docs server/api/api.yaml

Looks like https://github.com/Redocly/openapi-cli/pull/164 may have fixed this issue. Thanks @knidarkness!
npx<EMAIL_ADDRESS>preview-docs server/api/api.yaml
Using Redoc community edition.
Login with openapi-cli login or use an enterprise license key to preview with the premium docs.

  🔎  Preview server running at http://<IP_ADDRESS>:8080

Bundling...

  👀  Watching server/api/api.yaml and all related resources for changes

Created a bundle for server/api/api.yaml successfully

Closing as fixed.


Me

Type of issue
Typo
Description
[Enter feedback here]
Page URL
https://learn.microsoft.com/en-us/ef/core/providers/sql-server/?tabs=dotnet-core-cli&WT.mc_id=email
Content source URL
https://github.com/dotnet/EntityFramework.Docs/blob/main/entity-framework/core/providers/sql-server/index.md
Document Version Independent Id
af433371-e282-4b89-7b09-6caf43af83e0
Article author
@

Closing as no detail was provided in the issue.


Tray icon, app menu not working on Windows 10

Using Keybase GUI Version: 1.0.16-20160713110836+a1c00b3 on Windows 10:
Steps to reproduce:

Open up the tray by clicking on the keybase icon
Click "New private folder" button, OR "Open public folder"  button
Nothing happens
The menu in the top-left corner doesn't open anymore

I need to kill the tray process and start the app again to get the menu working. Other than that the whole keybase GUI on Windows doesn't really work.

Same issue as #3549

This seems to have been fixed in version 1.0.16-20160726182036+0c2a52a, update was rolled out today. All clickable UI elements seem to now work as intended. Please check whether this fixes it for you and close the issue if that's the case.

Thanks for the update! Checked, and at least the menu now seems to work again, even when clicking around the tray window.
No folders are opening though, when clicking on the "Open public folder"-button.

That's not what "Open public folder" does. You should see that if you press that button, a text box appears in its place where you can input a comma-separated list of users that have writing access to that folder. It's named misleadingly, I'll give you that. "New private folder" and "Open public folder" are doing the exact same thing, one for private, one for public folders.
To open your own public folder, click on the entry right below that button with your keybase username and profile picture on it. That didn't work either previously, but was fixed in the update.

Ah, OK, thanks for the clarification.
I had to restart the app to get the open folder to work. Seems to work now, so I'll close this ticket.
Keep up the great work!


Repair fpgabist DMA test for RC/DC

I won't merge until @nakulkorde gives me the all clear on auto rebase into master-x.
DMA bist finally works in 19.3: https://psg-sc-arc.sc.intel.com/arc/dashboard/reports/show_job/174954663

This one is ready now that master-x is in sync again. @OPAE/ci and @OPAE/tools please review.

Coverage remained the same at 78.744% when pulling 76c29b0aef7ef3bd9146e73853075a8a2c240e63 on mcadler/bist_dma_py into 440cf422117eb37a1c24acc5cfd677143aacec9a on master.

I can drop the bist_common changes and just keep the DMA part if you wish. The change to common is just a rearrangement of the existing code — functionality equivalent. I sort of agreed with your original comment that the new pattern makes more sense as the general case, so changed it as I thought you requested.

@buie1 here is fpgabist inside a VM: https://psg-sc-arc.sc.intel.com/arc/dashboard/reports/show_job/174956099


Warmup period, when should remotes be queried?

Current solution expects some kind of timer to have passed until you consider remote nodes to be mature.
However in a scenario where people join and leave frequently, new nodes might be worth quering for content even if they have not been around for long.
These problems are especially evident when doing testing where you connect to some node that have some data, and you are also a replicator, and immediately after connecting try to do a query.

Partial solution, added eager mode in eager mode in 57b8640ee53a25d81695f7175ca6747aa9dfe58a to force query all new nodes

It might be nice and allow for more intuitive/predictable behavior in certain circumstances if there was a setting where you could query remotes only if remote: true is set, regardless of whether or you and/or the remotes are still warming up.

Good point!


💡 collections: children of concepts should be automatically added to the collection

Description
children of concepts should be automatically added to the collection
( or is this already implemented ?)

This is not implemented.
On what use case where you thinking? Should this be a way to flatten a hierarchy to a list and access the list as collection?

just to make collections more convenient: one only needs add the top-most concept (in stead of entering every single narrower  concept) in the list, the hierarchy can still be kept.
To avoid unintentional additions, we could introduce a syntax, like v4c-bc:0000001+ (addition of a plus at the end of the IRI) to indicate, please also include narrower terms as specified in the vocabulary.

We need not only Excel -> SKOS but also SKOS -> Excel which may get difficult for this convenience feature. Another detail to consider: skos:broader / skos:narrower may form circles that need to be handled. Both are not transitive by default. This will get too complex in my opinion.
Another way is to find the narrower concepts and add them to a collection via a separate script. With the help of a SPARQL query or by filtering the RDF triples the concepts missing in a collection can be found easily. An update of the collection turtle file can then be submitted as PR. A new Excel file with the extended collection will be build by the pipeline.
Maybe we should add some jupyter notebooks that show how to do such things with rdflib.

I see your valid point, @dalito,
and I think a separate script (or jupyter notebook) will certainly help - it is a good idea, worth putting on the list of future features.


Add unit tests for chain syncing

This adds tests for db::storage::sync_chain and scanner::sync_chain.  I'm not certain they can be completed actually.
These tests help make sure forks/reorgs are handled correctly, and will help with ensuring no new bugs appear with "chain hardening".

Finally fixed linking issues.

Added tests for lws::scanner::run (inbound transaction outputs).


allow exclusion of resources using standard label

excludes any resources with the velero.io/exclude-from-backup=true label
fixes #1373
Signed-off-by: Adnan Abdulhussein<EMAIL_ADDRESS>

I guess we should also add some documentation about this new feature, where do we think that should go?

Not sure we have a place, but should go together with this: #1440

This ^ was supposed to be in response to:

I guess we should also add some documentation about this new feature, where do we think that should go?

Yeah, I'm not sure there's a great place to put docs for it right now -- agree it would make sense to go with https://github.com/heptio/velero/issues/1440, probably on a new page under "Use" in the TOC.

Yeah, I'm not sure there's a great place to put docs for it right now -- agree it would make sense to go with #1440, probably on a new page under "Use" in the TOC.

added a comment in #1440 to include this
Also updated based on review, PTAL!

@skriss really good, the existing tests are really helpful and pretty easy to work off of!


Add initial content of User Guide

Task progress list:

[x] Quick Start
[x] List Command
[x] Add Command
[x] Delete Command
[x] UI Mockup
[x] Saving and Loading Data

Closed as completed.


Run GenerateNavFilterEquations.m in matlab 2017a,Error！

When I ran GenerateNavFilterEquations.m using matlab 2017a, I got an error and I was wondering how I could fix this error(The function or variable 'QuatMult' is not defined) and run the script successfully.

it seems that you are in the default address of Matlab.
change the address where you copy the entire project. all folders and subfolders must be included in the matlab path.

@rafaelmendy Thank you for your help, the error has been resolved, because of the version problem, there is a library function error at runtime, after a little modification I solved. The link in ardupilot AP_NavEKF2_core.h (https://github.com/priseborough/InertialNav/blob/master/derivations/RotationVectorAttitudeParameterisation/GenerateNavFilterEquations.m) seems to have failed. I have opened this link before, but I am not sure if the content is the same as what I am running now. I remember the contents of this connection and can generate most of the C code for the EKF. I don't know if my memory is wrong.

I have the same problem，I use matlab 2016a
how you solve this, can you tell me.
thanks very much1


Terminal integration

At the moment we have the next 2 plugins:

terminal
processes

It doesn't make sense to have both.

plugin processes should be removed
output from plugin terminal should be redirected to the main output (session)
plugin terminal should be bound with each session (like processes plugin was)

Related #325 , #326


Move EntityHash related types into bevy_ecs

Objective
Reduce the size of bevy_utils (https://github.com/bevyengine/bevy/issues/11478)
Solution
Move EntityHash related types into bevy_ecs. This also allows us access to Entity, which means we no longer need EntityHashMap's first generic argument.

Changelog

Moved bevy::utils::{EntityHash, EntityHasher, EntityHashMap, EntityHashSet} into bevy::ecs::entity::hash .
Removed EntityHashMap's first generic argument. It is now hardcoded to always be Entity.

Migration Guide

Uses of bevy::utils::{EntityHash, EntityHasher, EntityHashMap, EntityHashSet} now have to be imported from bevy::ecs::entity::hash.
Uses of EntityHashMap no longer have to specify the first generic parameter. It is now hardcoded to always be Entity.

The migration guide could be changed to mention what needs to be adapter for the new changes, rather than the new changes themself. For example:

Migration guide

Uses of bevy::utils::{EntityHash, EntityHasher, EntityHashMap, EntityHashSet} now have to import them from bevy::ecs::entity::hash.
Uses of EntityHashMap no longer have to specify its first generic parameter. It is now hardcoded to always be Entity.

CI failure is real: looks like you forgot to feature flag imports.

It doesn't look like my problem. If it is, what am I doing wrong?


Allow throwing functions to be curried

Is there any reason why this hasn't been done? Seems like it would just require adding throws to the generated curry functions.
Usage would be like:
func foo(a: Int, b: Int) throws -> Int { ... }

let curried = curry(foo)
// curried: (a: Int) -> (b: Int) throws -> Int

try curried(1)(2)

Pretty sure I tried this at one point and it wasn't possible. If you can get it to compile I'd be happy to look at a PR.

Here's what I imagine the definition of a throwing 2-argument curry to be:
func curry<A, B, C>(f: (A, B) throws -> C) -> A -> B rethrows -> C {
  return { a in { b in { try f(a, b) } } }
}

But it doesn't compile:
error: repl.swift:1:54: error: only function declarations may be marked 'rethrows'
func curry<A, B, C>(f: (A, B) throws -> C) -> A -> B rethrows -> C {
                                                     ^

Without rethrows all uses of curry would need to be marked with try, so I'm not sure this is possible?

You're right. I'd be interested in better understanding why that error exists, but for now, I can get it working by overloading each curry function to have a throwing and non-throwing version:
func curry<A, B>(function: (A) -> B) -> (A) -> B {
    return { (`a`: A) -> B in function(`a`) }
}
func curry<A, B>(function: (A) throws -> B) -> (A) throws -> B {
    return { (`a`: A) throws -> B in try function(`a`) }
}

func curry<A, B, C>(function: (A, B) -> C) -> (A) -> (B) -> C {
    return { (`a`: A) -> (B) -> C in { (`b`: B) -> C in function(`a`, `b`) } }
}
func curry<A, B, C>(function: (A, B) -> C) -> (A) -> (B) throws -> C {
    return { (`a`: A) -> (B) throws -> C in { (`b`: B) throws -> C in function(`a`, `b`) } }
}
// and so on...

I'd be happy to put together a PR that does this, but I understand if you'd rather not double the number of functions being generated.

I definitely don't want to double the number of functions. That would seriously limit the number of implementations we could reasonably ship (we're already limited by exponential compile times). My assumption is that this would add an unreasonable amount of complexity for the compiler.
I'm not sure what the actual answer here is.

I tried rewriting the functions using the curry syntax being removed in Swift 3:
func curry<A, B>(function: (A) throws -> B)(_ a: A) rethrows -> B {
    return try function(a)
}

func curry<A, B, C>(function: (A, B) throws -> C)(_ a: A)(_ b: B) rethrows -> C {
    return try function(a, b)
}

These compile, but rethrows doesn't seem to be working properly, i.e. all types returned from these functions are throwing. In testing, I came across this:
func a2(x: Int) -> Int {
    return x+2
}
let ca2: Int -> Int = curry(a2)
// Error: 'curry' produces '(A) -> (B) throws -> C', not the expected contextual result type 'Int -> Int'

Given that error message, I'm guessing that under the old currying syntax, this edge case was handled by turning rethrows into throws, and there is (still) no way to return a rethrowing closure.
At the moment I'm looking to see if this limitation has been discussed before, or if I should file a Swift bug. Regardless, seems like the current answer to my original question is "it's impossible without further overloading" :(

I'm going to go ahead and close this due to inactivity but please feel free to reopen if this is still an issue.


how to compile asncdc.c file on windows using db2 community edition

Hi
I'm doing PoC synching db2 data to postgres. I'm able to sync db2 data with postgres using db2 container image because the ASN agent is enabled by default. however not able to enable ASN agent in DB2 community edition installed on windows because unable to compile asncdc.c file on windows.
kindly help how  to compile asncdc.c on windows using db2 community edition. I'm getting below error thought the utilcli.c and stdio.h are in path.
We are planning to migrate existing db2 data to postgres using debezium CDC. is it mandatory to have infosphere replication license to enable CDC on existing db2?
C:\Program Files\IBM\SQLLIB\samples\cli>bldrtn asncdc
Microsoft (R) C/C++ Optimizing Compiler Version 19.28.29336 for x64
Copyright (C) Microsoft Corporation.  All rights reserved.
asncdc.c
asncdc.c(1): fatal error C1083: Cannot open include file: 'stdio.h': No such file or directory
utilcli.c
utilcli.c(54): fatal error C1083: Cannot open include file: 'stdio.h': No such file or directory
Generating Code...
Microsoft (R) Incremental Linker Version 14.28.29336.0
Copyright (C) Microsoft Corporation.  All rights reserved.
LINK : fatal error LNK1181: cannot open input file 'asncdc.obj'
The system cannot find the file specified.

I'm able to generate asncdc.dll file however when i execute below statement it gives below error
SQL0444N  Routine "ASNCDC.ASNCDCSERVICES" (specific name "ASNCDCSERVICE") is
implemented with code in library or path "\asncdc", function "asncdcservice"
which cannot be accessed.  Reason code: "6".  SQLSTATE=42724

I'm able to generate asncdc.dll file however when i execute below statement it gives below error
SQL0444N  Routine "ASNCDC.ASNCDCSERVICES" (specific name "ASNCDCSERVICE") is
implemented with code in library or path "\asncdc", function "asncdcservice"
which cannot be accessed.  Reason code: "6".  SQLSTATE=42724

Hi,
I am sorry we are testing with Linux only so unfortunately we can't geve you any relevant advice right now.
If you'd be able to get it running we'd really appreciate if you could share the procedure so we can document it.
Thanks

Hi,
I am sorry we are testing with Linux only so unfortunately we can't geve you any relevant advice right now.
If you'd be able to get it running we'd really appreciate if you could share the procedure so we can document it.
Thanks

Hi Jiri Pechanec
Many Thanks .  I'm able to capture Db2 data change on windows. Though I compiled and generated .dll  that cannot accessed by db2 function gives above error.
However if I run the below command in db2 CLI then CDC works perfectly. Basically I executed same  command manually what we are doing with C program.
asncap capture_server=test capture_schema=ASNCDC
I'm preparing the document will share once it's done.
Thanks

Hi Jiri Pechanec
Many Thanks .  I'm able to capture Db2 data change on windows. Though I compiled and generated .dll  that cannot accessed by db2 function gives above error.
However if I run the below command in db2 CLI then CDC works perfectly. Basically I executed same  command manually what we are doing with C program.
asncap capture_server=test capture_schema=ASNCDC
I'm preparing the document will share once it's done.
Thanks

Hi Jiri Pechanec
I've quick question, in DB2 separate schema called ASNCDC is being created. to capture the change which means we need additional storage at DB2 side right. if we capture all tables then we may need to add same storage size what we have currently whereas this additional schema not required for other database like Postgres or mysql.
Please advice.

Could you please move the discussion to https://groups.google.com/g/debezium ?


Added my username as a maintainer

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/india/blob/main/CONTRIBUTING.md.

Which change are you proposing?

[ ] Adding/updating open source project/s
[x] Adding/updating maintainer/s

Adding/updating open source project/s

[ ] The project was built in or receives significant contributions from India
[ ] The project has at least 100 stargazers

Why should this open source project be featured?

Please replace this line with an explanation of why you think these changes should be made.

Adding/updating maintainer/s

[x] Maintainer's location on their GitHub profile indicates as India
[x] Maintainer has approved this PR to be included on the website

What open source project/s does the individual maintain?
https://github.com/MrKrishnaAgarwal/Drone-Course
https://github.com/MrKrishnaAgarwal/Random-Challenges
https://github.com/MrKrishnaAgarwal/30DaysOfCode
https://github.com/MrKrishnaAgarwal/Hacktoberfest2022

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing page, or at least not a single one

My changes adds me as a maintainer on the github India page

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

@mvkaran Please review this PR, and approve it.
I also want to get connected with you

Thanks for your contribution @MrKrishnaAgarwal 💙
Unfortunately, we would be unable to approve your PR currently since we are listing maintainers whose project/s have a significant traction within the community.
All the best in your open source journey!


SliverGeometry is not valid: The "maxPaintExtent" is less than the "paintExtent". with MultiSliver usage

Usage MultiSliver with SliverAppBar leads to exception SliverGeometry is not valid: The "maxPaintExtent" is less than the "paintExtent".
Scaffold(
    body: CustomScrollView(
      slivers: [
        const SliverAppBar(
          pinned: true,
          title: Text('Sample'),
        ),
        MultiSliver(
          children: [
            SliverToBoxAdapter(
              child: Container(
                height: 30, ///<--- changing this value to height bigger than app bar fixes issue
              ),
            ),
          ]
        ),
        MultiSliver(
            children: [
              SliverList(
                delegate: SliverChildBuilderDelegate(
                  (context, index) {
                    return Container(
                      height: 50,
                      width: double.infinity,
                      color: index.isEven ? Colors.red : Colors.blue,
                    );
                  },
                  childCount: 20,
                ),
              ),
              const SliverToBoxAdapter(
                child: Divider(),
              ),
            ]
        ),
      ],
    ),
  );

Flutter version: 2.0.5
Device: Android 11 (API 30) (emulator)

Hey @TatsuUkraine thanks for filing this.
Seems like an interesting bug indeed, I'll look into it ASAP
Does this affect how it renders or only show the error in the console?

Hey. Thank you for quick response) yes it affects. Second MultiSliver just stops moving

In more complex example , where I noticed this issue, it looks to me that other content is actually moving during the scroll, except multi sliver that goes just after a small one, but I could be wrong, didn't pay attention a lot) last hour was trying to reproduce this issue with very primitive example that I've sent you))

Could you test using PR #31

it seems it fixes the issue

Just published the fix as 0.2.2

thank you for fast reaction)

I faced the same Issue again today on the latest version (0.2.4)
https://user-images.githubusercontent.com/31625592/121203605-86e85000-c876-11eb-9283-94907cdfe4ca.mp4
As this weird behaviour is occuring I get the same error message.
"SliverGeometry is not valid: The "maxPaintExtent" is less than the "paintExtent"."
MultiSliver(
          children: <Widget>[
            SliverPersistentHeader(
              pinned: true,
              delegate: SliverHeader(
                title: title,
                index: index,
              ),
            ),
            SliverToBoxAdapter(child: formView),
            SliverPersistentHeader(
              pinned: true,
              delegate: SubtitleHeader(
                subtitle: subtitle,
                index: index,
              ),
            ),
          ],
        )

Flutter version: 2.2.1

@felixjaehn please open a new issue and include a minimal runnable code sample that reproduces the problem


Change Global Affiliate Network link on CC.org homepage to network.cc.org

This is a thing for @little-wow
Please change the "Global Affiliate Network" text at the top the CC homepage to "Creative Commons Global Network" and link it to http://network.creativecommons.org

Thanks.

Should be done. (Give it about an hour to update.) We lost the little icon, but I think that's okay.

Was hoping to keep the little icon as it called attention to it but if we can't get it back, that's ok.
Minor change: can the text read "Creative Commons Global Network" instead of just "Global Network"?
Is that's too much, how about "CC Global Network"?

So I'm not sure how to get the icon back because I had to change the link type to an external link and I think the icon came from it being an internal link. We can ask the dev team to return it if you think it's essential?
I changed the menu – see how it looks when it updates.

Not essential, but maybe @hugosolar have an idea how to get it back

Ok, now we've got the icon back. It was called in a silly way in the stylesheet


Training and validation loss graphs

Please post the training loss graph and validation loss graph for your trained model. I am using as a reference for my dissertation. It will be very helpful.

My apologies for not getting back to you on this - unfortunately, these graphs are lost to the sands of time, as I did not plot them out originally.

Closing out since no further comments.


Make BufferedImageFactory.fromNDArray synchronous

I made this PR based on this comment by @steinhae in #1278.
I also ran a quick experiment to verify the findings:
try (NDManager manager = NDManager.newBaseManager()) {
  NDArray array = manager.randomInteger(0, 255, new Shape(3, 400, 400), DataType.INT32).toType(DataType.UINT8, true);
  ImageFactory factory = ImageFactory.getInstance();
  for (int i=0; i<5000; i++) {
      factory.fromNDArray(array);
  }
}

Found that this change would reduce the time to run this test from about 62 seconds to about 22 seconds.

Codecov Report

Merging #1339 (03cdad5) into master (bb5073f) will decrease coverage by 0.55%.
The diff coverage is 54.92%.

@@             Coverage Diff              @@
##             master    #1339      +/-   ##
============================================
- Coverage     72.08%   71.53%   -0.56%
- Complexity     5126     5144      +18
============================================
  Files           473      478       +5
  Lines         21970    22295     +325
  Branches       2351     2394      +43
============================================
+ Hits          15838    15949     +111
- Misses         4925     5128     +203
- Partials       1207     1218      +11

Impacted Files
Coverage Δ

.../java/ai/djl/modality/cv/BufferedImageFactory.java
72.68% <ø> (+0.23%)
:arrow_up:

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <ø> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../modality/cv/translator/YoloTranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...i/djl/modality/cv/translator/YoloV5Translator.java
5.69% <0.00%> (ø)

...odality/cv/translator/YoloV5TranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...n/java/ai/djl/modality/nlp/bert/BertTokenizer.java
92.68% <0.00%> (-2.32%)
:arrow_down:

...modality/nlp/embedding/TrainableWordEmbedding.java
45.00% <0.00%> (-3.34%)
:arrow_down:

...pi/src/main/java/ai/djl/ndarray/BytesSupplier.java
54.54% <0.00%> (-12.13%)
:arrow_down:

... and 96 more

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 fa7ff1d...03cdad5. Read the comment docs.


Airbar with chromium

Hi,
I am planning to use Airbar with chromium. I connected it to the rpi's usb and the touch works perfectly on using android on pi and raspbian but on chromium, the screen goes black as soon as i connect the touchbar. seems like other people have problem with touchscreens as well. any help on this?
PS. Airbar is detected as a HID touchscreen device in windows.

we don't own one of these, if you can't provide more meaningful info or logs, it's unlikely that we can be any helpful.


Don't assume sys.stdout.encoding is not None

Supply the same default as Python will - 'ASCII' - in this case.
I ran into this issue when running wsgidav within PyCharm.  Python does not detect the output encoding and sys.stdout.encoding is None, which causes an error inside safeReEncode.

@mar10 Yes Sure......If you can provide me the build with the fix......I can test it out...........

Do you have any evidence that the values being logged here will ever contain non-ASCII characters?
Two of cases I am quite sure will not - URLs must be ASCII only, any non-ASCII characters must be escaped using %XX.
The one I'm not sure about is the getUserInfo one.  I can't recall what that field is supposed to contain.

I suggest that @vinay-is describes the concrete problems with the current solution in #95.
We can discuss there. This PR is already merged and closed anyway.

PR #87 was released with v2.3.0


write EPIPE error in when doing production build in docker

Hi there, I've got FE in docker container and running build there ends with following error from time to time. Anyone got idea what's causing it?
syscall: 'write'
code: 'EPIPE',
errno: 'EPIPE',
at processTicksAndRejections (internal/process/task_queues.js:79:11) {
at internal/child_process.js:810:39
Emitted 'error' event on ChildProcess instance at:
at /frontend/node_modules/terser-webpack-plugin/dist/TaskRunner.js:113:86
at enqueue (/frontend/node_modules/terser-webpack-plugin/dist/TaskRunner.js:89:35)
at TaskRunner.runTask (/frontend/node_modules/terser-webpack-plugin/dist/TaskRunner.js:41:26)
at JestWorker._callFunctionWithArgs (/frontend/node_modules/terser-webpack-plugin/node_modules/jest-worker/build/index.js:178:23)
at Farm.doWork (/frontend/node_modules/terser-webpack-plugin/node_modules/jest-worker/build/Farm.js:56:12)
at new Promise ()
at /frontend/node_modules/terser-webpack-plugin/node_modules/jest-worker/build/Farm.js:90:14
at Farm._push (/frontend/node_modules/terser-webpack-plugin/node_modules/jest-worker/build/Farm.js:159:12)
at Farm._enqueue (/frontend/node_modules/terser-webpack-plugin/node_modules/jest-worker/build/Farm.js:152:10)
at Farm._process (/frontend/node_modules/terser-webpack-plugin/node_modules/jest-worker/build/Farm.js:129:10)
at WorkerPool.send (/frontend/node_modules/terser-webpack-plugin/node_modules/jest-worker/build/WorkerPool.js:32:34)
at ChildProcessWorker.send (/frontend/node_modules/terser-webpack-plugin/node_modules/jest-worker/build/workers/ChildProcessWorker.js:330:17)
at ChildProcess.target.send (internal/child_process.js:677:19)
at ChildProcess.target._send (internal/child_process.js:806:20)
Error: write EPIPE
^
throw er; // Unhandled 'error' event

Happened the same exact error to me. Rollbacking (that is, fixing version) react-scripts to 3.4.0 was enough to get rid of the problem.

That's weird because we're running on version 3.4.0

Sorry @ketysek, I didn't intend to press the "Submit" button on my comment yet.

@ketysek I got rid of this problem by removing dangling images and stalled containers.
In my case, the "front-end" is just a temporary container running npm build (it's a create-react-app), which produces a build folder shared with a web server container via volumes. After this command finishes, it is expected for the container to exit, but I found that somehow some of these containers were kept running, despite the command they were running had finished long ago.
So, killing all the stalled ones (docker container ls to list all of them) and doing some clean up in my images fixed the problem for me. I still don't know what created these ghost containers in the first place, but it might be related with the fact that is a CI server that builds the front-end on every push... so maybe it's a weird race condition or just something related with the hardware on which Docker is running, no other servers (which receive pushes less often) have this problem

may be an issue with memory. Add a memory constraint to node, e.g. --max-old-space-size=1024. The flag can be added a number of ways, i.e. .npmrc, NODE_OPTIONS env variable, etc

@bfraz I'm not able to set it ... or shouldn't I see it via process.env? I've tried both .npmrc and also in scripts "start": "NODE_OPTIONS=--max_old_space_size=4096 react-scripts start"

was there ever any real solution to this?  I experience this on every build after 3.2.0 ( have tried 3.3.0, 3.3.1, 3.4.0, 3.4.3

@jdwilkin in my case when I see this error happening it's usually out of memory situations, the machine I'm using has 2GB only and my CRA bundles a bunch of heavy packages.

I ran into a very simular problem where yarn build worked just fine on my local machine, but when running the same command inside a docker build . command, i received EPIPE errors simular to above.
My issue was relatively easy to solve by creating a .dockerignore file to prevent copying of the node_modules on the local machine onto the docker image, like so:
// file  .dockerignore
node_modules
dist

// file: dockerfile
# build stage
FROM node:lts-alpine as build-stage
WORKDIR /app
COPY package*.json ./
COPY yarn.lock ./
RUN yarn install
COPY . .
RUN yarn build

# production stage
FROM nginx:stable-alpine as production-stage
COPY --from=build-stage /app/dist /usr/share/nginx/html
EXPOSE 80
CMD ["nginx", "-g", "daemon off;"]

Hope this helps someone else.


Use OS X friendly functions [phab:D1179]

Created by salguarnieri at 2016-12-09 01:39:57:
mktemp, stat, and others behave differently on OS X and Linux, so use
functions and command line options that are present on all systems.

salguarnieri wrote at 2016-12-09 01:45:06:
bazel test //kythe/cxx/tools:kindex_file_generator_test has been run successfully on OS X and Linux

shahms accepted this revision at 2016-12-09 01:45:28

salguarnieri closed via commit a8b5b5f8a1c5b2778c84c391fbc77d67f3878890 at 2016-12-09 02:06:45


No schema selected?

When I try to run db-migrate up using this, I'm getting this error. Are there docs showing how to use this library?
$ db-migrate up
[ERROR] AssertionError [ERR_ASSERTION]: ifError got unwanted exception: no schema has been selected to create "migrations" in
    at module.exports (/src/src/node_modules/db-migrate/lib/commands/helper/assert.js:9:14)
    at /src/src/node_modules/db-migrate/lib/commands/up.js:29:16
    at tryCatcher (/src/src/node_modules/bluebird/js/release/util.js:16:23)
    at Promise.errorAdapter (/src/src/node_modules/bluebird/js/release/nodeify.js:35:34)
    at Promise._settlePromise (/src/src/node_modules/bluebird/js/release/promise.js:566:21)
    at Promise._settlePromise0 (/src/src/node_modules/bluebird/js/release/promise.js:614:10)
    at Promise._settlePromises (/src/src/node_modules/bluebird/js/release/promise.js:689:18)
    at Async._drainQueue (/src/src/node_modules/bluebird/js/release/async.js:133:16)
    at Async._drainQueues (/src/src/node_modules/bluebird/js/release/async.js:143:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/src/src/node_modules/bluebird/js/release/async.js:17:14)
    at runCallback (timers.js:696:18)
    at tryOnImmediate (timers.js:667:5)
    at processImmediate (timers.js:649:5)
    at Connection.parseE (/src/src/node_modules/pg/lib/connection.js:553:11)
    at Connection.parseMessage (/src/src/node_modules/pg/lib/connection.js:378:19)
    at Socket.<anonymous> (/src/src/node_modules/pg/lib/connection.js:119:22)
    at Socket.emit (events.js:182:13)
    at addChunk (_stream_readable.js:283:12)
    at readableAddChunk (_stream_readable.js:264:11)
    at Socket.Readable.push (_stream_readable.js:219:10)
    at TCP.onread (net.js:635:20)

Your database.json is most probably not correctly setup. Check your configuration, look in the docs for examples.


BlitzWolf BW-SHP15 wrong readings (TS011F)

Hi,
The reading on the BlitzWolf BW-SHP15 are very wrong :(
In the photo there is a laptop charging, and shows 0.2V and 363A ...

Device info:
TS011F
by _TZ3000_mraovvmm
Running HomeAssistant (latest;2021.12.9) as docker on a Synology NAS
Thank you!

Hi,
After removing & adding the device again, the values now appears to be correct.
Thank you.


chore(templates/arc): node 18

arc docs don't mention it, but node 18 is supported
https://github.com/mcansh/remix-arc-node-18
https://cl2vu5dvj5.execute-api.us-west-2.amazonaws.com

Closes: #

[ ] Docs
[ ] Tests

Testing Strategy:

Nice!  I do see it mentioned here, unsure why it's not the default by now though 🤷


Elife interpolated from elife map from private repo

Changes only affect the ER model in xenon/x1t_sr1.py
The electron lifetime is interpolated from the electron lifetime map pulled from the XENONnT/Flamedisx private repository based on the event time of the event and added as a column in add_extra_columns. Subsequently, also in add_extra_columns, the cs2 value is calculated using this interpolated electron lifetime instead of the default value of 641us which was previously hardcoded in.
The electron lifetime information is then used in model function electron_detection_eff as a positional argument. It was previously a keyword argument (nuisance parameter) which had a default value of 641us.
Performance of this new model, along with ks p-value distributions from multiple MC toys, is documented in Convo note (under flamedisx tag).

@CristianAntochi, since this only affects the XENON models, would you like to take a look?

actually this is just 1T model and you can see the convo note as well cause I attached the flamedisx tag as well.


Connecting/Register to a community should check for the UID

0.11 is only looking for the pubkey. We should also check for the UID.
If the UID is present but with a different pubkey, we should raise an error.
If the pubkey is present but with a different UID, we should raise an error too.
The problem should be explicitly explained.

@M5oul, can you do some review on the feature please ? :)

After a cache cleaning, I think I can add the fair_money account anymore and I could add the meta_brouzouf account.

Any message ?

Nope!

Maybe it was related to the Identities registry bug I fixed this morning...


fix: time on mobile view

Close #997
I used the hover media query to display the full time on devices with touch screen (not able to hover)
** On Desktop **

** On Tablet **

I love this type of contribution that help with accessibility.
Thanks @dragnoir 🙏

@dragnoir Great work! Could you please fix linting so we ship it?

@He1DAr linting fixed.

:tada: This PR is included in version 1.110.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Allow splitting on "happy path" of common rules

In discussion with @tothtamas28 we realized that
a) overflows and underflows are rare issues in practice that you might want to investigate separately from the happy path,
b) in theory overflows and underflows can happen in a myriad of places in EVM code, and
c) modulo make SMT solving a lot harder.
We had the idea that it would be great -- at least for our project -- to always split on chop operations, where we would get one path where there are no chop present in the constraints.
A way to do this would be to modify the chop rule
rule chop ( I:Int ) => I               requires          0 <=Int I andBool I <Int pow256
 [concrete, smt-lemma]
rule chop ( I:Int ) => I modInt pow256 requires notBool (0 <=Int I andBool I <Int pow256) [concrete, smt-lemma]

Or something similar (we could also use priority on the rule which strips the modulo. This would then be handled separately by a Markdown code selector tag to not interfere with the ordinary semantics.
The same approach can be taken with other problematic rules. While simplifications and lemmas can often help, an approach like this could let us more aggressively find and explore happy paths and gather all the constraints on them. Ideally this type of splitting behavior would be hidden behind a flag or configuration choice.

@tothtamas28 Is it worth keeping this on the board? Given where we are with the MEV Searcher, should I move it to the Zombie list and we resurrect it when the Searcher comes back into play, or is there significant applicability to other projects that warrants keeping it around?