Refactor badges

Description
This PR bundles all different types of list/grid view badges into a single ds-component.
This way, badges will now be easier to move and align with other content.
This PR also added themed versions of certain components that hadn't been made themeable before.
The following components were made themeable:

SearchFiltersComponent
SearchSidebarComponent
All the existing badge components

Instructions for Reviewers
This PR can be tested by comparing different types of list/grid view and making sure they look the same to the current PR. The only difference should be that the "Private" and "Withdrawn" badges are now aligned with any of the other badges.
Test the themed components by creating a themed version of them and making sure they are rendered as intended.
Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes ESLint validation using yarn lint
[x] My PR doesn't introduce circular dependencies (verified via yarn check-circ-deps)
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.

Unfortunately, I'm going to have to move this to 7.6.  Just not enough time left to consider this in 7.5.  That said, I'd like to move this into 7.6 as soon as possible (after 7.5 is complete & once the merge conflicts can be resolved in this PR)

@LotteHofstede
we don't need to add the new input to all listable components, it's enough to add in the AbstractListableElementComponent that is extends by all. And to pass along by the ListableObjectComponentLoaderComponent you just need to add here the new input
For what i've seen in your code you only need to distinguish beetween the mydspace context and all the rest, is it correct ? in this case you could only provide a value for the new input in the mydspace list components only, that are a few.
maybe i'm missing something but it would not seem to require so much work, please correct me if i'm wrong

@atarix83 That would still require changing quite a few components, besides I'm not fond of the idea of adding an input to something as general and widely used as listableObjectComponents for something only necessary in a handful of specific cases.
The context was designed to deal with cases where a specific listable object component needs more info about the wider context in which it's being shown. So I think @LotteHofstede is using it correctly here. Since we have nested listableObject components, you have one, general context for the outer one (workflow or workspace) and one more specific context for the inner one: the status specific ones.
You could argue that context shouldn't be a simple enum, but rather a more complex object (something you could add a mydspace status property to). But changing the type of context would require us to change literally every component using context now, so that would definitely be out of scope of this PR

@tdonohue Thanks for the feedback, I added your proposed comment and did the same for some of the other more obvious contexts.

@LotteHofstede : Can you look at the minor fix I noted above in my last review?  I think it's a small bug...but if you disagree let me know. Once you have a chance to look at that and respond (or fix it), then we can merge this as it's already at +2.  Thanks!

@tdonohue I applied the fix for your feedback!

Thanks @LotteHofstede !  Merging as this is at +2


Reverse sort and adding tags to output?

