[VL] Enable split velox row vector

I enabled the velox splitter, but found it hash some bugs.

Old TestSingleSplitter partition name is wrong, so it test wrongly, I change it to partition name single, and then core dump
Other test expect result is wrong for null value, I add some logs, the split input vector is same, but result is wrong. Can you help to fix that? @zuochunwei

convert to velox vector 0: {1, 1, 1, null, -0.1234567, null, alice0, alice}
1: {2, -1, 2, null, null, 1, bob1, bob}
2: {3, null, 3, null, 0.1234567, 0, alice2, null}
3: {null, null, 4, null, null, null, bob3, null}
4: {4, -2, null, null, -0.142857, 1, Alice4, Alice}
5: {null, 2, 5, null, null, 1, Bob5, Bob}
6: {5, null, 6, null, 0.142857, 0, AlicE6, null}
7: {6, null, 7, null, 0.285714, 1, boB7, alicE}
8: {null, 3, 8, null, 0.428617, null, ALICE8, null}
9: {7, -3, null, null, null, null, BOB9, boB}
split this vector0: {1, 1, 1, null, -0.1234567, null, alice0, alice}
1: {2, -1, 2, null, null, 1, bob1, bob}
2: {3, null, 3, null, 0.1234567, 0, alice2, null}
3: {null, null, 4, null, null, null, bob3, null}
4: {4, -2, null, null, -0.142857, 1, Alice4, Alice}
5: {null, 2, 5, null, null, 1, Bob5, Bob}
6: {5, null, 6, null, 0.142857, 0, AlicE6, null}
7: {6, null, 7, null, 0.285714, 1, boB7, alicE}
8: {null, 3, 8, null, 0.428617, null, ALICE8, null}
9: {7, -3, null, null, null, null, BOB9, boB}

result [
  1,
  2,
  3,
  0,
  4,
  0,
  5,
  6,
  0,
  7
]
 expected [
  1,
  2,
  3,
  null,
  4,
  null,
  5,
  6,
  null,
  7
]

I enabled the velox splitter, but found it has some bugs.

Old TestSingleSplitter partition name is wrong, so it test wrongly, I change it to partition name single, and then core dump
Other test expect result is wrong for null value, I add some logs, the split input vector is same, but result is wrong. Can you help to fix that? @zuochunwei

convert to velox vector 0: {1, 1, 1, null, -0.1234567, null, alice0, alice}
1: {2, -1, 2, null, null, 1, bob1, bob}
2: {3, null, 3, null, 0.1234567, 0, alice2, null}
3: {null, null, 4, null, null, null, bob3, null}
4: {4, -2, null, null, -0.142857, 1, Alice4, Alice}
5: {null, 2, 5, null, null, 1, Bob5, Bob}
6: {5, null, 6, null, 0.142857, 0, AlicE6, null}
7: {6, null, 7, null, 0.285714, 1, boB7, alicE}
8: {null, 3, 8, null, 0.428617, null, ALICE8, null}
9: {7, -3, null, null, null, null, BOB9, boB}
split this vector0: {1, 1, 1, null, -0.1234567, null, alice0, alice}
1: {2, -1, 2, null, null, 1, bob1, bob}
2: {3, null, 3, null, 0.1234567, 0, alice2, null}
3: {null, null, 4, null, null, null, bob3, null}
4: {4, -2, null, null, -0.142857, 1, Alice4, Alice}
5: {null, 2, 5, null, null, 1, Bob5, Bob}
6: {5, null, 6, null, 0.142857, 0, AlicE6, null}
7: {6, null, 7, null, 0.285714, 1, boB7, alicE}
8: {null, 3, 8, null, 0.428617, null, ALICE8, null}
9: {7, -3, null, null, null, null, BOB9, boB}

result [
  1,
  2,
  3,
  0,
  4,
  0,
  5,
  6,
  0,
  7
]
 expected [
  1,
  2,
  3,
  null,
  4,
  null,
  5,
  6,
  null,
  7
]

OK, I will check it later

I enabled the velox splitter, but found it has some bugs.

Old TestSingleSplitter partition name is wrong, so it test wrongly, I change it to partition name single, and then core dump
Other test expect result is wrong for null value, I add some logs, the split input vector is same, but result is wrong. Can you help to fix that? @zuochunwei

convert to velox vector 0: {1, 1, 1, null, -0.1234567, null, alice0, alice}
1: {2, -1, 2, null, null, 1, bob1, bob}
2: {3, null, 3, null, 0.1234567, 0, alice2, null}
3: {null, null, 4, null, null, null, bob3, null}
4: {4, -2, null, null, -0.142857, 1, Alice4, Alice}
5: {null, 2, 5, null, null, 1, Bob5, Bob}
6: {5, null, 6, null, 0.142857, 0, AlicE6, null}
7: {6, null, 7, null, 0.285714, 1, boB7, alicE}
8: {null, 3, 8, null, 0.428617, null, ALICE8, null}
9: {7, -3, null, null, null, null, BOB9, boB}
split this vector0: {1, 1, 1, null, -0.1234567, null, alice0, alice}
1: {2, -1, 2, null, null, 1, bob1, bob}
2: {3, null, 3, null, 0.1234567, 0, alice2, null}
3: {null, null, 4, null, null, null, bob3, null}
4: {4, -2, null, null, -0.142857, 1, Alice4, Alice}
5: {null, 2, 5, null, null, 1, Bob5, Bob}
6: {5, null, 6, null, 0.142857, 0, AlicE6, null}
7: {6, null, 7, null, 0.285714, 1, boB7, alicE}
8: {null, 3, 8, null, 0.428617, null, ALICE8, null}
9: {7, -3, null, null, null, null, BOB9, boB}

result [
  1,
  2,
  3,
  0,
  4,
  0,
  5,
  6,
  0,
  7
]
 expected [
  1,
  2,
  3,
  null,
  4,
  null,
  5,
  6,
  null,
  7
]

done


Update timeout for .github/workflows/buildifier.yml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/buildifier.yml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

 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.

Closing in favor of https://github.com/Kong/kong/pull/12566.


How would you go about adding a max to the queue?

I think it's not always good to have an infinite amount of queued sounds,
since it could mean a sound is played minutes after it was supposed to.
(this often occurs if you have a lot of things making noise)
I'm not quite sure on how to add a maximum of noises in the queue or on how to remove the queue entirely.
How would you go about doing this?

The queue is just an array, so you can check its length before appending the sound.


[STK-212] Added code to get nodes and pods of Cluster

Getting nodes and pods in a cluster

@kahootali always have meaningful branch names

@kahootali Yikes!  Build 1 has Failed!

@kahootali Yikes!  Build 2 has Failed!

@kahootali Yikes!  Build 3 has Failed!

@kahootali Yikes!  Build 4 has Failed!

@kahootali Yikes!  Build 5 has Failed!

@kahootali Yikes!  Build 6 has Failed!

@kahootali Yikes!  Build 7 has Failed!

@kahootali Yikes!  Build 8 has Failed!

@kahootali Yikes!  Build 15 has Failed!

@kahootali Yikes!  Build 16 has Failed!

@kahootali Yikes!  Build 17 has Failed!

@kahootali Yikes!  Build 18 has Failed!

@kahootali Yikes!  Build 19 has Failed!

@kahootali Yikes!  Build 20 has Failed!

@kahootali Yikes!  Build 21 has Failed!

@kahootali Yikes!  Build 22 has Failed!

@kahootali Yikes!  Build 23 has Failed!

@kahootali Yikes!  Build 24 has Failed!

@kahootali Yikes!  Build 26 has Failed!

@kahootali Yikes!  Build 27 has Failed!

@kahootali Yikes!  Build 28 has Failed!

@kahootali Yikes!  Build 29 has Failed!

@kahootali Yikes!  Build 30 has Failed!

@kahootali Yikes!  Build 32 has Failed!

@kahootali Yikes!  Build 33 has Failed!

@kahootali Image is available for testing. docker pull stakater/chowkidar:PR-3-34

@kahootali Image is available for testing. docker pull stakater/chowkidar:PR-3-35

@kahootali Image is available for testing. docker pull stakater/chowkidar:PR-3-36

@kahootali Yikes!  Build 37 has Failed!

@kahootali Image is available for testing. docker pull stakater/chowkidar:PR-3-38

@kahootali Image is available for testing. docker pull stakater/chowkidar:PR-3-40


[Bugfix] Carefully handle TIC / EPIC ID overlap

This PR addresses a couple of issues. First, @christinahedges pointed out that certain searches returned overlapping targets and claimed a distance of 0 between them.
Example:

Now, the _query_mast() function parses the ID from queries beginning in EPIC or KIC, and assigns them their kplr... or ktwo... names. These queries now resolve to the correct objects.
🚨New bug alert 🚨
While fixing this issue, I found that there are targets observed by TESS with overlapping TIC and EPIC IDs. Depending on the syntax of the search, either one can be returned.
Example:

Next step:
Provide a warning when a target ID is within the overlap range of TIC and EPIC. Also, set the default formatting to TIC # when an int is passed into search_tesscut().

@barentsen said he will provide the EPIC ID range, which we can compare to the TIC ID range to constrain the possible overlapping targets. This will help us provide the warning message to targets which only have alternate observations with the same ID number.

The full range of target identifiers for observed Kepler targets is: [757076,	100004300].
The full range of target identifiers for observed K2 targets is: [200000811, 251816811].

If we merge this PR, calls like lk.search_lightcurvefile("EPIC 203348744") will directly search the K2 data archive for that exact target ID rather than resolving to coordinates, which solves the problem of the overlapping target being returned instead ("EPIC 203348270").
This is good, :rotating_light:  HOWEVER I ALSO NOTICED A NEW ISSUE :rotating_light:
...if we merge this PR, calls like lk.search_lightcurvefile("KIC 8462852") will ONLY return Kepler data.  This is a problem because KIC 8462852 has now been observed by TESS, and we can only show the TESS results to the user if we resolve the target identifier to coordinates.  That's bad! =(

The only good solution I can currently think of to resolve this issue is to add a new keyword, e.g. resolve, to let a user determine whether or not the target string will be resolved to coordinates.
For example:

lk.search_lightcurvefile("EPIC 203348744", resolve=False)
:arrow_right: this will ONLY return K2 observations made using the mask identifier "203348744"  (i.e. this translates to a MAST portal query on target_name='ktwo203348744').
lk.search_lightcurvefile("EPIC 203348744", resolve=True)
:arrow_right: this will resolve the coordinates of the target string to (ra, dec) positions and return ALL Kepler/K2/TESS observations within the search radius around those coordinates.

The default would be resolve=True.

This should now have been resolved by #796  :+1:


Resolvable external entity references: Melinda

Spefication
This validator cannot do fixing. It's purpose is to check whether the configured fields have resolvable references to external entities.
This validator supports Melinda's internal record references. The resolution is done by querying Melinda's SRU-service with record ids (CQL-query: rec.id=<id>)
SRU-query example
import {parseString} from 'xml2js';
import fetch from 'node-fetch';

const response = await fetch('https://melinda-test.kansalliskirjasto.fi:210/fin01?operation=searchRetrieve&maximumRecords=100&version=1&query=rec.id=5000');
parseString(await response.text(), (err, result) => {
  // Check that the resulting document contains only one record-element
});

