liquid-outlet not working in beta?

I have tried to get a very simple configuration working with the latest beta build of Ember but it appears to halt loading templates (aka, the outlet) but without reporting any errors. Is this a know issue? Is there any work-around?
As for general setup, I'm using:

ember-cli:  0.2.1
ember: 1.11.0-beta.5.92ef97e6
ember-data: 1.0.0-beta.16 (not that this should matter)

WRT to liquid-fire, I started out using v.0.18.0 but then switched to the latest commits from @master as I thought I saw some commits there which I thought might get me over the hump.

Ahh yes and I should add that when I am on 0.18.0 I do get an error:

Uncaught TypeError: Cannot read property 'container' of undefined

This error goes away when on @master but as mentioned above it doesn't load the outlet.

I now remember that it was this thread:

https://github.com/ef4/liquid-fire/issues/190

which pointed me in the direction of @master but unfortunately this was not a magic bullet for me.

I also have this problem. It has prevented me from upgrading to 1.18.0 and Ember 1.11 beta for now.

Hmm. I would expect liquid-fire master with ember beta to work.

Does your content render but not animate, or does it not render at all?
Is your content in the DOM, but hidden?
If you're using master, what sha?

Answer for me are:

Page renders but {{liquid-outlet}} does not render (on v0.18.0 it didn't load at all)
As for version, let me get it loaded up again but I'm fairly certain it was on 14752e71d3685c3890a1a9642cbc7c3818567b70 ... give me a sec and should be able to reproduce it on this build (I think)

Confirmed that I'm on 14752e71d3685c3890a1a9642cbc7c3818567b70

The liquid-outlet content is in the DOM, but it's hidden.

Did anyone figure anything out?

@abarax what commit are you on? As mentioned earlier, as of https://github.com/ef4/liquid-fire/commit/14752e71d3685c3890a1a9642cbc7c3818567b70 it was not working for me.

I have upgraded to e8e78a4b2cc83cace039420b48cba20e51e8bcf0 (the latest as of today) and it still doesn't work. :(

Were you using containerless=true? That's changed on master.
Please confirm that your own CSS is not breaking your layout and hiding your content.

Sorry I'm not sure what containerless=true is. As for it being my CSS, it seems unlikely considering that as soon as I switch the {{liquid-outlet}} back to just a normal {{outlet}} everything works fine (albeit without any animation). I have a very similar setup in an older project that's still on the 17.x tag with Ember v8 ... it works fine.

Here's the CSS on the liquid-container and liquid-child on the 17.x old project that works:

In comparison, here's a very similarly styled site but using Ember v11 w/ Liquid Fire v18:

Ok having gone through that it appears to be that the main difference is that the new one has no height component! That might be my doing. I'll look at that now but open to any ideas you have too.

Note: in both cases we have full-screen backgrounds which are being transitioned from page to page.

Well ok, for me my problem is solved. Sorry for pinning the fault on the product. As is often the case the problem was closer to home. :)
In my case I needed the following styling:
.liquid-container {
	position: absolute;
	top:0;
	left:0;
	width:100%;
	height:100%;
}
.liquid-child {
	height: 100%;
}

It still sounds like there's a behavior change in liquid fire that exposed this issue. Reopening so we can find and document for other people who may have difficulty upgrading.

I'm getting pretty much the same error(Uncaught TypeError: Cannot read property 'container' of undefined), but it instead on the liquid-with here.

I can confirm that master is working fine for me :)

@ef4, not sure if this is related.  I ran into the same error with ember-intl.  Originally was using 1.1.2 and switched to 1.2.2. At the location of the error, the container lookup was changed:
var intl      = this.container.lookup('intl:main');

to:
var intl      = view.container.lookup('service:intl');

Not sure if the container lookup needs to be done through the view explicitly or not.  Liquid-fire (0.18.0) is using the below while in master it looks like it's completely changed.
var View = this.container.lookupFactory("view:liquid-outlet");

Just a stab in the dark and a quick observation I found while upgrading dependencies and node.

Just as a note, changing this to env.data.view works although I'm not sure if the view is supposed to be accessed that way or not.  Currently using this in our prototype (manual changes) as moving to master breaks our liquid-if transitions.

Ember 1.12-beta.1, {{liquid-outlet}} instead of {{outlet}} in application.hbs:

Uncaught TypeError: Cannot read property 'container' of undefined

This is the same issue as https://github.com/ef4/liquid-fire/issues/190, closing in favor of that one.

@ksnyde thank you very much! I'll prob need to tweak things a bit but your style suggestions got me un-stuck!


A better solution on snippetsPreventQuickSuggestions

I finally found the only example on the Internet where snippetsPreventQuickSuggestions needs to be set to true, as follows:
this can backfire. if Intellisense suggests something and you click tab to go to the next snippet place, it will complete the suggestion. E.g. for snippet; typing i as variable; Intellisense suggestion:int; click tab; int inserted. But to avoid this, click escape before tab (only if Intellisense prompted) – Superior Nov 9 '19 at 9:58
@lisichka Thanks. – Mark Dec 23 '19 at 16:18
@SUPERIOR Why does visual studio not have this problem? Why can't vs code provide the same experience as visual studio? – Joke Huang Aug 16 at 10:59
@jokehuang Why is that question directed towards me? I don't know – Superior Aug 16 at 14:29
add a comment
https://stackoverflow.com/questions/55683145/intellisense-not-working-in-code-snippets-vs-code
However, it can be seen from the example that the problem is not Quick Suggestions, but TAB is responsible for moving the snippet place and confirming Quick Suggestions at the same time.
When it is true, the only convenience is not to manually press the ESC key, but not everyone needs to use TAB to move snippet place or confirm Quick Suggestions, so it is not necessary.
I have been using the Enter key to confirm Quick Suggestions, and I know TAB for the first time.
If you have to prevent something, you can prevent TAB from confirming Quick Suggestions in snippets, and only move the snippet place.

I will always pay attention to this issue until VSCode is improved.

I will always pay attention to this issue until VSCode is improved.

I will always pay attention to this issue until VSCode is improved.

@jrieken
People who are new to VSCode cannot know this setting from the beginning, so how to change it to false? Haven't you seen so many people asking questions on the Internet?
And, now I am not repeating the question, but giving a better solution. Have you read my text completely?

@jrieken
You just have to read one sentence: If you have to prevent something, you can prevent TAB from confirming Quick Suggestions in snippets, and only move the snippet place.


No examples of DOM manipulation using Browser.DOM

It's not intuitive how I should create a DOM element of a certain type using Fable. I'd like to see an example of this in the samples.
Updating an existing sample to demonstrate this would be nice. The basic canvas example could be modified to create the canvas before using it, as one possible solution to this issue.
Edit: it looks like the "advanced" example uses unbox. Further examples of creating things like nodelists and such would be welcome.

Yes, it's not very popular because normally Fable users prefer a more declarative style as with React. There's no particular Fable-way to manipulate the DOM you can just use the Browser APIs through the Fable.Browser.Dom bindings as you would do in JS (casting may be required sometimes). For examples, you could check Sutil which is a rendering library for Fable that manipulates the DOM, for example here: https://github.com/davedawkins/Sutil/blob/80b54124ffd86bc7e9a05d4ba88b05c52feadc3a/src/Sutil/Styling.fs#L68-L75

Gonna be honest I'm only just understanding that Elmish probably is what I want to use, but I'm also definitely not refactoring to use it quite yet.
Regardless, it feels weird that Browser.Dom.HTMLSomeElement.Create() exists but doesn't do what a beginner would expect. As someone transitioning from plain javascript, I'd like to see the correct way documented a little better.

