14 setting up tox

This is a simple version of tox which will test pytest with multiple python versions. Note, this does require pyenv to be installed. I also had a lot of trouble setting it up, so watch out for errors on mac or windows.

Gonna be honest, the makefile for ubuntu might just be completely wrong.


reset clock count when switching between clock tabs

issue : #179
I think this commit can solve the problem. I've done local test and it's running fine.
But if you come across any issues from this commit, I'd love to hear about it.

Looks good, thank you for the help! :)


zgrab not being detected as a bot

user-agent Mozilla/5.0 zgrab/0.x is not detected as a bot, which as far as I can research, it really should be.
https://developers.whatismybrowser.com/useragents/parse/285426-zgrab

Fixed in 2.1.1

Fixed in 2.1.1


Question: Setting up additional apps on top of (nextcloud+nginx reverse-proxy+letsencrypt + mariadb))

The question forums does not show up so I assume it was closed down.
I am using the docker compose file you made for (nextcloud+nginx reverse-proxy+letsencrypt + mariadb)
https://hackprogramming.com/how-to-setup-subdomain-or-host-multiple-domains-using-nginx-in-linux-server/
I am trying to add other containers for the reverse proxy to serve.  I planned on doing this by requiring domain.com/app1 ,  domain.com/app2, etc...   instead of having nextcloud on domain.com.    When all is done I will have a website hosted at domain.com that the reverse proxy serves but that website is not Nextcloud.   NextCloud will be at domain.com/app1 .  Other applications will be at domain.com/app2, etc...
In the docker compose file, I tried changing the virtual host to  VIRTUAL_HOST=domain.com/app1
but it broke the setup of NextCloud.  It was not possible to access then website.  When I did
VIRTUAL_HOST=domain.com
it worked fine.

VIRTUAL_HOST only accept host , no path ( you already know that)
The general approach to config /app1 /app2 using vhost.d -> location ^~ /app1 { and the nginx statement to reroute/forward to the other container }
Here is a answer to a similar question on StackOverflow:
https://stackoverflow.com/questions/39514293/docker-nginx-proxy-how-to-route-traffic-to-different-container-using-path-and-n


Some Suggestions

i have some suggestions for this mod:

let slabs can mixed double slabs together like mod Double Slabs (https://www.curseforge.com/minecraft/mc-mods/double-slabs)
can place torch on these slabs.(although on vertical slabs)
more fence like mod Custom Fence (https://www.curseforge.com/minecraft/mc-mods/customfence
can add custom images or gifs in game.
a computer can browse Internet
some toilet's furniture like washbasin and shower. Also compatible create's or mekanism's fluid system.

None of these things fit the vision I have for the mod.


Should not require requesterId

This code https://github.com/OpenConext/OpenConext-myconext/blob/53c1cee4f9247271cf1fe4a717e84075004af4f2/myconext-server/src/main/java/myconext/security/GuestIdpAuthenticationRequestFilter.java#L163 fails with a NPE when the AuthnRequest does not contain a Scoping element/RequesterID.
When using myconext with EB this will always be present, but when using it without a SAML proxy it's not. So please make it optional, e.g. by falling back to the remote entityID if there's no requesterId.

Fixed in https://github.com/OpenConext/OpenConext-myconext/commit/4cbc53ef0c36ca811d61f01b44438152a3f7bd07

Excellent, problem solved!


isNaN feature is added

It seems equal methods doesn't compare NaN. The PR is adding a NaN test
t.isNaN(NaN, 'It\'s NaN');
t.notIsNaN({}, 'It\'s not NaN');

I'm sorry. The spaces were deleted too. Should I create a new PR?

Thank you for this enhancement/bugfix, appreciate it.
And yes, please only keep the related changes in this PR.
You could do this by either create a new PR, or checkout the original version of the file and submit your changes again, this PR will be updated by your latest changes.

Thank you very much for the contribution!

@oshliaer Could you please also update the README.md to reflect your new methods?

@zixia Of course!
Please, let me time to re-fix my app https://github.com/oshliaer/golyfills/blob/master/app/tests.js#L42-L52


DBZ-5349 Run Oracle connector tests on pushes

https://issues.redhat.com/browse/DBZ-5349

Applied to main (=2.0) and branch 1.9


[WIP] Use ava instead of tape for unit tests

There is no big reason behind it except it's just modern tool. It's based on tape so writing tests isn't much different except some assertions are named differently. No need to specify t.end() for each test anymore.
It runs all tests in parallel so it's definitely faster to execute. It also has very useful watch mode which runs tests only for changed files so there is even more time saved during development.
Since there will be conflicts between this and PR #209 let's do that first and then I will update this.

Ava doesn't work in browsers, yet.

@ericelliott  I don't see any browser tests in stampit, so how does that relate?

Apologies for being so short. I'm not trying to be rude, I just have limited time to review PRs between my other responsibilities. I shut this down quickly because Ava is a non-starter for browser testing, and our tests must be browser compatible so that we get get the browser tests running again. Stampit used to have browser tests that ran in the cloud on Sauce Labs, and we need to make sure we can easily do that again.

Hey, Daniel. Thank you so much for the initiative. I believe ava is a better tool than tape.
Same time Eric is correct. We have plans to revive the browser tests. Here is the long living opened issue for it: https://github.com/stampit-org/stampit/issues/68
I really feel this PR is a great coding art. Tomorrow I'll investigate the ava usage inside browsers, there might be workarounds (via polyfills, or PhantomJS, or else). Will do my best and come up with more info sometime Friday or Saturday.

I don't see reason why it could not run in browser as well. If the problem is running in parallel, it possible to change it to serial execution. Otherwise I cannot imagine what would be difference.

There was a time when stampit tests were failing in IE8. Simply because i used a language construct not available in IE8. IE11 could become the new IE8.
As you suggested earlier, stampit will support Promises in the future. But IE11 and old Android <4.4.4 Browser do not support Promises natively.
It's just one of the possible scenarios. There might be other things, like iOS Safary, IE11, and Android browsers <5.0 do not support arrow functions.
etc. etc. etc.
 But people rely on us.

Getting Ava to work in browsers is not a small task. There is a long open Ava issue tracking the slow, not-a-top-priority progress.

Ok I see, but I believe that solution is simply running browser tests against bundle that is made to be executed in browser. With my rollup changes (#209) this should be a breeze.

I put a little time into trying to get browser tests working for stamp-utils. I felt like I was close, but I ran out of time to work on it. Getting the compose tests running from check-compose seems like the next logical step. =)

Ok I see, but I believe that solution is simply running browser tests against bundle that is made to be executed in browser. With my rollup changes (#209) this should be a breeze.

Yep! I was looking at doing the same thing with stamp-utils yesterday.

Sadly, getting Ava to work would be a lot harder than rolling a bundle. Ava uses a bunch of Node features that aren't available in browsers. Exploring all the alternatives and deciding on a strategy will take time, and I want to make browser tests a higher priority so that people can trust stamps not to break their browser apps.

Ok let's review what should actually happen to have those tests running in a browser.

bundle stampit code with all polyfills and stuff ... done
bundle test code that references bundled stampit
execute that bundle in the browser

Back in days I used tool Test'em which is essentially test framework agnostic framework which runs tests in browser natively. If we give that bundle from second step, it should work quite ok.

bundle test code that references bundled stampit

This doesn't have to be a separate step from #1. Just use the test index as the entry point for the test bundle instead of the stampit index, and Rollup will take care of the rest.
Last I looked, Test'em was local-only, and did not run on Sauce Labs for CI. Has this changed? I was looking at sauce-tap-runner. There's also Zuul, but it looked like a bunch of work manually hooking up the API.

Ok I see what you me although for ava there is no actual index.js because of the parallel running, but that can be added simply for sake of this.
There is this example for testem ... https://github.com/testem/testem/tree/master/examples/saucelabs so apparently it should be possible.

Yeah... if you want to work on the Ava stuff, a lot of work needs to be done in the Ava issue I linked earlier. Until Ava releases a browser-compat version, our tests will stick with Tape. Further browser test discussions should happen in https://github.com/stampit-org/stampit/issues/68
Thanks! =)

BTW, feel free to re-open this issue when Ava supports browsers. Ava seems legit, and I've been wishing I could use it in a real project -- sadly... all my current work is universal JS, which means I need working browser tests. );

Ok I will certainly try this, but the #209 should be merged first anyway to have rollup at hand.


useTimeZones not working in advanced filtering

Description
In API doc:
<useTimezones>: boolean = true
Set to true to be able to change timezones in the advanced filtering modal.
However in advanced filtering, timezone selection from the dropdown is always enabled regardless of <useTimeZones> set to false or true.
Should this be removed from the apidoc or should the functionality be implemented?
Version
Datagrid:  v1.0.0-beta.3

This api will most likely change after we discuss the timezone api/design today.

Close this after merging #773.  It was agreed that we would remove this additional property and force the timezone to show as abbreviated and text (not editable).

Closing as #773 has been merged.


Create new disk_type variable

GCP now has some kind of VMs that only accept SSD disks.
As the disk_type is a fixed value "pd-standard" we can use size like "c4-highcpu"
The proposed solution is to implements disk_type as a variable

Done:
https://github.com/Privatehive/gcp-hosted-github-runner/commit/5001a86fbf92322ab3b1d0ec71bce46b116e0a19


Infinite loop adapter

Hello,
I want to create an adapter that would scroll a list in a cycle.
The list is a classification for a tournament that I will show on a TV on an endless loop using some kind of animation to do the scrolling.
Is there any easy way to implement an adapter that would just simulate the effect ?
Thank you!!

@gpulido well you can simply add elements at the end and remove elements from the top. there is nothing in the adapter doing something like this.
seems like a very specific usecase

@mikepenz I know that is very specific, but not unusual,
For example it could be useful to implement an image carrousel showcase.
I was thinking if a "fastadapter wrapper" could be created to just generate the effect (no the animation, just the infinite loop).
Something like: InfiniteLooperAdapter that would wrap a fastadapter to just add the functionallity.
Seems that could be done manipulating the getItemCount and getItem methods for an Adapter:
https://stackoverflow.com/questions/31253555/how-do-i-create-a-circular-endless-recyclerview
Do you think it could be done that way using fastadapter?

@gpulido there is nothing preventing it, as basically everything is build upon the RecyclerView.Adapter interface. just add/remove the items, I don't see a problem with this.
you can also just move them for example which would be even better

@mikepenz I've tried to use that trick, for example with a list of just one element. And the adapter creates the viewholder for the element, but then it fills the screen with empty holders. Only when you scroll up a new element is shown.
Seems like it is no capable of reuse the holder until it disapears from the screen.
The test code that I use was just an extension of the GenericItemAdapter on which I override the getItemType and just "transforms" all positions:
    @Override
    public int getItemCount() {
        return Integer.MAX_VALUE;
    }

    @Override
    public void onBindViewHolder(RecyclerView.ViewHolder holder, int position) {
        super.onBindViewHolder(holder,  getAdjustedPosition(position));
    }

    @Override
    public int getItemViewType(int position) {
        return super.getItemViewType( getAdjustedPosition(position));
    }

    public int getAdjustedPosition(int position)
    {
        return position % super.getItemCount();
    }

This is what is happening:

sure it is not able to re-use a viewholder if it is still used.
I also don't think that it's clever to have MAX_INT items. just listen in the scroll and move items from the top to the bottom and visa vers. (it should theoreitcally work)
but the whole thing is not FastAdapter related. as it behaves and acts like a normal adapter.

Thank for the tip, I know that at this point is not just "fasteradapter". Sorry for the inconvenience.
I will try to do as you say and add items dinamically on the onScroll event. Probably I will take a look to your EndlessScroll example.
My main concern is that the list represents a "live" classification on a tournament that is being shown at any time but that is being refreshed (the order of elements) several times. So any time the "classification" changes I'm clearing the adapter data and refilling, and I don't know how it would work with the scroll adding.
But as you say, this is not just fasteradapter.
Again thank you for your help.
You can close this issue if you want.
Gabriel

@gpulido
Have you managed to implement this in meantime? If so, would you share your solution please? Thanks


A11y problems w/ Carousel Pagination on ExploreData

These issues were filed on the a117 report spreadsheet

[ ] Explore the data	SR	[A11Y - Health Equity Tracker]: No label associated with pagination buttons under Explore the Data page_Web_1.1	"ENVIRONMENT:
macOS 11.3 | Chrome Version 90.0.4430.212 (Official Build) (x86_64) | Voiceover
Chrome OS Version 91.0.4472.60 (Official Build) beta (64-bit) | Chromevox
PRE REQUISITES:
Turn on the respective screen reader
STEPS TO REPRODUCE:
1). Open https://healthequitytracker.org/exploredata
2). Linear navigate to the pagination buttons on the page
3). Observe the announcement.
EXPECTED RESULT:
Buttons should have a label associated with it and the same should be assigned properly by screen reader.
OBSERVED RESULT:
Pagination buttons under Explore the Data page don't have label. Screen reader announces them only as button.
Replication video attached"	https://drive.google.com/file/d/1gJ61P0fK5CH1Npxp7aAk2NtQroOlUMBD/view?usp=sharing	"Issue is replicable in the following Android environment:
Device: Pixel 4
Android Version: 11
Chrome Browser Version: 91.0.4472.101"

