helm_remote: Wait for namespace to exist before moving on

Code:
helm_remote('kafka',
  repo_name='bitnami',
  repo_url='https://charts.bitnami.com/bitnami',
  version='12.17.3',
  release_name='kafka',
  namespace='kafka',
  create_namespace=True,
  set=[
    'deleteTopicEnable=true',
    'defaultReplicationFactor=1',
    'offsetsTopicReplicationFactor=1',
  ]
)
k8s_resource(
  'kafka',
  labels=["databases"]
)
k8s_resource(
  'kafka-zookeeper',
  labels=["databases"]
)

Error:
STEP 1/1 — Deploying
     Applying YAML to cluster

Build Failed: namespaces "kafka" not found

It seems that sometimes it works and sometimes does not. When I triggered update via UI (on kafka resource) it worked. But , the first time when I saved Tiltfile the error occurred.

Hmm, I found helm_resource extension and it looks much better 🤔

Hi @tomislater
I've checked the helm_remote extension, and it should create the namespace before building anything. (https://github.com/tilt-dev/tilt-extensions/blob/master/helm_remote/Tiltfile#L139)
My best guess atm is that your cluster hasn't finished creating the namespace by the time the build starts, which would explain why it works on the second try, when you run helm_remote from the UI.
What does your tilt doctor output say?

Hi @lianmakesthings
Yeah, I also have checked the code and it looks that sometimes k8s needs more time to create a namespace thus the error. I should use something like wait_for and wait for creating a namespace.

Yeah, you could use the namespace extension to create the namespace yourself and then wait.
https://github.com/tilt-dev/tilt-extensions/tree/master/namespace
Although, I do believe it would be worth implementing waiting for namespace creation in the helm_remote extension itself.

Is there anything missing from your repro case here? I tried it out with tilt ci and got this output:
uncategorized │
uncategorized │ Initial Build • uncategorized
uncategorized │ STEP 1/1 — Deploying
uncategorized │      Applying YAML to cluster
uncategorized │      Objects applied to cluster:
uncategorized │        → kafka:namespace
uncategorized │        → kafka:serviceaccount
uncategorized │        → kafka-scripts:configmap
uncategorized │
uncategorized │      Step 1 - 0.30s (Deploying)
uncategorized │      DONE IN: 0.30s

