ERROR 403: Forbidden occurred.

When I ran DisCo_Demo.ipynb, ERROR 403: Forbidden occurred.
Cloning into 'sd-image-variations-diffusers'...
remote: Enumerating objects: 98, done.
remote: Counting objects: 100% (98/98), done.
remote: Compressing objects: 100% (54/54), done.
remote: Total 98 (delta 38), reused 98 (delta 38), pack-reused 0
Unpacking objects: 100% (98/98), 1.57 MiB | 4.59 MiB/s, done.
Filtering content: 100% (4/4), 5.77 GiB | 24.55 MiB/s, done.
--2023-10-24 06:28:16--  https://storage.googleapis.com/disco-checkpoint-share/checkpoint_ft/moretiktok_nocfg/mp_rank_00_model_states.pt
Resolving storage.googleapis.com (storage.googleapis.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to storage.googleapis.com (storage.googleapis.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 403 Forbidden
2023-10-24 06:28:16 ERROR 403: Forbidden.

Please let me know how to fix it.

It seems that it still used the google cloud link which has been deprecated. Could u please try the new download link in the repo?

Could you please let me know the new download link?
I don't know where it is.

Hi, I think you can check the link here: https://entuedu-my.sharepoint.com/:f:/g/personal/tan317_e_ntu_edu_sg/En3vpzKndr1OoBUoDhTCZvEBorQiXEhJzi6mYWNELVWnpg?e=AIrGlx


Inheritance is broken for WinRT types

I noticed this when trying to make async calls to WinRT. I believe this affects all WinRT types that extend a type other than IInspectable. e.g. IAsyncOperation(extends IAsyncInfo), IStorageFile(extends IStorageItem).
As of now, the generation script calculates vtable start index by iterating all of the interfaces that an interface extends and summing the number of methods in those interfaces:
https://github.com/timsneath/win32/blob/dcec79e0d1d0de9fc1a8c1836ad1cd96ff7c9b72/tool/projection/com_interface.dart#L32-L34
With this calculation, IAsyncOperation's vtable start index becomes 11 which I think it's wrong and I believe it should be 6. The function should only iterate for IInspectable and IUnknown interfaces.
The reason for that is I was looking into the header(.h) file for Windows.Foundation here and I found IAsyncOperation's vtable:
typedef struct __FIAsyncOperation_1_IInspectableVtbl
{
    BEGIN_INTERFACE
    HRESULT ( STDMETHODCALLTYPE *QueryInterface )(__RPC__in __FIAsyncOperation_1_IInspectable * This,
        /* [in] */ __RPC__in REFIID riid,
        /* [annotation][iid_is][out] */
        _COM_Outptr_  void **ppvObject);
    ULONG ( STDMETHODCALLTYPE *AddRef )(__RPC__in __FIAsyncOperation_1_IInspectable * This);
    ULONG ( STDMETHODCALLTYPE *Release )(__RPC__in __FIAsyncOperation_1_IInspectable * This);

    HRESULT ( STDMETHODCALLTYPE *GetIids )(__RPC__in __FIAsyncOperation_1_IInspectable * This,
        /* [out] */ __RPC__out ULONG *iidCount,
        /* [size_is][size_is][out] */ __RPC__deref_out_ecount_full_opt(*iidCount) IID **iids);
    HRESULT ( STDMETHODCALLTYPE *GetRuntimeClassName )(__RPC__in __FIAsyncOperation_1_IInspectable * This, /* [out] */ __RPC__deref_out_opt HSTRING *className);
    HRESULT ( STDMETHODCALLTYPE *GetTrustLevel )(__RPC__in __FIAsyncOperation_1_IInspectable * This, /* [out] */ __RPC__out TrustLevel *trustLevel);

    /* [propput] */ HRESULT ( STDMETHODCALLTYPE *put_Completed )(__RPC__in __FIAsyncOperation_1_IInspectable * This, /* [in] */ __RPC__in_opt __FIAsyncOperationCompletedHandler_1_IInspectable *handler);
    /* [propget] */ HRESULT ( STDMETHODCALLTYPE *get_Completed )(__RPC__in __FIAsyncOperation_1_IInspectable * This, /* [retval][out] */ __RPC__deref_out_opt __FIAsyncOperationCompletedHandler_1_IInspectable **handler);
    HRESULT ( STDMETHODCALLTYPE *GetResults )(__RPC__in __FIAsyncOperation_1_IInspectable * This, /* [retval][out] */ __RPC__out IInspectable * *results);
    END_INTERFACE
} __FIAsyncOperation_1_IInspectableVtbl;

interface __FIAsyncOperation_1_IInspectable
{
    CONST_VTBL struct __FIAsyncOperation_1_IInspectableVtbl *lpVtbl;
};

After I saw this, I changed the vtable start index to 6 in IAsyncOperation but it didn't work since IAsyncInfo's vtable start index was also 6 and they used the same pointer. Then I had the idea to use toInterface helper method to get the IAsyncInfo with the correct pointer and it worked.
As a temporary workaround, I modified IAsyncInfo.dart like this:
  // This pointer is used in methods and properties instead of the `ptr` from
  // `IAsyncOperation`
  late final Pointer<COMObject> _ptr;

  // vtable begins at 6, is 5 entries long.
  IAsyncInfo(Pointer<COMObject> ptr, {required Allocator allocator})
      : super(ptr) {
    // We need to query IInspectable to get the proper pointer for IAsyncInfo.
    _ptr = toInterface(IID_IAsyncInfo, allocator: allocator);
  }

I don't like this workaround at all because it breaks the inheritance for IAsyncInfo. What I mean is, I send ptr parameter that comes from IAsyncOperation to IInspectable in super(ptr). Because of that if we print the runtimeClassName property somewhere in IAsyncInfo it will output Windows.Foundation.IAsyncOperation'1. I'm worried that this will cause us some trouble as we use methods/properties from IInspectable and IUnknown in some places, especially inside IVector and IVectorView.
So far I haven't been able to find a way to fix this problem without breaking inheritance for both sides. You may have some thoughts about this.
If you want to check it out, my work on making async calls to WinRT is here. Inside that, there is a simple working example for IFileOpenPicker.

Inheritance is a little confusing in WinRT -- I didn't understand it at first, and I know it's not fully modeled in the projection (I was focused at first on just getting the basics in place, one interface at a time).
As I understand it, every WinRT interface inherits from IInspectable (and nothing else). So the vtable should begin at 6 for all of them, including IAsyncInfo and IAsyncOperation.
But WinRT interfaces can also "require" other interfaces. So when IAsyncOperation claims it implements IAsyncInfo, for example, that doesn't mean it inherits from it, it means that any object that supports IAsyncOperation must also support the IAsyncInfo interface (through QueryInterface). And of course, some interfaces may require more than one interface to be implemented, for example IPropertySet.
My sense is that the best way to implement this in Dart might be through mixins, which allows Dart to idiomatically support this.
One note -- the cacheVtableStart() method for WinRT interfaces should be provided by winrt_interface.dart, which overrides the implementation listed above in com_interface.dart. That should return 6 for all these objects; I'll test and fix this if that's not hte case.

Commit 72eba9e910478bd33fb8dd8936132305d1d9422a addresses the incorrect calculation of vtable start for WinRT types.

As I understand it, every WinRT interface inherits from IInspectable (and nothing else). So the vtable should begin at 6 for all of them, including IAsyncInfo and IAsyncOperation.

Right.

But WinRT interfaces can also "require" other interfaces. So when IAsyncOperation claims it implements IAsyncInfo, for example, that doesn't mean it inherits from it, it means that any object that supports IAsyncOperation must also support the IAsyncInfo interface (through QueryInterface). And of course, some interfaces may require more than one interface to be implemented, for example IPropertySet.

Yes, it was very confusing at first but now I understand better how it works.

My sense is that the best way to implement this in Dart might be through mixins, which allows Dart to idiomatically support this.

Using mixins might be a good idea. I'll investigate.

Lastly, the best page I've found for information on all this is here: https://docs.microsoft.com/en-us/uwp/winrt-cref/winrt-type-system#iinspectable-and-iunknown

Thanks, I'll check it out.

Commit 72eba9e addresses the incorrect calculation of vtable start for WinRT types.

Thank you!

Is there more we need to do to close this bug out?

Is there more we need to do to close this bug out?

No, we can close this.


32bit float comparison less_equal gives wrong result

I was comparing a value against 1.0, and it seemed to think that 1.99 <= 1.0 is true. Note that the plain less than works fine, so 1.99 < 1.0 return false. Also values bigger than 2.0 gave the correct result e.g. 2.001 <= 1.0 returns false.
Example function showing the incorrect result:
# input second value B = 1
scoreboard players set P0 io 1
function float:32/decompose/main
scoreboard players operation P3 io = R0 io
scoreboard players operation P4 io = R1 io
scoreboard players operation P5 io = R2 io

# input first float, A = 1.99
scoreboard players set P0 io<PHONE_NUMBER>
function float:32/from_int/main
scoreboard players operation P0 io = R0 io
scoreboard players operation P1 io = R1 io
scoreboard players operation P2 io = R2 io

# compare values, is A <= B?
function float:32/compare/less_equal/main
tellraw @p [{"text":"1.99 <= 1.00? result = "},{"score":{"name":"R0","objective":"io"}}]

Hi, could you try swapping the decompose and from_int functions? At the moment you are trying to compare
1073657938 <= 1.40129846432e-45
Obviously the answer should still be false but I feel like this might be an edge case since one of the numbers is the smallest float value.
I think once you swap the functions it should work, although I definitely will look into fixing this.

Yep, same issue with decompose and from_int swapped.
Sorry, that was my mistake, I meant it to be B set via from_int and A via decompose.

I will take a look at it, but probably not for a couple of weeks because it's finals season. In the meantime, if you want you can see if you can fix it yourself and submit a pull request.


Refactoring/scale encoding

Description of the Change
Faster scale encoding.

Removed internal deque for results.
Removed temporary vector.
Removed temporary vector on CompactInteger with large values.
BitVec, some buffers and integers transfers out by larger blocks.

Use concept StructureBindable, not Constructible
template <typename T>
concept Tup2 = requires(T &t) { [](T &t) { auto &[t0, t1] = t; }(t); };


Search Bar & Scrolling

Would it be possible for the creative gui to have a search bar and scrolling added? I created a datapack for pokecube support, but it has over a thousand mobs, making it difficult to find a specific mob. I have to spam click to get to the last page. :D
For example:
trofers.zip

Oh my, that's a lot of trophies XD! I'm not very familiar yet with making GUI's, that's why the menu is a bit bare bones right now. A search bar seems doable though. Honestly I'd love to just add the trophies to JEI if I could, but the fact that you can change data packs on the fly makes that rather difficult.

Ah thanks for looking into it. :)


Vertical Swipe

I still have to swipe horizontally in order for it to scroll.  Anyway to switch that to swipe vertically  instead?  Everything else works perfect, thank you :)

