Running into issues when using s2image

For some reason it is failing on the node.
[My Setup]
3 node centos 7.0. I used the ansible installer to provision the cluster. For some reason it is not allowing me to use source to image. I'm using openshift origin 3.1 .
I thought to open a ticket under openshift/origin but after looking at this discussion it could be openshift-sdn:
https://github.com/openshift/origin/issues/6071
Here are my logs.
[root@node1 ~]# docker logs f17d3c09dfec
I0327 19:42:11.517302       1 builder.go:57] Master version "v1.1.4", Builder version "v1.1.4"
I0327 19:42:11.521576       1 builder.go:145] Running build with cgroup limits: api.CGroupLimits{MemoryLimitBytes:9223372036854775807, CPUShares:2, CPUPeriod:100000, CPUQuota:-1, MemorySwap:9223372036854775807}
I0327 19:42:11.524237       1 sti.go:195] The value of ALLOWED_UIDS is [1-]
I0327 19:42:11.524291       1 sti.go:203] The value of DROP_CAPS is [KILL,MKNOD,SETGID,SETUID,SYS_CHROOT]
I0327 19:42:11.530869       1 docker.go:287] Image "openshift/wildfly-100-centos7@sha256:7127899a3cb475e2a160b9a932147fb0c8d6c21a87f16fcb625d8a880537aac8" not available locally, pulling ...
I0327 19:42:11.530909       1 docker.go:309] Pulling Docker image openshift/wildfly-100-centos7@sha256:7127899a3cb475e2a160b9a932147fb0c8d6c21a87f16fcb625d8a880537aac8 ...
I0327 19:44:39.763505       1 sti.go:222] Creating a new S2I builder with build config: "Builder Name:\t\t\tWildFly 10.0.0.Final\nBuilder Image:\t\t\topenshift/wildfly-100-centos7@sha256:7127899a3cb475e2a160b9a932147fb0c8d6c21a87f16fcb625d8a880537aac8\nBuilder Image Version:\t\tf127204\nBuilder Base Version:\t\t0c1672f\nSource:\t\t\t\tfile:///tmp/s2i-build416721069/upload/src#master\nOutput Image Tag:\t\taxep/sample-jee-app-1:7899687a\nEnvironment:\t\t\tOPENSHIFT_BUILD_NAME=sample-jee-app-1,OPENSHIFT_BUILD_NAMESPACE=axep,OPENSHIFT_BUILD_SOURCE=https://github.com/bparees/openshift-jee-sample.git,OPENSHIFT_BUILD_REFERENCE=master\nIncremental Build:\t\tdisabled\nRemove Old Build:\t\tdisabled\nBuilder Pull Policy:\t\tif-not-present\nPrevious Image Pull Policy:\talways\nQuiet:\t\t\t\tdisabled\nLayered Build:\t\t\tdisabled\nWorkdir:\t\t\t/tmp/s2i-build416721069\nDocker NetworkMode:\t\tcontainer:f17d3c09dfec8f42cc7df82d716a6e2a76d0e213959e3b086224ba98c1ae7b09\nDocker Endpoint:\t\tunix:///var/run/docker.sock\n"
I0327 19:44:39.785854       1 docker.go:291] Using locally available image "openshift/wildfly-100-centos7@sha256:7127899a3cb475e2a160b9a932147fb0c8d6c21a87f16fcb625d8a880537aac8"
I0327 19:44:39.860141       1 docker.go:291] Using locally available image "openshift/wildfly-100-centos7@sha256:7127899a3cb475e2a160b9a932147fb0c8d6c21a87f16fcb625d8a880537aac8"
I0327 19:44:39.860218       1 docker.go:411] Image contains io.openshift.s2i.scripts-url set to 'image:///usr/libexec/s2i'
I0327 19:44:39.863973       1 sti.go:140] Preparing to build axep/sample-jee-app-1:7899687a
I0327 19:44:39.864581       1 source.go:197] Downloading "https://github.com/bparees/openshift-jee-sample.git" ...
I0327 19:44:40.716517       1 source.go:208] Cloning source from https://github.com/bparees/openshift-jee-sample.git
I0327 19:44:42.220767       1 install.go:236] Using "assemble" installed from "image:///usr/libexec/s2i/assemble"
I0327 19:44:42.220845       1 install.go:236] Using "run" installed from "image:///usr/libexec/s2i/run"
I0327 19:44:42.220895       1 install.go:236] Using "save-artifacts" installed from "image:///usr/libexec/s2i/save-artifacts"
I0327 19:44:42.220964       1 sti.go:152] Clean build will be performed
I0327 19:44:42.221003       1 sti.go:155] Performing source build from file:///tmp/s2i-build416721069/upload/src#master
I0327 19:44:42.221017       1 sti.go:166] Running "assemble" in "axep/sample-jee-app-1:7899687a"
I0327 19:44:42.221042       1 sti.go:424] Using image name openshift/wildfly-100-centos7@sha256:7127899a3cb475e2a160b9a932147fb0c8d6c21a87f16fcb625d8a880537aac8
I0327 19:44:42.221075       1 sti.go:428] No user environment provided (no environment file found in application sources)
I0327 19:44:42.221231       1 sti.go:530] starting the source uploading ...
I0327 19:44:42.243301       1 docker.go:411] Image contains io.openshift.s2i.scripts-url set to 'image:///usr/libexec/s2i'
I0327 19:44:42.243338       1 docker.go:466] Base directory for STI scripts is '/usr/libexec/s2i'. Untarring destination is '/opt/s2i/destination'.
I0327 19:44:42.243365       1 docker.go:616] Creating container  using config: &{Hostname: Domainname: User: Memory:0 MemorySwap:0 MemoryReservation:0 KernelMemory:0 CPUShares:0 CPUSet: AttachStdin:false AttachStdout:true AttachStderr:false PortSpecs:[] ExposedPorts:map[] StopSignal: Tty:false OpenStdin:true StdinOnce:true Env:[OPENSHIFT_BUILD_SOURCE=https://github.com/bparees/openshift-jee-sample.git OPENSHIFT_BUILD_REFERENCE=master OPENSHIFT_BUILD_NAME=sample-jee-app-1 OPENSHIFT_BUILD_NAMESPACE=axep] Cmd:[/bin/sh -c tar -C /opt/s2i/destination -xf - && /usr/libexec/s2i/assemble] DNS:[] Image:openshift/wildfly-100-centos7@sha256:7127899a3cb475e2a160b9a932147fb0c8d6c21a87f16fcb625d8a880537aac8 Volumes:map[] VolumeDriver: VolumesFrom: WorkingDir: MacAddress: Entrypoint:[] NetworkDisabled:false SecurityOpts:[] OnBuild:[] Mounts:[] Labels:map[]}, hostconfig: &{Binds:[] CapAdd:[] CapDrop:[KILL MKNOD SETGID SETUID SYS_CHROOT] GroupAdd:[] ContainerIDFile: LxcConf:[] Privileged:false PortBindings:map[] Links:[] PublishAllPorts:false DNS:[] DNSOptions:[] DNSSearch:[] ExtraHosts:[] VolumesFrom:[] NetworkMode:container:f17d3c09dfec8f42cc7df82d716a6e2a76d0e213959e3b086224ba98c1ae7b09 IpcMode: PidMode: UTSMode: RestartPolicy:{Name: MaximumRetryCount:0} Devices:[] LogConfig:{Type: Config:map[]} ReadonlyRootfs:false SecurityOpt:[] CgroupParent: Memory:9223372036854775807 MemorySwap:9223372036854775807 MemorySwappiness:0 OOMKillDisable:false CPUShares:2 CPUSet: CPUSetCPUs: CPUSetMEMs: CPUQuota:-1 CPUPeriod:100000 BlkioWeight:0 Ulimits:[] VolumeDriver: OomScoreAdj:0}
I0327 19:44:42.617109       1 docker.go:623] Attaching to container
I0327 19:44:42.620547       1 docker.go:629] Starting container
I0327 19:44:59.135018       1 cleanup.go:23] Removing temporary directory /tmp/s2i-build416721069
I0327 19:44:59.135564       1 fs.go:156] Removing directory '/tmp/s2i-build416721069'
F0327 19:44:59.154433       1 builder.go:204] Error: build error: API error (500): Cannot start container 4d83d07a426cc97232eb49bcc4292205d2307240cb4d6060632d1d018e896a20: [8] System error: write /sys/fs/cgroup/memory/system.slice/docker-4d83d07a426cc97232eb49bcc4292205d2307240cb4d6060632d1d018e896a20.scope/memory.limit_in_bytes: invalid argument

@bparees : I wanted to know if I'm missing anything? I'm using your source.

this is a known issue w/ the latest docker packages which has been fixed in origin here:
https://github.com/openshift/origin/pull/8202
strictly speaking the fix is in the openshift/origin-sti-builder and openshift/origin-docker-builder images, so if you pull the latest versions of those images to your nodes, and run your openshift start with the "--latest-images" flag (which causes the system to use the latest image available on the node) you should be able to get things working.

(oh and it's definitely not related to openshift-sdn, but hopefully my above advice will get you going and we can just close this out rather than worry about moving it)

Yes. We can definitely close this as you mentioned I could pass in an option their to get the latest images:  Would I do this on the master. I'm suspecting this is the file I'll be updating as it loads options from here:
/etc/sysconfig/origin-master
[root@master system]# cat /etc/sysconfig/origin-master
OPTIONS=--loglevel=2
CONFIG_FILE=/etc/origin/master/master-config.yaml

# Proxy configuration
# Origin uses standard HTTP_PROXY environment variables. Be sure to set
# NO_PROXY for your master
#NO_PROXY=master.example.com
#HTTP_PROXY=http://USER:PASSWORD@IPADDR:PORT
#HTTPS_PROXY=https://USER:PASSWORD@IPADDR:PORT
[root@master system]# cat /etc/sysconfig/origin-master
OPTIONS=--loglevel=2
CONFIG_FILE=/etc/origin/master/master-config.yaml

# Proxy configuration
# Origin uses standard HTTP_PROXY environment variables. Be sure to set
# NO_PROXY for your master
#NO_PROXY=master.example.com
#HTTP_PROXY=http://USER:PASSWORD@IPADDR:PORT
#HTTPS_PROXY=https://USER:PASSWORD@IPADDR:PORT
[root@master system]# cat origin-master.service
[Unit]
Description=Origin Master Service
Documentation=https://github.com/openshift/origin
After=network.target
After=etcd.service
Before=origin-node.service
Requires=network.target

[Service]
Type=notify
EnvironmentFile=/etc/sysconfig/origin-master
Environment=GOTRACEBACK=crash
ExecStart=/usr/bin/openshift start master --config=${CONFIG_FILE} $OPTIONS
LimitNOFILE=131072
LimitCORE=infinity
WorkingDirectory=/var/lib/origin/
SyslogIdentifier=origin-master
Restart=always

[Install]
WantedBy=multi-user.target
WantedBy=origin-node.service

Even after trying that I'm getting a different error:
F0328 07:49:27.970687       1 builder.go:204] Error: build error: Failed to push image. Response from registry is: Put http://<IP_ADDRESS>:5000/v1/repositories/loans/cakephp-example/: dial tcp <IP_ADDRESS>:5000: no route to host
I'm suspecting that my registry isn't setup correctly. I'll work on this some more but just wanted to double check if this is a bug?