i.e., that there's usually a list of resources it's applying in the output (and there's not in your repro case, which might be a pointer to something going wrong).
usually tilt is pretty smart about applying the namespace before other resources. There also might be something wrong with Tilt's dependency ordering. But we also haven't heard any reports like this recently (despite a ton of users who use tilt to install Kafka), so I'm wondering if there's something unique about your dev env that might be triggering this bug.

@nicks I try to debug this case today and provide more info.

Btw, kafka was only an example. I can reproduce this with other charts also. I am still debugging this.

My Tiltfile:
version_settings(constraint='>=0.23.5')

load('ext://namespace', 'namespace_create', 'namespace_inject')
load('ext://local_output', 'local_output')
load('Tiltfile-dockerfile', 'dockerfile_contents')

secret_settings(disable_scrub=True)

include('Tiltfile-databases')
...

And in this Tiltfile-databases I put this code:
helm_remote('redis',
  repo_name='bitnami',
  repo_url='https://charts.bitnami.com/bitnami',
  version='15.4.0',
  release_name='test-redis',
  namespace='test-redis',
  create_namespace=True,
  set=[
    'architecture=standalone',
    'auth.enabled=false',
    'master.readinessProbe.enabled=false',
    'master.livenessProbe.enabled=false',
    'master.persistence.enabled=false',
  ]
)

I use tilt up --debug --verbose --klog 0 and in dashboard for this resource I see this:
Building with BuildOrder: BuildOrder{ *buildcontrol.LiveUpdateBuildAndDeployer *buildcontrol.DockerComposeBuildAndDeployer *buildcontrol.ImageBuildAndDeployer *buildcontrol.LocalTargetBuildAndDeployer }
Trying to build and deploy with *buildcontrol.LiveUpdateBuildAndDeployer

Falling back to next update method…
REASON: no targets for Live Update found

Trying to build and deploy with *buildcontrol.DockerComposeBuildAndDeployer

Falling back to next update method…
REASON: DockerComposeBuildAndDeployer does not support target type model.K8sTarget

Trying to build and deploy with *buildcontrol.ImageBuildAndDeployer
STEP 1/1 — Deploying
     No images injected into Kubernetes YAML
     No images injected into Kubernetes YAML
     No images injected into Kubernetes YAML
     Applying YAML to cluster

Build Failed: namespaces "test-redis" not found

Additionally, because my laptop does not have enough cpu/ram to run docker and 4k monitor... I set up k8s cluster via k3d on my local PC and I use local network to reach my PC from my laptop. Tilt use the right context of course.
.kube/config:
...
- cluster:
    certificate-authority-data: ***hidden***
    server: https://<IP_ADDRESS>:35027
  name: k3d-local-cluster
...

With this code I can achieve the same error:
namespace_create("test-redis")
helm_remote('redis',
  repo_name='bitnami',
  repo_url='https://charts.bitnami.com/bitnami',
  version='15.4.0',
  release_name='test-redis',
  namespace='test-redis',
  # create_namespace=True,
  set=[
    'architecture=standalone',
    'auth.enabled=false',
    'master.readinessProbe.enabled=false',
    'master.livenessProbe.enabled=false',
    'master.persistence.enabled=false',
  ]
)

Looks like a problem with ordering in Tilt? 🤔

Btw, output from tilt doctor:
➜ tilt doctor
Tilt: v0.23.5, built 2022-01-07
System: darwin-amd64
---
Docker
- Host: Error: Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?
- Compose Version: v2.2.1
---
Kubernetes
- Env: k3d
- Context: k3d-local-cluster
- Cluster Name: k3d-local-cluster
- Namespace: default
- Container Runtime: containerd
- Version: v1.21.7+k3s1
- Cluster Local Registry: {Host:localhost:5432 hostFromCluster:local-registry:5000 SingleName:}
---
Thanks for seeing the Tilt Doctor!
Please send the info above when filing bug reports. 💗

The info below helps us understand how you're using Tilt so we can improve,
but is not required to ask for help.
---
Analytics Settings
--> (These results reflect your personal opt in/out status and may be overridden by an `analytics_settings` call in your Tiltfile)
- User Mode: opt-in
- Machine: 5483cc923cdd5a03f0494263f5e0ec4f
- Repo: 80k0c/8Rc0EuL6DresFzSw==

Even if I refresh my Tiltfile (via UI) nothing happens. I need to trigger update (via UI also):

When I use tilt ci everything works smoothly. When I use tilt up at the first time also everything works correctly.
But when I comment the whole section, tilt refreshes itself and remove test-redis namespace from cluster. And when I uncomment the commented section, tilt refreshes itself, but now it has the problem.
So, it looks like a problem with live updates? 🤔

How to reproduce it:

run tilt up
add helm_remote with create_namespace=True to your Tiltfile
save a file
wait
you should get an error: Build Failed: namespaces "<your_namespace>" not found

I can also get this error using namespace_create. So, it looks like a problem with Tilt.

ooooooh the k3d info is interesting.
K3d tries really hard to be fast. Sometimes they make a mistake and say they're done doing something when it's still ongoing in the background. But that's just a hunch...will try to reproduce with a k3d cluster.

I'm interested in this as well, mainly having a way for helm_remote to wait for other resources to be ready. For example, we have a helm_remote for PostgreSQL and Redis and we would like that services that depend on these to only start after the aforementioned are ready.
I tried with wait_for_it but didn't do what I needed.

Did this get anywhere? I am still running into this issue.

No one's been able to come up with a repro case. tilt already confirms with the apiserver that the namespace exists before applying other resources, so it's not clear what additional thing tilt should wait on.


[DC-1942] Delete note_nlp table from AOU_REQUIRED and MAPPED_CLINICAL tables

DC-1942 Remove note_nlp from table lists

It was nice adding 'note_nlp' in 'other_cdm_tables' will still include it in 'cdm_tables' list for the integration tests. I did not know that excluding it in 'clinical_data_tables' will be fine, but if there is no mapping table for 'note_nlp', can it be included in the 'unmapped_clinical_data_tables' along with 'death'? So, it can be part of 'clinical_data_tables' list..
I'm not sure if it'll add any value or not.. please correct me if I'm wrong.

This is great feedback and I think makes sense! Will update this.


[BACKLOG-16360] Add invalid characters check for run configurations

Master PR: https://github.com/pentaho/pentaho-kettle/pull/3870
7.1 PR: https://github.com/pentaho/pentaho-kettle/pull/3878

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -B -fn -f plugins/pdi-engine-configuration/ui/pom.xml clean install && mvn -B -f plugins/pdi-engine-configuration/ui/pom.xml site

Cleanup Commands
mvn -B -f plugins/pdi-engine-configuration/ui/pom.xml build-helper:remove-project-artifact

Changed files
  plugins/pdi-engine-configuration/ui/src/main/java/org/pentaho/di/engine/ui/RunConfigurationDialog.java
  plugins/pdi-engine-configuration/ui/src/main/java/org/pentaho/di/engine/ui/RunConfigurationPopupMenuExtension.java
  plugins/pdi-engine-configuration/ui/src/main/resources/org/pentaho/di/engine/ui/messages/messages_en_US.properties

@mdamour1976 @dkincade @kcruzada


[Chatdown] Error parsing chatdown files with \n instead of \r\n as line breaks

Error when parsing .chatdown file with \n instead of \r\n for line breaks.
Repro on latest version (1.0.4). Did not happen with version 1.0.2.
Error message:
Error: Malformed configurations options detected. Options must be in the format optionName=optionValue
Possible issue
https://github.com/Microsoft/botbuilder-tools/blame/master/Chatdown/lib/index.js#L435
Possible solution
Should the line split be done using a RegEx that works on multiple systems?
E.g.: var parts = fileContents.split(/\r?\n/)

This has been fixed


Fix valuesForUndefined actual query being executed. Fixes #1268

This was a doosey and a half.. The main issue was that prepBindings was not being called in the QueryCompiler.
Interestingly however, some prepBindings that weren't being called also resulted in invalid tests, specifically for postgres. Reason for this is that the pg module and knex's own prepBindings -> prepareValue converts Numbers to Strings prior to executing the query. I'm mentioning this so you know why I had to change a few tests.
I also had to do a hacky-fix to one of the tests that involved undefined values for sqlite3 to prevent the TypeError being thrown. Since it does not support valueForUndefined, the entire test crashed after applying prepBindings, which was to be expected.
cc @rhys-vdw @elhigu @chrisfrancis27

I'll review this. I didn't even knew that other drivers had prepBindings too I thought it was only pg thing.

@wubzz I'm not sure yet, how and where bindings should be prepared... do you have any new ideas how to do this consistent way? I'm not a big fan of putting that call to every dialect either...

@elhigu :+1: I also dislike keeping such fundamental function calls seperated in each dialect, because that's doomed to fail one day when someone adds a new dialect and forgets to call prepBindings. The fact that postgres (and mysql?) was missed to begin with is proof of that.
What do you think about removing all current prepBindings calls and append it to the master class instead? (client)
https://github.com/tgriesser/knex/blob/master/src/client.js#L128
https://github.com/tgriesser/knex/blob/master/src/client.js#L139
The case about toSQL(), toString() and toQuery() calling prepBindings, the only way I can see of avoiding that is by calling prepBindings whenever we receive bindings, which would mean every single function in the API today that takes a set of bindings. That would result in even more calls, so I don't see it as a worthy tradeoff. Reason why it would have to be in every call is because you can do this for example:
var qb = knex(tableName);

qb.where({test: 1});
console.log(qb.toString());
qb.orWhere({test: 2});
console.log(qb.toString());

Both .toString() calls would have to be properly formatted to keep this consistent, which means .where and .orWhere would have to call .prepBindings. :-1:

@wubuzz how does it sound if prepBindings is ran when toSQL, but original bindings are not thrown away so that if client is attached/changed to query builder, prepBindings may be ran again on next toSQL call?

@elhigu I like that idea. Pushed changes!

@rhys-vdw do you have any idea about that tz argument discussed here?

do you have any idea about that tz argument discussed here?

No, we could perhaps blame it? Generally I'd say that if a feature is undocumented and untested then it is an implementation detail. But with Knex there seems to be a large amount of "discoverable" functionality. Might @bendrucker or @tgriesser be able to shed any light on this?

As this PR fixes a pretty crucial bug and is not a breaking change I am going to merge this so that it's included in the next release.
Should some information come to light regarding tz we can deal with that in the future. For now, it's still included in the source so there are no breaking changes in that regard.

good call :+1:

Original issue #1268


Creating a new dev box puts the environment card in a stopped state in the Environment page

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Create a new dev box
Expected result
Environment page should show the dev box created and running. Launch drop down should be functional.
Actual result
Environment page shows the dev box in a stopped state and the launch drop down is broken.
Included System Information
No response
Included Extensions Information
No response

Issue is fixed in the Azure Extension package. Fix would be part of the May release


Getting OutOfMemoryError for inferencing the model☹

Hi I am trying to run the model using on lambdalabs GPUs instances a10 and h100, However I am facing every time OutOfMemoryError on both of them.
torch.cuda.OutOfMemoryError: CUDA out of memory.
Tried to allocate 86.00 MiB. GPU 0 has a total capacty of 21.99 GiB of which 67.00 MiB is free.
Process 23489 has 21.91 GiB memory in use.
Of the allocated memory 21.69 GiB is allocated by PyTorch, and 1.55 MiB is reserved by PyTorch but unallocated.
If reserved but unallocated memory is large try setting max_split_size_mb to avoid fragmentation. See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Below is error trace with some GPU logs.

TORCH GPU IS AVAILABLE :  True
TORCH VERSION:  2.0.1+cu118
|===========================================================================|
|                  PyTorch CUDA memory summary, device ID 0                 |
|---------------------------------------------------------------------------|
|            CUDA OOMs: 0            |        cudaMalloc retries: 0         |
|===========================================================================|
|        Metric         | Cur Usage  | Peak Usage | Tot Alloc  | Tot Freed  |
|---------------------------------------------------------------------------|
| Allocated memory      |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| Active memory         |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| Requested memory      |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| GPU reserved memory   |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| Non-releasable memory |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| Allocations           |       0    |       0    |       0    |       0    |
|       from large pool |       0    |       0    |       0    |       0    |
|       from small pool |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| Active allocs         |       0    |       0    |       0    |       0    |
|       from large pool |       0    |       0    |       0    |       0    |
|       from small pool |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| GPU reserved segments |       0    |       0    |       0    |       0    |
|       from large pool |       0    |       0    |       0    |       0    |
|       from small pool |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| Non-releasable allocs |       0    |       0    |       0    |       0    |
|       from large pool |       0    |       0    |       0    |       0    |
|       from small pool |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| Oversize allocations  |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| Oversize GPU segments |       0    |       0    |       0    |       0    |
|===========================================================================|

TORCH GPU IS AVAILABLE :  True
TORCH VERSION:  2.0.1+cu118
|===========================================================================|
|                  PyTorch CUDA memory summary, device ID 0                 |
|---------------------------------------------------------------------------|
|            CUDA OOMs: 0            |        cudaMalloc retries: 0         |
|===========================================================================|
|        Metric         | Cur Usage  | Peak Usage | Tot Alloc  | Tot Freed  |
|---------------------------------------------------------------------------|
| Allocated memory      |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| Active memory         |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| Requested memory      |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| GPU reserved memory   |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| Non-releasable memory |      0 B   |      0 B   |      0 B   |      0 B   |
|       from large pool |      0 B   |      0 B   |      0 B   |      0 B   |
|       from small pool |      0 B   |      0 B   |      0 B   |      0 B   |
|---------------------------------------------------------------------------|
| Allocations           |       0    |       0    |       0    |       0    |
|       from large pool |       0    |       0    |       0    |       0    |
|       from small pool |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| Active allocs         |       0    |       0    |       0    |       0    |
|       from large pool |       0    |       0    |       0    |       0    |
|       from small pool |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| GPU reserved segments |       0    |       0    |       0    |       0    |
|       from large pool |       0    |       0    |       0    |       0    |
|       from small pool |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| Non-releasable allocs |       0    |       0    |       0    |       0    |
|       from large pool |       0    |       0    |       0    |       0    |
|       from small pool |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| Oversize allocations  |       0    |       0    |       0    |       0    |
|---------------------------------------------------------------------------|
| Oversize GPU segments |       0    |       0    |       0    |       0    |
|===========================================================================|
A new version of the following files was downloaded from https://huggingface.co/THUDM/cogagent-chat-hf:
- cross_visual.py
- visual.py
. Make sure to double-check they do not contain any added malicious code. To avoid downloading new versions of the code file, you can pin a revision.
Please 'pip install apex'
Loading checkpoint shards:   0%|          | 0/8 [00:00<?, ?it/s]
Loading checkpoint shards:  12%|█▎        | 1/8 [00:00<00:01,  5.63it/s]
Loading checkpoint shards:  25%|██▌       | 2/8 [00:00<00:01,  5.14it/s]
Loading checkpoint shards:  38%|███▊      | 3/8 [00:00<00:01,  4.83it/s]
Loading checkpoint shards:  50%|█████     | 4/8 [00:00<00:00,  4.80it/s]
Loading checkpoint shards:  62%|██████▎   | 5/8 [00:01<00:00,  4.57it/s]
Loading checkpoint shards:  75%|███████▌  | 6/8 [00:01<00:00,  4.24it/s]
Loading checkpoint shards:  88%|████████▊ | 7/8 [00:01<00:00,  3.83it/s]
Loading checkpoint shards: 100%|██████████| 8/8 [00:02<00:00,  3.00it/s]
Loading checkpoint shards: 100%|██████████| 8/8 [00:02<00:00,  3.76it/s]
Traceback (most recent call last):
File "/root/.pyenv/versions/3.11.7/lib/python3.11/site-packages/cog/server/worker.py", line 185, in _setup
run_setup(self._predictor)
File "/root/.pyenv/versions/3.11.7/lib/python3.11/site-packages/cog/predictor.py", line 66, in run_setup
predictor.setup()
File "/src/predict.py", line 33, in setup
.to("cuda")
^^^^^^^^^^
File "/root/.pyenv/versions/3.11.7/lib/python3.11/site-packages/transformers/modeling_utils.py", line 2271, in to
return super().to(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/root/.pyenv/versions/3.11.7/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1145, in to
return self._apply(convert)
^^^^^^^^^^^^^^^^^^^^
File "/root/.pyenv/versions/3.11.7/lib/python3.11/site-packages/torch/nn/modules/module.py", line 797, in _apply
module._apply(fn)
File "/root/.pyenv/versions/3.11.7/lib/python3.11/site-packages/torch/nn/modules/module.py", line 797, in _apply
module._apply(fn)
File "/root/.pyenv/versions/3.11.7/lib/python3.11/site-packages/torch/nn/modules/module.py", line 820, in _apply
param_applied = fn(param)
^^^^^^^^^
File "/root/.pyenv/versions/3.11.7/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1143, in convert
return t.to(device, dtype if t.is_floating_point() or t.is_complex() else None, non_blocking)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: CUDA error: out of memory
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with `TORCH_USE_CUDA_DSA` to enable device-side assertions.
ⅹ Model setup failed

I tried multiple times , but everytimes it fails
my inference code is via cog
Any help is highly appreciated🙏!!

Use load_in_4bit=True if you have limited GPU memory.

try to set different model device mapping because your two card are different, or try int4

I don't know what code you are running. It's your code problem. We have provided the code that can run on 24GB GPU.

python cli_demo_hf.py --from_pretrained THUDM/cogagent-chat-hf --fp16 --quant 4


External setup with Next-Server on Sophos Firewall DHCP does not work

Cannot seem to get this to work with the external DHCP configuration and my sophos firewall.

I have this setup under podman using a macvlan. I can reach the UI just fine over the assigned IP address and even PXE boot if I set iVentoy as the DHCP server. So iVentoy DOES work, just seems the external DHCP configuration does not? Could be an issue with sophos. Figured I'd post an issue in case others are having similar problems, happy test anything I can!

As a side note, I did set limit my main DHCP scope and am running both DHCP servers in non-overlapping scopes. PXE starts to boot but then loses the link somehow.

I was running external. This turned out to be a bug in the sophos version I was running. I apologize I forgot to follow up and update this.


Show the full number of likes/dislikes

It seems youtube's latest update has changed the way that like counts are displayed. Now, if there are 4,980 likes on a video, it just says "4K" next to the thumbs up button. I'm requesting that YouTubeCenter replace the "4K" text with the actual number "4,980".

+1

Well the same thing goes for the number of subscriptions in the hovercard.
something needs to be done.


add support of a unix-like poll function

poll() examines a set of file descriptors to see if some of them are ready for I/O or if certain events have occurred on them. The currenty implementation is using an asynchronous function to check all file descriptors.
This PR should solve smol-rs/polling#177
The current implementation doesn't support file descriptors, which points to the uhyve file system.

@mkroening Now, you can review the PR. I added some async lock to reduce the blocking time.

This is awesome :fireworks:


Can't read wave file, RiffReadError

Tried reading a wave file using the readtest demo file. Used different files from different sources, generated with different tools.
Win10
[...]\itersample.exe pink_250.0.wav
[...]\itersample.nim(79) itersample
[...]\itersample.nim(70) main
[...].nimble\pkgs\easywave-0.2.0\easywave.nim(245) openWaveFile
[...].nimble\pkgs\riff-0.2.0\riff.nim(790) exitGroup
Error: unhandled exception: Cannot exit root chunk [RiffReadError]

Can you upload the file here and I'll have a look? You might need to use ZIP extension or something that's in the allowed extensions list.

testrecord.zip
wav, scripts to generate and read wav & .exe's


ACM Certificate + Route53 dns validation example / tutorial

What is the URL of the document?
https://aws-controllers-k8s.github.io/community/docs/community/overview/

Which section(s) is the issue in?

Tutorials
What needs fixing?

A basic example on how to use the acm-controller should be provided.
Like creating a acm.services.k8s.aws/Certificate and how to use it together with aws-load-balancer-controller .  In particular it would be great to include the DNS validation as well,
Additional context

I googled and  can't find any example of the acm-controller.
So I think now my only options is to manually inspect the CRDs in acm-controller

Certificate

And then trying to guess how to use it, specially how to the DNS validation will be implemented.

I also didn't find any examples, so here's one:
apiVersion: acm.services.k8s.aws/v1alpha1
kind: Certificate
metadata:
  name: mydomain
spec:
  domainName: "mydomain.com"
  subjectAlternativeNames:
    - "*.mydomain.com"

This creates a certificate with two records. But I have no idea how to automate the route 53 verification, or if it is even possible atm.

After skimming the code in https://github.com/aws-controllers-k8s/acm-controller/blob/v0.0.9/pkg/resource/certificate/sdk.go I don't think this resource controller is actually capable of verifying the certificates it requests. In order to do so it would need to also use the API of Route53 (or some other DNS provider) and at the moment it only uses the ACM API.
This was previously discussed here where @jaypipes said it's not planned, as there is/was also a route53 controller in the works.

I can create a certificate with:
apiVersion: acm.services.k8s.aws/v1alpha1
kind: Certificate
metadata:
  name: "mydomain"
spec:
  domainName: "mydomain.com"

and using the route53-controller, add the cname resource record for validation.
apiVersion: route53.services.k8s.aws/v1alpha1
kind: RecordSet
metadata:
  name: "mydomain-validation"
spec:
  hostedZoneID:<hosted-zone-id>
  recordType: CNAME
  name: <ResourceRecord.name>
  resourceRecords:
    - value: <ResourceRecord.value>
  ttl: 300

but I can't figure out how to fully automate this. Are there examples on how to do this?

@john-r-swyftx Mutation webhook should do the trick then, see Kyverno for more info.


Fix source doc function summary

Function does not change the group ownership, it copies things. Source doc looks like copy&past ;)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.965%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3251

Relevant Lines:
3285

💛  - Coveralls


[Angular 13] persistent build cache is dependent on absolute folder path of the project

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
No
Description

When any parent folder of a project is renamed, the angular persistent build cache is recreated and will no longer use any "old" data
This makes the cache basically non-existent in any CI scenario where the workspace folder is generated on demand (e.g. DroneCI on Windows)

🔬 Minimal Reproduction

Folder: C:\angular
ng new app
ng build

Build at: 2021-11-11T15:53:43.980Z - Hash: d4f16607c301d89a - Time: 16540ms

ng build

Build at: 2021-11-11T15:53:59.406Z - Hash: d4f16607c301d89a - Time: 4076ms

Rename: C:\angular --> C:\angular2
ng build

Build at: 2021-11-11T15:55:55.631Z - Hash: d4f16607c301d89a - Time: 13954ms

ng build

Build at: 2021-11-11T15:56:08.950Z - Hash: d4f16607c301d89a - Time: 4124ms
🌍 Your Environment

Angular CLI: 13.0.2
Node: 16.13.0
Package Manager: npm 8.1.0
OS: win32 x64

Angular: 13.0.1
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1300.2
@angular-devkit/build-angular   13.0.2
@angular-devkit/core            13.0.2
@angular-devkit/schematics      13.0.2
@angular/cli                    13.0.2
@schematics/angular             13.0.2
rxjs                            7.4.0
typescript                      4.4.4

Anything else relevant?

While, I am not familiar with DroneCI, CIs typically offer ways to always checkout code to the same path, at least this was always the case for the CIs providers I used (Circle, BuildKite, CodeFresh, Travis, AppVeyor and Team City).
I am sorry but, this is not something that we can action ourselves, since Webpack which is used to bundle apps and Rollup which is used to bundle libs both take into account paths.


Text classification with numl

I am doing a software to classify texts about health.
So, I have many texts classified in "positive" and "negative" and these are the texts for my training.
The user will put the new texts and my software will evaluate and say with this text is "positive" or "negative".
Today, I use Weka and Naïve Bayes to this, but I would like of a framework specific to .NET.
So, I have found numl but I have not found a sample about this.
Is it possible?
Thanks

It sure is!
You will need to look at using the StringFeature attribute for using textual features.
This should help get you started:
            var strings = new []
            {
                new { Text = "So there is no way for me to plug it in here in the US unless I go by a converter.",  Label = false },
                new { Text = "Good case, Excellent value.",  Label = true },
                new { Text = "Great for the jawbone.",  Label = true },
                new { Text = "Tied to charger for conversations lasting more than 45 minutes.MAJOR PROBLEMS!!", Label = false },
                new { Text = "The mic is great.",  Label = true },
                new { Text = "I have to jiggle the plug to get it to line up right to get decent volume.",  Label = false },
                new { Text = "If you have several dozen or several hundred contacts, then imagine the fun of sending each of them one by one.",  Label = false },
                new { Text = "If you are Razr owner...you must have this!",  Label = true },
                new { Text = "Needless to say, I wasted my money.",  Label = false },
                new { Text = "What a waste of money and time!.",  Label = false },
                new { Text = "And the sound quality is great.",  Label = true },
                new { Text = "He was very impressed when going from the original battery to the extended battery.",  Label = true },
                new { Text = "If the two were seperated by a mere 5 + ft I started to notice excessive static and garbled sound from the headset.",  Label = false },
                new { Text = "Very good quality though",  Label = true },
                new { Text = "The design is very odd, as the ear \"clip\" is not very comfortable at all.",  Label = false },
                new { Text = "Highly recommend for any one who has a blue tooth phone.",  Label = true },
                new { Text = "I advise EVERYONE DO NOT BE FOOLED!",  Label = false },
                new { Text = "So Far So Good!.",  Label = true },
                new { Text = "Works great!.",  Label = true },
                new { Text = "It clicks into place in a way that makes you wonder how long that mechanism would last.",  Label = false },
                new { Text = "I went on Motorola's website and followed all directions, but could not get it to pair again.",  Label = false },
                new { Text = "I bought this to use with my Kindle Fire and absolutely loved it!",  Label = true },
                new { Text = "The commercials are the most misleading.",   Label = false },
                new { Text = "I have yet to run this new battery below two bars and that's three days without charging.",  Label = true },
                new { Text = "I bought it for my mother and she had a problem with the battery.",  Label = false },
                new { Text = "Great Pocket PC / phone combination.",  Label = true },
                new { Text = "I've owned this phone for 7 months now and can say that it's the best mobile phone I've had.",  Label = true },
                new { Text = "I didn't think that the instructions provided were helpful to me.",  Label = false },
                new { Text = "People couldnt hear me talk and I had to pull out the earphone and talk on the phone.",  Label = false },
                new { Text = "Doesn't hold charge.",  Label = false },
                new { Text = "This is a simple little phone to use, but the breakage is unacceptible.",  Label = false },
                new { Text = "This product is ideal for people like me whose ears are very sensitive.",  Label = true },
                new { Text = "It is unusable in a moving car at freeway speed.",  Label = false  },
                new { Text = "I have two more years left in this contract and I hate this phone.",  Label = false },
                new { Text = "Car charger as well as AC charger are included to make sure you never run out of juice.Highy recommended",  Label = true },
                new { Text = "You need at least 3 mins to get to your phone book from the time you first turn on the phone.Battery life is short.",  Label = false },
                new { Text = "It has kept up very well.",  Label = true }

            };

            var d = Descriptor.New("SENTIMENT")
                              .With("Text").AsString(StringSplitType.Word)
                              .Learn("Label").As(typeof(bool));

            var generator = new NaiveBayesGenerator(3) { Descriptor = d };

            var model = Learner.Learn(strings, .75, 1, generator);

            Console.WriteLine(model.Score);

If you have any other questions feel free to join us on gitter here

@bdschrisk How to use this model after training? I am getting an error when doing this "
Expression must be writeableParameter name: left
model.Predict(new { Text = "The hotel was good",  Label = true });


Quick Text Search: Improve the experience when there are many matches

Testing #191499

open the vscode repo
open command palette, type % semantic

There are many matches which makes the quick view less practical
Some suggestions:

show the total number of matches in a description (50 matches in 7 files)
reduce the number of matches per file if the list is long (e.g. 3 matches).
allow to filter for the file name as well

posted this on https://github.com/microsoft/vscode/issues/191683 to group together similar issues


alias options from 'data-inputmask' is not used anymore

This markup works in 3.1.34. Options integerDigits, allowMinus and allowPlus are applied to input. But in latest 3.2.3 release this is not works anymore.
<input type="text" class="inputmask" data-inputmask="'alias':'integer', 'integerDigits': 3, 'allowMinus': false, 'allowPlus': false" />

The issue is in importAttributeOptions function inputmask.js:328
data-inputmask json data parsed in dataoptions. But this object used to populate only general inputmask options. Alias options is not populated from this json and stay with default values.

@StennikovDmitriy ,
Fixed!


Custom Checkout useHook error

If you enable the custom checkout, when you click Proceed to Checkout, the front end crashes.
Full stack trace:
VM49945 use-checkout.js:21 Uncaught TypeError: Cannot read properties of undefined (reading 'fetcher')
    at useCheckout (VM49945 use-checkout.js:21:28)
    at CheckoutSidebarView (VM49943 CheckoutSidebarView.tsx:75:92)
    at renderWithHooks (react-dom.development.js?9c41:14985:1)
    at mountIndeterminateComponent (react-dom.development.js?9c41:17811:1)
    at beginWork (react-dom.development.js?9c41:19049:1)
    at HTMLUnknownElement.callCallback (react-dom.development.js?9c41:3945:1)
    at Object.invokeGuardedCallbackDev (react-dom.development.js?9c41:3994:1)
    at invokeGuardedCallback (react-dom.development.js?9c41:4056:1)
    at beginWork$1 (react-dom.development.js?9c41:23964:1)
    at performUnitOfWork (react-dom.development.js?9c41:22776:1)
    at workLoopSync (react-dom.development.js?9c41:22707:1)
    at renderRootSync (react-dom.development.js?9c41:22670:1)
    at performSyncWorkOnRoot (react-dom.development.js?9c41:22293:1)
    at eval (react-dom.development.js?9c41:11327:1)
    at unstable_runWithPriority (scheduler.development.js?2b4e:468:1)
    at runWithPriority$1 (react-dom.development.js?9c41:11276:1)
    at flushSyncCallbackQueueImpl (react-dom.development.js?9c41:11322:1)
    at flushSyncCallbackQueue (react-dom.development.js?9c41:11309:1)
    at discreteUpdates$1 (react-dom.development.js?9c41:22420:1)
    at discreteUpdates (react-dom.development.js?9c41:3756:1)
    at dispatchDiscreteEvent (react-dom.development.js?9c41:5889:1)

Somehow useHook is returning undefined packages/commerce/dist/utils/use-hook.js.
I've tried debugging and I can't narrow down the problem.

Duplicate of https://github.com/vercel/commerce/issues/533


Luci not show kernel and system log.

Openwrt 19.07.3 (openwrt-19.07 branch (git-20.145.59100-515bd92) / OpenWrt 19.07.3 r11063-85e04e9f46) don't show kernel and system log.

Works for me on a vanilla x86/64 image in Qemu.

I'm using TP-Link W8970. How can I check why this is not working?


service logging

closes #73

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 1min)