Did you change the Tag in your xml?

I have the same issue. What Tag do you need to change?


[Documentation]: Add info on how to backup the config file

Some pages of the TA Documentation (here for CLI, here for desktop) mention the need to "backup the config file" before updating or to save configs so they don't get lost.
Those statements on the pages hyperlink to #configuration anchors, which are non-existent -- so I can't find instruction on how to do that.
Could some easy documentation be written on how to backup the config file for both CLI (I'm on a Raspberry Pi/CompanionPi) and other OS's?
Or, more advanced, perhaps this turns into a feature request with an "export config" button so that it pops out the config file with a place to save it.

Hi @kenm-4682,
the export feature is tracked in #164 , however, updating the docs sounds like a good idea 👍


Update activate-using-active-directory-based-activation-client.md

Tidied up markdown.
Changed wording a few places.
Changed numbering style according to best practices found elsewhere.
Added a few line shifts for improved readability.

Rich Text Diff view, for easier overview of the changes:
https://github.com/MicrosoftDocs/windows-itpro-docs/pull/5080/files?short_path=ddbed77#diff-ddbed77f97e36e6828e7bdcae073f395
The MarkDown note format implementation looks good, so does the added spacing for easier reading (improved readability is definitely a plus).

@greg-lindsay Following up for updates.


bug (likely neovim upstream): noice.nvim causes vim.on_key to not trigger on count

Did you check docs and existing issues?

[X] I have read all the noice.nvim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of noice.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.0
Operating system/version
Arch Linux (all packages up to date)
Describe the bug
Noice.nvim partially breaks vim.on_key and causes it to not recieve the count key.
Steps To Reproduce

Use the below config (tested on both my machine as well as VM, both Arch Linux)
Press some number and other keys keys
Number keys don't show up, but other letters and symbols do.

Expected Behavior
noice should not break vim.on_key
Also this is very likely a NeoVim bug but I decided to report it here first as folke as the author of this plugin must have a better idea of what NeoVim API when used here is breaking vim.on_key
Repro
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
	vim.fn.system({
		"git",
		"clone",
		"--filter=blob:none",
		"https://github.com/folke/lazy.nvim.git",
		"--branch=stable", -- latest stable release
		lazypath,
	})
end

vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
	spec = {
		{
			"folke/noice.nvim",
			event = "VeryLazy",
			opts = {
				-- add any options here
			},
			dependencies = {
				-- if you lazy-load any plugin below, make sure to add proper `module="..."` entries
				"MunifTanjim/nui.nvim",
				-- OPTIONAL:
				--   `nvim-notify` is only needed, if you want to use the notification view.
				--   If not available, we use `mini` as the fallback
				"rcarriga/nvim-notify",
			},
		},
	},
})

local keylogger_ns = vim.api.nvim_create_namespace("keylogger")

vim.on_key(function(key, typed)
	vim.print(typed)
end, keylogger_ns)

local keylogger_ns = vim.api.nvim_create_namespace("keylogger")
local keys = {}
vim.on_key(function(key, typed)
  table.insert(keys, vim.fn.keytrans(typed))
  vim.notify(vim.inspect(keys))
end, keylogger_ns)

IT does work, but noice receives a cmd clear or cmd clear last and that hides the last print.
Not a Neovim, nor Noice bug.

Reason is that if you type a number, NEovim waits (since it could be a count).
It only sends the number on next character, so together with the next character.

Reason is that if you type a number, NEovim waits (since it could be a count).
It only sends the number on next character, so together with the next character.

No, it doesn't (atleast by according to the documentation). It should trigger for every key pressed and also see https://github.com/neovim/neovim/issues/15527.
Disabling noice and using the same init.lua as your suggested one printing the table, I am able to see number keys as soon as they are pressed without any other key. Noice.nvim is either calling a nvim function or setting a vim option which is likely triggering a bug in NeoVim, which is making it wait for another key to be pressed after the number.

You're right. Just add a redraw and you'll see them
local keylogger_ns = vim.api.nvim_create_namespace("keylogger")
local keys = {}
vim.on_key(function(key, typed)
  table.insert(keys, vim.fn.keytrans(typed))
  vim.notify(vim.inspect(keys))
  vim.cmd.redraw()
end, keylogger_ns)

The reason you see them right away without noice,
is because that would use the builtin notify, which is just an echo.
nvim-notify creates windows etc and those would only be displayed when doing a redraw.
Neovim automatically does redraws, but sometimes waits for a certain state to do it.
Adding the redraw like I did forces a redraw at that point.
So again, all pretty much how it's supposed to behave.

Thanks! Didnt think that way! And sorry for the dubious and dumb bug report


Parse verilog with preplacement

From Christian Kison:
Instances in verilog files like the example below (CLK_BUF_X3_0) are not parsed correctly, with our without the Parameters given. The PosX/PosY are meant for the layout to have a preplacement. I am coming from a placed and routed chip.
CLKBUF_X3
  #(
    .PosX(337060),
    .PosY(870240)
  )
  CLKBUF_X3_0
  (
    .A(_tmp_2697),
    .Z(_tmp_10493)
  );

See NANDGATE_45NM.yaml
@Christian Kison: We need test designs for that.

I'm not certain if this is actually a parsing problem, because the GUI doesnt have a layouter that takes preplacement into account yet. If it is in fact a parsing problem, please label this issue accordingly, if the parsing works correctly and its just a missing GUI feature please close this and create a new addition issue

@Slowpoke42 As far as I know this is a parsing problem. The problem is, that we don't have any example netlist, to actually see what exactly is the problem here.

Another example would be:
MUX2_X1
$MUX2_X1_897180_1990240_13345
(
 .S(_tmp_124),
 .B(_tmp_6249),
 .A(_tmp_6252),
 .Z(_tmp_6253)
);

aes_jun.v.zip

We have support for explicit gate placement since 9459da030672a7769d7d7a08e050efbcbcf9bd7c. Display of this data in the UI is work in progress. Is anybody working on updating the parsers?

As soon as we have more samples of netlists with placement information I will implement this feature.


⚠ Samsung has degraded performance

In ecfbcef, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2253 ms

Resolved: Samsung performance has improved in db5bb88.


Include coverage in mocha tests

This is unnecessary. The coverage is already shown in the test summary and in coveralls report.


Add Stick Figure related icons to Sketch Page

Sample Icons

Requirements:

Do not mess up any other part of the web app
Add an icon on the horizontal menu, on clicking which a drop down/another bar appears below showing options for icons representing stick figures
Add up to 10 different icons
Please test this thoroughly before giving a PR
Make use of any open source icons (or make custom icons) for this. You are required to give references in the PR description.

General PR Guidelines:

Each pull request should have an appropriate and short title like "Fixed Bug in Upload Page"

Describe your intended changes in the description section of the pull request (Use bullet points and phrases)

Refrain from using phrases like "Hi, I am ..", "Please merge me this OPR", "Thank You..", etc. We are only interested in technical parts

Attach a screenshot/clip of the change(s)

Make sure to refer to the respective issue in the respective PR using phrases like Resolves #issue_number or Closes #issue_number.

Please be patient enough. The project maintainers/mentors would review it as per their schedule. Please do not start putting comments like "Please check this" etc.

Although we support feedback from everyone in all phases of development, it is highly advised not to put any negative comments in other participant's pull requests.

Always keep a note of the deadline.

I want to work on this

I want to work on this

Assigned for 3 days


rasa data validate causes error with "action_two_stage_fallback"

Rasa Open Source version
3.0.0
Rasa SDK version
No response
Rasa X version
No response
Python version
3.8
What operating system are you using?
OSX
What happened?
When using the two-stage fallback, I get the following error when running rasa data validate
UserWarning: The form 'action_two_stage_fallback' is used in the 'Activate two stage fallback' block, but it is not listed in the domain file. You should add it to your domain file!
I have this in my rules:
version: "3.0" rules:

rule: Activate two stage fallback steps:
intent: nlu_fallback
action: action_two_stage_fallback
active_loop: action_two_stage_fallback

Command / Request
rasa data validate
Relevant log output
No response

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

Hi @nyejon sincere apologies for the delayed response here. Can you post your domain file too please?

Also experiencing the same problem here

I also had the same problem, is there a solution?

I have the same problem, when I add action_two_stage fallback in the rule.yml file and run the command rasa data validate I get the following error

UserWarning: The form 'action_two_stage_fallback' is used in the 'Implementation of the Two-Stage-Fallback' block, but it is not listed in the domain file. You should add it to your domain file!
More info at https://rasa.com/docs/rasa/forms
Project validation completed with errors.

I'm using the examples generated with rasa init and just adding the rule for the two stage fallback. These are my files:
domain.yml
version: "3.0"

intents:
  - greet
  - goodbye
  - affirm
  - deny
  - mood_great
  - mood_unhappy
  - bot_challenge

responses:
  utter_greet:
  - text: "Hey! How are you?"

  utter_cheer_up:
  - text: "Here is something to cheer you up:"
    image: "https://i.imgur.com/nGF1K8f.jpg"

  utter_did_that_help:
  - text: "Did that help you?"

  utter_happy:
  - text: "Great, carry on!"

  utter_goodbye:
  - text: "Bye"

  utter_iamabot:
  - text: "I am a bot, powered by Rasa."

  utter_default:
  - text: I'm sorry, I can't help you.

  utter_ask_rephrase:
  - text: I'm sorry, I didn't quite understand that. Could you rephrase?

session_config:
  session_expiration_time: 60
  carry_over_slots_to_new_session: true

rule.ym
version: "3.0"
nlu:
- intent: greet
  examples: |
    - hey
    - hello
    - hi
    - hello there
    - good morning
    - good evening
    - moin
    - hey there
    - let's go
    - hey dude
    - goodmorning
    - goodevening
    - good afternoon

- intent: goodbye
  examples: |
    - cu
    - good by
    - cee you later
    - good night
    - bye
    - goodbye
    - have a nice day
    - see you around
    - bye bye
    - see you later

- intent: affirm
  examples: |
    - yes
    - y
    - indeed
    - of course
    - that sounds good
    - correct

- intent: deny
  examples: |
    - no
    - n
    - never
    - I don't think so
    - don't like that
    - no way
    - not really

