alt text on image embeds should be imported and rendered in Frenklin

Currently images are imported without alt text. We should also investigate how authors would add alt text in google docs.

@icaraps How should we store this in the google document?

@hsaginor  Alt text is supported in google doc ... Isn't the alt text from the HTML retained when you copy paste it to google doc ?

@hsaginor There is no way to import directly into google doc. The importer saves everything as word documents and then I manually upload them. I am not sure how we transfer this information yet.

@icaraps My guess is that this is something that could be fixed in the Helix Importer. Our transformation script just selects the markup we want to import and returns that. But even if the img tags we are returning contain alt attribute the alt text doesn't get preserved even though there is a way to pass it (as image title) in markdown - https://www.markdownguide.org/basic-syntax/#images-1
Here is an example importing a page with imageEmbed - https://www.golfdigest.com/content/golfdigest-com/en/golf-news-tours/_default/article/2010/1/justintimberlake.html
The imported img tag is <img class="photo-gallery-launch a-Image" src="//golfdigest.sports.sndimg.com/content/dam/images/golfdigest/fullset/2015/08/14/55ce4f06b91019d74c98ef80_Justin-Timberlake-Interview-Staff.jpg.rend.hgtvcom.966.690.suffix/1573513695924.jpeg" style="object-position: 50% 10%; background-image: none;" loading="lazy" alt="Justin-Timberlake-Interview-Staff.jpg">
Markdown created by importer from this is ![Justin-Timberlake-Interview-Staff.jpg][image0]
From there I think it should also be preserved when the document is saved as MS Word and later converted to Google Doc when we upload it to gdrive. But assuming alt text is not lost in those steps at the moment it looks like the importer is not handling it through markdown.

@icaraps Actually you can ignore my last comment. It looks like alt text is actually imported if you check that document - https://docs.google.com/document/d/19PKC1k5_RHivYuD5J2_SgBr42CFMV4cxH9Z0csJDLdg/edit

I should make sure the import script adds alt text to all images. But do you have any examples where it's not imported?

@icaraps I see. The best I have seen on golfdigest do for these images is actually use the file name which is slightly better. If we reimport this page it will use the very cryptic "1491590551" as alt text. This is what the data is. The question is should we really force the importer to do better than the data it receives if we can't even apply a consistent rule? I see this as an editorial job.

I see this as an editorial job.

You're right, this would be the ideal world where editors define the right alt data for their images. I was hoping we could find a fallback solution that of course wouldn't be perfect but still better than the non human readable alt ...

@icaraps Well we can set it to file name if alt attribute is not set, empty or is a number like in this case. Leave it unchanged otherwise.

@icaraps See re-imported document with alt text set to file name https://docs.google.com/document/d/1RIMNgGL_zs11IPH7BXwuzba-Z1yCvG45roxTWT6kRg4/edit
But I don't see alt text being rendered by Franklin when I preview the page https://main--helix-sportsmagazine--headwirecom.hlx.page/content-v2/golf-news-tours/_default/article/2023/7/how-historic-would-scottie-scheffler-pga-tour-season-be-with-average-putting-stats
That wouldn't me in import issue.

@hsaginor indeed it looks like a bug on hlx side because it's still documented https://www.hlx.live/docs/authoring#images-and-videos. I'll ask on their channel ...


Not working at the moment

Is it still "Work in Progress" state? Can't enable alerts and has internal error

Yes, the bot is still in experimental phase.
It started as a handy thing for myself but it seems that the bot is actually getting attention.
The error seems a failure in retrieving guild information and I will look into it and try to fix it.

I see, no worries tho, please do update me if it's fixed! also you can upload your bot on top.gg for more exposure

I made some changes to the config code. Try again and let me know if the issue persists!

I tested it, hmm still can't, tried on public and private, same issue

From the logs, it looks like the bot could not resolve guild information. It probably happens because in its current state, the bot updates the config only on code retrieval. I will update the code tomorrow to account for this. It will probably be a bandaid fix until I find the event handler for when the bot joins a server. Illtry to restart the bot to force an update and force your server into the config.

Oh I see, no worries, I'm happy to help and provide feedback to make sure everything is working for everyone who uses your bot to be 100% working. Do update me

Issue should be fixed now. The bot checks on every interaction if the guild is in the config so it should update instantly. Try it out and let me know!

I just checked, and it's working now, thank you! I will notify you if the bot sends a code when there is a new code

Suggestion: A feature to get codes that are still fresh or those codes that do not expire, it may be called manual to get codes manually?
Also, for gathering codes, include this website, Prydwen, which shows the active codes or those codes that do not expire.

I'm happy to hear that the issue was solved. For the features you mentioned I opened two new issues: #2 and #3 to track progress on those features.

Thanks! no rush, but it would be cool indeed to have those additional 2

I'm happy to hear that the issue was solved. For the features you mentioned I opened two new issues: #2 and #3 to track progress on those features.

I'm happy to hear that the issue was solved. For the features you mentioned I opened two new issues: #2 and #3 to track progress on those features.

There's a new code by the way on the livestream today 2.1, it did not send yet nor the previous ones recent ones
SB87P52BTVQ3
2.1
#1 - 3SRN6L3AADLK
#2 - YTRN743TSUL7
#3 - 2S8N6M3ATV6T


KIV Features

Enhancements to the dictionary lookup:

[x] double char check
[x] single char remove check (for typos)
remove words
gui

the more I work on it the less i think a gui is needed. KIV.


Importing material/mwc-dialog library from jsDelivr CDN does not work

Describe the bug
Importing a library from a CDN does not work:
https://cdn.jsdelivr.net/npm/@material/mwc-dialog/+esm
The associated dependency cannot be resolved:
<EMAIL_ADDRESS>To Reproduce
https://demo-source.github.io/mwc-dialog.v1.html
Expected behavior / Screenshots
https://github.com/lit/lit/issues/2190
Additional context
Is this problem solved by changing the dependencies in the material or do you need fixes from lit?

Thanks for the issue! Unfortunately we can't support all CDN services and the variety of behaviors that they offer. We recommend installing from npm directly and bundling for production.
For short demos, we try to maintain compatibility with https://unpkg.com and recommend using that when not bundling for production.

@IgorNovozhilov this was an issue at jsDelivr that we just fixed - the demo works now 😄


Could not require 'angular-cli-build.js - clean setup

Please provide us with the following information:

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Windows 10

Versions. Please run ng --version. If there's nothing outputted, please run
in a Terminal: node --version and paste the result here:

$ ng --version
(node:15292) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.
angular-cli: 1.0.0-beta.10
node: 6.2.2
os: win32 x64

Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.

ng new app

cd app

ng serve

The log given by the failure. Normally this include a stack trace and some
more information.

$ ng serve
(node:16180) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.

Running without elevated rights. Running Ember CLI "as Administrator" increases performance significantly.
See ember-cli.com/user-guide/#windows for details.

Could not require 'angular-cli-build.js': Cannot find module 'angular-cli/lib/broccoli/angular2-app'
Error: Could not require 'angular-cli-build.js': Cannot find module 'angular-cli/lib/broccoli/angular2-app'
  at Function.Module._resolveFilename (module.js:440:15)
  at Function.Module._load (module.js:388:25)
  at Module.require (module.js:468:17)
  at require (internal/module.js:20:19)
  at Object.<anonymous> (C:\Users\jamie\Documents\Gitlab\flyvesteder-webapp\angular-cli-build.js:7:19)
  at Module._compile (module.js:541:32)
  at Object.Module._extensions..js (module.js:550:10)
  at Module.load (module.js:458:32)
  at tryModuleLoad (module.js:417:12)
  at Function.Module._load (module.js:409:3)

Mention any other details that might be useful.

When I switch to opening a prompt with administrator rights almost the same error is displayed. Just without the Ember CLI note

Thanks! We'll be in touch soon.

see issue #1600

@eikooc are you trying to use the webpack version?

I don't think so. Did a regular npm install -g angular-cli

Hm... what is your NPM version?

3.10.3

Odd. It kinda seems like a bad install. Can you reinstall the global and local angular-cli?

Sorry I don't have time as of now. You can close it if there is no need for the issue

Roger, closing.


ReDos mitigation

I'm a student from the kent university session

Sorry @ethull, we enjoyed reviewing your fix but another was selected. We appreciate your effort and look forward to reviewing more of your fixes in the future! :hammer: :sunglasses:


A12 Storage perms

I'm pretty sure that on android 12 the storage perms have to be added by executing the following adb command:
adb shell appops set --uid com.beatgames.beatsaber MANAGE_EXTERNAL_STORAGE allow

I just found out this a few days ago as well. I have already made a pr and it is merged #122


Correct API endpoint performance benefits sentence

Change When where to When

@evelyne24 Just a small typo improvement 😉


current_page? returns false if on root_path

If i am on the root_path, the crumb does not gets the styles.
A solution could be to check by controller and action instead of the url.
Example:
# routes.rb
root "publications#index"
resources :publications, only: :index

# publications_controller.rb
breadcrumb "Publications", publications_path

If i am on the /publications the crumb is highlighted. If im on / instead, its not.

As breadcrumb second argument you can use anything that Rails accepts including parameterized hash {controller: :publications, action: :index} ?

Wow thanks. It really worked.

Idea how to solve it if its namespaced?

I would try again Rails conventions, so as controller parameter do controller: '/admin/publications' or similar. The leading slash will ensure that the url gets resolved as Admin::Publications. Does it work?

Ok i found the problem. You are currently calling url_for(controller: ...., action: ....) before passing it to current_page?.
If i have:
# routes.rb
root "publications#index"
resources :publications, only: :index

and call url_for the following is returned:
url_for controller: "/publisher/publications", action: :index
# => http://localhost:3000/de/publisher

The Problem
When i go to http://localhost:3000/de/publisher everything is fine, and it is highlighted. But when i go to http://localhost:3000/de/publisher/publications, current_page? returns false with this URL.
Would be cool if you could pass the hash directly to current_page?. I think that should be the solution :) Thanks and sorry for distubing!

Ok, understood. Do you want to take a stab at it and submit pull request? I will happily review and merge - provided there are tests demonstrating the issue :smile:

It's been a long time since this issue has been posted - sorry!
However, I've finally fixed the problem in master branch and made the solution super flexible. I've added :match to allow for granular setting of matching behaviour when highlighting current breadcrumb. Hope you like it!


Set "Programmed" status of custom entities

Is there an existing issue for this?

[X] I have searched the existing issues

Does this enhancement require public documentation?

[X] I have added an Acceptance Criteria item for adding and/or adjusting public documentation (if applicable)

Problem Statement
#6055 implements the major part of reconciling and translating KongCustomEntity CRDs into custom entities in Kong declarative configuration. But the Programmed condition in status of KongCustomEntity is not set to True in the PR.
Proposed Solution

Set Programmed condition to True when a KongCustomEntity CRD is reconciled or translated

Additional information
No response
Acceptance Criteria

[ ] Programmed condition is set to True when a KongCustomEntity is reconciled/translated in KIC's controller

Included in #6055.


Less broken, somewhat working counter