Context File is returned with the Mimetype "plain/text"

... however, "application/json" or "application/ld+json" is required.

@mkollenstart I just tested the JSON-LD messages with Jena, which worked out-of-the-box. Looks like at least Jena doesn't give much about the mime type when resolving the context.

Since Gtihub Pages actions only support pushing a single artifact, the only viable solution is to include the releases into the repository.
I've created a fork with an example of how this could work at https://github.com/mkollenstart/ids-specification where I've created a releases folder and a Gtihub Action for creating the Github pages.
The JSON-LD context can then be accessed via https://mkollenstart.github.io/ids-specification/0.8/schema/context.json
The repository settings should then be updated to use Github Actions as source for Github Pages (In Settings > Code and automation > Pages > Source).
The question is how to serve the working version (i.e. the non-released version) of the context, should that be in a folder within releases or in the current directory structure? For the latter the github action should be updated to include those files in the Github Pages.
Note: The next two weeks I'm out of office, so feel free to merge these changes without me.

I also need the following three JSON files being exposed the same way:

https://raw.githubusercontent.com/International-Data-Spaces-Association/ids-specification/main/negotiation/message/schema/contract-schema.json
https://raw.githubusercontent.com/International-Data-Spaces-Association/ids-specification/main/catalog/message/schema/catalog-schema.json
https://raw.githubusercontent.com/International-Data-Spaces-Association/ids-specification/main/catalog/message/schema/dataset-schema.json

