Named RateLimit Middleware not working

Laravel Version: 9.6.0 | 9.7.0
PHP Version: 8.0.14
Database Driver & Version: MySQL 5.7

Description:
Based on the request throttling documents, it says you can place the RateLimiters inside the configureRateLimiting function under App\Providers\RouteServiceProvider (created function, not currently there).
However, it appears to not take effect. The rate limiter doesn't respect my configuration and seems to be running some defaults (limit of 1)
Steps To Reproduce:

Make function configureRateLimiting in App\Providers\RouteServiceProvider
Add the following for a rule

RateLimiter::for('public-registration', function (Request $request) {
            return config('app.env') == 'local'
                ? Limit::none()
                : Limit::perMinute(200);
        });

Add middleware(['throttle:public-registration']) to a route
Load the route, then refresh again right away
Triggers the throttle exception

I expect for local environment to never trigger throttle as per the rule defined.
Am I missing something here or is there actually a bug? Followed what the documentation says https://laravel.com/docs/9.x/routing#defining-rate-limiters.
Stepped through the code as it executed and it appears to not hit the return inside this block
if (is_string($maxAttempts)
            && func_num_args() === 3
            && ! is_null($limiter = $this->limiter->limiter($maxAttempts))) {
            return $this->handleRequestUsingNamedLimiter($request, $next, $maxAttempts, $limiter);
        }

The ->limiter doesn't appear to return anything so its like its not actually registering the middleware rule.

Update:
I added the Rule to boot() and it appears to be loading it up.
Question is should that work as intended without any effects, and if so, documentation should probably be updated unless its something else which I've missed.

Well... great
Yeah i'm adding it in now
Upgrading the project from a previous version, and I did go through and overwrite all the main files, including providers and configs and bring our changes back in so its inline with updates made.
Thanks @ankurk91


Create a SECURITY_CONTACTS file.

As per the email sent to kubernetes-dev[1], please create a SECURITY_CONTACTS
file.
The template for the file can be found in the kubernetes-template repository[2].
A description for the file is in the steering-committee docs[3], you might need
to search that page for "Security Contacts".
Please feel free to ping me on the PR when you make it, otherwise I will see when
you close this issue. :)
Thanks so much, let me know if you have any questions.
(This issue was generated from a tool, apologies for any weirdness.)
[1] https://groups.google.com/forum/#!topic/kubernetes-dev/codeiIoQ6QE
[2] https://github.com/kubernetes/kubernetes-template-project/blob/master/SECURITY_CONTACTS
[3] https://github.com/kubernetes/community/blob/master/committee-steering/governance/sig-governance-template-short.md

@jessfraz I would say we can safely close this as we already resolved this by #495. Can you close this?


Stash index Zi show error

Description
when i try to stash 1 file when i have other files modified,
when just only 1 file modified that i can use Zz and stash it without any error, but Zi still have the same error
Neovim version
NVIM v0.7.0
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>Operating system and version
macOS M1 Montery 12.4
Steps to reproduce
open neogit status
stash unstaged changes with index ( Z i )
Expected behavior
stash only file i have cursor on
Actual behavior
Error

Minimal config
vim.cmd([[set runtimepath=$VIMRUNTIME]])
vim.cmd([[set packpath=/tmp/nvim/site]])
local package_root = "/tmp/nvim/site/pack"
local install_path = package_root .. "/packer/start/packer.nvim"
local function load_plugins()
  require("packer").startup({
    {
      "wbthomason/packer.nvim",
      {
        "TimUntersberger/neogit",
        requires = {
          { "nvim-lua/plenary.nvim" },
          { "sindrets/diffview.nvim" },
        },
        config = function()
          print("loaded neogit")
          require("neogit").setup()
        end,
      },
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. "/plugin/packer_compiled.lua",
      display = { non_interactive = true },
    },
  })
end
if vim.fn.isdirectory(install_path) == 0 then
  print("Installing neogit and dependencies.")
  vim.fn.system({ "git", "clone", "--depth=1", "https://github.com/wbthomason/packer.nvim", install_path })
end
load_plugins()
require("packer").sync()

@TimUntersberger i have also observed when pressing x key on a stash, It is prompting with Discard confirmation. but, opting "Yes" does not delete it. is it working as expected for you?


Companion app occasionally fails.

At least once a day the updates for BG stop getting sent to the watch.  When this happens the graph goes blank, the most recent value spot goes blank but the counter for time since last update continues to run.  Tracker values for steps and heartrate continue to update normally.
When this happens the settings option has disappeared in the companion app.  The only way to address this is to re-establish dev link on phone and watch and re-push the app and companion.  I'm not yet ready to attribute this to code quite yet as I suspect it is a side-loading artifact or the fitbit development environment oddity.

Hey, welcome back! I think we can close this one as with newer ionic firmwares and other dev env changes it seems to have gone away. Also, you hang in the fitbit sdk discord server?

If you do, poke NiVZ about nightscout.....


test build failed in jenkins in windows

installed jdk8 through choco
downloaded Jenkins.war
Jenkins setup is done..while creating new item with freestyle project created shell file as shown in example, while building it is saying build failed..

Your executing shell script in windows

if you are using windows select "execute windows batch command" instead of "execute shell" in build and you can find commands that works on windows from this link click here

yes
try execute batch commands

Select execute batch file give the script to there it will execute

Resolved


how to display key path and language context together

in v1.15.5

in v2.1.4

how to display together, thanks

"i18n-ally.annotationInPlace": false,

it work, but only display key when select line

Fixed in 2.1.5, please try it out :)

thanks, it fixed!


feat(SIP-39): Async query support for charts

SUMMARY
Adds support for running chart queries in async workers, as proposed by SIP-39, under a new GLOBAL_ASYNC_QUERIES feature flag.

Formalizes async event publishing using Redis Streams, with this implementation using a new /api/v1/async_event/ endpoint for polling.
Adds async event middleware to superset-frontend to poll for events when the feature flag is enabled and there are chart components in a loading state.
Adds a new JWT token cookie (name configurable) to the client for validating async requests.
This PR affects charts using both the new /api/v1/chart/data API and the legacy /superset/explore_json endpoint in both dashboards and Explore (behind GLOBAL_ASYNC_QUERIES feature flag).
Abstracts and moves set_and_log_cache out of viz.py.

Remaining issues:

Multi-line chart not rendering correctly (component will need updating as it does its own data fetching)

TODO in separate follow-up PRs:

Query cancellation for async queries (see limitations in SIP-39)
Support async operation for CSV, raw results and sample data
Add support for websocket event handling on the frontend
Update SQL Lab async query execution to use new async architecture (backwards compatible)

TEST PLAN

Enable the GLOBAL_ASYNC_QUERIES feature flag in your test environment and ensure correct chart rendering in both dashboards and Explore
Ensure synchronous chart rendering with the feature flag disabled functions correctly.

ADDITIONAL INFORMATION

[x] Has associated issue: https://github.com/apache/incubator-superset/issues/9190
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[x] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

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

@@            Coverage Diff             @@
##           master   #11499      +/-   ##
==========================================
- Coverage   66.63%   62.70%   -3.93%
==========================================
  Files         863      445     -418
  Lines       41235    15155   -26080
  Branches     3721     3721
==========================================
- Hits        27476     9503   -17973
+ Misses      13657     5473    -8184
- Partials      102      179      +77

Flag
Coverage Δ

#cypress
?

#javascript
62.70% <ø> (ø)

#python
?

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

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

I'm currently unable to reproduce any of these CI errors. It looks like it may be running cached code?

@ktmud thanks for the feedback! Some answers below, others inline:

Errors are included with the async event payload, and should render as they do with synchronous requests (see attached screenshot below). What kind of error are you seeing a load issue with? The example below uses a virtual dataset with a bad SQL query.
Query cancellation support in Superset is limited (see SIP-39), but I've added it to the TODO items for follow-up PRs

Async chart error example:

Oh, just found out why I was forever in pending state: I didn't have my Celery worker running. Might want to mention this somewhere in the doc.
Managed to make it work but I feel the round trip for the polling might have made the page appear slower, especially when cache already exists. Can we let explore_json and chart/data API return the cached query results directly when the cache key exists?

@villebro the last commit (066504f) should take care of the component not found for job_id console warning

Managed to make it work but I feel the round trip for the polling might have made the page appear slower, especially when cache already exists. Can we let explore_json and chart/data API return the cached query results directly when the cache key exists?

I had a similar experience, and this could be a best of both worlds solution. @robdiciuccio does this make sense to you?

@villebro the last commit (066504f) should take care of the component not found for job_id console warning

Thanks, I'll try it out again 👍

Managed to make it work but I feel the round trip for the polling might have made the page appear slower, especially when cache already exists. Can we let explore_json and chart/data API return the cached query results directly when the cache key exists?

I did some performance testing on my local machine this morning with the "USA Birth Names" example dashboard, and the timing differences are fairly negligible (see below).
Async:

Sync:

I'm curious what your celery concurrency is set to @ktmud @villebro. These are the settings used in this test:
celery worker --app=superset.tasks.celery_app:app --pool=threads -c 20 --loglevel=DEBUG

Keep in mind also that the ultimate goal here (with SIP-39) is to use websockets as the event transport, which will eliminate the delay between polling requests, and the authentication overhead therein. The plan was to include short polling as a fallback if running a websocket server is not an option for whatever reason. As an MVP, it made sense to leverage polling to test the full e2e solution without introducing additional dependencies.
To the point about returning cached data with the initial request, I'll look into the refactoring work necessary to accomplish this (as the caching logic currently lives in the get_payload methods which execute the queries), but architecturally, I think there's value in having consistent return values from the APIs regardless of the cache status. I suggest we revisit this once websockets are introduced, and we can do some additional performance testing.

Overall, looks good! One thing I noticed was a bit of hand-rolling around JWT handling. I suggest leaning on flask-jwt-extended for this, along with the ability for other teams to customize their "jwt manager" as to adhere to their internal use cases (flask-jwt-extended allows for customization of all things JWT related via a set of config options). I can see wanting to keep this functionality separate from other global JWT handling. At the very least, it would be great to make the security-related functionality pluggable so that it can be easily overridden and extended

OK @villebro @ktmud @dpgaspar @craig-rueda finally got all tests passing, and have addressed the majority of the feedback here. How are folks feeling about merging this MVP, with the understanding that this feature is still experimental and will continue to be iterated upon?

Can we update the JWT structure at least?


HYD-6355 For testing purposes limit tests that are run

It would be nice and resource friendly to be able to limit the tests we
run when trying to diagnose a problem.
Allow two commit message pragmas "Run-tests" and "Skip-tests" to specify
a list of tests to either run or skip.

enhance check_for_autopass() to look for a commit pragma and limit
tests being run based on that
pass the tests that were skipped
reorder passes so that distribution support is before gui-bump

