Request: MissingMoviesSearch

Hi Vivien,
Can you please implement the following method?
Merci!!
def MissingMoviesSearch(self) -> json_data:
"""Perform an Missing missing movie search.
Returns:
json response
"""
return self._sendCommand({"name": "MissingMoviesSearch"})

I didn't add it on purpose because the wanted API just doesn't exist in radarr. As you can see in the UI, radarr has a "wanted" section, but not radarr.
You can only get the movie list and filter out by status those missing (not downloaded).
so basiccally you can do something like
missing = [ missing_movie for missing_movie in radarr_cli.get() if missing_movie.downloaded is False]

wait, is there an existing command "MissingMoviesSearch" ?

Yes
On 4 Dec 2021, 13:09 +0100, Vivien Chene @.***>, wrote:

wait, is there an existing command "MissingMoviesSearch" ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

nice. i didn't see it in radarr api. I will implement it, thanks !

Fixed in v1.0.19


Permission to view emails (Old 258)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:08
Add a permission to display user email address.

In gitlab by @A-Lawrence on Aug 31, 2015, 01:13
Implemented all mship/account permissions!  Resolves issue #488484 (259).
Removed emails from view for those that don't have permission.  Resolves issue #258.
Account permissions are fully implemented now! Resolves issue #488490 (1)
ÔåÆ <<cset 53f321068c43>>
ALawrence66 - 2015-02-21T21:57:56.757130+00:00


feat(IAM Identity): add support for inactivity reports

PR summary
adding new endpoints: create report, get report
adding new optional query parameter include_activity for the existing get calls of apikey/serviceid/profile by id.
PR 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)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior
users will now be able to generate report for inactive identities.
create report: will return a reference
get report: this will return a detailed report of inactive identities
existing get apikey/serviceid/profile by id will now include additional optional parameter include_activity
if include_activity is passed as true the response will include a new section activity with last authn time and count.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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

@michaelbeck Could you please post a screenshot of the results when running the integration tests and examples?

You see the screenshot of the integration tests under "Other Information" Will add one for the examples...

:tada: This PR is included in version 0.24.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[BUG] VM of existing RKE1 node is not removed after the node is deleted from the cluster after upgrading to Rancher 2.9.3+

Rancher Server Setup

Rancher version: 2.9.3 / 2.9.4
Installation option (Docker install/Helm Chart):  any

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: any
Cluster Type (Local/Downstream): Downstream, node-driver

User Information

What is the role of the user logged in? Admin

Describe the bug

After upgrading from 2.9.2 to 2.9.4, when deleting a downstream RKE1 cluster node (e.g., scaling down a node pool) that was created prior to the upgrade, the node is successfully removed from Rancher and the downstream cluster. However, the VM is not physically deleted from the infrastructure provider.
To Reproduce

create a new single node ( or docker-install) Rancher 2.9.2
add cloud credentials (Azure in my case, but vSphere for customer)
create a suitable node template for rke1
deploy a rke1 downstream cluster (e.g. single cp, and 3 workers for convenience)
upgrade Rancher to 2.9.3 (or 2.9.4 / 2.10.0)
select rke1 cluster from cluster management
select a node from a node pool and scale it down
the node should disappear from both Rancher and the downstream cluster
checking the hypervisor (Azure in my case, or vSphere for customer) we can see that the virtual machine is still there and running

Result
The actual virtual machine is not removed from the hypervisor/cloud
Expected Result

The actual virtual machine should be removed from the hypervisor/cloud
Workaround
As a workaround, we can manually delete the VM from the cloud as long as we see the cluster comes back in the Active state and the node disappears from the cluster.
Additional Notes

The issue is reproduced on the Docker-install Rancher v2.9.2 upgraded to 2.9.4 with a Digital Ocean node-driver DS cluster.
The issue affects only RKE1 clusters. RKE2/k3s node-driver clusters are not affected.
This is a regression caused by https://github.com/rancher/rancher/pull/47419 (which was backported to earlier release lines as well)

SURE-9483

Root cause
A file permission issue prevents rancher-machine from accessing the necessary files to delete the VM.
What was fixed, or what changes have occurred
Set the file permissions properly during the file creation process.
Areas or cases that should be tested
Testing should include the scenarios outlined in the issue and extend to multiple infrastructure providers for comprehensive validation.
What areas could experience regressions?

Cluster provisioning in general
for existing RKE1 or RKE2/K3s clusters created before upgrading Rancher

deletion of  "old" nodes
addition and deletion of new node pools or  nodes

Deletion nodes of newly-created  RKE1 or RKE2/K3s clusters

In all cases, we need to confirm the VM is destroyed on the infrastructure provider side
Are the repro steps accurate/minimal?
Yes

The fix can be validated on the latest v2.11-head image


Add CUDA tests to main

my understanding that this will be the main development repo for cuda/Gpu.
and this Pr is for activating the tests. then the next step now is merging cuda_main_temp here?

That is correct. The branch master will be the equivalent of cuda_main, development will be the equivalent of cuda_dev. If you haven't added anything to cuda_dev yet then yes the next stage is to merge cuda_main_temp into cuda_main


How to get RemoteAddr()?

How to get RemoteAddr()?

https://github.com/panjf2000/gnet/blob/36258e2/connection_unix.go#L251


PKS 1.8 Documentation - Change recommendation to a single NVDS to NSX-T Edges

In the section - Create Transport Zones

"However, the default TZs cannot be used by TKGI. If you do use them, you will receive an pks-nsx-t-osb-proxy BOSH error when you try to deploy TKGI."

Working with Francis internally we have tested using the default transport zones and as long as you specify the nvds name on the NSX-T Edge to be the same that is defined on the default transport zones (nsxHostSwitch) then you wont receive the BOSH error when you deploy TKGI.  This would allow us to continue recommending the practice of a single n-vds design so we can leverage multi-tep in production.
This would remove the need for the creation of a TEP port group in the following steps and you would need to add additional steps for creating another trunk port group.

Create EDGE-UPLINK-PG-A (VLAN TRUNK) with vmnic0 as active and vmnic1 as standby
Create EDGE-UPLINK-PG-B (VLAN TRUNK) with vmnic1 as active and vmnic0 as standby

We should also add a section on the creation of uplink profiles to support multi-tep and a single-nvds design. This would include the creation of custom failover teaming policies as well as setting up the vlan tag and MTU for the TEP interfaces for the compute hosts as well as the edges.  https://communities.vmware.com/docs/DOC-37591 (Page 26)
We would also want to cover setting up a Transport Node Profile as a part of the workflow since the recommendations for AZs are generally based on cluster boundaries and to ensure consistent configuration on the TNs within a cluster we leverage these in standard engagements.
Please reach out if you have any questions.

Thanks. Working on this.

This PR addresses most of what is in this issue: https://github.com/pivotal-cf/docs-pks/tree/address-issue-1169. Still need to follow up on the port groups.

this is addressed here: https://docs.pivotal.io/tkgi/1-9/nsxt-3-0-install.html#configure-the-n-vds-switch
closing as fixed
@kalenarndt please review and open new issue if anything is amiss, thanks


Identity threebot name is changed to lower case

https://github.com/threefoldtech/jumpscaleX_core/blob/3c6ca936a1ab5cd5a432e4f36e2a73a210c41e85/JumpscaleCore/me/Me.py#L38
If the threebot name contains both upper and lower case letters, the registration to the phonebook fails when the identity is already saved there because locally the name is lower cased.

https://github.com/threefoldtech/zos/issues/706


[Bug] Cookie name and subtext are not clickable #51 resolving

Description

Basically the cursor stays the normal cursor on the text and changes to the pointer on the checkbox
Issue Ticket Number
Fixes #51

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes needed to the documentation

Thanks a lot @Gismo1337!!


Reload the runner if structure.sql changes

Closes #326
There already exists a test that modifications to structure.sql ends up reloading but without proper integration tests this behaviour isn't really testable.
To confirm this works:

Get a rails app with structure.sql
Make a change and save, look at the log output to validate it reloaded.
Make some changes to a *.rb file to confirm that this new listener doesn't overwrite the main one.

Thank you! I've tested and it's working well.

Let's also update workspace_did_change_watched_files to remove structure.sql? We can add a comment that it's only for Ruby files.

Just needs a rebase, but LGTM.


Drag element not mainting styles

When dragging an element in a container the element attached to the cursor doesn't maintain it's css styles while being drug around (which makes things visually confusing for the user since everything collapses into and ugly state). I've made sure the dragula.min.css file is imported.  My html is setup as follows and I pass the section object to dragula. I'm not using npm or a package manager of any kind. Just downloaded the source and placed the dist files into django's static folder.

Resolved the issue by having the mirror container be the same as the element i'm moving instead of the default body.


Logout still not working on PWA?

See #141 this was seemingly fixed but on the installed homescreen app its still off.

Maybe also tied to #168


NOT working if I capture image from camera.

NOT working if I take an image from camera. It does not fire the callback function. Any solution?

Hi Motiur,
I am having the same issue and I didn't find solution anywhere. Did you managed to fix this somehow?
Thanks
Rajbir

seems to be happening on android 10, works fine on 8 & 9... someone found a solution?

Well I didnt  found a complete solution but found a workaround to open the
camera with basic click function.
On Wed, 21 Jul 2021 at 11:56 PM, andy0404 @.***> wrote:

seems to be happening on android 10, works fine on 8 & 9... someone found
a solution?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prashantsaini1/android-titanium-imagecropper/issues/9#issuecomment-884399199,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQC7IB3KKR4PYBA64S33TWDTY4GMVANCNFSM4KJ74CRQ
.

the camera opens for me it's the callback function which is not triggered when the user takes a picture


fix values references and bump version

PR to fix https://github.com/quickwit-oss/helm-charts/issues/106

Thanks, @coutug!


Strange Icons

I'm not sure what I'm missing or why things are rendering like this.
Thought this maybe a common one.
Everything looks fine except when responses don't have a schema.
Example:

An example css class that makes that:
zippy-empty[_ngcontent-grp-11] .zippy-indicator[_ngcontent-grp-11]:before {
    content: "â€”";
    font-weight: bold;
}