We wait for @mkollenstart


[HUDI-1852]Add SCHEMA_REGISTRY_SOURCE_URL_SUFFIX and SCHEMA_REGISTRY_TARGET_URL_SUFFIX property

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
Before, only schemaRegistrySuffix did not distinguish between targetSchemaRegistrySuffix and sourceSchemaRegistrySuffix, which is not suitable for our needs.
Verify this pull request
Committer checklist

[x] Has a corresponding JIRA in PR title & commit

[x] Commit message is descriptive of the change

[x] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #2884 (6f9f3ec) into master (9bbb458) will increase coverage by 16.64%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #2884       +/-   ##
=============================================
+ Coverage     52.93%   69.58%   +16.64%
+ Complexity     3742      375     -3367
=============================================
  Files           488       54      -434
  Lines         23511     2002    -21509
  Branches       2501      237     -2264
=============================================
- Hits          12446     1393    -11053
+ Misses         9966      478     -9488
+ Partials       1099      131      -968

Flag
Coverage Δ
Complexity Δ

hudicli
?
?

hudiclient
?
?

hudicommon
?
?

hudiflink
?
?

hudihadoopmr
?
?

hudisparkdatasource
?
?

hudisync
?
?

huditimelineservice
?
?

hudiutilities
69.58% <0.00%> (-0.13%)
375.00 <0.00> (+1.00)
:arrow_down:

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

Impacted Files
Coverage Δ
Complexity Δ

...s/deltastreamer/HoodieMultiTableDeltaStreamer.java
76.19% <0.00%> (-2.34%)
19.00 <0.00> (ø)

.../java/org/apache/hudi/HoodieDataSourceHelpers.java

.../org/apache/hudi/hadoop/utils/HoodieHiveUtils.java

.../hudi/common/bloom/InternalDynamicBloomFilter.java

.../java/org/apache/hudi/common/util/RateLimiter.java

...i/bootstrap/SparkParquetBootstrapDataProvider.java

...di/common/table/timeline/HoodieActiveTimeline.java

.../hadoop/utils/HoodieRealtimeRecordReaderUtils.java

...in/java/org/apache/hudi/common/model/BaseFile.java

...e/hudi/table/format/mor/MergeOnReadTableState.java

... and 426 more

@pratyakshsharma : fyi.


Added a /thread list command that lists active unsolved threads

Adds a /thread list command that lists active unsolved threads
This variant sends links to the threads as a DM to the user who ran the command. It could be nicer to have them sent as an ephemeral message, but that would require creating a new command, something like /threads list. That solution might also be preferable from a code structure point of view, because currently this command has to have a separate if statement at the start in order to skip the permission checking that happens later that prevents the /thread command from being ran anywhere but inside a thread.
EDIT:
The command is now /threads list [filter]

👋


Auto heigth with fixed width

Hey, i want to set a auto heigth of my pdf generate. Heigth based on content of pdf, and witdth set to 80mm i tried with page-size="80mm:auto" but it didnt work.
 Im using the plugin in symfony2 with your bundle.
Thanks a lot!

Maybe elastic-page tag would help you? https://github.com/psliwa/PHPPdf#standard-tags

Yes, thank you and sorry. I look in doc and i did not see it :$
Thank you!


Add version: JetBrains.Kotlin.Compiler version 1.2.60-eap-44

Pull request has been created with Komac v1.9.1 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116337&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


Getting close error while invoking the lambda

Hi Team,
I am getting a very critical issue in the lambda plugin while connection to lambda from kong.
below is error
handler.lua:188 [-aws-lambda] closed,  client: <IP_ADDRESS>. , server: kong, request: "GET /apis HTTP/1.1", host: "*.com"

related Kong issue: https://github.com/Kong/kong/issues/5246

Since this repo was merged with https://github.com/Kong/kong, I'm closing this issue; let's continue the discussion in https://github.com/Kong/kong/issues/5246.


Add Tofino to "Built with Electron"

Mozilla is working on an experimental browser called 'Tofino', using electron.

Awesome, thank you! ✨

I love that logo!


AWSSDK.DAX.Client GetItemAsync throws NullReferenceException when permissions are missing

Describe the bug
We have a service where we have implemented DAX using AWSSDK.DAX.Client for .Net Nuget package version 2.1.0. If we don't give permission 'dax:GetItem' to the service while fetching data from DAX, we don't get a specific error. Instead we get a general error 'NullReferenceException'.
Sample Code -
ClusterDaxClient client; await client.GetItemAsync(request);
Expected Behavior
We expect to see a specific exception like 'AccessDeniedException' or 'MissingPermissionsException' instead of NullReferenceException.
Current Behavior

We are getting NullReferenceException while accessing DAX when permissions are missing.
Reproduction Steps
Skip the permission 'dax:GetItem' from user IAM policy accessing DAX.
call clusterdaxclient getitemasync method.
Sample code -
ClusterDaxClient client; await client.GetItemAsync(request);
Possible Solution
No response
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
AWSSDK.DAX.Client 2.1.0
Targeted .NET Platform
.Net 6
Operating System and version
AmazonLinux

Communicated to the DAX team to look at the issue.

P85058576


Issue #455: Add docs on environment specific configs

A first draft of the functionality.

I avoided giving specific suggestions as I'm not entirely sure what workflow we should be recommending.
Personally I would not entirely trust the environment variable and would instead have the security hardening configurations in config.yml while I override them with development tools in vagrant.config.yml. This way a production environment will never be provisioned with development tools even if I make a typo in the command.

That's... a good point. Do we want to update the docs and recommend cloning default.config.yml to vagrant.config.yml instead of config.yml? Then in the docs we can say "if you consider using Drupal VM for production, you should set the secure overrides in config.yml and customize your local environment using vagrant.config.yml and production-specific overrides in prod.config.yml... something along those lines?

Do we want to update the docs and recommend cloning default.config.yml to vagrant.config.yml instead of config.yml

You mean in README, all docs, etc?

@oxyc - Yeah, maybe... though it might be a bit heavy handed. If Drupal VM becomes used more for multiple environment configuration (e.g. prod, stage, local), then using config this way is a little more correct.
But for the majority use case, currently just mentioning config.yml seems to be the simplest/least confusing. So I guess I just convinced myself to not recommend changing things up right now :D

@oxyc - Can you also update the README in the prod directory to point to the docs page instead of having duplicated info? Just have the note about 'security is your responsibility', then direct people to the docs URL for more information.

@geerlingguy you want me to move over all the docs from the README?

Yeah.

This is fine for now... we can iterate on it later :)


cob_perception_common: 0.6.16-1 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository cob_perception_common to 0.6.16-1:

upstream repository: https://github.com/ipa320/cob_perception_common.git
release repository: https://github.com/ipa320/cob_perception_common-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.9.8
previous version for package: 0.6.15-1

cob_3d_mapping_msgs
* Merge pull request #101 <https://github.com/ipa320/cob_perception_common/issues/101> from fmessmer/test_noetic
  test noetic