Trying to get a handle on an explosion of snapshots.  Would be useful to do a reverse sort on the Created column (for now, resorting [snicker] to running in csv output and then doing a | tac.
Also, any way to get the tags into the output?

Indeed a reverse sort flag will come in handy, so we will add it.
For the tags of a snapshot, we do sync them when fetching a snapshot. We do not display them as part of the listings, as we have not find a way to do so while keeping a readable listing output.
Although less scriptable, the snapshot's tags will be shown with the awless show output if I am not mistaken. For example: awless show snap-071573255d4a764ce.
Note: when running multiples times a listing to craft my output, I find very useful the global flag --local to avoid re-fetching everything each time. (Of course at least a sync or cmd triggering a sync should have been done). I even have an alias alias awl=awless --local.

You might know as well that you can filter by tags. Doing a awless ls -h:
...
--tag stringSlice         Filter EC2 resources given tags (case sensitive!). Ex: --tag Env=Production
--tag-key stringSlice     Filter EC2 resources given a tag key only (case sensitive!). Ex: --tag-key Env
--tag-value stringSlice   Filter EC2 resources given a tag value only (case sensitive!). Ex: --tag-value Staging
...

Also tags are so central to good AWS management that we will add other features around them. @deinspanjer If you have ideas let us know.

Thanks, this is very helpful.  One more that I hope is related enough to tag along as a comment.
I ran list instances, list volumes, list snapshots and copied each of those outputs into an Excel workbook.  I then used COUNTIF and SUMIF to count up the number of snapshots per volume and the number of volumes per instance.  This gives me a pretty good accounting for the bad actors and low hanging fruit.
I grabbed one of the volume ids and I wanted to get just the snapshots for that volume so I tried running a command that gave an error.
➜  aws git:(master) ✗ awless list snapshots --ids --filter Volume=vol-91f5176b
[error]   Invalid filter key 'Volume'. Expecting any of: ID, Name. (Note: filter keys/values are case insensitive)

Is it intended that if you use --ids, you can only filter on the ID or the Name?

That is a good point about the inspector.  Let me take a look at the bucket sizer one and see what I can do.  I'll open a new issue if I get stuck or have something to offer.

When you say I wanted to get just the snapshots for that volume, since the volume in our model is the parent of a snapshot, if you awless show a volume you should see in the # Relations section the snapshots children - if they are any (otherwise it is a bug). Ex:
awless show --local vol-00f2d4cea0738f55e

But as you see the output of the show command is not yet really scriptable or grepable.

Yes, I do see the snaps listed for a volume.  What I was missing was that the show instance doesn't list the volumes attached to it.  The only mention of them is in the Depending on: section where it shows their aliases -- @instance.name[volume], @instance.name[volume]


SDK gem 1.0.3 is broken by depending on opentelemetry-common 0.19.3

I noticed test failures in our distro of NoMethodError: undefined method truncate_attribute_value for OpenTelemetry::Common::Utilities:Module and tracked it down to I think being that SDK gem 1.0.3 was published the other day and still depends on opentelemetry-common gem from December, ~> 0.19.3 which does not have this method.

Is this not working at all? Or is it an issue with certain use cases?

@arielvalentin any setting of attributes on a span when the attribute length limit is configured. Since the default is nil I guess there is a good chance many won't encounter this?

Ok. Are you available to submit a fix for this?

I think you just need to release a new version for the opentelemetry-common gem (and it's good to update the dependency in the SDK gem):
https://github.com/open-telemetry/opentelemetry-ruby/blob/ee5ece7d04a9922db9d9600c5f4de27b71491907/common/CHANGELOG.md?plain=1#L1-L3
We are also waiting for the release.

After a few attempts we were able to release the latest version of common 0.19.5
https://github.com/open-telemetry/opentelemetry-ruby/pull/1247
Thank you all for your patience.


[BUG] /ep settings and /ep setup notworking

Details
Spigot Version:
please run /ep debug and attach a screenshot

Describe the bug
A clear and concise description of what you expected to happen.
/ep setup and /ep settings unable to load
Expected behavior
A clear and concise description of what you expected to happen.
should be loaded well
Screenshots
If applicable, add screenshots to help explain your problem.

I am running into the same issue if more info is needed to help debug. Currently, I'm not able to try out the plugin with this error occurring. I hope this can be fixed soon!

Please attach the  log.
Thank you!

Those are screenshots that it doesn't work

Please try the latest version (1.6.5-R3)

It still doesnt work.

and there is no error in the console?

There only show 'Null' in the console, just like the pictures on the top comment.

Please delete or rename the main folder from EasyPrefix, so that the files can be regenerated


Proxy localhost:port -> subdomain.localhost:another_port doesn't work

Expected behavior
I have two running dev servers on localhost. I need to proxy a request from localhost:4000 to subdomain.localhost:3000
Actual behavior
On server launch I receive:
[HPM] Proxy created: /  ->  http://agent.localhost:3000
On request I see following message in console:
[HPM] Error occurred while trying to proxy request /config from localhost:4000 to http://agent.localhost:3000 (ENOTFOUND) (https://nodejs.org/api/errors.html#errors_common_system_errors)
And in a browser I receive following 504 response:
Error occurred while trying to proxy to: localhost:4000/config
At the same time requesting that endpoint in Chrome works fine.
Setup

http-proxy-middleware: 0.18.0
server<EMAIL_ADDRESS><EMAIL_ADDRESS>
proxy middleware configuration
  app.use(require('webpack-dev-middleware')(compiler, {
    noInfo: true,
    publicPath: config.output.publicPath
  }));
  app.use(require('webpack-hot-middleware')(compiler));

  config.devServer.proxyConf.endpoints.forEach((endpoint) => {
    app.use(endpoint, proxy(config.devServer.proxyConf.config));
  });

  proxyConf: {
    endpoints: [
      "/config",
    ],
    config: {
      target: "http://agent.localhost:3000",
      secure: false,
      changeOrigin: true
    },
  }

server mounting
  app.listen(port, function (err) {
    if(err) {
      console.log(`Error during app launch: ${err.message}`.red);
    } else {
      console.log(`Server is listening on ${port}...`.green);
    }
  });

Has anyone found a solution to this issue yet?
I'm also getting a ENOTFOUND error, but my target is slightly different, but more or less the same concept.

Care to create a minimal working setup so it can be investigated?

Afterall I just started to proxy to another environment. So solution wasn't found

Is this helpful?
https://stackoverflow.com/questions/36745592/node-http-proxy-middleware-not-working-with-local-servers-as-targert

I've hit this as well. My path is correct for the target server. I can hit the same endpoint in Chrome (admin.localhost:8080), but the proxy middleware will not route it.

putting my local subdomains on hosts file solved the issue to me, not the best solution but ok


Create Automation  test script for certificate Api

Steps

[ ] write the script for certificate Api
/api/v1/Certificates

Outcome :

[ ] Running certificate Api successfully
@Andolepriti
@krishnakr6

[ ] Need to refer the below link
https://docs.google.com/spreadsheets/d/1LOOIJ-BM9-3YxgGdyjHe8v9HPMLaut5SJlxvgQncrZw/edit#gid=0

[ ] I written the script
[ ] found few error
[ ] need to check

[ ] I completed the test script for certificate Api

[ ] Output : expected and actual result are matched

[ ] for negative scenario getting java.lang.AssertionError

https://docs.google.com/spreadsheets/d/1LOOIJ-BM9-3YxgGdyjHe8v9HPMLaut5SJlxvgQncrZw/edit#gid=0

[ ] From above sheet i completed the script for certificate Api

[ ] Output : expected and actual result are matched


[NW Reconfig] add header to network  requests with epoch

@akichidis there is a test case for that one in epoch_filters tests


Left, not right

"On the right hand side of the Routing and Remote Access management panel you will see an icon with your servers name next to it, right click this and select "Configure and Enable Routing and Remote Access"."
It's not on the right hand side but left.
Florent

Do you have a document reference or URL?

This: https://www.google.com/url?sa=i&source=images&cd=&ved=2ahUKEwiIgZf-9-DgAhUBxxoKHfpUDREQjRx6BAgBEAU&url=http%3A%2F%2Ftechgenix.com%2Fconfigure-vpn-windows-server-2012-r2%2F&psig=AOvVaw0a21ki52KMzgK27YrZOcQY&ust=1551529510905299

WEll - that is a third party site.
I suspect you mean https://docs.microsoft.com/en-us/virtualization/hyper-v-on-windows/user-guide/nested-virtualization-azure-virtual-network
I found it, corrected it!

Oh yes sorry, you’ve right. It’s on this thread. I just mentionned the screenshot to confirm that it’s on left and not right.

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Add bloom, blur and colourmap filters

From constellation!

Having thought about it a bit, having colourmaping as a built in default filter is a little weird. I'll make weird filters an extension later!


Error after GIS Server site creation - "Undefined property IsWorkflowManagerDeployment

After the ArcGIS GIS Server site has been created I am getting the following error, even though there is no Workflow Manager deployment involved.

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request

Module Version

3.3.0

Affected Resource(s)

ArcGIS_Server

Configuration Files

"ConfigData": {
        "Version": "10.8.1",
        "ServerContext": "hosting",
        "PortalContext": "portal",
        "FileShareLocalPath": "F:\\Software",
        "FileShareName": "\\\\mneu-d-a-agsh10.anglo.local\\Software",
        "ServerRole": "GeneralPurposeServer",
        "Credentials": {
            "ServiceAccount": {
                "Password": "<removed>",
                "UserName": "<domain account>",
                "IsDomainAccount": true,
                "IsMSAAccount": false
            }
        },
        "Server": {
            "LicenseFilePath": "\\\\mneu-d-a-agsh10.anglo.local\\Software\\Esri\\ArcGIS\\10.8.1\\Licenses\\gis-server-adv.ecp",
            "Installer": {
                "Path": "\\\\mneu-d-a-agsh10.anglo.local\\Software\\Esri\\ArcGIS\\10.8.1\\Installs_DSC\\ArcGIS_Server_Windows_1081_175203.exe",
                "InstallDir": "E:\\Program Files\\ArcGIS\\Server",
                "InstallDirPython": "E:\\Python27",
                "PatchesDir": "\\\\mneu-d-a-agsh10.anglo.local\\Software\\Esri\\ArcGIS\\10.8.1\\Patches_DSC\\Server"
            },
            "ServerDirectoriesRootLocation": "F:\\arcgis\\hosting\\directories",
            "LocalRepositoryPath": "F:\\arcgis\\hosting\\local",
            "ServerLogsLocation": "F:\\arcgis\\hosting\\logs",
            "ExternalLoadBalancer": "<removed>",
            "InternalLoadBalancer": "",
            "PrimarySiteAdmin": {
                "UserName": "siteadmin",
                "Password": "<removed>"
            },
            "ConfigStoreCloudStorageAccount":{
                "UserName": "mneustdevarcgishub001.file.core.windows.net",
                "Password": "<removed>",
                "CloudStorageType": "AzureFiles",
                "AzureFileShareName": "arcgis",
                "CloudNamespace": "dev"
            },
            "ServerDirectoriesCloudStorageAccount":{
                "UserName": "mneustdevarcgishub001.file.core.windows.net",
                "Password": "<removed>",
                "CloudStorageType": "AzureFiles",
                "AzureFileShareName": "arcgis",
                "CloudNamespace": "dev"
            }
        },

Expected Behavior

It should carry on with the install.
Actual Behavior

Failing to continue due to the error.
Steps to Reproduce

I just run the script. Not sure what is causing the error.
Important Factoids

References

Hi @vassilo,
The error you are experiencing can be caused by having multiple versions of the ArcGIS Module installed. Let's ensure all previous versions of the module are removed.
For example, check the following location C:\Program Files\WindowsPowerShell\Modules\ArcGIS to ensure only one version exists. If there are multiple versions, remove all but 3.3.0, close the powershell cmdlet, then re-launch a fresh cmdlet and re-run.
https://github.com/Esri/arcgis-powershell-dsc/issues/354#issuecomment-975073570
Thanks,
Cameron K.

Hi @cameronkroeker
I will go and hang my head in shame. It was indeed the problem. Should have checked that first before posting.
Thanks for the help.
Regards,
Vassilo


Center footer content

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

https://github.com/openprivacytech/openprivacytech.org/issues/9

Hi, thanks for the PR! Closing this as an issue was raised first by someone else.
Please check #3 if you can contribute.

Hi, only started working on this because someone else took the issue that I was already working on so I thought first come first served... anyway, I can have a look at 3

Hey @zghp, going to open this PR again, this issue seems to be hanging for some time.
Once reviewed It will be merged.


Bug/multiple of with a float

Reasons for making this change
Fixes #997
If this is related to existing tickets, include links to them as well.
Checklist

[ ] I'm updating documentation

[ ] I've checked the rendering of the Markdown text I've added
[ ] If I'm adding a new section, I've updated the Table of Content

[x] I'm adding or updating code

[x] I've added and/or updated tests
[ ] I've updated docs if needed
[x] I've run npm run cs-format on my branch to conform my code to prettier coding style

[ ] I'm adding a new feature

[ ] I've updated the playground with an example use of the feature

@n1k0 @glasserc would you be able to take a look at this?


Mac OSX build

I'm trying to download the binary version of phantom.js (as suggested by the website). I'm getting an error when trying to download the Mac OSX static build. I will build from source if necessary, but I'd really prefer not to.
Specifically, using wget, I get ERROR 403: Forbidden
If I tryi to download directly from the website, I get the following error:
<Error>
    AccessDenied
    <Message>Request has expired</Message>
    <Expires>2016-03-29T12:04:25Z</Expires>
    <ServerTime>2016-03-29T12:13:09Z</ServerTime>
    <RequestId>113CB62C81A5180C</RequestId>
    <HostId>
        nZ3hFFa0HhF2jQet6enqWDPwA48gFmLU8G04iVeF4oXvNbFSVfTa8NPOxGy64K40pUJ2UXRwVoo=
    </HostId>
</Error>

Hi. Sorry, but this is not the PhantomJS issue. It might be a problem with your network. We received message from people about problems with bitbucket. But we can't do anything with it. Sorry.
You can find more information here #13953


WebkitBrowser Sync-up from Metro plugin version

Config changes syncup
GLIB API updates + MemoryObserverImplmentation moved to WebKitBrowser class
InjectedBundle: GLIB APIs implementation added
Library restructuring changes

Copyright scan failure
Commit: 7354e151f03a7b0e259189e710edb2204379a6a6
Report detail: https://gist.github.com/e7aff49c5ee39a61d33153b2b8f895ad

@emutavchi
Could you please review and merge this? LGI needs issues fixed in these changes (splitup of Plugin and implementation part,m these are now in separate libraries to facilitate process containers)

@HaseenaSainul @MFransen69 current sprint is 2201. The code in this branch differs a lot from what we have in the active sprint. Would you mind creating PR against 2201?

Sure @emutavchi. will create new one against latest sprint.


Claims default dock is not a valid dock plist

I'm getting these errors:
/System/Library/User Template/English.lproj/Library/Preferences/com.apple.dock.plist does not seem to be a home directory or a dock plist
Well, it certainly seems to be a dock plist.  Why is dockutil refusing to work with the template dock plist?  I'm using the same syntax as the -h help page.   This is in 10.10.3 and dockutil 2.0.2.
The only workaround I can think of is for it to alter the current user's dock and then copy that .plist to the template, changing the owner, group, and permissions.  But I'd prefer not resort to such a kludge.
Thanks for any thoughts!

There is no default file at /System/Library/User Template/English.lproj/Library/Preferences/com.apple.dock.plist on my 10.10.3 system.  You may need to put one there.  Or if you have one there, make sure you are running dockutil as root.  Or maybe that plist is corrupt or invalid.
sudo plutil -lint  '/System/Library/User Template/English.lproj/Library/Preferences/com.apple.dock.plist'

Also getting the '/System/Library/User Template/English.lproj/Library/Preferences/Library/Preferences/com.apple.dock.plist does not seem to be a home directory or a dock plist' message when attempting to update the template dock (sending the command as root).
sudo plutil -lint '/System/Library/User Template/English.lproj/Library/Preferences/com.apple.dock.plist' returns '/System/Library/User Template/English.lproj/Library/Preferences/com.apple.dock.plist: OK'
Am I understanding correctly that I should be able to modify the template dock using dockutil 2.0.2. on 10.10.3?

Dock templates move in 10.6 to:
/System/Library/CoreServices/Dock.app/Contents/Resources/*.lproj/default.plist

Is this still an issue.  I'm not able to reproduce it.

I had this issue when I was not running docuktil as root. "dockutil --add.." versus "sudo dockutil --add". Once I added a "sudo", everything started working.

Closing this old inactive issue.


chore: 🤖 Pin transitive polkadot deps

Pinning transitive polkadot dependencies in case they have breaking
changes
Description
Pins transitive dependencies for polkadot in the case of a dependency of the dependency not respecting semver.
If we upgrade polkadot to a new version with incompatible deps we will need to update the resolutions. It will give warnings when running yarn when that happens.
For util-crypto I left a few as they were:
"@polkadot/wasm-crypto" "^6.0.1"
 "@polkadot/x-randomvalues" "9.0.1"

My reasoning being that the risk profile in crypto deps are more likely to be critical bug fixes than user facing breaking api changes.
Breaking Changes

JIRA Link
Disovered as part of DA-348
Checklist

[ ] Updated the Readme.md (if required) ?

On second thought I pushed a feat commit.
I am thinking this could clear up the version mismatch warnings for consumers.

:tada: This PR is included in version 15.0.0-alpha.3 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 15.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Drum Kit challenge

@KapilKumar7 Kindly assign me this under Hacktoberfest 2022.

Can you please elaborate more about this challenge, How you will gonna add Drumstick to this project?

@KapilKumar7 In this project , by pressing different keys from keyboard , it produces different types of drum sounds .

But we have just readme the file, We still don't have deployed that project on the web, then where will these sounds produce? On which part you are thinking of adding these sounds ?

Sir , in readme file we can ellaborate this project and by adding picture of my project we can explain the project clearly.

You can work on this by forking the repo and Let me show some work (by attaching some screenshots) on that,  If it can be added to our repo then I will definitely Merge the code with the main codebase. Thanks.


Memory leaks in typeinference module

It seems that certain input source code can cause memory leak in typeinfer module. We are still locating the leak and more details will come soon.

After CFG module was fixed, the memory leaks in typeinfer module seem to have gone. As such, we are closing this issue


Article index page

What is this PR trying to achieve?
Adding a first stab at the article index page.
We are using our current promo styling.
There are some questions around the designs that need addressing before implementing those (as a separate PR).
What does it look like?

TODO

[ ] Pagination controls

For interface components

[ ] Browser testing - Have you checked the component in our supported browsers
[ ] No javascript - Have you checked the component with javascript disabled
[ ] A11y testing - Have you run pa11y against the component and fixed/verified all errors, warnings and notices?

Have the following been considered/are they needed?

[ ] Tests?
[ ] Docs?

Waiting for #383 to be merged first.


Delete codeql-analysis.yml

https://github.com/peterjc/flake8-black/issues/50

Codecov Report

Merging #72 (8361932) into main (1f587f4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   48.71%   48.71%
=======================================
  Files          11       11
  Lines         273      273
  Branches       51       51
=======================================
  Hits          133      133
  Misses        140      140

Continue to review full report at Codecov.

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


Fluid Tank's NBT data is ignored when placed on existing Fluid Tank

Describe the Bug
If you place a Fluid Tank with NBT data (containing a fluid) on top of another Fluid Tank, its contents (NBT data) gets ignored.
This doesn't happen on Create Forge.
Reproduction Steps

Place a Fluid Tank
Place a different fluid tank, put a liquid into it and pick it up with CTRL + middle mouse click
Place the fluid tank with NBT data on top of the first fluid tank

Expected Result
Create Forge: https://youtu.be/8TatW1hOBjw
Screenshots and Videos
Create Fabric: https://youtu.be/s-XmPCm25w8
Crash Report or Log
No response
Operating System
Windows 11
Mod Version
0.5.0g
Minecraft Version
1.19.2
Other Mods
none
Additional Context
No response

On Forge, all items are placed during block snapshots, so the state isn't updated immediately. The data is updated, then the block updates, so the 2 tanks merge.
On Fabric, the tanks merge as soon as the second one is placed. When the NBT is loaded, since the tank is not a controller, it just doesn't load the data.


Redirect from a set of configurable sources

From https://advancedweb.hu/how-to-use-cloudfront-functions-to-change-the-origin-request-path/
function handler(event) {
	var request = event.request;
	request.uri = request.uri.replace(/^\/[^/]*\//, "/");
	return request;
}

CloudFront Functions event structure.

I can route based on the Host header. The CloudFront Functions event's request object has the headers. So I can have some-form.douglas-naphas.org route to a Google Form, and douglas-naphas.org route to a Google Site, using the same distro.

Testing
Before delegating DNS with the hosting provider, can I send, with CURL or fetch, a request to the distro (with its CloudFront domain name) where the HTTP Host header is the domain from a from-value?

I'd better consider the CloudFront Functions JavaScript runtime features.

For testing: https://daniel.haxx.se/blog/2018/04/05/curl-another-host.


Publish to npm

@dmtrKovalenko could you let me know your npm username so I can give you access to publish this package?

@cpojer -> dmtr.kovalenko

Awesome! I added you – can you let me know once you published this package?

@cpojer Published to npm - https://www.npmjs.com/package/@react-native-community/status-bar
Also I would like to add to this repo:

Tests
Typescript support
Better documentation + badges 😮

And also will create a commit to depracte status-bar module in core


Error when starting container

Getting this error when trying to start a specific container:
Cannot start container 20e0e0d4c73f65b6c4cac2d7efff6267ba316f106ae1b55b638b7efba5b82fe6: write /sys/fs/cgroup/cpuset/docker/20e0e0d4c73f65b6c4cac2d7efff6267ba316f106ae1b55b638b7efba5b82fe6/cpuset.cpus: invalid argument
This container image was built, pushed to a private registry (running in the "official" docker-registry container), pulled, and a container was created with default options before attempting to start it
Running Ubuntu 14.04 in a VMWare VM on Windows 8.1.
Docker info:
Client version: 1.1.2
Client API version: 1.13
Go version (client): go1.2.1
Git commit (client): d84a070
Server version: 1.1.2
Server API version: 1.13
Go version (server): go1.2.1
Git commit (server): d84a070

So, this issue doesn't merely manifest as a consequence of NULLs in JSON -- I'm seeing it with:
sudo docker run --cpuset 2 <long list of options that work fine without cpuset>

... it may be a case of the machine not having 3+ CPUs, in which case this is simply a confusing and unhelpful error message that could be better handled by Docker ensuring that the value given to cpuset is sane for the machine at hand.


Multiple artifact signers from metadata

-- Modify framework to allow multiple artifact-signers to be created from single metadata file
-- Will help in avoiding roundtrip to remote key store/vault services (such as upcoming FSecure Interlock HSM)

Implemented alternate approach to cache session with same credentials.


Template selection improved

Now you can select the templates by passing as an argument only the name of this one

Hello @Jalkhov! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file jamstack/__main__.py:

Line 41:5: E265 block comment should start with '# '
Line 42:5: E265 block comment should start with '# '


Concourse deployment of docs.cloud.gov

Acceptance criteria

[ ] Create reusable Concourse pipeline
[ ] Site is deployed to docs.fr.cloud.gov by pipeline


Code cleanup + reorganising db_info + add projects + standarisation update movie

This MR is linked to several issues: it resolves issue #89 , #98 .
There is more information in the commit messages. But in short:

This commit gets rid of db_info_dict and uses server_connection instead.
Renames the "db_info" attribute of the projectprocessor for "csv_paths"
I moved most of the "get_db_table" from the projectprocessor to db_utils and called it using the "project.db_connection" in different utils
I deleted widgets that were only specific to uploading movies to spyfish aotearoa (these were very project-specific and will be redundant with chloudina)
I also fixed an issue when we loaded the "Project" class from panoptes (e.g. it got confused with our "Project" attribute)
It adds two new projects
It standardizes  the uploading of movies

When we merge this MR, we should also merge the MR in data management "Deleted unnecessary tut#11-13 and updated t#3 #78". Right @victor-wildlife ?

Yes, @Diewertje11 - We should also MR the "Deleted unnecessary tut#11-13 and updated t#3 #78" in the data management repo


Help: No username when running tmux in a docker container with alpine

Hi
I have tried to get things working by installing tmux in a docker image and apply your config for tmux and wonder how you are getting the username because when I do a whoami I get the username but i cant get a username in the right corner in status bar (See picture below).

More info:
Found some a issue where some investigated was done,  you asked them to do this cmd
 cut -c3- ~/.tmux.conf | sh -s _username

in ubuntu it outputs correctly
but in alpine i get this error
>   ~/.tmux$ cut -c3- ~/.tmux.conf | sh -s _username

ps: unrecognized option: t
BusyBox v1.33.1 () multi-call binary.

Usage: ps [-o COL1,COL2=HEADER] [-T]

Show list of processes

        -o COL1,COL2=HEADER     Select columns for display
        -T                      Show threads

So I suppose alpine is different in this way handling ps!
Do you know of a workaround that don't include changing OS?

Hello @Looooopy 👋
Indeed Oh my tmux! doesn't support BusyBox. I think you can try installing the procps package.

Hello @Looooopy 👋
Indeed Oh my tmux! doesn't support BusyBox. I think you can try installing the procps package.

Ok I will try, thanks for the info

Yes it did work now!
Thanks again for the tip!


feat(operation-summary): add new 'operation-summary' rule

PR summary
This commit includes two additions:

added the new operation-summary rule
enabled the spectral:oas operation-operationId rule

and two subtractions:

removed the legacy no_summary rule
removed the legacy no_operation_id rule

PR Checklist
General checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)
[x] Dependencies have been updated as needed
[x] .secrets.baseline updated as needed?

Checklist for adding a new validation rule:

[x] Added new validation rule definition (packages/ruleset/src/rules/*.js, index.js)
[x] If necessary, added new validation rule implementation (packages/ruleset/src/functions/*.js, updated index.js)
[x] Added new rule to default configuration (packages/ruleset/src/ibm-oas.js)
[x] Added tests for new rule (packages/ruleset/test/*.test.js)
[x] Added docs for new rule (docs/ibm-cloud-rules.md)

Checklist for removing an old validation rule:

[x] Removed old rule implementation (packages/validator/src/plugins/validation/*.js)
[x] Removed or adjusted testcases (packages/validator/test)
[x] Updated default configuration to deprecate old rule (packages/validator/src/.defaultsForValidator.js)
[x] Removed docs of old rule (docs/ibm-legacy-rules.md)

:tada: This PR is included in version 0.10.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.61.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


SOLR-17285: Move RemoteSolrException to SolrClient in v10

https://issues.apache.org/jira/browse/SOLR-17285
Description
Moved RemoteSolrException to SolrClient from BaseHttpSolrClient
Solution
Moved RemoteSolrException class to SolrClient. Manually changed import statements from BaseHttpSolrClient to SolrClient if is not needed anymore for RemoteSolrException. Changed BaseHttpSolrClient.RemoteSolrException to SolrClient.RemoteSolrException
Tests
OUTPUT-org.apache.solr.cloud.HttpPartitionOnCommitTest.txt
OUTPUT-org.apache.solr.cloud.ReplaceNodeTest.txt
OUTPUT-org.apache.solr.handler.TestSolrConfigHandlerConcurrent.txt
OUTPUT-org.apache.solr.pkg.TestPackages.txt
I ran ./gradlew dev. It worked fine
./gradlew check did not work. It said there were failing tests on unrelated files
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[ ] I have created a Jira issue and added the issue ID to my pull request title.
[ ] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

I am looking at BaseHttpSolrClient again, and I see that RemoteExecutionException is also something that'll need to be moved.  It's not used in many places.  I think it's reasonable to include that in the scope of this JIRA issue and PR; WDYT?  But I don't love the name "RemoteExecutionException" and I'm not even convinced it needs to exist; like couldn't RemoteSolrException take a payload?  Publishing it to SolrClient would give it undeserved visibility IMO.  Hmmm.  On the other hand, based on how it's used, this is so internal that maybe it should simply move to HttpSolrClientBase (which is the successor to BaseHttpSolrClient).  I welcome your thoughts.

@samuelrivascoding how shall I credit you (name you) in CHANGES.txt?  If I don't hear back, it'll be literally that.

Will merge as soon as Monday.  I also plan to handle RemoteSolrException in a follow-up.


API not working in PROD?

I don't know if I am doing something wrong. I have installed the plugin and uncommented /foo route so that it replies with {msg: 'foo'}. It works great in dev. If I go to "localhost:3000/foo" I will get msg and in the app (running at 8080) if I do axios.get("/foo") I also get msg as well.
However I can't get the same results in PROD. I npm run build then served the "dist" folder with serve -s dist. Doing localhost:5000/foo (serve serves on 5000 by default), localhost:5000/#/foo or axios.get("/foo") inside the app doesn't work at all.
How can I fix this?

Hi,
To serve your app in production you have to use the yarn express:runcommand.
See the documentation:
https://github.com/mathieutu/vue-cli-plugin-express#use-your-app-in-production


Importing @starkware-industries/starkware-crypto-utils returns an empty object

Describe the bug
I'm upgrading from 0.0.2 to 0.2.0.
Importing from '@starkware-industries/starkware-crypto-utils' returns an empty object.
To Reproduce
const starkwareCrypto = require("@starkware-industries/starkware-crypto-utils");

console.log({ "starkwareCrypto": starkwareCrypto });
// => { starkwareCrypto: {} }

console.log({ "starkwareCypto.keyDerivation": starkwareCrypto.keyDerivation });
// => { 'starkwareCypto.keyDerivation': undefined }

Here's a CodeSandbox
Expected behavior
I should be able to do this without an error:
const starkwareCrypto = require("@starkware-industries/starkware-crypto-utils");

const ethSignature =
  "0x21fbf0696d5e0aa2ef41a2b4ffb623bcaf070461d61cf7251c74161f82fec3a43" +
  "70854bc0a34b3ab487c1bc021cd318c734c51ae29374f2beb0e6f2dd49b4bf41c";

starkwareCrypto.keyDerivation.getPrivateKeyFromEthSignature(ethSignature);
// TypeError: Cannot read properties of undefined (reading 'getPrivateKeyFromEthSignature')

Might be because of https://github.com/starkware-libs/starkware-crypto-utils/commit/01e5c58b09b46bac6c4b832b4a8a8601f6075db0 since it only exports as an ESM module
@dan-ziv

Doesn't work with ESM import either: CodeSandbox

:tada: This issue has been resolved in version 0.2.1-dev.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Use purescript-record in Data.Lens.Record

Shall we use purescript-record get and set functions?

Yes, I think we should


Fatal error during startup

What issue do you have? Please be as thorough and explicit as possible.
This plugin causes a fatal error during the startup of Homebridge:
# Fatal error in , line 0
# unreachable code
#FailureMessage Object: 0x75afd790

When I uninstall the plugin the error disappears, when I install it again it's back.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
I'm using the Homebridge UI, Homebridge is running in a docker container on a Raspberry Pi.

Which version of Homebridge/HOOBS do you have?
Homebridge v1.4.1 (HAP v0.10.2)

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Raspberry Pi, Docker image oznu/homebridge:ubuntu (latest version) - and respective Node.js/npm versions.

Which version of this plugin (homebridge-deebot) do you have? Has the issue started since upgrading from a previous version?
Issue occurred with version 5.0.4 and 5.0.5.

Which ECOVACS devices do you have that are causing issues? Please include product models if applicable.
None. I'm waiting for/working on the support of Yeedi robots.

Please paste any relevant logs below.
[6/10/2022, 10:18:32 PM] [HB Supervisor] Starting Homebridge with extra flags: -I -P /homebridge/node_modules
[6/10/2022, 10:18:33 PM] [HB Supervisor] Started Homebridge v1.4.1 with PID: 2500
[6/10/2022, 10:18:35 PM] Loaded config.json with 9 accessories and 3 platforms.
[6/10/2022, 10:18:35 PM] Loaded 0 cached accessories from cachedAccessories.
[6/10/2022, 10:18:35 PM] ---
[6/10/2022, 10:18:38 PM] Disabled plugin<EMAIL_ADDRESS>[6/10/2022, 10:18:38 PM] ---
[6/10/2022, 10:18:38 PM] The plugin "homebridge-deebot" requires Node.js version of ^14.19.3 || ^16.15.1 which does not satisfy the current Node.js version of v16.15.0. You may need to upgrade your installation of Node.js - see https://homebridge.io/w/JTKEF

#
# Fatal error in , line 0
# unreachable code
#
#
#
#FailureMessage Object: 0x762fe790

... just spotted the hint in the logs re: the node.js upgrade - this is new for version 5.0.5 of the plugin - it did not occur for version 5.0.4, while the fatal error was the same. But upgrading node.js would need to be done by the container provider...

Hi @apfelnutzer
Can you have a look at this section of this wiki and potential solutions in the issue links?
https://github.com/oznu/docker-homebridge#2-container-will-not-start-on-older-versions-of-raspbian
Let me know how you get on
Thanks!

I had a quick lock at the issues referenced on that page and for me they look different. Anyway: As I run Homebridge in a docker container (image oznu/homebridge) I'm not able to change anything I the included OS, am I? And I don't think it would help to change the OS on the Raspberry itself (which is up to date, of course).
Moreover a I mentioned the hint re: the outdated node.js version did not occur with 5.0.4, while the error was the same...

https://github.com/oznu/docker-homebridge/issues/445#issuecomment-1150516764
This looks a very similar issue and was resolved with an OS update.
I'm not sure what changes I can make to the plugin since I don't think this is an issue with the plugin, rather an issue with the environment that the plugin is running in.
You can ignore the message about the node version. This won't have anything to do with it.

oznu/docker-homebridge#445 (comment)
This looks a very similar issue and was resolved with an OS update.
I'm not sure what changes I can make to the plugin since I don't think this is an issue with the plugin, rather an issue with the environment that the plugin is running in.
You can ignore the message about the node version. This won't have anything to do with it.

You are right, that comment looks quite similar - I must have missed it earlier. I'll follow up on this later (today?)! Still the OS of my Raspberry is up to date, but we'll see...

@bwp91 Hm. I'm running Buster too, so maybe I should consider upgrading to Bullseye? Might have too little space left...

Upgrading from Buster to Bullseye solved the issue - thanks again for the hint! The log show still
[6/12/2022, 2:15:21 PM] The plugin "homebridge-deebot" requires Node.js version of ^14.19.3 || ^16.15.1 which does not satisfy the current Node.js version of v16.15.0. You may need to upgrade your installation of Node.js - see https://homebridge.io/w/JTKEF

though, but everything seems to work. Will now try out the new beta with my Mob Station!


[Coverity] fix coverity issue

Add checking return value

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #513. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


cleaner local state?

right now running orbit locally ( node index.js) uses some local state in its home directory. its not clear what is "dirty" and can be removed. (its not super clear from the gitignore either)
it may be better to have either:

a clear account of what those files are in the readme, (or via clean target in a Makefile /  Gruntfile)
or put all those under a directory, like orbit-data or something.

This should be fixed now in the pubsub branch. I'm gonna keep this open until the changes are in master but do let me know if you still find issues with the dev directory state.


Add new debug questions & function question

Adds 5 new debug questions and 1 function/parsons
They are probably on the upper end of difficulty; factorial (debug) and the existing factorial question are likely too difficult, should I remove the two?

Adds 5 new debug questions and 1 function/parsons
They are probably on the upper end of difficulty; factorial (debug) and the existing factorial question are likely too difficult, should I remove the two?

@lucyturn3r what do you think about factorial questions? My first instinct was they might not be appropriate as non-maths teachers might find it intimidating, but since there is already one in there and both of them have an example of a factorial then it might be fine? They're good questions.

Adds 5 new debug questions and 1 function/parsons
They are probably on the upper end of difficulty; factorial (debug) and the existing factorial question are likely too difficult, should I remove the two?

@lucyturn3r what do you think about factorial questions? My first instinct was they might not be appropriate as non-maths teachers might find it intimidating, but I'm probably being overly cautious. Since there is already one in there and both of them have an example of a factorial then it might be fine? They're good questions.

To give some history to the existing factorial question; I wrote it early on in codeWOF and accidentally merged it straight to develop. I was told not to undo and it would be reviewed later but that might not have happened. I also have an issue with it (#20)

Adds 5 new debug questions and 1 function/parsons
They are probably on the upper end of difficulty; factorial (debug) and the existing factorial question are likely too difficult, should I remove the two?

@lucyturn3r what do you think about factorial questions? My first instinct was they might not be appropriate as non-maths teachers might find it intimidating, but I'm probably being overly cautious. Since there is already one in there and both of them have an example of a factorial then it might be fine? They're good questions.

To give some history to the existing factorial question; I wrote it early on in codeWOF and accidentally merged it straight to develop. I was told not to undo and it would be reviewed later but that might not have happened. I also have an issue with it (#20)

I think it is fine to leave it in considering that the function question is already out there.


Update Character “roger”

Automatically generated by Netlify CMS

👷 Deploy Preview for dynamic-cheesecake-5fbd8b processing.

Name
Link

🔨 Latest commit
5311180b2f7d91266d0891d91e505625e3a2b6c7

🔍 Latest deploy log
https://app.netlify.com/sites/dynamic-cheesecake-5fbd8b/deploys/632631b0e32b890008dc23dc


Recognize extension ".F90" as free-format Fortran

Very simply, can ".F90" be added as a recognized extension for free-format Fortran?

Looks like it should already be supported: https://github.com/Glavin001/atom-beautify/blob/master/src/languages/fortran.coffee

In this instance, I means specifically capital "F" .F90.  It is case-sensitive.  If I add "F90" to the extensions in fortran.coffee, it works beautifully.  Pun intended.

Huh did not even think of this. Thanks for bringing it up!
I think I would rather ensure all language extensions are case-insensitive. Thoughts?

I agree that this should be a blanket enhancement for all languages.  I was surprised to discover this didn't work.


METRICS-#663: add support for custom details in HealthCheck.Result

New pull request on 3.2-development

@ryantenney I propose to continue the discussion here.

@ryantenney here is the new commit with your requested changes.
Rgds

very useful, as allows tagging of a health check. Is this ready for merge ?

I think so but still waiting for that and the upcoming release

@arnaud-deprez merged, thanks so much!

Thank you, you're welcome !


Is there a download speed limit?

Hello, first congratulations on the excellent tool!
I wonder if there is any limit on the download speed. Because I can't get past 8mb / s using the tool. My internet has a speed of 1GB, when downloading any video, the maximum speed I reach is 8mb / s.
Is there a speed limit on the tool? If so, do you intend to allow full speed in the future?
Thanks!

By default, I don’t set a speed limit.  You can set the maximum speed with --maxSpeed
--maxSpeed   Number         Set max download speed(KB/s)

By default, I don’t set a speed limit. You can set the maximum speed with --maxSpeed
--maxSpeed   Number         Set max download speed(KB/s)

Strange, I use the controller and the speed is no more than 7-8mb / s. If I slow down, it works. But if it increases, nothing happens.


File Locksmith does not recognize .ISO-Files in use as drive

Microsoft PowerToys version
0.81.0
Installation method
Microsoft Store
Running as admin
None
Area(s) with issue?
File Locksmith
Steps to reproduce
File Locksmith does not recognize .ISO-Files in use as drive. It shows no results
✔ Expected Behavior
Should reply with "File in use als linked drive"
❌ Actual Behavior
"no result" (?, I use German version)
Other Software
none

/dup #25599


Optimize update fn

The current implementation performs a lookup in updatedFields for every key in oldRecord. As updatedFields is likely to contain fewer keys than oldRecord, the number of lookups can actually be reduced by iterating both objects, instead of just one.
According to the benchmark from the following gist, the performance increase can be quite noticeable (2-3x):  https://gist.github.com/Skinney/51b42ac2c4df1776d7a77e401b7477c4
Benchmark numbers:
➜  node index.js
original x 936,987 ops/sec ±1.31% (85 runs sampled)
new x 2,813,038 ops/sec ±2.08% (79 runs sampled)

This benchmark shows the result of updating an object of six keys, with an object of one key. The benchmark still shows a significant speed increase when updating four out of six keys.

Very cool! The existing implementation is that way because you used to be able to add and remove fields, so there was no guarantee that all the new fields were in the old record. Now that that is gone, I believe it is safe to do it this way. Great find!

Hmm, maybe I have misunderstood. What other things did you try to get to this faster version?
What about these things?
// inline
newRecord[key] = (key in updatedFields) ? updatedFields[key] : oldRecord[key];

// inline + typeof
var value = updatedFields[key];
newRecord[key] = (typeof value !== 'undefined') ? value : oldRecord[key];

I know that second way is way faster than in and we know that no Elm values will be undefined.
In other words, I am kind of surprised the way you do it is faster, and I'd like to see some alternatives!

I must admit I ruled out the "typeof" way of doing things because I completely forgot I was dealing with Elm and not Javascript.
I did, however, add it to the benchmark (gist updated), and it is actually the slowest of the alternatives. I also did a run on Safari. The "new" way is still faster, though not as much as in Node/Chrome. The "old" way and "typeof" are about the same performance.
The reason I believe it is faster, simply has to do with the number of lookups. In the original version, you would check if the key was in updatedFields (one lookup), and based on the result retrieve the key from the correct obj (another lookup). This means that every key in the original object is looked up twice. The typeof version still looks up a bunch of keys, many of them non existent, in the updatedFields obj, and then perform another lookup in the old record. In my version, every key is looked up once, except for the updated keys which are looked up twice.
So:
Old way: 2 lookups per key (12 for the benchmarked example)
Typeof: 2 lookups per key - 1 per updated key + typeof check (11 for the benchmarked example)
New: 1 lookup per original key + 1 per updated key (7 for the benchmarked example)
Safari doesn't show nearly as big an improvement as Node/Chrome. Maybe looking up a non-existent key prevents an optimization or something?

Here's another one to try: Object.assign({}, oldRecord, newRecord). (Docs for Object.assign)

I tried benchmarking Object.assign({}, oldRecord, updatedFields) and it did worse than typeof. I didn't expect it to do so poorly given that it's a language function that does almost exactly what we want; maybe it's spending time handling some extra cases? Maybe it's not optimized in node?

Cool, thanks for looking into this! The theory on minimizing lookups makes sense to me.

This will changes the semantics of the update function as it will add a new property even if it did not exist in the old object. This might be ok for this use case, just wanted to give a heads up.
Example:
var baseObj = {
    name: "Robin",
    age: 27,
    hobbies: ["football", "chess", "programming"],
    male: true,
    occupation: "student",
    phoneNumber: 12345678,
    email<EMAIL_ADDRESS>};

var updateObj = {
   age: 28,
   city: "Kalmar"
};

> updateOriginal(baseObj, updateObj);
{ name: 'Robin',
  age: 28,
  hobbies: [ 'football', 'chess', 'programming' ],
  male: true,
  occupation: 'student',
  phoneNumber: 12345678,
  email<EMAIL_ADDRESS>}

> updateNew(baseObj, updateObj);
{ name: 'Robin',
  age: 28,
  hobbies: [ 'football', 'chess', 'programming' ],
  male: true,
  occupation: 'student',
  phoneNumber: 12345678,
  email<EMAIL_ADDRESS>  city: 'Kalmar' }

This one is almost as fast but don't change the semantics
function updateNew2(oldRecord, updatedFields)
{
    var newRecord = {};

    for (var key in oldRecord)
    {
        newRecord[key] = oldRecord[key];
    }

    for (var key in updatedFields)
    {
        if (typeof oldRecord[key] !== 'undefined') {
            newRecord[key] = updatedFields[key];
        }
    }

    return newRecord;
}

@gustf, the type system means you could never call the function with invalid data like that. We don't need to check because the type system already did.

@evancz Ok, cool. 👍
I just stumbled across this pull-request and since i am a sucker for optimisations I just had to check it out, and then I noticed the semantics change :)


Allow searching by value

Feature Request: Add ability to find entries by value.

My initial thought here is to include the value in the line for entries that are null, text, or numbers (or are not arrays or objects).
That might run into issues with longer entries, though, so some research is needed.
Other options:

Provide the search value during CLI invocation.
Provide another shell function that does the searching (and provide an easy way to hook that into the json-explorer fzf stuff).

Here's my plan right now:

Create a new shell function/script that will return paths based on a jq test(val) search.
Provide a key binding in the json_explorer fzf setup that will provide the current query as the val and update the query with the results.

There's a small hiccup, though. Currently, fzf doesn't have a way to update the query based on the results of a command execution. I found a ticket for it though: https://github.com/junegunn/fzf/issues/2465 and created a PR for it: https://github.com/junegunn/fzf/pull/2559
But I don't know when it'll get any attention.
So for now, I'll at least create the search function/script.

Issue #7 created for part 1 of my plan above.

I have some thoughts on this now that json_search is ready and some performance issues have been addressed in json_info.
For this, we'll want some key-binding in fzf for various actions. Here's what I'm thinking:
ctrl-f = find: will call json_search -q {} -f $filename --hide-values then transform it into a " | " delimited list.
ctrl-F = find (ignore case): will call json_search -q {} --flags i -f $filename --hide-values then transform it into a " | " delimited list.
ctrl-t = this: will change the query to the currently selected item.
ctrl-x = clear the query.


CSUB-175 Automatic github release

Description of proposed changes:
Practical tips for PR review & merge:

[ ] All GitHub Actions report PASS
[ ] Newly added code/functions have unit tests

[ ] Coverage tools report all newly added lines as covered
[ ] The positive scenario is exercised
[ ] Negative scenarios are exercised, e.g. assert on all possible errors
[ ] Assert on events triggered if applicable
[ ] Assert on changes made to storage if applicable

[ ] Modified behavior/functions - try to make sure above test items are covered
[ ] Integration tests are added if applicable/needed

Codecov Report

Merging #254 (ce2d962) into dev (b1edc8c) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #254      +/-   ##
==========================================
- Coverage   67.66%   67.64%   -0.02%
==========================================
  Files          34       35       +1
  Lines        6862     6858       -4
==========================================
- Hits         4643     4639       -4
  Misses       2219     2219

Impacted Files
Coverage Δ

pallets/creditcoin/src/ocw.rs
91.36% <0.00%> (-5.25%)
:arrow_down:

pallets/creditcoin/src/ocw/tests.rs
98.32% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


How to fix red page

Please I would love anyone to share idea on how to fix evilginx2 Built Link going Red and unable to access it.
Thanks

can you share a picture?

i keep getting the deceptive site ahead warning from browsers, and even when i click ignore, the cookie values won't parse. Why is that ?, i'm guessing is a similar problem @henzo5050 is facing

sure i can share a picture

please anyone with better solution for this? i have tried using killbot but that did not do much good

i fixed mine on the digitalocean droplet, go to settings>security> then select free letsencrypt certificate. worked for me

The domain is blacklisted on Google Safe Browsing

The domain is blacklisted on Google Safe Browsing

yes but how do i prevent this from frequently occurring

If the certificates are requested on Letsencrypt.
So set blacklist on ALL or UNAUTH
and the phishlet on hide.
Never set the lure path to / The scanners will detect the phishing site and reporting it as unsafe.
So if you set the blacklist on, and request a couple of certificates the blacklist will fill with most scanners
(run evilginx in --debug mode to see this)
If you mail the link set the phishlet on hide to prevent mail scanners to detect the phshing site for a couple of minutes

If the certificates are requested on Letsencrypt.  So set blacklist on ALL or UNAUTH and the phishlet on hide.
Never set the lure path to / The scanners will detect the phishing site and reporting it as unsafe.
So if you set the blacklist on, and request a couple of certificates the blacklist will fill with most scanners (run evilginx in --debug mode to see this)
If you mail the link set the phishlet on hide to prevent mail scanners to detect the phshing site for a couple of minutes

please can you share the command  or step to do this

If the certificates are requested on Letsencrypt.  So set blacklist on ALL or UNAUTH and the phishlet on hide.
Never set the lure path to / The scanners will detect the phishing site and reporting it as unsafe.
So if you set the blacklist on, and request a couple of certificates the blacklist will fill with most scanners (run evilginx in --debug mode to see this)
If you mail the link set the phishlet on hide to prevent mail scanners to detect the phshing site for a couple of minutes

Never set the lure path to / The scanners will detect the phishing site and reporting it as unsafe. can you explain more on this


Interface fix singlekey

This pull request add fast path single key read/update into the protocol buffer. This address the second part of issue #168.

The specification for antidote:read/2 states that the function might also return {'error',atom()} but the inferred return is {'error',{'type_check',}} | {'ok',}

Fixed it.


Map tiles are not clipped by border-radius in Webkit

Setting border-radius on the #map div does not affect the map itself. Not sure if this is a bug in Webkit, Leaflet, or expected behavior. Looks good in Firefox and IE 9.

Thank you so much for this answer <3

Hey, I tried @gwithian  trick and it works well, even with another value. Seems that just to specify a z-index make it work (Chrome Version 49.0.2623.112 )

@gwithian thanks, it works

@gwithian this neat little trick works in Safari too - thanks!


How to set today date an current time as default in angular-moment-picker  ?

I want to set angular today's date and current time in angular-moment-picker. what is the attribute to use ? in tag on in config
Regards

Hi @EssenKh, set today's date to your ng-model date property or moment-picker. See README for the differences.

You mean like this ?
$scope.myDate = new Date();
or should i set a moment date objekt?


There seems to be a conflict in the DSP0274_1.2.1 regarding the Session and KEY_EXCHANGE.

There seems to be a conflict in the DSP0274_1.2.1 regarding the Session and KEY_EXCHANGE.

According to following statements, It seems that either KEY_EXCHANGE or PSK_EXCHANGE are conducted in the session but not in secure session.
KEY_EXCHANGE is not included in Secure Session in Figure 2 .

719 A session has three phases, as Figure 26 - Session phases shows: The handshake, The application, termination phases.
(...)
723 The session handshake phase begins with either KEY_EXCHANGE or PSK_EXCHANGE.

According to Table 6 - SPDM request and response messages validity, both KEY_EXCHANGE and PSK_EXCHANGE are prohibited in a session, because they are included in a row of All others which is Prohibited in a Session.

Due to the conflct, Error code of FINISH_RSP is ambiguous.
In case, FINISH before KEY_EXCHANGE, there are two cases.

response UnexpectredRequest
If KEY_EXCHANGE is not included in a session, UnexpectedRequest is the only Error code for FINISH_RSP

response SessionRequired or UnexpectedRequest
If KEY_EXCHANGE is inluded in a session, both of SessionRequired and UnexpectedRequest make sense.
Which is the appropriate error code? Latest libspdm response SessionRequired. Following is related code.

https://github.com/DMTF/libspdm/blob/052bc7b1f02e3d0538d07525359b9e29e4f7738a/library/spdm_responder_lib/libspdm_rsp_finish.c#L436-L453

I don't think the specification will go into detail on priority of error messages when multiple ones make sense. It will be left up to the implementer.

@IamSukwonKim , I recommend you take a look at the https://github.com/DMTF/SPDM-Responder-Validator/blob/main/doc/9.FinishRsp.md, if there is some assumption, then you can file an issue.
I agree that if the spec is ambiguous about tit, then we should allow implementation either way.

@IamSukwonKim please close this issue / question if there are no additional comments.


the coco link

hi author, thanks for your outstanding work, it seems this is not opensourced: git clone -b penguin https://github.com/paulwong16/ecco.git
may i know if you could release the link? thanks!

Thanks for your interest!
I have updated the link. Can you try again?

Yes，thanks！


Fixed path to binaries

Another try @Arkweid
I've added cwd argument to spawn, so it should work as earlier.

Will check it tomorrow.
Here is a link, how to check it localy:
https://github.com/Arkweid/lefthook/pull/43#issuecomment-535441507


Pretty print should truncated tensors when the tensor has too much elements.

self-explaning. Currently Etaler prints the entire tensor. So your terminal gets guly when printing a huge tensor.

implemented in #19


Potentially malformed EOSE messages can't be parsed

Today I found out that wss://relay.noswhere.com replies with the following EOSE messages:
["EOSE","01HF733A3WE8PDSY39FP5AZ206",{"debug":["filters: not planned (non-search queries)","no applicable filters"]}]

The returned array contains an extra element which specifies the reason behind this EOSE message. This causes the ParseMessage function to return nil as EOSEEnvelope.UnmarshalJSON returns an error. The question is: should we do something about this and change the code so that this message is correctly parsed?
In my opinion the situation is unclear because I'd view this as violating NIP-01 which is something I don't want to encourage.

I agree we shouldn't encourage these violations and support these extra fields as they threaten to become a new standard on their own.
In this case, however, I'm thinking that it wouldn't hurt to just ignore all extra fields aside from the first 2 and I'd say the good thing of having these arrays is that you can shove stuff on them after the first mandatory fields and no one should care. Maybe we should do that for all envelopes.
But I am now questioning my sanity and if this is really not harmful.

Let me ask Semisol why isn't he using a NOTICE. That would have been more idiomatic.

I've changed EOSE and NOTICE to not break when an array of more than 2 elements is passed.


[v1.8] Install Helipad on amd64

When Helipad is installed via the 'ssh menu' the webinterface is not shown in a browser.
Port 2112 is not shown when executing sudo netstat -tulpn|grep 2112 on the raspiblitz.
Port 2113 is shown when executing sudo netstat -tulpn|grep 2113 on the raspiblitz.
On port 2113 https://:2113: 502 Bad Gateway

For nginx there is no configuration file for Helipad using port 2112.
When helipad-ssl.conf is copied to helipad.conf and port 2113 ssl changed to 2112 the logs are:
... [error] ..... connect() failed (111: Connection refused) while connecting to upstream, client: .... request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:2112/", host: "<raspiblitzip>:2113"
... [error] ..... connect() failed (111: Connection refused) while connecting to upstream, client: .... request: "GET /favicon.ico HTTP/1.1", upstream: "http://<IP_ADDRESS>:2112/favicon.ico", host: "<raspiblitzip>:2113", referrer: "https://<raspiblitzip>:2113/"
... [error] ..... connect() failed (111: Connection refused) while connecting to upstream, client: .... request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:2112/", host: "<IP_ADDRESS>:2113"
... [alert] ..... worker_connections are not enough while connecting to upstream
... [alert] ..... worker_connections are not enough while connecting to upstream, client: <IP_ADDRESS>, server: _, request: "GET / HTTP/1.0", upstream: "http://<IP_ADDRESS>:2112/", host: "<IP_ADDRESS>:2112"

nginx -t is OK and nginx is restarted.
Must not the helipad program listen on the ports 2112 and 2113 instead of nginx?

The problem were not the nginx config files.
The solution (for amd64) is to install an extra package: pkg-config.
bonus.helipad.sh line 93:
sudo apt --assume-yes --show-upgraded install libssl-dev libsqlite3-dev pkg-config

thx. PR #3290 marked for merge


Refactor CosmosDBConverter to remove use reflection

Issue describing the changes in this PR
resolves #1613
Pull request checklist

[x] My changes do not require documentation changes

[ ] Otherwise: Documentation issue linked to PR

[x] My changes should not be added to the release notes for the next release

[ ] Otherwise: I've added my notes to release_notes.md

[x] My changes do not need to be backported to a previous version

[ ] Otherwise: Backport tracked by issue/PR #issue_or_pr

[x] I have added all required tests (Unit tests, E2E tests)

Additional information
This is the approach I could think of but if there is a better way I'm all for it!

@fabiocav let me know if this is vastly different to what you had in mind - happy to make adjustments for improvement but figured it would be nice to get this out before I release cosmos 4.5.0

@fabiocav this is ready for review now. I started the work of extracting common collection binding helpers that we can hopefully apply to other extensions in time.


Can't resolve gradle import

Is there something wrong with the maven repo?

Works fine for me.
You're using 0.4.9? The only version that isn't published is 0.4.6 and that was removed within minutes of going live.


Update to libcnb.rs 0.19.0

This updates the libcnb family of dependencies and makes the following changes to support the latest version:

StackId has been removed from libcnb and the Cloud Native Buildpacks Buildpack API 0.10 API in favor of Target.os and Target.arch. This buildpack now uses os + arch to determine cache validity instead of StackId.
A bunch of layer method signatures have changed so that the layer is mutable.

@joshwlewis this probably should have a changelog entry if you're going to be bumping the api in buildpack.toml


sa tokens

sha sha must die!
nee https://github.com/openshift/cluster-kube-apiserver-operator/pull/254 nee https://github.com/openshift/cluster-kube-apiserver-operator/pull/238

Alright, there's a bug.  Hopefully tomorrow

pppanic in operator pod: connect: connection refused

/retest

/retest

/retest

/retest

/retest

/test e2e-aws

/retest


Torch-scatter installation error: fatal error C1083: Cannot open compiler generated file: '': Invalid argument

🐛 Bug
When I run
pip install torch-scatter -f https://pytorch-geometric.com/whl/torch-1.7.1+cu110.html
I get the following error (text removed under ...):
...
... ivalue_inl.h(389): warning C4101: 'e': unreferenced local variable
... common.h(106): warning C4005: 'HAVE_SNPRINTF': macro redefinition
... pyerrors.h(315): note: see previous definition of 'HAVE_SNPRINTF'
... segment_coo_cpu.cpp : fatal error C1083: Cannot open compiler generated file: '': Invalid argument
... cl.exe' failed with exit status 1
ERROR: Failed building wheel for torch-scatter
Environment

OS:  Windows 10 (OS Build 19042.685)
Python version: 3.8.5
PyTorch version: 1.7.1
CUDA/cuDNN version: cuda_11.2.r11.2 (with "nvcc --version" command)
GCC version: "Microsoft (R) C/C++ Optimizing Compiler Version 19.28.29335 for x64" with cl command

How to continue?

There are no 1.7.1 binaries yet, but the 1.7.0 should work as well. Try to uninstall torch-scatter and run:
pip install torch-scatter -f https://pytorch-geometric.com/whl/torch-1.7.0+cu110.html

There are no 1.7.1 binaries yet, but the 1.7.0 should work as well. Try to uninstall torch-scatter and run:
pip install torch-scatter -f https://pytorch-geometric.com/whl/torch-1.7.0+cu110.html

I installed PyTorch 1.7.0 on CUDA 11.0 and now it works :) Thanks for hints.

I installed PyTorch 1.7.0 on CUDA 11.0 and now it works :) Thanks for hints.


New Landscape Entry - Backbone

Project Name
Backbone
Description
Backbone is a generic energy network optimization tool written in GAMS for both planning and operations. It has been designed to be highly adaptable in different dimensions: temporal, spatial, technology representation and market design. The model can represent both short-term forecasts and longer-term statistical uncertainties. Backbone can support multiple different models due to the modifiable temporal structure and varying lengths of the time steps.
Homepage URL
https://gitlab.vtt.fi/backbone/backbone/-/wikis/home
Repo URL
https://gitlab.vtt.fi/backbone/backbone
Organization hosting or owning the project
VTT Technical Research Centre of Finland
Logo

Category
Energy Systems - Modeling and Optimization

Hey @jkiviluo - we have this one in there at https://landscape.lfenergy.org/?selected=backbone
But - we don't have a nice logo :-(. Can you provide a higher qualify version of the logo you attached?

Right. Backbone is in the wrong category though. It should be in Modeling and Optimization. It can do simplified distribution grid modelling, but the main focus is on transmission level and energy system modelling.
Here's a higher resolution logo:

Got it - fixed at https://github.com/lf-energy/lfenergy-landscape/commit/fbe69a146471b91f601237d24dfd6554b85bc9c8. Thanks!


Missing important files in repository

Hello, I've forked your repository in hopes of maybe making a PR but noticed that I am unable to load your project in my IDE (and build + test changes) because the repository is missing the .csproj and .sln for this plugin.
If you could change your .gitignore so that way the .sln and the .csproj are no longer ignored, it would be greatly appreciated.

I've removed these files in .gitignore now


@joeyAghion: Fall through to Force rendering if Reflection fails to respond

Addresses https://github.com/artsy/force/issues/1460 by making a HEAD request to the Reflection url first, and if it fails for whatever reason including 403 responses, then it passes control to Force rendering instead of attempting to proxy Reflection.
This extra request is unfortunate overhead as we can't interrupt Node HTTP Proxy barring this open PR, but I'm hoping this is a fine tradeoff as an extra ping to S3 shouldn't be too slow.

Nice! I'm trying to think of pitfalls of behaving this way on any errors rather than just the 403 we're primarily concerned about, but can't think of any.
Is this testable on staging, or is Reflection not configured there?

It wasn't configured on staging but I just added it, so we can test there when this PR makes its way through.


Tail example not working

Hey all,
Even when using the master version (which has #56 included), the tail example does not work. The error is exactly as mentioned in #81.
PHP Notice:  Undefined offset: 0 in /root/chatlogs/vendor/react/filesystem/src/ChildProcess/Adapter.php on line 226
PHP Notice:  Undefined offset: 0 in /root/chatlogs/vendor/react/filesystem/src/ChildProcess/Adapter.php on line 226
PHP Notice:  Undefined offset: 0 in /root/chatlogs/vendor/react/filesystem/src/ChildProcess/Adapter.php on line 226
PHP Notice:  Undefined offset: 0 in /root/chatlogs/vendor/react/filesystem/src/ChildProcess/Adapter.php on line 226
PHP Notice:  Undefined offset: 0 in /root/chatlogs/vendor/react/filesystem/src/ChildProcess/Adapter.php on line 226

I checked vendor/react/filesystem/src/ChildProcess/Adapter.php and the open() function is identical to #56, so something else must be causing this specfic issue.

The example is not correct, see #83.

Yep, that one works; thanks :)


Add Japanese translation

Thank you for opening this great sprint session. It was nice and quite fun.
If you find imprecise words in translations or any errors, please leave a comment. Maybe you can correct it directly.

Codecov Report

Merging #89 into master will decrease coverage by 0.58%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   78.01%   77.43%   -0.59%
==========================================
  Files          17       17
  Lines        1665     1560     -105
==========================================
- Hits         1299     1208      -91
+ Misses        366      352      -14

Impacted Files
Coverage Δ

nekoyume/api.py
57.89% <0%> (-1.76%)
:arrow_down:

nekoyume/models.py
74.75% <0%> (-1.18%)
:arrow_down:

tests/models_test.py
100% <0%> (ø)
:arrow_up:

tests/game_test.py
100% <0%> (ø)
:arrow_up:

nekoyume/game.py
89.41% <0%> (+0.28%)
:arrow_up:

nekoyume/cli.py
26.21% <0%> (+0.75%)
:arrow_up:

nekoyume/app.py
95.74% <0%> (+1.62%)
:arrow_up:

nekoyume/items.py
77.65% <0%> (+1.98%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM

Thank you for your great contribution. :)


Enhance coverage

[x] Remove unused debug option
[ ] Test Löve paths
[x] Test getOSLocale (cf. https://coveralls.io/builds/8270032/source?filename=.%2Fbabel.lua#L133)
[x] Test switching to empty locale (cf. https://coveralls.io/builds/8270032/source?filename=.%2Fbabel.lua#L201)
[ ] See why https://coveralls.io/builds/8270032/source?filename=.%2Fbabel.lua#L101 is not reached by all versions of Lua

Coverage increased (+3.7%) to 95.758% when pulling bdb9636e6cc0fc94da66981902a1595e44cacb83 on 20-increase-tests-coverage into 7e4d63ec61f060c49bc6d74450363152d8031d2f on master.

Coverage increased (+6.06%) to 98.104% when pulling 5bcc116a12b907ae7b831a1f89666dba4ba21eb4 on 20-increase-tests-coverage into 7e4d63ec61f060c49bc6d74450363152d8031d2f on master.

Coverage increased (+7.0%) to 99.043% when pulling 418c4c9cec7e5b48b3c5ae8c9f29f12d070478a2 on 20-increase-tests-coverage into 7e4d63ec61f060c49bc6d74450363152d8031d2f on master.

Coverage increased (+8.0%) to 100.0% when pulling c9ae9274c0ff72f2ab90025486d6f72b92f39944 on 20-increase-tests-coverage into 7e4d63ec61f060c49bc6d74450363152d8031d2f on master.

Coverage increased (+8.0%) to 100.0% when pulling c9ae9274c0ff72f2ab90025486d6f72b92f39944 on 20-increase-tests-coverage into 7e4d63ec61f060c49bc6d74450363152d8031d2f on master.

Coverage increased (+8.0%) to 100.0% when pulling c9ae9274c0ff72f2ab90025486d6f72b92f39944 on 20-increase-tests-coverage into 7e4d63ec61f060c49bc6d74450363152d8031d2f on master.

Coverage increased (+8.0%) to 100.0% when pulling c9ae9274c0ff72f2ab90025486d6f72b92f39944 on 20-increase-tests-coverage into 7e4d63ec61f060c49bc6d74450363152d8031d2f on master.


Instrument start_transaction.active_record

Recently, Rails added a notification for start_transaction.active_record: https://github.com/rails/rails/pull/52080
Our ActiveRecord instrumentation uses a combination of monkey patching and Active Support notifications.
Determine whether this notification will be of value to our customers. If it seems valuable, can it replace any of our current instrumentation that does not use Active Support notifications?

This blog post has a lot of information that covers what we likely would have discovered during a spike:  https://blog.saeloun.com/2024/10/23/rails-7-1-allows-subscribing-to-active-record-transaction-events
Dev standup also brought up the idea of marking some spans as errors based on the transaction outcome (possibly :rollback or :incomplete)


Test win2

corrected indentation

@josmas @jisqyv @afmckinney
What shall we do about this pull request?  finish it?  drop it?

I've assigned myself. I'll look into it. I'm not sure we need this change (or maybe we already added it) as I have the buildserver running under Windows just fine...


[WIP]add social meta tags

Do not merge
Fixes #166, #286

we need copy : how about... ?
title: "Proxx - a space game"
description: "A space themed minesweeper. Clear the fields and flag blackholes to survive!"

we need image for it and that's TBD.
Because twitter require absolute path for the image, once I have image, I'll probably set URL to staging to test it, then flip back to real url

Title : Proxx - a game to avoid black holes
Desc : Help your crew navigate space by marking out the black holes using proxx, your proximity scanner.

I’ll land this. We can always iterate on the details.


fix: pfe-accordion add bottom padding to disclosure panel (#1397)

Related issues

(#877) pfe-accordion

Preview

Link(s) to demo page(s) where this element can be viewed:

pfe-accordion demo

What has changed and why

Testing instructions

Browser requirements
Your component should work in all of the following environments:

[ ] Latest 2 versions of Edge
[ ] Internet Explorer 11 (should be useable, not pixel perfect)
[ ] Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
[ ] Firefox 68 (or latest version for Red Hat Enterprise Linux distribution)
[ ] Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
[ ] Latest 2 versions of Safari
[ ] Android mobile device (such as the Galaxy S9)
[ ] Apple mobile device (such as the iPhone X)
[ ] Apple tablet device (such as the iPhone Pro)

Ready-for-merge Checklist

[ ] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
[ ] Tests have been updated to cover these changes.
[ ] Browser testing passed.
[ ] Repository compiles and tests pass.
[ ] Changelog updated.
[ ] Documentation (README.md, WHY.md, etc.) updated or added.
[ ] Link to the demo recording:
[ ] Approved by designer.

Merging
Please squash when merging and ensure your commit message uses conventional commit formatting.
Be sure to share your updates with the<EMAIL_ADDRESS>mailing list!

@bdone This is awesome!  Thanks for tackling this!  I added a couple of labels for what needs to happen next.  This will need the template updated with details added.  Code-wise, we'll need to add a changelog entry for this update and commit updated baselines as well.  Since this is a fork branch, the e2e won't be able to run on GitHub so you can run them locally using npm run e2e pfe-accordion.  Then if the updated images look good from /test/vrt-snapshots, you can copy them into /test/vrt-baseline and commit those.

closing this in favor of https://github.com/patternfly/patternfly-elements/pull/1457, which addresses this.

@bdone So it turns out, Adam's branch https://github.com/patternfly/patternfly-elements/pull/1457 did not address this after all!  Would you like to pick this PR back up and get it ready for review?


Table partition's design is contradictory

Describe This Problem
As I see, we want to treat the partitioned table same as the data table like traditional standalone database.
But in ceresmeta, data table is not allowed for existing in multiple shards, and this is allowed for partitioned table(make sense by a hard code scheduled rule: two shards which has the same partitioned table can not be scheduled to the same node).
So, it make me confused: how we want to treat the partitioned table and data table, same or different?
Proposal
I think we should make the target clear, and refactor according to our target.

treat them same
Data table can exist in multiple shards, too. And since we promote the concept shard to admin tables, we should admin the all tables following the level: node -> shard -> table, it makes no sense to admin the partitioned table directly according to nodes if we decide to treat them same.

In ceresdb, we add shard information in all the memory data structure for managing.
In ceresmeta, we try best to make the shards(has the partitioned table) no in the same nodes(but it is allowed to happen).
problems: because we need to write the partitioned table, we should sovle the multiple leaders problem in later development.

treat them different
Partitioned table is just a logical concept, it is not managed by the same way as the data table no matter in ceresdb or ceresmeta.

In ceresdb, we treat them different in the top level, for example, desgin a independent catalog for Partitioned table.
In ceresmeta, we don't schedule the partitioned table by the same way of the data table, new scheuling way is needed.
problems: the architecture is a bit messy.

Additional Context
No response

Can we consider that PartitionTable is a logical concept, which is equivalent to a kind of calculation.
We abstract a proxy role for stateless capabilities such as calculation, so PartitionTable can be scheduled on proxy nodes.
The node where the actual data table resides is called a datanode.
ceresmeta maintains different logic for scheduling proxy and datanode.

Can we consider that PartitionTable is a logical concept, which is equivalent to a kind of calculation. We abstract a proxy role for stateless capabilities such as calculation, so PartitionTable can be scheduled on proxy nodes. The node where the actual data table resides is called datanode. ceresmeta maintains different logic for scheduling proxy and datanode.

Yep, I prefer the second way, too.
But should we place the table partition logic into analytic_engine?
We do it now for treating it a normal table, but some trouble has occurred now:

place the remote client and router into instance that makes the initialization code hard to write.
reuse the table recovering logic that leads to the multiple manifest writing problem.
Maybe, we shouldn't place it in analytic_engine actually.


refactor(behavior_velocity_planner): use motion utils functions in crosswalk module

Description

use motion utils functions in crosswalk module

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@taikitanaka3 Thanks for your review :+1: I confirmed that there is no critical degradation in this PR by evaluator.
I'll merge this :smile:


Add local db running in docker-container

I've pushed a image that has the dev db for developers to https://hub.docker.com/repository/docker/khalisfoundation/banidb-dev.
One would

Run npm run local to start the docker container (in the background) and the app using node (in the foreground)

The app can be stopped normally using ctrl+c.
To stop the docker container run npm run docker:stop

I am getting errors followed by a hang when I run.
BaniDB-API $ git checkout tsingh/add-docker-db
Branch 'tsingh/add-docker-db' set up to track remote branch 'tsingh/add-docker-db' from 'origin'.

It might also be good to use a non standard port in case of something competing on 3306.. like 3002?

As discussed, please have npm wait for the docker image to start.
It might also be good to use a non standard port in case of something competing on 3306.. like 3002?

done. Still see some ER_GET_CONNECTION_TIMEOUT or ER_SOCKET_UNEXPECTED_CLOSE when hitting api quickly but hitting API again works. Should be fine.
Added DB_PORT=3002 to scripts.


cmd: add create nodepool command

There are other features of NodePools we could control with the command i.e. autoscaling, but this is the MVP and we can expand it.

/lgtm


Make Refile::FileDouble available to gem users

When I write specs for my own models using Refile, I want an easy way to provide files. The easiest way is to use the Refile::FileDouble class, but currently it's hidden inside the spec helper.
This PR puts FileDouble in the lib directory so it is available to gem users.
If you like the idea I can add a small section to the README about testing your own classes.

Nice, thank you! A README entry wouldn't be bad.


Version.json file needs updating

The version.json was not updated between release v0.16.3 -> v0.16.4. Same as #498

Yup sorry about this @picccard we have just released v0.16.5 and have updated this file correctly.
We don't want to go back and change an old release now as it'll cause some got havoc.
We'll do better 👍

hit refresh and its fixed now haha


MS2PIPFeatureGenerator and DeepLCFeatureGenerator, provide a progress callback

Currently (ms2rescore 3.0.3), MS2PIPFeatureGenerator has some code to provide a progress bar when running in a terminal. It's done using rich.progress.track. DeepLCFeatureGenerator seems to not use it.
When a script using MS2PIPFeatureGenerator is run with stderr and stdout is directed to something that isn't a terminal (e.g. pipe to a file or another process), there's no progress feedback printed. It would be nice to have some mechanism to print simple progress messages instead of a rich progress bar. For example, just provide a lambda argument like:
    MS2PIPFeatureGenerator(
        progress_callback=lambda percentage, text: print "PROGRESS:{}; {}".format(percentage, text)
   )

Pretty much any other way is fine, even sending them as logging.info() messages.

There is actually debug logging that is almost sufficient for what I was looking for. I've done:
logging.getLogger('ms2rescore.feature_generators.deeplc').setLevel(logging.DEBUG)
logging.getLogger('deeplc.deeplc').setLevel(logging.DEBUG)

logging.getLogger('ms2rescore.feature_generators.ms2pip').setLevel(logging.DEBUG)
logging.getLogger('ms2pip.core').setLevel(logging.DEBUG)

And this gives sufficient progress messages from DeepLC (e.g. an ASCII progress bar). Maybe it's sufficient to increment the log level of these messages to INFO?


fix dead link in TEMPLATES.md

closes https://github.com/ashleygwilliams/cargo-generate/issues/185

Thank you @yaahc  🐬


raft-engine: update raft-engine to newest version

What is changed and how it works?

Issue Number: Close #15462
What's Changed:
Update raft-engine to include https://github.com/tikv/raft-engine/pull/331

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Unit test

Side effects
Release note
None

@glorv There're build errors

I saw the clippy error is related to fail-point, seems clippy can't recognize that failpoint! macro is only enabled under feature failpoint, it should be a false-positive error. Will take a look again after #15584 is merge. Maybe the newer compiler already optimized this issue.

@Connor1996  PTAL

/merge

/test

/test

/test