Not sure about the issue on the top, but there is no bug for the connectivity to the docker registry.
Please make sure your docker registry pod is running well. You could be able to access your docker-registry on any of your nodes or any of the running pod with curl <IP_ADDRESS>:5000/v2/ (the ip may change if you re-created registry).

On the master:
[root@master centos]# curl <IP_ADDRESS>:5000/v2/
curl: (7) Failed connect to <IP_ADDRESS>:5000; No route to host

on node1:
[root@node1 ~]# curl <IP_ADDRESS>:5000/v2/
curl: (7) Failed connect to <IP_ADDRESS>:5000; No route to host

on node 2:
[root@node2 ~]# curl <IP_ADDRESS>:5000/v2/
{"errors":[{"code":"UNAUTHORIZED","message":"authentication required","detail":null}]}

In conclusion it looks like the registry was setup on node 2 and not reachable via curl on any of the other pods.

ok, then the docker-registry is running well on your node2.
So it should be some networking configuration problem for your cluster.
I suggest you to check
a) if the iptables rules for the docker registry in nat table exist iptables -nL -t nat | grep docker-registry
b) if the udp port 4789 on all the nodes is opened in default table iptables -nL | grep 4789

Sounds like you're past the cgroups issue, over to @knobunc to triage the network config.

Yes. thank you. I did get past the cgroup problem. Hoping to get some help on the network/sdn?

https://raw.githubusercontent.com/openshift/openshift-sdn/master/hack/debug.sh is a script you can run that will gather data about the cluster that can help us to debug. (It has to be run from a machine/account that can ssh as root to the master and each node.)
Did you reinstall everything after fixing the previous problem? This seems like the sort of thing that might just boil down to there being some lingering leftover brokenness from the previous bug.

@zmhassan could you try to restart one of your node (where the curl if failing) with :
systemctl stop atomic-openshift-node docker
rm /run/openshift-sdn
systemctl restart iptables openvswitch
systemctl start atomic-openshift-node

And try your curl again


Simplify controlled gate for SumOfProducts

Simplify the SumOfProducts control value when possible (if there's only a single product in it) when constructing a ControlledGate. This allows the X.controlled() -> CX optimization when using a SumOfProducts control value.
Add a two-step test to check X.controlled(num_qubits=2) -> CCX optimization, and include SumOfProducts check there too.
Make use of this simplification where we were hacking it in previously.
Fixes #5883

most optimization and inferences were implemented for the vanilla ways (as ints, lists ..etc) of defining control values, those were ported to ProductOfSums, but for SumOfProduct they didn't make sense in most cases, however in this case what you do in the constructor effectivelly transforms SumOfProducts to ProductOfSums representation, looks good to me

Automerge cancelled: A required status check is not present.
Missing statuses: ['Misc check', 'Pytest Ubuntu (3.7)', 'Pytest Ubuntu (3.8)', 'Pytest Ubuntu (3.9)', 'Pytest Windows (3.7)', 'Pytest Windows (3.8)', 'Pytest Windows (3.9)', 'Typescript lint check', 'Typescript tests', 'Typescript tests coverage']


avoid loop for closed polygons

Hi Daniel,
Thank you very much for ruby-geometry :-)
I've modified good_ray to avoid infinite loops, in case of closed polygons.
What I mean by "closed polygon", is a polygon which is defined by points A, B, C, D, A for example.
In such case, the function would return false infinitely.

Any test to reproduce?

Hi @kmmndr, thank you for adding this. At first I thought that zero length edge is a degenerate case, but I do understand that some people do define closed polygons as A, B, ..., A.
@PikachuEXE, good point on tests, this is my point of grief. It is very hard to write a test which reproduces particular cases in an algorithm based on randomness. Back in the day I didn't even know how to do it. Nowadays I think it is possible by further separation of responsibility of choose_good_ray method, but in expense of readability, I'm afraid. So, it is not fair to ask @kmmndr to write a test for his case when we don't have a "framework" for such tests in our codebase. What do you think?

This seems to fix #5
I just run my gist https://gist.github.com/PikachuEXE/71c6d0961e6bdc31846f for that issue
It does not take so much time to run choose_good_ray
Just remove git: 'https://github.com/DanielVartanov/ruby-geometry.git' then the bug can be reproduced

Hi all :-)
It is hard to write tests based on randomness, that's true.
But instead of choosing a fully random ray, one could use a kind of clockwise ray, what do you think ?
def random_ray(seed = 1.0)
  random_direction = (Math::PI / 2.0 * seed + Math::PI / (seed + 1.0)) % (Math::PI * 2.0)

  ray_endpoint = Point sufficient_ray_radius * Math.cos(random_direction), sufficient_ray_radius * Math.sin(random_direction)
  Segment point, ray_endpoint
end

In such case, we may have a randomly~predictable way to have a ray.
And we could test by fixing seed.


Passing the mouseover event from the hook : Update on useHover.js

Also passing event object on mouseover event.
Update in example:
const [ref, isHovered, event] = useHover();

Then:

{isHovered ? "Hovered ! at " + [event.pageX, event.pageY].join(",")   : "Hover me !"}

Yo @AnuragNtl !
This is a really nice improvement. Thank you for your proposal, I merge it !


"from the waist up"

Uttered while having a photo taken: "I like from the waist up."
Is this analogous to the double complementation in "from Boston to Providence", where "from" licenses two complements ("Boston" and "to Providence") (p. 641)? And is the NP complement an object?
(PP
    :Head (P :t "from")
    :Obj (NP [the waist])
    :Comp (PP
        :Head (P :t "up")))

I think it is analogous and that the NP complement must be an object.

Updated the CGELBank docs to mention this.


create APP_ROUTE constant

create an APP_ROUTE constant, so we don't have to mess around with paths like ../lib/foo

I think what you mean is the root path. Have a look to the root-patch branch

fixed with PR #66


init.lua:L467 "See :help cmp" is probably not correct

Describe the bug
The :help comment is misleading
-- [[ Configure nvim-cmp ]]
-- See `:help cmp`
local cmp = require 'cmp'

You probably want to point to nvim-cmp documentation, I don't know how to do that, though.
To Reproduce

go to line 467 in init.lua
follow instructions
be confused

Desktop

OS: Debian
Terminal: URxvt

Neovim Version
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/__w/neovim/neovim/build/nvim.AppDir/usr/share/nvim"

I don't really understand cmp very well so I don't think I can help here.
I'd love to, and maybe someday I will have a weekend to dedicate to plumbing its depths :)

Both aliases work. Try it :)

I don't think we need to change it.

@mech-a Works for me! Thank you for as always being so helpful.

Closing this. :help cmp works like a charm. And I have some reading to do :)


#122: Fix a path for querying bank connections method

@rultor merge

@rultor merge

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

@rultor merge

@proshin-roman Done! FYI, the full log is here (took me 2min)


Use hie-plugin-api

This way, plugins can be shared with HIE, and will make adopting GHCIDE more desirable for both HIE and end-users.

I think this project is a replacement for hie-plugin-api. The h-i-e plugin interface will have to be modified in a different way to work with ghcide.

I did read the hie plugin interface first, but it seems like a lot of code to turn products into sums and vice versa. I think our plugin interface is much simpler and more powerful. That said, no one has actually written a plugin against ghcide, so we need a first volunteer. It is probably possible to write a compatibility layer between the two systems, if someone wants to, but that's up to someone stepping forward.

First, let me say that I would very much like to see a merge or at least more collaboration between hie and ghcide but we are still working out how exactly that will proceed.
That said, I don’t think reusing hie-plugin-api is feasible since the two approaches differ fundamentally. hie-plugin-api basically lets you add commands that run in GhcModT monad which gives you a GHC session. In ghcide a plugin as a Shake rule that can depend on other Shake rules and we don’t even depend on ghc-mod.
I’m closing this issue since while as I mentioned, I would like to see more integration with hie, I don’t believe using hie-plugin-api is a good way of achieving that.


feat: eslint-plugin-case-police and VS Code plugin

Please read the Contributions Guide before making Pull Requests.

feat eslint-plugin-case-police
The effect is as follows

Also supports template string

current rules

Check the correctness of the string

For https://github.com/antfu/case-police/issues/116

Now also supports template string

The configuration options are the same as for the cli

I think I will use this to make a plugin for VS Code

Hi @antfu , I hope this message finds you well. Is there anything more I can do to help move it along?  Thank you for your time.

Add case-police for vue and jsx

Do we really need a VS Code plugin if ESLint is already able to handle it?

Do we really need a VS Code plugin if ESLint is already able to handle it?

I'm not sure if anyone needs it, so I haven't finished the vscode plugin. I can then remove the vscode plugin for now

I see. Let's focus on ESLint plugin first :) - Ping me when you are done and I will start reviewing. Thanks!

I see. Let's focus on ESLint plugin first :) - Ping me when you are done and I will start reviewing. Thanks!

OK, the vscode plugin has been temporarily removed. The review is ready to begin


perf: isSmartAccountActive param cache

Closes: #XXX
What is the purpose of the change
Inspired by https://github.com/osmosis-labs/osmosis/pull/8148
This param retrieval currently gets called 6 times for each tx. This requires unmarshalling all the params from the module, which is very wasteful.
Testing and Verifying
All currently existing tests that test the circuit breaker should behave the same. I will also be setting breakpoints in my QA for v25 on this feature to ensure the behavior is as expected.
Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[ ] N/A

Blocked on https://github.com/osmosis-labs/osmosis/pull/8148#discussion_r1586772250

This looks good. To make this easier in the future, I think we should have a global app.BlockCache and app.AppCache (caching for the duration of the block and caching for as long as the app runs) that we pass to keepers so it's easy to reuse.


Nick/precommit

Implemented Flake8 pre-commit and modified files to comply to Flake8 standard. One thing to note is that there was one small file that was deleted. This file was test/performace_check/check_memory_reading_ASDF.py. It was a small file containing 18 lines of code. Flake8 marked unused variables in this file and when I went to delete them it marked new unused variables and kept doing this until the file was basically empty. I discussed it with Carlos, and we assumed it was a one-off script and decided to delete the file. @niyiyu or @mdenolle could you confirm that the deletion of this file won't impact the rest of the project.

@nrich20 can you update the PR's title to something more descriptive since it'll become part of the commit history when you squash-merge?

Requested changes have been implemented. Please review and make any more suggested changes

New commit just dropped with the requested changes


fix: mark seed script as ESM after transform

This allows the seed script to use top-level await with Node 14, which the Babel plugin generates

Might need more changes, marking as draft till I figure out what they all are.

Okay done.
I've made changes in Cloud to start seeding via an ESM script, so these changes are required here. More details in the Slack message on #engineering-developer-productivity that I'm going to write.

https://prisma-company.slack.com/archives/G01R4M0GCRJ/p1632743034025200


Pseudo Irradiance Pipeline

Pull Request
Description
This adds the training datapipe, and related ones, for training the Pseudo Irradiance forecasting and labeling models.
Fixes #160
Fixes https://github.com/openclimatefix/pseudo-labeller/issues/5
How Has This Been Tested?
Unit tests

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Codecov Report

