[AIRFLOW-3355] Fix BigQueryCursor.execute to work with Python3

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-3355
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

BigQueryCursor.execute uses dict.iteritems internally,
so it fails with Python3 if binding parameters are
provided. This PR fixes this problem.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

tests.contrib.hooks.test_bigquery_hook:TestBigQueryCursor
Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes flake8

CI failures occur at test_scheduler_sla_miss_callback_exception and test_scheduler_sla_miss_email_exception and are irrelevant to this fix.

Codecov Report

Merging #4198 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #4198      +/-   ##
=========================================
+ Coverage   77.69%   77.7%   +<.01%
=========================================
  Files         199     199
  Lines       16309   16309
=========================================
+ Hits        12672   12673       +1
+ Misses       3637    3636       -1

Impacted Files
Coverage Δ

airflow/models.py
92.37% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


Sensors attributes in table markdown card

Two days ago, after update from 2022.5.5. to 2022.6.6 all my trakt's markdown cards not working. I am not sure if the issue is related to HA or Trakt integration. Sensors working as before. Could anybody, please, test this code?
    type: markdown
    content: |-

          Show

          Episode
          Date

      {% for item in state_attr('sensor.trakt_upcoming_shows', 'data') %}
      {% if item.airdate %}

          {{item.title}}
          {{item.number[1:6]|regex_replace(find='E', replace='x', ignorecase=False)}}
          - {{item.episode}}
          {{as_timestamp(item.airdate) | timestamp_custom('%d.%b')}}

      {% endif %}
      {% endfor %}

Thank you!

Hello, I don't think it is related to my card.

Hi,
ok, I thought. Could you please, test that code, if it is working with any of your trakt sensor? I am trying to find if it is problem in my HA or any HA.

In case of someone in future will have same problem. After few evenings and some help. Markdown card works only if I delete few lines in attributes:
{'title_default': '$title', 'line1_default': '$episode', 'line2_default': '$release', 'line3_default': '$rating - $runtime', 'line4_default': '$number - $studio', 'icon': 'mdi:arrow-down-bold'}


Create README.md

Copy form Google Play Store and add back old description, for respect the Brevent's work.
In this issue:

In previous, @RikkaW refused to mention Brevent and said it would violate play policy. But RikkaW violate play policy to show third party payment in appops.

I think Shizuku should do something to avoid the Google Play policy and mention the contributors. For example, use a link to mention them.

Won't fix


Retry middleware logic for network errors

In the Retry middleware, the default function validateRetry catches all requests that returns a 500 response from the server.
It would be nice to also retry the request if some client error occurs (one of ECONNRESET, ENOTFOUND, ESOCKETTIMEDOUT, ETIMEDOUT, ECONNREFUSED, EHOSTUNREACH, EPIPE, EAI_AGAIN), when there was no response from the server.
At the moment, the response object from such an error is a status 400, Network error. If the error code could be attached to the response object somehow, it would be easier to add the logic needed to modify the validateRetry option. Also, it might be worth considering adding these to the default retry function.

Take a look at response#error, fixed in version 2.22.0


[Milestone] v2

Here's what I would love to see for a v2 of this package
History: https://www.npmjs.com/package/node-navigator?activeTab=versions
Current state:
module.exports = {
    Navigator,
    NavigatorConcurrentHardware,
    NavigatorID,
    NavigatorLanguage,
    NavigatorOnLine,
    NavigatorGeolocation
};

Things to do for v2:

[ ] #11
[x] #1
[ ] #12
[ ] #13
[x] #3
[ ] #14
[ ] Expose NavigatorGeolocation mixin as a separate package
[ ] #10
[ ] Improve readme & package metadata
[ ] #15

For the extension mixins, I'm imagining that this package is the "html-system-state" package (implements all things from https://html.spec.whatwg.org/multipage/system-state.html) and there's other packages that implement things from https://storage.spec.whatwg.org/ ("storage") and https://www.w3.org/TR/geolocation/ ("geolocation") etc.
I could see:

This package has an additional packages/ folder with many sub-projects in it pertaining to each of those "geolocation" "storage", etc. navigator-related implementations
Recommend other polyfills like a fictitious jcbhmr/geolocation (that I have yet to create) or something

For the documentation website, I'd love to see something that describes how each of the more complicated things gets its information. For instance, saying ".hardwareConcurrency uses os.cpus().length" or something is a 👍 from me! This can be generated from JSDoc-like /** */ comments using TypeDoc: https://typedoc.org/
I'd also like to see a docs/example/ folder that uses StackBlitz https://stackblitz.com/ to insta-launch a demo project that a potential user can use as a playground.
@skdhg Any other improvements you think could be made? Looking for ideas and feedback on my ideas

Don't worry about existing consumers! That's what major versions are for, remember! 😉
{
  "dependencies": {
    // Will NOT auto-install v2. Stays on v1.0.1!
    "node-navigator": "^1.0.1"
    //                 👆
    // This is the default when 'npm install node-navigator' is run:
    // The "^" caret range is vN.x.x
  }
}

@jcbhmr I've sent you the transfer request of this repository, please check your email. I have also invited you to npm registry.

@skdhg Awesome! ❤ I have accepted that transfer and the npm thing. 👍


Inv no work sometimes

Good job, but something wrong.
I'v tryed invers of:
M138.1463,-141.438C120.1315,-128.3117 100.4597,-113.1211 83.4474,-98 72.8076,-88.543 61.8617,-76.9712 53.3179,-67.4481
Give:
M83.4474,-98 C100.4597,-113.1211 120.1315,-128.3117 138.1463,-141.438
But supposed to be:
M53.3179,-67.4481 C61.8617,-76.9712 72.8076,-88.543 83.4474,-98 100.4597,-113.1211 120.1315,-128.3117 138.1463,-141.438

And it
M9490 180c-328 0-458-147-816-124-287 19-549 230-1017 248-501 18-780-179-1129-170-308 7-489 115-703 128-247 15-417-93-651-92s-393 69-667 69c-509 0-735-166-1274-166-788 0-1304 300-1879 300S748 180 0 180
result to:
M748,180 SNaN,300 NaN,NaN c69,-509 -393,NaN NaN,NaN sNaN,NaN NaN,NaN cNaN,NaN -328,NaN 9490,180
https://embed.plnkr.co/plunk/rIhZfI


Feature xxx

Contribution guidelines

[X] I've read the contribution guidelines and wholeheartedly agree

I want to suggest an idea and checked that ...

[X] ... to my best knowledge, my idea wouldn't break something for other users
[ ] ... the documentation does not mention anything about my idea
[ ] ... there are no open or closed issues that are related to my idea

Description
-- 查看建库语句
show create database A
-- 删除数据库
drop database A

能支持以上操作就更完美了
Use Cases
-- 查看建库语句
show create database A
-- 删除数据库
drop database A
Screenshots / Mockups
No response

Message that will be displayed on users' first issue


really fix install_t in containers

I'm not sure if something changed here, or my code never actually worked but basically we currently fail to bootc install with unknown types.

I hit this bug while trying to create a bootc image based on Arch Linux
It seems that it can be worked around by disabling SELinux