[ ] Explore Data > Compare rates of x between y and z	SR	Missing label for page navigation dots.	"ENVIRONMENT:
OS | Browser | Screen Reader
macOS Big Sur:11.4|Version 90.0.4430.212 (Official Build) (x86_64) | VoiceOver
ChromeBook | Chrome Version: 90.0.4430.218 (Official Build) (64-bit) | ChromeVox
Window | Chrome Version 91.0.4472.77 (Official Build) (64-bit) | JAWS/NVDA
Window | Firefox Version: 87.0 (64-bit) | JAWS
PREREQUISITE:
Enable the respective screen reader.
STEPS TO REPRODUCE:

Open this Url: https://healthequitytracker.org/
Navigate to the explore the data tab and select.
Navigate to the page navigation dots to observe this issue.
ACTUAL RESULT: Missing label for page navigation dots. Screen reader is announcing button only.
EXPECTED RESULT: Provide proper label for page navigation dots."	https://drive.google.com/file/d/1rZthVjjWhVo2m5-zApBoka9QgUflRLb0/view?usp=sharing

[ ] Explore Data > Compare rates of x between y and z	SR	Missing description of page navigation dots.	"ENVIRONMENT:
OS | Browser | Screen Reader
macOS Big Sur:11.4|Version 90.0.4430.212 (Official Build) (x86_64) | VoiceOver
ChromeBook | Chrome Version: 90.0.4430.218 (Official Build) (64-bit) | ChromeVox
Window | Chrome Version 91.0.4472.77 (Official Build) (64-bit) | JAWS/NVDA
Window | Firefox Version: 87.0 (64-bit) | JAWS
PREREQUISITE:
Enable the respective screen reader.
STEPS TO REPRODUCE:

Open this Url: https://healthequitytracker.org/
Navigate to the explore the data tab and select.
Navigate to the page navigation dots to observe this issue.
ACTUAL RESULT: Missing description about the page navigation dots. Screen reader is only announcing button but no indication that which page(dot) is selected.
EXPECTED RESULT: Provide an appropriate description to indicate the presence of dots and which page is selected."	https://drive.google.com/file/d/1rZthVjjWhVo2m5-zApBoka9QgUflRLb0/view?usp=sharing

[ ] Explore the data	DOM	[A11Y - Health Equity Tracker]: Improper DOM order under Explore the Data page_Web_1.8	"ENVIRONMENT:
macOS 11.3 | Chrome Version 90.0.4430.212 (Official Build) (x86_64) | Voiceover
Chrome OS Version 91.0.4472.60 (Official Build) beta (64-bit) | Chromevox
PRE REQUISITES:
Turn on the respective screen reader
STEPS TO REPRODUCE:
1). Open https://healthequitytracker.org/exploredata
2). Navigate through next & previous buttons on the top portion of the page
3). Observe the navigation & DOM order
EXPECTED RESULT:
Screen reader focus should move in a logical manner, ideally from left to right.
OBSERVED RESULT:
Screen reader focus goes to Next button and then to previous button, ideally it should move from previous to next button as previous button is placed before next button.
Replication video attached"	https://drive.google.com/file/d/1t-YsMBv1VGFayBApF5_kYzKWI4eQV035/view?usp=sharing	"Issue is replicable in the following Android environment:
Device: Pixel 4
Android Version: 11
Chrome Browser Version: 91.0.4472.101"

npm package providing the carousel component seems to not offer a solution. I subscribed to a filed issue and will research further: https://github.com/Learus/react-material-ui-carousel/issues/105

Order of "previous" and "next" buttons being rendered inside package... not sure it's possible nor necessary to reorder...


HADOOP-18470. More in the index.html about security

Expands on the comments in cluster config to tell people they shouldn't be running a cluster without a private VLAN in cloud, that Knox is good here, and unsecured clusters without a VLAN are just computation-as-a-service to crypto miners

For code changes:

[X] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

+@lmccay ...comments about Knox OK?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 46s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ trunk Compile Tests _

+0 :ok:
mvndep
15m  6s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
33m 53s

trunk passed

+1 :green_heart:
mvnsite
1m 49s

trunk passed

+1 :green_heart:
shadedclient
76m  0s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 26s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 21s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
1m 37s

the patch passed

+1 :green_heart:
shadedclient
27m  4s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
asflicense
0m 33s

The patch does not generate ASF License warnings.

108m  0s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5383/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5383

Optional Tests
dupname asflicense mvnsite codespell detsecrets

uname
Linux 78ba10114c9e 4.15.0-200-generic #211-Ubuntu SMP Thu Nov 24 18:16:04 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 988b0aa0226ed82e6a844f1488a6aaa9ceac415d

Max. process+thread count
591 (vs. ulimit of 5500)

modules
C: hadoop-project hadoop-common-project/hadoop-common U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5383/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@mukund-thakur @mehakmeet @HarshitGupta11 @lmccay I'm planning to add this as a continuation of the other 3.3.5 release note changes; does anyone see anything wrong with it before I do the merge?


[5.8] do not JSON encode strings for cast model attributes

if you cast a Model attribute to 'array', 'json', 'object', or 'collection', they are stored as JSON in the database.  To set them on the model, you must pass either an array, object, or Collection. Currently you cannot pass a properly formatted JSON string that would be decoded into one of the types.  This PR allows you set a property to  a JSON string, and to skip the encoding, so when it is decoded it comes out properly.
In 5.7 if we cast a property to an array, and pass an array, everything goes fine.
class User extends Model
{
    protected $casts = [
        'about' => 'array',
    ];
}

$user = new User();
$user->about = [
    'gender' => 'male',
    'height' => '72',
    'eyes' => 'brown',
];
$user->save();

dd($user->about);

returns the expected array
[
    'gender' => 'male',
    'height' => '72',
    'eyes' => 'brown',
]

However, if we pass a JSON string of the encoded array, we may expect to get the same array back...
$user = new User();
$user->about = '{"gender":"male","height":"72","eyes":"brown"}';

dd($user->about);

but it actually returns our original string
{"gender":"male","height":"72","eyes":"brown"}

What's happening behind the scenes is Laravel is running json_encode on the JSON string, which results in it being stored in the database like this:
"{\"gender\":\"male\",\"height\":\"72\",\"eyes\":\"brown\"}"

and when we pull it back out, it is getting json_decoded back to the original JSON string.
What I'm proposing for 5.8 is a breaking change, and that we not json_encode any strings that are passed to an attribute cast to any of the "json castable" values (json, array, object, collection).
Passing an array, object, or Collection would still behave identically as before.  However, passing a non-properly formatted JSON string will now result in a null value stored in the database.  I would not expect anyone to be storing normal strings as JSON in their database, so I would say this is probably a low probability of impact for the upgrade, but definitely something to consider.

I would not expect anyone to be storing normal strings as JSON in their database

I disagree, I think there are valid use cases. I've been doing that myself.
I see that your suggested feature is useful, but I don't think it justifies the breaking change. You could achieve the same behavior with a custom mutator.