In a few situations, bindings can differ a bit from what JS looks like. When two interfaces are used to represent (as it's normally done in the Browser bindings for "historical" reasons), Create denotes the constructor by convention (as interfaces cannot be built directly). The trick is to remember the "native" JS methods normally start with lowercase :) You can read more about calling JS code from F# with Fable here: https://fable.io/docs/communicate/js-from-fable.html


Add cmctl upgrade migrate-api-version --dry-run

Describe the bug:

As an admin, I want to know what a command will do before I run it. This especially applies to destructive commands.
The cmctl upgrade migrate-api-version command doesn't appear to have any way for me to see what it's going to do.
Expected behaviour:

Running cmctl upgrade migrate-api-version --dry-run should yield a list of things it's going to do without actually doing them.
If there's an equivalent command outside cmctl that I could use to achieve the same effect, it could even tell me about it / generate output in that format, but that isn't strictly speaking necessary.
Steps to reproduce the bug:

Anything else we need to know?:
Environment details::

Kubernetes version: client "v1.23.1", server :"v1.21.5-gke.1302"
Cloud-provider/provisioner: GKE
cert-manager version: "v1.7.0-beta.0" (from cmctl version)
Install method: Direct download of https://github.com/jetstack/cert-manager/releases/download/v1.7.0-beta.0/cmctl-darwin-arm64.tar.gz

/kind bug

Re-classifying this as it is not a bug 😄
/kind feature
/remove-kind bug
/good-first-issue
/help-wanted
/area ctl

I think it might be worth going one step further and doing a dry-run by default and adding a --no-dry-run flag.
That way someone who is playing around with cmctl (perhaps having installed the completion) rules won't accidentally make changes to the cluster in referenced by their KUBECONFIG.

I want to work on this issue.
/assign

Some notes that would hopefully help:

https://github.com/cert-manager/cert-manager/pull/4711 PR that added cmctl upgrade migrate-api-version subcommand

the subcommand and its current flags are defined in https://github.com/cert-manager/cert-manager/blob/master/cmd/ctl/pkg/upgrade/migrateapiversion/command.go

this command essentially does two things- patches statuses of CustomResourceDefinitions and re-applies all custom resources for each CustomResourceDefinition that it patched. It also does it conditionally, if there is no need to patch statuses and --skip-stored-version-check option is not passed then it will not perform any actions.
I think we'd want the --dry-run option to only print the names of CRD whose statuses would be patched and names/namespaces of custom resources that would re-applied

I think it might be worth going one step further and doing a dry-run by default and adding a --no-dry-run flag

Agree that cmctl picking up current KUBECONFIG might cause issues same as with kubectl doing that, but I am not sure if CRD names/resource names alone will be sufficient to tell what kube context is current and would also argue that it might make sense to make this option similar to kubectl which has a --dry-run option

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

If it doesn't require many lines, I'm not absolutely opposed to implementing...


Add an async option on macOS that returns a file descriptor for a pipe to the subprocess

This PR is intended as part of a solution to https://github.com/atom/atom/issues/15267. The plan is to run dd via this module as we currently do in pathwatcher, but instead of passing the contents of the file to be written synchronously, we'll obtain a file descriptor and pass it to a new method save method on superstring's buffer.
This PR also removes CoffeeScript, Grunt, and Jasmine dependencies.
A new test is added using Mocha, but the test prompts for a password, so isn't really ideal for CI.
/cc @maxbrunsfeld

Could this also be added to Windows? I need to run something async with admin privileges.

@MickdeGraaf Yes! We've decided to create a new module instead of modifying runas. See https://github.com/atom/spawn-as-admin.


Preserve access tokens when refreshing tunnel

When calling management APIs like getTunnel() or updateTunnel(), if the input Tunnel object has one or more tokens inn its accessTokens property, those tokens should be preserved in the returned Tunnel object if not refreshed by the request.

Please add unit tests.

Please add unit tests.

Done.


Add JVM shutdown hook to report the metrics from the current TimeWindow.

Currently all the TimeWindow based reporter has to wait for the next time window to report the metrics. To make it work from a batch job or from a Lambda function, client will have to sleep for at least the reporting frequency (60 seconds by default) at the end of the job.
Instead we need to add a JVM shutdown hook that would flush and report the metrics from the current time window.

I would suggest - as a solution - that the existing "com.yahoo.yamas.YamasReporter#close()" would flush the report before returning...


Allow to set acl during creation of the document

This is available on the master branch and will be in the next release


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Issue with implementing filtering on nested collection

Hi there,
I'm having an issue with trying to implement a filter on a nested collection. I have the following ViewModel set up:
`public class MailoutViewModel
{
private int MailoutTotal => Mailouts.Count();
private int ResponseTotal => Responses.Count();
    public int ParticipantId { get; set; }
    public bool HaveAnyMailouts => MailoutTotal > 0;
    public bool HaveAnyResponses => ResponseTotal > 0;
    public int NoOfDaysSinceActive
    {
        get
        {
            var active = Responses.FirstOrDefault(x => x.Type.Code == "Y2");
            if (active != null)
            {
                return (int)(DateTime.Now - active.DataReceived).TotalDays;
            }
            return 0;
        }
}

    public IEnumerable<MailComm> Mailouts { get; set; }
    public IEnumerable<Response> Responses { get; set; }
    public IEnumerable<CustomItem> CustomItems { get; set; }
    public IEnumerable<ConsentItem> ConsentItems { get; set; }

}`
I'm using this so that I can query several tables to check if a participant has a mailout due e.g. if they have a particular response but no letter has been sent then we can send them a specific letter.
I've implemented as such:
`
var searchFilterTest = new QueryBuilderFilterRule()
{
Condition = "and",
Rules = new List()
{
new QueryBuilderFilterRule()
{
Condition = "and",
Field = "HaveAnyMailouts",
Id = "HaveAnyMailouts",
Operator = "equal",
Type = "boolean",
Value = new [] { "true" }
},
new QueryBuilderFilterRule()
{
Condition = "and",
Field = "HaveAnyResponses",
Id = "HaveAnyResponses",
Operator = "equal",
Type = "boolean",
Value = new [] { "true" }
},
new QueryBuilderFilterRule()
{
Condition = "and",
Id = "Responses.Ambiguous",
Field = "Responses.Ambiguous",
Input = "NA",
Operator = "equal",
Type = "boolean",
Value = new[] {"false"}
}
}
};
        var mailouts = _participantManager.GetAll().Where(x => x.Responses.Any()).Select(x => new MailoutViewModel()
        {
            ParticipantId = x.ParticipantId,
            Mailouts = x.Mailouts,
            Responses = x.Responses,
            CustomItems = x.CustomItems,
            ConsentItems = x.Consent
        });

var result = mailouts.BuildQuery(searchFilterTest).ToList();
`
However, upon building the query, I'm getting a System.ArgumentNullException: 'Value cannot be null.
Parameter name: property'
Here is my Response class:
`public class Response : BaseEntity
{
public Response()
{
ResponseItems = new List();
}
    [Key, DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    public int ResponseId { get; set; }
    public int ParticipantId { get; set; }
    public int TypeId { get; set; }
    [Index]
    public DateTime DataReceived { get; set; }
    [Index]
    public bool Ambiguous { get; set; }

    #region Navigation properties

    public virtual Participant Participant { get; set; }
    public virtual ResponseType Type { get; set; }
    public virtual OptOut OptOut { get; set; }
    public virtual ICollection<ResponseItem> ResponseItems { get; set; }
    #endregion
}`

I've been following your example within your Unit Test but it doesn't seem to work (also it's worth noting that in your unit test, you are using FilterRule which has been marked as deprecated!).
Can someone offer any advice please?

I think I've finally solved it...
In my MailoutViewModel, I was using IEnumerable instead of List. Because of using an interface, it could not find the property within the list.
Thanks anyway :-) this is a great library and I'm looking forward to using it in my project.

In my MailoutViewModel, I was using IEnumerable instead of List. Because of using an interface, it could not find the property within the list.

Thanks @harpsicord86 , that was my issue as well! Thank you for commenting your solution and not leave the thread without an answer


Rules with :style() are wrongly converted

I guess it's related to this - https://github.com/AdguardTeam/CoreLibs/issues/880
Steps to reproduce

Enable AdGuard Base filter and add these rules:

@@||example.com^$generichide
example.com##div > h1:style(background: red!important;)
example.com#%#AG_onLoad(function() { var el=document.body; var ce=document.createElement('div'); if(el) { el.appendChild(ce); ce.setAttribute("class", "luhb-div"); } });

Go here - http://example.com/

Expected behavior

CSS rule should be applied only to h1
Actual behavior

CSS rule is also applied to .luhb-div
Screenshot:

Your environment

AdGuard for Windows 7.2.2943 NIGHTLY (CL 1.5.60)

Yes, ## are not converted to #$#, only the content

@AdamWr, by the way, you can check result of the conversion in logs, if you turn on debug log level

I didn't know about that, thank you for the information.

I might be wrong, but it seems that some rules are still incorrectly converted.
Steps to reproduce

Add these rules:

@@||example.com^$generichide
example.com##a[href*="://"]:style(color: red!important;)

Go here - http://example.com/

a[href*="://"] should be red, but it seems that it doesn't work.
Screenshot

Most likely you are not wrong:)

It's not a conversion but validation issue
fixed in core/pull-requests/1565

One more issue:
@@||example.com^$generichide
example.org##p:not(.test) > a:style(color: red)
Screenshot

One more issue:
@@||example.org^$generichide
example.org##p:not(.test) > a:style(color: red!important;)
Screenshot

resolved in core/pull-requests/1577

And maybe one more?
@@||example.org^$generichide
example.org##p::before:style(content: "Test"!important; color: red!important;)
Screenshot

Works fine with:
example.org#$#p::before { content: "Test"!important; color: red!important; }
so I guess it's a bug.
Screenshot

fixed in core/pull-requests/1595

Rule radikal.ru#?#.render_body_main:style(margin-top: 0 !important) should have been converted to radikal.ru#$?#.render_body_main { margin-top: 0 !important }, but it's not

core/pull-requests/1596

It seems that now rules with :style() don't work.
Example:
example.org##body:style(background: red!important;)
or
example.org##div > h1:style(background: red!important;)
Link - http://example.org/
AdGuard for Windows 7.4 nightly 25 (build 3209, CL 1.5.256)

@sfionov please re-test

fixed in core/pull-requests/1884


fix(interactive): Rename modern_graph.yaml to graph.yaml

To align with the file naming convention of Interactive, we are renaming modern_graph.yaml to graph.yaml.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 27.78%. Comparing base (5fb62ef) to head (845d87a).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3603   +/-   ##
=======================================
  Coverage   27.78%   27.78%
=======================================
  Files         178      178
  Lines       16233    16233
=======================================
  Hits         4511     4511
  Misses      11722    11722

Continue to review full report in Codecov by Sentry.

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


Book IV Armies added and minor fixes

Added all the armies on the Book IV and changed original armies names to mantain the same format with the new ones.

Wrong project, this should go to dba3


Added showPlaceholder option

In some cases you don't need input placeholder. This option allows to turn off it.
(There's placeholder, when input field in empty. Some users may think it's phone number inputted in the field. I know it sounds unlikely, but can happen)

This is a good idea, thanks. I think autoPlaceholder would be a more descriptive name. If you change that, and update the readme, I will gladly accept the pull request. Thanks again.

Thanks!


GitHub pages: add organization page

https://libertem.github.io/ should be some kind of landing page that links to the documentation of our packages - so currently LiberTEM itself and the blobfinder. AFAIU this can be done by having a repository libertem.github.io in our organization. How to structure this and what tools to use?
See also: https://help.github.com/en/github/working-with-github-pages/about-github-pages#types-of-github-pages-sites

Currently, there is a github.io page for Blobfinder. So, is our goal is to create a new website that has both LiberTem and the blobfinder? If so, we can use Jekyll.

Since our documentation is built with sphinx so far, I'd prefer to stick with it for the organization page as well. That way we can use the familiar structure, syntax, and things like intersphinx links.
A quick research on what to do:

[ ] Create empty libertem.github.io repo in the LiberTEM organization
[ ] Create separate document tree in LiberTEM main repo or a dedicated repository for the sources

Separate repo: Only build when page is updated, not on every CI run
Main repo: Use existing Travis and Tox set-up

[ ] Include additional build commands in tox.ini and instruct travis-sphinx to use this tree via travis-sphinx build --source=other/dir/doc and the target repo via GH_REPO_SLUG = 'LiberTEM/libertem.github.io'.

So, is our goal is to create a new website that has both LiberTem and the blobfinder?

An overview page for the project that links to both.

Since our documentation is built with sphinx so far, I'd prefer to stick with it for the organization page as well

Agreed.

Create empty libertem.github.io repo in the LiberTEM organization [...]
main repo or a dedicated repository for the sources