We need to not do this in the render method as it can reexecute and therefore creates lots of buggy behaviour

Replace LWWWins with the AddSubRegister

Requested a review from everybody as this will be a core part of our code

Remember to run npm install when testing this. The first build will currently take ages because it creates Scala.js bindings from Typescript dependencies. We may replace that workflow again in the future as we don't use it much but otherwise it seems like a nice thing to get correct typings.

11h


Consider adding "DisableMaintenance" setting

Especially in virtual environments (with SAN based storage) it is important to disable the scheduled defragmentation. This is done by disabling the corresponding scheduled task, which can only be done from a SYSTEM account, not from a regular Admin account.
Usually what you'd have to do is something like (note the -s for psexec which means "run as SYSTEM")
psexec \\<IP_ADDRESS> -u Administrator -p "p4ssw0rd" -s schtasks /change /tn "\Microsoft\Windows\TaskScheduler\Maintenance Configurator" /DISABLE

Ah I just realized that this particular setting is the Win8.1/Win2012 setting, not the Win10 setting..
For Windows 10 the setting is hidden behind the "Optimize" button in the properties of any disk drive. I don't know (yet) which task or registry setting is behind this all

"Optimize" button on Win10 maps to "\Microsoft\Windows\Defrag\ScheduledDefrag" task, therefore
Disable-ScheduledTask -TaskName "\Microsoft\Windows\Defrag\ScheduledDefrag"

seems to do the trick. I'll check if it works the same on Server 2016 and add it to the next version. Although I won't put it to default preset as your use case is a bit uncommon and Win10 are generally smart enough not to defragment SSD.


Fix test_recurring_job_labels_with_backing_image

Proposal Change
The test test_recurring_job_labels_with_backing_image tests 2 backing image files.
When calling recurring_job_labels_test, clean up the recurring job setting to prevent create_recurring_jobs failed due to the previous recurring job still there when running the 2nd backing image file test.
Issues
https://github.com/longhorn/longhorn/issues/2876

Because test_recurring_job_labels also shares this function and it doesn't require extra cleanups.
I prefer this fix go into test_recurring_job_labels_with_backing_image with the other cleanups here.


Blazor doc versioning and revert test docs

Addresses #22831

Removes versioning test docs
Moves a few Blazor docs to the new approach

Rick, Wade ... I think even if you sign-off today, I'll wait until Monday morning to merge. I'm beat 😩 ... and this will need full testing after merging. It's also best to merge when fresh in case I need to quickly patch anything.
WRT using a debug-topic folder of static assets for the debug.md file instead of an expected debug folder: We have a .gitignore rule that prevents a "debug" folder from making it into the repo. Therefore, I tacked on the word "topic" to get it in.

btw - I realized this morning that the .gitignore I keep at the Blazor root was allowing me to use the debug folder name before, so I've moved a copy of that .gitignore into each of the versioned blazor folders. Now, I can keep my debug folder of static assets for the Debug topic.


fixed: webgl errors involving null/undefined arguments break the whole process

glFunctionArgToString in webgl-debug.js was broken for null and undefined arguments.
In fact the whole thing looks pretty old. We should maybe consider updating to the newer version by Kronos. Buut haven't tested that in replace, so just patched this simple bug for now :P.

Thanks!
I'll try Khronos' version in right now, and see if there are any issues. If not, we'll move forward with it in the next release, otherwise, I pulled your fix, so we can roll it back to your patched version if it does not work out :smile:


Edit Bindings to sync where it makes sense

When we are running on iOS we will advertise simultaneously on the multi-peer connectivity framework and when we have a Wi-Fi AP on SSDP as well. Which means that other iOS devices will discover us twice.
We have a similar problem on Android where if we have a Wi-Fi AP we will use SSDP but we will also use BLE/Bluetooth since not all the devices in the room may be on the same AP or have permission to join the AP. So again we can be discovered twice.
Do we want to set up the bindings so that devices can easily figure out when they have found the same peer twice and not waste battery trying to connect to them twice?

I view this as a perf feature and so I am deprioritizing it until it matters.

We have actually put in logic to address this in #359


Fast version

Dude, the prec/recall function is very slow. Use numpy if possible for batch calculations:
def calc_precision_recall(contours_a, contours_b, threshold):
        x = contours_a
        y = contours_b

        t1 = np.tile(x, (len(y),1))
        t2 = np.repeat(y, len(x), axis=0)
        t = np.concatenate( (t1,t2), axis=1)

        dx = (t[...,0] - t[...,2]) * (t[...,0] - t[...,2])
        dy = (t[...,1] - t[...,3]) * (t[...,1] - t[...,3])
        thr = (dx + dy) < (threshold * threshold)

        blocks = np.split(thr, len(y))
        block_has_hit = [np.any(x == True) for x in blocks]
        top_count = np.sum(block_has_hit)

        precision_recall = top_count/len(y)
        return precision_recall, top_count, len(y)

It should be the same, you can validate it with your own data.
Thanks, Balint

Even faster to keep the outer loop:
    def calc_precision_recall(contours_a, contours_b, threshold):
        x = contours_a
        y = contours_b

        xx = np.array(x)
        hits = []
        for yrec in y:
            d = np.square(xx[:,0] - yrec[0]) + np.square(xx[:,1] - yrec[1]) < (threshold * threshold)
            hits.append(np.any(d == True))
        top_count = np.sum(hits)

        precision_recall = top_count/len(y)
        return precision_recall, top_count, len(y)

Thanks, all the best!

Thank you very much, @bpfliegel for your suggestions. (y)

Thanks to you for your work! :)

Hey, @bpfliegel , just wondering could you please create a pull request with your improvements? That would be really great. Thank you very much.

@minar09 too busy, sorry, but I posted you the source code above ^^^ thanks

@bpfliegel , same here. No problem. Thank you very much for the codes. I will try to update the repository.

Hi! Thanks for the repo.
I implemented @bpfliegel's 2nd version and did a PR, all the details are there: #2

Thank you @theodumont for your contributions.


[SPARK-12889][SQL] Rename ParserDialect -> ParserInterface.

Based on discussions in #10801, I'm submitting a pull request to rename ParserDialect to ParserInterface.

cc @hvanhovell

Merged build finished. Test FAILed.

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

Test build #2402 has started for PR 10817 at commit 5d0e1c8.

Test build #2402 has finished for PR 10817 at commit 5d0e1c8.

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

Going to merge this.


chore(ci): update test_nightly.yaml GH workflow

What this PR does / why we need it:
Running KIC integration tests on demand with the latest version of Kong Gateway is useful, thus updating the respective workflow.

currently nightly builds can be found in the registry kong-gateway-dev, so let's use it
PR https://github.com/Kong/kubernetes-ingress-controller/pull/4389 introduces running tests for enterprise dbless, so let's run them
add a comment to PR that tests were triggered and remove label ci/run-nightly from PR

Which issue this PR fixes:
It unblocks PRs that have to be tested with newer Gateway images than the last stable version e.g. https://github.com/Kong/docs.konghq.com/pull/5865

Special notes for your reviewer:
Probably we should have a broader discussion about our workflows and their names. This PR doesn't deduplicate code (it can be improved in the same way as other workflows. Food for thought - maybe introduce support for a more complicated label or comment that will allow specifying a custom tag instead of always using the latest.
Enterprise suites are run with enterprise image and default one with OSS image. This distinction has been made because EE Gateway EE will be detected no matter the license and skipping tests with an invalid/missing license (works like OSS, but still presents EE version) is risky, because it may lead to overlook of e.g. expired license.  See what happened when only EE image was used

Unfortunately, it doesn't evaluate, see this, furthermore I need to juggle more with images, see updated PR description @pmalek

Unfortunately, it doesn't evaluate, see this, furthermore I need to juggle more with images, see updated PR description @pmalek

Got it. It might be that defining the defaults for inputs doesn't make sense to use them in a different event (i.e. not workflow_dispatch where we define them in).
🤷 Sad. Let's stick to using those verbatim as it was.


iOS: (Feedly) Read articles switch back to unread

We've been seeing articles that should have been marked as read either not marking as unread or switching back to unread.  This is probably how it is happening.

We begin to send our article statuses to the sync service by selecting all available statuses from the sync database.
An article is marked as read and added to the sync database.
We complete sending our statuses.
We request article statuses from the sync service.
The article just marked as read in NNW is unread in the sync service, so we mark it as unread in NNW, putting us in sync with the sync service but out of sync with what we will send to the sync service and what the user is expecting.
Eventually the entry in the sync database gets processed and we go back into a read state for the article or the user manually marks it as read again.

To rectify this we will need to check the sync database for pending entries while processing statuses from the sync service.  If we have a pending entry we should discard the sync service status since we know that we have more recent status information than the sync service.

I added two functions to SyncDatabase, selectPendingReadStatusArticleIDs and selectPendingStarredStatusArticleIDs to get the article ids that we don't want to update when receiving statuses from the sync service.

Used https://github.com/Ranchero-Software/NetNewsWire/commit/c3fbf88fbb9776ed11e8a024b98440f733bfc785 for reference.


Cannot navigate dropdown menus when  voiceover is on

Update: Dropdown menus can be navigated and read aloud however, the pop-up window is hidden and only the field itself changes. This happens because under the hood, the selection is being changed and not the highlight, when the selection is changed the pop-up menu is auto closed. A potential work around is overwriting the field select method to disable auto closing and instead only close when enter is pressed.
Old:Voiceover uses the arrow keys by default so dropdowns should be controlled with WASD to be consistent.

This now works with WASD

Great!
Steph
Sent from my iPad

On Jun 7, 2016, at 1:04 PM, MaryS94<EMAIL_ADDRESS>wrote:
This now works with WASD
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[docs] Typo in the "Test HTTPS Features" article

https://devexpress.github.io/testcafe/documentation/guides/advanced-guides/test-https-features-and-http2-websites.html

Most modern browsers thread localhost as a secure origin, and some built-in browser features that require a secure origin (Service workers, GeoAPI) should work.

I believe, we meant treat, not thread.

Fixed by https://github.com/DevExpress/testcafe/commit/81672b1fbf591d176200a9c7892a462de5760c9d
Thanks!


Follow standard convention for Arrays

According to common convention (i.e. php, python, ruby) named fields ending with square brackets [] should be treated as arrays.
This example HTTP POST should produce a parsed object like the one shown here below:
Shorter snippet from link above:
-----------------------------114772229410704779042051621609
Content-Disposition: form-data; name="attachments[]"; filename="file1.txt"
Content-Type: text/plain

This is file 1

-----------------------------114772229410704779042051621609
Content-Disposition: form-data; name="attachments[]"; filename="file2.txt"
Content-Type: text/plain

This is file 2
It has three lines
And was created on OS X

Parsed Object:
{
    name: "AJ ONeal",
    email<EMAIL_ADDRESS>    // <Object HTML5::FileAPI::File>
    avatar: {
        name: "smiley-cool.png",
        size: 869,
        type: "image/png",
        path: "/tmp/ab489fe1d9a4df.png"
    },
    attachments: [
        // <Object HTML5::FileAPI::File>
        {
            name: "file1.txt",
            size: 15,
            type: "text/plain",
            path: "/tmp/c3e29fa4de1d9f.png"
        },
        {
            name: "file2.txt",
            size: 58,
            type: "text/plain",
            path: "/tmp/9a4ab48dffe1d9.png"
        }
    ]
}