so we tried this on a fedora host with SELinux disabled and --disable-selinux flag.
This allowed us to skip the install_t issue when creating an bootc image from arch linux (they don't ship selinux policies so everything is unlabeled). However we ran into some problems.
We realized matchpathcon and chcon are both used even when selinux is disabled at the kernel level and --disable-selinux has been passed so the target installed system should not necessarily have labels (though I guess it is nice to have them in case the user wants to turn selinux on later, it seems the idea behind --disable-selinux is to support non-selinux enabled system)
We were able to patch this by simply overwriting the binaries, matchpathcon was hardcoded to output a specific selinux context and chcon was overwritten to be equal to /bin/true so both commands would succeed unconditionally. (we're trying not to touch bootc itself for now, just trying to make it work on a non-fedora system as proof of concept)
is this behaviour expected?

Thanks for the report; any chance you could test out https://github.com/containers/bootc/pull/96 ?

This should be fixed by https://github.com/containers/bootc/pull/284


My Calendar View Didn't Expand its Height When That Month Needs An Extra Row [Help Needed]

When I switch from July to August, it didn't add an extra row which it suppose to. However, in the demo app, it behaves normally. I wonder where I made a mistake for that. Please give me some hint and I will really appreciate.
Here is how I write the code. I have the override viewDidLayoutSubviews() of course. I didn't use storyboard at all. Everything I created is programmatically. The way I create the calendarView is I give it a frame size and init it using CVCalendarView(frame: frame) and assign the calendarDelegate.
I have tried to solve this bug for a long time. Any help will be appreciated. Thank you so much for your time.

The way it resizes you're calendar is based off the height constraints. You need to set a height constraint on your CalendarView, and it should fix the problem

I really appreciate your hint. However, I didn't solve the problem . I am sorry that I have to bother you again.
I fixed my code and add constraints instead of giving the calendarView a fixed height. But that still didn't work. It doesn't affect anything at all.  Here is how I init the calendarView
func calendarViewSetup () {
    self.mainCalendarView = CVCalendarView()
    self.mainCalendarView.setTranslatesAutoresizingMaskIntoConstraints(false)

    self.mainCalendarView.calendarDelegate = calendarDelegate
    self.mainCalendarView.backgroundColor = UIColor.whiteColor()
    self.addSubview(self.mainCalendarView)

    // add constraint
    let horizontalConstraint = NSLayoutConstraint(
      item: self.mainCalendarView,
      attribute: NSLayoutAttribute.CenterX,
      relatedBy: NSLayoutRelation.Equal,
      toItem: self,
      attribute: NSLayoutAttribute.CenterX,
      multiplier: 1,
      constant: 0)

    let verticalConstraint = NSLayoutConstraint(
      item: self.mainCalendarView,
      attribute: NSLayoutAttribute.CenterY,
      relatedBy: NSLayoutRelation.Equal,
      toItem: self,
      attribute: NSLayoutAttribute.CenterY,
      multiplier: 1,
      constant: 0)

    let widthConstrant = NSLayoutConstraint(
      item: self.mainCalendarView,
      attribute: NSLayoutAttribute.Width,
      relatedBy: NSLayoutRelation.Equal,
      toItem: nil,
      attribute: NSLayoutAttribute.NotAnAttribute,
      multiplier: 1,
      constant: calendarView_width)

    let heightConstrant = NSLayoutConstraint(
      item: self.mainCalendarView,
      attribute: NSLayoutAttribute.Height,
      relatedBy: NSLayoutRelation.Equal,
      toItem: nil,
      attribute: NSLayoutAttribute.NotAnAttribute,
      multiplier: 1,
      constant: calendarView_height)

    self.addConstraints([horizontalConstraint, verticalConstraint, widthConstrant, heightConstrant])
  }

Also in ViewController, I have
  override func viewDidLayoutSubviews() {
    super.viewDidLayoutSubviews()

    // calView is the whole view
    // mainCalendarView is the CVCalendarView
    self.calView.mainCalendarView.commitCalendarViewUpdate()
    self.calView.mainCalendarView.changeMode(.MonthView)
    self.calView.calendarLabelView.commitMenuViewUpdate()
  }

Thank you very much for your time. Looking forward to your reply.

Hello @shan199105,
Please check out  develop branch along with issued #101 #104. It seems the height calculation doesn't work properly for your case but a newer version should help.
If it doesn't please let me know!
Regards,
Eugene

Thanks Problem Solved


macvlans example rendering broken

Describe the bug

To Reproduce
Steps to reproduce the behavior:
https://nixos.org/nixos/options.html#macvlans
Expected behavior
show the actual example
https://github.com/NixOS/nixpkgs/blob/54ad1864617eb2a66b762e67d9a6ea06641bcbbf/nixos/modules/tasks/network-interfaces.nix#L690-L694

literalExample needs a string - see other occurences of it.
This seems quite an easy fix - can you file a PR for it?

I can't reproduce this on 20.09 docs and master. Probably has been fixed by 2e4a45c92171ad1712dcdcebffcd178f794c7d7f already.


feat(CreateEdit): Navigation highlight follows user's scroll #155

In the future, please rebase -i to clean up local commits (like "pushing for clark to test") and rebase your feature branches against develop (as opposed to merging them) so that other people's commits don't show up in your PRs.

this works, but it's weird how you can't scroll to 'Owner Information & Contacts'. Maybe add some whitespace to the bottom of the createEdit form?

@sdeighan added some whitespace to fix the issue but on larger displays the issue is still apparent, perhaps in the future we can add more space to the bottom or something else


Warc sidecar cdxj

This is only half of what is needed to complete issue 5. Here we have a script that will turn each record from the sidecar payload into one line in a cdxj file. We also include the surt and timestamp for future reference.

@ldko @somexpert This is ready for review.

@ldko @somexpert This is ready for review again. The logic did not change in the script. I just extracted the parts out as Lauren mentioned to do so.


Add badges

Travis, Coveralls, license!

Done with 23, coveralls can wait until not 10pm at night


Clicking on a component in the DepMap triggers an animation that moves the diagram out of the screen.

The behavior is not consistently reproducible, only seems to happen right after appmap file is opened. If the viewer has a vertical scroll bar (lots of events), the diagram's animation will have it scroll down out of the screen.
Likely related to issue https://github.com/applandinc/vscode-appland/issues/91

Fixed by https://github.com/applandinc/appmap-js/pull/49

Fixed by https://github.com/applandinc/appmap-js/pull/49


Document why Decoder::new might return Err

Migrating from flate2 to the zstd crate, it was surprising to me to find that zstd::stream::read::Decoder::new returns an io::Result:
https://github.com/gyscos/zstd-rs/blob/e470f0021d25cd5f79431acba0b7456301c00b82/src/stream/read/mod.rs#L24-L31
Compare this to flate2::read::GzDecoder::new, which returns the decoder directly:
https://docs.rs/flate2/latest/flate2/read/struct.GzDecoder.html#method.new
I was hoping the rustdocs for zstd::stream::read::Decoder::new might explain why it is fallible, so I could determine whether specific user facing error handling was required for this failure mode, or whether it was unlikely to occur in practice.
From inspection, it seems the only two fallible calls during the creation of the decoder are here:
https://github.com/gyscos/zstd-rs/blob/e470f0021d25cd5f79431acba0b7456301c00b82/src/stream/raw.rs#L147-L150
...which are:

Initialising the decompression context using zstd_sys::ZSTD_initDStream
Loading the dictionary using zstd_sys::ZSTD_DCtx_loadDictionary

Checking the zstd source for ZSTD_initDStream I see:
https://github.com/facebook/zstd/blob/ff7a151f2e6c009b657d9f798c2d9962b0e3feb5/lib/decompress/zstd_decompress.c#L1747-L1754
...where the code comment suggests it can't fail?
Similarly, checking ZSTD_DCtx_loadDictionary I see the fallible paths appear to only be encountered if trying to set the dictionary when the context is in the wrong state (which can't happen unless zstd-safe has a bug), or when a non-empty dictionary is provided (and Decoder::new sets an empty dictionary):
https://github.com/facebook/zstd/blob/ff7a151f2e6c009b657d9f798c2d9962b0e3feb5/lib/decompress/zstd_decompress.c#L1697-L1711
...so unless I'm missing something it appears Decoder::new() can never fail in practice?

Hi, and thanks for the report!
Indeed, it looks like this should never return an error in practice.


Actuator proxying

The SSH actuators should have this. Perhaps modify the config to have the following:
{
  "tunnels": [
    {
      "type": "reverse",
      "srcport": 5671,
      "dsthost": "<IP_ADDRESS>",
      "dstport": 5671
    }
  ]
}

MINA SSHD would create those tunnels upon connection. The user can specify custom AMQP/Transfer URIs in the resource configuration as normal, except this time they'd reference the head node.

Needed for Nord III


Allow bypassing graph rescaling

When resizing the container the graph seems to be squashed to fit to the smaller viewport size. What I would expect is the world coordinates remain the same and simply we are looking through a smaller viewport.
From what I have gathered from my searches on the web there used to be a field 'scalingMode' and when set to  'outside' it alleviated this problem. However, this field does not seem to exist on the newer versions.
Edit for clarity:
I would like to suggest a bypass flag to turn off graph rescaling to frame. The behaviour would allow the graph size to remain constant regardless of changes to the viewport size. The disadvantage would be that it would require more work when the graph first loads to ensure something is on screen.

The scalingMode: outside refers to a very old version of sigma.
The way sigma behaves now is the following (it is further described in the Coordinate Systems page in the doc):

It normalizes the node positions into a [0,1] x [0,1] square (this coordinates system is called framdGraph within sigma)
It fits the graph (as a rectangle, not as a square) the best it can within the viewport, for the default camera
Finally, it applies the camera modification

The thing is:

Either we fit in the viewport (then, necessarily, when the viewport is resized, then the graph appearance changes as well)
Either we don't, in which case, we cannot even guarantee you will see the graph when we load it

There are two tips that might interest you, though:

If you use the setting itemSizesReference: "positions", the node sizes will always be relative to their sizes, so when the graph shrinks, the node sizes shrink proportionally. This, used with the other setting zoomToSizeRatioFunction: (x) => x, will allow having a graph aspect seem more stable (example here).
In Gephi Lite, we have side panels that can come and reduce the visible space used by the graph stage. To preserve the aspect ratio, the graph stage size stays the same when the side panels are opened, and instead, the side panels are "hover" the graph stage. You can try to open the side panels directly on Gephi Lite to see what I mean.

I hope this helps!

Thank you for the reply, I believe this issue is still an enhancement. The case of the graph possibly not being visible on load is fine because one can simply calculate the centroid or find the node with the highest centrality and move the camera to it. The lack of control surrounding the fitGraphToFrame behaviour is a challenging problem. Even a flag like turn off graphFitToFrame with the description of 'You should know what you are doing. Here be dragons' would suffice. I can always add functionality but if it is behind a black box that can be challenging.

OK, fair enough, if the issue becomes as clear as "Allow bypassing graph rescaling", then I agree, it is an enhancement.

Ok, I modified the title and original comment for clarity. Thank you!

Hi @Zac-hills,
I added a new autoRescale setting, true by default, so that you can disable sigma's auto rescaling by setting as false.
I updated an existing story to showcase how it behaves:
https://www.sigmajs.org/storybook/?path=/story/fit-sizes-to-positions--story
You can resize the viewport to check that the graph ratio remains independent.
I hope it meets your need!

@jacomyal This is exactly what we needed. Thank you!

Nice, thanks for the quick feedback :)


Fail to create a container app which has a "Linux/arm64" image in it

OS: M1
Build Version: 20220405.6
Regression: Not a regression
Repro Steps:

F1 -> Azure Container Apps: Create Container App... -> Select one Container App environment -> Enter a container app name -> Select a container registry -> Select a repository which has a Linux/arm64 image in it -> Select the next items to create the container app.
Check whether succeeds to create a container app or not.

Expect:
Succeed to create a container app which has a Linux/arm64 image in it.
Actual:
Fail to create a container app which has a Linux/arm64 image in it.

Container Apps currently does not support arm images.

This issue also reproduces after executing "Deploy Image to Container App..." for the image in the Registry view.
Repro Steps:

Get project using "git clone https://github.com/MicroFish91/aca-hello-world.git" and open it.
Amend Dockerfile with the following option:  "--platform=linux/amd64".
Right click Dockerfile -> Build Image.
Navigate to the Images view -> Right click the above image -> Push -> Azure -> Select a registry -> Enter.
Navigate to the Registry view -> Expand the selected registry -> Right click the image -> Click "Deploy Image to Container App..." -> Select a container apps environment -> Select a container app -> Skip for now -> Deploy.

This issue also reproduces for the DockerHub and GitHub registries.
This issue doesn't reproduce when executing "Build Image in Azure..." to push the image for a registry.


Memory Leak issue

I am using a simple configuration:
input {

	file {

		type => "one_type"

		start_position => "end"

		path => [

		"/Users/Gabriel/Documents/ElasticSearch/Test/logstash-2.1.2/logs/*"

		]

		sincedb_path => "/Users/Gabriel/Documents/ElasticSearch/Test/.sincedb"

		codec => multiline {

		auto_flush_interval => 120

		max_lines => 100

		max_bytes => "10 MiB"

		pattern => "NVRAZBYCX"

		what => "next"

		negate => true

		}

	}

}

filter {

	drop{}

}

And generating random lines of logs using a simple command line that writes new lines to a log inside logs folder.
The memory consumption is going up and up with no stop, and looks like the GC is not being able to collect much garbage:

I generated a heap dump from the logstash instance and got the following:
HTML to be opened on a browser (remove the .txt since Github prevents people from uploading html):
Objects-by-class.html.txt
Image:

Same test without the auto_flush_interval is showing better results:

I can confirm that a library we use concurrent_ruby is not fully removing cancelled timer tasks.
I am working on two solutions for two different scenarios.

when ML is used without identity mapping, e.g. stdin input. I have a solution for this that uses one timer.
when ML is used with identity mapping, .e.g. file input. Here we can't have a timer per codec as each file path (identity) seen creates a copy of the codec and therefore a timer and a timer thread - 20K files = 20K codecs = 20K timer threads.  I have a solution and am coding it now.

I can confirm that #29 PR fixes this issue, I used @gabriel configuration and a script that generate log line. I ran the scripts for 2 hours and monitored the memory usager. GC was correctly behaving with no leak.

Thanks for taking care of this @ph @guyboertje


symfony webpack-encore

Any advice on how to load the ckeditor using webpack-encore ?

To be honest, I have no idea if CKEditor is buildable with webpack... Is there an official CKEditor doc about it?

I have give a try to your issue but unfortunatelly, it's not working:
var webpack = require('@symfony/webpack-encore');

webpack
    .setOutputPath('web/build/')
    .setPublicPath('/build')
    .cleanupOutputBeforeBuild()
    .addEntry('app', './vendor/egeloen/ckeditor-bundle/Resources/public')
    .enableSassLoader()
    .enableSourceMaps(!webpack.isProduction());

module.exports = webpack.getWebpackConfig();

@bogdaniel Have you been able to make it working? I'm interesting to put such infos in the docs :)

Closing as there is no feedback, if someone knows how to build CKEditor with webpack, share it :)

Hey guys,
I'm not using encore, but I managed to make ckeditor work with webpack. There is a CopyWebpackPlugin for webpack, which should be configured to copy ckeditor bundle to build directory:
    plugins: [
        new CopyWebpackPlugin([
            {from: './web/bundles/ivoryckeditor', to: 'bundles/ivoryckeditor'}
        ]),
    ],

In config.yml ckeditor should be configured not to do autoloading :
ivory_ck_editor:
    autoload: false

and in your twig where you use ckeditor you should add script before any ckeditor field is rendered (in <head> probably):
<script src="{{ asset('build/bundles/ivoryckeditor/ckeditor.js') }}"></script>

Hope it helps someone.


Tab completion for the pol binary

We should investigate how to enable tab completion in the console for the pol binary.

See: https://docs.rs/clap_complete/latest/clap_complete/


Redirected to auth/login with 404 after install

After the installation was successful, I got redirected to "url/auth/login" with a 404 Not Found page.
Ubuntu 16.04
PHP 7.2.2-3
Apache2 2.4.18
mod_rewrite is enabled

Same problem

I just got this working, I noticed that the htaccess file didn't copy over in the main directory so I copied that one to the server and everything seems to be working

@joerou I tried adding the .htaccess, but it didn't do anything...

Not sure if this will help anyone else, but I was able to correct this issue by following the below guide to test mod_rewrite and htaccess (turns out I missed a configuration step in Apache for mod_rewrite). This is written for the Bolt CMS, but applies to any Apache setup:
https://docs.bolt.cm/3.4/howto/making-sure-htaccess-works

@Jolva Thanks a lot for the link, I actually thought I had enabled .htaccess via the apache2.conf - but I was mistaken! Thanks a lot :D

I just got this issue as well.   I had to update the default httpd.conf with "AllowOverride All".   Shouldn't the requirements or install docs be updated with this info?

Can anyone explain what the solution to this problem is with nginx? I've just installed Cockpit on a Vagrant box with nginx and had this same problem.

Can anyone explain what the solution to this problem is with nginx? I've just installed Cockpit on a Vagrant box with nginx and had this same problem.

@boticello Did you find the solution for Nginx?

@fais3000 No. In the end I switched to a cloud solution (Prismic). It was easier to get started. I’ll try Cockpit again another time.

@boticello Did you try the steps outlined here? I was pulling my hair out with the same 404 issue.

@Jolva I haven't yet tried the solution you posted. Thanks for pointing me to it. When I try Cockpit again I will use it and post my results to this thread.

@boticello I ended up solving it by following https://github.com/agentejo/cockpit/issues/704
I am hosting cockpit in a subdirectory. I was blaming Nginx and htaccess for the 404 errors. Found out that cockpit automatically redirects to the root of the domain after auth instead of relative to the project root. I added a defines.php to overwrite those paths. Which solved it for me.

For all the ones who have problems running it under nginx. This problem was already discussed in #19 and there you can find the fix. Also there is a link to the nginx server config which worked for me
(hence the try_files $uri $uri/ /cockpit/index.php?$args;)

I'm having a similar issue, but mine is with the api routes and not the CMS routes. When I try to hit /api/collections/get/ i get this error:
php-fpm_1          | <IP_ADDRESS> -  20/Feb/2019:14:52:52 -0500 "GET /index.php" 404
nginx_1            | 2019/02/20 14:52:52 [error] 6#6: *1 FastCGI sent in stderr: "Primary script unknown" while reading response header from upstream, client: <IP_ADDRESS>, server: mysite.com, request: "GET /api/collections/get/projects?token={token} HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "cms.mysite.com"

So it appears that using routes nginx does not go back to the default index.php but tries to load one from {root}/api/collections/get/
Here is my nginx config:
    location / {
        try_files $uri $uri/ /index.php$is_args$args;
    }

    location ~* \.php$ {
        fastcgi_split_path_info ^(.+\.php)(/.+)$;

        fastcgi_pass    php-fpm:9000;
        include         fastcgi_params;
        fastcgi_index   index.php;
        fastcgi_param   SCRIPT_FILENAME     $document_root$fastcgi_script_name;
        fastcgi_param   SCRIPT_NAME         $fastcgi_script_name;
        fastcgi_param   PATH_INFO           $fastcgi_path_info;

        fastcgi_param COCKPIT_URL_REWRITE On;
    }

So for me it was pretty simple. I was installing cockpit on a2hosting with cpanel shared hosting. When i uploaded the cockpit zip file and extracted it. I moved all the files and folders to my subdomain's root directory and ran the install. When i attempted to access the login page, it showed 404. What i didn't realized was that when i moved all the files and folders over it didn't move over hidden files, once i chose the option to show hidden files and folders and moved those over to the root directory of my install. I was then able to access my cockpit login page. Hope this helps someone else with a similar issue.

I just got this issue as well. I had to update the default httpd.conf with "AllowOverride All". Shouldn't the requirements or install docs be updated with this info?

Definitive requirement in Apache Configuration (do not forget to add this to both virtual hosts ports 80/443)
<Directory /var/www/<cockpit_directory>
    Options Indexes FollowSymLinks
    AllowOverride All
    Require all granted
</Directory>

Genius! Now it worls for me on Win10, PHP8 (threaded php-8.2.6-Win32-vs16-x64) and Apache 2.4


Added search

I grabbed the search component from the helix website, but the problem was it was highly embedded and combined with their navigation component. I hacked it apart to get just the search component, and this was the result. There's probably some remnants of functionality that's not needed anymore, but that can be enhanced later.
The search results are currently populated from our query-index sheet. I set that up using instructions from here: https://www.aem.live/developer/indexing , but the instructions were a bit confusing to set up a custom sheet so I couldn't figure that out. I think in the future we can do things like make another page within that sheet that filter out pages we don't want to set up in search results. For now, I set up a page with the name.
I added a search block to the homepage for now. I set up a junk page to test the search with, you can currently search for the nav too. Searching "Example OG title" will pull up the example page, "logo" should pull up the nav, and "LoveSac" will pull up the homepage.

Search works and is returning results!


Updating content which changes alert size does not update position

I'm updating my modal after I have loaded some data which then changes the modal size significantly. After this change, the modal stays in the same position from the top of the document.
Is there any way to recalculate the modal position?

Can be fixed with the same styling changes from issue #46 with pull request #47

I'll check if the #47 PR fixed this also.

Fixed in PR #47 and tested


maddy: enable pam support

Description of changes
This allows to use pam for user authentication.
Config snippet to define a recv_authdb table:
auth.pam recv_authdb

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 202926 run on x86_64-linux 1

1 package built:

maddy

Result of nixpkgs-review pr 202926 run on x86_64-linux 1

1 test built:

nixosTests.maddy

Thank you!


Correctly free the QUIC_CONNECTION on failure during construction

We need to ensure we clear any references to the SSL_CTX and crypto_ex data.
Addresses the issue found in:
https://github.com/openssl/openssl/pull/21668#issuecomment-1787221129

Darn...dupe.


Numba ReferenceError: underlying object has vanished

I get this error whenever I run claspy segmentation in Jupyter notebooks after restarting kernel.
If I run it again without changing anything, it runs fine.
Do you have any idea what might cause this?
Numba version 0.56.4
Reproducible code
np.random.seed(1379)
dataset = "Noise"
cps = np.array([400, 1600, 1000])
time_series = np.concatenate((
    np.random.rand(cps[0]),
    1 + np.random.rand(cps[1]),
    -1 + np.random.rand(cps[2])
))
cps = np.cumsum(cps)[:-1]

clasp = BinaryClaSPSegmentation().fit(time_series)
clasp.plot(heading=dataset, gt_cps=cps)
plt.show()

Error
KeyError                                  Traceback (most recent call last)
c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\caching.py in save(self, key, data)
    468             # If key already exists, we will overwrite the file
--> 469             data_name = overloads[key]
    470         except KeyError:

KeyError: ((array(float64, 1d, C), int64, int64, ListType[UniTuple(int64 x 2)], ListType[array(int64, 1d, C)], type(CPUDispatcher(<function znormed_euclidean_distance at 0x000001F5E79D3D30>)), type(CPUDispatcher(<function sliding_mean_std at 0x000001F5E79D38B0>))), ('x86_64-pc-windows-msvc', 'tigerlake', '+64bit,+adx,+aes,-amx-bf16,-amx-int8,-amx-tile,+avx,+avx2,-avx512bf16,+avx512bitalg,+avx512bw,+avx512cd,+avx512dq,-avx512er,+avx512f,+avx512ifma,-avx512pf,+avx512vbmi,+avx512vbmi2,+avx512vl,+avx512vnni,+avx512vp2intersect,+avx512vpopcntdq,+bmi,+bmi2,-cldemote,+clflushopt,+clwb,-clzero,+cmov,+cx16,+cx8,-enqcmd,+f16c,+fma,-fma4,+fsgsbase,+fxsr,+gfni,+invpcid,-lwp,+lzcnt,+mmx,+movbe,+movdir64b,+movdiri,-mwaitx,+pclmul,-pconfig,-pku,+popcnt,-prefetchwt1,+prfchw,-ptwrite,+rdpid,+rdrnd,+rdseed,-rtm,+sahf,-serialize,-sgx,+sha,+shstk,+sse,+sse2,+sse3,+sse4.1,+sse4.2,-sse4a,+ssse3,-tbm,-tsxldtrk,+vaes,+vpclmulqdq,-waitpkg,-wbnoinvd,-xop,+xsave,+xsavec,+xsaveopt,+xsaves'), ('19597f6ba93d4431633b3bf3a936017a49578d550391ab41686d4ffc7c35fbb2', 'e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855'))

During handling of the above exception, another exception occurred:

ReferenceError                            Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_10548/1821199485.py in <module>
      9 cps = np.cumsum(cps)[:-1]
     10
---> 11 clasp = BinaryClaSPSegmentation().fit(time_series)
     12 clasp.plot(heading=dataset, gt_cps=cps)
     13 plt.show()

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\claspy\segmentation.py in fit(self, time_series)
    220         if self.n_segments > 1:
    221             prange = 0, time_series.shape[0]
--> 222             clasp = ClaSPEnsemble(
    223                 n_estimators=self.n_estimators,
    224                 window_size=self.window_size,

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\claspy\clasp.py in fit(self, time_series, knn, validation, threshold)
    400
    401         if knn is None:
--> 402             knn = KSubsequenceNeighbours(
    403                 window_size=self.window_size,
    404                 k_neighbours=self.k_neighbours,

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\claspy\nearest_neighbour.py in fit(self, time_series, temporal_constraints)
    382             if end > start: pranges.append((start, end))
    383
--> 384         self.distances, self.offsets = _parallel_knn(time_series, self.window_size, self.k_neighbours,
    385                                                      pranges, List(self.temporal_constraints),
    386                                                      self.distance, self.distance_preprocessing)

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\dispatcher.py in _compile_for_args(self, *args, **kws)
    485                     e.patch_message('\n'.join((str(e).rstrip(), help_msg)))
    486             # ignore the FULL_TRACEBACKS config, this needs reporting!
--> 487             raise e
    488         finally:
    489             self._types_active_call = []

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\dispatcher.py in _compile_for_args(self, *args, **kws)
    418         return_val = None
    419         try:
--> 420             return_val = self.compile(tuple(argtypes))
    421         except errors.ForceLiteralArg as e:
    422             # Received request for compiler re-entry with the list of arguments

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\dispatcher.py in compile(self, sig)
    970                         raise e.bind_fold_arguments(folded)
    971                     self.add_overload(cres)
--> 972                 self._cache.save_overload(sig, cres)
    973                 return cres.entry_point
    974

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\caching.py in save_overload(self, sig, data)
    650         """
    651         with self._guard_against_spurious_io_errors():
--> 652             self._save_overload(sig, data)
    653
    654     def _save_overload(self, sig, data):

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\caching.py in _save_overload(self, sig, data)
    660         key = self._index_key(sig, data.codegen)
    661         data = self._impl.reduce(data)
--> 662         self._cache_file.save(key, data)
    663
    664     @contextlib.contextmanager

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\caching.py in save(self, key, data)
    476                     break
    477             overloads[key] = data_name
--> 478             self._save_index(overloads)
    479         self._save_data(data_name, data)
    480

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\caching.py in _save_index(self, overloads)
    520     def _save_index(self, overloads):
    521         data = self._source_stamp, overloads
--> 522         data = self._dump(data)
    523         with self._open_for_write(self._index_path) as f:
    524             pickle.dump(self._version, f, protocol=-1)

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\caching.py in _dump(self, obj)
    548
    549     def _dump(self, obj):
--> 550         return dumps(obj)
    551
    552     @contextlib.contextmanager

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\serialize.py in dumps(obj)
     55     with io.BytesIO() as buf:
     56         p = pickler(buf, protocol=4)
---> 57         p.dump(obj)
     58         pickled = buf.getvalue()
     59

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\cloudpickle\cloudpickle_fast.py in dump(self, obj)
    566     def dump(self, obj):
    567         try:
--> 568             return Pickler.dump(self, obj)
    569         except RuntimeError as e:
    570             if "recursion" in e.args[0]:

c:\ProgramData\Anaconda3\envs\base\lib\site-packages\numba\core\types\functions.py in __getnewargs__(self)
    484         obj = self()
    485         if obj is None:
--> 486             raise ReferenceError("underlying object has vanished")
    487         return (obj,)
    488

ReferenceError: underlying object has vanished

Hi @voloddia,
thanks for mentioning this. I'll have a look.
Best, Arik

Hi @voloddia,
actually, I cannot reproduce your error. I ran
conda create -n claspytest python=3.9 conda activate claspytest pip install claspy pip install notebook pip install numba==0.56.4 jupyter notebook
and then created a notebook with your short example, restarting and running the kernel in all kinds of ways. But it worked, regardless. I also tried other numba versions, which all work fine.
However, maybe I can help you nevertheless: The ReferenceError you posted comes up, because we cache the results of the nearest neighbour computation with numba. I've had this error a few times myself in random scenarios. It sometimes comes up when numba confuses something in the caching (I guess). More importantly, I always fixed it by cleaning the cache. So, please go to your claspy installation you find with
pip show claspy
Then head to the pycache folder and delete it. This hopefully should solve your problem. Please let me know!
Best regards,
Arik

Thank you.
After restarting the computer, I also cannot reproduce the error.
That object vanished error still happened seemingly randomly in other places a few times.
Clearing pycache of claspy worked in those times, so I'll close this.


Feature/add nku

JIRA [SHARE-312]

Coverage decreased (-0.09%) to 93.702% when pulling 9a56adb72e4ba298238d0b5e5a8039037f8bd9b7 on kms6bn:feature/add_nku into 669f933b21b986e6cfe92413dfbb4790d8655634 on CenterForOpenScience:develop.


add categories

hi
I added categories in config and  display this error
Type '{ root: { render: ({ children, editMode }: RootProps) => JSX.Element; }; categories: { layout: { components: string[]; }; typography: { components: string[]; }; interactive: { title: string; components: string[]; }; }; components: { ...; }; }' is not assignable to type 'Config<Props, RootProps>'.   Object literal may only specify known properties, and 'categories' does not exist in type 'Config<Props, RootProps>'.ts(2322)
my vesion is 0.10.0
is there any other file that i need to change
my config type is
type Config<Props extends {     [key: string]: any; } = {     [key: string]: any; }, RootProps extends DefaultRootProps = DefaultRootProps> = {     components: {         [ComponentName in keyof Props]: ComponentConfig<Props[ComponentName], Props[ComponentName]>;     };     root?: ComponentConfig<RootProps & {         children: ReactNode;     }, Partial<RootProps & {         children: ReactNode;     }>>; };

Hi @PCosta1990 👋,
This will be because the categories feature has yet to be released but will once 0.11.0 is ready.  In the meantime, you can install a canary 0.11.0-canary.4613df4, and that should be able to get it working!

Yes, @xaviemirmon is right. This hasn't been released yet. I'm working on a few more changes to adaptors before canary is stable and I can do a release.


Use object shorthand for properties

This rule is on its way into the latest Standard ☺
ref: https://github.com/standard/eslint-config-standard/pull/166

Coverage remained the same at 100.0% when pulling a2620172773f7ebbe1a88b565417e57032a4b333 on LinusU:object-shorthand into 497078211ac9450b84282b6a9122b1519ed1eb77 on mcollina:master.

Coverage remained the same at 100.0% when pulling a2620172773f7ebbe1a88b565417e57032a4b333 on LinusU:object-shorthand into 497078211ac9450b84282b6a9122b1519ed1eb77 on mcollina:master.

Coverage remained the same at 100.0% when pulling a2620172773f7ebbe1a88b565417e57032a4b333 on LinusU:object-shorthand into 497078211ac9450b84282b6a9122b1519ed1eb77 on mcollina:master.

Coverage remained the same at 100.0% when pulling a2620172773f7ebbe1a88b565417e57032a4b333 on LinusU:object-shorthand into 497078211ac9450b84282b6a9122b1519ed1eb77 on mcollina:master.

Coverage remained the same at 100.0% when pulling a2620172773f7ebbe1a88b565417e57032a4b333 on LinusU:object-shorthand into 497078211ac9450b84282b6a9122b1519ed1eb77 on mcollina:master.

I would not force a semver-major bump down the chain for a change due to the linter (it’s true for all the modules I maintain).

I would not force a semver-major bump down the chain for a change due to the linter (it’s true for all the modules I maintain).

Of course not, I made a mistake when checking for support Node.js versions! Sorry about this!


Prompt suggests "UNLICENSED" but says it's invalid when entered

Hi! My work is currently licensed under “The Unlicense” license. Now, running ahkpm init, the prompt said something like,

…a valid SPDX license identifier or "UNLICENSED"

Emphasis mine, especially the 'D' character at the end.
And so, naturally, I typed UNLICENSED, but it nagged to me that it was invalid. Then I noticed that, “The Unlicense” doesn't end in 'D'. So, I typed UNLICENSE, but it nagged again. So in the end, it really only accepts case-sensitive SPDX identifiers. (Note that, I had to google what “SPDX license identifier” mean, just to get pass that prompt :P)
Conclusion: It's the prompt's fault :P
Proof of my little trials and tribulations:

P.S. Thanks for considering my little DSV library as one of the first few packages available via ahkpm. Also, a package manager for AHK is indeed a really cool idea btw! Something like that would definitely simplify my current AHK setup.

Thanks for the bug report, and sorry for the time you wasted fighting it!
Should have a fix for it in the next couple of days that:

Allows "UNLICENSED"
Clarifies that "UNLICENSED" is if you don't want others to use it.
Provides a link to the SPDX license list to help find the correct identifier.
Compares identifiers case insensitively, and automatically converts it to the correct casing in ahkpm.json.

@jasonsparc I've merged a fix for 1, 2, and 3 above. See #187. I expect to publish a 0.7.0 with the fixes this evening or tomorrow morning.
Number 4 from above will take a bit longer, but I can probably have that done sometime next week.

Is "UNLICENSED" equivalent to having no license? If so, how about "NO LICENSE" instead? This avoids confusion with "The Unlicense" (i.e., https://choosealicense.com/licenses/unlicense), which is actually a valid license as a dedication to the public domain (while also covering legal stuffs).
Also, I think my suggestion ("NO LICENSE") plays nicely with GitHub's warning about what may happen if you don't choose a license whatsoever for your repo (i.e., https://choosealicense.com/no-permission).

Looks good 👍
I'll keep this open just in case you still want to implement number 4. Although, even if you don't implement the case-insensitive lookup, I think people would just go to the provided link and look up their license identifier there, in case the program doesn't accept their incorrectly cased input. Feel free to close this issue once that's implemented.

I just tried it too:

P.S. I could see the new "scripts": {} line too. I might try it for my test script when I find the time :)


inkscape: Migrate to caskformula/caskformula

Not sure if an unofficial tap is acceptable for a migration from an official formula.
But, I think that for most users, having a drop-in replacement for the formula they were already using would be preferable to being downgraded from 0.92 to 0.91 in the cask.

If we wanted to go down that road, we'd just reinstate the formula. Your efforts are best directed at convincing upstream to reverse course on their decision to stop publishing new binaries for macOS, so that Cask can get the new version.


Moving a player?

I cannot seem to be able to move the player. I have tried sending position packets with very small increments and a small delay, but no dice. Would you mind providing an example of simply moving the player across the x, or z axis?

Some info regarding player movement can be found here: https://github.com/Steveice10/MCProtocolLib/issues/32

Thanks for the quick reply, but I have already seen both issues regarding it. It looks like the ClientPlayerPositionRotationPacket constructor has changed since then and I am not sure what the 'stance' is.

It doesn't exist anymore. It's probably safe to ignore.

Alright, I got it working, but do you have any idea how to make it more natural looking instead all jumpy

Smooth out your movement, e.g. set a destination to walk to and do it gradually little by little.

It works. The only thing left is implementing gravity. Right now I am replying to 'ServerPlayerPositionRotationPacket' packets. I have a for loop that adds 0.1 each time to the x position and sends a movement packet with a small delay afterwards. The constructor for the 'ClientPlayerPositionPacket' takes a boolean that represents whether the player is on the ground, or not. I am not sure how I can tell if the player is on the ground and whether I should decrement the y axis, or not.

For properly telling whether the player is on the ground, you could store block info from chunk packets and check if they're standing on a solid block. I don't recall if there's an easier way to do it or not, although if I remember correctly the server doesn't really mind if the on-ground value is incorrect, although it may cause some glitches with certain actions.


Enforce sidecar clean install

npm ERR! code EUSAGE
npm ERR!
npm ERR! The `npm ci` command can only install with an existing package-lock.json or
npm ERR! npm-shrinkwrap.json with lockfileVersion >= 1. Run an install with npm@5 or
npm ERR! later to generate a package-lock.json file, then try again.

npm ci requires package-lock.json to be present and one will not have it on clean clone, because it is ignored. Alternative might be npm install --clean...


jsonpb: unexpected output for small negative google.protobuf.Duration

What version of protobuf and what language are you using?
Version: v1.3.1
What did you do?
https://github.com/AlekSi/go-bug/tree/master/jsonpb
func TestDuration(t *testing.T) {
	d := -time.Nanosecond
	dp := ptypes.DurationProto(d)
	m := &jsonpb.Marshaler{}
	s, err := m.MarshalToString(dp)
	if err != nil {
		t.Fatal(err)
	}
	if s != `"-0.00000001s"` {
		t.Errorf("Unexpected result: %s", s)
	}
}

What did you expect to see?
"-0.00000001s"
What did you see instead?
Unexpected result: "0.-00000001s"

Thank you for the report.
@cybrcodr, can you confirm whether this was fixed in v2?

Also, if the fix is simple, it may be doing in v1 as well.

This is fixed in v2.
Minor, I think the posted sample code is comparing with slightly wrong value, a negative nanosecond is "-0.000000001s"  (8 zeroes).
The bug in V1 is that it puts the negative sign after the decimal.  Yeah, I can look into fixing this for v1.  Logic can be carried over from --
https://go.googlesource.com/protobuf/+/refs/heads/master/encoding/protojson/well_known_types.go#651

Thank you. Do you plan to release v1.3.2 any time soon?


NY Times bridge missing items, article text, and subtitle

Describe the bug
The bridge for the New York Times, bridges/NYTBridge.php, is out of date. The full article is missed, as well as not updating correctly and missing the subtitle of the story (subtitles missing seems to be common in RSS feeds for some reason).
Proposed fixes
The article CSS element has changed, not sure of the best one, but works with:
$articlePage->find('section.meteredContent')
To fix updating interval and missing articles, I followed other news ones here (like The Guardian) and set it to something lower, like 15 minutes:
const CACHE_TIMEOUT = 900;
as well as removing the limit in fetched articles in collectExpandableDatas.
Finally, to handle including the subtitle before the article image, I followed https://github.com/RSS-Bridge/rss-bridge/blob/47dc26c775a2158950cffe085c1781b6e6a71e26/bridges/BundesbankBridge.php#L63
// subtitle
if($subtitle = $articlePage->find('p.css-w6ymp8', 0)) {
               $article .= '' . $articlePage->find('p.css-w6ymp8', 0)->plaintext . '';
}
// figure contain's the main article image
$article = $article . $articlePage->find('figure', 0);
// p > css-exrw3m has the actual article
foreach($articlePage->find('section.meteredContent') as $element)
        $article = $article . $element;

Here is the complete bridge as modified:
class NYTBridge extends FeedExpander {

	const MAINTAINER = 'IceWreck';
	const NAME = 'New York Times Bridge';
	const URI = 'https://www.nytimes.com/';
	const CACHE_TIMEOUT = 900; //15 minutes
	const DESCRIPTION = 'RSS feed for the New York Times';

	public function collectData(){
		$this->collectExpandableDatas('https://rss.nytimes.com/services/xml/rss/nyt/HomePage.xml');
	}

	protected function parseItem($newsItem){
		$item = parent::parseItem($newsItem);
		// $articlePage gets the entire page's contents
		$articlePage = getSimpleHTMLDOM($newsItem->link);
                $article = '';
                // subtitle
                if($subtitle = $articlePage->find('p.css-w6ymp8', 0)) {
                        $article .= '' . $articlePage->find('p.css-w6ymp8', 0)->plaintext . '';
                }
		// figure contain's the main article image
		$article = $article . $articlePage->find('figure', 0);
		// p > css-exrw3m has the actual article
		foreach($articlePage->find('section.meteredContent') as $element)
			$article = $article . $element;
		$item['content'] = $article;
		return $item;
	}
}

gj on suggesting fix!


History hash is missing terms after ajax form submission (homepage)

To reproduce:

Load homepage
Fill out term a in submission form
Hit submit
(at this point the legend displays (no term) for both)
Adjust the slider

fixed in bc93b43


Set default Provision/Retirement Entry Point for Generic & Orchestration Catalog Item.

Issue #2090
@bzwei please review/test

It works. Can you apply the same change to generic service (both retirement and provisioning) and update the title of this PR?

@bzwei made suggested changes, please retest.

All working now!


Add Attributes request

It is a new functionality in Flinks can be very useful for analyzing and getting information about customer. Documentation: https://docs.flinks.io/docs/enrich-your-data

Codecov Report

Merging #2 into master will decrease coverage by 10.21%.
The diff coverage is 12.5%.

@@             Coverage Diff             @@
##           master       #2       +/-   ##
===========================================
- Coverage     100%   89.78%   -10.22%
===========================================
  Files           5        6        +1
  Lines         121      137       +16
===========================================
+ Hits          121      123        +2
- Misses          0       14       +14

Impacted Files
Coverage Δ

flinks/entities/attributes.py
0% <0%> (ø)

flinks/client.py
89.74% <33.33%> (-10.26%)
:arrow_down:

Continue to review full report at Codecov.

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


strings in moment fields

I was trying to get moment.Moment objects for dates.
Setting TypeScriptGeneratorSettings.DateTimeType to TypeScriptDateTimeType.MomentJS does indeed change the TypeScript type of the respective fields, but I get only strings in them at runtime.
Looking at the generated TypeScript, I can't see any conversion code. It seems that the result from the server is simply parsed with JSON.parse and passed unmodified to the application code.
I just updated to the most recent NuGet package.

this only works if you use DTO classes - this cannot work with DTO interfaces

I don't see why.
And if it's not supported, it should probably better throw.

This is because the received JSON is just parsed and returned - we cannot convert it to the correct structure. But you could inject a custom reviver and parse the datetime objects yourself...

I don't see why it has to be that way - the client knows the type structure, so it can deserialize the json with the correct types.
Thinking about it, I'm no longer sure whether it's such a good idea in my own case specifically though. I'm using MobX, and that thinks all types with ctors (like moment.Moment) are user-defined, which would be bad in this case.

Ok, since my motivation to have this work is gone you can have this closed if you feel the current behavior is as intended.

Have you checked the generated code for the DTO interfaces? It's just JSON.parse and returning the result... If you want to have correct date objects or rename properties, we have to post process this result and this means generating a lot of code...

Yes, it would generate more code.
By the way, I've got this related suggestion that one could generate only the client interface and the type information in a form that's available at runtime so that you could use this one definition with all of the different ways of calling the server (jquery, fetch, angular, etc.).
That way, the generated code would stay really lean (much more so even than it is now) and it could also be used in a more flexible manner.
Just a thought, after all it's very different from how it's being done now. I just always wanted to drop this thought.
I'm closing this then.


Does not allow to delete a created group when '+' special character is used in creating the Group using SCIM2

Suggested Labels
Affected : 5.8.0- alpha2
Severity: Major
Priority: Highest
Component: SCIM2
Type-Bug, Type: QA-Testing,
Environment
IS 5.8.0-m27
2 node cluster against Mysql 5.7
Ubuntu 16.04
Steps to reproduce.

Enable JDBC userstore in usr-mgt.xml
Set Enable SCIM to true
Create a group by including the special character + in the group name. Will allow to add the group successfully and can view the created group without any issue through the management console. Sample Request

curl -v -k --user admin:admin --data '{"displayName":"managerrrr+"}' --header "Content-Type:application/json" https://is580.qa.com:443/scim2/Groups

Login to the management console and try to delete the created group in step 3. Will not allow to delete the group. Will prompt a error message as attached in [1]

Could not delete role managerrrr . Error: 31806 - Cannot delete non-existing role
[1]. error-message.png

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


On branch edburns-msft-dd-2321245-empty-resource-group apply the "allowExisting" property.

modified:   pom.xml
modified:   weblogic-azure-aks/src/main/arm/createUiDefinition.json

Hello @edburns this PR is good to go. I've finished testing.


Add option to enable Lucene 8 index compatibility

My solution to #1952 is to add a flag -lucene8 that abandons consistent tie breaking, so retrieval doesn't need to touch the docvalues. In the regression script, a similar option --lucene8 allows the score matching to be more lenient.

Codecov Report

Merging #1953 (9a3e282) into master (2725655) will decrease coverage by 0.01%.
The diff coverage is 40.00%.

@@             Coverage Diff              @@
##             master    #1953      +/-   ##
============================================
- Coverage     60.10%   60.09%   -0.02%
  Complexity     1061     1061
============================================
  Files           178      178
  Lines          9886     9891       +5
  Branches       1370     1371       +1
============================================
+ Hits           5942     5944       +2
- Misses         3439     3441       +2
- Partials        505      506       +1

Impacted Files
Coverage Δ

...main/java/io/anserini/search/SearchCollection.java
41.63% <0.00%> (-0.26%)
:arrow_down:

src/main/java/io/anserini/search/SearchArgs.java
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Thundering herd prevention

The current implementation only works when the data is already cached. In this case, every time the data is read from the cache, the algorithm calculates the expiration time and, if key is expired, recompute the cache. But imagine the cache is empty(for the first time) in high traffic. If we get a thousand simultaneous requests, they all hit the empty cache and the system calculates the data for all of them. Which again causes the cache stampede problem.
Is there a way to prevent the calculation of it for all requests in the first time, when our key and data are not cached?
For example lock the key or anything else ?
$now = microtime(true);
$delta = ceil(1000 * ($now - $this->updated)) / 1000;
$expired = $now - $delta * $beta * log(random_int(1, PHP_INT_MAX) / PHP_INT_MAX);

return $this->expiry <= $expired;

One of the obvious solutions — warmup. How would you solve it?

The situation you have isn't "Cache stampede" but "Thundering herd".

Cache stampede, also known as a thundering herd :
When a large number of concurrent requests attempt to access the same data from the cache simultaneously, but find it missing or expired.
https://www.thetechplatform.com/amp/how-to-prevent-cache-stampede-thundering-herd-problems

Yeah. Seems to be different names used for the same thing. I found that "thundering herd" if usually used for empty cache situation and stampede is used for when many cache items are expiring at the same time.

Why locking is not good idea ?

Higher latency.
Reduced concurrency.
Potential for deadlocks.

I see no other immediate solutions to empty-cache problem i.e. around https://github.com/yiisoft/cache/blob/master/src/Cache.php#L79 we can do the following:
public function getOrSet(
        mixed $key,
        callable $callable,
        DateInterval|int|null $ttl = null,
        Dependency $dependency = null,
        float $beta = 1.0
    ) {
        $key = $this->keyNormalizer->normalize($key);
        /** @var mixed */
        $value = $this->getValue($key, $beta);

       if ($value) {
           return $value;
       }

       // We'll get additional N cache reads. Better than N reads from DB but...
       while ($lease = $this->psr->getRaw('lease_' . $key)) {
          usleep(500); // The higher the value the worse latency is.
       }

       // How to choose correct expiration time here?
       // That will double the amount of keys and reads/writes for cache overall.
       $this->psr->set('lease_' . $key, 1, max($ttl, 1500));
       return $this->setAndGet($key, $callable, $ttl, $dependency);
    }

It's should be optional and disabled for cases where user own resolve this problem on application level.


Fix pending integration test by moving postTx outside of the atomically block

[x] I have re-enabled pending test STAKE_POOLS_JOIN_05
[x] Moved postTx out of the atomically block, this makes test suite pass.

Issue Number
ADP-2662

bors try

bors try

bors try

bors try

bors merge


Lint models/admin.go

LGTM

@andreynering  Updated

@andreynering my bad, should have seen that one.

LGTM

LGTM

resolved #70


Clarify that enterprise features are available with CC

Also change the CC FAQ UX so that it feels like a single FAQ
page with CC and CC Free options.

This change is


Conditional breakpoints stop at wrong predicate after a period of time

It seems like conditional breakpoints are not working as intended.
I tried to debug a specific case which I wanted to find via a simple comparison of local variable against a string.
Reproduction:

Open file and add conditonal breakpoint here: unpickleScalaFlags

The condition should be name.toString == "DelayedLazyVal"

Try to debug test local1

You will see that the first match was at:

From this point conditional breakpoint will stop working.
The fix is to disable and then enable it, but it will stop on another wrong name after a few seconds.
This happened on scala dap 4.0.1

Thanks for the report!

Do you have any error in Metals log before you hit the conditional breakpoint?

I reproduce it and I have:
2024.02.26 13:59:14 ERROR com.sun.jdi.ObjectCollectedException


Config Eslint and Prittier

need to to config Eslint and  Prittier config for all packages

ref: https://rushjs.io/pages/maintainer/enabling_prettier/


[Question] Use EasyClick Only

Question
Is there a way to configure win-vind such that it only listens for the triggers for EasyClick/"EasyMotion-like Hints" but passes all other keybinds through to the OS?  Basically, I'd like to disable all other functions and modes and just use the EasyClick feature.
So far, I've tried removing all entries from keybinds.json besides the ones for EasyClick but that effectively blocks all normal keyboard input.

Right after posting this, I figured it out:

Remove all non-EasyClick from the default keybinds.json
The default EasyClick binds are added to GUI NORMAL mode / guin.  Either remove them or move them to guii
Either use the win-vind GUI or edit settings.json directly to make the default mode be GUI INSERT

Thank you for using it.
GUI Insert has only change_to_normal that change mode to GUI Normal in default and passes all inputs to OS. Therefore, you can use only EasyClick in the following steps.

Make the initial mode to GUI Insert.
Set bindings of easy_click_*** in GUI Insert (guii).

Example: (Ctrl + Shift + f)
    {
        "name": "easy_click_left",
        "guin": ["FF", "Fo"],
        "guii": ["<ctrl-F>"],   <----- THIS
        "guiv": [],
        "edin": [],
        "edii": [],
        "ediv": [],
        "edivl": [],
        "cmd": ["ecl"],
        "mycwn": [],
        "mycwi": [],
        "en": "EasyClick (Left Click)",
        "ja": "EasyClick (左クリック)"
    },

You can refer to keywords in Keyword List - win-vind.
If you want to disable change_to_normal (<Esc-Left>, etc.), set an empty string to its guii.
Example:
    {
        "name": "change_to_normal",
        "guin": ["<Esc-Left>", "<Esc-Right>", "<Esc-Up>", "<Esc-Down>", "<F9>"],
        "guii": [""],    <----- THIS
        "guiv": ["<guin>"],
        "edin": ["<guin>"],
        "edii": ["<guin>"],
        "ediv": ["<guin>"],
        "edivl": ["<guin>"],
        "cmd": [],
        "mycwn": ["<guin>"],
        "mycwi": [],
        "en": "GUI Normal Mode",
        "ja": "GUI ノーマルモード"
    },

The next version improves the performance of EasyClick and adds a preset feature. Please look forward to it.

If someone wants to do it now using .vindrc, they can use this:
set initmode guii
imap <ctrl-f> <easy_click_left>


Implemented PHPUnit, Comply to PSR4, Allow custom \Redis objects

I did a bunch of commits that i needed for this to run against my system.
You might want to cherry-pick what makes sense.

Comply to PSR-4 by putting the RedLock class into a vender namespace RedLock (might break backwards compatibility for people using dev-master to install package.
Tagging changes to allow freezing to a specific version in composer. This will prevent stuff from breaking in the future.
Adding PHPUnit test cases, and a bash script to start and stop 3 redis-server instances used in testing.
Allow user to pass an existing instance of a \Redis object instead of the [$host, $port, $timeout] array. (if user is already dealing with a connection, why open a new one)
Since custom \Redis objects can now be passed, the $token used to unlock the lock must be serialized with the \Redis objects serializer. This is important if the user is using something than the default serializer (like igbinary).

Ping @ronnylt

Since @ronnylt has abandoned the project, I've created a new codebase and packagist entry signe/redlock-php which contains this pull request, and more.


Disputes, Verdicts and Judgements STF Test Vectors (GP Section 10)

README

[x] Cover most error conditions
[x] Tiny and Full set
[x] ASN.1 schema
[x] Tests description in README
[x] Signatures verification using current/previous validator sets:

requires age to be correctly instanced
requires the validators to be within the test state

Thanks for keeping the format similar to the SAFROLE tests, it helps a lot with parsing them.

Would it be possible to also include current/previous validator keys to check that the vote signatures are valid?
The age probably indicates from which past epoch the judgement came? Currently it is only ever set to 0.

Thanks for keeping the format similar to the SAFROLE tests, it helps a lot with parsing them.
* Would it be possible to also include current/previous validator keys to check that the vote signatures are valid?

* The `age` probably indicates from which past epoch the judgement came? Currently it is only ever set to `0`. Eventually this would determine which key set to use IIUC.

@ggwpez Done

Would it be possible to set the disputes extrinsic to null in progress_with_no_verdicts-1.json to ensure that the "one or more verdicts" property of it is not violated? Like this the extrinsic is invalid, since it contains on verdicts.

Why is work report hash 0x11da6d1f761ddf9bdb4c9d6e5303ebd41f61858d0a5647a1a7bfe089bf921be9 being removed from rho in progress_invalidates_avail_assignments-1.json?

I think tau should advance by 1, otherwise tau == tau' in all tests.

I did not validate the markers yet, but the STF tests pass for Graymatter. Thanks!

Would it be possible to set the disputes extrinsic to null in progress_with_no_verdicts-1.json to ensure that the "one or more verdicts" property of it is not violated? Currently the extrinsic is invalid, since it contains on verdicts.

2. Why is work report hash `0x11da6d1f761ddf9bdb4c9d6e5303ebd41f61858d0a5647a1a7bfe089bf921be9` being removed from `rho` in `progress_invalidates_avail_assignments-1.json`? I dont see any verdict or anything against it.

3. I think `tau` should advance by 1, otherwise `tau == tau'` in all tests.

I did not validate the markers yet, but the STF tests pass for Graymatter. Thanks! And just to clarify, the tests transition to rho_dagger, not to rho_prime, right? Since there is no availability extrinsic or anything (i appreciate this to make it easier to test).
And thanks again for your effort, the format is pretty good - hopefully other conformance tests could adapt it.

Would it be possible to set the disputes extrinsic to null in progress_with_no_verdicts-1.json to ensure that the "one or more verdicts" property of it is not violated? Currently the extrinsic is invalid, since it contains on verdicts.

Since $\textbf{v}$ is defined to belong to a set of sequences, it can include an empty sequence. If I understand correctly, this applies to all sequences in the GP (e.g., (76)). If they are empty, they are simply empty sequences.
In other words, this would be represented as vec![], not Option::None.
Therefore, perhaps the quoted text from the GP should specify "zero or more verdicts"? @gavofyork

Why is work report hash 0x11da6d1f761ddf9bdb4c9d6e5303ebd41f61858d0a5647a1a7bfe089bf921be9 being removed from rho in progress_invalidates_avail_assignments-1.json? I dont see any verdict or anything against it.

The hash you see as the prefix of dummy_work_report items is actually the work package hash,  and not the work report hash. Specifically, it is the $h$ found in $\mathbb{S}$ by going from equation (118) to (119) to (122).
I populated that entry ($h$) with some random data to make the work report hash unique.
The report hash is then calculated by serializing the entire $w \in \mathbb{W}$ as per equation (118). If you hash the dummy_work_report octet string from the removed entry, you'll find that its hash is 376fd230cc3710d737382e6e7cf4eaa46f3bf89589e061492e8ffdf6bb5f90d4, which is the hash of the work report in the bad verdict.
I realize that including random data there might be misleading. I could change it to just a sequence of a single byte or something similar, or perhaps we should explicitly state what I just told in the test vector description? Or maybe both? WDYT?

I think tau should advance by 1, otherwise tau == tau' in all tests.

This is intentional because the disputes subsystem STF doesn't directly affect $\tau$. My goal is to construct the test vectors such that the post-state is strictly determined by what the GP prescribes, with timekeeping being part of section 6 (safrole, auth rotation, etc.).

just to clarify, the tests transition to rho_dagger, not to rho_prime, right? Since there is no availability extrinsic or anything (i appreciate this to make it easier to test).

Yes, that's correct. It might be better to rename it to rho_dagger to make this explicit.

The report hash is then calculated by serializing the entire w ∈ W as per equation (118). If you hash the dummy_work_report octet string from the removed entry, you'll find that its hash is 376fd230cc3710d737382e6e7cf4eaa46f3bf89589e061492e8ffdf6bb5f90d4, which is the hash of the work report in the bad verdict.

Okay i see. I am just hashing the data now without trying to decode it.

I realize that including random data there might be misleading. I could change it to just a sequence of a single byte or something similar, or perhaps we should explicitly state what I just told in the test vector description? Or maybe both? WDYT?

Yea, i was wondering since the 0x11da6d1f761ddf9bdb4c9d6e5303ebd41f61858d0a5647a1a7bfe089bf921be9 appears as work-report-hash in some other tests. So it is not really random and not unrandom either 😅 Just setting to 0... is also fine i think.

This is intentional because the disputes subsystem STF doesn't directly affect τ.

Yes okay. Am ignoring tau now.

The hash you see as the prefix of dummy_work_report items is actually the work package hash, and not the work report hash. Specifically, it is the h found in S by going from equation (118) to (119) to (122).
I populated that entry (h) with some random data to make the work report hash unique.
The report hash is then calculated by serializing the entire w ∈ W as per equation (118).

Actually i went through the whole decoding logic for a Work Report now, and it seems to me that the first hash should be the authorizer_hash, not the work package hash from the availability spec:

^ is the encoding equation for a Block. There is no other encoding defined for W, hence i assume this should be used?
(either way, i can decode it and then hash it back also when using the other decoding function)

@ggwpez https://github.com/gavofyork/graypaper/pull/67

Hello @davxy , the https://github.com/davxy/jam-test-vectors/blob/disputes/disputes/tiny/progress_with_faults-7.json test says ( in readme) this:
Auditor included as culprit following a verdict in his favor

can you clarify please? Because my test fails (correctly I guess) but the description on why that test should fail makes me question my understanding of how the verdicts/disputes system works.
In my point of view the test fails because this
https://github.com/davxy/jam-test-vectors/blob/04c2c14340abdefe6a12ab37b1470816c3147684/disputes/tiny/progress_with_faults-7.json#L38-L44
voted true just like the verdicts of 2/3+1 validators. so it's not really a fault.
If this is correct there is no real formalism that prevents this in the paper besides:

derivative logic based on written text (not math)
102 which states this:

Since the report hash ends up to be in posterior psi_g (2/3+1 validators voting yay) and since v is 1 then (to me) this is the reason why the test should fail. I dont see how culprits (which are not valued) have anything to do with the test failing?

@vekexasia indeed there was a typo.
The auditor, classified as an offender, had his vote aligned with the final verdict.

Can you explain where is this 269 coming from? https://github.com/w3f/jamtestvectors/blob/bcc43b76d5a06007022f624a786c5482b8227432/disputes/disputes.asn#L39
The minimal valid work report size that I am getting is 355 bytes (with assumption that core index is 4 bytes as I can't find the defs in GP).
I am decoding it into a WorkReport struct and if the data is invalid, well, I won't be able to decode it.

Can you explain where is this 269 coming from?
https://github.com/w3f/jamtestvectors/blob/bcc43b76d5a06007022f624a786c5482b8227432/disputes/disputes.asn#L39
The minimal valid work report size that I am getting is 355 bytes (with assumption that core index is 4 bytes as I can't find the defs in GP).
I am decoding it into a WorkReport struct and if the data is invalid, well, I won't be able to decode it.

For example given this structured content:
{
    "package_spec": {
        "hash": "0x30466e0ae1b05dde5249872475f6beeac368fd014b5a3413ceb32d3872143284",
        "len": 42,
        "root": "0x7af11fdaa717c398e223211842b41392f18df4bbc4ea0f4cfb972f19c7a64949",
        "segments": "0xc2fdd53ecd96faa2b4a53b5acfb113483e4b4a28d17e239824acd22fa05649b3"
    },
    "context": {
        "anchor": "0xc0564c5e0de0942589df4343ad1956da66797240e2a2f2d6f8116b5047768986",
        "state_root": "0xf6967658df626fa39cbfb6014b50196d23bc2cfbfa71a7591ca7715472dd2b48",
        "beefy_root": "0x9329de635d4bbb8c47cdccbbc1285e48bf9dbad365af44b205343e99dea298f3",
        "lookup_anchor": "0x60751ab5b251361fbfd3ad5b0e84f051ccece6b00830aed31a5354e00b20b9ed",
        "lookup_anchor_slot": 33,
        "prerequisite": null
    },
    "core_index": 3,
    "authorizer_hash": "0x022e5e165cc8bd586404257f5cd6f5a31177b5c951eb076c7c10174f90006eef",
    "auth_output": [],
    "results": []
}

I get the following encoded data:
30466e0ae1b05dde5249872475f6beeac368fd014b5a3413ceb32d38721432842a0000007af11fdaa717c398e223211842b41392f18df4bbc4ea0f4cfb972f19c7a64949c2fdd53ecd96faa2b4a53b5acfb113483e4b4a28d17e239824acd22fa05649b3c0564c5e0de0942589df4343ad1956da66797240e2a2f2d6f8116b5047768986f6967658df626fa39cbfb6014b50196d23bc2cfbfa71a7591ca7715472dd2b489329de635d4bbb8c47cdccbbc1285e48bf9dbad365af44b205343e99dea298f360751ab5b251361fbfd3ad5b0e84f051ccece6b00830aed31a5354e00b20b9ed21000000000300022e5e165cc8bd586404257f5cd6f5a31177b5c951eb076c7c10174f90006eef0000

minimal of 1 result is required so your data is invalid

@davxy I don't think that the new dummy work reports are correct.

@davxy I don't think that the new dummy work reports are correct.

@dakk can you elaborate? As the content is "dummy" for sure the content doesn't make sense. But maybe you're referring to the size of the raw data?

@davxy I don't think that the new dummy work reports are correct.

@dakk can you elaborate? As the content is "dummy" for sure the content doesn't make sense. But maybe you're referring to the size of the raw data?

I didn't want to send the comment without explaination (it was a mistake), I'm still debugging my decoder which works fine with CODEC tests, but fails with this: basically I'm reading 0 when it comes to read the number of workresults, but first I'm assuring that I'm following the GP correctly.

@dakk, as you correctly processed the vectors in the "codec" PR, I assume you're using the new JAM compact integers codec (i.e. the one in the gray paper)? These vectors are still utilizing the old SCALE compact integer encoding. I will regenerate them today I promise 😅 , and then we can retry.

@dakk, as you correctly processed the vectors in the "codec" PR, I assume you're using the new JAM compact integers codec (i.e. the one in the gray paper)? These vectors are still utilizing the old SCALE compact integer encoding. I will regenerate them today I promise 😅 , and then we can retry.

Oh gosh, I think that is the problem  😅 I'll wait for it, thank you

@davxy now my tests are passing, but you should also update the json files since dummy_work_report is encoded differently between the binary and the json.

Hi @davxy , do you had a chance to check discrepancies between the binary and the json format regarding the dummy_work_report? This is an example of difference between the twos:
progress_invalidates_avail_assignments-1.json
rho - prestate - first dummy work report
0x11da6d1f761ddf9bdb4c9d6e5303ebd41f61858d0a5647a1a7bfe089bf921be9000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000c0302010400000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000002a00000000000000000c010203
progress_invalidates_avail_assignments-1.bin
rho - prestate - first dummy work report
0x11da6d1f761ddf9bdb4c9d6e5303ebd41f61858d0a5647a1a7bfe089bf921be9000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000c0302010400000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000002a000000000000000003010203

@dakk currenty the JSON is constructed using the old codec (which is why you're seeing the dummy work report blob encoded in the older format). This happens because we first encode using the "classic" scale codec and then convert it to JSON using the scalecodec Python library: https://github.com/polkascan/py-scale-codec.
To produce a blob compatible with the new codec, we would need to fork the py-scale-codec and introduce the new compact encoding (more or less here: https://github.com/polkascan/py-scale-codec/blob/f676638998ce612d9f0b38b135f77a95eacd51ce/scalecodec/types.py#L31).
It shouldn't be too difficult, and since we're using that library for test vectors, I may implement this at some point (unless someone has already done it :-) ).

't be too difficult, and since we're using that library for test vectors, I may implement this a

Got it (y)
No, it is not a blocker; I disabled those test on jampy for the moment.

Just wanted to mention "progress_with_bad_signatures-2" seems to match two different errors I'm not sure if that was intended.
Depending on which order the implementers code their validation rules it could result in "bad signature", or "already judged" since the target exists in psi_b pre-state already.

Just wanted to mention "progress_with_bad_signatures-2" seems to match two different errors I'm not sure if that was intended.
Depending on which order the implementers code their validation rules it could result in "bad signature", or "already judged" since the target exists in psi_b pre-state already.

Yes, and also can match with culprits not sorted unique error.

This is another reason why it’s better not to be picky about the error codes returned by the STF. Invalid inputs may trigger different errors depending on the order of checks, which is unspecified in the GP.
That said, I agree that it’s very helpful to have test vectors that don’t rely on a specific order of checks. In future revisions, I’ll fix these signaled vectors to make the returned errors not ambiguous. The feedback gathered here is not being overlooked

Temporarily closed in favor of https://github.com/w3f/jamtestvectors/pull/28


(Design) Show if a Transaction is Not Completely Funded

Create a design for the partial payment message and payment screen when a payment has been made while purchasing a listing, or in the transaction modal, that is insufficient to fund the transaction.
Related UI Story: https://github.com/OpenBazaar/OpenBazaar-Client/issues/1571
Related Server Story: https://github.com/OpenBazaar/OpenBazaar-Server/issues/416

I did this while coding it, since the design was very minimal.


[BEAM-12650] Jenkins README updated with all existing tests

Jenkins jobs README updated with missing jobs with corresponding data for each one (Trigger phrase, CRON, etc)

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @kileys

Can you take a look at the ones with the broken build icon and see if the links need to be updated or removed?
https://github.com/apache/beam/blob/1c7fd7a4093c870db9ae7344e45bcb077672e98d/.test-infra/jenkins/README.md

@kileys - Is this ready to be merged? If so, could you do that?

Waiting on changes from https://github.com/apache/beam/pull/16610#issuecomment-1021499058

Waiting on changes from #16610 (comment)

Did you mean to link to another PR?

I am fixing the ones that has broken link
Will submitting the changes today @aaltay

@kileys changes applied, please see the following file


Unexpected late submission

Hello.
Validation on March 30th ended for me and my friend with the status late submission, even though all the transactions with short/long session answers have been sent on time.
Addresses:
0x75d6cE9A43A681BD21B79ccB148C07DA65345072
0x052cD56Ec984298E2E4dceCf26410eC964D3554d
I can also attach the log files here if they don't contain any sensitive data.

Did you use web app or desktop app ? If web app, what shared node did you use, private or from marketplace ?
Late submission is not only related to the time answers submitted.

Addresses:
0x75d6cE9A43A681BD21B79ccB148C07DA65345072
0x052cD56Ec984298E2E4dceCf26410eC964D3554d
In the case of both addresses, the late submission is related to created flips were not available on time for validation, decryption keys didn't reach on time the network.

Thanks for the answer.

Did you use web app or desktop app ?
Web app, the node is private.
decryption keys didn't reach on time the network.
What are decryption keys? Should they be transmitted from client to the shared node?
Is it possible to definitely say whether the problem was in the shared node or in the clients’ network?

Is it possible to definitely say whether the problem was in the shared node or in the clients’ network?

Is possible if you have logs from client and from shared node to check

While I was discussing uploading and sending logs with the owner of the shared node, they were overwritten with more recent ones. And we don’t have logs from the last validation.
Btw is there a method to check if the flips are available for the next validation (before the validation)?
Maybe some node API call?

Flips are available as soon are submitted, you can check in scan.idena.io but encrypted for all the network, and only decrypted at ceremony time, when ID that created the flips send keys to decrypt

Now I got it - the issue is about delivering the decryption key, not the flips themselves.
Thank you for the info, I'm closing the issue.


Only increment message count on server msgs

this should fix #14

LGTM :+1: will be merging shortlyt

@DrVilepis These kinds of joke commits will not be tolerated. I trust that this will be the last time we see these from you.


improve Python unit testing and refactor _validators.py

Which issue this PR addresses:

Improvement of Python code testing and refactor of _validators.py (application of Guard clauses, give some names to magic numbers, etc).
What this PR does / why we need it:
Before this PR there was no execution of any Python unit tests (even though we have a prompt saying "Python Unit tests", we were just executing the Python linter).
Now we have a lot more Python code covered by unit tests.
This PR simplifies the code in python/az/aro/azext_aro/_validators.py applying Guard clauses, avoiding having nested ifs and complex boolean conditions in order to improve the understanding of the code at first sight.
This PR also splits the Python test folder in:

unit tests (no external dependencies) and
integration tests (depends on Microsoft testing libraries and always fails -> this is why we did not execute Python tests before this PR). Now that we split tests in unit vs integrations we can safely execute unit tests without depending on MSFT things and without the tests failing all the time.

This PR adds Unit Tests for all the functions in _validators.py (it uses pytest.mark.parametrize for a cleaner data-oriented tests).
Lastly, it includes directives in the Makefile to execute python UNIT TESTS (no integration). -> this could be done directly in the Makefile but I decided to write a simple script because

the Python unit tests are invoked in two recipes of the Makefile:

one to just run the unit test (unit-test-python)
and another one to also run the linter (test-python)

(there are two different recipes in the Makefile because when programming, you just one to make your tests go green without worrying about the linter, and just before committing the code, execute the linter to make sure there are no linting errors. To sum up: it is more convenient to execute N times
make unit-test-python

and just once
make test-python

at the end, all that for a faster feedback loop).

IMHO it's better to have a big independent script and a clean Makefile rather than a bloated Makefile. If in the future we change how to execute the unit tests, we will touch a single file with that responsibility, the script, and NO the whole Makefile.

Is there any documentation that needs to be updated for this PR?

N/A. One could argue that the new Unit tests also serve as executable documentation.

@SudoBrendan Thank you for your review.
I have applied the proposed changes and fixed a few typos you mentioned.
Answering your question: yes, I have run all the tests with the code previous to the refactor.
A side comment, also: even though I was adding unit tests to existing code, I was ALWAYS making my test go Red before going Green to ensure no false positives.

/azp run ci

@SudoBrendan I see @AldoFusterTurpin fixed everything requested. I am dismissing your review so i can merge. Thank you for understanding.


Swipe gesture doesn't respond after navigating away and back

Describe the bug
After navigating to another screen and coming back to the screen with the carousel, the swipe gesture no longer works. The carousel stops responding. I can however, still use the pagination to navigate.
To Reproduce
Steps to reproduce the behavior:

Create a screen with the carousel
Navigate to another screen
Come back to the screen with the carousel
No longer can use swipe gesture

Current code:
<View style={carouselStyle.container}>
  <View style={carouselStyle.wrapper}>
    <Carousel
      ref={carouselRef}
      data={data.slider.items}
      onProgressChange={carouselProgress}
      height={(screenWidth * 9) / 16}
      loop={true}
      pagingEnabled={true}
      snapEnabled={true}
      width={screenWidth}
      mode="parallax"
      modeConfig={{
        parallaxScrollingScale: 0.8,
        parallaxScrollingOffset: 80,
        parallaxAdjacentItemScale: 0.8,
      }}
      renderItem={({item}) => {
        let image = hasImage(
          serverUrl,
          item.field_featured_image,
          network,
        );

        if (item.thumbnail) {
          image = hasImage(
            filesystem_root,
            item.thumbnail,
            network,
          );
        }
        return (
          <View style={carouselStyle.item}>
            <TouchableOpacity
              activeOpacity={1}
              style={carouselStyle.itemWrapper}>
              <Image
                source={{uri: image}}
                style={carouselStyle.image}
                resizeMode="cover"
              />
              <View style={carouselStyle.overlay} />
              <View style={carouselStyle.textContainer}>
                <Text style={carouselStyle.title}>
                  {item.title}
                </Text>
                <Text style={carouselStyle.type}>{item.type}</Text>
              </View>
            </TouchableOpacity>
          </View>
        );
      }}
    />
  </View>

  <Pagination.Basic
    progress={carouselProgress}
    data={data.slider.items}
    containerStyle={carouselStyle.paginationContainerStyle}
    dotStyle={carouselStyle.paginationDotStyle}
    onPress={number => {
      carouselRef.current?.scrollTo({
        count: number - carouselProgress.value,
        animated: true,
      });
    }}
  />
</View>

Expected behavior
You should still be able to use swipe gesture
Screenshots
NA
Versions (please complete the following information):

react: 18.3.1
react-native: 0.76.3
react-native-reanimated: 3.16.3
react-native-reanimated-carousel: 4.0.0-canary.22 (also on 4.0.0-alpha.12, haven't tried other versions)
react-native-gesture-handler: 2.21.2

Smartphone (please complete the following information):

Device: iPhone 16 Pro Max, Vivo x200 pro, Google Pixel 8, iOS simulators, Android emulators
OS: iOS & Android
Browser: na
Version: iOS, 18, Android (14, 15)

Additional context
I thought I had seen a similar issue created here, came back to see the state of it but couldn't find it anymore so apologies for duplicate.

same bug in expo go also


feat: A simple and easy to use casbin.js library for Vue based

feat: #1

You should redesign the api, but not only rename all variable.

Why rename all "authorizer" to "enforcer"?

A better solution is just provide enforcer instance to user. But not a asmalle set of available API.
Looks like
this.$enforcer.add/remove/update

@Zxilly ok，working on this！

@Gabriel-403 If you are free, please make a demo on codesandbox, this demo will install vue-authz from github repo, you need to show the code of README.md.

 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.


about attention mechanism...

nice work and highly configurable.
Is there a plan to increase the implementation of the attention mechanism?

Thanks :)
Currently there is no plan to  add  attention mechanisms. If you implement it, feel free to create a pull request so that it can be included.


Application Supersedence Attempts to Supersede Retired Applications

I've ran into an issue with some applications that have many versions where, when the recipe has supersedence configured for "true", that it will attempt to supersedence really old or retired applications.  For my case (Firefox ESR), we had 5 versions in SCCM with one in production and 4 superseded and retired.  When the newest version came out, it tried to supersede the first two applications, not the most recent.
In line 1390 of the CMPackager.ps1 (under the Invoke-ApplicationSupersedence function), would it make sense to exclude retired applications from the latest applications (Where-Object {$.IsExpired -eq $False})? Or maybe even excluding apps that are already superseded (Where-Object {$.IsSuperseded -eq $False})?  Maybe even just doing the "Select-Object -last 2" instead of "Select-Object -first 2" to account for the newly created app and the most recently deployed application?

I hit a similar issue where supersedence is updating the previous release instead of the new release.
When trying to package 81.0.4044.138
Application Supersedence
Supersedence is True
Latest 2 apps = Google Chrome 81.0.4044.113 Google Chrome 81.0.4044.122
Old: Google Chrome 81.0.4044.113 New: Google Chrome 81.0.4044.122


(maint) Synced file(s) with chocolatey/.github

Synced local file(s) with chocolatey/.github.

Changed files

Synced local  LICENSE with remote  .github/licenses/apache-2.0.txt

This PR was created automatically by the repo-file-sync-action workflow run #3044391886

@choco-sync thanks for this PR! 😄
This change brings the line-endings between the files into sync, which is also something that we really want to have in place.


Fix formatting of action/workflow files (OSOE-518)

And include recommendation for editing files (VS Code + https://marketplace.visualstudio.com/items?itemName=me-dutour-mathieu.vscode-github-actions) in the documentation.
Static analysis would also be nice, although this plugin still has false positives, because the underlying language server is not GHA-specific, but the GH team is working on an official one: https://github.com/github/roadmap/issues/564
Maybe https://github.com/oxsecurity/megalinter/blob/main/docs/descriptors/yaml.md?
Jira issue

FYI: https://github.com/Lombiq/NodeJs-Extensions/issues/11