Typer --ignored-modules type conversion failure

Hi,
There is a command line argument type conversion issue (related to Typer).
When "ignored_modules" is set multiple times, the variable is not converted into a list but into a tuple.
$ lazydocs ./app --ignored-modules "main" --ignored-modules "tests" --overview-file README.md
Source path:...  AttributeError: 'tuple' object has no attribute 'append'

@hellowhistler wrote a fix, but it is not merged:
Typer PR -> https://github.com/tiangolo/typer/pull/143#pullrequestreview-805875789

Issue has been closed by Github bot, however is still existing..
@tiangolo, is the project dead ?


Any plans for wasm64 support?

Are there any plans to support wasm64?
It looks to me that WASMTIME has begun to implement supports for WASM64, which allows a wasm application to use more than 4GB of memory. I am aware that WASI-SDK is not ready for 64 bits. I just wonder if WAMR has any plan to support wasm64 in the future.

@qinkunbao currently there is no plan to support wasm64 yet.

@qinkunbao currently there is no plan to support wasm64 yet.

Thank you for your reply.


Frame Selection (2D) / Focus Selection (3D) shortcut doesn't work reliably when focusing Scene tree (and if shortcut is a letter, selects node starting with that letter instead)

Godot version
v4.0.2.stable.official [7a0977ce2]
System information
Linux Ubuntu 22.04 with Unity desktop
Issue description
Moved from https://github.com/godotengine/godot-proposals/discussions/6853
Most of the time, when I use the 2D Frame Selection shortcut F (or Shift+F, apparently the new default) or 3D Focus Selection F, I have just selected the node in the Scene tree (otherwise I would already have it in scene view), so the Scene tree is focused. Pressing the shortcut at that moment will fail to focus on selection. If the shortcut is one letter (optionally with Shift), the Scene tree panel will focus the next node whose name starts with that letter (e.g. Floor) instead.
Inconsistency:

sometimes, esp. after repeating letter presses, it would stop searching nodes by name
sometimes, mapping Focus to another shortcut like F4 or Ctrl+Alt+F can help working more reliably even when Scene tree is focused; but not always (it worked yesterday, not today)

To make Focus truly work, you must click on the Scene view first, then use the shortcut, but it's cumbersome.
I think we should give priority to any meaningful shortcut when Scene tree is focused. The only feature we'd "lose" is if the user really wants to search for a node whose name starts with F. But I suppose the default for 2D was switched to Shift+F precisely to distinguish the two use cases (and reversely, a user could decide to revert it to F as it's the most common operation, and only press "Shift+F" to search for a node whose name starts with F).
I suppose this could then be documented somewhere.
Also, if we really want to support first letter searching we should fix the bug that blocks it sometimes, to make it work reliably.
Steps to reproduce

Create a scene with nodes with various names, some starting with F like "Floor", and at different positions, in 2D and/or 3D
Move the scene view away
Verify your current shortcut for Frame Selection (2D) / Focus Selection (3D): it should be F or Shift+F (you can try other letters too, or even F4).
Select a node and press the corresponding shortcut for the dimension (below, we assume the shortcut is F to simplify)
=> the nodes whose name start with F are selected
Press F more times => selection iterates over more nodes whose name start with F, but does not focus
Try various letters and press them multiple times => it may select nodes based on their names, but sometimes the selection just fails and nothing happens
Focus the scene view and press F => this time, the view frames / focus on the selected node or group of nodes
Repeat 4-7 with various shortcuts, letters or not

Minimal reproduction project
N/A

I think we should give priority to any meaningful shortcut when Scene tree is focused. The only feature we'd "lose" is if the user really wants to search for a node whose name starts with F. But I suppose the default for 2D was switched to Shift+F precisely to distinguish the two use cases (and reversely, a user could decide to revert it to F as it's the most common operation, and only press "Shift+F" to search for a node whose name starts with F).

I'm not fond of having incremental search work for all letters except F.

There is indeed a choice to make about priority, when we enable Scene View shortcuts while Scene Tree is focused. Ideally we'd let the user set their favorite context to detect a shortcut, but that requires a more elaborate shortcut editing system (like VS Code).
Otherwise, we must hardcode priority in advance, but then we must make a statement and align all default shortcuts to match this (I'm thinking about Shift+F for 2D vs F for 3D in particular).
I also realized there are other pure letter shortcuts like the transform tools QWER... I didn't really pay attention to it so far because I generally focus the scene when moving nodes around, but I could imagine a user selecting a node in the Scene Tree then W to translate it. I just checked Unity and... it doesn't have node selection by first letter at all.
To sum up the possibilities:
a. Any existing shortcut gets priority over name search. In this case, we allow Shift + Letter to still search Letter. Advantage: we can use F, QWER, etc. Disadvantage: we cannot search nodes whose names start with just F, QWER, etc. but it's possible using Shift + Letter.
b. Any simple letter press always searches nodes by name. However shortcuts with modifier (including Shift this time) work even when Scene Tree is focused. Advantage: searching node is consistent. Disadvantage: just F won't work, but user can always add a shortcut Shift+Letter to make it work. While Shift+F is OK, they probably won't want to use Shift+QWER for every transform tool, but Godot allows to add additional shortcuts, so they could be added as alternative shortcuts.
c. Add an option to switch priority between a. and b.
d. Add an option to disable searching nodes by node when typing letter entirely. Better combined with a. Users who don't need node search by first letter can check the option so they can just use F to focus.
Options c and d require more work, but that's much less work that a full VSCode-style shortcut editor with custom context field.

I'm also struggling with this in Windows, having a hard time locating nodes on the scene.


Automatic TF docs generation

Adding support via a pre-commt hook https://github.com/hookdeck/terraform-provider-hookdeck/pull/103


ci(Mergify): configuration update

This change has been made by @testgithubadmi from Mergify config editor.

@Mergifyio refresh

@Mergifyio copy testgithubadmi-patch-1


Help Needed

Hello,
I updated this portfolio for my own here is the Link.
I want to add a video clip that starts on hover, instead of normal image of my project in Work section, I tried to use this React-hover module but it is not working because of project structure, react-dependencies. I will be thankful if you can guide me for this

Hi AbhishekTiwari23,
It's been a while since your issue was published, but I saw on your site that you haven't implemented it yet so I'll share my two cents with you. In my opinion, you don't need to use any 3d party libraries, html video element (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/video), onMouseEnter & onMouseLeave event handlers and useRef hook may be sufficient for this.
Here is a basic example of how you should implement this:
const videoRef= useRef(null);
const onMouseEnter = () => ref.current.play();
const onMouseLeave = () => ref.current.pause();

<video ref={videoRef}>
    <source src={source} type='video/mp4' onMouseEnter={onMouseEnter } onMouseLeave={onMouseLeave } />
</video>

Hope this helps you :)

Hi, I need little help, I updated the code as per the latest update and made the required changes, you can check them here - https://github.com/ladunjexa/reactjs18-3d-portfolio/compare/main...AbhishekTiwari23:tiwariabhishek23:main why after deploying it is becoming like


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2469

Questions ?
Please refer to the Backport tool documentation


Prevent fieldsets from resulting in double data entries

Fixes: https://github.com/rails/jquery-ujs/issues/459

I created another PR where I fix both the issue I reference here and another issue. The code is somewhat different though, so the reviewer can have a look in what is more appropriate.

This fix is more correct in my opinion. Thank you for the pull request.


Weirdness in Resolv.getaddress