Not sure why this happening though (minus sign "-" being switched with whatever that symbol is)

Duplicate of https://github.com/Rebilly/ReDoc/issues/213 and https://github.com/Rebilly/ReDoc/issues/29
tl;dr
Just add <meta charset="utf-8"/> to your html


Adjusted focus outline for buttons

Why
PR for #776
Changed up focus outline for button to color that meets contrast guidelines with page's white background and button's light blue color.

Pre-Merge Checklist

[ ] All new features have been described in the pull request
[x] Security & accessibility have been considered
[x] All outstanding questions and concerns have been resolved
-- Would like to note, I don't have experience with Ruby + Bulma SCSS but looked through documentation and believe I've added CSS correctly to bulma_simple_form_customization. Please let me know if there are any errors!
[x] Any next steps that seem like good ideas have been created as issues for future discussion & implementation
[ ] High quality tests have been added, or an explanation has been given why the features cannot be tested
[ ] New features have been documented, and the code is understandable and well commented
[ ] Entry added to CHANGELOG.md if appropriate
-- Unsure if focus CSS change is needed for CHANGELOG but happy to add it if needed!

What

[x] Tested current focus in Mutual Aid app using Chrome DevTools

[x] In Chrome DevTools, adjust colors and use one that passes 3:1 ratio (see WCAG 1.4.11 Non-text Contrast)

[x] Add CSS to bulma_simple_form_customization.scss

How
WCAG 1.4.11 states UI elements should meet 3:1 ratio. Black/dark gray is used throughout the site which I chose to stick with color scheme. In adjusting the colors, the dark gray outline with the white background reaches 10.9:1 contrast ratio which is well past the requirement

Next, meeting 1.4.11 again, I tested the turquoise/light blue color with the same dark gray outline. The color combination reaches 5.6:1 which is again past the requirement.

Please note - this doesn't mean you have to go with the dark gray color. The color can be changed but it needs to meet 3:1 in both cases.
Testing
Struggled a bit with testing as I was trying to get serve up the changes but using npm run serve sounds like it was the wrong way to go. A lot of what I tested was in Google Chrome DevTools. I know my PR probably needs a lot of work 😓 If anyone can lead me to spinning this up with my changes to test, I'm happy to work on this further!
Next Steps
Only note I have here is to consider changing the text color for the button as well to pass.

For text contrast, we would need to meet 4.5:1. Using black/dark gray would work here as well (albeit there are other options that work too)
Outstanding Questions, Concerns and Other Notes
My main note is just not having been able to test which I will continue to work on. I figured since this is a color change and somewhat minor but need to test and see that this worked.
Accessibility
I've covered a lot of the accessibility issues above but this is to resolve an accessibility issue that's already open #776. Happy to open another issue for the text on the button or submit a pull request to change this as well.
Security
N/A
Meta
I always enjoy working on accessibility and helping to the best of my ability and appreciate being able to work with @exbinary and @metamoni . Please let me know if you have any feedback on this or need anything else 😄

@exbinary can you please review this? I'm not a contributor to this repo so I can't approve PRs. Looks good to me unless you know of a different colour that works better and still meets the requirements. Thanks!

Thank  you so much @natastro !!! All accessibility contributions are so welcomed! :)


Cannot create App plan in sub A for ASEv3 in Sub B

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az appservice plan create
Errors:
App service environment 'subscriptions/<subB>/resourceGroups/ase-rg/providers/Microsoft.Web/hostingEnvironments/begimase' not found in subscription.

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az appservice plan create --name {} --resource-group {} --app-service-environment {} --location {} --number-of-workers {} --sku {}

Expected Behavior
Environment Summary
Windows-10-10.0.22000-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.29.0

Extensions:
aks-preview 0.5.34

Additional Context

route to service team

Template works, so it's just CLI and PS1
{
"$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
"contentVersion": "<IP_ADDRESS>",
"parameters": {},
"variables": {},
"resources": [
{
"type": "Microsoft.Web/serverfarms",
"apiVersion": "2020-06-01",
"name": "begim-ase-wap2",
"location": "uksouth",
"sku": {
"name": "I1v2"
},
"properties":{
"capacity": 1,
"hostingEnvironmentProfile": {
"id": "/subscriptions//resourceGroups/ase-rg/providers/Microsoft.Web/hostingEnvironments/begimase"
}
}
}
]
}
#deploy context subA


FRI verify_remainder_64 Assembly Routine

This PR implements FRI remainder verification using probabilistic argument, see https://github.com/0xPolygonMiden/miden-vm/issues/592 for more details.
Cost of assembly routine: 4056 -cycles.

After implementing all the suggestions from https://github.com/0xPolygonMiden/miden-vm/pull/644#pullrequestreview-1252632080 , https://github.com/0xPolygonMiden/miden-vm/pull/644#discussion_r1072968761 and https://github.com/0xPolygonMiden/miden-vm/pull/644#discussion_r1072980848 along with some more optimizations we're down to 3178 -cycles.

Note
From 4056 -cycles, we've reduced cycle count by ~21.65 % 🔥. Now we're at 3178 -cycles.

Exactly! We can use repeated squaring which boils down to $\log(l) = 6$ multiplications to compute $\tau^l$.

At commit 93921cf6a09024ed25e0b7a2a267c1cd7c0ee717, I implement new formula for computing β, following @Al-Kindi-0 's suggestion https://github.com/0xPolygonMiden/miden-vm/pull/644#pullrequestreview-1265141094. This brings down cost of verify_remainder_64 routine to 2561 -cycles --- which is ~19.41% cheaper compared to our last optimization attempt https://github.com/0xPolygonMiden/miden-vm/pull/644#issuecomment-1397066807 🔥

At commit 93921cf, I implement new formula for computing β, following @Al-Kindi-0 's suggestion #644 (review). This brings down cost of verify_remainder_64 routine to 2561 -cycles --- which is ~19.41% cheaper compared to our last optimization attempt #644 (comment) 🔥

This is really awesome! Great job @Al-Kindi-0 and @itzmeanjan!!
@itzmeanjan - could you address the comments from https://github.com/0xPolygonMiden/miden-vm/pull/644#pullrequestreview-1264590590 and then we can merge?

could you address the comments from #644 (review) and then we can merge?

I address them in 00fb2b23

