crypto/ssh/knownhosts: check more than one key

I believe this fixes https://github.com/golang/go/issues/36126 .
The problem was that it was keeping only the first known key of each type found. If you have a server advertising multiple keys of the same type, you might get a missmatch key error.
Per sshd(8) man page, it should allow reapeatable hosts with different host keys, although it don't specify anything about hosts being from different types:

It is permissible (but not recommended) to have several lines or
different host keys for the same names.  This will inevitably happen when
short forms of host names from different domains are put in the file.  It
is possible that the files contain conflicting information;
authentication is accepted if valid information can be found from either
file.

So, this changes knownhosts behavior to accept any of the keys for a given host, regardless of type.
closes https://github.com/golang/go/issues/36126

My first investment with Charlotte give me profits of over $7,000 and ever since then she has never failed to deliver and I can even say she's the must sincere broker I have ever known'' communicate with her to be a successful investor. On WhatsApp:+𝟭𝟰𝟳𝟱𝟮𝟰𝟮𝟲𝟯𝟯𝟴

Any progress? Love to see this merged. Thanks!

One potential downside to this PR is it changes the documented behavior of knownhosts.KeyError.Want, re: "For each key algorithm there can be one hostkey"
type KeyError struct {
	// Want holds the accepted host keys. For each key algorithm,
	// there can be one hostkey.  If Want is empty, the host is
	// unknown. If Want is non-empty, there was a mismatch, which
	// can signify a MITM attack.
	Want []KnownKey
}

@lonnywong out of curiosity, do you really have 2 keys of the same type for the same hostname (same situation as https://github.com/golang/go/issues/36126, which would be solved by this PR)? Or do you have a server presenting multiple hostkeys of different types, some of which aren't in the client knownhosts file (see https://github.com/golang/go/issues/29286)?
Both issues result in the same user-facing error on the client side, but they have different root causes. In my experience, the latter situation is much more common.
I have a widely-used package, github.com/skeema/knownhosts, which uses some trickery to provide a workaround for https://github.com/golang/go/issues/29286, although it doesn't help with https://github.com/golang/go/issues/36126.
Just mentioning this here because github.com/skeema/knownhosts currently assumes the "For each key algorithm there can be one hostkey" logic remains the case, although I don't think it would break anything if that changes -- pretty sure it would just lead to potential duplicate entries in ssh.ClientConfig.HostKeyAlgorithms which hopefully is harmless? Anyway that should be easy to solve if it is a problem. Not sure if other packages out there depend on that logic more strictly though.

@evanelias Thanks very much for your help. My case is multiple hostkeys of different types. I didn’t find https://github.com/golang/go/issues/29286 before. I’ll try your awesome package https://github.com/skeema/knownhosts this weekend. Thanks again.

anything I can do to help move this forward?


 Errors should be at least logged.
If a channel no longer exists it should be automatically unsubscribed.

As part of this we need to also check if APR is in the requested channel, we can do this when someone subscribes.


Change http/https schemas in kubernetes service instance

Fixes https://github.com/spring-cloud/spring-cloud-kubernetes/issues/486

Cherry-picked and merged with a test in 1.1.x via 521195d08d81b482802020fe46f8ee79ddda9081


fix: update logback version to fix vulnerability

Logback core module recently added a 1.2.7 version that fixes an existing vulnerability. This updates to that new version.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

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


No indication what install is running

While testing #29449.
There is not indication whether I'm running the 32bit and 64bit Windows version of VS Code. The name of the program, as well as all information in the About dialog are the same.
The about dialog should tell me which version I'm running.

Maybe consider adding this to the issue template too(including but not limited to the template vscode generates).

This is fixed.


Iris Core + Plugins

Separating Iris API into 'core' API and multiple 'plugins' is a popular ambition offline, so deserves to be recorded publicly, especially for voting purposes 😉 .
Benefits:

Smaller package(s)
Quicker to install - fewer core dependencies
Can run on Iris on smaller machines e.g. AWS lambda
Faster testing - plugins (with accompanying dependencies) get their own separate testing routines
Clearer development philosophy

Examples of elements that could be plugins:

Plotting
Regridding
File loaders/savers

There are various small packages out there that do specific things that are useful for scientists.  For example, I am currently having a go with xeofs which currently has interfaces for pandas, xarray and numpy types.  If one were to think about adding an Iris interface to such a package, it would be useful (for testing) to have an Iris flavour that knows about the cube class and probably not much else.

@HGWright pointed out : The big cost of this, not just the effort of doing it but the ongoing extra repo maintenance (e.g. multiple dependabot hits)

I am still in favour of establishing a core 'concept' - defining Iris core determines Iris' core dependencies - but I don't think plugins would be worth the hassle. We can instead cut the Conda recipe down to the core dependencies and introduce errors/warnings for when a user needs a core dependency but has not yet installed it.

An important consideration:

#5936

Something not well captured above:
Iris-grib and iris-esmf-regrid have benefitted from being separated from the release cadence of Iris itself. This may be an opportunity for other areas of Iris too.
If any part of Iris is unlikely to reap this benefit (e.g. our plotting capabilities do not evolve very frequently) then I am still keen to explore optional Conda dependencies rather than full blown repo separation.


Feat/verify api

Description

Added context to auth_request based on the spec here to support the verify api
Added context to tests
Added context type to the AuthRequestEventArgs type

Resolves #72
How Has This Been Tested?
Modified unit tests in client.spec.ts to check if the verify data after a auth_request is correctly "unknown"

Due Dilligence

[ ] Breaking change
[ ] Requires a documentation update
[ ] Requires a e2e/integration test update

Looks like the spec PR just got merged with a few further edits, could you double check if any of the recent edits affect the implementation here so we know it’s aligned? 🙏
https://github.com/WalletConnect/walletconnect-docs/pull/436


Fixing a few things

Tooltip for carrion and infernal siegeworks

oil spawn i had indeed already fixed in the past


Is this project alive?

I see a bunch of issues and open PRs, but they stay untouched since some time. Is this project still alive?

For anyone looking for a maintained version, there's a fork that is being updated regularly: https://github.com/seblj/roslyn.nvim

I've been wanting to work go through issues and PRs for a long time, unfortunately I haven't had the time to do it. I still hope to be able to come back to it in the near future. But if not maybe I'll consider redirecting people to one of the forks.


Implement Debug for ColumnarValue

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
I'm frustrated because ColumnarValue does not have Debug implemented which makes debugging harder.
Describe the solution you'd like
I'd like to implement Debug for ColumnarValue. Since it is already implemented for both Array and ScalarValue it should be very straightforward.
Describe alternatives you've considered
Something like columnar_value.map(|cv| cv.into_array(1)) is doable but it is still not very straightforward.

Proposed fix: https://github.com/apache/arrow-datafusion/pull/3076


[Kernel][Model] logits_soft_cap for Gemma2 with flashinfer

Add logits_soft_cap for flashinfer, which is needed by Gemma2 model, also add a simple gemma2 test.

I think this warning can be removed to avoid confusion: https://github.com/vllm-project/vllm/blob/7cd2ebb0251fd1fd0eec5c93dac674603a22eddd/vllm/model_executor/models/gemma2.py#L140

I am able to run and reproduce the reported MMLU scores for both 9b and 27b models 👍
However, if I don't disable CUDA graph, vLLM will crash with this error:
ERROR 07-04 03:34:04 async_llm_engine.py:53]   File "vllm/vllm/worker/model_runner.py", line 1202, in execute_model
ERROR 07-04 03:34:04 async_llm_engine.py:53]     model_input.attn_metadata.decode_wrapper = self.graph_runners[
ERROR 07-04 03:34:04 async_llm_engine.py:53] IndexError: list index out of range

I am able to run and reproduce the reported MMLU scores for both 9b and 27b models 👍
However, if I don't disable CUDA graph, vLLM will crash with this error:
ERROR 07-04 03:34:04 async_llm_engine.py:53]   File "vllm/vllm/worker/model_runner.py", line 1202, in execute_model
ERROR 07-04 03:34:04 async_llm_engine.py:53]     model_input.attn_metadata.decode_wrapper = self.graph_runners[
ERROR 07-04 03:34:04 async_llm_engine.py:53] IndexError: list index out of range

Thanks for reporting! Could you give me an minimal reproducible example since I can run gemma-2 with flashinfer cudagraph on my end. Thanks!

I am using the run_batch script:
python -m vllm.entrypoints.openai.run_batch -i requests.jsonl -o /dev/null --model google/gemma-2-9b-it --disable-log-request

requests.jsonl.zip

I am using the run_batch script:
python -m vllm.entrypoints.openai.run_batch -i requests.jsonl -o /dev/null --model google/gemma-2-9b-it --disable-log-request

requests.jsonl.zip

