[Bug] Discovery of k8s service when using AWS ELB

What happened: When using k8s service discovery in clusters running on AWS, discovery of public IP of LoadBalancer services fails
What you expected to happen: I am able to use AWS load-balancers as targets by all standard probes
How to reproduce it (as minimally and precisely as possible): create a k8s cluster with AWS as a cloud-provider. Create a LoadBalancer service and an RDS deployment that can discover it. Connect cloudprober to the RDS and attempt to probe it. The following is observed in the logs
I0622 18:43:12.014801 3299756 prober.go:309] Starting probe: pod-to-endpoints
W0622 18:43:14.015205 3299756 ping.go:214] Bad target: cloudprober. Err: no IP address for the resource: cloudprober
I0622 18:43:14.015270 3299756 ping.go:268] Skipping unresolved target: cloudprober
I0622 18:43:14.040403 3299756 ping.go:268] Skipping unresolved target: cloudprober

Anything else we need to know?: this is caused by the fact that AWS ELB returns a hostname (cname/alias) instead of IP, which is what gets populated in a service's status field. Currently, RDS code assumes that svc.Status.LoadBalancer.Ingress can only contain an IP, whereas hostname is also a valid option and is what gets populated by the AWS cloud-provider code.

I've implemented a naive fix like this:
@@ -93,7 +93,12 @@ func (lister *servicesLister) listResources(req *pb.ListResourcesRequest) ([]*pb
                        if len(svc.Status.LoadBalancer.Ingress) == 0 {
                                continue
                        }
-                       res.Ip = proto.String(svc.Status.LoadBalancer.Ingress[0].IP)
+                       if svc.Status.LoadBalancer.Ingress[0].IP != "" {
+                               res.Ip = proto.String(svc.Status.LoadBalancer.Ingress[0].IP)
+                       } else if svc.Status.LoadBalancer.Ingress[0].Hostname != "" {
+                               res.Name = proto.String(svc.Status.LoadBalancer.Ingress[0].Hostname)
+                       }
+
                } else {
                        res.Ip = proto.String(svc.Spec.ClusterIP)
                }