can you provide an example of JSON encoding a string? I've struggled to come up with something.

I am using it to store setting values with different data types in a single database column.
By encoding the value as JSON, I can preserve the type:
class Setting extends Model {
    protected $casts = ['value' => 'json'];
}

$setting->value = true;
$setting->value = 123;
$setting->value = 'foo';

could you provide a real world example? I'm still struggling to understand how that would be useful.  How do you know what to do with the data if they all have different meaning?

Hmm, yeah I think I'll try to avoid the breaking change here for now. Would probably need to use a custom mutator. Or, hopefully in 5.8 we may be able to ship some kind of custom casts.

Thanks for the feedback guys. @staudenmeir if you've got that real world example, I'd still be interested to hear, so I can understand what type of use case people would have for that.


Allow ECD to redefine wording of requirements.

I don’t know whether the functionality to alter the ECD version of an SFR has been put in. If there has been, please advise on the schema (didn’t see an example of it in WIDS). There is a situation in the VPN Client Module where it applies. Specifically, the VPN Client Module defines two instances of FCS_CKM_EXT.2 as follows:
When GPOS PP is the Base-PP: The [selection: VPN client, OS] shall store persistent secrets and private keys when not in use in OS-provided key storage.
When App PP is the Base-PP: The [selection: TOE, TOE platform] shall store persistent secrets and private keys when not in use in platform-provided key storage.
What I would like to do is define the ECD version of the SFR more broadly with an assignment, so that each of these two versions can have their more restrictive selections formatted as assignments. So ideally I would want the Appendix D “original” version of the SFR to say something like this:
The [assignment: entity responsible for key storage] shall store persistent secrets and private keys when not in use in platform-provided key storage.
But in the current schema it looks like the ECD still picks up the exact text of the body text component with no ability to make the ECD version more generic.

Can we get an update on this? The ECD should not be a duplicate of the SFR and needs to be more generic as described above.

Fixed


Not Working on swift 2.3

Unfortunately the project doesn't build on 2.3
in dg_center:
> if usePresentationLayerIfPossible,
>             let presentationLayer = layer.presentationLayer() as? CALayer {
>             // Position can be used as a center, because anchorPoint is (0.5, 0.5)
>             return presentationLayer.position
>         }
>         return center

returns the following error:
Downcast from 'CALayer?' to 'CALayer' only unwraps optionals; did you mean to use '!'?
I am not sure why that downcast is there as layer.presentationlayer() seems to return a CALayer. If its unneeded then removing the downcast allows the project to build

I will try to look into that this week.

@gontovnik I have a fix for this on my swift-2.3 branch. Its very simple, here is a diff of my branch on your 1.0.3 tag.
I'm unable to make a pull request because the 1.0.3 tag doesn't exist as the head of any branches. If you could make a swift-2.3 branch based on your 1.0.3 tag I'd be happy to make a pull request for you to merge in.


Bug: error handling join predicate

Without the datafusion logical optimizer enabled, the join predicate would be stored in filter instead of the on field. This makes equal condition join break because we do not handle that.
We need to reconsider the logic of handling join conditions.
> explain select * from t1 join t2 on t1.t1v1 = t2.t2v1;
Join.on: []
Join.filter: Some(
    BinaryExpr(
        BinaryExpr {
            left: Column(
                Column {
                    relation: Some(
                        Bare {
                            table: "t1",
                        },
                    ),
                    name: "t1v1",
                },
            ),
            op: Eq,
            right: Column(
                Column {
                    relation: Some(
                        Bare {
                            table: "t2",
                        },
                    ),
                    name: "t2v1",
                },
            ),
        },
    ),
)

thread 'main' panicked at optd-datafusion-bridge/src/lib.rs:320:14:
called `Result::unwrap()` on an `Err` value: unsupported join filter: Some(BinaryExpr(BinaryExpr { left: Column(Column { relation: Some(Bare { table: "t1" }), name: "t1v1" }), op: Eq, right: Column(Column { relation: Some(Bare { table: "t2" }), name: "t2v1" }) }))

I think our join plan node supports filter by default? Just use a LogOp::And to make the filter into our join condition should be fine.

Solved by this pr: https://github.com/cmu-db/optd/pull/115


Errors in new project created in VS Code

STEPS TO REPRODUCE:

I clean-installed VS Code on Linux and ran it.
I added the Flutter extension.
I chose to create a new project using the "skeleton" template in a new directory.
A prompt told me to press F5 to run it, so I did.

EXPECTED RESULTS
App runs, either on my attached Android device, or using Linux, or as a web app.
ACTUAL RESULTS
It immediately complained about build errors.

@DanTup If you could post an update when you have a chance that'd be great. Thanks!