* Bump CMake version to avoid CMP0048 warning
* Contributors: Felix Messmer, fmessmer

cob_cam3d_throttle
* Merge pull request #101 <https://github.com/ipa320/cob_perception_common/issues/101> from fmessmer/test_noetic
  test noetic
* Bump CMake version to avoid CMP0048 warning
* Contributors: Felix Messmer, fmessmer

cob_image_flip
* Merge pull request #101 <https://github.com/ipa320/cob_perception_common/issues/101> from fmessmer/test_noetic
  test noetic
* reduce dependencies
* Bump CMake version to avoid CMP0048 warning
* Contributors: Felix Messmer, fmessmer

cob_object_detection_msgs
* Merge pull request #101 <https://github.com/ipa320/cob_perception_common/issues/101> from fmessmer/test_noetic
  test noetic
* Bump CMake version to avoid CMP0048 warning
* Contributors: Felix Messmer, fmessmer

cob_object_detection_visualizer
* Merge pull request #101 <https://github.com/ipa320/cob_perception_common/issues/101> from fmessmer/test_noetic
  test noetic
* reduce dependencies
* Bump CMake version to avoid CMP0048 warning
* Contributors: Felix Messmer, fmessmer

cob_perception_common
* Merge pull request #101 <https://github.com/ipa320/cob_perception_common/issues/101> from fmessmer/test_noetic
  test noetic
* remove cob_vision_utils
* Bump CMake version to avoid CMP0048 warning
* Contributors: Felix Messmer, fmessmer

cob_perception_msgs
* Merge pull request #101 <https://github.com/ipa320/cob_perception_common/issues/101> from fmessmer/test_noetic
  test noetic
* Bump CMake version to avoid CMP0048 warning
* Contributors: Felix Messmer, fmessmer

ipa_3d_fov_visualization
* Merge pull request #101 <https://github.com/ipa320/cob_perception_common/issues/101> from fmessmer/test_noetic
  test noetic
* Bump CMake version to avoid CMP0048 warning
* Contributors: Felix Messmer, fmessmer

Holding for the pending Kinetic sync


add buildout support (to replace run.py script)

to boostrap and buildout Cactus for local development run:
$ python bootstrap.py
$ bin/buildout

then, to run Cactus:
$ bin/cactus ...

to run tests:
$ bin/test

Reference: zc.buildout

We're currently using tox for development. I'm not sure I understand why adding another tool is desirable here.
As I mentioned in the other thread, I don't really see why this "replaces" the run.py script? The run.py script essentially just does whatever cli.py does. It's not really useful (and not very usable, since it strongly depends on how your your PYTHONPATH is setup and whether Cactus is installed).

@krallin ah ok. so tox is what I missed.

Eh that might be why! You should be able to run all tests by just installing tox and then running tox from the root of the project (where there is tox.ini).
As far as running cactus in development, what I usually do is install Cactus in development (python setup.py develop) in a virtual environment. If that's not usable I'm happy to hear suggestions / etc. : )
Cheers,


Fix one shift/reduce and one reduce/reduce conflict

Implements the fix suggested here, which performed better than expected!
I'm not too fond of this, but this seems like the right way.
This leaves a single shift/reduce conflict (!) which is lookahead confusion with T_POP_SET. I don't know how to tell Yacc it can figure it out with sufficient lookahead, if at all possible.

It doesn't look like a bad fix.


test(e2e/twap): convert TestTWAP to operate to multi-asset pool

Background
e2e TestTWAP should operate on a multi-asset pool to enable holistic testing of TWAP pruning.
Suggested Design
Convert TestTWAP to operate on a multi-asset pool
Acceptance Criteria

TWAP pruning tests pass on a multi-asset pool

blocked by https://github.com/osmosis-labs/osmosis/pull/2638


Pricing (or any other value) category label wrangle

Wrangle that sorts values into ranges (labels) using pandas.cut()
Should this be separate from filter?
filter reduces size of dataFrame, this will keep the same size always.
Data sample:

cost
category

200
under $250

503
$500-$1,000

800
$500-$1,000

1300
$1,000-$2,000

2500
$2,000-$3,000

3013
$3000 and above

possible wrangle:
wrangles:
 - value_category:
     input: cost
     output: category
      values:
         - 200
         - 250
         - and so on... # could import as variables, keep recipe short
       labels:
         - 'under $250
         - '$250-$500
         - and so on # could import as variables, keep recipe short

code:
data['category'] = pd.cut(
    x=data['cost'],
    bins=[0, 250, 500, 1000, 2000, 3000, math.inf],
     labels = [
        'under $250',
        '$250-$500',
        '$500-$1,000',
        '$1,000-$2,000',
        '$2,000-$3,000',
        '$3000 and above'
    ]
)

Seems like a good addition. I'd prioritise this one.

Yes, I'd consider this different from filter.
I'd name it create.bins - I think binning is the most widely recognised term for this.
Test the behaviour of the outer bounds. They won't be able to add infinity in the yaml, so perhaps automatically add +/- infinity at either end if that is needed/works well.
Allow an integer or list for the bins. The interval index looks like it needs to pass in a separate data structure, so wouldn't work naturally. I'm struggling to think when that would be usefully beyond the simple list though, so maybe just skip that.

wrangles:
  - create.bins:
      input: cost
      output: cost bins
      bins:
        - 200
        - 250
        - and so on... # could import as variables, keep recipe short
      labels:
        - under $250
        - $250-$500
        - and so on # could import as variables, keep recipe short

When passing values as variables (from python variables to recipe ${variable}) the list must be converted to a string, otherwise an error occurs (typeError)
In recipe, when using " " around the ${variable} notation, an error occurs
Exception has occurred: ValueError could not convert string to float: "['-', 250, 500, 1000, 2000, 3000, '+']"
below is the combination that works
python
bin_labels = [
    'under $250',
    '$250-$500',
     and so on...
]

bin_values = [
    '-',
    250,
    and so on...
]

variables = {
    # Pricing Categories, must be strings
    'bin_values': str(bin_values),
    'bin_labels': str(bin_labels),
}

Yaml
- create.bins:
      input: IMAP_Value
      output: Pricing
      bins: ${bin_values}  # cannot be surrounded by ""
      labels: ${bin_labels}


[Enhancement]: Users mis-interpret minimum spend requirement on Keepers

What would you like us to improve?
Users are misinterpreting the minimum spend requirement. They are reading that it is per transaction, when it is in fact per upkeep. Need to ensure this is crystal clear.
How important it is?
This is scale of 1 to 2 on a range of 1 = High, 5 = Low

Resolved


Bump plasma-contracts SHA to Nov 12

A bit of work to change the deployment interface as @boolafish pointed out:

https://github.com/omisego/plasma-contracts/commit/db824055bb3037d2fc1034b8ec751494cbe1dd4b
https://github.com/omisego/plasma-contracts/commit/43968f6fdab815fb5b46cfcf2d9803d3a26ea1e9

Build errors: https://circleci.com/workflow-run/856730f1-ada9-4c4c-9017-53a00806b033
TODOs:

[x] Bump https://github.com/omisego-images/docker-elixir-omg/blob/master/tester/Dockerfile.plasma_deployer#L15
[ ] Bump elixir-omg to new SHA
[ ] Fix elixir-omg contracts deployment errors during tests (align with the 2 plasma-contracts commits above)
[ ] Let devops know when elixir-omg PR with new contracts SHA is merged.

I tracked to these two files from the error message:
https://github.com/omisego/elixir-omg/blob/master/apps/omg_eth/test/support/bundle_deployer.ex
https://github.com/omisego/elixir-omg/blob/master/apps/omg_eth/test/support/deployer.ex


Add some blocks to the main template for fast customization

As pointed in the pull request #79 it is actually possible to change the main docs view overriding the index.html file. But overriding the whole file is not the best idea if you want to be sure that your custom template uses the latest Javascript files, for example.
Adding blocks it is possible to make some fast customisation of the main docs view, still making sure that users are using the latest javascript files.
(The pull request comes with some minor CSS tweaks, especially regarding the line-height of the description block).

sweet! squash please.

It should be ok now.


Default generated unit test for a Java function doesn't compile

I just spoke with @julianduque about this problem and he suggested I file an issue here, so hopefully it's the right place.
It looks like the auto-generated unit test for a Java function is no longer valid. It includes:
Record firstRecord = mock(Record.class);
when(firstRecord.getStringField("Id")).thenReturn(Optional.of("5003000000D8cuIQAA"));
when(firstRecord.getStringField("Name")).thenReturn(Optional.of("Account One, inc."));

Record secondRecord = mock(Record.class);
when(secondRecord.getStringField("Id")).thenReturn(Optional.of("6003000000D8cuIQAA"));
when(secondRecord.getStringField("Name")).thenReturn(Optional.of("Account Two, inc."));

when(mockResult.getRecords()).thenReturn(Arrays.asList(firstRecord, secondRecord));

which yields the following compilation error on the last line for thenReturn:
java: no suitable method found for thenReturn(java.util.List<com.salesforce.functions.jvm.sdk.data.Record>)
    method org.mockito.stubbing.OngoingStubbing.thenReturn(java.util.List<com.salesforce.functions.jvm.sdk.data.RecordWithSubQueryResults>) is not applicable
      (argument mismatch; inference variable T has incompatible bounds
          equality constraints: com.salesforce.functions.jvm.sdk.data.RecordWithSubQueryResults
          lower bounds: com.salesforce.functions.jvm.sdk.data.Record)

The fix is to change the type of firstRecord and secondRecord to be RecordWithSubQueryResults.
I know this is a relatively minor thing overall, but I can easily see someone trying out Java functions for the first time being frustrated that the default generated unit test won't compile or run.
Note that this is using the following versions:
$ sfdx --version
sfdx-cli/7.183.1 win32-x64 node-v18.12.1
$ sf --version
@salesforce/cli/1.60.2 win32-x64 node-v18.12.1
$ sf plugins
functions 1.17.3

thanks @SCWells72.  the bug with the code you're referring to is located in another repository that handles the template generation process but i'll see that it gets addressed.

@SCWells72 this issue has been fixed in the latest sf CLI release candidate (v1.63.1) and will be included in the next weekly release.


Secret Scanning Alerts

In Github, I have over 2,000 Secret Scanning alerts. In Splunk, I only see about 500 of them. How can I pull the rest of the screts from Github or why am I not able to see historical data?