@@ -117,7 +122,8 @@ type serviceInfo struct {
        Status struct {
                LoadBalancer struct {
                        Ingress []struct {
-                               IP string
+                               IP       string
+                               Hostname string
                        }
                }
        }

However this feels like the wrong approach as I'm overwriting/hiding the real name of the resource. We also can't use hostname since RDS targets do not have an option to use local/client resolver and will try to resolve hostnames in local cache (if I understood your code correctly).
So it seems like the way forward is to do a resolution on the RDS server? (bearing in mind that AWS's ELB returns 2 public IPs).

@networkop Thanks a lot for reporting this bug.
You're right about this not being straightforward. There are a few possible approaches:

Handle this as a special case and return hostname as resource name.
Add another field "hostname" to RDS resource, and on the client use local resolver if there is a hostname in the resource but no IP.
Since RDS resource's IP field is a string, return hostname in that field itself and handle the distinction on the client side. This doesn't seem that crazy after seeing the AWS's documentation: https://aws.amazon.com/premiumsupport/knowledge-center/eks-kubernetes-services-cluster/; towards the end you'll see that kubectl shows hostname as external IP.
Resolve hostname on the RDS server side itself.

(2), and, to a certain extent (3), seem like the right approach. (2) will require more careful thought as it's changing the protobuf message.
(1) and (3) are more like workarounds for now.
Let me give it a bit more thought and get back on this.

bearing in mind that AWS's ELB returns 2 public IPs.

Are these IPv4 and IPv6?

Since RDS resource's IP field is a string, return hostname in that field itself and handle the distinction on the client side. This doesn't seem that crazy after seeing the AWS's documentation: https://aws.amazon.com/premiumsupport/knowledge-center/eks-kubernetes-services-cluster/; towards the end you'll see that kubectl shows hostname as external IP.

FYI, I am working on the client side changes to make this option work. So far this seems like a reasonable thing to do.

great, thanks. there's just one concern I've got with this approach. Every load-balancer service will also have a unique spec.clusterIP in addition to what's reported in the status field. If I understand it correctly, this IP will always be hidden by the status.loadbalancer.ingress, even now (before this issue gets fixed).
However, if we consider option #2 and extend the RDS resource, this would make it a bit cleaner. what do you think? For example:
message RDSTargets {
  repeated k8s.loadbalancer.ingress ingresses = 5;
}

Of course, this implies changing the schema and much more code compared to option 3 :man_shrugging:

spec.ClusterIP is in fact the default IP you'll get. To get the ingress IP, you've to specify the ip_type as PUBLIC:
https://github.com/google/cloudprober/blob/c003b5cbd7db790dfd346bfaf33643234ce0d027/rds/kubernetes/services.go#L152
rds_targets {
  ...
  ip_config {
    ip_type: PUBLIC
  }
}

Did I get your question correctly.

ah yes, you're right, my apologies.

Closing this. It will get released with v0.10.8:
https://github.com/google/cloudprober/milestone/6


Command Line Continuous Integration Placeholders

Hi there,
I am looking to use tabular editor via the command line to set up on continuous deployment pipeline using azure devops.
I understand pretty well what I need to do apart from the how to update the data source using place holders as outlined in the wiki. I understand that I could put %database%, %server% etc in the model.bim but I'm not clear then what the development workflow is. I.e. if I am looking to continue to use visual studio how do I set %database% and %server% locally so that I can process, get new tables etc.
Any help appreciated and thanks for efforts on this. Looks great.

I generally recommend not to put a placeholder value in the connection strings within your Model.bim file. The reason is that Analysis Services will mask the entire connection string (*********), if it determines that it contains sensitive information OR if it can't parse the string (better safe than sorry). Once you add a placeholder to a connection string, it is likely that Analysis Services won't be able to parse it, and therefore, it will change the full string to ********** when you save the model to a .bim file. At this point, swapping placeholder values for their proper values using the Tabular Editor command line will of course no longer work... Also, as you've observed, working with Visual Studio to process tables will also not work properly, when you have placeholders in the connection string.
What I recommend, is to leave a valid connection string (without any placeholders) in your Model.bim file. This will ensure that you can still use Visual Studio to process the model just fine. In your deployment pipeline, you will use the -S switch with TabularEditor.exe, to change the entire connection string of your data source into a placeholder value. In the same command, you will also use the -D, -O and -C switches to actually do the deployment, over write data sources AND swap placeholders with proper connection strings, which you populate using variables in your Azure DevOps pipeline.
First, put the following script into a new file, ClearConnectionStrings.cs and add the file to source control:
// This will replace the connection string of all Provider (legacy) data sources in the model
// with a placeholder based on the name of the data source. E.g., if your data source is called
// "SQLDW", the connection string after running this script would be "SQLDW":

foreach(var ds in Model.DataSources.OfType<ProviderDataSource>())
    ds.ConnectionString = ds.Name;

In your DevOps pipeline, use the following TabularEditor.exe command, to execute the script above and then replace the placeholders with proper values before deployment:
TabularEditor.exe "Model.bim" -S ClearConnectionStrings.cs -D "$(TargetServer)" "$(TargetDatabase)" -O -C "SQLDW" "$(SQLDWConnectionString)" -W -V -E

In the example above, I assume your model contains just a single data source called "SQLDW". At the same time, your pipeline should have at minimum 3 variables:

TargetServer: The Analysis Services instance that you want to deploy to
TargetDatabase: The ID of the database to be deployed
SQLDWConnectionString: The actual connection string to use for the "SQLDW" data source within the model. Mark this as sensitive, if it contains username and password.

This way, you're only applying the proper connection string just-in-time, before deploying the model, without saving the connection string as plaintext anywhere - the only place it's stored, is as an Azure DevOps variable. For a release pipeline, you can even have multiple values for the same variable, depending on the stage - so for example, one connection string for UAT, one for PROD, etc...

what about if we have multiple data sources , where we need to update. How to change the Impersonation type and give username and password for that

You may also use a script to update properties on specific data sources, like so:
// Assuming your model contains a provider (legacy) data source called SQLSrc:
var ds1 = (ProviderDataSource)Model.DataSources["SQLSrc"];
ds1.ImpersonationMode = ImpersonationMode.ImpersonateAccount;
ds1.Account = Environment.GetEnvironmentVariable("SQLSrcUsername");
ds1.Password = Environment.GetEnvironmentVariable("SQLSrcPassword");

// Assuming your model contains a Power Query (structured) data source called PQSrc:
var ds2 = (StructuredDataSource)Model.DataSources["PQSrc"];
ds2.AuthenticationKind = "UsernamePassword";
ds2.Username = Environment.GetEnvironmentVariable("PQUsername");
ds2.Password = Environment.GetEnvironmentVariable("PQPassword");

Here, I use Environment.GetEnvironmentVariable to assign the data source properties from environment variables. You can pass a DevOps pipeline variable into an environment variable as shown here. This is to avoid saving sensitive data in the script, and this solution is easier to manage than the one provided in my original answer above. So in other words, no need to use the connection string placeholder replacement options (after the -C switch) when calling TabularEditor.exe from the command line.
If you have many data sources that needs to be updated to use the same values, you can also just loop over them:
foreach(var ds in Model.DataSources.OfType<ProviderDataSource>())
{
    ds1.ImpersonationMode = ImpersonationMode.ImpersonateAccount;
    ds1.Account = Environment.GetEnvironmentVariable("DataSourceUsername");
    ds1.Password = Environment.GetEnvironmentVariable("DataSourcePassword");
}

in the wiki , it is mentioned that "Remember that for this to work, the agent running the build pipeline must be able to reach the data source." our datasources are from onpremises , will it work with is method

without doing the schema check , is it possible to use   the .cs file to get the datasource values
-s setconnectionstringfromenv.cs
foreach(var dataSource in Model.DataSources.OfType())
{
var evName = dataSource.Name.Replace(" ", "") + "ConnectionStringDev";
var evValue = Environment.GetEnvironmentVariable(evName);
if (evValue != null)
dataSource.ConnectionString = evValue;
dataSource.ImpersonationMode = ImpersonationMode.ImpersonateAccount;
dataSource.Account = Environment.GetEnvironmentVariable("ServiceAccountUsername");
dataSource.Password = Environment.GetEnvironmentVariable("ServiceAccountPassword");
}

I have updated the commandline task  and given below command and also given environment variables
TabularEditor.exe "$(Build.SourcesDirectory)/Tabular/AsqQuantificationTabular/AsqQuantificationTabular/Model.bim"  -S "$(Build.SourcesDirectory)/Tabular/AsqQuantificationTabular/AsqQuantificationTabular/config/Azure-cube-dev/SetConnectionStringFromEnv.cs  -D "$(ASConnectionString-dev)" "$(ASDatabase)" -O -C
but I am getting the fallowing error
##[error]Cmd.exe exited with code '1'

when I have command line
TabularEditor.exe "$(Build.SourcesDirectory)/Tabular/AsqQuantificationTabular/AsqQuantificationTabular/Model.bim"  -D "$(ASConnectionString-dev)" "$(ASDatabase)" -O -C "$(Build.SourcesDirectory)/Tabular/AsqQuantificationTabular/AsqQuantificationTabular/config/Azure-cube-dev/SetConnectionStringFromEnv.cs
the deployment is happening but the datasources are not changing .

when we have given like this
TabularEditor.exe "$(Build.SourcesDirectory)/Tabular/AsqQuantificationTabular/AsqQuantificationTabular/Model.bim"  -S  "$(Build.SourcesDirectory)/Tabular/AsqQuantificationTabular/AsqQuantificationTabular/config/Azure-cube-dev/SetConnectionStringFromEnv.cs" -D "$(ASConnectionString-dev)" "$(ASDatabase)" -O -C
it is working fine , but how can we link the serviceaccount password from keyvault.

Linking a password from key vault is outside the scope of Tabular Editor. You should set up a variable group in Azure DevOps that connects to the key vault. Here’s an article with more details on how to do that: https://zimmergren.net/using-azure-key-vault-secrets-from-azure-devops-pipeline/
Please open a new GitHub issue if you have any further questions.

Hi,
I am trying to run TabularEditor in Azure DevOps with switches -S , -C, -O and -D. I need to deplopy same Model.bim in 2 different environments. however, we want to change the DataSource for each environment. Eventhough deployment works , it does not change the ConnectionString in DataSources. I tried using -B switch to generate the updated .bim file, however, I don't see the affect of using -S and -C switrches. Please help here. Here is the masked Datasources from model.bim :
"dataSources": [
{
"type": "structured",
"name": "Some Name for Data Source",
"connectionDetails": {
"protocol": "tds",
"address": {
"server": "Azure SQL Server FQDN",
"database": "Azure SQL DW Database"
},
"authentication": null,
"query": null
},
"credential": {
"AuthenticationKind": "OAuth2",
"kind": "SQL",
"path": "Azure SQL Server FQDN;Azure SQL DW Database",
"Expires": "Mon, 12 Jul 2021 16:40:35 GMT",
"RefreshToken": "********"
}
}
],
with -C Switch I am trying to update the SQL Server and DW Database names using a valid connection string.


Quick Start is broken

Check CONTRIBUTING guideline first and here is the list to help us investigate the problem.
Describe the bug

Quick Start is broken at git master.
out_forward with @id may not work.
It works on v1.7.3.
To Reproduce

$ mkdir tmp
$ bin/fluentd -s tmp
$ bin/fluentd -c tmp/fluent.conf

Expected behavior

Fluentd works.
Your Environment

Fluentd or td-agent version: fluentd --version or td-agent --version
git master b1bd1dab520f4a4592e96e4ba64908a4e6aa6c65
Operating system: cat /etc/os-release
Ubuntu 18.04

NAME="Ubuntu"
VERSION="18.04.3 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.3 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Kernel version: uname -r
5.0.0-31-generic

If you hit the problem with older fluentd version, try latest version first.
 Check whole configuration and fix it."
$

Additional context
It works on v1.7.3.
#2541 seems to break Quick Start.

Thanks. I'll check it next Monday!

this issue has been fixed in https://github.com/fluent/fluentd/pull/2661 . Thank you for the report!

It works!
Thank you for fixing.


Low-latency storage backends

So far we've only used cloud storage (S3 and GCS) for storing intermediate Zarr data in Cubed. It would be interesting to try other storage backends that have lower latency.
Here are some examples, I'm sure there are more:

Google Cloud Memorystore
AWS Lambda and EFS
Modal Shared Volumes

Note that Google Cloud Memorystore is not serverless, so you'd need to start a cluster before running your computation, and shut it down afterwards.
We'd also want to be more aggressive in deleting intermediate data that is no longer needed in Cubed. Currently we just let it get deleted automatically in the background, well after the computation has completed. Low-latency storage has smaller capacity and is a lot more expensive, so we'd want to delete data that is no longer a direct dependency of any node in the DAG immediately as the computation progresses.
cc @TomNicholas @rabernat

This would be very interesting.

Low-latency storage has smaller capacity

I would be concerned about the maximum size of the low-latency storage limiting scaling. For example I noticed:

Amazon EFS supports up to 25,000 connections per file system.

With 100MB chunks, this means the maximum size Zarr array you could process in one step before losing parallelism would be 2.5TB.

Jacob Tomlinson on twitter suggested momento, Dragonfly, and ElastiCache as possibilities too.

Also relevant to this issue is the work going on in Zarr to improve performance.

Discussion thread: https://github.com/zarr-developers/zarr-python/discussions/1479
Cubed use case (by @TomNicholas): https://github.com/zarr-developers/zarr-python/discussions/1486#discussioncomment-6732054

Amazon S3 Express One Zone might have solved this problem.

Is fsspec + s3fs + Zarr likely to "just work" with this new storage class? It says it implements most of the same S3 API...

Not yet, by the look of it. I had a quick dig and found that fsspec uses aiobotocore, which in turn depends on botocore. The latest version of aiobotocore is pinned to botocore < 1.33.2, and support for Amazon S3 Express One Zone was added in botocore 1.33.2.
Also, quoting from https://github.com/aio-libs/aiobotocore/pull/1057#issuecomment-1830899886:

botocore>=1.33.2 includes many new codepaths and will likely be a major piece of work for us.

Yikes! Reading a bit more to see why botocore itself doesn't support async io, in https://github.com/boto/botocore/issues/458 I was amused to find a comment from scanpy's Philipp Angerer. We're all hitting the same issues!
I went ahead and filed https://github.com/aio-libs/aiobotocore/issues/1065 to have a place to track the S3 Express One Zone support work in aiobotocore.

Let's just create python bindings for Rust's object store crate and start using that instead. Looks very powerful and performant.

More performance would be welcome! It doesn't support S3 Express One Zone yet either: https://github.com/apache/arrow-rs/issues/5140

the fsspec abstraction is too broad and deep (e.g. caching)

To defend fsspec: what do you think could possibly be getting in the way? Methods like fs.cat([list of paths]) are very minimal and all that zarr actually needs. Caching and such are totally optional. It would be better if you can affect optimisations there, perhaps hooking into a rust implementation if you like.
Does object-store-python support anything async?
I again refer readers to rfsspec: rust/tokio does not magically make things much faster, it may help a little.

A month ago I was doing comparative benchmarks between Spark/Dask/DuckDB/Polars on cloud data.  My observations were that, as long as the projects don't do anything dumb (a big assumption) the game was entirely about optimizing S3 access. I starting pushing on Arrow timings (see https://github.com/apache/arrow/issues/38389) and found that it has much less to do with the language that is used, but rather due to how gently one handles S3.  What I found at the time was that S3 liked ...

Reads of 5-15 MB in size
Plenty of concurrency (2x-3x threads than cores)

On an m6i.xlarge (4 cores) I could get about 60 MB/s from a single read (regardless of whether or not it was coming from boto, fsspec, or arrow (C++ backed)), or about 500 MB/s if I fully saturated the machine.
This was all done before the new S3 single-zone stuff was launched.
I came to the following conclusions:

The framework orchestrating downloads (zarr, dask, xarray, ...) needs to add in some concurrency
It's nice if the downloading library (fsspec, boto, arrow) is pretty simple

All this is to say, I wouldn't expect "Just reimplement everything in Rust" to solve the problem.  I think that you'll need to be more clever than that here.  However I also don't think it would hurt.  I found myself leaning more towards Arrow over fsspec in profiling because things felt simpler to profile.  I was pretty confident that there wasn't some background IOLoop or GIL issue slowing me down (although I never had solid evidence that there was).
In Cubed's case I'd recommend trying to build some concurrency in to a single function call.
After talking to devs in other projects, the 2-3x threads over cores approach is common in Dask (or will be soon), DuckDB, and Polars.

Just a note that concurrency X threads is normal and should be expected in zarr-on-dask workflows. fsspec supports concurrency, so this achieved by setting the dask partition to be a few (4-20) zarr chunks, depending on memory.

build some concurrency in to a single function call

That of course is the tricky thing, in whichever language. If you need a sync gate at all (because the whole application is not async), then you need to decide where that gate is.

I never had solid evidence that there was

This is quite opposite to claims that were made before. IF we can reach near equivalent performance in python, the simplicity of the code and speed of development are big benefits. My experiments in rfsspec showed there is probably little difference.

What I found at the time was that S3 liked ...
Reads of 5-15 MB in size
Plenty of concurrency (2x-3x threads than cores)

These are interesting findings that are hard to tease out from the linked very long thread. Perhaps there can be some sort of summary of findings?

A summary of the finding sounds great.  If someone wants to do that that would be welcome.

 On the design for Zarr-Python v3.0, there's some discussion of implementing an async Store.get_items() method.
Compiled languages
On the topic of implementing IO libraries in compiled languages (like Rust)... There are use-cases which would benefit from being able to read millions of small chunks per second to a single machine (for example, here's an overly long blog (!) post I wrote about my main use-case: training large ML models from multi-dimensional data). To achieve that kind of performance, it would appear necessary to implement at least some of the code in a compiled language. (And to use all the tricks discussed in the thread above: especially using native async IO that modern operating systems provide. I'm tinkering with some ideas in Rust in this repo).
But I recognise that this current GitHub discussion is about low-latency cloud storage buckets (using cubed). And my main focus is somewhat different: my main focus is on local, modern SSDs (which, today, can do up to 3 million IOPS. And the performance is rapidly improving).
As far as I can tell, it's not yet possible to perform millions of IOPS to a single machine from a cloud storage bucket (although Amazon S3 Express One Zone gets us to 100,000 IOPS... although I assume that performance is only possible when talking to the bucket from many VMs). And, of course, if you're using large chunks sizes (100 MB) then you're probably only doing on the order of 10 IOPS per machine, and so your performance is dominated by the network bandwidth, and no one cares if you can shave a few milliseconds off your code's runtime per IO operation!
But there are folks (like me) who require on-prem hardware and fast, local storage. Amazing things are happening in the storage industry right now: I've heard it said that there's been more innovation in storage in the last two years than in the last twenty years! Folks like me need code which can keep up with these innovations in hardware!
And - who knows - maybe cloud storage buckets will soon deliver millions of IOPS, too. But - wait -  is it even possible to do that over a network? Well... even the lowly 1 Gbps NIC in your laptop can do almost 1.5 million packets per second (PPS)... and 400 Gbps NICs are available, which can do almost 600 million PPS. And, in the future, it'd be nice if cloud storage APIs allowed us to submit multiple disk operations per packet... you know... like HTTP has been capable of since 1996 :slightly_smiling_face:.

Thanks for all the comments! Really interesting.

I starting pushing on Arrow timings (see apache/arrow#38389) and found that it has much less to do with the language that is used, but rather due to how gently one handles S3.

That's a really good point @mrocklin. There is a PR in aiobotocore to support Amazon S3 Express One Zone, so hopefully we'll be able to try it out (via fsspec) soon. That would be the first thing I would try to address this issue, which is particularly acute in Cubed as it uses cloud storage for intermediate results (i.e. it doesn't use S3 very gently).

In Cubed's case I'd recommend trying to build some concurrency in to a single function call.

Agreed. When we start reading multiple chunks per task in more sophisticated ways (e.g. for tree-reduce in #284, #331) then we'll need the ability to load N chunks concurrently - sometimes in a streaming fashion - and no more than N chunks to meet Cubed's memory guarantees. It sounds like fsspec can already do that, which is great, also there's active discussion about how to improve this more generally (like the Zarr V3 discussion @JackKelly linked to).

aiobotocore is pinned to botocore < 1.33.2, and support for Amazon S3 Express One Zone was added in botocore 1.33.2.

The newest release of aibotocore seems to have fixed this now right? So we can now try out using fsspec with S3 Express One Zone?

botocore>=1.33.2,<1.33.14 is what I see now, so give it a go

I assume not... But, just to check: would cubed be happy with high latency if it got high bandwidth and a large number of IO operations per second?

Let's just create python bindings for Rust's object store crate and start using that instead. Looks very powerful and performant.

This seems to have been done https://github.com/developmentseed/obstore

And it's almost implemented in Zarr itself: https://github.com/zarr-developers/zarr-python/pull/1661
Would also love to plug in Icechunk to Cubed.

Would also love to plug in Icechunk to Cubed.

Opened #628


rustc turning into a fork bomb

When opening a rust file initially ale uses cargo, then it starts using rustc and spawning a lot of rustc.

Is it forking, or is it running too frequently and not being killed early enough? Which version of Vim are you using? Do you have any other information? Can you paste the output of :ALEInfo?

Yeah it isn't forking, it is just spawning too frequently.
I have updated to a more recent version of rust so cargo works and that seems to fix the problem.
NVIM 0.1.7
:ALEInfoToClipboard with the updated version of rust
 Current Filetype: rust
Available Linters: ['cargo', 'rustc']
  Enabled Linters: ['cargo']
 Linter Variables:

 Global Variables:

let g:ale_echo_cursor = 1
let g:ale_echo_msg_error_str = 'Error'
let g:ale_echo_msg_format = '%s'
let g:ale_echo_msg_warning_str = 'Warning'
let g:ale_enabled = 1
let g:ale_keep_list_window_open = 0
let g:ale_lint_delay = 200
let g:ale_lint_on_enter = 1
let g:ale_lint_on_save = 0
let g:ale_lint_on_text_changed = 1
let g:ale_linter_aliases = {}
let g:ale_linters = {'systemverilog': []}
let g:ale_open_list = 0
let g:ale_set_highlights = 1
let g:ale_set_loclist = 1
let g:ale_set_quickfix = 0
let g:ale_set_signs = 1
let g:ale_sign_column_always = 1
let g:ale_sign_error = '❯❯'
let g:ale_sign_offset = 1000000
let g:ale_sign_warning = '~❯'
let g:ale_statusline_format = ['%d error(s)', '%d warning(s)', 'OK']
let g:ale_warn_about_trailing_whitespace = 1
  Command History:

(started) 'cargo build --message-format=json -q < /var/folders/7b/44cxjcbn5q3c9chky5q9h_f00000gn/T/nvimYyAdMN/4/main.rs'

This looks like the same issue as #380.

Additionally, since the cargo processes stick around, cargo cannot be run from command line.

As far as I can tell, there is a bug in NeoVim where it isn't killing processes correctly. I could be wrong about that, but that seems to be the case. I'd recommend trying Vim 8, or a newer version of NeoVim in the meantime.

I'll move discussion on this to the original issue raised for pylint: #380


Do not use -t flag on install command (does not exist on OSX)

/cc @themalkolm The -t flag does not exist on OSX. I've removed it and added the destination directory at the end of the command. Although I know that you don't have budget for Makefiles any more this week, are there any downsides to doing this? :-)

-D should be available on Mac, could you check?

Unfortunately not: https://apple.stackexchange.com/questions/201029/install-illegal-option-d
Only the lower-case -d can be used.

-d? :)

Doesn't really matter of Travis says green actually.

yeah, I'll add the -d flag and merge. It's working as expected without the -t already.

LGTM


Local network discovery

Grav should be able to discover nodes on the local network using multicast/broadcast.

https://github.com/schollz/peerdiscovery

Initial version is done!


Redesign offers slideshow

Please assign this issue to me. I have added these types of carousels or slideshows in e-commerce and food delivery projects.

I will assign you this issue when DevIncept will commence from 10th July.

Ok sir.

I would like to do this work.
Please assign me this issue.

I would like to do this work.
Please assign me this issue.

okay :)


split and extend modes

This PR adds missing values in data.MODE and splits them up into data.MODE.cmn, data.MODE.gregorian, and data.MODE.extended, of which the last one is a semi-open list.
closes #627

I also remember that there've been issues with nested data types in libmei

Yes (until something else breaks it)


! Undefined control sequence. l.13 \documentclass                  [localFont,alternative]{yaac-another-awesome-cv}

Hi~ Thanks for your great work!
I met a problem while compiling:
This is LuaTeX, Version 1.12.0 (MiKTeX 2.9.7300 64-bit)
 restricted system commands enabled.
(./cv.tex
! Undefined control sequence.
l.13 \documentclass
                 [localFont,alternative]{yaac-another-awesome-cv}
?

Anyone can help?
Thanks!

I will try on windows to see if I can reproduce the issue

Met the same issue on Windows with TexLive 2021

same here

I'm not using lualatex directly, could you try with the following command : latexmk -cd -f -lualatex -interaction=nonstopmode -synctex=1 cv.tex


parts of c++ strings recognized as names

(Original issue 1433 created by None on 2018-03-29T17:48:55.388238+00:00)
I don't know if this bug has already been found but parts of these strings are recognized as names. Tested with 2.2.0.
Example:
#!C++

         unsigned int ExprScriptFunction::getArity() const {
                 if (_pPrototype != NULL) return _pPrototype->getArity();
                 throw UtlException("internal error in ExprScriptFunction::getArity(): this case shouldn't occur");
         }

         const char* ExprScriptFunction::getName() const {
                 if (_pPrototype != NULL) return _pPrototype->getFunctionName();
                 throw UtlException("internal error in ExprScriptFunction::getName(): this case shouldn't occur");
         }

Doesn't reproduce in the latest version, closing.


core: service account auth

Implement authorization using the Service Account flow.

All authentication uses a JSON keyfile. Is that sufficient to close this?


Broken Sixpoint links

The links to "Sixpoint" in the README.md file are broken.
The Android link (https://play.google.com/store/apps/details?id=com.sixpoint.sixpoint&hl=en_US) simply does not work:

The iOS link also does not seem to work for me, but I might be in the wrong region.

The company probably went out of business. The maintainer seems inactive.
I have uploaded a new version of the package (4.1.0) with the issue resolved: https://pub.dev/packages/qr_widget
You can report issues with the package in my new repo: https://github.com/vanyasem/qr.widget


Added missing stop() method for "ksolo stop" command

thanks for spoting it, and for the PR, will cut a new release tomorrow

Thanks for accepting + merging.

thanks for the PR 👍
btw is the latest app version working fine for, specially VM's UP, as there is issue opened #151 whcih I cannot reproduce

I am not sure about #151 but I used to experience a bug where I was unable to re-connect to the cluster after letting my mac go to sleep. This is now working.

sleep I know was the issue before for sure. @AntonioMeireles can you confirm/explain about sleep?


Remove redundant check

If the first operand of the disjunction (!obj.loading) is false, obj.loading is truthy.
Thus there is no need to test it again in second operand.

@jomo your logic is sound :) thanks for the contribution \o/


Implement image message model

Images handled by (branch)[https://github.com/ben-cunningham/pybot/commit/b78eaba5e299334029c925a28fbdf32095e41c21]


[SPARK-10054][Streaming]Add a timeout for launching Receiver

Add a timeout for launching Receiver. StreamingContext will be stopped if any of the receivers cannot register in time.

Merged build triggered.

Merged build started.

Test build #41023 has started for   PR 8242 at commit ae78ee6.

Test build #41023 has finished for   PR 8242 at commit ae78ee6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41023/
Test PASSed.

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #41183 has started for   PR 8242 at commit ca08e97.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41181/
Test FAILed.

Merged build finished. Test FAILed.

Test build #41183 has finished for   PR 8242 at commit ca08e97.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41183/
Test PASSed.

Merged build triggered.

Merged build started.

Test build #41829 has started for   PR 8242 at commit c17d36c.

Test build #41829 has finished for   PR 8242 at commit c17d36c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41829/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #41856 has started for   PR 8242 at commit 1427476.

Test build #41856 has finished for   PR 8242 at commit 1427476.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SQLTransformer(JavaTransformer):

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41856/
Test PASSed.

Merged build finished. Test PASSed.

I'm going to close this one. We can reopen it later.


Wobbly edges

Hello,
I’m trying to train a credit card detector(card is held on forehead). I have a training set of 80 images, test set of 18 images and finally validation set of 21 images. I know my dataset is small for training but before moving forward I want to be sure whether I’m on the right track or not.
My problem is, when I train the detector, it’s able to find the card but especially when card is tilted, it detects edges wobbly. I’m sharing the two versions of the same image(one shows how I labelled the other shows how detection resulted).

My config file is as below:
_base_ = '../mask_rcnn/mask_rcnn_r50_caffe_fpn_mstrain-poly_2x_coco.py'

model = dict(
    roi_head=dict(
        bbox_head=dict(num_classes=1),
        mask_head=dict(num_classes=1)))

dataset_type = 'COCODataset'
classes = ('card',)
data = dict(
    train=dict(
        img_prefix='data/base_set/training/',
        classes=classes,
        ann_file='data/base_set/training/training_labels.json'),
    val=dict(
        img_prefix='data/base_set/testing/',
        classes=classes,
        ann_file='data/base_set/testing/testing_labels.json'),
    test=dict(
        img_prefix='data/base_set/validation/',
        classes=classes,
        ann_file='data/base_set/validation/val_labels.json'),
)

load_from = 'checkpoints/mask_rcnn_r50_caffe_fpn_mstrain-poly_3x_coco_bbox_mAP-0.408__segm_mAP-0.37_20200504_163245-42aa3d00.pth'

I want to increase precision of how card is detected. I mean card edges should be detected with straight and more precise lines. I have been searching through the documents to find out where I'm doing wrong  but couldn't find anything right now.
So I appreciate any recommendations for achieving my goal. Is my dataset too small? or should I use a different config file as base? etc.Thanks in advance.

The reason should be that the mask output size is too small.
mask_roi_extractor=dict(
      type='SingleRoIExtractor',
      roi_layer=dict(type='RoIAlign', output_size=14, sampling_ratio=0), # 14
     out_channels=256,
     featmap_strides=[4, 8, 16, 32]),

You can consider modifying output_size  to 28 or change to PointRend

If you use pointrend, you should modify the following configuration
    # configs/point_rend/point_rend_r50_caffe_fpn_mstrain_1x_coco.py
    roi_head=dict(
        type='PointRendRoIHead',
        mask_roi_extractor=dict(
            type='GenericRoIExtractor',
            aggregation='concat',
            roi_layer=dict(
                _delete_=True, type='SimpleRoIAlign', output_size=14),
            out_channels=256,
            featmap_strides=[4]),

@hhaAndroid thanks for your recommendation. Now edges are much better:

Yet I still need improvement in accuracy so will be working on that. Thanks :)


Publish v0.0.39 on npm

The current version of angucomplete-alt on npmjs.org is v0.0.36. Could you update it to the current version?

Sorry. I thought I've been doing this. Done.


Detect same domain conf cloud attachments and don't transmit via proxy to use existing client permissions

Idea was not to proxy same-domain content, but this isn't worth the complexity.


Early feedback from Netlify

Hi there!
After our initial small exchange on Twitter, I had a quick look at this plugin. We can dig a little deeper when this is submitted to the plugins directory.
This is looking great! I do not see anything that's a major issue. The only two things I would note are the following:

Plugin event handlers should not return values (we are reserving doing this for some potential unrelated features).
The HTML injection logic might potentially be slow because each file is being read/write serially, one at a time. Instead, there might be some benefits to executing this on ~100 files at a time (up to 1000 might be ok, you'd just want to ensure it does not hit the OS limits), which should dramatically improve performance when the site has 10,000s or even 100,000s of HTML files. I personally find p-map particularly helpful to do this easily (but there are other libraries as well) thanks to its concurrency option.

Thanks again for working on this, this is awesome!

Hey @ehmicky, thank you for looking at this so quickly!

The event handlers either return undefined (same as a function without an explicit return), or the result of calling utils.build.failBuild, or utils.build.failPlugin (docs). The errorResponse() function is a wrapper around failBuild and failPlugin so I can switch owhichne is used based on the user's inputs/settings.

Should I not be returning a failBuild and failPlugin from the event handlers? Is it enough to just call those functions without returning their result?

Great catch on the HTML injection logic. p-map looks great so I'll probably end up using it as suggested, and I'll borrow a little bit more from the New Relic Python agent and limit how much of file contents I need to check with the regex.

Does plugin execution have a time limit?
Would it be faster to use cache? Does the cache expire? I'm thinking something like:

Use utils.git.fileMatch() to find new/modified HTML files
Perform injection logic only on those files, and cache the results
Restore the rest of the HTML files from the cache

About utils.build.fail*: this is an issue with our documentation. I just reported it to our documentation team. Those methods do not return any value. Instead, they throw an error. So you should be able to use those without any return.
Plugin execution does not have a time limit (beyond the time limit of the build itself).
Using caching is a nice idea! That being said, there might be several gotchas to get it right. For example:

utils.git.fileMatch() is comparing the HEAD with CACHED_COMMIT_REF which is basically the current commit and its parent commit.
Saving cached files happens asynchronously and can sometimes end after the build has completed. One build might be missing the cache from the previous one if that one was run very recently (or is still running).
Therefore, any files not returned by utils.git.fileMatch(), i.e. not modified by the current commit, might have been modified by the parent commit, but not cached yet.

This might lead to situations where some files are accidentally skipped by the injection logic.
From a performance standpoint, the main bottleneck is reading then writing each file. The performance should be roughly similar to copying the publish directory recursively, which can be slow, but probably not that much. From that perspective, it looks like caching might be a good candidate for a later optimization instead, once the plugin is stable. However, that's just my two cents!


Add support for exporting metrics to a pushgateway

Add support for exporting metrics to a pushgateway. By default the fqdn is sent is instance wasn't provided.

I've been holding off on adding this until https://github.com/prometheus/pushgateway/issues/18 is resolved, as the 'instance' handling isn't quite compatible with the usual use cases.

Resubmitted using urllib2. Once prometheus/pushgateway#18 is sorted out, I'll update the PR. The use_fqdn_as_instance param was a hack to overcome it.

I messed this up, deleted the branch by accident and that closed the PR.

@brian-brazil @mmoya - just noticed this PR sitting here for almost a month - is it still current?

This is still current, we're waiting on the instance label cleanup on the push_gateway side.

Any idea when this might land? Would love to see that functionality.

Looks like we're pretty close to having all the required changes elsewhere.
This needs to be updated to set the instance label to the empty string if it's not present already before exporting.

This needs to be updated to set the instance label to the empty string if it's not present already before exporting.

Wait, that's not right. No change required there as the pushgateway handles that.
You'll need up pull in the latest changes to the repo, and things have been split into separate files.

I've updated this and merged it. Thanks!


Using traits for an association

Can not use traits from association
factory :user do
  trait :with_profile do
    name "John Doe"
  end
end

factory :subscription do
  association :user, :traits => [:with_profile]
end

Gives the error "undefined method `traits=' for #User:0xced2284"

This sounds like a good idea! I'll close this and keep an eye out for the pull request! Thanks!

Is it still available? Current docs don't mention that.

@Nowaker yep, it's still there, in the Traits section. The commit adding the documentation is here: https://github.com/thoughtbot/factory_girl/commit/c9ed3d88e5aa8d02ffd3e8b63ea80542caee1a01

I wanted to use a trait in an association in my factory definition. I'm not sure if things changed, but the syntax suggested here doesn't work.  In case it helps someone else, here is what I got to work:
FactoryWhatever.define do
  factory :subscription do
    association :user, factory: :user_factory do
      with_profile
    end
  end
end

@gnfisher Thanks. It works

Looks like the documentation has changed locations and now lives here:
Traits Documentation


chore: remove nx try import except ImportError

Historically, a try import except ImportError was introduced in 2019, when this NApp wasn't pinning its requirements/run.txt dependencies. We no longer need this. Following the installation with python setup.py develop will take care of everything.

This has landed too


make remote application effect on consumer side too

What is the purpose of the change
make sure RpcContext.getContext().getRemoteApplication() returns provider's app name on client side. It may useful when a custom filter is implemented.
a follow up for pull request #4722
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #5029 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5029      +/-   ##
============================================
+ Coverage     63.88%   63.95%   +0.06%
- Complexity      450      453       +3
============================================
  Files           769      769
  Lines         33147    33182      +35
  Branches       5206     5232      +26
============================================
+ Hits          21177    21222      +45
+ Misses         9549     9544       -5
+ Partials       2421     2416       -5

Impacted Files
Coverage Δ
Complexity Δ

...apache/dubbo/rpc/filter/ConsumerContextFilter.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...che/dubbo/remoting/transport/mina/MinaChannel.java
43.42% <0%> (-10.53%)
16% <0%> (-1%)

...dubbo/common/status/support/LoadStatusChecker.java
46.15% <0%> (-3.85%)
0% <0%> (ø)

...ting/exchange/support/header/HeartbeatHandler.java
83.72% <0%> (-2%)
0% <0%> (ø)

.../main/java/org/apache/dubbo/qos/textui/TTable.java
83.79% <0%> (-0.87%)
0% <0%> (ø)

...org/apache/dubbo/registry/redis/RedisRegistry.java
48.15% <0%> (-0.26%)
31% <0%> (ø)

...n/java/org/apache/dubbo/common/utils/UrlUtils.java
78.04% <0%> (-0.2%)
0% <0%> (ø)

...mon/src/main/java/org/apache/dubbo/common/URL.java
50.22% <0%> (-0.08%)
0% <0%> (ø)

...g/apache/dubbo/registry/consul/ConsulRegistry.java
61.87% <0%> (ø)
29% <0%> (ø)
:arrow_down:

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 23 more

Continue to review full report at Codecov.

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


Website

Hello,
I have created an website interface that looks better, I'll include a download once I'm done with testing.
Regards,
Sander Jochems

I couldn't found any errors. here is the zip with all the files in the data folder
data.zip

Hello
Looks much better indeed. Any chance to get an offline version and ESP fs friendly please?
Best wishes
Grzegorz

Hello,
Could you tell me what you mean with an offline version?
I don't know what a ESP fs friendly is.
Regards,
Sander

Hi
I use isolated network with local icecast server without internet access. It looks like your nice pages need access to internet. I was thinking if I could download external files down to ESP fs.
Best
Grzegorz

On 28 Dec 2016, at 10:36, Sander Jochems<EMAIL_ADDRESS>wrote:
Hello,
Could you tell me what you mean with an offline version?
I don't know what a ESP fs friendly is.
Regards,
Sander
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes, i use some internet css for the style. I will include the css in the data map

I'll add a new donwload link in a few minutes

Amazing! Thank you, much apriciate.
Best wishes
Grzegorz

On 28 Dec 2016, at 10:43, Sander Jochems<EMAIL_ADDRESS>wrote:
I'll add a new donwload link in a few minutes
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm done, it should work now
data.zip

I like looks!  But 1.5 MB is quite a lot of space...

Yes, that is because the css is in the data map, if you take remove the css map and change te url its a lot smaller

Personally, I was planing to make a SD http serveur, and let the radio module connect on it for local .mp3 playing. Cause if I remember well, the VS10x3 don't like to be on the same bus the the SDcard reader, some conflict on MISO line, unless you uuse a 3states buffer to Isolate the sdCard.

I have two version now:
Internet CSS --> data.zip
Included CSS -->data.zip
I would use the Internet CSS, becuase then you have more space left in your ESP8266.

I like the "Included" version for safety and independency.  I will try to make a smaller version with the same looks.  My aim is 95 percent of the looks for 1 percent of the size...

Yes, the best thing you should do is to remove all the parts from the css which are not used.

New interface now in data-directory (14 KB).

Nice work,
There are a few things I would change to make it even nicer.

White background of the textboxes
No URL in the "ESP Radio" title in the topbar
3 Links on every page, not 2

If you want, i can change it and send it for the next update

I'm done, data.zip (17,1 kB)

I think the white background & links etc can be fixed by editing the ESP-radio.ino file.
Go to function String getContentType ( String filename )
and add this line:
else if ( filename.endsWith ( ".css"  ) ) return "text/css" ;
Without this, the esp internal web sever returns ccs code as text/plain whereas it needs to be text/ccs.
I hope this helps.

I added Sanders website to the data directory.
I added the ContentType "text/css" to the sketch.

The webinterface is now in PROGMEM.  More reliable than in SPIFFS.

I found out that the ESP8266 will not handle multiple connections to the async webserver.  Most browsers will open multiple connections to get index.html, radio.css and favicon.ico simultaneously.  In Mozilla I changed the "network.http.max-persistent-connections-per-server"to 1 (was 6).

Edzelf, maybe replace <link rel="stylesheet" type="text/css" href="radio.css"> with
<script type="text/javascript"> var stylesheet = document.createElement('link'); stylesheet.href = 'radio.css'; stylesheet.rel = 'stylesheet'; stylesheet.type = 'text/css'; document.getElementsByTagName('head')[0].appendChild(stylesheet);  </script>
and add <noscript><link rel="stylesheet" href="radio.css"></noscript> after </html>
It should work.

I will try that.  Can you explain the difference?

radio.css should start loading after page is loaded, so multiple connections will not be needed.
Tested in google chrome and firefox

I'm also trying additional headers.
changed(every file):
request->send_P( 200, ct, index_html );
to:
AsyncWebServerResponse *response = request->beginResponse_P( 200, ct, index_html ) ;
and added:
response->addHeader("Server","Esp-radio Web Server");
response->addHeader("Cache-Control","max-age=3600");
response->addHeader("Last-Modified","Sat, 07 Jan 2017 21:00:00 GMT");
request->send(response);

Pavulon87:  I updated the project according to you suggestions.

What tool should I use to convert those resource files into C array?
I had custom icon need to put into PROGMEM. Don't know how to do it.

I used "od" on a Linux machine, but Googling on "dump file in ascii" or "hexdump" will give you all the answers.


Builder removes the NODE_ENV variable

Weird one as it only just started to occur, but here's a minimal repro:
now.json
{
  "version": 2,
  "name": "now-env-test",
  "build": {
    "env": {
      "NPM_TOKEN": "@npm-token",
      "NODE_ENV": "hello",
      "OTHER_VAR": "world"
    }
  },
  "builds": [
    { "src": "package.json", "use": "now-sapper" }
  ]
}

package.json
{
  "name": "now-env-test",
  "version": "1.0.0",
  "description": "fff",
  "main": "index.js",
  "scripts": {
    "build": "echo Build: $NODE_ENV $OTHER_VAR",
    "now-build": "echo Build: $NODE_ENV $OTHER_VAR"
  },
  "author": "",
  "license": "ISC"
}

index.js
module.exports = (req, res) => {
  res.end(`Hello from Node.js on Now 2.0!`);
};

If you deploy this project, the build output is simply "world" instead of "hello world"

After a weekend of working through the issue with Zeit - turns out it wasn't us - it was them 0 https://github.com/zeit/now/pull/3862
Closing :tada:


Crash when deleting GameObjects with Mesh and Textures

Currently there is a bug where deleting GameObjects with Mesh and/or Textures will crash the engine.
Suspect it's because Vulkan Objects are not being destroyed correctly or are being destroyed before the commandBuffer has been submitted.

Bug has been patched per latest update on main.


 You cannot put a (discord/non-unicode) emote within the codeblock, so we cannot get a true cowsay of discord text.
Potential alternatives include:

Printing only the emote name (e.g. :yeet:)
Removing all emotes before the message is cowsaid
Something else?

Personally, I think printing only the emote name would work best.

@lcox74 is looking into it

closed with the merging of #89


kubectl doesn't show gameon deployments by default

The go-run.sh setup script sets the deployments' namespace to gameon-system.
By default, kubectl commands will run with the default namespace, so the commands (and minikube dashboard) show nothing about the gameon deployments.
Fix: Switch the namespace to gameon-system either by:

Changing the default namespace in the kubectl config with kubectl config set-context minikube --namespace=gameon-system

or making a new config? kubectl config set-context minikube-gameon --cluster=minikube --user=minikube --namespace=gameon-system

Appending --namespace=gameon-system to every kubectl and minikube commands

Alternatively, change the gameon namespace to use the default namespace instead of having its own namespace

Messing with 1) seemed to mess up how helm finds its charts.
Could add to docs: to view these items on the kubernetes dashboard you must switch namespace to gameon-system in the left menu

The intention was that you, the intrepid developer, would be writing your own additional services, which would then be in the default namespace for you to control.
If you could try the iterate branch (beware, I force-push that thing.. so you need to be prepared to hard reset periodically) , go-run k ....  will do the kubectl command with the gameon-system namespace

The main issue was that we didn't know why Kubernetes didn't list our deployments and services when helm did. When the script handled these automatically we weren't aware it had neatly organized them for us. It's nice and I can agree now but the docs should at least mention this

I can add that. Thanks for the feedback. I will add to the iterate branch, and you can close this once I've done the job to your liking. =)

PR updated. Let me know what you think.

looks good! 👍


Universal Action Model Schema Released and Documented

Link to feature: https://adaptivecards.productboard.com/feature-board/planning/features/5302870

Closing this, with #3076 tracking the remaining work (javascript implementation)


On reusability: how to stop suffering from “idtech is meant to be forked” mindset

We have a problem, not that particular “playerstat” thing that is just a trigger for this talk, but we have a problem in general with the dæmon engine. Like all idtech3 mods and games, we have suffered enough from the fact the id tech engines were meant to be forked.
Id tech engine is great and efficient, but it's like every part of it assumes you will fork the whole and never contribute back neither mutualize the development efforts.
Already 20 years of suffering have passed, 20 years of developers suffering from that.
Id tech was meant to be sold and forgotten from the seller. It looks to be designed for only one social relation: the sale. That's the absolute opposite mindset we had when we created this repository. That's the absolute opposite mindset we have when we develop a game.
Either map formats, either materials, either net code, either many things, there is no place to extend, versionnize, and in many case, no bits to even detect the format.
Everything looks to be done in a way the only future exists in forking. At the social scale, this is pure entropy.
Let's make an analogy: not fighting entropy is exactly the same as to run something without backup because the only purpose of a backup is to fight entropy. The issue we have is that not only id tech engine does not fight entropy, but it is designed in a way it induces entropy. To pursue the analogy with a backup, on the social scale an id tech engine behaves like an object with a design preventing lossless copies to be made.
I don't know if you have read Robert Sheckley, but on the social scale id tech engines are like Tangreese, forks of them are produced from thin air as time passes, drawing power from the environment, and we don't have a Laxian key to stop that.
I'm looking for a Laxian key. And before we find it, I'm OK to change the engine to make it more reusable.
We have to fight entropy or our project is already dead: we're not selling discs of plastic that are deads as soon as they are produced and sold. We are building free software and we are building a community. That means we are producing two living things: imagination and social interaction.
There is no "it had been like that since 20 years” to say, in 2020 no one game developer is looking for an engine that requires to be forked and maintained alone.
At some point, that's the reason why we did the NaCL thing and are looking at Wasm: make a game, build it once, run it everywhere the engine is installed. But that's not enough: at first id tech3 did the same with QVM, but it appears that was basically a way for them to ship updates easily.
The fact QVM also makes possible to mod the game is just there for free without guarantee, a game will even face trouble to extend itself without having to fork the engine at some point. We are there.
We already seen that problem with PK3 VFS. PK3 is good enough to ship updates. But PK3 is not mod friendly. Because PK3 is good enough to ship updates, the ability to mod the game is also there for free but without guarantee neither any failsafe. That's a fact players discovered with pain and suffered from for 20 years: PK3 VFS is not designed for modding, it's only designed for updates coming from one source only: the seller. As soon as you have more than one source, the PK3 VFS does not prevent the player's game to be broken. That's why we created the DPK VFS, and… we had to make the hard choice to break compatibility.
The game is so tied to the engine it's even hard for us to extend it without forking our own engine, I mean by breaking compatibility.
Like when we implemented DPK, I'm OK to break compatibility once for some design that makes the engine less prone to compatibilty breaking and less prone to forks on the project and social scale. If we don't find a Laxian key, that would be our Laxian key.

For the specific case of the playerState_t netcode, it actually has a nice table-driven structure. Although currently the definition of the player state fields is hard-coded in the engine, it would be fairly easy to load the table from the gamelogic instead.

I just read this sentence in Wikipedia English page about Unreal Engine:

Unlike id Software, whose engine business only offered the source code, Epic provided support for licensees and met with them to discuss improvements to its game development system.

For some reason, it's also the fact id Software used to just dump the code in a transaction that made them free enough to dump the code again as free software when next iteration was profitable.
I use to say that forking is like giving birth: without attempt to revive it, it's already a dead child. Id Software was basically selling the child without parenting it.
In software development, a fork is always about code being dead, or killed. At the time of the fork, both original branch and the new one are now dead. Only revive those who receive huge efforts put on life support.
Id Software engine development, business model, and open source release was all about forking.
When sold, the new child was delivered without life support. Internally, new engine iteration was fork of the previous tree, with the new child vampirizing all life support. The open-sourcing of the dead child then happened when the body was cold. That's a bit of irony, but that's also because the body was cold it was easier to open-source it. Also, when you consider the children are all dead, there is no such concern like license compatibility among them, that adds to irony.
I don't want the engine to be killed anytime it reaches puberty. I do not think that children are worth only their childhood. I want the engine to grow up, learn from others, have a full life and become a wise man.
Speaking about myself and Dæmon, I want to provide support and met with people to discuss improvements to our game development system. This is all about free software: free software is not about software, it's about people. This time my words are not metaphorical: only living people can be free. I care about living people.

Her is an extended response and description of the technical and philosophical problems we face:

Building a community as a service: how to stop suffering from “this code is meant to be forked”
https://unvanquished.net/building-a-community-as-a-service/

For the limited playerStat_t issue, some solutions were implemented there: https://github.com/DaemonEngine/Daemon/pull/341
For the engine being repurposable to another game without recompilation, see https://github.com/DaemonEngine/Daemon/pull/588
For other specific issues where the engine is too deeply tied to Quake 3 things and where even other idTech3 games had to fork and ship incompatible engines (like not being able to extend BSP format without breaking compatibility or support multiple BSP formats), let's open specific tickets.


Delete useless doc

Signed-off-by: tumao<EMAIL_ADDRESS>

Codecov Report

Merging #434 (f04c48b) into main (f4e9152) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #434   +/-   ##
=======================================
  Coverage   61.99%   61.99%
=======================================
  Files         141      141
  Lines        5420     5420
  Branches      805      805
=======================================
  Hits         3360     3360
  Misses       1838     1838
  Partials      222      222

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 f4e9152...f04c48b. Read the comment docs.

/approve

/approve
/lgtm


UI - CIS Scan - No error when junk values entered in config map

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Go to Cluster --> System Project --> Config --> Config map --> Edit
Currently we are allowed to input invalid values. Enter invalid values and click on save.
Run Scans again on the cluster.
Scan fails to generate report
On clicking on the scan generated in Step 4, no error is seen on the UI.
But when a user clicks on a scan where a report has been generated like cis-1575938047053880138 in the screenshot below, error is seen on the UI.

Expected Result:
In both the cases, error should be seen on the UI.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head commit id: 5f917419b
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): DO
Kubernetes version (use kubectl version):

1.16.3

Verified on master head - commit id: 48a2c4ffc
Steps:

Go to Cluster --> System Project --> Config --> Config map --> Edit
Currently we are allowed to input invalid values. Enter invalid values (like @#a)and click on save.
On clicking on any of available scans.
Clicking on Skip or Unskip, gives an error on the UI:

Clicking on Run Scan - Runs a security scan on the cluster. and report is generated successfully.


Accurate bar charts, starting at 0, with trend line

Several recent issues are closely related so I'll work on them together:

[x] Switch charts to vertical bars https://github.com/project-vagabond/covid-green-app/issues/81

[x] Add 7-day rolling average line
[x] Add labelling to line (needs input on text)

[x] Start the charts' Y-axis at 0 https://github.com/project-vagabond/covid-green-app/issues/80
[x] Ensure the charts accurately represent the latest API data https://github.com/project-vagabond/covid-green-app/issues/79
[x] No non-integers in y axis labels, and don't show very low numbers as spikes https://github.com/project-vagabond/covid-green-app/issues/94
[ ] Change charting period to 30 days https://github.com/project-vagabond/covid-green-app/issues/78

Not getting the 30-day data through the API yet, this is with fake data:

With latest commits fake data has been removed and it should be able to handle any amount of data, here is how it handles 7 days of data:

Rated this as High because it contains multiple issues,


Please unlock #138128

Yesterday @sandy081 reopened #138128 (Green dot for pre-release feels random), but two days earlier the bot had locked the issue. Please unlock it to allow for more feedback.

Done


exec() chef-client to resume the run after updating the omnibus install

#64 rebased onto develop per comment.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

This issue has been solved in the new chef_client_updater cookbook. I'm going to close this issue out here since we've deprecated this cookbook in favor of the new method. We believe it provides a much better experience and encourage everyone to try it out.


Add support for importing default exports

This should fix #1006
This PR doesn't mangle reserved JS words, only default. This is an alternative to #1091

O_o. For some reason failure crate fails to build. Can't reproduce it locally... Is it some kind of CI bug?

No worries that was my own mistake!


television: 0.5.0 -> 0.5.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for television is: Television is a blazingly fast general purpose fuzzy finder
meta.homepage for television is: https://github.com/alexpasmantier/television
meta.changelog for television is: https://github.com/alexpasmantier/television/releases/tag/0.5.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.1 with grep in /nix/store/j66kxjbnw1ay5nzfjmnnwjql2b6ih9vv-television-0.5.1
found 0.5.1 in filename of file in /nix/store/j66kxjbnw1ay5nzfjmnnwjql2b6ih9vv-television-0.5.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

television

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/j66kxjbnw1ay5nzfjmnnwjql2b6ih9vv-television-0.5.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A television https://github.com/r-ryantm/nixpkgs/archive/8171d5b3a676521fb9c7c511f6fbe2c00df27a7f.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8171d5b3a676521fb9c7c511f6fbe2c00df27a7f#television

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/j66kxjbnw1ay5nzfjmnnwjql2b6ih9vv-television-0.5.1
ls -la /nix/store/j66kxjbnw1ay5nzfjmnnwjql2b6ih9vv-television-0.5.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

television

Maintainer pings
cc @louis-thevenet @getchoo for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot  merge


Reduce false-positives emitted from Y026 TypeAlias check

Fixes #205.
I've temporarily renamed the check from Y026 to Y038 so that I can see the impact using typeshed_primer in CI. I'll revert the renaming once the CI has completed.
(EDIT: The output looked good, so I reverted the renaming.)

I've done some random skimming of the output from typeshed_primer here, and it looks like this gets rid of the vast majority of the false positives while retaining the vast majority of the true positives.

There are still some false positives, even with this patch. I'm working on it.

I know it's a draft, but I want to clear my requested review list :)
I'm wary of a heuristic-heavy approach like this since if it goes wrong, it will be hard for new contributors to understand the problem.
What if we limit ourselves to extremely obvious cases like those where the RHS has a | at the top level?

I'm moving in that direction as well after discovering more false positives when working on https://github.com/python/typeshed/pull/7630 :)

With the latest version of this patch, there are now only 4 hits in typeshed:
./stdlib/builtins.pyi:1265:1: Y03[8](https://github.com/PyCQA/flake8-pyi/runs/6046702244?check_suite_focus=true#step:6:8) Use typing_extensions.TypeAlias for type aliases
./stdlib/builtins.pyi:1384:1: Y038 Use typing_extensions.TypeAlias for type aliases
./stdlib/typing.pyi:1141:5: Y038 Use typing_extensions.TypeAlias for type aliases
./stdlib/typing.pyi:1153:5: Y038 Use typing_extensions.TypeAlias for type aliases

These are all places where we can't currently use TypeAlias due to some kind of bad interaction between PEP 604 and PEP 612 for mypy.


Unable to login

Additional technical information:
Correlation ID: c2423490-3795-4097-9100-c8a6d0fabcc8
Timestamp: 2017-08-23 19:17:29Z
AADSTS70001: Application with identifier '0abb4c32-97b5-432b-a6ea-42ca08be' was not found in the directory ********.onmicrosoft.com

Hell @cognologixguru.
Did you register you app with the Azure portal (https://portal.azure.com/) as explained in the README.md of this sample?
Or would you by any chance have registered it with the Application Registration Portal (https://apps.dev.microsoft.com), where you would have added a converged application?
If this is the later, then the error is expected because the Application Registration Portal is used to add applications for the Azure AD V2 endpoint (applications that can be accessed both by Work/School accounts and Microsoft personal accounts), whereas this sample uses adal.js which only supports the V1 endpoint (only Work/School accounts and on premise Active Directory (ADFS)).
You need to register your application with the Azure portal  (https://portal.azure.com/)

Hello @jmprieur ,
Thank you for reply. I will try as per your suggestion and I will inform you if any any issue persists.

Closing for now. Feel free to re-open if this does not work for you.


Add support for "eslint-plugin-qunit"

Similar to #8 but using https://github.com/platinumazure/eslint-plugin-qunit

@sapeien I changed the test code to use fixture files now

merging as all the comments were addressed


Edit alert screen for nodes, etcd, events always has "scheduler" selected by default

Rancher Version: Master build from Dec 6th
Steps:

Create a DO cluster, enable monitoring
The default alerts for etcd, nodes, etcd, events are displayed.
Edit alert for nodes, etcd, events etc.
Edit screen always displays "scheduler" by default.

Example:

The fields which are not relevant to that alert should be disabled

Version - 2.2 master 12/7
Verified fixed


Add list_series and delete_series to query series v2

So far, this looks fine to me (pending a missed not-frozen string).
Do you mind squashing the spec commits and implementation commits into one each? Or shall I?
--Dominik

@dmke
Thanks for your comment.
I'll try it.
Please wait for a couple of days.

Sure :-)
Let me know if I can be of any assistance.
--Dominik

Thank you for your kindness!
I made another PR(https://github.com/influxdata/influxdb-ruby/pull/186)

#186 is merged, so I'm closing this.
Did you run into problems updating this brach, or why did you open another PR? (I'm just curious, this is absolutely fine)


Fix username in k8s auth token

The k8s auth token was using the username which is able to change and then does not match clusterrolebindings and rolebindings in target clusters anymore.

Coverage remained the same at 55.32% when pulling 45eb72590956696b7cf9c9e79fa5e3829764a8d8 on fix/k8s-auth-username into c66b1d8e4c3bfeae6c211ea0b5b61a97238104c7 on main.


Mips64 architecture kernel

Hi , I came across a firmware which belongs to netgear （WMS5316_FW_V2.1.7），
The architecture of the firmware is mips64 , but the mips kernel provided by FirmAE is mips32.
Is there some script or tool which can compile the mips64 architecture kernel

Currently, FirmAE does not support mips64 cause the toolchain is the musl-cross of the Firmadyne.
However, the latest musl-cross supports x86_64 architecture. link
Moreover, FirmAE uses the kernel-v4.1 for all the architectures, so it may be compatible with the latest musl-cross.

Currently, FirmAE does not support mips64 cause the toolchain is the musl-cross of the Firmadyne.
However, the latest musl-cross supports x86_64 architecture. link
Moreover, FirmAE uses the kernel-v4.1 for all the architectures, so it may be compatible with the latest musl-cross.

Sorry for my misunderstanding, musl-cross does not support mips64. However, I found musl-cross-make repository which supports mips64 and also aarch64.
The kernel-v4.1 is old, so it only supports the old version of gcc (5.3.0) and the other libraries, too. I'm testing now to build the compiler.

Sorry for my misunderstanding, musl-cross does not support mips64. However, I found musl-cross-make repository which supports mips64 and also aarch64.
The kernel-v4.1 is old, so it only supports the old version of gcc (5.3.0) and the other libraries, too. I'm testing now to build the compiler.


Auto-responder email vacation & vacation_notification

https://sourceforge.net/p/postfixadmin/code/HEAD/tree/trunk/VIRTUAL_VACATION/

This feature is already available with rainloop and Pigeonhole Sieve plugin. You can set a vacation auto-responder like this :

Sieve rule :
require ["vacation"];

vacation :days 1 :addresses<EMAIL_ADDRESS>:subject "Out of office" "My vacation message...";
stop;

Mailserver log :
dovecot: lmtp(522, user@domain.tld): xxxxxxxxxxxxxxxxxxx: sieve<EMAIL_ADDRESS>sent vacation response to<EMAIL_ADDRESS>
Headers :
X-Sieve: Pigeonhole Sieve 0.4.2
Subject: Out of office
In-Reply-To: ...
References: ...
Auto-Submitted: auto-replied (vacation)

hi there
is the Pigeonhole sieve plugin still available and if yes hwo to install it (it is not in rainloop>plugins>available plugins)
thank you!
Cheers

@digitalkram you must enable Sieve support in rainloop admin panel.

Pigeonhole plugin is already enabled in Dovecot configuration.

Great, thank you!


Fixes for newer versions of jsoncpp

Also changed out uses of memberName to name since it was deprecated.

Note this pull request was closed as an unintended side-effect of our removal of the release branch.   Please feel free to file a new pull request against the main branch, If these changes are still relevant.  Thank you!


charCount of undefined error in LineNode.walk causing invalid errors in editor

TypeScript Version:  3.7.x-dev.201xxxxx

Search Terms:

charCount
Cannot read property 'charCount' of undefined

Expected behavior:
Reliable responses from tsserver
Actual behavior:
tsserver returns errors that aren't accurate. Sometimes the errors use mangled versions of my identifiers (I have an identifier addresses and got an incorrect type error for addresseses on the same line). I do see an exception in tsserver:
[Trace  - 9:54:40 AM] Sending request: navtree (205). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/src/the-pill-club/web-infrastructure-2/sites/dashboard/src/account-renew/index.tsx"
}
[Trace  - 9:54:40 AM] Response received: navtree (205). Request took 1 ms. Success: false . Message: Error processing request. Cannot read property 'charCount' of undefined
TypeError: Cannot read property 'charCount' of undefined
    at LineNode.walk (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:141571:68)
    at LineIndex.edit (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:141452:31)
    at ScriptVersionCache._getSnapshot (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:141242:47)
    at ScriptVersionCache.getSnapshot (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:141235:82)
    at TextStorage.getSnapshot (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:133768:32)
    at ScriptInfo.getSnapshot (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:133928:41)
    at ConfiguredProject.Project.getScriptSnapshot (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:134542:39)
    at SyntaxTreeCache.getCurrentSourceFile (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:130325:44)
    at Object.getNavigationTree (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:130952:71)
    at IOSession.Session.getNavigationTree (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:140539:44)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:139433:61)
    at /src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:140870:88
    at IOSession.Session.executeWithRequestId (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:140861:28)
    at IOSession.Session.executeCommand (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:140870:33)
    at IOSession.Session.onMessage (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:140893:35)
    at Interface.<anonymous> (/src/the-pill-club/web-infrastructure-2/node_modules/typescript/lib/tsserver.js:142208:27)
    at Interface.emit (events.js:193:13)
    at Interface._onLine (readline.js:308:10)
    at Interface._normalWrite (readline.js:449:12)
    at Socket.ondata (readline.js:165:10)
    at Socket.emit (events.js:193:13)
    at addChunk (_stream_readable.js:296:12)
    at readableAddChunk (_stream_readable.js:277:11)
    at Socket.Readable.push (_stream_readable.js:232:10)
    at Pipe.onStreamRead (internal/stream_base_commons.js:150:17)
[Trace  - 9:54:43 AM] Event received: projectsUpdatedInBackground (0).
Data: {
    "openFiles": [
        "/src/the-pill-club/web-infrastructure-2/sites/dashboard/src/components/Greeting.tsx",
        "/src/the-pill-club/web-infrastructure-2/sites/dashboard/src/account-renew/index.tsx"
    ]
}
[Trace  - 9:54:44 AM] Sending request: geterr (206). Response expected: yes. Current queue length: 0
Arguments: {
    "delay": 0,
    "files": [
        "/src/the-pill-club/web-infrastructure-2/sites/dashboard/src/components/Greeting.tsx",
        "/src/the-pill-club/web-infrastructure-2/sites/dashboard/src/account-renew/index.tsx"
    ]
}
[Trace  - 9:54:44 AM] Event received: syntaxDiag (0).
Data: {
    "file": "/src/the-pill-club/web-infrastructure-2/sites/dashboard/src/components/Greeting.tsx",
    "diagnostics": []
}
[Trace  - 9:54:44 AM] Event received: semanticDiag (0).
Data: {
    "file": "/src/the-pill-club/web-infrastructure-2/sites/dashboard/src/components/Greeting.tsx",
    "diagnostics": []
}

I notice it is returning an empty diagnostics list after this, is it possible the exception is a red herring and coc-tsserver is not responding properly in this case?
Playground Link:
Related Issues:
This issue has come up before. #21818, #16456, #20799.

Can you please share tsserver log when the error reproduces for us to be able to investigate this.
Is there sample code and edits that follow that cause this.
You can upload logs by following these steps:

Set "typescript.tsserver.log": "verbose"
Restart vscode or vscode insiders
Reproduce the problem
In vscode or vscode insiders, run the TypeScript: Open TS Server log command
This should open a large file called tsserver.log. Please share this file. Additionally you can also share file ti.log created next to it.


Bug in inserting link in text editor

Framework7 version: 5.0.4
Platform and Target: Latest chrome browser, macOS
Live Link: https://framework7.io/docs/text-editor.html

Describe the bug
The insert link button for the text editor is broken. If you select it with no text selected, and insert a URL, just blank lines are inserted. If you have text selected, and select insert link, then the text is replaces with several blank lines. No link is inserted.
To Reproduce
Steps to reproduce the behavior:

Go to the posted links to the docs for the text editor
Go to the live demo
Try inserting a link

Expected behavior
A link should be made/inserted
Actual Behavior
Blank lines are inserted

Nope, can't replicate it. Would be good to see more complete replication steps or screen recording

Closing due to inactivity and/or no responses from issue author


Domains for 7 More Emerging Markets

WIP: This is currently blocked on the v1.11.0 release of trading calendars.
Adds Indonesia, Malaysia, Pakistan, Philippines, Russia, Taiwan, Thailand.

Coverage increased (+0.2%) to 87.749% when pulling 9c96916fc56647157ac4b5d9bb7bea573c68ebd4 on more-emerging-markets into f0e34f88f9f80884295f005d303d433ab504cd81 on master.

Coverage increased (+0.2%) to 87.749% when pulling 9c96916fc56647157ac4b5d9bb7bea573c68ebd4 on more-emerging-markets into f0e34f88f9f80884295f005d303d433ab504cd81 on master.


Remove get_ compatibility layer

This has been deprecated for some time now. Getting the attribute manually is now the supported way.

This PR seems like a good candidate for also changing the way getters/setters are in Manim - see the Discord developement thread


GLXBadContext when using mujoco.viewer.launch_passive()

Hi, I'm running into this error when running episodes with the viewer
X Error of failed request:  GLXBadContext
  Major opcode of failed request:  152 (GLX)
  Minor opcode of failed request:  4 (X_GLXDestroyContext)
  Serial number of failed request:  355
  Current serial number in output stream:  355
X Error of failed request:  GLXBadContext
  Major opcode of failed request:  152 (GLX)
  Minor opcode of failed request:  4 (X_GLXDestroyContext)
  Serial number of failed request:  355
  Current serial number in output stream:  355

I'm not entirely sure what could be causing this.
My code for launching the viewer is as follows:
import mujoco as mj
import mujoco.viewer as mjv

def run_episode(path, episode_length=2000):
  m = mj.MjModel.from_xml_path(str(path))
  d = mj.MjData(m)

  t = 0
  data = []

  with mjv.launch_passive(model, data) as viewer:
    while viewer.is_running() and t < episode_length:
      # policy code omitted

      mj.mj_step(m, d)
      viewer.sync()

      # data collection code. collect omitted
      data.append(collect(m, d))

      t += 1
      time.sleep(0.01)

  return data

if __name__ == '__main__':
  run_episode(sys.argv[1])

I have not included the mjcf, but can provide it if that may be contributing to the issue.

Include the following context:

Ubunutu 22.04
Python 3.10.13
mujoco.mj_version() returns 300
Using official python bindings

have same problem.
`import time
import mujoco
import mujoco.viewer
global angle
def key_callback(keycode):
global angle
if chr(keycode) == '1':
angle += 0.1
m = mujoco.MjModel.from_xml_path('./kayraCircle.xml')
d = mujoco.MjData(m)
angle = 0
#with mujoco.viewer.launch_passive(m, d) as viewer:
with mujoco.viewer.launch_passive(m, d, key_callback=key_callback) as viewer:
Close the viewer automatically after 30 wall-seconds.
start = time.time()
while viewer.is_running() and time.time() - start < 30:
step_start = time.time()
# mj_step can be replaced with code that also evaluates
# a policy and applies a control signal before stepping the physics.
mujoco.mj_step(m, d)

# Example modification of a viewer option: toggle contact points every two seconds.
with viewer.lock():
  viewer.opt.flags[mujoco.mjtVisFlag.mjVIS_CONTACTPOINT] = int(d.time % 2)
  d.actuator("Right Calf 3BR").ctrl = 1.57 - angle

# Pick up changes to the physics state, apply perturbations, update options from GUI.
viewer.sync()

# Rudimentary time keeping, will drift relative to wall clock.
time_until_next_step = m.opt.timestep - (time.time() - step_start)
if time_until_next_step > 0:
  time.sleep(time_until_next_step)`

X Error of failed request:  GLXBadContext
Major opcode of failed request:  152 (GLX)
Minor opcode of failed request:  4 (X_GLXDestroyContext)
Serial number of failed request:  1042
Current serial number in output stream:  1042
X Error of failed request:  GLXBadContext
Major opcode of failed request:  152 (GLX)
Minor opcode of failed request:  4 (X_GLXDestroyContext)
Serial number of failed request:  1042
Current serial number in output stream:  1042

What graphics card do you have out of curiosity?

I have an NVIDIA RTX 3090, also running Ubuntu 22.04

Sorry for the late reply, I just returned to work. Can you identify which line of code is triggering the error?

Going to close due to inactivity. Please re-open if this is still an issue.


<EMAIL_ADDRESS>

seems to be continuously deploying with the same exemption, must restart ram issue on my side

All sorted was a load issue on my end


Adding constants for doorbell service

Hey guys I couldn't find anything for the doorbell service.  I found the values in the HAP-Spec.  Let me know if it needs adjustment.

hi,@kellyp, as you may have noticed, this public repository is for 'Hobbyists and Makers', and non-commercial usage, so a lot of services are not contained.

Oh I see.  I am a bit unsure why hobbyists wouldn't want to mess with doorbells.


List unlocking answer is not being evaluated

From @ksteph
List Slicing > Suite 1 > Case 1
>>> x = [1, 2, 3, 4]
...
>>> x[::-1]
? [4, 3, 2,1]
-- Not quite. Try again! --

? [4, 3, 2, 1]
-- Not quite. Try again! --

? [4,3,2,1]
-- OK! --

This should have been handled by https://github.com/Cal-CS-61A-Staff/ok-client/pull/101; will investigate further.

If the user input doesn't match the expected answer exactly, then repr(eval(input)) is compared to the expected answer. However, in order for this to work, the expected answer has to be in its canonical form. I think the bug above is due to the expected answer being [4,3,2,1] when it should be [4, 3, 2, 1].

Just checked the sample test, and it seems your right. Thanks!


goghud

Also it seems like you haven't added the pictures listed under assets, please make sure you upload those


"Collapse useless groups" collapsing not useless groups

Hello! This issue was noticed in svgo v1.0.5.
Option "collapseGroups" (or "Collapse useless groups" in SVGOMG) removing  tags and filter attributes to images, that does not work -- image doesn't displayed properly.
Attached file demonstrates the problem -- just try to compress it with and without "collapseGroups" option.
Charivniy-Vechir.zip

I'm running into this same issue.
  <defs>
    <filter id="a">
      <feDropShadow dx="0" dy="25" stdDeviation="25" flood-opacity="0.65" />
    </filter>
</defs>

is being turned into this:
<defs><filter id='a'/></defs>

Fixed in https://github.com/svg/svgo/releases/tag/v2.6.1


Inline Series.astype documentation

Codecov Report

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

@@            Coverage Diff             @@
##           master     #337      +/-   ##
==========================================
- Coverage   94.31%   94.28%   -0.03%
==========================================
  Files          35       35
  Lines        3485     3485
==========================================
- Hits         3287     3286       -1
- Misses        198      199       +1

Impacted Files
Coverage Δ

databricks/koalas/series.py
93.05% <ø> (ø)
:arrow_up:

databricks/koalas/__init__.py
88.88% <0%> (-3.71%)
: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 a831052...aa3bba5. Read the comment docs.

LGTM other than the two minor comments .... and this reminds me, we should implement DataFrame.astype as well.


Error starting the project after installation

Environment

Operating System: Windows_NT
Node Version:     v20.11.1
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      modules, typescript
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
No response
Describe the bug
The error "Cannot find package 'next-auth' imported from D:\sidebase-p.nuxt\dev\index.mjs" is reproduced immediately after installing the Merino Stack. It is strange that the error appears immediately after installing the stack. All dependencies are installed.

Additional context
No response
Logs
WARN  "next-auth/providers/github" is imported by "server/api/auth/[...].ts", but could not be resolved – treating it as an external dependency.                             00:30:25

 WARN  "next-auth/providers/credentials" is imported by "server/api/auth/[...].ts", but could not be resolved – treating it as an external dependency.                        00:30:25

[00:30:25]  WARN  "next-auth/core" is imported by<EMAIL_ADDRESS>but could not be resolved – treating it as an external dependency.

[00:30:25]  WARN  "next-auth/jwt" is imported by<EMAIL_ADDRESS>but could not be resolved – treating it as an external dependency.

✔ Nuxt Nitro server built in 1407 ms                                                                                nitro 00:30:25

 ERROR  [worker reload] [worker init] Cannot find package 'next-auth' imported from D:\sidebase-p\.nuxt\dev\index.mjs                                                         00:30:25

  at packageResolve (node:internal/modules/esm/resolve:853:9)
  at moduleResolve (node:internal/modules/esm/resolve:910:20)
  at defaultResolve (node:internal/modules/esm/resolve:1130:11)
  at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:396:12)
  at ModuleLoader.resolve (node:internal/modules/esm/loader:365:25)
  at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:240:38)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:85:39)
  at link (node:internal/modules/esm/module_job:84:36)

Have you added next-auth as a dependency?
https://sidebase.io/nuxt-auth/getting-started/installation#specifics-authjs-provider


SZCC improvements to exe/lib detection

Improve detection of executables

Check for links back to self; ignore and retry search without that path

Pushed updated commit


Overriding the namespace prefix with empty prefix

In some cases, we need to remove namespace prefix for generating valid soap request message.
I encountered this problem with complex WSDL.

Remove namespace prefix of param

  client.MyService.MyPort.MyFunction({':name': 'value',}, function(err, result) {
      // request body sent with `<name`, regardless of what the namespace should have been.
  }, {timeout: 5000})

Thanks @SunJang !  I feel this is a great feature.

Thoughts @herom ?

@jsdevel So..What should I do? request another PR?

Thanks for your contribution @SunJang !