Mapping bypass_auth not always honored

Describe the bug
I have an AuthService with a Mapping that has bypass_auth set to true. Sometimes a request to auth service goes through AuthService instead of the Mapping.
The situation where we're frequently seeing this is:

Browser requests /app without jwt cookie
ext_authz request sent to /auth
Auth returns 302 to /login
Login returns 302 to /auth/callback
Auth is sent an ext_authz request instead of the full browser request
The OAuth callback fails because the Content-Type header is missing
If we retry the request from the browser, it usually succeeds and our Auth service is sent the full, non ext_authz request. But sometimes we can retry up to 5 times before it goes through correctly

Our Auth service manifest:
apiVersion: getambassador.io/v3alpha1
kind:  Mapping
metadata:
  name: auth
spec:
  host: "*"
  prefix: /auth/
  service: http://auth.default:80
  bypass_auth: true
  rewrite: /auth/
---
apiVersion: getambassador.io/v3alpha1
kind: AuthService
metadata:
  name: auth
  namespace: default
spec:
  auth_service: "auth.default:80"
  proto: http
  timeout_ms: 5000
  include_body:
    max_bytes: 4096
    allow_partial: true
  status_on_error:
    code: 502
  failure_mode_allow: false

To Reproduce
Steps to reproduce the behavior:

Configure an auth service like above with request logging
Log in through that auth service
You'll likely see that the auth service was sent an ext_authz request on the callback redirect

Expected behavior
The full browser request makes it to the callback endpoint on the auth service
Versions (please complete the following information):

Ambassador: 3.3.0
Kubernetes environment: Azure Kubernetes Service
Version: 1.24.6

Additional context
Comparison of original request (authz) and browser refresh:

Workaround
Allow the accept and content-type header to be passed through to the authz endpoint by adding this to the AuthService definition:
  allowed_request_headers:
    - accept
    - content-type

Hi @carlin-q-scott, thanks for raising this issue and the workaround.  I wouldn't think ext_authz would be interfering if bypass_auth is used but perhaps there is some lag/ordering issue in chain of events.  Also found this Envoy issue which may also be related.

@cindymullins-dw Yeah, looks like they implemented the same workaround as me. They were also using emissary-ingress.
I don't think I can give much input on the lag/ordering you mention since I don't know the internal workings of emissary and envoy. Are the request chains executing in parallel and first one to respond wins? So when I make a request to my bypass_auth Mapped service, emissary goes ahead and makes a request to the AuthService and would cancel that request if the target service responds faster? One theory I had was that emissary or envoy is reusing the request forwarder for the AuthService when fulfilling the request to the Mapped service since they are in fact the same service.


Fixing #141

Fixing default_box doesn't first look for safe attributes before falling back to default (thanks to Pymancer)

Coverage increased (+0.0006%) to 99.875% when pulling 733dfa199bacee324c0383f87204a5b04db92fa0 on bugfix-141 into 9685a2e08a654b2c20bb2a00d94eb3dcbc6471ce on development.


Add sync-WsusClient to force computers to check in with update server

I use this one all the time to make sure my servers are in sync with the WSUS server(s). I thought it would fit well in your collection! Please let me know any feedback. Thanks!

Thanks @rpstester !


Why are the hidden states for each layer the same as the transformer hidden states instead of the language adapter's hidden state?

While trying to check the hidden states before and after the language adapter in XLM-Roberta (using AutoModelForMaskedLM), I found that the output of the language adapter is not the hidden state of the layers, but instead the hidden state of the Transformer model is the hidden states for each layer.
Below is my code snippet where I checked the output for the Transformer's hidden state and
from transformers import AutoAdapterModel
from transformers import AutoTokenizer, AutoModelForMaskedLM

# A simple hook class that returns the input and output of a layer during forward/backward pass
class Hook():
    def __init__(self, module, backward=False):
        if backward==False:
            self.hook = module.register_forward_hook(self.hook_fn)
        else:
            self.hook = module.register_backward_hook(self.hook_fn)
    def hook_fn(self, module, input, output):
        self.input = input
        self.output = output
    def close(self):
        self.hook.remove()

tokenizer = AutoTokenizer.from_pretrained("xlm-roberta-base")
model = AutoModelForMaskedLM.from_pretrained("xlm-roberta-base")
model.load_adapter("en", model_name = 'xlm-roberta-base')
model.set_active_adapters("en")

However, the hidden states from each layer are the same as the output from the Transformer's hidden states.
For example
module1 = list(model.get_adapter('en').items())[1][1]  #adapter module output
hook1 = Hook(module1['output_adapter'])
module2=model.roberta.encoder.layer[0].output.LayerNorm # roberta encoder output
hook2 = Hook(module2)

input = tokenizer("Test input", return_tensors="pt")
output = model(**input, output_hidden_states=True)

print("Adapter module output = ", hook1.output[-1])
print("Roberta FFN output = ", hook2.output)
print("encoder hidden state = "output['hidden_states'][1])

hook2.output is the same as "output['hidden_states'][1]. Why is this so? Are the transformer hidden states not the input to the language adapters?