With changes made at 00fb2b23, which makes code easy to comprehend, reduces duplication etc. our cycle cost increases little bit ( compared to https://github.com/0xPolygonMiden/miden-vm/pull/644#issuecomment-1402162674 ). Now one can execute verify_remainder_64 routine at cost of 2618 -cycles.

I left a few more comments inline.

Address in b33313f6bf176da61b8b7dbd82c5b8a5d0ea9b21

Also, could we update the hash function to RPO in this PR?

Done in d1b9fd26a83778abb412a2e434abc97da9f6e139

All looks good! Thank you!

Thank you for the review.


How to load a jpeg file efficiently ?

Hello,
I am trying to load rather large JPG files (4800x3600 pixels) into holoviews. Thusfar I have not found a way to replicate the fast/efficient method from Bokeh:
p = figure(x_range=(0, xdim), y_range=(0, ydim), tools=['box_zoom','reset'], width=1000, height=800, match_aspect=True, title='Load Image')
p.image_url( url=[link],  x=0, y=0, w=xdim, h=ydim, anchor="bottom_left")

In the above example I know the dimensions (xdim, ydim) of the JPG file and the name of the JPG is in the link variable. This loads very fast but when using the Image option in holoviews that only seems to read PNG.
Any ideas ?
regards
Cor

The quickest workaround might be to use PIL to load your JPG to an array and stick that into hv.RGB (if you want to preserve the color channels). If it is monochrome, you can still use PIL to load an NxM image and load that with hv.Image.
I'm a little surprised plt.imread(filename) doesn't support JPG. If you are using hv.Image, you might also want to use the regrid datashader operation to avoid passing such a large image to the browser.

Hi,
Thanks for
I have tried using imageio
import imageio
img = imageio.imread(file)
and img=hv.RGB(nimg, bounds=(0,0,xdim,ydim))

My default image takes over 10 seconds to load in this fashion, using
img=hv.Image(nimg, bounds=(0,0,xdim,ydim))
takes only 5 seconds but has only one channel ...
Cor

How long does it take to create the RGB or Image element without the imread loading step and also without display? It should be very quick but I can expect bokeh to struggle with rendering such a large image...

I just checked and its indeed the final stage where the image gets rendered that slows everything down. Loading the image and createing the hv.RGB or hv.Image takes 0.5 seconds only.

In bokeh I have used the code below, that works really fast ... so I was amazed when bokeh could not deliver the same
p = figure(x_range=(0, xdim), y_range=(0, ydim), tools=['box_zoom','reset'],
           title='Test')
p.image_url(link)
show(p)

There is a difference to the way image_url renders and how image renders, drawing an actual jpg or png on a canvas is very cheap, serializing and deserializing a huge RGB array is very expensive. I'd be open to supporting image_url directly in some form.

Hi,
Ok I can understand that. I will be - and probably more people - hoping to see this arrive. Still I want to thank you for the effort in making packages like holoviews that allow many to work with data a lot easier. My application (building something where people have to interactively  set markers on images) is not so standard so I can understand that it doesnt work yet.


Refactor npm dependencies update

What this PR does / why we need it:
Refactor the logic to handle @grafana/* packages when updating dependencies in the migrate and update commands
The previous logic was spread in different methods and was harder to follow.
I renamed the option for a more verbose one and made it always to move grafana FE dependencies from dev to dependencies. This is the way grafana dependencies should be declared in plugins.
Which issue(s) this PR fixes:

Fixes https://github.com/grafana/plugin-tools/issues/122
Fixes https://github.com/grafana/plugin-tools/issues/163
Special notes for your reviewer:

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Testing this out by running npx<EMAIL_ADDRESS>update I'm seeing some strange behaviour.
Testing against jackw/grafana-hetzner-cloud-datasource

If I answer no to updating the @grafana/* packages I'm then asked if I want to update an empty list of dependencies. The current version<EMAIL_ADDRESS>hides the Would you like to update the following dependencies in the package.json?  prompt when there is nothing to update.

✔ Would you like to update the @grafana/* packages dependencies?

 (y/N) · false
? Would you like to update the following dependencies in the package.json?

  Dependencies

 …
❯ Yes, all of them
  Yes, but only the outdated ones
  No

If I answer yes to updating the @grafana/* dependencies I'm then greeted with the following but they're the same versions as the plugin is currently using. I was expecting it to bump all grafana dependencies to 9.3.8. 🤔

✔ Would you like to update the @grafana/* packages dependencies?

 (y/N) · true
? Would you like to update the following dependencies in the package.json?

  Dev Dependencies
  @grafana/e2e - 9.2.5 (new)
  @grafana/e2e-selectors - 9.2.5 (new)

 …
❯ Yes, all of them
  Yes, but only the outdated ones
  No

Testing against  grafana-plugin-examples/examples/app-basic

With plugins that use version 9.2.1 of the Grafana dependencies it mentions the latest version (9.3.8) but updates it to 9.2.5.

✔ Would you like to update the @grafana/* packages dependencies?

 (y/N) · true
? Would you like to update the following dependencies in the package.json?

  Dependencies
  @grafana/data - 9.3.8 -> 9.2.5 (updated)
  @grafana/runtime - 9.3.8 -> 9.2.5 (updated)
  @grafana/ui - 9.3.8 -> 9.2.5 (updated)

  Dev Dependencies
  @grafana/e2e - 9.2.5 (new)
  @grafana/e2e-selectors - 9.2.5 (new)

 …
❯ Yes, all of them
  Yes, but only the outdated ones
  No

@jackw this refactor appears to be more complicated than I thought. I'll close this PR and open a simpler one only to not update the grafana dependencies and then I'll do some follow ups for the refactor.


Remove TUAW from Vienna's default subscriptions

It now redirects to some AOL feed of non-Apple related stuff. Probably best to replace with six colours or something?

Six Colors seems like a safe/neutral choice. I've just started using Vienna so I wouldn't mind making such a tiny contribution. 😂


⚠ GitLab Server has degraded performance

In 034a159, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11459 ms

Resolved: GitLab Server performance has improved in 6eca913.


FII-030 - Display the Inline files on the File inspector - UI impleme…

…ntation (LEAN-920)

Can one of the admins verify this patch?


Create an archivebox server page with a UI & REST API endpoint to add links to the archive

Page Requirements

[x] ability to add a single page to the archive (like echo <url> | archivebox add)
[x] ability to import a list of pages / feed of URLs into the archive (like  archivebox add <url>)
[X] link to the homepage of the index /
[X] link to the django admin list of URLs for editing the archive /admin/
[X] link to the archivebox github repo & documentation

Setup
To set up your dev environment, run the following:
pip3 install --upgrade pip setuptools virtualenv pipenv

cd ~/path/to/ArchiveBox
git checkout django
git pull

pipenv install
pip install -e

Then to create a new collection and run the server, do:
mkdir data
cd data/
archivebox init
archivebox add https://example.com
archivebox info

archivebox server
# then open: http://<IP_ADDRESS>:8000

To edit the templates you can change files in:

archivebox/themes
archivebox/legacy/templates

CC: @mlazana

This is planned for v0.5. There is already an admin UI for editing and removing existing links in v0.4, so v0.5 will add the ability to submit new links via UI as well.

This is done and available on the latest django branch (soon to be master).
git checkout django
git pull
docker build . -t archivebox
docker run -v $PWD/output:/data archivebox init
docker run -v $PWD/output:/data archivebox add 'https://example.com' --only-new

+1 here, would love to trigger it from https://github.com/dgtlmoon/changedetection.io

this is a very old issue @dgtlmoon 😄 ! if you're looking for a UI page we've already had one for several versions now, if you're looking for a REST API endpoint to add links, you can just POST to /core/snapshot/add/ (using archivebox:dev) and find more info here:

https://github.com/ArchiveBox/ArchiveBox/issues/496
https://github.com/ArchiveBox/ArchiveBox/issues/577#issuecomment-872640967


[QUESTION] Please add transitions support

Recently I have been playing with this API and I am willing to add a transition to the slides, how can I do that?

Clone the repo, make and test your changes, submit a pull request. There are several guide and videos available on submitting pull reqs.


#148476333 Add nagging feature

What the PR does

Enables user to configure nag time
Sends emails to the user when the check is down
Adds check on nag mode with a nag icon

Changes Unknown when pulling fb4ec0e7118cd8f6b965be7a48d929f8b5f8be97 on ft-nag-user-148476333 into ** on master**.


Add more iot resources

Issue-id
PR Checklist

[x] I have read the CONTRIBUTING.MD instructions
[x] I have changed the resourceDefinition.json
[x] I have generated the resource model (there's a models_generated.go file in my PR)
[x] I have updated the README.md#resource-status
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?

Description

Does this introduce a breaking change

[ ] YES
[x] NO

Testing

@arnaudlh can we go forward with this PR?


[docs] R-packages/man not generated

The R reference manual is missing most of its content: http://mxnet.io/api/r/mxnet-r-reference-manual.pdf
This is where the R docs are generated: https://github.com/dmlc/mxnet/blob/master/docs/sphinx_util.py#L19
The files in R-packages/man are generated using roxygen2::roxygenize(), which are then used to make the reference manual. You may need to install the packages "roxygen2" on the docs build host, but without access to the host or logs I'm not sure.

the man files are generated by make in the root folder I think

I think the man files in the root folder you're referring to are unrelated (that is, not these .Rd files). Building the docs locally by running build-preview.sh shows R-package/man full of .Rd files after running R -e "roxygen2::roxygenize()" as part of the sphinx_util script and a full PDF after Rd2PDF, as expected.
Besides running install.packages("roxygen2"), I'm not sure how else to troubleshoot the docs build without access to the host or logs -- @kevinthesun has some experience building the R docs, and I think only @mli has access to the host.
Previously, the R-package/man files were just committed in the repo -- as a stopgap, I could check in the .Rd files, but the right way would be to generate these.

@andremoeller I just did
sudo R

install.packages("roxygen2")
and exit.
Is it enough to fix this?

Unless the build hasn't run yet (I think it's hourly?), then it didn't seem to fix it -- there should be tons of .Rd files in R-packages/man after sphinx_util.py runs, and a PDF with 43 pages in docs_/build/api/r/.
As a stopgap I'll send a PR that commits the .Rd files, though they should be generated or R doc changes won't get picked up automatically

All good now: http://mxnet.io/api/r/mxnet-r-reference-manual.pdf


[feature-request] ability to bypass blueprint validation

Feature Request: The ability to bypass Blueprint validation.
I'm continually testing the newest Blueprint functionality and would like to use CloudBreak for this.
CloudBreak currently has it's own validation (separate from Ambari's) which blocks certain Blueprints.
For example: HA Blueprints. CloudBreak marks them as invalid as it does not have that logic yet.
Long-term the logic for such things should be added, but in the short term and as new Ambari versions are released, the ability to bypass validation would be useful. Or a separate CloudBreak environment for such testing.
--
Sean Roberts
Hortonworks

Hi,
We added an option field to be able to disable the blueprint validation. We'll push the change to production with the next release.


Add-TxtCsvHelper

Please add TxtCsvHelper to test

Hey @camdrudge, is this something you'd be able to provide a PR for? Take a look at some of the closed PRs for examples.
If you provide the PR, I'm happy to run the benchmark again and update my blog post.

Fixed with https://github.com/joelverhagen/NCsvPerf/pull/32.


Warning message

Would you have any suggestions of how to fix a warning message I am getting every few seconds:

The message means that the scheduled heartbeat fires 0.3 to 0.4 seconds late.  That could indicate an overloaded system (not the case, looking at the CPU and memory usage) or something blocking NodeJS, most likely another plugin doing synchronous I/O.
Did you try restarting Homebridge?  Have you seen these messages for all 14 days that Homebridge has been running, or did they appear more recently?

The message means that the scheduled heartbeat fires 0.3 to 0.4 seconds late.  That could indicate an overloaded system (not the case, looking at the CPU and memory usage) or something blocking NodeJS, most likely another plugin doing synchronous I/O.
Did you try restarting Homebridge?  Have you seen these messages for all 14 days that Homebridge has been running, or did they appear more recently?

I restarted Homebridge and the warning is gone.  Thank you!

I restarted Homebridge and the warning is gone.  Thank you!


TrackDetailActivity: landscape layout broken

The landscape layout does not show all data fields (although it should).

Removed landscape for TrackDetailActivity and it should be reimplemented completely after refactoring portrait mode.
See #31.


Add initial electricity time series dataset

Here is an initial prototype time series dataset

@kashif Some commits on the PR branch are not authored by you, so could you please open a new PR and not use rebase this time :)? You can copy and paste the dataset dir to the new branch.

making a new PR


PokéCenter Heals Rule Change

This: https://discord.com/channels/827408079272017930/833822819073654834/985968512289898526
Given that now PC heals don't apply until after the first non-rival trainer battle, we will need to make a change to my recent implementation of tracking PC heals.
@GB127 had suggested that this be done manually in my PR and I wanted it to be automatically tracked because nothing else in the Player's Pokémon page is tracked manually. Perhaps that'll have to change for now, until we can figure out a way to know if the player fought the first trainer outside of the rival fights. Unfortunately, I don't see a way to do that at the moment. Perhaps we could have the player tell the tracker to start tracking heals?
I'm asking for help on this because I am about to be unavailable for about two weeks. If anyone wants to take on improving the groundwork I laid, have at it! I will still be able to review/approve as long as you tag me in the PR. Thanks!

Having a way to tell the tracker to start sounds like a good option. It'd be good if it could be flexible with a way to manually adjust it if needed, as well as a way to turn auto mode off again. Unfortunately with my setup I can't contribute, but good luck to anyone who can take this on. 😄

Yes! All of that is an excellent idea! Obviously, space considerations are an issue with all of this as well, and GBA button configurations aren't going to be easy to adapt to using this, so it may take a bit more effort to get the Input class built up to better understand the overall state of the tracker. If it doesn't get done when I get back from my vacation, then I'll get it done.

Closing this as completed with #80. I think Fell's implementation solves the problem the best way we can with the immediacy of the changes to the rules.


[DOR-81]: Temporarily hide records without thumbnails from search results

Ticket URL: https://national-archives.atlassian.net/browse/DOR-81
About these changes
Hides any results which don't contain a thumbnail. This is a temporary fix for user testing, to hide parent records in our subset of record results which don't contain thumbnails.
Note: this will need to be reverted after user testing (or at an agreed time) to ensure we don't unintentionally hide any records without thumbnails in future (see ticket https://national-archives.atlassian.net/browse/DOR-82)
How to check these changes
Where possible, provide guidance to help your reviewer
Before assigning to reviewer, please make sure you have

[ ] Checked things thoroughly before handing over to reviewer
[ ] Checked PR title starts with ticket number as per project conventions to help us keep track of changes
[ ] Ensured that PR includes only commits relevant to the ticket
[ ] Waited for all CI jobs to pass before requesting a review
[ ] Added/updated tests and documentation where relevant

Merging PR guidance
Follow docs\developer-guide\contributing.md
Deployment guidance
Follow docs\infra\environments.md

Is this meant to be merged into develop?

Is this meant to be merged into develop?

Nope 🤦♀  updated! thanks :)


[Feature] show up or down color of candle bar based on previous close and current close

Feature Description
Can add a new candle type to show up or down color of candle bar based on previous close and current close.  Currently up or down color depends on current close > current open or current close < current open
To Do
CandleBarView add logic to check up or down color based on previous close >current close  or previous close < current close.

Definitely, this is a must. I'm just beginning to use klinecharts so I don't know how to do it but I will try to implement it and if I'm able to, I will do a PR.


Add support for Netty 4.1.115+

This commit adds additional reflection metadata for MpscUnpaddedArrayQueueConsumerIndexField, MpscUnpaddedArrayQueueProducerIndexField and MpscUnpaddedArrayQueueProducerLimitField in the io.netty.util.internal.shaded.org.jctools.queues.unpadded package.
It also refines the metadata with a "default-for" entry that ensures that previous metadata continue to be used for Netty 4.1.0 to 4.1.114, uses consistently Netty 4.1.80 for the tests of the previous version, and adds related tests for Netty 4.1.115.

Thanks @sdeleuze, that looks good to me.

@zakkak as well


removing log statements

Short description outlining key changes/additions
removing log statements
JIRA Ticket Number
Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: Security warnings found!


add --scenarios flag

add a --scenarios flag which takes a list of string scenario names and to allow users to specify multiple scenarios per state. The launch_experiment script should then be able to pass along each scenario to the run_task.py runner script via the --scenario flag on the runner.
This will expand the launch_experiment's functionality and make it less likely that users need to write their own custom versions of the script.
Can also introduce an explicit mode which instead reads in a CSV that defines each task and the keywords passed along to it. For example, if your run_task.py script takes in 2 arguments --state, --scenario, and the obligatory --jobid you could provide a CSV with the two changing columns columns state, scenario and then each row of that CSV could be launched as its own task. Since the user already provides the --jobid flag to the launcher a similar row is not needed for that argument. I am curious if you would use such a mode @kokbent

CSV idea is pretty neat 🤔 although it's seldom that we would run differential scenarios for each state at the moment... I think there's a good chance I would use it to be sure the book-keeping is right but not frequently...

Copied from another chat:

or you thinking of maximum flexibility: if for whatever reason run_task.py has three arguments, I can then flexibly csv of three columns to control this?
This is good and a bit more future proofing yeah

edit: breaking this into two separate issues, --explicit flag will now be tracked here: https://github.com/CDCgov/scenarios-hpc-azure/issues/9

CLOSED BY #14, explicit mode covers all use cases for this sort of functionality anyways


force numpy<2.0

see #69, no idea how to fix, It can be reproduces locally but its very odd.
since we will release a new version later, we can maybe stick to numpy<2.0 here.
feel free to check for alternatives
add matplotlib deprecation cm.viridis to cmap=plt.get_cmap('viridis')

This is not the issue.
The issue is that aspect ratio handling doe not work properly within Matplotlib in python 3.8 in combination with numpy 2.x
Specifically, this fails:
ax.set_box_aspect([1, 1, 0])

which for some reason works on all other python versions we're testing


[YUNIKORN-524] use apache approved action for push

Apache limits the actiona that are allowed in gthub actions. The action
used for the auto push is not on that list. Replace with standard push.


Added optional MODSEC_DISABLE_BACKEND_COMPRESSION env variable

The ModSecurity SecDisableBackendCompression could not be configured in this docker image. This PR adds that option.

Maybe we can add some shell scripting to only generate the variable if running on apache. This error doesn't look nice, and might confuse users:
nginx-nightly            | nginx: [emerg] "modsecurity_rules_file" directive Rules error. File: /etc/modsecurity.d/modsecurity.conf. Line: 224. Column: 19. SecDisableBackendCompression is not supported.  in /etc/nginx/conf.d/modsecurity.conf:2

If we can live with that, then no problems

Good point. I agree that we should remove it from httpd.

I've tried to recreate the issue you had with nginx and the variable but I can't. I believe it was already solved with additional RUN instruction that comments the directive for v3.
I've rebased onto develop, so that builds should run again.


terminated by signal SIGABRT

Unfortunately can't make public the source repo. But here's the stacktrace.

<--- Last few GCs --->

[4286:0x105002600]   104823 ms: Mark-sweep 2058.2 (2064.7) -> 2058.1 (2064.7) MB, 1351.9 / 0.0 ms  (average mu = 0.063, current mu = 0.000) last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

    0: ExitFrame [pc: 0x1006db52d]
Security context: 0x0ce93c000921 <JSObject>
    1: toString [0xce9f846ef09] [buffer.js:~756] [pc=0x195c047b9962](this=0x0ce95ead40e9 <Uint8Array map = 0xce94cea3ee1>,0x0ce9f8467cb9 <String[#4]: utf8>,0x0ce9d93004b9 <undefined>,0x0ce9d93004b9 <undefined>)
    2: arguments adaptor frame: 1->3
    3: readFileWorker(aka readFileWorker) [0xce9e294f259] [/Users/abhishiv/work/gratico/leela/node_modules/typedoc/...

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
 1: 0x100b9f982 node::Abort() (.cold.1) [/usr/local/bin/node]
 2: 0x100084338 node::FatalError(char const*, char const*) [/usr/local/bin/node]
 3: 0x100084460 node::OnFatalError(char const*, char const*) [/usr/local/bin/node]
 4: 0x10017dfc1 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 5: 0x10017df6b v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/usr/local/bin/node]
 6: 0x100297f23 v8::internal::Heap::FatalProcessOutOfMemory(char const*) [/usr/local/bin/node]
 7: 0x1002992a8 v8::internal::Heap::MarkCompactPrologue() [/usr/local/bin/node]
 8: 0x100296e76 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [/usr/local/bin/node]
 9: 0x100295955 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/usr/local/bin/node]
10: 0x100295bd2 v8::internal::Heap::CollectAllAvailableGarbage(v8::internal::GarbageCollectionReason) [/usr/local/bin/node]
11: 0x10029d36d v8::internal::Heap::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/usr/local/bin/node]
12: 0x10027a4f5 v8::internal::Factory::AllocateRawWithImmortalMap(int, v8::internal::AllocationType, v8::internal::Map, v8::internal::AllocationAlignment) [/usr/local/bin/node]
13: 0x10027c8e8 v8::internal::Factory::NewRawOneByteString(int, v8::internal::AllocationType) [/usr/local/bin/node]
14: 0x10027c997 v8::internal::Factory::NewStringFromUtf8(v8::internal::Vector<char const> const&, v8::internal::AllocationType) [/usr/local/bin/node]
15: 0x100195e8b v8::String::NewFromUtf8(v8::Isolate*, char const*, v8::NewStringType, int) [/usr/local/bin/node]
16: 0x10010e6df node::StringBytes::Encode(v8::Isolate*, char const*, unsigned long, node::encoding, v8::Local<v8::Value>*) [/usr/local/bin/node]
17: 0x10006ba10 void node::Buffer::(anonymous namespace)::StringSlice<(node::encoding)1>(v8::FunctionCallbackInfo<v8::Value> const&) [/usr/local/bin/node]
18: 0x1006db52d Builtins_CallApiCallback [/usr/local/bin/node]
fish: './node_modules/.bin/typedoc  --…' terminated by signal SIGABRT (Abort)

Environment

Typedoc version: 0.16.11
Typescript version: TypeScript 3.7.5
Node.js version: v13.5.0
OS: OSX

I'm afraid there isn't much I can do here without a reproduction... the stacktrace doesn't reveal much about the potential memory leak. The last bug that caused this was #1150, where a recursive type wasn't handled properly by TypeDoc. Are you using any recursive types by any chance?

Hey @Gerrir0, ok I understand. Just FYI --excludeExternals fixed it for me, so maybe it was something in my node_modules causing this.
closing


feat: directional move focus #55

that's still wip, I'll write the tests later on today

I have tried the branch locally and it's working great for me. Just leaving some nitpicks which came to my mind from the first look.

Hey @denismaxim0v - on the surface this looks great. Before I go deeper into it, do you think you can add some tests for these? There are quite some edge cases here that we should probably be testing. (eg. making sure it picks the terminal with the largest overlap in each case, etc.) You can look at some of the resize tests in this repo for inspiration.

Hey @denismaxim0v - on the surface this looks great. Before I go deeper into it, do you think you can add some tests for these? There are quite some edge cases here that we should probably be testing. (eg. making sure it picks the terminal with the largest overlap in each case, etc.) You can look at some of the resize tests in this repo for inspiration.

Yup, working on it, gonna commit those tomorrow


frozen description equates it to allow_mutation, but it is the inverse

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Thanks for the awesome library! I recently migrated to v2 and am loving it. I did notice a very minor, though potentially important issue in the description of frozen config argument:
https://github.com/pydantic/pydantic/blob/c067c1b9d0e5ffdf5b0fd53bf0e138cb5924f750/pydantic/config.py#L128
Though the names are clear, one could potentially assume that they are equivalent based on this description. The docs clearly indicate that frozen is the invers of allow_mutation, perhaps could also indicate that on this line as well.
Something like "On V1, the inverse of this setting was called allow_mutation, and was True by default."
Let me know what you think, and then I can open a PR.
Example Code
No response
Python, Pydantic & OS Version
pydantic version: 2.5.3
        pydantic-core version: 2.14.6
          pydantic-core build: profile=release pgo=true
               python version: 3.12.0 (main, Nov 29 2023, 03:32:06) [GCC 10.2.1 20210110]
                     platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.31
             related packages: typing_extensions-4.9.0

@themattmorris,
Sounds like a great change. Please open a PR and I'd be happy to review!


cleanup(libscap): Allow retries after encountering SCAP_EOF

Call gzclearerr if gzread returns fewer bytes than expected. This lets us "tail" a file being written by another process.

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: geraldcombs
Once this PR has been reviewed and has the lgtm label, please assign gnosek for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone TBD

LGTM label has been added.  Git tree hash: a3841cf57628ac62d83d8928fb0fe41122b1cfcb

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geraldcombs, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone TBD

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, geraldcombs, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.17.0


How to make use of the news?

Has anyone figured out a way to take the news headline and scroll the text? Or even better scroll the article that's tied to the headline?

Ops didn't mean to close it.

To scroll the text you could create a Text widget and put all the headlines in there, and then set the size to show only 5 articles. Once you call the method again to refresh the articles you could have it scroll up to fill in the new text. However, you would want to erase the old headlines after 10 refreshes just so you are not making too many necessary variables.

Hmmm...good suggestion. I'll take a look at this. Thanks for the idea!


idea: hosting community project docs under a single location

Currently most community projects have their own websites, likely using RTD and their own themes.
The only exception I am aware of is ansible-lint which is nicely hosted under docs.ansible.com but the publishing process went broken and remained broken for more than an year. Mainly the project docs are out of sync with the project itself.
As I am aware of the sensitivity regarding what gets published to docs.ansible.com, I can understand that Ansible core team wants to assure that goes there is properly vetted. This is a bit of a conflict with the needs of the community projects, which may need or want to release docs at their own cadence, and more important not to depend on others regarding doc publishing.
A: subfolder under docs.ansible.com

maximum visibility
easy search (make use of same search engine)
cannot use RTD build process as we would need our internal publishing job to update production

B: subdomain like community.ansible.com

good visibility, makes it clearly visible it is community driven
still indexable and we could make results visible from docs searches
we can make use of RTD subproject feature
can use RTD custom domain feature to host all community docs, even disable advertising, with minor costs. If we decide to go DIY we can do it without affecting the user-experience
Using RTD brings serious benefits, like being able to preview the pull-request builded docs, making much easier to review changes made to them.

C: each project with its own site (current)

no search across projects
much harder to distinguish between projects under community umbrella and random projects from internet
documentation theming will obviously differ a lot between projects, affecting now only browsing but also maintenance
prevents us from analizing how community documentation is displayed, consumed or to make more holistic improvements.

Looks like everyone's going for option C for now.


Review of Wyre flow UX

Need to review the Wyre flow and get UX feedback/ correction.

UX team Sumaiah is working on this. ETA - Thursday.

UX team Sumaiah is working on this. ETA - Thursday.

Wyre/ramp screen color
https://xd.adobe.com/view/00485057-d5b7-4b2b-641b-20216b3a38d5-7679/screen/a3f25a38-b42a-4374-b63d-d188414f35b2/
https://xd.adobe.com/view/00485057-d5b7-4b2b-641b-20216b3a38d5-7679/screen/8a780aed-2930-43f7-bd27-96edd0c1693e/

Tested by Prasad


Remove python C-API refs from numpy_returns.cpp

Hi @davisking and @visionworkz, this PR would be my approach to remove the Python C-API from the numpy_returns wrapper. This could be used in other places as well, but i'd like to review and test this first in isolation.
I have written a very basic regression-test for #1220 , but marked the test-cases as "skip" because i have not invested time for the CI setup but used it continuously during my local development (@davisking i've seen you typically solve test-data with a base64-encoded string, i could give it a try here too).
Tell me what you think and how to proceed.

Awesome, I was also bothered by the C API.  You can save some test files in the test folder. Don't worry about base64 encoding it or anything funny like that, just stash them in python/test and access them as appropriate.
Also make sure the automated tests pass.  It seems like they are failing because opencv isn't installed on the test servers. It would be better if the tests didn't require opencv to be installed.

Thanks @supervacuus!

Shall we introduce a load/save image method in the dlib library? This will remove any dependencies on OpenCV or skimage. Sample code is given below:
#include <dlib/image_io.h>
py::object load_rgb_image (const std::string &path)
{
    matrix<rgb_pixel> img;
    load_image(img, path);
    return create_array_from_rgb_image(img);
}

bool has_ending (std::string const full_string, std::string const &ending) {
    if(full_string.length() >= ending.length()) {
        return (0 == full_string.compare(full_string.length() - ending.length(), ending.length(), ending));
    } else {
        return false;
    }
}

void save_rgb_image(py::object img, const std::string &path)
{
    if (!is_rgb_python_image(img))
        throw dlib::error("Unsupported image type, must be RGB image.");

    std::string lowered_path = path;
    std::transform(lowered_path.begin(), lowered_path.end(), lowered_path.begin(), ::tolower);

    if(has_ending(lowered_path, ".bmp")) {
        save_bmp(numpy_rgb_image(img), path);
    } else if(has_ending(lowered_path, ".dng")) {
        save_dng(numpy_rgb_image(img), path);
    } else if(has_ending(lowered_path, ".png")) {
        save_png(numpy_rgb_image(img), path);
    } else if(has_ending(lowered_path, ".jpg") || has_ending(lowered_path, ".jpeg")) {
        save_jpeg(numpy_rgb_image(img), path);
    } else {
        throw dlib::error("Unsupported image type, image path must end with one of [.bmp, .png, .dng, .jpg, .jpeg]");
    }
    return;
}

and
    m.def("load_rgb_image", &load_rgb_image,
	"Takes a path and returns a numpy array (RGB) containing the image",
	py::arg("path")
    );

    m.def("save_rgb_image", &save_rgb_image,
	"Saves the given (RGB) image to the specified path. Determines the file type from the file extension specified in the path",
	py::arg("img"), py::arg("path")
    );

@visionworkz From my pov it would be great not to depend on external libraries for basic image io. I can integrate your code in this PR. We could use this for some of the examples too (in another PR).
@davisking In this case I was specifically talking about the trained model for the shape predictor. It has ~9MB and i think there was a reason why you did not include it in the repository. If you can provide a lighter weight model to feed the test predictor that would be great.

@supervacuus from my pov as well, it will be very useful to have image io exposed from Python. I also suggest the inclusion of a show image function:
#include <dlib/gui_widgets.h>
void show_rgb_image(py::object img, const std::string &window_name) {
    if (!is_rgb_python_image(img))
        throw dlib::error("Unsupported image type, must be RGB image.");
    image_window the_window(numpy_rgb_image(img), window_name);
    the_window.wait_until_closed();
}

and
    m.def("show_rgb_image", &show_rgb_image,
	"Shows the image in a window",
	py::arg("img"), py::arg("window_name")="Image"
    );

I am adding the code samples here instead of in a PR as they depend on your create_array_from_rgb_image function included in this PR.
With these changes, we can remove skimage and OpenCV dependencies from all Python examples.
@davisking thoughts?

@supervacuus Ah, if you want a shape predictor test then just run the trainer on the example test data and then load the resulting model and test that it does what it's supposed to.  You could train it on one image, using really mild settings that train very quickly and then test that the model works well on the image it's trained on.
@visionworkz Yeah, removing the external dependencies in the examples is great.  I don't think you need show_rgb_image though.  Just using the dlib.image_window is good right? I guess it needs to be updated to deal with numpy arrays but that's not a big deal, just another overload of set_image().
Also, can't a def_buffer statement be added to the dlib::matrix classes so that the image loader can just return a matrix<rgb_pixel> directly without doing a copy? That would run faster. See http://pybind11.readthedocs.io/en/stable/advanced/pycpp/numpy.html.  I haven't used that pybind11 feature but it looks good.

@davisking the following code works perfectly:
import dlib
img = dlib.load_rgb_image("someimage.jpg")
window = dlib.image_window()
window.set_image(img)

I think we should leave one Python example with OpenCV as it demonstrates that images returned from dlib can be used with OpenCV too after swapping Red and Blue channels.

Yeah. Leaving one opencv example is a good idea.

@davisking, @supervacuus  one last consideration:
If we return an object via def_buffer, on the Python side, we would have to use something like:
np.array(matrix_instance, copy = False)

This does introduce more complexity on the Python side.
With the memory copy approach, it is already a Numpy array and can be used directly.
For me personally, I prefer the convenience of just getting a Numpy array without additional complexity.
Ideally, if we can find an approach which can return a Numpy array without the memory copying, that would be best in my opinion.

Yeah, I agree.  We shouldn't require python users to write weird code like
that.  It seems like there should be a way to make it transparent, but
maybe that's not the case.
I'm not adverse to adding a method to dlib::matrix like "steal_memory()" or
something that just returns the pointer to the matrix's memory block and
causes the matrix to go into an empty matrix state since it no longer has
any contents (since the memory was taken). Then that memory block could be
stuffed into something else.

So to summarize:

introduce basic image io interface to the python module
rewrite the tests in terms of this interface to drop dependencies
trains simple predictor model for the test to drop dependency on the heavy *.dat

with that we can close the PR.
further steps:

evaluate options in pybind to bind array_t to matrix without a copy
adapt examples to use basic image io but leave/create one example to show possible interoperability with opencv, skimage or pillow.

I just have to add, that i won't be able to work on this before the weekend. So in case you want to hijack this in the meantime @visionworkz feel free.

@supervacuus I never intended to hijack this thread :-) I know you were the person who wrote the pybind11 stuff and that's why I mentioned you, so that you could eyeball the new stuff better. I appreciate your contributions!
@davisking I did some further research and here is what I found:
How to bind and return matrix<rgb_pixel> directly:
#include <pybind11/numpy.h>
matrix<rgb_pixel> get_face_chip (
    py::object img,
    const full_object_detection& face,
    size_t size = 150,
    float padding = 0.25
)
{
    if (!is_rgb_python_image(img))
        throw dlib::error("Unsupported image type, must be RGB image.");

    matrix<rgb_pixel> chip;
    extract_image_chip(numpy_rgb_image(img), get_face_chip_details(face, size, padding), chip);
    return chip;
}

and
    typedef matrix<rgb_pixel> type;
    py::class_<type>(m, "image", py::buffer_protocol())
   .def_buffer([](type &m) -> py::buffer_info {
        return py::buffer_info(
            image_data(m),                                      /* Pointer to buffer */
            sizeof(npy_uint8),                                  /* Size of one scalar */
            py::format_descriptor<npy_uint8>::format(),         /* Python struct-style format descriptor */
            3,                                                  /* Number of dimensions */
            { num_rows(m), num_columns(m), 3UL },               /* Buffer dimensions */
            { sizeof(npy_uint8) * num_columns(m) * 3,           /* Strides (in bytes) for each index */
              sizeof(npy_uint8) * 3,
              sizeof(npy_uint8) }
        );
    });

