Implement state space object

Codecov Report

Merging #12 (945f66a) into main (ed28a13) will increase coverage by 0.10%.
The diff coverage is 100.00%.

:exclamation: Current head 945f66a differs from pull request most recent head f263b6c. Consider uploading reports for the commit f263b6c to get more accurate results

@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   98.58%   98.68%   +0.10%
==========================================
  Files           5        6       +1
  Lines         423      457      +34
==========================================
+ Hits          417      451      +34
  Misses          6        6

Flag
Coverage Δ

unit
98.68% <100.00%> (+0.10%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/dcegm/consumption_retirement_model.py
100.00% <100.00%> (ø)

src/dcegm/egm_step.py
100.00% <100.00%> (ø)

src/dcegm/solve.py
100.00% <100.00%> (ø)

src/dcegm/state_space.py
100.00% <100.00%> (ø)

src/dcegm/upper_envelope_step.py
96.49% <100.00%> (ø)

tests/test_integration.py
100.00% <100.00%> (ø)

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


Fix locksmith etcd-lock reboot strategy

Right now, the reboot-strategy "etcd-lock", which is the only suitable one for a HA cluster, doesn't work and is disabled by default (reboot-strategy: "off").
To make a production ready cluster, automatic updates which don't impair or disrupt the cluster's operation have to be possible.

@Thermi Thanks for creating the issue 👍
I personally prefer to turn it off for my production clusters so that I have a complete control over when to automatically upgrade Container Linux for nodes.
Would it be enough to allow selecting a reboot strategy per sub-clusters including etcd, controller, each worker node pool?

Related: https://github.com/kubernetes-incubator/kube-aws/issues/240

I think selecting a reboot strategy per sub-cluster would be fine.
I like clusters to take care of themselves as much as possible to keep down OPEX and possibility for human error.

Thanks for the confirmation 👍
I generally agree to you.
JFYI, probably you may already know but to be clear; I believe we need an extra care to have enough replicas and/or extra nodes to allow your services to be kept up and runnig while upgrading Container Linux over all the nodes. Please see the relevant discussion at https://github.com/kubernetes-incubator/kube-aws/issues/240#issuecomment-272134598

Yes, sure. That's the point of the etcd-lock: To ensure only one node reboots at a time.

Could we replace this one and https://github.com/kubernetes-incubator/kube-aws/issues/240 with an issue to support https://github.com/coreos/container-linux-update-operator?


There is no authenticated way to download installers.

The site www.googleplaymusicdesktopplayer.com is not accessible securely via https. This means there is no way to verify the authenticity of the downloaded installers which users are meant to run. This is a security risk. Please fix this by getting a certificate for the site/domain and using https, ideally by default.

If you do a trace the installers are not fetched from www.googleplaymusicplayer.com.
They are fetched from a SSL secured GitHub artifact domain and proxied to S3.

That doesn't help if the site linking to the artifact isn't secure. If someone can spoof the website, they can put any link they want there.

Getting an SSL cert for www.googleplaymusicdesktopplayer.com is an absolute nightmare because "google" is in the name.
TLDR, if you want a secure response use https://gpmdp.xyz

Alright, that seems reasonable.

FWIW as well, I managed to get a Let's Encrypt cert for the core site now as well.  It means not using Cloudflare though so we shall see how it goes
https://www.googleplaymusicdesktopplayer.com/


docs: use name for rule-doc-title-format

PR Checklist

[x] Addresses an existing open issue: fixes #133
[x] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview

@all-contributors please add @hyoban for doc.

🤖 Beep boop! This comment was added automatically by all-contributors-auto-action.
Not all contributions can be detected from Git & GitHub alone. Please comment any missing contribution types this bot missed.
...and of course, thank you for contributing! 💙


Output is not working when extension is active

Can't open Output using any ways. But when I disable this extension, everything is ok.

Closing for now due to inactivity


bug(python): num_sub_vectors validation doesn't trigger

LanceDB version
v0.5.3
What happened?
:assertion failed: sub_vector_idx < num_sub_vectors when trying to search vectors with an index.
Are there known steps to reproduce?
import lancedb
import numpy as np

rng = np.random.default_rng()

db = lancedb.connect("test")

data = [
    {"vector": rng.standard_normal(size=512, dtype="float32")} for _ in range(5_000)
]

tbl = db.create_table(
    "test",
    data=data,
)
tbl.create_index(
    num_partitions=256,
    num_sub_vectors=96,
    metric="dot",
)

tbl.search(rng.standard_normal(size=512, dtype="float32")).limit(1).to_list()

thread 'lance-cpu' panicked at /home/runner/work/lance/lance/rust/lance-index/src/vector/pq.rs:124:5:
assertion failed: sub_vector_idx < num_sub_vectors
thread 'lance_background_thread' panicked at /home/runner/work/lance/lance/rust/lance/src/utils/tokio.rs:34:24:
called `Result::unwrap()` on an `Err` value: RecvError(())
thread 'lance-cpu' panicked at /home/runner/work/lance/lance/rust/lance-index/src/vector/pq.rs:124:5:
assertion failed: sub_vector_idx < num_sub_vectors
thread 'Traceback (most recent call last):
lance-cpu' panicked at /home/runner/work/lance/lance/rust/lance-index/src/vector/pq.rs:124:5  File "<stdin>", line 1, in <module>
:
thread 'assertion failed: sub_vector_idx < num_sub_vectorslance-cpu
' panicked at /home/runner/work/lance/lance/rust/lance-index/src/vector/pq.rs:124:5:
assertion failed: sub_vector_idx < num_sub_vectors
thread 'lance-cpu' panicked at /home/runner/work/lance/lance/rust/lance-index/src/vector/pq.rs:124:5:
assertion failed: sub_vector_idx < num_sub_vectors
thread 'lance-cpu' panicked at /home/runner/work/lance/lance/rust/lance-index/src/vector/pq.rs:thread '124lance-cpu:' panicked at 5/home/runner/work/lance/lance/rust/lance-index/src/vector/pq.rs:
:assertion failed: sub_vector_idx < num_sub_vectors
124:5:
assertion failed: sub_vector_idx < num_sub_vectors
  File "/usr/local/lib/python3.10/dist-packages/lancedb/query.py", line 268, in to_list
    return self.to_arrow().to_pylist()
  File "/usr/local/lib/python3.10/dist-packages/lancedb/query.py", line 500, in to_arrow
    return self._table._execute_query(query)
  File "/usr/local/lib/python3.10/dist-packages/lancedb/table.py", line 1304, in _execute_query
    return ds.to_table(
  File "/usr/local/lib/python3.10/dist-packages/lance/dataset.py", line 397, in to_table
    ).to_table()
  File "/usr/local/lib/python3.10/dist-packages/lance/dataset.py", line 1883, in to_table
    return self.to_reader().read_all()
  File "pyarrow/ipc.pxi", line 757, in pyarrow.lib.RecordBatchReader.read_all
  File "pyarrow/error.pxi", line 91, in pyarrow.lib.check_status
OSError: Io error: Execution error: External error: Execution error: ExecNode(Take): thread panicked: task 16974 panicked

Hi @pshashk, num_sub_vectors should be set to an interget that divides the number of dimensions. I'm not sure why the safety check didn't trigger in your case. Could you try changing num_sub_vectors to something like 64.
ps: your num_partitions is also a bit too large. Usually you want to shoot for num_rows // 5000 ish (depending on the number of dimensions in your embedding vectors.

@chebbyChefNEQ Thanks a lot! That solves the issue.

Hi @chebbyChefNEQ. I was also seeing the same issue when using lancedb v0.5.3. I created an index with num_sub_vectors = 96  but my original vector size was 512. I only got the assertion error at search time and not during index creation.
Changing num_sub_vectors to 64 fixed the issue though. Thanks 😊


Add "your sessions" data inside dropdown

Currently the dashboard page becomes very long due sessions, which isn't the main interest. Adding each session inside it's own dropdown could work (or even consider removing the current session implementation after key data is visualized in charts).

Fixed, this is what it looks like now:

I will eventually remove the whole sessions list as you suggested, but that requires more charts, so it shouldn't be removed yet.


[WIP] Fixing TPFFactory to take in WCS

Adding in WCS coordinates so that they are not pulled from the template file.

Codecov Report

Merging #169 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #169   +/-   ##
=======================================
  Coverage   92.27%   92.27%
=======================================
  Files          17       17
  Lines        2292     2292
=======================================
  Hits         2115     2115
  Misses        177      177

Impacted Files
Coverage Δ

lightkurve/targetpixelfile.py
89.02% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@amcody, @christinahedges This is now finished, so if you could please review it that would be great!
Possible further work that Ann Marie and I discussed was to adapt the Cutout2D line of code (line949) in from_fits_images to account for the possibility of images without WCSs being passed in, as currently the code assumes the files have WCS.

I think the channel keyword is missing when you make TPFs with this? We need it for psf photometry.

There is a now a if wcs=None: in the code which should be wcs==None:
Not sure who did that, but can you fix it?

This is looking good to me now and it is working locally on M35 superstamp images. Anybody else have a comment, or can we merge? @barentsen?

@barentsen This is now updated with all the changes you mentioned, and @amcody 's comments. These changes did mean that I restructured my code such that the two helper functions _get_hdu and _get_pos_corrs were merged with from_fits_images. This does reduce unnecessary recalculation of certain values, but it makes me concerned about how from_fits_images is maybe not as general now, which is why Christina and I first decided to introduce the helper functions. If you have any thoughts on this, I'd love to hear them!

Also, does anyone know why one of the tests is now failing? It says something about the code taking too long to build.

@Emmavt Thanks Emma!  I do think it's a good idea to put the superstamp-specific aspects in a helper function, just not within the KeplerTargetPixelFile class, unless it is a direct operation on a specific target pixel file (which the function wasn't).  It can be a function in the module though (i.e. the same python file).
(also see comment above re the sky-to-pixel-to-sky conversion which confuses me a bit)

@amcody I wouldn't worry about the failing tests reported by AppVeyor. I'm pretty sure the failures are unrelated to this PR.