No significant progress yet - I have a CL that refactors things a little so the watcher is set up more reliably, but it doesn't yet handle what's happening here. Hoping to make some more progress this week though (although it may mean making some sync code async and I'm not certain if that will be straight-forward yet).

Actually, I think this is a different issue. While trying to repro this I noticed something strange..

run flutter create -t skeleton my_project
note that the output contains Running "flutter pub get" in my_project...
look in .dart_tool/ and notice there is no flutter_gen folder (this is why the analysis errors occur)

But note if you manually run flutter pub get the .dart_tool/flutter_gen folder is created.
So I think the issue here is that when flutter create fetches packages, it's not doing the same thing as when it's run manually or when it's (presumably implicitly) run by flutter analyze. I'll have a look and see if I can understand why.

Ok, figured out what's happening...
When you manually run flutter pub get it triggers creation of .dart_tool/flutter_gen by calling generateLocalizationsSyntheticPackage inside _runPubGet here:
https://github.com/flutter/flutter/blob/9e88fe328efcf42d3037a965b39269f0805cd4ce/packages/flutter_tools/lib/src/commands/packages.dart#L134
When you run flutter analyze, it triggers the same code inside FlutterCommand.verifyThenRunCommand here:
https://github.com/flutter/flutter/blob/9e88fe328efcf42d3037a965b39269f0805cd4ce/packages/flutter_tools/lib/src/runner/flutter_command.dart#L1265
However, when running flutter create, this code is never run. It goes pub get from here, but never calls the code to generate the localisations:
https://github.com/flutter/flutter/blob/9e88fe328efcf42d3037a965b39269f0805cd4ce/packages/flutter_tools/lib/src/commands/create_base.dart#L458
It does pass generateSyntheticPackage: true, to the call to pub get but that controls only injecting the package into _package_config.json` and not the actual creation of the project that it references.
I think the fix is to call generateLocalizationsSyntheticPackage around line 458 above so I'll do some testing and file a PR.

PR at https://github.com/flutter/flutter/pull/92975.

Is it bad that I've never heard of .dart_tool/flutter_gen before...

Nor had I until the race condition came up 😄
It's also slightly unfortunate that there's an unrelated published Pub package of the same name https://pub.dev/packages/flutter_gen (and I don't know what happens if you need/try to have both).


:zap: create pos_order when paying invoice

Successfully tested.  Great work!

will be tested and merged in #1412


Add instructions to install and link to the repo in onnxscript.ai/

Because it is the landing page from PyTorch doc, we should make it clear how users should install the package and add a link to the repo from https://onnxscript.ai/

cc @shubhambhokare1 this would be useful too

#1494 should add this info.


added readyState trigger to renderEvents

Added a readyState trigger at the end of each renderEvents function.  This is necessary for code which needs to act after a set of events is rendered.  One use-case for this is when events are pulled via ajax within Drupal. Drupal.attachBehaviors needs to be run after new content is added to the DOM but is a fairly heavy function and so cannot reasonably be run against each event individually as it is rendered.  This is a possible solution to issue #753.

This would also be useful for my needs.  Decorating all events after rendering seems to be a pretty common pattern.


Use the same middleware mecanism for App and Route

This is a work in progress.
The target is to build a common entity that can manage app middleware's stack as well as the route middleware's stack

That 's is ready for review

@codeguy since the Middleware class is exists only for internal business I think we can transform it into a more abstract class to wrap the app to, to do it we just have to make the next param optinal.
Since it shouldn't be used outside of the library that seems ok for me. Maybe we should change its name.

Ok, this is next on my todo list. Will be posting feedback soon.

Here's some basic code I put together really quick based on what you've done so far. Let me know what you think. Basically, all we'd need to do is change the Middleware class in this code to a trait, and drop that into the Slim and Route classes.
<?php
class Middleware
{
    protected $stack = [];

    public function initializeStack()
    {
        $this->stack[] = $this;
    }

    public function add(callable $callable)
    {
        $next = $this->stack[0];

        array_unshift($this->stack, function ($req, $res) use ($callable, $next) {
            return $callable($req, $res, $next);
        });
    }

    public function call($req, $res)
    {
        return $this->stack[0]($req, $res);
    }

    public function __invoke($req, $res)
    {
        $res .= '-CENTER-';
        return $res;
    }
}

$mw = new Middleware();
$mw->initializeStack();
$mw->add(function ($req, $res, $next) {
    $res .= 'Foo';
    $res = $next($req, $res);
    $res .= 'Bar';

    return $res;
});
$mw->add(function ($req, $res, $next) {
    $res .= 'Xyz';
    $res = $next($req, $res);
    $res .= '123';

    return $res;
});
$out = $mw->call('', '');

echo $out;

So much has changed I went ahead and implemented a rudimentary version in https://github.com/slimphp/Slim/commit/f45fe83c6f976210b5b334229bd406cc932db00e. Can you look over and make sure it's ok? Also, can you integrate into the Route class and make sure Route middleware is working correctly?

Hello.
Afk for the all week.
I will take a look next week.
Just to be sure you want me to add your implementation in the route class ?
Do we leave my pr ?
Le 4 mars 2015 04:19, "Josh Lockhart"<EMAIL_ADDRESS>a écrit :

So much has changed I went ahead and implemented a rudimentary version in
f45fe83
https://github.com/slimphp/Slim/commit/f45fe83c6f976210b5b334229bd406cc932db00e.
Can you look over and make sure it's ok? Also, can you integrate into the
Route class and make sure Route middleware is working correctly?
—
Reply to this email directly or view it on GitHub
https://github.com/slimphp/Slim/pull/1009#issuecomment-77094731.

I think your existing PR can be closed, but open a new PR for new changes and we can move discussions there.


Add PostgreSQL inttest

https://github.com/retz/retz/tree/wip-postgres-inttest is the working branch.

https://github.com/retz/retz/commit/bb1638deb24d86b314c544b819dce8413f1573da


Incorrect indent when auto-save beautifying

switch('...'){
  case 1:
  case 2:
  case 3:
  case 1024:
    alert('I like it!');
    break;
  default:
    alert("???");
}
var a = "We'll at the start without tabs";

after beautify:
switch ('...') {
  case 1:
    case 2:
      case 3:
        case 1024:
          alert('I like it!');
          break;
        default:
          alert("???");
        }
        var a = "We'll at the start without tabs";

I run ACode 1.4.164 with Webview 55.0 (com.google.android.webview)

Update the app.
You can join telegram group Acode editor.
Latest version - 1.4.168

Thank you for your concern.
I may update it after it's official publish, as the Telegram is banned in my country.


spacedrive: 0.2.4 -> 0.2.13

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for spacedrive is: An open source file manager, powered by a virtual distributed filesystem
meta.homepage for spacedrive is: https://www.spacedrive.com
meta.changelog for spacedrive is: https://github.com/spacedriveapp/spacedrive/releases/tag/0.2.13
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.13 with grep in /nix/store/pm7zn0iv4g25npsbs6z9j2fwapsw47wc-spacedrive-0.2.13
found 0.2.13 in filename of file in /nix/store/pm7zn0iv4g25npsbs6z9j2fwapsw47wc-spacedrive-0.2.13

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

spacedrive

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pm7zn0iv4g25npsbs6z9j2fwapsw47wc-spacedrive-0.2.13 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A spacedrive https://github.com/r-ryantm/nixpkgs/archive/8a74634fe7e088a6d123d0c4ffa86f85834416ec.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/8a74634fe7e088a6d123d0c4ffa86f85834416ec#spacedrive

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pm7zn0iv4g25npsbs6z9j2fwapsw47wc-spacedrive-0.2.13
ls -la /nix/store/pm7zn0iv4g25npsbs6z9j2fwapsw47wc-spacedrive-0.2.13/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

spacedrive

Maintainer pings
cc @DataHearth @heisfer @MikaelFangel @stepbrobd for testing.

Result of nixpkgs-review pr 307301 run on x86_64-darwin 1

1 package built:

spacedrive

aarch64-darwin and x86_64-darwin was not updated
Superseded by #307630


Added an open source project

Changes proposed
Added Drifty, an open source project.

Hey @SaptarshiSarkar12, could you please provide a description about what the project does, what is it about, how can people contribute, etc?
The documentation of the project provides very little insight.

Ok @siddhant-khisty . I shall let you know after I have included the above points in the documentation.

@siddhant-khisty I have successfully made the documentation more insightful by adding the points that you have mentioned and have added some other points as well.

@siddhant-khisty Please re-review this PR.

@kaiwalyakoparkar Please review this PR.

@kaiwalyakoparkar Please check it again. I have fixed it.

Thanks @kaiwalyakoparkar for reviewing this PR


blog deletion

Trying enable deletion on thoughts in this PR, blog deletion would be nice too.

Thanks for your work. will check the code today : )

I am working on this one, hope to finish it very soon.

Hi, I've finished the delete/edit functionality for both posts and blogs. I hope you haven't worked on that yet.
Now I'm really interested in these two features:

Add a public URL so everyone can share their homepage, like tinymind.me/metrue, where metrue is the user's Github user.login.

Add a "drop/tap to upload image/audio" feature, where we create an "assets" section so users can tap to upload their images or audio (video may be too large), rather than currently using the ![](xxx) format.

Both two features are very useful, I can start working on the photo drag and drop feature.

Tips: use Cursor + Claude 3.5 Sonnet for faster coding : )


Various issues

this menu does not do anything

"My Issues" leads to 404

the functionality of this page is not clear for me

pretty high but requires attention

Terms of Service is 404 as well

"Sign up for free" does not work

Yeap, i know about this 404 pages, it will be implemented in nearest future (summer)😋
The idea behind this project, is for tracking team issues, so this editor will be used for description editing. As for low optimisation for this page, if you open two instances of this in different windows, you find out, that this editor supports live update, so i think it's impossible(or very hard) to implement better optimisation

Oh, thanks, I already forgot about "Sign up for free" button😅


fix #263 merging fluent / fluentimpl

Hopefully this is still valid.
There is a usage of the fluent interface/impl in fabric8 that can be easily changed to just the concrete fluent.
cc @iocanel @manusa

@shawkins: Thank you for putting time on this. This is a far from trivial task. In the past I tried myself and gave up.
If I remember correctly the major pain point for me was what to do with intrefaces annotated with @Buildable as ATM we do generate a Fluent that is implemented by Builders of classes implementing the original interface.
Eitherway, before we merge this. I need to ensure that all downstream projects work. So, it will take some time.

@iocanel would it be possible to include additional test cases for the things that may be problematic?
I tried with a buildable interface and that caused the pojo to be generated as well.  From there the fluent and builder looked as expected.
With an abstract buildable I see only the fluent being generated.  Just like the change needed in fabric8, downstream projects would no longer use the fluent interface and just extend the concrete fluent.

@iocanel would it be possible to include additional test cases for the things that may be problematic?

I think that the shapes example should cover most of cases.

I tried with a buildable interface and that caused the pojo to be generated as well. From there the fluent and builder looked as expected.

Before adding support for Pojo generation when interfaces where annotated a Fluent was generated.
Implementing types had their Fluent extend the interface Fluent (or so I think). This seems to be broken now anyways and most probably this isn't even used.
I do see some failures in the downstream check which are due to explict references on the Impl class.
Can you please address that downstream so that we can re-run the check.

Can you please address that downstream so that we can re-run the check.

Won't I need at least a snapshot release of sundrio before I can do that?

@iocanel how do you want to proceed on this?

Can you please address that downstream so that we can re-run the check.

Won't I need at least a snapshot release of sundrio before I can do that?

I think that you might be able to fix that without a new release just by changing from FluentImpl to Fluent.

I think that you might be able to fix that without a new release just by changing from FluentImpl to Fluent.

How would that work given that Fluent with existing releases of sundrio will be an interface?
I'll just wait for the new release and update then.

How would that work given that Fluent with existing releases of sundrio will be an interface?

Well, unless you explicitly implment the Fluent I don't see why that is a problem.

State before this pr:
fabric8 has RunConfigNestedImpl extends RunConfigFluentImpl implements RunConfigNested - https://github.com/fabric8io/kubernetes-client/blob/5fb4ea82dfe55932a10b646fecde7812b0a28ef0/kubernetes-client-api/src/main/java/io/fabric8/kubernetes/client/extended/run/RunOperations.java#L31
After this pr, RunConfigFluentImpl will no longer exist.  The fabric8 code will to instead become:
RunConfigNested extends RunConfigFluent
I cannot make that change ahead of time as RunConfigFluent is an interface.

Ok, I am creating a 0.95.0 release with all the other changes that have recently made it in.
Then I'll branch and we can go for 0.100.0.

Then I'll branch and we can go for 0.100.0.

Sounds good.  Will there be snapshot builds available to update the downstream against?
There are some additional changes that may be good before the final 0.100.0 release.  Please see https://github.com/sundrio/sundrio/pull/379/commits/6168288168615b5bdf691dc4b25e842aa878e6dc

Sounds good. Will there be snapshot builds available to update the downstream against?

I haven't configured snapshot releases. Will work on that.


BlackWeb Mouse Support

Adding support for BlackWeb gaming mice would be nice.
my particular mouse is BlackWeb Wireless Mouse  : Model No.: BWA19HO027

+1


[PostgreSQL] Adding comments to constraints

DBeaver lacks the ability to add comments to table constraints (FK's, PK's and check constraints). It also doesn't detect and show existing comments of constraints. It would be great if DBeaver supported this feature. I think this should consist of:

When using Edit Constraint: new field for comment/description in the properties tab.
When using Create New Constraint: new field for comment/description.
Creation of comment DDL's like this: COMMENT ON CONSTRAINT constr_name ON public.table_name IS 'constraint comment';
Displaying constraint comments as mouse-over tool tips in the Navigator pane (just like for table columns).

Constraints and foreign key comments are now read from DB. Comment edit is also fully supported.
All constraint comments are also included in table DDL.


[ENH] Consider moving the Icon Thumbnail at a fixed position (next to the Sequence Name) on the Sequence Editor

Is your feature request related to a problem? Please describe.
I think that the icon thumbnail takes a lot of space at the top of the Sequence Editor.
I mean... not the icon itself but the way it moves when you resize the window increases the height of this "header" area.
Depending on how you resize the window, the icon thumbnail moves to the right side of the window, or moves below the Sequence Name.

I understand that you wanted to place the icon on the right side but I tend to think that the icon thumbnail would also fit nicely next to the Sequence Name input box, with a fixed position. Don't you think?
Describe the solution you'd like
Place the Icon Thumbnail next to the Sequence Name, at a fixed position (to avoid it moving below it).
Mock-up

Describe alternatives you've considered
Or just keep it on the right (it's a good location too, one I even prefer) but in this case, would it be possible to make the icon not move below the Sequence Name when the window is resized?

I've moved it to the left and shrunk it a bit to give more editor space as part of #871


Fix #20140: Fix compatibility with PHP 8.4: calling session_set_save_handler()

… 8.4

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
✔

Fixed issues
#20140

Thank you!


plugins

Plugins are not installed by running the file. For a step by step guide on installing plugins you can see this website.


Custom basemaps using http don't work in public maps

When using a custom basemap over http (e.g: http://tile.stamen.com/watercolor/{z}/{x}/{y}.jpg), it works in the editor but fails to load in the public map view. It seems that the public view is trying to load the basemap over https, even though the vizjson specifies the HTTP url.
@xavijam has been looking at this.

There is a function in CartoDB.js that transforms some URLs to https, that stamen one is included :joy:.
https://github.com/CartoDB/cartodb.js/blob/2983b2fdcef914afdb1f4fdae173471143930452/src/vis/layers.js#L11
As @javisantana pointed me, we did that in order to prevent problems visualizing maps with https.
Let me close it!

Perfect then, just went over the smoke tests results and didn't want to lose track of this.

Really appreciated man :heart:


Plan to publish to pypi?

Maybe publish to pypi so users can install straight from pip instead of copy from github?

This is planned and I'll get around to it sooner or later.

This should be complete now. If any further issues arise feel free to comment on this issue.
https://pypi.org/project/slash-util/

:0


Enhancement / Wishlist

It would be great to adopt the following practices

[ ] use xo to lint the code.
[ ] use conventional commit message

[ ] enforce conventional commit message

[x] semantic release
[x] Use ES2015
[ ] let the browser compatible version be generated

Fhfhf

Push.js uses eslint, which as it stands, is the most popular linting tool right now.
Commit messages should indeed follow the standard convention. Completely agreed. I would try to ensure merges to master follow the conventions.
Push.js is already using semver for releases.
Push.js is wriiten in ES2015.
Rollup is our build tool which has been configured to spit UMD builds.

Grateful for your recommendations @sarbbottam  :)


Reformat action thread comments

The thread comments not looks fancy than other projects:

codecov https://github.com/cpp-linter/cpp-linter/pull/21#issuecomment-1312488830
sonarcloud https://github.com/cpp-linter/cpp-linter/pull/21#issuecomment-1312673279
dependabot https://github.com/shenxianpeng/blog/pull/59#issue-1411851725

Maybe we could reformat thread comments, I draft a new one (WIP), and I will continue to update if there is a new idea.

Cpp-Linter Report ⚠
Some files didn't pass the configured checks.
Check the details
The following files need to be formatted (report from clang-format)

demo.hpp
demo.cpp

The following files have static analysis errors (report from clang-tidy)

demo.hpp

demo.hpp:10:11: warning: [modernize-use-trailing-return-type]

use a trailing return type for this function

    void *not_usefull(char *str){
    ~~~~~~^
    auto                         -> void *

demo.hpp:12:16: warning: [modernize-use-nullptr]

use nullptr

        return 0;
               ^
               nullptr

demo.cpp
demo.cpp:5:5: warning: [modernize-use-trailing-return-type]

use a trailing return type for this function

int main()
~~~ ^
auto       -> int

demo.cpp:7:13: warning: [readability-braces-around-statements]

statement should be inside braces

    for (;;)
            ^
             {

demo.cpp:11:5: warning: [cppcoreguidelines-pro-type-vararg]

do not call c-style vararg functions

    printf("Hello world!\n");
    ^

Have any feedback or feature suggestions? Share it here.

Cpp-Linter Report ✔
No problems need attention.
Have any feedback or feature suggestions? Share it here.

I have no objections to this.

If this is to be shown after successful runs, then I think it would be good to be more concise.

Cpp-Linter Report ✔
No problems need attention.
Have any feedback or feature suggestions? Share it here.

Instead of showing the errors, we could show a simple table that summarizes the results.

file
clang-tidy
clang-format

demo/demo.cpp
🔴
🟢

complete emoji choices

Or we could hide the summary table in a details tag
Cpp-Linter Report ⚠
Some files didn't pass the configured checks.
sumary table

file
clang-tidy
clang-format

demo/demo.cpp
🔴
🟢

You know me: anything is possible. I just need a concrete decision before coding the solution.
So, if tidy-checks is '-*' or style is '', then we go with a list. Otherwise we show a table. Yes?

I only suggested the table because

it will reduce the occupied screen real estate when user expands the details.
not showing the info about the reported errors will allow us to fit more summary info in the comment. Remember that the comment's size is limited by the REST API max payload size, so a thread comment about a very large diff (with lots of errors/warnings to report) will be truncated by the REST API's max payload size. There isn't a way around this max limit. I know other actions hide the ending portion of their potentially large comments in a <details> tag just in case the payload is truncated.

One thing I noticed about all the example comments you linked to in the OP was that they all show an overview instead of detailed info. So, I thought that was partly what you wanted to mimic.

LGTM

I just found an additional feature in github actions: The GITHUB_STEP_SUMMARY will post a markdown content at the bottom of the run's summary. Its not exactly like thread-comments (the markdown payload is the same), but at least we wouldn't have to worry about duplicate bot comments.
Someone even uses the feature in an existing action that produces a unit test summary comment in the workflow's run summary. For example, this run summary shows the custom summary that we could create.

It wouldn't be too hard to implement, we just need to add an input option to turn it on/off and add
with open(os.environ["GITHUB_STEP_SUMMARY"], "a") as summary:
    summary.write(Globals.OUTPUT)

Yep! We don't have to check for duplicate comments and all that entails either.

sorry this still on my todo list, but I've been busy.

Note
I recently noticed that github added support for admonitions (AKA "callouts"). See example here.

Warning
But it looks like only "note" and "warning" are supported (in English only). I also don't like how this looks in dark mode. Block quotes' dimmed text was a bad choice.


Behavior of error message?

If I add my-email<EMAIL_ADDRESS>to the email field of the demo, then remove the .com part, blur the field, nothing happens. Isn't it supposed to say "invalid input" in this situation?

Isn't it supposed to say "invalid input" in this situation?

No. The [domain part] (http://en.wikipedia.org/wiki/Email_address#Domain_part) must not necessarily be end with a TLD. However, it needs to identify a host name. foo@localhost is a syntactically correct email address.
Sidenote, valdr uses the same regex for email validation as Angular (also see tests).


error in binding-details

When running documentarian on fugue, I get this traceback:
error: expected integer key in range [0, 0), got :ns
  in binding-details [src/documentarian.janet] on line 198, column 29
  in extract-bindings [src/documentarian.janet] on line 278, column 11
  in generate-doc [src/documentarian.janet] (tailcall) on line 427, column 17

Lemme know if this rings any bells!

An interesting trace:
trace [src/documentarian.janet]: (ns-or-alias name ns) is "fugue"
trace [src/documentarian.janet] on line 198, column 30: meta is @{:doc "(match x & cases)\n\nPrototype-aware version of `match`. Introduces one new case form:\n\n- `(@ <prototype-name> <dictionary>)`: Will pattern match against an instance\nof `prototype-name`. Additionally, will validate at compile-time that every\nkey in `dictionary` is a field that's present on the specified prototype." :macro true :source-map ("fugue.janet" 877 1) :value <function match>}
trace [src/documentarian.janet]: (ns-or-alias name ns) is "fugue"
trace [src/documentarian.janet] on line 198, column 30: meta is @[]

@subsetpark Thanks for reporting and looking into this. You're right, I'm not familiar with the :macro-lints change. I'll have to have a look at what's changed in Janet.

Oh, and thanks for submitting PR #5! I want to understand the problem better but hopefully that's the fix (less code for me to write! :P).

I got the same error when trying to run on the testament library.


docs: set correct links to documentation

Fixes the links to the docs.
Closes #794

Deployed at: https://alva-issues-796.surge.sh

Deployed at: https://alva-site-issues-796.surge.sh


Change error cleanup in objstore.DownloadDir to delete files not destination dir

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Change error cleanup in objstore.DownloadDir to delete files not directories:
dst is always a directory. If any file after the first fails to download, the cleanup will fail because the destination already contains at least one file. This PR changes the cleanup logic to clean up successfully downloaded files one by one instead of attempting to clean up the whole dst directory.
This PR does not account for the case where there are nested directories. While we can do os.RemoveAll on any child directories, it's unsafe. We don't know for sure that all files in child directories were downloaded as a result of this DownloadDir call and didn't exist before it.
Verification

Even though, replacement hasn't done in main repo, better to target https://github.com/thanos-io/objstore from now on.


Error message added to training.py

Error message added to training.py.

Closing outdated PR. If you still care about the content of the PR, please submit a new PR to master, updated for the Keras 2.0 API.


[Run] Globalization.CultureNotFoundException: Culture is not supported

Microsoft PowerToys version
0.80.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
General, PowerToys Run
Steps to reproduce
Having Powertoys Run enabled.
✔ Expected Behavior
No errors should be displayed.
❌ Actual Behavior
No matter if I run the app as administrator or not, if PowerToys Run is enabled it will keep throwing errors.
2024-05-19.txt
Other Software
No response

System.Globalization.CultureNotFoundException: Culture is not supported. (Parameter 'culture')
4096 (0x1000) is an invalid culture identifier.
   Source: System.Private.CoreLib
   TargetAssembly: System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e
   TargetModule: System.Private.CoreLib.dll
   TargetSite: Void .ctor(Int32, Boolean)
   at System.Globalization.CultureInfo..ctor(Int32 culture, Boolean useUserOverride)
   at System.Windows.Input.InputLanguageSource.get_CurrentInputLanguage()
   at PowerLauncher.ViewModel.MainViewModel.GetLanguageFlowDirection()
   at PowerLauncher.MainWindow.SearchBox_UpdateFlowDirection()
   at PowerLauncher.MainWindow.OnLoaded(Object sender, RoutedEventArgs e)
   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
   at System.Windows.UIElement.RaiseEventImpl(DependencyObject sender, RoutedEventArgs args)
   at System.Windows.BroadcastEventHelper.BroadcastEvent(DependencyObject root, RoutedEvent routedEvent)
   at System.Windows.BroadcastEventHelper.BroadcastLoadedEvent(Object root)
   at System.Windows.Media.MediaContext.FireLoadedPendingCallbacks()
   at System.Windows.Media.MediaContext.FireInvokeOnRenderCallbacks()
   at System.Windows.Media.MediaContext.RenderMessageHandlerCore(Object resizedCompositionTarget)
   at System.Windows.Media.MediaContext.RenderMessageHandler(Object resizedCompositionTarget)
   at System.Windows.Media.MediaContext.Resize(ICompositionTarget resizedCompositionTarget)
   at System.Windows.Interop.HwndTarget.OnResize()
   at System.Windows.Interop.HwndTarget.HandleMessage(WindowMessage msg, IntPtr wparam, IntPtr lparam)
   at System.Windows.Interop.HwndSource.HwndTargetFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

xref: #24172

I had the same issue and it was fixed by changing the language/keyboard layour combination
I switched from English(Europe)/US-INT to English(US)/US-Int
similar solution here: https://github.com/microsoft/PowerToys/issues/24172#issuecomment-1449476792


Update jailbreaks.yml

Update the unc0ver max version supported and remove duplicate as the current version supports them all. Also bump checkra1n's max version.

thanks for this, i've applied your changes and moved the jailbreak order in my most recent commit to master.


[Mailing List] Make all the items width equal.

Signed-off-by: Arunima Chaudhuri<EMAIL_ADDRESS>Description
This PR fixes #868
Closes #868
Notes for Review

ers
Signed commits

[x] Yes, I signed my commits.

Hey @legendarykamal, Can you plaese review the changes. By mistake I uncommitted the changes of the previous PR. Hence it was close. So, can you please look at the changes and approve them?

Looking good, @tinniaru3005

The buttons are still not of identical width.

Instead of adjusting the padding for each button, keep the padding the same for all of them and adjust other CSS properties like display, and min-width. And, set the width: auto, margin:auto to the parent p element.

hey @Nikhil-Ladha, are the changes fine now?

Hey @Nikhil-Ladha, are the changes fine now?

The text is not center aligned, please fix that.
Also, you will have to use min-width (probably 8rem) for a else, the button width won't remain the same for all of them.


Zippy throws RangeDefect when parsing file permissions on 32-bit systems.

When compiled for Arm 32-bit and run on a Raspberry Pi Zero, zippy throws the following RangeDefect:
Error: unhandled exception: value out of range:<PHONE_NUMBER> notin -2147483648 ..<PHONE_NUMBER> [RangeDefect]
This is because the proc parseFilePermissions() is set up to read the platform-flexible signed int, and externalFileAttr: uint32 is converted to an int when parsing file permissions. It should instead be used as the uint32 it already is. We can fix this by changing the following two lines:

internal.nim line 285:

proc parseFilePermissions*(permissions: uint32): set[FilePermission] =

ziparchives.nim line 386:

        filePermissions: parseFilePermissions(externalFileAttr shr 16)

These changes negate the issue on the Raspberry Pi Zero, and hopefully should do the same on other 32-bit systems.

This should be fixed in release 0.10.16


upper menus and configure lables desappears

Please see the attached image.

Thanks for letting me know, this was an SSL misconfiguration on my side.  I've fixed the issue now.


[Web] Combine Positioned with Draggable (with button as child) throws when dragging

Description
When on web, a Draggable or LongPressDraggable with a child containing an onTap() method will throw an exception and fail to update when starting to drag.
Similar to #69774, the only difference is that the Draggable is under a Positioned widget
Steps to Reproduce
Example code on DartPad
  Widget build(BuildContext context) {
    final fab = FloatingActionButton(
        onPressed: () {}); // works if switch to a widget without onTap/onPress...

    return MaterialApp(
      home: Stack(
        children: [
          Container(key: overlayCtnKey),
          DraggableWidget(containerKey: overlayCtnKey, child: fab),
        ],
      ),
    );
  }

To reproduce the bug when on Web, drag & drop the FloatingActionButton to a new position on screen
Expected results: The Draggable's new position should be updated
Actual results: The Draggable crashes. In the console, exceptions are thrown:

Logs
Error: This widget has been unmounted, so the State no longer has a context (and should be considered defunct).
                    Consider canceling any active work during "dispose" or using the "mounted" getter to determine if the State is still active.

Error: Assertion failed: file:///C:/Users/jaden.tang/sdk/flutter/packages/flutter/lib/src/rendering/mouse_tracking.dart:301:12
           !_debugDuringDeviceUpdate is not true

[√] Flutter (Channel beta, 1.24.0-10.2.pre, on Microsoft Windows [Version 10.0.18363.1256], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Chrome - develop for the web
[!] Android Studio (not installed)
[√] VS Code (version 1.52.0)
[√] Connected device (3 available)

When on Android emulator, the dragging behaves as expected.

Hi @tcd93
Thanks for filing the issue, I can reproduce the issue on beta, However, it seems to be fixed in the dev and master channel

code sample
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  final overlayCtnKey = GlobalKey();

  @override
  Widget build(BuildContext context) {
    final fab = FloatingActionButton(
        onPressed:
            () {}); // works if switch to a widget without onTap/onPress...

    return MaterialApp(
      home: Stack(
        children: [
          Container(key: overlayCtnKey),
          DraggableWidget(containerKey: overlayCtnKey, child: fab),
        ],
      ),
    );
  }
}

class DraggableWidget extends StatefulWidget {
  final Widget child;

  final GlobalKey containerKey;

  DraggableWidget({@required this.child, @required this.containerKey});

  @override
  DraggableWidgetState createState() => DraggableWidgetState();
}

class DraggableWidgetState extends State<DraggableWidget> {
  double top, left;

  @override
  void initState() {
    top = 50;
    left = 50;
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    final RenderBox backgroundRenderBox =
        widget.containerKey.currentContext.findRenderObject();
    return Positioned(
      top: top,
      left: left,
      child: Draggable(
        dragAnchor: DragAnchor.child,
        feedback: widget.child,
        child: widget.child,
        childWhenDragging: const SizedBox(),
        onDragEnd: (drag) {
          final localOffset = backgroundRenderBox.globalToLocal(drag.offset);
          setState(() {
            top = localOffset.dy;
            left = localOffset.dx;
          });
        },
      ),
    );
  }
}

Issue exist

flutter doctor -v
[✓] Flutter (Channel beta, 1.24.0-10.2.pre, on macOS 11.0.1 20B29 darwin-x64,
    locale en-GB)
    • Flutter version 1.24.0-10.2.pre at /Users/tahatesser/Code/flutter_beta
    • Framework revision 022b333a08 (4 weeks ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[!] Android Studio
    • Android Studio at /Applications/Android/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Android Studio not found at /Applications/Android/Contents
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.52.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.17.0

[✓] Connected device (5 available)
    • RMX2001 (mobile)     • EUYTFEUSQSRGDA6D                     •
      android-arm64  • Android 10 (API 29)
    • Taha’s iPad (mobile) • 00008020-000255113EE8402E            • ios
      • iOS 14.2
    • iPhone 12 (mobile)   • 2188C52E-1D22-45B2-98B8-48401D2B6716 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-2 (simulator)
    • Web Server (web)     • web-server                           •
      web-javascript • Flutter Tools
    • Chrome (web)         • chrome                               •
      web-javascript • Google Chrome 87.0.4280.88

! Doctor found issues in 1 category.

No issue on version below

multiple flutter doctor -v
[✓] Flutter (Channel dev, 1.25.0-8.0.pre, on macOS 11.0.1 20B29 darwin-x64,
    locale en-GB)
    • Flutter version 1.25.0-8.0.pre at /Users/tahatesser/Code/flutter_dev
    • Framework revision a12e2a473a (5 days ago), 2020-12-08 23:13:04 -0500
    • Engine revision df39e5c515
    • Dart version 2.12.0 (build 2.12.0-133.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[!] Android Studio
    • Android Studio at /Applications/Android/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Android Studio not found at /Applications/Android/Contents
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.52.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.17.0

[✓] Connected device (5 available)
    • RMX2001 (mobile)     • EUYTFEUSQSRGDA6D                     •
      android-arm64  • Android 10 (API 29)
    • Taha’s iPad (mobile) • 00008020-000255113EE8402E            • ios
      • iOS 14.2
    • iPhone 12 (mobile)   • 2188C52E-1D22-45B2-98B8-48401D2B6716 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-2 (simulator)
    • macOS (desktop)      • macos                                • darwin-x64
      • macOS 11.0.1 20B29 darwin-x64
    • Chrome (web)         • chrome                               •
      web-javascript • Google Chrome 87.0.4280.88

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 1.25.0-9.0.pre.43, on macOS 11.0.1 20B29
    darwin-x64, locale en-GB)
    • Flutter version 1.25.0-9.0.pre.43 at /Users/tahatesser/Code/flutter_master
    • Framework revision 63062a6443 (22 hours ago), 2020-12-13 23:19:13 +0800
    • Engine revision 4797b06652
    • Dart version 2.12.0 (build 2.12.0-141.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[!] Android Studio
    • Android Studio at /Applications/Android/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Android Studio not found at /Applications/Android/Contents
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.52.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.17.0

[✓] Connected device (5 available)
    • RMX2001 (mobile)     • EUYTFEUSQSRGDA6D                     •
      android-arm64  • Android 10 (API 29)
    • Taha’s iPad (mobile) • 00008020-000255113EE8402E            • ios
      • iOS 14.2
    • iPhone 12 (mobile)   • 2188C52E-1D22-45B2-98B8-48401D2B6716 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-2 (simulator)
    • macOS (desktop)      • macos                                • darwin-x64
      • macOS 11.0.1 20B29 darwin-x64
    • Chrome (web)         • chrome                               •
      web-javascript • Google Chrome 87.0.4280.88

! Doctor found issues in 1 category.

Labelling the issue to track the fix for beta channel
CC @pcsosinski

Please see https://github.com/flutter/flutter/issues/67044 . closing this one as duplicate.


Check out "Grindr - Gay chat"

https://play.google.com/store/apps/details?id=com.grindrapp.android

Please check out the project before opening any issues


fix metalink size for url

Overview
This PR adds a "size" checker in URLHandler.
When the metalink document is created it is checking the "file" size of a metafile item. When the metafile is representing an external URL it is using the URLHandler.
Issue: when we were calculating the size of an "URL" it was first downloading the URL and the calculating the file size. We did not recognize ... files were small before ;) This is probably the root cause for issue #571.
Now, the URLHandler and the FileHandler have their own methods to calculate "size". Avoiding unwanted downloads.
Related Issue / Discussion
#571 #580
Additional Information
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[ ] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[ x] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

Coverage remained the same at 0.0% when pulling 4d24fb92d999391232ef25aa5568d252f33462a7 on cehbrecht:fix-metalink-size-for-url into bbca1e42b63cf9c31c40fa3e2a731a1528adb2b9 on geopython:pywps-4.4.


[Feature Request] Change locale at runtime

Context

[X] This issue is not a bug report. (please use a different template for reporting a bug)
[X] This issue is not a duplicate of an existing issue. (please use the search to find existing issues)

Description
Currently there is no way to change the locale after initializing the editor.
Monaco Editor Playground Link
No response
Monaco Editor Playground Code
No response

Thanks for the feature request!
This is not a regression, right?

right


Expiring credit card email not sent

My credit card (#592538) for the yearly contribution #57662 expired last month. I didn't receive any templates/emails/payment.creditcard.expiring.hbs email to warn me about that it was about to expire.

The root cause is that the expiryDate column for this credit card is null, despite the data field properly containing an expYear and expMonth. It seems to be the case for many cards:
SELECT COUNT(*), MAX("createdAt"), MIN("createdAt")
FROM "PaymentMethods"
WHERE "expiryDate" IS NULL
AND data ->> 'expYear' IS NOT NULL
AND data ->> 'expMonth' IS NOT NULL
AND "deletedAt" IS NULL

count
max
min

68377
2024-11-01 10:00:02.880000 +00:00
2021-02-15 21:27:38.132000 +00:00


Can't work on AndroidStudio3.0

@siosio any news here?

I confirmed it in Android Studio, but it works.
As version 1.10 or higher is not supported, please install 1.9.
This plugin works with Smart Completion (Ctrl + Shift + Space), not Basic Completion (Ctrl + Space).

How do I customize shortcut keys?


Review ObjectSerializer proposal for Python

https://github.com/Azure/azure-sdk/issues/1454 tracks the proposal for ObjectSerializer from the .Net perspective.
Java has something similar.
We need to review the discussion there and come up with corresponding proposal in Python for the ObjectSerializer
One upcoming use case for this would be the Avro serializers for Azure Event Hubs.
cc @lmazuel

Related issue in JavaScript: https://github.com/Azure/azure-sdk-for-js/issues/10069

C#: https://github.com/Azure/azure-sdk-for-net/pull/12258

No more planned work for now. Closing.


[BUG]: repctl compiling failed

Bug Description
On CentOS 8, with go lang version 1.6, compiling repctl failed without clear reason.
After change to go 1.9, the compiling went well.
Logs
[root@ins repctl]# make build
cp ../scripts/gen_kubeconfig.sh ./pkg/cmd/scripts/gen_kubeconfig.sh
cp ../scripts/config-placeholder ./pkg/cmd/scripts/config-placeholder
go build ./cmd/repctl
/root/go/pkg/mod/sigs.k8s.io/json@v0.0.0-20220713155537-f223a00ba0e2/json.go:24:2: //go:build comment without // +build comment
make: *** [Makefile:6: build] Error 1
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
On CentOS 8, install golang via yum, and run "make build" to compile repctl.
Expected Behavior
Make command finished successfully. Command repctl was compiled.
CSM Driver(s)
CSM replication 1.3
Installation Type
No response
Container Storage Modules Enabled
Replication 1.3
Container Orchestrator
Kubernetes 1.24
Operating System
CentOS 8

The repctl/go.mod file indicates that we need to use "go 1.18" at a minimum. What is your ask? Is there a documentation that you would like us to update?

Hi @seanzhan2014.  There does not appear to be an issue here so I will close the issue.  Please feel free to reopen the issue if you have additional information for us.  Thanks.


framed draw crash

i made a framed draw from functional storage changed texture didnt like it so tried to change the same texture in the tinkers crafting station, game crashed on placing final item reloaded opened crafting station with same items already in crashed again
Did more testing it seems to crash when trying to use slime as one of the decoration blocks
In any crafting table
Crashlog: https://pastebin.com/69CPSLL3
Version: functionalstorage-1.18.2-1.0.7.jar

Got similar issue when using a cobblestone chest on a crafting station...
java.lang.NullPointerException: Cannot invoke "net.minecraft.core.Direction.m_122421_()" because "side" is null
at com.buuz135.functionalstorage.client.loader.FramedModel$Baked.getMinMaxPosition(FramedModel.java:228)
...
It doesn't like some items textures and then crashes.

I got a crash when trying to use sand and a cactus for a framed drawer.

Should be fixed in https://github.com/Buuz135/FunctionalStorage/pull/127
Backport soon


fix: remove trailing slash from repo path

Quick follow-up to https://github.com/jrmoulton/tmux-sessionizer/pull/35 because I noticed it too late
Workdir paths seem to always have a trailing slash, which is displayed wrong in my fish shell prompt, and also different from how the paths looked before
Canonicalize removes these trailing slashes
Before:

After:

Thanks for catching that!


update readme

updating readme to add branching, pull requests, and merging.

I think the references to "your branch" are confusing; e.g., "All posts should be placed in the _posts directory of the your new branch" could be just "All posts should be placed in the _posts directory."
Actually, I wonder if it's worth separating out process for adding something from where files should be place?

I'd agree with @bonniegee's comment above - let's separate process from conventions.
A couple of other things:

Last sentence of "Image Location" section should read "Make sure your post links to the correct images."
In the "Create a pull request" section how about "You should create a pull request from your newly created branch which points at the gh-pages branch." (PRs have directionality)
I think any other RAC staff member should be able to review and approve the post.

Updated! Let me know.

@bonniegee I agree, but I also liked @HaSistrunk 's suggestion to give guidance on naming. Maybe just the name of the post? Or the date?

If you were to create a branch name to add a post, what would your instinct be?

For the post I just did, I used the date and then just the word "post" to differentiate from other types of changes.

@HaSistrunk that works for me. We could differentiate between posts and development PRs


BUG: Unable to create plain text nodes

If I pass in an undefined "sel", this line blows up, even though everything after that seems to work fine.
https://github.com/snabbdom/snabbdom/blob/master/src/h.ts#L35

According to issue creation date, you were looking at
https://github.com/snabbdom/snabbdom/blob/091dc5f96154ebde238bfecd7adcd8d6809cae0d/src/h.ts#L35-L37
And it seems like to this day an undefined first argument to h is not supported:
https://github.com/snabbdom/snabbdom/blob/aac0233b1ff616ffbb47d06c39505fad0ec50476/src/h.ts#L20-L49
So I'm going to close this as unsupported. Text nodes can be created as children of elements using h or using vnode in this way:
https://github.com/snabbdom/snabbdom/blob/aac0233b1ff616ffbb47d06c39505fad0ec50476/src/h.ts#L37-L41
Or manually:
https://github.com/snabbdom/snabbdom/blob/aac0233b1ff616ffbb47d06c39505fad0ec50476/src/vnode.ts#L13-L20
Seems like it's something like { text: 'foo' } and that's it.


update release_4.0 time period list

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

Project coverage is 16.85%. Comparing base (b77276b) to head (53b71bb).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #73   +/-   ##
=======================================
  Coverage   16.84%   16.85%
=======================================
  Files          41       41
  Lines        7063     7062    -1
=======================================
  Hits         1190     1190
+ Misses       5873     5872    -1

Flag
Coverage Δ

unittests
16.85% <ø> (+<0.01%)
:arrow_up:

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

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


Emporia Smart Plug support

QUESTION: Does anyone know of any sources for similar reverse engineering efforts for the Emporia smart plugs?
Background
So, I've ordered a Vue 2 main unit, and managed to get through pulling the flash contents and working through the NVS. Thanks for the great work!
I've also ordered (they arrive later this week) the smart plugs though, as I've got a few hot spots that are on otherwise pretty broad circuits (e.g., my home office setup is on the same circuit as every light and plug on our second floor... 🤦♂), and I'd like to use them for both monitoring and control.
My gut is suggesting to me that internally they are probably very, very similar since their features compared to a full unit seem very similar (so I'm expecting a similar ES32 with almost identical firmware, and a single current loop, with a relay).
My end goal, I think, is to replace the certificates in the flash (that are almost certainly the AWS IOT Core CA certs) with custom certs, and MITM the traffic to get the best of both worlds (on-prem data retention, with cloud connectivity for control), but I'm not there yet (the certs are in some mutant format I haven't put much time into deciphering).

Not a chance. Sorry. It just doesn't make sense to put the high-end parts needed to handle 20 channels in a cheap wall thing.
Here's a teardown: https://blog.horner.tj/emporia-smart-plugs-tasmota/. Looks like it's a white-label product. It uses a BL0937 for power monitoring. It's also impossible to take apart cleanly.
I would return them and replace them with the Sonoff S31, which is easy to disassemble and flash with ESPHome or Tasmota.


Null Reference Exception and 64bit Nvidia Driver Error while building from source

So I'm able to get run the application from the installer just fine, but I tried building it from source and I get the following issues
This is the gui error

But this is what I get while trying to debug it

Anyone have any advice?

I am also currently getting this error after pointing to CUDA 10.1 and generating the shared files


Add browser-esm-rspack sample

Add browser-esm-rspack sample.
Rspack (pronounced as /'ɑrespæk/, ) is a high performance Rust-based JavaScript bundler that offers strong interoperability with the webpack ecosystem, enabling faster development cycles and efficient collaboration between the two tools.

@microsoft-github-policy-service agree

@hediet cc

This PR is out of scope. We cannot add examples for every bundler.


install/upgrade/rollback commend support add  label flag

I need to support add labels when I install / upgrade / rollback without modifying the contents of the template in the chart package.
The helm cli does not seem to support  label flag for install or upgrade and rollback.for example:
$ helm install release_name ./chart_package --label  key1=vaule1,key2=value2

The same as kubectl.
$ kubectl label pods foo  key1=vaule1,key2=value2

Duplicate of https://github.com/helm/helm/issues/1950. Please check the issue queue before opening a new ticket. Thanks!

Sorry,I got it.


Abstract at least common dependencies into an AssemblyLoadContext

Summary of the new feature / enhancement
PSGet 3.0 should follow the example of PSES and abstract its common dependencies such as NuGet into an AssemblyLoadContext, so as not to conflict with other legacy modules which might use those assemblies.
Per @SteveL-MSFT:
https://twitter.com/Steve_MSFT/status/1542983766901026816?s=20&t=lguXc5lP3bbwtRahi34EUw
Proposed technical implementation details (optional)
No response

We're actually going to try to remove all dependencies so that we don't need to use an ALC.  We'll invoke PowerShell for functionality such as parsing json data.  If for some reason in the future we need to pick up more dependencies, we'll implement an extension of an ALC.


2.3, mail icon is different and I can't find it in icons list page.

2.3, mail icon is different and I can't find it in icons list page.

Good spot, the icon is now called envelope, I am going to add an alias for mail to help with compatibility.

Should also have mail

Fixed in next


Color mask and Occlusion queries

I am totally unsure if what I'm doing is legal vulkan, but I think this is the best place to ask since I have this issue exclusively on Molten. And again is a pretty corner case which probably stretches the normal usage of the specs.
I have a pipeline which is setup to not write to the depth attachment (VkPipelineDepthStencilStateCreateInfo::depthWriteEnable is false) and to mask both color attachments (VkPipelineColorBlendAttachmentState::colorWriteMask is 0) (so a pipeline to do nothing, essentially). However, the renderpass and the framebuffer are setup so that they have 2 color attachments and a depth attachment.
The shader does a discard, but otherwise specifies no color attachment (so the fragment shader has no output layout). Maybe this is illegal but I also tried with two color attachments writing dummy values and nothing changes.
Then I have an occlusion query counting the samples passed by the shader, which executes either a discard, or writes noting to the attachment. I want the query to count only the times no "discard" is issued.
"25.8. Sample Counting" does not mention color masks, so I would assume as for the specs fully masked colors should still count:

The occlusion query sample counter increments by one for each sample with a coverage value of 1 in each fragment that survives all the per-fragment tests, including scissor, sample mask, alpha to coverage, stencil, and depth tests.

I have the feeling somewhere along the way Metal is optimizing away something that I do not want.
Now: one alternative would be just have a renderpass with no attachment, but I have a feeling that this would also result in the wrong occlusion counts (I will test this later on).
The alternative seems to be not to mask the color attachments, which works on MoltenVK, but in theory (if I understand the specs correctly) results in undefined values written to the attachment (did not observe this in practice).
My question is:

does somebody know what should happen according to the specs? and if I'm doing something illegal?
is the alternative of not masking the color a good solution (workaround) for molten (since I do not know what metal does under the hood)? Or am I shooting myself in the foot?

By the way, it seems this issue happens only on Intel Iris mac, but on a Mac book pro it seems to work as expected, so it might be a Metal issue.

So what's happening is, because your fragment shader has no output layout, when MoltenVK transforms it to MSL so it can be passed to Metal, the shader returns void. I'm wondering, then, if you are correct, and Metal is discarding the fragment whether or not the shader calls discard_fragment(), because no color is produced. Maybe it shouldn't. I'll file a feedback.


Update elixir-lang "linter" added.md

Information:
Hello... add linters for "elixir-lang"
### ie:

Dogma
Credo
elixir_linter

Checklist:

[ ] Only added one linter?
[x] Is it inside the right language container?
[x] Is it sorted alphabetically inside the container?
[x] Does the title follow the template:  "Add [LINTER] for [LANGUAGE]."?

For reference, there's some contributing guidelines.

@odyright Can you create the pull request in caramelomartins/awesome-linters, please? I would like your changes to be included in the project but don't want to take credit for your suggestions. :)


Configs for initialization with custom non-root PODs

Feature description:
I noticed that there is now a ConfigPodInitializer so in my understanding it should now be possible to define some configs that create a few sample users on startup (for example for a test environment).
Are there sample configs on how to do this for non-root users - (I can only see ones for root users in the initializers configs). I made the following attempt of my own, but was not sure how to correctly configure the WebID - nor what the appropriate values for storageKey were.
{
  "@context": <EMAIL_ADDRESS>  "@graph": [
    {
      "comment": "Makes sure the root container exists and contains the necessary resources.",
      "@id": "urn:solid-server:default:PodInitializer",
      "@type": "ConditionalHandler",
      "storageKey": "foo",
      "storageValue": true,
      "storage": { "@id": "urn:solid-server:default:SetupStorage" },
      "source": {
        "@type": "ContainerInitializer",
        "args_baseUrl": { "@id": "urn:solid-server:default:variable:baseUrl" },
        "args_path": "/wright",
        "args_store": { "@id": "urn:solid-server:default:ResourceStore" },
        "args_generator": {
          "@type": "TemplatedResourcesGenerator",
          "templateFolder": "@css:templates/pod",
          "factory": { "@type": "ExtensionBasedMapperFactory" },
          "templateEngine": { "@type": "HandlebarsTemplateEngine" }
        },
        "args_storageKey": "foo",
        "args_storage": { "@id": "urn:solid-server:default:SetupStorage" }
      }
    }
  ]
}

Happy to create a couple of such configs for others to use in dev environments if I can be pointed in the right direction for solving these problems.

I noticed that there is now a ConfigPodInitializer so in my understanding it should now be possible to define some configs that create a few sample users on startup (for example for a test environment).

This is actually an "old" feature that was added before the first release of CSS, but not iterated upon afterwards due to shifting priorities. It's not exactly what you think it is though so I'll cover what it does here.
ConfigPodInitializer is part of the dynamic pod creation feature, which is enabled by the imports in the dynamic.json config. The idea behind dynamic pod creation is that every individual pod could have its own custom Components.js configuration. This would allow one user to have a file backend for their pod, and another user to have a SPARQL backend or even a composite backend where part of it is used for their hue lights. Registering a pod in such a server takes an extra template parameter, as seen in the corresponding integration test. This string refers to one of the stored templates on the server. The server then uses a router store to the correct instantiated mini-store based on the podname found in each URL.
One issue is that if the server restarts it needs to instantiate all these pods again. That is where the ConfigPodInitializer comes in. Every time a new pod is created on such a dynamic server, its settings (owner, template, name, etc.) are writting to a JSON config. When starting the ConfigPodInitializer it then reads that config file and calls Components.js for each entry.
What the ConfigPodInitializer does NOT do is:

Create files in those pods, since those would have been stored in whatever backend.
Register those pods in the IdP, since that would have already have happened during registration.

So i don't think the initializer would fit for your use case. If you want to quickly set up a bunch of pods your best bet is to have a JSON list where each entry is a set of parameters that can be sent to the registration endpoint and then a small script that iterates over them and does the POST requests.


[FLINK-6789] [optimizer] Remove duplicated test utility reducer in op…

Removed DummyReducer class, keep SelectOneReducer instead.

LGTM, merging ...

Thanks for review.


Investigate valgrind processes dying on qcl hub

From https://2i2c.freshdesk.com/a/tickets/1161

Several users are having issues where processes are seemingly randomly getting killed.
Personally I was running profiling tests with valgrind (3 hour single threaded process using ~10% of the ram), and maybe 1 in 3 of these processed would be killed (exit process 127, not sure if thats helpful)
Looking at grafana, I confirm that these are not out of memory kills... do you have any diagnostics, or watch dog tools that kill CPU hogging processes? This is somewhat of a critical issue for us as we need to run analysis that takes days to process

This user's pod is no longer running, so we should first figure out where it was.
I can find all events related to this user's pod with the following query on stackdriver:
resource.type="k8s_cluster" OR resource.type="k8s_node" OR resource.type="k8s_pod" OR resource.type="k8s_container"
log_id("events")
jsonPayload.involvedObject.name="jupyter-jtkmckenna"

This shows me that the pod was assigned to Successfully assigned prod/jupyter-jtkmckenna to gke-qcl-cluster-nb-n2-highmem-4-daaf2382-wgxm so I can look at that node's logs.
We can then look at all the logs on that node (not k8s related) with:
resource.type="gce_instance"
labels."compute.googleapis.com/resource_name"="gke-qcl-cluster-nb-n2-highmem-4-daaf2382-wgxm"

I can then search through these messages, with textPayload=~".*Kill.*" on to the previous query, to see OOM kills (because those messages look like [11686.043641] Out of memory: Kill process 2603 (flasherav) score 761 or sacrifice child).
This produces no kills. Just to be sure, I removed the resource name check there, and it found nothing either.
So with this, we can confirm that these are not being OOM killed.

I then looked at the notebook logs with resource.labels.pod_name="jupyter-jtkmckenna" and found nothing relevant there either.

Since the whole pod didn't die, this isn't from the jupyterhub idle culler.

Another example of what I suspect is the same issue, (although they were batch processes that weren't being actively watched as closely), were the failure cgrid jobs (an in house python program). These would randomly get killed in a similar way (under the user carhels)
On Saturday:
"
2i2c crashed again right before 12 today when I tried to run the final CATF simulations. According to the history there were no out of memory kills, which there shouldn't have been.

Now I cannot even restart the server again

get the feeling that they cannot handle close to 100% load for extended periods.
They rarely failed for me when I'm not running something hard.
Which brings me to the obvious question, why should we have a server if it only can handle light loads?
[2:53 PM]
Ok, got back in again but first my account was wiped clean. Logged out and back again and now I see my stuff again.
[4:56 PM]
Crashed again

We got a follow up ticket that this issue is now time critical https://2i2c.freshdesk.com/a/tickets/1163

I've responded on the ticket with some suggestions and changes.

This solution is working well for us. The system seems stable and our executing until completion. Would it be add a system to ping us when we've had a server on for more than 24h (since we'll have to remember to shutdown our sessions now)
A https webhook to mattermost would be ideal for us https://developers.mattermost.com/integrate/webhooks/incoming/
This seems something grafana seems to directly support https://grafana.com/docs/grafana/latest/alerting/alerting-rules/manage-contact-points/webhook-notifier/
Would you like us to make a new ticket for a uptime alert webhook?

We dealt with the uptime alert via freshdesk I think. Closing this as completed!


Migration to CircleCi 2

https://circleci.com/docs/2.0/migrating-from-1-2/
https://circleci.com/docs/2.0/ios-migrating-from-1-2/