@Ali-Azmoud  Thanks.....

We now handle all those scenarios
        simple<input type="text" name="simple">

        array text 0<input type="text" name="atext[]">

        array text 1<input type="text" name="atext[]">

        file simple<input type="file" name="filesimple">

        file attribute multiple<input type="file" name="multiplefile" multiple>

        file html array0<input type="file" name="filearray[]">

        file html array1<input type="file" name="filearray[]">

        file html array and mulitple0<input type="file" name="mfilearray[]" multiple>

        file html array and mulitple1<input type="file" name="mfilearray[]" multiple>

attribute multiple + array merges into one array.
Open a new issue if you still have a problem

We now handle all those scenarios
attribute multiple + array merges into one array.
Open a new issue if you still have a problem

@GrosSacASac link to commit?
Does this enhancement you speak also handle multidimensional arrays?
ie.)
client:
<input type="text" name="people[0]['firstName']" value="Jerry">
<input type="text" name="people[0]['middleName']" value="T">
<input type="text" name="people[0]['lastName']" value="Mouse">

<input type="text" name="people[1]['firstName']" value="Tom">
<input type="text" name="people[1]['middleName']" value="R">
<input type="text" name="people[1]['lastName']" value="Cat">

server (JSON output):
{
  "people": [
    {
      "firstName": "Jerry",
      "middleName": "T",
      "lastName": "Mouse"
    },
    {
      "firstName": "Tom",
      "middleName": "R",
      "lastName": "Cat"
    }
  ]
}

note: you get this functionality built-in now, in express.js v4.16+, see express.urlencoded

please make a new issue for extra functionality, so we can keep track of it

Yup. It's not working with multidimensional arrays. We should introduce querystring parsing for this. New issue please :)

We chose to not do it. form inputs with the same name, (regaredless if they have []) will be in the same array


The badge is no status

Are you experiencing an issue with...
shields.io
🐞 Description

Why is it no status?
🔗 Link to the badge
https://img.shields.io/github/actions/workflow/status/zilliztech/GPTCache/pylint.yaml?branch=dev&label=Pylint
💡 Possible Solution
want it to show normally

The badge is set to report the status of builds on the branch called dev on the repo zilliztech/GPTCache
The build in your screenshot ran on a branch called dev on a forked copy. That's why it says SimFG:dev under branch not dev
If you look at the badge for a workflow which has a run on the dev branch on the source repo zilliztech/GPTCache (e.g: CI_integration_test_main.yaml ) you will see a build status: https://img.shields.io/github/actions/workflow/status/zilliztech/GPTCache/CI_integration_test_main.yaml?branch=dev

@chris48s thank you so much!!!


Minor code improvements

This seems to have some conflicts, are you able to resolve those?

Alright, merge conflicts resolved.
Actually, I think this might be a breaking change because of the parameter type change from status: &u16 to status: u16; see this commit. Looking through the commit diff I see that I also had to update the parameters of files inside examples.

I genuinely have no clue why that comment was in italics, sometimes the GitHub mobile app is magical.
I don't see any changes to the parameters that examples take, I think they've always taken owned values haven't they?

Yeah @Asha20 I'm pretty sure you've just needed to change some things in .perseus/ (which we work on inside examples/basic/, I know...), which doesn't constitute a breaking change.


Plot multiple plots in a shell

Here is my test.nu script
#!/usr/bin/env nu

let one = (seq 0.0 0.01 20.0 | math sin)
let two = (seq 1.0 0.01 21.0 | math sin)
let three = (seq 2.0 0.01 22.0 | math sin)
let four = (seq 3.0 0.01 23.0 | math sin)

$one | plot
[$one $two] | plot
[$one $two $three $four] | plot -l -t "Four sine lines!"

The result only show last plot
    Four sine lines!
    ⣥⠖⠋⠉⠳⣤⠖⠋⠉⠳⣄          ⢀⡴⠊⠉⠙⢦⡴⠊⠉⠙⢦⡴⠊⠉⠙⢦⡴⠊⠉⠙⢦⡀          ⣠⠞⠉⠉⠲⣤⠞⠉⠉⠲⣤⠞⠉⠉⠲⣤⠞⠉⠉⠲⣄          ⢀⡴⠋⠉⠓⢦⡴⠋⠉⠓⢦⡴⠋⠉⠓⢦⡄ 1.0
    ⠍⢳⡀ ⡴⠉⢳⡀  ⠈⢳⡀       ⣠⠏   ⣠⠏⠙⣆ ⣠⠏⠙⣆ ⣠⠏⠙⣆   ⠙⣆       ⢀⡞⠁  ⢀⡞⠉⢧⡀⢀⡞⠉⢧⡀⢀⡞⠉⢧⡀  ⠈⢧⡀       ⣰⠋   ⣰⠋⠱⡄ ⣰⠋⠱⡄ ⣰⠋⠁
    ⠂ ⢱⡞   ⠱⡄   ⠱⡄     ⣰⠃   ⣰⠃  ⠘⣶⠃  ⠘⣶⠃  ⠘⣆   ⠘⣆     ⢀⡞   ⢀⡞   ⢳⡞   ⢳⡞   ⢳⡀   ⢳⡀     ⡴⠁   ⡴⠁  ⠘⣶⠁  ⠘⣶⠁
    ⡁ ⡞⠹⡀   ⠹⡀   ⠱⡀   ⣰⠃   ⣰⠃   ⣰⠛⣆  ⣰⠛⣆   ⠘⣆   ⠘⣆   ⢀⡞   ⢀⡞   ⢀⡞⢳⡀ ⢀⡞⢳⡀   ⢳⡀   ⢳⡀   ⡸⠁   ⡸⠁   ⡸⠙⡆  ⡸⠙⡆
    ⠄⡼  ⢳⡀   ⢳⡀   ⢳⡀ ⢠⠃   ⢠⠃   ⢠⠃ ⠘⣆⢠⠃ ⠘⣆   ⠘⣆   ⠘⣆ ⢀⡞   ⢀⡞   ⢀⡞  ⢣⢀⡞  ⢣    ⢣    ⢣  ⣰⠃   ⣰⠃   ⣰⠃ ⠸⡄⣰⠃ ⠸⡄
    ⣻⠁   ⢳    ⢳    ⢳⢠⠏   ⢠⠏   ⢠⠏   ⢸⡏   ⠸⡄   ⠸⡄   ⠸⡄⡼    ⡼    ⡼   ⠈⣿   ⠈⢧   ⠈⢧   ⠈⢧⢰⠃   ⢰⠃   ⢰⠃   ⢹⡃   ⠹⡀
    ⡁⢯ ⠁⠈ ⢧⠈ ⠁⠈⢧⠁⠈ ⢁⣯ ⠁⠈⢀⡏⠈ ⠁⢈⡏⠁⠈ ⢁⡎⠹⡅⠈ ⠁⠹⡄⠁⠈ ⠹⡌ ⠁⠈⡽⡅⠈ ⠁⡼⠁⠁⠈ ⡽⠉ ⠁⠈⡼⠉⣏ ⠁⠈⠈⣇⠈ ⠁⠈⣇⠁⠈ ⢩⣏ ⠁⠈⢠⠇⠈ ⠁⢨⠇⠁⠈ ⢡⠏⢳⡁⠈ ⠁⠁
    ⠄⠈⢇   ⠈⢇   ⠈⢇ ⢀⡞⠈⢇ ⢀⡞   ⢀⡞   ⢀⡞  ⢳⡀   ⢳⡀   ⢳⡀ ⣰⠃⢳⡀ ⣰⠃   ⣰⠃   ⣰⠃ ⠘⣆   ⠘⣆   ⠘⣆ ⢀⠏⠘⣆ ⢀⠏   ⢀⠏   ⢀⠏  ⢣
    ⠂ ⠘⣆   ⠘⣆   ⠘⣆⡜  ⠘⣆⡜    ⡜    ⡞    ⢳⡀   ⢳⡀   ⢳⣰⠃  ⢳⣰⠃   ⣰⠃   ⣰⠃   ⠘⡄   ⠘⡄   ⠘⣄⡞  ⠘⣄⡞   ⢀⡞   ⢀⡞   ⠈⢧
    ⡁  ⠘⣆   ⠘⣆  ⢀⡞⣆  ⢀⡜⣆  ⢀⡜   ⢀⡜      ⢳⡀   ⢳⡀  ⣰⢳⡀  ⣰⢳⡀  ⣰⠃   ⣰⠃     ⠘⣄   ⠘⣄  ⢀⡞⣄  ⢀⡞⣄  ⢀⡞   ⢀⡞     ⠈⢧
    ⠄   ⠈⢧⡀  ⠈⢧⣀⠞ ⠈⢦⣀⠞ ⠈⢧⣀⠞   ⢀⠞        ⠳⡄   ⠳⡄⡴⠃ ⠳⡄⡴⠃ ⠳⣄⡴⠃   ⡴⠁       ⠘⢆   ⠘⢆⣠⠎ ⠘⢆⣠⠎ ⠘⢆⣠⠎   ⣠⠎        ⢳⡀
    ⠂     ⠑⢦⣀⣠⠴⠛⢦⣀⣠⠴⠛⢦⣀⣠⠴⠛⢦⣀⣠⠴⠋          ⠘⠦⣄⣀⡤⠞⠦⣄⣀⡤⠞⠦⣄⣀⡤⠟⠦⣄⣀⡤⠞          ⠈⠳⢤⣀⣠⠞⠳⢤⣀⣠⠞⠳⢤⣀⣠⠞⠳⢤⣀⣠⠖⠁            -1.0
    0.0                                                                                2000.0
    Line 1: --- Line 2: --- Line 3: --- Line 4: ---

Sorry for late response, you'll have to do something like this, as I think nushell only returns the last item in a script?
#!/usr/bin/env nu

let one = (seq 0.0 0.01 20.0 | math sin)
let two = (seq 1.0 0.01 21.0 | math sin)
let three = (seq 2.0 0.01 22.0 | math sin)
let four = (seq 3.0 0.01 23.0 | math sin)

let first = $one | plot
let second = [$one $two] | plot
let third = [$one $two $three $four] | plot -l -t "Four sine lines!"

print $first $second $third

So you'll have to save intermediates and print when you like.
M

Closing this, as I think this is addressed!


Inner error when injecting ValidationConfig

I need to switch locale globally, but cannot access ValidationConfig.
Anywhere, app.js or any component, when injecting ValidationConfig I get inner error.
But,
import {ValidationConfig} from 'aurelia-validation';
console.log('ValidationConfig', ValidationConfig); // => function ValidationConfig()