I think we should be able to simplify this code further.  The documentation on astropy's Cutout2D() says that it returns a WCS that is adjusted for the cutout, so I'm not sure I understand why we have to meddle with the WCS keywords ourselves?
Is it correct to say that we need to select one image to use as the reference WCS for the TPF, and use the WCS for all other images to compute the poscorr values?  (That's not quite what the code is doing right now?)

@barentsen that's exactly what I was thinking. I believe the reason why I was originally calculating the WCS keyword values myself was that I was under the impression we should have the TPF WCS info in the PC system, not the CD system (as are the superstamp images). But if it's fine to have the spatial information stored in CD keywords, then we should be able to eliminate a lot of code by just having cutout2D transfer the WCS from the middle image. Sound good?

Okay, we're working on these changes now. We do still need to know the detector coordinates of the corner of the TPF, and it looks like Cutout2D does not calculate this. So we have to leave a few lines of code in to do that. It's a bit complicated, as we need to know the exact center of the cutout in order to find the corners. And that depends on how Cutout2D is choosing the middle pixel (is it simply rounding the object's x,y position?).

Why do you need to know the corner coordinates?

Well, we figured out the behavior of Cutout2D just now.
We need to know the corner because TPF.column,row reports that (i.e., it goes into the the 1CRV5P and 2CRV5P keywords of the TPF header).

Hi all, I deleted some of the excess code to do with WCS header entries, as those should be filled by the Cutout2D function. I left in the entries to do with finding the corner coords. I also added in an if/elif statement to deal with the tpf possibly having even sides. The code still assumes a square tpf.

Whoops, the WCS wasn't getting passed into the new TPF. Emma is making more fixes... stay tuned.

@barentsen This should be finished now, I fixed a few bugs in the cutout2d line.


[Bug?] Proxy request causes infinite loop

If you add a handler that has proxy option like this
FakeXMLHttpRequest.addHandler({
    url: /projects\/(.*)\/services\/lostmycode/,
    proxy: (url, data) => {
        return "http://localhost:9090/" + url;
    }
});

you will get maximum call stack size exceeded error
Uncaught RangeError: Maximum call stack size exceeded
    at FakeXMLHttpRequest.open

So i think Line#166 should be new window.OriginalXHR() instead of new XMLHttpRequest()
https://github.com/thommyhh/fake-xmlhttprequest/blob/c4602cf1da63efda4f4a790808c2b5ac5e542bd6/lib/fake-xmlhttprequest.js#L165-L167
new window.OriginalXHR() worked for me
@thommyhh Sry for mentioning your old project. It's been 4 or 5 years since the last update, but I just wanted to help to improve this repo coz it really nice and helped me so much. Thank you.

@LostMyCode I'm sorry for the extremely late reply. I abandoned this, as there are better ways to solve the issue I tried to solve with this. Please see https://mswjs.io/.


[Feature] Have a feedback loop for Beta Widget.

Summary
Since we are launching Beta Labeled newer widget I think we can use some feedback from realtime users who will get to use it. This was we can also have a feedback system across the application.
Motivation
Building a non intrusive feedback system will help us understand user needs better.

@Debsourabh nice feature request. This also ties back to #3035 and should work for both. Can you look into how do products like Gitlab / Rudderstack do this?

Stats

Stat
Values

Reach
50

Effort (months)
0.25


Validate all modules are included in a GitHub Actions workflow

Background
Following up from:

https://github.com/gchq/sleeper/pull/3540#discussion_r1812579794

Description
We'd like the build to fail if any relevant module is missing from the chunks configuration for GitHub Actions.
Analysis
Chunks are configured in .github/config/chunks.yaml.
There's a step in the workflow .github/workflows/java-status.yaml to validate the chunk build configuration. Currently this will fail if there's a module with a Java test directory that is not included in a chunk.
We'd can validate that modules without a test directory are still included in a chunk. We'll need to come up with another validation criteria that excludes parent modules from the validation. We may also want to exclude the distribution module.

It looks like the current behaviour for this is fine.


[feature request] Local speech recognition

Token and data plan saver: perform the transcription of audio locally, which seems to have very good results, as it is part of Apple Siri and Google Assistant products, as well as voice input as used to speak text messages.

https://discord.com/channels/963222352534048818/984966420603482182/1210382075363065906

Ran into all sorts of problems because monocle couldn’t handle any sort of bandwidth. Using the phone across the room was more accurate also. It’s possible if it’s chunked but the lag makes for a slow  experience. Maybe frame has more throughput I’ll have to try it out when they land.

The Frame device does have a better Bluetooth bandwidth.
It is possible to choose a trade-off between low/high-resolution and low/high-bitrate audio for a compromise between bandwidth and speed.
There was not yet anyone to experiment with audio compression using StreamLogic, and audio compression for Frame was suggested here: https://github.com/brilliantlabsAR/frame-codebase/issues/134#issuecomment-2093907836
But it seems like the FPGA is already full with JPEG encoding, so a trade-off would be needed to fit FPGA-based compression.


deleting a node in linked list

deleting a node in linked list

Pr will be closed and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.


Fixing changed pushed without edit

Fixing structure of step.
Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

ping @jamtur01 @fredlf @SvenDowideit

@fredlf thanks for the catch Fred. Fixed.

LGTM

author LGTM

LGTM too, just waiting on Janky.


Implement Get Artifact API

Purpose

Retrieve a certain artifact in the cellery hub

Goals

Implement an API to fetch artifact data

Approach

User stories

Release note

Documentation

Training

Certification

Marketing

Automation tests

Unit tests

Integration tests

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Samples

Related PRs

Migrations (if applicable)

Test environment

Ballerina 0.991, MySQL 8.0.16

Learning

Codecov Report

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

@@           Coverage Diff           @@
##           master      #33   +/-   ##
=======================================
  Coverage   30.35%   30.35%
=======================================
  Files          42       42
  Lines         883      883
  Branches      137      137
=======================================
  Hits          268      268
  Misses        561      561
  Partials       54       54

Continue to review full report at Codecov.

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


CLOUD-677 Update README.md - pg helm

add helm
add raw files installation via kubectl

Test name
Status

demand-backup
failed

init-deploy
failed

upgrade
failed

scheduled-backup
failed

scaling
failed

smart-update
failed

recreate
failed

version-service
failed

affinity
failed

monitoring
failed

self-healing
failed

operator-self-healing
failed

clone-cluster
failed

tls-check
failed

users
failed

ns-mode
failed

commit: https://github.com/percona/percona-postgresql-operator/pull/203/commits/5aab99f45ab2cf25d2c6c114d19e6fe4f3abfcf8
image: perconalab/percona-postgresql-operator:PR-203-5aab99f4-pgo-apiserver
image: perconalab/percona-postgresql-operator:PR-203-5aab99f4-pgo-event
image: perconalab/percona-postgresql-operator:PR-203-5aab99f4-pgo-rmdata
image: perconalab/percona-postgresql-operator:PR-203-5aab99f4-pgo-scheduler
image: perconalab/percona-postgresql-operator:PR-203-5aab99f4-postgres-operator
image: perconalab/percona-postgresql-operator:PR-203-5aab99f4-pgo-deployer


npc

hello i am running a paper 1.14.4 server and i am having trouble on quest version 3.8.7 where i create a npc start quest and when i have fished setting up the quest i click on the npc to open up what quests they have and a gui opens with the quests inside i then try and click on the quest and it says in chat that i need to speak with the npc to start this quest but the only thing that comes up is the gui and that it i cant start the quest

I do have the same problem on my 1.15 server.
Even after removeing the files, redownloading it and staring from 0.

Duplicate of #1081
Use latest dev build from https://ci.codemc.io/job/PikaMug/job/Quests/

I had the same problem, but version 3.8.8 fixed it.
Thanks


Windows + Vagrant + Docker

I'm on Windows and using Vagrant to kick off a simple Dockerfile:
FROM phusion/baseimage:latest
When it runs:
RUN apt-get update
I hit:
Err http://archive.ubuntu.com trusty InRelease

Err http://archive.ubuntu.com trusty-updates InRelease

Err http://archive.ubuntu.com trusty-security InRelease

Err http://archive.ubuntu.com trusty Release.gpg
  Could not resolve 'archive.ubuntu.com'
Err http://archive.ubuntu.com trusty-updates Release.gpg
  Could not resolve 'archive.ubuntu.com'
Err http://archive.ubuntu.com trusty-security Release.gpg
  Could not resolve 'archive.ubuntu.com'
Reading package lists... Done
W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty/InRelease

W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty-updates/InRelease

W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty-security/InRelease

W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty/Release.gpg  Could not resolve 'archive.ubuntu.com'

W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty-updates/Release.gpg  Could not resolve 'archive.ubuntu.com'

W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty-security/Release.gpg  Could not resolve 'archive.ubuntu.com'

W: Some index files failed to download. They have been ignored, or old ones used instead.

Seems like a connection issue. Is there something I need to do in the Vagrant or Dockerfile in order for this connection to work?

Is this issue still open?


Check correct font: It should be "Roboto", often size 20.

In questionnaire/survey it's size 20
In consent it's 20 bold (titles) and 18 regular (body text)

Please review it.


Fix bug with fixed steps where transitions were not recomputed

Codecov Report

Merging #213 (d32c09b) into main (126603f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

: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     #213   +/-   ##
=======================================
  Coverage   95.61%   95.62%
=======================================
  Files          26       26
  Lines        1072     1074    +2
=======================================
+ Hits         1025     1027    +2
  Misses         47       47

Impacted Files
Coverage Δ

src/caches.jl
98.46% <100.00%> (ø)

src/perform_step.jl
98.09% <100.00%> (ø)

src/priors.jl
99.14% <100.00%> (+0.03%)
:arrow_up:

src/initialization/common.jl
95.83% <0.00%> (-0.17%)
:arrow_down:

src/diffusions.jl
98.63% <0.00%> (-0.02%)
:arrow_down:

src/solution.jl
98.64% <0.00%> (-0.02%)
:arrow_down:

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


Add example for usage with localhost

Triggered by #43

Closed with https://github.com/bartveneman/wallace-cli/commit/e011a394db0ffae283f1d3884b824911cf45592a


Unable to install multiple environments on the same machine (with default agent names) when running as a service

Agent Version and Platform
2.188.2 Windows Server 2019 Standard
Azure DevOps Type and Version
dev.azure.com
https://bolinda.visualstudio.com
What's not working?
Unable to install multiple environments on the same machine when running as a service. By default the agent name is the machine name. When installing an environment as a service the service name is defined as:
vstsagent.{0}.{1}.{2}
{1} is replaced with the poolName - however for environments - poolName is empty which results in:
vstsagent.bolinda..machinename
When we config another environment on the same machine (with the default agent name) - the existing service is replaced.
It would be ideal if the service name / display name included the environment name in place of where the pool name exists on build agents.
src/Agent.Listener/Configuration/ServiceControlManager.cs  generates the service name - but poolName is empty in the case of environments.
Workaround
Specify a different agent name (other than the default machine name) for each environment.

Hi, @bwgit Agent that contains the fix for the described issue has been released (v2.189.0). Could you test - does it works for you? Feel free to ping us if you will have any problem with it.

Yes this works perfectly now.
Thankyou!


Can't access apps after installing. Dev 0.6

Hello,
I am having issues with accessing apps after installing, i.e. I get 503 Service Temporarily Unavailable
I am working with 0.6 version of the Platform (dev environment), since newer ones don't support UI. Here's my configuration
inventory.yml configuration: https://paste2.org/JHxGjPOw
Output of a ansible-playbook ./install.yml run (at least the last part): https://paste2.org/yEVVWXZC
Error when trying to connect to the apps (portal, admin, formio): https://i.imgur.com/JJ60Ghn.jpg
Log files from assets, authentication, cases, cms, identities, records, services: log_files.zip (found at /srv/<log_name>/var/logs/dev.log)
Some additional information on the similar issue can be found here
https://github.com/DigitalState/Platform/issues/34
When deleting the platfrom I did docker system prune -a --volumes just in case.
List of images and containers: https://i.imgur.com/DGYMieJ.jpg
System specifications:
Windows 10 Pro
12GB RAM
Docker version 18.06.0-ce, build 0ffa825 (Docker for Windows)
2nd OS. Same error
Red Hat Enterprise Linux 7.5
Docker version 18.06.0-ce, build 0ffa825
Update: There is a warning in all of the logs:

{"message":"Warning: unlink(web/bundles/apiplatform): No such file or directory","context":{"exception":{"severity":2,"file":"/srv/api-platform/vendor/symfony/symfony/src/Symfony/Component/Filesystem/Filesystem.php","line":178}},"level":100,"level_name":"DEBUG","channel":"php","datetime":{"date":"2018-08-06 19:25:37.463896","timezone_type":3,"timezone":"UTC"},"extra":[]}

And, authentications has this after "extra":[]

", app":"authentication"

The problem was I have been using *lab.ds instead of *dev.ds (which is the correct link for dev environment).


Fix circle_avoidance

Short Description:

spin_forts was assigned twice in config

fort related arguments are now embedded in fort keyword

added embedded flag for parser, in order to handle nested keywords

Fixes:
#1544
circle avoidance now works as intended

This has another issue: inner keys lacks any reference in the same to outter ones and this is bad. I have one that will handle it properly. Will open PR for u to see.

Have a look #1590


Performance issues between version on official mxnet and version on mxnet-ssd

I train a model on both version :

the version in official repos example
the version here

I got really bad results when I use the version in official repository
(I use exact same input parameters and datas)
Have you experience same issues ?

Could you provide more details?

I use the exact same data and configuration :

on your repos,  I got:

INFO:root:Epoch[26] Train-Acc=0.886437
INFO:root:Epoch[26] Train-ObjectAcc=0.671642
INFO:root:Epoch[26] Train-SmoothL1=5.021330
INFO:root:Epoch[26] Time cost=2307.235

that was already a quite good detection

on mxnet/example/ssd :

INFO:root:Epoch[26] Train-Acc=0.785770
INFO:root:Epoch[26] Train-ObjectAcc=0.182737
INFO:root:Epoch[26] Train-SmoothL1=20.170220
INFO:root:Epoch[26] Time cost=1794.627
...
INFO:root:Epoch[55] Train-Acc=0.782617
INFO:root:Epoch[55] Train-ObjectAcc=0.163265
INFO:root:Epoch[55] Train-SmoothL1=21.588834
INFO:root:Epoch[55] Time cost=1795.653

I think the issue was the convolution tuning :
without it I already got (and it's already in ascending slope):
INFO:root:Epoch[0] Batch [2780]	Speed: 32.27 samples/sec	Train-Acc=0.805153
INFO:root:Epoch[0] Batch [2780]	Speed: 32.27 samples/sec	Train-ObjectAcc=0.366649
INFO:root:Epoch[0] Batch [2780]	Speed: 32.27 samples/sec	Train-SmoothL1=15.288175

I see, could be something related to convolutional padding mode. I will track this.

Is it possible to convert model from your repos to official mxnet/example/ssd ?

I will try other parameters tuning, according to your answer :
https://github.com/dmlc/mxnet/issues/4120#issuecomment-266305244

Maybe i got my mistake, on my pc the version of official repos don't use cudnn.
And it seem to work know.


[WIP] Change postgres port

Solves #231
The community says the best way to run two projects in parallel is to just switch port, so I think we should do that. Is there any other options, like a command that creates a new database within the original container.

I think it is easiest to use the default port in ci tho

You might be right there. I just have to find out how

I'm closing this because I made a new branch to try to solve the whole parallel thing, and not just address the PostgreSQL side of the issue.


⚠ Bofrost.be has degraded performance

In 7d9198b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1855 ms

Resolved: Bofrost.be performance has improved in ffe4de3.


Support guest use of VS Code extensions that currently access the local filesystem directly

Note that developers starting the collaboration session (hosts) do not have this limitation. It is specific to guest joining a session.

Many VS Code extensions use standard interface specifications like the language server protocol that abstract them away from local file system access.  These along with colorization, theming, key maps, snippets, etc will "just work" for guests that have joined a collaboration session.
However, there are extensions that use features that either:

Go directly at the file system (e.g. via the fs module)
Shell out and execute command line tools that are expected to be installed locally

In some cases these extensions just go to the file system for a configuration file (e.g. eslint) and otherwise function. Others will error out. There are a number of potential solutions to these problems that we are investigating.
Note that the person that initiated the collaboration session (host) will not run into any of these limitations.
Sound off in the comments below with any critical extensions you need that are not working as expected.

Beyond that, it'd be awesome for guests to be prompted to install workspace/project-wide extension recommendations automatically (if this doesn't already happen), to make the editing experience (esp. wrt colorization and other in-editor behavior) more homogeneous between guests and host.

@tdd Really appreciate the feedback!  One follow up question since I really like your suggestion.
In your view, would it be preferable to:

Prompt for each extension
Prompt for all extensions that are missing at once (and hitting OK would then install them)
Automatically do the install regardless
Automatically install but sandbox the extensions so that they're only available during the collaboration session

Hey @Chuxel !
There are two bits that inform my reply here:

I'm giving a lot of training classes where we strongly recommend Code as the editor stack, and our trainees are perfectly happy installing a whole bunch of recommended exts.  Not only do we provide a .vscode/extensions.json they can then click "Install all" for, we even streamline it with our custom npm package, vscode-setup, so they can just open a terminal and go npx vscode-setup, and voilà! (this also offers to merge our recommended user settings with theirs).
Live Share sort of assumes your guests work on similar areas / techs as you do, so the chance they'd benefit from your recommended exts is higher. They also know you, either IRL or online, as you probably wouldn't LS with a complete stranger most of the time. So there's higher trust.

For these reasons, I think option 1 is not a good bet. Option 3 is also super-rude. IMHO the best-case scenario would be a banner prompt, much like the local opening of a workspace with recommended exts, where the options would be:

Install all the missing ones (like your option 2)
Sandbox all the missing ones (and sandbox config, perhaps, for the existing ones), like your option 4.
Leave my extensions alone 😉

Best,

@Chuxel and @tdd
I think there's going to be training/teaching use cases where having an install all option would be good but I think with more experienced people it'd potentially be really annoying getting prompts if they don't want to change their workflow/setup. Personally I'd probably really only want to see it as a "what's the host using, maybe there's an extension I'd like to try out" sort of thing but not every session. Maybe the better approach would be to give it a UI element to open a list of the hosts active extensions and let the guest user decide to install from there rather than having a prompt every session?

Hey all,
We were specifically talking about recommended exts present in the
project's .vscode files, so not something you'd see on any session and
any project. Such workspaces recommend exts for their contributors and
sometimes expect these to be there.
On Fri, Feb 2, 2018, at 20:08, Ryan Wilson wrote:

@Chuxel[1] and @tdd[2]
I think there's going to be training/teaching use cases where having
an install all option would be good but I think with more experienced
people it'd potentially be really annoying getting prompts if they
don't want to change their workflow/setup. Personally I'd probably
really only want to see it as a "what's the host using, maybe there's
an extension I'd like to try out" sort of thing but not every
session. Maybe the better approach would be to give it a UI element
to open a list of the hosts active extensions and let the guest user
decide to install from there rather than having a prompt every
session?> — You are receiving this because you were mentioned. Reply to this
email directly, view it on GitHub[3], or mute the thread[4].>

--
Christophe Porteneuve
<EMAIL_ADDRESS>Links:

https://github.com/chuxel
https://github.com/tdd
https://github.com/MicrosoftDocs/live-share/issues/39#issuecomment-362676847
https://github.com/notifications/unsubscribe-auth/AAAOqKwnLvsKrWgCdQ3L4IRXZnnhVpokks5tQ10ggaJpZM4RcsZI

Great suggestions.  Anchoring to extensions.json makes sense since this is where the majority of the functional gaps would be addressed.  You then wouldn't be forced to look someone's "hot dog stand" VS Code theme which would be a problem with "all" extensions.

I'm seeing code snippets appear in intellisense when I connect as a guest to a session, but when I attempt to use those snippets they do not complete in the editor window.  Is this the same problem?

I’m going to close this issue in favor of extensions being able to use our extensibility API to synchronize state amongst participants.


structure reorganisation

Alright, I think this is the structure we agreed on slack so... :shipit:


NIP-05 login bug

Reproduce with<EMAIL_ADDRESS>reason appears to be text/plain response header

Also support fiatjaf.com (auto prepend _@


proposal: cmd/vet: flag benchmarks that don’t use b

I propose that cmd/vet flag any benchmark that doesn’t have any reference to b in its body.

Frequency: Low among experienced gophers, but not an uncommon mistake among new gophers.
Correctness: Benchmarks written without using b do not function as benchmarks. Often this gets caught by a user who is puzzled by the output, but not always, particularly if the benchmark is slow.
Precision: 100%, or very near it. You cannot write a correct benchmark without using b.N, b.Run, b.RunParallel, or somehow passing b or one of its fields to another function to do that for you.

The implementation should be fairly straightforward. Pick out benchmarks. Pick out the identifier (usually b). Look for any uses of that identifier. If none, complain.
cc @robpike @mvdan

I've actually seen this twice in the past few months. Something else I tend to see is slow benchmarks that do a lot of heavy initialization work, but don't take care of resetting the timer.
I'm not sure how far vet can get here, though. Writing a decent benchmark is more than just using b, and I worry that the vet check could give a false sense of security when their benchmark is "fixed".
Wouldn't there be a way for the testing package to catch this? For example, for any benchmark that can be run with N=1 quickly (which should be the vast majority of them), any run with N>100 should at least take 10x more time. If it does not, we are either benchmarking something else (like the initialization code, or nothing at all), or the benchmark doesn't use b properly.
It would be hard to cover very expensive benchmarks by default there, such as those that take one second per iteration, since the default is -benchtime=1s. But I think that's an okay tradeoff.

Writing a decent benchmark is more than just using b, and I worry that the vet check could give a false sense of security when their benchmark is "fixed".

True. I'd hope that being made aware of the problem and referred (by the vet error message) to some docs would set most people on a better path.

Wouldn't there be a way for the testing package to catch this?

There's been scattered discussion between @rsc, @bcmills, @aclements, myself, and maybe others, about having the benchmark framework detect non-linearity. But that's a heavy lift, and you'll get false positives, particularly since you have to base the decision on a single run.
There's also been discussion of having the benchmark framework print all intermediate b.N runs and have benchstat detect non-linearity, bi-model distributions, and other distribution problems. But then you have to know about benchstat, at which point, you probably don't aren't the target audience for this vet check.
I definitely wish we could somehow build benchstat into package testing. But the "before"/"after" problem is a thorny one. (See my failed machinations in https://github.com/golang/go/issues/10930.)

I'm not in favor of this. It seems a minor problem and opening vet up to analyzing people's tests leads us down a long and slippery path.

@josharian your thoughts are very interesting - you've clearly been thinking about this for a while :)
I agree that, in general, writing good benchmarks in Go requires reading quite a lot of content and using multiple tools together carefully. Even though it's a difficult task, I still think that the only way to truly solve that problem is to make the tooling better. Be it by making the testing package smarter, or bundling benchstat into testing.
To me, fuzzing and benchmarking are kind of similar, in the sense that they are very helpful and necessary for many developers, but they are difficult to use correctly simply because not enough hours have gone into properly including them as part of go test.
Adding small checks to vet could possibly help in the near term, but we can only catch the most trivial mistakes there. So while this could meet the bar for inclusion in vet, I don't think it's the direction we want to be heading in.

It seems a minor problem and opening vet up to analyzing people's tests leads us down a long and slippery path.

A Benchmark function that doesn't depend on b.N at all is guaranteed not to produce valid results.
In contrast, a Test function that ignores its *testing.T could potentially be valid: it might indicate a tests whose failure mode is (say) an uncaught panic that terminates the test program with a non-zero status.
So I don't think there is a slippery slope here: the argument for Benchmark functions already does not apply to Test functions.

We do vet API usage, checking things like printf format strings.
And we vet test code, checking things like t.Fatalf's format strings.
In general it seems OK to vet misuse of test APIs, same as other APIs,
provided the checks meet the same bar.
That said, I am not convinced this specific check needs to be in vet.
Checking "did it use b at all" would catch some but not all problems
and would be easily defeated. It doesn't seem specific enough to me.
And I think the mistake can be detected more precisely at runtime instead.
As Josh noted, we've talked before about having a linearity check.
That can be a bit sensitive to timings but might be worth doing anyway.
Whether to do that is maybe borderline pending experiments
(that none of us have ever made time for).
But a highly accurate linearity check is not needed to detect
"this line (plotting t against b.N) has slope 0 instead of slope 1".
I sent CL 230978.
With that change, a vet check doesn't seem worthwhile.
(The only benefit would be that the vet check would catch
the bug in a benchmark that is never actually run, but if you
never run your benchmark, do you really care if it has a bug?)

Austin pointed out various problems with CL 230978 so I ended up writing a vet check after all. Results of running that vet check on the public Go ecosystem are at https://gist.github.com/rsc/4ba2eacb2d1a00339f4539e7b1bee576. There is one false positive I need to correct involving assigning b to a global that is then read by another function called via yet another function, but other than that, the check seems quite accurate and useful.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

@ChrisHines I agree that those look like false positives and should not be flagged. The final implementation should probably be conservative when a callee may use b.


⚠ Arturia Pendragon Website has degraded performance

In 7a04d92, Arturia Pendragon Website (https://www.ovh.com/fr/) experienced degraded performance:

HTTP code: 200
Response time: 5615 ms

Resolved: Arturia Pendragon Website performance has improved in 668d127.


Proposal to fix the script errors for customizing Tomcat

https://learn.microsoft.com/en-us/azure/app-service/configure-language-java?pivots=platform-windows#automate-creating-custom-tomcat-on-app-start
Sample startup.ps1 script doesn't work due to invalid option and the lack of folder creation. This PR will fix these issues.

@cephalin
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Is it configuration issue only? Because I have tried seed method and it works fine.

Not a configuration issue,I merged the latest branch, Use commands
dotnet ef migrations add InitAdminIdentityDb -c AdminIdentityDbContext -o Data/Migrations/AdminIdentityDb
dotnet ef migrations add InitIdentityServerConfigurationDb -c IdentityServerConfigurationDbContext -o Data/Migrations/IdentityServerConfigurationDb
dotnet ef migrations add InitIdentityServerPersistedGrantDb -c IdentityServerPersistedGrantDbContext -o Data/Migrations/IdentityServerPersistedGrantDb
dotnet ef migrations add InitAdminLogDb -c AdminLogDbContext -o Data/Migrations/AdminLogDb

Ok, I will check it. Thank you

I have tried it and works. Can you check please - @xmichaelx ? :)

I tried again, download the zip file of the dev branch, also throw error

From the command line I see that you've added the migrations but it seems that you haven't updated the database with them.
dotnet ef database update -c AdminIdentityDbContext
dotnet ef database update -c AdminLogDbContext
dotnet ef database update -c IdentityServerConfigurationDbContext
dotnet ef database update -c IdentityServerPersistedGrantDbContext

Additionally it looks like you're failing during login so please check if your connection strings are correct.

Previous versions can use /seed to create a database, using /seed to create a database automatically in docker.

I did not modify any files after downloading the zip file.

You are right - but we had only one DbContext, now it is necessary to do this manually - like Michal mentioned above.

This will cause some difficulties for newcomers to use Docker in the future.

This basically boils down to either issuing those commands (which you can automate in Docker build) or adding to code for each of 4 builtin context:
context.Database.Migrate();
From my POV it's a matter of preference and I would like the community to voice their preferences.

Cannot be manipulated in Dockerfile, Dockerfile is only used to build images, and building image is independent of database operations.

@Seaear - Please give us your preference about this issue. We'll modify it especially for better docker support.

Use the command in the docker-compose.yml file to add the /seed parameter at runtime to automatically create the database

@Seaear could check automatic-database-creation-during-seed branch? I've added database creation during seeding process.

@xmichaelx Missing "" in path when using localdb

Server=(localdb)\\mssqllocaldb;Database=IdentityServer4Admin;Trusted_Connection=True;MultipleActiveResultSets=true where here is the missing character?

Great, use SQL Server it's working, localdb problem, I can try to find

@xmichaelx Localdb problem, I can't find it, it's not affected in production environment or Docker, it doesn't matter for the time being.

@Seaear - OK, I'll merge the changes from @xmichaelx .

Done on dev branch @Seaear - you can test it. :)
Thanks!

@skoruba Great, I tested it is working fine.
Thanks

Thank you @xmichaelx as well.
May I close it?

Thank you @xmichaelx as well.
May I close it?


fix $_SERVER['LARAVEL_START_TIME']and$_SERVER['LARAVEL_START_MEMORY']

Q
A

Bug fix?
No

New feature?
Yes

Description

Fixed $_SERVER['LARAVEL_START_TIME'] and $_SERVER['LARAVEL_START_MEMORY'] for roadrunner worker.

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made changes in CHANGELOG.md file

I will temporary merge with another branch and make some changes them

ok. thx.


Writing operators misses a space

?- writeq(1 is X),nl.
1 is_112
X = _112.

Expected: 1 is _112
See 7.10.5 h 1:

... In all cases, a space is output to separate an operator from its argument(s) if any ambiguity could otherwise arise.

Similarly variable_names#73.

Thank you very much for reporting! Fixed in v1.2.1.
$ go install github.com/ichiban/prolog/cmd/1pl@latest
go: downloading github.com/ichiban/prolog v1.2.1
$ $(go env GOPATH)/bin/1pl
Top level for ichiban/prolog v1.2.1
This is for testing purposes only!
See https://github.com/ichiban/prolog for more details.
Type Ctrl-C or 'halt.' to exit.
?- writeq(1 is X),nl.
1 is _79
X = _79.
?- S=(1 is T),N='X',write_term(S,[quoted(true), variable_names([N=T])]),nl.
1 is X
N = 'X',
S = 1 is _90,
T = _90.
?-

Back to perfect scores!


2.0.4 Release -> U11 Certification

Please note, I've updated the project to support SPM as the ArcGIS and ArcGISToolkit dependency manager. As such, the project is automagically grabbing the 100.10 public release. I've tested the app using the U11 daily build installed manually, now the project is configured to pull the 100.11 dependency upon release.

I've tested the app using 3114 and I think we're good to go here. I'll keep the PR in draft until Runtime and Toolkit 100.11 are released when I'll need to update the toolkit dependency and use Xcode to resolve the swift packages for U11 releases.

Alright @mhdostal I think we're ready for final PR review here. Thanks!


"Open with vscode" in the explorer context menu always select default profile

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.76.0 (system setup)
OS Version: Windows_NT x64 10.0.22621

Steps to Reproduce:

Select a user defined profile and close vscode.
Right click some file in the explorer and select "open with vscode", the profile is not the selected one in the last step but the default profile.

This is expected because profile associations to empty windows are not persisted and an empty window takes the profile from the active window, or default profile if there is no active window.

The described behavior is how it is expected to work. If you disagree, please explain what is expected and what is not in more detail. See also our issue reporting guidelines.
Happy Coding!


Add optional listdir method support

Coverage increased (+0.47%) to 97.45% when pulling f8a9a89caa65d0b5a6e1548518b095285553ca3c on subuk:master into be7c8ca255e6c67ceed16a4146d752003334e0c5 on marazmiki:master.

Coverage increased (+0.47%) to 97.45% when pulling f8a9a89caa65d0b5a6e1548518b095285553ca3c on subuk:master into be7c8ca255e6c67ceed16a4146d752003334e0c5 on marazmiki:master.

Coverage decreased (-1.28%) to 95.7% when pulling 0d430a783cee565ab7875a128da2681d8c81291e on subuk:master into be7c8ca255e6c67ceed16a4146d752003334e0c5 on marazmiki:master.

Coverage increased (+0.51%) to 97.49% when pulling 0d430a783cee565ab7875a128da2681d8c81291e on subuk:master into be7c8ca255e6c67ceed16a4146d752003334e0c5 on marazmiki:master.

It's awesome. Thank you very much for your contributions!


Canary route

If Bouncer goes down outside normal working hours, the people on call should be notified. The app and the check should also be documented in the opsmanual so that they are able to take appropriate action if necessary.
This PR adds a canary route at /__canary__ which checks to see if it can select from 3 of the tables. If any of these 3 checks throw an error or can't connect, we return a 503. Ticket.
Original work done by @jennyd. Props to her for starting and helping me with this. ✊✊✊✊✊

Nice small, specific commits :small_blue_diamond: :small_orange_diamond: :black_small_square:

We should test that the canary route returns 200 when everything is working, as well as testing the error conditions - we're relying on that to not call people unintentionally :)

Fixed the PR based on your feedback. Also added a test for the 200 case. I have some questions about the caching so let's talk then.

:soccer: :maple_leaf: :bird: :candy: :camel: :cake: :minidisc:


ImportError:No module named pycocotools.coco

When I am in your STEP test and train,the terminal show me the error.
from pycocotools.coco import COCO
I cannot find the folder or document named pycocotools,can you give me some suggestions?Thank you so much for your help,hope to hear from you.

You need to create a link in data folder, following what you do when you set up pay-fast-rcnn.

For training with pascal_voc why do I get this issue?


Error

When i npm start I get this..

Do you able to run "android" and "ant" command?

Windows is not supported. It is written in the docs.


Atomix, AWS 2 Kinesis, AWS 2 Lambda, Azure Storage Blob and Azure Storage Queue JVM support

JVM jobs passed


Update docs links to point to v5 release

Details
Updates docs links to point at (upcoming) v5.0 release so the correct links get baked into the tag
Pivotal Story (if relevant)
Story URL: https://www.pivotaltracker.com/story/show/187772613
Related PRs
N/A
Submitter PR Checks
(tick as appropriate)

[X] PR completes task/fixes bug
[ ] API version has been incremented if necessary
[ ] ADR status has been updated, and ADR implementation has been recorded
[ ] Documentation updated (README, etc.)
[ ] PR added to Pivotal story (if relevant)
[ ] Follow-up stories added to Pivotal

Reviewer PR Checks
(tick as appropriate)

[ ] PR completes task/fixes bug
[ ] Design makes sense, and fits with our current code base
[ ] Code is easy to follow
[ ] PR size is sensible
[ ] Commit history is sensible and tidy

Info on PRs
The checks above are guidelines. They don't all have to be ticked, but they should all have been considered.

LGTM. I assume the version in the spec itself will get updated in a separate PR?

It's part of the release process


Compile error - missing import

Awesome work, thank you!
Just the compiling broken since last week, related to a new class.
Cheers,
bithug

Thanks!


Game Stage Refinement Needed

Currently the script detects controller level of our main room, and uses that to assign a game stage from Charmander, to Charmeleon, to Charizard. We should localize the game stage to be per room, so that our main room can operate at a higher stage than a newly claimed room.

Fixed in spawn logic refinement Issue - separated room stage check to be during the for each room loop rather than for only the first room hardcoded.

Fixed in spawn logic refinement Issue - separated room stage check to be during the for each room loop rather than for only the first room hardcoded.


Fix to work with yq from snap

This way it works with yq from snap.
On the other hand we now have a dependency on sponge. We can resolve it by moving around tmp files if we prefer

Hm, with my non-snap yq I'm able to run yq eval -i '.asdf = "bar"' /dev/stdin < asdf.yaml. Can you check whether that works for you too?
That should bypass the sandbox without needing sponge, since your shell (or make) would be opening the file and just passing the fd to yq.

Fancy! Works fine


Systemd for create_bridges

Similar to #525 done by @SahilTikale , we need to make the call to create_bridges into a separate systemd script, as it only needs to run once per boot.
This will also entail updating INSTALL.rst.
Once this is done, users will no longer need to modify /etc/rc.local!!!

Also, we need to use 'IP' commands to create vlan and stop using vconfig/net-tools because they are deprecated.
--Sahil
-----Original Message-----
From: "henn"<EMAIL_ADDRESS>Sent: 3/16/2016 9:47
To: "CCI-MOC/haas"<EMAIL_ADDRESS>Cc: "Sahil Tikale"<EMAIL_ADDRESS>Subject: [haas] Systemd for create_bridges (#555)
Similar to #525 done by @SahilTikale , we need to make the call to create_bridges into a separate systemd script, as it only needs to run once per boot.
This will also entail updating INSTALL.rst.
Once this is done, users will no longer need to modify /etc/rc.local!!!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

So I think #552 is no longer needed, right? Shall I close the PR?

This issue becomes redundant once we have the openVPN feature.
Should we close it then @henn ?

Sure

@shwsun We can close this issue. It's a duplicate of #671 which has been fixed by #674


what is the sent_limit?

Hi
I would like to know what is the sent_limit? It may be hard to understand this parameters due to that there is no annotation for parameters.
Kind regards
Tang

It's the maximum number of sentences we process (and therefore use in the model) for each paragraph.


Add link for component availability

This link is a community-supported history of component availability in nightly releases.
(Generated automatically from each nightly release)
I always forget the link, so probably having it up visible on rustup.rs is a good idea.
wdyt?

Anybody maintaining this repo at all?


Syncing with upstream psf-bch-wallet

:tada: This PR is included in version 1.2.0 :tada:
Your semantic-release bot :package::rocket:


WIP:  Python modernization

What was wrong?
Need to remove Python 2 code since we are dropping support for it in the next release.  Also - take advantage of native Python 3 methods where it makes sense.
Reference issue #339
How was it fixed?

Remove support for Python2
Add tests for Python 3.6
Add docker based test environment

Cute Animal Picture
   __
 .^o ~\
Y /'~) }      _____
l/  / /    ,-~     ~~--.,_
   ( (    /  ~-._         ^.
    \ "--'--.    "-._       \
     "-.________     ~--.,__ ^.
               "~r-.,___.-'-. ^.
                YI    \      ~-.\
                ||     \        `\
                ||     //
                ||    //
                ()   //
                ||  //     -Row
                || ( c
   ___._ __  ___I|__`--__._ __  _
 "~     ~  "~   ::  ~~"    ~  ~~
                ::
                .:
                 .

Tests are passing when I run pytest -n 4 -f -k "not goethereum".  I am having issues getting go-etheruem installed in my docker container, but I don't think any of my changes should affect this part of the test suite.
============= 852 passed, 29 skipped, 2 warnings in 68.35 seconds ==============

OK, I think this one is ready for review.  The only travis test that is failing is the flake8 part for some reason, not sure what is going on there.

@jasonrhaas If you click into the flake8 test, it shows you log output, which includes the flake8 failure info. (flake8 will also run quickly in your local dev environment)

All review comments have been implemented.  I added one other commit 6edd103 that removes __future__ imports across the project.  Tests are passing 💯

@carver @pipermerriam this one should be ready to merge.

Removed tests test Python 2 code, but I don't see Python 2 code removed from main codebase. Without tests it will be harder to locate it.

@techtonik I understand the sentiment, and it would have been reasonable to make those changes in this PR, but it's also okay to put it in a separate one. It won't be too hard to find, because they use sys.version_info.

I don't understand why dropped support for python2, actually a lot dev favour python2 over python3, including me.

@zhuyuecai if you can list reasons why you favour Python 2, then there is a chance to make Python great again in Python 4 and add support for that.

@zhuyuecai I'd be interested in hearing your reasons for preferring python2 as well.


Data can be stale after the first load

A user of this plugin recently asked if the plugin caches because the data did not refresh on a page reload during development.
This issue occurs as a consequence of how webpack works and how it caches resources during development.
During development, webpack will cache a resource it has already loaded before and will only re-compile that source if it sees a change in the contents of the file loaded. In the case of prevals, the file doesn’t actually change but the data does.
This is known issue on my side and there are ways to fix it but it involves having the data source send some sort of event to tell webpack to re-compile the preval’ed file.
For now, workarounds are:

touch the preval file (e.g. hit save). the update to the file is enough to tell webpack to refresh the preval.
restart the dev server.

A better solution would most likely involve creating a mechanism for data sources to tell webpack to refresh the file (or maybe just polling). I'm open to any suggestions!

@ricokahler as an update to this, restarting the dev server does not fix the cacheing problem. I've had to either touch the file, or delete the .next directory.
Also, I host my site on Netlify, where I have to manually select Clear cache and deploy site to have the changes reflected -- since they aren't when a build is triggered via git. This causes a bit of a problem when I'm trying to use build hooks, where clearing the cache isn't an option.


[18.09] Initialize nss libraries in Glibc so that the dynamic libraries are l…

bring in https://github.com/moby/moby/pull/39612
Although current releases from 18.09 branch are not affected because we vendor custom tar package this was recently changed in this branch and requires patching for future releases.
@thaJeztah @tiborvass

I think this was related to a jenkins configuration
13:54:12 /go/src/github.com/docker/docker/hack/ci/janky: line 7: GIT_SHA1: unbound variable
13:54:13 Build step 'Execute shell' marked build as failure

and we need to remove codecov in this branch as well

I'll ignore that failure: experimental is green and runs the same tests


extractToken should support oauth and/or client credentials

With this change two kind of tokens are supported: Oauth and basic/client-credentials.
E.g. in case the "Authorization" header is not provided, and basic or client-credentials is supported it tries to return the password or client-credentials token.

@mwdb
can you please do a review?

basic and JWT always use the authorization header.
Either:

Authorization: basic base64(username:password)
Authorization: Bearer

Also the change is in case a null/emty string value is returned, the checks continue


changing application results in multiple appConfig

Describe the bug

With the latest master branch.
Each time modify an existing Application will trigger to create a new appConfig. And multiple appConfig exist simultaneously.
To Reproduce
Apply below app.
apiVersion: core.oam.dev/v1alpha2
kind: Application
metadata:
  name: website
spec:
  components:
    - name: backend
      type: webservice
      settings:
        image: busybox
        cmd:
          - sleep
          - '1000'
      traits:
        - name: scaler
          properties:
            replicas: 2

Make any modifications and apply again. Repeat. Check the appConfig
kubectl get appconfig

NAME         AGE
website-v1   7m50s
website-v2   7m21s
website-v3   5m32s

ping @wonderflow @ryanzhang-oss

fixed by
https://github.com/oam-dev/kubevela/pull/1166


Using in multi-threaded multi-user app?

How would you suggest using your mail library in multi-threaded application to retrieve and send mail using different mailboxes in different threads? The settings singleton is not thread safe. If multi-threaded use is currently not possible, how would you suggest solving this in an acceptable pull request?

@wrzasa you could use the retriever/delivery classes directly instead of the static configuration.
So, roughly Mail::POP3.new({...}).find

Thank you for your answer! That leaves two questions. First, is the retriever/delivery part of the API subject to future changes? It's rather an internal API. And second, don't you consider changes to the main API that would make the gem thread-safe?

These classes are part of this library's stable public API.
Default config is global, not thread-local, so thread safety is not a consideration here.

Thanks a lot for your answers!


Add client operation for partial listing

What changes are proposed in this pull request?
Adds client API for partial listing of directories which returns a batch of listings from an offset based on the options inout.
Does this PR introduce any user facing changes?
Adds the API to BaseFileSystem
public ListStatusPartialResult listStatusPartial(AlluxioURI path, final ListStatusPartialPOptions options)

As suggested by @rongrong I modified how the startAfter parameter works.
If startAfter starts with "/", then the it is expected to be a full path, otherwise it is expected to be a partial path that will be appended to the listing path.
For example say we are listing "/dir"
Then if startAfter is "/dir/f1" then the listing will start after "/dir/f1"
If startAfter is "f1" then the listing will start after "/dir/f1".
If startAfter is "/f1" then an error will be thrown saying the parameter is invalid.

alluxio-bot, merge this please


get bright shows 100 if the bulb is off

The get bright command outputs 100 if the bulb is off.

That's how the yeelight API works, this is simply a wrapper around it. Their rationale is probably that you may want to check the brightness of the light without having to turning it on.
You should check power to check if the light is on or off. You can do it in a single call with: get power bright and the results will be in separate lines like so:
$ yeelight <MY_BULB> get power bright
off
40


TvSettings: Add Support to DolbyVision Game Mode

Check and update L1 and L2 cases as required for the specification update
issue : https://github.com/rdkcentral/rdkv-halif-tvsettings/issues/27

merged and tag 1.3.1


New release tag

Hello,
Would it be possible for a new release tag to be created that includes changes for cleaning up the use of copy? Would save me referring to master branch in my package.json :)
Thanks!

+1 on this! Test runner output has been pretty noisy due to this.

Release please.

Kindly ping to @poteto or @mike-north :)

@kellyselden and @poteto are the only ones who are listed as npm collaborators. If either of them gives me the appropriate permissions, I'm happy to cut a new release

@kellyselden? @poteto?

Should we e-mail npm and see if someone else can get access?


[Feature] Support log profile to file

Why I'm doing:
The profile should be downloaded to ensure that the profile information will not be lost.
Using the API to obtain data cannot avoid loss.
What I'm doing:
Support log profile to file
Fixes #issue
What type of PR is this:

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@Mergifyio backport branch-3.3 branch-3.2 branch-3.1


Shortening cropped image file names

The current autocrop script produces filenames like the following for a cropped version of a source book image.
crop--ocean-projects-hum160002p-shared-books-test_autocrop-anon_R300_hrc_8_paradiseregaind1680-anon_R300_hrc_8_paradiseregaind1680-0052.tif
Broken down in pieces this is:

crop-
The full path to the book directory with its directory separator character ('/') replaced with dashes
The original name of the enumerated book image tif file

This renaming originates in page_parser.py in the Page.__init__ method:
        if name is not None:
            self.name = name
        else:
            if img_file is not None:
                self.name = (img_file[2:] if img_file[0:2] == './' else img_file).replace('/','-')
            else:
                name='default.jpg'

and further down in Page.write_side_images:
        if self.num_sides == 1:
            cv2.imwrite(path+'crop-'+self.name,self.side1)

        if self.num_sides == 2:
            cv2.imwrite(path+'crop1-'+self.name,self.side1)
            cv2.imwrite(path+'crop2-'+self.name,self.side2)

Something shorter and more descriptive would be: crop-<cropping-type>-<enumerated-book-image-filename>
For example: crop-threshold_by_inside-anon_R300_hrc_8_paradiseregaind1680-0052.tif
I believe the idea behind the original filenaming method was to keep track of the actual source images the cropping was taking place over, but as @nvog has noted he is renaming them back to the original name in run_workflow0.sh:
python3 $SHARED_DIR/books/code/auto_crop.py --threshold_by_inside "*.tif"
# move precropped, raw originals to the originals_precrop directory
mkdir originals_precrop
mv *.tif originals_precrop/
pushd originals_precrop
# rename the cropped images to their original filenames
# in top, book-level dir to prepare for workflow1
for f in crop-*; do
    mv $f ../${f#*-}
done
popd

(A brief explanation of the bash script - particularly the ${f#*-} part: All files in originals_precrop beginning with crop- are renamed with a filename that removes all strings matching any set of characters followed by a '-'. See Substring Removal in Parameter Expansion with ${} in Bash: A Comprehensive Guide.)
NOTE: page_parser.py and run_workflow0.sh are currently in soon to be reorganized and 'print-and-probability' repository. This is the books/code folder that Nikolai had been updating on GitHub in the icebergnlp account.

In discussion with @nvog, we decided to add a --test flag to autocrop.py and output cropped images into the separate folders in <source_dir>/results with nothing prepended since he is currently removing prepended information from the original image name in run_workflow0.sh


++ and += not parsed

e.g.

fn foo() {
    var b = 1;
    b+=1;
    b++;
}

I'll get these added.

I pushed an update with the recent WGSL changes: ++, --, +=, -=, *=, /=, |=, &=, ^=, >>=, <<=, const, and static_assert.


added name to an action

We decided to take a command approach in our application, where we wanted to send commands to our server, where the command namespace would be the store and the command the action, i.e. UserStore.createUser. To achieve this we had to know the names of the actions, and this is why we made a change where we inject the name of the action into the created action as a property when creating multiple actions at once.
Atm we used __name but if you prefer a different name then we can change that as well

There is an actionName in current versions of reflux that is used by the multiple actions creator. Creating an action with a string sent to createAction will use that string as actionName See #200. Closing this PR.


fix find_program ~ don't report errors for failed simple checks

When searching for tools, errors were emitted from the "check" function when a tool was missing from a searched path. That is expected behavior and I would say should have only an informational note or nothing at all.
This PR moves any check with a non-function opt.check together with the check done with a missing opt.check. That check never emits an error, so this change suppresses the, IMO unnecessary, error noted above.

This is unnecessary, I need to add -v argument to run xmake -v for knowing why the detection tool failed.
And if you do not add the -v parameter, this error message is already suppressed.

Ok, I understand that you might want to see information of the detection tool failed. But to me an "error" denotes something that is configured incorrectly and needs to be fixed. It's concerning when I see an "error" or "warning" during builds, even if only during a verbose (-v) build. And I think lots of other devs would see the "error" and spend time searching for a repair as well.
From my reading of the source code, the check function can be run several times while searching through paths for a tool and will, during normal operation, "fail" for earlier paths which don't contain the tool until finding it in a later path (driven by the find_program() function). I initially thought about adding a "warning" and/or "info" function, eg...
-- the verbose warning function
function utils.vwarning(format, ...)

    -- enable verbose?
    if option.get("verbose") and format ~= nil then
        utils.warning(format, ...)
    end
end

-- the verbose information function
function utils.vinfo(format, ...)

    -- enable verbose?
    if option.get("verbose") and format ~= nil then
        -- ... bright blue info message ...
    end
end

... and then changing check to issue a vwarning or vinfo instead of verror. But I thought it better to move the simple check up into the same context. when I saw that the initial exec can fail without issuing any error:
https://github.com/tboox/xmake/blob/d130b8049af3250f3504580d146e3c54ed052a5f/xmake/core/sandbox/modules/import/lib/detect/find_program.lua#L57-L59
It seems to me that given what you want (and I agree that knowing which paths failed for a tool search is useful), check should issue an informational note if it doesn't find the tool on a specific path (including in the above notes section of code), and maybe find_program() or some higher level function should issue an error if the tool is truly, finally, not found in all searched locations.

Ok, I understand and agree with you, as the error message is really not friendly during a verbose (-v) build. But I haven’t thought about how to improve the display detection failure information. Maybe utils.vinfo, add xmake -v --more or xmake -V arguments  to get more verbose info only for developers, and etc.
I need to take some time to think about it.

I add -D and --diagnosis arguments to get more diagnosis information only for developers.
d3d05b0c620dbfc318d4c693bc2fb7b556d51d3c
$ xmake -h
  -v, --verbose      Print lots of verbose information for users.
  -D, --diagnosis    Print lots of diagnosis information (backtrace, check info ..) only for developers.
                     And we can append -v to get more whole information.
                           e.g. $ xmake -v -D

And I have improved the check errors output:
    if not ok and option.get("diagnosis") then
        utils.cprint("${yellow}checkinfo: ${clear dim}" .. errors)
    end


HEAD request ignores 'data' parameter

When I call:
qwest.map('GET', 'example.com', {canApprove: true})
it successfully requests 'example.com?canApprove=true'
But, when I call:
qwest.map('HEAD', 'example.com', {canApprove: true})
it requests 'example.com' with no query string
It looks like 'data' parameter is ignore for HEAD request.

It's not a bug, it comes from the spec of the HTTP protocol : https://stackoverflow.com/questions/4529066/post-data-with-head-request

Hi,
Well, I don't want to post data with HEAD request, I want my 'data' object
to be converted to query strings.
This is what happen for a GET request, but not for a HEAD request.
According to spec, HEAD request should accept query string (same as GET
request)
Thanks,
Mederic
On Fri, Aug 11, 2017 at 1:59 AM, Aurélien Delogu<EMAIL_ADDRESS>wrote:

It's not a bug, it comes from the spec of the HTTP protocol :
https://stackoverflow.com/questions/4529066/post-data-with-head-request
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pyrsmk/qwest/issues/142#issuecomment-321762626, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACa9SA6EPEZ5-0EdlrkR-f1sVlTgqrPyks5sXBgCgaJpZM4O0CuG
.

Sorry, I misunderstood your issue. Anyway, this feature request is tricky. This is not a bug, but a design issue. I think I should add a new dataType option, like queryString. I will work on this by sunday/monday.

Great!
Thanks
On Sat, Aug 12, 2017 at 6:49 AM, Aurélien Delogu<EMAIL_ADDRESS>wrote:

Sorry, I misunderstood your issue. Anyway, this feature request is tricky.
This is not a bug, but a design issue. I think I should add a new dataType
option, like queryString. I will work on this by sunday/monday.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pyrsmk/qwest/issues/142#issuecomment-321982302, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACa9SIhi6ZSt_EG12mgpH2LGDJNKvcucks5sXa1SgaJpZM4O0CuG
.


Publish chart to ghcr

The idea would be to publish to both sources for a few releases and promote the older approach as deprecated.
Advantages for ghcr:

Better visibility and "closeness" to the repo than using gh-pages
Simpler and cleaner release code
OCI registries are the "new" standard for helm charts

Todo:

[x] gh_token secret must have packages scope
[ ] should be tested locally by someone with access to gh_token

RFC @xoxys @anbraten

Yes, I need to check whether a token from me works or if I need to ask for one.

@anbraten @6543
If you're ok with this, the following needs to be done:

Add a GH token with "packages" scopes
Enable "packages" for this repo

there is no package scope :/

It should be Contents, packages is enabled for the org. We just have to set the chart to public after publishing the first time.

why is github's registry used when we use quay.io and docker-hub for other images?

It was proposed earlier in a discussion to drop quay
DH has been denied once already and nobody looks there for charts (arguably)
ghcr makes the packages more visible and accessible as they are stored alongside the source

as we have a fallback via our website ... I dont reject. I still think we can improve on the delivery side of artifacts ...
... and I did not get the discussion to drop quay where is it!?!

https://github.com/woodpecker-ci/woodpecker/discussions/4110

I'd call the gh-pages approach "legacy" meanwhile. It was not meant for it in the first place and isn't easily discoverable.
We should focus on OCI registries (yes GH also isn't the best solution here but better than the current approach) and also link these on artifacthub.

woodpecker-ci/woodpecker#4110

well another reason i personally can not work with the discussions  ... I dont get notifyed nor can you proper search in them ...

... shuld work

Please test locally if not yet done - you can also delete the asset afterwards again. Just to make sure we don't have another failed release here.


Filter open positions by tags

#1190 must be done first
Use TagSelect component to filter open positions based on tags. Get positions by tags from backend.
Implement this on OrganizationRecruitmentPage.

sames as #686


ajax example broken

Trying to get started with Ajax and Halogen but it looks like the ajax example latest commit in master won't compile.

If you're trying to use the version of Halogen in master, please refer to this branch: https://github.com/garyb/purescript-halogen/tree/new-guide
Otherwise, please use the tag for the latest release https://github.com/slamdata/purescript-halogen/tree/v0.12.0 :)

Ah, I didn't know about tags in git/github and didn't realize that master wasn't the latest stable version.


Provide arithmetic operations on IPv4 and IPv6

Hi, this functionality was ready since a long time but unfortunately never made the merge. Would appreciate a lot to get it merged into the new gem. Thanks a lot for your time and effort!

Thanks for the pull request. I managed to merge it with master and will put a new gem version out soon.


Fix race condition where pending Ack IDs can be modified by another thread.

Closes #5913

I kicked off a rebuild, hoping to work around the logging systest failure.

apologies if I am out of line, but will this make it into the 0.38.0 release? (#5962)

Oh good catch, @jordanhurwitz! I thought this was merged.


fix: minor wording adjustment

for your components which are wrapped inside the context

but also from the components providing context (which is what the example shows).


added skipped count in json output format

Checklist

[ ] make test-all (UNIX) passes. CI will also test this
[ ] unit and/or integration tests are included (if applicable)
[ ] documentation is changed or added (if applicable)

Description of change

@aelsabbahy Please share your review comments, if changes made won't impact the overall or any specific goss functionalities, let me know in case of any otherwise.
Thanks!!

Looks good to me, I think I had a branch here that was essentially the same but never pushed it.

Thanks for looking into @ripienaar, it will be really helpful, if these changes can be merged and release a new version as GA.
Thanks!!

released: https://github.com/goss-org/goss/releases/tag/v0.3.23


Add Emscripten barebone build implementation

This adds the first version of Emscripten-specific Makefile definitions,
allowing to compile the C/C++ code under WebAssembly. This new
functionality can be enabled by setting the TOOLCHAIN environment
variable to "emscripten" (the variable still defaults to "pnacl" to use
Google Native Client, as before).
Note that this is only a barebone implementation, since it doesn't
support several important features, namely:

Auto-detecting related .h files and triggering recompilation of .cc
files on their changes;
Installation of headers into shared location;
Building unit tests;
etc.

These will be addressed in follow-ups, as tracked by #177.
Another note - Right now only very few files in the project can be
compiled using this new Emscripten toolchain, since many files are using
Native Client specific definitions (like the pp::Var class). Their
migration is a separate ongoing effort.

Thanks!


[Bug] [clickhouse] clickhouse无法正常使用

Search before asking

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

Operating system information
Linux
Python version information
3.10
DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[X] Chat DB
[ ] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[ ] Installation From Source

[X] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[X] Other

Device information
Distro : Ubuntu 22.04.3 LTS aarch64
Processor : Neoverse-N1
编译模式：bash docker/base/build_image.sh -b ubuntu:22.04 -i https://pypi.tuna.tsinghua.edu.cn/simple --language zh --install-mode openai
Models information
LLM_MODEL=proxyllm
EMBEDDING_MODEL=text2vec
What happened
clickhouse无法链接
本机curl能连接数据库但是db-gpt里面无法连接

What you expected to happen
连接clickhouse
How to reproduce
clickhouse无法链接
本机curl能连接数据库但是db-gpt里面无法连接
Additional context
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

DB-GPT  server和clickhouse都是部署在本地吗？

DB-GPT server和clickhouse都是部署在本地吗？

对，都用docker compose部署在同一个linux服务器中

try replace <IP_ADDRESS> into your ip address?

try replace <IP_ADDRESS> into your ip address?

我尝试了<IP_ADDRESS>和本机ip，8123和9000端口，均无法连接。
使用docker/base/build_image.sh编译的镜像缺少clickhouse_connect，连接时出现缺少依赖的错误。我在dockerfile中加入了RUN pip3 install -i $PIP_INDEX_URL clickhouse_connect后，依然无法连接，出现图中的错误。

try replace <IP_ADDRESS> into your ip address?

ssh连接服务器，用curl -u 账号:密码 'http://<IP_ADDRESS>:8123/?query=SELECT+1&database=数据库' 可以正常连接clickhouse

have your clickhouse set password or just install your from

have your clickhouse set password or just install your from

我修改了初始数据库用户名和密码，docker compose配置如下：
version: '3.8'
services:
  clickhouse:
    image: 'clickhouse/clickhouse-server:${CHVER:-latest}'
    container_name: clickhouse
    hostname: clickhouse
    environment:
      - CLICKHOUSE_DB=DB_GPT
      - CLICKHOUSE_USER=修改的用户名
      - CLICKHOUSE_PASSWORD=修改的密码
      - CLICKHOUSE_DEFAULT_ACCESS_MANAGEMENT=1
    volumes:
      - /123/clickhouse/DB-GPT:/var/lib/clickhouse
      - /123/clickhouse/log:/var/log/clickhouse-server
      - /etc/localtime:/etc/localtime
    ports:
      - '<IP_ADDRESS>:8123:8123'
      - '<IP_ADDRESS>:9000:9000'
    depends_on:
      - clickhouse-keeper
  clickhouse-keeper:
    image: 'clickhouse/clickhouse-keeper:${CHKVER:-latest-alpine}'
    container_name: clickhouse-keeper
    hostname: clickhouse-keeper
    volumes:
      - ./fs/volumes/clickhouse-keeper/etc/clickhouse-keeper/keeper_config.xml:/etc/clickhouse-keeper/keeper_config.xml
    ports:
      - '<IP_ADDRESS>:9181:9181'

you can add some log info in conn_clickhouse.py, print some connect info and client info and see what happened.


v2.0.0 to v2.0.1

Fix tabindex of CheckboxSVGMap
Update changelog

Codecov Report

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

@@          Coverage Diff          @@
##           master    #29   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           3      3
  Lines          76     76
  Branches       17     17
=====================================
  Hits           76     76

Impacted Files
Coverage Δ

src/checkbox-svg-map.jsx
100% <ø> (ø)
:arrow_up:

src/radio-svg-map.jsx
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Remove test dependency

The pitch_detector_dart package published on pub.dev has:
dependencies:
  test: ^1.17.10

dev_dependencies:

However the test dependency need to be moved into the dev_dependencies section.

Sorry it took so long. Done. Version 0.0.4
Thanks for the heads up.


[TECH] Use a ManyToMany relation between Answer and Choice

Now that an answer can have multiple choices, the Doctrine relation between Answer and Choice should be ManyToMany instead of duplicating answers.

done in 4e75576dcfdb215b5d1811d3909298626d605f1d


Import sqlalchemy-enum-tables

Description
Importing this repo because we need to make changes to it.  I am working to clear this with the open source team @ CZI.
We are importing from https://git.heptacle.fr/neshiraini/sqlalchemy-enum-tables @ f2d832ad78aeac2acd6c0a6e22a6d43049a2c409
Test plan
none

Ok Shawn Liu approved this. :)


IDbContextOptionsBuilderInfrastructure seems too esoteric

There is an interface, IDbContextOptionsBuilderInfrastructure, that exists solely for this purpose:
((IDbContextOptionsBuilderInfrastructure)optionsBuilder).AddOrUpdateExtension(extension);

This is kind of a weird thing to figure out while writing an extension.  Can there be a similar method, like UseExtension<T>(T extension) where T : class, IDbContextOptionsExtension?

Oops, let this one slip me.
I am writing an extension that provides an alternative query compiler subsystem. It needs to register some things with the service collection.

@tuespetre That's reasonable. The reason it is done like this is because we wanted to keep it out of the API that is seen by application developers since they should never need to use this. Agreed that it is a bit ugly, but given that it is for advanced scenarios I'm not sure there is enough value in trying to put something prettier over the top of it.


refactor(VirtualElement): remove contextElement and add docs

There's no need for contextElement anymore since the offsetParent is always relative to the popper element.

Fixed the $FlowFixMe


[DO NOT MERGE] - TEST COMMIT TO TEST PR JOB

Signed-off-by: Kalyan Nidumolu<EMAIL_ADDRESS>

Hey @kalyan-nidumolu -- can we close this now plz? ;-)


Unable to Reply to Resolution Center For BETA REVIEW(betaNotes) via Spaceship reply_resolution_center, Please check on this

New Issue Checklist

[ ] Updated fastlane to the latest version
[ ] I read the Contribution Guidelines
[ ] I read docs.fastlane.tools
[ ] I searched for existing GitHub issues

Issue Description

Complete output when running fastlane, including the stack trace and command used

[INSERT OUTPUT HERE]

Environment

[INSERT OUTPUT HERE]

This is working as intended. The method currently does not provide access to reply to beta review.


CRD link is not working

Description
I am installing k8up and during CRD installation its failing ,As link  is not working
kubectl apply -f https://github.com/k8up-io/k8up/releases/download/k8up-4.0.2/k8up-crd.yaml -n k8up-operator
error: unable to read URL "https://github.com/k8up-io/k8up/releases/download/k8up-4.0.2/k8up-crd.yaml", server reported 404 Not Found, status code=404
wget https://github.com/k8up-io/k8up/releases/download/k8up-4.0.2/k8up-crd.yaml
Not FOUND
Additional Context
No response
Logs
error: unable to read URL "https://github.com/k8up-io/k8up/releases/download/k8up-4.0.2/k8up-crd.yaml", server reported 404 Not Found, status code=404_

Expected Behavior
Link should work for CRD installation
Steps To Reproduce
No response
Version of K8up
4.0.0
Version of Kubernetes
1.21
Distribution of Kubernetes
kubernetes

Hi @vfolk30
Thanks for pointing this out!
v4.0.2 of the helm chart has not yet been released. v4.0.1 should work though.
So could you try:
kubectl apply -f https://github.com/k8up-io/k8up/releases/download/k8up-4.0.1/k8up-crd.yaml -n k8up-operator
And see if it works for you?

HI @Kidswiss ,
Thank you for explanation. provided link is working.
I have a question, How to use custom CA root ca in k8up to work with   on-premises s3 compatible storage .

Hi @vfolk30
Custom CA is currently not supported. A similar discussion is held in this issue here: https://github.com/k8up-io/k8up/issues/792

Thank @Kidswiss .
You may close this and I will use QA forum for queries.


added NotificationService

Coverage increased (+0.02%) to 95.775% when pulling 966e3dd3ebe27a9732983ad17176aeea559b0056 on aportner:add-notification-service into 35b90d04dc7a7714e0d4ef30d39696ecb1ede065 on LPGhatguy:master.


[Enhancement] Extraction of constants [AbstractProducer RocketMQProducerImpl RocketMQConsumerImpl]

Search before asking

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

Enhancement Request

Describe the solution you'd like
Located at:eventmesh-connector-rocketmq/src/main/java/org/apache/eventmesh/connector/rocketmq
AbstractProducer RocketMQProducerImpl RocketMQConsumerImpl
Extract the constants from these classes to the Constants class in the common directory.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

I am interested to solve this issue, so can you please assign me for this issue ?

I am interested to solve this issue, so can you please assign me for this issue ?

@soundarrajan-official Sorry, this issue is finished and I will close it, are you interested in other issues? eventmesh you are more than welcome to participate.


.civ to not working after 1.2.0 release

You can't roll random from entire pool after the update that added search.
https://github.com/AuroraBTH/mr-civ/blob/d6b08eaef7e3dde5a66c073896a578a44433d689/commands/civilization.js#L18-L22
The above code needs to check if there is an argument sent, instead of just checking if if it's .civ lekmod or .civ base

Fixed in v1.2.2.


Fix SchemaRegistry live tests

/azp run java - schemaregistry - tests

API change check
API changes are not detected in this pull request.


Routing dynamically learnt services

Bug description
We have a service architecture where services come in and go out on a regular basis. These services register to a service registry, which is used by consumers to "discover" and consume. We want all external service traffic to flow via the egress gateway, since only those nodes (where the egress pods are scheduled) have external connectivity. So this creates a problem when the service that we are trying to dynamically discover and consume is outside the cluster, since each external service needs a ServiceEntry to be created.
One way out of this is for the service registry to create the required istio resources when new service registers to it. This would mean that we would perform infrastructure layer operations from application. May not be an ideal solution. Apart from this, there is no guarantee that the service registry itself will be within the cluster. So in such cases, the only way out is to manually configure the routing rules. And this breaks the dynamic nature of the service architecture.
Is there any "default" routing that can be applied so that all services that are not part of the istio service registry flow through the egress gateway? Some form of http2 proxy. Can this be introduced if its not already available?
Affected product area (please put an X in all that apply)
[ X] Configuration Infrastructure
[ X] Docs
[ ] Installation
[X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version)
Istio 1.2.5
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

in theory I think you could write an EnvoyFilter that replaces the passthrough cluster with a cluster that points to egress gateway.. I think I'll try it out if you don't have the chance/envoy experience since this sounds useful

Something like this maybe
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: passthrough-egress
  namespace: istio-system
spec:
  workloadSelector:
    labels:
      app: shell
  configPatches:
  - applyTo: CLUSTER
    clusterMatch:
      name: PassthroughCluster
    patch:
      operation: REMOVE
  - applyTo: CLUSTER
    clusterMatch:
      name: PassthroughCluster
    patch:
      operation: ADD
      value:
        connectTimeout: 1s
        dnsLookupFamily: V4_ONLY
        hosts:
        - socketAddress:
            address: istio-egressgateway.istio-system
            portValue: 80
        name: PassthroughCluster
        type: STRICT_DNS

Thanks a lot @howardjohn for the sample! I'll try this out and check if it satisfies our needs.
Do we know what would be the performance impact on this? Is this like any other routing evaluation?
I see the following note in istio docs. So isn't there a chance of breakage during Istio updgrades?
NOTE 1: Since this is break glass configuration, there will not be any backward compatibility across different Istio releases. In other words, this configuration is subject to change based on internal implementation of Istio networking subsystem.

So, if all you want is to forward external service traffic through the egress gateway, why dont you launch an envoy as a HTTP proxy on some unique port (see envoy docs https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/dynamic_forward_proxy_filter.html). And then in all your applications, you add a HTTP_PROXY environment variable to forward traffic via this forward proxy.
you might have some issues on the sidecar rejecting the hosts as it wont match any virtual host in the outbound. But you can fix that using EnvoyFilter CRD to overwrite the vhost on that specific port.
Another trick is what @howardjohn did [overwrite the passthrough cluster's endpoints by deleting the cluster and then adding a cluster of same name, but with your forward proxy as the endpoint.


Bai 19 - A

Phan main execise cua bai 19

Hi,
Your homework 19 has been reviewed.
Great! Keep up the good work!
Best wishes from Automation Test Pro,
Khanh


Implement emissive strength for PBR materials

This PR adds support for exporting emissive strength/intensity on PBR materials in VRM via the KHR_materials_emissive_strength extension.
https://github.com/KhronosGroup/glTF/tree/main/extensions/2.0/Khronos/KHR_materials_emissive_strength
Our users want to use HDR bloom so that parts of their avatar glow like this:

While the code works, i'm not entirely familiar with python or blender plugins so happy to improve this based on any feedback you may have.

OK. I'll tweak your commit and merge it later. Thanks!


TINKERPOP-1301 Provide Javadoc for ScriptInput/OutputFormat's

This trivial patch consolidates the Javadoc and links to the official reference documentation for the very powerful ScriptI/OFormats. The relevant Jira issue can be found at
https://issues.apache.org/jira/browse/TINKERPOP-1301

PR updated to current for documentation references. Nice work folks.

VOTE +1.

VOTE: +1

Actually, before we merge this after the next positive vote...

can you please squash the commits?
any reason we shouldn't merge this to tp31 and then master?

Commits merged sorry about that folks. Bad house keeping on my part.

Can you also please retarget this PR for the tp31 branch?

Replaced by #327 - going to close this one from our end.


Integration with Auth

Once I have Drywall set up and handling my user creation and management, how do I actually use these users to log in to my App?
Is it expected that I will use something like Passport's local strategy and just feed from the same DB?
Or does Drywall provide some sort of SSO functionality too?
Also, how is all of this affected by the use of subdomains?
Thanks

Once I have Drywall set up and handling my user creation and management, how do I actually use these users to log in to my App?

Your copy of Drywall is the app you'd login to. By default it's not an OAuth provider provider itself. You may want to checkout https://github.com/jaredhanson/passport-oauth2. I think this is similar to #169 if I'm not mistaken.
Apologies if I misunderstood the question. Let me know if this helps.

Sorry to let this close, and thanks for your efforts to answer.  I am not looking for Oauth per se.   Let me try to articulate this better.
Let's say I am creating a simple todo program.  I will need to have pages for user registration, administration of those users, profile data etc.  It seems that drywall handles most or all of that out of the box.
Then I will need to code pages/functions for entering and viewing todo lists, and those pages will need to know what user is active, and their details.  So how should I go about doing this, or is it even a feature that drywall is supposed to provide?
I am going to try to answer that myself with some perusal through the source, but please let me know if there is some intended way to interact with user login and data from within the code of the greater application.
Thanks ,
Adam

Checkout the /account area and related pages. For example /account/settings allows the currently logged in account to update their settings. You could do something similar and manage other data like todo lists and items.


Extension broken in incognito mode

When trying to open the addon in incognito mode, the error message "ERR_BLOCKED_BY_CLIENT" appears.
Looks like "split mode" has to be selected, see https://developer.chrome.com/docs/extensions/mv3/manifest/incognito/

Running second copy of the background page will surely break things so I'll have to set it to not allowed to avoid unforeseen issues


repo2docker for julia+jupyter?

Is there a solution yet for a julia notebook via repo2docker? I'm pretty new to Julia, but I put together my first Jupyter+Julia container (not using repo2docker, but using the continuous-builder) and it would probably be similar/trivial to have the same sort of execution for a julia notebook, just with some list of julia requirements and copying files to the place where julia expects them.

I'm still futzing around like a lost penguin (I need to do testing to figure out how to best run the container and pull to Singularity) but if it's helpful, here is what I was working on so far - this is generated from a repo with a modified circleci to also push metadata back to Github pages, and just build a standard Dockerfile, with the anchor bits to reference different "functions" in the config.

https://vsoch.github.io/julia-share/
https://hub.docker.com/r/vanessa/julia-share/
https://circleci.com/workflow-run/dfa766a9-821a-4214-b5d6-74715d97e17e

I'm going to have a writeup for the containershare (it's a nice little registry that anyone can pretty easily deploy to get the resulting table and linked containers) but for now I hope the links above can suffice.
I have no idea how jupyter can handle ipython and R and julia! It's like, magic. :sparkles:

To install Julia dependencies check out http://repo2docker.readthedocs.io/en/latest/config_files.html#require Is this what you were looking for?

Yes! This one --> https://github.com/binder-examples/julia-python is exactly what I would want! Thank you :)