I rather like the variant of using branches for this: have the documentation source and CI pipeline in the master branch of the libertem.github.io repository and directly visible in github, and put the result into the gh-pages repo. That way, the source is directly "attached" to the built pages and easily findable.
So that would mean copying a bit of infrastructure (sphinx setup, maybe but not necessarily tox, travis config) to the new repo, but for this purpose, it should be quite minimal: we likely only need to change it when we add a new repo/python package to the libertem organization.

Include additional build commands in tox.ini and instruct travis-sphinx to use this tree via travis-sphinx build --source=other/dir/doc and the target repo via GH_REPO_SLUG = 'LiberTEM/libertem.github.io'

This last point is then not necessary, the config can be a simplified version of what is in the main repo.
I went ahead and created the repository - https://github.com/LiberTEM/libertem.github.io - feel free to open a PR there!
We should maybe also discuss what each item on that landing page should consist of. From my side this should be at least:

title
short description what it is about
links: github repo, documentation, pypi

https://help.github.com/en/github/working-with-github-pages/about-github-pages#publishing-sources-for-github-pages-sites
"The default publishing source for user and organization sites is the master branch. If the repository for your user or organization site has a master branch, your site will publish automatically from that branch. You cannot choose a different publishing source for user or organization sites."
Maybe we can reverse, with source in a side branch and published version in master?

Hm, the target branch gh_pages seems to be hard-coded in travis-sphinx: https://github.com/Syntaf/travis-sphinx
Since updates will be infrequent, we can perhaps just pull from gh_pages into master manually?

We can also not use travis-sphinx - if we want to move to azure, we will need to use a different flow anyways

Good point, maybe a good time to test this at this opportunity?

Indeed! Mostly this involves finding out how the authentication will work from both travis and azure, as we will need to push the results to a git branch. I guess the GH_TOKEN that is available in the CI should be enough?

Depends on #565

The page exists now with quite a bit of content. Certainly can still be improved, but we can maybe close here at this point.


expression: do not bind column for count(*) in binder

https://github.com/risinglightdb/risinglight/blob/40db687e7424da4ef96661c9cfe65adcbcf5a1a6/src/binder/expression/agg_call.rs#L65-L83
Currently, we always select the 0th column in binder for count(*). This has caused problems like https://github.com/risinglightdb/risinglight/issues/200. In the future, we should be able to do both of the following:

bind all columns in binder, and do column prune in optimizer
rewrite count(*) to using statistics information (only for queries like select count(*) from table)

I will solve this problem after @st1page finishes his work.


Images disappearing in dark theme

In article 'Icons for UWP apps' there are transparent images with black stroke. So they are invisible in dark theme.

Looks like this is still an issue as of 7/20/17

The article is being revised; the new version will fix the image issues.

The fix is live.


Issue in quick-sort python

@aashutoshrathi You have chosen first element as pivot element. This may lead to worst case scenario where time complexity is O(n2).  For average case complexity, it is advised to use median of 3 partitioning technique . For more information, read https://www.khanacademy.org/computing/computer-science/algorithms/quick-sort/a/analysis-of-quicksort
Another issue is auxiliary space required by quicksort. You have mentioned that it is O(n), but it can be reduced to O(log N), if algorithm is optimized carefully. For more information, read https://en.wikipedia.org/wiki/Quicksort#Space_complexity.
What are your views @aviaryan @singhpratyush @yashLadha @saurabhjn76

@Prakash2403 Should I randomize pivot ?

Take three different random temporary pivots, take there median as pivot.
That's median of 3 pivot technique. That will guarantee you that time
complexity will be O(nlogn) with a probability over 65%. This method is
used to avoid worst case.
On 18-Apr-2017 1:12 AM, "Aashutosh Rathi"<EMAIL_ADDRESS>wrote:

@Prakash2403 https://github.com/Prakash2403 Should I randomize pivot ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/iiitv/algos/issues/144#issuecomment-294569600, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATGchENHA7rRatncc1O4ylLEn3q1POuAks5rw8CBgaJpZM4M_dqk
.

@Prakash2403 I tried to randomize pivot, but it went wrong 😕.
Please tell where I went wrong.
http://ideone.com/aYkFHB

@aashutoshrathi There seems to be a very small mistake there, when you take median you will have to iterate through all the elements of array but you have missed the last element in each recursion i.e.
change for j in range(p, r): to for j in range(p, r+1): on line number 28.

@Monal5031 I changed r to r+1 still, result isn't sorted.

@aashutoshrathi Oh sorry, I forgot to mention this one too. array[p], array[pivot_index] = array[pivot_index], array[p] is also the cause the trouble.


[LLM] Support pre_caches input of llama

PR types

Function optimization
PR changes

Others
Description

Support pre_caches input of llama

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: wj-Mcat:white_check_mark: xiaoxiaohehe001:x: rootroot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


words like constructor and hasOwnProperty tag as undefined

Words that are inherited from Object are not tagged correctly and are instead tagged as "undefined"
const base_folder = path.join(path.dirname(require.resolve("natural")), "brill_pos_tagger");
const rulesFilename = base_folder + "/data/English/tr_from_posjs.txt";
const lexiconFilename = base_folder + "/data/English/lexicon_from_posjs.json";

const lexicon = new natural.Lexicon(lexiconFilename, "n");
const rules = new natural.RuleSet(rulesFilename);
const tagger = new natural.BrillPOSTagger(lexicon, rules);

console.log(tagger.tag(["constructor"])); //prints undefined
console.log(tagger.tag(["destructor"])); //prints NN
console.log(tagger.tag(["asdfadfasdf"]));//prints "n" which is default
console.log(tagger.tag(["hasOwnProperty"]));  //prints undefined

This is probably because on this line:
https://github.com/NaturalNode/natural/blob/29a1c21dd281fe8c624cb5ec910af576c03d62ca/lib/natural/brill_pos_tagger/lib/Lexicon.js#L56
this.lexicon = {};

should be:
this.lexicon = Object.create(null)

This way prototype methods are not pulled in and this.lexicon["constructor"] will return a word instead of the actual constructor. I haven't tested it to see the cause of the issue however

Another approach is simply putting a prefix on every word when adding or checking the lex.
// Returns a list of categories for word
const PREFIX = "CATEGORY:";

Lexicon.prototype.parseLexicon = function(data) {
  // Split into an array of non-empty lines
  var arrayOfLines = data.match(/[^\r\n]+/g);
  this.lexicon = {};
  var that = this;
  arrayOfLines.forEach(function(line) {
    // Split line by whitespace
    var elements = line.trim().split(/\s+/);
    if (elements.length > 0) {
      that.lexicon[PREFIX + elements[0]] = elements.slice(1);
    }
  });
};

Lexicon.prototype.tagWord = function(word) {
  var categories = this.lexicon[PREFIX + word];
  console.log(categories);
  if (!categories) {
    categories = this.lexicon[PREFIX + word.toLowerCase()];
  }
  if (!categories || (typeof categories == "function")) {
    categories = [this.tagWordWithDefaults(word)];
  }
  return(categories);
};

Fixed it by testing for function object.


versions/2.0: Add a 'time' type

Having human-readable times that are not associated with dates is
frequently useful.  For example, this lets you say things like:

"tea is at 3:15pm" (localtime)
"the weekly meeting is at 12:00 UTC"

Using the ISO 8061 time formats should make it easy for both
people and software to interact with the values.
Since Swagger-specified APIs are often consumed and produced from
JavaScript, and since JavaScript has limited time-handling
capabilities, I thought I'd dig a bit deeper in that direction.
Support for date-less ISO 8601 times in native JavaScript is unclear,
but Firefox 31.5 in seems to support times with a 'T' prefix:

new Date('12:34')
Invalid Date
new Date('T12:34')
Date 1970-01-01T20:34:00.000Z
new Date('T12:34Z')
Date 1970-01-01T12:34:00.000Z

which, in conjunction with Date.prototype.toTimeString() and other
time-specific maniplations is probably sufficient for most purposes.
Chromium 41.0, on the other hand, gave 'Invalid Date' warnings for
each of those cases, so JavaScript consumers may want to pad their
times with dummy date information.  Moment.js also requires a date
part and doesn't support stand-alone times.
On the other hand, there are time-only pickers in JavaScript.
Python also has a time-only format as does Java.  Go just
has a date-time type, but it can parse time-only strings.

Same comment applies as the previous PR... sorry.


chore(): update to Beta.6, including having our own typings.

This took longer because of ts2dart which was trying to find stuff in the wrong place.

@jelbourn: please review.

The build system is not in good shape, but this is the best I can do considering.

LGTM


Reverted to eceeef176448ee348e1461ac64bc9f6cfbcdf404

I needed to restore the upload scripts that were overwritten. Also, it seems quillz did break something. This commit was my last one yesterday, right before the one from quillz.
Commit: https://github.com/OpenSprites/OpenSprites/commit/eceeef176448ee348e1461ac64bc9f6cfbcdf404
If anyone has code they need back that I may have deleted, comment here and I can get it off of dev.opensprites before another update.
Also do you like my brand new "short term" label?

Snazzy new label.

Resolved


Option to eliminate striped letter

Hi. Would you consider providing an option to play the game without the striped letter hint? Hard mode unfortunately left the striped letters making it easier to solve.
thank you for the consideration

Hi there! I have already done this yesterday https://github.com/flackr/crosswordle/commit/04972b2e41d05392f92d47d4c2e73a2961b9005d 😀
If you're still seeing it today it's probably that your browser is still showing you a cached version and hasn't loaded the new code yet but you should no longer get striped clues by default unless you turn on the word-specific clues setting in the settings.


ARROW-15698: [Integration] Privatized some code in tests

Privatized some members of the concrete Testers
Added typing information to the IntegrationRunner
Simplified some code in the IntegrationRunner

There are too many changes to say "MINOR": https://github.com/apache/arrow/blob/master/CONTRIBUTING.md#minor-fixes
Could you open a JIRA issue?

@kou , thanks. Done.


Feature Request: Remove transparency

Feature Requested
Hi! If the theme is used with the Sliding Pane plugin uses a distracting (and resource hungry) transparency between panes. I couldn't find any option to stop it in the styling options.
Thanks for this awesome theme by the way :)
Relevant Screenshot
No response
Relevance

[X] The feature would be useful to more users than just me.

Style Settings

[X] I have checked that the feature is not already available via the Style Settings plugin.

Could you add a screenshot what kind of transparency you mean? The "relevant screenshot" field is there for a reason, I can't tell for your description alone what you mean ;)