I tried the script and data on H100, it seems work. Could you report your environment? Flashinfer only supports GPU with compute capability greater than 8.0 (https://developer.nvidia.com/cuda-gpus). Not sure if that might be the problem.

I am using the run_batch script:
python -m vllm.entrypoints.openai.run_batch -i requests.jsonl -o /dev/null --model google/gemma-2-9b-it --disable-log-request

requests.jsonl.zip

I tried the script and data on H100, it seems work. Could you report your environment? Flashinfer only supports GPU with compute capability greater than 8.0 (https://developer.nvidia.com/cuda-gpus). Not sure if that might be the problem.

I am using H100 with CUDA 12.5. Can you try sync you branch to the latest? https://github.com/vllm-project/vllm/pull/4412 might be related (it refactors graph_runners).

I am using the run_batch script:
python -m vllm.entrypoints.openai.run_batch -i requests.jsonl -o /dev/null --model google/gemma-2-9b-it --disable-log-request

requests.jsonl.zip

I tried the script and data on H100, it seems work. Could you report your environment? Flashinfer only supports GPU with compute capability greater than 8.0 (https://developer.nvidia.com/cuda-gpus). Not sure if that might be the problem.

I am using H100 with CUDA 12.5. Can you try sync you branch to the latest? #4412 might be related (it refactors graph_runners).

yes, it's a merge conflict. Just fixed, please try again. Thanks!

I am using the run_batch script:
python -m vllm.entrypoints.openai.run_batch -i requests.jsonl -o /dev/null --model google/gemma-2-9b-it --disable-log-request

requests.jsonl.zip

I tried the script and data on H100, it seems work. Could you report your environment? Flashinfer only supports GPU with compute capability greater than 8.0 (https://developer.nvidia.com/cuda-gpus). Not sure if that might be the problem.

I am using H100 with CUDA 12.5. Can you try sync you branch to the latest? #4412 might be related (it refactors graph_runners).

yes, it's a merge conflict. Just fixed, please try again. Thanks!

Thanks for the fix. It works now, w/ or w/o CUDA graph.


Starky proof is generated but can't be verified

I am trying to use the starky_bls12_381 library, and there is a specific case where a certain STARK is generated but fails to verify, showing a Mismatch between evaluation and opening of quotient polynomial error. I have described this issue in more detail here: issue link.
Because the proof is generated successfully and has the correct public inputs, I suspect this might actually be a problem with starky. The code uses an old version of Plonky2, but I am also reproducing the error with the latest Plonky2 version here: example link.

Hi! I've had a quick look at your code. It seems the issue lies in the following function call when defining the constraints:
add_fp12_multiplication_constraints(local_values, next_values, yield_constr, FINAL_EXP_OP_OFFSET, Some(local_values[FINAL_EXP_MUL_SELECTOR]));

I checked that native & recursive constraints match, so you most likely replicated the error in both methods. I haven't had the time to dive much more into it, but I'd suggest spending some time auditing the method to try identify where the issue lies.
Orthogonally, you'd probably benefit from testing all individual modules within your main FinalExponentiateStark, especially during development, as it should be pretty helpful for debugging to identify easily where the issue lies (although at the moment it seems your test suite is broken).

Hi!
I've had a quick look at your code. It seems the issue lies in the following function call when defining the constraints:
add_fp12_multiplication_constraints(local_values, next_values, yield_constr, FINAL_EXP_OP_OFFSET, Some(local_values[FINAL_EXP_MUL_SELECTOR]));

I checked that native & recursive constraints match, so you most likely replicated the error in both methods. I haven't had the time to dive much more into it, but I'd suggest spending some time auditing the method to try identify where the issue lies.
Orthogonally, you'd probably benefit from testing all individual modules within your main FinalExponentiateStark, especially during development, as it should be pretty helpful for debugging to identify easily where the issue lies (although at the moment it seems your test suite is broken).

Just realized this isn't your repo 😅 You'll probably want to report this finding upstream for them to investigate!


feat: rename Record to Download

Rename Record to Download.

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

@fcgxz2003 just only rename? are there any logic changes?

@fcgxz2003 just only rename? are there any logic changes?

@fcgxz2003 just only rename? are there any logic changes?

This is for refactoring storage, we need to store another data type,  "Record" represents all storage types and needs to be distinguished from one storage type. The other data type is NetworkTopology and Probes in another pr: https://github.com/dragonflyoss/Dragonfly2/pull/2252.

@fcgxz2003 why close?

@fcgxz2003 why close?

I noticed that some comments haven't been modified after renaming, so close and modify it.

@fcgxz2003 got. just do it.


'step' does not work with mu_slider_ex

It appears that the 'step' argument does not work with sliders. No matter which value I use the value is not rounded to multiple of 'step'. I briefly looked at the code and it seems cast to int or floor call is missing from microui.c line 873:
if (step) { v = (((v + step / 2) / step)) * step; }

Thanks!


Proxy target missing trying to run image rothnic/anaconda-notebook

Hi I'm probably doing something wrong.
I want to get rothnic/anaconda-notebook running as tmpnb.
I have no prob getting the jupyter/minimal running.
But getting this to work with the following command is give me a proxy target missing error.
docker run --net=host -d -e CONFIGPROXY_AUTH_TOKEN=$TOKEN --name=tmpnb -v /var/run/docker.sock:/docker.sock jupyter/tmpnb python orchestrate.py --image='rothnic/anaconda-notebook' --container_port=8000 --command="ipython notebook --NotebookApp.base_url={base_path} --port {port}"
Could you help me out? I would really love to use this configuration with my team.

I met the same problem, can you share your solution?

I encountered the same issue recently. I forgot the expose port 8888 from the container to the host. I added 'EXPOSE 8888' to the Dockerfile and rebuild the image.
You can check whether you're facing the same problem with the command 'docker ps -a'. If the ports of the spawned containers of rothnic look like '<IP_ADDRESS>:0:8888'. If correct it should like '<IP_ADDRESS>:32768:8888'.

I don't think you need to expose any ports since the --net=host option is specified. See https://docs.docker.com/articles/networking/

That's true for the proxy container and the tmpnb container that you're launching. But not for the spawned containers from the tmpnb (with the --image="dummy" option). If the dummy image doesn't expose the port from the ipython notebook, you'll encounter 'proxy target missing'.

Oh that's right. You need it in the dockerfile.

Can someone give a step-by-step on how to solve this that a n00b like me can follow?


Allow View with Python syntax to be formatted

Context:
I was playing around with some Python scripts that has no extension and noticed that...
Currently the format command accept Views if only filename extension is .py:
https://github.com/thep0y/python-black/blob/19166c3b948dd3a4168d77975a96bc5062f5800c/commands.py#L35-L41
Do you mind if it also accepts View which Python syntax is set (but not with .py extension)?
If that's OK, I'll open PR.

You are welcome to create a pr to solve this issue.


Kinematic hydrostatic pressure and buoyancy

This PR updates the code to use kinematic (density normalized) hydrostatic pressure and buoyancy rather than density perturbation. Using kinematic pressure simplifies the code because the reference density no longer affects the algorithm. Using buoyancy instead of the density perturbation will ultimately simplify the code further because it is needed for turbulence closures (for example).
The docs should be updated before merging.
Also, the golden master tests should be updated elsewhere; then it can be confirmed here that these changes preserve the functioning of the code (these changes do not affect the outcome of the algorithm).
I have also copy/pasted an interpolation operator that will become part of the code once #234 is merged. Prior to this PR, the interpolation of the buoyancy field to z faces was done manually.
We need a function to calculate buoyancy for the turbulence closures; so I think we are better off deleting the density perturbation function, ultimately. After this change the reference density ceases to be used in the algorithm.
I have also included salinity in the calculation of buoyancy.

Looks good to me! Agree that it should just pass the golden master test.
Would also be good to get @jm-c's comments before merging.

Hi there,
2 comments:

does not look like the most urgent thing in the to-do list so we could wait until
we have discussion in person.
with realistic EOS, we will need to comute density to get the hydrostatic
pressure AND N^2 (as input of mixing/SGS closuse scheme). Can express
everything as normalised by rho_c but could easily get confusing.

On Fri, May 24, 2019 at 06:08:42AM -0700, Ali Ramadhan wrote:

Looks good to me! Agree that it should just pass the golden master test.
Would also be good to get @jm-c's comments before merging.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/climate-machine/Oceananigans.jl/pull/236#issuecomment-495619852

We need a function to compute buoyancy for the turbulence closures. I chose to work on this for a reason.

We need a function to compute buoyancy for the turbulence closures.

I can hack in a function to compute buoyancy for the purposes of implementing the turbulence closures that we need in the very immediate future --- I was hoping that this would provide a more robust and permanent solution.

We can have both a δρ(eos, T, i, j, k) and a buoyancy(i, j, k, grid, eos, g, T, S) function so why can't the update_buoyancy! function continue using δρ (and maybe ρ in the future) while the turbulence closures can use the new buoyancy function?

@ali-ramadhan that will work. I just wanted to have the assurance that buoyancy was being computed identically everywhere (if tests that stress the buoyancy function pass, we know it works).

Fair enough, that's a good point. Seems like motivation to implement a unit test for the update_buoyancy! function (so that it can be tested with δρ and b), although not sure what a good test would be...

This isn't moving, so I'm closing it.


Scene not updated when changing an effect's input

(adapted from the notebook example in this repo):
In [2]: mesh2 = TetraMesh.from_vtk('piston.vtu')
   ...
   ...  iso2 = IsoColor(mesh2, input=('RESU____DEPL', 'DX'), min=-1.3931281e-06, max=1.3929895e-06)
   ...
   ...  scene2 = Scene([iso2])

In [3]: scene2

I'd like to change the isocolor input, e.g., from another notebook cell
In [4]: iso2.input = ('RESU____DEPL', 'DY')

But it has no effect on the scene that is already rendered in cell 3 output. To see the new input color I need to re-execute that cell.
Is it the normal behavior? I see that Effect.input is synchronized so I'd rather expect that changing it would update the scene(s) already shown in output cells.

It looks like a bug indeed, thanks for reporting

Hmm I've tracked the input change down to https://github.com/QuantStack/GanyJS/blob/ebfdea022ddf068b56d6d18b0707091d4d885809/src/NodeMesh.ts#L166 in GanyJS and everything seems to be updated properly. I don't know much three.js so I'm afraid I can't do more to tackle the issue here. In the meantime I've found a workaround for my use case.

Thanks for tracking it down. I fear this is due to the ThreeJS update. I'll try to find time to fix this.

@Krande my workaround is the same than yours 🙂

No rush @martinRenou! That workaround is not ideal but it's only minor annoyance.


Remove unnecessary vendored packages

PR#3002 removed dependencies to pillar but vendored files were not removed
Thanks to @shjala for pointing this out

LGTM.


interpreter->Invoke(); Segmentation fault in c++ inference

I am trying to deploy own object detection model in c++.
Once I run the code I am kept getting segmentation fault.
seems like cerr << memcpy(interpreter->typed_input_tensor<uchar>(0), image.data, image.total() * image.elemSize())<< endl; print out 0x7fff8c2fe580
However  it doesn't print 5 which is having problem from interpreter->Invoke()
This is what I am getting on debugger
#4  0x00007ffff7cda369 in tflite::optimized_ops::Conv(tflite::ConvParams const&, tflite::RuntimeShape const&, unsigned char const*, tflite::RuntimeShape const&, unsigned char const*, tflite::RuntimeShape const&, int const*, tflite::RuntimeShape const&, unsigned char*, tflite::RuntimeShape const&, unsigned char*, tflite::CpuBackendContext*) ()
#5  0x00007ffff7cdabc9 in void tflite::ops::builtin::conv::EvalQuantized<(tflite::ops::builtin::conv::KernelType)2>(TfLiteContext*, TfLiteNode*, TfLiteConvParams*, tflite::ops::builtin::conv::OpData*, TfLiteTensor*, TfLiteTensor*, TfLiteTensor*, TfLiteTensor*, TfLiteTensor*, TfLiteTensor*) ()
I am getting stuck and I am not sure how to fix this problem

Hi @sangyo1, can you please share the file or code snippets/text instead, Also please include your steps to compile and execute. Thanks.


[Feature Request] add support for the official mastodon app

I'd love it if the official mastodon app could be supported, assuming its possible from a technical standpoint
Official app https://www.f-droid.org/packages/org.joinmastodon.android/

The last time I checked, they didn't have way to open links externally

Checking again and it looks like they do have support now, so this will be in the next release.

https://github.com/zacharee/MastodonRedirect/releases/tag/1.3.3


proposal: net/url: url manipulation after creation/parsing

What version of Go are you using (go version)?

go1.14.2 darwin/amd64

Does this issue reproduce with the latest release?
It is reproducible also on Playground, go1.14.4
What operating system and processor architecture are you using (go env)?

GO111MODULE="on"
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"

What did you do?
net/url presents weird behaviour in some situations
Now, the issue I spot, is that there is no way to re-parse the same struct on url structure change.
All changes to the url.URL struct are direct property manipulation and not with proper struct fucntions
package main

import (
	"fmt"
	"net/url"
)

func parseURLWithoutScheme() *url.URL {
	u, err := url.Parse("somedomain.com/firstpath")
	if err != nil {
		return nil
	}
	return u
}

func parseURLWithScheme() *url.URL {
	u, err := url.Parse("https://somedomain.com/firstpath")
	if err != nil {
		return nil
	}
	return u
}

func main() {

	fmt.Println("1 path WITH scheme intialized:", parseURLWithScheme().Path)
        // output: /firstpath

	u := parseURLWithoutScheme()
	fmt.Println("2 path WITH NO scheme intialized:", u.Path)
        // output: somedomain.com/firstpath

	u.Scheme = "https"
	fmt.Println("3 path WITH NO scheme intialized, scheme was added later:", parseURLWithoutScheme().Path)
        // output: somedomain.com/firstpath

	u, _ = url.Parse(u.String())
	fmt.Println("4 same path of url struct in example 3, but reparsed into url.URL:", u.Path)
        // output: /firstpath

}

Available here
https://play.golang.org/p/3SkxRVIgmYS
The inability to change the url structure is such a way after creation is quirky.
Proposal
Add manipulation methods such as
func (*URL) SetPath(path string) { // re-parse and set properties respectively}

or allowing deep copy
func (*URL) DeepCopy() *URL { // re-parse and create a new URL struct}

Why not just add the scheme to your string?
func ParseURL(rawurl string) (*url.URL, error) {
	u, err := url.Parse(rawurl)
	if err != nil || u.Scheme != "" {
	    return u, err
	}
	return url.Parse("https://" + rawurl)
}

Why does this needs to be in the standard library?  How often does it come up?

Why does this needs to be in the standard library?  How often does it come up?

It's comes up every time that a URL is being manipulated in such a way. Maybe indeed a rare use-case but it creates an inconsistency that I cannot reason, if someone can do it, I would love to hear why the behavior is as presented and why a re-parsing function won't be an acceptable solution.
Thanks for your comment

@CommoDor64 what does your proposed SetPath method do?

@CommoDor64, in your example, the problem is that the definition of URLs is that "example.com/foo" is the same as "foo/bar" - they are both path-only URLs, with no domain at all. Even though example.com looks like a domain, it is not in this case. That is, it is not just a "URL without scheme". It is a "URL without scheme and without domain".
If you want a URL without scheme but with domain, that syntax is "//example.com/foo". Parsing that, setting u.Scheme, and reparsing u.String does work.
The problem here seems to be confusion about the URL syntax as defined by the RFC, not a bug in the Go library, which follows the RFC.

@rsc Thanks for taking the time to reply.
After looking at the RFC https://tools.ietf.org/html/rfc3986 I have to agree completely.
Thanks everyone, I will close the issue

ISTM that the odd thing here is that when there's a URL with a scheme but no host, the string representation makes the first element of the path into the host. Is there something better that it could do in that case? I suspect not.

FWIW, I've had several instances where I needed a Clone() *url.URL method.
It's not that hard to write, but I still think, that it would be nice to have in the
standard library.


Links in blog/atom.xml feed are broken

In the blog's atom feed, links to articles are broken.
They point to, e.g.,

/blog/2018/01/18/implementing-twitters-app-loading-animation-in-react-native.html (which redirects to https://code.facebook.com)
instead of /react-native/blog/2018/01/18/implementing-twitters-app-loading-animation-in-react-native.html

I think the feeds are still not working, as they point to https://facebook.github.io/blog/atom.xml.

@bcalaars this will be fixed when this repository updated their docusaurus version 😊

Docusaurus has been updated, it should be fixed by now. Let's close this, thanks!


Cloud 100 load balancer doesnt work

Codecov Report

Merging #243 into master will decrease coverage by 0.27%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #243      +/-   ##
==========================================
- Coverage   65.03%   64.76%   -0.28%
==========================================
  Files          38       38
  Lines        1945     1947       +2
==========================================
- Hits         1265     1261       -4
- Misses        581      588       +7
+ Partials       99       98       -1

Impacted Files
Coverage Δ

pkg/pod/k8s/pods.go
81.25% <ø> (-4.69%)
:arrow_down:

pkg/pod/k8s/task.go
46.49% <0%> (-1.73%)
:arrow_down:

Continue to review full report at Codecov.

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


Builtin endpoints do not follow CRUD

get() and receive() and other endpoints which get data from serial port should only be accessible via GET requests, and sending data should be accessible via POST requests.
Additionally, the names/terms of some RestApiHandler methods, such as add_endpoint(), are wrong, because it is actually adding a resource. Endpoints are different for different request methods. The method/decorator should be called something like add_conn_resource() instead.

This isn't breaking change because we can create an alias for the add_endpoint() method but rename it to something else.

I will fix the second part of this issue later because there are too many places where "route", "endpoint", and "resource" are used interchangeably. This includes the documentation, comments in the code, variable names, method names, parameter names, etc.


Add Dockerfile for consistency with other projects

We can close this issues, we don't see any scenario on which apply a dockerized  version of yochu.


Improve tests for handling additionalProperties

Signed-off-by: Alan Cha<EMAIL_ADDRESS>

@Alan-Cha Great work, leading to better responses in light of incomplete JSON schema definitions.


[Feature]:  Support for Pydantic v2

Brief Description
I propose that we add support for Pydantic v2 in the Vocode project. This will allow us to leverage the latest features and improvements in Pydantic, such as enhanced performance and strict mode for data validation.
Rationale
Pydantic is a critical part of our project, and keeping up with the latest version ensures we can leverage all its features and improvements. This can lead to better performance, improved features, and fewer bugs. Pydantic v2, in particular, offers significant performance improvements and a new strict mode for data validation, which could be beneficial for our project.
Suggested Implementation

Update the Pydantic version in our project dependencies to v2.
Review the Pydantic v2 migration guide and update our existing code to ensure compatibility with Pydantic v2.
Test our application thoroughly to ensure that the update does not introduce any regressions.
If necessary, use the code transformation tool provided by Pydantic to assist with the migration.

If any issues are encountered during the migration, we can report them to the Pydantic team using the bug V2 label on GitHub.

Just to add to this, it may also be beneficial to move to support pydantic's Settings Management with a v2 transition. Probably worthy of a whole separate issue though.
I know theres a lot of dotenv work being done but this drastically simplifies a project's environment management and can even open dynamic importing from remote SecretsManagers (happy to provide PoC code for GCP Secret Manager I wrote in the past)
See docs here:
https://docs.pydantic.dev/latest/concepts/pydantic_settings/

That's an interesting proposition! Particularly for handling dynamic imports from remote Secrets Managers.
To ensure this suggestion gets the attention it deserves, I agree that creating a separate Feature Request is the best course of action.

Fix #490


Create some test transactions to feed to device

Create a set of bts_transaction_xxx.json files to guide the development of the serialization utilities (#1) and for feeding to the Nano S to test decoding of serialized transactions (#2).
E.g, test jsons of the following transaction types would be useful:

Transfer (Op 0)
...

See also: #4

Transfer Operation Example:
An example un-signed transaction containing a Transfer operation is seen here in JSON:
{
  "ref_block_num": 35221,
  "ref_block_prefix":<PHONE_NUMBER>,
  "expiration": "2018-12-04T03:54:15",
  "operations": [[
      0,{
        "fee": {
          "amount": 10420,
          "asset_id": "1.3.0"
        },
        "from": "1.2.1152620",
        "to": "1.2.1152699",
        "amount": {
          "amount": 4000000,
          "asset_id": "1.3.0"
        },
        "extensions": []
      }
    ]
  ],
  "extensions": []
}

This can be serialized in the cli_wallet.  However it will choke on the line breaks.  If the above is in a file called bts_transaction_transfer.json then we can strip the eol's and excess whitespace with:
$ cat bts_transaction_transfer.json | tr -d "\n" | tr -s " "
Then, within the cli_wallet, we get the serialized form with the serialize_transaction command, which take the json object as an argument, as follows:
>>> serialize_transaction { "ref_block_num": 35221, "ref_block_prefix":<PHONE_NUMBER>, "expiration": "2018-12-04T03:54:15", "operations": [[ 0,{ "fee": { "amount": 10420, "asset_id": "1.3.0" }, "from": "1.2.1152620", "to": "1.2.1152699", "amount": { "amount": 4000000, "asset_id": "1.3.0" }, "extensions": [] } ] ], "extensions": []}

And produces the following result:
"958934be51b867fa055c0100b42800000000000000ecac46bbad4600093d00000000000000000000"
TODO:

[ ] Attempt to manually interpret that result.  Is it DER encoded?  Does the format appear to be the same as expected by device?

Another Transfer example, with Memo and Signature:
serialize_transaction { "ref_block_num": 18714, "ref_block_prefix": 966826885, "expiration": "2018-12-08T03:37:57", "operations": [[ 0,{ "fee": { "amount": 10940, "asset_id": "1.3.0" }, "from": "1.2.1152620", "to": "1.2.1152699", "amount": { "amount": 1000000, "asset_id": "1.3.121" }, "memo": { "from": "BTS6UAQW11iC7AnsbwA8GRnzGFW7xZ6Eap1cBHWtepAhMQF9w1L9i", "to": "BTS6tQzY1aVqcu63bWho7Nqp1RL8PXUNGkKUCX8dch8kJE6yzvAbG", "nonce": "11963104850544806207", "message": "ca5d847eb9ca3776d2e268ba3c6eb2ba" }, "extensions": [] } ] ], "extensions": [], "signatures": [ "1f46cf6a430a5f9add45d063cead9afa0c9df99c1e5598895fa375c3057862b0dc2819ab2e22b953f023600dd444ed9e4117b4442af2970a272511eacd3a6e6743" ]}

Result:
1a49859ba039953c0b5c0100bc2a00000000000000ecac46bbad4640420f0000000000790102d027cd80d88195318e0e99013a7fc053fc2a4ef21697efdd9674268538dbfa8b030738b236cceabdfaaec1af4e1a4aad8544220176419c1a40b6cbf990f95337f43fc979737a7c05a610ca5d847eb9ca3776d2e268ba3c6eb2ba0000011f46cf6a430a5f9add45d063cead9afa0c9df99c1e5598895fa375c3057862b0dc2819ab2e22b953f023600dd444ed9e4117b4442af2970a272511eacd3a6e6743

Decoding:

Field
Data

ref_block_num
1a49

ref_block_prefix
859ba039

expiration
953c0b5c

num_operations
01

operation_id
00

fee.amount
bc2a000000000000

fee.asset
00

from.id
ecac46

to.id
bbad46

amount
40420f0000000000

amount asset id
79

memo bool
01

memo.from
02d027cd80d88195318e0e99013a7fc053fc2a4ef21697efdd9674268538dbfa8b

memo.to
030738b236cceabdfaaec1af4e1a4aad8544220176419c1a40b6cbf990f95337f4

memo.nonce
3fc979737a7c05a6

memo.message_length
10

memo.message
ca5d847eb9ca3776d2e268ba3c6eb2ba

num_op_extensions
00

num_extensions
00

num_signatures
01

signature
1f46cf6a430a5f9add45d063cead9afa0c9df99c1e5598895fa375c3057862b0dc2819ab2e22b953f023600dd444ed9e4117b4442af2970a272511eacd3a6e6743

Test transactions in JSON files are included in example-tx/ folder.  As of the 0.0.0 release, these serialize correctly and are capable of serving as examples both of Tx formats and wire APDU formatting.


expose useful infomations

Hi,
I'm doing some webdriver testing which client UI is in a browser, your package just match my requirement and it work fine.
However, when I try to access some response data form server, I found that some of them are discarded in the package.
So, here is some modify to expose the error type in error message and the accurate capabilities that what environment browser is using.
Btw, not sure it's my problem, the lint in pre-commit work strange, it trying to add brackets and commas in some place I'm not modify, so I disable it in my commit, just let you known that.

and it looks like I need some help, can't pass your testing, any idea?


setFormState设置values，在动态visible设置的时候会出现设置不上的问题

Help us to manage our issues by answering the following:

How would you tag this issue?

Question
[x] Bug
Discussion
Feature request
Tip
Enhancement
Performance

Describe your issue:

What is the current behavior?
What is the expected behavior?
Which versions of UForm?
Did this work in previous versions of UForm?

If your issue is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via Bug Report Template on CodeSandbox

https://codesandbox.io/s/spring-pond-gz23e
.

想了一下，visible不能开放设置，因为这个开放了之后，会和visible设置为false删值的行为互斥，要实现隐藏状态下设置值，只能用display


Add libsfml package

Github Repository: SFML

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Note: CMake based

A FindSFML.cmake module is included, should that be used or is patching it still the better option?

A FindSFML.cmake module is included, should that be used or is patching it still the better option?

Generating SFMLConfig.cmake automatically is the only correct way to use package. Usually Find*.cmake modules doesn't support more than 2 build types for example (and if you change postfix it will not work at all). Plus if Find*.cmake doesn't create imported targets the package will not be relocatable, e.g. you will not be able to upload it to the binary server.

I see, anyways there are talks upstream of making a SFMLConfig.cmake or a sfml-config.cmake file. Should we wait or patch in the meantime?

Should we wait or patch in the meantime?

we can help :) can you show discussion (or issue)

SFML#937

https://github.com/SFML/SFML/issues/937

Thanks. They have components. Will not be as trivial as usual but is doable, will take a look tomorrow.

this short (forum duscussion)[http://en.sfml-dev.org/forums/index.php?topic=5789.msg37888#msg37888] might be relevant?

this short forum duscussion might be relevant?

It mean quite the same what I said: generate SFMLConfig automatically instead of writing FindSFML

just for your info: https://github.com/SFML/SFML/pull/1103

Will not be as trivial as usual but is doable, will take a look tomorrow.

Change my mind. Will not doing it, sorry.

Hello @ruslo sfml upgraded their cmake process. https://en.sfml-dev.org/forums/index.php?topic=24070.0 is this now easier for you to add the library?

is this now easier for you to add the library?

@maluramichael I personally have no such plans, if you want to work on this I can consult you, e.g. review CMake code.

There are 4 bundled libs (3 of which is already in hunter (flac is missing)):
https://github.com/SFML/SFML/tree/master/extlibs/libs-ios

[ ] libflac : https://github.com/xiph/flac (autoconf)
[x] libfreetype : https://docs.hunter.sh/en/latest/packages/pkg/freetype.html?highlight=freetype
[x] libogg : https://github.com/xiph/ogg (autoconf)
[x] libvorbis : https://github.com/xiph/vorbis (CMake)


Remove extern crate directives

These are no longer needed as of the 2018 edition

This one seems innocuous enough


Two user profile pages - let's make it one

Make sure these boxes are checked before submitting your issue - thank you!

[X] I have checked the superset logs for python stacktraces and included it here as text if any
[X] I have reproduced the issue with at least the latest released version of superset
[X] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.22.1
Expected results
I'd like to have:

one user profile page
welcome page when clicking on Superset logo

Actual results
There is two profile pages:

when clicking on superset logo:
/superset/profile/admin/
when clicking on user icon / profile
/users/userinfo/

What's the recommendation ?

I would say the /users/userinfo/ page should be linked to on /superset/profile/admin/ as a tab. Similar to the recent activity and security & access.

/users/userinfo/ is the page we get out of the web framework we use. It'd be hard to customize, the vision is to improve the other profile page, we should probably make it the welcome page as well.

So idea is to merge information from /superset/profile/admin/ to /welcome ?

@mistercrunch Maybe then not use /users/userinfo and put that information on the other page ignoring /users/userinfo altogether..?


Create vspheredatacenterconfig reconciler

… that validates vcenter config and update its status
Issue #, if available:
#870
Description of changes:
API changes

Add SpecValid to VSphereDatacenterConfigStatus

Doc changes

Add VSphereDatacenterConfigStatus session to cluster full lifecycle API design

Code changes

Refactor vsphere setup method
export vsphere validator and defaulter struct
initialize validator and defaulter during controller setupmanager time
fetch vsphere-credentials secret from cluster and use it for Govc calls
validate vcenterconfig and update its status within reconciler

Follow up

Make govc executable independent from using Env Vars
Add FailureReason and FailureMessage to VSphereDatacenterConfigStatus
Unit tests

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/retest

/lgtm


False positive for rule "Using Components with Known Vulnerabilities."

Hello,
I used the latest version 0.9.16-java of jruby-openssl gem in my project, but when I use sonarqube to scan my code, it reports many "Using Components with Known Vulnerabilities.". issues caused by jruby-openssl, for example CVE-2015-0292, when I used the latest version 0.3.5 of thread_safe gem in my project, sonarqube also reports one "Using Components with Known Vulnerabilities." issue caused by thread_safe, for example CVE-2011-4838.  The rule "Using Components with Known Vulnerabilities."  is from OWASP Dependency-Check plugin.
I asked developers of project jruby-openssl  and thread_safe, they confirmed these two gems don't subject to these vulnerabilities,  please check https://github.com/jruby/jruby-openssl/issues/89 and https://github.com/ruby-concurrency/thread_safe/issues/25 for details.
Thanks,
Sam

I am not the author of a sonar plugin for dependency-check - although I believe a couple of plugins exist. Dependency-check will absolutely report false positivies - most of these are extremely easy to identify as false positives. I would highly recommend reading "How It Works" and "Suppressing False Positivies". As I am not familiar with exatly which plugin you are using I'm not sure if it is possible to add the suppression file.
If you have access to the full dependency-check report (example here) if you can post the Details, Evidence, and Identifiers sections of the report for both gems I can add suppression rules to the core engine.
Best Regards,
--Jeremy

@jeremylong , thanks your reply. it's much clear to me now. The following is the detailed info of two gems, because there are a lot of rules are violated, can't list all, just list some of them.
(1) jruby-openssl-0.9.16-java
the  jopenssl.jar file in this gem violated many rules:
CVE-2010-0742 | CVSS Score: 7.5 | Category: CWE-310 Cryptographic Issues
CVE-2009-3245 | CVSS Score: 10.0 | Category: CWE-20 Improper Input Validation
CVE-1999-0428 | CVSS Score: 7.5
CVE-2011-4838 | CVSS Score: 7.8
...
the jruby-openssl-0.9.16-java.gemspec file in this gem also violated many rules:
Reference: CVE-2016-0704 | CVSS Score: 4.3 | Category: CWE-200
Reference: CVE-2016-0703 | CVSS Score: 4.3 | Category: CWE-200
Reference: CVE-2015-4000 | CVSS Score: 4.3
...
the bcprov-jdk15on-147.jar file in this gem also violated the following rule:
Reference: CVE-2015-7940 | CVSS Score: 5.0 | Category: CWE-200 Information Exposure
(2) thread_safe-0.3.5-java
the jruby_cache_backend.jar  file in this gem violated many rules:
Reference: CVE-2012-5370 | CVSS Score: 5.0 | Category: CWE-310 Cryptographic Issues
Reference: CVE-2010-1330 | CVSS Score: 4.3 | Category: CWE-79 Improper Neutralization of Input During Web Page Generation ('Cross-site Scripting')


IOS - undefined is not an object (evaluating 'tns_core_modules_application__WEBPACK_IMPORTED_MODULE_9__["android"].context')

Hi,
We have an issue with a project who was suppose to be only on Android and now wanted to be also on IOS.
The thing is, when I try to run the project with "tns run iOS", it successfully build and run but I have a white screen and an error message.
I'm on iPhone 8 emulator.
Nativescript Version : 6.2.0.
Xcode Version : 11.3
Package.json :
{
  "nativescript": {
    "id": ".......",
    "tns-android": {
      "version": "6.3.1"
    },
    "tns-ios": {
      "version": "6.2.0"
    }
  },
  "description": "NativeScript Application",
  "license": "SEE LICENSE IN <your-license-filename>",
  "repository": "<fill-your-repository-here>",
  "dependencies": {
    "@angular/animations": "~8.2.0",
    "@angular/common": "~8.2.0",
    "@angular/compiler": "~8.2.0",
    "@angular/core": "~8.2.0",
    "@angular/forms": "~8.2.0",
    "@angular/http": "8.0.0-beta.10",
    "@angular/platform-browser": "~8.2.0",
    "@angular/platform-browser-dynamic": "~8.2.0",
    "@angular/router": "~8.2.0",
    "@ngx-translate/core": "^11.0.1",
    "@ngx-translate/http-loader": "^4.0.0",
    "@nstudio/nativescript-loading-indicator": "^1.0.0",
    "nativescript-angular": "^8.20.4",
    "nativescript-background-http": "^3.4.0",
    "nativescript-camera": "^4.4.0",
    "nativescript-imagecropper": "^1.0.7",
    "nativescript-nfc": "4.0.1",
    "nativescript-permissions": "^1.3.8",
    "nativescript-plugin-firebase": "9.0.2",
    "nativescript-theme-core": "~1.0.4",
    "reflect-metadata": "~0.1.10",
    "rxjs": "^6.4.0",
    "tns-core-modules": "^6.3.2",
    "zone.js": "^0.9.1"
  },
  "devDependencies": {
    "@angular/compiler-cli": "~8.2.0",
    "@nativescript/schematics": "~0.5.0",
    "@ngtools/webpack": "~8.2.0",
    "nativescript-dev-webpack": "^1.4.1",
    "tns-platform-declarations": "6.0.1",
    "typescript": "~3.5.3"
  },
  "readme": "NativeScript Application"
}

Tell me if you need something I forgot.
Log :

Installing on device 4D7E8B60-329A-4B3B-B7D3-50325F2D6394...
Successfully installed on device with identifier '4D7E8B60-329A-4B3B-B7D3-50325F2D6394'.
Successfully transferred all files on device 4D7E8B60-329A-4B3B-B7D3-50325F2D6394.
Restarting application on device 4D7E8B60-329A-4B3B-B7D3-50325F2D6394...
Successfully synced application ..... on device 4D7E8B60-329A-4B3B-B7D3-50325F2D6394.
CONSOLE INFO file:///node_modules/nativescript-dev-webpack/hot.js:3:0: HMR: Hot Module Replacement Enabled. Waiting for signal.
CONSOLE LOG file:///node_modules/@angular/core/fesm5/core.js:25638:0: Angular is running in the development mode. Call enableProdMode() to enable the production mode.
CONSOLE LOG file:///src/app/app.component.ts:25:26: firebase.init done
CONSOLE ERROR file:///node_modules/@angular/core/fesm5/core.js:4002:0: ERROR Error: Uncaught (in promise): TypeError: undefined is not an object (evaluating 'tns_core_modules_application__WEBPACK_IMPORTED_MODULE_9__["android"].context')
CONSOLE ERROR file:///node_modules/@angular/core/fesm5/core.js:4002:0: ERROR {
CONSOLE ERROR file:///node_modules/@angular/core/fesm5/core.js:4002:0: ERROR {

Thanks !

Sorry my bad, I had code who used app.android and because the app was suppose to be only android we didn't put if(isAndroid) , now it works


ci: Update dependabot automerge job

Instructions

PR target branch should be against development
PR title name should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-title-check.yml
PR branch prefix should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-branch-check-name.yml

Summary
Update automerge dependabot job
Testing Plan

[ ] Was this tested locally? If not, explain why.
{explain how this has been tested, and what, if any, additional testing should be done}

Reference Issue (For mParticle employees only.  Ignore if you are an outside contributor)

Closes https://go.mparticle.com/work/SQDSDKS-5204

:tada: This PR is included in version 2.20.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add ActionFilter to be used when you want to automatically delete files after post action executed

This is not always desirable, so you could annotate actions/controllers with this actionfilter to have files deleted after an action that has a PluploadFileCollection in the posted model.

Also include an option to only delete files after the action executed and returned a Redirect(Route)Result. That usually indicates the Post was a succes. Usually when there are validation errors, you'll return a ViewResult or something and you might want to keep the files in the temp folder still, to avoid re-uploading the files.

Some tests are needed for this feature. Works fine in the sample application.


How to use PubSub on TypeScript ?

How to use PubSub on TypeScript ?
I want to use PubSub on Typescript  and try to use the sample code on this page
https://helia.io/modules/_helia_ipns.html
However when I run the code, Some issues emerged.
These error is below .These error happend during compile .
src/main.ts:347:5 - error TS18048: 'libp2pOptions.services' is possibly 'undefined'.

347     libp2pOptions.services.pubsub = gossipsub()
        ~~~~~~~~~~~~~~~~~~~~~~

src/main.ts:363:11 - error TS18046: 'helia.libp2p.services.keychain' is of type 'unknown'.

363     await helia.libp2p.services.keychain.importKey("self_key_pair",exported,user_name)
              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

src/main.ts:368:18 - error TS2345: Argument of type 'HeliaLibp2p<Libp2p<ServiceMap>>' is not assignable to parameter of type 'PubsubRoutingComponents'.
  The types of 'libp2p.services' are incompatible between these types.
    Property 'pubsub' is missing in type 'ServiceMap' but required in type '{ pubsub: PubSub<PubSubEvents>; }'.

368           pubsub(helia)
                     ~~~~~

  node_modules/@helia/ipns/dist/src/routing/pubsub.d.ts:10:13
    10             pubsub: PubSub;
                   ~~~~~~
    'pubsub' is declared here.

src/main.ts:375:9 - error TS2322: Type 'HeliaLibp2p<Libp2p<ServiceMap>>' is not assignable to type 'HeliaLibp2p<Libp2p<DefaultLibp2pServices>>'.
  Type 'Libp2p<ServiceMap>' is not assignable to type 'Libp2p<DefaultLibp2pServices>'.
    Type 'ServiceMap' is missing the following properties from type 'DefaultLibp2pServices': autoNAT, dcutr, delegatedRouting, dht, and 5 more.

375         helia: helia,
            ~~~~~

  src/main.ts:34:5
    34     helia:HeliaLibp2p,
           ~~~~~
    The expected type comes from property 'helia' which is declared here on type 'peer_ctx'

Found 4 errors in the same file, starting at: src/main.ts:347

Is this issue still occuring with the latest? We had an issue with typings a few months ago that I believe is now resolved. Can you try again and share repro code if you are still seeing issues?

By building Helia node myself, It move.
like:
var libp2p = await createLibp2p({
    services: {
      autoNAT: autoNAT(),
      dcutr: dcutr(),
      delegatedRouting: () => createDelegatedRoutingV1HttpApiClient('https://delegated-ipfs.dev'),
      dht: kadDHT({
        validators: {
          ipns: ipnsValidator
        },
        selectors: {
          ipns: ipnsSelector
        }
      }),
      identify: identify({
          agentVersion
      }),
      ping: ping(),
      relay: circuitRelayServer({
          advertise: true
      }),
      pubsub: gossipsub({ allowPublishToZeroTopicPeers: true }),
      keychain: keychain()
    },
})

const helia = await createHelia({libp2p})

name = ipns(helia,{
    routers: [
      pubsub({
        datastore: helia.datastore,
        libp2p: {
          peerId: helia.libp2p.peerId,
          services: {
            pubsub: helia.libp2p.services.pubsub as PubSub
          }
        }
      })
    ]
})

fs = unixfs(helia)

https://github.com/polus-arcticus/helia-react-components/pull/2 has an attempt to bring pubsub with typescript and i'm seeing some similar type errors as well
build

><EMAIL_ADDRESS>build
> aegir build

[11:49:54] tsc [started]
src/helia-provider.tsx:2:28 - error TS2459: Module '"libp2p"' declares 'ServiceMap' locally, but it is not exported.

2 import { type Libp2p, type ServiceMap } from 'libp2p'
                             ~~~~~~~~~~

  ../../node_modules/libp2p/dist/src/index.d.ts:21:23
    21 import type { Libp2p, ServiceMap, ComponentLogger, NodeInfo, ConnectionProtector, ConnectionEncrypter, ConnectionGater, ContentRouting, Metrics, PeerDiscovery, PeerRouting, StreamMuxerFactory, Transport, PrivateKey } from '@libp2p/interface';
                             ~~~~~~~~~~
    'ServiceMap' is declared here.

src/usePubsubRoom.ts:26:37 - error TS2345: Argument of type '"message"' is not assignable to parameter of type 'keyof Libp2pEvents<any>'.

26       helia.libp2p.addEventListener('message', (event) => {
                                       ~~~~~~~~~

src/usePubsubRoom.ts:27:36 - error TS2339: Property 'topic' does not exist on type 'PeerId | Connection | Libp2p<any> | PeerUpdate | PeerInfo | IdentifyResult | Listener | Connection[]'.
  Property 'topic' does not exist on type 'RSAPeerId'.

27         const topic = event.detail.topic
                                      ~~~~~

src/usePubsubRoom.ts:28:38 - error TS2339: Property 'data' does not exist on type 'PeerId | Connection | Libp2p<any> | PeerUpdate | PeerInfo | IdentifyResult | Listener | Connection[]'.
  Property 'data' does not exist on type 'RSAPeerId'.

28         const message = event.detail.data
                                        ~~~~

src/usePubsubRoom.ts:29:21 - error TS2345: Argument of type '(old: []) => [{ topic: any; message: any; }]' is not assignable to parameter of type 'SetStateAction<[]>'.
  Type '(old: []) => [{ topic: any; message: any; }]' is not assignable to type '(prevState: []) => []'.
    Type '[{ topic: any; message: any; }]' is not assignable to type '[]'.
      Source has 1 element(s) but target allows only 0.

29         setMessages(old => [...old, {topic, message}])
                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Found 5 errors in 2 files.

https://github.com/ipfs/helia/blob/b723a1d9910c69064eafba965d4d96d753b91297/packages/helia/src/helia-p2p.ts#L14C14-L14C22
seems like the candidate to import, though i cant seem to import { type HeliaP2P} from helia

got it working by referencing this https://github.com/ipfs/helia/blob/b723a1d9910c69064eafba965d4d96d753b91297/packages/interop/src/ipns-pubsub.spec.ts#L36


Added ng-content to formly-group. Updated Examples to inline remove btn

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Added ng-content to formly-group. So that users can add content inside the field group such as Remove button.
What is the current behavior? (You can also link to an open issue here)
#840
What is the new behavior (if this is a feature change)?
Users able to add html content inside field group
Please check if the PR fulfills these requirements

[ ] The commit messages follow our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] A unit test has been written for this change.
[ ] Running npm run build produced a successful build. (Unit testing can be done by running npm test;)
[ ] My code has been linted. (npm run lint to do this. npm run build will fail if there are files not linted.)

Please provide a screenshot of this feature before and after your code changes, if applicable.
Other information:

This change is

Codecov Report

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

@@           Coverage Diff           @@
##           master     #843   +/-   ##
=======================================
  Coverage   92.91%   92.91%
=======================================
  Files          22       22
  Lines         918      918
  Branches      252      252
=======================================
  Hits          853      853
  Misses         65       65

Impacted Files
Coverage Δ

src/core/src/components/formly.group.ts
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 295974b...c6fe2db. Read the comment docs.

Thank you @n3xus!


Add proper key usage to haproxy certs to avoid strict browser complaints

Add key usage to haproxy generated cert

it should, but how much do we want to diverge from the core?

Ideally we should set proper key usage for all certs and PR upstream eventually for core components themselves


[837] Add google tag manager

Context
Analytics
Changes proposed in this pull request
We can add Google Analytics (and other tracking tools) and set up goals etc with GTM
Guidance to review
🚢

 [S] Add Google Analytics

 [S] Add Google Analytics


Introduce finch-generic module

Before:
import io.finch._
val e = Endpoint.derive[Foo].fromParams

After:
import io.finch._
import io.finch.generic_
val e = deriveFor[Foo].fromParams

Resolves #528

Current coverage is 78.64% (diff: 100%)

Merging #716 into master will increase coverage by 0.30%

@@             master       #716   diff @@
==========================================
  Files            34         35     +1
  Lines           665        665
  Methods         642        644     +2
  Messages          0          0
  Branches         23         21     -2
==========================================
+ Hits            521        523     +2
+ Misses          144        142     -2
  Partials          0          0

Powered by Codecov. Last update d27f085...e8335a1

@vkostyukov @rpless any chance to get a feedback here?

Hey @ilya-murzinov! Any chance you'd be able to update this branch today/tomorrow? I was hoping to get the release out and wanted to include these changes there as well.

Sure, will do tomorrow!
On Wed, Mar 29, 2017, 20:34 Vladimir Kostyukov<EMAIL_ADDRESS>wrote:

Hey @ilya-murzinov https://github.com/ilya-murzinov! Any chance you'd
be able to update this branch today/tomorrow? I was hoping to get the
release out and wanted to include these changes there as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/finagle/finch/pull/716#issuecomment-290164519, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADiGKxE6q9vXxheyoAq6mS2uiLiRQTaOks5rqpYhgaJpZM4Ldr1x
.


What format for SBOM would be best for Liquibase?

For people using Liquibase out there in an environment that needs/would benefit from a Software Bill of Materials, I am interested in which format(s) would be most useful to you.
The three most frequently used now are SPDX (Linux Foundation), SWID (NIST), CycloneDX, (OWASP)

Don't know if that helps the decision: CycloneDX is simple enough and has some tooling to generate it (see https://github.com/CycloneDX). It is also possible to generate a SPDX (e.g. with https://github.com/CycloneDX/cyclonedx-cli with convert command).

Very helpful @robertk3s thank you. I was definitely looking at CycloneDX! Are you consuming any SBOMs on your end in this format? Producing them seems like the easy part, and I want to make sure what we end up producing is digestible/usable for users who need them. @stevespringett saw you 👍🏻 as well... why do you prefer CycloneDX? Thanks all!

HI @kataggart. Yes, my organization has standardized on CycloneDX due to the wide array of cybersecurity use cases it supports - and also supports most license use cases as well. There are a lot of tools that can consume CycloneDX today. The Maven plugin validates the generated BOM when it's produced.  Because of this, you can be sure that the generated BOM will be consumable by any application that can ingest and analyze them. It's actually pretty simple.
I know the NTIA folks have a paper that mentions SWID. Do not consider it though. The NTIA standards and formats group does not consider SWID to be a viable format, and SWID is not being considered by CISA now that the SBOM work from NTIA has shifted over to them. OWASP considers SWID to be a software identity format (which it is), and SWID is natively supported in CycloneDX because its decent at describing the identity of commercial software.

@nvoxland is checking out CycloneDX during our current sprint


How to Use.

I'm new in the Cheats World,
How to Use this.
Build in Release x86
Close Steam, and Open .exe with admin and Inject Cheat?
Only This?
The Bypass work with every Cheat? Or Only Osiris?

Correct, for security reasons use Osiris because it gets updated very often
I repeat the steps for being sure with you:

Close Steam
Compile in Release-x86
Run (as administrator ofc) and check the "initialization successful" prompt
Run CS:GO
Run your Injector as Administrator too or it won't inject for some reasons

You can use any cheat, just make sure it's a reliable cheat :).

Ok, but i need to open vac bypass every time i want to cheat or i just need to open 1 time?


Results page: Registration Date values not inline with search from to quarters

Search to and from quarter dates not inline with the values shown on Registration date on results page, for example search March 1947 to Sept 1947 shows other quarters like Jan to March 1947.

I have checked. Search March 1947 to Sept 1947 includes three quarterts (January, February, March; April, May, June; July, August, September), and the results also include information from three quarters.  So, the results and search are correct.
Mar, Jun, Sep, Dec means quarters, not months.

@richardofsussex do you agree with my comment? I think the story can be closed.

retested it works when I ask for dates between March and March, it picks up
the first quarter.
thank you
unless i missed something I want to see records between March 19
On Tuesday, December 10, 2024, KristinaGadzhieva @.***>
wrote:

@richardofsussex https://github.com/richardofsussex do you agree with
my comment? I think the story can be closed.
—
Reply to this email directly, view it on GitHub
https://github.com/FreeUKGen/FreeBMD2/issues/607#issuecomment-2532315826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDP72B656MMWPIGQIHLKOED2E4OA7AVCNFSM6AAAAABSRKHPCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZSGMYTKOBSGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes I think this story can be closed. The system is working as intended.


How to open last open project/file on restart

Hi there, amazing work on this package.
Just curious, no matter what project I am working on, when I close and re-open Atom it always seems to fallback to the same project, not the one I was last working on.
Thanks for your help!

hey @jaredreich, sorry for this, it may be related to the new versions of atom, I know they changed a bit the workflow for projects.
Just be patient and thanks for the vote of confidence.

I'm on Atom 1.12.6

hummmm, need to investigate then! :/

i can replicate this, will investigate this more today

this is getting harder to get the workaround. :\

@jaredreich this has been fixed, will run some more tests, and try to add some more features / issues to the next release!


Introduce a ConstExpr representation for arrays, simplify the array/string representation

This array representation overlaps with aggregate, and I'd like to eliminate
that, but I'd like to get this in as a precursor to the more interesting
patch.  I'll look to refactor if possible in a follow-on.

@swift-ci please test tensorflow

Looks spurious.  Will retry.


Failed to install deiver-os-monitor using databox-install-component

I had a clean install, and after running databox ( which works well ), I tired to install deiver-os-monitor  using databox-install-component script, I've got this error:
dyn903-206:databox malekzadeh$ ./databox-install-component me-box/driver-os-monitor databoxsystems 0.5.0
[2018-10-02 13:10:52 ]: Getting the code.....
Cloning into 'driver-os-monitor'...
remote: Enumerating objects: 17, done.
remote: Counting objects: 100% (17/17), done.
remote: Compressing objects: 100% (12/12), done.
remote: Total 190 (delta 4), reused 15 (delta 3), pack-reused 173
Receiving objects: 100% (190/190), 80.63 KiB | 0 bytes/s, done.
Resolving deltas: 100% (92/92), done.
[2018-10-02 13:10:53 ]: Pulling the code from https://github.com/me-box/driver-os-monitor.git

  OK
[2018-10-02 13:10:53 ]: Starting build driver-os-monitor ...
invalid argument "/driver-os-monitor-amd64:" for "-t, --tag" flag: invalid reference format
See 'docker build --help'.
[2018-10-02 13:10:53 ]: Build driver-os-monitor
ERROR: 125 FAILED
dyn903-206:databox malekzadeh$

dyn903-206:databox malekzadeh$ ./databox-install-component me-box/driver-os-monitor databoxsystems 0.5.0
[2018-10-02 13:13:43 ]: driver-os-monitor already installed, rebuilding and reseeding
[2018-10-02 13:13:43 ]: Pulling Changes driver-os-monitor ...
[2018-10-02 13:13:44 ]: Git pull driver-os-monitor OK
[2018-10-02 13:13:44 ]: Starting build driver-os-monitor ...
The command '/bin/sh -c npm install --production && npm run clean' returned a non-zero code: 1
[2018-10-02 13:14:23 ]: Build driver-os-monitor
ERROR: 1 FAILED
dyn903-206:databox malekzadeh$

OS: Mac OS X 10.13.6
Docker: 18.06.1-ce

Should be fixed now, thanks for letting me know 👍


update e2e AMI

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm


Quickstart transformation miscategorizes bill line

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
I am working with a customer to bring in Quickbooks data by GL.
I see a transaction the GENERAL_LEDGER transformation as being associated with Account 7410, but in Quickbooks it is clearly classified as 7620.
I believe this is because the inference of the account via the item number associated with the bill line is not happening or is happening incorrectly. The Bill itself is associated with account 7410, but the Bill Line in question is a specific item that should be categorized as 7620.
Relevant error log or model output
As described above, there is a diff where more transactions that expected are being categorized in account 7410, since bill lines that are directly associated with an account are not being categorized as such

Expected behavior
Use the ACCOUNT_EXPENSE_ACCOUNT_ID column on Bill Line table to associate bill line items with accounts as part of the transformations, instead of only relying on the parent bill. line items with direct associations to accounts should override their parent bill.
dbt Project configurations
I'm using Fivetran's quickstart transformations for Quickbooks
Package versions
I'm using Fivetran's quickstart transformations for Quickbooks
What database are you using dbt with?
snowflake
dbt Version
I'm using Fivetran's quickstart transformations for Quickbooks
Additional Context
No response
Are you willing to open a PR to help address this issue?

[ ] Yes.
[X] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Hi @rishi-tripathy thank you for opening the issue--you bring up a very interesting case. I will be looking into this in the next week and will let you know if I have any questions!

Hi @rishi-tripathy, following up to my last message, are you seeing this issue in the credit or is this issue with the debit transactions?

Hi @rishi-tripathy have you had a chance to check into this? Thanks!

Ah sorry for the delay here Cat! Notifications were piping to my personal email instead of my work one. I'm seeing the impact on debit items in this case. What's the implication of handing them separately?

Thanks @rishi-tripathy ! This is useful to narrow down where the issue might be. I had actually suspected credits, so it's very interesting the issue lies with debits. For debit transactions, we already use account_expense_account_id as the first option to determine account_id for debits (see this line in our model), so I will have to investigate a bit more.

Hi @rishi-tripathy. I discussed this issue with my team, and it would be really helpful if you could show us live what is happening. Would you be available for a call in the next week? If so, please send an email to<EMAIL_ADDRESS>with your availability, and I'll set up the meet!

Hi @rishi-tripathy just following up if you would be available for a call with our team?

Yes, can do cc @@.*** - how is this ( https://calendly.com/r-tripathy/30 ) availability?
Rishi Tripathy | He, Him, His
Twitter ( https://twitter.com/rishi_tripathy_ )
609-651-2559
On Wed, Nov 15, 2023 at 12:56 PM, fivetran-catfritz < @.*** > wrote:

Hi @ rishi-tripathy ( https://github.com/rishi-tripathy ) just following up
if you would be available for a call with our team?
—
Reply to this email directly, view it on GitHub (
https://github.com/fivetran/dbt_quickbooks/issues/105#issuecomment-1813001450
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AJRJLWASOEWU2JXA5FML5FLYET65ZAVCNFSM6AAAAAA6DCEPE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJTGAYDCNBVGA
).
You are receiving this because you were mentioned. Message ID: <fivetran/dbt_quickbooks/issues/105/1813001450
@ github. com>

Thanks @rishi-tripathy! I have scheduled a meeting.

Great! Good timing as well - I just saw this issue pop up with another customer account.
The issue this time is with debit transaction types in credit account types. There are transactions included in the model (they're associated with the account number - in this case 4802) that should not be (i.e. they are not showing up as associated with that account in the customer's Quickbooks instance).
Happy to show you more live on our call tomorrow.
If you have a chance to poke around, two affected transactions from the QBO general_ledger transformation can be found by just filtering the entire transform model (org id is 709) by
adjusted_amount =-811 or =-292.40.
Rishi Tripathy | He, Him, His
Twitter ( https://twitter.com/rishi_tripathy_ )
609-651-2559
On Tue, Nov 21, 2023 at 6:27 PM, fivetran-catfritz < @.*** > wrote:

Thanks @ rishi-tripathy ( https://github.com/rishi-tripathy ) ! I have
scheduled a meeting.
—
Reply to this email directly, view it on GitHub (
https://github.com/fivetran/dbt_quickbooks/issues/105#issuecomment-1821856257
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AJRJLWEKAYNV5M5Z4O6BPDTYFU2GNAVCNFSM6AAAAAA6DCEPE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRRHA2TMMRVG4
).
You are receiving this because you were mentioned. Message ID: <fivetran/dbt_quickbooks/issues/105/1821856257
@ github. com>

In this second case (Would need to go back to confirm for the other case) — I can confirm that:

the bill line in quickbooks.bill is associated with the same account (using account_expense_account_id) that it shows up in the model — though this is different than how it's categorized in the customer account. Asked the customer for more context on why this might be the case.

Hi @rishi-tripathy thanks for meeting with us on the call yesterday to go over this issue! To summarize what we found:

The 7410 instead of 7620 account was associated with the bill line in question since in the source bill_line table, the account_expense_account_id did not have a value populated.
This causes the coalesce clause in this line to move on to another field to determine the account to use.

Next steps:

During the call we were unable to establish where the association of 7620 was coming from, so @rishi-tripathy will determine where that path is coming from.

Keep us posted with what you find!


ld: warning: -no_pie is deprecated when targeting new OS versions

I saw this message on the CI.
https://github.com/Code-Hex/vz/actions/runs/3247329051
There is a workaround solution: https://github.com/golang/go/issues/54482#issuecomment-1251124908

As far as I understand it's just a warning, and it will be fixed in newer go versions, not sure it's really useful to add the workaround for a matter of weeks

@cfergeau There is no supported for 1.17.x

Resolved as the latest version has been released.
https://groups.google.com/g/golang-announce/c/dRtDK7WS78g


Add formspec

This commit adds a formspec and fixes a minor bug.

I'm looking at this now sorry for the wait. I do want to point out that (according to github) the "Fix undeclared global" commit changed nothing. I assume that's an accident.

TeTpaAka  answered on the forum topic so I'll just paste here the relevant info for documentation:
1 Accident.
2 I think, this is due to me simply adding the gui to the existing display function. Or is this something different? I didn't add any chat messages to the mod.
3 Me neither. It happens occasionally for me, too. It's the same with my other mods, but I can't do anything about it. The function gets called, but the formspec simply doesn't open.
I have merged this pull request and will push it to the github repo soon. Again thanks for this TeTpaAka formspec looks really nice.


change script's current_dir to 'android-constraint-layout-performance'

The project root directory name is 'android-constraint-layout-performance'. So I think the script's current_dir should be same with the project name.

The change looks LGTM. Could you sign the CLA so that we can merge it?

I signed it!


Feature/775

Fixes #775
https://github.com/airswap/airswap-web/assets/86911296/fad8a96b-9a5b-40f2-b23c-02bc57e4ac5d
When choosing ETH / WETH use gets the right amount in metamask he needs to wrap:

Would it be a better UX if these displayed the token logo instead of a blank circle?

Would it be a better UX if these displayed the token logo instead of a blank circle?

That's just goerli that has missing token images.


Create a GetSchema Operation

This operation should be capable of returning either the original full schema (currently held in the Graph class) or the compact schema held in the store class. To determine which to return a boolean field "compact" should be added. The compact field should default to false. To achieve this the schema held in the graph class may need to be moved to the Store.
The main reason for this is for use in the Federated Store. So we will need to write a special handler for this operation within the FederatedStore to return the federated schema.

I've done most of the code required on branch gh-1462-getschema. It still needs some unit tests writing.

Merged into develop.


With .inverse, all text and headings should use clr-white

Only exceptions to clr-primary-400 headings are:

a span in the h1 in hero
h3s in the herosub side blue boxes
main 'just scratching the surface' header in the footer

All links and buttons have clr-white text.   Only difference is buttons which add the background.


REPORTS - Debt Bars Showing Up Inconsistently

Defective ToolKit Feature Name:
Reports Time Filter Slider
Expected Behavior
When using a maximum date range, "Debts" consistently show as either credit card debt, or expenses for the month.
Actual Behavior

When at maximum date range, red "Debt" bars only appear next to the months in which I incurred credit card debt that was subsequently paid off.
When decreasing the earliest month to be the second earliest month (from July 2014 - Current to August 2014 to Current), red bars show up labelled as "Debts" which are my expenses for the month.
When decreasing the most recent month to the be the second most recent month (From July 2014 - August 2016 to July 2014 to July 2016), no additional red bars appear. Instead, the "Debt" bars remain as credit card debt.

Steps to Reproduce the Behavior

Navigate to the NYNAB app
Log in and ensure you have months with both expenses and credit card debt, as well as some months without a credit card account (my first 16 or so months there are no credit accounts present)
Navigate to the reports tab
Adjust the filter to include all dates
Decrease the maximum date by 1
Increase the maximum date by 1
Decrease the minimum date by 1
Increase the minimum date by 1
Decrease both the minimum and maximum date by 1

Export of Toolkit Settings (Bottom Left of Settings Page)
[{"key":"accountTransactionSearch","value":true},{"key":"accountsDisplayDensity","value":"0"},{"key":"accountsEmphasizedOutflows","value":false},{"key":"accountsSelectedTotal","value":true},{"key":"accountsStripedRows","value":false},{"key":"activityTransactionLink","value":true},{"key":"budgetBalanceToZero","value":true},{"key":"budgetProgressBars","value":"pacing"},{"key":"budgetQuickSwitch","value":false},{"key":"budgetRowsHeight","value":"2"},{"key":"categoryActivityPopupWidth","value":"0"},{"key":"changeEnterBehavior","value":false},{"key":"checkCreditBalances","value":true},{"key":"checkNumbers","value":false},{"key":"collapseExpandBudgetGroups","value":true},{"key":"collapseSideMenu","value":true},{"key":"colourBlindMode","value":false},{"key":"currentMonthIndicator","value":false},{"key":"daysOfBuffering","value":true},{"key":"daysOfBufferingHistoryLookup","value":"3"},{"key":"editButtonPosition","value":"2"},{"key":"enableRetroCalculator","value":false},{"key":"enlargeCategoriesDropdown","value":true},{"key":"enterInRegisterNow","value":true},{"key":"goalIndicator","value":false},{"key":"goalIndicatorWarningColor","value":true},{"key":"googleFontsSelector","value":"0"},{"key":"hideAOM","value":false},{"key":"highlightNegativesNegative","value":true},{"key":"importNotification","value":true},{"key":"incomeFromLastMonth","value":"0"},{"key":"l10n","value":"0"},{"key":"largerClickableIcons","value":true},{"key":"monthlyNotesPopupWidth","value":"0"},{"key":"options.dark-mode","value":true},{"key":"pacing","value":"2"},{"key":"printingImprovements","value":true},{"key":"privacyMode","value":"0"},{"key":"reconciledTextColor","value":"0"},{"key":"removePositiveHighlight","value":true},{"key":"removeZeroCategories","value":true},{"key":"reports","value":true},{"key":"resizeInspector","value":true},{"key":"rightClickToEdit","value":true},{"key":"runningBalance","value":true},{"key":"seamlessBudgetHeader","value":true},{"key":"showIntercom","value":true},{"key":"splitKeyboardShortcut","value":true},{"key":"splitTransactionAutoAdjust","value":true},{"key":"squareNegativeMode","value":true},{"key":"swapClearedFlagged","value":false},{"key":"toBeBudgetedWarning","value":true},{"key":"toggleSplits","value":true},{"key":"toggleTransactionFilters","value":"0"},{"key":"warnOnQuickBudget","value":false}]

System Settings
ToolKit Version:
Operating System (Mac/Windows/Etc): Windows 10
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: Version 52.0.2743.116 m
Other Extensions: Lastpass

@Frostcrest We have a hotfix just about to go out for this one :D

This is now released as part of v0.7.1. Any better?

@joshmadewell @blargity
Yes, thank you! A little late to the party, I guess. Keep up the wonderful work!


docs: devstack settings

Description:
Plugins may have a "devstack" setting, and this was not documented (as far as I know). I was bitten by this as I was trying to override settings in a plugin, and my settings were being overridden by devstack settings.

Thanks for the pull request, @regisb!
What's next?
Please work through the following steps to get your changes ready for engineering review:
:radio_button: Get product approval
If you haven't already, check this list to see if your contribution needs to go through the product review process.

If it does, you'll need to submit a product proposal for your contribution, and have it reviewed by the Product Working Group.

This process (including the steps you'll need to take) is documented here.

If it doesn't, simply proceed with the next step.

:radio_button: Provide context
To help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:

Dependencies

This PR must be merged before / after / at the same time as ...

Blockers

This PR is waiting for OEP-1234 to be accepted.

Timeline information

This PR must be merged by XX date because ...

Partner information

This is for a course on edx.org.

Supporting documentation
Relevant Open edX discussion forum threads

:radio_button: Get a green build
If one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green.
:radio_button: Let us know that your PR is ready for review:
Who will review my changes?
This repository is currently maintained by @openedx/2u-arch-bom.  Tag them in a comment and let them know that your changes are ready for review.
Where can I find more information?
If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources:

Overview of Review Process for Community Contributions
Pull Request Status Guide
Making changes to your pull request

When can I expect my changes to be merged?
Our goal is to get community contributions seen and reviewed as efficiently as possible.
However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:

The size and impact of the changes that it introduces
The need for product review
Maintenance status of the parent repository

:bulb: As a result it may take up to several weeks or months to complete a review and merge your PR.

Thanks @regisb. I’ll take this to our teams because:

I’m not aware of us needing to do this for our plugin, and maybe that is just personal lack of awareness and maybe there are limited circumstances where this is required?
2U is helping with a DEPR around the devstack.py files. See this relevant comment: https://github.com/openedx/public-engineering/issues/247#issuecomment-2344711918. It would be nice to understand how this all fits in with the plan.

That said, I understand your intent is simply to get the docs in line with reality as a step.

I’m not aware of us needing to do this for our plugin, and maybe that is just personal lack of awareness and maybe there are limited circumstances where this is required?

We are currently working on a plugin (the new forum) that needs to override the COMMENTS_SERVICE_URL -- including in development. I did add the updated setting in common.py, but these values are immediately overridden by the values from edx-platform's devstack.py. A quick search revealed that the devstack settings are already used in the event routing backend (here).

2U is helping with a DEPR around the devstack.py files. See this relevant comment: https://github.com/openedx/public-engineering/issues/247#issuecomment-2344711918. It would be nice to understand how this all fits in with the plan.

I wasn't aware of that. I'll comment there.

Hi @robrap - any update on this?

@regisb: What is the status of this PR? Do you know how it relates to any changes that might be happening in terms of "devstack" => "development", or do we need to bring in Kyle? Thank you.

I am not familiar with the recent updates, so I'm not in the best position to comment. AFAIK this documentation change is still relevant.

@kdmccormick: Is this ready to land as-is? Will it need to be updated in the future? Thank you.

We had planned to replace devstack.py with a simpler, YAML-free development.py. But, we found that Tutor's use of devstack.py is tightly coupled to production.py and the YAML-based settings schema. Replacing devstack.py without also replacing production.py would actually increase the overall complexity of the system. So, we are going to leave it as-is for now, and then introduce simpler YAML-free production+development settings all at once further down the road.
In short-- we will replace devstack.py eventually, but not in the immediate future. It would be reasonable to add this "devstack" slot to Django App Plugins now, with the understanding that we'll eventually need to deprecate it and replace it with a "development" slot.


[5.2] ResourceRegistrar where patterns

Add ability to define where patterns on resource parameters.
Route::resource('test', 'TestController', ['where' => ['test' => '[0-9]+']]);

In reference to Issue #11945

I would suggest just using Route::pattern


Make validation error messages more friendly

We don't want to sound scary.

This issue has been moved to dart-lang/pub#349.


Import BiLouvain raises error

It seems like sknetwork.clustering doesn't support BiLouvain clustering methods. Could you please suggest a new clustering method? Thanks.

here:
0.24.0 (2021-07-27)
Merge Bi* algorithms (e.g., BiLouvain -> Louvain) by Thomas Bonald (#490)
Transition from Travis to Github actions by Quentin Lutz (#488)
Added sdist build for conda recipes
Added name position for graph visualization
Removed randomized algorithms
should be Louvain

Good looking out! It needed a couple of changes to account for the updates to Sci-Kit Network; change to Louvain as well as change in how modularity is calculated.
It should be good now, and please let me know if its not!
Also, I'll get a tutorial on how to use the code into the Repo soon as well.


Make DualAxis::right_stick() use the correct y-axis

It was incorrectly using the y-axis from the left stick.

Thank you very much!


nxstyle: "Operator/assignment must be preceded with whitespace"

nxstyle doesn't like the following line in apps/examples/sotest/sotest_main.c
  testfunc = (CODE void (*)(FAR const char *))dlsym(handle2, "testfunc1");

to me, the style seems perfectly fine.

nxstyle doesn't like the following line in apps/examples/sotest/sotest_main.c
  testfunc = (CODE void (*)(FAR const char *))dlsym(handle2, "testfunc1");

to me, the style seems perfectly fine.

I guess nxstyle like this:
 testfunc = (CODE void ( * )(FAR const char *))dlsym(handle2, "testfunc1");

@xiaoxiang781216
actually it doesn't.
error: Space follows left parenthesis
error: Space precedes right parenthesis

Seems to be fixed by https://github.com/apache/incubator-nuttx/pull/4880

So, let's close this issue.


Service Create Selectors - Show Target Pods Real Time

Adds a banner message similar to Fleet Cluster Groups that shows the live pod count for the selectors section of service create.
Adds required to External Name input.
rancher/dashboard#717
rancher/dashboard#2179

@vincent99 updated per your comments, thanks!

@vincent99 updated per your comments, thanks!


Using php keywords breaks fpp

E.g. when you want to use php keywords like "new" or "class" breaks fpp:
e.g.
data Bar = Old | New deriving (Enum);

results in:
Parse Error: Syntax error, unexpected 'New', expecting 'T_STRING' at line 5 on file 'foo.fpp'

ping @lunetics

I close this, if you still feel that there is something I should do about it, please reopen and leave your arguments.


SOLID-274 add support for .button:disabled

add :disabled everywhere the .button--disabled class is styled

this lgtm

This lgtm too! 👍


two small grammar changes

@elipousson what do you think of the proposed changes? This is a great project, by the way!

Thanks so much for these corrections! I'm relatively new to using GitHub so this is actually my first time merging any pull request other than my own. 😀 Look out for an email from me soon.


Duplicate implementations of punionWith

punionWith appears in both Plutarch itself and LPE (for both sorted and unsorted varieties).

Actually, I'm realizing that there's duplicate functions in plutarch itself (one in Value and one in Assoc map). Just going to leave this issue up as a reminder to check whether any of our instances duplicate their instances.

This seems to already be resolved: we don't have punionWith in LPE. Closing.


OpenCost Plugin For MongoDB Atlas

Is your feature request related to a problem? Please describe.
Add support for MongoDB's cloud product, Atlas
Describe the solution you'd like
Very basic support for atlas costs
Describe alternatives you've considered
N/A
Additional context
This issue tracks all work related to MongoDB Atlas plugins

implemented by #41


fix shown property of the resource security group rule is error.

The value of property port_range_min is error.

I will fix it.


Fix error in clusterMaxZoomLevel on new arch

Description
On new arch when the map re-renders it causes the following error when using the clusterMaxZoomLevel prop. This is because this prop cannot be updated in mapbox-ios. This isn't a problem in the old architecture since the prop handler is only called when the prop changes, in the new arch it needs to be handled manually by comparing the props values.
Mapbox [error] RNMBXShapeSource | clusterMaxZoomLevel  Cannot set property clusterMaxZoom for the source symbolLocationSource

After this the error no longer appears, unless actually changing the value of clusterMaxZoom.
It might be a good pattern to adopt generally for all components and props, but I thought it was out of scope of this since it would become a pretty big refactor.
Tested this change in an app that makes use of clustering.
Checklist

[x] I've read CONTRIBUTING.md
[x] I updated the doc/other generated code with running yarn generate in the root folder
[ ] I have tested the new feature on /example app.

[ ] In V11 mode/ios
[ ] In New Architecture mode/ios
[ ] In V11 mode/android
[ ] In New Architecture mode/android

[ ] I added/updated a sample - if a new feature was implemented (/example)

Screenshot OR Video

Component to reproduce the issue you're fixing

@janicduplessis thanks that is great.
One approach I was experimenting with is:
https://github.com/rnmapbox/maps/blob/b74fc6f77f332b00ca84b537bdb55b4ff41de526/ios/RNMBX/RNMBXNativeUserLocationComponentView.mm#L60-L67
Those macros are defined here:
https://github.com/rnmapbox/maps/blob/main/ios/RNMBX/RNMBXFabricPropConvert.h
What do you think?

Looks good, I've seen this approach used too in some components in react-native core. I will update RNMBXShapeSourceComponentView to use those.

@mfazekas Updated to use macros


[Possible Improvement] Check for SSL in sitemap

Hello Stephen and Mezzanine community,
It's a pleasure to greet you again.
I was reading about the Mezzanine sitemap and its code, and I ended up in Django's docs. There is an attribute called protocol which you can set to indicate if it uses http or https in the generation of the sitemap.
Currently, as far as I've checked, Mezzanine doesn't set in any way that attribute. If you set SSL_ENABLED as True so you can tell Mezzanine that you're using a secured protocol, the class DisplayableSitemap should check for it, with something like this (the code I edited for my project is mezzanine/core/sitemaps.py):
class DisplayableSitemap(Sitemap):
    """
    Sitemap class for Django's sitemaps framework that returns
    all published items for models that subclass ``Displayable``.
    """

    # Check if SSL is enabled so we force the sitemap to show URLs with HTTPS
    if settings.SSL_ENABLED:
        protocol = 'https'
    else:
        protocol = None

    def items(self):
        """
        Return all published items for models that subclass
        ``Displayable``, excluding those that point to external sites.
        """
        return list(Displayable.objects.url_map(in_sitemap=True).values())

    def lastmod(self, obj):
        if blog_installed and isinstance(obj, BlogPost):
            return obj.updated or obj.publish_date

    def get_urls(self, **kwargs):
        """
        Ensure the correct host by injecting the current site. Also, we set the
        correct protocol in case SSL_ENABLED is set to True
        """
        kwargs["site"] = Site.objects.get(id=current_site_id())
        kwargs["protocol"] = self.protocol
        return super(DisplayableSitemap, self).get_urls(**kwargs)

According to the docs (https://docs.djangoproject.com/en/1.11/_modules/django/contrib/sitemaps/), if protocol is None it goes straight to http, but if it is not None (the other option, according to https://docs.djangoproject.com/en/1.11/ref/contrib/sitemaps/#django.contrib.sitemaps.Sitemap.protocol is https), it goes to self.protocol, which means https can come into play in the URLs definition of the sitemap.
It's a small thing, but I believe it's a good thing to add because I think it helps with SEO.
What do you think?
Regards,
Melvin

Hi Melvin,
Looking at two mezzanine sites' sitemap.xml files:

http://mezzanine.jupo.org/sitemap.xml
https://bldg92.org/sitemap.xml

It seems the protocol is honored without any extra work on the part of the developer. You don't even need to set SSL_ENABLED = True.

I can confirm on the demo site linked above, which has a self-signed cert and can be accessed over HTTPS also - the sitemap uses whichever protocol is being used to access it.


Sheets imps

appendResults and validation stuff moved into separate Class, some tests

@paulirish what do you think about using TypeScript in future? It hurts a bit to check all arguments for each method.

@paulirish what do you think about using TypeScript in future? It hurts a bit to check all arguments for each method.

yup i'd be happy to move to TS

this PR looks great. sorry for letting it sit.

No problem at all :)
Do you wanna merge it? Looks like smth wrong with gulp-example which is run with Travis...

Fixed via https://github.com/paulirish/pwmetrics/commit/b3d2fc91b45c62266bb373a32046506789f2c57e


GH900: Fix AnyCPU platform target when building a project

Fixes #900

@thomaslevesque Looks good to me! Can you rebase against develop and I'll merge as soon as I get an opportunity.

Can you rebase against develop

Done

@thomaslevesque apologies!  Can you rebase one more time, and I will get this merged.  Thanks!

@thomaslevesque apologies! Can you rebase one more time, and I will get this merged. Thanks!

Sure, not problem. Done ;)

@patriksvensson @devlead  any thoughts on why the Travis build failed here?

@gep13 Hmm. Maybe old Cake is cached for some reason. Not sure why that fails. I think that error was fixed. @devlead What do you think?

@gep13 @patriksvensson I've cleared all caches.

Now the build seems to be stuck in a loop... it keeps restarting from zero, reinstalling Mono every time. Package restoration is taking forever (very long response times from MyGet)

@thomaslevesque I've restarted the build, NuGet seems especially cranky on Travis Linux today 😉

@thomaslevesque thank you for your contribution to Cake! :+1:


Alon High School Init

Adding Alon High school to access Student' license.

@Atalyeu First stage of verification passed, official domain is il.linkedin.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://il.linkedin.com/company/municipality-of-ramat-hasharon

@Atalyeu First stage of verification passed, official domain is il.linkedin.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://il.linkedin.com/company/municipality-of-ramat-hasharon

Hello when do you think I will get the final verfication ?

@Atalyeu Regrettably, this request cannot be satisfied. The domain ramhash.co.il does not belong to the school, it belongs to the municipality, and not only students and teachers have inboxes in this domain.
Please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
I have to close the request without merging. Thank you for your understanding!


acknowledge licensing issues in AbstractNodeQueue

https://github.com/apache/incubator-pekko/blob/532b7173c175c0c379074cb022e98f5002860c22/actor/src/main/java/org/apache/pekko/dispatch/AbstractNodeQueue.java#L21-L26

@jrudolph @mdedetrich  could the text in https://github.com/apache/incubator-pekko/pull/433 be adapted to include this too? AbstractNodeQueue does not have any inspiration from Andriy Plokhotnyuk's code but it is inspired by the MPSC code.

@jrudolph @mdedetrich could the text in #433 be adapted to include this too? AbstractNodeQueue does not have any inspiration from Andriy Plokhotnyuk's code but it is inspired by the MPSC code.

So I just had a look at the referenced code at https://www.1024cores.net/home/lock-free-algorithms/queues/non-intrusive-mpsc-node-based-queue and this does look like its a similar case, i.e. the code is in a different language (C) and is minimal compared to AbstractNodeQueue.

Closing as completed, will re-open if more needs to be done.

@mdedetrich I think it is better if we modify the text from #433 to also mention 'AbstractNodeQueue.java'.

#434 merged