I want to report a bug
What is the current behavior?
The problem is in DNS resolution and the different behaviour is, on the same machine, same RVM environment, between padrino in console and irb when calling Resolv.getaddress
From within a padrino console session, a DNS resolution went in timeout
2.4.0 :012 > Resolv.getaddress('www.google.it')
Resolv::ResolvError: no address for www.google.it
	from /Users/thesp0nge/.rvm/rubies/ruby-2.4.0/lib/ruby/2.4.0/resolv.rb:95:in `getaddress'
	from /Users/thesp0nge/.rvm/rubies/ruby-2.4.0/lib/ruby/2.4.0/resolv.rb:45:in `getaddress'
	from (irb):12
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/gems/padrino-core-<IP_ADDRESS>/lib/padrino-core/cli/base.rb:43:in `console'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/gems/thor-0.20.0/lib/thor/command.rb:27:in `run'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/gems/thor-0.20.0/lib/thor/invocation.rb:126:in `invoke_command'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/gems/thor-0.20.0/lib/thor.rb:387:in `dispatch'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/gems/thor-0.20.0/lib/thor/base.rb:466:in `start'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/gems/padrino-core-<IP_ADDRESS>/bin/padrino:9:in `<top (required)>'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/bin/padrino:22:in `load'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/bin/padrino:22:in `<main>'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/bin/ruby_executable_hooks:15:in `eval'
	from /Users/thesp0nge/.rvm/gems/ruby-2.4.0@engage/bin/ruby_executable_hooks:15:in `<main>'

From an IRB session, same ruby version, same gemset, same machine, I've got and immediate resolution :
2.4.0 :004 > d.getaddress('www.google.it')
 => "<IP_ADDRESS>"

What is the expected behavior?
Also in Padrino, the address should be resolved
Which versions of Ruby, Padrino, Sinatra, Rack, OS are you using?
ruby 2.4.0p0 (2016-12-24 revision 57164) [x86_64-darwin16]
Padrino v. <IP_ADDRESS>
sinatra 2.0.0
Rack 1.3 (Release: 2.0.3)
Darwin pollon 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64 x86_64

Why are you using d.getaddress('www.google.it') in the second example? What is d?
I don't think this is a Padrino issue. Is this a new Padrino app with no modifications?
$ padrino c
=> Loading development console (Padrino v.<IP_ADDRESS>)
=> Loading Application Foobar::App
irb(main):001:0> require "resolv"
=> true
irb(main):002:0> Resolv.getaddress('www.google.it')
=> "<IP_ADDRESS>"

Hi @adam12 sorry, d was a Resolv::Dns.new object.
Here you can find an irb session.
2.4.0 :001 > require 'resolv'
 => true
2.4.0 :002 > Resolv.getaddress('www.google.it')
 => "<IP_ADDRESS>"

It's not a new Padrino app... it's a big multi application portal.
It's too weird something stucking DNS resolution... any idea if not padrino?

Can you post the application's Gemfile?

source 'https://rubygems.org'

# Project requirements
gem 'rake'
gem 'sinatra-flash', :require => 'sinatra/flash'

gem "capistrano", "~> 2.15.5"
gem 'ipaddress'

gem 'bcrypt'
# gem "bcrypt-ruby", :require => "bcrypt"

# Component requirements
gem 'haml'
gem "pony", "~> 1.6.2"

gem 'net-ldap'
gem 'puma'

gem 'dm-postgres-adapter'
gem 'dm-aggregates'
gem 'dm-validations'
gem 'dm-timestamps'
gem 'dm-migrations'
gem 'dm-constraints'
gem 'dm-core', '~>1.2.0'
gem 'dm-types'
gem 'dm-serializer'
gem 'dm-ar-finders'

gem 'will_paginate', '~> 3.0'
gem "nexpose"
gem "nexty", :group=>"development", :git=>"https://github.com/thesp0nge/nexty.git"
gem "trimmy", :git=>"https://github.com/thesp0nge/trimmy.git"
# gem "fastercsv"
# Test requirements
gem 'rspec-padrino', :group => "test"
gem 'rspec', :group => "test"
gem 'dm-rspec', :group => "test"
gem 'capybara', :group => "test"
gem 'cucumber', :group => "test"
gem 'factory_girl', :group => "test"
gem 'rack-test', :require => "rack/test"#, :group => "test"

gem 'rack-ssl', :group => ['test', 'production']
# Padrino Stable Gem
gem "padrino"

gem 'spreadsheet'
gem 'sanitize'
gem 'dawnscanner', :group => "development"

Nothing super exciting in there.
Any references to Resolv in your application? Perhaps in an initializer? Maybe someone manually set nameserver IPs.

Does not look like a Padrino issue.


Remove blank line at the start from init file (Debian)

There was a blank line at the start of the file before #! /bin/sh, which declares which interpeter it has to use. With this blank line the init script will not work and will throw error "exit status 1", meaning the deamon doesn't start.
The change was introduced in version 4.22.
This simple edit to the file will fix that.

@dnobori, could you please merge this one? That's a real problem for all Ubuntu \ Debian users and we need to fix it before introducing package to Debian repo.

In case it might be still needed: I choose option 1

It's preventing newly installed vpnserver service from start, and package doesn't count as installed properly. Very trivial change, please merge.

Thank you so much for your contribution to enrich the SoftEther VPN source code.
Your patch has been merged on the main source-tree of SoftEther VPN.
As a token of our gratitude, your GitHub username will be added on the AUTHORS.TXT file and on the header of the related source file.
(It will be done within a few days.)
https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT
Thanks again for your contribution.


⚠ 2 Minutes de génie (staging) has degraded performance

In 73f4372, 2 Minutes de génie (staging) (https://2mdgstaging.gitlab.io/) experienced degraded performance:

HTTP code: 200
Response time: 2558 ms

Resolved: 2 Minutes de génie (staging) performance has improved in 628aafa.


i18n.dir is not a function

Describe the bug
You must provide a clear and concise description of what the bug is.
i18n isn't populated with anything. It's an empty object.
Occurs in next-i18next version
You must provide the npm module version, git tag, or even better the explict commit causing the issue.
"^8.2.0"
Steps to reproduce
You must provide a reproducible repository, or copy/paste blocks of code which are sufficient to reproduce the bug.
  const { i18n } = useTranslation();
  const direction = i18n.dir(language);

Expected behaviour
You must provide a clear and concise description of what you expected to happen.
That you would be able to use i18n off the useTranslation. We switched from react-i18next and it worked without having to do anything else.
Screenshots
If applicable, add screenshots or a GIF to help explain your problem.
OS (please complete the following information)

Device: [e.g. MBP 2018 13"]
Browser: [e.g. Chrome 70.0.3538.77]
MBP 2017 16''
Chrome

Additional context
Add any other context about the problem here.

Cannot reproduce in the simple example. Please provide a reproducible repo. Thanks!

@isaachinman I created two examples:
Typescript - https://codesandbox.io/s/condescending-grass-qg6nr
Javascript - https://codesandbox.io/s/heuristic-villani-nyhzy
Typescript I'm still working on getting going it's throwing an error. Javascript the pages need the getStaticProps | getServerSideProps to work properly. I didn't realize that was a requirement for the library. I thought it was optional for some reason. I will add that and report back.

I have added getServerSide and had to change the serialization to import the config directly.
Now, I've got this error:
Error: You are passing a wrong module! Please check the object you are passing to i18next.use()

@birdwell I've loaded your JS codesandbox, and don't see any errors. Please provide a reproducible repo.

I finally got it all set up correctly. It was a mix of making the next-i18next.config.js use export default instead of module.exports once it's imported into _app.

Glad to hear it!


Narrative Script Writing for Chef Gus's Introduction

Description:
Write a comprehensive narrative script for Chef Gus, focusing on his backstory, the mission of "Green Paw Chefs," gameplay mechanics, and emphasizing the significance of sustainability and combating food waste. The script should be engaging and informative, laying the foundation for the game's educational goals.

Game script: https://docs.google.com/document/d/1C7HuhPjJ-w1rRJScn5LRRWgutfaHyTdw3iSj3oT_Skw/edit?usp=sharing


Make molecule work with ansible-base

Issue Type

Feature request

Molecule and Ansible details
molecule 3.0.4
   ansible==2.10.0b1 python==3.7

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Desired Behavior
I have been doing some testing with both ansible-base and ansible, and noticed that Molecule fails to work at all (at least with the default Docker driver) under ansible-base, since it calls the docker_container module directly—which is not part of the base distribution anymore.
It would be nice if Molecule were updated to install the community.general collection and use the community.general.docker_container module if it detects it is not available... somehow.
This is probably a bit of a niche use case, though, but it would be helpful for me so I could test whether a bug affects ansible-base and ansible (2.10 and later), or just ansible-base or ansible (2.10 and later).
Right now I basically can't use molecule if I'm testing with ansible-base locally.

There are serious issues with ansible wheels with 2.10 especially related to upgrading from 2.9.x but they have nothing to do with molecule itself.
We cannot start using FQCN until we drop support for 2.8, which would be after 2.10 is GA.
At this moment we cannot really make molecule depend on ansible-base as we have modules that need full ACD. Still, once we move them out of core and we also move minimal version to 2.10, we could switch to requiring ansible-base. In case it was not obvious this cannot be sooner than one year from now.

Ansible 2.10 has been released !
https://groups.google.com/g/ansible-announce/c/QxIOml3i62A
https://pypi.org/project/ansible/2.10.0/#files

Yep, but we will be able to add dependency on ansible-base only when we drop support for 2.9, which will happen when 2.11 is released.

Since 3.1.3 this is already true, molecule no longer needs ansible.

How did you get this to work?  Does molecule support ansible 2.10? I figured out that I have to pip install molecule[docker] and ansible-galaxy collection install community.docker to get the docker functionality, but molecule still fails:
ERROR! couldn't resolve module/action 'docker_container'. This often indicates a misspelling, missing collection, or incorrect module path.
because molecule_docker/playbooks/destroy.yml uses docker_container: instead of community.docker.docker_container:?  Or is there some way to get the ansible docker modules in a "legacy" format?

How did you get this to work?  Does molecule support ansible 2.10? I figured out that I have to pip install molecule[docker] and ansible-galaxy collection install community.docker to get the docker functionality, but molecule still fails:
ERROR! couldn't resolve module/action 'docker_container'. This often indicates a misspelling, missing collection, or incorrect module path.
because molecule_docker/playbooks/destroy.yml uses docker_container: instead of community.docker.docker_container:?  Or is there some way to get the ansible docker modules in a "legacy" format?

We must keep the short names for backwards compatibility but please add a PR to add the "collections:" bit which is ignored on older ansible versions.

Thanks for the details @tadeboro. For me, dropping this tests/molecule/default/collections.yml file fixed the problem (using ansible-base 2.10 and molecule 3.2.3:
collections:
  - name: community.docker
    version: ">=1.2.0,<2"
  - name: community.general
    version: ">=2,<3"

FYI these aren't an issue with either Ansible 3.0 or 3.1 which include the necessary versions. Only a problem if you have an older version installed in your collections.
Ansible 3.0
$ ansible-galaxy collection list |grep -e docker -e general
community.docker              1.2.2
community.general             2.0.1

Ansible 3.1
$ ansible-galaxy collection list |grep -e docker -e general
community.docker              1.3.0
community.general             2.2.0


Bot works but GUI doesn't

After I log in using the GUI it just stays frozen and unresponsive. The only thing I can do is use the top bar menu.
Checking the logs I can see the bot works fine and throw no errors.

Windows 10

Gonna try a bit more, I'll reopen if needed or update here if I can solve myself.

have same Problem ;(

do you have a solution


Store pertinent info from payload_response on order_item or *product

@brittballard you might already have done this?


Auto-Update Feature

This feature should notify users through the CLI whenever a new update is available. The goal is to streamline the update process, making it more efficient and user-friendly.

Implemented and will be in next release.


Highly available vmalert setup skips some recording evaluations at restart

I'm currently setting up a HA vmalert cluster. The relevant part of its definition is:
apiVersion: operator.victoriametrics.com/v1beta1
kind: VMAlert
metadata:
  name: vmalert-cluster

spec:
  replicaCount: 2

I have a group of recording rules with 30s period. When I do kubectl rollout restart deployment ... I observe missing points in recording rules results.
AFAICT, the problem is that vmalerts are being restarted too quick and that vmalerts rule evaluation starts after (not so) random delay: https://github.com/VictoriaMetrics/VictoriaMetrics/blob/cluster/app/vmalert/group.go#L226-L231
I propose that liveness check should become true only after all rule evaluations has been started.

I think this problem is more common before https://github.com/VictoriaMetrics/VictoriaMetrics/commit/96db7ac52c5323eacc4005493cdf0aa37b1462bc, because new vmalert could take too much time to restore data instead of creating new.
And I think it's hard to determine whether a vmalert is healthy, even if all group started, it doesn't mean all the group is configured right and starting creating new series. So the origin problem[some missing points] is still there.
@hagen1778 WDYT

Yes, vmalert indeed has delayed groups evaluation time to avoid thundering herd problem. Restarting of vmalert could result in missing evaluations. However, it is not expected for vmalert to restart frequently as its configuration can be hot-reloaded.
And even if one rules evaluation is skipped - that shouldn't cause many issues as VM's query engine should gracefully handle a single missing point.
I think, the same problem is related to vmagents restart as they could miss some scrapes, couldn't they?

Closing the issue as stale. Feel free to reopen if needed.


Nested create

The following should be possible:
client
    .user()
    .create(
        "Name".to_string(),
        user::image::create("some url".to_string(), vec![]),
    )
    .exec()
    .await;

Hi! Is this feature in development or planned to be shipped recently? I am using this library to integrate with MongoDB, so nested creation is an essential feature to support my development.
I can see from the code, FieldTypeExt has not supported type(sub-docuemnt) for now.
Open to contributing some code here!

From what I understand about MongoDB, I think you're referring to composite types. These are being tracked in #82.
This issue is for creating records for a relation in a relational DB.
Check #182 for the status on MongoDB support.

Hello, any update on this? Will there be any estimated date for this? Thanks.

still got a lot to go but the types for these are completed
client.user().create(
    "Brendan".to_string(),
    vec![user::posts::connect(vec![
        post::id::equals("0".to_string()),
        post::id::equals("1".to_string()),
    ])],
);

client.user().create(
    "Brendan".to_string(),
    vec![user::posts::create_many(vec![
        user::posts::create("Brendan".to_string(), false, vec![]),
        user::posts::create("Brendan".to_string(), true, vec![]),
    ])],
);

client.user().create(
    "Brendan".to_string(),
    vec![user::posts::connect_or_create([
        (
            post::id::equals("bruh".to_string()),
            user::posts::create("Brendan".to_string(), false, vec![]),
        ),
        (
            post::id::equals("bruh".to_string()),
            user::posts::create("Brendan".to_string(), true, vec![]),
        ),
    ])],
);


Parse the Markdown source code, please.

This is a fantastic plugin. It would be even better if it could parse Markdown instead of outputting the source code.

Hi, I'll try to get to it as soon as I find some time.
Closing this since it's a duplicate of #39


Try non-default user id to create non-root user in devcontainer.

closes https://github.com/ros2/ros2_documentation/issues/4590

@sloretz requesting another review.

@clalancette friendly ping.


[release-1.6] Fail with exit code in check-no-modify (#23425)

Manual cherry-pick of https://github.com/istio/istio/pull/23425.

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.6
6726e68df296f34cc74c0c05a451642536bf2471
link
/test gencheck_istio_release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test gencheck_istio_release-1.6

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.6
6726e68df296f34cc74c0c05a451642536bf2471
link
/test gencheck_istio_release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Hi @ostromart this is failing with:
operator/pkg/vfs/assets.gen.go: FAILED
md5sum: WARNING: 1 computed checksum did NOT match
Files above should not be modified in the repo. Please remove the change from your PR or run update_no_modify.sh if you are sure.

@ostromart: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ostromart: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.6
4d5d48010c259a7eb70312637a2725fee794e698
link
/test unit-tests_istio_release-1.6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Let configure pluginRespository in Gradle Wrapper

I have lots of separated projects with our custom plugins stored in Artifactory Maven repo. I want to use plugins { id '' version ''} syntax for them, so I need to configure plugin repository.
I want to configure this in custom gradle distribution that is used by Gradle Wrapper instead of configuring it for each project. Currently it needs to be configured in settings.gradle which is parsed before all init.d scripts so I guess it is not possible to do it from there.
Expected Behavior
It should be somehow possible to configure plugin repository inside gradle distribution (like it is possible to do for build scripts & and common dependencies inside init.d scripts)
Current Behavior
Currently, this plugin repository is defined in settings.gradle and hence can not be set 'globally'.

This will be possible once we remove the special handling for pluginRepositories and make them a normal method on Settings (see #1343). Then you can use the settingsEvaluated() hook in init scripts to add plugin repositories from a custom distribution.


[SPARK-10226][SQL] Fix exclamation mark issue in SparkSQL

When I tested the latest version of spark with exclamation mark, I got some errors. Then I reseted the spark version and found that commit id "a2409d1c8e8ddec04b529ac6f6a12b5993f0eeda" brought the bug. With jline version changing from 0.9.94 to 2.12 after this commit, exclamation mark would be treated as a special character in ConsoleReader.

Can one of the admins verify this patch?

cc @dragos can you take a look at this?

Jenkins, test this please.

Merged build triggered.

Merged build started.

Test build #41558 has started for   PR 8420 at commit 2e178d9.

Test build #41558 has finished for   PR 8420 at commit 2e178d9.

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

Merged build finished. Test PASSed.

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

LGTM @dragos

Sorry, I missed the Github notification. LGTM. The Scala REPL does the same thing. Probably the ancient version of jline that was used before didn't perform event expansion.

Thanks, I'm going to merge this into master and 1.5


Will Cloudpathlib support HDFS path?

I found that Cloudpathlib only supports prefixes with ['az://', 's3://', 'gs://'] currently.
Is there any plan to support HDFS path hdfs:// in the future?

Hi @daviddwlee84, we're open to supporting HDFS as a provider. None of the core developers use it regularly, so it would be great to have it from a contributor.
A few questions:

What's the best Python library for interacting with HDFS? There seem to be a quite a few options.
Is there a good Dockerfile for a single-node HDFS deployment? We'll want testing infrastructure that's easy to maintain.

Otherwise, implementation will be creating the HdfsClient and HdfsPath like for the existing providers, a test rig, a mocked backend for unit testing, and any provider specific tests.

Thanks @pjbull, I see.
For the first question, as far as I know, pyarrow.fs.HadoopFileSystem might be a good choice. (Some other libraries are just wrapping up Hadoop CLI which requires complex environment setup and version matching.)

Filesystem Interface
pyarrow.fs.FileSelector

Here is an example of how fsspec manipulates this.

fsspec.implementations.arrow
API Reference

For the second question, I haven't used Hadoop within the container before.
I found this might be usable but not very active.
big-data-europe/docker-hadoop: Apache Hadoop docker image
For single-node deployment, this requires a matched Java & Hadoop version. With minimal configuration on $HADOOP_HOME/etc/hadoop/hdfs-site.xml like
<configuration>
    <property>
        <name>dfs.replication</name>
        <value>1</value>
    </property>
    <property>
        <name>dfs.namenode.name.dir</name>
        <value>file:///nvme/HDFS/HadoopName</value>
    </property>
    <property>
        <name>dfs.datanode.data.dir</name>
        <value>file:///nvme/HDFS/HadoopData</value>
    </property>
</configuration>

Then can start with $HADOOP_HOME/sbin/start-dfs.sh.
I think these installation steps can be done in a Dockerfile easily.


sms logged as calls on samsung note II

Brilliant app.
However on Samsung Galaxy Note II sms's are logged as calls. So essentially unusable on that device. Would imagine it has to do With Samsung's way of integrating SMS in the call list..

device specific, closing


build: add godep-save script with vendoredApps manifest

Depends on https://github.com/tools/godep/pull/343
This removes dummy.go in favor of vendoredApps file which parsed by scripts/godep-save. Makes godep process to work with go 1.5.

Updated the PR, but not I haven't addressed all the issues yet.
cc @jonboulle

The godep PR just got merged, so this PR is ready for review and merge.

a few comments, but overall, woohoo!!

Fixes #1424

super, a few small style things, LGTM after that!


Added a start script for linux

The script performs these functions:

Checks for node and npm, installs the latest version from the official binaries.
Launches server.js.

I propose having a separate installer script that would run npm i for both Windows and Linux. The start script could just be node server.js.

If you copy the same file but give it an ".command" extension it also works as a startup for MacOS! :)


difftest:Surrport VCS use sim-jtag

The JTAG driver is packaged within the difftest shared object (.so) file, initialized with dpic, allowing the use of simjtag in the VCS environment.

Format check failed, please run make format locally and push again.

Format check failed, please run make format locally and push again.

Done ! Thanks for your response.


Fix some eslint issues and update some package versions

Fix Eslint Issues
Update Packages

https://github.com/react-scheduler/react-big-schedule/issues/92


Try fix issue #11438

What does the pull request do?
Try to fix issue #11438.
What is the current behavior?
Currently, the code can not ensure the list is not null when trying to iterate through it.
What is the updated/expected behavior with this PR?
Added one more check to ensure the list is not null.
Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Fixed issues
Fixes #11438

@dotnet-policy-service agree

@dotnet-policy-service agree

You can test this PR using the following package version. 11.0.999-cibuild0035175-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

Please, provide a unit test that demonstrates the issue

Please, provide a unit test that demonstrates the issue

You mean the repro for this bug? Actually I didn't figure out what caused this problem. I tried to find it out and make a repro for that but eventually I failed :(

Please find repro here: https://github.com/AvaloniaUI/Avalonia/discussions/11491#discussioncomment-5993054

Fixed by https://github.com/AvaloniaUI/Avalonia/pull/11552
@laolarou726 thanks!


app.logger logs messages multiple times when using mod_wsgi and application factory

Hi,
I'm not sure if this bug is in Flask, python or mod_wsgi but since I can only reproduce it when I use an "application factory", I'm opening it here.
When I create an application that

uses an application factory (create_app function to create the app object)
add a WatchedFilehandler logger to the app.logger
Create a basic route that logs something

If I deploy this application using Apache and mod_wsgi and then send 3 requests to the application, the log file contains more than 3 lines.
For the first request, it will contain one line, for the second request it will contain two more lines, 3 for the 3rd request, ...
I've created a git repository with an example to reproduce it:

Clone https://gitlab.esat.kuleuven.be/Rik.Theys/brol.git
In the repository is a Containerfile that can be used with buildah (or Docker) to build a container image:
buildah bud -f Containerfile -t brol
Run the resulting container:
podman run -dt --rm -p 5000:80 --name brol localhost/brol
Verify that there's no log file in the container image:
podman exec -it  brol cat /srv/app/logfile
Send a single request:
curl http://localhost:5000/app/
Check the log again. It will show one line.
podman exec -it  brol cat /srv/app/logfile
Send another request.
curl http://localhost:5000/app/
Check the log again. It will show 3 lines: one for the 1st request and two for the second request.
podman exec -it  brol cat /srv/app/logfile
Repeat the request/cat a few times. For each request, there will be multiple lines in the log now.

When I don't use a create_app function to create the app, but instantiate the object directly in app.py, I can not reproduce this issue.
I'm using the following configuration for mod_wsgi:
WSGIDaemonProcess myapp python-home=/srv/app/venv user=apache group=apache processes=2 threads=10 home=/srv/app
WSGIProcessGroup myapp
WSGIScriptAlias /app /srv/app/myapp.wsgi

<Directory /srv/app>
    Require all granted
</Directory>

I'm not sure but it seems like the file logger is added to the list of loggers with each request?
Environment:
I'm using these versions as they are the defaults on RHEL 8 and Flask 2.1 is not supported on Python 3.6

Python version: 3.6
Flask version: 2.0.1

Hi,
I've updated my test to return the list of currently registered log handlers.
Running this shows that for each request, an additional WatchedFileHandler is registered.
Regards,
Rik

Sounds like mod_wsgi is configured in such a way that it calls the factory for each worker it creates. And you're unconditionally registering a logging handler in that factory. Neither of these things are something Flask controls.


refactor(idl): unify the *.thrift files used by cpp and java-client

https://github.com/apache/incubator-pegasus/issues/1283
This patch including changes:

move thrift file from src/runtime/security/security.thrift to idl/security.thrift
remove java-client/idl/*.thrift
add struct request_meta to *.thrift which is used by java-client only
add java related namespace to *.thrift
remove java-client/idl/apache-licence-template while it is not used now
move recompile_thrift.sh from java-client/idl to java-client/scripts
update some structures' and fields' name, there are differents of *.thrift files between idl/* and java-client/idl/* before

Compatiblity changes
Because Java client is more wildly used than cpp-client and go-client, so I do the following refactor to make sure Java-client has less compatiblity issue:
cpp-client

old name
new name

configuration_query_by_index_request
query_cfg_request

configuration_query_by_index_response
query_cfg_response

go-client

old name
new name

ConfigurationQueryByIndexRequest
QueryCfgRequest

ConfigurationQueryByIndexResponse
QueryCfgResponse

I think we'd beter bump the major version to include this break changes?
And do we have a compatibilty guide about what things can be broken while upgrading?
And it is a bit strange that, why java client and cpp client can use different thrift defination while they connect to the same server?

That would be better, but I'm afraid there may be some more compatiblity changes in the future, the major version will inscrease too frequently then.
We will point that out obviously in the release note, as how we do before.
No matter what the name of the struct or field is, they are compatiable. When decode a request message from client on server, it depends on what the rpc code is, then use the related struct to decode it. For the fields, only the numberic tags are used, no matter what the names are.

OK, found this on stackoverflow
https://stackoverflow.com/questions/52882370/is-safe-to-rename-a-field-in-thrift-idl
SO thrift only relies on the field id, not the name.
Then I think it is less hurt for our users to upgrade.


add view

Codecov Report

Merging #8 (5dbb231) into main (0c2df71) will increase coverage by 1.90%.
The diff coverage is 64.89%.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
+ Coverage   59.13%   61.03%   +1.90%
==========================================
  Files           8        9       +1
  Lines         602      675      +73
==========================================
+ Hits          356      412      +56
- Misses        204      216      +12
- Partials       42       47       +5

Impacted Files
Coverage Δ

jenkins/types.go
0.00% <0.00%> (ø)

jenkins/view.go
64.78% <64.78%> (ø)

jenkins/job.go
70.81% <73.68%> (+1.99%)
:arrow_up:

jenkins/jenkins.go
59.71% <100.00%> (+0.29%)
:arrow_up:

jenkins/build.go
60.52% <0.00%> (+7.89%)
: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 0c2df71...5dbb231. Read the comment docs.


dynamically load compressed data

A popular use-case for genshindb is to load it as a script in browser. Unfortunately the library is over 20MB uncompressed and takes too long for a webpage to load.
It should be possible to first load the genshindb script with no data,
then send http request to get specific compressed data,
then decompress,
then register the data into the genshindb script.

2a68347d48e815aa17a651e41cc8c90c7b185f72


TF32 Support for NVIDIA A100

This is an update of https://github.com/tensorflow/tensorflow/pull/39764
Attn: @reedwm

@reedwm, PTAL

Seems auto-merge is not happening but the changes are now committed so we can close this. Thank you for the PR.


Reload the flask dynamic blueprint by using the @app.route('/reload')

Hi,
i am creating the new routes by having  new entry in the backend table.  here new routes are assigned to blueprint .
the new routes are not reflecting instantly it requires restart the app to get the new blueprint route .
ping me the possibilities based on the requirements.
Thanks

Routes cannot be changed at runtime. Define one route with the static part (e.g. /content/<key>) and then query for the related data in your view function.
That aside, this is not the right place for support questions; use Stack Overflow or our discord.

Routes cannot be changed at runtime. Define one route with the static part (e.g. /content/<key>) and then query for the related data in your view function.
That aside, this is not the right place for support questions; use Stack Overflow or our discord.


Validate removal of C_FLAGS = -Xcc -fblocks for linux

@rolivieri also tested on Kitura-Starter-Bluemix
https://travis-ci.org/IBM-Swift/Kitura-Starter-Bluemix/jobs/165881014
and it passes


No-Fork provider no longer applies terraform $$ escapes

What happened?

For resources where we fork a terraform CLI process to reconcile them, if there is a ${something} in a string parameter, in order to prevent terraform from interpreting the value as a terraform variable, users needed to set the crossplane spec.forProvider field to $${something}. The corresponding status.atProvider field would be ${something}. This always felt like a bug.
This syntax is only used in a handful of use cases, to execute some sort of AWS-specific logic. The two I'm familiar with are an ssoadmin InstanceAccessControlAttribute (which I'm adding in #928) and an iot TopicRule with a destination type of kafka, where it's used to extract credentials from secretsmanager. I bet there are more, too, but I didn't find any in the existing /examples directory.
Now for resources that aren't forking a terraform CLI, this behavior no longer applies, and resources with $${something} in the spec.forProvider fields get an error from AWS because the doubled $ is now being passed through to AWS and that is not valid syntax.
This is definitely an improvement, but it's a breaking behavior change that came as a surprise, and should be documented.
How can we reproduce it?

https://github.com/upbound/provider-aws/commit/aab2f0d547f06f57636736852095a886acbc91f9
I had to make this change to what was (in the fork version) a passing test in order to make it pass in the no-fork version.

For the release notes, I'd offer a recommended migration strategy of

Annotate all resources with a $$ escape in their spec with crossplane.io/paused="true"
Do the provider upgrade
Edit the paused resources to remove the extra $
Remove the crossplane.io/paused annotation.


Texture display wrong in IOS browsers

I found in ScupltGL can not display correctly in all IOS browers, but IOS claims it support WEBGL, and i test another webgl matcap sample base on matcap, and it's work fine.
pls check this link for test video:
https://vimeo.com/185645917

Thanks for the report, unfortunately I have an android,, so for now I can't really debug it.

something wrong with me too.
when i // if (this._showGrid) this._grid.render(this)
the texture can not display correctly( look like color lump / color spot ).
the video : https://youtu.be/YX-jsQu-4gI

This commit appeared to have broken the shader on some iOS environments.

Thanks, I suspect something wrong with shader derivatives, can you try https://goo.gl/m9C6Bx and see if it works?

yes, i work ( https://goo.gl/m9C6Bx ). seen like as desktop

Yes, it works fine in IOS10 Chrome. So if it means your suspect not established?
In my further test for SculptGL shader in IOS10 Chrome, I found PBR,matcap,normal shader are also not work well, but UV shader is OK.
So I guess the bug is from Reflection/Environment Mapping, pls check Jaume Sanchez's blog about Spherical reflection (or environment) mapping (SEM)
[https://www.clicktorelease.com/blog/creating-spherical-environment-mapping-shader](url
Someone also report a bug in IOS8, and the solve the problem, but Jaume Sanchez's tutorial is base on three.js framework, I don't know if it work in native webgl.

I'm almost sure it's not the matcap shader as PBR and normal have the issue too.
It can be many things, but until I debug on an iOS for real, it's tricky to solve :p.

i came to ( https://www.browserstack.com/ ) test on IOS online within a free account.
it can test on locale need a plugin

Ok I'll see if I can reproduce and if it's not too complex to setup!


Update ExampleStep readme

Need further clarification why we removed the word "Aperture" from "Aperture Data Studio". If my understanding is correct, "Aperture Data Studio" is the correct brand name.

I think it's fine to refer just to 'Data Studio' in the body of the text where it improves readability, just as we do in the main application user guide here: https://www.edq.com/documentation/aperture-data-studio/user-guide/


How to destroy a already registered $mdExpansionPanelGroup

I need to destroy the whole $mdExpansionPanelGroup in order to update it.
How do I do this?

I'm not clear on what you mean by "destroy", but here are some things that may help.

You can use the removeAll method

$mdExpansionPanelGroup('theComponentId').removeAll();

You can remove the group element from the DOM. This will remove and destroy all panels.

Will close this after 30 days of no response

Closed due to inactivity


The oauth state was missing or invalid

I'm getting an exception when trying to use Facebook OAuth in my application:
Exception: The oauth state was missing or invalid.
Microsoft.AspNetCore.Authentication.RemoteAuthenticationHandler+<HandleRequestAsync>d__12.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
System.Runtime.CompilerServices.TaskAwaiter.GetResult()
Microsoft.AspNetCore.Authentication.AuthenticationMiddleware+<Invoke>d__6.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
NWebsec.AspNetCore.Middleware.Middleware.MiddlewareBase+<Invoke>d__2.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
NWebsec.AspNetCore.Middleware.Middleware.MiddlewareBase+<Invoke>d__2.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
NWebsec.AspNetCore.Middleware.Middleware.MiddlewareBase+<Invoke>d__2.MoveNext()
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware+<Invoke>d__7.MoveNext()

It's being configured successfully in Startup.ConfigureServices as such:
services.AddAuthentication()
    .AddFacebook(facebookOptions =>
    {
	facebookOptions.AppId = Configuration["facebookAppId"];
	facebookOptions.AppSecret = Configuration["facebookAppSecret"];
    });

The Challenge result is being created in my AccountController:
[HttpPost]
[AllowAnonymous]
[ValidateAntiForgeryToken]
public IActionResult ExternalLogin(string provider, string returnUrl = null)
{
    // Request a redirect to the external login provider.
    var redirectUrl = Url.RouteUrl("SigninFacebook", new { returnUrl });
    var properties = SignInManager.ConfigureExternalAuthenticationProperties(provider, redirectUrl);
    return Challenge(properties, provider);
}

However the callback method is not being reached:
[HttpGet("signin-facebook", Name = "SigninFacebook")]
[AllowAnonymous]
public async Task<IActionResult> ExternalLoginCallback(string returnUrl = null, string remoteError = null)
{
    // etc...
}

, and I am getting the exception. I've setup my Facebook developer account successfully and added the valid Valid OAuth Redirect URIs as per the Docs.

https://docs.microsoft.com/en-us/aspnet/core/security/authentication/social/facebook-logins?view=aspnetcore-2.1&tabs=aspnetcore2x

The Facebook authentication configured later in this tutorial will automatically handle requests at /signin-facebook route to implement the OAuth flow.

The /signin-facebook route is handled by the middleware, not by your MVC controller. Your external login should route to something like /ExternalLoginCallback.

Ah thanks, I read that about 1 million times and it just wasn't sinking in.


There isn't a distinction between model_name/identifier and model_path

Hi team,
I have a LMQL server running on one machine like this:
MODEL_NAME = "llama.cpp:/home/ubuntu/Desktop/models/TheBloke/Yarn-Llama-2-13B-128K-GGUF/yarn-llama-2-13b-128k.Q4_K_M.gguf"

lmql.serve(model_name=MODEL_NAME, n_gqa=8, n_gpu_layers=100, n_ctx=2000)

and I am making request to this server this way:
import lmql
import nest_asyncio
nest_asyncio.apply()

@lmql.query
def hello():
    '''lmql
    MODEL_NAME = "llama.cpp:/home/ubuntu/Desktop//models/TheBloke/Yarn-Llama-2-13B-128K-GGUF/yarn-llama-2-13b-128k.Q4_K_M.gguf"
    argmax
        "Hello[WHO]"
    from
        lmql.model(MODEL_NAME, endpoint="localhost:1212")
    where
        len(TOKENS(WHO)) < 1500
    '''

hello()[0].prompt

The problem is that my model name is the path of the model on the server machine. The request machine doesn't have the same file structure.
Is there a way to use a model_identifier that would be set in lmql.serve and used in lmql.model?

Currently the string you use on the client side must match the server-side string exactly, so you cannot customize this. In the case where this includes paths, it indeed is a bit odd to use server paths on the client.
If you need to specify a tokenizer that is differently located, you can pass a tokenizer=... argument to lmql.model.
For consistency, and in the future though, I think you are right in that it would make sense to disentangle the path and the model identifier. Let's keep the issue around to track that as an enhancement.
Is this mainly a consistency issue/question for you or did you encounter other issue because of this?

Thank you @lbeurerkellner  for the quick reply.
It is mainly a consistency issue/question. I had really long path to access my models and it was odd to do it on the client side.


CSS regression tests

https://github.com/garris/BackstopJS
https://css-tricks.com/automating-css-regression-testing/

BackstopJS
Needs to be tested on Windows


updating some dependencies.

What changed?
Updating a few dependencies.

Why?
In general good practice to keep dependencies up to date to avoid security vulnerabilities in old versions and to get benefits of new features/improved performance.

How did you test it?
Relying on tests.

Potential risks
Some of the new dependency versions might now work correctly, work differently. There is always a risk when upgrading dependencies.

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 01856350-8265-421c-95b9-1209ca7705b8

0 of 0   changed or added relevant lines in 0 files are covered.
109 unchanged lines in 19 files lost coverage.
Overall coverage decreased (-0.02%) to 57.219%

Files with Coverage Reduction
New Missed Lines
%

common/types/shared.go
1
41.76%

client/history/client.go
2
38.1%

client/history/metricClient.go
2
45.3%

common/persistence/executionManager.go
2
78.89%

common/persistence/statsComputer.go
2
95.71%

common/task/fifoTaskScheduler.go
2
87.63%

common/util.go
2
52.54%

service/history/execution/mutable_state_builder.go
2
68.79%

service/history/handler.go
2
47.28%

service/history/task/transfer_active_task_executor.go
2
72.22%

Totals

Change from base Build 01855e4e-1a58-4c8e-a8bf-5e154102e721:
-0.02%

Covered Lines:
84902

Relevant Lines:
148381

💛  - Coveralls

Closing as part of old PR cleanup effort.


Why is there a 2px padding/space on the right side of the lightbox?

To confirm this issue, I just tested this from http://sorgalla.com/lity/ by using Image demo.  Check the image below.

For small boxes this is not noticeable but for the lightbox with 100% width, the white line on the right can be seen clearly.

This cause by https://github.com/jsor/lity/blob/2.x/src/lity.less#L51
TBH, i'm not sure anymore why this is there, you can probably just remove it.

No probs, I added below code to my stylesheet, and it fixed the issue.
.lity-container{
    margin-left:3px;
}

as the modal box was going out of screen on left side. I thought may be some error in my stylesheet but it is on the demo too. It's just because of dark background, it's not noticeable.  But for the full width modals with like 10-15px spacing on left-right side( client request :P ), it is noticeable.

This should be in Lity. I made a bug report about this, thinking that .lity-wrap:before didn't need a margin-right of -.25em.


cratify zingo-memo-utils

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@a3821f9). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@          Coverage Diff           @@
##             dev     #236   +/-   ##
======================================
  Coverage       ?   56.34%
======================================
  Files          ?       35
  Lines          ?     5113
  Branches       ?        0
======================================
  Hits           ?     2881
  Misses         ?     2232
  Partials       ?        0

Impacted Files
Coverage Δ

zingo-memo-utils/src/lib.rs
72.91% <ø> (ø)

zingolib/src/blaze/fetch_full_transaction.rs
85.60% <ø> (ø)

zingolib/src/lib.rs
64.28% <ø> (ø)

zingolib/src/wallet.rs
57.41% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

I propose the name zingomemo or zingo_memo or zingo-memo, such that when a developer uses the public interface they express things like:
zingomemo::unpack
zingomemo::pack
zingomemo::authenticate_nym
zingomemo::record_nym
zingomemo::distrust_nym
zingomemo::sign_message


Allow faceting for world locations, policy areas and taxons

World locations and policy areas are used in the whitehall finders like https://www.gov.uk/government/announcements. We're building new finders for those, so to keep feature parity we need to be able to facet on those attributes.
The taxons facet we'd like to use to experiment in finders too.

🚢


unit tests for implicit type conversions

Unit tests were written before 724a40baec3ded142 - we should circle back and write a few more now to capture all usecases.

stale: closing - we are working on test coverage overall right now leading up to release: #78


Parallel Joins

After the amazing work by @peterneyens in https://github.com/47deg/fetch/pull/84, we've acquired a tech debt that we might consider to implement, based on the conversation in gitter:
https://gitter.im/typelevel/cats?at=5828ac28e097df7575a76077
Adelbert suggested to implement it with Free#fold.

After the last re-design, I think this is no longer needed. Closing this issue...


A way to add EPOLLPRI event type to the epoll selector

While working with Video4Linux2 I am searching for a way to (a)wait POLLPRI events coming from the kernel.
I do see, that an Event can be checked if it is_prioprity:
https://github.com/tokio-rs/mio/blob/fa4e4b3c58af76909d047abd81d3eaca1d8c5736/src/sys/unix/selector/epoll.rs#L227-L229
But I wonder how to register at the Selector in a way that it receives this type of events?
Looking at https://github.com/tokio-rs/mio/blob/fa4e4b3c58af76909d047abd81d3eaca1d8c5736/src/sys/unix/selector/epoll.rs#L114-L123 it seems, that some kind of  PRIORITY Interest would be needed, but it is not there: https://github.com/tokio-rs/mio/blob/fa4e4b3c58af76909d047abd81d3eaca1d8c5736/src/interest.rs#L19-L34
I searched a bit through the history and found something similar was already there:
https://github.com/tokio-rs/mio/blob/434405a66c66c1e2b15a07388dd0ed4edcc7e3c8/src/event_imp.rs#L613-L622
But it is gone and I don't know why.
So my questions:

Why is the is_priority function there? It is of no use at the moment, right?
What happened to the old Ready:PRIORITY? Is there some reason why this got removed or was it a mistake?
What would be a good (upstreamable) way to implement the support for POLLPRI type events?

EPOLLPRI is enable by default when using readable interest.

EPOLLPRI is enable by default when using readable interest.

Really? I don't see this backed by the code and I do not get any EPOLLPRI events from the kernel.
Have a look here: https://github.com/tokio-rs/mio/blob/1f8cd972f7a575e097c29da1e5aa2d4b7f399d8d/src/sys/unix/selector/epoll.rs#L170-L182
This sets the event type flags up for the syscall. And it does not set EPOLLPRI.
I made a "horrible hack" here: https://github.com/poeschel/mio/commit/538970c98c7dd66ad1c58b185fa14fb91c6467cd
With this change I do receive the EPOLLPRI events.

EPOLLPRI is an output flags only if I recall correctly, so you should see it when use register readable interest. If you want to test this for yourself you can send some Out-Of-Bound (MSG_OOB) data to a TcpStream and you should see an event with EPOLLPRI.

Hello, I am working with @poeschel on this problem. We have created a small test project which can be found here: https://github.com/poeschel/mio-pollpri-test
The program sends messages over a TCP socket and checks the received events. We also send a MSG_OOB message.  There you can see that the MSG_OOB is in fact received, but does not contain the EPOLLPRI flag, and does not have the priority set to true. Output from the program (only the second message to keep it brief) in this case:
[src/main.rs:43] event = Event {
    token: Token(
        1,
    ),
    readable: true,
    writable: false,
    error: false,
    read_closed: false,
    write_closed: false,
    priority: false,           <<<<<
    aio: false,
    lio: false,
    details: epoll_event {
        events: EPOLLIN,       <<<<<
        u64: 1,
    },
}

With the "horrible hack" posted above, the output changes to:
[src/main.rs:43] event = Event {
    token: Token(
        1,
    ),
    readable: true,
    writable: false,
    error: false,
    read_closed: false,
    write_closed: false,
    priority: true,                   <<<<<
    aio: false,
    lio: false,
    details: epoll_event {
        events: EPOLLIN|EPOLLPRI,     <<<<<
        u64: 1,
    },
}

You can see that the flags contains EPOLLPRI and the priority field is set to true. That likely means MSG_OOB was received only because it also contains the EPOLLIN flag.
As mentioned above, we work with V4L2, and our events do only contain EPOLLPRI. Changing the selector to generally also accept EPOLLPRI events like in our hack, seems like a big breaking change, so we are not quite sure how to proceed.

Hello, I am working with @poeschel on this problem. We have created a small test project which can be found here: https://github.com/poeschel/mio-pollpri-test

This program is incorrect. You should get only a single event for the send on line 20. You should drain the readiness, see https://docs.rs/mio/latest/mio/struct.Poll.html#readiness-operations, to get another event.

I  created pull-request #1646 that demonstrates the issue a bit better.

... and I would suggest something like this as a possible solution: https://github.com/poeschel/mio/commit/6a42efe332719b55cceba17cc854a5deabc3f9c0


number limit difference between redis and javascript

Hi,
In redis as per doc http://redis.io/commands/incr
the limit is 64 bit signed integer, max is 2^63 = 9 223 372 036 854 775 807
But for javascript, the max is 2^53 = 9 007 199 254 740 992
http://blog.vjeux.com/2010/javascript/javascript-max_int-number-limits.html
You can see the problem here when I increment the value beyond 2^53, it will not hit error, but it will loose precision.
 Could node_redis return string instead of number to retain the precision if the number return goes beyond the limit? Since the are other tools we can use like bignum to do the operation that we need.
Kind Regards
Andy

This is implemented on master


[FR] FR-010: os-proc-regex supports blacklist

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Manually writing a blacklist regular expression is very difficult for most people, and Rust's regular expression library does not support all regular expression syntax.
It is recommended to add an action configuration item to os-proc-regex, which can be configured as drop to implement the blacklist function.
Prometheus also supports similar features: https://prometheus.io/docs/prometheus/latest/configuration/configuration/#relabel_action
Use case
https://github.com/deepflowys/deepflow/blob/main/server/controller/model/agent_group_config_example.yaml#L986
  ## Example:
  ##   os-proc-regex:
  ##     - match-regex: python3 (.*)\.py
  ##       match-type: cmdline
  ##       rewrite-name: $1-py-script
  ##     - match-regex: (?P<PROC_NAME>nginx)
  ##       match-type: process_name
  ##       rewrite-name: ${PROC_NAME}-%HOSTNAME%
  ##     - match-regex: (sleep|top)
  ##       action: drop
  ##     - match-regex: .*

Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

CC @eddietanshuo


Fix/14681 EOL "more" arrow has wrong placement

Description
Fix a layout issue with the chevron in HomeAppClosureNoticeTableViewCell.
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-14681
Screenshots

❤


how does FRASER compare to SpliSER?

Hi,
thanks for creating such a comprehensive framework for splicing analysis.
I have recently come across SpliSER (https://github.com/CraigIDent/SpliSER) a tool that, I understand, computes splice site strength estimation in a similar way to how you compute your theta values. Do you agree with that or are they conceptually different scores?
Thanks very much! Best,
Miquel

Hi Miquel, we haven't heard of it. As you say, it does sound similar, but we couldn't find exactly how the values are quantified in the repository you sent. There's also no publication linked to it.

Hi Vicente, thanks for your reply. Here's the doi of the publication: https://doi.org/10.1093/nargab/lqab041
I was curious about it because their calculation of SSE looks similar to FRASER's
Thanks!!

Thanks for sharing. I checked the formula in Fig 1 and it seems the metric is a combination of FRASER's theta for the donor and the acceptor sites combined into one.


Add bower.json and register

I wonder if we could add a bower.json and register it with bower? Thanks in advance!

Thanks man

It's registered! Thanks!


AgoraRtcEngineModule undefined

Hi Guys. I am trying to access to NativeModules.AgoraRtcEngineModule but I got undefined. How I can link that AgoraRtcEngineModule to NativeModules. Thanks.
import {NativeModules, findNodeHandle, Platform, UIManager} from 'react-native';
import AgoraRendererView from './AgoraRendererView'
let AgoraRtcEngine = Object.create(NativeModules.AgoraRtcEngineModule);

Do you have copy the agora sdk AograRtcEngineKit.framework file into the ios/RNapi folder ?

Close due to no activity for more than 1 month

Also facing this issue. I added the project file to the libraries and copied the framework file from the pods directory.

TypeError: TypeError: _AgoraRtcEngineModule.default.createEngine is not a function. (In '_AgoraRtcEngineModule.default.createEngine('dd2592e6b5a34c159cc26c30bad84fc3')', '_AgoraRtcEngineModule.default.createEngine' is undefined)
Iam creating basic #AgoraRtcEngine.createEngine('App ID')
importing AgoraRtcEngine from
import AgoraRtcEngine from './AgoraRtcEngineModule'
import AgoraRendererView from './AgoraRendererView'
what went wrong please help me !  @CavanSu @plutoless

@HemanthSurimani  Hi bro, This project already deprecate. We recommend use react-native-agora.

Ho ok Tq @matrixbirds


chore: release 2.1.0

:robot: I have created a release *beep* *boop*
2.1.0 (2022-02-25)
Features

it is no longer necessary to fill in the parameter, if no parameters (05c0f44)

Bug Fixes

generate error when response body is an array (a4cdfc0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/keq-request/keq-cli/releases/tag/v2.1.0 :sunflower:


Task-based login

User story
As a site user, in order to access appropriate section of the portal, I would like to be able to select a task I want to accomplish during the login process.
Acceptance Criteria:

[ ] A user can select a task they want to accomplish in order to login to the portal (wording of the below labels might change):

[ ] Create and manage a challenge
[ ] Manage submissions and build evaluations
[ ] Evaluate submissions
[ ] Solve challenges

[ ] A user is logged in to appropriate section of the portal based on their selected task and assigned role

[ ] Create and manage a challenge => challenge manager => Phoenix app (#13 )
[ ] Create and manage an evaluation => challenge manager => Rails app (#13 )
[ ] Review and evaluate submissions => evaluator => Rails app (#16 )
[ ] Submit a solution to a challenge => public solver => Phoenix app (#23 )

[ ] A user is logged in to all site sections (based on their role) with a single login

Definition of Done
Doing (dev team)

[ ] Code complete
[ ] Code is organized appropriately
[ ] Any known trade offs are documented in the associated GH issue
[ ] Code is documented, modules, shared functions, etc.
[ ] Automated testing has been added or updated in response to changes in this PR
[ ] The feature is smoke tested to confirm it meets requirements
[ ] Database changes have been peer reviewed for index changes and performance bottlenecks
[ ] PR that changes or adds UI

[ ] include a screenshot of the WAVE report for the altered pages
[ ] Confirm changes were validated for mobile responsiveness

[ ] PR approved / Peer reviewed
[ ] Move card to testing column in the board

Testing (dev team)

[ ] Security scans passed
[ ] Automate accessibility tests passed
[ ] Build process and deployment is automated and repeatable
[ ] Feature toggles if appropriate
[ ] Deploy to staging

Staging

[ ] Accessibility tested (Marni)

[ ]  Keyboard navigation
[ ]  Focus confirmed
[ ] Color contrast compliance
[ ] Screen reader testing

[ ] Usability testing: mobile and desktop (Tracy or Marni)
[ ] Cross browser testing (tool to be determined) (Tracy or Marni)

[ ]  UI rendering is performant

[ ] AC review (Renata)
[ ] Deploy to production (production-like environment for eval capability) (dev team)
[ ] Move to production column in the board

Production

[ ] User and security documentation has been reviewed for necessary updates (Renata/Tracy/Dev team)
[ ] PO / PM approved (Jarah or Renata)
[ ] AC is met and it works as expected (Jarah or Renata)
[ ] Move to done column in the board (Jarah or Renata)

A question for login.gov:

can they filter state and local accounts

@stepchud Here's a task-based login diagram to help visualize this user workflow:
Task-based login diagram_1.pdf

Figma link: https://www.figma.com/design/VfdBFG8ShVnOStdSNYy9fR/Challenge.Gov-Component-Library?node-id=29-3530&t=STCLmrRsXj3cnZKx-0

@stepchud If there aren't any concerns with this story, should we move it back to the sprint backlog?


CI "unexpected end of file" in slugrunner based job

ControllerSuite.TestExampleOutput failed with:
    t.Assert(errs, c.HasLen, 0, c.Commentf("%s validation errors: %v\ndata: %v\n", cacheKey, errs, string(jsonData)))
... obtained []jsonschema.ValidationError = []jsonschema.ValidationError{jsonschema.ValidationError{Keypath:[]string{"response", "body"}, Description:"Additional properties aren't allowed, found \"message\" as one of its keys"}}
... n int = 0
... https://flynn.io/schema/examples/controller/job_update validation errors: [{[response body] Additional properties aren't allowed, found "message" as one of its keys}]
data: {
	"request": {
		"body": {
			"app": "6030514e9e394e699624c8e61caf0a15",
			"id": "192081b6-45c4437d8631433dbe327b9afd4479db",
			"release": "e53440ec01c948a58e637525b4830617",
			"state": "down"
		},
		"headers": {
			"Authorization": "Basic OjZhMDM3ZmU2NTFiNGY5MWE=",
			"Content-Type": "application/json"
		},
		"method": "PUT",
		"url": "/apps/6030514e9e394e699624c8e61caf0a15/jobs/192081b6-45c4437d8631433dbe327b9afd4479db"
	},
	"response": {
		"body": {
			"code": "validation_error",
			"detail": {
				"field": "state"
			},
			"message": "state pq: invalid job state transition: crashed -\u003e down"
		},
		"headers": {
			"Content-Type": "application/json"
		}
	}
}

The problem here is that the controller example tries to set the job state to down, but the job was in the crashed state (leading to an invalid state transition) due to the following error in it's log:
HostID: 3d89767a - flynn-host log --init 192081b6-45c4437d8631433dbe327b9afd4479db
...
gzip: stdin: unexpected end of file
tar: Child returned status 1
tar: Error is not recoverable: exiting now

https://s3.amazonaws.com/flynn-ci-logs/20150223001629-720e9a9c-build-121f09d518886c9ba7e635c082c240c9a0ae1ee4-2015-02-23-00-25-49.txt

@jvatic even though there was an underlying issue, I think the response from the controller was in fact valid, does the schema check need updating?

@lmars it's testing the response body against website/schema/controller/job.json. The test is not currently aware of error responses which can make the output of such failures confusing.

https://s3.amazonaws.com/flynn-ci-logs/20150226202401-fbd7841e-build-38707ef28bdf489d97e3fea69e2a0d501906133a-2015-02-26-20-34-20.txt

https://ci.flynn.io/builds/20150228053200-3a10d6dc

https://ci.flynn.io/builds/20150310193522-64b520dc

This hasn't happened recently.


Add formatting to the deno.json file

Why? To make deno fmt work as well as IDE-specific auto formatting features.
Suggestion based on the current coding style:
  "fmt": {
    "indentWidth": 2,
    "semiColons": true,
    "singleQuote": false,
    "useTabs": false
  }

Running this against the current code base shows minimal changes (only dangling commas are added in ts files) as well as a double quote usage in the README.
Happy to do a PR if needed.

Oh, this should actually all be formatted using the default Deno settings. Feel free to submit a PR (without the fmt config) which is just formatting changes.

fair enough. I'll do when I get some time.


InvalidCreatePolicyAssignmentRequest - The policy definition specified in policy assignment 'Deny-Subnet-Without-Nsg' is out of scope.

Describe the bug
I received the following error during deployment of Enterprise-scale for Azure Government today. It may be a fluke but figured I would report it. I have a workaround noted in this report.
{
"status": "Failed",
"error": {
"code": "DeploymentFailed",
"message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.",
"details": [
{
"code": "BadRequest",
"message": "{\r\n  "error": {\r\n    "code": "InvalidCreatePolicyAssignmentRequest",\r\n    "message": "The policy definition specified in policy assignment 'Deny-Subnet-Without-Nsg' is out of scope. Policy definitions should be specified only at or above the policy assignment scope. If the management groups hierarchy changed recently or if assigning a management group policy to new subscription, please allow up to 30 minutes for the hierarchy changes to apply and try again."\r\n  }\r\n}"
}
]
}
}
Steps to reproduce

Deploy Enterprise-scale for Azure Government.

I had pretty much all options set at default in the wizard except for the following options which were declined:

Deploy CI/CD Pipeline
Azure DDoS Protection Standard
Azure Firewall
Azure VPN Gateway

The deployment failed after about 20 minutes with the below screenshot and included error.

Workaround
I was able to get past this issue by running a new deployment with all of the exact same settings right over the top of what had already been deployed. It sounds like the operation failed because it was attempted too quickly after a management group was created.
Screenshot

Thanks - and glad it got sorted. Did you by any chance re-deploy this initially, using a management group prefix you have used in the past?

Yes, I was using the same prefix from past runs for the initial deploy as well as the re-deploys.

I've just had the same issue whilst deploying Enterprise-scale hub and spoke today, and yes I think I may have used the same prefix I have used in a past deployment that has since been deleted. Deploying again over the top of the initial deployment completed successfully.
"message": "{\r\n  "error": {\r\n    "code": "InvalidCreatePolicyAssignmentRequest",\r\n    "message": "The policy definition specified in policy assignment 'Deny-Public-Endpoints' is out of scope. Policy definitions should be specified only at or above the policy assignment scope. If the management groups hierarchy changed recently or if assigning a management group policy to new subscription, please allow up to 30 minutes for the hierarchy changes to apply and try again."\r\n  }\r\n}"

Hey @dalejmarshall,
Have you been able able to retry by deploying it with exactly the same input parameters as the first run, this should resolve the issue👍
You may need to wait 5-10 mins, or even 30 as the error states, before trying the redeployment to allow the backend to update fully.
Let us know

Hi @jtracey93 , yes I have thank you. It worked after leaving it for around 15 minutes. Thanks.

This issue cannot be solved within the ESLZ implementation as it requires backend work by Azure engineering. Leaving this open with "wontfix" label in case others are running into the same.

Closing this to track in master issue #902 for hygiene


[download-meta] Restyling

Describe your changes
Restyling download meta page

Issue ticket number and link
Checklist before requesting a review

[ ] I have read and understand the Contributions section
[ ] Target branch is releaseCandidate and not master
[ ] I have performed a self-review of my code
[ ] I ran the unit tests and my PR does not break any tests
[ ] I documented the changes I've made on the CHANGES.md and followed actual conventions
[ ] I added a new section on how-to.md (optional)

#271

May you put them on 2/3 columns?
It would make it easier to find the required metadata.


[3plus] Add new information extractor

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x] New extractor
[ ] New feature

Description
3 Plus is a Swiss private TV station. They publish their own TV production on their website. It would be nice to have youtube-dl support for it. Here is a suggestion for a information extractor.

3qsdn is already supported.

Hi dstftw,
I've seen that you have "self-assigned" this pull request. Does this mean that you will take care of this, or should I try to improve the mentioned points anyway? Thanks.

This means I'll take care of merging this PR.

I know it's old but because I'm currently using the tool to download something from the site I wanted to ask for an update on this? The download seems to work with the generic extractor but there is probably still room for improvement.

@heubergen This is a very old pull request. I have just had a look at 3plus.tv and it looks completely different than it used to look, so I assume that these changes won't work anymore anyway. If the download of videos works with the generic extractor, that's great. Do you miss anything? If so, you can open a feature request to support [3plus.tv] natively.


Domain activation is Failed - Trying to assign a subdomain to a new AWS Amplify webapp

App Id: d20el7a7f463ab
Region: us-east-1
Step: Domain activation
Status: Failed
Issue/question
The CNAME gets verified, but I get this error message, in spite of this being the one and only CNAME I have pointing to AWS: "Error message: One or more of the CNAMEs you provided are already associated with a different resource."
Error message
"Error message: One or more of the CNAMEs you provided are already associated with a different resource."
Additional information
New to this, wondering if I might be missing something obvious. The AWS Amplify webapp works well, but with the native AWS URL. All I am trying to do is use it with my own domain.
Thanks!

I followed the troubleshooting guide, thanks. No luck though. I do not have any distribution on cloudfront.
https://console.aws.amazon.com/cloudfront/home?#
Two things that may be relevant though:
(1) I have a custom subdomain setup with Meteor Cloud.
(2) I also have another subdomain setup with Mongo Atlas (Cloud service for MongoDB).

I followed the troubleshooting guide, thanks. No luck though. I do not have any distribution on cloudfront.
https://console.aws.amazon.com/cloudfront/home?#
Two things that may be relevant though:
(1) I have a custom subdomain setup with Meteor Cloud.
(2) I also have another subdomain setup with Mongo Atlas (Cloud service for MongoDB).

This helped me. Mine was failing due to a cloudfront distribution using the same domain I want to use in amplify under its CNAME. Even though distribution was disabled, I still had to remove the CNAME from it.

@carlosalvidrez I see on our end that your domain association is currently waiting to validate that a CNAME exists pointing to the cloudfront distribution for your app. You should create an entry in your DNS records like this:
www.example.com       CNAME          123456789.cloudfront.net <--- replace with the correct URL for your app

Thanks Fernando... as soon as I deactivated Mongo Atlas hosting (I don't handle those things directly, they are a bit of a black box to the end-user), the thing worked. They must have their own cloudfront records for it.
I appreciate the help anyway.
Cheers


Verification result when the token is expired

I'm wondering what's the proper way to verify a token and get the decoded version even if it's expired. My current use case requires me to use some data inside the token object both when it's expired and when it's still active, as long as the token is technically valid. The questions that come specifically to my mind are:

If I use verify() without options.throwError set to true and the token is expired but all the other factors (like the structure and signature) are valid, would the response be the decoded token object or undefined?
Alternatively, when options.throwError is set to true and an EXPIRED error is thrown, does this mean the token is technically valid but just the exp is inferior than now, or there could be other invalid things?

An example for how to correctly read the data inside the token object both when it's valid+active and valid+expired and handle both cases would be highly appreciated!
PS. sorry for mixing up the terms valid and expired but that's the best way I come up with to formulate my question.

Using decode() you can always extract the header and payload from the token, regardless if it has expired or not.


[API Store] [UI] Regenerate button is disabled after generating a token and show keys button is clicked

Regenerate button to regenerate a token is disabled when the show keys button is clicked after generating a token for an application. The Regenerate button is re enabled only when the page reloads.
Steps to reproduce

Create a new application in store

Generate tokens by clicking "Generate keys" button in the "Production Keys" tab

Now we can observe that the "Regenerate" button is enabled

Now click "Hide Keys" button

Then the "Regenerate" button is automatically disabled

The will not be re-enabled even if we click "Show Keys" button

The issue is fixed only when we refresh the web page

Fixed in https://github.com/wso2-support/carbon-apimgt/pull/1532


IIS 10 + ModSec 2.9.1 breaks websocket/socket.io even if turned off

IIS 10 + socket.io = OK
Install mod sec 2.9.1 x64
Use default config file
Socket.io = Broken (can't connect from client)
Turn off Mod Sec in Server = no change
Turn off Mod Sec in web.config in site = no change
Uninstall Mod Sec = IIS broken (some config files broken)
Reinstall Mod Sec = see step 3
Pls help

Found one error : when uninstalling modsec (windows server/IIS), there are some leftovers in applicationHost.config (<ModSecurity enabled=false"/>) if you have set that rule per site and not globally. This causes IIS to refuse to start with an error 500.
That being fixed, my sockets are now properly working again (running on port 8080).

Hi @globules-io, looks like the uninstaller is a bit buggy. I'll have a look at it.
In the meantime, were you able to solve your issue? If not, please try going over these instructions to see if it solves your problem:
https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual#installation-for-microsoft-iis
https://github.com/SpiderLabs/ModSecurity/wiki/IIS-Troubleshooting

I could solve the issue by removing by hand any mention of ModSecurity inside applicationHost.config. My only issue left is that, for some reason, ModSec blocks another process running on port 8080 (socket.io). Note that socket.io here is not handled by IIS so any modification to IIS should not have effect on this I suppose. So my only issue is how to allow that process on 8080 while ModSec is installed on IIS?

Any update on this? Any of it solved?

Hi @globules-io
As a follow up from uninstallation issues: When uninstalling, please try to run the .msi as Administrator or using msiexec. See https://github.com/SpiderLabs/ModSecurity/issues/1868#issuecomment-435532593 for more information.
As for the issue you described about ModSec blocking other process on a different port, I could not reproduce it here. I'm wondering if it's any particularity on your environment?
Please check if with a fresh installation of Windows + IIS + ModSecurity 2.9.3 + Socket.io the issue stills persists. If so, please let us know and we can reopen the issue to investigate further if it's an issue with ModSecurity for IIS.
Thanks.


Projection error on collection field

My code:

Here is the sql HQL generated:
SELECT          prnindexcl0_.prn    AS col_0_0_,
prnindexcl0_.NAME   AS col_1_0_,
.                   as col_2_0_,
tags2_.id           AS id1_2_,
tags2_.created_time AS created_2_2_,
tags2_.prn          AS prn3_2_,
tags2_.tag          AS tag4_2_,
tags2_.updated_time AS updated_5_2_
FROM            prn_index prnindexcl0_
LEFT OUTER JOIN element_tag tags1_
ON              prnindexcl0_.prn=tags1_.prn
INNER JOIN      element_tag tags2_
ON              prnindexcl0_.prn=tags2_.prn
WHERE           prnindexcl0_.DELETE<>?
ORDER BY        prnindexcl0_.create_time DESC
I got this error:
"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'as col_2_0_, tags1_.id as id1_2_, tags1_.created_time as created_2_2_, tags1_.pr' at line 1"
I'm confused with "." column in the select statement. what is that mean?

Hi, I'm running into the same issue, did you manage to find a way around?

*.tags sounds like a list/1:n maybe this is a problem here.
Just select the tag and join the Prn to it. Stream it into a map with prn as key and you get all the tags for one prn.


Add who started poll and when

lol, copying my code

https://github.com/panpawn/Pokemon-Showdown/commit/5840eefb78bd07f48e9eea6634c373b21a5c9f7f
you didn't copy and paste right, either

nah it isnt
i used moment coz Tolls.duration fails

Except you used moment wrong. Without actually adding moment to package.json, this code will crash.
And you'd be better off figuring out why it failed than bringing in an outside dependency. It might be because you spelled Tools wrong.


Assert in SRE during Boo compiler run with Mono 6.x

I am the package maintainer of boo in Fedora.
For a while, there is an issue that the build does not work anymore, with Mono 6.x.
This is the bug in Fedora: https://bugzilla.redhat.com/show_bug.cgi?id=1799197
I have reported it upstream: https://github.com/mono/mono/issues/18970 but that did not cause any action (yet).
I can reduce it to these steps:
cd /root/rpmbuild/BUILD/boo-unstable
nant -D:install.prefix=/usr -D:docs.dir=/usr/share/doc -D:install.booexamples=/usr/share/boo/examples
mono build/booc.exe -v -noconfig -debug -out:build/Boo.NAnt.Tasks.dll -target:library -r:NAnt.Core.dll -r:NAnt.DotNetTasks.dll -srcdir:src/Boo.NAnt.Tasks -lib:/usr/lib/NAnt/bin/

Start of the output:
Boo Compiler version <IP_ADDRESS> (<IP_ADDRESS> (tarball Sat Jun 27 04:51:00 UTC 2020))
* Assertion at sre-encode.c:290, condition `count > 0' not met