Oh yes I am sorry,

ah, I see!
Thanks for the info! It's actually a side effect of a feature that I implemented a couple of weeks ago. Since I don't use Sliding Panes, I didn't know that this has this sort of side effect... :P
Fixed with the latest release!

btw, I finally found some time to make the rotated headers work properly with my theme. If you update, the rotated headers correctly now :)


Update FW failed

I tried to update FW from 2.6 to 2.8/3.1.
And I got:
[2022-04-13 21:12:42.458] [debug] [FwUpdater] cannot find DNET : blk 0 sz=524288
[2022-04-13 21:12:42.458] [debug] [FwUpdater] Stopping reader thread..
[2022-04-13 21:12:42.566] [debug] [FwUpdater] Reader thread stopped
[2022-04-13 21:12:42.567] [error] [FwUpdater] Did not receive 'dl ack'
Firmware update failed

Thank you for your help

Can you run rsid-viewer before upgrading the FW?
which upgrade tool do you use? I recommend rsid-fw-update v0.25.0

The problem "update FW from 2.6 to 2.8/3.1." is Firmware update failed.
Why can not update directly from 2.6 to 4.3 ?
I want to use FW 4.3


upstream merge changes from dash v0.16.x

In compliance to PR #66 I am closing that Pull Request cus is not valid anymore


[feature request] shuffle

Consider the following trait method:
pub trait Itertools: Iterator {
    fn shuffle(self, buffer_size: usize) -> Shuffle<Self> {}
}

The Shuffle iterator would create a buffer (Vec::with_capacity(buffer_size)) and have two states:

empty buffer: on the next next() call, the buffer is filled with items from the underlying iterator and the buffer is shuffled. Fallthrough to non-empty buffer case.
non-empty buffer: items are popped from the buffer.

I'd happily implement this if people like the idea.

In order to work on embedded systems, the shuffler would be a parameter:
fn shuffle<S>(self, buffer_size: usize, shuffler: F) -> Shuffle<Self>
where F: FnMut(&mut Vec<Self::Item>)
{ ... }


TAJO-1332 Implement Python Client for Tajo

This is implement of python tajo rest client.

This is a PR just for discuss more.
we should discuss about testing and distribution and directory of this kind of clients.
I just put the python client into contrib/tajo-python
but it should be discussed for right position.

Great work! I think that contrib seems to be a right position. I'll review it soon.

Nice work! On the proper location, I agree with you guys.

I'm sorry for late review. The protocol between client and TajoMaster has been rapidly changed for better error propagation (https://issues.apache.org/jira/browse/TAJO-1625). This changes will actually affect REST API specification. I thought that this patch is conflict to the on-going error propagation work.
The solution is that we have to improve REST API to reflect the error propagation before we can continue this Python client. I think that REST API improvements would be trivial. We can put this python client into the next release.

Hi @charsyam
Thank you for your contribution.
Python has an API to make it easy to move from one SQL-based DB to another called DB-API. Each DB may have multiple different modules available for it. And if python users use it, they can easy integrate with ORM. It's documented in https://www.python.org/dev/peps/pep-0249.

Hi, @charsyam
May I take this on to continue?

@eminency Thanks for your suggestion.
but, If you excuse. I will finish it ASAP
Thanks.


[Bug] No available options

Note! Try updating/reinstalling the extensions. Maybe this will help.
I did. I even deleted all the content of user_settings.

I restarted the browser to no avail.
Describe the bug

Browser: Firefox 114.0.2 (64 bits)
Tampermonkey 4.18.1
Nova 0.41.0

When I click the nova button to go to options, the plugins tab is empty. It used to work. I have no idea what happened other than a browser update.

Same for the map

The other tab is ok.

Here is the full console log when opening https://raingart.github.io/options.html?tabs=tab-plugins

Solved by installing the 0.41.9 version available here https://openuserjs.org/scripts/raingart/Nova_YouTube

Solved by installing the 0.41.9 version available here https://openuserjs.org/scripts/raingart/Nova_YouTube

Can confirm it works after the update, thanks
Gosh, OpenUserJS' rate limit whatever is so god awful, Violentmonkey couldn't ever auto update, ever


fontStyle & textAlign tokens

Request:
fontStyle & textAlign tokens
These font props are visible in Figma's inspect panel
Key Information:
These tokens are not specified as part of the W3C Design Token Format Module

We have an issue for fontStyle. Can you rename this to be about text align?
Also, can you describe what you'd like to see in it?

+1
being able to specify alignment horizontally (text: align-left, align-center, align-right, and align-justified) and vertically (align-top, align-middle, and align-bottom) would be nice to have.

probable duplicate of #1138
+1
being able to specify alignment horizontally (text: align-left, align-center, align-right, and align-justified) and vertically (align-top, align-middle, and align-bottom) would be nice to have.

Thanks, agree lets close this in favor of #1138


Add anti-DPI blocklist mode

The anti-DPI blocklist mode makes AdGuard to modify only HTTP packets that is related to a website existing in the blocklist.
The requested feature will help to improve user experience in some region [^1] that internet censorship apply.
[^1]: Example: South Korea

@piquark6046 As a workaround you may try the rule like
@@*$stealth=dpi,domain=~www.example.org,~www.example.com

The cons is that rule will be in Filtering log for every request.

We leave this issue open but it is not on any milestone yet. The same is about another similar issue: https://github.com/AdguardTeam/CoreLibs/issues/1702
They may be solved together in future.
Voting on issue may speed up this process

https://github.com/AdguardTeam/CoreLibs/issues/1795#issuecomment-1673166827
I think that the option has a bug.
AdGuard for Windows
Screencast from 2023-10-10 03-04-14.webm
Screencast from 2023-10-10 03-05-51.webm

AdGuard for Android
Please see AdGuard CoreLibs Slack channel.

Maybe add other features from https://github.com/dovecoteescapee/ByeDPIAndroid also?


Different query result between 2.2.37 and 3.0.30

We are migrating one of our applications from OrientDB 2.2.37 to 3.0.30.
The following query returns different results depending on the OrientDB version.
SELECT  FROM Detail WHERE (name = 'foo' AND master.labels LIKE '%bar%')
labels is an EMBEDDEDSET.
With 2.2.37 it does return the expected results.
With 3.0.30 it returns 0 documents, regardless we use the old or the new query api.

com.orientechnologies.orient.core.db.ODatabase.query(OQuery<?>, Object...)
com.orientechnologies.orient.core.db.ODatabase.query(String, Object...)

Hi @diegomtassis
What type of objects are master and labels?
Thanks
Luigi

Hi @diegomtassis
That's one of the strange behaviours of version <= 2.x that we could not re-implement in the new executor (just because they don't make much sense in general).
LIKE operator is defined on strings, not on EMBEDDEDLIST.
you can try one of the following:
SELECT FROM Detail WHERE (name = 'foo' AND master CONTAINS (labels LIKE '%bar%'))

Thanks
Luigi

Thanks @luigidellaquila
master is another document connected via a OType.LINK
labels is an EMBEDDEDSET of String
Everything we do is via the java API.
The query you suggested doesn't return any document either.
I'll try to isolate the behavior in an integration test without our application code.

@luigidellaquila  I've done some rearch on the topic.
In 2.2.37, LIKE works for EMBEDDEDSET but only when the set contains one element. See the following test. First scenario passes, second fails.
In 3.0.30 none of the scenarios pass, which seems correct to me. As you said, LIKE doesn't seem a correct operator for a collection. The query you suggested doesn't provide the expected result either.
See the scenarios here.

@luigidellaquila @tglman any progress on this topic?


implemented the feedback from starter scenario testing team

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

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

Done, signed now.

Done, signed now.


Resource property validation should be aware of action-specific reqs

Signed-off-by: Steve Abatangle<EMAIL_ADDRESS>
Description

The get and set methods in the Property class, meant to be used by resources, validate that a property is required if its value is nil. However, they don't check to see whether the property was required for the specific action the resource is using, so any use of get and set will throw an exception if used with a resource that doesn't use a required property, even when the property is not required for that action.
That's a vague explanation, so I'll get specific: the current version of the chef-client cookbook has a recipe (cron.rb) with a resource that does a delete. When you try to get that resource, it throws an exception, explaining that :command is a required property—but it's only required for the :create action, not for :delete. https://github.com/chef-cookbooks/chef-client/blob/master/recipes/cron.rb#L94-L96
This PR simply fixes that by passing the resource's action to the required? method, and allows the required? method to compare arrays of actions rather than just expecting a single action.
Related Issue

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[X] I have read the CONTRIBUTING document.
[X] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.
[X] All commits have been signed-off for the Developer Certificate of Origin.

Okay with a skim of the code this looks correct, but I'm going to need to play around with some failure cases and maybe try to write an integration test for this one.

Okay with a skim of the code this looks correct, but I'm going to need to play around with some failure cases and maybe try to write an integration test for this one.

I did add some unit tests but I honestly don't know enough about the Chef testing setup to have attempted doing the integration test myself.

I'm just writing a note to myself here to get it to the top of the list so that I look at it again because this got buried and fell off my radar.

Naw, this is already fixed. Wasn't exactly my code but it's definitely
fixed now.
On Mon, Aug 23, 2021 at 10:34 AM Lamont Granquist @.***>
wrote:

I'm just writing a note to myself here to get it to the top of the list so
that I look at it again because this got buried and fell off my radar.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chef/chef/pull/11451#issuecomment-903973527, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQYHEUPT4CJV3FQW6V5ZLT6KBDNANCNFSM43QM7EUA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

@lamont-granquist Can you take a look at this one again?


fix style issues

Codecov Report

Merging #514 into master will not change coverage.
The diff coverage is 86.76%.

@@           Coverage Diff           @@
##           master     #514   +/-   ##
=======================================
  Coverage   92.35%   92.35%
=======================================
  Files          37       37
  Lines        2877     2877
=======================================
  Hits         2657     2657
  Misses        220      220

Impacted Files
Coverage Δ

vent/api/actions.py
92.8% <86.76%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Associate ssh key pair to dataplane node to help debug

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

asgs: 1
An error occurred (InvalidKeyPair.NotFound) when calling the DescribeKeyPairs operation: The key pair 'scaletest-us-west-2-nodegroups-ssh-key' does not exist
KeyPair already exists
sh: out of range
An error occurred (InvalidParameterException) when calling the CreateNodegroup operation: Remote access configuration cannot be specified with a launch template.