Name
resolvable-ext-references-melinda
Description
Checks if Melinda entity references are resolvable
Configuration

endpoint: URL pointing to the SRU endpoint (Mandatory)
prefixPattern: Pattern which must match to the subfield value and is also removed from the value to get the actual record id
fields: List of subfields from where the references are selected. An object with field tag patterns as keys and subfields codes as values (Represented as an array) (Mandatory)

Examples
Config
"endpoint": "https://foo.bar",
"prefixPattern": /\(FOOBAR\)/,
"fields": {
  "773": ["w"]
}

Record with references according to config
{
  "fields": [
    {
      "tag": "001",
      "value": "123456"
    },
    {
      "tag": "035",
      "subfields": [
        {
          "code": "a",
          "value": "(FI-MELINDA)123456"
        }
      ]
    },
    {
      "tag": "773",
      "subfields": [
        {
          "code": "w",
          "value": "(FOOBAR)4567"
        }
      ]
    }
  ]
}

Record without references according to config
{
  "fields": [
    {
      "tag": "001",
      "value": "123456"
    },
    {
      "tag": "035",
      "subfields": [
        {
          "code": "a",
          "value": "(FI-MELINDA)123456"
        }
      ]
    },
    {
      "tag": "773",
      "subfields": [
        {
          "code": "w",
          "value": "(FI-MELINDA)4567"
        }
      ]
    }
  ]
}

Implemented in 5d4bbdce6eb8f60d4dff0a883f41fc8a81a01a5b


Fix blank spaces between partitions

Description
Fix huge blank spaces between partitions
Why is this needed
rootio partition leaves huge blank spaces between partitions
Fixes: #98
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

The change has made its way into upstream. Thanks.


Scry preview page doesn't have a profile link

I opened a previewer link for scries on my skincent here and the human icon that's supposed to lead you to your tester profile isn't there. I checked on another skin and none of the scry links have it.

Fixed


Re-word "Cryptic Password" to be more clear on functionality

Summary
When generating passwords using gopass create, the user is asked if they want a "Cryptic Password", which seems not very clear to me. I actually searched for a while to find out how to create xkcd-style passphrases for a system where I might be expected to read the password over the phone, before finally stumbling onto the fact that gopass uses "non-cryptic" to mean a passphrase style password of English words.
Steps To Reproduce
Run gopass create and read the test
Expected behavior
A more clear explanation of what "Cryptic Password" means
Environment

OS: Gentoo
OS version: Linux 4.19.71-gentoo,
gopass Version: 1.8.5
Installation method: Installed via Gentoo's portage

Additional context
I didn't want to submit this issue until I had some ideas on what could replace "cryptic passphrase", so here are my best ideas on what could replace it. Note that the previous line will be "Generate Password?" so the intent is that users should already be aware that password generation is happening

- by concatenating randomly selecting words?
- consisting of words?
Generate human-pronounceable passphrase?

I don't know if any of these are best, but I at least wanted to talk about newer wording for this feature.

Good suggestions, thanks. This feature is indeed poorly named.

This has been fixed in  #1194 so closing.


code editorial screen shots are not visible

The code editorials are not visible for any questions. Im not sure if they have been taken down. Any updates

Hey, nothing has been changed recently. The editorial code isn't available for all problems, just some of them. I can still view the code here as usual: https://github.com/akhilkammila/leetcode-screenshotter/tree/main/editorial_code
If you are experiencing something different, could you lmk in more detail

The editorial screenshots , which should contain the images of the editorial concepts and code of the problems of leetcode questions (.png files) are not visible in the repository.
Im not sure if this repository is just the compilation of the code solution of leetcode problems. I was expecting it to be like a compilation of the whole editorial solution with detailed concepts and code for the problems.

Oh ok, I see. Yeah it is supposed to show the whole editorial. I am not having the same issue. It should look like this (and as you scroll down, the rest of the article is there):

Could you lmk if the download button works? What browser are you using, and does it work on a different browser?
To me, it seems to be an issue with github displaying the png file. This might help: https://github.com/orgs/community/discussions/56585. I think trying a different browser or clearing cache could be a good place to start.

My issue was same as the https://github.com/orgs/community/discussions/56585#discussioncomment-7150482. I have switched the network and its working now. Thanks a lot. You have made a really helpful repository.  I was thinking if one could make a pdf book of the screenshots and the questions. It would be like a DSA problems exercise book lol.
You have really interesting projects. Could you please help me and guide me on how to build such bots .I would like to connect and work together on something.


New Typeform Entry

A new entry with ID 4969bac7b56e83a82ad060f0ae57faed was submitted at 2018-08-24T09:56:39Z. It took  to complete.A new entry was submitted in Typeform with ID 4969bac7b56e83a82ad060f0ae57faed at 2018-08-24T09:56:39Z. Time to complete:

newentry


WIP: Add ipv4_ipam_pool_id & ipv4_netmask_length

what

Add ipv4_ipam_pool_id & ipv4_netmask_length variables
Update cidr_block with default value of null

why

In order to support assigning cidr_block for VPC from AWS VPC IPAM pool

@Nuru this is a modification I was working on locally and mentioned in last weeks office hours. Was suggested I run past you to get your thoughts.

/test all

@korenyoni I'll go back and take another look at this... The use case I'm working on is being able to use AWS IPAM when deploying out the VPC so it would either use cidr_block or ipv4_ipam_pool_id & ipv4_netmask_length.  The two should be mutually exclusive. You couldn't set all 3 variables obviously as they would conflict.

/test all


Disable cookie authen

Fixes https://github.com/aspnet/AspNetCore.Docs/issues/6063 details in https://github.com/aspnet/Security/issues/1394
Internal review URL

@John0King let me know if you can work on this.  On the file changed tab, edit in browser.

@John0King sorry, I forgot you did the work in  #12424

I'm not familiar with the DocFx style markdown , and I'm learning it now,  and I will provide a full sample for this page tomorrow or later.

do I need to crate another branch to push new change

Yes. But you can edit in browser and it will create a new branch for you.

At the top right of the article, select the Edit pen icon:

Select the pen icon again. Save.

@Rick-Anderson  I create a new PR #12466  now

@severisv @John0King  @serpent5  please review

@John0King  the following code from @severisv looks simpler
services.AddAuthentication(CookieAuthenticationDefaults.AuthenticationScheme)
        .AddCookie(options =>
         {
             options.Events.OnRedirectToLogin = context =>
             {
                 context.Response.Headers["Location"] = context.RedirectUri;
                 context.Response.StatusCode = 401;
                 return Task.CompletedTask;
             };
         });

See his SO post

without handle the browse request case, it won't redirect to login page

without handle the browse request case, it won't redirect to login page

That's true, but I don't think we need that. In most cases the clients are not browsers.

@serpent5
To review the doc build

download https://www.dropbox.com/s/kg7bll4gio6lo74/1.zip?dl=0
unzip
view HTML file in FF (Fire Fox). It best to view the doc with the widest window that has no left or right TOC (Table  of contents)

Page looks good. Will there be another commit to add it the ToC?
Thanks for adding me as an author. :tada:

@serpent5  yes, just added, see this commit

There's no discussion of WHY you would want to do this
The short title "Disable automatic cookie authentication" is simply wrong. There is not automatic cookie authentication, it's a choice a developer makes to include in their project
The initial paragraph doesn't read well, and isn't true if a policy or role restriction is specified.

This needs to cover why you want to use cookies, the dangers of using cookies (CSRF) and how it may be mitigated by same site cookies (which need to be configured to be strict), and the more standard approach of JWT.

There's good coverage of CSRF protection in the docs already (here). Would it be worth updating that to include details of SameSite mitigation and then linking to the page in a big "dragons ahead" style warning on this page being talked about here?
From what I know, there's no official docs page or guidance on using JWTs. Is this something that can be created and then linked to from here too? I think I've seen this come up before and be partially countered with the new IS4 integration that's coming up, but I guess for the simpler one-site apps that might be too heavy.


This doesn't seem to work at all...

I don't seem to be able to get this to work at all with the current Facebook JS API. First off, the getSessionFromCookie function uses the cookie prefix "fbs" where its now "fbsr".
Though I have tried changing that and I don't seem to be able to get this var session = qs.parse(cookie); to decode the cookie, if I console.log(session); I just get an object with the same encrypted string in that the cookie has.
Anyone else having this issue? I don't know enough about node to get digging :(

@tarnfeld: Unfortunately, you are right. Personally, I haven’t used this library in a while but since it’s direct port of the official Facebook Python SDK which has been deprecated, it also stopped working. Feel free to issue a pull request or fork the project.

Alright, thanks ;) I needed a lib for a quick hack project so havnt had the chance to write a fix for this, though maybe when I get a chance I will!
Sent from my iPhone
On 9 Jan 2012, at 08:20, Daniel<EMAIL_ADDRESS>wrote:

@tarnfeld: Unfortunately, you are right. Personally, I haven’t used this library in a while but since it’s direct port of the official Facebook Python SDK which has been deprecated, it also stopped working. Feel free to issue a pull request or fork the project.

Reply to this email directly or view it on GitHub:
https://github.com/gasi/node-facebook-graph/issues/3#issuecomment-3408528


Update functions-networking-options.md

@andresnava : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Correction confirmed. Thanks.


Avoid spawning Flink cluster when user control annotation job-cancel is submitted

Currently when cluster is in creating state or stuck in this state due to bad supplied image or absence of underlying resources it's not possible for users to cancel the current submission and abort the creation of the cluster without deleting the FlinkCluster resource.

Closed via #376


Create a headless account in ArcGIS Online

In production we are refreshing hosted feature layers owned by a headless account with a password.  We use the same headless account and password to perform the refresh.
Is it possible to create another headless account for running integration tests?  If not, attempt to set up test data using my personal account.

From the AGOL organization

Add members without sending invitations (user password)
Add a new member in the creator user type with publisher role

See #2 for next steps/possible alternative approach


⚠ Bofrost.de has degraded performance

In 92e0741, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1148 ms

Resolved: Bofrost.de performance has improved in 66048c2 after 6 minutes.


SIL: remove the private_external SILLinkage

The private_external linkage was a relict from the sil-serialize-all days. This linkage doesn't make any sense because a private function cannot be referenced from another module (or file, in case of non-wmo compilation).
This PR also contains another change: diagnose body-less private functions with an _silgen_name attribute
Functions with an @_silgen_name attribute can be declared without a body - for referencing externally defined C functions. But this does not make sense for private functions.
So far, this error was not diagnosed - and it didn't cause any problem, because such functions were generated with a private_external linkage.
With removing the linkage, it does not work anymore and would result in SILVerifier errors.

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - 8e0fad2629a8452a515edede2fd8a8fd77daa27e

Build failed
Swift Test Linux Platform
Git Sha - e526abc792714880c63a6ba35bc31bdc1df2cedc

@swift-ci test


Use muslrust:stable instead of rust-musl-builder

fixes: https://github.com/LukeMathWalker/cargo-chef/issues/122

I'd prefer switching over to clux/muslrust:stable as base image for musl - what do you think?