Hey @lorrainelopezFTFCU - The alerts in Splunk are webhook notifications.  When a new alert, or change to an alert occurs, a webhook is fired and sent to Splunk.  This means that historical alerts (before you configured the webhook) are not available. There is no backfill capability.
My suggestion is to use the security overview as the canonical list of alerts.
This Splunk integration is very good at showing activity during a specific time.  It's also good for building workflows and automation when an event occurs.  It's not able to provide a point in time count of open alerts.


Bump notifications-ruby-client to 6.2.0

Version 6.2.0 is compatible with this library. The only backwards incompatible change was removing the is_csv parameter from prepare_upload, which this library does not use.
We want to update to version 6 to make use of the ability to set the file name for file uploads, introduced in version 6.0.0.

@stephencdaly A new version of the gem has been released, 3.0.0. Thanks for bringing this to my attention.
I also took the opportunity to add support for a new feature (unsubscribe urls) introduced in version 6.1.0 of the notifications client 👍

@zheileman Amazing! Thanks so much for pushing this out so fast! ⭐


Improve request-response cycle with axios

Originally created by @gomorizsolt
We often use node-fetch. It does seem axios is much more suitable for client- & and server-side promise-based requests. Looking at its configuration, it's highly customizable and has several advantages as opposed to the (built-in) fetch library, e.g.:

supports cancel tokens
a base url can be set so that each request is prefixed with it
facilities error handling
it's no longer needed to convert the response to json/text
intercept requests & responses before they resolve/reject

We can also attach an adapter to axios so that the responses are cached in-memory for the given time(see #243).

I like fetch for keeping things simple and browser compliant. You do need to some things manually but you can use a function to do all that. E.g.
return fetch(url).then(response => {
  return response.text().then(textResponse => {
    if (response.ok) {
      try {
        return JSON.parse(textResponse);
      } catch (error) {
        throw new Error(`Could not JSON parse reponse: ${textResponse}. Error: ${error}`)
      }
    } else {
      throw new Error(`HTTP error: ${response.status} - ${textResponse}`)
    }
  });
})

supports cancel tokens

This is a very special use case. When needed let's use axios for sure.

a base url can be set so that each request is prefixed with it

Easily handled by a function.

facilities error handling

What does it do which isn't possible with fetch?

it's no longer needed to convert the response to json/text

Question for me is how does error handling work is this case? I don't want to see JSON parsing failed because ' is invalid, that isn't helpful.

intercept requests & responses before they resolve/reject

Is this useful for testing? Why is it better than e.g. mocking?

What does it do which isn't possible with fetch?

Refer to their example. There are several if-else branches for different error-cases, not sure it if makes sense at all though.
I can't really provide a clear explanation for the other two questions due to the lack of experience with the package. I also prefer to keep things simple if possible, but axios is quite popular for some reason.

What does it do which isn't possible with fetch?

Refer to their example.

I got that but I'm not sure how fetch behaves in this case. If we run into an actual issue that axios handles better I'm all for changing our defaults, but I need a use case. :)


Add unit tests

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@c2e68b8). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #52   +/-   ##
=======================================
  Coverage        ?   86.37%
=======================================
  Files           ?        7
  Lines           ?      279
  Branches        ?        0
=======================================
  Hits            ?      241
  Misses          ?       38
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


feat(Apple Notes): Added the function to open notes in separate windows

Description
Added the function to open notes in separate windows.
A new preference entry is also introduced to do this by default.
Closes #5496
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Gold! Thanks so much!


PACT-864 Modified to get legal statuses from message broker

@userjd , did you want to add the ticket key to your commit message?
It only says:

Modified to get legal statuses from message broker

Done

@userjd , did you want to add the ticket key to your commit message?
It only says:

Modified to get legal statuses from message broker

Done


bad links to sketch?

Summary
The page https://www.carbondesignsystem.com/resources#theme-libraries  has links that don't work for me (using Chrome browser, on Windows), but can be made to work with a simple modification:
Not Work:
sketch://add-library/cloud/JaVzz
Work:
https://sketch.cloud/s/JaVzz
Please consider making all the sketch://  links work for users who do not have sketch installed.

You need to have Sketch in order for these links to work, unfortunately they won't work on Windows.


Step counter goal

Most of the fitness tracker watches have some configurable step goal for each day (e.g. 10.000 steps).
So the watch can show the progress of reaching the set goal and can notify when the daily goal is reached.
Wouldn't it be great to have something similar here for the people who use this as a sports tracker as well?

Yes, sounds good. Maybe it could be added to the pedometer widget?

Good idea. Although I'm not sure how the information should fit in the limited space of the widget.
Maybe a clock with the step indicator would be a better idea?

I am currently working on my own version of the pedometer widget, which filters out arm movements.
This includes an active state (or inactive state).
My idea was to show this with the color of the widget.
Green = Active
Red = Inactive.
Maybe the goal could be displayed with the fill state of the icon?

I changed the display to one big number and a small number.
The big one shows 11k for 11834 steps.
The small one shows 11834.
The big one is red when inactive, green when active.
That improves readability quite a bit, unfortunately it does not leave any room for progress bar.

Didn't see the last few comments until I already made my pull request which displays the goal as a partial circle in the background. Maybe that could be adjusted to work with your changes?

@rigrig
Sorry to post here but after https://github.com/espruino/BangleApps/pull/260 got merged the widpedom widget seems to crash on my watch.
After i installed it and pressed BTN3 to reload interface the screen of the watch stays black and i can not open the main menu or anything else.
I can bring back the watch into a working mode only by removing the widpedom.
I am not sure how i can debug this on the watch itself as it seems not to happen on yours.

My editor made a bunch of autoformatting whitespace changes which I undid manually to not clutter up the commit. I must've broken it then and somehow failed to retest the final commit :-(


Add a tunnel option between the SSH and HTTP

It would be nice to have an option for the user to send some code snippet via SSH and block until the other side hasn't clicked the link, and it would stream the contents of the file directly to that user, and therefore eliminating the need to persist it on the server side altogether.

ssh codesnap.sh tunnel=true < example.go

// link returned, and the command is blocking, until someone clicks the link and a tunnel is established


fix: uppercase "Input" instead of "input"

I guess the code example in section "Integrating with global state" was supposed to use "input" (all lowercase) instead of "Input" component. Because there is no "Input" component being defined.

thanks


Fix 2519

Before:

Now (the list will be expanded if less or equal to 10 files):

Codecov Report

Merging #2539 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2539   +/-   ##
=======================================
  Coverage   94.41%   94.41%
=======================================
  Files         160      160
  Lines       19453    19453
=======================================
  Hits        18367    18367
  Misses       1086     1086

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 e7c5832...e356545. Read the comment docs.


an illegal memory access was encountered

CUDA_LAUNCH_BLOCKING=1 python train.py --name nyuv2_VGGdeeplab_depthconv --dataset_mode nyuv2 --flip --scale --crop --colorjitter --depthconv --list dataset/sunrgbd_training.lst
------------ Options -------------
batchSize: 1
beta1: 0.5
checkpoints_dir: ./checkpoints
colorjitter: True
continue_train: False
crop: True
dataroot:
dataset_mode: nyuv2
debug: False
decoder: psp_bilinear
depthconv: True
depthglobalpool: False
display_freq: 100
display_winsize: 512
encoder: resnet50_dilated8
fineSize: [480, 640]
flip: True
gpu_ids: [0]
inputmode: bgr-mean
isTrain: True
iterSize: 10
label_nc: 40
list: dataset/sunrgbd_training.lst
loadfroms: False
lr: 0.00025
lr_power: 0.9
max_dataset_size: inf
maxbatchsize: -1
model: DeeplabVGG
momentum: 0.9
nThreads: 1
name: nyuv2_VGGdeeplab_depthconv
nepochs: 100
no_html: False
phase: train
pretrained_model:
pretrained_model_HHA:
pretrained_model_rgb:
print_freq: 100
save_epoch_freq: 10
save_latest_freq: 1000
scale: True
serial_batches: False
tf_log: False
use_softmax: False
vallist:
verbose: False
warmup_iters: 500
wd: 0.0004
which_epoch: latest
which_epoch_HHA: latest
which_epoch_rgb: latest
-------------- End ----------------
CustomDatasetDataLoader
dataset [NYUDataset] was created
#training images = 795
model [BaseModel] was created
create web directory ./checkpoints/nyuv2_VGGdeeplab_depthconv/web...
/home/cgl/miniconda3/envs/torch0.40/lib/python3.6/site-packages/torch/nn/functional.py:1749: UserWarning: Default upsampling behavior when mode=bilinear is changed to align_corners=False since 0.4.0. Please specify align_corners=True if the old behavior is desired. See the documentation of nn.Upsample for details.
"See the documentation of nn.Upsample for details.".format(mode))
/home/cgl/code/Git/DepthAwareCNN/models/Deeplab.py:106: UserWarning: invalid index of a 0-dim tensor. This will be an error in PyTorch 0.5. Use tensor.item() to convert a 0-dim tensor to a Python number
self.averageloss += [self.loss.data[0]]
error in depthconv_col2im: an illegal memory access was encountered
THCudaCheck FAIL file=/pytorch/aten/src/THC/generic/THCTensorMath.cu line=26 error=77 : an illegal memory access was encountered
Traceback (most recent call last):
File "train.py", line 55, in
model.backward(total_steps, opt.nepochs * dataset.len() * opt.batchSize + 1)
File "/home/cgl/code/Git/DepthAwareCNN/models/Deeplab.py", line 112, in backward
self.loss.backward()
File "/home/cgl/miniconda3/envs/torch0.40/lib/python3.6/site-packages/torch/tensor.py", line 93, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph)
File "/home/cgl/miniconda3/envs/torch0.40/lib/python3.6/site-packages/torch/autograd/init.py", line 89, in backward
allow_unreachable=True)  # allow_unreachable flag
File "/home/cgl/code/Git/DepthAwareCNN/models/ops/depthconv/functions/depthconv.py", line 91, in backward
grad_weight = weight.new(*weight.size()).zero_()
RuntimeError: cuda runtime error (77) : an illegal memory access was encountered at /pytorch/aten/src/THC/generic/THCTensorMath.cu:26

I'm facing the same error. Any clue on solving this?

I have the same problem.

I have the same problem ，Could you solve the problem?

l also met this problem.
  self.depth = Variable(data['depth']).cuda() RuntimeError: cuda runtime error (77) : an illegal memory access was encountered at /pytorch/aten/src/THC/generic/THCTensorCopy.c:21
