Issue loading models quantised with older GPTQ code: ValueError: QuantLinear() does not have a parameter or a buffer named bias.

As discussed in: https://github.com/oobabooga/text-generation-webui/pull/1668
Trying to load for example: https://huggingface.co/anon8231489123/vicuna-13b-GPTQ-4bit-128g
We get:
The safetensors archive passed at /workspace/models/anon8231489123_vicuna-13b-GPTQ-4bit-128g/vicuna-13b-4bit-128g.safetensors does not contain metadata. Make sure to save your model with the `save_pretrained` method. Defaulting to 'pt' metadata.
Traceback (most recent call last):
  File "/workspace/test_gptq_cuda.py", line 29, in <module>
    model = get_model("vicuna-13b-4bit-128g", triton=False, is_llama=True, model_has_desc_act=False)
  File "/workspace/test_gptq_cuda.py", line 21, in get_model
    return AutoGPTQForCausalLM.from_quantized(quantized_model_dir, use_safetensors=True, model_basename=model_base, device="cuda:0", use_triton=triton, quantize_config=get_config(model_has_desc_act))
  File "/usr/local/lib/python3.10/dist-packages/auto_gptq/modeling/auto.py", line 66, in from_quantized
    return quant_func(
  File "/usr/local/lib/python3.10/dist-packages/auto_gptq/modeling/llama.py", line 107, in from_quantized
    model = accelerate.load_checkpoint_and_dispatch(
  File "/usr/local/lib/python3.10/dist-packages/accelerate/big_modeling.py", line 479, in load_checkpoint_and_dispatch
    load_checkpoint_in_model(
  File "/usr/local/lib/python3.10/dist-packages/accelerate/utils/modeling.py", line 946, in load_checkpoint_in_model
    set_module_tensor_to_device(model, param_name, param_device, value=param, dtype=dtype)
  File "/usr/local/lib/python3.10/dist-packages/accelerate/utils/modeling.py", line 131, in set_module_tensor_to_device
    raise ValueError(f"{module} does not have a parameter or a buffer named {tensor_name}.")
ValueError: QuantLinear() does not have a parameter or a buffer named bias.

This model loads fine with the latest GPTQ-for-LLaMa code, both CUDA and Triton branches.
Could such models be made to work in AutoGPTQ as well? If necessary, including some compatibility code for whatever has changed to cause this error?
I know this is a pain, but some of the most popular models on HF are encoded in this way, eg the model above which has 32,227 downloads.
I think being able to support the widest range of models will help the community as a whole move to AutoGPTQ as the one standard for GPTQ.

This seems to be resolved by #43

I'm seeing this issue with a many models as well

@SirWaffle Hi, can you try the main branch and see if this problem still exists?

And set strict=False in from_pretrained() on any model that throws this error

using the latest from the main branch, I fail to import due to missing triton - I'm on windows. The previous version didn't require titon, only spit out a message if I did not have it installed. I will spend some time and see if I can get it compiled manually for windows and setup to see if this addresses the issue

perhaps, as a temporary mitigation an alternate load_quantize function should be written. The function would auto-detect the basename based on common model suffixes and guess the quantize_config from the model/folder name. The function could be marked as deprecated with instructions on how to convert old format models to new without needing to re-quantize.

Hi @Interpause
Unfortunately quantize_config doesn't help us: two models can have identical quantization config and one will work and one will not.
This is because there was a silent change in the GPTQ format.  Models produced with older GPTQ-for-LLaMa code will throw the error described. There is no obvious indication as to whether a model will fail or not.
But we do already have a solution for this - we can pass strict=False to AutoGPTQForCausalLM.from_quantized() and then both old and new format models will load.  That is the current recommended solution for this issue.
We have been discussing whether strict=False should therefore be default.  I think perhaps it should be, although PanQiWei has been explaining that it might have implications in multi-GPU inference scenarios. So that needs to be tested.

Im having a bit of an issue getting triton to work in windows, is there any hacky thing i can do to remove the reliance on triton from the main branch? or is triton now a required package that cant be detangled easily?

using the change here: https://github.com/PanQiWei/AutoGPTQ/pull/85
I can confirm the older models load with strict=false, without triton issues under windows

i can confirm the new change merged in fixed issue with strict=false setting on model load, i would say set it as default, most of my gptq model would simply not load without it

What if the model name & version could be specified within quantize_config.json? The first would solve auto-loading models with custom names, and the second could be used to automatically determine if strict=False or other compatibility measures are needed. Maybe using version:-1 for GPTQ-for-Llama quantized models to automatically turn on strict=False.


RadioGroup validator

It would be nice to allow RadioGroup validation with saripaar.

@ararog Nice suggestion, will do.

Is there an example of how you could custom implement this behavior?

+1

I was just looking to find how to create a custom ViewDataAdapter for a radioGroup
Is there an example ?
Thanks

You can make use of the @Checked annotation on a RadioGroup. The rule will pass if one of the RadioButtons is checked.

@createS0ftware Can you check the latest snapshot and let me know if it works as expected?

@Checked
RadioButton rm;
@Checked
RadioButton rf;
I made a form in which I placed two radio buttons - Male and Female.
I have added the validation as shown above.
The problem is even after selecting one of the radio buttons, it still gives me a message (Must be checked).
Can you please explain this issue ?

@rishabutd Place your RadioButtons inside a RadioGroup and put a @Checked annotation on it.
@Checked
RadioGroup genderRadioGroup;

This should work. You should use your technique only on CheckBoxes.

Yeah! I just tried that. Your code has been of great use.

Thank you :)


Cloaks

It would be great to add the option to define a default cloak for every users, which would replace their real hostname. If I am not mistaken, this is similar to the vhost parameter for opers (?). Being able to attribute a custom cloak to an user would be even better.

Heyo, thanks for the issue! Implementing traditional cloaking is something I've been thinking about, definitely. It's difficult to do it in a way that preserves privacy of your original address as well as making sure people can't ban-evade, which is the main irritating factor about implementing this.
The way I'd likely go about this would be to have another cloak parameter on Client that gets populated when they join. Ban masks and similar would get checked against both the cloak and their actual hostname. However, when using a vhost then ban masks and similar should really only compare their values against that vhost.
I think doing something like this helps general day-to-day people from just seeing your IP address, and since the only action that can be affected by someone being on a banlist (or similar list) is someone sending a reply PRIVMSG/NOTICE to a message, this should help make sure people can't do something similar to that attack on Anope/Atheme where people can discover your real IP address.
I need to dig a bit deeper into this, see exactly what depends off the relevant ban/invite/etc mask lists and make sure there's no way to automagically discover someone's real address (easily) using any of those methods. After I'm reasonably happy with that, I can implement this no problem.
Doing vhosts is... planned, I'll say. But I'm not sure exactly how to do it on the admin side since I haven't digged into how that works with other systems. Maybe just creating a similar HostServ fake client like we have NS and all now.

I'm working on this in devel+cloaks, starting with 0d3b5b23818dccd66bd43978279b67536a51baf0 that adds the cloaking mechanism for IPv4 addresses.
These are the separate things that need cloaking:

[x] IPv4 Addresses (0d3b5b23818dccd66bd43978279b67536a51baf0)
[ ] IPv6 Addresses
[ ] Hostnames

I'm not sure exactly how I should do this, but right now I'm inspiring it off a comment in plexus4's cloaking code that outlines how it approaches this for different address types. For IPv4, this is how I'm approaching it:

IPv4 addresses can be represented as a.b.c.d, where a is the least unique part of the address and d is the most unique part.
a is unique for a given a...*, and d is unique for a given, specific a.b.c.d address. That is, if you have <IP_ADDRESS> and <IP_ADDRESS>, the d part of both addresses should differ to prevent discoverability. In the same way, if you have <IP_ADDRESS> and <IP_ADDRESS> then the a part of those addresses will be the same value. This ensures chanops can properly ban dodgy people as they need to do so.

To do this, I go through something like this method:

Separate out the parts of the IP address into a, b, c, d.
Create an empty string, the key that will be hashed to create the IP octet replacement.
Append the current part to the key, then all less-specific parts that apply to this octet (when hashing c, the key is c+b+a, etc).
After all this, append a static hashing key that's used specifically for this octet of the IP address. Each octet has its own key (so for IPv4 addresses there's 4 hashing keys, etc).
Sha512 the key.
Base64 the result, lowercase it, make sure all separators are - instead of the special chars. Trim it to the first 16 characters and trim any leading/ending - characters. (Thinking about looking at base36 or something similar)
Output a.b.c.d.cloaked-net, where a is the hashed output of a, and net is the netname.

This seems like it should work alright. It avoids double-hashing results like P4's impl does, since I'm iffy about double-hashing things. However, I'm not a crypto guy so I dunno how well it'll work in practice really. Here's some examples:
<IP_ADDRESS>
    -> fhzkguhmza63y8u5.gnqmmmbesue-nawp.ypq8cphg-yjvkygx.fovu8s8zps23y2ck.cloaked-toast
<IP_ADDRESS>
    -> 3d8jjp-w7llcoidc.7gvy4ronc324klje.u5rtmy09f2rbmxmc.qg-cnkwnrt66-6er.cloaked-toast
<IP_ADDRESS>
    -> fhzkguhmza63y8u5.tqnsrgz278qixhwt.taijstboi8ibu27a.ewlrvcls2nod4ikg.cloaked-toast
<IP_ADDRESS>
    -> fhzkguhmza63y8u5.tqnsrgz278qixhwt.taijstboi8ibu27a.w2-tkhrfejfcspsf.cloaked-toast

As a side note, I've split the VHOST system stuff in particular into #109, have been meaning to get to that for a while but it's a liiittle lower priority than just cloaking and some other general features/enhancements.

I've made it actually use a similar HMAC+sha256 -> base32 system as the P4 system above uses. Also shortened it so that they turn out nicer:
<IP_ADDRESS>
    -> xjtqvg6mb2.4grk5upshl.bjip2p4uup.mydoahopjs.test-cloaked
<IP_ADDRESS>
    -> 4yfdxndgf3.jpbkkfnt3g.fhbk6ukspg.5jqti37n7o.test-cloaked
<IP_ADDRESS>
    -> xjtqvg6mb2.w6j3lbv4pb.6g5gtfflr5.itzwzkd73v.test-cloaked
<IP_ADDRESS>
    -> xjtqvg6mb2.w6j3lbv4pb.6g5gtfflr5.7cudw6ngmi.test-cloaked

I think it makes sense to go ahead with this.

Quick question... if the solely reason for host mangling/cloaking is to hide one's IP adress, then why not just replace the host with "users.<network_name>", and a ircd.yaml entry, to use it or not?
e.g. "users.test.oragono" instead of IP, by default...

Good question. I looked back over the ticket and saw a mention of (channel) ban masks, so I assume one reason is to allow channel operators (who are not privileged enough to see the underlying IPs) to ban IPs from their channels.

Someone showed up in #oragono and mentioned the desire for a relatively anonymous chat system.
I'm jumping into this without much context, but rather than hashing every byte of the IPv4 address (and every two bytes of the IPv6 address) separately, I think I might prefer an approach similar to connection limiting:

Identify a CIDR length that is the appropriate target for anti-abuse measures, i.e., a CIDR length that is relatively likely to be associated with a single user (typically, /32 for IPv4 and /64 for IPv6)
Hash or encrypt that portion of the address

This errs on the side of user privacy --- for example, if alice has cloak xjtqvg6mb2.w6j3lbv4pb.6g5gtfflr5.itzwzkd73v.test-cloaked, and bob has cloak xjtqvg6mb2.w6j3lbv4pb.6g5gtfflr5.7cudw6ngmi.test-cloaked, and you know that Alice is from Zagreb, then you can deduce that Bob is probably from Zagreb as well.

We might also want to see if we can avoid deanonymization via repeatedly banning different regions of the IP space, as alluded to here: https://gist.github.com/maxteufel/1e2cf7ada079c271bd3c
AllNickmasks currently returns a nickmask based on the raw IP address, if cloaking is enabled then we should not do that.

Hmm. Something like that sounds... like it makes a lot of sense, honestly. We could probably even output the same sort of masks we do for IPv4 for IPv6 (just choosing our splits differently). Probably makes sense to preserve the order of 'hashed octets', from most specific to least specific, as that makes it a lot easier for users to ban on and is what they expect (e.g. banning *.xjtqvg6mb2.w6j3lbv4pb.6g5gtfflr5.text-cloaked rather than xjtqvg6mb2.w6j3lbv4pb.6g5gtfflr5.*.test-cloaked).
Definitely up for this, yeah.

Ah, I'm suggesting a solution where we take the entire "significant portion" of the address (its CIDR) and scramble that entire portion atomically, not exposing subnet relationships within the portion anymore.

Ah I see, that's very interesting. I could see something along those lines, yeah.

Closed in #481


Update _legal_cookie_chars to include / and =

Both / and = may legally occur in a cookie value without requiring that the value be quoted.
Reference https://www.ietf.org/rfc/rfc6265.txt
While the presence of added quotes should be inconsequential, this is motivated by interacting with a server that did not seem to unquote the cookie value for an auth token, which strictly speaking did not need to be quoted.

You may want to re-commit. You need to configure git with your name and email.


zlib: unexpected end of file

Link to reproduction
No response
Payload Version
Latest Beta
Node Version
20.x
Next.js Version
14.2
Describe the Bug
Getting the zlib: unexpected end of file Error no matter how I try to create a payload project
I have tried adding it to existing next.js project
Creating new blank project
Creating new website template project. I've always got the same error.
Please note I have tried after cleaning the cache it still gives the same error.

Reproduction Steps
npx create-payload-app@beta
Adapters and Plugins
No response

Related to degit's issue here: https://github.com/Rich-Harris/degit/issues/313
The workaround until this is resolved is to delete the .degit directory in your home directory. Depending on your operating system this will likely be ~/.degit or something like C:/Users/username/.degit.

It solved the earlier error, but I still can't add it to an existing Next.js project.
It doesn't even detect the existing Next.js project.


Deploy

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].
In raising this pull request, I confirm the following (please check boxes):