It seems we can't specify launch template and remote access at the same time.
https://tekton.scalability.eks.aws.dev/#/namespaces/scalability/pipelineruns/awscli-eks-load-5k-v131-slos-t6mln-r-s6g99?pipelineTask=create-mng-monitoring-nodes&step=create-nodegroup


API changes and support for GLFW 3.1 features.

This PR contains changes from the devel branch. That includes minor API changes that make the API easier to use and more idiomatic, as well as support for new features of GLFW 3.1. See changelog in README for a complete list of changes.
It is planned to be merged into master when GLFW 3.1 is released.
Closes #40.
Closes #8.
Closes #56.
Closes #106.

92%! @elmindreda has been doing and awesome job and making progress.

:smile:

Hey guys,
Great news! GLFW 3.1 (the C library) has been officially released today!
https://twitter.com/elmindreda/status/556914293984075776
I will make a PR to update this devel_glfw3.1 branch to GLFW 3.1, and then we'll want to review this huge PR before merging it into master.

:+1:

We are now in final review stage. This is the time to bring up any points about the breaking API changes that we are introducing with the 3.1 update, and make sure all potential issues and points of concern are resolved before we pull the trigger.
Please do not merge this PR until it has an explicit +1 from everyone in go-gl organization; and let's give it at least a few days after that to make sure no more issues are raised.
Remember, this will be a breaking API change for people using the master version. We should be very careful about that, and only do it once.

I wonder if we can write some gofmt/gofix style rules to make the lives of other people porting easier. We might also consider creating a glfw3-201501-legacy package or something like that so that there is a drop-in replacement for the old dependency.

I'm beginning work on #118: until I have a PR in for that please don't work on anything except the README to avoid collision as that change will likely touch every .go file in various places.

I've finally updated all my code to the latest commit on this branch, including API changes. It was definitely a win and made my user code simpler/shorter by removing panic(err) and , _ things.
I'll keep testing it over the next few days and report any issues I notice.

Sorry to bother you but is this ready to be merged yet or is there still some work to do?
(Just asking because I am using a glfw version with a manually patched v3.1 feature in a personal project for half a year and I would love to switch to the official package)

@mbertschler should be any day now. one issue left to resolve #123

@tapir Thank you!
I am really looking forward to switch to the official 3.1 which makes it possible to open source my package, because I don't want people to have to use a patched version of 3.0 just for this tool.
My package tries to provide a cross platform GUI that should make GUI programs as easy to write as a web app. (The programming model is based on React.js.)


New body-parser helper resets the body, breaks middleware that expect streams

Related to #256.
I'm using rocky to proxy some of the routes of my express.js application, and using the express-stormpath library breaks the request body because the body-parser helper is "forcing" a default body of an empty object. The proxy expects the default body of an express request(a string or a buffer), so resetting the body to an empty object is breaking the request flow.
To fix this issue I commented out the following in stormpath.js in the sdk itself:
router.use(bodyParser.forceDefaultBody())

Not sure if this breaks some other functionality(doesn't seem to) but I would say modifying the req.body on every request is not such a great idea anyway, so maybe there is a better way to handle this within the library?

Thank you for reporting this issue. I just labeled this as a bug and I'm going to be looking into this shortly.

I agree, our library should only modify req.body when we are parsing POST bodies for routes that our body binds to, such as /register, etc.


Add Crystal::System::FileHandle

Crystal::System::FileHandle represents a handle to a file, or file-like
object (pipe, tty, etc.).
The specs currently fail due to debug line-numbers being broken for some weird reason. I just wanted to PR this so people could criticize the technique.

One thing I'm concerned about is that we've separated Socket and FileDescriptor but we still have pipes and consoles and similar things. I think it's pretty unreasonable to have a seperate hierarchy of code under IO for each type but I want to remove FileDescriptor too and leave just Socket < IO, File < IO, etc.

It's unclear for me about Socket < IO, now it's just
class Socket
    include IO::Buffered
    include IO::Syscall
....
end

So, what is the purpose of making Socket < IO?

@txe By that comment I simply meant that Socket would be an IO directly, instead of inheriting from IO::FileDescriptor. That work has already been done, but I thought about it a bit more and removing IO::FileDescriptor and being able to do the same to File would be great. The unfortunate part is that there are a lot of IO objects in unix systems which are neither files or sockets; they're pipes or ttys or character devices or block devices. Separating all these types into different classes is unreasonable. So I'm questioning the design decisions behind this PR and #4707 a little bit.

@RX14 It would be great if you could finish this work. It will really help me with adding functionality for windows.

Maybe IO::FileDescriptor can be kept for unix, and for Windows we can have a different hierarchy. The only thing IO::FileDescriptor exposes is fd, which shouldn't be normally used.

I don't think that changing the class hierarchy based on platform will turn out well...

It could be an included module, just an implementation detail.

@asterite Except my concern is that we have to have file descriptors which are neither sockets or files. Modules can't be instantiated by their own so that doesn't solve much.

I mean, File, Socket and pipes could include IO::FileDescriptor. Or Socket in Windows doesn't inherit IO::FileDescriptor. In any case I don't think a different hierarchy is a problem, with an IO you want to read and write.

Well most of the the role of IO::FileDescriptor was surplanted in this PR, which is why I wanted to remove IO::FileDescriptor. Are you suggesting having a seperate class for pipes? What about device nodes and character devices and all the other weird and wonderful uses of file descriptors? I'm not entirely sure I understand your proposed design.

unix: IO::FileDescriptor exists for files, sockets, pipes, etc.
windows: IO::FileDescriptor exists for files, pipes, etc. Sockets have a different implementation.

That's what I'm saying.

@asterite The problem with that proposal is that it means that Socket will have to be split into 2 platform specific files. So much code will change in the Socket class that it would be impossible to follow if the platform-specifics were done using macros. I don't think changing inheritance based on platform is even practical without splitting the files entirely.

IMHO IO::FileDescriptor is left spineless. It delegates everything to Crystal::System::FileHandle and does nothing. Why bother keeping it?

I'm not sure we should be exposing Crystal::System to the user, and there are some file descriptors which don't fit into any of the subclasses. I can't see a way to remove it.

I think we should merge #4901 before this PR though.

Still from wikipedia:

In computer programming, a handle is an abstract reference to a resource.

@RX14 I think it's important to emphasis the fact that it is a resource handler, and not simply a generic handler, or a more precise one like file handler.
I also agree on this definition of a resource handler, and how a FD, socket, PID, etc.. are also resource handlers. But in our case, I think it's a concept too generic to use in a programming language, and we should have classes for the abstraction level above, which we can actually implement using syscalls and stuff.

Superseeded by #5333


[License] Keep the Apache License for not changed files

Many files come from Apache Doris and have not been changed.
According to the License rule, the files should keep the Apache License.
What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[ ] refactor
[x] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)


New methods required on Frame, e.g. setKey and setFamily

For my HBase work I set the key and the column family in the schema fields in the Frame - for now I have some implicits to do this:
    JdbcSource(() => dataSource.getConnection, query)
      .withFetchSize(10)
      .toFrame
      .withLowerCaseSchema()
      .setKey("name")
      .setFamily(cf = "cf1", fields = "age", "salary", "creation_time")
      .to(HbaseSink("hannes", "test7"))

Could you update the Frame with methods set the key and family using a name of your choice.

Note I also have another implicit to build the Frame's schema from a Hive External Table mapped to a HBase table - this is remaining as an implicit as I don't think this method has any business belonging to the Frame.
However if you have any suggestions on how I  could interface this seamlessly would be much appreciated.

What's the concept of a family on a frame? Does that fit into how a frame is modelled, which is a table?

Well we have concepts for manipulating the frame's schema at the field level...

addField, addFieldIfNotExists
replaceFieldType
removeField
updateField (unsupported atm)
renameField

So if I have a frame coming from JDBC I need to ensure that the key and column families are set before passing it to the HbaseSink.

What I'm saying is a Frame is model of a table. How does a key and column sit in with that concept? If HBase doesn't model things as a table, then is it the right fit for the frame abstraction?

Key = Primary Key and I would argue it is a model of table.
Column Family is a models a HBase and Cassandra tables - you could argue this is additional metadata which I use atm.

I could compromise and provide a mapping for key and cf on the HBaseSink - I don't need to do that on the HbaseSource.
What do you think?

Having the key on the Field is not really any use unless you say HBaseSource -> KuduSink - both have concepts of a key.
For ColumnFamily I can just provide mappers in my HbaseSource and HbaseSink and you don't have to worry about adding that to field.

Closing this, as we will be adding the column mapper to hbase sink, covered in another issue.


Is there a way for me to run it without installing the Jmeter

Like, is it possible for me just to point to the bin folder of Jmeter and get it rolling rather than setting it up in the environment variable? That would be easier for everybody who works in a corporate setting where asking for installation permission takes days.
something like what sparklyR does...
Please do let me know.

# devtools::install_github("tmobile/loadtest")
# install java and jmeter - hourray /s
Sys.setenv("LOADTEST_JMETER_PATH"="C:/Program Files/apache-jmeter/bin/jmeter")

library(loadtest)

Thanks for pointing it out. But even after using it my code doesn't work.
library(loadtest)
Sys.setenv("LOADTEST_JMETER_PATH"="C:\\Users\\vikram.rawat\\Downloads\\apache-jmeter-5.3\\bin")
loadtest:::check_jmeter_installed()
results <- loadtest(url = url,
                    method = "GET",
                    # headers = c("version"="v1.0"),
                    # body = list(sentences = list("I love this band")),
                    # encode="json",
                    threads = 8,
                    loops = 32,
                    delay_per_request=100)

it gives me an error.
loadtest - beginning load test
loadtest - completed load test
Error in file(file, "rt") : cannot open the connection
In addition: Warning messages:
1: In doTryCatch(return(expr), name, parentenv, handler) :
  Unable to find JMeter installation. https://github.com/tmobile/loadtest#installation
2: In file(file, "rt") :
  cannot open file 'C:\Users\VIKRAM~1.RAW\AppData\Local\Temp\RtmpKqhWBa\file5c903cec5f88.csv': No such file or directory