Works well, updated the PR. PTAL. Thanks for recommending muslrust:stable and cargo-check is amazing.


chore: bump next version to v10.0.8

What kind of change does this PR introduce?
Chore
What is the new behaviour?
Update Next dev and peer version to ^10.0.8 and update router mock to include new typed props
Does this PR introduce a breaking change?
Yes.
Other information:
Please check if the PR fulfills these requirements:

[ ] Tests for the changes have been added
[X] Docs have been added / updated

It seems I can't reproduce the CI failure on my machine. Need investigation.

Update: it's a timeout error. We should make sure whether it's an issue introduced with async configuration loading, or we should just increase timeout threshold.

Coverage remained the same at 100.0% when pulling fa8e031140ff238eb27f560a50b405162ee6e786 on next-v10.0.8 into c3dc2913071d9389a7d802cd10166f21a47d5e05 on master.

I figured what's causing x4 execution time.
loadConfig makes a new loadWebpackHook call which seems to be the origin of the issue.

With this hack makes me really sad, it's definitely an order of magnitude worst then any workaround we had to put up in this project :) @Meemaw I'll let to you the final word.
An alternative solution might consist of re-implementing config loading.


Select input ugly rendering in non-chrome browsers

Steps
I'm trying to display some select input on a ui form but it seems results are inconsistent between browsers. I'm not sure if I am doing something wrong or it is a rendering bug of semantic ui (browser rules takes priority?)
Expected Result
Chrome   67 and 69 (Mac Os 10.11.6 )

Actual Result
Firefox 60.0.2 (Mac Os 10.11.6 )

Safari 11.0.3 (Mac Os 10.11.6 )

Version
2.2.10
Testcase
https://jsfiddle.net/dae83sp6/

You need to add .ui.dropdown to your select elements and initialize them, take a look at the dropdown docs page. https://semantic-ui.com/modules/dropdown.html
Here is a working example: https://jsfiddle.net/mzn93agh/


Disable metrics-based autoscaling by default when scaleUpTriggers are enabled

Relates to https://github.com/summerwind/actions-runner-controller/pull/379#discussion_r592813661
Relates to https://github.com/summerwind/actions-runner-controller/issues/377#issuecomment-793266609
When you defined HRA.Spec.ScaleUpTriggers[] but HRA.Spec.Metrics[], the HRA controller will now enable ScaleUpTriggers alone and insteaed of automatically enabling TotalNumberOfQueuedAndInProgressWorkflowRuns. This allows you to use ScaleUpTriggers alone, so that the autoscaling is done without calling GitHub API at all, which should grealy decrease the change of GitHub API calls get rate-limited.

@mumoshu how does scale down happen when we use Github Webhook based scaling and no metrics?

@Puneeth-n Each scaleUpTrigger expires after the configured duration passed. Let's say duration was 5m(5 minutes), RD's desired replicas increases by the configured replica(s) for 5 minutes. After 5 minutes the RD's desired replicas decreases by 1.
You'll also find https://github.com/summerwind/actions-runner-controller#faster-autoscaling-with-github-webhook useful!

@mumoshu Thanks for clarifying. I didn't know it would scale down as well after the "duration"


GROOVY-7624: Collection asImmutable() methods aren't immutable

The proposed PR provides asUnmodifiable() aliases for the current behavior. The asImmutable() variants are then using a copy of the original, making them effectively immutable.


Unhandled exception in YARD::Handlers::Ruby::MixinHandler:   in `lib/rubocop/ext/processed_source.rb during Building YARD (yri) index for rubocop

I upgraded my Ruby version and wanted to install solargraph which has rubocop dependency and during installation it threw some errors.

Expected behavior
No warnings or errors?
Actual behavior
ruby -v                                                                                main
ruby 3.3.2 (2024-05-30 revision e5a195edf6) [x86_64-linux]
nfedyashev@mxlinux  nextgen main gem install solargraph                                                                                main
Fetching unicode-display_width-2.5.0.gem
Fetching ast-2.4.2.gem
Fetching parser-<IP_ADDRESS>.gem
Fetching tilt-2.3.0.gem
Fetching ruby-progressbar-1.13.0.gem
Fetching yard-0.9.36.gem
Fetching rubocop-ast-1.31.3.gem
Fetching rainbow-3.1.1.gem
Fetching parallel-1.24.0.gem
Fetching language_server-protocol-<IP_ADDRESS>.gem
Fetching rubocop-1.64.1.gem
Fetching reverse_markdown-2.1.1.gem
Fetching rbs-2.8.4.gem
Fetching kramdown-2.4.0.gem
Fetching kramdown-parser-gfm-1.1.0.gem
Fetching jaro_winkler-1.5.6.gem
Fetching e2mmap-0.1.0.gem
Fetching backport-1.2.0.gem
Fetching solargraph-0.50.0.gem
Successfully installed yard-0.9.36
Successfully installed tilt-2.3.0
Successfully installed unicode-display_width-2.5.0
Successfully installed ruby-progressbar-1.13.0
Successfully installed ast-2.4.2
Successfully installed parser-<IP_ADDRESS>
Successfully installed rubocop-ast-1.31.3
Successfully installed rainbow-3.1.1
Successfully installed parallel-1.24.0
Successfully installed language_server-protocol-<IP_ADDRESS>
Successfully installed rubocop-1.64.1
Successfully installed reverse_markdown-2.1.1
Building native extensions. This could take a while...
Successfully installed rbs-2.8.4
Successfully installed kramdown-2.4.0
Successfully installed kramdown-parser-gfm-1.1.0
Building native extensions. This could take a while...
Successfully installed jaro_winkler-1.5.6
Successfully installed e2mmap-0.1.0
Successfully installed backport-1.2.0
Successfully installed solargraph-0.50.0
Building YARD (yri) index for yard-0.9.36...
Building YARD (yri) index for tilt-2.3.0...
Building YARD (yri) index for unicode-display_width-2.5.0...
Building YARD (yri) index for ruby-progressbar-1.13.0...
Building YARD (yri) index for ast-2.4.2...
Building YARD (yri) index for parser-<IP_ADDRESS>...

Building YARD (yri) index for rubocop-ast-1.31.3...
Building YARD (yri) index for rainbow-3.1.1...
Building YARD (yri) index for parallel-1.24.0...
Building YARD (yri) index for language_server-protocol-<IP_ADDRESS>...
Building YARD (yri) index for rubocop-1.64.1...
[error]: Unhandled exception in YARD::Handlers::Ruby::MixinHandler:
  in `lib/rubocop/ext/processed_source.rb`:20:

	20: RuboCop::ProcessedSource.include RuboCop::Ext::ProcessedSource

[error]: NoMethodError: undefined method `mixins' for an instance of YARD::CodeObjects::ConstantObject
[error]: Stack trace:
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/code_objects/base.rb:379:in `method_missing'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/code_objects/proxy.rb:180:in `method_missing'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/ruby/mixin_handler.rb:44:in `process_mixin'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/ruby/mixin_handler.rb:12:in `block (2 levels) in <class:MixinHandler>'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/ruby/mixin_handler.rb:10:in `each'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/ruby/mixin_handler.rb:10:in `block in <class:MixinHandler>'

Building YARD (yri) index for reverse_markdown-2.1.1...
Building YARD (yri) index for rbs-2.8.4...
[error]: Unhandled exception in YARD::Handlers::Ruby::ExceptionHandler:
  in `lib/rbs/errors.rb`:155:

	155: raise new(type_name: type_name, location: location)