see attached files for details.
boo_build_failure.txt
Is there anything that could be done from the side of boo?

it looks quite similar to https://xamarin.github.io/bugzilla-archives/45/45136/bug.html but is probably different.

Yeah, from the build log, that really does look like a project in the Mono runtime.


Download File Only

Hello. Is there any way to download the embedded file only?
Because the artist that I'm supporting only post the preview on the image section and the 'real thing' on the embedded file.
Thanks in advance.

I also want this

v0.26.1 now provides --skip-images flag to download files only.

thanks, love your work


Implemented functionality from dir_writable to dir

When variable POWERLEVEL9K_DIR_SHOW_WRITABLE is set to true, dir prompt
will have new state NOT_WRITABLE.
This state will be set when user doesn't have permission to write in
current working directory as in prompt dir_writable.
When mode that allows icons is set, LOCK_ICON is used.
Closes #588.

Nice work, @tippl! I'm really excited to see someone take up the torch from #588 :)
A few code review comments:

We need to make sure that POWERLEVEL9K_DIR_SHOW_WRITABLE is set to a default value. Check out some of the other segments, and you can see how we use a P9k utility function set_default to do that before the function.
Be sure to document this change in the README! We want users to know about this cool new feature you are adding =)
Since this is a new feature, it actually needs to get merged into the next branch rather than master (https://github.com/bhilburn/powerlevel9k/wiki/Developer's-Guide). Would you mind rebasing your changes onto next and then re-submitting the PR? Let us know if you need help with this!

I'm really excited about getting this in. Thanks for taking it on! =)

@bhilburn This should address all your comments. I was thinking of adding the lock icon to the table at the start of the dir section, but i had no idea on how to upload an image to github.

Excellent work, @tippl! Nice work incorporating the review feedback and updating the PR. This is a great addition to the project.
Welcome to P9k as a contributor =)