However the httr::GET(url = url)  request works with result. I can assure that URL is working fine.
Response [http://localhost:9090/api/dimensions/country]
  Date: 2020-10-14 21:01
  Status: 200
  Content-Type: text/plain
  Size: 158 B

This worked for me... Thanks for replying.. and I sorry to not go through the entire document before asking straight away... We can close this I guess

@vikram-rawat I guess you can close it.

Thanks for helping me

Just installed. You should clarify in the readme that it's not just the path to bin but bin/jmeter. That got it working for me, even though jmeter --version worked.


Move update channel setting to Cody & Sourcegraph

https://github.com/sourcegraph/jetbrains/pull/9 has been prematurely merged.
This PR moves the update channel setting from Cody to Cody & Sourcegraph.

Test plan
Manual

runIde
go to Settings / Plugins / gaer icon / Manage plugin repositories
verify there are no repositories
go to Settings / Tools / Sourcegraph & Cody
Switch Update channel to Nightly
go to Settings / Plugins / gaer icon / Manage plugin repositories
verify there is alpha repository added
go to Settings / Tools / Sourcegraph & Cody
Switch Update channel back to Stable
go to Settings / Plugins / gaer icon / Manage plugin repositories
verify there are no repositories

Looks good to me :+1: (except for one var visibility - you must have overlooked it while moving the code).

Fixes https://github.com/sourcegraph/sourcegraph/issues/57369


Benchmarks reformat

Refactored benchmarks for query & insert to be more consistent and use a more compact format.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@695b52f). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #91   +/-   ##
=======================================
  Coverage        ?   98.49%
=======================================
  Files           ?       24
  Lines           ?     2065
  Branches        ?        0
=======================================
  Hits            ?     2034
  Misses          ?       31
  Partials        ?        0

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


Cargo install is broken

Thanks to a breaking change in protobuf-pure, our current master will not build with cargo install.
A workaround is to clone, and then compile it with --locked but as per https://github.com/rust-lang/cargo/issues/7169 I don't see a way to pass the flag to cargo install.
cc #473 #460
@awiouy @sashahilton00 Could you make a patch release with 6ee2dbaad4406628f77cd59adebb2ed3ed70fcaa in the meantime?

What about vendoring the required crates and using only the vendored crates at build time? This way, the source code would be complete and consistent.
https://doc.rust-lang.org/cargo/commands/cargo-vendor.html

Doesn't vendoring requiring you to clone the repo first?
You might as well just build it with --locked if you already cloned it right?

What about vendoring the required crates and using only the vendored crates at build time? This way, the source code would be complete and consistent.
https://doc.rust-lang.org/cargo/commands/cargo-vendor.html

could you not do that commit to master then rebase dev onto master ? A none compiling master seem the greater evil.

If you're trying to compile it on a Raspberry Pi (as I did, expect 40m+ of compiling time) I'd suggest you to try this command: cargo install --features alsa-backend -- librespot.


groq-builder: implement core methods

Implement core methods, including:

[x] star
[x] filter
[x] filterByType
[x] projection
[x] deref
[x] order
[x] slice
[x] slug

Complete!


LocalGPUTest fails and leaves GPU in an invalid state

Setup
I followed the Condential Computing Deployment Guide - (Intel TDX & KVM).
nvidia-smi
Driver Version: 560.35.05  (nvidia-driver-560-open)
cuda-toolkit
Version 12.6
Problem
The script fails and leaves my GPU in an invalid state.
cudaGetDeviceCount returned 802
-> system not yet initialized

This is the error I get when I run the script.
[LocalGPUTest] node name : thisNode1
[['LOCAL_GPU_CLAIMS', <Devices.GPU: 2>, <Environment.LOCAL: 2>, '', '', '']]
[LocalGPUTest] call get_evidence()
Nonce generated: 931d8dd0add203ac3d8b4fbde75e115278eefcdceac5b87671a748f32364dfcb
Number of GPUs available : 1
Fetching GPU 0 information from GPU driver.
Exception in thread Thread-14 (function_caller):
Traceback (most recent call last):
  File "/usr/lib/python3.12/threading.py", line 1073, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.12/threading.py", line 1010, in run
    self._target(*self._args, **self._kwargs)
  File "/shared/nvAttest/lib/python3.12/site-packages/verifier/utils/__init__.py", line 206, in function_caller
    result = function(*arguments)
             ^^^^^^^^^^^^^^^^^^^^
  File "/shared/nvAttest/lib/python3.12/site-packages/pynvml.py", line 5097, in nvmlDeviceGetConfComputeGpuAttestationReport
    _nvmlCheckReturn(ret)
  File "/shared/nvAttest/lib/python3.12/site-packages/pynvml.py", line 919, in _nvmlCheckReturn
    raise NVMLError(ret)
pynvml.NVMLError_Unknown: Unknown Error
        The call to fetch attestation report timed out.
[LocalGPUTest] call attest() - expecting True
Evidence is empty.. skipping attestation..
[LocalGPUTest] call attest() - result :  False
[LocalGPUTest] token :
[LocalGPUTest] call validate_token() - expecting True
[LocalGPUTest] call validate_token() - result:  False
Token is empty

Running snvidia-smi conf-compute -srs 1 after executing the script, returns Failed to set Conf. Compute GPUs Ready State: Invalid Argument.

For attestation at least, 560 is not supported for confidential compute, only 550 right now. In addition, nvidia-persistenced needs to be run before any SMI commands are executed.

For attestation at least, 560 is not supported for confidential compute, only 550 right now. In addition, nvidia-persistenced needs to be run before any SMI commands are executed.

Thank you. I installed 550 and now the script works. I believe this driver is not completely compatible with cuda-toolkit 12.5 which is the version that the guide uses. Will confidential compute still work if I switch to 12.4?

Will confidential compute still work if I switch to 12.4?

As a feature, yes, but newer CUDA toolkits and can run on older drivers. See https://docs.nvidia.com/deploy/cuda-compatibility/#id3.


Fine-tuning for classification

Hello,
do you plan on adding the code to fine-tune your model in a classification task (like RVL-CDIP)?
PS : Nice work, splitting the layout and the language models is a really good idea!

Yes it seems like an option.
However, in the run_funsd.py script would replacing the AutoModelForTokenClassification with AutoModelForSequenceClassification and adapting the dataset to the classification task work?

No, it wouldn't work, since there is no LiLTRobertaLikeForSequenceClassification implemented in https://github.com/jpWang/LiLT/blob/main/LiLTfinetune/models/LiLTRobertaLike/modeling_LiLTRobertaLike.py for now. You can implement it as you wish.
Or, if you want to implement classification task based on run_funsd.py, you can focus on https://github.com/jpWang/LiLT/blob/main/examples/run_funsd.py#L180-L208, and give the classification category id for [CLS] token to label_ids, and give -100 for other tokens to label_ids.


remove multithreading from CollectHiSeqXPfFailMetrics

Multithreading is not worth the errors and must be removed from hellbender code. GATK4 is a single-thread toolkit.

depends on https://github.com/broadinstitute/hellbender/issues/403. Tests must be in place before this ticket is attempted

deleted code #470


Update user story and use case

Tackles part of https://github.com/AY2122S1-CS2103T-W16-2/tp/issues/206

Codecov Report

Merging #225 (774397a) into master (cb6cbb4) will decrease coverage by 1.22%.
The diff coverage is 1.28%.

@@             Coverage Diff              @@
##             master     #225      +/-   ##
============================================
- Coverage     78.74%   77.52%   -1.23%
  Complexity     1149     1149
============================================
  Files           139      139
  Lines          3289     3341      +52
  Branches        377      373       -4
============================================
  Hits           2590     2590
- Misses          604      656      +52
  Partials         95       95

Impacted Files
Coverage Δ

...22s1_cs2103t_w16_2/btbb/commons/core/Messages.java
0.00% <ø> (ø)

..._cs2103t_w16_2/btbb/model/util/SampleDataUtil.java
0.00% <0.00%> (ø)

...ava/ay2122s1_cs2103t_w16_2/btbb/ui/MainWindow.java
0.00% <0.00%> (ø)

..._w16_2/btbb/logic/descriptors/OrderDescriptor.java
85.71% <100.00%> (ø)

Continue to review full report at Codecov.

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


"HTTP ERROR 403" on download migration archive GHE server

"HTTP ERROR 403" on download migration archive GHE server.  But I can download it manual with curl.
Could be a race condition https://sourcegraph.com/github.com/github/gh-gei@8e1ef90ccf2a42b0e24351df0426ac4c6c064483/-/blob/src/Octoshift/AzureApi.cs?L31-39?

from<PHONE_NUMBER>0429.octoshift.verbose.log

  35   │ [2:04 PM] [INFO] Downloading archive from https://test.[ghes].com/storage/migrations/29/archive/1032932e-d797-401d-afc3-e23bdf0c1ca3?token=AAAABEYIENZMMTOZW7PEI5TDICRHS
  36   │ [2:04 PM] [DEBUG] HTTP GET: https://test.[ghes].com/storage/migrations/29/archive/1032932e-d797-401d-afc3-e23bdf0c1ca3?token=AAAABEYIENZMMTOZW7PEI5TDICRHS
  37   │ [2:04 PM] [DEBUG] RESPONSE (Forbidden): <truncated>
  38   │ [2:04 PM] [ERROR] [HTTP ERROR 403] System.Net.Http.HttpRequestException: Response status code does not indicate success: 403 (Forbidden).
  39   │    at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
  40   │    at OctoshiftCLI.AzureApi.DownloadArchive(String fromUrl)
  41   │    at OctoshiftCLI.GithubEnterpriseImporter.Handlers.MigrateRepoCommandHandler.GenerateAndUploadArchive(String ghesApiUrl, String githubSourceOrg, String sourceRepo, String azureStorageConnectionString, String awsBucketName, String awsAccessKey, String awsSecretKey, String github
       │ SourcePat, Boolean skipReleases, Boolean lockSourceRepo, Boolean noSslVerify)

manually download