Merging #161 (6274b59) into main (50a3e42) will decrease coverage by 2.15%.
 Consider uploading reports for the commit 3a6fffb to get more accurate results

: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
@@            Coverage Diff             @@
##             main     #161      +/-   ##
==========================================
- Coverage   80.05%   77.91%   -2.15%
==========================================
  Files         118      121       +3
  Lines        4677     4835     +158
==========================================
+ Hits         3744     3767      +23
- Misses        933     1068     +135

Impacted Files
Coverage Δ

ocf_datapipes/training/pseudo_irradience.py
0.00% <0.00%> (ø)

...tapipes/transform/xarray/convert_to_numpy_stack.py
37.50% <37.50%> (ø)

...cf_datapipes/transform/xarray/create_time_image.py
40.00% <40.00%> (ø)

ocf_datapipes/transform/xarray/__init__.py
100.00% <100.00%> (ø)

...f_datapipes/transform/xarray/pv/create_pv_image.py
72.46% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


hi ,in my project，Some text change to a picture

how to resolve this problem。 thinks

Sorry, I don't know what you mean by this but this library doesn't have anything to do with images, it is solely used for text linking.


runtimes/rocm: bump ROCm to 6.2.2

This PR updates the packages to the one at https://repo.radeon.com/rocm/apt/6.2.2/dists/focal/main/binary-amd64/Packages and bump and bump the ROCM_VERSION constant.
Tested on MI210 with Llama 3.1 8B Instruct.

thanks


Feature for auto commenting on a new issue creation

I would like an automated feature that posts a predefined comment on an issue as soon as it is created. This feature aims to address the problem of delayed and inconsistent communication following the creation of issues. By automatically commenting on issues as soon as they are created, it ensures that contributors receive immediate feedback and acknowledgment for their efforts. This fosters a positive and supportive environment for collaboration, encouraging continued participation and engagement from contributors.
Could you please assign me this issue under GSSOC'24.

Issue already raised for this..
Issue #185


Material Design launcher icon

Please use a launcher icon in SVG based upon Material Design icons and colors. For this, special websites exist to create a launcher icon without having to open graphical design software.

We will be maintaining our current launcher icon for the foreseeable future. We'll switch to SVG if it's possible, I'll confer with our artist and see if there's a vector form of the original.


Dont assume that unit attribute exists

Fixes #769 .

@jenshnielsen I tested your changes. They do not fix the problem.
Looking a bit more at the traceback I think it may have to do with having multiple subplots. I can see that the first plot get's created correctly (out of 2 I am trying to create) but the second one does not.

Do you get the same traceback? or a different one?
Can you inspect how the  plot.traces dict looks normally there should be an element in the traces list for each subplot which in turn contains a config element which contains all the axis

I.e this is how it looks for me for a simple plot of a line graph and a 2d plot in
[{'config': {'figsize': (1200, 500), 'x': DataArray[6,6]: dac_ch2_set
   array([[ 0.,  1.,  2.,  3.,  4.,  5.],
          [ 0.,  1.,  2.,  3.,  4.,  5.],
          [ 0.,  1.,  2.,  3.,  4.,  5.],
          [ 0.,  1.,  2.,  3.,  4.,  5.],
          [ 0.,  1.,  2.,  3.,  4.,  5.],
          [ 0.,  1.,  2.,  3.,  4.,  5.]]), 'y': DataArray[6]: dac_ch1_set
   array([ 0.,  1.,  2.,  3.,  4.,  5.]), 'z': DataArray[6,6]: dmm_voltage
   array([[  25.,   52.,    2.,   17.,   73.,   68.],
          [  51.,   38.,   79.,   62.,   73.,  100.],
          [  60.,   72.,   20.,   37.,   56.,   51.],
          [   2.,   98.,   79.,   81.,   70.,   99.],
          [   4.,   71.,   10.,   39.,   11.,   30.],
          [  11.,    8.,   69.,   25.,   86.,   93.]])},
  'plot_object': {'cmap': 'hot',
   'hist': <ObjectProxy for process 24930, object 0x1db: <pyqtgraph.graphicsItems.HistogramLUTItem.HistogramLUTItem object at 0x117b7f948> >,
   'histlevels': (2.0, 100.0),
   'image': <ObjectProxy for process 24930, object 0x1d8: <pyqtgraph.graphicsItems.ImageItem.ImageItem object at 0x117b77a68> >,
   'scales': {'x': TransformState(translate=-0.5, scale=1.0, revisit=False),
    'y': TransformState(translate=-0.5, scale=1.0, revisit=False)}}},
 {'config': {'x': DataArray[6]: dac_ch1_set
   array([ 0.,  1.,  2.,  3.,  4.,  5.]), 'y': DataArray[6]: dac_ch1_set
   array([ 0.,  1.,  2.,  3.,  4.,  5.])},
  'plot_object': <ObjectProxy for process 24930, object 0x200: <pyqtgraph.graphicsItems.PlotDataItem.PlotDataItem object at 0x117b9c678> >}]

@jenshnielsen Below the traces and the plot.traces.
The problem occurs when in the same loop the second trace get's added.
I have to focus on my experiment now but I think this info should be useful.
Traceback :
d:\githubrepos\pycqed_py3\pycqed\measurement\measurement_control.py in measure(self, *kw)
    171     def measure(self, *kw):
    172         if self.live_plot_enabled():
--> 173             self.initialize_plot_monitor()
    174
    175         for sweep_function in self.sweep_functions:

d:\githubrepos\pycqed_py3\pycqed\measurement\measurement_control.py in initialize_plot_monitor(self)
    570                                      subplot=j+1,
    571                                      color=color_cycle[j % len(color_cycle)],
--> 572                                      symbol='o', symbolSize=5)
    573                 self.curves.append(self.main_QtPlot.traces[-1])
    574                 j += 1

d:\githubrepos\qcodes\qcodes\plots\base.py in add(self, updater, *args, **kwargs)
     89         # TODO(giulioungaretti): replace with an explicit version, see expand trace
     90         self.expand_trace(args, kwargs)
---> 91         self.add_to_plot(**kwargs)
     92         self.add_updater(updater, kwargs)
     93

d:\githubrepos\qcodes\qcodes\plots\pyqtgraph.py in add_to_plot(self, subplot, **kwargs)
    147         if prev_default_title == self.win.windowTitle():
    148             self.win.setWindowTitle(self.get_default_title())
--> 149         self.fixUnitScaling()
    150
    151     def _draw_plot(self, subplot_object, y, x=None, color=None, width=None,

d:\githubrepos\qcodes\qcodes\plots\pyqtgraph.py in fixUnitScaling(self, startranges)
    547
    548                     # set limits either from dataset or
--> 549                     setarr = self.traces[i]['config'][axis].ndarray
    550                     arrmin = None
    551                     arrmax = None

AttributeError: 'list' object has no attribute 'ndarray'

Inspecting traces:

More readable
[{'config': {'color': '#1f77b4',
   'symbol': 'o',
   'symbolSize': 5,
   'x': [0],
   'xlabel': 'Frequency',
   'xunit': 'Hz',
   'y': [0],
   'ylabel': 'Magn',
   'yunit': 'V'},
  'plot_object': <ObjectProxy for process 12880, object 0x1f: <pyqtgraph.graphicsItems.PlotDataItem.PlotDataItem object at 0x000000000564DE58> >}]

@AdriaanRol Thanks I will have a look

Your config object is quite different so the auto scaling will not work as is but I think this should be enough to keep the crash from happening

@AdriaanRol did you have a chance to test the latest version?

@jenshnielsen I didn't see you made any further changes. I'll have to check that out. Won't be today though, we are kind of deadline pressed at the moment.
On a related note, I think it would be good to add some unittests to the plotting to prevent these kind of issues in the future. I'd say they should only test if the code runs as testing for visual correctness is notoriously hard. If you are interested, I have a travis build script (based on the QCoDeS one) that also includes proper installation of pyqtgraph.

@AdriaanRol thanks thats fine whenever you have time. Unittesting is great but given that you are using it with a config object that is completely foreign to qcodes I don't see how we could have tested this without better visibility to what you are doing

@jenshnielsen . I'll try to create a unittest using only the default QCoDeS QtPlot API that reproduces the problem.
I think that should provide the visibility that you need and should prevent these kind of problems in the future.

Sounds good thanks

@jenshnielsen I've made a new PR containing a self contained test using only the default QCoDeS API that also contains all the work from your branch. Looking forward to your input.

Super-seeded by #805


fix freqstr in BaseOffset being a property

[ ] Closes #xxxx (Replace xxxx with the Github issue number) [does not apply]
[ ] Tests added: Please use assert_type() to assert the type of any return value

The freqstr is supposed to be a property, not a method returning a str
with this examples
from pandas import DateOffset

offset = DateOffset(minutes=10)
reveal_type(offset.freqstr)

and running mypy on this results in:
tt.py:4: note: Revealed type is "def () -> builtins.str"
Success: no issues found in 1 source file

with this patch it is properly captured as str:
tt.py:4: note: Revealed type is "builtins.str"
Success: no issues found in 1 source file

I ran into to this while doing this:
import pandas

delta = pandas.Timedelta(minutes=10)
offset = pandas.tseries.frequencies.to_offset(delta)

freq = None
if offset is not None:
    freq = offset.freqstr
    full_idx = pandas.date_range('2021', '2022', freq=freq)

tt.py:9: error: Argument "freq" to "date_range" has incompatible type "Callable[[], str]"; expected "Union[str, timedelta, Timedelta, BaseOffset]"  [arg-type]
Found 1 error in 1 file (checked 1 source file)

I'd be happy to add a test for this, but I'm not sure how - maybe you can give me a hint. I tried this, but it passed both times, so does not capture the error where Offset becomes a Callable[[], str].
def test_freqstr() -> None:
    offset = DateOffset(minutes=10)
    check(assert_type(offset.freqstr, str), str)

I'd be happy to add a test for this, but I'm not sure how - maybe you can give me a hint. I tried this, but it passed both times, so does not capture the error where Offset becomes a Callable[[], str].
def test_freqstr() -> None:
    offset = DateOffset(minutes=10)
    check(assert_type(offset.freqstr, str), str)

The above test should work.  If you remove your change, at least pyright picks up that assert_type will fail.  mypy should do that as well, although I didn't test that.
Can you add the test, then I'll enable the CI test.

thanks for looking into this! I added the test, but it passes also on main, so not verifying my fix.

Can you indicate how you tested it on main?
I did poetry run poe mypy and got this:
tests\test_timefuncs.py:1036: error: Expression is of type "Callable[[], str]", not "str"  [assert-type]

oh I was just using pytest.
When using poetry run poe mypy I get an error on main but it's passing on freqstr-property so exactly what I want!
But with poetry run poe pytest everything is passing (like when I was just using pytest)
I was not aware how the tests are supposed to be run - sorry.

oh I was just using pytest.
When using poetry run poe mypy I get an error on main but it's passing on freqstr-property so exactly what I want!
But with poetry run poe pytest everything is passing (like when I was just using pytest) I was not aware how the tests are supposed to be run - sorry.

If you do poetry run poe test, it will run mypy, pyright and pytest.  The first two do the type checking, which is what you are fixing. The last one checks that what we expect in the types is what happens at runtime.  At runtime, assert_type(x, a) just returns x which is why it worked with pytest on main


add start/completion timestamps to PodVolumeBackups, PodVolumeRestores

Currently we don't have start/complete timestamps on the restic operation CRDs. This would be useful for visibility purposes.

Do we want to expose the timestamps anywhere in the CLI output (e.g. when describing a backup)?

I don't think it's strictly necessary - will make the output a lot busier - but open to doing it if you want to.

Yeah I couldn't really see a place that made sense, and the backups already have their own start/completion timestamps so I imagine it might just be more confusing. I guess this will be more useful for when specifically debugging PodVolumeBackups/Restore operations.

yeah, the intent was mainly for debug, so I think that's fine for now. we can always revisit if needed.


Allow category tree filters on vtex legacy filters

Feature description:
This PR Allows the category tree filters to Vtex Legacy ProductListingPage. By sending to front the categories below the current department or category.
Use case:
https://www.austral.com.br/masculino?O=OrderByScoreDESC
https://australroupas.deco.site/masculino?O=OrderByScoreDESC

This PR is the same of the STD PR: https://github.com/deco-sites/std/pull/179 however without correcting the URL that returns the same page as this feature has already been corrected


Visualize Class doesn't show UI on Android

At Startup I have this page
https://www.dropbox.com/s/eb88uumttnjzfvo/Screenshot 2015-12-25 22.22.20.png?dl=0
If I press "Visualize Class" menu, I have this
https://www.dropbox.com/s/j5dzen5nral2m1l/Screenshot 2015-12-25 22.23.44.png?dl=0
Is it correct?

I believe this is correct. If you look at the visualization support for iOS and Android, you will notice that @praeclarum put some more love in the iOS one :)