- intent: mood_great
  examples: |
    - perfect
    - great
    - amazing
    - feeling like a king
    - wonderful
    - I am feeling very good
    - I am great
    - I am amazing
    - I am going to save the world
    - super stoked
    - extremely good
    - so so perfect
    - so good
    - so perfect

- intent: mood_unhappy
  examples: |
    - my day was horrible
    - I am sad
    - I don't feel very well
    - I am disappointed
    - super sad
    - I'm so sad
    - sad
    - very sad
    - unhappy
    - not good
    - not very good
    - extremly sad
    - so saad
    - so sad

- intent: bot_challenge
  examples: |
    - are you a bot?
    - are you a human?
    - am I talking to a bot?
    - am I talking to a human?

config.yml
recipe: default.v1

language: en

pipeline:

- name: WhitespaceTokenizer
- name: RegexFeaturizer
- name: LexicalSyntacticFeaturizer
- name: CountVectorsFeaturizer
- name: CountVectorsFeaturizer
  analyzer: char_wb
  min_ngram: 1
  max_ngram: 4
- name: DIETClassifier
  epochs: 100
- name: EntitySynonymMapper
- name: ResponseSelector
  epochs: 100
- name: FallbackClassifier
  threshold: 0.3
  ambiguity_threshold: 0.1

policies:

- name: MemoizationPolicy
- name: TEDPolicy
  max_history: 5
  epochs: 100
- name: RulePolicy

I am using version 3.0.5 of Rasa Open Source version