❯ http https://test.[ghes].com/storage/migrations/29/archive/1032932e-d797-401d-afc3-e23bdf0c1ca3\?token\=AAAABEYIENZMMTOZW7PEI5TDICRHS
HTTP/1.1 200 OK
Accept-Ranges: bytes
Connection: keep-alive
Content-Length: 13134
Content-Security-Policy: default-src 'none'
Content-Type: application/x-gzip
Date: Fri, 07 Oct 2022 21:15:29 GMT
ETag: "cc39245da0d1af6e71475bf8240b04b95bafa9e91f63512b7e7082cbc1a7982c"
Last-Modified: Fri, 07 Oct 2022 21:04:41 GMT
Server: GitHub.com
Strict-Transport-Security: max-age=31557600
X-Content-Type-Options: nosniff
X-Frame-Options: deny
X-GitHub-Request-Id: 2b5d68e1-4685-11ed-80b9-58651b5897f1
X-XSS-Protection: 1; mode=block
timing-allow-origin: https://test.[ghes].com

+-----------------------------------------+
| NOTE: binary data not shown in terminal |
+-----------------------------------------+

local version

❯ gh version
gh version 2.17.0 (2022-10-04)
https://github.com/cli/cli/releases/tag/v2.17.0
❯ gh ext list
gh gei  github/gh-gei  v0.27

Reproduction Steps on MacOS
1. gh gei generate-script --github-source-org ${SOURCE_ORG} --github-target-org ${DESTINATION_ORG} --download-migration-logs --output ${FILENAME}.ps1 --no-ssl-verify --ghes-api-url ${GHES_API_URL} --azure-storage-connection-string "${AZURE_STORAGE_CONNECTION_STRING}"
1. chmod +x ${FILENAME}.ps1
2. ./${FILENAME}.ps1

AWS WAF rejected the request due to NoUserAgent_HEADER.  We are able to download the migration archive after AWS WAF rule adjusted.