I try to the following methods, and finally succeed to train. l wish these can help you .
1.you can try to check your class of your datasets.  l use SUNRGBD datatset but l forgot to change the label_class.
2. try to add this sentence:  torch.backends.cudnn.benchmark = True
3.you can try to print data to check whether all right
4. l remove the data augmentation and it can succeed  to train , so l locate this error  to this. And then l add data augmentation one by one , it occurred to the scale, so l decrease the scale.

d

大佬，我没懂你的解决方法


[FEATURE] show amount of pixels immediately in box shadow

Description
Like the other generators where you get to see the amount of degree, updating when the slider is moved. The same should be for the sliders in box shadow generator, but instead of degree it should be pixels
Screenshots

Before contributing, read 👉🏾 this 👈🏾

It's great having you contribute to this project by creating an issue Make sure you are assigned to this before you work on it and you read the Contributing.md file, Thank you! Welcome to the community :nerd_face:

@Dun-sin Can I work on this?

@Dun-sin Can I work on this?

Definitely, thanks for wanting to contribute, Make sure you follow the rules here, else your PR will not be accepted
Good Luck

:tada: This issue has been resolved in version 2.14.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Hotfixes needed for v2.10.3

[x] Crashes on google group forum
[x] #2720
[ ] #2832
[ ] #2035
[ ] #2818

This has been closed in favour of 2.11 as discussed with @ashishkumar468 . There is no purpose in releasing such an old version when the upload refactor is nearly complete, and one of the major bugfixes in this list depends on the refactor.


^ appears before version number

A ^ character appears before the version number.

That's by design as it's updating the requirement (in JS the caret is the SemVer operator). If you have a lockfile then Dependabot will know exactly which version you're using, but without one it can only update the requirement.


Add support for redirects in plain routes

Fixes https://github.com/rackt/react-router/issues/1632
e.g.
let routes = [
	{ from: '/', to: '/begin' },
	{
		path: '/',
		component: App,
		childRoutes: [
			{ path: '/begin', component: Begin },

			{ from: '/about', to: '/about-us' },
			{ path: '/about-us', component: AboutUs }
		]
	}
];

This functionality is repeated here and in Redirect.js so there migh be a case for abstracting it out into a util

ping @ryanflorence @mjackson

I'm not 100% sure we want "plain routes" to be public API at this point. It all just feels like machine-generated code.

I agree it doesnt really feel like public facing API. But still quite limiting to not be able to do redirects.

How would you feel about them being more like a normal route object with path? e.g.
{
    path: '/',
    redirectTo: '/begin'
}

we don't need to touch the existing code, just need something like this:
I've done this, maybe should add it to the api? I dunno. Closing in favor of