Add missing @throws to docblocks

Related: @return type in NotFound.php had wrong types in docblock (fixed)
Closes #2666

Coverage remained the same at 97.314% when pulling 1b6df21d3461ee22f670124c85d239e7168c70ec on RyanNerd:add-throws-2-doc-blocks into 833fea9b839f7792da7c73fe829c986ce5b99b8e on slimphp:3.x.


Safari: isFreemium() on null

I got this error in Safari when I open a link with the Billable middleware attached.
Error: Call to a member function isFreemium() on null in /vendor/osiset/laravel-shopify/src/ShopifyApp/Http/Middleware/Billable.php:50

In Google Chrome it works perfectly.
Do you have an idea if it has something to do with CORS?

 My guess is you're not detected as being logged in thus the user is null, thus isFreemium doesnt work. I am not too sure how to fix this at the moment.

 I have no support for this at the moment, its my next initiative .. closing until I can add support.

See issue #522 as follow up

I am having the same bug as mentioned TS but it is happened in every browser even on main page of the app.
Package version is:
"osiset/basic-shopify-api": "^7.0",  "osiset/laravel-shopify": "^10.3.1",

I am having the same bug as mentioned TS but it is happened in every browser even on main page of the app.
Package version is:
"osiset/basic-shopify-api": "^7.0",  "osiset/laravel-shopify": "^10.3.1",


Logos

Update the logo to the version as supplied by Jonah in SVG format to remove the "Online"
Resolves #57

Done!