@rzheng-sxm thanks for reporting this, I'm wondering if there's anything we can do on the CLI side to improve things here. I had thought we were always setting a UserAgent header in all HTTP calls, however I see that for calls to API's we always explicitly set a custom UserAgent, but for the call to download this file we're just leaving the default (which I didn't think was blank, but perhaps it is).
I'm going to leave this open so we can revisit it when we have some time to see if we can improve things here.

Thanks Dylan.
From: Dylan Smith @.>
Date: Wednesday, October 19, 2022 at 2:44 PM
To: github/gh-gei @.>
Cc: Zheng, Rong @.>, Mention @.>
Subject: [EXTERNAL] Re: [github/gh-gei] "HTTP ERROR 403" on download migration archive GHE server (Issue #676)
@rzheng-sxmhttps://urldefense.com/v3/__https:/github.com/rzheng-sxm__;!!NUnUjx3wvH5xgA!aMQRACutiAcvFNrvHGPyBf5xunAhBGbasK73V-ROhtn__3uprgS75eUesw3MN7oyoYFKeO8-ndCLuTkYkbzHUXhtADg$ thanks for reporting this, I'm wondering if there's anything we can do on the CLI side to improve things here. I had thought we were always setting a UserAgent header in all HTTP calls, however I see that for calls to API's we always explicitly set a custom UserAgent, but for the call to download this file we're just leaving the default (which I didn't think was blank, but perhaps it is).
I'm going to leave this open so we can revisit it when we have some time to see if we can improve things here.
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/github/gh-gei/issues/676*issuecomment-1284606187__;Iw!!NUnUjx3wvH5xgA!aMQRACutiAcvFNrvHGPyBf5xunAhBGbasK73V-ROhtn__3uprgS75eUesw3MN7oyoYFKeO8-ndCLuTkYkbzHvlzHFmU$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/A3RIBK323MPJ6ZQZUVFPQLTWEBTUJANCNFSM6AAAAAAQ75VORU__;!!NUnUjx3wvH5xgA!aMQRACutiAcvFNrvHGPyBf5xunAhBGbasK73V-ROhtn__3uprgS75eUesw3MN7oyoYFKeO8-ndCLuTkYkbzHT3g0SHM$.
You are receiving this because you were mentioned.Message ID: @.***>


[BUG] CLI: plasmo dev is freezing & unresponsive

What happened?
When saving a file the Extension re-packaged is not showing. Then when trying to exit the process with Ctrl + C it is unresponsive, must kill the pid with kill -9.
Is there a way to get more verbose logs so that I can identify the issue - DEBUG=* doesn't reveal any additional information.

Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
No response
Relevant log output
No response
(OPTIONAL) Contribution

[X] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

Ok think this was caused by me overriding the @parcel/watcher module, because I was unable to install plasmo (cc https://github.com/parcel-bundler/watcher/issues/156).

I have now changed the override to the following - and so far seems to work great!
// package.json
{
  ...
  "overrides": {
    "@parcel/watcher": "2.1.0"
  }
}


Remove types.Value Type() in favor of types.TypeOf()

BREAKING CHANGE
This removes the Type() method from the types.Value interface.
Instead use the types.TypeOf(v types.Value) bool function.
Fixes #3324

This does not do any optimizations. It only replaces val.Type() with types.TypeOf(val). Later PRs will update the code to compute the type from the value as well as skipping asking for the type when we do not need it any more.


open redirect for target_link_uri parameter

see https://github.com/zmartzone/mod_auth_openidc/discussions/671, thanks @Meheni
Recently we have forged a URL for a phishing attack that redirects the user, after their authentication on our OP, to any site of our choice.
the forged url is as follows:
https://<oidc_callback>?iss=<issuer>&target_link_uri=https://FQDN_phishing\.<domain_your_oidc_callback>/

example:
https://myapplication.local/app/redirect_oidc?iss=<issuer>&target_link_uri=https://google.fr\.myapplication.local

After authentication, user is redirect to https://google.fr.myapplication.loca
According to the OpenID Connect documentation,
https://openid.net/specs/openid-connect-core-1_0.html#ThirdPartyInitiatedLogin

"target_link_uri
OPTIONAL. URL that the RP is requested to redirect to after authentication. RPs MUST verify the value of the target_link_uri to prevent being used as an open redirector to external sites."

Does the module verify the value of the target_link_uri to prevent being used as an open redirector to external sites? and how to configure it in the module?

Hi,
Thank you for your work.
Meheni

this fix is now included in release <IP_ADDRESS>

Hi,
I just tested the new version <IP_ADDRESS> and the fixed works very well.
I have the following message on my browser :

Error:

URL not allowed

Description:

value does not match the list of allowed redirect URLs: https://google.fr/.myapplication.local

Thank you again for your work.
Best regards,
Meheni


Feature - Add any in operator

I'm not sure if it's a good idea to extend external libraries by modifying their source codes. Why not build on top of it?
What if, for example, JsonLogic decides to add its own implementation for the any_in operator down the road, and we upgrade to this version?

@linhyojee This is the forked repo of public library. So I don't see any issues in updating it directly.
If we need to extend it without updating directly, we might have to take the approach mentioned in this PR https://github.com/liuming/json_logic_elixir/pull/3


fix indentation typo in dashboard reporting

Fixes a an indentation problem introduced with #6082
@ericvaandering please review as that pull request was yours.

Beginning pylint and unit tests.
See https://cmssdt.cern.ch/jenkins/job/DMWM-PR-test/351/ for details

Btw, at the very least this means for 1.0.9.pre3 to 1.0.9.pre5 the dashboard reporting code will throw an error for every single job in the JobAccountant. Didn't investigate further if its only cosmetic or if dashboard reporting is completely broken for these tags or if there are further consequences also for overall job accounting.

No pylint warnings for pull request 6158:

Unit test changes for pull request 6158:

WMCore_t.ReqMgr_t.Service_t.Auxiliary_t.AuxiliaryTest:test_A_HelloWorld_get changed from error to success


Add --no-sanity-check to iptables-wrapper-installer.sh

Description
Add --no-sanity-check to iptables-wrapper-installer.sh for architectures other than amd64
This is required as when performing QemuUserEmulation, it is not possible to actually execute the iptables-nft to determine the version. We are still executing the sanity check on amd64 as that is a native architecture for the build pipeline.
The correct links are still created and the end iptables-wrapper is correct, i.e.:
root@jobsite:~/go/src/github.com/coreos/flannel# git tag v0.13.0-rc1
root@jobsite:~/go/src/github.com/coreos/flannel# make release
#### SNIP ####
root@jobsite:~/go/src/github.com/coreos/flannel# docker run --rm -it --entrypoint /bin/sh quay.io/coreos/flannel:v0.13.0-rc1-arm
/ # ls -l /sbin | grep iptables-wrapper
lrwxrwxrwx    1 root     root            22 Aug 20 16:19 ip6tables -> /sbin/iptables-wrapper
lrwxrwxrwx    1 root     root            22 Aug 20 16:19 ip6tables-restore -> /sbin/iptables-wrapper
lrwxrwxrwx    1 root     root            22 Aug 20 16:19 ip6tables-save -> /sbin/iptables-wrapper
lrwxrwxrwx    1 root     root            22 Aug 20 16:19 iptables -> /sbin/iptables-wrapper
lrwxrwxrwx    1 root     root            22 Aug 20 16:19 iptables-restore -> /sbin/iptables-wrapper
lrwxrwxrwx    1 root     root            22 Aug 20 16:19 iptables-save -> /sbin/iptables-wrapper
-rwxr-xr-x    1 root     root          1797 Aug 20 16:19 iptables-wrapper
/ # cat /sbin/iptables-wrapper
#### SNIP ####
# Update links to point to the selected binaries
for cmd in iptables iptables-save iptables-restore ip6tables ip6tables-save ip6tables-restore; do
    rm -f "/sbin/${cmd}"
    ln -s "/sbin/xtables-${mode}-multi" "/sbin/${cmd}"
done 2>/dev/null || failed=1
if [ "${failed:-0}" = 1 ]; then
    echo "Unable to redirect iptables binaries. (Are you running in an unprivileged pod?)" 1>&2
    # fake it, though this will probably also fail if they aren't root
    exec "/sbin/xtables-${mode}-multi" "$0" "$@"
fi
#### SNIP ####

Release Note
None required

@rajatchopra @AleksandrNull @luthermonson PTAL -- This resolves: https://github.com/coreos/flannel/issues/1339


Added startertemplaterepo to the CMintS as part of

This is a followup to the current discussion.
Currently the startertemplaterepo contains the build script and also netlify.toml file.
There is also a guide how to deploy CMintS application to the Netlify see -> https://cmints.io/en/documentation/getting-started/deployment
This PR is part of the JAMstack Hackathon, so I will highly appreciate if current PR will get an attention before the Hackathon deadline.
Thanks in advance.

CC: @erquhart @verythorough

Tested and worked! 🎉


Add support for request action plugins

Goal
This PR provides plugin developers the same type of action hook available to workspaces and request folders for individual requests. See the documentation and examples below.
Backstory
While looking through a recently submitted plugin I realized that this individual was using the existence of a header property to create an action for a request. While it is an ingenious work-around, it showed that there is an obvious gap.
You can create actions on request folders, and on the workspace, but not the actual requests themselves.
Documentation
Request Actions
Actions can be added to the bottom of the request dropdown by defining a request action plugin.
type RequestAction = {
    label: string,
    action: (context: Context, {
        requestGroup: RequestGroup,
        request: Request
    }): void | Promise<void>,
    label: string,
    icon?: string,
};

// Request actions are exported as an array of objects
module.exports.requestActions = Array<RequestAction>

Example: Plugin to get request details in a modal

module.exports.requestActions = [
  {
    label: "See request data",
    action: async (context, data) => {
      const { request } = data;
      const html = `<code>${JSON.stringify(request)}</code>`;
      context.app.showGenericModalDialog("Results", { html });
    },
  },
];

Example: Send request

module.exports.requestActions = [
  {
    label: "Send request",
    action: async (context, data) => {
      const { request } = data;
.      const response = await context.network.sendRequest(request);
      const html = `<code>${request.name}: ${response.statusCode}</code>`;
      context.app.showGenericModalDialog("Results", { html });
    },
  },
];

Additional Details

Replicates implementation of already existing feature RequestGroupActions
Converts RequestActionsDropdown to be stateful component to support plugin actions
Adds support for activeEnvironment to the RequestActionsDropdown for network requests

Open questions

What do you think about putting plugins above settings instead of below, this way we can slowly move actions over to being plugins, and their position does not jump around (Generate Code for example is an example).

Closes https://github.com/Kong/insomnia/issues/2119

Fixes INS-168

I'd say to leave plugins below settings for now. I don't believe there is currently any control over the ordering of plugins, so some thought will need to be put towards that, in order to ensure that the other actions converted to plugins (eg pin, delete, duplicate, generate code) stay towards the top, while new/other plugins have their own order as well. The ordering will need to be applicable for request group and workspace plugins too.

Ah, perhaps my intent wasn't clear, but I do agree about ordering, however, it was more placing plugins above settings and below the application items. It looks awkward, imo. But, I think I'm okay with this for now.

it was more placing plugins above settings and below the application items

Oh I get ya, misunderstood before. That'd look nice, the slightly different dropdown divider UI does look kind of strange 😅

it was more placing plugins above settings and below the application items.

"application items" are Duplicate, Generate Code, Pin, Copy as Curl and Delete ?
I would prefer putting plugins actions above Settings. But I can't find any argument I guess it is just a personal preference 🤷♂

"application items" are Duplicate, Generate Code, Pin, Copy as Curl and Delete ?

Yes,

I would prefer putting plugins actions above Settings.

I agree, it seems better, will update


How to build attestation-agent/attester/src/bin/evidence_getter.rs to bin to get evidence on AMD snp machine?

How to build attestation-agent/attester/src/bin/evidence_getter.rs to bin to get evidence on AMD snp machine?

I used something like cargo build --features=bin --bin evidence_getter from attestation-agent/attester/ directory

@fffmonkeyking Hi, thanks for bringing this up. Let me add a docs then.


[Snyk] Security upgrade node from 18.13.0-bullseye-slim to 18.15-bullseye-slim

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

Dockerfile

We recommend upgrading to node:18.15-bullseye-slim, as this image has only 43 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

686
Insecure Permissions SNYK-UPSTREAM-NODE-3326666
No Known Exploit

678
Access of Resource Using Incompatible Type ('Type Confusion') SNYK-UPSTREAM-NODE-3326668
No Known Exploit

614
Denial of Service (DoS) SNYK-UPSTREAM-NODE-3326688
No Known Exploit

586
Privilege Escalation SNYK-UPSTREAM-NODE-3329554
No Known Exploit

686
Denial of Service (DoS) SNYK-UPSTREAM-NODE-3329555
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)
🦉 Denial of Service (DoS)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
101

Relevant Lines:
101

💛  - Coveralls


iosevka: 11.0.1 -> 11.2.4

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Superseded by #155015. Forgot to check if anyone had tried to update recently, sorry! For two minor versions I probably wouldn't have bothered...


fix(cloudid): cloudid resource sync worker

What this PR does / why we need it:

[ ] Smoke testing completed

If yes, write the release branches name in the below format and submit the related cherry-pick PR:

release/3.11

/lgtm


Allow configure without --global on git?

Today I tried to re-generate an ebuild and I'm using a machine that's shared so I don't have set  global email and username. Maybe there is a way to allow to do this just locally?
--git-email<EMAIL_ADDRESS>--git-name "My name" flags maybe?
sam $ superflore-gen-ebuilds --ros-distro kinetic --only rosbridge_library --output-repository-path /home/user/sam/ros-overlay
>>>> Creating new branch gentoo-bot-XUuscxloxw...
>>>>
>>>> Regenerating package 'rosbridge_library'...
>>>> failed to parse website for package rosbridge_library
!!!! Failed to resolve required dependencies for package rosbridge_library!
!!!!  unresolved: "python-bson"
>>>> Pulling docker image 'allenh1/ros_gentoo_base:latest'...
>>>> Running docker image...
>>>> Generating manifests...
>>>> Running container with command string 'bash -c 'cd /tmp/ros-overlay/ros-kinetic/rosbridge_library && repoman manifest''...
>>>> Docker container exited.
>>>> Adding changes...
>>>> Committing to branch gentoo-bot-XUuscxloxw...
Traceback (most recent call last):
  File "/usr/local/bin/superflore-gen-ebuilds", line 9, in <module>
    load_entry_point('superflore==0.2.1', 'console_scripts', 'superflore-gen-ebuilds')()
  File "/usr/local/lib/python3.5/dist-packages/superflore/generators/ebuild/run.py", line 175, in main
    overlay.commit_changes(args.ros_distro)
  File "/usr/local/lib/python3.5/dist-packages/superflore/generators/ebuild/overlay_instance.py", line 44, in commit_changes
    self.repo.git.commit(m='{0}'.format(commit_msg))
  File "/usr/local/lib/python3.5/dist-packages/git/cmd.py", line 550, in <lambda>
    return lambda *args, **kwargs: self._call_process(name, *args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/git/cmd.py", line 1009, in _call_process
    return self.execute(call, **exec_kwargs)
  File "/usr/local/lib/python3.5/dist-packages/git/cmd.py", line 820, in execute
    raise GitCommandError(command, status, stderr_value, stdout_value)
git.exc.GitCommandError: Cmd('git') failed due to: exit code(128)
  cmdline: git commit -m regenerate ros-kinetic, Sat Apr 28 21:18:54 2018
  stderr: '
*** Please tell me who you are.

Run

  git config --global user.email<EMAIL_ADDRESS>  git config --global user.name "Your Name"

to set your account's default identity.
Omit --global to set the identity only in this repository.

fatal: unable to auto-detect email address (got 'user@machine.(none)')'

I know it's a bit of pain, but thank you for considering it.

That sounds quite reasonable to me. Ultimately, I'd like superflore to have config files, as the options are starting to get extensive (and nobody is going to enjoy typing them all out every time).
For now, though, I can probably introduce that.


Issue since change on deno polyfill

Hey team,
Since change on deno polyfill (last commits) we have an issue when we use our package from esm:

Example of usages:
export { parseStringPromise } from<EMAIL_ADDRESS>

any way to stay on v15?
Something like this
export { parseStringPromise } from<EMAIL_ADDRESS>

@RoXuS Try importing from like<EMAIL_ADDRESS>or something like that

yeah it is ok with v15!

But we have to focus the file directly
<EMAIL_ADDRESS>-> redirect to v16
<EMAIL_ADDRESS>-> ok on v15

@ije Do you think for the time being until v16 is patched and stuff we can have esm.sh redirect to v15 and require explicit setting to use v16 with cdn.esm.sh/v16/?

i just disabled the events polyfill of std/node, please try again deno --reload

I get

During aleph build...


Python3 -m venv doesn't work properly and keeps giving 'not installed' error message

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [version 10.0.15063]

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)

What's wrong / what should be happening instead:
Basically I try to build virtual environment by 'python3 -m venv [foldername]', which keeps saying the venv module is not installed even it is definitely installed.

It actually creates the folder however lacks of some core files like ./bin/activate.sh ,etc..

I've just tried this on my version, works fine, but I do remember having this issue at some point.. I think I was still running 14.04 at that time?
Are you running 16.04 or 14.04?

Yep my one is 16.04. Sorry about not mentioning that.

Not really sure what's wrong then:

Yes. I did.

That will likely be the reason, if you upgrade form 14.04 it can cause odd bugs like that.
I'd recommend you do a full reinstall using lxrun, will mean you will lose your data inside of bash for windows though. May be worth copying it somewhere.

Thanks for that. Will give it a try.

Yep I did the Ubuntu upgrade by 'do-release-upgrade' which is probably the cause.
This time I just use lxrun/uninstall and lxrun/install to get a clean 16.04 distro and the python module works fine though I got some ipv6 issues during apt operations.
So should I close this ticket or just leave it ?

Closing out old issues, if you're still having issues on newer versions of Windows please open a new issue.


drivers: gpio: gpio_mcux_lpc: fix lost interrupts

if ISR got called for an event on pin X but
another event occurred on pin Y in between the call
to GetStatus() and ClrStatus(), pin Y event was getting
cleared without being processed
Signed-off-by: Maxim Adelman<EMAIL_ADDRESS>

@luchnikov, can you submit an issue report on this for the project's traceability

@dleach02  - https://github.com/zephyrproject-rtos/zephyr/issues/46697
Thanks

@luchnikov Thank you for the fix, can you please share the test application that you used to uncover this problem.

@luchnikov Thank you for the fix, can you please share the test application that you used to uncover this problem.

The bug was discovered during testing of a large internal application I am working on, which runs on custom hardware - sharing the application would therefore be impossible and wouldn't help anyway, without the hardware. Because the bug is very much timing and concurrency-dependent, creating a synthetic test for it would also be quite difficult.