Sorry @victorgarciaaprea I have not understand. You say that I can only visualize my page on iOS, and in Android I see those lists of words?

You're seeing the "fallback" visualization which @victorgarciaaprea noted is a bit lacking.
What is the inheritance hierarchy of the class you're visualizing? I'll make sure to add support.

It's a simple ContentPage

Ah, Xam Forms isn't support on Android yet.

Ok Frank, so can I use it with XF and iOS?

Yes, iOS should render the UI


Make extend_vec_zeroed, insert_vec_zeroed fallible

They now return AllocError on allocation failure. insert_vec_zeroed still panics if its position argument is out-of-bounds.
Closes #1653

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.01%. Comparing base (f998114) to head (21d63a2).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1658   +/-   ##
=======================================
  Coverage   88.00%   88.01%
=======================================
  Files          16       16
  Lines        5801     5805    +4
=======================================
+ Hits         5105     5109    +4
  Misses        696      696

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


docs: fix grammar and remove patch number from Go version

Fix a minor grammar issue

Summary by CodeRabbit

Documentation

Clarified the distinction between the Blobstream P2P network and other networks in the README.
Updated the installation instructions for improved clarity.
Corrected a minor grammatical error to enhance the text quality.

I don't have write access so @SweeXordious can you please merge after reviewing?


Broken queue for HP LaserJet 4 Plus in Mac OS Sierra

Operating System:
Mac OS Sierra 10.12.6 (16G1114)
Printer Software:
Driver Version: 10.4
PPD: HP LaserJet 4 Plus v2013.111
PPD File Version: 19.13
PostScript Version: (2013.111) 0
CUPS Version: 2.2.0 (cups-450)
The printer is on my LAN with a static IP address and a JetDirect card.
Problem Description:
Ever since upgrading to Sierra, the queue for my HP LaserJet 4 Plus is not working properly.  I can print a single job, the first job in the queue, but subsequent print jobs simply vanish from the queue whenever the printer is "busy".  The next job goes into the queue window saying "Ready to Print".  Then the pending job is finishing up and says "Waiting for printer to finish" until it disappears, and the next job goes to the top and says "Printing", but only for a second... it very quickly vanishes...
Watch Screen Capture Movie
It never makes it to the printer (no busy light, no blinking), it never prints, and there are no log entries or errors; it's as if this print job never existed.  If there are x number of jobs waiting, they'll each quickly fail in the same manner until the printer status changes back to "ready", and then if there are any jobs left, only the top one will print.
There is nothing wrong with the printer as another Mac running 10.5.8 can queue up as many jobs as desired and they all print.  This other Mac is using the same PPD version "HP LaserJet 4 Plus v2013.111" and Driver version 10.4.  My previous OS, Yosemite never had any trouble printing all jobs in the queue for this same printer & driver setup.
I've deleted and reinstalled the printer.  I've reset the printing system as per Apple procedure.  I've also tried a different HP JetDirect card.   None of this has resolved the issue.
I realize this printer is very old, however, it's a little workhorse and this model is on Apple's official list of supported printers.  The actual printing works perfectly well, and only the queue is not working properly.
On the same OS, I also have an old HP Color LaserJet 5/5M printer on the network and its queue is working properly with all computers, FWIW.

Please file macOS bugs with Apple directly (bugreport.apple.com) as CUPS does not include the driver you are using.


how to use inpaint.pt with img2img and Strength

it seems that the stength parameter is not available, it would be interesting to have an area where inpaint is needed and an area where the model paints based on the original image via the Strength parameter, and the area where it stays the same as the original image.
It could be a three color mask, the white area where the model will not paint, the gray area where the model will paint based on the Strength parameter and the black area where it will inpain.

with this repo you can do partial inpainting with the --skip_timesteps flag. So 90% denoising would be --steps 50 --skip_timesteps 5
this only works for inpainting though, and it would apply to the entire inpainting region.


remote window should occupy more screen

check the screenshot.
the remote window is just a small rectangle in the middle and i do not see any option to maximize the window. the blur area should be occupied by the window and hopefully an option to go fullscreen

Please attach to Screenshot.

I see you using Rustdesk, why did you used codext remotecontrol desktop and how did you found it?

@danielehrhardt its a long story. i was using and contributing to RD for a long time but lately they are migrating to paid model by removing features from "open source" version and adding them behind paywall. your project looks like a good alternative so i tried it. looks good enough


Video Version of the Tutorial: Publish Azure Static Web Apps with Azure DevOps

I have created a video version of this tutorial. You can have a look here: https://dotnetrumbles.wordpress.com/2021/04/01/publish-to-azure-static-web-apps-with-azure-devops/
It might be beneficial for somebody to go through the video version of the tutorial. If its possible, my video version can be added to the comment section of the page.
Thoughts ?

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

ID: 63230b97-2269-3fed-7fca-882bb380c5f6
Version Independent ID: feb2708e-5cea-4ad8-eaaa-f3061e18c11a
Content: Tutorial: Publish Azure Static Web Apps with Azure DevOps
Content Source: articles/static-web-apps/publish-devops.md
Service: static-web-apps
GitHub Login: @scubaninja
Microsoft Alias: apedward

@lohithgn Thanks for your feedback.
Assigning to content author @scubaninja for review.

Thanks for making this post and video, @lohithgn!
Right now we aren't able to provide a direct link to this page, but we really appreciate your support!
#please-close


Advanced exception rules

Why does this build have problems with the MagicNumber rule again? Does anyone have any idea?

Maybe you haven't branched this branch from the newest master where detekt RC4 is used for CI?

@arturbosch This might have caused the problem. However, my branch includes the update to RC4. Since all tests ran successfully I'd grateful if someone could review this changes.

Will do!

I like the new rules! Just wondering why you should not throw an exception in the main function?

@vanniktech Does it improve the readability? I think it's just syntactic sugar.

I prefer it that way since it's shorter


Improve Auditor Visibility & Stability

What would you like to be added:
Log more behavior from the Auditor:

[x] Log HTTP requests to GCR #358
[x] Log OS signals #389 #391
[ ] Enable & log graceful termination

Why is this needed:
In the past, the Auditor has experienced scaling and restarting issues. To improve future stability, it's important to capture behavior patterns that reflect current instabilities. Therefore, all Auditor issues will have more insight into how the the program is behaving.

/remove-lifecycle stale


feat!: remove ForestSimulator

Description
As part of the refactor of Backends we decided to remove Forest integration and thus the ForestSimulator class. This PR does so by removeing ForestSimulator along with the accompanying tests.
Please verify that you have completed the following steps

[x] I have self-reviewed my code.
[x] I have included test cases validating introduced feature/fix.
[ ] I have updated documentation.

Codecov Report
Base: 92.98% // Head: 96.51% // Increases project coverage by +3.52% :tada:

Coverage data is based on head (95959f7) compared to base (36567c7).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   92.98%   96.51%   +3.52%
==========================================
  Files           4        3       -1
  Lines         271      201      -70
==========================================
- Hits          252      194      -58
+ Misses         19        7      -12

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.


Weird bug on Chrome using Linux

Hello. I found a weird bug that happens on ArchLinux (at least) using Chrome. The screen flickers and movement seems dragged.
Brave and Firefox do not have this bug. Also works on Windows (Chrome included). I'm using a i5-1135G7 (with Intel TigerLake-LP GT2 [Iris Xe Graphics] GPU) , but it also happens on my old i3 (6th gen).
Curiosly only the arcade example works. Other libraries also seem to work fine (tested svelte-cubed and react-three-fiber).
https://user-images.githubusercontent.com/4981855/211649458-cb142932-50de-4fcf-9056-5478117b6d62.mp4

Hey @cristovao-trevisan, this is an interesting one. I have no idea where to start, the colors seem odd as well. Other examples don't work as well but the arcade example does? The arcade example uses a different rendering pipeline, but all other ones use the standard three.js WebGLRenderer, so there should not be any problems, especially since you tested other libraries relying on that as well.
I'd like to dig a bit deeper but i have the feeling this is not library-related. Feel free to join our Discord server so we can investigate your problem.

Knowning that the renderer is the problem I just tried it's configs and setting powerPreference to 'default' or 'low-power' removes the bug. The problem is probably related to the GPU driver, but I'm happy I got a fix.
Should high-performance be the default instead of 'default'?


[ruby] Add generic PURL for ruby

Syft has since added a bunch of additional binary classifiers, and since all of those are recognised as generic, we should use that list to create a PR of generic PURLs.


3D Block Renderer 2.0

[ ] Hangs indefinitely

http://scratch.mit.edu/projects/22721176/
http://phosphorus.github.io/#22721176

Seems fine now?

Yep. I guess they changed it.


[Bug] The default oap startup in .env under the docker directory fails

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
OAP server (apache/skywalking)
What happened
Started by docker compose, failed to start normally, here is the startup command, docker-compose --profile banyandb up -d

What you expected to happen
Start normally through docker compose, and give examples of mounting configuration files and data stores outside the disk
How to reproduce
Started by docker-compose file, the configuration has not been modified
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Follow quick start doc. Don't report bugs as you don't follow our guidelines.

Use the script in the document to start normally https://skywalking.apache.org/docs/main/v10.0.1/en/setup/backend/backend-docker/
Use docker directly The mirror in .env under the directory failed to start BANYANDB_IMAGE=ghcr.io/apache/skywalking-banyandb:84f32b3969cdcc676aaee428383b34b3b67dbdf5
OAP_IMAGE=ghcr.io/apache/skywalking/oap:latest
UI_IMAGE=ghcr.io/apache/skywalking/ui:latest
Changed to
BANYANDB_IMAGE=apache/skywalking-banyandb:0.6.1
OAP_IMAGE=apache/skywalking-oap-server:10.0.1
UI_IMAGE=apache/skywalking-ui:10.0.1
Can start normally,
Start command: docker-compose --profile banyandb up -d
Please check again if there is any problem.