This works, and in Python, you would show the image as follows:
# Get the aligned face image
dlib_image = dlib.get_face_chip(img, face, size=320)
count = sys.getrefcount(dlib_image)
assert count == 2, "Memory leak detected in get_face_chip"
image = np.array(dlib_image, copy=False) # This step is necessary as dlib_image is of type dlib.image
count = sys.getrefcount(dlib_image)
assert count == 3, "Unexpected ref count for dlib image"
count = sys.getrefcount(image)
assert count == 2, "Unexpected ref count for numpy array"
window = dlib.image_window()
window.set_image(image)
window.wait_until_closed()

We have to initialize the array with np.array(dlib.get_face_chip(img, face, size=320), copy=False) which is non-ideal. So I prefer @supervacuus approach, unless we can find something else.
One more thing. Since, we are returning a numpy array, we know the dimensions will be H x W x 3. I did not get why we need to use sizeof(rgb_pixel). Shouldn't the code be:
pybind11::array_t<uint8_t> create_array_from_rgb_image(const matrix<rgb_pixel> &rgb_image)
{
    const size_t dtype_size = sizeof(uint8_t);
    const auto rows = static_cast<const size_t>(num_rows(rgb_image));
    const auto cols = static_cast<const size_t>(num_columns(rgb_image));
    const size_t channels = 3;
    const size_t image_size = dtype_size * rows * cols * channels;

    auto arr = new uint8_t[image_size];
    memcpy(arr, image_data(rgb_image), image_size);

    return pybind11::array_t<uint8_t>(
        {rows, cols, channels},                                                     // shape
        {dtype_size * cols * channels, dtype_size * channels, dtype_size},          // strides
        arr,                                                                        // pointer
        pybind11::capsule{
            arr, [](void *arr_p) {
                delete[] reinterpret_cast<uint8_t *>(arr_p);
            }
        }
    );
}