```js
function redirect (from, to) {
  return {
    path: from,
    onEnter (nextState, transition) => {
      transition.to(to)
    }
  }
}

and then
var routes = {
  path: '/'
  component: App,
  childRoutes: [
    redirect('/users', '/people'),
    // ...
  ]
}

@ryanflorence ok this seems like a better solution

transition no longer has 'to' method so it should be just:
function redirect(from, to) {
  return {
    path: from,
    onEnter: (nextState, transition) => {
      transition(to);
    }
  }
}

The given alternatives don't work with parametrised routes whereas <Redirect /> does. Is there a way to do this without repeating/copypasting Redirect's functionality?


Status line doesn't show messages when running with --singleThread flag

Steps:

Run the program with the --singleThread flag.
Save the file.
'File saved.' will not appear on the status line.

Ah, I'd accidentally caused two (drawing) updates per (keyboard) event. So it would draw the 'File saved' and then draw again, clearing the text.


Update xorm version

Test new xorm version compatible with Gitea

One more question, shall we backport this to 1.20 because of https://gitea.com/xorm/xorm/pulls/2309 ?


[utils] Don't try to install or remove missing dlls

Same goal as https://github.com/doitsujin/dxvk/pull/1064 but touching only setup_dxvk.sh.
I can think of one instance this is going to silently fail. If the user already has dxvk installed in a prefix, with more dlls than their new build includes, it won't remove the missing ones. It is a side-effect of trying to make partially-built dxvk and d9vk co-exist under the same prefix without trying to remove each other. Is it something that should be handled?

Is it something that should be handled?

I don't think that's relevant. Most users are going to install the whole thing anyway, and d9vk should be merged eventually.


Improve the UI update after indexer/watcher

During testing, it appears that data is received but the UI is not updated with the new transactions.

This has been improved a lot and should not be an issue any longer.


Insert timestamp() problem.

I'm with this error in my tests
1) test users list_users/0 returns all users (NeopagApiStore.AccountTest)
     test/neopag_api_store/account/account_test.exs:22
     Assertion with == failed
     code:  assert Account.list_users() == [user]
     left:  [%NeopagApiStore.Account.User{__meta__: #Ecto.Schema.Metadata<:loaded, "users">, active: true, cpf: "some cpf", email: "some email", id: "5a217a7735dbc00cd7aeb96e", name: "some name", password: nil, password_hash: nil, phone: "some phone", inserted_at: %NaiveDateTime{calendar: Calendar.ISO, day: 1, hour: 15, minute: 51, month: 12, second: 19, year: 2017, microsecond: {33000, 6}}, updated_at: %NaiveDateTime{calendar: Calendar.ISO, day: 1, hour: 15, minute: 51, month: 12, second: 19, year: 2017, microsecond: {42000, 6}}}]
     right: [%NeopagApiStore.Account.User{__meta__: #Ecto.Schema.Metadata<:loaded, "users">, active: true, cpf: "some cpf", email: "some email", id: "5a217a7735dbc00cd7aeb96e", name: "some name", password: nil, password_hash: nil, phone: "some phone", inserted_at: %NaiveDateTime{calendar: Calendar.ISO, day: 1, hour: 15, minute: 51, month: 12, second: 19, year: 2017, microsecond: {33525, 6}}, updated_at: %NaiveDateTime{calendar: Calendar.ISO, day: 1, hour: 15, minute: 51, month: 12, second: 19, year: 2017, microsecond: {42219, 6}}}]
     stacktrace:
       test/neopag_api_store/account/account_test.exs:24: (test)

Basically, the user returned from my create_user function has a different timestamp when compared with the database one, returned from get_user.
def create_user(attrs \\ %{}) do
  %User{}
  |> User.changeset(attrs)
  |> Repo.insert()
end

def get_user!(id), do: Repo.get!(User, id)

def user_fixture(attrs \\ %{}) do
  {:ok, user} =
    attrs
    |> Enum.into(@valid_attrs)
    |> Account.create_user()

  user
end

test "get_user!/1 returns the user with given id" do
    user = user_fixture()
    assert Account.get_user!(user.id) == user
end

Can you please provide the version of the adapter that you are using?

@ankhers ecto-2.1

(https://github.com/ankhers/mongodb_ecto/blob/ecto-2.1/lib/mongo_ecto.ex#L503-L510) or load the data from the database.
Thanks for reporting this. I will look into it when I have a bit more time.


User Export of Object Metadata

As a digital scholar,
I would like the ability to download raw metadata about any subset of digital collections (including an entire collection), using fields of my choosing (title, creator, date, etc.) in some sort of regular format (.csv, json, etc.),
So that I may re-purpose the metadata in my own research projects.

Related: #99


Update WindowsBuild.md

Ensure that symlinks are handled properly on Windows, which are disabled by default.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test


Implement Chacha20 and Poly1305

Hello again!
This pull request implements the ChaCha20 cipher, the Poly1305 authenticator,
and the ChaCha20+Poly1305 AEAD described in RFC 7539. This addresses Issue #346
ChaCha20 and AEAD_ChaCha20_Poly1305 have been added to the Cipher module also.
I've implemented these algorithms by following RFC 7539. I referred to one of the RFC's informative references here to help fix some multiplication correctness issues that I had with Poly1305 whilst implementing the algorithm.
The test suite has been updated to include the test vectors from RFC 7539 for each of the algorithms.
Below are the benchmark results for ChaCha20 and Poly1305 individually, tested
on my system: Ubuntu 14.04 64-bit, Intel Core i7-2630QM, built using GCC 4.9.3.
AES-CBC-128              :     321493 Kb/s,          5 cycles/byte
AES-CBC-192              :     288173 Kb/s,          6 cycles/byte
AES-CBC-256              :     256147 Kb/s,          7 cycles/byte
AES-GCM-128              :     125628 Kb/s,         15 cycles/byte
AES-GCM-192              :     121279 Kb/s,         15 cycles/byte
AES-GCM-256              :     115015 Kb/s,         16 cycles/byte
AES-CCM-128              :     178437 Kb/s,         10 cycles/byte
AES-CCM-192              :     161727 Kb/s,         11 cycles/byte
AES-CCM-256              :     147757 Kb/s,         13 cycles/byte
CAMELLIA-CBC-128         :      60861 Kb/s,         32 cycles/byte
CAMELLIA-CBC-192         :      46225 Kb/s,         47 cycles/byte
CAMELLIA-CBC-256         :      47150 Kb/s,         40 cycles/byte
ChaCha20                 :     209225 Kb/s,          9 cycles/byte
Poly1305                 :     700353 Kb/s,          2 cycles/byte
BLOWFISH-CBC-128         :      71843 Kb/s,         26 cycles/byte
BLOWFISH-CBC-192         :      73082 Kb/s,         26 cycles/byte
BLOWFISH-CBC-256         :      71474 Kb/s,         28 cycles/byte

This change currently only adds the ciphers to mbedTLS. I haven't added any of the ChaCha20 cipher suites described in draft-ietf-tls-chacha20-poly1305-04, as I'd like some feedback first.

Gosh that was quick! You didn't really do this in a day did you?
Thanks very much for the contribution! It's going to take us a little time to review, and properly integrate, which we'll do at the next opportunity.

Haha no, I started working on it for mbedTLS a few days ago, since I'd like to use these along with mbedTLS on one of my own projects. I'm also familiar with ChaCha20 having previously implemented it in Ada about a year ago, so that helps.

Hello again,
I've just rebased my branch with the latest mbedTLS development branch to resolve the merge conflicts and to keep it more up-to-date.

@damaki Thanks again for your contribution and sorry for keeping you waiting for so long. I'm happy to announce that integrating these new primitives finally made it into our short term roadmap and we are aiming to include your contribution in our next release.
Even though your PR is very high quality already, there are inevitably a few things to rework. Since we feel it wouldn't be fair to ask you to do this rework in a short delay after keeping you waiting for  almost two years, we're going to do it ourselves in this new PR: https://github.com/ARMmbed/mbedtls/pull/1617
Please note that this is not how we intend to handle contributions in general, it is only a work-around while we try to handle our backlog of overdue PRs. In the future we certainly intend to react more quickly to incoming contributions and work with submitters on finalizing them.

Continued in: https://github.com/ARMmbed/mbedtls/pull/1617


Make out-of-scale postgres user defined duckdb type

I would like to followup on issue: https://github.com/duckdb/pg_duckdb/issues/467
I think current implementation is not ideal:

If user creates a table with duckdb unsupported numeric type, we silently fallback to double type
Later write operation doesn't always fail, based on the input numeric value, which adds more confusion to the users

Here I propose two improvements:

method-1: No silent fallback on type conversion "failure"; so we print a warning message with elog(WARNING, ...), and get users aware of the fallback behavior;
method-2: Instead of falling back to double type, we fallback to user-defined type, since it's not supported by duckdb (system) anyway.

In this PR, I selected method-2, but I'm open to other feedbacks and suggestions.
The goal here is to disallow silent fallback and surprising write behavior.

Thank you for the contribution.
The fact that we silently fall back to double has been bothering me too (see #309). I think it would be nice to make this a configurable setting that should probably be off by default, called something like duckdb.use_double_for_large_numerics.
However, afaict in the case you're describing this is actually not what is happening. In this case NUMERIC is converted to DECIMAL(18, 3), not double.

To be clear, I think it's probably reasonable to disallow specifying plain NUMERIC as the type for duckdb tables and require specifying a width and scale. But I don't think the current code actually has that effect.

Both suggestion makes sense to me, thank you! I will make changes these days.

Hi @JelteF , I tried to implemented your suggestion, which fails to create table if column type is not supported by duckdb.
I verified by existing SQL test statements with make check.

Hi @JelteF , I'm wondering if you could take another look when you have some time? Thank you!
Let me know if there's anything else I should change.

There's branch conflict and unit test failure, let me resolve them before re-requesting reviews.

@JelteF Sorry for the late response, and thanks for helping me trigger the CI last time.
I have fixed the failed sql test last time, should be good now.


modify ring

ring has been modified to not complain about ring_nf. Douple-check and remove ring_nf from the story.

This is still an issue! Interestingly enough, marking it as new (yellow) works as expected, but it's still available beforehand


Fix process.created -> created_time

Codecov Report

Merging #157 into master will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #157      +/-   ##
==========================================
- Coverage   93.97%   93.91%   -0.07%
==========================================
  Files          62       62
  Lines        5824     5764      -60
==========================================
- Hits         5473     5413      -60
  Misses        351      351

Impacted Files
Coverage Δ

stix2validator/v21/enums.py
94.80% <ø> (ø)

stix2validator/v21/musts.py
66.86% <0.00%> (-0.50%)
:arrow_down:

stix2validator/v21/shoulds.py
90.59% <0.00%> (-0.29%)
:arrow_down:

stix2validator/validator.py
89.23% <0.00%> (-0.25%)
:arrow_down:

stix2validator/v20/shoulds.py
95.33% <0.00%> (-0.16%)
:arrow_down:

stix2validator/v20/musts.py
93.33% <0.00%> (-0.14%)
:arrow_down:

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 40b6d2b...ba45a98. Read the comment docs.


Moving to virtualenv directory when activated

When you create a virtualenv through the following:
mkvirtualenv somevir -a ~/some/dir/

You should be taken to ~/some/dir/ when calling workon somevir
I realized that the path that a virtualenv should go to is saved in a file called .project within the virtualenv dir
Currently solving it by doing the following:
(add-hook 'venv-postactivate-hook
          '(lambda ()
             (when (file-exists-p (expand-file-name ".project" venv-current-dir))
               (cd (string-trim (with-temp-buffer
                        (insert-file-contents (expand-file-name ".project" venv-current-dir))
                        (buffer-string)
                        )))
               )))

Huh, I was totally unaware of this virtualenvwrapper feature! I agree we should move to the project directory if one exists. Better yet would be having a way to do the equivalent of the -a option from within emacs! For the first part, your implementation above looks good and I would happily merge a pull request to add it.

ref #35, which this is dup of, but this time around we have code to fix the problem and I have a better understanding of what the feature actually is 😁

I forked and made a commit but once testing it, it fails. Evaluating venv-postactivate-hook results in nil. Not sure what's up. Let me know if I missed something
https://github.com/porterjamesj/virtualenvwrapper.el/compare/master...gopar:master


Add the two clear cache APIs

Signed-off-by: alicejw<EMAIL_ADDRESS>Description
Added the POST //_cache/clear and POST /_cache/clear APIs. Both are in one file.
Note: This PR replaces the original content from PR 1093. We had an issue getting it checked in.
Also note - Alice's review changes are incorporated here, but technically we need 2 more doc reviewers.
Issues Resolved
N/A
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I've created an Asana request for editorial review.


Add CI

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@4e57548). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage        ?   24.88%
=======================================
  Files           ?       10
  Lines           ?      418
  Branches        ?        0
=======================================
  Hits            ?      104
  Misses          ?      314
  Partials        ?        0

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 4e57548...9d92fd3. Read the comment docs.


Header not clickable

@kanelloc-deploy Please check the video, header is not clickable may be some overlapping on there, please let me know if I am doing something wrong.
https://github.com/kanelloc/react-native-animated-header-scroll-view/assets/124288110/3cb330c0-5394-466c-8eb3-bb0eb2253d28

did you get around this? Facing the same issue rn

hi @dcnxx1,
Yes, I solved it by adding a condition while rendering the OverflowHeaderComponent in the library's source (src/components/AnimatedNavbar.tsx), and I added a patch to the library.
Before
<Animated.View         style={[           styles.container,           styles.overflowHeader,           {             zIndex: overflowHeaderOpacity,             height: headerHeight,             opacity: overflowHeaderOpacity,           },         ]}       >         {OverflowHeaderComponent}       </Animated.View>
After
{OverflowHeaderComponent && <Animated.View         style={[           styles.container,           styles.overflowHeader,           {             zIndex: overflowHeaderOpacity,             height: headerHeight,             opacity: overflowHeaderOpacity,           },         ]}       >         {OverflowHeaderComponent}       </Animated.View>}


Create integration tests for the "Validator" node

Integration tests need to be created for the Validator class (Thenewboston.Nodes.Validator) in order to test that all the operations work.

place the tests in a new file, Thenewboston.Tests/Nodes/ValidatorIntegrationTests.cs;
use xUnit;
create a "setup" method - the used bank should be customizable, not hard-coded;
make sure all the tests run;
if you find endpoints that are unusable, remove them;

Relevant discussion: #36

Hello I can take this one

Hello I can take this one

Hey :) Thanks for the interest, but unfortunately this project has been discontinued.


add dracut i18n_module required for systemd

add dracut i18n module required for systemd-vconsole-setup.

To reduce size we could remove the /usr/share/unimaps/ and /usr/share/keymaps/ folders

To reduce size we could remove the /usr/share/unimaps/, /usr/share/consolefonts/ and /usr/share/keymaps/ folders

The best way is adding a Gentoo package install mask for these paths.

To reduce size we could remove the /usr/share/unimaps/, /usr/share/consolefonts/ and /usr/share/keymaps/ folders

The best way is adding a Gentoo package install mask for these paths (needs to be done for the package that introduces them to the base image).

will try it out asap, thanks.

To reduce size we could remove the /usr/share/unimaps/, /usr/share/consolefonts/ and /usr/share/keymaps/ folders

The best way is adding a Gentoo package install mask for these paths (needs to be done for the package that introduces them to the base image).

Just to be sure on what we expect to be the reduced size, I took a snapshot of those 3 dir sizes:
1.3M    ./rootfs-1/usr/share/keymaps/
368K    ./rootfs-1/usr/share/unimaps/
960K    ./rootfs-1/usr/share/consolefonts/

To reduce size we could remove the /usr/share/unimaps/, /usr/share/consolefonts/ and /usr/share/keymaps/ folders

I tried this:
diff --git a/sdk_container/src/third_party/coreos-overlay/profiles/coreos/targets/generic/prod/make.defaults b/sdk_container/src/third_party/coreos-overlay/profiles/coreos/targets/generic/prod/make.defaults
index 6a34f638ba..2dc5af6747 100644
--- a/sdk_container/src/third_party/coreos-overlay/profiles/coreos/targets/generic/prod/make.defaults
+++ b/sdk_container/src/third_party/coreos-overlay/profiles/coreos/targets/generic/prod/make.defaults
@@ -61,3 +61,13 @@ INSTALL_MASK="${INSTALL_MASK}
   /sbin/ebtables-save
   /sbin/xfs_scrub_all
 "
+
+# remove unnecessary files from the sys-apps/kbd module.
+INSTALL_MASK="${INSTALL_MASK}
+  /usr/share/keymaps
+  -/usr/share/keymaps/include
+  -/usr/share/keymaps/i386/include
+  -/usr/share/keymaps/i386/qwerty/us.map.gz
+  /usr/share/consolefonts
+  /usr/share/unimaps
+"

But it is not enough, because it will do changes in the /usr partition, not on the initrd. So the previous way of trying this, with a custom dracut i18n module is required. Will continue by readding the custom module with the above required files.

I think it could work if you use sdk_container/src/third_party/coreos-overlay/profiles/coreos/targets/generic/make.defaults because then dracut won't be able to find these files in /build/amd64-usr and thus they don't get included in neither the initrd nor the production image.

Ready for review, tested and working with https://github.com/flatcar/scripts/pull/1679.

Looks good, but still would let @pothos review.


TypeError: Cannot read property 'theme' of undefined after first start

git clone --depth=1 https://github.com/ixartz/Next-js-Blog-Boilerplate.git website-next-js
cd website-next-js/
yarn
yarn run dev
results in:
yarn run v1.22.10 $ next ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000 info  - Using webpack 5. Reason: future.webpack5 option enabled https://nextjs.org/docs/messages/webpack5 error - ./node_modules/next/dist/compiled/css-loader/cjs.js??ruleSet[1].rules[2].oneOf[6].use[1]!./node_modules/next/dist/compiled/postcss-loader/cjs.js??ruleSet[1].rules[2].oneOf[6].use[2]!./src/styles/main.css TypeError: Cannot read property 'theme' of undefined

Hi @tbsvttr,
The project currently doesn't support yarn.
Please run the following command: git clone https://github.com/ixartz/Next-js-Blog-Boilerplate.git website-next-js
cd website-next-js/
npm install
npm run dev
I'm currently working to add the yarn support.