I also seem to get an error when the two stage fallback is actually run:
in _get_active_loop_ignored_intents
    form_ignored_intents = domain.forms[active_loop_name].get(
KeyError: 'action_two_stage_fallback'

I also seem to get an error when the two stage fallback is actually run:
in _get_active_loop_ignored_intents
    form_ignored_intents = domain.forms[active_loop_name].get(
KeyError: 'action_two_stage_fallback'

I am running into the same error as described by nyejon when I reach the TwoStageFallbackAction and the user clickes one of the buttons. In my case the default settings for the TwoStageFallback were used as described here.
Rasa Version      :         3.0.4
Minimum Compatible Version: 3.0.0
Rasa SDK Version  :         3.0.2
Rasa X Version    :         None
Python Version    :         3.7.11

@pythenry
Not sure when this change was made and if this is related, but the two stage fallback appears to be implemented as a form now and as such the _get_active_loop_ignored_intents method is called. This checks for ignored intents, which are usually defined in the domain when declaring a form.
I did not manage to find the part where the two stage fallback is registered as a form internally, but apparently it is not added to the mapping-dictionary that is checked in the previously mentioned method.
What fixed it for me was to (locally) expand the check in this line to the following:
if active_loop_name and active_loop_name != 'action_two_stage_fallback':

I guess the better way to fix this would be to add the entry to the checked dictionary with an empty list or whatever the desired behavior was meant to be.

Hi all,
@akelad @wochinge maybe you have an idea, why rasa data validate fails when using The two_stage_fallback mechanism?
C:\DEV\miniconda\envs\rasa3\lib\site-packages\rasa\shared\utils\io.py:99: UserWarning: The form 'action_two_stage_fallback' is used in the
'Implementation of the Two-Stage-Fallback' block, but it is not listed in the domain file. You should add it to your domain file!
  More info at https://rasa.com/docs/rasa/forms
Project validation completed with errors.

I tried it with both, Rasa Version 3.0.9/ Python 3.8.0 and 3.1.0/ Python 3.9.0. They both resulted in the same error.
We are very close to the documented example:
#rules.yml
- rule: Implementation of the Two-Stage-Fallback
  steps:
  - intent: nlu_fallback
  - action: action_two_stage_fallback
  - active_loop: action_two_stage_fallback
---
#config.yml
pipeline:
...
- name: FallbackClassifier
  threshold: 0.7
  ambiguity_threshold: 0.1
policies:
- name: RulePolicy
  core_fallback_threshold: 0.4
  core_fallback_action_name: action_default_fallback
  enable_fallback_prediction: true

Running into the same issue. The two stage fallback also causes this error in normal rasa operation when using rasa run.
My version details:
Rasa Version      :         3.1.0
Minimum Compatible Version: 3.0.0
Rasa SDK Version  :         3.1.1
Rasa X Version    :         None
Python Version    :         3.8.9
Operating System  :         Windows-10-10.0.19041-SP0

Hi, I am having a similar issue. I overwrote action_default_fallback as well as action_default_ask_affirmation just to customize the bot a little, but I keep getting the No registered action found for name 'action_two_stage_fallback' error. Would love some insight. Here are all my specs:
Rasa Version      :         2.8.28
Minimum Compatible Version: 2.8.9
Rasa SDK Version  :         2.8.5
Rasa X Version    :         None
Python Version    :         3.8.13
Operating System  :         macOS-10.16-x86_64-i386-64bit

same problem for precisely same versions as pixpack report above while migrating a working bot from rasa 2.8.x - is there no resolution to this given the prevalence of the problem? Appears fallback implementation changed?

Fix released in version 3.1.3

still seeing KeyError: 'action_two_stage_fallback' when the user clicks one of the buttons. I'm also using the default settings for the TwoStageFallback were used as described here.
Rasa Version : 3.2.0
Python Version : 3.8.10

@cqx931 That seems to be a different issue to the one described in the original issue. Please submit a separate bug report with more details please.

@ancalita thank you for pointing this bugfix. rasa data validate does work without program now but the bot still hangs with the same key error in 3.2.1. Please see the ticket: https://github.com/RasaHQ/rasa/issues/11294

the bug has not been resolved in version 3.2.1 or 3.2.4, it seems to be a problem with the SLOTS, because when SLOTS are defined it is when the problem related to KeyError: 'action_two_stage_fallback' occurs

@borisjota The bugfix released in version 3.2.1 targeted solely the rasa data validate reported issue here. I'm currently looking into the separate issue related to the KeyError.

@borisjota The bugfix released in version 3.2.1 targeted solely the rasa data validate reported issue here. I'm currently looking into the separate issue related to the KeyError.

Is there also an opened issue we could link here?


Model ->delete() returns true for race condition where two instances both attempt to delete the same database row.

Laravel Version: 6.0
PHP Version: 7.3.7
Database Driver & Version: MySQL

Description:
In race conditions where two separate instances attempt to delete the same model, both instances return true. Using the DB facade to delete the row simultaneously on two different instances has one return 0 and the other return 1.
This causes double model events and the like when only one deletion occurred. Considering the DB returns the number of rows deleted, one should just return false.
Steps To Reproduce:
(You may want to use a split pane terminal for maximum race condition reproducing)

Open two tinker shells and get the same model from both.

Call $model->delete() in both shells. BOTH return true.

A model indeed always returns true at the end of a delete statement. The thing is that concurrency is something you'll need to deal with in your own app and can't be done within model class itself. We have no way of knowing if a record is being manipulated at the same time.


Tutorial1 - BiLSTMPOSTagger，don't have pack pad？

For varying-length sequences，Why don't have pack pad？
def forward(self, text):
        embedded = self.dropout(self.embedding(text))

        outputs, (hidden, cell) = self.lstm(embedded)

        predictions = self.fc(self.dropout(outputs))

        return predictions

Oh，there is a line
self.embedding = nn.Embedding(input_dim, embedding_dim, padding_idx=pad_idx)
point out padding_idx=pad_idx，but why isn't pack_padded_sequence&pad_packed_sequence needed？

You only need to use pack_padded_sequence and pad_packed_sequence if you want to do something with the final hidden (and cell) states, denoted in the code as hidden and cell, respectively from outputs, (hidden, cell) = self.lstm(embedded). This is because the packing and padding allows us to get the final hidden state from the last non-pad element in each sequence within the batch.
When doing POS tagging we want to do something with a sequence of hidden states, which we get from the outputs variable from outputs, (hidden, cell) = self.lstm(embedded).  Yes, some of those hidden states will be from padded tokens, but I don't believe there is a nice way in PyTorch to get a sequence of hidden states up to a certain point. However, this doesn't really matter because when we pass the padding tag index to the CrossEntropyLoss we're telling PyTorch not to calculate losses over these pad tokens. We're only losing the time taken to run the LSTM over the pad tokens, which should be minimized by the BucketIterator that is designed to reduce the amount of padding within a batch.
The padding_idx=pad_idx argument to nn.Embedding does something different. It means whenever a token that has been numericalized to pad_idx is passed to the nn.Embedding it will return a tensor of zeros at that position, see: https://stackoverflow.com/a/61173091

Thank you for your reply.

It means whenever a token that has been numericalized to pad_idx is passed to the nn.Embedding it will return a tensor of zeros at that position.

But why we need a tensor of zeros at the position of pad_idx ？Will the parts of pad that haven't been packed but still here affect the training of LSTM？OR, what's the purpose of PACK other than reducing training time？
Besides，can I use pack_padded_sequence and pad_packed_sequence instead of padding_idx=pad_idx？


nimble: add monitor protocol/interface + improve logs

monitor protocol core is now available in separate PR: https://github.com/apache/mynewt-core/pull/372
this will be for logging part only
See here for details: http://mail-archives.apache.org/mod_mbox/incubator-mynewt-dev/201706.mbox/<CADAkoF4LQOdqH%3DLh%3D5GgBPC9nEaXV95TV_uQPXXWuhrpCXfbtQ%40mail.gmail.com>

will push upload modified logging stuff in another PR


Remove unnecessary generic constraints (IEiffelClient)

Description
Remove unnecessary parameterless constructor constraints from Publish and Subscribe methods in IEiffelClient. Based on that, an update will be required in EiffelEvents.Clients.RabbitMQ as it implements IEiffelClient.
Motivation
To be able to build new abstract layers on the EiffelEvents SDK.
Exemplification
NA
Benefits
Make the codebase clean and remove not needed constraints.
Possible Drawbacks
NA

Close here #87


Add new Ironic tags

Add the Ironic containers to bring in this patch: https://review.opendev.org/c/openstack/ironic/+/876626

Looks good. Might be nice to add a reno.

Or will that come in via the xena sync?


Changes to permissions

After user testing, we found this was overwhelming for a user and the additional gallery permission led to negative feedback
Current permissions flow for a first time user
Home screen -> AI Camera button -> camera permissions -> AI Camera -> Shutter button -> Location permissions -> Saving to gallery permissions -> Additional gallery permissions -> Suggestions
Desired permissions flow for a first time user
Home screen -> AI Camera button -> camera permissions -> AI Camera -> Shutter button -> read AND write to gallery permission -> Suggestions -> "Improve these suggestions by using your location" button -> location permissions
Key changes to make for permissions

[x] whenever we ask to read OR write to the gallery, ask for a single modal to read AND write to the gallery - this should be the case whether the first permission a user sees is to save to the gallery or upload from the gallery.
[x] prevent location permissions from automatically triggering on suggestions
[ ] Instead, location permissions should trigger opt-in on suggestions and explore and projects, and any time the user opens the location picker on obs edit

[x] On suggestions, the user should see a persistent button that says "Improve these suggestions by using your location" if they have not allowed location access. Clicking this should open the location permission gate or location permissions denied screen. (this should be covered by #1592)
[ ] On explore, when a user is viewing species nearby (the default state), they should see an opt-in message to allow location access. Clicking this should open the location permission gate or location permissions denied screen.
[ ] On obs edit, the user will by default see an error state for location that says "add location." Opening the location picker on obs edit when a user has not allowed location should open the location permission gate or location permissions denied screen.
[ ] On any map screen (including explore map, taxon detail map, obs detail map) whenever a user clicks the "current location" button, the location permission gate or location permissions denied screen.
[ ] On projects, when a user is viewing nearby projects and hasn't given location access, they should see an opt-in message to allow location access. Clicking this should open the location permission gate or location permissions denied screen.
[ ] If a user denied photo Read/write permissions, user should see the gallery or camera permissions denied screen on opening cameras and gallery
[ ] If a user denied microphone permissions, user should see the microphone permissions denied screen on opening sound recorder

Also:

[ ] Fix: Location permission gate does not show correct title. Why?


Appstudio update test-component-pac-dmus

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-dmus-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
29 seconds
build-container

✅ Succeeded
7 seconds
sast-snyk-check

✅ Succeeded
14 seconds
clair-scan

✅ Succeeded
43 seconds
clamav-scan

✅ Succeeded
10 seconds
inspect-image

✅ Succeeded
8 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
4 seconds
show-summary

✅ Succeeded
8 seconds
show-sbom

Pipelines as Code CI/test-component-pac-dmus-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
29 seconds
build-container

✅ Succeeded
7 seconds
sast-snyk-check

✅ Succeeded
43 seconds
clamav-scan

✅ Succeeded
14 seconds
clair-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
10 seconds
inspect-image

✅ Succeeded
8 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
show-sbom

✅ Succeeded
4 seconds
show-summary

Pipelines as Code CI/test-component-pac-dmus-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
7 seconds
sast-snyk-check

✅ Succeeded
26 seconds
build-container

✅ Succeeded
10 seconds
inspect-image

✅ Succeeded
15 seconds
clair-scan

✅ Succeeded
6 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
12 seconds
sbom-json-check

✅ Succeeded
8 seconds
show-sbom

✅ Succeeded
5 seconds
show-summary


updating code to check the execution point in release pipeline

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign sumitsharma22
You can assign the PR to them by writing /assign @sumitsharma22 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

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

Needs approval from an approver in each of these files:

OWNERS [sohimanshu]

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

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

Needs approval from an approver in each of these files:

OWNERS [sohimanshu]

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


0.34.1 and 0.34.2 releases on Github but not electron.atom.io

0.34.1 and 0.34.2 show up as Github releases but not on the official site — as a result, we're stuck with 0.34.0 because grunt-electron cannot download anything newer.
I'm not familiar with your release process, but is it possible a few steps were omitted for the last two? Thanks!

edit: Pretty sure my grunt-electron troubles have nothing to do with releases being listed on electron.atom.io, my bad.

@fasterthanlime No worries! Just for brevity's sake we only put the major releases on the website.

Makes sense, thanks for the clarification!
(For the record, grunt-electron downloads the latest electron releases no problem, my problem did indeed lie elsewhere)


[Driver] Use simpler idiom for forwarding -target-cpu

No functionality change, but now at least we're testing this more explicitly than just in test/Misc/tbi.sil.

@swift-ci Please smoke test and merge


Use unsigned int for decision type flags

I tried to remember one phrase about unsigned types in LightGBM. It turned out that comment was about size types. But I still think that link to that comment will be good reminder for maintainers.
https://github.com/microsoft/LightGBM/issues/2077#issuecomment-503881128

@tcmt thanks for your interest in LightGBM. Please add a description to this pull request explaining how this proposed change improves the project. Without that, maintainers don't have enough context to review this.

@tmct Thank you for your contribution. Could your explain the motivation to replace the signed integers with unsigned ones for the flags?

I'm going to close this due to lack of response.
@tmct , please feel free to come back and contribute in the future when you have time to work with maintainers her.


ITMS-91061

I am getting these mails from Apple when submitting my apps for review
ITMS-91061: Missing privacy manifest - Your app includes “Frameworks/FirebaseCore.framework/FirebaseCore”, which includes FirebaseCore, an SDK that was identified in the documentation as a privacy-impacting third-party SDK. Starting November 12, 2024, if a new app includes a privacy-impacting SDK, or an app update adds a new privacy-impacting SDK, the SDK must include a privacy manifest file. Please contact the provider of the SDK that includes this file to get an updated SDK version with a privacy manifest. For more details about this policy, including a list of SDKs that are required to include signatures and manifests, visit: https://developer.apple.com/support/third-party-SDK-requirements.
ITMS-91061: Missing privacy manifest - Your app includes “Frameworks/FirebaseMessaging.framework/FirebaseMessaging”, which includes FirebaseMessaging, an SDK that was identified in the documentation as a privacy-impacting third-party SDK. Starting November 12, 2024, if a new app includes a privacy-impacting SDK, or an app update adds a new privacy-impacting SDK, the SDK must include a privacy manifest file. Please contact the provider of the SDK that includes this file to get an updated SDK version with a privacy manifest. For more details about this policy, including a list of SDKs that are required to include signatures and manifests, visit: https://developer.apple.com/support/third-party-SDK-requirements.
Is there any solution for this?

@cbjerg hm, what value do you use as IOS_FIREBASE_POD_VERSION?

I have not added that variable.
I just have

in my config.xml
Which version should I be using?

Added
  <plugin name="cordova-plugin-firebase-messaging" spec="8.0.1" source="npm">     <variable name="IOS_FIREBASE_POD_VERSION" value="11.3.0" />   </plugin>
And after pod repo update, it seems like it is building. Thank you


Authentication failed caused by 302 redirect

`for cookie in cookie_file:
self.session.cookies.set(cookie["name"], cookie["value"])
    # url = "https://www.bing.com/turing/conversation/create"

    url="https://cn.bing.com/turing/conversation/create"

    # Send GET request

    response = self.session.get(
        url,
        timeout_seconds=30,
        headers=headers,
    )`

Get the url https://www.bing.com/turing/conversation/create will get a 302 response which redirect to new url,The Authentication fail.
change the new url works.Maybe it’s about the region.

I might be able to fix this with a allow redirect flag. Do you mind hanging around to test it?

https://github.com/acheong08/EdgeGPT/releases/tag/0.0.41

If this doesn't work, it means that you can't use this in China

The new version(bf93dd7)is work


Add Bubbles for Courses.

Add Charmbracelet BubbleTea and Bubbles Table or List component (Whichever is more suitable) to Select Courses.
https://github.com/charmbracelet/bubbles

I feel it would be in best interests of the project to go forward with a TUI-first setup instead of relying on CLI. Charmbracelet offers powerful support for the same.
Here is an example of how it could potentially look like

Let me know if this is something you would want as it would make the tool feature complete.

This looks good now.
But this much of information will it actually make sense to show?

But hav to say this
it looks super cool!

But hav to say this
it looks super cool!

It is also fairly straightforward to make with charmbracelet and lipgloss packages

Then let's try it out @IAmRiteshKoushik!
Can you start working on this one?

Then let's try it out @IAmRiteshKoushik! Can you start working on this one?

Sure,  can do that in a few days time. Can we sit down sometime to discuss the TUI requirements for the application ?

Yup sure
@Abhinav-ark what do you think >>

Yup sure @Abhinav-ark what do you think >>

Not sure, TUI might be an overkill. But if we are doing a TUI, we have to design it properly.

Yeah
True
Let's give it a try! See how it goes. If it works out or it doesn't, either way we'll learn something new!

Hi , @Abhinav-ark I'd like to work on this issue, kindly give me the permission.

@RD-Tarun the issue is assigned to you. For the list do something like this https://github.com/Melkeydev/go-blueprint. Come up with a partially completed code by atleast january 8th. The issue will be reassigned in case of no progress.


New game prompt no longer works

The new game prompt no longer works; the prompts don't export automatically or show the new game dialog when "No" is clicked. There is some sort of mobx issue going on where it doesn't re-render the new game dialog or the export dialogs.

Found the source of the bug. We set the right dialog to show, but then we close the message dialog, which clears all of the modal dialogs. We need to be smarter about which dialogs we hide.


Update README to provide instructions for base64-encoding image inputs

Context
Update sign-off prior commit in git history + Add basic base64 example

@GothReigen Thanks for opening this PR. As I mentioned in https://github.com/replicate/replicate-python/pull/244/files#r1493371435, I believe this base64-encoding behavior is automatically provided for file handles. Can you think of any reason not to use that?

@GothReigen I think the discussion a few lines up in the README does a sufficient job of conveying this information. I just pushed 6725e8274b6929fd49a5e3f3c6859b4707f86e91, which updates the existing example with your changes.
Thanks again for your contribution. Merging now.


Builder proposer settings / profile

Problem description
As discussed on discord it would be useful to be able to configure a different builder URL per validator. It is relevant for DVT setups when running different solutions like Obol and SSV together via same beacon node which requires builder relays to be different. This can be achieved by running two local mev-boost instances with the specific relay configuration but right now it also requires to run multiple beacon nodes as it is only possible to configure one mev-boost instance per node.
Solution description
Solution 1
Similar to the proposer settings file on the validator client we could introduce a  (builder) proposer settings file on the beacon node which allows to configure a builder URL per validator pubkey.
Solution 2
Could have a profile based approach instead of key based (as SSV is all automated) and the builder_profile would need to be passed by Obol or SSV in produceBlockV3. The builder_profile option could be integrated into the validator's proposer settings config and keymanager to allow key based configuration. And on the beacon node side we would require a profilename => builder url mapping with a default (current builder.url cli options will override/set the default).
Additional context
No response

I think from a design perspective, if we had a profile based solution, does this give us much more flexibility to adapt to various DVT infrastructures? Both Obol and SSV function slightly differently and if we think about other upcoming infrastructures like Diva, it seems like with profiles we can customize more to adapt to the individual stack differences.

Could be additional to the alpha functionality we have with proposer_config.yaml

Could be additional to the alpha functionality we have with proposer_config.yaml

The main issue with this is that the proposer config is on the validator client side and it would have to communicate that information somehow to the beacon node which manages the builder connection.
I think this needs to be configured on the beacon node side.

It is relevant for DVT setups when running different solutions like Obol and SSV together via same beacon node which requires builder relays to be different

This is already solved by Vouch which you can run as a mev sidecar as well and configure different relays per pubkey.


Update edit-guests-information.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/6083
Typo

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@Kateyanne PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


[Bug]: Checkbox widget - Label -> Position - On selecting Right, it still remains where it is.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I select the Right option from the Label -> Position property of Checkbox widget, it remains on the left.
No difference is seen if we select Left or Right.
Expected behavior - Previously, on selecting Right, the Label text used to be aligned to the right of the widget.
Steps To Reproduce

Drag and drop a checkbox group widget and increase the width of the box.
Select the Right option from the Label -> Position property
Note that there is no change in the position of the Label text.

Earlier behavior -

Public Sample App
No response
Version
Release

Stats

Stat
Values

Reach
392

Effort (months)
0.5

Working fine now, moving to done


Player disconnected when type on sign.

DEBU[0725] 2023/12/08 01:38:28 unexpected packet with ID 56  src=gophertunnel

This was fixed in https://github.com/Sandertv/gophertunnel/commit/539a193680b2df72ef627cd34fbba01105a358dc


Cleeng TVOD support

TVOD is the opposite of SVOD. Rather than giving viewers access to a broad content catalog, content owners would give them access to a single piece of content. It's was a way to monetize popular premium content and recent new releases.
TVOD can either be purely transactional (pay for unlimited access), or it can be a rental model.
Scope

TVOD only
TVOD titles, in an SVOD model
TVOD with upgrade to  SVOD

Details
https://docs.google.com/presentation/d/1zxn-XxFYSaFr9o7YcGdm4iC17DzsmaLkswStda4zRNA/edit#slide=id.g63129a3642_0_0
Notes

Align with Applicaster productId
Support SVOD as well
Exclamation mark (!) to override global offers for premier access)

Could you please confirm that TVOD for live streams (including count down to live event and purchasing something that is not available to watch yet, etc) is not a requirement for this sprint?

Marked it as out of scope.

Fixed by #66


refactor(api): remove InstrumentContext.speed on PAPI v2.14

This PR adds a version guard + error to the InstrumentContext.speed property setter used to set the pipette's plunger speed. We made this change, after some discussion, for the following reasons:

Name confusion: InstrumentContext.speed sets plunger speed, while InstrumentContext.default_speed sets movement speed.
Inexact behavior: Pipettes have a step-wise speed function that is determined by the current volume in the tip, and the HW API keeps track of flow rate in µL/s. Setting a "plunger speed" in mm/s via the Hardware API involves backing a flow rate out of the step-wise function using an arbitrarily selected volume, meaning the plunger speed that is actually used approximates the desired plunger speed without matching it exactly.
Implementation effort: Setting the plunger speed in mm/s has some value for hardware validation testing, but seems to lack value for actual scientific protocols. Given the extra effort it would take to implement this concept on the ProtocolEngine system, and given the relative lack of user-value in a world where the flow_rate setter exists, we would rather not sink time into getting this corner of the Protocol API working in the 2.14 release.

Closes RCORE-621
Test Plan
No special testing required
Changelog

remove InstrumentContext.speed on PAPI v2.14
update versioning docs
knock out the last few remaining NotImplementedErrors in the engine core while I was here

Risk assessment
Low

Do we need to remove this note (or keep it and revise the "is equivalent to" language)? https://github.com/Opentrons/opentrons/blob/fb50d91d3287246fb4612d013a737b5c39eaea06/api/src/opentrons/protocol_api/instrument_context.py#L1319-L1326

@ecormany merging this PR to unblock 6.3.0 testing, but your docs feedback is noted and will open a separate docs-only PR to address


Slack Bolt with Flask and sqlalchemy context

I'm trying to get context for getting access to my Db models using python3  / slack_bolt / flask
Reproducible in:
app.__init__.py
def create_app(**config_overrides) -> Flask:
    app = Flask(__name__)
    app.config.from_object(get_config())
    with app.app_context():
        from .db import init_db
        db.init_db(app)

        from .views import init_views
        init_views(app, db)

    return app

def create_slack_app():
        from slack_bolt import App
        slack_app = App(
            token=get_env("SLACK_BOT_TOKEN"),
            signing_secret=get_env("SLACK_SIGNING_SECRET")
        )
        return slack_app

slack_app = create_slack_app()

app.db.py
from flask_sqlalchemy import SQLAlchemy
from flask_migrate import Migrate

db = SQLAlchemy()

def init_db(app):
    db.init_app(app)
    migrate = Migrate(app, db)

    return db

app.models.py
from .db import db
....

class User(BaseModel):
    __tablename__ = 'user'

    slack_id = db.Column(db.String, nullable=False)

and for example, one of my view
app.views.view1.py
from app import slack_app

handler = SlackRequestHandler(slack_app)

@slack_app.action("multi_ext_select")
def handle_submit(client, body, ack, logger):
     from app.models import User
     User.query.first()

User.query.first() will fail with RuntimeError: working outside of application context
It seems that i'm not the only one coming accross this issue https://bigbunny.io/handling-flask-application-context-for-slack_bolt.html, but i still quite dunno how to solve it , could you help me out please ?
Thanks a lot
The slack_bolt version
slack-bolt==1.18.0
slack-sdk==3.21.3

Python runtime version
Python 3.11.3

if i remove all occurences of slack_bolt and use purely my flask app, i get the context to access db, but i would have prefer to try slack_bolt for a chance

HI @krab-skunk, thanks for writing in!
I haven't tried to reproduce your issue in your app design, but we have a simpler example here: https://github.com/slackapi/bolt-python/blob/main/examples/sqlalchemy/oauth_app.py As long as you simply use an Engine object in listeners, you won't be botherred by the thread local related issues (I mean Flask heavily relies on thread local data sharing inside the framework, which prevents async ack() mechanism in bolt-python).
If you still have a certain reason to depend on Flask app context for managing database connections, checking our Django ORM support (https://github.com/slackapi/bolt-python/blob/v1.18.0/slack_bolt/adapter/django/handler.py#L61-L91) may be helpful for you to learn what you can do. That being said, this project does not recommend app context for the use case. Thus, we won't be able to provide further help on this topic. It'd be appreciated if you could understand this.
I hope this helps.

Hi @krab-skunk , I am facing the same issue. The only way I was able to resolve it was by using the app context before the query. For example
with flask_app.app_context():
    User.query.first()

Have you been able to find a better way?


Can not export results when using local storage

Describe the bug
I am using the latest docker version of the label studio, it is a really great tool and I loved to use it. However, I found a bug when I would like to export my labeling results.
First, there are two exact same cards in the Cloud Storage when I chose Local Files as my storage type, I haven't tested it for a remote server yet, but I think it produces the same problem.

Secondly, when I set Local Files as my preferred Target Storage type, there is no output in the destination folder at all. I am sure that the output path is correctly mapped since the input files are stored in the same parent folder and everything works fine except for not showing the source file name.

What's more confusingly, the output files generated by the download link did not contain the source images. As you can see in the following picture, the size of the image folder is 0.

To Reproduce
Steps to reproduce the behavior:

Choose a project, Go to 'Settings'
Click on 'Cloud Storage' on the navbar
Select local files as preferred storage type
Go back to the project page and click the export button on the top right corner
Choose YOLO as the export type
See the error

Expected behavior
The exported files should be in the corresponding local path and the zip file that the download link provides should contains the original images
Screenshots
See above
Environment (please complete the following information):

OS: Windows 10 Enterprise N 20H2, OS build: 19042.1165
Label Studio Version [docker 1.2.0+ DIGEST: 9a1562eff204]

Additional context
None

Hey, I have the exact same problem with Local Cloud Storage.
That cancels the automatic training of my model by the way because there is no images in the export, so the model can't start training without data and only labels...

@ChainYo @Mano-Liaoyan Downloading and preparing archives with tons of images can be a complex task and we tend to drop support of this. So, I can recommend to manually copy your images into 'images' folder. Is it possible solution for you?

Hey, I have the exact same problem with Local Cloud Storage.
That cancels the automatic training of my model by the way because there is no images in the export, so the model can't start training only with labels.
@Mano-Liaoyan Did you find a solution without copying manually images data ?

I apologize that it took so long to reply to your message, and I'm sorry to tell you that I ended up solving the problem by manually copying the files as well. Luckily I stored it on a relatively fast private NAS and it didn't take me much time, hopefully, your problem will be solved well too.
Have a nice day

I solved this issue by adding a personal script to copy images from the initial directory to the exported one.
My comprehension of the problem: There is a problem between local storage path and exported images path. Something is weird because the links are d/? + ìmage_name`. There is 3 digits/letters before the image name, maybe referencing to the local storage path, but the real local storage path isn't found by the app.
I will probably go with cloud storage next time instead of local storage, but it was easier for the PoC I was on.
Best regards

Hello, is there any news about this problem of image path when using a local synchronization?
Best regards

@macfly1202 Could you clarify, what's wrong with your image path? Do you use target storage and an image path is incorrect there?


Add a capability to pass an option to remove the initialContent autom…

…atically pushed in the aposArea local.
Let me know if this is going to break something, my goal was to make it so I could make a limit: 1 area that doesn't allow text and doesn't require you to initially delete a placeholder text block.

Please try this out with a "boring" ordinary area just as a sanity check
first.
On Wed, Feb 24, 2016 at 10:07 AM, Jimmy Harrison<EMAIL_ADDRESS>wrote:

Assigned #446 https://github.com/punkave/apostrophe/pull/446 to @boutell
https://github.com/boutell.
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/pull/446#event-563742991.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Tested in a normal area with no option set. Initial content appears as expected, can click into it and edit it. Verified that the initial content does not appear for non-editors. Added a few widgets, everything seems to work.


add my-info

I've added my info in the last paragraph of the code as suggested in the Readme file. Kindly Check it !

Can you tell me where is the conflict in this pull request?


build-fhs-userenv: fix ssl certificates mount point

Motivation for this change
The host SSL certificates at /etc/ssl/certs were not being mounted on the FHS environment as they were getting overriden by the default /etc/ssl path in the FHS environment. Similar issue to #110825.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer


Fabric 1.20.1 Release 0.15.4 status effect crash

Describe the Bug
Upon loading up the create-fabric-0.5.1-d-build.1161+mc1.20.1 along with fabric-api-0.91.0+1.20.1 on Fabric - 0.15.4 and loading a world if you receive any status effect and proceed to open your inventory you will crash.
Reproduction Steps
1.Create a Fabric - 0.15.4 Profile.
2.Download create-fabric-0.5.1-d-build.1161+mc1.20.1 along with fabric-api-0.91.0+1.20.1
3.Load up the game.
4.Create a new creative/survival world.
5.Acquire any status potion effect or otherwise.
6.Open Inventory.
7.Crash.
Expected Result
The player Inventory opens without any crash due to effects.
Screenshots and Videos
No response
Crash Report or Log
https://gist.github.com/ZeonEsper/7f3c39a995b6c73b2fbb0d9d27933ec3
Operating System
Windows 11
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods
No response
Additional Context
No response

update to patch F


Clean up spec for Metal Gateways

Taken from a comment on #128.
There are a couple inline schemas for Metal Gateways, which leads to duplicated code, an inability to reuse objects across methods due to type mismatches.
The spec section that corresponds to FindMetalGatewayById200Response is here: https://github.com/equinix-labs/metal-go/blob/main/spec/oas3.fetched/paths/metal-gateways/id.yaml#L56-L58
oneOf:
- $ref: '../../components/schemas/MetalGateway.yaml'
- $ref: '../../components/schemas/VrfMetalGateway.yaml'

This chunk should be moved to a file under components/schemas/, but we'd need a good name for it.  Maybe rename the existing MetalGateway.yaml to NonVrfMetalGateway.yaml?  Is there a better name we could come up with?  Names like GenericMetalGateway.yaml (for the new component) have caused concern in the past, but I'm not sure how to give the existing MetalGateway.yaml a more specific name without referencing VRF.
There is a similar inline schema inside MetalGatewaysList.yaml; that one uses an anyOf instead of a oneOf, but we should really use the same relationship in both places so that we can improve reused of our components.

Normally I'd say we should use a oneOf, since we expect the API to return either a MetalGateway or a VrfMetalGateway, but given that non-VRF gateway properties are a subset of VRF gateway properties, aligning on anyOf may lead to a better user experience.

After some experimentation, we've found that if we include=ip_reservation, we can rely on the IP reservation to identify what type of Metal gateway was received (a VRF gateway has an IP reservation with type vrf, whereas a non-VRF gateway has an IP reservation with one of the types [global_ipv4, public_ipv4, private_ipv4, public_ipv6]).
That means in the near term we could keep the oneOf construct, which is IMO preferable, and document that the IP reservation must be included to find the type.
For a more permanent fix we could introduce a type field on Metal gateways on the API side so that there is a way to determine what kind of Metal gateway was received without needing to include other resources.

This repo is deprecated and a corresponding issue has been created in equinix/equinix-sdk-go


[BUG] After refreshing a novel, the manually changed info and cover reset to the default one.

I changed the cover of a novel, after that refreshed the chapter list. Then the cover re-sets itself to the default one. Please fix, thanks!

This also happens to the edited info.


Use out of band docker build with sbt assembly

let me run through some tests in a test deployment before we merge this

gonna close this, the builds seem to take longer, not sure if this gives us too much advantage elsewise


feat(v2): add pt-PT translations for docusaurus-theme-classic

Motivation
Following this comment  asking if it was possible to also have the translation for pt-PT translations, I wanted to help.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Since I only added a new translation JSON, there is no test plan, the existing tests should do fine.
Related PRs
None

thanks!

Was wondering, if a user choose locale = "pt", should we apply PT or BR translations by default?

That is actually a good question! If you think with maths, it would probably be better to use pt-BR, since there are waaaay more brazilian people than portuguese people, however pt-PT is the "original" ahahah
If I had to choose only one, maybe pt-BR, since it more probable that someone would brazilian rather than portuguese.

I see
But if you want a Docusaurus site in Brazilian, would you try using "pt" instead of using directly "pt-BR"?
Maybe we should have pt.json (for pt + pt-PT) and pt-BR.json?

Yeah, that's probably the best approach, since I believe most brazilian people are more used to seeing "pt-BR" than simply "pt".

Apparently, it's pt-BR :)
https://twitter.com/sebastienlorber/status/1376825837417340931

Fair enough then, we learn things every day ahahah

"pt" -> "pt-BR"  implemented in https://github.com/facebook/docusaurus/pull/4581


error 500 on methodes page

 production.ERROR: Trying to access array offset on value of type null {"view":{"view":"/Users/jeong-gwang-il/Downloads/SREAL/WebErpMesv2/resources/views/methods/methods-index.blade.php","data":{"errors":

@darrelljung111 I cannot recreate the error, have you entered data on the methods?

As you put on manual, I followed below instead of php artisan db:seed
php artisan db:seed --class=PermissionTableSeeder
php artisan db:seed --class=CreateAdminUserSeeder
Now I don't have 500 ERROR on Dashboard or Methods. I appreciate your fast fix the issue!!
I will test more and will let you know more if I found out the problem.
Thank you so much!!!

Cool, Listening to you, thank you for your interest


Update README.md

What problem does this PR solve?
Now the architecture picture is much larger(perhaps too large?).
What is changed and how it works?
Replaced URL.
Check List
Tests

No code

Side effects
Related changes

/run-all-tests


Code of Federal Regulations citations

Many examples, almost as complex as US Code citations. Some:
14 CFR part 25
Title 41 CFR Sections 102-74.435
38 CFR Part 74.2
48 CFR § 9903.201
23 CFR 650, Subpart A
Appendix A of 10 CFR Part 440
24 CFR 85.25(h)
47 CFR 54.506 (c)
5 CFR §531.610(f)
5CFR, part 575, subpart C
19 Code of Federal Regulations (CFR) Parts 12
13 CFR Parts 125 and 134
parts 121 and 135 of Title 14 of the Code of Federal Regulations
45 C.F.R. 3009.4

probably not going to get:
parts of title 22 of the Code of Federal Regulations (CFR) including Parts 226, 228 and 216

Declaring this full-done


upload failed

about 78% it break.
test twice , show same result.

tested
upload file size cannot over 1199MB.
dont know why.

I cannot reproduce the issue.
An upload of a 1.75GiB file works fine on my end in both Firefox and Chromium on Linux, with the server deployed through Docker on an x86_64 machine:

Bear in mind your server currently caches the whole file in RAM while receiving data before committing it to disk. I understand this is not ideal, but could explain issues on the backend.
If you want me to actually help you, you must provide details to reproduce the issue:

What version are you running?
Are you running from source, or with Docker? Have you tried both?
What are the specs of the server machine? Host operating system, processor, RAM, available storage, etc.
What operating system, browser and broswer version are you using to make the upload?
Especially important are the logs of both client and server.

You can open the browser console with Ctrl+Shift+J and check for errors on the client.
You can check the server's logs in Docker with docker compose logs.
If you're running from source the log is printed directly to STDOUT.
There should be a clue here as to why the server could not process the request.

That said, while testing I did notice that the WebCrypto-API imposes a 2GiB limit on the encrypted filedata. This is not the issue you're experiencing, however, as the error would manifest during encryption, not during upload.
I will update the docs and configuration limits to account for this.


TypeError: undefined is not an object (evaluating 'pushNotification.areNotificationsEnabled')

Hello,
When I call areNotificationsEnabled method, I receive that error :
TypeError: undefined is not an object (evaluating 'pushNotification.areNotificationsEnabled')

How to properly call this method ?
I call that method like that :
        var options;
        if (device.platform == 'android' || device.platform == 'Android' || device.platform == 'amazon-fireos')
        {
            options =   {
                            "senderID":"661780372179",
                            "ecb":"onNotificationAndroid"
                        };
        }
        else
        {
            options =   {
                            "badge":"true",
                            "sound":"true",
                            "alert":"true",
                            "ecb":"onNotificationIOS"
                        };
        }

           pushNotification.areNotificationsEnabled(
                function()
                {
                    console.log('OK ?? areNotificationsEnabled');
                },
                function()
                {
                    console.log('NOTOK ?? areNotificationsEnabled');
                },
                options
            );

sorry, it was my bad. just changed pushNotification object to window.plugins.pushNotification, it is ok now.


Add info about how to fail the build from AzurePowerShell task

It would be really helpful to include a Write an error section that documents the Azure PowerShell task requires the code to use
[Environment]::Exit(1);

if the developer wants the error to fail the build.
The PowerShell task's documentation includes a Write an error section with a tip documenting that to fail a build developers should use:
exit 1;

We have used the PowerShell task a bunch in our pipelines. The first time we used the Azure PowerShell task we lost a lot of time trying to figure out why the Azure PowerShell task wasn't failing the build even though the exit code was being set.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: dcda4373-62ae-e1ae-c15d-63f441179b47
Version Independent ID: 305f78c0-2e89-78f2-d2d1-89471602610c
Content: Azure PowerShell task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-powershell.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

@robhalevt -- Rob, thank you for your considered feedback.
@RoopeshNair -- Roopesh, please look into this issue.


Revision update

Description of your changes

Added new sh script to automatically install environment variables.
Re-added the old/original gene.output_slice.
Added name attribute to gene class.
Added logic for optionally using old data transforms in hound_data.
Cleaned up seqnn.gradients function.
Added missing stream class.
Updated README with install instructions.

Type of change

[X] Bug fix
[X] New feature
[X] Refactoring / code clean-up
[X] Documentation add / update

(If applicable) How has this been tested?
Tests:

Re-ran all scripts from 'borzoi-paper' repo and 'borzoi' tutorials.
Checked that all major results from the paper reproduced (test evals, e/s/paQTLs).

Environment:

Python 3.10
TensorFlow 2.15

All requested changes are made.


⚠ ClamAV has degraded performance

In 5416793, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in e253243.


AssetLib: Only notify when unavailable in verbose mode

Follow-up to #62531. Makes this less of an annoyance in Web editor logs.

Cherry-picked for 3.5.


Flex gap

Implements rowGap and columnGap. Creating a draft PR for testing. Will update this section soon! Feedbacks are welcome.
Approach

This PR adds React native binding for this Yoga PR

To test the example

Run rn-tester as per the instructions in readme.md. I've modified RNTesterAppShared.js with flex gap example.

https://user-images.githubusercontent.com/23293248/146173072-bce93401-cd37-413a-bd3e-40efda38a59a.mov

Would also be cool to support gap property which is a shorthand for both row-gap and column-gap

Added gap property 🎉

@TheSavior this is a patch that implements one of the layout tasks on the convergence roadmap 🥳

Hey Eli and Nicolas! Can you tell us more about the convergence roadmap please? Thanks 🙏

yoga and browser flexbox layout are slightly different and we'd like yoga to better match the flexbox spec

OK cool! Feel free to share your plans with the community, I'm sure we can help

Updated gap approach with @jacobp100's implementation. Margin based implementation felt hacky and it also used owner_ property directly as discussed here. Thanks to @jacobp100 for pointing it out. Once the Yoga PR gets merged, we can move this to ready for review. 🥳

Any updates? This is a highly demanded and killer feature in my opinion which should be expedited

Does anyone have a patch for react-native + yoga for using flex gap?

@nandorojo this is the new PR. will close this one.


improve handling of data folder input

could have an option to call it from the command line by 'dragging' the data folder as an argument:  python -m run_tetrodes "Folder that I drag here"
could have an option to select the data folder with the os explorer

Now doing it with 2 possible ways: write as an input to the code or give an empty folder input and select it by navigating to it instead


Fix bug in update_shipment hook

The bug occurs whenever an order contains a variant with the same SKU as a deleted variant with a lower id. The error thrown is a "Can't find line items with SKU . . ." error created here.
I've had a lot of these errors clogging up my instance of wombat on a recurring basis. I imagine anyone else who has replaced old variants with new versions that have the same SKUs has them as well.

Thanks I've merged w/ https://github.com/spree/spree_wombat/commit/5dadea37c19fd017722366d16d7fab003095ada9


Sort environments, fix preview

Fix a string comparison in zfsbootmenu-preview.sh to make it an exact
match instead of a regexp match. This will prevent multiple environments
showing up as (default).
In populate_be_list, reverse sort the list of candidate boot
environments and then append the BOOTFS value if defined. Combined with
fzf/sk --tac, this pushes the boot environment to the top of the list so
that it's the first selected item on the main screen.

Validating bootfs before the countdown timer could be useful; except it'll produce weird behavior on systems with native encryption turned on. There will be a password prompt and then the countdown timer will start - the reverse of the current process.

I didn't think about the effects of encryption on pre-validation of $BOOTFS to short-circuit the menu. Asking for a passphrase before presenting the countdown timer seems backwards. The current behavior seems best for now.


import pkg_resources triggers a DeprecationWarning on Python 3.9

I'm pretty sure this is a bug because I don't think pkg_resources is deprecated in its entirety.
$ PYTHONWARNINGS=error python3 -c 'import pkg_resources'
Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/packaging/version.py", line 57, in parse
    return Version(version)
  File "/usr/lib/python3.9/site-packages/packaging/version.py", line 298, in __init__
    raise InvalidVersion("Invalid version: '{0}'".format(version))
packaging.version.InvalidVersion: Invalid version: '/usr/lib/python3.9/site'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 3239, in <module>
    def _initialize_master_working_set():
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 3222, in _call_aside
    f(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 3251, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 558, in _build_master
    ws = cls()
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 551, in __init__
    self.add_entry(entry)
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 607, in add_entry
    for dist in find_distributions(entry, True):
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 2056, in find_on_path
    path_item_entries = _by_version_descending(filtered)
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 2026, in _by_version_descending
    return sorted(names, key=_by_version, reverse=True)
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 2024, in _by_version
    return [packaging.version.parse(part) for part in parts]
  File "/usr/lib/python3.9/site-packages/pkg_resources/__init__.py", line 2024, in <listcomp>
    return [packaging.version.parse(part) for part in parts]
  File "/usr/lib/python3.9/site-packages/packaging/version.py", line 59, in parse
    return LegacyVersion(version)
  File "/usr/lib/python3.9/site-packages/packaging/version.py", line 127, in __init__
    warnings.warn(
DeprecationWarning: Creating a LegacyVersion has been deprecated and will be removed in the next major release

Cross-posted to https://github.com/pypa/packaging/issues/368 because I'm not sure whether the bug is in packaging or in setuptools.

The issue seems to be that _by_version_descending() in setuptools attempts to pass invalid version strings to packaging. With this patch
diff --git a/pkg_resources/__init__.py b/pkg_resources/__init__.py
index 737f4d5f..6ca1c7b4 100644
--- a/pkg_resources/__init__.py
+++ b/pkg_resources/__init__.py
@@ -2020,7 +2020,8 @@ def _by_version_descending(names):
         Parse each component of the filename
         """
         name, ext = os.path.splitext(name)
-        parts = itertools.chain(name.split('-'), [ext])
+        parts = list(itertools.chain(name.split('-'), [ext]))
+        print(parts)
         return [packaging.version.parse(part) for part in parts]

     return sorted(names, key=_by_version, reverse=True)

I got
['/usr/lib/python3.9/site', 'packages/MarkupSafe', '1.1.1', 'py3.9', '.egg-info']

With that observation, I created a fix(?)
diff --git a/pkg_resources/__init__.py b/pkg_resources/__init__.py
index 737f4d5f..84df31dd 100644
--- a/pkg_resources/__init__.py
+++ b/pkg_resources/__init__.py
@@ -2020,8 +2020,15 @@ def _by_version_descending(names):
         Parse each component of the filename
         """
         name, ext = os.path.splitext(name)
-        parts = itertools.chain(name.split('-'), [ext])
-        return [packaging.version.parse(part) for part in parts]
+        parts = list(itertools.chain(name.split('-'), [ext]))
+
+        def parse_version_ignore_invalid(version):
+            try:
+                return packaging.version.Version(version)
+            except packaging.version.InvalidVersion:
+                return version
+
+        return [parse_version_ignore_invalid(part) for part in parts]

     return sorted(names, key=_by_version, reverse=True)

This is also hit in https://github.com/dask/fastparquet/issues/558. @yan12125's investigation makes it clear: passing an invalid version (the first parts of the name here) throws the DeprecationWarning of LegacyVersion.
https://github.com/pypa/setuptools/blob/c121d289da5d19cf6df2bf6b64ac28916a060161/pkg_resources/__init__.py#L2025-L2027
https://github.com/pypa/setuptools/blob/c121d289da5d19cf6df2bf6b64ac28916a060161/setuptools/_vendor/packaging/version.py#L48-L58


Backward looking years

Currently the Years component is hard-coded to be forward looking, and for users to see past years they need to navigate there via the buttons. I have a use case where the date can't be the future (an easy example of this is a birthday) and it would be useful in that instance to have a prop I could pass, like isForwardLooking={false} so that I could have the years looking backwards.

Result:


slingshot: add README with quick outline of all components

When we share a link to this repo, it'd be good to have a map of all the projects with related links all in one place.

Closed by #165


Allow specifying nav_shape, sig_shape and sync_offset for K2IS

Continuation of #793.
nav_shape and sig_shape can be easily implemented. sync_offset implementation was almost finished, check this comment and this one.

Replying to this comment:

Now I've a better understanding on how K2IS's sectors are synced. My implementation works for minor changes to the offset but has some issues for example when sync_offset is set to 0 and all the data without the shutter_active flag is read too, but I'm working on it.

I think having sync_offset=0 return all frames from the beginning is actually the right behavior. Would this problem be possibly fixed by considering a different default value for sync_offset? If the default were None, it could signal that the shutter active flag should be used and sync_offset should be set automatically. Thoughts?

I think having sync_offset=0 return all frames from the beginning is actually the right behavior.

I agree.

If the default were None, it could signal that the shutter active flag should be used and sync_offset should be set automatically. Thoughts?

Yup, that was the plan.

but has some issues for example when sync_offset is set to 0 and all the data without the shutter_active flag is read too

I should have better explained the issues - actually I was getting weird frames when I set an offset <= 0. But a negative offset should mean blank frames so something was wrong with my implementation. I'll definitely look into this while working on our K2 simulator!

I'll definitely look into this while working on our K2 simulator!

Great! :+1:


[VS Code Insiders] Issue with icons in "Open Editors" when pane is narrow

Bug description
If left pane is narrow, file names cover icons. This does not happen with default icon sets that come with VS Code, so it may be an issue with your extension.

Not a duplicate
VS Code version:

Screenshot of issue:

I cannot reproduce it. I made the left pane very narrow and I even tried to add a file with a long path unsuccessfully. @JimiC, @jens1o, maybe you'll be luckier than me.

If this issue is not present on stable vscode then it's not an issue with the extension but with vscode itself.

I don't think it's an issue with VS Code because the default icon sets don't have this issue. I get a new Insiders build each day, so I'll report back if it gets fixed... otherwise, I'll just change the title in a couple of weeks when they fork and a new stable release goes out.

The thing is I wasn't able to reproduce it in both stable and insiders version. Can you provide a video or animated gif of how you do it?

Also there is a possibility that another extension is messing with ours.

Here you go: https://streamable.com/a5bao
This is on my personal machine (MacBook). Original repro was on my WIndows 10 machine.

I also reproed the issue with all of my other extensions disabled.

@martellaj I'll try to test in on a Mac, but I definetely can't reproduce it on my Windows machine.

@martellaj a few questions:

Does this happens to you in the stable version?
Does this happens to you to any other non-default icon set (i.e. Material Icons...)

Oh man this due to the new feature in insiders that shows modified file of scm.

I still don't see where this could be our fault. :c

It isn't, that's what I'm saying from the beggining.

You're right @JimiC. It's the new vscode feature. See this happening with Material Icons.

(didn't found some more matching labels)

@jens1o, I would remove bug and confirmed - as this is not our bug but vscode`s - and close the issue.


Motion permission

2.0 b5-develop: Gave all permissions for motions to "Delegates"-group, but if you log in as delegate, you can't click at a motion to see more details (text etc.).

Looks like the delegate user has no 'see_see_hidden_items' permission. Then you get a 404 error, see https://github.com/OpenSlides/OpenSlides/pull/2030#issuecomment-192706200

Not really sure that I give the correct permission...
[I 160306 18:57:31 wsgi:355] 200 GET /rest/motions/motion/12/ (<IP_ADDRESS>) 48.40ms
[W 160306 18:57:31 wsgi:355] 403 GET /rest/mediafiles/mediafile/ (<IP_ADDRESS>) 21.45ms

I can reproduce it now:
If a user has no permission to see mediafiles, hidden agenda items OR users the loading of the motion detail view failed with a 404 error (see resolve object of motions.motion.deatail). That's an important (general) issue for 2.0 - because defining reduced see permissions for users caused in these 404 errors...
@ostcar Is there a possibility to catch these 404 errors? Checking if operator has required permissions in resolve part looks a bit to much... and does not work for me)

It seems, that angular ui-router has a problem, if a promise in the resolve part returns a 403.
One solution would be, that you do not load the data in resolve, but in the directive. Another solution would be, that you write your own promise, that does not return 403 but an empty list.
This seems to be another solution, but I could not understand it: https://stackoverflow.com/questions/23877304/angularjs-handle-rejected-response-with-ui-router


Added SAP-samples/security-services-tools

Added SAP-samples/security-services-tools

@larshp ah, sorry, didn't think about thta.


drivers: can: add missing can_get_max_filters() syscall handler

Add missing syscall verification handler for can_get_max_filters() (fixes #44687).
Add test for the can_get_max_filters() optional API function.

Implementing this test revealed a few issues, which are also fixed by this PR in order for the added test to pass:

CANopenNode needs not depend on the optional can_get_max_filters() API function.
#44721
#44724

Implementing the test for the can_get_max_filters() syscall also revealed an issue in the STM32 bxCAN driver, which is not addressed by this PR (except for removing support for can_get_max_filters() from that driver for now). I have opened #44725 for dealing with that.

Should get_max_filters() be made mandatory? The testcase sure looks generally useful

@karstenkoenig Thank you for reviewing. I have considered making the can_get_max_filters() syscall mandatory, but I am not convinced all drivers are able to implement it.
Consider for instance a driver for a CAN controller with native support for either fully-masked 32-bit filters or two semi-masked 16-bit wide filters. Depending on the masks of the added filters, they could either fit in one of the 16-bit filters or one of the 32-bit filters. This would make the number of supported filters dependent on the masks of the added filters and thus not a constant.
The current STM32 bxCAN driver attempts to do something like this at the moment and its filter limitations (and can_get_max_filters() implementation) are completely broken, which is why I removed the syscall from that driver and opened #44725. This, at least, needs to be fixed and the syscall re-added to that driver before making it mandatory.


My Fire HD 10 won't connect to scrcpy

[X] I have read the FAQ.
[X] I have searched in existing issues.

Environment

OS: Chrome OS 11.0.5563.71
scrcpy version: 1.17
installation method: manual
device model: Fire HD 10 (2021, 11th gen)
Android version: 9

Describe the bug
A clear and concise description of what the bug is.
Okay so, I have been trying to connect my fire tablet to scrcpy, but, it never works. I always get the warning
INFO: OpenGL version: 3.1 Mesa 21.2.6
INFO: Trilinear filtering enabled
INFO: Initial texture: 1920x1200
WARN: Device disconnected
WARN: Killing the server..

Now, I know you might say try another cable, which I've tried, I've tried doing it wireless, but nothing works. The screen does turn on when I run the command scrcpy -s [serial] But nothing happens. And sometimes it says it can't retrieve my device information. Whatever/whoever can help/fix this problem, I appreciate it.

scrcpy version: 1.17

Could you try the latest? (currently, 2.0)
Could you copy the whole console output, please, with scrcpy -Vdebug?
Also try with another --video-encoder: https://github.com/Genymobile/scrcpy/blob/master/doc/video.md#codec

Could you try the latest? (currently, 2.0)

Uh, I don't know how to reply to this, but, I can't, Chrome OS linux uses debian 11, and on the version page it says debian 11 runs 1.17 so i just went with it

Could you copy the whole console output, please, with scrcpy -Vdebug?

INFO: scrcpy 1.17 <https://github.com/Genymobile/scrcpy>
DEBUG: Using server: /usr/share/scrcpy/scrcpy-server
/usr/share/scrcpy/scrcpy-server: 1 file...ushed. 0.7 MB/s (34015 bytes in 0.046s)
DEBUG: Screensaver enabled
[server] INFO: Device: Google rammus (Android 11)
DEBUG: Starting stream thread
DEBUG: Starting controller thread
DEBUG: Starting receiver thread
[server] DEBUG: Using encoder: 'c2.v4l2.avc.encoder'
INFO: Renderer: opengl
INFO: OpenGL version: 3.1 Mesa 21.2.6
INFO: Trilinear filtering enabled
INFO: Initial texture: 1920x1080
DEBUG: User requested to quit
DEBUG: quit...
DEBUG: End of frames
DEBUG: Receiver stopped
[server] DEBUG: Controller stopped
WARN: Killing the server...
DEBUG: Server terminated

Also try with another --video-encoder: https://github.com/Genymobile/scrcpy/blob/master/doc/video.md#codec
Just tells me

scrcpy: unrecognized option '--video-codec=h265'


feat: map GitHub user to ontology

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


The child element is not rendered

My code:
Scaffold(           resizeToAvoidBottomInset: false,           body: FooterLayout(             footer: Container(               child: InputField(controller: null, placeholder: 'Type Here'),             ),             child: Text('PRUEBA'),           ),         ),       );
The result:

The text widget with PRUEBA is not shown
Flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.17.5, on Microsoft Windows [Version 10.0.18363.1016], locale es-AR)
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[√] Android Studio (version 3.6)
[√] VS Code, 64-bit edition (version 1.48.2)
[√] Connected device (2 available)
• No issues found!

Hi! Thanks for using the package and for raising this issue.
From the information you provide, my first guess it's that the InputField you are using is somehow pushing/expanding to fill all the space available to it, leaving no room for the child to be rendered. This is because of the way FooterLayout is built. From the FooterLayout class documentation:
/// A widget that can lay out a child widget above a footer.
///
/// This widget builds a [CustomMultiChildLayout] whose delegate loosely
/// positions a footer widget at the bottom of the available space first,
/// and then positions a child widget in the remaining space with loose
/// constraints.
class FooterLayout extends StatelessWidget {
  ...
}

As you can see, FooterLayout renders the footer first and then places the child in the remaining space. This means that if the footer is trying to take all the available space, the child might not have any room available.
I would suggest you to check if this might be the case in your code, maybe replacing your InputField with a standard TextField, which has a fixed height, just to be sure that the footer is not trying to take all the space available to it.
If the problem still persists, please provide a minimal reproducible example of the bug so I can take a deeper look at it. Thanks!


Enable blank GitHub issues

Oren mentioned on Discord that he couldn't report a general issue/concern, only fill out the bug report template. This change should allow starting from a blank template as an option, while still making the bug report template prominent at the top.

I see.. Biggest reason I went this way is I want people to use discussions more and try to keep the issues to things that we acknowledge as actual issues. Bugs are the obvious one. But sure lets see how this goes.

The alternative would be to add some more templates, I guess, but I feel like it's pretty hard to find the blank template at the bottom after all the other options. So this should still cut down on extraneous issues. Anyway, as you said, we'll see what happens!

Biggest reason I went this way is I want people to use discussions more and try to keep the issues to things that we acknowledge as actual issues.

The user probably could have directed the issue or question into the discussion's forum?

The alternative would be to add some more templates - @edemaine

You could also create other types of forms that will appear when the user clicks "new issue"
For example a feature_request.yml  file

https://github.com/cliffordfajardo/foam/blob/master/.github/ISSUE_TEMPLATE/feature_request.yml

The options would then be displayed as an option (not shown in image below, simply illustrating the idea)

Yes I like this even better.


Fixes Theme not changing on System Dark Mode

This PR addresses the issue: #1411.
As it stands, Get.changeTheme() doesnt seem to work when the system theme is set to Dark Mode. This potentially fixes that issue without caring for current themeMode or the system themeMode.
However there is a fix around this issue:
Setting themeMode to light:
return GetMaterialApp(
      home: Page(),
      themeMode: ThemeMode.light,
    );

Pre-launch Checklist

[] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or @jonataslaw said the PR is test-exempt.
[] All existing and new tests are passing.

there is an enum value that set the theme based on the system :
try it :
themeMode: ThemeMode.system

there is an enum value that set the theme based on the system :
try it :
themeMode: ThemeMode.system

Hmm, if i remember it correctly this didnt work because the internal thememode of get doesnt get changed with this. I'll do more checks and update this later today.

GetMaterialApp(
  title: "Application",
  theme: ThemeData.light().copyWith(primaryColor: Colors.blue),
  darkTheme: ThemeData.dark().copyWith(primaryColor: Colors.blue),
  themeMode: ThemeMode.system,
)

to update primaryColor, use this code:
Get.rootController.theme = ThemeData.light().copyWith(primaryColor: Colors.red);
Get.rootController.darkTheme = ThemeData.dark().copyWith(primaryColor: Colors.red);
Get.forceAppUpdate();

to update theme mode, just use changeThemeMode:
Get.changeThemeMode(ThemeMode.dark);

Get.changeTheme() does not run as expected.
It can be changed like this:
/// /lib/get_navigation/src/extension_navigation.dart
-  void changeTheme(ThemeData theme) {
-    _getxController.setTheme(theme);
-  }
+  void changeLightTheme(ThemeData theme) {
+   _getxController.setLightTheme(theme);
+  }
+  void changeDarkTheme(ThemeData theme) {
+   _getxController.setDarkTheme(theme);
+  }

/// /lib/get_navigation/src/root/root_controller.dart
-  void setTheme(ThemeData value) {
-    if (darkTheme == null) {
-      theme = value;
-    } else {
-      if (value.brightness == Brightness.light) {
-        theme = value;
-      } else {
-        darkTheme = value;
-      }
-    }
-    update();
-  }
+ void setLightTheme(ThemeData value) {
+    if (value.brightness == Brightness.dark) return;
+    theme = value;
+    /// or change theme to a light theme if value is a dark theme
+    /// theme = value.copyWith(brightness: Brightness.light);
+    update();
+ }
+ void setDarkTheme(ThemeData value) {
+    if (value.brightness == Brightness.light) return;
+    darkTheme = value;
+    /// or change theme to a dark theme if value is a light theme
+    /// darkTheme = value.copyWith(brightness: Brightness.dark);
+    update();
+ }

use it like this:
Get.changeLightTheme(ThemeData.light().copyWith(primaryColor: Colors.red));
Get.changeDarkTheme(ThemeData.dark().copyWith(primaryColor: Colors.red));

or change Get.changeTheme like this:
/// /lib/get_navigation/src/root/root_controller.dart
  void setTheme(ThemeData value) {
-    if (darkTheme == null) {
-      theme = value;
-    } else {
      if (value.brightness == Brightness.light) {
        theme = value;
      } else {
        darkTheme = value;
      }
-    }
    update();
  }

use it like this:
/// Need to call twice
Get.changeTheme(ThemeData.light().copyWith(primaryColor: Colors.red));
Get.changeTheme(ThemeData.dark().copyWith(primaryColor: Colors.red));

If current mode is dark, changing theme to a light theme would not be working util device theme mode changed to light.
These would be OK.