[ ] I have read and understood the contributors guide.
[ ] I have checked that another pull request for this purpose does not exist.
[ ] I have considered, and confirmed that this submission will be valuable to others.
[ ] I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
[ ] I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

[ ] I have never used the project.
[ ] I have used the project briefly.
[ ] I have used the project extensively, but have not contributed previously.
[ ] I am an active contributor to the project.

{pull request content here}

Codecov Report
Base: 83.19% // Head: 83.59% // Increases project coverage by +0.40% :tada:

Coverage data is based on head (f47bf48) compared to base (be29732).
Patch coverage: 85.85% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   83.19%   83.59%   +0.40%
==========================================
  Files          36       36
  Lines        3653     3676      +23
  Branches      434      432       -2
==========================================
+ Hits         3039     3073      +34
+ Misses        484      466      -18
- Partials      130      137       +7

Flag
Coverage Δ

unittests
83.59% <85.85%> (+0.40%)
:arrow_up:

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

Impacted Files
Coverage Δ

mod_ci/controllers.py
72.45% <81.39%> (+1.98%)
:arrow_up:

mod_upload/controllers.py
74.44% <81.81%> (ø)

mod_auth/controllers.py
84.39% <83.33%> (+0.26%)
:arrow_up:

mod_sample/media_info_parser.py
85.03% <87.50%> (-0.79%)
:arrow_down:

mod_test/controllers.py
71.71% <88.88%> (-0.29%)
:arrow_down:

mod_ci/cron.py
100.00% <100.00%> (ø)

mod_ci/models.py
93.87% <100.00%> (-0.13%)
:arrow_down:

mod_customized/controllers.py
97.50% <100.00%> (ø)

mod_deploy/controllers.py
86.32% <100.00%> (+0.35%)
:arrow_up:

mod_regression/controllers.py
94.14% <100.00%> (ø)

... and 7 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Test | Fix Unit Tests for GetSqlServerSPN

This PR is for the addition of Unit Tests of the GetSqlServerSPNs function of SNIProxy class.
The unit tests is to ensure that the GetSqlServerSPNs function returns a port number for TCP protocol and returns an instance name for NP protocol.
In addition, unit tests were added to test the DataTestUtility.ParseDataSource function.

/azp run


CONTRIBUTING: Document PR approval and link to OWNERS

Remove a dead “Becoming a Maintainer” link and that stub section, since libpod currently doesn't document that process.  By leaving it undocumented, maintainer changes falls back to the usual pull-request review process.  I've added a link the OWNERS docs and our local OWNERS file so folks can discover the usual pull-request review process.  This portion of the PR is similar to kubernetes-incubator/cri-o#1272.
I've also updated OWNERS itself to rename assignees to approvers.  This is recommended in kubernetes/test-infra#3851, and is similar to what happened in kubernetes-incubator/cri-o#1103.

LGTM

Those tests are actually known (though infrequent) flakes - timing issues with running commands too close together in a potentially-slow travis environment

@rh-atomic-bot r+


feat(gamedao): add gamedao

Fixes # .
Changes proposed in this pull request:

add gamedao
thank you
success

I'm closing this PR, it's not relevant anymore.


Add metrics about CVE vulnerability ID

Description
This is a proposition about feature request 562.
Which permit to expose new metrics (enable by default but disabled via operator env variable) about CVE vulnerabilities find in VulnerabilityReports
Related issues

Close #562

Checklist

[x] I've read the guidelines for contributing to this repository.
[ ] I've added tests that prove my fix is effective or that my feature works.
[ ] I've updated the documentation with the relevant information (if needed).
[ ] I've added usage information (if the PR introduces new options)
[ ] I've included a "before" and "after" example to the description (if the PR is a user interface change).

@MatthieuFin this change could be a nice addition to trivy-operator metrics capability, however I agree with @JAORMX about the impact of metrics cardinality , I would suggest to make it metricsVulnIdEnabled disabled by default until we gather enough feedback on it usage.

Hi, yes i'm agree with that too !
ok to set it disabled by default, it will avoid bad surprise on upgrade.
But i have no really good workaround on this issue.
We could perhaps record only CVE with a minimum severity. Or reduce number of label maybe set it as configurable. By example we could only keep namespaces and not image repo / name / version.

LGTM 🚀  thank you for the contribution

I was looking for this, but reading the legit concerns I'd like to know if any of you had feedbacks on whether or not cardinality proved to be an issue with this enabled ?


Small fixes of Mako alerter on Github

Do not add the issue detail in the comment that reopens the issue
The first comment in the issue should be index 0, instead of 1

/cc @vagababov
/cc @adrcunha

/cc @mattmoor

/approve
/lgtm


当前版本不再支持lmdb了吗

Go 1.6 can't compile LMDB package now,we should wait the package fixes it first or choose another proper one.

Isn't that proper library: https://github.com/bmatsuo/lmdb-go ? It looks like lmdb has better performance on Sorted Sets and Lists operations

Cool, but I have no time to embed it now, very busy in this month. 😭


refactor: Do not use Roslyn

Description
Refactored code generator to no longer uses Roslyn.
The reason for this is that Roslyn imports certain Span<> based types, which crashes all our type lookups, including type lookups from external SDKs such as Facebook.

This pull request is included in version 1.5.0


Change initials

Is it possible to change the acounts initials at the overview.
I have multible acounts from the same domain and it's hard to differ.

Duplicate of #8381


Missing Object.assign member

I have a piece of JS code that does the following:
    var a = Object.assign({}, state, {
      isAuthenticating: false,
    });

where state is an empty object, and even though it runs fine in the browser when bundled, it fails running that same bundle in goji with the following error:
TypeError: Object has no member 'assign'