The showcase has matched version up and running.
I don't know how you do with random versions.
oap 10.0.1 should be with BanyanDB 0.6.1
Dev versions(latest) is more about you knew what you are going to do. Showcase has versions listed in.


can't search keyword by :s or :o after upgrading to Firefox 46

The following error message printed:
resource://dactyl/util.jsm: 1583: TypeError: can't assign to properties of (new String(" ")): not an object

This is fixed in pull request https://github.com/5digits/dactyl/pull/151 so if you are building pentadactyl yourself you can merge that in manually.
But after you fix that you run into further issues with following links (backspace does not work and you get more errors of similar kind).

Fixed in git master. This can be closed.


Cleanup table of contents for select documentation

The existing one is very ugly, to the point of being embarrassing. I would not have approved that change originally. So we can either do this, or we can remove the TOC until we have something better. Staying the the existing is not an option.
Existing:

New:

@mosabua I agree with you in general, although this is the SELECT reference page. I'm willing to bet anyone looking at this already has a basic understanding of what this does.
I think we should change the documentation to include the inline table of contents on the right side, in a consistent way. The GitHub docs are a good example: https://help.github.com/en/actions/reference/workflow-syntax-for-github-actions

Yeah. Agree .. I was already looking at adding general ToC on left and page specific ToC on right.


Unavailable tables

I'm currently working with the dbt lab for snowflake and I have some issues relate to the tables. The database: "KNOEMA_ECONOMY_DATA_ATLAS" on the dbt lab works with the tables "exratescc2018" and "usindssp2020", but in the moment that I got this database on the marketplace, it didn't include those tables.
This a screenshot from the dbt lab:

That are the tables that I got from "KNOEMA_ECONOMY_DATA_ATLAS" database:

@carlohakkoda1 the extraescc2018 and usindssp2020 are views in that database.
If you expand the 'Views' you should see them.  Let me know if you are unable to see them, or if you are getting an error.

Yes, I know that they are a views, but the problem is there is a moment
that I call them as table not as a views, and that moment, dbt give me a
error
On Thu, 18 Nov 2021 at 09:02 Jason West @.***> wrote:

Closed #176 https://github.com/Snowflake-Labs/sfquickstarts/issues/176.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Snowflake-Labs/sfquickstarts/issues/176#event-5639407435,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWOS3D6BEOY56AHF6EXCNA3UMUIRBANCNFSM5H5E4LAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@carlohakkoda1 does the error state that the usindssp2020 view was defined as 12 columns, but the view query returns 13 columns?