Signed-off-by: Brian J. Murrell<EMAIL_ADDRESS>

This is a backport of 5e7e164d91fb36e01f6b4143d5b223d640a2c7eb from b4_0:
Change-Id: Ibd8f915bd90f376810275e9e85584328a91f6c14
Reviewed-on: http://review.whamcloud.com/22757
Tested-by: Jenkins
Tested-by: Chroma Test User
Reviewed-by: Joe Grund<EMAIL_ADDRESS>Reviewed-by: Chris Gearing<EMAIL_ADDRESS>
Per our recent policies, it probably doesn't even really need reviewing considering it's been reviewed and landed already.  But I did have to change the test names slightly in this patch to match the new test names on the new Jenkins so I thought it ought to be reviewed at least for that.
But to answer some questions:

so this allows us to Skip a selection of tests,

Yes.

presumably this doesn't allow patches to land without passing all tests,

Correct.  It fails tests that are skipped.

it's just for debug purposes on a patch?

Yes.  For when you have a failure that only happens in CI that you need to debug and don't want to waste time and resources running the whole battery of tests when you are just interested in one or two.

or does this allow landing a patch which only lands a subset of tests?

It can't since any tests skipped report a failure.

there are no tests for this change, how do we know we've verified most of the potential codepaths that might occur?

There is only one (new) codepath and that's to exit a test with a failure if it's on on the skip list.  What kind of test (which would be a test of a test) should we add?

there might be unintended results of this change.

If you can think of any, we could write a test for them but as I can't think of any, I can't imagine what kind of test we need to add.


Add tree-sitter syntax highlighting support

This PR adds parser-based syntax highlighting using the new tree-sitter support in Emacs 29.  The mode automatically switches to tree-sitter-based highlighting when it is available, falling back to the old version otherwise.
The PR also rearranges some code, and adds some missing keywords.
The README is updated with instructions for enabling the new feature; installation is easy provided that one has compiled a version of Emacs with tree-sitter support.  The mode uses the MLang tree-sitter grammar from my repository, which we may want to consider adding to the miking group on Github as well.
In more detail, the tree-sitter library enables Emacs to parse code files incrementally and provide syntax highlighting based on the actual syntax tree of the file, rather than simply highlighting select keywords or regexes.  This means we can easily highlight e.g. the function in an application or the name and parameters in a declaration.
The two images below give a peek at how the highlighting looks before and after.

One can control the level of fontification using the treesit-font-lock-level variable (which is an integer in the range [1,4]); the screenshot above uses level 3.
I also want to recommend the excellent treesit-explore-mode which can be used to explore a syntax tree interactively (although this is not directly connected to the PR).

This looks great! I'm looking forward to playing with it.


Added struct for pet, modified contract functions

Added a new struct of Pet to better define the kind of pet, modified functions accordingly, added a new function to get unadopted pets, made some other modifications for better security and gas optimizations.

The contract was tested on remix for quick test, for hardhat test, a new pr can be made.

Overall did it worked with the reach app? if yes attach a screenshot or video of the UI.

I haven't integrated the changes with the frontend yet, I would if this pr is accepted because since it is dealing with the contract itself, any change would have to be implemented for the frontend again.
The frontend is still able to render though.

Overall did it worked with the reach app? if yes attach a screenshot or video of the UI.

I haven't integrated the changes with the frontend yet, I would if this pr is accepted because since it is dealing with the contract itself, any change would have to be implemented for the frontend again.
The frontend is still able to render though.

Well, I don't think so. I think the contract and the frontend should be in sync, so do the changes which are required. Don't take stress about the labels and all, these things are for learning only.

Ok sure, will create a new PR with the frontend integrated too. However, please take care of the labels since it was told by a mentor in wob discord that contributions will be counted through labels. Thank you.

Ok sure, will create a new PR with the frontend integrated too. However, please take care of the labels since it was told by a mentor in wob discord that contributions will be counted through labels. Thank you.

Sorry, I can't merge any PR before I ensure it end to end working. Otherwise the project will go unstructured, I don't want that.

Ok sure, will create a new PR with the frontend integrated too. However, please take care of the labels since it was told by a mentor in wob discord that contributions will be counted through labels. Thank you.

Sorry, I can't merge any PR before I ensure it end to end working. Otherwise the project will go unstructured, I don't want that.

Yes, I meant a new pr including the frontend and contract changes. This one can be closed.


pkg/redirectpolicy: Make code robust against incorrect policy configu…

See commit message.
Fixes: #16204

@aditighag hey, what's the status of this PR? I've been waiting for it to be marked ready for review before having a look.

test-only --focus="K8sServicesTest.* LRP" --kernel_version="net-next"

test-only --focus="K8sLRPTests" --kernel_version="net-next"

Focused tests relevant to the code changes (LRP tests) have passed.

Focused tests relevant to the code changes (LRP tests) have passed.

No need to re-run the tests since the latest commits addressed a simple formatting review comment - https://github.com/cilium/cilium/pull/16216#discussion_r673697235.

test-only --focus="K8sLRPTests" --kernel_version="net-next"

Focused LRP tests have passed - https://jenkins.cilium.io/job/Cilium-PR-Tests-Kernel-Focus/293/.
Based on the out-of-band comment from Jussi, I've added a comment in the commit description and the code. No need to re-run the test.

checkpatch and a connectivity test are failing


[WIP] Add test for podiooutput with collections only in first event

BEGINRELEASENOTES

This test creates some collections only in specific events, then uses PodioOutput to write the events.
It will crash on writing the collections.

ENDRELEASENOTES

Podio assumes all collections will be there at the moment.

This is a PR with more than two years and still hasn't been merged, so I'm closing it.

This will also not be easy to implement with ROOT as an I/O backend, since that assumes the same structure for every event.


Updated language definitions and translation engine for new steps

Creating and sending a PR for updated TagUI language definitions and translation engine for the new steps (keyboard, mouse) and helper functions (mouse_xy(), mouse_x(), mouse_y()) created in #370, #368, #361 and #365.
This lets users use the new steps and helper functions in 21 different languages. More details on the multiple languages feature of TagUI in below link of documentation -
https://github.com/kelaberetiv/TagUI#native-languages

Committed to master. Prior to packaged release, feature available from cutting edge version here -
https://github.com/kelaberetiv/TagUI#set-up


Fix completion related issues

@ethagnawl if it wasn't noticeable, my comment https://github.com/tmuxinator/tmuxinator/pull/706#issuecomment-546596771 had a patch that would close #706 and #705. But this PR includes an update to the README as well.

Thanks a bunch for this!  My shell now loads without an error message, and I'm a very happy chappy 😄


Apiary Blueprint Docs: Lack of line feeds on attribute example text

In the below screenshot, you'll see that the "Example" is always appended to the end of the line, and there seems to be no way to force it onto a new line. Having it appended to the end doesn't feel clean and makes for much harder reading. Ideally it would put this info on a new line by default.
+ Parameters
    + user_id: 161yTi2eZvKYlo2CHUYEN9jP (string) User ID
    + card_id: 00AA11CCDDEE (string) - Card ID (serial number)

Hey @foxx,
Thanks for the suggestion. We will look into this and get back to you.

To be addressed in #191


reduce_string_templates reports issues for string templates with variable padding

The code
REPORT zfoobar.

PARAMETERS p_width TYPE i.

DATA(str) = |{ '' WIDTH = p_width PAD = '=' }|.
WRITE str.

produces the issue "/zfoobar.prog.abap[5, 16]: Constant string in text template, reduce(reduce_string_templates)".
However, this is not a constant string template that can be easily reduced since the WIDTH is not a constant. In fact, this is the most ergonomic way I know to generate a sequence of a single character with variable length.

@bjoern-jueliger-sap try again, 2.100.6 published

Thanks, works now as expected

fyi
DATA(str) = repeat( val = `=` occ = p_width ).

https://help.sap.com/doc/abapdocu_latest_index_htm/latest/en-US/index.htm?file=abenrepeat_functions.htm


Error when trying to install on web server Please help!

error: could not create '/usr/lib/python2.6/site-packages/youtube_dl_server': Read-only file  system
I am using a shared hosting so i could  not use sudo, any suggestions?

Probably you are trying to install https://github.com/jaimeMF/youtube-dl-api-server, report the issue there.

it does the same thing when i use youtube-dl
error: could not create '/usr/lib/python2.6/site-packages/youtube_dl': Read-only file system

Since you haven't post the command you are running and the full output, we can't provide full help. If you are using pip install youtube-dl, you either need to run it with sudo or use the --user option.

When I use the command pip install youtube-dl this is the full output i get: Downloading/unpacking youtube-dl
Running setup.py egg_info for package youtube-dl
Installing collected packages: youtube-dl
Running setup.py install for youtube-dl
error: could not create '/usr/lib/python2.6/site-packages/youtube_dl': Read-only file system
Complete output from command /usr/bin/python -c "import setuptools;file='/home3/nekle/build/youtube-dl/setup.py';exec(compile(open(file).read().replace('\r\n', '\n'), file, 'exec'))" install --single-version-externally-managed --record /tmp/pip-GJTTWZ-record/install-record.txt:
running install
running build
running build_py
running install_lib
creating /usr/lib/python2.6/site-packages/youtube_dl
error: could not create '/usr/lib/python2.6/site-packages/youtube_dl': Read-only file system

Command /usr/bin/python -c "import setuptools;file='/home3/nekle/build/youtube-dl/setup.py';exec(compile(open(file).read().replace('\r\n', '\n'), file, 'exec'))" install --single-version-externally-managed --record /tmp/pip-GJTTWZ-record/install-record.txt failed with error code 1 in /home3/nekle/build/youtube-dl
Storing complete log in /home3/nekle/.pip/pip.log
I cannot run sudo since its a shared hosting and is not letting me use it. How can i use the --user option?

I cannot run sudo since its a shared hosting and is not letting me use it. How can i use the --user option?

pip install --user youtube-dl?

Thank you so much !! it worked!! you saved my life :)


Prevents code from running early

Added a safety to prevent the script from running while the game hasn't loaded properly yet, which could result in some minor issues.
Added a check for the alertOnRun variable, so that it actually disables the alert when set to false.

Shouldn't need anymore


[Update Request]: Update TheDocumentFoundation.LibreOffice to Version 7.6

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
TheDocumentFoundation.LibreOffice
Package Version
7.6
Please describe the changes you would like to see
A new Update 7.6 is released:
https://wiki.documentfoundation.org/ReleaseNotes/7.6
Download:
https://www.libreoffice.org/download/download-libreoffice/

thanks to @krystofkrticka for providing update in #117603 👍


add "and" and "or" aggregators

I would suggest adding a test for and/or to https://github.com/brimsec/zq/blob/master/reducer/reducer_test.go#L73 which exercises the xPart() calls in multiple configurations (and likewise for collect/union but I missed those trains...)