I think the same issue here: http://aurelia.io/validation/#/i18n (errors in console)
Error: Error invoking undefined. Check the inner error for details.
------------------------------------------------
inner error: TypeError: Type is not a function
    at Object.invoke (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:334:14)
    at InvocationHandler.invoke (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:309:168)
    at Container.invoke (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:557:25)
    at StrategyResolver.get (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:145:37)
    at Container._get (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:508:23)
    at Container._get (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:505:28)
    at Container.get (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:491:28)
    at Container.elementContainerGet [as get] (http://localhost:3030/jspm_packages/npm/aurelia-templating@1.0.0-beta.1.0.1/aurelia-templating.js:1504:17)
    at Object.invoke (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:349:81)
    at InvocationHandler.invoke (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:309:168)
    at Object.invoke (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:334:14)
    at InvocationHandler.invoke (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:309:168)
    at Container.invoke (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:557:25)
    at StrategyResolver.get (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:145:37)
    at Container._get (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:508:23)
    at Container._get (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:505:28)
    at Container.get (http://localhost:3030/jspm_packages/npm/aurelia-dependency-injection@1.0.0-beta.1/aurelia-dependency-injection.js:491:28)
...

@valichek I was just about to open new issue with d.ts file and I saw your issue. I am switching validation locale globally with Validation class. I am injecting Validation class and then on validationInstance is property config:ValidationConfig where you can switch locale.
I am not sure if this is proper so somebody here could confirm or denie it.

@kdeberni , thx, will try

@valichek @kdeberni Yes I think this is the proper way to change the locale, if that isn't working please feel free to open this back up and we can figure out the right way.
In the end we need to get the locale setup out of validation anyway :)  That is tracking in #174


Blocks source attribute (API and DB) is empty

Description
source in blocks api is empty, the corresponding database similarly shows no value. Other data columns are correctly filled.
Steps to Reproduce

Database migration from 0.19 to 0.20

Reproduces how often
100% of the time
Version
Release 0.20 w/ db migration steps. Coming from release 0.19 on Ubuntu 16.04

Block source is only used for clustering scenarios where the value is supposed to contain of the geographic peer receiving and validating a share. When that share results in a block, the block's source property will reflect the originating share's source property. If you want that value to be filled, configure the "clusterName" property of the main config.


Haneke Exception

I am using Haneke in my app. I'm seeing this exception in the crash logs. Any ideas what the issue might be?
Thread : Crashed: io.haneke
0  Haneke                         0x1004723ec _TTSf4g_gs_n___TFC6Haneke9DiskCacheP33_700FC6C008DA9A15639B525F3849CF5611setDataSyncfS0_FTCSo6NSData3keySS_T_ + 1740
1  Haneke                         0x100472e8c _TFFC6Haneke9DiskCache7setDataFS0_FTKT_GSqCSo6NSData_3keySS_T_U_FT_T_ + 96
2  libdispatch.dylib              0x19b0cd6e8 _dispatch_call_block_and_release + 24
3  libdispatch.dylib              0x19b0cd6a8 _dispatch_client_callout + 16
4  libdispatch.dylib              0x19b0d96ec _dispatch_queue_drain + 864
5  libdispatch.dylib              0x19b0d11ac _dispatch_queue_invoke + 464
6  libdispatch.dylib              0x19b0db5bc _dispatch_root_queue_drain + 728
7  libdispatch.dylib              0x19b0db2dc _dispatch_worker_thread3 + 112
8  libsystem_pthread.dylib        0x19b2e1470 _pthread_wqthread + 1092
9  libsystem_pthread.dylib        0x19b2e1020 start_wqthread + 4

This looks like a duplicate of #207, can you try with the current development branch or provide a way to reproduce the crash ?

This looks like a duplicate of #207, can you try with the current development branch or provide a way to reproduce the crash ?

Hi,
I don't see the correlation between this and #207. I can't reproduce this. The crash log was retrieved from Crashlytics.
Thank you.

right, sorry

#211 is in the same function setDataSync


Issue with Command Block

Description

When a player activates a command block via pressure plate with the command block executing this command "lp user @p parent add (role here)", the server instantaneously crash with this server log.
Reproduction steps

Place a command block on the floor with a pressure plate above
Key this into the command block  "lp user @p parent add (role here)".
Step on the pressure plate
Crash

Expected behaviour

It should give the player the parent rank/role.
Environment details

Server type/version: 1.16.4 - 288
LuckPerms version: v5.2.29

https://pastebin.com/c7WCxZcs
Any other relevant details

It did promote player to the said role but the server crashed afterwards.

Please follow the issue template.

Looks like the crash report you provided was not an actual crash/error report. That message is generated when a server is lagging. If your server did exit unexpectedly, please provided the actual error log. Additionally, I was not able to reproduce with 1.16.4 Paper and the latest version of LuckPerms.

Looks like the crash report you provided was not an actual crash/error report. That message is generated when a server is lagging. If your server did exit unexpectedly, please provided the actual error log. Additionally, I was not able to reproduce with 1.16.4 Paper and the latest version of LuckPerms.

Problem is, server is not lagging at all before the pressure plate was activated. Its a brand new server with 2 players on it, pretty sure isn't server side lag.

Activating the command block via pressure plate, causes the lag, then causes the "crash"

It's interesting because I know since 1.13 selectors (like the @p) don't work on plugin commands. I'll review the relevant processing code, but I don't think LP has special support for them either. Are you using spigot/paper?

It's interesting because I know since 1.13 (on spigot) selectors (like the @p) don't work on plugin commands. I'll review the relevant processing code, but I don't think LP has special support for them either. Are you using spigot/paper?

Yes, I'm using paper 1.16. Do note that, the command did actually execute and promoted the player who stepped on the pressure plate. Just that it crashed afterwards.

Okay, may I have a list of your active plugins then. It would seem one of them is doing the processing, and taking long at that. Again, @p shouldn't work as of 1.13, it was a spigot decision.

Looks like it works fine by itself when I add CommandHook. You may be running out of resources. Either way, it's not a problem with LP.

Ok, let me try upgrading the server and try once more

LP has special support for target selectors.

Well, I don't know if that's a separate issue, but target selectors do not work when using LP by itself.

Well, I don't know if that's a separate issue, but target selectors do not work when using LP by itself.

Did you enable their parsing in LP config?
https://github.com/lucko/LuckPerms/blob/798a5390248516954bfd50a40b1e6569f7ecc6ed/bukkit/src/main/resources/config.yml#L635-L637

Right. It’s a config setting that needs to be enabled. Completely forgot 😅

Nothing here to suggest there's an issue with LP itself.


Web UX: Map marker positioned differently than mobile

On the web version, the map marker appears to be positioned bottom-middle vs. middle on mobile. This makes for inaccurate presentation of wonder location. Because web version also includes the ability to zoom in via scroll wheel, you can watch this effect continue as the location gets closer to the actual position.
Web:

Mobile:

Interesting, this is likely an issue with the maps plugin for web but I'll take a look.

@jaredbell This is a result of this bug with the maps plugin where custom anchor positions are ignored for the marker:
https://github.com/flutter/flutter/issues/80578
Web only supports bottom-center anchoring as that is the default. We're currently using a custom anchor point because of the white background around our pin:

Options:

Leave it as-is until web plugin bug is fixed
Replace marker for all platforms removing the background circle
Create custom marker just for web removing the background circle

Fixed by d8833d39


@tball about circle_finder

Hello, @tomball ,
My question is whether the circle_finder can find all possible circle references, can it find circle refence in container like hashmap, arraylist?  We just verify each found circle references, then pick which circle is true circle reference？
Maybe the found circle references were just a part of all possible circle references？ We need the found the reminder circle references from the source code?
Can circle finder automaticly  set the @weakOuter for us? then we just don't care the circle reference problems？
Thank you for your great j2objc product!

The cycle_finder uses static analysis to locate cycles, and static analysis is limited in what it can find (that's why compilers can't find all bugs). So no, cycle_finder won't find all cycles. It also reports "false positives," where cycles are reported where they don't exist. Its main use is to report the easier cycles, and when used as a unit test, to catch some cycles when they are later created.
The way to find cycles is with Xcode's Leaks profiler. Since it examines the actual program execution instead of just its source code, the Leaks profiler will report real leaks and not report false positives. Each leak will have a stack trace to locate the offending class.
@WeakOuter is not a general-purpose cycle fixer: it's specifically used with non-static inner classes where an inner class object may live longer than the outer class. The easiest way to avoid issues with non-static inner classes is to make them static, as Effective Java recommends. We cannot make that change in j2objc's version of the Java Runtime since it's a public API, but you can make that change in your product.


Want To Add Check Box but note working?

Summary of problem or feature request

I need To add A check box which will showed that is user request accept or pending. If Accept It will show checked id pending will show uncheck.
but every time it showed unchecked if Status active or pending in database
Code snippet of problem

->addColumn('status', function ($item) {
$togolebutton = '<input  ' .$item->status.' == "Active" ? "checked" : ""}}   type="checkbox" class="status" id="status"  data-id="'.$item->id.'" />';
return $togolebutton;
})
System details

Operating System : Windows 10
PHP Version :  7.3
Laravel Version : 8
Laravel-Datatables Version: 9.3

I guess it should be
$togolebutton = '<input ' . ($item->status == "Active" ? "checked" : "") .' type="checkbox" class="status" id="status" data-id="'.$item->id.'" />';

I also try this way and it also work
```
if($item->status == 'Active'){
$togolebutton = '';
}
else{
$togolebutton = '';
}


support setting passive mode in bgp peer

For https://github.com/projectcalico/calico/issues/1603
This allow use to set passive: true in bgp peer spec so we can leverage that in calico/node to set proper bird config.
I think this should touches all the required place here. Then we would want to modify calico/node for confd config and calicoctl for updating new fields.
cc @caseydavenport

 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.

@dqminh thanks for this pr!
I think there's a bit more to do here. Namely, update the model.BGPeer to include this field (this is what confd sees)
Is there a corresponding PR for the BIRD templates? It would be good to review them together since they rely on each other.
Also, looks like the bot would like you to sign the CLA :)

@caseydavenport I've gone ahead and updated the PR with a change to model.BGPPeer.
Would it be worth modifying api.BGPPeer as well to allow filling in the full model.BGPPeer struct?
As a potentially unrelated side-note,  how does back-porting a change work? We'll be migrating to 2.6.x; I just want to make certain this change would land there as well.

As a potentially unrelated side-note, how does back-porting a change work? We'll be migrating to 2.6.x; I just want to make certain this change would land there as well.

So, we'll need to work through this one. Strictly speaking, this is an API change and so would require a minor bump rather than a patch release (e.g v2.7.0, or v3.2.0). I don't think we have any plans to cut a v2.7.0 release.
I'll need to discuss internally and get back to you, but in the absence of that I imagine this would land in v3.2.0.

Looks like the fomatting check is failing, proabably need to run goimports over the new code.
You'll also need to run make gen-files to update some autogenerated code.

@caseydavenport sorry about that!  I've let go imports fix the formatting (shame shame shame).  I've also re-run gen-files; though, that only netted a change to deepcopy (the import order).

This looks good to me.
Before we merge I'd like to be able to review it side-by-side with the corresponding confd template changes to make this work. Is there a PR for that as well?

There isn't one yet ... but ... I can make one! :)

Any update on this guy?

@mattalberts @dqminh this has gotten out of date and needs a rebase.
I'm going to close for now since there hasn't been any recent activity, but do ping me if you find time to work on this and we can reopen - I'd quite like to see it go in!


Import automake changes from vmod-example.

Tells the user in more user-friendly way if we could
not find varnish.m4.
make dist should work now.

Looks good, thanks


[Bitcoin] Implement Send all functionality

In support of https://github.com/hirosystems/stacks-wallet-web/issues/2691


Fix tabs in project.pbxproj of iOS template

Should be tab instead of space.

By analyzing the blame information on this pull request, we identified @alexeylang and @tonyxiao to be potential reviewers.

@facebook-github-bot shipit

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.


split_out=True doesn't work with groupby aggregations

I'm liking the shuffle groupby-aggregations stuff, especially when we have many grouping columns and large data (I suspect that this would be a sensible default).  Unfortunately, it's not working quite right.  Trying the following ...
import dask_expr
df = dask_expr.datasets.timeseries()
df.groupby("name").x.sum(split_out=True).optimize(fuse=False).pprint()

Sum(TreeReduce): split_every=8
  Sum(GroupByChunk): chunk=<methodcaller: sum> columns=x numeric_only=False
    Timeseries: seed=1082116929 columns=['name', 'x']

This is because 1 and True are equal to each other.  We need something like the following:
--- a/dask_expr/_reductions.py
+++ b/dask_expr/_reductions.py
@@ -412,7 +412,7 @@ class ApplyConcatApply(Expr):
         chunked = self._chunk_cls(
             self.frame, type(self), chunk, chunk_kwargs, *self._chunk_cls_args
         )
-        if self.split_out == 1 or sort:
+        if self.split_out is not True and self.split_out == 1 or sort:
             # Lower into TreeReduce(Chunk)
             return TreeReduce(
                 chunked,

However, after this we still have a problem:
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
<ipython-input-3-966392224809> in ?()
----> 1 df.groupby("name").x.sum(split_out=True).optimize(fuse=False).pprint()

~/workspace/dask-expr/dask_expr/_collection.py in ?(self, combine_similar, fuse)
    174     def optimize(self, combine_similar: bool = True, fuse: bool = True):
    175         return new_collection(
--> 176             self.expr.optimize(combine_similar=combine_similar, fuse=fuse)
    177         )

~/workspace/dask-expr/dask_expr/_expr.py in ?(self, **kwargs)
    541     def optimize(self, **kwargs):
--> 542         return optimize(self, **kwargs)

~/workspace/dask-expr/dask_expr/_expr.py in ?(expr, combine_similar, fuse)
   2222     # Manipulate Expression to make it more efficient
   2223     result = result.rewrite(kind="tune")
   2224
   2225     # Lower
-> 2226     result = result.lower_completely()
   2227
   2228     # Cull
   2229     result = result.rewrite(kind="cull")

~/workspace/dask-expr/dask_expr/_expr.py in ?(self)
    376         """
    377         # Lower until nothing changes
    378         expr = self
    379         while True:
--> 380             new = expr.lower_once()
    381             if new._name == expr._name:
    382                 break
    383             expr = new

~/workspace/dask-expr/dask_expr/_expr.py in ?(self)
    333     def lower_once(self):
    334         expr = self
    335
    336         # Lower this node
--> 337         out = expr._lower()
    338         if out is None:
    339             out = expr
    340         if not isinstance(out, Expr):

~/workspace/dask-expr/dask_expr/_reductions.py in ?(self)
    199             shuffled = SetIndexBlockwise(shuffled, split_by, True, divisions)
    200
    201         # Convert back to Series if necessary
    202         if is_series_like(self._meta):
--> 203             shuffled = shuffled[shuffled.columns[0]]
    204
    205         # Blockwise aggregate
    206         result = Aggregate(

~/workspace/dask-expr/dask_expr/_expr.py in ?(self)
    789     @property
    790     def columns(self) -> list:
    791         try:
    792             return list(self._meta.columns)
--> 793         except AttributeError:
    794             return []

~/mambaforge/envs/test-env/lib/python3.11/functools.py in ?(self, instance, owner)
    997             with self.lock:
    998                 # check if another thread filled cache while we awaited lock
    999                 val = cache.get(self.attrname, _NOT_FOUND)
   1000                 if val is _NOT_FOUND:
-> 1001                     val = self.func(instance)
   1002                     try:
   1003                         cache[self.attrname] = val
   1004                     except TypeError:

~/workspace/dask-expr/dask_expr/_expr.py in ?(self)
   1117     @functools.cached_property
   1118     def _meta(self):
   1119         args = [op._meta if isinstance(op, Expr) else op for op in self._args]
-> 1120         return self.operation(*args, **self._kwargs)

~/workspace/dask-expr/dask_expr/_shuffle.py in ?(df, new_divisions, *args, **kwargs)
   1042     @staticmethod
   1043     def operation(df, *args, new_divisions, **kwargs):
-> 1044         return df.set_index(*args, **kwargs)

~/mambaforge/envs/test-env/lib/python3.11/site-packages/pandas/core/frame.py in ?(self, keys, drop, append, inplace, verify_integrity)
   5869                     if not found:
   5870                         missing.append(col)
   5871
   5872         if missing:
-> 5873             raise KeyError(f"None of {missing} are in the columns")
   5874
   5875         if inplace:
   5876             frame = self

KeyError: "None of ['name'] are in the columns"

I stopped investigating at this point.  cc @rjzamora I think

For motivation, this is where I am today with TPC-H on Scale-1000.

Query 3 spends more than half of its time on this tree reduction, which I think given the number of columns and partitions we can pretty comfortably set the default to use the shuffle system.  With that change we would be almost always faster than Spark at this scale.

Gah. I didn’t realize this wasn’t tested! I’ll take a look as soon as I can.

Thanks Rick!
On Fri, Oct 27, 2023 at 8:48 PM Richard (Rick) Zamora <
@.***> wrote:

Gah. I didn’t realize this wasn’t tested! I’ll take a look as soon as I
can.
—
Reply to this email directly, view it on GitHub
https://github.com/dask-contrib/dask-expr/issues/376#issuecomment-1783660040,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTHCYEHXUBZPJ6YSXBTYBRP7ZAVCNFSM6AAAAAA6TQ434CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGY3DAMBUGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I am curious how many workers were you using there?
Is this with your read parquet branch?
The size optimization PR should get us over the hump  for query 1
Matthew Rocklin @.***> schrieb am Sa. 28. Okt. 2023 um
03:54:

Thanks Rick!
On Fri, Oct 27, 2023 at 8:48 PM Richard (Rick) Zamora <
@.***> wrote:

Gah. I didn’t realize this wasn’t tested! I’ll take a look as soon as I
can.
—
Reply to this email directly, view it on GitHub
<
https://github.com/dask-contrib/dask-expr/issues/376#issuecomment-1783660040>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AACKZTHCYEHXUBZPJ6YSXBTYBRP7ZAVCNFSM6AAAAAA6TQ434CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGY3DAMBUGA>

.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/dask-contrib/dask-expr/issues/376#issuecomment-1783661227,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOYQZGDSXQA5OIAIBSV3YELYBRQUDAVCNFSM6AAAAAA6TQ434CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGY3DCMRSG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This is my read-parquet branch, yes.  This is run with 32 4-core workers.  (everyone has 128 cores in this workflow)
I'll cherry-pick that commit for future runs.

#377 for the shuffle issue

the prs are merged, current main should work now (query 3 runs in 85-100 seconds on my cluster)


feat: define Newton's method and prove decomposition as nilpotent + root

This is just a modified version of the code provided by Antoine Chambert-Loir here: https://leanprover.zulipchat.com/#narrow/stream/116395-maths/topic/jordan-chevalley.20decomposition/near/411402670
Co-authored-by: Antoine Chambert-Loir<EMAIL_ADDRESS>

bors merge


Choose GroupAlsoByWindows implementation based on streaming flag

This change depends on https://github.com/apache/beam/pull/3315.

R: @aaltay

Looks good to me. Lets wait on #3315 to be merged, rebase and test after that.

Coverage increased (+0.007%) to 70.589% when pulling 8b77b37be1a7537eb07b7196972542e829831bed on charlesccychen:streaming-gabw-refactor into 911bfbdef62a7bb750426da9cc59537c4ffd9e1f on apache:master.

Coverage increased (+0.01%) to 70.592% when pulling 8b77b37be1a7537eb07b7196972542e829831bed on charlesccychen:streaming-gabw-refactor into 911bfbdef62a7bb750426da9cc59537c4ffd9e1f on apache:master.

Coverage decreased (-0.003%) to 70.658% when pulling 894ac8feb361d70d753ce705df36fd5cebf36a61 on charlesccychen:streaming-gabw-refactor into f9d51aa5cc718c44c82ec4773dbd16a57e7d394e on apache:master.

The Java tests were flaky--the Python tests passed.  Can you PTAL?  Thanks!

LGTM, except for one comment.

Coverage decreased (-0.02%) to 70.645% when pulling 503518df18486e45d0d78ed9ea83caf2b2418d1f on charlesccychen:streaming-gabw-refactor into 646caf255510c735488b88ecc69edd9b9bde4081 on apache:master.

Thanks, PTAL.

Coverage decreased (-0.01%) to 70.648% when pulling 503518df18486e45d0d78ed9ea83caf2b2418d1f on charlesccychen:streaming-gabw-refactor into 646caf255510c735488b88ecc69edd9b9bde4081 on apache:master.

Coverage decreased (-0.003%) to 70.677% when pulling 4226e55ca25a7e5a864439d2900b48abf14a100e on charlesccychen:streaming-gabw-refactor into dd9abc397c67b47f95414bfd5debb08b7b6a6eaf on apache:master.

Can you please take another look?  Thanks!

LGTM. thank you.


Writing an actual rendering engine

Currently in the process of writing an actual opengl-based rendering engine for feather instead of just using sdl's default renderer. Our own renderer will give us more control over the rendering process, and give me the oppurtunity to add some cool new features in the future.

out of scope, might re open later


Quote generator

I would like to add the code for a random quote generator using html, css and javascript under GSSOC'21.

Similar project already exist


[sca] Add AES fixed-vs-random Data capture mode

The first commit enables AES serial fixed-vs-random data capture
The second commit does the same for uJSON

Note that so far I've reviewed the simpleserial side only. Let's first iterate on this and then carry the final changes over to the uJSON implementation. At this point, it will be good if also @nasahlpa can review.

Thanks @vogelpi and @nasahlpa for the reviews. I've made the required changes.
Thanks @nasahlpa for explaining to me how to generate the binaries with the new process.
I've updated ot-sca PR 295 with the new binaries. Both serial and uJSON captures work on my local CW310 setup.


Fix heap overflow from variable length arrays

FiniteElementCollection contains a few instances where variable length arrays are being declared that can be of length zero. When a variable length array is of length zero, it does not result in a nullptr. it instead declares a single byte rather than a pointer matching the size of the data type. This can result in some rather opaque bugs https://github.com/mfem/mfem/issues/3671.
Recreating this on godbolt gives the same error message, and looking a little further, I have just about convinced myself this is undefined behaviour. The consequence of this is, is that when these arrays are checked against nullptr, they return false, as they do address memory, just a piece of memory totally inconsistent with the type, causing the overflow when they're dereferenced.
The best solution here is that VLA should never be used, but that's a much more sprawling change, so instead to address this error I propose this much smaller bug fix of placing such initializations in fe_coll.cpp behind a ternary checker. I also opted to put the check on all instances for consistency, even those where with some logic the VLA size can be deduced to always be non-zero.

There is some confusion regarding variable-length arrays (VLAs): VLA is a stack allocated array with size that is only known at runtime, e.g.
void test(int s)
{
   int a[s]; // a is VLA
}

VLAs are NOT part of the C++ standard -- some compilers support them since they are allowed in C99. I think we don't actually use VLAs anywhere in MFEM.
The arrays you refer to are dynamic arrays and having size zero is acceptable in C++. One problem that can happen is if one tries to dereference such a pointer. Was that the issue you ran into in #3671?

I think the issue is a little different to dereferencing a nullptr. The triggering example for me is this line https://github.com/mfem/mfem/blob/master/fem/pfespace.cpp#L2165 which is checking if the array is a nullptr before addressing it, but even if that array was of length zero, then this still says it is not a nullptr, and ind[0] after the short circuit triggers the overflow. new int [6*TriDof] (line) is allocating a single byte when TriDof=0, which is incompatible with the 4 bytes required for an int pointer, but isn't a nullptr, hence when checked for null status, it's ok, but when accessed with [0] it causes a heap overflow.
Tbh, I wasn't very familiar with VLAs (as I am a C++ programmer by background and avoid new and delete like the plague, so have basically not encountered them before debugging this), if I mislabeled those it was in error. Either way, whatever they are called, new int x[0] is assigning a one byte region, when reading it I would've believed it would have been assigning nothing (nullptr). I believe this interpretation was also being assumed by that line in pfespace.cpp above, hence the heap overflow. The godbolt link demonstrates the issue, as sizeof() and the nullptr comparison are both indicating there's an address assigned (somewhat counterintuitively in my opinion). I strongly suspect this assumption is elsewhere in the code base, this was just the first time I ran into it.

I think the dereferencing of ind in https://github.com/mfem/mfem/blob/f1c83d85cb933b526fd9ddb3a077f3f30e4f56a4/fem/pfespace.cpp#L2165
is an indication of an error somewhere else. If the fec has 0 dofs associated with a mesh entity, there should be no tries to dereference the ind pointer.
Basically, it looks like the ids indicates that there are dofs associated with entities of dimension ent but the fec has 0 dofs for that dimension.

That would be quite plausible, as I just recently discovered a situation where exactly that could occur: when a triangle with no dofs is incorrectly indexed into by searching the var_face_dofs structure, see here. I believe this can be fixed by tweaking the use of std::lower_bound(begin, end, val) to std::upper_bound(begin, end, val) - 1, as the entity -> dof relation there can look like 0,0,0,1,2,3,3,3,4, for tri, tri, quad, quad, quad, tri, tri, quad if p = 2.  Upper bound will catch the first index beyond the interval matching the predicate, whereas lower_bound is the first within the interval. Running this locally the np=2 adapts and the resulting solution and mesh looks as expected.
This change looks like it solves my issues for np = 2, and thinking about it, might resolve this scenario too. However, for np = 3, can still end up with R2 waiting for dofs to arrive after R0 and R1 are complete, but that's unrelated to this behaviour.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

@hughcars, apart from the changes in NeighborRowMessage::Decode, is there a use case where the other changes in this PR are still necessary? As discussed above, I don't think the allocation of dynamic arrays of size 0 is really a problem.

The main use I can think of, is for warning if an array could or could not be accessed. As it stands, a developer cannot tell if x[0] will segfault or not before doing it, as the dynamic array has undergone pointer decay, so the size information has been lost, and a pointer to a dynamic array will report its size as the same for 0 or 1 elements, see https://godbolt.org/z/hq3a4YrWe.
Yes, with the other fixes to pfespace.cpp, this particular trigger for the failure should never be activated, but leaving the developer without the means to safeguard oneself, turns debugging it into hunting a heap overflow, rather than a nice MFEM_ASSERT(x != nullptr, "!"); which can be easily found without recompiling with sanitizer turned on. That there already existed a nullptr check on the output of these arrays, suggested I was not the first person to assume that new int x[0] would have size of 0/be a nullptr.
Really this ends up boiling down to the dangers of the implicit pointer decay of C style arrays.

Okay, maybe you are right -- having these 0-size pointers be NULL could be helpful and should not have any negative effects.

The macos failures appear to be unrelated to the changes in this pr.

Restarting the checks after a fix in one of the gh-actions.

Merged in next for testing...

@mlstowell, this looks ready for merge in master.


Working Directory

When I set the job directory using
defaults:
  run:
    working-directory: ./ARM_Lite_Compiler

the action fails in Configure, which yields
CMake Error at CMakeLists.txt:3 (PROJECT):
  Running

   'ninja' '--version'

  failed with:

   No such file or directory

CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
See also "/home/runner/work/ARM_Lite/ARM_Lite/ARM_Lite_Compiler/build/CMakeFiles/CMakeOutput.log".
CMake Error at /home/runner/work/_temp/5f649009-61c2-41ed-8dbc-2bbb832adf25:32 (message):
  Bad exit status

Error: Process completed with exit code 1.

The work I am doing is here.

Instead of setting working-directory: ./ARM_Lite_Compiler you can set:
         COMMAND cmake
            -S ./ARM_Lite_Compiler
            -B build

At https://github.com/Teddy-van-Jerry/ARM_Lite/blob/master/.github/workflows/build_compiler.yml#L130

It works as expected. Thank you so much!


Use 1ES pool

All Windows and Linux pipelines should use 1ES pools by default, unless they need to run in the DevOps Pool for a specific reason.

@jhendrixMSFT: I don't have permissions to merge, so you or one of the other maintainers will need to merge.


Publish dual RNA-Seq host-pathogen data sets in PATRIC

Host-‐Pathogen Response (Option 3)
Based on the survey of dual RNA-Seq data in public repositories, we will select those that we consider to be exemplar data sets. These exemplar data sets will be chosen based on the pathogen, host, and associated publication(s) so as to select data that is most relevant to high priority NIAID goals. We will complete changes to PATRIC website that enable users to search for pathogen genomes for which dual RNA-Seq data is available, search for genes that are differentially expressed in the pathogen and the host, visualize the data using standard genome browser technology, and cross-link with other data available in PATRIC, as well as to link-out to the external data sources, namely NCBI SRA, and to the published literature.

03/15/16: Based on the paucity of public data sets that were found that met this criteria, we need to restructure this task accordingly to better match the type of data that are being generated.


Map shorthand unit names to their fully-qualified names

ESRI uses shorthand unit names instead of of the fully-qualified names like the web/RSS feeds use ex MED865  -> MEDIC 86-5. With enough datapoints, this should be as simple as just building an internal mapping.

Going to document edge-case unit names with their respective fully-qualified names here:

Shorthand
Fully-Qualified

FMPSP
FIREMARSHAL PSP


Fix toolbar dropdown menu issues

ListPopupWindow equipped a DropDownListView as its content list view, which extends from ListViewCompat, that's to say, items shown in it have a themed selection background (usually is RippleDrawable after API 21) themselves. The original item layout set a duplicated background which can lead to undefined behavior in some scenario. So I removed it.
Another enhancement is supporting Drag to  Open feature, that's very easy to achieve but can improve user experience. And to support this, I have to remove the background I mentioned before, because...if don't do that, the final effect can be so cool.

Oops, I forgot to sync with the upstream origin, and the first commit shouldn't be there again, I will close this and reopen a new PR.


Strange workaround (*hack) for converting PATCH to PUT in RESTAdaptor

See buildRequest method, line 816

I think this is here because at the time I couldn't test PATCH requests and decided to convert to testable PUT requests and come back to it.  It seems now like it is easily testable with jsonplaceholder api and should work similarly to PUT.

I think this is here because at the time I couldn't test PATCH requests and decided to convert to testable PUT requests and come back to it.  It seems now like it is easily testable with jsonplaceholder api and should work similarly to PUT.


Restored window position should be dpi-adjusted

To repro:

Run DragablzDemo.net45 on a screen running at a DPI higher than 100% (96dpi). The issue is more evident if you have two monitors and move the window to a monitor to the right so the coordinates involved are larger.
Maximize the "Simple form of bound tabs" window.
Click and drag down in the title bar of the maximized window to trigger a Restore and the window will either disappear or be positioned incorrectly.  The window can disappear b/c it's Top,Left coordinates are off-screen.

Looking through other issues, I suspect that this may be what's described in issue #140 though the wording isn't too specific so I'm not positive.


Dark mode (maybe even as default?)

GitBook seems to have an option for dark mode: https://docs.gitbook.com/tour/customization/space-customization#theme-mode
Since the editor and the website are in dark mode by default, too, I think I'd make sense for the docs as well.

PRs being in sync is a big one, stylistic consistency is one. I'd love to be able to reference data from zed/zed.dev in the docs directly as well.
I think if we used something else I'd just move the docs to github itself otherwise. Simplest option. No plans right now though.

Any updates on this?


packlite option for non-developers

Can we also add an option for a packlite?
Before we had this, I had written a bash script that ran git archive on each dependency to extract tarballs.
pushd KVTree.git
git archive --format=tar --prefix=kvtree/ -o ../archive/kvtree.tar master
popd
pushd spath.git
git archive --format=tar --prefix=spath/ -o ../archive/spath.tar master
popd
...

It doesn't capture the git history, so it's not as useful for SCR developers.  But it generates in a top level tar file that is closer to 5MB instead of 320MB.
  5099520 May  7 15:56 archive.tar
320807060 Jun 12 13:21 scr-top-dev.tgz

Some code teams want to include this tarball as part their own source repo, so having that smaller file will help.

Sure thing!
I added the make pack-lite command, and simplified things a bit (I hope).
Once you’ve run make clone you can then do make dist, make pack, or make pack-lite. Each creates an scr-top-xxx.tgz.
After you untar the scr-top-xxx.tgz file, you can run make unpack and make build.
As an aside, the git-all script is designed to pass a git command to each of the components. It wouldn’t quite work if your use-case since each archive command requires a different prefix argument, but I thought I’d mention it.

Wow, that was fast turn around!  Thanks!

let me know if you want any more features :)


FakeBackends are inefficient because noise model is repeatedly built

What is the expected enhancement?
When using FakeBackends there are multiple calls to noise_model.from_backend that lead to hug performance hits. E.g from profiling one VQE run, 40 out of 58 seconds is just building noise models repeatedly:
201    0.019    0.000   40.505    0.202 noise_model.py:209(from_backend)

However, doing the building direct in Aer removes all of this overhead.  Namely:
sim = Aer.get_backend('aer_simulator')
nsim = sim.from_backend(FAKE_BACKEND_INSTANCE)

is dramatically faster.
I am guessing this is because FakeBackend was created before Aer had this capability.

This should be fixed by: https://github.com/Qiskit/qiskit-terra/pull/7912 I've tagged it for backport to 0.20.1 (which will hopefully go out next week).


feat: Add new profile modal popup to access profile page when wallet connect

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3359

PR #3360

PR #3361

PR #3362

PR #3363

PR #3364

PR #3365

PR #3366

@wraeth-eth merged this pull request with Graphite.


NestedScrollView's outer scrollable jumping with BouncingScrollPhysics due to double precision errors

This PR fixes scrolling issues with NestedScrollView using the BouncingScrollPhysics. In one of the steps of the calculation, we can reach a state where the position of the inner scrollable is set to a double value that falls within precisionErrorTolerance of 0 but we were using == with 0 rather than checking for a precision. My posts in the linked issue show the current behavior, and how I reached to conclusion (the code in this PR). This PR only addresses the "jumping" of the outer scrollable.
Fixes #136199
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

I have not finished a test for this since I have never done so and therefore have 0 experience writing tests in Flutter, so any help there would be appreciated. I am also not sure how to test double precision errors in general. I did run all the nested_scroll_view_tests.dart locally and there are no failures.

@Michal-MK thanks for the contribution. Looks like this PR is failing some checks. Can you please take a look and fix those up? Thanks!

@goderbauer I can decode the Linux Analyze check, which is clear to me, I have a TODO in tests, but the issue is I do not know how to write tests for this fix. For the other 6 failing checks I have no idea what the output is trying to tell me. I looked at the "undestading a LUCI build failure", but that to me appears to describe something else.
â��05:52:02â�� Test failed.
â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��â��
For your convenience, the error messages reported above are repeated here:
  ð���  ð���

Apart from tree-status all tests should pass, I'll wait and see if a test for this will be needed/asked for.

Okay so I tried something, the issue is that I do not fully understand the test as I just while(true)'d until a set of inputs caused the if to be hit with the faulty values. Then I searched for a place where the wrong value would appear, and it appeared in the outer scrollable's controller.
I replaced another == 0 check that may render the first one obsolete, and added a test that will fail if the outer scroll controller's listener is called. Since that is the place I found manifests the precision error to the user.

This is the output before the fix.
After the fix it does not trigger the outer controller's callback:

All other tests in NestedScrollView pass.

Tests are failing randomly, as noted here: https://github.com/flutter/flutter/pull/138319#issuecomment-1813239714. No idea how to decode the output. I just merged current master into it and it worked again. That was the case even before I added my test so i do not know what is causing it.

So it seems that Linux web_canvaskit_tests_4 and Linux web_tests_0 are now failing consistently, but their output is still illegible: â��â��â��â��â��â��â��â��â�

After spending more time with it I did find the tiny "show more details" button. There I finally got the actual output. Hmm why is GitHub's so broken then...? I modified the test to hopefully pass the two failing cases.

Happy new year to you and all of the Flutter team. Awesome framework, glad I could contribute. :)

@Piinks  Is there something on my part that needs doing? The "Google testing" test is now running for more than 5 days, is that intended?

This still needs a second review, but I have rebased it to kick the Google testing shard. 👍
Following up on the second review in https://discord.com/channels/608014603317936148/804482791517192273/1193960740856545290

Hmm, I was not able to rebase since there is a merge conflict. Can you rebase with the master branch @Michal-MK?

Struggling with the rebase, give me a minute :)

So, I finally won agains git rebase, unfortunately, the bot added some people for review, that was not intentional. @Piinks I do not see the Discord channel and I have not found any info on how to enable it on what is publicly available.

Thanks for rebasing! Looks like that channel is restricted to contributors with commit access (members of flutter-hackers): https://github.com/flutter/flutter/wiki/Contributor-access
Sorry for sharing something inaccessible. The channel is for seeking reviewers, so it should get picked up soon. 👍


docs(overview): fix overview flow example

we noticed a minor compilation error in the docs overview guide.
We official made it with the amazing NgNepal community!
this closes #4570

Pull Request Test Coverage Report for Build 8101

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.2%) to 96.962%

Files with Coverage Reduction
New Missed Lines
%

src/internal/observable/fromPromise.ts
1
87.5%

src/internal/observable/fromIterable.ts
2
95.24%

src/internal/observable/fromObservable.ts
2
83.33%

Totals

Change from base Build 8098:
0.2%

Covered Lines:
5777

Relevant Lines:
5958

💛  - Coveralls


Add regexp function - regexp_instr()

Is your feature request related to a problem or challenge?
Add regexp_instr()
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
discussed in #11946

take


Admin pass

Hello, what is the password for the administrator panel?

Is it possible to participate without sign in
Or option open quiz with out login?

Is it possible to participate without sign in Or option open quiz with out login?

No. Ideologically, the admin needs to log in to manage quizzes, and the student needs to provide the information to attempt a quiz.


Feature request: Disabling keepalive option for OpenVPN clients

OPNSense automatically adds the --keepalive option to the OpenVPN client, but there is no option in the GUI to disable it.
The ping and keepalive options are mutually exclusive and some VPN providers like NordVPN rely on the ping option instead of keepalive, which results in reconnects every other minute.

Hi @Chrzi,
Sounds reasonable. I assume this is most relevant for OpenVPN clients so we only need the option there?
Cheers,
Franco

Sorry for the late answer.
I did read the man page of openvpn again and a bit closer this time. --keepalive <interval> <timeout> is for clients an alias for --ping <interval> bundled with --ping-restart <timeout>.
By adding the keepliave into the advanced config you can set the values however you want (except 0) and there is no need for another option to disable it.
https://community.openvpn.net/openvpn/wiki/Openvpn24ManPage

@Chrzi ok, thank you for posting the answer 👍


Form builder: Submit data + additional types for FormInputFieldWidget

Please note, that further validations (e.g. one form attribute may only be set once) will be addressed in a different PR.
Documentation for the new types: https://docs.google.com/document/d/1Dbr7Y-ODFsQ4YSyYtgfyWM74YwkZTCovSz4xstvkVzc/edit#heading=h.icv9ytbve50m

FormCheckboxWidget

REVIEW (2870d32)

REVIEW (4138ec3)
Let's discuss validation message tone independently with Andreas.


Add Slash cmd search

add search command like soul music , if user enter word it searchs from yt and display it  like this

🤣

ncs kinda sus tho

😄

https://discordjs.guide/interactions/autocomplete.html#enabling-autocomplete this is autocomplete thing idk how to add yt search to this


fix for comparison of NSNumber and NSDecimalNumber

it is a fix for issue #118, where .equal() provides false negative results when testing equality of a NSDecimalNumber and a NSNumber

Rockin, thanks for the PR!


Initial implementation of blend modes

Add blend and composition mode enums to API
Mirror these in the shaders
Add new public blend function to PietGpuRenderContext that mirrors clip
Plumb the modes through the pipeline from scene to kernel4

This is certainly caused by my incorrect alpha computation in mix_compose (https://github.com/dfrg/piet-gpu/blob/90774f1f46305a91ed99b770b301aedf6688dbe0/piet-gpu/shader/blend.h#L257). This generates a^2 alpha components which are obviously wrong. If you replace it with the correct vec4(cs, as) * fa + vec4(cb, ab) * fb then the cli test works as expected, but some of the blend modes render with artifacts.

Got it, thanks. I'm interested in coming back to this, but don't think it's critical path. Since my current work is based on this rev, what we might do is merge this, get variable encoding in, and develop a "more correct" blending computation as a separate PR. It sounds like that should be localized to fine rasterization, so unlikely to cause serious merge conflict.


Fatal error: Uncaught Error: Call to a member function execute()

When I followed wiki's instructions to install ezxss, the following error occurred when accessing exss.
https://github.com/ssl/ezXSS/wiki/Installation#installation-with-docker-compose
Fatal error: Uncaught Error: Call to a member function execute() on bool in /var/www/html/src/Database.php:64 Stack trace: #0 /var/www/html/src/Database.php(91): Database->fetchAll('SELECT setting,...', Array) #1 /var/www/html/src/Route.php(331): Database->fetchSetting('killswitch') #2 /var/www/html/src/Route.php(31): Route->verifySettings() #3 /var/www/html/init.php(86): Route->__construct() #4 {main} thrown in /var/www/html/src/Database.php on line 64

Hi @shmilylty, please share the following information to help us troubleshoot.
What version of ezxss are you trying to install?
What version of docker-compose are you using docker-compose --version?
Can you post the output of docker-compose up without the -d flag?
Thanks

docker-compose --version
docker-compose version 1.27.4, build 40524192

docker-compose up
Creating ezxssdb ... done
Creating ezxss_ezxss_1 ... done
Attaching to ezxssdb, ezxss_ezxss_1
ezxssdb    | 2020-10-26 05:21:23+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 8.0.22-1debian10 started.
ezxssdb    | 2020-10-26 05:21:23+00:00 [Note] [Entrypoint]: Switching to dedicated user 'mysql'
ezxssdb    | 2020-10-26 05:21:23+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 8.0.22-1debian10 started.
ezxssdb    | 2020-10-26T05:21:24.938402Z 0 [System] [MY-010116] [Server] /usr/sbin/mysqld (mysqld 8.0.22) starting as process 1
ezxssdb    | 2020-10-26T05:21:24.972059Z 1 [System] [MY-013576] [InnoDB] InnoDB initialization has started.
ezxss_1    | AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
ezxss_1    | AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
ezxss_1    | [Mon Oct 26 05:21:25.212033 2020] [mpm_prefork:notice] [pid 1] AH00163: Apache/2.4.38 (Debian) PHP/7.4.11 configured -- resuming normal operations
ezxss_1    | [Mon Oct 26 05:21:25.221081 2020] [core:notice] [pid 1] AH00094: Command line: 'apache2 -D FOREGROUND'
ezxssdb    | 2020-10-26T05:21:25.457776Z 1 [System] [MY-013577] [InnoDB] InnoDB initialization has ended.
ezxssdb    | 2020-10-26T05:21:25.616664Z 0 [System] [MY-011323] [Server] X Plugin ready for connections. Bind-address: '::' port: 33060, socket: /var/run/mysqld/mysqlx.sock
ezxssdb    | 2020-10-26T05:21:25.722046Z 0 [Warning] [MY-010068] [Server] CA certificate ca.pem is self signed.
ezxssdb    | 2020-10-26T05:21:25.722945Z 0 [System] [MY-013602] [Server] Channel mysql_main configured to support TLS. Encrypted connections are now supported for this channel.
ezxssdb    | 2020-10-26T05:21:25.726771Z 0 [Warning] [MY-011810] [Server] Insecure configuration for --pid-file: Location '/var/run/mysqld' in the path is accessible to all OS users. Consider choosinga different directory.
ezxssdb    | 2020-10-26T05:21:25.751611Z 0 [System] [MY-010931] [Server] /usr/sbin/mysqld: ready for connections. Version: '8.0.22'  socket: '/var/run/mysqld/mysqld.sock'  port: 3306  MySQL Community Server - GPL.

docker ps -a
CONTAINER ID        IMAGE                         COMMAND                  CREATED              STATUS                    PORTS                                                          NAMES
70636c53538c        ezxss_ezxss                   "docker-php-entrypoi…"   About a minute ago   Up About a minute         <IP_ADDRESS>:8080->80/tcp, <IP_ADDRESS>:4443->443/tcp                ezxss_ezxss_1
d7b00d9a709f        mysql                         "docker-entrypoint.s…"   About a minute ago   Up About a minute         3306/tcp, 33060/tcp                                            ezxssdb

Thanks @shmilylty.
I was able to replicate the error using the same setup/version. It seems it's not exclusive to docker but rather a recent code change in ezxss 3.9. Try again with ezxss version 3.8 and let me know if it this works for you.

There is no problem using docker to build version 3.8 before, please fix this bug, I want to use the latest version, thank you.

@shmilylty Try again with version 3.9 but this time in Database.php change line 62 from
$this->DB->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);
to
$this->DB->setAttribute(PDO::ATTR_EMULATE_PREPARES, true);
This seemed to solve the issue for in my test case, however I am not 100% why. @ssl will need to weigh in.

It can be used normally after modification.

Glad to hear, keep in mind this is definitely not a permanent fix but rather a temporary one until a proper fix is pushed.

We look forward to your release of the repaired version.

Fixed this in https://github.com/ssl/ezXSS/commit/9e91b20a507948eeaf6e0cb947098b567f92da23 - thanks for letting me know!


Fix #10: Add hover:text-slate-100 class to all links in header

Fixes #10
I have added the hover:text-slate-100 class to the links in the Header.tsx file.

Pull request merged

This is an automated message from devgallery.io [BOT].

Thank you for your contribution! Your pull request has been merged and your name has been added to the List of Contributors in README.md.
Please leave a star if you enjoyed working on this project.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects here.


⚠ Turnos 24x7 has degraded performance

In e267d1d, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 963 ms

Resolved: Turnos 24x7 performance has improved in ca5a783.


Add a license (e.g. MIT)

Hey, thanks for the awesome package!
Could you add a LICENSE file with a permissive license, such as MIT or Apache-2.0? This would allow people to freely use and modify your code, and is also a blocker for #4.
Thanks!

done! but it's not the main thing that blocks #4

Thank you!


4 ROI selection produces 12 TIFF images

The TIFF images appear to be duplicates.
It could be related to a click-order issue with multiple exports. See attached export.json.
export.json.zip

I'll try to explain bugs (as I understand them) when appropriate since I think it is illustrative for a project like this.
This is caused by a bug in the javascript code.
function onHyphaePick(b64_hyphae, onPathClose) {
    hyphae.src = 'data:image/png;base64,' + b64_hyphae;
    hyphae.onload = () => {
        let imgInstance = new fabric.Image(hyphae);
        canvas.clear();
        canvas.setBackgroundImage(imgInstance, canvas.renderAll.bind(canvas));
        onDrawStart();
        canvas.on('path:created', (d) => onPathClose(d));
    }
}

The last line there is the culprit.
canvas.on('path:created', (d) => onPathClose(d));

Apparently the on() function will append callbacks when called repeatedly. So each time this onHyphaePick() function is called, the core processing event that happens in onPathClose() will be done another time.
This explains why each time a new image was rendered after clicking the update button, the number of paths created from a single lasso would be an increased multiple.
The fix was to simply move the callback registration line outside the function so it only runs once.


feat(packages): add additional exports to support apps/whale-api

What this PR does / why we need it:
Since we cannot rely on /dist imports within jellyfish we need to provide the expected values out of our core packages for use in our apps. These are plucked exports as export * from '...' results in naming conflicts.
Which issue(s) does this PR fixes?:

Partially Fixes #977
Additional comments?:
This is a precursor PR for:

https://github.com/DeFiCh/jellyfish/pull/1168

Setting the PR back to draft as I am trying out another path of least resistance suggested by @fuxingloh

Closing this as its not longer required, in migration changing /dist to /src in the imports seems to do the trick.


Feature request: Appending query strings to domains

I'd like to be able to hook onto one or all domains specified in my config.yaml file and append a query string to a specified domain.
Something like:
domains:
  current:  "https://www.example.com"
  new:
    url: "https://www.example.com"
    append: "?env=dev"

paths:
  home:     /
  contact:  /contact-us

With the output capturing urls  as follows:
  current:  https://www.example.com/
            https://www.example.com/contact-us

  new:      https://www.example.com/?env=dev
            https://www.example.com/contact-us?env=dev

The documentation doesn't mention anything like this but is this something I could achieve now and if not is this a feature that might be developed in the future?

This would be AWESOME. Am working on a project where the only way to load the test theme is to pass a 'preview' parameter to prod. So the configuration would have to know to ONLY pass the param to one domain and not the other. Just as you have in your example above.
Please, please, please!

Would also be useful for busting caches, such as on sites behind a reverse proxy.


[stable/prometheus-postgres-exporter] add ability to pass additional container arguments such as log level

Signed-off-by: Yash Bhutwala<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

@sameersbn @prydonius can you please take a look?

/ok-to-test

@cpanato I made the fix you asked for.  Can you please review again and lgtm if possible?

/hold
Let’s hold PRs to stable prometheus charts for a short time until prometheus-community/community#28 (comment) is resolved. If you want to help with that effort, to test etc, that would be great!

🚧 This chart is now deprecated.
Further development has moved to prometheus-community/helm-charts.
If applicable, please re-open this PR there. Thanks!


Solax error config

Just connected a Solax to the Wifi network and I'm trying to get the data out of the system. I have seen a previous bug #26626 that is similar to this problem, but it is slightly different.
If I go to the http://<ip address>/api/realTimeData.htm I get an error saying that it is 404 Not Found.
The model of the Solax is SL-TL5000T
2019-10-07 14:09:02 ERROR (MainThread) [solax] Received non-JSON data from Solax API endpoint
2019-10-07 14:09:02 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up platform solax
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 150, in _async_setup_platform
    await asyncio.wait_for(asyncio.shield(task), SLOW_SETUP_MAX_WAIT)
  File "/usr/local/lib/python3.7/asyncio/tasks.py", line 442, in wait_for
    return fut.result()
  File "/usr/src/homeassistant/homeassistant/components/solax/sensor.py", line 29, in async_setup_platform
    resp = await api.get_data()
  File "/usr/local/lib/python3.7/site-packages/solax/__init__.py", line 140, in get_data
    3)
  File "/usr/local/lib/python3.7/site-packages/solax/__init__.py", line 107, in async_solax_real_time_request
    raise SolaxRequestError
solax.SolaxRequestError

This should be fixed in the next version of home assistant. The fix was included in #26705. This link describes how to run the development version so you can verify: https://www.home-assistant.io/docs/installation/updating/

I just updated to version 0.100.1 and i have the same issue

@phoniclynx what inverter do you have?
Can you please check this thread and try the curl requests mentioned: https://github.com/squishykid/solax/issues/5
Depending on your inverter you might need to connect your home assistant instance to the inverter’s wifi access point and then curl: curl -X POST http://<IP_ADDRESS>/?optType=ReadRealTimeData

Hi @squishykid, When I try the curl command, I just get 401 Unauthorized.  Where do I authenticate and sort that part out?
How would I set this integration up in Home Assitant, since I need to authenticate etc.
Currently I just have the following set up in my HA config:
sensor:
  - platform: solax
    ip_address: <IP_ADDRESS>

I'm starting to get a slightly different error message now
2019-11-27 14:57:40 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up platform solax Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 150, in _async_setup_platform     await asyncio.wait_for(asyncio.shield(task), SLOW_SETUP_MAX_WAIT)   File "/usr/local/lib/python3.7/asyncio/tasks.py", line 442, in wait_for     return fut.result()   File "/usr/src/homeassistant/homeassistant/components/solax/sensor.py", line 34, in async_setup_platform     api = await real_time_api(config[CONF_IP_ADDRESS], config[CONF_PORT])   File "/usr/local/lib/python3.7/site-packages/solax/__init__.py", line 37, in real_time_api     i = await inverter.discover(ip_address, port)   File "/usr/local/lib/python3.7/site-packages/solax/inverter.py", line 74, in discover     raise DiscoveryError() solax.inverter.DiscoveryError
Which is the same as @KiLLeRRaT

@squishykid Sorry it took ages for me to get to your message, but I didn't see it until now
I have SL-TL 5000T, if that means anything to you?

Log Details (ERROR)
Logger: homeassistant.components.sensor
Integration: Sensor (documentation
https://www.home-assistant.io/integrations/sensor, issues
https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A"integration%3A+sensor"
)
First occured: 9:37:26 AM (1 occurences)
Last logged: 9:37:26 AM
Error while setting up solax platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py",
line 179, in _async_setup_platform
await asyncio.wait_for(asyncio.shield(task), SLOW_SETUP_MAX_WAIT)
File "/usr/local/lib/python3.7/asyncio/tasks.py", line 442, in wait_for
return fut.result()
File "/usr/src/homeassistant/homeassistant/components/solax/sensor.py",
line 33, in async_setup_platform
api = await real_time_api(config[CONF_IP_ADDRESS], config[CONF_PORT])
File "/usr/local/lib/python3.7/site-packages/solax/init.py",
line 37, in real_time_api
i = await inverter.discover(ip_address, port)
File "/usr/local/lib/python3.7/site-packages/solax/inverter.py",
line 74, in discover
raise DiscoveryError()
solax.inverter.DiscoveryError
On Fri, 28 Feb 2020 at 21:35, stale[bot]<EMAIL_ADDRESS>wrote:

There hasn't been any activity on this issue recently. Due to the high
number of incoming GitHub notifications, we have to clean some of the old
issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check
if that solves the issue. Let us know if that works for you by adding a
comment 👍
This issue now has been marked as stale and will be closed if no further
activity occurs. Thank you for your contributions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/27319?email_source=notifications&email_token=AA5UBEGN7SYH2ZLK56T7O5LRFDVXDA5CNFSM4I6Q5WRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENIE2JA#issuecomment-592465188,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5UBEH5KCR3GMS4UOT7A5TRFDVXDANCNFSM4I6Q5WRA
.


Blocks by slot-index returns the same cursor

Bug description
Blocks by endpoint always returns the same results and cursor
ex.
https://chronicle.iota2-alphanet.iotaledger.net/api/explorer/v3/commitments/by-index/44249/blocks
Rust version

Rust version:

Chronicle version

Chronicle version (version number, commit, or branch):

Hardware specification

Operating system:
RAM:
Cores:
Device:

Steps to reproduce the bug
send subsequent requests to the provided link with returned cursor
Expected behaviour
cursor not returned on the last page
Actual behaviour
cursor always returned
Errors

Paste error here

@DaughterOfMars This issue can be closed