Object.assign() is not part of ES5.1. If you want to run ES2016 code you may want to use Babel with appropriate polyfill plugins (in your case, this one might work: https://babeljs.io/docs/plugins/transform-object-assign/)

I thought about that.. but it turns out that the babel plugin is actually turning:
Object.assign(a, b);

into
var _extends = ...;

_extends(a, b);

and that doesn't really look much ES5.1 as well..

It does to me. Have you actually tried it?

@tanis2000
I think you're taking the var _extends = ...; too literally. Pretty sure the ... is meant to represent whatever the polyfill puts in its place.
Having ... as an expression isn't valid in any version of ES.

You guys are right. I took it literally for the spread operator. I tried it and it works so it's definitely not literally that :)
Cheers!


Build card-based layout (testing is a separate card)

What:
We want to know whether pre-defined searches would be helpful to our users both at GSA and outside of GSA.
Why:
To understand whether this will simplify their workflow and is worth pursuing further.
Potential design options:

we might want to have buttons on the top to do some of the most important searches 🤷♀
we might want SSN, ICAM (usernames, PIV), High risk systems more visible 🤷♀
maybe there could be combined term searches so that a user types "usernames, PIV" and it returns results that have those two separate terms available.

Or something else . . .
@nikzei and @igorkorenfeld to pair on research + testing
Acceptance (we know we're done when):

[x] we have a live webpage based on the mock ups and all its features
[ ] @igorkorenfeld to verify that webpage meets the features he designed

First round of Acceptance

[x] Move matching count to the right. Float?
[x] Matching count not updating on first SORN and PIA checkbox click.
[x] Quick filters styling and functionality. It should show as an active filter.
[x] Styling of the pagination
[x] Maintain same distance between filter and the card . Match right side of card to top hr.
[x] Gutter spacing between PII columns
[x] Quick filters show up on Active Filter pills
[ ] Work with Igor on max height of filter section background

Check our notes from a recent session with Marcela.
Have a conversation with Marcela about other use cases for search.

Still need to test mocks.
Next:

recruiting
figure out how we want to test - so we know whether Invision is fine or if we want to build this out before testing

So far, we want:

we want to test for the usefulness of pre-defined search buttons (SSN, etc)
card lay out (working prototype would be best here)

Mockup of the card direction:

With Annotations:

Notes
(Giving a little more context to the annotations)

For the System, SORN-ID, and PII boxes I was picturing these would be "type-ahead" filters where a user could start typing a query and it would show them the available options. That way the filters are limited to the options that are actually in the system, but in way that is more manageable than a long dropdown. These boxes are important to helping narrow down a list of results and doing a targeted search for a specific field, rather than seeing where a keyword might show up.

For the list of PII in within a card, I think it would be helpful to the user if the list was in two-columns and alphabetized. My guess is that this would be easier to scan through and read than grouped in a paragraph or an unsorted list.

"Matching x of y" on the top right should show how many results are left after filtering to help the user decide if they need to apply more filters.

Mockup of the table layout:

@igorkorenfeld I'm struggling to find a way to be able to do multiple separate searches on the same data. Its very complicated to code and it doesn't seem like a common web pattern. I don't know if I've seen multiple search bars for a single data source before.
I suggest we do a single search bar across all the fields. As we talked about in our meeting, for now we've pulled out most of the other columns of data that were polluting the search results anyways.
If we find through user testing that we do need more advanced search options, I vote for doing a hidden advanced search menu. It could offer filtering of fields for the single search bar. That is a more common pattern on the web and will be a lot easier to build.
See the advanced search options at https://lucaong.github.io/minisearch/examples/

Also the USWDS recommends not offering advanced search as the default.

Don’t offer advanced search as the default. The majority of people will do a simple search with one or two search terms. If advanced search is offered, it increases the likelihood of mistakes.

Feel free to push back on any of this. You've got to represent what you've learned from user testing, while I'm going try and be lazy and ask for the easy way every time.

WIP ready at https://cg-9341b8ea-025c-4fe2-aa6c-850edbebc499.app.cloud.gov/site/18f/privacy-dashboard/
For data it has system name, url, and pii. To get the type and sorn id in there, we'll need to do some spreadsheet munging. There is a lot to clean up in our data workflow still.
For features it has a single search and pagination. I still need to figure out typeahead, and pre-filters. I've got ideas for both.

This is great!
The advanced search suggestion could definitely work. I agree that without the other fields it seems like the user would get more relevant results and we may not need the more granular filtering options upfront. I think we can find out in feedback sessions.
Got it on the type and ID. I think if we could at least get the type on there that would helpful. Right now it's hard to tell what the result is if there is both a PIA and a SORN. For instance, searching for "child" returns two boxes both with the title "Child Care Subsidy."
For the multiple boxes, I think we might be framing it slightly differently. I don't thing of them as separate search boxes but rather as filter boxes. The way I think of it, search is additive, so you are adding to the screen all items that match the keyword. While filter is subtractive, and you are removing anything that doesn't match the keyword.
Here are two examples that have multiple typeahead filters:
Job Search – Where each category of filter has a typeahead
Sofa search  – Here certain lists like material and style have a typeahead style filter.
I am guessing the filtered out results are not happening client side in this examples though.

@igorkorenfeld I figured out how to do multiple search boxes filtering on the list. It wasn't as hard as I thought it would be.

awesome!

[ ] Move matching count to the right. Float?
[ ] Matching count not updating on first SORN and PIA checkbox click.
[ ] Quick filters styling and functionality. It should show as an active filter.
[ ] Styling of the pagination
[ ] Maintain same distance between filter and the card . Match right side of card to top hr.
[ ] Gutter spacing between PII columns

Just noticed this bug:

I noticed another strange bug.
Whenever any of the filters have just character, the tag for the filter changes label is "System Name." This is true for System too, it switches from "System Name" to just "System" (the latter being the correct label)

2 questions for when we review this work:

Should we show checkbox filters as active and inactive in the toolbar section? They do in the mock-up, but the boxes themselves are pretty easy to grasp.
If we don't do that, should we rename the toolbar section to 'Active Searches' instead of 'Active Filters'?

That's a good point, I think it makes sense to include them as active filters in the toolbar area for both clarity and consistency. I think it'll reinforce how thee filters work and how they should be read.

@igorkorenfeld I got the "Quick Filters" to show up as pills in the "Active Filter" section.

I added a PII: label to them. Is that how you wanted it?

Yes, that works great


.vimrc sourced after saving: perhaps include sourcing when user extensions are saved

The .vimrc file maintained by the hvn script triggers sourcing of the rc file when saved.  Is it possible to have the event trigger include changes to vimrc.local and plugins.vim?

Sorry, can you explain the problem in more detail?

@EdmundsEcho I see what you mean, the augroup here - https://github.com/begriffs/haskell-vim-now/blob/master/.vimrc#L281-L288
It's definitely possible to source the other files as well. Probably just add the filename after $MYVIMRC. Want to try it locally and send a pull request?

Motivated by your feedback I found a way for it to work.  If what I propose makes sense (i.e., it's not considered a hack), then the place to make the changes would be to update the documentation for how to create and use vimrc.local.
In my vimrc.local I now include the following:
" Source the vimrc file after saving it
augroup sourcing
  autocmd!
  autocmd bufwritepost vimrc.local source $MYVIMRC
  autocmd bufwritepost vimrc.local source ${HOME}/.config/haskell-vim-now/vimrc.local
augroup END

For some reason, I can't seem to just trigger the sourcing of vimrc.local with just the first line... However, when I include these two lines in my vimrc.local, I do not need to change the .vmrc (or the nvim equivalent) to get both files to be sourced when either file has been changed.


Build pipeline

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

Ad build tasks


add test that tests binary values get converted to categorical

Close #19

I'm going to close this PR because of inactivity. That probably tells us something about the underlying issue.


add dataProxy parameter

Allows users to use this plugin when using the prisma data proxy.

Can you enable "Allow edits from maintainers." please?

Doesnt seem i can 'allow from maintaners" since i forked it into an Organization. Lame. I invited you with write access to the fork. Hopefully that fixes it or I can refork into a persona github. LMK

Amazing! will watch for NPM release. Thanks


Patch Reaper for multiple CVEs

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./LICENSES-AND-NOTICES/SPECS/data/licenses.json, ./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md, ./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Change Log

CVE-2024-45296
CVE-2024-43799
CVE-2024-43800
CVE-2024-47764
CVE-2024-48949

Does this affect the toolchain?

NO
Associated issues

#xxxx

Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2024-45296
https://nvd.nist.gov/vuln/detail/CVE-2024-43799
https://nvd.nist.gov/vuln/detail/CVE-2024-43800
https://nvd.nist.gov/vuln/detail/CVE-2024-47764
https://nvd.nist.gov/vuln/detail/CVE-2024-48949

Test Methodology

Local Build
PR Checks

Auto cherry-pick results:

main :x: -> https://github.com/microsoft/azurelinux/pull/10735

Auto cherry-pick pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=658001&view=results


[FLINK-18399] [table-api-java] fix TableResult#print can not print the result of unbounded stream query

What is the purpose of the change
In current implementation of PrintUtils, all result will be collected to local memory to compute column width first. this can works fine with batch query and bounded stream query. but for unbounded stream query, the result will be endless, so the result will be never printed. To solve this, we can use fix-length strategy, and print a row immediately once the row is accessed. This pr aims to fix the bug.
Brief change log

[hotfix] correct the logic of truncateString method in PrintUtils
Correct the logic of PrintUtils for unbounded stream query case

Verifying this change
This change added tests and can be verified as follows:

Extended PrintUtilsTest and CliTableauResultViewTest  to verify the fixes

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc @twalthr @wuchong

@wuchong @twalthr Thanks for the suggestions, I have updated the pr based on comments, please take another look

They are rather confused about having no output by default.

agree with you, for 1.11, we throw an exception if checkpoint is not enabled for streaming query. @TsReaper have open a pr https://github.com/apache/flink/pull/12750. In 1.12, we will support at least once semantics soon

Merging...

FYI: I merged the one for release-1.11 #12752

Thanks for the notice @wuchong. I didn't know that there were two PRs for the different branches.


LOVE_GT_090 incorrectly set to false, Windows 7

I've had some issues running the script on my new Windows 7 setup. It seems that the variable LOVE_GT_090 is incorrectly set false for my 0.9.2 game, meaning that the script tries to curl love-0.9.2-win-x86 which doesn't exist in https://bitbucket.org/rude/love/downloads/.
Note: the version wasn't detected at all at first: it was using the default 0.9.1 despite the 0.9.2 in the conf.lua file. I hard set the default LOVE_VERSION=0.9.2 and LOVE_GT_090=true in the script in order to fix this.
This may have something to do with some grep and gawk errors I'm getting:
$ ./do-build.sh > log.txt
grep: invalid option -- o
Usage: grep [OPTION]... PATTERN [FILE]...
Try `grep --help' for more information.
grep: invalid option -- o
Usage: grep [OPTION]... PATTERN [FILE]...
Try `grep --help' for more information.
gawk: cmd. line:1: END { exit ( !(  >= 0.8)); }
gawk: cmd. line:1:                  ^ parse error
gawk: cmd. line:1: END { exit ( !(  >= 0.8)); }
gawk: cmd. line:1:                         ^ parse error
gawk: cmd. line:1: END { exit ( !(  >= 0.8)); }
gawk: cmd. line:1:                          ^ parse error
gawk: cmd. line:1: END { exit ( !(  >= 0.9)); }
gawk: cmd. line:1:                  ^ parse error
gawk: cmd. line:1: END { exit ( !(  >= 0.9)); }
gawk: cmd. line:1:                         ^ parse error
gawk: cmd. line:1: END { exit ( !(  >= 0.9)); }
gawk: cmd. line:1:                          ^ parse error

Hi ! Thank you for the error report. For now I don't really know how to fix it, but I've been rewriting love-release for a week and I'll be done in a couple of days, so it will certainely fix your problem.
You can check the progress on the rewrite branch. To get a look at the new compare_version function (which is used to... compare two LÖVE version strings), you can check this Gist (I hope I haven't done any basic/logic mistakes on such a simple thing).
Oh, and it never should have been named LOVE_GT_090 but LOVE_GE_090 as the variable is set to true if LÖVE is greater-or-equal (and not greater-than) 0.9.0. I'm sorry that love-release is so broken ! But hopefully the rewrite will fix the problems you've encountered.

Righto, I'll give that a look tomorrow night!


How to load data on demand with paging ?

Actually this is not an issue.
I want to know how to load data through ajax request based on the pages selected ?
Say for example
I have 1000 records in my database, I want to show only 10 at a time. When I click on the second page, it should load the next 10 records.
Hopes this understands. Please help me with example.
Thanks.

@heiovsky, thanks for the lead!


Using validator.addMethod to extend 'required' throws 'e.validator.methods[o] is undefined' error

With the following references:
script src="http://code.jquery.com/jquery-1.9.1.min.js"
script src="http://code.jquery.com/ui/1.10.1/jquery-ui.min.js"
script src="http://ajax.aspnetcdn.com/ajax/jquery.validate/1.11.0/jquery.validate.min.js"
Extending the 'required' validator using this code:
jQuery.validator.methods.oldRequired = jQuery.validator.methods.required;
jQuery.validator.addMethod("required", function (value, element, param) {
if (!frm.isNotHint(value, element, param)) { return false; }
return jQuery.validator.methods.oldRequired.call(this, value, element, param);
},
jQuery.validator.messages.required
);
Causes the line "return jQuery.validator.methods.oldRequired.call(this, value, element, param);" to throw this error:
TypeError: e.validator.methods[o] is undefined
http://ajax.aspnetcdn.com/ajax/jquery.validate/1.11.0/jquery.validate.min.js
Line 4
FYI - The method frm.isNotHint indicates whether the input field contains 'hint' text, and is executed without error.
If the script reference is changed to version 1.10 (http://ajax.aspnetcdn.com/ajax/jquery.validate/1.10.0/jquery.validate.min.js), no error is thrown.

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually.
To the reporter (or anyone else interested in this issue): If you're affected by the same issue, consider opening a new issue, with a testpage that shows the issue. Even better, try to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!

Good call. As the reporter on this (aka, mtnrambler) I've long moved on and could probably not even reproduce it at this point!


(#6335) Allow optional trailing comma in argument lists.

Support optional comma at the end arguments lists for parameterized
class and user defined resources. This makes it behave similar to
resource attributes.
The parser.rb file is generated with racc 1.4.6 against the grammer.ra
file and if this conflicts please remove parser.rb file and run make in
the directory to generate it.

I cherry-picked, and merged into 2.7.x in e2aec6375fe0d25e31bade0e35f39ec25423e844 (cleaned up commit message).


Error: column "base_url" and start_address.geofence_id does not exist

Hello,
having a slight issue with postgresql (and therefore nothing will display in some of the Grafana views).
Getting the following messages in postgresql:
2019-10-02 16:59:31.496 CEST [50] ERROR: column "base_url" does not exist at character 8
2019-10-02 16:59:31.496 CEST [50] STATEMENT: select base_url from settings limit 1;
2019-10-02 16:59:32.382 CEST [50] ERROR: column start_address.geofence_id does not exist at character 2488

It seems that teslamate cant update the database with the new columns added in some of the newest version(s) somehow? I have added the teslamate database user as a superuser. Even tried running:
/opt/app/bin $ ./teslamate eval "TeslaMate.Release.migrate"
17:11:16.823 [info] Already up
/opt/app/bin $

Everything else is working fine.
Thank you for a great software :)
edit: ah, I may have found out why this has happened. I am running version 1.9.1, not 1.10 - but the Grafana views I have are running the newest straight from github. Doh! I guess everything will be alright when 1.10 will be released :)

I was really wondering how this could have happened. I'm glad you found the solution! :D


Support multiple patient email addresses - Frontend [3/3]

Related Issue or Background Info

Closes #1951

Changes Proposed

Adds a ManageEmails component for use in patient self-registration and the admin "edit patient" forms

Overall this very closely resembles the component to manage multiple phone numbers - this was by design, but there's a decent amount of copy-and-paste. As we go forward we may find a way we like to DRY them out.

Introduces to the UI the notion of a "default" or "primary" email

This field cannot be deleted by the user like additional email fields can, but it is still optional

Additional Information

N/A

Screenshots / Demos
Add/edit person form
https://user-images.githubusercontent.com/27730981/141360991-c1213799-6338-42e2-af9d-aaba6da77e77.mov
AoE form

Re-send test results via email

Multiple emails are sent
https://user-images.githubusercontent.com/27730981/141379923-06a4db2f-eddf-4444-9bf9-e0b47649f7c4.mov
Checklist for Author and Reviewer

[ ] Getting the yup validation right was a bit of a pain - did I miss any sneaky edge cases?

UI

[ ] Any changes to the UI/UX are approved by design
[ ] Any new or updated content (e.g. error messages) are approved by design

Testing

[ ] Includes a summary of what a code reviewer should verify

Changes are Backwards Compatible

[x] Database changes are submitted as a separate PR

[x] Any new tables that do not contain PII are accompanied by a GRANT SELECT to the no-PHI user
[x] Any changes to tables that have custom no-PHI views are accompanied by changes to those views
(including re-granting permission to the no-PHI user if need be)
[x] Liquibase rollback has been tested locally using ./gradlew liquibaseRollbackSQL or liquibaseRollback
[x] Each new changeset has a corresponding tag

[x] GraphQL schema changes are backward compatible with older version of the front-end

Security

[x] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[x] Any dependencies introduced have been vetted and discussed

Cloud

[x] DevOps team has been notified if PR requires ops support
[x] If there are changes that cannot be tested locally, this has been deployed to our Azure test, dev, or pentest environment for verification

It seems like the backend is not returning the emails correctly.

@zdeveloper I'm not seeing that... at the risk of asking a dumb question, do you need to restart your backend or something?

@Rebecca-LM now that we support multiple email addresses, this text will need to be changed:

Would you like to receive results at this email?

Probably the only question there is whether "email address" is more correct than "email" in this context.

@Rebecca-LM now that we support multiple email addresses, this text will need to be changed:

Would you like to receive results at this email?

Probably the only question there is whether "email address" is more correct than "email" in this context.

@nathancrtr do you have a screenshot of what it looks like now for context? Without seeing that I'd recommend changing to this. Any singular result is just result. Multiple results are results.
"Would you like your test result sent to this email?"

@Rebecca-LM now that we support multiple email addresses, this text will need to be changed:

Would you like to receive results at this email?

Probably the only question there is whether "email address" is more correct than "email" in this context.

@nathancrtr do you have a screenshot of what it looks like now for context? Without seeing that I'd recommend changing to this. Any singular result is just result. Multiple results are results.
"Would you like your test result sent to this email?"

Yes! See below.

I think this is what you're saying, do I have it right:
One email address - "Would you like your test result sent to this email?"
Multiple email addresses - "Would you like your test result sent to these emails?"

@zdeveloper good call on the "primary" and "additional" language. I've removed those words from the labels, although the person form component still maintains a notion of "primacy" for some UI reasons (e.g. so you can't delete every email field from the form). See 6ae9b5e.

@Rebecca-LM now that we support multiple email addresses, this text will need to be changed:

Would you like to receive results at this email?

Probably the only question there is whether "email address" is more correct than "email" in this context.

@nathancrtr do you have a screenshot of what it looks like now for context? Without seeing that I'd recommend changing to this. Any singular result is just result. Multiple results are results.
"Would you like your test result sent to this email?"

Yes! See below.

I think this is what you're saying, do I have it right:
One email address - "Would you like your test result sent to this email?" Multiple email addresses - "Would you like your test result sent to these emails?"

Thanks for the screenshot and yes, that sounds good to me! Thanks @nathancrtr


Fix error catcher tests

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
✔

Tests pass?
✔

Fixed issues

1) Yiisoft\Yii\Web\Tests\Middleware\ErrorCatcherTest::testDefaultContentType
Error: Call to protected method Yiisoft\Di\Container::set() from context 'Yiisoft\Yii\Web\Tests\Middleware\ErrorCatcherTest'

Merged manually. Thanks!


Use Settings.app_scheme instead of hardcoded protocol in helper test

This fixes a test that was failing for me locally.  In a helper test, it uses the value from Settings.app_scheme instead of a hardcoded http:// to construct a URL.  This is related to the switch to SSL.

Rebased to develop


Server boost badges don't work

Title says it.

hopefully fixed,

Yeah, it works now! Thanks @mr-miner1.


Select output device

Select audio output device on chrome

https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/setSinkId
In FF it is behind flag, but it is coming.

duplicate #94


got a segmentation fault while trying to login to jsng shell

Got a segmentation fault while trying to login to jsng shell (jsng has been installed on docker)

Commit_ID abbeba613319e6ff3b02143a6ba18310575ca266 , Branch development

Cannot reproduce, tried with this image again https://hub.docker.com/r/threefoldtech/js-ng

I can reproduce on a fresh manual installation using poetry on ubuntu:19 container.
(js-sdk-w2FClMcC-py3.7) root@5c2a4182417d:/js-sdk# jsng
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
/usr/lib/python3.7/importlib/_bootstrap.py:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
  return f(*args, **kwds)
Segmentation fault (core dumped)
(js-sdk-w2FClMcC-py3.7) root@5c2a4182417d:/js-sdk#

Some related issues in other projects suggest downgrading greenlet, main issue is here https://github.com/gevent/gevent/issues/1260.

Now this issue should not happen in js-ng as long as we install without poetry update (current locked deps are ok).
But in the manual installation for js-sdk, we do poetry update, which should not happen, as we already track a poetry.lock file, people do not need to do this, as this might produce an installations with dependencies we didn't test, with something like the current greenlet broken ABI for python3.7 on ubuntu 19.10 (what I tests).
I suggest we close this here, and re-open a new issue in js-sdk repository to set current greenlet version to greenlet==0.4.16 (as it works for me), and also push an updated and working poetry.lock file, and remove poetry update step from all installation instructions as this need to be done by maintainers only.

Current working versions in js-ng environemnt:
gevent==1.4.0
greenlet==0.4.16

Current broken versions in js-sdk environment:
gevent==1.4.0
greenlet==0.4.17

Related issue:
https://github.com/python-greenlet/greenlet/issues/178

They yanked the broken version for now:

I yanked 0.4.17 for the time being.

https://github.com/python-greenlet/greenlet/issues/178#issuecomment-696723056

closed in favor of https://github.com/threefoldtech/js-sdk/issues/1713


Devcompiler's Object.toString doesn't match other platforms.

If you do
class C {}
main() {
 print(C());
}

other platforms print [Instance of 'C'], but dartdevc prints Instance of 'C' without the braces. I'm guessing this is the JS Object.prototype.toString behavior, but it is inconsistent.
If anyone depends on the format (which they obviously shouldn't since it isn't specified, but when has that ever stopped anyone) then they get different results in the dev-compiler.
(This was tested with dartdevk, the same test file failed for a different reason (#34806) with dartdevc).

FYI, it's not calling JavaScript toString. DDC just has an older copy of the SDK impl code (originally from dart2js), so it's defined in our core_patch.dart file as:
  @patch
  String toString() =>
      "Instance of '${dart.typeName(dart.getReifiedType(this))}'";

Easy fix. :)

errr, I'm confused... other impls like Dart VM/dart2js seem to also print it without the brackets. I get the same output. Was there a different repro for this?

I'm confused too. I can't see any change to the other platforms' behavior, and yet they now all print without square brackets. Maybe I was seeing things?
So, everything is fine. Keep up what you are doing! :)


Init GTM after nuxtServerInit

Hey,
Im getting correct GTM code from backend API. So I get data after nuxtServerInit How I can init GTM after this?
I tried to use plugins, but those are before nuxtServerInit as far as I tested, so I don't have any data in my store yet.
Nuxt.js @ v2.14.7
Thanks

No one? :D

No one? :D


Supported hardware?

Is there a list of supported hardware somewhere? We are planning on migrating our current solution to a self hosted Jitsi-setup and we have some fairly pricy Cisco SX10 web conference systems which we would ideally like to keep.

This repository is about ‘Jitsi Desktop’, a desktop app for the operating systems Microsoft Windows, Apple macOS, and Linux (Debian, Ubuntu). It is very much like a client. Are you about another product, camera hardware, or interoperability with a specific product?


Poloniex USDT balance trouble

The current balance in USDT from Poloniex, does not come in daily messages. But with Bittrex has no problem. ver 0.192

Poloniex working OK for me for some time now.


Changed scripts to fix broken Docker builds

Fixes: #1216
Ref: #1073
These changes should allow for the Docker Builds to work, and subsequently the buildDocker.sh script.
The unbounded variables problem has been fixed by the if statements in docker-build.sh. It's not a very clean way of fixing it, but it works if the variables haven't been used in the case of not using --debug-docker or --ssh with make-any-platform.sh.
The problem of the ./gradlew script no being found has been fixed by adding that into the Docker container, in docker-build.sh by the addition of -v "${hostDir}/pipelines":/openjdk/pipelines \ .
With the current version of Gradle that is put onto the containers, it can only be executed with Java-8. Therefore, all Dockerfiles have zulu-8, along with the JDK version needed for the JDK_BOOT_DIR var, with zulu-8 being set to their JAVA_HOME variable.
build.sh had to be changed slightly, as in its current form it is putting JAVA_HOME to be what the JDK_BOOT_DIR is.

I think I overrode my other comment.  Is it also possible to use an adopt binary as opposed to zulu?  Just for dog fooding purposes.

@karianna That makes sense, and @sxa555 suggested it too. I mainly put in zulu for ease, as a placeholder :-)

Approved but not landing until Java 13 is out


JavaScript native keystore

Makes CIP8 very unusable
Getting it into CLI; don’t have to worry about browser capability, reads from file from file
Extend wallet interface
~ 1d work
Wallet channel cannot access what they need to

RPC with geth node with native node — some keys kept in plan text (file system store); use contract kit and hook up to this store; contract kit uses this in memory wallet (DEK)
Do run a geth client and some things (Signing Key) native bridge

Would Validators like js keystore — filesystem and local store interface
Where does Metamask store private key?
.celo file

Added old description for posterity

Makes CIP8 very unusable
Getting it into CLI; don’t have to worry about browser capability, reads from file from file
Extend wallet interface
~ 1d work
Wallet channel cannot access what they need to

RPC with geth node with native node — some keys kept in plan text (file system store); use contract kit and hook up to this store; contract kit uses this in memory wallet (DEK)
Do run a geth client and some things (Signing Key) native bridge

Would Validators like js keystore — filesystem and local store interface
Where does Metamask store private key?
.celo file


Terminal and Git issues

Type: Bug
On Linux when using the terminal to do git commands, it will overide the user credentials with the credentials stored in vscode. Prevented new connections to github, as I could not get the credentials to refresh. Attempting to modify git global credentials would show as working, however when attempting a git clone, you still run into the same issues. It also will ignore the global when setting password manager.
VS Code version: Code 1.72.2 (d045a5eda657f4d7b676dedbfa7aab8207f8a075, 2022-10-12T22:16:26.920Z)
OS version: Linux x64 4.15.0-142-generic
Modes:
Sandboxed: No

There should be better documentation regarding what all vscode does with the environment when spawning terminals/shells (and the same could be said of other processes vscode spawns).
For instance, I see vscode sets GIT_ASKPASS:
$ env | grep GIT
VSCODE_GIT_ASKPASS_NODE=/usr/share/code/code
GIT_ASKPASS=/usr/share/code/resources/app/extensions/git/dist/askpass.sh
VSCODE_GIT_ASKPASS_EXTRA_ARGS=--ms-enable-electron-run-as-node
VSCODE_GIT_IPC_HANDLE=/run/user/1000/vscode-git-c4e92f925a.sock
VSCODE_GIT_ASKPASS_MAIN=/usr/share/code/resources/app/extensions/git/dist/askpass-main.js

Searching online, the only reference I can find is here: https://code.visualstudio.com/docs/terminal/advanced#_extension-environment-contributions

Extension environment contributions
Extensions are able to contribute to terminal environments, allowing them to provide some integration with the terminal. For example, the built-in Git extension injects the GIT_ASKPASS environment variable to allow VS Code to handle authentication to a Git remote.
If an extension changes the terminal environment, any existing terminals will be relaunched if it is safe to do so, otherwise a warning will show in the terminal status. More information about the change can be viewed in the hover, which also includes a relaunch button.

I get that extensions can add arbitrary additional env vars, but it would be nice for vscode to document what (and why) it adds to the environment.

@bigowlrrww, by terminal do you mean the VS Code integrated terminal? If so, you can toggle the git.terminalAuthentication setting so that VS Code will not add the VSCODE_GIT_ASKPASS* environment variables. Could you please give that a try and let me know if that resolves your problem. Thank you!
@cstrahan-blueshift, thanks for your feedback. We will look at improving the documentation.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Bare gits needs to have refs/heads/ included in ref name (go-git.v4?)

When trying out latest Zoekt (@master), it seems that the full branch name (**refs/heads/**master) needs to be specified for bare gits to allow Zoekt to actually index them.
In our previously deployed version of Zoekt (pre go-git.v4, so I'm guessing this is the change that brought on the issue) it was fine running zoekt-git-index --branches  master,stable-* on bare gits and have Zoekt indexing the specified branches. This is not the case anymore and is a bit problematic since the full ref-spec is now stored into the index and presented in the UI.
When branches are presented as "refs/heads/master" in the UI it hinders readability and wastes screen real estate in the UI. I'm guessing this also this affects branch searches.
I can try making a patch for it, though I'm not sure if we should change how zoekt-git-index interpret branch names, or just change how the UI presents the information.

yeah, this could have been the go-git transition. The relevant code would be under gitindex, and there is an option BranchPrefix that should hold refs/heads/ . The prefix should not go into the actual branch name, afaict:
https://gerrit.googlesource.com/zoekt/+/f3e8b54db17ec67f8108123bac197194f51d1288/gitindex/git.go#342
maybe expandBranches doesn't behave as expected?

Sven made a match which seems to work for us:
https://gerrit-review.googlesource.com/c/zoekt/+/137410

can you tell me more about the failure mode? I can't reproduce the problem.
$ git clone --mirror https://gerrit.googlesource.com/zoekt
$ cd zoekt.git
$ git show-ref|grep master
e25552763cd3c371a52cda438d8e194b775b8072 refs/heads/master
$ zoekt-git-index --branches=master .
2017/11/05 14:03:11 finished /home/hanwen/.zoekt/gerrit.googlesource.com_zoekt_v14.00000.zoekt: 1261902 index bytes (overhead 3.4)

That will work because the default -prefix is "refs/heads/".
Since we index Gerrit configurations (refs/meta/config) we need to set "-prefix" to an empty string or else we cannot index the "refs/meta/"-branches.
Pre go-git.v4 we could set -prefix to "" and it would still find and index a ref like "master". That way we could index both "master" (not "refs/heads/master") and "refs/meta/config", which was perfect for us.
Since go-git.v4, zoekt-git-index wont find "master" anymore with -prefix "", so we have to specify the full branch ref: "refs/heads/master" which clutters the UI (and I guess affect the searches).
I guess people that index refs/notes and similar namespaces will have the same problem.

This problem is solved by this patch:
https://gerrit-review.googlesource.com/c/zoekt/+/137410

fixed by https://github.com/google/zoekt/commit/e1a95db42e6a5292b63b6f5a9a4e2d24ffcbbd68


New fatigue example

closes #313
Based on #396 because I first tried to implement it in terms of the function implemented there. Turns out this does not work because of a bug in dpf core (see comment).

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (a67ea92) 91.97% compared to head (79d9bc9) 92.03%.

Files
Patch %
Lines

src/ansys/dpf/composites/ply_wise_data.py
97.50%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #405      +/-   ##
==========================================
+ Coverage   91.97%   92.03%   +0.06%
==========================================
  Files          48       49       +1
  Lines        3201     3241      +40
==========================================
+ Hits         2944     2983      +39
- Misses        257      258       +1

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

@janvonrickenbach I changed the base branch to feat/ply_wise_plots to minimize the shown differences. It should automatically revert back to main once #396 is merged.


Haproxy: DrainTimout breaks service-restart

Since upgrading to 1.6.12 we've noticed that setting a drain_timeout_ms (as per #2777) breaks restarting services.
You can reproduce this very easily: Create a stack with a single HAProxy and a single service with one container.
Then, restart the single container and observe the proxy setting the backend-weight to 0 (to drain the connection).
NOTE: You don't even need an active connection for this to break!
As the service is not upgraded but restarted, the lb-controller will pick up the change in rancher but generate the very same config file, thus not re-evaluating it.
In the end, this leads to the backend being stuck at weight 0 and the service not receiving any new connections, even though it was restarted successfully - returning 503 to every request.
In the HAProxy stats you can actually see the service being stuck with weight 0:

I guess a good approach would be to reset the weight to 1 once the service is running again (in rancher); this way there would be really no downtime (as long as there's at least a second container).

Useful
Info

Versions
Rancher v1.6.12 Cattle: v0.183.30 UI: v1.6.30

Orchestration
Cattle

Can you try v1.6.22, please? It should be fixed.
Please let us know in a comment if you can still reproduce the issue, and we'll reopen it


All content vanished after update

I use YunoHost  <IP_ADDRESS> (stable) and now Yunowiki Version: 3.2.12~ynh5
Since my last update all my content of the wiki is gone. I didn't expected this and use a couple of apps in my yunohost, so I can't say it for sure when it happened. I did an greater update last friday but didn't noticed something strange. today I want add some new content  in my wiki but there is only the default lionwiki content. Luckily it's not my main wiki, but could this really happen and how to prevent?
Add: I did this complete update with my Browser at the Yunohost Backend.

@Struppi I'm also investigating the recent update changes.
As the wiki is text based, all your content should still be there.
Log with ssh if you can, then cd to /var/www/lionwiki-t2t (if you have several instances of the wiki, it might have several others such as /var/www/lionwiki-t2t__2/ etc)
all pages should be inside /var/www/lionwiki-t2t/var/pages/
it might be the main.txt page has been altered by the update
What happens if you browse on the wiki the "recent" pages? All the ones in var/pages should be there
you can get it at youryunohostserver.com/lionwiki-t2t/index.php?action=recent

yes, I can login via ssh. I have one instance. the main.txt is there but just contain the new content.
The History (recent changes) shows only changes from the default installation and since the last update. The main site exists only since the update.
[While I wrote this, there was a new update for Lionwiki]
I just update now and it remove again the content. But something is strange. Before the update the timestamp of most files in /var/www/lionwiki-t2t/var/pages/ was from July 2023, except main.txt was from 5.2.2024
Now it is all from 2.2.2024

@Struppi did you wrote other files beside main.txt?
main.txt shouldn't be modified but if is had been, it should be fixed. Anyway, even if this happened, other txt files shouldn't be removed at all!
I'm trying to upgrade 2 different instances of this app, on 2 different servers, from  3.2.12~ynh4 à 2024.02.02~ynh1 and it failed in both case so I can't really see what happened...

Yes, I createtd some other sites inside the wiki, but they are all gone. I know have "2024.02.02~ynh1"

I'm sorry about that. At least there should be an archived with backup before the upgrade inside  /home/yunohost.backup/archives/ (it should be named like lionwiki-t2t-pre-upgrade2.tar)
I've checked in my (failed) upgrade, the archive contained .txt files I've created in some of my wikis:
apps/lionwiki-t2t/backup/var/www/lionwiki-t2t/var/pages/links.txt
apps/lionwiki-t2t/backup/var/www/lionwiki-t2t/var/pages/mails.txt
apps/lionwiki-t2t/backup/var/www/lionwiki-t2t/var/pages/main.txt
apps/lionwiki-t2t/backup/var/www/lionwiki-t2t/var/pages/office.emails.txt

It is not important for me - I use the wiki just for sharing some stuff between two computers.
there is a backup file, but it's from before the last of today, but after the content was vanished.

I could reproduce this, I've changed the part in my upgrade (color problem) which prevented to continue, and now it's even worse:

I could reproduce the failing to complete the upgrade
it didn't manage to automatically restore the app, resulting in an half installation, with all wiki pages replaced by the default ones.
I've tried to restore from the backup, it complained the /var/www/lionwiki-t2t folder was still present, so I've removed it by ssh and the I could restore everything (all was still there before the upgrade

It looks like there is a problem with the backup script:
2024-02-07 14:54:11,091: DEBUG - Could not run script: /var/cache/yunohost/app_tmp_work_dirs/app_ivv80kpf/scripts/backup

https://paste.yunohost.org/raw/eqecehehir

I've tried another upgrade on the same server (not changing anything from the testing git repo), this time it didn't fail, and could come to completion, however, again it removed all user data from var/pages/ and replaced it by the default ones from the installation.
I managed to uninstall and put the backup online without problem (without data loss)

fixed with https://github.com/YunoHost-Apps/lionwiki-t2t_ynh/pull/25

YunoHost <IP_ADDRESS> (stable)
Lionwiki 2024.09.27~ynh1
After this update I did again lost all my data. I did no deeper investigation, just looked in the directory where the data is. It just contain the example files.

@Struppi I've fixed it again, it was because of "full_replace" which was added again with this commit: https://github.com/YunoHost-Apps/lionwiki-t2t_ynh/commit/6606b873f0f854542294caae034c1bcfb9256df4#diff-57aeb84da86cb7420dfedd8e49bc644fb799d5413d01927a0417bde753e8922f
you can upgrade with sudo yunohost app upgrade lionwiki-t2t -u https://github.com/YunoHost-Apps/lionwiki-t2t_ynh/tree/testing --debug meanwhile it's coming from testing into main branch.

Thanx.
I don't mind. the wiki isn't the most important app for me. Loosing the data of nextcloud or even more freshRSS, would be a pain. But as I told before the wiki is just a helper between several computer, so I won't try to restore the data.

I've put a warning in the code so I hope nobody will add this full_replace again in the future...

Oh, and btw despite this problem it works better than before. At the last version I always got an empty(white) screen after saving a page (my workaround was to recall the start page everytime I saved). This is now as it should be.

@Struppi well, maybe a previous version was set to php8, which is not totally compatible at the moment. I had also the white page problem, and the only workaround was to reload the page after saving. Now it's back to php7.4, which is not a long term solution. In the next release it will be possible to select the version of php (7.4 or 8 and above), and I'll try to find a fix for this empty page error.


[1.12.2] Fallen Knights stand in place and fail to attack while flailing wildly

The fallen knights stop chasing the player, stand in place, and start spazzing out.

I encountered a frozen knight doing what appeared to be the same thing.
It looked like he was locked onto a millienaire villager and just stood in place and tracked the villager. the knight was raising and lowering his sword over and over all the while.

It also looks like fallen mounts are bucking their mounted knights' off.

I assume this was in latest 1.3.5 version?   I redid some AI in latest version to fix some ai problems,,, might have introduced some secondary issues  https://minecraft.curseforge.com/projects/emberroot-zoo/files/2535187

I think I've figured it out. Just did another test on 1.3.5. The odd behavior wasn't showing up until I was spawning the knight during the day. Something's happening when they decide to hide under tree cover. The knights can't decide whether to attack or to avoid the sunlight.
They'll also run into and out of water repeatedly.

Thanks thats useful information. Must be something with conflicting priorities

https://minecraft.curseforge.com/projects/emberroot-zoo/files/2546169


Give packaged desktop apps access to more WinRT APIs

Proposal: Give packaged desktop apps access to more WinRT APIs
Summary
This page https://docs.microsoft.com/en-us/windows/apps/desktop/modernize/desktop-to-uwp-supported-api details the WinRT APIs available to packaged desktop apps. However there are quite a few APIs missing that should be able to work. In particular I am interested in being able to use Windows.System.Profile.SystemIdentification.GetSystemIdForPublisher. I haven't actually tested whether this works or not (I am currently forward planning). If it already works, then the docs should be updated.
Rationale
The Windows.System.Profile.SystemIdentification.GetSystemIdForPublisher API is the main way for identifying the device in a UWP app (I think) and is very useful for things like device-specific licences or notifications etc. If the publisher produces desktop or UWP variants of an app or a suite of apps, some of which are desktop and some UWP, then it is necessary to be able to call this API from both packaged desktop apps (or perhaps even desktop apps with an identity) and UWP apps.

When you don't use the appropriate proposal format, it makes it much harder to read the proposal and understand the need for what you're asking. So now we have to go back/forth on questions.
That said, it sounds like the scenario is: You have a codebase that runs on Windows, both outside and inside AppContainer and want an API to call to determine what context you're in. The pattern for that is calling the GetCurrentPackageFullName API and keying off whether or not it returns APPMODEL_ERROR_NO_PACKAGE. If writing the P/Invoke code to do so isn't your cup of tea, you can bring in DesktopBridge.Helpers from Nuget and call its IsRunningAsUwp API.
Here's a good rundown of feature detection with API contracts that's worth a read too https://blogs.windows.com/windowsdeveloper/2015/09/15/dynamically-detecting-features-with-api-contracts-10-by-10/

@riverar - I think you have completely misinterpreted the proposal. It does follow the suggested format, but omitting the optional sections because they weren't really relevant.
I thought the proposal was clear in its simplicity. Simply make as many WinRT APIs as possible available for use within packaged desktop apps. As I mentioned before, there are may APIs missing which ought to be available (using this doc as the source of what's available https://docs.microsoft.com/en-us/windows/apps/desktop/modernize/desktop-to-uwp-supported-api).
The rationale is basically to be able to use the same APIs from desktop and UWP apps (which is the whole point of Project Reunion). One use case for this is being able to create a UWP and packaged desktop version of the same app from almost the same code base. Also ensuring that the app has a future path of becoming either a UWP app or a desktop app depending on how the Windows app development story pans out.
I called out the specific API Windows.System.Profile.SystemIdentification.GetSystemIdForPublisher because that's the one I happened to be checking for availability to desktop apps, and there are some additional reasons for wanting this to be available to both types of app, as I mentioned, for example tying a license to a device. For future-proofing, I want to ensure I will be able to call this from a desktop app in the future and get the same result on the same device if the publisher is the same.

So while this is all still fresh in our heads, in the short term can we at least get @benstevens48 an answer on whether or not this API is safe for desktop use? If so, which guarantees can they rely on?

This API is safe for desktop use, with the caveat that every app without publisher identity will get the same advertising ID.


feat(tidb): tidb split release 8.1 jobs

Split tidb release 8.1 jobs.

/hold
wait new branch release-8.1 to be created.

/unhold


Feature/travis env testing

Splits up travis tests into two jobs to deal with the 50 min job limit

[x] Check that you pass the basic style checks and unit tests by running mvn clean install

Changes Unknown when pulling 9d7ff71aafadbb243b4fb8b3e3ad9e5c14b32a23 on feature/travis_env_testing into ** on develop**.


How could I disable the red block with vimrc?

How could I disable this kind of block? Thank you!

my vim broken too,after update win10 to the version of 1903

:redraw!

Please try to update both Vim and the latest version of this vimrc. I hope it fixes it; if not, please reopen this issue.


Improving the Metadata change event

Please make all changes to the event with the Metadata category.
If you now submit an mp3 file to the Player, the event does not occur. But already from the reader.buffering event, player.mediaInfo.metadata already contains all the tags of the file (artist, album, etc.).
This is very cool, despite the fact that no other actions are needed to show the cover. Mp3 file covers are immediately displayed on the texture.

added in the latest mdk-sdk


Compile error with v0.1.2

Since today I'm getting this error when trying to compile my project:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':bitmap:externalNativeBuildDebug'.
> Build command failed.
  Error while executing process /home/dlednik/Android/Sdk/cmake/3.6.4111459/bin/cmake with arguments {--build /home/dlednik/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/bitmap-0.1.2/android/.cxx/cmake/debug/armeabi-v7a --target bitmap}
  [1/2] Building CXX object CMakeFiles/bitmap.dir/home/dlednik/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/bitmap-0.1.2/ios/Classes/bitmap.cpp.o
  FAILED: /home/dlednik/Android/Sdk/ndk/22.0.7026061/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++  --target=armv7-none-linux-androideabi16 --gcc-toolchain=/home/dlednik/Android/Sdk/ndk/22.0.7026061/toolchains/llvm/prebuilt/linux-x86_64 --sysroot=/home/dlednik/Android/Sdk/ndk/22.0.7026061/toolchains/llvm/prebuilt/linux-x86_64/sysroot  -Dbitmap_EXPORTS  -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -march=armv7-a -mthumb -Wformat -Werror=format-security   -O0 -fno-limit-debug-info  -fPIC -MD -MT CMakeFiles/bitmap.dir/home/dlednik/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/bitmap-0.1.2/ios/Classes/bitmap.cpp.o -MF CMakeFiles/bitmap.dir/home/dlednik/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/bitmap-0.1.2/ios/Classes/bitmap.cpp.o.d -o CMakeFiles/bitmap.dir/home/dlednik/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/bitmap-0.1.2/ios/Classes/bitmap.cpp.o -c /home/dlednik/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/bitmap-0.1.2/ios/Classes/bitmap.cpp
  In file included from /home/dlednik/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/bitmap-0.1.2/ios/Classes/bitmap.cpp:1:
  In file included from /snap/flutter/current/usr/include/c++/8/stdlib.h:36:
  In file included from /snap/flutter/current/usr/include/c++/8/cstdlib:41:
  In file included from /snap/flutter/current/usr/include/x86_64-linux-gnu/c++/8/bits/c++config.h:508:
  In file included from /snap/flutter/current/usr/include/x86_64-linux-gnu/c++/8/bits/os_defines.h:39:
  In file included from /snap/flutter/current/usr/include/features.h:448:
  /snap/flutter/current/usr/include/x86_64-linux-gnu/gnu/stubs.h:7:11: fatal error: 'gnu/stubs-32.h' file not found
  # include <gnu/stubs-32.h>
            ^~~~~~~~~~~~~~~~
  1 error generated.
  ninja: build stopped: subcommand failed.

Anyone else seeing this?
Looks like 32bit headers are missing. I installed flutter via snap on my Ubuntu 20.04

Does this happen for debug, release of both?
Could you provide outputs for flutter doctor and the target sdk versions of your android app?


Hero 7

Thanks for the library
I had to add labels "UNIF" & "SROT" for the Hero 7
I haven't investigated what they are for, but my data requirement is very basic - just GPS.
Will look to ignore missing additional labels at some stage when I work out the best way to do it
Cheers

Hello Dunha,
we need to purchase some gopros for an experiment where we reall need to output the metadata.
We would get Hero 7 if it works or older models if it does not.
At minimum, we need the GPS timestamps to synchronize with other systems at minimum.
Additionally, the gyro and accelerometers would be useful.
What can you read in your Hero 7 videos?
Do you have access to all outputs below? (and unif and srot are extra?).
Thanks for any info.
Philippe
~400 Hz 3-axis gyro readings
~200 Hz 3-axis accelerometer readings
~18 Hz GPS position (lat/lon/alt/spd)
1 Hz GPS timestamps
1 Hz GPS accuracy (cm) and fix (2d/3d)
1 Hz temperature of camera

Hi Philippe
You should be able to extract all the available metadata. I used code from https://github.com/JuanIrache/gopro-utils which combines this repository with others to extract the binary video stream and convert to csv depending on which sensor data is required.
The multiple sensor frequencies make it difficult to have one size fits all output.
I've modified the code for my purposes but GoPro seem to add fields for each model which need to be added or skipped.. It looks like these fields have been added in he master now so it should be fine with the 7. Just make sure you get the black hero version for the GPS


distil can't handle spaces in filename

Hi, could you provide a description? In which way are you using the distillery? Any error messages?
If you are using is via the command line, you'll need to wrap filenames with spaces in quotes--but that is normal for all command line expressions.

Sorry 'bout that. Updated.


Azure App Configuration Task exposes secret values on logs

When an Azure App Configuration entry references an Azure KeyVault key AND the value of that key is a multiline string, its plain/text value (from the second line only) is revealed on the task execution log, which is a serious security breach.
Steps to Reproduce:

Create a secret entry on Azure KeyVault using Azure CLI, Azure PowerShell, or even Terraform, and set a Multiline String as it value. (Azure Portal does not support creating a secret with multiline value);
Create an entry on Azure App Configuration that references the multiline secret;
In your Azure Pipeline, add the task AzureAppConfiguration@6, referencing the entry created above;
Run the pipeline, and inspect the log - it will display the secret entry from the second line;

Again, for single line secrets from KeyVault, it works as expected and it value is not shown. The problem occurs only for multiline values.

Hello @samuel-barbosa , thanks for reporting the issue. A fix has been released in AzureAppConfiguration@7 (release notes  ). The task will now fail if any Key Vault references cannot be resolved or if the resolved values cannot be set as a pipeline secret properly. We also introduced a new option that allows errors related to the Key Vault references to be treated as warnings. It could be useful for users to fix any invalid secrets during the migration to the new version.


Profile page not updated when navigating to own profile from other user profile

This is also a bug with the Angular2 version.  When viewing another profile and then clicking the header link to your own profile, the post listing remains as the previous user and the follow button is visible(with current user's username).  The profile image and username update however.

Yup, you're totally right. Will merge any PR's that fix!

I've got a fix for this, what's the procedure for creating a PR on this project?


How to not include 'authenticity_token' in form

In my current set-up of a Rails4.2 application and SimpleForm 3.1.0, I got a problem with the authenticity tokens generated and included as a hidden field in the forms.
According to http://www.alfajango.com/blog/rails-4-whats-new/ Rails4 does no longer include the token in the form by default in order to prevent errors when using fragment-caching.
I, however, do use fragment-caching and thereby the authenticity tokens in my SimpleForms are incorrectly being cached. As a result I have these 'invalid authenticity token'-errors. Is it possible to turn off the rendering of these hidden fields in SimpleForms? As I already include the csrf token in the headers of the page...

Rails should not be including the auth token in remote forms, but it does include it by default in non-remote ones if I'm not wrong.
You can check the documentation from form_for, it explains about passing authenticity_token: false on a per-form basis, and a global configuration for remote forms. And please note that Simple Form does nothing related to the authenticity token, it's all up to Rails.
Please use the mailing list or stack overflow for questions, we reserve the issues tracker for issues. Thanks!


moving authentication docs to their own section

Signed-off-by: susanev<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?
The config menu item is getting kinda big so I was thinking it might be helpful to users to have an authentication specific section.
:athletic_shoe: How to Build and Test the Change
make serve from components/automate-chef-io
:white_check_mark: Checklist

[ ] Tests added/updated?
[x] Docs added/updated?

chatted with the team, and for now we are going to leave this in config
it may be good to consider the number of pages in that section in the future
i think there is some cognitive load to finding the authentication pages
tertiary menus would be a rad solution here


Tutorial Creation Ease of Use

I was thinking that just like in the Scratch forum post creator, we could have built-in buttons that add Markdown to the text, that way people don't need to be discouraged because they have to learn markdown, but can still use more advanced features if they would like.

Darkmode please

Seems nice, I like UI. I like ease of use.

only forumers know bbcode. it's a lot worse than markdown and less intuitive.

but isn't everyone here at least somewhat familiar with the Scratch forums?

Seems nice, I like UI. I like ease of use.

only forumers know bbcode. it's a lot worse than markdown and less intuitive.

but isn't everyone here at least somewhat familiar with the Scratch forums?

maybe we could make two modes for editing? BBCode and Markdown

Actually, I'm planning to go 100% wysiwyg

Actually, I'm planning to go 100% wysiwyg

wysiwyg like discord, or like google docs? I think having it like discord would be better...

Look at my https://TheColaber.github.io/reflect example.

https://TheColaber.github.io/reflect

hmmm... I think discord's system is better.

I don't really want to have users learn markdown to use this website

I don't really want to have users learn markdown to use this website

There's nothing to learn, though - markdown was designed to be intuitive. and with discord's wysiwyg combined with buttons for ease of use like github's, you won't even need to know markdown to use it.

i mean we will still implement MD as an option, ig


Add graphql lint and graphql lint schema

Opening this in case someone has time.
GraphQL schema linter: https://github.com/cjoudrey/graphql-schema-linter/
GraphQL query linter: https://github.com/apollographql/eslint-plugin-graphql/

I think this is sufficiently covered in #33 and can be closed?

That’s so cool, didn’t know someone implement lint.
I think the missing part might be to also include query linting somehow?

Yup! @cjoudrey would you mind opening a PR for this?

Currently we have new structure for GraphQL CLI and in the new alpha version 4.0.0-alpha.0
We have bunch of commands for fixing and validating GraphQL files in the new structure.
Checkout new instructions and we will be happy to hear your feedback. Feel free to contact us using our Discord channel;
https://discord.gg/xud7bH


Add tags variable

This PR let's us being able to set tags on Azure resources created for PostgreSQL.

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


Mouseout : Popover does not disappear  as I mouse out

Mouseout : Popover does not disappear  as I mouse out

+1 for an option like that


Align metric exposition with prometheus best practices

This PR does couple of things.

Implement common naming schema for updating metrics in DataManager (allows to implement common interface for influx and prometheus without distinguishing it at every execution)

Implicitly update metric responsible for number of all updated containers.

Implements some fixes to prometheus exposed metrics and aligns them with prometheus best practices. Mainly it namespaces metric names with application name.

Removes all_containers_updated gauge. This metric can be simply replaced by a counter as it is only going up. In such case application can reuse already present counter holding number of updated containers.

Applying this patch should result in exposing metrics as follows:
# HELP ouroboros_containers_updated_total Number of containers updated
# TYPE ouroboros_containers_updated_total counter
ouroboros_containers_updated_total{container="monitoring_grafana_1",socket="unix://var/run/docker.sock"} 1.0
ouroboros_containers_updated_total{container="",socket="unix://var/run/docker.sock"} 1.0

# TYPE ouroboros_containers_updated_created gauge
ouroboros_containers_updated_created{container="monitoring_grafana_1",socket="unix://var/run/docker.sock"} 1.558525377606453e+09
ouroboros_containers_updated_created{container="",socket="unix://var/run/docker.sock"} 1.558525377606453e+09

# HELP ouroboros_containers_monitored Number of monitored containers
# TYPE ouroboros_containers_monitored gauge
ouroboros_containers_monitored{socket="unix://var/run/docker.sock"} 6.0

@circa10a Prometheus is your baby. Take a look for me

from the lint: pyouroboros/dataexporters.py:28:1: W293 blank line contains whitespace

Sorry for the trouble going through this PR. This was a mistake.


[ALLUXIO-2743] Support specifying Mount Options for each Mount Point: cleanup (part5)

https://alluxio.atlassian.net/browse/ALLUXIO-2743

@calvinjia  this is the follow up PR to wrap Map<String, String> into UnderFileSystemConfiguration.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14859/Failed Tests: 3org.alluxio:alluxio-tests: 1alluxio.master.MasterFaultToleranceIntegrationTest.workerReRegisterorg.alluxio:alluxio-underfs-hdfs: 2alluxio.underfs.hdfs.HdfsUnderFileSystemTest.getUnderFSTypealluxio.underfs.hdfs.HdfsUnderFileSystemTest.prepareConfiguration
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14860/Failed Tests: 3org.alluxio:alluxio-tests: 1alluxio.master.MasterFaultToleranceIntegrationTest.workerReRegisterorg.alluxio:alluxio-underfs-hdfs: 2alluxio.underfs.hdfs.HdfsUnderFileSystemTest.getUnderFSTypealluxio.underfs.hdfs.HdfsUnderFileSystemTest.prepareConfiguration
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14864/Build result: FAILURE[...truncated 864 lines...][JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.5.0-SNAPSHOT.jar to org.alluxio/alluxio-core-common/1.5.0-SNAPSHOT/alluxio-core-common-1.5.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.5.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-common/1.5.0-SNAPSHOT/alluxio-core-common-1.5.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/common/target/alluxio-core-common-1.5.0-SNAPSHOT-tests.jar to org.alluxio/alluxio-core-common/1.5.0-SNAPSHOT/alluxio-core-common-1.5.0-SNAPSHOT-tests.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/assembly/pom.xml to org.alluxio/alluxio-assemblies/1.5.0-SNAPSHOT/alluxio-assemblies-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/s3/pom.xml to org.alluxio/alluxio-underfs-s3/1.5.0-SNAPSHOT/alluxio-underfs-s3-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/proxy/pom.xml to org.alluxio/alluxio-core-server-proxy/1.5.0-SNAPSHOT/alluxio-core-server-proxy-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/shell/pom.xml to org.alluxio/alluxio-shell/1.5.0-SNAPSHOT/alluxio-shell-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/oss/pom.xml to org.alluxio/alluxio-underfs-oss/1.5.0-SNAPSHOT/alluxio-underfs-oss-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/keyvalue/client/pom.xml to org.alluxio/alluxio-keyvalue-client/1.5.0-SNAPSHOT/alluxio-keyvalue-client-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/common/pom.xml to org.alluxio/alluxio-core-server-common/1.5.0-SNAPSHOT/alluxio-core-server-common-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/hdfs/pom.xml to org.alluxio/alluxio-underfs-hdfs/1.5.0-SNAPSHOT/alluxio-underfs-hdfs-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/swift/pom.xml to org.alluxio/alluxio-underfs-swift/1.5.0-SNAPSHOT/alluxio-underfs-swift-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/pom.xml to org.alluxio/alluxio-examples/1.5.0-SNAPSHOT/alluxio-examples-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/s3a/pom.xml to org.alluxio/alluxio-underfs-s3a/1.5.0-SNAPSHOT/alluxio-underfs-s3a-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/pom.xml to org.alluxio/alluxio-core-server/1.5.0-SNAPSHOT/alluxio-core-server-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/keyvalue/common/pom.xml to org.alluxio/alluxio-keyvalue-common/1.5.0-SNAPSHOT/alluxio-keyvalue-common-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/pom.xml to org.alluxio/alluxio-underfs/1.5.0-SNAPSHOT/alluxio-underfs-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/glusterfs/pom.xml to org.alluxio/alluxio-underfs-glusterfs/1.5.0-SNAPSHOT/alluxio-underfs-glusterfs-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/pom.xml to org.alluxio/alluxio-integration/1.5.0-SNAPSHOT/alluxio-integration-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/protobuf/dependency-reduced-pom.xml to org.alluxio/alluxio-core-protobuf/1.5.0-SNAPSHOT/alluxio-core-protobuf-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/protobuf/target/alluxio-core-protobuf-1.5.0-SNAPSHOT.jar to org.alluxio/alluxio-core-protobuf/1.5.0-SNAPSHOT/alluxio-core-protobuf-1.5.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/protobuf/target/alluxio-core-protobuf-1.5.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-protobuf/1.5.0-SNAPSHOT/alluxio-core-protobuf-1.5.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/keyvalue/pom.xml to org.alluxio/alluxio-keyvalue/1.5.0-SNAPSHOT/alluxio-keyvalue-1.5.0-SNAPSHOT.pomchannel stopped
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14865/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.master.MasterFaultToleranceIntegrationTest.workerReRegister
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14866/
Test PASSed.

Merged build finished. Test PASSed.

@calvinjia  could you take a look?

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14878/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14877/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14883/
Test PASSed.

@calvinjia PTAL?

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14888/
Test PASSed.

Merged build finished. Test PASSed.


Add "Bearer" as an auth option

This PR addresses issue #1919
Adds Bearer Token Authorization to Client Requests.
Added needed test.

@Nyholm FWIW, I found this PR while wondering why RequestOptions::AUTH didn't recognise bearer as built-in, as it is widely used because of OAuth 2.0 nowadays. I mean, it's not just a user-defined method in which you're expected to manually add the Authorization header.
It is a great feature for tools that saves configuration options so that I don't have to switch (...) or similar just for the sake of adding the header or using RequestOptions::AUTH.

Sorry for the long delay. Could you please rebase this PR?

Hi.. @hamzahejja thank you for the contribution..
This was my bad marking the original issue as accept PRs
For now this seems like a not needed change. As @Nyholm said developers can easily set the Authorization header