Will do but I prefer it as a ztest... coming up!

Just realize I had the spill trigger wrong in the union PR.  I'll fix and make sure both spill tests work exercise the partials path of union/logical/collect.

Will do but I prefer it as a ztest... coming up!

if a ztest can cover the conditions that reducer_test.go, I will be pleased to eat my hat.

Will do but I prefer it as a ztest... coming up!

if a ztest can cover the conditions that reducer_test.go exercises, I will be pleased to eat my hat.

My bad for not writing tests that covered the partials path on union and collect.  See PR #1503.  Do you take sauce on your hat?

@mccanne: no hats have been served yet... this doesn't exercise all the conditions covered in reducer_test.go.

@henridf: RE: "undocumented"... is this is a reference to user-facing ZQL docs?

@philrz yes that's what I meant by "undocumented". I wouldn't suggest putting it in the docs at this point since I don't see a use for it. I think it's more of an "advanced tuning" option if anything.

Thanks @henridf. That's what I guessed, but wanted to be sure. I know we discussed -sortmem recently and reached a similar conclusion. I figure once we have some experience building our own setups that tune all these settings to work at scale we could work on an advanced config guide with best practices for users that actually want to try to run this stuff themselves at scale with proper tunings.


Failed to install the apk after merged

While trying to install the app via adb
01.778 I: [MERGE] Writing apk ...
05.458 [MERGE] Writing: total=35638710 bytes : play-services-tasks.properties
05.558 I: [MERGE] Zip align ...
05.639 I: [MERGE] Saved to: out.apk
05.640 I: [MERGE] Done

└─$ adb install out.apk
Performing Streamed Install
adb: failed to install out.apk:
Failure [INSTALL_PARSE_FAILED_NO_CERTIFICATES: Failed collecting certificates for /data/app/vmdl808427888.tmp/base.apk:
Failed to collect certificates from /data/app/vmdl808427888.tmp/base.apk: Attempt to get length of null array]

Sign the apk , will solve this
Can you automate the signing process just after the merge ?

The main objective of this project is to promote and provide application example of ARSCLib but survey likes this project than ARSCLib :) .
Let see too many people request such feature :)

Does manifest protect auto implement after execute res protect with the new update ? @REAndroid

Does manifest protect auto implement after execute res protect with the new update ? @REAndroid

Yes planning to change to user optional from CLI like -protect-manifest


problem with logic of rasa'rule

Rasa version: 2.5.0
Rasa SDK version (if used & relevant): 2.6.0
Rasa X version (if used & relevant):
Python version: 3.7.9
Operating system (windows, osx, ...): Ubuntu 18.04
Issue:
Rasa did not the output what I want in the conversation. Check the rule.yml file below and here is my expected conversation:
- user: X
- rasa: Ok, now you are at Y
- user: other
- rasa: Nothing here

But in fact, when I entered "X", rasa did not reply anything.
To reproduce result:

run rasa train -c config.yml -d domain.yml --out models/ --augmentation 0 --data data/ .
run rasa run actions and rasa shell --debug in two seperate terminals.

In my understanding, rasa should executes in following steps:

Recognize user message 'X' as intent_x intent
Go to action_goto_Y
Assign Slot goto = Y (return [SlotSet("goto", "Y")])
Since the slot goto = Y, the condition of rule: goto Y is met, so rasa outputs action utter_wentto_Y
...
...

Did my understanding go wrong ? please explain to me clearly . Thanks a lot ~
Error (including full traceback):

Command or request that led to error:

Content of configuration file (config.yml) (if relevant):
language: en
pipeline:
  - name: WhitespaceTokenizer
    token_pattern: (?u)\b\w+\b
  - name: CountVectorsFeaturizer
  - name: DIETClassifier
    number_of_transformer_layers: 1
    number_of_attention_heads: 1
    epochs: 100
policies:
  - name: RulePolicy
    restrict_rules: true
    check_for_contradictions: true

Content of domain file (domain.yml) (if relevant):
version: '2.0'
config:
  store_entities_as_slots: true
session_config:
  session_expiration_time: 480
  carry_over_slots_to_new_session: true
intents:
  - intent_X
  - intent_other
slots:
  goto:
    type: addons.my_custom_slots.goto
    influence_conversation: true
actions:
  - action_goto_Y
  - utter_wentto_Y
responses:
  utter_wentto_Y:
    - text: "Ok, now you are at Y"
  utter_other:
    - text: "Nothing here"

Content of Slot file (addons/my_custom_slots.py) (if relevant):
from rasa.shared.core.slots import Slot

class goto(Slot):
    def feature_dimensionality(self):
        return 2

    def as_feature(self):
        r = [0.0] * self.feature_dimensionality()
        if self.value:
            if self.value == "Y":
                r[0] = 1.0
            else:
                r[1] = 1.0
        return r

Content of Rule file (data/rules/rule.yml) (if relevant):
version: "2.0"
rules:
  - rule: my_rule
    steps:
      - intent: intent_X
      - action: action_goto_Y
    wait_for_user_input: false

  - rule: goto Y
    condition:
      - slot_was_set:
          - goto: Y
    steps:
      - action: utter_wentto_Y
      - intent: intent_other
      - action: utter_other

Content of NLU file (data/nlu/nlu.yml) (if relevant):
version: "2.0"
nlu:
  - intent: intent_X
    examples: |
      - X
      - XX
      - XXX
  - intent: intent_other
    examples: |
      - other
      - others

Content of action file (actions/actions.py) (if relevant):
from typing import List, Text
from rasa_sdk import Action, Tracker
from rasa_sdk.types import DomainDict
from rasa_sdk.executor import CollectingDispatcher
from rasa_sdk.events import (
    SlotSet,
    EventType,
)

class ActionGoToY(Action):
    def name(self) -> Text:
        return "action_goto_Y"

    def run(
            self,
            dispatcher: CollectingDispatcher,
            tracker: Tracker,
            domain: DomainDict,
    ) -> List[EventType]:
        return [SlotSet("goto", "Y")]

Thanks for the issue, @erohmensing will get back to you about it soon!
You may find help in the docs and the forum, too 🤗


Ref: Use match statements

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Used match statements

I personally think that this is a net-negative change in terms of code readability for most programmers, so I would prefer not to incorporate this. I appreciate the attempt at the contribution, but I think we should close this for now unless any other maintainers feel strongly that this is a positive change.

@frankxu2004
https://github.com/OpenDevin/OpenDevin/blob/de74c7a0a1a3a18f8a68ef28ab5710f6d48729ba/agenthub/browsing_agent/prompt.py#L492
@iFurySt https://github.com/OpenDevin/OpenDevin/blob/de74c7a0a1a3a18f8a68ef28ab5710f6d48729ba/evaluation/agent_bench/helper.py#L61


[Mention The Feature]:  Adding favicon in login page

Is your feature request related to a problem? Please describe.
favicon in login page is not added
Describe the solution you'd like.
adding a favicon will enhance website more
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots

Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

@charrviwadhwa  assigning this issue to you.
Make sure to complete this fast.
As I have said earlier this project and I are going to be inactive from today night
Reason being due to some of my works.
This will be the only issue that I am gonna assign you.
I hope you understand.

@charrviwadhwa assigning this issue to you.
Make sure to complete this fast.
As I have said earlier this project and I are going to be inactive from today night
Reason being due to some of my works.
This will be the only issue that I am gonna assign you.
I hope you understand.


Heuristics implementation

This is an initial implementation for computing heuristic decision rules.
We have to discuss the arrangement of our original struct_env: shall we slightly modify it or just inherit it here?
In this case, we need to retrieve inspection observations (detected, no-detected).

For the struct_env_heurisitc, either we inherit from struct env and we only modify the step function or we can modify the step function of struct_env directly and check its use in the other classes.

I am ok with this one


Let users see clip thumbnail same as regular thumbnail

Currently the preview thumbnail for the clips is different than that for the main gallery preview. This was done on purpose, but I could see how someone would prefer the same screenshot in both.
This would probably be a super-easy thing to toggle in the settings 👍