[error]: NoMethodError: undefined method `source' for nil
[error]: Stack trace:
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/ruby/exception_handler.rb:18:in `block in <class:ExceptionHandler>'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:114:in `block (2 levels) in process'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:112:in `each'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:112:in `block in process'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:111:in `each'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:111:in `each_with_index'

[error]: Unhandled exception in YARD::Handlers::Ruby::ExceptionHandler:
  in `lib/rbs/errors.rb`:171:

	171: raise new(super_decl)

[error]: NoMethodError: undefined method `source' for nil
[error]: Stack trace:
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/ruby/exception_handler.rb:18:in `block in <class:ExceptionHandler>'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:114:in `block (2 levels) in process'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:112:in `each'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:112:in `block in process'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:111:in `each'
	/home/nfedyashev/.rbenv/versions/3.3.2/lib/ruby/gems/3.3.0/gems/yard-0.9.36/lib/yard/handlers/processor.rb:111:in `each_with_index'

Building YARD (yri) index for kramdown-2.4.0...
Building YARD (yri) index for kramdown-parser-gfm-1.1.0...
Building YARD (yri) index for jaro_winkler-1.5.6...
Building YARD (yri) index for e2mmap-0.1.0...
Building YARD (yri) index for backport-1.2.0...
Building YARD (yri) index for solargraph-0.50.0...
Done installing documentation for yard, tilt, unicode-display_width, ruby-progressbar, ast, parser, rubocop-ast, rainbow, parallel, language_server-protocol, rubocop, reverse_markdown, rbs, kramdown, kramdown-parser-gfm, jaro_winkler, e2mmap, backport, solargraph after 211 seconds
19 gems installed

A new release of RubyGems is available: 3.5.9 → 3.5.11!
Run `gem update --system 3.5.11` to update your installation.

Steps to reproduce the problem
switch to ruby 3.3.2
gem install solagraph -v 0.50
or probably gem install rubocop -v 1.64.1
RuboCop version
1.64.1

Thank you for the feedback. This seems to be an error in the YARD, not RuboCop. Can you open an issue to the YARD repository?
https://github.com/lsegal/yard/issues


Bring e2e-tls fixes to main

What this PR does / why we need it:

Brings the fix from https://github.com/openshift-knative/serving/pull/871 to main.

Does this PR needs for other branches:
No, done manually
Does this PR (patch) needs to update/drop in the future?:
No
/assign @skonto

/lgtm
/approve

/hold for 1.15

for 1.15

Should not be affected by the main branch, no?

/unhold


Driver does not work when plugged into USB 2.0

As in description driver does not work when plugged into USB 2.0 port, when plugged into USB 3.0 port it works fine.
This is the relevant dump from dmesg
[ 3407.854535] usb 1-1.3: new full-speed USB device number 33 using ehci-pci
[ 3413.049856] usb 1-1.3: unable to get BOS descriptor or descriptor too short
[ 3418.169816] usb 1-1.3: unable to read config index 0 descriptor/all
[ 3418.169829] usb 1-1.3: can't read configurations, error -110
[ 3418.254524] usb 1-1.3: new full-speed USB device number 34 using ehci-pci
[ 3423.545786] usb 1-1.3: unable to get BOS descriptor or descriptor too short
[ 3428.665730] usb 1-1.3: unable to read config index 0 descriptor/all
[ 3428.665743] usb 1-1.3: can't read configurations, error -110
[ 3428.670490] usb 1-1-port3: attempt power cycle
[ 3429.274476] usb 1-1.3: new full-speed USB device number 35 using ehci-pci
[ 3434.553710] usb 1-1.3: device descriptor read/all, error -110
[ 3434.638467] usb 1-1.3: new full-speed USB device number 36 using ehci-pci
[ 3439.929695] usb 1-1.3: device descriptor read/all, error -110
[ 3439.934443] usb 1-1-port3: unable to enumerate USB device

It turns out that this bug occurs also when plugging device into APU2 (USB 3.0).
Probably this is the same bug that caused USB to stop working at all after enabling optimizations in Fobnail firmware.
USB tends to fail randomly in various ways, sometimes errors happen, enumeration freezes for 3 seconds or so and the continues (here fails to read string descriptors).
[193586.025268] usb 3-2: unable to get BOS descriptor or descriptor too short
[193596.265257] usb 3-2: string descriptor 0 read error: -110
[193596.265269] usb 3-2: New USB device found, idVendor=1234, idProduct=4321, bcdDevice= 0.10
[193596.265282] usb 3-2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[193596.281722] cdc_eem 3-2:1.0 usb0: register 'cdc_eem' at usb-0000:00:14.0-2, CDC EEM Device, 22:4f:24:61:e7:db

This looks like either usb-device or nrf-usbd bug, chance that the bug is located here is minimal.

Actually problem is in Fobnail firmware, when formatting persistent storage NVMC halts CPU for too much delaying USB interrupts for ~90 ms. Worked around by formatting storage before initializing USB https://github.com/fobnail/fobnail/pull/23


Bump rexml from 3.2.4 to 3.2.5

Bump rexml from 3.2.4 to 3.2.5
This issue is related to https://github.com/fastai/fastpages/pull/518

/preview


Availability of Exp(.) and Log(.)

Does this library have the capability mentioned in #14? I should be working in PGA (G+ 3,0,1) and I'm trying to grok the text mentioned below as to whether I need CGA (4,1) or Lie Alg/Group transformations.
Utilizing chained applications of screws/motors on MediaPipe landmarks without actual kinematics should be sufficient (though I'm hand-waving here without fully understanding much...)
Depending on how complicated things become, I may abandon using GA anyways, as the code I'm writing for Kaggle cloud instances requires Python 3.7 and code that diverges on my local machine is already an existential risk. The TF Lite submission can be developed wherever and only needs to be submitted online.
However, I have an AMD GPU and I'm running into some compatibility issues anyways. I got a docker container to run with ROCm and I'm checking the functionality now. Your library seems to extend TF without adding C++ code, so AFAIK it wouldn't need a rebuild. The errors I'm getting now involve SEE3/etc and AVX/2 instructions in oneDNN which indicates I need to rebuild anyways. I think I can handle that.
I don't really need help with the model and that may be against the rules. I'm not sure. To give you some background on what I'm doing:
The project is for a Kaggle competition using MediaPipe data on ASL and I think using GA would give me an advantage. I have this GA Applications text for Robotics to help me, but the math is a bit over my head. I understand some aspects from a high-level.
What I'm hoping to gain via GA is using some multivector values for data analysis & training, potentially using this method for live classification, but the rules require the TFLite build to be less than 40MB. Other submissions are reportedly less than 5MB which seems low.

Ahhh so it seems to be working with my environment. I just need to watch the types. I still might bail though if it gets too complex ... or double or dual.

On master there are only approximations for exp and log, however there's a PR which implements the invariant decomposition in #21 and uses it for exp(), but not log() yet. It was tricky making it work with tensorflow natively.
If you want to learn more aobut GA you could join our Discord server https://discord.gg/vGY6pPk (https://bivector.net/), we have channels for beginner questions and programming which you might find interesting :)


Support for running cattle-cluster-agent with trace logging

Is your feature request related to a problem? Please describe.
Currently, it's easy to run cattle-cluster-agent with debug-level logging by setting CATTLE_DEBUG or RANCHER_DEBUG env var to true, see https://github.com/rancher/rancher/blob/67422772955d5830b237fe84dbf1a26d266b55c1/cmd/agent/main.go#L60-L62
However, running with trace-level logging is not directly supported.
Describe the solution you'd like
Following the established pattern it's suggested to add support for CATTLE_TRACE/RANCHER_TRACE env vars to enable trace logging for cattle-cluster-agent.
Describe alternatives you've considered
As an alternative it's already possible to exec into running agent container and run loglevel --set trace to set logging level to trace; however:

This is less than trivial and is only temporary
In case trace-level logging is needed for the initial startup of the agent, some solution to execute it prior to starting the agent needs to be figured out (not immediately obvious how it will work but likely possible)
This sets the log level to trace globally for the container, not just for the agent executable

Additional context
While at it, we also want to explicitly document setting logging for cattle-cluster-agent on https://ranchermanager.docs.rancher.com/troubleshooting/other-troubleshooting-tips/logging as it's missing that information.

Merged on https://github.com/rancher/rancher/pull/42252

Validation Template
Root Cause
We had no way of setting the trace-level for rancher-agent .
What was fixed, or what change have occurred
Now is possible to to add trace-level using   Agent Environment Vars.
CATTLE_TRACE or RANCHER_TRACE must be  set to true
Areas or cases that should be tested

Create a new cluster with the Envs set.

What areas could experience regressions
None.
Are the repro steps accurate/minimal?
Create new cluster setting  CATTLE_TRACE or RANCHER_TRACE  to true.
run kubectl logs {podname}  -n cattle-system Trace and debug level logs must appear.


schema dbmirror2 does not exist

New install using:  "pipx install git+https://github.com/acoustid/mbslave.git"
database imports fine using: "mbslave init --create-database"
running command "mbslave sync" fails with the following:
INFO:mbslave.replication:Downloading https://metabrainz.org/api/musicbrainz/replication-170231-v2.tar.bz2?token=***
INFO:mbslave.replication:Packet was produced at 2024-08-21 01:05:40.896471+00
Traceback (most recent call last):
File "/root/.local/bin/mbslave", line 8, in
sys.exit(main())
File "/root/.local/share/pipx/venvs/mbslave/lib64/python3.9/site-packages/mbslave/replication.py", line 801, in main
args.func(config, args)
File "/root/.local/share/pipx/venvs/mbslave/lib64/python3.9/site-packages/mbslave/replication.py", line 516, in mbslave_sync_main
process_tar(packet, db, config, ignored_schemas, ignored_tables, schema_seq, replication_seq, hook)
File "/root/.local/share/pipx/venvs/mbslave/lib64/python3.9/site-packages/mbslave/replication.py", line 460, in process_tar
importer.load_pending_data(member_file)
File "/root/.local/share/pipx/venvs/mbslave/lib64/python3.9/site-packages/mbslave/replication.py", line 355, in load_pending_data
cursor.execute('TRUNCATE dbmirror2.pending_data')
psycopg2.errors.InvalidSchemaName: schema "dbmirror2" does not exist

I've just merged https://github.com/acoustid/mbslave/pull/8 which should fix this.


pin psutil version to >=2.2.1,<3.0

There is a backwards incompatibility issue with psutil >=3.0, which is what pip installs by default.  While this is probably just a stop-gap measure, pinning the version as such avoids the following failure case:
root@3f9f6155ac7b:/opt/gantryd# pip list | grep psutil # 3.* fails
psutil (3.2.1)

root@3f9f6155ac7b:/opt/gantryd# ./gantry.py /etc/gantry/gantry.conf update test
[test] Starting container f09d2fcbf913
[test] Waiting for health checks...
[test] Running health check: http
Checking HTTP address in container f09d2fcbf913: http://<IP_ADDRESS>:3333
No handlers could be found for logger "health.healthcheck"
[test] Health check failed
[test] Sleeping 3 second(s)...
[test] Running health check: http
Checking HTTP address in container f09d2fcbf913: http://<IP_ADDRESS>:3333
[test] Redirecting traffic to new container
Finding running containers...
Updating proxy...
[test] Terminating container: 8af203b1f02c
10/09/2015 04:08PM | Monitor check started
[test] Sending 0 termination signals
[test] Waiting for 1 termination checks
[test] Running termination check: connection
Traceback (most recent call last):
  File "./gantry.py", line 105, in <module>
    run()
  File "./gantry.py", line 102, in run
    cleanup_monitor(None, None)
  File "./gantry.py", line 96, in cleanup_monitor
    manager.join()
  File "/opt/gantryd/runtime/manager.py", line 225, in join
    self.monitor_futures.get().get()
  File "/usr/lib/python2.7/multiprocessing/pool.py", line 558, in get
    raise self._value
AttributeError: 'Process' object has no attribute 'get_connections'

root@3f9f6155ac7b:/opt/gantryd# pip uninstall -q psutil
root@3f9f6155ac7b:/opt/gantryd# pip install -q "psutil>=2.2.1,<3.0" # downgraded
Proceed (y/n)? y

root@3f9f6155ac7b:/opt/gantryd# ./gantry.py /etc/gantry/gantry.conf update test
[test] Starting container 44804fefacfa
[test] Waiting for health checks...
[test] Running health check: http
Checking HTTP address in container 44804fefacfa: http://<IP_ADDRESS>:3333
No handlers could be found for logger "health.healthcheck"
[test] Health check failed
[test] Sleeping 3 second(s)...
[test] Running health check: http
Checking HTTP address in container 44804fefacfa: http://<IP_ADDRESS>:3333
[test] Redirecting traffic to new container
Finding running containers...
Updating proxy...
[test] Terminating container: f09d2fcbf913
10/09/2015 04:15PM | Monitor check started
[test] Sending 0 termination signals
[test] Waiting for 1 termination checks
[test] Running termination check: connection
Container has no remaining connections: f09d2fcbf913
10/09/2015 04:15PM | Monitor check finished
10/09/2015 04:15PM | Shutting down container: f09d2fcbf913

LGTM


fix(checks): handle file: and multi: in AVD-DS-0011

The reverse engineered Dockerfile of an image doesn't exactly match
the original Dockerfile. For example, it doesn't have the original
source files names. Instead, it uses file:<hash> in:
COPY file:8b8864b3e02a33a579dc216fd51b28a6047bc8eeaa03045b258980fe0cf7fcb3 in /__cacert_entrypoint.sh
Such commands should not trigger AVD-DS-0011.

I've never used rego before and I'm having trouble grokking it and I'm very confident that the tests aren't actually running when I run make test so I doubt this works and I can't test it... but hopefully it's at least a good starting point and someone can help me complete it :)

You are right, the tests are in the process of being brought over to this repo. We'll take a look at your PRs once we merge the tests in to ensure nothing breaks.

@simar7 can you please help me progress this MR?
I'm very eager to eliminate this false positive.

@nikpivkin could you take a look?

Hi @candrews !
Can you make the same recommendations as in this comment on this PR?

Hi @candrews !
Can you make the same recommendations as in this comment on this PR?

Done!
Thank you very much for your help!


feat(operate): propagate global flag to Operate...

to enable/disable multi-tenancy
Which problem does the PR fix?

What's in this PR?

Checklist
Please make sure to follow our Contributing Guide.

Before opening the PR:

[x] There is no other open pull request for the same update/change.
[x] The commits follow our Commit Guidelines.
[x] Tests for charts are added (if needed).
[x] The main Helm chart and sub-chart are updated (if needed).
[x] In-repo documentation are updated (if needed).

After opening the PR:

[x] Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
[x] Did all checks/tests pass in the PR?

@sdorokhova thanks for your contribution :bow:


Fixes for various issues against 2015.12

Issue #666
Print meaningful error message when unable to create /var/run/snabb:
$ ./snabb
core/shm.lua:111: Unable to create /var/run/snabb: Permission denied
stack traceback:
	core/main.lua:126: in function <core/main.lua:124>
	[C]: in function 'assert'
	core/shm.lua:111: in function 'mkdir'
	core/shm.lua:84: in function 'map'
	core/counter.lua:48: in function 'open'
	core/app.lua:28: in main chunk
	[C]: in function 'require'
	core/main.lua:116: in function 'initialize'
	core/main.lua:35: in function <core/main.lua:32>
	[C]: in function 'xpcall'
	core/main.lua:155: in main chunk
	[C]: at 0x0044fa80
	[C]: in function 'pcall'
	core/startup.lua:1: in main chunk
	[C]: in function 'require'
	[string "require "core.startup""]:1: in main chunk

Issue #658

Print useful error message when unable to open PCI device:

(When the target device of device_info does not exist)
$ sudo SNABB_PCI0="0000:05:00.0" SNABB_PCI1="0000:05:00.1" ./snabb snsh -t apps.intel.intel_app
selftest: intel_app
lib/hardware/pci.lua:38: No such device: 0000:05:00.0
stack traceback:
	core/main.lua:126: in function <core/main.lua:124>
	[C]: in function 'assert'
	lib/hardware/pci.lua:38: in function 'device_info'
	apps/intel/intel_app.lua:150: in function 'selftest'
	program/snsh/snsh.lua:29: in function <program/snsh/snsh.lua:29>
	core/lib.lua:405: in function 'dogetopt'
	program/snsh/snsh.lua:63: in function 'run'
	core/main.lua:58: in function <core/main.lua:32>
	[C]: in function 'xpcall'
	core/main.lua:155: in main chunk
	[C]: at 0x0044fa80
	[C]: in function 'pcall'
	core/startup.lua:1: in main chunk
	[C]: in function 'require'
	[string "require "core.startup""]:1: in main chunk

(When the selected device exists but is not suitable)
$ sudo SNABB_PCI0="0000:04:00.0" SNABB_PCI1="0000:04:00.1" ./snabb snsh -t apps.intel.intel_app
selftest: intel_app
SNABB_PCI_INTEL[0|1]/SNABB_PCI[0|1] not set or not suitable.

Make packetblaster bail out if no suitable devices are selected:

$ sudo ./snabb packetblaster replay program/snabbnfv/test_fixtures/pcap/64.pcap 0000:05:00.0
program/packetblaster/packetblaster.lua:49: <PCI> matches no suitable devices.
stack traceback:
	core/main.lua:126: in function <core/main.lua:124>
	[C]: in function 'assert'
	program/packetblaster/packetblaster.lua:49: in function 'run'
	core/main.lua:58: in function <core/main.lua:32>
	[C]: in function 'xpcall'
	core/main.lua:155: in main chunk
	[C]: at 0x0044fa80
	[C]: in function 'pcall'
	core/startup.lua:1: in main chunk
	[C]: in function 'require'
	[string "require "core.startup""]:1: in main chunk

Issue #579
Make snabbnfv traffic warn when the configuration file does not exist
initially:
BINARY    2.0M snabb
$ sudo ./snabb snabbnfv traffic 0000:03:00.1 foobar baz
snabbnfv traffic starting
WARNING: File 'foobar' does not exist.
load: time: 1.00s  fps: 0         fpGbps: 0.000 fpb: 0   bpp: -    sleep: 100 us
[...]

Great! Merged onto next, sorry about the wait.
What do you think about a future policy of suppressing stacktraces for well-defined detected errors? (Showing them only for unexpected errors caused by bugs.) For example with a main.fatal(msg...) function to print and terminate. That would seem friendly to end-users.

Suppressing stack traces for configuration/setup errors and the like would be great.

In that case I would like to retain the ability to always print stack traces (a debug option or similar). I understand that end users might be put off by stack traces but I think developers profit from seeing the guts no matter how trivial the error.


Feature/domain management

Hey
Here is an update that uses / leverages the updates to sitehostnz/gosh to add in DNS management to the terraform provider
Cheers

Hi @yakmoose,
Thank you for contributing!!!
Similar to the GoSH PR, I will merge it into our dev branch and update the code once we have updated GoSH.
Cheers,
Tommy


[Gssoc23] prime number detector

Describe the project you want to add with tech stack
want to add a program in C (C_projects) to detect if input number is prime or not

Expected behavior
prints out the factors of the input number

Screenshots (optional)

Additional context (optional )

@MohitGupta121 Please assign me this issue.


Multiple inheritance in C#

I want to use multiple inheritance, is there a way to use it directly without jumping in garden?
If not, why does .net not support it?

.net doesn't support multiple inheritance to avoid the "deadly diamond of death". Use interfaces.


Extend Cycle concept

Description
We should be able to make cycles start date be different from the default one.
Examples:
I want my Daily cycles to start at 8AM and not 12PM.
I want to have a business day frequency.
I want my monthly cycles to start on the 5th day of the month.
I want a business month frequency.
etc.
This link could help to understand the need.

Should be driven by a real use case.


May I ask how many resources and time are needed for training?

@dflkasjfh  SAMURAI is a training-free method. The checkpoints we used are directly from SAM 2, you can find related details in the appendix D of their paper.


Sending transaction via QUIC (TimedOut)

Couldn't send transaction in mainnet via QUIC tpu connection.
The most interesting thing is that sometimes the transaction is successful.
2024-05-16T23:09:42.127766Z ERROR quic::quic_manager: Attempt 1: Failed to send transaction via QUIC: Custom(
    "\"ConnectionError(TimedOut)\"",
)

BUT in devnet - all works good.
Code available here
https://github.com/solana-labs/solana/issues/35559

Some receivers are just blocking connections from other validators

https://github.com/RAprogramm/solana-quic-client


all: Add a length variable to blas64.N

Alongside, fix the implementation of mat.VecDense and mat.Diagonal, as well as other changes needed to fix this change.
Fixes #736.
Please take a look.

When you squash, I think this should be "blas64,mat: add length field N to blas64.Vector" or something similar (even "blas64: add length field N to Vector" given that the flow on to mat is obvious).


get signed URLs for GCS objects

fixes #251

@ChengyuanZhao Also, I noticed that this PR does not add support for creating signed URLs for buckets. One way to fix it would be to just go back to having a single GoogleStorageResource class and represent a GCS resource and deal with bucket vs. object internally, as I suggested previously.

My understanding of #251 was that when a blob is created, we should return (e.g., by logging) the GCS URL to users. Although now that I'm re-reading #251 I'm not totally sure what it's about.
Maybe I'd just add a log statement to GoogleStorageResourceObject#createBlob() logging the URL of the newly created object (same for buckets).

On a second thought, I may have misinterpreted the meaning of signed URL.
So maybe disregard my comment about logging the signed URL on blob creation.

@meltsufin
I must say I feel completely the opposite. We had a previous PR that exposes setting metadata, which only makes sense for an object ( not a bucket), and that was elegantly handled by having the two separate types.
Also ,the function for getting signed url is from Storage, and only works for objects, not buckets, so again I feel our current setup with 2 objects is very fitting. I definitely don't think we should be implementing our own url building functionn for buckets especially since Storage only provides one for objects. not buckets.

@meltsufin
Ultimately it's because while a gs://bucket and gs://bucket/object path look very similar to a user, they represent very different things and have different functionality even from the user perspective, so I really don't think we should be hiding both behind a single type and instead warning the user at runtime that one operation is supported or not. For example, setting meta data or even getting a write stream.

I'm also OK with having the two types vs single type and internal ifs (and I've read something on this recently in Effective Java where multiple types are encouraged vs internal ifs, Item 20 - Prefer class hierarchies to tagged classes).
In this case, this comes at the price of having a duplicated createSignedUrl(), which is OK for now. If duplication grows, we might think of doing something else.

If it's true that you can't create a signed URL for a bucket, then this makes total sense and two classes are good.


Add build matrix for MPI

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

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

This is a work in progress and should not be merged until after #20.

@conda-forge-admin, please rerender

@vijaysm, could you take a quick look at the failing test in CI? It seems like one of the Fortran tests isn't compiling.  Is there an environment variable I'm not defining that is needed?

Anyone have suggestions for checking that moab with MPI works?  Keeping in mind it has to work in CI so something trivial?

The built in tests include some that are only invoked for parallel builds - is that enough?

Yes, I think that's enough. Just wanted to make sure there are some tests to make sure it's not failing.
Which it apparently still is in a few cases :-(

Looks like only OpenMPI is failing - can't remember if it's officially supported?

It seems like the problem is missing ssh.  I'll see if adding openssh helps.

@vijaysm, I made your suggested change.  Pointing to the MPI directory using --with-mpi wasn't enough on its own, that's why I needed to define the compilers as well.  Maybe this is because the CI configurations still point to the non-MPI compilers, as you pointed out.
I believe this is ready to merge, once CI has completed successfully again.  Let me know if further changes are desired.

Maybe @vijaysm can sign off on this before he disappears on vacation and I'll merge...

I like the changes in this PR @gonuke The CI builds are all passing cleanly and I don't have any comments at the moment that would stall the merge. So I'm approving the PR as well, but leave it up to you (given my shortage in time) to merge when the documentation updates are in place.

Typically, the feedstock is not considered the right place to document usage. Few users will land here and typically only when something has gone badly wrong with the conda package(s). The right place for the documtation you're requesting is the main moab docs, which I don't have time or interest to get involved in at this level. I'm happy to edit/ review changes either of you want to make.

@conda-forge-admin, please rerender

@gonuke, a rerendering of the feedstock just deleted the text you added again.  I don't think you can edit the README.md and have it stick.  The only files you can edit are the ones in the recipe directory.

Thanks @xylar

Thank you both for the review.


Update Glyphicon to version 1.9.2

@glebm Glyphicon was updated to 1.9.2 on Dec 15, 2015, the assets/stylesheets/bootstrap/_glyphicons.scss isn't updated since Nov 24, 2015

@glebm I could do this Glyphicon update if you give me the directions :smile:


Mark ONNX-MXNet experimental

Description
mark onnx-mxnet as experimental in release notes.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] Changes are complete (i.e. I finished coding on this PR)
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html

Comments

@anirudh2290 @zheng-da

@eric-haibin-lin can this be merged ?


Allow specifying Hasher used by std:collections::HashMap

The default std::collections::HashMap uses SipHasher, which is quite slow. For higher performance, one might want to use FxHasher, e.g. as provided by the following crate: https://docs.rs/rustc-hash/latest/rustc_hash/.
Currently, there is no way to specify which Hasher to use in memoization.

This is a good idea. I will attempt an implementation in the next couple days, as time permits.

AHash is even faster and I'd like to use it with memoize.
@NickHu It's also interesting to read the criticisms of FxHash: https://github.com/tkaitchuck/aHash/blob/master/compare/readme.md#fxhash
So yeah, it would be great if users of the crate could choose the hasher they want to use :)

I am aware of AHash, however the last time I looked at it it had significant limitations on a platform that isn't x86 with hardware AES instructions (e.g. wasm), which would be my primary use case. It was either a hassle to compile or slower than FxHash at the time, or possibly even something else, I don't remember which.
In either case, I think a generic implementation is the correct thing to do.

Folks, keep in mind -- this crate is supposed to speed up a slow function by 100x through storing its result. At that point, a few nanoseconds on the hash function don't matter that much anymore IMO. It is not the best solution for high performance or general purpose hash tables, in which case one should simply use a hashmap of one's choice directly.
That being said, of course I accept PRs for such features :)

I'll try to implement this.

Only @dermesser can release a new crate version. Also, thanks! :D


use-cases: use 'dvc cache dir' command in shared dev server

In the shared server use case https://dvc.org/doc/use-cases/shared-development-server we set up the shared cache by changing config dvc config cache.dir /path/to/dvc-cache while a separate command was introduced dvc cache dir /path/to/dvc-cache.
I'm trying to understand the motivation behind the command dvc cache dir... It feels like we should deprecate the command dvc cache dir or change the docs.
@efiop could you please explain the motivation behind the command?
UPDATE: also probably we should update dvc cache dir doc as well - it's not that clear from the doc (from https://github.com/iterative/dvc.org/issues/986#issuecomment-586541617)

The doc is just outdated.
There are two things behind this command:
https://github.com/iterative/dvc/issues/1284
https://github.com/iterative/dvc/issues/1421#issue-388849219
https://github.com/iterative/dvc/pull/1467
(didn't find specific discussion re the dvc cache dir itself, but the point is - we want to keep dvc config w/o any magic, w/o any transformations, while higher level commands like dvc cache dir provide some use case specific, higher level interface)

@shcheklein thanks for the clarification.

we want to keep dvc config w/o any magic, w/o any transformations, while higher level commands like

It makes sense. The current dvc cache dir still seems to provide very little value for a dedicated command but it can be extended in the future. If we follow this logic dvc config cache.shared group should be also replaced to something higher level.
So, we can keep this issue to update the docs or close it if duplication. @jorgeorpinel you should know better.

Sure, I'll look into updating the docs and close this then. Thanks

The doc is just outdated.

Why is https://dvc.org/doc/use-cases/shared-development-server outdated @shcheklein ? For an external cache you would probably prefer to use dvc config cache.dir instead of dvc cache dir because its easier to use the absolute path than a path relative to the project.

https://dvc.org/doc/command-reference/cache/dir#example-using-absolute-path - Absolute path /path/to/dir saved as is. (Probably we should update dvc cache dir doc as well - it's not that clear from the doc).
If we have two APIs and one is made to be user friendly why not advertise it?
The only value that comes to mind to keep the low level config manipulation is that we already have a lot of low level commands in the same doc (like group related, protected modes, etc).
It brings a good point if we don't have even a single place to recommend dvc cache dir in docs - why do we have it indeed. I understand and clearly remember the motivation behind it, but if we don't even recommend it in the doc that deals with changing cache location in the most common pattern, why bother?
(^^ this is not me trying to defend the dvc cache dir - it's a question that came to my mind, want to recalibrate my understanding of this command)

Good points @shcheklein and sorry for taking so long to reply.

The only value that comes to mind to keep the low level config manipulation is that we already have a lot of low level commands in the same doc (like group related, protected modes, etc).

I think this is the reason to keep it as is: keeping the sequence of commands to a minimum complexity.

It brings a good point if we don't have even a single place to recommend dvc cache dir in docs - why do we have it

It's mentioned in a few docs (4). Not strongly, but more than dvc config cache.dir (just in this use case) at least... Perhaps we should first try to find more places to mention/link to it and if not, consider facing it out?

Also dvc cache dir has some magic sprinkled on it, it resolves your relpaths so they work with the location that the config is in. I'm sure you guys remember that issue from last year, where we've introduced that logic for local remotes because dvc cache dir ../path in repo root should be resolved to ../../path, because the config is in .dvc/config.

Do we have to modify the docs in this section of the shared development server according to the cache dir documentation?

Hi @PseudoNerd thanks for finding this issue in the context of GSoD. Yes, that is correct. Also please note

Probably we should update dvc cache dir doc as well - it's not that clear from the doc
(from https://github.com/iterative/dvc.org/issues/986#issuecomment-586541617)

and

Perhaps we should first try to find more places to mention/link to it (dvc cache dir)
from https://github.com/iterative/dvc.org/issues/986#issuecomment-594342255

It would be great to also try to include these ☝ in the PR. Please lmk if you have any further questions.

It's only logical to have references to the dvc cache dir link, I'll go through relevant docs and find suitable places for the same and make a PR regarding this issue.


Fix #1795

thanks merged.

@vsn4ik should we also remove the sourceMap options in the less build step?

@acrobat for development is not necessary to remove. I deleted sourceMap only for production.

So this will generate the sourceMaps but don't link to them in the minified css files, if I understand correctly?

All right.


Need ADD for Shopcarts

As a Customer
I want to add products to my shopcart,
So that I can select the items I wish to purchase and view them later.
Details and Assumptions

We have the required machine and will use PostgreSQL
Performance is not a priority at this stage

Acceptance Criteria
Given a connection attempt to the DB
Then the connection is successful

Given a command to create a new table
When no table with the same name exists
Then the table is created

Given a command to create a new table
When a table with the same name already exists
Then an error message is returned

Given a valid SQL command on the tables
Then the correct result is returned


Router login page

Hi, that is great script thanks for it 👍
I have interesting problem.
When i take script and run it it works. It changed my  domain ip on cloudflare. But when i am triying to visit my web site. It sends me to the Router page after scripts run.
My site: altanalaybeyoglu.com

i think it is about status section on cloudflare. when scripts run status screen is being gray


[MM-37670] System console about modal missing header border

Summary
Fixes an issue where the header border in the System Console's About Mattermost modal was faded and difficult to see.
Ticket Link
https://mattermost.atlassian.net/browse/MM-37670
Related Pull Requests
None
Screenshots
Before fix:

After fix:

Release Note
Fixes About Mattermost modal styling

/cherry-pick cloud

Cherry pick is scheduled.


Cannot debug binary of godot(from scratch version)

Edit: template fill
Godot version:
Tried both master branch and 3.2 branch.
OS/device including version:
Windows 10
Issue description:
Cannot debug. Whenever I attach the debugger at launch it throws this exception:

Exception has occurred: W32/0xc0000008
Unhandled exception at 0x00007FFC430EEC9A (ntdll.dll) in godot.windows.tools.64.exe: 0xC0000008: An invalid handle was specified.

This is what the devenv configuration and the exception's call stack look like in a screenshot: ss.
Steps to reproduce:
Clone, add the .vscode folder, run debug.
Minimal reproduction project:
vscode.zip
Drop in project root for vscode to pick it up. (name it a .vscode folder)
Thanks in advance.

cc @bruvzg, as they implemented WinTab API support.

Please fill the issue template which was present when you created the issue.

Is it crashing when started normally or only when attaching VS debugger?
This error does not make much sense, LoadLibraryW should load library or return NULL.
Probably you have broken WinTab driver library installed (or it has invalid debug symbols, which debugger fails to load).

Is it crashing when started normally or only when attaching VS debugger?
This error does not make much sense, LoadLibraryW should load library or return NULL.
Probably you have broken WinTab driver library installed (or it has invalid debug symbols, which debugger fails to load).

Only with debugger.
I did install a driver for my tablet.

Perhaps a broken wintab32.dll. And a small update: works if hit continue through the debugging. No need to be that alarmed.

Update: I hit continue, it doesn't crash Godot but it won't hit any breakpoints so I'm guessing the gdb hook didn't work.


Getting error when running docker conatienr -> validate() got an unexpected keyword argument 'extra_validators'

Hi Team,
we are trying to install the application on Ubuntu machine . we have followed the same procedure mentioned in the document .when we are running the docker image make using command "docker-run-superset" than getting below error.

Can you please guide what we have to do to resolve this error .
Regards,
Akash

@akash-goel I'm unable to replicate. What version of ubuntu are you running? by default I am running this in 20.04

I get the same issue.
I don't think ubuntu is relevant here, as I'm also running in a docker environment, and the error is the same.
"meltano.yml->utilities->superset" pins the versions of many libraries, but for superset it has a >=1.5.0, which afaik would allow pip to pick a different major version. I tried to change to ~= but didn't work either, but I'm afraid I didn't cleanup the env correctly.
I'll try later this week when I've some time, but so far that's what I got.

Oh yeah, superset might be on a new version. I can pin to 2.0. or to 1.6.2. let me take a look at this stuff.

Intereting, I'm getting a 500 when running in the dev container (after authentication).

It looks like 2.0.1 released on pypl in December, so it is possible that broke it. I'm pinning it to 2.0.0 and will see if that fixes it.

So nothing to do with superset, I'm going to work back through the dependency chain on this one. paging @gregwdata since you pinned the dependencies on superset originally. It looks like something changed w/Flask.

Looks like we are seeing this issue: https://github.com/wtforms/flask-wtf/issues/547
Quick fix I'll look at is pinning an older version of flask-WTF, but longer term may need to look at how validate gets invoked on the superset login page and get it fixed there.

Yep that worked! Thanks Greg!!


[BESU-176] JRPC response to eth_getTransactionCount is different from other node implementations.

PR description
Impact
Someone is using their BESU node as they use the other implementations and following good practices, their transactions will eventually get stuck, while in the other implementations this wouldn´t happen.
How it works
When asking for transaction count with the param ‘pending’, Besu returns the latest pending transaction nonce + 1, other implementations return the next user nonce non used by the pending transactions.
This means that if a transaction with a future nonce reaches a Besu node (Sending many nonce requests, latency issues, dropped transaction, planning future transactions), all the responses to getTransactionCount will return future nonces.
This means that none of the transactions are mined, and after a short while they are rejected by other nodes.
In other node implementations, getTransactionCount would return the missing nonce, enabling the execution of all broadcasted transactions.
Fixed Issue(s)

fixes https://jira.hyperledger.org/browse/BESU-176
Signed-off-by: Abdelhamid Bakhta<EMAIL_ADDRESS>

Feels kinda bad to find the bug, create tests, make a report on it, and just having someone copy the issue you created word by word and not credit you at all as the original reporter...
https://jira.hyperledger.org/browse/BESU-176
Thanks for the fix thou

Feels kinda bad to find the bug, create tests, make a report on it, and just having someone copy the issue you created word by word and not credit you at all as the original reporter...
https://jira.hyperledger.org/browse/BESU-176
Thanks for the fix thou

Sorry, indeed you are right. It was not intentional. I will update the description of the pull request for tracking record. Thanks for this valuable work.

@kikoncuo Sorry about this slip up on our part. Your contribution was ideal, lots of details and it really let us understand the problem.
Send me your address and I can get some Besu stickers to you. PM me at https://twitter.com/felipefaraggi


BUG: keybinding values undergo globbing, may break mappings

I have configured the copycat search key to '?' (set -g @copycat_search '?' in .tmux.conf), and all of a sudden the mapping was assigned to prefix + 6 instead of prefix + ?. Turned out I had a file ~/6, and the ? mistakenly is expanded to 6.
The problem is the unquoted variable reference "for key in $key_bindings". As we need the word splitting for the (undocumented?) feature of allowing multiple key mappings, it cannot be quoted.
As a fix, use read with a here-string to perform word splitting in an array variable, and then iterate over the array elements.

Thanks for the fix.
I'd be totally ok with removing the "feature" of multiple key mappings to reduce the complexity (and guessing) for whoever is reading the code. Just open the PR if think that would be a good idea. Thanks.

Thank you for accepting this so quickly (and for the great plugin)!
If the problem were spread among more places, I'd also be in favor of dropping the feature (or factoring this out). As it stands, why not keep it in; maybe someone really uses multiple key mappings for convenient search?!


XML source

Hello, and congratulation on the amazing job!
Is it possible to use a .xml file as a source data for autocompletion?
Luca

Hello @lguariento,
Many thanks I'm glad you like it! :)
Unfortunately, there is no XML support out of the box. But still you can covert the XML data source into JSON and plug it in.
Hope that's helpful.

Thanks anyway - if, say, someone is working with a native XML database, then what you suggest is out of the question.


Driver Hub at 6.2.2 gives warning when connected to FtcRobotController built from 6.2

The Driver Hub from REV is at Driver Station 6.2.2
This builds a robot controller to version 6.2
So the Driver Hub gives a warning that the two don't match.   I am concerned about the students learning to ignore that warning...
Is there any way you could release a robot controller 6.2.2.  (or a version of both of them that is larger)
Thanks in advance,
Alan
Coach FTC #16072, Quantum Quacks

This is a known issue.  The mismatch is harmless from a functional perspective, but a new APK for the Driver Hub was necessary, and there was no reason to release a new robot controller, so here we sit.  When v7.0 is released for Freight Frenzy this will go away.

I would humbly suggest that not training students to ignore warning messages is a reason to release a new robot controller. But I understand that with limited resources that might not be enough of a reason.

@AlecHub - Thanks but since we only have one driver hub then we'll have the same problem when a regular driver station tries to connect to our robot controller since we can't rebuild the driver station....  (except that will be 6.2 trying to connect to 6.2.2)

I suggest that Version X.Y.Z1 of RC/DS/DH be compatible with Version X.Y.Z2 of RC/DS/DH, and vice-versa, by design.  No version mismatch warnings should be displayed if first two numbers (X.Y) of the versions match.

@AlecHub I agree and thought about that as I composed my initial response.   Filter version mismatch by major.minor, disregard point releases.  We'll put it on the list to discuss with the tech team.

@AlecHub That sounds like a great suggestion that would solve the problem in the future.

7.0 is out now, and implements @AlecHub's suggestion.


Incorrect output of prediction start date

Description
I am not getting the anticipated prediction start date. My dataset ends at 07-10-2019, and I am trying to predict for the next couple of days. However, my prediction starts at a much later date, specifically: 10-26-2021
To Reproduce
Please try executing the following stand-alone code as a minimal working example of my problem:
import pandas as pd
from gluonts.dataset.common import ListDataset
from gluonts.model.simple_feedforward import SimpleFeedForwardEstimator
from gluonts.trainer import Trainer
from pathlib import Path

# CSV File is available here: "https://gist.github.com/sayonsom/af6604bb38d50e366e53d1ccc994b269"
csv_url = 'Text_example.csv'
df = pd.read_csv(csv_url, index_col=None, names=['Timestamp', 'measured_value'], parse_dates=False)

prediction_length = 96
freq = "15min"
start = pd.Timestamp(df['Timestamp'][0], freq=freq)
target = df['measured_value'].to_numpy()

# train dataset: cut the last window of length "prediction_length", add "target" and "start" fields
train_ds = ListDataset([{'target': target[:-prediction_length], 'start':start}], freq=freq)
# test dataset: use the whole dataset, add "target" and "start" fields
test_ds = ListDataset([{'target': target[-prediction_length:], 'start': start}], freq=freq)

estimator = SimpleFeedForwardEstimator(
    num_hidden_dimensions=[10],
    context_length=96*3, # input window length, 3 days data
    prediction_length=prediction_length,
    freq=freq,
    trainer=Trainer(ctx="cpu", epochs=4, learning_rate=1E-3, hybridize=True, num_batches_per_epoch=200),
)

predictor = estimator.train(train_ds)

target = df['measured_value'].to_numpy()
test_ds = ListDataset([{'target': target, 'start': '2019-04-11 00:00:00'}], freq=freq)

predictions = list(predictor.predict(test_ds))

known_data = ListDataset([{'target': target, 'start': '2017-04-27 00:00:00'}], freq=freq)
print(f"Dimension of samples: {predictions[0].samples.shape}")
print(f"Start date of the forecast window: {predictions[0].start_date}")

Error message or code output
...
INFO:root:Using CPU
INFO:root:Using CPU
Dimension of samples: (100, 96)
Start date of the forecast window: 2021-10-26 15:00:00

(Expecting 07-11-2019, instead)
Environment

Operating system: macOS Catalina 10.15.3
Python version: 3.7.4
GluonTS version:0.4.2

(Expecting 07-11-2019, instead)

@sayonsom why are you expecting that start date? Running the following
import pandas as pd
csv_url = "https://gist.githubusercontent.com/sayonsom/af6604bb38d50e366e53d1ccc994b269/raw/7fa603a4618286aa4376556043627e9383d27117/Test_example.csv"
df = pd.read_csv(csv_url, index_col=None, names=['Timestamp', 'measured_value'], parse_dates=False)
freq = "15min"
# This is the data you fed into the predictor
target = df['measured_value'].to_numpy()
# This is the start date of the data you fed into the predictor
start = pd.Timestamp('2019-04-11 00:00:00', freq=freq)

print(start + len(target) * start.freq)

yields 2020-03-31 07:00:00

@lostella Thanks for clarifying. I was not able to comprehend how to define a "begin forecast from this date onwards" from the documentation. I will close this issue.

I see, happy if that helps. Just to clarify: is that the forecast start date that you get? If not, there might still be an issue... I couldn’t yet run your full example, will do that to double check though 👍🏻


Hotfixes to blog

Refactors the blog component

expands data option
reduces number of examples blog post to two
adds additional author metadata

Additionally, this commit contains moment.js library in action. Also includes lodash library.

This PR resolves #67


practice pull request

Practice

Unfortunately you deleted the repository and I can no longer resolve the conflicts. Closing PR.


Spatialite integration

Hello!
We are trying to load a sqlite database that have tables with GEOMETRY field.
Is there any way to load this field with spatialite module or transform it to other that capacitor sqlite plugin can read it?
Let me know if you need more information about it, and sorry for the inconveniences.
Thanks, Kartano.

@KartanoDev thanks for using the plugin i already answer that one issue#45. If you can provide a wrapper specification from spatialite to capacitor sqlite plugin and give me tables schema and data with GEOMETRY field. this wrapper should use capacitor sqlite to store and make the link to spatialite.

Hello @jepiqueau, we don't have any  wrapper specification from spatialite to capacitor sqlite plugin.
This is the schema for one of the tables:

@KartanoDev Can you give me an example of data  corresponding to that schema and how is the GEOMETRY data type?

@KartanoDev is the Geopoly sqlite3 extension (https://www.sqlite.org/geopoly.html) can be used? is it the same?

@jepiqueau I need to transform this GEOMETRY field to AsGeoJson like this:

@KartanoDev Is it something that you would like to do or not. How to you get the Geometry data from? which software. you have to give me a bit of your scenario so i can understand. Is spatialite an app install on Windows or Mac ...

@KartanoDev What is the tool you are using to  show me the data

@jepiqueau We want to display data on a map. For this we need to have transform GEOMETRY field to AsGeoJson type like:
"{"type":"Point","coordinates":[-72.98352140181473,41.25610730677585]}"
To transform this we are using spatialite_gui in Windows:
https://www.gaia-gis.it/fossil/spatialite_gui/index

@KartanoDev for me the complete scenario step by step is not clear Could you describe it
it seems somehow that you first enter all your GEOMETRY in spatialite_gui so if it is the case where do you use sqlite and why would you like to store GEOMETRY, we could store AsGeoJson type using the sqlite Json1 extension?

@KartanoDev No news from you so i close the issue


Snackbar is hidden behind RecylerView if there are enough items to allow to for scrolling

I’m using the snackbar to notify when an item has been deleted from the recyclerview list. I’m using: https://github.com/krossovochkin/Android-SwipeToDismiss-RecyclerView
My snackbar displays fine if there are too few items for the recyclerview to scroll, but once the list is surpasses the height of the screen, the snackbar is hidden behind the recyclerview.
In the image, I’ve just started swiping at the bottom of the screen, and I can see the snackbar appear.

This is in 5.0.

Can you post the code you are using to display the snackbar?

I've got this issue in our application too, and after much experimentation I got a test app that shows the problem: https://github.com/jonathan-caryl/SnackbarIssue109.
It looks like it's happening when the android.support.v4.widget.DrawerLayout is given an elevation, which means it's getting drawn over the Snackbar.

It's worth noting that the problem doesn't show up when you run on a pre-Lollipop version of Android, where Views have no elevation.

I've made a PR with a fix in, I hope it's suitable.

 :)


[Bug]Malicious validators can prevent legitimate transactions from being executed.

https://hackerone.com/reports/2498849
Summary:
Malicious validators can prevent legitimate transactions from being executed.
Proof-of-Concept (PoC)

When a malicious validator receives any transaction, it can create a FakeTransaction using the following method: FakeTransaction = Transaction + sample_transiton().
FakeTransaction and Transaction have different TransactionIDs but share the same TransitionIDs, InputIDs, OutputIDs, and TPK. According to the current logic in the code, if the FakeTransaction is sorted before the Transaction in the DAG, it will cause the Transaction to be aborted.
Malicious validators broadcast the FakeTransaction externally via WorkerPing, potentially causing honest validator nodes to prioritize the FakeTransaction over the Transaction.
Furthermore, validators can fabricate FakeTransactions for uncommitted transactions upon proposing each proposal. If a malicious validator is chosen as the leader, it will result in all transactions in the block being aborted.

Impact
This attack can lead to a large number of legitimate transactions being aborted, and the abort_transaction_id will be recorded in the ledger, requiring users to regenerate transactions.

This looks like a (low priority) issue to me as well. Trying to phrase it in my own words: we abort on a malleable part of the Execution, before verifying integrity of the full Execution.
Fortunately, deployments don't have the partial malleability issue, so a quickfix can be to only call early should_abort_transaction on deployments: https://github.com/AleoHQ/snarkVM/pull/2452
I'll keep thinking about the range of issues and more comprehensive solutions. Similar attacks may be possible if a malicious client or validator frontruns a transaction, creating the same txid but replacing the proof with a false one.

After additional consideration, we have decided to only partially mitigate this for now and prioritize DoS prevention. Slashing can likely properly mitigate this censorship issue in the future.

IIRC we weren't considering any slashing methods; has anything changed since then?

IIRC we weren't considering any slashing methods; has anything changed since then?

I view it as likely to be essential to add post-mainnet launch.

Doesn't sound like we'd block mainnet for this.  Marking as such.  Feel free to argue.


 Unfortunately that will be quite hard to fix because I’m not a font designer — all I’ve done here is repackage what was in Edward Tufte’s original fonts, and they didn’t include most of them. I’d especially love IPA support myself, but it’s unlikely to happen.
In the meanwhile I can point you in the direction of XETBook, ETbb, and Cardo which have a bit better character support, though I don’t know if all of them include all the characters you mention here. Cardo probably has the best character set, but is an original revival of Bembo not based on ET Book.

@dpk thanks for the info. Is anybody maintaining Cardo anymore?

Not sure. Certainly David J. Perry, the designer, still seems to be actively designing fonts, and if there were need for revisions I’m sure he would make them. But as it stands it’s a decently modern OpenType font, even if the design doesn’t quite match the elegance of ET Book and its derivatives

Closing this as support for this complete character set is unlikely, but better support for various languages is a goal of ET Book OT 2.0.

Where is ET Book OT 2.0 hosted?

At the moment, in this branch: https://github.com/dpk/et-book/tree/2.0 — it’s still in development (you’ll need to build the fonts from source yourself atm), but hopefully a release will be coming soonish, as I’m making fairly decent progress today. I’m also considering submitting a PR to the upstream @edwardtufte/et-book repo to make this the new official version of the fonts, especially since there are a number of issues open in that repo which are basically fixed in this fork.


Refactor 'Find' to 'Filter'

Codecov Report
Base: 69.78% // Head: 69.78% // No change to project coverage :thumbsup:

Coverage data is based on head (b851095) compared to base (34bf2ec).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #131   +/-   ##
=========================================
  Coverage     69.78%   69.78%
  Complexity      508      508
=========================================
  Files            87       87
  Lines          1655     1655
  Branches        169      169
=========================================
  Hits           1155     1155
  Misses          436      436
  Partials         64       64

Impacted Files
Coverage Δ

...rc/main/java/gim/logic/commands/FilterCommand.java
100.00% <100.00%> (ø)

...n/java/gim/logic/parser/ExerciseTrackerParser.java
80.95% <100.00%> (ø)

...ain/java/gim/logic/parser/FilterCommandParser.java
100.00% <100.00%> (ø)

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

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


Explore Callback Based Internal Representation for ZQuery

This PR explores a change to the internal encoding of ZQuery to a callback based one:
final class ZQuery[-R, +E, +A] private (
  private val start: (Result[R, E, A] => UIO[Any]) => ZIO[(R, QueryContext), Nothing, Any]
)

This lets us address the issue in ghostdogpr/caliban#577 where we were incurring additional overhead for forking queries that did not actually involve effects in zipWithPar. In this encoding, a query can either call the callback synchronously or asynchronously, so we can push the forking into the fromEffect constructor and avoid forking when the query is not actually effectual.

Replaced the Promise with a Ref. Will run all the benchmarks after I finish going through everything. Latest performance on the complex query benchmark is:
[info] Benchmark                          Mode  Cnt    Score   Error  Units
[info] GraphQLBenchmarks.complexCaliban  thrpt    5  120.123 ± 2.299  ops/s
[info] GraphQLBenchmarks.complexSangria  thrpt    5  101.097 ± 2.752  ops/s

@ghostdogpr Reran the benchmarks for this. Unfortunately, we lose the recent bump in performance we had for the simple benchmark. I think the issue is that now when we have a single effect we still have to fork a fiber whereas we didn't in the previous encoding. Not sure it is possible to avoid that while addressing the complex query case. Do we have a benchmark for more of a mixed "real world query" that we could try this with to see how the two compare on net?

@ghostdogpr Reran the benchmarks for this. Unfortunately, we lose the recent bump in performance we had for the simple benchmark. I think the issue is that now when we have a single effect we still have to fork a fiber whereas we didn't in the previous encoding. Not sure it is possible to avoid that while addressing the complex query case. Do we have a benchmark for more of a mixed "real world query" that we could try this with to see how the two compare on net?

Hmm I don't have another one. Real world use cases are very widely split, I guess there might be a lot of simple ones as well as complex ones. People tend to use one extreme or the other. Can you share the benchmark result?

You could try changing the query to this:
val simpleQuery: String =
    """{
          characters{
            name
          }
          character("Amos Burton"){
            name
          }
       }""".stripMargin

This would trigger 2 effectual fields instead of 1.

Looks like it is still slightly slower with two effects, which potentially makes sense given we still have to fork two things in both encodings and there is some overhead to doing everything in terms of callbacks, but different is much lower.
Before:
[info] Benchmark                             Mode  Cnt      Score     Error  Units
[info] GraphQLBenchmarks.introspectCaliban  thrpt    5   2276.862 ± 128.473  ops/s
[info] GraphQLBenchmarks.introspectSangria  thrpt    5    585.397 ±  63.128  ops/s
[info] GraphQLBenchmarks.simpleCaliban      thrpt    5  17877.155 ± 290.458  ops/s
[info] GraphQLBenchmarks.simpleSangria      thrpt    5   8415.909 ± 252.670  ops/s

With callback encoding:
[info] Benchmark                             Mode  Cnt      Score     Error  Units
[info] GraphQLBenchmarks.introspectCaliban  thrpt    5   2439.555 ± 193.674  ops/s
[info] GraphQLBenchmarks.introspectSangria  thrpt    5    593.164 ±  54.117  ops/s
[info] GraphQLBenchmarks.simpleCaliban      thrpt    5  15751.370 ± 123.156  ops/s
[info] GraphQLBenchmarks.simpleSangria      thrpt    5   8784.278 ± 184.473  ops/s

Your "before" seems much worse than this https://github.com/ghostdogpr/caliban/issues/577#issuecomment-687717451, I wonder why? In your before/after the degradation is not much.

I wonder if it's worth it 🤔 I did a bunch of benchmarks before/after, the gains are quite marginal and they benefit the case where you have a huge query full of ZQuery. However it's worse for simpler queries and queries using ZIO (fromEffect), which I think is the majority.

@ghostdogpr Agreed. This adds significant complexity so if it isn't a clear win in terms of performance I think it is a no. We can always come back to this later if we need to.


About paper

hi,
We think that mdetr has great potential, but we look at table 6 in the paper and find that the metics of moment retrieval on the charades-sta dataset is not much higher than that of ivg-dcl (in particular, ivg-dcl adopts C3d feature for video extractor and glove for text embedding), and your work uses clip feature + slowfast). Have you ever tested on other video grounding dataset, like activitynets?

我已经收到您的邮件，将尽快回复。

Hi @BMEI1314, in our work, we primarily focus on collecting the QVHighlights dataset and developing the MomentDETR model on top of this collected dataset. On CharadesSTA, we did not quite tune the model, but we still notice significant performance improvement (e.g., +3 on<EMAIL_ADDRESS>or +5 with pretraining). We did not test on other datasets.

Thanks for your quick reply and look forward to your follow-up work


Move KMC to its own repository

Description
The control plane repository will be cleaned up in the future. To accomplish this we will have to move KMC to its own repository in the kyma-project organisation.
Acceptance

[x] create kyma-project/kyma-metrics-collector (based on template repository)
[ ] move all code from control-plane to the new repository keeping the commit history (see)
[ ] setup post-build jobs to work on the new repository
[ ] setup test pipelines as github actions
[ ] setup security scans on the new repository

Checklist:

[ ] update codeowners
[ ] update gitignore

enable dependabot: https://github.com/kyma-project/kyma-metrics-collector/pull/14


Fio: pre-filling the attached disk does not work

Look at the fio_benchmark.Prepare() method:
fill_size = str(DISK_USABLE_SPACE_FRACTION *                         1000 * disk.disk_size) + 'M'
Let's say that disk.disk_size is 3 and DISK_USABLE_SPACE_FRACTION is 0.9.
fill_size will be exactly 2700.0M. This format is not accepted by fio and as a result, pre-filling the scratch volume will not work. What is worse, fio command will fail with exit_code=0 and we will not even see that something went wrong.
My proposition would be:

convert fill_size to int before converting it to str (format like 2700M is accepted by fio)
add more logging around fio command as it sometimes exits without error code.

@stfeng

I think we should get rid of the DISK_USABLE_SPACE_FRACTION parameter, it's pretty flaky (and setting it to 90% is also just a rough estimate that don't really work for all file systems).
If someone really wants to test against the file system (which isn't that encouraged for fio command anyway), and if they want to pre-warm the device before the testing, then the implementation here in FIO should reuse the device filling logic in the "against device case", fill the device first using /dev/*** first, then format the device again using a file system, and then mount it for subsequent testing.

Hi Mateusz,
This issue is already fixed in the latest fio_benchmark.py in the master
branch. It looks like the fix was committed 13 days ago, so it may not have
made it into the last release, but it will be in the next one.
Thanks,
Noah
On Tue, Sep 29, 2015 at 10:19 AM, stfeng<EMAIL_ADDRESS>wrote:

I think we should get rid of the DISK_USABLE_SPACE_FRACTION parameter,
it's pretty flaky (and setting it to 90% is also just a rough estimate that
don't really work for all file systems).
If someone really wants to test against the file system (which isn't that
encouraged for fio command anyway), and if they want to pre-warm the device
before the testing, then the implementation here in FIO should reuse the
device filling logic in the "against device case", fill the device first
using /dev/*** first, then format the device again using a file system, and
then mount it for subsequent testing.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/issues/497#issuecomment-144126299
.

If you want a fix right away, what you suggested with int( ... ) is exactly what I implemented. You can patch your copy of fio_benchmark.py with that change, or cherry-pick commit 84dfd924fb6a4cd51485e8ae615527b9beb06a39 . I'm closing the issue, but please re-open if there's anything else I can do to help.

I think to properly fix this issue, we should get rid off the DISK_USABLE_SPACE_FRACTION and merge the fill the file code with fill the device code.

@noahl - I have tested with a pretty fresh master version. PR with the fix was merged 2 days ago, so I was unlucky ;) anyway, thanks!

Fixed in #502.


Workspace gets overwritten if workspaceFolders returns non-empty, breaking 'find by' features

I'm using Emacs as an LSP client. It happens to report both a root URI on initialize and a non-empty list of workspace folders.
What ends up happening is the workspace that's created in the server by the initialize callback is overwritten by the one created in the workspaceFolders callback. When commands like find definition are used, the wrong workspace gets used and nothing will be found.
Here is the first assignment:
https://github.com/sumneko/lua-language-server/blob/da43d801df94ed8dce217499afb83aedb1ed24d5/script/method/initialize.lua#L18-L25
And here is where it gets overwritten:
https://github.com/sumneko/lua-language-server/blob/da43d801df94ed8dce217499afb83aedb1ed24d5/script/method/initialized.lua#L4-L9
Based on the code it seems like there's only support for a single global workspace. Is this intentional, or will there be support for redirecting the LSP commands to multiple workspaces based on the current document?

It only supports one workspace for now.


Dashboard for Loki

Signed-off-by: Matus Jenca<EMAIL_ADDRESS>

Please start every commit line with verb in imperative mode, e.g. Add dashboard for distributed loki

Please also make a template for panel aggregating overall 'health' of loki, like here for cadvisor. It will be on L1 dashboard (one level above loki dashboard) and it should aggregate health of loki to single panel.