With this, the Python code becomes dead simple:
# Get the aligned face image
image = dlib.get_face_chip(img, face, size=320)
count = sys.getrefcount(image)
assert count == 2, "Memory leak detected in get_face_chip"
window = dlib.image_window()
window.set_image(image)
window.wait_until_closed()

@visionworkz "hijacking" was meant jokingly... just wanted to make sure that you know that i am not married to the code :-)

@davisking and @supervacuus. I have consolidated the changes discussed here in PR https://github.com/davisking/dlib/pull/1258

@davisking I think you can close this then?

Yeah, that makes sense.  Thanks for working on this with @visionworkz. The overall API is now a lot nicer :)


Create MakeItSmall

NB : github automatically added my previous pull request "extraplanetory launchpads" to this pull request, why? Don't know how to remove it from this pull request

It's included in the PR because you had that commit in your master. For individual PRs you want to use individual branches.

Thanx, should i have to make another pull request or this one is ok for now?

This isn't ideal, but also not important enough to make a fuss about.

Thanx, the next time i contribute to this repo, i will do it better. This is really great to have this tool, CKAN is the best!

Noted, you push it? I push it? Is there any form of list of metadata i can use in a .ckan?

I can't push changes into your PR because it's your local branch. Metadata fields are listed in the spec: https://github.com/KSP-CKAN/CKAN/blob/master/Spec.md