Hey @ajesujoba,
In general, your assumption that the Transformer's hidden status are the inputs to the adapters is correct.
Your sample code shows the same output because of the specifics of the used adapter config. The "pfeiffer" config used for the lang adapter applies the original layer norm after the adapter module (see original_ln_after in https://docs.adapterhub.ml/_images/architecture.png). Thus, if you hook into the layer norm of the Transformer layer, you get the hidden states after forwarding through the adapter. For retrieving the Transformer layer hidden states before the adapter layer, you could e.g. hook into model.roberta.encoder.layer[0].output.dense.
(Alternatively, you should also be able to see different hidden states when adding an adapter where original_ln_after is set to False in the config.)
Hope this clears things up!

This issue was closed because it was stale for 14 days without any activity.


Mark hover and floating markdown buffers to avoid loading ftplugins

I was hitting a performance issue, I realized that auto completion was very slow, when I set the profiler I realized that my ftplugin/markdown.vim was the root cause. I have quite hefty ftplugin, plus multiple plugins that are loaded for markdown buffers.
This is an issue because these buffers are created to display signature info, it would be great if there was a way to mark these buffers and floating windows so that it could be avoided to load ftplugin and other plugins on these buffers.
This is probably a problem for nvim-lspconfig as well.
What I am thinking is probably setting something like: b:floating_signature or something of the sort, that could be checked in the after/ftplugin/markdown.vim and not source the file if found. Ideally these could be a special  kind of window where no after/* plugins are loaded, I am thinking like nerdtree/ctrlp/tagbar that have special window names that can ignored in multiple plugins.

Hmm I think that's because the hover and signature help callbacks set the floating filetype as markdown. A workaround is to manually trigger hover and signature by disable the auto hover and auto signature. Setting a buffer variable shouldn't be too hard though.

I've setup a buffer variable b:lsp_floating to true for the floating window in hover and signature_help. Please check if it resolve your issue.

Thank you. This does fix the performance problems. However does require user action. I'll experiment with seeing these buffers to a different filetype and then applying the markdown highlight.


Use memory mapped files for minidump

Contributes to #660
Interestingly enough, the simplest use of MMF increases performance significantly for my test case.
The MMF doesn't allow any file sharing by default so there can only be one stream at the same time. (Why do we need two streams?) I made MinidumpReader the IDataReader responsible for the disposal.

Thanks!  I haven't had a chance to review it yet, but one concern I have is memory usage when inspecting large crash dumps.
I haven't used MemoryMappedFile before, so could you check what happens when you load a very large x86 dump file of a process (like with the /3gb bit set)?  Does it consume the entire address space or just a reasonable amount of it?
One of the reasons I started with a Stream (and hoped eventually to do some memory mapping) is that loading the whole file into the address space would run OOM in some x86 scenarios.  This is a super common use-case for ClrMD at Microsoft (inspecting Visual Studio dumps).  I want to make sure we don't eat too much memory.

I'd guess it will take up the entire address space. Let's make this a 64-bit specific feature?

I'd guess it will take up the entire address space. Shall we make this a 64-bit specific feature?

I also am a little concerned about 32gb dumps out of SQL.  I would suggest instead making it a feature based on dump size.  For example, if the filesize of the dump we are loading is bigger than 256 megs on x86 or 4gb on x64 we fall back to streams?  (I picked those numbers at random, please feel free to choose better ones.)
We can always go back and let the user tune the numbers themselves later through CachingOptions.

I will say my original plan for memory mapped files ages ago was to use a Least Recently Used cache to map segments into memory and keep them there for a while.  It'd let us use memory mapped files larger than we would allow with a filesize based approach, but it's also a lot more complicated to code up.
I'd say long term I'd still like to do that, but since it doesn't affect the public surface area of the API it's something I could always go back and do later.

to use a Least Recently Used cache to map segments into memory and keep them there for a while.

Yeah it would be my preference as well.

I would suggest instead making it a feature based on dump size. For example, if the filesize of the dump we are loading is bigger than 256 megs on x86 or 4gb on x64 we fall back to streams

This can act as a transition. The size threshold on x64 can be much larger as the address space limit is in the TB level (128 TB on Windows 10).
I'll hold this for a while until you finish the perf testing to make sure this really improves performance.

Sounds great!  I'll take a look at this as soon as I can get back to working on ClrMD.

@NextTurn Still a draft, or ready to merge?

@leculver Please feel free to merge.


Update POSIX Dependency to 5.x

Hello, i just got an issue uodateing to DART 3.x with your lib.
POSIX 4.1.x doesen't seem to work with dart 3, i wasn't able to compile.
Updating the dependency to posix 5.x worked well, please update your pubspec to the following:
dependencies:
  posix: ^5.0.0

Fixed with shelf_unix_domain_socket ver 1.1.1 .
Thank you for your patience.


Allow "selfupdate" to specify specific version

embulk selfupdate command gets the latest version.
When the latest version caused a problem, I want to revert back to old version.
In this scenario, embulk selfupdate <version> is very good to have.

I try to implement this feature this branch: https://github.com/cosmo0920/embulk/tree/specific-version-selfupdate.
But this branch will be conflict for selfupdate subcommand fix.
After selfupdate subcommand fix at master, I will create PR with this branch.

I've merged selfupdate fix.

https://github.com/embulk/embulk/compare/master...cosmo0920:specific-version-selfupdate#diff-dcb0e3c8126ddf8d77d7de608e44e48aR257
** why don't you include :version in options?
https://github.com/embulk/embulk/compare/master...cosmo0920:specific-version-selfupdate#diff-0f4dfcf1becb38c679f5f5e979faadc7R86
https://github.com/embulk/embulk/compare/master...cosmo0920:specific-version-selfupdate#diff-0f4dfcf1becb38c679f5f5e979faadc7R86
** 404 Not Found is expected error because user can specify version which doesn't exist. So, how about saying "Specified version does not exist. See https://github.com/embulk/embulk/tree/master/embulk-docs/src/release for released versions" if status is 404?

master...cosmo0920:specific-version-selfupdate#diff-dcb0e3c8126ddf8d77d7de608e44e48aR257

why don't you include :version in options?

Ah! I didn't notice about it.

master...cosmo0920:specific-version-selfupdate#diff-0f4dfcf1becb38c679f5f5e979faadc7R86

404 Not Found is expected error because user can specify version which doesn't exist. So, how about saying "Specified version does not exist. See https://github.com/embulk/embulk/tree/master/embulk-docs/src/release for released versions" if status is 404?

I see. I think that above exception message is better.


test(transformer): prevent lint error when running miri

#7873  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

Trivial change, so merging without review.

Merge activity

Dec 13, 10:09 PM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


added my self

add 'hacktoberfest-accepted' tag on merge request

Hi @Almas-Ali Please add something use full also then pull a combine request.

ok


Edit single order modal

I changed the dynamic page title, modified the modal content for edit order to form, prop binding for the item name in item list of current order.

looks good 👍


Class 'Searchy' not found

I followed the instruction to install this package but I can't make it works.
I'd like to use it in my RegisterController.php
namespace App\Http\Controllers\Auth;

use Searchy;

class RegisterController extends Controller {
    public function example () {
        Searchy::search('companies')->fields('name')->query('test')->getQuery()->limit(1)->get();
    }
}

I got the error Class 'Searchy' not found

use TomLingham\Searchy\SearchBuilder; .. Use this ...

Here is what I got : Class 'App\Http\Controllers\Auth\Searchy' not found


Features/1.12/texture refresh

Pull Request Description
Update wolf armor textures to make them more visually congruent with Minecraft base armors.

Fixes #195


Add support for VSCode 1.70.x again please

Hi, thanks for developing the amazing extension!
I’m now requested to use VSCode version not newer than 1.70.x, while twinny not newer than v2.6.21 supported VSCode 1.70.x, making new features not available any more. Would you please add support for lower VSCode version again?

Please could you provide more instructions as to what is not supported, many thanks.

@rjmacarthy thanks a lot for the quick reply! In version v2.26.1, only ollama is allowed to be provider, while custom OpenAI api like LMStudio or some other type of local custom OpenAI api is needed. I'd like to try the newest version of twinny but it's not competitive for vscode 1.70.x

Hmm, that is not correct you can configure many different providers in the latest version.  Please check the docs.  Many thanks.


the ref_convolution.cl implementation not aligned to the mkl-dnn dev guide

https://github.com/intel/mkl-dnn/blob/master/src/ocl/ref_convolution.cl#L46
            const int id = od * SD - PD + kd * (1 + DD);
            const int ih = oh * SH - PH + kh * (1 + DH);
            const int iw = ow * SW - PW + kw * (1 + DW);

this implementation I think has some problem;  and also not aligned to
[http://intel.github.io/mkl-dnn/dev_guide_convolution.html]

And I think the formula in the dev guide also is wrong.
I think the formula should be
            const int id = od * SD - PD + kd * DD;
            const int ih = oh * SH - PH + kh * DH;
            const int iw = ow * SW - PW + kw * DW;

please help check this issue. Thanks!

Hi @joye,
Thanks the report!
Actually the problem is in the documentation, and not in the source code. In DNNL we use the convention dilation = 0 for regular convolution. Hence the formula in users guide should be:
dst(..., oh, ...) = ... src(..., oh + kh * (DH + 1) - PH_0, ...)

Please note, that other libraries or frameworks may use dilation = 1 for regular convolution, e.g. this is the case for PyTorch.

Thanks for explanation, so the dilation in Pytorch or Tensorflow models will be dilation-1 in DNNL? If this is true, can close this question.

Hi @joye,

Thanks for explanation, so the dilation in Pytorch or Tensorflow models will be dilation-1 in DNNL?

Yes, that's correct.
I will leave the question open till we promote the fix to the documentation.
Thanks for noting the issue!


CNN for Text -- Reshape(), Flatten() issues

Hello again ---
I am training a CNN for a text classification task. I have documents, each with a maximum n_words, and I am calling pretrained word2vec vectors of dimension D for each word.
I am trying to pass documents to the network, which are (padded to) n_words by D. I am only using one type of embedding, so I only have one channel.
Unlike the canonical MNIST examples, I want the convolutions to happen over words taken over the entire word2vec embedding width D. If I try to set the kernel=c(5,D), though, I get the error: "kernel size exceed input". If I instead pass kernel=c(5,D,1,1), the code runs without error.
Is this the way to make sure that the kernel is being taken over a window of words (i.e., 5 words) but over all 100 dimensions of the word embedding vectors? Or am I seriously misunderstanding how to pass data to the CNN?
Thanks!

https://github.com/dmlc/mxnet/tree/master/example/cnn_text_classification this example might be quite related

Thanks! That is related. I've been basing my model on the one in that example. In fact, I basically just copied it and changed the python conventions to R.
Here's the issue I'm having: when I get to the
total_filters = num_filter * length(filter_list)
concat = mx.symbol.Concat(pooled_outputs, num.args=length(filter_list), dim=1)
h_pool = mx.symbol.Reshape(data=concat, target_shape=c(batch_size, total_filters))

The last line, using Reshape(), gives me an error, from.shape() == to->shape() operands shape mismatch I've tried changing the arguments to the Reshape() and nothing works.
If, instead, I use the mx.symbol.Flatten(concat) (as in the MNIST examples), the model will run, but the results are also way off. But that could just be my data or the untuned model parameters.
I can't seem to find what "Flatten" is doing, or if it's accomplishing what the text classification example was trying to do with Reshape(). If not, how do I use Reshape() correctly?
If it's helpful, here's the model code. The input data is an array, 50 (sentence length) by 300 (word2vec embeddings) by 1 (channel) by 30000 (training samples). The target is dichotomous.
# fit the CNN
library(mxnet)
pooled_outputs = c()
sent_length = 50
data = mx.symbol.Variable("data")
filter_list = c(3,4,5);
batch_size=100;
num_filter=100
for(f in 1:length(filter_list))
{
  convi = mx.symbol.Convolution(data=data, kernel=c(filter_list[f],300),num_filter=num_filter)
  relui = mx.symbol.Activation(data=convi, act_type='relu')
  pooli = mx.symbol.Pooling(data=relui, pool_type='max', kernel=c((sent_length-filter_list[f]+1),1))
  pooled_outputs = c(pooled_outputs,pooli)
}
total_filters = num_filter * length(filter_list)
concat = mx.symbol.Concat(pooled_outputs, num.args=length(filter_list), dim=1)
#h_pool = mx.symbol.Reshape(data=concat, target_shape=c(100, total_filters)) # Doesn't work?
h_pool = mx.symbol.Flatten(data=concat)
h_drop = mx.symbol.Dropout(data=h_pool,p=0.2)
fc1 = mx.symbol.FullyConnected(data=h_drop, num_hidden=100)
act1 = mx.symbol.Activation(fc1,act_type='sigmoid')
drop1 = mx.symbol.Dropout(act1,p=0.2)
fc2 = mx.symbol.FullyConnected(drop1, num_hidden=2)
# softmax output
sm = mx.symbol.SoftmaxOutput(data=fc2, name='softmax')
mx.set.seed(0)
devices=mx.cpu()
model = mx.model.FeedForward.create(sm, X=x,
                                    y=as.numeric(as.factor(subideoForCNN)),
                                    ctx=devices,num.round=200,array.batch.size=batch_size,
                                    learning.rate=0.005,momentum=0.9,
                                    wd=0.00001,eval.metric=mx.metric.accuracy,
                                    initializer=mx.init.uniform(0.01),
                                    lr_scheduler=FactorScheduler(step=batch_size*50, factor = 0.95),
                                    eval.data=list(data=xt,
                                                   label=as.numeric(as.factor(subideoTest))))

@bdlacree  does your input_shape is (100, 1, 50, 300) ?

The total data input shape is (50,300,1,30000), with batchsize 100. The model seems to work on training and test data, but I just want to be sure that the Flatten() is doing what it should.

@bdlacree  could you provide the dataiter code? I think your input shape may be wrong,
you can run the following code to see what Flatten is doing:
import mxnet as mx
import numpy as np
x = mx.symbol.Variable('data')
shape = (2, 2, 3, 4)
data_tmp = np.ones(shape)
arr_data = mx.nd.array(data_tmp)
arr_grad = mx.nd.empty(shape)
test = mx.symbol.Flatten(data=x)
exe_test = test.bind(mx.cpu(), args=[arr_data], args_grad=[arr_grad])
exe_test.forward()
print exe_test.outputs[0].shape

@bdlacree: Did you manage to replicate the example in R? I'm having a hard time doing so and would love to see your version if you managed to do it?
Kr,
Daniel

Closing for now. Please feel free to reopen if the needed.


ID3v2: Popularimeter

Examples:
Popularimeter                            : 5 stars (255/255, MusicBee)

Popularimeter                            : 1 star (1/255, Windows Media Player 9 Series)

I keep the raw number because the detail may be useful.
Fix https://github.com/MediaArea/MediaInfo/issues/620, cc @stax76

That will be useful for me, thank you. The other format I use is opus (YouTube), and there, Rating is already supported by mediainfo.

The other format I use is Opus (YouTube), and there, Rating is already supported by MediaInfo.

I guess that this is by a "pass through" of unknown Vorbis comment, please share a sample file so I can have coherency between formats.

Art Farmer - Soulsides.zip
In the app Mp3tag it shows this:

Unfortunately, I cannot tell you how it was tagged because currently I experiment with various tools because I want to give up MusicBee and migrate to open source tools such as mpv.net.


LZ helper users should be able to deploy aks cluster with custom azure policy initiative

Is your feature request related to a problem? Please describe.
Customers usually use custom policies in an enterprise setting. LZ helper should have an option to deploy aks cluster with custom azure policy intiative
Describe the solution you'd like
LZ helper with provide an option to add custom policy initiative resourceId
Additional context on the request

Currently LZ helper can add only Baseline and Restricted PSP initiative to AKS cluster. But if user wants to deploy AKS cluster with custom azure policy initiative then they won't be able to do so.

So the environment will already have a custom policy definition/initiative. (as per these docs)
All the helper will need to do is make an assignment with the right effect against the custom policy.


Adding nitpicky option

I think the nitpicky option should be turned on sooner rather than later. Right now there are about ~100 broken links already in the docs.
If you prefer I can try to fix them during the slightly toxicated sprint moments this week ;)

@bsipocz – If you can clean up those warnings, that would be great!
Today is probably not a good time though ... there's a few PRs on the docs already in progress that will get merged later today.

OK. After the PR's are merged I'll clean it up during the weekend. It's
getting late anyway, and I'm already in the deepest of multitasking...
On 28 August 2015 at 21:54, Christoph Deil<EMAIL_ADDRESS>wrote:

@bsipocz https://github.com/bsipocz – If you can clean up those
warnings, that would be great!
Today is probably not a good time though ... there's a few PRs on the docs
already in progress that will get merged later today.
—
Reply to this email directly or view it on GitHub
https://github.com/astropy/astroplan/pull/82#issuecomment-135885701.

@cdeil - would you ping me on this once you merge the those PRs you mentioned above? Then I'll try to do this before you release.

The other PRs you should wait for is mainly #77  and probably also #79.
I'll ping you here once they are in.

@bsipocz - the two PRs @cdeil mentioned are done... But #87 now also touches some of the docs.  It may not conflict with what you're planning here, though, because #87 is mostly touching code blocks rather than narrative docs (where you are likely to find most of the links).

@bsipocz - Thanks for doing this.
There's some nitpicky warnings left that make the travis-ci build fail:
https://travis-ci.org/astropy/astroplan/jobs/78436519#L746
@bmorris3 as 0.1 release manager – This is currently labeled 0.1, but IMO can go in 0.2 if you want to cut the release today.
(There will likely be other things we miss and new features soon, so we can release 0.2 in a few weeks if we like.)

@cdeil - yes I know, but wanted to commit the stuff I had before leaving
the office (also to see what's is still missing as my local build is having
some SSL issues).
On 2 September 2015 at 21:48, Christoph Deil<EMAIL_ADDRESS>wrote:

@bsipocz https://github.com/bsipocz - Thanks for doing this.
There's some nitpicky warnings left that make the travis-ci build fail:
https://travis-ci.org/astropy/astroplan/jobs/78436519#L746
@bmorris3 https://github.com/bmorris3 as 0.1 release manager – This is
currently labeled 0.1, but IMO can go in 0.2 if you want to cut the release
today.
(There will likely be other things we miss and new features soon, so we
can release 0.2 in a few weeks if we like.)
—
Reply to this email directly or view it on GitHub
https://github.com/astropy/astroplan/pull/82#issuecomment-137240224.

@bsipocz Thanks for this! I've moved this to 0.2 so this doesn't block 0.1.

@bmorris3 - sure.
FYI: I've temporarily disabled most of the travis builds, since as I mentioned my local has a mess up. If I manage to get a pass on it tonight, I'll ping you.

@bmorris3 - This should pass now.

Travis-ci tests passed.
:+1:  to merge.
Thanks!
This might conflict with #87, so whichever is merged first, the other might have to be rebased.

Especially after #92 has been merged, this will need a rebase. Sorry for the extra work @bsipocz!

Rebased, and travis runs passed (This latest one can be cancelled if you like, as the only difference is that I've rebased it again to remove the temporary minimization of the travis matrix)

@eteq - I've included the new AstroplanWarning class in the public API, thus the other warning classes can resolve its link for their baseclass. Hope you don't mind.

@bsipocz: Beautiful! Looks good, merging.


LSP[id=1] client has shut down after sending the message - when opening a haskell (.hs) file

nvim --version: NVIM v0.5.0-639-g46e74142a-dirty
nvim-lsp version(commit hash): latest version (executed : PlugUpdate)
:checkhealth result: -> https://pastebin.com/AcenMLg2
What language server(If the problem is related to a specific language server):
hie - > Haskell IDE Engine
Operating system/version: Gentoo Linux Kernel  5.3.2
Terminal name/version: rxvt-unicode
$TERM:xterm-256color

How to reproduce the problem from neovim startup
-Install stack
-Install ghc version 8.8.3
-Install hie version 8.8.3
-Install hoogle
-Install neovim builtin lsp
-For more accuracy, run neovim with the following dotvim configs:
aesthetics.vim:
https://pastebin.com/p5eDf1ZE
general.vim:
https://pastebin.com/aCvdEmSs
mapped.vim:
https://pastebin.com/PVry35Dt
initial_plugins.vim:
https://pastebin.com/SBYT7pjf
lsp_config.vim:
https://pastebin.com/PekTjkRE
init.vim:
https://pastebin.com/jxb5gcK6
-Open any haskell file inside a haskell project directory (run hoogle generate prior!)
Actual behaviour

When opening any haskell file inside a project directory, the following error message gets thrown by neovim during startup:
"LSP[id=1] client has shut down after sending the message"
Virtual text for errors seem to work fine..

For some functions, when you press HoverDocumentation button, documentation appears:

complementing neovim lsp with completion-nvim in haskell partially works. hie lsp references appear during completion asynchronously, although when hovering over the selection in the popup menu, no documentation appears (not sure if this is expected behaviour ):

Expected behaviour
The expected behaviour should be: No error should occur when opening a haskell file in neovim. if HIE supports it, Hovering documentation should be appearing when selecting a suggestion from the completion popup-menu.
Example of *expected method documentation in popup menu for hie and haskell (but using pyls and a python file):

Thanks for your report. The configs here are best-effort:

It is hoped that these configurations serve as a "source of truth", but they are strictly best effort. If something doesn't work, these configs are useful as a starting point, which you can adjust to fit your environment.

If you have an improvement or fix, please send a PR!
If you have a specific question we'll try to answer it (keeping in mind the configs in this repo are best-effort and unsupported).
If you found a bug in the core lsp module, the best way to get it fixed is to describe steps to reproduce it without the particular LSP server and other factors particular to your environment. Or better, by adding a failing test case to test/functional/plugin/lsp_spec.lua, which has code to setup a fake LSP server to simulate various scenarios.

Thank you @justinmk . I've tried configuring the lsp client for hie but it's been difficult for me since I'm an absolute beginner with neovim-lsp. I'll keep trying configuring hie (or maybe attempt to use ghcide) since nvim lsp works really well for other language servers for me, except for a few (Haskell, ruby and go).


executor: sys/linux: arm64: implement SYZOS_API_MEMWRITE

The new API call will be used to write values to guest memory specified by base+offset.
Writing to e.g. MMIO registers for VGIC (or any other MMIO ranges) may result in new coverage.

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

PTAL


update generated bin files names

About this change - What it does
simplifies generated RPM executable name to guardian-(backup|restore) instead of guardian-for-apache-kafka-cli-*

Why this way
guardian-for-apache-kafka-cli-* is a bit of a tongue twister

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 75.683%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
277

Relevant Lines:
366

💛  - Coveralls


Update to Swift 4.2

Maybe think about updating to Swift 4.2

Now comes Swift 5.2

I am doing it myself, in my own branch


Unable to use @include mat-core(); in my style.scss in Angular4 Webpack 2

Bug, feature request, or proposal:
Unable to use @include mat-core(); in my style.scss in Angular4 Webpack 2. here is my package.json's dependencies and devDependencies.
"dependencies": {
"@angular/animations": "^4.0.2",
"@angular/common": "^4.0.2",
"@angular/compiler": "^4.0.2",
"@angular/core": "^4.0.2",
"@angular/forms": "^4.0.2",
"@angular/http": "^4.0.2",
"@angular/material": "^2.0.0-beta.3",
"@angular/platform-browser": "^4.0.2",
"@angular/platform-browser-dynamic": "^4.0.2",
"@angular/platform-server": "^4.0.2",
"@angular/router": "^4.0.2",
"@angularclass/conventions-loader": "^1.0.13",
"@angularclass/hmr": "^1.2.2",
"@angularclass/hmr-loader": "^3.0.2",
"angular2-cookie": "^1.2.2",
"async": "^2.4.0",
"core-js": "^2.4.1",
"font-awesome": "^4.7.0",
"http-server": "^0.9.0",
"ie-shim": "^0.1.0",
"jasmine-core": "^2.5.2",
"reflect-metadata": "^0.1.9",
"rxjs": "^5.1.0",
"zone.js": "0.8.5"
},
"devDependencies": {
"@angular/compiler-cli": "^4.0.0",
"@types/hammerjs": "^2.0.33",
"@types/jasmine": "2.5.38",
"@types/node": "^7.0.0",
"@types/selenium-webdriver": "~2.53.39",
"@types/source-map": "^0.5.0",
"@types/uglify-js": "^2.0.27",
"@types/webpack": "^2.0.0",
"add-asset-html-webpack-plugin": "^1.0.2",
"angular-router-loader": "^0.4.0",
"angular2-template-loader": "^0.6.0",
"assets-webpack-plugin": "^3.5.1",
"awesome-typescript-loader": "~3.0.0-beta.18",
"codelyzer": "~2.0.0-beta.4",
"copy-webpack-plugin": "^4.0.0",
"css-loader": "^0.26.0",
"exports-loader": "^0.6.3",
"expose-loader": "^0.7.1",
"extract-text-webpack-plugin": "~2.0.0-rc.3",
"file-loader": "^0.10.0",
"find-root": "^1.0.0",
"gh-pages": "^0.12.0",
"gulp": "^3.9.1",
"html-webpack-plugin": "^2.28.0",
"imports-loader": "^0.7.0",
"install": "^0.6.0",
"istanbul-instrumenter-loader": "2.0.0",
"jasmine-core": "^2.5.2",
"json-loader": "^0.5.4",
"karma": "^1.4.1",
"karma-chrome-launcher": "^2.0.0",
"karma-coverage": "^1.1.1",
"karma-jasmine": "^1.0.2",
"karma-mocha-reporter": "^2.2.2",
"karma-remap-coverage": "^0.1.4",
"karma-sourcemap-loader": "^0.3.7",
"karma-webpack": "2.0.1",
"lodash": "^4.17.4",
"ng-router-loader": "^2.1.0",
"ngc-webpack": "1.1.0",
"node-sass": "^4.5.2",
"npm-run-all": "^4.0.1",
"optimize-js-plugin": "0.0.4",
"parse5": "^3.0.1",
"protractor": "^4.0.14",
"raw-loader": "^0.5.1",
"rimraf": "~2.6.0",
"sass-loader": "^4.1.1",
"script-ext-html-webpack-plugin": "^1.6.2",
"source-map-loader": "^0.1.5",
"string-replace-loader": "1.0.5",
"style-loader": "^0.16.0",
"to-string-loader": "^1.1.4",
"ts-helpers": "1.1.2",
"ts-node": "^2.0.0",
"tslib": "^1.5.0",
"tslint": "~4.5.1",
"tslint-loader": "^3.3.0",
"typedoc": "^0.5.3",
"typescript": "~2.2.1",
"url-loader": "^0.5.7",
"v8-lazy-parse-webpack-plugin": "^0.3.0",
"webpack": "2.3.1",
"webpack-dev-middleware": "^1.10.0",
"webpack-dev-server": "2.4.1",
"webpack-dll-bundles-plugin": "^1.0.0-beta.5",
"webpack-merge": "~3.0.0"
},
It shows below error:
ERROR in ./~/css-loader!./~/sass-loader!./src/app/styles/style.scss
Module build failed:
@include mat-core();
^
No mixin named mat-core
Backtrace:
stdin:4
in D:\Bankim\local\Delivery Skeleton\src\app\styles\style.scss (line 4, column 10)
@ ./src/app/styles/style.scss 4:14-124
@ ./src/app/app.module.ts
@ ./src/app/index.ts
@ ./src/main.browser.ts
@ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main.browser.ts
Child html-webpack-plugin for "index.html":
chunk    {0} index.html 542 kB [entry]
+ 4 hidden modules
webpack: Failed to compile.
What is the expected behavior?
What is the current behavior?
It gives compile time error and stuck at that point.
What are the steps to reproduce?
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: https://goo.gl/DlHd6U
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, browsers are affected?
Angular 4 , ^2.0.0-beta.3, windows
Is there anything else we should know?
Thanks,
Bankim Sutaria

Can we see your style.scss file?
It should have the following line on top of the file:
@import '~@angular/material/theming';

@Chan4077
Here  is my style.scss
@import "common";
@import<EMAIL_ADDRESS>body{
min-width: 600px;
position: absolute;
top: 0;
right: 0;
bottom: 0;
left: 0;
margin: 0;
color: #272727;
font-family: 'Open Sans','Helvetica Neue','Helvetica','Arial',sans-serif;
font-style: normal;
font-weight: 300;
font-size: 14px;
line-height: 1.6;
overflow-x: auto;
-webkit-font-smoothing: subpixel-antialiased;
-webkit-font-smoothing: antialiased;
}
When I add @include mat-core(); it gives above mention error. What is exact use of @include mat-core(); ?

The mat-core mixin is responsible for adding styles for elevation, ripple, overlay, etc that are not theme dependent.
It has already been included in the prebuilt theme files before being compiled to css, so that's why you it's not available to you in your current styles.scss. If you chose to build a custom theme instead of using the prebuilt indigo-pink theme, then you would need to include it. Otherwise it is fine to omit.

What @willshowell said.


Get-AdobeAcrobatReaderDC

I'm not yet sure if this is a bug or a change on the Adobe site. Still, Get-AdobeAcrobatReaderDC appears to return the latest Adobe Acrobat Reader DC; however, after downloading and installing the file, it appears to be Adobe Acrobat DC, not Reader.

Yes at least the installer for English is misplaced at Adobe. The German one is still Reader.

looks like English (UK) too, but looks like the expected file is still on the anticipated path.
http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120135/AcroRdrDC2100120135_en_US.exe
PS C:> Get-AdobeAcrobatReaderDC
Version      Type      Language     URI

21.001.20135 Installer English      http://ardownload.adobe.com/pub/adobe/acrobat/win/AcrobatDC/2100120135/AcroRdrDCx642100120135_en_US.exe
21.001.20135 Installer English (UK) http://ardownload.adobe.com/pub/adobe/acrobat/win/AcrobatDC/2100120135/AcroRdrDCx642100120135_en_US.exe
19.008.20071 Installer Danish       http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_da_DK.exe
19.008.20071 Installer Dutch        http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_nl_NL.exe
21.001.20140 Installer French       http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_fr_FR.exe
19.008.20071 Installer Finnish      http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_fi_FI.exe
21.001.20140 Installer German       http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_de_DE.exe
19.008.20071 Installer Italian      http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_it_IT.exe                                21.001.20140 Installer Japanese     http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_ja_JP.exe                                19.008.20071 Installer Korean       http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_ko_KR.exe                                19.008.20071 Installer Norwegian    http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_nb_NO.exe                                21.001.20140 Installer Spanish      http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_es_ES.exe
21.001.20142 Updater   Neutral      http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120142/AcroRdrDCUpd2100120142.msp
21.001.20142 Updater   Multi        http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120142/AcroRdrDCUpd2100120142_MUI.msp

The installer and shortcut after install shows as Adobe Acrobat DC, however, once installed it is Reader. There's definitely a change in the installer as I'm seeing the 64-bit version installed now instead of the 32-bit version of Reader.

The shortcut points to C:\Program Files\Adobe\Acrobat DC\Acrobat\Acrobat.exe; however, once launched it is Reader:

@bmitchell76 @Deyda could you please test the development branch 2302a2c2acc0c052a8c4140e76e065a0e8e7d131 for Get-AdobeAcrobatReaderDC? Note that this makes some changes to the output by returning x86 (the old installer) and x64 installers (the new installer with the behaviour above). This also removes the .MSP download at this time as I don't have a good way yet to determine whether the updater is a required or optional update (note the version differences in the list below).
Version      : 21.001.20135
Language     : English
Architecture : x64
Name         : Reader DC 2021.001.20135 English Windows(64Bit)
URI          : http://ardownload.adobe.com/pub/adobe/acrobat/win/AcrobatDC/2100120135/AcroRdrDCx642100120135_en_US.exe

Version      : 21.001.20140
Language     : English
Architecture : x86
Name         : Reader DC 2021.001.20140 English for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_en_US.exe

Version      : 21.001.20135
Language     : English (UK)
Architecture : x64
Name         : Reader DC 2021.001.20135 English UK Windows(64Bit)
URI          : http://ardownload.adobe.com/pub/adobe/acrobat/win/AcrobatDC/2100120135/AcroRdrDCx642100120135_en_US.exe

Version      : 21.001.20140
Language     : English (UK)
Architecture : x86
Name         : Reader DC 2021.001.20140 English UK for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_en_US.exe

Version      : 19.008.20071
Language     : Danish
Architecture : x86
Name         : Reader DC 2019.008.20071 Danish for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_da_DK.exe

Version      : 19.008.20071
Language     : Dutch
Architecture : x86
Name         : Reader DC 2019.008.20071 Dutch for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_nl_NL.exe

Version      : 21.001.20140
Language     : French
Architecture : x86
Name         : Reader DC 2021.001.20140 French for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_fr_FR.exe

Version      : 19.008.20071
Language     : Finnish
Architecture : x86
Name         : Reader DC 2019.008.20071 Finnish for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_fi_FI.exe

Version      : 21.001.20140
Language     : German
Architecture : x86
Name         : Reader DC 2021.001.20140 German for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_de_DE.exe

Version      : 19.008.20071
Language     : Italian
Architecture : x86
Name         : Reader DC 2019.008.20071 Italian for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_it_IT.exe

Version      : 21.001.20140
Language     : Japanese
Architecture : x86
Name         : Reader DC 2021.001.20140 Japanese for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_ja_JP.exe

Version      : 19.008.20071
Language     : Korean
Architecture : x86
Name         : Reader DC 2019.008.20071 Korean for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_ko_KR.exe

Version      : 19.008.20071
Language     : Norwegian
Architecture : x86
Name         : Reader DC 2019.008.20071 Norwegian for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/1900820071/AcroRdrDC1900820071_nb_NO.exe

Version      : 21.001.20140
Language     : Spanish
Architecture : x86
Name         : Reader DC 2021.001.20140 Spanish for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_es_ES.exe

Looks good
PS C:> Get-AdobeAcrobatReaderDC | Where-Object {($.language -eq "english") -and ($.architecture -eq "x86")}
Version      : 21.001.20140
Language     : English
Architecture : x86
Name         : Reader DC 2021.001.20140 English for Windows
URI          : http://ardownload.adobe.com/pub/adobe/reader/win/AcrobatDC/2100120140/AcroRdrDC2100120140_en_US.exe

Yeah the dev module looks good. I can confirm Aaron's statement as well. Installation and icons (even in Program & Features) looks like Adobe DC, with the prod module. But when I launch it, it is Adobe Reader DC. Strange :-/

I wonder how this bodes for VDI environments with Acrobat and Reader side-by-side

It does not work. Tried it earlier on my test machine with my script.

What about the Acrobat Reader DC Multi Language download? I can't get it anymore with Get-AdobeAcrobatReaderDC

Adobe Reader updates are included in Get-AdobeAcrobat. This has been split from Get-AdobeAcrobatReaderDC to keep similar retrieval methods in the same functions. Getting the installers is a different approach from getting the updaters.


Put a release onto composer?

There have been a lot of changes to this repo since the last composer release. Could maintainers drop a release by chance? @lkorth
I do see the "master" note in README, but would be good to actually bump this. Latest release is over a year old

Several breaking changes have been made since the last release and I am waiting on finalizing all of the changes before releasing a 2.0 version. In the meantime if you would rather not just use dev-master, you can specify a SHA in composer with dev-master#SHA.


Suggestions for chapter 5

Hi,
I am reading through chapter 5, and have a few comments:
Line 332 (chapter5-database-setup.md) with comment: # Print out what we have added to the user.
should probably be changed to something like # Print out the categories we have added.
I think lines 337-341 are duplicates of the ones above. But that was also mentioned in another Issue.
Method in line 349 (chapter5-database-setup.md) does not save the category, should it. I could make it work without (I guess because of the foreignkey constraint on the Page class). But when I arrived at the exercises, it didnt work without adding a c.save(), just before the return. Rightfully there is a hint to remember, but I wonder if the save should always be there.
In the screenshot on page 53 (pdf) the ordering of the elements is different than mine. Not that it matters but might confuse newbies into thinking they did something wrong.
In the hints for the exercise it is written that you should add a class. Is it "known" that you have to put the class above the code that registers the models, or is it worth saying it?
In the rango.tests, I think all the response.content in the assertIn method calls should be enclosed in str(). I also had to add the following two lines after the imports in the file:
os.environ['DJANGO_SETTINGS_MODULE'] = 'tango_with_django_project.settings'
django.setup()
Hope this helps, I was thinking about doing a pull request, what is easier for you guys?
Thank you for a great tutorial

Hi B.V. I've sorted out the first 3 points.
Hopefully they should be able to work out that the class needs to be
declared before they use it, of course they can look at our repo if they
have major problems.
Re:the tests.. are you using Python 3+ ?
Re the imports of os.environ etc... which file did you add these too?
Thanks
L
On Mon, May 16, 2016 at 3:31 PM, Binary Viking<EMAIL_ADDRESS>wrote:

Hi,
I am reading through chapter 5, and have a few comments:
Line 332 (chapter5-database-setup.md) with comment: # Print out what we
have added to the user.
should probably be changed to something like # Print out the categories we
have added.
I think lines 337-341 are duplicates of the ones above. But that was also
mentioned in another Issue.
Method in line 349 (chapter5-database-setup.md) does not save the
category, should it. I could make it work without (I guess because of the
foreignkey constraint on the Page class). But when I arrived at the
exercises, it didnt work without adding a c.save(), just before the return.
Rightfully there is a hint to remember, but I wonder if the save should
always be there.
In the screenshot on page 53 (pdf) the ordering of the elements is
different than mine. Not that it matters but might confuse newbies into
thinking they did something wrong.
In the hints for the exercise it is written that you should add a class.
Is it "known" that you have to put the class above the code that registers
the models, or is it worth saying it?
In the rango.tests, I think all the response.content in the assertIn
method calls should be enclosed in str(). I also got a problem with the
response.templates are not populated, but I dont know if that is down to me
missing something or this is a new "feature" in Django 1.9. I also had to
add the following two lines after the imports in the file:
os.environ['DJANGO_SETTINGS_MODULE'] = 'tango_with_django_project.settings'
django.setup()
Hope this helps
Thank you for a great tutorial
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/leifos/tango_with_django_19/issues/20

I am using python 3.5.1
I added the os.environ etc to the tests.py file.
.Kim

Hi Kim,
We have just updated the tests.py (
https://github.com/leifos/tango_with_django_19/blob/master/code/tango_with_django_project/rango/tests.py
)
In Python 3, we needed to convert the strings to bytes code (apparently).
So you will see that the strings have a b in front of them. Python 2 will
ignore these, but Python 3 will do the conversion.
If you try these tests now hopefully they should work - there should be no
need to add:
os.environ['DJANGO_SETTINGS_MODULE'] = 'tango_with_django_project.settings'
django.setup()
to the file.
Let me know how you get on.
Cheers
L
On Wed, May 18, 2016 at 1:28 PM, Binary Viking<EMAIL_ADDRESS>wrote:

I am using python 3.5.1
I added the os.environ etc to the tests.py file.
.Kim
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/leifos/tango_with_django_19/issues/20#issuecomment-220011071


IconPack demo

Maybe I'm missing something obvious but where is the IconPack demo I can see at https://github.com/MahApps/MahApps.Metro?
I've cloned the latest source code and run the MetroDemo which gives this, which isn't the same

and MetroDemo.NET45 just opens https://github.com/MahApps/MahApps.Metro.IconPacks in the browser.

@PhilPJL I'm currently working on a new demo


New logo

Simplify -- just an "EU Yellow" shield on "EU Blue" background.

@dmarti  Hi, my name is Richard. I'm a graphic designer and I love making contributions for open source projects. I see that they want a new logo for your application. If you wish, I can do it for you without any problem ... I can present some modern ideas that would improve the appearance and image of your project.
My Portfolio

Hi @richardbmx  -- this sounds great.  I believe we need a 32x32 and a 96x96 PNG.  Happy to look at your ideas.

I'm going to lock on this ... I see that you have an idea (Simplify -- just an "EU Yellow" shield on "EU Blue" background.) in mind but I do not understand it very well, you can explain it a little better...

A simpler logo would be: take the color of the stars, apply that color to the shield symbol.
Then remove the stars and make the shield symbol bigger.
I don't know where the original SVG for the shield is -- it's the same one that Firefox uses for Tracking Protection.

Found it -- it's "tracking-protection.svg"
<!-- This Source Code Form is subject to the terms of the Mozilla Public
   - License, v. 2.0. If a copy of the MPL was not distributed with this
   - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->
<svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" viewBox="0 0 16 16" fill="context-fill" fill-opacity="context-fill-opacity">
  <path d="M8 15.006l-.112-.012a6.244 6.244 0 0 1-4.012-2.427 9.26 9.26 0 0 1-1.8-5.286C2 6.442 2 4.6 2 3.575a1.845 1.845 0 0 1 1.527-1.822L8 .985l4.471.768A1.845 1.845 0 0 1 14 3.576c0 1.023 0 2.866-.08 3.705a9.26 9.26 0 0 1-1.8 5.286 6.244 6.244 0 0 1-4.012 2.427zM4 3.7C4 5.325 3.951 6.46 4 7a7.572 7.572 0 0 0 1.487 4.382A4.223 4.223 0 0 0 8 12.987a4.221 4.221 0 0 0 2.512-1.605A7.572 7.572 0 0 0 12 7c.049-.54 0-1.675 0-3.3l-4-.685z"/>
  <path d="M8 4.537l-2.5.428c.009.942.03 1.655.062 2a5.765 5.765 0 0 0 1.13 3.53 2.685 2.685 0 0 0 1.3.943H8z"/>
</svg>

@richardbmx For the icon/logo in context, you can try installing the Global Consent Manager in Firefox, and clicking the toolbar icon.
I went looking for the SVG yesterday, and found permissions.svg in the same directory that might be even more appropriate for GCM (with the same EU colors applied):
https://searchfox.org/mozilla-central/source/browser/themes/shared/controlcenter
(Those icons are provided under MPL, meaning we are allowed to use, reuse and modify them as needed.  Coincidentally, GCM is a Mozilla project as well.)
Feel free to come up with a design based on one of these two starting points, or something completely different if you have a better idea that could better fit the project.

Removing this from the experiment milestone because what we have seems to be good enough for now. More ideas welcome.


Add notes on how to migrate from using IndexRedirect

May be helpful for some, I didn't really find it too apparent offhand without searching the issues first on how to do it.

@pshrmn yup totally right on those, should have looked it a bit closer, fixed!

LGTM. Thanks!


Stable Migrations Are Essential

The current setup with some migrations being automatically generated on different machines (as part of the deploy-hs script) is unstable. Having the development heads (either the develop or master branches) stand with missing migrations is unstable.
1) The missing migrations must be committed and merged
2) The makemigrations command must be a part of creating a new branch, not deployment (running them should be part of deployment, though)
3) A policy must be adopted that no branch or pull request be accepted for merge that does not provide a complete set of migrations. No branch should be accepted for which the makemigrations command would try to generate anything that isn't already included.

Agreed. Not conforming to this is why we are having so much trouble with migrations.
Alva L. Couch, Ph.D.
Senior Architect, CUAHSI Water Data Center
http://wdc.cuahsi.org
+1<PHONE_NUMBER> x 203
From<EMAIL_ADDRESS>Sent: ?Wednesday?, ?May? ?27?, ?2015 ?8?:?00? ?AM
To<EMAIL_ADDRESS>The current setup with some migrations being automatically generated on different machines (as part of the deploy-hs script) is unstable. Having the development heads (either the develop or master branches) stand with missing migrations is unstable.

The missing migrations must be committed and merged
The makemigrations command must be a part of creating a new branch, not deployment (running them should be part of deployment, though)
A policy must be adopted that no branch or pull request be accepted for merge that does not provide a complete set of migrations. No branch should be accepted for which the makemigrations command would try to generate anything that isn't already included.

Reply to this email directly or view it on GitHubhttps://github.com/hydroshare/hydroshare/issues/407.

I've edited the policies wiki document https://github.com/hydroshare/hydroshare/wiki/Policies-&-Procedures to include this statement.  Please feel free to clarify aspects of it that may not be right or complete.  I noticed much deprecated content in this policy document.  I cleaned up some of it, but not all.  I'll create a separate ticket for wiki clean up to work on.

@ironfroggy Can you clarify what you mean by "merge" in point 1.  Do you mean taking migrations committed to a feature branch and merging them into develop, and/or also squashing migrations once they are merged into develop?

I mean the feature branch being merged into develop.

@mjstealey is this resolved by your recent migration resets


Improve run error message

It would be nice to improve the error message when the command fails.
Current
$ runme run echo-ok
ok
failed to run command "set -e -o pipefail;echo ok;false;\n": exit status 1

For readme
```sh
echo ok
false
```

Just not printing the generated bash is probably fine for now:
failed to run command: exit status 1

Closed in #115


Docs improvements

Hello
I fixed some typos in the docs,
Hope it helps.
Br,
Tudor.

thanks for fixing these typos! added these fixes in https://github.com/NillionNetwork/nillion-docs/pull/69


Remove legacy code from Python 2

Not needed with Python 3 because it is default lib.

@giovannicimolin This also looks good to me. Let me know what you think.
This looks like it cleans up the last of the "six" references https://github.com/search?q=repo%3Ajazzband%2Fdjango-rest-knox+six&type=code


testing this out

trying to move on to the next step. this is a test

clicking c"lose with comment". this is a test


Test gem in Windows

There are a number of issues using this gem on Windows.  Installation doesn't work, detection of administrative privileges does not work and the bat files which windows_server_manager uses have been depreciated in favor of a PowerShell module.
I may begin working on a few of these at some point in the near future.

Anything you can do to help in this area would be awesome!  I hope to be in a place where I might be able to boot into Windows in the coming months, but for now it's not really feasible ;(


README Updates

Updated README with examples of starting all 5 supported setups
Added removeVistaSource.sh to cover required post install requirements
Updated rpms and vxvista post installer to use removeVistaSource.sh
Added entry build arg to adjust for when running Cache

Joe,
Give me some time. I am working with George Lilly now.
On Wed, May 9, 2018 at 3:00 PM, Christopher Edwards <
<EMAIL_ADDRESS>wrote:

@ChristopherEdwards approved this pull request.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA/docker-vista/pull/3#pullrequestreview-118863821,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEtwh-PH2FX1QIdwAp2fU67QovALSklks5twzy3gaJpZM4T40tR
.

I tried one yesterday, and it failed like this: Error processing tar
file(exit status 1): write /home/osehra/Dashboard/VistA-M/Packages/Lexicon
Utility/Globals/757.033-3+CHARACTER POSITIONS.zwr: no space left on device
I haven't seen this before. Any hints?
--Sam
On Wed, May 9, 2018 at 3:07 PM, Sam Habiel<EMAIL_ADDRESS>wrote:

Joe,
Give me some time. I am working with George Lilly now.
On Wed, May 9, 2018 at 3:00 PM, Christopher Edwards <
<EMAIL_ADDRESS>wrote:

@ChristopherEdwards approved this pull request.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA/docker-vista/pull/3#pullrequestreview-118863821,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEtwh-PH2FX1QIdwAp2fU67QovALSklks5twzy3gaJpZM4T40tR
.

Googled it. Common problem on Macs apparently. Trying again now.
On Thu, May 10, 2018 at 8:53 AM, Sam Habiel<EMAIL_ADDRESS>wrote:

I tried one yesterday, and it failed like this: Error processing tar
file(exit status 1): write /home/osehra/Dashboard/VistA-M/Packages/Lexicon
Utility/Globals/757.033-3+CHARACTER POSITIONS.zwr: no space left on device
I haven't seen this before. Any hints?
--Sam
On Wed, May 9, 2018 at 3:07 PM, Sam Habiel<EMAIL_ADDRESS>wrote:

Joe,
Give me some time. I am working with George Lilly now.
On Wed, May 9, 2018 at 3:00 PM, Christopher Edwards <
<EMAIL_ADDRESS>wrote:

@ChristopherEdwards approved this pull request.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA/docker-vista/pull/3#pullrequestreview-118863821,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEtwh-PH2FX1QIdwAp2fU67QovALSklks5twzy3gaJpZM4T40tR
.

@shabiel the docker machine is probably out of space (macs use a virtual machine to provide a docker environment either through HyperKit or VirtualBox) delete some old images/containers and it should free up some space.

@josephsnyder Did you see these failures before in the Tests?
https://code.osehra.org/CDash/viewTest.php?onlyfailed&buildid=38397

Okay. It didn't work. It builds fine; but something to do with the entry
point not being defined:
docker run -p 9431:9431 -p 8002:8002 -p 2223:22 -p 8080:8080 -d -P
--name=osehravista osehra-vista
saichiko:docker-vista sam$ docker ps -a
CONTAINER ID        IMAGE                 COMMAND
CREATED                  STATUS
PORTS
NAMES
e0f8e36cd2b9        osehra-vista          "/bin/sh -c ${entry_…"   Less
than a second ago   Exited (126) 3 minutes
ago
osehravista
What am I doing wrong?
On Thu, May 10, 2018 at 9:00 AM, Christopher Edwards <
<EMAIL_ADDRESS>wrote:

@shabiel https://github.com/shabiel the docker machine is probably out
of space (macs use a virtual machine to provide a docker environment either
through HyperKit or VirtualBox) delete some old images/containers and it
should free up some space.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA/docker-vista/pull/3#issuecomment-388046231,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEtwmaWZ7iqKKEDqSNCRPD0Znq6mfDXks5txDnvgaJpZM4T40tR
.

@shabiel I'm looking into this right now.  Will have an update for you shortly

+1 for being communicative!
On Thu, May 10, 2018 at 10:01 AM, Joe Robinson<EMAIL_ADDRESS>wrote:

@shabiel https://github.com/shabiel I'm looking into this right now.
Will have an update for you shortly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA/docker-vista/pull/3#issuecomment-388062219,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEtwnh77ba1v84pGXL3omss6euDs7I8ks5txEgkgaJpZM4T40tR
.

@shabiel @ChristopherEdwards
I just ran the build and it looks like YottaDB is failing:
Something went wrong downloading ydbinstall
Creating osehra...
ls: cannot access /opt/yottadb/: No such file or directory
ls: cannot access /opt/yottadb/: No such file or directory
ls: cannot access /opt/yottadb/: No such file or directory```

The `Something went wrong downloading ydbinstall` is from GTM/install.sh:92.  I may need help troubleshooting that since the YottaDB sections were updated just after I updated the Dockerfile.

This is due to the SHA hash updating of the ydbinstall script (due to changes i made there :) ). please remove the line that does the SHA hashing I think i'm being too careful. The only thing i'd do is make sure the file looks like a shell script (file, sed, etc can be used to check) so that we don't try to execute an html file...

Hi Sam,
We have seen those errors before. I don't have a good explanation as to why it happens during these "headless" runs.  I'll run a local one to be sure.

@jrobinsonkrm
Update this file:
$ cat GTM/ydbinstall_SHA1
f563850db56e19aa3d482a595474445cf29228bb  ydbinstall

On Thu, May 10, 2018 at 10:12 AM, Joseph Snyder<EMAIL_ADDRESS>wrote:

Hi Sam,
We have seen those errors before. I don't have a good explanation as to
why it happens during these "headless" runs. I'll run a local one to be
sure.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA/docker-vista/pull/3#issuecomment-388065512,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEtwreH1QcQWuBnu6eWOpLAH5XxdFcGks5txErPgaJpZM4T40tR
.

i have a feeling that the entry point isn't defined as it isn't looking at the dockerfile any more (hunch/guess) and since we refer to an environment variable it doesn't exist. if you run docker run --env entry= it should work.

I did a check on the entry_path variable in the dockerfile and it returned correctly:
Step 25/27 : RUN echo ${entry_path}
 ---> Running in edfaa10a9528
/home/osehra/bin/start.sh

I am a dummy reading the instructions: Here's what I did:
docker build -t osehra-vista .
docker run -p 9431:9431 -p 8002:8002 -p 2223:22 -p 8080:8080 -d -P
--name=osehravista osehra-vista

I don't see any updated run instructions, so that is what i would have ran. not sure that I like having the entry point using an environment variable due to this issue.

There isn't a reason for using --env entry= since that's a build arg.
ARG entry="/home"
ENV entry_path="${entry}/${instance_name}/bin/start.sh"
And since entry has a default you don't need to supply it.  entry_path will build the correct entry path dynamically (this is because cache is in a different location than GTM/YottaDB)

Apologies i misspoke about the build arg (i was just using sam's posting without going back to the code to get the exact arg name) - yes, entry_path should be the right variable to pass to the run command.
Maybe the $basedir for cache should change so it is more in-line with GT.M/YottaDB and we can avoid this whole mess from the beginning. I quickly added Caché support without much thinking and it appears I was short sighted with how I set $basedir, keeping the cache in /opt/cachesys still makes sense though - just the location of start.sh should be different.

@shabiel I have fixed the entrypoint and permission issues with the post installer script.  Can you please pull and test again

Okay. I will try it now.
On Thu, May 10, 2018 at 12:30 PM, Joe Robinson<EMAIL_ADDRESS>wrote:

@shabiel https://github.com/shabiel I have fixed the entrypoint and
permission issues with the post installer script. Can you please pull and
test again
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OSEHRA/docker-vista/pull/3#issuecomment-388107706,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEtwpl6K3r0uf4MpjhdBoyvmBcz6j9uks5txGsNgaJpZM4T40tR
.

Okay. I had a problem with a stuck CMake; but when I did it again, it worked fine.
I will test the others now and see how they go.

Update:
I tried osehra vista; worldvista. Both work fine. Now trying vxvista. There is a stray password prompt near the end. Not sure what that's doing there, but it's not affecting anything that I can see.

That password prompt is the script running one of the clean up post install scripts.  You can see them here:
https://github.com/OSEHRA/docker-vista/pull/3/files#diff-e8c809c0f2fa8c0bf8d137fa12edf882R3

@shabiel there is only one reference for the CACHE database in cache.cpf:
https://github.com/jrobinsonkrm/docker-vista/blob/readme-update/Cache/cache.cpf#L13
Is there a way you can get your copy of the cache.cpf so I can examine it?  Also, what steps did you take to get to this error so I can try to recreate it?

@shabiel , did you call your instance "cache"?  I know that the script does some "find and replace" with the FOIA string to add in the instance name: https://github.com/jrobinsonkrm/docker-vista/blob/readme-update/Cache/install.sh#L124.  That would give us the duplication of that named file.
Sorry about the duplication of emails, @jrobinsonkrm

@shabiel @josephsnyder looks like the instance name was the issue.  I've updated it locally to cachevista but would like input on the name in case there is something better to call it.

A - OK.
There are some minor issues with the README which I can take care of. I am not in the office my now in front of the computer I had the changes on.


Restore state class of the Sensor and create HistoricalSensor

As per discussion here #30:

HistoricalSensor should have an undefined state_class and should keep all the logic related to historical data imports
Sensor should have proper state_class and should keep all the logic but historical data imports (as it was before v2.0)

I’ll take this…
I’ll open a new branch, can I pull form master?

Yes, everything is merged into the master.
Thank you!

@disforw It sounds like all the workarounds did not work and we need to implement this solution.


Implement statement node in ast

Implement null and return statement.

@Lai-YT Applied fixes, removed unused parser rules and code.


textured-mesh bug

video: https://youtu.be/5jsK8Vv0B8k
Example: http://pixijs.github.io/examples/?v=v4.2.3#/basics/textured-mesh.js
my device: http://www.gsmarena.com/xiaomi_mi_note-6944.php
Pixi 4.02 and 3.0.11  does not have this problem.

Hi @lizhenwen, which browser is that?

@staff0rd Chrome

Hi there! HAve you tested on the latest dev version? Also have you tried adding this line before creating the renderer:
PIXI.glCore.VertexArrayObject.forceNative = true
Cheers!

Hi @lizhenwen, have you had a chance to try this with the latest pixi (v4.5.0)? Also, have you had a second to try @GoodBoyDigital's recommendation?

Closing due to inactivity. Please create a new issue if you have any other problems


⚠ Samsung has degraded performance

In c8d8405, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2746 ms

Resolved: Samsung performance has improved in 88a8d05.


Add clap parameter for mainnet, or testnet

Add a configuration option, which indicates, whether we are using the testnet (default), or the mainnet of Bitcoin.
Then, according to this setting, choose the correct code path (e.g. checking test net instead of mainnet on blockstream)
@mtbitcr Do we need to think about labelling, if bills/quotes etc. were created in testnet mode, or mainnet mode? If yes, we might have to put in a layer in the filesystem, where we differentiate (e.g. ./bills/testnet and ./bills/mainnet, or ./testnet/bills, ./mainnet/bills etc.)

@zupzup as we discuss, let work on this

here is my though process regarding this issue,
i will add .env variable called environment, then write a function that return the network type depending on the environment i.e testnet, regtest, segtest and mainnet
this where everywhere we need the network type, we can call the function and get the right network type base on the environment set when running the program.
@zupzup @mtbitcr

here is my though process regarding this issue,
i will add .env variable called environment, then write a function that return the network type depending on the environment i.e testnet, regtest, segtest and mainnet
this where everywhere we need the network type, we can call the function and get the right network type base on the environment set when running the program.
@zupzup @mtbitcr

Sounds reasonable :+1:
As a reference, the new variable should be implemented in the same way as the other config options within config.rs

@mubarak23 Thank you for your contribution! You will be rewarded with 500 tokens at the beginning of the next quarter.
Please provide me with your taproot address of bitcoin wallet.


Basic CSP setup broken

Versions

nuxt: 2.15.4
node: 14.16.1

Reproduction

stieben/nuxt-csp-bug-poc
Steps to reproduce

yarn create nuxt-app nuxt-csp-bug-poc

Additional Details
? Project name: nuxt-csp-bug-poc
? Programming language: JavaScript
? Package manager: Yarn
? UI framework: None
? Nuxt.js modules: None
? Linting tools: None
? Testing framework: None
? Rendering mode: Universal (SSR / SSG)
? Deployment target: Server (Node.js hosting)
? Development tools: None
? What is your GitHub username? stieben
? Version control system: Git

cd nuxt-csp-bug-poc

Set render.csp to true (as per Nuxt Documentation)

yarn build

yarn start

What is Expected?
Hydration and a working Vue app, with the security of a CSP.
What is actually happening?

Vue inactive
2 console errors:

Content Security Policy: The page’s settings blocked the loading of a resource at eval (“script-src”)
Content Security Policy: The page’s settings blocked the loading of a resource at eval (“script-src”)

Hi @danielroe, thanks for getting back to me so quickly.
Indeed I had tested this only on ❌ Firefox 87.0 and just verified that the problem persists. On ✔ Chromium 90.0.4430.72, however, I don't get any errors as well. Both browsers include the following response header (which looks correct):
Content-Security-Policy
	script-src 'self' 'sha256-kAVGvMFWDkuqkyA4xnLi3h5jk8dWz0XiySrLWG+6PjU='

OS: elementary 5.1.7 64-bit (based on Ubuntu 18.04.4 LTS)
Is it possible that those browsers calculate the hash differently? 🤔

Trying a different port didn't change anything, but there are indeed no errors in a private/incognito window. I'm clueless about why that would make a difference...
Anyway, I embarrassed myself. Thanks a lot for your time!

I just went through my browser extensions and curiously enough it's the Vue Devtools causing the error output, which means that my initial assessment was incorrect – it seems that Vue is hydrated; it's just the extension that's blocked.


Unable to uninstall OLM

WHAT:
How to uninstall OLM?
Expected

Script to uninstall and make command
Documentation over it

+1 !!
I tried oc delete -f deploy/okd/manifests/latest/
and now my namespaces are blocked in the terminating state and not going away.
$ oc get namespace openshift-operators
NAME                  STATUS        AGE
openshift-operators   Terminating   20h
$ oc get namespace openshift-operator-lifecycle-manager
NAME                                   STATUS        AGE
openshift-operator-lifecycle-manager   Terminating   20h

There are finalizers listed - so I assume a finalizer failed and now the namespaces are blocked:
$ oc get namespace openshift-operators -o yaml
apiVersion: v1
kind: Namespace
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"v1","kind":"Namespace","metadata":{"annotations":{},"name":"openshift-operators","namespace":""},"spec":{"finalizers":[]}}
    openshift.io/sa.scc.mcs: s0:c22,c14
    openshift.io/sa.scc.supplemental-groups:<PHONE_NUMBER>/10000
    openshift.io/sa.scc.uid-range:<PHONE_NUMBER>/10000
  creationTimestamp: 2019-04-11T17:45:09Z
  deletionTimestamp: 2019-04-12T13:35:44Z
  name: openshift-operators
  resourceVersion: "1048433"
  selfLink: /api/v1/namespaces/openshift-operators
  uid: 8d16cc01-5c81-11e9-81da-54e1addf59b3
spec:
  finalizers:
  - kubernetes
status:
  phase: Terminating

$ oc get namespace openshift-operator-lifecycle-manager -o yaml
apiVersion: v1
kind: Namespace
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"v1","kind":"Namespace","metadata":{"annotations":{},"name":"openshift-operator-lifecycle-manager","namespace":""}}
    openshift.io/sa.scc.mcs: s0:c22,c9
    openshift.io/sa.scc.supplemental-groups:<PHONE_NUMBER>/10000
    openshift.io/sa.scc.uid-range:<PHONE_NUMBER>/10000
  creationTimestamp: 2019-04-11T17:45:09Z
  deletionTimestamp: 2019-04-12T13:35:44Z
  name: openshift-operator-lifecycle-manager
  resourceVersion: "1043578"
  selfLink: /api/v1/namespaces/openshift-operator-lifecycle-manager
  uid: 8d161d42-5c81-11e9-81da-54e1addf59b3
spec:
  finalizers:
  - kubernetes
status:
  phase: Terminating

This is an ordering issue -
If you delete the apiservice first (packagemanifests), then the crds, and then the other resources, you should be in the clear.
We should have a tool to do this for you so I'll leave this open to track that.

Hi @ecordell,
What about we create a make command to do it?

deleting the packagemanifests doesn't seem to work
$ kubectl delete packagemanifests.packages.operators.coreos.com -n olm --all
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource
Error from server (MethodNotAllowed): the server does not allow this method on the requested resource

The following worked for me:
kubectl delete crd catalogsources.operators.coreos.com clusterserviceversions.operators.coreos.com installplans.operators.coreos.com operatorgroups.operators.coreos.com subscriptions.operators.coreos.com
followed by
kubectl delete ns olm

PR to solve it : https://github.com/operator-framework/operator-lifecycle-manager/pull/898

The PR to address it is : https://github.com/operator-framework/operator-lifecycle-manager/pull/1438

Hello, I'm trying to add OLM as an addon for Minikube with this PR, but I'm not able to uninstall OLM in a clean way.
minikube addons enable olm runs:
kubectl apply -f https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.14.1/crds.yaml
kubectl apply -f https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.14.1/olm.yaml

minikube addons disable olm runs:
kubectl delete -f https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.14.1/crds.yaml
kubectl apply -f https://github.com/operator-framework/operator-lifecycle-manager/releases/download/0.14.1/olm.yaml

The error is because of order when deleting those I guess (can't find CRDs since they are deleted in first step), but if I invert the order then I got deleting pending (finalizers?) with olm namespaces in Terminating.
I'm wondering what is the correct procedure to uninstall olm, I wasn't able to reproduce the procedure I see in this PR for make uninstall successfully.
Thanks!

Why was this closed? This seems to be an issue still...


[SPARK-46752][SQL][TESTS] Use default ORC compression in data source benchmarks

What changes were proposed in this pull request?
This PR aims to use the default ORC compression in data source benchmarks.
Why are the changes needed?
Apache ORC 2.0 and Apache Spark 4.0 will use ZStandard as the default ORC compression codec.

https://github.com/apache/orc/pull/1733
https://github.com/apache/spark/pull/44654

OrcReadBenchmark was switched to use ZStandard for comparision.

#44761

And, this PR aims to change the remaining three data source benchmarks.
$ git grep OrcCompressionCodec | grep Benchmark
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/BuiltInDataSourceWriteBenchmark.scala:import org.apache.spark.sql.execution.datasources.orc.OrcCompressionCodec
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/BuiltInDataSourceWriteBenchmark.scala:      OrcCompressionCodec.SNAPPY.lowerCaseName())
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/DataSourceReadBenchmark.scala:import org.apache.spark.sql.execution.datasources.orc.OrcCompressionCodec
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/DataSourceReadBenchmark.scala:      OrcCompressionCodec.SNAPPY.lowerCaseName()).orc(dir)
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/FilterPushdownBenchmark.scala:import org.apache.spark.sql.execution.datasources.orc.OrcCompressionCodec
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/FilterPushdownBenchmark.scala:      .setIfMissing("orc.compression", OrcCompressionCodec.SNAPPY.lowerCaseName())

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual review.
Was this patch authored or co-authored using generative AI tooling?
No.

Oh, I missed your approval, @HyukjinKwon ! Thank you!
Merged to master.

Late +1

Thank you, @yaooqinn .


CI + Testsuite Setup and move ES6

This PR setups the full code-quality suite

ci via GitHub actions
lint using standard
tests using meteortesting:mocha and puppeteer for headless client tests (which are required for proper ci tests)
moved all code to ES6 (to satisfy linter)
updated some test dependencies
added ecmascript package

@StorytellerCZ re-requested review. We should update this soon, so we can update dburles:mongo-collection-instances because there are many packages depending on it and may be prevented from updating because of the indirect dependency to accounts.

Merging. Also @jankapunkt maybe it would be nice to add test to the package template so that we can get nice tests like these on all packages?


Update to 1.8rc1

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I'm guessing you are referring to this bug. Hmm...

Please see issue ( https://github.com/conda-forge/conda-forge.github.io/issues/147 ). Basically we haven't formalized a way to do dev releases. We should probably try to do that first. Otherwise users may be forced into an rc or dev release when they want stable releases.

@jakirkham, thank you for the pointer.
In the case of OpenTURNS we always push rc's as they're almost stable and bring lots of bug fixes, also the rc period is rather short (1 month or so).

Still we should have a proper way of doing RCs that don't fall in the main label. It would be nice if we could resolve that first and then proceed.


Highlight Moments

[x] MVP
[x] Allow some click area margin for better UX
[x] Finalize component styles

can't wait to see this on staging!


Unhelpful error messages when parsing of Cargo.toml failed

In the updating process of one of my libs, I needed to add these lines in my Cargo.toml
[dev-dependencies]
wayland-client = { features = ["dlopen"] }

But at first, I made a mistake, and forgot the [ ] around "dlopen", thus writing
[dev-dependencies]
wayland-client = { features = "dlopen" }

the error message generated by cargo was
expected a value of type `string`, but found a value of type `table` for the key `dev_dependencies`

which wasn't really helpful to see my mistake.
Would it be possible to make more precise error messages in cases like this?

Wow, that is a pretty bad error message! This originates from the toml crate, but I'd want to take some time to get a minimization before closing in favor of an issue over there.


gnunet: Add gnunet_git.

Motivation for this change
Gnunet didn't have a release in years but they are preparing a new one. Also they seem to have switched from svn to git for development, so I added a new git package. I tested nothing but the actual build and the start of a few binaries so far since gnunet is somewhat convoluted and I didn't manage to get it running with the old version. Going forward I'd like to add a pkg switch to the gnunet service and try to get that up to date for the next release.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Pinging @viric for easy access.

I merge it, but please appoint yourself as maintainer. :)

Alright. Will do. Do you want to get rid of the entire gnunet maintainership? I think I understand the package enough to take over.


Shellquote for path escaping seems to not work

I am setting a new python project with pipenv. While installing packages, i got the following error.
I am using the 11.10.0 version of pipenv.
I think pipenv is picking up incomplete path. something similar to #1621
Command executed:
pipenv install [package]
Here is the log trace...
D:\Development\project>pipenv install request
Installing request…
Looking in indexes: https://pypi.python.org/simple
Collecting request
  Downloading https://files.pythonhosted.org/packages/f3/73/e72e0b5e771ca4dbd48fbef81ddca96fcaa98153f401d797e6682b8cec65/request-1.0.1.tar.gz
Collecting get (from request)
  Downloading https://files.pythonhosted.org/packages/65/e5/5fc6b9bdeb11a94fc2cd1c761ab8b81b590ca831517a223edf7285903d34/get-1.0.2.tar.gz
Collecting post (from request)
  Downloading https://files.pythonhosted.org/packages/3b/e4/dcf3750dc9bd4cbb7e04933109219295dd9e6f6b9e51e3baf76be0091a54/post-1.0.1.tar.gz
Requirement already satisfied: setuptools in c:\users\nitish mutha\.virtualenvs\project-r3blgo3n\lib\site-packages (from request) (39.0.1)
Collecting query_string (from get->request)
  Downloading https://files.pythonhosted.org/packages/e7/3d/ba02d224ca7df8c12ef6d894cb97d5eea0a11bf0beacac3aa3681927ce82/query_string-1.0.1.tar.gz
Collecting public (from query_string->get->request)
  Downloading https://files.pythonhosted.org/packages/4d/c3/71608a2b17ac8369376415c0e696e412beccb9c30b47fb308d5a8278a36d/public-1.0.1.tar.gz
Building wheels for collected packages: request, get, post, query-string, public
  Running setup.py bdist_wheel for request: started
  Running setup.py bdist_wheel for request: finished with status 'done'
  Stored in directory: C:\Users\Nitish Mutha\AppData\Local\pip\Cache\wheels\85\e5\f5\34ee71ff1fd375984eba09d6aad563a6bb5912f0639b7f1af4
  Running setup.py bdist_wheel for get: started
  Running setup.py bdist_wheel for get: finished with status 'done'
  Stored in directory: C:\Users\Nitish Mutha\AppData\Local\pip\Cache\wheels\c4\a7\06\00fa9ec8a0fdf555f5432f91f738d37c55280175e1cd46a77e
  Running setup.py bdist_wheel for post: started
  Running setup.py bdist_wheel for post: finished with status 'done'
  Stored in directory: C:\Users\Nitish Mutha\AppData\Local\pip\Cache\wheels\15\a9\c1\42bfa000a5184863f31df0b881280d7d113e7f54f89f6634a2
  Running setup.py bdist_wheel for query-string: started
  Running setup.py bdist_wheel for query-string: finished with status 'done'
  Stored in directory: C:\Users\Nitish Mutha\AppData\Local\pip\Cache\wheels\5f\f6\7c\6d4c06b8fe517fe986c93d18f8bb6fa38a7a699d84762de9f5
  Running setup.py bdist_wheel for public: started
  Running setup.py bdist_wheel for public: finished with status 'done'
  Stored in directory: C:\Users\Nitish Mutha\AppData\Local\pip\Cache\wheels\06\a4\da\909d4238750ba4b6761907dbfc321be99dd40ced54e3d085fa
Successfully built request get post query-string public
Installing collected packages: public, query-string, get, post, request
Successfully installed get-1.0.2 post-1.0.1 public-1.0.1 query-string-1.0.1 request-1.0.1

Adding request to Pipfile's [packages]…
Pipfile.lock (011179) out of date, updating to (0d4f18)…
Locking [dev-packages] dependencies…
Locking [packages] dependencies…
Updated Pipfile.lock (0d4f18)!
Installing dependencies from Pipfile.lock (0d4f18)…
An error occurred while installing click==6.7! Will try again.
An error occurred while installing flask==0.12.2! Will try again.
An error occurred while installing get==1.0.2! Will try again.
An error occurred while installing itsdangerous==0.24! Will try again.
An error occurred while installing jinja2==2.10! Will try again.
An error occurred while installing markupsafe==1.0! Will try again.
An error occurred while installing post==1.0.1! Will try again.
An error occurred while installing public==1.0.1! Will try again.
An error occurred while installing query-string==1.0.1! Will try again.
An error occurred while installing request==1.0.1! Will try again.
An error occurred while installing werkzeug==0.14.1! Will try again.
  ================================ 11/11 - 00:00:13
Installing initially–failed dependencies…
     -------------------------------- 0/11 - 00:00:00
Invalid requirement: 'Mutha\AppData\Local\Temp\pipenv-bpyndvgh-requirements\pipenv-931zhika-requirement.txt'
It looks like a path. File 'Mutha\AppData\Local\Temp\pipenv-bpyndvgh-requirements\pipenv-931zhika-requirement.txt' does not exist.

     ================================ 0/11 - 00:00:05

I do have a space in my path as FirstName LastName. But pipenv gets incomplete path.
python -m pipenv.help output
Pipenv version: `'11.10.0'`

Pipenv location: `'C:\\Users\\Nitish Mutha\\Anaconda3\\lib\\site-packages\\pipenv'`

Python location: `'C:\\Users\\Nitish Mutha\\Anaconda3\\python.exe'`

Other Python installations in `PATH`:

  - `3.6`: `C:\Users\Nitish Mutha\Anaconda3\python.exe`

  - `3.6.4`: `C:\Users\Nitish Mutha\Anaconda3\python.exe`
  - `3.6.5`: `C:\Users\Nitish Mutha\AppData\Local\Programs\Python\Python36-32\python.exe`
  - `3.6.4`: `C:\Windows\py.exe`

(Thinking out loud) Basically we need to quote all arguments in the call, not just the command. This can probably be better generalised to avoid duplication…

It’s kind of weird we haven’t encountered this before though...

P.s. the topic isn’t quite right. Shellquoting works fine. We just aren’t doing it here

happy to change the topic as appropriate. Thanks


suggest some changes to comments on LSOA rastor to help new reader un…

…derstand

Thanks @alanbrennan1965 I agree these would be big improvements so I'll take a look and write these over the next few days.


Performance different from reported for SAE based models

Hi!
Thanks for providing this excellent demo. However, when I tried to run some baseline models for unsupervised open intent discovery like "SAE-KM", "DCN" and "DEC", I found that the performance is different from what is reported in paper "Discovering New Intents with Deep Aligned Clustering".
For instance, with "DCN" model in this demo, after averaging 3 runs on "banking", I get NMI 54.87, ARI 18.19, ACC 33.03. As opposed NMI 67.54, ARI 26.81, ACC 41.99 reported in the paper.
Notice that these models are all related with "SAE" embedding. I first wondered if it is because of different hyper-parameter choice. But the parameters I used strictly follows your implementation except the paths.
Also notice that I can not install all the dependencies by running pip install -r requirements.txt. Thus I have to change the versions for several packages by myself. The differences are:

keras-nightly==2.5.0.dev2021032900
tensorflow-gpu==2.5.1
Pillow==8.4.0
mkl-random==<IP_ADDRESS>
mkl-fft==1.0.6
numpy==1.19.5
remove torchaudio

To make it clearer in a future version, could you please:

add the performance list using your environment into README as a comparison for reproduction.
clean and remake the requirements.txt.

Thanks a lot! And appreciate your great work!

Hi, thanks for your recognition in our work! For DCN and DEC, you need to first train SAE and save the trained model and then, use the trained SAE model to perform DCN and DEC training.

Thanks for your quick reply!
It seems that the code will run "SAE" training automatically. But it still produces lower performance.
if os.path.exists(self.sae_feats_path):
    self.logger.info('Loading SAE features from %s' % self.sae_feats_path)
    self.sae.load_weights(self.sae_feats_path)
else:
    self.logger.info('SAE (emb) training start...')
    self.sae.fit(self.tfidf_train, self.tfidf_train, epochs = args.num_train_epochs_SAE, batch_size = args.batch_size, shuffle=True,
    validation_data=(self.tfidf_test, self.tfidf_test), verbose=1)
    self.logger.info('SAE (emb) training finished...')

    if args.save_model:
        os.makedirs(self.sae_feats_path)
        self.sae.save_weights(self.sae_feats_path)

@zhang-yu-wei  Hi, "DCN" and "DEC"  does will run training automatically if the "self.sae_feats_path" not exists. However, some of DEC's or DCN's generic hyperparameters, such as Batch size, are not necessarily optimal for SAE.So you'd better train
"SAE" manually and save the model. And set "self.sae_feats_path"  to the SAE saved model. Hope to help you.

I see. As I would like to use this results, it would be great if you could maybe provide the pre-trained SAE features since it will save me a lot of time. Thanks! :-)

Of course, SAE's hyperparameter file path is in open_intent_discovery/configs/ sae.py.You can just run the open_intent_discovery/examples/run_SAE.sh and  make sure you add the parameter "save model" to the run_sae.sh file.


Implement the Credential Management API

It allows streamlining logins for returning customers including synchronizing credentials between multiple devices.
Details: https://developers.google.com/web/updates/2016/04/credential-management-api

#1224


Unable to save more than one flag when running PHP 7.2

We're experiencing an issue running Cache-Flag in PHP 7.2 which is related to a known issue in Yii: https://github.com/yiisoft/yii/issues/4167.
We have one flag currently set in the CP. When we try to add additional flags beneath it we get the error PHP 7.2: count(): Parameter must be an array or an object that implements Countable and when the page is reloaded the new flag is gone. If we add a flag above the existing one we see the same error but when the page is reloaded the new flag remains.
We have not been able to replicate the issue in PHP 7.1 with the same data store (postgres). There are no errors appearing in the PHP or Web log for Craft.

To Replicate

When running PHP 7.2 add a new flag in the CP.
If that was your first flag, attempt to add a second flag.
You should see the red flash notice (screenshot attached)

Environment Information:
PHP version | 7.2.8
PostgreSQL 9.6.6
Imagick 3.4.3 (ImageMagick 6.7.8-9)
Craft Pro 3.0.25
Yii Version <IP_ADDRESS>

Hi @bernsno – thanks for the detailed report! I'll take a look at that ASAP.

@bernsno Fixed in v. 1.0.3.


Enh/mark users with display name collision

fixes #827

added a group named "Colliding Display Names"
users with colliding display names will be added to the above-mentioned group in load_v1_users

It appears that users whose display name is the same as their login name get added to the collision group.
This should be fixed.

I cherry-picked 8b14834 895d964 b7e1ccc 0eb5676 7f78bf2 e633537.
Your fix for the infinite redirection works if there are multiple site objects, but causes an infinite re-direction if there is only one (which was working previously). Thus I reverted the Site-object related commits so we just create one Site object during the migration. Will set the Site object manually for now until we figure out what to do about them.


from security import safe_requests

from security import safe_requests

I've had a similar PR opened on https://github.com/akaszynski/keepa/pull/162, it doesn't add value and might be suspect.

Closing as doesn't seem to add anything (thanks @akaszynski for the helpful reference).


Agility Shortcuts Broken

Not accounting for agility short cuts

config

My agility level is more than sufficient for the local obstacles.

This was added in version 1.11 and is now live in the client.


Able to specify jshint function

I want to use this jsxhint plugin, in combination with this gulp-jshint task. An option to provide/overrule the linter would be awesome.
.pipe(jshint({ jshint: require('jshint-jsx').JSXHINT }))

@jtangelder I like this as it also supports using gulp-jshint with a different version of jshint than what is bundled. Can you try out the PR to see if it works for you?
With recent versions of npm you can install it by running  this in your project:
npm install spenceralger/gulp-jshint#overrideJshint

Using this, I end up getting "C:\Users\Mike\Documents\Git projects\collaborations\mofo-example-app\react code\editor-app.jsx: line 0, col 0, Bad option: 'jshint'." errors for my .jsx files. Seems like the linter override option is being passed on to jshint/whatever linter was bound?


timestamps are parsed incorrectly in report on windows

Filenames with : do not work well on windows. Seems the files are possibly renamed while transfering from linux to windows, where : is replaces by _. Next, the parsing of the date in the filename fails.
Example output:
Unable to parse date time from filename: histo.dump.my-service.2022-06-09T19_33_07.txt, will now use file creation date.

File creation date after transfer/copy is also not correct. Modified date might be better. But best is to use filename.

Default timestamps in filenames are now with dashed instead of semi-colons for time, like so: 2021-03-03T00-30-52
Filenames with underscores or semicolons for time are also parsed correctly for compatibility reasons.