Thank you @martincaron for implementing this feature (#422) 🎉 🙇  🥇


Cannot import roslib.message

Python 2.7.15rc1 (default, Nov 12 2018, 14:31:15)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> from rospy_message_converter import message_converter
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/pablo/src/freedomrobotics/freedom_python/env/local/lib/python2.7/site-packages/rospy_message_converter/message_converter.py", line 34, in <module>
    import roslib.message
ImportError: No module named roslib.message

I just created a simple virtualenv and installed the library with pip.
Is there anyother package that should be installed for running this? It would be nice to add it to the README.

@cjds I've installed both pyros and the pip version of the message converter, however it still does not work out of the box. Is there something I'm misisng?
It would be realy nice to be able to use this library without a full ros installation.

@annesteenbeek Sorry again for the extremely late reply (not been as on top of notifications as I should have). I am currently working on a re-release of this library that will not have the above traceback.
It's caused due to a change in requirements from catkin_pkg and the fact that I last released this 2 years ago. When I do the update I will let you know


feat(solc): Strip experimental pragma from all imported contracts

Motivation

forge verify-contract failed because flatten doesn't strip the experimental pragma from imported contracts.
Related issues:
https://github.com/gakonst/foundry/issues/852
https://github.com/gakonst/ethers-rs/issues/1115
Solution

This PR enable solc.flatten to strip pragma from all imported contracts.
Fix #1115
PR Checklist

[x] Added Tests
[ ] Added Documentation
[ ] Updated the changelog

@mattsse @gakonst is there any simple way to test this out directly on foundry?
AFAIK foundryup only support installing forge using specific github repo, cannot specify which ethers-rs is used.

nvm guys, I just learned that I can patch the ethers-rs using my local repos. https://github.com/gakonst/foundry/blob/6130af7d3fd581c22b42d232011fa1576bfc6ac0/Cargo.toml#L25-L32
btw here is the example contract successfully verified after stripping the experimental pragma
https://arbiscan.io/address/0x1B82DF960F5470a17606abAd2E541CC552870098#code
Previously it always failed to verify coz flatten doesn't strip the experimental pragma on imported files

@mattsse ty ser!


test_resnet50_untilize_with_halo_and_conv_v2 sproadically failing with watcher enabled in stress test pipeline

We recently enabled watcher on stress test (24h of post commit in a loop) pipelines and got several consecutive runs to pass (last one being most recent):
https://github.com/tenstorrent-metal/tt-metal/actions/runs/7430070758
https://github.com/tenstorrent-metal/tt-metal/actions/runs/7441497962
https://github.com/tenstorrent-metal/tt-metal/actions/runs/7456505379
However, stress test on GS started failing here:
https://github.com/tenstorrent-metal/tt-metal/actions/runs/7470195336
Around this time, CI on other pipelines were also experiencing issues, so this may be a regression issue.
Passing commit: https://github.com/tenstorrent-metal/tt-metal/commit/828ccd648e3bccc20f8a60877a7ed62210ae1a1b
Failing commit: https://github.com/tenstorrent-metal/tt-metal/commit/0dd85e4e9d33ff8f945598be4b6263596be3558b
2024-01-11T09:07:34.1246428Z FAILED tests/tt_eager/python_api_testing/unit_testing/test_resnet50_untilize_with_halo_and_conv_v2.py::test_resnet50_conv[LoFi-activations_BFLOAT8_B-weights_BFLOAT16-64-16-115-115-4-4-1-1-0-0-batch_20] - AssertionError: Output should be identical to old conv!
2024-01-11T09:07:34.1246548Z assert False

I have been able to repro this locally on a VM by running the stress test locally
@pgkeller @mywoodstock

cc: @TT-BrianLiu

Stress testing failed again on same test for GS https://github.com/tenstorrent-metal/tt-metal/actions/runs/7483006866/job/20367564268

Is this still failing @TT-billteng?

Seeing hanging issues on same test here on N150 machines
https://github.com/tenstorrent-metal/tt-metal/actions/runs/8262747162/job/22602917413

triaging this into the kernel op bucket now that we have that process.


Overlays

The commits here implement overlays, that is, the ability to draw alpha-blended overlays on top of the camera image in the various types of preview window, without changing the actual camera image underneath. This entire feature is made up of 6 commits:

The first commit implements overlays in the OpenGL preview window. There's also a very small amount of plumbing just to send the overlay from the Picamera object to the preview.
The second commit repeats this work for the simple Qt software-rendered version of the preview window.
Next we do it all again for the DRM/KMS version of the preview.
Finally we do the same for the NullPreview. Of course this doesn't do anything, we just want it to have the same API.
This commit adds some simple overlay examples and also puts them into the standard test list.
The final commit updates the docuementation in the README.md file.

@chrisruk Here's a much less trivial pull request to look at when you have time! Probably a lot of it won't mean that much (TBH it wouldn't have meant much to me a week ago) but a second set of eyes is probably good nonetheless. Thanks!

I've not used the OpenGL etc. bindings before, but been looking through the commits, which look good to me, will merge now. Cheers.


Clicking in overview bar does not draw annotations in raw.plot

If plotting a Raw object with annotations, these are not drawn in the main window when navigating through the data by clicking on the little overview bar. It's probably just an update() call that's missing. For a MWE, the example in from #8686 can be used. @drammock do you have time to look?

Yes, I'll look. I actually noticed this myself late on Friday so it was already on my to-do list for today 🙂


Using apt to install k4a package in a non-interactive environment fails

Describe the bug
If you attempt to install the Debian K4A package using apt (or dpkg) the install we fail by default. This is due to the interactive EULA which users need to accept (due to guidance from Microsoft) to install the Debian package.
A common scenario for this would be attempting to use apt to install libk4a1.4 in a Docker container.
To Reproduce

Create a Dockerfile with a step to run apt-get install libk4a1.4
Build that docker image
See error -->Sub-process /usr/bin/dpkg returned an error code (1)

Desktop (please complete the following information):

OS with Version: Ubuntu 18.04

This is a known issue due to the EULA we are required to present to users. We have a work around to this issue.
The EULA acceptance is done using Debconf. You can manually set entries in the debconf database to accept the EULA before installing the debian package.
The steps to do this are as follows (please substitute in the correct version of the package):

Set the environmental variable DEBIAN_FRONTEND to noninteractive. If you are in a Dockerfile you can do this with the ARG command.
ARG DEBIAN_FRONTEND=noninteractive

This tells dpkg and debconf that you are in a non-interactive environment.

Manually set the hash of the accepted EULA in the debconf database.
echo 'libk4a1.4 libk4a1.4/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | sudo debconf-set-selections

NOTE: That hash may change if the EULA ever changes.

Manually set that the user has accepted the EULA.
echo 'libk4a1.4 libk4a1.4/accept-eula boolean true' | sudo debconf-set-selections

Install the libk4a package. In the stdout from the install process you should see

License has already been accepted

We are looking at adding support for allowing users to set an environmental variable to also skip this.
This would look something like
ACCEPT_EULA=Y apt install libk4a1.5

Note: You'd have to be root to get this to work as I don't think sudo would pass the proper environmental variables.

This seems to work for libk4a1.x, but I don't have the same luck with libk4abt1.0.

ENV DEBIAN_FRONTEND=noninteractive
RUN echo 'libk4a1.3 libk4a1.3/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | debconf-set-selections
RUN echo 'libk4a1.3 libk4a1.3/accept-eula boolean true' | debconf-set-selections
RUN echo 'libk4abt1.0 libk4abt1.0/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | debconf-set-selections
RUN echo 'libk4abt1.0 libk4abt1.0/accept-eula boolean true' | debconf-set-selections
RUN apt-get install -y --no-install-recommends k4a-tools=1.3.0
libk4a1.3-dev
libk4abt1.0-dev

Gives me:

0 upgraded, 13 newly installed, 0 to remove and 193 not upgraded.
Need to get 0 B/446 MB of archives.
After this operation, 777 MB of additional disk space will be used.
Selecting previously unselected package libk4a1.3.
(Reading database ... 109924 files and directories currently installed.)
Preparing to unpack .../libk4a1.3_1.3.0_amd64.deb ...
License has already been accepted
Unpacking libk4a1.3 (1.3.0) ...
Setting up libk4a1.3 (1.3.0) ...
Selecting previously unselected package libk4abt1.0.
(Reading database ... 109935 files and directories currently installed.)
Preparing to unpack .../libk4abt1.0_1.0.0_amd64.deb ...
eula agree question could not be asked
libk4abt1.0 will not be installed
dpkg: error processing archive /var/cache/apt/archives/libk4abt1.0_1.0.0_amd64.deb (--unpack):
new libk4abt1.0 package pre-installation script subprocess returned error exit status 2
Selecting previously unselected package libk4a1.3-dev.
Preparing to unpack .../libk4a1.3-dev_1.3.0_amd64.deb ...
Unpacking libk4a1.3-dev (1.3.0) ...
Errors were encountered while processing:
/var/cache/apt/archives/libk4abt1.0_1.0.0_amd64.deb

Any advice?

This seems to work for libk4a1.x, but I don't have the same luck with libk4abt1.0.

ENV DEBIAN_FRONTEND=noninteractive
RUN echo 'libk4a1.3 libk4a1.3/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | debconf-set-selections
RUN echo 'libk4a1.3 libk4a1.3/accept-eula boolean true' | debconf-set-selections
RUN echo 'libk4abt1.0 libk4abt1.0/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | debconf-set-selections
RUN echo 'libk4abt1.0 libk4abt1.0/accept-eula boolean true' | debconf-set-selections
RUN apt-get install -y --no-install-recommends k4a-tools=1.3.0
libk4a1.3-dev
libk4abt1.0-dev

Gives me:

0 upgraded, 13 newly installed, 0 to remove and 193 not upgraded.
Need to get 0 B/446 MB of archives.
After this operation, 777 MB of additional disk space will be used.
Selecting previously unselected package libk4a1.3.
(Reading database ... 109924 files and directories currently installed.)
Preparing to unpack .../libk4a1.3_1.3.0_amd64.deb ...
License has already been accepted
Unpacking libk4a1.3 (1.3.0) ...
Setting up libk4a1.3 (1.3.0) ...
Selecting previously unselected package libk4abt1.0.
(Reading database ... 109935 files and directories currently installed.)
Preparing to unpack .../libk4abt1.0_1.0.0_amd64.deb ...
eula agree question could not be asked
libk4abt1.0 will not be installed
dpkg: error processing archive /var/cache/apt/archives/libk4abt1.0_1.0.0_amd64.deb (--unpack):
new libk4abt1.0 package pre-installation script subprocess returned error exit status 2
Selecting previously unselected package libk4a1.3-dev.
Preparing to unpack .../libk4a1.3-dev_1.3.0_amd64.deb ...
Unpacking libk4a1.3-dev (1.3.0) ...
Errors were encountered while processing:
/var/cache/apt/archives/libk4abt1.0_1.0.0_amd64.deb

Any advice?

I have found a workaround for libk4abt (it needs a different hash). Here is a snippet from my Dockerfile that seems to work for Ubuntu 20.04 base images:
RUN apt-get install -y curl
# Install k4a SDK from their repository.
# Bit of a hack, see https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/1263
# The simple solution by @vinesmsuic does not seem to work. This uses the manual
# solution by @atinfinity
# Also, https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/1190 is part of this solution (needed to non-interactively accept EULA)

RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/libk/libk4a1.3/libk4a1.3_1.3.0_amd64.deb > /tmp/libk4a1.3_1.3.0_amd64.deb
RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/libk/libk4a1.3-dev/libk4a1.3-dev_1.3.0_amd64.deb > /tmp/libk4a1.3-dev_1.3.0_amd64.deb
RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/libk/libk4abt1.0/libk4abt1.0_1.0.0_amd64.deb > /tmp/libk4abt1.0_1.0.0_amd64.deb
RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/libk/libk4abt1.0-dev/libk4abt1.0-dev_1.0.0_amd64.deb > /tmp/libk4abt1.0-dev_1.0.0_amd64.deb
RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/k/k4a-tools/k4a-tools_1.3.0_amd64.deb > /tmp/k4a-tools_1.3.0_amd64.deb
RUN echo 'libk4a1.3 libk4a1.3/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | debconf-set-selections
RUN echo 'libk4abt1.0	libk4abt1.0/accepted-eula-hash	string	03a13b63730639eeb6626d24fd45cf25131ee8e8e0df3f1b63f552269b176e38' | debconf-set-selections
RUN dpkg -i /tmp/libk4a1.3_1.3.0_amd64.deb
RUN dpkg -i /tmp/libk4a1.3-dev_1.3.0_amd64.deb
RUN dpkg -i /tmp/libk4abt1.0_1.0.0_amd64.deb
RUN dpkg -i /tmp/libk4abt1.0-dev_1.0.0_amd64.deb
RUN apt-get install -y libsoundio1
RUN dpkg -i /tmp/k4a-tools_1.3.0_amd64.deb

If I might add, I was able to also use the yes command in conjunction with DEBIAN_FRONTEND to get an automated workflow. Here's what worked for me:
yes yes | DEBIAN_FRONTEND=dialog apt-get install -y /tmp/*.deb

Thank you very much @jackjansen!!!
Your comment helped me solve my issue with the build of my Dockerfile.
Looks like there are newer .debs available:
libk4a1.3_1.3.0_amd64.deb > libk4a1.4_1.4.1_amd64.deb
libk4a1.3-dev_1.3.0_amd64.deb > libk4a1.4-dev_1.4.1_amd64.deb
libk4abt1.0_1.0.0_amd64.deb > libk4abt1.1_1.1.0_amd64.deb
libk4abt1.0-dev_1.0.0_amd64.deb > libk4abt1.1-dev_1.1.0_amd64.deb
k4a-tools_1.3.0_amd64.deb > k4a-tools_1.4.1_amd64.deb

How did you obtain the following EULA hash strings?
RUN echo 'libk4a1.3 libk4a1.3/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | debconf-set-selections
RUN echo 'libk4abt1.0	libk4abt1.0/accepted-eula-hash	string	03a13b63730639eeb6626d24fd45cf25131ee8e8e0df3f1b63f552269b176e38' | debconf-set-selections

I would like to obtain new EULA hash strings for the new .debs.
Thanks again! Greatly appreciate your comments.

How did you obtain the following EULA hash strings
Once you manually accept EULA, you can see those hash by typing sudo debconf-show libk4a1.3, sudo debconf-show libk4abt1.0

Thank you very much @jackjansen!!! Your comment helped me solve my issue with the build of my Dockerfile.
Looks like there are newer .debs available:
libk4a1.3_1.3.0_amd64.deb > libk4a1.4_1.4.1_amd64.deb
libk4a1.3-dev_1.3.0_amd64.deb > libk4a1.4-dev_1.4.1_amd64.deb
libk4abt1.0_1.0.0_amd64.deb > libk4abt1.1_1.1.0_amd64.deb
libk4abt1.0-dev_1.0.0_amd64.deb > libk4abt1.1-dev_1.1.0_amd64.deb
k4a-tools_1.3.0_amd64.deb > k4a-tools_1.4.1_amd64.deb

How did you obtain the following EULA hash strings?
RUN echo 'libk4a1.3 libk4a1.3/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | debconf-set-selections
RUN echo 'libk4abt1.0	libk4abt1.0/accepted-eula-hash	string	03a13b63730639eeb6626d24fd45cf25131ee8e8e0df3f1b63f552269b176e38' | debconf-set-selections

I would like to obtain new EULA hash strings for the new .debs.
Thanks again! Greatly appreciate your comments.

Thanks for the solutions. I found that for different version of libk4a and libk4abt the EULA hash strings is exactly the same so we can directly use the hash strings by @jackjansen.

I have found a workaround for libk4abt (it needs a different hash). Here is a snippet from my Dockerfile that seems to work for Ubuntu 20.04 base images:
RUN apt-get install -y curl
# Install k4a SDK from their repository.
# Bit of a hack, see https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/1263
# The simple solution by @vinesmsuic does not seem to work. This uses the manual
# solution by @atinfinity
# Also, https://github.com/microsoft/Azure-Kinect-Sensor-SDK/issues/1190 is part of this solution (needed to non-interactively accept EULA)

RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/libk/libk4a1.3/libk4a1.3_1.3.0_amd64.deb > /tmp/libk4a1.3_1.3.0_amd64.deb
RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/libk/libk4a1.3-dev/libk4a1.3-dev_1.3.0_amd64.deb > /tmp/libk4a1.3-dev_1.3.0_amd64.deb
RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/libk/libk4abt1.0/libk4abt1.0_1.0.0_amd64.deb > /tmp/libk4abt1.0_1.0.0_amd64.deb
RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/libk/libk4abt1.0-dev/libk4abt1.0-dev_1.0.0_amd64.deb > /tmp/libk4abt1.0-dev_1.0.0_amd64.deb
RUN curl -sSL https://packages.microsoft.com/ubuntu/18.04/prod/pool/main/k/k4a-tools/k4a-tools_1.3.0_amd64.deb > /tmp/k4a-tools_1.3.0_amd64.deb
RUN echo 'libk4a1.3 libk4a1.3/accepted-eula-hash string 0f5d5c5de396e4fee4c0753a21fee0c1ed726cf0316204edda484f08cb266d76' | debconf-set-selections
RUN echo 'libk4abt1.0	libk4abt1.0/accepted-eula-hash	string	03a13b63730639eeb6626d24fd45cf25131ee8e8e0df3f1b63f552269b176e38' | debconf-set-selections
RUN dpkg -i /tmp/libk4a1.3_1.3.0_amd64.deb
RUN dpkg -i /tmp/libk4a1.3-dev_1.3.0_amd64.deb
RUN dpkg -i /tmp/libk4abt1.0_1.0.0_amd64.deb
RUN dpkg -i /tmp/libk4abt1.0-dev_1.0.0_amd64.deb
RUN apt-get install -y libsoundio1
RUN dpkg -i /tmp/k4a-tools_1.3.0_amd64.deb

You may have to install these dependencies first.
RUN apt-get -y install python3.9-dev libgl1 libxrandr2 libxinerama1 libxcursor1


Outdated Readme

After merging #198 the real-life-example got moved but the readme has stayed the same. This makes it really hard to figure out how to run those in the current master branch.

@IanCStewart  do you know how?

I am closing the Issue as a part of an effort to cleanup and revive the project. If this issue persists after v1.0.0, you are more than welcome to reintroduce the issue.
Thanks for your contribution.


5 Column layout ?

I noticed that making a 5 column layout (full width), is impossible hard.
Bootstrap has the same issue and must be tweaked to support it.
https://stackoverflow.com/questions/10387740/five-equal-columns-in-twitter-bootstrap
One solution to this is to do like what pure-css does and implement multiple 5ths and 24ths based grids.
Maybe we could look into implementing something like this for beauter ?
Chris

Hello Chris,
Thank you for opening such an important and useful issue. Of course this is the power of open source, improving beauter together, I welcome your suggestion. Forming 5 and 7 columns layout would be a great pain indeed.
I saw how bootstrap and purecss have implemented their solution. Is having a custom grid generator plugin (just as purecss also provide) a nice idea, considering beauter as a CSS only framework? Coz this would add a lot of flexibility to the designer. (At the cost of JavaScript)
So Chris, what's your take on this issue? It would be nice of you to help us implementing such things properly for beauter 3.0 updates :)

Well, as you said I am not sure creating a grid generator is the way to go.
Maybe simply creating 5ths-Based Units and 24ths-Based Units column grid layouts ?
They could compliment the current 12 unit base grid.
Ex:
m1-5 to m5-5    => medium col 1 to 5 of a 5 column layout
m1-24  to m24-24 => medium col 1 to 24 of a 24 column layout
Any thoughts ?
Chris

I clearly got your idea for complementing the current 12 unit based grid with 5 and 24 based. But I am thinking about the possibilities and (difficulties) we can achieve on having JavaScript working behind. Not only for grids, but in general. Although I considered Beauter to be a vanilla CSS Initially, I believe hardly any site would exist without using JavaScript now a days. Specially concerned about size, there is a lot we can save and make things more DRY.
What's your opinion on Dynamic styling using JavaScript for keeping Beauter lighter and flexible?

Although beauter is pure css by source, it uses javascript for many of its elements like responsive navigations, alerts, modals etc. And creating them with pure CSS is a pain in neck with compatibility issues. Hence for full utilization of beauter, user always has to use javascript along side beauter css. Having a javascript generator would not that much of trouble.

I agree with @vyasnavneet adding some sort of basic JavaScript to implement standard features would be nice.
As for the dynamic grid system not too sure, we could maybe use css variables.
I found this medium article with a working prototype.
https://medium.com/@fionnachan/dynamic-number-of-rows-and-columns-with-css-grid-layout-and-css-variables-cb8e8381b6f2
But support for variables is not 100%
http://caniuse.com/#feat=css-variables
Maybe we can inspire ourselves with it an implement a js version if it is more flexible I am not sure.
Any thoughts ?
Chris

I found @chrisribe suggestion the best yet. We shall add new classes for new sizes. 5 columns or 7 or 24 etc.

JavaScript will add a render delay due to dynamic styling and this will not be convenient for many browsers.
CSS variables are still facing compatibility issues.
Using hacks like offsets etc (commonly used with bootstrap grid) is a bit complex, I don't consider it EASY for a beginner, considering beauter a simple framework.

Any thoughts ?


[EPIC] Git performance & UX

Can you guys share some context here? Missed this on our planning call because of network issues.

@abhvsn @AnaghHegde @IAmAnubhavSaini @hiteshjoshi can we close this epic if we reached the goals that we set out for it? We could move the remaining items to the next epic to bring git version control out of beta.

Yes, let's close it. We are not going to get any more time on it. Next git efforts will be on gitV2

I agree with @vuiets. We can close this and move the remaining issue to v2.

Yeah I will just need to move the issues to Git V2.


[Doc] Add some faq

Proposed changes
Issue Number: close #xxx

add some sql-faq : "s3 load ssl cert"

run buildall

run buildall


2.1.5版本拍照成功 上传服务器 显示图片空白

UIScrollView *scrollView = [[UIScrollView alloc] initWithFrame:self.bounds];
scrollView.alwaysBounceVertical = YES;
scrollView.scrollEnabled = NO;
[self.image_pho_view addSubview:scrollView];
self.scrollView = scrollView;
CGFloat width = scrollView.frame.size.width;

HXPhotoView *photoView = [[HXPhotoView alloc] initWithFrame:CGRectMake(kPhotoViewMargin, kPhotoViewMargin-5, (width - kPhotoViewMargin * 3)/1.5, 0) manager:self.manager];
photoView.spacing = 4;
photoView.lineCount = 4;
photoView.spacing = kPhotoViewMargin;

photoView.delegate = self;
photoView.backgroundColor = [UIColor whiteColor];
[photoView refreshView];
[scrollView addSubview:photoView];
self.photoView = photoView;

相机拍的照片直接用model.thumbPhoto  或者 model.previewPhoto  这个


Looking forward to support for LLM on Android

Just letting you know I am watching your progress.

How is it going?
I think you are not able to use external storage on Android, but you could use the /mnt/sdcard for some internal storage on the (if available) sdcard?
You might find this interesting if you are trying to use a small llama-compatible model:
https://learn.microsoft.com/en-us/dotnet/maui/platform-integration/storage/file-system-helpers?view=net-maui-8.0&tabs=android


Go back to handling CH companies by company_number

Can one of the admins verify this patch?

Codecov Report

Merging #110 into master will increase coverage by <.01%.

@@            Coverage Diff            @@
##           master    #110      +/-   ##
=========================================
+ Coverage   93.79%   93.8%   +<.01%
=========================================
  Files          28      28
  Lines        1016    1017       +1
  Branches       46      46
=========================================
+ Hits          953     954       +1
  Misses         55      55
  Partials        8       8

Impacted Files
Coverage Δ

datahub/company/views.py
100% <100%> (ø)
:white_check_mark:

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 1a641ca...4c31fa9. Read the comment docs.


Compatible with Swift 2.2

Current coverage is 79.34%

Merging #270 into master will not affect coverage as of 353be1a

@@            master    #270   diff @@
======================================
  Files           12      12
  Stmts         1375    1375
  Branches         0       0
  Methods          0       0
======================================
  Hit           1091    1091
  Partial          0       0
  Missed         284     284

Review entire Coverage Diff as of 353be1a

Powered by Codecov. Updated on successful CI builds.


BoxNotFound(moov) error in mp4::Mp4Reader::read_header

.m4a files saved by QuickTime Player result in a BoxNotFound(moov) error when running mp4::Mp4Reader::read_header.
The file can be played in QuickTime Player, Chrome and VLC just fine.
The moov doesn't seem to be missing:

And output from ffprobe seems ok:
  Metadata:
    major_brand     : M4A
    minor_version   : 0
    compatible_brands: M4A mp42isom
    creation_time   : 2021-01-31T05:24:23.000000Z
    iTunSMPB        :  00000000 00000840 00000240<PHONE_NUMBER>7CA580 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000
  Duration: 00:03:05.23, start: 0.047891, bitrate: 127 kb/s
    Stream #0:0(und): Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 126 kb/s (default)
    Metadata:
      creation_time   : 2021-01-31T05:24:23.000000Z
      handler_name    : Core Media Audio

Here's the m4a file:
https://user-images.githubusercontent.com/11315492/106375747-5441be80-638f-11eb-84fe-80629af044de.mp4

Wait a minute, it is actually missing in the screenshot I took, my bad. But, it does still plays fine and such.
Looks like the moov is near the bottom of the file:

Wait a minute, it is actually missing in the screenshot I took, my bad. But, it does still plays fine and such.
Looks like the moov is near the bottom of the file:

Good find. I think it's a bug when parsing a largesize mdat. I'll check it out.

Good find. I think it's a bug when parsing a largesize mdat. I'll check it out.

@probablykasper Can you try the branch in the PR? #49

@probablykasper Can you try the branch in the PR? #49

@alfg Sure, but will have to do it tomorrow

@alfg Sure, but will have to do it tomorrow

@probablykasper No worries. Thanks for finding this. Have a good night.

@probablykasper No worries. Thanks for finding this. Have a good night.

@alfg Works like a charm! And thanks for also adding Freq7350

@alfg Works like a charm! And thanks for also adding Freq7350

@probablykasper Great! It's merged. I'll be bumping the mp4 crate version later today.

@probablykasper Great! It's merged. I'll be bumping the mp4 crate version later today.

Also, this helped me fix the issue I was having with qt files on https://alfg.github.io/mp4-inspector/ 😄

@alfg Oh shit, found that website while debugging, had no idea it was made by you. Nice!


"A KeyUpEvent is dispatched, but the state shows that the physical key is pressed on a different logical key" error for Alt key breaks input on Linux

Keyboard input is broken on Linux when typing "@". This problem did not occur when using flutter 2.2.3. It was introduced upon upgrading to 2.5.0.
Steps to Reproduce

Run flutter create bug.
Update the files as follows: Just add a TextFormField().
Type<EMAIL_ADDRESS>into the TextFormField.

Expected results:
I expected to see<EMAIL_ADDRESS>Actual results:
Only "hello@" is shown and text input no longer works.

Logs

[+146188 ms] ** (bug:31682): WARNING **: 15:30:43.554: Unable to retrieve framework response: Message is not valid JSON
[   +6 ms]
           ══╡ EXCEPTION CAUGHT BY SERVICES LIBRARY ╞══════════════════════════════════════════════════════════
           The following assertion was thrown during a platform message callback:
           A KeyUpEvent is dispatched, but the state shows that the physical key is pressed on a different
           logical key. If this occurs in real application, please report this bug to Flutter. If this occurs
           in unit tests, please ensure that simulated events follow Flutter's event model as documented in
           `HardwareKeyboard`. This was the event: KeyUpEvent#de59f(physicalKey:
           PhysicalKeyboardKey#70034(usbHidUsage: "0x00070034", debugName: "Quote"), logicalKey:
           LogicalKeyboardKey#00027(keyId: "0x00000027", keyLabel: "'", debugName: "Quote Single"), character:
           null, timeStamp: 5:16:04.135000) and the recorded logical key LogicalKeyboardKey#00040(keyId:
           "0x00000040", keyLabel: "@", debugName: "At")
           'package:flutter/src/services/hardware_keyboard.dart':
           Failed assertion: line 444 pos 16: '_pressedKeys[event.physicalKey] == event.logicalKey'

           Either the assertion indicates an error in the framework itself, or we should provide substantially
           more information in this error message to help you determine and fix the underlying cause.
           In either case, please report this assertion by filing a bug on GitHub:
             https://github.com/flutter/flutter/issues/new?template=2_bug.md

           When the exception was thrown, this was the stack:
           #2      HardwareKeyboard._assertEventIsRegular.<anonymous closure> (package:flutter/src/services/hardware_keyboard.dart:444:16)
           #3      HardwareKeyboard._assertEventIsRegular (package:flutter/src/services/hardware_keyboard.dart:452:6)
           #4      HardwareKeyboard.handleKeyEvent (package:flutter/src/services/hardware_keyboard.dart:543:5)
           #5      KeyEventManager.handleRawKeyMessage (package:flutter/src/services/hardware_keyboard.dart:821:35)
           #6      BasicMessageChannel.setMessageHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:73:49)
           #7      BasicMessageChannel.setMessageHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:72:47)
           #8      _DefaultBinaryMessenger.setMessageHandler.<anonymous closure> (package:flutter/src/services/binding.dart:379:35)
           #9      _DefaultBinaryMessenger.setMessageHandler.<anonymous closure> (package:flutter/src/services/binding.dart:376:46)
           #10     _invoke2.<anonymous closure> (dart:ui/hooks.dart:205:15)
           #14     _invoke2 (dart:ui/hooks.dart:204:10)
           #15     _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:42:5)
           #16     _Channel.push (dart:ui/channel_buffers.dart:132:31)
           #17     ChannelBuffers.push (dart:ui/channel_buffers.dart:329:17)
           #18     PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:544:22)
           #19     _dispatchPlatformMessage (dart:ui/hooks.dart:92:31)
           (elided 5 frames from class _AssertionError and dart:async)
           ════════════════════════════════════════════════════════════════════════════════════════════════════

Analyzing bug...
No issues found! (ran in 1.1s)

Relevant parts:
[✓] Flutter (Channel stable, 2.5.0, on Debian GNU/Linux 10 (buster) 4.19.0-17-amd64, locale en_GB.UTF-8)
    • Flutter version 2.5.0...
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4cc385b4b8 (5 days ago), 2021-09-07 23:01:49 -0700
    • Engine revision f0826da7ef
    • Dart version 2.14.0

[✓] Linux toolchain - develop for Linux desktop
    • clang version 7.0.1-8+deb10u2 (tags/RELEASE_701/final)
    • cmake version 3.13.4
    • ninja version 1.8.2
    • pkg-config version 0.29

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Debian GNU/Linux 10 (buster) 4.19.0-17-amd64
    • Chrome (web)    • chrome • web-javascript • Chromium 90.0.4430.212 built on Debian 10.9, running on Debian 10.10

Same here macOS. Even we don't input text in textfield. Just press and hold cmd or other keys and move focus between flutter app and another program, sometimes(not always) the error will be raised.

Same here macOS

@narumishi  please file a separate issue with exact steps and logs, This issue seems a bit different from what you are reporting.

This exception is also raised when Right Alt is pressed and, as I mentioned in this issue and when you combine Shift with any other number key (not only 2 for @), question mark and possibly others.

A similar exception is being thrown on Flutter web, on Chrome 93 on Linux, with Flutter 2.5.0.
In this case, after a minute of typing and changing focus in the web app, every keystroke generates an exception on the console, but the key also has its expected effect when an input field has focus.
The exception is "A KeyDownEvent is dispatched, but the state shows that the physical key is already pressed."  The KeyDownEvent physical key and logical key are the "Arrow Right" key, even though I am pressing a completely different key.
I am trying to find a reproducible sequence of key presses, copy-pastes, and focus changes that cause this error to start, but haven't found it yet.  I can consistently enter it after about a minute of hitting keys.
The web app is "current_results_ui" at https://github.com/dart-lang/dart_ci/tree/master/current_results_ui.
I will continue verifying the problem, by testing with earlier versions of Flutter web and finding a reproducible test case.
The error stack is
Exception caught by services library
The following assertion was thrown during a platform message callback:
Assertion failed: file:///src/flutter/packages/flutter/lib/src/services/hardware_keyboard.dart:437:16
!_pressedKeys.containsKey(event.physicalKey)
"A KeyDownEvent is dispatched, but the state shows that the physical key is already pressed. If this occurs in real application, please report this bug to Flutter. If this occurs in unit tests, please ensure that simulated events follow Flutter's event model as documented in `HardwareKeyboard`. This was the event: KeyDownEvent#8e77c(physicalKey: PhysicalKeyboardKey#7004f(usbHidUsage: \"0x0007004f\", debugName: \"Arrow Right\"), logicalKey: LogicalKeyboardKey#55c9d(keyId: \"0x100000303\", keyLabel: \"Arrow Right\", debugName: \"Arrow Right\"), character: null, timeStamp: 0:00:51.308000)"

When the exception was thrown, this was the stack:
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 251:49  throw_
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 29:3    assertFailed
packages/flutter/src/services/hardware_keyboard.dart 437:47                   <fn>
packages/flutter/src/services/hardware_keyboard.dart 451:14                   [_assertEventIsRegular]
packages/flutter/src/services/hardware_keyboard.dart 543:5                    handleKeyEvent
packages/flutter/src/services/hardware_keyboard.dart 821:35                   handleRawKeyMessage
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 84:54            runBody
dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 123:5            _async
packages/flutter/src/services/hardware_keyboard.dart 808:51                   handleRawKeyMessage
...

I wasn't able to reproduce this by just writing hello@ so I randomly typed for a while and found that left shift + left alt + any other alphanumeric key will cause all input to break and start throwing these
** (master:15753): WARNING **: 16:10:33.322: Unable to retrieve framework response: Message is not valid JSON

======== Exception caught by services library ======================================================
The following assertion was thrown during a platform message callback:
A KeyUpEvent is dispatched, but the state shows that the physical key is pressed on a different logical key. If this occurs in real application, please report this bug to Flutter. If this occurs in unit tests, please ensure that simulated events follow Flutter's event model as documented in `HardwareKeyboard`. This was the event: KeyUpEvent#e93cb(physicalKey: PhysicalKeyboardKey#700e2(usbHidUsage: "0x000700e2", debugName: "Alt Left"), logicalKey: LogicalKeyboardKey#00106(keyId: "0x200000106", keyLabel: "Meta Left", debugName: "Meta Left"), character: null, timeStamp: 0:17:08.056000) and the recorded logical key LogicalKeyboardKey#00104(keyId: "0x200000104", keyLabel: "Alt Left", debugName: "Alt Left")
'package:flutter/src/services/hardware_keyboard.dart':
Failed assertion: line 444 pos 16: '_pressedKeys[event.physicalKey] == event.logicalKey'

Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
  https://github.com/flutter/flutter/issues/new?template=2_bug.md

When the exception was thrown, this was the stack:
#2      HardwareKeyboard._assertEventIsRegular.<anonymous closure> (package:flutter/src/services/hardware_keyboard.dart:444:16)
#3      HardwareKeyboard._assertEventIsRegular (package:flutter/src/services/hardware_keyboard.dart:452:6)
#4      HardwareKeyboard.handleKeyEvent (package:flutter/src/services/hardware_keyboard.dart:543:5)
#5      KeyEventManager.handleRawKeyMessage (package:flutter/src/services/hardware_keyboard.dart:821:35)
#6      BasicMessageChannel.setMessageHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:77:49)
#7      BasicMessageChannel.setMessageHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:76:47)
#8      _DefaultBinaryMessenger.setMessageHandler.<anonymous closure> (package:flutter/src/services/binding.dart:387:35)
#9      _DefaultBinaryMessenger.setMessageHandler.<anonymous closure> (package:flutter/src/services/binding.dart:384:46)
#10     _invoke2.<anonymous closure> (dart:ui/hooks.dart:205:15)
#14     _invoke2 (dart:ui/hooks.dart:204:10)
#15     _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:42:5)
#16     _Channel.push (dart:ui/channel_buffers.dart:132:31)
#17     ChannelBuffers.push (dart:ui/channel_buffers.dart:329:17)
#18     PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:544:22)
#19     _dispatchPlatformMessage (dart:ui/hooks.dart:92:31)
(elided 5 frames from class _AssertionError and dart:async)
====================================================================================================

Left Alt on its own will already cause this message before input breaks.
** (master:24714): CRITICAL **: 16:13:58.419: void synchronize_pressed_states_loop_body(gpointer, gpointer, gpointer): assertion 'pressed_logical_key_before_event == 0 || pressed_logical_key_before_event == logical_key' failed

This was known before as https://github.com/flutter/flutter/issues/89729 but this can happen without hot restarting once. I was able to reproduce this on the latest master 2.6.0-6.0.pre.76 as well as stable 2.5.0

flutter doctor -v
[✓] Flutter (Channel master, 2.6.0-6.0.pre.76, on Ubuntu 20.04.3 LTS 5.11.0-34-generic, locale en_US.UTF-8)
    • Flutter version 2.6.0-6.0.pre.76 at /home/dad/Development/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fad5e4cf89 (17 hours ago), 2021-09-13 13:37:07 -0700
    • Engine revision abb1980f65
    • Dart version 2.15.0 (build 2.15.0-96.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /home/dad/Android/Sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /home/dad/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2020.3)
    • Android Studio at /home/dad/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000
    • Flutter plugin version 60.1.2
    • Dart plugin version 203.8292
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 20.04.3 LTS 5.11.0-34-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 93.0.4577.82

• No issues found!

Doesn't reproduce on 2.2.3 so its a regression

flutter doctor -v
[✓] Flutter (Channel unknown, 2.2.3, on Linux, locale en_US.UTF-8)
    • Flutter version 2.2.3 at /home/dad/Development/flutter_old
    • Framework revision f4abaa0735 (3 months ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /home/dad/Android/Sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /home/dad/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2020.3)
    • Android Studio at /home/dad/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000
    • Flutter plugin version 60.1.2
    • Dart plugin version 203.8292
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Linux
    • Chrome (web)    • chrome • web-javascript • Google Chrome 93.0.4577.82

• No issues found!

Confirming what @whesse said it also reproduces on Chrome on Linux host. Caused by the same thing?

TLDR: The flutter web failure is due to fragile key handling code that fails if an exception is thrown by scrollable.position during handling of an arrow key.  The arrow key is registered as pressed, but never reset to unpressed because the exception happened.
The solution is not to have broken ScrollControllers, by not using the default primary ScrollController for multiple widgets on the page.
The Flutter web exception is being thrown because an error in the application is causing the ScrollController to throw an exception in the middle of the key handling code.  Because the application is using the default primary ScrollController for many Scrollable objects, that controller throws an exception when it is referenced, as in the "position" call made by a key press that might be caught by a scrollable, like arrow left, right, up, or down.  The key handling code then has set the "pressed" values to include that arrow key, but exits with the scrollable exception before it processes the key up and removes the "pressed" state of that key.
So this issue report can be ignored, unless the opinion is that key handling should be robust when exceptions are thrown by the scroll controller.
The flutter web KeyDown exception for right arrow reproduces reliably on flutter dev channel  2.6.0-5.1.pre  if right arrow is the first key pressed when the web page is opened.
The error does not reproduce on Flutter version 2.3.0-24.1.pre

I should mention that I have a UK keyboard layout. ' is the non-shifted character for the same key.

This seems to be occuring on Windows too, based on this https://github.com/flutter/flutter/issues/90050. Adding platform-windows label as applicable.

/cc @dkwingsmt

Same happens to me running MacOS with iOS emulator.. Can't press any key!
Only sometimes I can see a letter has been wrote in the TextField.

Bumping priority given how serious this is.

I have the same issue on iOS simulator using Channel dev, 2.6.0-5.1.pre.

Doctor
[✓] Flutter (Channel dev, 2.6.0-5.1.pre, on macOS 11.4 20F71 darwin-x64, locale en-US)
    • Flutter version 2.6.0-5.1.pre at /Applications/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f2903d3da5 (2 weeks ago), 2021-09-08 12:44:29 -0700
    • Engine revision 0bd9be379a
    • Dart version 2.15.0 (build 2.15.0-82.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/larry/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 12.5.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 50.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (4 available)
    • iPhone 11 (mobile) • 149F69C8-D689-47AC-AA39-1A224BA1B090 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5
      (simulator)
    • iPhone 12 (mobile) • 25E330C8-D67C-41FD-A04F-DD696084B2A0 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5
      (simulator)
    • macOS (desktop)    • macos                                • darwin-x64     • macOS 11.4 20F71 darwin-x64
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 93.0.4577.82
    ! Error: Apple TV is not connected. Xcode will continue when Apple TV is connected. (code -13)

I have this issue on Linux using Flutter 2.6.0-12.0.pre.85 • channel master. It occurs while I press shift + 8 on a TextField instance; however, only the left shift button raises the exception:

Also having this issue on Linux desktop builds.

Any clue that might cause this, so maybe we can find a workaround or even a possible fix?

This thread contains multiple issues that are possibly unrelated. They show the same error because they are triggered by the same assertion that checks the regularity of the incoming event sequence, the failure of which can come from various sources.
I'm restraining this thread exclusively for Linux for clarity. I'm looking at each issue one by one, but I can't reproduce all of the issues.
When reporting issues, please make sure to add your flutter doctor -v, keyboard layout, and the reproduction steps. Thank you.

I think this is the same issue discussed in #68713.
The problem with this issue is that usually Flutter just displays the error in the terminal... but sometimes, the TextField stops working.

Any expectations when this is getting fixed? Just asking because I am making a game for GitHub's Game Off (submission ends on December 1st) and this issue is a total deal breaker since the game suddenly crashes on desktop 😢

@mirbostani Thank you! I can reproduce it on Linux. I'm taking a look.

A fix has been merged and landed onto master.  https://github.com/flutter/engine/pull/29550
It should resolve the problem raised by @mirbostani.
Can everyone try to see if your problem persists?

@KyleKun That is not a desired move... I just saw the list of issues that were closed in favor of this one. I'll take a look at them one by one.

I've just gone over the list of issues that were "marked as duplicates" above. Some of them have been confirmed fixed. Some I can't reproduce, but I've unlocked and requested to verify on the latest master.
Thank you for all the reports.

I'm marking this issue as fixed. If the same or similar issues occur, feel free to comment, reopen, or file a new issue.
(Yes I'm aware that other issue are in the wild, namely:

https://github.com/flutter/flutter/issues/68713
https://github.com/flutter/flutter/issues/93278


Sentiment Analysis of lyrics

Predict Sentiment Analysis of the lyrics, which will be used for elastic search later on

Done with making the model


Use inspect.getmembers instead of dict

The usage of __dict__ requires manual processing of __mro__ as well, which inspect.getmembers handles automatically, as well as providing built-in filtering via predicates.
~This also adds a JujuHookEvent base class for distinguishing hook events from non-hook events. It also removes the Base suffix from the events defined in juju.charm, since it feels a bit superfluous.~
Proposed as an alternative to #15 with the additional logic that @dshcherb mentioned in Telegram.

I included both JujuHookEvent and JujuEvent in case we find some future need to distinguish other types of Juju-generated events in the future.  For example, potentially auto-linking action handlers like we do hooks, which was a commonly requested feature in reactive.

@niemeyer Maybe this should be split into two PRs.  The change to inspect.getmembers() is to address @dshcherb's issue that the use of __dict__ doesn't handle subclasses, and it seems much more standard and clearer than resorting to the nested loops:
        for cls in type(self).__mro__:
            for attr_name, attr_value in cls.__dict__.items():

On the other hand, the JujuEvent base classes are mainly there to more cleanly filter events by type.  The immediate use-case being preventing hook symlinks being created for custom charm-defined events.  If we can move away from symlinks sooner rather than later, then they become unused for now.  I think they still provide some helpful hierarchical information about the event types but I'd have no objection to dropping them until another actual need arose.

getmembers also relies on mro but has some additional logic: https://github.com/python/cpython/blob/v3.6.9/Lib/inspect.py#L317-L359
I think replacing the explicit for loop with it is OK.

Thanks for this.


Feature request: support for -A and -B grep flags

I am by no means a go developer or I would submit a pull request / prototype for this request.
It would be nice to have a way to pass -A# or -B# in the gf binary and/or encoding a field in the .json object.
Main use case is making the search more visibly identifiable, i.e. adjacent headers or other multi-line entries that might be relevant or helpful in recon.

@kmjones1979 it's already supported, you just need to wrap your grep arguments in quotes like: gf -save csharp_deserial '-HnriEC 5' 'serial.*GetType()'

Oh nice thanks for the tip Zach
On Sun, Feb 16, 2020 at 8:17 PM Zach Grace<EMAIL_ADDRESS>wrote:

@kmjones1979 https://github.com/kmjones1979 it's already supported, you
just need to wrap your grep arguments in quotes like: gf -save
csharp_deserial '-HnriEC 5' 'serial.*GetType()'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tomnomnom/gf/issues/29?email_source=notifications&email_token=ABUDLS77Z6ZQLH54H6RLJL3RDIFWLA5CNFSM4KC5DXU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL5AITY#issuecomment-586810447,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABUDLSZ7MWEHAGOHPZSVIBLRDIFWLANCNFSM4KC5DXUQ
.

--
Kevin Jones
<EMAIL_ADDRESS>YouTube.com/kevinjonescreates
Instagram: @kevinjonescreates


Add pool definition to prepare-pipelines.yml set to Ubuntu 20.04, and wrap the steps in a job

This PR addresses https://github.com/Azure/azure-sdk-tools/issues/4888 by doing the following:

It wraps the steps definition in jobs/job definition, so that pool definition can be added. The job is named PreparePipelines, mimicking the name GeneratePipelines as seen in pipeline-generation.yml.
It adds the pool definition with value copy-pasted from pipeline-generation.yml, which points to the ubuntu 20.04 1ES hosted pool:

pool:
  name: azsdk-pool-mms-ubuntu-2004-general
  vmImage: MMSUbuntu20.04

Testing
I am not going to merge this PR until I test if it is going to work in expected way. I will do so by submitting PRs against some target azure-sdk-for-* repos, changing their derived prepare-pipelines.yml. I will provide an update here once done with these tests. Note that the file updates made by these test PRs will be transient, as they will be overridden by changes made by this PR once fully deployed, as explained by the Common Engineering System doc.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Ftdi fixes

Fixes for #51

@smontanari1 Can you please confirm that these fixes work on windows 7 64 bit as well?


First stories

Let's discuss what they should be!
I think we should have a short quick one that gets the ball rolling. Also, I can add chapter one of my story.

Good idea on short stories. I think 3-5 "flash fiction" stories that get across some major themes we want to explore further would be a great start


Why "valid_acc" always better than "train_acc"?

In the summary of different task logs,  "valid_acc" always better than "train_acc", it seems unreasonable.
These are logs of different tasks:

I did not observe this behavior in Vision task

I did not observe this behavior in Vision task

But it always appears in all my Vision tasks, which dataset did you use?

Are you defining tuning set explicitly in predictor function or AG split it self

Yes, I did not defin my own val_data, just use the tuning data in predictor function split it self

Yes, I did not defin my own val_data, just use the tuning data in predictor function split it self

Are you defining tuning set explicitly in predictor function or AG split it self

Yes, I did not define my own val_data, just use the tuning data in predictor function split it self, Is there any different between these two mode？

I used a dataset from kaggle, the yoga posts. And I have observed the same problem.
As you can see, the training accuracy is more than 20% less than validation accuracy, and when I used a self-split test set, I got accuracy similar to the training.
Would you please look into the problem? Maybe test with this kaggle dataset?
PS: My autogluon version is 0.2.0.

Thanks for reporting! This is definitely strange and might have to do with some kind of index misalignment in how training data is scored, hard to tell without deep diving. We will plan to look into this issue and resolve it for v0.3.

@Fanxingye / @kevin666aa Could you provide a kaggle notebook / colab notebook that reproduces the issue reliably? This would help us debug.

@Fanxingye / @kevin666aa Could you provide a kaggle notebook / colab notebook that reproduces the issue reliably? This would help us debug.

Here is a demo from the docs of autogluon, I tested it  and observed the same problem. https://auto.gluon.ai/stable/tutorials/image_prediction/hpo.html

Thanks, it does appear to be quite widespread. The primary maintainer of the vision portion of the codebase is currently on vacation this week, but we will try to resolve as much as we can once he is back.

I used a dataset from kaggle, the yoga posts. And I have observed the same problem.
As you can see, the training accuracy is more than 20% less than validation accuracy, and when I used a self-split test set, I got accuracy similar to the training.
Would you please look into the problem? Maybe test with this kaggle dataset?
PS: My autogluon version is 0.2.0.

Hi, @zhreshold and @Innixma  I would like to hear your opinion on @kevin666aa comment as he is saying he got similar accuracy when he do splitting him and much lesser accuracy when splitting is performed by predictor. Or may I misunderstood what he is saying

I used a dataset from kaggle, the yoga posts. And I have observed the same problem.
As you can see, the training accuracy is more than 20% less than validation accuracy, and when I used a self-split test set, I got accuracy similar to the training.
Would you please look into the problem? Maybe test with this kaggle dataset?
PS: My autogluon version is 0.2.0.

Hi, @zhreshold and @Innixma I would like to hear your opinion on @kevin666aa comment as he is saying he got similar accuracy when he do splitting himself and much lesser accuracy when splitting is performed by predictor. Or may I misunderstood what he is saying

colab notebook
https://colab.research.google.com/drive/1_tZ9y-gmzelaLLLX6eSO_NLUg-ksD6qy?usp=sharing

Deep dived into this and it is indeed a bug.
When validation data isn't explicitly specified to predictor.fit(), 100% of data is used for both training and validation.
The reason why is not so simple however... Everything looks like its going correctly until deep in GluonCV code when the ImageDatasets are transformed into their mxnet versions via train_data.to_mxnet() and val_data.to_mxnet(). The output of to_mxnet() somehow returns the full original data prior to splitting for both.
Here is a colab notebook showing the bug:
https://colab.research.google.com/drive/1pJScwfjj1lNIC07AR4jJxLw__9of639K?usp=sharing

Opened #1168 which fixes ImagePredictor functionality.


Added Swift Multihash

Thanks @NeoTeo !

I think you guys are doing awesome work. :)

Thanks for the help + kind words @NeoTeo :)


Can not load pretrained weight of PointNext in finetuning on ModelNet40,  in_channels is 3  not  4

First of all, thank you for sharing such an excellent work !!
The pretrained weight of PointNext is released, where '-- use_height' is used. However, during finetune on ModelNet40 or ScanObjectNN, there was no "-- use_height" in the data, which resulted in the pretrained weight being unable to be loaded.
The  in_channels is  4  in pretrained weight of PointNext. But the in_channels is  3  in setting of finetune on ModelNet40 or ScanObjectNN

RuntimeError: Given groups=1, weight of size [32, 4, 1], expected input[16, 3, 2048] to have 4 channels, but got 3 channels instead
how to solve?

Hi I think if you are using the pointnext version, you need both the weights and the input to have four dimensions, including the height information. in the code i used the --use_height to enable this.


Move dependencies out of devDependencies

Closes #27

Thank you so much!
Electron dep should stay in devdeps. Could you fix that?

Thank you again! :clap:


WIP: Add Django 3.1 support

@FinalAngel LOL I just opened #1202  for 3.1 :D

@FinalAngel I switched #1202 as a PR against your branch

@yakky lol I just pushed a bunch of stuff too :D

@yakky checkout this small beast:
https://github.com/divio/django-filer/pull/1201/commits/39621f5fbb3e5ba364f2627acb81dc390171b92b
:D

@FinalAngel I closed mine :)

@yakky you also included these changes:

https://github.com/divio/django-filer/pull/1202/files#diff-71f149c65e62bb3b8fdcd50b4b00a845
https://github.com/divio/django-filer/pull/1202/files#diff-d08b27fd1826e47143342eee8a59a02a
https://github.com/divio/django-filer/pull/1202/files#diff-8d2f6449635dadfd7a030ac92dfb1fd5
https://github.com/divio/django-filer/pull/1202/files#diff-2eeaed663bd0d25b7e608891384b7298R10
https://github.com/divio/django-filer/pull/1202/files#diff-5b96e697e7a218b995b7fdf9b754c366

shall I port them too?

@yakky you also included these changes:
* https://github.com/divio/django-filer/pull/1202/files#diff-71f149c65e62bb3b8fdcd50b4b00a845

* https://github.com/divio/django-filer/pull/1202/files#diff-d08b27fd1826e47143342eee8a59a02a

* https://github.com/divio/django-filer/pull/1202/files#diff-8d2f6449635dadfd7a030ac92dfb1fd5

* https://github.com/divio/django-filer/pull/1202/files#diff-2eeaed663bd0d25b7e608891384b7298R10

* https://github.com/divio/django-filer/pull/1202/files#diff-5b96e697e7a218b995b7fdf9b754c366

shall I port them too?

Yes, I pinpointed them in the review :)

we have to wait the release of this https://github.com/django-polymorphic/django-polymorphic/pull/452 for properly support 3.1, unfortunately

awesome @yakky, I'll have a closer look tomorrow. Also cms will be dropped from the tests, that was mostly copy & paste. Not yet ready :) marked WIP.
And yeah, I added the first comment on polymorphic xD

@yakky I'll remove the integration tests for now as they are not stable anymore. Needs more work in the future...
Would you know what's up with the documentation, I tried several code changes without any positive result...

@yakky I'll remove the integration tests for now as they are not stable anymore. Needs more work in the future...
Would you know what's up with the documentation, I tried several code changes without any positive result...

I see fe tests are enabled again now
I will work with 3baal to have a new release of polymorphic soon
I see docs tests failing due to missing sphinx

@yakky sorry it's this one: https://travis-ci.org/github/divio/django-filer/jobs/718533457
I don't get why filer is not installed

@FinalAngel I run local tests using soon-to-be released new version of polymorphic https://github.com/django-polymorphic/django-polymorphic/archive/1d719a0e605632bb761fb002673c5530896e6f32.zip and tests pass :top:

@yakky will wait for the release though 👍  Any ideas about the docs tests?

@yakky will wait for the release though +1 Any ideas about the docs tests?

wrt docs tests https://github.com/divio/django-filer/pull/1201/files#r472112884

Codecov Report

Merging #1201 into master will decrease coverage by 0.56%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #1201      +/-   ##
==========================================
- Coverage   71.34%   70.77%   -0.57%
==========================================
  Files          63       63
  Lines        3144     3052      -92
  Branches      441      436       -5
==========================================
- Hits         2243     2160      -83
+ Misses        747      738       -9
  Partials      154      154

Impacted Files
Coverage Δ

filer/admin/__init__.py
100.00% <ø> (ø)

filer/admin/permissions.py
75.00% <ø> (-1.48%)
:arrow_down:

filer/admin/thumbnailoptionadmin.py
100.00% <ø> (ø)

filer/contrib/django_cms/cms_toolbars.py
0.00% <0.00%> (ø)

filer/fields/image.py
100.00% <ø> (ø)

filer/management/commands/filer_check.py
78.78% <ø> (-0.63%)
:arrow_down:

filer/management/commands/generate_thumbnails.py
0.00% <ø> (ø)

filer/management/commands/import_files.py
0.00% <0.00%> (ø)

filer/migrations/0002_auto_20150606_2003.py
100.00% <ø> (ø)

filer/migrations/0003_thumbnailoption.py
100.00% <ø> (ø)

... and 53 more

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 d45a9a2...e089f94. Read the comment docs.

@yakky this is ready from my end, would you be able to review and approve?

awesome, once CMS passes we are ready for the release marathon :D