Ok then, i re-do it

Sorry, i just tried to make a new branch and did it but... the launchpad plugin show up again, i'm very new to git, can you do it for me?
https://github.com/labo-css/CKAN-meta/blob/labo-css-makeitsmall/MakeItSmall/MakeItSmall-1.1.ckan
I just don't understand where i fuckup

I found git via github really difficult. I had a much easier time using smartgit (http://www.syntevo.com/smartgit/), it just made more sense to me.
If you're using the github web UI though, the easiest way to submit new files is probably to do it via the KSP-CKAN-owned repo. So I'll give a few screenshots of me submitting to your fork -- when you submit to our repo it'll be the same except you'll be at https://github.com/KSP-CKAN/CKAN-meta/ instead of https://github.com/labo-css/CKAN-meta/

Hit the "new file" button http://puu.sh/orsaW/593c24b637.png http://puu.sh/orsev/79339091fc.png
Enter the file name and info http://puu.sh/orska/4f1958c134.png
Give a description of what the change is and then his "Propose New File" http://puu.sh/orsmU/0c1dcbce59.png
Click "Create Pull Request" http://puu.sh/orsot/e4e1b2f545.png
Add any additional info then hit "Create Pull Request" again http://puu.sh/orsr2/03ce76f695.png

Lastly I'm going to ping @politas since I believe he uses the GitHub web UI and might have extra info or corrections to make to the info I provided.

I use a mix of Github web interface and a local git clone on my computer.
If you commit a change to your master branch, that's always going to cause issues for you. Ideally, make a branch, add or modify files in that branch, then create a pull request from that branch. If you have two changes applied to your master and can't figure out how to return to being in sync with the upstream, the easiest way would be to generate a pull request from your master to the KSP-CKAN master, including both changes.
The trickier and less elegant way would be to make local copies of your changed files, delete your fork of the KSP-CKAN repo and create a fresh fork, then create individual branches for your changes as @plague006 mentioned above.
If you wish to use git locally, feel free to jump into #ckan IRC channel on espernet and ask me questions. I'm not an expert on git, yet, but I'm starting to get a hold of how it all works.

Thanx everybody.


fix firehose encryption logic

Summary
The logic to enable the Kinesis firehose encryption was flawed because it referred to "var.kinesis_firehose_encryption_key_arn" rather than "local.kinesis_firehose_encryption_key_arn" (which has additional logic). Switching from the var to local ensures that encryption is enabled if it is set to be enabled and the local key arn exists.
How did you test this change?
Tests performed: 1) ensure kinesis encryption was enabled with default settings 2) ensure kinesis encryption was enabled when a key arn was supplied and set to be enabled 3) ensure kinesis encryption was disabled when a key arn was supplied and set to be disabled
Issue
n/a

Closing PR in leu of more changes that need to be made. Will open a new one that include the other changes.


MIGRATE.md doesn't mention the transition from Collector to CollectorManager

Description
Spinoff from this downstream issue.
This API migration confused this user (understandably!), and I thought MIGRATE.md really should spell it out, but I looked quickly (at main/10.0 lucene/MIGRATE.md) and couldn't find any mention?
Let's fix this before 10.0.0 release!

Hmm is there some way to mark issues as blocker for release?  I want to make sure we address this for 10.0.0.

FWIW I plan on treating all issues that have their milestone set to 10.0 as needing discussion if we want to exclude them from 10.0.

FWIW I plan on treating all issues that have their milestone set to 10.0 as needing discussion if we want to exclude them from 10.0.

Awesome, thanks @jpountz.

Thanks @mikemccand for catching this! I've opened a PR to add it https://github.com/apache/lucene/pull/13238

is this for 10.0.0

is this for 10.0.0

This is for 10.0.0, yes, but is also helpful when upgrading to 9.x since these APIs are deprecated there (and removed in 10.0).
@zacharymorn -- I think we can close this now?  #13238 resolves this issue?

Addressed by #13238


Global search

Dear jarektkaczyk,
First i want to say, i really like your package!
But for my application i need a global search feature (search trough (all) models).
My temp fix for now is:
`
$users = User::search($id, [
'email',
'country.name',
'country.demonym',
'country.native_name'
])->get();
		$companies = Company::search($id, [
				'company_code',
				'statutory_name',
				'company_name',
				'display_name',
				'telephone_number',
				'email_address',
				'website',
				'btwnr',
				'kvknr',
				'taalcode'
			])->get();

		$addresses = Address::search($id, [
				'street',
				'number',
				'zip_code',
				'city',
				'direct_phone',
				'mobile_phone',
				'description',
	  		'country.name',
	  		'country.demonym',
	  		'country.native_name'
			])->get();

		$projects = Project::search($id, [
			'project_code',
			'naming_convention_free',
  		'Countries.name',
  		'Countries.demonym',
  		'Countries.native_name'
			])->get();

$app = app();
$results = $app->make('stdClass');
$results->users = $users;
$results->companies = $companies;
$results->addresses = $addresses;
$results->projects = $projects;`

But this by far not the best way to go is guess...
I want the results of all items displayed at relevance of the searchterm.
I hope you can help me out with this braincracker!

Have you managed to resolve this already?
In such case I would probably go with DB level relevance scoring using eg. levenshtein or better something more robust, like algolia, solr etc.
Joining all these tables might not be the best for perf.