I will do the lab again because honestly I did it last week and I didnt
remember the error the dbt gave me.
But I think that it is a compilation error. (I'm not sure).
On Thu, Nov 18, 2021 at 9:16 AM Jason West @.***> wrote:

@carlohakkoda1 https://github.com/carlohakkoda1 does the error state
that the usindssp2020 view was defined as 12 columns, but the view query
returns 13 columns?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Snowflake-Labs/sfquickstarts/issues/176#issuecomment-972960776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWOS3D4PF3D4BIWRSM4KSEDUMUKGDANCNFSM5H5E4LAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This is a first error that I got it:
How it's supposed to be:
[image: image.png]
What I got:
[image: image.png]
On Thu, Nov 18, 2021 at 9:28 AM Carlo Vargas @.***>
wrote:

I will do the lab again because honestly I did it last week and I didnt
remember the error the dbt gave me.
But I think that it is a compilation error. (I'm not sure).
On Thu, Nov 18, 2021 at 9:16 AM Jason West @.***>
wrote:

@carlohakkoda1 https://github.com/carlohakkoda1 does the error state
that the usindssp2020 view was defined as 12 columns, but the view query
returns 13 columns?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Snowflake-Labs/sfquickstarts/issues/176#issuecomment-972960776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWOS3D4PF3D4BIWRSM4KSEDUMUKGDANCNFSM5H5E4LAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Can't connect to VPN hosted on Azure when on LAN.

It started yesterday but it works fine when we connect to to it over WiFi.

If you are getting this error since yesterday, then it's something related to CIC, they are changing some things. It should be fine over a few days. Currently, I am out of campus so I can't debug or find a fix.

I will debug once I reach campus after the weekend

Sure thing,

Just updating, it doesn't work on WiFi now also 🥲

This will be resolved, will pursue CIC regarding this.

Any updates on this? Has CIC just straight-up blocked VPNs in general?

Sorry, no update yet. There is negotiation going on with the CIC. Please wait a couple more days

This place is not convenient to carry on the discussion. Prefer joining (MetaKGP Slack)[https://bit.ly/metakgp-slack]. And in that join #kgp-network channel.


MockedDOMSource is not assignable to DOMSource

The release of @cycle/dom v22.2.0 seems to have resulted in introducing a breaking change, though it's a minor version bump.
In 90645d6, DOMSource has been rewritten to a union type, and MockedDOMSource is intentionally excluded from this union.
https://github.com/cyclejs/cyclejs/blob/c0bb763a56d74dba63a8ddee1113973b1b7bace3/dom/src/DOMSource.ts#L14-L16
Before this commit, it was defined as an interface, which allowed us to assign MockedDOMSource to DOMSource.
https://github.com/cyclejs/cyclejs/blob/79344a4d288845bbb923a5548d8178681436971b/dom/src/DOMSource.ts#L11-L23
As a result, MockDOMSource is no longer assignable to DOMSource.

Additionally, it seems that we don't have any precise types to which both DOMSource and MockedDOMSource can be assign.
type Sources = { DOM: T }; // what should we expect for T?
type Sinks = { DOM: Stream<VNode> };

function Component(sources: Sources): Sinks {
    // ...
}

Of course we can define something like type T = DOMSource | MockedDOMSource, but this definition tsc will claim as following (as of typescript v3.2.2).
path/to/Component.ts:156:9 - error TS2347: Untyped function calls may not accept type arguments.

156         DOM.select('.incr')
            ~~~~~~~~~~~~~~~~~~~
157             .events('click')
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
158             .mapTo(1),
    ~~~~~~~~~~~~~~~~~~~~~~

To me, it appears that it is not a very good idea to define DOMSource as a union type, as TypeScript's inference over structual subtypes are not strong enough. (Or can it be possibly fixed with better typings?)
If possible, it would be great that DOMSource is defined as an interface again, with enabling TypeScript's strict mode.
Thanks!

Thanks for reporting this, @ryota-ka, I was fearing that this use case could appear as a breaking change, and it indeed did. The move to union type was a clever way of allowing us to "delete" types through TypeScript's "type ternary condition".
I understand your need for an interface.

@jvanbruegge Perhaps we could have some internal type used _DOMSource which is a union type, and then DOMSource which is an externally provided interface (or object type alias)? Not sure if that works, but just trying to look for a solution.

Is there any workaround for this or should I downgrade to a previous version to use mockDOMSource in a TypeScript project ?

@staltz @jvanbruegge While investigating this, I've noticed this is a problem caused by the variance of Streams and MemoryStreams.
(see also: https://github.com/staltz/xstream/issues/286)
I found that @jvanbruegge has supported TypeScript's strict mode in 90645d669f360edd792618e42512ea0d90da189a, but until the time we have {co,conta}variant subtypes of Streams, how about disabling strictFunctionTypes option?
My commit fixing this is here: https://github.com/ryota-ka/cyclejs/commit/e8532c96d2d9d5e2e177b2854837f285d0664d70
If it looks okay, I can submit a pull request.

Waiting for the fix, I've been doing the following :
export function intent(DOM: DOMSource | MockedDOMSource): Actions {
  // https://github.com/cyclejs/cyclejs/issues/869
  if (DOM instanceof MockedDOMSource) {
    return {
      addOne$: DOM.select(".add")
        .events("click")
        .mapTo(null),
      removeOne$: DOM.select(".remove")
        .events("click")
        .mapTo(null)
    };
  } else {
    return {
      addOne$: DOM.select(".add")
        .events("click")
        .mapTo(null),
      removeOne$: DOM.select(".remove")
        .events("click")
        .mapTo(null)
    };
  }
}

This is quite ugly, is there a better way ?

Sadly not at the moment. This won't be an issue in the new version, but that will still take a bit of time. Another workaround (what I've been doing) is in the tests type MockedDOMSource as any and pass that in.


[flutter_tools] initialize web stack trace formatter

Description
Ensure source maps are correctly initialized in debug mode, will be tested by https://github.com/flutter/flutter/pull/50652

Note that the stringy-ness of this code will be cleaned up when we can move to the require config defined in: https://github.com/dart-lang/webdev/pull/898


Migrate RemovePodsHavingTooManyRestarts to plugin

/retest

@BinacsLee thank You for helping us with the migration to plugins. It's great to see improvements in the code.

hi @ingvagabund , thanks for your review! I have fixed the code, PTAL. :)

@BinacsLee thank You for addressing the comment. Would you please rebase the PR as well?

@ingvagabund sure, rebase done. PTAL

sorry, will fix the test soon

/approve
/lgtm


How to show refresh indicator at starting when the user visits the page?

I want to show the refresh indicator every time the user visits the page at first for sometime

You can do this using initialRefresh property like this:
RefreshController _refreshController = RefreshController(initialRefresh: true);

It's not working...or may be it's because I am using page view and in one of the pages whenever the user swipes I want to show the refresh indicator...

@jagadish-pattanaik you can use _refreshController.requestRefresh when onPageChange is called.

用多个 tabpageview里都放一个SmartRefresher 那么点击某个按钮 只显示最后_refreshController.requestRefresh是成功的

用多个 tabpageview里都放一个SmartRefresher 那么点击某个按钮 只显示最后_refreshController.requestRefresh是成功的


fix: Change response tab name

This PR changes the response tab name from "Body" to "Response"

Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: fix/response-body-tab-name

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
56.49 (-0.01)
37.99 (-0.01)
36.29 (0)
56.71 (0)

:red_circle:
app/client/src/selectors/commentsSelectors.ts
83.61 (-1.64)
61.76 (-2.95)
73.33 (0)
88.24 (-2.35)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=bc9585d


While dragging a finger on the device, more than one tooltip shows up. How to only show one tool tip at a time?

Bugs and Questions
Checklist

[X] This is a victory-native specific issue. (Issues that also appear in victory should be opened here)

[x] I have read through the FAQ and Guides before asking a question

[X] I am using the latest version of victory-native

[X] I have checked to make sure that my versions of react-native and react-native-svg are compatible with this version of victory-native. Read about version requirements

[X] I've searched open issues to make sure I'm not opening a duplicate issue

The Problem
As a user scrolls across the graph, all the tooltips show up. I would like to only show one at a time, at the respective scatter dot location.

"victory-native": "^30.0.0"
"react-native-svg": "^6.4.1",
"react": "^16.4.1",
"react-native": "~0.56.0",
Reproduction
  <View style={styles.container}>
    <VictoryGroup
      containerComponent={
        <VictoryVoronoiContainer
          voronoiDimension="x"
          onTouchStart={this.props.disableScrolling}
          onTouchEnd={this.props.enableScrolling}
        />
      }
    >
      <VictoryArea
        groupComponent={<G />}
        style={{
          data: {
            fill: '#f7f7f7',
            stroke: 'rgba(200, 200, 200, 0.3)',
            strokeWidth: 1,
          },
        }}
        data={this.state.values}
        interpolation="monotoneX"
      />
      <VictoryScatter
        style={{
          data: { fill: '#00ff00' },
          labels: { fill: '#000' },
        }}
        size={(datum, active) => (active ? 4 : 2)}
        labels={d => `$${d.y}\n ${d.x}`}
        labelComponent={<VictoryTooltip />}
        data={this.state.values}
      />
    </VictoryGroup>
  </View>

@VicFrolov you might try adding your labels to VictoryVoronoiContainer and letting it be responsible for rendering the tooltip rather than VictoryScatter. You can avoid duplicate tooltips from VictoryArea with the voronoiBlacklist prop: https://formidable.com/open-source/victory/docs/victory-voronoi-container/#voronoiblacklist


error in extracting all tweets

hello!
I was trying to extract tweets by running
python3 /Users/Jham/src/getoldtweets3/bin/GetOldTweets3 --usernames-from-file userlist.txt --since 2017-01-01 --until 2017-12-31
but I keep getting the following error:
Found 94 usernames in userlist.txt
Downloading tweets...
Saved 600Traceback (most recent call last):
File "/Users/Jham/src/getoldtweets3/bin/GetOldTweets3", line 206, in main
got.manager.TweetManager.getTweets(tweetCriteria, receiveBuffer, debug=debug)
File "/Users/Jham/src/getoldtweets3/GetOldTweets3/manager/TweetManager.py", line 88, in getTweets
rawtext = TweetManager.textify(tweetPQ("p.js-tweet-text").html(), tweetCriteria.emoji)
File "/Users/Jham/src/getoldtweets3/GetOldTweets3/manager/TweetManager.py", line 190, in textify
if "u-hidden" in attr["class"]:
KeyError: 'class'
'class'
Done. Output file generated "output_got.csv".
Thank you!!

I got only small number of tweets..

I got the same error. My hypothesis is that a tweet contains an emoji that causes the error.  But I was not able to confirm this so far.

this is new right?  because I got a good result a few months a go

I found a workaround and created a pull request: https://github.com/Mottl/GetOldTweets3/pull/49

Fixed 49


perf($compile): only use document fragments and copy data when necessary (replaceWith)

This does a few things, all within the internal replaceWith...

only creates the document fragment if needed and clarifies why it's there
only copies data if the node hasData to avoid creating empty data to copy (a jqLite version of hasData was added)
uses the static data methods to avoid creating jqLite/jQuery objects
cleans the data of all replaced elements the same way (this will no longer trigger $destroy on nodes 1+, like node 0, which seems correct to me)
uses jqLite/jQuery/Array.splice to do the array modifications

So is this worth it? I think the code is simpler and explains some TODOs which is worth it on its own. The new benchmark app I'm not so sure about though, has anyone looked at it...?

These changes (excluding the benchmarks) have been broken up into smaller PRs, closing this...


O3.2.4 Moist Held-Suarez Perfect-Model Calibration

The Climate Modeling Alliance
Software Design Issue 📜
Purpose

We will implement a pipeline to calibrate on the moist held-suarez experiment in ClimaAtmos. We will calibrate on the horizontal temperature gradient.
Once this is complete, we will generalize to a pipeline that can handle other cases.
Cost/Benefits/Risks

Primary risk is developer time, but this will save us a lot of time for manual calibration in the long run.
People and Personnel
 @szy21 @trontrytel
Components

EKP interface: Configure EKP and construct the prior distribution.
Atmos interface: Configure and run the atmosphere model.
Data processing utilities: Needed for constructing the observation map
Visualization utilities: Used for comparing model output and determining useful information for the observation map

Inputs

EKP configuration: ensemble and iteration sizes
Prior distribution TOML
Observation map
Steady-state restart file

Results and Deliverables

[ ] Fully working and reproducible perfect-model calibration for moist held-suarez
[ ] Simple experiment configuration interface with tests
[ ] Inverse problem for the HS case

Task Breakdown And Schedule

[x] #5
[ ] Create interface
[ ] Moist HS observation map
[ ] Demonstrate moist HS perfect model calibration

SDI Revision Log

CC
@tapios @simonbyrne @cmbengue

Looks good to time. Let's target running this on GPUs right away, to increase throughput. This may then also necessitate some GPU compatibility work for EKP (if it isn't already compatible).


connect_proxy.js

Loading "connect_proxy.js" tasks...ERROR

TypeError: The super constructor to inherits must not be null or undefined.

Running "serve" task
Warning: Task "configureProxies:server" not found. Use --force to continue.
Aborted due to warnings.
And the app blows up from that point on

This repository is being archived. Sorry your question was not answered. The original owner of this repo left no one to take care of it.


Add eas support (sdm650)

Hello, would it be possible to add a config for the sdm650 with eas support?
Recently a kernel with eas support is being implemented. The device is kenzo (xiaomi redmi note 3 snapdragon)
https://github.com/trax85/Citrus-EAS
I would like to thank you for the work of the project. Thank you! I was using it for other kernels on my device, and I would like to be able to continue using it. Thank you again and I'm looking forward to the project!
Spanish user!😉
你好，是否可以为sdm650添加一个支持eas的配置？
最近正在实施一个支持eas的内核。该设备是KENZO（小米红米Note 3 snapdragon）。
https://github.com/trax85/Citrus-EAS
我想感谢你为这个项目所做的工作。Thank you! 我在我的设备上使用它的其他内核，我希望能够继续使用它。再次感谢你，我很期待这个项目!

No problem, sdm650-eas will be add in the next pre-release version, which is expected to be released this Sunday.

Great!! Thank you @yc9559!!!


Upgrade jdk version to 1.8

jdk=1.7 currently exists in some configurations, do we need to replace it?
https://github.com/apache/incubator-eventmesh/blob/efb2c118de077f82f83011ed13823e2c6f8b35c2/eventmesh-common/gradle.properties#L17-L19

@ruanwenjun sure, since it is just a configuration, you can set it default to 1.8.


PR policy

Thanks for posting all these data! :fireworks: :+1: :laughing: :1234:
So, do you have a pull request policy?
Since the data here are (presumably) generated from your internal collections management system, the usual pull request system may not work, as you would want to effect changes to either the content or presentation of these data in the upstream CMS and/or export scripts. If you want all suggestions to go through the link you provided, you might consider having a heading within your README.md on Pull Requests reiterating that. Similarly, if you would accept PRs for the documentation in the README itself, or to include new script examples, consider saying so explicitly (see the Carnegie Museum docs).

I assembled a collection of resources that may offer clarity on this topic. See http://bfy.tw/3aZm.

@dericed I think my question is not about "how pull requests work" per se, but about how the NYPL team would handle them for this particular repo. e.g. if I see a data formatting change I'd like done, is it worth me making the changes myself and submitting a PR, or would it be more efficient to submit an issue, assuming NYPL would have to replicate the scripted changes within their own data export workflow?

I don't at all speak for the project, but unless there is some documentation within the repo to indicate an exception to the expected use of GitHub, I'd suggest the following defaults:

a new pull request be initiated by sending a new pull request
a new issue be initiated by creating a new issue

@mdlincoln Thanks for the feedback! We've updated our documentation to include a pull request policy and to clarify our preferences for data correction suggestions. In short, we would love your pull requests for code examples, but we are not accepting pull requests for our datasets. You can send us metadata corrections through our Digital Collections feedback form or through email to<EMAIL_ADDRESS>If you have other suggestions for improving this repository, this is the place to do it.
Hope this helps,
Shawn

Brilliant! :+1:


Update FreeRTOS configuration

https://github.com/ARM-software/CMSIS-RTOS2_Validation/commit/3c25def91bb2dd67e04bb2b34cb4bfac3c1a11da


Clarify when the parent activity or window param can be actually optional

Clarify when you can omit the optional parent window parameter for WithParentActivityOrWindow  parameter in AcquireTokenInteractive  function. It'd be great to clarify that in the doc so this sort of customer question can be avoided.

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

ID: a7a27a93-a596-114e-6bc2-7a8ab5b20ef2
Version Independent ID: 54077d4d-6f1c-5a92-9d61-0533f85628fe
Content: Acquire a token to call a web API (desktop app) - Microsoft identity platform
Content Source: articles/active-directory/develop/scenario-desktop-acquire-token.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@chitturs Thank you for your feedback . We will investigate and update the thread.

We have updated the documentation as per the request and internal conversation. The section for withparentactivityorwindow  has been updated woth the information that you can omit to create a window or use the optional paramter WithParentActivityOrWindow if you do not want to control where the sign-in dialogs appear on the screen. The changes will be live in the next 48 hours.
Thank you.


Add GitHub Workflow for issue management

What type of PR is this:
/kind feature
What does this PR do / why we need it:
To ease project and issue management, this PR adds a GitHub Workflow that automates some manual tasks:

automatically labels new or reopened issues with the "needs-triage" label
automatically adds new or reopened issues to the odo Project.

Which issue(s) this PR fixes:
—
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:
Once merged, new or reopened issues should be added to the Project and labeled accordingly.

I see two jobs with one step each. Since both have runs-on field, I assume they would each need/spin up a container. Is it possible to combine them into one job to reduce the resources it would need?
My reasoning is to only be resourceful.

I understand your point.
For now, we have one step in each job, but I wanted to leave room for other steps we might think of later.
Also, the jobs being quite independent, it made sense to me to have different jobs that would run in parallel.

/approve

/override ci/prow/v4.11-integration-e2e
Not related to code, and tests are passing on IBM Cloud.


How to run repo on a mac with local folder?

I am having trouble due to setup not complete so I did the following
nltk
Download nltk files from https://www.nltk.org/nltk_data/
Punkt and extract to: Users › korayem > nItk_data › tokenizers > punkt
averaged_perceptron_tagger and extract to: Users › <username> > nitk_data › taggers > averaged_perceptron_tagger
Update requirements.txt
Many packages where not there. Add the following:
networkx
python-pptx
xlsxwriter
pillow
pandas
openpyxl

I am not sure what else is missing
Thought of sharing in case anyone else is stuck like I was

Hey @Korayem can you share the error logs?

Hey @assafelovic
It's was all trial an error night yesterday :)
I took a fresh git clone, added .env then started to get error loading networkx
Added that to requirements, run again, get an error loading python-pptx
and so on...
As for NLTK, I got
Resource punkt not found.
  Please use the NLTK Downloader to obtain the resource

import nltk
>>> nltk.download('punkt')
[nltk_data] Error loading punkt: <urlopen error [SSL:
[nltk_data]     CERTIFICATE_VERIFY_FAILED] certificate verify failed:
[nltk_data]     unable to get local issuer certificate (_ssl.c:1000)>

So after digging around, I found this answer that gave me a hint on how to structure the files
Btw, I still can't get it to work with local files. Last thing it asked for LibreOffice. I installed it. Then got another error. will share when I am back home tonight.

Welcome @Korayem ,
Perhaps a smoother path for you would be to run it through Docker Desktop.
Tutorial is here

Yeah, forgot to mention docker. I want to use it with my local files as mentioned, so I tried the new File Sharing >Virtual File shares feature in docker. I added the path to my local mac folder in env users/<username>/documents/folder. Also added DOC_PATHtodocker-compose.yml` to carry the value into docker container.
I got an error


Examples: make use of core utils in more places.

@twbs/css-review: If anyone wants to tackle the other examples, feel free to push on this branch.

Ping @twbs/css-review

BTW, not sure if we should backport this to v4?

BTW, not sure if we should backport this to v4?

Skip it

Someone needs to approve it :P


Can you add the tempo or bpm to flac ?

HI,
Can you add the tempo or bpm to flac ?
{
"danceability" : 0.596,
"energy" : 0.869,
"key" : 10,
"loudness" : -11.970,
"mode" : 1,
"speechiness" : 0.0370,
"acousticness" : 0.0116,
"instrumentalness" : 0.173,
"liveness" : 0.0678,
"valence" : 0.949,
"tempo" : 109.902,
"type" : "audio_features",
"id" : "05wIrZSwuaVWhcv5FfqeH0",
"uri" : "spotify:track:05wIrZSwuaVWhcv5FfqeH0",
"track_href" : "https://api.spotify.com/v1/tracks/05wIrZSwuaVWhcv5FfqeH0",
"analysis_url" : "https://api.spotify.com/v1/audio-analysis/05wIrZSwuaVWhcv5FfqeH0",
"duration_ms" : 238733,
"time_signature" : 4
}
Thanks

I can always try. Give me a little time to take a look.
On Sat, Jan 14, 2017 at 1:16 PM Pancho111<EMAIL_ADDRESS>wrote:

HI,
Can you add the tempo or bpm to flac ?
{
"danceability" : 0.596,
"energy" : 0.869,
"key" : 10,
"loudness" : -11.970,
"mode" : 1,
"speechiness" : 0.0370,
"acousticness" : 0.0116,
"instrumentalness" : 0.173,
"liveness" : 0.0678,
"valence" : 0.949,
"tempo" : 109.902,
"type" : "audio_features",
"id" : "05wIrZSwuaVWhcv5FfqeH0",
"uri" : "spotify:track:05wIrZSwuaVWhcv5FfqeH0",
"track_href" : "https://api.spotify.com/v1/tracks/05wIrZSwuaVWhcv5FfqeH0",
"analysis_url" : "
https://api.spotify.com/v1/audio-analysis/05wIrZSwuaVWhcv5FfqeH0",
"duration_ms" : 238733,
"time_signature" : 4
}
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hbashton/spotify-ripper/issues/11, or mute the thread
https://github.com/notifications/unsubscribe-auth/AL97hMO60f2mBR7fydxxXdZnZX3jB6wXks5rSR8cgaJpZM4LjtuV
.


LUA - Lua Uppercase Accident

nerdies keep using "don't talk to me until I had my LUA" so this punishes them for typing words without knowing what it means
caveats: needs message permissions explicitly enabled in the discord bot dashboard

Thanks a lot for your submission and your work!
While this feature can surely cause some fun for a short while amongst some people, it and its messages appear mostly annoying, spammy and even offensive at a level I wish not to introduce to the guild. Therefore, we've had some internal discussion with the staff off MCM, we'll not be merging this PR in its current form. Thanks for your understanding!


Update git2 to compile with latest rustc nightly

This should work around the issue rust-lang/rust#85574 by including https://github.com/rust-lang/git2-rs/pull/712.
(I haven't tested this change locally.)

Thanks for opening the PR!
0.13.12 is semver-compatible with 0.13.20, so you should be able to cargo update -p git2 on your project to get the latest version. Does that work for you?

Thanks for opening the PR!
0.13.12 is semver-compatible with 0.13.20, so you should be able to cargo update -p git2 on your project to get the latest version. Does that work for you?

Good point, thanks! Updating git2 on our project is enough to work around the issue.

Great! Closing the PR then.


Use more detailed exceptions when possible

This PR changes some error messages around to carry more details, but more importantly removes some roadblocks that were suppressing already-extant detailed error messages from being available for analysis. We've also added hooks on OneLogin_Saml2_Auth to retrieve the most recently sent request XML body, and the most recent XML response body.
Dependencies: None
Author notes and concerns:

I'm getting a segfault on the integration testing. That really shouldn't be happening for Python, period, so that's a Bad Thing. I'm gonna look into that further, but the really odd thing is that it's segfaulting even on a particular commit for my branch that passes on the main branch. Still not sure why this is happening, but I can't replicate when running the tests locally, so I have to assume it's something up with Travis.

[ ] @bradenmacdonald for initial review

@bradenmacdonald, this is ready for a first pass. Basically, we're updating the library to use more of its own low-level exceptions in error logging, and we've added the ability to retrieve the most recent request and response bodies.

@haikuginger The changes to auth.py (the ability to retrieve the most recent request and response bodies) are great and are the main thing we need here. With that, the next part of ENT-4 should be easy.
I'm not feeling as good about the other changes to the logging. A lot of the code in the Utils module is technically part of the public API, so changing methods to raise Exceptions rather than return False is a breaking API change. I think it's going to make this PR much more controversial than it should be. I think we'll get further faster by limiting the changes we make to ones that are backwards-compatible, even if that means we can't add more details to every place you've currently added details.

I commented on https://github.com/onelogin/python-saml/issues/98#issuecomment-241603717 because just using logging would really be the way to go here.
For now, I think you should limit your changes to those that don't break the API, then consider this done.

@bradenmacdonald, what would you think of having a raise_exceptions kwarg on methods in Utils, defaulting to False? There's a wealth of detail in that module that's just impossible to get to if we only "return False" form those functions. And, that'd preserve the default behavior of the API.

@haikuginger Yeah, that or an error callback param. Kinda ugly but that works within the constraints we have. However, could you split those changes into a second PR? The ability to retrieve the XML is the most important thing, and I'd like to separate that out and build on it now (ideally today) to complete ENT-4 by the end of the week; the other changes are "nice to haves" that we can submit soon and discuss with upstream to see if we can arrive at a workable solution over a longer term.

@bradenmacdonald, the first part has been split into #3.


Signature always returning empty true

Describe the bug
Since upgrading from 0.0.21 to 0.1.0 when checking for if the signature is empty it is always returning true, even if the signature as been signed.
The code that always returns true:
if (viewModel.signaturePadAdapter?.isEmpty == true) {
Smartphone (please complete the following information):

Device: Sony H8416
OS: Android 10
Library Version 0.1.0

Hey @ian-sayles
This should be fixed in https://github.com/warting/android-signaturepad/pull/207


Java and PHP didn't work for me.

Giving "awesome java" or "awesome php", returns:
>awesome java
Fetching the README from cache...
Done.
Traceback (most recent call last):
  File "/usr/local/bin/awesome", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.4/dist-packages/awesome/__main__.py", line 46, in main
    awesome_blocks = get_awesome_blocks(args.title, args.force)
  File "/usr/local/lib/python3.4/dist-packages/awesome/__main__.py", line 19, in get_awesome_blocks
    awesome_blocks = parser.parse()
  File "/usr/local/lib/python3.4/dist-packages/awesome/parsers/__init__.py", line 67, in parse
    awesome_blocks = self.parse_awesome_content(content)
  File "/usr/local/lib/python3.4/dist-packages/awesome/parsers/java.py", line 28, in parse_awesome_content
    plain_line, link = self.parse_link_line(line)
  File "/usr/local/lib/python3.4/dist-packages/awesome/parsers/__init__.py", line 116, in parse_link_line
    name, link, desc = self._markdown_link_line_regex.findall(line)[0]
IndexError: list index out of range
>

I'm on Debian Jessie (oldstable) x64 KDE.  Python3 v.3.4.2-2. PIP3 v.1.5.6-5 from repositories.

I've reproduced those issues on my env. I'll fix it ASAP.
Thanks for reporting bugs.
I'll inform you if done.

Congrats for this project!!! ;-)

@Achaean Thanks :)
I've fixed this issue and updated the version to 1.0.6 with https://github.com/mingrammer/awesome-finder/releases/tag/v1.0.6
Check it plz. If done, you could close this issue.

Success!!! ;-)
Now everything works flawlessly!!!


jax.ShapeDtypeStruct isn't documented

It is helpful for AOT compilation!

We should also probably update the documentation of jax.eval_shape to make use of jax.ShapeDtypeStruct rather than suggesting that users define a similar structure themselves.

We use types.SimpleNamespace to a similar end as an example throughout our reference documentation, e.g. here.
On the one hand, this isn't ShapeDtypeStruct. On the other hand, it helps emphasize that anything that duck-types is meant to work. Should we keep those examples as is, or give up on emphasizing the duck-typing point as part of the docstring example, and switch it over to ShapeDtypeStruct as well?

We should either document ShapeDtypeStruct or remove it from the API.
If we expect users to use duck types, we should describe clearly what attributes the duck types are expected to have.

Do you mean to state those as mutually exclusive? It seems at least possible to have SDS, document it, allow for anything that duck types like SDS on the input side, and always output SDS. With that, it might make sense to have the SDS symbol in the API for type annotations. (?)
We could entertain a protocol for annotations, but by the time we've written a class to define the protocol, it seems easy to fill it out into a concrete class for optional use too?

I didn't mean they were exclusive. I just mean without documentation, it's hard to work out what attributes matter, and maybe the best form of that documentation is a class like SDS.

I agree with @hawkinsp – using ShapeDtypeStruct in the docs is helpful in that it's self-documenenting. We could also mention that any duck-typed object will work. but defining a struct in the example makes it look more complicated than it needs to be.

Yeah, great, I agree!


Webhook Notifications - Add Instructions for Microsoft Teams

This minor change adds a row on how to enable Webhook notifications for Microsoft Teams.

lgtm

Thanks for the suggestion @benrfairless :)


Fix loaders, remove unnecessary annoying effects, separate errors

:pushpin: References

pdbe-molstar PR: Add toasts and remove clipping #11
Issue: Closes https://app.clickup.com/t/8693ftchc, https://app.clickup.com/t/8693cmvzu, https://app.clickup.com/t/8693cpf6m

Note: Some subtasks remain undone

:memo: Implementation
Main changes:

Separate the errors that are related to the viewer and those that are related to data.
Removed clipping
Fix MolecularStructure for undesired loads
Added toasts to pdbe-molstar
Decouple UniprotID

Fixes:

If main PDB is set hidden on URL params on init is not being applied on molstar (only on UI state)
If chain is added on URL params on init, highlight doesn't occur
Fix "Load PDB data" after any change on selection or visual change
When EMDB is set hidden, it shouldn't trigger new "Load PDB data"

:art: Screenshots

:fire: Testing
Params to try all at once: !6zow:B+EMD-11328|6aba+!6w9c on init

Thanks @tokland for your review! The lint warnings actually are kinda intended, it's a task related to "go to top" button. Now is bugged, so I removed it, that's why lint marks as unused


[DON'T MERGE] Remove barrier rdd for random forest

This draft PR is trying to remove BarriereRDD dependency for random forest algorithm. I will do the benchmark test, and if the random forest perf can't benefit from this PR, I will close this PR.

This PR doesn't help performance. Close this PR


added wrapper around build container

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Feature # (details)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration. Is it a  breaking change which will impact consuming tool(s).

[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@Tatsinnit any ideas on a better way to do this? The extension we are interfacing with doesn't return anything with the executeCommand function. This is extremely hacky but the only workaround unless we submit a PR into that repo (which we will probably want to do sometime). this is just a temporary workaround for demo.

@Tatsinnit any ideas on a better way to do this? The extension we are interfacing with doesn't return anything with the executeCommand function. This is extremely hacky but the only workaround unless we submit a PR into that repo (which we will probably want to do sometime). this is just a temporary workaround for demo.

☕☕ Cool, It is indeed, very interesting. Here are 2 options, It is absolutely normal for many extension co-exist but as long as its well thought as a user scenario. (Otherwise we could land in cross copy/paste code or hard to maintain features)
2 ways possibly could be:

Option 1: Let vscode-docker extension experience take over after exec call - Communicating directly with the extension command: (Let the run experience of the vscode-docker take over and don’t try and tenable the promise and handle separately. (Again I have no idea what this scenario is)

you can first do a getExtension() api check for the extension name publisher.name aka ms-azuretools.vscode-docker from package.json metadata of the docker extension.
Then check for activate() is important to make sure you have the extension object.
Once activation is true then: Invoke the command you want to execute.
Error handle and let the other extension take over the experience.

For example Kuberentes extension and AKS extension has couple of cross shared commands.
Option 2: Tricky but long shot - Use of pacakge.json -- When clause and just hook the command directly on the command of other dependent extension in this case docker extension .

Other user option: Otherwise let the docker extension take over the whole scenario, just let them invoke the docker command.
fyi @bfoley13, @qpetraroia, @sabbour  just for keeping this as fyi when proper delivery for specific aspect will happen. ❤☕
Thanks,


Word wrap needs to wrap multiple lines at a time

For the tab detail page.  See https://stackoverflow.com/questions/60794616/wrap-two-lines-at-a-time-in-an-android-textview-with-spans

Not quite done; need to fix zoom broken by this feature


Search queries should be case insensitive

Bug Description
When doing a search, an exact match should not be required, and a search should work regardless of the search query case.
Steps to Reproduce
Steps to reproduce the behaviour:

Go to FAQ page (https://thenewboston.com/faq/all)
Type "coins" (for example) and hit search. You will see some results. Now change the case of any letter (Coins or coinS or CoinS).
No results will be shown

Expected behaviour
The search should display results regardless of the case of the query
Actual behaviour
Results are shown only when a query is entirely in lower case
OS and Browser

OS: Windows
Browser: All

Additional context
Account Number
18c07b4227cb0993fa7a1db834834e42461a0d9f61b2b157dfbfee625bbf3de6

Hi, I will be working on this if it's alright

@avinashupadhya99 as a note, in the future, if there is a PR Reward label on an issue, feel free to begin working on it right away without getting permission from us!


fix readme

Coverage remained the same at 96.721% when pulling 091a292cce42ae66f1c5b75c805e9e22e9e2baf7 on feature/fixReadme into e88314dd51fa3b581e48e3fa47615a3a9f568190 on develop.


Create a shadow job  in CI for Spoon

Shadow job should call maven repair every commit in a pull request.

I'm working on it. FH

Hi @monperrus, do you think we should create this issue under issues with [spoon] https://github.com/INRIA/spoon/

no need, this issue is good here :)

FTR created https://github.com/SpoonLabs/spoon-shadow-job

Shadow job exist one with Spoon - https://ci.inria.fr/sos/job/shadow-job-spoon-maven/
The one with NPEFix - https://ci.inria.fr/sos/job/shadowJob-NPEFifx/

Shadow job exist one with Spoon - https://ci.inria.fr/sos/job/shadow-job-spoon-maven/
The one with NPEFix - https://ci.inria.fr/sos/job/shadowJob-NPEFifx/

A shadow job is for a target project to repair. So the one for Spoon makes sense, but the one for
NpeFix does not really.

A shadow job is for a target project to repair. So the one for Spoon makes sense, but the one for
NpeFix does not really.


PutObjectTaggingRequest: XML malformed error on Dell ECS

Discussed in https://github.com/aws/aws-sdk-net/discussions/2114

Originally posted by Tobias-08 September 20, 2022
Hi,
we are using Amazon S3 SDK (.NET) with Dell ECS. When sending a PutObjectTaggingRequest Dell ECS responds with a "xml malformed" error. I debugged this issue and learned that it is caused by a missing xmlns-attribute (xmlns="http://s3.amazonaws.com/doc/2006-03-01/") in the tagging-element. When I added the xmlns-attribute (manually in Fiddler) the request succeeded.
My question: Is it possible to inject the xmlns-attribute with means of the Amazon SDK (are there extensions points?)?
Thanks!
Tobias

While, PutObjectTagging does specify the below syntax for request, looks like the use of xmlns is not fully enforced by AWS S3 service.
PUT /{Key+}?tagging&versionId=VersionId HTTP/1.1
Host: Bucket.s3.amazonaws.com
Content-MD5: ContentMD5
x-amz-sdk-checksum-algorithm: ChecksumAlgorithm
x-amz-expected-bucket-owner: ExpectedBucketOwner
x-amz-request-payer: RequestPayer
<?xml version="1.0" encoding="UTF-8"?>
<Tagging xmlns="http://s3.amazonaws.com/doc/2006-03-01/">
   <TagSet>
      <Tag>
         <Key>string</Key>
         <Value>string</Value>
      </Tag>
   </TagSet>
</Tagging>

I tried using Fiddler, and see the below request sent to AWS service:
PUT https://<<some-bucket>>.s3.us-east-2.amazonaws.com/src.txt?tagging HTTP/1.1
User-Agent: aws-sdk-dotnet-coreclr/<IP_ADDRESS> aws-sdk-dotnet-core/<IP_ADDRESS> .NET_Core/6.0.9 OS/Microsoft_Windows_10.0.19043 ClientAsync
amz-sdk-invocation-id: 9bc9098c-8aa2-4e07-91c9-a6a24387513f
amz-sdk-request: attempt=1; max=5
x-amz-security-token: <<REMOVED>>
Host: <<some-bucket>>.s3.us-east-2.amazonaws.com
X-Amz-Date: 20220921T213857Z
X-Amz-Content-SHA256: <<REMOVED>>
Authorization: AWS4-HMAC-SHA256 Credential=<<REMOVED>>/20220921/us-east-2/s3/aws4_request, SignedHeaders=content-md5;content-type;host;user-agent;x-amz-content-sha256;x-amz-date;x-amz-security-token, Signature=<<REMOVED>>
Content-Length: 130
Content-Type: application/xml
Content-MD5: NI28Rt55/vKkjuZr+ZitIg==

<Tagging><TagSet><Tag><Key>tag1</Key><Value>value1</Value></Tag><Tag><Key>tag2</Key><Value>value2</Value></Tag></TagSet></Tagging>

Java SDK does add xmlns attribute to request XML:
2022-09-22 11:48:40 [main] DEBUG software.amazon.awssdk.request:85 - Sending Request: DefaultSdkHttpFullRequest(httpMethod=PUT, protocol=https, host=duck-mcbucket-asia-pacific-two.s3.ap-south-1.amazonaws.com, port=443, encodedPath=/this_file_has_%C2%AE_in_the_name.txt, headers=[amz-sdk-invocation-id, Content-Length, Content-MD5, Content-Type, User-Agent], queryParameters=[tagging])
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "PUT /this_file_has_%C2%AE_in_the_name.txt?tagging HTTP/1.1[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "Host: <<some-bucket>>.s3.ap-south-1.amazonaws.com[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "amz-sdk-invocation-id: 63ee5888-348b-48fc-8e5a-eaf23d6e8b58[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "amz-sdk-request: attempt=1; max=4[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "Authorization: AWS4-HMAC-SHA256 Credential=<<REMOVED>>/20220922/ap-south-1/s3/aws4_request, SignedHeaders=amz-sdk-invocation-id;amz-sdk-request;content-length;content-md5;content-type;host;x-amz-content-sha256;x-amz-date;x-amz-security-token, Signature=<<REMOVED>>[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "Content-MD5: vN/wdX/b89dvCvFA7dnJuQ==[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "Content-Type: application/xml[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "User-Agent: aws-sdk-java/2.17.267 Mac_OS_X/12.6 OpenJDK_64-Bit_Server_VM/11.0.14+9-LTS Java/11.0.14 kotlin vendor/Amazon.com_Inc. io/sync http/Apache cfg/retry-mode/legacy ft/s3-transfer[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "x-amz-content-sha256: UNSIGNED-PAYLOAD[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "X-Amz-Date: 20220922T184840Z[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "X-Amz-Security-Token: <<REMOVED>>[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "Content-Length: 169[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "Connection: Keep-Alive[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "Expect: 100-continue[\r][\n]"
2022-09-22 11:48:41,673 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "[\r][\n]"
2022-09-22 11:48:41,951 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "HTTP/1.1 100 Continue[\r][\n]"
2022-09-22 11:48:41,951 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "[\r][\n]"
2022-09-22 11:48:41,953 [main] DEBUG org.apache.http.wire -  http-outgoing-0 >> "<?xml version="1.0" encoding="UTF-8"?><Tagging xmlns="http://s3.amazonaws.com/doc/2006-03-01/"><TagSet><Tag><Key>tag1</Key><Value>value1</Value></Tag></TagSet></Tagging>"
2022-09-22 11:48:42,251 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "HTTP/1.1 200 OK[\r][\n]"
2022-09-22 11:48:42,251 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "x-amz-id-2: DGaAROftR+4y4X6wHYcmoNLmMWaJn6boPH4hsTbPD5YA69WdT886fJX3uphtxMm967e/bXXpZJc=[\r][\n]"
2022-09-22 11:48:42,251 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "x-amz-request-id: FMAKSKFACYQYSH9F[\r][\n]"
2022-09-22 11:48:42,251 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "Date: Thu, 22 Sep 2022 18:48:42 GMT[\r][\n]"
2022-09-22 11:48:42,251 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "Server: AmazonS3[\r][\n]"
2022-09-22 11:48:42,251 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "Content-Length: 0[\r][\n]"
2022-09-22 11:48:42,251 [main] DEBUG org.apache.http.wire -  http-outgoing-0 << "[\r][\n]"
2022-09-22 11:48:42 [main] DEBUG software.amazon.awssdk.request:85 - Received successful response: 200, Request ID: FMAKSKFACYQYSH9F, Extended Request ID: DGaAROftR+4y4X6wHYcmoNLmMWaJn6boPH4hsTbPD5YA69WdT886fJX3uphtxMm967e/bXXpZJc=

Even though S3 service does not complain and customer is using 3rd party service, we should be complaint with the API specification.

Looks like AmazonS3Util. SerializeTaggingToXml() is not setting the xmlns attribute.
Since S3 service is customized in SDK, we should check for other S3 service API operations that have the requirement of xmlns attribute.

Looks like almost all S3 bucket operations (https://docs.aws.amazon.com/AmazonS3/latest/API/API_Operations_Amazon_Simple_Storage_Service.html) should have xmlns attribute.