Thanks for your reaction!
I tried elasticsearch before and a few others, but i find it a bit complex for what i am trying to achieve at the moment.
For now i am testing the new [https://laravel.com/docs/5.3/scout]


docs(documentation-website):add code factor rating to root readme

The Pull Request is ready

[x] fixes #757

please don't mix tasks


Create Schedule Task Error

I am creating a timing task error
Caught exception while handling a request: Failed to create schedule Failed to create schedule remain-cron
Resolved [org.springframework.cloud.scheduler.spi.core.CreateScheduleException: Failed to create schedule Failed to create schedule remain-cron]
why？

This is all logs
2018-12-13 00:29:29.736 INFO 1 --- [ main] o.s.c.d.s.k.KubernetesDataFlowServer : Started KubernetesDataFlowServer in 33.835 seconds (JVM running for 36.474)
2018-12-13 00:33:01.028 INFO 1 --- [nio-9393-exec-1] o.a.c.c.C.[Tomcat].[localhost].[/] : Initializing Spring FrameworkServlet 'dispatcherServlet'
2018-12-13 00:33:08.473 WARN 1 --- [nio-9393-exec-8] i.f.k.client.internal.VersionUsageUtils : The client is using resource type 'cronjobs' with unstable version 'v1beta1'
2018-12-13 00:33:17.805 INFO 1 --- [nio-9393-exec-1] o.s.c.d.r.s.DownloadingUrlResource : Downloading [http://<IP_ADDRESS>:8888/repository/analysis-release/com/hoolai/analysis/tasks/remain_task/1.0/remain_task-1.0.jar] to /tmp/spring-cloud-deployer3345023133535560099/9650af8ca87b3c2374be8a82ca8c9d7fd0aa10a8-remaintask10jar
2018-12-13 00:33:20.435 WARN 1 --- [nio-9393-exec-1] o.s.c.d.s.c.RestControllerAdvice : Caught exception while handling a request: Failed to create schedule Failed to create schedule remain-cron
2018-12-13 00:33:20.451 WARN 1 --- [nio-9393-exec-1] .m.m.a.ExceptionHandlerExceptionResolver : Resolved [org.springframework.cloud.scheduler.spi.core.CreateScheduleException: Failed to create schedule Failed to create schedule remain-cron]
2018-12-13 00:42:05.202 INFO 1 --- [nio-9393-exec-4] o.s.c.d.r.s.DownloadingUrlResource : Downloading [http://<IP_ADDRESS>:8888/repository/analysis-release/com/hoolai/analysis/tasks/remain_task/1.0/remain_task-1.0.jar] to /tmp/spring-cloud-deployer2068221042014573662/9650af8ca87b3c2374be8a82ca8c9d7fd0aa10a8-remaintask10jar
2018-12-13 00:42:06.726 WARN 1 --- [nio-9393-exec-4] o.s.c.d.s.c.RestControllerAdvice : Caught exception while handling a request: Failed to create schedule Failed to create schedule remain-cron
2018-12-13 00:42:06.728 WARN 1 --- [nio-9393-exec-4] .m.m.a.ExceptionHandlerExceptionResolver : Resolved [org.springframework.cloud.scheduler.spi.core.CreateScheduleException: Failed to create schedule Failed to create schedule remain-cron]
How to view other logs?

it would be helpful to know all the details here... what cron expression did you use? what can we do exactly step by step to reproduce you problem, etc?...

container.log
There are so many logs

only docker images are supported in kubernetes, looks like your pulling a jar:
DownloadingUrlResource : Downloading [http://<IP_ADDRESS>:8888/repository/analysis-release/com/hoolai/analysis/tasks/remain_task/1.0/remain_task-1.0.jar] to /tmp/spring-cloud-deployer3345023133535560099/9650af8ca87b3c2374be8a82ca8c9d7fd0aa10a8-remaintask10jar


[REG2.067a] Issue 13729 - One not detected case of not purity

https://issues.dlang.org/show_bug.cgi?id=13729

Auto-merge toggled on


Add checkstyle rules as part of the project for all sources

I am a fan of light code standard enforcement using static analysis tools like checkstyle.
The Agrona project shows how we can incorporate checkstyle into a gradle build.  I'm not sure I agree with all the rules that are enforced there (in particular bracing style).  An alternative that needs looking over is the nowucca checkstyle rules.  I think
The sevntu checkstyle extensions are interesting and a little under documented.  The Agrona project uses some of these.
This task to take Agrona gradle concepts, then the Agrona, nowucca and the sevntu checkstyle example rules  (as applied to themselves) and come up with a checkstyle file for this project.  That file should be applied as part of the gradle build.

If we do add a static analysis tool, we should check in the tool separately from the formatting changes that it drives, so we have a commit that "fails" a build with all the checks turned on.
This let's us have a clear separation of the tool from the formatting changes in the commit history.


Unable to use typings in a typescript node project

Getting this error:
Could not find a declaration file for module 'ollama-js-client'. '/Users/kristian/repos/personal/svelte-apps/agent-communication-hub/node_modules/ollama-js-client/dist/node/index.js' implicitly has an 'any' type.
  There are types at '/Users/kristian/repos/agent-communication-hub/node_modules/ollama-js-client/dist/node/types/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'ollama-js-client' library may need to update its package.json or typings.ts(7016)

Should be fixed in the latest version. Let me know if it works.

I've switched to the official ollama package in the meantime. Thanks anyways. Might well be useful in the future :)


Add versioning to the docs

As we start to add more features and release new versions, we should version the docs so that users can find docs for the particular version they are using.
Couple different ways to do it:
mkdocs with mike - https://github.com/squidfunk/mkdocs-material-example-versioning
mkdocs with readthedocs - https://docs.readthedocs.io/en/stable/versions.html

Works and includes a banner message if someone is browsing the main docs: https://ibm.github.io/argocd-vault-plugin/main/


grabbing single object from a fake pile?

Imagine having a 'pile' that looks like, say, 1000 objects, but is actually just 1 model. Now, I want to 'drag' a single item 'from' this pile, and drop it somewhere else.
So, when I start a drag event, I want to drag not the event I aim at, but a generated child object (since I don't want to have an actual thousand objects in my scene).
What's the best way to do something like this? Is this plausible in superhands, or would it require a workaround (or just an update)?

There are several ways you could approach this:

Have the pile transform into a single object when grabbed and then spawn a new pile in it's old spot
Have the pile be non-interactive, but keep a single object that stays invisible until grabbed in the same space, spawning a replacement when it is taken
Make a custom reaction component for the pile that listens for grab events, spawns a single-item grabbable child, and then forwards the original grab-start as well as the next grab-end event to that new child

Thanks!


Problem with Keyboard char in Terminal

@dobin I'm using a french AZERTY keyboard and to type @ or [, }, \, I will use a combinaison keys like  Alt Gr + 8 for \ ,  Alt Gr + 0 for @ etc...
In the Web Terminal, these keys do not work. I'm not able to type an @ or #.
It's a problem to do an SSH connection like<EMAIL_ADDRESS>Could you solve it please. Thank you.

@saldru can you try on aarnaud.github.io/lxd-webui because i don't reproduce.

@aarnaud It's working fine with your web service but not with @dobin web gui.

Strange. I'll investigate.

Could not reproduce.
I changed keyboard layout in ubuntu to "french", and i was able to insert the @[.
Can you tell me your OS + Browser?

Could not reproduce, no feedback.


Add CODE_OF_CONDUCT.md

Hey @Satakshijain assign this issue to me and I will add a code of conduct and also add hacktoberfest tag

hey Satakshi please assign this issue to me :)

i am done with it. kindly merge it
On Wed, Oct 26, 2022 at 3:29 PM Satakshi Jain @.***>
wrote:

Assigned #19 https://github.com/Satakshijain/Coding-Hustlers/issues/19
to @anjaligoswami https://github.com/anjaligoswami.
—
Reply to this email directly, view it on GitHub
https://github.com/Satakshijain/Coding-Hustlers/issues/19#event-7671230387,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASAHUD4BMMQR2I3KI4GLKTDWFD6J3ANCNFSM6AAAAAARLZWWXI
.
You are receiving this because you were assigned.Message ID:
@.***>


Design for 'Create Organization on the Fly' instance

🎨 See latest design on figma
There are a couple of places in the platform, generally inside a flow (e.g., Add funds, start accepting financial contributions) where you are prompted to create an organization 'on the fly.'
See #2713 and https://github.com/opencollective/opencollective-frontend/pull/3247
In the current create collective flow redesign its needed when selecting your organization as funds holder. So we are revamping that instance platform-wide.
Status: Design ready for implementation
@Betree  @alanna

Finished design already on implementation
I think @sbinlondon it's going to implement this during the 'Start accepting financial contributions flow' @Betree

@sbinlondon can you estimate the complexity and add the corresponding label? I think you can re-use most of the things you did for CCF, for example the component to add admins?

Closing this because it's being implemented here #3468


bug: linea_getProof / SparseMerkleProof.sol

Description
linea_getProof returns wrong values for storage values that don't exist in the trie.
Steps to Reproduce (Bug)
Here is a simple demo, view-source to see what it's doing exactly, but it does account and storage proof verification for a single slot.
The default contract is USDC but you can change it with query paramater address.
For slot 0xcb0cbc8493baf4a7b1972914ba0be89040e56e4a3c98d60268fe37b8c8e546d8 (the default): https://raffy.antistupid.com/eth/linea-proof-bug.html
2024-09-21T22:58:30.386Z accountProof is valid: true
2024-09-21T22:58:30.555Z leftStorageProof is valid: false
2024-09-21T22:58:30.670Z rightStorageProof is valid: false

For slot 0, which is owner and exists: https://raffy.antistupid.com/eth/linea-proof-bug.html?slot=0
2024-09-21T22:59:18.777Z accountProof is valid: true
2024-09-21T22:59:18.976Z storageProof is valid: true

For slot 49, which is also and does not exist: https://raffy.antistupid.com/eth/linea-proof-bug.html?slot=49
2024-09-21T23:02:21.272Z accountProof is valid: true
2024-09-21T23:02:21.527Z leftStorageProof is valid: true
2024-09-21T23:02:21.625Z rightStorageProof is valid: false

we did a fix in the last release. could you try again ?

Yes, I believe that fixed the issue AFAICT.  I'll get back to you with further analysis.  Thank you!


Bump resolver and libraries

Bump resolver to latest in lts-18 series and bump dependencies on the unbeliever packages to the latest released as well (no longer needs to depend on unreleased core-program)

Thanks much!


boards: merge repo. boards as folder

Contribution description
First step in the groupping repositories task
Testing procedure
make should work with m4a boards:
make BOARD=m4a-mb

or
make BOARD=m4a-24g

should build the firmware
Issues/PRs references
#69

I see some warnings, from a group of files in example folder...

squash to merge soon :+1:

squash to merge soon fireworks

Today should be ready to merge


device_class: running in some entities

Hi!! I found some entities of this component with the device_class set to 'running'.
I think this kind of device class is designed for process or threads that can being executing or not (running or not running). For example a Home Assistant addon is running, but a switch or a pump is not running.
For example the 'Aquarea Main Power' switch has the device_class 'running', but a switch musnt'n have this device_class: https://www.home-assistant.io/integrations/switch/#device-class

Hello,
I understand what you mean. Indeed the running device_class is not explicitly mentioned as possible device class by the documentation. Do you some some (visual or behavioral) downsides with the current setup?
I have considered switch entities as natural extension of  binary sensors (with the addition that you can control it) so it made sense.
Regarding the semantic, I am not a native speaker, however a "running" pump feels ok to me. I'll research to see if there is something more appropriate.

@kamaradclimber thanks por your answer. Yes, my problem is that i have an automation that alert me when a software instance is not running. The automation inspects the entities which device class is running.
It has been stranger to get an alert because my pump is not "running"... So i've decided to open this issue. In my opinion, the pump is not working or the pump is powered off. Device class "running" is being used for addons or software instances, but not for a machine.
I can exclude heishamon entities from my automation, but i thought it deserves to discuss this theory in an issue. I could be wrong.
Thanks!

Thanks for the context.
I looked at other device_class without finding anything more appropriate. I also checked the semantics, it makes sense to say a heatpump is "running" (or any mechanical process apparently).
I understand it makes your automation a bit more complex (and kudos for the automated discovery of addons).
For the time being I'll keep the code as-is. We can of course reconsider the discussion in the future.

@kamaradclimber it's ok. No problem for my automation.
Only for curiosity. For my understanding. What are the advantages of consider a power switch as a "running" entity? Consider that no other 'switch' type entity is. Or at least I had never seen it.


stix2 3.0.1 ❄

stix2 3.0.1 ❄
Destination channel: Snowflake
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_c1nhpzvid7/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:65: missing_description: The recipe is missing a description
clone/recipe/meta.yaml:21: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:65: missing_license_file: The recipe is missing the about/license_file or about/license_url key.
clone/recipe/meta.yaml:64: documentation_specifies_language: Use the generic link, not a language specific one

===== ERRORS =====

clone/recipe/meta.yaml:60: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, closest match: 0BSD
===== Final Report: =====
1 Error and 7 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_d6h7f_wgbn/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:62: missing_description: The recipe is missing a description
clone/recipe/meta.yaml:61: documentation_specifies_language: Use the generic link, not a language specific one
clone/recipe/meta.yaml:62: missing_license_file: The recipe is missing the about/license_file or about/license_url key.

===== ERRORS =====

clone/recipe/meta.yaml:57: incorrect_license: The recipe's about/license key is not an SPDX compliant license or license exception, closest match: 0BSD
===== Final Report: =====
1 Error and 3 Warnings were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_1flp4ifl8x/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:59: documentation_specifies_language: Use the generic link, not a language specific one
===== Final Report: =====
0 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_1425w8gou0/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:59: documentation_specifies_language: Use the generic link, not a language specific one
===== Final Report: =====
0 Errors and 1 Warning were found


Update dashboard Kubernetes / Views / Pods ( 33 → 34 )

This PR contains the following updates:

Package
Update
Change

Kubernetes / Views / Pods
major
33 -> 34

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Deserialization error: Wrong directory fields

2017-01-15 02:41:20,537:INFO:requests.packages.urllib3.connectionpool:756: Starting new HTTPS connection (1): acme-v01.api.letsencrypt.org
Traceback (most recent call last):
  File "build/bdist.linux-x86_64/egg/simp_le.py", line 1401, in main
    return main_with_exceptions(cli_args)
  File "build/bdist.linux-x86_64/egg/simp_le.py", line 1386, in main_with_exceptions
    persist_new_data(args, existing_data)
  File "build/bdist.linux-x86_64/egg/simp_le.py", line 1282, in persist_new_data
    client = registered_client(args, existing_data.account_key)
  File "build/bdist.linux-x86_64/egg/simp_le.py", line 1224, in registered_client
    client = acme_client.Client(directory=args.server, key=key, net=net)
  File "build/bdist.linux-x86_64/egg/acme/client.py", line 63, in __init__
    self.net.get(directory).json())
  File "build/bdist.linux-x86_64/egg/acme/messages.py", line 169, in from_json
    raise jose.DeserializationError(str(error))
DeserializationError: Deserialization error: Wrong directory fields

Unhandled error has happened, traceback is above

Looks like this was an error from an old version. I've updated my containers and everything seems fine now.


Corrected all errors

Corrected all error told by you everything is working perfectly fine. 405 error is also gone.... using gmail I have checked for many times now 405 is not there but couldn't try using Zoho since I don't have that account and it is not free to use.

Summary by CodeRabbit

New Features

Unified success messages across different generators for a consistent user experience.

Refactor

Updated email authentication credentials to enhance security and maintain environmental variable standards.
Refined API response handling to prioritize success messages and streamline user feedback.

Bug Fixes

Adjusted HTTP status code responses for confirmation processes to ensure accurate communication with the client.

What I think is due to missing return statements mail was encountering 405 error otherwise it will not, you may try using your zoho id using by locating that in env file with same value as given in code @ank1raj

@singhcoder694 No, it's still not fixed. let's sit together tomorrow and work on this and fix this asap.

Ok sir, I fixed it properly but if not we can meet to fix it.... but how can we meet? @ank1traj


SDSTOR-10986 btree leaf node pagination and range query

Testing:
Basic testing passed with 2 leaf nodes;
Load (with more leaf nodes) testing is being added and tested...
Command example:
Pagination:
curl -X GET "http://<IP_ADDRESS>:5000/api/v1/getStatus?name=btree_node_test_files/vol0&type=btree_node"
Range_query:
curl -X GET "http://<IP_ADDRESS>:5000/api/v1/getStatus?name=btree_node_test_files/vol0&type=btree_node&next_cursor=10995116277760" | jq .

Codecov Report

Merging #118 (39a985e) into stable/v3.x (15dcf46) will increase coverage by 0.54%.
The diff coverage is 6.97%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@               Coverage Diff               @@
##           stable/v3.x     #118      +/-   ##
===============================================
+ Coverage        45.00%   45.55%   +0.54%
===============================================
  Files              262      126     -136
  Lines            33720    16225   -17495
  Branches          3890     1925    -1965
===============================================
- Hits             15176     7391    -7785
+ Misses           17115     8127    -8988
+ Partials          1429      707     -722

Impacted Files
Coverage Δ

src/engine/homeds/btree/btree.hpp
42.96% <2.59%> (-2.22%)
:arrow_down:

src/engine/index/indx_mgr.cpp
26.58% <40.00%> (+0.07%)
:arrow_up:

src/homeblks/volume/mapping.cpp
32.07% <50.00%> (ø)

... and 141 files with indirect coverage changes


Update generating-a-new-ssh-key-and-adding-it-to-the-ssh-agent.md

Why:
Fix some typos regarding what is actually shown in the terminal, and also fix a command which previously didn't work.
"ssh-add c:\Users\YOU.ssh\id_ed25519" (with backslashes) leads to error "c:UsersYOU.sshid_ed25519: No such file or directory".
Notes:
I had to run "eval `ssh-agent -s`" in bash before being able to run ssh-add (link), should maybe add this to the docs.
Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@charleskimbac Thanks so much for opening a PR! I'll get this triaged for review ✨

Hi @charleskimbac, thanks so much for opening this. This is interesting because we've only just changed the direction of the slashes based on previous feedback.
It was my understanding that the direction of the slashes wouldn't actually matter to Powershell, which can cope with both. To confirm this, I created a fresh Windows 11 VM to test. It looks like ssh-add really doesn't care how you provide the slashes:
PS C:\Users\adminadmin> ssh-add c:/Users/adminadmin/.ssh/id_ed25519
Identity added: c:/Users/adminadmin/.ssh/id_ed25519<EMAIL_ADDRESS>
PS C:\Users\adminadmin> ssh-add c:/Users/adminadmin\.ssh\id_ed25519
Identity added: c:/Users/adminadmin\.ssh\id_ed25519<EMAIL_ADDRESS>
PS C:\Users\adminadmin> ssh-add c:\Users\adminadmin\.ssh\id_ed25519
Identity added: c:\Users\adminadmin\.ssh\id_ed25519<EMAIL_ADDRESS>
The same is true with ssh-keygen.
I'm curious if the errors you're seeing are due to a different version of ssh-add (for example, MinGW).
We can only really support a default installation but, as both slash-directions work on a default install, and it looks like switching back to forward slashes will help some folks such as yourself, let's do that!
I'll approve this and get your changes merged in. Thank you! ✨✨✨✨

I had to run "eval ssh-agent -s" in bash before being able to run ssh-add (link), should maybe add this to the docs.

Did you run Get-Service -Name ssh-agent | Set-Service -StartupType Manual and Start-Service ssh-agent first? As I had the fresh VM, I tested this too, and was only able to use ssh-add once I'd ran those two commands from our docs. As I've confirmed this is working as expected with a default installation, I think we should stick to our current procedure.

Hey @stevecat so I want to start by saying that I only have a bare knowledge of all of this, so apologies if I say something or am saying something incorrectly.
In-line with step 2 of "Adding your SSH key to the ssh-agent", it starts by saying "In a terminal window without elevated permissions..." so I went back (from powershell) to my bash terminal (mingw) in VSCode, and attempted ssh-add. I guess you don't need the "eval ssh-agent -s" if you do it in Powershell vs bash, and it also explains the forward/back-slash discrepancy.

Ooh, thanks @charleskimbac! I'm experimenting with this and it appears that running the ssh-add command in Powershell might set things up so the key works in Powershell and Git Bash. This surprised me tbh!
Launching ssh-agent in Git Bash (using the eval command) does then allow you to run ssh-add as you mentioned.
I'm currently working on a project to revamp our SSH docs, and am paying extra attention to the Windows procedures. I'll work with our team to decide on the best option for that second step; either sticking with Powershell or adding the eval command.